bigdata-ustc / EduKTM

The Model Zoo of Knowledge Tracing Models
Apache License 2.0
197 stars 63 forks source link

[Refactor]Split KPT into KPT and EKPT #13

Closed Ljyustc closed 3 years ago

Ljyustc commented 3 years ago

Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.

Description

Split the original KPT into KPT and EKPT.

What does this implement/fix? Explain your changes.

Split the original KPT into KPT and EKPT.

Pull request type

Changes

Split the original KPT into KPT and EKPT.

Does this close any currently open issues?

N/A

Any relevant logs, error output, etc?

N/A

Checklist

Before you submit a pull request, please make sure you have to following:

Essentials

Comments

codecov-commenter commented 3 years ago

Codecov Report

Merging #13 (7e64642) into main (a0502ab) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              main       #13    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           21        23     +2     
  Lines          870       976   +106     
==========================================
+ Hits           870       976   +106     
Impacted Files Coverage Δ
EduKTM/EKPT/EKPT.py 100.00% <100.00%> (ø)
EduKTM/EKPT/__init__.py 100.00% <100.00%> (ø)
EduKTM/KPT/KPT.py 100.00% <100.00%> (ø)
EduKTM/__init__.py 100.00% <100.00%> (ø)

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 a0502ab...7e64642. Read the comment docs.