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

Context 1.0 #58

Merged
merged 1 commit into from
Jul 18, 2024
Merged

Context 1.0 #58

merged 1 commit into from
Jul 18, 2024

Conversation

JOUNAIDSoufiane
Copy link
Contributor

Added the following new methods:

  • list_sites(show: [None, “widget”, “text”] = None) -> [str]
  • use_site(site_name: str = DEFAULT_SITE) -> None
  • use_project(project_id: str = None) -> None
  • choose_site() -> None, displays a dropdown widget to choose the site
  • choose_project() -> None, displays a dropdown widget to choose the project
  • check_credentials() -> None, prints authentication metadata
  • set_log_level(debug: Bool), sets openstack debug logging to true, including HTTP request logs
  • _is_ipynb() -> Bool, checks if the code is running within an ipy notebook. Used to determine whether to execute widgets

Copy link
Contributor

@Mark-Powers Mark-Powers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and works well! I have a few minor improvements/thoughts left as comments.

chi/context.py Show resolved Hide resolved
chi/context.py Outdated Show resolved Hide resolved
chi/context.py Outdated Show resolved Hide resolved
chi/context.py Show resolved Hide resolved
@JOUNAIDSoufiane JOUNAIDSoufiane force-pushed the soufianej-context branch 2 times, most recently from 9e8faab to bf43eba Compare July 15, 2024 03:31
@JOUNAIDSoufiane
Copy link
Contributor Author

@Mark-Powers Mark-Powers changed the base branch from master to dev July 15, 2024 15:07
@JOUNAIDSoufiane JOUNAIDSoufiane changed the title Python-chi 1.0 context module Context 1.0 Jul 15, 2024
Added the following new methods:

- list_sites(show: [None, “widget”, “text”] = None) -> [str]
- list_projects(show: [None, “widget”, “text”] = None) -> [str]
- use_site(site_name: str = DEFAULT_SITE) -> None
- use_project(project_id: str = None) -> None
- choose_site() -> None, displays a dropdown widget to choose the site
- choose_project() -> None, displays a dropdown widget to choose the project
- check_credentials() -> None, prints authentication metadata
- set_log_level(debug: str), changes logging level to either ERROR or
  DEBUG, including HTTP request logs if the latter is chosen
@Mark-Powers Mark-Powers merged commit 83d4f0c into dev Jul 18, 2024
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.

2 participants