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.

Sign up for GitHub

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

Jump to bottom

Add aliens optimization draft #70

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add aliens optimization draft #70

wants to merge 5 commits into from

Conversation

llshang
Copy link

@llshang llshang commented May 14, 2023 •
edited
Loading

  • Read Contribute to NTHU-CPP.
  • Rebase to the latest main branch.
  • List all the references.
  • Successfully build the website by mdBook with no error after the change.
  • Exclude any irrelevant files.
  • Link to the issue which is related to your change.
  • Review the content by myself.
  • Pass tests/CI.

@llshang llshang changed the title Add aliens optimization draft Draft: Add aliens optimization draft May 14, 2023
@llshang llshang changed the title Draft: Add aliens optimization draft Add aliens optimization draft May 14, 2023
@llshang llshang marked this pull request as draft May 14, 2023 12:35
@harry900831 harry900831 linked an issue May 18, 2023 that may be closed by this pull request
Add Aliens Optimization #44
Open
@llshang llshang marked this pull request as ready for review June 22, 2023 11:34
@llshang
Copy link
Author

llshang commented Jun 22, 2023

@harry900831

I have completed my task, please review.
Sorry for the delay in requesting the review.

harry900831
harry900831 requested changes Jul 10, 2023
View reviewed changes
Copy link
Contributor

@harry900831 harry900831 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please try to reorganize the whole article and think about how to make your explanation clear.
  • Please re-request review after your revision. Thanks.

src/SUMMARY.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
src/DP/aliens.md Outdated Show resolved Hide resolved
harry900831
harry900831 requested changes Jul 10, 2023
View reviewed changes
Copy link
Contributor

@harry900831 harry900831 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please re-request review after your revision. Thanks.

@llshang
Copy link
Author

llshang commented Jul 11, 2023 •
edited
Loading

@harry900831 感謝 Review，我會在修改好後，到時再麻煩你再次確認。

@harry900831
Copy link
Contributor

harry900831 commented Jul 11, 2023 •
edited
Loading

順便提一下，事實上這個技巧在各大論壇也有不少人討論，例如：https://codeforces.com/blog/entry/49691

你可以多看一些 codeforces 上的討論。

@harry900831
Copy link
Contributor

friendly ping

@llshang llshang closed this Aug 25, 2023
@harry900831
Copy link
Contributor

why are you closing this?

@llshang
Copy link
Author

llshang commented Aug 25, 2023

why are you closing this?

I accidentally closed this while resolving conflicts. I'll reopen it after completing the modifications in the next few days. Thanks.

@llshang llshang reopened this Sep 5, 2023
@llshang llshang closed this Sep 5, 2023
@llshang llshang reopened this Sep 5, 2023
@llshang
Copy link
Author

llshang commented Sep 6, 2023

@harry900831
I have made the modifications. Thanks for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harry900831 harry900831 harry900831 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Aliens Optimization
2 participants
@llshang @harry900831