[DOC] fix link in doc #2944
Merged
[DOC] fix link in doc #2944
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
documentation
CodSpeed Performance ReportMerging #2944 will not alter performanceComparing Summary
|
ccmao1130
approved these changes
Oct 4, 2024
ccmao1130
requested changes
Oct 4, 2024
| @@ -30,7 +30,7 @@ Daft does not use an index | |||
|
|
|||
| Dask aims for as much feature-parity with pandas as possible, including maintaining the presence of an Index in the DataFrame. But keeping an Index is difficult when moving to a distributed computing environment. Dask doesn’t support row-based positional indexing (with .iloc) because it does not track the length of its partitions. It also does not support pandas MultiIndex. The argument for keeping the Index is that it makes some operations against the sorted index column very fast. In reality, resetting the Index forces a data shuffle and is an expensive operation. | |||
|
|
|||
| Daft drops the need for an Index to make queries more readable and consistent. How you write a query should not change because of the state of an index or a reset_index call. In our opinion, eliminating the index makes things simpler, more explicit, more readable and therefore less error-prone. Daft achieves this by using the [Expressions API](../user_guide/basic_concepts/expressions). | |||
| Daft drops the need for an Index to make queries more readable and consistent. How you write a query should not change because of the state of an index or a reset_index call. In our opinion, eliminating the index makes things simpler, more explicit, more readable and therefore less error-prone. Daft achieves this by using the :doc:`Expressions API <../user_guide/basic_concepts/expressions>`. | |||
There was a problem hiding this comment.
i think its supposed to be redirecting to https://www.getdaft.io/projects/docs/en/stable/api_docs/expressions.html since it's referencing Expressions API so it should be ../api_docs/expressions?
|
@ccmao1130, updated! |
raunakab
approved these changes
Oct 7, 2024
ccmao1130
approved these changes
Oct 7, 2024
|
Thank you @amitschang ! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I noticed a small error in documentation, looks like markdown style link used mistakenly.