-
Notifications
You must be signed in to change notification settings - Fork 79
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
Notebooker Windows Issues #174
Comments
FYI... I think that the windows compatibility started to be looked at in this branch to address slashes issues |
Thanks for the report, we will try to take a look. If you were able to fix locally in the meantime then contributions are very welcome! |
ok... when I get a moment I will submit some fixes |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Love the project and concepts but having a few issues in getting this up and running on windows 11
In window virtual env the executable notebooker-cli is in the script folder so
notebooker/notebooker/execute_notebook.py
Line 522 in d7549c5
should be replaced by
os.path.join(sys.exec_prefix, "Scripts", "notebooker-cli")
Babel needs a locale defined
notebooker/notebooker/utils/results.py
Line 147 in d7549c5
should be replaced by
metadata["time_diff"] = babel.dates.format_timedelta(datetime.datetime.now() - metadata["latest_run"], locale = "en")
open statements should include encoding="utf-8")
notebooker/notebooker/execute_notebook.py
Line 117 in d7549c5
notebooker/notebooker/utils/conversion.py
Line 27 in d7549c5
i.e. open(ipynb_path, "r", encoding="utf-8")
this still appears to be an issue with a blank results page output
Results page is empty in Windows 10 #77 (comment)
this returns an empty array
http://127.0.0.1:11828/core/get_all_available_results?limit=100&report_name=sample/plot_random
but this works... it is to do with slashes
http://127.0.0.1:11828/core/get_all_available_results?limit=100&report_name=sample\plot_random
I couldnt work out where best to fix this... any advice.
The text was updated successfully, but these errors were encountered: