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

Data not being plotted/fitted correctly? #10

Open
robineappen opened this issue Jul 21, 2020 · 5 comments
Open

Data not being plotted/fitted correctly? #10

robineappen opened this issue Jul 21, 2020 · 5 comments
Labels

Comments

@robineappen
Copy link

Hi,

The code doesn't seem to produce the right result or maybe I'm looking at it the wrong way. I have attached a plot of the result.

chk1

The data seems to plot an oblate spheroid but the fit model (the orange plot) shows a prolate spheroid. Why is that? Is it because it is reading the wrong coordinates maybe?

coordextract1_01259.txt

I have attached the coordinate text file as well. Please let me know if you get the same result or not.

Thank you
Robin

@aleksandrbazhin
Copy link
Owner

aleksandrbazhin commented Jul 22, 2020

The red one is actually a sphere - two days ago i fixed incompatibility with newer matplotlib #9 - they removed option for equal axes scale. The red dots are just the green ones with transformation which makes the green fitted ellipse into an orange sphere.
You can install 2.x version of matplotlib and uncomment lines in plot_ellipsoid.py hack for equal axes

Maybe I should rewrite readme and add fixed version of matplotlib to requirements.txt, I don't know.
Also are you sure your data belong on the ellipsoid surface? If you are not sure, please, take a look at #5

@aleksandrbazhin
Copy link
Owner

Figure_1-4

That is your plot stretched to fullhd and rotated a bit. I think it is a sphere, just uneven scale on axes is confusing now.

@aleksandrbazhin
Copy link
Owner

Figure_1-5
And here is top-view, Which is perfectly correct for data not belonging on the surface of an ellipse. I think, I'll post an explanation as a separate issue.

@aleksandrbazhin
Copy link
Owner

Take a look at #11

@robineappen
Copy link
Author

@aleksandrbazhin Yes, that was really helpful. I'll try to refine the data before using and will update here.
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants