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

docs(rust): Add Rust example for tolerance join using join_asof in user guide #18696

Closed
wants to merge 0 commits into from

Conversation

arnabanimesh
Copy link
Contributor

No description provided.

@arnabanimesh arnabanimesh changed the title Add Rust example for tolerance join using join_asof in user guide docs: Add Rust example for tolerance join using join_asof in user guide Sep 11, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation python Related to Python Polars rust Related to Rust Polars and removed title needs formatting labels Sep 11, 2024
@arnabanimesh arnabanimesh changed the title docs: Add Rust example for tolerance join using join_asof in user guide docs(rust): Add Rust example for tolerance join using join_asof in user guide Sep 11, 2024
@@ -218,11 +218,7 @@ If you want to make sure that only quotes within a certain time range are joined
the `tolerance` argument. In this case we want to make sure that the last preceding quote is within 1 minute of the
trade so we set `tolerance = "1m"`.

=== ":fontawesome-brands-python: Python"
Copy link
Member

Choose a reason for hiding this comment

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

I don't understand. Why is this removed?

Copy link
Contributor Author

@arnabanimesh arnabanimesh Sep 14, 2024

Choose a reason for hiding this comment

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

I presume you are talking about why I have removed line no. 221. Python and Rust logos are already part of the auto-generated code_block template. No need for another Python logo since it will be added by the code_block anyways.

Copy link
Member

Choose a reason for hiding this comment

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

Alright, thanks!

@ritchie46
Copy link
Member

Can you do a rebase? Then it's good to go.

@arnabanimesh
Copy link
Contributor Author

Can you do a rebase? Then it's good to go.

Done

@arnabanimesh
Copy link
Contributor Author

Fixed in PR #19127. Closing this PR.

@arnabanimesh arnabanimesh deleted the patch-1 branch October 13, 2024 21:03
@rodrigogiraoserrao
Copy link
Collaborator

Sorry for the overlapping work, @arnabanimesh, but I really needed to include the Rust example in the PR I made.
Thanks for your work 🚀

@arnabanimesh
Copy link
Contributor Author

No problem @rodrigogiraoserrao. It happens. 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants