Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create sprint-1-2_18inchPipePile_bridgeSI_Georef_A #40

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ashishlov-uf
Copy link

File

Provide a short summary (3-5 lines):
For example, is it a new file, an update to an existing one?
Provide some bullet points about the content of the file, e.g. terrain model, tunnel structure, etc.

This is a new IFC file that satisfies the requirements set forth by sprint 1.1, and it has been generated wholly by the Bridge Software Institute (University of Florida).

The file describes a steel pipe pile that has been driven into a layer of sand.

Covered Concept Templates - Usages Included

Please list all Usages you have implemented.

Project Unit Assignment
-Plane Angle Unit is satisfied
Project Representation Context
-3D is satisfied
Project Declaration
-IfcActor is satisfied

Miscellaneous

Please use this area to convey anything else.

This IFC file satisfies the requirements set forth by sprint 1.1, and it has been generated wholly by the Bridge Software Institute (University of Florida).
@pjanck
Copy link
Member

pjanck commented Feb 8, 2023

Please rename the file to have a file ending .ifc, otherwise the checker doesn't react.

…e_bridgeSI.ifc

Renamed file to include ".ifc" extension.
@ashishlov-uf
Copy link
Author

Please rename the file to have a file ending .ifc, otherwise the checker doesn't react.

File has been renamed, thanks for catching that.

@SergejMuhic
Copy link
Collaborator

SergejMuhic commented Feb 9, 2023

One question, how did you get IfcSolidStratum into an IFC4 file? This is quite disconcerting. Not checked in this sprint but that would be quite a hard violation i.e. foundation for "IFC not working" statements.

@ashishlov-uf ashishlov-uf changed the title Create sprint-1-1_18inchPipePile_bridgeSI Create sprint-1-2_18inchPipePile_bridgeSI_Georef_A Feb 24, 2023
…2_18inchPipePile_bridgeSI_Georef_A.ifc

-Revised file based on comments related to sprint 1-1.
-Added content based on requirements of sprint 1-2 Georef_A.
Copy link
Member

@pjanck pjanck left a comment

Choose a reason for hiding this comment

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

The file has a lot more content than required - this approval only covers the concepts as part of Sprint 1.2. It may be that the file will fail future checks and will need amendment.

#91 = IFCSITE('2q0A6$eDbEJx$bZGM8f7tv', $, 'Default Bridge', $, $, #24, $, $, .ELEMENT., (0, 0, 0), (0, 0, 0), 0., $, $);
#92 = IFCRELAGGREGATES('2w4vtdhVoEH810nVKpCtY1', $, 'Project Bin', 'Project to Spatial Element', #1, (#91));

#93= IFCPROJECTEDCRS('EPSG:12364',$,$,$,$,$,$);
Copy link
Member

Choose a reason for hiding this comment

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

Out of curiosity: I can't find any reference to this EPSG code - can you provide a link to a source? (If you have simply made up the number, we encourage you to use a valid EPSG number, e.g. from https://epsg.io/ - the files are observed by domain experts as well 🚀)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants