Skip to content

[Version 3] feat: allow_multiple_runs is true by default #780

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

Merged
merged 29 commits into from
Apr 13, 2025

Conversation

inimaz
Copy link
Collaborator

@inimaz inimaz commented Feb 6, 2025

The lock file has posed several issues lately. Setting as default allow_multiple_runs=True to not block users while we decide how to deal with it.

@benoit-cty
Copy link
Contributor

Thanks !
I prefer to merge this in the branch codecarbon_v3_rc as it change the default behavior.

Maybe we have also to change some documentation and test ?

@benoit-cty benoit-cty changed the title feat: allow_multiple_runs is true by default [Version 3] feat: allow_multiple_runs is true by default Feb 6, 2025
@benoit-cty benoit-cty force-pushed the feat/allow-multiple-runs-true branch from 0eca00d to 06dfe40 Compare April 13, 2025 14:24
@benoit-cty benoit-cty changed the base branch from master to codecarbon_v3_rc April 13, 2025 16:05
inimaz and others added 22 commits April 13, 2025 18:15
* feat(api): add auth to project token endpoints

* fix: add auth_context to project-token-service container

* chores(api): run api unit tests on the CI

Signed-off-by: inimaz <93inigo93@gmail.com>

* add logs to project-token call

* chores: custom CORS taken from env variable

* fix: log fief response when there is no access token

* fix: credentials included in fetch for project tokens

* fix: remove use server from project token api calls

Signed-off-by: inimaz <93inigo93@gmail.com>

* fix: remove useless prints

Signed-off-by: inimaz <93inigo93@gmail.com>

---------

Signed-off-by: inimaz <93inigo93@gmail.com>
Add public flag to project
Allow getting data for public project without authentication
Bumps [next](https://github.com/vercel/next.js) from 14.2.15 to 14.2.21.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](vercel/next.js@v14.2.15...v14.2.21)

---
updated-dependencies:
- dependency-name: next
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: settings page update project

* add animation saying changes saved

Signed-off-by: inimaz <93inigo93@gmail.com>

---------

Signed-off-by: inimaz <93inigo93@gmail.com>
* fix: fix account creation with cli login

---------

Co-authored-by: patrick.lloret@protonmail.com <patrick.lloret@protonmail.com>
Signed-off-by: inimaz <93inigo93@gmail.com>
* ci: deploy app from ci

* test triggering the deploy

Signed-off-by: inimaz <93inigo93@gmail.com>

* fix: add reference to local clever cli

* unshallow the fetch

Signed-off-by: inimaz <93inigo93@gmail.com>

* fix: deploy to multiple applications using --alias APP_NAME

Signed-off-by: inimaz <93inigo93@gmail.com>

* reduce verbosity of deploy logs (no need for app logs)

Signed-off-by: inimaz <93inigo93@gmail.com>

* .

---------

Signed-off-by: inimaz <93inigo93@gmail.com>
* chores: split github workflows

* docs: typo in the name of the build jobs

Signed-off-by: inimaz <93inigo93@gmail.com>

---------

Signed-off-by: inimaz <93inigo93@gmail.com>
inimaz and others added 4 commits April 13, 2025 18:27
* feat: add experiment modal

Signed-off-by: inimaz <93inigo93@gmail.com>

* fix: timestamp optional

* fix: timestamp

* rearrange the ui

* fix: experiment creation

* fix: show id of created experiment

Signed-off-by: inimaz <93inigo93@gmail.com>

* fix: remove unnecessary fields from experiments

Signed-off-by: inimaz <93inigo93@gmail.com>

---------

Signed-off-by: inimaz <93inigo93@gmail.com>
Bumps [@babel/runtime](https://github.com/babel/babel/tree/HEAD/packages/babel-runtime) from 7.24.8 to 7.27.0.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.27.0/packages/babel-runtime)

---
updated-dependencies:
- dependency-name: "@babel/runtime"
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* improvement: Upgrade to Next 15

* fix: organization navbar select not empty

* feat: improved member page + loader + breadcrumb layout

* feat: project settings page + code refactoring + public project page

* lib: upgrade next for security

* fix: clarify api client/server side with fief auth

* feat: export json and csv + refactorings

* improvement: create org and create project loading and refresh logic

* fix: downgrade react-day-picker

* build: update package.json

* doc: updated readme to use pnpm

* build: trigger build

* feat: export to json button and data fetching

* fix: build issue after rebase

* fix: githbu build webapp with pnpm

* fix: build-ui github action use pnpm build

* fix: prettier fixes

* fix: prettier pnpm lock

* fix: ignore lock files in prettier
Intel(R) Xeon(R) W-11955M CPU @ 2.60GHz
@benoit-cty benoit-cty force-pushed the feat/allow-multiple-runs-true branch from 06dfe40 to 289ba32 Compare April 13, 2025 16:28
@benoit-cty benoit-cty merged commit 1bc0982 into codecarbon_v3_rc Apr 13, 2025
4 checks passed
@benoit-cty benoit-cty deleted the feat/allow-multiple-runs-true branch April 13, 2025 16:40
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.

9 participants