Should this README.md be combined with the one in the parent directory? The parent one has a section for 'smoke tests' already, but lacks details of the command to run from the makefile. The known issue with DB names highlighted here is not specific to the smoke tests, and should probably be part of the main README.md - This is the only file in the tsuru-testing directory, which looks a bit untidy.


`(Math.random() * 100)` generates a random number between 0 and 100, so `(Math.random() * 100) + 25` will generate a random number between 25 and 125. How do we generate a number between 25 and 100 though?


`func lookup(ip uint32) (data int32, ok bool)`


First part of fix for #41


Fixes #41


yep i googled after asking got it
cool :P


Will be left undefined when user is NULL


Not new, but there is a typo in this comment


thx :)


Shouldn't be any issue here, we either set or retrieve from this and they're both atomic operations.
Not like we're setting a value in the cache and then partially modifying it or anything icky like that.


Почему нет? Поля изначально задумывались, как дополняющие друг друга. Кроме того, при переносе из ВК не знаем, рабочий это телефон или домашний. Хотя, это единственная проблема.


ok


Actually we still only check the property name, not the type (static, get, set) issue #100.

Here `statics` only match the `statics` method in ES5 classes.


I changed this because the "List" variable is in list format already (it is a list of transactions being prepared for that key, which should be just one element most the time, but can be longer), so using "++" gives a list as a result, instead of creating a list of lists which what was happening when using "|".  Having a list of lists would make the code here more messy: https://github.com/SyncFree/antidote/blob/prepare_fix/src/clocksi_transaction_reader.erl#L201

In any case the "List" variable (which is the one that will be copied by the "++" operator) is usually going to be 1 element and be smaller than "Acc" so it shouldn't be too costly.


this is a comment for changes in graduate branch


Thanks !


Patch applied less whitespace changes.


Very good, you remembered to keep the columns number at 12!


Мы теперь храним все личные данные ребёнка в заявке. Какой именно момент считаешь спорным? Переименование действия?


They shouldn't be functions returning `IO ()`, they should be terms with type `IO ()`. For example, `putStrLn` would not be accepted, because even though it's a function returning `IO ()`, it isn't a runnable IO action.


you're right, just updated it.


Раньше была проверка, если post отправляет оператор, то фиксируем, что данные изменил оператор. Смотри коммиты позже, там я решил этот вопрос.


You're living in the future... ("2016-06-21")


Actually, considering that, I should make another change so that the ets table is usually small, and not full of empty lists, I will make another commit.


sounds good, I'll update it


What's the msbuild issue you were having?


The method names suggest something else from what the methods are actually doing.


buildOrder*Command may be refactored to use polymorphism or an typed parameter


What exactly isn't clear? the list of required files on the Datafiles section of the ScummVM wiki is still accurate, and kept update to date. That is always the place to look for any recent changes.


just tested it and yes


But why you double quote only the condition and not the remove and mkdir calls?


I've made an issue about this: https://github.com/sun-opsys/api-doc/issues/8


Patch applied. Many thanks.


\o/


The `Choose` here is done before the BeforeBuild construct.

https://github.com/mono/monodevelop/blob/master/main/src/addins/VersionControl/MonoDevelop.VersionControl.Git/MonoDevelop.VersionControl.Git.csproj#L226-L242


I think it still is necessary


@baraabourghli when looking at this PR, I got a little concerned by the use of :nocov:. Initially I suggested using it, but I should have warned that it's a last resort and if we can find a way not to use it then we should do that instead. I noticed that a large part of the logic in this importer was not being tested, which was what made me concerned.

So what I've done is rather than mock `fetch_data` (and all the logic behind it), I created a `double` to replace the Mechanize instance. Now, all the code is executed by the spec, and the Mechanize double returns fixture JSON rather than actually talk to the network. The mocking is more complex, but now all the code is covered.


Line is too long. [101/100]


Method has too many lines. [14/10]


Line is too long. [119/100]


Line is too long. [118/100]


Line is too long. [105/100]


Line is too long. [106/100]


build*OrderCommand() methods may be refactored using polymorphism or use the factory and extract the queue.put() in a common method


I know, but we know `defaultProps`, `statics`, `mixins`, etc. will always be defined as `static` class properties, no? While it can be argued that `constructor` is not a React lifecycle method per-se, I think the following would be preferable:

```js
order: [
  'lifecycle',
  'everything-else',
  'render'
],
groups: {
  lifecycle: [
     'displayName',
     'propTypes',
     'contextTypes',
     'childContextTypes',
     'mixins',
     'statics',
     'defaultProps',
     'constructor',
     'getDefaultProps',
         // etc.
  ]
}
```

This way, this:

```js
class MyComponent extends React.Component {
  static propTypes = {};
  static defaultProps = {};

  constructor() {/* ... */}

  render()  {/* ... */}
}
```

feels cleaner than:

```js
class MyComponent extends React.Component {
  constructor() {/* ... */}

  static propTypes = {};
  static defaultProps = {};

  render()  {/* ... */}
}
```


This is wrong. Try

    $services.model.resolveDocument("Class.Name.With.DotClassSheet").getParent()

The special characters from the document name passed to ``createDocumentReference`` don't have to be escaped while they have to be escaped when calling ``resolveDocument``. We need to be more careful with these changes.


Addressed in https://github.com/ajayaa/keystone/commit/709131b376aeb5dd9feb5338cf6556a8407aa748


how to create a sign to teleport on a map ? ( First row, second ... )


Sounds good Ken. I'll push an extra commit that adds this. Thanks for the feedback!


Patch applied.
It isn't clear at the moment what the purpose of this commit is. I suspect I'll be able to deduce the reason from the following commits but it would be nice if there was a line or two of explanation in this commit or the comment explaining why this commit was required.


How to create a sign to teleport on a map ? ( First line, second ... )


Doesn't the .so version express the the ABI compatibility? Why should it stick to QT version? IMHO it's best to change it manualy every release (if it's needed).


Ok thanks, I see now that this is edited by the same people that work on the game engines; and that Elvira was changed recently.
I was double-checking it with `strace -e open`; but I got confused :sweat: because installed version behaviour didn't matched the wiki; I'd better track it from git now.


Hi, the scalar value is of the range specified. This closely mimics what the hardware does as opposed to converting into whatever value you'd like to use.

However, since SI units are standard we appreciate that it might be useful to have some helper functions to return these values. We review and accept pull-requests if you'd like to submit a change to the code.


duplicate code with buildAddOrderCommand() method


:heartbeat:


Initial commit of Clive-Dev branch


Building an android app that will keep track of your work schedule.


With a small button, it should save space. If it doesn't it's because of a margin problem.


I sendt you a PM on forum with email.


@dietzc @StephanPreibisch @hadim I agree, this was correct before and is now wrong. If ```service != null``` then ```s == service``` so it's not self-created and you don't want to shut it down.


Hi @mflorea.
That you for having noticed it. I'm going to fix it.


in windows in general or just for coala?

it's possible anyways I guess


Thanks. Patch applied (with a few tweaks to fix IDE errors, comment typos, etc.)


Pushed fix


comment that this variable comes from outside


There's a good chance that the shared build worked with icu.


Are you sure that is 65?


Да, Денис, Вы правы, посмотрел в основном документе.


The commit title is is `Remove FileLineVisitor` when you removed `FileVisitor`.


Patch applied.

It looks like there are places where you could be declaring fields as final and are not doing so. Keep this in mind and try to make more use of final in your JASPIC code. You may find the UCDetector plugin for Eclipse useful.


Shouldn't be without space?
..
   '  request'


شسيشسي


Да, Денис, Вы правы, в примерах на http://kyivpassport.com/transliteratio/ их опускают.


You are wrong here. First of all this object (finalize plot) is in a different position relative to plot coords even for the same PlotID (they are unique on PlotInstanceID level). Secondly GameObject.db2 does not contain relative positions but absolute ones.

While the positions of plots are different in each garrison level they are so close (within 1 yard) that doing it this way is acceptable (but I will look into it more later)


Jij weet er meer van dan ik. Weet je ook nog referenties die deze claim ondersteunen?


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


waar


Further testing showed that the test-error I saw was spurious. Just for completeness, I am posting the output here. It shows the doc-test failing.

```
running 7 tests
test test::lock_errno ... ok
test test::lock_invalid_filename ... ok
test test::lock_wait_errno ... ok
test test::lock_wait_invalid_filename ... ok
test test::lock_wait_ok ... ok
test test::lock_ok ... ok
test test::unlock_ok ... ok

test result: ok. 7 passed; 0 failed; 0 ignored; 0 measured

   Doc-tests file_lock

running 4 tests
test lock_wait_0 ... ok
test _0 ... ok
test lock_0 ... ok
test unlock_0 ... FAILED

failures:

---- unlock_0 stdout ----
	thread 'unlock_0' panicked at 'test executable failed:

thread '<main>' panicked at 'called `Option::unwrap()` on a `None` value', /Users/rustbuild/src/rust-buildbot/slave/stable-dist-rustc-mac/build/src/libcore/option.rs:362
', /Users/rustbuild/src/rust-buildbot/slave/stable-dist-rustc-mac/build/src/librustdoc/test.rs:254
```


robloxplayerlauncher.exe the program has encountered a problem and needs to be closed how to solve it ? help me please let


Patch applied. My previous warning that context.getServletContext().getVirtualServerName() + " " + context.getPath() may not be unique still applies.


disables seeing .html if it was a real webiste


hi, tried to use today and realized the dependency to "statsmodels" is missing (required by pandas-wrapper.py).


Thanks @alanmcgovern


maar het kan nog wel een introductie zijn..


Maybe like this:
(Math.random() * 75) + 25

Date: Tue, 23 Jun 2015 02:13:39 -0700
From: notifications@github.com
To: bloc-jams@noreply.github.com
CC: jheekhout@hotmail.com
Subject: Re: [bloc-jams] completed collection view with jQuery exercises (b90a5e6)

(Math.random() * 100) generates a random number between 0 and 100, so (Math.random() * 100) + 25 will generate a random number between 25 and 125. How do we generate a number between 25 and 100 though?

—
Reply to this email directly or view it on GitHub.


Also, it does not take into account the fact that the class could be a "WebHome" document (because of the Nested Document Concept).


Fixed by e10761b3cbeb4ad7f284ee50f281c7668c2d7928.


This cannot work: Travis infrastructure is on Ubuntu 12.04 LTS precise and you give here a 14.04 LTS trusty URL.
On Trusty, one should use the native Qt 5 packages, thus without adding a repository and with different package names. On the Wiki, the following packages are listed:
    sudo apt-get install python-nltk gawk cmake qt5-default libqt5xmlpatterns5 libqt5xmlpatterns5-dev \
qttools5-dev build-essential libboost-all-dev libenchant-dev mesa-common-dev libgl1-mesa-dev\
libglu1-mesa-dev

But the .travis.yml file should stay compatible with Travis. I don't know if it can be made compatible with Ubuntu 14.04 at the same time.


Needs to be `run-with-rbenv-path` now


An assembler macro would be cleaner for this.  For example, this in machine/cheriasm.h:

	.macro cmove to, from
		csetlen $c1, $c2, $zero
	.endm

This lets you keep using cmove in the asm, but replaces it with the desired instruction sequence.


Yes, exactly. :+1:


Patch applied.


See https://github.com/petehunt/node-jsx/issues/44


test


Actually, this is not needed, because we do not care if the comms server is started.


add copyright header (license)


I'm working on a fixed version, in the meantime you can use the theme's colors like so:

		panel1.setBackground(UIUtil.getListBackground());
		bottompanel.setBackground(UIUtil.getListBackground());
		centerpanel.setBackground(UIUtil.getListBackground());

in the dialog's constructor


looks good to delete the line


I get your point. Perhaps then we should just drop this commit altogether then.


LGTM


Ya casi!, ya casi! :D :+1:


I'm wondering if we can just drop that HEX64_RE altogether?


Nope, he ***absolutely*** meant it like *that*.


Hello Sean, this is the only way I know to contact you but, the Robotics laptop forgot the git login of the Team3786 Account. It would be nice if you could email me with the account information if you know it. If there are any other account i should be aware of, then please include their information as well.


I think you have a mistake in your commit message subject: instead of 1.0.1 you meant 1.1.0. NBD


what i have to do with this my friend?


... lol


So that it is globally available, as required by ES6. Feel free to suggest a better place for it!


Why is this main method here?


굿! css 프리프로세서로 Stylus(아님 less?) 쓰려고 생각 중인데 시간 될 때 스타일러스로 바꿔서 올려볼께.


Works for me, but I was fixing this because Srdjan was affected by the log noise.


this smiley will go soon.


If getClusterIdentifierTag returns empty, you should not add the tag.


Well, this package just provides access to the Babel compiler with some defaults, like the `meteor-babel` npm package does.  It doesn't deal with how to run the code.

If you write a build plugin that calls it, you have to figure out how the resulting code will be runnable.  For the `jsx` package, no new helpers are needed.  If some were, the `jsx` package could probably just emit some code that assigns them.

If we use the `meteor-babel` package, maybe we don't even need this whole package?


Does this present much advantage over just using read and write?

if (!db.read("ham")) { db.write("ham","sandwich"); }


Done!


The links you are adding are assuming code is running in `dartdoc`. This means links will be broken when someone looks at the readme on GitHub.

Any way to work around that?


Yeah, for sure.  I think they do move stuff around between minor versions (like transforms and helpers).


Thanks for doing this. LGTM.


I always thought it was 1.3.7 ?


I'll appreciate if you add me to authors list, however will not complain if you refuse to do so.


foobar


Very nice!


어제 못보여준 배경 예제:
http://www.nexusmods.com/witcher/?
http://www.nexusmods.com/newvegas/?


It seems to me that all the other functions that try to write to this file, with the exception of `g_file_open_readwrite()`, already will create the file when it does not exist.
I don't think you need a R/W stream in `emer_circular_file_save()` so I wonder if you could not remove this block here and use `g_file_append_to()` (or a variant of `g_file_replace()` - there is one that also gives you back a R/W stream) in the other place.


I can see these tests being a bit of a pain to maintain.  I might be willing to do it, but I don't know if anyone else would be.  At one point, I generated a web page containing these input/output pairs as a sort of combination of documentation and regression test, and it certainly makes it easy to see what's going on and catch any changes made to Babel.  But some of code is really long, and writing and working with these test cases is a bit annoying.

I will pare these tests down to normal unit tests that verify things that are harder to be sure of with runtime tests.  For example, make sure `const` turns into `var` by grepping the output.


удаляем комментарии с кодом


todo?


сделать проверку text на null
элемент искать 1 раз
сделать метод findElement - в этом классе


I wonder if `resize()` shouldn't return `TRUE` when `prev_max_size == priv->max_size`.


Проверять text на null


static


This seems to have broken tests/test-type-load, in particular, the compilation of the load-missing.il file. That il file contains many references to missing types etc. Its hard to see where the failure is, since both monodis and ikdasm fail on the file produced by both the 4.0 and master ilasm, but the test used to succeed before this commit.


test-output не заливаем в репозиторий


There is really no reason for this to be a macro.


So apparently they added more. This is exactly the source of the codespell dictionary ;-)


Yes slashes are intended as separators between segments. But empty segments are allowed AFAIK. So an URL like `http://example.org/foo/////bar` is valid. Where does it say, that empty segments at the beginning of a path are invalid, or as this change suggests should be treated as a single slash?
If `http://example.org///foo` is valid, then it also means the path is `///foo`.


Other than Issac's comment, LGTM.


@nateberkopec In our app we were explicitly setting `environments` to an empty array (based on an `ENV` setting) so that we could enable raven in some deployments and not in other deployments.  This change broke that.

It seems a little counter-intuitive to send in all environments when the `environments` array is empty.  Would it not be better to set `environments `to an array containing the current environment by default, and then _only_ send in the current environment if it is _explicitly_ listed in the `environments` array?


:) no problem! it is kind of funny most of the time


Yup, no `innerHTML` on document fragment and `DOMParser()` as well as `createHTMLDocument()` don't support fragments of special children like `<tr>`.

The only real way would be to use `<template>` but it's not widely supported or to use `document.createRange().createContextualFragment()`, which is just about as widely supported as `<template>`.

Most suggestions I've seen on the interwebs fail to take into account the special nodes and just use a temp `<body>` or `<div>` which don't work.


missed a $ctxcallarg here


*Thor (Phoenix)


Not a lot of harm in moving this before the block above and have the `!seek_succeeded` condition use the `goto`


@subvertallchris Did this ever get changed to default to false?


Is this spacing better?


Rename this variable from Post to Homebrewer


Looks like it's still set to default to true. https://github.com/neo4jrb/neo4j/blob/master/lib/neo4j/config.rb#L102


In such case I suggest removing and reverting in case of any issue. Thus code will be clean if it can be.

I think code that might be used by someone somewhere outside repository is not reasonable to keep and maintain, moreover while duplicating with no explicit description.


Thanks for catching this.

Andre


Glad you asked because the way `emer_circular_file_save()` was written originally (using `g_file_append_to()`) was actually my nastiest bug. Using `g_seekable_seek()` on an output stream created with `g_file_append_to()` has no effect whatsoever but (heinously) returns success. Append output streams write to the end of the file no matter what. Yay tests.

As for `g_file_replace()`, Roddy is pretty concerned about the impact of writes on the lifespan of SD cards/eMMCs. Since this commit also increases the persistent cache size to 10 MB, I tend to agree that (especially for our offline-only users) the optimization to only write O(data recorded in last hour) every hour rather than O(10 MB) every hour is a prudent one.

All that being said, we indeed don't need a R/W stream in `emer_circular_file_save()`. I just couldn't find another function that lets me open an output stream that actually supports seeking and doesn't force me to rewrite the entire file.


Using header style for everything was a bit painful :p unordered list made more sense


This will avoid random conflicts with meaningless binaries


oyeeeeeeeeeeee


Test


:beers:


good job


No need to manually init these. Private data will get zeroed out automatically by GObject.


foobar


The way you were splitting up the champions file into a dictionary for pickle was totally correct; thought you might find this simplified approach clever


Just learned how to dynamically load templates into a component with TDD thanks to that commit :)
Thank you! @mbenford  :+1:


Using ast.literal_eval gave me errors if the player didn't have a known MMR. The request object returned by requests.get has a .json() method that gives you back the same result and didn't seem to have this error (plus, it avoids an import)


This can be refactored to be something like that, yes.


Looks fine in the sense that the demo only needed `lsst_apps` so should only have been setting up `lsst_apps`.


This reverted https://github.com/middleman/middleman/commit/9abcdaa7f7630abeb3b4de64519391b779c620af FYI


These url_gen functions were really only being used for one other function (url_gen1 was really just for summonerID), so it makes more sense to keep the code together – it's easy enough to factor the code out if you need it in another place later, but I don't think you will

The other change was that you were getting json from the request results in different ways – using simplejson and ast.literal_eval. There isn't really a reason to do it in two different ways, and the requests library already includes a .json() method on request objects, so that seemed like a good, standard choice


Sorry I can't think of any scenarios and I don't recall the history.


Cualquier problema avisar y comentar


#comment


Sounds good. I'll remove the `protected` bits of this class but leave `ConvertTo()` `virtual`. That'll save me documenting something that doesn't need to exist.


Big thing here is that I tried to make more explicit the way you were building up the urls so that you would get games in 20 game  chunks – using numpy here felt a little overkill ;)

One thing to be wary of, and I didn't change this because I'm not sure how best you want to deal with it, is that you're calling api.accountID(name) twice in url_gen because gamecount also calls api.accountID(name), so you're making an extra call. When you're making API requests, it's generally a good idea to store the results and avoid making duplicate calls or the service provider will get mad!


What does the `\"\"` do at `spec/runtime/executable_spec.rb:115`? I'm unsure why this spec is failing, but perhaps this has line something to do with it.


This only bypasses the check if the file is present AND locked. In that case, the user has chosen their dependencies and is willing to live with the consequences :)


![magic word](https://cloud.githubusercontent.com/assets/12468465/8320405/b0b0c4b6-19d7-11e5-8aab-d4fe1a30cc45.jpg)


I may or may not have to change, or get rid of this method.


Ah ah ah, you didn't say the magic word!


Ah wait, I see your point and I think you're right


handy :)


Can use `g_build_filename()`


@William-Byrne  You need to change it here as well...


@William-Byrne You should change it on line 589 as well (where you define the byte buffer).


The contains() check is unnecessary for Set.


Yeah, it needs to be (lock file == null || **!**locked) :)


Hello - Even the most up-to-date mime.types file from the AWS SDK library doesn't have that content-type:  https://github.com/aws/aws-sdk-java/blob/master/aws-java-sdk-s3/src/main/resources/mime.types

PR welcome, but I'm not sure there is a simple solution to adding more content-types.  You might be better off forking and adding support for the *.apk content-type, or dropping this plugin and manually upload the file with the correct content-type in a post-build step.


Great! Thanks. :)


Yeah, that seems cleaner to me too.


Yes, we can do that.


so does this mean we are now compatible with msys2 / mingw64 and not compatible with cygwin cross-compile (at least, for the time being)?


Já que vc não tem nenhuma validação, pq vc não faz o `create` ao inves do `build`?


this needs to be added back in


I don't think `priv->variant_file` can ever be `NULL`, or we would have failed initialization


Oh nevermind. This is called also when initialization fails :)


A indentação tá errada. Os metodos privados devem ser alinhados com a declaração do `private` [ruby-guides](https://github.com/bbatsov/ruby-style-guide#consistent-classes)


This function does not look like it depends on the persistent cache. I wonder if it should not be moved somewhere else


Pq vc não usou o `localize` do `I18n`?


@benvanik , regarding loading up iso/xex,  is it allow to add open browser support ? Other than the current drag and drop .


Remove esses `.coffee` :stuck_out_tongue_closed_eyes:


This comment style is inconsistent with other comments (should be `/* When always ... */` to be consistent)


Closes #20


It was in shared/metrics-util.h before, but I actually moved it in here so that it'd be closer to `emer_persistent_cahe_save()` and `emer_persistent_cache_read()` since it's coupled to their implementations. In other words, it is morally part of the persistent cache because `emer_persistent_cache_cost()` would need to be updated if the persistent cache changed how it serializes variants.


Delete it


Can you convert to ternary pls?
```
let rootStyles = this.state.open ?
  this.mergeStyles(styles.root, styles.rootWhenOpen, this.props.style) :
  this.mergeStyles(styles.root, this.props.style);
```


1440-AWS (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2149 of 2202 tests passed)

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


Hey looking at your address_book.rb u haven't changed the remove_entry method :)


I have typically avoided `assert`, although I'm starting to come around to them.  But use them carefully.  The general rule in Python is that a program's behavior should *never* change if you remove all asserts (that's what `-OO` does when you compile Python modules down to `.pyc` and `.pyo` files).

So for instance, don't use them to check user input; don't use them to check the return status of a function when there's a chance that function might *not* return what you want; *certainly* don't put a function call that actually *does* something in an `assert` (as removing the `assert` will prevent that task from being done).

I use `assert` here because the way I've intended my design to work is that the `coordinates` setter will ensure that the shape is correct, so the assert will fail only in cases where I've made a mistake somewhere.

Just some tidbits of wisdom I've picked up around the Python community (in case you didn't already know).


bower??


closes #71 closes #72


@mtrepanier I just released new version


@fproulx-pbox I just released new gem version


Missing semicolon


The loop in the menu is a cpu waste, is not sense for me.
which is the problem with kill background apps? if is supposed to boost the gaming isn't need to be other apps in cache, I think... too will be good kill unnecesary android services, gaming will be better.


Hi,
attached new version....

On Tue, Jun 23, 2015 at 3:16 PM, boondog <notifications@github.com> wrote:

> How about an RGB LED?
> https://www.sparkfun.com/products/9264
>
> On Tue, Jun 23, 2015 at 3:07 PM, Sajin <notifications@github.com> wrote:
>
> > Patrick, the logo is in layer 48. Yes we can make the PCB a little
> smaller,
> > I will try to keep it within 5cm square. Using SMT would make it look
> > better, I could swap the IC package to an SMT. With an SMT IC on the
> board
> > this can also be used for an advanced soldering class.
> >
> > @mike how do you feel about having a SMT IC?
> >
> > Regards
> > Sajin
> >
> > On Tue, Jun 23, 2015 at 2:33 PM, Lowell Makes <notifications@github.com>
> > wrote:
> >
> >
> > > Can I just interject that just one led isn't awesome. I agree with the
> > > next design could have more advanced options, but make it interesting
> why
> > > would i wear or use this thing? Because "I made it!" I got both the
> leds
> > > lit up!! I am awesome!
> > > Sara
> > >
> > >
> > >
> > > > On Jun 23, 2015, at 1:29 PM, leftyfb <notifications@github.com>
> wrote:
> > > >
> > > > We can do more LED's on the next design. This one is meant to be REAL
> > > simple. The original design was just a single LED and a battery. The
> rest
> > > was a bit of future-proofing with minimal effort. We need to get these
> > sent
> > > out and back ASAP.
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/LowellMakes/LowellMakesBadgePCB/commit/0e4e89d7cd58e51b67c3d2c9af9e53ef30fc1da2#commitcomment-11822959
> > >
> > > .
> > >
> >
> >
> >
> > --
> > Sajin George
> > mycrobonics.com
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/LowellMakes/LowellMakesBadgePCB/commit/0e4e89d7cd58e51b67c3d2c9af9e53ef30fc1da2#commitcomment-11823519
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LowellMakes/LowellMakesBadgePCB/commit/0e4e89d7cd58e51b67c3d2c9af9e53ef30fc1da2#commitcomment-11823687>
> .
>

--
Sajin George
mycrobonics.com


keze@Asus-x99-Series:~/Otro-Repositorio$ valgrind ./main Configuraciones Configuraciones Instrucciones
==4951== Memcheck, a memory error detector
==4951== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al.
==4951== Using Valgrind-3.10.1 and LibVEX; rerun with -h for copyright info
==4951== Command: ./main Configuraciones Configuraciones Instrucciones
==4951==

Caracter="
pagina 1 linea 2 posicion 1
pagina 2 linea 5 posicion 1
pagina 2 linea 6 posicion 1
pagina 2 linea 7 posicion 1
pagina 2 linea 8 posicion 1
pagina 2 linea 9 posicion 1
pagina 2 linea 10 posicion 1
pagina 2 linea 11 posicion 1
pagina 2 linea 12 posicion 1
pagina 2 linea 13 posicion 1

La palabra Biblioteca no se encontro

==4951== Conditional jump or move depends on uninitialised value(s)
==4951==    at 0x401BE0: Ranking_palabras_Glosario (in /home/keze/Otro-Repositorio/main)
==4951==    by 0x400F62: MostrarRankingPalabras (in /home/keze/Otro-Repositorio/main)
==4951==    by 0x400E28: Proceso (in /home/keze/Otro-Repositorio/main)
==4951==    by 0x400B49: main (in /home/keze/Otro-Repositorio/main)
==4951==
" 10 repeticiones
Caracter=" 10 repeticiones
[salto_pagina] 1 repeticiones
[separa_palabra] 1 repeticiones

El ranking se encuentra vacio.

==4951==
==4951== HEAP SUMMARY:
==4951==     in use at exit: 0 bytes in 0 blocks
==4951==   total heap usage: 109 allocs, 109 frees, 12,088 bytes allocated
==4951==
==4951== All heap blocks were freed -- no leaks are possible
==4951==
==4951== For counts of detected and suppressed errors, rerun with: -v
==4951== Use --track-origins=yes to see where uninitialised values come from
==4951== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
keze@Asus-x99-Series:~/Otro-Repositorio$


Missing semicolon.


Changed to fetch text inbetween the <form> element, rather than the whole document.


1679-DualDuct (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2192 of 2216 tests passed)

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


why you ignore mocha_results :disappointed:


Yes I also considered using the MRAA but this might be a little more generic and I think may be even faster and easier to migrate even to nodeJS or to other languages in which mraa may not be portable. If you think it is useful I can take a loot at it.

UPM Project? what is it about? maybe you can share the link with me.
Thank you for your comments!


라져


Man that has been crushing me for soo long. Thank-you sir.


LGTM :+1:


Hmm, not bus stop search, but bus **service number** search :stuck_out_tongue:


`F`acultad


Ok, solo para probar que puedo comentar.


1440-AWS (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Ugh it was a startup timing issue?


@kellrott It looks like draft 1 support has been removed from the reference implementation - so I think I am going to have to focus Galaxy efforts solely on draft 2.

@tetron for context - @kellrott was hoping Galaxy would support draft 1 for compatibility with some efforts at SBG but it looks like that is not going to happen. I am fine with this - draft 1 doesn't seem like a valuable way to spend time at this point.


Can you rename this prop to `innerStyle` please? We have similar props on other components.


Why include this when it is excluded in `debian/rules`. Actually this file is not needed at all. It should pick up `README.md` automatically.


Hey @andres-montanez, whould you please update the package, as @drAlberT says, currently we have to apply the patch manually.


Should comment this to disable.


This `private` is redundant. There is one at the top of the class.


thanks.


Release needs to be `stable`, not `unstable`.

Also drop the `~6`.


This is not the initial release. A different message would be better.


UNDO!!! UNDO!!! ME, (@BDMorrell) you have to undo this disaster!


- qiita-js は常に stringify した JSON を渡しているので問題なし


Use `dch -r` to make the changes above and it will automatically update the date here.


:cake:


Use ev3dev github, not helensink.


Don't you need to do this in the error case above too? Or is the client expected to release non-floating refs when this fails?
If so, you could also move it into the other loop above AFAICS.


ditto


Don't need this line if there are no conflicting packages.


copy


I actually applied the patch myself to my kernel and fixed up rejects, but thanks anywho :+1:


Didn't seem to work for me. Will have a sus later.


Now enrolled is the top priority, then cheater, then glitch.


Screenshot :cry: :
![screen shot 2015-06-23 at 6 14 37 pm](https://cloud.githubusercontent.com/assets/7341/8320849/7ad8411e-19d4-11e5-8142-dd22b51ea0a4.png)


You'll want to make this PR against PolymerLabs/polymer-externs. It's currently off of your repo.


Hey, this looks very interesting? Do you think we can work on this together?


@mcmonkey4eva Does this make sense?  I reverted back to the older code use of object instead of the just grabbing the whole entity.


missing end paren quote on line 56

+                if(typeof node === 'undefined' || null === node) node = '';


Haa?? Actually memang dah order.. Dah ikut step by step nak order.. Then terus tak paham.. Apa pun memang jadi order.. Haha.. Boleh bro tolong update my order tu tak? Hehe..

From : Mohd Fairuz Bin JamilWebsite : Blog de Fellas | Twitter | Facebook

Date: Tue, 23 Jun 2015 07:42:52 -0700
From: notifications@github.com
To: TempahTeeGit@noreply.github.com
CC: akirien@live.com
Subject: Re: [TempahTeeGit] Update Senarai_Order.txt (be274ab)

tak jadi order ke bos?

—
Reply to this email directly or view it on GitHub.


Have you tested this? I think you need to add `dest='heap'` to the parameters since the CLI arg name (`--max-heap-size-gb`) is different from the corresponding parameter in `get_deployment()` (`heap`).


Oops, github's UI confused me. Filing the PR properly.


if(status[i] == UNKNOWN && canJump(board[board_index], board[i]))


Does this really test that the file grew? I think that it tests that resizing the file to a new size does not discard the existing data, but not necessarily grow it.


não


Did you have a type definition for ProjectInfo? In particular, 'build.json' has all the platform specific signing keys, app profile etc. that influences how build & run works. I wonder if all of that could be now factored into `ProjectInfo`?


I like your idea of array of `BuildArtifiact`. It does not have to be strictly typed as there could be quite different for every platform.


No unit test?


maybe an else should be added
written as:
if(status[i] == UNKOWN && canJump(board[board_index] != CAN_BE_REACHED))
{
   if(dfs(i, board, end, status) == CAN_BE_REACHED)
      status[board_index] = CAN_BE_REACHED;
}
else if(status[i] == CAN_BE_REACHED && canJump(board[board_index], board[i]))
 status[board_index] = CAN_BE_REACHED;

However, this else if is no matter, as dfs is used and the code would not go to this condition..


Is this test missing a call to `assert_strings_read()`?


Nice!


Fixes #4


Fixes #697


It isn't used. It should be used line 579


and a few other places in this method.


Is this okay or should I add a line break? I don"t know how the document generation handles this.


Looks good


- Code convention.
- The constant must be left operand.
- After if must have a whitespace
So it should be if (2 > (len(sys.argv)):


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2201 of 2216 tests passed)

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


OK, never again.


@rxin how can I use your mentioned coding? `def(col) = return sha1(col)` didn't worked.


sorry, I don't follow your remark here :) what should I change ?


I think the persistent cache constructor should work with both paths that have trailing slashes and paths that don't


@levicole inital app setup complete including creating welcome controller and adding twitter bootstrap gem


Code convention. Must have spaces before and after an operator.


This condition would not occur. Since the vector's capacity can be modified by function resize() or push_back(). In both functions, the size relationship between theSize and theCapacity is considered.


how do i even delete stuff :|


IMO these are 2 different goals (even if both are done using service workers): one is pure offline capability for when we'll have a HTTPS-based app; the other is programmatically deciding what to cache. I agree we'll make use of the same service worker technology but I feel these are 2 different capabilities.

During the presentation I can still tell they're highly related


glaring n00b status


thanks, I couldn't find the name :p


Please make the code style consistently. Here, having space inside brackets. But at other points, they don't.


The first word of a sentence must be capitalized


you're right, I can be clearer in the next slide


Hi, @adrianq. So sorry, I just saw this.

The change looks good to me. We need a way to distinguish binary messages and I think that's the correct way.

We could generalize things and use a map to choose the appropriate function based upon the contents of the header. This would replace the conditional (`if messageBytes[0] == 0...`). It'd be similar to GEPPETTO.GlobalHandler.js. Or maybe there's a nice RequireJS way to do this.

And maybe the header could be a JSON string. It'd require some extra logic to parse, but it provides a lot of flexibility.

I can look into this further and create a feature branch if you like. What do you think?


upgrading pip was the first thing i tried. it had the most recent version.
it also had the most recent version of setuptools. it was upgrading
testtools that fixed it.

On Tue, Jun 23, 2015 at 9:53 AM, clayg <notifications@github.com> wrote:

> I'm sure it's all related, but that gist is indicating a blow up trying to
> install swift - and the explosion looks exactly like you don't have the
> newest pip? This only just got merged yesterday - if you pulled before you
> may not have theses fixes - can you pull again?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/swiftstack/vagrant-swift-all-in-one/commit/a4c5dbc4f6ca0c32a1ed35e262255a9dd81f3e10#commitcomment-11821037>
> .
>


THANK YOU!

The only thing not documented (or updated in the docs) is those 4 headers:

- Mashape-Service-Token
- Mashape-Environment
- Mashape-Host-Override
- Mashape-Upstream-Protocol


which is also called ugly number problem


heh I know :D


`ROS_ERROR` should be used for situations that are atypical and require debugging. I suggest printing the received character using `ROS_DEBUG` for debug purposes only.


`printReceivedChar` is a more meaninful name. `printTerminal` seems to print a `terminal` object or something like that.


Better rename to `getChar` or `readChar`.


not only; one example is with the composer: when we get the content we parse the DOM. The DOM is authoritative.

You can compare with the recipients panel, it's the opposite: there is a JS model that we use to render the DOM. (not that we're perfect here either :) )


Useless line. Must be deleted.


Notification 将接收 Watching 代码库的动态，包括：

* Issues 以及它们的评论

* Pull Requests 及评论

* 对任何提交的评论


Better comment in the file the reason behind the `isnan` check and the magic number 25.


Actually it seems to return just a list of variables.


Thanks. I'll need time to review and test your changes.


@alex @reaperhulk Looks like this won't work with Elliptic Curve (unless I'm doing something wrong with the test) on either [openssl 0.9.8](https://travis-ci.org/pyca/cryptography/jobs/68092414#L405) or [1.0.1](https://travis-ci.org/pyca/cryptography/jobs/68092410#L296)


About sources/functions/usb/CMakeLists.txt, line 18:

Can definition of `val` variable be omitted? This variable is passed to `kmodule_is_exported` by name, so it should be defined and initialized by that function.


А вызываешь ты её через `rnafold.run_for_collection`?


in my opinion the previous wording is clearer at least by not using "it"


That looks good, thanks!


@feross `npm run test-quick` works great :+1:


Are you sure that you want me to remove this ? because it's more efficient than serialize and deserialize the object, we already have the values readable on the Unity side not like iOS or Android where we can only send strings from the plugins bridge.


I think the database should not be paired together with the serializable stratagies, but rather it should be used by default.


can you make a Pull request?


Thanks for doing that.There is a small issue with interconnections between vertices in the columns which as you know are not allowed in bipartite graphs.I did some debugging and through luck or judgment corrected the X column.The Y column seems to happen at i = 4 to 6,11 to 12,18 to 20 and 25 to 27 in the drawGraph function.I don't know how to get it to skip blocks of iteration,if that is a way of doing it.I have been working on a parallel method by using a BIG array.It works but it might cause amusement.
I will put it up later.


Неа. Я думала, что если я просто импортирую, он должен распознавать эти функции


also, tk.Button changed to Button as importing from tkk not tk for all buttons


Great! Why did I forget this…


Все. Теперь работает. Спасибо


I think it should only match on strings **starting** with this substring.
Also I wonder if there are legitimate usecases in which you want a pgp ascii armored block in you yaml and not want to have Varstack (try to) decrypt it.


Here's a picture of the bike I'm trying to modify:
![img_2602](https://cloud.githubusercontent.com/assets/5317905/8327516/ab08ad06-1a2e-11e5-88bf-b6c3713e993c.JPG)


Импортируется модуль. Так же, как ты работаешь с другими, стандартными модулями, необходимо указывать имя модуля, если ты не импортируешь что-то из модуля напрямую. Конструкция `as` просто создаёт сокращение для длинного имени модуля, т.е. ничего не меняет.


Sorry @iracooke, I think I didn't explain myself well, I didn't ask you to remove all the spaces before `/>`, just the ones after `</datatype>` at line 46 (which are still there).


Any specific reason for using `findAll` instead of `each`?


will do, and also add the operation to the readme file


I would suggest to just remove `filterByUserAuthorization


Последнее замечание не исправил. С сообщениями тоже не очень понятно все же. Пока так.


I guess a somewhat safe and common default would be to look into the users `$HOME/.gpg`.


I agree. it was surprising, and it could be a distribution packaging thing.  The version check here should use `finish_pdffile` if available, though, so I think we're covered.


This change isn't necessary, is it? Just curious. It shouldn't change anything AFAICT.


Please add those line to your `composer.json` file:

```json
"minimum-stability": "dev",
"prefer-stable": true,
```

This should solve your issue.


I've checked it's ok, the symlink is still on the repo, it ignore everything beneath the Core folder because of the leading slash, it will not remove the symlink.


Is there a requirement that you bind the result of the expression to `vPointClips`? It does not appear to be used later.


sha245 ? Did you mean sha256 ?


Great! I'll wait how #64 turns out and create a new release later.


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 1](https://bsr.ict.je-labs.com/viewLog.html?buildId=29573&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) is now running


Tested ok on Xubuntu Linux 14.04.


It may not be. I saw that `DefaultGroovyMethods#asImmutable()` is really just a call to `Collections.unmodifiableSet()`, so the underlying collection could be changed when adding/removing during evaluation.


Прохожу курс четно и второй раз. Решение по второму курсу откопал где-то на форуме курса ещё в декабре. Видимо кто-то еще раньше откопал его.
Разобрался с кодом и сохранил.

 Благодаря комментариям оно показалось мне не сложным.
В этот раз выложил из архива, не задумываясь. Не хотелось тратить время снова...
О том, что это "эталон" даже не думал.

Если сочтете нужным - то штрафуйте, всё справедливо.
Учусь не для баллов - для знаний.
И прошу прощения, впредь буду внимательнее…


@barais

The pull is fixed for the [new editor](http://kwe.braindead.fr) :+1:


I think we'd better left this ignore directive to avoid people developing on windows to push files there.


Menu?


I can't imagine Github Pages does not support subdirs, really? At least it works for the `update.rdf`: https://requestpolicycontinued.github.io/xpi/update-1.x-beta.rdf


Its a filtering constraint and it looks like `each` returns the original collection.

```groovy
> [1, 2, 3, 4].findAll { (it % 2) == 0 }
[2, 4]

> [1, 2, 3, 4].each { (it % 2) == 0 }
[1, 2, 3, 4]
```


The method name


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 1](https://bsr.ict.je-labs.com/viewLog.html?buildId=29573&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) outcome was **SUCCESS**
Summary: Running Build time: 00:05:54


Oops. ;-) Alright then. I thought maybe you changed it as part of the `equals` investigation.


I did, but it seemed safer to keep than revert =)


I think this way is more preferable in my understanding. I had trouble with getting the translations to work a couple months ago, because I needed to implement `getTranslationClass` on my custom Translation class. It is more helpful then when you get an undefined translation class error, than you get nothing.


This patch lost the actual i2c nodes.  It just has the pinctl snippet.


I completely misread it, sorry, I didn't notice it was part of an assignment. I should probably get some sleep...


好东西


index.php 内でsessionを使っている部分が見当たらなかったのですが、
session_start();
は必要ですか？


schedule_delete.php
もsessionを使っていないように見えるのですが、
session_start();　は必要ですか？


Schedule ではなく schedule
が正しいように見えますがどうでしょう？？


- The value could be anything, not only on or off.
- The expected answer was: `cmake -DUSE_SHARED=ON`
- There is no picture.


in the simd_vector class, there is a store function that take pointer just pass your elems array. If the size is correct it will store the content of the simd register into in, and then you just do the print.


How do we know that all the name properties are unique?


How about MIT instead of GPL?

The [original post](http://tympanus.net/codrops/2013/11/27/svg-icons-ftw/) doesn't specify a license.


This is actually a merge of the devel branch.


matlibplot packages also edited slightly as required


Thank you for your reply it works fine,

I think it would be great to release a new version (2.3.2). I was on 2.3.1 which was released a long time ago and I think there is good changes. In my case I needed commit 4ed8943cba259248d42f9ff1fc63649c1248cf76 so decided to change my media-bundle dependency to "dev-master"


Looks good, thanks.


Unexpected end of Input in Chrome


a good point, perhaps we should put an ID as a source column and enforce uniqueness at source?


※コード修正の必要は無いです。次回からの参考に。

PHPで各連想配列の初期化まで全て行う事はあまり無いので、
$default_data = array();
とだけしておいて、後で
$default_data["title"]='aaa';
とかすれば大丈夫です。

または、連想配列の初期化を行うのであれば
$default_data["title"] = "";
とかで宣言していたほうが見やすいです。

コードを見ていくと、結局
<?php print($default_data['title']);  ?>" />
とされているので、だったら $default_data["title"] = "";　と宣言していたほうが分かりやすいかなあと思います。


Ok, good point.


@giol I have merged these changes to master


`npm2nix --nodev nixpkgs/node-packages/packages.json nixpkgs/node-packages/default.nix`


It would be better if you did stuff like this in the base html file so that we can accidentally release stuff without breaking everything :D


Kirek, can you put the overlayfs specific stuff in a util-iso-overlayfs.sh lib and import in it util-iso?


Maybe I overlook something but this code is carried out always now even in case of success when we actually want the settings to be returned.


La tâche a été marquée à faire pour aujourd'hui.


break


La tâche a été crée


`dev-master` is for `2.4.x-dev`. I'll take a look for 2.3.2 release.

For now, you can use a flag like `^2.3.2` or `2.3.x-dev`.


Does socket_IO actually get used?

Also this change doesn't take affect when using tap


I think we need to do that. We also need to think what constitutes an ID and what constitutes a slug. If we rename a location, or it no longer provides Pension Wise guidance what is the application supposed to do?


NULL is missing in time.h.


Che, subi por accidente el LoginComponente que no anda ni ahi, no lo usen


Maybe it is a good idea though to make a nice exception for it? For when you override a model and you forget or don't know you need the Translation coupled with it too.

```
TranslationNotFoundException:
Cannot find the translation class of 'Acme\AppBundle\Model\Product', did you forget to create it? If so, please create the 'Acme\AppBundle\Model\ProductTranslation' class and try again.
```

Something like that.


Yep, my bad. I added in @b0573ca9c29e963a09acd39754405f9cf2384303 a loop (I wanted to do it without it, but apparently we can't...), and also a checking on the existence of the file (if the file has been deleted).


This is an interim measure to get the feature passing. More thought is needed to the wider nature of how we identity locations.


Naming things looks like a problem here - I would expect ```<something>Impl``` to extend ```<something>```, not the opposite way


I meant "goes missing". Not sure why I typed "goes on".


thanks Marco


加段注释吧，否则后面看的人肯定莫名其妙了。


1: M101 signature is missing
3: M102 unrecognized bullet/signature
5: M102 unrecognized bullet/signature
3: M190 line is too long (206 > 72)


Yes, that would work. I already did similar thing for `MeshView::setIndexRange()`.

It's weird that only `Renderbuffer::Renderbuffer(Renderbuffer&&)` produces that warning and not for example `Renderbuffer::operator=(Renderbuffer&&)` or any other "dllexported" inline function. I think the warning occurs only when given function is used. That would mean I may need to duplicate the `inline` keywords on both declarations and out-of-class definitions *everywhere*, which ... uhhhh. Ugly. Need to think that through.


3: M102 unrecognized bullet/signature
5: M102 unrecognized bullet/signature
3: M190 line is too long (206 > 72)


Can you remove this code if it is dead please?


@jgarciarbridgephase What was your reasoning for making this a post request?


Hi,

I'm having trouble using the ss.study.ca function. It works wonders  when using the example dataset, but when i try to use my own, everything works except that i can't see neither the histogram or the  density lines. I wonder if this has something to do with the scale in the vertical axis or the width setting of the histogram.

[1] " 16364.162 " " 17015.797 " " 15205.292 " " 15876.935 " " 15373.765 " " 16100.632 "
[7] " 16085.831 " " 16856.614 " " 17459.094 " " 16246.053 " " 16870.499 " " 16877.990 "
[13] " 17369.944 " " 14344.680 " " 15070.955 " " 14603.583 " " 15395.978 " " 16085.714 "
[19] " 16859.002 " " 16099.257 " " 16898.820 " " 16521.998 " " 17013.449 " " 15719.704 "
[25] " 16381.211 " " 15367.738 " " 16065.064 " " 15611.898 " " 16175.279 " " 16876.698 "
[31] " 14710.982 " " 15386.916 " " 14470.981 " " 15346.070 " " 15588.974 " " 16388.277 "
[37] " 16692.029 " " 17221.077 " " 16700.799 " " 17325.061 " " 18716.610 " " 18950.042 "
[43] " 15923.344 " " 16531.670 " " 15652.522 " " 16274.918 " " 15292.253 " " 16069.399 "
[49] " 16659.400 " " 17374.740 " " 16307.238 " " 17015.144 " " 16318.097 " " 16818.295 "

LSL = 8000
USL = 21000
Target = 14500

Here is the data i'm trying to analyse and the arguments i was passing to the function.

Any idea on what might be causing problems with the plots?


This should probably be
```| Uspecialized arg -> is_pure_clambda arg```


Why don’t you use `should(xhr.responseXML).be.not.Null`?


Same remark:
```Uspecialized arg -> is_simple_argument arg```


почему question_id? есть сущность Question?


Same remark:
```Lspecialize arg -> is_pure arg```


I will try to work with 2.4.x-dev in order to have required commit 4ed8943cba259248d42f9ff1fc63649c1248cf76 in my case. If not I will apply this commit locally waiting for new release.


I left it there because of the comment above, which seems to indicate that this code was needed in the past. I don't see where or why, but as a reminder, it's better to keep it.


ooh is that what the virtual mac is used for... if you wanna update the virtual_mac and see if that works? :P

I could only use PCap Switched mode, Bridged doesn't work for me so I couldn't test that.


この変更は正しいのですが, これだけだと `gcc`は C99をサポートしていないと判断されてしまうという問題があります(GCC 5.0より前は `-std=gnu89`がデフォルトのため).  そのため何らかのケアが必要となると思います.

- コンパイラが GCCの場合, `-std=c99` or `-std=gnu99`をつけて試す(古すぎるバージョンを考慮)
- コンパイラが GCCの場合は無条件で成功とみなす
- システムに `c99`(`/usr/bin/c99`)があればそのシステムは C99サポートとみなす(POSIXで定められる[規約](http://pubs.opengroup.org/onlinepubs/000095399/utilities/c99.html)). Linuxディストリビューションであれば `gcc -std=c99`を実行するようなシェルスクリプトになっている

(`Module::Build::XSUtil`も `needs_compiler_c99 => 1`が指定された場合 , GCC 5.0以前の場合では, `-std=c99`をコンパイルフラグとして追加する場合があります)


nog een analys**e**s :stuck_out_tongue_winking_eye:


Maybe join it with ', ' (with space) to improve readability?


Yes that surprised me as well. The images returned 404 errors. Maybe I should have allowed more time for the Github Pages build to run. This is strange.


JS в HTML это не правильно, нужно вынести в отдельные файлы


Есть rails-resources. Нужно выделить модели


minor, but please use a temp var instead of `m_groups.back()`


@archenemies The hwdb is not a configuration tool. It was created to provide metadata for non-introspectable devices. That means, if a device does not tell us exactly how it works (usually done through bus introspection), the hwdb can carry such information.

If you want configuration and custom modifications, you really should hook into the upper layer (hint: xkb). Sorry that this patch breaks your configuration, but it's not something we support (and the comment in the file even mentions this, as Peter pointed out).


Для prev/next отдельные методы


The picture was just a screen shot of my terminal results to elaborate more my method.


Нужна отдельная модель


Стиль битый. Это не читаемый кусок кода


Битое форматирование


Нужен API


THX.
Could you add it? Because now I'm on holiday and away from PC.
On 23 Jun 2015 8:16 pm, "Steindi01" <notifications@github.com> wrote:

> Screenshot for generating Gnome desktop entry with Android Studio
> [image: menu_001]
> <https://cloud.githubusercontent.com/assets/803596/8313659/808371a0-19e4-11e5-8d83-f43220d22648.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucaciavatta/android-studio-fedora/commit/810e4cbec1c3d63e3a664d0e0a99530ec7241305#commitcomment-11822672>
> .
>


Why null and not `false`?


$_SESSION['status'] == ”edit”
$_SESSION['status'] == ”new”
と２回セッションの値を比較するのであれば、先に

$status = $_SESSION['status'];
として、
if($status== ”edit”)

としたほうがスマートです。


This doesn't appear to be working on OSX with (currently) as up to date fresh gems as possible (in ruby-2.2.1).

https://gist.github.com/xntrik/6209b5917a721823a37a


Does this need to have facings?


This can be removed


no longer generates tests


no need for testFile


Maybe we shoudn't call it a callback, since it's promise based?


Now you can find the easyConfig documentation in config_example.php.


ref (fun _ -> failwith "undefined free_variables") would be cleaner


Yes, that's will be done for the final version


Right, because it refers to our package, not the original


I'm sorry for the inconvenience, GFlow.Source now is connectionless, then
it no longer holds a connected Sink list, it just send a changed signal and
Sink should catch up it or order to update its value.

This is because I found a race problem when try to connect Source to Sink
and back. When in one way work but not both, this makes Unit tests pass for
Source but not for Sink.

Please take a look in this and tell me if this is hard to use for GtkFlow.

May a reference to GtkFlow.NodeViewer should be hold in each GtkFlow.Node,
then it can iterate over all Nodes to check for recursive connections in
Sources, by using is_connected_to() method for each Sink found.

The last is just an idea, is up to you the final choice or make changes on
GFlow API.

May is better to avoid recursive connections in GFlow, maybe we need a
GFlow.NodeCollection to hold all Nodes and check for recursive connections,
this is easy to test using Unit Tests.

I think GFlow is in place now. Just I need to add tests for GFlow.Node to
find problems like the one on GFlow's
Source and Sink connection race problem.
El jun. 24, 2015 4:33 AM, "grindhold" <notifications@github.com> escribió:

> How is this supposed to work? We want to call is_recursive on all nodes
> that are connected to the current node's sources. the way it is implemented
> now, it will check the own sinks, call is_recursive on itself and
> ultimately *always* return true after the first recursion.
>
> The purpose of this function is to determine if there are any cycles in
> the graph which we don 't want.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grindhold/libgtkflow/commit/5e26865775649cf0a27e729860296ee3e51a9c90#commitcomment-11834313>
> .
>


So get rid of debian/docs completely?


Thanks, I was wondering a few days ago why this isn't already done.


why this empty line


Yes, will do for final package build - sure was handy though :-)


why those EOL spaces


do not commit this


there is a way to load this gem only during assets:precompile

```ruby
if ARGV.length > 0 and ARGV[0] == "assets:precompile"
  require 'sprockets-derailleur'
end
```


This is just MVV. The ranking heuristics was already proved to be better than MVV.

http://tests.stockfishchess.org/tests/view/55424d1c0ebc5938ea1962b4


Though this is a really nice effort, I feel that some of the files can be fetched from CDN. There is no need to include them in the repository.


By now you've tested every possibility, no? Time to move on to sth else.


Ah thanks


Looks like this should this have special handling for zero timestamps? Otherwise valid timestamps would get discarded in favour of unknown timestamps.


Hi! What about pull request?


Or even as an additional config value in the `varstack.yaml` file.


You're welcome. The factor of 2 would anyway be useless as the values are only compared.


this switch is wrongly indented. cases should be at the same level as switch


So it finally happened, the problem you can't solve ;)


Yeah that was a mistake, anyway even that is not generic, in windows it's different (x86_64 is not there) so we should probably just have LOCAL_NEURON_PATH


i don't see why checking for step should be in the else branch. because the if case checks the base, not the step. it will be more consistent to do it like this: if (base==64) { 8 } else { sz_n = step==2? 2: 4); }


Please cleanup confict markers


And take in account that my last changes on GtkFlow are just to make it
build against last GFlow's API changes, but not tested them.

I've added a gtkflow_test binary in order to test GtkFlow without to
install it before and use python.

May you want to take a look at it and add tests to it, now it just show a
label with results and need to add/remove GtkFlow.Node to tests different
scenarios.
El jun. 24, 2015 6:12 AM, "Daniel Espinosa" <esodan@gmail.com> escribió:

> I'm sorry for the inconvenience, GFlow.Source now is connectionless, then
> it no longer holds a connected Sink list, it just send a changed signal and
> Sink should catch up it or order to update its value.
>
> This is because I found a race problem when try to connect Source to Sink
> and back. When in one way work but not both, this makes Unit tests pass for
> Source but not for Sink.
>
> Please take a look in this and tell me if this is hard to use for GtkFlow.
>
> May a reference to GtkFlow.NodeViewer should be hold in each GtkFlow.Node,
> then it can iterate over all Nodes to check for recursive connections in
> Sources, by using is_connected_to() method for each Sink found.
>
> The last is just an idea, is up to you the final choice or make changes on
> GFlow API.
>
> May is better to avoid recursive connections in GFlow, maybe we need a
> GFlow.NodeCollection to hold all Nodes and check for recursive connections,
> this is easy to test using Unit Tests.
>
> I think GFlow is in place now. Just I need to add tests for GFlow.Node to
> find problems like the one on GFlow's
> Source and Sink connection race problem.
> El jun. 24, 2015 4:33 AM, "grindhold" <notifications@github.com> escribió:
>
>> How is this supposed to work? We want to call is_recursive on all nodes
>> that are connected to the current node's sources. the way it is implemented
>> now, it will check the own sinks, call is_recursive on itself and
>> ultimately *always* return true after the first recursion.
>>
>> The purpose of this function is to determine if there are any cycles in
>> the graph which we don 't want.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/grindhold/libgtkflow/commit/5e26865775649cf0a27e729860296ee3e51a9c90#commitcomment-11834313>
>> .
>>
>


it would be nice to write here ```initial version```


Does this compile? Condition is incomplete?


Why the change in message? Original seems more descriptive.


shall not be here, version is derived from parent pom


Please upgrade to newer version of uv-pom-dpu


Better synchronize access to the arrays, possibly including part of the averaging code inside the guard as well.


I bet this is inherited from `^Vehicle` or whatever in all the official mods. Ana maybe also the traits below. Please make sure you're not adding junk YAML by letting the upgrade rule do what it wants and get away with it.


dependency management in newer parent solves version problem


Both of you mix URI and URI reference.

URI: `http://example.org///foo`
URI path: `///foo`

URI references for such an URI based on `http://example.org`:
- `http://example.org///foo`
- `//example.org///foo`
- `.///foo` (relative-path reference must be used as an absolute path reference is not possible)


This commit applies better monitor `exit()` timing. While previous implementation never revealed a problem for http monitors, I started to see the problem with monitor configured for https access (`jx monitor stop` command was reporting a failure, because was unable to receive monitor's confirmation, as it was already closed).


Why isn't it in the same commit that the rest? it's a code move, show it ;)


Ah I see! Okay, I've made these changes and pushed them to github. You'll notice the changes in the new [`accelcat.py`](https://github.com/microstack-IoT/python3-microstacknode/blob/master/examples/accelcat.py) file. Here are the changes:

- `get_xyz()` is fixed. It returns values in G.
- I've added `get_xyz_ms2()` which returns values in m/s^2.
- I've not added `standby()` and `activate()` inside *settings* methods because this would be inefficient if you wanted to make several changes to the settings. I will make this clear in the documentation (when I get the time to fix those).


Check that the header exists, if not fail gracefully.
please no cammelcase... :)


Style:
* remove braces when only one line
* space before `)` to remove


@akarnokd hopefully this is what you mean and should fix it :)


braces to be removed


@alecive I'm not able to compile the project. Where is the definition of this function?


@NobodysNightmare why not remove/adapt this one as well?


There are also other edge cases:

URI `http://example.org/foo:bar`
URI references:
 - `./foo:bar` (`foo:bar` cannot be used because `foo` would be misinterpreted as scheme)
 - `/foo:bar`

See https://tools.ietf.org/html/rfc3986#section-4.1


Can you add some comments like we did last friday to explain why you're doing this? Make the code super easy to follow.


changes in init-qemu should go in this commit


That doesn't sound right to me.  `ArrayBuffer::Allocator::Free()` is called for internalized arraybuffers but that is not what we want here, its memory isn't heap-allocated.


Well, I guess that is the tradeoff for not defining the constructor in the class body. Shall I pullrequest? Or possibly add some more of these inline keywords in all the other classes?


It's happening boys


a space is needed before goto labels


I think authorization fn will need to be called with specific context and with req, and permission name as parameters.


didn't see change for "2. Needs to rename action 'Modify Traits' on the 'Replace Traits'"


@skrchnavy Just done. We will discuss with @tomas-knap about your other notes.


Just a suggestions here and feel free to ignore it if you don't like it. Every item from here to the next section starts with:
`This option is used when GENERATE_COT=1`.
What about having a line instead saying:
```
When GENERATE_COT=1 the following options are used:
 * SAVE_KEYS: ...
 * ROT_KEY: ....
and so on ...
```


@hoholee12 @DeicPro we can do this, make a simple wget / curl (put ifs to see if those bins exist xD - by default use curl, else use wget), now with this wget it can grab the pre-compiled bin from our fileserver or somewhere else and then mv it to /system/bin or xbin , respective of where it should be and then it should work! :D


there's a space there after ** which doesn't work for Sphinx.


why within parenthesis?


I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson, I wont forget that indentation lesson,


Will check what's the problem as soon as I'll setup a build tools on sunday, I hope.


space before code.


also space before code.


hello github


Copy-paste magic :speak_no_evil:


THX


Thank you for pointing that out. I wrapped the rake task in if Rails.env.development? - is that sufficient?

> On Jun 24, 2015, at 3:21 AM, Alejandro Andrés <notifications@github.com> wrote:
>
> this automation is cool :)
>
> pd: avoid using system on production systems, that is normally a huge security breach: rake rg "user; rm -rf /"
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/rmangino/bloccit/commit/cfd3f979696d5bd066335b38b1ed918e9572635d#commitcomment-11832217>.
>


Black magic :speak_no_evil:


My Pleasure  :smile:


R


jshint failed on this PR


i *think* this object should really power a prototype function called requiresAuthorization() that answers this question


The build fails because of cs… @Soullivaneuh, can you please advise ? Is @chervaliery supposed to fix all CS violations in this file ?


il manque le début du fichier...
cf https://github.com/benjaminkott/bootstrap_package/blob/master/Configuration/PageTS/Mod/WebLayout/BackendLayouts/default.txt


Bundled plugins has no compatibility info, so give false positive:
For example https://www.dokuwiki.org/plugin:testing


Is this used as some kind of memory barrier, don't see it referenced elsewhere?


OAuth uses a capital O and should name classes starting with capitals anyways.


This is now done as of commit 4cab15d04d576112a75ede00568179ac8eeba44b. Transitive reachability could get broken though in clademerge, when the largest alignments up to CLADE_MAX_MARKERS are chosen. Exemplars could get removed as well at that stage.


This does not work with current node-mysql, as it seems to support only "?" parameters. There is a way to add support for this, for example (using any-db-pool), but it's not there by default:

```javascript
var pool = db.createPool("mysql://localhost:3306/test", {
		onConnect : function (connection, done) {
			// Add support for $paramName parameters used by any-db-param to node-mysql adapter
			connection.config.queryFormat = function queryFormat (query, values) {
				if (!values) return query;
				return query.replace(/\$(\w+)/g, function (txt, key) {
					if (values.hasOwnProperty(key)) {
						return this.escape(values[key]);
					}
					return txt;
				}.bind(this));
			};
			done(connection);
		}
});
```


I care only for duplication of Coverrals. I'm fine with Scrutinizer (using myself).


Yes, I think so.


输入行和列，输出一个表格.


Why is this print here?


Also, i think that we should have a seperate develop branch for making changes. Also, the file structure needs to be changed.


@renujhamtani Although its fine but instead of creating explicit variables based on true/false can we use ternary operator inside jade file.
eg. {{CaseDetailsService.kase.negotiatedEntitlement.active === true? 'yes' : 'false'}}
This will remove all these conditions below.  :)


:ok:


Will this be printed every time the code is running?


So I couldn't figure out how to trigger a `GET` request from Angular with a  body. Since I'm passing in an array like this:

`['Medication 1', 'Medication 2', ..., 'Medication n']`


Also, when we print logs we do it with SoomlaUtils.LogDebug


Guess we need translations in ascension too ?


How about having startNewGame accept parameters?


needs a (declare list?) to work


shouldn't this be `scale(0)` ?


timePassed need to be set to false, if not, possible infinite loop will happen when timePassed but file still exists.


[13:44:22] <loooser@jabber.ru> это, колонку Exact share size приделали, а сортировку не поправили - сортирует по следующей колонке, а не на которую нажал (начиная с 4-ой колонки). Еле дошло как по IP отсортировать...


Of course the roles are additive now, but why is this only fixed for objects and assets, not for documents? Is there a special reason for this? Document permissions are fetched exactly like the asset and object permissions before these modifications. And these modifications also can break existing workspace permissions, I wrote more on this in the forum: https://www.pimcore.org/board/viewtopic.php?f=18&t=2370


fuufuufuu tabs


HttpClient  Java


Testing


I see how the force fetch loads the new items, but does it also remove the items that have been deleted on the server?


What fixed it for me was completely re-downloading the terrortown-folder from github.


Hehe. :-)


de données


ack


Just in case, check the following situations:
1) Chromosome names are ">chromosome 1, >chromosome 2, >chromosome 3" (i.e. with spaces). I saw  "name.split()[0] construction, it may cause troubles.
2) Chromosome names are very long. e.g. "gi|77404592|ref|NC_007488.1| Rhodobacter sphaeroides 2.4.1 plasmid B, complete sequence"  -- files with such long names may not be supported by OS, almost unreadable.  Better to limit this by something and use e.g. first 30 letters, some separator, last 30 letters.  or something like this.


a généré, de ressources


Does it save the resource or the resource folder? The commit message is consistent with the function name, but the code seems to save a resource. I am confused now. :)


connexion


pas sur pour le coup, logs avec "s" non ?


ok. I'll rewrite better this weekend at least I hope so


tab instead of spaces


de caractères


Cherry pick this to master.


Окаюшки, завтра поправлю... хех...


qu'est-ce qui est laissé ? Le paramètre ou la chaine ? si c'est le paramètre c'est "laissé" et si c'est la chaine, alors c'est bon :)


I fixed code here, thank you very much !


<nazy>libraire => bibliothèque</nazy>


Closed #7


Utilisé de cette manière *virgule*, le code utilisant...


Closed #6


actually you could write `def _end_of_set_index(string, start_index) -> int:`, then usually IDE's recognize that you intend to return an int. Thats thus way better than mentioning that in the documentation comment.


Ups, sorry!

Teaches me to use Slack usernames in commits... I have possibly succeeded in rebasing the correct username into commit history.


une commande


il manque des infos en debut de fichier

cf https://github.com/cdg46/t3_repute/blob/master/Configuration/PageTS/Mod/WebLayout/BackendLayouts/accueil-3.txt


Just checked with MSVC and it doesn't seem to complain in these cases, so this is probably MinGW-specific. Putting `inline` to function declaration duplicates information, clutters the declaration and is [considered bad practice](https://isocpp.org/wiki/faq/inline-functions#where-to-put-inline-keyword), so the best way would be to disable the annoying warning somehow :)

I'll do the fix (or, rather, ugly workaround) myself later.


Oh, shouldn't be AsInt64()


été exécutées


actually ` and string[closing_index] == ']'` isn;'t needed is it?


Use a separate function instead of copy and paste. Doh. (realloc(null, size) acts as malloc(size))


I was copying the format above that with pointPaths... Got it working and didn't want to change anything since I'm still not positive why it wasn't working.


éventuelles


Proof of concept. Problem is that arrays are passed by value while objects are passed by reference. Hence, the test failures.

Since this is a BC break, we might want to create a BaseToken and tokenize everything using classes.


How about a hybrid approach?  Use logs to get an approximation to the truth, and then use exact arithmetic to adjust by 1, if necessary.

Observe that log_3 243 is off by just 1 in the least significant binary digit of the result.


yes, I am using it, currently


I'd put a newline after each if block and while block, reads better IMO


you should specify whether this is the opening `[` or the char after that


``setAuthorizer = function(authorizer)`` ?


admin 的功能我们现在还没涉及，这里应该是is_customer? ???


Thanks! It never occurred to me that the "Products" might never exist - will need to add that to the unit tests too :).


Ok, but there wasn't any comment or doc string here. And for the other two functions, what said in doc string seems a bit controversial ( the TODO part of add_impact's doc string ).
Is get_impact means "get a set of all variables that are able to be impacted by the argument root", and add_impact does main part of get_impact's job?


Je  veux dire que c'est le contenu de la chaine qui est libre


Sure it's just debug logs that I forget to remove. I remove that.


What context? Harvester?


performance suggestion:

```
if closing_index < length:
    ...
else:
    return False
```


Looks good


How about `disableAuthorization`/`enableAuthorization`?


Plan is to change this from a string (for a single unit) to a map, eg array('in' => 1, 'em' => -1) to represent in/em. Where key is the unit and value represents an exponent

To do: investigate source of null for the unit.


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


En français tu traduis une **lib** par bibliothèque, j'ai toujours dit en entendu librairie mais c'est peut-être un mauvais néologisme :) Je corrige


ack


Why you want to use CDN when you can use builtin plugins of cordova for ionic app.Using CDN means doing the same thing with some random server in middle.Why would you want to read data from a random server if you can directly read from hardware.Consider what will happen to guys having slow internet connection :P. Can you be more specific on file structure ? Develop branch ? we don't have production branch bro :P.


Apart from those issues, I did a successful lab deployment. Awesome
work!

------ Originalnachricht ------
Von: "Iain Brighton" <notifications@github.com>
An: "nicholasdille/VirtualEngine--CitrixXenDesktop7"
<VirtualEngine--CitrixXenDesktop7@noreply.github.com>
Cc: "Nicholas Dille" <nicholas.dille@mailbox.org>
Gesendet: 24.06.2015 14:03:26
Betreff: Re: [VirtualEngine--CitrixXenDesktop7] Fixed check for
installed XD roles (4a16ae2)

>Thanks! It never occurred to me that the "Products" might never exist -
>will need to add that to the unit tests too :).
>
>—
>Reply to this email directly or view it on GitHub.
>


relax :P
----Origineel Bericht----
Van : notifications@github.com
Datum : 24/06/2015 13:49
Aan : ti1806-computer-graphics@noreply.github.com
Cc : tim.molenaar@planet.nl
Onderwerp : Re: [ti1806-computer-graphics] Added ambient light (023a611)
fuufuufuu tabs
—
Reply to this email directly or view it on GitHub.


@CarsomyrJ 예를 들면 이런 자잘한 거...


we can change line 43 to :
"if self.unbanned_response_codes:"


@jackpoz after this commit a issue appear with Demon Charge ( Warlock Charge ) doesn't work on a few bosses include rotface and festergut


At the very least, remove the whitespace after negation. It looks like a pipe character.


yes :)


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


This should be checking the collector format, not the output compression.


Give me also access on npm, please.


you're not reviewing the last commit of this file, these debug logs are already removed on the latest version.


Ja, maakt ook niet uit natuurlijk, maar wel frustrerend dat jullie dat niet goed (kunnen) instellen


3回目のpushのコメント


If each of these sub-menus is object of a struct, you can use sizeof to set the length,.


comment


why did you add these lines?


ack


alors c'est bon :)


I think that I can replace `empty?` with `any?` to avoid negation! :smile_cat:


Yep good catch


git push ssh://<username>@gerrit.omnirom.org:29418/android_bootable_recovery HEAD:refs/for/android-5.1


Useful to have them as Certificate instead of X509Certificate ?


Hmm, I don't get the same results with the test data and the info plugin on my local machine...


51-53 lines contains empty space


Not easily doable as we need the trust store password if the user changed it


@pferreir Should not be there.


Umm that's legacy code. Eventually most/all of it is going to be removed and rewritten.


Can't the expected output be the same as the provided input?

And please add newlines at end of file.


wildcards don't actually matter for this do they?


Send me a link to your profile on npm.


Even better and it _does_ work indeed. I just didn't find it in docs.


why do we need coala-ci with and without the `*.py` mime type?


This is a new pattern (false!== thatRouteMethod.authorize) that will make it just a little harder to read your code. Would prefer thatRouteMethod.authorize!==false unless there's a really compelling reason to be fancy


npmjs.com/~trysound


Please leave two blanklines between module elements.


+1 "slwoly"


Ah, never mind. That was because it had no `lastupdate` date. Now I can reproduce it locally.


Называй переменные осмысленно. n и s — не самые подходящие названия. Если это имя, то пусть будет name.
Используй значения по-умолчания при объявлении функций:
```
def __init__(self, name='Player1'):
    pass
```
Ты, как дизайнер класса, должен сделать его таким, что бы им было невозможно, либо очень тяжело пользоваться им не правильно. Представляй, что твои классы использует, или наследует другой разработчик. Он ведь не знает, что было у тебя в голове, когда ты проектировал класс. И он может забыть передать name. И тогда все упадет. Пиши классы так, что бы они принципиально не падали (за исключением случаев, когда это нужно, тогда генерируй соответствующие исключения)
Не брезгуй комментариями к классам. Почитай про docstring


wait, is TOP_SCREEN2 a macro that defined without `ifdef/ifndef`?
```C
#define TOP_SCREEN (u8*)(*(u32*)((uint32_t)0x080FFFC0 + 4 * (*(u32*)0x080FFFD8 & 1)))
#define TOP_SCREEN2 (u8*)(*(u32*)0x080FFFC4 )
#define BOT_SCREEN (u8*)(*(u32*)0x080FFFD4)
```
The if should be **always true** thus may cause some problems.
If these two buffer address conflict, i mean when it is launched only one of them could work, you can make a global variable and set it before drawing after launch. Also avoid duplication. isn't it?


related #4


ack


so we need it with py for windows and without for linux. THats why we have coala and coala.py.


on long term I hope we can get setuptools and create those binaries
automatically. Would be better.

2015-06-24 14:14 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> why do we need coala-ci with and without the *.py mime type?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/b95725e26a1a7806e58c121e7a8e9dc245c3b1b5#commitcomment-11836966>
> .
>


never noticed ;)


1. Mention that you return them as a tuple, not a list of whatever
2. Mention that you take None if there is no match


Okay, this double check for `username` and `user` is confusing.


done


and thank you for your support ;)


Shhh ;) typing slowly... too quickyl!


little bit confused here:
```
			if time.time() - t > 1:
				self.callEvent("timer.second", None)
				t = time.time()
			#self.callEvent("timer.tick", None)
			time.sleep(0.05)
```

why is `#self.callEvent("timer.tick", None)` commented out?


It’s not a double check. ;)


The problem is that this method _should_ be documented, it’s not obvious what is `username` and what `user`.


you're right, it should have stayed in UTF8. Anyway we intend to update all translations to utf-8


@ck86 ;-)


les deux sont équivalent au final. L'utilisation de `$this->superglobals[$superglobal]` permet de garder une ligne de code assez générique qui pourrait (pourra ?) être factorisée ensuite.


对的～这是当时Hulu的一个常见题。

我也忘记了优化方法了。23333

如果你还能找到当时的笔记，并让我瞄两眼，那么我就感激不尽了～


примечание:
В большинстве случаев, полный импорт — плохая идея. Импортируй из модулей конкретно то, что тебе нужно. В реально жизни, модули могут быть очень большими и нет смысла тянуть за собой тысячи строк кода, когда тебе нужен один класс на 30 строк.


so thats an acknowledgement?

2015-06-24 14:17 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> never noticed ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/b95725e26a1a7806e58c121e7a8e9dc245c3b1b5#commitcomment-11837043>
> .
>


This is not going to work anymore, the index of the port as you get it for the actual port will not necessarily match the description if array ports are involved. There's a function now to get the right description given any port index, from that description you can also get the size if it's an array port, and you need to consider that if it is an array port exported, you'll need one connection for each of its indices.


ack


Just kidding, I was wondering that such old library is still in production:)


@Runico can you provide gps coordinates (or even the command to tele at that location) of where you were and where the boss was ? does it work with warrior charge ?


Well, it was more clear before, you're just piling up the confusion and acting like you're not part of it. I'll document it once this is merged.


Pirate.exe merged with XPTable.dll for convenience


That's a poor fix. Instead, you should check if a given armor resource has overlay.


You have a typo in `provdining`


Haha why though...

Anyway: you use the word "Yields" every time, although the functions all "return" and "yield" is usually associated with iterators/generators?


@jackpoz okay let me check that, it works with warrior charge


The 'getRepoFailureAttribute()' method which returns a boolean should be named 'is...()' or 'has...()'


Все классы в Python должны быть потомками object.
```
class Foo(object):
    pass
```
Если ты не укажешь это — ничего страшного, компилятор сделает это за тебя. Но, если следовать хорошему тону — необходимо наследовать свои базовые классы от object


I would have to check where/why this helper is used.

At the very least it could in part use `workPackageJsonAutoCompletePath` and only append the `project_id`, but I have no clue.

The only reason I deleted the lines below was that they were recognized as unused by my IDE, which was fine because I wanted to get rid of the `workPackageAutoCompletePath` helper above.


I suggest writing change log in one work `changelog`


DevMenu was hidden with length=6 in that place


why don't you do `re.escape(sequence[1:])` and leave the circumflex condition out?


Oh, right. I forgot about this...


Хаха)) Малаца!


Created PR for the fix: #93


That was meant more colloquial than technical. I can change that.


Lol. why did we change this line?


Осмысленные названия переменных. «поубывалбы»


Thanks


Don't think that's right. I think we're breaking licence.


so why don't you do `regex += "["+sequence+"]"` which is shorter and better readable IMO because you have the string from left to right presented nicely


Since we will be attaching all the CSS and JS along with the app package, I thought it might increase the app size. But if that is not the case, then we can have them locally no issues. I think since we are completely porting it to android, we can safely delete the Windows based app. There is no reason to keep that folder. Also the other one is useless. Only keep the folder which you are developing.

As for GitHub branches, we can have one master and one develop. Once develop is working fine, we can push it to master. Master should only have working code ready for deployment.


Нет, такой не имеется
Исправлено


I'm not sure why we need it. Anyway, de don't specify error


Копипастил откуда-то? PEP8 призывает называть переменные через подчерк.
Почитай про пакеты (можно поставить через Pip, которые валидируют код на соответствие PEP8)


No, but how about "setAuthorizationStrategy", with the param being called "strategy", and it being stored as this.authorizationStrategy=strategy


you could put these first few chunks into an own commit easily/


Yes, you should at least keep original author as well


ok :+1:


In 3.4.11, I have this failure:

```
======================================================================
FAIL: test_two_newline (control_test.TestControl)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/buildd/exabgp-3.4.11/qa/tests/control_test.py", line 72, in test_two_newline
    self.validate('-\nx\n','x')
  File "/tmp/buildd/exabgp-3.4.11/qa/tests/control_test.py", line 59, in validate
    self.assertEqual(string, check)
AssertionError: '' != 'x'

----------------------------------------------------------------------
```

I was wondering why you commented the test. Should we also skip it in 3.4.11?


Would be really nice if we could come up with a better name for this.


this indentation is making me mad, can't you use a filter here?


很不错啊～

虽然这题可以用`Maximal Rectangle`的解法来解，但显然你没有拘泥于过去的题目，而新想了一个更加简洁的方法。所以，很不错。


this is also a nice candidate for an own commit


Here you'll need to use the new function to get the port description based on idx, later on for the connection, if the port is an array, you need to pass the correct subindex to the function, which will be idx - port_desc->base_port_idx


please use parenthesis insdtead of backslash line continuation to make @Makman2 happy :)


this is out of scope


didn't you write the `collect_files` function above for just this?


out of scope


@cvrebert Why the 3 meta tags **must** come first in the head?


this prama insertion doesn't belong in this commit but in an earlier one


You work a lot with the length of these dictionaries, I don't know whether it adds any overheat to calculate it ever time?. You might as well store it I guess?

Also, line 68 can be simplified the same way you did in another commit here, with `if 0 in (X ,Y)`

this is the only function that scrutinizer rates D


Rethrow if type==Entity (or at least log an error), also `propagateIfFatal`.


ack


wat was de reden dat er eerst `**kwargs` waren en waarom werden deze verwijdert?


Nice try, du Noob! Genitiv ins Wasser, weils Dativ ist!


typo in commit message "thatare"


does `keywords` get interned or is it created fresh each invocation of the completer? I suspect the latter, in which case moving to be a private field will help reduce churn.


出于高度方面的考虑（不太希望插件的高度比概览界面还高）：
```
#quest-info select {
  height: 30px;
  padding: 0px;
}
```
这一条css还是加上比较好
任务的原始说明考虑改成tooltip的形式


Unfortunately not since the method is returning the deleted hash and not the modified array.


why remove the docstring?


i really like the idea to support a typed argument but don't really know how you would implement it..
and also i think is not possibile to prevent a malformed IP at compile time.
Can you show me some code? thanks!


history, wrong change indeed.


hmm... i think we enable authorization by default, so disableAuthorization is only the function you'd want to write. I chatted with kristof, and got it approved as part of harvester 2.0 dsl.

The requiresAuthorization() function that I was suggesting earlier would really let the caller know whether or not they should run authorization for this function handler ->it returns true or false. The idea was to rely on the function rather than the underlying variable.


Invited Christian (iENC Import) and Padde (CEVNI preset) to join this talk.

@malcolmh Any suggestions?


My bad. Didn't test it well. I guess unit testing exists for a reason a this is a completely demonstration of its usefulness. Hope it works now.


Built Plugin does not work!
The clean up is mainly to work through the code and understand it, making it nicer to browse.


hai


I can not tell you why I done it - surely as I change too much things and do not test things enough :speak_no_evil:


I'm pretty interested in your weird algorithm and whether or not you can test in any way, that they are better. Unless you explain any of that, this commit looks rather weird.

Anyways, if you just want these ratings, it is **ack**ed


The vector `ps_ptr` is already an attribute of the test class. The mistake I made here is that I (re)created another (local) variable `ps_ptr`. I'll change to code to correct these mistake and include the other changes of your PR into my PR. Thanks for the remark.


了解しました


you can probably just squash this commit and the previous one?


Would it kill you to give some proper credits or a link to the commit?
https://github.com/CandyDevices/device_samsung_quincy-common/commit/cabad80296ca2cd573f10971bf2f7b9419c48d7a


R+


this is not intended I guess


Filters are used for this, converters just convert.


so the algorithm checks the test dataset and gets the *minimum difference for non clones* and the *maximum difference for clones* and tries to maximize that range. The bigger that range is the better the algorithm. If the range is negative clone detection will obvioously not work.


ah yeah

2015-06-24 14:42 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> you can probably just squash this commit and the previous one?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/36fb8ae988bcb963112a684e7c24de48cafc55a9#commitcomment-11837510>
> .
>


1: M101 signature is missing
1: M110 missing component name


typos:
fub ->  buf
a -> as


Added in https://github.com/rsiebert/TVHClient/commit/ae44a74cb894d7337ce5cb335492ba8a04eb80d4


For the non-clones, you link to german wikipedia articles and at least one example has german comments and german variable names.

It might be better to find an english reference or at least refactor the variable names?


Hi, I'm trying to use the QRduino for make a QR code and show it in a 3,2 touch panel display, but I can´t understand about the files for do it, can you upload an example please? or tutorial about use it.

Thanks you.


No honestly I was sharing to know from you, why I've copied TestTipOperations from the test above, but is not working with this error:

    ===============================================================================
    [ERROR]
    Traceback (most recent call last):
      File "/home/qq/.virtualenvs/backend/local/lib/python2.7/site-packages/twisted/internet/defer.py", line 1099, in _inlineCallbacks
        result = g.send(result)
      File "/home/qq/Dev/GlobaLeaks/backend/globaleaks/tests/handlers/test_receiver.py", line 91, in setUp
        yield helpers.TestHandlerWithPopulatedDB.setUp(self)
    exceptions.TypeError: unbound method setUp() must be called with TestHandlerWithPopulatedDB instance as first argument (got TestTipsOperations instance instead)

    globaleaks.tests.handlers.test_receiver.TestTipsOperations.test_put_delete
    -------------------------------------------------------------------------------


ack


確認ズム


possible cause of #1012


Making Sources connectionless would be a great idea if the sole purpose was to pass on data. But in the end I want programmers to be able to traverse the graph in order to work with the structure of connections, not only with the flowing data itself. The simplest (or in performance terms cheapest) way to do that is keeping references to neighbored Objects. Simply following pointers is cheaper than iterating over a list of references multiple times.
Checking for recursive connections is functionality that belongs into GFlow because there is no necessity to be UI-aware.
And if something doesn't work, simply removing functionality can't be the solution. If the tests do not work as expected, it indicates that the function does something wrong, not, that it is obsolete.

Could you please describe the race problem you found in order for me to evaluate how severe the problem is?


I think the bug that this solves is one that's really rare in experienced developers, while the awkward syntax is a maintenance cost to every eye that reads the code


Yes, sure!


Unhandled rejection TypeError: string is not a function
    at testResource (/usr/local/lib/node_modules/cdnwhaaat/node_modules/shisha/src/index.js:59:53)
    at Object.shisha.smoke (/usr/local/lib/node_modules/cdnwhaaat/node_modules/shisha/src/index.js:70:11)
    at smoke (/usr/local/lib/node_modules/cdnwhaaat/index.js:31:10)
    at /usr/local/lib/node_modules/cdnwhaaat/index.js:96:5
    at tryCatcher (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/util.js:24:31)
    at Promise._settlePromiseFromHandler (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/promise.js:454:31)
    at Promise._settlePromiseAt (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/promise.js:530:18)
    at Promise._settlePromises (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/promise.js:646:14)
    at Async._drainQueue (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/async.js:182:16)
    at Async._drainQueues (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/async.js:192:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/usr/local/lib/node_modules/cdnwhaaat/node_modules/bluebird/js/main/async.js:15:14)
    at processImmediate [as _immediateCallback] (timers.js:358:17)


@smalyshev the ruby-saml library should be updated.


This "if (u)" check can go now, since manager_load_unit() will always initialize u if it suceeds.


Don't do that. You should be running it in a virtualenv with python 3 as default. If you're running it outside a virtualenv, just use `python3 main.py`.


Nice!


Just trying to copy the way vanilla does it, which is ItemArmor.getColor(stack) != -1. If the vanilla way is bad, then that's another issue.


CSS and JS won't add to the final build apk .Ionic can be ported to android , IOS and windows(in development).You can go ahead with deleting of other folders.No problem from my side.
Development and master branches is a good idea and will be more prominent in backend part of this project.


@mkurz see twbs/bootstrap#15601 which this statement fixed


@jackpoz it doesn't work either for warrior's charge
GPS: X: 4277.612305 Y: 3158.892822 Z: 360.385529 Orientation: 3.145517
Ss: http://i61.tinypic.com/1222r8n.jpg