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

Definice default type HarvestConfiguration #594

Closed
JanMeritus opened this issue Jul 29, 2021 · 2 comments
Closed

Definice default type HarvestConfiguration #594

JanMeritus opened this issue Jul 29, 2021 · 2 comments
Assignees
Labels
dataType Data types dev and optimalisation.

Comments

@JanMeritus
Copy link
Contributor

ad navrh @Fasand v #402

  • jestvuje pro kazdy typ Sklizne (Serials, Topics, Tests, Totals)
  • navyse existuje tzv default, priradzen najpr nasledne podle typu sklizne updatovan / pokud typ sklizne nema definovan harvestConfiguration tak default
  • key value pairs pole:
    • "seedsNo": 300,
    • "duration": 259200,
    • "budget": 10000,
    • "dataLimit": 10000000000,
    • "documentLimit": 0,
    • "deduplication": "PATH",
@Fasand
Copy link
Contributor

Fasand commented Aug 2, 2021

  • Novy model HarvestConfiguration(BaseModel): { harvest_type, seedsNo, duration, ..., deduplication }
  • Harvest bude mit nove required fields "seedsNo, duration, ..., deduplication", ktere se nastavi v poradi: Manuálně -> HarvestConfig -> model default (ty specifikovane výše)
  • V obou pripadech budou fields nastavene na PositiveInteger pro validaci az na deduplication (Char)
  • Pri vytvareni sklizne se po vyberu typu sklizne pres AJAX stahne nastaveni pro dany typ sklizne – pokud neexistuje, ukaze se nejake upozorneni
  • HarvestConfig budou moci menit pouze uzivatele ve skupine "Tech" (a superusers)

@JanMeritus JanMeritus added the dataType Data types dev and optimalisation. label Aug 4, 2021
@Fasand
Copy link
Contributor

Fasand commented Aug 17, 2021

@JanMeritus sorry že jsem si toho všiml až teď, ale mělo by tam opravdu být i seedsNo? To jsem chápal jednoduše jako celkový počet semínek ve všech kolekcích ve sklizni a každá kolekce pak má i vlastní seedsNo, viz #402 (comment)
Chápu to tedy jako vypočítanou hodnotu, nikoli přednastavenou.

EDIT po telefonátu: seedsNo by samozřejmě mělo být dynamické, tedy není součástí HarvestConfiguration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dataType Data types dev and optimalisation.
Projects
None yet
Development

No branches or pull requests

2 participants