Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@portaloffreedom
Copy link
Contributor

No description provided.

@portaloffreedom portaloffreedom changed the base branch from master to development December 4, 2019 17:12
Copy link

@DaanZ DaanZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am a bit worried that some of the changes will affect the way we do experiments in the future, and I am not sure how much of these changes will actually be reusable.

: next_state(nullptr)
const DifferentialCPG::ControllerParams &params,
const std::vector<std::shared_ptr<Actuator>> &actuators,
std::shared_ptr<AngleToTargetDetector> angle_to_target_sensor)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does every future version need to specify these parameters?

const NEAT::Genome &gen)
: next_state(nullptr)
const NEAT::Genome &gen,
std::shared_ptr<AngleToTargetDetector> angle_to_target_sensor)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary for future experiments?


double DifferentialCPG::output_function(double input) const
{
return this->output_signal_factor
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split up formula for better readability.

message(WARNING "Using local Gazebo @ ${gazebo_DIR}")
else()
find_package(gazebo 9 REQUIRED)
find_package(gazebo 10 REQUIRED)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this change generally needed?

const std::vector<std::shared_ptr<Actuator>> &_actuators);
const DifferentialCPG::ControllerParams &params,
const std::vector<std::shared_ptr<Actuator>> &_actuators,
std::shared_ptr<AngleToTargetDetector> angle_to_target_sensor = nullptr);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed for future experiments?

this->previous_position_.Reset();
this->start_position_.Reset();
this->locomotion_type = "directed"; // {directed, gait}
this->locomotion_type = "turing_left"; // {directed, gait}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the new standard approach?


/////////////////////////////////////////////////
void Evaluator::Reset()
void Evaluator::reset()
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No more camel case for functions?

std::cout << "Loading controller " << controller_type << " and learner " << learner << std::endl;
auto brain_sdf = _sdf->GetElement("rv:brain");
auto controller_type = brain_sdf->GetElement("rv:controller")->GetAttribute("type")->GetAsString();
auto learner_type = brain_sdf->GetElement("rv:learner")->GetAttribute("type")->GetAsString();
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert tabs?

{
auto currentTime = _info.simTime.Double() - initTime_;
const gz::common::Time current_time = _info.simTime - initTime_;
const double current_time_d = current_time.Double();
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revert tabs?

evaluator->simulation_update(world_pose, current_time_d, delta_time);
}

if (brain_)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where did the brain go?

fuda van Diggelen and others added 5 commits October 7, 2020 23:55
 - Continue run
 - Load best genome

IMC:
 - Modularity from yaml
 - open-loop logging

BO:
 - LHS initialization

Evaluator:
 - Logging components of the fitnesses
 - Reset robot correctly with fitness

Positionmotor:
 - Added possibility for motor input noise

python_code:
 - Added yaml functionality for IMC

plane.world:
 - Added realtime and camera #commented <!-- -->
 - Continue run
 - Load best genome

IMC:
 - Modularity from yaml
 - open-loop logging

BO:
 - LHS initialization

Evaluator:
 - Logging components of the fitnesses
 - Reset robot correctly with fitness

Positionmotor:
 - Added possibility for motor input noise

python_code:
 - Added yaml functionality for IMC

plane.world:
 - Added realtime and camera #commented <!-- -->
 - Continue run
 - Load best genome

IMC:
 - Modularity from yaml
 - open-loop logging

BO:
 - LHS initialization

Evaluator:
 - Logging components of the fitnesses
 - Reset robot correctly with fitness

Positionmotor:
 - Added possibility for motor input noise

python_code:
 - Added yaml functionality for IMC

plane.world:
 - Added realtime and camera #commented <!-- -->
 - Continue run
 - Load best genome

IMC:
 - Modularity from yaml
 - open-loop logging

BO:
 - LHS initialization

Evaluator:
 - Logging components of the fitnesses
 - Reset robot correctly with fitness

Positionmotor:
 - Added possibility for motor input noise

python_code:
 - Added yaml functionality for IMC

plane.world:
 - Added realtime and camera #commented <!-- -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants