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

[Enhancement]: clamp (ceil) graph values to first unit #1060

Closed
dicktyr opened this issue Feb 12, 2021 · 6 comments · Fixed by #2000
Closed

[Enhancement]: clamp (ceil) graph values to first unit #1060

dicktyr opened this issue Feb 12, 2021 · 6 comments · Fixed by #2000
Labels
enhancement Issue that suggests an enhancement good first issue Great issues for first-time contributors help wanted Issue with which contributors need help

Comments

@dicktyr
Copy link

dicktyr commented Feb 12, 2021

please consider implementing an option to force rounding up of minimal values
which otherwise fall below the threshold of visibility in a given graph
i.e. values between zero and the first visible unit

for example
network graphs can be troublesome as their values span such a large range
causing small values to go by without being displayed
(logarithmic scaling is not satisfactory for some cases)

this feature is especially useful for small graphs
such as those in a menu bar or status panel/strip
to be sure that even small values will be displayed

in consideration of high resolution displays
perhaps the option should specify a number of pixels
(as a single pixel may still be nearly invisible)

e.g. a option value of 4 would cause any graph value between zero
and the normal threshold to paint the first 4 pixels
(default option value: 1 pixel)

@su8 su8 added enhancement Issue that suggests an enhancement good first issue Great issues for first-time contributors help wanted Issue with which contributors need help question Issue where reporter seeks information labels Feb 22, 2021
@github-actions
Copy link

This issue is stale because it has been open 365 days with no activity. Remove stale label or comment, or this issue will be closed in 30 days.

@github-actions github-actions bot added the Stale Issue/PR that requires attention because it hasn't been updated in over a year label Dec 11, 2022
@dicktyr
Copy link
Author

dicktyr commented Dec 13, 2022

still relevant

@github-actions github-actions bot removed the Stale Issue/PR that requires attention because it hasn't been updated in over a year label Dec 14, 2022
@prrajput1199
Copy link

can I work on this issue?

@harshit110927
Copy link

I want to work on this issue.

@Caellian
Copy link
Collaborator

@prrajput1199 and @harshit110927, are you still interested in implementing this feature?

@Caellian Caellian removed the question Issue where reporter seeks information label Apr 11, 2024
@Caellian Caellian changed the title graph improvement: force round up to first unit [Enhancement]: clamp (ceil) graph values to first unit Apr 11, 2024
@Caellian
Copy link
Collaborator

This should be pretty straightforward to implement. Graph drawing code is in conky.cc:1282.

Be sure to document the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue that suggests an enhancement good first issue Great issues for first-time contributors help wanted Issue with which contributors need help
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants