aeon icon indicating copy to clipboard operation
aeon copied to clipboard

[MNT] `binder` install option does not install optional dependencies used in notebooks

Open MatthewMiddlehurst opened this issue 2 years ago • 17 comments

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 avatar Apr 19 '23 15:04 MatthewMiddlehurst

@MatthewMiddlehurst hey i am interested in solving this issue, if its not yet assigned or worked upon

RohitP2005 avatar Dec 17 '24 05:12 RohitP2005

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.

baraline avatar Dec 18 '24 21:12 baraline

Okay @baraline thanks for the help! Can u also tell me how to assign myself using the bot

RohitP2005 avatar Dec 19 '24 05:12 RohitP2005

like this : @aeon-actions-bot assign @RohitP2005

baraline avatar Dec 19 '24 08:12 baraline

Thanks @baraline

RohitP2005 avatar Dec 19 '24 08:12 RohitP2005

Is this still being worked on?

MatthewMiddlehurst avatar Jan 09 '25 12:01 MatthewMiddlehurst

Yes , i will probably finish it within two days, sorry for the delay

RohitP2005 avatar Jan 09 '25 17:01 RohitP2005

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

RohitP2005 avatar Jan 11 '25 03:01 RohitP2005

Seems the site is currently down.

MatthewMiddlehurst avatar Jan 11 '25 14:01 MatthewMiddlehurst

I guess, I will wait for the site to be back up running

RohitP2005 avatar Jan 11 '25 17:01 RohitP2005

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.

RohitP2005 avatar Jan 16 '25 20:01 RohitP2005

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.

MatthewMiddlehurst avatar Jan 20 '25 01:01 MatthewMiddlehurst

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?

RohitP2005 avatar Feb 03 '25 18:02 RohitP2005

That should not be the case. I just checked tsfresh.ipynb and got the following as expected.

Image

MatthewMiddlehurst avatar Feb 11 '25 00:02 MatthewMiddlehurst

That should not be the case. I just checked tsfresh.ipynb and got the following as expected.

Image

I see, I will look into it once again and update you

RohitP2005 avatar Feb 13 '25 08:02 RohitP2005

Is this still in-progress?

MatthewMiddlehurst avatar Mar 21 '25 22:03 MatthewMiddlehurst

I find it difficult with the binder enviroment. I think you can assign this to someone else Sorry about it

RohitP2005 avatar Mar 22 '25 17:03 RohitP2005