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

AI Photography #1357

Merged
merged 7 commits into from Sep 17, 2013
Merged

AI Photography #1357

merged 7 commits into from Sep 17, 2013

Conversation

AndroidSFV
Copy link

Gives the AI two verbs, Take Picture and View Picture. The AI can also print these pictures at a photocopier if there is enough toner, and on a newscaster. The AI is given an ai_camera in New(). The picture is taken centered on the AI's eyeobj. I broke up some of the original photography code so that it could be used with the AI camera.

Added some checks to photocopier.dm that when the AI tried to eject a photo from a photocopier, it would end up in the AI's hand. Also increased the amount of toner a photocopier holds (and how much a toner cartridge adds) to 40. I also gave a toggle option for non-ai mobs to copy in greyscale or color.

Consolidated some datacore code into datacore.dm.

To come in future pulls: A way for AI's to delete pictures they don't like so it isn't cluttered, targeting of the center of the picture based on a mouse click, giving this functionality to borgs, unifying picture storage/viewing among all silicons, making the selection interface into some form a window with html interaction.

Please offer any feedback or criticism in a detailed manner so I can address it properly.

Notes: code/datums/obj.dm and /code/defines/obj.dm, where I found those bits of datacore code, do not really serve a purpose and should have their contents shifted into more appropriate .dm's. Also, when you try to take a picture of the kitchen, the kitchen turfs appear black in the picture. I will open issues for both.

Also, a big thanks to Nodrak and Giacom for providing invaluable advice and assistance.

@GunHog
Copy link
Contributor

GunHog commented Sep 13, 2013

Still new to Git. How do I test this without mucking up my /tg/ Repo?

@caelaislinn
Copy link
Contributor

Here's a direct download https://github.com/AndroidSFV/-tg-station/archive/AIphotography.zip

Alternatively just pull from his repo

@Petethegoat
Copy link
Contributor

@AndroidSFV The photo issues are documented and explained in an existing issue report(s)- Baystation12/Baystation12#2725

There's no good way to fix them, that I know of, but you're welcome to look into it. Might be worth asking about it on the byond forums.

@Cheridan
Copy link
Contributor

@GunHog you can easily test pull requests with https://gist.github.com/Giacomand/bbe7c557d329fdb680c4

@GunHog
Copy link
Contributor

GunHog commented Sep 14, 2013

I modded the file as instructed...but I do not know what "fetch from the upstream for your repo" means. You do not need to bother holding my hand through this, I will just...try stuff until something happens. Thanks for the link though, it is quite useful to someone who knows how to use GitHub properly.

@caelaislinn
Copy link
Contributor

Upstream is the internet repository which you originally cloned from, but you can change that if you want. Fetching is a way of getting the updates, but not putting them into your branch (the commits are stored in a special branch called FETCH_HEAD).

Basically you just want to update your local branch with -tg-station:master

@@ -218,13 +248,79 @@
P.icon = ic
P.img = temp
P.desc = mobs
P.pixel_x = rand(-10, 10)
P.pixel_y = rand(-10, 10)
// P.pixel_x = rand(-10, 10)
Copy link
Contributor

Choose a reason for hiding this comment

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

IIRC This makes it so when photos are printed they're offset randomly on the tile, so they're not hidden underneath each other.
It'd be better to leave it in

Copy link
Contributor

Choose a reason for hiding this comment

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

Yup, that's what they do. Uncomment these please!

@Metacide
Copy link
Contributor

This sounds like an unbelievably wonderful thing, thank you for this.

@AndroidSFV AndroidSFV mentioned this pull request Sep 17, 2013
Cheridan added a commit that referenced this pull request Sep 17, 2013
@Cheridan Cheridan merged commit 12b1de9 into tgstation:master Sep 17, 2013
LemonInTheDark pushed a commit to LemonInTheDark/tgstation that referenced this pull request Dec 12, 2021
…on#1357)

* Adds feedback when successfully emaging a generator (tgstation#62524)

* Adds a span_notice when emaging a PACMAN

* Adds feedback when successfully emaging a generator.

* Adds feedback when successfully emaging a generator

Co-authored-by: Stonetear <89315023+Stonetear@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

7 participants