-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updating data.py for simplicity and z-mass and running 3particle update #2
Changes from all commits
fe18562
25d3c1e
6a37a48
8bf27d7
12e40ca
ff50da3
9770f31
8452c02
e6957f6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,3 +2,5 @@ | |
*.DS_Store | ||
**/.ipynb_checkpoints | ||
*.out | ||
batch**.sh | ||
*.o |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,12 @@ | ||
data_path: "/glade/p/cisl/aiml/ai4ess_hackathon/holodec/" | ||
out_path: "/glade/p/cisl/aiml/ggantos/holodec/conv2d_1particle/" | ||
path_data: "/glade/p/cisl/aiml/ai4ess_hackathon/holodec/" | ||
path_save: "/glade/p/cisl/aiml/ggantos/holodec/xyz3/base1/" | ||
model_name: "cnn" | ||
num_particles: 1 | ||
random_seed: 328942 | ||
output_cols: ["x", "y", "z", "d"] | ||
input_scaler: "MinMaxScaler" | ||
scaler_out: "MinMaxScaler" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does scaler_out handle the scaling of both the inputs and outputs or just the outputs? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Based on further review of the code, it looks like it handles the outputs, which is fine. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. sorry, yes, just outputs. I renamed scaler_vals to scaler_in to mimic the naming convention in my latest commit. |
||
num_z_bins: False | ||
subset: False | ||
metric: "mae" | ||
conv2d_network: | ||
filters: [8, 16, 32] | ||
|
@@ -16,7 +18,6 @@ conv2d_network: | |
lr: 0.001 | ||
optimizer: "adam" | ||
loss: "mae" | ||
batch_size: 32 | ||
batch_size: 256 | ||
epochs: 20 | ||
verbose: 1 | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
path_data: "/glade/p/cisl/aiml/ai4ess_hackathon/holodec/" | ||
path_save: "/glade/p/cisl/aiml/ggantos/holodec/z3_flatten/base/" | ||
model_name: "cnn" | ||
num_particles: 3 | ||
random_seed: 328942 | ||
output_cols: ["z","hid"] | ||
scaler_out: "MinMaxScaler" | ||
num_z_bins: False | ||
flatten_coord: True | ||
subset: False | ||
metric: "mae" | ||
conv2d_network: | ||
filters: [8, 16, 32] | ||
kernel_sizes: [5, 5, 5] | ||
conv2d_activation: "relu" | ||
pool_sizes: [4, 4, 4] | ||
dense_sizes: [64, 32] | ||
dense_activation: "relu" | ||
lr: 0.001 | ||
optimizer: "adam" | ||
loss: "mae" | ||
batch_size: 256 | ||
epochs: 20 | ||
verbose: 1 |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
#!/bin/bash | ||
wget ftp://ftp.ucar.edu/pub/mmm/bansemer/holodec/synthetic_holograms_v01.nc | ||
wget ftp://ftp.ucar.edu/pub/mmm/bansemer/holodec/project_datasets/synthetic_holograms_v01.nc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for the batch**.sh scripts to be ignored by git? Are they autogenerated by another program?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not autogenerated but I figured since they were native to our specific casper environment, we don't necessarily want them on the git repo. What do you think?