Full-resolution silent pictures (silent.mo)

Started by a1ex, July 01, 2014, 05:11:15 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Levas

@NikFreak, Thanks for helping :D

Downloaded the build in the link, but the modules folder only contains one file (6D_113.sym) for the rest it is empty...that's not good is it ?
I expected at least a silentpics module and raw_diag module.

Can you take a look at it ?

Happened before, a build with no modules, audionut said this about it:

Quote from: Audionut on July 16, 2014, 04:21:25 PM

@nitfreak, for future reference, run make in the source directory, then browse to the required platform directory (6D.113 in this case), and run make zip.
This will compile the modules and produce a nightly type zip in the platform directory.



nikfreak

Yeah I did it like Audionut instructed. But you are right about missing modules. They don't get added cause of some missing rst2html:


Building module silent...
REBUILDING
[ README   ]   module_strings.h
cat README.rst | grep -v -E "^:([^:])+:.+$" | rst2html --no-xml-declaration | python ../html2text.py -b 700



/bin/sh: 1: rst2html: not found

(<type 'exceptions.SystemExit'>, SystemExit(1,), <traceback object at 0x2ad947187368>)
make[4]: *** [module_strings.h] Error 1
[size=8pt]70D.112 & 100D.101[/size]

Levas


nikfreak

[size=8pt]70D.112 & 100D.101[/size]

Levas

Downloaded your build, it has indeed modules now  :D

RAW_DIAG module is not in it, downloaded RAW_DIAG module from the first page of this thread and put in the module folder.
Unfortunately the RAW_DIAG is not loaded at startup, it says, "OldAPI" instead of "OK",
And displays, "Wrong version, (v5.0, expected v6.0)" 
So at this moment I can't do the test Alex asks for...

Is this message for RAW_DIAG module "Wrong version, (v5.0, expected v6.0)" , something you can fix, Nikfreak ?
Or do we need Alex to ask what's going on (Alex is probably working on a newer RAW_DIAG, V6.0, but it's not yet online ?)


nikfreak

I have really no clue. I compiled "fullres-silent-pics" branch and what's that about v5.0 / 6.0 I don't know cause I didn't try myself. If a1ex tells what to do I will do and we can do the requested test for 6D. Sorry still freshman over here....
Btw that raw_diag module from 1st page is prebuilt. I have no idea where the sources are so that I could increase a version number in source and compile myself

@a1ex: This is what I did: http://magiclantern.fm/forum/index.php?topic=12523.msg124160#msg124160

Where did I wrong? Can you teach plz?
[size=8pt]70D.112 & 100D.101[/size]

Levas

No problem, already glad you could help this far  :D

@A1ex, Can you help us out here ?

The RAW_DIAG module on page one of this thread does not work with currently compiled builds.
The module won't load at startup, OldAPI.
It shows the message "Wrong version, (v5.0, expected v6.0)" 




ayshih

The version of the module API was bumped up a few days ago (this commit) due to the expectation of bugs resulting from changes in FIO return values.  That's why previously-built modules will no longer run.

You should be able to build a new raw_diag.mo yourselves from the iso-research branch, after first locally merging it with the current unified.  If merging is difficult, you might be able to just copy over the raw_diag source files from that branch to the current unified and build it that way.
Canon EOS 50D | 17–40mm f/4L & 70–300mm f/4.5–5.6 DO IS | Lexar 1066x

nikfreak

Thx for your response. Merging gets me headache. i am still uncomfortable with hg mercurial compared to using git / github. I tried copying raw_diag into unified and tried compiling as you suggested but that gave me compile errors. Dunno at least me and Levas tried. Hopefully this is honored and 6D won't be thrown out. Maybe ayshih you can instuct me or upload a raw_diag.mo for us so we can use my built zip from "fullres-silent-pics" branch from today which I compiled for 6D??
[size=8pt]70D.112 & 100D.101[/size]

ayshih

Yeah, I'm much better at Git than Mercurial too.

I won't be back at my ML build environment for many many hours, so hopefully someone else can figure out how to get raw_diag.mo to build in the meantime.  If you'd like, you can PM me the compile errors you're getting, but I may not be able to diagnose them from remote.
Canon EOS 50D | 17–40mm f/4L & 70–300mm f/4.5–5.6 DO IS | Lexar 1066x

a1ex

Updated the link, should be OK now.

You can simply compile from iso-research branch and take the .mo file; the branch is up to date.

Levas

Thanks, Nikfreak, Ayshih and A1ex!
It works finally  :D

The Cyan box fits perfectly inside the noisy borders (for 6d that is)
.
See "OB-Zones-6d-31Juli2014.png" on my google drive
https://drive.google.com/folderview?id=0B1BxGc3dfMDaazVGRF9JOTF2VzQ&usp=sharing

Although, I don't get the bright noise on the bottom left in the white box, what's that A1ex ?

a1ex

Looks alright. Can you also take a screenshot showing some overexposed raw zebras?

Levas

How do I take a Screenshot of the zebra's ?

I Did try to use the "screenshot -10s" option in the Debug menu, but the zebra's don't show up on the screenshot written on the memory card...

The zebra's do show up on my camera LCD screen, but their not visible in the screenshot ?

Levas

Wait, problem solved.
Zebra's were on Luma, now they're on RGB and they show up in the screenshot.

Levas

For Raw zebra's screenshot from the 6d:
Check my google drive for "Raw-Zebras-6d.png"
https://drive.google.com/folderview?id=0B1BxGc3dfMDaazVGRF9JOTF2VzQ&usp=sharing


Edit, wrong screenshot

a1ex

Zebras must be RAW, not YUV.

Also, they must be in photo mode, not LiveView. Either the silent full-res or the regular photo mode, they are the same.

Levas

Took 2 pictures the normal way and with picture review it makes a colored overlay of the overexposed areas.

See "Raw-Zebras-6d.png" and "Raw-Zebras-6d-.png"
https://drive.google.com/folderview?id=0B1BxGc3dfMDaazVGRF9JOTF2VzQ&usp=sharing

a1ex


Levas

Whoehoe!

So we're done (for) now  8)

Full-res-silent pics will be introduced in the nightly builds soon and Canon 6d will be in those nightly builds  :D

Thanks everyone who helped

a1ex

Now you're gonna kill me: I've deleted a bunch of code, is it still working?

https://bitbucket.org/hudson/magic-lantern/commits/5925e186ed67

You no longer need to check any offsets; if there are problems with the new method, silent pictures and zebras would be totally broken (easy to notice, unlike the possible problems caused by wrong offsets).

nikfreak

Still the skip offsets were worth it plus I learned to correct some mistakes
[size=8pt]70D.112 & 100D.101[/size]

Levas

Quote from: a1ex on July 31, 2014, 08:22:25 PM
I've deleted a bunch of code, is it still working?

If it ain't broken, don't fix it! 
  >:(
No kidding  ;D , you probably had a good reason for it, like cleaner code...

Do I have to test some stuff or are you asking Nikfreak to look at the code ?

nikfreak

Same game. Fullres-silent-pics branch needs to be compiled and retested. we do this tomorrow ok?
[size=8pt]70D.112 & 100D.101[/size]

a1ex

Yeah, but this time there's nothing special to test - just see if it works.