Skip to content

Small tweaks to improve Durvo diagram scaling with high-TDS data#18

Open
Benito7349 wants to merge 2 commits into
jyangfsu:mainfrom
Benito7349:durvo
Open

Small tweaks to improve Durvo diagram scaling with high-TDS data#18
Benito7349 wants to merge 2 commits into
jyangfsu:mainfrom
Benito7349:durvo

Conversation

@Benito7349
Copy link
Copy Markdown

Hi there,

I’m a geologist with some programming background, and I’m currently working with water samples from deep wells
(over 1,400 feet deep). These samples have very high TDS concentrations, and I’ve run into scaling issues when
using the Durvo diagram.

🧩 Summary

This pull request introduces a few adjustments to the function used for generating the Durvo diagram.
The main goal is to improve the handling of scaling issues when working with water samples that have very
high TDS concentrations, such as those from deep wells (>1400 ft).

⚙️ Changes

Modified the function to better handle samples with high TDS values.

Improved axis scaling to prevent data overlap.

(Temporary workaround) Exporting to .svg and adjusting elements manually in Inkscape to fix legend overlap.

🐞 Known Issue

The legend still overlaps the diagram in some cases. I haven’t found a robust solution for this yet,
but exporting to .svg and rearranging elements manually in Inkscape works as a workaround.

🙏 Acknowledgments

Many thanks for developing this tool — it’s been extremely helpful in my work.
I hope these small changes can be useful to others facing similar challenges.

@jyangfsu
Copy link
Copy Markdown
Owner

Hi, thank you so much for your contribution and for the clear explanation!
These improvements for handling high-TDS samples are very helpful. I appreciate your effort in identifying the scaling issue and proposing a practical workaround.
I’ll review the changes shortly and will keep you updated.
Thanks again for supporting the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants