-
Notifications
You must be signed in to change notification settings - Fork 16
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
Implement ComplexZmanimCalendar #5
Comments
I have no plans to do this at this time. Every method of the Complex calculations can be performed using the overloaded ZmanimCalendar methods, and I didn't want the responsibility of keeping the code in sync as new opinions are being continuously added to the Complex class. |
Can you give an example of how I would be able to do it with the ZmanimCalendar methods? |
Do you have a specific zman and opinion you'd like an example for, or would any suffice? |
Any |
Here's how 5 (randomly selected) calculations may be implemented:
|
@plonibarploni can I assume this has been resolved? |
Yes, but it would be nice if this was included in a readme or wiki page. Thanks |
@pinnymz I know that this issue is all but resolved, however, I felt like I should say that just because it is easy to create all the methods in the ComplexZmanimCalendar class, it shouldn't mean that you can't create a bunch of convenience methods for your users. It's not like there are all new zmanim opinions popping up everyday. I feel like someone who wants to use your repo, wouldn't want to have to keep a separate python file that implements the proper methods wherever he goes to use Python and this repo. I know it might be a lot of work, but I believe that it would be a very good idea to implement the ComplexZmanimCalendar class to the best of your ability. The result being that someone who wants to use your repo, can just install it and use the bainHasmashosRT13Point5MinutesBefore7Point083Degrees() method right away for example. In the end, it is your repository, so do what you want. However, I felt like I should give the idea a push. Kol Tuv! |
Are there any plans to port over the ComplexZmanimCalendar class from KosherJava? Or is it already implemented elsewhere?
The text was updated successfully, but these errors were encountered: