I would make this `entity.hidden_indexable_content + entity.body` so that when the user they're both sent to rummager, instead of `hidden_indexable_content` overwriting what normally would be sent. That might not be what the user is expecting.


`objectWithID:` throws an exception if the blog has been deleted. It's usually safer to use `existingObjectWithID:error:` instead


I don't think so.


Thanks for replying. I was trying to use the jj-abrams branch of ember-simple-auth locally using npm_link. Feel free to disregard, I'm focused elsewhere right now.


@cosimoc Definitively not simpler since `old_path`, `old_entry->path`, and `new_entry->path` all need to be converted to a `GFile` and then unlinked as soon as you call `g_file_get_relative_path()`/`g_file_resolve_relative_path()`; it turns these 5 lines into 10-15ish that are much harder to follow.


The PrefixArrayElasticMethodCreatesPrefixWhereCriteria should take more than one field in the prefix call to ensure it's correctly emitting multiple ones?


I am also lost at the meaning of `complement`. Can this variable be renamed to something that evokes exactly its purpose? (it can even be longer, as long as it is clear what it does :smile: )


So, All can't be cast silently -> now using Unknown quite widely :-)

--
Olivier
http://ochafik.com
http://twitter.com/ochafik

2015-08-26 21:02 GMT+01:00 Leaf Petersen <notifications@github.com>:

> So I guess the idea is that all generics just get mapped to the raw type,
> and then all type parameters get mapped to All? Given invariance of
> generics in Closure, this might be the best that can be done. Closure
> allows things of type All to be silently cast to other types? Or is that
> what Unknown is for? If the latter, I think you'll need to map type
> variables to unknown.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dart-lang/dev_compiler/commit/3760837ee30b4c062509530f1b78058149da028f#commitcomment-12912167>
> .
>


*cylinder :)


you should merge this patch to fix the sample ASAP.

Also, would be nice to rework the oic samples to not need fixed `hwaddr`


Ahh good catch, I'll fix that now


It would be better to send both the `hidden_indexable_content` and the `entity.body`.  There's little harm if there's duplication between those, and there's definite benefit to making sure any text we have available is searchable.

```
entity.hidden_indexable_content + "\n" + entity.body
```


Yes I can read, but it's not clear to me why one would choose 5.


http://php.net/manual/en/mysqli.overview.php :)


@ksachs if you can amend following the inline comments (you can just re-push this very same branch) and you then make a pull request against the usual inspirehep/invenio:prod branch than we can merge it)


This wont work for state ids other than SYSTEM_SUSPEND. I have modified this logic a bit for different state ids


`r.Aliases` is an array


Let's keep it uppercase then. But this is exactly why I do not use this extension: I'd rather set it manually in my xsl template rather than having php do it. Do you want send a PR reverting this ?

> Note that if you don’t uppercase DOCTYPE in an XHTML document, the XML parser will return a syntax error.

I'd like to see the spec for that.


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: mixer


1: D100 Missing docstring in public module
22:1: I100 Imports statements are in the wrong order. from invenio.ext.mixer should be before from six
23:1: I201 Missing newline before sections or imports.
32: D101 Missing docstring in public class
36: D101 Missing docstring in public class
40: D101 Missing docstring in public class
45: D101 Missing docstring in public class
49: D101 Missing docstring in public class


You use variable "$NAME" but there is no such a variable in this script, you replaced it with "$NAMES".


OK, just for the record..
PEX is now included in the key_response message.
Instead of a single hidden-seeder-intro-point, a list of 50 of them is returned.
This may be a whopping long list, as it included 50 x (pub_key + IP:Port).


So when you hit the form submit button the redirect to the features page doesn't happen until node-git has finished working. It is a bit messy, I'll try to simplify it soon.


Okay, will do. [Here's the doc on the DOCTYPE](http://www.w3.org/TR/html-polyglot/#doctype).


How is it going to work if according to above condition `this` doesn't have a `_keyClasses` property?


> ... among other possibilities

Tell me more about these possibilities...


There were no changes here, just copied it over


No harm done. I will catch those kind of things sooner or later.


Ok great. If one spec says it's CI and the other one says it's CS, let use the one that works everywhere.


thanks, that makes sense. and I will have it renamed.


How do values from this._keyClassList get to the this._keyClasses object?


Doesn't handle the number of dimensions, at least not without getting another type in the C code, which seems not worth it.


I think we may want to create an article about invoices in the "Needs Review" state and link it from here. Maybe it is too short to create an article but I think we do need a section that explains what "Needs Review" means.


I wrote 1 in commit and 0 on code. My fault.
About the hack, I didn't understand exactly how do you want it, so no problem :) You can do it.
I found a bug on Windows 8.1 + Indy, simple problem with readtimeout I'll create another pull request.


Looks good


Cool, thanks for updating.


donde esta el documentation?


Good simple comments here. Without them, I'd probably have no idea WTF your intention was. :)


What is the difference between `$(npm bin)/mocha` and `$(npm bin)/_mocha`?


Nit: missing new line.


I don't see any errors related to this merge.

I do see
```
Host Machine Init:
Operating System = Microsoft Windows XP Professional Service Pack 3 (build 2600)
```

So, this merge actually works.

The only errors I see are unrelated to this merge:

```
Path: C:\Games\PCSX2\plugins\DEV9ghzdrk.dll
File is not a valid dynamic library.
[wx] Failed to load shared library 'C:\Games\PCSX2\plugins\DEV9ghzdrk.dll' (error
0: the operation completed successfully.)
Some kinda plugin failure: C:\Games\PCSX2\plugins\DEV9ghzdrk.dll
Path: C:\Games\PCSX2\plugins\GSdx32-AVX.dll
File is not a valid dynamic library.
[wx] Failed to load shared library 'C:\Games\PCSX2\plugins\GSdx32-AVX.dll' (error
0: the operation completed successfully.)
Some kinda plugin failure: C:\Games\PCSX2\plugins\GSdx32-AVX.dll
Path: C:\Games\PCSX2\plugins\GSdx32-AVX2.dll
File is not a valid dynamic library.
[wx] Failed to load shared library 'C:\Games\PCSX2\plugins\GSdx32-AVX2.dll'
(error 0: the operation completed successfully.)
Some kinda plugin failure: C:\Games\PCSX2\plugins\GSdx32-AVX2.dll
```

But you have:

```
x86 Features Detected:
SSE2.. SSE3
```

and you need https://www.winpcap.org/install/ to use DEV9ghzdrk. I'm surprised the SSE4 plugin was loaded when no SSE4 was detected unless that one was deleted from the folder.


libc is unix only. If you really want to add some c/c++ implementations to this project, introduce shared library (and this library may be extended in future to frontend for VCMI_Lib).


I copied this from some other code, for exmple http://cmslxr.fnal.gov/lxr/source/CondCore/SiStripPlugins/src/plugin.cc#0068 , so I assume it should be fine.


<3 <3 <3


This really requires a comment


Personally I'd do away with keeping the reference to the tags altogether, as at the present time they are unused. Storing and exposing them now is dead code that can be confusing in future, and it's also likely that when the future use case arrives, the code we've put in to make the future easier won't exactly fit what we want anyway and will need changing again.

That said, this is much more understandable now than the iteration before.


Addresses #93.


I think what we need is a wildcard route so that all requests route to `home_page` and then the URL is handled by the client-side router. Let's think of a way to do this without conflicting with API endpoints later.


@Kevin-Wei change subber to a meaningful name.


Minor: you can put this code in a function and timeit call the function instead..


This is actually a pretty decent fix. LYC-STAT IRQs for Line 0 are actually triggered during VBlank. It seems that Line 153 is technically Line 0. At that time the LCD has reached the top scanline, but it waits 456 cycles (the time it takes to process an entire scanline) before exiting VBlank.

So, that is to say, the timings were off. LYC-STAT IRQs for Line 0 get a sort of head-start before the LCD enters Mode 2 (OAM processing) and before VRAM becomes inaccessible. Several commercial games demonstrate this behavior such as Tomb Raider and Aladdin on the GBC, and Kirby's Dreamland 2 on the DMG (1st scanline of Rick's head didn't work because of this). Others probably rely on this behavior.

Once again, these kind of things are explained absolutely nowhere (maybe in the source code of other emus? certainly not in more formal documentation).


Are you not deprecating the "space" attribute at least? If so this needs some comments!


**This is dangerous to wait for completion here!**
 What if you're trying to thread-shift being in the lebevent thread already? This can be due to finished collective. I'm seing this in our test suite.
You won't give libevent a chance to progress because you're stuck in the current event.
This is what I'm seing for our test suite (both "simple" and "regular").


No, it seems:

E   SyntaxError: unqualified exec is not allowed in function 'rhs' it is a nested function (integrate.py, line 151)


This also needs some comments


left join will work. Thanks.


Agreed.


On Thu, Aug 27, 2015 at 08:03:13AM -0700, John A. Bachman wrote:
> This is nice fix of the inline issue. Just curious though, under what situations/scipy packaging scenarios does scipy.weave fail to import?

It was missing on some arches (arm?).


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.472.27a54b71a38b980a464e4792d5d0ded6349e48bc](http://build.palaso.org/viewLog.html?buildId=62009&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 137 Build time: 00:07:07


Consider moving this model to `core` for parity with the backend.


Here we do a factor 50 amplification attack! :-)

On a more serious note, this will be fixed in the future with the multi-chain community and preference for well-behaving peers.


@hoholee12 #titandev :Cannot join channel (+r) - you need to be identified with services

lol


haha that works too. I was just about to move the function def'n above out of the callback


Any reason why we aren't using variables for these?


@mpinsky the folder `ai` is unzipped, but it only contains zip files. Are you suggestion this for space-saving reasons? There isn't a significant (if any) space-savings from zipping up zip files (I guess you're compressing the small metadata files, but those are so tiny, and all the other files are .zip, so they're already compressed).

I'm pushing back on this b/c I don't know if my `read.zip.csv` function will work well on nested zip files, and I don't want to futz with it if I don't need to. And, to be clear, my new approach is relying very heavily on these folders  existing ... i.e., I need the contents of `data_raw` to stay the same (a folder for each region, and inside the regional folder there are .zip files named only by date, and that name has to be in the current format (but the dates can change)).

Repeat, I am relying heavily on this current setup being a stable organization. If anything changes, everything will likely break. It does, however, expect new files to be added etc. Just naming patterns and directory organization need to be the same.


This commit is being reviewed as a part of review issue #91.


can this be const?


availability returns "availability", as its getText() is the same as we use in the REST-path. How could you compile this with invalid type? javac should catch that, this isn't String -> MetricType.


Yes, I was already thinking about something like that as well. We could add a `boolean isAvailableOnPlatform()`method to `SupportedGroup`which would try to initialize a `KeyPairGenerator`with the corresponding `ECGenParameterSpec`instance for the group's name... I will implement this and push the branch so we can review and discuss, ok?


[Done](https://github.com/symphonists/html5_doctype/pull/17/files). Thanks.

Semi-related, I did take a quick look at doing HTML5 without this extension, but there were references to problems with textareas or something so I backed away.


Where's this comment come from? There's no bootstrap here.


ok :)


I'll ask the compiler.


Also, we plan to add: Alert Center


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 41](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=125&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 02:54:28


Nice try, but I don't think it is better.


Great thanks


You'll need a changelog entry for the version bump here as I don't see it in the parent PR yet. We'll also likely want to merge this one last after the other 2 so we have only one PR with a version bump :smile:


It is split in 2 parts setup + stmt.
possible to setup in a fucntion ?


Brilliant logo…


@schrepfler The current means of configuring both query and collector should be considered throw-away. The main launcher (binary) needs to be redone as it depends on an ancient system (ostrich) cc @wadey


I'd like to figure out a better place to put the current user instance at some point. Settings are usually configuration points, but the user is more of an application state property.


Shouldn't we remove this -webkit prefix? The autoprefixer should handle it for us. At the least move it above the standard rule name.


There's the race condition again.. would it not be easier to have the thread serve forever and listen for an RPC command to shut down?


Right, for virt-install it has to use a partitioned disk so disk_size is the sum of all the partitions.

filesystem images are not partitioned and don't have swap, etc. They are only /, so in my tests it was working fine with no-virt (4.5G /) but the same kickstart used for virt was 4G because of the 0.5G swap space. This change will make sure the / is consistent so you aren't surprised when using the same ks for virt and no-virt.


Actually we have a separate user app on the backend (since there are views associated with it).


We can live with that for moment.
The 2 problem with ObjectProxy/DeferredObjectProxy approach:
  * it duplicate tyhe pyqtgraphg stuff
  * we must be carefull to not loss compatibilty with other language.

If all __getattr__ __getitem__ are proxied:
nodegourp.nodes['mynode'].outputs['signals'].configure is super good in python
but not compatible in javascript for instance.

Lest think about it later.


From https://github.com/gotwarlost/istanbul/issues/44#issuecomment-16093330 it seems that _mocha is what is actually running the tests and so that is what istanbul needs to instrument.


What about just doing `npm install patternpack --save-dev`?


You should revert this commit it simply stop showing my images, it is a bit specific for example is use @web this don't work fine in some project for example advanced template.

the file_exists also create some problems. Revert it please


1) Ok, but the `getSession()` mainly used to send alert will always used the established session. This means if an alert should be sent during the session resumption it will be encrypted. (with the establised session connection state)
2) This will not work because, in case we do a session resumption the peer address could change, but does it make sense to have 2 connections with sessionEstablished which have the same sessionID ?


OK


Should we do something similar to https://github.com/alphagov/whitehall/commit/640cd7da1dcf95ddf074b026ab9f4dfc5bb46b49?


Like:
```
def setup():
   ...
def stmt():
   ...
timeit("stmt()", "setup()")
```
?


slash commands? *slash commands?* cmon, man, you're better than that


How about adding patterns? The template?


Is `grunt patternpack` the default task, or just `grunt`?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.450.3e3a07049aba97d573e354bba958b7526f8fd111](http://build.palaso.org/viewLog.html?buildId=62010&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3353, ignored: 134 Build time: 00:19:41


Specify this flag to include metrics data in the snapshot


https://tu-dresden.de/die_tu_dresden/fakultaeten/fakultaet_informatik/studium/studiengaenge/master_informatik/ma_inf_zugangsvoraussetzungen/index_html/document_view?set_language=de und die dort verlinkte Ordnung https://tu-dresden.de/die_tu_dresden/fakultaeten/fakultaet_informatik/studium/dateien/sonstige_dokumente/zugangsvoraussetzungen/zv_EFO_INF_2014_05_25.pdf sind jetzt doch aktuell und schreiben B2 vor.


Bracket is not closed.


No because RPCServer._process_one() must done in the mainthread not in the listennning one.


 `XWiki.EntityReference.relativeTo` can help avoid this mess.

Example:
`new XWiki.DocumentReference('xwiki', ['Main'], 'WebHome').relativeTo(new XWiki.WikiReference('xwiki'))` creates a local reference that can then be serialized with `XWiki.Model.serialize(...)`.


I should have said "among other possibilities in the future". Maybe we'll want to have a project model that's always available.


this should be if (sessionStorage && sessionStorage.getItem("serverUrl")). serverurl is needed.


Vu les notifs que j'ai reçues, tu as trouvé :)


ok !


No need to add main.js here


I would recommend verifying your connections again between all 3 boards. The issue you are seeing typically comes from a lack of pull-up resistors on the I2C lines, or due to a lack of logic conversion. However, the sensor breakout includes pull-up resistors, and the I2C block should be converting the logic between the two boards. You should double check that the logic jumper on the I2C block is set correctly.

I would also recommend attempting to test the sensor board separately from the Edison if possible, and test a different I2C sensor on the I2C block. This will help narrow down where the issue may be.

If you're still having problems, please contact techsupport@sparkfun dot com. They'll be able to assist you if you need a replacement with either board.


Done


not_in_table = not self.project()


Yeah, the "_RESERVED" notation is used in other documents already, so we should keep it to remain consistent. Also, "deprecated" wouldn't correctly describe all the things currently labeled "reserved".


@SandeepRaparthi1 I have fixed issues and committed.


help: Specify this flag to prevent storing credentials in the snapshot


I like it ! It enforce the fact that it's linked with issues.


Yeah, the second point is difficult. I think it's safe to assume that the API will look different across different languages. So in JS it might be `nodegroup.get_node('mynode').get_ouptut('signals').configure` or something..


It seems that the Markdown code is broken if the `<target>`tag is not escaped. But into a code block I can not escape with backslashes for example...


Var dette feilen som var snakket om før sommeren? :) GET på http redirecter jo til https på tv.nrk.no?


Some comments based on a quick look:
* This feature should probably be used via --order-by=whatever rather than --whatever as a flag like "apt search --status" suggests we are searching in the status, rather than that we 'just' order by it.
* Please don't use 0 as a boolean false (in FindB calls)
* std::tie can be handy for writing comparators
* a failing test-acquire-same-repository-multiple-times is "normal". The test is rather time-dependent as what it tests is pretty much a (positive) race condition. We will have to invent something to make it more predictable… The other two should work™ – they at least do on my machine(s). We have our fair share of problems with Travis-CI at the moment through for its usage of the old Ubuntu Percise while we want to use C++11 (and preferable even the new default ABI)…
* sorting versions by string comparison is wrong as it is a lot more complex than that: 10 < 2 but 1 < 1~
* the code before the pull-request is slow in a multiarch environment (as it iterates over Pkg which means pkg:amd64 and pkg:i386 are looked at in a amd64+i386 system). After the pull request it will even show both packages as results (as a vector isn't deduplicating). We should be iterating over Grp and looking at the most preferred Pkg first and stopping whenever we find a Pkg in the Grp matching the search (I wonder if there are packages in the archive which have different descriptions in different architectures).

And as Julian already noted, whitespace changes are bad. The "3 spaces indent, 8 spaces are a tab" rule is crazy, but we stick to it for keeping our long history intact (to git blame someone else).


I am trying to make the changes in my graph but I am not able to do it. Could you please direct me where I have to go?  I have been doing those changes in my sigma.js but I didn't get the awesome feature that you got.
Many thanks


parentheses just make it harder to copy this url


@houtanb Could it be that the manual entry is still missing?


fuck off


For this scenario wouldn't you also need another folder for the other target frameworks such as lib/net45/System.Bananna.dll?


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1280 of 1280 tests passed)

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


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1797 of 1797 tests passed)

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


@jgarboza813 DWMR. Good work here. :+1:


@bgerstle Not currently - because a search result doesn't have a site property so it can't generate a title without help.

Thought about injecting the site afterwards, but that makes the MWKSearchResult mutable and I didn't like that either.

It could sit on WMFSearchResults… since it has the title. Maybe I can try that.


OK


You can pass parallel bash functions and a makefile would be quite nasty
and an extra file to include. I think it will make the code a bit more
readable actually.

On Thu, Aug 27, 2015, 02:06 lethalman <notifications@github.com> wrote:

> Honestly builds are so fast it will be useless and hard to read. Btw if
> you are really going to use parallel, maybe it's better to even write a
> makefile instead.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/75ed15e174996139e439fdca407f03fadfe929da#commitcomment-12921494>
> .
>


langages en français :)


Good to know! Thanks.


Uhh.. can I ask why this is removed? it's basically causing the entire _Completed_-UI to not react?


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Consider changing to `user.isAuthenticated()` or something to that effect.


In 1.6.4 this one was called: inferToAsym


So would that change this block to:
```
def after_sign_in_path_for(user)
   user_path(user)
end
```
?


I am wondering. Should I remove the prefix from `font-smoothing` as well? WDYT?


I am afraid it is not that simple, specially when the value is read from the user's input.

This will create the usual problems of space name vs space reference for unescaped space names that will be handled as references.

The only thing we can do is to deprecate this constructor and still support it, but with the mention that it can only be used to construct top level documents, i.e. the `space` variable can only hold a top level space name.

IMO, any improvements we want to make to initializing from a (page, serializedSpaceReference, wiki) group should be made in `entityReference.js` so that we always end up passing the reference to XWiki.Document.


Consider changing to `router.go('/login')`.


Thanks. However, please note that the text is still at an early stage. It will undergo several reviews (and revisions) until the end of tomorrow. Hence, I'd recommend you to defer your review until then.


If you talk about incremental analysis, I imagine that this method will also be called on type VIEW, no ?
As there's not always sub view, and a view can contain some projects.


I'm open to suggestion - how do you propose to support a blocking function call to Get without blocking? Can you provide more info on how a finished collective is impacting here? Is the collective for some reason calling Get while in the callback (that doesn't sound right)?


The number of dimensions is part of the type.


This issue affects ECDSA too. Instead of removing it, change it from DSA to ECDSA.

Apparently, lack of randomness in the ECDSA "k" param is specifically what was exploited for the 2010 Sony PS3 hack.


Consider moving this redirection logic to the controller.


No, in order for a FillerSet to be global it must be in a filler- file. Also, i was supposed to rename FillerSet to shell, but forgot.


Si, le pregunté a Hector y me dijo que mientras siguiese la misma estructura que el documento "plantilla", no había problema en hacerla en LATEX.


You can add the if else statement to check if neighbour is 0.


This is just topic, not topic.name, I think


@benrudolph fyi


Not easy to be backward compatible. Deprecate the constructor is the only solution I think. I hesitated to call it "legacyInitialize" or "deprecatedInitialize"


@bnoordhuis this isn't correct anymore. :)


avoid the `cp`


do not change root password everytime, just if the root password is empty


@bgerstle So we still do need some concept of an app site because:
- We we need a site to use for nearby searches
- We need a site for "Today" and "Random"

This seems scoped to Home (for now) - but I am not refactoring this out of the settings at this time because:
- Thats not in scope for this ticket
- We need to decide if we want to move the UI for changing the site to the home screen - and then design would need to whip something up for that (I'm inclined to do this option, it just hasn't been discussed yet)

So basically yeah… we can do something better… but lets wait until we have the final resting place of configuring the "Home Site" decided before we start to refactor.


this name could be `vehicle-locations` perhaps? (definitely don't need "demo" in here)


We are.  You're looking at the compiled CSS. :)


you missed the N, something like `</dev/urandom sed 's/[^A-Za-z0-9+_]//g' | head -c 10` is more readable


see comment for yaml on the other side.. maybe change to `vehicle-locations`


@Ninikitos There are missing files in this commit. The xcode project file is missing for certain, and there may be more. If you can add the missing files then I know I can help you.


Good point


Although not really needed, I find this very convenient for checking which species are assigned to which clade and how many of them end up in the clade tree. Further, one could now go into a specific clade directory and do e.g. the clademerge manually.


shouldn't need the connectionFactory


avoid the creation of other files in the rootfs


Wups.  One documentation PR coming up.


@chalkie666 Still on the PR thread, yeah. Because that commit was once part of the PR's branch. GitHub never forgets! At least in my experience.


:+1: to this commit too.


yep  - agreed - lets talk soon and evaluate the other fetchers to work out the requirements - maybe at code review this week.


@snoyberg, you might want to check that this makes sense to you.


I remenber : setup is templated with some kargs (compression, protocol, transfert mode)
I prefer this.

Didi you try to run it.

On linux theplaindata+ tcp+localhost is as fast as the SharedArray approach.
The plaindata+inproc is always the faster. At least on my PC.


should be auto-configured


Awesome!!  @dukedorje


No problem. I realized after making the first one that I was reading a draft.


"service" or "fleet"?


@BrandonStrevell Please ignore this meaningless merge per 4c416b8220d9fa10f7324b0ee2c8f7abf414fb81


@briankalwat If the Attribute is configured as one of the Rock Audio or Video field types, it will get generate the full tag as the item.AudioUrl, which would include the js-media stuff.  It looks like you should just change the attribute type to text, then the Lava will do what you want.  (Before the fix, even the text attribute would have been affected by the js-media plugin after the above Lava was applied)


I know this approach works when fetching items but am not sure if it works when creating or updating since the field be found by the framework,  can you test this on a 1.10 instance?


"service" or "fleet"


done


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: personal_collection


23:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
26:1: I100 Imports statements are in the wrong order. from invenio_search.api should be before from six


I'd prefer using an `assert` there, having an exception throw an exception in production seems dangerous (even if it should never happen).


This won't get things going.  You still need to configure the source and build locations


Fix #217


MIT License


I know, I meant the install path where you `npm install` in a project, and then manually configure that stuff. It's probably worth documenting.


nice thanks


NAL1, NAL2, NAL4 are working as they did. TODO: NAL3, NAL5, NAL6, and then let's finally go temporal. ^^


@mikepetersonccv Right, and i'm saying we pulled in the fix, changed it to text and it appears to be still wrapping it.


When i originally built it i had it fail loading the file if the block wasn't found, in order that the error would be noticed. If the block is just skipped, and a modification was made, then it could be a very long time before the error is actually noticed, since it could be a very small detail. Requiring the mods tag makes it easier for us to tell when a block is expected to fail and when it isn't expected to fail.

I think we should undo the change you made removing the throw when a block errors.


`grunt` -> `grunt default`
`grunt default` -> `grunt patternpack` but only if you setup a task to do this.

I thought it was a little clearer to describe what task actually runs


that means i can get rid of the one in `south_migrations` right?


This might be fancier using [map()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map) and [join()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/join):
```
var spaceSegments = this.spaces.map(function(spaceSegment) {
  return encodeURIComponent(spaceSegment);
}).join('/');
```
:)


For reference: "Related to KrauseFx/deliver#316"


Is this sleep temporary ?


Is there nothing that should be set as a default? Not even the step name? I agree with @nnairftb that if there is nothing in it, is there something that relies on it being there at all?


Bueno pues como no me da tiempo a hacerlo a mano todos los cuadritos y todo, borro el repo y lo hago en .doc :/


That won't work with GlotPress (strings will be trimmed), so we should add space chars programatically around the word "and".


fixed


I guess as the whole class is soon obsolete. Stupid question...


@briankalwat looking at the rendered output above, it looks like it is putting the full tag in the src property.  I wonder if the page is cached or something.


Maybe we can extract this in a method to avoid duplication and passing the separator as a parameter.


What do we think about getting rid of the many many many lines of commenting code that the example step placed in there to begin with?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.451.27a54b71a38b980a464e4792d5d0ded6349e48bc](http://build.palaso.org/viewLog.html?buildId=62011&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3353, ignored: 134 Build time: 00:21:37


This case could occur but the rxe_add_index must called before rxe_srq_from_init, because "srq->pelem.index" initialized inside rxe_add_index and used as srq_num inside rxe_srq_from_init.


I had considered that, but was trying to strike a balance to keep the getting started small.  We do have plans for documentation around creating a pattern.  I was going to add that and see how big it got.


I don't know that you need to change the first line - your paragraph is enough.


если это пок, то persLink.nodeName === 'B' >>> persLink.href === undefined


Okay. Thanks for the info. I will see what I can do.

This branch is an experimentation with the purpose to propose merging code from vcmi and vcmi_editor together.

Since few people know pascal (myself included), it would be beneficial to share code in a single VCMI_Lib.


yeah


Теперь в куклотреде у каждого сообщения будет предпоследний коммит с которого взят скрипт.


Wau, thanks ! This will help many people, using "poorly" maintained machines


*commit changes reverted: #976eca8e41e944e31a582d6b0ca9ea30a19c3b48


Не забудь выполнить `npm update`.


updated


Yes, it's super temporary, as this will all get thrown away once https://github.com/KrauseFx/deliver/pull/259 is ready :+1:


Hello @pageauc thank you for the kind words. I just got into my office, eager to build that arm binary for you, but looked through my cables and realized I forgot my ethernet to thunderbolt cable (it connects my Pi to the network via Macbook) at home. However in about 9 hours, I'll get back home and get it up and running.


Same as SRQ rxe_add_index must called before rxe_qp_from_init to initialize qp->pelem.index which used to init the qp_num inside rxe_qp_init_misc


Get is not related. I've commented in different place.
The problem appears with fence.

четверг, 27 августа 2015 г. пользователь rhc54 написал:

> I'm open to suggestion - how do you propose to support a blocking function
> call to Get without blocking? Can you provide more info on how a finished
> collective is impacting here? Is the collective for some reason calling Get
> while in the callback (that doesn't sound right)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-mpi/pmix/commit/2cdb930a1e0586c3592741cb25c90563b794e0f1#commitcomment-12929686>
> .
>

--
-----
Best regards, Artem Polyakov
(Mobile mail)


This is a welcome change since getResource was quite restrictive.


@amaseda Wooooo!


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.29.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62014&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:39:00


The only issue is that this needs to be non final so that it can be overridden by the `InterceptingTransportService` in the `IndicesRequestIT` integration test suite. Otherwise, it's a thing of beauty. :)


should be 5 spaces before "http:" here (we all should update templates; this crept in at some point and has proliferated)


This was the most profoundly prolonged research process. The docs for Middleman are garbage. Furiously scoured the internet for a method that ended up being buried in the [Sitemap](https://middlemanapp.com/advanced/sitemap/#accessing-the-sitemap-from-code) section. Feeling better, now.


develop (axelstudios) - 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)


As a FILE is also a LEAVE, I would tend to not create a UT on the Views tree, as you're testing twice the same thing, and maintaining both tests will generate some additional cost.


Fixed the commented-out code (src/main/java/org/ambraproject/article/service/XslIngestArchiveProcessor.java).


@bgerstle good catch on about the possibility of a nil location, updating that now.


create a login(final String username, final String password) and perform some authentication for dummy and return true or false


`find_resource_by_path` is the final piece in making Middleman a suitable alternative to (ahem) something like [Kirby](http://getkirby.com/)


Doesn't matter, you can see the contents of the commit...


if view is empty, you're right, it can happen


I guess we don't need this.


I tend to thing that this test class could be removed, as everything is already tested in ReportUnitTestMeasuresStepTest. Or you could have only one test to be sure that the step is well executed on a view, but without testing everything.


'<module rename-to="net.mostlyoriginal.ContribPluginProfile">'


In fact only one test could be done on a views tree -> Check that's well executing, but not testing every use cases


This is git getting confused by removing and adding classes.


newline


since these are by definition *launcher* properties, can we keep this simple: "aggregate" (or maybe there's a better term.... I was thinking maybe "colocate" but that's not quite accurate since regardless of this boolean value the modules are launching within the same process).


если это пок, то persLink.nodeName === 'B' >>> persLink.href === undefined


Perfect, and well done for deciphering my confusing instructions!


Line 1846 is of interest.
Sorry that I posted a comment in a wrong place :).

четверг, 27 августа 2015 г. пользователь Artem Polyakov написал:

> Get is not related. I've commented in different place.
> The problem appears with fence.
>
> четверг, 27 августа 2015 г. пользователь rhc54 написал:
>
>> I'm open to suggestion - how do you propose to support a blocking
>> function call to Get without blocking? Can you provide more info on how a
>> finished collective is impacting here? Is the collective for some reason
>> calling Get while in the callback (that doesn't sound right)?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/open-mpi/pmix/commit/2cdb930a1e0586c3592741cb25c90563b794e0f1#commitcomment-12929686>
>> .
>>
>
>
> --
> -----
> Best regards, Artem Polyakov
> (Mobile mail)
>

--
-----
Best regards, Artem Polyakov
(Mobile mail)


Where these 'names' come from, @red-trela ?


indentation was correct before


permittedCrossDomainPolicies and contentSecurityPolicy though added dont play any role at the moment it seems


#466 hopefully, we have a nice plan out sorted before october, but in case
not, we could have a working session on it.
https://docs.google.com/document/d/1nFFoIFKsaL5u_XV5k3VpXzFvM2fr2ysYgmKSlFQNJi0/edit#heading=h.jtynkol5bppc


comment: this addresses the special case when the shared channel is of a different type than the one exposed by the proxy


Looks good, I'll probably add an extra abstraction later so we can stack strategies for different plugins.


We might have to check the other parameters too, since the constructor could be used as `(null, 'someSpace', 'wiki')`.

So `(typeof pageNameOrReference === 'string' || typeof space === 'string' || typeof wiki === 'space')`.

Does that make sense?


It is general way as I understand (I saw it in nova).
ec2api.conf.sample used only in install.sh - it was copied to system etc folder and configured.
another way of installation - devstack plugin generates ec2api.conf from sources right in installation.
conf.sample is not a good file because we can't support it in actual state. it is depended on third party libraries that can be changed without our confirmation.

Do you really need sample conf?
Or maybe you need some document with available config options for ec2 api?
Or maybe you need something else?
We need you opinion to be better :)


from here call LoginService#login() method and based true or false return the user to success or failure pages (success,jsp and failure.jsp)

@lavanyakomarasamy


Same remark as before, as the code is the same than the report test, I would only check that the step is well executed on a views tree.


@rakaramos shouldnt it be:
Nevertheless, `UIMotionEffects` is still overlooked.


which UT are you talking about?


This could have been done in another commit :)


I'm fine with the GDX dependency outside of core, still serves most users.


Add @Service("loginServiceImpl ")


Must be a better way to write this...


Yeah. I had it as 'aggregate' at some point and then somehow I convinced myself it's too generic.


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


Add @Service("regUserServiceImpl")
@lavanyakomarasamy


Any particular reason for the renaming?


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


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


@juansanzone Fixed in 0c58f41abe257401a0ec73e7bacd0a4fd819f19f


does this work with GWT? We'll have to check. Can take care of that!


why not use an enum ?


follow jsp coding convention the js file name must be success.jsp not Success.jsp


this test varies enough from ReportUnitTestMeasuresStepTest to exist. eg. it tests that no measure is added on PROJECT_VIEW components
and in fact, it's too much similar to ReportUnitTestMeasuresStepTest, the tested tree does not contain an empty SUBVIEW nor testes behavior on empty VIEW


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


As usual I don't get this at all since it's totally ad-hoc and fragile, but as long as it works for you... :D


What the difference?????????????


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


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


@bgerstle Just checked that out and it doesn't look great either… really doesn't solve any "dry"-ness.

Also to be honest, this logic really not that un-dry… each controller is getting the title using internal knowledge of the results it has. This is logic is similar but slightly different in the related section controller and the nearby section controller.

So basically the only other viable option is the one you originally brought up: putting the title logic in the search result itself. But as I mentioned earlier, requires it be mutable since the site isn't returned in the response, and then setting it in the fetcher, and I would say this is a worse trade off than having similar logic in 2 places.


Voy a revisarlo


LGTM!


nop, this test should be bullet proof, so that any change to the production code made with the report only in mind would be ensured to not be incompatible with views


good idea.


It make sense.


Ah, I see - yeah, that doesn't look right. Let me poke at it while you are off tonight


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 44](http://teamcity.otenv.com/viewLog.html?buildId=1480352&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 34; exit code 1 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 43](http://teamcity.otenv.com/viewLog.html?buildId=1480351&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 34; exit code 1 Build time: 0:0:0


:heart:


Code is very similar between this method and previous one, you could add a private method to share the behaviour, something like :
```
private static Iterable<Period> supportedPeriods(Component.Type type, List<Period> periods) {
    if (type.isReportType()) {
      return periods;
    }
    return from(periods).filter(viewsRestrictedPeriods());
  }
```


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.449.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62021&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:23:08


slider was refactored in https://github.com/enyojs/moonstone/pull/2424  I think we should be able to simulate the same behavior and work on both pre/post if we position relative to the slider itself.


> , but there were references to problems with textareas or something so I backed away.

Never had this problem, unless you render using xml (which will I never use), because your textarea (and other empty elements) are rendered as empty, self-closing elements

```html
<textarea />
```
which does not works.

[Look at how I've use it](https://github.com/DeuxHuitHuit/symphony-2-template/blob/master/workspace/utilities/master/master.xsl#L34-L43)


Add method level comments so that we understand the what is the responsibility of the method
@lavanyakomarasamy


Nice, I only learned recently about embedding. Thought it would be useful here.


I am a bit confused by this pattern.

If we expose this generally, how do we know which callback is for which method call?


@PieCrafter314 Thanks! I just shaded them and added a little more shape. I'll get to work on the soda thing as soon as I finish revamping bridge blocks(again).


Oops; I missed one: "Excution" is still in there somewhere, as it shows up on the server console output.


@dmatteo I'm curious why the last assertion here would fail: https://gist.github.com/marr/9e3f77e978b6aac132cd. Any hints?


Here also should be a comment: `# amazonaws.com is the official download host per the vendor homepage`.


@bgerstle updated `fetchNearbyArticlesWithLocation:` to address the nil location issue.

Also just to answer the other question, it is intentional to only refetch… not fetch - the first fetch is initiated by the location manager.

TL;DR Unlike other fetching logic, where the fetch is initiated because of user interaction, this fetch happens because the location changes. The fetch is implicit. Because the initial fetch happens in response to the location manager there is no external "fetch" method. You can't force a fetch until the location is received. You can only tell the location manager to start updating, and maybe you get a location at some time in the future.

All of that is to say, that the logic in the here is necessarily different than most other places.


Should the return type be "[size_t](https://github.com/akumuli/Akumuli/blob/c17dbc26f830797a55fda18264df0bce6c2074fa/include/akumuli.h#L287 "Check function signature for aku_cursor_read()")" for the member function "read"?


How do you suggest to use `assert` here -- since we're not dealing with a boolean property?


Cool


Nice job on creating an element we did not discuss in class here.

* Instead of using br tags to add spacing to the top and bottom of your hr element. We discussed this last class so I'm sure that makes more sense now.


Cool :v


Examples can be found here:
https://gist.github.com/JanBessai/6da2e5ca1e1196c6d8b7


:trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:  :trophy:


upload first file named README.MD


Let me know if that works.


Is it actually spearphishing or just phishing?


In the other commit you said QA explicitly requested that "no qa" shouldn't be removed ;)


I guess this is ok, but there should be a defines section outside of the OS checks to apply defines to all platforms


Looks like outerClass is not used by any handler except `normalKey`.


Battery may suffer if the max value is set 2000 as our battery is non-removable . Just a query


Typo here: `use_cassette`


The above 6-line block appears many times throughout the tests. Would suggest refactoring into a single setup function and using `@Before`.


Ah, I see from the definition that `jl_array_uint8_type` should probably be called `jl_vector_uint8_type`.


More generally, similar stuff should be handled ideally by the same code that in main SF app, so that the behavior doesn't diverge. You shouldn't reimplement the same (or even similar) functionality at both places.


I was trying to implement [event-based async pattern](https://msdn.microsoft.com/en-us/library/wewwczdw(v=vs.110).aspx), but not quite. It should look something like
```csharp
client.GetTimeCompleted += ...;
client.GetTimeAsync();
```

But this is not inline with other client libraries. That's why I chose the current naming
```csharp
client.TimeReceived += ...;
client.Time();
```


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


There is a double check here, no reason to check value == NULL twice


не, лучше так:
```
language: d

d:
  - dmd
```


Hello @rafaelfranca :ribbon:

This commit completely removed the version of `sprockets-rails`, so both version 2 and version 3 could be used.

However, when I tried to update an existing Rails 4 app to Rails 5 (master), that resulted into `sprockets-rails (0.0.1)` being installed! :weary:

I think we should at least keep the minimum version requirement `'>= 2.0.0'` as initially suggested by @arthurnn in https://github.com/rails/rails/pull/17752/files

Is that okay? Should I open a PR for that?


See http://hub.qgis.org/issues/13277


Hello,

I installed kibana 4.1.1 and I have the same problem. This patch don´t exist:

src/ui/public/agg_types/AggType.js

The patch is: /opt/kibana/src/public/components/agg_types, but don´t have this file. I have looked for file in server, but I din´t find it.

Could you help me?

Thanks


Thank you for explanation.

> Thanks for the heads up on the proto3 support. Are you working with anyone here at Google on this?

No, currently I'm working on it alone. Need to finish some very basic things and release alpha version for testing. I use [protobuf google group](https://groups.google.com/forum/#!forum/protobuf) for questions. Feng Xiao answers quite quickly :)


I re-thought about that :
1) Does it really useful to keep the previous established session ? I mean what is the purpose ? I not sure a client will send application data as the same time it try to do a rehandshake. If this is to prevent from a malicious handshake, I'm not sure we are able to do something. We avoid the IP spoofing attack with cookie exchange, so we face a man in the middle attack. In this case prevent the session termination seems to be a really complex challenge. And probably useless as it could filter packet if he want.

2) There must be only one connection in the store with session established. This means when the session resumption is done (after Finished Message), we must remove the previous connection from the store.

Does it make more sense ?


Will there be one build or two? eg original raspberry pi ARM1176JZF-S and
newer raspberry Pi 2 Cortex A7
I think there are slightly different instructions sets but not sure if this
is need for the new compile?  I have seen this on several applications that
have specific builds for each cpu platform.

If you post on your github I can get from there.

Thanks
Claude

On Thu, Aug 27, 2015 at 12:06 PM, Emmanuel Odeke <notifications@github.com>
wrote:

> Hello @pageauc <https://github.com/pageauc> thank you for the kind words.
> I just got into my office, eager to build that arm binary for you, but
> looked through my cables and realized I forgot my ethernet to thunderbolt
> cable (it connects my Pi to the network via Macbook) at home. However in
> about 9 hours, I'll get back home and get it up and running.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pageauc/pi-timolo/commit/8c18ec7be66d578bb273ecc24a04b6d3b9ca3ea9#commitcomment-12930079>
> .
>

--
See my YouTube Channel at http://www.youtube.com/user/pageaucp


New shortcut methods


@zont - isn't this essentially a breaking change if folks were calling the same pipeline twice?


Thanks Humeira. I applied the fix manually.


Right, so:
```
timeit("stmt(args)", "setup(args)")
```
It's not really important. I was just confused why there was so much python code inside a string literal..


Hrm that does not look right, if you look at https://github.com/InfusOnWoW/WeakAuras2/commit/6dfb997413dcafd5113f1ca87a0fe4dc91362a26 you can see that I did not even touch that line. I *did* touch it here https://github.com/WeakAuras/WeakAuras2/commit/08f83cf49543061e879381788ad6ead7fe78038c but you are somehow missing that commit?


Yes, that would be much better.


Don't think you meant to commit this, right? :)


I'm not sure why this key was "voter" since no Pupa class has a property by that name!


1. Avoid putting h3 tags inside a span tag. The h3 tag should be by itself.
2. Try to avoid writing inline styles such as "<h3 style="text-align:center">".
3. In this is situation since your listing out links, I'd be better semantically to write this as a unordered list.


The app itself is free, so the license for this app seems to be `:gratis`.


nameカラム=thumbnailモデルの旧カラム名　＆ tagsモデルのカラム名　と二重になっていてややこしかった 且つ nameだけでは何なのかイメージしづらかったので、
thumbnailモデルのカラム名を、name→image_urlに変更しました！　確か！


Yes, that may be a typo. Interestingly enough, it will still work because `A-z` is the same as `A-Za-z`, so in the current regex, `A-Z` is not necessary since `A-z` already covers the same range.


I've rewrote the action: https://github.com/tnataly/url-shortener/commit/8a27e18148f448c4c5ef174c6ebae1f4cb576840


I couldn't find another way to pass strings to c without using PChar. This is what was recommended.

Needed to typecast all the calls this way.

The unittest passed.


Using padding before we discussed this property! Nice!


Hm, this is the domainname, not the hostname...


Deleting a dup. This was a test.


Nice, ended up being not much code after all!


Hmm, it will be one build. However, you could actually compile the binaries yourself if you have the devices.
```shell
$ go get -u github.com/odeke-em/drive/drive-gen && drive-gen
```
should cut the deal.

Please let me know what you think.


Good job linking out to a page within some paragraph text. This is great practice for SEO.


You are not supposed to call mkapp yourself, maven does it. If that does not work (the way you expected), please report.


This change is going against our style guide https://github.com/cbdr/ruby-style-guide#consistent-multi-line-chains

I think that rubocop did the right thing for multi-line chaining and am only questioning if it belongs on multiple lines.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.450.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62025&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:17:14


Not sure what this tag is being used for here but it can be removed.


This seems like a good addition because showIndexes() does not definitively identify primary indexes in postgres, seeing as the index name is not "PRIMARY". For consistency and to make it even more useful, I think it should be changed to return the same format as showIndexes. In the current form, it only returns the columns, so you wouldn't be able to identify the primary key's name in Postgres. index_type could also be useful - I believe MySQL allows HASH indexes as primary keys for MEMORY tables, not just BTREE.


But in that doc, it states:

> If you use the multiple-invocation overloads, your code will need to keep track of the userState objects (task IDs) for pending tasks

I appreciate Time is a contrived example, but we will certainly need this for any methods that require a callback based on the method being called when multiple can be done concurrently such as publish. I think we need to adopt a pattern that will work across the board.


May be we can use/extend `ParentContextCloserApplicationListener` from spring-boot?


update the comment


Why the `!`?


New to Github. I'm testing the Delete in Github for the Data Scientist's Toolbox evaluations. One submission doesn't look correct. I am attempting to duplicate what they did.


It was me! Not sure what happened the the merge...


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


I think this needs to be a regular ->quote() call, because $table is being used as a string in the query as opposed to a keyword.


Centralizar a Autenticação da aplicação


Since you're writing out multiple values for the border property here, you should be using "border" instead of "border-style".


How do you imagine an external user would handle loading of leases? We currently don't provide an external way to construct a lease (apart from loading if from file, which this now removes).


Either we like it how it was or we like it how it changed.  Im not willing to do more than that in these PRs as they are already too big.  We can do a separate PR for some stuff like this


just merge my damn code, i had this >.<


I think this else can be removed now, was there for backward compatibility but now the REST is always setting `lfn` if I am not wrong.


👍

Russell Ingram
Tennessee Data Commons
(901) 289-9551
On Aug 26, 2015 4:35 PM, "Brad Montgomery" <notifications@github.com> wrote:

> you can tag old commits! [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tndatacommons/android-app/commit/82432f0343579ecc3a5aefbf022d5c8e179b3f67#commitcomment-12913850>
> .
>


Actually, it looks like 175d6d7 broke things on Node.js v10.x..


:+1: sounds fine to me


"normal" is the default value for the line-height property, therefore there is no need to add this here.


Why not unsigned? I use that everywhere else (and protect underflows with asserts).


@DonKult  and @julian-klode Thank you by the feedback. I will close the pull request in order to work on those points. As i should make a rebase to recover the originals lines from whitespace changes, avoid commits about travis being travis and simplify the pull request with only one commit by feature.


Great job overall George!. Seems like you're really getting the hang of everything. Be sure to let us know if you have any questions.


oops


I believe the current configuration is easy and functional enough, but that's me. It's a mix of events and callbacks, similar to what we have in other client libraries (if we say that events == observers).

We definitely need some feedback from guys actually using the library. My bet is to go ahead like this now and change it later when we have a reason to.


to the Orion landing page


In 6dfb997 you remove this code:

for class in pairs(WeakAuras.class_types) do
      if(db.talent_cache[class] and db.talent_cache[class][talentId]) then
        --Get the icon and name from the talent cache and record it in the table that will be used by WeakAurasOptions
        WeakAuras.talent_types_specific[class][talentId] = "|T"..(db.talent_cache[class][talentId].icon or "error")..":0|t "..(db.talent_cache[class][talentId].name or "error");
      else
        --If there is no data in the cache, just use the default
        WeakAuras.talent_types_specific[class][talentId] = WeakAuras.talent_types[talentId]
      end
end

Which does ensure that talent_types_specifc is initialized for all classes. The new code only does that for the current class.


Those quotes don't work on mac. I see the following (which made me bail out of letting it run as root):

`performing a single-user installation of Nix...
directory /nix does not exist; creating it by running ‘�� using sudo`


Travis' build has failed: https://travis-ci.org/mezuro/kalibro_processor/builds/77537093 :disappointed:


But I didn't look too long, so maybe I'm wrong.


:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:
:birthday: :cake:


Resolves https://github.com/hence-io/images/issues/1


So it won't fail silently. We wouldn't check the errors at all otherwise.


can't this be static const?


unsigned short?

Let's use uint16_t here please...


A Namespace-package should not have any code, too.


unsigned int? Let's use uint32_t instead here please.


I really wanted to keep the *Model* like function in mortgage_calculator_service, well away from the *Controller* or *App* specific stuff. I think it's bad practice to mix business functionality - with view and controller stuff (which is what my first draft was doing until I moved and refactored this). Having this in a seperate place helps avoid the temptation to mix the two.

I'm not sure where the best place to put it is - (it may be it isn't an Angular module). Some day this should get data from the server (the hard coded numbers at the top should come from a server).

What are your thoughts?


fix indentation


Thanks for taking care of this.


Just trying to eat as much dogfood as possible.


no it can't. seeing htat now...


Not sure about that. It listens to the same event, but does something different.


I believe it was due to the kalibro_configurations version not being right. I'll look at the PR and hopefully accept it, then bump the version.


The whitespace-only changes make it harder to find the changes which actually matter. Please revert them


That is true, but may change in the future. I think this was a remnant of that notion, so I'll remove it here and add it in when it's needed.


Ah yes, I forgot about that. Thanks


revert whitespace-only changes


It looks like this isn't resilient to concurrent inserts. If someone else is inserting concurrently, won't MAX(id) be a race condition?

minor things, but the other keywords in this query should be capitalized for consistency: AS, MAX(id), AS


No need to use an `init` function here, you can just do:

    var configData = make(map[string]map[string]string)
    var re = regexp.MustCompile("^\\s*([\\w-]*)\\s*:\\s*(.*)\\s*")


Hm, we don't use this style anywhere else (at least not I don't), you sure about this? If so something to put in coding-style?

FTR I always do:

`r = ioctl()
if (r < 0)
         return -errno;`


mulit


You're probably correct :smile:


gcloud fails to build: http://hydra.nixos.org/build/25136699/nixlog/1/raw


Nice thanks!


Would be cool to have a log_ipv4ll_errno() variant here, and print the actual error...


Don't use the two variable version of map fetching if you don't need the second variable. Just do `return namespaceMap[setting]`.


sounds unnecessary to reset ll->fd here, if we overwrite it as next step anyway...


This fixes #1


Not sure I grok this all, but looks OK to me, from my superfical PoV


it might be less C++11. but will `std::strftime()` + `snprintf()` be faster?


Using PChar to pass strings to C is right, the devil is in the details: you passing const PChar but modifying it.
I think test will fail if you check that initial string is not changed.
smth like:
```pascal
initial := '/pp\bin/win32\ppc386.txt';
UniqueString(initial);
AssertEquals('/PP/BIN/WIN32/PPC386', NormalizeResourceName(PChar(initial)));
AssertEquals(initial, '/pp\bin/win32\ppc386.txt');
```


`e` no longer exists as a variable, you renamed it to `event` above.


I was debugging a bug when IPcam only has H264 encoding (no MPEG4).
This API change caused a
-0:34-:: ::[TypeError: Cannot read property 'media' of undefined]
and I have to start over.... Orz


Only for clarity.


Please remove control character at the beginning of the line and provide a an useful description.


nettl2.7 fails to build, do we even need it? http://hydra.nixos.org/build/25151613/log/raw


Okay, will reduce it.


I think you can drop this use. The other one is cleaner.


If you make the configuration user writable after initialization you should explicitly document that none of your routines are safe for concurrent use. Or, since that's likely to be a common requirement, add appropriate synchronization to make it safe for concurrent use.


@rzarzynski make it log level 0 here, errors should be verbose


You're removing a newline here.


I have renamed those functions, in https://github.com/basho/riak_test/commit/5eab8d8d0c0d101b25c3b03ad0f72935d364cff3.


I'm writing the integration tests and I'll that scenario


seems good so far. does this occur on each win 8.1 platform? and everytime?


But @yavor87 with that pattern, how will that work for the publish method?  It does not make sense.  For example:

```
channel.Publish(...)
channel.Publish(...)
channel.Publish(...)
channel.PublishSucceeded += ...;
```

How will the developer know which succeeded is the correct one and relates to which event published? That is the exactly the problem that link is describing.  Am I missing something?


Usage:

 1. Add config that's in the comment
 2. Call IIS.configureApp (OK "My Cool App")


in general, I wonder how would the out of order writes affect recovery process


They don't. For `Schema.keyClassList()` I can't simply use `return Object.keys(this._keyClasses);` because key classes can exist up the prototype chain and `Object.keys` only returns a object's own properties. So, I had to choose between this method and using a `for..in` loop, which I dislike because it can pick up stray properties added to `Object`'s prototype by third-party code.

Come to think of it, that would be solved by `_keyClasses: Object.create(null)`, so I may go with that. What do you think?


I wouldn't use a temporary tensor for the example here, as the whole point is that you can re-use tensors. So maybe just create a LongTensor and use that:

```lua
indices = torch.LongTensor()
x.nonzero(indices, x)
```


Indeed, will fix


Made changes here: https://github.com/kristophM/Blocitoff/commit/823d4d29e8f717ae9240e0fc244749eb38b668c2
and here:
https://github.com/kristophM/Blocitoff/commit/8bc394de857bb1a0f9d467299a2976a5b432b7a6


The user needs to run `` eval `opam config env` `` after this, or trying to compile anything with the OCaml backend will still fail.


This means that when the hours>23 a timedelta object is returned. For Excel spreadsheets this does not happen. Instead, a datetime object is returned relative to the epoch. I think using a timedelta object makes more sense to me because the user choice to view the value of the cell in time metrics, not dates. However, this results in a difference between handling of Excel and ODF spreadsheets.


Eliot, my test isn't passing because it doesn't recognize TestFactories- any idea what's going on?


The other thing, if you care about space, is that you should order members in a struct to be from biggest to smallest. So the Trans pointer should be first, Label next, and then your bools. You're also not using `#pragma pack(1)`, which is the thing to do if you really care about space.


退回去


Sure,

we can respin the patch with 1024 and see how it goes...

On Thu, Aug 27, 2015 at 12:48 PM, Marco Costalba <notifications@github.com>
wrote:

> @lucasart <https://github.com/lucasart> Of course, my point is to chose
> the correct K 1024 or 512? Or perhaps is even better 2048?
>
> This is something we have to be aware now, because we are validating the
> patch now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zamar/Stockfish/commit/4b80422eaea2c5e7e74676caaf5baf81475c0760#commitcomment-12924424>
> .
>


As Motorola gave us a slow charger to improve battery life. :)


this is a leftover I guess?


The condition only checks to see if `_keyClasses` is `this`' _own_ property. If not, it attaches a fresh copy of `_keyClasses` directly to `this` so the one up the prototype chain isn't modified (which would be modifying a base class - a bad thing).

`this` is guaranteed to be an instance of `Schema` (line 174 & 178), which is guaranteed to have `_keyClasses` (line 89)


could you squash the restructure and the format changes into the commit of 95b7929 , so only the new timer implementation is left in this commit?


The logic behind this is that I came across two data sets which, in a single column of data, mixed seconds with decimals and seconds without decimals. This appears to have been due to a one-time change in capture granularity of the original data. Since the decimal is interpreted as a separator when splitting the date-time elements, a mix of decimal and non-decimal seconds results in incorrect separation (which fortunately fails).


Well, the old code returned "r", not "-errno"...

if you write

```
r = foo();
if (r < 0) ...
```

or

```
if (foo() < 0) ...
```

doesn't really matter too much. If you want to use the "r" elsewhere (for printing an error msg for example), then the first way is of course the right way, but otherwise it really doesn't matter...

The change I made here was really just about returning the correct error, I just dropped the `r` as side effect, because it allowed me to drop the whole declaration of r here...


these `System.out` should go away I think


woot


...really?


A regression has been reported: http://trac.cppcheck.net/ticket/6954#ticket


I *wouldn't*. It seems looking at the "self-hosted" section of the [main docs](http://google.github.io/material-design-icons/) that they want it specifically for webkit only (maybe due to rendering issues it has but not others?) However, I'm also noticing other rules such as `text-rendering: optimizeLegibility;` that are not present here.

Maybe we need to reach out to the icon team to get some guidance on what exactly is needed? Seems odd they'd have one thing in the CDN version but another in the docs.


I believe we need to make the key case insensitive here. May be using ConcurrentSkipListMap ?


This worked for me. Thanks!


yeah.. sorry about that.


for this section--how about we just style the elements within div.relatedcontent, instead of asking for more classes?

Example:

div.relatedcontent {

h5 {
...
}

}

instead of

h5.relatedheader {
...
}


How about:
```
<link type="text/css" rel="stylesheet" href="<? autov(\"/webGui/styles/default-{$display['theme']}.css\"">
```
And in Helpers.php we have:
```
function autov($file) { echo "$file?v=" . filemtime($docroot.$file); }
```


Indeed, it won't. As I said, currently it is mixed. Specifically, publishing is done like this:

```csharp
channel.Publish("test", "data", (received, error) => { ... });
```
Where the 3rd parameter is a function, which will be called depending on ack and nack. I believe this is the same way this method is implemented in other client libraries. So generally it depends on the case.


On Thu, Aug 27, 2015 at 07:18:07AM -0700, John A. Bachman wrote:
> imp.reload is also available in Python 2, so we could use the "from imp..." statement without the try block.

I didn't know that. This makes things simpler indeed.

I looked at the reload method, and it tries to remove a .pyc file.
Under Python3 the file will __pycache__ so the removal will fail
(and this failure will be ignored). So I guess that's OK.

I wonder if the .pyc removal is necessary for anything... Python will
use the .py file if it is newer anyway.


how about just a { } instead of a.bodylink {} @amcpherson @jkondis ?


 @ExceptionHandler(MethodArgumentNotValidException.class)
 @ResponseStatus(value=HttpStatus.BAD_REQUEST)
 @ResponseBody
 public MessageDTO exception( MethodArgumentNotValidException ex ) {

                String errorMessage = localizeErrorMessage("error.bad.arguments");
                String errorURL = req.getRequestURL().toString();

                ErrorFormInfo errorInfo = new ErrorFormInfo(errorURL, errorMessage);

                BindingResult result = ex.getBindingResult();
                List< FieldError > fieldErrors = result.getFieldErrors();
                return processFieldError( fieldErrors);
        }


I think that changing the -@{floatdirection} is not enough!
I many cases you need to reset (make zero) the opposite value, so we probably need to set a @{counterFloatDirection} with relevant value.


You may implement algorithm in C with some tweaks
1) declare C function
```cpp
void NormalizeResourceName(char *, size_t*);
```
second parameter receives initial length and return trimmed length.
2) declare on pascal side same way
```pascal
procedure NormalizeResourceName(PChar s, var SizeUInt size);cdecl; external;
```
3) wrap on pascal side
```pascal
function NormalizeResourceName(s: AnsiString): AnsiString;
var
   size: SizeUInt;
begin
   UniqueString(s); //(!) important if you modifying string as char pointer even in pascal code
   size  := Length(s);
   NormalizeResourceName(@s[1], size);
  SetLength(s, size);
  Exit(s);
end;
```


I see...  I should specify that the pointer is a const, then and copy the string in c.

Am I right in assuming that the pascal `NormalizeResourceName` doesn't have side effects (changes to the initial string) and neither do the functions it calls?


don't you also need to check if the contents are the same when the checksum matches?
`contentsChanged = *i.Value.Checksum != *existingInfo.Value.Checksum || !bytes.Equal(i.Value.Bytes, existingInfo.Value.Bytes)`


Why not move this if() check before the va_start()? If so you can make this shorter, because one va_end() goes away...


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


We need to add JunitTest for it


chrome://flags/#enable-npapi
把npapi打开即可
chrome44(64bit)+switchsharp+ss2.56可用


Not much, actually: It outputs some data we could add to the index in
the future => removing this option again.

Kind regards,

    Gunter.

On Do, Aug 27, 2015 at 4:57 , Andrej Vodopivec
<notifications@github.com> wrote:
> My makeinfo does not recognize this new option. What does it do?
>
> —
> Reply to this email directly or view it on GitHub.
>


> Not sure if using a similar name is ok for filenames ('_add_or_remove.html.erb' / 'add_or_remove.js.erb')

I think this should be fine, especially since these two files are related.

> I am not too sure at this point why when I click "Add", the "Remove" button renders without a refresh, but clicking on the resulting button triggers an html response, any thoughts?

So if you check out the difference between the Add and Remove button, you'll see one small difference here and that's the `remote: true` option passed into the button_to helper. This adds a `data-remote="true"` attribute to the form element which in turn tells javascript to submit the form as an AJAX request. Once you add that, you'll see in the rails server logs that instead of:

`Processing by IngredientsController#remove_from_bar as HTML`

you see

`Processing by IngredientsController#remove_from_bar as JS`

Which will result in the format.js block being used instead of format.html (which redirects to the selected ingredient's show page):

~~~ ruby
    respond_to do |format|
      format.html { redirect_to @ingredient, notice: 'Ingredient was removed from bar' }
      format.js { render action: "add_or_remove" }
    end
~~~

> I don't know if there is a specific reason for why I represented the two methods differently in routes, I'll probably end up following the complete format as with favorite and unfavorite.

Nice, your code here is more idiomatic than what I added (i.e. the `post '/ingredients/:id/add_to_bar', to: 'ingredients#add_to_bar'`). You can move that route into the member block of the ingredients resource as well.

> Any other suggestions at this point?

Tests? hahahahaha :cry: :sob: :crying_cat_face:


Note that the latest version of choroplethr (v3.2.0), makes it easier for you to override the polygon style: http://www.arilamstein.com/blog/2015/08/27/choroplethr-3-2-0-is-now-out/


Did I miss something? I can't see anything new.


@rzarzynski I'm not sure you solve much by having it sharded by timestamp. The idea of sharding is to be able to distribute concurrent work across many objects. This will just make it move the hotspots when time progresses.
Maybe just do it the same way we handle the GC? Shard by object names, and the collector iterates through all the shards anyway?


I can capitalize the other keywords.  You're right, I didn't think of that, but just thought of a way to go about this without race conditions. Look here ab3b31ce09009022d04f783f160cd1e2816f418d


Since we are in the lucky position here that we don't actually have to allocate memory, but could just return an pointer into the name we should probably do so here. Of course, this would need a clear comment, so that users understand this and don't end up free()ing the result...


I'm not sure. This is a `uint32`, which means the probability of collision is 1 in 4 billion. I think I'm ok with a false positive at that frequency


Should this also account for plain `default`? What about `search_quote_default`? Also, what about the search analyzer taking the `default` when `default_search` is not defined?


Well actually @nsoranzo ... typos in typo fixing commits.


When I test locally and I do not log in to Bloccit, I get "undefined method ‘admin?’ for nil:NilClass.”

That’s why I placed the check for @user.nil? above the other checks.


@kdkocev :(


memory leak. the returned memory is never freed... (if you make the change I suggested above, then the test becomes magically correct ;-))


return -EIO here


Note line 39, `<div class="col-md-3"></div>`.
You may wonder why this empty class (?). I want to have stable grid system even when there is no context. Left side is used mainly for mapping.


hola que onda, esto sirve solo para comentar ?


Also note that you can override the scale by setting the "ggplot_scale" member variable on the Choropleth object. This might allow you to quickly experiment with na colors that you like without rebuilding the package. You can also type e.g. "state_choropleth(...) + <your own custom scale>", and ggplot will use the last scale written.


Thanks guys. I'll analyse and test your service with our custom implementation.


Thanks @trustin !


ah para comentar por fuera del programa


cxzv


pygame codigo facilito/Juego Codigo Facilito sonidos 4 v18.py

lalalalalalal


@Anbranin this is a one-line way to solve this problem; I'm wondering which solution you prefer.


Good call! That would be more intuitive for the rest of the producers.


Sure. If any I was suggesting this as a followup patch.


kk right. sorry!


I'll be rebasing the branch this commit is in once feature/vk/spark-master-test is merged in, and ditching this commit.


Thanks to Ah___ Sh___ for reporting the bugs.


@TheBigBear gitk comes with msysgit. I'm not sure if there is a way to install it separately. git-gui should also come with it. Again, I'm probably the wrong person to ask about that -- I do all my git in Linux through virtualbox/vagrant. gitg is a gnome app and gitq is a qt/kde app and I'm not sure if either of them are windows capable. The only other client I can think of is tortoise-git


Always use angular imports


Also note that you can override the scale by typing e.g. "state_choropleth(...) + <your scale type>". The choroplethr functions just return ggplot objects, so if you add your own scale, ggplot2 will use the last scale added. Also, you can set the member variable "ggplot_scale" on the base Choropleth objects.


line 46 is closing it. Again , note the previous `<div class="col-md-3"></div>`


Is Windows Vista supported? I saw in the past a couple of issues that mentioned that Electron supports Windows 7+ (no XP or Vista).


on split /usr systems unit files can be in both /lib and /usr. You hence need to check both... really needs to be a search path logic here...


Pascal strings use copy-on-write and are managed (reference counting) objects. If string is passed by value it is assumed that there will be no side effects. If string is passed by value and not const you can change it but changes are only local to that function (COW).


Of course feel free to do the changes you think are needed Yaacov :D and submit a different pull-request.
Though I don't know, in this specific case, where are the margin-right set, so don't know where to set the counter float direction.
You're much more inside the RTL world than me :D and your contribution is greatly appreciated ;)


Following the idea of str->bytes and unicode->str, then `__str__()` should return the unicode version of `serialize()` for Python 3, isn't it?


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Why the stat()? Can't we just do the readlink() directly here? If it's not a symlink, it will fail with EINVAL, and that's as good as an S_ISLNK() check really.


@hallyn Thanks man, you saved my day!


I think 1000 might be too much. I think we usually use 100 for omap operations (although I'm not sure what we do when we list buckets).


claro


We may add package based logging levels also
<logger name="com.sinet.provisioning.tool" level="DEBUG" />


The route should be /admin/person to not be public.


@rzarzynski not sure what's the function of need_trim here. Should we just have need_trim true if !entries.empty()?


I think it would be really good if there'd be a comment here somewhere, that explains what this function precisely does...


OK, I added a commit that removes `contentsChanged`. It was being used in one place as an optimization, which I removed.


Isn't .gitconfig some well-known parseable format like YAML?


AJJAJ holiss :3 jajaj marcos cuando lo instales y clones el repositorio, 1 fijate si aparece el # hola antes del sapaceInvader y 2 modifica el codigo medieante el idle y sincronizalo a ver si se cambia


This discussion is losing me a bit.  I guess I need to read the *Determinism Theorem* that you mention?  I took a peek at it.  I'll need a while to fully digest it, I think.

In the meantime, your changes look fine.


I tried the modified test:
```
procedure TTestEditorUtils.TestNormalizeResourceName;
var path: string;
var ptrPath: PChar;
begin

  path := '/pp\bin/win32\ppc386.txt';
  AssertEquals('/PP/BIN/WIN32/PPC386', NormalizeResourceName(PChar(path)));
  AssertEquals('/pp\bin/win32\ppc386.txt', path);

  ptrPath := '/pp\bin/win32\ppc386.txt';
  AssertEquals(PChar('/PP/BIN/WIN32/PPC386'), NormalizeResourceName(ptrPath));
  AssertEquals(PChar('/pp\bin/win32\ppc386.txt'), ptrPath);

end;
```

It passes. Do you think it might be because `PChar(path))` creates a copy of the contents of `path` and that they are not in the same address space? I can't explain the second part, though...

I checked if the test fails if I fail with `'garbage'` as the string and it does.


This was a mistake


I suspect they're about equivalent, but I've no objection to changing if we think there's an improvement to be had.


In previous commits I deleted various static files and folder images. Inside images was those two and are necessary for map markers so i want to upload them once again.
I would like to squash this one with the rest.


I'm not fully convinced, that this really solves the issue about user defined interfaces.

I already used the user options to add an interface, see http://www.bernhard-ehlers.de/projects/ostinato4gns3/install-qemu.html .

In the previous implementation that interface was inserted as the first interface eth0. But as GNS3 doesn't know about that, the GNS3 created interface are numbered starting with Ethernet0, while in the VM those interfaces start with eth1. So you have an offset between the VM and GNS3 interface numbering, which is unfortunate. In my ostinate VM I solve this with some udev trickery.

Now the user defined interfaces are created after the GNS3 defined interfaces. The big advantage is, that there is no longer an offset between the interface numbering in the VM and GNS3. The drawback is, that the user defined interface has no longer a fixed number.

My suggestion is to revert this change, instead add an interface offset field in the GUI that is added to the GNS3 interface numbers. That way the special user supplied network interfaces keep their fixed numbering and the offset between VM and GNS3 can be easily handled.


Do you know either that stories without labels have an empty list, or that lodash smart about attempting to iterate over an undefined value?


лучше поставь android первой линией


Yes, sorry about that, I hadn't realized it had leaked out like that.


:fire:


лучше так
0 0 android
0 1 POWERVR_ROGUE


It's been removed, sorry.


We should really make all fields accessible via getter functions so that they can serialize this themselves. I mean, we are pretty good at that after the clean-up. The only field they cannot access yet is the private options stuff.


Ok, now it makes sense. An explicit comment about avoiding prototype modification would be appreciated though :).


It is not a good practice to commit credentials in the repository
Remove them!


:laughing:


This may be related to my inability to implement `LeToNInPlase`


clever. I think this should do it. Technically it looks like you can configure auto increment to values other than 1, but we can probably assume for any "id" related functionality in our library that the user is using the standard auto increment IDs.


Wouldn't logging the switch value be useful here if this ever does occur?


because mistake. will fix.


Sure thing.

On Thu, Aug 27, 2015 at 12:32 PM, Debbie Paul <notifications@github.com>
wrote:

> Merge pull request #6 from brymz/patch-1
>
> Fix link to openrefine.org
> thanks @brymz
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/datacarpentry/OpenRefine-ecology/commit/6e776bfbeae36e302f53600e77e62c5d31d9dd66
>


Christian, is there something else I need to do to make this work? I can send the `c` command no problem but I am not getting back the response that I am expecting. I am thinking that I need to set the camera into a particular run/trigger mode prior to sending the `c` command. The documentation states that when the camera is in "free run" trigger mode, it will start sending the image data upon a TCP connection, which is consistent with what I am seeing.

Also, there seems to be ambiguity in how to use the PCIC interface. In the documentation that I have (see `docs` folder in repository) one sections suggests crafting the messages as you have above and in another section it suggests sending commands like: <Ticket><length>CRLF<Ticket><contents>CRLF


Hi! could you elaborate how the %graph.csv has to be set up? Everytime I try to import it, my Graph only contains the entry "1" in the top left cell.


did you leave this out on purpose?


Skip is not allowed by Azure Table queries. @raizam, in your code, you actually get every single error (up to 1000), then skip, then take. In mine, I take enough to cover the skip (that must be executed after the Storage call returns) and then the following take. It's not pretty (and thus why I added the comment above, in the code), but that's how it has to be with the limitations by Azure Table Storage.

So in my code, each subsequent page is slightly less performant, but on @raizam's code, the page load will always be the same as the last page.


```ruby
fields.last.try(:number).to_i + 1  #Perhaps?
```


An exception has been thrown during the rendering of a template ("The "PumukitLDAPBundle" (from the _controller value "PumukitLDAPBundle:Person:button") does not exist or is not enabled in your kernel! Did you mean "PumukitLiveBundle:Person:button"?") in PumukitNewAdminBundle:Person:listautocomplete.html.twig at line 16.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1286 of 1286 tests passed)

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


Fixed, thanks!


Checkmarx scan results 27 Aug 2015 13:39 UTC -04:00

High
Medium
Low
Info

New
2
4
2
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

549d113ba6b69c54670a2ee2d3c2587765eb3081	@virginiacc


In one case above you have a leading `/` and in the other you don't.  Is that right?

r=me otherwise.


A few people have mentioned compile problems and that the pre compiled
binary was much better possibly due to different directives or maybe
support file issues. Personally the current binary has worked OK for me but
then again I may not be pushing the envelope.
Thanks

On Thursday, August 27, 2015, Emmanuel Odeke <notifications@github.com>
wrote:

> Hmm, it will be one build. However, you could actually compile the
> binaries yourself if you have the devices.
>
> $ go get -u github.com/odeke-em/drive/drive-gen && drive-gen
>
> should cut the deal.
>
> Please let me know what you think.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pageauc/pi-timolo/commit/8c18ec7be66d578bb273ecc24a04b6d3b9ca3ea9#commitcomment-12930797>
> .
>

--
See my YouTube Channel at http://www.youtube.com/user/pageaucp


By definition, `serialize()` is producing bytes, ready for file writing. I don't think we need the `__str__()` implementation in Python 3. And this would even be impossible for some formats which are binary by essence. Take for example the built-in file object, when you call `str(my_file)`, you don't get the content of the file, but a string representation (e.g. `<open file './README', mode 'r' at 0x7fbb29178f60>`). This is the essence of this proposed patch, depart from the `str()` thing by adding a specific serialization API (while keeping compatibility on Python 2 only).


You`ve missed UniqueString(initial);
It is possible that you even changed literal constant inside BSS.


Either way, I think, probably merits a comment. Or just writing what @Anbranin did:

```ruby
if fields.present?
  fields.last.number + 1
else 1
end
```


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.473.4f18395488d892066e720b2f231301d19211342e](http://build.palaso.org/viewLog.html?buildId=62033&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 138 Build time: 00:06:06


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Looks good to me.


sure about this? sql.commit() maybe?


Yeah I agree. The controller should just be used to bind the data to the view, it shouldn't contain the business logic. The only thing I would suggest is that are contained in the same module. A module (at least the way I do it) is like a feature. So for example we could have a mortgage calculator module that contains all the logic and controllers and views for that. Then we could have another module called mortgage application that contains everything to do with that, each one would have it's own module.

I do think this sort of thing is easier to be shown :/


This code does not compile with gdal-2.0 anymore http://www.vtk.org/Bug/view.php?id=15692


I dunno if I feel this should belong to a concerns module or something else. This can be refactored in some way that is super easy to read.


> Do you think it might be because PChar(path)) creates a copy of the contents

It shouldn`t do that - 100% memory leak.


Okay, that fails the test.


configure_obect -> configure_object


can't judge, sorry


@maxogden i don't think we can say this yet because dat doesn't yet do reproducibility for code


@Reinmar thanks for adding it as a commit. I had added it locally, and run the build, just not checked it in. I'm running  new build now, and will get it up on the staging server shortly.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.452.4f18395488d892066e720b2f231301d19211342e](http://build.palaso.org/viewLog.html?buildId=62032&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3354, ignored: 134 Build time: 00:11:36


@dizzzz How come?


@Stasik0 i have to pack it into internal header #350


Yay! I helped! :) lol


I'll stick with search as it checks the entire string while match checks
only the beginning of the string. It just makes more sense to me.
On Aug 26, 2015 12:53 PM, "Nadav Samet" <notifications@github.com> wrote:

> Thanks for sending this. Since search() is more permissive than match(),
> this may break it for existing users. Instead of this change, you can add
> re.MULTILINE as the 3rd parameter to re.match(), and use .* at both ends of
> your RE? Would definitely accept that change as a PR.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chromerobv/ansible-health-check/commit/c3c98259e570952eb4b2d157c43dcecd05dd08f2#commitcomment-12911983>
> .
>


FYI, I've cherrypicked useful commits from this branch into marginal-mixer-support and completed support for SDLMixer class.


Yes it was redundant.


Since we have `AggregateBuilder` that configures direct binding to all the modules without the use of the binder while this branch changes would address module composition but still binding to the binder adapter, the word  `aggregate` might get confused with `AggregateBuilder` as well.


I already contributed once to CMake and I'm aware of how welcoming they can be but I don't think that is something to be added to CMake. It would be nice if the Clang folks could provide such files. I think they are actually in the process of getting their CMake more complete.


hi, this commit seems to create problem for 15.05 build. failing saying no rule to make libbz2/host. (tp-link archer c7 v2).


Everything helps :smiley


Misspelled "canola"!


If you take a look at the latest source I have added an example. See commit 4a49fd2


"Laboratory" is the string used in the component's row. There is also "Recitation" and "Seminar". Probably others. Perhaps this is customizable by each school?


https://github.com/imonology/Hydra/issues/34 actually


:+1: :domain-knowledge.


I've given dfd142f2aa2c2b7790fa944edcd8abef3918f614 a go on an AArch64 dev board and it gives decent results on unittest_bufferlist too. The performance is similar to the packed structs case and the generated code looks optimal to me too.

FWIW: Tested-by: Steve Capper <steve.capper@linaro.org>


seems fine to me.


O nível tá bom hein, auhaushas. #faltaUpdatesNesseRepo


Yes, it is, but I think there are good reasons for executing the `git config` command instead of parsing the file(s) directly:
1) There are multiple possible locations for git config data, most notably in the the current repo working directory and in the users home directory.
2) The `git pivotal` command can be executed anywhere in the current repo's directory tree, so even if we only looked in `.git/config`, we possibly have to walk up the directory tree to find the .git directory.

FYI, the specific changes I made here were to support multi-word labels. The code previously assumed one label with no spaces in the label, which worked for all of the use cases I cared about at the time, but was clearly too restrictive.


Hi, can you please send me your test csv file?  If you are consistently
getting a graph with a single populated "1", then the graph file is not
found, or there was an error in processing your graph file.

To use graph variables, you have to have individual graphs as separate txt
or csv files.
Attached are some screenshots to see a sample project.
The "prepend path" feature is being used to prepend the file path to the
name of the file that has the graph data or image, inside the
VariableImporter UI.

Also attached is a sample graph.

I'm interested to know what tasks you will be using this feature for, if
you can tell me , please do for my own curiosity.

Thank you,
-_Vasily


Just FYI, m3 is getting retired at end of year.


Nutno podotknout, že žádná diskuze neproběhla. Kniha je u mě, o víkendu či v pondělí jí mohu vrátit @encukou :-)


I think we should remove AggregateBuilder - or maybe refactor it to use ModuleAggregator. Right now it just does what SCD does, in a way.


Yay, I was just going to send a PR ;9


right now i see `public class LogEntryTypes {` in the set of changes; looks quite nice i'd say


I saw from inspection that there was always an array, which was empty when there were no labels.


Why do you need this fucking \?


Yes. Yes it was.


Checkmarx scan results 27 Aug 2015 13:58 UTC -04:00

High
Medium
Low
Info

New
2
4
2
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

549d113ba6b69c54670a2ee2d3c2587765eb3081	@virginiacc