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

Histogram process #41

Open
jdries opened this issue Feb 27, 2019 · 7 comments
Open

Histogram process #41

jdries opened this issue Feb 27, 2019 · 7 comments
Labels
help wanted Extra attention is needed new process
Milestone

Comments

@jdries
Copy link
Contributor

jdries commented Feb 27, 2019

In some of our zonal statistics use cases, we compute histograms. I believe this is basically a type of reducer.
When the input data cube uses a type with a limited range, like byte, we compute a histogram with 256 buckets, and keep track of nodata. For float/double data types, we might need user definded buckets.

@m-mohr
Copy link
Member

m-mohr commented Feb 27, 2019

Could you come up with a concrete JSON-based proposal? If it's required for the four core use cases it should be in the 0.4 release.

@jdries
Copy link
Contributor Author

jdries commented Feb 27, 2019

I don't think it needs to be in 0.4!

@m-mohr m-mohr added this to the v0.5 milestone Feb 27, 2019
@m-mohr
Copy link
Member

m-mohr commented Nov 17, 2019

@jdries Could you come up with a concrete JSON-based proposal for the next version?

@m-mohr m-mohr added the help wanted Extra attention is needed label Nov 17, 2019
@m-mohr
Copy link
Member

m-mohr commented Nov 26, 2019

Telco: Agreed to add. Not sure whether it's a reducer though, depends on the outcomes of #73, I guess. If there's a volunteer to help define this function, I'd appreciate it.

@bossie
Copy link

bossie commented Nov 29, 2019

Histogram as a reducer process with discrete buckets might look like this: #97. No user-defined buckets are supported yet.

@m-mohr m-mohr added work in progress and removed help wanted Extra attention is needed labels Dec 13, 2019
@m-mohr m-mohr added the has PR label Jan 15, 2020
@m-mohr m-mohr modified the milestones: v1.0-rc1, v1.0 Jan 20, 2020
@m-mohr m-mohr added help wanted Extra attention is needed and removed accepted has PR work in progress labels May 28, 2020
@m-mohr m-mohr modified the milestones: v1.0-final, future May 28, 2020
@m-mohr m-mohr added the minor label Aug 4, 2020
@m-mohr m-mohr modified the milestones: future, 1.1.0 Apr 28, 2021
@m-mohr
Copy link
Member

m-mohr commented Apr 28, 2021

Seems to be relevant for UC3 (Platform), so giving it some priority. @jdries

@m-mohr m-mohr self-assigned this Apr 28, 2021
@m-mohr m-mohr modified the milestones: 1.1.0, 1.2.0 May 18, 2021
@m-mohr m-mohr modified the milestones: 1.2.0, 1.3.0 Oct 25, 2021
@m-mohr m-mohr changed the title Histogram reducer Histogram process Dec 15, 2021
@m-mohr
Copy link
Member

m-mohr commented Dec 15, 2021

Would this really go through reduce_dimension? I think it might need to go through apply_dimension (with a target_dimension?) instead as the histogram returns more than a single value, right?

@m-mohr m-mohr linked a pull request Dec 15, 2021 that will close this issue
@m-mohr m-mohr removed their assignment Oct 12, 2022
@m-mohr m-mohr removed the platform label Oct 12, 2022
@m-mohr m-mohr modified the milestones: 1.3.0, future Oct 12, 2022
@m-mohr m-mohr removed the minor label Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed new process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants