Skip to content

Conversation

@Sosuke23
Copy link

@Sosuke23 Sosuke23 commented Nov 9, 2025

i think it is better to make coding language consistent.


We can find arbitrary range sums by computing the prefix sums for $l-1$ and $r$ and taking the difference of them again.

```python
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we actually keep both Python and C++?

Copy link
Author

@Sosuke23 Sosuke23 Nov 11, 2025

Choose a reason for hiding this comment

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

do you want it for the whole fenwick module or just this one?

Copy link
Contributor

@kostka-dev kostka-dev Nov 22, 2025

Choose a reason for hiding this comment

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

Whole if possible.

Copy link
Author

Choose a reason for hiding this comment

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

done.... can you approve the workflow so i can see the preview?

Copy link
Contributor

Choose a reason for hiding this comment

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

Previews are available on the website website contributing.

Copy link
Author

Choose a reason for hiding this comment

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

preview looks good, can you approve the workflow so i can see if that error is resolved.

Copy link
Author

Choose a reason for hiding this comment

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

haven't made any changes in test fenwick file..... The build is getting failed bcz of that missing .h files

@Sosuke23 Sosuke23 requested a review from kostka-dev November 11, 2025 01:53
try messing with space.
pretty sure spaces are screwing up test file generation.
@mhayter
Copy link
Contributor

mhayter commented Dec 6, 2025

It looks like the extract_snippets.py is not matching indented named C++ files. Maybe we should change that extractor @adamant-pwn ? I've resolved the error in a cursory way by unindenting the named C++ files (perhaps we should uniformly unindent the code blocks?

@mhayter mhayter closed this Dec 6, 2025
github-actions bot added a commit that referenced this pull request Dec 6, 2025
@mhayter mhayter reopened this Dec 6, 2025
@mhayter
Copy link
Contributor

mhayter commented Dec 6, 2025

I screwed up. I need to undo that force push. It seems like stuff finally worked locally but I'm dumb and somehow incorporated incorrect changes (lack of updates).

@mhayter
Copy link
Contributor

mhayter commented Dec 6, 2025

Hmmm... I'm gonna revisit this. I screwed up incorporated old code into a push. It was working locally (tabs on the fenwick article). If you're interested @Sosuke23 , the https://cp-algorithms.com/contrib.html is useful. You can run the website locally and test it locally.

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.

3 participants