Skip to content
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

[Umbrella/RFC] Improvements to the scheduler & platforms layer #36

Open
3 of 6 tasks
berekuk opened this issue Mar 29, 2022 · 2 comments
Open
3 of 6 tasks

[Umbrella/RFC] Improvements to the scheduler & platforms layer #36

berekuk opened this issue Mar 29, 2022 · 2 comments
Labels
feature A new feature

Comments

@berekuk
Copy link
Collaborator

berekuk commented Mar 29, 2022

So I took a closer look at how backend/platforms are implemented, and I have a few ideas on how to improve.

@NunoSempere, let me know if you're ok with all these or if you have other suggestions.

This is a longer term roadmap, and the last two items are a bit more complex, but I believe that they're quite beneficial. Getting closer to the "metaforecast data is never stale" goal would be great.

@NunoSempere
Copy link
Collaborator

Most of these sound good, no strong opinions on unifying the platforms list.

@berekuk berekuk mentioned this issue Mar 31, 2022
6 tasks
@berekuk
Copy link
Collaborator Author

berekuk commented Mar 31, 2022

One more thing: I'd like to change the platform field values in the DB and other related code (e.g. stars.ts) to the short platform name/id, e.g. "goodjudgment" instead of "Good Judgment", and leave longer titles just for displaying. This is useful for consistency, for type safety, etc.

(from #33 (comment), but relevant here too)

I also think that stars.ts code maybe should be split and moved into platforms files too. I see the benefit of having it all in a single file, but I don't like the fragility of "if I make a typo in a platform's name then it'll just fallback on 2 stars default without any signal that something went wrong".

@berekuk berekuk added the feature A new feature label Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature
Projects
None yet
Development

No branches or pull requests

2 participants