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

Authoring Atom User Guide | Section: Materials #28

Merged
merged 12 commits into from
Mar 9, 2021
Merged

Authoring Atom User Guide | Section: Materials #28

merged 12 commits into from
Mar 9, 2021

Conversation

chanmosq
Copy link
Contributor

No description provided.

Copy link
Contributor

@carrcain carrcain left a comment

Choose a reason for hiding this comment

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

edits

@chanmosq chanmosq changed the title [WIP] Authoring Atom User Guide | Section: Materials Authoring Atom User Guide | Section: Materials Mar 4, 2021
Copy link
Contributor

@santorac santorac left a comment

Choose a reason for hiding this comment

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

I noticed a few more things, some of it maybe new, some maybe things I missed in my first pass.

Copy link
Contributor

@sptramer sptramer left a comment

Choose a reason for hiding this comment

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

Looks like this PR was mangled somehow - we should figure out what happened to it on Monday AM. No need to stress before then.

@chanmosq chanmosq requested a review from sptramer March 7, 2021 21:37
Copy link
Contributor

@sptramer sptramer left a comment

Choose a reason for hiding this comment

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

Very minor suggestions, pretty much all copyedit. Can just click the accept suggestion buttons and I'll sign off!

content/docs/atom-guide/core-systems/materials/pbr.md Outdated Show resolved Hide resolved
- castle_brick_02_red_**rough**\_1k → castle_brick_02_red\_**roughness**


## Create material with Material Editor
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Create material with Material Editor
## Create a material with the Material Editor

- **AutoBrick**
A procedural brick material type that includes settings for color, ambient occlusion, brick (width, height, offset), and grout line (width, depth). This material type is an example of a completely custom material type, expanding from the MinimalPBR material type.

This file is located in *Gems/Atom/TestData/TestData/Materials/Types/AutoBrick.materialtype*.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This file is located in *Gems/Atom/TestData/TestData/Materials/Types/AutoBrick.materialtype*.
This file is located in `Gems/Atom/TestData/TestData/Materials/Types/AutoBrick.materialtype`.

@chanmosq chanmosq requested a review from sptramer March 7, 2021 22:37
For more information, see [Material File Specification](material-file-spec.md).

### Material File Example
In the following example, the material inherits from the **StandardPBR** material type and contains the properties `baseColor`, `roughness`, and `normal`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
In the following example, the material inherits from the **StandardPBR** material type and contains the properties `baseColor`, `roughness`, and `normal`.
In the following example, the material uses the **StandardPBR** material type and contains the properties `baseColor`, `roughness`, and `normal`.

### Download texture files
Download the [castle brick material](https://texturehaven.com/tex/?t=castle_brick_02_red) in any resolution from Texture Haven. When downloading, make sure to choose **All Maps** to download all textures that make up this material.

After downloading, move and extract the zip file to the folder *\<project-folder\>/Assets/Materials/TextureHaven/*.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the "Assets" folder. This isn't part of a standard project: <project-folder>/Materials/TextureHaven/


## Create material with Material Editor
To create a material using the Material Editor:
1. Open the Material Editor. If you are in the Open 3D Engine Editor, go to *Tools > Material Editor*, or press *M*. Otherwise, you can open the Material Editor as a standalone application from `\<build_folder\>/bin/profile/MaterialEditor.exe`.
Copy link
Contributor

Choose a reason for hiding this comment

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

The md here isn't showing up right, notice the extra slashes:
<\build_folder\>/bin/profile/MaterialEditor.exe


2. Create a new StandardPBR material by choosing **File** > **New** > **Standard PBR**. This opens the file browser and prompts you to save the new file. In the **Inspector** tab, you can verify that the material type is `StandardPBR` by checking the `Material Type` property in the `Details` property group.

{{< note >}}
Copy link
Contributor

Choose a reason for hiding this comment

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

the note md isn't rendering right

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not a standard .md tag. It renders with our Hugo site configurations.

Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
Update material images.

WIP PR fix.

Update content, syntax, and grammer

Rename folder.

Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
…3de file directory.

made edits, add headers

minor fix

added weights and minor edit

Fix PR: Dev suggestions, removed old comments, updated path files for o3de file directory

Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
…PBR material' to tutorials.

Signed-off-by: Chanelle <chanmosq@amazon.com>
Signed-off-by: Chanelle <chanmosq@amazon.com>
@erickson-doug erickson-doug merged commit 4361b87 into o3de:main Mar 9, 2021
@chanmosq chanmosq deleted the materials branch March 9, 2021 19:47
@chanmosq chanmosq self-assigned this Jun 12, 2021
chanmosq added a commit to chanmosq/o3de.org that referenced this pull request Nov 18, 2022
Signed-off-by: chanmosq <75444793+chanmosq@users.noreply.github.com>
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.

None yet

5 participants