Beta
Data Correlation
10crouzet
Loading description...
Algorithms
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
This is really YOLO:
If you do stuff like this, I expect a clear specification of a "correct" log entry.
Thx for the feedback :)
i've changed it to: "- log lines without request id, variant A or B declaration or price must be ignored"
(and reordered the list)
Reference solution used for the random tests is incorrect: it only detects the first matching entry from each line.
not sure to understand. there is always 1 match per line
There are lines with multiple entries inside a line.
oh, right! i fix that. thx
done
It's unclear what the "global increase rate" is and how
12.93is derived, or even what it means (is it a percentage? How is it calculated?).thx, i update the instructions asap
Done
Random tests should be added.
i add that thx :)
i hope you liked the kata
fixed, thx to update your solution :)