-
Notifications
You must be signed in to change notification settings - Fork 541
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
Wavelet Tree Module #4534
Wavelet Tree Module #4534
Conversation
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks so much!
content/6_Advanced/Wavelet.mdx
Outdated
Let's define: | ||
c[i] = as 1 if index[i] is smaller than mid otherwise 0 | ||
prefixB[i] as prefix sum of c[i] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you check how the formatting looks on markdown lol
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you also use language sections around code blocks per usaco.guide standards?
this should also fix your build
for more information, see https://pre-commit.ci
Changed time complexity Added braces
Changed the uniqueID
for more information, see https://pre-commit.ci
content/6_Advanced/Wavelet.mdx
Outdated
However, let's explore a different approach that can also be extended to the | ||
second type of query. | ||
|
||
### A different way |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you don't need to split up the module into this many headers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would using ### instead of ## also do the job? or would you prefer to erase some of the headers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it's best if you erase some of the headers and just split up your ideas into paragraphs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unaddressed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have erased some of the headers. Would you also like me to erase the first type of query and the second type of query header as well?
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
This pull request has been automatically marked as stale because it has not had recent activity. Please address the requested changes and re-request reviews. Thank you for your contribution! |
This pull request has been automatically marked as stale because it has not had recent activity. Please address the requested changes and re-request reviews. Thank you for your contribution! |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool diagram
"isStarred": false, | ||
"tags": ["Wavelet"], | ||
"solutionMetadata": { | ||
"kind": "none" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use auto-gen from site (cf)
https://usaco.guide/general/working-mdx#problem-lists
Co-authored-by: Ryan Chou <81596991+ryanchou-dev@users.noreply.github.com>
Co-authored-by: Ryan Chou <81596991+ryanchou-dev@users.noreply.github.com>
for more information, see https://pre-commit.ci
This pull request has been automatically marked as stale because it has not had recent activity. Please address the requested changes and re-request reviews. Thank you for your contribution! |
Changes requested have not been made. Free free to create a new PR. |
Place an "x" in the corresponding checkbox if it is done or does not apply to this pull request.