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

Compatible-features inspired overhaul of FA #65

Merged
merged 13 commits into from
Sep 15, 2019
Merged

Conversation

tspooner
Copy link
Owner

@tspooner tspooner commented Sep 3, 2019

This PR includes a ridiculous amount of changes and will take too long to explain in detail. It's taken a long time to get here but the framework is much more usable now. There are still a lot of things that need refinement (such as the gradients module in fa) but on the whole it's a major improvement.

@tspooner
Copy link
Owner Author

tspooner commented Sep 3, 2019

This addresses #45

@tspooner
Copy link
Owner Author

tspooner commented Sep 3, 2019

And #41

This change makes things a little simpler and justifies the methods a
little more (as far as I'm concerned). In future we will want to improve
the linalg code and probably just use components from external crates.
The problem we have is that most existing frameworks don't support
simple versions sparse features or gradient matrices.

My view is to leave it like this for now (i.e. messy AF) and then
rewrite it all to make liberal use of GATs when they are stable. This
would allow us to hide _all_ the underlying code and just implement
lightweight interfaces as with the Entry approach taken in the std
library.
@tspooner tspooner merged commit 7ee9a85 into master Sep 15, 2019
@tspooner tspooner deleted the ft-compatible_features branch September 15, 2019 16:11
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.

1 participant