Closed MattMonk closed 2 weeks ago
The changes involve updates to the _exists
, mtime
, and size
methods in the snakemake_storage_plugin_xrootd/__init__.py
file. The modifications replace the use of self.path
with self.url.path_with_params
when invoking the stat
method on self.file_system
. This adjustment allows for more accurate file system operations by incorporating all relevant URL components, which is critical for file existence checks, modification time, and size retrieval.
Files | Change Summary |
---|---|
snakemake_storage_plugin_xrootd/__init__.py |
Modified _exists , mtime , and size methods to use self.url.path_with_params instead of self.path for file system operations. |
snakemake_storage_plugin_xrootd/__init__.py
file, enhancing the mtime
and size
methods in relation to file system operations and URL handling.[!TIP]
OpenAI O1 model for chat
- We have deployed OpenAI's latest O1 model for chat. - OpenAI claims that this model has superior reasoning capabilities than their GPT-4o model. - Please share any feedback with us in the [discussions post](https://discord.com/channels/1134356397673414807/1283929536186155099).
snakemake_storage_plugin_xrootd/__init__.py (1)
Pattern `**/*.py`: Do not try to improve formatting. Do not suggest type annotations for functions that are defined inside of functions or methods. Do not suggest type annotation of the `self` argument of methods. Do not suggest type annotation of the `cls` argument of classmethods. Do not suggest return type annotation if a function or method does not contain a `return` statement.
snakemake_storage_plugin_xrootd/__init__.py (3)
`316-316`: **LGTM!** The change to use `url.path_with_params` instead of `self.path` ensures that the complete URL including parameters is considered when checking file existence. This aligns with the PR objective. --- `331-331`: **LGTM!** The change to use `self.url.path_with_params` instead of `self.path` ensures that the complete URL including parameters is considered when retrieving the modification time. This aligns with the PR objective. --- `341-341`: **LGTM!** The change to use `self.url.path_with_params` instead of `self.path` ensures that the complete URL including parameters is considered when determining the file size. This aligns with the PR objective.
Addresses issue https://github.com/snakemake/snakemake-storage-plugin-xrootd/issues/12 by ensuring that URL parameters are propagated correctly.
Summary by CodeRabbit
New Features
Bug Fixes