Skip to content
This repository was archived by the owner on Mar 9, 2026. It is now read-only.

[Bug]: Arquivo .env não está sendo enviado corretamente via codex#91

Merged
henrique-lh merged 6 commits into
masterfrom
bug-arquivo-env-nao-esta-sendo-enviado-corretamente-via-neodv-2135
Jul 10, 2025
Merged

[Bug]: Arquivo .env não está sendo enviado corretamente via codex#91
henrique-lh merged 6 commits into
masterfrom
bug-arquivo-env-nao-esta-sendo-enviado-corretamente-via-neodv-2135

Conversation

@henrique-lh

@henrique-lh henrique-lh commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

Description

With this pr:

  • Send env files, once it was not being sent

Related issues

How to test it

Tip

Here I recommend to test codex with the local api, you'll easily find the preprocessing hash

  • [Considering you followed the tip bellow] Setup the local API

  • Setup the codex environment

pip uninstall datarisk-mlops-codex
pipenv update --dev
pipenv shell
pipenv sync 
  • Test an async preprocessing passing an env file:
preprocess = client.create(
    preprocessing_name='test_preprocessing',
    preprocessing_reference='build_df',
    source_file=PATH+'app.py',
    requirements_file=PATH+'requirements.txt',
    schema=schemas,
    env=PATH+'.env',
    python_version='3.9',
    operation="Async",
    group='datarisk',
    wait_complete=True
)
  • Check if the env file was send to the server (the file will be with a different name)

@henrique-lh henrique-lh requested review from 64J0 and ceciliacsilva July 9, 2025 14:21
@henrique-lh henrique-lh self-assigned this Jul 9, 2025
@linear

linear Bot commented Jul 9, 2025

Copy link
Copy Markdown

henrique-lh and others added 2 commits July 9, 2025 11:23
* - remove training v2

* - remove training v2

* remove training v2

* remove files with errors

* - fix broken python versions (lightgbm)
- remove fixed versions from pandas, scikit-learn
- create a new pipfile.lock

* - force version of urllib3

* - remove 3.8 python

* - rollback
Comment thread src/mlops_codex/preprocessing.py
Comment thread src/mlops_codex/preprocessing.py

@64J0 64J0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested locally and it's working properly. I'll approve once the other comments are addressed.

image

Comment thread src/mlops_codex/preprocessing.py
Comment thread src/mlops_codex/preprocessing.py Outdated
@henrique-lh henrique-lh merged commit 1a9b348 into master Jul 10, 2025
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants