-
Notifications
You must be signed in to change notification settings - Fork 1
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
Default Term edge cases #33
Comments
We experienced this issue today, since Fall term has started but it is August 31st. The default term fails because there is no eecs485su22. We could adjust the behavior to be "current or latest", such that today (8/31) |
This might make sense |
This is annoying! I think we should do one or both of the following:
|
Quick fix but if the next term autograder gets added before the current term is over this breaks. Better solution could be just making the "boundaries" of a term correspond to a day in a month instead of just a month. You don't need to sync with UM if you just hardcode reasonable dates e.g. August 25 or something as start of fall. |
I like it! I think this is where the code change would need to happen. https://github.com/eecs485staff/agio-cli/blob/develop/agiocli/utils.py#L97 |
As mentioned in #32, the support for empty, "cur", and "current" semester designations that defaults to the current term ignores an edge case when the Fall term starts at the end of August.
The text was updated successfully, but these errors were encountered: