-
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
Refactor TF Dataset #46
Comments
Yep :-) but I'm a bit confused, I thought we had already done all of that. I remember discussions like here: #39 |
Otherwise I want to say no to knots for now, and we will need to keep fairly faint galaxies in order to apply the selection cuts. |
And the only form of augmentation we can make is generating the noise on the fly instead of hardcoded in the dataset |
We did, but the current state of the module is not satisfactory, still.
This is kind of a problem. The COSMOS dataset is small, isn't it? We need more than 80k galaxy models. |
No it's not really a problem, you can apply random rotations to galaxies, and generate independent noise realisations. At least for training a NN it's more than enough |
Ok, that's kind of the data aug I was talking about. |
Yep but this can be done on the fly most likely. |
After talking with @EiffL and @b-remy, we decided to converge on a more usable format for our tfds.
We will have a simple galaxy generator dataset and one based on parametric COSMOS galaxies drawn as CFIS-like stamp images.
This issue is to discuss some design choices and track/discuss this development.
Issues/todo:
gal_type='parametric'
)What do you think about these open questions, @EiffL ?
The text was updated successfully, but these errors were encountered: