NIEHS / PrestoGP

Penalized Regression on Spatiotemporal Outcomes using Gaussian Processes a.k.a. PrestoGP
https://niehs.github.io/PrestoGP/
0 stars 0 forks source link

GitHub Action yaml, badges, and rules #17

Closed kyle-messier closed 7 months ago

kyle-messier commented 8 months ago
kyle-messier commented 8 months ago

Only 3400 Linting warnings! 🙉

kyle-messier commented 7 months ago

@ericbair-sciome @brian-bk22 @sigmafelix @sciome-bot Thoughts on changing the naming conventions. The linting points out that things like "PrestoGP_Model" do not abide by Camel_Case or snake_case. The latter doesn't typically use capitalization. If I change things to "presto_gp_model", then it will abide, but we'll have a lot of places that will need adjustment and likely create a few bugs. Nothing that we can't handle of course. I just wanted to get folks thoughts before I start proposing sweeping changes in pull request #15

kyle-messier commented 7 months ago

@ericbair-sciome I figured out how to exclude some linters such as the object_name. I'll proceed with updating the lints on the package the SET-actions branch, then merge once the lints are passing.