Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

Exclude certain users from leaderboard stats

Open akhil-datla opened this issue 2 years ago • 12 comments

akhil-datla avatar Jan 17 '23 00:01 akhil-datla

:x: pre-commit failed. Please run pre-commit run --all-files locally and commit the changes. Find more information in the repository's CONTRIBUTING.md

github-actions[bot] avatar Jan 17 '23 00:01 github-actions[bot]

:x: pre-commit failed. Please run pre-commit run --all-files locally and commit the changes. Find more information in the repository's CONTRIBUTING.md

github-actions[bot] avatar Jan 17 '23 01:01 github-actions[bot]

:x: pre-commit failed. Please run pre-commit run --all-files locally and commit the changes. Find more information in the repository's CONTRIBUTING.md

github-actions[bot] avatar Jan 17 '23 07:01 github-actions[bot]

:x: pre-commit failed. Please run pre-commit run --all-files locally and commit the changes. Find more information in the repository's CONTRIBUTING.md

github-actions[bot] avatar Jan 17 '23 07:01 github-actions[bot]

@akhil-datla Should this PR be closed? As it is it would lead to gaps in the leaderboard stats which we don't want. One of the main intentions behind adding the enable-stats flag was to allow 'system' users to import data without appearing on the leaderboard etc.

andreaskoepf avatar Jan 23 '23 12:01 andreaskoepf

@andreaskoepf I'll work on the task this week.

akhil-datla avatar Jan 23 '23 19:01 akhil-datla

:x: pre-commit failed. Please run pre-commit run --all-files locally and commit the changes. Find more information in the repository's CONTRIBUTING.md

github-actions[bot] avatar Jan 27 '23 06:01 github-actions[bot]

Why is the website .env file deleted?

isnt .env in the .gitignore

@akhil-datla could rename it to .env.example

AlexanderHott avatar Jan 27 '23 07:01 AlexanderHott

or we could add

!website/.env

to the .gitignore file

AlexanderHott avatar Jan 27 '23 07:01 AlexanderHott

or we could add

!website/.env

to the .gitignore file

We include a .env in the website directory with working values so devs don't have to go create them. They aren't secret in any way. I'd like to keep that file.

fozziethebeat avatar Jan 27 '23 07:01 fozziethebeat

@fozziethebeat, the website CI is failing. However, all the other CI workflows pass.

akhil-datla avatar Jan 27 '23 07:01 akhil-datla

@fozziethebeat, the website CI is failing. However, all the other CI workflows pass.

Please ignore that failing for now. Its a problem we need to investigate but is unrelated to your changes.

fozziethebeat avatar Jan 27 '23 07:01 fozziethebeat

replaced by #961

andreaskoepf avatar Jan 27 '23 19:01 andreaskoepf