-
Notifications
You must be signed in to change notification settings - Fork 37
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
Trouble exporting notebooks with otter-grader: Notebook is empty #4477
Comments
@NSagan271 Seems like issue was reported in Otter grader repo - ucbds-infra/otter-grader#444 |
@balajialg Could increased CPU resources help? If so, I can make a Github issue for that. I'm asking because I haven't encountered this issue at all locally, but most/all of the students are having issues exporting their submissions. |
Tagging @shaneknapp @ryanlovett if they have input on whether increasing CPU would have any effect. EECS hub has a CPU limit of 2 and a guarantee of 0.1. Should we increase the limit and/or guarantee further to test whether it reduces the save time of notebooks? @NSagan271 Can you take a look at this code snippet used by Data 8 when a similar error happened to their students - #2107 (comment) |
BTW might be helpful: it seems to be happening because the lab involves processing a video file. The lab deadline has passed, but we had students do the following and it seemed to work:
Since the lab deadline has passed, no changes need to be made at the moment (as long as our next lab doesn't have the same issue). |
@NSagan271 That's an interesting hack. Thanks for sharing it. I will add it to the FAQ documentation I will close the issue for now but feel free to reopen if this issue persists during the next lab. Thanks! |
@NSagan271 @balajialg it looks like the EECS hub is using Otter v3.1.4. Otter's force-save functionality was updated to be a bit more robust in ucbds-infra/otter-grader#476, which was included in v3.3.0. Is it possible to upgrade the version of Otter being used on this hub? |
@chrispyles Thanks for the info! Yes - that should not be a problem. Is the version upgrade backward compatible or will that result in instructors rewriting their existing test cases? |
I don’t think there are any breaking changes |
Bug description
Hi! Some EE 120 students are experiencing trouble exporting their notebooks with otter-grader; they're getting "AssertionError: Notebook lab4.ipynb is empty. Please save and checkpoint your notebook and rerun this cell" even after saving and checkpointing, restarting the page, restarting their server, etc.
Here's the lab in question, it's one of our more compute-heavy ones.
According to one student, it worked after "some combination of server restarts, kernel restarts, and refreshing the page."
It could be a problem on our end, or maybe the students are having network connectivity issues, but I'm bringing it up here in case it's something that you all can help with.
Environment & setup
How to reproduce
It definitely doesn't happen all the time, so I'm not sure how easy it is to reproduce this behavior.
The text was updated successfully, but these errors were encountered: