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

[BUG] switch to dwh_connector #98

Open
5 tasks
SanderDevisscher opened this issue Dec 17, 2024 · 1 comment · May be fixed by #99
Open
5 tasks

[BUG] switch to dwh_connector #98

SanderDevisscher opened this issue Dec 17, 2024 · 1 comment · May be fixed by #99
Assignees
Labels
bug Something isn't working

Comments

@SanderDevisscher
Copy link
Collaborator

Welke functie/dataset
label_selecter

Beschrijf de bug
Met de huidige flow worden alle bestanden van de DWH binnengehaald en weggeschreven, dit is niet nodig met de dwh_connector

Stappen om de bug te reproduceren:
run fistools::label_selecter(update = true)

Verwachte uikomst
run fistools::label_selecter(update = true) is a lot faster

Screenshots
Indien van toepassing

Extra context
sessionInfo

package - versie
resultaat van utils::packageVersion("fistools")

Checklist

  • corrigeer bug
  • voer devtools::check() uit in de console
  • los eventuele errors, warnings en notes1 op
  • increment versie dmv usethis::use_version(which = "patch")2
  • maak een pull request met @soriadelva of @SanderDevisscher en eventueel andere relevante gebruikers als reviewer.

1in de mate van het mogelijke

*2Indien het nodig is om de functie grotendeels te herzien (bvb tgv een depricated package) om de bug op te lossen of als er meerdere functies en/of datasets bij betrokken zijn, gebruik: usethis::use_version(which = "minor"). *

@SanderDevisscher SanderDevisscher added the bug Something isn't working label Dec 17, 2024
@SanderDevisscher
Copy link
Collaborator Author

@LynnPallemaerts FYI

@SanderDevisscher SanderDevisscher self-assigned this Dec 17, 2024
SanderDevisscher added a commit that referenced this issue Dec 17, 2024
SanderDevisscher added a commit that referenced this issue Dec 17, 2024
@SanderDevisscher SanderDevisscher linked a pull request Dec 17, 2024 that will close this issue
@SanderDevisscher SanderDevisscher linked a pull request Dec 17, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant