Python generator inserts wrong file_mtime call

[lars-reimann]

Describe the bug

Running the code pasted below leads to the following error log:

2024-04-08 19:58:32.568 [info] Launching Pipeline (b886301b-34a2-47fe-8830-0366fa424c7d): file:///c%3A/Users/Lars/OneDrive/Desktop/test/test.sdstest - myPipeline
2024-04-08 19:58:32.568 [info] [Runner] Sending message to python server: {"type":"program","id":"b886301b-34a2-47fe-8830-0366fa424c7d","data":{"code":{"test":{"gen_test":"# Imports ----------------------------------------------------------------------\r\n\r\nimport safeds_runner\r\nfrom safeds.data.tabular.containers import Table\r\n\r\n# Pipelines --------------------------------------------------------------------\r\n\r\ndef myPipeline():\r\n    titanic = safeds_runner.memoized_call(\"safeds.data.tabular.containers.Table.remove_columns\", Table.remove_columns, [safeds_runner.memoized_call(\"safeds.data.tabular.containers.Table.from_csv_file\", Table.from_csv_file, ['C:/Users/Lars/OneDrive/Desktop/test/titanic.csv'], [safeds_runner.file_mtime('C:/Users/Lars/OneDrive/Desktop/test/titanic.csv')]), ['passenger_id', 'name', 'ticket', 'cabin', 'port_embarked']], [safeds_runner.file_mtime(['passenger_id', 'name', 'ticket', 'cabin', 'port_embarked'])])\r\n    safeds_runner.save_placeholder('titanic', titanic)\r\n","gen_test_myPipeline":"from .gen_test import myPipeline\r\n\r\nif __name__ == '__main__':\r\n    myPipeline()\r\n"}},"main":{"modulepath":"test","module":"test","pipeline":"myPipeline"}}}
2024-04-08 19:58:35.688 [info] [Runner] Message received: '{"type": "runtime_error", "id": "b886301b-34a2-47fe-8830-0366fa424c7d", "data": {"message": "argument should be a str or an os.P<truncated>'
2024-04-08 19:58:35.689 [error] [Runner] Runner-RuntimeError (b886301b-34a2-47fe-8830-0366fa424c7d): argument should be a str or an os.PathLike object where __fspath__ returns a str, not 'list' 
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_pipeline_manager.py line 269
	at <frozen runpy> line 226
	at <frozen runpy> line 98
	at <frozen runpy> line 88
	at test/gen_test_myPipeline line 4
	at test/gen_test line 9 (mapped to 'test.sdstest' line 4)
	at E:\Repositories\safe-ds\Runner\src\safeds_runner\server\_pipeline_manager.py line 360
	at C:\Users\Lars\AppData\Local\Programs\Python\Python312\Lib\pathlib.py line 1162
	at C:\Users\Lars\AppData\Local\Programs\Python\Python312\Lib\pathlib.py line 373
2024-04-08 19:58:35.689 [error] [Runner] Safe-DS Error (b886301b-34a2-47fe-8830-0366fa424c7d): argument should be a str or an os.PathLike object where __fspath__ returns a str, not 'list' 
	at file:///c%3A/Users/Lars/OneDrive/Desktop/test/test.sdstest#4 (c:\Users\Lars\OneDrive\Desktop\test\test.sdstest line 4)

It seems that the culprit is the incorrect call safeds_runner.file_mtime(['passenger_id', 'name', 'ticket', 'cabin', 'port_embarked']). It works fine if the chaining is replaced by an explicit assignment to a placeholder.

To Reproduce

Run the following code (replace the absolute path to the CSV file):

package test

pipeline myPipeline {
    val titanic = Table
        .fromCsvFile("C:/Users/Lars/OneDrive/Desktop/test/titanic.csv")
        .removeColumns(["passenger_id", "name", "ticket", "cabin", "port_embarked"]);
}

Expected behavior

No error.

Screenshots (optional)

No response

Additional Context (optional)

No response

[lars-reimann]

@WinPlay02 Do you have an idea what could cause this?

[WinPlay02]

No, not directly. I'll try to investigate this issue and propose a fix

[lars-reimann]

Thanks. It's not necessary to have this ready for this Friday, if you are otherwise occupied. We can tell people to "unchain" their calls for now.

[lars-reimann]

🎉 This issue has been resolved in version 0.11.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• npm package (@latest dist-tag)
• v0.11.0
• GitHub release

Your semantic-release bot 📦🚀