Skip to content
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

Add support for BAPS OJ #387

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Add support for BAPS OJ #387

merged 2 commits into from
Oct 19, 2023

Conversation

touhidurrr
Copy link
Contributor

resolves #385

@jmerle jmerle merged commit b300fe1 into jmerle:master Oct 19, 2023
1 check passed
@jmerle
Copy link
Owner

jmerle commented Oct 19, 2023

Thank you! I can't test it because I have no BAPS OJ account and there's no public registration page, but I don't see any problems in this PR. Will be released soon.

@touhidurrr touhidurrr deleted the baps branch October 19, 2023 18:05
@touhidurrr
Copy link
Contributor Author

touhidurrr commented Oct 21, 2023

I can't test it because I have no BAPS OJ account and there's no public registration page

Yes, BAPS OJ is a private judge that was made to run ICPC Dhaka Regional 2023 (and possibly future ICPC Dhaka Regional Contests) replacing CodeMarshal (although no official announcement about it was made about it but i kind of guessed it since they have taken CM down for now). The judge offers no registration page as username / password is made for participating teams by the OJ maintainers emailed before the contest for contests like this. Since I was also a participant this time, I could test it and made a PR.

Although I cannot share my team credentials in public, since the API is weirdly open for the judge you should be able to test it by making API requests.

Fun fact is the issue was actually made some time before the preliminary contest began and i responded to it because i was bored as the contest was delayed. I also wrote some test code that worked (again a little while before the contest) but didn’t upload it because it did not look stable. Later implemented the API to make a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not working with BAPS Oj
2 participants