From af19713a2778709fa178d011e6211f73666ed27a Mon Sep 17 00:00:00 2001 From: Enrico Turri Date: Thu, 13 Sep 2018 15:15:00 +0200 Subject: [PATCH 1/5] 1st installment of 3d offset for ModelInstance --- xs/CMakeLists.txt | 1 + xs/src/libslic3r/Format/3mf.cpp | 16 +++++ xs/src/libslic3r/Format/AMF.cpp | 80 ++++++++++++++++++++++++ xs/src/libslic3r/Format/PRUS.cpp | 30 ++++++++- xs/src/libslic3r/Model.cpp | 88 ++++++++++++++++++++++++++- xs/src/libslic3r/Model.hpp | 39 +++++++++++- xs/src/libslic3r/ModelArrange.hpp | 25 ++++++++ xs/src/libslic3r/PrintObject.cpp | 12 ++++ xs/src/libslic3r/Technologies.hpp | 12 ++++ xs/src/libslic3r/libslic3r.h | 4 ++ xs/src/slic3r/GUI/3DScene.cpp | 8 +++ xs/src/slic3r/GUI/GLCanvas3D.cpp | 16 +++++ xs/src/slic3r/GUI/GLGizmo.cpp | 41 +++++++++++++ xs/src/slic3r/GUI/GLGizmo.hpp | 8 +++ xs/src/slic3r/GUI/GUI_ObjectParts.cpp | 10 +++ xs/xsp/Model.xsp | 17 ++++++ 16 files changed, 400 insertions(+), 7 deletions(-) create mode 100644 xs/src/libslic3r/Technologies.hpp diff --git a/xs/CMakeLists.txt b/xs/CMakeLists.txt index b7d99ed69..1f2a93375 100644 --- a/xs/CMakeLists.txt +++ b/xs/CMakeLists.txt @@ -177,6 +177,7 @@ add_library(libslic3r STATIC ${LIBDIR}/libslic3r/SurfaceCollection.hpp ${LIBDIR}/libslic3r/SVG.cpp ${LIBDIR}/libslic3r/SVG.hpp + ${LIBDIR}/libslic3r/Technologies.hpp ${LIBDIR}/libslic3r/TriangleMesh.cpp ${LIBDIR}/libslic3r/TriangleMesh.hpp ${LIBDIR}/libslic3r/SLABasePool.hpp diff --git a/xs/src/libslic3r/Format/3mf.cpp b/xs/src/libslic3r/Format/3mf.cpp index 464cbb8e6..ddd3de69e 100644 --- a/xs/src/libslic3r/Format/3mf.cpp +++ b/xs/src/libslic3r/Format/3mf.cpp @@ -1252,9 +1252,17 @@ namespace Slic3r { // we extract from the given matrix only the values currently used // translation +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec3d offset(transform(0, 3), transform(1, 3), transform(2, 3)); +#else +//################################################################################################################################ double offset_x = transform(0, 3); double offset_y = transform(1, 3); double offset_z = transform(2, 3); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ // scale double sx = ::sqrt(sqr(transform(0, 0)) + sqr(transform(1, 0)) + sqr(transform(2, 0))); @@ -1287,8 +1295,16 @@ namespace Slic3r { double angle_z = (rotation.axis() == Vec3d::UnitZ()) ? rotation.angle() : -rotation.angle(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance.set_offset(offset); +#else +//################################################################################################################################ instance.offset(0) = offset_x; instance.offset(1) = offset_y; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ instance.scaling_factor = sx; instance.rotation = angle_z; } diff --git a/xs/src/libslic3r/Format/AMF.cpp b/xs/src/libslic3r/Format/AMF.cpp index 81617ad72..b783f4973 100644 --- a/xs/src/libslic3r/Format/AMF.cpp +++ b/xs/src/libslic3r/Format/AMF.cpp @@ -29,7 +29,16 @@ // VERSION NUMBERS // 0 : .amf, .amf.xml and .zip.amf files saved by older slic3r. No version definition in them. // 1 : Introduction of amf versioning. No other change in data saved into amf files. +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET +// 2 : Added z component of offset. +const unsigned int VERSION_AMF = 2; +#else +//################################################################################################################################ const unsigned int VERSION_AMF = 1; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ const char* SLIC3RPE_AMF_VERSION = "slic3rpe_amf_version"; const char* SLIC3R_CONFIG_TYPE = "slic3rpe_config"; @@ -119,19 +128,39 @@ struct AMFParserContext NODE_TYPE_INSTANCE, // amf/constellation/instance NODE_TYPE_DELTAX, // amf/constellation/instance/deltax NODE_TYPE_DELTAY, // amf/constellation/instance/deltay +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + NODE_TYPE_DELTAZ, // amf/constellation/instance/deltaz +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ NODE_TYPE_RZ, // amf/constellation/instance/rz NODE_TYPE_SCALE, // amf/constellation/instance/scale NODE_TYPE_METADATA, // anywhere under amf/*/metadata }; struct Instance { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Instance() : deltax_set(false), deltay_set(false), deltaz_set(false), rz_set(false), scale_set(false) {} +#else +//################################################################################################################################ Instance() : deltax_set(false), deltay_set(false), rz_set(false), scale_set(false) {} +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ // Shift in the X axis. float deltax; bool deltax_set; // Shift in the Y axis. float deltay; bool deltay_set; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + // Shift in the Z axis. + float deltaz; + bool deltaz_set; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ // Rotation around the Z axis. float rz; bool rz_set; @@ -254,6 +283,12 @@ void AMFParserContext::startElement(const char *name, const char **atts) node_type_new = NODE_TYPE_DELTAX; else if (strcmp(name, "deltay") == 0) node_type_new = NODE_TYPE_DELTAY; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + else if (strcmp(name, "deltaz") == 0) + node_type_new = NODE_TYPE_DELTAZ; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ else if (strcmp(name, "rz") == 0) node_type_new = NODE_TYPE_RZ; else if (strcmp(name, "scale") == 0) @@ -314,7 +349,19 @@ void AMFParserContext::characters(const XML_Char *s, int len) { switch (m_path.size()) { case 4: +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + if (m_path.back() == NODE_TYPE_DELTAX || + m_path.back() == NODE_TYPE_DELTAY || + m_path.back() == NODE_TYPE_DELTAZ || + m_path.back() == NODE_TYPE_RZ || + m_path.back() == NODE_TYPE_SCALE) +#else +//################################################################################################################################ if (m_path.back() == NODE_TYPE_DELTAX || m_path.back() == NODE_TYPE_DELTAY || m_path.back() == NODE_TYPE_RZ || m_path.back() == NODE_TYPE_SCALE) +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ m_value[0].append(s, len); break; case 6: @@ -354,6 +401,16 @@ void AMFParserContext::endElement(const char * /* name */) m_instance->deltay_set = true; m_value[0].clear(); break; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + case NODE_TYPE_DELTAZ: + assert(m_instance); + m_instance->deltaz = float(atof(m_value[0].c_str())); + m_instance->deltaz_set = true; + m_value[0].clear(); + break; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ case NODE_TYPE_RZ: assert(m_instance); m_instance->rz = float(atof(m_value[0].c_str())); @@ -498,8 +555,16 @@ void AMFParserContext::endDocument() for (const Instance &instance : object.second.instances) if (instance.deltax_set && instance.deltay_set) { ModelInstance *mi = m_model.objects[object.second.idx]->add_instance(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + mi->set_offset(Vec3d((double)instance.deltax, (double)instance.deltay, (double)instance.deltaz)); +#else +//################################################################################################################################ mi->offset(0) = instance.deltax; mi->offset(1) = instance.deltay; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ mi->rotation = instance.rz_set ? instance.rz : 0.f; mi->scaling_factor = instance.scale_set ? instance.scale : 1.f; } @@ -800,12 +865,27 @@ bool store_amf(const char *path, Model *model, Print* print, bool export_print_c " \n" " %lf\n" " %lf\n" +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + " %lf\n" +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ " %lf\n" " %lf\n" " \n", object_id, +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance->get_offset(X), + instance->get_offset(Y), + instance->get_offset(Z), +#else +//################################################################################################################################ instance->offset(0), instance->offset(1), +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ instance->rotation, instance->scaling_factor); //FIXME missing instance->scaling_factor diff --git a/xs/src/libslic3r/Format/PRUS.cpp b/xs/src/libslic3r/Format/PRUS.cpp index 3cf3fc075..0f3807c7a 100644 --- a/xs/src/libslic3r/Format/PRUS.cpp +++ b/xs/src/libslic3r/Format/PRUS.cpp @@ -166,7 +166,15 @@ bool load_prus(const char *path, Model *model) float trafo[3][4] = { 0 }; double instance_rotation = 0.; double instance_scaling_factor = 1.f; - Vec2d instance_offset(0., 0.); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec3d instance_offset = Vec3d::Zero(); +#else +//################################################################################################################################ + Vec2d instance_offset(0., 0.); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ bool trafo_set = false; unsigned int group_id = (unsigned int)-1; unsigned int extruder_id = (unsigned int)-1; @@ -207,8 +215,16 @@ bool load_prus(const char *path, Model *model) for (size_t c = 0; c < 3; ++ c) trafo[r][c] += mat_trafo(r, c); } +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance_offset = Vec3d((double)(position[0] - zero[0]), (double)(position[1] - zero[1]), (double)(position[2] - zero[2])); +#else +//################################################################################################################################ instance_offset(0) = position[0] - zero[0]; instance_offset(1) = position[1] - zero[1]; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ trafo[2][3] = position[2] / instance_scaling_factor; trafo_set = true; } @@ -360,8 +376,16 @@ bool load_prus(const char *path, Model *model) ModelInstance *instance = model_object->add_instance(); instance->rotation = instance_rotation; instance->scaling_factor = instance_scaling_factor; - instance->offset = instance_offset; - ++ num_models; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance->set_offset(instance_offset); +#else +//################################################################################################################################ + instance->offset = instance_offset; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + ++num_models; if (group_id != (size_t)-1) group_to_model_object[group_id] = model_object; } else { diff --git a/xs/src/libslic3r/Model.cpp b/xs/src/libslic3r/Model.cpp index 19c474cad..f7d9df79b 100644 --- a/xs/src/libslic3r/Model.cpp +++ b/xs/src/libslic3r/Model.cpp @@ -242,10 +242,27 @@ void Model::center_instances_around_point(const Vec2d &point) for (size_t i = 0; i < o->instances.size(); ++ i) bb.merge(o->instance_bounding_box(i, false)); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec2d shift2 = point - to_2d(bb.center()); + Vec3d shift3 = Vec3d(shift2(0), shift2(1), 0.0); +#else +//################################################################################################################################ Vec2d shift = point - 0.5 * to_2d(bb.size()) - to_2d(bb.min); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ for (ModelObject *o : this->objects) { for (ModelInstance *i : o->instances) +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + i->set_offset(i->get_offset() + shift3); +#else +//################################################################################################################################ i->offset += shift; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ o->invalidate_bounding_box(); } } @@ -311,8 +328,17 @@ bool Model::arrange_objects(coordf_t dist, const BoundingBoxf* bb) size_t idx = 0; for (ModelObject *o : this->objects) { for (ModelInstance *i : o->instances) { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec2d offset_xy = positions[idx] - instance_centers[idx]; + i->set_offset(Vec3d(offset_xy(0), offset_xy(1), i->get_offset(Z))); +#else +//################################################################################################################################ i->offset = positions[idx] - instance_centers[idx]; - ++ idx; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + ++idx; } o->invalidate_bounding_box(); } @@ -336,7 +362,15 @@ void Model::duplicate(size_t copies_num, coordf_t dist, const BoundingBoxf* bb) for (const ModelInstance *i : instances) { for (const Vec2d &pos : positions) { ModelInstance *instance = o->add_instance(*i); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance->set_offset(instance->get_offset() + Vec3d(pos(0), pos(1), 0.0)); +#else +//################################################################################################################################ instance->offset += pos; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } } o->invalidate_bounding_box(); @@ -366,13 +400,29 @@ void Model::duplicate_objects_grid(size_t x, size_t y, coordf_t dist) ModelObject* object = this->objects.front(); object->clear_instances(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec3d ext_size = object->bounding_box().size() + dist * Vec3d::Ones(); +#else +//################################################################################################################################ Vec3d size = object->bounding_box().size(); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ for (size_t x_copy = 1; x_copy <= x; ++x_copy) { for (size_t y_copy = 1; y_copy <= y; ++y_copy) { ModelInstance* instance = object->add_instance(); - instance->offset(0) = (size(0) + dist) * (x_copy-1); - instance->offset(1) = (size(1) + dist) * (y_copy-1); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + instance->set_offset(Vec3d(ext_size(0) * (double)(x_copy - 1), ext_size(1) * (double)(y_copy - 1), 0.0)); +#else +//################################################################################################################################ + instance->offset(0) = (size(0) + dist) * (x_copy - 1); + instance->offset(1) = (size(1) + dist) * (y_copy - 1); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } } } @@ -676,12 +726,27 @@ void ModelObject::center_around_origin() this->translate(shift); this->origin_translation += shift; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + // set z to zero, translation in z has already been done within the mesh + shift(2) = 0.0; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + if (!this->instances.empty()) { for (ModelInstance *i : this->instances) { // apply rotation and scaling to vector as well before translating instance, // in order to leave final position unaltered +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + i->set_offset(i->get_offset() + i->transform_vector(-shift, true)); +#else +//################################################################################################################################ Vec3d i_shift = i->world_matrix(true) * shift; i->offset -= to_2d(i_shift); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } this->invalidate_bounding_box(); } @@ -1004,8 +1069,17 @@ BoundingBoxf3 ModelInstance::transform_mesh_bounding_box(const TriangleMesh* mes } // Translate the bounding box. if (! dont_translate) { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + bbox.min += this->m_offset; + bbox.max += this->m_offset; +#else +//################################################################################################################################ Eigen::Map(bbox.min.data()) += this->offset; Eigen::Map(bbox.max.data()) += this->offset; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } } return bbox; @@ -1032,7 +1106,15 @@ Transform3d ModelInstance::world_matrix(bool dont_translate, bool dont_rotate, b Transform3d m = Transform3d::Identity(); if (!dont_translate) +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + m.translate(m_offset); +#else +//################################################################################################################################ m.translate(Vec3d(offset(0), offset(1), 0.0)); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ if (!dont_rotate) m.rotate(Eigen::AngleAxisd(rotation, Vec3d::UnitZ())); diff --git a/xs/src/libslic3r/Model.hpp b/xs/src/libslic3r/Model.hpp index 8a8af481c..39633162d 100644 --- a/xs/src/libslic3r/Model.hpp +++ b/xs/src/libslic3r/Model.hpp @@ -224,15 +224,42 @@ public: friend class ModelObject; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET +private: + Vec3d m_offset; // in unscaled coordinates + +public: +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + double rotation; // Rotation around the Z axis, in radians around mesh center point double scaling_factor; +//################################################################################################################################ +#if !ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ Vec2d offset; // in unscaled coordinates - +//################################################################################################################################ +#endif // !ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + // flag showing the position of this instance with respect to the print volume (set by Print::validate() using ModelObject::check_instances_print_volume_state()) EPrintVolumeState print_volume_state; ModelObject* get_object() const { return this->object; } +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + const Vec3d& get_offset() const { return m_offset; } + double get_offset(Axis axis) const { return m_offset(axis); } + + void set_offset(const Vec3d& offset) { m_offset = offset; } + void set_offset(Axis axis, double offset) { m_offset(axis) = offset; } + +// void add_to_offset(const Vec3d& shift) { m_offset += shift; } +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ + // To be called on an external mesh void transform_mesh(TriangleMesh* mesh, bool dont_translate = false) const; // Calculate a bounding box of a transformed mesh. To be called on an external mesh. @@ -252,9 +279,19 @@ private: // Parent object, owning this instance. ModelObject* object; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + ModelInstance(ModelObject *object) : rotation(0), scaling_factor(1), m_offset(Vec3d::Zero()), object(object), print_volume_state(PVS_Inside) {} + ModelInstance(ModelObject *object, const ModelInstance &other) : + rotation(other.rotation), scaling_factor(other.scaling_factor), m_offset(other.m_offset), object(object), print_volume_state(PVS_Inside) {} +#else +//################################################################################################################################ ModelInstance(ModelObject *object) : rotation(0), scaling_factor(1), offset(Vec2d::Zero()), object(object), print_volume_state(PVS_Inside) {} ModelInstance(ModelObject *object, const ModelInstance &other) : rotation(other.rotation), scaling_factor(other.scaling_factor), offset(other.offset), object(object), print_volume_state(PVS_Inside) {} +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ }; diff --git a/xs/src/libslic3r/ModelArrange.hpp b/xs/src/libslic3r/ModelArrange.hpp index a8e7936d6..68d842722 100644 --- a/xs/src/libslic3r/ModelArrange.hpp +++ b/xs/src/libslic3r/ModelArrange.hpp @@ -517,8 +517,17 @@ ShapeData2D projectModelFromTop(const Slic3r::Model &model) { if(item.vertexCount() > 3) { item.rotation(objinst->rotation); item.translation( { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + ClipperLib::cInt(objinst->get_offset(X) / SCALING_FACTOR), + ClipperLib::cInt(objinst->get_offset(Y) / SCALING_FACTOR) +#else +//################################################################################################################################ ClipperLib::cInt(objinst->offset(0)/SCALING_FACTOR), ClipperLib::cInt(objinst->offset(1)/SCALING_FACTOR) +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ }); ret.emplace_back(objinst, item); } @@ -655,11 +664,27 @@ void applyResult( // appropriately auto off = item.translation(); Radians rot = item.rotation(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec3d foff(off.X*SCALING_FACTOR + batch_offset, off.Y*SCALING_FACTOR, 0.0); +#else +//################################################################################################################################ Vec2d foff(off.X*SCALING_FACTOR + batch_offset, off.Y*SCALING_FACTOR); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ // write the tranformation data into the model instance inst_ptr->rotation = rot; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + inst_ptr->set_offset(foff); +#else +//################################################################################################################################ inst_ptr->offset = foff; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } } diff --git a/xs/src/libslic3r/PrintObject.cpp b/xs/src/libslic3r/PrintObject.cpp index d03b99416..fac653d08 100644 --- a/xs/src/libslic3r/PrintObject.cpp +++ b/xs/src/libslic3r/PrintObject.cpp @@ -102,8 +102,20 @@ bool PrintObject::reload_model_instances() copies.reserve(this->_model_object->instances.size()); for (const ModelInstance *mi : this->_model_object->instances) { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + if (mi->is_printable()) + { + const Vec3d& offset = mi->get_offset(); + copies.emplace_back(Point::new_scale(offset(0), offset(1))); + } +#else +//################################################################################################################################ if (mi->is_printable()) copies.emplace_back(Point::new_scale(mi->offset(0), mi->offset(1))); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } return this->set_copies(copies); } diff --git a/xs/src/libslic3r/Technologies.hpp b/xs/src/libslic3r/Technologies.hpp new file mode 100644 index 000000000..5c4f8617d --- /dev/null +++ b/xs/src/libslic3r/Technologies.hpp @@ -0,0 +1,12 @@ +#ifndef _technologies_h_ +#define _technologies_h_ + +// 1.42.0 techs +#define ENABLE_1_42_0 1 + +// Add z coordinate to model instances' offset +#define ENABLE_MODELINSTANCE_3D_OFFSET (1 && ENABLE_1_42_0) + +#endif // _technologies_h_ + + diff --git a/xs/src/libslic3r/libslic3r.h b/xs/src/libslic3r/libslic3r.h index 925e93031..2a96682f0 100644 --- a/xs/src/libslic3r/libslic3r.h +++ b/xs/src/libslic3r/libslic3r.h @@ -13,6 +13,10 @@ #include #include +//################################################################################################################################ +#include "Technologies.hpp" +//################################################################################################################################ + #define SLIC3R_FORK_NAME "Slic3r Prusa Edition" #define SLIC3R_VERSION "1.41.0" #define SLIC3R_BUILD "UNKNOWN" diff --git a/xs/src/slic3r/GUI/3DScene.cpp b/xs/src/slic3r/GUI/3DScene.cpp index ed2f8690d..a25e29928 100644 --- a/xs/src/slic3r/GUI/3DScene.cpp +++ b/xs/src/slic3r/GUI/3DScene.cpp @@ -684,7 +684,15 @@ std::vector GLVolumeCollection::load_object( } v.is_modifier = model_volume->modifier; v.shader_outside_printer_detection_enabled = !model_volume->modifier; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + v.set_offset(instance->get_offset()); +#else +//################################################################################################################################ v.set_offset(Vec3d(instance->offset(0), instance->offset(1), 0.0)); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ v.set_rotation(instance->rotation); v.set_scaling_factor(instance->scaling_factor); } diff --git a/xs/src/slic3r/GUI/GLCanvas3D.cpp b/xs/src/slic3r/GUI/GLCanvas3D.cpp index fdee693ee..b037e57ee 100644 --- a/xs/src/slic3r/GUI/GLCanvas3D.cpp +++ b/xs/src/slic3r/GUI/GLCanvas3D.cpp @@ -2375,7 +2375,15 @@ void GLCanvas3D::update_gizmos_data() ModelInstance* model_instance = model_object->instances[0]; if (model_instance != nullptr) { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + m_gizmos.set_position(model_instance->get_offset()); +#else +//################################################################################################################################ m_gizmos.set_position(Vec3d(model_instance->offset(0), model_instance->offset(1), 0.0)); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ m_gizmos.set_scale(model_instance->scaling_factor); m_gizmos.set_angle_z(model_instance->rotation); m_gizmos.set_flattening_data(model_object); @@ -5355,8 +5363,16 @@ void GLCanvas3D::_on_move(const std::vector& volume_idxs) ModelObject* model_object = m_model->objects[obj_idx]; if (model_object != nullptr) { +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + model_object->instances[instance_idx]->set_offset(volume->get_offset()); +#else +//################################################################################################################################ const Vec3d& offset = volume->get_offset(); model_object->instances[instance_idx]->offset = Vec2d(offset(0), offset(1)); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ model_object->invalidate_bounding_box(); update_position_values(); object_moved = true; diff --git a/xs/src/slic3r/GUI/GLGizmo.cpp b/xs/src/slic3r/GUI/GLGizmo.cpp index 9c209035b..892ae5ca4 100644 --- a/xs/src/slic3r/GUI/GLGizmo.cpp +++ b/xs/src/slic3r/GUI/GLGizmo.cpp @@ -1233,10 +1233,27 @@ void GLGizmoFlatten::on_render(const BoundingBoxf3& box) const else ::glColor4f(0.9f, 0.9f, 0.9f, 0.5f); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + for (Vec3d offset : m_instances_positions) { + offset += dragged_offset; +#else +//################################################################################################################################ for (Vec2d offset : m_instances_positions) { offset += to_2d(dragged_offset); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ ::glPushMatrix(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + ::glTranslated(offset(0), offset(1), offset(2)); +#else +//################################################################################################################################ ::glTranslatef((GLfloat)offset(0), (GLfloat)offset(1), 0.0f); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ ::glBegin(GL_POLYGON); for (const Vec3d& vertex : m_planes[i].vertices) ::glVertex3f((GLfloat)vertex(0), (GLfloat)vertex(1), (GLfloat)vertex(2)); @@ -1255,9 +1272,25 @@ void GLGizmoFlatten::on_render_for_picking(const BoundingBoxf3& box) const for (unsigned int i = 0; i < m_planes.size(); ++i) { ::glColor3f(1.0f, 1.0f, picking_color_component(i)); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + for (const Vec3d& offset : m_instances_positions) { +#else +//################################################################################################################################ for (const Vec2d& offset : m_instances_positions) { +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ ::glPushMatrix(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + ::glTranslated(offset(0), offset(1), offset(2)); +#else +//################################################################################################################################ ::glTranslatef((GLfloat)offset(0), (GLfloat)offset(1), 0.0f); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ ::glBegin(GL_POLYGON); for (const Vec3d& vertex : m_planes[i].vertices) ::glVertex3f((GLfloat)vertex(0), (GLfloat)vertex(1), (GLfloat)vertex(2)); @@ -1275,7 +1308,15 @@ void GLGizmoFlatten::set_flattening_data(const ModelObject* model_object) if (m_model_object && !m_model_object->instances.empty()) { m_instances_positions.clear(); for (const auto* instance : m_model_object->instances) +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + m_instances_positions.emplace_back(instance->get_offset()); +#else +//################################################################################################################################ m_instances_positions.emplace_back(instance->offset); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } if (is_plane_update_necessary()) diff --git a/xs/src/slic3r/GUI/GLGizmo.hpp b/xs/src/slic3r/GUI/GLGizmo.hpp index fc2912d3a..41ccb4d77 100644 --- a/xs/src/slic3r/GUI/GLGizmo.hpp +++ b/xs/src/slic3r/GUI/GLGizmo.hpp @@ -328,7 +328,15 @@ private: SourceDataSummary m_source_data; std::vector m_planes; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Pointf3s m_instances_positions; +#else +//################################################################################################################################ std::vector m_instances_positions; +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ Vec3d m_starting_center; const ModelObject* m_model_object = nullptr; diff --git a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp index af57db8ed..1570a4ccf 100644 --- a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp +++ b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp @@ -1564,9 +1564,19 @@ void update_position_values() auto og = get_optgroup(ogFrequentlyObjectSettings); auto instance = (*m_objects)[m_selected_object_id]->instances.front(); +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + og->set_value("position_x", int(instance->get_offset(X))); + og->set_value("position_y", int(instance->get_offset(Y))); + og->set_value("position_z", int(instance->get_offset(Z))); +#else +//################################################################################################################################ og->set_value("position_x", int(instance->offset(0))); og->set_value("position_y", int(instance->offset(1))); og->set_value("position_z", 0); +//################################################################################################################################ +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ } void update_position_values(const Vec3d& position) diff --git a/xs/xsp/Model.xsp b/xs/xsp/Model.xsp index a5925775d..9032323c7 100644 --- a/xs/xsp/Model.xsp +++ b/xs/xsp/Model.xsp @@ -358,15 +358,32 @@ ModelMaterial::attributes() %code%{ RETVAL = THIS->rotation; %}; double scaling_factor() %code%{ RETVAL = THIS->scaling_factor; %}; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + Vec2d* offset() + %code%{ RETVAL = new Vec2d(THIS->get_offset(X), THIS->get_offset(Y)); %}; +#else Ref offset() %code%{ RETVAL = &THIS->offset; %}; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ void set_rotation(double val) %code%{ THIS->rotation = val; THIS->get_object()->invalidate_bounding_box(); %}; void set_scaling_factor(double val) %code%{ THIS->scaling_factor = val; THIS->get_object()->invalidate_bounding_box(); %}; +//################################################################################################################################ +#if ENABLE_MODELINSTANCE_3D_OFFSET + void set_offset(Vec2d *offset) + %code%{ + THIS->set_offset(X, (*offset)(0)); + THIS->set_offset(Y, (*offset)(1)); + %}; +#else void set_offset(Vec2d *offset) %code%{ THIS->offset = *offset; %}; +#endif // ENABLE_MODELINSTANCE_3D_OFFSET +//################################################################################################################################ void transform_mesh(TriangleMesh* mesh, bool dont_translate = false) const; void transform_polygon(Polygon* polygon) const; From 3faaff85d52db84947c93da6a8aa94a3c440e41a Mon Sep 17 00:00:00 2001 From: Enrico Turri Date: Thu, 13 Sep 2018 16:08:19 +0200 Subject: [PATCH 2/5] Code cleanup --- xs/src/libslic3r/Format/3mf.cpp | 8 ------ xs/src/libslic3r/Format/AMF.cpp | 30 --------------------- xs/src/libslic3r/Format/PRUS.cpp | 12 --------- xs/src/libslic3r/Model.cpp | 38 --------------------------- xs/src/libslic3r/Model.hpp | 14 ---------- xs/src/libslic3r/ModelArrange.hpp | 12 --------- xs/src/libslic3r/PrintObject.cpp | 4 --- xs/src/libslic3r/libslic3r.h | 2 -- xs/src/slic3r/GUI/3DScene.cpp | 4 --- xs/src/slic3r/GUI/GLCanvas3D.cpp | 10 ++----- xs/src/slic3r/GUI/GLGizmo.cpp | 20 -------------- xs/src/slic3r/GUI/GLGizmo.hpp | 4 --- xs/src/slic3r/GUI/GUI_ObjectParts.cpp | 4 --- xs/xsp/Model.xsp | 4 --- 14 files changed, 2 insertions(+), 164 deletions(-) diff --git a/xs/src/libslic3r/Format/3mf.cpp b/xs/src/libslic3r/Format/3mf.cpp index ddd3de69e..c5c37a128 100644 --- a/xs/src/libslic3r/Format/3mf.cpp +++ b/xs/src/libslic3r/Format/3mf.cpp @@ -1252,17 +1252,13 @@ namespace Slic3r { // we extract from the given matrix only the values currently used // translation -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec3d offset(transform(0, 3), transform(1, 3), transform(2, 3)); #else -//################################################################################################################################ double offset_x = transform(0, 3); double offset_y = transform(1, 3); double offset_z = transform(2, 3); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // scale double sx = ::sqrt(sqr(transform(0, 0)) + sqr(transform(1, 0)) + sqr(transform(2, 0))); @@ -1295,16 +1291,12 @@ namespace Slic3r { double angle_z = (rotation.axis() == Vec3d::UnitZ()) ? rotation.angle() : -rotation.angle(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance.set_offset(offset); #else -//################################################################################################################################ instance.offset(0) = offset_x; instance.offset(1) = offset_y; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ instance.scaling_factor = sx; instance.rotation = angle_z; } diff --git a/xs/src/libslic3r/Format/AMF.cpp b/xs/src/libslic3r/Format/AMF.cpp index b783f4973..3f3839d43 100644 --- a/xs/src/libslic3r/Format/AMF.cpp +++ b/xs/src/libslic3r/Format/AMF.cpp @@ -29,16 +29,12 @@ // VERSION NUMBERS // 0 : .amf, .amf.xml and .zip.amf files saved by older slic3r. No version definition in them. // 1 : Introduction of amf versioning. No other change in data saved into amf files. -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET // 2 : Added z component of offset. const unsigned int VERSION_AMF = 2; #else -//################################################################################################################################ const unsigned int VERSION_AMF = 1; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ const char* SLIC3RPE_AMF_VERSION = "slic3rpe_amf_version"; const char* SLIC3R_CONFIG_TYPE = "slic3rpe_config"; @@ -128,39 +124,31 @@ struct AMFParserContext NODE_TYPE_INSTANCE, // amf/constellation/instance NODE_TYPE_DELTAX, // amf/constellation/instance/deltax NODE_TYPE_DELTAY, // amf/constellation/instance/deltay -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET NODE_TYPE_DELTAZ, // amf/constellation/instance/deltaz #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ NODE_TYPE_RZ, // amf/constellation/instance/rz NODE_TYPE_SCALE, // amf/constellation/instance/scale NODE_TYPE_METADATA, // anywhere under amf/*/metadata }; struct Instance { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Instance() : deltax_set(false), deltay_set(false), deltaz_set(false), rz_set(false), scale_set(false) {} #else -//################################################################################################################################ Instance() : deltax_set(false), deltay_set(false), rz_set(false), scale_set(false) {} -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // Shift in the X axis. float deltax; bool deltax_set; // Shift in the Y axis. float deltay; bool deltay_set; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET // Shift in the Z axis. float deltaz; bool deltaz_set; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // Rotation around the Z axis. float rz; bool rz_set; @@ -283,12 +271,10 @@ void AMFParserContext::startElement(const char *name, const char **atts) node_type_new = NODE_TYPE_DELTAX; else if (strcmp(name, "deltay") == 0) node_type_new = NODE_TYPE_DELTAY; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET else if (strcmp(name, "deltaz") == 0) node_type_new = NODE_TYPE_DELTAZ; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ else if (strcmp(name, "rz") == 0) node_type_new = NODE_TYPE_RZ; else if (strcmp(name, "scale") == 0) @@ -349,7 +335,6 @@ void AMFParserContext::characters(const XML_Char *s, int len) { switch (m_path.size()) { case 4: -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET if (m_path.back() == NODE_TYPE_DELTAX || m_path.back() == NODE_TYPE_DELTAY || @@ -357,11 +342,8 @@ void AMFParserContext::characters(const XML_Char *s, int len) m_path.back() == NODE_TYPE_RZ || m_path.back() == NODE_TYPE_SCALE) #else -//################################################################################################################################ if (m_path.back() == NODE_TYPE_DELTAX || m_path.back() == NODE_TYPE_DELTAY || m_path.back() == NODE_TYPE_RZ || m_path.back() == NODE_TYPE_SCALE) -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ m_value[0].append(s, len); break; case 6: @@ -401,7 +383,6 @@ void AMFParserContext::endElement(const char * /* name */) m_instance->deltay_set = true; m_value[0].clear(); break; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET case NODE_TYPE_DELTAZ: assert(m_instance); @@ -410,7 +391,6 @@ void AMFParserContext::endElement(const char * /* name */) m_value[0].clear(); break; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ case NODE_TYPE_RZ: assert(m_instance); m_instance->rz = float(atof(m_value[0].c_str())); @@ -555,16 +535,12 @@ void AMFParserContext::endDocument() for (const Instance &instance : object.second.instances) if (instance.deltax_set && instance.deltay_set) { ModelInstance *mi = m_model.objects[object.second.idx]->add_instance(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET mi->set_offset(Vec3d((double)instance.deltax, (double)instance.deltay, (double)instance.deltaz)); #else -//################################################################################################################################ mi->offset(0) = instance.deltax; mi->offset(1) = instance.deltay; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ mi->rotation = instance.rz_set ? instance.rz : 0.f; mi->scaling_factor = instance.scale_set ? instance.scale : 1.f; } @@ -865,27 +841,21 @@ bool store_amf(const char *path, Model *model, Print* print, bool export_print_c " \n" " %lf\n" " %lf\n" -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET " %lf\n" #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ " %lf\n" " %lf\n" " \n", object_id, -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance->get_offset(X), instance->get_offset(Y), instance->get_offset(Z), #else -//################################################################################################################################ instance->offset(0), instance->offset(1), -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ instance->rotation, instance->scaling_factor); //FIXME missing instance->scaling_factor diff --git a/xs/src/libslic3r/Format/PRUS.cpp b/xs/src/libslic3r/Format/PRUS.cpp index 0f3807c7a..45eb56c63 100644 --- a/xs/src/libslic3r/Format/PRUS.cpp +++ b/xs/src/libslic3r/Format/PRUS.cpp @@ -166,15 +166,11 @@ bool load_prus(const char *path, Model *model) float trafo[3][4] = { 0 }; double instance_rotation = 0.; double instance_scaling_factor = 1.f; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec3d instance_offset = Vec3d::Zero(); #else -//################################################################################################################################ Vec2d instance_offset(0., 0.); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ bool trafo_set = false; unsigned int group_id = (unsigned int)-1; unsigned int extruder_id = (unsigned int)-1; @@ -215,16 +211,12 @@ bool load_prus(const char *path, Model *model) for (size_t c = 0; c < 3; ++ c) trafo[r][c] += mat_trafo(r, c); } -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance_offset = Vec3d((double)(position[0] - zero[0]), (double)(position[1] - zero[1]), (double)(position[2] - zero[2])); #else -//################################################################################################################################ instance_offset(0) = position[0] - zero[0]; instance_offset(1) = position[1] - zero[1]; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ trafo[2][3] = position[2] / instance_scaling_factor; trafo_set = true; } @@ -376,15 +368,11 @@ bool load_prus(const char *path, Model *model) ModelInstance *instance = model_object->add_instance(); instance->rotation = instance_rotation; instance->scaling_factor = instance_scaling_factor; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance->set_offset(instance_offset); #else -//################################################################################################################################ instance->offset = instance_offset; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ++num_models; if (group_id != (size_t)-1) group_to_model_object[group_id] = model_object; diff --git a/xs/src/libslic3r/Model.cpp b/xs/src/libslic3r/Model.cpp index f7d9df79b..1088c2e87 100644 --- a/xs/src/libslic3r/Model.cpp +++ b/xs/src/libslic3r/Model.cpp @@ -242,27 +242,19 @@ void Model::center_instances_around_point(const Vec2d &point) for (size_t i = 0; i < o->instances.size(); ++ i) bb.merge(o->instance_bounding_box(i, false)); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec2d shift2 = point - to_2d(bb.center()); Vec3d shift3 = Vec3d(shift2(0), shift2(1), 0.0); #else -//################################################################################################################################ Vec2d shift = point - 0.5 * to_2d(bb.size()) - to_2d(bb.min); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ for (ModelObject *o : this->objects) { for (ModelInstance *i : o->instances) -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET i->set_offset(i->get_offset() + shift3); #else -//################################################################################################################################ i->offset += shift; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ o->invalidate_bounding_box(); } } @@ -328,16 +320,12 @@ bool Model::arrange_objects(coordf_t dist, const BoundingBoxf* bb) size_t idx = 0; for (ModelObject *o : this->objects) { for (ModelInstance *i : o->instances) { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec2d offset_xy = positions[idx] - instance_centers[idx]; i->set_offset(Vec3d(offset_xy(0), offset_xy(1), i->get_offset(Z))); #else -//################################################################################################################################ i->offset = positions[idx] - instance_centers[idx]; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ++idx; } o->invalidate_bounding_box(); @@ -362,15 +350,11 @@ void Model::duplicate(size_t copies_num, coordf_t dist, const BoundingBoxf* bb) for (const ModelInstance *i : instances) { for (const Vec2d &pos : positions) { ModelInstance *instance = o->add_instance(*i); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance->set_offset(instance->get_offset() + Vec3d(pos(0), pos(1), 0.0)); #else -//################################################################################################################################ instance->offset += pos; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } } o->invalidate_bounding_box(); @@ -400,29 +384,21 @@ void Model::duplicate_objects_grid(size_t x, size_t y, coordf_t dist) ModelObject* object = this->objects.front(); object->clear_instances(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec3d ext_size = object->bounding_box().size() + dist * Vec3d::Ones(); #else -//################################################################################################################################ Vec3d size = object->bounding_box().size(); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ for (size_t x_copy = 1; x_copy <= x; ++x_copy) { for (size_t y_copy = 1; y_copy <= y; ++y_copy) { ModelInstance* instance = object->add_instance(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET instance->set_offset(Vec3d(ext_size(0) * (double)(x_copy - 1), ext_size(1) * (double)(y_copy - 1), 0.0)); #else -//################################################################################################################################ instance->offset(0) = (size(0) + dist) * (x_copy - 1); instance->offset(1) = (size(1) + dist) * (y_copy - 1); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } } } @@ -726,27 +702,21 @@ void ModelObject::center_around_origin() this->translate(shift); this->origin_translation += shift; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET // set z to zero, translation in z has already been done within the mesh shift(2) = 0.0; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ if (!this->instances.empty()) { for (ModelInstance *i : this->instances) { // apply rotation and scaling to vector as well before translating instance, // in order to leave final position unaltered -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET i->set_offset(i->get_offset() + i->transform_vector(-shift, true)); #else -//################################################################################################################################ Vec3d i_shift = i->world_matrix(true) * shift; i->offset -= to_2d(i_shift); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } this->invalidate_bounding_box(); } @@ -1069,17 +1039,13 @@ BoundingBoxf3 ModelInstance::transform_mesh_bounding_box(const TriangleMesh* mes } // Translate the bounding box. if (! dont_translate) { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET bbox.min += this->m_offset; bbox.max += this->m_offset; #else -//################################################################################################################################ Eigen::Map(bbox.min.data()) += this->offset; Eigen::Map(bbox.max.data()) += this->offset; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } } return bbox; @@ -1106,15 +1072,11 @@ Transform3d ModelInstance::world_matrix(bool dont_translate, bool dont_rotate, b Transform3d m = Transform3d::Identity(); if (!dont_translate) -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET m.translate(m_offset); #else -//################################################################################################################################ m.translate(Vec3d(offset(0), offset(1), 0.0)); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ if (!dont_rotate) m.rotate(Eigen::AngleAxisd(rotation, Vec3d::UnitZ())); diff --git a/xs/src/libslic3r/Model.hpp b/xs/src/libslic3r/Model.hpp index 39633162d..e8cd5473b 100644 --- a/xs/src/libslic3r/Model.hpp +++ b/xs/src/libslic3r/Model.hpp @@ -224,41 +224,31 @@ public: friend class ModelObject; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET private: Vec3d m_offset; // in unscaled coordinates public: #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ double rotation; // Rotation around the Z axis, in radians around mesh center point double scaling_factor; -//################################################################################################################################ #if !ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ Vec2d offset; // in unscaled coordinates -//################################################################################################################################ #endif // !ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // flag showing the position of this instance with respect to the print volume (set by Print::validate() using ModelObject::check_instances_print_volume_state()) EPrintVolumeState print_volume_state; ModelObject* get_object() const { return this->object; } -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET const Vec3d& get_offset() const { return m_offset; } double get_offset(Axis axis) const { return m_offset(axis); } void set_offset(const Vec3d& offset) { m_offset = offset; } void set_offset(Axis axis, double offset) { m_offset(axis) = offset; } - -// void add_to_offset(const Vec3d& shift) { m_offset += shift; } #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // To be called on an external mesh void transform_mesh(TriangleMesh* mesh, bool dont_translate = false) const; @@ -279,19 +269,15 @@ private: // Parent object, owning this instance. ModelObject* object; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET ModelInstance(ModelObject *object) : rotation(0), scaling_factor(1), m_offset(Vec3d::Zero()), object(object), print_volume_state(PVS_Inside) {} ModelInstance(ModelObject *object, const ModelInstance &other) : rotation(other.rotation), scaling_factor(other.scaling_factor), m_offset(other.m_offset), object(object), print_volume_state(PVS_Inside) {} #else -//################################################################################################################################ ModelInstance(ModelObject *object) : rotation(0), scaling_factor(1), offset(Vec2d::Zero()), object(object), print_volume_state(PVS_Inside) {} ModelInstance(ModelObject *object, const ModelInstance &other) : rotation(other.rotation), scaling_factor(other.scaling_factor), offset(other.offset), object(object), print_volume_state(PVS_Inside) {} -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ }; diff --git a/xs/src/libslic3r/ModelArrange.hpp b/xs/src/libslic3r/ModelArrange.hpp index 68d842722..3c2c2aa29 100644 --- a/xs/src/libslic3r/ModelArrange.hpp +++ b/xs/src/libslic3r/ModelArrange.hpp @@ -517,17 +517,13 @@ ShapeData2D projectModelFromTop(const Slic3r::Model &model) { if(item.vertexCount() > 3) { item.rotation(objinst->rotation); item.translation( { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET ClipperLib::cInt(objinst->get_offset(X) / SCALING_FACTOR), ClipperLib::cInt(objinst->get_offset(Y) / SCALING_FACTOR) #else -//################################################################################################################################ ClipperLib::cInt(objinst->offset(0)/SCALING_FACTOR), ClipperLib::cInt(objinst->offset(1)/SCALING_FACTOR) -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ }); ret.emplace_back(objinst, item); } @@ -664,27 +660,19 @@ void applyResult( // appropriately auto off = item.translation(); Radians rot = item.rotation(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec3d foff(off.X*SCALING_FACTOR + batch_offset, off.Y*SCALING_FACTOR, 0.0); #else -//################################################################################################################################ Vec2d foff(off.X*SCALING_FACTOR + batch_offset, off.Y*SCALING_FACTOR); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ // write the tranformation data into the model instance inst_ptr->rotation = rot; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET inst_ptr->set_offset(foff); #else -//################################################################################################################################ inst_ptr->offset = foff; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } } diff --git a/xs/src/libslic3r/PrintObject.cpp b/xs/src/libslic3r/PrintObject.cpp index fac653d08..9d1d58d6d 100644 --- a/xs/src/libslic3r/PrintObject.cpp +++ b/xs/src/libslic3r/PrintObject.cpp @@ -102,7 +102,6 @@ bool PrintObject::reload_model_instances() copies.reserve(this->_model_object->instances.size()); for (const ModelInstance *mi : this->_model_object->instances) { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET if (mi->is_printable()) { @@ -110,12 +109,9 @@ bool PrintObject::reload_model_instances() copies.emplace_back(Point::new_scale(offset(0), offset(1))); } #else -//################################################################################################################################ if (mi->is_printable()) copies.emplace_back(Point::new_scale(mi->offset(0), mi->offset(1))); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } return this->set_copies(copies); } diff --git a/xs/src/libslic3r/libslic3r.h b/xs/src/libslic3r/libslic3r.h index 2a96682f0..cf0a19f7a 100644 --- a/xs/src/libslic3r/libslic3r.h +++ b/xs/src/libslic3r/libslic3r.h @@ -13,9 +13,7 @@ #include #include -//################################################################################################################################ #include "Technologies.hpp" -//################################################################################################################################ #define SLIC3R_FORK_NAME "Slic3r Prusa Edition" #define SLIC3R_VERSION "1.41.0" diff --git a/xs/src/slic3r/GUI/3DScene.cpp b/xs/src/slic3r/GUI/3DScene.cpp index a25e29928..6b6f44a8c 100644 --- a/xs/src/slic3r/GUI/3DScene.cpp +++ b/xs/src/slic3r/GUI/3DScene.cpp @@ -684,15 +684,11 @@ std::vector GLVolumeCollection::load_object( } v.is_modifier = model_volume->modifier; v.shader_outside_printer_detection_enabled = !model_volume->modifier; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET v.set_offset(instance->get_offset()); #else -//################################################################################################################################ v.set_offset(Vec3d(instance->offset(0), instance->offset(1), 0.0)); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ v.set_rotation(instance->rotation); v.set_scaling_factor(instance->scaling_factor); } diff --git a/xs/src/slic3r/GUI/GLCanvas3D.cpp b/xs/src/slic3r/GUI/GLCanvas3D.cpp index b037e57ee..1e81965a1 100644 --- a/xs/src/slic3r/GUI/GLCanvas3D.cpp +++ b/xs/src/slic3r/GUI/GLCanvas3D.cpp @@ -1140,8 +1140,10 @@ bool GLCanvas3D::Gizmos::init(GLCanvas3D& parent) if (!gizmo->init()) return false; +#if !ENABLE_MODELINSTANCE_3D_OFFSET // temporary disable z grabber gizmo->disable_grabber(2); +#endif // !ENABLE_MODELINSTANCE_3D_OFFSET m_gizmos.insert(GizmosMap::value_type(Move, gizmo)); @@ -2375,15 +2377,11 @@ void GLCanvas3D::update_gizmos_data() ModelInstance* model_instance = model_object->instances[0]; if (model_instance != nullptr) { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET m_gizmos.set_position(model_instance->get_offset()); #else -//################################################################################################################################ m_gizmos.set_position(Vec3d(model_instance->offset(0), model_instance->offset(1), 0.0)); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ m_gizmos.set_scale(model_instance->scaling_factor); m_gizmos.set_angle_z(model_instance->rotation); m_gizmos.set_flattening_data(model_object); @@ -5363,16 +5361,12 @@ void GLCanvas3D::_on_move(const std::vector& volume_idxs) ModelObject* model_object = m_model->objects[obj_idx]; if (model_object != nullptr) { -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET model_object->instances[instance_idx]->set_offset(volume->get_offset()); #else -//################################################################################################################################ const Vec3d& offset = volume->get_offset(); model_object->instances[instance_idx]->offset = Vec2d(offset(0), offset(1)); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ model_object->invalidate_bounding_box(); update_position_values(); object_moved = true; diff --git a/xs/src/slic3r/GUI/GLGizmo.cpp b/xs/src/slic3r/GUI/GLGizmo.cpp index 892ae5ca4..b461e6657 100644 --- a/xs/src/slic3r/GUI/GLGizmo.cpp +++ b/xs/src/slic3r/GUI/GLGizmo.cpp @@ -1233,27 +1233,19 @@ void GLGizmoFlatten::on_render(const BoundingBoxf3& box) const else ::glColor4f(0.9f, 0.9f, 0.9f, 0.5f); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET for (Vec3d offset : m_instances_positions) { offset += dragged_offset; #else -//################################################################################################################################ for (Vec2d offset : m_instances_positions) { offset += to_2d(dragged_offset); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ::glPushMatrix(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET ::glTranslated(offset(0), offset(1), offset(2)); #else -//################################################################################################################################ ::glTranslatef((GLfloat)offset(0), (GLfloat)offset(1), 0.0f); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ::glBegin(GL_POLYGON); for (const Vec3d& vertex : m_planes[i].vertices) ::glVertex3f((GLfloat)vertex(0), (GLfloat)vertex(1), (GLfloat)vertex(2)); @@ -1272,25 +1264,17 @@ void GLGizmoFlatten::on_render_for_picking(const BoundingBoxf3& box) const for (unsigned int i = 0; i < m_planes.size(); ++i) { ::glColor3f(1.0f, 1.0f, picking_color_component(i)); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET for (const Vec3d& offset : m_instances_positions) { #else -//################################################################################################################################ for (const Vec2d& offset : m_instances_positions) { -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ::glPushMatrix(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET ::glTranslated(offset(0), offset(1), offset(2)); #else -//################################################################################################################################ ::glTranslatef((GLfloat)offset(0), (GLfloat)offset(1), 0.0f); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ ::glBegin(GL_POLYGON); for (const Vec3d& vertex : m_planes[i].vertices) ::glVertex3f((GLfloat)vertex(0), (GLfloat)vertex(1), (GLfloat)vertex(2)); @@ -1308,15 +1292,11 @@ void GLGizmoFlatten::set_flattening_data(const ModelObject* model_object) if (m_model_object && !m_model_object->instances.empty()) { m_instances_positions.clear(); for (const auto* instance : m_model_object->instances) -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET m_instances_positions.emplace_back(instance->get_offset()); #else -//################################################################################################################################ m_instances_positions.emplace_back(instance->offset); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } if (is_plane_update_necessary()) diff --git a/xs/src/slic3r/GUI/GLGizmo.hpp b/xs/src/slic3r/GUI/GLGizmo.hpp index 41ccb4d77..e51dca5b2 100644 --- a/xs/src/slic3r/GUI/GLGizmo.hpp +++ b/xs/src/slic3r/GUI/GLGizmo.hpp @@ -328,15 +328,11 @@ private: SourceDataSummary m_source_data; std::vector m_planes; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Pointf3s m_instances_positions; #else -//################################################################################################################################ std::vector m_instances_positions; -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ Vec3d m_starting_center; const ModelObject* m_model_object = nullptr; diff --git a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp index 1570a4ccf..9f4e1c151 100644 --- a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp +++ b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp @@ -1564,19 +1564,15 @@ void update_position_values() auto og = get_optgroup(ogFrequentlyObjectSettings); auto instance = (*m_objects)[m_selected_object_id]->instances.front(); -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET og->set_value("position_x", int(instance->get_offset(X))); og->set_value("position_y", int(instance->get_offset(Y))); og->set_value("position_z", int(instance->get_offset(Z))); #else -//################################################################################################################################ og->set_value("position_x", int(instance->offset(0))); og->set_value("position_y", int(instance->offset(1))); og->set_value("position_z", 0); -//################################################################################################################################ #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ } void update_position_values(const Vec3d& position) diff --git a/xs/xsp/Model.xsp b/xs/xsp/Model.xsp index 9032323c7..6ae7d6415 100644 --- a/xs/xsp/Model.xsp +++ b/xs/xsp/Model.xsp @@ -358,7 +358,6 @@ ModelMaterial::attributes() %code%{ RETVAL = THIS->rotation; %}; double scaling_factor() %code%{ RETVAL = THIS->scaling_factor; %}; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET Vec2d* offset() %code%{ RETVAL = new Vec2d(THIS->get_offset(X), THIS->get_offset(Y)); %}; @@ -366,13 +365,11 @@ ModelMaterial::attributes() Ref offset() %code%{ RETVAL = &THIS->offset; %}; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ void set_rotation(double val) %code%{ THIS->rotation = val; THIS->get_object()->invalidate_bounding_box(); %}; void set_scaling_factor(double val) %code%{ THIS->scaling_factor = val; THIS->get_object()->invalidate_bounding_box(); %}; -//################################################################################################################################ #if ENABLE_MODELINSTANCE_3D_OFFSET void set_offset(Vec2d *offset) %code%{ @@ -383,7 +380,6 @@ ModelMaterial::attributes() void set_offset(Vec2d *offset) %code%{ THIS->offset = *offset; %}; #endif // ENABLE_MODELINSTANCE_3D_OFFSET -//################################################################################################################################ void transform_mesh(TriangleMesh* mesh, bool dont_translate = false) const; void transform_polygon(Polygon* polygon) const; From 578792be31d57eb084b39a1a04540946765264aa Mon Sep 17 00:00:00 2001 From: Enrico Turri Date: Fri, 14 Sep 2018 11:37:26 +0200 Subject: [PATCH 3/5] Fixed #1229 --- lib/Slic3r/GUI/Plater/ObjectPartsPanel.pm | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/Slic3r/GUI/Plater/ObjectPartsPanel.pm b/lib/Slic3r/GUI/Plater/ObjectPartsPanel.pm index 783c1a9f5..4032886f3 100644 --- a/lib/Slic3r/GUI/Plater/ObjectPartsPanel.pm +++ b/lib/Slic3r/GUI/Plater/ObjectPartsPanel.pm @@ -362,13 +362,17 @@ sub on_btn_load { } foreach my $object (@{$model->objects}) { + $object->center_around_origin; foreach my $volume (@{$object->volumes}) { my $new_volume = $self->{model_object}->add_volume($volume); $new_volume->set_modifier($is_modifier); $new_volume->set_name(basename($input_file)); # apply the same translation we applied to the object - $new_volume->mesh->translate(@{$self->{model_object}->origin_translation}); + my $delta_x = $self->{model_object}->origin_translation->x - $object->origin_translation->x; + my $delta_y = $self->{model_object}->origin_translation->y - $object->origin_translation->y; + my $delta_z = $self->{model_object}->origin_translation->z - $object->origin_translation->z; + $new_volume->mesh->translate($delta_x, $delta_y, $delta_z); # set a default extruder value, since user can't add it manually $new_volume->config->set_ifndef('extruder', 0); From eb2b56a87f1d16f334c6d1df4933a26014a4731b Mon Sep 17 00:00:00 2001 From: YuSanka Date: Fri, 14 Sep 2018 13:59:56 +0200 Subject: [PATCH 4/5] Fixed settings list for objects + fixed showing of an info_manifold_warning_icon --- lib/Slic3r/GUI/Plater.pm | 15 +++++++------ xs/src/slic3r/GUI/GUI.cpp | 14 ++++++++++-- xs/src/slic3r/GUI/GUI.hpp | 2 +- xs/src/slic3r/GUI/GUI_ObjectParts.cpp | 31 ++++++++++++++++----------- xs/src/slic3r/GUI/GUI_ObjectParts.hpp | 2 ++ 5 files changed, 41 insertions(+), 23 deletions(-) diff --git a/lib/Slic3r/GUI/Plater.pm b/lib/Slic3r/GUI/Plater.pm index 050d2dee8..0cb07023c 100644 --- a/lib/Slic3r/GUI/Plater.pm +++ b/lib/Slic3r/GUI/Plater.pm @@ -607,11 +607,12 @@ sub new { #$self->{object_info_manifold_warning_icon}->Hide; $self->{"object_info_manifold_warning_icon_show"} = sub { if ($self->{object_info_manifold_warning_icon}->IsShown() != $_[0]) { + # this fuction show/hide info_manifold_warning_icon on the c++ side now Slic3r::GUI::set_show_manifold_warning_icon($_[0]); - my $mode = wxTheApp->{app_config}->get("view_mode"); - return if ($mode eq "" || $mode eq "simple"); - $self->{object_info_manifold_warning_icon}->Show($_[0]); - $self->Layout + #my $mode = wxTheApp->{app_config}->get("view_mode"); + #return if ($mode eq "" || $mode eq "simple"); + #$self->{object_info_manifold_warning_icon}->Show($_[0]); + #$self->Layout } }; $self->{"object_info_manifold_warning_icon_show"}->(0); @@ -2478,11 +2479,11 @@ sub select_object { if (defined $obj_idx) { $self->{objects}->[$obj_idx]->selected(1); # Select current object in the list on c++ side, if item isn't child - if (!defined $child){ - Slic3r::GUI::select_current_object($obj_idx);} +# if (!defined $child){ +# Slic3r::GUI::select_current_object($obj_idx);} # all selections in the object list is on c++ side } else { # Unselect all objects in the list on c++ side - Slic3r::GUI::unselect_objects(); +# Slic3r::GUI::unselect_objects(); # all selections in the object list is on c++ side } $self->selection_changed(1); } diff --git a/xs/src/slic3r/GUI/GUI.cpp b/xs/src/slic3r/GUI/GUI.cpp index 7d8a9b9d5..433acea1d 100644 --- a/xs/src/slic3r/GUI/GUI.cpp +++ b/xs/src/slic3r/GUI/GUI.cpp @@ -259,6 +259,15 @@ void set_show_print_info(bool show) void set_show_manifold_warning_icon(bool show) { g_show_manifold_warning_icon = show; + if (!g_manifold_warning_icon) + return; + + // update manifold_warning_icon showing + if (show && !g_info_sizer->IsShown(static_cast(0))) + g_show_manifold_warning_icon = false; + + g_manifold_warning_icon->Show(g_show_manifold_warning_icon); + g_manifold_warning_icon->GetParent()->Layout(); } void set_objects_list_sizer(wxBoxSizer *objects_list_sizer){ @@ -1118,11 +1127,11 @@ void show_buttons(bool show) } } -void show_info_sizer(const bool show, const bool is_update_settings/* = false*/) +void show_info_sizer(const bool show) { g_info_sizer->Show(static_cast(0), show); g_info_sizer->Show(1, show && g_show_print_info); - g_manifold_warning_icon->Show(show && (!is_update_settings && g_show_manifold_warning_icon)); + g_manifold_warning_icon->Show(show && g_show_manifold_warning_icon); } void show_object_name(bool show) @@ -1142,6 +1151,7 @@ void update_mode() show_info_sizer(mode == ConfigMenuModeExpert); show_buttons(mode == ConfigMenuModeExpert); show_object_name(mode == ConfigMenuModeSimple); + show_manipulation_sizer(mode == ConfigMenuModeSimple); // TODO There is a not the best place of it! // *** Update showing of the collpane_settings diff --git a/xs/src/slic3r/GUI/GUI.hpp b/xs/src/slic3r/GUI/GUI.hpp index 7e9acfabf..448c37773 100644 --- a/xs/src/slic3r/GUI/GUI.hpp +++ b/xs/src/slic3r/GUI/GUI.hpp @@ -182,7 +182,7 @@ bool select_language(wxArrayString & names, wxArrayLong & identifiers); // update right panel of the Plater according to view mode void update_mode(); -void show_info_sizer(const bool show, const bool is_update_settinfs = false); +void show_info_sizer(const bool show); std::vector& get_tabs_list(); bool checked_tab(Tab* tab); diff --git a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp index ff69eaffe..40b066b1d 100644 --- a/xs/src/slic3r/GUI/GUI_ObjectParts.cpp +++ b/xs/src/slic3r/GUI/GUI_ObjectParts.cpp @@ -731,11 +731,8 @@ void select_current_object(int idx) { g_prevent_list_events = true; m_objects_ctrl->UnselectAll(); - if (idx < 0) { - g_prevent_list_events = false; - return; - } - m_objects_ctrl->Select(m_objects_model->GetItemById(idx)); + if (idx>=0) + m_objects_ctrl->Select(m_objects_model->GetItemById(idx)); part_selection_changed(); g_prevent_list_events = false; } @@ -748,12 +745,8 @@ void select_current_volume(int idx, int vol_idx) } g_prevent_list_events = true; m_objects_ctrl->UnselectAll(); - if (idx < 0) { - g_prevent_list_events = false; - return; - } - - m_objects_ctrl->Select(m_objects_model->GetItemByVolumeId(idx, vol_idx)); + if (idx >= 0) + m_objects_ctrl->Select(m_objects_model->GetItemByVolumeId(idx, vol_idx)); part_selection_changed(); g_prevent_list_events = false; } @@ -1186,8 +1179,8 @@ wxMenu *create_add_settings_popupmenu(bool is_part) menu->Append(menu_item); } #ifndef __WXMSW__ - menu->Bind(wxEVT_MENU, [menu](wxEvent &event) { - get_settings_choice(menu, event.GetId(), true); + menu->Bind(wxEVT_MENU, [menu,is_part](wxEvent &event) { + get_settings_choice(menu, event.GetId(), is_part); }); #endif //no __WXMSW__ return menu; @@ -2027,5 +2020,17 @@ void update_double_slider_from_canvas(wxKeyEvent& event) event.Skip(); } +void show_manipulation_sizer(const bool is_simple_mode) +{ + auto item = m_objects_ctrl->GetSelection(); + if (!item || !is_simple_mode) + return; + + if (m_objects_model->IsSettingsItem(item)) { + m_objects_ctrl->Select(m_objects_model->GetParent(item)); + part_selection_changed(); + } +} + } //namespace GUI } //namespace Slic3r \ No newline at end of file diff --git a/xs/src/slic3r/GUI/GUI_ObjectParts.hpp b/xs/src/slic3r/GUI/GUI_ObjectParts.hpp index 70c077bd7..e66b4d1db 100644 --- a/xs/src/slic3r/GUI/GUI_ObjectParts.hpp +++ b/xs/src/slic3r/GUI/GUI_ObjectParts.hpp @@ -140,6 +140,8 @@ void reset_double_slider(); // update DoubleSlider after keyDown in canvas void update_double_slider_from_canvas(wxKeyEvent& event); +void show_manipulation_sizer(const bool is_simple_mode); + } //namespace GUI } //namespace Slic3r #endif //slic3r_GUI_ObjectParts_hpp_ \ No newline at end of file From d139274da8a43215ff8acb10a8060342113807af Mon Sep 17 00:00:00 2001 From: Enrico Turri Date: Fri, 14 Sep 2018 14:37:13 +0200 Subject: [PATCH 5/5] Fixed gizmo move 3d axis Z --- xs/src/slic3r/GUI/GLGizmo.cpp | 27 +++++++++++++++------------ xs/src/slic3r/GUI/GLGizmo.hpp | 3 ++- 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/xs/src/slic3r/GUI/GLGizmo.cpp b/xs/src/slic3r/GUI/GLGizmo.cpp index b461e6657..e23958c1d 100644 --- a/xs/src/slic3r/GUI/GLGizmo.cpp +++ b/xs/src/slic3r/GUI/GLGizmo.cpp @@ -1032,6 +1032,7 @@ GLGizmoMove3D::GLGizmoMove3D(GLCanvas3D& parent) , m_position(Vec3d::Zero()) , m_starting_drag_position(Vec3d::Zero()) , m_starting_box_center(Vec3d::Zero()) + , m_starting_box_bottom_center(Vec3d::Zero()) { } @@ -1065,17 +1066,19 @@ void GLGizmoMove3D::on_start_dragging(const BoundingBoxf3& box) { m_starting_drag_position = m_grabbers[m_hover_id].center; m_starting_box_center = box.center(); + m_starting_box_bottom_center = box.center(); + m_starting_box_bottom_center(2) = box.min(2); } } void GLGizmoMove3D::on_update(const Linef3& mouse_ray) { if (m_hover_id == 0) - m_position(0) = 2.0 * m_starting_box_center(0) + calc_displacement(1, mouse_ray) - m_starting_drag_position(0); + m_position(0) = 2.0 * m_starting_box_center(0) + calc_projection(X, 1, mouse_ray) - m_starting_drag_position(0); else if (m_hover_id == 1) - m_position(1) = 2.0 * m_starting_box_center(1) + calc_displacement(2, mouse_ray) - m_starting_drag_position(1); + m_position(1) = 2.0 * m_starting_box_center(1) + calc_projection(Y, 2, mouse_ray) - m_starting_drag_position(1); else if (m_hover_id == 2) - m_position(2) = 2.0 * m_starting_box_center(2) + calc_displacement(1, mouse_ray) - m_starting_drag_position(2); + m_position(2) = 2.0 * m_starting_box_bottom_center(2) + calc_projection(Z, 1, mouse_ray) - m_starting_drag_position(2); } void GLGizmoMove3D::on_render(const BoundingBoxf3& box) const @@ -1144,40 +1147,40 @@ void GLGizmoMove3D::on_render_for_picking(const BoundingBoxf3& box) const render_grabbers_for_picking(box); } -double GLGizmoMove3D::calc_displacement(unsigned int preferred_plane_id, const Linef3& mouse_ray) const +double GLGizmoMove3D::calc_projection(Axis axis, unsigned int preferred_plane_id, const Linef3& mouse_ray) const { - double displacement = 0.0; + double projection = 0.0; - Vec3d starting_vec = m_starting_drag_position - m_starting_box_center; + Vec3d starting_vec = (axis == Z) ? m_starting_drag_position - m_starting_box_bottom_center : m_starting_drag_position - m_starting_box_center; double len_starting_vec = starting_vec.norm(); if (len_starting_vec == 0.0) - return displacement; + return projection; Vec3d starting_vec_dir = starting_vec.normalized(); Vec3d mouse_dir = mouse_ray.unit_vector(); unsigned int plane_id = select_best_plane(mouse_dir, preferred_plane_id); - switch (plane_id) + switch (plane_id) { case 0: { - displacement = starting_vec_dir.dot(intersection_on_plane_xy(mouse_ray, m_starting_box_center)); + projection = starting_vec_dir.dot(intersection_on_plane_xy(mouse_ray, (axis == Z) ? m_starting_box_bottom_center : m_starting_box_center)); break; } case 1: { - displacement = starting_vec_dir.dot(intersection_on_plane_xz(mouse_ray, m_starting_box_center)); + projection = starting_vec_dir.dot(intersection_on_plane_xz(mouse_ray, (axis == Z) ? m_starting_box_bottom_center : m_starting_box_center)); break; } case 2: { - displacement = starting_vec_dir.dot(intersection_on_plane_yz(mouse_ray, m_starting_box_center)); + projection = starting_vec_dir.dot(intersection_on_plane_yz(mouse_ray, (axis == Z) ? m_starting_box_bottom_center : m_starting_box_center)); break; } } - return displacement; + return projection; } GLGizmoFlatten::GLGizmoFlatten(GLCanvas3D& parent) diff --git a/xs/src/slic3r/GUI/GLGizmo.hpp b/xs/src/slic3r/GUI/GLGizmo.hpp index e51dca5b2..2430b5af5 100644 --- a/xs/src/slic3r/GUI/GLGizmo.hpp +++ b/xs/src/slic3r/GUI/GLGizmo.hpp @@ -287,6 +287,7 @@ class GLGizmoMove3D : public GLGizmoBase Vec3d m_position; Vec3d m_starting_drag_position; Vec3d m_starting_box_center; + Vec3d m_starting_box_bottom_center; public: explicit GLGizmoMove3D(GLCanvas3D& parent); @@ -302,7 +303,7 @@ protected: virtual void on_render_for_picking(const BoundingBoxf3& box) const; private: - double calc_displacement(unsigned int preferred_plane_id, const Linef3& mouse_ray) const; + double calc_projection(Axis axis, unsigned int preferred_plane_id, const Linef3& mouse_ray) const; }; class GLGizmoFlatten : public GLGizmoBase