-
Notifications
You must be signed in to change notification settings - Fork 32
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
mybinder from zenodo and some notebook issues #75
Comments
Thanks a lot @vuillaut. By the way, checking the notebooks in the binder I forgot to make a new release with the newest documentation (and newest notebooks). I will do it next week. The issue you reported with Thank you! |
I would not pretend to tell you how to manage your project :)
No problem |
Hi @vuillaut, I re-organised the material in the sub-directories and the installation dependencies to be sure all the notebooks can run on binder. For the rest, I added a README.md in So I think all the problems you reported have been fixed :) and I can close this. |
Hi
First point is not an issue.
I just wanted to mention that I tried to run the published version of agnpy (on zenodo) in mybinder and that it worked fine. I thought you might be interested :)
https://mybinder.org/v2/zenodo/10.5281/zenodo.4311271
A couple of things that did not work:
sys.path.append("/home/jsitarek/zdalne/agnpy/agnpy/")
innotebooks/experiments/basic/SSDisk.ipynb
- you probably don't want to keep this pathfrom agnpy.synchrotron import R, epsilon_equivalency
innotebooks/experiments/basic/trapz_loglog_test.ipynb
does not work../../data/sampled_taus/tau_dt_figure_14_finke_2016.txt
innotebooks/experiments/basic/check_tau_dt.ipynb
should be replaced by../../agnpy/data/[...]
Quantities
inSynchrotron.evaluate_sed_flux([....]
innotebooks/experiments/basic/synchro_fit_trial.ipynb
Cheers,
The text was updated successfully, but these errors were encountered: