Improvements for mercury driver

[SilasM2001]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 43 lines in your changes are missing coverage. Please review.

Project coverage is 31.48%. Comparing base (92d4147) to head (4480251).
Report is 49 commits behind head on master.

Files	Patch %	Lines
basil/HL/mercury.py	0.00%	43 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
- Coverage   32.67%   31.48%   -1.20%     
==========================================
  Files          85       87       +2     
  Lines        5800     6026     +226     
==========================================
+ Hits         1895     1897       +2     
- Misses       3905     4129     +224     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[YannickDieter]

Please rebase to check if pipeline is now working.

[YannickDieter]

Looks good to me, except the minor comments. In general it would also be good to change to str.format.

[YannickDieter]

Thanks for the update. Actually we still should replace the prints with proper loggers like it is done for example here

[cbespin]

Looks good to me now, thanks! Great addition to basil

[cbespin]

Since quite some lines are added that are not (can not?) be covered by tests, the coverage decreases, but we should ignore that.

[YannickDieter]

Yeah, this was also my thought. So lets merge ;)