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

Inconsistencies in dataframe due to chained_indexing #24

Closed
AyanSinhaMahapatra opened this issue Nov 10, 2020 · 0 comments
Closed

Inconsistencies in dataframe due to chained_indexing #24

AyanSinhaMahapatra opened this issue Nov 10, 2020 · 0 comments

Comments

@AyanSinhaMahapatra
Copy link
Member

Docs : https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#returning-a-view-versus-a-copy

Group Location wasn't being added to the dataframe properly because of chained_indexing in the DataFrames (which handles different sizes of data differently even with the same code).
AyanSinhaMahapatra added a commit to AyanSinhaMahapatra/scancode-analyzer that referenced this issue Nov 10, 2020
@AyanSinhaMahapatra
Fixes issue with chained_indexing
f0e07de 
Fixes aboutcode-org#24 by
changing functions in `divide_cases`. Also updates test related data
into folders.

Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
pombredanne pushed a commit that referenced this issue Nov 10, 2020
@AyanSinhaMahapatra @pombredanne
Fixes issue with chained_indexing
fda3c56 
Fixes #24 by
changing functions in `divide_cases`. Also updates test related data
into folders.

Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
AyanSinhaMahapatra added a commit to AyanSinhaMahapatra/scancode-analyzer that referenced this issue Nov 10, 2020
@AyanSinhaMahapatra
Do not use chained_indexing in pandas aboutcode-org#24
acc30aa 
Using chained indexing in pandas is a source of problems. The pandas
documentation provides some details at:
https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#returning-a-view-versus-a-copy
This commit fixes this by changing functions in
results_analyze/divide_cases.py.
Also updates test related data into folders.

Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
@AyanSinhaMahapatra AyanSinhaMahapatra linked a pull request Nov 24, 2020 that will close this issue
Adds tests for divide_cases module with pytest #28
Closed
AyanSinhaMahapatra added a commit to AyanSinhaMahapatra/scancode-analyzer that referenced this issue Dec 10, 2020
@AyanSinhaMahapatra
Do not use chained_indexing in pandas aboutcode-org#24
f80b0a3 
Using chained indexing in pandas is a source of problems. The pandas
documentation provides some details at:
https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#returning-a-view-versus-a-copy
This commit fixes this by changing functions in
results_analyze/divide_cases.py.
Also updates test related data into folders.

Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
AyanSinhaMahapatra pushed a commit that referenced this issue May 5, 2022
@pombredanne
Merge pull request #24 from nexB/use-venv-embeds
1c84e9e 
Use virtualenv-embedded libraries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds tests for divide_cases module with pytest AyanSinhaMahapatra/scancode-analyzer
1 participant
@AyanSinhaMahapatra