Skip to content

Conversation

@pradal
Copy link
Contributor

@pradal pradal commented May 7, 2025

  • Move README.rst to README.md
  • move setup.py to pyproject.toml

@pradal pradal self-assigned this May 7, 2025
@pradal pradal added packaging Related to packaging the software release2025 OpenAlea 2025 release refactoring Adapt to the guidelines labels May 7, 2025
@pradal
Copy link
Contributor Author

pradal commented May 7, 2025

  • update meta.yaml (guidelines)

@pradal pradal moved this to In progress in OpenAlea Release May 7, 2025
@pradal pradal linked an issue May 7, 2025 that may be closed by this pull request
9 tasks
@pradal
Copy link
Contributor Author

pradal commented May 7, 2025

  • Documentation : generate automagically Developper documentation

@pradal pradal requested review from baugetfa and thomasarsouze and removed request for thomasarsouze May 7, 2025 19:24
Copy link

@baugetfa baugetfa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May be add in meta.yaml

- cd test

in Test/commands

Copy link
Contributor

@thomasarsouze thomasarsouze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Local building of documentation creates a lot of warnings.
Not sure how bad it is, but this includes:

/Users/arsouze/Documents/Modeles/Codes/OpenAlea/sconsx/doc/_autosummary/openalea.sconsx.rst:24: WARNING: autosummary : échec de l'importation de qt5.
Indications possibles :
* AttributeError: module 'SCons.Warnings' has no attribute 'Warning'
* AttributeError: module 'openalea.sconsx' has no attribute 'qt5'
* ValueError: not enough values to unpack (expected 2, got 1)
* ModuleNotFoundError: No module named 'qt5'
* KeyError: 'qt5'
.../OpenAlea/sconsx/src/openalea/sconsx/config.py:docstring of openalea.sconsx.config:1: WARNING: description dupliquée de l'objet openalea.sconsx.config, autre instance dans _autosummary/openalea.sconsx.config, utiliser :no-index: pour l'un d'eux

@pradal
Copy link
Contributor Author

pradal commented May 9, 2025

I have added an option to create API automatically.
However it may try to import modules.

@thomasarsouze
Copy link
Contributor

If you think this is fine for you, then ok for merging.

@pradal
Copy link
Contributor Author

pradal commented May 9, 2025

No. I will try to fix the errors

@pradal
Copy link
Contributor Author

pradal commented May 9, 2025

Solved...

Copy link
Contributor

@thomasarsouze thomasarsouze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@pradal pradal merged commit 238fd4a into main May 9, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in OpenAlea Release May 9, 2025
@thomasarsouze thomasarsouze deleted the release branch May 9, 2025 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging Related to packaging the software refactoring Adapt to the guidelines release2025 OpenAlea 2025 release

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Release 2025 sconsx

4 participants