@amitit is this even compiling? you're returning an `error` from a function that can only return the `Handler` type based on the signature.


Forge 1395, OptiFine_1.7.10_HD_U_B6
No Progress screen and crash:
crash-report - https://gist.github.com/Bogdan-G/3429db1f362a8428603a
Loading screen debug info - https://gist.github.com/Bogdan-G/fe00e6907712f63d09d7


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


There's nothing like a reference to a reference in PHP. It's the same value… but the & is unnecessary in this case.


@amitit forget that, github diff view made me believe you're returning from NewHandler here


gofmt please :)


`$initialClone´ is not available in this block


as you've pointed out in commit comment, the other issue is that the `assert()` raises an exception which is then caught and `pass`-ed

the code there should be something like this:
```
try:
    connection.handshakeServer(...)
except TLSException:
    pass
else:
    assert()
```

I'd prefer to keep `TLSRecordLayer.fault`-related code to minimum in non test classes, so I don't like the below solution


Not able to test the code right now (will be able later today), but from a brief look I am afraid this could introduce problems with accessibility – you are animating alpha from 0 to 1, for this to work you removed the `webViewContainer.hidden = true` on , but this needs to be set in order for accessibility not being able to access the webView when home panel is shown – could you add a completion handler to the alpha animation that would set the `hidden` to `true` there after the animation finishes (which should be fine then)?


Thank you :). I thought it could be useful info :).
Do you watch all my commits? :)


Yes, I think you are correct, I should leave this part as it is, so that we continue to test `__eq__` and `__ne__`. Similar lines e.g. L334 should probably be left unchanged as well, otherwise I am changing the meaning of the test.


-1
This is a matter to be decided collectively, not in a unilateral commit. If you advanced a usability argument, that would be one thing. But marketing strategy is not decided in individual core commits.


I just copied it from the line above, but it seems you're right. The old tests were broken too. (This seems to be a common theme. I spent quite a long time before realizing that the fault-based "test framework" was a no-op and never tested anything.)


Thx, but I am now confused about the DeepCL, should i make a pull request to the 4.x.x version?


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1133 of 1133 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


`$future` could be `null` at this point


Missing Full Stop


@emord I did this same fix in https://github.com/dimagi/Vellum/commit/381ffa577b845b3a7a9dfa3dbdaafa3df33c20b0, which is on the copy-paste branch. I forgot to merge copy-paste back into staging after writing that fix though. Looks like that needs to be done anyway since there are other changes on copy-paste that will not go to staging if a new version of vellum is pushed to staging.


the overall assignment is covered in README.md


like I said, feel free to revert. For things that seem obvious, I'm not interested in wasting time on process though.

In any case, since this has already provided enough drama, let's revert it.


this commit fixes #3


I could leave opts out of the params altogether and just operate on the typeof rate or callback. I thought it was nice to have a placeholder, but I think in practice if someone is using opts it will probably be the only parameter passed.


`$cleanUp` vs. `$cleanup`on line 196


You, as always are correct. I forgot to delete it. I can create another PR if you like.


`$auth` is not available here


Möchtest Du Leerzeichen um `==` spendieren, damit der Quellcode einheitlich formatiert ist?


So now, you need to think always about 3 levels for a spell, so 3 textures from the lesser powerful to the most powerful spell
you can play with size and details :)


Commit


try to hack #27


What?


test


nice


Testing. That's all. I love to tinker.


I certainly agree that a properly designed test system would not inject so much fault code into the real implementation. (It shouldn't inject any! BoringSSL intentionally keeps all injected faults in a completely independent TLS implementation.) But this is the framework that exists. I think, if the weird `Fault.faultAlerts` framework is to go (which it definitely should), that should be a follow-up, since I think some of the other tests are relying on that machinery (or would have been if this framework worked). This commit is to make these tests do anything at all.


マージします


Why was this change introduced? :((


Thanks!


I think these lines are missing trailing semi-colons as well.


You could use the AsyncScheduler for this instead of your own thread.


Entspricht die Formatierung den Konventionen im Upstream-Projekt? Persönlich finde ich, dass man nur mit `/` ohne irgendeine Einrückung erst zweimal gucken muss, um zu sehen, dass die Zeile fortgesetzt wird.


This causes migration to fail when running in production mode for Tutor:

stderr: rake aborted!
LoadError: cannot load such file -- factory_girl_rails
/home/osttutor/src/tutor-7d0d354/vendor/bundle/ruby/2.1.0/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:274:in `require'
/home/osttutor/src/tutor-7d0d354/vendor/bundle/ruby/2.1.0/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:274:in `block in require'
/home/osttutor/src/tutor-7d0d354/vendor/bundle/ruby/2.1.0/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:240:in `load_dependency'
/home/osttutor/src/tutor-7d0d354/vendor/bundle/ruby/2.1.0/gems/activesupport-4.2.0/lib/active_support/dependencies.rb:274:in `require'
/home/osttutor/src/tutor-7d0d354/lib/tasks/sprint/010mix.rake:1:in `<top (required)>'


So we do not want to submit a pull request to upstream, then?


Foto


Hi Benjamin,

here's were I stand at the moment:
* I don't want to hold your progress
* I've only briefly looked into docstrap
* I'm not yet clear how to separate the user and the developer documentation

Personally, at this stage I don't see much benefit in using docstrap, but if you feel it will help you, let's go for it. Could we discuss first how to separate the user and the developer documentation? I would like the landing page to focus on the needs of IJavscript users (something similar to the root `README.md` in github).

Nico


sorry about that. I'm going to find a global solution for this now. It needs to be over git as IntelliJ won't play along.


There is some weird interaction going on. Maybe try this @teonlamont ?

```py
from mne.report import Report
from mne.datasets import sample
from mne import read_evokeds, read_cov
report = Report(verbose=True)

path = sample.data_path()
fname = path + '/MEG/sample/sample_audvis-ave.fif'
evoked = read_evokeds(fname, condition='Left Auditory', baseline=(None, 0),
                      verbose=True)
noise_cov = read_cov(path + '/MEG/sample/sample_audvis-cov.fif')
evoked.plot_topomap(show=False)
evoked.plot_white(noise_cov=noise_cov, show=False)
```


Do you prefer it the other way?


when will this be released?


@jlucerond done with my review. checkpoint complete.


hehe ok :p just asked to learn


Yeah :D It was actually trailing initially, but we switched to leading dot once ruby 1.8 support was dropped... IMO leading dot makes it a bit easier to read.


What should I see? Cannot reproduce the problem.


+1
btw, looks like this plugin is supported only for Linux, it will be very nice to have it for AIX and Solaris as well.


this can be unit tested inside `HandshakeSettings` and in general tests by monkey patching, without adding another ugly fault condition

r-


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (345 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.71%25-yellow.svg)


Remove that stray line.

Also, put shlibs changes for a different package into a different patch.


No figure pops up?


Interessante Änderung.


Es macht einfach einen besseren Eindruck, wenn der Code von der Formatierung einheitlich ist.

# Whitespace nach Komma fehlt
# Mal hast Du bei Blöcken mit geschweiften Klammern Whitespace innerhalb, mal nicht. Mal hast Du Whitespace vor der öffnenden Klammer, mal nicht. Alleine in den Zeilen 79-82 kommt das alles vor.


personally, i'd suggest you switch these two conditions:
```
    if (room >= (header.max_size >> 1) &&
        room - size < (header.max_size >> 1)) {
```

so one can interpret it as

> before this write, the size is smaller than half of max, after this write the size will pass the half full mark.

what do you think?


"maybe consider moving the imported interfaces (IReservedSpace etc.) into the API package?"
It shouldn't be any different really, because of Maven. You're getting the entire mod with that (and you shouldn't have the IGW-Mod API in your jar anyway :P).

"Oh, and one more small suggestion: maybe change the interpret method to include the name of the page? That could make things a good bit easier to detect, because one could just go by file ending, e.g. (unless page names don't include file ending?)"
File ending? Like .txt? All pages should end with .txt... I was thinking that users would add '[opencomputers]' or a tag like that in the beginning of the file, which your ITextInterpreter would look for and would strip off before interpreting or something like that. But your idea might work too... but at the moment it's all .txt though..


@jlucerond done with my review. checkpoint complete. great to see you testing ideas.


Although, I'm OK to stick with trailing dot instead of leading, if that makes things easier to others :D


Auch hier: `=>` mit oder ohne Whitespace darum?


Dodaj sobie wszystkie pliki z folderu "./.idea" do gitignore. Mozesz to zrobic przez IntelliJ, nie musisz ręcznie edytowac pliku .gitignore.
Ogólnie nic poza pom.xml i zawartoscią folderu src się nie commituje, chyba, że na prawde potrzeba (no... w githubie jeszcze plik README.md, bo się wyswietla na stronie głównej repo, nie wiem jak w innych repach).
Czyli szybko podsumowując - foldery:
./target
./.idea
i inne które nie są plikami które tworzyłeś - dodajesz do .gitignore


To tez do .gitignore


Bocs, itt tényleg arról van szó, hogy egy RMNY-ben van egy kézzel írott vers, viszont egy másik megfeleltetésben tényleg hibásan szerepelt az RMNY, azt már javítva töltöttem fel. Most már remélem, hogy ez a helyes változat.


I think the only change we need is to change setCredit_card to setCreditCard, I'm using facade , for example amount function will return an instance of Amount class ..


Could you add this class into `tutor.less` and make it big and [blinking](https://stackoverflow.com/questions/16344354/how-to-make-blinking-flashing-text-with-css3/16344548#16344548) (or marquee)?


Cos I don't have old the mappings lah..


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (346 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I did use Snapshot 75, or rather, I built from the same commit that committed Snapshot 75.

I don't know how to do any video recording, and it will be a while before I can even start researching how to do it.


@kennethlimcp please see this screenshot taken a moment ago:

http://i.imgur.com/7BsZSdy.png

The setup/loop methods and variables are still present in the Spark version, and they can collide with projects that use this Stepper library as a dependency.


Poczytaj o wykorzystywaniu String.format i o StringBuilder (zamiast ręcznego sklejania stringów możesz wykorzystac któres z nich). Dodatkowo zapoznaj się z różnicą między StringBuilder i StringBuffer - popularne pytanie


Fixed.


Hast Du irgendeine Konvention, wann Du Single-Quotes `''` und wann Double-Quotes `""` nimmst?


From my point of view the whole `fault` framework is deprecated, and it needs to go as soon as the library has proper unit test coverage.

We won't make its removal easier by adding more of code that depends on it to main library.


No Github is ok. I will look at it this week.


Auch wieder ein Beispiel ohne Whitespace nach dem Komma. Im else-Zweig drei Zeilen später gibt es dann wieder Whitespace.


Yep, this is the correct fix.  Added benefit is that it no longer needs to iterate through `allFrameworks` though.


Oops, I missed another one here. Line 514 should also be `javascript`.


De ce se cheamă proprietatea asta index?


ah yes. thanks


Varför for-loopar ni inte bara laddningen av ljud-effekter?


Niepotrzebnie zmienną sum inicjalizujesz zaraz przed zwróceniem jej. Nie widze innego miejsca gdzie są wykorzystane w klasie. Możesz spokojnie zwracać wartość na żywca z JTextFielda


`index` ăsta e același cu field-ul de mai sus sau e altfel sau cum?


this could be just `scceed` instead of `when`?


Yes, no figure. Note that you may not use IPython in pylab mode. Show will always be true otherwise.


Ich frage mich gerade, was der Vergleich mit `0` macht, aber inhaltlich habt Ihr das schon gereviewt, oder?


@jogleasonjr , seems like the version was not bumped. Just did it so please check ;)


Jag är inte en gud som du eller stoffe okej chilla ska fixa det vänta bara
okej
 Den 27 apr 2015 16:54 skrev "Tom Leonardsson" <notifications@github.com>:

> Varför for-loopar ni inte bara laddningen av ljud-effekter?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DriverInput/lbs-game-awards-2015/commit/49e0978b7cb78357217f9a6bbe7f47298d2737eb#commitcomment-10917766>
> .
>


@Omegatcu well done mate :)

we should also add the `offcanvas` at the bottom with the "Mobile Menu" particle so it can be there by default.

I tried the following codes but I cannot figure out what is the one for the "Mobile Menu" particle:

```
offcanvas:
    - mobile-menu
```

and

```
offcanvas:
    - mobile_menu
```


Ah yep, that slipped through when I did the commit in my client, thank you.
Yes, you're right with my committing behavior. This project is the first time I've truly used GitHub or similar and it wasn't too important as not that many people were interested in this repo. I'll try to adjust.


could be `succeed` instead of `when`


Hmmm... well... plausibly I should have merged into 4.x.x, and then merged from 4.x.x into my current dev branch.  I guess that if I do merge into 4.x.x now, that any conflicts are easy-ish to deal with, so I can merge into 4.x.x if you want?


Aș vrea să pot folosi lista asta pentru mai mult de 50 de elemente.


That's one of the reasons why I'm asking for unit tests, and not those connection tests.

Yes, they are useful as a sanity check, but they shouldn't be the main way to show that the code is working.


I understand the naive approach well; here I am implementing a stack-based virtual machine for my was-an-assembler-now-a-compiler where upon I have to write the assembler in the new VMs assembly language -- all so that I can compile my Z80 source code so that I can get my level editor working. It's quite maddening.


yeah as default ... that's looks good ;)


I have tried both pylab mode and without pylab mode and I can still reproduce this issue on master. Try switching the last two lines:

```py
from mne.report import Report
from mne.datasets import sample
from mne import read_evokeds, read_cov
report = Report(verbose=True)

path = sample.data_path()
fname = path + '/MEG/sample/sample_audvis-ave.fif'
evoked = read_evokeds(fname, condition='Left Auditory', baseline=(None, 0),
                      verbose=True)
noise_cov = read_cov(path + '/MEG/sample/sample_audvis-cov.fif')
evoked.plot_white(noise_cov=noise_cov, show=False)
evoked.plot_topomap(show=False)
```


I don't think you needed to change this ' blog.layout = "article_layout" '.


I figured Sponge would add something similar, is there any major benefit to using that other than consistency?


Since you seem to like the `^FOO^` pattern, maybe the previous line should be `^TMPDIR^` for consistency? In this way, it will be clearer that `$TMPDIR` is not an environment variable.


If there's no separate API JAR on Maven then the first point is moot (hadn't had a look when I wrote that).

As for tags... I guess something along those lines could work. Just not super happy with it because that'd still show in "normal" Markdown rendering. Hmm, oh! I guess I can add support for HTML-style comments and pack the annotation in those, though. Allrighters, I'll see how far I get with that :-)

How often is the Maven updated, btw? The last build on it (1.1.1-17) doesn't have the new API goodies in it, yet (unless I'm being blind again >_>).


Could make this a list:

```md
## Examples

- [WhatDoTheyKnow](https://www.whatdotheyknow.com)
- [KiMitTud](http://kimittud.atlatszo.hu)
- [Informace Pro Všechny](http://www.infoprovsechny.cz)
- [fyi.org.nz](https://fyi.org.nz)
```


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (346 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@ashfurrow Why are you conforming to `: class` here?


Cînd ai ajuns la elementul de la poziția `index`, ai putea să te oprești.


Nikt nie lubi wykorzystywać standardowych Javowych bibliotek do dat i czasu. Są zwyczajnie zjebane. Spróbuj zamienic GregorianCalendar na coś z biblioteki YodaTime, którą każdy lubi.
Żeby móc korzystac z YodaTime dodaj sobie zależność w pliku pom.xml. Zależność (dependency) możesz wziąć z Maven Repository - zwyczajnie googluj "YodaTime Maven". Dependency które szukasz to jest coś podobnego jak mysql-connector który masz w pliku pom.xml - dodaj to obok niego


Buna,

Am citit toate comentariile tale, da ai dreptate mai trebuie sa mai lucrez
putin la partea asta si nu numai. O sa incerc ca pe viitor sa fac mai bine.

Multumesc pentru comentariile constructive !

În data de 27 aprilie 2015, 10:27, anitzapredescu <notifications@github.com>
a scris:

> Avand in vedere ca toate fieldurile sunt protected, ca nu ai un
> constructor prin care sa le initializezi, daca vrei sa faci o instanta de
> tip Persoana(clasa nu este abstracta asa ca cineva se poate gandi ca vrea
> sa creeze un obiect persoana) ex: var pers = new Persoana(); (folosind
> constructorul implicit) nu vei putea niciodata sa schimbi valorile
> fieldurilor si ele ar ramane cu valorile default.
> O rezolvare la aceasta problema este ori sa faci un constructor de genul
> public Person(string lastName, string firstName, string dateOfBirth) si sa
> initializezi, fie sa faci niste propietati gen public string
> LastName{get{return lastName;} set {lastName = value;}}(eu as alege
> constructorul pentur ca il poti folosi sa il apelezi cand faci
> constructorul claselor care deriva din clasa Persoana. vezi comentariu de
> la constructorul Employee)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabrielnicolae/C-Fundamentals/commit/b2cdb235ae6499ac05fcc92688ea6c9b23e6040d#commitcomment-10910241>
> .
>


This is very helpful.  Thank you.


You're right, you should keep it `xml` to keep backwards-compatibility.


+1 Right, that's what we expect after our discussions


No hay forma de que con localizedDescription funcione bien ? no es comun llamar al objectForKey... en el error


E un bug pe linia de mai sus.


I think that might be the same one but with a new line number


Hey, are you be able to provide iOS sources to successfully built project or rebuilt it to support iOS 64 bit ?
Thanks


TeamCity MonoGame :: Develop (Win) [Build ???](http://teamcity.monogame.net/viewLog.html?buildId=6387&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


plot_topomap seems broken though, I get a separate figure for each topomap.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (346 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I tried using Yank and how does it actually maps the result to object
class?

Regards,
Lakshmisri Gopalan
IDEAS2IT Technologies

On Wed, Apr 22, 2015 at 8:20 PM, Lakshmisri Gopalan <lakshmisri@ideas2it.com
> wrote:

> What are the dependancies do I need for that?
>
> One more important point is am using dropwoxard 0.8
>
> Also with sundial how do I inject dependencies
>
>
> On Wednesday, April 22, 2015, Tim Molter <notifications@github.com> wrote:
>
>> I use Yank, and demos of that are found in XDropWizard. Once you have it
>> integrated you can call
>>
>> Yank.execute(...);
>>
>> directly in any Sundial Job. That's how I do it, and it's really easy. If
>> you want to use a different ORM or JDBC directly, different setup will be
>> needed.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/timmolter/XDropWizard/commit/c1290b69eb6801fb247fe2d080d788c5bca053d8#commitcomment-10847402>
>> .
>>
>
>
> --
> Regards,
> Lakshmisri Gopalan
> IDEAS2IT Technologies
>
>


I think the wording is a little confusing here.

I think something like "If you always want to use the default development version used by the alaveteli team (currently 2.0.0) you can create a `.ruby-version` symlink with a target of `.ruby-version.example`.


De ce faci asta? Ce se întîmplă dacă vrei să adaugi alte elemente după ce ai făcut `Clear()`?


Aha


Ce te faci dacă `t` este `null`?


Commituj częściej :D


There is not an API JAR, but there is a dev jar, which I was refering to really.

Well, I assume you'll have to do that thing in the PageChangeEvent anyway, so you could as well in there add that tag to the returned text?;) That way it still is seperated from your normal rendering.

Hmm... the maven should be updated with every build I do... It might be broken, I'll take a look.


"*Sync application*" maybe?


This is a first take at issue #1.


no, I have some postInstall task to call here??


Hum, indeed show=False is ignored if you switch the order ...


Iarăși, e derutant că parametrul se cheamă la fel cu field-ul, încearcă să eviți de astea.


No problem, I'll look into this tomorrow.

--
Douglas Drumond
*Café Linear <http://www.cafelinear.com/> – GDG Campinas
<http://www.gdg-campinas.org/>*
*google.com/+DouglasDrumond
<http://google.com/+DouglasDrumond> – @douglasdrumond
<http://twitter.com/douglasdrumond>*

On Mon, Apr 27, 2015 at 11:51 AM, chdiza <notifications@github.com> wrote:

> I did use Snapshot 75, or rather, I built from the same commit that
> committed Snapshot 75.
>
> I don't know how to do any video recording, and it will be a while before
> I can even start researching how to do it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/macvim-dev/macvim/commit/b517b310559bb4c8d297f9b2dec6aa972446b504#commitcomment-10917717>
> .
>


Well there is still a `coffeescript` code tag in there 7 lines down from the one where you changed `html` to `javascript`. Also, there is an `html` tag 20 lines below the last `html` tag as well (above `<Person />`).

I'm actually not sure you wanted to change the `html` tags as they seem to be valid. But you _do_ want to change the `coffeescript` one to `javascript`.


amazing code! O(∩_∩)O


De ce faci asta cu default(T)?


Wäre das `get_zk` ohne `()` nicht idiomatischer?


same here, $postInstall needs to be called?


@kennethlimcp looks great! I see v0.0.2 on the server and the example code is removed. Thanks for doing that, it should prevent confusion to newcomers.

Out of curiosity, is it possible for folks like me to submit pull requests for fixes like this on the Spark library side of things? I noticed that the change you made isn't something that appeared in this GitHub repository.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (346 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


N-ai de ce să faci commit cu cod comentat?


and mention moon tacos


@maxiloc could you replace this loop with a `removeProducts` and `removeCategories` call invoking `deleteObjects`? We didn't have the "batch delete" at the beginning so I think that was the reason why it used to be an async task -> removing a lot of products would then create too much API calls, producing timeouts.


offcanavas:
  container:
     - mobile-menu


I have a pojo class and it has a field called spotfix_id which is the same
same as column in the database. But Yank is not mapping the values
correctly and its passing null. Not sure what i am missing here.

@Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
@Column(name="spotfix_id", unique = true, nullable = false)
    private Long spotfix_id;

Regards,
Lakshmisri Gopalan
IDEAS2IT Technologies

On Mon, Apr 27, 2015 at 8:29 PM, Lakshmisri Gopalan <lakshmisri@ideas2it.com
> wrote:

> I tried using Yank and how does it actually maps the result to object
> class?
>
> Regards,
> Lakshmisri Gopalan
> IDEAS2IT Technologies
>
> On Wed, Apr 22, 2015 at 8:20 PM, Lakshmisri Gopalan <
> lakshmisri@ideas2it.com> wrote:
>
>> What are the dependancies do I need for that?
>>
>> One more important point is am using dropwoxard 0.8
>>
>> Also with sundial how do I inject dependencies
>>
>>
>> On Wednesday, April 22, 2015, Tim Molter <notifications@github.com>
>> wrote:
>>
>>> I use Yank, and demos of that are found in XDropWizard. Once you have it
>>> integrated you can call
>>>
>>> Yank.execute(...);
>>>
>>> directly in any Sundial Job. That's how I do it, and it's really easy.
>>> If you want to use a different ORM or JDBC directly, different setup will
>>> be needed.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/timmolter/XDropWizard/commit/c1290b69eb6801fb247fe2d080d788c5bca053d8#commitcomment-10847402>
>>> .
>>>
>>
>>
>> --
>> Regards,
>> Lakshmisri Gopalan
>> IDEAS2IT Technologies
>>
>>
>


why do we want to disable the resaturation test?


Meinst Du `if path.empty?`?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


misses assign for return value


Era bine cum era, acum ai stricat. E perfect valid ca într-un `MyList<string>` să zic `myList.Add(null)`, deci să am elemente `null`, nu am de ce să mă opresc cu enumerarea cînd dau de ele.


Thanks :)


La ce-ți trebuie `using System.Runtime.Remoting.Messaging`?


`&& !path.empty?`


I think someone modified a .gns3 file because this is always the same projects/hosts. We will have to use  schema validation for gns3 files I think (for 1.4 for instance).


Thank you! ;)


Era ok cu acolade, de ce le-ai șters? Lasă rîndurile să respire, nu le înghesui așa.


Ist `watchers.first` evtl. lesbarer als `watchers[0]`?


I guess those are the hashes you pulled out?


Can't synthesizers figure out one-hot encoding themselves?


This commit breaks C_DecryptInit in SmartCard-HSM as key->ops->can_do is not set for emulated cards.


Generell fehlt bei Dir gerne mal ein Newline am Ende der Datei.


This fixed the swi_demo.gba from TONC demos. The table values are generated from the GBA itself. The code is implemented from VBA-M (horray for the GPL!) but cleaned up to avoid the unholy amount of casting that goes on. Of course the casts here are implicit, but it's much more readable than what's in VBA-M (with comments too).


It's strange multiple users have the same issue


I hate floating point, so I'd recommend ``(flen(signal)+7)//8`` instead.


or a conversion error coming from old old version


`fail` instead of `when`?


Hi,
# This script is proprietary software and cannot be published or redistribute.
Why making such simple scripts proprietary?
IMHO, being able to cluster ejabberd is the main advantage of ejabberd over other XMPP servers (e.g., prosody).
So far, it's quite difficult to set an mnesia cluster without manual intervention, and even harder to fix netsplit issues afterward.
I've done some scripts in that direction, and the work at https://github.com/chadillac/ejabberd-easy_cluster/ is also helping in that regards, but I don't understand why ProcessOne would not accept contributions from the community.
If I want to distribute Ejabberd 15.04, should I delete those scripts from the tarball? So far, they are included in the tarball even though the COPYING file says GPLv2.


Sure, I'm working on it.


@littleguy77 we should also modify the createGLContext method in NativeSDL.java in order to be able to initialize an GLES3.1 context.


No, why? You can't fail($closure) and expect the Closure to be executed on failure…

Whatever is passed to fail(), will be returned as failure value.


感动


add `show=False` to the `plot_topomap` call on line 1088 of `topomap.py`, then you should be good


`gaussian_filter` uses `nearest` as the default boundary. Two others use `reflect` and one uses `constant`. I'm not sure what the right thing to do is.


@Gillou68310 Great point, you're right.  Feel free to push the change, otherwise I can probably get to it tonight.


But this is probably fine.


My feeling is, it should be possible to support multiple clusters inside one VPC. So, perhaps it is better to identify the VPC via the "Name" tag (as it was prior to v0.15) or introduce a new variable "VPC_NAME"?


(`show=True` was added to `plot_topomap`, which means functions like `plot_evoked_topomap` might need to call it with `show=False` now)


Whether we should care should depend on our supported list of browsers.  Since we still support IE7+, then yes we should care about producing javascirpt errors.  This one is easy though.  Just test for it first.

if (window.console && console.log) {
    console.log(...);
}


Hi

i want add MyBB Forum script css and js file please help me how to add it


This commit broke just having chat throttle off.

Also, for the record I think having throttle effect mods too would be a great if it was optional, but having it mandatory is terrible.


Nice @spsaucier !

Would it make sense to send a pull request to angular-gantt/angular-gantt and get this back into the main repo?


understood, i could have emailed these notes separately


s/blocks/chunks/ ?


Ok I'll check that. Could you send me Gliden64 source? I don't have access to paulscode.com at work (stupid minecraft reference!!!!)


Lucas, this is a masterpiece.
You are giving much needed bones to those tables.
And when the structure is there, improvements are easier to find.

Especially for the Rook PST, you removed all the nonsense.
How much ELO gain you got just from that one will probably deserve a test on its own.

I really like this Center array in your formulas.
I wonder if tuning that array, keeping the 0 fixed, will give even better results.

Good luck with all this.


Only this? ;)


Thanks for spotting this! It looks like your change fixes in a better way. Would you send a pull request to re-enable IEEE?

You're right that I'm frequently busy and will occasionally miss things. Right now I'm spending about 75% of my time on Let's Encrypt. I definitely appreciate you attention to detail to catch things like this: it's a huge help in keeping HTTPS Everywhere working. I'm sorry if I sometimes step on work you've done.

In terms of fixing this particular miscommunication: It would be helpful if you would close or comment on Trac and GitHub issues once you've fixed them. That would both help avoid future miscommunications and save both of us time in reviewing bugs that are already fixed.

Thanks,
Jacob


-- Classe(tabela) Pessoa com variaveis inicializadas.

local Pessoa = {nome = 'João', idade = 30, nacionalidade = 'brasileira'}

-- Construtor de Pessoa.
function Pessoa:new(pessoa)
pessoa = pessoa or {}
pessoa.nome = pessoa.nome or self.nome
pessoa.idade = pessoa.idade or self.idade
pessoa.nacionalidade = pessoa.nacionalidade or self.nacionalidade
setmetatable(pessoa, self) ; self.__index = self
return pessoa
end

-- Criando dois objetos de Pessoa usando o construtor,
-- o primeiro sem passagem de parametro para o construtor
-- e o segundo com passagem de parâmetros.

local cliente = Pessoa:new()

local fornecedor = Pessoa:new{nome = 'Luis', idade = 21}

-- Criando um objeto similar mas sem usar a classe.

local funcionario = {nome = 'Robert', nacionalidade = 'americana',
idade = 42, cargo = 'gerente',
filho = {
{nome = 'Fernando', idade = 18, nacionalidade = 'mexicana'},
{nome = 'Mônica', idade = 13, nacionalidade = 'japonesa'}
}
}

-- Criando um cachorro, nao e uma pessoa mas possui
-- alguns atributos iguais.

local cachorro = {dono = 'Fernando', nome = 'Rex', idade = '5',
nacionalidade = 'brasileira'}

-- Tabela sociedade vai armazenar todas as pessoas e o cachorro.

local sociedade = {fornecedor, funcionario, cliente, funcionario.filho[1],
funcionario.filho[2], cachorro}

-- Imprime dados dos objetos(tabelas) criados. Observe que
-- este metodo tambem e usado para apresentar as informacoes
-- do cachorro, que apesar de nao ser uma pessoa, possui
-- todos os atributos tratados neste metodo.

function showPessoa(pessoa)
print('--------------------------------------')
print('Nome: '..pessoa.nome)
print('Idade: '..pessoa.idade)
print('Nacionalidade: '..pessoa.nacionalidade)
end

function initialize() -- Varre a tabela sociedade.
for i=1, table.getn(sociedade), 1 do
showPessoa(sociedade[i])
end
end

initialize()

-- Inicia a aplicação.


I'm not sure if it's a general condition -- no one else has reported having
the issue, so it may have to do with our setup only. One side effect of
adding the '-1' is that if there is *not* a rounding error, it's 1 pixel
too short -- acceptable for our purposes, but if it's not an issue
elsewhere, it wouldn't be ideal.

_______________
*Stephen Saucier*
(706) 225-9155

On Mon, Apr 27, 2015 at 11:13 AM, Brandon Beacher <notifications@github.com>
wrote:

> Nice @spsaucier <https://github.com/spsaucier> !
>
> Would it make sense to send a pull request to angular-gantt/angular-gantt
> and get this back into the main repo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gather/angular-gantt/commit/120cdf5dc33451c40cf4db30811d1741f27e6855#commitcomment-10918207>
> .
>


my issue with your inline ifs has nothing to do with your inline ifs.  it has to with consistency.  the rest of the developers have already discussed which SONAR issues to enforce.  i don't think asking you to be on the same page as the rest of your developers is a huge request.


Yes yes, that's what I meant to confirm :-) (I originally just assumed, because it's not uncommon for projects to have a separate API JAR).

As for adding the tag in the `PageChangeEvent`, that's really clever. I'll probably do that, thanks for that idea!


Me during the push
![Giphy](http://media3.giphy.com/media/7h5IURRx9IhZC/giphy.gif)


ok perfect, it works. Fix added to my PR


using `g_signal_connect_after()` at connection time should work too


SONAR rules.  HTML uses double quotes. JS uses single quotes.  It helps with consistency, especially when mixing HTML and JS.

document.write('
...
');


you can put this onto a single line


`found` is still not set to the actually used tag when not updating


why is usage of the last_tmpfile variable necessary?


/cc: @exalted


not very important, but I'd love to see the fix included in the commit


I actually can't build it yet, so it probably won't make a difference if you don't have it.  Just comment out the new section of Android.mk while you debug the java part.


rax retry tests


Fix verbosity please.


If possible (should be, I think), it would be better to split this into lines as "one ``for`` at a line"


(Andrew Connolly) ColorImage
initial color image generation code


Maybe we should do something so the user can know not every tags are shown?  I don't know, but I could imagine someone having a hard time getting what's going on, and reporting us a bug because the 2345th tag doesn't appear.

I however have no idea how would be a good way to show this…


done


Arghs, I am missing jQuery-style promises  with amphp-style ones


Cheated here: copy paste from somewhere online: http://www.sinatrarb.com/faq.html#auth


Dar, sigur e bun? `items[count]` vrei să-l setezi la `default(T)`?


Nice catch guys! Thanks!


LGTM


GDC will test either ``(arr.ptr | arr.length)`` or ``arr.ptr || arr.length`` - though I think the latter is dead code. (I'll just check)


No, just this one :).


See http://sourceforge.net/p/erlyaws/mailman/message/32340912/ .


* do not port to 6.x *


Actually, why do we bother having `reflect` here? The main interface, is the dask array method which will override this with `None` by default anyway.


By removing this validation, all the non-Debian operating systems will not have their package_gpg_key validated.


Yep, actually you are right, this is maybe an issue from gns3-converter...


Isn't it exactly the same as the commit I pushed? Have you changed anything?


Did some research; it's not really necessary. The "DEVICE_RESOLUTION" flag is deprecated anyways ("TW_THEME" should be used instead), but it doesn't matter since the new way TWRP handles these themes is the same for 720p and 1080p (flag value is "portrait_hdpi").
My Git is also broken so pushing anywhere creates a huge mess (just check any of my repos lol), wouldn't want to contribute chaos :P


this is the real fix


add shutdownNow() to make sure no threads are leaked


LOL


This could be split into two lines by assigning ``", ".join(str(d) for d in unavailable_dependencies)`` to a variable (even to ``unavailable_dependencies``, I think).


All comments and identifiers should be written in english, please.


Typo, param should be :
DJANGO_SETTINGS_MODULE='amoki_music.settings.production'


fix 2/2


While this dependency might be useful for deployed environments, it has no use in development or research. Anyone bothered if we remove this?


What kind of errors do you get when trying to build it?


sorry.. where is the typo?


diff without whitespace: https://github.com/Microsoft/nodejstools/commit/8a178a025a508a2240c31d0b70ebb39b821a56e6?w=1


Fair enough. Just leave a note to that fact.


Good call.


Messages should be internationalizable. Commit 2550c40a45dfd6550891fb37b9b2fbd8f697748a adds a `$application->translator` object that allows i18n. See [Symfony/Translation](http://symfony.com/doc/current/components/translation/introduction.html). Translations are stored in [`src/i18n`](https://github.com/CosyVerif/paquito/tree/master/src/i18n).


Why don't you use defines for tunables? You can place them at start of file to have better overview of code. It will be easier to tune governor(s) and to keep up with numbers and changes across commits :)

Example:
```
....(code)
#define SAMPLING_DOWN_FACTOR 100000
.....(code)

static unsigned int sampling_down_factor = SAMPLING_DOWN_FACTOR
```

So later you can tune defaults with easy change of number in #define macro :)


I'm fine to do this change in next patch set version


Should the new icon replace all old icons? Now system tray icon and window icon have different image than docker.


Why do you return the parsed result? You are within a command, the user can do nothing with your result. Instead, store it in a field in `$application`, for instance `$application->data`. It can then be used and modified by other commands.


I like the concept of opts, just thinking through ordering of params (along with backwards compat) that is consistent elsewhere. I just made a quick scan through on my phone. I can look more in detail later.


Yes, there are uses for == and != but often that means the statement could be rewritten to utilize the more efficient === and !==.  If there is no way around it then a comment should be left that effect.


I don't think this should be done here.

Indeed, the timer belongs to the SoC, not the board.
I mean, it does not use pins.


Can you fix this typo within this patch as well? s/bufefr/buffer/


LGTM :+1:


mmmmm


Well I guess my concern is that there is no a priori knowledge on the
device how many Server or Security object instances a bootstrap server
would create (it could create one, five).  Similar case could be made for
any of the other objects that allow for multiple instances (a server could
create 0 instances, a hundred, etc).  Would then a solution be to create a
map of several factories inside of ObjectEnabler?

   - A ReuseInstanceFactory that would have a map of known InstanceEnablers
   passed in that will reuse instances passed in a
   ObjectEnabler.setInstanceForObject() method.
   - User-defined factories that could be set in an
   ObjectEnabler.setInstanceFactoryForObject(), so a device could build the
   InstanceEnablers as it needs them.
   - A DefaultInstanceFactory that would do the existing default
   constructor for creation (and be the default if no other factory exists and
   it would create the SimpleInstanceEnabler).

Or am I overthinking this?  I guess I am trying to find the most flexible
way for a device to get data in and out of the Leshan library...

-J

On Mon, Apr 27, 2015 at 9:23 AM, sbernard31 <notifications@github.com>
wrote:

> I don't know, what would be the benefits from :
>
>    - a way to initialize object with a list of instance
>    (initializer.setInstancesForObject(1, {server1,server2})
>    - and the use of default constructor for creation
>    <https://github.com/eclipse/leshan/blob/master/leshan-client-core/src/main/java/org/eclipse/leshan/client/resource/ObjectEnabler.java#L68>
>    ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eclipse/leshan/commit/0679bced772a2f2f5d7171278d7be0f580222eb5#commitcomment-10917067>
> .
>


What does this argument mean? What is a "structure"?
Instead, this command should take nothing or a yaml filename as input. When the filename is given, it can use the `Parse` command to parse it. In both cases, it should use the `$application->data` gobal field (that should have been set by the `Parse` command and/or modified by other commands).


Casey,

I'm sorry I haven't gotten to these yet. I had a clients website go down Sunday and I'm trying to get everything back up. I expect to be done within the next hour or so and I will start working on them right after that.

Ruben


@jstypka there are some differences. Please review them carefully.


O to mi chodziło. Dobra robota!


FIXME using Francis' remark in last meeting.


Thanks for your editorial skills nick :) :+1:


@Pacoup, just an fyi but you can wrap code only meant for the polyfill with `.no-details {}`.

This will only apply to the polyfill
```scss
.no-details{
    summary{

    }
}
```

You can also use `.details` to target only unpolyfilled summaries


Maybe you could change U-Boot clock settings to keep 180MHz?
7% speed gain not much, but always welcome on such a small CPU.

But maybe I missed something and you gave a good reason to do so?


That'd be great. It's really the ugliest part of this and I'd love to simplify while still maintaining backwards compatibility.


Arguments should be named `input` and `output`, because `structure` does not mean anything and `filename` does not say if it is the input or output one.


new font is good .
but don't change any other things they are best


I normally wouldn't push directly to master for this, but since the travis config only works with my SF account because of encryption limitations, it's not a huge deal.


>iw4sp.exe
:+1:


No benchmarks, but should be trivial - and depends on the system.

The idea was that Ruby's detect gets used for all builds, which have no explicit buildpack configured.


Not sure if Apple approves of this commit message :stuck_out_tongue_winking_eye:


We dont need to create a new table, we can assume that all hierarchy relationships in "SAB" = MSH will be RB...

so lets load into ISA


You should remove the open bracket **{** on line 181 after *case self::CRYPT_STD_DES:*

It brakes future builds on Travis

Zephir\ParseException: Syntax error in /home/travis/build/phalcon/cphalcon/phalcon/security.zep on line 185
	    let saltBytes = this->getSaltBytes(2);


Apparently the code does not handle correctly the case where no algo_refs are defined for a key and can_do() returns CKR_FUNCTION_NOT_SUPPORTED.


This code is too deeply indented. Instead, you should maybe create several functions with dedicated tasks.


main problem was here: sel_prod was taken from the initial value of the GUI, andnot from the "saved" product selected in the RData file. Thus, inconsistency in bands selected for analysis


I'm currently waiting on Gideon to check and publish a the release, he's quite busy at the moment. You can grab the latest code from the master branch if you like.


Do we need the code between comments.


I'm not sure if we have a standard set on this one but to me, using spaces over tabs for network transmitted code makes no sense.  Production code should be minified so this really only affects the developers but I think the principle still holds.


Fixed.


This comment could be omitted if the variable name was more explicit.


Could you also check the compile performance? (using force compile option in both versions?)


IMO, this entire Debian specific `if` block is unnecessary. You could leave the `validate_string($package_gpg_key)` line where it was previously, add an additional `validate_string($key_content) if $key_content != undef` and continue on with your intended fix in https://github.com/nnathan/puppetlabs-rabbitmq/commit/0295ca1ecdca4849ba5afd3fc124a8b36d349b0f#diff-d3ff06b1697648f0bbba8fb053f59366R23.

I don't think that the manifest should fail if both params are set, that may cause more problems than it creates. If you want to alert, it should be a warning at most. Otherwise the intended usage was always noted in the README that the `$key_content` param `Overrides 'package_gpg_key' behavior, if enabled`.


Ack. I was following what the other existing workloads had, so I can fix this.


@jirikuncar I'm afraid I can't see any changes, I also cloned your repository and run the tests which have failed. Are you sure you pushed all your work?


Ahh, makes sense when you put the two together....


All these **13** closing blocks show that this code **must** be refactored in smaller functions. It will become unmaintainable very quickly.


I have removed this code already. It was commented out in an intial comment. You should not see this in final copy.


This code has to be somewhere inside ggrc_workflows, we should never ever reference workflow object outside the module...


@phalcon


new font is good .
but don't change any other things they are best


Hmm...
I don't need to do that with my busybox.
Any idea why you need with yours?


I forgot to take into account that the user may want to run the workload for more than 30 seconds occasionally. Noted.


On another subject I worked on an OpenSLES audio plugin. It's not finished yet but it would be great if you could test it on your devices just to see if it's running okay. Resampling is not implemented yet so changing speed factor won't have any effect and some games might run too fast or too slow depending on the requested output frequency.

https://github.com/mupen64plus-ae/mupen64plus-ae/tree/audio-sles


When do you plan to release this code?
Look like that we introduced API breaking change in version 0.9.0
It is breaking on current pypi version if you have something like this

RedisDict('key', redis.StrictRedis(host=redis_host, port=int(redis_port), db=int(redis_db)))


Thanks for the suggestion.


FYI, DMD uses ``or`` too:
```
mov    QWORD PTR [rbp-0x10],rdi
mov    QWORD PTR [rbp-0x8],rsi
mov    rax,QWORD PTR [rbp-0x8]
or     rax,QWORD PTR [rbp-0x10]
```


No, I just know what I'm doing.


That look good to me.

Thanks,
Maxime


Referencing from pull request https://github.com/AOEpeople/Aoe_CacheCleaner/pull/3


Noted.


蛋疼的发现了貌似没支持iOS8以下的路径，明天再处理一下。


Good to know. Will fix this. Thanks!


I don't see any story in the unit tests for testing handshake things. It seems mostly low-level bits like parsing and the like. I can certainly imagine building a unit-testing framework that can test a handshake and connection behavior (which would amount to basically a full TLS implementation...), but a proper connection-based framework seems much simpler. We've gotten a ton of mileage out of BoringSSL's setup, and it's all connection-level tests.

I certainly don't have the time to build out any missing test frameworks from scratch here. I have enough TLS implementations to maintain as it is. This is mostly because Trevor asked me to upstream some patches. Being able to avoid carrying some of them is also convenient, but at some point it becomes less of a burden to just carry the patches.


Nice


No problem, love your distro!


But i have a better one


Wait a bit for it


*0.0 ? Maybe *0.01 (and use some names, no more magic number ;D )


@NokiaGuy This is still valid, and can be used for updating without running the launcher


What can I say? My brain just sees thing differently.


@hantek Can you explain the changes made to the test_roc_auc file? They seem unrelated.


Seems this change is breaking creation of pull requests (and probably other things) since json Map is no more used.


It'd probably be useful also to add something related to server usage with multiple users


I don't think this should prevent the merge, but troubleshooting is harder for me when an ivar has the same name as a method


TeamCity MonoGame :: Develop (Win) [Build ???](http://teamcity.monogame.net/viewLog.html?buildId=6390&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


SPELL_EFFECT_SCRIPT_EFFECT, does nothing, in any case scripting it should do the same as this


train_y_roc_auc was causing KeyErrors if we don't update them to a right channel name. See #1480.

Quote:
The original test was buggy. The written channel name doesn't exist. The issue was not provoked before, before the EpochCounter terminates the training before even MonitorBased being called.


oh, that's nice, I always tried to pass the reporter as another arg to tsc() and ran into the filter arg not being documented...


This might explain why some USB audio apps said that "Linux detected audio hardware, but Android didn't".
Also now Android's native USB audio output may work on stock KK.

Have someone try it on USB DAC.


ID's are a more efficient CSS selector.  Browsers can make optimizations because an ID can legally only be used once.  I thought through how this tool would be used and I didn't see a need for multiple windows.  Classes are important for reusability -- reusing styles across elements.  Classes also have a lower performance rating on the jQuery selector performance list.


You should add a docstring to every test that describes what you're testing, since it's not always obvious.


But, there are 3 different spells, how to handle that with area_aura table? they should be put one after another.


Why PiP was removed? I can't find any issue indicating it, neither an external plugin.


It seems that you duplicated the name for rxn12184 here


Is this simply-busy-looping? or does it have a blocking call?


Done here 7efb99f


And again.


That should be setup(newChannel) instead of setup(channel), I suppose?


Missing javadoc


I made a mistake here when producing plantdefault_obs, it appears that by separating out previously merged reactions, I didn't restore their former ids, I'll look into it.


Dit stuk moet worden teruggedraaid, anders werkt het niet op Heroku.


> But when you check man setenv, _DEFAULT_SOURCE isn't mentioned.

That is irrelevant, the man page just isn't updated. Anyway the later info you found confirmed that this commit does the Right Thing[tm].


You're fine, don't worry about it, we have till Wednesday to get this done,
and we're pretty close. Most of the methods really aren't that hard to
implement. The only one I ever had any problems with was ==, and that's
just because this language doesn't do good static type checking.

On Mon, Apr 27, 2015 at 8:35 AM, Ruben Niculcea <notifications@github.com>
wrote:

> Casey,
>
> I'm sorry I haven't gotten to these yet. I had a clients website go down
> Sunday and I'm trying to get everything back up. I expect to be done within
> the next hour or so and I will start working on them right after that.
>
> Ruben
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RubenSandwich/CS420HW2/commit/9befe827c4616521b8704837abd26b85fde940f7#commitcomment-10918636>
> .
>


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (345 of 346 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.71%25-yellow.svg)


Ik zou de regels omdraaien, dat leest net logischer.


Missing javadocs for this function and below few functions


Encapsulation, the team is saved in the InstanceScript, shouldn't be accessed from the AT script. (at least not directly)


Niet nodig?


Hey @blacktm, @sankarravi, and the rest of the BlueButton.js community!

I just wanted to let you know that the [CTS-IT team at the University of Florida](http://ufl.to/ctsit) has started porting BlueButton.js to Python for use in our applications. We hope a native-Python version will expand the community. We will try to keep the functionality as up to par as possible with the original JS code.

Thank you for your initial and continuing work and please let me know if you have any thoughts or concerns.


Wat gebeurt er als de les niet bestaat?


Looks promising, and if this approach makes sense to you then stick with it!


Fixed


I wonder, why is this a GZDoom-only change?


cf https://github.com/dobtco/dobt/issues/1135


I got rid of the `!important` in #6724


@gssiyankai

I have changed some of this code for two reasons:

1. We want to preserve little endian encoding.

2. It seems wasteful to use 4 bytes to store the number of runs: this should never happen... you would be much better off with an array container at this point.


tutorial 2


@thiswangba   这个貌似没放md里


do it also with buylist :+1:


sounds good


OK, I just pushed a thing that addresses this: b67aa03908b69c7ffef49f4def48a3ef765bf533


buylist should also modify? :+1:


You haven't see the end result of my style update.  Clicking on the entire DIV won't make any sense when I'm done. Once again, my favorite word comes into play: consistency.  Both Windows and Mac seem to agree in this area now.  Just the titlebar or upper area of the window will move a window.  I know that wasn't the standard for Mac back in the day but most apps have moved that direction.  I believe "I like it this way" isn't a good enough reason to go against a standard.


I had a look at this, it seems you're aiming to remove abbreviations that are the same as the name?


Yup, agreed. That's the right direction to go with that.


Why?

Also, use [`event.stop()`](http://api.prototypejs.org/dom/Event/prototype/stop/) if you really need to stop it.


1) `if len(bracket_string) == 0` has the same result as `if not bracket_string`
2) Have you considered moving it to the beginning of the functon? Having `return` in the body of this `if` branch allows you to drop `else`.


Man, OK, just noticed you also did some work on this in your PR. That's what I get for starting from the beginning. I'll pull that down and run the tests.


@cernceph the backport tab should be 'Backport', not 'Backports'. Can you also add 'Fixes: #11473' to it?


This function is just findConnecting()


`USER` is more canonical than `LOGNAME` these days, I think. But +1 to ssh'ing as root - that relies on the implicit assumption that the user syncing daemon is functional.


This assertion is vague. Is there anything else in that console warning to ensure it's related to the prop you care about?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14266/testReport/) for this commit**


No. Add:

`@Inject private Provider<XWikiContext> xcontext;`

Then use `this.xcontext.get()` where you need the context, don't keep only one instance for the whole class.


Looking at the code, the script fails if the argument is not given.


You shouldn't need to call this because of https://github.com/react-bootstrap/react-bootstrap/blob/v0.21.0/test/index.js#L10


Tyler,
     I don’t know about ASCII art, but the Piet language is as artsy as it gets :o)
Ciao Mike

From: Tyler Carter [mailto:notifications@github.com]
Sent: Sunday, April 26, 2015 6:39 PM
To: Team3128/frc-java-3128
Subject: Re: [frc-java-3128] Add ASCII art to buildfile (c0a9c6d)

We should create a programming language that we can program via ASCII art. It will probably be very easy to debug.

—
Reply to this email directly or view it on GitHub<https://github.com/Team3128/frc-java-3128/commit/c0a9c6d7bf861f1c1e91a5dabb8d49a729c611d5#commitcomment-10908219>.


The link is contained within a clickable container. I do not want the click event on the link to propagate to the container.


Alter meta returns the items' full metadata, including the new version number. Is there a reason why file.version is behind by 2 versions?


I see.


> I certainly prefer fewer dependencies. However, if the library provides features that cannot be attained without it (or it is very difficult to do so) and it is required for running the tests only (and not using the library), I have no objections.

I certainly don’t want to rewrite something just for NIH reasons, but I have yet to find the case using `nosetests` which couldn’t replaced by the standard Python library.


Defaults to **"**foo**"** **if** RKT-NETCONF-FILE is **"**10-foo.conf**"**


In other languages that allow you to "freeze" things, it means that the arrays become immutable, not just of fixed-length. Arguably, immutability might be a more useful language feature than fixed length.


Any plan to merge this in upstream? Was struggling with getting PyML installed too


Dobro, hvala na pitanju! :) :+1:


De aquí en adelante no esta traducido el documento... es un work in progress?


I'll see if that will work, or if it's sufficient to grab $DISPLAY before
the imports.


I missed this when I merged the other changes in this PR, but we shouldn't be doing this. The config file should stay configured so that it runs against the local sandbox env so that when you run the app locally the files only ever changes if there are new config options. In which case there should be an additional change to the `config/environments/*` files.

This has been fixed with #365


Сделал


Do I get the exit code from the command in `err`? The ssh command normally returns it. If so, you should be printing stdout/stderr here and not below, because err might encapsulate "error from either SSH issues or from exit issues"?


same


the name of the bundle is everything up to the first space character. the old code was generating names like `bootstraps/facia - bootstraps/app` because it was string-replacing arithmetic expressions.


Actually there is: http://search.cpan.org/~mob/Acme-AsciiArtinator-0.04/lib/Acme/AsciiArtinator.pm


Is it worth mentioning up front that it doesn't support array types?


@samdroid-apps,  You are changing the way the configuration is changed, in the practice reverting
https://github.com/sugarlabs/sugar/commit/2c95a2d9ed97ea751a0744c2f56d9409ab64b398
What is the rationale for the change?


Maven is fixed ;).

I did think of adding the page path to the ITextInterpreter.. however, there doesn't necessarily have to be one. For example, with the plan you have, you won't have a path, as you'll be supplying it via the PageChangedEvent ;). But, I think you agreed already on adding a custom tag :P.

If you think your text interpreter might be a nice contribution to IGW, be sure to PR it to IGW :). But no scala... you decide ;)


this notice was a mistake and is no longer present in 15.04.


@khadijaibrahim @maryam-osama  YALAAA B2AA N MERGEEE :dancer: :dancer:


I can't find where this undo method is implemented.


workunit **_as_** an xml...


@alonecuzzo - This allows the variable to be declared weak.


Oops, I committed this to the wrong repo.


I don't understand what you are doing. Zach already made a menu and I made it functional, why do you keep adding this menu?


This is a new dependency for Sugar (right now is a dependency for Browse, anyway). We should document, to make life easier to packagers.


Yeah, sorry, re-read `SSH()`. Your `err` here is encapsulating both the exit code and the stdout/stderr, so if a command exited non-0, I think it would just bail out here (at line 78), and you'd never see stdout/stderr, losing diagnostic data.


Er, is encapsulating both exit code and whether whether there's a session error, rather.


@andresgutierrez


MySQL. Registration works


Yeah, good catch, you are correct.

I wonder if there was a reason to use write instead of touch. touch wouldn't work on a file system which is mounted as `nomtime`, but nevertheless, the action shouldn't be named "touch" then :)

/cc @DoriftoShoes


Might be better to [define a type](https://atom.io/docs/api/v0.194.0/Config#defining-settings-for-your-package).


curious how this compares to spawn-async which I've been using elsewhere (eg. launching clang-format)


You are probably right. Would freeze_size be better ? It's not working yet anyway so there is time to discuss terminology ;)


of **_the_** types of activities (rather than 'which', I'm not sure why I missed this before)


love all the deletes


I can merge your pull request if you move the preferenceDir creation into the preferenceDir function ( no code duplication that way ).


I think translator are used to the %s placeholder. Probably is better continue using the same.


Alex you have some misspellings in the ChromaKey and Keyer warning text code - the word "dit": "OFX::Message::eMessageError, "", "OFX Host dit not take into account output components""


You didn't fix the users of this API. This is odd.


3assssssh @samaasallam


This seems like a really strange path to be heading down. Are we removing `map!` from the language? If we're not adding this new invariant there, I'm worried about the disconnect between `map` and `map!` that's being created by this.


Did I put in a `CHECK_OK`? This is fine too (the important is that it prints the status if it's not OK).


Here are some benchmarks:

    % cat hello.sh
    #!/bin/sh
    echo "Hello"

    % cat hello.rb
    #!/usr/bin/env ruby
    puts "Hello"

    % time ( for i in {1..1000}; do; ./hello.sh; done; )
    ...
    0,23s user 1,96s system 103% cpu 2,110 total

    % time ( for i in {1..1000}; do; ./hello.rb; done; )
    ...
    29,78s user 6,49s system 100% cpu 36,238 total

And using `/usr/bin/env bash`:

    % time ( for i in {1..1000}; do; ./hello.bash; done; )
    ...
    1,08s user 3,22s system 101% cpu 4,251 total


Not quite sure why this isn't just `SELECT hash FROM ...`? Whatever.


(Scott Daniel) Bugfix/CATSIM-188 arcseconds
There were some errors in sims_catUtils converting from arcseconds (as stored in the database) to radians (for manipulation in the stack).  This pull requests adds functions to facilitate the fix to that bug.

* added functions to convert between radians and arcsec to
observationMetadataUtils.py
* cleaned up whitespace


(Scott Daniel) This is the other pull request


(Yusra AlSayyad) Thank you.


(Scott Daniel) Bugfix/CATSIM-179 phoSim test is null
This is the pull request that was declined, plus modifications to the API for ObservationMetaData and SpatialBounds so that they store angles as radians (except in explicitly identified cases... there are variables like RAmaxDeg, RAminDeg that are in degrees)

* convert OpSim RA/Dec to degrees for obs_metadata
* updated spatialBounds.py to internally store degrees
* updated ObservationMetaData to store bounds in radians
* small bug
* forgot how I imported numpy in spatialBounds.py
* had to handle the possibility that Nones are passed to ObservationMetaData
* updated testObservationMetadata.py
* updated testSpatialBounds.py
* fixed whitespace
* cleaned up whitespace


(Simon Krughoff) Do these comments need to be corrected?


(Simon Krughoff) Unfortunately, I think you need to copy the docstring here even if you are inheriting.  Happy to be shown wrong!


(Simon Krughoff) Same here re: docstring.


(Simon Krughoff) This is right isn't it?  We should just get rid of this if we are convinced.


(Scott Daniel) I'm not sure I understand.  Are you questioning what our policy should be if min > max?  We could raise a RuntimeError in that case, just to be sure (this is legacy code, so I'm not aware of the philosophy behind it).


(Scott Daniel) yes. done


(Scott Daniel) done


(Simon Krughoff) Sorry, I was talking about the comment.  It's been in here for a while and we should just remove it if we fell confident that what we are doing after it is correct (and I think I'm convinced).


(Simon Krughoff) Feature/CATSIM-24 generate camera model for catsim
Added ability to limit the test database in RA


(Scott Daniel) Feature/CATSIM-159 galsim galaxy images
modifications in support of the galSim unit tests

* added half light radii to the phoSim test DBs.  Also made sure that major axis
is, in fact, larger than minor axis in galaxies
* allow for variable radius when constructing test DBObjects
* added ability to specify location of objects relative to center
to the makePhoSimTestDB method
* added docstrings


(Scott Daniel) Feature/CATSIM-111
Stores m5 as either a dict or a float in the ObservationMetaData class.  Makes the value accessible using the m5() method of that class.

* ObservationMetaData now stores m5 in either a float or a dict called m5value; it is returned using a method m5(filterName)
* indentation error
* misspelled 'isinstance'
* adding a unit test for the __init__ on ObservationMetaData
* test that ObservationMetaData __init__ throws an exception when it should
* testObservationMetaData.py runs


Actually, since the outputDirectory comes from the step view controller itself, it would make more sense to do this in a prepareStep override in the audio step view controller, so the recorder itself never gets instantiated, and so it's more obviously in the right codepath. An alternative would be to do it in recordersDidChange, just after the recorders are instantiated.


(Simon Krughoff) Can you update the documentation as well?


(Simon Krughoff) Do you want to check that the filterName is the same as bandassName?  Or, maybe the bandpassName should be removed from the metadata?  Hmm, I see now that it let's you set a single depth for all bands.  I guess that's nice.
I definitely think the m5 and filters should match up somehow, but now I'm not exactly sure how.


(Scott Daniel) At this point, m5 and bandpassName are variables that the user arbitrarily sets (I believe bandpassName is supposed to correspond to the PhoSim input Opsim_filter).  In a future ticket, should we add some kind of functionality to automatically calculate m5 given actual telescope and pointing data?


(Scott Daniel) Bugfix/CATSIM-211 rotskypos in radians
This branch fixes the inconsistency in how ObservationMetaData stores rotskypos.  It now stores the angle in radians.  init still accepts degrees (in keeping with our philosophy that "every angle humans see should be in degrees).

There are related branches in
sims_catalogs_measures
sims_coordUtils
sims_catUtils (this branch also updates the findChipName example to apply corrections to ra and dec before feeding them into findChipName)

* ObservationMetaData now stores rotskypos in radians
(it is still input in degrees if input by hand)
* tried to make docstring easier to read
* added capability to sample densely about a field of view
in unit test databases
* fixed bug in sampleFocus


(Scott Daniel) Bugfix/CATSIM-96 galaxies have no size
Sorry I'm burying you in pull requests.

This one is just a few lines of code.  I've moved the phoSim spatialModels out of sims_catalogs_generation and into sims_catUtils, as we discussed

* changed value of spatialModel in SsmModels.py to agree with what phoSim expects
* changed value of spatialModel in StarModels.py
* changed spatialModel in testUtils.py
* removed all mention of spatialModel from dbConnection.py
* removed all mention of spatialModel from GalaxyModels.py
* removed all mention of spatialModel from SsmModels.py
* removed all mention of spatialModel from StarModels.py
* removed all mention of spatialModel from testUtils.py
* removed all mention of spatialModel from MetaDataDBObject.py


(Scott Daniel) Bugfix/CATSIM-103 update obsMetaData
ObservationMetaData now sets unrefracted(RA,Dec) as follows

if box_bounds/circ_bounds exists and the user specifies a point in the bounds, use the user-specified point

if box_bounds/circ_bounds exists and the user specifies a point outside of the bounds (or does not specify a point at all), use the center of the bounds

otherwise, set them to None

* test behavior of unrefracted[RA,Dec] in presence of box_bounds
* put comments in testObservationMetaData.py
* refactored testObservationMetaData
* oops... one more conflict to resolve
* fixed bug relative to sims_catUtils/examples/generatePhosimInput.py
* did not realize that Opsim actually did store filters as strings, rather than ints
* fixed
* moved epoch from the DBObj class to ObservationMetaData
* oops... never mind; should not have pushed that last commit


(Simon Krughoff) I think this example will actually raise a KeyError, since it is passing a box_bounds to a circ_bounds.


But overall I'm coming round to throwing an exception.


For 180 MHz configuration the USB PLL will be impossible to configure. So 180 MHz is okay but this means no USB.
In general I think it would be good to create scaling clk driver to let the user choose. My friend has already made some initial code related to this (see code from my other branch: https://github.com/Varcain/linux_ARMv7M/blob/stm32_playground/drivers/clk/stm32/clk-stm32f429.c)


Have you considered naming these fields something like `location[address_2]`, that way it might be possible to reuse validation and other logic when creating locations or child forms.


hi, i want build some lollipop rom for p1 aka u9200, but where is kernel and hardware trees for cm12?


eclagentexec? Is this the actual name for this or just shorthand?


Yeah that's definitely possible. For the moment I'd like to get things up and running, but refactoring that to a more reusable state should be pretty quick. Right now there's nowhere else to "edit" your saved locations, but I imagine that will be a feature we add in the future.


Believe me I don't like the disconnect either. We can surely add an explicitely typed `map(f, ReturnType, [])` which is closer to `map!`.
On the plus side we then get rid of all the ugliness of `type_goto`, the fact that comprehensions don't act the same in toplevel vs in functions, the fact that `map(symbol, ["a"]) == [:a]` but `map(symbol, String[]) == String[]` (which by the way you cannot `push!` symbols into either...), ...

I'm not making any decision here, but it would be easier to discuss this with actual code to look at in a pull request than suppositions in an issue thread.


@jaykayone why is the scale set in the general attributes as opposed to be in the map specification?


I think my elf2flt utility doesn't set this bit in entry address correctly (and I guess it is the correct place to set it). I could hack this in the elf2flt itself but then I really, really don't like the FLAT binfmt (and the support we have for it for ARM).
Anyways - it was faster and easier to temporarily do it here. Which binfmt your busybox is compiled in? If it is in FLAT then where did you get elf2flt from?


First commit of project Arduino - File README.md


@littleguy77 everything else should be okay for gles3.x context creation ;-)


Noted, thank you for your detail explanation @richtr.


(Scott Daniel) Bugfix/CATSIM-188 arcseconds
There were some errors in sims_catUtils converting from arcseconds (as stored in the database) to radians (for manipulation in the stack).  This pull requests adds functions to facilitate the fix to that bug.

* added functions to convert between radians and arcsec to
observationMetadataUtils.py
* cleaned up whitespace


(Scott Daniel) This is the other pull request


(Scott Daniel) Bugfix/CATSIM-179 phoSim test is null
This is the pull request that was declined, plus modifications to the API for ObservationMetaData and SpatialBounds so that they store angles as radians (except in explicitly identified cases... there are variables like RAmaxDeg, RAminDeg that are in degrees)

* convert OpSim RA/Dec to degrees for obs_metadata
* updated spatialBounds.py to internally store degrees
* updated ObservationMetaData to store bounds in radians
* small bug
* forgot how I imported numpy in spatialBounds.py
* had to handle the possibility that Nones are passed to ObservationMetaData
* updated testObservationMetadata.py
* updated testSpatialBounds.py
* fixed whitespace
* cleaned up whitespace


(Simon Krughoff) Do these comments need to be corrected?


(Simon Krughoff) Feature/CATSIM-24 generate camera model for catsim
Added ability to limit the test database in RA


(Scott Daniel) Feature/CATSIM-159 galsim galaxy images
modifications in support of the galSim unit tests

* added half light radii to the phoSim test DBs.  Also made sure that major axis
is, in fact, larger than minor axis in galaxies
* allow for variable radius when constructing test DBObjects
* added ability to specify location of objects relative to center
to the makePhoSimTestDB method
* added docstrings


(Scott Daniel) Feature/CATSIM-111
Stores m5 as either a dict or a float in the ObservationMetaData class.  Makes the value accessible using the m5() method of that class.

* ObservationMetaData now stores m5 in either a float or a dict called m5value; it is returned using a method m5(filterName)
* indentation error
* misspelled 'isinstance'
* adding a unit test for the __init__ on ObservationMetaData
* test that ObservationMetaData __init__ throws an exception when it should
* testObservationMetaData.py runs


(Simon Krughoff) Can you update the documentation as well?


We may want to call this "parallel" instead of partitioned.


Regression: Need to update pre-build step with correct file names.


(Scott Daniel) Bugfix/CATSIM-211 rotskypos in radians
This branch fixes the inconsistency in how ObservationMetaData stores rotskypos.  It now stores the angle in radians.  init still accepts degrees (in keeping with our philosophy that "every angle humans see should be in degrees).

There are related branches in
sims_catalogs_measures
sims_coordUtils
sims_catUtils (this branch also updates the findChipName example to apply corrections to ra and dec before feeding them into findChipName)

* ObservationMetaData now stores rotskypos in radians
(it is still input in degrees if input by hand)
* tried to make docstring easier to read
* added capability to sample densely about a field of view
in unit test databases
* fixed bug in sampleFocus


(Scott Daniel) Bugfix/CATSIM-96 galaxies have no size
Sorry I'm burying you in pull requests.

This one is just a few lines of code.  I've moved the phoSim spatialModels out of sims_catalogs_generation and into sims_catUtils, as we discussed

* changed value of spatialModel in SsmModels.py to agree with what phoSim expects
* changed value of spatialModel in StarModels.py
* changed spatialModel in testUtils.py
* removed all mention of spatialModel from dbConnection.py
* removed all mention of spatialModel from GalaxyModels.py
* removed all mention of spatialModel from SsmModels.py
* removed all mention of spatialModel from StarModels.py
* removed all mention of spatialModel from testUtils.py
* removed all mention of spatialModel from MetaDataDBObject.py


(Scott Daniel) Bugfix/CATSIM-103 update obsMetaData
ObservationMetaData now sets unrefracted(RA,Dec) as follows

if box_bounds/circ_bounds exists and the user specifies a point in the bounds, use the user-specified point

if box_bounds/circ_bounds exists and the user specifies a point outside of the bounds (or does not specify a point at all), use the center of the bounds

otherwise, set them to None

* test behavior of unrefracted[RA,Dec] in presence of box_bounds
* put comments in testObservationMetaData.py
* refactored testObservationMetaData
* oops... one more conflict to resolve
* fixed bug relative to sims_catUtils/examples/generatePhosimInput.py
* did not realize that Opsim actually did store filters as strings, rather than ints
* fixed
* moved epoch from the DBObj class to ObservationMetaData
* oops... never mind; should not have pushed that last commit


(Simon Krughoff) I think this example will actually raise a KeyError, since it is passing a box_bounds to a circ_bounds.


Is it merged to the 3.0 m1 ?


Is it possible to create a base class for the ArrayPrepend and ArrayAppend that was already done and make the common code go to the base class?  I think it could be possible.  It would ensure that these initial checks that we do can go inside that. What do you think?


:cool:


FYI I was able to figure out how to get rid of the targets when using ```-debugMode``` see #58


WTF?


mercadojeferson.github.io


Update this to redirect to actual site. Hardcoding would work, but consider using something like <%= @baseurl + "/exams/new?exam=12" %> to allow for local debugging and live functionality all in one.


Needs to be wrapped in an `unchecked` box


Apresenta a lista com o resultado e volta pra tela inicial.


documento ref #1


(Scott Daniel) Bugfix/CATSIM-188 arcseconds
There were some errors in sims_catUtils converting from arcseconds (as stored in the database) to radians (for manipulation in the stack).  This pull requests adds functions to facilitate the fix to that bug.

* added functions to convert between radians and arcsec to
observationMetadataUtils.py
* cleaned up whitespace


(Scott Daniel) This is the other pull request


(Scott Daniel) Bugfix/CATSIM-179 phoSim test is null
This is the pull request that was declined, plus modifications to the API for ObservationMetaData and SpatialBounds so that they store angles as radians (except in explicitly identified cases... there are variables like RAmaxDeg, RAminDeg that are in degrees)

* convert OpSim RA/Dec to degrees for obs_metadata
* updated spatialBounds.py to internally store degrees
* updated ObservationMetaData to store bounds in radians
* small bug
* forgot how I imported numpy in spatialBounds.py
* had to handle the possibility that Nones are passed to ObservationMetaData
* updated testObservationMetadata.py
* updated testSpatialBounds.py
* fixed whitespace
* cleaned up whitespace


(Simon Krughoff) Do these comments need to be corrected?


(Simon Krughoff) Feature/CATSIM-24 generate camera model for catsim
Added ability to limit the test database in RA


(Scott Daniel) Feature/CATSIM-159 galsim galaxy images
modifications in support of the galSim unit tests

* added half light radii to the phoSim test DBs.  Also made sure that major axis
is, in fact, larger than minor axis in galaxies
* allow for variable radius when constructing test DBObjects
* added ability to specify location of objects relative to center
to the makePhoSimTestDB method
* added docstrings


(Scott Daniel) Feature/CATSIM-111
Stores m5 as either a dict or a float in the ObservationMetaData class.  Makes the value accessible using the m5() method of that class.

* ObservationMetaData now stores m5 in either a float or a dict called m5value; it is returned using a method m5(filterName)
* indentation error
* misspelled 'isinstance'
* adding a unit test for the __init__ on ObservationMetaData
* test that ObservationMetaData __init__ throws an exception when it should
* testObservationMetaData.py runs


(Simon Krughoff) Can you update the documentation as well?


(Scott Daniel) Bugfix/CATSIM-211 rotskypos in radians
This branch fixes the inconsistency in how ObservationMetaData stores rotskypos.  It now stores the angle in radians.  init still accepts degrees (in keeping with our philosophy that "every angle humans see should be in degrees).

There are related branches in
sims_catalogs_measures
sims_coordUtils
sims_catUtils (this branch also updates the findChipName example to apply corrections to ra and dec before feeding them into findChipName)

* ObservationMetaData now stores rotskypos in radians
(it is still input in degrees if input by hand)
* tried to make docstring easier to read
* added capability to sample densely about a field of view
in unit test databases
* fixed bug in sampleFocus


We need a better and more friendly general error message. And also we need to differentiate error messages by http status code (5XX,4XX).


(Scott Daniel) Bugfix/CATSIM-96 galaxies have no size
Sorry I'm burying you in pull requests.

This one is just a few lines of code.  I've moved the phoSim spatialModels out of sims_catalogs_generation and into sims_catUtils, as we discussed

* changed value of spatialModel in SsmModels.py to agree with what phoSim expects
* changed value of spatialModel in StarModels.py
* changed spatialModel in testUtils.py
* removed all mention of spatialModel from dbConnection.py
* removed all mention of spatialModel from GalaxyModels.py
* removed all mention of spatialModel from SsmModels.py
* removed all mention of spatialModel from StarModels.py
* removed all mention of spatialModel from testUtils.py
* removed all mention of spatialModel from MetaDataDBObject.py


(Scott Daniel) Bugfix/CATSIM-103 update obsMetaData
ObservationMetaData now sets unrefracted(RA,Dec) as follows

if box_bounds/circ_bounds exists and the user specifies a point in the bounds, use the user-specified point

if box_bounds/circ_bounds exists and the user specifies a point outside of the bounds (or does not specify a point at all), use the center of the bounds

otherwise, set them to None

* test behavior of unrefracted[RA,Dec] in presence of box_bounds
* put comments in testObservationMetaData.py
* refactored testObservationMetaData
* oops... one more conflict to resolve
* fixed bug relative to sims_catUtils/examples/generatePhosimInput.py
* did not realize that Opsim actually did store filters as strings, rather than ints
* fixed
* moved epoch from the DBObj class to ObservationMetaData
* oops... never mind; should not have pushed that last commit


(Simon Krughoff) I think this example will actually raise a KeyError, since it is passing a box_bounds to a circ_bounds.


This feels suboptimal : we are looping over each field name, and then performing a table lookup on the field name. Can't we just iterate all the pairs of (field name, value) instead?


It's actually due today:

Code due: Monday 27th April 2015 at or before 13:00

I'm gonna get to work on it right now.
Ruben


walk-through**,** details


this commit breaks images not having the latest PLi-changes regarding skin-parameters (e.g. openatv).

Traceback (most recent call last):
  File "/usr/lib/enigma2/python/Components/ActionMap.py", line 68, in action
  File "/usr/lib/enigma2/python/Components/ActionMap.py", line 48, in action
  File "/usr/lib/enigma2/python/Screens/Menu.py", line 55, in okbuttonClick
  File "/usr/lib/enigma2/python/Tools/BoundFunction.py", line 9, in __call__
  File "/usr/lib/enigma2/python/Plugins/Extensions/AutoTimer/plugin.py", line 121, in main
  File "/usr/lib/enigma2/python/Plugins/Extensions/AutoTimer/AutoTimerOverview.py", line 16, in <module>
  File "/usr/lib/enigma2/python/Plugins/Extensions/AutoTimer/AutoTimerList.py", line 8, in <module>
ImportError: cannot import name parameters


Other than moving all this to a separate file, I believe the only change was that I discovered that AddHintBubbles() was sending two more parameters to MakeHelpBubble() that weren't ever getting used. For instance, MakeHelpBubble() figures out 'whatToSay' by itself even tho' AddHintBubbles() seems to try to send it as a parameter. So I just modified the AddHintBubbles() to not bother getting information that wasn't going to be used anyway.


I love it when you're forceful :)


Good idea, i have implemented the idea in feature/parentContainerFix branch.


atv now dead :(


You need to provide validation for all of these ui components.


change castIdentities to passthrough too.


Applying John Thomson's earlier code review...


:) lol


You need to make sure all errors are handled here.


should create test coverage for all AMD invocations?


r=MonsieurNicolas


(Scott Daniel) Bugfix/CATSIM-188 arcseconds
There were some errors in sims_catUtils converting from arcseconds (as stored in the database) to radians (for manipulation in the stack).  This pull requests adds functions to facilitate the fix to that bug.

* added functions to convert between radians and arcsec to
observationMetadataUtils.py
* cleaned up whitespace


(Scott Daniel) This is the other pull request


(Scott Daniel) Bugfix/CATSIM-179 phoSim test is null
This is the pull request that was declined, plus modifications to the API for ObservationMetaData and SpatialBounds so that they store angles as radians (except in explicitly identified cases... there are variables like RAmaxDeg, RAminDeg that are in degrees)

* convert OpSim RA/Dec to degrees for obs_metadata
* updated spatialBounds.py to internally store degrees
* updated ObservationMetaData to store bounds in radians
* small bug
* forgot how I imported numpy in spatialBounds.py
* had to handle the possibility that Nones are passed to ObservationMetaData
* updated testObservationMetadata.py
* updated testSpatialBounds.py
* fixed whitespace
* cleaned up whitespace


(Simon Krughoff) Do these comments need to be corrected?


(Simon Krughoff) Feature/CATSIM-24 generate camera model for catsim
Added ability to limit the test database in RA


(Scott Daniel) Feature/CATSIM-159 galsim galaxy images
modifications in support of the galSim unit tests

* added half light radii to the phoSim test DBs.  Also made sure that major axis
is, in fact, larger than minor axis in galaxies
* allow for variable radius when constructing test DBObjects
* added ability to specify location of objects relative to center
to the makePhoSimTestDB method
* added docstrings


(Scott Daniel) Feature/CATSIM-111
Stores m5 as either a dict or a float in the ObservationMetaData class.  Makes the value accessible using the m5() method of that class.

* ObservationMetaData now stores m5 in either a float or a dict called m5value; it is returned using a method m5(filterName)
* indentation error
* misspelled 'isinstance'
* adding a unit test for the __init__ on ObservationMetaData
* test that ObservationMetaData __init__ throws an exception when it should
* testObservationMetaData.py runs


(Simon Krughoff) Can you update the documentation as well?


(Scott Daniel) Bugfix/CATSIM-211 rotskypos in radians
This branch fixes the inconsistency in how ObservationMetaData stores rotskypos.  It now stores the angle in radians.  init still accepts degrees (in keeping with our philosophy that "every angle humans see should be in degrees).

There are related branches in
sims_catalogs_measures
sims_coordUtils
sims_catUtils (this branch also updates the findChipName example to apply corrections to ra and dec before feeding them into findChipName)

* ObservationMetaData now stores rotskypos in radians
(it is still input in degrees if input by hand)
* tried to make docstring easier to read
* added capability to sample densely about a field of view
in unit test databases
* fixed bug in sampleFocus


(Scott Daniel) Bugfix/CATSIM-96 galaxies have no size
Sorry I'm burying you in pull requests.

This one is just a few lines of code.  I've moved the phoSim spatialModels out of sims_catalogs_generation and into sims_catUtils, as we discussed

* changed value of spatialModel in SsmModels.py to agree with what phoSim expects
* changed value of spatialModel in StarModels.py
* changed spatialModel in testUtils.py
* removed all mention of spatialModel from dbConnection.py
* removed all mention of spatialModel from GalaxyModels.py
* removed all mention of spatialModel from SsmModels.py
* removed all mention of spatialModel from StarModels.py
* removed all mention of spatialModel from testUtils.py
* removed all mention of spatialModel from MetaDataDBObject.py


(Scott Daniel) Bugfix/CATSIM-103 update obsMetaData
ObservationMetaData now sets unrefracted(RA,Dec) as follows

if box_bounds/circ_bounds exists and the user specifies a point in the bounds, use the user-specified point

if box_bounds/circ_bounds exists and the user specifies a point outside of the bounds (or does not specify a point at all), use the center of the bounds

otherwise, set them to None

* test behavior of unrefracted[RA,Dec] in presence of box_bounds
* put comments in testObservationMetaData.py
* refactored testObservationMetaData
* oops... one more conflict to resolve
* fixed bug relative to sims_catUtils/examples/generatePhosimInput.py
* did not realize that Opsim actually did store filters as strings, rather than ints
* fixed
* moved epoch from the DBObj class to ObservationMetaData
* oops... never mind; should not have pushed that last commit


(Simon Krughoff) I think this example will actually raise a KeyError, since it is passing a box_bounds to a circ_bounds.


@ufl-taeber That's awesome – glad you've found it useful! Speaking of other language implementations, we've floated the idea in the past of creating a language-agnostic description of the parsing instructions ([see this comment](https://github.com/blue-button/bluebutton.js/issues/79#issuecomment-29119229)), so the only thing we'd have to write from a language-specific perspective is the engine to parse those instructions and build the resulting JSON. It'd be a much cleaner, declarative approach, but as always developing a [general solution](https://xkcd.com/974/) may be more effort than it's worth.

If you're ever interested in talking strategy here, let me and @sankarravi know.


This reader tools stuff goes to readerTools.ts in a later commit.


The previous pull for this (#50) used the flag `--no-refresh` to gain quite a boost in execution time. Did you consider using it here?


saw approval from MonsieurNicolas
at https://github.com/gmarceau/stellar-core/commit/e00658d5d71b52464a7b58687a5f9e2681756632


merging gmarceau/stellar-core/item-fetching-cleanup = e00658d5 into auto


gmarceau/stellar-core/item-fetching-cleanup = e00658d5 merged ok, testing candidate = 6899cb86


Это явно лучше, но всетаки название комментария должно отражать суть изменений, а не ссылаться на прошлое.


This moved to bloomQtipUtils, since more than one functional area uses it.


just moving stuff around


An example of how to use it in drupal render array definition.
      $image['#attributes'] = array(
        'data-parent-container' => '.Slideshow-items',
      );
Ad you can see you have to pass the parent selector and not the class.


Did this "API" been upgraded since this time ?


It's funny how you always end up in the same place as a coder, if you want to do something truly interesting.

On the one hand, it's fun to work on solving a *new* problem, instead of just playing catch-up to existing solutions.

On the other hand, there's always those moments when you realize *why* other people haven't tackled this particular problem.  ;)


qtip stuff that either didn't fit in any of the functional areas, or needed to be used by more than one of them.


The guarantee made by the datasource here is only "We will always send the appearance events to the right component tree" :
- Use dequeue and prepare for reuse to stash the output item in the cell, could use the appearance events but then it would require the comsumer to call willAppear to have didDisappear being called, possible to do so.
- The output item will obviously be stashed privately (private property or even associated object). An external index cell->model could be used as well but we would like a weak to strong relationship to not retain cells for no reason. NSMapTable has issues with those configurations and might keep output items in memory longer than needed (i.e even when the cell is gone) if a didDisappear is missed (which happens in general and is also out of our control because the consumer decides to call this). Quite unlikely there will be much of those cases because of reuse though.
Anyway those solutions are conceptually equivalent it's mostly an implementation detail.

If we want to move the contract to "We will send the disappearance events to the component tree if the appearance events were called", we can set and unset the output item in the ` announceWillAppearForItemInCell:atIndexPath` and the ` announceDidDisappearForItemInCell:atIndexPath`.
The same discussion applies regarding wether or not we stash the output item in the cell or keep a weakToStrong index and a weakToStrong NSMapTable would be more appropriate in this scenario IMO.


unfinished moving business


I disagree that we have to import almost everything into extensions. We have a lot of UI code. Almost none of it needs to be in there. Neither does any of the storage code (because its a framework, and we can import it only where its needed). Encapsulation is good, especially in Swift where keeping code in small well contained files and access-control seem to be at odds with one another (I assume they just assume the XCode interface is so awesome it doesn't matter).

This buys us quite a bit. Importing the TabManager into Extensions requires importing Browsers in which requires importing the WebServer in (along with other things). WebServer.swift gets "Undefined symbols for architectures X" errors when you include it in extensions. If I go into GCDWebServer itself and start including files in the project you see things like "UIApplication.sharedApplication is unavailable in App Extensions". I don't want to derail this sync feature rewriting GCDWebServer. I'm agree with renaming things. "ExtensionProfile"? That would make it clear to only put things in there if extensions need it (but doesn't make it clear that its not a subset of "Profile"?) Maybe "ExtensionProfile" and "AppProfile"?


I don't understand how this is different from the original use of local.


I don't mean to bikeshed, but I've more often seen "err" used as the error key. That's what I use, and see also https://github.com/inconshreveable/log15/issues/14#issuecomment-49221518


Webstorm never did like this... even tho' it seemed to work. So I changed how it is done when I moved it to .ts by making it a function.


(Scott Daniel) Bugfix/CATSIM-188 arcseconds
There were some errors in sims_catUtils converting from arcseconds (as stored in the database) to radians (for manipulation in the stack).  This pull requests adds functions to facilitate the fix to that bug.

* added functions to convert between radians and arcsec to
observationMetadataUtils.py
* cleaned up whitespace


(Scott Daniel) This is the other pull request


(Scott Daniel) Bugfix/CATSIM-179 phoSim test is null
This is the pull request that was declined, plus modifications to the API for ObservationMetaData and SpatialBounds so that they store angles as radians (except in explicitly identified cases... there are variables like RAmaxDeg, RAminDeg that are in degrees)

* convert OpSim RA/Dec to degrees for obs_metadata
* updated spatialBounds.py to internally store degrees
* updated ObservationMetaData to store bounds in radians
* small bug
* forgot how I imported numpy in spatialBounds.py
* had to handle the possibility that Nones are passed to ObservationMetaData
* updated testObservationMetadata.py
* updated testSpatialBounds.py
* fixed whitespace
* cleaned up whitespace


(Simon Krughoff) Do these comments need to be corrected?


(Simon Krughoff) Feature/CATSIM-24 generate camera model for catsim
Added ability to limit the test database in RA


(Scott Daniel) Feature/CATSIM-159 galsim galaxy images
modifications in support of the galSim unit tests

* added half light radii to the phoSim test DBs.  Also made sure that major axis
is, in fact, larger than minor axis in galaxies
* allow for variable radius when constructing test DBObjects
* added ability to specify location of objects relative to center
to the makePhoSimTestDB method
* added docstrings


(Scott Daniel) Feature/CATSIM-111
Stores m5 as either a dict or a float in the ObservationMetaData class.  Makes the value accessible using the m5() method of that class.

* ObservationMetaData now stores m5 in either a float or a dict called m5value; it is returned using a method m5(filterName)
* indentation error
* misspelled 'isinstance'
* adding a unit test for the __init__ on ObservationMetaData
* test that ObservationMetaData __init__ throws an exception when it should
* testObservationMetaData.py runs


(Simon Krughoff) Can you update the documentation as well?


(Scott Daniel) Bugfix/CATSIM-211 rotskypos in radians
This branch fixes the inconsistency in how ObservationMetaData stores rotskypos.  It now stores the angle in radians.  init still accepts degrees (in keeping with our philosophy that "every angle humans see should be in degrees).

There are related branches in
sims_catalogs_measures
sims_coordUtils
sims_catUtils (this branch also updates the findChipName example to apply corrections to ra and dec before feeding them into findChipName)

* ObservationMetaData now stores rotskypos in radians
(it is still input in degrees if input by hand)
* tried to make docstring easier to read
* added capability to sample densely about a field of view
in unit test databases
* fixed bug in sampleFocus


(Scott Daniel) Bugfix/CATSIM-96 galaxies have no size
Sorry I'm burying you in pull requests.

This one is just a few lines of code.  I've moved the phoSim spatialModels out of sims_catalogs_generation and into sims_catUtils, as we discussed

* changed value of spatialModel in SsmModels.py to agree with what phoSim expects
* changed value of spatialModel in StarModels.py
* changed spatialModel in testUtils.py
* removed all mention of spatialModel from dbConnection.py
* removed all mention of spatialModel from GalaxyModels.py
* removed all mention of spatialModel from SsmModels.py
* removed all mention of spatialModel from StarModels.py
* removed all mention of spatialModel from testUtils.py
* removed all mention of spatialModel from MetaDataDBObject.py