Duplicated Face identifiers were found

After running the PO_ValidateModel, I frequently get the error The following duplicated Face identifies were found and then a list of Room_ID…Face_ID. Looking at the same, I see two different faces with the same Face_ID.

What I’m confused about is why these errors are occurring in the first place. I am using the PO_DrawRoom command almost exclusively.

Is there a way to fix this duplicate Face_ID without redrawing the space?

Hi @justinshultz, That should not be happening. Can you help us to recreate the issue? Does it happen when you copy the rooms? Is there a way to recreate the issue in a new model? @mingbo should be able to help you with fixing it quickly.

I can share a Python script with you that resets all the IDs for all the faces. Or if you can share the model I can fix the particular faces that are causing issues. Let me know which one do you prefer.

P.S. If you export this model to IES this error shouldn’t matter as we use display names for the IES GEM file export.

Hi @mostapha,

I noticed it happens most in spaces that have been PO_SplitRoom or rooms with more than 6 sides.

Can you share the python script? That would be a huge help!

Good to know that it is not important in the GEM file. Sorry I can’t share the model! I just finished fixing it. If it happens again I’ll isolate the space and see if you can find out any additional info.

1 Like

Hi @justinshultz,

@mingbo and I had a chat and we are going to add this functionality as a new command for the Rhino plugin which should also address your other wish for using the display name for the IDs to make the IDF file and the EnergyPlus reports human-readable.

We will also try to fix the issue in the first place. You should not get duplicated faces using the Rhino plugin. That’s a bug on our side.

Hi @justinshultz,

Without a sample geometry, it is hard to recreate this issue on my side.

Meanwhile, as @mostapha mentioned above, we just added a new command: PO_ResetIdentifier which has an option to reset object id by its display name.

A new installer should be ready in an hour.

Hi @mingbo,

If I find a more reliable way to reproduce the Duplicate Face_ID error, I will let you know!

Thank you for adding the new command! You guys are amazing with your turnaround time. I will update the plugin in an hour.

Best,
Justin

4 posts were split to a new topic: Rooms are not loading due to a geometry problem

@mingbo and @mostapha I’ve found a pretty reliable way to create the duplicate surface ID error. One of the ways I like to model spaces that are not rectangular is to use the Draw Room command with the Rectangular option then drawing the Room within the whole void.

This creates an overlap of room with adjacent spaces. I then use the SplitRoom command to subtract the excess.

I then delete the extra spaces that were leftover.

I’ve used this method several times and frequently get the Duplicate Face ID error. The ResetIdentifier command has been very helpful but sometimes Pollination is still really confused. Here’s an example for VENDOR STORAGE 112:


Here is the model if you would like to dive deeper:
https://eypae-my.sharepoint.com/:u:/p/jshultz/Ed7sDTEQceNKgsqMCL_4FosBoA50U8cY61euFqytqkMMxg?e=cgdkgU

Thanks @justinshultz I will look into this issue and see if I could recreate it on my side.

Hi @justinshultz, I think the issue is with the combination of SplitRoom and ResetIdentifier commands.

So SplitRoom resets faces ID by adding room id to the original face id. So that even there are two faces that have the same display name (“Face bfd59” in the following video), but their ids are still different.

Now the issue is with ResetIdentifier command which resets the face id based on the display name, which creates the duplicated IDs in this case.

I think I will add a new option to the ResetIdentifier command, from which we can create a new ID based on the combination of the display name and a random ID. So that we can ensure this duplicated id issue could be minimized.

Hi @justinshultz I added a new ID generating option: ByDisplayNameAndID, which adds a random id to the display name for the identifier so that it could avoid your issue happening again. A new installer will be available in a few hours, I will keep you posted.

Hi @mingbo this is a good catch, though PO_ValidateModel finds an error with those duplicate IDs, or at least it did in my case. Are these false positive errors/warnings in PO_ValidateModel?

I like this workaround. I like to use the DisplayName so that when I’m troubleshooting problems in the model, I know what Room I need to go to. I don’t mind if there are extra characters added.

Hi @justinshultz, I don’t think these errors are false-positive errors, and I think PO_ValidateModel command should catch these duplicated IDs no matter how they were created.

A new installer is released with above update, please let me know if it works for you.

Thanks again for reporting this issue.

1 Like