They are feature files (number of students x features) that we were using
to test the regression experiments. Should I remove them?

On Sun, Nov 15, 2015 at 12:58 PM, Alexander Lerch <notifications@github.com>
wrote:

> I see that you committed .mat files. What are these? In general, I would
> hesitate to commit data except the ground truth...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GTCMT/FBA2013/commit/f60c6ec6f5130f047baa4e85dd17489dad2e5d7b#commitcomment-14403053>
> .
>


"refer" is a terrible word here. The E docs are pretty terrible on this front, too; they've aged terribly.

I changed this to use "contain" and cleaned up the language.


Hmm, are these IP ports? If so, there should be additional checks in place, to ensure the values are > 0 and <= 65535.


Also `tmp`.


Is this an IP port? if so, the type should be uint16_t really.


This document is all about satiating PLT folks who want to see this document exist. It's also for writing down niggly bits like order of execution in more complex situations. @washort and I continually run into being unsure of order of execution.


The functional syntax is right, and I like it.

Is `Restaurant[pid]` a function? And do we want to assign to `this.restaurants` or `this.restaurant_detailed`?


I got time right now to fix it. I'll just revert these changes and apply whatever needs to stay changed.


what is target? thanks


I think `toAvoid(tp.underlying)` might catch more things here.


I see, so you're just checking out this git repo and then using it. That makes sense, but it's not how I have been using it. I've been including statebus as a separate package.


Hmm, is this really the right API? we pass two BE 16bit values as a single NE 32bit value via netlink? Really?


`IBond.Order.DOUBLE.numeric()`


it this an IP port? If so, this should be uint16_t, no?


This needs `connections[socket.id]` too.


please elaborate on what kind of "port" is meant here. Is this an UDP/IP port? If so, say so.


Hm. In one sense, yes; we can have a priori definitions of ``Char`` and ``Int`` which do not overlap, and that assumption is core to intuitive use of these primitives.

On the other hand, no; it's legal to construct ``Any[Char, Int]`` and use it as a guard.

On the gripping hand, we can view ``Char`` and ``Int`` as subsets of all Monte objects, and then axiomize that they are distinct.


I think it sounds better if you replace "DestinationPort" with "the destination port"...


so you got the original files recovered and back onto git
Sorry about that again >.<

On Sun, Nov 15, 2015 at 11:12 AM, wwcao <notifications@github.com> wrote:

> I just pushed my original files and the recipe.php that you modified. If
> you have changed other than recipe.php, my suggection is that you can
> create a new branch before pulling original/master so that you have a old
> master on a branch and a recovered master.
>
> Git Command you may need:
> git branch
> show branchs
> git checkout -b <branch name>
> create a new branch <branch name> and switch to <branch name>
> git branch -D <branch name>
> force delete merged branch, you will get a warning when you use -d
> by the way, if you don't push the branch in your local machine, there will
> be no such branch in the remote.
>
> git push
> if there is a warning message, you need to use git pull to update
> first; we should AVOID to use *git push -f* again.
>
>
> Weipeng Cao
>
>
>
> On Sat, Nov 14, 2015 at 10:11 PM, Weipeng Cao <Weipeng.Cao@colorado.edu>
> wrote:
>
> > No worry it can be recovered.
> >
> >
> > On Saturday, November 14, 2015, Rikka12580 <notifications@github.com>
> > wrote:
> >
> >> Ugg. I should have them on my laptop. I'll worry about it tomorrow.
> >> On Nov 14, 2015 8:16 PM, "wwcao" <notifications@github.com> wrote:
> >>
> >> > You pushed the old files, we need to recover the old version.
> >> >
> >> > —
> >> > Reply to this email directly or view it on GitHub
> >> > <
> >>
> https://github.com/Rikka12580/group8/commit/adb75e403bca4371777b6ce9a295bdb6df33208d#commitcomment-14400226
> >> >
> >> > .
> >> >
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/Rikka12580/group8/commit/adb75e403bca4371777b6ce9a295bdb6df33208d#commitcomment-14400447
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rikka12580/group8/commit/adb75e403bca4371777b6ce9a295bdb6df33208d#commitcomment-14403112>
> .
>


Shouldn't this be as simple as
```
if (first == null || second == null) return false;
if (first == Unknown || second == Unknown) return false;
return first.compare(second) < 0;
```

and like wise for higher order?


If the config has no system section, there definitely is something wrong with the config.xml... The additional check is harmless, but I do think the user should have more issues when he experiences this one (no users, etc).


BTW, I think we should either unify the concept of a "part" of a type (as defined by `existsPart`/`forallParts`/...) and a "named part" of a type (as defined by `namedParts`/`namedPartsWith`/...) or give them completely different them to avoid confusing them. For example, `namedParts` will look at the underlying type of a `ThisType`.


Please don't include _LAST in the documentation, that is for internal use only...


Awesome


Hi François. I think this should be at the end of the previous line, no?
I was installing this package today and Emacs asked me about what coding system to use, so apparently it didn't see this spec.


Why call the bal method and then convert the string? The bal method just does what this method does but doesn't format it


Each CharExpr expression node produces an object which passes ``Char``. Maybe I should add "Expr" and "Patt" to the end of every subsection?


```java
    public static boolean isLowerOrder(IBond.Order first, IBond.Order second) {
        if (first == null || first == Order.UNSET || second == null || second == Order.UNSET)
            return false; // should probably be assertion error!
        return first.compareTo(second) < 0;
    }
```


That did not seem to be the case. He was just not able to configure the OPT1 device. But maybe I missed the correct line?!


Is there any reason to remove a configuration option from MP2?


Because I wasn't thinking, I will fix that to be other way around.


I get a warnig that 'runHtmlT' is now exported twice.


I thought we agreed not to break master?


A "touch OpenMBV.i" solved the problem.


I guess I need to upgrade to VS2015 and start using Jake. I've been using only the CMake files and building for VS2013.


No it was the exact line. Let's see what he reports back. The timing in bug / fix seems to align in this case.


Just saw the later revert… nvm then.


8ebf733e346242d24c269c36d1c68cb9deb30b17 :-)


Switch here
```java
     switch (order) {
            case SINGLE:
                return Order.DOUBLE;
            case DOUBLE:
                return Order.TRIPLE;
            case TRIPLE:
                return Order.TRIPLE;
            case QUADRUPLE:
                return Order.QUINTUPLE;
            case QUINTUPLE:
                return Order.SEXTUPLE;
            default:
                return order;
        }
```


switch here also


Strange. Your right, let's wait for feedback.


switch here too


@srob650 Did some test, looked pretty good. Will continue to test!


In sass, the convention is to prefix files with an underscore to mark them as imports instead of putting them in an `imports/` folder. Could the `isRoot` function be used for styles that are originally compiled by a compiler that inherits from `MultiFileCachingCompiler` ?


Чого "An"? Хотіли показати знання англійської, а в результаті - ні англійської, ні нормальної назви. :-(
Чого "Ex**ta**ctor"?


Thanks for this library, it's working great! :bowtie:


Player 2 movement for testing


Done.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.784](http://teamcity.monogame.net/viewLog.html?buildId=11361&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


We do not *need* it, but it is commonly done to keep the travis.yml manageable and maintainable (without folding too much into bash scripts).  Travis itself will not support different behaviors with different branches.

See this for an exploration:

https://github.com/travis-ci/travis-ci/issues/879


Awesome, happy it's working well for you!


Fixed. Also specified that ``Binding``, in particular, is used to enforce the type of bindings.


Interesting. Thank you!


An від Annotation


Git commits -s


a little explanation why wäre skipping these would be great.


This is really smart. Good call.


@cedrictools thx, committed too quickly. will update now


(Strange, Brady's message disappeared and now it is back!)

The second fix is wrong.

For timezones with a negative UTC offset, new Date("2015-11-22") gives this output...

Sat Nov 21 2015 19:00:00 GMT-0500 (Eastern Standard Time)

So using getUTCDate() on date object created in line 1008 is necessary.

Probably the only date method that should have been changed is in line 1017...

~~~~~~
if (tmp.getDate() - d.getUTCDate() < 7) {
~~~~~~

as tmp was created with different date input format and tmp.getUTCDate() will give a different answer depending on positive or negative UTC offset.

(Deleting this branch)


The build need to look like this: https://ideone.com/owy3fT
 just copy and paste!


1) надо реализацию функций из arithmetic.h перенеcти в arithmetic.сpp
2) тесты выглядят очень странно. can_check_the_brackets надо разбить на 4, can_do_partition не должен содержать никаких алгоритмов.
3) (продолжение предыдущего) думаю, функция Partition должна что-то возвращать (type1 и type2?)


I'm fine with unifying this.


Does `ParallelCommandsGen` itself give any guarantees whatsoever?


YearDescription_Results (macumber) - Win64-Windows-7-VisualStudio-12: OK (2337 of 2371 tests passed)

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


Ball of Duty Server/Domain/Entities/Bullet.cs

Why?


Ah, I see I missed this one too. Yeah I think you're right that we probably want the full connection object.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 97](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=484&buildTypeId=XbimEssentials_XbimEssentials) is now running


eyup o7


Submitting pull request with fixes would be the right course of action IMO, provided you've tested that the changed module code does work. I haven't touched this code, or my HMC5883, in a year and a half...


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 97](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=484&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:18


Making this be just `std::cout << st` would also work, maybe a little bit more wonkily


Windows (via Python) can take either way ("C:\" or "C:/"). But os.path.join use the native format for each system ("/" for Unix and "\" for Windows).

That means os.path.join will mess things up really badly when mixing the anchor in Unix with anchor and filenames in Windows and vice-versa.

Remember that the master send the filenames to the clients.


concurrent Modifikation exception?


Ball of Duty Server/Utility/LightEvent.cs

why this instead of Timer?


Since the statement in the try-clause will either return or raise an exception, this else clause is imho unreachable, @lukasjuhrich .


Hello, I wonder, why prepareQuery is going before the preMiddleware? Seems in this case there is no way to control query parameters on Node.js layer? Only by touching private req._ermQueryOptions inside of preMiddleware?


iirc, we want to send the incrementals from the peon to the osd. `OSDMonitor::send_incremental(..., session, ...)` will send said incrementals to that one session. But the session we're passing here is the op's session, which ought to be the monitor that sent this routed request. I think what you wanted was `rr->session` instead.


Git commits -s


having 2 physics update right after eachother where one is updating position, the other is updating collision could be a problem in the end if we want something not to change position if it collides for example.


The other problem is that the connection objects don't have a key. I tried just adding a 'key' property to connection objects on creation, but that doesn't solve the problem.

My guess is that I'll need to declare an on_fetch function for '/connection/:id'. Also, if I am using the user_bus_func, will I have to add the '/connections' method as well? The code right now does not fall back to the master bus if a new userb is created.


AFAIK it should have the same guarantees as CommandsGen, i.e. pre-conditions are met (checked by invoking apply).


Could it be that this is not in master but in v1.5.x?


Of course the branches are checked individually starting with the post state that came out of the last command in the prefix.


Wouldn't `center = Vec3(0.0f);` first create a new vector, then copy its content to `center`? Or does this syntax execute the Vec3 constructor on center directly?


Oh, they don't have a key... Ok, maybe I should think more deeply about this.


I tried to make IntegrationControllerManager work with the new system, without implementing grouping.
It seems to mostly work good, but 20 tests don't pass in the IntegrationProcessSuite.


Umumi ve daha abstrakt meseleler:
- quiz.js object oriented dizayn edilse ela olar.
- Debugging toollar (console.log, uncommented codes, etc) kodun, en azindan production kodun icinde olmamalidir.
- Coding convention problemleri cox az da olsa var (double vs single quote, indentation, etc)


We should not do timeout events like we are doing right now (udp), we should do it with TCP now that we have a TCP connection running. I know this wasnt what this commit was about.


Mmm... good points... so much for no radical changes :) I'm on it...


I discovered this week that `:+=` and co are **not** calling out to `enqueue` (at least in scala 2.10), so this is not as efficient as you'd think it is (we do the same thing in the classfile indexer, which I'll address)


1. setirdeki function ucun contexte gore mence function deyil, object daha uygundur. Hetta, ayri bir faylda json objectleri kimi olsa lap yaxsi olar. Bu sual ve cavablarin hamisi quiz classindan musteqil bir sekilde hemin faylda durmalidir


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.784](http://teamcity.monogame.net/viewLog.html?buildId=11361&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:24:28


I'd leave the Linux portion generic, since it's not really possible to enumerate all the distros.  Something like, "Please install FUSE development packages, for instance libfuse-dev or fuse-devel."  That's a hint for the big two, deb and rpm, but leaves the message open for others.

I don't know what BSD requires, but similarly I'll bet there's no one answer to cover all of FreeBSD, OpenBSD, DragonFlyBSD, etc.


Rubricist warning :) :

Colophon still says "Textus et cantus officii divini secundum Antiphonale Monasticum, Solesmis 1933."
The Compline included in this commit is not monastic. It's secular. Monastic Compline has no Nunc dimittis and there may be some more differences, too. See the abovementioned Antiphonale, pp. 167ff.


Should only call these methods if characters health and score was updated, which isnt likely to be true every 0.3 second.


Also, in monastic Compline the psalms are chanted in directum, not with an antiphon.


best practise olaraq, bu cur variable declerationlar tek bir `var` ile en ustde yazilmalidir. Meselen:
```javascript
var pTag = document.getElementsByTagName('p')[0],
    currentIndex = 0, numOfAnswered = 0;
```


`mainDiv` burda da, bir sonraki function'da da isledilib. Yuxaridakilar kimi global declare edilse adina uygun olar mence.


bad console print.


should add this to PendingReleaseNotes.


niye `mainDiv.innerHTML('');` yox?


As said before, this way of unregistering could easily lead to errors, you'd have to know what observations the character is registered to be sure to unregister all. There should be a Unregister all method in observable.


Grande @blackleg!
@davidromani como lo ves ahora, se puede subir a produccion?


Same as before, should be a unregister all. Also unregister is one word.


eyni sekilde function scope ucun de bu kecerlidir: https://github.com/Azer28/Simple-Javascript-quiz/commit/c794af945ac4626c8847a8023a9483a3911b7cf1#commitcomment-14403500


I don't think there are any implementations that change wide character encoding at runtime.  I don't remember anything in the C11 spec specifically disallowing it, but it would be weird…  OTOH there are platforms which mix endianness at runtime, so maybe.

I considered the idea of just having this be a macro (it actually was in an earlier version), but opted against it since a function is much more flexible and I was thinking about the possibility of adding some runtime detection if the standard macros aren't defined.  A macro would probably work if we added detection to the build system, but I'm not sure there is a way to do it without breaking cross-compilation.  Besides, I'm betting the CC does a pretty good job of optimizing this away (especially with LTO enabled, which we do for release builds).  Maybe an "inline" would be helpful, but I doubt it's necessary.

One interesting thing here is that `__STDC_UTF_16/32__` don't actually say anything about `wchar_t`; they provide information about `char16_t` and `char32_t`.  I have trouble imagining a situation where `wchar_t` isn't the same as one of those, but it is possible.

We should probably also be distinguishing between little-endian and big-endian UTF-16/32 encodings.  Maybe something like…

```C
#if defined(_WIN32)
/* We don't need to do anything here; just assume little-endian */
#elif defined(__APPLE__)
#  include <libkern/OSByteOrder.h>
#elif defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(__bsdi__) || defined(__DragonFly__)
#  include <sys/endian.h>
#else
#  include <endian.h>
#endif

#if defined(__GNUC__)
__attribute__((__const__))
#endif
const char*
squash_charset_get_wide (void) {
#if defined(_WIN32)
  /* Windows always encodes wchar_t as UTF-16LE. */
  return "UTF-16LE";
#elif (WCHAR_MAX == INT32_MAX) && defined(__STDC_UTF_32__)
  /* AFAIK this should take care of most modern Linux and BSD
     distributions.

     This is basically known to be buggy in theory, but in practice I
     think it will work.  Problems:

     * __STDC_UTF_*__ technically refers to char16_t and char32_t; it
       says nothing about wchar_t.  I have no idea why wchar_t
       wouldn't be the same as one of those, but it's possible....
     * We assume that the data is native-endian encoded.
     * Someone could use -fwide-exec-charset to request a
       non-UTF-16/32 charset.  But really, who does that?

     If anyone encounters issues with it in practice, please let us
     know. */
#  if __BYTE_ORDER == __LITTLE_ENDIAN
  return "UTF-32LE";
#  else
  return "UTF-32BE";
#  endif
#elif (WCHAR_MAX == INT16_MAX) && defined(__STDC_UTF_16__)
  /* I'm not aware of any platforms which would trigger this (Windows
     seems to be the only one using UTF-16, and they don't define
     __STDC_UTF_16__.  That said, the same issues as the 4-byte
     case above apply. */
#  if __BYTE_ORDER == __LITTLE_ENDIAN
  return "UTF-16LE";
#  else
  return "UTF-16BE";
#  endif
#elif (WCHAR_MAX == INT32_MAX) && defined(__APPLE__)
  /* AFAIK OS X is actually always little-endian, but they do allow
     for other values in their API... */
#  if __BYTE_ORDER == __LITTLE_ENDIAN
  return "UTF-32LE";
#  else
  return "UTF-32BE";
#  endif
#else
  /* Fall back on runtime detection.  Currently we only know about
     UTF-16 and UTF-32, but hopefully if anyone has a platform with
     another encoding they'll let us know.  I'll add an error message
     somewhere in the library if the encoding is unknown with a link
     to the issue tracker. */
  wchar_t test_char = L'ѡ';
  uint8_t* alias = (uint8_t*) &test_char;

  switch (sizeof(wchar_t)) {
    case 2:
      if (memcmp (alias, (uint8_t[]) { 0x61, 0x04 }, 2) == 0)
	return "UTF-16LE";
      else if (memcmp (alias, (uint8_t[]) { 0x04, 0x61 }, 2) == 0)
	return "UTF-16BE";
      break;
    case 4:
      if (memcmp (alias, (uint8_t[]) { 0x61, 0x04, 0x00, 0x00 }, 4) == 0)
	return "UTF-32LE";
      else if (memcmp (alias, (uint8_t[]) { 0x00, 0x00, 0x04, 0x61 }, 4) == 0)
	return "UTF-32BE";
      break;
  }

  return "Unknown";
#endif
}
```


Wow, yes ... brain lapsed on that one. Thanks!


comment


Thanks for this fix. I was thinking something like that yesterday while browsing the Coverity defects, but was too uncertain about exactly how to write it. I'm glad to see that my thinking was not too far off. :smirk_cat:


NPCs need to be added for multiplayer.


We can always store a dict in memory for as long as the program is running. When testing new functionality it is a bit overkill to actually save a file. Of course, in the production code we will need to dump the state to file.


@GeorgeTaveras1231 add your own ssh key :)


If collection is None, shouldn't we raise a 404 too?

Also, in that case, might want to bring the following line outside the try/except.


alphabetize


2-space indent


The way I read it, and correct me if my interpretation is wrong, that this says "We've half-heartedly tried to do this before, but haven't really put the right effort into it, and so those previous efforts failed. We're going to do this right, and actually succeed this time". This is a message I agree with 1000%. I can see how not knowing the previous history could leave the previous wording feeling confusing though.

Perhaps something like "Increasing diversity seeks to actively and effectively focus on onboarding people from diverse backgrounds to the Node.js project and maintain their participation". I don't love it, it doesn't feel as pointed as the previous wording to me. Maybe we can spin on this wording some more, as I think this message is really important.


I think itemprop="description" is used for page snapshots in social media shares. So, can we do something like this:

```
    <span class="protractor-test-collection-header">
      Collection
      <span class="oppia-navbar-breadcrumb-separator"></span>
      <span itemprop="description">{{collection_title}}</span>
    </span>
```


The .suo file is a "solution user options" file and should not be checked in to github.


I'm not a big fan of the name `recurseInImage`. You should rarely need your method to expose the fact that its using recursion (in fact Im tempted to say you never will, but Im sure there are at least a couple use cases). I mean, you dont call your methods `iterateOverImage`.

Instead, what is this method //doing//. Its doing something, and it happens to need the result from operating on a subproblem to figure it out.

I like to think of recursion as "Can I break down this big problem into the same thing on a smaller set". Its like how you solve tree problems. "I need to figure out X for this tree. Well if I figure out X for the left, and figure out X for the right, its really easy to figure out X for my node - so lets just recursively call myself until `left` and `right` dont exist"


Call this "URL parameter values", or "Parameter values passed into interpolateUrl"? The reason I say this is because we use the term "parameters" elsewhere.


HBIRE_vector_take2 (amirroth) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Move model logic into model (repo use_count > 4)


Think you might need quotes (") around the collection title


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5196](https://teamcity.mesosphere.io/viewLog.html?buildId=75896&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:12:53


I'm not sure if you're familiar with using `map` and `reduce` in python, but I'm a big fan. The goal is to abstract away the concept of a loop so that you think in terms of whats happening, now how you do it.

In this case, you could do

    total = reduce(lambda total, val: return total + val, avePixelValue, 0)

Which is saying "Start at 0, add the val from every entry in `avePixelValue` to the total and return that".

Alternatively, since Python is awesome, there's a built in function to do that called `sum`


@berrange This was pulled in here: https://github.com/jaypipes/os_vif/commit/5e17661eae89769bd9beda6034923b41daade844


Hehe LOL :) I can understand what you mean.

Read this -- https://www.python.org/doc/humor/


OctomapServerContact and CollisionDetector is not compiled if robot_self_filter is not found.
In travis, these nodelets are not build, so we can not generate the instace of these classes.
https://github.com/mmurooka/jsk_recognition/commit/fe0b143e284be06ffae4549533430c3f1e7289fd


I had actually done that originally and it doesn't compile. Since AMDSRunTimeSupport is a namespace rather than a class you can't use the const modifier. I just pulled down master, tried to build, failed, changed them back, built, and pushed back up to master. I was confused last night too when it didn't build at first.


Python has a built in function to do this for you `tuple()` - https://docs.python.org/2/library/functions.html#tuple


Where are TODO instructions ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5197](https://teamcity.mesosphere.io/viewLog.html?buildId=75897&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:05:12


Again, like with the `CreateCollage` class - this is named more as an Action than a Noun. Classes should be Objects, and they perform Actions. It should be a `StockPhotoOrganizer` or something.

Also, why does it subclass `CreateCollage`? Conceptually it seems like it shouldnt, and in general you should prefer composition to inheritance.


Lots of unnecessary whitespace here.


I'm trying to work my way through the node.js tutorials so I can understand how the code is working, so tackling any of this is a bit over my head. But I'll keep trying. How does one even go about writing a driver for something? The Arduino libraries already exist and work pretty well, but I'm guessing they can't just be translated into javascript.
Have you encountered the error I mentioned before, where after about a minute the readings suddenly stop and freeze? If I can at least get it to read continuously I can start integrating it with the heater relays and get moving on to testing the rest of the system. So that's my main priority at the moment.

Yes I'll need my other chip back to test multiple devices... although I could probably get by with my one device and an analog chip, but I'd prefer them both be digital, it seems to be both more accurate and more resistant to noise and other interference.


@Arthasvivin You're very welcome.  I tried my best to make the code easy to understand haha.  If you don't mind me asking, what did you use this for?


That is great news.

On Sat, Nov 14, 2015 at 5:31 PM, legoboyvdlp <notifications@github.com>
wrote:

> @IAHM-COL <https://github.com/IAHM-COL>
> We have permission to change the liscence to GNU GPL.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FGMEMBERS/A320neo/commit/54df8031807e124c2ad1cca927cae5563cdb5880#commitcomment-14399989>
> .
>


You removed this field, which comes from the port binding information in Neutron, but didn't add any OVS VIF subclasses...


I know about the hash pressure; I was just curious how it would perform without.


Oof, so this has bitten me in the past before (on Ice actually), but initializers should be lightweight. There shouldn't be any side effects to creating an object. What happens if you do that is that creating one object calls some method that causes another object to be created, which then kicks off some other work that causes another object to be created etc etc. Then if an error occurs anywhere down the line you're SOL. I used to have code that looked like this:

```
engine = IceEngine()
try:
  engine.run()
except Exception as e:
  logger.error("There was an error running Ice!")
raw_input()
```

The goal of just catching "Exception" there was that no matter what happened while running Ice I would still run `raw_input`. That was because running the script opened a new terminal window and if I didnt do a raw_input the app would error and close before anyone could see what happened.

Well the problem I ran into was that initializing `IceEngine` had a bunch of side effects like I described above, so about half of the time we ran into issues it would happen outside of the try block and my app would just close for seemingly no reason. This got even worse with imports, since at one point I also initialized a bunch of global state  whenever the file got imported. Well that was even worse, cause basically no code was able to run before terrible things happened.

Anyway, this is a long story to suggest a simple idea - try to keep initializers as lightweight as possible. Creating objects shouldnt be an expensive operation - using them should be.


So, I've been thinking about this more. This code is what would be executed in Nova -- for instance, in nova/virt/libvirt/vif.py. One of the ideas behind os_vif was to remove the need for Nova to have a whole bunch of conditional logic that handles the different vif types (or vif plugins in the os_vif world). The problem I see with the proposed object class hierarchy here is that Nova will still need to have a bunch of conditional logic that looks kind of like this:

```python
# do some communication with Neutron -- i.e. the port negotiation stuff...
port = neutronclient.ports.find(...)

# Create a VIF object subclass of the type that the port binding information returned
vif_plugin = port['vif_type']
if vif_plugin == 'ovs_hybrid':
    vif = vif_objects.VIFOVSHybrid(...)
elif vif_plugin == 'vhostuser':
    vif = vif_objects.VIFVhostUser(...)
...
```
and so on. Which is kind of what we're trying to get away from, no?

That is partly why I wanted a single VIF object that handled the polymorphism needs internally. We also need to consider that if we have multiple VIF subclass objects then the `os_vif.plugin.PluginBase.describe()` method interface that returns an `os_vif.plugin.PluginInfo`, what object version would that be? The subclass? If we have a single VIF object, we can evolve the schema of that single object in a measured and consistent manner.

Thoughts?


Should be ------


this looping could be refactored and cleaned for clarity. e.g., your `start` doesn't remain a start conceptually, really it's your current location as you search. and line 90 feels unintuitive when reading; maybe partly b/c it's poorly formatted?


I think you're supposed to close this file handle when you're done. Since this is a short lived script I dont think it makes a big deal, but when working with files an easy way to make sure everything gets cleaned up correctly is to use the `with` keyword (which Files support in python)

```
with open(self.jsonFileName, 'w') as jsonFile:
  data = json.dumps(colorAverages, indent=4, sort_keys=True)
  jsonfile.write(data)
```

When the with block is exited the file will be closed and you'll be a happy camper.

(You can use this same style to do easy perf logging, which I will describe in a little bit)


http://forum.flightgear.org/viewtopic.php?f=4&t=25961&start=165#p264517

-----------------------------------------------------------
>Thorsten Renk Wrote
> I think you may want to read what the GPL license has to say about dependencies and code. That your code is written by you is a necessary, but not a sufficient condition for licensing the bundle as you like.
> Edward would argue that merely using a 3d model to visualize flight dynamics triggers the relevant GPL provision for everything in a bundle if, say, the flight dynamics is GPL licensed. I'm less sure in this case, but your code calling GPL'd routines? That's a rather clear-cut dependency and automatically makes your code GPL.


this search seems easier to read and makes me wonder whether you can repurpose/DRY along with the previous code for `delete`


Verbs Nouns etc etc.


Lightweight initializers


this looping could be refactored and cleaned for clarity. e.g., your start doesn't remain a start conceptually, really it's your current location as you search. and line 90 feels unintuitive when reading; maybe partly b/c it's poorly formatted?


Externalized ES IP


oh noes, someone forgot to do a key piece of the assignment  :-P


Hi,
I am now reading the paper '3D Reconstruction with Monocular Vision using Supervised Learnig ', and want to redo this test. But I find it difficult for me. I can't get the dataset. Could you please give some suggestions?
Best,
Wenju Xu


"parse" is probably confusing naming (unless there's a precedent?). generally "parse" has fairly important meaning, that isn't really what you're doing here.


I don't quite understand this for loop enough to refactor it into a forEach or something else (forEaches and non-final variables can be icky). Could @Flaxbeard or @elifoster explain to me what edge case this is trying to catch to see if I can refactor it?


DirectX libraries that we ship have the same name for 32bit and 64bit anyway. We will never be able to have both versions in the same folder without issues. Distinguishing mpc-hc.exe from mpc-hc64.exe made sense because of some Windows quirks with associations, the rest... not much.


This is used just for locating some objects beside color blocks in 1st step.
Actually, it's very ideal for me.


Yep, I was away, sorry :)


These files can be regenerated by gradle.


I did not write this code, so Flax will need to explain it.


Ah, now I know why you were asking me about `.DS_Store` the other day :P

Anyway, instead of having an if statement here I think your intention is clearer if you filter out non-image files you should do something like this:

```
dirContents = os.listdir(self.subredditFolderPath)
picNames = filter(lambda f: f.startsWith('.'), dirContents)
for pic in picNames:
  img = ...
```

(Along with `map` and `reduce`, `filter` is an incredible utility. 10/10 would recommend - https://docs.python.org/2/library/functions.html#filter)


how do i run these tests i can't figure it out where're the docs omg


nit: Class names should always be capitalized. Im assuming thats a mistake though since you did it for all of your other classes :P


OK, Good to know


This is what I mean about initializers having side effects - creating this object ends up making a network request to the Reddit API. A single line of code has far reaching implications


added :P


Well I kind of agree. But why have we done it for LAV Filters then?


The naming on this is a little weird. The type of your `image` variable is `URLopener`?


Arrays and other collections should have plural variable names to suggest that you need to iterate through them. This should be `thumbnails`


In general you should prefer adding method parameters rather than instance variables on a class. Instance variables can change at any time for no apparent reason, but once you have a parameter passed in its never going to change without you knowing about it.

Really this function should be `callRedditAPI(self, subreddit, limit)`

And hell honestly you could even make that a regular function, not associated with a class at all. Pure functions are great, much easier to test.


Uh, is this supposed to be here? Looks like you're doing the same thing with `urllib2


These look like they should be command line arguments - check out the `argparse` module in Python, it makes it super easy.


Ouch :(


whoops, this was already done by another contributor. For some reason EGit still pushed it though


Not sure what exactly happened here. EGit did this on its own


@parshap does this seem right to you?


Having different travis.yml on branch `develop` and `master` is common practice.

The weekly merge down has to be handled a little more carefully.

@RocketChat/core - any strong preference?


@Arthasvivin That's pretty cool!  It's pretty strange that you stumbled across my code tho haha


THis is weird


for issue #12


So I mentioned the `with` keyword earlier - `with` is basically a way to let an object know both when you start an operation and when you end it. This is useful for things like files, cause when you start your block you want the file to open and when you finish your block you want it to close. You can do all sorts of fun things with this, but consider a simple class:

```
class PerfLogger(object)
  def __init__(self, label):
    self.label = label

  # __enter__() and __exit__() are the magic functions which get invoked at the beginning and end of the with block
  def __enter__(self):
    self.starttime = datetime.now()
    print "[%s] Starting" % self.label

  def __exit__(self):
    endtime = datetime.now()
    elapsed = endtime - starttime
    print "[%s] Ending. Duration: %s" % (self.label, str(elapsed))
```

Then you use it as follows:

```
with PerfLogger("Scraping Photos"):
  creator = ThumbnailCreator()
  creator.fetchThumbnailsFromSubreddit(subredditName, limit)
```

This will log when you start the thumbnail creation process and when you end, along with the elapsed time. You can then easily tell where all of your time is being spent. Especially since your current implementation is a compounded time - `starttime` doesnt change per step that you're doing, so every step supposedly takes longer than the one before it.


I guess because people tend to request "x64 and x86 builds to coexist in the same folder". Which really doesn't make sense anymore now when madvr has 64bit version.


This is a pretty silly/ugly format to use.


First Commit.


@Tjoosten i think this can be handful to use with the provider


vljrwvklrwk


@elethom :/


I'm using them both alongside each other for now. At some point I'm going to make complete transition to Ocornut's one, but for now that's not a priority so I'm trying to make them play nicely together :)


Can you make the default indexSeparator a forward slash instead, so its not language specific `'  /  '`.


since the default corner has no border radius can we remove this :)


Hi master I enabled the multiplayer mode,there is small problem about the bitly, How can I create short url? anyone help me?


same here, default to `/` slash instead of `of`


@skeptical can we use this information to update the playback timers on the status tab? For example:

```
double currentPosition = (System.currentTimeMillis() - res.getLastStartSystemTime()) / 1000;
currentPosition = currentPosition + res.getLastStartPosition();
```

Then set the timer to that


You know, you didn't have to copy my code word for word... And why the heck are you committing one file at a time?


@Arthur2e5 The guy who made this commit is likely to be an ignorant idiot. Confirmed.


LGTM


Teste


weird whitespace change; may be github messing the identation, so feel free to ignore :)


Why should this be Authenticated Remembered?


Note that it's preferable to use `include` over `exclude` (whitelist over blacklist). I know it's nit-picking but still. :)


:+1: Thanks for this fix, nice to see this added because of my PR dealing with the Hallow's End wands.


After trying owncloud as carddav server it was noticed that the response code of 200 being returned from radicale on a delete appears to cause the issue that prompted this commit and is responsible for the undesired behavior noted by ggtd.


Why not [url-loader](https://www.npmjs.org/package/url-loader) with limit set instead? That will allow you to inline small assets and avoid some unnecessary requests.


Sessions aren't int's, are they? @Griatch


@mannol would it be possible for core to have a way to chose complete data over fast data for video? like the above example, if you're doing tech support, 1 high quality frame every 3 seconds would be better than 30 that you can't read. I.e. using lossless vs lossy, and just having core silently drop frames while the last transfer is still in progress?


Is this addition necessary?


Git commits -s


These properties should check for Lite.


> However, for the sake of completeness `output_charset` and `input_charset` are assumed to point to UTF-8 encoded strings, so `_mbsicmp()` may be a better choice here.

Why?  Maybe I'm not understanding something, but based on the documentation from MSDN, it seems like the only "advantage" of `_mbsicmp` is that it operates on a character level whereas `_strcmpi` is a bit more naïve.  Since we're just trying to detect if the strings are exactly equal, not collate them, I don't think it should matter.  Also, `_mbsicmp` has the added restriction that it ["… cannot be used in applications that execute in the Windows Runtime"](https://msdn.microsoft.com/en-us/library/k59z8dwe.aspx).


Yes. I use clang-format to keep code consistent. There is a .clang-format file supplied.


@EvilOlaf - Ditto!
Thus, configurable XP amounts per-tree species should be added... *wink* lol


Thanks. Much cleaner!


Nice!


@ahmedcharles

Ah, thanks no, that's a leftover from sessids. Fixed in latest push.


They do `<OutDir Condition="'$(Configuration)|$(Platform)'=='Release Lite|Win32'">`


Spelling mistake


```
    <VisualStudio>
      <UserProperties RESOURCE_FILE="mpc-hc.rc" />
    </VisualStudio>
```
Any `UserProperties` might not be needed too.


By the way try to remove those "leftover" subrecords too, I doubt that FO4 has them.


indeed! will fix!


What is `requirements.txt~`?


Please check the indentation on the CSS and the HTML file.


remove whitespaces


Isn't `WEB` missing here?


True that, I missed the obvious there. So does everything compile fine for lite without mixed dirs etc?


I am the originator of the post mentioned above ...
Before I Update to 15.7.19: Is it sufficient to patch the file only for testing ?


# Adventure
A strange game. The only way to not be disappointed is not to play.

![a](http://media.peakprosperity.com/images/chris-mish-screen2.jpg)

-- (LOL)


#include <windows.h>

BOOL HardwareBreakpoints_GetThreadContext()
{
	/* Hardware breakpoints are a technology implemented by Intel in their processor architecture,
	and are controlled by the use of special registers known as Dr0-Dr7.
	Dr0 through Dr3 are 32 bit registers that hold the address of the breakpoint */

    unsigned int NumBps = 0;

    // This structure is key to the function and is the
    // medium for detection and removal
    CONTEXT ctx;
    ZeroMemory(&ctx, sizeof(CONTEXT));

    // The CONTEXT structure is an in/out parameter therefore we have
    // to set the flags so Get/SetThreadContext knows what to set or get.
    ctx.ContextFlags = CONTEXT_DEBUG_REGISTERS;

    // Get a handle to our thread
    HANDLE hThread = GetCurrentThread();

    // Get the registers
    if(GetThreadContext(hThread, &ctx) == 0)
        return -1;

    // Now we can check for hardware breakpoints, its not
    // necessary to check Dr6 and Dr7, however feel free to
    if(ctx.Dr0 != 0)
        ++NumBps;
    if(ctx.Dr1 != 0)
           ++NumBps;
    if(ctx.Dr2 != 0)
           ++NumBps;
    if(ctx.Dr3 != 0)
        ++NumBps;

    if(NumBps>0)
		return TRUE;
	else
		return FALSE;
}


That's probably better :)   Just double-check whether *you* use `pyret_keywords` anywhere in the highlighter; the uses of it in the indenter are already set. (Note to self: I should document the code in the pyret mode more thoroughly.  I'd make an issue out of it, but Github isn't showing me the issues controls for Joe's repo...)


@bringha1 let me answer on the forum in a few minutes


did not mean to include the jingle tiddler in this commit/push.


Yes, it does. I confirmed [*slick* uses `name`](https://github.com/kamicane/slick/blob/6eb8da55711e3b0ca81777b96cf45da868af382f/parser.js) and not `key`. Nice find.

The specificity algorithm is a beast. I tried using an [external specificity module](https://github.com/parshap/juice-next/blob/master/index.js#L268) (also where I experimented with rewriting juice's core inlining functionality) and it worked well.


Start of the proposed checker for people to leave elevator.

Cons: How to delete after people have left the elevator?


I think the current approach of pairing a Bluetooth LE peripheral is flawed. Once the PairAsync method returns the GATT services should already have been cached by the operating system, or the BluetoothLEDevice object should raise a GattServicesChanged event once the services are initially cached. Waiting for a certain amount of time or initiating a second monitoring till a generic GATT service is available are not optimal solutions. And in other scenarios (for example on desktop where the user pairs in system settings and immediately returns back to the app which is unaware of the pairing operation) they are even not applicable. I hope this gets fixed in later releases.


I think we can remove this too since we use W4 in common.props.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5209](https://teamcity.mesosphere.io/viewLog.html?buildId=75942&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:10:05


Sure thing ...

> Am 15.11.2015 um 21:14 schrieb Franco Fichtner <notifications@github.com>:
>
> @bringha1 <https://github.com/bringha1> let me answer on the forum in a few minutes
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/opnsense/core/commit/249c74c6d661c75e5d9c5231031cfda6d719de90#commitcomment-14403594>.
>


Object-oriented design deyende neyi nezerde tutursuz?


Yeah, the colophon is outdated.  All the recent Conventus notes are meant to be according to Roman rite (~ Divino Afflatu), with some extras (148+149+150 psalms instead of 148 only, full Gospel + Te Decet Laus added to Matins), and responsorium breve added to Vespers/Laudes.  I'm still using amon33 for the notes, but the content is checked with arom12, lusualis62 and divinumofficium.com.  The older SGP booklets were often a mixture of Monastic and Roman rites, but for Matins we usually have only Roman sources (Nocturnale Romanum), so the agreement was to prefer Roman rite.


On font names that longer than one word use `"` so write it like `"Josefin Sans"`


Indentation


I guess you added `FATAL_WARNING=off` because of this error:`
> remote-processor/Message.cpp:54:41: warning: unused parameter 'offset' [-Wunused-parameter]

Ie: an assert that uses function parameters not used elsewhere in the function. I would rather fix the function than remove the warning.
I still really like the `FATAL_WARNINGS` option though. But more for unsupported platform (integration) than in the development cycle.


If you need to use a different database name you can override that in the local_settings.py file, no need to edit this file. Is common for all devs and is better to not enforce this things, devs have different database environments.


Ok.


The name of this commit isn't right!


Try to come up with more descriptive class names. Data could be everything.


This seems not to effect the link behind the github icon. @kba Any ideas?


@SthephanShinkufag у тебя на форчане посты отправляются нормально?


Git commits -s


Please use some block element to put something in a new line instead of `<br>`-s, for example `<p>` or `<div>`


This is the RMD file for assignment 1


@Azer28 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Introduction_to_Object-Oriented_JavaScript


`<br>` again


Review from Group 12:

Things you did well:
* You did a great job in profiling and found the part that is most time-consuming, which gives you idea about how to tune the code later.
* With vectorization and re-ordering loop, you got a good improvement. That is great.

Things you could improve on:
* I didn't see your MPI code, but I assume that it could be improved because in general the MPI code should be more efficient than the OpenMP version. Actually we have the same situation in our MPI code.
* Maybe you could try blocking strategy in the square function, which should potentially improve the performance.


this is acomment slash


The list was way better than the `<br>` it is more semantic


Oops.

Thanks for fixing that one!


Missing space between the selector and the opening curly brace


Indentation


Maybe the unit test I added would be interesting to check that see : https://github.com/twbs/bootstrap/pull/17640


Опиши, пожалуйста, как минимум тесты для самого драйвера с конкретными проверками, тут про них ничего толком нет. Насчет кэша и скедулинга (скедулинга!) всё понятно, этого достаточно.


Guess & -> and would have worked as well. :-)


O QuizItem funksiyasi constructor rolunu oynayir. Teklif edirsiz bir obyekt yaradim ve sonra ona esases Object.create() -den istifade edim constructorun yerine?


You could use the `left` property since it is `position: absolute`, in this case you don't have to mix margins with position properties.


Да, для кэша хост -> дц сейчас этот дефолт и используем.


You don't have to use uppercase in HTML you can use the `text-transform: uppercase;` in CSS


Looks like the example have semicolons that is not really should be there.


Typo!


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


just go back in the commit history, on mobile now so don't have access to it but if you go back a few commits on github you can copy the file content and save that to a .obj file


Travis-CI handles this magically, any reason to force it?


Teshekkurler) Nezere alacam)


good catch


It is unnecessary to wrap your sections to these `<div>`-s you could add these classes to the `<sections>`


You could done it without repeating yourself, like:
```
section {
  border-right: solid 3px black;
  max-width: 600px;
  padding-right: 1em;
}

.text2 section,
.text4 section {
  text-align: right;
}

.text1 section,
.text3 section {
  text-align: left;
}
```


> After reviewing this function, I now understand why you have been calling `vsnprintf()` initially. `scprintf()` and `vscprintf()` simply do not exist in the Unix/Linux world, consequently neither does `scwprintf()` and `vscwprintf()`.

That's a really misleading way to phrase it.  Calling `vsnprintf` to calculate the length of the string is the standards-compliant way of doing this.  Microsoft's implementation of `vsnprintf` is not C99/C11 compliant, `_vscprintf` is a Microsoft extension to make up for the deficiency.

In other words, the reason I was calling vsnprintf is that it's the right way (a.k.a., the standards-compliant way) to do it.  The reason I was calling it with a small stack-allocated buffer was it's an optimization (which, BTW, MS's version doesn't allow).  The reason I wasn't doing it on Windows is that the implementation there isn't standards-compliant and doesn't support that pattern.

> The implementation could be optimized a bit if the FILE* stream to /dev/null was setup as a global variable at load time of the shared object (and destroyed upon unloading or termination).

Yeah, that's probably a good idea.  Destructors aren't standard, but it would at least be possible to support it for GCC, Clang, and MSVC.  Other platforms would just never close the file, which isn't a big deal (only people who run valgrind but don't understand the results would care).

That said, I'm thinking about re-adding the original version alongside this version as a much better optimization.  Outside of Windows nobody really uses wide character strings, and on Windows we can avoid this function completely, so it's unlikely anyone would hit the slow version.


Data chyba nie do konca poprawna : )


Please use a folder for the images and the CSS as well.


This should probably be `< 4.0` or at minimum less than `3.1`


Это на переходный период. Модуль inventory я выпилю, когда перейду на исключительное использование inventory-воркера, потом inv.py -> inventory.py и бам! Напишу, пожалуй, об этом при импорте модуля.


Should this function be exposed in SaguaroQL?


LoggerInterface $logger=null


That's funny, because the very first version of "In Nativitate BMV" was actually according to the secular Roman office, strictly following the "Divino Afflatu" order and the 1912 Antiphonale Romanum https://github.com/jakubjelinek/Editio-Sti-Wolfgangi/tree/a60ada506522fe44e7982a2593cade43e89b727c/InNativitateBMV

However, Filip protested against Vespers without Lord's Prayer chanted by the whole assembly (it's recited secretly in the secular Vespers) and after some discussions it turned out that monastic office should be followed instead.

(Well, actually he first wanted just secular office with some random "reforms" applied and tunes taken from Antiphonale Monasticum, like you make it now, but I am quite stubborn when it comes to self-decided liturgical "reforms".)


- 84
 - `$mysql->fetch_assoc()`


if($logger===null) { $logger=new Psr\Log\NullLogger(); }
$this->logger = $logger;


Exposed upon error? pls specify


See #317


that is, on old versions of MinGW (such as those we can install on TravisCI right now)


`mysql_real_escape_string` does not exist in SaguaroQL as `$mysql->escape_string()`.


@geshan My pleasure! Thanks for sharing :)


:+1: :heart:


this first version was done on 11/14/2015


Не. Висит проверка.


Not sure ... Github handles the "&"as a "-"in the named link (giving two "-" in a row), but the pandoc-generated html and epub expected only a single "-"... this seemed like the simplest way to elimintate the conflict :-/


:<

А почему там html5 выключен?


@lucasart
It was not to achieve "atomicity" but to see if there was a performance gain from more efficient cashing by only accessing the hash once and keeping a copy on the stack.


Не было смысла с HTML5. Там запрещено встраивание, и добавление байт. И еще кажись какие-то проблемы были.


Gah, redbox-react requires `'react'`, which is causing 2 versions to get imported


"at AGH-UST".


Why are we redirecting to the root route here? This will cause problems with implicitly rendering the show view


Best practice would be to redirect back to the show view for that post you are updating


Try to write more descriptive commit messages like `increase container with`.
Git will already show that you modified this file


Hi Andrew,
that is correct. The limit ranges between 4 and 5 magnitudes. I can test it ones again to be super sure.
I can agree with you to put that not in this place but then we would need some Htautau specific entry point into this code. I like the idea to have a general part to be usable for all, but i would also like to have Htautau specific codes available. I think there are cases which will be analysis specific and still should be sharable between people instead of private code.


Nah, "assert_return()" is supposed to be used only for programming errors, when things are part of a public API. This is an internal interface, hence the assert() is at the right place. However, we should really validate the PID before we invoke the function, already in method_get_machine_by_pid(), where we first receive the data from the client. Will prep a fix for that.


will also fix similar mistakes in logind.


Смысл есть, поскольку так ьбыстрее, ибо не копируется строка с `outerHTML` после каждого ответа.


Please use English classes :)


We need build artifacts from both right?  If its different on master / develop come time for merge that's a step we have to constantly mess with.  Seems like it would be best if the travis file was the same for both.


Да и DOM-дерево не строится внутри iframe.


That's a big question.  :)  You can ?predict in R for details.  Hit me up
in the office.

On Sat, Nov 14, 2015 at 2:41 PM, Conor <notifications@github.com> wrote:

> Line 91. Wondering how you're using `predict()` here and what the
> capabilities of this function is.
>
> 2015-11-13 20:14 GMT-06:00 ecocarlisle <notifications@github.com>:
>
> > Line 91 or 92?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ecocarlisle/Blood-Donor/commit/4266cd1e1bcafbb7004bf6b42bbf1cbd98920246#commitcomment-14395230
> >
> > .
> >
>
>
>
> --
>
> Conor B. Murphy
>
> Regional Grants Officer | Rotary Grants
>
> Tel 1.847.424.5305
>
> rotary.org <http://www.rotary.org/>
>
>
>
> [image: Description: Description: Description: Description:
> cid:1B9D9986-621D-4D75-841B-EF8661B3CCC1]
>
>
>
> ROTARY INTERNATIONAL | One Rotary Center | 1560 Sherman Ave. | Evanston, IL
> 60201 USA
>
>
>
> Like us: facebook.com/rotary <http://www.facebook.com/rotary>
>
> Follow us: twitter.com/rotary
>
> Watch us: youtube.com/user/RotaryInternational
> <http://www.youtube.com/user/RotaryInternational>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ecocarlisle/Blood-Donor/commit/4266cd1e1bcafbb7004bf6b42bbf1cbd98920246#commitcomment-14398990>
> .
>


De ce revert?


You could add a class for each section, and only write a selector for that.


Опечатка-копи-паста. Поправлю.


Спутал handle и handler, в данном случае более верно handle. Поправлю комментарий к коммиту.


maybe not needed to be such hacky, we can do it by closure and re-assignments


innerHTML axi funksiya deyil, o bele `innerHTML('');` yazila biler? yegin siz `mainDiv.innerHTML = ""` nezerde tutursuz, he, bele daha asandir


something along this line
```r
log.callback  = function() {
    tmp = res
    callback(env) {
        env$train.metric$get()
    }
}
log.callback  = function() {
    tmp = res
    callback(env) {
        res = env$train.metric$get()
        tmp<<- append(tmp, res)
    }
    get() {
        tmp
    }
    return(list(callback=callback, get=get))
}
```


Let ``gulp-less`` be only in devDependencies..


Git commits -s


Use ```<<-``` to rebind the local variable in the closure, and use get to return the result


HTML5 там работает и при этом показывает ошибку в отличии от iframe, который я починил.


Still TO DO:
- robust handling strategy for errors arising from a deep stack level
- additional Analyser checks
- More precise parameter splitting by class Function
- Consistent return value handling
- Improvement of handling and functionality of the Arranger component


:thumbsup:


> On a side note, it may also be advisable to not export this function in the final shared object.

Sorry, forgot about this part.  It's marked as static, shouldn't be an issue.


coment


See: https://github.com/freifunk-gluon/gluon/issues/557


no, just no! :P


see: https://github.com/freifunk-gluon/gluon/issues/557


I have no problem if you want to do that.  Last time this
http://tests.stockfishchess.org/tests/view/543562c90ebc593533cac2ac
updated version was about 2% slower so about 2 elo weaker.  You can just bench it because if you define keyWidth as 16 it's a non functional change.


Well, they are kind of "normal" method. I'd like to follow the Qt convention, which is to use camelCasing starting with a small character. There's no better or worse here I guess, mostly just convention.


Hello,

Rather than cleaning the \n, if any, I have created a new argument, "..." to python.assign. It goes to toJSON, the internal interpreter, and this function has options to control the \n that it sometimes inserts. Mind that your solution may kill \n inside user strings and produce hard to track errors..


Ja dat moet .getGameId() zijn. Dat was om te checken ofdat het bericht van het juiste game komt. Ik wilde eerst checken ofdat dat van de owner van het spel kwam, maar daarvoor moet de lobby het id van de owner bepalen.


I think the changing the height of the body affected the whole thing and cut off the logo


Salut Mihai! Georgiana a folosit o masina pe care era configurat contul meu de github (am folosit-o eu in trecut). Cand a facut push-ul (cu credentialele ei), sistemul a facut update-ul cu datele mele. Ciudat, nu imi explic cum. Tu ai idee?
Apoi am vrut sa facem revert la commit-ul acela, insa din greseala cred ca am copiat id-ul acestui commit si nu al celui facut de mine. Prin urmare, de aici revert-ul. Am vrut revert ca sa faca commit-ul de pe contul sau. Imi cer scuze pentru greseala, te rog sa repari tu proiectul. Mersi!


You could use `list-style-type: square;` instead of `:before`


TeamCity LabkeyRelease15.3 :: CDS postgres 9.4 [Build 1](https://teamcity.labkey.org/viewLog.html?buildId=285313&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 15 (15 new), passed: 11 Build time: 00:51:40
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSHelpCenterTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.verifyAntigenVariableSelector: <no details avaliable>

##### there are 5 more failed tests, see build details
```


Ah, ok, nu-i nimic, repar eu.


Usage in standalone project :
In main.cpp , Just change the 'standalone' value in setUp(standalone) to setUp(plugin) to see the effects .


If I don't misunderstand you, you want

```
train.model <- function(callback = log.callback(train_metric)){
  f_lst <- log.callback()
  f_lst[[1]]()
  train_metric <<- f_lst[[2]]
}
```

But I don' think this will work.


I could use: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/time


Hi, I was wondering how I get the greyscale off of this theme. Thanks :)


Git commits -s


Should be `trailing` but nitpicking :P


nitpick: see CODING_STYLE: we place opening bracket on same line as function name.


@volter9 thx, works great


What's this for?


Please uppercase the first letter in english sentences


same here.


This is the version test class.


Two spaces between the tag an the class


I've had a go - the main problem seems to be that when you pass through the `potion_str_eval` function, at that point you have a reference to the closure of the `potion_str_eval` function and not the closure representing the code that is actually calling the eval.

One way we could possibly get round this is to have an OPCODE for the current closure that can be invoked with the message "current" or something similar. We can then make an optional parameter on the str_eval function that can be passed a closure in which to execute the string. Often, it won't matter the context in which something is executed.

I tried making a function that would return the current closure, but I ran into the same problem that the closure was already set by the time it hit the function. So, I think it would have to be an opcode, or something built-in to the runtime.


Missing space between the selector and the opening curly brace.


"CloudFoundry" should be "Cloud Foundry" cause branding.


Please put the images in to a separate directory from the CSS


Nope, docs should still be correct.  It's still a two step process - join
the community and/or group, then visit the link.
On Nov 15, 2015 8:00 AM, "Ben Bodenmiller" <notifications@github.com> wrote:

> Is this still correct? I thought you mentioned somewhere that someone
> could now just signup by going to
> https://play.google.com/apps/testing/com.joulespersecond.seattlebusbot
> without joining the G+ Community but I can't seem to find the comment.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OneBusAway/onebusaway-android/commit/e2c6c5ebbf5c1dddd9bbe34de1b45f0e903dc75c#commitcomment-14401764>
> .
>


Please use sd_bus_error_set_errnof() and then pass in the load_error as error number and use %m to get the string for it.

Also, I think we should not access the load_error in the catchall case, but only in the load states where we set it... Othwerwise it's going to be null, and hence will say "success"


fnction should probably use a single switch block, rather than multiple if checks?


git revert la id-ul revertului merge? (id 63bb767c70eb48fd4bfd4dc2308bc6b160b156a1)


Does this one by any chance fix http://trac.kodi.tv/ticket/16336 ?


That's defined in the header which is pulled via AJAX from the infolis-web machine. Will update it.


you tell us when there's a nightly build tomorrow.


Probabil. vezi cu git diff ce se intampla, sau git show HEAD


@vburlak you're welcome :)


I don't understand your statement. If a translation is missing from a .po file, the original message is displayed, which is English. The de-facto way of extracting translation from source files is the xgettext command and creating a `.pot` file, which is a translation template.

But in your case, translators extract string by hand, and there is no way to see if there is a new string to translate since there is no automated way of doing translation things in recalbox. That is why I put the msgid's in po files, so transators can see their translation files are not fully translated at all :)

I can remove them if you insist though.


I disagree, you sure this is wrong?


Why don't you check for v3_ table before deleting them?


We should use a different force mode here that isn't effected by mass. Perhaps `ForceMode.Acceleration` or `ForceMode.VelocityChage`


no wonder #12 doesn't want to work, this is wrong here, should clone/copy the committed version not what the current master.


@jomo yes, looks like it. we're developing a new bot though, so ¯\_(ツ)_/¯


check if lc is null. put unknown if it is null..


Because there is no need too, as this is intended to run on empty v3_  tables.


Please remove the extraneous newline


more semantically called as:

```csharp
rb.AddTorque (transform.up * torque);
```


thanks t'as tout fait waqil :p
moi je viens d'arriver a la section DAO en base de données. j'avais pas assez de temps


If you end emulation in a game, change plugin from Edit Game Settings menu in rom browser, restart same game, plugin does not change because the Game_TempLoaded is true.


These are modified PHP files–containing improvements to display on-screen names in localized order if language is Hungarian.


It is still in use - e.g. https://github.com/jruby/jruby/blob/9570c7331f4d17b8d3a6917edb7702e39fca785c/truffle/src/main/java/org/jruby/truffle/nodes/core/array/ArrayNodes.java#L2132


My VPN server wouldn't start with this line of code, but simply removing it seems to fix the issue.


This will break the paging for purchasing new credits (cause it relies on the index)?


Separate bash script is much cleaner. So assuming it still works, LGTM!


So we have doze ?


You likely have already seen this, but: https://wiki.remotestorage.io/Apps


Git commits -s


It's a bit much to take a small, one line helper into it's own file, IMHO.


Yes. But it is not linked anywhere from the wiki it seems?! I arrived at the 'old' page!


Nice, very helpful.

I'm going to create a page at some point that documents what each effect does. That'll be useful I think. Have started on an app. It'll eventually have a node editor, but for now I will get it talking to the daemon.


JSON with comments?

worth mentioning, since we support it.


Gebruik _done_ in assert om async stuff te testen: https://jasmine.github.io/2.3/introduction.html#section-Asynchronous_Support, _failed_ is dan overbodig.


Thus, derived Bootstrap variables can depend on values from here. See the Bootstrap  `_variables.scss` file for examples of derived Bootstrap variables.


Isn't it better to move confirmConflict to a config.... now after each enigma2 restart it will re-occur.... With e.g. add it to the common settings so once enabled you can still disable it....


Thus, you may customize Bootstrap variables based on the values established in the Bootstrap `_variables.scss` file..


will do tomorrow morning . . . .


That's not a problem.


Try to use more descriptive class names


Use snake_case for method names.


@shalabhc I did this mostly because all the other lexers use `type` in this case, but let me know if you have any objections.


What motivated this? Just incorrect previously?


Indentation


Missing a space?


We'll need to get these examples in the docs somewhere.

CC: @robwise


nice! :+1:


Git commits -s


camelcase used later here https://github.com/netguru/people/blob/master/app/react/components/statistics/statistics.jsx#L19


I meant to address this on Friday, but can you figure out a way to just reference the post you created on line 28 instead of querying the db for it again?


@Dannonew Thanks for the patches. Keep in mind though to test if things still compile before sending a PR. I fixed it up manually this time.


I looked it up, and in fact it is not wrong, but:
> "Is to be used" is grammatically acceptable but stylistically weak.

(from answer of http://english.stackexchange.com/questions/147574/has-to-be-or-is-to-be)

and to be honest for me it sounds like pain in the ass when reading it, sorry :/


because destroy is a func not an action


alternatively there could be another version of lightevent that uses funcs


Hey dude,

Well, I hacked this up b/c I was on the plane and couldn't get to the handbook but I'm sure what I want is doable.

1. I want `testdouble.replace()` for the browser to be swapped with a different function that throws an error along the lines of "This is only supported on Node.js"
2. I want to have a way to indicate node-only and browser-only tests
3. I want it to not look like a big mess


possibly, however no functionality has changed as of now


Hi Pascal,

Would you be interested in contributing to this app?
Let me know if it is the case.

Simon.


TeamCity LabkeyRelease15.3 :: Git postgres 9.4 [Build 79](https://teamcity.labkey.org/viewLog.html?buildId=285328&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24 (2 new), passed: 263 Build time: 01:27:33
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

##### there are 14 more failed tests, see build details
```


:+1: I was just getting to this


We should probably have a look at detecting which method is best, or have a better fallback, because edge scrolling is quite awful to use on modern trackpads like macbooks for example


score is quite likely due to decay


TeamCity LabkeyModules15.3 :: Git sqlserver 2014 [Build 19](https://teamcity.labkey.org/viewLog.html?buildId=285326&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:23


beats yours :#


there is, just didnt apply it


Is this appveyor specific? Would it be worth including something like [is-ci](https://github.com/watson/is-ci)?


it doesnt run in its own thread. and we have more control over events this way


as before, it does exist


bugou o commit hahaha


HelloWorld.md
## This is a markdown file


Ooops, forgot to switch to master...


Should we add pictures here? Pictures are worth a thousand words.


Thanks. I have a few others bits to do @smilespray. I'll add them in as tasks in mine


literally my life lol


Thanks. Removing my own implementation right now :+1:


By the way, are you ok with standing as maintainer of the zile package? It's done by adding a

    TERMUX_PKG_MAINTAINER="firstname lastname <email@example.com>"

line to the `build.sh` file, and gets added as a header in the package so that anyone running `apt show zile` will see you as the package maintainer.


I really would like to, but honestly, I don't have the faintest idea how Kodi links with ffmpeg. The only thing that I know is that previously if Kodi had a <streamurl>||HttpProxy=http://<host>:<port> format, ffmpeg would use that proxy instead of the configured proxy (if there was) in Kodi.

But since 15.x that does not work. Kodi does pass it on to ffmpeg. ffmpeg does see it but passes it on as a HTTP header instead of using it as a proxy. Is that something somebody could go with?


This text seems outdated. It still references `generate_entities.sh`. Also, this is not the right place to talk about "modify[ing] the entities.yml file", these are just the short build instructions!


"Compare two different folders on a local filesystem without recursion." - is better


This is not the place to bump your issue. Update your bugreport instead ...


There is a buildtime dependency section above. You could make a subsection for gamelogic buildtime dependencies there if you insisted on splitting that, though I fear this may bloat the readme.


This should have the same naming prefix as the parent class, e.g. `class FileMIMETester_Validator` to ensure it sticks with SS coding conventions


I thinks it is to make routing only in the country you are, so if you are
in Spain, it would try to make routing in Spain only, avoiding border
crossing to Portugal or France.

--
Cumprimentos,
D.er Hugo Barrocas


See if you can also use `implements TestOnly`. I'm not sure if it will do anything, but it should mean that you can't use that validator in normal code.


Śmiesznie Was znaleźć w odmętach internetów.


> this.num

Альзо, пофиксил скрытие ветки при Cfg.hideRefPsts === 1, не работала.


Maybe don't bother with disabling the form, and just let people upload another file if they want to?


Sorry, MartijnKaijser more or less asked? I just asked a curious question? Sorry if offended somebody.


Better to move this into a new file - or keep it inside diff.go .

common-methods.go is only for common functions for entire 'mc'.


Maybe wrap this in a `if(function_exists('finfo'))` to ensure that it doesn't `E_FATAL` if it doesn't exist because the module isn't installed. Make sure you return an appropriate error message in that case, so this can be used to test for that too.


No probs
>On a related note, the reordering and standardizing of ability text is a nice side effect.
Agreed, I'd love to be able to sweep through the whole lot, but there's still a fair bit of manual work which the scriptbuilder can't do yet, these image-checks are a good middle-ground. (And an alert for when webpages on magiccards change is helpful too!)


Если `Скрывать ответы на скрытые посты*` выключено, то все работает как раньше. Первый пост скрывается через `setUserVisib`, ответы через `setUserVisib`. Пункта в меню `Скрывать с ответами` нет.
Если опция выключена, то доступен пункт меню. По клику на `Скрывать с ответами` пост вместе с ответами скрываются через `setUserVisib`.


unnecessary to have the / on the end of the tag


Travis is rock solid without it.
This is to make our own tests pass.
Our tests rely heavily on stdout and stderr. Which are flaky on windows CI (it occasionally discards the last line / three of output)

I doubt other people's tests are going to parse AVA's IO stream. Really, I probably should change this to a unique environment variable that we set in appveyor.Yaml


Hey man. I have an issue. Can you help me out?
https://github.com/huytd/agar.io-clone/issues/346#issuecomment-156797251

this clone doesnt work on ipv6 for whatever reason and I wanted to fix it


Please use English class names


comment could be: `Brand new sync/async implementation, a lot of changes in publish mechanisms. Should be used with care`


Missing space


*куча опечаток.
Если Скрывать ответы на скрытые посты* включено, то все работает как раньше. Первый пост скрывается через setUserVisib, ответы через setVisib. Пункта в меню Скрывать с ответами нет.
Если опция выключена, то доступен пункт меню. По клику на Скрывать с ответами пост вместе с ответами скрываются через setUserVisib, игнорируя прошлые состояния постов.


Typo for "canola" oil


This causes the plugin to not load at all...


Можно просто редактировать комент :grinning:


Yes, I agree, and would also would prefer this method.

Blue button is more often used, so it could be as it was in day 10-11-2015,
just one tap and go navigate (well, one more to give it the 'Go' after
viewing the planned route, but it was ok).

The waypoints button could hold the other options, when needed.

--
Cumprimentos,
D.er Hugo Barrocas


.seek +30 and also .seek 00:10:39 work :)
Thanks.


Not needed to be fixed now, but we've got a bool trap here.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5210](https://teamcity.mesosphere.io/viewLog.html?buildId=75949&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:13:40


Used to create a new database on a server

•	Checks to see if the database exists, throws error if it does.
•	Sets setting variables based on what has been entered


Ну короче ты понял.
Я тут думаю, как сделать, чтобы можно было отменить "зашквар" пользовательского скрытия. Иногда сталкиваюсь, что хочу протестить спелл на каком-либо посте, а он уже раскрыт пользовательски.
А с этой фичей одновременно десятки постов в треде зашкварить можно.


Thanks to @Tjitse-E for feedback about the problem this release fixes.


Можно пункт в меню "Отменить выбор", или что-то вроде того. Пункт должен появляться, если пост зашкварен пользователем.


b? I'll request you avoid single letter parameters if possible. Let's keep understandable words.


Можно сделать так: если пост после раскрытия может быть скрыт спеллом, то зашкваривать его, в противном случае просто раскрывать и удалять инфу о пользовательском скрытии.


Sure, anything is possible!


Правда на доске и в треде могут работать разные спеллы, потому может работать криво.


This is one is confusing IMHO, really. :)


Git commits -s


Не, сейчас логика работает идеально. Просто как-то добавить возможность снять пользовательский выбор.


Generally in WP docs, I believe it's preferred to use [backticks around code or variables](https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#other-descriptions) within in PHP DocBlocks, because those end up getting parsed into the HTML documentation.

This doesn't affect us that much since we're not automatically generating our docs, but it's good to be in the habit since some of our docs end up getting copied over to core.

For regular inline comments, backticks can be dropped in favor of single quotes.


Двойной клик?


Makes error in online test, but not on my windows


thanks @dalcde


С зажатым контролом или шифтом.


Ну или так.


Naja. Mit dem hier bin ich nicht ganz einverstanden. Idee dahinter war ja, dass SetCenter einen false zurück gibt, wenn der Task abgebrochen wurde. In diesem Fall ist der Ladeprozess nicht vollständig und es ist nicht gewährleistet, dass alle notwendigen Chunks geladen sind.

Ein Cancel des Tasks sollte aber immer zu einem neuen Ladeprozess (mit eigenem Success-Handler) führen und früher oder später ein true erhalten. Einen Freeze hab ich hier noch nicht erlebt, könnte mir aber vorstellen, dass der Success-Handler schneller zurück kommt als die kommende Zeile (ReadyState = false) ausgeführt wird.

Meine Lösung für den nächsten Commit: Den false von unten vor die SetCenter-Methode zu verschieben.


@mikaelpatel was fairly thorough with testing of this, my W5500 changes have seen more real world usage and testing and they were derived from this W5200 source as well. I think it should be worth your while starting from this set of source mods.


Пункт в меню - годная идея, только назвать "Снять пометку", потому что про какой выбор идёт речь, может быть непонятно, а вот зелёная пометка видна. А про контрол-клик большинство не узнает.


Два действия - не гут. лень тянуться в меню.
Контрол плюс клик по кнопке снимает пользовательское скрытие/раскрытие, и возвращает его в перевозданный вид. И сразу же проверка на спеллы. Если пост подпадает под спелл, то сразу скрывается спеллом.


Или лучше меню? В принципе меню тоже норм. Я не знаю. Как Йоба захочет. Могу и я заняться этим.


Yes, just yes.


Проверка не нужна. Просто проверяешь поле `spellHidden` и всё.


Правда тогда не будет показываться имя спелла, который скрывает пост. Можно проверять повторно при `this.spellHidden` равным `true`.


Big refactoring! :)


Git commits -s


Да, два в одном по клику это очень гуд, но ведь придётся длинный тултип с инфой об этом присобачивать. Хотя если он будет только на пользовательски скрытом, то наверное норм.


Может просто двойной клик? Чтобы не тянутся к клавиатуре. Просто ставить таймаут на скрытие/раскрытие и если до того как таймаут сработал появился второй клик ничего не делать и просто снимать выделение.


Thanks a lot for all the effort on that part.


この修正で、ブロック内でのbを宣言＆定義したと認識しているのですが、何か自分が誤解しているのでしょうか？

お手数をお掛けして申し訳ありませんが、ご説明頂けないでしょうか。


Yes ^^
But it's almost only code move, no addition.


Контрол-клик уже много где используется, будет единообразно, а двойной - нигде.


Thanks! ;)


强迫症


No, not like that

```
logger = log.callback()

train_model = function(callback = logger$callback)
metric = logger$get()
```


Welcome ^^
I've achieved what I wanted in using the factories ie only have functional code in the room/trap and the building part in the corresponding factory :)


Caspiterina @GigaDev90 we forgot to initialize those variables, grazie @jackpoz for having done it :+1:


Couldn't it be `int` instead of `Integer`? The port shouldn't be rather null.

Btw, I've sent it already via mail, but cannot find it.


That's because it's a generic boolean. It's use depends on the CreatureActionType. In reality, the best would be to use some polymorphism here. I will open an issue for that.


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


Yes. As I said, using polymorphism should remove the confusion. I will have a look soon.


And why would I sign it?


Delete blank line.


Great! you can even close an issue by using some syntax in the commit comment. I think Fixed #{issue} does that :-)


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 53](https://teamcity.labkey.org/viewLog.html?buildId=285336&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:07


Is it possible to move prism.js into a separate file that can be included optionally? I already include Prism.js outside of uno-zen and use plugins that aren't included in the version you include. I don't know if by adding them again will cause errors.


yep. no more concerns
2015年11月14日 上午10:09，"Ola Bini" <notifications@github.com>写道：

> OK, so your concern here is resolved?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twstrike/otr3/commit/7c27d600436986a7dfefc6d74ca70ecd69dc57b9#commitcomment-14397740>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5211](https://teamcity.mesosphere.io/viewLog.html?buildId=75952&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:05:02


Where do you plan to pass the arguments like `train_metric`?

You think what I done here is hacky, because by default we can not pass arguments by reference in R.

```
> test <- function(x) {x <<- 1}
> a <- 10
> test(a)
> a
[1] 10
> x <- 10
> test(x)
> x
[1] 1
```

What I have done here is one of the common tricks to pass argument by reference.


Алсо, быстрореквест - сделайте плиз, чтоб при закрытии постформы крестиком она при этом очищалась. (А по "Ответить на пост" и "Скрыть форму" - нет.) Это удобнее чем Ctrl-A + Del нажимать при необходимости её очистить, и при этом логично - раз крестом закрываешь, то содержимое не нужно.


...but only when running the packaged release. I wonder what's different when running from inside my IDE.


mmk fine


Will this work better? Maybe I was in Master


Вот запилил по двойному клику: https://gist.github.com/Y0ba/86a389c70b2c84657618/raw/dd89a5eee706b877a4cf91e89b3d1c6582a01dea/DET.user.js

Двойной клик работает только на раскрытых постах. Алсо не запили синхронизацию и сделал поверх старой версии без этого коммита.

@SthephanShinkufag ну как тебе ^


Можно таймаут ожидания сделать больше, если 150мс мало.


That is what I mean, do not pass train_metric by reference, but put it as ```tmp``` inside the closure, which can be updated by reference using ```<<-```, and returned by ```get()```


Git commits -s


Sorry I still don’t understand. How do users get the train_metric in the last?

> On Nov 15, 2015, at 5:40 PM, Tianqi Chen <notifications@github.com> wrote:
>
> That is what I mean, do not pass train_metric by reference, but put it as tmp inside the closure, which can be updated by reference using <<-, and returned by get()
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/thirdwing/mxnet/commit/1deb2fa0dcc0ac4f3e0ac9bf23554b167d0cd7a0#commitcomment-14404271>.
>


Ya might dig [npmrank](https://github.com/anvaka/npmrank) which can be used to get this kind of data too.


You need to show() the email field again if they change back to any of the other fields after selecting inpatient.


`center = Vec3(0.0f);` will indeed create a new object and assign it, though the generated code will be the same as assigning every component.

I doubt `center *= 0.0f` will be optimized to zero-assignment, since with floats it will still need to check for special values, (ie: NaN, ±Inf)


patch is not doing what you advertise. you **are** saving draw scores, but with VALUE_NONE = 32002, instead of VALUE_DRAW = 0.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5212](https://teamcity.mesosphere.io/viewLog.html?buildId=75953&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 199 Build time: 00:05:08


@josevalim do you recall which was the fix for this? We are revisiting this piece on #3774 and I wonder how the changes will affect the issue mentioned by @jDeppen here.


What is this getting us?


```r
get.logger = function() {
     metric = 0
    callback(env) {
        res = env$train.metric$get()
        metric<<- append(metric, res)
    }
    get() {
        return(metric)
    }
   return(list(callback=callback, get=get))
}

logger = get.logger()
tran_model = do.train(callback=logger$callback)
train_metric = logger$get()
```


I think I created some problem, there are two master branches, do you know how I can merge your changes?


Ok, next time I will be more precise!


why do we want a center_vertical here? It is height==wrap_content


see above


Maybe reference class is another simply way to deal with this problem
http://adv-r.had.co.nz/OO-essentials.html


Hi @golosio, apparently your branch does not contain the code from my pull request anymore. I can't make a new pull request right now as this would conflict with some other code, apparently. I'll try at some point between tomorrow and wednesday to make a new pull request which should solve the problem.


it's not "layoutGravity" - this aligns the text with the drawable.
hold on I'm about to post some screenshots


Is it auth against docker hub or private registry?


ah the images - then it makes sense - ack


It's a version converted directly from CoffeeScript source. That's why this code may seem so dirty.


How can I get the API Key?


do you understand even vaguely how search works ? here you store a draw with +32002. what happens next time you probe the TT and prune with it ? you end up seeing "wins" and "losses" +/-32002 with draws from the TT.


 >> apparently your branch does not contain the code from my pull
request anymore
yes, I see, but I don't understand why
 >> I'll try at some point between tomorrow and wednesday to make a new
pull request
ok, thank you!

On 11/15/2015 11:51 PM, Alfio Emanuele wrote:
>
> Hi @golosio <https://github.com/golosio>, apparently your branch does
> not contain the code from my pull request anymore. I can't make a new
> pull request right now as this would conflict with some other code,
> apparently. I'll try at some point between tomorrow and wednesday to
> make a new pull request which should solve the problem.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AlfioEmanueleFresta/annabell/commit/9f724b2e915a2921675bcc3b597ea3b575563152#commitcomment-14404313>.
>

--
-----
Dr. Bruno Golosio (Assistant Professor, PhD)
University of Sassari (Italy), PolComIng Department,
Section of Engineering and Information Technologies
Ph. +39 079 229587 Mob. +39 320 4220312
Home page: http://www.neuralsystems.net/~golosio/
-----


Sorry Haley for using your user name to commit that.


After you destroy systems, do they get reinitialized when requested again?


For what you changed this?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5213](https://teamcity.mesosphere.io/viewLog.html?buildId=75954&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:10


This depends how many changes we want in `model.R` and there are many ways to implement.

Right now, if passing arguments by reference like here, nothing needs to be changed in `model.R`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5214](https://teamcity.mesosphere.io/viewLog.html?buildId=75955&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:08


please do not commit noop style fixes to 1.12.  We are trying to stabilize for release and this creates noise that we don't have time to mentally filter out


Да, можно чутка побольше.


Could you add a comment to the effect that we can use use `origin` in the 12am case since the time of day will be discarded anyway? I was confused at first.


It's Gtk.FileChooserButton instead.


`CanvasLayer._canvas` is a canvas element inside the `#cr-stage` div that holds all Canvas entities.


This throws an error- no such field


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 5](https://teamcity.labkey.org/viewLog.html?buildId=285330&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 18 (2 new), passed: 29 Build time: 01:20:56
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 8 more failed tests, see build details
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5215](https://teamcity.mesosphere.io/viewLog.html?buildId=75956&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:08


session ilk satırda başlasın yoksa ilerde başını ağrıtabilir


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5216](https://teamcity.mesosphere.io/viewLog.html?buildId=75957&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:07


TeamCity LabkeyRelease15.3 :: Git sqlserver 2014 [Build 74](https://teamcity.labkey.org/viewLog.html?buildId=285340&buildTypeId=LabkeyRelease153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:33


Hi,

Nice work. Hard to find out how to boot an entire linux OS to RAM only.

I got on github to ask you some question since I did not know how otherwise. I realize you did this many year ago but maybe you can help.

1. Will this work as written on CentOS 6.5, 6.6, or 6.7?  What about CentOS 7?
2. Where do you put files?  I think the ks can go in any folder as long as I run the livecd command in the same folder.  I only need the dmsquash-live-root.patched file, not the original or the middle file, yes? What folder does that go?
3. I am using basic CentOS TFTP PXE server. Can I put the same lines in the pxelinux.cfg/default file? If not, what extra special options do I need to boot the image and kernel to RAM?  Only "toram"?

Thank you very much for your reply.
Arturo


php doc kullanımı yanlış biraz araştırabilirsin :smile:


RS Agregar interfaz gráfica componente NAND #37
RS Agregar interfaz gráfica componente OR #32
RS Agregar interfaz gráfica componente Reloj #22


RS Orden de piezas #46


login işlemini sqlde hallet php ile değil
yani
```
 select * from uyeler where kullanici_adi = ? ...
```
şeklinde olsun


The PR creates problem on Chrome / Chromium : form's fields use the whole page height.


nit: I don't think that you need to use simplejson. The builtin python json library is good enough


nit: debug -> warning or error?


Ditto


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5217](https://teamcity.mesosphere.io/viewLog.html?buildId=75958&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:08


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5218](https://teamcity.mesosphere.io/viewLog.html?buildId=75959&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:07


Is the WebGL div somewhere located in the DOM tree so we can document that?


Shouldn't there be a `return` or something else here? Otherwise you're checking the result, but not doing anything about it (there's a stray semicolon).


*the* other


Was this supposed to be 224 for Octillery?


Hi,

this is an old commit but based on it you may be able to give me some explanations:
I see that calibration was already partially part of the code (src/ANTChannel.cpp:748).
I've added FE-C calibration but without GUI (at present messages are shown as overlay on the video but calibration start automatically when device is not calibrated. This is not the best way to proceed as user should have the possibility to choose if he want to calibrate or not).
Was there a GUI for calibration at the time of this commit?
If not I propose to add a menu entry to start calibration and then if overlay is not there (due to platform or video not selected) then a popup to describe the severals steps and what user have to do.


Não vão dar mais suporte? A gente estava pensando em usar isso pra evitar erros bobos no formato das contas. Recomendam algo?


Endorse. I wouldn't have thought to put down structured metadata, which I like. However, I wouldn't want to imply that there isn't a place for unstructured, narrative explanations for the inevitable weird parts of data set that require it.


After running `./configure`, the file *znc.service* contains literal `ExecStart=${exec_prefix}/bin/znc -f`. This is not valid for a systemd service file. And I got the same invalid line installed on my system (this is Arch Linux).
Seems broken to me.

See also [comments](https://aur.archlinux.org/packages/znc-git/#news) and [workaround](https://aur.archlinux.org/cgit/aur.git/commit/?h=znc-git&id=32e142b60b6ad0d58093a1f1e5deba27f121121a) at AUR.

@Barthalion what do you think?


We need to come back to this in v4, and include Jon Escombe in the design.
We want to change the flow to connect>start>pause/calibrate>stop>restart/disconnect


I am working on an 1.12 fix for the scratching settings Bug #1482163 and Bug #829538. Having these changes in the pull request will make it hard to read. So I have decided to commit it directly.


Format!!!!!!!!!


Missing space between the selector and the opening curly brace.


Окей, але з цього скорочення це не зрозуміло. Або пиши повністю, або взагалі не пиши.


maybe it will not be the same for ANT FE-C trainers calibration


Usually we use `-` in class names rather than `_`


missing v :)


bug fixes need to be as small and targeted as possible for the release branch.  Please do not also include style fixes, it makes things much harder to review.  If the fix you have includes style fixes, revert the style fixes and just include the bug fixes.  Please do this out of respect for the time of the other people who need to review the code and make sure the release is stable.