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

Update asn_from_list to accept target argument #1408

Closed
stscijgbot-rstdms opened this issue Sep 11, 2024 · 2 comments · Fixed by #1411
Closed

Update asn_from_list to accept target argument #1408

stscijgbot-rstdms opened this issue Sep 11, 2024 · 2 comments · Fixed by #1411

Comments

@stscijgbot-rstdms
Copy link
Collaborator

Issue RCAL-910 was created on JIRA by David Davis:

Update asn_from_list to accept a command line argument to set the target field.

@stscijgbot-rstdms
Copy link
Collaborator Author

Comment by David Davis on JIRA:

Adding this has reveled some additional errors in the association code, 

from romancal.associations.registry import AssociationRegistry
>>> registry = AssociationRegistry()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/ddavis/src/Roman/romancal/romancal/associations/registry.py", line 93, in _init_
    self.populate(
  File "/Users/ddavis/src/Roman/romancal/romancal/associations/registry.py", line 267, in populate
    self.Utility = type("Utility", (obj, self.Utility), {})
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: Cannot create a consistent method resolution
order (MRO) for bases Utility, Utility```
I will use this ticket to investigate/fix this error so that we can add the target field.

If this goes too far afield I'll pause this ticket and write a specific ticket for the above error

@stscijgbot-rstdms
Copy link
Collaborator Author

Comment by Nadia Dencheva on JIRA:

Code changes in #1411

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

Successfully merging a pull request may close this issue.

1 participant