Command loggers use the base name and not the full name

[amikhalev]

@PlasmaPower @Beta-Ultra

[PlasmaPower]

No?

CommandBase::CommandBase(const std::string& name) :
Command(name.c_str()), log(name.c_str()) {

[amikhalev]

DriveContinuous(YAML::Node config) : CommandBase(GetBaseName()) {
}

[PlasmaPower]

Hmm, should Commands be passed their name? That would be the only way to do this.

[amikhalev]

They are, you could do

GetBaseName() + config["name"].as<std::string>()

but that is gross

[PlasmaPower]

@amikhalev You would have to have a shorthand if to account for commands which only have one instance. We aren't doing that.

[amikhalev]

You could just have a method on CommandBase and Logger to set the logger name. That would be a Nick thing though

[amikhalev]

@PlasmaPower @Beta-Ultra Bump