You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This will simplify and fortify the code base. Functions like vis_drake_graph() take a bunch of arguments and then construct a drake_config() list if one is not already supplied. This generates a lot of confusion, repeat code, and room for error. Only make() and drake_config() should take arguments like parallelism and jobs and verbose in the usual way. The others should be required to get them from a drake_config() list.
The text was updated successfully, but these errors were encountered:
This will simplify and fortify the code base. Functions like
vis_drake_graph()
take a bunch of arguments and then construct adrake_config()
list if one is not already supplied. This generates a lot of confusion, repeat code, and room for error. Onlymake()
anddrake_config()
should take arguments likeparallelism
andjobs
andverbose
in the usual way. The others should be required to get them from adrake_config()
list.The text was updated successfully, but these errors were encountered: