Skip to content

Allow renaming of class methods#10

Open
Thoemi09 wants to merge 1 commit intoflatironinstitute:unstablefrom
Thoemi09:rename_methods
Open

Allow renaming of class methods#10
Thoemi09 wants to merge 1 commit intoflatironinstitute:unstablefrom
Thoemi09:rename_methods

Conversation

@Thoemi09
Copy link
Contributor

@Thoemi09 Thoemi09 commented Dec 9, 2025

  • add new method get_python_name to the worker class
    • checks if a given function declaration has a rename attribute and renames it if so
  • use get_python_name for functions as well as class methods

@parcollet
Copy link
Collaborator

  • To be rebased, I slighlty cleaned the implementaiton of the function in the other PR.
  • Can we add a simple test for this ?

@Thoemi09
Copy link
Contributor Author

There is already a PR in c2py: flatironinstitute/c2py#7

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 this pull request may close these issues.

2 participants