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

bug: Head Circumference is missing from the vitals display chart due to incorrect unit conversion #6527

Closed
juggernautsei opened this issue Jun 8, 2023 · 1 comment · Fixed by #6556
Milestone

Comments

@juggernautsei
Copy link
Member

Describe the bug

This is what was shared with me after we installed v7.0.1 (1)
We have 3 graph that are always checking by the Dr. Chen

  1.   The height is plotted properly now
    
  2.   The weight red dots need to be fixed to a bigger size like the dots in height graph
    
  3.   **The head circumference is does not plot on the graph ,** this stature graph only seems to show on the old patients ( eli yates 8 yrs old) but not with new borns.
    

To Reproduce

Untitled

Untitled

Untitled

Steps to reproduce the behavior:

  1. Add a '....'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

show readings on the chart.
I have tested this in the demo site and the readings don't show on the chart their either.

Client configuration

  • Browser: All
  • OpenEMR version: 7.0.1 (1)
  • Operating system: Ubuntu 22.04
@stephenwaite
Copy link
Sponsor Member

stephenwaite commented Jun 13, 2023

hi @juggernautsei , pretty likely it's because the birth section of chart.php wasn't refactored for the metric locale.

probably need a function to convert cm to in when $head_circ is defined

@adunsulag adunsulag added this to the 7.0.2 milestone Nov 10, 2023
@adunsulag adunsulag changed the title Head Circumference is missing from the vitals. bug: Head Circumference is missing from the vitals display chart due to incorrect unit conversion Nov 16, 2023
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 a pull request may close this issue.

3 participants