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
Computation rewriting #463
Comments
Fully agreed. Aside from the obvious "just go to a webpage instead of downloading yet another tool", their damage calculations currently sweep the offline planner out the door. A rewrite that focuses both on maintainability and calculation efficiency (plus integration with the GA) could put us back into competition. I'm still quite unsure about the effort of keeping things up to date wrt new uniques and such, but I guess that's what open source is for. I'll have some spare time to devote in the coming weeks, so if you think I can help you in any way, do let me know! |
My goal is that adding a mod for a new unique would only be one line of code in the data part that handles conditions, values, affected stats and the likes (which obviously only works if the mod is not a completely new mechanic). That would be not much effort and also easy for others to contribute if the syntax/functions used are easy to understand.
I didn't do anything more on the topic yet than writing this issue and collecting points on my todo list over time, but I'm sure another helping hand would be great! |
Took a while, but I'm done with everything I wanted to do before this and I should have at least some time now (and more time in a few weeks) to spend on this. So if you still got time and want to help me, that'd be great! |
I won't have a terrible amount of free time in the near future, but I'm totally down for working on this. What do you suggest for coordination? Github tickets in your repo (so we don't spam this one), meeting on Skype again or something entirely different? |
I suggest GitHub issues here for the bigger things, so others can also easily join the discussion, and Skype (or Discord) for coordination |
We could create a poeskilltree discord. It would give everyone a way to
talk with each other. We can also create support channels and what not
|
Sounds like an excellent idea! |
I'm also totally up for that! And welcome as Collaborator, @MLanghof! |
Thanks! (Also thanks @EmmittJ :D) |
:) no problem
|
I have created a Discord server. I have no idea about Discord server administration, but I guess I can just invite you, give you enough permissions and we can figure it out. |
I've edited the issue with a roadmap on what I'm planning to do so that there's an overview for my plans and the current state somewhere:
|
Our current calculation code is heavily outdated and hard to maintain, so I plan to completely rewrite what Compute.cs does from scratch and at least do major refactoring to ItemDB.cs and other calculation related stuff.
Not sure when I'll get to start with it, but might as well open an issue for discussion now.
Some thoughts:
Path of Building is obviously a big source of both motivation and inspiration, as it makes our current calculation stuff completely obsolete. (as a side note, from what I've seen from its code, it doesn't look to good on the maintainability side either)
With the parsing now having an open Pull Request (#500), here is what I'd say makes sense as milestones:
Full UI integration doesn't necessarily need to be done in one part, some parts might make sense before jewel support and some might be less important than Tree Generation integration. Jewel support is not directly related and still the easiest thing for someone else to pick up from these, but I do think it is required for the program to be called a full build calculator.
I'll open a new issue for discussion and status updates about the next milestone once I'm starting working on it. I'll try to divide that into multiple PRs and comment about the current state more often.
The text was updated successfully, but these errors were encountered: