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

Aanvullingen correcties al #33

Closed
wants to merge 168 commits into from
Closed

Conversation

leymanan
Copy link
Collaborator

@leymanan leymanan commented Oct 1, 2020

kleine aanvullingen aan develop

Zie ook issues #31, #32 en #33 voor aanpassingen die nog moeten gebeuren

…n terwijl er geen gegevens in deze klassen kunnen zitten
…dat een overzicht geeft van de gegevens die niet voor de berekening gebruikt worden
…n een tekstfile in het package bewaard kunnen worden.
…n een tekstfile in het package bewaard kunnen worden.
…ctory te bepalen (om fout in online test te vermijden)
…liciet mee bij render, dezelfde aanpassingen gemaakt voor validatierapport (ook: .Rmd met hoofdletter)
…en werden zo op de verkeerde plaats opgeslagen...
nog enkele functies aangemaakt of aangepast:
- outputIVANHO
- initiatie: rapport als extra uitvoer
- leesFile
!!! Voor deze laatste functie moeten de paths uit het hoofdscript lokaal opgeslagen worden in tekstfiles!
…niet meer opnemen als alle afwijkende metingen goedgekeurd zijn en de curve niet afwijkend is
ElsLommelen and others added 26 commits September 10, 2020 20:53
- enkele typo's
- filter = top toegevoegd bij tabellen (extra filterkeuze)
- enkele aanvullingen ter verduidelijking
- toevoegen van outline-headings
- unieke ID toegevoegd aan bosdatmetingen en nieuwe hoogtemetingen
spatie toegevoegd na ##: om zo domeinnaam en bms als kop weer te geven
spatie toegevoegd na ##: om zo domeinnaam en bms als kop weer te geven
- leesbaarheid verhogen
- updatequery "te controleren" nieuwe hoogtemetingen
@ElsLommelen
Copy link
Collaborator

Misschien logischer om je branch eerst te mergen met de develop? Dan kan ik ook mijn aanpassingen eerst met de develop mergen, en als alles in orde is in de develop, kunnen we deze mergen met de master tot onze versie 0.1. In dit geval zit er in de master nog een oudere testversie en is het nog geen echt probleem als we het niet zo te doen, maar binnenkort, als onze eerste afgewerkte versie in de master zit, gaan we het wel zo moeten doen: een branch met nieuwe aanpassingen altijd naar de develop mergen, en deze pas met de master mergen als we een nieuwe versie willen uitbrengen. Gebruikers kunnen dan kiezen wat ze willen installeren: de afgeklopte masterversie met een duidelijk versienummer, of de develop met de laatste aanpassingen.

Ik weet niet of je het nu als oefening even opnieuw wil doen? (Ik weet niet of je het nog kan aanpassen, waarschijnlijk ga je deze pull request moeten verwijderen en een nieuwe aanmaken.)

@leymanan
Copy link
Collaborator Author

leymanan commented Oct 1, 2020 via email

@leymanan leymanan closed this Oct 1, 2020
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.

4 participants