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

Enhance the MET compilation script and its documentation #1395

Closed
8 of 19 tasks
jprestop opened this issue Jun 26, 2020 · 1 comment · Fixed by #1605, #1796, #1804 or #1930
Closed
8 of 19 tasks

Enhance the MET compilation script and its documentation #1395

jprestop opened this issue Jun 26, 2020 · 1 comment · Fixed by #1605, #1796, #1804 or #1930
Assignees
Labels
component: build process Build process issue component: documentation Documentation issue requestor: METplus Team METplus Development Team type: enhancement Improve something that it is currently doing type: task An actionable item of work
Milestone

Comments

@jprestop
Copy link
Collaborator

jprestop commented Jun 26, 2020

Enhance the compilation script and its documentation.

Describe the Enhancement

  • DONE - Remove --enable-modis from configure when COMPILE_HDF = 0 and COMPILE_HDFEOS=0 (need to account for when using an existing build of HDF and HDFEOS and not compiling)
  • DONE - Create defaults for default BUFRLIB_NAME and GRIB2CLIB_NAME
  • DONE - Changes George suggested in Slack
    ** cp include/*.h ${LIB_DIR}/include/. : line 443 you could remove the . at the end. The copy fails if you already instealled HDFEOS and you try to rerun the script. The 2nd time the files weren't overwritten in $LIB_DIR/include.
    ** Line 667: You could change to: if [ "${SET_D64BIT}" == "TRUE" ]; then; There were no quotes around ${SET_D64BIT} so if that is unset accidentally, it would evaluate to "" and fail the if test. Without it, you get a unary operator error; That could be applied to other checks for environment variables that may not be set by the user
    **CLARIFIED IN DOCUMENTATION UPDATES - in the example install_met_env.kiowa file, I suggest commenting out the 2nd section of variables and change the comment to say uncomment if you have already compiled these. I was being hasty and didn't read the comment, so it skipped all of those and failed.
  • DONE - Add syntax to check if path exists (eliminating extra colons) when adding to, for example, LDFLAGS
    **export PATH=/usr/local/anaconda3/bin${PATH:+:$PATH}
    **export PYTHONPATH=${PYTHONPATH:+$PYTHONPATH:}${HOME}/git/ConfigMaster
  • DONE - Add LIBPNG_LIB to LDFLAGS (and potentially other libs) (this fixed Logan's MRMS problem on WCOSS)
  • DONE - Add option for --bindir call to MET
  • DONE - Improve documentation for script based on changes above which will simplify a recalling of the script
  • CLARIFIED IN DOCUMENTATION - Add comments for each var to the install_env_.conf files or just to the generic file
  • DONE - "python --version" should really be "${MET_PYTHON}/bin/python --version"
  • DONE - Add check for GNU version 10. If 10 or greater, have ‘-fallow-argument-mismatch' in BUFRLIB compilation. If less than 10, have ‘-Wno-argument-mismatch'' in BUFRLIB compilation.

Time Estimate

Estimate the amount of work required here.
1 - 3 days of work.

Sub-Issues

Consider breaking the enhancement down into sub-issues.

  • Add a checkbox for each sub-issue here.

Relevant Deadlines

List relevant project deadlines here or state NONE.

Funding Source

Split 2771021 AF and 2700042 WPC

Define the Metadata

Assignee

  • Select engineer(s) or no engineer required (Julie Prestopnik)
  • Select scientist(s) or no scientist required

Labels

  • Select component(s)
  • Select priority
  • Select requestor(s)

Projects and Milestone

  • Review projects and select relevant Repository and Organization ones
  • Select milestone

Define Related Issue(s)

Consider the impact to the other METplus components.

Enhancement Checklist

See the METplus Workflow for details.

  • Complete the issue definition above.
  • Fork this repository or create a branch of develop.
    Branch name: feature_<Issue Number>_<Description>
  • Complete the development and test your changes.
  • Add/update unit tests.
  • Add/update documentation.
  • Push local changes to GitHub.
  • Submit a pull request to merge into develop.
    Pull request: feature <Issue Number> <Description>
  • Iterate until the reviewer(s) accept and merge your changes.
  • Delete your fork or branch.
  • Close this issue.
@jprestop jprestop added type: enhancement Improve something that it is currently doing component: documentation Documentation issue component: build process Build process issue type: task An actionable item of work priority: high requestor: NCAR/RAL NCAR Research Applications Laboratory labels Jun 26, 2020
@jprestop jprestop added this to the MET 10.0 milestone Jun 26, 2020
@jprestop jprestop self-assigned this Jun 26, 2020
@JohnHalleyGotway JohnHalleyGotway added the alert: NEED CYCLE ASSIGNMENT Need to assign to a release development cycle label Sep 10, 2020
@jprestop
Copy link
Collaborator Author

Added ability for MET_GSL to be read in and have GSL compiled if MET_GSL is not set.

@jprestop jprestop removed the alert: NEED CYCLE ASSIGNMENT Need to assign to a release development cycle label Sep 18, 2020
@jprestop jprestop linked a pull request Dec 10, 2020 that will close this issue
8 tasks
@jprestop jprestop modified the milestones: MET 10.0.0, MET 10.1.0 Feb 18, 2021
@jprestop jprestop linked a pull request May 19, 2021 that will close this issue
12 tasks
@jprestop jprestop linked a pull request May 20, 2021 that will close this issue
12 tasks
@jprestop jprestop reopened this May 24, 2021
@TaraJensen TaraJensen added the alert: NEED ACCOUNT KEY Need to assign an account key to this issue label Sep 13, 2021
@TaraJensen TaraJensen removed the alert: NEED ACCOUNT KEY Need to assign an account key to this issue label Sep 23, 2021
jprestop pushed a commit that referenced this issue Sep 30, 2021
@jprestop jprestop linked a pull request Oct 2, 2021 that will close this issue
12 tasks
JohnHalleyGotway added a commit that referenced this issue Oct 2, 2021
jprestop pushed a commit that referenced this issue Oct 4, 2021
…stent indentation and modified to use two spaces throughout to keep it consistent and easier to read.
jprestop added a commit that referenced this issue Oct 4, 2021
* Per #1395, add fix for gcc10 compilation and work on other enhancements

* Per #1395, cleaned up commented code and set comilation of HDF(4), HDFEOS, FREETYPE, and CAIRO to 0

* Per #1395, updated documentation within script

* Per #1395, ran sed to remove lots of trailing whitespace that was causing odd wrapping of lines.

* Per #1395, removed mixture of tabs and spaces that results in inconsistent indentation and modified to use two spaces throughout to keep it consistent and easier to read.

Co-authored-by: Julie Prestopnik <jpresto@seneca.rap.ucar.edu>
Co-authored-by: John Halley Gotway <johnhg@ucar.edu>
@jprestop jprestop closed this as completed Oct 4, 2021
@jprestop jprestop added requestor: METplus Team METplus Development Team and removed requestor: NCAR/RAL NCAR Research Applications Laboratory labels Oct 4, 2021
@JohnHalleyGotway JohnHalleyGotway changed the title Enhance compilation script and its documentation Enhance the MET compilation script and its documentation Mar 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: build process Build process issue component: documentation Documentation issue requestor: METplus Team METplus Development Team type: enhancement Improve something that it is currently doing type: task An actionable item of work
Projects
None yet
3 participants