pyrevitlabs / pyRevit

Rapid Application Development (RAD) Environment for Autodesk Revit®
http://wiki.pyrevitlabs.io
GNU General Public License v3.0
1.3k stars 332 forks source link

4.4 conflicts with dynamobim #216

Closed epeter- closed 7 years ago

epeter- commented 7 years ago

the new version is conflicting with dynamobim (both 1.3.1 and 0.92). when opening a python window dynamo crashes since installing pyrevit.

eirannejad commented 7 years ago

I can't replicate the error here. Can you share a GIF screen cap and the error report if anything? @gtalarico Is this happening to you as well?

epeter- commented 7 years ago

ehsan, thanks for replying! unfortunately there are no error messages and significant pics. only when hitting the python note in dynamo for editing the python code, dynamo freezes. -> revit 2017 on windows 10 64 bits. setup is user with restricted permissions & administrative user best peter Am 2017-07-20 21:00, schrieb Ehsan Iran-Nejad:

I can't replicate the error here. Can you share a GIF screen cap and the error report if anything? @gtalarico [1] Is this happening to you as well?

-- You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub [2], or mute the thread [3].

*

Links:

[1] https://github.com/gtalarico [2] https://github.com/eirannejad/pyRevit/issues/216#issuecomment-316798750 [3] https://github.com/notifications/unsubscribe-auth/AGm4a1Xr99dvQBfcyjQpaqFD8LgdYf2aks5sP6O_gaJpZM4OeEgT

eirannejad commented 7 years ago

Honestly this is really a conflict issue between IronPython versions of pyRevit and Dynamo

Dynamo is not specific about which version of the IronPython it should use so it defaults to the latest version which is the one loaded by pyRevit (currently 2.7.7) and it crashes.

epeter- commented 7 years ago

ehsan, that was my expectation. what do you suggest to use both pyrevit and dynamo? best peter

eirannejad commented 7 years ago

Let me spend some time today and test a few things

gtalarico commented 7 years ago

Just wanted to reference this here since the discussion is still active on DynamoDS: https://github.com/DynamoDS/Dynamo/issues/8028#issuecomment-316165224

One user posted that uninstalling rps and pyrevit fixed it.

Possible Solutions

  1. Dynamo is not specific about which version of the IronPython it should use Maybe we can submit an issue/suggestion to Dynamo with correct code so it loads only specific assembly instead of latest? (edit: just asked it. Let's see what they say.) https://github.com/DynamoDS/Dynamo/tree/dec6240ded0c4369617775336b9af60c2aba4103/src/Libraries/DSIronPython

  2. Is there a way to load the pyrevit IronPython (2.7.7) in a way that Dynamo would not pick it up? (Rename it? Different app domain?)

eirannejad commented 7 years ago

We need to recompile the IronPython assembly with a completely different name. It's not as easy as renaming the file. I think the correct solution is for Dynamo to update to 2.7.7 IMHO

epeter- commented 7 years ago

didn't work on my side.

have immediately deinstalled dps and dynamo. reinstalled but still confilicting.

only when disabling the addin (pyrevit) no problems in dynamo.

best peter

Am 07.08.2017 um 17:46 schrieb Gui Talarico:

Just wanted to reference this here since the discussion is still active on DynamoDS: DynamoDS/Dynamo#8028 (comment) https://github.com/DynamoDS/Dynamo/issues/8028#issuecomment-316165224

One user posted that uninstalling rps and pyrevit fixed it.

  Possible Solutions

1.

/Dynamo is not specific about which version of the IronPython it
should use/
Maybe we can submit an issue/suggestion to Dynamo with correct
code so it loads only specific assembly instead of latest?
https://github.com/DynamoDS/Dynamo/tree/dec6240ded0c4369617775336b9af60c2aba4103/src/Libraries/DSIronPython

2.

Is there a way to load the pyrevit IronPython (2.7.7) in a way
that Dynamo would not pick it up? (Rename it? Different app domain?)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/eirannejad/pyRevit/issues/216#issuecomment-320701059, or mute the thread https://github.com/notifications/unsubscribe-auth/AGm4a1aNTrbs70-BE3D8L-iIpWmEopGBks5sVzFRgaJpZM4OeEgT.


Diese E-Mail wurde von Avast Antivirus-Software auf Viren geprüft. https://www.avast.com/antivirus

epeter- commented 7 years ago

+1

Am 07.08.2017 um 18:46 schrieb Ehsan Iran-Nejad:

We need to recompile the IronPython assembly with a completely different name. It's not as easy as renaming the file. I think the correct solution is for Dynamo to update to 2.7.7 IMHO

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/eirannejad/pyRevit/issues/216#issuecomment-320717043, or mute the thread https://github.com/notifications/unsubscribe-auth/AGm4axAItv-Ltudoipg9MLWIgZp7ZICfks5sVz90gaJpZM4OeEgT.


Diese E-Mail wurde von Avast Antivirus-Software auf Viren geprüft. https://www.avast.com/antivirus