-
Notifications
You must be signed in to change notification settings - Fork 78
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
Potentially upstreaming some site changes to DMOJ #381
Comments
Hello there, sorry for the late reply, I haven't checked issues in a while, A few years ago, we were interested in pushing our changes to DMOJ. However, the more we update our site, the more we diverge from DMOJ's, making it quite difficult to push changes upstream. Because we need to run the DMOJ codebase and test it. We couldn't use the same database because our database is different from DMOJ quite a bit. Besides, some of the modifications we've made were implemented in a hacky way (such as the frozen scoreboard), merging that into DMOJ is not a good idea. Moreover, some of the features listed were actually fixes for bugs specific to our system. For instance, the Fix submit button when using back button, this problem arose because we prevented the button from being clicked twice. Another example is the On a final note, we are quite understaffed :kek:, you can see there aren't a lot of new features got added to our site recently. The effort to make PR to DMOJ would increase our workload quite a bit. On the other hand, what is your thought on cherry-picking our commits yourself instead of us making the PR? |
As much as i want to push the frozen ranking to DMOJ, i have no idea how to make its implementation match DMOJ. You can take a quick look here: #212 I can share how i implement it and you can re-implement it on your end |
Hi!
I'm one of the authors of DMOJ, love what you've done with VNOJ!
Over the years, you've added many features and fixed many bugs on top of the DMOJ branch, great job on that. Some of them are features we have open tickets for upstream, and some are bugs we also have. (Whoops.)
What are your thoughts on upstreaming some of your changes?
Hopefully, that should also reduce your maintenance burden in the long run, with merge conflicts becoming less likely. We'd of course greatly appreciate this too--most of our effort nowadays goes into the administrative parts of running https://dmoj.ca and making sure the judge chugs along, and less on feature work.
Having skimmed through the changelog, I've listed some changes that stood out to me as "wow I wish we had this upstream" below:
And some that sound relevant, but I wasn't sure applied to upstream:
As well as anything else you think I missed here.
Let me know what you think, I'm happy to chat more.
The text was updated successfully, but these errors were encountered: