kscience/kmath
9
0
Fork 1
Code Issues Pull Requests 6 Packages Projects Releases 16 Wiki Activity

KMP library for tensors #300

Merged
grinisrit merged 215 commits from feature/tensor-algebra into dev 2021-05-08 09:48:04 +03:00
Conversation 1 Commits 215 Files Changed 40 +4062 -17
grinisrit commented 2021-04-30 11:30:53 +03:00 (Migrated from github.com)
Copy Link

Closes #178

Closes #178
altavir commented 2021-04-30 11:36:31 +03:00 (Migrated from github.com)
Copy Link

Please fix tests and ensure that CI build completes.

Please fix tests and ensure that CI build completes.
CommanderTvis (Migrated from github.com) requested changes 2021-04-30 12:43:58 +03:00
AndreiKingsley (Migrated from github.com) reviewed 2021-04-30 12:55:27 +03:00
AndreiKingsley (Migrated from github.com) reviewed 2021-04-30 12:55:54 +03:00
CommanderTvis (Migrated from github.com) reviewed 2021-04-30 12:58:51 +03:00
altavir (Migrated from github.com) requested changes 2021-04-30 14:32:06 +03:00
altavir (Migrated from github.com) left a comment
Copy Link

All unnecessary inlines must be removed.
Some classes dublicate existing functionality.

Otherwise, awesome!

All unnecessary inlines must be removed. Some classes dublicate existing functionality. Otherwise, awesome!
grinisrit (Migrated from github.com) reviewed 2021-04-30 17:10:33 +03:00
grinisrit (Migrated from github.com) reviewed 2021-04-30 17:12:32 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-04-30 21:41:44 +03:00
grinisrit (Migrated from github.com) reviewed 2021-04-30 21:54:03 +03:00
CommanderTvis (Migrated from github.com) reviewed 2021-05-01 12:03:23 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-05-01 12:20:58 +03:00
CommanderTvis (Migrated from github.com) reviewed 2021-05-01 14:49:53 +03:00
CommanderTvis (Migrated from github.com) reviewed 2021-05-01 14:51:16 +03:00
CommanderTvis (Migrated from github.com) reviewed 2021-05-01 14:51:46 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-05-01 19:18:58 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-05 21:03:59 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-05 21:10:46 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-05-06 19:28:05 +03:00
altavir (Migrated from github.com) requested changes 2021-05-07 10:09:19 +03:00
altavir (Migrated from github.com) left a comment
Copy Link

The major concern is that I don't understand reasoning behind the hierarchy of implementation of algebras. Probably better to have one singl implmentation that inherits all interfaces.

The major concern is that I don't understand reasoning behind the hierarchy of implementation of algebras. Probably better to have one singl implmentation that inherits all interfaces.
grinisrit (Migrated from github.com) reviewed 2021-05-07 10:27:25 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 10:31:16 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 10:31:52 +03:00
altavir (Migrated from github.com) reviewed 2021-05-07 11:28:37 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:20:55 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:21:24 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:26:32 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:28:34 +03:00
altavir (Migrated from github.com) reviewed 2021-05-07 13:38:19 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:41:49 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 13:43:26 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 14:37:47 +03:00
grinisrit (Migrated from github.com) reviewed 2021-05-07 14:41:11 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-05-07 15:43:43 +03:00
CommanderTvis (Migrated from github.com) requested changes 2021-05-07 17:42:05 +03:00
CommanderTvis (Migrated from github.com) approved these changes 2021-05-07 23:35:15 +03:00
altavir (Migrated from github.com) approved these changes 2021-05-08 09:47:54 +03:00
Sign in to join this conversation.
Reviewers
No reviewers
CommanderTvis
altavir
Labels
Clear labels   
bug

Something isn't working

  
dependencies

Pull requests that update a dependency file

  
discussion

   
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
feature

A new feature request

  
good first issue

The issue awaits its hero. Contributions are welcome

  
misc

Infrastructure tasks, cosmetic changes and miscellaneous tasks

  
performance

Performance optimization issue

  
question

Further information is requested

  
test

Add tests for some functionality

  
use case

A use case for library or feature

  
wontfix

This will not be worked on

No Label
bug
dependencies
discussion
documentation
duplicate
feature
good first issue
misc
performance
question
test
use case
wontfix
Milestone
Clear milestone
No items
No Milestone
0.3
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: kscience/kmath#300
No description provided.
Delete Branch "feature/tensor-algebra"

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?