Skip to content

Conversation

@georgi
Copy link
Contributor

@georgi georgi commented May 30, 2025

Summary

  • add CSVRowIterator node for iterating CSV strings
  • add LoadCSVFileStream node for streaming CSV files
  • document new nodes in nodetool.data
  • update node index and tests
  • skip tone test when tone node unavailable

Testing

  • black src/nodetool/nodes/nodetool/data.py src/nodetool/nodes/nodetool/os.py tests/nodetool/test_csv_stream.py tests/nodetool/test_transform.py
  • ruff check src/nodetool/nodes/nodetool/data.py src/nodetool/nodes/nodetool/os.py tests/nodetool/test_csv_stream.py tests/nodetool/test_transform.py
  • mypy src/nodetool/nodes/nodetool/data.py src/nodetool/nodes/nodetool/os.py tests/nodetool/test_csv_stream.py tests/nodetool/test_transform.py (fails: Library stubs not installed)
  • flake8 tests/nodetool/test_csv_stream.py src/nodetool/nodes/nodetool/data.py src/nodetool/nodes/nodetool/os.py tests/nodetool/test_transform.py (fails: E501 line too long)
  • PYTHONPATH=src pytest tests/nodetool/test_csv_stream.py tests/nodetool/test_transform.py::TestTone::test_tone_generation -q

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant