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.

Sign up for GitHub

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

Jump to bottom

Review laatste serie aanpassingen nav trendanalyse rapportage #9

Open
jspijker opened this issue Jan 22, 2021 · 2 comments
Open

Review laatste serie aanpassingen nav trendanalyse rapportage #9

jspijker opened this issue Jan 22, 2021 · 2 comments
Assignees
@jspijker

Comments

@jspijker
Copy link
Collaborator

jspijker commented Jan 22, 2021 •
edited
Loading

Review van commits v1.1 vs 4323c8f:
4323c8f...v1.1
@jspijker jspijker self-assigned this Jan 22, 2021
jspijker added a commit that referenced this issue Mar 1, 2021
@jspijker
Eerste ronde review.
b4eac7c 
* toevoegen van veld instantie aan data verwijderd, is geen onderdeel
  van datamodel
* tests voor data series weer toegevoegd
* return value mktrends.R is weer NA igv te weinig meetwaarden (was print
  value, hoe verwerk je dat?)
* nieuwe functie plot_all.R: reformat code

zie issue #9
@jspijker
Copy link
Collaborator Author

jspijker commented Mar 1, 2021 •
edited
Loading

Twee issues met huidige versie code:

  • in functie monsterFreq wordt variabele nbemonsterd gebruikt, deze is geen onderdeel van de functie environment, waar komt die var vandaan?
  • aan functie trendReversal is optie toegevoegd om coefficienten terug te geven, maar via geneste if..else structuur. Erg onoverzichtelijk en volgens mij klopt logica niet.

jspijker added a commit that referenced this issue Mar 1, 2021
@jspijker
Functie monsterFreq aangepast
c42375c 
In functie werden variabelen gebruikt die of niet bestonden, of geen
doel hadden. Output van functie weer hersteld zoals in v1.0 van package.

Zie issue #9
jspijker added a commit that referenced this issue Mar 1, 2021
@jspijker
Functie trendReversel aangepast
d925ace 
* veld 'instantie' verwijderd, geen onderdeel data model
* extra optie om data.frame met coefs terug te geven verwijderd (d.f.
  werd al terug gegeven)
* output hersteld, bij geen meetwaarden komt leeg data.frame terug in
  plaats van een print statement.

Overige aanpassingen overgenomen

Zie issue #9
@jspijker
Copy link
Collaborator Author

jspijker commented Mar 2, 2021

De volgende bijdragen zijn nu dus overgenomen: v1.0...master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jspijker jspijker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jspijker