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

add requirements.txt at the top level for all dependencies #145

Merged
merged 6 commits into from
Jan 3, 2022

Conversation

geek-yang
Copy link
Member

This PR closes the issue in dianna.

A requirements.txt at the top level for all dependencies (necessary for preparing data and executing xai notebooks) is added after checking all the notebooks in the folder "example_data".

@egpbos
Copy link
Member

egpbos commented Dec 21, 2021

I used Binder to test whether the setup works: https://mybinder.org/v2/gh/dianna-ai/dianna-exploration/26-add-dependencies-list

The captum version specification doesn't work, maybe the x should be replaced with a number?

@geek-yang
Copy link
Member Author

Binder

@egpbos Thanks for the review, and the solution. After specifying the version of captum, it passes the installation/check in the Binder.

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