George Tay

Results 24 comments of George Tay

@gok99 After some preliminary testing and profiling, it seems that the refactoring does not have much of an impact on the overall performance of the code. The increases or decreases...

@sopa301 and @gok99, I have implemented the suggestion to consolidate all commonly used Regex patterns into the `StringsUtil` class. Please take a look and let me know where I can...

@gok99 Sure thing, will look through the codebase to find other places with repeated regex patterns!

@gok99 I have provided a proof of concept for the `clone` feature in the draft PR linked to this issue, for your reference. I agree that allowing deep cloning would...

@gok99 @chan-j-d Apologies for the late reply, but thank you for your feedback and comments! I agree with @gok99 in that this may be rather troublesome to maintain, and perhaps...

@gok99 Perhaps we could first take a look at how we can make our model classes immutable before we take a look at the other classes. I believe the model...

@sopa301 Let me work on that and update you in the upcoming days!

@sopa301 I have implemented new test cases for BlurbMap, please do let me know if there are any other aspects that are lacking in tests!

@gok99 @ckcherry23 I have added the new changes as requested!

@gok99 @damithc The feature is still being worked on at the moment (I have not finalised the precedence to read in config files/CLI arguments, and subsequently have not updated tests...