The usual reasons for persistent cross-track error are AHRS roll offset (different between AHRS zero and wings level) and vehicle sideslip (poor rudder trim). This is usually in to be in the same direction, i.e. it tends to fly to the same side of the track between turns, resetting the integrator throws this information away and requires it to be learnt each time.

Ones solution would be to apply a clamp to the integrator input that prevents the integrator winding up in the direction of the demanded cross track acceleration if it exceeds an equivalent bank angle demand of TBD degrees. This would effectively clamp the integrator whilst acquiring the next track, but still retain the learned offset.


WTF? why push this to master? Did we not agree to issue pull requests?


TeamCity pull requests :: netty [Build 255](https://garage.netty.io/teamcity/viewLog.html?buildId=1578&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4765, ignored: 21 Build time: 00:28:57


It might be best to return `Collections.emptyList()` from here, the [`dropCookiesByRegexArray`](https://github.com/Adobe-Consulting-Services/acs-aem-commons/commit/bbab6614c18119c4fd44895aa83f54026faf9c58#diff-15e58f1aa2096800fd143b65cb76c912R143) method adds the result of this to a `List<Cookie>` directly.


> I don't see linesize updated here, only the samples.

why should it be updated? assume linesize is 2000 and we want to skip 10 bytes. in this case we copy 1990 bytes. does not matter if used samples are only 20 or 40.


てすや


See previous comment wrt limiting the input of an integrator.


OK I see your point. I think this is because you don't have any HTablePool in james-mailbox.


Есть небольшая разница между 0 и 0.0:
0 - это целое число, а 0.0 - действительное.
Поэтому запись float D = 0 сначала создаст целый ноль и лишь потом приведет его к действительному и запишет в D.
Все действительные переменные лучше инициализировать действительными значениями.
Вместо 0 лучше написать "0.0", хотя будут работать и записивида "0." или даже ".0"


Hmm, I didn't think pk_backend_job_get_proxy_http() returned a strdup'd version. I guess it shouldn't really... the PkBackendJob is going to outlive the thread it creates.


てすとだ


`<mof-file-uri>` => `<meta-mof-file-uri>`?


sending the message as str() dictionary would have been a better choice, as it would be more flexible and extendable.For example, if you would have to re-arrange this message, you would also have to change the controller. Plus, on the controller side, it would have been much easier to have stuff like this:

    timestamp = message['timestamp']
    metric_value = message[''metric_value']
    metric_type = message['metric_type']
    ...


Not quite. The cmake scrips must be *supported* by the build server (for obvious reasons) The backport and contribs are for a wide support. So the build server works (boost 1.55 is max on quite some linux distros using the package manager) and developers don't need to gather all the components that are required. (check e.g. the msvc contrib archive, it is obviously not for the build server but for the windows devs)
But of course they should also work for developers. There are some issues left which is no surprise but we try to iron them out (as e.g. now boost 1.56 is supported)
Using static boost libs is preferred (you can distribute your binaries e.g. to a friend without needing him to install something) but dynamic ones should also be supported if a dev chooses so.


Add some more?


Почему именно -3 ?
Вообще принято возвращать -1 в случае ошибки. Ну или можно сделать дефайн
#define ERROR_CODE -3
и использовать вместо числа его


Was ich bisher gesehen habe, wuerde es reichen das "geo" aus dem Servernamen der Download-URL zu loeschen. Finde allerdings gerade kein Beispiel. Scheint eh nicht immer aktiv zu sein.


In the last comment, I said to add one for the `tests` directory as well.


Because it is a drop, and it is not made from a dirt block.

On Wed, Sep 23, 2015 at 8:18 PM, TrueArstotzkan <notifications@github.com>
wrote:

> What's this?
> Why is there a need to remove the DemonSkin recipe?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/incaterradev/Incaterra/commit/e0626dbec1c163a60173b550a6a568e7040116a2#commitcomment-13384982>
> .
>


![Uploading 123.gif…]()


。。我加完了，看起来好像可以弄一下，不知道行不行


Correct, though I think with the fixed callbacks (that don't call N times for N plugins) this code may not be needed afterall.


@afnogueira
Looping over the calls permits you tu build the progress bar as soon as the first data arrive and defer the render of the other bars.
It's very usefull when you have tags with a lot of executions and the servlet answer take a lot of time. If you do just one call, it blocks the page rendering


Seems to work, I can 5 travis builds and none failed.


no need to convert it to float.


Ups, gerade erst den Commit davor gesehen :)


But I'll double it, just to be sure


We do not support higher version than 14.04 LTS, right?


Fair enougth!


Это плохое архитектурное решение. Очень. Подумайте как исправить.


Waere praktisch, wenn das konfigurierbar waere, oder? Ich glaube es wird doch langsam Zeit fuer eine Config-Datei.


Oops, I mixed up the issue number. I meant #24 here.


Maybe a short explanation for the fields (took me a while to figure them out)?
- `ImportDeclarationType` ^= hidden or explicitly named imports
- `Maybe ModuleName` ^= eventual qualifier


>Remove unapproved alias /personalroom
You didn't say it had to be gone :|


This change looks unrelated and unintended


Nope, see the issue is here:

```
class CSoundPacket
{
public:
  CSoundPacket(SampleConfig conf, int samples);
  ~CSoundPacket();
  uint8_t **data;                        // array with pointers to planes of data
  SampleConfig config;
  int bytes_per_sample;                  // bytes per sample and per channel
  int linesize;                          // see ffmpeg, required for planar formats
  int planes;                            // 1 for non planar formats, #channels for planar
  int nb_samples;                        // number of frames used
  int max_nb_samples;                    // max number of frames this packet can hold
};
```

This sound packet was made to be read right from the beginning, e.g. the data ptrs are continuous. The only way to stop reading is  when ``nb_samples`` is reached. Now when you want to skip data out of it, you can only reduce ``np_samples`` but therefore the data must be moved, e.g. data is skipped by moving the real data minus the offset to the front.

You are right with the linesize, though - it is quite inefficient. As if the Packet is let's say always only "half filled", you would always copy arround half the buffer for skipping just 20 samples.

The real problem is the SoundPacket - perhaps it should get an additional plane_index? which is zero by default, but when we want to skip, we increase that one and whenever someone is reading from the planes this index is just added on top as an offset? That way no move needs to happen.


YES, 9.0 is also failed.


Ah thats cool I didnt realise Modernizr had that functionality


ArrayList is bigger than necessary now


Подумайте, как сделать вывод сообщения об ошибке более универсальным (для логина и регистрации). Загляните в топики нашей группы вк, я там некоторым студентам отписывал возможное решение.


table names should not contain spaces or special characters as /. Also, they typically are lowercase. The __tablename__ maps to the actual table name in mysql / mssql / plsql or whatever, and they probably don't support such table names.


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


right-hand ?


skipping less than nb_samples happens a single time during sync. I don't see an issue here.


Look a lot better now. I still have to review code and unit tests, but until now, there aren't any major flaws, mostly suggestions / ideas.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for autocompleteFor


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillReceiveProps


**Eslint**: 'return' outside of function


Okay - so basically: not worth the hassle?


Haha, that was stupid.


nope :)


The question is how `g_string_new` behaves. I read [this](https://developer.gnome.org/glib/unstable/glib-Strings.html#g-string-new) as it makes a copy.
In any case one or another place is wrong.


Oki :-) then let's memmove. If we some day have some million samples in such a class, we can revisit :-)


set two commands, how about using the code block enclosed between pair of triple back-ticks ?
```
docker pull centos
docker run -t -i centos /bin/bash
```


You can get the context from the view.


@sindresorhus btw, we appreciate you helping us improve the dependency situation with our tools. Much :heart_eyes:


Using the box from the Host


public function setHttpClient(Client $httpClient)


Pretty thorough test.  Here's some feedback:

* Are num_write and num_read swapped?  I think that num_write should be 8, not 5.
* Please rename psync; since psync means something special in SHMEM and this is not a psync array.
* Target, source, and psync should be statically initialized to something invalid, like -1.
* If you require that each test touch different buffers, this should be captured in a comment.
* The argument processing is kind of complex.  You could use getopt for processing the args to simplify things.  You could make each test take a number of iterations argument, and capture this in putfence_niter, gettest_niter, ...  variables that start out with a default value.  You can have a '-a' option to run all with a specific number of iterations.  Process all of the arguments at once and then proceed to code that runs the tests.  Getopt is part of libc and is really easy to use.
* You could pass the __func__ string to pre_op_check.  You can also pass __FILE__ and __LINE__ to get more detailed debugging messages.
* Check indentation.  putfence has some bad indentation.
* Is there a race in gettest, between PE 0 initializing source[] and PE 1 reading it?
* Line 131: put iterations, instead of i?  I know they are the same, but the former seems to self-document better.
* Line 221: The extra odd/even variables are unnecessary, "x % 2 == 0" should be clear to any reasonable reader.  You could add a comment to make it clearer, but the extra variables are overkill.
* Instead of calling exit(), you should call shmem_global_exit().


I DONT LIKE THIS ITS TERRIBLE


URGHHH SO BAD


LEARN TO BE GOOD


U MAD BROOOOOOOOO


IM SO MUCH BETTER


M9 GET REKT


麻烦尽快让支持IOS9  和 安卓5.1 等新系统 。


Not even Readline 0/10


M9 YOU SUCKKKKKKKKKK


Looking at the implementation of `DefaultConvert` it seems that it would work since it converts based on types but wouldn't it be better to keep the conversion explicit here? Not a strong preference, just a thought.


0.5/10 IGN


Read/write more common files :

case x if x.endsWith(".csv") | x.endsWith(".nlogo") | x.endsWith(".nls") | x.endsWith(".py") | x.endsWith(".R") | x.endsWith(".gaml") | x.endsWith(".txt") ⇒ TEXT


METACRITIC SCORE 0


comment for cosmetic change: how about adding the code block for it?

```
 config.vm.network "forwarded_port", guest: 2376, host: 2379, auto_correct: true
```


Hi am thanks but I am just wondering that there is no app.yaml and php.ini files generated should I add them manually. I also have memcached installed but have the error in the error as below pls help
![mem](https://cloud.githubusercontent.com/assets/10448809/10041843/4c313b4c-61df-11e5-9cb2-655f05b025b9.JPG)


heyyyyyyyyyyyyy


#TURNIP IS  LIE


> While that assertion would currently be correct, if any flow names were duplicated between smartdown and smart answers, it would no longer be correct.

Indeed, but ideally that'd be the prompt we need to write the test properly and check the behaviour we're interested in (i.e. that it excludes duplicates).

> I think I'd go with any or even just assert that the method can be called without an exception...

I think my main problem is that I don't think it's too much work to write a useful test and that I'm not sure going for one of these easier options is necessarily the right thing to do.


I wanted to commit to supporting `/makechatroom`, but nothing else until we could come to an agreement.


#WE HATE TURNIPS


#TURNIPBANTER


#NOFILTER


#TURNIPSARENOTPOTATOS


Nice pic Sam


ASDA SMART PRICE WOULDNT BUY THIS


sup


Lol


We have the Project.exists_by_name? function for such checks. It might not make a difference here atm, but would be anyway better to use IMHO if we need extra checks later.


updated


Hello, I have a question related to this part of code. Why redis connects only to master redises?


#MOREACTIVETHANREDDIT


MR JARVIS HAS A NICE BEARD


#TURNIP SUPPORT


#SAVETHETURNIPS


#TURNIPFORPRESIDENT


removed this and the ensued dead code


#JOSHMHASNOLIFE


fixed


Nice, thanks!

Please not merge it as is though, the diff is huge as on top of the added documentation, this commit also syncs the classes doc with the current code base. This should be done in a separate commit for clarity (so that the actual added doc for vector2 and vector2array can be properly reviewed).

Changes like that are also somewhat puzzling:
```
-		<method name="cos">
+		<method name="cos"  >
```
@reduz Is it an intended/expected change in the doctool, or is it a bug that should be fixed prior to updating the classes reference?


#JOSHMCANTWRITELINE #NOOB #GETOFFTHECOURSE


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1350 of 1350 tests passed)

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


k


Just do `new Date()` here and call the .toISOString, etc. inside the hash function.


É porque quando o Orc ataca ele causa um certo dano, então eu deixai um método nos personagens que recebe esse dano. Na classe Elfo ele apenas herda este método, na classe Dwarf ele sobrecarrega este método e na classe Orc ele sobrescreve. Assim quando o Orc atacar informando o dano o outro personagem vai receber este dano.


测试添加商品代码


$testって名前はどうだろう


maybe coding style should used from PSR? see https://github.com/php-fig/fig-standards .


So it would return true for their home projects now even when they are not allowed to create it themselfs?


A possible option to write a useful test without stubbing would be to set the load path for the two flow registries to a fixture directory with a few very simple flows.


D'oh, forgot a couple of `with pytest.raises` there. But let's first decide whether we want `create_params()` to look like this or get an `alternative_shapes` kwarg.


It does seem funky, but unfortunately I think it's still needed for when `return-to-pool` is called on a `RetryPoisonPill` (https://github.com/rlinehan/puppet-server/commit/67afeb70eb1bf813009bdf2a8d5c28137f135eb7#diff-fa19d0dfb74026c0ab162ee83ae7b72fR306)


Would it be possible to get a release with this included?  A lot of Linux package build systems build in sandboxes without internet access for security reasons, so being able to build without internet access is critical.


V3 out now!

https://github.com/turnip99/demo/commit/723738cf5b66e8485aaf393abd7cc5063ef49da7


Looks like we need another test case where the user tries to create it's home project via the api when the config option is set to false. I can add one ...


too complicated add annotations


As Kanye West said:

> We're living the future so
> the present is our past.


Ok, will take a look at it how to make it more logical then.  <feature


+1 chrismann37


Will do m8


hey everyone go comment on this one
https://github.com/Termin8or/3DEngine


#5112-Array-bounds-error-with-ControllerMechanicalVentilation-and-no-DesignSpecificationOutdoorAir-objects (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (812 of 812 tests passed)

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


bad


@jayapalu If it's commented out, why can't it be removed?


When 0.2.6 will be available via Homebrew?


/*添加颜色*/
insert int p_color (id, qr_code, code, fabric_code, name, remark, pic, create_time) values
(555,'F201509231745000100', '101','red121', '浅橘色', '棉900%涤纶5%氨纶5%','','2015-09-23 17:45:56');


Added magic variable.


+1
We're using this class. What we should use instead ?


Hi @Snesha , the sentence after `hutil.do_exit` won't be executed. Please refer to the following link.
https://github.com/Azure/azure-linux-extensions/blob/master/Utils/HandlerUtil.py#L309


Hi @Snesha , this line should not be remove. It will update the status of the extension.


arm


Rotav m9  annotations here
https://github.com/turnip99/demo/commit/0ac5c206b31dec2c557cd626c3b8302aad70880e


Testing on `master`, `bitcoin-0.11.0`:

* `20150430` (@nckx fix) compiles
* `20150730` (after revert) fails
* `20150907` (before revert) compiles

Note that I haven't tested altcoins! I suppose most of them break with `20150907`, so we can either maintain two versions or stick with `20150430` for now.


shouldn't mods simply return `[]` in the other case?


Some of them may not need any translation anymore: all strings that contain just the Unicode sign and nothing else.
I'd love to replace double flats and double sharps with their Unicode representations too, but unfortunatly These don't render properly, they are just missing from most fonts.


@Arthur2e5

> And this thing contain some typography terms, don't translate them if you are not in the field and can't use search engines properly.

Got it. Since I translate them in school (and in offline mode), I won't translate those terms. However I do not think I *can't use search engines properly*

> AND please don't use plain text editors.

I use classrooms' computer, and extra software cannot be installed, so I use plain text editor (Notepad++) on my flash disk.

I've installed a set of `gettext` tools on my flash disk. Days after I will find a way to install a `.po` editor on my flash disk.

**P.S.** I use my little free time to do this, but if you think I'm not good enough to do this, tell me and I will stop it.


This is also a very misleading name for this test case ;)


Hyaw


Thanks this is much clearer now. Best turnip counting program on the internet.


I'm having some issues building with this change. I'm using headers with CL_VERSION_1_2 defined however clEnqueueBarrier is deprecated. The change to unconditionally use this symbol seems to be causing a problem. I think I would need to define CL_USE_DEPRECATED_OPENCL_1_1_APIS to be able to compile this with version 1.2 headers. Does the travis build use 1.1, 1.2 or both?


Thanks for the kind words!


Yes, probably no. In any case I'd leave it in place for robustness.


Does this responsiveness work also on "not-the-wallet" content areas?


Please see my comment @ ln 1208


I also removed test_random_batch because all it does is compare DataSets with different capacities to make sure they behave the same. But capacity is no longer variable.


Addressed in https://github.com/alphagov/smart-answers/commit/7bfb563b84ad6bbeeddacc9fca31a9f0f4d76104


@borisroman we can remove that.


Do we need the 'me' bit here? "google me" works, but "let me google that for you me"... not so much


Weil der Regionsbesitzer automatisch bewacht? So langsam komme ich mit den verschiedenen Regelwerken durcheinander. Also, Bewacher angreifen, es sei denn, sie sind nur deshalb Bewacher, weil sie Regionsbesitzer sind? Warum überhaupt Bewacher angreifen?
Ursprünglich war der Grund, den Bewacher anzugreifen, dass man dann Steuern eintreiben kann (Drachen haben ja früher TREIBE gemacht). Das ist jetzt vielleicht gar nicht mehr richtig.
Muss ich wohl mal drüber grübeln.


Hm, this will probably make cherry-picking commit between master and release-15.09 painful...


Glad that you did that. Looks good to me.


I did a few commits under your name too haha

Sent from my Windows Phone
________________________________
From: jpcsupplies<mailto:notifications@github.com>
Sent: 23/09/2015 7:00 PM
To: jpcsupplies/Economy_mod<mailto:Economy_mod@noreply.github.com>
Cc: Toby Wilkinson<mailto:tangentspy@outlook.com>
Subject: Re: [Economy_mod] Some debug logic for sell command (5a6f550)

woops toby was logged in at work.. that was my commit ;)

---
Reply to this email directly or view it on GitHub:
https://github.com/jpcsupplies/Economy_mod/commit/5a6f550c5037ebf47257359aaed1acc66bed5ea8#commitcomment-13385689


fdsafdsafds


この時点ではテキストが確定していないので、locationがずれる


pirou de vez


And I dropped python 3.4 support.


Should multiple ways of calling these be shown
``inc(i)``, ``inc i``, ``i.inc() ``
or keep it as just the preferred idiomatic way to help newbees to style their code the preferred way?

for len(), is it worth adding a for loop example, which is a common usage?
``for i in 0..<arr.len:
  echo arr[i]
``


nit: `; // If`


var shuffleArray = function (oldArray) {


Missing phpdoc comment.


var array = oldArray.slice(0),
  currentIndex = array.length,
  temporaryValue,
  randomIndex;


This is very good. What are you trying to say?


Thanks for figuring this out!


abc


The HTML diff here will get reverted whenever someone runs `npm run build:compilers` - the change needs to be made in `data-es6.html`.


If you dont mind me asking, What do you use to convert the image to byte array? that section to that webpage does the job, but with one problem, at the end of the file I get this:  0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x|........|,
	0x0000afc8,  <--- Now that isnt right.............. So I dont get it.. I have used everything I can think of, I got the splash to show.. and default works and just for testing, I used yours and that worked.. so its gota be something with the image format that im doing wrong. So im asking what do you use to convert your gfx over.. as well as the image format.............
thanks man..


Thanks, just want to make sure everybody understand it


actually its not white space . its typo :)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 705](https://teamcity.labkey.org/viewLog.html?buildId=273871&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:20
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Not sure if we could save this as a long since we are trying to retrieve it as a long.


@ljharb as you can see, all build after `html` docs (es5, es6, es7, intl, non-standard) contain correct background. And `compilers` not required for most cases.


@PinpointTownes thanks, the id_token can be in the ClaimsIdentity.Bootstrapcontext, so there is possibility of duplication here. Did anyone think about putting this in the bootstrapcontext? Adding 'claims' in the middleware always makes me nervous.


I was thinking either that, or whatever the unix equivalent call would be to authenticate a system user @mdickson


The compilers build is what's run every time I update the compilers results. If that's going to be wrong, for the first time ever, then the `npm run-script` should be updated to do the right thing.

There's also no other `npm run-script` to run - so I'm not familiar with the "html docs" build. Please add an `npm run-script` if it's what contributors should be using.


This is the same non-portable conversion issue we encountered before:

name = xasprintf ("%s%"PRId64"", jname, id);


Spíš bych tam napsal něco jako: „Na základě příkazu rektora č. 5/2015, Ochrana osobních údajů na ČVUT v Praze, vám tato data bohužel nemůžeme zobrazit.“
@jnv, co myslíš?


I pushed the HelloWorld file a few times.


Definitely planned


The same source and destination index for a `NSFetchedResultsChangeMove` does not indicate a bug.

Consider:

1. The FetchedResultsController initially fetches objects `A`, `D`, `E` from the store
2. Code inserts object `B` and updates object `E` to `C`, and saves
3. The FRC should see an `.Insert` at index 1 and a `.Move` from index 2 (before insertion) to index 2 (after insertion)
4. The resulting object list is `A`, `B`, `C`, `D`.

In iOS 8, the FRC would dispatch an `.Insert` and an `.Update` (instead of a `.Move`), resulting in an out-of-order object list: `A`, `B`, `D`, `C`. In other conditions, the .Update would conflict with an .Insert or .Delete, violating UIKit's "one operation per row" rule for table view updates, causing an exception to be thrown.


Detecte un problema al crear nuevas matrices con diferentes tamaños tratare de solucionarlo.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 21](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=186&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


Yes, I understand that spirit. I also understand that you shouldn't do things that don't belong to the task you're doing right now. That being said, I just don't want to explain later to collectd guys why I've made a non-specific Windows change in a Windows-specific PR. Unless you want me to actually take the time to figure out what from unistd.h is needed and why it's not needed on Linux.


Why not just call it 'port'? less typing


@ljharb I do not know what you need from me. Please, explore the project if you don't understand something.


@zloirock what command do you run that generates the es6 html file, such that this diff is not reverted?


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 21](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=186&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:02:39


Second, why is this the first time in a very long time that `npm run build:compilers` doesn't produce the correct output?


Shouldn't it reflect 3.4.0, 24th September 2015 ?


`node build`


Good catch!


TeamCity pull requests :: netty [Build 268](https://garage.netty.io/teamcity/viewLog.html?buildId=1608&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5603, ignored: 21 Build time: 00:38:50


TeamCity pull requests :: netty [Build 270](https://garage.netty.io/teamcity/viewLog.html?buildId=1619&buildTypeId=pulls_netty&guest=1) is now running


OK, so `node build` produces a diff for me as well.


You're right... bah, it was merged already.


Yeah something like ActiveDirectory interfaces in .NET I would think but I'm not familiar with what Mono implements.  Hopefully there is a way to map to PAM or some other system mechanism. I'll dig a bit more.


I've published the new output. Please be sure to run `node build` and commit the output before pushing your commits.


Буде конфлікт, бо будуть дві find links адреси, і гарантувати потім звідки
береться пакет буде непросто

On Thursday, 24 September 2015, selurvedu <notifications@github.com> wrote:

> Тобто мати і те, і інше з різними URL'ами не можна?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/selurvedu/openprocurement.chronograph/commit/b80b2c2d9ca44a8b8a6d8cdedaca3289cb270ae2#commitcomment-13402557>
> .
>

--
....................................................................................................................................
Myroslav Opyr   ▪   CTO   ▪    Quintagroup   ▪   +1.917.475.4725   ▪
http://quintagroup.com
˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙


We'd like to have signal-like support for Windows also. That being said, I might just extract this piece of code to another function and guard the call with HAVE_SIGNAL.


Run `node build` one more time ;)


um - why is the output non-deterministic? Every time I run it it flips.


Absolutely. My intention was simply to fix broken things first. I believe this is worth a ticket in vagrant-libvirt issue tracker.


`node build` should always produce the exact same result. I'm not sure why that suddenly isn't the case, but we need to fix that.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 706](https://teamcity.labkey.org/viewLog.html?buildId=273879&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 34, passed: 9 Build time: 00:25:33
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Hmm. OK, the data is getting in there. Weird formatting. And I can't yet figure out how to intersperse Historical and Personal. Would you think less of me if I didn't bother, and kept them separate?


No ideas. I noticed it some days ago. I will explore it when I will have some time. Or you can do it :)


![screen shot 2015-09-23 at 8 32 16 pm](https://cloud.githubusercontent.com/assets/8370772/10062620/601f78dc-6232-11e5-8398-fe3fb8c1caa6.png)


Configurado uma nova classe no arquivo models


This is breaking my build.
```
Microsoft.Dnx.Compilation.CSharp.RoslynCompilationException: D:\EntityFramework\test\EntityFramework.Sqlite.FunctionalTests\SqliteTestStore.cs(127,17): DNX,Version=v4.5.1 error CS0117: 'SqliteConnectionStringBuilder' does not contain a definition for 'Cache'
D:\EntityFramework\test\EntityFramework.Sqlite.FunctionalTests\SqliteTestStore.cs(127,39): DNX,Version=v4.5.1 error CS0103: The name 'SqliteConnectionCacheMode' does not exist in the current context
D:\EntityFramework\test\EntityFramework.Sqlite.FunctionalTests\SqliteTestStore.cs(127,74): DNX,Version=v4.5.1 error CS0103: The name 'SqliteConnectionCacheMode' does not exist in the current context
```

Any fix to resolve this?


Hi,

You are perfectly right, I thought I add the volatile keyword on this reference.

I don't have the code there but from what I remember having the synchronized keyword do not prevent a thread to get the reference and use it while another one perform a logout.
That's why I though this keyword was not so smart and better to avoid.

On the other side, at last on x86, the volatile or synchronized word will generate a memory barrier asm instruction only when performing a write, not a read. But the volatile hint is need to be sure that the reference will be read from memory like L1 cache and not keep on a register.
In fact we may have the bug you say, ie not having the reference change visible to the thread that send a message, only if he do very few things between two calls to send(). Something like sending messages in a loop, which is a non sens. But even in that case I'm sure that there is enough instructions done by the thread to perform the send(), to prevent him to keep the reference on a register... The whole code beneath the send() is far too big to have this kind of optimization.
But yes, volatile keyword is the correct and right thing to do.
Sorry for the mistake !!

> Le 24 sept. 2015 à 01:33, Chris <notifications@github.com> a écrit :
>
> At a guess the sync on the responderLock was trying to achieve a volatile read (it does look a bit odd). Are we ok with removing this ? i.e. can another thread set responder in which its change might not be visible to another thread calling this method.
> Should we consider if responder should be volatile or an AtomicReference ?
>
> —
> Reply to this email directly or view it on GitHub.
>


@smitpatel Switch the code back temporarily in your local copy while we wait for a coherence build to pass.


TeamCity pull requests :: netty [Build 270](https://garage.netty.io/teamcity/viewLog.html?buildId=1619&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4783, ignored: 16 Build time: 00:10:52
### Failed tests
```
io.netty.resolver.dns.DnsNameResolverTest.testNegativeTtl: <no details avaliable>

```


@xDefine I just updated the code to work properly.


97668326-WholeBuildingLoad (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1884 of 1892 tests passed)

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


increment RELEASE yo


@Brit200313 better. Still a little weird.
![screen shot 2015-09-23 at 8 42 50 pm](https://cloud.githubusercontent.com/assets/8370772/10062710/bf7dce2c-6233-11e5-88b5-6ad7b25644a7.png)


@wido could add a label before `Mirrors` like:
```rst
.. _mirrors:

Mirrors
=======
```

and reference it later:
```rst
.. tip:: For international users, there are various mirrors globally. See :ref:`mirrors`.
```


@henare I feel like this could do with another sentence about why this is interesting—why did you add this item? ;-)


#4298-Recurring-error-with-Heat-Exchanger-when-using-cycling-fan-system (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1355 of 1355 tests passed)

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


This was intentionally left as implemented before passing the year.  The year should be passed in for all use cases, but I had to ensure I did not disrupt existing code that was too obfuscated to identify.  Honestly, I don't believe any of these parameters should have defaults.


The description lies.  Working more correct info based on first re-generating all pages with no source changes...


Adding this line causes the render_in_browser() function unable to work. self.render_tree() function takes no arguments, but In line 84, kwargs is passed into this function, which leads to a TypeError:render_tree() got an unexpected keyword argument 'force_uri_protocol'


fixed in 57310f0


Heh... Oops


빠른 대응 감사합니다 :+1:


#4402-VRF-outputs-and-multiplied-floor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Renamed to VerilatedScopeNameMap because that's what things seem to be named up here.


Looks like this line was mistakenly left in.


Is this used anywhere?


No longer need the return here.


continue instead of return?


74564912Issue4347BBRadConvHW (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1889 of 1892 tests passed)

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


I'm never sure what to do about formatting quandaries like this.  I prefer not to touch lines I don't have to, but if desired, I can normalize.


For an email, it actually makes a lot of sense to keep them separate. On the index page on the app, I would expect them to be interwoven.


Had to come along for the ride.


Yeah, when I looked at it I said "hmm, that's cool, actually".


Hey man, this is very close... the only issue is that since the function callback isn't bound to anything "this" on lines 1131 and 1133 aren't the right object either. If you don't support IE8 you could rewrite it as

```
    this.modifySourcesQueue.push(function() {
         console.log('modify sources done');

         callback();

         var newSdp = new SDP(this.peerconnection.localDescription.sdp);
         console.log("SDPs", oldSdp, newSdp);
         this.notifyMySSRCUpdate(oldSdp, newSdp);
     }.bind(this));

```


I meant 0.15.1


@smitpatel you can also pull the latest Microsoft.Data.Sqlite (≥ 15626) from CI drops (or the aspnetvolatiledev feed).


Python 2's `zip` will eagerly create a list, which is unnecessary here. I don't think `six.moves.zip` should be a problem, and it definitely seems to exist in the packaged version of `six.py`.


@udaysagar2177 This should be empty right?


I think this should still be restart.


We don't have originsnap, which appears to be related to some previous snapshot-patch we do not yet have.


cant get the source code 404 not found


corrects stupid mistake in `8166d47` caught by ci


Thank you for this fix. Would mind submitting a PR to the upstream repo so that we may merge? Sorry for the issues. https://github.com/blueboxgroup/knife-bluebox


TeamCity pull requests :: netty [Build 269](https://garage.netty.io/teamcity/viewLog.html?buildId=1618&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4765, ignored: 21 Build time: 01:02:20


Many of these scripts (see Blackwing Descent, Bastion of Twilight, some from Firelands, some of the dungeons) are written or remade exclusively by me and once belonged to the leaked 4.3.4 SkyMist source :)). Nice to see they are still going around, even if the headers are gone from passing through countless hands of people that have assumed credit for or sold my work :)).


I believe this was supposed to become a standalone project - curious what @markpollack still thinks.  It could live in Spring Cloud Stream after all, but we may want to move it to a higher level, for wider availability.


Yes, which makes for even more duplicate code, and is an even more compelling argument for sharing one function.


Dang.  Yep.


VerilatedScopeNameMap accessed here.


![screen shot 2015-09-23 at 8 59 45 pm](https://cloud.githubusercontent.com/assets/8370772/10062942/23c58ec2-6236-11e5-8cf2-d73686815515.png)


Oops, meant to remove this, using rel_filepath.


I believe this needs to be configurable per-binding (just like partitioning and everything else).


close #9


It's not really pseudocode, but it contains the logic of the program. Executing it won't do anything since none of the functions are defined yet.


@underscorediscovery do you know if result is an absolute filename or a relative one? If relative, it could be relative to the CWD that was set by the dialog. You might need to prefix the new CWD to the result before returning it.


#4402-VRF-outputs-and-multiplied-floor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1886 of 1892 tests passed)

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


D'oh, yes.


Yes.


97668326-WholeBuildingLoad (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1884 of 1892 tests passed)

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


VERSION and RELEASE are not the same thing. VERSION is for the software and RELEASE is for RPM packaging (the .spec file).


I think we should do this wrapping in the `ChannelBindingService` who already has access to the `ChannelBindingProperties`.  Or ideally in the Binder implementation, perhaps as high as `MessageChannelBinderSupport`.


@dmnsgn Temporary, I fixed PostCSS version to 5.0.5 to pass the tests. and just released CSSfmt v1.3.4.


update babel plugin config to use new react-transform syntax (see [Release v1.1.0](https://github.com/gaearon/babel-plugin-react-transform/releases/tag/v1.1.0))


Should this logic be here or should it be part of the type ? Each type contains it's own serialization/deserialization logic.


It's very unlikely, but we'll solve that problem when/if it actually exists.


Is this commit only related to UKSM? Can I use this patch as a port of uksm to 4.1?


Calling invert() on your background color gives a nicely complimentary font color that really stands out.


Setting entries for feature requested in https://github.com/KC3Kai/KC3Kai/issues/690


This commit (and I believe this line) leads to an infinite loop on Mac and breaks load_cfl calls. According to  [This StackOverflow page](http://stackoverflow.com/questions/11926883/scanf-dynamic-allocation), `%m` is a GNU extension.

Why not allocate temporary memory and have a policy for longest allowable keyword length?


I wonder if we can merge this. Coverage is 77%

I quickly reviewed the specs; here's a strange one:

![screen shot 2015-09-23 at 9 10 05 pm](https://cloud.githubusercontent.com/assets/8370772/10063104/00d4b4a4-6238-11e5-8cde-89d4b8aef684.png)


The existing message feature does support user->patient and vice versa messaging. To see how this works in the pnotes table and the OpenEMR messaging gui, create a patient message on ZH's offsite portal on the 4.1.0 demo. ZH also stuck a encryption flag on the sql table to track whether message is encrypted. All that is missing is a gui on the patient portal to support this, which Terry has been working on:
https://github.com/teryhill/openemr/commits/patient_portal_messages

Regarding deleting, is there a work around to salvage that?

let me know your thoughts


(The main trouble with that is that the convertor is set during the channel creation process, so it seems like some refactoring would be necessary) - but in general, if the configuration was per-binding, we could avoid this check and the subsequent environment resolution.


@latobarita: retry


oniichan uses the dank branch so the memes live on


comment test


Watch out for the difference between == and =.


You need to make sure the formatting of the date matches. Change it to eq("Thu, 20 Aug 2015").

I'll merge it if you submit a PR with this test fixed and your current Simplecov score.


please go and do a pull request to the main website :)
looks good to me :) thanks


## This is a markdown


This part of notice is used to specify that some other OS/hardware may work too but we did not test that.
I change it to

~~~~
The New OS and New Hardware which listed in the table have been fully tested. The OS which comes with the same source code or Hardware comes with the same CPU should also work, but you need to try it by yourself.
~~~~


Had to handle this similarly on the test_perf_tracking branch, I'll make mine copacetic.


B/c database interprets null and false as the same value:
  - add default value of false on questions table multiple_choice column
  - remove presence of multiple_choice validation from Question model


Count your parentheses.


Why the change to the price * 1000?


You're not using 'num' anywhere else in your code.


我打算不在列表给网络链接，要么不给，要么给文档内部的链接


Your last function should be named 'sum_series' or else the pytest won't recognize it.


I'm a bit surprised how little the parameters moved from the starting values.


Hmmm. I'm building with 5.2 SM.


Comments please.


我想如果一个数据源需要详细介绍，那么可以考虑一个数据源一个你博客的链接。比如IRIS的介绍，你有多篇博文，最好你那边可以有一个 集中 的博文接手，不知道表达清楚没有


Hmm. Well, I couldn't fix it! My coverage is 77.78%

![screen shot 2015-09-23 at 9 24 16 pm](https://cloud.githubusercontent.com/assets/8370772/10063255/aa4614c8-6239-11e5-8f2b-cc6482c6c8b8.png)


Well on kernel side. Still 4.8 on ROM


The way this block is indented, it's going to run only as part of your else statement on line 46.  Is that when you want it to run?


You're welcome :P. By the way, this is based on the leaked 4.3.4 SkyMist core so i know what i coded here for a few years, if you need any info let me know :). Beware archaeology and other things were not completely implemented at the time.


because DailyBarWriterFromDataFrames itself multiplies the relevant fields by 1000, so we don't need to do it.


This is clever!


One possible solution is for the `ChannelBinderService` to check whether the channel being bound is an `AbstractMessageChannel` and call `setConverter()` on it. If it isn't, it can bridge it with a converting transformer. Thus we are covered in the most likely case, and can accommodate more exotic situations that can occur in the future.

As the values under `bindings.*` continue proliferate, I think  that we need to introduce (*outside* this PR) a type safe BindingProperties object as the map value, and use a Converter to populate said value when we use the short form `bindings.output=`. I don't think that it needs to be part of this PR, but added it as a reminder.


#4402-VRF-outputs-and-multiplied-floor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (818 of 818 tests passed)

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


Benign but good to know: http://stackoverflow.com/a/9853116/1597274


For this test model, the compiler actually produces:

```
Constraint(equal(min(joinRef(global(c0_A))), max(joinRef(global(c0_B)))));
```

However, when I try to instantiate this model, I get

```
Unhandled compilation error occured. Please report this problem.
Can't find method org.clafer.ast.Asts.equal(org.mozilla.javascript.Undefined,org.mozilla.javascript.Undefined). (minAndMax.js#10)
```
Line 10 is the constraint above.


@blackbbc 这边需要单独开一个权限


Here's the spec. Looking closer now.

```
describe "POST #update" do
    it "valid params" do
      @person = FactoryGirl.create(:person, user_id: @user.id, name: "Laura Vandervoort")
      @personal_event = FactoryGirl.create(:personal_event, person_id: @person.id)
      post :update, id: @personal_event.id, personal_event: {description: "MyNewString"}, format: :json
      expect(flash["notice"]).to eq("Your event was updated.")
      expect(@personal_event.reload.description.to_s).to include("MyNewString")
      expect(@personal_event.reload.starting).to eq("Thu, 20 Aug 2015")
    end
  end
```


What's the `_runtimeEnv.OperatingSystem == "Windows"` part doing?


Don't touch this part of the mono thing. I'm making changes here now that @BrennanConroy 's changes are in


Added an issue for this commit:
https://github.com/CM-zenfone2/android_device_asus_mofd-common/issues/7


你好！！！


Use labels to break out of outer loops: http://stackoverflow.com/a/886979/1597274


`vertical = (y1 != y2);`


@tomantonyp Eventually the change will be pushed to http://generator.swagger.io, which is used by http://editor.swagger.io. Currently, there's no regular schedule to deploy the latest master of https://github.com/swagger-api/swagger-codegen to editor.swagger.io.

For the time being, please pull the latest swagger-codegen (master) if you need the latest bug fixes and enhancements.

You can even use your own generator server (at port 8001 by default) as follows:
```
git clone https://github.com/swagger-api/swagger-codegen.git
cd swagger-codegen/modules/swagger-generator
mvn jetty:run
```

To use generator.swagger.io or your own generator server with curl, please try the following command:
```
curl -X POST -H "content-type:application/json" -d '{"swaggerUrl":"http://petstore.swagger.io/v2/swagger.json"}' http://generator.swagger.io/api/gen/clients/php
```
Please let us know if you've further questions.


Whatever formatting you are applying to @personal_event.starting, do the same to "2015-08-30". Put them on the same playing field so they will equal one another.

Headed to bed. G'night!


You need the commits in this topic:

http://review.cyanogenmod.org/#/q/status:open+topic:quick-charge-ui


@latobarita: retry


Hey @juliocamarero

when enable staging, the asset publisher will show two same web contents, one is from Live site, Another is from Staging site, this will be strange, why the staging site is referred to a live site.

so in order to prevent this situation, currently in asset publisher configurations, we are only allow its own and Global site.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/458e87211eb04c83b443f97c269294d479565594


merging graydon/stellar-core/dh-based-mutual-auth = 458e8721 into auto


graydon/stellar-core/dh-based-mutual-auth = 458e8721 merged ok, testing candidate = 1366d513


Good night.

I don't understand, so I think I'll move on and do a PR. I'm not applying any formatting, I don't think, since it is at the controller level. I apply different formats in different views using methods from PersonalEventsHelper.

Anyway, I can work on that **whenever** gem and *thegravelpitband*.

thanks for your help.


^ OOPS! I mean "move on and **not** do a PR"


Why +50?  A bit of documentation about what this is for would be nice.


`sortFacesByMaterialIndex()` and `materialIndexSort()`, instead?


Ok, let's do this, then: add a TODO right before the #include that this header may not be Windows-specific to let the maintainers know about this, and let them make this decision (now or after the merge).


102966118-HeatRecoveryBypass-4568 (Myoldmopar) - i386-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-85.57%25-red.svg)


so in order to prevent this situation, currently in asset publisher configurations, we are only allow its own and Global site.

so prevent the user changing from UI side.


This should not have been committed. It'll break our sonatype publishing.


You are right, I will learn how to write ReadMe just today.


Cara menyelesaikan persamaan akar-akar kuadrat
Pseudocode
1. start
2. input a,b,c
3. diskriminan b^2-4*a*c
4. disk >0 ?
5. =0
6. x1=-b/2*a   x2=x1
7. >0
8. x1=-b+sqrt b^2-4*a*c/2*a  x2=-b-sqrt b^2-4*a*c/2*a
9. <0
10. akar imajiner
11. finish


sdfghkl;


A custom Fibonacci-Sequence maker:
def cust_fibb(num1, num2, length):
        base = [num1, num2]
        for i in range(length-2):
                base.append(base[i]+base[(i+1)])
        print(base)


Thanks.  Given the projects it lands on, I will have to figure out a strategy in the coming days to make sure the cherry pick works across ROMs.


#Yay!


Thanks @risacher .

I found CLI option problem reported in #9 was still reproduced with pagent.exe of 6645bcf on https://risacher.org/putty-cac/testing/ .


hey, i know you


Only for methods? lookupThis is called for fields as well.
Is lookupThis specific to features?


strange variable name - this isn't a FIeld


asdasdad


extra 'to' here.


TeamCity pull requests :: netty [Build 271](https://garage.netty.io/teamcity/viewLog.html?buildId=1637&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4765, ignored: 21 Build time: 00:29:55


Hi JeGa, Nice to have opportunity to see you project that use Philip twin-eye laser sensor PLN2033, Recently I have a project want to use the sensor but I can't find the RAM code, Can you send that to me?My email address 297020063@qq.com,  Very appreciate for you share the code anyway!


how can u make it cname  comment...my constructor is not working


It'd be great if the docs could reflect these changes at some stage soon


bah dans KVFAZIA ca a suffit a régler le pb...

Quoting John David Frankland <notifications@github.com>:

> à mon avis c'est dans KVMultiDetArray.h qu'il fallait le rajouter...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/kaliveda-dev/kaliveda/commit/8ce10655bc6034ec7ec3011d586d4902bd68b980#commitcomment-13410891


For one, people should stop adding applications to `go-packages.nix` (see discussion at https://github.com/NixOS/nixpkgs/commit/d0179b917ee24e4735051f06a65c908d622c7a96), which has already bloated to 3000 lines. These belong in separate files.

For applications, binaries should be installed in `out`.


TeamCity pull requests :: netty [Build 272](https://garage.netty.io/teamcity/viewLog.html?buildId=1638&buildTypeId=pulls_netty&guest=1) is now running


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (820 of 820 tests passed)

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


TeamCity YaccConstructor :: YC.Core General Build [Build 499](http://teamcity.codebetter.com/viewLog.html?buildId=199557&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 278 Build time: 00:17:31


Actually neither mosaic nor highland use the default headers in their design. So these headers should be disabled, since there is no frontend output available for them.

Each theme should only contain those headers in the selectbox that are actually usable with that particular theme. Anything else will fall back to "default" in the frontend after switching a theme and give a hint about that header layout being not available in the backend.

You just have to make sure, that none of the themes is abusing the original header layouts from 0 to 6 for their own special headers.


Magic number, replace with constant


Да, недавно стал посматривать его видео)
В принципе пожно написать разрабам об этой проблеме.(смотрю ты создал страничку на вики, есть идея  - можно начать переводить официальную вики по RimWorld, или это лишнее?)


Nice :)


@wkennington Are you going to revert this? Otherwise I'll do it, but I don't want to spend a day reverting your changes again...


sadasda


not necessary to use this Image Button. Just replace your LinearLayout to EditText, and set rightDrawable with that image. Event onSearch should be fired when user taps "Done" button on keyboard.


I agree, goPackages should be only firo libraries, where binary packages
should be in separate files. Also in case of binary packages default output
should be a binary and in case of libraries in goPackages, output should be
library.

V čet., 24. sep. 2015 12:03 pop. je oseba Eelco Dolstra <
notifications@github.com> napisala:

> For one, people should stop adding applications to go-packages.nix (see
> discussion at d0179b9
> <https://github.com/NixOS/nixpkgs/commit/d0179b917ee24e4735051f06a65c908d622c7a96>),
> which has already bloated to 3000 lines. These belong in separate files.
>
> For applications, binaries should be installed in out.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/2f3eb5e0eff9958999245fb8ae9f2702480cf6c6#commitcomment-13410995>
> .
>


What is this commit for ? and why is there a getter/setter for EnsureUniqueNaming  ?


This is inside a `@font-face` definition block, so the `font-family` rule just defines the name of the font. Accidentally adding those fallbacks meant that references to Source Sans Pro didn't work.


Also, please check the cluster name, the `ITFourStoreGraphDriverRead.java` test specifies it as `TRAINBENCHMARK_CLUSTER`, while the shell scripts use the lowercase `trainbenchmark_cluster`.


And there's another issue with this setup: It now contains all the headers from both themes, while there is i.e. no "H2 Block Header Row" in mosaic. As already mentioned above: Mosaic must only contain those header layouts in its setup, that are available in Mosaic.


@wkennington can we come to consensus here please. I know you maintain a lot of go packages, but I think @edolstra has a point.


Why have we ignored Path for templates? We need the path for templates to sync the tree.


I think it might be worth returning the `IncomingMessage` object here to refer to directly in the `RequestMailer.new_response` call later - just getting the last incoming message seems more brittle - could be possible to get into some weird race condition.


First Commit


we have it already


Just fixed your test and pushed. Please take a look so you understood what I meant.

![popkey](https://m.popkey.co/HL3uJ92kUwEsfzdlRx1zWsfwKaU=/original-0470574f-738a-4e08-a5b1-0c94ef142ea4.gif/b5xDV.gif?c=desktop&l=search&f=.gif)


And the filter


it's icon color - not menu background image


@edolstra @wkennington  let's try to do it right, can we instead of reverting, split binary packages from go packages. We depend on this in production.


Is ``(EE)`` always followed by single space? Is it the case even if the log line contains only ``(EE)``?


getPrimaryHighlightsColor()=getPrimaryHighlightColor()


TeamCity pull requests :: netty [Build 272](https://garage.netty.io/teamcity/viewLog.html?buildId=1638&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 4786, ignored: 16 Build time: 00:10:00
### Failed tests
```
io.netty.resolver.dns.DnsNameResolverTest.testNegativeTtl: <no details avaliable>

```


esto es un comentario


Do you mean the Zwave controller ? You cannot choose the group for the Zwave controller. The ZWave controller is in the group 1 by default.
However all my energy or power devices are in the group 1.

What is the relationship with the short log content ?


change name of parameter from "size" to "offset"


You know what I'm going to say :) Do some binding and step through the mailer to make sure it is the right data, If you're going to do @user = user (which is good), be consistent about using @user going forward.

For sanity's sake, you also might want to test with the some real data so you don't confuse everything.


currently you can add parameter limit=100


Consider `[ reason | Left reasons <- ..., reason <- reasons ]`


Ah I see! :+1:


hlint all the redundant parens away, they confuse me :)


Tagged! Thanks


I think a tuning session is needed anyway.


if lookupInput is only used here, it may be nicer to define it uncurried in the first place


I agree that it doesn't hurt, and I'm not familiar enough with C++ exception to conduct an intelligent discussion here. Keep it.


The commit is for avoiding duplicate data type names, so instead of just throwing an exception it adds (1) to the end of it , the getter/setter is just following the way the content has a a unique name toggle


Thanks


And you keep me guessing is this good I think so like all of humanity she will call again did I just step into a trap I see the broken chair I wanna feel sincere break free I want all that I have made like all of humanity is this good you keep me guessing is this good I think so like all of humanity don't let them win.....once you let go and you let it show and choosing sides becomes difficult and you want to grab a hold of the first person you see this is getting old but it's what sets us free the fire that lights our hearts it burns for us all it doesn't discriminate deep down inside you carry the guilt of what you left behind they don't know how you feel so go ahead and take the deal that sound then will know how you feel look up fill cup when the global planet of humanity spins around.push used keep end voice of my inner thought now go vomit and puke lol


Ok... if you are going to add features or fix bugs like this please make sure it's a task on the tracker and that your commit references only this.

Also, there's no reason to have a public getter/setter to toggle this options, if we want it to be this way, then we just make it that way.


Think i have fixed it, but can you give it a try on this plunkr before i
release it?

http://plnkr.co/edit/ZNXKNb?p=preview

On Thu, Sep 24, 2015 at 10:59 AM, Carles Company Soler <
notifications@github.com> wrote:

> Use the same plunkr as before: http://plnkr.co/edit/lcgwKd?p=preview
>
> Set the date as a random string (whichever that is not a date) and it's
> still valid.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gillardo/bootstrap-ui-datetime-picker/commit/81016b775a8343eaf89ada80fb3ca7ff5a83dc92#commitcomment-13410913>
> .
>


The purpose of embedded ES is testing/demoing and it is limited by some hard coded config so
- we are sure that embedded is easy to debug : single shard, cors, no failure if your disk is full at 95%
- use limited number of threads
The message is if you want full control use a production setup with a dedicated ES cluster


@ST-DDT just type a http link in the chatbox normally


glad to help ﾟ ∀ﾟ)ノ


The status in these places are redundant, e.g line 154, 157, 161, 162.


@flavour I tried with this for defualt options:
```
rows.append(("pr_subscription_repository_id__row",
                         "",
                         multiselect(synctable.id,
                                     "",
                                     _id="pr_subscription_repository_id"),
                         repository_id.comment))
```
It gives me UnboundLocalError. The full trace is http://pastebin.com/i3Q8wMW4
I debug to see but the 'id''s requires field has no attribute 'options' as here: https://github.com/web2py/web2py/blob/master/gluon/sqlhtml.py#L312

Sorry for such small trouble, but its killing me :(


alpha sort.


These places could be wrapped in one line, e.g line 46, 56, 77, 83, 88 and so on.
```if (path == null) return 0;```


Use a "dictionary" (record) of callbacks/vtable, instead of many params


This hunk of change is very large for such a small change: I believe the same result could be acheived with this one line patch given all the other arguments to `subprocess.Popen` are the same:

```diff
         session.xenapi.VDI.get_by_uuid(vdi)
     except:
         raise ArgumentError("VDI parameter invalid")

     process = subprocess.Popen([
-        "/opt/xensource/debug/with-vdi", vdi, "/opt/xensource/bin/xe-install-supplemental-pack"],
+        "/opt/xensource/debug/with-vdi", vdi, "/opt/xensource/bin/xe-install-supplemental-pack"] + (["-u"] if "upgrade" in args else []),
         shell = False,
         stdout = subprocess.PIPE,
         close_fds = True)
```


Yes, (EE) is should be always followed by single space.


tjena Anton! om du går in på "strings.xml" filen i "resource" mappen kan du lägga in strings som du vill ha i projektet. när du vill anvinda stringen som i det här scenariot, kan du istället göra @string/work_in_progress, vilket refererar till stringen du lagrat i strings.xml. det här betyder att du inte behöver hårdkoda in det varje gång du lägger in det nånstans.

 om du går in på resource mappen, sen strings.xml, kan du se alla strings som projektet använder.

rekommenderar inte att hårdkoda. om du går in på resource mappen, sen strings.xml, kan du skapa strings. när du vill använda den kan du skriva @string/work_in_progress.


I have an error when I run +reagent option.
I run this command below.

```
$ lein new descjop tmp8 +reagent

Generating fresh descjop +reagent project.

$ cd tmp8

$ lein new descjop help

java.lang.Exception: Error loading /Users/***/project.clj
 at leiningen.core.project$read_raw$fn__6862.invoke (project.clj:942)
    leiningen.core.project$read_raw.invoke (project.clj:939)
    leiningen.core.project$read.invoke (project.clj:953)
    leiningen.core.project$read.invoke (project.clj:954)
    leiningen.core.main$_main$fn__6480.invoke (main.clj:386)
    leiningen.core.main$_main.doInvoke (main.clj:383)
    clojure.lang.RestFn.invoke (RestFn.java:436)
    clojure.lang.Var.invoke (Var.java:388)
    clojure.lang.AFn.applyToHelper (AFn.java:160)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.core$apply.invoke (core.clj:630)
    clojure.main$main_opt.invoke (main.clj:316)
    clojure.main$main.doInvoke (main.clj:421)
    clojure.lang.RestFn.invoke (RestFn.java:482)
    clojure.lang.Var.invoke (Var.java:401)
    clojure.lang.AFn.applyToHelper (AFn.java:171)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.main.main (main.java:37)
Caused by: clojure.lang.Compiler$CompilerException: java.lang.UnsupportedOperationException: nth not supported on this type: Keyword, compiling:(/Users/***/project.clj:1:1)
 at clojure.lang.Compiler.load (Compiler.java:7239)
    clojure.lang.Compiler.loadFile (Compiler.java:7165)
    clojure.lang.RT$3.invoke (RT.java:319)
    leiningen.core.project$read_raw$fn__6862.invoke (project.clj:940)
    leiningen.core.project$read_raw.invoke (project.clj:939)
    leiningen.core.project$read.invoke (project.clj:953)
    leiningen.core.project$read.invoke (project.clj:954)
    leiningen.core.main$_main$fn__6480.invoke (main.clj:386)
    leiningen.core.main$_main.doInvoke (main.clj:383)
    clojure.lang.RestFn.invoke (RestFn.java:436)
    clojure.lang.Var.invoke (Var.java:388)
    clojure.lang.AFn.applyToHelper (AFn.java:160)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.core$apply.invoke (core.clj:630)
    clojure.main$main_opt.invoke (main.clj:316)
    clojure.main$main.doInvoke (main.clj:421)
    clojure.lang.RestFn.invoke (RestFn.java:482)
    clojure.lang.Var.invoke (Var.java:401)
    clojure.lang.AFn.applyToHelper (AFn.java:171)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.main.main (main.java:37)
Caused by: java.lang.UnsupportedOperationException: nth not supported on this type: Keyword
 at clojure.lang.RT.nthFrom (RT.java:933)
    clojure.lang.RT.nth (RT.java:883)
    leiningen.core.project$dependency_map.invoke (project.clj:77)
    leiningen.core.project$dep_key.invoke (project.clj:229)
    leiningen.core.project$reduce_dep_step.invoke (project.clj:233)
    clojure.lang.PersistentVector.reduce (PersistentVector.java:333)
    clojure.core$reduce.invoke (core.clj:6518)
    leiningen.core.project$setup_map_defaults$fn__6606.invoke (project.clj:325)
    clojure.core$merge_with$merge_entry__4649.invoke (core.clj:2932)
    clojure.core$reduce1.invoke (core.clj:909)
    clojure.core$merge_with$merge2__4651.invoke (core.clj:2935)
    clojure.core$reduce1.invoke (core.clj:909)
    clojure.core$reduce1.invoke (core.clj:900)
    clojure.core$merge_with.doInvoke (core.clj:2936)
    clojure.lang.RestFn.invoke (RestFn.java:439)
    leiningen.core.project$setup_map_defaults.invoke (project.clj:319)
    leiningen.core.project$make.invoke (project.clj:373)
    leiningen.core.project$make.invoke (project.clj:359)
    leiningen.core.project$eval599.invoke (project.clj:1)
    clojure.lang.Compiler.eval (Compiler.java:6782)
    clojure.lang.Compiler.load (Compiler.java:7227)
    clojure.lang.Compiler.loadFile (Compiler.java:7165)
    clojure.lang.RT$3.invoke (RT.java:319)
    leiningen.core.project$read_raw$fn__6862.invoke (project.clj:940)
    leiningen.core.project$read_raw.invoke (project.clj:939)
    leiningen.core.project$read.invoke (project.clj:953)
    leiningen.core.project$read.invoke (project.clj:954)
    leiningen.core.main$_main$fn__6480.invoke (main.clj:386)
    leiningen.core.main$_main.doInvoke (main.clj:383)
    clojure.lang.RestFn.invoke (RestFn.java:436)
    clojure.lang.Var.invoke (Var.java:388)
    clojure.lang.AFn.applyToHelper (AFn.java:160)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.core$apply.invoke (core.clj:630)
    clojure.main$main_opt.invoke (main.clj:316)
    clojure.main$main.doInvoke (main.clj:421)
    clojure.lang.RestFn.invoke (RestFn.java:482)
    clojure.lang.Var.invoke (Var.java:401)
    clojure.lang.AFn.applyToHelper (AFn.java:171)
    clojure.lang.Var.applyTo (Var.java:700)
    clojure.main.main (main.java:37)
```

I think it need to change project.clj code that descjop generated.

```
  :plugins [[lein-cljsbuild "1.0.5"]
            [lein-externs "0.1.3"]
            [lein-shell "0.4.1"]
            [lein-figwheel "0.4.0"]
```

to

```
  :plugins [[lein-cljsbuild "1.0.5"]
            [lein-externs "0.1.3"]
            [lein-shell "0.4.1"]
            [lein-figwheel "0.4.0" :exclusions [org.clojure/core.cache]]]
```

reference link : https://github.com/technomancy/leiningen/issues/1739

thanks.


@jezdez it worked for envdir a while ago, where you've added @ticosax and me.
Also @RonnyPfannschmidt has offered his help multiple times.
Please consider creating a team for maintaining it - especially since the most pressing issue is "just" creating a new release.


Yep. Therefore I will create a PR afterwards ;)


I think this should probably go to the end of `create_response!` so it can be close to the `self.awaiting_description` call which is what makes it necessary.


```()``` is needless?


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


Instead of synctable.id use s3db.sync_task.repository_id
- this has a Validator with an options() method


Might be clearer to say that it logs the rejected reason.


Oh, does that work?


Could you send a PR please?


Perhaps worth adding a test for `ActionMailer::Base.deliveries.size` just to confirm that there's only one mail there.


Maybe "value" should be called "leaf" or even "a" and be last?

"key" and "keyDesc" look like "key" and "key2". In the context of a generic Trie, it seems weird to have specifically 2 levels.

The outer NonEmptyMap is never looked up, right? It's only an assoc-list in practice? Maybe it could be a Functor given from the outside instead of "NonEmptyMap key" and "keyDesc" should be "key"?


jqyTRAWETKUYAWRITYUAEIRUYAEOTULESTULEUTWUttrwgaDSTWRTEH


It should work the whole way, I can add another test for that.


### New Feature

`mesh.prototype._destroyElement(componentName, callback)`

The opposite of `_createElement()`

* Removes reference to component from the API (event and exchange) on all remotes (other nodes and browser)
* Removes relevant requestors from the all appropriate messengers (local and remote).
* Removes the local request listener (component/requests/* subscription from the datalayer)
* Removes all associated middleware from the connect stack
* Calls the component stop method if started
* Deletes it completely


Jo precis, såg det. Men tänkte inte lägga in den där strängen eftersom vi har den bara som temporär text så hårdkodade den istället. Men ska lägga in alla strängar på det där sättet i fortsättning så det blir mer korrekt


Worth having a line to check that there's no incoming message associated with the request?


@martinklepsch hmm, I'm not seeing the dependency on my setup.  I ran the following in a fresh `virtualenv`:

    pip install enum34
    pip freeze

I see this:

    enum34==1.0.4
    wheel==0.24.0


I have slightly changed this query in the last push: the extra 15s grouping is needed to get more results back; influx will round it to a single value otherwise.


How about this?
```
        for (String expName : expDir.list()) {
            if (!expName.startsWith(expType)) continue;
            if (expName.endsWith(".apk")) {
                expPathApk = expDir + "/" + expName;
            } else if (expName.endsWith(".obb")) {
                expPathObb = expDir + "/" + expName;
            }
        }
```


I think we also want to test rejection of a non-authority-domain mail here.


Don't know if this is all you intend to do, but do send me a pull request when you're done!


Ah, I see these are later - sorry.


> I agree that it doesn't hurt

Actually, logically it doesn't hurt, but doesn't adding `throw()` have performance implications? So if it ends up never throwing anything because you catch all inside it, it just seems to me not required.

If it helps Coverity then it's another thing, but does it actually help coverity? Why would it help it?


On travis, `Strict` conflicts ... This came as a surprised !


As Nameable and Enumerable are defined here in protocol, why are they re-defined in endpoints?


Perhaps more clear as 'does not bounce responses to external requests'


To prevent future omissions, couldn't this version be read from `package.json`?


### TODO

Pending __necessary__ functionalities for `_destroyElement()`

* Remove dangling event and data subscriptions left by remotes in the local data layer for the destroyed component.


信用取引ナンピンテーブルが崩れないように、発注から５秒後に信用取引ナンピンテーブルのルーチンを呼び出す


Write the code comments in English later on.
We will have international engineers working on this.


This function declared as ```private```?


Might be worth making clear that this is the notification email that there's something new in the holding pen.


remove empty line


Possibly worth a line to demonstrate that there's nothing in the holding pen.


"Prop should be positiv**e**"


This description is right?


Could you explain ```JNI'd```?


It seems to work, but I think it breaks the validation introduced with this commit ('2015-09-11T21:47:20.866Z'). Check the pull request I sent you.


This should be `encodeURI()`, not `encodeURIComponent()`.


Cool :grinning: thanks @snowdream


Could we create a constant for `"leg"`?


```
--pageToOpen page/to/open
--skipPageOpen
```


其实放在一个文件夹下就行，不用非得有week之类的


sant, vi lär ta bort sånt från strings.xml på slutet


What is the reason of for distinct `_timeout` and inner `wrap` functions? Why not write it like this:
```
def _timeout(func):
    @defer.inlineCallbacks
    def __timeout(*args, **kwargs):
        ...
    return __timeout
```
?

(but it will cause change of usage syntax: `@_timeout` instead of `@_timeout()`)


```
--pageToOpen page/to/open

--skipPageOpen
```


I would rather make the core more robust:
* do not use `strstr` because in the unlikely case where the string does not follow ``]`` the `skip_pfx()` would cut more than necessary
* do not count on ' ' following  `(EE)`


It is good practice to add `@functools.wraps(func)` decorator on replacement function in order to save docstrings


The methods were fixed in the following commit:
https://github.com/netguru/review/commit/83acc854922bf36b21df65d881b42a187eb3cc23


Should this be in this commit?


request pull


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1358 of 1358 tests passed)

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


extra line.


I think it is cleaner to write like this:
```
deadline = kwargs.pop("deadline", None)
seconds = kwargs.pop("timeout", None)
```

This will remove them from `kwargs` automatically


you can't know whether sth was removed simply based on count, if you remove one tag and add another the count will not change even though the tags did


Just seen your pull request.  I have merged the request and released a new
version 1.2.5.

Thanks for the contribution

On Thu, Sep 24, 2015 at 11:39 AM, Carles Company Soler <
notifications@github.com> wrote:

> It seems to work, but I think it breaks the validation introduced with
> this commit ('2015-09-11T21:47:20.866Z'). Check the pull request I sent you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gillardo/bootstrap-ui-datetime-picker/commit/81016b775a8343eaf89ada80fb3ca7ff5a83dc92#commitcomment-13411535>
> .
>


@Awea Qu'est-ce que t'en penses ?


It is a task on the tracker: http://issues.umbraco.org/issue/U4-7058 I'll remove the toggle


Translated all of the sentence, except some words and the "Research".
And change of translation that was misleading.
「一部の表示される場所のわからない文」と「研究」を除いた全ての文を翻訳しました。
そして、「Equal Trade」の「等価交換」という翻訳が誤解を招いていたため「置き換え」に変更しました。


They are working on different requests, so this is not possible to add and remove a tag at the same time without having a request for addition and subtraction of a tag.


Thanks! Just needed those two methods for a quick demo*. I'll try to add more endpoints covered before PR.

* * the demo is [here](https://github.com/IGZgustavomarin/eddystone-emt-demo)


This is great attempt, because tuning require too much games with carefully chosen ck vales to obtain convergence. I think that tuning should be combined with other tests.


@yohanology I think you are right.
My plan right now is to release `angular:1.0.0` and then release a new version of this tutorial with support for ES2015 and best practices like using the Meteor API from the start.
I'll update it with the new version, thank you


That did the trick. :)


Might be worth using a block of `|name, email_address, incoming_message_id|` rather than `|row|` for clarity - then we can just refer to `email_address` below.


this is an obsolete case class that I forgot to remove. It's been superceded by BusinessDetails where the name is, in fact, not optional.


yes, but we shouldn't depend on it – what if we change the frontend to add or remove tags in batches? we end up with a bug that might be hard to track later on


```1024 * 1024``` is large enough to get input stream of lib*.so? Maybe misunderstanding.


_switchOn -> enable() - I don't think we need to rename the method to fix the issue / add the feature. For the wording: yours is actually better. But those methods are protected, maybe make them private then as well as you could only rename them if they are only used within the same class.


No problem. Thanks to you for the great control.


I really don’t like the look of this border… why is it there?


As you use it only once I'm wondering if it's actually saving lines in this small program :)


what if someone wants to use it in a play app? will that not pull in an otherwise unnecessary dep tree?


Should be `[allow, reason]` ?


Thanks.


copypaste from examples > me

go 1 - francesco 0


@westcripp Sorry for the mistake. I'll be more careful. Thank you for editing


Small typo of mulitpart in the commit message.


Because that's what the example code used ^^


I think the commit comment here should say that the methods are being turned into classes.


The links to the images in this chapter are all broken for the spanish version of the book. To whom should I notify about it?


Tabs instead of spaces (which is used in this file).


Ok so content comparison of arrays? Can I assume that the arrays will be always sorted ?


graph is not easy to understand when scaled to 100, scaling down makes it much easier to read and more visually appealing


Sorry... I'll fix it soon


Separate WPA-PSK and WPA-2 features in secobj objects, dladm and nwam.


Prop**e**ability


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 26](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=192&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


This looks bad. Indent the "int" by one more to align with the one above.


1. Digsites:

PLAYER_FIELD_RESERACH_SITE_1 (RESEARCH, typo there) is not used to show completed projects but store digsites. there are 2 offsets on each (PLAYER_FIELD_RESERACH_SITE_1 + 0, + 1 etc.), that together compose the uint32. I used for this a uint16 setting for each of the site id's to store them on the correct offset. check _UpdateResearchFields() for that. There are 8 of these, as seen in UpdateFieldFlags.cpp (Type: TWO_SHORT), so you can have up to +7. 8  x 2 offsets = 16 and that solves the digsites part (4 for each map, Kal, EK, Out, North).

2. Projects:

PLAYER_FIELD_RESEARCHING_1 also has size 8 (0 to +7) and the projects storage is the same only it does not store branches. it stores the actual projects id's, and i did that using bitshifts. There are 2 projects on each offset (project_now_1 / 2), like digsites are stored (Type is also TWO_SHORT). Check GenerateResearchProject for that. When a project is completed another one is generated for that branch. I stored them in a map considering their branch id (sResearchProjectStore : ResearchBranchID), so that they are retrieved based on the branch you select in the window. This works in corelation with the opcode handlers. Branches 1-8 are there but 9 is 27 as seen in dbc for vrykul (jumps to 27 after 8), thus the stuff in GenerateSavedArtifacts(). project_1 is for the current branch, project_2 is for the current branch + 1, so all branches get a project you can complete if you opened it by getting a fragment for that branch, this being handled by the client. I used bitshifting but you should be able to use the same method as for digsites by setting uint16 values for each offset with no problem :). m_researchProject[branchId] is the project you must currently complete for the branch you are looking at. Complexity of and fragment / stones count needed for projects are based on some calculations i made that you can find in the code.

Idk what else to say, if there is still something that is not clear please write here. Have a nice day!


camelCase :trollface:


no need to be public, right?


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 26](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=192&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:58


Two spaces before EOL-comments, please.


`getCeisumImageryProvider` is already sethere -> https://github.com/geoadmin/mf-geoadmin3/blob/c886ae7f71010c659b472c904d5a9955dbcdb504/src/components/map/MapService.js#L1200

or am I missing something ?


This doesn't work.  sigwaitinfo() returns -1 on error.  You don't want to propagate that.


What's the purpose of  the "// 8"?
It's not like the numbers make a lot of sense anyway.


Najs


Doesn't look right. The script should result in an error, if gcrypt support was explicitly requested but the configure script can't do it.


Why the prefix?


Yes, SPSA (at least in the form we are use) is a local, not a global, tuning method, so good starting values are important. If we are to far from a local optimal solution convergence needs many more games.


good!


Possibly change this to a `compare_exchange_strong()` to remove the possibility of re-entry.


Use package:test, not package:unittest.


赞！


Esto lo ha hecho el JJ para subir en el ránking, trampa


So many nested if-else inside this function.
Seems like everything is dependent on it.

Refactor it to distribute load of this function to other functions (if they don't exist, create new functions.)


That's just the buffer size, how much is transferred on each read() / write().


I am not sure whether this is detected, but: Git autodetects renames / moves in a single commit via a heuristic:

>        -M[<n>], --find-renames[=<n>]
           Detect renames. If n is specified, it is a threshold on the
           similarity index (i.e. amount of addition/deletions compared to the
           file’s size). For example, -M90% means Git should consider a
           delete/add pair to be a rename if more than 90% of the file hasn’t
           changed. Without a % sign, the number is to be read as a fraction,
           with a decimal point before it. I.e., -M5 becomes 0.5, and is thus
           the same as -M50%. Similarly, -M05 is the same as -M5%. To limit
           detection to exact renames, use -M100%. The default similarity
           index is 50%.

Copies can be detected by setting `-C`.


children could contain some object unrelated to the one you provided.
advice: store `BatchReport.Component.newBuilder().build()).setKey("CHILD_KEY").setUuid("CHILD_UUID").build()` into a variable and check the object in the list if the same (do not use equals) as the one in the variable


`strip()` is not defined on `None`?

```
>>> d={'a':'b'}
>>> d.get('c').strip()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'NoneType' object has no attribute 'strip'
```


Extract libxwalkcore.so so it can be loaded by JNI (java native interface).


For correct operation of the long-lived Hibernate all classes must have a
constructor (which may be not public) to NHibernate can create objects of
classes using Constructor.newInstance ().

2015-09-24 12:50 GMT+03:00 Kuzmenko Alexander <notifications@github.com>:

> It's not about liking or disliking. Please leave comments why it has to be
> there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kalexi/MailBot/commit/ba1c787edf8de990d4697dacfad38126a0733d4d#commitcomment-13410723>
> .
>


For correct operation of the long-lived NHibernate all classes must have a
constructor (which may be not public) to NHibernate can create objects of
classes using Constructor.newInstance ().

2015-09-24 14:09 GMT+03:00 Алена Шрамко <shramko.zp@gmail.com>:

> For correct operation of the long-lived Hibernate all classes must have a
> constructor (which may be not public) to NHibernate can create objects of
> classes using Constructor.newInstance ().
>
> 2015-09-24 12:50 GMT+03:00 Kuzmenko Alexander <notifications@github.com>:
>
>> It's not about liking or disliking. Please leave comments why it has to
>> be there?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kalexi/MailBot/commit/ba1c787edf8de990d4697dacfad38126a0733d4d#commitcomment-13410723>
>> .
>>
>
>


Hello, does post-processing work?

How does one do it?

I am attempting with `python3 rsigh.py "/path/to/Downloads/complete/[rsigh] example"` and not getting anything out.


Have you solve all the warnings in 64-bit build?


I agree that this should no longer be considered as a valid alternative, but I wonder whether it's worth having a section somewhere about historical approaches i.e. someone might find it confusing when trying to trace the git history of a flow if they don't know about this old v2 workflow.


Using .map() here doesn't seem warranted. A regular for loop with multideclaration would be clearer.


Ains!! Lo perdono por ser en los #juevesenlaetsiit :+1:


looks weird that `ComponentRootBuilder` takes no parameter


You shouldn't import this, just do `implicit def ec: ExecutionContext` and let the user of the trait define it


Fine, but can we remove it? I think it’s ugly :)


I believe this is taking things in a completely opposite direction than what I meant :)

What I meant was that we should rather have something like `spring.cloud.stream.bindings.output.contentType` - as multiple inputs and outputs can have different needs.


Yeah, I know the feeling :)

I am fine with commits of this size on devel, as those are easy to review.  As soon as one needs to make mental connections over different sections it becomes a different story altogether.  So, I don't mind, really...


I had tried this function and it worked for enrichResult instance but did not for compareClusterResult.  If I understand correctly, the problem is that the latter one doesn't have 'ontology' attribute.


@alohaug It seems to be working again in the latest release build.

https://risacher.org/putty-cac/testing/Release/pageant.exe


(we can still accommodate that for SCDF where we prepopulate properties)


method `buildChildren` should be after its first use


create a new package 'helper' and move all the helpers there


I think it's better to let whoever extends this abstract class define this method, as they may have other routes they wish to add


@waxspin yes, Toby sent an email out on the lmms-devel list .  I tested on Ubuntu 12.04 both 32-bit and 64-bit mingw packages and then updated accordingly.


Issue number should be #99 .

<configuration,layout,documentation


could be refactored to read

```java
ComponentDto project = persist(newProjectDto("ABCD").setKey(REPORT_PROJECT_KEY));
```


+1 on sending it to master

On Thu, Sep 24, 2015 at 1:08 AM, Simone Dalmasso <notifications@github.com>
wrote:

> cool, no drowbacks is just a dedicated api and is fine.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simod/geonode/commit/355de365b9d1bd55cbda5a8b66e2c49f32baa92e#commitcomment-13407441>
> .
>


@jennybc After loading gapminder, do you still want to do `str(gapminder)` every time after loading like we did with gDat?


No, it's more like save as copy. It just saves the world in a new directory.

With the cfg and logs, we'd just need to save them for the new world so that it can be started with the same settings.


could be renamed to 'indexAllComponentsInTreeByRef' or something


Ok you're right, 'bytearray' when writing, 'str' when reading.


I'm not completely convinced of the use of italics for the filename interpolation here. I wonder whether using a `code` block and an ASCII tree with angle-brackets around "interpolated" values along the following lines might be clearer:

    lib
    └── smart_answer
        └── calculators
            └── <calculator-name>.rb

i.e. the output of the unix `tree` command


> [...] I should commit more often

:+1:

Pro Tip: Take advantage of `git add` and `git stash` to split unrelated changes into separate commits. The former even allows adding selected lines of a file to the staging area (via `git add -i`, see https://git-scm.com/book/en/v2/Git-Tools-Interactive-Staging#Staging-Patches). PyCharm has a similar feature called "Change Lists".


you want to push the build to github?


@jakepruitt - Looking good, great start!

 I can see some issues with current grammar :

*  it's not safe to pass as an argument  `int*`. This is c++ so we can use std::array<int, 4> or adapt custom struct e.g
```c++
struct view_box
{
   int x0; int y0; int x1; int y1;
};
```
*  Current grammar will fail on `0    0   100   100` input (NOTE: more then one space)
Here the fix
```c++
...
using skip_type = boost::spirit::ascii::space_type;
if (!phrase_parse(str, str + std::strlen(str), int_[ref(list[0])=_1] >> -lit(',')
                >> int_[ref(list[1])=_1] >> -lit(',')
                >> int_[ref(list[2])=_1] >> -lit(',')
                >> int_[ref(list[3])=_1]))
///

```

I changed `parse` to `phrase_parse` and passed `skipper` and made commas optional - `-lit(',')`

*  Does viewBox really expects integers ? I'm not familiar but doesn't feel right


There is a note here "Note: the above is true when run interactively but currently not true in the rendered document. That is a glitch in knitr that is getting straightened out." referring to an error message specifying the function it was called from. From the knit document it looks like this bug is fixed, but maybe I'm misunderstanding the bug. I didn't touch it, just wanted to bring to attention


I think it'd be clearer to say it's returned as a "normalised postcode" and maybe add an example.


No I wrongly removed that.


This will not use the bundle $ for now right?


@jennybc
With gDat, we always looked at the number of rows to make sure we have the right data. Are we still going to do that with gapminder? Because this blurb doesn't make any sense anymore if we don't do these checks anymore


Thanks! :-)


Lol, so many derp here... thanks bro 😆


should not be needed if no need to push the bundle to git.

otherwise one can use `npm run dev`


Iff `document.forms` is up-to-date before the mutation observer is fired, AND `LoginManagerContent.loginsFound` can fire twice with no performance or correctness cost, I think going through all the added nodes looking for `FORM` elements on every mutation is sufficient.

Having `subtree: true` seems sufficient to catch the `FORM` within `FORM` case recursion and string matching was trying to solve (follow up bug for test-case and/or regression).


> This will not use the bundle $ for now right?

It will since autocomplete.js is now UMD compliant and will `require('jquery')` internally to define `$`.


@wilr this breaks the Travis build.


perfect


For this to work, you can define "main" autocomplete.js package.json entry to be dist/jquery.plugin.js and then you can just require('autocomplete.js'), will work

It will also define the main autocomplete.js export to be the jquery plugin, which should be what we want


Minor. It doesn't look as though you need `current_node` now that you're relying on using `next_node`.


> It will since autocomplete.js is now UMD compliant and will require('jquery') internally to define $.

Only the dist/* are umd compliant


No, actually lookupThis could exist in KOOL, too.


I agree that most of this is pretty redundant now, but I think it might still be worth mentioning that all leading whitespace is removed before the text is processed by Govspeak, in case there's a scenario where this leads to confusing behaviour with a Govspeak construct that we haven't run into yet.


There are some things here which could be could be ported back to KOOL to make a better design.


> Only the dist/* are umd compliant

Ah yes my bad :)


> For this to work, you can define "main" autocomplete.js package.json entry to be dist/jquery.plugin.js and then you can just require('autocomplete.js'), will work

No I don't want the jquery version to be the main one.


I would go so far as to say "Avoid deeply nested contexts".


How are you???


throw() give an user guarantee to the compiler that code won't throw. Code can still throw an exception however it will likely call an abort due to unexpected exception. Most updates are for the GUI so performance impact is null.

Coverity knows which functions can throw an exception (I think it is limited to our BaseException) so it can find unintended behavior (methods marked as not-throw but that are still throwing).


There are a number of spurious spacing changes in this file. Please revert them.


Well I happen to think it looks ugly without, lust loose elements floating on a white page ;)


Is it worth using the "DEPRECATED" marker that you've used elsewhere to indicate that tests with deeply nested contexts exist, but we shouldn't be adding new ones.


its working ok now just stops when cat error but that can be ignored


Is it possible to move this up on `.btn` instead or do we need this specificity to override bootstrap defaults?


What problems did you have so you needed reset?


Why? I need this for corrrect order of startup, this breaks my ipsec setup!


Huh? Floating? This is what I see:
![with-border](https://cloud.githubusercontent.com/assets/2346599/10072268/7835af7a-62c0-11e5-8097-0e134f421e49.png)
And with the border removed:
![without-border](https://cloud.githubusercontent.com/assets/2346599/10072275/847388ac-62c0-11e5-860c-ce99947294ec.png)
(I hope those tabs don’t give too much away ;) )


Should be "method of steepest descent"?


No, engine.sdk2013.txt only has "#default" sections, with nothing
game-specific.


My bad, 'USB2Driver' has a default timeout of 1000ms.
But:
>  Anyway, could you explain why do you need to increase this value ?


typo


@SubJunk I don't think it will take me more than 5 minutes to do, so if you're not planning to tag today there's really no reason to do it. We just got some more French translations in, so every minute counts :wink:


Well, perhaps I commented on the wrong style line? For the second screenshot above, I removed the border from the following element: `<div style="display: block; top: 0px; border: 1px solid rgb(223, 223, 223); padding: 0px; overflow: hidden;" class="w2ui-panel-content">` (unique selector: `#layout_all_panel_top > div:nth-child(4)`)


and it was fixed :)


Aoutch. 2 years later, finding this bug I introduced myself. Should be v:t(). Will fix momentarily.


Code looks good. More a question potential problem is that no one knows why we have 8px there, is this some common rule we can use elsewhere? Is this some calculation based on some base value?


The point of using LookPath is that it will return an error if the file does not exist.


If you define it in french. :) The function name was translated below, so I changed it here for consistency.


I could have missed that and my cluster would start failing, please next time make pull request.


It is giving the red x (which is considered fail by current test) only if qgis_app lib contains ${IMAGE_RCC_SRCS} and if qgis_app lib is loaded during the test (if a provider/plugin is linked to qgis_app (currently GRASS plugin).


You forgot to put a break; here - so 'System Information' would go to this Debug screen instead.


Como es que lo has añadido y lo has dejado desactivado? Algún problema o algo?


the "for spam reasons" here seems a bit misleading now - perhaps better to just end this comment at "prevented"


sure, os.stat is more direct, though.

I'd rather have a separate function, since now you dup the code for /bin/umount.


Is it the memo about Galaxy installation/running/customization ?


Shouldn't code used for tests be in /test not /src ?


Oh duh. I'm silly!


Addressed in the amended version of the commit - a347ce8a6ca30107fcf19b87ce81f59f48f20cc0


It seems to me wrong as no organization CommonJ exists.
Commonj.timer and commonj.workmanager are APIs created by BEA and IBM together. see http://docs.oracle.com/cd/E13222_01/wls/docs100/commonj/commonj.html#wp1057962


I was originally eating the @_timeout(seconds) and had to pass it through, it is no longer necessary because we eat it from kwargs... makes it cleaner. Thanks!  I'm ok with @timeout


Shouldn't the default be a head request since all you need is the status code?


I'll look this up, put this above def timeout(func) ?


The duplication is not nice, yes. I will rewrite it as a wrapper around LookPath(). I'd rather not use stat() directly because you have to check for a number of things - the file must exist, be executable, not be a directory, ...


This is just how dartfmt does its thing. Talk to Bob, file an issue, etc.


@deads2k mind reviewing this before I squash it to different commits?


Should this fixup commit also set the default value for the `rejected_reason` param on the `receive` method to `nil` rather than an empty string?


Talk to Bob. File an issue w/ dartfmt


I think I answered default constructor required NHibernate.


Adding a comment to a long literal list makes sure the formatter doesn't format these as a one-item-per line.

// 8 is just saying how many items are in a line


А что, количество изменённых строчек имеет хоть какое-то значение? =)


Do you know what setting `CURLOPT_SSL_VERIFYPEER` to `false` does? Opens us up to MiTM attacks. Check the code, you can pass through custom cURL parameters if you need this.