-
Notifications
You must be signed in to change notification settings - Fork 967
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
Unable to schedule from a specific time and following a specific calendar #597
Comments
@SijmenHuizenga could you please assess feasibility and complexity arising from scheduling
I agree with @dbader sentiment to keep this library lightweight and not turn it into next large unmaintainable project (#219). However, I believe the above could considerably increase expressiveness at little cost. -- A huge fan of this library |
So, the disregard and silence of @SijmenHuizenga for several months, meaning this suggested improvement won't be considered? |
This is one of the prominent use cases for me. I do it via hacky ways for now by refreshing schedule before and after the EDIT: Another issue with the same requirement exists. Track it here. #568 In fact, this issue should be closed in favor of the one I linked. |
@SijmenHuizenga could you please confirm if above requirements (day calendar, time upper bound) significantly increase complexity / decrease maintainability? |
The library supports scheduling until a certain time but not from time. Consequently, it is difficult to implement logic such as every hour on Monday-Friday from 11:30 to 12:30.
We created a custom function that generates a list of options, in this case
This implies 15 calls, as opposed to only 5 calls per each day, or just 1 call per calendar that sets Monday to Friday eligible.
With a more frequent schedule, this becomes even more prohibitive. In such a case, we just run the job at every().minute and exit early when outside of schedule from job function, i.e. manage schedule externally.
Any enhancements planned? Any idea how to enhance internally the library?
The text was updated successfully, but these errors were encountered: