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

Dualsum #50

Merged
merged 42 commits into from
Nov 30, 2023
Merged
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
dc81d44
Try to isolate Breeze
kwalcock Nov 8, 2023
c777301
WIP
kwalcock Nov 8, 2023
6018fdf
Rename to Math, transpose
kwalcock Nov 8, 2023
eab249a
Complete Breeze isolation
kwalcock Nov 9, 2023
97e3d2b
Fix it again
kwalcock Nov 9, 2023
2ab8e60
Try Emjl math
kwalcock Nov 2, 2023
2f91702
Test BreezeMath
kwalcock Nov 9, 2023
f038c40
Unit test Emjl math
kwalcock Nov 9, 2023
92c269e
Organize math better
kwalcock Nov 9, 2023
42b2679
Clean up types
kwalcock Nov 9, 2023
c3851f4
Fix up test app
kwalcock Nov 9, 2023
e50c05d
Test better
kwalcock Nov 9, 2023
b21c0a6
Make column matrix directly
kwalcock Nov 10, 2023
0d8a1ed
Make column matrix directly
kwalcock Nov 10, 2023
8148de7
Remove OnnxMath
kwalcock Nov 10, 2023
fd29280
Rename things
kwalcock Nov 10, 2023
b7c2326
Move the BlasInstanceApp to apps subproject
kwalcock Nov 10, 2023
fb0d70d
Only include necessary ejml parts
kwalcock Nov 10, 2023
cddb890
Add CommonsMath
kwalcock Nov 10, 2023
79084ca
Add some mkRowVector to tests
kwalcock Nov 10, 2023
49ac525
Add some logging
kwalcock Nov 11, 2023
f16b818
use sum instead of concat in dual mode
MihaiSurdeanu Nov 13, 2023
a75b72c
Add CluMath
kwalcock Nov 14, 2023
72d210f
Remove .t from main interface
kwalcock Nov 14, 2023
ff7c5d7
Write some comment on CluMath
kwalcock Nov 14, 2023
f3b5362
Hide math files that aren't used
kwalcock Nov 14, 2023
2721d6e
sum not concat in scala
MihaiSurdeanu Nov 15, 2023
21c0f34
use sum instead of concat everywhere.
MihaiSurdeanu Nov 16, 2023
a41971e
Merge branch 'dualsum' into kwalcock/mathWithDualSum
kwalcock Nov 16, 2023
aa39eb9
Do the sum
kwalcock Nov 16, 2023
c095106
Simplify
kwalcock Nov 16, 2023
21cf682
Remove log
kwalcock Nov 16, 2023
b3d901d
Format
kwalcock Nov 16, 2023
32e07af
deberta full model
MihaiSurdeanu Nov 22, 2023
8bc4471
Merge branch 'dualsum' into kwalcock/mathWithDualSum
kwalcock Nov 29, 2023
f207514
Clean up the apps
kwalcock Nov 29, 2023
635bfa7
Remove breeze and go with ejml
kwalcock Nov 29, 2023
9c7463f
Add documentation
kwalcock Nov 29, 2023
cbc99d7
Shade EJML
kwalcock Nov 29, 2023
e7e417e
Shade the other class as well
kwalcock Nov 30, 2023
70f44f2
Update CHANGES
kwalcock Nov 30, 2023
09218f7
Merge pull request #51 from clulab/kwalcock/mathWithDualSum
kwalcock Nov 30, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Rename things
kwalcock committed Nov 10, 2023
commit fd2928049303a4cf97c0de69a7171b476ec23717
Original file line number Diff line number Diff line change
@@ -10,8 +10,8 @@ object BreezeMath extends Math {
type MathRowVector = Transpose[DenseVector[MathValue]]

def fromResult(result: Result): Array[MathRowMatrix] = {
val array = result.get(0).getValue.asInstanceOf[Array[Array[Array[Float]]]]
val outputs = array.map(mkMatrixFromRows(_))
val arrays = result.get(0).getValue.asInstanceOf[Array[Array[Array[Float]]]]
val outputs = arrays.map(mkMatrixFromRows(_))

outputs
}