Skip to content
GitHub no longer supports this web browser. Learn more about the browsers we support.
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

Check continuity on O2 cryogenic tanks before allowing stir #3

Closed
NoraCodes opened this issue Jul 7, 2016 · 216 comments
Closed

Check continuity on O2 cryogenic tanks before allowing stir #3

NoraCodes opened this issue Jul 7, 2016 · 216 comments

Comments

@NoraCodes
Copy link

@NoraCodes NoraCodes commented Jul 7, 2016

A customer has had a fairly serious problem with stirring the cryogenic tanks with a circuit fault present. To reproduce:

  1. Build CSM
  2. Perform mission up to translunar coast
  3. During translunar coast, attempt to stir cryo tanks

If a wiring fault exists, the issue may be replicated. Be aware that this may be hazardous to the tester attempting it.

@electron0zero

This comment has been minimized.

Copy link

@electron0zero electron0zero commented Jul 7, 2016

i think this issue should be closed because we can't reproduce it

@lbotinelly

This comment has been minimized.

Copy link

@lbotinelly lbotinelly commented Jul 7, 2016

Does it happens only with translunar coast (sol-3-a), or any moon coasting? It may be a problem with the moon. Just trying to narrow down the issue.

@pjanaya

This comment has been minimized.

Copy link

@pjanaya pjanaya commented Jul 7, 2016

Have you tried turning it off and on again?

@atpage

This comment has been minimized.

Copy link

@atpage atpage commented Jul 7, 2016

This is fixed in Apollo 14.

@adampats

This comment has been minimized.

Copy link

@adampats adampats commented Jul 7, 2016

This is a hardware problem. I suggest a wontfix and close.

http://history.nasa.gov/SP-4029/Apollo_13a_Summary.htm

In other words: "What was that gauge reading before you hit the switch?"

@NoraCodes

This comment has been minimized.

Copy link
Author

@NoraCodes NoraCodes commented Jul 7, 2016

@adampats Hey, don't tell me how to fly the CM, all right? They brought me in here to do a job, they asked me to stir the tanks, and I stirred the tanks!

@leommoore

This comment has been minimized.

Copy link

@leommoore leommoore commented Jul 7, 2016

As a temporary solution, you could try gently rolling the CM/LAM to agitate the O2. You may need to do it on the X and Y axis to get optimal agitation.

@chrislgarry

This comment has been minimized.

Copy link
Owner

@chrislgarry chrislgarry commented Jul 7, 2016

@SilverWingedSeraph could you provide a screen shot of the error I should expect to see during repro?

@AdamFreemer

This comment has been minimized.

Copy link

@AdamFreemer AdamFreemer commented Jul 7, 2016

Let's work the problem people.

@NoraCodes

This comment has been minimized.

Copy link
Author

@NoraCodes NoraCodes commented Jul 7, 2016

@chrislgarry It looks a little something like this:

screenshot_20160707_104443

screenshot_20160707_104357

@khash

This comment has been minimized.

Copy link

@khash khash commented Jul 7, 2016

Which version of Apollo are you running? I suggest upgrading to Apollo 12

@lbotinelly

This comment has been minimized.

Copy link

@lbotinelly lbotinelly commented Jul 7, 2016

Not specifically for this issue, but maybe we should consider trimming down. I'm pretty sure we can automate this, eliminating both meatbags and O2 tanks from the final implementation.

@Wazzaps

This comment has been minimized.

Copy link

@Wazzaps Wazzaps commented Jul 7, 2016

Have you tried migrating your project to jQuery?

@AdamFreemer

This comment has been minimized.

Copy link

@AdamFreemer AdamFreemer commented Jul 7, 2016

Many bug fixes were addressed in Apollo 18 mission software, until the bugs infected the astronauts.

@predicat

This comment has been minimized.

Copy link

@predicat predicat commented Jul 7, 2016

It works for me

@lparkermg

This comment has been minimized.

Copy link

@lparkermg lparkermg commented Jul 7, 2016

So, I think it might be a hardware issue. Have you tried taking the tanks out and putting them back in again? Maybe that will fix it.

@g0ddish

This comment has been minimized.

Copy link

@g0ddish g0ddish commented Jul 7, 2016

@atpage Savage

@stkrzysiak

This comment has been minimized.

Copy link

@stkrzysiak stkrzysiak commented Jul 7, 2016

We're going to need a label for good first bug, agreed?

@lparkermg

This comment has been minimized.

Copy link

@lparkermg lparkermg commented Jul 7, 2016

@stkrzysiak It's got to be a hardware tag if what @atpage said is true.

@autoferrit

This comment has been minimized.

Copy link

@autoferrit autoferrit commented Jul 7, 2016

@Wazzaps
Have you tried migrating your project to jQuery?

we have tried converting to node. heres a mockup.

http://svtsim.com/moonjs/agc.html

@AstraSerg

This comment has been minimized.

Copy link

@AstraSerg AstraSerg commented Jul 7, 2016

Which exactly Universe do you use when face this bug?

@jerry-bachmann

This comment has been minimized.

Copy link

@jerry-bachmann jerry-bachmann commented Jul 7, 2016

Just had this issue happen to me, I took a screenshot.

apollo-13

@JohnTendik

This comment has been minimized.

Copy link

@JohnTendik JohnTendik commented Jul 7, 2016

Did you try uninstalling the drivers then reinstalling them?

@bstiller1

This comment has been minimized.

Copy link

@bstiller1 bstiller1 commented Jul 7, 2016

Looks like there could be an ID10T error, along with some memory leaks.

@ireun

This comment has been minimized.

Copy link

@ireun ireun commented Jul 7, 2016

Well, can I just update the firmware to fix this issue? Or do I have to take my Apollo to local NASA-spacecraft dealer?

Really frustrating!

@AndrewFeeney

This comment has been minimized.

Copy link

@AndrewFeeney AndrewFeeney commented Jul 7, 2016

This software has reached end of life and is no longer receiving security updates. NASA strongly recommends that you migrate to a supported version. Unfortunately no such version exists at this time.

@jerry-bachmann

This comment has been minimized.

Copy link

@jerry-bachmann jerry-bachmann commented Jul 7, 2016

Ok I'm in a real pickle here, after you experience this issue apparently you can't simply reboot to fix it. Does anyone have a good workaround? Also my CO2 is getting pretty high, please answer quickly this is a production issue for me.

@kiweb

This comment has been minimized.

Copy link

@kiweb kiweb commented Jul 7, 2016

It's not a bug, it's a feature.

@luislobo

This comment has been minimized.

Copy link

@luislobo luislobo commented Jul 7, 2016

Well, I run the unit tests and it says 0 errors...

@AnthonyRinaldi

This comment has been minimized.

Copy link

@AnthonyRinaldi AnthonyRinaldi commented Jul 7, 2016

Did you try SCE to AUX?

@ShariqKhan2012

This comment has been minimized.

Copy link

@ShariqKhan2012 ShariqKhan2012 commented Jul 16, 2016

I am on Mars for a client assignment right now. So it will be a while before I look into the issue.
Also, I note that you are using a legacy setup that we no longer actively develop.
We will however maintain backward compatibility till 31st Dec 2016. This is part of our strategy to make a streamline move towards cost-effective SpaceX components.
We advice you to upgrade your infrastructure to reflect the same.

That said, we do have a patch for your problem but it wont be until the next release that you can see the fix.
However, if it is very critical for your project, then kindly send over your time signature (in Unix timestamp format) and the value of gravity at your test facility, and one of our engineers will travel back in time to examine and fix the issue.

Thanks.

@panarchist

This comment has been minimized.

Copy link

@panarchist panarchist commented Jul 16, 2016

Is this happening in sandbox, or a production environment? Engineering is saying the described behavior is impossible, it's got to be an instrumentation issue.

@kaleemullah360

This comment has been minimized.

Copy link

@kaleemullah360 kaleemullah360 commented Jul 17, 2016

Leave it.
until they provide hardware too :)

@yPhil-gh

This comment has been minimized.

Copy link

@yPhil-gh yPhil-gh commented Jul 17, 2016

Your call might be recorded for quality and training purposes

@vickz84259

This comment has been minimized.

Copy link

@vickz84259 vickz84259 commented Jul 18, 2016

You forgot to import apollo_11.cms...

@Domminique

This comment has been minimized.

Copy link

@Domminique Domminique commented Jul 18, 2016

let me check it out

Regards,

Dominic A. Imbuga *
Blogs: http://imbuga.blogspot.co.ke || imbugad.wordpress.com
*Fullstack Developer | Blogger

Github:Domminique
Twitter:@dominicimbuga

Skype:Dominic-imbuga_+254 715 180 530 | +254 734 983 442_

_" _Science and technology revolutionize our lives, but memory, tradition
and myth frame our response"

On Mon, Jul 18, 2016 at 9:48 PM, Victor Otieno notifications@github.com
wrote:

You forgot to import apollo_11.cms...


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#3 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGX1iZKhwYKY2mTiS8n9ezBpcATSgQ_4ks5qW8prgaJpZM4JHJnx
.

@dreamyguy

This comment has been minimized.

Copy link

@dreamyguy dreamyguy commented Jul 20, 2016

🚫 USER ERROR 🚫

@ahmadawais

This comment has been minimized.

Copy link

@ahmadawais ahmadawais commented Jul 20, 2016

I just solved the issue. Here's how, have a listen!

🔘──────── 00:21

@Pablo-Ivan

This comment has been minimized.

Copy link

@Pablo-Ivan Pablo-Ivan commented Jul 26, 2016

I suggest assigning Stanley Kubrick to fix this issue.

@chrislgarry chrislgarry added the humor label Aug 6, 2016
@pantaluna

This comment has been minimized.

Copy link

@pantaluna pantaluna commented Aug 17, 2016

Which version of Apollo are you running? I suggest upgrading from Apollo 11 to Apollo 12

@khash We cannot upgrade simply because by our 60's semver rules the V12 contains "breaking" changes.

@humantorch

This comment has been minimized.

Copy link

@humantorch humantorch commented Sep 15, 2016

This never would have happened when Steve Jobs was alive.

@sdhar86

This comment has been minimized.

Copy link

@sdhar86 sdhar86 commented Sep 15, 2016

And that's why we should use docker, could have reproduced the issue easily.

@cristianmedeiros

This comment has been minimized.

Copy link

@cristianmedeiros cristianmedeiros commented Sep 16, 2016

works on my machine

@aniruddhas

This comment has been minimized.

Copy link

@aniruddhas aniruddhas commented Sep 17, 2016

You guys are trying to solve an issue which doesn't exist in future.
The Doctor has already solved it in past future.
In his words:
"People assume that time is a strict progression of cause to effect, but actually from a non-linear, non-subjective viewpoint - it's more like a big ball of wibbly wobbly... time-y wimey... stuff."

So, let's stop being geeky and go back to humour, ehh?

@brossow

This comment has been minimized.

Copy link

@brossow brossow commented Jan 24, 2017

Unless someone can point to the specific user story in the functional design document that proves this is not working as intended, this should be considered a billable change request. Please submit this to the internal review board for discussion at next month's meeting.

@timlindsay

This comment has been minimized.

Copy link

@timlindsay timlindsay commented Feb 5, 2017

if we split the tank and stirrer into separate micro-services and implement some sort of ESB, then at least we can deal with faults in isolation. I highly recommend [flavour of the day] hosting service, in my experience they have been A1 and their 24hr support is awesome.

@Silmar

This comment has been minimized.

Copy link

@Silmar Silmar commented Apr 3, 2017

There is a plugin for jQuery that fixes it. Google4it.

@1337ninja

This comment has been minimized.

Copy link

@1337ninja 1337ninja commented Apr 5, 2017

And that's why we have not sent anyone to the moon for 47 years! Please use upcoming Android O which comes with blazing vibration support for stirring (android.os.Vibrator.vibrate())

@orcaman

This comment has been minimized.

Copy link

@orcaman orcaman commented Apr 5, 2017

I think it's time we re-write this node.js code in golang.

@brianmay

This comment has been minimized.

Copy link

@brianmay brianmay commented Dec 2, 2018

Why hasn't anybody submitted a PR to fix this issue yet?

@j-degreef

This comment has been minimized.

Copy link

@j-degreef j-degreef commented Mar 26, 2019

This never would have happened when Steve Jobs was alive.

Who ?

@Stevie-C

This comment has been minimized.

Copy link

@Stevie-C Stevie-C commented Sep 3, 2019

This is just like that time you complained that the autopilot's "hold altitude" function caused a 200 foot per minute descent. No matter how hard I try, I have yet to successfully reproduce this problem on the ground!

@brianmay

This comment has been minimized.

Copy link

@brianmay brianmay commented Sep 3, 2019

Don't understand why this was tagged "Won't fix" and "Humour". Seems to me you don't appreciate the seriousness of this report.

@theodesp

This comment has been minimized.

Copy link

@theodesp theodesp commented Sep 3, 2019

@brianmay what do you expect? They are the same people that worked on the Apollo13 and we all know what happened...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
You can’t perform that action at this time.