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

Data import/export updates #7

Open
wants to merge 69 commits into
base: main
Choose a base branch
from
Open

Data import/export updates #7

wants to merge 69 commits into from

Conversation

wright13
Copy link
Collaborator

No description provided.

Irene Foster and others added 24 commits November 13, 2023 11:29
adding photo and seedling QC functions
94fe117
adding pine tree QC function
3e67ae8
update filter logic and adding explicit function calls
9626a2d
adding event data QC function
55b03b8
addng first two exploratory pine data viz
502b713
adding visualizations for individual sites
f5f9744
adding general pine visualizations
94346ad
improvements and fixes to data import/export
05505d3
Clean up visualizations
e952112
reducing number of bins for height and DBH graphs
0002ac6
adding beginning of ojs
4022917
Changing graph titles
3d55e07
adding basal area and visit number calculations
c4001b7
adding ecological measures functions and graphs
6e321f6
adding relative frequency and relative dominance functions and graphs
4e90b4a
adding importance value and dead tree graphs
266fb7e
cleaning up function code
d0e2c07
adding vitality, mistle toe, and cone count graphs
071db49
adding canker and pine beetle graphs
3e4f75f
moving data cleaning to function
c5e738d
Merge if-dev2
bb41ebe 
Merge branch 'if-dev2' into sarah-dev

# Conflicts:
#	DESCRIPTION
Add magrittr dependency, add documentation, comment out scratchwork
460c8ec
Tweak qmd files to work with db/fetchaccess updates
2bc71ee 
DB path is set to relative path assuming FNP_MOJN_Primary.accdb is in root project folder
Update gitignore to prevent tracking of qmd outputs
1307744
@wright13 wright13 requested a review from i-m-foster December 18, 2023 23:45
i-m-foster
i-m-foster approved these changes Dec 19, 2023
View reviewed changes
Copy link
Collaborator

@i-m-foster i-m-foster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! loadPine() worked for me but writePine() and filterPine() didn't

Irene Foster and others added 30 commits January 5, 2024 13:16
modifying graph aesthetics
ad55e68
creating fake data for unit tests
8adcea7
adding script to make fake data for unit tests
e63d2b2
update unit tests to include checking the number of rows in the fake …
538d4ee 
…data
generalizing paths and cleaning up code
d8691b8
moving visualization files and deleting old man files
80b4775
moving files around and adding more unit tests
3cbf1f7
deleting report template
5a750e1
adding explicit function call
adde3f4
progress on data pkg/EML creation
62ef0fb
refactor tests
7282605
Only read empty fields in CSV as NA
66895cd
Resolve merge conflict from if-dev2
ef46bb6 
Merge branch 'if-dev2' into sarah-dev

# Conflicts:
#	DESCRIPTION
#	R/calculations.R
#	scripts/visualizations/pine_exploratory_viz.qmd
Generate documentation files
d25be45
switching absolute paths to relative paths
0ac69b5
adding visualization scripts for individual networks
2cbedfb
deleting visualization first draft
b9e1522
adding function documentation
0fff613
fix merge conflict
5c96d3b
add summary tables to KLMN visualization file
0c1a1f7
update description w/roxygen version
e0f0351
start adding lat/long to site table (not functional)
3eea828
Merge branch 'if-dev2' into sarah-dev
211c8fa 
merge updates from if-dev2
Document return value for filterPine
e8c5257
adding function to write QC results to excel file
2bd4fe3
merge Irene's updates
cd7b6b3 
merge Irene's updates

# Conflicts:
#	R/utils.R
update data pkg creation
b672982
deleting utm conversion that was added to loadPne function
0da2b86
adding README
d27a8c7
@wright13
Fix R cmd check issues
9f47f70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@i-m-foster i-m-foster i-m-foster approved these changes

Assignees

@i-m-foster i-m-foster

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wright13 @i-m-foster