Ok, i commented on the last commit.
It's nice to have  really "short" bullet point, like this one, but I wonder if its worth having it in more details to really strongly make that point, because I believe it is our **"MAIN FEATURE"** to put a focus on extreme collaboration :-)

#### So:
`We encourage individuals to collaborate on projects` + `We encourage them to invite others to join their projects`
The sentence in `line 18` covers both directions of collaboration, but it's easy to miss the "co-" while reading, especially because people are probably not used to collaborate intensively.

What do you think?


ack :)


I did consider this, but checked that no builtin types defined `__hash__` and so thought that it would be ok.  But I agree that if a native type defines `__hash__` then this block of code should be executed.  So I'd say this check (this line) should be reverted.


I think that should be an exception, because whenever we actually reach the point that we can call the compileTree function the result will be set. (Same goes for all the cases). Would you prefer that?


Gilles,

just curios, the error that I get right now on master with ROMIO, is it
related to this?

  Component file data does not match filename:
/home/gabriel/OpenMPI/lib64/openmpi/mca_io_romio314 (io / romio314) !=
io romio -- ignored

Thanks
Edgar

On 5/7/2015 8:36 AM, Gilles Gouaillardet wrote:
> @jsquyres <https://github.com/jsquyres> following a previous discussion
> with @hppritcha <https://github.com/hppritcha> I had 3 goals :
> 1. upgrade to romio314
> 2. allow several romio to coexist
> 3. provide an easy way to refresh romio when needed
>
> this commit, which does resolve some real issue, makes the third step
> more painful that I had expected.
>
> right now, I am thinking of "reverting" most of this commit and
> (heavily) use macros to :
>
>   * have symbols called romio314
>   * minimize the efforts needed to refresh to romio315/romio320
>
> any thoughts ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-mpi/ompi/commit/db257cdbc007b0c805dffe3157809f803c9342d3#commitcomment-11087066>.
>

--
Edgar Gabriel
Associate Professor
Parallel Software Technologies Lab      http://pstl.cs.uh.edu
Department of Computer Science          University of Houston
Philip G. Hoffman Hall, Room 524        Houston, TX-77204, USA
Tel: +1 (713) 743-3857                  Fax: +1 (713) 743-3335
--


It sounds good


ki


How do you want to proceed? I could do a part of it tonight, but I'm not sure working both at the same time would work well... Or I can start right now and you tell me when I have time to take the mutex token...


I suppose I could get rid of them.  I just have them there because I like to be able to see what my class fields are.


@gidili For what you need it was working fine, it was Neuron that used a different approach.


Aca podes hacer directo `@posts = article.posts`, en esa linea se te setea @article


There was a version in the beginning, which stated in more detail, that we invite others to give feedback and/or debate our program, which is "for free"...
Do we still have that point somewhere else? (Because now it doesnt include that its transparent and feels like it should be listed under "Collaboration" instead of "Transparency"


It's better to name file the same as class, so file should be connection.rb.

Also when you call it mixin, it should be module, not class.


reack


I like this last bullet point `line 29`, but wonder if that fits more under `collaboration` then under `open organisation / transparency`


podes hacer directo post.update


У почетку је била само *страна* онако како је и сада, не сећам се да ли сам ја тако именовао или сам преузео. Међутим, у међувремену је додавано више пута и за другу употребу и ни мени некад није било јасно на шта се мисли док не погледам код па сам зато раздвојио да не би било забуне као у енглеском.


There is an error, the added lines should be before you check if you can use save link, on line 132. This way will cause in some browsers finish execution before prepending the BOM


Yes.
By default you can omit "management_network" for slave nodes because in such cases manament_network gets from config that used for create management node. Look to CFY-2153.


När jag rebasar på den här commiten så rullar det inte på min maskin längre.
Har du någon ide vad det kan bero på?


Would adding a `boolean` class with `._is_eqn` set to True be okay? That way, `simplify()` could avoid reducing equations to an expression.


why don't you use close.png from mobileweb/theme/images?


I think this is 1.8+


You're welcome ;)


ok, no problem.


Refactored Tabs LESS into an easier format - Top is default in large view, left is default view in other views - can be changed through json.


So, if you don't always want to autoresume if no one wants to resume, you can solve the problem slightly differently:
- instead of a boolean, have a 3-state variable (enum).
- Where you call clear() now, just set a neutral state.
- Where you call want(true) now, set a  request for resume (but only if state is not, DoNotResume)
- Where you call want(false) now, set the state to DoNotResume

I just want to avoid what seems a fairly useless list at the moment, since I don't think we actually need to store all those booleans.


<body><body>


defaults to true really?


I've dealt with the new `ORKResultPredicate` feedback.


mrtrix_version support as in issue #111. Not sure if you want to write the version number via write_mrtrix.m though.


@Azure7111 you don't need so many back-ticks :)


I would add the comment above the statement which should make it easier to read and should be fine with hound


I think this should be a cache.Store now.  Since we are not indexing by namespace, since the clusterPolicy has no namespace, it makes no sense to kill the internal index where everything is hashed to same no namespace key.


The v2 registry doesn't include any search functionality at the moment, so if you want to search for image repositories under a given namespace, this allows you to specify the search index URL to use for that namespace


same here, this should be a cache.Store


Questo lo hai già seguito ?
INSTALLATION NOTES:

In questa seconda versione sono state utilizzate le librerie XLForm per la gestione dei form di input.

Per poter soddisfare le dipendenze della libreria XLForm occorre inanzitutto installare  CocoaPods :

$ sudo gem install cocoapods

successivamente è necessario risolvere le dipendenze spostandosi nella cartella root del progetto e da terminale lanciare:

pod install -V

una volta scaricate le dipendenze è sufficiente aprire il progetto non più tramite file

Rate It!.xcodeproj

Ma

Rate It!.xcworkspace


I am confused why somewhere you have it under module Undercloud and somewhere not. It's usually clean to follow the structure of files. You can choose from which level to do that, but then it's better to be consistent.


I copy-pasted and didn't figure out what to replace them with.


So wait, I'm not sure what the issue is now. When can result be null?

In general we should use exception/command errors if the error is the user's fault and they can do something about it. When possible prefer command errors over exceptions.

If an error is a result of some condition that the user can't control, then this is a programming bug and we need to capture it with an assertion.

So which one is it in this case?


A copy of mapper is already provided by using mapper.use()


Attention aux message de commit. Ici cela ressemble à ce que j'ai fait avant uniquement en faa4d959bdd5c75ee9bc873d7880d0135633b559


Shouldn't it be
```
`csc`/`cse`
```
in this line? Or am I mistaken here?


do we enforce there is only one clusterPolicy ?


have no juice left to rewrite it in more functional style, will try to revisit later.


I hoped for markdown as at least some emojis seems to be supported.


The macro way could be very interesting but its advantage is also its flaw. You have to use literal values for the path (at least in this implementation) and so you cannot use it with variables in place of path.


Ugly as shit - but it works


same here, do we enforce there is only one clusterPolicyBinding?


I tested it now, and it seems like Streamer_GetRadiusMultiplier returns weird huge values:

>CMD:gr( playerid, params[ ] )
{
	new Float:lfRadius;
    Streamer_GetRadiusMultiplier( STREAMER_TYPE_OBJECT, lfRadius, playerid );
	format( gsString, 128, "Your radius: %f", lfRadius );
	SendClientMessage( playerid, -1, gsString );
	return 1;
}

>CMD:sr( playerid, params[ ] )
{
	new Float:lfRadius;
	if( sscanf( params, "f", lfRadius ) )
	    return SendClientMessage( playerid, -1, "Float !" );

>	Streamer_SetRadiusMultiplier( STREAMER_TYPE_OBJECT, lfRadius, playerid );
	format( gsString, 128, "Your set radius: %f", lfRadius );
	SendClientMessage( playerid, -1, gsString );
	return 1;
}`

When I firstly enter on the server, /gr shows radius as 1.000000, everything is ok. But when I use /sr with any number above 0.000000 then /gr shows a insanely huge number. When I set the radius to 0 everything is ok (objects dissapear and /gr shows 0.000000). If I set it back to 1 it shows a big number. I think that the radius isn't being set correctly.

Incognito, you tested this feature ? It looks untested.


node.js


Is this really a recommended way to iterate strings for now? urgh ((.


It can only be null when either source is null or manager is null. I think both would mean an Error in the code?


@cnbeining
HTML5 API 换域名了。你那边也更新一下。


done


> same here, do we enforce there is only one clusterPolicyBinding?

Definitely no to this.


Crappy crap - there must be some more sugar for such things, i think i just missed it.


I had missed the last part of its description. Thanks for the clarification.


> do we enforce there is only one clusterPolicy ?

For now?  I can certainly see a case for allowing more than one and permissioning them differently.  The rest of the code handles multiples.


l'image est à remplacer, pas d'angular ici ;)


you don't delete the thread object. if you only want to start one thread, it is better to do that explicitly


Tout pareil, balises angular en trop je crois ;)


This is only for U8800, which has Qualcomm WLAN+BT+FM chip. U8800Pro and U8860 have Broadcom WLAN+BT+FM, so unfortunately this won't help. The reason why Broadcom is hard to implement is its communications type, which goes through Bluetooth. Qualcomm FM sits on I2C bus, meaning kernel can communicate with it without much effort. However Broadcom BT solution needs BT to be turned on, firmware has to be sent to the chip and then it can communicate with the FM chip. However, the bluedroid stack we use now has some support for sending the vendor commands to BT, however I never managed to get a proper response, so I left it as it was.


Forgot this one.


You can actually chain processors on assets by appending more file suffixes. I'm using that to get the search api path in this file: https://github.com/concord-consortium/rigse/blob/wip-react-itsi-bin/app/assets/javascripts/components/search/result_group.js.jsx.coffee.erb

You just have to wrap your head around the processing order and make sure the syntax will be ok for the next step in the chain. And debugging a compilation failure is a nightmare, because afaik, the intermediate steps aren't easily findable after it fails.

Oh, and you have to remember that there's no request environment, so you can't get sane values using _url helpers (you have to use _path), or be able to translate anything to a language other than the default, etc.


Well, you could never opt out before and now you can.

-Samuel E. Giddins

On May 7, 2015, at 5:14 AM, Kyle Fuller <notifications@github.com> wrote:

Enhancements, wasn't this just a bug introduced in 0.37?

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


@nirvdrum Fixed here: 440a90ffd5d7ea9019a60c0f71b5c966b4c4e7bd


that does not look right. I would expect OGM_ON to be set here.


Source will never be null, feel free to write a Q_ASSERT in the beginning of the function to check that.
As for manager, you should probably also write a Q_ASSERT that checks for that. In addition, to assure that this assert is never false, in the canInterpret method of Command, you can that condition.
finally assert that result is not null.


May be "Entity #%s does not found." ?
sprintf("Entity #%s does not found.", $id)


not sure what are all those connections, but at least the hooks are repeating, so putting them to base class will be good. E.g. class BaseConnection, or just the connection, then e.g. class MockConnection < Connection.

You can also use class as namespace, no only module. But it's always better to require the class first,

class Connection
   class MockConnection < Connection

then addressed as Connection::MockConnection, where mock_connection.rb will be in the connection dir


yes this is true. i think at the two possibilities. if is passed a constant string (without variable but as a symbol) the path can be processed at compile time, otherwise at runtime.


With the introduction of the `quiet` option, this seems pretty pointless. I don't know why that was added and have little interest investigating. The changes were pretty trivial anyway.


Eu exclui as do cadastro de usuário, apenas.

--

Atenciosamente,

*Prof. Dr. Rodrigo Noll*
Coordenadoria de Desenvolvimento Institucional - Câmpus Canoas - IFRS
Professor de Informática - Engenharia de Software e Programação
Instituto Federal de Educação, Ciência e Tecnologia do Rio Grande do Sul -
IFRS - Câmpus Canoas
Rua Dra. Maria Zélia Carneiro de Figueiredo, 870-A
Bairro Igara III - CEP 92412-240 - Canoas/RS
Fone: (51) 3415.8200– Ramal: 220
E-mail: rodrigo.noll@canoas.ifrs.edu.br
Website: http://www.canoas.ifrs.edu.br

Em 7 de maio de 2015 01:02, juarezmonteiro <notifications@github.com>
escreveu:

> Por que excluiu minhas linhas de inserts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rodrigonoll/tri-health/commit/a6660fc54333733359fe6c1254fa8aa107fbc531#commitcomment-11079582>
> .
>


### Quick Reach is accessibility folder which stays on top of all apps and home screens.you can add your favorite apps here.


Fixed notices and changed input validation routine... in contact.php


":100:" - @almartin82


not needed return


good, i was just about to push this change :)


У реду, преправићу кад окачиш у Глотпрес


If you don't want to set yourself, maybe "Buildhck Team". I have no idea for email though, if nothing else it would probably fall to mine mailroxas <at> gmail <dot> com.


Well I admit the current "doc" is poor ;) Hopefully the (future) sphynx doc will clarify these kind of usage.
I will try to push several of the current unit tests and associated samples that also provide more examples.


Yes, not sure about that either... I probably wouldn't write the version number back out, it would make the image look like it was generated using MRtrix. In the actual C++ version, the version number is always updated in the output, regardless of what it was in the input. I think we could probably replace that entry with 'matlab' or something like that...?


I don't think I want to do this a first class citizen. Let's drop the `ems_ref` for now. Let's use `container_node_id`, `container_pod_id` and `name` as references.


397[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:22:5: warning: '{' should be on the previous line.

398[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:27: warning: First sentence should be present.

399[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:31:5: warning: '{' should be on the previous line.

400[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:32: warning: Line is longer than 150 characters (found 192).

401[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:38:5: warning: '{' should be on the previous line.

402[ant:checkstyle] src/main/java/com/mcgoodtime/gti/client/gui/GuiCarbonizeFurnace.java:46:9: warning: '{' should be on the previous line.

403[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/BlockCarbonizeFurnace.java:22: warning: First sentence should be present.

404[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/BlockCarbonizeFurnace.java:50:104: warning: Reference to instance variable 'sideFront' needs "this.".

405[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/BlockCarbonizeFurnace.java:67:5: warning: '{' should be on the previous line.

406[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/BlockCarbonizeFurnace.java:94:105: warning: Reference to instance variable 'isBurn' needs "this.".

407[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/BlockCarbonizeFurnace.java:95:101: warning: Reference to instance variable 'isBurn' needs "this.".

408[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/blocks/GenGasKu.java:0: warning: File does not end with a newline.

409[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/init/GtiBlocks.java:28: warning: Wrong lexicographical order for 'com.mcgoodtime.gti.common.blocks.BlockCarbonizeFurnace' import.

410[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/init/GtiBlocks.java:30: warning: Wrong lexicographical order for 'com.mcgoodtime.gti.common.blocks.GenGasKu' import.

411[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:16: warning: First sentence should be present.

412[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:26:5: warning: '{' should be on the previous line.

413[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:34:9: warning: '{' should be on the previous line.

414[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:36:13: warning: '{' should be on the previous line.

415[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:42:9: warning: '{' should be on the previous line.

416[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:48:5: warning: '{' should be on the previous line.

417[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:59:5: warning: '{' should be on the previous line.

418[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:63:9: warning: '{' should be on the previous line.

419[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:67:13: warning: '{' should be on the previous line.

420[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:72:13: warning: '{' should be on the previous line.

421[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:77:13: warning: '{' should be on the previous line.

422[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:89:5: warning: '{' should be on the previous line.

423[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:91:9: warning: '{' should be on the previous line.

424[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:96:9: warning: '{' should be on the previous line.

425[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:101:9: warning: '{' should be on the previous line.

426[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:107:5: warning: '{' should be on the previous line.

427[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:115:5: warning: '{' should be on the previous line.

428[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:120:9: warning: '{' should be on the previous line.

429[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:125:13: warning: '{' should be on the previous line.

430[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:127:17: warning: '{' should be on the previous line.

431[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:132:13: warning: '}' should be on the same line.

432[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:134:13: warning: '{' should be on the previous line.

433[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:136:17: warning: '{' should be on the previous line.

434[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:138:21: warning: '{' should be on the previous line.

435[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:141:17: warning: '}' should be on the same line.

436[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:143:17: warning: '{' should be on the previous line.

437[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:145:21: warning: '{' should be on the previous line.

438[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:148:17: warning: '}' should be on the same line.

439[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:150:17: warning: '{' should be on the previous line.

440[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:152:21: warning: '{' should be on the previous line.

441[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:155:17: warning: '}' should be on the same line.

442[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:157:17: warning: '{' should be on the previous line.

443[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:160:13: warning: '}' should be on the same line.

444[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:162:13: warning: '{' should be on the previous line.

445[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:167:13: warning: '{' should be on the previous line.

446[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:169:13: warning: '}' should be on the same line.

447[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:171:13: warning: '{' should be on the previous line.

448[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/inventory/ContainerCarbonizeFurnace.java:176:13: warning: '{' should be on the previous line.

449[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:0: warning: File does not end with a newline.

450[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:25:1: warning: '{' should be on the previous line.

451[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:29: warning: First sentence should be present.

452[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:31: warning: First sentence should be present.

453[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:33: warning: First sentence should be present.

454[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:35: warning: First sentence should be present.

455[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:43:5: warning: '{' should be on the previous line.

456[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:47: warning: First sentence should be present.

457[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:51:5: warning: '{' should be on the previous line.

458[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:60:5: warning: '{' should be on the previous line.

459[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:62:9: warning: '{' should be on the previous line.

460[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:66:13: warning: '{' should be on the previous line.

461[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:70:13: warning: '}' should be on the same line.

462[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:72:13: warning: '{' should be on the previous line.

463[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:76:17: warning: '{' should be on the previous line.

464[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:82:9: warning: '}' should be on the same line.

465[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:84:9: warning: '{' should be on the previous line.

466[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:94:5: warning: '{' should be on the previous line.

467[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:96:9: warning: '{' should be on the previous line.

468[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:100:9: warning: '}' should be on the same line.

469[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:102:9: warning: '{' should be on the previous line.

470[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:111:5: warning: '{' should be on the previous line.

471[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:115:9: warning: '{' should be on the previous line.

472[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:120: warning: First sentence should be present.

473[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:124:5: warning: '{' should be on the previous line.

474[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:128: warning: First sentence should be present.

475[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:132:5: warning: '{' should be on the previous line.

476[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:137:5: warning: '{' should be on the previous line.

477[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:141: warning: Abbreviation in name must contain no more than '1' capital letters.

478[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:142:5: warning: '{' should be on the previous line.

479[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:148:9: warning: '{' should be on the previous line.

480[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:153:13: warning: '{' should be on the previous line.

481[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:163:9: warning: '{' should be on the previous line.

482[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:168: warning: Abbreviation in name must contain no more than '1' capital letters.

483[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:169:5: warning: '{' should be on the previous line.

484[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:176:9: warning: '{' should be on the previous line.

485[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:178:13: warning: '{' should be on the previous line.

486[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:189:9: warning: '{' should be on the previous line.

487[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:198:5: warning: '{' should be on the previous line.

488[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:202: warning: First sentence should be present.

489[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:208:5: warning: '{' should be on the previous line.

490[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:212: warning: First sentence should be present.

491[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:218:5: warning: '{' should be on the previous line.

492[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:220:9: warning: '{' should be on the previous line.

493[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:227: warning: First sentence should be present.

494[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:231:5: warning: '{' should be on the previous line.

495[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:236:5: warning: '{' should be on the previous line.

496[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:241:9: warning: '{' should be on the previous line.

497[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:246:9: warning: '{' should be on the previous line.

498[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:248:13: warning: '{' should be on the previous line.

499[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:250:17: warning: '{' should be on the previous line.

500[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:254:21: warning: '{' should be on the previous line.

501[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:258:25: warning: '{' should be on the previous line.

502[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:262:29: warning: '{' should be on the previous line.

503[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:263:61: warning: Reference to instance variable 'furnaceItemStacks' needs "this.".

504[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:263:109: warning: Reference to instance variable 'furnaceItemStacks' needs "this.".

505[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:270:17: warning: '{' should be on the previous line.

506[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:274:21: warning: '{' should be on the previous line.

507[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:279:17: warning: '}' should be on the same line.

508[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:281:17: warning: '{' should be on the previous line.

509[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:287:13: warning: '{' should be on the previous line.

510[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:294:9: warning: '{' should be on the previous line.

511[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:303:5: warning: '{' should be on the previous line.

512[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:305:9: warning: '{' should be on the previous line.

513[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:307:9: warning: '}' should be on the same line.

514[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:309:9: warning: '{' should be on the previous line.

515[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:311: warning: 'if' construct must use '{}'s.

516[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:312: warning: 'if' construct must use '{}'s.

517[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:313: warning: 'if' construct must use '{}'s.

518[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:314:26: warning: Reference to instance variable 'furnaceItemStacks' needs "this.".

519[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:315: warning: Line is longer than 150 characters (found 165).

520[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:319: warning: First sentence should be present.

521[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:323:5: warning: '{' should be on the previous line.

522[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:325:9: warning: '{' should be on the previous line.

523[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:329:13: warning: '{' should be on the previous line.

524[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:331:13: warning: '}' should be on the same line.

525[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:333:13: warning: '{' should be on the previous line.

526[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:340:13: warning: '{' should be on the previous line.

527[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:346: warning: First sentence should be present.

528[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:351:5: warning: '{' should be on the previous line.

529[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:353:9: warning: '{' should be on the previous line.

530[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:355:9: warning: '}' should be on the same line.

531[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:357:9: warning: '{' should be on the previous line.

532[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:359: warning: 'if' construct must use '{}'s.

533[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:360: warning: 'else' construct must use '{}'s.

534[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:365:5: warning: '{' should be on the previous line.

535[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:366: warning: First sentence should be present.

536[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:373: warning: First sentence should be present.

537[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:377:5: warning: '{' should be on the previous line.

538[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:378: warning: Line is longer than 150 characters (found 218).

539[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:389:5: warning: '{' should be on the previous line.

540[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:398:5: warning: '{' should be on the previous line.

541[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:407:5: warning: '{' should be on the previous line.

542[ant:checkstyle] src/main/java/com/mcgoodtime/gti/common/tiles/TileCarbonizeFurnace.java:416:5: warning: '{' should be on the previous line


Ok, I'll change it.


OK, I'll have to investigate how to do a NEWS entry... thanks!


This looks fine, assuming there are no false negatives on jQuery UI.


This change also causes the overall sort to be in the wrong order, being sorted by the oldest after this change...


Овде чак није реч да ли је превод у реду или не, пошто је то назив који се већ користи у Вордпресу, ако би се и променио онда мора свуда, не ваља да буду различити јер је онда нејасно да се ради о истој ствари.

Пре сам присталица да има мање преведених сродних пројеката али усклађених око назива ствари које се у њима користе (не баш сваке споредне речи, разумљиво је да понека буде различита) и да буду квалитетнији (као што си ти радио) него да има више али и са више имена и различитог квалитета па се не зна шта је шта.


IIRC it's here that you have to specify `container_node_id`, `container_pod_id` and `name` instead of `ems_ref`.


We are dealing with two types of the same simple name.


Let's use `name`.


Or how about a suffix to the version: "_matlab"?


Should we? After all `Configuration` is still there.


You have to use `t.bigint`. Also add `container_group_id`.


Re your understanding of MVVM - yep that's basically it. The cleaner separation you keep between your V and VM, the easier it will be to structure your application properly and the easier it will be to test your VM logic. If you find you need VMs need to communicate with one another you can use a messenger or something like that


`\.ph(p[2-7]?|tml)$` :)


in ruby the ? in method name should be reserved for methods returning boolean.

also you can define setter method as, attr_setter :completed, which is equivalent to defining method

def completed=(bool)
   @completed=bool
end

then you call it as Object.new.completed = true


Termine tout ce que tu as faire et merge pas.

facilities part de 47, je m'occuperai de ce merge un peu tricky


Canonical appears to be  httpS now for all YouTube URLs in the browser.


The problem is that the git version refers to the version used to generate the input image, and this might have absolutely nothing to do with what happened to the image in matlab... I really don't think we want to keep any reference to the original git version here.


In a non JPA bootstrap, non `OgmConfiguration` world, how do you set OGM_ON to true only when OGM needs to be started (mixed environments)


Why mixed?

There can something like `ClassName|bool|null`.


same setter concern


wqedsdsdsdf


instead of expecting error to be null, you could make a get request to check if the information was actually saved


I think that constraints on fixed parameters are ignored - but I'm not 100% sure.


Regression.

With this change, it's not possible to set values into deep hashes any more.
For example, consider a property path like `someProperty[foo][bar]` with a class like:

```php
class Foo
{
    /**
     * Example: $this->someProperty['foo']['bar'] = 'baz'
     * @var array
     */
    public $someProperty;
}
```

With this change, it will not be possible to set a value for `bar` any more.

```php
$obj = new Foo();
$obj->someProperty = ['foo' => ['bar' => 'some_value']];

$propertyAccessor->setValue($obj, 'someProperty[foo][bar]', 'another_value');

echo $obj->someProperty['foo']['bar'];
// Before this patch: 'another_value'
// After this patch: 'some_value' => fail
```


Nice work.
Don't push commented out codes


This is what I was asking about on the call yesterday. We've added a feature to Red Hat's Docker RPM so if you do `docker pull foo/bar`, the daemon will search through all the configured registries until it finds `foo/bar`, e.g. redhat.com, othercompany.com, docker.io. I'm wondering if this is a hard requirement, or is it sufficient instead to try to find `foo/bar` in all of the *registries* configured to provide images for the *default namespace*? In other words, instead of searching redhat.com, othercompany.com, and docker.io for `foo/bar`, expand `foo/bar` to `defaultnamespace.com/foo/bar` and search all the registries preconfigured for `defaultnamespace.com`. Does that make sense? Would that work for us?

/cc @rhatdan


The model layer has always been a little funny to me - when I hear "model" I'm usually thinking a collection of structs or classes that represent the objects your application work with. The model in MVVM refers to this as well as repositories, which are classes that you're VMs use to retrieve or store objects somewhere.


Yes, that is how I intended to do it :-)


See my comment on the previous bullet


On 07/05/15 14:30, Wei Liu wrote:
> Can you elaborate what model you have in mind and how I can structure this driver? Even though I don't have time to do it now I might rework it in the future.

E.g. the networking driver, where the kernel side and the hypercall side
is explicitly divided into different modules.  I think that makes for
very easy reasoning about what can be held where, etc.

Ian didn't like that style for whatever reason, so he invented his own
style for xendev where symbols are "leaked" from one side to the other.


So basically being able to map *scope* to index with config line

    docker.io/        index        https://index.docker.com

and using the url of index during `docker search` solves it.


I kind of liked this list of classes, what't the reason for getting rid of it?


Edgar,

this is quite likely.
/* I did not run any romio tests on master with Jeff's commits */

just to be sure, did you remove your install dir before you run make install ?
there is a risk an old module is causing this.

I will investigate this from tomorrow


I feel like ```delete``` should return nothing. I can think of two use cases for the boolean:
1. entity exists, but we failed to delete it
2. entity does not exist

Case 1 is error-handling, and should be dealt with elsewhere. For Case 2, the user should just call ```get``` to check for existence.


kshf


Yes, it made no difference (IDL is weird).  Ignore this.


So then we can't just have a single resource version in order to invalidate the cache.

Sent from my iPhone

> On May 7, 2015, at 10:45 AM, David Eads <notifications@github.com> wrote:
>
> do we enforce there is only one clusterPolicy ?
>
> For now? I can certainly see a case for allowing more than one and permissioning them differently. The rest of the code handles multiples.
>
> —
> Reply to this email directly or view it on GitHub.
>


Done. 1507ece5ae1cf0c3ff945b95298396cf532f0a6d


I think keeping the .lock outside GIT (as the vendor folder) is the best way :
you clone the project without the content of vendor and you will never save it in git, so same thing for the `.lock` file.


The setup script actually makes sure that the values are WITHIN the constraints, so that’s not an issue.

I was more worried about speed, e.g. as they are being constantly checked.
By the sounds of it, they are not, so I’ll leave the code as is in that point.


Need more consts


Суштина је где повући линију код употребе новијих туђица, негде мора, али треба пазити да се не оде у другу крајност.


感谢，已经push。


Oops.  Fixed in 5bdc31a42bd9760437f6563c31a3a5a93353b6ba.


So in canInterpret I should have something like this: ?
```
if (CommandWithNameAndFlags::canInterpret(...))
{
    while (source && !source->node()) source = source->parent();
    if (source) return  source->node()->manager();
}
return false;
```


Hello,
Did you test this module on Amazon Linux Server?
thanks.


Done. Hopefully, the PR meets your expectation. :-)


yes, I did remove everything, that was my initial suspicion as well.
Just to be safe I rm -rf the installation directory once more, redid the
make install, and the error still occurs.

Thanks
Edgar

On 5/7/2015 9:57 AM, Gilles Gouaillardet wrote:
> Edgar,
>
> this is quite likely.
> /* I did not run any romio tests on master with Jeff's commits */
>
> just to be sure, did you remove your install dir before you run make
> install ?
> there is a risk an old module is causing this.
>
> I will investigate this from tomorrow
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-mpi/ompi/commit/db257cdbc007b0c805dffe3157809f803c9342d3#commitcomment-11089011>.
>

--
Edgar Gabriel
Associate Professor
Parallel Software Technologies Lab      http://pstl.cs.uh.edu
Department of Computer Science          University of Houston
Philip G. Hoffman Hall, Room 524        Houston, TX-77204, USA
Tel: +1 (713) 743-3857                  Fax: +1 (713) 743-3335
--


No se que significa


*Insurance

I don't think "refund" is the right word here. Insurance refund is about reimbursing premiums. It should probably be "compensation" or "indemnity".


ah... I use it in several points when "something was broken" ;)


Yes, that could be definitely an option.

Il giorno gio 7 mag 2015 alle ore 08:50 francesco agati <
notifications@github.com> ha scritto:

> yes this is true. i think at the two possibilities. if is passed a
> constant string (without variable but as a symbol) the path can be
> processed at compile time, otherwise at runtime.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fponticelli/thx.core/commit/bbf5538b4176283cc43268d6c07d4e0ad2f64961#commitcomment-11088810>
> .
>


Looks about what I had in mind.


This would only make Travis to install and run tox on Python 3.3 also. But tox under Python 2 and 3 run exactly same set of environments.


I think I'd still have to do this comparison, because of the default flavors that are created.


Потпуно се слажем :+1: Али с друге стране, не идемо у "умножницу" него у "(фото)копирницу"


I agree in this point, this is weird. Only just noticed, too.

More dangerous, actually. in new versions of IDL
arr[-1] is the last element.
as -1 is what an empty where statement returns, this is dangerous! Now you always have to check manually!

In old versions, it was a great way to break the code if where is empty.

> On 7 May 2015, at 15:58, Steven Bamford <notifications@github.com> wrote:
>
> Yes, it made no difference (IDL is weird). Ignore this.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/MegaMorph/galapagos/commit/440196bc8d031924529a3f83f14d4e88787f2db1#commitcomment-11089026>.
>


Tape is in place. Changes made


I'm sorry .....
can you come that time i'm said you


Fixed in https://github.com/whatwg/mediasession/commit/063f961b92076ff722923f4c6745087796a25757.


#player,#player a{ color:#fff; }
#player{ background:url(sizerAndVol.png); background-position:bottom; }
#player .mainImage{ background:url(main.png); }

#player #baseLeft{ width:93px; }
#player #baseRight{ width:145px;}
#player .button{  width:22px; }
#player #play{ background:url(playBtn.png); }
#player #play,#player #pause{ left:32px; }
#player #previous{ left:8px; }
#player #next{ left:55px; }
#player #title{ left:95px;  }
#player #tooltip{ right:196px; }
#player #timer{ right:122px;  }
#player #list{ right:5px; }

#player #seeker{ right:320px; }
#player #seekerBase{ border-color:#dbf8ff; }
#player #seekerBuffered{ background-color:#86caff; }
#player #seekerSliderBase{ background-color:#fff; }

#player #volume{ width:70px; right:33px; }
#player #volumeSlider{ width:14px; background:url(sizerAndVol.png); }

#playlist,#playlist .module,#playlist h3{ border-color:#60AFFF; color:#1265ff; }
#playlist h3{ background:url(bar.jpg); text-shadow: white 1px 1px; }
#playlist .even{background-color:#FFF;}
#playlist .odd{ background-color:#dff6ff;}
#playlist .focus{ background-color:#0099FF; color:#FFF;}


Multiple python versions in `.travis.yml` make Travis-CI run each env (travis env, not tox env) under each python version. You should rather leave one python version in travis config (no matter which one) and let tox deal with the rest.


k

On Thu, May 7, 2015 at 8:32 PM, maheshikaHA <notifications@github.com>
wrote:

> I'm sorry .....
> can you come that time i'm said you
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PCWithana/Black-Board/commit/8b66cb2b03c7cfdc3f09c52076f70c95f6b145e9#commitcomment-11089164>
> .
>


I think that's wrong, it should be that you enable it when you want OGM to run.


@tvsonar what is the purpose of this file?


`Care::id` does not exist.


@edgargabriel @jsquyres should this be "romio314" instead of "romio" ?


I don't think you meant to leave this here? pry isn't in the Gemfile for starters.


*UniiqCare


@magnumripper should this get_integer be (renamed and) moved into common.c, top be reused by other formats?


This change breaks the build. there are entries like il_seq_points with 0 length in at least amd64 and x86.


I guess this has set some timeoutout set? Should there be exception catch for that? Or it is somehow handled?


Commented code?


Actually, let's rather say that it depends, so let's discuss that.


Ttttt


import com.sun.org.apache.xpath.internal.SourceTree;

import java.util.Scanner;

public class Task1 {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        System.out.print("Enter your name: ");
        String s1 = scanner.nextLine();

        System.out.print("Enter your surname: ");
        String s2 = scanner.nextLine();

        System.out.print("Enter your age: ");

        if (scanner.hasNextInt())
        {
            int a = scanner.nextInt();
            if (a < 0 || a > 120)
            {
                System.out.println("IllegalAge");
            }
            else
            {
                System.out.println("I'm " + s1 + " " + s2 + ", " + a + " years old.");
            }
        }
        else
        {
            System.out.println("IllegalAge");
        }
        scanner.close();
    }
}


A multiline string would be cleaner here.


correctly sorted array


@tritao can the code be changed so that the build is no longer broken?


Done (https://bro-tracker.atlassian.net/browse/BIT-1394)
Contains the error, log files, and the environment+everything configured/installed.

For the proxies - we ended up doubling the proxies per system yesterday, and that seems to have fixed the overload problem. For every 10Gbps stream that we process, we now have 4 proxies. That seems to be the magical number where it splits the peers nicely and is not overloaded.


I think that you can refactor this part to make less API calls by using filters for list() modules, instead of 3 calls per resources.

Making a call to `Ip.list({'ip': ['1.2.3.4', '5.6.7.8']})` should filters the first time to retrieve the ifaces_id list and then another call to `Iface.list({'id': [42, 202020]})` to retrieve all the vm_ids.

But maybe there is a easier way if you retrieve all ips, ifaces and vm ids, as done in `gandi ip list --vm`  and then filter by what you have in resources.


Yes, that is much better.  I'll revise my commit.


Account status false? This should probably be replaced by an enum or use a more descriptive name.


Fixed in https://github.com/whatwg/mediasession/commit/408ef219ff9383f233890288882e334f7cd08cc1.


Ah. Sorry if this is the wrong place to ask, but how then do you use T*index* in scripts?


@orgkhnargh I'm new to Tox, could give me an example for that?


Is in galapagos now as a start option.
Haven't tested, but should work. Will test the next time I run it.


I'm not sure but this `_detach` method was removed above so you must update this part so the delete automatically detach the ifaces first.


Ouch.


#2  수정 완료.


Using `str.has(re.to_re)` would be cleaner, slightly faster and very slightly more memory efficient.

Also the type of the method arguments could be merged.


Consider running
```console
$ gen_travis.bash > .travis.yml
```
to add a list of all tox environments into `.travis.yml`,


done


```perl
    next unless /=encoding \s+ ( \S+ )/ix;
```


I use 1/10. for galaxies within the postage stamps now (your first block).
and 1/5. for the galaxies outside of this (your second block)


changed


eg,
- for an OGM only app, true is a good default
- for a hybrid ORM/OGM app, false is a good default
- in a WF embedding both ORM and OGM, false is a good default if it comes transparently in. If ogm added is an explicit thing, then the first two rules make sense.


I am not sure to understand.
Previously initializeScheduler was called only once per program execution and resyncThread was never destroyed. Now initializeScheduler may be called each time that World_New is used and so I start resyncThread only if it does not exist yet to avoid having several resyncThreads


Nope, it's not :P


The only reason `symfony/symfony` is required is for its `debug:container` command. It won't be this way forever, though. I will pare down the original command and include it with the application.


Ficheiros a ignorar..


no need for `the` here


Should be fixed by https://github.com/mono/mono/commit/d105620caa9c38b33b38b85a4d2d4ec5f0b6c31e.

Thanks for the heads-up.


declaration -> definition


Redefinition


I think I looked at this one befrore and the better solution wold be to change ptr into char.


In principle you could also do `$tree->clone`, which is computationally cheaper (but deep clones are a bit scary)


Please translate %LOCALESET%/homepage.php too.


But the gen_travis.bash is only.

```
#!/usr/bin/env bash

cat <<"END"
language: python
python:
  - 2.7
install: pip install tox
script: tox -e $ENV
env:
END

if [[ $1 == "--all" ]] ; then
  tox --list | (
    while read i ; do
      echo "  - ENV=$i"
    done
  )
else
  # a few representative envs...
  cat <<"END"
  - ENV=py27-django15-pmlatest
  - ENV=py27-django16-pmlatest
  - ENV=py33-django15-pmlatest
  - ENV=py33-django16-pmlatest
END
fi
```

This is hardcoded, right?


there are tests that test this behavior already. besides the setTimeout test will fail after the extraneous zone spawning is fixed. so I think it's good as is.

I also can't change the order because the next commit depends on this one.

and the last option to squash the two commits doesn't feel right since one adds a feature and the other fixes a bug with a breaking change.

I suggest keeping as is.


Actually, though, your logic is not quite correct.  But, point taken.  I'll clean up the code while maintaining the existing logic.


Its important that simplify indeed simplifies trivial equations when it can (example: eax==eax simplifies to cst(1,1)) but unfortunately z3 makes a difference between a bitvec of size 1 and a boolean, so indeed the two forms are not equivalent for z3...reason why 'cast_z3_bool' is defined is smt.py.

What could be done to avoid an assert is to add to the solver the cast_z3_bool(e) which would allow to have bit0/bit1 in the provided eqns list and add them as "0x0==0x1" or "0x1==0x1" z3 equations.

I will commit this, let me know if its ok for you.


Not yet. I meant to port it, but ran out of time back then. Just created [OGM-816](https://hibernate.atlassian.net/browse/OGM-816) to track it. Should not take long.


为什么没 review 就和了 PR？


Nope, it's not. Look closer at this part:

```bash
if [[ $1 == "--all" ]] ; then
  tox --list | (
    while read i ; do
      echo "  - ENV=$i"
    done
  )
```

Oh, didn't mentioned that option myself...
You should use `--all` option to get all environments:

```console
$ gen_travis.bash --all > .travis.yml
```


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: Tests Failed

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


I recommend making this a hyperlink, rather than requiring the people copy and paste it.

Once I have some time to refine this install script we should probably put it on this page rather than merely linking to it.


@sipa sounds good?


this should be LOGF_DEBUG, right? Does it even run without an F?


`perlpod` says:
- `=encoding` affects the whole document and must occur only once
- put a `=encoding` command **early** in the document

This basically implies that a single pass on the document is necessary. No need to use a second loop and a seek just to find the encoding.


Log.h is not thirt-party, it should be under "// LSST headers" section.


Can you give me until 12:00 US/Eastern (45 minutes from now)?


I thought about it, but that'd would require ORM users to deal with the wildcard type `SessionFactorybuilder<?>` which seems not so nice.


ack - I'll keep an eye on these warnings (so far, I've only seen them in active_merchant / offsite_payments IIRC).


This is corrected in 24243dc


i think we should only add "log" here, and "log" should trigger log4cxx automatically


remove log4cxx


can you try to fix io_romio_component.c as I explained in the github commit
comment ?

On Friday, May 8, 2015, Edgar Gabriel <notifications@github.com> wrote:

> yes, I did remove everything, that was my initial suspicion as well.
> Just to be safe I rm -rf the installation directory once more, redid the
> make install, and the error still occurs.
>
> Thanks
> Edgar
>
> On 5/7/2015 9:57 AM, Gilles Gouaillardet wrote:
> > Edgar,
> >
> > this is quite likely.
> > /* I did not run any romio tests on master with Jeff's commits */
> >
> > just to be sure, did you remove your install dir before you run make
> > install ?
> > there is a risk an old module is causing this.
> >
> > I will investigate this from tomorrow
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/open-mpi/ompi/commit/db257cdbc007b0c805dffe3157809f803c9342d3#commitcomment-11089011
> >.
> >
>
> --
> Edgar Gabriel
> Associate Professor
> Parallel Software Technologies Lab http://pstl.cs.uh.edu
> Department of Computer Science University of Houston
> Philip G. Hoffman Hall, Room 524 Houston, TX-77204, USA
> Tel: +1 (713) 743-3857 Fax: +1 (713) 743-3335
> --
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-mpi/ompi/commit/db257cdbc007b0c805dffe3157809f803c9342d3#commitcomment-11089090>
> .
>


Awesome javadoc :+1:


While you're at it you might also want to remove the static, or did I miss something?


I note in passing I have /no/ idea how this works.  gencode is meant to return a string, but for these determinant nodes it instead returns an ast fragment, that is somehow inserted?


lsst not 3rd party


Don't give the container the same id as the modal.


Move into external .js file


not qserv, lsst


I wonder if these things should not actually be nodes but instead functions that return the appropriate ast fragment.


Hello,
I am learning SDN and trying out the application features related to it.
I am able to understand this code. Could you please let me know how to run this code. I have already ryu installed in my system and functional. Need your help to know the steps for running this particular stateful firewall.

Thank You !! Happy Learning !!


Yes, the idea being that if you have OGM on the classpath, you likely want to use it. If needed, you can opt-out and go with plain ORM (e.g. in mixed set-up with several PUs) through some public property which is added in the last commit of this PR.


I use Brackets.


首先要校验product是不是为null或者empty，user是不是为null


It is on by default.


"Built"


如果哈希表里面按照product找没有记录，那么应该先add一条，而不是exception


Minetest very consistently uses spaces for identation. Why make an exception here?


In addition to the explicit property, we also could think about turning OGM off if e.g. "hibernate.dialect" is set.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: Tests Failed

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


Am I right that TextRenderer::nonStaticTextBound() is the specific part which I should look at ?


`subscriptions.get(product).contains(u)`
这样的 语法是不推荐的，因为第一步`subscriptions.get(product) `的结果可能是null


@michael-e I do have the same problem sometimes.

> When I commented this line

Which one ?


Get some tips on how to do this with ObjectGuid instead of Creature*


page.find('#toggle_advanced_options').click

In futur, if we add tests for translation, the text will change so it's better to use the id. Why do you use .trigger instead of .click?


Multiple stories in the same commit? Difficult to review.


Why not leave a comment?


page.find('#toggle_advanced_options').click


Symlink references can be circular so we'd need to keep track of how many we have traversed and stop at some limit. It would add some complexity, but it is certainly doable. A PR would be welcome if you're so inclined.


I have analysed the new version and the patches no longer apply.  He is using a different esp so I have removed the patch from Compatibility Patches page.  Users will be directed there but they won't find a patch.  This should not be reverted.


Fixed


Could you remove this empty line?


Also, if devs decide to keep it as a pointer, it should be initialized as
````c++
jules = NULL;
````
or
````
jules = nullptr;
````


yes


consistent with java list
https://docs.oracle.com/javase/8/docs/api/java/util/List.html#remove-java.lang.Object-


Please increase the TTL (to let's say 24h) as this query takes quite long during DB load spikes.


Holly molly :1234:


因为是多线程程序，所以这里的单例需要doublecheck


加上 volatile关键字


> Should we?

Hum, good question :) It was always my impression that we want to get rid of it. I think it will go away in ORM also some time soon? So felt why not do it right now in OGM before even more people get in touch with it.

> how do you set OGM_ON to true

It would be enabled by default. In a mixed set-up, where you want to bootstrap ORM actually, you could disable it through a public property you set on the `ServiceRegistryBuilder`.


@L2G can you push this to the supermarket https://supermarket.chef.io/cookbooks/timezone-ii the version on there is quite old. Thanks


Ah, sorry, I got spoiled by GitHub's preview showing a hyperlink automatically...
Fixed in f49a8989e132cc44af7deb4d09eb3f08a69d5a26.

I agree we should eventually put it on the website directly.


This should be considered: Version system changes: Part 2


I was indeed a bit scared by the 'use with caution' comment in the TreeRole docu. I'll try if it works on my data!


Yeah, there's already enough refactoring changes and I don't want to make the patch too large.


done


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: Tests Failed

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


Ah I see, its only to replace default tab identation. Then its ok.


you're not paid by the line, don't put too many empty ones :)


If we did want to get rid of the joined tables, I think an easy way to do it would be to just store it as JSON in the database and then parse it when it comes back out.  That should handle most every data type you would want, and as long as we configure the parser correctly, it could even handle dates, etc.


отображение дерева, конечно, жесткое


It's still there for the most tests. Only this one doesn't extend `OgmTestCase` anymore as it needs to customize the SF bootstrap, so it does everything itself.


Litt fort i svingene her, ja 😜


I have analysed the new version and the patches no longer apply.  He is using a different esp so I have removed the patch from my Compatibility Patches page.


I think these will be overwritten by the next Transifex pull, or has it been abandoned? There are a couple of strings yielding to FCs in Italian and I would like to fix those, but I am unsure where.


(it's funfetti)


Infos die am 07.05 während der Vorlesung mitgeteilt wurden.


That's actually a perfect timing for me


ok


This is still inside the <form> element, any reason why? Doesn't seem like it should be.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: Tests Failed

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


This cite is undefined,can someone add the source.


The style guide states "Use soft-tabs with a two space indent. Spaces are the only way to guarantee code renders the same in any person's environment." Can you update your editor to follow this?

http://localhost:8080/#/styleguide


I'm done.  It seems to work, but the numbers might be off due to typos.


@koikonom what was happening before you made this fix? Was this just returning all minions?

I don't really get the logic because install_master is setting `ec2.set_instance_tags(instance_ids, {'SaltMasterPrvIP': master_prv_ip})` then we are doing this in install_minions: `to_install = list(set(candidates).difference(set([x.id for x in existing_minions])))` but the existing minions are just anything with that `SaltMasterPrvIP` tag which is everything?


If that helps they should definitely go to the namespace `Event\Trait` instead of `Traits`.


Hey, you fixed my bug! thanks!


> for an OGM only app, true is a good default

+1

> for a hybrid ORM/OGM app, false is a good default

Is one really better than the other? I don't think it matters much which way around it is. WDYT about also disabling it if a typical ORM prop such as "hibernate.dialect" is set?

> in a WF embedding both ORM and OGM, false is a good default if it comes transparently in. If ogm added is an explicit thing, then the first two rules make sense.

AFAIK you still need to add the manifest entry to get the OGM modules, so you need to opt in by that.

The reason for the current behavior is that there is no way to disable OGM by default when bootstrapping through the new API (`MetadataBuilder` et al.). Our integrators etc. are discovered through the Java service loader and they run un-conditionally. The only way to effectively disable them is through some prop which then will stop them from actually doing anything.


Hi,

Thanks for your PR !


Viewer and editor are full of 4am code. Source of the problem is definitely there.


good point. will fix, amend, push -f


Error at clean install of latest master > Fatal error: Can't use function return value in write context in /home/****/apps/blog/handlers/post.php on line 54


sure


This came from https://github.com/ministryofjustice/bootstrap-salt/pull/18

If I remember what K said it was a problem where some objects had a `.id` and others had a `.instance_id`


We don't want this.  We want to generate a new salt for every password that is hashed.  Generate a salt, use that salt to hash a password, save the hashed password in the database.  Also, we want to use the `async` methods instead of `sync` like so:

```javascript
// Hashing a password for a new user
var user = req.body.user
bcrypt.hash(user.password, 10, function(err, hash) { // auto-gen's a salt when called with a number
  user.hash = hash
  delete user.password
  // Save user to the database
  var queryPromise = usersTable.insert(user)
  // etc...
});
```


rm this ... see below comment.


Why is 'rule.class_distribution == 0" needed? Wouldn't not rule.class_distribution already cover it?


master (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (348 of 348 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)


would it be worthwhile to keep an instance of the cache, rather than querying multiple times?


I will never be able to see this env var without thinking of [this](http://en.wikipedia.org/wiki/Manifest_destiny).


This *also* broke mono_mixdown, which now defaults to on because you added the default value before the persist value and "true" maps to double 1.0


that was my half formed guess.


Ah, interesting. It seems I never read RFC4627 sufficiently thoroughly :)


This is how it looks,is it meant this way?
I would suggest that a userstory begins on a new line.

![temp](https://cloud.githubusercontent.com/assets/5946438/7519157/c128d3a0-f4df-11e4-803f-6769e913c9f7.PNG)


I don't think we need to leave this comment in... we have git history.


well dialect is optional but I get the idea.


For anyone in 2015 that is still looking for this problem

try:
```javascript
var nic		= new nicEditor({fullPanel : true});
var nicinstance= nic.nicInstances[0];
nicinstance.setContent('wwoot')
nicinstance.getContent()
```

That works for me.


а как ее проверить? надо бы show еще реализовать хотя бы какой-нибудь


Works great, thank you for the portation :)


So everywhere in the code where we set a control object to persist=true, we're actually setting the default to 1.0 and not persisting.


Hi debashis,

You can just run any of the firewall applications (for example:
inefficient_stateless_firewall.py) by passing the file as an argument to
ryu-manager command.!!

Write me back if you have any problem with the code.

Regards,
Jay

On 7 May 2015 at 17:19, debashish216 <notifications@github.com> wrote:

> Hello,
> I am learning SDN and trying out the application features related to it.
> I am able to understand this code. Could you please let me know how to run
> this code. I have already ryu installed in my system and functional. Need
> your help to know the steps for running this particular stateful firewall.
>
> Thank You !! Happy Learning !!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jms30/SDN_Firewall/commit/a7d09dc3e466c58fa1e9139bef368861f8bb24a1#commitcomment-11089523>
> .
>


when there are default arguments, new arguments need to go on the end of the call chain to prevent this from happening


@inca `.do.wait()` without parameters will wait page loading and doesn't make sense in this case. [fix](https://github.com/nodeca/navit/commit/49905a14ba7844bc5a80469a3b5816e259774a8a)


I updated code addressing these comments.


This works, but couldn't you just use `unlinkat()`?


Also, the BOM won't be added on IE 10+


is it worthwhile to protect the singleton queryinstance method?


@astaric @thocevar @martinmozina Can we make this line universal? The other question is, do we want to - or do we just assume Python 2.7? (The third thing is that Python 2.6 is already long dead and nobody noticed the problem. What does this tell us?)


OK, `jules = nullptr;` it is, then.


I've never even heard of that. I just liked the pleasant rhyme  ¯\_(ツ)_/¯


@emilykggray can you take a look at this intro copy? Trying to incorporate a more spacey/futurey feel to go along with the space theme.

See this design from Marshall for context.

![pasted_image_at_2015_05_06_04_56_pm](https://cloud.githubusercontent.com/assets/1403188/7519271/040e05aa-f4ae-11e4-9be3-b3c27bead819.png)


master (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (348 of 348 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)


@Linuksiarz LOL, ktoś tu się wziął za magisterkę? :-D


We should probably get rid of default argument usage in Mixxx. This is the reason they aren't allowed at Google.
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Default_Arguments


> Hum, good question :) It was always my impression that we want to get rid of it. I think it will go away in ORM also some time soon? So felt why not do it right now in OGM before even more people get in touch with it.

My preference would be to kill it when ORM does. We have been talking about killing Configuration for a looooong time "any time soon" :)

> It would be enabled by default. In a mixed set-up, where you want to bootstrap ORM actually, you could disable it through a public property you set on the ServiceRegistryBuilder.

Let's discuss that but let's set this value explicitly when you know. e.g OgmConfiguration knows it builds an OGM SF. Same for a JPA perssitence unit started from HibOgmPersistenceProvider.


How often is this cache expired?


Review complete.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 26](http://teamcity.otenv.com/viewLog.html?buildId=1293752&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 27](http://teamcity.otenv.com/viewLog.html?buildId=1293753&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 22](http://teamcity.otenv.com/viewLog.html?buildId=1293754&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 26](http://teamcity.otenv.com/viewLog.html?buildId=1293752&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 23](http://teamcity.otenv.com/viewLog.html?buildId=1293756&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


'<' should be escaped


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 27](http://teamcity.otenv.com/viewLog.html?buildId=1293753&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


Pretty simple – just add an entry to https://github.com/JuliaLang/julia/blob/master/NEWS.md.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 22](http://teamcity.otenv.com/viewLog.html?buildId=1293754&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


You're right. I guess this isn't covered under our tests. I'll have to fix that too. But I'll just remove this for now.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 23](http://teamcity.otenv.com/viewLog.html?buildId=1293756&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


Good job, boyo.


@yasunobuigarashi
@tejitak さんのレビュー指摘内容がまだ反映されていないようですので、対応お願いいたします。

https://github.com/vuejs-jp/vuejs.org/pull/16#discussion_r29566996


It's not a validation, just a restriction I suppose. Can be refactored probably.


Awesome, thanks :+1:


I just landed a big cleanup to move all the autocleanup macros over to `g_autoptr()`.  Can you fix this instance and any others in the patch set I may have missed?


Ok -- please revert this until it's ready to go.

Before we mess around with this again we need some tests for persisting and default arguments that show that our first run configuration isn't broken -- because we can't keep going back and forth like this. It's not worth the human effort of verifying everything over and over so it should be automated first.


Currently. never. Adding code to make a configurable timeout...


Sweet :)


This isn't related to this PR, but nevermind.


We usually use a simpler check `options.searchDefaultTokenKeys !== false`. It works well for `true`, `false` and `undefined`, even though it behaves differently for edge cases like `null` or non-boolean value.


@cqql This made CI go red. I missed this the first time I went through it. I noticed it while working on [another branch](https://github.com/browserify-rails/browserify-rails/tree/make_dependencies_more_flexibile) and made some changes there but I don't use the feature and it isn't clear it is tested enough to make sure it doesn't break.

I asked the original author here about it: https://github.com/browserify-rails/browserify-rails/pull/70


Very nice work but he was crashing my TrinityCore-2013.
Something wrong with kick.


You're welcome! ;)


Added a comment, but probably in the wrong spot. Sorry! https://github.com/sparkbox/apprenticeship-web-project/commit/c25dfbea9133642120bb2a65ac5ad8800f2d1403


OK, sorry about that.


done


Any idea why diff includes every line of this table rather than just the new line?


`produces = MimeTypeUtils.APPLICATION_JSON_VALUE`


Hmm, ideally even edited todo state should be stored in global app state.


Yes


Shouldn't the brackets be surrounding the values? Something like
``
bquery + "('%s')" % ", ".join([row[0] for row in results])
``


Do we need both 7 & 8 here, if yes the key should be different (which is what I'm going to merge for now)


Yes, join it on shutdown would be better but it is a coreaudio-only issue and I dont feel sure touching many things without trying


No need to be sorry.  No harm no foul. I have contacted him on Steam for permission so if he denies my request then I can't make a new patch.  What I'll do instead is mark the file with a warning about the out of order subrecord and leave it at that.


All the issues are not addressed and the refactor is not fully debugged yet.
TBD


master (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


`om` could be an instance variable so that it's reused in subsequent requests


@tvsonar We will want to minimize the usage of Parent - because in theory you have to check for nullity at each stage.
Moroever, in theory you'd also have to check that the parent of the declarator is indeed a declaration, and so on.

Working from the parent down to the children is way better, because if the model ever changes, you'll get compilation errors instead of runtime errors.


mispelled 'canola'


Thanks @emilykggray. @cromwellryan mentioned another point about how this is a little too geared towards the potential apprentice, and not so much towards the offerer of the apprenticeship. Maybe instead of the "earn money while learning" he suggested something along the lines of "a sustainable way to teach web-workers of the future." Thoughts on that?


你想干嘛？


You can use http://sphinx-doc.org/markup/para.html#directive-note
```
.. note::

    On OS X ...
```


Nope, that would generate:
```sql
INSERT INTO packages (pkg_name) VALUES ('firefox, nethack, guake');
```
which would insert one package with a big long comma separated name.  Instead we want

```sql
INSERT INTO packages (pkgname) VALUES ('firefox'), ('nethack'), ('guake');
```

Which inserts three rows, each with a different name.


This filtering of resource seems to be using the old dublin core data model (key/value). I should have removed this when removing the dependency on 'dublin_core' submodule. Should I create a github issue for this and clean that up?


This link points to **rc6 release** and not to the new **0.1.0 release**


@davidmiller is this a moan at my expense?! ;)


I'll go with that then.


no


Not at all - it's deploying it to elcid04-test :)


This breaks the latest RbacCollector.


Is this how it is supposed to behave?
I mean, we are summing an int [s->tick()] to a pointer(?) [cr].
Shouldn't the parenthesis be s->tick() + (cr ? cr->actualTicks() : 0) instead?


Too bad. Seems just as good as the rank idea...

If you have time for this, may I suggest to try this locally ?

m.value =  PieceValue[MG][pos.piece_on(to_sq(m))] -
PSQT[pos.piece_on(to_sq(m))][to_sq(m)];


BTW - looks beautiful! Just what i hoped for


See previous comment.


Looks good!


ytutyutyu


h2 Change Life


hgfhjghjhjfhgj


    #1\grenobreak#2\relax %

should be enough... I'll do in a future commit


Hmm that's interesting. It stopped working for me after removing () so it seemed like that would be required.


ah defs not definitions... :)


Hi,

How can you use these two events to get confirmation from the user before removing the picture from the server, and remove it only if they confirm?

Thanks,
Ali


master (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (347 of 348 tests passed)

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


Hi.  Newbie question here.  Should the version of angular be tightened up and made more specific?  On my system, my "bower install" gave me 11 choices of angular to choose from.


This fixes #49


+++ Nick Wellnhofer [May 07 15 07:18 ]:
>   This saves a call to strlen but doesn't avoid the extra allocation in
>   cmark_node_set_literal. I think the best solution is a helper function
>   to detach a strbuf and store it in a chunk.

Isn't that what this does?

    static inline cmark_chunk
    cmark_chunk_buf_detach(cmark_strbuf *buf)


:+1: thanks!


@koying

Results in quite a few warnings now http://pastebin.com/raw.php?i=EJxxnXvJ


please open a new ticket, preferably as a PR with a failing test


`'ascii'`is not one of the codings registered with the IANA Content Coding [Registry](http://www.iana.org/assignments/http-parameters/http-parameters.xhtml#content-coding).  For clarity, can we make the default here either `None` or `'identity'`?


@alexpowerup Well I found it on my old *Romhacking tools* folder and I felt like it was nice to share it, even if it isn't my tool or source, but it's really simple to understand in my opinion.


Those are the minimum needed to create the encounter. I will look at adding some others.


Thanks for pointing this.
I understand that sending null and using another StringBody constructor without the encoding param have the same result, but I haven't looked into the actual StringBody implementation.

Anyway I'm more concern about the if check above as if compressByGzip cannot compress the data, it should throw an exception instead of returning the original data. I will leave this to @hideki to do a proper fix.


Yea I think the names let is useful when you would just be defining one helper function anyway

```racket
(define (fact n)
  (define (loop a n)
    (if (= a 0) a (loop (* n a) (sub1 n))))
  (loop 1 n))

(define (fact n)
  (let loop ([a 1]
                 [n n])
    (if (= a 0) a (loop (* n a) (sub1 n)))))
```
both are ok to me.


@AnisGitHub If you can, please commit all the various native files as well. gbxml, sketchup, openstudio, trace/ etc. Thanks.


Yeah, @kevincorcoran and I were discussing this, and we couldn't come up with a reason why the service itself would report `:unknown` (that it didn't know whether it was running or not). But I also don't necessarily see a reason why we shouldn't allow a service to return this, plus it would allow us to reuse this schema.


I know for a fact that @Divran is often being defensive!

*trigger*


Would something like Lifespan be more concise, perhaps?


That looks good, I don't even know how we could end up with  :

```java
existing.versionId != loadedVersion.versionId
```
since there's only one place the insertion is made it is made through :

```java
versions.putIfAbsent(loadedVersion.versionId, loadedVersion)
```


90179052-SetupEMSActuatorMissingDuplicates (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1644 of 1644 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)


@sbaechler
This is the cause for the logout. ".delete()" clears the whole session. To only remove the id you should use "del request.session['pledge_id']"


Perhaps this part should not have been removed.
Without the more accurate timestamp, when the client is reusing connections, I'm getting a very early request.timestamp_start, equal to when the connection started read-blocking


As someone unfamiliar with ExpAdv2, I have questions:
- Does it make use of E2's function repertoire? If not, why not? :)
- is it a true C-like without the silly whitespace dependency and swapped &&/& operators?


fix #43


Without some kind of notification to the player, or better yet (in my opinion), a "Rotten ______" remains in its place, it might be confusing why items are just vanishing away.


クラスの場合はinlineは、コンパイラが自動的に読み取って書かなくてもinlineのものはしてくれるよ～
まぁ、書いても書かなくてもどっちでもいいって感じですねｗ


The main reason behind that was to make the export files self-documented on defaulted parameter values. So we don't have to dig into the SDK while browsing the code.


@rodrigopastrana pleased rereview


We could have different versions of the lib on the classpath


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Build Failed

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


Does this mean items in Locations other than where the player is are frozen in time?


"[...] we will one of the ones with with the simplest deployment process" seems broken, maybe "we will use one of the easiest in term of deployment process"?


Hey,
FYI. This commit was pushed empty by mistake, but the change to sitereadiness v2.5 was actually included in the commit that follows:
https://github.com/cms-sw/cmsdist/commit/2eed43cf0fc2f97a5fbc8908ff1a93b39573351f


Yeah, I was basically going with that second option when writing this - basically not validating the `StatusCallbackResponse` that we get back, just providing it for someone registering a callback function to be able to use. But I guess I agree that I'm not sure that's the best option? But I think the question of whether we should catch and swallow vs re-raise gets at the question of what happens for multiple services, where only one errors? But as you pointed out, there's another ticket for that (which I didn't realize) so we could choose one now and figure that out later?


Good catch :bug:  - could you please send us a quick PR that fixes this?


issue #3


|**formato** |**PROS**| **CONTRAS**
|---------------|------------------|------------------|
|**NS**        | *es mas prolijo cuando es extenso*| *ocupa mucho espacio*
|                  |*se confeccion de manera decendente* | *en algoritmos extensos puede ser dificil su lectura*
||*es facil seguir el recorrido*| *cuando hay que volver a iniciar despues de una instruccion es complicada su lectura*
||*Favorece la programación estructurada*|
|**DDF**      | *es facil de entender*| *es complicado de leer y elaborar cuando son algoritmos extensos*
|                  | *ocupa menos espacio que otros metodos* | *no ilustra la estructura del programa*
||*Favorece la programación estructurada*|
|**Pseudocodigo** | *se adapta al leguaje y la idea del usuario* | *su lectura se dificulta ya que cada persona maneja su codigo*
| |*facil de crear y modificar*|
|**C++**| *despues de su escritura el programa esta listo para ser compilado*| *es el mas complejo de elaborar*|
||*muy eficiente para realizar programas cortos*|*Es difícil programar bases de datos*


Last thought here: Maybe instead of individual rot times on every item, have some categories? "RAW_MEAT", "SALTED_MEAT", "COOKED_MEAT"? Since I don't see a lot of animals having very meaningful variations in the time it takes their meat to decay.


`* Fixes a typo: s/Tru/True/` you can remove the bullet ...


Making renderAsNormalBlock return false prevents things like beds, torches, and monsters from spawning. See issue #433

When renderAsNormalBlock returns false, the Block.isNormalCube() returns false and the World.doesBlockHaveSolidTopSurface() returns false, preventing a lot of item and entity interaction.

Was this intentional? If so, can just close that bug as those issues are intended.


Cf. https://github.com/openmicroscopy/openmicroscopy/commit/19c3aeef6e873594dc5156a05ad80394340da9d9.


`INCOMPATIBLE Renames `run` to `run_py_func` and adds documentation.`


Does this fix the issue with the font icons: play, pause etc.. not showing up for users with IE 9?


But before we start modifying any code, it's probably a good idea to document use non-Concerto-plugin use cases to avoid undocumented/untested functionality.


ok


Different version with the same id then


Может всё же форму для вставки ключа зойпилите и сохранение там же где и настройки скрипта хранятся? А то править каждый раз при обновлении будет не айс.


probably this can be closed now?


Maybe should use watchcolleciton


..'s aren't needed, in the same parent dir.


Note for future: GDS style is to not indent "private" specifiers here (ie, it should have 2 spaces on the line before it).  Not going to let that block merge, though.


Thanks.  Penalties still confuse me, and you had said to "surround it" with nobreak.


I like this pattern (of deriving this type of class from Struct).


Could you please test https://github.com/django/django/pull/4621?


i agree that 1 is an exception. For 2 I don't have strong feelings.

On Thursday, May 7, 2015, Matt Sooknah <notifications@github.com> wrote:

> I feel like delete should return nothing. I can think of two use cases
> for the boolean:
> 1. entity exists, but we failed to delete it
> 2. entity does not exist
>
> Case 1 is error-handling, and should be dealt with elsewhere. For Case 2,
> the user should just call get to check for existence.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/broadinstitute/rawls/commit/0dba4c5ff32af68a90492a4a4ef1472ac5762b2d#commitcomment-11089023>
> .
>


@elibarzilay libcmark currently distinguishes between `strbufs` and `chunks`. `strbufs` are used to build strings, `chunks` are immutable. The "literal" property is stored in a `chunk`. So in order to append to it, a `strbuf` must be created first. It's not possible to append to a `chunk` directly.

@jgm `cmark_chunk_buf_detach` returns a new `chunk`. We'd need a helper that frees the previous contents of a `chunk` first, then sets it to the detached `strbuf`.


Is true .He helped me.


oups.


Der null Fehler liegt daran, dass der inflator nicht auf die activity gesetzt werden muss, sondern auf den alert Dialog. Deswegen sind die Werte null.


Definitely connection based. With all your kernel stuff and queueing and application management it's pretty much up to you. I won't put a single line into that, since this is not related to the tcp/ip framework and software collection


> On 07 May 2015, at 17:33, Gunnar Morling <notifications@github.com> wrote:
>
> AFAIK you still need to add the manifest entry to get the OGM modules, so you need to opt in by that.
>
>

Today yes. But what do we want for our users. Not necessarily to fiddle with these manifests.


Yeah, the bower config on the master branch causes bower to get confused and ask for help. I've been working on the events-refactor branch for the last month, and it includes an upgrade to angular v1.13.15. I believe I've gotten the config set up so that it installs smoothly but have yet to double check it. I'll be merging events-refactor back into master later this weekend.


The idea of this was to prevent accidental access to non "test" indexes.   Do we really need to make this more permissive?  I'd really like to keep this property.


looks good.


Your correct, thanks!


Is it post_info or poll_info?


Does this compile? I thought Base64 was new in Java 8.


@tvsonar These lines are not tested, and I would argue that you only need to test the method's name: It's misleading to return null from a "ToString" method, and it's also misleading to have multiple "ToString" methods.


Oh never mind. This is the android one.


Да, именно так, и я пока не поняла, почему.


Oops -- I missed fixing that one, yes, I'll fix it right now...


Hopefully better now.


I am fucking retarded.


"id CHAR(12)" should be "npi CHAR(12)" or the "primary key (npi)" should change to "primary key (id)"


We can remove the branch then?


Add comment for the last argument:
```java
byte[] decodedByte = Base64.decode(dataUrl.substring(contentStartIndex), 0 /* flags */);


script.src = require.toUrl('dist.pep.js');


Also moved (un)license information to homepage.


Ммм... У меня long_bed была логической переменной, смысл немного другой. Это не та константа, которая может измениться снаружи. Впрочем, подумаю.


Same here:
```java
icon = BitmapFactory.decodeByteArray(decodedByte, 0 /* offset */, decodedByte.length);


Much appreciated. Thanks for your time.

-b
On May 7, 2015 9:16 AM, "Joshua McMichael" <notifications@github.com> wrote:

> Yeah, the bower config on the master branch causes bower to get confused
> and ask for help. I've been working on the events-refactor branch for the
> last month, and it includes an upgrade to angular v1.13.15. I believe I've
> gotten the config set up so that it installs smoothly but have yet to
> double check it. I'll be merging events-refactor back into master later
> this weekend.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/genome/civic-client/commit/83c6be72dc2da65cc20bc91a2accc31f19caf648#commitcomment-11090864>
> .
>


Hi @theoryshaw, I just did. Please let me know if you have any questions.


You are correct.  I let the different bench of your version confuse me but that is for a different reason.  I have updated the patch to remove the refresh.  Thanks!


I don't see any direct import of Jackson.  Shouldn't these be transient dependencies, left for Maven to infer from com.amazonaws:aws-java-sdk-s3:pom?


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


Thats right. I change the name of Const to DataType.CHAR.

So if you want to use the character-Obejct use: DataType.ref("Character")


Authorship?
2015/04/27:
https://github.com/bauner/cm-android_device_samsung_galaxys2-common/commit/eaf0b5ff0a7269c69960f9b911542d545d5e0a18


Can you provide some examples that this solves?

Also, this isn't a case type. If we want to support acronyms, then each method should be able to handle them as well.


@amirroth So something is still definitely going on.  Here is the space 1-1 sensible cooling rate between develop and this resorted surfaces branch on the UnitVent5Zone.idf file:

![image](https://cloud.githubusercontent.com/assets/849824/7520297/de89fc86-f4a2-11e4-803a-07ad65055e83.png)

Do you want me to dig into that routine and see what's up?  I might step back a bit and use a simpler shoebox model as it might be easier to debug.  However, given that this has to do with zone pointers, you might need at least 2 zones to see the difference.  Anyway, I'll wait for you to give me the green light to spend time on it, just let me know.  <oncall>


Что касается функций read_tsv и write_tsv: с комментарием согласна, но пока я не определилась, могут ли они быть полезны извне. Когда определюсь, наверное, изменю названия.


@tvsonar there should be a case to testing the "Write" method, and another one testing another method (such as ReadLine)


Sjekk ut denna @I-Simen-I  jeeeeeeee.


Hi,great man. Can this governor work for M8? or how to port to m8?


Remove `can and` double word in "...and can and set the maximum discount as..."


Супер, може ли да го шипнеш на Master-a да си го взема :)


is this "fast" maybe a typo?


We have both `LOG_DEBUG` which accepts printf-style arguments and `LOGF_DEBUG` with boost::format-like arguments.


why `c_str()`?


This edit ought to be left out or reverted, if this work is submitted for inclusion in stock DSpace.


b97123437914297b3485d916652666b6379aa70a : Fixed, thank you!


These should be commented in stock DSpace.


Authorship:
http://review.cyanogenmod.org/#/c/95656/


Added , checked you can add a room number when yo create the appointment


-    to_install = list(set(candidates).difference(set(existing_minions)))

The bug in this line was that candidates and existing_minions weren't of the same type. Specifically, candidates was a set of IDs and existing_minions was a list of EC2 instance objects. Also get_minions was also passing a list of EC2 instance objects to the filter instead of IDs which also caused the method to return an empty list.

Bottom line is that this bug would cause the fab task to install salt on *all* instances, not just the ones without salt installed.


No. The icons not showing is probably a configuration problem in IE cause by your IT department. (Font icon download disabled)


shen me a


This change breaks core on at least base-nova, e.g. it will not start.


missing newline


The weird thing is `to_review` already should exclude missing topics. If it stops the errors though I am cool with it.


Yes, thank you :-)


Ok, look good now


I'm already trying to suggest a PR for jQuery after this - http://stackoverflow.com/a/30089307/274502


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2133 of 2209 tests passed)

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


Can't this be accomplished by just @with_config? https://github.com/spotify/luigi/blob/master/test/helpers.py#L33


Super ! :)

On Thu, 7 May 2015 at 16:54 GabPoll <notifications@github.com> wrote:

> BTW - looks beautiful! Just what i hoped for
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openhealthcare/opal/commit/6289747de29796b7fb17da88bc2cd9357b1d3b50#commitcomment-11090429>
> .
>


nit: s/trigerred/triggered/


Oh, I thought that this was really in `lesson-template`and made a [PR there](https://github.com/swcarpentry/lesson-template/pull/235)

In any case, I think it's better to have 0 as min, instead of 1. In case the lesson needs no extra files. Or maybe, we can leave it like this and have a default: "You're ready to go!"


This dependency breaks arm/stm32_vl build


Looks good.


Is there a reason for having the slash-dot outside the single quotes? i.e. Why not:

```
io.shell("cp -r '%s/.' '%s'" % (repo, dest))
```


ngrok ?


that was my fault, not a clue of how that even happened.


To stay consistent with the interface, could you rename `handle` to `state`?


[This line](https://github.com/symphonycms/symphony-2/commit/d361f366b66be0b45194e2acd15c6ed8589c2408#diff-f3ff2ff6810310452f63883fac0df71fR99)

```php
cleanup_session_cookies();
```


This should probably continue to be "abstract" since you never want to instantiate it directly.