Skip to content

Load libaries in Impacts#238

Open
amarini wants to merge 1 commit intocms-analysis:mainfrom
amarini:load_libraries_impacts
Open

Load libaries in Impacts#238
amarini wants to merge 1 commit intocms-analysis:mainfrom
amarini:load_libraries_impacts

Conversation

@amarini
Copy link
Copy Markdown
Contributor

@amarini amarini commented Feb 12, 2020

Impacts do not intercept and load external libraries passed to combine.

@ajgilbert
Copy link
Copy Markdown
Collaborator

Yes thanks, indeed we're missing this at the moment - I might implement it in a slightly different way (in CombineToolBase.py) so it can be used in any combineTool.py method, but let's leave this PR open as a reminder that we should fix it.

@ajgilbert ajgilbert changed the base branch from master to main April 25, 2023 08:26
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