Skip to content

31 register callback decorator#51

Open
JAuriac wants to merge 17 commits intodeisa-project:mainfrom
JAuriac:31_RegisterCallbackDecorator
Open

31 register callback decorator#51
JAuriac wants to merge 17 commits intodeisa-project:mainfrom
JAuriac:31_RegisterCallbackDecorator

Conversation

@JAuriac
Copy link
Copy Markdown
Collaborator

@JAuriac JAuriac commented Apr 23, 2026

Close #31

Copy link
Copy Markdown
Contributor

@benoitmartin88 benoitmartin88 left a comment

Choose a reason for hiding this comment

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

I think this is a good start.

Comment thread test/test_deisa.py Outdated
Comment thread src/deisa/dask/deisa.py
Comment thread src/deisa/dask/deisa.py Outdated
@benoitmartin88
Copy link
Copy Markdown
Contributor

benoitmartin88 commented Apr 26, 2026

Is this PR still a draft or is it ready for review ?

@JAuriac JAuriac marked this pull request as ready for review April 27, 2026 08:18
Copy link
Copy Markdown
Contributor

@benoitmartin88 benoitmartin88 left a comment

Choose a reason for hiding this comment

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

A few changes are needed.
Could you rebase on main to take into account the DeisaArray changes ?

Comment thread CHANGELOG.md Outdated
Comment thread src/deisa/dask/deisa.py Outdated
Comment thread src/deisa/dask/deisa.py Outdated
Comment thread src/deisa/dask/deisa.py Outdated
Comment thread src/deisa/dask/deisa.py
Comment thread test/test_deisa.py
@JAuriac JAuriac marked this pull request as draft April 27, 2026 13:31
@JAuriac JAuriac marked this pull request as ready for review April 28, 2026 09:18
@JAuriac JAuriac requested a review from benoitmartin88 April 28, 2026 09:18
Comment thread src/deisa/dask/deisa.py
Comment thread src/deisa/dask/deisa.py Outdated
Comment thread test/test_deisa.py Outdated
Comment thread test/test_deisa.py Outdated
Comment thread test/test_deisa.py Outdated
Comment thread src/deisa/dask/deisa.py Outdated
Comment thread src/deisa/dask/deisa.py
@JAuriac JAuriac requested a review from benoitmartin88 May 6, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Register callback decorator

2 participants