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

Expose complexity checks without annotations #138

Merged
merged 11 commits into from
Dec 6, 2021

Conversation

chrispyles
Copy link
Contributor

@chrispyles chrispyles commented Dec 6, 2021

Closes #137

@codecov-commenter
Copy link

codecov-commenter commented Dec 6, 2021

Codecov Report

Merging #138 (3c77b63) into main (489f821) will increase coverage by 0.18%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
+ Coverage   95.10%   95.29%   +0.18%     
==========================================
  Files          29       30       +1     
  Lines        1655     1720      +65     
==========================================
+ Hits         1574     1639      +65     
  Misses         81       81              
Impacted Files Coverage Δ
pybryt/execution/__init__.py 100.00% <ø> (ø)
pybryt/execution/complexity.py 100.00% <ø> (ø)
pybryt/__init__.py 100.00% <100.00%> (ø)
pybryt/complexity.py 100.00% <100.00%> (ø)
pybryt/execution/tracing.py 100.00% <100.00%> (ø)
pybryt/student.py 95.59% <100.00%> (-0.17%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 489f821...3c77b63. Read the comment docs.

Copy link
Contributor

@leestott leestott left a comment

Choose a reason for hiding this comment

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

Complexity Analysis

PyBryt exposes its code complexity analysis tools so that they can be used to analyze blocks of code without necessitating the use of the annotation framework. These tools can be used to give students a way to test the complexity of their implementations, or just to analyze code complexity in a manner that is more reliable than using a timer.

@leestott leestott merged commit af7b80b into microsoft:main Dec 6, 2021
@chrispyles chrispyles deleted the complexity branch January 6, 2022 21:41
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.

Allow users to use time complexity analysis tools without annotations
3 participants