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

Update docstrings #18

Merged
merged 5 commits into from
Jul 26, 2023
Merged

Update docstrings #18

merged 5 commits into from
Jul 26, 2023

Conversation

kyosek
Copy link
Owner

@kyosek kyosek commented Jul 25, 2023

Reference issue

What does this implement/fix?

Additional information

@kyosek kyosek self-assigned this Jul 25, 2023
@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #18 (445af8c) into master (5cec6f0) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 445af8c differs from pull request most recent head c92953b. Consider uploading reports for the commit c92953b to get more accurate results

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   93.12%   93.12%           
=======================================
  Files           5        5           
  Lines         262      262           
=======================================
  Hits          244      244           
  Misses         18       18           
Files Changed Coverage Δ
cfepy/focus/core.py 89.67% <ø> (ø)
cfepy/__init__.py 100.00% <100.00%> (ø)
cfepy/focus/utils.py 94.28% <100.00%> (ø)
cfepy/tests/test_focus.py 100.00% <100.00%> (ø)
cfepy/tests/test_utils.py 100.00% <100.00%> (ø)

@github-actions
Copy link

github-actions bot commented Jul 25, 2023

🤖 OpenAI


Chat with 🤖 OpenAI Bot (@openai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @openai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Ignoring further reviews

  • Type @openai: ignore anywhere in the PR description to ignore further reviews from the bot.

Files ignored due to filter (1)

Ignored files

  • environment.yml
Files not summarized due to errors (12)

Failed to summarize

  • README.rst (nothing obtained from openai)
  • cfxplorer/init.py (nothing obtained from openai)
  • cfxplorer/tests/test_utils.py (nothing obtained from openai)
  • cfxplorer/tests/test_focus.py (nothing obtained from openai)
  • cfxplorer/focus/core.py (nothing obtained from openai)
  • cfxplorer/focus/utils.py (nothing obtained from openai)
  • cfxplorer/version.py (nothing obtained from openai)
  • docs/conf.py (nothing obtained from openai)
  • examples/hyperparameter_tuning_example.py (nothing obtained from openai)
  • examples/focus_example.py (nothing obtained from openai)
  • examples/utils.py (nothing obtained from openai)
  • setup.py (nothing obtained from openai)

In the recent run, only the files that changed from the base of the PR and between 445af8c72a21a8e76fbf18b0e81bea9aa1d79bc7 and c92953ba7f4f6b621ad667e2d8b2a3afb16f2ca5 commits were reviewed.

Files not reviewed due to errors in the recent run (12)

Failed to review in the last run

  • cfxplorer/init.py (no response)
  • cfxplorer/focus/utils.py (no response)
  • cfxplorer/tests/test_utils.py (no response)
  • README.rst (no response)
  • cfxplorer/focus/core.py (no response)
  • cfxplorer/tests/test_focus.py (no response)
  • docs/conf.py (no response)
  • cfxplorer/version.py (no response)
  • examples/hyperparameter_tuning_example.py (no response)
  • examples/utils.py (no response)
  • examples/focus_example.py (no response)
  • setup.py (no response)

@kyosek kyosek merged commit 6c77145 into master Jul 26, 2023
9 checks passed
@kyosek kyosek deleted the readme branch July 28, 2023 07:59
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.

None yet

1 participant