Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

3d artist objects need clear and unified way to get/set 3d coordinate data #1483

Open
WeatherGod opened this Issue Nov 11, 2012 · 3 comments

Comments

Projects
None yet
5 participants
Member

WeatherGod commented Nov 11, 2012

Because get_offsets() need to remain 2-D based for draw(), a separate function is needed to get and set the 3d coordinate data. Currently, one would set the 2d offsets, and then use set_3d_properties() to set the 3rd dimension data, which is awkward.

Perhaps this is an opportunity to create a function in the base artist class strictly for raw coordinate data? get/set_offsets() would then be intended more for display/transform purposes but would often be the same as the new methods. Also, this would help clear up confusion where some artists have get/set_array(), get/set_data() and get/set_offsets() all for coordinate data. A unified method would be valuable for documentation purposes.

@ghost ghost assigned WeatherGod Nov 11, 2012

@tacaswell tacaswell added this to the v1.5.x milestone Aug 17, 2014

@tacaswell tacaswell modified the milestones: v1.5.x, unassigned Aug 18, 2014

Member

WeatherGod commented Jun 30, 2015

This would be a great low-hanging fruit task for SciPy 2015 sprints

Member

pelson commented Jul 1, 2015

I agree that the doing of this is simple, but I don't think it is an easy thing todo unless we can be very specific about what is required (unless one of us were doing it, rather than a newcomer to the project).

Contributor

OceanWolf commented Jul 4, 2015

Hmm, by the description above, do I understand that we want to separate out the screen stuff from the axes stuff because we shouldn't assume them as identical...?

I would imagine we should make this generic for all coordinate systems, not just 3d, what do we do with polar axes coordinates, and basemap?

@WeatherGod WeatherGod removed the Duplicate label Jan 3, 2017

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