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

Prepare next release #34

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Prepare next release #34

wants to merge 64 commits into from

Conversation

hsonne
Copy link
Member

@hsonne hsonne commented Sep 15, 2023

This change is Reviewable

into aggregate_export_fst() and use this function within
the aggregate_export_fst_* functions. Pass functions that
are to be used to process the different kinds of data.
Use Unicode instead of latin1-code to encode Greek
character "my"
any more. Read from the yml files. Use new function
get_calc_config().
No more need for the get_calc_info_<site>() functions
Remove package_file(). It is not used any more.
- use dplyr::mutate()
- simplify variable names (e.g. simply "files", "file")
- use do.call(rbind, lapply(...)) instead of for loop
hsonne and others added 30 commits November 12, 2021 21:04
The lapply() call did not seem to be closed correctly.
The call data.table::rbindlist(...) must be called after
the lapply() call, otherwise "raw_list" does not yet
exist!
to-do: roll back once GH actions completed successfully!
Merge branch 'dev' into clean

# Conflicts:
#	DESCRIPTION
#	R/read_mbr4.R
to (hopefully) remove last R CMD check warning
from the base package. It seems that the
function was renamed between package versions
I would have liked @mrustl to review this...
Merge branch 'dev' into clean

# Conflicts:
#	R/group_datetime.R
#	R/import_operation_berlin_t.R
#	man/calculate_operational_parameters.Rd
#	man/calculate_operational_parameters_berlin_f.Rd
#	man/import_data_berlin_f.Rd
#	man/import_data_haridwar.Rd
- add commented lines for foldable sections
- improve indentation
- find better names (gather_cols -> parameters)
- extract functions
- use pipe operator
- use kwb.utils::removeColumns()
Clean some functions, start using yml files for configuration
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.

2 participants