odoomrp / odoomrp-wip

WIP of OdooMRP
http://www.odoomrp.com
GNU Affero General Public License v3.0
117 stars 248 forks source link

[mrp_production_real_costs].Traspaso numero de ciclos de la ruta a la OF #1006

Open cor9003 opened 8 years ago

cor9003 commented 8 years ago

Hola, El "numero de ciclos" que se define en una ruta de produccion no se traspasa a la orden de produccion, tanto en la ficticia como en la real. Al darle al boton de calcular datos de la orden de trabajo, lo calcula todo Ok excepto este valor, que siempre lo deja a 1.

Este valor, el modulo si lo utiliza para el calculo del coste estimado de la maquina, que es el "numero de ciclos" por "coste por ciclo" del centro de trabajo, con lo que ¿ no deberia traspasarse a la OF para que el calculo de costes sea acorde con la ruta de produccion definida ?

Por otro lado, he intentado desde la OF ficticia modificar el valor "numero de ciclos" de una orden de trabajo y si me da la opción de hacerlo, pero le doy a guardar y no lo guarda. Además sale con valor a cero cuando tiene el valor de 1. Tiene pinta de ser un bug.

Saludos David

pedrobaeza commented 8 years ago

David, aunque los cálculos debería hacerlos correctamente, sí que es cierto que en el formulario emergente los valores no se mostraban correctamente. De todas formas, le he dado una revisión grande que culmina con en este PR: https://github.com/odoomrp/odoomrp-wip/pull/1061. Puedes por favor probarlo y decirme tu opinión?

cor9003 commented 8 years ago

Hola Pedro, No conozco muy bien github y para probarlo lo que he hecho es borrar en mi sistema odoo todos los modulos de odoormp, descargarlos de nuevo de github en .zip, pasarlos a mi sistema y hacer un update all a toda la BD, pero he visto que no es asi, pues en lo que he bajado no estan todos los cambios que veo el PR #1061 ( veo que se han modificado 10 ficheros ). Si que esta el nuevo hook instalado, etc, pero no los cambios que veo el PR- En definitiva, no se como he de probar un PR. Si no te importa decirme como puedo probar este PR lo hare encantado, asi me servira para el resto de veces que tenga que hacerlo.

pedrobaeza commented 8 years ago

Entra en el enlace de runbot del propio PR: https://odoo-community.org/page/review

cor9003 commented 8 years ago

tested in: http://3119739-1061-1205f9.runbot2.odoo-community.org/web?db=3119739-1061-1205f9-all Created routing named: BJ626147-B with 2 operations named "punzonado" and "preparation", Both have '5' as number of cycle value Created ficticious MO named FMO000002. As you can see the number of cycles is '1' and should be '5' because these are the values created in number of cycles fields for the operations moved to the MO Result: Error moving number of cycles, but now the form show the number of cycles . Before to create the PR #1061, always was '0' and now at least show the value '1''.

anajuaristi commented 8 years ago

Hola David. Estamos probando lo mismo... echa un vistazo por favor a mi último comentario aquí https://github.com/odoomrp/odoomrp-wip/pull/1061

Una OF ficticia es exactamente igual que una real, lo único que tiene el flag activo a false. Por lo que lo que encuentres en ellas aplica también a las OFs normales y al revés. Creo que el lío del número de ciclos se debe a una etiqueta que lleva a confusión, como pongo aquí: The only confusing thing is the label "Number of Cycles" on routing line. It's showing capacity per cycle, so it should be renamed to "capacity per cicle" On example "capacity per cicle" on machine is 500, so "number of cycles" is 1, but it's showing "number of cycles"=500 on routing line and it's not true.

Gracias por ayudar con el testing. La verdad es que nos es sumamente útil el feedback

cor9003 commented 8 years ago

Si, si , he visto el comentario y efectivamente el lio es debido a la etiqueta. Iba a probar ahora que cuando en una lista de materiales se consume el mismo material en 2 operaciones diferentes funcione ahora con el hook_mrp. Si te acuerdas se hizo una correcion en el codigo (issue closed #1007) pero tenia el problema de que se recalculaba, y creo que el hook_mrp en parte era para evitar esto. Lo he probado en mi sistema (con el hook_mrp instalado y la ultima version de odoomrp ) y no me ha funcionado. He intentado probarlo en el sistema del PR #1061 pero de momento me da el errorValueError Expected singleton: mrp.bom()

Cuando este solucionado probare esta casuistica y te dire.

Por cierto, cuando he bajado todos los programas de odoomrp a mi sistema, al hacer un update all de la BD me daba el error de que no encontrabal el modulo quality_control porque ya no se encuentra en la version de odoomrp, he tenido que recuperarlo para que funcionara, no se si es correcto que ya no este.

Muchas gracias

pedrobaeza commented 8 years ago

El módulo quality_control forma ya parte de OCA en OCA/manufacture. Acerca de la etiqueta, no está corregido. @Daniel-CA o @agaldona, podéis encargaros de revisarlo y hacer el PR aquí y en OCA?

Daniel-CA commented 8 years ago

Lo único que esa etiqueta viene así definida de base en el módulo mrp, meto la modificación en el mrp_operations_extension?

pedrobaeza commented 8 years ago

Sí, y recuerda hacerlo también en el que está en OCA

JMODaERP commented 8 years ago

Feliz año a todos. Me gustaría comentar que en http://3138909-8-0-4bbf95.runbot1.odoo-community.org/web este error, que ya habíamos comprobado y reportado hace tiempo, está corregido y el número de ciclos que aparece en la orden de producción (ejemplo MO00010) en la pestaña correspondiente a las órdenes de trabajo (bajo el texto número de ciclos) aparece, felizmente, la división realizada entre las unidades a fabricar y la capacidad del ciclo expresada en el centro de trabajo o modificada en la ruta si así hubiésemos hecho.

Quizá lo preocupante es que errores que existen en algunas versiones estén corregidos en otras y quepa la posibilidad que; al arreglar algunas incidencias en alguna versión de Odoo se provoquen regresiones a versiones que contienen errores ya erradicados.

pedrobaeza commented 8 years ago

No hay posibilidad de errores, ya que los módulos tienen nombres distintos (uno en plural y el otro no). La idea será acabar eliminando estos módulos de aquí y dejar sólo los de OCA, migrando los clientes que los tengan instalados.