Any optimizer would have performed the multiplication at compile time, so this won't make a difference in speed.  The style may be better though.


space before `(`


@night Any idea why it's doing this?


`seq` and `sig` are exactly the same class, simply with a different contained class.  Why not make them one generic class?


@sameersbn thanks for letting me know!


why does uniq have false as its first param? specify is sorted?

and then filter (return item.pictures.length !== 0 was so because i dont want tags with no pictures. why did u just filter by length? is that "if length is 1 or greater, its true, so filter"?


![scan0003 jpg](https://cloud.githubusercontent.com/assets/11039724/8165314/142d0508-13ad-11e5-9281-e9e9a1909120.jpg)


This doesn't seem complete to allow using "a.b" notation to define/show columns.  Are there further associated changes?  The columns in the columnProperties are still defined using _.keys, which ignores nested values.


Why are you changing this back?


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 552](http://build.prxm.it:8811/viewLog.html?buildId=3037&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 284 Build time: 00:11:31


OOPS.  :-D


This is in scalacheck/master now.


Still got the old link in there.


This change adds a proxy at:

http://akira.w3.org:8888/wot/switch12

This breaks demo.js for me since I cannot connect to that.  Is this intentional?


I'll have to update this plugin again, I'm sure, anyway I think is a too little change to force a global update.
Is so annoying this warning?


Usually formatted with spaces as such:
```
if (redPos >= 72) {
```


@xuantain ive been really busy recently but maybe i could update the tests for you next weekend?


This is a large method with procedural logic that may be able to be broken out into several private methods with descriptive names.


This test is failing with a timeout, and now that it's release-critical, channel updates are failing: http://hydra.nixos.org/build/23107185


Should be `if (location.state && location.state.nextPathname) {`.


```cxx
GTEST_ASSERT_EQ(expected, actual)
```
i'd admit that we are not always very consistent in this aspect...


Resolve to actual response data.


Consider putting this in a utility type factory.


Ah right, good point


Strange, don't remember I have modified this


@stomar Thanks for the heads up. :+1: Noted. Will remove in the future.


@ksingh7 yeah we can update the build-deps for cepnt7 zeromq3-devel. You'd be changing this file https://github.com/ceph/calamari/blob/master/vagrant/centos-build/salt/roots/build_deps.sls

Take a look at https://github.com/ceph/calamari/blob/wip-f21-devmode/vagrant/devmode/salt/roots/build_deps.sls for an example of how to differentiate between cent6 and 7


Thanks. Fixed in https://github.com/cloudfoundry-incubator/routing-api/commit/eae3b9ccb05dd591fcd86d400b82e990dceb87a.


Thanks for this, but I'm still having trouble with defaults when using a JSON document. If I specify a value as being None in JSON on the client side, the soft validation fails. The only way around this for me is to not set the attribute on the client side if the value is None, which results in an awful lot of if statements.


Yes, it's a broken tests because I added extra parameters to grub install
that need to be fixed for grub1

On Mon, Jun 15, 2015, 09:55 Daniel Fox Franke <notifications@github.com>
wrote:

> This test is failing with a timeout, and now that it's release-critical,
> channel updates are failing: http://hydra.nixos.org/build/23107185
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/cd28c5517c224c0c969f6bccb1f85dcc280ed7f2#commitcomment-11684717>
> .
>


Might want to use the $scope.activeEditId.


Wouldn't it make more sense to set these members in the definition of the cb variable? That should allow the compiler to resolve them at compile time and add them to the data segment, making the code more efficient.


Yeah -- since these variables are only used in mouseClicked, then it's better for Java garbage collection and code design to declare them here (rather than as static class vars)


could also use `mkstemp(3)`


Feels like this would be very appropriate as a directive.


Ur a god @Paul19988


Please don't default switches that work on enums and handle _all_ cases. Without a default statement we get a nice compile-time warning when that enum changes. The `default:` suppresses that warning.


Nice commit!!


@vouillon can you confirm the fix.
before the fix, we had `string_of_float 3e100 =  "3e+1"`


See comment about `default:` above.


It is not percentage, so it is just MAX_INT MIN_INT


Lots of these ui-sref=adminHome(storeId) things. This is a good opportunity to possibly try creating an attribute directive.


I believe this broke staging: http://hydra.nixos.org/build/22516199/nixlog/1/raw . Hope it's temporary but builds need to be restarted.


!== undefined necessary?


(btw, it currently needs snarfed/webutil#2 for the tests to run. not sure what the actually right fix is yet. meh.)


Thanks Max. When you get a chance, can you throw into 1. client >> html and 1. client >> css? Thanks.


Ah I see, thanks for checking it out. Didn't think of that. There was some problem with that line which removed the wrong number of characters according to the test. I'll have another look


it should do on every push to master. Translation strings should be fully synched with master. So I see the only conditions:

- Only do it on master.
- Only do it once per push.


@sergiogonzalez, I used a wrong import (`com.sap.dbtech.util.StringUtil`, damn you Idea) in the tests. Please close it, I'll fix that and resend again.


Args should rather be splitted all the way (I guess I was reaching the hard limit of 120 chars here)


This conditional expression should be on one line because it's short.  But if you did have to break the line the correct style is with two more levels of indentation as in:
```Lua
if foo
		and bar then
	...
end
```
See http://dev.minetest.net/Lua_code_style_guidelines


This broke building the bootstrap glibc


I'd probably just check for `.then` to be compatible with other Promise implementations.


Hi sir help me to get IP address so that I will get my high level if clash of clans thank you


Ah - fair enough.


This spelling wordlist is a mess but please use ``` ``cURL`` ``` instead of adding this to the dictionary. The way I think of it is "mmm, this should really be in the dictionary, so I'll add it to the wordlist"


It is best practice to always format if/else as so:
```
if (thing) {
    doStuff();
} else if (otherThing) {
    doOtherStuff();
}
```
Particularly -- the else cannot become detached from the preceding `if` block if it's written `} else {` in one line. This is mainly a concern for collaboratively edited code.


jj


package com.zombiefortress.server.Object.Tile;
+
+public class TileWindow extends TileObject{
+
+ public TileWindow(int X, int Y) {
+  super(X, Y, 4);
+ }
+


@adin oops, sorry about that! This diff was sync'd from an internal repo, and apparently this version of the sync script doesn't convert Facebook's internal unixnames into GitHub usernames in commit messages when pushing them out.


Move this code into partial


@jfoug thanks for handling this! :+1:


I'd break this out into a private method, `private void handleGameBoardClick(int x, int y)` or something.


Move into partial


Something (prob eclipse git client) ruined the project structure and rolled us back about 25 commits, this is an attempt to fix that, please back up locally before doing anything.

Also make sure to sync your local files before committing anything!


Should there also be respondResourceUpdated() and respondResourceDeleted()?


The outer lock is unconditionally unlocked before it gets here.  Here:
https://github.com/rec/rippled/blob/06f3a73197a1a5b2eb41d84c8b1af7dd4a55aa69/src/ripple/app/misc/SHAMapStoreImp.cpp#L291


Use rails' methods for convert string format


You don't need to manually wrap here. Can just:

```js
return maybePromise.then(() => {
  return undefined;
})
```

And for the `catch`, I'd continue with:

```js
.catch((err) => {
  fluxEmitter.emit('error', err);
  throw err;
});
```

That way, we can log the error if we want. Otherwise, it will (hopefully) be picked up by whatever mechanism (i.e., chrome) is picking up uncaught errors.


This line should be:

```ruby
favorite = @user.favorites.create(post: associated_post)
```

We just want to make sure that ```favorited``` wont return the other post that the user favorited. And in order to do that, we need to set up another post.


If locally installed versions of jade (or other dependencies) differ, this happens. It's not a big deal and can be ignored.


originally I had it as ```+ 2:```.  That may be the actual correct splice out...


I've removed the repeated attempts to create the bundle file since I don't think its needed at all now. I spotted a rather large bug in our `deleteDirectoryFromBottomUp` code which wasn't deleting populated directories properly which may well be the root cause of this and is probably the reason why we had `FileUtils.deleteQuietly()` everywhere.


This looks like if the click is somewhere else, then this entire method is a no-op (no-operation), and just stops.
Instead of wrapping this entire method in this if(), try this:
```
// Is the click on the game board (and is the game still alive)?
boolean isValidClick = e.getX() >= 40 && e.getX() <= 510 && e.getY() >= 40 && e.getY() <= 510 && !gameOver;
if (!isValidClick) {
    return
}
```
This is an early return that makes sense at the very top of methods, and breaks the general rule that multiple return statements are undesirable.


It fails despite not setting `nullable=True` or is there any other reason?


@sahrens where is this linter.js file?


Only methods inside classes start with `self`.


ack


This method is only for when you're in a class based view. Not in a plain view function.


Output error to global as details are lost this is not done


Naming conventions for these would suggest using `isGameOver` and `isRedsTurn` or something like that -- the "is" helps developers see the code below and know that it's a boolean.


I suggest you look around at other view **functions** in that `views.py` and see how they look like and how they take the `slug` and use that to fetch the `event`.


team/photos/Siddharth Jain_Bharati Vidyapeeth's College Of Engineering


camelCased::snake_cased -- seems inconsistent


Some servers are not returning an image ID


The 12 number will change. Why not leave this just looking for Edge?


Actually, the correct indices are probably something like ```m.start() + 1 + len(m.group(2)) + le + 1:```


Don't you mean; "NOT Event.PRIVACY_COMPANY"?
I.e. if you're a contributor you can see all events **except** those that are `PRIVACY_COMPANY`.


acl


missing newline before EOF


Nice pics dude
On 15 Jun 2015 22:43, "Siddharth2" <notifications@github.com> wrote:

> [image: 11034373_344033552457113_3887244712741802707_o]
> <https://cloud.githubusercontent.com/assets/11039724/8165649/d1234f44-13af-11e5-8ea1-f3d80d9107c3.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/5harad/crowds/commit/3051e1011ea5e10fb262c6d4fc65866e2b5d1e75#commitcomment-11685041>
> .
>


There's a big difference between the string `"Event.PRIVACY_PUBLIC"` and the variable value `Event.PRIVACY_PUBLIC` :)


Se decidió consultar en el show de Person si el campo updated_by corresponde a un usuario que no haya sido eliminado, ya que no se puede actualizar el campo updated_by de Person como 0 o NULL porque el campo se definió como requerido en la migration.


I'm not wild about overloads as a matter of style - I'm a fan of them if you're going to break an existing interface by adding a default, but I don't want to break `track("Event", null)` everywhere (which is a common use case).


Please squash with commit 75fdaa.


:clap: :bow:


In `minetest.conf.example` comments are supposed to be **above** the settings they document -- same as the C++ and Lua style.


can't you check here if you're on circle?


Applied to trunk for 9.0.x


You could also be transparent (in addition to explicit) about image pixel coordinates by prepending the region command with "image; ", i.e.
regions.append('image; ellipse %g %g %g %g %g%s' % ...

If you prefer to keep the "i" suffixes, please squash commits 60b9b35 and 1b9b5e1


I'd prefer `Submitting coverage data...`


or so


Actually, I see a kind of scenario here which would need a change in expected behavior of usbg_rm_gadget.  To remove gadget directory, it needs to be empty (e.g. we can't remove just the gadget directory while maintaining configs or functions directory), means it has to be recursively removed anyways. Hence USBG_RM_RECURSE should be the only option here and gadget directory always removed recursively. I am leaving this decision for open discussion as of now while I explore more.
This fix just makes the exit in case of opts=0 more graceful.


My configuration for the attribute 'dimension' is:
('dimension',        Unicode(min_occurs=1, default='dimensionless', nillable=True)),

On the client side, I say: {'dimension': None} (simplified)

The resulting error is: Client.ValidationError:The value "('dimension', None)" could not be validated.

Have I missed something?


Please squash with 88104bd


did a quick check with ipython and it should be ```m.start() + 1 + len(m.group(2)) + le```


fsdf


ack


and remove it from travis, right?


typo


what exactly is the owner?


Gulp tasks is missing });


Using an X-Y coordinate system on these class variables may make some of the logic below a bit more straightforward. Java provides a `Point` class that may be useful:

http://docs.oracle.com/javase/7/docs/api/java/awt/Point.html


This is too difficult to maintain. I would do something like that:
```js
var disFun = distance[sim];
if (!disFun)
  throw new TypeError('Invalid distance function');
```


Nice use of `compact` for readability of code.


No else.

This is now about doing that second level of filtering. That should be combined with the query on the IDs.
I don't want to reveal exactly how you should do it but you can find the answer by looking at other places where it uses `privacy_filter` and `privacy_exclude`.


I didn't know. :) What can drive this change considering Edge has no versions planned?

I'll change it to `/edge\//i`. This is just for unit tests, though, so it doesn't matter too much.


This could be replaced with simply:
```Lua
if minetest.get_item_group(minetest.get_node(
		{x = pos.x, y = pos.y - 1, z = pos.z}).name, "soil") == 0 then
	return
end
```


Thanks Iain!

On June 15, 2015 at 10:19:47 AM, Iain Diamond (notifications@github.com) wrote:

Gulp tasks is missing });

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


It is arguable whether or not this code is cleaner. The changes to the line breaks make it harder rather than easier to read. Because of this I'm not going to apply this patch as is. I'll apply the (factory != null) part but not the rest.


could use the `classNames` module here (`import classNames from 'classnames';`)

something like `<div className={classNames('grid__cell u-padding-Vxl u-padding-Rxs', {'u-size-1of2': hasPercentagePricing})}>`


very good


nitpick: by avoiding curlies you make the code more susceptible to bugs in future changes because it's too easy to miss that curlies need to be added if another statement is needed in this block.


Ref:
* [ruby on rails - ActionController::InvalidAuthenticityToken - Stack Overflow](http://stackoverflow.com/questions/3364492/actioncontrollerinvalidauthenticitytoken)


"so" what? :)


This looks like a docstring -- there are particular JavaDoc standard ways of placing them.
```
/**
 * Docstring Title goes here.
 * Longer description here.
 * More lines, and so on.
 **/
public class BreadthFirstSearcher {
```
Classes are usually named as nouns. In this case, "BreadthFirstSearcher"


I realize the use of `tribe_event_format_date()` predates this change - but is that the best function to use here? Given users may set up a filter on the `tribe_event_formatted_date` hook, might a direct call to `date_i18n()` be a better choice where we are setting query vars?


In the future, we should probably use language files, e.g.
`return Lang::get('ayah_api.resource_added', ['name' => $this->repository->getResoureName()]);`

along with a corresponding language file along the lines of:
```
<?php

return array(
    'resource_added'' => 'New :name successfully added.'
);
```


don't you want to reset the flag in the `clear` method?


I deleted all my node_modules from my AppData folder and ran npm install and it uses the version in package.json.

Can I still commit even if it changes that? Are you cool with that?


They could live in another branch (they are not related with subscriptions in
any way), but we are testing them in the same testbed with a subscriptions
enabled server application.

If we split the branch, I will end up merging them before every testrun in the
coming days.

On Monday 15 June 2015 18:20:16 Sten Gr�ner wrote:
> Do method calls need subscriptions? Or can they live in anorher branch?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/acplt/open62541/commit/4717f4f3057a25f31df6ab79b3ee2117aa
> 8a3c37#commitcomment-11683983


Good idea.


use single quotes


bird6: Enforce version from backports.


this should start with lowercase like the other controller methods


Please, don't review this commit. It is part of another PR.


missing a " at the end of line 46 fails to run.


Yes, I am interested in the support of building:part and height.
Please note, there are the lines in the begining of io_import_scene_osm.py:
# This is the release version of the plugin file io_import_scene_osm_dev.py
# If you would like to make edits, make them in the file io_import_scene_osm_dev.py and the other related modules
# To create the release version of io_import_scene_osm_dev.py, executed:
# python plugin_builder.py io_import_scene_osm_dev.py:

I'd suggest the following:
I'll convert tabs to spaces in all related files. Then you resubmit your patch. All handlers (e.g. BuildingsParts and Natural) should go in separate files.
Ok?


Interesting. Could you please distill it to a testcase? `nillable=true` should mean soft validator passes null values.


I want tis to be published on my computer


The issue here is: we will never know which name the folder will have ...


how do you know that it wasn't called? just because done wasn't called? that seems very brittle. could you use a spy instead?


remove it from the private array too then
And the comment


Don't you need to click on the <option> created for SF. I feel this test would pass even without your changes.


You can't return a tem* error code this late. Whether or not a transaction is malformed can't hinge on, for example, the amount it delivers. If you want to validate the DeliverMin field and return a tem* code if it's not valid, you need to do it in preCheck. (This can break consensus because some servers can cache that the transaction is malformed just because they executed it against an open ledger at a time when it failed this check.)


Some old packages don't have a `depends` keys, but this is only for historic reasons.  The `depends` key is required nowadays.  Anyway, thanks for the fix, it makes things more robust.


Nice one, that's super useful - thanks!


As stated, when there's only a few lines changed it's not a big deal.


tecPATH_PARTIAL is better. The path(s) could deliver some funds, but not enough.


@AzGoalie thanks for updating the parameterized logging


@jakemac53 Is this code still needed with the new PR ?


No reason for bracket notation for layers and filling. Should be .layers, .filling etc


This is a clean and straightforward controller but it is quite repetitive. All we're really doing is:
```
public function fooFromBar($barId=null)
{
    return $this->extractCollectionFromResource('foo', 'bar', $barId);
}
```

Can we just implement one function that is configuration driven like this:
```
$sub_resources = [
    [
        'owned_resource' => 'gamepools',
        'owning_resource' => 'game'
    ]
];
```
(ehh, this is a rough draft but the idea is to read the above config and automatically implement a `gamepoolsFromGame($id)` call based on it)


We do know by querying 'xdg-user-dir <DIR>' as defined in /etc/xdg/user-dirs.{conf,defaults}
I thought, if it is in Desktop dir in /etc/.skel/, it will be copied on user creation at runtime?


That makes sense. I'd whipped this together very specifically to one platform, so steps away from that are a good thing!


initial commit


I know there would be some minor considerations around pluralization, e.g. we are only asking about a specific Game but the table is called Games, but I don't think that this should stop us from seeking a config-driven approach.


@masteinhauser @haad @sethvargo - What do you think of this approach for issue https://github.com/mitchellh/packer/issues/590 ? Too hacky? Full story here - http://blog.session.it/2015/06/15/berkshelf-and-packer/


_perror_fatal ("execp: ");

:+1: with this change


FWIW, Chrome 45 is in the wild and the UA follows the same pattern as <=44.

Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2427.7 Safari/537.36


Don't want to be a codestyle nazi but single and double quotes on the same line :(


switch statement?


Deleted contacts-details class


for example...

      $ xdg-user-dir DESKTOP
      /home/devtux/Schreibtisch


This doesn't seem to do anything if no cake is specified?


uniq has three parameters, when you chain you only fill the first. You need
to fill out the isSorted parameter, right?

In JavaScript and a few other languages, some values act as true/false even
though they aren't booleans. 0 will act as false. Other numbers are true.
So saying !== 0 is unnecessary.

On Mon, Jun 15, 2015, 9:53 AM marcoajonker <notifications@github.com> wrote:

> why does uniq have false as its first param? specify is sorted?
>
> and then filter (return item.pictures.length !== 0 was so because i dont
> want tags with no pictures. why did u just filter by length? is that "if
> length is 1 or greater, its true, so filter"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saiichihashimoto/poptags/commit/3c5034594928b9a303fe8dee6b21cdefa2fc17ad#commitcomment-11684664>
> .
>


I'm quite sure. usb-KINGSTON_USB_DRIVE_0118488030033-0:0 is my disk and usb-KINGSTON_USB_DRIVE_0118488030033-0:0-part1 is my partition.

http://ubuntuforums.org/showthread.php?t=1528529 is the thread that suggest to me that reformatting my disk may solve this problem.


ok, sounds good.

What is the easiest way to test edited code; is it necessary to rebuild the plugin and then reimport it into blender, or is there an easier way?


Also, I noticed that the buildings and highways code is added twice to the final file.


There's no reason to do this transformation here.  System calls can take three register-based arguments, which exactly matches the arity of this function.  I think you can get rid of `__upcall` and just use the normal wrapper `cos_syscall_3`.


callback? NOOOOOOOOOOOO!!!!


If unused, it should be removed.


Pound isn't illegal. Dollar is a common token meaning 'variable'.


Much better!


rootScope? There is likely a better way.


should have been a relative link


no hay drama :)


Welcome KSV


same


wait 5 min second commit


Or 1.580.1.


It now looks like new problem page :
![image](https://cloud.githubusercontent.com/assets/2932687/8166271/0274607e-1397-11e5-9630-b0fba7aa8655.png)


It isn't obvious to me that maintaining the old logic around the "recovery thread" is the correct thing here, but I don't have any better ideas for the time being.


FYI, I would not recommend use of this library, due to [JENKINS-25625](https://issues.jenkins-ci.org/browse/JENKINS-25625).


Eh, not sure why I thought it was :S Sorry for wasting your tiem :P


I normally test it with development version io_import_scene_osm_dev.py version and then update the release version.
If you need convinient debugging please see my notes here: https://github.com/vvoovv/prokitektura-blender/wiki/Developement-Enviroment


Sweet, thanks! I'll give that a go


This lock is not within `mCompleteLock`'s scope.  However it is within the scope of `sl(m_mutex)`:

https://github.com/rec/rippled/blob/06f3a73197a1a5b2eb41d84c8b1af7dd4a55aa69/src/ripple/app/ledger/impl/LedgerMaster.cpp#L1014

That is, this looks like a useless recursive lock of `m_mutex` to me.  I think we should remove it.


Nice example!


blz


Shouldn't this test to validate the that operation is DESTROY, and that the argument is passed correctly?


It is to emphasize that this function does not return a magic value (opposite to 373 line)


ok that's a hard habit xd


Please add javadoc indicating that small does not necessarily need to be equal or smaller than big, or change names to something smarter.


For posterity, I change my partition table from

    Device     Boot Start    End Sectors   Size Id Type
    /dev/sdb1  *       32 250623  250592 122.4M  6 FAT16

to

    Device     Boot Start    End Sectors  Size Id Type
    /dev/sdb1  *     2048 251903  249856  122M  6 FAT16

and now GRUB 2 installs without error.


:heart:


Drobna sprawa: nie psujmy innym statystyk :laughing:


Thanks @nemobis . The Wiki Ed logo is correct; we abbreviate the organization as either Wiki Ed or Wiki EDU ( eee dee yew), but that's the only logo. We'll update the README at some point, but we'll probably rename the whole thing soon-ish, as it is evolving into more than just a dashboard.


@MaruthiPrasadA Please do not post under commit threads unless they are directly related to the code that was committed. Please post all issues related to the project here:https://github.com/bchavez/Coinbase/issues


magic numbers are not cool, here we could use `ldb.ERR_NO_SUCH_ATTRIBUTE`


Awesome!


great!


I've tested this patch as requested in 87b9b5778caf57f957028574a7665318966a17f9

I've tested it with Mario64 and Conker. Both seem to work fine


:tada:


These for me seems like spurious tabs.. why are these tabs necessary? can you explain further


Totally unnecessary. I dunno how that happened.


Tested it with Mario64 and Conkers. Both seem to work fine


this looks so much better :+1:


Chyba warto to usunąć, nie psujmy innym statystyk (id skopiowane z cs231n.github.io)


Deveria funcionar. Aliás, segue o log do travis do projeto rodando no saucelabs com a mesma solução, perfeito:

https://travis-ci.org/flaviohenriquealmeida/mean-livro-cap-12-com-saucelabs

Talvez seu problema seja outro e para que eu possa ajudá-lo preciso do seu link do seu repositório e o log ao rodar o teste.

Temos um fórum no Github que também pode ajudá-lo a descobrir o que está acontecendo:

https://www.facebook.com/groups/1534163683531297/

Antes de qualquer coisa, tenha certeza de que usou as versões dos módulos que indico no livro, inclusive do protractor. Não tenho controle sobre as novas versões, aliás, não migro toda hora que uma nova sai.

Aguardo seu feedback.


Since the parents of this node are just containers, it would seem that you won't be able to commit a configuration to the router without configuring the AS number. Is this intentional? Am I misinterpreting the YANG (I'm trying to interpret in my head) or would this block me if I want to configure a router without BGP?


This function doesn't return anything.


Suspect that ```|| big == Double.NEGATIVE_INFINITY)``` could simply be removed here ... if you are adding up 0 + X the result is X. So return big would always work.


@maoo Interesting solution you came up with. Though, this would have still failed in the scenario I was originally working in. I needed to handle environment differences on user systems where the wrapper scripts were run, and between my local system and our CI/CD tools.

Personally, I still would not use this and feel that it couples the packer template too intimately with the execution environment packer is run in. However, it does also help provide a *full* template to an end-user whereas my templates without the wrapper scripts cannot rebuild the entire artifact.


Yay! Great error handling.


These styles can be removed since we are not using blanket anymore.

My previous comment about styles was regarding the testing container div but you have addressed this conversation in the sl-ember-test-helpers repository.


why is this 2 if the goal is to blur 1px?
tiny detail: antialiasing should depend on the devicePixelRatio since it should blur 1 screen pixel, not 1 standard pixel unit. Line antialiasing: https://github.com/mapbox/mapbox-gl-js/blob/06eece43a88141c281b7664cc225ed7753dbfb98/js/render/draw_line.js#L29


It's a common case because we didn't allow a way to track an event without
properties before ;). We can discuss this more when the time comes.

On Mon, Jun 15, 2015 at 10:15 AM, Joe Bowers <notifications@github.com>
wrote:

> I'm not wild about overloads as a matter of style - I'm a fan of them if
> you're going to break an existing interface by adding a default, but I
> don't want to break track("Event", null) everywhere (which is a common
> use case).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mixpanel/mixpanel-android/commit/2802e7178d550eaddb970b7b82efeed06571de38#commitcomment-11685087>
> .
>


Yes!  Lets keep ``backend_managers`` clean, see comment in the main MEP27 thread.


You can use a .map here.


See https://github.com/mapbox/mapbox-gl-js/commit/6d9ddefc960a2c4cc227fab4e33945b54a92af67 - already fixed this


## This is a markdown file


Broken, obviously.


Was `getc` supposed to be removed?  It looks like this change is failing master now.


How do u download this mod


This is available in fsa-prebuilt


Repository of elite committed crimes in Hungary since 1990


On Fri, Jun 12, 2015 at 07:18:37PM -0700, daurnimator wrote:
> I'm surprised you added in `POLLRDBAND` and `POLLWRBAND` (when yielding
> via number). Any reason you added that but not `POLLRDNORM` and
> `POLLWRNORM`?

I realized that I should add a macro for the bitmask rather than
copy+pasting the entire expression to multiple locations. So I just
committed what was there.


Changed to ```this(new Random())```


Great file! Consider using a variable to prefix all the urls (instead of /api/blends over and over).


Cherry-picked that onto maint.  Sigh.  That was clearly a commit for maint.


[Deployment available online](https://s.brightspace.com/lib/vui/dev/61bffe2f25acc318f0679ee733d4a9a59b1e0b2a/valenceui.css)


What is User here??


Write instead ```Utils.nonNull(rnd,"the random number generator cannot be null")```


how do i run bot?????


it's preferable to log function calls, so that you can detect when functions are called more than once or in wrong order.

I usually just use an array I push logs into or have a test logger that can  return an array of logs.


Pls commit


What is this used for? Also, what kind of attribute is `type`?


Ok, well not everybody likes to have it on the desktop on installed systems ...


I meant #1526.


I think this should be `data-type`.


This should come from the server I think, just in case new levels are introduced.


That's fixed as well. Thank you so much for your help !


"Open Source" :D


Should this be WriteExecutor?


as


I would substitute all the calls to this method to ```String.repeat(s,nCopies)``` directly; alterantively do proper argument checking with ```Utils.nonNull``` and making sure nCopies is not a negative value. Otherwise what is the value added by having this method in utils?


if you call done here and there are more callbacks to be executed, you'll get weird results because the callbacks might get executed during the next test execution.

it would be better to write the test in a way where async.done() is called from a place where the whole system is at rest and we have a guarantee that no more future work is pending.


I guess this should actually be 1.1.3.1


Whoops, I must have forgotten to sync before I pushed my other change. Fixed now.


This does not work the way you except. `.login` is being called at the same time as `.createUser`.


`_query0` -> `_query1` ?


This is the only section I really changed.

Pull request left active to allow you to see the actual Blizzy test and handling code.

You can close the pull if you desire.


`_query0` -> `_query2` ?


Hi,
That's my fault. I have totally missed the fact that the init method blocks until a joystick is detected.
The reason why I've moved the init method to the constructor was because I needed the joystick id before loading the settings.
You are right that the joystick won't be initialized if the dialog has loaded and it's even worse the joystick won't be initialized if not plugged in before application start because I've moved the init to the constructor.

TODO for me:
- move the init method back to run and load the settings if a new joystick is initialized. Update the UI whenever a new joystick is detected.
- Think about possible test cases and test or even better provide unit tests

Sorry for this faulty commit. I will provide a fix within the next few hours.


There is a ```Strings.repeat(char,int)``` that does the job. Thus the comment above on dupString(String,int) applies here as well.


Why do you have a ng-click if it does not do anything?


`response` is misnamed. It is not the HTTP response.


No worries, I am gonna fix it with the inline editor, as I am too lazy to invoke all this git command ;)


Do it, if you want.


OK.


just a thought, because the guys on IRC pointed out that this folder is not localized on livecd.
I am not sure if cli-installer desktop file is in localized folder.


delete scriptTSE2


Should we check for numeric values here?


It totally should! Will fix it now.


Consider ng-src so your HTML doesn't request a pre-interpolation value.


:cat:


should we cast the 'page' query param as an integer here to ensure we always return a numeric value?


The html shouldn't be all uppercase. We should use the css text-transform uppercase to make the text display in all caps. This is for accesbility reasons. Also, if the page gets shared on facebook or google, I don't think we want this content all uppercase there either


good change.   i'll merge.


This `$state.go` flies in the face of the `$scope.listing` assignment above it.


@MichaelCurrie Yes, basically, I am just going from contour to skeleton. I am not really introducing anything new. I am just taking the functions from SegWorm, reorganising them and translate them into Python and Cython. The code is quite complicated and introduces a lot of redundancy, but I think (I didn't test it) that this might be useful to deal with difficult cases. I like the idea to compare it with the pre-features code.
Do you have an example on how to go from a contour to a skeleton using @JimHokanson code? I could test it in the same data test (or even generate more examples) and compare what are the best features of each code.


fragro, I don't know if you have seen the story about Colorado letting employers fire medical cannabis users.   Maybe the amendment should prohibit being fired for medical cannabis use.


Using an event here ... interesting.


JQuery is totally fine, but if we don't really use it, then let's not bloat up the site.

See: http://youmightnotneedjquery.com/


Maybe not. Please check and adopt if needed ...


@alecive remember 1.0 != 1


You could alternatively get all blends in a resolve and then just return the result of that here (this won't work if all blends changes while the person is on the page)


link tags should go in head


This should be .cssFloat rather than .float... https://developer.mozilla.org/en-US/docs/Web/CSS/float#Summary


@alecive use [] in place of ()


the script tags should go after all of your content in the body, it makes the page load faster


it makes the page load faster


@alecive use `lenght()` in place of `size()`


All tests passing.


should be total not totla


But then I'd have to make sure that all the tests have their indices set to 1, in the case where I don't need to duplicate anything.


@alecive don't use `{ }` for one instruction


When we get a change, we should add meta description and favicon tags.

See http://realfavicongenerator.net/
this is a great resource


$viewValue? Whaaaa?


We should use some free webfont for mooc house i guess...


I just had some small nitpicks, the great work overall!


Hello, please see my update. I used inserted sort into hash for priorities, I also changed the r-uri matching value.


Hey Roger! I'll swap out the css when I get home from work tonight. As for
JQuery, I was using it to attempt form verification, but I ended up not
using it. I will take out the link. Also, I noticed the link tags, and I
put them in the head of a newer file version (I'm not certain I pushed it
to github, so I can check on that tonight). Finally, I did put the script
after all page content (I think this probably relates to which version you
are viewing).

I'll confirm these edits and push the newest version to github.

On Mon, Jun 15, 2015 at 2:07 PM, Roger Zurawicki <notifications@github.com>
wrote:

> JQuery is totally fine, but if we don't really use it, then let's not
> bloat up the site.
>
> See: http://youmightnotneedjquery.com/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackharvard/H2Website/commit/fa31191274546bb52c00a9e480da755404c742d3#commitcomment-11685968>
> .
>


Why not just `formData`?


What if we write the 3 desktop files instead of installing them?
documentation could go in somewhere /usr/shared/... and we create the installer desktop files for live session user on the fly?


Do you know what the site domain name will be/which host we will use?

On Mon, Jun 15, 2015 at 2:10 PM, Roger Zurawicki <notifications@github.com>
wrote:

> I just had some small nitpicks, the great work overall!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackharvard/H2Website/commit/fa31191274546bb52c00a9e480da755404c742d3#commitcomment-11686044>
> .
>


Where does `listing` come from?


Actually this one is located in a different Strings utility class joptsimple.internal.Strings, perhaps is better to avoid using it as is an "internal" utility.... Don't know but I think the calling code could do well just using Strings.repeat(String,int) instead... does not seem to be performance critical (quite a few should actually be constants).


Not sure what's going on here, haven't you already summed these?


Thanks, is it tomorrow already :-p
I'll make more tests by my side and if everything is okay I'll open a pull request for this.


so I replaced tabs for spaces everywhere except built release versions


You can add some error handling to a lot of the $http calls


I think you want to calculate this each day so you know when it's dipped below 4


never assigned...


I'm good with this


I included this link for connivence purposes.


Why not `formData`?


Nice!


Great use of filters in this file.


@Eilon - fixes are here


@masteinhauser thanks a lot for the quick and rich feedback! CI/CD integration is definitely what I'm aiming at too; I'm planning to use --var-file packer option, so that I can re-define `pre_local_commands` on different environments, if needed.

I must agree that having to write full bin paths feels wrong and hacky though, I perfectly understand your point.


Very nice nested states.


@cmherd DWMR. The text sections/divs of the landing page no longer animate and show. I'm actually not sure if that is new in this checkpoint or a previous one. It's worth 15 min of your time to sort it out. Probably some JS was renamed or dropped.


hackharvard.org, bluehost I think


I've had the problem where I'm running several instances for different combinations of mysql flavor and topo flavor. Rather than try to make the subdirectory names unique for each combination of settings, I figure we'll just make every invocation unique.

I thought about adding a -clean option, but in most cases 'rm -rf _test' should work fine.


Paoloo... I'm holding off any platform merges until I make sure this doesn't break the current cross-platform implementation and works with libuv, but keep looking at these issues.  The old platform-dependent code lives in the various directories under src/styx/<plattform> and src/styx/hosting.


> If you're having issues, make sure to raise them on those repos as they are the canonical source.

This might be confusing for people as some issues are actually subject to be discussed here.


I would put these in an array of "excluded inputs" somewhere near the top so these are not buried in the code.


All those files get created on the fly but installed to normal directories also. Check those packages to find out where we install them originally.


Nice use of relative states in ui-sref!


Nice, I'll be waiting.


Not sure if it's related to this commit or another, but the output of PlaylistItems is different than PlayListItemsWithTags.  Which is fine, I was able to modify cleanup to mostly work but still can't duration.  Any clue about that?


OK!


Is a sub-resource sync not required when deleting? (e.g. removal of entries from pivot tables, clearing the foreign key on the related resource?)


How come this didn't move to the binder?


Thank you Severin, I made some change and also used your example to improve the readability.

```XML
<?xml version="1.0" encoding="utf-8"?>
<courseStructure p1:any_Attr="anySimpleType" xmlns:p1="otherNS" xmlns="http://www.adlnet.gov/cmi5/CourseStructure.xsd">
  <course p1:any_Attr="anySimpleType" id="http://uri1">
    <title p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Course Title langstring1</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring2</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring3</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring4</langstring>
    </title>
    <description p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Course Description langstring5</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring6</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring7</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring8</langstring>
    </description>
    <languages p1:any_Attr="anySimpleType">en-US fr de as</languages>
  </course>
  <objectives p1:any_Attr="anySimpleType">
    <objective id="http://uri2">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Title langstring9</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring10</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring11</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring12</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Description langstring13</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring14</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring15</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring16</langstring>
      </description>
    </objective>
    <objective id="http://uri3">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Title langstring17</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring18</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring19</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring20</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Description langstring21</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring22</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring23</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring24</langstring>
      </description>
    </objective>
    <objective id="http://uri4">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Title langstring25</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring26</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring27</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring28</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Objective Description langstring29</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring30</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring31</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring32</langstring>
      </description>
    </objective>
  </objectives>
  <au p1:any_Attr="anySimpleType" id="http://uri5" moveOn="NotApplicable" masteryScore="0" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType1">
    <title p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring33</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring34</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring35</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring36</langstring>
    </title>
    <description p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring37</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring38</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring39</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring40</langstring>
    </description>
    <objectives p1:any_Attr="anySimpleType">
      <objective idref="http://uri2" />
      <objective idref="http://uri4" />
    </objectives>
    <url>http://uri5</url>
    <launchParameters>anyType</launchParameters>
    <entitlementKey>anyType</entitlementKey>
  </au>
  <block p1:any_Attr="anySimpleType" id="http://uri15">
    <title p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Title langstring41</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring42</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring43</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring44</langstring>
    </title>
    <description p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Description langstring45</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring46</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring47</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring48</langstring>
    </description>
    <objectives p1:any_Attr="anySimpleType">
      <objective idref="http://uri2" />
      <objective idref="http://uri3" />
      <objective idref="http://uri4" />
    </objectives>
    <au p1:any_Attr="anySimpleType" id="http://uri6" moveOn="NotApplicable" masteryScore="0" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType1">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring49</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring50</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring51</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring52</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring53</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring54</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring55</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring56</langstring>
      </description>
      <objectives p1:any_Attr="anySimpleType">
        <objective idref="http://uri2" />
        <objective idref="http://uri3" />
        <objective idref="http://uri4" />
      </objectives>
      <url>http://uri6</url>
      <launchParameters>anyType</launchParameters>
      <entitlementKey>anyType</entitlementKey>
    </au>
    <block p1:any_Attr="anySimpleType" id="http://uri7">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Title langstring57</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring58</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring59</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring60</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Description langstring61</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring62</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring63</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring64</langstring>
      </description>
      <objectives p1:any_Attr="anySimpleType">
        <objective idref="http://uri2" />
        <objective idref="http://uri3" />
        <objective idref="http://uri4" />
      </objectives>
      <au p1:any_Attr="anySimpleType" id="http://uri8" moveOn="NotApplicable" masteryScore="1" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType2">
        <title p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring65</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring66</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring67</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring68</langstring>
        </title>
        <description p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring69</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring70</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring71</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring72</langstring>
        </description>
        <objectives p1:any_Attr="anySimpleType">
          <objective idref="http://uri2" />
          <objective idref="http://uri3" />
          <objective idref="http://uri4" />
        </objectives>
        <url>http://uri8</url>
        <launchParameters>anyType</launchParameters>
        <entitlementKey>anyType</entitlementKey>
      </au>
      <block p1:any_Attr="anySimpleType" id="http://uri9">
        <title p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Title langstring73</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring74</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring75</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring76</langstring>
        </title>
        <description p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Block Description langstring77</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring78</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring79</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring80</langstring>
        </description>
        <objectives p1:any_Attr="anySimpleType">
          <objective idref="http://uri2" />
          <objective idref="http://uri3" />
          <objective idref="http://uri4" />
        </objectives>
        <au p1:any_Attr="anySimpleType" id="http://uri10" moveOn="Passed" masteryScore="0.1" passIsFinal="false" authenticationMethod="Basic" launchMethod="OwnWindow" activityType="activityType3">
          <title p1:any_Attr="anySimpleType">
            <langstring p1:any_Attr="anySimpleType" lang="en-US">langstring81</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="fr">langstring82</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="de">langstring83</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="as">langstring84</langstring>
          </title>
          <description p1:any_Attr="anySimpleType">
            <langstring p1:any_Attr="anySimpleType" lang="en-US">langstring85</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="fr">langstring86</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="de">langstring87</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="as">langstring88</langstring>
          </description>
          <objectives p1:any_Attr="anySimpleType">
            <objective idref="http://uri2" />
            <objective idref="http://uri3" />
            <objective idref="http://uri4" />
          </objectives>
          <url>http://uri10</url>
          <launchParameters>anyType</launchParameters>
          <entitlementKey>anyType</entitlementKey>
        </au>
        <au p1:any_Attr="anySimpleType" id="http://uri11" moveOn="Completed" masteryScore="0.9" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType4">
          <title p1:any_Attr="anySimpleType">
            <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring89</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="fr">langstring90</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="de">langstring91</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="as">langstring92</langstring>
          </title>
          <description p1:any_Attr="anySimpleType">
            <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring93</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="fr">langstring94</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="de">langstring95</langstring>
            <langstring p1:any_Attr="anySimpleType" lang="as">langstring96</langstring>
          </description>
          <objectives p1:any_Attr="anySimpleType">
            <objective idref="http://uri2" />
            <objective idref="http://uri3" />
            <objective idref="http://uri4" />
          </objectives>
          <url>http://uri11</url>
          <launchParameters>anyType</launchParameters>
          <entitlementKey>anyType</entitlementKey>
        </au>
      </block>
      <au p1:any_Attr="anySimpleType" id="http://uri12" moveOn="CompletedAndPassed" masteryScore="0.2" passIsFinal="false" authenticationMethod="Basic" launchMethod="OwnWindow" activityType="activityType5">
        <title p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring97</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring98</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring99</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring100</langstring>
        </title>
        <description p1:any_Attr="anySimpleType">
          <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring101</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="fr">langstring102</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="de">langstring103</langstring>
          <langstring p1:any_Attr="anySimpleType" lang="as">langstring104</langstring>
        </description>
        <objectives p1:any_Attr="anySimpleType">
          <objective idref="http://uri2" />
          <objective idref="http://uri3" />
          <objective idref="http://uri4" />
        </objectives>
        <url>http://uri12</url>
        <launchParameters>anyType</launchParameters>
        <entitlementKey>anyType</entitlementKey>
      </au>
    </block>
    <au p1:any_Attr="anySimpleType" id="http://uri13" moveOn="CompletedOrPassed" masteryScore="0.8" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType6">
      <title p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring105</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring106</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring107</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring108</langstring>
      </title>
      <description p1:any_Attr="anySimpleType">
        <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring109</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="fr">langstring110</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="de">langstring111</langstring>
        <langstring p1:any_Attr="anySimpleType" lang="as">langstring112</langstring>
      </description>
      <objectives p1:any_Attr="anySimpleType">
        <objective idref="http://uri2" />
        <objective idref="http://uri3" />
        <objective idref="http://uri4" />
      </objectives>
      <url>http://uri12</url>
      <launchParameters>anyType</launchParameters>
      <entitlementKey>anyType</entitlementKey>
    </au>
  </block>
  <au p1:any_Attr="anySimpleType" id="http://uri14" moveOn="NotApplicable" masteryScore="1" passIsFinal="true" authenticationMethod="Basic" launchMethod="AnyWindow" activityType="activityType2">
    <title p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Title langstring113</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring114</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring115</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring116</langstring>
    </title>
    <description p1:any_Attr="anySimpleType">
      <langstring p1:any_Attr="anySimpleType" lang="en-US">Assignable Unit Description langstring117</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="fr">langstring118</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="de">langstring119</langstring>
      <langstring p1:any_Attr="anySimpleType" lang="as">langstring120</langstring>
    </description>
    <objectives p1:any_Attr="anySimpleType">
      <objective idref="http://uri3" />
    </objectives>
    <url>http://uri14</url>
    <launchParameters>anyType</launchParameters>
    <entitlementKey>anyType</entitlementKey>
  </au>
</courseStructure>
```


I populated the table with fake data for illustrative purposes.


I guess here we use not statuses, but actions. And from this side it seems that it should be "suspend", "pause", "unpause" etc.


Estou criando um banco de questões em Gestão e Segurança da Informações utilizando os conceitos da aula de XML e AJAX.


Okie Dokie, I've removed the `.smlTitle .title` and the font reverts back to the default size of the original rxCollapse object.


Fair enough. This specific check is really not that deterministic.


instead of comparing all attributes, create two Handler instance and compare against those


Proper namespacing! Yay! :)


I don't think this will be the final resting place for this code, but as this refactor progresses I think homes for things will become more apparent. In the meantime, I find it useful to split out unit-testable code from the denseness of the model layer.


pattern match instead of isInst + asInst


>  once the scheduler is reimplemented in bash.

(in the commit message). s/bash/python/


Ejercios/POO C++/Mi primer proyecto/Auto Moviles/LLantas.h

Objeto Llanta


Huh ?!!


Hey @mkristian, are you planning publishing release to rubygems :)?


instead of comparing attributes, create a method type to compare against


I think @bgruening is correct, lines 149-155 can be simplified to:

```
if self.strict_min:
    op1 = '>'
if self.strict_max:
    op2 = '<'
```


Seems easier here just to attach it to the artwork itself (or have the reviews already available via population on the server)


`scope.rb` has a `resource` accessor, but `to_s` explicitly calls itself as mainly used for logging. So that seems like the intended behavior.

`rights.rb` and `templatewrapper.rb` aren't a Provider or Type, so should fall to the else clause. They override `to_s` in a way that makes sense.


You are mocking out too much by doing this. You really want to simulate an option selection because this behaves differently on different browsers.


`value=` is not necessary with `ng-model`


Avoid NPE, use Utils.nonNull.


Avoid NPE, use Utils.nonNull on both arguments.


Use a `| json` filter for even more readability!


Sorry that is a private method so I guess you don't need to do that.


Root of all evil.


How is this related to the balance?


@@ -0,0 +1 @@


Thanks for the tip!


I'm just curious about the purpose of this class.


Thank you Swapnil.  I agree w/ Andrew's comments about setters, length, etc.  It seems like a hindustani folder is the best way to go forward, for this and the tabla class.  Can we wait until the class is more full featured w/ documentation (or at least links to explanations of what the various objects represent) before considering it for a pull request? I try to only merge at the point when the project will be generally useful to people doing research in the field.  Thanks!


Why would we want to call out a specific version? I'd think we'd want :latest tag. Do we really want to have to go through and update all these simple template examples? I guess one could argue that the :0.10 will remain around for a long time.


make reverse trick more explicit, or at least add some doc.


![Uploading Siddharth Jain_Bharati Vidyapeeth's College Of Engineering.png…]()


You may want to use a third-party localStorage abstraction (like $ngLocalStorage) next time.


This change should have triggered a clirr failure, but it was masked by a previously existing exclusion that was much too generic for what it was supposed to do: https://github.com/xwiki/xwiki-platform/commit/4a85f108cf096f692ce9a9e72a243200b1a8f7be#diff-f7d9f26837a9903e7c4befe7db15b03fR503

Caty has just added it to the [clirr report of 7.1](http://www.xwiki.org/xwiki/bin/view/ReleaseNotes/ReleaseNotesXWiki71#HAPIBreakages) with the message `Young API. This method should not have been public from the start` which seems to be the intention of this change.

Please confirm.


Yeah, do you have another way you'd prefer it be phrased? Maybe something like "if your'e having issues with the template files generated for those projects, make sure to raise them on those repos.."


south


![siddharth jain_bharati vidyapeeth s college of engineering](https://cloud.githubusercontent.com/assets/11039724/8167058/612d3132-13b9-11e5-8ccf-9175457b5a7b.png)


@@ -0,0 +1,2 @@


arg previous value is totally thrown out here so I guess that ```String arg``` could be declared-initilized as final within the previous loop.


The statement `preg_match("/on/i", $flag)? 1 : 0;` is used a lot among the various resources and I was wondering if it should be replaced with a generalized utility function, e.g. `isFlagSet("on", $flag); /* returns a 1 or a 0 */`


Get ... picture?


`var antialias = 1 / browser.devicePixelRatio / layer.paint['circle-radius'];` would be slightly blurry on retina screens.


comment: could make prodCons lazy. `invocationsToRewrite` would then traverse all instructions of the method (instead of looking at the consumers of indy). so prodCons would only run if there is some callsite that looks like the closure body call (name + signature).


I would call the second `arg` variable `lastArg` and declared-initialize as final.


Is this key named correctly?


Ahhh, good ol' `/api/someroute`.


Are we using spaces or tabs?


Achei seu repositório, colocando meus dados e passou pela parte de autenticação, só deu erro na parte dos testes do protractor (esse vemos depois). Sem dados concretos, fica difícil ajudá-lo, mas não se preocupe, vai dar tudo certo.


No need for this `.then`.


Daniel...

Você já está admitindo que a pasta já está em `app/data/colecoes`?


Same here.


+*.dat


Looks like a bad diff. The change here is just a slight reorganization of the eval/arguments strict mode check (which is a different check from the reserved words).


Probably belongs in a factory.


Sounds good.


+static


I found this: http://www.1001freefonts.com/afta_sans.font


+* use per-app templates folders


I don't know the internals of the current solution but shouldn't you store this as it was done before?


tabs are evil


+42-test template


> Note: This implementation still has a regression vs the original
> implementation in bash.  This will be fixed in subsequent commits.

(in the commit message). Should probably mention *what* the regression is, briefly: "This implementation still has a regression (in the Control-C handling) vs the original ...".


I see the purpose of getAdtagsListAttribute and other related methods (getVideoadsListAttribute, getGamesListAttribute), but I feel that these are misplaced in the Resource class, which is the parent class for all resources; this smells of weak cohesion because it tries to deal with special cases that are specific to certain resources. It also means that I can call e.g. PublisherSetting::getVideoadsListAttribute() with unexpected results.


In case you need to store you need to do the inheritance the other way around


I'm not thrilled with my implementation although I wasn't thrilled with the original implementation either. More testing on this would be great. This should be brought into an issue in the movement validation repo to compare the two. What's really missing however are the test cases.

Relevant code:
https://github.com/openworm/movement_validation/blob/master/examples/test_normalized_worm_creation.py

See NormalizedWorm.from_BasicWorm_factory

Let's move this conversation into an issue


Wondering if I could ship this code as a Packer plugin - http://www.packer.io/docs/extend/plugins.html


I wouldn't call this "failing cleanly", I expected that we would issue a warning and continure with the convergence if we this cookbook was running on non-ec2 hardware. (See #1 #2)


Sim. Em outro commit submeti minha coleção.


I would push up the nested if condition into the enclosing one with a ```&&```


I decided to go with my own classes for math, because I wanted to learn and practice all the math theory behind the glm library.


Awesome, thanks! One potential issue with Bluehost is speed. The site might
be a bit laggy (I've noticed this for some of my personal sites on
Bluehost). What about AWS? (most AWS services - including site servers -
are also free for the first year)

On Mon, Jun 15, 2015 at 2:13 PM, Roger Zurawicki <notifications@github.com>
wrote:

> hackharvard.org, bluehost I think
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hackharvard/H2Website/commit/fa31191274546bb52c00a9e480da755404c742d3#commitcomment-11686106>
> .
>


In the long term I wouldn't even expose root, after all this it should be more like something internal to the format


Całe szczęście że to zauważyłeś :P Myślałem ze to już zrobiłem wcześniej, ale musiałem postawić repo drugi raz i nie powtórzyłem już tego xd Dzięki


I believe you've defined it as .createListing in the factory.


@tgoyne: Addressed your feedback here.


Transform NPE to IAE with Utils.nonNull


I'm only exposing to enable people to use a custom_root if they want to.


This leads to crash on

```
import jnius
```

On Nexus 5 Android 5.1.x. Reverting this fixes the issue,  we need to test this on different versions of Android. To see where the initial issue was.


this code was used to compare with the "ticks_in_main" StatCounter in `jit.cpp`.  without this I don't think we need ticks_in_main either.


Didn't see that you added in the ubuntu package... Might make sense to use
that since the service manager still tries to keep uwsgi alive if there are
problems?

On Sat, Jun 13, 2015 at 11:24 PM, Dean Chen <notifications@github.com>
wrote:

> Do we still need this now uwsgi is installed by PyPi?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/atomic-labs/docker-flaskbase/commit/acc02866afa48cd4e3736baa6037fe292fe082ec#commitcomment-11668944>
> .
>


I have already used my first year of hosting. AWS is alright. If we have any free credits then we can use whatever. We will cloudflare the site in any case.


This import should really be moved to the proper import block.


CutomError


On my machine the save doesn't work unless first the directory is created


Created the wiki for the gem
Are there any errors in Readme or gemspec? or any improvements for them?


I don't think that should happen. In my mind a format is defined without the possibility to change stuff, that's why we are providing this two things.

If someone wants to change something they should provide an adapter, inheriting if they want to take advantage of something.


Defined as "quantity"  not "qty" in your Listings schema.


Yes this method was public by error; it was never meant to be public.


It was done this way in C^3, and I did it to minimize the delta. I'll provide a non-transformative version, it requires modifying the assembly that implements cos_syscall_upcall.


I imagine this will catch cases like:

- argument: "-t"
- help: "One-third of the values  .... "

(or any other hyphenated words)


Any plans on modifying this to edit forms already created?


I looked over the api design doc at https://docs.google.com/document/d/1BfqulTKscALY5itrhKt7KlOymVigR3eZK2gixWuxE2Q/edit and I can't discover the rationale behind always returning a HTTP 200. I always thought that full REST "compliance" included proper use of HTTP verbs and status codes.


The beep was originally removed by Shiv as he wanted to get rid of "OK" message. I put it back as the beep helps user to not move too fast (moving too fast will crash the machine in analysis/kibitz.) Removing the "OK" message should be done in some other way ...


Seems that ArrayUtils.addAll covers this functionality.


Is this change solely to support a larger attr size, and not to use the new fields? If so, we have just landed a change to support reading attr sizes other than the static size of the struct.

https://chromium-review.googlesource.com/#/c/275345/


Ok!


reack


I got it and agree, but "deprecating" it now might make it harder to ppl willing to update to AMS 0.10.x.
What is your thought about it? Should we take advantage that we are on a RC and remove it, or wait for it?


No problem since the changes were not related at all. I think the policy for direct push should be only emergencies (critical bugs) or really small fixes (e.g. typos).


Just bear in mind that the log could, in theory show:

```
Top commit: sh: git: command not found
Branch: sh: git: command not found
```

It's not common to have .git and *not* having `$ git`, but maybe you want to go the extra mile:

```
   status, git_commit = commands.getstatusoutput('git show ...`)
   if os.path.exists('.git') and os.path.exists('avocado.spec') and (status == 0):
```


I'm not sure it's important, but I'd used a separate configuration-error because I wasn't sure that a config file error should be treated the same as a cli-error, and because Russell asked whether or not the TK cli-error and cli-help exceptions were actually a public interface.


 :sweat_smile: Not sure what you meant by
> store this as it was done before


aircrack-ng 1.2-rc2 fails to build with this make file

/home/user/openwrt/staging_dir/toolchain-i386_geode_gcc-4.8-linaro_uClibc-0.9.33.2/lib/gcc/i486-openwrt-linux-uclibc/4.8.3/../../../../i486-openwrt-linux-uclibc/bin/ld: cannot find -lssp_nonshared
/home/user/openwrt/staging_dir/toolchain-i386_geode_gcc-4.8-linaro_uClibc-0.9.33.2/lib/gcc/i486-openwrt-linux-uclibc/4.8.3/../../../../i486-openwrt-linux-uclibc/bin/ld: cannot find -lssp
/home/user/openwrt/staging_dir/target-i386_geode_uClibc-0.9.33.2/usr/lib/libcrypto.so: warning: gethostbyname is obsolescent, use getnameinfo() instead.
collect2: error: ld returned 1 exit status


I would remove this method: ```arrayFromArrayWithLength```; is only used twice in the same place in testing code to basically repeat the same byte over... that is already covered by dupBytes method above.


can't think of a better way at the moment. if there's a specific case where this fails, i can dig in


Is `full` the right name for this variable?  Perhaps `minor` or `patch`?


one question what is this {envlogdir} variable, I wanted to see how much code is covered by our tasts, but I couldn't execute the command.


@gnarf Looks :+1:  to me. Thanks!


Gotcha.  I think making that change is appropriate here.  These transformations just make life difficult for everyone ;-)


In the code I had, there were these lines at the beginning of runSim that first deleted then created the working directory and then set that as the working directory. Was there a reason this wasn't working for you?

  # Check/setup folder structure
  if(file.exists(test.params@wd)) {
    unlink(test.params@wd, recursive = TRUE, force = TRUE)
  }
  dir.create(test.params@wd)
  wd <- setwd(test.params@wd)


```Arrays.fill(bytes[],byte)``` does the job, so please remove it and update calling code.


Proposed package changes:

Remove from manjaro-documentation /etc/.skel/Manjaro

Remove from cli-installer /etc/.skel/Desktop
add desktop file to /usr/share/applications

Let us create symlinks to the desktop files in the live session desktop folder on the fly.

should I also change manjaro-documentation?.


Yup. My mistake. Thanks for pointing it out.


Sorry :).

`@result = { root => @result }`

As long as I can see the whole result was being stored in @result.
Exactly what is happening in Line 41 https://github.com/joaomdmoura/active_model_serializers/commit/5932da64efacc60b986a967d8dd465a0039654d5#diff-7b562c84aa219fb9dbfb8112b95df4d9L41


Oh, wait, I see. It's the subdirectory (label) that doesn't get created by fastsimcoal on your machine, right?


You may be able to change calls to this method with ```Utils.validIndex```.


yep!  that's it


You can use ```Utils.nonNull``` here.


Not sure to be honest, according to what I see the migration is already a bit hard so I'm not sure if the root things changes anything.


Hi @ruslanskorb,
I think the subclassing approach is not ideal. To use custom `RSDFDatePickerDayCell` means the subclass of `RSDFDatePickerView` needs to reimplement everything in `collectionView:cellForItemAtIndexPath:`.


You can use ```Utils.nonNull``` here


@nsoranzo yes, this was in my latest addition.  It took me a couple of minutes to get my eyes uncrossed, but finally saw it.  :)


Nice commit message :p


This may be too picky on my part, but it would be nice to have a generic "print_table" of sorts on `avocado.utils` and reuse it here.

We don't lack the code itself, as I remember there are implementations on both autotest/virt-test and arc. If you feel like doing it, it would improve readability here by many orders.


Howabout removing the inner tryCatch and changing it to this:

if(!dir.exists(label)) dir.create(label)


Fixed in b8fdb2542d728d23d75e1f9d54ac4973a1ae2508


Oh got it.
So, there is particular need to store this right now because this is the last line / return from AMS.
We can store it if we want to, even without change the inheritance itself, by storing it into some other variable in both adapters, not sure about the name but something like ```serialized_object``` or  ```serialized_return``` idk.


Same string formatting logic, as mentioned on previous commit.


```o == null``` is covered by ```!o instanceof ...``` so you can remove that if.


That sounds good to me!


ack


reack


Oddly it seemed to work just fine for me on Arch and CentOS otherwise I would have caught it.


could you use master only here? As you already tested this?


You can use ```Utils.nonNull``` here. I think that ISE instead IAE es a bug.


Neat. Thanks!


You can use ```Utils.nonNull``` here.


`your'e` -> `you're`?


I'm thinking this is better - because any branch with a name ending as "/circleci" can test it.

What do you think ?


oh. Already fixed.


Well, do the needful. We have to remember those changes and backport it to master branch of manjaro-tools if needed, or should I already start to use the devel branch again for my 0.9.0-dev builds, so also I test your changes?


Would checking for arguments in parenthesis make more sense i.e. '(--argument)'?


Actually, a plugin is a great idea. Specifically, using or [creating](http://www.packer.io/docs/extend/provisioner.html) a `local shell` provisioner similar to the [provided `shell` provisioner](http://www.packer.io/docs/provisioners/shell.html) would be able to achieve the same effect you need.


Done.


@he1000: retry


Also include username.  (Not password!)


This is a pattern that occurs in several transformers where we really just return what's in the item. Have you thought about implementing a list of "pass-thru fields" where we simply list all array keys that get passed through from the item to the output, e.g. `$passthru_fields = ['use_modal', 'auto_start', 'max_height', 'max_width'];` and then below we just loop through them and grab them from the item? This would be implemented in the parent class, `Transformer`


yep! Parentheses is probably the best option. I used to stick mine at the
beginning, so checking place in the string isn't so useful.

man. 15. jun. 2015 kl. 13.48 skrev Aysam Guerler <notifications@github.com>:

> Would checking for arguments in parenthesis make more sense i.e.
> '(--argument)'?
>
> —
> Reply to this email directly or view it on GitHub.
>


Just a suggestion, or maybe just a matter of pure taste: most tree implementations that I know of tend to refer to the initial point of the tree as the `root`, that is, they do not emphasize the whole tree thing.

One example from libxml2:
```
getRootElement(self)
     Get the root element of the document (doc->children is a list containing possibly comments, PIs, etc ...).
```

This is really minor, but a suggestion nonetheless.


That's how it was originally written. Do you want me to add a `try`/`finally` here too?


my current version of php is 5.4.24, I guess I will have to update my version. Thanks for your answers guys!
@pespantelis  @brayniverse


Looks good.


That would be cool.
I had just the idea, that you may want to do a xdg-user-dirs fix release for 0.8.13 and we then start to use new pkglist tags for manjaro-0.9.0?


why did you do that


OK.


The client already gets back HTTP status code 200.  Is there any value in having a message?  Or maybe it should be really simple and generic like "Request successful."


Agreed.


Done


Told Jekyll where to place content


Since we are here, and this method is public it should check for null input arrays and throw an IAE if there is any.


the $fillable array should probably go in the config file `config/resources/games.php`? (same for other models)


I think there is a typo at the protocol part.


I'm not sure about this but I'm kind of inclined to say that this should also go in the config file as it effectively describes data types for each field. (this applies to all models, I guess)


Same as above. Check for nulls and through IAE if so: use ```Utils.nonNull``` for that.


Same, check for null insQuals. Use ```Utils.nonNull```


What do you think about removing the following sentence?

> We all first met doing some late night hackathons at a coworking space about 4-5 years ago.


`| currency` built into Angular


1667_SketchUpInspectorNewZone (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


should also have last_char_dot = false;


should also have last_char_hyphen = false;


@rpetti are you fine with merging those or should we split them apart?


This one looks safe to be merged :+1:


It's actually 7+ years for me :smile:

Instead of

> Eric and I were previously creating web and app solutions at VERBS for the past 9+ years.

How about

> Eric and I were previously creating web and mobile apps at VERBS for the past 9+ years.


I think move these into check_bucket_name_characters_for_DNS


self.find_by_xpath seems to take one argument only


Deleted {{ content }} to see what would happen
Added an extra blank line (unintentional)


I wonder if our resource configuration files are complete and comprehensive enough to generate forms instead of maintaining one form per resource, and if that would even be a better approach.


and what about

> Between us three, our values and interests always mesh well.

becoming

> Our values and interests have always meshed well.


move 'case 1:' label to here, drop the 'case 1:' body, then you don't have to repeat this code
But also moot if we validate the input to the config option


Wasn't it merged and released already??


sure, why not. changed :)


why? nil used less memory?


The context's provide you JObject wrappers that exposes some of the provider specific fields without you having to go lookup all the keys for them. I don't know if that's sufficient justification for their existence, but that's the value they currently add.  A few of the fields are harder to process than others: https://github.com/aspnet/Security/blob/dev/src/Microsoft.AspNet.Authentication.Google/Notifications/GoogleAuthenticatedContext.cs#L89

@HaoK What do you think about just eliminating these provider specific contexts?


The unit test on this one is still there; I suggest running phpunit locally before committing.


Ups! I update it as soon as possible


I disagree, we want those strings from master only, I usually use wip/sils/circleci for anything that I do on circle ^^


1. hidden assumption that ```\n``` is the newline separator.  If we're requiring newlines there should be explicit documentation up front about which format(s) are legal.
2. I like "history" better than "record" because it's less ambiguous.
3. I think it's cleaner to have nested JSON objects than to use newlines.  "record": [ "1g": "Ra1 Cc2", "2g": "Ra1n Ra2n Ra3n Ra4" ]
(I don't know if my JSON syntax is correct but you know what I mean.)
4. Another option, even more verbose: { "move_num": "1g", "move": "Ra1 Cc2" }

(I think option 4 is better than option 3.  The few JSON APIs I've seen always have standard things on the left side of the colon.)


angular.copy


Oh right, I forgot to do that.


Yep, true.


confirmed. this works for my project now. thank you for this.


added title to page layout
deleted some code that was no longer needed
changed wording of paragraph


And how about changing

> A couple months ago we began having conversations about creating a new company that was focused on helping startups and businesses create products that were valuable and solved real problems.

to

> A couple months ago we began having conversations about creating a new company that was focused on customer-driven product development.


Experimental comment. This is awesome.


Is this redundant with record/history?  Will it be used by the caller?


typo? http://github.com/capistrano/capistrano-symfony


added page title to page layout
deleted code no longer needed
added a new blank line
added a new paragraph to explain what page this is


OK, sure, I guess.


Conda build clones the repository automatically. There's no way to add this to the git config that way.


Oh, and added support for probe wrapped in a VLAN header.


Done! I'm working on documentation. Be pacient.

And yes, it's easy to start because is compatible with original ArrestDB, but it's complete if you want to do a lot of things. I'm using it in a project and it's powerfull.


invokespecial (java emtis / allows private body methods, constructors). check out more java examples (https://github.com/scala/scala-java8-compat/blob/master/src/test/java/scala/compat/java8/LambdaTest.java)


One issue here is that the API version is hardcoded right in the form. Updating our API version to v2 would be non-trivial as we would need to locate and replace all such occurrences. I recommend that we insert the same API version as requested by the user.

So, if a user comes in to the v2 api, our forms would always refer to v2. Likewise, a v1 user would always see forms referencing v1.


I know this wasn't done before, but neither was tidying up of open files. If you are going to do it then you may as well fix this up too. You are making sure the file is closed, but the message may not have been released (think freed) in gribapi, which is not ideal. I guess the way to handle it is with another `try`/`finally` block within the loop yes. I don't know if this will have any negative impacts though.


removing code no longer needed
adding site title and page title to default page layout
adding page menu to default page layout


The above comment is applicable to all similar views.


I like the, "sure" and then "I guess". :smile:

Now, as I said before, this is really a matter of taste, so if it also fits your taste, good. If not, it's a tie and you win.


add comment


More inclusive language, love it. I'll approve it.


giving kittens page a default layout


giving home page a default layout


comment on this opcode offset trickery


If we're adding new dependencies, https://github.com/shama/userhome might be a closer fit and it has less dependencies.


Could this be done in it's own pull request (is it independent of the other changes)?


$ grep -r AU_LINUX system/vendor/lib64/
Binary file system/vendor/lib64/egl/libEGL_adreno.so matches
Binary file system/vendor/lib64/libCB.so matches

$ strings system/vendor/lib64/egl/libEGL_adreno.so | grep AU_LINUX
Remote Branch: refs/tags/AU_LINUX_ANDROID_**LA.BR.1.1.2.C2**.05.00.00.059.008

LA.BR64.1.1_RB1.10 + 26.1.A.1.112 -> 565 files changed (many CAF changes and FIH changes)
LA.BR.1.1.2.C2 + 26.1.A.1.112 -> 316 files changed (only FIH changes)

It is clear that FIH did not use the RB1.10 branch, despite what they may have told you.


(If we anticipate that these forms are very v1-specific, then they should live in a v1 subdirectory or be otherwise organized by api version)


created default layout page with paragraph and instruction where to place content


edited config file to confirm GB timezone (not operating system generated)


The function "default.node_sound_gravel_defaults()" is properly defined (by Carbone to sound like dirt) in the file 'functions.lua'.


By the way, unrelated, but you can add #alternaterecolor to items to use alternate recoloring method. It's a far better method since it doesn't require brightening up greyscaled sprites, but it was originally made alternate because majority of the graphics are made with normal recoloring in mind.

I'd like to swap to use just it, but with UnitGen it's way too late.


Why not do it like AEI?
> [RRRRRRRRCDHEMHDC_______________________________cdhemhdcrrrrrrrr]

(I am writing underscore instead of space because GitHub is collapsing my whitespace.)

This is so much easier to read and write than FEN.
You are worried about bandwidth?


Do you mind pulling each of these out into their own pull requests?


edited code to include site title from newly created config file
created a link on kittens.html to index.html


This will fix the issue with a negative timezone as mentioned in issue [92](https://github.com/nuodb/nuodb-python/issues/92)


system "xhyve", "-h"


Additionally, instead of writing out the actual URL here, we can ask the router to give it to us if we know the route's name. http://laravel.com/docs/4.2/routing#named-routes


fixed!


removed code no longer needed
inserted site title on to page from newly created config file


use better log message.


created config file
inserted site title


Caues a StopIteration exception instead. Is this the behavior we want?


use better log message. no `[DELETE_HANDLER]` prefix, please


To be more specific, allows this in fields.yaml
```
fields:
    name:
        label: Name
        type: text
        attributes:
            maxlength: 16
```


changed "To the kittens!" from a standard paragraph to a html link


Is it necessary to specify the floating-point precision somewhere?

If the caller is just displaying the info it probably doesn't matter but if the caller is doing numeric calculations it certainly does.


It's not well worded, but I think the original is more correct.


unused field.


created kittens page
inserted first paragraph
inserted image of kitten


Yes, for sure. I just created this request to validate if I have taken correct initial steps. Obviously, there is a lot to be added on top of this and a merge makes sense only with at least one fully functional use case. Thanks.


If someone could explain why every file should be "executable"...


Specify result code somewhere.


no `public` needed


this is good to go!
<3 love the links to the teams, too!


updated initial paragraph to a h1
deleted unnecessary text
added a new paragraph "To the kittens!"


please format javadoc nicer


This is game length?  Or the date and time the game ended?


description ?


added a second paragraph "Cheese!" to test out the <p> function


Fixed, according to Seema


"ply" is an AI term.
You mean number of moves completed so far?


Sure. I guess the commented out part should also be included as *ToDo*.


You know this is testing branch right?


created index file
inserted paragraph "Hello World!" to test


Initial commit, repo created


Fixed in https://github.com/ensingm2/SteamMonsterGameScript/commit/b8e56dcd69d2439d6aa2468c11e98ef3836764f8


Specify format for boolean somewhere.  "T"?  "true"? 1?


good work done !!


It could eventually use less (zero) memory. However `nil` is currently represented as `i1` in LLVM.
Also, if you compare two sets, it also must compare the values which is trivial if they are nils.


CoffeeScript似乎没有这个语法
应该是when a, b, c
而不是when a || b || c


logout


Cool idea here! May use this.


I suppose some people might see it that way :-)

On Mon, 15 Jun 2015 at 15:26 Matthias Kurz <notifications@github.com> wrote:

> Shouldn't this be
>
> DeadBolt 2 Java - Quick start
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schaloner/deadbolt-2-java/commit/ff095d32032d0d6eef5020c8f1726a3e14926c2d#commitcomment-11680094>
> .
>


This template has too much responsibility and it's very complex because of it. You should have one template that is the blends list, one that is a blend detail and one that is blend creation.


I think the move times should be grouped with the moves.


`master -> methods -> dev`

Merging methods to dev is the same as merging master to dev.
And you will get both features faster into master as they are easier to review.


This is epoch time the game was started?  Don't people normally use integer/long for that?


I made a layer to separate my code as much as possible from the source main_out.js as you can see, I made functions from line 1312 to 1414. What I need to update between each update is pretty much only what is returned in those functions.


I pushed a fix for that. Bad merge.


I don't think Listings.listing is a thing but I can be wrong.


unused import


This is the manpage, I think the reference to scripts/ is wrong


Shouldn't these const addition be in a separate patch, before this one?


I think this is much better (it's definitely more efficient), and a more realistic test is good too :)


How about changing the isEqualTo to containsAll? This way the test will not fail in case the exchange adds new exchanges. I think it's good to make sure that Coinbase provides a specific set of exchanges (hence this test).


"Now" seems funny but I don't know what would be better.
Maybe I just have to see it in action to see if I like it.


td and ng-model? Is this working?


"games" or "game"?  I think "game".


Indeed, that will be direction I'll be going for in the next couple streams.


:bulb: the `if (true)` is unnecessary. Just use a raw block.


Looks like you are only adding recvfrom and sendto was already there. /me confused with the commit message.


Have a look :

![image](https://cloud.githubusercontent.com/assets/2932687/8168604/3382a38a-13a4-11e5-9e8c-1235b474b4b8.png)


Think about which HTTP response code for all the error conditions.


What is the return type of ParseTokenResponse? I guess I should take a closer look ( I just joined the team to help out/take over). Seems like we should change the return type to string if all the usages are doing the cast to string (we'd need to look).


`ng-src` will make this work better.


yes i know, i case i want to build an test version, but if it's not ok then i delete it


Yeah, I think that is where it is looking. It's all good, everyone started somewhere :).


Require move number as well as move string?  This is a known issue with AEI.


On Mon, Jun 15, 2015 at 11:33:35AM -0700, Simon Que wrote:
> Is this change solely to support a larger attr size, and not to use the new fields? If so, we have just landed a change to support reading attr sizes other than the static size of the struct.
>
> https://chromium-review.googlesource.com/#/c/275345/

Yes it's only to tolerate the larger attr size. It does not use anything
in it.

Note that there are also other changes, such as handling the
FINISHED_ROUND pseudo record.

-Andi


:exclamation: This is locking on the wrong object. It should only lock on a single DFA state.


whoops, my bad. I'll fix this.


@warden This was merged and released in April. There's no need to split them apart after the fact.


:exclamation: This is locking on the wrong object. It should only lock on the DFA for a particular decision.


I have instaled hola player, but when i try to play a torrent video, hola offers to me to instal again. I tried to reinstal player, but stil dont work. Do you now what is the problem ?


Using `this` instead of `$scope`?


awesome sauce.

I'm happy to help with the webstack migration stuff, my haskell is probably
not good enough to help with the parser just yet.

Seems like a good time for me to learn React : )

I'll start with migrating to browserify and updating to latest version of
React.

On Thu, Jun 11, 2015 at 8:57 PM, Parnell Springmeyer <
notifications@github.com> wrote:

> Minor cabal file update
>
> @OR13 I dumped an old project that did exactly this in here and
> renamed some things. I need to rewrite the parser from using Parsec to
> using Attoparsec + Aeson since I no longer need specific look-ahead
> features from Parsec in these logs.
>
> I dropped an example log file from one of our event routers in the
> `./etc` dir.
>
> Another thing I'd like to do is get away from RequireJS and switch to
> browserify and gulp - I also need to update the version of react since
> most of this code is like a year old at this point.
>
> Additionally, in order to stream-tail a file into Redis (which this
> application uses as its faucet), this utility I wrote handles that:
> https://hackage.haskell.org/package/pub
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/OR13/haskell-event-algebra-learning-project/commit/30787fddefc0a7c4e309b2e71b762289f52cf39f
>


Whaqt is that


This line can be eliminated completely. In your template, you are already passing in `$scope.whichName` as the argument on `ng-click`.


Since we have a visible label for the field here, we should really be trying to associate it with the textarea by wrapping it or using the `for` attr, instead of duplicating it with an aria-label.


Hello! Thanks for the fork! What exactly does adding ``py:`` do? I'm new to Sphinx documentation.


Removed STL call, used local typedef.


>>> setx path "%path%;C:\Program Files (x86)\Arduino"
  File "<string>", line 1
    setx path "%path%;C:\Program Files (x86)\Arduino"
            ^
SyntaxError: invalid syntax
>>> `setx path "%path%;C:\Program Files (x86)\Arduino"`
  File "<string>", line 1
    `setx path "%path%;C:\Program Files (x86)\Arduino"`
    ^
SyntaxError: invalid syntax


close spring-projects/spring-boot#3208


ack


eh...that sounds like straining to make something "generic" where it doesn't need to be


I'm not sure right now.  I can see value in both approaches (leaving the catch-all or removing it).

In thinking about this, I feel like the catch-all topic is a bigger problem than can be solved in this PR.  For now I'm probably just going to add the catch-all's back.


Fixes /f kick <player>


Yep, looks better.


Fixes /f info <faction>


What?


Adds "ClaimingEnabled" to the default prefs.yml file


Btw, why are we removing Blu-ray from the results? we handle that in quality regex now.


The API code is fantastic and very well organized!


@cguardia: can you speak about why we are not using the pipeline for now?  Was this a temporary fix that should be removed?  Or a required change that must stay for our changes to function?


LGTM!


The test should test that the game is not finished when the board has just been created.


Or maybe add them back but with lower certainties for the appropriate fingerprints?


One note: this is a public repo! :)


My comment was just for commit b354882.

Since it is only to accommodate the larger attr size, it may longer be necessary. Try syncing to the the upstream quipper code and see if it works for you.


@cguardia: can you tell me a bit more about this change?  Is it required for the fixes to self-compled courses to work?


I believe these changes may have caused https://github.com/chef/chef/issues/3535.


I'm curious why you didn't implement authentication as a package?


changed all h1 to hot pink


do we have functionality in place to remember where the user wanted to go when they failed authentication, and then take them there once they are logged in?