Skip to content

Conversation

@IgorTatarnikov
Copy link
Member

Currently, http://cng.gmu.edu:8080/api/health returns a 503 when queried. Interestingly, if you query the API regardless it returns valid output e.g. http://cng.gmu.edu:8080/api/neuron/id/1.

This PR adds a fallback URL for querying API health if http://cng.gmu.edu:8080/api/health doesn't return 200. In this case self._base_url is swapped for http://neuromorpho.org/api/neuron instead of http://cng.gmu.edu:8080/api/neuron.

@codecov
Copy link

codecov bot commented May 12, 2025

Codecov Report

Attention: Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.

Project coverage is 72.37%. Comparing base (bb21a0c) to head (f26038f).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
morphapi/api/neuromorphorg.py 77.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
+ Coverage   72.14%   72.37%   +0.23%     
==========================================
  Files          10       10              
  Lines         718      724       +6     
==========================================
+ Hits          518      524       +6     
  Misses        200      200              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@IgorTatarnikov IgorTatarnikov requested a review from a team May 12, 2025 10:09
@adamltyson adamltyson merged commit 0d5c593 into main May 12, 2025
22 of 23 checks passed
@adamltyson adamltyson deleted the update-URL branch May 12, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants