KitwareMedical / KitwareMedical/GirderMedViewer

Use a comprehensive name for configs

Open
#34 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
5
Forks
5
PR merge metrics
No merged PRs in 30d

Description

Identifying a Girder server by its url is not great. It would be great to have the URL be a setting instead of the title section. Potentially, if the url is missing, it would use the section title

e.g.
```
[Data @ kitware]
url = https://data.kitware.com
api_root = api/v1
default_location = {'_id': '5b22b7198d777f2e6225658c', '_modelType': 'folder'}

[https://data.kitware.com]
api_root = api/v1
default_location = {'_id': '5b22b7198d777f2e6225658c', '_modelType': 'folder'}

[https://data.kitware.com]
url = https://data.kitware.com/a/longer/path
api_root = api/v1
default_location = {'_id': '5b22b7198d777f2e6225658c', '_modelType': 'folder'}
```

Also:
Support URLs with ending '/'

Contributor guide

Open the contributing guide

Research direction

No files, tests, or entry points are named in the issue. Start by locating the configuration parsing and Girder URL handling, then compare the three example section formats and trailing-slash behavior; done means the URL setting, title fallback, longer paths, and ending '/' are handled consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.