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

MAPS, WADS: New e1m5 replacement: Gloomliths #173

Merged
merged 3 commits into from
Oct 5, 2024
Merged

Conversation

Zungrysoft
Copy link
Collaborator

@Zungrysoft Zungrysoft commented Oct 2, 2024

Description of Changes


Introduces a new replacement for e1m5 called Gloomliths. Taking place within the winding halls of two monolithic black stone structures, this map has tons of secrets and puzzles to unlock secret items and alternate paths. There is in fact an entire second path through the level you can access right from the start.

Also added new textures for this level.

This replaces the previous e1m5 map Reading of the Dead. I have moved it into the Episode 1 archive folder for now.

Visual Sample


fte-20241001230137-0
fte-20241001230150-0
fte-20241001230204-0
fte-20241001230233-0
fte-20241001230258-0
fte-20241001230309-0
fte-20241001230319-0
fte-20241001230333-0

Checklist


  • I have read the LibreQuake contribution guidelines
  • I have thoroughly tested my changes to the best of my ability
  • I confirm I have not contributed anything that would impact LibreQuake's licensing and usage
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible

Copy link
Collaborator

@Nolcoz Nolcoz left a comment

Choose a reason for hiding this comment

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

After compiling the map with the branch, it seems that at the start there is a missing texture here.
e1m5_2024-10-02_22-51-12
Apart from that, everything seems fine

@MotoLegacy
Copy link
Collaborator

After compiling the map with the branch, it seems that at the start there is a missing texture here. e1m5_2024-10-02_22-51-12 Apart from that, everything seems fine

Weird, our CI should've flagged this. @pnahratow can you investigate?

@Zungrysoft
Copy link
Collaborator Author

@Nolcoz Whoops. I just fixed it.

Copy link
Collaborator

@Nolcoz Nolcoz left a comment

Choose a reason for hiding this comment

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

Yep, fixed. Thanks!
e1m5_2024-10-03_01-05-17

@pnahratow
Copy link
Collaborator

Weird, our CI should've flagged this. @pnahratow can you investigate?

It shows a warning. My guess is that the warning doesn't make qbsp fail. I could either try to find out if there is some fail-on-warnings in ericw-qbsp or I could parse for the WARNING string manually.

@lavenderdotpet
Copy link
Owner

lavenderdotpet commented Oct 3, 2024

Weird, our CI should've flagged this. @pnahratow can you investigate?

It shows a warning. My guess is that the warning doesn't make qbsp fail. I could either try to find out if there is some fail-on-warnings in ericw-qbsp or I could parse for the WARNING string manually.

could maybe do a qbsp | grep "warning" and some how out put that to the git build process

@MotoLegacy
Copy link
Collaborator

Weird, our CI should've flagged this. @pnahratow can you investigate?

It shows a warning. My guess is that the warning doesn't make qbsp fail. I could either try to find out if there is some fail-on-warnings in ericw-qbsp or I could parse for the WARNING string manually.

Thanks for taking a look. Can I assign an issue to you for this?

@RandomBrushes
Copy link
Collaborator

Also had a playthrough. It's a very fine map that fits the other e1-maps better than the current e1m5.

As usual, I didn't find any secrets and thus missed half the enemies, but surprisingly didn't get lost.

Visually, I guess some parts are looking very blocky and that gray stone wall texture is really carrying most of the structual load, but the mood is excellent and I guess some visual palette cleaners can be integrated later on.

I'd say things are very merge-able.

@pnahratow
Copy link
Collaborator

Weird, our CI should've flagged this. @pnahratow can you investigate?

It shows a warning. My guess is that the warning doesn't make qbsp fail. I could either try to find out if there is some fail-on-warnings in ericw-qbsp or I could parse for the WARNING string manually.

Thanks for taking a look. Can I assign an issue to you for this?

No

#174

@lavenderdotpet
Copy link
Owner

this ready to merge?

@Zungrysoft Zungrysoft merged commit 7b7ab38 into main Oct 5, 2024
12 checks passed
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.

6 participants