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
model factory refactor #146
Conversation
ml4cvd/tensor_generators.py
Outdated
sample_weights = [None] * len(out_batch) | ||
return (in_batch, out_batch, sample_weights, paths) if return_paths else (in_batch, out_batch, sample_weights) |
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.
This is the same as master, not sure why it's presented as a change
ml4cvd/tensor_generators.py
Outdated
sample_weights = [in_batch.pop(sample_weight.input_name()).flatten()] * len(out_batch) | ||
return (in_batch, out_batch, sample_weights, paths) if return_paths else (in_batch, out_batch, sample_weights) |
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.
This is the same as master, not sure why it's presented as a change
9870821
to
7b61c75
Compare
401a5a1
to
96b4055
Compare
…nstitute/ml into nd_model_factory_refactor
…nstitute/ml into nd_model_factory_refactor
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.
Still a few cases where master performs marginally better, but I think we are close enough!
After running tests pushing new dockers for CPU & GPU and pre-commit hooking I think this is good to merge. |
Lots of changes to model factory! List here https://docs.google.com/document/d/15sZsTnDVOiBPlW16FX36IzEx-FS7p0YxQ7alY_-npjE/edit?usp=sharing
Lots of changes to model factory! List here https://docs.google.com/document/d/15sZsTnDVOiBPlW16FX36IzEx-FS7p0YxQ7alY_-npjE/edit?usp=sharing
Lots of changes to model factory! List here https://docs.google.com/document/d/15sZsTnDVOiBPlW16FX36IzEx-FS7p0YxQ7alY_-npjE/edit?usp=sharing
Lots of changes to model factory! List here https://docs.google.com/document/d/15sZsTnDVOiBPlW16FX36IzEx-FS7p0YxQ7alY_-npjE/edit?usp=sharing
Lots of changes to model factory! List here https://docs.google.com/document/d/15sZsTnDVOiBPlW16FX36IzEx-FS7p0YxQ7alY_-npjE/edit?usp=sharing
make_variational...
, which will soon be deprecatedHere is a model with
u_connect
, multiple inputs, multiple outputs, and parented outputs