Skip to content

Commit

Permalink
temp fix- change repo name in DESCRIPTION
Browse files Browse the repository at this point in the history
hardcode new name for repo, for now, so ghpages pkgdown site links will work
  • Loading branch information
ejanalysis committed Feb 20, 2025
1 parent 3d774e6 commit 3337849
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
10 changes: 6 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,9 @@

## Note on "EJAM" versus "EJAM-open" as the repository name

Please note that as of early 2025, changes are still being made to exactly where the open source project will be shared, documented, and available for contributions (such as submitting Issues or Pull Requests on GitHub). In the short term, it is important to note that the "EJAM" package is shared at the "EJAM-open" repository even though the documentation website may still refer to and try to link to a different, nonpublic repository called "EJAM" which may cause some confusion. You can still clone and work with the EJAM-open repository and just avoid using any weblinks pointing to USEPA/EJAM rather than USEPA/EJAM-open. The open source EJAM package was initially made available via the [EJAM-open repository](https://github.com/USEPA/EJAM-open), but the [EJAM package documentation](https://usepa.github.io/EJAM/index.html) was initially published from a non-public development repository, so that documentation's links to source code did not work (because they required access to a separate, non-public repository).
Please note that as of early 2025, changes are still being made to exactly where the open source project will be shared, documented, and available for contributions (such as submitting Issues or Pull Requests on GitHub).

The open source EJAM package was initially made available via the [EJAM-open repository](https://github.com/USEPA/EJAM-open), but an early version of the EJAM package documentation was initially published from USEPA/EJAM, so the documentation's links to source code did not work (because they required access to a separate, non-public repository). That has been corrected so that documentation is associated with the EJAM-open repository.

## Welcome!

Expand All @@ -12,9 +14,9 @@ The worst that can happen is we’ll politely ask you to change something.
We appreciate all friendly contributions.

We encourage you to read about this R package and project, and see the project’s
[CONTRIBUTING](https://usepa.github.io/EJAM/CONTRIBUTING.html) policy (this page),
[LICENSE](https://usepa.github.io/EJAM/LICENSE.html), and
[README](https://usepa.github.io/EJAM/index.html).
[CONTRIBUTING](https://usepa.github.io/EJAM-open/CONTRIBUTING.html) policy (this page),
[LICENSE](https://usepa.github.io/EJAM-open/LICENSE.html), and
[README](https://usepa.github.io/EJAM-open/index.html).

All contributions to this project will be released under the MIT dedication
or the CCO-1.0 license file dedication.
Expand Down
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ Description: Web app and R tools to easily summarize
type of facility, shapefile, or Census areas.
Very fast identification of nearby blocks and distances, and
aggregation of indicators within each distance by site and overall.
URL: https://usepa.github.io/EJAM, https://github.com/USEPA/EJAM
URL: https://usepa.github.io/EJAM-open, https://github.com/USEPA/EJAM-open
Depends:
R (>= 3.5)
Imports:
Expand Down
2 changes: 1 addition & 1 deletion R/doaggregate.R
Original file line number Diff line number Diff line change
Expand Up @@ -767,7 +767,7 @@ doaggregate <- function(sites2blocks, sites2states_or_latlon=NA,
results_overall <- cbind(results_overall, results_overall_wtdmeans) # many columns (the popwtd mean cols)
}
############################################### #
## >>> TEMPORARY PATCH UNTIL FORMULA FIXED - SEE ISSUE #498 https://github.com/USEPA/EJAM/issues/498 ####

if (!showdrinkingwater) {
print(results_overall[ , .( drinking)])
print(results_bysite[ , .(ejam_uniq_id, drinking)])
Expand Down

0 comments on commit 3337849

Please sign in to comment.