Skip to content

CSV Export: Adapt for multiple column_names in FUNCTION_INFO#396

Open
w31t1 wants to merge 1 commit intoterryyin:masterfrom
w31t1:master
Open

CSV Export: Adapt for multiple column_names in FUNCTION_INFO#396
w31t1 wants to merge 1 commit intoterryyin:masterfrom
w31t1:master

Conversation

@w31t1
Copy link
Copy Markdown

@w31t1 w31t1 commented Oct 10, 2024

Some extensions like IO (Fan-In/Fan-Out) generate more than one data column. All these columns shall appear in the CSV output.

Some extensions like IO (Fan-In/Fan-Out) generate more than one data
column. All these columns shall appear in the CSV output.
@w31t1
Copy link
Copy Markdown
Author

w31t1 commented Oct 10, 2024

Tests executed on Python 3.12.5 successfully.

I considered an additional security check in L27 (if hasattr(...)):

            if hasattr(extension, 'FUNCTION_INFO') and isinstance(extension.FUNCTION_INFO, dict):

but removed it as the data structure is defined in class OutputScheme.

@w31t1
Copy link
Copy Markdown
Author

w31t1 commented Oct 10, 2024

Unfortunately I cannot identify the build failures raised by the continuous integration pipeline. Can somebody point me to the problem?

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.

1 participant