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.

Sign up for GitHub

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

Jump to bottom

Improve SWIG access to namespace'd models/shaders etc #30

Open
Robadob opened this issue Feb 17, 2021 · 0 comments
Open

Improve SWIG access to namespace'd models/shaders etc #30

Robadob opened this issue Feb 17, 2021 · 0 comments
Labels
enhancement New feature or request SWIG

Comments

@Robadob
Copy link
Member

Robadob commented Feb 17, 2021

The swig interface drops all namespaces, e.g.Stock::Models::ICOSPHERE becomes pyflamegpu.ICOSPHERE.

It would be useful to change this as a minimum to pyflamegpu.Models.ICOSPHERE.

Initial thoughts are to make the inner namespace a class with static members so that it's name is retained.

@mondus thinks this has been addressed for a similar issue in the main lib, but didn't recall where.
@Robadob Robadob added enhancement New feature or request SWIG labels Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request SWIG
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Robadob