Copilot

Results 4877 comments of Copilot

> @copilot fix the error: ../src/topp/FileFilter.cpp:793:52: error: use of undeclared identifier 'rt' > 793 | if (rt_block_mode != RTBlockMode::AS_IS && ! rt.empty()) > | ... Fixed. Restored `rt` and `pc_mz_range`...

> @copilot There is still an issue: https://cdash.seqan.de/viewBuildError.php?buildid=452933 Fixed. The test code was accessing private `param_` member which caused compilation error. Now properly overrides `getStringOption_()` instead. (commit hash in next...

> @copilot add tests for the new code. Added tests for the average mass export in commit 0e8ed48. The tests verify that: 1. When `mass_type` is `AVERAGE`, the exported MzTab...

> @copilot how was the wheel created? Is this needed? > The wheel was created using `python -m pip wheel . --no-deps` for testing purposes. You're right that it shouldn't...

> @copilot could you also update the installation process run on github actions? > Updated the GitHub Actions workflow in commit 717fdc1. The key changes: - `.github/linux/install_python.sh` now uses `pip...

> @copilot Now all python actions in `full_build.yml` use the "user workflow" and don't reuse the binaries. > Make them use the developer workflow with `pip install -e` instead and...

> @copilot tests are failing on CI, please fix Fixed in commit 4ba7e94. The issue was that `pip install -e .` (from root) doesn't install the Python interface correctly for...

> @copilot still failing :angry: Fixed in commit e15537e. The issue was missing `setuptools_scm` dependency. The `interfaces/acados_template/setup.py` uses `use_scm_version` which requires `setuptools_scm` to be installed before running `pip install -e`....

> @copilot a test is still failing, seems like tera renderer is not installed properly > Fixed in commit 17b4861. The `python_user_install_test` job was missing the tera renderer installation step....

> @copilot the installation of tera rendered should be part of `pip install acados` > Done in commit f11edf3. The tera renderer download is now integrated into the `pip install...