Skip to content

Hexa 1330 current run and pipeline#292

Merged
nazarfil merged 2 commits into
mainfrom
HEXA-1330-current-run-and-pipeline-id
Aug 5, 2025
Merged

Hexa 1330 current run and pipeline#292
nazarfil merged 2 commits into
mainfrom
HEXA-1330-current-run-and-pipeline-id

Conversation

@nazarfil

@nazarfil nazarfil commented Jul 8, 2025

Copy link
Copy Markdown

implements:

  • current_run.id property to pin to the run_id inside pipeline context
    implements:
  • current_pipeline object to display pipeline parameters

fixes runtime import OpenHexa and instantiation

@nazarfil nazarfil requested a review from yolanfery July 8, 2025 20:06
@nazarfil nazarfil force-pushed the HEXA-1330-current-run-and-pipeline-id branch 4 times, most recently from c165371 to 79c4382 Compare July 9, 2025 06:58
Comment thread openhexa/sdk/pipelines/run.py Outdated
Comment thread openhexa/sdk/pipelines/run.py Outdated
Comment thread openhexa/sdk/pipelines/run.py Outdated
@nazarfil nazarfil force-pushed the HEXA-1330-current-run-and-pipeline-id branch from ed80a7a to eb2c9f1 Compare July 9, 2025 09:13
@nazarfil nazarfil requested a review from yolanfery July 9, 2025 09:14
Comment thread openhexa/sdk/client/__init__.py Outdated
@nazarfil nazarfil force-pushed the HEXA-1330-current-run-and-pipeline-id branch from 46d4e3c to 176a805 Compare July 9, 2025 11:00
@nazarfil

nazarfil commented Jul 9, 2025

Copy link
Copy Markdown
Author

After a discussion the pipeline run id, and other pipeline description parameters are already exposed in env vars.
I have added pipeline code to env vars as well. So we can expose all of it here in methods.
BLSQ/openhexa-app#1251

(outside of this topic : reflect on how to manage imports to avoid instantiation of OpenHexaClient)

@nazarfil

nazarfil commented Jul 9, 2025

Copy link
Copy Markdown
Author

to update the doc if methods change : https://github.com/BLSQ/openhexa-toolbox/edit/main/docs/lineage.md

@yolanfery

yolanfery commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

After a discussion the pipeline run id, and other pipeline description parameters are already exposed in env vars. I have added pipeline code to env vars as well. So we can expose all of it here in methods. BLSQ/openhexa-app#1251

Looks way better, nice !

(outside of this topic : reflect on how to manage imports to avoid instantiation of OpenHexaClient)

It seems my IDE is giving me good warnings already to help with importing it from the correct place
Screenshot 2025-07-09 at 13 12 26

@nazarfil nazarfil requested a review from yolanfery August 4, 2025 13:42
@nazarfil nazarfil merged commit d1dc1a4 into main Aug 5, 2025
4 checks passed
@nazarfil nazarfil deleted the HEXA-1330-current-run-and-pipeline-id branch August 5, 2025 09:09
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.

2 participants