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

possible feature ? BMI on Graph #462

Closed
sinedo opened this issue Jun 14, 2020 · 18 comments · Fixed by #497
Closed

possible feature ? BMI on Graph #462

sinedo opened this issue Jun 14, 2020 · 18 comments · Fixed by #497
Milestone

Comments

@sinedo
Copy link

sinedo commented Jun 14, 2020

Although the BMI is not as good as other measurements it gives you a rough estimate where your weight should be located
It would be cool to have a checkbox which enables the a transparent color of the bmi displayed in the background of the graph

Graph_with_BMI

@rolandgeider
Copy link
Member

Yeah, this sounds useful

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 5, 2020

@rolandgeider I'm new to Github, but I'd like to try this one.

@rolandgeider
Copy link
Member

Hi @Svn-Sp ! Sure, do you have a working development instance? I still haven't uploaded the new Docker images... 🙈 We can coordinate us in the gitter channel, or here if you want

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 7, 2020

Have installed wger for developement, but when I go to the weight overview page I just get the table but no graph...

@rolandgeider
Copy link
Member

Strange, if you have entries there, it should show you something. Does the development server show 404 errors?

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 9, 2020

screen
There are no 404's in the console...

@rolandgeider
Copy link
Member

I'll take a look at master. I've been working on a different branch with mobile templates (which I'll hope I'll be able to merge soon) and there it's definitely working

@rolandgeider
Copy link
Member

I've merged the branch into master and the weight overview is working (other things are ugly, but should still be functional). The CSS/JS libraries are downloaded with yarn now, the easiest thing is probably to use docker-compose, but you can download the dependencies yourself with yarn install

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 12, 2020

Excuse me if this isn't the smartest question, but could you post the complete yarn install command?
I still can't get the graphs to show...

@rolandgeider
Copy link
Member

yarn install is the whole command 😄. You call it from the source folder and it should download everything to wger/core/static/yarn/

image

I really can't think of a reason why it wouldn't show anything

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 14, 2020

Still the same problem; No graph is showing and no 404's in the console.
I think yarn install did not work properly
yarn install
There is still no /yarn folder in wger/core/static !

@rolandgeider
Copy link
Member

it's strange that yarn says there's a lock from another package manager, did you pull the latest stuff from master? What does git log -2 say?

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 15, 2020

Ok, so I don't really know how I did it, but now it runs! I am gonna start working on the feature now.
Thank you for your patience!

@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 15, 2020

How shall I implement this, I mean should I draw the lines in the svg or has d3 a function for this or what should I do?

@rolandgeider
Copy link
Member

Ok, so I don't really know how I did it, but now it runs! I am gonna start working on the feature now.
Thank you for your patience!

😂😂 I feel you. When it suddenly starts (or stops) working and you don't know why...

@rolandgeider
Copy link
Member

For the weight chart we use metricsgraphics, you can look there if it is possible to draw on top of an existing chart. D3 is used in the background, perhaps it's possible to access that?

Svn-Sp pushed a commit to Svn-Sp/wger that referenced this issue Aug 19, 2020
@Svn-Sp
Copy link
Contributor

Svn-Sp commented Aug 19, 2020

Have opened a PR...

rolandgeider added a commit that referenced this issue Sep 20, 2020
Feature/bmi on graph at weight overview (Issue #462)
@rolandgeider rolandgeider added this to the 2.0 milestone Sep 20, 2020
@rolandgeider rolandgeider linked a pull request Sep 20, 2020 that will close this issue
@rolandgeider
Copy link
Member

PR was merged, closing here

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