I just did a small test, to decide whether UtpSocket is really a stream or message passer. In the test the sender sent one message of size 20 bytes and the receiver tried to receive twice into a buffer of size 10 bytes. If the socket was message based (I'm switching the terminology from `datagram` to `message` as `datagram` is often associated with non reliability), then I would only receive once and the second call would block. On the other hand, if it was a stream, I would be able to receive twice.

To my surprise, it was indeed behaving like a stream. I personally think this is a bad design decision from the library creator, but oh well...

> uTP sockets add congestion control and reliability (including delivery order)

I think it is important to note that reliability and congestion control are *not* mutually exclusive with being message based. Or put differently: reliability and congestion control do not imply `stream based`. On the contrary, I had expected that UtpSocket was _reliable message passer_ and then UtpStream was _stream based_ (note that the terminology would correspond to UdpSocket and TcpStream Rust objects). This would make a lot of sense to me because it is efficient to implement stream on top of a _reliable message passer_, but to do it the other way around you'd need to add bytes indicating where a message ends (so less efficient). Also, having a separate UtpStream object where UtpSocket is also a stream seems kind of pointless, but again, oh well... :)

> We can know that Rust treats Ok(0) as EOF.

I am not claiming otherwise here, what I am saying is that what happens in this commit doesn't look like EOF. I'll elaborate...

> I believe the point of EOF in Rust is that, unless you do something else (like seek(0) in a File-based implementation), you'll never get bytes again.

That is not the complete story. If you're reading from a file, and you hit EOF, you can still read more data out of it if another thread or process appends to the file. See [this example](https://gist.github.com/inetic/f1c2a5cdb703fbbff7eb). Note that this is *not* what the lines in this commit are about. That is, if the socket is closed and we try to receive, we'll get EOF (Ok(0, _)), but we'll never receive any more data out of it, even if the other end tries to send more.

Now imagine we have one file handle shared by two threads, one thread writes to it and the other end reads from it. Then, when the writing thread is done, it closes the file handle. After that, (IMO) the reading thread should get an error, not EOF, because the reader wont be able to read any more bytes out of the handle. This other scenario is - I believe - more alike to what is happening in this commit.

If I'm right, and we revert the commit instead of modifying the tests, we'll be creating a technical debt. It probably wouldn't be too big of a deal, as we're already in debt by using blocking uTP API (it's still an itch though).


@vlj j'ai Eternal Sonata qui utilise ces 2 valeur la, il m'indique 0x0 et il y a deux 0 dans le enum ciblé, il y a soit "CELL_GCM_SURFACE_TARGET_NONE" soit "CELL_GCM_SURFACE_CENTER_1" j'ai tester avec les 2 ça crash, mais opengl a aucun probleme lui, je vois pas du tout quoi mettre la, si tu a une idée :/


I think not, as this is a new feature. The same goes for its cider.el counterpart.


It will resolve ok since the socket will go away, but will look into doing it there as well.


I think Electron in BRM is the same (Omnitrom Defense System).


Программа использует данные о прогнозе погоды для разных городов на ближайщую неделю с сайта http://openweathermap.org/. Записывает данные в базу данных MySql. Данные используются для получения средней ошибки для прогноза разной давности и составления таблицы для вывода ежедневных ошибок. На основной странице сайта отображаются ссылки для разных городов.


关于滤波以及相移的问题，还需要更细致的考虑。某些数据不需要重采样，所以没有相移，而需要重采样的数据，则有可能有相移。这可能会导致走时标记出现问题


Nevermind.. bufferptr(data, len) makes a copy.


See f951884


See 54838b2


Thanks! BitHub has sent payment of  $35.35USD for this commit.


See 65c5c97


See c144b00


See d516e07


Yes, it's implemented.


Without this patch, the build fails.


See d36212e


oh right i forgot about the race...
probably because everything was shown correctl<


Surely we should be suggesting `wrapping_neg`. Unfortunately it won't work for `const`s though.

Note that the following overflows so isn't a good suggestion in general:
```rust
let x: u8 = 128;
-(x as i8) as u8;
```


It looks good and it is surely a better solution than mine.
I would either remove the last `else` check in `OpenBlocksWikiTab` or add a warning to it before the fallback, since the lenght of an array is never less than 0.


I'm no Javascript expert but shouldn’t it be?
auth = new XiVOAuth("https://192.168.1.1:9497");


Ну да, должно. Спасибо!


No. I checked other IO operations
* [loading a layer definition file](https://github.com/SebDieBln/QGIS/blob/9ed33af7a64c9c253e65f75b063b5ebc51c3e511/src/app/qgisapp.cpp#L4608)
* [saving a layer definition file](https://github.com/SebDieBln/QGIS/blob/9ed33af7a64c9c253e65f75b063b5ebc51c3e511/src/app/qgisapp.cpp#L5544)
* [loading a style from file/database](https://github.com/SebDieBln/QGIS/blob/9ed33af7a64c9c253e65f75b063b5ebc51c3e511/src/app/qgsvectorlayerproperties.cpp#L842-L843)
* [saving a style to file/database](https://github.com/SebDieBln/QGIS/blob/9ed33af7a64c9c253e65f75b063b5ebc51c3e511/src/app/qgsvectorlayerproperties.cpp#L961-L962)

and there are no attempts to recover either. The user is only presented with an error message, either via `QMessageBox` or `pushMessage()`.

I would have gone this way for the `ParseError`, too, to stay consistent. But it is much more convenient for the user to just load the backup with a simple click of the mouse plus we can do some little extras like marking the project dirty.


Change made.  Thanks.
I actually dislike these couple sentences in general.  I feel like they're
trying to say NuGet and NPM support is not as good as Mavens (true) but it
does exist without making us sound too bad.  I left this as it was in NX2
really.

On Wed, Dec 23, 2015 at 6:20 PM, Steve Henty <notifications@github.com>
wrote:

> I'm finding, 'NuGet proxy repositories support {rhc} and identification of
> components is performed,' a little stilted. Can we rephrase to avoid the
> passive voice for '...identification of components is performed?'
>
> Ex: 'NuGet proxy repositories support {rhc} and component identification.'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sonatype/nexus-book/commit/0799cc90b034ac6a08808bd75dbd39a3176f7e3d#commitcomment-15147462>
> .
>


Better roll it back:)


Not sure, but maybe it would be worthwhile to have a such assert in other .get()'s as well.


is this not supposed to be `str(uuid.uuid4())`?


It is empty when user clicks one of non-item links.


I think you need to import logger


@ljacqu `getLabelCount` is a terribly confusing name


So if already doing it the nice way for ParseError, why not also for IOError?


Oh ignore that I just noticed you did inside the function.


Done thanks.

Aside: I feel like either how you're reviewing or how I'm viewing your
review, isn't showing comment association with specific lines.  So without
your example, I'd have no idea what sentence you wanted split.
If something on my end, I'll need to learn it, so thought I'd comment.
2 cents with especial thanks for the examples=)

On Wed, Dec 23, 2015 at 6:26 PM, Steve Henty <notifications@github.com>
wrote:

> Recommend splitting this sentence into two for readability.
>
> Ex: 'You can also remove these tasks manually. Adding them, however,
> cannot be done through the tasks administration and must be done as
> described previously.'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sonatype/nexus-book/commit/0799cc90b034ac6a08808bd75dbd39a3176f7e3d#commitcomment-15147538>
> .
>


lol. It seems that 5.8 will come soon. XD. I hope my next "pull request" will ctach it up.


Yes! Good catch


Reverted for what?


Hi, in the original Swimat repo also support space indent and with lots of bug fixed, you can give it a try.
Thank you.


Please add error messages to the `assert_raises`


Done.

On Wed, Dec 23, 2015 at 6:30 PM, Steve Henty <notifications@github.com>
wrote:

> Consider replacing 'Alternatively...' with 'Rather than...', or changing
> the phrasing entirely to remove the explicit reference to '...enabling and
> disabling...'
>
> Ex 1: 'Rather than enabling and disabling...'
> Ex 2: 'Alternatively, you can enable...'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sonatype/nexus-book/commit/0799cc90b034ac6a08808bd75dbd39a3176f7e3d#commitcomment-15147574>
> .
>


:spades:


Yep the both are correct :)


I know that - the reason for my question was your initial assessment stated as

>I have a feeling that the <style>'s within a custom element are not disposed properly


see 283d780ba2bf458dc4a3a7522e4392cad184c1b9


@jordan-brough Thanks for the quick response!!


I didn't knew that. I always thought that on an entity selection, IGW would fall back to the entity page.


You forgot to modify the `webpack.config.js` file. It should specify: `entry: './app/entry.js'` instead of `entry: './src/entry.js'`. Committing this in a sec.

You definitely should *launch the generator* before each commit.


Oh, I missed your point here :smile:
You want to load the backup in case there was an IO-Error loading the project file, correct?

I decided not to do that and in fact it is the only reason to have the `ErrorCode`. I assume an IO-Error will affect the backup file, too. IO-Errors are usually something like
* insufficient permissions
* network errors
* flash drive removed

But if you wish, I will remove the `ErrorCode` logic and simply do the check for the backup file regardless of the type of the error.


Quite a few lint suggested changes. I appreciate your willingness to do your best :-)


I have no idea, how it can happen, but it looks possible.

From javadoc on `onPageChange`:
```
metadata can be empty, or containing an itemstack or Class<? extends Entity>
```


aaaa


Bit laggy. no, too laggy


Sorry, this was a straight up copy-paste from the `IndentHash` comment. should I document `align_braces` there as well?


This also takes care of #420.


Wrong bracker correction I was looking at the script since it seems to not recognize the names in it. Please dismiss my proposed correction


Why?


@digitalentity
Can you check weather this works?

I tried applying this on luxfloat dterm, but ends up with erratic numbers.

Code I used.
           // Apply median filter for averaging
            for (deltaCount = 6; deltaCount > 0; deltaCount--) {
                deltaOld[axis][deltaCount] = deltaOld[axis][deltaCount-1];
                debug[3] = deltaCount;
            }
            deltaOld[axis][0] = delta;
            deltaSum = quickMedianFilter7f(deltaOld[axis]);


That's not my fault. N64 emulation itself is slow. Nothing to do with 64DD.


For causing troubles I guess.

@Jusonex should know more.


The style we've been going with on bio pages was capital for roles (even though they don't precede the name)— `Content Designer`


Yeah :) I'd say go for it, for network and flash drive errors the other condition will trigger anyway.


Never say never (says the assigned reviewer...)!


Needs` ConfigureAwait(false)`.


This build is broken. [It caused some text to be really small](https://dl.dropboxusercontent.com/u/7177600/aaa.jpg)


please see http://forums.openpli.org/topic/39743-skin-displayfor-et8500-not-works/?view=findpost&p=520932


@Sairii Well, you got a warning when the script was loaded^^


Total time of surgery steps rounded up. Can be scaled upwards if needed, or maybe the effect of manipulators could be increased? Right now worst manipulators should take 2.5 times the time listed with femtos doing it in the time.


you should also put the username as an option or hardcode `admin`


RE Aside: Figured it out, sorry my bad.


This seemed cleaner than the current way of an arbitrary function but its not really in scope of the change


Thanks for giving me credits on the donation part, keep making awesome scripts :)


because we didn't need to link with zlib on windows.


So far, I'm pretty pleased with that effect wich is pretty close to a "lighten" effect but also impacts image links wich is nice IMHO.
Will see if I get comments about this.


THANKS CELT


It should ceil health instead of casting it to int, otherwise health like between 0 and 1 will make client think that it's dead, while in reality it is not, and that will make player unable to respawn


I did a bit of research and it seems that if you quit the wiki while on an OpenBlocks page and then you press `I` while looking at an entity, it will try to open the entity page on OpenBlocks's tab.
I guess we will add an entity renderer as a fallback, sooner or later.


Cool!


Maybe live patients would have to confirm the operation from inside the machine before it starts? I'm open to suggestions


So, i converted the SM64 Disk Version to an .n64 format works with MAME but laggy Does not work with PJ64 i have the latest dev version. with 64DD support other games work fine. so how do i Play it on PJ64.
MAME really sucks in N64 emulation.


I took a quick look at a .cdt file in hex editor, and its format doesn't seem that hard. I can't promise anything, but I'll take a better look at it later, then *maybe* I'll make something for it...


Would this make sense to do something similar in /web/app/controllers/system_controller.rb?  Make sure to reset the GPIOReader on a stop event?

  def stop_race_session
    @race_session = RaceSession::get_open_session
    if @race_session
      @race_session.update_attribute(:active,false)
	  load "#{Rails.root}/lib/ir_daemon_cmd.rb"
	  IRDaemonCmd::send("RESET#\n")
    end
    redirect_to action: 'index'
  end


I thought putting the reason in the string is better than actually commenting about it


Lets introduce `err.notFound` early in the docs. It's a central concept imo.


Excellent work, going to merge this and fix a few copy/paste errors.


Done.


Check out aglio - it generates static HTML from API blueprint files and it has fantastic themes.

Just drop that HTML file in your project and have a route that points to it and you're done :smile:

> On Dec 23, 2015, at 8:06 PM, blacktop <notifications@github.com> wrote:
>
> I have been building a lot of Flack APIs at work and have been using http://apidocjs.com/ for that, I wish there was a API doc generator that built from the comments in Go. :( api-blueprint looks pretty great. Is there a way to host the docs or do you have to use Apiary?
>
> —
> Reply to this email directly or view it on GitHub.
>


Attention, tu as déclaré que le contenu renvoyé était du JSON, donc c'est une bonne idée de spécialiser le code de retour avec un HTTP 204, mais tu dois quand même renvoyer du JSON, en l'occurrence [] plutôt que du texte, sinon l'API n'est pas cohérente.


I think more faster will be Reflection.Emit.


Removed this because it's a bit misleading. There might have been an I/O error going on here so `'foo does not exist'` isn't necessarily true


Cool :-) This addresses all the issues I came across setting up my [Hello world](http://jean.github.io/posts/hello-world/) page.


Ah, but (Bertrand) Russell wants to know who reviews the reviewer?


Project64 does not really support 64DD. It just supports 64DD ports which is a different thing.
Besides, why would you play a buggy version of Super Mario 64...


I don't think this middleware should have the !guest check as authentication is completely optional.


Навіщо двічі логувати?


Формат дати винести в Constants, або використати існуючий там.


Hi @naveen and @rouson,
I'm looking into tidying up the CMake files a little bit (and also implementing the `FindMPI` module logic...) and was curious about this line. Please forgive my poor understanding of assembly etc. but I was looking at some CMake documentation and it seems that there are two usual ways that people detect information about the architecture:

 1. To detect the "bitness" (i.e., 32 vs 64) [the canonical approach seems to be to check `if ( CMAKE_SIZEOF_VOID_P EQUAL 8)`](https://cmake.org/cmake/help/v3.4/variable/CMAKE_SIZEOF_VOID_P.html) which indicates 64 bit system, or 4 bytes if it is a 32 bit system.
 2. The second way to detectcect the architecture is through the [`CMAKE_SYSTEM_PROCESSOR`](https://cmake.org/cmake/help/v3.4/variable/CMAKE_SYSTEM_PROCESSOR.html) variable, which, when not cross compiling, [is set to the output of `uname -p`](https://cmake.org/cmake/help/v3.4/variable/CMAKE_SYSTEM_PROCESSOR.html).

AFAIK, the output of `uname -p` will be `x86_64` on linux, but something else on Mac and Windows... on recent Mactels, the output is usually `i386` I believe. This leads me to my main question:

**Wouldn't it be simpler and more "canonical" to just query `CMAKE_SYSTEM_PROCESSOR` (which holds the output of `uname -p`) rather than to check if Linux AND then check if x86_64 by invoking a shell process?**

As I said, I may be missing something... Can you think of cases when `uname -p` might give something other than `x86_64` on `x86_64` Linux, or could give a false positive `x86_64` on a system other than `x86_64` Linux?


no punctuation no problems


Pourquoi une 418 ???


Changed the first 1/2 of the first recommendation.  Leaving the comma debate alone=)
I left 'Status' as if I went with your example, I think I'd be inclined to remove the word status altogether, which I think provides guidance on how to tell if it's offline currently.  If others feel this is confusing feel free to speak up (or change in later iterations=)).


Il aurait été mieux de changer l'auteur, mais d'en laisser un


How does it increase the number of match calls?


Done.


Так не можна парсати дату. в тебе є Date об*єкт,

Date date; // your date
    Calendar cal = Calendar.getInstance();
    cal.setTime(date);
    int year = cal.get(Calendar.YEAR);
    int month = cal.get(Calendar.MONTH);
    int day = cal.get(Calendar.DAY_OF_MONTH);


Dear Sir, You Put Wrong Logic There.. In your Code you trying to say that..If Given Input User is equal to DataBase User Than show User Already Exist.. Else Create New User.. This is Actually NOt Happening.

The Happen is When give input It Find Input in database if it exist it show User already Exist. But It Can not Create New User..

In My Code Im saying that Check if User is Exist or NOt..

That is why If Put if User is Null It means if User not Exits. then Creat User if not Than SHow Error.. This was the simple Logic I put..


Ah nice thanks for the info! Will definitely use that.


Le commit s'est fait sur tous les node_modules, ce qui rend son analyse plus compliquée.


main func


Can you walk through the `isSymbol` method.
Like first this because of that, then this thing because of this other thing.


Went with Detailed, thanks=)


Il vaut mieux laisser l'info de MAINTAINER, en changeant bien sûr pour ton identité. Ce n'est pas obligatoire à proprement parler, mais c'est la pratique dans les Dockerfile.


таке замінюється на  if(!$scope.selectedData.numberProtocol)


Ca aurait été bien de mettre un commentaire de commit pour expliquer le pourquoi du changement de port.


yeh, that's a typo - i reverted it in the next comment


was meant to not have the negation


This has 4 additions and 1 deletion


Neues Icon.


для чого постійно преприсвоювати calibrationModel . Достатньо присвоїти його тоді, коли знайшов


All I can tell you is that the machine outpaced me by multiple steps with basic parts.


As i said Earlier I dnt Just COpy Paste or Clone it.. I just Write it With my Hand In hole day.. SO that is why i just leave YOur Comment session.. and Didnt Read it..


Il y a un double // qui se balade, et aucun code source mis à jour pour les entrées d'API ajoutées. Je ne compterai pas des masses de point pour ce commit.


для чого тобі змінна model? $scope.manufacturerNumbers.push(data[i]); ??


if (!currentClibrationModel) {


I Remove becuse i think its not there may its useless. That is why i remove it..


OK, per semina gli inseguitori
>
> CC @kitsunenosaraT <https://github.com/kitsunenosaraT>
> Direi che è veramente troppo diretta :facepunch:
>
> Visto che è un pulsante, deve essere un'azione/verbo, quindi
> l'alternativa sulla volpe è da scartare a prescindere. Qualche idea:
>
> |Semina gli inseguitori Sfuggi agli inseguitori Fai perdere le tue tracce |
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla-l10n/www.mozilla.org/commit/9110e1bd311da8f7778b54f2a14abc81e818f912#commitcomment-15149894>.
>

--
Al Di Meola – Mediterranean Sundance
<http://www.last.fm/music/Al+Di+Meola/_/Mediterranean+Sundance>
Two things are infinite: the universe and human stupidity; and I'm not
sure about the universe.


This seems to create new session if it doesnt exist, and breaks the functionality.


$scope.IsScanDoc = ($scope.pathToScanDoc != null)


Oh yeah, looks like I put max instead of min in the time calculation. Pushing a fix soon


Bonne réécriture, même si une relecture aurait pu éliminer quelques fautes de ponctuation ou de frappes qui empêcheront d'avoir 100% des points.


Si, molto meglio Associato.
>
> Direi che "associato" è abbastanza generico da non causare problemi.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla-l10n/www.mozilla.org/commit/9110e1bd311da8f7778b54f2a14abc81e818f912#commitcomment-15151746>.
>

--
Al Di Meola – Mediterranean Sundance
<http://www.last.fm/music/Al+Di+Meola/_/Mediterranean+Sundance>
Two things are infinite: the universe and human stupidity; and I'm not
sure about the universe.


почистити закоментований код.


кльово, але видалити)


looks like you use a wrong word


L'implémentation du lock est TRES optimiste, mais je mettrai quand même des points pour l'effort. Attention, en vrai, il faudrait quand même quelque chose de plus costaud.


Согласен. Упустил этот момент, когда переписывал sql-запрос


Un petit commentaire de commit pour expliquer le require sur le fichier aurait été le bienvenu.


Le commentaire de commit devrait expliquer ce choix.


slack_api_url? Is  the webhook+channel?


I revamped the sentence slightly.

I am leaving the note markup and legalese alone as that's how it was and I don't want to emphasize something that was previously decided not to.
Since we've not redone CLM/PRO, I assume this will be reevaluated and maybe even change.


в наступних комітах це вже замінено на нормальне повідомлення )


Hey @timrogers
Wondering if you could do me a quick favour, and test out the holding page submission forms on this PR? Before, we used to have `prospect[metadata][sepa_country_interest]` on the 'register your interest in SEPA' form - any idea whether it's still set up to receive that value? Are we able to push in other country names such as Australia and the USA?
Thanks!


Non-urgent btw, so please go and enjoy your Christmas first ;-)


Why is this kind of change stuffed inside the arithmetic commit?


@Mrmaxmeier I thought Pootle would merge your german translations but something went wrong :( I'm sorry, the server wasn't ready eyt.


@arnova ping
http://trac.kodi.tv/ticket/16469#comment:11


видалив


So now we can start weekly builds (I think...)

Please build a new APK, put it on ```master/update/UltimateBrowserProject.apk``` and change the ```Update.txt``` to ```2015122401```, what means **first release** on **December 24, 2015**

I need to change the **version name** on my web server to **2.0.1**.


After that, let me test the updater, and if it works, you can release it.


I made a wording adjustment, but not really sure how to make this more clear.
That's one of my writing failings=\  Hopefully it helps.

Aside: I also found this paragraph out of "114" sizing, so I resized it.  Not sure why I didn't before.  It may make my specific change hard to read...if so, I apologize.


Thanks! BitHub has sent payment of  $34.60USD for this commit.


Well can you change the update.txt file to something else because if I change that right now I'll have to upload an APK to the server. So, do you want me to really create a distributional update or is it just a test?


лишня , видаляю


A couple of things. First, you shouldn't need to call `locksByName.get(name)` and then call `locksByName.computeIfAbsent(...)`; the latter should be sufficient and (for the `ConcurrentHashMap` type used) will be thread safe. Second, by doing this the synchronization is not required whatsoever.


Опечатка: не перемещён, а создан.


Seems like this logic will not clean up locks if unexpected problems occur (like an NPE somewhere in the call stack).


Isn't it possible that the WeakHashMap will drop NamedLocks that are still locked?


We need the correct certificate to do this. If you give me the password of the key store, as well as the password for the alias, I could do this myself.

I need to test if it works when you release a new version. We can do this and then change it back to the old version, no need to release a distributional update right now.


Just upload the apk to the new folder ```update``` in the repository and change the Update.txt


This is just a funny name for lower and upper bounds. Note that the acl2 procedure is essentially Fourier Motkin + heuristics for nonlinear arithmetic. I know they use this funny name in their paper.


Did you ship all cef_*_percent.pak files in the first place? Normally branch builds of CEF *should* be stabilized enough for typical use cases, OSR should be one of those.


No, what I'm saying is, changing the number in `update.txt` would trigger an update on all devices, so do you want me to release an update or is it just a test and you can change the `update.txt` file to something else just for the test?


- [ ] Can be omit `logout-url`. Please refactor to use default settings provided Spring Security.


Why did you comment this out?


chop comma
"the model"


Why is this done here and not atomically inside the `NamedLock.lock()` method?


Just to change Update.txt... To not trigger the update I would suggest that I create a new file and just change the URL in the source code for testing purposes.

You need to build the apk and upload it to the new folder I created in the repository.


where could a NPE occur from ?


I don't see it used anywhere.


Since there's no documentation for this, I think it would be helpful to describe the 'attr' and 'kwarg' options here as well.


"Writing R Extensions", section 1, "Writing R Packages", sub-section 6, "Writing Portable Packages", states, and I quote,

> If your package has a Makevars or Makefile file, make sure that you use only portable make features...  If you really must require GNU make, declare it in the DESCRIPTION file by

```
SystemRequirements: GNU make
```


Btw, this is how the updater looks like:
![screenshot20151214-upd](https://cloud.githubusercontent.com/assets/6959245/11996235/3f5d57b6-aa5f-11e5-9886-a986b9ab9c69.png)


Now I just have to check if the update procedure goes well.


I thought I told you this approach is not good. We should break ties based on the order simp rules have been inserted.


The entire approach is based on entries being evicted from this Map (I described it in the JavaDoc). So the use of `WeakHashMap` is intentional, otherwise there is no way to reliably clean up/remove locks which will cause OOMs in the end.

The trick is that the values in the `WeakHashMap` will strongly refer the keys until/while the locks are being used. As long as the key is double referenced (weakly from the key and strongly from the value) it will not be removed from the Map.


@Thunderbottom Wait, I think I can do this myself


> Commonly misused GNU extensions are conditional inclusions (`ifeq` and the like), `${shell ...}` and `${wildcard ...}`, and the use of `+=57` and `:=`.


Well, yes we can :-) I don't know what I was thinking about.


Unexpected. Merry Christmas!


And now I know. I copied that from the hugo docs :-)


And hey, I think we should remove the "Check for updates" which manually checks for updates and replace it with "Check for updates" which triggers the updater instead.


That was an example. The `validateLock` method is non-trivial since it uses synchronization, so it's possible that this might cause a runtime exception. Or, the system might generate a runtime exception at an inopportune time. Shit happens.

My point is that any problem within this for loop can leave locks locked, which would require a restart to fix. We need to avoid that situation at all costs, so the cleanest way to do this is to have a try-catch that handles any runtime exception


>You shouldn't need to call locksByName.get(name) and then call locksByName.computeIfAbsent(...); the latter should be sufficient and (for the ConcurrentHashMap type used) will be thread safe.

the get is used to improve concurrency, because if a lock is already there I don't need to synchronize.

>The synchronization makes this quite messy, but I understand it's needed to ensure unlocking is done atomically. Have you thought about not using synchronization above when adding missing locks, and then to hold onto the locks (rather than just the names) so that you can unlock them around like 93 without having to go back to the map?

not sure I understand

>This would greatly improve concurrency, since we wouldn't be locking/synchronizing on the already-concurrent map.

the map is not concurrent, which is why the `synchronization` is required in the first place


Already done


Also, where will you be getting the changelog from?


I suppose it shows the old changelog instead of the new one?


It's currently saved on my webspace


Then remove it, don't comment it out.


Unfortunately, I cannot download files from other servers on my website because the webhoster doesn't allow me to do that because I don't have pro.

Can you run PHP on GitHub servers? Or should we create a new webspace which supports that?


Isn't this always true?   The existence of the sbn file should be checked and the existence of the qix file should also still trigger (re)creation of the index.


картинок не обязательно ровно 5


need to add try - except block if one of the value is None otherwise it gives the following error:

<pre>
[2015-12-24 18:04:02,163] Mesuts-MacBook-Pro.local/ERROR/stderr: Traceback (most recent call last):
[2015-12-24 18:04:02,163] Mesuts-MacBook-Pro.local/ERROR/stderr: File "/Library/Python/2.7/site-packages/gevent/greenlet.py", line 327, in run
[2015-12-24 18:04:02,164] Mesuts-MacBook-Pro.local/ERROR/stderr: result = self._run(*self.args, **self.kwargs)
[2015-12-24 18:04:02,164] Mesuts-MacBook-Pro.local/ERROR/stderr: File "/Library/Python/2.7/site-packages/locust/stats.py", line 515, in stats_persist
[2015-12-24 18:04:02,165] Mesuts-MacBook-Pro.local/ERROR/stderr: store_stats(filename, locust_runner.request_stats);
[2015-12-24 18:04:02,165] Mesuts-MacBook-Pro.local/ERROR/stderr: File "/Library/Python/2.7/site-packages/locust/stats.py", line 504, in store_stats
[2015-12-24 18:04:02,165] Mesuts-MacBook-Pro.local/ERROR/stderr: f.write("%s,%s,%d,%d,%d,%d,%d\n" % (r.name,r.method,r.num_requests,r.num_failures,r.min_response_time,r.max_response_time,r.avg_response_time));
[2015-12-24 18:04:02,165] Mesuts-MacBook-Pro.local/ERROR/stderr: TypeError: %d format: a number is required, not NoneType
[2015-12-24 18:04:02,165] Mesuts-MacBook-Pro.local/ERROR/stderr: <Greenlet at 0x1069fd0f0: stats_persist('result.log')> failed with TypeError
</pre>


а потом поменяешь структуру таблицы, поле станет не 4, а 5-м и все сломается


Thanks! BitHub has sent payment of  $33.94USD for this commit.


Done.


Тебе нужна одна картинка, а ты запрашиваешь все


Maybe `attr_name` and `kwarg_name` would be better names?


Ok, it works.


@Mrmaxmeier it still isn't, running into some scaling issues with pootle. Hang on.


https://validator.w3.org/check?uri=https%3A%2F%2Fnameless-thicket-6873.herokuapp.com%2FList.php&charset=%28detect+automatically%29&doctype=Inline&group=0

нужно, чтобы документ проходил валидацию, надо исправить все ошибки


> the map is not concurrent, which is why the synchronization is required in the first place

Okay, you're right that the map is not concurrent. My bad for mixing the removed lines with the added ones. So I agree that you have to synchronize.

> > The synchronization makes this quite messy, but I understand it's needed to ensure unlocking is done atomically. Have you thought about not using synchronization above when adding missing locks, and then to hold onto the locks (rather than just the names) so that you can unlock them around like 93 without having to go back to the map?

> not sure I understand

If you're not able to obtain a lock on one of the locks, you unlock all of the locks you've already acquired by iterating through the lock names, getting the lock from the map, and then unlocking that lock.

Can you not simply hold onto the *lock* itself after you successfully acquire it? Then, you have a list of the acquired locks, and if you need to unlock them (because you couldn't acquire another lock) you can unlock them without back to the map by iterating over the list of locks. That saves you from having to i) synchronize on the map and ii) look up the locks by name.


Am I right that this will make dirlist always empty?


Thanks! BitHub has sent payment of  $33.35USD for this commit.


Why are you concatenating strings rather than using a second variable?


Maybe a comment about why these are commented out, if we want to leave them in?


I added --override-modules option because web module was affected. I had this doubt before sending the pull request. I think it should be a good option as generic demo deployment procedure. I'm not sure if --override-all should fix the error without creating new problems. I'll test on this demo.


hey, did you test the docker file? If it works fine, do you mind if I merge this commit into the master branch?


At the moment, we only accept `sepa_country_interest` in our metadata  - this gets passed to relevant fields in Salesforce and Mixpanel ("SEPA Country Interest"). For now, I'd suggest maybe putting it in there, and we should think about renaming it to "Country Interest" in the New Year.


You control where the `tryLock` methods are used, so you can change the signature to pass in the name (if that's needed).


It's temporary.  They won't be commented out for long, just pushed WIP on tests.


Yeah. Do this in a separate commit.


Borked on android :(


These changes need making on literally 100 packages on GitHub...


This needs fixing on laravel/framework 5.1 please. ;)


test


Maybe you could merge these renames separately so this patch doesn't look so big.


// cc @jbrooksuk


Hi. Can you disable rain over (above) biomes (areas) with default:snow and with snow blocks and disable snow  above non snow areas? Did yours mod work right with 0.4.13 and mapgen 7?


This line of code ```while (input.hasNext()){``` don't have a good style, it should be like this ```while (input.hasNext()) {```


앞으로 프로젝트 소개를 수정해야 함


Yes, the check should be QFile::exists( sbnIndexFile ).
Is it really necessary to check for the qix file? I had that in the branch before, but in my experience, gdal updates the qix file correctly.


*Ready for new release


No GitHub only allows static webpages, I know cause I tried to host a contact form once and it didn't work


Well, I suggest we could look for a good webhoster.


My route is
`
  devise_ios_rails_for :users

  devise_scope :user do
    post 'users/auth/facebook', to: 'devise_ios_rails/oauth#facebook'
  end
`

I successfully auth from mobile to use 'users/auth/facebook'.
And if you refer 'https://github.com/netguru/devise-ios-rails-example/', above problem and your #13 issue will be resolve too.

Plus, This library will be rollback to past commit without notice for business-related problem. So be careful above comments will removed without notice too.


The test would only fail on Python 3. I'm not sure it's worth it... I can add an explanation in the commit message though, no worries :)


Please squash the commits; you should use `git commit --amend` or `git rebase -i`; we don't want unnecessary commits in the history. (You can update the PR with `git push --force`)


To be honest @popey456963 that's a bad example because the PDF is blank.


Adds MonitoredProperty from feature/LinkedOceanData


Yes, it will, until #299 is merged (which I imagine will happen in short order anyway). :)


Fantastic work!


Love the page pings plug!


BTW, what I'm proposing is essentially what the `ClusteringService` is already doing [here](https://github.com/ModeShape/modeshape/pull/1496/files#diff-7a3817f69891df411ce49d5aee1ead1bR194).


Ну я синтаксис по памяти писал, возможно там надо что-нибудь типа all satisfy или ещё как-то. Но смысл такой


thanks! I will play with it further.


Very good catch, thanks. https://github.com/rackt/react-redux/commit/04693ca0cabe021b27b62eb9240b51459ffe8c32


You're right -- I'll explain it in an inline comment.


I have a hosting space but it's my Dad's and there's a lot of space. We can use the FTP there if you want.


test for github


Да, all satisfy будет работать (и это как раз то место, где satisfy уместен)


So you mean to say you already have a hosting space?


Да, матчер included_to хорошая идея :)
Вообще, правило «всегда тестировать subject» есть, но оно скорее рекомендация, чем Непреложный Закон. То есть сам бы я в реальном проекте не стесняясь написал вот так, как показал (хотя когда мне понадобилось бы написать так второй и третий раз — тоже сделал бы матчер included_to)


Thanks Craig. I will switch out the references to FQCN.


ack


I think these can be set in one of the constructors - I did not touch that code in this pull, trying to keep the blast radius to the minimum ;) I will capitalize the constants.


Yes we can do that. We just need PHP.


So it appears that the CMake documentation is incorrect, because on my Mac `uname -p` gives `i386` but the contents of `CMAKE_SYSTEM_PROCESSOR` is `x86_64`... So I have added back the test for Linux, but am going to try just using that CMake variable rather than invoking a shell process during configure.

**TL;DR** I'm tweaking this, but think I have everything sorted out.


I have one, but it's better if we do it how you said.

So don't release a new version, we first switch over to your webspace.


@akx then maybe put these commits on top of #299 if they're dependent on that?


this way it appears correctly in the history


They're not strictly dependent. I prefer keeping different features/bugfixes in separate PRs...


> The test would only fail on Python 3

What's wrong with that? The test is there so our CI system will automatically detect it in case somebody in some future things "why's he importing that there? Throw it away" and makes a PR :)


Thanks for the tips, Daniel! So awesome to get feedback from you!

I'm going to experiment with the "extract method" options you noted... that sounds extremely helpful!


Yes but won't it be hosted on my dad's personal webpage?


does this fix the laravel 5.2 issues?


Please notify me too @fasterthanlime when pootle became available  :)


That's interesting... the online lessons I took instructed me to capitalize the first letter in a public method... where private methods should begin with a lower case.

e.g.,

PublicMethod
privateMethod


Don't even try to download that. The site is already deleted... (And if not, the file is.)


But wouldn't that require a domain name?


Changed to "tabs" after further discussion.

On Wed, Dec 23, 2015 at 6:34 PM, Steve Henty <notifications@github.com>
wrote:

> If 'pills' refers to either tabs or vertical navigation, we should change
> the verbiage accordingly. 'Pills' is not a widely-recognized term for those
> UI elements, even within UX circles.
>
> Ex: "... accessible via the tabs below the 'Delete' button."
> Ex: "... accessible via the navigation links below the 'Delete' button."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sonatype/nexus-book/commit/b7fe1c8942bb6079aa0777fb9771f109cbefc5f1#commitcomment-15147618>
> .
>


Yes just do it if you want


Melhorias importantes, de uma olhada nos meus também?

https://github.com/Alzemand/Kefir-After-Install-Script

https://github.com/Alzemand/Kefir-After-Install-Tools (GTK)


nice!


add `enter whitespace`
```css
top:0px;

td {
}
```
instead of
```css
top:...
td:{
}
```


Did you get any `no provider for http` issues at all? Your change worked for me yesterday, but now it's breaking. If I revert back to `injector.get(Http)`, it works again.


You are right: $(git ls-files) lists files only, hence dirlist and lsfileslist are disjunct.


òtimo


No, but I don't want it to redirect it to my dad's page, cause it's his work site and he'd have problems with it if I asked him about it...


Não sabia lançar isso no gitignore, tnx


I thought I did, but I guess I forgot something or replaced the wrong version. So, I'm just going to (re)apply the latest CEF build as soon as I'm back home.


Well, alright. I was thinking about a separate test just for this case, though it's more an issue of increasing test coverage to cover this line in general. Will do.


I guess singleton is laravel5 style,  share is laravel4 style, but i'm not sure.

I found that : 'share' bind a service to a closure, and 'singleton' bind a service to an  array within Application. I developed [LaravelFly](https://github.com/scil/LaravelFly)  to run laravel faster,   and LaravelFly make backup  for Application before any requests and restore data after each request,  I'ts not possible to backup  services binded to closures

So I prefer singleton.


add constant variable and use it instead `12`
e.g. you can put
```ruby
# user.rb
class User
  PER_PAGE = 12
end

# and use it inside controller like User::PER_PAGE
```


thanks :)

2015-12-24 17:44 GMT+01:00 Aarni Koskela <notifications@github.com>:

> Well, alright. I was thinking about a separate test just for this case,
> though it's more an issue of increasing test coverage to cover this line in
> general. Will do.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/python-babel/babel/commit/7e80c475b504916d73898568be681481a7f8075a#commitcomment-15155432>
> .
>


this is bad. You can use simple `format js: render 'partial'`


remove it. immediately =)


Ok, forget it, I suggest we create a native updater so we don't need any additional webserver


look [here](https://github.com/zdubzdab/schoolTest/commit/be970666950fa95e592d1c1b3bf19c49f88b8a38#commitcomment-15155447)


Я удовлетворил ваше любопытство по подключенным и не используемым инструментам?


Singleton is the only thing that makes sense if you want to actually work using contracts.


Let me do this later, tomorrow or maybe later...


You can't assume the container supports array access.


i found the solution :+1: split pattern+dynamic routing


The throws here will run into the following code: https://github.com/liferay/senna.js/blob/master/src/app/App.js#L627

If `app.navigate' is manually called it will explicitly throw an error, which is expected.
If you click on links it will try/catch and navigate natively without senna.

Make sense?


Hi @ijansch, thanks for answering here. We're concerned because Buildozer does not add the GIT commit number to the version in the About-Screen of our app, @larsgrefer already sent to your colleague Ivo:

>The exact problem is the following line in our build.gradle file:
```
gitSha = 'git rev-parse --short HEAD'.execute([],
project.rootDir).text.trim()
```
>On buildozer gradle fails at this line, because it cannot find the program git to execute. I assume git is not available in the PATH environment variable or something like this.


Wait I think I can give access to my IP


D.C.


Any extension


Digital Transformation Lead


Product Manager


There is no branch called `current` in this project


Hm I can't get it to work as a separate statement either when there values already in the queues table that need to be migrated. I get

    sqlalchemy.exc.IntegrityError: (IntegrityError) column "durable" contains null values
 'ALTER TABLE queues ALTER COLUMN durable SET NOT NULL' {}

both as a separate alter_column() statement and if I try putting nullable and server_default into add_column.

I'm not entirely sure why this is the case, but we can just leave it as you have it now, without those extra options, since it seems your code handles null values fine.


Attention, il y a des symboles de merge qui ont été commités.


With reason for that file?


If you really want to do this in that way, you can use [DuckDns](http://duckdns.org) as Dynamic DNS.

But I think it would be better if we make the updater native. Today I won't have time anymore for that but in the next days maybe.


Yeah, I meant to fix that at some point but never really cared enough. I
don't think you can install PETSc without gnumake, so no big deal.

On Thu, Dec 24, 2015 at 10:57 AM, Nick Janetos <notifications@github.com>
wrote:

> Commonly misused GNU extensions are conditional inclusions (ifeq and the
> like), ${shell ...} and ${wildcard ...}, and the use of +=57 and :=.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jtilly/taoR/commit/d4a6e64dcceec6430e4d58ba895418c2250f1fbc#commitcomment-15155139>
> .
>


Yep, exactly.


Interesting way of doing this one, I did it with a pencil and paper in about 30 seconds after trying and failing to write code to do it for me...


Très bonne idée d'avoir ajusté le DataContract avec des attributs dédiés.


I dont understand how you change from wrong power factor to correct power factor,


Can you add an exception for Gyro Ball? It absolutely should not be tagged as viable. Tag it as viable only if the Pokemon has 40 speed or lower (40 because Forretress)


Maybe you don't have to change this. Maybe you can just say "x stocks remaining" and everything will be fine. Also, make it red when you go over. That would be good.


ready 쪽   jquery 를 흉내내야하는거 아님 ?

그대로 소스를 가져와 버리면  이게 우리가 작성한게 아닌데

문제가 되지 않을까?


Alright, as you wish


ok, makes sense.  although we might now be creating a qix file where none was previously present - but that's proabably a good thing.


just write `$.get "", formData, (html) ->` it should send request to the same page where you are.


remove it


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


why do we need the same changes on diff pull requests. If you need some feature for all changes just create difference small pull request


^^ and https://github.com/zdubzdab/schoolTest/commit/be970666950fa95e592d1c1b3bf19c49f88b8a38#commitcomment-15155497


Although you mention it in the README, there should be a comment here as well that this value is overridden with config.database_url.


So this is great now! Except I can't get the tests to work. I just ran them and got this for every test:

======================================================================
ERROR: test_abnormal_queue_name (__main__.GuardianTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test/runtests.py", line 127, in setUp
    dbinit.init_and_clear_db()
  File "/Users/mcote/projects/pulseguardian/src/pulseguardian/pulseguardian/dbinit.py", line 39, in init_and_clear_db
    for queue in Queue.query.all():
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/orm/query.py", line 2292, in all
    return list(self)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/orm/query.py", line 2404, in __iter__
    return self._execute_and_instances(context)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/orm/query.py", line 2419, in _execute_and_instances
    result = conn.execute(querycontext.statement, self._params)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 720, in execute
    return meth(self, multiparams, params)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/sql/elements.py", line 317, in _execute_on_connection
    return connection._execute_clauseelement(self, multiparams, params)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 817, in _execute_clauseelement
    compiled_sql, distilled_params
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 947, in _execute_context
    context)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1108, in _handle_dbapi_exception
    exc_info
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/util/compat.py", line 185, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 940, in _execute_context
    context)
  File "/Users/mcote/projects/pulseguardian/lib/python2.7/site-packages/sqlalchemy/engine/default.py", line 435, in do_execute
    cursor.execute(statement, parameters)
OperationalError: (OperationalError) no such column: queues.durable u'SELECT queues.name AS queues_name, queues.owner_id AS queues_owner_id, queues.size AS queues_size, queues.warned AS queues_warned, queues.durable AS queues_durable \nFROM queues' ()

Do they run for you?


This breaks badly for elements without any `position` explicitly set, `.style.position` would just return `''` and thus the branch would be executed for wrong elements.
This in turn contributes to dropdown being mis-positioned vertically when it changes "directions" from above to below, when, say, new ajax results come in.

However, `this.$dropdownParent.css('position')` would return `'static'` even though `position` is not set in CSS.


```javascript

// The DOM ready check for Internet Explorer
			function doScrollCheck() {
				if ( ReadyObj.isReady ) {
					return;
				}

				try {
					// If IE is used, use the trick by Diego Perini
					// http://javascript.nwbox.com/IEContentLoaded/
					document.documentElement.doScroll("left");
				} catch(e) {
					setTimeout( doScrollCheck, 1 );
					return;
				}

				// and execute any waiting functions
				ReadyObj.ready();
			}
			// Cleanup functions for the document ready method
			if ( document.addEventListener ) {
				DOMContentLoaded = function() {
					document.removeEventListener( "DOMContentLoaded", DOMContentLoaded, false );
					ReadyObj.ready();
				};

			} else if ( document.attachEvent ) {
				DOMContentLoaded = function() {
					// Make sure body exists, at least, in case IE gets a little overzealous (ticket #5443).
					if ( document.readyState === "complete" ) {
						document.detachEvent( "onreadystatechange", DOMContentLoaded );
						ReadyObj.ready();
					}
				};
			}

```

이게 핵심 코드고 나머지는 필요 없는 부분일 듯 한데.
이것만 우리쪽으로 잘 가지고 오면 될 듯 하다.


OK fixed in 9e86ef! Thanks for the suggesting the fix!


only for gl2?


Not sure I follow, but old contractObligations don't have the LastRevisionTxn field, so when they're loaded from disk they'll be initialized with an empty txn (and thus no revisions).


Can't we keep the fallback for LogPrint?


Hey @gboone - I'm in Nashville, TN - twitter is cgoranson - thx!


Oh nevermind, that's because I had an old test database lying around. I wonder if we should clean that up after the test... anyway it's fine for now.

Last step after you address the one little comment I have in the diff above: can you squash all the commits down to one clean one?

Thanks, this is good stuff! :)


I think that this code needs to be inside the IF.  No need to recreate the key if you are not invalidating anything.


this change I don't understand.
Signature for the constructor is still "address", so this now accepts either a list or a string?
I think the type (or the interface) should be more clear.


Thanks!


why `raw t('some_translate)` instead `t('some_translate)`?


Feel free to replace ifeq ...end by if ... fi or whatever it is.
On Dec 24, 2015 9:55 AM, "Jan Tilly" <jantilly@gmail.com> wrote:

> Yeah, I meant to fix that at some point but never really cared enough. I
> don't think you can install PETSc without gnumake, so no big deal.
>
> On Thu, Dec 24, 2015 at 10:57 AM, Nick Janetos <notifications@github.com>
> wrote:
>
>> Commonly misused GNU extensions are conditional inclusions (ifeq and the
>> like), ${shell ...} and ${wildcard ...}, and the use of +=57 and :=.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jtilly/taoR/commit/d4a6e64dcceec6430e4d58ba895418c2250f1fbc#commitcomment-15155139>
>> .
>>
>
>


я бы предложил взять отдельную функцию для получения страниц пользователя
сейчас работаю с этим
<img width="689" alt="screen shot 2015-12-24 at 20 13 45" src="https://cloud.githubusercontent.com/assets/15259299/11996733/d9b99bce-aa7a-11e5-870b-c4803ecd4487.png">


это совсем неправда.

решение
<img width="554" alt="screen shot 2015-12-24 at 20 15 00" src="https://cloud.githubusercontent.com/assets/15259299/11996745/06ba3f0c-aa7b-11e5-8e52-df054a099e25.png">


возьми обновления из моей ветки


but if we reach inside the if. we are guaranteed to regenerate the key. I think outside is fine.


Please preserve the formatting of the message, like so:

/fetch [player/id]


Is this even a sentence? Please use correct sentences.


storage_list.xml is deprecated and not needed...remove...all you need is the fstab entry....


A correct sentence would be: "LVP has a TeamSpeak server!"


In other words, we are not regenerating the keys when it is not necessary.


阿斯顿


There are only a few (or even one) commits in both your pull request, so it would be pretty easy just to do make a new branch off of dev and only keep the commits in these PRs that you want. Simple with `git cherry-pick`:

```bash
git checkout dev
git pull
git checkout -b new_branch
git cherry-pick 4657576   # SHA of the commit you want to add to the new branch
git push origin new_branch
```

Then make a new pull request. The SHA I used above is your commit ["Make all title references to K Clusters"](https://github.com/catterbu/receptormarker/commit/4657576b1311954e2b80704cb3a3148630137221).


the comment won't make sense when reading the code outside the scope of the review. IMO you might as well leave it out since the changed code will now simply conform to the contract.

But, have you followed the uses of this method to make sure that they are not depending on [] in the case of 'not paths'? It might be that callers *do* depend on this behavior, in which case it seems fine to me to leave the code and change the documentation.


m********s@hotmail.com


feku gedit


Thanks, @OpenGlobe! I'll get that updated now.


You have a syntax error here.


Ты ж хотел UnaryExpr добавить?


this line is a fast positive bailout in an env with native symbols, since typeof can't be faked.


```js
var rawPlayers = stdout.split('\n');
var players;

if (rawPlayers[0] === 'No Players Connected') {
  players = [];
} else {
  players = rawPlayers.map(function(player) {
    var out = {};
    var info = player.split(' ');
    out.steamid = info.pop();
    info.shift();
    out.name = info.join(' ');
    return out;
  });
}
var count = players.length;

callback({count: count, players: players});
```

This should output both name and steamid


this is a fast negative bailout in an env without `Symbol.toStringTag`, or, in an env with `Symbol.toStringTag` but someone has intentionally modified it to not report "Symbol".


What's this "in 3min" referring to? Do players need to spray every tag within 3 minutes? I don't think so, but this sentence does suggest this now.


and this is the final, slow check. if it throws an exception, it's not a symbol object (the first line already checked if it's a primitive)


This is a known issue (see https://github.com/select2/select2/issues/3970) that has not yet been fixed for the 4.0.2 release. Interested in making a pull request with the change?


@Mrbanks thanks for the heads up bro.. :)
Joey informed abt this ...so will remove storage list.👍


Include this in the previous sentence.

Also please don't use brackets in sentences, use something like: "or type /taxi 10 to quickly go to the LVP Main bank."


Fixed.
I'm not sure that this option actually does something useful.


@nadavrot after second thoughts, you are right.


Sober should be with a lowercase letter.


Good eye, going to hold on fixing it, though, until I know that's the correct title.


`Symbol#valueOf` is weird in node 0.11, and 0.12 possibly, and thus in some v8's. This certainly isn't robust against someone passing a Symbol object with a faked `valueOf`, but it's a fast negative bailout for someone who's mutated a Symbol object to have a non-symbol-producing `valueOf`. Essentially, in spec text, this is a "if Type(ToPrimitive(value)) is Symbol" check, but a "true" doesn't mean it's not a normal object with a Symbol-producing `valueOf`.


 :+1:
我本地代码最新代码还未来得及提交，你都已经发 pull request 了。


`symToStr.call` will throw if the value isn't a native Symbol with the proper internal slot. The symStringRegex test is just one of many ways I could have checked that it's a value wrapped with `Symbol(…)`.


added a number of comments


https://github.com/ljharb/object.assign/blob/master/hasSymbols.js may also be useful


Then could you please fix this as this is breaking setting mtime for dirs?


This example really helps me.


i am not saying that just pointing you that work on session to store logged in user then check user logged in before quiz route


@zbeekman I too accept that we can tidy up the checks. Regarding the check on whether the system is a LINUX system is kind of conservative, I suppose we can remove that.

Said that, I am little confused on what can be done on the second conditional check. We check whether the system is a 64-bit variant. I am not sure, whether this is completely correct. This was attempted to check whether the GNU build is a 32-bit or 64-bit variant. Due to lack of better analysis, it was assumed that if the system is a 64-bit variant, then the GNU build should also defn be the same.

I am not sure, whether someone would just build (or even if it is possible to build) just a 32-bit variant on a 64-bit system. If it is possible, then the check would pass and the test will fail.


yes checked that and fixed.


Rock, thanks! :santa:


this  (1/0) can be factored out at the top of the function


And, off course any change committed on this file should also be reflected on the navier stokes Make file.


Unnecessary. Source: http://publications.lib.chalmers.se/records/fulltext/144105.pdf


I don't like the name of this class.
Maybe KeyDocumentsUpdater?

A litte class docstring explaining what it does would be nice too.


I can't remember why I did this, but I'm kind of irritated that I added a bower component here. ****shrug****


that's the point. I have no idea why someone would use negation except it some super efficient inner loop math. So basically something like `if x == 0 { 0 } else { 255 - (x - 1) }` would be better


No I didn't  Add that..


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


@shahan-mik3  No problem :)


@neetVeritas, anyway to bypass RF ban detection ? When you log in and it says your account has been flagged.


some documentation notes saying that this int should be bumped each time that the document schema changes.


Why a placeholder ?


more than "update docs" (which sounds like a CRUD op), i'd use 'migrate documents schema'


I would do the opposite
`.has-state[aria-busy="true"]` should be display as `block` by default


`$fz--m` ?


@sukimanhung This will do a deep watch of `values`, which can get very slow if we have a lot of fields with complex values. Did you try something like this:

    scope.$watch(function() {
        // parse and evaluate the show_if Angular expression, providing
        // the values of the fields as the locals of the expression
        return scope.$eval(scope.field.show_if, scope.values);
    }, function(newValue, oldValue) {
        // newValue is the value of the show_if expression
        elem.toggle(newValue);
    });

This way, the `$watch` will only watch the expression specified in `show_if` field, keeping things nice and quick.


@Mrmaxmeier @matiasbeckerle looks like http://weblate.itch.ovh/projects/itch/itch/ is working!


Actually you could avoid the callback entirely...

```js
function players(callback) {
  execute('arkmanager rconcmd "ListPlayers"', function(stdout, stderr) {
    var rawPlayers = stdout.split('\n');
    var players = [];

    if (rawPlayers[0] !== 'No Players Connected') {
      players = rawPlayers.map(function(player) {
        // split the string in an array [number, name, steamid]
        var info = player.split(' ');
        // return an object containing name and steamid
        return { name: info[1], steamid: info[2] };
      });
    }

    return {count: players.length, players: players};
  });
}
```

and use it with

```js
server.get('/players', function(req, res, next) {
  res.send(players());
  return next();
}
```


maybe name a constant for this offset?


same comment than in previous commit. maybe it makes sense to use a wrapper that builds the content dict.


I tried it out. It's a lot slower and it might conceivably lead to some memory issues because it only works on lists, not arrays (but actually I don't think this will be a problem for us). We could maybe have it again be an optional dependency?


This indentation confused me at first, but now that I see the column at which this multi-line string begins, it all makes sense. :stuck_out_tongue_winking_eye:


maybe add a comment in Misc saying that this is a backward-incomp change?


its


It would be better to generate the icon when a Report instance is initialized rather than as a side effect of accessing a property. You could override `Report.__init__` or add a `post_init` handler. So, a slight refinement of what I suggested before is:

- Keep `Report.generate_icon` as a thin wrapper around `utils.generate_icon` (it does the `if icon exists` check)
- Call `Report.generate_icon` from the initializer (`__init__` or `post_init` handler)


This will almost certainly conflict with https://github.com/apple/swift/pull/762. If that one is merged in first, you could probably just remove this commit from the branch and all will be well.


Hey @jasonthomas, I think per https://bugzilla.mozilla.org/show_bug.cgi?id=1232617, we need https://github.com/mozilla/zamboni/commit/710241fe85591a9080bb3b60b2ae03357a9bbc99#diff-1a388eaa8d81768c26bb5381e8732f63L121 in production. Can we get this for the next push (1/12)?


Change filter


* int, realも予約語なので`int`,`real`としたほうがよいと思います


* タイポだと思うのですが「`arra`y型」になっています。


There is README. Instantiate the class, set allowed tags, call Filter::xss(). I don't know your situation, but usually it's better to filter data before output, not before saving to DB. You must use PDO and placeholders to communicate with DB.


* 細かいかもしれませんが、xとmもStanコードの中の変数を指すので `x` `m`とするほうがよいかと思います。


![Merry Xmas](http://www.decexcel.com/wp-content/uploads/2015/12/Merry-Christmas-Meme-3.jpg)


Ew grammar.


:star: :santa: :star2:


Will this work for 12.1 -
cm-12.1-YOG4PAS3JM-tomato-signed-fastboot.zip
??


Thanks! BitHub has sent payment of  $32.56USD for this commit.


* `real` a[10,10,10] → `real a[10,10,10]` a[10,10,10]までがStanコードなので。


You should do this in the `Player.spellpower` method instead, checking directly for the spellpower tag.


This commit adds support of Ruby and MATLAB/Octave syntax highlighting to WCM Commander


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1064](http://teamcity.monogame.net/viewLog.html?buildId=12931&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:23:53


Do you know if this fix works? There are 3 commits saying attempt to fix?


@rjmccall I changed APInt to const-ref.  What name would you use?


@slavapestov I agree. I just didn't want to re-indent so many lines of code because it makes the patch difficult to review.


It brings fanciness. Ok, I will delete it.


Is there a problem with people trying to use it on fast pokemon?


Can you please convert *.classpath* back to Unix line endings?


This is always true


What for? I added this function together with levels, because was too lazy to change it everywhere.
However if you need it for anything, just put it back


I am not 100% familiar with Prism codebase and my comment may be stupid, but what will happen you call CreatePageFromSegment(string.Empty)?


i think this is actually `bridgy_omit_link`. see publish.py:322.


@Jcubed99, I just picked up programming my sophomore year of highschool and it stuck. As for how I got into reverse engineering, before I was an active member of the rumblefighter "hacking" community, I was involved with MapleStory's -- which is to date, surprisingly the most robust. I just studied different public trainer sources, and studied the foundations of computer science. My first programming language was also technically Lazarus ( Free Pascal ), which I wouldn't recommend to any new programmers.

@neetVeritas, I haven't messed with GC's RumbleFighter enough -- the detection could have occurred while tripping any "hacking" errors.


```(job.title in icker.mode.disabled_jobs)``` is faster.


thanks Francisco.


curious.. if this an up coming feature or something already complete?  and if complete any docs?   curious about what it does (I'm assuming uses some external randomization website)..    Thanks!


`.has-state--inline` then?


Yes. It fits in the "usually useless" category.


Thank you! Merry Christmas!


controrls


Ça, c'est un commit douloureux… Je suis content qu'on ait des tests !


This isnt markdown, but using html only gains something if you wish to center image, else markdown all the way.

Image wise we can get sneaky and sly and use images already in the source.

![Kodi logo](https://raw.githubusercontent.com/xbmc/xbmc/2a4e1a03436901c9655c5103be2c8ce41e7fcb13/media/icon256x256.png)

```![Kodi logo](https://raw.githubusercontent.com/xbmc/xbmc/2a4e1a03436901c9655c5103be2c8ce41e7fcb13/media/icon256x256.png)```

Thers more ways and even better ways to do this however.


Excluding only from the `vendor/` directory was leaving behind `.css.map` and `.scss` files in `src/client-s/css/` when building ZenCache.


I'm not sure this makes sense; isn't the list of system directories dependent upon each user's system? For system-specific configuration, bfg tries to support this at the command line by setting the appropriate environment variables. For this, you'd do something like `CPPFLAGS="-isystem path" bfg9000 builddir`.


When I originally did this it was intentional to separate kodi intro from the remainder which isnt related. but meh.


Who knew a simple console could be so frustrating?


Like the isystem commit, the search_dirs are system-specific, and I think we should prefer a command-line way to specify these, rather than putting them in bfg files themselves. Static-only support seems ok, maybe, although that seems like more of a user-level configuration thing.

I could see some arguments to putting this in bfg files (e.g. if you wanted to do some magic autoconfiguration for people's systems), but I think we'd probably want to do that another way...


@blesh In fact, we might have to wrap **all** `Subject` subscriptions in a `Subscriber`, so that disposing of a subscription doesn't dispose of the Subject. For example:

```es6
const subject = new Subject();
subject.subscribe((x) => console.log(x));

// `subscriptionA` and `subscriptionB` === `subject`
const subscriptionA = Observable.interval(50).subscribe(subject);
const subscriptionB = Observable.interval(100).subscribe(subject);

setTimeout(() => {
  // this unintentionally kills `subscriptionB` too :(
  subscriptionA.unsubscribe();
}, 500);
```


你在干嘛...


•	Developed python application which can be used by employers for predicting future employee salary.
•	Data set from UCI Machine-Learning repository is used.
•	Implemented Decision Tree Classifier CART algorithm to predict the salary of future employees.
Python is used as a primary language of development. Library: scikit-learn, pandas, matplotlib.


Could you check again this logic ? What is supported is `self.divide==true`, right ? Currently if it's nil (old model from nn) it will pass silently. But the problem is that old models from cudnn doesn't have the divide field, and they are supposed to divide...


> Regarding the check on whether the system is LINUX is kind of conservative, I suppose we can remove that.

I thought so, but from testing it seems, at least on OS X, that this is still required. At any rate I've tidied everything a little bit.

The `Makefile`s are @afanfa's domain so I'll leave any edits required there to him, unless you, @naveen-rn, want to submit a PR, but I don't *think* that anything really needs to change there. The changes I'm making to the CMake system regarding this issue are less substantive than I originally anticipated....


This already is a 'from': 'to'.


Yes


Hmm, could this be the cause of #260?


That not what I was suggesting but I guess yes, actually its a better default behavior to have centered vertically and horizontally by default.


Sorry I'll update it. Need to pass `{ silent: true }` as a param into load. I would say until that other pr is merged this would be nice to have. I cannot use LaserWeb without it.


controrls


Probably. If you want to provide a PR feel free to work on it.


Doesn't this need a corresponding change to the menu as well?


conrorls


@nitsanw  Curious to know why you did not chose to use @Contended here for padding?


I totally forgot about bithub,  can I refund this as a freebie?  Do I just send it back to the address that sent it to me


Версия должна быть целым числом и она только увеличивается.
И в манифесте надо новую версию прописать


Merged here, right now.


That annotation is unfortunately not available without a command line flag
to the JVM.
On 24 Dec 2015 6:30 p.m., "kirtiteja" <notifications@github.com> wrote:

> @nitsanw <https://github.com/nitsanw> Curious to know why you did not
> chose to use @Contended here for padding?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/real-logic/Agrona/commit/f2ab87e5e84ff569d26aad41b735b30455388087#commitcomment-15156088>
> .
>


There are 13 additions and 9 deletions in this commit. Therefore to better understand what is happen after the commit it is better to take a look at equal by content pull request what have only 4 deletions:
https://github.com/official-stockfish/Stockfish/pull/532/files
If bits=64 CXX is either x86_64-w64-mingw32-c++ or x86_64-w64-mingw32-c++-posix.
If bits=32 CXX is either i686-w64-mingw32-c++ or	i686-w64-mingw32-c++-posix.

> The error is becuase there are no i686-w64-mingw32-c++ in my setup.

This means two things:
   1. Your development environment can be used for 64-bit targets only.
   2. Your invoked `make` for building of 32-bit target e.g. `make build ARCH=x86-32 COMP=mingw `
similar to command that you were using in comment  at the top of the current page:

>Testing config sanity. If this fails, try 'make help' ...

>mingw32-make[1]: Leaving directory 'C:/stockfish/src'
>c:/MinGw/bin/mingw32-make ARCH=x86-32 COMP=mingw all
>mingw32-make[1]: Entering directory 'C:/stockfish/src'

You need to re-test "Makefile before revert" for ARCH=x86-64.

> I have x86_64-w64-mingw32-g++ but in general it is really a bad idea to hardcode the binary name

1. If you have x86_64-w64-mingw32-g++ you surely have x86_64-w64-mingw32-c++, each mingw-w64 package contains it.
2. Agree with you that it is generally a bad idea of using hard-coded binary name, but particularly here it was smart one. Especially for mixed environments capable of building 32/64-bit targets. In addition Makefile became more fool-tolerant.


Thanks, I will merge this and expand on it a little, because right now it really only talks about how to debug values in show variables, not the other kinds of dynamic parameters. This is a good start, and I will add a bit about that to round it out.


这儿为啥要bind, current.hide就行了吧


好像。。没区别？


Any reason not to call the constructor argument rootSectionName to avoid this confusion? Seems to match well with layoutName anyways...


将ram.txt重定向到sim下面, 感觉更符合其用途.


...Sorry about the refactor, my IDE puts all "{" in a new line :P


rebase用的还不熟练......


And fixed face of buttons in the inventory :+1:


disagree


single assignment is easier to understand than mutation


note: commit message was supposed to read `"$ssmf_splat_normal:X"` before terminal ate it


:thumbsup:


Greate Work


There are 2 ways to use it:
1. Directly, as u did, and then there's no need for constant. This is
tightly coupled to Webpack and also a bit confusing for new comets as for
did it came from.
2. Use it only to define a constant plus a comment on where it comes from.
Fixes tge above.

I usef to use #1, but then the guys offered to use #2, and now it sounds
more logical.
On Dec 24, 2015 2:34 PM, "Maayan Glikser" <notifications@github.com> wrote:

> Why not? I didn't see it defined anywhere
>
> On Thu, Dec 24, 2015, 14:15 Alex Ilyaev <notifications@github.com> wrote:
>
> > @morsdyce <https://github.com/morsdyce> The whole point was to not use
> it
> > directly, and ENV.production should be true only after a build.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/500tech/bdsm/commit/e39b84922d41ea988fad21ae7a17db1794efee71#commitcomment-15153364
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/500tech/bdsm/commit/e39b84922d41ea988fad21ae7a17db1794efee71#commitcomment-15153513>
> .
>


Ah didnt know. Found this thread now http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-November/036509.html.


This should be listed as an incompatibility


Sincerly I've not tested it, but I'm happy to see that you want to grow the project!
If you need help, I'm here :wink:


That's a very good point. It is done.


What about adding to the non-standard table?


wow this is great


1. Ошибка-утечка памяти: не освобождается память, выделенная под узлы `*this`.

2. Формально, можно просто добавить вызов `clear()`, то есть перемещение будет выполняться как копирование. Но на практике конструктор перемещения (и оператор перемещающего присваивания) нужен затем, чтобы более эффективно реализовать перемещение (так как не нужно сохранять состояние перемещаемого `rhs`). В данном случае, можно, очистив текущий список, скопировать `first`, `last` и `size_`, в в `rhs` их занулить.


I committed fixes here https://github.com/phenotips/phenotips/commit/4c9c92812ada63c21ec1675bf6d44d2c89f17424


Yeah, I remembered about it afterwards, sort of added just now


You forgot to bump the version here
https://github.com/haf/DotNetZip.Semverd/blob/master/src/SolutionInfo.cs


closes https://github.com/okulbilisim/ojs/issues/1381


I am reverting your changes to package.json line 19 and 22, because it breaks how the crypto code works. If necessary we can debug it later, but the reversion is part of 42019591eb242c251d872bfc0aacd42535c4de43


reducing severety to warning is often better than completely disabling it


The reason why I introduced `ValueObject` class is that sometimes there is a need to store metadata. In this case, `LocalDirectoryAdapter` returns `ValueObject`s consisting of file contents as the data and file name as one of the properties (metadata). I would argue that in most cases there is no need for `ValueObject` as it raises unnecessary complexity to the code. So I would suggest either (a) adding another worker that would transform the data so that is complies with `YamlToArrayTransformer` or (b) rewrite `LocalDirectoryAdapter` (or make a new one and set that as default) so that it only returns raw file contents (not a `ValueObject` object with the data in it). Option (a) might look something like this:

```
->addSource(new LocalDirectoryAdapter(array('directory' => __DIR__.'/../resources/yaml')))
->addWorker(function (ValueObject $object) {
    return $object->data;
})
->addWorker(new YamlToArrayTransformer())
```

The first worker simply extracts the file contents (a string) and that gets passed directly to the second worker.

Option (b) might be something like this:

Ref. https://github.com/transfer-framework/transfer/blob/1.0/src/Transfer/Adapter/LocalDirectoryAdapter.php#L94
```
    public function currentCallback(CallbackIterator $iterator)
    {
        $clientFilename = $this->fileNames[$iterator->key()];
        $filename = $this->directory.DIRECTORY_SEPARATOR.$clientFilename;

        return file_get_contents($filename); // new code

        /* OLD CODE $fileObject = new ValueObject(file_get_contents($filename));
        $fileObject->setProperty('filename', $filename);
        $fileObject->setProperty('client_filename', $clientFilename);
        return $fileObject;*/
    }
```

@htollefsen What are your thoughts?


Response is set but never returned?


Right, should be just $services.families.deleteFamily($familyId, $removemembers)


The whole thing (code included) should probably be ssmf_splat_normal**s** though: https://github.com/spring/spring/blob/develop/rts/Lua/LuaOpenGLUtils.cpp#L69


Why Yahoo?
Also, I recently found this : https://github.com/charlierguo/gmail


Great! Thank you!

On Thu, Dec 24, 2015, 14:38 Amos Wenger <notifications@github.com> wrote:

> @Mrmaxmeier <https://github.com/Mrmaxmeier> @matiasbeckerle
> <https://github.com/matiasbeckerle> looks like
> http://weblate.itch.ovh/projects/itch/itch/ is working!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itchio/itch/commit/fdf32ae1d4dd98feffb50937b426694b50d6672a#commitcomment-15155770>
> .
>


@nuno-silva feel free to put real names if you want to, this was done before our talk on Facebook


We shouldn't be adding mock data. This will be loaded even in production. Mock data is for testing, not for the real thing.

We should be adding some real data so we can test.

Can you add instead:

## Technologies

Id | Technology | Icon
------------ | ------------ | -------------
1 | AngularJS | https://angularjs.org/favicon.ico
2 | GIT | https://git-scm.com/favicon.ico

## Courses

id | title | link | description | score | status | enabled | technology_id
------------ | ------------ | ------------- | ------------ | ------------ | ------------- | ------------- | -------------
 1 | Learn Angular JS | https://www.codecademy.com/learn/learn-angularjs | Learn AngularJS with the codeacademy course, direct from the browser, no environment setup required. | 10 | PUBLISHED | true | 1
 2 | AngularJS Tutorial | https://docs.angularjs.org/tutorial | AngularJS tutorial from angularjs.org. All you need to know to start understanding and developing AngularJS apps. Requires environment setup | 20 | PUBLISHED | true | 1
 3 | Try Git from Github | https://try.github.io | Try git using the github in browser platform, so you can understand the basics of this wonderful VCS, no environment setup needed. | 10 | PUBLISHED | true | 2
 4 | Learn Git | https://www.codecademy.com/learn/learn-git | The codeacademy course to learn git. Detailed yet basic, no environment setup needed. | 20 | PUBLISHED | true | 2


There seems to be a constant `MAX_CRUMBS`, another alternative is ARRAYSIZE.


Please use (example):
git fetch https://github.com/nitrogen-project/nitrogen_manifest.git
git cherry-pick <hash>
git push


That looks great, thank you!


@osheroff - Yeah, I'm in the same boat. Would love this to be in the next Ruby client-bindings release.


You might consider making this bean id more specific (e.g. "serialization.xstreamSerializer") just in case some other module naively tries to define "xstreamSerializer"


Можно ли это сделать другим способом? :+1:


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


It could be made MSBuild-agnostic way: `cmake --build . --config Release`. :smile:


Forgot to include "@" + domain


LVDS misspelled


Shouldn't this be 0x0000FF00?


Данная конструкция, типовая в операторе присваивания (кстати, там у вас её нет), не имеет смысла в конструкторе копирования.


`.nil?`


Makes sense but there is several modules i believe that use this module, don't you think it might break some of them if we assign it a new bean id?


Derp. I had my test script pointed at the production rig and assumed everything was peachy; that was dumb.


As you can see on line 304, an exception will be thrown.  Honestly, the only way that could happen is if you pass null in your navigate call, or you have a malformed URI.


Oh, if it's already "public" under this bean id it's fine to leave it as-is.

On Thu, Dec 24, 2015 at 11:24 AM, Wyclif Luyima <notifications@github.com>
wrote:

> Makes sense but there is several modules i believe that use this module,
> don't you think it might break some of them if we assign it a new bean id?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-serialization.xstream/commit/958317fd6169099af58738a42f9ca1e04db5fc56#commitcomment-15156320>
> .
>


I love you


Of course - not being critical just talking. And yes I noticed it wasn't in a <label> tag or anything which means that has to be overwritten.


Same here... all builds after this commit seems to crash on OSX(in my case El Capitan 10.11.2) after pressing play button.


I've also disabled the unit test for some functions that I have not checked or written yet.


I think this will remain empty string to support the case where no path is specified when creating route. Any other way you want us to address that?


You should not committed such a change. This prevents us from deploying the project to a Tomcat server which is NOT embedded


Fantastic - did you want me to do that?

or are you happy to?

kind Regards,
 Haydon

Kind Regards,

*Haydon Ryan | Advisory Solutions Architect | Pivotal Cloud Foundry
Services *

Mobile: 415-400-9377 | Email: hryan@pivotal.io

<http://www.pivotal.io>

www.pivotal.io

On Wed, Dec 23, 2015 at 7:02 PM, Bin Xia <notifications@github.com> wrote:

> @haydonryan <https://github.com/haydonryan>
> The script will fail if the user inputs "East Asia" as the location.
> We can change $4 to "$4" to avoid the error.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/haydonryan/bosh-azure-cpi-release/commit/55836f14afdefe971236618ba3edd05fb0604f4e#commitcomment-15149019>
> .
>


Hi Shawn, it turns out the original code was right! The JSON-RPC 2.0 specifications were designed to follow the XML-RPC specifications, where the explanation is clearer:
http://xmlrpc-epi.sourceforge.net/specs/rfc.fault_codes.php

I've updated the documentation to clear up any confusion caused by the JSON-RPC 2.0 specifications document. Thank you for bringing this up!


That's right. I think that was intentional....path is supported only when you have host name but we can check with @shalako and address it accordingly.


Лишняя строка попала


Great!


correct type for canola oil


> It is up to the translators to translate 'release_group' in English as 'release group' (and similarly for other languages).

This is just not going to work. It's better to explicitly specify two strings.


So... can't help me, right?


GREAT JOB!


Incidentally I love how much simpler you've made the code in this commit.  Good job.


Is it working now?
What a great Xmas gift If so.


Right, that wasn't the case I was worried about. I guess when you mark something as a singleton, it's just marked as shared isn't necessarily kept around as a singleton. Apparently laravel's container is smart enough to only keep one singleton instance around.

Here's an example of what I was worried we'd run into if we had two singletons (which I don't think we do) https://3v4l.org/veSPJ


Hey, thanks for very quick reply am so humbled. Am grateful for the filter
it has helped me.

My situation is that I have a small forum am using to test xss & other
injections. I wanted to filter the data before its saved on the DB & also
the data before it can be viewed from the DB on the web application. My
challenge is will I need to instantiate the filter every time am saving
data to the DB or is there a configuration I do without changing the apps
code

Regards,

Chris Njogu.

On 2015 Dec 24 20:50, "ymakux" <notifications@github.com> wrote:

> There is README. Instantiate the class, set allowed tags, call
> Filter::xss(). I don't know your situation, but usually it's better to
> filter data before output, not before saving to DB. You must use PDO and
> placeholders to communicate with DB.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ymakux/xss/commit/0c5c50f0bca64f62b8db61f9afc63fc59779ead9#commitcomment-15155892>
> .
>


Yes, that was my intent, but apparently I only remember to remove the ifdef around the initializer and not the field itself.


As I wrote - https://github.com/kangax/compat-table/pull/700#issuecomment-161471615


Ok :)


:+1: I did wonder if that would be better or not.


oh, maybe it was causing a segfault at the FCRs[0]


is this if ok? there's something weird about it.


Maps are not showing on minimap, and some other rgb items,...didn't tested many things, but found this to not work.


Shit, I've been meaning to amend existing PR so that you folks (@ljharb @zloirock @webbedspace) can take a look and double-check. Anyway, I added constructor test and fixed test overall as it was still using old syntax with subtests.


Better database for faster lookup?
Do not save database to disk for every entry, one per a number of inserts (20?) should be good enough.


I don't see any issues here on a first glance. Seems like a good idea. I'll take a closer look when I have some time in front of a computer.


The port should be moved to a config file


Yes, this should be done as a query or url parameter.


2GB files only right


Please review this branch


@maxmanuylov Thanks for the heads up! I'll send a fix.


Deleted.


Still not working.


Смотри, тут начинается сложный момент.
Разновидностей тестов в нашем мире существует много!
В данном случае нам нужно различить две большие вещи:
* модульный тест (unit test) — это тест на то, что каждый маленький кусочек системы и все их сочленения работают предсказуемым образом;
* функциональный тест — это тест на то, что «с точки зрения пользователя, система функционирует».

В данном случае, у тебя на сложную функциональность только тест что «она вообще работает». Это _функциональный_ тест как раз — он тоже нужен; но разработка без _модульных_ это грустно и неприятно. Напр., если ты что-то написал не так, ты знаешь только что Весь Тест Упал да и всё (а не какая конкретно часть поломана); то же самое если вдруг у TMDB _слегка_ изменится формат отдачи данных.

Эта ситуация заставляет разделить тебя тесты (и код! в этом-то и смысл! хорошее тестирование само собой обеспечивает хорошее проектирование кода) на что-то вроде:
* при вызове `top250tmdb` мы забираем из TMDB результат такого-то запроса
* получив результаты запроса, мы превращаем их в фильмы
* конкретный фильм из представления TMDB парсится таким-то образом
* получив от TMDB такую-то ошибку, код ведёт себя так-то.

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


Just "append" for the char and StringRef ones.  The one taking an APInt should be called "mangleNatural", since "natural" is the name of the production in the mangling grammar in ABI.rst.


And the item level should keep some corner for position,...if i put it on some +50, it's ok for big items but for small items it's in the middle...dunno how was in the past but with -3 to position...was ok for all items.


That makes sense. It's basically a side effect of combining Subscription and Observer into Subscriber. Before it would have had to be wrapped in a Subscription


I think there should not be a dash before $script, otherwise it wouldn't catch "fuckadblock.min.js" script from the official example https://github.com/sitexw/FuckAdBlock/tree/v4.x


aaa


Oops, I forgot that one :p
Yes it's correcr


I discovered today that my code was not scoring correct answers properly in Microsoft Edge when the submitAnswer() function gets called by way of a keypress event. It's a mystery as to why the numbers being compared in the submitAnswer function were being altered only in Microsoft Edge, but moving $getNumberButton.focus() from the keypress event code to the end of the submitAnswer() code seems to have solved the problem.


nice work ive been following along but didnt want to distract you. gui is really good, i knew its pieces could fit together somehow i couldnt decide any particular way so i guess i left it unfinished for others to decide.

in hijacked1 there's a lot of changes much i can not yet justify but a few of the important things that keep me focusing on it:
 * cleaner, less OOP-ish Term classes. everything is pretty much a GenericCompound. makes some things easier and other things harder but it can get sorted out
 * unified (per-Memory) TermIndex containing both Terms and Concepts. this is in essence a central symbol table for the memory allowing it to share references as often as possible. not only does it share Terms/Concepts in its index, but also the subterm vectors. so <a --> b> and <a ==> b> will share a common TermVector (a,b) defining their contents. this reduces memory usage, improves comparison speed (early-terminating == better than exhaustive .equals() ) and keeps hash/equality calculations consistent across types (most important).
 * functional adaptations of certain rule pre/post conditions. this is still not bug-free i'm sure it is violating certain order assumptions the rules were depending on but these can be discovered and made explicit.  this includes second-layer matching being performed within a function application, eliminating the distinction so it is all essentially one continuous phase.
 * ellipsis and term transform operators used throughout all rules. no fixed permutations deciding arity. this also seems to break some things but can definitely be repaired
 * constraints which are checked during substitution, to prevent invalid mappings as early as possible. this improves speed but also throughput since a substution can potentially recover from a miss that otherwise would only be discovered at the end of the rule application.
 * as a result of the ellipsis and constraints stuff mostly working i decided to convert the FindSubst "match any" behavior to a "match all" iterator pattern which delays all combinatoric choices until after all fixed components are matched.  these choices form a chain of Termutators which are essentially iterators that can produce zero or more permutations/combinations from the same set that i originally shuffled and choose one per premise match; this seems to produce much more output but it's AIKR adjustable because you can simply stop termutating at any point (it's a predicate visitor/iterator). this control replaces the unification power parameters and basically determines how many termutations a premise is allowed to generate.  this also can be used for determining solutions, since it uses FindSubst, allowing question to generate many answers per cycle.
 * ArrayBag (wrapped by CurveBag) is redesigned with a simpler API that is faster and more accurate. it uses a double buffering scheme to accumulate float-scaled deltas to each of the budget components until the next 'commit' where the bag applies them.  the scale parameter allows precise division between activation spreads without reallocating new Budget instances when they are applied to a recipient's bags.  the timing of these needs examined but the entire activation process is streamlined and more precise hopefully allowing more subtle budget dynamics to manifest.  termlinks are just BagBudget<Term> and Tasklinks are BagBudget<Task>. they stilll preserve the different budget value between the Task and the TaskLink.
 * Concept forgetting parameter is removed and concept budget is a dynamically determined value based on, for example, the average budget summary of a concept's tasklinks.  this makes concept bag seem more predictable, and i bet it is
 * improved Operator API anticipating the needs of advanced NAL9 operators.  the old API was clumsy for defining new operators and how they feedback results.  instead, operators just need an Execution instance that they Consume<> and it includes utility methods for all the different ways of applying feedback to the NAR (including asynchronously).
 * GUI experiments

its down to about 37 test faliures out of 700.  these bugs are critical and no correct behavior can be expected until they are addressed. these remain a challenge for me in the meantime i try to do more intersting things. still yesterday i managed to fix all NAL4 tests which have been broken in this branch for a while. so NAL1-4 seem pretty solid, some small error in NAL5, then NAL6 the problems begin and continue upwards.

![screenshot_2015-12-22_18-11-00](https://cloud.githubusercontent.com/assets/157093/11988125/98d52492-a9b8-11e5-9b7d-06e2328945e4.png)
![screenshot_2015-12-23_03-25-29](https://cloud.githubusercontent.com/assets/157093/11988129/98d984f6-a9b8-11e5-9606-ade43e63792f.png)
![screenshot_2015-12-23_03-27-10](https://cloud.githubusercontent.com/assets/157093/11988128/98d970ec-a9b8-11e5-886b-ed2bb7018909.png)
![screenshot_2015-12-23_03-43-45](https://cloud.githubusercontent.com/assets/157093/11988130/98dc9b6e-a9b8-11e5-824a-318a92654a7b.png)
![screenshot_2015-12-23_03-44-05](https://cloud.githubusercontent.com/assets/157093/11988127/98d9657a-a9b8-11e5-88c6-774ac499b62c.png)
![screenshot_2015-12-23_03-44-20](https://cloud.githubusercontent.com/assets/157093/11988126/98d929c0-a9b8-11e5-9811-7f5753e34bea.png)
![screenshot_2015-12-23_03-46-43](https://cloud.githubusercontent.com/assets/157093/11988131/98de6282-a9b8-11e5-974d-bd81ab94c6e5.png)
![screenshot_2015-12-23_03-49-53](https://cloud.githubusercontent.com/assets/157093/11988132/98e3cee8-a9b8-11e5-91b3-3b2e2e0f6cf4.png)
![screenshot_2015-12-23_13-18-57](https://cloud.githubusercontent.com/assets/157093/11988133/98e46c4a-a9b8-11e5-911c-a7812657b0e1.png)
![screenshot_2015-12-23_13-19-32](https://cloud.githubusercontent.com/assets/157093/11988134/98e6445c-a9b8-11e5-9c43-0ad512bd9db4.png)


I have been building a lot of Flack APIs at work and have been using http://apidocjs.com/ for that, I wish there was a API doc generator that built from the comments in Go. :( api-blueprint looks pretty great.  Is there a way to host the docs or do you have to use Apiary?


I was using jQuery to hide the quiz and instructions, but in the Edge browser you would see a flash of the instructions before jQuery hid them. It makes much more sense to hide some of the components by default using CSS since the browser will look for the CSS before rendering and take care of hiding those divs before anyone sees them. Avoid FOUC, grasshopper!


HTTP is basically what is happening between a client and a server. It has mostly nothing to do with what is happening on the computation part on the server side. To help me each time something like this is happening I'm trying to think about implementing HTTP as a desk/counter interaction in a shop between two persons.

1. A person (client) is making a request through a form to deposit something into the storage area of the shop. (The way the storage area is manage is not the client business here)
2. The person (server) behind the counter receives the something and check the parameters of the object and determines if it complies with their internal rules. The internal rules say two things:
  * Can I open it to check what is inside? "Oh. This is taped and locked. We can't open it. We can't accept that."
  * What is the content? "Hmmm that seems to be food. It will stink in a couple of days, or will attract animal. We can't accept that."
3. The person (server) sends back a response to the client.

**Quizz**: What should be the response?

* `500 Server Error`: The server says I have no idea what happened internally, but we fucked up.
* `415 Unsupported Media Type`: "The format problem might be due to the request's indicated Content-Type or Content-Encoding, or as a result of inspecting the data directly." [RFC7231](http://tools.ietf.org/html/rfc7231#section-6.5.13)

We used PIL as a tool to read (open the parcel) and inspect the image format (what type of content of the parcel). The reading error `IOError` is just part of the strategy we are using for this.

The useful answer to the user is `415` and probably with a message explaining why. :)


#3. → ## 3. たしか#のあとにスペースが必要だったはず。あとchapterは ## のレベルからお願いします。3.1は ### のレベルになります。以降も同様。

* データの一貫性 → データの一貫性のチェック


FYI, it looks like this broke the live site. i'm getting a 500 server error.


should call trim(input). wait.


BRING BACK MR. SKELETAL WELCOMING PLAYERS TO GAME START


Aww, that's very kind of you


The log class isn't available in this package so this won't work, I can see what you mean but I've just modified the original Laravel MemcachedConnector.php https://github.com/laravel/framework/blob/7f62d81acb2001faafb21a32544464ebfc92476c/src/Illuminate/Cache/MemcachedConnector.php

There must be a way of catching that exception somewhere else in the app


+1 LGTM


This commit broke square_codes and knight_path.


why would the copy prevent the null pointer? Why there's two copies?


delete


Do not blindly copy a PKGBUILD.  This is obviously a first version in KCP, so pkgrel is 1 (not 5!!!)
Run pckcp on your PKGBUILD before submitting:
http://kaosx.us/kcp-guide/#Submitting_a_Package


r+

(D'oh)


Can you elaborate on why you added this line?


Doesn't seem to have a parameter.


* データの一貫性 → データの一貫性のチェック
`data`ブロックで`<lower=0, upper=10>` とかでデータが範囲内かどうかをチェックする仕組みを示しています。


This line is blasphemous. It must be updated immediately, or else.