Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(release): v2.17.6 #2159

Merged
merged 60 commits into from
Sep 25, 2024
Merged

chore(release): v2.17.6 #2159

merged 60 commits into from
Sep 25, 2024

Conversation

skamril
Copy link
Member

@skamril skamril commented Sep 25, 2024

No description provided.

skamril and others added 30 commits August 2, 2024 12:34
* refactor main.yml workflow to separate lint, test, and build jobs
* implement caching for npm dependencies to speed up CI
)

Configuration were arbitrarily dispatched between setup.py, setup.cfg,
and the newer pyproject.toml.

---------

Signed-off-by: Sylvain Leclerc <[email protected]>
This allows to remove an unnecessary trick from CI, which could break if
anything change on github side.

Signed-off-by: Sylvain Leclerc <[email protected]>
Uses antares-timeseries-generation to perform thermal clusters
timeseries generation.

The generation is performed as a background task on file studies,
since it can take several seconds or more.

Note that generated matrices are not "inner matrices" of the
command since they are generated on each command execution.
Generated matrices that will be stored later in the matrix store will
be saved from garbage collection as all other snaphot matrices.
The changelog is maintained by hand in documentation, it's better to use
it everywhere for consistency sake. Also, generated changelog did not
seem to match
the released version (see changelog in v2.17.5 zip which only contains
changes up to v2.17.4).

---------

Signed-off-by: Sylvain Leclerc <[email protected]>
Depending on the BC operator, only relevant matrix files are
now expected and considered present.
…tion (#2092)

Context: 
for the aggregation `apis`, we want to add additional endpoints that
perform aggregation on the folder `economy/mc-all`
Addind the new installer to the package requires to update the
documentation with new directions about how to install the application.
In this PR, I added instructions for both CLI and GUI version of the
installer.
The installer is now packaged together with antares desktop.
For windows, we provide the GUI version, for linux we only
provide the command line version.

Installer code is added as a git submodule and built with its
hatch build system.


JIRA ANT-1966

---------

Signed-off-by: Sylvain Leclerc <[email protected]>
Co-authored-by: Sylvain Leclerc <[email protected]>
…2128)

Context: 
In the job listing of `AntaREST`, we call two endpoints: 
- `GET`, `v1/launcher/jobs`
- `GET`, `v1/launcher/jobs/{job_id}/progress`
It turns out that when user does not have a permission for some job,
this job, though listed (using the first endpoint of `jobs`), spits out
an error `403` when it is called for the second endpoint `progress`.

Issue:
For the endpoint `jobs`, the logic beyond the permissions includes a
second option where when an orphan job is created recently enough would
become visible to everyone. This logic was not included in the
`progress` endpoint which leads to the `Exception` above.

Solution:
After consulting the application users, they think that this `orphan`
visibility logic is not that much relevant and should be removed (at
least temporarily).

---------

Co-authored-by: Sylvain Leclerc <[email protected]>
This PR adds AUTHORS.txt and a copyright header for every python 
files inside folders `antarest` and `tests`.
It also introduces a new github action that checks that every one of
these files contains a header.
It's based on a home-made Python script (ugly and unoptimized but runs
quickly enough) inside the `scripts` folder.

See ANT-1916
- improve checker to skip licensed subfolders
- remove some wrong copyrights from forked projects

---------

Signed-off-by: Sylvain Leclerc <[email protected]>
Enable the use of pytest-xdist to run tests in parallel, in CI
or developer env.

Following issues with parallel execution of tests are solved :

- some checks depended on the consistency at 2 moments in time of the
result of disk_usage, which was anyway not a good idea (dependent on
what happens elsewhere on the machine). It's now replaced with mocking
- some parameterized tests had random sort order, which is not allowed
by pytest-xdist (all workers must have exactly the same tests in the
same order)
- the command_factory test checked something in its teardown, assuming
all tests were executed in the same process, which is not the case
anymore. The same check is done as a separate unit tests.

Possible remaining issue which could cause some timeouts:
some services hold FileLock which have the same identifier
accross multiple tests.

Signed-off-by: Sylvain Leclerc <[email protected]>
* install and use @mui/x-tree-view instead of the beta version of TreeView
* update style
* use SplitView
* fix data type
* fix issue with `list.txt` file displayed like a folder
@skamril skamril self-assigned this Sep 25, 2024
@skamril skamril requested review from hdinia and sylvlecl September 25, 2024 16:29
@skamril skamril added the release New release label Sep 25, 2024
@skamril skamril merged commit 6ff5520 into master Sep 25, 2024
8 of 10 checks passed
@skamril skamril deleted the release/2.17.6 branch September 25, 2024 18:33
@skamril skamril restored the release/2.17.6 branch September 25, 2024 18:33
@skamril skamril deleted the release/2.17.6 branch September 26, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release New release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants