Conversation
Draft Radix impl and tests with docs and commentary on areas of future development.
Haven't quite got the algorithm properly defined yet. Need to revisit data flow, but also need to sign off tonight. Till the next session.
Implementation left in progress to pursue priority venture
Just a lil somethin to work with collections, maybe it'll work lol
Also len() to .Len()
Behavior fails test, but logically is almost correct
Several changes are logged below
Removed the collectable type, now just interface{}
Unexported help function
Correct Name of FilterFunc in Select Header
Change Pointer to actual reference
Move interface type, Collection, to parameters for ImmutableMapReduce()
All exported functions now have documentation comment
Add runtime type check before cast within switch block, twice
Change incorrect `type` keyword to `func` where necessary
First test passes!
Index was being recorded instead of expected string values. Oops
Build some initial types and work in an example Write up idea for range extender that may be relevant
I am stuck because of old code. Some kind person in the past already deleted it for me.
To test pkg.sort in files elsewhere
Good start, the test runs and the code fails. Now we iterate till green.
Behavior mostly as expected, but does not solve the right problem. Ranges were not extended. Better test needed. Fixes: panic: assignment to entry in nil map panic: runtime error: invalid memory address or nil pointer dereference [recovered] panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x110b8c7]
This will demonstrate the strategy used in the range extension method. Since the simpleSync should not be exported We create an example instead to run tests on top of.
simpleSync is a struct referenced to increment a specific instance of integer a number of times at a Later point in execution. It was not incrementing because the method call generated the nil version of the struct each time. The fix was simply to change the struct references to pointers all around. Now the correct struct is used during runtime
Leave some additional ideas commented or unused. Now we run tests and modify
Results showed that order is not preserved in this version This commit adds some initial debugging attempts at the time of rediscovery.
Tests initiated to test construction
Item by item level decision for tosses WorryLevel now reflected in each item, rather than across all items Constructors for and usage of Operations and Decisions refelct the above Toss Items one at a time to different receivers, rather than as a block to one
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.