If we decide to stop building 32-bit, all we have to do is change the value
in the .ini, delete build64 and rename it to build32.

We could add a check so that if "arch=3" and build32 exists do that above.
Then if we move again to "arch=1" it all goes back to normal and build64
gets local git.

On 10 April 2015 at 16:57, jb-alvarado <notifications@github.com> wrote:

> Does this make a local git clone into the build64 folder? Maybe is better
> to make a copy, otherwise we can not run later only the 64bit compilation.
> It will be always depend on the 32bit git clone/update.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wiiaboo/media-autobuild_suite/commit/72c4808d840a495ec1c161238f48fd87964274c6#commitcomment-10669043>
> .
>


because what I want to validate is that a string can be parsed into a number of a given type, in a given range


yes


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


This until we figure out an easy way to just use a single build dir, which would be ideal since there's no more wasted space.


This is proof that even Cpt. Obvious gets it wrong sometimes too.


it is a first party lib not to be confused with th actual libexif.


I recommend doing `setTimeout(hitCallback, 0);` here instead... Often code written with callbacks assumes that the callback will be called after the current code runs to completion, and really weird bugs can creep in if this assumption is violated.


Это для первого курса на питоне, ваша группа  group-1652-2013


Being firm gives the opportunity to provide more descriptive and helpful error messages in the 400 response. Plus being liberal in what you accept effectively creates a pseudo-API that is more painful to break in the future.


Yep.  No point in changing the commit at this point, but the issue title was corrected.


since the `else` clause calls `hitCallback`, should this default no-op assignment actually be at the very beginning of the function, rather than within the `if` clause?


This introduces a bug as it always returns after the first if. I'm on mobile and can't see how to revert but it needs some braces.


I realize this delay fixed OoT, but have you tested other games?

According to the int03 datasheet, "With an official controller attached, there is a typical interval of about 6ms between successive updates. In fact, I believe that the update rate is controlled by the game, perhaps divided from the video frame rate. Each update lasts around 348us. The sequence starts with a 24-bit command from the console."


This one might have been written by Cpt. Obviously Lying...


Is `\phi_n` correct? This seems to indicate we have a `\phi` for every `g`, but `g`s correspond to data points and `\phi`s correspond to groups.


site.getsitepackages() returns ['/opt/conda/lib/python3.4/site-packages', '/opt/conda/lib/site-python']

The first list element is the right one, but the second directory doesn't exist.  I'm curious what's happening there.  Do you know?

I considered using a 'try' and change matplotlibrc in any directory that has it, but that feels hacky.  Other thoughts?


maker.type adds a dot to the left side of the type, which makes it undentifiable to java.
proposed solution:
JCVariableDecl var = maker.VarDef(maker.Modifiers(0), itVarName, maker.Ident(typeNode.toName(collectionType.toString())), null);


Recommend consistency in single/double quotes https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Strings#Strings


@mika OK, done in a later commit


 The reasons I prefer to use this image are:
* no need to use Q since the image comes with io.js
* small footprint (22 MB)
* isolation - I can use wide variety of technologies all isolated from each other (node.js, nginx, go, php, mysql etc)

I don't have NPM/Node installed on my laptop anymore. everything I need is inside containers.
But I understand that this approach is a bit extreme so it might not be appealing to majority of the users.


thank you Petr - fixed


Removed.


Yeah: *types to the right-hand side*!


fixed


maybe ?
char const* localeNames[OLD_TOTAL_LOCALES] =


Done.


hey man can you give me a call at 5:02 203_8602


pushed and rebased.


According to Oanda's API documentation trade expiration should be sent in UTC time if you use something other than UTC you risk setting the expiration to a date that has already passed.


bonjour
je fais des tentavives mais je rame sévère !


job.getClassType() == ClassType.STAGE_IN.value
keep the good coding style


You'd just want to use the version specific default of `site.getsitepackages()[0]`


@simongoffin
Good job, great that this got fixed!
Unfortunately, you missed the use case in case you choose to display the list view of variants.

Javascript is going to yell at the user every time you try to check the price ;-)
Uncaught TypeError: Cannot read property 'forEach' of undefined


listo aceptado el cambio


@majianpeng IIUC, in the case of an EC pool, if the `bad_peer` is a replica, we should also put the `ok_peers` into `missing_loc`, so that `ECBackend::get_min_avail_to_read_shards()` can take *all* available shards into consideration when trying to figure out a minimum set of shards to re-encode required ones when performing recovery, right ?

i agree that this change does help with the recovery in EC pool. but your change also impacts the way how we repair the bad objects in replicated pool:

- before this change: we append the bad object to `peer_missing[bad_peer]` if the `bad_peer` is a replica. so the primary is responsible to share the object with the `bad_peer` later on. and the `missing_loc` is not updated in this case.
- after this change: in the case of a replica `bad_peer`, we append the location (pg_shard) of good peers to `missing_loc[hoid]`. and in `ReplicatedBackend::prepare_pull()`, we try to find the object in `missing_loc`. but the primary does pull only iff when it has a missing object locally. so i think it's not necessary to append the good peers to `missing_loc` in this case. does it make sense?


It is better if you could just leave it unchanged here since we don't want everyone to change this daxPath


Fixes the search with spaces https://github.com/joomla-extensions/install-from-web-server/issues/18


It's a little more concise to use the x = obj.get("key", False) idiom, which lets you skip many of the default initializers above


Similar here for (Task task : job.getTaskList()) {
add space


Space between parameters.


TeamCity xBIM :: XbimWebUI :: Build [Build 47](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1343&buildTypeId=XBIM_XbimWebUI_Build) is now running


Bad package name.


LGTM


Log.print(task.getCloudletId() + ",");


Some odd spacing and indenting here.


Aqui tienes un pequeño detalle


Whhops, wrong branch!


It's more comforting to see something like `if not valid continue` because it easier for a code reviewer to inspect the block and confirm that nothing can possibly happen after this unless the validation has passed. Otherwise an indentation error might let something icky creep in.


test


TeamCity xBIM :: XbimWebUI :: Build [Build 47](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1343&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:44


TeamCity xBIM :: XbimWebUI :: Build [Build 48](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1344&buildTypeId=XBIM_XbimWebUI_Build) is now running


Tricky options :-)
Commented on https://github.com/odoo-dev/odoo/commit/875f6756077163e877b482a6d5ae423b09b0dc88


TeamCity xBIM :: XbimWebUI :: Build [Build 48](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1344&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:10


pour le moment je fais joujou avec les profils et les follows....


no whitespace around default arguments https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Default_Argument_Values#Default_Argument_Values


@vinkla Broken how? Did you open a new terminal after installing it?


Missing space between `if` and `(`.


jsonlds is not very readable. I can't tell what "lds" is supposed to mean.


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


nits


Use Title Case Here, or You Will Be Ripleyed :)


is the pidfile directory always going to be owned by root for apache? It's possible to run apache under user's besides root. Does this cause any problems?


Just curious: what's the advantage of ignoring mapping notify events?


Thanks, I've applied this fix.
Doug


;) I work that phrase in to as many things as possible.


Sorry about that, I also noticed it earlier this morning and had a commit on my fork. The story goes that make_unique was an oversight for C+11 and "most" C+11 compilers have it. Clang is a bit more "not in my spec, lol"


Thanks a lot @JDutil!


Would you like some more help with other textures?

On Fri, Apr 10, 2015 at 9:06 AM Sandra <notifications@github.com> wrote:

> Made new textures the textures. Thanks @AceK7
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/AwesomeSauceMods/OpenAutomation/commit/
> 6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28
>


Working on both of them now.


Why not use:
````
VMWARE_VERSION=$(vmware-toolbox-cmd -v | cut -d. -f -3)
````


I have been playing with Karaf this afternoon. :alien:
See https://issues.apache.org/jira/browse/KARAF-3654

Add sed lines to update Karaf's **etc/org.ops4j.pax.logging.cfg** file.
Replace **${karaf.log}** by **$ROBOCONF_AGENT_VAR_LOG**. And do not move KARAF_DATA anymore. They can remain in /opt...


lets add an input from the user of their name, and display it,


Lack of an empty constructor being is good practice to perform its implementation and facilitates system compatibility.


Thanks for the review!  Some of your nits are on code that existed prior, and I didn't want to update too much at once -- but I'll make a new patch to address them.


This respect pe8, but I'm not sure it tests what we want.  Because now the reference is automatically dropped before we check the refcount below.


I'm less concerned about field order in the UI and more concerned with checking specific values before acting on others at time of registration. For instance I only set the rewrite bits if the general "rewrite" is true. Easier to just handle myself than rely on trying to set things in some sort of UI order.

For i18n text review, go with 1.1, as that's the next major version. I'm doing 1.0.x updates as necessary and for hopefully quick changes.


Still needs more spaces


Seems my GIT completion doesn't work anymore.


Guillermo, can you please remove this pull request since you're giving the solution to everyone..
Thanks,
Doug


Thanks for inflict those penalties :+1: Although "*just test*" server is still enthroned on top of the list...


@Benglin It contains code for the original version of search, which I had deprecated as a part of the modularization work.


The third argument of this ```memset``` call is wrong, as Cache is not an array, but a pointer.


test


test 2


היי,
זה נראה שלאחר העדכון הכתבה הראשית בדף הבית של ספורט 5 נחסמת בטעות


Could you please put ShellJS back in?  I cannot use pdfjs anymore because of this.  I do not have npm, so I can't install ShellJS on my own; I need it to be a part of the pdfjs git repository, as it used to be.  I'm very limited where I work in what I am allowed to install, so I cannot install npm (I already had git).


I needed to add --use-standard-socket to the invocation to enable gpg2 to communicate with gpg-agent running as a daemon. Without this gpg2 would start a separate gpg-agent in --server mode and I'd need to enter my passphrase every time I used gpg2.


This was really motivating.


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (336 of 336 tests passed)

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


Can you also add a test that covers this case.  It's pretty important part of your diff, and currently isn't exercised by any test code.


Its just a suggestion, we can create separate properties file for each service and load it in each service test case as required.


Cool thanks. I was trying to do this with the `some` above, but this seems like a better way of doing it.


You should only need one click handler on .leftbutton - you can delete the one on lines 5-7, it is redundant w/ line 21


you can delete this


thanks


Could you file an issue at svn.boost.org so it doesn't get dropped?


You missed a few.


gpg2 communication with gpg-agent also works correctly when I add: use-standard-socket to ~/.gnupg/gpg-agent.conf


Thanks for the suggestions, I've implemented the prescaler in software as you suggested, it's working so I reverted the prescaler core in the Timer.


Please also check the return value of `CXBMCTinyXML::Parse()`.


Why don't you use `doc.RootElement()`?


celui la recuperait mal les pochettes


Well IMO it's just a matter of tweaking the reconcile to log the test names and failures...


done


I think something is wrong here. The LUKSDevice, which can have any format, always has a single parent whose format is an instance of LUKS.


i think `.compile` should only compile? Now it looks more like `.render` - rendering templates.

like in [jstransformer-toffee](https://github.com/jstransformers/jstransformer-toffee/blob/master/index.js).

I think `resolveToString` should beuse like `render`


Is it ok that we only really provide the return value of the last XML file (independent of whether it works or fails)?


згодны з ідэямі, наконт шматзадачнасці, то я разумею, што у сучасных машынах Ось і адказвае за шматзадачнасць, проста магчыма у проц трэба будзе дадць якіх небудзь інструкцый, ае гэта стане ясным пры канкрэтным праектаванні


@piosz how did you enable dns in this example? As I see you switched to v2 docker images, but not sure where kube-dns is configured.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/3c7b0ea069751cb0d24811f25f3da4cb2e1566ed/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/3c7b0ea069751cb0d24811f25f3da4cb2e1566ed/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/3c7b0ea069751cb0d24811f25f3da4cb2e1566ed/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/3c7b0ea069751cb0d24811f25f3da4cb2e1566ed/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/3c7b0ea069751cb0d24811f25f3da4cb2e1566ed/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


You'll have to add those as explicit requirements in lorax now that blivet doesn't pull them in. You may have to do something similar for live-kickstarts or whatever it's called.


thanks bb


І яшчэ я пакуль дрэнна уяўляю як ажыцявіць віртуальную странічную памяць, па ідэі гэта усё ведае сістэма але дзёргаць што раз яе на кожны мов у агульнай памяці неяк дзіўна, але можа так і робяць


It was removed in the original repository, and was brought over


TeamCity xBIM :: XbimWebUI :: Build [Build 49](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1346&buildTypeId=XBIM_XbimWebUI_Build) is now running


hmm or
```js
exports.compile = function _compile(str, options) {
  return compile(str, options);
};
exports.render = function _render(str, options, locals) {
  var renderer = exports.compile(str, options);
  return renderer(locals);
};
```


TeamCity xBIM :: XbimWebUI :: Build [Build 49](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1346&buildTypeId=XBIM_XbimWebUI_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:23


TeamCity xBIM :: XbimWebUI :: Build [Build 50](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1347&buildTypeId=XBIM_XbimWebUI_Build) is now running


TeamCity xBIM :: XbimWebUI :: Build [Build 50](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1347&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:03


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 tests passed)

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


Як варыянт перакінуць мов у групу  сістэмных каманд, каб на апаратным узроўне забараняць яго любыя вызавы для агульнай памяці. А прыкладным праграмам даць які не будзь сіскол па капіраванню памяці ў хуткую


it seems there no so much diffs between this `.render` implementation and `resolveToString` idea


- probably we should return right away, after calling `reply_op_error()`.
- i see this change as an optimisation for the case performing ops on a missing object before `find_object_context()` returns the same result. but is it worthwhile to optimise for a negative case ? just my two cents.


should probably return right away.


Rebased as 098d12f988f40da66171275538fd44d2b89f845d. Thank you!


what about in the linear scalar as part of the code that does the reversing -- getScaled  "if (rate_add_new * rate_add_old < 0)"  since this is the only case where this applies.


Rebased as 6a30cce13ae85f23268919fec99548247f40a2a0. Thank you!


@AlexandreAbraham Why do we not simply use a list to solve the problem that we do not know the required memory size in advance? Because it is at the cost of needing to double memory usage at the memory when we convert it to a numpy array at the end?


Inline without_coercion


MarkdownDocs (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1633 of 1633 tests passed)

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


"how to authenticate" (if that's right) for first usage, then "auth" for the rest is fine


That's correct. I thought that what I had added was another possible spot, but it looks like I might have added an extra spot that will never be used. The reason I'm hesitant about asserts in this case is because the only way I can do them is to replicate the calculations I did in `Chord::ResetAccidSpace` by making sure the width/height are greater than `4*(m_accidList.size() )`, and I know for a fact those were set correctly.

Two options to suggest:
- An assertion at every iteration of the while loops to make sure it's within the boundaries of accidSpace.
- Use `currentLeft` and `currentRight` or something instead of `currentX`, but the code might get much messier if I do so.

Would either of these feel safer to you? I can also add the assertions for the vertical dimension, but at some point this just borders extra documentation =)


Rebased as 23c838c4ae5671b2ec2bce732867b8d58613a16e. Thank you!


@john-a-m  Sorry, I have not directly commented on the line I am talking about. What has changed in line 79? The change in line 41 is fine for me. I am not sure what has been changed in line 79. Probably it is just a new line at the end of line 79 which would be fine.

Just as a note: I wrote a mail to api@oanda.com and asked them if they are still interested in the development of `oandapy`. At the moment there are a lot of hanging PRs and it looks like nobody maintains this repository.


Nice job man !


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


Was this committed accidentally ?


By default an agent token is created for each entity. However, you may prefer to use the same agent token for all entities. In that case, specify the agent token here.


Please do not make little cosmetic changes like this. It makes it harder for me to push our fixes out.


or if there are other issues with


LGTM - do you mind adding the same to indigo-devel as well?


shall we log the error here?


"This flag indicates if...", same for the next two


[JSTransformer](http://github.com/jstransformers/jstransformer) handles implementing the render function if compile exists. Don't need a render() if you have compile().


No problem, yea it was probably just a newline or something. I didn't intend to change anything on line 79.


@Sfolt, but #273 :)


This variable defines the entity name of the load balancing device against which the remote MaaS checks and alarms are configured.


drop=FALSE


No it's intended, why?


@aturon how about adding `font-size: inherit` to `code` in css? That way any inline code snippets are the size of its surrounding text. I tried it briefly using the console but have don't know if it would produce weird side effects.


Thank you very much!


I get it, the --args


Know that. The question is what is better to use `.resolveToString` or built-in render from jstransformer.

Anyway, this is the correct .compile implementation for me. Because users will expect that compile returns function instead of already compiled and rendered string.


Yes, especially because ```scriptArgs``` is not defined.


Fixed, thx!


This is handled like in all other functions. In case `Parse` failed we end up with no `RootElement`.


do we still need to check this? root has a rootRole, right?


Serwus. Das ist unser repository in Github!!


Thank you !
"patch-1" ? "patch-2" is comming ? *_*

Tips : "Update foundation_form_div_layout.html.twig" <- It's not a "good" commit name, try something which describes more what you really did ^^


added this
If the the version specified by the Client is not supported by the Server, it
MUST return `412 Precondition Failed` with a `Tus-Version` header; see
[Tus-Version](#tus-version) for the details. In addition, the Server MUST NOT
process the request.


IMO we shouldn't make the same "mistakes" in new code just because old code does it.


why do we need a special case here? could we let "root" always have "rootRole"?


warn about dropped probes


``` javascript
exports.compile = function _compile(str, options) {
  return compile(str, options);
};
```
`compile` does not return a `function(locals)`, so we resolve that by returning a new function to wrap it to the correct parameters.


Why do you need this here?  The include will come from the tope CMakeLists.


Nadal mi sie to nie podoba. :P
Obczaj: http://en.wikibooks.org/wiki/Java_Persistence/JPQL#Aggregation_functions
`SELECT COUNT(e) FROM OpwObwodowaKomisja o WHERE o.pkwId = :pkwId`


Will change.


@vrenaville Good question :)


This constant is not defined on 2003, and causes this method to fail with

```
The system call level is not correct. - QueryServiceConfig2: The system call level is not correct.
```


can we make the root role a global var?

``` go
var (
rootRole = Role {Role: RootRoleName}
)
```


恩，这样是很丑，我改一下


See [this commit and comment](https://github.com/johngrogg/ics-parser/commit/2fd2db6e59229bde7bed2b2745677bc995b6611b#diff-943ec5e7e6b8ee5e1a4a129123fa94d4R256)


I think `security.Error` is better than `security.SecurityError` here.
Reference: https://github.com/golang/go/wiki/CodeReviewComments#package-names


resizable returns a boolean indicating whether or not a thing can be resized. As I see it, that includes whether the required utilities are available.

The end result of this project should be an expanded awareness, but it should be possible to achieve that without requiring that the user perform additional checks in the course of setting up already-supported operations.


I don't know the details about the technical details regarding the `Clear` function but avoiding the crash you are describing should be easy. It's just a matter of querying both interface and calling `Clear` only when `ISubRenderConsumer2` was successfully queried.


Actually my +=($@) it buggy I think, I should insert the remaining args into the array one by one.


Do the find_package later on, and make it conditional on JSONCPP_INCLUDE_DIRS being undefined.
Then put the include in the bigger block at line 58.
Also, name it JSONCPP_INCLUDES for consistency with the others.


I actually authored this, sorry about the confusion. I am using someone else's computer.


setNames rather than structure


```
~/W/m/m/mono (master)$ ag --cc -s '\sexit \(-?[0-9]+\)'
dis/main.c
1774:		exit (1);
1940:	exit (1);
1971:					exit (1);

metadata/attach.c
428:		exit (1);

metadata/boehm-gc.c
191:						exit (1);
196:					exit (1);
207:				exit (1);

metadata/domain.c
599:		exit (1);
739:		exit (1);

metadata/pedump.c
412:	exit (1);
717:		exit (1);

metadata/object.c
3648:			exit (-1);
3702:			exit (-1);
3712:			exit (-1);
3726:			exit (-1);
3737:		exit (-1);

metadata/sgen-memory-governor.c
251:	exit (1);

metadata/sgen-marksweep.c
2046:			exit (1);

metadata/sgen-split-nursery.c
394:			exit (1);
405:			exit (1);

metadata/threadpool.c
1835:				exit (255);

mini/aot-runtime.c
865:				exit (1);
1524:				exit (1);
1526:				exit (0);
1798:			exit (1);
1827:			exit (1);
2067:			exit (1);

mini/aot-compiler.c
3815:					exit (1);
6377:		exit (1);
6382:		exit (1);
6400:		exit (1);
6502:			exit (0);
6539:			exit (0);
6542:			exit (1);
8642:					exit (1);

mini/driver.c
206:				exit (1);
279:	exit (1);
948:		exit (1);
1033:				exit (1);
1043:					exit (1);
1049:				exit (1);
1056:			exit (1);
1304:#define error_if_aot_unsupported() do {fprintf (stderr, "AOT compilation is not supported on this platform.\n"); exit (1);} while (0)
1391:				exit (1);
1406:			exit (1);
1417:			exit (1);
1861:		   exit (1);
1865:       exit (1);
1868:       exit (1);
1893:			exit (1);
1899:		exit (1);
2011:			exit (1);

mini/debugger-agent.c
870:		exit (1);
885:	exit (1);
925:			exit (0);
938:			exit (1);
954:		exit (1);
959:		exit (1);
966:			exit (1);
1024:			exit (1);
1092:			exit (1);
1244:			exit (1);
1262:				exit (1);
1324:				exit (1);
1330:			exit (1);
1355:			exit (1);
1364:		exit (1);
1422:		exit (1);
1426:		exit (1);
1495:		exit (1);

mini/main.c
63:			exit (1);

mini/mini-exceptions.c
2360:			exit (1);
2401:		exit (-1);

mini/mini-gc.c
2479:		exit (1);

mini/method-to-ir.c
3702:			exit (1);

mini/mini-runtime.c
2683:			exit (1);
2920:		exit (1);

mini/wapihandles.c
76:		exit (-1);
82:		exit (-1);
121:	exit (0);
201:		exit (0);
213:	exit (0);
245:		exit (0);
270:	exit (0);

profiler/mono-profiler-aot.c
81:			exit (1);

profiler/decode.c
2561:			exit (1);
2581:		exit (1);

profiler/proflog.c
2926:		exit (1);
2992:		exit (1);
3269:			exit (0);
```


To me it looks fine.


scoped_ptr.h has to go. The fact that it was missing is an incomplete push out of stuff. We now use std::unique_ptr exclusively. If that is not true for our code I will push a fix for that today. Remove basic_types.h as well.

util::Status I need a deeper look at.  The master branch is currently broken w.r.t. that.


I thinke "Cannot modify %s role, RootRoleName" is good enough.

Or we will introduce an unnecessary concept `superuser role`.


"No root.."


These `if`s could be combined.


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (336 of 336 tests passed)

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


You need clearProgress() at the beginning of each page, otherwise it'll estimate the total number of tasks including the previous step's tasks and never make it to 100 percent.


Don't you mean: '2.0.0beta1' (no dash)?
https://rubygems.org/gems/roo/versions/2.0.0beta1


well were not Pentoo , I pushed them to community overlay for testing and or bumping (emphasis on version bumping/research)   main repo Spike-overlay their are a few bumps in ebuilds , I've submitted things to Pentoo witch we sync from upstream, sometimes i get trolled for my dyslexia , ok make that any bleeping typo in chat. .... sometimes  dose my research go ignored. even after unpaid 100's of hours of it.
even my'n own  carefully git-r3 class ebuilds seem to fly over you know live low maintenance ebuilds
 we are but 3 Graduate student's pentoo has more , we are a Sabayon based distro , and so is  BlackNoxis's .
https://github.com/schism/haking4xemu/commits?author=necrose99
, I have found a few bumps already.

I'm trying to test and find good quality packages for the tinderbox, trying to find updates not in pentoo
is often a pain in the ass,  however maintain parity with upstream sources be they you , or Pentoo in a timely fashion as a Graduate student in IT security is difficult time-wise ...
sometimes APP:Witchcraft finds similar ebuilds, or already has them, from pentoo, etc. https://github.com/Spike-Pentesting/App-witchcraft
the tinderbox euscans, if minor bump without a patch and do QA testing Dajore and build it for bin repo.
I'm not the best programer in the world but I can try, yet i start to grasp foreign languages stupid fast go figure, , but yet programing languages and dyslexia it take chronic levels of practice , sometimes I feel like a trip though a wood-chipper, the code start hurting my eyes and head after a good bit.
 I've been more on the infrastructure side of IT & Security hands on.

My Vacation, is going to be spent house hunting in Dallas , since our jobs are relocating in ~June...
so doing research over the hotel slow-fi is going to blow chunks ... 56k nerner nichshssh comes to mind @ how slow..........................
I've also been quietly doing proxy maintenance on packages on Gentoo,
so feeding back bumps or other finds should be of service , your ebuilds have been handy for study /furthering research and from them I have made a new  Mallorca ebuild and etc, returning improvements as I have time to is the least I can do, i just droped into new workspace , pruning ones already bumped making a note of them , not everything needs to be on that repo so i'm trimming ,  to the few I'd like to keep or move to tinderbox for bin-repo host and CI server to build till cos come home,

I could have not cited my sources and outright plagiarized. , however for now i'm beating on packages as i can seeing if i can bump the versions , many packages , . plaso was already upstream ?, however i was wrong ,  once in the tinderbox they wont be as viewable on that repo...
schism/haking4xemu is a repo that is know by few and Greatly appreciated.


single quotes


cc @fdansv https://github.com/CartoDB/torque/issues/169
@xavijam


Missing space


Reformat the comment to respect the codestyle, 80 characters.  Avoid orphan words on the line like in ` * nb_columns-1`.


Just already enabled?

We already have the "security: " prefix.


just already disabled?


Shouldn't this return false?


By the way @tarsius, don't forget to add `(ido-ubiquitous "3.0")` to magit's package dependencies.


are you sure? I checked it before comment.
https://github.com/medikoo/es6-template-strings#usage

part of the example there
```js
var compile = require('es6-template-strings/compile')

  , compiled = compile('Welcome to ${siteName}, you are visitor number ${visitorNumber}!');
```


Remove condition


Does this mean that `ContainsFiles()` always has to be called before `GetDirectory()`? That seems like an odd implication.


This check is not correct.  The function accesses `relation->m[i][column + 1]`, thus with `column = relation->nb_columns` it is off by 2.  The proper condition should be something like `column >= relation->nb_columns - 1`, which, by the way, would correspond to the documentation comment.


GO!


Why is "git clean" necessary here?


Ca se passe ici : http://labs.iri-research.org/catedit/


Export makeSummarizedExperimentFromExpressionSet


Missing documentation for Mapper functions


that's why I'm against this too simple testing flow that does and test nothing. @jstransformers/owners


I don't believe there is any mandatory rule. I started doing it because I think it looks cleaner. It is easier to find strings and where they are used, quickly see any misspellings or errors, etc. A bonus is that if they end up being used more than once, there is no need to go back and create the constant. It is already there. If there is a need to externalize, the strings are already collected and easy to move. I have no problem with changing this if there is an official directive on this.


do we want to support '/a*' matches '/aa' too? If not, this func can confuse users.


it shouldn't be - i can't quite work out why the tests aren't passing, but the errors look like this:
```
Invalid proto descriptor for file "9dde3724-e852-4aa3-8f94-a95432ab2086.proto":
  Foo2.uint64_field: "Foo2.uint64_field" is already defined in file "7ce7f340-f090-43a9-93c8-6527771ebae1.proto".
Invalid proto descriptor for file "9dde3724-e852-4aa3-8f94-a95432ab2086.proto":
```
which might suggest extra files lying around. I can't reproduce the failure locally either, so I'm stabbing in the dark right now.


rename man page


Note about rownames used for matching


code reuse. see filedirectoryfactory and it should be clearer.


Can this be done directly without the lambda?


There should be code that adds a margin-bottom when you open the composer so you can see the last post when you are typing.

If we just remove this the margin-bottom will stay on the topic.


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (336 of 336 tests passed)

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


Add example (coercing sample.ExpressionSet)


Yeah, I'm totally with you. Before we do that we just need a solid code test coverage.


`pages.times do |page_num|` ?


LGTM


Simple Tests


MarkdownDocs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


Can we define nullptr to 0 if it's not C++11?


i thought we were gonna do that, but using this as a skeleton base thing for now


Exactly, so could you not remove weapon categories entirely?


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (336 of 336 tests passed)

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


This is a good start.


The approach for these annotations is that there is an apply method for both Analysis and Data Source Wizard data source types. The annotations are applied differently in each of those cases. Each annotation supplies its own logic for applying the change represented by annotation for each of the respective models. In this case, we are following the pattern established by architecture and also being followed by the SDR team (See CreateMeasure, CreateAttribute). I do agree to some extent, which is why I have worked to abstract some of that logic to helper methods of a handler class in the case of Mondrian schemas. If there is a larger architectural concern, we can bring this up with Benny and the Venom team.


Wouldn't it make more sense for this function to accept loop depth as `is_loop_carried` does?


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/0308481c1fdbb1c1778dfb8e883f67c04e381852/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/0308481c1fdbb1c1778dfb8e883f67c04e381852/appconfig.json)


Thanks for the suggestion. I've been reluctant to formalize the version system, just because it doesn't match particularly well with the continuum of code changes (I'm not _really_ preparing "releases"). But we probably might as well since the seeds are based on them.


Because there are separate tabs in the f4 menu. In the end I never should
have made weapons a shipment thing.


Are you sure it's useless and not mistake? Maybe it ought to be ```group->state == SPU_THREAD_GROUP_STATUS_WAITING_AND_SUSPENDED ``` ? In cases like this it shouldn't be hastily deleted without finding out what the original plan was.


This is actually not new, and this generated class has been removed in the subsequent check-in, as per Benny's previous feedback. As far as the values being applied, this is the convention currently used for applying formatting in the Analyzer reports. I was simply attempting to utilize the same pattern for the models, as opposed to introducing a new way of managing formatting in Analyzer.


Woops, resolves #2 is what I meant...


Man [rodo](https://github.com/python-dirbtuves/it-brandos-egzaminai/compare/master...dz0:patch-1), kad šakoje patch-1 yra tik šis vienas keitimas.


LGTM!


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (336 of 336 tests passed)

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


Again, this class has been removed from modeler in the next commit, and is the way Analyzer currently applies formatting in reports.


return_value_handling (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


You totally right! Made that change in https://github.com/slackhq/SlackTextViewController/commit/2b6e48d8d702ab986a75127bc004f539ff74de7d


On Fri, 10 Apr 2015 07:20:56 -0700
dacook <notifications@github.com> wrote:

> | @schism it's easier to keep the ebuilds in the distro-based overlay.
>
> Not really; I've been doing this in my own overlay for years and was
> doing this because I chose to.  After submitting literally hundreds
> of g.o bugs (old alts) with patches and ebuilds and mostly silence or
> ignorance from both them and pentoo, it's been far simpler to
> maintain them for myself.  I should be indifferent or happy - they're
> by default public and someone else is benefitting from my labors, but
> I can't help but be frustrated that you've taken a large,
> carefully-curated solo repo that was explicitly out of "circulation"
> and imported it wholesale without even contacting the developer first.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Sabayon-Labs/spike-community-overlay/commit/65e72a49ab531cf3ddae98af94a25c00c51e23c2#commitcomment-10667089
There is no "need/enforcing" rule that says someone should contact you
before forking, or copying , or else your "private" work. That's why
GPL is so good. I know you think there is a principle involved, but
there is none. You will have the right to be pissed when someone forks
your work, makes some improvements and he never contributes back.


I would add some tests for these functions, e.g a couple scops with dependencies.  Something simple like polynomial multiplication or vector operations should do it.

Otherwise LGTM.


What about use CreateRole and UpdateRole here?
you always need to figure whether it is created.


Keep reading. `compiled` is not what you think it is...

``` javascript
var compile = require('es6-template-strings/compile')
  , compiled = compile('Welcome to ${siteName}, you are visitor number ${visitorNumber}!');

// Welcome to MySite, you are visitor number 137!
console.log(resolveToString(compiled, { siteName: "MySite", visitorNumber: 137 }));
```

`compiled` is the compiled string template, in the form of:
``` json
{
  literals: [ 'Hello ', '! ', ' + ', ' = ', '\n' ],
  substitutions: [ 'name', 'x', 'y', 'x + y' ]
}
```
JSTransformers Tranforms' `compile` function is expected to return `function(locals)`. So, we wrap it into that return render function, taking in the locals.

> that's why I'm against this too simple testing flow that does and test nothing. @jstransformers/owners

The test is here: https://github.com/jstransformers/jstransformer-es6-template-strings/tree/master/test

If you have an issue with the way it's set up, then please feel free to create an issue, perhaps a PR so that it's tested. It's very difficult to track in a commit comments.


wrong function name, will confuse doxygen parser

Personally I am against writing `function_name function:` at all unless your comment does not relate to the function immediately succeeding it.


Can you put the assets in the assets branch. Just to keep this directory clean


Needs a database migration to set the type column to EmsCluster for existing records.


Unfortunately it seems that ES6 `Map` isn't quite a drop-in -- I haven't looked into the details here yet. Ran this test with IOJS v1.6.5 nightly:

     + protobufjs.complexProto                          2.134 ms     10 assertions
     x protobufjs.complexJSON                           1.056 ms      0 assertions

     Unexpected end of input
        SyntaxError: Unexpected end of input
          at Object.parse (native)
          at Object.ProtoBuf.loadJsonFile (.../ProtoBuf.js/dist/ProtoBuf.js:5350:70)
          at Object.suite.complexJSON [as test] (.../ProtoBuf.js/tests/suite.js:638:40)
          at Domain.<anonymous>

     [object Object] is not iterable
        TypeError: [object Object] is not iterable
          at new Map (native)
          at Object.suite.mapContainer [as test]     (.../ProtoBuf.js/tests/suite.js:1793:17)
          at Domain.<anonymous>     (.../ProtoBuf.js/node_modules/testjs/test.js:278:37)

     x protobufjs.mapField                              1.305 ms      0 assertions

     [object Object] is not iterable
        TypeError: [object Object] is not iterable
          at new Map (native)
          at Message (.../ProtoBuf.js/dist/ProtoBuf.js:1830:58)
          at Object.suite.mapField [as test] (.../ProtoBuf.js/tests/suite.js:1902:27)
          at Domain.<anonymous>     (.../ProtoBuf.js/node_modules/testjs/test.js:278:37)

     x protobufjs.mapEncodeDecode                       0.570 ms      0 assertions

     [object Object] is not iterable
        TypeError: [object Object] is not iterable
          at new Map (native)
          at Message (.../ProtoBuf.js/dist/ProtoBuf.js:1830:58)
          at Object.suite.mapEncodeDecode [as test] (.../ProtoBuf.js/tests/suite.js:1942:27)
          at Domain.<anonymous> (.../ProtoBuf.js/node_modules/testjs/test.js:278:37)

     x protobufjs.proto3Json                            1.376 ms      0 assertions

     [object Object] is not iterable
        TypeError: [object Object] is not iterable
          at new Map (native)
          at Message (.../ProtoBuf.js/dist/ProtoBuf.js:1830:58)
          at Object.suite.proto3Json [as test] (.../ProtoBuf.js/tests/suite.js:2008:27)
          at Domain.<anonymous> (.../ProtoBuf.js/node_modules/testjs/test.js:278:37)


Maybe a more descriptive commit message (unless you intend to squash it into another commit) as to which flag is changed.


Not sure when RStudio Re-runs `devtools::document()`.  I would just run that yourself and it should update the namespace file.  All you should need to export a given function is `#' @export` which you have.  Check your `NAMESPACE` file and see if it changed.

I have a few unsolicited comments about the code inline above as well.


Can you adjust the size of the images here, they are a little large. You can just use html tags and limit the height.


Naming inconsistent with `osl_dependence_n_sprint`


More descriptive description please :)


Hm? Maybe a comment as to why?


This causes a compile error on -Dlegacy:
"T:/Development/Git/openfl/openfl/text/TextField.hx:1080: characters 15-34 : Unknown identifier : systemFontDirectory"


yea yea, you're right, huh.
I ran some tests here and I see what it returns. hm.


0 is correct for C++ < 11. It has no functional value to play around with C++ 11 #ifdefs because it looks a little dated.


That's bizarre. This whole file is wrapped with `package openfl.text; #if !flash #if !openfl_legacy`


Would returning the newly inserted ID make more sense? Then you could use it in the UPDATE query below instead of relying on the file path.


This conditions is spurious.  Next `while` will not iterate basing on the same condition.


why do we make this change?
I understand that the original code doesn't want to return the plain-text password back in the response.


that would work but .times is 0 indexed, so I'd have to add +1 anywhere so IMO it's not any more readable/compact


I think we should remove this one commit and figure out a good solution for a build dir where every source goes in. We also can clone only in one source folder, and when we make a git update and let say the 32 bit build make a source update we put a variable in a array. And when the 64 bit compiling starts it checks the array for the update variable. Is that a idea?


ah darn forgot about this hack. it is thread timings - need to make sure decoder thread has started, if not it eats the seek  but ignores it as a seek(0) is done on decoder side. not a proper solution.


Codestyle.


Ah, perhaps it's because I'm using:
<set name="legacy" value="true"/>
?


@alban wanna do this in your #749?


I would probably make this an S3 function (you will first have to define a generic `filter` method).  Also you can use `Inf` for the maximum length, and you should probably use 0 for the min length.  Right now you are dropping any sequence of 1 base (not that those would be useful).

```r
filter = function(x, ...) UseMethod("filter")

filter.primerTree = function(obj,


Ah right, good call.


could do 1.upto(pages) if upto is easier to read than remembering which one is .. and which one is ...


Yup, no issues if I use:
```<set name="openfl-legacy" value="true"/>```

Do note that flixel does this in include.xml:
```<set name="legacy" unless="next" />```


You don't need the paste here, message automatically concatenates all of its arguments.


Although I incorporated the LLN version in the code, I'm not sure if it is valid as a test.


This commit LGTM


This code duplicates `osl_generic_clone`.

Better fix the branch with `n < 0` so that it does not use that function (e.g. by counting the actual size of the list) and use this function to simplify `osl_generic_clone`.


is_loop_carried need a loop index because it compute if the dependence is carried by *this* particular loop index.
A loop independent dependence is not carried by any depth or loop index in particular, so I don't think it need a depth/loop index.


+1. The verboser, the better.


Good.


This fixes Python 3, but I'm not sure how it affects Python 2; if it can't be taken as-is perhaps it will at least provide guidance toward fixing the GDB server for Python 3.


`m_pDS->fv(0).get_asInt()` and you can even drop the `as NumSongs` from the query.


Ok. Thank You Jamil. I think I still need to send you the info for lynda.com
so I'll send that to you soon.

On Fri, Apr 10, 2015 at 7:32 AM, Jamil Evans <notifications@github.com>
wrote:

> Hey guys... please rename the folder from EAT-1 to "upload-form", and the
> HTML file can be index.html or upload.html.... folders and filenames should
> be descriptive and represent their content and purpose. It's good enough to
> just put EAT-1 in your commit message... that's enough linkage back to the
> Jira story.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ECLabs/Eaton/commit/d34c257826401fd100dedbeeae5b067dfc54483e#commitcomment-10664184>
> .
>


Great, this will also make it a lot easier to change reporting as needed in the future.


I agree that it should be configurable.  I'll leave that for the next guy to implement.


Ok! Done.


rather than looping through the objects four times to get the lengths (min and max here and at 18 above) store the results and use it twice.

lengths <- unlist(lapply(primerTreeObj$sequence, length)
above_max <- lengths > max
below_min <- lengths < min
message(sum(above_max), " sequences above ", max) # same for min
primerTreeObj$sequence <- primerTreeObj$[lengths < max & lengths > min]
```


Extra `(`?


Hey Skiphs, is it possible you could add support for modded versions?  Currently they cause this crash report when using them to launch AMIDST:
[info] Searching for biome color profiles.
[info] Unable to find biome color profile folder.
[info] Beginning latest version list load.
[info] Attempting to download remote version list...
[info] Scanning for profiles.
[info] Successfully loaded profile list.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1230.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1291.jar is missing or a directory.
[warning] Unable to load MinecraftVersion at path: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10 because jarFile: C:\Users\joshl\AppData\Roaming\.minecraft\versions\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10\1.7.10-LiteLoader1.7.10-1.7.10-Forge10.13.2.1340-1.7.10.jar is missing or a directory.
[info] Successfully loaded version list. URL: https://s3.amazonaws.com/Minecraft.Download/versions/versions.json
[info] Reading minecraft.jar...
[info] Jar load complete.
[info] Searching for classes...
[debug] Found: [ByteClass are] as WorldType | CCStringMatch
[debug] Found: [ByteClass bpa] as GenLayer | CCLongMatch
[debug] Found: [ByteClass boy] as IntCache | CCStringMatch
[debug] Found: [ByteClass od] as unknown |
[debug] Found: [ByteClass bsp] as WorldType | CCRequire
[debug] Found: [ByteClass bsp] as BlockInit | CCRequire
[debug] Found: [ByteClass bsp] as GenLayer | CCRequire
[debug] Found: [ByteClass bsp] as IntCache | CCRequire
[info] Class search complete.
[info] Generating version ID...
[info] Loading libraries.
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\minecraftforge\forge\1.8-11.14.1.1341\forge-1.8-11.14.1.1341.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\minecraft\launchwrapper\1.11\launchwrapper-1.11.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\ow2\asm\asm-all\5.0.3\asm-all-5.0.3.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\com\typesafe\akka\akka-actor_2.11\2.3.3\akka-actor_2.11-2.3.3.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\com\typesafe\config\1.2.1\config-1.2.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-actors-migration_2.11\1.1.0\scala-actors-migration_2.11-1.1.0.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-compiler\2.11.1\scala-compiler-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\plugins\scala-continuations-library_2.11\1.0.2\scala-continuations-library_2.11-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\plugins\scala-continuations-plugin_2.11.1\1.0.2\scala-continuations-plugin_2.11.1-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-library\2.11.1\scala-library-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-parser-combinators_2.11\1.0.1\scala-parser-combinators_2.11-1.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-reflect\2.11.1\scala-reflect-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-swing_2.11\1.0.1\scala-swing_2.11-1.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-xml_2.11\1.0.2\scala-xml_2.11-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\lzma\lzma\0.0.1\lzma-0.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\sf\jopt-simple\jopt-simple\4.5\jopt-simple-4.5.jar
[info] Loading libraries.
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\minecraftforge\forge\1.8-11.14.1.1341\forge-1.8-11.14.1.1341.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\minecraft\launchwrapper\1.11\launchwrapper-1.11.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\ow2\asm\asm-all\5.0.3\asm-all-5.0.3.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\com\typesafe\akka\akka-actor_2.11\2.3.3\akka-actor_2.11-2.3.3.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\com\typesafe\config\1.2.1\config-1.2.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-actors-migration_2.11\1.1.0\scala-actors-migration_2.11-1.1.0.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-compiler\2.11.1\scala-compiler-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\plugins\scala-continuations-library_2.11\1.0.2\scala-continuations-library_2.11-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\plugins\scala-continuations-plugin_2.11.1\1.0.2\scala-continuations-plugin_2.11.1-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-library\2.11.1\scala-library-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-parser-combinators_2.11\1.0.1\scala-parser-combinators_2.11-1.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-reflect\2.11.1\scala-reflect-2.11.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-swing_2.11\1.0.1\scala-swing_2.11-1.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\org\scala-lang\scala-xml_2.11\1.0.2\scala-xml_2.11-1.0.2.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\lzma\lzma\0.0.1\lzma-0.0.1.jar
[info] Found library: C:\Users\joshl\AppData\Roaming\.minecraft\libraries\net\sf\jopt-simple\jopt-simple\4.5\jopt-simple-4.5.jar
[crash] Unable to find critical external class while loading.
Please ensure you have the correct Minecraft libraries installed.
[crash] java.lang.NoClassDefFoundError: com/google/common/util/concurrent/ListenableFuture
	at java.lang.Class.getDeclaredFields0(Native Method)
	at java.lang.Class.privateGetDeclaredFields(Unknown Source)
	at java.lang.Class.getDeclaredFields(Unknown Source)
	at amidst.minecraft.Minecraft.<init>(Minecraft.java:200)
	at amidst.gui.version.ProfileVersionComponent$2.run(ProfileVersionComponent.java:119)
	at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.ClassNotFoundException: com.google.common.util.concurrent.ListenableFuture
	at java.net.URLClassLoader$1.run(Unknown Source)
	at java.net.URLClassLoader$1.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(Unknown Source)
	at java.lang.ClassLoader.loadClass(Unknown Source)
	at java.lang.ClassLoader.loadClass(Unknown Source)
	... 6 more

[crash] Amidst has encounted an uncaught exception on thread: Thread[Thread-7,6,main]
[crash] java.lang.NullPointerException
	at amidst.minecraft.Minecraft.<init>(Minecraft.java:206)
	at amidst.gui.version.ProfileVersionComponent$2.run(ProfileVersionComponent.java:119)
	at java.lang.Thread.run(Unknown Source)


meh. didnt quite refactor my tree properly i see..


Ah this fixes the extra space from the other commit.


The link doesn't automatically switch to 3.4 (on Firefox/Linux), nor does it with the anchor `#py34`.
Probably you should mention "Python 3" in the text ...


Why?  Looking through the code, it would seem that removing this would make the command unavailable, but I'm not super familiar with how this is used.  Check that this is intended?


Hello Yaguang,

The issue appears when I was using remote Driver-nodes, and unfortunately has very bad consequences (ie. completely ruins the test-run - the test will never complete, and will have to be aborted manually).  Even with the fix, and with the NTP-sync enabled on all my nodes (using CentOS Linux), I still see the "Negative time interval..." warning pop up occasionally in the logs (ie. not hard to reproduce, with a fast S3 server).

Because I have only seen it in a multi-Drivers environments, this is why I assumed the issue is related to collecting/calculating the time samples from the remote Driver-nodes, where I do see a milliseconds-level difference in their system time, even if they all have NTPD running.

I agree that flipping the time might not be the best solution (as it introduces one incorrect measurement sample), but we have to deal with the negative time somehow, to avoid the ArrayIndexOutOfBoundsException and the crash of the test.  I was thinking about "swallowing" the measurement (ie. not add a negative value into the counter), but I was not sure about the consequences if _all_ of the samples produce a negative time (ie. if a user really has not configured the NTP across their nodes).

Hope this helps.

Best regards,
                     Zoran


It's so great that those are finally gone!


MarkdownDocs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


I think using the wrong url here was causing the problem earlier


This url returns a newline character at the end of the URL, the others don't.  We should be consistent in what we return, not rely on the caller to sanitize the output when they don't have control over which service we use.


gonna refactor the `human_referrer` method into a one line method. since we're always only looking for one record, I like to use the `find()` methods rather than calling `.first` on an array


version -> release?


@danilaml Thanks I would mark it instead of just remove it.. Well, at least before it is confirmed by the original writer of this section.


These `xs:int` should be all of type `xs:positiveInteger` shouldn't they?


I would make this function an S3 generic as well

```r
table <- function(...) UseMethod("table")
table.default <- base::table

table.primerTree <- function(x, ...) {
  lengths <- unlist(lapply(primerTreeObj$sequence, length)
  table(as.character(lengths))
}
```


oui et je fais quoi avec je vais  où et question subsidière (dans l'état j'erre ) ;-)
désolée d'être génétiquement blonde


I've just extended Elemental to provide a C routine which returns ``sizeof(bool)`` within a C ``uint`` and modified the python interface to use this result to decide between using a C ``uchar`` or ``uint`` for representing ``bool``:
https://github.com/elemental/Elemental/commit/c0be00f48c38f5533e6fc19a3854f2142837adcb

Perhaps the same C routine would be of use here.


m(_ _)m


It makes sense in some applications to know that dependence is not carried by a particular loop, allowing to, for example, parallelize this loop.

However it is not as important as having tests and/or evidence that `is_loop_independent` is equivalent to `is_loop_carried(depth)` is false for any possible depth.

> On 10 Apr 2015, at 19:46, Kuntz Thomas <notifications@github.com> wrote:
>
> is_loop_carried need a loop index because it compute if the dependence is carried by this particular loop index.
> A loop independent dependence is not carried by any depth or loop index in particular, so I don't think it need a depth/loop index.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Haerezis/candl/commit/e3b3b6eeada8b1fb00f0809fd7a549f5eecd6bf6#commitcomment-10670780>.
>


You don't have to use `return`, just put the object as your last line

```r
primerTreeObj
```


I'll try to do it monday.


I think the arguments are mixed up. Should be:

target.triggerHandler('toggled', [**target**]);
tabs.triggerHandler('toggled', [**tab**]);


addtionnaly, could it cause #6105 ?


Oops, these names are flipped.


    E: Unable to locate package libsndfile

The package is actually called `libsndfile1`:
https://packages.debian.org/search?keywords=libsndfile1
http://packages.ubuntu.com/search?keywords=libsndfile1


Also style comments.  I use `snake_case` throughout primerTree rather than `camelCase`.  (package names have to be camelCase however)

You should try to put spaces around any operators you use, otherwise it is like tryingtoreadasentencewithoutanyspaces.  You can do it but it takes some effort.


Return targetToken.iskind...


typo (unqoute)


disons que tout l'univers est en anglais ce qui m'oblige à soit travaillé avec languee pour être sûr de ne pas me tromper de bouton
soit y aller à l'intuitif ce que je fais en testant des commentaires
je suis vraiment perdue sur le but du jeu
avec la règle du jeu


It's so blue :o


Numpy -> NumPy (this is inconsistent in the whole text)


`sequence` looks unused, is it important?


Me neither but every time I look in the code of openscop or candl I see this coding style, so I was just trying to do same for continuity.
The doxygen function name will be changed. I was an error of copy/paste going wrong :/


11/10 would like to notice again.


@JDutil it looks like there is a namespacing problem with this commit (`spree/frontend` vs `frontend/spree`) that has caused exception handling issues in the `2-3-stable` branch. I'm going to adjust accordingly.


In a few places here we forgot about "append = true" parameter:

				FileUtils.write(sqlFile, sb.toString(), true);
				writeFileRaw(sqlFile, sb.toString());

So that this caused currently failing Service Builder, https://issues.liferay.com/browse/LPS-54883


Mmm, I'm not sure that we want to solve this by adding a new notification. I think we may want the loading to not appear if there is no need for it to show up in the first place.


TeamCity Ybackend :: synteny/ybackend [Build 19](http://192.168.30.225:8111/viewLog.html?buildId=1107&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


Wouldn't this be printed very, very often? Like, everytime you open a page like the Abstract Bus Card?


You missed the ```&& position_->canStop()```


Wrong message. Adding crontab to scripts.


What am I missing, why can't this just be implemented as
```
    latestOperation.remove(operation.getKey(), operation);
```
?


Hi, you should provide resources for specific screens instead of changing the defaul value.

http://developer.android.com/guide/practices/screens_support.html


Or should I throw UnknownLocaleException here?


Well, this does'nt belong in this PR, but heyyy.


That was ultimately the time and space saver I was thinking.
That would need another variable for each git/hg lib or a file in each dir to mark that updates were made during git/hg pull and then delete that file after the last build.

Check my latest commit meanwhile. I haven't tested with the full build yet but using a simple [test.bat ](https://gist.github.com/fc39079ce87fee047c8d) the logic seems to work with what i had in mind.


TeamCity Ybackend :: biocad/ybackend [Build 184](http://192.168.30.225:8111/viewLog.html?buildId=1108&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


It was introduced by this https://github.com/Syphurith/rpcs3/commit/9e49a33b3c2027715eb7b323e909976b3d0f3137
commit by @Nekotekina


Ah well, it looks simple enough.


I kept the "if" because of its "else".


rax retry tests


"Weź się już nie ośmieszaj."
if aka kamilkime


Be more explicit please.


How come these say email but are actually validating slugs?


Oh wait, is there really `CELL_EINVAL` returned and not `CELL_ESTAT`?
BTW, according to API doc (maybe outdated?), that are these three should return ESTAT..
* `SPU_THREAD_GROUP_STATUS_NOT_INITIALIZED`
* `SPU_THREAD_GROUP_STATUS_INITIALIZED`
* `SPU_THREAD_GROUP_STATUS_WAITING`

.. All right, since i know little about how its SPU actions.


Can we have a more descriptive name for this package? SGD is really not very informative.


wtf?
To samo masz wyzej w onExecute


docs.lightxlab.com


UVA/10109 - Solving Systems of Linear Equations.cpp

To see final Matrix


We can dispense with these comments. They were useful for GSON because the private ctors appeared never to get called.  But in your JSON version, there are explicit calls to them, so it's clear why they're necessary.


lol, cool beans. I'll look them over in a bit and make changes. I got the namespace file updated. I had to delete it and rerun devtools::document().


Can we keep the logging, please?  SLF4J is very lightweight if you don't install a logging target (the default).


Also, @IainNZ has decreed that we no longer wish to register packages without tagged versions.


@danilaml I've uploaded the new version which restored the condition. BTW the `CELL_EINVAL` may be a typo. Please refer to newer source for notes.


These are for container commands, i.e. shell scripts. publicip is no longer implemented as a shell script. Notice here: https://github.com/replicatedcom/libcmd/commit/2636280c4571475bd6163aa4b52f6857b26ce928#diff-bb6c3b32e68f715378f981838fc9e91dR28


Oriana, le travail collaboratif ne se fait pas sur github.com mais sur http://labs.iri-research.org/catedit/
Je vous invite à reprendre le tutorial sur https://www.france-universite-numerique-mooc.fr/courses/univnantes/31003/Trimestre_1_2015/discussion/forum/i4x-univnantes-31003-course-Trimestre_1_2015/threads/552667359b101dd7da0002b7


This looks good to me. I'll leave it to @cooperq to merge.


these params should not be part of the state of each Workflow instance; this is state that should be managed by the WorkflowManager (hence, no initialization of email, pw attributes for each workflow)


Fixed in db79365


Looks good. My only concern is the removal of the logging.


Do we need to update FrameworkBundle's PreviewExceptionController as well?


TeamCity Ybackend :: synteny/ybackend [Build 19](http://192.168.30.225:8111/viewLog.html?buildId=1107&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 48 Build time: 00:10:44
### Failed tests
```
ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest.AnnotatorRunner should work with aminoacid files: class java.util.NoSuchElementException: None.get
java.util.NoSuchElementException: None.get
      at scala.None$.get(Option.scala:344)
      at scala.None$.get(Option.scala:342)

```


TeamCity Ybackend :: biocad/ybackend [Build 185](http://192.168.30.225:8111/viewLog.html?buildId=1109&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


@rstacruz thanks for the documentation improvements, they're released in `yargs@3.7.1`.


attempt_more_exception_reduction (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (335 of 335 tests passed)

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


TeamCity Ybackend :: biocad/ybackend [Build 184](http://192.168.30.225:8111/viewLog.html?buildId=1108&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 48 Build time: 00:10:55
### Failed tests
```
ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest.AnnotatorRunner should work with aminoacid files: class java.util.NoSuchElementException: None.get
java.util.NoSuchElementException: None.get
      at scala.None$.get(Option.scala:344)
      at scala.None$.get(Option.scala:342)

```


Updated in 559da32b30bb243bd87


Thanks for the tip!


I can't remember why we added the redirect on failure, I think it should be fine that they are just redirect to the error page. If you tested failed updates/deletes through the UI and it behaves as expected, then I think this is fine.


All of these dont invoke subcommands.
`ping -a` is a flag, `bcast vm` is used to specify a target (read: stop accidental broadcasts) and `math` requires an expression to parse. All of these have nothing to do with subcommands.


Currently breaks unlike 623a3d9fdbcf4c7688cf456c2c12c55ae23a7fcd, with

```TypeError: Cannot call method 'createSearchIndex' of undefined```

i.e. can't find the collections.


I don't get why we don't want registered packages without versions.


I'm not sure I'm following you.
Right now is_loop_independent add constraint for every shared iterator dimension, so the maximal shared depth.
Because if a dependence is loop independent, it is so for _all_ depth.

Maybe you want that is_loop_independent check *up to* a certain depth, allowing to check if a dependence is loop-independent for the first N shared iterator dimensions, but I found that a little weird.


I certainly prefer it, but my boss says it's too complicated and wants
it to be foolproof. You know, requirements. Maybe I should keep it as a
separate branch.

Thanks for the advice.

On 04/10/2015 11:14 AM, Areski Belaid wrote:
>
> @LICEBMI <https://github.com/LICEBMI> contact table have a json field
> in order to offer flexibility on the contact fields, I would strongly
> recommend you use that. This will keep the system more flexible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LICEBMI/newfies-dialer/commit/1b7e7e88866a8f5ddb9b496f5987fccd24869ff7#commitcomment-10669355>.
>


I wonder why it got stuck in the first place.
If it ran for a new day, it would send the kModelRefresh notification.
That would hit the refreshPageView selector, and the last step of refreshPageView selector would call hideLoading...


Also cleaned up PointCalc!


attack


sorry but i don't understand your question
what seems to be the problem?


@JDutil After 794870a, the first spec at https://github.com/spree-contrib/spree_tax_cloud/blob/2-3-stable/spec/features/checkout_spec.rb#L41-L56 was failing, due to the exception handling in https://github.com/spree-contrib/spree_tax_cloud/blob/2-3-stable/app/controllers/spree/frontend/checkout_controller_decorator.rb#L8-L11 not taking place. Reverting the file move turned this spec green again in my `2-3-stable` branch.

Do you have any idea what's going on here? I'm happy to put this decorator into the `lib/` path, but can't figure out how to get the exception handling (and hence this spec) to work. Have fiddled around with a couple different paths and can't figure it out. Suggestions welcome!

No idea if this affects other branches as well, but will look into that in the coming days.


superfluous variable


how to correct the typo ?


`match_tags` is the actual setting name here.


:beers: doesn't matter anyways though; got :-1:


[Meaningful  variable names](https://github.com/TheJokr/vmbot/commit/6066efa02ca0969c6dbccb346e93241ab7bf85db#commitcomment-9789210)


*expressed


This must go somewhere else.


rather uncommon


I don't want to assume it's in the PATH because I want people to be able to pass dummy names for generators for the purpose of setting the order of custom scripts.

Maybe path manipulation should be possible with a config variable so people can explicitly set the search path of generators.


Indeed. e5ef8e3


Ouch.


Great Work!


Most of the mapping notify are pointless (I noticed that particular keys on my keyboard would systematically generate a mapping notify events after being pressed) and will break any ongoing chord chain.

I don't know of any method that would allow me to differentiate meaningful mapping notify events from pointless ones.

It seems that, when a non-QWERTY keyboard is used, the first mapping notify events is useful.


lol


This is great.  I was doing something like this to make docker-machine work with RHEL on ec2 (ami-4dbf9e7d).

```shell
## ... prior to this, docker-machine create fails ... cannot connect to ssh...
MACHINE_IP=`docker-machine ip $MACHINE_NAME`

# Add support for installing RHEL7.1 with docker-machine
# change the docker-machine user from 'ubuntu' to 'ec2user'
CONFIG_DIR=$(docker-machine inspect $MACHINE_NAME | egrep StorePath | head -n 1 | awk -F '"' '{print $4}')
CONFIG_FILE=$CONFIG_DIR/config.json
# replace the user that gets used in ssh commands
sed -i 's/SSHUser":"ubuntu"/SSHUser":"ec2-user"/g' $CONFIG_FILE

# ssh to the machine (skip hostkey check) ...
#  ... and change the sudoers setup to not require a tty when running 'sudo' commands
ssh -t -t -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i $CONFIG_DIR/id_rsa ec2-user@$MACHINE_IP "sudo sed --in-place 's/Defaults    requiretty/\#Defaults    requiretty/g' /etc/sudoers"
#  ... continue using docker-machine
```

What about changing the username for all available drivers .. or at least the ones that would support such a thing?


TeamCity Ybackend :: biocad/ybackend [Build 185](http://192.168.30.225:8111/viewLog.html?buildId=1109&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 48 Build time: 00:10:23
### Failed tests
```
ru.biocad.ybackend.api.businesslogic.toolrun.AnnotatorRunnerTest.AnnotatorRunner should work with aminoacid files: class java.util.NoSuchElementException: None.get
java.util.NoSuchElementException: None.get
      at scala.None$.get(Option.scala:344)
      at scala.None$.get(Option.scala:342)

```


Oh, yes, sorry


Having worked extensively with nagios-style monitoring systems I'm familiar with what they expect. Frankly I don't think anyone should use crm_mon for that purpose (there are better special-purpose plugins for that) but this does make it more usable if they choose to. In particular, previously if the cluster wasn't running at all, the output would not be monitoring-compatible, but now it will print a monitoring-friendly warning status. As an aside I'll note that we only print OK or warning status, never critical, but I don't think that's a big deal until there is demand for more.


LGTM


fixed


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


The end of the file should have a new line.


```CFile::Exists(url)``` is checked at the beginning of CotainsFiles


nice I did not know this library existed very tight!


json data is reliable, plus give plenty of flexibility.
Additionally this is turning into a great feature of PostgreSQL http://www.postgresql.org/docs/9.4/static/datatype-json.html


what is a precache?


Just a bit of cleanup that helped me understand what the code was doing.


lol who uses OnGUI anymore


we want to enforce this for all the validators that this extension provides OOTB. But it's not something that in my opinion should be imposed to every validator that a third party can provide


Regarding "unknown" -> "none", that case was printed as "none" in the three other places it occurred, so just making it consistent.


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1129 of 1129 tests passed)

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


Diese Übersetzung sagt irgendwie gar nichts aus.
Das Wort "Rendern" gibt es in deutsch nicht.
Vielleicht sollte man eher  "Asynchrone Wiedergabe zu den Aktualisierungen"(Aber irgendwie ergibt das auch wenig Sinn.)


What about `return "Requires one of show, index, edit, chown, log, delete{} or revert as an argument".format(random.choice(["",","]))`? :joy:


OK I went overboard with refactoring but it makes the code easier to follow and sets the stage for streamlining the command-line options in the future. I imagine something like --include/-I "headers,operations,bans".


This closes #108


Why are you changing case on me?  And names in general?  iOS style is camelCase and descriptive.


Just one thing that needs to change

Shib will fail if people try to login from rep.lib, so if you could make it the fully qualified url https://repository.library.northeastern.edu/users/auth/shibboleth

That'll stop any errors people might encounter from our old url style


Why bother?  I hate aligning like this.


Thanks for breaking these into a specifically deprecated section. Makes it easier to reason when skimming through.


Why?


Will do


nice looks good


Neither do I, rules in METADATA.jl are becoming annoying rather than
helpful.

Packages without tagged version can be thought as rolling release.


I'd feel more comfortable with this change if we knew exactly why the move to local resolved the issue with the celery workers' startup process.  But, with @BenjiLee giving his approval based on his knowledge of the module in question and his assessment of the risk of impact to the rest of the system, I'll give a thumbs-up as well -- :+1:


I don't want to access them directly, but by initializing them as `self.name` they become variables of the object anyways. I'm fine with using `self.getAccessToken()` for lookups.


Overwriting .meteor in a new app which uses this package, with .meteor contents from 623a3d9fdbcf4c7688cf456c2c12c55ae23a7fcd makes it work.


Thanks!


I don't support this.  We've discussed it multiple times; I think it makes it too easy to be lazy.  It's basically impossible to distinguish `NSError` instances, and I think you pretty much always want to know the type of the error you're getting back.


e


Is it just me or is this indentation all kinds of crazy?


Why are you changing the name of the service here? This is a breaking api change that will require a major version bump.


Approved for pull.


\`udev\`


Fixed in master.


Thank you for the explanation.


codes looking tight, niiiiice


This refactoring makes it much easier to deal with the multiple output formats in the later commits that implement more HTML/XML output. It also sets the stage for more CLI option streamlining; I'm thinking e.g. --as/-a (cgi|html|xml|monitor|plain|console|none) --output $FILENAME.


I'm fine with using `self.getAccessToken()` for lookups.
In my opinion this should be moved to `vmbot.py` anyways, because (Authenticated-)CREST could be used for other lookups as well.


@V3n3RiX: no, I'm not suggesting there is any such rule or principle.  I'm saying it would have been more polite to do so, and I would have been a lot more inclined to help, even pushing forward some additional changes to help your transition.  Rules become necessary when politeness has failed, but their absence does not preclude politeness.

As it is, the attitude here is tending toward the "screw you, we're perfectly within our rights" approach rather than the cooperative notion of "yeah, we probably could have checked to see what you're doing and how we can cooperate."  Yes, you could have taken my work with even less (zero) credit given and you'd be perfectly right, but don't be a jerk about it.  The OSS world is built mostly on cooperation, so don't squander the default goodwill for expediency's sake.

Be considerate.  I worked a lot of personal hours two weeks ago to put together and actually test (!!?!) a bunch of bumps, go on a week's disconnected vacation, and come back to find this grumpy mess.


SICK. It was all Anthony. But I'll take the credit.

And the green.

Managing Editor
@dosomething <https://twitter.com/dosomething> @bkassoy
<https://twitter.com/bkassoy>

On Fri, Apr 10, 2015 at 2:29 PM, Joe Kent <notifications@github.com> wrote:

> nice looks good
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DoSomething/DS-Realtime-Feed/commit/077f0fafd66c5574fee5445a2d6843a7c7567845#commitcomment-10671623>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.187](http://teamcity.monogame.net/viewLog.html?buildId=6113&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 481, ignored: 6 Build time: 00:14:32


Finalizando projeto 10 e alterando login de commit.


My understanding is that between `window.caches` and `window.fetch`, it's possible to use the Cache API for some useful use-cases independent of Service Worker. Is that the case, or am I misunderstanding the scope of it being exposed globally? :)

If they can be used in a useful manner outside of SW, I think it may be worth adding in both this example and a simpler one (maybe it just extracts or moves portions of the current example) to a dedicated `cache-api` sample.

Will wait for confirmation before providing more feedback. Again, I could be totally wrong here.


Big push for consistency between output formats (in terms of what is displayed rather than how). Rather than have separate functions for each format, which repeat the logic and just change the output, I'm using a function per displayed item, so the logic only appears once, and the output is handled by "switch (output_format)".


I'm always super skeptical of commits that are named like this...


nit: Consider inserting newline after #include <glib.h>.


Please format the code in the class to add spaces as appropriate (e.g. after "for", "if", ...)


I've changed osl_dependence_n_sprint instead.


Can `putIfAbsent` help here instead of that `synchronized`?
```
SourceEventBucket bucket = new SourceEventBucket(source, timestamp, bucketDuration);
SourceEventBucket existingBucket = eventsBySource.putIfAbsent(source, bucket);
if (existingBucket != null) {
    bucket = existingBucket;
}
```


Should this file have an h1 title similar to [CONTRIBUTING.md](https://github.com/anselmbradford/open-source-project-template/blob/master/CONTRIBUTING.md)? Maybe "Installation instructions" or "How to install this software".


Sorry, what's this?


Bug? `return newBucket` ?


\o/


don't need .row .panel or .panel-default here


I think we should edit this section:
In the case of siblings, write once buckets use an algorithm for resolving conflicts that has the appearance of random behavior from the user's point of view, but is in fact determinisitic. Effectively, Riak will choose the "least" entry among siblings, where the ordering relation is based on a deterministic SHA-1 hash of the object. This has the appearance to the user of "random write wins", even though the behavior is repeatable and determinisitic (assuming the same sibling inputs).

to

In the case of siblings, Riak will choose the "least" entry among siblings, where the ordering relation is based on a deterministic SHA-1 hash of the object.

I am not sure we should talk about appearing random.


Замените на вызов `::operator new` или переопределяйте `operator delete`. Ни в коем случае нельзя выделять `malloc` и освобождать `operator delete` либо наоборот


Поправьте везде - операторы обрамляются пробелами


+1 looks good.


Hey @sc0ttkclark I don't think I ever got an official :+1: from you on this. Does it meet all the requirements you asked for in #711?


This commit includes other changes.


Missing the standard header.


What is the point of registering a package without a tag? You can't `Pkg.add` it, so you have to `Pkg.clone` it. It then updates whenver you do Pkg.update, just like I'd `git clone`d it myself. So what is the utility of putting it in METADATA? If you can answer that, then I'll drop my objections. I'm still not putting it on pkg.julialang.org either way unless you twist my arm, because installing a 3rd party package that can break at any time due to a simple Pkg.update is the definition of madness and I don't want to encourage that.


Hello World


Yes, but led me down to #712, but this on it's own accomplishes what I was after.


d


While we're churning names, something should give here.  Why is `.clients` not `.clientsAndTabs`?


ew, pass the `IApplicationEnvironment`


I think it would be better (since you've used "fish" operator) also use `fmap`

    decode . BS.pack <$> (getResponseBody <=< simpleHTTP  $ getRequest request)


OK thank you!


I vote for calling this "hasMemberWithProperty" because I think it's sufficiently clear, and shorter.

I would probably make the first argument a List<Object> and use PropertyUtils, rather than assuming it's a Map.


Also, the rules are there because this is a curated collection with a single namespace. As long as its a single namespace everyone has to play in together, there needs to be namign rules. I'm all for someone creating an alternative METADATA-like thing with per-user namespaces or whatever people want - go right ahead. I think the current consensus-derived rules and discussion are a pretty good system but its not for everyone.


you can keep this code just do:

```
QAmqpChannel *channel = qobject_cast<QAmqpChannel*>(object);
if (!channel) return;
// work with channel
```


`clientsAndTabs`?


I agree this is ugly, but I suppose we can live with it. (Is it not possible to execute some JS against the engine to define a function? Or is it just not obvious how to do that?)


Figured out the save problem, you have to type the .pdf after the filename


注意：本次Commit中虽然解决了业务逻辑层表格懒惰修改的问题，但是没有解决访问信号量的处理问题，将在下个commit中解决。


The index of the stop is the actual index in the list of stops, not the number that's displayed to the user. This should just be setIndex(i)


Is it possible to reference `Class.asPair` directly as a higher order function?


I'm familiar with *how* to version, that's not what I'm talking about.


I think it's more Swift-y to say `keyBundleFromPair()`.


Cool thank you!


That is a good point Ian, I regret what I've said about versions.

Pkg.clone() with the full URL isn't that bad.


That's not my point.  The point is there's this unwritten rule (maybe it is written somewhere other than on this pull request?) that's not even adhered to.

How about we have a document in the mono repo with these unwritten rules and I'm happy to fix all the places where we violate them?


Should be index + 1


Same here


Rewrite this to 'when the route exists'


Thanks.  Done.

On 2015-04-10 2:46 PM, Chris Bethune wrote:
>
> Missing the standard header.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/unchartedsoftware/aperture-tiles/commit/61ba78f4d8d5d2b917ded5cc67ee45cc80d760c5#commitcomment-10671872>.
>


I batted around the idea of doing it that way, actually. However I
decided to split it a little because I have a habit of going too crazy
with monad/applicative combinators, and then my code starts to looks
like noise. :-)

On 2015-04-10 14:48, dima-starosud wrote:
> I think it would be better (since you've used "fish" operator) also
> use fmap
>
> decode . BS.pack <$> (getResponseBody <=< simpleHTTP $ getRequest
> request)
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/bstamour/weather/commit/f6acacbaed852bdfa67c3783d0ac61428d612e2b#commitcomment-10671894


How about .build(input, url)?


Can we use a Hash now (people more likely to know about hashes!).


I actually don't work for/with spike team, I just threw my 2 cents
about your rant. If you think that makes me a jerk, thats fine, no
heart feelings. You do know I'm right, and at some point you're right
also.

On 4/10/15, dacook <notifications@github.com> wrote:
> @V3n3RiX: no, I'm not suggesting there is any such rule or principle.  I'm
> saying it would have been more polite to do so, and I would have been a lot
> more inclined to help, even pushing forward some additional changes to help
> your transition.  Rules become necessary when politeness has failed, but
> their absence does not preclude politeness.
>
> As it is, the attitude here is tending toward the "screw you, we're
> perfectly within our rights" approach rather than the cooperative notion of
> "yeah, we probably could have checked to see what you're doing and how we
> can cooperate."  Yes, you could have taken my work with even less (zero)
> credit given and you'd be perfectly right, but don't be a jerk about it.
> The OSS world is built mostly on cooperation, so don't squander the default
> goodwill for expediency's sake.
>
> Be considerate.  I worked a lot of personal hours two weeks ago to put
> together and actually test (!!?!) a bunch of bumps, go on a week's
> disconnected vacation, and come back to find this grumpy mess.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Sabayon-Labs/spike-community-overlay/commit/65e72a49ab531cf3ddae98af94a25c00c51e23c2#commitcomment-10671721


Really don't see reason to get deal with `@Scheduled`:
````java
@InboundChannelAdapter(value = "split",
				poller = @Poller(fixedDelay = "5000", maxMessagesPerPoll = "1"))
public List<SourceEventBucket> releaseBuckets() {
         long now = new Date().getTime();
         return eventsBySource.values()
                          .stream()
                          .filter(bucket -> now - bucket.getLastEventTime() >= bucket.getDuration())
                          .map(bucket -> eventsBySource.remove(bucket.getSource()))
                          .collect(Collectors.toList());
}

@Spliter(inputChannel="split", outputChannel="output")
public List<SourceEventBucket> split(List<SourceEventBucket> payload) {
    return payload;
}
````


Right, we don't have a simpler way to specify a default `splitter`, like we can do that in DSL using just `.split()`


For some reason, after this change Terraform stop reading my AWS credentials from `terraform.tfvars`


zeroAddInt is fixed.  easierRoadmapProg1 is quite hard!  [see messages to ppaml-l]


nice


The question is whether to try to fix issues in titles in the code, vs. a file or data structure allowing arbitrary tweaks, but requiring more maintenance.
I don't regenerate my Rpackages.bib very often, but when I do, it takes a lot of work to make it actually work with BibTex.  Anything to reduce the number of problems is appreciated.


How about 'You can then use [...] to stub 'http://localhost/backdrop/example' with the following JSON:? It's the stubbing that's important here. (Or mocking, or whatever.)


Maybe we should `let response = response!` or include the response type as `Response!` to get rid of all the bangs.


Thanks @c4milo – taking a look


Not wanting to cause any merge conflicts, so not committing changes, but https://calamares.io/  doesn't seem to work, http://calamares.io/ does
Also the Copyright 2014 needs adjusting > Copyright 2014-2015


@kjlai please review


We frequently make small modifications to our jobs and resubmit them when prototyping them. With the cache turned on, we have to rename the class file or else the old version is picked up by the cache, whereas disabling the cache allows us to easily run the new version by creating a new context.

Although thinking about it, seems the cache takes into account the uploadTime, so maybe I was  mistaken about how the caching works or there is a bug in here somewhere..


As you understood there won't be a reason to have that `setOutputChannel`.
There is just enough to mark this class with `@Component` and add `@ComponentScan` to your `AggregatorConfiguration` and remove the `@Bean` for this class.

`eventAggregator().getEventsBySource()` can be replaced with:
````java
@Value("#{eventAggregator.eventsBySource}")
private Map<String, SourceEventBucket> eventsBySource;
````
in this class.


This is major!


Why bother: "Boyscouting". Leave things cleaner than you found them.

I find it much easier to read.


lk


http://pastebin.com/MmwCKyHS - BedWars failed to load


Something similar for this `@Bean`. No reason in it.
Just mark your `EventAggregator` with `@Component`
and move this `@ServiceActivator` to the `aggregate` method


Hmm, I kind of want `dist/` in git as that's the downloadable browser bundles.
Also, `try-dist` should be `docs/try-dist`? :D


Since we are moving to the `@InboundChannelAdapter`, there is no reason in this `@EnableScheduling` stuff at all.


I find it much harder to maintain :(


You'd want some indentation there:

``` ruby
unless something_is_true
  lets_do_it
end
```


Do we have a straightforward way to automatically infer shiny for R Markdown docs with runtime: shiny ?


Do you know about http://www.webjars.org/bower i.e. "org.webjars.bower" % "swagger-ui" % "2.0.24"?


I'm glad you figured git out :raised_hands: