aburrell / apexpy

A Python wrapper for Apex coordinates
MIT License
34 stars 25 forks source link

DOC: Updates to installation documentation #102

Closed ljlamarche closed 1 year ago

ljlamarche commented 1 year ago

Description

This PR makes some updates and clarifications to the apexpy installation instructions, specifically troubleshooting unusual cases. Addresses https://github.com/aburrell/apexpy/issues/101 by specifying apexpy has not been tested with versions of gfortran below v7.

Type of change

How Has This Been Tested?

Compiled sphinx documentation locally.

Test Configuration:

Checklist:

ljlamarche commented 1 year ago

Is numpy still required to be pre-installed with the new build system?

aburrell commented 1 year ago

I think not. I believe it will grab it, and just check if you have it locally.

From: Leslie Lamarche @.> Reply-To: aburrell/apexpy @.> Date: Thursday, December 1, 2022 at 12:42 To: aburrell/apexpy @.> Cc: Subscribed @.> Subject: Re: [aburrell/apexpy] DOC: Updates to installation documentation (PR #102)

Is numpy still required to be pre-installed with the new build system?

— Reply to this email directly, view it on GitHubhttps://github.com/aburrell/apexpy/pull/102#issuecomment-1334125844, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABVYF7XQLX677HLMCUPSZ3DWLDPP7ANCNFSM6AAAAAASRBMAGU. You are receiving this because you are subscribed to this thread.Message ID: @.***>

aburrell commented 1 year ago

The test failure issues are fixed in #98

ljlamarche commented 1 year ago

Does apexpy require just gfortran or also gcc? Or do you only need gcc if you're building it locally?

aburrell commented 1 year ago

Does apexpy require just gfortran or also gcc? Or do you only need gcc if you're building it locally?

You need gcc and gfortran if you're installing using anything other than a wheel.

ljlamarche commented 1 year ago

@aburrell , can you read through this version of the installation page? It's supposed to be geared towards what the installation process WILL be in v2.0.0, but I wasn't sure about some of it.

ljlamarche commented 1 year ago

Awesome, I accepted your suggestions! Feel free to merge when convenient!