-
Notifications
You must be signed in to change notification settings - Fork 5
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
2023 weather data updates #22
Conversation
Janewyx
commented
Mar 27, 2023
- Removed duplicate files from data/ directory (already updated the R/shiny/data folder
- Keeping intermediate files necessary for each year's update in data folder
- Tested all functionalities - did not experience bugs on windrose
- Deleted duplicated files to `R/Shiny/data/`, only keep updated files as intermediate files as a part of each year's data update - All `hourly_****.csv` files must be kept a record of as the they are needed to generate each year's windrose plots (summarised from hourly data) - Need to unzip the zipfile for next year's run
Please accept incoming changes with the data updates in |
Hmm, lot's of data conflicts when I pull this : R/shiny/data/NBCClimateStns_VF.xlsx $ git pull https://github.com/Janewyx/nbcclim.git master
|
Good morning Alex,Thank you for taking a look at my PR -Would you like me to schedule a teams meeting with you, and we could to go over the conflict? I have not handled the csv conflicts before and wonder what the best path forward is..If you have some time, could you let me know your availability this week, please? Thanks!Jane On Mar 27, 2023, at 8:44 AM, Alexandre Bevington ***@***.***> wrote:
Reopened #22.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
We manually removed the files from the app and replaced with the PR code |