Add adapters of scalar functions to MST and vice versa #150
No reviewers
Labels
No Label
bug
dependencies
discussion
documentation
duplicate
feature
good first issue
misc
performance
question
test
use case
wontfix
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: kscience/kmath#150
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "kotlingrad"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
closes #149
$ ping @breandan
Please add tests and at least some documentation. Also an example in
examples
would be welcome.change to toMst()
I think simple kmath-kotlingrad is appropriate
@ -15,3 +23,4 @@
mavenCentral()
}
group = "kscience.kmath"
I do not like jitpack dependency here as well as other custom dependencies. You'd better move it to specific projects and explicitly list them in README both for the module and for examples.
@ -11,3 +12,3 @@
repositories {
maven("https://dl.bintray.com/mipt-npm/kscience")
jcenter()
See the comment above. I think it makes sense to leave them here for people to see which repositories to use.
@ -0,0 +34,4 @@
@Test
fun simpleFunctionShape() {
val linear = "2*x+16".parseMath().toSFun<KMathNumber<Double, RealField>>()
if (linear !is Sum) fail()
Better to use AssertFalse
@ -0,0 +34,4 @@
@Test
fun simpleFunctionShape() {
val linear = "2*x+16".parseMath().toSFun<KMathNumber<Double, RealField>>()
if (linear !is Sum) fail()
No.
fail
is better since it returns Nothing, so Kotlin DFA makes smart-cast oflinear
toSum