Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #66 +/- ##
=======================================
Coverage 97.70% 97.70%
=======================================
Files 62 62
Lines 2045 2045
=======================================
Hits 1998 1998
Misses 47 47 ☔ View full report in Codecov by Sentry. |
|
@jnytra If you can, would you mind looking through this tutorial/example and letting me know if anything is unclear or doesn't make sense? |
|
This is absolutely amazing. This is exactly what I needed to write a new adaptor. I don't have any comments yet, but I plan to write a new adaptor according to this tutorial, so I'll see. |
@jnytra Sounds good! If you're planning to add any new adaptors or algorithms please file an issue first (I'll mark it as an enhancement and assign you to "fix" it) so that we don't end up working on the same thing :) |
| * convenient. | ||
| */ | ||
|
|
||
| struct flux_sequence_traits { |
There was a problem hiding this comment.
Change to struct flux_sequence_traits : flux::default_sequence_traits {
|
This tutorial is great, it's exactly what I needed to get started. One issue I ran into is that struct flux_sequence_traits {to struct flux_sequence_traits : flux::default_sequence_traits {resolves the issue. I can't add a commit to this pr, but I didn't think that this change warranted a completely new pull request. Thanks again for the tutorial! |
No description provided.