aeon
aeon copied to clipboard
[MNT] `binder` install option does not install optional dependencies used in notebooks
The binder install option should install these so that all notebooks can be run. I'm not even sure if all notebooks run in their current state, so this would require a bit of a review.
@MatthewMiddlehurst hey i am interested in solving this issue, if its not yet assigned or worked upon
Hey @RohitP2005, I don't think anyone is working on this currently, as this is an old issue, we should first check its status. A first step would be to try to run the notebooks on binders to see if you get any failures due to missing optional dependencies.
Then these dependencies should be added in the binder install options until no more errors occurs. You should be able to use the bot to assign yourself to the issue.
Okay @baraline thanks for the help! Can u also tell me how to assign myself using the bot
like this : @aeon-actions-bot assign @RohitP2005
Thanks @baraline
Is this still being worked on?
Yes , i will probably finish it within two days, sorry for the delay
The last time I tried, the build was successfully , but didn't run the notebooks. It seems that the binder site is not working currently. Can u guide me forwards
Seems the site is currently down.
I guess, I will wait for the site to be back up running
Hey @MatthewMiddlehurst, if you don't mind, could you help by building the Binder and sharing the URL? My builds are currently failing. Once it's up, I'll check the notebooks and update the Binder dependencies myself.
I don't think it works like that unfortunately. but I'm also not so familiar 🙂. I just tried the link and It works for me, just takes a bit if it the first time an image is created for the commit.
hey @MatthewMiddlehurst i tried runnig those notebooks and most of them ran with no issue, but several notebooks ran into this erro
OMP: Info #276: omp_set_nested routine deprecated, please use omp_set_max_active_levels instead.
am i missing smth here?
That should not be the case. I just checked tsfresh.ipynb and got the following as expected.
That should not be the case. I just checked
tsfresh.ipynband got the following as expected.
I see, I will look into it once again and update you
Is this still in-progress?
I find it difficult with the binder enviroment. I think you can assign this to someone else Sorry about it