Je commit les icons de SIMU. Il faudra faire quelques icons avec la SIMU
pour les couleurs, qui on reçu pour le moment un simple tick.

Je commit trois, deux, un ........

On Wed, Nov 23, 2016 at 5:48 PM, Ingwie <notifications@github.com> wrote:

> Yes !! S'il te plais
>
> Je dois finir le splash perso en priorit�.
>
> ----- Original Message -----
> From: jmentero <notifications@github.com>
> To: Ingwie/NextStepRc_Dev <NextStepRc_Dev@noreply.github.com>
> Cc: Ingwie <theingwie@gmail.com>, Comment <comment@noreply.github.com>
> Date: Wed, 23 Nov 2016 08:47:19 -0800
> Subject: Re: [Ingwie/NextStepRc_Dev] Menu icons added. (9f8543b)
>
> > Si tu veux, je me met a �a. C'est facile
> >
> > On Wed, Nov 23, 2016 at 5:32 PM, Ingwie <notifications@github.com>
> wrote:
> >
> > > ?? Cool But ... How about Simu witch have NO icons ;-(
> > >
> > > Thanks Miguel
> > >
> > > �
> > > You are receiving this because you authored the thread.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/Ingwie/NextStepRc_Dev/commit/
> 9f8543bf62cdee2b7c2580fed0901187dbde6b43#commitcomment-19940712>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/AVF8dQhx-
> PUfiYpRboQVIGTFPEAU6u09ks5rBGqhgaJpZM4K6y4s>
> > > .
> > >
> >
> >
> > --
> > You are receiving this because you commented.
> > Reply to this email directly or view it on GitHub:
> > https://github.com/Ingwie/NextStepRc_Dev/commit/
> 9f8543bf62cdee2b7c2580fed0901187dbde6b43#commitcomment-19940981
> >
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/Ingwie/NextStepRc_Dev/commit/9f8543bf62cdee2b7c2580fed0901187dbde6b43#commitcomment-19941017>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AVF8dROClCdEip4qfmjwFRs3Yk6e0hFeks5rBG54gaJpZM4K6y4s>
> .
>


Je vois ça parfois.  Là j'ai utilisé ça pour être consistent avec nr_rows.


yes, i was too lazy to fix it. i was thinking maybe i should have four labels, two of them dont change just say "Memory : " and "Text: " and the other two are the ones updated by the ActionListener


done


Wooops, thanks!
Maybe we shouldn't make this special case for LLVM error.


i had to cheat and copy some code from the oracle tutorials, but it works!


I got an email from them because it's my account that the project is on,
and they must've found it using a bot of some kind haha

On Nov 23, 2016 10:39 AM, "Jayme Greer" <notifications@github.com> wrote:

> LMAO this is so funny. How did they find this? Also, how did you find it
> Kale?
>
> > On Nov 22, 2016, at 6:39 PM, JDeeth <notifications@github.com> wrote:
> >
> > Hah. Your commit got picked up by a Twitter bot. Well done :D
> >
> > https://twitter.com/callouscommits/status/801243412288643072 <
> https://twitter.com/callouscommits/status/801243412288643072>
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub <https://github.com/
> CaptainKirk1609/CSCI499/commit/ab1aa7e24304991d18a80391e73beb
> 3ff62cf22f#commitcomment-19931738>, or mute the thread <
> https://github.com/notifications/unsubscribe-auth/
> AQgKCTx2uSzS5zEfyBM49KvkcXqLlfogks5rA6dUgaJpZM4K6Kv6>.
> >
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/CaptainKirk1609/CSCI499/commit/ab1aa7e24304991d18a80391e73beb3ff62cf22f#commitcomment-19942478>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ARzjDGIlzn2ftS6FUp1fRS0eoQwkWDmmks5rBIh7gaJpZM4K6Kv6>
> .
>


Your condition if(pos.pieces(Us, BISHOP) & ei.attackedBy[Us][KING]) seems odd, because its triggers if one bishop around the king exists. So if you have two bishops but only one defends the king the bonus is added for each bishop.

I think if(ei.attackedBy[Us][KING] & s) is better because its triggers only if the current bishop is near the king.

Hint: the order of operands is important else the result is changed!


Oui si possible.


I considered that, but ultimately chose to keep the API simple. Since the event listeners are closures, I assumed that you'd capture a weak reference to the view if you needed it. What do you think?


Either seems like a reasonable choice but I chose "settings_redis" as it is more specific (it is a real block of configuration required for settings in redis).


@matumaros I understand. Sorry for my bad knowledge ;(


Starting from Ruby 2.3.0, `Pathname.new('a/b/c').ascend` automatically returns an enumerator, so we shouldn't need the `enum_for`.


Natasha Ruf


I tried that - we only have 2.1.0 in CI.


I'm now getting a bunch of redeclaration warning, even with this patch


I had a problem with the launch of an animation on a particular view. I had to add the weak links


But apparently we use a old outdated rubysl-pathname.


@walkah Oh, the suspense -- it's killing me! ;-)


Решил проблему с утечкой памяти ProductPresenter
Пикассо не загружает картинки хотя все делал как в уроке


@Godod Don't be sorry, I'm glad that I can teach you something too :D You are teaching me a lot of new and better things about Django.


the function declaration is fine
```javascript
function theVault(){
  ...
}
```
however, when you use the module, you must invoke theVault() function and keep a reference to the module that is returned
```javascript
let myVault = theVault();

myVault.setValue('playerName', '');
myVault.setValue('score', 0);
myVault.setValue('isDead', false);
myVault.setValue('promoCode', undefined);

let playerName = myVault.getValue('playerName');
let score = myVault.getValue('score');
let isDead = myVault.getValue('isDead');
let promoCode = myVault.getValue('promoCode');

console.log('playerName should be "", actual value:', playerName);
console.log('score should be 0, actual value:', score);
console.log('isDead should be false, actual value:', isDead);
console.log('promoCode should be undefined, actual value:', promoCode);
```

example jsBin: [https://jsbin.com/foguviziqa/edit?js,console](https://jsbin.com/foguviziqa/edit?js,console)


@PostPollux I replicated a Nuke plugin, but it was my own, so it's cool :)
I meant "real name" as I intended it to be. In the case of Feather, I have some sentimental value to the naming, as the tool roots from my very first supervisor/mentor from whom I learned a lot. Some sort of respect, you know. I mean after all it could be named 'Matty McSoftedge' and it wouldn't be a real difference :)
_adv stands for advanced, I like to keep the names of all my tools across apps identical though, it makes not much sense in Natron I'm with you on that.

@olear where's the fun in generic names :P
all jokes aside, I feel like it really helps to distinguish between stock-nodes and PyPlugs. Personally I have a collection of tools I choose myself, but with an installer lots of users have tools installed they have never heard of, or don't know how to use. Therefore I like to keep it a bit "funny", like in ChillSpill or Tweaky, to make the cut between stock+external clear.
If you insist, I can rename all of the tools to their exact use (like Feather and RoughenEdges lately), we can also drop the lp_ if you like. Adding of the initials is somewhat normal for Nuke that's my motivation behind it, but as explained above I don't insist on using that in general; if it's not appreciated, we can drop all of that.

I plan some updates on ChillSpill, Tweaky and even the very new Feather, but I won't be able to pull it off that fast as I also have to do some additional testing and development. So those will more likely get into a next release, if there is one :)


That is what the "skip_run" and "skip_loop" functions do, pass an empty lambda. The problem is that the thread that skips may not be the assigned thread, so isMyNextTask will be correctly false. Thus, we need a way to detect and handle that situation.


@IlyaUmanets in case you really want unique arrays, use sets


et donc faire une requête ajax pour chaque modification ?
Je vois pas trop...


What the hell! I'm lost.


This is a great idea!


Too soon, Reto. Too soon.


I think this should have been under the Interpreter heading, the DLL is already there as it should. Opening an issue would be nice for future updates. Thanks.


FYI we have already this in dda_math.h
https://github.com/Traumflug/Teacup_Firmware/blob/experimental/dda_maths.h#L26
`#define UM_PER_METER (1000000UL)`


@Zaicon Then why is it not fixed, clearly using this.


Do you see the weak links as a problem or non-obvious? I'm not opposed to adding the views as associated values, but I'd also want to pass the config as well. Just seemed a little messy to me.


I'm also pretty sure this will transfer the text "Input text" together with the typed text. It's fine if that's what you want, but I assume you don't. Otherwise bretty gud!


I'm pretty sure you don't need this if you're already typing into the text field.


FixDaylightingTableZone (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2340 of 2340 tests passed, 570 test warnings)

 * 570 tests had: AUD diffs.

![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)


Just a month… in comparison to eleven ;-)


Okay.


@jdalton

I didn't know how else to ask you a question.  You appear to have blocked me from the entire lodash github org.  Was that deliberate?  And if so why?


This is only "szőrszálhasogatás", but according to our styleguide these should be on separate lines:
https://google.github.io/styleguide/javaguide.html#s4.8.2-variable-declarations


new api the BEST


test comment


Good catch.

That's fine.  Or maybe make preload_index() not static and call it directly from here.  That might make the intent more clear.


@thc202 would this Override be required here?


@BrennanConroy says it needs to be done and the reasons are "it's complicated"... I did not dig further


I'm pretty sure I know what the issue is, but a GSDump could be helpful.


This is my final project for the year. I am making a website that links to other websites to get the latest news from the verge and cnet.


@derick-hess
You removed ph5API_object from the constructor, causing the web service code to break. We need to be more careful, as we cannot have mistakes like this when the service is live.


The `nil` check is not needed: https://golang.org/pkg/builtin/#len

> if v is nil, len(v) is zero.


Removed in bb5186f597f9d33cf7c44e02c721bb07d1acc2e3.


I would probably change the name to setUpFrame, because you set the fields of the UIControls class, which extends JFrame.


But this is only run by JRuby+Truffle itself, isn't it?


I don't think we need this. We already have settings as a String -> Object map in cluster configuration. The value is not evaluated until execution time when we write the map into elasticsearch.yml. So you can do things like this using GString and closures to delay evaluating something until runtime.

```
integTest {
  cluster {
    setting 'some.setting.name', "${-> some.expression.that.will.be.evaluted.at.runtime()}"
  }
}
```


@FlatOutPS2 : Already added to the first post.


Hmmm yeah good point. I think it was failing in CI because of it.


So 0x02 is name length and 0x03 is the start of the name depending on it's length. Well, that sure as hell helps when dealing with the inventory and trying not to crash the Wii U LOL


This tests that non-existing paths are not added to the chroot. Was it removed because it breaks in some unexpected way with the patch? If that's the case, it sounds like a regression.


I see why you did this though, since you can't support the * wildcard and have a ph5API object passed to the constructor. Is this correct? If this is the case, I should probably handle this somehow in the web service code, and we should revert back to the previous version of ph5tomsAPI.


Thanks. 👍 Your new post did show up, but the edit with the dump didn't show up until I refreshed the page.


Funny names are ok :) I just think that they should be one word without _ prefix/suffix, but it's not important ;)

When you make custom plugs for own use etc then I of course understand that _ may by normal, but when we distribute plugins for Natron we should just blend in with the rest.

Anyway, thanks for your continued contributions :)

I will do a quick test of the plugins and the installer a bit later, then release shortly after Natron 2.1.9.


Yep. That's why it's a hell of a fun to work with the offsets for this game... and why i dont use hardcoded offsets except for the name length and name start offset.


Why is radix included in the test? It doesn't seem to be directly used by this file; seems weird that adding an assignment `ctx.ChrootEnvTree = GetChrootEnvTree(ctx.ChrootEnv)` would require the implementation ...


Unlike the others props, `changed` will be set `true` on `onChange` and will not be reverted to `false` after you interact with the input.


Yes, it's correct


Yes, it should have it, it's implementing a method of ValueGenerator.


Actually, I personally don't remember Sen Meg being a thing until G2 and Bears didn't start dropping Empty Treasure Chest until some time after launch which would corresponds with the area being introduced along with Fleta for G2. It's very unlikely that a Sen Meg existed without either Fleta or Peaca Dungeon. I kind of remember Peaca dungeon being closed until G3.


Sorry, you are totally right, I will fix it in the next release.


Thank you for your suggestion! Will be done.


pode trocar por
```
for _, label in ipairs(labels) do
  box.child.simple:append_text(label)
end
```


See this ? So revisit boot sequence bowing down to the complexity accumulated by developers 10 years now. In fact I have an issue for that - which I have put in the backburner cause that's tricky - and standalone as you see in this module needs hacks on its own. So just make small changes - well tested

https://github.com/wrye-bash/wrye-bash/issues/244

If you are to look into the boot sequence the number one thing that needs to be addressed is wx importing. I want this to be done high up and once and for all. This blocks the work on wx3/phoenix - once we have this done in a central place we can then add a command line switch to test wx3

Actually I have done this in bass but I am not satisfied with this code - just pushing for you to see:

https://github.com/wrye-bash/wrye-bash/commit/fcdf47d4a99a0b269fe8619748c7e2235b5e16a6

EEE in my wip commits stands for edit...


BTW I forgot to add that the game also suffers from flickering in progressive scan mode (can be seen with the gs dump since it was made in progressive scan mode).


Fixed.


thx a lot for this! :+1:  must have been very annoying to debug


nunca use a variável `_`; se você precisar do valor, renomeie a variável (aqui, para `i` estaria bom)


ta zmiana nic nie wniosła


Isso não é uma closure.


commit and push more, otherwise there is nothing to review! :octocat:


a wniosła bardzo dużo


nie widać


nice one, thanks for fixing.


That's similar to how how we ran our cashregister.js and calculator.js pages so I could access my calculator functions. I did this initially, based off your comments to DavidE's code, and I received 'null' for all answer and thought I should receive the values by what was created in setValue. Like it created the 'key' but didnt give it a 'value'?


The function is generalized, but still refers to something specific ("cluster code")


Thanks for fixing it @cdlaguna


ouin j'avoue que je suis pas sur non plus


źle patrzysz


I thought we were supposed to overwrite the memory label with just the previous input, but this might make more sense. Though if you end up with a long list, using a text area might be better, so you can wrap the lines.


`somPonto` não é um `enum`.
Os nomes possíveis de arquivos não são finitos, então não pode ser um `enum`.


good point, maybe "related code"?


En fait j'en ai pas mis dans get-next-pc et ca ete ok


UP Next is ERD:
Helpful resources - https://docs.oracle.com/cd/E19253-01/817-6089/mg_appendixA.html


Ha ouin je comprend vraiment pas la


Nice, easy-to-read codes. 👍


Постарался также исправить ошибки в остальных уроках


FYI https://github.com/foxhound87/mobx-react-form/issues/154


Initial Queries


It would be good to find some official confirmation =|


chwila


Here you read in the file I send earlier today. So from here on, it is possible to create the plot with home ranges according to catch location


Ha j'avais l'idee que ca indentais tout de meme mais ok en effet ca pas besoin


chyba widzę


@andrewnisbet74 was this:
[https://rubygems.org/gems/jquery-turbolinks/versions/0.2.1]
the turbo-links jquery gem you were referring to?


DOH! bwahahaha


Ho nah I did not notice the clash


nie, jednak nie


a ideia do grid é bacana, mas para listas realmente longas isso vai gerar um scroll horizontal enorme. No caso do instalador do GoboLinux nós temos um checklist da lista de pacotes, com mais de 100 entradas. Ao invés de colocar as coisas pro lado, o melhor é ter uma tabela com scroll bar.
![screenshot_2016-11-23_17-18-29](https://cloud.githubusercontent.com/assets/245621/20575648/64f1604a-b1a1-11e6-9ed7-e3de32df3ef2.png)


Did you check Mabinogiworld Wiki? It even says Sen Meg Plateau was was introduced in G2 under map updates: http://wiki.mabinogiworld.com/view/Patches#Generation_2_-_Paladin


There's an existing in game constant - MAX_CREEP_SIZE


$i=1;
Do
{
$Text = Get-Content -Path C:\Users\Guest\Documents\passwords.txt
Start-Process powershell -Verb runAs
echo $Text[$i]| clip
Read-Host $Text[$i] |Out-Null
$i
$i++
}
While ($i -le 168)
break
exit


is there any precise documentation for use? i dont know what 2captcha and data-sitekey means...


I tested serveral bots on different servers and talked to some users. Nobody did change it. In case anyone needs this function the admin now needs to enable it.


@xZeroMCPE lol 👍


Hi, yes we have run e.g. the test suite of the 'coala' project and another project. The gzip process simply did not finish to write the content. Unsure why that happened, I would need to investigate.

So any 'bigger' program we ran on the sprint just hang.


That's why I blocked it recently, but then two people told me they remember being there in G1, and it's not listed as a new area in the official news: http://mabinogi.nexon.net/News/All/151/0008v/generation-2-season-1-update-list


well done!!! 👍


**Code Quality - Critical**
Once you `return false` the function breaks out from the loop. So, I don't see the use case for your `else` construct. You want to avoid so much nesting with conditional blocks if necessary and I think you can do some refactor here.


Como fazer para eliminar essa variável de estado?
(é trivial)


Bravo !


![](https://github.trello.services/images/mini-trello-icon.png) [MAINT: Let sparks-design-system feed css into sparks-frontend](https://trello.com/c/oKYlnTTp/173-maint-let-sparks-design-system-feed-css-into-sparks-frontend)


O comportamento está dependento da taxa de FPS. O ideal seria a `update` dar resume passando `dt`.


Sounds good.


quando os campos têm chaves constantes strings, prefira a notação `screen.widgets` (`screen.title`, `page.content`, etc.)


That was simple, when I tried for the first time it was breaking everything. Surprised it now "just works".


The new autotools building doesn't work for me. I am trying to install binutils which needs to install flex. First flex broke and complained about gettext being too old. After I did a `spack install gettext` and `spack load gettext`, that error went away, but now I get the error:

==> './autogen.sh'
libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, 'build-aux'.
libtoolize: linking file 'build-aux/config.guess'
libtoolize: linking file 'build-aux/config.sub'
libtoolize: linking file 'build-aux/install-sh'
libtoolize: linking file 'build-aux/ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIRS, 'm4'.
libtoolize: linking file 'm4/libtool.m4'
libtoolize: linking file 'm4/ltoptions.m4'
libtoolize: linking file 'm4/ltsugar.m4'
libtoolize: linking file 'm4/ltversion.m4'
libtoolize: linking file 'm4/lt~obsolete.m4'
Copying file ABOUT-NLS
Copying file build-aux/config.rpath
Copying file m4/codeset.m4
Copying file m4/extern-inline.m4
Copying file m4/fcntl-o.m4
Copying file m4/gettext.m4
Copying file m4/glibc2.m4
Copying file m4/glibc21.m4
Copying file m4/iconv.m4
Copying file m4/intdiv0.m4
Copying file m4/intl.m4
Copying file m4/intldir.m4
Copying file m4/intlmacosx.m4
Copying file m4/intmax.m4
Copying file m4/inttypes-pri.m4
Copying file m4/inttypes_h.m4
Copying file m4/lcmessage.m4
Copying file m4/lib-ld.m4
Copying file m4/lib-link.m4
Copying file m4/lib-prefix.m4
Copying file m4/lock.m4
Copying file m4/longlong.m4
Copying file m4/nls.m4
Copying file m4/po.m4
Copying file m4/printf-posix.m4
Copying file m4/progtest.m4
Copying file m4/size_max.m4
Copying file m4/stdint_h.m4
Copying file m4/threadlib.m4
Copying file m4/uintmax_t.m4
Copying file m4/visibility.m4
Copying file m4/wchar_t.m4
Copying file m4/wint_t.m4
Copying file m4/xsize.m4
Copying file po/Makefile.in.in
Copying file po/Makevars.template
Copying file po/Rules-quot
Copying file po/boldquot.sed
Copying file po/en@boldquot.header
Copying file po/en@quot.header
Copying file po/insert-header.sin
Copying file po/quot.sed
Copying file po/remove-potcdate.sin
libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, 'build-aux'.
libtoolize: copying file 'build-aux/ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIRS, 'm4'.
libtoolize: copying file 'm4/libtool.m4'
libtoolize: copying file 'm4/ltoptions.m4'
libtoolize: copying file 'm4/ltsugar.m4'
libtoolize: copying file 'm4/ltversion.m4'
libtoolize: copying file 'm4/lt~obsolete.m4'
configure.ac:31: option `dist-lzip' not recognized
autom4te: closing standard output: Broken pipe
autoreconf: automake failed with exit status: 1


aaaa


**Nitpick - Code quality**
Whereas `forEach` looks terse and doesn't have issues with smaller array size, I'd suggest you avoid it because of performance issues. It's terrible when your  data size increases and worst still, it constitutes a nuisance with some IE browsers. See [here](https://coderwall.com/p/kvzbpa/don-t-use-array-foreach-use-for-instead)


Does this mean it should be called only message id?


Well, I can't really say anything that would contradict the two people who claimed it was available other than be a counter to to those anecdotes. But for the official update list from Nexon missing it, I think that's just their incompetence.


Uploading my thesis to Grappa failed because of the removed single quotation mark here. The error I received was:

`{"message":"Command failed: pdftk /var/www/grappa.cs.helsinki.fi/grappa-backend/tmp/1479928713380/thesis.pdf dump_data | grep NumberOfPages | awk '{print $2}\n/bin/sh: 1: Syntax error: Unterminated quoted string\n"}`


What about `if(entry.hasRaw()) return this.emit('done'....`?


@mbluj FYI


`isPaper` is a Java variable name. `paper` would be Ruby-like.


Oh, I didn't check the pull #112 - so it's being handled already. :)


do you need these ?


To be fair, the wiki isn't all that reliable either at times.


Sounds good, but better document this somewhere.

Actually, would be nice to start a readme file for plugins.


Hi,

Thanks for addressing this issue, but I still don't see the custom HTML.  While there is still an excellent chance I am doing something wrong, I have spent several hours over several days trying to eliminate that possibility before getting back to you on this.  Currently, I have started the installation procedure over from scratch with a brand new clone of the repository.  I copy in my exsiting custom views and edit the proxy in gulp/config.js.
I run the gulp run with view argument. I can see all my hits in the command window with claims of what it is serving, but the HTML is the same stuff as the live site rather than as the attached.

I tried debugging the behavior but quickly ran into my own limitations.  I noticed in the sources panel under my custom view, the css, img and js show up, but the HTML folder does not.  Not sure how meaningful that is, but seemed suspicious that it is the only one missing.

[home_en_US.zip](https://github.com/ExLibrisGroup/primo-explore-devenv/files/609729/home_en_US.zip)


**Nitpick - Code quality**
Remember that indexArray inherits from its prototype `Object` and doing a sanity check that  `indexArry` hasOwnProperty of `element` might come in handy here.


The original intent here is for programs to be able to read the entire contents of a params file and parse it as a JSON object. The way you're writing multiple params iterations to a single file makes this no longer work. There are a couple of options to make this work again:

1.  Each time you write, write to a separate file. The drawback here is that there could be an explosion of files.
1.  Write the params to a single file in a JSON array (i.e. start the file with '[' and close it with ']'). The drawback here is that the file will be malformed JSON if the process crashes (since it'll be missing the trailing ']').

I would also add an option for the user to specify whether they want the entire history or just the most recent params (defaulting to just the most recent).


oh that's.... very true... Will make the change.


wait, weird. the ph5API_object shouldn't have been removed. Revert back to one commit back and I'll fix that and fold in the fixes for wildcards. It looks like something happened with the local version on the server here at PASSCAL.


I don't understand this.


Someone speak Chinese, who could tell me if this mentions whether Sen Mag was new, and not only Fleta?
http://luoqi.tiancity.com/homepage/zhuanti/g2/s3more/s3more.html


FixDaylightingTableZone (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2346 of 2346 tests passed, 573 test warnings)

 * 573 tests had: AUD diffs.

![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)


Where is that southern seal stone that was supposed to be blocking Emain Macha anyways? Is it In Sen Meg, or is it in Giarech?


There is a syntax error here: the last item should not have a comma...


Sen Mag, the path from Gairech to Sen Mag is not narrow enough for a Seal Stone, that's why I had to use 3 =<


nice


Can you add to docstring convention of how dangling in_ports and out_ports are handled


**Nitpick - Code quality**
`EOF`. Please use `editorConfig` to automatically format your code and add EOF.
See [here](http://editorconfig.org/)


FixDaylightingTableZone (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2380 of 2380 tests passed, 574 test warnings)

 * 574 tests had: AUD diffs.

![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)


Ну вообще была такая мысль, я думал использовать [autobahn](https://github.com/crossbario/autobahn-python) в качестве реализации вебсокетов, и в принципе там есть реализация WSS (несмотря на то, что в примерах есть только реализация для **twisted**, уверен, что для **asyncio** это запустить тоже реально).
Но тут хотелось бы услышать ваше мнение @zhebrak @FZambia: стоит ли вообще возиться с WSS, если нужно защитить только данные авторизации? Ведь по сути основной объем передаваемых данных - это передача текущего состояния игры от сервера к игроку (это происходит каждый тик игры, то есть около 30-60 раз в секунду - это еще предстоит протестировать), а также передача событий игрока обратно на сервер, и их в принципе можно передавать в открытом виде. Тогда можно использовать обычный протокол ws, а защитить данные авторизации можно путем обычного асимметричного шифрования.


No biggie. So if support the station, location, channel, and array wildcards for the PH5 experiment that is passed to the constructor, then I will expand the network wildcard in my code and make only one request per network. Does this sound right to you?


The following doc(s) have been deployed to the [terminusof](https://terminusof-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/terminus/examples](https://terminusof-static-docs.pantheonsite.io/docs/terminus/examples)
- [source/_partials/footer_scripts.html](https://github.com/pantheon-systems/documentation/commit/08dfc80c7a9ba9773d25b867a499a5b0cb4963cf/source/_partials/footer_scripts.html)
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/08dfc80c7a9ba9773d25b867a499a5b0cb4963cf/source/docs/assets/css/main.css)
- [source/docs/assets/scripts/reset-dev-to-live.sh](https://github.com/pantheon-systems/documentation/commit/08dfc80c7a9ba9773d25b867a499a5b0cb4963cf/source/docs/assets/scripts/reset-dev-to-live.sh)
- [source/docs/assets/scripts/reset-dev-to-live.txt](https://github.com/pantheon-systems/documentation/commit/a0a8647349f622e8ede89a87d8c3923820e3425a/source/docs/assets/scripts/reset-dev-to-live.txt)


lprint isnt defined by that point, right? and It exits early if gitpython isnt found so lprint doesnt really matter anyway?


торопился однако :(


yes, that is correct. so the vault does have it's methods exposed correctly.
the issue now is the `getValue(key)` code.
figure out why you are getting `null` instead of the actual value that i wanted to set 🍻


Good to know


Rubric comments:
Criteria - Get ip address from user (v4 and v6) in cidr notation
Teacher comment - "Fails when the host bits are set."
Points - 6 / 9.0 pts


**Code quality - Critical**
Please add @params option to your comment sections to specify the signature of your functions


noice!


А как мне реализовать проверку именно в __init__ ?


FixDaylightingTableZone (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@jmjiang You should probably create an issue, rather than leave this as a comment on the license file.

It would be interesting to hear why this approach was taken.  I do understand that it fits into a 2-categorical setting much more easily.  The classical approach from CT really assume a lot of strictness (and possibly more) which may not be as suitable in a constructive setting parametrized by ≡.  But that's just my best guess.


comentei no código da Anny


Think this needs to be e.rule.name === rule. The rule property is an object while the rule parameter in the firstOf function is a string


This "fixes" issue #1 by removing the feature :)


Pojebało cie?


added a link to SV grammar


Looking through the NA news I noticed that Fleta was added in G2S3 and Peaca in G2S4. I guess someone could run around Sen Mag for a while until the stone was broken and those two features were added. @Cosrnos?


5925---VRF-terminal-unit-max-iterations-exceeded (rraustad) - x86_64-MacOS-10.9-clang: OK (2340 of 2340 tests passed, 0 test warnings)

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


@phillipadsmith stay tuned! ❤


@nikki93 This update requires some native adjustments as well, see this [issue](https://github.com/exponentjs/exponent/issues/16).


I change it to a much more reliable in 90662219f66eef9690acab6717eb0a7c2fe18db0 which also fits on one line.

I'll make note of that, though.

Also, let it be of note that any # FOFF/FON comments are what i have setup to tell pycharm "Dont mess up my formatting pls"

Very handy things to use. Should probably be rebased out during the final steps, since they wouldent belong on dev


**Nitpick**
You wouldn't need an extra `else` block here since a truthy `if` or `else if` expression will return from the function. Just a nitpick tho' except it violates the specs. of sims


I think it may be good to have 2 files: log and checkpoint, so that checkpoint will have valid JSON in a case of a crash and log will be used for history. Each can have its own flag.
Another alternative, though I have to check if it's possible, is maybe to write every time the last update of params with the "]" in the end and then when appending new params, truncate first the last "]", append the params, and add it again. Then the JSON array will be valid even in a case of crash.


Get a better exception here


Just took a quick glance of what was dependent... Do we really need `buffer-to-arraybuffer` I found this http://stackoverflow.com/a/19544002/1008999


TODO: pass install dependencies via `install_requires` rather than requirements.txt file


Would be nice if there was some meta hints that could solve this easily such as the textures or objects in that area being labeled with a g2 prefix. Well, I guess if you want to be on the side of caution you can have Sen Mag open, but without Fleta available because her existence definitely altered the bear's drop list post launch. It is truly unusual that you have to put the boulders in yourself rather than the game using the seal stone in the right place. Though at least twice there has been situation where a map had forest turn into opening.


Тут более общее условие: нельзя добавить станцию в маршрут, если она уже есть. Не важно на каком месте.


I can confirm that this does allow online use.


Логичнее добавлять в конец станции, но т.к. последняя у нас фиксированная, то надо добавлять предпоследнюю


И используй [] вместо fetch.


Good to hear. We are using the netsight version in production at the moment - intensively. We are interested in a mac version and are happy to test and help where we are able, but don't have any mac packaging know how.

I was in contact with Thierry from atReal and he wrote this:
```
Hi Alex,

I built the last version of ZopeEdit for OSX. The issue is that the library that I used for packaging python app for OSX isn't maintained anymore and is not usable on latest versions of the OS. As we don't develop for OSX we didn't find an alternative but I'm confident that it does exist.
In order to pack zopeedit for OSX you need :
- to make a package from the python app ZopeEdit
- insure that the path where the options are saved is still accurate
- insure that the editor can be launched with : from LaunchServices import LSOpenFSRef
- find the way to create a python installable application with python included
- when the application is installed, associate it the files with the extension .zem and type application/x-zope-edit

I hope that helps. As we don't have any osx developper it is very difficult and unefficient for us to make it. I'm still available for you if you have questions.

Best regards,
Thierry
```

And last not least we found that you need to downpin p.a.contenttypes to 1.2.10 because this change to the webdav support returns all content encoded. https://github.com/plone/plone.app.contenttypes/commit/1b6c75dae22cee9ad13b99fb013cd02189ff2d9a#commitcomment-17982379


Done, will be in commit with random arrow generator


Если у тебя имя передается в качестве параметра, то надо это отразить в названии аргумента, написав name или station_name, т.к. сейчас непонятно что надо передавать - объект или имя.


No space after opening brace. - Check JS Style Guide or try to use some code quality tool!


I am not too experimented on JS, but I am thinking on using of "===" instead of "==". Or let me make it clear why == is enough :)


brutal


Ну тут еще один вариант может быть, он остановлен, но вагонов у поезда нет. Оба варианта попадут в else


Favor explicar no código qual linha de código **cria** a closure e quais upvalues são **capturados** na criação.


Лучше хранить именно индекс станции в маршруте, и при необходимости по идексу получать сам объект из маршрута


Se ha hecho un primer acercamiento al proyecto juego web para la practica solicitada


don't know what this is


It's not there, because you haven't added it.


То есть ты объявляешь методы в другом методе, да еще и в бесконечном цикле? Зачем?


> However, this appears to run roughly 4 times slower than the suggested toArrayBuffer function on a buffer with 1024 elements.

We still support 0.12, which does not use TypedArray-backed Buffers.


I'm checking for `GaxError` now.

Would it be more appropriate to raise `GatewayTimeout`? I think that could be weird with the `return_immediately` logic though.


Having this here is stupid but I didn't find a good way yet. I'm sure with the new DAGNode API and giving some thought to this, we can get rid of the need to pass the request function.


Зачем тут столько пустых строк? Достаточно одной


Опять лишние строки


Você trocou 6 por 1/2 dúzia.
Quero eliminar esses `if`. Do jeito que está, não é muito diferente de uma callback. O corpo do `while` executa por completo a cada `resume`. Nesse caso, qual a vantagem de usar uma co-rotina??


**Code quality - Critical**
Technically, `arrays` are objects but you'd want to specify the exact of your values. Your function signature from your comment is a `object` but `result` is an array. Can you reconcile?


@vishalmamidi it's installed when you install vscode using the rpm/deb packages. It's not shipped with the tarball (if that's what you meant).


Можно избавиться от вложенного if, объеденив условия через логическое "и"


In here, you could check if the username exists, and if the project id exists for this user, and if not, you could render a 404 page (being worked on in #198)


this doesn't make any verification about the stride. Numpy does some. Do we want to add that?


new & disabled


and Revlink added


thanks, this is my mess...
I fixed this on this commit:
https://github.com/AndroPlus-org/android_kernel_sony_msm8996/commit/61c014a96e9d02dd21f4b1f8f717bbec6264bb21


you want the package to be the same as the name of the folder it is in, so in just the root src directory there is no package.


@mat_e  all the translators has been removed from the header of PO file except me.


same as above


Does it apply to code comments too?


HybridVentilationEnhancement (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2379 of 2379 tests passed, 0 test warnings)

![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)


nice improvement


@Jay-Song Why did you change "100644" value to "100755"?


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1770 of 1770 tests passed, 0 test warnings)

![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)


No worries !
I've just didn't saw your v2 yet ...
Thanks !


u_f.h was not in drivers/usb/gadget/function, so I changed the path.
Maybe we can move u_f.h to drivers/usb/gadget/function, but it will break something.


HybridVentilationEnhancement (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Sorry but I cannot understand what you mean. Mauvais Anglais.


Not sure I love this--since these are temporary classes intended to be merged into nipype in the near future, I think we should keep them as thin as possible.


why ocf_ssl? did you actually create certs for this? (i think not, since puppet is erroring)


(i'm guessing this was copypasta from one of the few staff VMs that are special snowflakes and have ssl)


春运交通40天


@Tyxiaomi 不要在这里评论，大家都能看到的。看一个 github 项目，首先要把 readme好好读一读。jjc 给出了如何加到群里的指引。


Would it be simpler/better to do what the other fields do and get/set the
value from the item?  The default would only be changed when using a menu
editor and saving and fully loading the menus.  I did find some seting of
visibility in the MenuManager but not sure I fully understand if I would
need to mimic that as well.

Thoughts?

I also had missed updating the serialize class so I will update that.

On Tue, Nov 22, 2016 at 7:04 PM, Sean Stuckless <notifications@github.com>
wrote:

> If isVisible is setup that way, then likley it's right :) Not all menu
> properties are stores in the UserRecord, but, because visibility can be
> easily changed by the user, it is stored in the UserRecord and applied
> during the loading. You might need to look in the Menu Builder or GetMenu
> code and see where it's applying isVisible from UserRecord, and then do the
> same for the isDefault
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/stuckless/sagetv-phoenix-core/commit/3a8a47195cb4e0769a99eee8be180c5ed39ba718#commitcomment-19931122>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AGT63L3arcX-tHKZ79kaZIhNd19sGzveks5rA5EkgaJpZM4K56ir>
> .
>


That was a mistake, I just copied the config from a different staff VM. But I did create and install the SSL cert, and puppet is not erroring looking for a cert.


Site hospedado?


Puppet is actually not erroring at all right now.


Issue #18 still exists,
![image](https://cloud.githubusercontent.com/assets/16886014/20549296/ee5b2086-b0f8-11e6-9bf8-5a3a6dc142b9.png)


oh, i didn't actually check the last timestamp, just saw 20 emails from puppet


You are right though, I did just copy another config and think about the fact I had to create a cert for that. My cert manager password had expired and everything.


array map 直接返回就是一个数组

```js
const items = texts.map((text,index)=>{
    return(
        <Text key={index} id={index+1} onEditItem={onEditItem}  {...text} />
    );
});
```


Why is it set to false? Shouldn't mask generation be turned to true if the
report generation is requested?

On Nov 22, 2016 6:07 PM, "Shoshana Berleant" <notifications@github.com>
wrote:

> Not sure I love this--since these are temporary classes intended to be
> merged into nipype in the near future, I think we should keep them as thin
> as possible.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/poldracklab/niworkflows/commit/b027f731022afc3f0377134a3661dd55a19def77#commitcomment-19931574>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAOkp758CJUXYE2ABgJ3tuMJTv5pkniUks5rA5_XgaJpZM4K6Jxw>
> .
>


Text 这个组件的命名 ， 改为 Row, 或者TableRow 之类的更合理


@Sija What do you mean? I fixed it for code comments. Does it happen somewhere else?


Since you never need a reference to this object, you can just move it to a `before` block.


site hospedado?


qishiwohrnzaihu


Since you never reference this object, you can just make it a before block.


Why did you add this to the test scenario?


why are we creating these?


Will be tested once I figure out why vita-elf-create is segfaulting.


另外，这里的 <td id={id}> 和  e.target.id 这个真的是多余的操作，没有从传统的Dom操作转变过来， component 自己本身就带有props。  给组件声明一个 props 是 id ，在点击调用的时候 `onEditItem(this.props.id);` 就可以了。


![](https://github.trello.services/images/mini-trello-icon.png) [A* Pathfinding](https://trello.com/c/Hh6l6Oft/10-a-pathfinding)


texts 这个命名需要改一下，你可以叫 item 或者 user


Hey thanks


I've observed it both, in type restrictions (`def method(argument : ::Type)`) and comments (`# Comment mentioning ::Type`)


Site hospedado?


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1773 of 1773 tests passed, 0 test warnings)

![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)


d


HybridVentilationEnhancement (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1197 of 1197 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-36.14%25-green.svg)


start first comment.


We never say `typename` elsewhere. Is this different because it's code rather than a synopsis or declaration?


Hah. Your commit got picked up by a Twitter bot. Well done :D

https://twitter.com/callouscommits/status/801243412288643072


urg point


Thanks, helpful to know the list. I still think this is a good move for this module since it allows it to be focused on generating UUIDs and not deal with any parsing related issues which naturally lead to folks wanting a "validation" api.


Also happy to add a section to the README outlining other modules to use to perform this function or basic code that will do it for those that needed it.


hi


I am a postgrad working in petrochemical QC. I am working to gain a bit of knowledge about data science.


fttfct


With this change, what is the recommended way to display components that display absolutely positioned elements that overflow the borders of the preview container (something like a Tooltip or a Dropdown)?


Site hospedado?

Faltou fazer o pull request de cada commit.


Works for me too.


This would allow scrolling in those cases, wouldn't it?

Either way, I'd probably wrap my component in something that gave it more space so that the tooltip would not extend beyond the boundaries of the example.


When the hell did the game update!? Why the hell did they have to change offsets too damn it? Oh well... Just a minor setback I guess... Btw, I will also start assisting with development if you don't mind


Should be hashing password!


There is still a corner case, when updating `xen-hypervisor` package on non-EFI system. Then `xen.efi` lands directly in `/boot/efi`. `[ -n "${EFI_DIR}" ]` check was correct...
But at least the update works as expected when it should.


Und, ich du Deutsch? Ich spreche ein wenig.


这个命名也很不好 _state 这里的_ 代表是私有的，但是 通过`mapStateToProps`这个名字就可以知道它是把state转成component的 props， 既然是 props怎么能是私有的呢？ 而且_state这个命名还模糊了， 你首先需要给组件先设计几个属性，你这个组件名字叫 AddItem ，首先名字就定了他是做添加一个项目的，但是你又用来做编辑 有点不合理，你可以叫 UserForm, 或者叫ItemForm 代表这个一个表单组件， 然后它有个props : data 用于填充表单的数据，而不是 _state


@chrisfilo that's a mistake. I wish I had seen your comment 27 minutes ago


I think the semicolon should be inside the quote here


@NoahKunin I changed these headers to all `###` specifically so the policy could be easily copy/pasted into the 18f.gsa.gov repo, into a part of [this file](https://github.com/18F/18f.gsa.gov/blob/master/pages/vulnerability-disclosure-policy.md) below the header. Since that is the canonical location for _display_, I want to privilege the header level that that location needs.


So Mario wanted to time of the computers to do a network graph. My argument is not all platforms will have synced time. So I would go with the original idea and only send a ping if haven't received a message is x amount of time. Just use your local timer. Number 2 seems like the right approach.


Reverted in 97600ade825987eec79e87c3d0d7dc083a6bee20.


sfdsfsdfds


Com esse commit aumentando os packet tao alto assim...
Sabe que é possivel crashar o ot né?
Com WPE...
Por que aumentou ?


海贼王登陆


Thanks :)


and mention IT initiatives


Note: This compiles on Linux 64 bit but doesn't actually work yet!

Seg Faults with the Generic Parser code coming from QVM in UI.
G2 Dedicated server files are likely out of date and need syncing with some of the changes to the vanilla G2 files.

I also believe we may have to switch to ghoul handles like jk2mv but for all OS' not just !id386


Site hospedado?


Bug:but only after calling both ;)


Fixed URL to correct format


Well yes @xfurry  and @killerwife already told me about this problem.
The real issue is that hook was not in the good place before i fixed AIM_Initialize. So after the patch i made wich is normaly not related the core crashed at that hook because it was caled before that AIM_Init. So to be sure it will be called after AI init AND in respawn case i found this place conveniant (before it was called by AIM_Initialize)
So long story short yes it need a better place. But this hook will need some research because it will probably need to be set in different place.
That research have to be done in perfect world by someone who use script and know implication for other scripts. That's why i let time to see what is realy broken and if a @xfurry would investigate it.
So long story short i can work on it fix this issue but i cant guarantite i will not create new :/


I don't think the re-wrapping is necessary


А это зачем?


In `getProject`, you should check if the username exists/a project with that id exists for that username, and if not, return a 404 error (e.g. `res.status(404).json({message: "Username does not exist, project with that is does not exist"})`)


I was wondering, thanks  - yes not on dev


Here is what was missing ;-)
https://github.com/igorpecovnik/lib/commit/8a036a860096fb5338a7263de1f28344ee7c2d52


Dmitriy Lishko added


Actually I'm not sure whether the error was before the patch or after. A colleague reported that issue. I missed the fact that only a function is returned. So you are probably right. I'll let you know if there is still an issue.


Почти верно, только надо не кол-во считать, а выводить поезда заданного типа


You like empty newlines?


Sorry, been very ill...
NO... Unfortunately not, but I really do appreciate it...
But if you'd like to help me out some more, the easiest way to get ahold of me is via Messenger. Just search Facebook for Scorpio2177! 😁😉
Basically(& I'm not sure if I'm allowed to swear on here, but...)
My SHIT AIN'T SYNCIN'!!! LOL!! 😁... But not really, haha... 🙁


Чтобы не использовать elsif, можно объеденить условия в первом if через логическое  "и"


Принято так записывать условия, чтобы внутри if был "оптимистический" сценарий (прицепление вагона), а в else - "пессиместический"


bah... is it only me who want to ditch 0.12 so badly 😠


Тут тоже надо поменять содержимое if и else местами, ну и условие для этого изменить, само собой


"this" is null in a function not used as callback, replace by "EnemyBuffTimers" in those and stop caring.
"CastSpellByName" takes up to 2 arguments in case of a cast on self ("onSelf" ==1).
"EnemyBuffTimers:CHAT_MSG_SPELL_PERIODIC_CREATURE_BUFFS" has a duplicate.
"guids" and "abilities" should be global.


You have class "Segment". Why than method "isSegmentsHaveIntersection" is located in the different class?
* Locate it in the Segment class and let it get only one argument.
* Rename the method so that when it's called it looks like segment.hasIntersection(that) or segment.hasIntersectionWith(that).


I would like to write it in three lines, so it's too long for me.


Проще всего хранить индекс текущей станции у поезда. А при необхоимости по нему получать саму станцию из маршрута. И тогда перемещение поезда между станциями, это всего лишь имзенение индекса на 1 или -1


если не ошибаюсь лучше в данном случае писать ++sizeclass, чтобы в памяти не создавалась временная копия этой переменной перед тем как она увеличиться.. если я не права, исправьте меня пожалуйста:)


Wow, that's awesome! Glad you got it working all by yourself. I'll add in the Cygwin part for Windows users, thanks for your suggestion!

I'm working on finishing a new program called Monitor, which you can boot into and manipulate, view and jump to memory. Stayed tuned, if you're interested ^^


Hmm, this is unsatisfactory because it loses the name of the binding argument.
But I think I will punt that to another bug because this is fixing a crash.


Hi,

No worries,

Just send me your zipped package+the PROXY_SERVER definition from config.js
and I'll take a look

On Wed, 23 Nov 2016 at 21:36 quist00 <notifications@github.com> wrote:

> Hi,
>
> Thanks for addressing this issue, but I still don't see the custom HTML.
> While there is still an excellent chance I am doing something wrong, I have
> spent several hours over several days trying to eliminate that possibility
> before getting back to you on this. Currently, I have started the
> installation procedure over from scratch with a brand new clone of the
> repository. I copy in my exsiting custom views and edit the proxy in
> gulp/config.js.
> I run the gulp run with view argument. I can see all my hits in the
> command window with claims of what it is serving, but the HTML is the same
> stuff as the live site rather than as the attached.
>
> I tried debugging the behavior but quickly ran into my own limitations. I
> noticed in the sources panel under my custom view, the css, img and js show
> up, but the HTML folder does not. Not sure how meaningful that is, but
> seemed suspicious that it is the only one missing.
>
> home_en_US.zip
> <https://github.com/ExLibrisGroup/primo-explore-devenv/files/609729/home_en_US.zip>
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ExLibrisGroup/primo-explore-devenv/commit/13e478a5c487973a1297fbf0e70f544c41781e66#commitcomment-19943146>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHqcn57nU58Jvot56I3zLppZzECI4ysNks5rBJXSgaJpZM4K6-ST>
> .
>


Though the regression has been introduced due to this PR, it's actually an issue with `m_regs->DISP[i].DISPFB.DBY` reporting 2047 as the top value(this is usually 0) when getting the frame rectangle: https://github.com/PCSX2/pcsx2/blob/master/plugins/GSdx/GSState.cpp#L442

I can easily fix it locally by not allowing these extreme values, but it should be looked into further.

@ssakash Probably something you'd want to take a closer look at.


Кстати, у маршрута нет метода stations, к которому ты здесь обращаешься. Идея правильная, но надо дописать класс Route, чтобы такой метод заработал.


I [just ditched 0.10](https://github.com/bitinn/node-fetch/commit/25d139a4d0a4ea1d703abbacec8fb31589a9086e), so relax :joy:


Did you read the comments too?

> Late addition: @trevnorris says "starting in [V8] 4.3 Buffers are backed by Uint8Array", so possibly this is faster now...

> Tested it with v5.6.0 and it was the fastest


Oh, I think this is issue #263 anyway!


No but dymola does and i did not run the clean up script yet. So shut up
;-)

23. nov. 2016 9.03 p.m. skrev "tbeu" <notifications@github.com>:

> You like empty newlines?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/dietmarw/FirstBookExamples/commit/81065c0df831f07d839142e838fec7f462523d73#commitcomment-19943522>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAAkdEN0KMv-zLgxHMHj63Yrb6baB0pqks5rBJvygaJpZM4K6_v2>
> .
>


el atributo quéCosa, loadMap(position, true), true es para mostrar un icono en la ubicación encontrada en google maps.
if (useMarker) { // es el parámetro que recibe true o false
        new google.maps.Marker({
          position: latLng,
          map: map
        });
}


И метод index_next_station нет. От того, что ты вызываешь метод у объекта, он там не появляется. Или руби должен сам "догадаться", как такой метод должен работать? Нет, мы должны четко "сказать" (написать в классе), что наш класс имеет такие-то методы и эти методы работают так-то


Кстати, такой метод тебе не нужен. Достаточно уметь получать список станций в маршруте и управлять индексом текущей станции у поезда. Этого достаточно, чтобы получить нужную станцию.


OK! I'll definitely try that.


И метода last_station у тебя нет.


Yeah that was what I was referring to. If you read the module carefully, you'll see that it has two branches: a fast branch for when `Buffer extends Uint8Array` and a slow branch for support for older V8/Node.js versions.


Any ideas why?

(I'm here to help!)


Extraneous `end`.


I don't think momentum is compatible with this technique, you should probably assert that momentum is zero.


More specific commit messages please: include what you actually did. In this case, took gear ratios into account


И next_station тоже нет в маршруте. Ты посмотри на свой класс Route. Вот что там написано, ровно это и есть, больше нет ничего. И когда я смотрю на него, то не вижу там всех этих чудесных методов, которые ты вызываешь.

Вообще, это не так страшно. Это даже один из способов проектирования кода, когда ты используешь объект и его методы еще до их реализации. Так ты понимаешь, что тебе вообще от объекта будет нужно и какие там должны быть методы. И возникающие ошибки как раз указывают на то, что еще нужно реализовать. Просто надо сделать следующий шаг - написать реализацию нужных методов в нужном классе. На этой идее основан подход Test Driven Development, который мы будем проходить в Продвинутой части.


Is there any specific reason for all the `if x is not None` checks? Isnt that the same as `if x`?

So we want to use bass as a central place for wx imports, and everything else go from there?

Looking at that code, it would fail to import if wx actually werent there since wxversion wouldent be.

Judging from the [docs](https://wxpython.org/docs/api/wxversion-module.html#select) it could just try and select and then catch VersionError and abort. Of course, it will also fail horribly in standalone since the [wxversion docs](https://wxpython.org/docs/api/wxversion-module.html) say it will fail in a bundled enviroment

I'll figure something out.

How about just have bash.main set bass.wx to the correct version, everything else imports wx from bass.

Though then dump_enviroment will need to be altered?.. Since we dont want to abort on a missing wx problem before we can print atleast that to the debug log, and it imports wx itself. Does wxpython Phoenix use different imports or? [wxversion doesnt exist on Phoenix, though.](https://wxpython.org/Phoenix/docs/html/MigrationGuide.html)


Implemented grab and drop the pizza


It'd be really good if you put the class to src.main.java folder rather than having it in src.test.java folder.


Take care of tabs


Prepare Release v1.1.0 (#226)

Preview: docs
Coverage: core | datetime


See #2398. I'm currently working on resolving this problem. Hope to have a fix out by the end of the day.


Je bricole un truc et vous montre ça. Pour le moment, ne changez rien, c'est pas très grave.


ohhh .... i didn't know :P οχι νταξει δεν λεω! δεν ξερω απο css οποτε good luck with that.


FixDaylightingTableZone (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1198 of 1198 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-36.15%25-green.svg)


Зачем тут concat и еще и новый массив. Нам не надо добавлять кол-во вагонов, нам надо сохранять в массиве сами объекты вагонов.


@cyberium sorry, I didn't have time to investigate the escorts, in-game yet. Will do it soonish.


Good you've thought of that.


Передавай не кол-во, а сам объект вагона, который нужно удалить


Mhm, at least four times. To Cobh, to Abb Neagh, and in Dugald and Emain to the regions which's names I can't remember.

>It is truly unusual that you have to put the boulders in yourself

That's the thing, either the path to Sen Mag didn't exist in G1 and was added in G2, or it did exist, which would mean Sen Mag existed as well.


Используй метод delete для удаления. И поменьше обращай внимание на кол-во вагонов, а больше - на сами объекты. Например, вагон можно удалить только, если он есть в составе поезда. Это можно проверить при помощи метода `include?`


Please note, that I changed the names of some quantities related to gen. matched lepton/tau four-momenta, that previously were the same as in the ValidGenTausProducer. I hope, that I changed every configuration correspondingly.


jai juste ajouter des montants :P


Думаю, в таком случае лучше взять просто какой-нибудь protobuf over ws, а аутентификационные данные шифровать отдельно. Хотя интересно было бы посмотреть на оверхед все равно.


I meant to set the title to "Applied lightweight status to **branch** "


> Is there any specific reason for all the if x is not None checks? Isnt that the same as if x?

Readability - read it if wx is set to None - None ? - Yes have a look where it's that defined - Ah I see, we failed to import

----

Re the rest - I told you this was a quick patch

> fail horribly in standalone

EEE standalone

Don't spend brain power in this - I am not even decided if this should go to bash.py (which should be renamed to main or something leads to many issues as has the same name as parent package)

Just to make you see what's in my local branches.

When I have a specification of how exactly needs be handled will let you know.

What we do need is:

- have a way to switch wx versions from cli so we test
- have wx imported once and yes warn appropraitelly
- cater for standalone which has wx packed in anyway

Btw this is working wx3 Bash - and chops off a significant amount of time from BAIN refresh cause the dialog is not native anymore !!


Нет, это вообще как броадкастинг работает.


I have looked a bit closer and its even worst because knights counts also. So if you have additionally two knights the bonus is given four times.
so following should be used

if(Pt == BISHOP && (ei.attackedBy[Us][KING] & s))


Hm, on which project did you run coala? Just tried it using their `.coalarc` on the mypy source code, works!


Outside of needing to clean up the command, why was this removed?


Ok thanks again. thought it would auto gen like other settings.


Hey @ssaunier quick question: you're mentioning here the `dotfiles` step, as if the students would already have completed it.
But they are going to read this part about the public GitHub email before the `dotfiles` step. I don't get it :(


So, since we're doing socket I/O, the right way to send a stream of n bytes is in a loop, calculating remaining bytes to send on each loop iteration. The loop terminates when n bytes were sent. Each loop iteration checks a few error conditions (by inspecting errno and the return val of the write) and returning failure only when error conditions occurred. The code currently returns failure if a partial write occurred (which is normal).


Thanks for this.   Would you consider
A.  Adding 180 to the list after "Default" in the comment
B.  Enable lockscreen rotation by default  (You used to be able to do that in the build prop--not sure how it is done in the default file.
C.  Move this also to the wip-knext branch  ?

Note:  Some apps (like the cm setup wizard, ha ha) insist on coming up with 0 degrees, set up wizard  will not rotate until you touch "next".   I've run into a couple of others--not a big deal.

thanks.


Thanks, I  just love compact code. =)


I like the idea of splitting up large methods to but I thinks we can improve on the name of the smaller methods.

Instead of having a naming convention `convert*`, e.g `convertMetaData`, `convertSheetPoints`, `convertSheetAdvantages` etcetera, I would prefer to name the results. I.e. `metaData`, `points`, `advantages` etcetera.

I prefer this for the following reasons

1. It is [declarative](https://en.wikipedia.org/wiki/Declarative_programming). It tells you what a thing is, in this case an `SheetSheet`, instead of a recipe for the thing.
2. It removes noise. The `convert*` or even the `convertSheet*` convention is increasing the signal-to-noise ratio.


This is not necessary


Zostawiam komentarz


Apple's changes are even more substantial this time around, then they were in the last releases.
In addition they're now using more blocks instead of methods in some places, which means that we can no longer simply hook into them, but have to completely re-implement some internal functionality of Mail, to place our code in between.
We've already rewritten a lot of parts of GPGMail and hope to have a first beta in the next few weeks.


tu też zostawiam komentarz


$override should be named something more descriptive.


Sorry for the wrong placement.

I see. It does seem more "robust" than the normal approach. Thank you for replying! Please delete or move it to an issue as you see fit.


same with the read, read until reply_size bytes were read from the socket. It is perfectly normal to get EINTR and a partial read, you just continue reading until you've read the full expected buffer. Failure should not be returned on partial reads. Failure is typically returned when the error conditions are anything other than EINTR  or EAGAIN (for non-blocking I/O).


Guys, this pretty much finishes up the drip creation feature. How is your bucket list feature coming along?


What kind of verification do you have in mind?


Agreed. I had thought of that also, but some devil whispered in my ear that a verb should be part of the method name. I will adjust them.


I am a bit puzzled, I don't know what was wrong with travis.
I get that you gain reliability by having a build external to your environment but other than that ..

I'm having a problem of my own. Switched to the hardware setup that I'd like to move in production, instead of the test setup I used until now. It's an Arduino Uno WiFi so one board with both the Arduino and the ESP.

The trouble is the two don't seem to connect. Sketches run, but the network functionality won't start. It just shows a continue stream of this :
�F*i�EL-Client sync failed!
�F*i�EL-Client sync failed!
�F*i�EL-Client sync failed!

Any clue ?


Hey this task should be on another branch. Reber this commt


Have a look at Bogus -> https://github.com/bchavez/Bogus. I have been using it to generate fake data for my unit tests. I have found it quite useful.


Those are good ideas!


Hey this task should be on another branch. Revert this commit


amazing script man


I did download your S3 build, and ran it. Same result as above though :-(


Always question devils.

And angels for that matter.


dyskusja


super dyskusja


gadam ze sobą


coś jest nie tak


to podyskutowaliśmy


Great! then I'll look more into implementing one of these and update the PR accordingly.


I just got told again that Sen Mag was *definitely* available, as they remember going there when Bangor's seal stone wasn't even broken yet.


Somehow the version I was working on(and was still open from yesterday) got reverted to a version almost a month old and the IDE didn't tell em the file change so this morning I was working on very old code. I grabbed the previous commit and then added in the fixes for array, station and channel wildcards.
It should work now and be correct with the ph5APIobject in place.

Making one request per network makes sense to me. That should work.


This doesn't seem to be the task I've given.
What's required in java language is
1) to find if there are repeating numbers in int array. Like
Arrays.asList(2,5,3,6,3,6) should return exception with 3 because 3 is the first int that repeats.
Arrays.asList(2,3,4) should return message that all the numbers are unique.
2) find if there are segments that start/end with the same number. Segment(1,3) means there is a starting point 1 and ending point 3. All the segments are on the same line, it's 1 dimensional.
Arrays.asList((1,4), (5,7), (4,8)) should return exception with (1,4) as far as some other segment starts with 4.
Arrays.asList((1,4), (5,7)) - returns a no intersection message.
3) find segments that intersect
Arrays.asList((1,4), (2,5)) - returns exception with (1,4) because (2,5) starts before (1,4) is finished.


@xen0n @fire855 Well........ After a very well needed break from the porting :wink: I FINALLY
have a working cm13 for my device, based on fires patches and lightly based on the device and vendor tree :+1: GPS, FMRadio, AND the video camera ACTUALLY saves the files with the correct colors!
Thank you both sooooo much with all your help! I would never have been able to get this far without you.
Ive learned so much from you both, even down to just reading through your commits, and code...
I owe you guys a drink big time! Its just a shame the import/export would be so high! haha!

One last thing though, i did have a slight issue with no network service. i added in my rild files from my device to see if this would sort the problem, it did. is this a normal issue? as surely building ril is enough?


Och kappa, brONzowy... No nieźle


Typo - errorn


Just fo full reference to others:
We discussed in two places. The other part happened here and the issue is solved. I had a corrupted download or some other part of rolling release distro was updated. This discussion is fixed and closed.

https://github.com/graysky2/profile-sync-daemon/issues/188


preload saves 12 bytes but costs 8 cycles.


The built-in jsonb function, `jsonb_strip_nulls`, recursively strips nulls. This is undesirable in our cases. While it might nifty to support an option of stripping nulls at the base level, recursively, or not at all, this commit restores the behavior to how `slice(hstore, text[])` works.

Implementation of `slice(hstore, text[])` becomes irritating, because there is no similar method for jsonb.

pg doesn't allow for the following:
```
'{"a": "b", "c": "d"}'::jsonb - ['a', 'c']
```
It only allows subtracting one key at a time. For this reason, the statement is written as:
```
```
('{"a": "b", "c": "d"}'::jsonb - 'a') - 'c'
```
And so forth, for more keys.

@data-axle/developers


What is Dymola?


Doesn't this prevent 2 constructors from sharing the same name?  I don't think names are meant to serve as a global identifier.


@slevyrosetti have a look.


I would like this to dev - but please first test what happens if we failed to  import wx - test it over dev (that is rebase this on dev and then rename the python/site-lib/wx so it fails to import) - edit accordingly


Eyyy... How Download This, I Need Port It For My Huawei Y360 Mtk6582.


Not sure if this commit is the cause, but webif broken.
![capture](https://cloud.githubusercontent.com/assets/2105351/20577962/c354cd50-b1bc-11e6-8cdb-f6a98bdba0f3.PNG)


Ready for dev ?


@alexey-sidnev Лабораторная работа №2.


Add another bulky comment here:

// -------------------------------------------- //
// LISTENER
// -------------------------------------------- //

We use these to seperate sections and usually use these for Constants, Fields, Instance & Construct (or seperate instance | construct if necesarry), aswell as Listener or Override.


This is not what I meant by a helper.  https://github.com/mhevery/jasmine-node/blob/6c6099b6a33ae65b5b3c9a141e75ea797fa95a69/lib/jasmine-node/index.js#L41 or https://github.com/opentoken-io/opentoken/tree/master/spec/helper may help


Added final changes


ok noted, marking Ticket: #19087 as reviewed on redmine


Here, and elsewhere, you don't document the returned data type nor the type of data being passed in.


@sjoelund Looks like the release note thing went to the wrong file.


@lglopezf tenemos que deshacernos de ese atributo misterioso; que `#loadMap` solamente haga una cosa. Si tienes que agregar un marker entonces hacemos otra interfaz para eso en el servicio, puede ser un `#displayMarker`


Nope, I have discontinued my work on YOURLS and authmgr. Feel free to pick up where I left off.


And also should go to tables, but not internal


Nope! Does not work! Since operator way too dumb (me). Of course `userpatches/linux-pine64-dev.config` was still there. :(

Will report back in 5 minutes :)


Please look at other enums within MassiveCore or Factions. They follow a specific pattern and it will feel much more integrated into the code base if you follow it.

Rel is a really good place to look at:
https://github.com/MassiveCraft/Factions/blob/master/src/com/massivecraft/factions/Rel.java


The branch or this commit?


User const instead of let for variables which won't change in current scope


Are there any defaults? If so, add the using the MassiveMap varargs constructor:

`
public Map<String, String> entityHostilityDefaults = new Massivemap<>(
    "defaultKey", "defaultValue",
    ...... // As many pairs as you wish
);

Also, you are fine to use the actual enum as the value type. Gson will manage to serialize/deserialize that with ease.

`


Well, I know how to fix it without making new issues and without moving JustRespawned :) All we need to do is add "is temporary summon" to JustRespawned of EscortAI to prevent reseting after constructor init.


Всё же я настаиваю на том чтобы оставить только `@route`. Нам достаточно хранить список станций маршрута в переменной экземпляра `@route`. Если потребуется найти первую или последнюю станции мы можем использовать выражение `@route.first` и `@route.last`. Мы также можем определить для этого методы:

```ruby
def first_station
  @route.first
end
```


Ah, i see your point. The wx checking code isnt added in this commit, so it would fail in this commit without wx.

I could squash whichever commit adds it with this one, maybe? Or edit this one, depending on what else the commit that adds the wx check adds.


Sure, although I don't usually change their documentation comments, and the upstream CM default is still those three. I may amend the change if a related overlay or prop change can enable lockscreen rotation by default. Once there's confirmation that this overlay actually works, I'll propagate it to all M & N branches.


> Don't spend brain power in this - I am not even decided if this should go to bash.py (which should be renamed to main or something leads to many issues as has the same name as parent package)

Why does it have issues? Something to do with the WRONG WRONG WRONG relative imports of python 2? Couldent any problems like that be fixed through, say, importing \_\_future__.absolute_import (Though i suppose such a change is wildly out of scope for this atm)

> have a way to switch wx versions from cli so we test

you mean between wxpython phoenix and wxpython classic? Or between arbitrary wxpython classic versions? Because phoenix doesnt have wxversion so we cant just switch like that?(And phoenix has the same imports, too?)(I dont think you can install both classic and phoenix, anyway?)

> Btw this is working wx3 Bash - and chops off a significant amount of time from BAIN refresh cause the dialog is not native anymore !!

Is this saying BAIN refresh is faster or slower on wx3? Im confused. Native?


Don't use underscore variable naming. You have to use only one style of naming(camelcase)


> This is undesirable in our cases.

Can you give an example of why this is bad?  I don't immediately see why.


Looks good


komentarz pod commitem 1


komentarz pod commitem 2


Don't use assign syntax for adding new props to object, use spread syntax instead


@edolstra Yes, I just did.


Please break this line up. Cast to LivingEntity on it's own line (then you only have to cast it once, determine the hostility on it's own line, create the new event on it's own line .... you get what I mean. :)

We need to create tidy and readable code that takes one look to see what is going on.

I'd say do the same for line 51, but in case you actually use an enum in that map, it should be fine.


I left some inline comments regarding socket I/O.


This commit


Use destructing feature for variables which could be used in code below


Please get rid of that empty line.


This file has issues with indentation (2 vs 4)


Should be fine?


Yep!


On P19 LVDS connection, colors appear correct, but the display is dim and their are screen artifacts visible. The display is 24-bit, so it needs a different setup.


hola hello


This is great!  This way you are able to have less code and reuse the same function.  When you had to write them all out, it is more code and more chances for typos or mistakes.


Bulky comment with the word FIELDS please and put getters and setters right underneath the actual fields here.


Тут ещё такой момент: что мы храним в переменной экземпляра `@route`? **Объекты** станций или их названия? Похоже что объекты, но тогда что мы передаём в параметре `name`? Похоже что тоже **объект** станции. Но это значит что `name` на самом деле сама станция, а не только её название. Поэтому ещё лучше будет назвать этот параметр: `station`.


Where do you use i(var i = 0)?


То же самое, мы ведь передаём объект станции, не только её название?


As you saw editing the boot process is tricky so I'd rather we go in small steps. So I'd rather have this reworked to exit - or at your option include the wx checking code - but this means more testing and it's really not the time to break bash.py


troll install
troll start
troll run samcouch


You didn't use the correct styling for the code, please make sure to take another [look at this page](https://www.massivecraft.com/tech-standards#code-style) and see what is wrong for yourself.


this should be parametric


I have not attempted connecting an arduino to the esp.
My first instinct would be your pin setup is wrong?

I'm assuming that debug output is from serial? Or is the unit connecting to
Wi-Fi and such?

If it serial, try changing baud rate.. Esp uses an odd baud rate during
boot up(78600?)

As far as Travis build goes... For me, I like these positives
1) It means no matter where I am I don't need to setup a build enviroment
2) easier to encourage people to actually try the build for testing or
input.
3) rules out the build enviroment of being the cause of errors or problems
when collaborating
4) no need to checkout or clone the current remote commit.(likewise somehow
losing my own local changes)

On Wed, Nov 23, 2016, 2:30 PM dannybackx <notifications@github.com> wrote:

> I did download your S3 build, and ran it. Same result as above though :-(
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/fuzzball03/esp-link/commit/c59b2766facffb71f26cb71c269dd1f2d45976c5#commitcomment-19943839>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AFimNBlPZDHSu4_YuJ0cY82F0W7DXWBuks5rBKJhgaJpZM4K6YVU>
> .
>


no need to call `all` on `@owner.units` - this will automatically return all of the specific owner's units.


Hi hi


Нужно ли нам сохранять следующую станцию в переменной экземпляра `@index_next_station`? Я думаю нет, ведь этот же объект маршрута может быть у разных поездов. И один поезд может быть на одной станции, а другой - на другой. Это значит что значение этого (`index_next_station`) метода для них (поездов) будет разным и сохранять его в переменной экземпляра `@index_next_station` особого смысла нет.


this method makes more sense in your `Renter` model since it's doing a calculation on the data that a renter has


c'est dommage mais apparemment la conversion de l'objet en tableau php ne garde pas les tableaux vides, Du coup ça levait une exception parcequ'il n'y avait pas de champ notes dans l'objet track


this method makes more sense in your `Owner` model rather than in this ApplicationHelper


>  (Though i suppose such a change is wildly out of scope for this atm)

Yep - plus we are in mid refactoring so the relative imports are there for a reason

> Or between arbitrary wxpython classic versions?

This

> Because phoenix doesnt have wxversion so we cant just switch like that?

That's why it's tricky - we have to figure something out really

> Is this saying BAIN refresh is faster or slower on wx3? Im confused. Native?

Much faster - the dialogs are pythonland so avoids a gazillion of system calls


Dodaję komentarz, żebyś mógł przeprowadzić tu dyskusję (zadanie z działu code review)


I did something similar ;)
would be good to know, how we list them under each other.


nice find @mmoynihan .
Fortunately it _shouldnt_ cause any issues here since its just a key used to reference the field element. I'll make sure its sorted anyways for a future release. Let us know at support@octopus.com if you are hitting any issues thats preventing you from deploying.


Dunno ?


This is bizarre. Github demanded this merge, but there's nothing in it.


you guys probably don't want to make periodic upstream change merges that only consist of the tracking revision.txt and no other changes... :-\


Now that you're down to one assertion per value, I wonder if it would be more, or less, legible to relace each three-line case with something like...

```
number = UInt8(1)
XCTAssertEqual(Data(byte: number), number.ip_data)
```

This would

1. avoid having each value hard-coded twice (once in the `number` assignment, and again on the assertion line), and
2. simply inline the Data() initializer call into the assertion line.

Of course, maybe those are both bad ideas, or maybe only one of them is worth doing.  Any thoughts?


since we're calculating something that belongs to units, this probably makes more sense on your `Unit` model.

be mindful of where your putting your methods to handle the logic related to making calculations. if the calculation has to do with data that's in our database, we should put the method in the corresponding model.


Нужно ли нам сохранять следующую станцию в переменной экземпляра `@next_station`? Я думаю нет, ведь этот же объект маршрута может быть у разных поездов. И один поезд может быть на одной станции, а другой - на другой. Это значит что значение этого (`next_station`) метода для них (поездов) будет разным и сохранять его в переменной экземпляра `@next_station` особого смысла нет.


I think i'll rework it to keep the try..except but still remove the tkinter stuff, instead just exiting there.


Ignore* :laos:


notice how you're doing this same thing on line 6 of your `views/owners/index.html.erb` - this is a perfect example of something that could go in our helpers folder to DRY up our views 👍


se realiza modificacion de codigo


Improvement ideas:

- Check if copying the file is even necessary (is the file outside of `$HOME/workspace`?)
- Sync file changes immediately, not just on close


> В том то и дело, что когда станции нет в списке, данный блок возвращает false

Как так? если станции нет в маршруте `@route.include?(name_station)` вернёт `false`. Приведи пример, если это не так.


be careful, if you do it like this and you want to edit your samples from the electribe, it will not work properly.
This is why I used this workaround.
If it works properly for you let me know (maybe a bug has been fixed with update 2.02 of the electribe firmware?)


we want to avoid querying our database in our views. the `Owner.all.collect {}` piece here should live in your controller and you can pass it in as an instance variable


I understand those benefits to travis, but I don't know what's wrong with it for you. What are you trying to improve?
As to my issue : yes the output I show is from Serial.
I just changed my app so it only loops a couple of times, and the app does work.
Tried all the baud rates too, maybe I need to do that again.

`
avrdude done.  Thank you.

--- Miniterm on /dev/ttyACM0: 9600,8,N,1 ---
--- Quit: Ctrl+]  |  Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
Yow !

Arduino Uno Wifi Chicken Hatch
(c) 2016 by Danny Backx

Starting WiFi ...
��|�ELClient sync failed
��|�ELClient sync failed
��|�ELClient sync failed
��|�ELClient sync failed
��|�ELClient sync failed
��|�ELClient sync failed
��|�Initializing BMP180 ... ok
���Unable to sync with the RTC
Set up hatch ...
Hatch : use motor 3.
Sensor UP is on pin 2
Sensor DOWN is on pin 3
No Button UP
No Button DOWN
Light sensor is on pin 17
��api.thingspeak.comP�W3�
                         kippenReady, IP address 0.0.0.0��Ready, IP address 0.0.0.0
TS timeout /update?api_key=DWT274BS6CU0MKJU&field1=23.95&field2=1012&field4=759
�
 kippen23.95,1012,759�i�����TS timeout /update?api_key=DWT274BS6CU0MKJU&field1=24.03&field2=1012&field4=220
�
 kippen24.03,1012,220D�
`


> That's why it's tricky - we have to figure something out really

Barring editing wxpython itself to have different imports, i dont see how it would work?

Unless.. would wxversion.select(version) correctly change the wx import, even if the wx import currently refers to phoenix. Though that would probably be buggy and unreliable? Eh, i'll try.


you could also use that helper method here!


Thanks for having a look.

var PROXY_SERVER = 'https://calstate-primoprod.hosted.exlibrisgroup.com:443';

[01CALS_FUL.zip](https://github.com/ExLibrisGroup/primo-explore-devenv/files/609984/01CALS_FUL.zip)

Dave


The UML will be updated to follow the implementation


It just changes the names, so i dont see why not?


Oh, one more thing. When using cygwin, it installs itself to the C: folder. Move sector1.bin to C:Cygwin64\home\[computer name. For me, it's home\Ethan-Campbell.

![2016-11-23](https://cloud.githubusercontent.com/assets/18752458/20578486/64227d4a-b17c-11e6-9cea-6c1a2cf809ab.png)


Could do - just test it and document what happens from the user perspective in the commit message.
Btw congrats for wrapping those commit messages :)

Now let me rebase will you ? :P


try now with commit 7dd18bb4c08d358605b8ac543e61de94dc270c8b


1. Please add .gitignore file and list DevExtreme libs there (or `bower_components`, depending on your decision regarding the point 2)

2. I'd suggest using the standard `bower_components` dir, not `lib`. I understand that you copy directory structure of old ASP.NET Core projects, but is it of any use?


i don't see tests for the methods in this helper. if we're going to create methods to help us achieve a specific piece of functionality, we always need to test them!


Don't spend brain power in this


esto ocurrio desde el servidor github


Hi, sorry for being so ignorant, but where would Ipaste the above code so that I can use WeeWx ?


None of this shows up in the Telnet session though, it's as if the two processors don't talk to each other.


Cause you couldn't believe how easy it is to break stuff - just be sure it's ok and let me know


Do they really work and put breaklines under? It did nothing in my code :(


Martin, remain where you are. Am about to come by and hug you:

    root@pine64:/mnt/btrfs# lsusb
    Bus 002 Device 001: ID 1d6b:0001 Linux Foundation 1.1 root hub
    Bus 001 Device 002: ID 174c:55aa ASMedia Technology Inc. ASM1051E SATA 6Gb/s bridge, ASM1053E SATA 6Gb/s bridge, ASM1153 SATA 3Gb/s bridge
    Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

But of course `# CONFIG_USB_UAS is not set`. Gnaaaa... next try.


Yep... we need the Protocol 📝


I'm curious about this one


Ok. Thank you very much for the note. I am going to check the JavaScript code quality with http://www.jslint.com/.

About the "==" vs "===" question:

"The identity (===) operator behaves identically to the equality (==) operator except no type conversion is done, and the types must be the same to be considered equal."

From: http://stackoverflow.com/questions/359494/which-equals-operator-vs-should-be-used-in-javascript-comparisons


nubetest?


gg 👍


That's an interesting idea @rajyraman but this thing is just used to generate literal Guids.  Changing them could have bad consequences for any tests that have partial data in the System.


Classic!


Do we still need to reset the stream after writing the response?


@matumaros not better, but we teach each other)


Yes. Go ahead.


If you look in the WeatherPi.py file in my Github, it is on Line 394, and it is called on line 525.
Note: I have not updated this for the new Switchdoc code, so it's not a direct plugin. I do plan on updating as soon as I have the time and motivation.

If you need anymore help please let me know.


Se cambiaron las urls tanto de las colecciones como de los modelos,se cambio la propiedad unitname del PersistenceContext y otros cambios menores


That is not what i have in mind. JustRespawned should be moved in more correct place.


Thumbs up!


Why null?


Okay, how about this: https://github.com/git-for-windows/git/compare/master...dscho:fixup-preloaded-add


Ok. I think you are right.


nice 👍


_TRUNCATE копируется столько символом, сколько можно разместить в строке назначения вместе с завершающим символом, и добавляется нулевой завершающий символ
А вообще _TRUNCATE разрешает усечение, если передается в качестве параметра. Усечение строки меняет возвращаемые значения функции strncpy_s


![patiencegravity](https://cloud.githubusercontent.com/assets/7444037/20578820/0d29abe2-b17e-11e6-8cf6-5e213035ab20.gif)

i made a gif


My opinion is that we convert bucket names to lowercase. It may not be the most visually pleasing, but I think it is the safest option at this point. Which do you think will be an easier fix?


FixDaylightingTableZone (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2346 of 2346 tests passed, 573 test warnings)

 * 573 tests had: AUD diffs.

![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)


You have a new commit for this now. (Although I wonder why we have a changelog in the textfile when you can just svn/git blame the file to see the history...)


@CameraRick  Don't get me wrong, I like funny names like ChillSpill, too! And I think there is no problem as long as there is one word in the name that describes the main purpose of the plugin. Well that's not exactly the case with Tweaky, but I still like that name! It has so many functions that this word describes it best.

By the way, the plugins can be named different inside Natron. It doesn't have to be the filename. Just open the .py file of the Plugin and look for the following at around line 20:

def getLabel():
    return "just put your name here and save"

Thanks so much for all the contributions! As olear already said, the naming is not so important for now. I'll just rename it to "lp_FeatherAdvanced" so it's more the original name but blends a bit better with the rest. The node is so powerful that it's totally ok to put an "advanced" in the name ;)


I have taken care of some of the issue. For the rest I have created a new task: https://github.com/BujakiAttila/MapIT/issues/7


You're welcome to it :)


rename to IGMP Proxy?


("Hi Igor!");
("Bye Igor!")
)))


I don't know what you had in mind, but I realized that we have to adjust the flags to avoid problems so I added that.


Unfortunately, it seems like this change also removed the result tag from `jenkins.job.completed`! That's really problematic. Any chance you'll be able to fix that soon?


Please set this in YML via mysite config.

```yaml
---
Name: mysite
After:
  - 'framework/*'
  - 'cms/*'
---
# YAML configuration for SilverStripe
# See http://doc.silverstripe.org/framework/en/topics/configuration
# Caution: Indentation through two spaces, not tabs
SilverStripe\View\SSViewer:
  themes:
    - '$theme'
    - '$default'
SilverStripe\i18n\i18n:
  default_locale: 'en_NZ'
```


@TheAppleGamer Thanks pmmp xD


I haven't thought it through all the way, but I think converting to lowercase shouldn't be too hard. I will revise the requirements with this in mind.


os_2_0_develop (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Qua code is er weinig op aan te merken.

Het enige wat ik wil noemen is dat dit tegen de wens van @pebeling gaat. Hij wilde absoluut geen attributen op de yaml hebben.

Maar dat vechten jullie lekker zelf uit :8ball:


No.


Gabriell. I was pushing my file but needed to pull before because i was behind. Then it accidentally merged. Can you check it so i can fix


FixDaylightingTableZone (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1773 of 1773 tests passed, 0 test warnings)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-71.17%25-green.svg)


Ik ben een voorstander van `aDefault` implaats van `defolt`


I did test  this on CM that I built  (3.4K) (just stuck the file in the
proper directory)--that is where I discovered the lock-screen issue.  I
will confirm that the build prop edit still works and let you know.   The
problem that I see in not changing the comments is that the "default"  line
shows 3 of the orientations being "defaults" but our device has all 4
enabled,  which one can realize since 1 +2 + 4 +8 =15 --but it's'
confusing.  perhaps just an additional  comment line?  :-)

On Wed, Nov 23, 2016 at 3:45 PM, Andrei Măceș <notifications@github.com>
wrote:

> Sure, although I don't usually change their documentation comments, and
> the upstream CM default is still those three. I may amend the change if a
> related overlay or prop change can enable lockscreen rotation by default.
> Once there's confirmation that this overlay actually works, I'll propagate
> it to all M & N branches.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/airend/android_device_bn_common/commit/34fe0350e7b1b1f3e10ccc1c5a2012a9e461eda5#commitcomment-19943998>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AQR-2GrPV4Z9gYtNaN0gwIrlP5_C8oW6ks5rBKYCgaJpZM4K7A7i>
> .
>

--
Marti Packer


os_2_0_develop (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


FixDaylightingTableZone (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


This change (adding `slotId` argument) leads to some inconsistency. We store `gtLastProcessedSlot` and use it in `sscProcessMessage`. And in `applyGlobal` we use another approach and take `SlotId` as argument.


Thanks, @jscaber, not sure. If you have a fix, please create a fork and submit a pull request.
Currently in the middle of unicode refactoring.


@dscho  That looks good.  Thanks!


I am a strong proponent of removing the `sheet*` noise as well.


No in this specific instance. But more download the line, if you wanted to add functionality to setup fake data.


Good to know, thanks!


Yeah that should be it. It's those two, and internal_meta_keys for anything extra we wouldn't want loaded.


I've revised the requirements so that bucket names are obtained by converting to lowercase. Now how does this look?


So, 180 degree rotation definitely works out of the box, right? I might remove the comment altogether when I amend this change, to account for lockscreen as well.


os_2_0_develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Oops. This commit was meant to be fused with b848bd9d52ff1d834dd1c360e703750fecf6d2f7.

See that for a better commit message.


@smcavallo I believe this is a regression. Why would we not let the cookbook randomly generate a password in chef_zero -- state is saved on the filesystem, not "lost" like it is in chef-solo mode. Can I ask what is the use-case where this needs to fail out?


The more I think about it, the more convinced I am that they were having Sen Meg available in G1, with the seal stone intact, unbreakable. It's very strong evidence. Also now that I think about it, the bridge from Emain Macha to Sen Men plateau is so awkward that it probably both zones were designed at different times. I retract my statements about it not being there in light of this.


This commit was meant to be fused with 25df84fbe41bc6151394320b1702e3467fa89bcd.

The rest of the refactor is there.


I might mention that ntpd should be run on the nodes running Zookeeper, too. Hopefully that's a given :) but the distributed lock acquisition can fail / take a long time if any of the clocks are horribly out of sync.


Does this mean isInstance is broken?


Isn't the IDView what CitationView was in FOAM1?

The problem with putting this here is that if someone just wants to use DAO's on a node server they're now forced to bring in the u2 package because IDView is an inner-class and it extends Element. Move it someone in the u2 package so that doesn't happen.


This fixes rocknsm/SimpleRock#51


I dont't kow :)
its generated automatically this way (ctrl + n / gen. constr.)
I'll make a question card on trello with this


Good job!


os_2_0_develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


It was implementing a CLASS, not an INTERFACE, so it may have been
working-as-intended. If implements is supposed to support classes and
interfaces, then maybe there's a problem.

On Wed, Nov 23, 2016 at 4:24 PM Kevin Glen Roy Greer <
notifications@github.com> wrote:

> Does this mean isInstance is broken?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/foam-framework/foam2-experimental/commit/466d969e430540d208c5f3625ff37b20c8fcde5f#commitcomment-19944407>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AIKjMIcRXHO9Z_ZshBdsPNLMLiPEvozYks5rBK8ggaJpZM4K7D5I>
> .
>


The issue was if you went to releases, it would have 10+ releases with an old commit msg from the master branch.

So a new build on testing shows up with the wrong commit message, and even if it was just built it would show as "10 days old".

Very confusing if you were trying to locate a build.

I believe github only like placing releases with the master branch.

Now... Built in any branch, its easily found and identified.

Certainly seems like you've got a comm issue.
Ill see if I can find you more info.
Any chance you could post your current connection scheme between the two devices?

Btw, I've used these with good success... And they're cheap.
https://www.amazon.com/gp/product/B01IK9GEQG


We should add to next design review agenda.

On 23 November 2016 at 16:27, Jackson Stone <notifications@github.com>
wrote:

> It was implementing a CLASS, not an INTERFACE, so it may have been
> working-as-intended. If implements is supposed to support classes and
> interfaces, then maybe there's a problem.
>
> On Wed, Nov 23, 2016 at 4:24 PM Kevin Glen Roy Greer <
> notifications@github.com> wrote:
>
> > Does this mean isInstance is broken?
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/foam-framework/foam2-experimental/commit/
> 466d969e430540d208c5f3625ff37b20c8fcde5f#commitcomment-19944407>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AIKjMIcRXHO9Z_
> ZshBdsPNLMLiPEvozYks5rBK8ggaJpZM4K7D5I>
> > .
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/foam-framework/foam2-experimental/commit/466d969e430540d208c5f3625ff37b20c8fcde5f#commitcomment-19944438>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AHHXsw6Z0enOS0E8NByVePwD3R6psjDfks5rBK_EgaJpZM4K7D5I>
> .
>


Note, that this code is untested yet.


yes, though ideally it should rename automatically. this annoys me. :)