Skip to content
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

Tensorflow example #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Tensorflow example #8

wants to merge 1 commit into from

Conversation

wjpearson
Copy link

Optical example for AA use in Tensorflow. Includes a script to load the GalaxyMNIST dataset as a tensorflow dataset.

@mb010
Copy link
Owner

mb010 commented Jul 6, 2022

I think if I merge this right now, then anyone who uses astroaugs will have to have both torch and tf installed. I will try to figure out a way to make that optional and then merge this?

My current plan is to remove any direct reference to data sets, and instead link them or provide them as auxiliary files which can be used, but wont be installed by pip (in the same way as the notebooks aren't installed).

I'm open to suggestions though.

Thanks for setting this up @wjpearson ! 🎉

@wjpearson
Copy link
Author

I checked in a clean environment. I only installed: numpy, astropy, h5py, torch, torchvision (plus any dependencies they install to work). When importing AA it does not complain tensorflow is missing via the following:

>>> import sys
>>> sys.path.append('/home/pearsonw/dev/deep_learning/AstroAugmentations')
>>> import astroaugmentations as AA

So I don't think it requires tensorflow to use it.

But please double check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants