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

sklearn_flatten in dea_tools/classification not working as expected #1260

Open
Kooie-cate opened this issue Aug 7, 2024 · 2 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@Kooie-cate
Copy link
Collaborator

sklean_flattern appears to be dropping pixels.

Expected behaviour
when using this model to flatten an array I expect the resulting 1D array to have the same size and the original array (ie an array where x=200 and y = 400 should return a 1d array with a size of 80000 pixels.)

when I run this with data that has data masked out for cloud or shadow it seems to drop those pixels from the 1D array.

Environment information
DEA Sandbox

@Kooie-cate Kooie-cate added the bug Something isn't working label Aug 7, 2024
@Kooie-cate Kooie-cate self-assigned this Aug 7, 2024
@robbibt
Copy link
Member

robbibt commented Aug 7, 2024

Thanks @Kooie-cate! I think Ness may have run into something similar... would be great to make sure this is working as expected! There may have been some changes to numpy/xarray logic in recent Sandbox updates

@Kooie-cate
Copy link
Collaborator Author

Kooie-cate commented Aug 7, 2024

Oh no! @robbibt I talked to Damien about it just by chance and it seems like the problem is me! or rather it doesn't do what I expect but it does do what is intended, but the documentation doesn't make that clear that this is the intended behavior. Maybe improving the documentation and or adding a flag to turn this on/off is needed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants