Cool, thanks for the improvements.
I'll be really interested to see when we can troubleshoot our slow build!

On Sun, Aug 2, 2015 at 1:51 AM Dieter Adriaenssens <notifications@github.com>
wrote:

> Hi Alex,
>
> Happy to inform that I deployed two small improvements to production :
> - auto-refresh : add ?refresh=<refresh_rate in minutes> to your dashboard,
> fe. :
>
> http://buildtimetrend.herokuapp.com/dashboard/angular/angular/index.html?refresh=10
> (currently 10 minutes is the minimum refresh rate, due to caching of the
> query results that supply data for the charts), more info [here](
> https://github.com/buildtimetrend/service#dashboard)
> - the average buildtime metric on top of the dashboard now shows minutes.
>
>
> > - SHA, I assume you mean the commit-hash? Definitely a possibility to
> > > group by commit-hash, or by build ID? The chart might get overcrowded
> for
> > > longer timeframes (month, year), though, but should be OK for week or
> > day :
> > > buildtimetrend/python-lib#102
> > > <https://github.com/buildtimetrend/python-lib/issues/102>
> > >
> > Yeah, I'd like a zoomed-out view at first, but ability to zoom into
> > individual items from the 'Build History' tab in Travis. It's a
> 'travisty'
> > that they don't show this (instead they show cumulative time including
> > sitting in the queue)
> >
>
> I'll see what I can do. I'll create an overview chart grouping on build ID
> and figure out a way to display build job duration without overcrowding the
> chart.
>
>
> > >
> > > -
> > > - kiloseconds : it's on the todo-list to display in 'xxh yym zzs'
> > > format, just like the badge in the top right corner :
> > > buildtimetrend/python-lib#39
> > > <https://github.com/buildtimetrend/python-lib/issues/39>
> > > - travis buildstage labels (script.1) : as far as I know, the Travis
> > > stage names can't be customized. But there are other possibilities. The
> > > chart could show the executed command (fe.
> > './scripts/ci/build_and_test.sh
> > > ${MODE}' in case of 'script.1'), is that something that could work for
> > you?
> > > Another possibility, is to generate custom timestamps with labels
> during
> > > the build (that's how Buildtimetrend recorded timing data before Travis
> > > started adding timestamps to their logs), but you'll have to add those
> to
> > > your .travis.yml file and the script that parses the logfile has to
> pick
> > > them up (currently not supported).
> > >
> > I think I'd like to just emit lines into the build log in some format you
> > recognize, and have these shown instead/in addition to the times that
> > travis writes. I imagine I would need to write startYYY/stopYYY pairs so
> > you know the duration, you can't assume that a particular build step has
> > finished the same moment the next one starts.
> >
>
> Sure, I assume you'll want to add some timestamps inside your
> build_and_test.sh script?
> By the time I can support this, I can provide you with a script that
> generates the timestamps in a format that the script recognises.
>
>
>
> > >
> > >
> > > Currently the charts show averages of builds during a timeslot (per
> hour
> > > on the 'day' timeframe, per day on the week and month timeframe), so
> > > digging into specific builds is not possible at the moment, but
> > definitely
> > > something I'd like to add.
> > > Would a chart with maximum times be useful?
> > >
> > Actually the easier thing for me is, I know the dart stable build is the
> > slowest, so if I could just select a single matrix build dimension at the
> > top, and all the graphs narrowed to show just that build, it would be
> good.
> >
>
> Good idea, filtering on build matrix items. I'll create a Feature request
> for it: https://github.com/buildtimetrend/python-lib/issues/103
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/30bd2a3fc72f8f1cdb0ffc4b9d11b4ca42e9ece7#commitcomment-12482354>
> .
>


"SerDe" according to the Hive wiki


"Output format"


Added content from Sandrino's suggestion.  Added content, example and links to show that scope and expiration of JWT can be altered in Rules


"Output format %s with SerDe %s is not supported"


Make query number a dropdown box, make it multiple of 10


That's the method I'm going with now. Seems to work out ok so far.


@rlf Was already fixed in a more recent commit.


5058-fix-readvars-cli (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)


Subproject commit 9c9912f64e994af45a9adbb6306d2a101f2cbbe3
 1  +Subproject commit dfbc2379064ca0fad5f2413f84134d086956a794


Done.


the point isn't "< 0 is better than !"... ! is wrong, it was using it to identify an issue.

it could be "if (sml_process(sml))" , but it seems a bit non-intuitive, since we have some functions returning true / false, it could seems like, if "sml_process is true". Writting sml_process() < 0 makes it clear that it's in case of issues, imo.


Hi guy


Is it guaranteed that the image is RGB? I don't remember all VTK specific on all platforms, that's why asking.


Removing snapshot completely seems overkill. Limiting the webUI to a single folder would make it quite unpractical to use in many cases. I would say that a reasonable solution for now is to add an option to disable the snapshot feature. This option should be disabled by default. A warning could be issued when enabling it (or just be shown in a tooltip).


Actually, I don't see why the check is required here. `getHiveStorageFormat()` already checks and throws `PrestoException`


I need to either
* A) start training on history (with disregard of additional features)
* B) transform color (feature) into a number and add it to the training set


@cjmcgraw  comments?


```java
Map<String, Object> properties = ImmutableMap.of(STORAGE_FORMAT_PROPERTY, getHiveStorageFormat(table.get()));


Sure - but I won't be able to get to it until this evening.

On Mon, Aug 3, 2015 at 12:57 PM, Mark Trostler <notifications@github.com>
wrote:

> @donato <https://github.com/donato> will you make a pr for that? If
> args.real_mobile is not set do not set it for BrowserStack? Thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/karma-runner/karma-browserstack-launcher/commit/85c4764199c9c78c7a725385d055b9d1293f6272#commitcomment-12497823>
> .
>


Añadido, gracias por la aclaración.


BuildDir is known at primary builder ninja generation time, it is only unknown in bootstrap build.ninja.in.  Could the private bootstrap.binDir ninja variable reference $buildDir, but then define a bootstrap.BinDir go variable with the value of BuildDir embedded in it?


Please, use blank lines to separate parts of code responsible for different things.


Why pass fast pv list as names instead of `StorageDevice` instances? It seems like you're open to problems with things in `/dev/md/` or `/dev/mapper/`, or is that not possible?


I would think about writing `class vtkImageMatSink`. This would give better convenience to reuse in future.


yo creo que veo bien... :) tu version final es

extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
      ciaaDevices_deviceType * device, uint8_t const oflag)
{
   ciaaDioDevices_deviceType * dioDevice =
      (ciaaDioDevices_deviceType*) device->layer;

   ciaaPOSIX_assert(dioDevice->device->open(path, (ciaaDevices_deviceType *)device->loLayer, oflag) == device->loLayer);

   //ciaaPOSIX_assert(device->open(path, (ciaaDevices_deviceType *)device->loLayer, oflag) == device->loLayer);

   return device;
}

esta el assert comentado, y el device en vez de drv, yo lo haria igual que las otras funciones :) o lo dejaria como estaba ya que no veo el error...


What exactly are you doing here?


I'm not entirely sure if this is the best way to mention the Discount requirement. It took me a while to figure out which packages were required AND where the DISCOUNT_LIBDIR & DISCOUNT_INCDIR paths were before luarocks install discount actually worked for me.

So I think a quick tutorial should be added somewhere on how to get this setup on Debian/Ubuntu. Perhaps I should add a wiki page? Let me know your thoughts on this.


@LoreReut
Please be sarcasm


Having the two static methods with the same name is confusing. How about name `lookupHiveStorageFormat` so it is different, then you can static import the other one


yo sigo sin ver el error. pero bueno, para mi lo que hace falta es correr los tests :) los mismos se deberian poder correr en windows/linux y ahi probar si funciona todo bien sin el hw.


Why do we need this in touchMove and not on touchEnd?

The reason I am asking, is because many native Cocoa/iOS apps are able to scroll from a valid view and allow your finger to go beyond the `UIScrollView` bounds.

If the `touchEnd` event is not firing, this is actually a Safari bug, so I would like to file a bug against Mobile Safari.


I think you forgot one `%011lx` further below...


ops. my bad about the !, hehe.
+1


You are writing to Slobberchops (aka Rafe)!  Patrick is working with me on
the forked source code project.  I should explain...

You definitely don't want to integrate all of our changes, we're working on
a specific configuration for a burning man project.  I was suggesting that
you either cherry pick the changes that pertain to things you find
interesting (specifically life.py) or just copy the files in without
preserving the history.

Those config files are indeed specific to the hardware we've been working
on.  It was done that way because I wanted to avoid writing my own
equivalent of art.py at the time.  I now understand more about what art.py
does and am ready to either write my own harness, which would allow me to
write stuff completely independently from the rop fork as we are doing now,
or contribute improvements (if you like them) to the existing harness.

Anyway, some changes to the original rop project would make it easier for
someone to use that project independently while injecting their own config
and art details rather than how I did it.  If you would like some
contribution to that effect, I'll be happy to do that separately.

In the meantime, I could not figure out another way to communicate with you
outside of issuing a pull request :)

On Mon, Aug 3, 2015 at 9:58 AM, Patrick Ball <notifications@github.com>
wrote:

> oops sorry thought I was writing to slobberchops! maybe blinkymirror.json
> should be git ignore'd?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/slobberchops/rop/commit/926e645aea800618d372ae870d413bbe2d2e73d6#commitcomment-12497841>
> .
>

--
  - Rafe


Removed user & token as suggested


"Undefined behaviour" should be reported if a module is not resolved, too, don't you think?


Add a new exception `HIVE_UNSUPPORTED_FORMAT`


I'm working on going through the latest additions (mostly to combine replacements where possible) and I came across this one.  I'm a bit confused about the intention of this replacement.

Effectively all this appears to do is change "Millennialis" to "Millennialism" (regardless of the capitalization of the leading m) while maintaining the original capitalization.  Any other matches are simply replaced by themselves (e.g. "millennialism" is replaced with "millennialism") which isn't too useful.

Am I missing something?  If all we want to do is replace "Millennialis" with "Millennialism" then we can remove the "m?" at the end of the regex and it will only match what we want (avoiding identical replacements).


I had encountered some troubles related to this too. Thanks for the deeper fix!


Oh, you have a separate inspection for this, I see. (https://github.com/ignatov/intellij-erlang/pull/653)

I think, these two inspections should be one, but reporting different messages in each case.


showdown hangs if doc.index is nil. This occurs if there is no title set in the current markdown document. Adding this check allows for displaying the document so the user can see there is no title set, so they can set a proper title.


find average of different channel.


Done.


Don't add this if you aren't going to go all the way to a .recipe.


Catch `PrestoException` since that method can't throw checked exceptions


Ah, I may have misunderstood the intent. I don't mind maintaining multiple csproj files, but I question the value of having a target strictly for .NET 4 vs 4.5...


93469234-DemandResponse2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1619 of 1789 tests passed)

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


5058-fix-readvars-cli (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (746 of 746 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.14%25-green.svg)


that should have been returnIfFalse()
On Jul 15, 2015 9:19 AM, "Dima Tisnek" <notifications@github.com> wrote:

> Are those typos in assert or was that spelling intended, i.e. is that a
> special function used in this project?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/radu124/fretscpp/commit/084fb70613a0747b4790a08b5222892eb71d26c5#commitcomment-12172531>
> .
>


Algorithm layer


this is a test repository.


Would it be possible to get a release tagged with this fix included (e.g. RC6.1) ? I'm wary of relying on a production maven build that tracks a moving snapshot.


Don't know. Thanks for the heads up. I will try to make time to update mint
this weekend. If you solve it in the meantime, a pull request would be
graciously accepted.
On Aug 3, 2015 10:17 AM, "Shaun Eaton" <notifications@github.com> wrote:

> @kferrio <https://github.com/kferrio>
>
> Have tried several times to get this to work with Latest Nemo 2.6.7 that
> comes with Mint 17.2, but is only creating the "Select Left Side..."
> context menu. Is there an update that needs to be done to support the
> latest Nemo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kferrio/mint-beyond-compare/commit/b74291bb204c6b831652ab5f7a47c6797678bc9d#commitcomment-12498216>
> .
>


Oops. Only `AllowV2` is exported.


> It depends on toolchain. If you are using universal one you must have 5 archs.

I forgot to specify I was using `--toolchain ios-nocodesign`, my Xcode version is 6.1.1.

I am starting to think that this is a CMake generator bug, because if I go into Xcode, and edit the "Header Search Path" by adding and element, and then deleting it. The search path that I created pointing to `<root-it>/Install/multi-arch/arm7` gets taken into account when building from the command line with `xcodebuild`. I also trying to build with https://github.com/facebook/xctool , a replacement for `xcodebuild`

> What if we start with it, but add them using another approach - not in CMake code but in C++ code (?) Just create headers with the same name and do the switch inside

I will propose a mid-term, I will try to implement some CMake logic that will merge header files (everything inside the `include` directory tree) into a single one when their contents differ by architecture.  I'll guard the contents with `#ifdefs` by their architecture using these macros as references for a starting point: http://sourceforge.net/p/predef/wiki/Architectures/ .

It is just that I don't really know how to patch the source to get rid of the `config.h`, plus I think that if we start patching all the libraries, it could become quite a laborious process.

> By the way such approach is iOS framework creation friendly.

My main interest in `frameworks` was to see if they solved the problem of diverging contents by architecture that was happening to `config.h`, which they don't. If you think they can be of any benefit, let me know and we can try to build them at a second stage.


develop (mmatts) - x86_64-MacOS-10.9-clang: OK (1261 of 1261 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)


97903100_SolarCollectors (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


Change to a new image for HW


ohai


LGTM.


$db is not yet defined, was this supposed to be "$old_db"?


`RedisClusterClient` is not closed and occupies resources


Included commits

Commit ID	Owner

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud


I know why it's so complicated after wrapping my head around this for 10 minutes. The `_are_colors_supported` concept is just wrong. Better:

```python
@property
def supports_colors()
    return ((self.print_colored is None and
             not self._supports_colors()) or
             self.print_colored == False)

def _supports_colors():
    """ Override if needed. """
    return True

def _print(output, color=None, **kwargs):
    if color is not None and self.supports_colors():
        try:
            return self._print_colored(output, **kwargs)
        except:
            pass

    return self._print_uncolored(output, **kwargs)
```


I am going to make it so it moves the database into ./local/lib if it does exist


I like keeping the name in with the code, so our urls are still human-readable.  This will make the awcs show up in filters like `Sohran Bigha (34)`, right?  That's probably fine.
You can also use a tuple as a dict key:
```python
# instead of
hierarchy[block][gp][awc_name_with_code] = None
# use a tuple
hierarchy[block][gp][(awc_name, awc_code)] = None
```
That way you can split them up later when you send to the filter, for finer control over rendering.  Probably not worth changing unless they complain.


this is a global because the last line doesn't have a comma. also, we use a trailing underscore to denote private variables.


I believe IPP here is supposed to omit patients older than 85 years.  I'm not seeing that test anywhere...?


I am always interested in things getting simpler. I would suggest you
create the PR after we get this one worked through.

On Fri, Jul 31, 2015 at 8:39 PM, Dylan Wen <notifications@github.com> wrote:

> About the building process, I found it's really hard for me to jump around
> files(Makefile -> build_support/core-build.mkf -> bootstrap.mkf ->
> build.erl -> joxa-compiler) to dig out what happens. Along with the
> complication of self-bootstrapping, it's really hard for beginners.
> I tried to gather all building steps into one single Makefile and made it
> clearer, e.g. it's better codes in build-support/build.erl are put into a
> Makefile. Changes are finished at branch
> https://github.com/hhkbp2/joxa/tree/clearer_build. I am not sure whether
> anyone else would like it, please take a look at it if you feel
> interesting. Probably I should just raise another PR after this PR is done
> sometime later. :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hhkbp2/joxa/commit/83ff5330a3519f81568c8c14dc201dd6e775b388#commitcomment-12478047>
> .
>


Typo.


nvm_alloc_ppalist?
nvm_free_ppalist?


alloc_ppalist?
free_ppalist?


Can be removed


veo q solo se usa dentro del run, podrias definirlo alli y te evitas la creacion de una variable de instancia.


5574 and 5575 does not compile. There is an cannot find symbol error:
symbol:   variable FLAG_PROPERTY_NAME
  location: class FeedbackPanel
How should be fixed?


vdsa


acquire


sss


Remove


mdr->session->check_access(in, MAY_WRITE, ...


Sure. I expect that might happen sometime this coming weekend, if not earlier. I'd like to tackle #117 before cutting the next RC.


This comment should only remove logging. pyramid_sawing replaces it.


What is the reason for going backward? What happens if failing midways?


don't need to pass the session.. that's is implicit when you call the method on a specific object

We can call it check_access() too, with the underscore, since there's no ambiguity with a different version


drop the leading _ and session argument


Same, what happens if it fails in the middle?


Session *session = static_cast{Session *}(m->get_connection()->get_priv());
  mds->server->_check_access(session, in, MAY_WRITE, 0, 0, 0, 0); // call _check_access
  session->put();

curly brackets represent angle ones


It looks like the alignment of this may be off


```java
StorageDescriptor descriptor = table.getSd();
if (descriptor == null) {
    throw new PrestoException(HIVE_INVALID_METADATA, "Table is missing storage descriptor");
}
SerDeInfo serdeInfo = descriptor.getSerdeInfo();
if (serdeInfo == null) {
    throw new PrestoException(HIVE_INVALID_METADATA, "Storage descriptor is missing SerDe info");
}
String outputFormat = descriptor.getOutputFormat();
String serializationLib = serdeInfo.getSerializationLib();


`HTMLElement` is the valid Symbol which should be used when referencing DOM Nodes. :)


Add a new `HiveErrorCode.HIVE_UNSUPPORTED_FORMAT`

```java
throw new PrestoException(HIVE_UNSUPPORTED_FORMAT, format("Output format %s with SerDe %s is not supported", outputFormat, serializationLib));


why is this an event? seems like maybe it should just be a private function. it's weird to set a handler here, every time a notebook is loaded.


```js
return $resource('http://api.petfinder.com/pet.getRandom?key=f07bd0733c5bc6925509c31d5dd84b95&location=:location&animal=:animal&size=:size&output=full&format=json', {
    callback: "JSON_CALLBACK"
}, {
      query: {
        method:'JSONP',
        params:{
          location: 'Seattle,us',
          animal: null,
          size: null
        },
        isArray:false
      }
    });
```


borrado


I created a new commit, this time with the "sign-off".
To be able to create the commit, I had to change the board.cpp file once more.  I add a space character.
I'm an unexperienced git user.  Was there another way to fix this?


@alexmarsev: This will do the job :)


"Hive storage format for the table"


orzGodTang();


What about these from the eMeasure specification:
* Diagnosis of End Stage Renal Disease
* Diagnosis of Chronic Kidney Disease
* Procedure, Performed: Vascular Access for Dialysis
* Encounter, Performed: ESRD Monthly Outpatient Services
* Procedure, Performed: Kidney Transplant
?


Not sure I got why it is unnecessary. We have to initialize the AKE context before sending the dh commit message otherwise it will throw a null pointer exception. This is the first moment Bob needs the `ake` property.

The same thing is currently done in `dhKeyMessage` (the first moment Alice needs the `ake` property).


Könntest du das etwas näher beschreiben?
Ich hatte diese Funktionalität erst als statische Methode in der TupleClassDeclaration-Klasse, allerdings gibt das einen Haufen anderer Probleme (unter anderem, weil das ganze halt statisch ist und wenn während der Laufzeit eines Programms ein zweiter AST generiert wird, ist der Zustand der statischen Variablen noch da).
Daraus resultierend würde ich dann im ASTBuilder als Attribut eine Instanz von der TypeDeclarationFactory vorhalten, die die Tuple-Typen zählt und erzeugt. Wenn der ASTBuilder "gegarbage-collected" wird, fliegt auch die Factory-Instanz raus.
So ok?


93469234-DemandResponse2 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1791 of 1791 tests passed)

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


This change forced me to initialize the `ake` when I switched from the `newConversation` helper to the vanilla exported `Conversation`. This is probably how the client will initialize, and it will fail for the same reason.

See: https://github.com/twstrike/otr3/blob/master/full_conversation_test.go#L50


yep, but no mds-> ... just session->check_access(...)


LGTM


no need anymore.


Adding rss is just a new template. The "feed" icon should link to which one ?


You can do User.create(...) and pass all those fields as arguments.


The second sentence in RoleSensitive.java doesn't read correctly.  Did you mean to say
"use the standard abstract base classes instead"?


Put this in a finally block


PreviousMap constructor only takes 2 args, but this passes a 3rd.


Nice!


Hola Pablo,
Creo que a la gráfica se le podría agregar algo más de detalle, en los clientes y el servidor.
Voy a basarme en lo que tienes para enviarles una propuesta y luego si te parece la completamos.

Creo que las gráficas son en OmniGraffle así que la trataré de hacer igual o si tienes el archivo .graffle mejor.


probably would make sense to load this in notebooks_frame.js, it only has to happen once, not on every notebook load.


If we want the same behavior as the super class, this should be in a finally block, to invalidate the cache even if the storage cleanup fails.


Is it OK to put this outside of the try block now?


Maybe nit-picking but shouldn't the Event be triggered when the build is actually successful? aka move it after evaluator.go#L200 ?

Thanks for the quick turn-around @duglin


Hello Humans


Also call this method from `createTable()` and `renameTable()`


esto es un comentario acerca de este cambio.  se agrego la linea 6 y se agrega este3 comnentario referente al camnbio en el readme.md


@mike-tutkowski what part should or shouldn't be put outside what try block? I can not link your comment to a particular piece of code.


We have rather little cache sources which are not refreshable at all. I've only noticed this with the newly added terracaching, as we can only import those from GPX but never refresh them. But we would have seen the error also with waymarks before.


apagar esse método


So, @BrendanEich was the one who changed it from .io to .com. The website seemed to work for me the same way beforehand as afterwards. Brendan, what was the problem?


So picky, sheesh. It's pronounced the same, isn't that good enough? ;)

Fixed.


comentar essa linha


It doesn't work in Firefox over HTTPS with .com. .com redirects to .io so I don't see why .com would be good.


This is a bit inconsistent: The site.conf option is called "ap", but the function is called "client". Why is that?


nome ta incorreto. paRamento


if we're posting ourselves as the object, there's no need to pass `sectionId` in the `userInfo`, since observers can:

``` objc
TitleOverlayLabel* sender = notification.object;
NSLog(@"Long pressed on section %d", sender.sectionId);
```


Добрый вечер, Ольга.
Очень удивлена следующим совпадением  https://github.com/IvanShymanouski/ASP.NET.1501.Shymanouski.Day8/tree/master/GenericMatrix.


see [other comment](https://github.com/wikimedia/wikipedia-ios/commit/ec2a4a191a7befea197b6a2383be43a796d33004#commitcomment-12499448) about getting `sectionId` from `notification.object`.


yes, we can add those parameters with empty values by default in setMemberIdentifiers().


Noob


Do we need to concern ourselves with undefined here?


Can you run it through pretty-print or some such so that it's easier for a human to follow the formatting?


TIL git removes anything between backticks `` ` ``, this is supposed to read ``put the `!important`s back.``


Doesn't it currently print the message ("Run with -u"...) twice?
I think the `else` should be removed since the message was moved into the `printUpgrades` function.


Implemented and added to README.


Esto también incluye el CRUD de inventarios


@aheritier It would be nice to update this url to the Jenkins Wiki


you rang?


Why the stylistic difference here (i.e. `this`) vs. [`flatMap` in same file](https://github.com/fpinscala/fpinscala/commit/4c2730847a43defd4030247608977d3bddb14e26#diff-3c96b421e257314f60c3e16588fa8fcaR12)?

/cc @zhengcao


Kindly correct the spelling of Imaginea in a future release :)


okay.


ups. :)


I think `exit 1` will not work from inside a subshell `(...)`. Well, actually it might because you're under `set -e` but I think it's somewhat sloppy... Use `{...}` instead. Also, instead of `&&` for that case I'd just use two separate lines:

```
  check_criu || {
    finish
    exit 1
  }
```


Perhaps call this `cleanup` instead?


old code wasn't like this and I forgot to remove from here. updating.


should - sorry.


No, not so picky. But i was just interested in the changes of the README so stumbled over the typo. :-)


Do their other tests use `sudo` here? I'm not sure you're guaranteed to have it working all the time... If the other tests just call `$CRIU` directly then assume they're using `sudo make test` etc. instead and don't `sudo` here yourself.


A couple of issues here.  procedure_code may be lab-specific and not a standard code.  procedure_result.result_code is supposed to be a LOINC code and is surely the better thing to check when looking for a particular lab result.  However dialysis is not a lab test.  Are your clients using the procedure ordering system for dialysis?  Perhaps you should be checking referrals (the transactions table) instead or in addition?


remover essa linha


TEST COMMENT


Buenas, Mario. He añadido el .graffle al repo, por si quieres meterle mano. Si no, dime qué cambios hay que hacer y lo hago yo. Saludos!


Like what?


We can omit the "Erlang" prefix for nested classes, as they're referenced through enclosing class which has the prefix.

Name it FunctionTextProvider.


Perhaps simply:
  Errors detected below.  Please correct.


I had to pin this guy to 0.0.1 since master isn't working with curation_concern's master


@datakurre another way is to use xpath for that: http://stackoverflow.com/questions/30018/how-do-i-select-an-xml-node-based-on-its-content http://stackoverflow.com/questions/247135/using-xpath-to-search-text-containing


Ah, a discussion point that we've had before for other events.  Do you report it "before" or "after", or both?
When the event is related to a single action (like container create) then I think reporting it after makes sense.  However, when there's a series of commands/events that are grouped by another event I think it makes sense to let people know the bigger context.  So, if we only have the build event upon (and after) a successful build, then you'll never know that the other commands in the event stream are all related to a build if the build, overall, fails.

I think the last time this came up was due to some CLI action that didn't have a clear ending to it (like streaming from a container or something like that), so waiting until after it completed didn't make a lot of sense.   And I think I let that situation influence my decision to do it before in this case too - I like consistency - along with what I said above.  But, perhaps that was a special case though.


Why do you make every implementation of this interface store a function name in enclosing field? How is it different from computing arguments text? (Hint: there's no difference =) )


Nullability annotation.


i was thinking about that, with the extra "baggage" debating if it's worth splitting the email pieces out to py-cifsdk-email.


i think Raster should imply georectification, so this placement is wrong perhaps and could be just a generic Image


Did you change your sort method?
`sort -uf -o spammers.txt spammers.txt` does not give that sorting order


why does git do this crap srsly


shouldn’t this be removed?


We mistakenly compiled 1.5.0 with JDK 8. 1.5.1 is compiled with JDK 7, there are no code changes.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2212 of 2232 tests passed)

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


*Modern


Only name something "SomethingImpl" if it's scope is limited. Also, avoid using such names if you can come up with a more meaningful name - in this case - you're implementing a FunctionTextProvider with default arguments - why not name it "DefaultFunctionTextProvider"?


Double yey !


Mind fixing the tests? Will merge it. Thank you very much.


I would think using ``available`` should be fine for a file, since it knows immediately (unlike a network stream) how many bytes there are? We don't want to just read one byte a time. If ``available == 0`` isn't reliable, can we can use one call to ``read`` after grabbing whatever is ``available`` first, so most of it is chunked?


mostly the weight of the text. the navigation should look something like this:
<img width="1168" alt="screen shot 2015-08-03 at 2 34 48 pm" src="https://cloud.githubusercontent.com/assets/4105343/9044777/e6f29b14-39ec-11e5-84ff-c8692014a553.png">


`XCTAssertNil` ?


Ok, le meto mano y lo regreso,

saludos

On Mon, Aug 3, 2015 at 11:30 AM, Pablo García Sánchez <
notifications@github.com> wrote:

> Buenas, Mario. He añadido el .graffle al repo, por si quieres meterle
> mano. Si no, dime qué cambios hay que hacer y lo hago yo. Saludos!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JJ/modeling-volunteer-computing/commit/b86371f978361f8eecf608ba5f525bb697c3782d#commitcomment-12499626>
> .
>


That's what it is. It is a proxy_pass problem. I have Nginx running in front of Emby passing port 80 traffic to 8096. Apologies. Emby sees the traffic as internal. I need to find a better proxy solution.


Por qué hiciste esto?


No, undefined doesn't pass the below check because `typeof undefined === 'undefined'`


Note that this conflicts with example usage here: https://github.com/projecthydra-labs/hydra-works#usage which uses inclusion


`Cannot invoke 'XCTAssertNil' with an argument list of type '(File?)'`


I'd remove %user% from here, it's not supported anymore in Gold and you didn't include any other commands from original game.
There is also %cat% for category directory (for example data/levels/missions/)


Profile validation should validate that the cpu/mem resources specified are non-negative long/double values.

Here, we are validating that the nmInstances provided have non-zero cpu/mem capacities. This is different from profile validation itself.


Sure. Will change it.


5058-fix-readvars-cli (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1264 of 1264 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.96%25-green.svg)


I don't know if you still care about that sample, but instead of launching an external executable to do the disassembly, you can use DbgEng's. Just run `IDebugControl::Disassemble` repeatedly on the memory range and you get nicely formatted strings a la WinDbg's `u` command.


Note that I can get this done before the end of the week if you don't have time or don't want to bother with an option.


This is now fixed;  the workaround was:

```ruby
.select { |name| name['mods:role'].field('mods:roleTerm').map(&:value).include?('Contributor') }
```


I still don't get it. Where are the styles you're expecting it to use?


I already had a debug.c/.h pair in my project, that served for all modules. So I changed debug.h to debug_scpi.h. I also found a "#include "scpi/debug_scpi.h" somewhere in the original code.

Maybe I messed up my include paths when starting integrating the code to my project, but I realized that I should set the include path only to ./inc. Thank you.


@sankhesh what is pyspharm bit about?


Thanks, fixed d5707f0


also in the middle of refactoring `spawn` and webworkers, which is broken here.


Bitte alle Sache die für andere Sachen sind wie z.B die LongMap in other packen danke.


I'd leave it to support also old game.

Wait, there was more in original? O_o


Hans,
All the tests passed with the correction.
The patch is safe.

Regina


Download page should check if user's cookie matches the one stored in the database. It should also unset the cookie after file is sent.


キャプションが図6のコピーだ


Might as well change this to compileCmd, the gc name seems to be going away


@ggouaillardet which async threads are you referring to here?  This commit causes hangs at finalize for
some non mpirun launched jobs.  I'm trying to narrow down when in fact there is a thread to be woken up by this logic in the case of non-mpirun launched jobs.


This is going to make unexpected errors if you set GOROOT wrong on 1.4.  Maybe check that $GOTOOLDIR/compile exists, and give an error if neither exists?


Why do you need to yell in your commit message?


Yep haha. whoops. i watched a treehouse video about through associations and must have typed the example symbol in there.


press* :octopus:


Let's just put the branch name here. You will change this to the (merge) commit once corresponding PR in escalus is merged.


Thanks. I'm not opposed to having a solution in the code though that will
allow it to just work, so if we can figure that out that'd be fine with me.

On Mon, Aug 3, 2015 at 2:36 PM, Oakington <notifications@github.com> wrote:

> That's what it is. It is a proxy_pass problem. I have Nginx running in
> front of Emby passing port 80 traffic to 8096. Apologies. Emby sees the
> traffic as internal. I need to find a better proxy solution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MediaBrowser/Emby/commit/91fd835e9bfaa6642e86ae43ea6886749b103034#commitcomment-12499736>
> .
>

--
Luke Pulverenti


Ah. I didn't look at the guard code below. Why does `typeof null === 'object`? WAT.


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


add css class


We can actually simplify this entire block of code into a single line.
```php
$home_page_flag = drupal_is_front_page();
```


Because the config.h isn't cleared when configuring twice (we do that when building semi-static for Debian and CentOS in a single go) and the compiler warns about redeclarations.


develop (mmatts) - i386-Windows-7-VisualStudio-12: OK (1264 of 1264 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)


Oh, sorry. I thought I added those styles into the recent updates to reactive table but they were exclusive to EIDR.


Hi Nimish,

Quick thought is that this commit looks like it changes the call signature to the cbdatasource.NewBucketDataSource() API.

Unfortunately, cbdatasource is a reusable library that's in use by other projects (not just cbgt/cbft), so...

A) we can either break the API, and have those apps break in turn, and then get them (and/or help them) to upgrade to the new API.  I'm loathe to do this, and have users point fingers at poor library maintainership.

B) or, perhaps there's a way to do add this support to the API in a non-breaking way?  (And, be lazier!)

Perhaps the 6th parameter (auth) to the existing cbdatasource.NewBucketDataSource() can be overloaded so that you wouldn't have to add an extra authMemchd parameter?

    http://godoc.org/github.com/couchbase/go-couchbase/cbdatasource#NewBucketDataSource

That is, currently auth is a couchbase.AuthHandler, but maybe NewBucketDataSource should try to cast the auth value to a (new, does-not-exist-yet) interface, like couchbase.MemcachedAuthHandler (or couchbase.KVAuthHandler?  or couchbase.SASLAuthHandler?  naming is hard).  The extra interface, if the cast succeeds, would have the optional memcached/KV/SASL auth information.

With plan B, then all the existing users of cbdatasource wouldn't break.

Lemme know if that doesn't make sense or there's another way/idea.

-- steve


URI must be `-`.


Nice Readme :+1:


Thanks for the credits!


97903100_SolarCollectors (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 252](https://teamcity.labkey.org/viewLog.html?buildId=264202&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:11:24
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


Yes, right now only allowV2 is needed for xmpp-client


@bricelam does this fix the "System.Runtime" issue? :smile_cat:


As the original script, when copied in, is showing the same problem, this seems to indicate a problem with copying in the script rather than an issue specific to my version.

However you said you set the permissions correctly which is the only common mistake I could think of.

Does the boot log contain any hints?


Bravo!


huh.. didn't see you working in here.. gonna have to merge after doing the course (still changed a few things)


First, constant names have to be in upper case (see VERSION and INSTANCE fields above, for example).

Second, you should probably move this key to the index extension itself, as the key's users don't need to know it's related to ErlangFileElementType.


@Synapz1 I deleted the repo because everyone was contributing when i didnt want them to. I have the source on my android tho.


I don't think so. I think that one is something wrong with the Immutable Collections package and/or NuGet/DNX.


@sym3tri Later for the party, but why redirectURL is required here? E.g. I think I can use the client for just token verification or token refreshing, which doesn't need the redirectURL info.


I don't want to rename it, but I think, I can remove debug.c/h completely in future version.


@danielo515 tendrás algún ejemplo de como se realizaria? yo lo e intentado con uno de los métodos de object "hasOwnProperty" pero no e dado con la solucion adecuada


@MrMindImplosion surely you need that because otherwise you have to install them with `npm`?


Hey, this looks great and all but I'm pretty sure that I don't actually have anything to do with this project :smile:


In the case of an exception, the directory stream is not closed creating a file descriptor leak.


argh. forgot about @-mentions. that was meant for @timelyportfolio :-) apologies


Add a comment about why you need these two packages


OMG -that could take a while to fix, then, I guess. I will enjoy the sun in the meantime, then :sunglasses:


yeah, but you just run `npm install` when you clone the repo, which can be put in the `README.md` file. just saves having a million lines in the repo when you don't need the repo to be that large.


@lusv rm these comments


@doutriaux1

Just cleaned up the code. My change does the same thing as the previous code in a succinct way. Moreover, it sticks to the convention with other modules.


Here it is!

Viel spass damit!

Todo:
* [ ] put this in the form you want to have error messages
* [ ] I think that's it....


LGTM


Why was this important?


"Adding SignalFx apt repositories"


No boot log. If you mean messages/dmesg, no nothing - it's puzzling. I even checked the script with a hex editor in case it had been edited on windows machine, but looks fine. I run scripts/py files all the time, so it's not new to me. My desktop is ubuntu 15.04 - rarely use Windows.


But it might; I haven't investigated that one yet. There's a ton of churn going on in DNX right now making day-to-day development extremely painful.


my mistake, you need package.json first which defines what npm should install. when you run `npm install <package-name> --save`, it adds the packages to this so that when you run `npm install`, it grabs these packaegs automagically and installs them.


this here is the bootstrap styling for the form. it makes it red.
http://getbootstrap.com/components/#alerts

(you can change the colour if you want)


Will give this a go then, when it is in the daily build.


Change to ```rm -f ${repo_name}```


Hi, awesome post, but what are you doing here? Why not just render the `TodoApp` directly?


f


Thanks for your feedback @MrMindImplosion , but I believe that we will need to include mode_modules, because otherwise, it would take a while to do `npm install 'module'` on each? In order to do what you are suggesting, one would need to include a `package.json` file which would then allow the user to do `npm install` inside the project, downloading all of the dependences. We did not do this, because otherwise, it would mean that each of us would need to install the modules manually, as we when one of use committed to github, we wanted all the modules to sync properly so that they worked. Once we get this project to a deployable stage, of course we would use `package.son` to install the required dependencies.


"ap" is the actual mode name of the interface in the mac80211 layer while "client" is the purpose of the interface. Likewise, both "ibss" and "mesh" would be used for meshing.


I would vote for us to accept this pull request.


@popey456963 it's not a PR...


To be honest, it basically is @developius - It's a gist suggesting us to make a change.  Also, this can be automatically synced with us.


Test @OmaymaS


When will the update?


you don't do npm install module on each, just `npm install` which then reads from package.json and installs all dependencies. with a good script on serverside, you can listen in to pushes to github using webhooks and automatically run `npm install` to get new dependencies and then restart the node.js server. The only reason i bring this up is because github has a limit of 1gb for a repo and including loads of libs under yours is inefficient. it also, make it seem like your repo has a LOT more code than it actually has. Also, i would be willing to make a PR if you give me some time. It's not here because i did this during the finals and i had really bad internet so couldn't clone in order to delete node_modules and make a package.json


Please incorporate the following:
* make the class public
* add short javadoc to the class


Crazy cows drink milk


I accidentally deleted line 99!
 https://github.com/radumas/coppe-ltc/commit/9f94521782711bc31885b96db078712ed483fe5d#diff-663f387b6a1a407ab38de055a12bc7c8L99


add short javadoc pls. desribing purpose of the class and version when it has been introduced


Лишнее?


@MrMindImplosion That was my mistake, I'm a very tired person.

I would be more than willing to accept your pull request if you make one.


Thank you for the explanation behind it, as I missed previous conversations; what you described makes sense, the build was started and thus should trigger such event especially if it's consistent with the rest of the events.

:+1: LGTM


We could say that one has to run `npm rebuild` in projects with native dependencies.


Should be a constant


@larsiusprime thanks! i've figured out how to handle game scaling correctly on native targets. Now i need to check everything else again (i'm sure that there are couple issues left)


`fail` is still on the Jenkins side.  One option is to have the Jenkins script `try` this (and other calls into this module) and `fail` if it `catch`es any trouble.


Please remove one of the empty lines, add javadoc


add javadoc, this is quite important method, it deserves it :-)


It's [4.4.63.26](https://github.com/nodejs/io.js/pull/2220) now ;)


This probably shouldn't be a notification… but some sort of delegate system or target/action with a UIControl.

Since this behavior isn't actually required or defined yet  - I basically "didn't review it". When we decide how to do editing, I'll come back to it.


Mariano, la linea comentada es la linea origina y la otra es la linea que
queda. Es decir yo cambie la siguiente linea (la comente en lugar de
borrarla)

ciaaPOSIX_assert(device->open(path, (ciaaDevices_deviceType
*)device->loLayer, oflag) == device->loLayer);

por

ciaaDioDevices_deviceType * dioDevice =
(ciaaDioDevices_deviceType*) device->layer;

ciaaPOSIX_assert(dioDevice->device->open(path, (ciaaDevices_deviceType
*)device->loLayer, oflag) == device->loLayer);

con lo cual el assert no cambia, lo que cambia es que hago
device->layer->device->open en lugar de device->open.

Lo que pasa es que ademas hice este cambio de formato, estaba así

extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
ciaaDevices_deviceType * device,
uint8_t const oflag)

y quedo así

extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
ciaaDevices_deviceType * device, uint8_t const oflag)

como los dos cambios esta seguidos toma la llave como referencia y por eso
es mas difícil de leer.

El 3 de agosto de 2015, 14:42, Mariano Cerdeiro <notifications@github.com>
escribió:

> yo sigo sin ver el error. pero bueno, para mi lo que hace falta es correr
> los tests :) los mismos se deberian poder correr en windows/linux y ahi
> probar si funciona todo bien sin el hw.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12498710>
> .
>


You dont have a donate version in the play store so donate_link and donate should be false after that i will merge


I believe this change broke both Firefox and IE (in specific scenarios that I'm trying to track down).  According to the spec, the value should be off|on.

Example plnkr: http://run.plnkr.co/plunks/OnkTwKgzfHRUcJHj7BTI/

![firefox](https://cloud.githubusercontent.com/assets/124591/9045397/7d877964-39e8-11e5-8b68-5406a1b02b85.png)


For some reason I didn't see the message doubled, but yes, I missed that. Will patch. Thanks!


@"sectionID" should be a constant

Basically for all notifications, the Notification Name and any keys should all be defined constants in the header of the class that posts them. That way it is documented on how to use a particular notification.


Nix `new` here since this doc will probably survive long after the variable could be considered "new."


I'd like to see that commit but it doesn't exist

https://github.com/inverse-inc/packetfence/commit/9248e9b801c78983d5309a6774e93cc22e542910


Yes, thanks.


So, if there is no logger the user won't receive any message? by the way, is it possible to have a null logger?


OK, maybe we can think of having libscpi specific debug stuff in debug_scpi.c/.h and platform/project specific stuff in debug.c/.h.

In my embedded project (e.g.) I have no stdout or sterr but I have itm_printf() which is something specific to ARM Cortex-M CPUs (ITM = integrated trace macrocell, that is a peace of memory in the core of the CPU where the application can write to and a conected debugger can read from. This is very comfortable in embedded design.)


Se podría hacer modificando la vista layout.ejs de la siguiente forma:

<% if (typeof(e) !== 'undefined' && e.length>0) { %>

¿Lo veis correcto?


This won't do what it's intended to do. Included files will still not be logged because subPath is not set when files are included by default. I'll submit a pull request that has the functionality I was imagining.


This can throw NPE, when except does not have inner exception.


fixes #3


LGTM


Oops, thanks @chriddyp!


shouldn't this have been:

  --protocol PROTOCOL

and explain what PROTOCOL is in the description?


What cookie? There are no cookies.


Did you miss a comma between "positions" and "loaded"?


Same here


I know this could be a old bug. if currentHadoopUser is null and keytabePath is not set, doesn't this code throw NPE ?


question... sorry for late... if there is conflict between url ^ and port v - what will kick in?


resp not closed? (need e.g. defer)


You still don't have Ctrl+Q, and Ctrl+Shift+I working properly. It looks like you're gonna need your ```org.intellij.erlang.stubs.index.ErlangBehaviourFileIndex``` to become an index over ```org.intellij.erlang.psi.ErlangModule``` as opposed to ```org.intellij.erlang.psi.ErlangFile```.


This is full of leaks. Also why is it necessary?


oh... DNS here, so the README is out of date?


this code path is never reached


rename "flag" to e.g. "url" or "root"


No way Android Studio let you compile this code! This android:padding="16dp" line seems to have lost its way. Let's figure out what this does and where it should go during our meeting today.


Thanks again, @MrMindImplosion , but as I have explained, this would involve everybody in our team to run extra commands when dependencies are added, this is inefficient and takes time, and so therefore, we just include the dependencies, as this is fast, quick and hassle free for the people in our group who develop front-end, and aren't as good at using command lines? Many thanks for your point, and we will definitely include this at a later stage, once the project is complete, and we are ready for deployment. If we do reach the stage when our repo is full, we will add dependencies to `.gitignore`, rather than the whole node_modules folder, so that extra dependencies are still added. Also, when you say that our project contains more code than it actually does, this is caused by the HTML generated by our PDF2HTML converter, and not by the absurd amount of modules contained within our project...

Many thanks again,
Team Refresh.


Whoops! :grimacing:


Arghh, that's frustrating, thanks for the explanation. Sed -i is not portable however, at least OpenBSD doesn't have it in any released version that I know of.


Ahh yes, I forgot about that part.  That's a little trickier.


Before the fix, the columns `['int_', 'long_', 'byte_']` would be returned in the right order, but the dtypes still have the original order, causing the dtypes of the columns to be resp. `[np.int8, np.int16, np.int32]`


This will capture patients of age 1 or 2 as of the begin date.  However as I read the eMeasure spec, they should not be included if they turn 1 exactly on that date.  You might instead calculate their age as of the end date, and make sure it's exactly 2.  That only works for calendar year measurement periods so you might want to improve it even further.


 wordpress-visual-comopser-extension
This is an extension of wordpress visual comoper. This plugins will help to replace the wordpress menu with the page builder.


:blush: very kind of you @IonicaBizau
:computer: coding is :cool:


저는 String 에 대한 함수에 대해서는 가변 길이로 바이트를 할당해줘야 하기 때문에 (예전에 논의된 바로는 널 문자가 올때까지 읽는 등의 작업을 하자고 말씀하신것 같습니다) TypeLength함수를 호출하지 않을 계획이었습니다. 기본적으로 타입에 따라서 할당받아야 할 사이즈가 정해져있을 때만 이 함수를 호출하구요. 일단 새로 보낸 PR에서 보시면 제가 STR은 0을 반환하도록,
우리가 정의한 타입 이외의 값이 들어올 때만 error 메세지를 출력하도록 변경하였습니다.

* 참고로 string을 처리할 때는 그 메세드 안에서 바이트를 순차적으로 읽어들이면서 널문자가 들어올 때까지 확인하고, 그 만큼의 크기를 할당하여 처리하려고 계획하고 있었습니다.


This patch is bigger than I'd like, big enough that it's not obvious what it's doing


sure I can add metavar. You think that 'protocol to be used to communicate with ldap' is not enough? Not sure how better to describe in 1-2 lines


Поддержка типа 'page' ?


This method seems very similar to `select`, but adds some new functionality.

We could already write
```
t = Table([1, 2, 3], 'points`)
t['counts'] = 0
t['letters'] = ''
t.select(['points', 'counts', 'letters'])
```


Sticking `None` into the table scares me — I think I'd rather have an error in this case.


@zeeshanlakhani Out of curiosity, how do you test changes like this?


I can't image situation how they can conflict ? You can't speicfy 'port' in --ldap-server param now


Had to adjust the y-coords here, since they go through `unproject` and then `project`, which now gives different results.

I must confess I didn't check this very thoroughly, but the new numbers make more sense to me than the previous, given the expected dimensions at this zoom, so I'm pretty confident it works as expected.

Also, I went through some of the debug-examples and they work as expected.


make(.., len, capacity) - capacity goes 3rd (optional)


I think a `KeyError` is more appropriate behavior. Is there some particular example that motivated this change?


Is it better to check for Python version and act accordingly rather than catch an ImportError? Catching an exception would also catch import errors from misconfigured python installations or PYTHONPATH issues.

I am not very familiar with Python conventions, do what you think is best.


wordpress-visual-comopser-extensio


the metavar should be simple term PROTOCOL for example, then in the description you write: "protocol to be used to communicate with server, can be x, y or z".


DNS is correct , we use --port instead of ldap-urls now, no?


Hey, I was checking out the fork network of the original plugin.

How has implementing CDN/CDN Invalidation?  Seems easy enough from what I've seen peeking at your code.


so it is not url :)


not sure I understand, above README.md states: --ldap-server URL


Yes, setters on name is primarily for serialization. Better immutability is more complicated (there are various levels of immutability), and I really want to keep it simple.

For database objects of a specific type, I think name (and sometimes schema) is always the unique identifier. Just think of the DDL statements - create/alter table dbo.X can only apply to one table. Within table X, you can add/alter column Y, but it there can only be one Y in table X. Y could be an int or a nvarchar or a guid. Any sql statement will execute unambiguously against schema/table/column by name.

SqlServer actually has another id, object_id. If you execute the same create database scripts against two different database instances, you'll have different object_ids. The object_id function takes the string name and type.


@aenhsaihan Thats interesting, actually. The single period refers to the "working directory", which is not necessarily where this file lives, but where the file is that is looking for that directory lives. Since the express.js file isn't actually using the directory, but passing it through the app.use, it doesn't point to the config directory. So I'm guessing the directory is either being used by the server.js file or express just works from the root directory. Either way, thats interesting. On other projects I've done, I've just used 'public' and that works. I think it would be a good idea to take a look at this the next time we meet.


To ensure safety of the server, we need to check to make sure to add this under line 13: if (string.IsNullOrEmpty(rawHosts) || !rawHosts.Contains(';')) return hostsList;


        if     flag =~? '^-\%[no]quickfix$' | let s:flags.quickfix = flag !~? '^-no'

I think you mean `\%(no\)\=` here.  `\%[no]` matches `n` or `no` not an optional `no`.


develop (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1264 of 1264 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)


Otherwise invalid hosts given by the Client could easily cause a server crash.


oh, ok


I have just tested pub and sub. It still works.


Also, add: <code>... where (!string.IsNullOrEmpty(host) && host.Contains(':'))</code>


for delete and undo text


@HowardHinnant Agreed.

@vinniefalco It's worth noting that, for all the effort, these math changes should only be different with extreme values - values that we are not very likely to see. However, with those values, it is certainly true that these kinds of changes will give different results. Does that make it worth tagging this PR, and/or changes to `mulDiv` as "Tx Change"?


@wardVD the samples DYJetsToLL_M50_HT200to400,DYJetsToLL_M50_HT400to600,DYJetsToLL_M50_HT600toInf
In DAS are still in status=PRODUCTION, will merge when status goes VALID


oh yes! it's out of date.. :) should be DNS


ExactlyEqual( const TInput & x1, const TInput & x2 )
@ajjl

There is nothing about this function that limits it to floating point values.  It could be used for classes as well or integers.

An important point in the documentation is that this is a complementary function to the AlmostEquals where a small tolerance is acceptable.  In this case the values to be be exactly equal (with no tolerance) to be considered the same.


which you have already fixed. Thanks!


I'll put it back.


What version of OS X did you use for this? This actually breaks fonts for me on 10.10.4.


As a final note, perhaps we could ignore the host since it is invalid instead of throwing an Exception. By throwing an Exception in here for an invalid host, we now will not obtain the other hosts that likely were valid.


If you want to name the functions after their purpose ("client" in this case), both other functions would have to be called "mesh", as you said.

On the other hand, if you name the sections in the site.conf after the interface mode in mac80211, "ibss" would have to be "adhoc", because that's the mode name for the interface (see [line 75](#diff-ef20480af207f56edad2b9b2a3bfba70R75)).


Firstly, you can add bash/batch script to the root directory that runs all commands needed to run the server (```npm install;
node app.js;``` or similar dependant on OS) which those less familiar with the command line could simply double click to get their development server ready. Secondly, the node_modules folder is 18.1MB out of your repos 24.4MB full size (not including `.git`) which is ~74% of the code in your repo.


taka generalna zasada, w commitcie dajesz referencje do issue n.p. #4, ale opis commit'a postaraj się dać bardziej znaczący. Referencja do issue daje możliwość podejrzenia issue w github, natomiast przeglądając commity w lokalnym repozytorium ciężko zorientować się który commit czego dotyczył. W naszym przypadku dałbym commit "#4: page1 refactored to projects"


This would be easier to read if you just tracked a flag for "hasNulls":

```java
boolean hasNulls = false;
for (...) {
    if (isEntryNull(position)) {
        hasNulls = true;
        newIds[position] = values.length;
    }
}

if (hasNulls) {
    ...
    return ...
}

return ...
```


- What does this have to do with it being a login shell?
- You mean "$TERM"
- Doesn't this define fish_prompt twice, the second definition prevailing?


NeiAddons removed ExNihilo support with version 1.12.8 as ExNihilo handles it itself with a newer version


Thanks!


I think we do not want to execute the %sh{...} block either if the exec fails, so it should be moved inside the try block.


Esteban, fijate si podes re usar el test que acabo de hacer commit en tu
branch:
https://github.com/ciaa/Firmware/commits/master

tenes que hacer un rebase o un merge del master en tu branch y luego correr
el tests y explicarme donde el codigo original estaba mal, antes de cambiar
el codigo me gustaria aclarar donde esta mal, yo sigo sin ver el
problema... y es más si lo cambiamos me gustaria cambiarlo como las otras
funciones y no como esta ahora... :(

Saludos.
Mariano.-

2015-08-03 21:15 GMT+02:00 Esteban Volentini <notifications@github.com>:

> Mariano, la linea comentada es la linea origina y la otra es la linea que
> queda. Es decir yo cambie la siguiente linea (la comente en lugar de
> borrarla)
>
> ciaaPOSIX_assert(device->open(path, (ciaaDevices_deviceType
> *)device->loLayer, oflag) == device->loLayer);
>
> por
>
> ciaaDioDevices_deviceType * dioDevice =
> (ciaaDioDevices_deviceType*) device->layer;
>
> ciaaPOSIX_assert(dioDevice->device->open(path, (ciaaDevices_deviceType
> *)device->loLayer, oflag) == device->loLayer);
>
> con lo cual el assert no cambia, lo que cambia es que hago
> device->layer->device->open en lugar de device->open.
>
> Lo que pasa es que ademas hice este cambio de formato, estaba así
>
> extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
> ciaaDevices_deviceType * device,
> uint8_t const oflag)
>
> y quedo así
>
> extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
> ciaaDevices_deviceType * device, uint8_t const oflag)
>
> como los dos cambios esta seguidos toma la llave como referencia y por eso
> es mas difícil de leer.
>
>
> El 3 de agosto de 2015, 14:42, Mariano Cerdeiro <notifications@github.com>
> escribió:
>
> > yo sigo sin ver el error. pero bueno, para mi lo que hace falta es correr
> > los tests :) los mismos se deberian poder correr en windows/linux y ahi
> > probar si funciona todo bien sin el hw.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12498710
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12500571>
> .
>


I think there's a flaw with this proof. `a` and `b` get sorted, but `c` (`feeFactor`) is not sorted with the other two. But now that I fiddle with a calculator a bit, I realize that since `feeFactor` is 32-bit, if `c` is larger than `a` and `b`, then `b * c` can't overflow anyway.

So I'm convinced.


2015-08-03 13:14 GMT-05:00 antonylesuisse <notifications@github.com>:

> Adding rss is just a new template. The "feed" icon should link to which
> one ?

Yes it is a simple "advance" xml constructor that was what we did in the
module.

And by default, I think you can have both. (even they are very little icons
that can be added).,

--
--------------------
Saludos Cordiales

CEO at Vauxoo <https://www.vauxoo.com> Odoo's Gold Partner.

[image: --]
Nhomar Hernandez
[image: http://]about.me/nhomar
<http://about.me/nhomar?promo=email_sig>


You can factor out some code from the `if` and `else` branches:

```java
Slice[]  newValues;
if (hasNulls) {
    newValues = Arrays.copyOf(values, values.length + 1);
}
else {
    newValues = Arrays.copyOf(values, values.length);
}

return new DictionaryBlock(getPositionCount(), new SliceArrayBlock(newValues.length, newValues), wrappedIntArray(newIds));
```


What's that?


I somehow managed to make a mistake in commit message...


Sorry for delay, just getting back to real world now. It seems like an implicit flush on start would be valuable, but I'm blanking on cases where you wouldn't want to the flush


if we rename next to grep-next, then I'd say jump should become grep-jump.


No debería ser 'usuarios'? digo, devuelve una coleccion de varios usuarios, no uno solo


> Why didn't you just submit a PR to glob2base?

no prob, that's the intention, [per my comment above](https://github.com/floatdrop/gulp-watch/commit/b1afde8bc4668e46ef70e750ab88d62be0a03bcf#commitcomment-12481368) (bottom):

> There shouldn't be two modules for this, seems like a big waste of time on both ends

fwiw, I didn't create either module. I just did the benchmarks because I made that initial comment and started having doubts about what I said, so I wanted to make sure.

anyway, happy to do a pr. I'll send it over in a bit


Is there any reason why this doesn't use `newIds`? (maybe memory usage downstream?) It's going to be exactly the same as `ids` in the case where there are no nulls.


From `iw help`: Valid interface types are: managed, ibss, monitor, mesh, wds.


Added @deprecated  (might have pressed the wrong button in website again :)

On 8/3/2015 12:46 PM, GaryO wrote:
>
> What's that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PierreJasmin/openfx/commit/58f6369fd177c506fb7a4061ba93e1a3b7a6a25f#commitcomment-12501098>.
>


Is it possible for immunizations to be recorded as CPT4 procedures instead of as medications?  If so would that be a problem here?  The eMeasure spec says to check both.


saw approval from mrallen1
at https://github.com/basho/cuttlefish/commit/1d6982b3d3f32887ba3d3bbbcfdaf386f3103858


merging basho/cuttlefish/bugfix/jrd/explicit-utf8 = 1d6982b3 into borshop-integration-199-bugfix/jrd/explicit-utf8


basho/cuttlefish/bugfix/jrd/explicit-utf8 = 1d6982b3 merged ok, testing candidate = 1e9ca4d8


Ovo mi deluje ok. Za sada nam to vrsi posao, ako zatreba, koristicemo onu fju compare u types :)


Looks like a merge got messed up.


@bucaran I tried to simplify the code a bit. Looks good to you?


Nice catch


Thanks for the pull request! I hope I will process it faster than the previous one.


Duplicate issue at #329


Mrik - I didn't know you removed the GMap interface.

The list and map views are documented in the page layout. Let's have an open dialogue before adding/removing so nobody's confused.


@aenhsaihan Okay, I did a little research and I think this explains it : http://stackoverflow.com/questions/16727045/node-js-express-js-relative-paths-dot-or-dirname-or-without-any-prefix

If I'm understanding this correctly, It appears that paths are relative to the node process, which runs in the same directory as the server file, which in our case is server.js.


remove minus sign from the beginning of lines, in editor I see

```
    <!--[if lt IE 9]>
-     <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
-     <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
-   <![endif]-->
```

While it should be

```
    <!--[if lt IE 9]>
     <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
     <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
   <![endif]-->
```


czy jest możliwość edycji wpisu już wykonanego commita?


Right, I should've added the info that c is 32 bit.  Thanks for highlighting that.


I've rebased this against your Devirtualize rebase, and am testing the combo now.


Would love to not commit commented out code.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 4](https://teamcity.labkey.org/viewLog.html?buildId=264211&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:35:26
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

```


Update the following code:

```
         <nav>
           <ul class="nav nav-justified">
             <li class="active"><a href="#">Home</a></li>
             <li><a href="page1.html">Projects</a></li>
             <li><a href="#">Services</a></li>
             <li><a href="#">Downloads</a></li>
             <li><a href="#">About</a></li>
             <li><a href="#">Contact</a></li>
           </ul>
         </nav>
       </div>
```

1) Note that there is a reference to page1.html should be projects.html
2) Missing link to Home, update href accordingly


Added some comments related to measures CMS117v3 and CMS165v3 which I happened to know a bit about.  These might apply to some other measures that I didn't look at, so please consider them for other contexts also.


Will look into this.

On Monday, August 3, 2015, Ian Clarke <notifications@github.com> wrote:

> Also, is it necessary to configure the RandomDecisionForestBuilder?
> Wouldn't the defaults do? A more concise usage example would more
> effectively communicate the simplicity of the library.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sanity/quickml/commit/3090ce209ec0e257188a1106e33d65799a3aa69b#commitcomment-12488676>
> .
>


I can't merge it now in mine.. Because it doesn't work on arm.....


Are we going to have this function somewhere in Boomstrap as well? (Not just in the docs.)


A simplification/refactor should not cause tx breaking changes


General comment: please perform tests of your code whenever you change something. If you would do some basic sanity tests, e.g. clicking Projects link being on Projects page, then the error could be captured


After having read this through a few times today, I'd really love for the word "rename" to appear somewhere in there, ex,

    ... existing label name (so `job` is renamed to `exported_job`, and so on) and then attaches ...


I don't think I can change this without potentially invalidating existing reports. We're storing the name of the backend field (contact_type) in the database. In order to display, we split on "_", do a " ".join on the pieces, and .title() it.


Why not just import it?


991757360-DocIssues-4956-4997 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1769 of 1772 tests passed)

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


You wanted to know specific examples of what TypeScript would fix for us. I think this may be a good example here. I'm converting some code now and this gives me a red flag.

Under what scenario would `parser.parser` ever be true, since it's coming from the `parse` function you defined, where there is no `parser` property. I'm not able to find anywhere where this `parser` property is being set.


hello-world


Okay. Created a PR: https://github.com/apache/incubator-zeppelin/pull/177


so like this?
```python
        use_supported = self.print_colored is None and self._are_colors_supported()
        if (
                kwargs.get("color") is None or
                or self.print_colored == False or
                not use_supported):
```


Why upgraded angular if you are resolving to version 1.2.15?


I don't know, @Udayan12167 would Linux color codes work in Mac too?


Remove this file from source code, use instead from bower path


Yeah, I believe I wrote this before `tsconfig.json` was working fully, or maybe I just didn't know how to use it at the time.  :)  My new stuff all is reference path free.  Also, TypeScript now includes Promise definitions, so the typing for it isn't necessary anymore.


Remove this on as well.


Use bootstrap css instead from bower path


Why is this code in main at all?  Can't the entire block be moved to ConfigManager or is that the wrong place for such a message?

-Tres


When you create a variable, this should start with any lowercase letter (a-z) or underscore (_), if you start your variable name with  uppercase it is considered as a Constant.


You can use demo-twig to test this PR


Nice Job! all the statements required are included in your file.


Is there a specific reason _End was marked sealed, but the rest of the shadow methods are not?


Are you sure you can do that? These two lines not identical - comparison operators are different. This would probably result in larger-than-needed bounds.


Why hiding supertype combo on selection of category? Keep it visible everytime.


well well well


this is inetersting


Wrong validation method. Why doing each validation in javascript file. Use angular form validations.


and.... derp. -_-


Also see how angular form validation can be hooked w/ notification service. Angular validation as just a popup/tooltip near invalid element, instead try if you can use notification service to display validations.


I got an error from this line: `src/bson_binary.erl:55: illegal use of variable 'Name' in map`.


In ruby if you are defining a local variable it should be written with lowercase letter (a-z) or underscore (_) , otherwise, if you write the name with uppercase it is considered as constant.


Why created branch from HDPDGI-124, it would become mess while merging. Create branch from master and then add your changes related to refactoring.


qq


Agreed, good catch.


may be but i already forund caf source with this commit
https://github.com/k2wlxda/android_kernel_N958St/tree/LA.BR.1.2.4_rb1.10

and the device tree matching this branch also removes the uncompressed kernel line from boardconfig....so i am just going to try to make new kernel with xiaomi commits on that branch...may be it will work...if it does not then it is still harmless.


can probably revise this line as well


may be...:)
and welcome to mi4i -dev group :)


Nice job! I like to see that you are trying to do also different things besides the practice described, only a suggestion in your code is to try to use the name of the variables with a clear name, for example when you are counting bananas you should say bananas=10, instead of a=10, in this way you perform your code more easy to maintain.


Move 0x02 to proper spot in code


LGTM


这个 url  和上面删掉的那行代码里的 url 是一样的啊


Thanks! :pray: :heart:


This comment is alien language. I cannot read.


Hello @anamewing,

晚一點的時候我試試, 謝謝.


아나


why


do


you


keep


deleting


the


lines


between


all


of


properties


It was the only one that wasn't marked virtual in the previous version. I'm all for sealing the rest of them.


WindowComplexManager_Speedup (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


I pushed the code to git


Sorry, I don't understand the above. As described in the expopt documentation (here: http://docs.mosek.com/7.1/capi/Exponential_optimization.html), the objective is just an ordinary posynomial, which can of course have any coefficients c_k > 0. If x_j does not affect the objective, the corresponding a_{k,j} is 0, which is also allowed.


I've noticed now that any time i draw a marker on the map it will crash on the phone. been working on it now and I can't even make a dummy marker show up with the map. (copy pasted a marker from google sample). Works 100% in the simulator, crashes my phone.


Ok. I'm working on commenting some of the command classes right now, so i'll do it right now. However, in Java, they are not marked as final. so maybe we should just remove the sealed from that one so it matches Java. Since it's internal it can't be modified by teams anyway.


Commit and description should say PHP5.5.


because we need to append "_head"/"_snapdir" to the name


呵呵呵


Please split this out from the "Added support for .jpg files" pull request, so they can be reviewed and pulled in separately.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 364](http://192.168.30.225:8111/viewLog.html?buildId=1867&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Nice Job! I like the way to try using also different options.


This will absolutely work, but because you're using `users.sample`, you're creating items for randomized users, which means not every user will necessarily have items.  That's probably fine for your seed data, but if you did want each user to have some items, you could do something like this:

```
User.each do |user|
  5.times do
    Item.create!(user: user, name: Faker::Lorem.sentence)
  end
end
```

That would ensure each user gets 5 items.

Your way is absolutely fine, just wanted to show you another option.


@Raesu It crashes on the very line that adds the marker? Or does adding the marker fire some method in the delegate which in turn is causing the crash? Can you tell if the offending code is in your classes or the library classes?

Reset the settings and contents of the simulator and try it there again. Still works?

There would be crash logs on your iPhone that you can look at. Check out the Organizer...


And yes, I know that the way I'm saving the config is hideous af, I'll work on it later.


Thanks for that :smile:
BTW, do you have some articles about `let` and `const`? I know it's a basic question, but I'm still confused about use them.


nie, te commity zostają tak jak są, postaraj się zastosować regułe przy następnych

Generalnie, nic się nie stało. Tutaj możemy się mylić, od tego jest ten projekt.


all tests pass:
success: http://buildbot.bos1/builders/test-cuttlefish/builds/159


looks good to me


Just saw this. Big fan :+1: :hash: :one:


Accidentally pushed this directly instead of as pull request.


The rewrite isn't exactly a simplification.  The rewrite gets more correct digits of accuracy when the numbers are extreme.  And when the result can't fit in a uint64_t even if given infinite storage and precision for the intermediate computation, an exception is thrown.  I think that best as we don't want to silently overflow here.


This is not how an AJAX call against WordPress should be done. Have a look at this page, to learn on how to improve it: https://codex.wordpress.org/AJAX_in_Plugins


Should this be extracted to into a js file?


Hm. Counter example:
```
a ~= 2^38
b ~= 2^36
c ~= 2^31
d ~= 2^45
```
Assume those values are approximate and, `a`, `b`, and `c` are all relatively prime to `d`, so nothing gets reduced by common factors. Now, if we don't worry about overflow, `a * b * c / d ~= 2^60`.  But, since `b * c ~= 2^67`, we'll throw the exception, even though `a / d` rounds to 0. OTOH, there doesn't seem to be any way to do this calculation within 64-bits without getting either an overflow or 0, so, once again, nevermind.

I'm still convinced.


Error:
asciidoctor: WARNING: <stdin>: line 74: section title out of sequence: expected level 1, got level 2
Warning: (intermediate value)(intermediate value)(intermediate value).$read is not a function Use --force to continue.


> @neappx How is that relevant? GitHub isn't a monopoly right now and if it's ever gonna be one is not clear.

It is. There is no other realistic choice.

> And just because it has the most users and therefore makes it the most viable option doesn't require them to host stuff that would damage themselves as a business.

Yes, it does. That's the other side of the deal. You want the userbase/community, you have to take the accompanying social responsibility. Give and take.

> For its success, sure. But keep in mind that a company has employees and therefore a (partial) failure of a business can have more impact than one of a hobby project (in terms of actual stuff happening in real life).

The world doesn't just consist of "companies" and "hobby projects". And that is a horribly inaccurate generalization, and one that I'm getting rather tired of hearing, frankly.

> I guess enterprise planes may have different ToS/better contracts (when running on an Enterprise plan), but how would you grant yourself the right to delete content in the case you think you have to without being too vague?

You don't. If you can't define it, you shouldn't be in a position to remove it. Catch-all clauses hurt every party involved.

> Fair point, but I guess it's still better to be specific what is prone to get removed instead of writing something along the lines of "what we don't like gets deleted", even though the "sole discretion" part already does that.

There really *is* no difference, in practice. Just one makes you feel a little safer in using the service. Which I suspect is exactly the reason why it's worded as it's worded.


no valio verga


In the future lets not create fixes for these kind of things. It can get out of hand very fast


Let's limit that to published only:
NewsPage.objects.live()


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18635/testReport/) for this commit**


Plus, if campers use this exploit it will show up on their solutions


Which was removed, but I still missed getDisplaySlotForColor.


Hi!
No problem, I have many ideas for the code... and work for it... Because
I want to help you and the others for the coding of soundrts...
regards

Am 03.08.2015 um 21:50 schrieb soundmud:
>
> Thanks for the pull request! I hope I will process it faster than the
> previous one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chpross/soundrts/commit/e8f7fd971f951eb1cf05d68a83545639dc826414#commitcomment-12501175>.
>


Will there be a way to dump and load tiles that are flagged as flipped (Bit5 and Bit6) separately?


Sure! I only did it as it is the only one that we have come across and it was easy to fix!


Ahhhh ahora me queda mejor la idea... Che igual, si dejás el .coffee ¿no te lo toma el intérprete y trata de convertirlo a .js? Por eso te decía lo de la extensión. Igual no te digo que lo cambies ni nada de eso, sólo trato de entender nomás como funca node en gral :p


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18636/testReport/) for this commit**


for some reason testthat was stopping with an error https://travis-ci.org/tdhock/animint/builds/73946422#L1704

seems like it had something to do with putting variables/functions in `aes` which are not defined explicitly in the data.frame.

anyways to avoid that error I just created new variables in each of the data.frames and use those variables inside of each `aes`.

now on my machine on firefox I get 3 test failures

```
> tests_run(filter="malaria")
1. Failure (at test-malaria.R#205): one line is rendered for each region -------
length(region.lines) not equal to nrow(malaria$regions)
14 - 12 == 2

2. Failure (at test-malaria.R#213): geom_point(aes(color)) legend shows as circle stroke
some[["fill"]] == none[["fill"]] isn't true

3. Failure (at test-malaria.R#250): none legend entry opacity 0.5 after clicking
none.after not equal to 0.5
0.5 - 1 == -0.5

testthat results ================================================================
OK: 10 SKIPPED: 0 FAILED: 3
1. Failure (at test-malaria.R#205): one line is rendered for each region
2. Failure (at test-malaria.R#213): geom_point(aes(color)) legend shows as circle stroke
3. Failure (at test-malaria.R#250): none legend entry opacity 0.5 after clicking

Error: testthat unit tests failed
```


this commit was based on a old version and should be ignored/deleted. Cannot do it myself.


Makes no sense. The old sample set up the concepts and the following examples (inheritance etc) build upon that.  Now it just jumps right into Inheritance. As an API guide, it should show actual examples too. Don't you hate referring to a formal API doc that has no examples? It drives 98% of developers nuts, and only people who don't want to do the work to make it right would be the other 2%. : \    This change is a cop out.


Add Bitdeli for analysiis


in _widely use method_  - s/use/used


An identical fix is needed for the other call to PlayTeleportChargeEffects a bit lower down for FakeGateInUnit.


(Might as well do that now as well)


Is there a particular reason why logging these is important?


This commit breaks test for relative imports, but only for Python 3.
```
PYTHONPATH= PYINSTALLER_RUNTESTS_WORKDIR= python3 ./runtests.py import/test_*relative*.py
Running test:  import/test_onefile_relative_import2
Running test:  import/test_onefile_relative_import3
Running test:  import/test_onefile_relative_import
test_onefile_relative_import2 (__main__.GenericTestCase) ... FAIL
test_onefile_relative_import3 (__main__.GenericTestCase) ... FAIL
test_onefile_relative_import (__main__.GenericTestCase) ... FAIL
...
... when running the exe:
Traceback (most recent call last):
  File "<string>", line 11, in <module>
ImportError: No module named 'relimp2'
```


Esta noche lo veo en casa y para mañana tienes la respuesta.

Saludos

El 3 de agosto de 2015, 16:44, Mariano Cerdeiro <notifications@github.com>
escribió:

> Esteban, fijate si podes re usar el test que acabo de hacer commit en tu
> branch:
> https://github.com/ciaa/Firmware/commits/master
>
> tenes que hacer un rebase o un merge del master en tu branch y luego correr
> el tests y explicarme donde el codigo original estaba mal, antes de cambiar
> el codigo me gustaria aclarar donde esta mal, yo sigo sin ver el
> problema... y es más si lo cambiamos me gustaria cambiarlo como las otras
> funciones y no como esta ahora... :(
>
> Saludos.
> Mariano.-
>
>
> 2015-08-03 21:15 GMT+02:00 Esteban Volentini <notifications@github.com>:
>
> > Mariano, la linea comentada es la linea origina y la otra es la linea que
> > queda. Es decir yo cambie la siguiente linea (la comente en lugar de
> > borrarla)
> >
> > ciaaPOSIX_assert(device->open(path, (ciaaDevices_deviceType
> > *)device->loLayer, oflag) == device->loLayer);
> >
> > por
> >
> > ciaaDioDevices_deviceType * dioDevice =
> > (ciaaDioDevices_deviceType*) device->layer;
> >
> > ciaaPOSIX_assert(dioDevice->device->open(path, (ciaaDevices_deviceType
> > *)device->loLayer, oflag) == device->loLayer);
> >
> > con lo cual el assert no cambia, lo que cambia es que hago
> > device->layer->device->open en lugar de device->open.
> >
> > Lo que pasa es que ademas hice este cambio de formato, estaba así
> >
> > extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
> > ciaaDevices_deviceType * device,
> > uint8_t const oflag)
> >
> > y quedo así
> >
> > extern ciaaDevices_deviceType * ciaaDioDevices_open(char const * path,
> > ciaaDevices_deviceType * device, uint8_t const oflag)
> >
> > como los dos cambios esta seguidos toma la llave como referencia y por
> eso
> > es mas difícil de leer.
> >
> >
> > El 3 de agosto de 2015, 14:42, Mariano Cerdeiro <
> notifications@github.com>
> > escribió:
> >
> > > yo sigo sin ver el error. pero bueno, para mi lo que hace falta es
> correr
> > > los tests :) los mismos se deberian poder correr en windows/linux y ahi
> > > probar si funciona todo bien sin el hw.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12498710
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12500571
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/e61f3ea2f32273ec351d4e7e99b7697217591af6#commitcomment-12501072>
> .
>


:+1: :+1: :+1:


Ok, I've missed this check, now it works properly with local files. Can I change the files by editing straight via github edit or provide another commit?


All that's left is getting `Module#prepend` merged


Silly question, but are tokens the only thing this could possibly affect? Are there other cases where we might send different credentials? (I'm not entirely clear on our credential situation)


This is what happens when someone is tasked with 'clean a bunch of stuff off the change list.'  Work that people spent hours on, and learning how to do it properly in the system get stepped on by someone who doesn't really care.


Hayoc konichiwa


ini stores nicks in square brackets so they cant be part of the nick and need to be replaced. as ~ is not a valid character for a nickname, its a good replace.
If not replaced. Key isnt saved ...

```
[~bla~]
key=+OK TaFzr0u6b3U1
```


As the original creator of #6261, this would be an acceptable fix for the accessibility requirements our team has, and I would be in favor of merging this into PDF.js.


Not currently using this yet, just installing for future use.


We had already decided not to go forward with maps on Saturday or thats what i thought we had. If i remember correctly we went through each and every point before we left that day. I may be wrong though since I had just bought a car that day :)


I accidently changed the namespace. Just change it back to NpmTaskRunner and then it works fine


can we use `segment` here? :)


I think it doesn't matter, whatever you like, but one way or the other it will result in commit anyway. You cannot update files in repository without commit.


Al principio de todo se carga este archivo
<https://github.com/rodri042/scoremonkey-server/blob/master/globals.js> que
le hace *require* al compilador de coffee. Al hacer eso no hace falta nadie
convierte los coffee's a js's y los almacena, sino que al *requerir* un
archivo el compilador lo hace por vos.

Como nadie hace ni querría hacer require("src/config/env.sample") el
archivo nunca se carga, no hay drama con que el archivo termine en *.coffee*
 :P

El 3 de agosto de 2015, 17:06, JSorella <notifications@github.com> escribió:

> Ahhhh ahora me queda mejor la idea... Che igual, si dejás el .coffee ¿no
> te lo toma el intérprete y trata de convertirlo a .js? Por eso te decía lo
> de la extensión. Igual no te digo que lo cambies ni nada de eso, sólo trato
> de entender nomás como funca node en gral :p
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rodri042/scoremonkey-server/commit/3c7f27c440f7e3a58a0095d826e85cb4f9b876b6#commitcomment-12501417>
> .
>


No, I mean the scene commands, some of them were changed in Gold and you don't have legacy versions listed here


@dchambers The test was added back in https://github.com/BladeRunnerJS/brjs/commit/810bd64eddde819473efb163f2e66dafb187f1c0.


FIX:
**
Al hacer eso no hace falta convertir los coffee's a js's y almacenarlos,
sino que al *requerir* un archivo el compilador lo hace por vos.
**

El 3 de agosto de 2015, 17:13, Rodrigo Alfonso <rodri042@gmail.com>
escribió:

> Al principio de todo se carga este archivo
> <https://github.com/rodri042/scoremonkey-server/blob/master/globals.js> que
> le hace *require* al compilador de coffee. Al hacer eso no hace falta
> nadie convierte los coffee's a js's y los almacena, sino que al *requerir* un
> archivo el compilador lo hace por vos.
>
> Como nadie hace ni querría hacer require("src/config/env.sample") el
> archivo nunca se carga, no hay drama con que el archivo termine en
> *.coffee* :P
>
> El 3 de agosto de 2015, 17:06, JSorella <notifications@github.com>
> escribió:
>
>> Ahhhh ahora me queda mejor la idea... Che igual, si dejás el .coffee ¿no
>> te lo toma el intérprete y trata de convertirlo a .js? Por eso te decía lo
>> de la extensión. Igual no te digo que lo cambies ni nada de eso, sólo trato
>> de entender nomás como funca node en gral :p
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/rodri042/scoremonkey-server/commit/3c7f27c440f7e3a58a0095d826e85cb4f9b876b6#commitcomment-12501417>
>> .
>>
>
>


@javajolt https://github.com/basho/aws-ansible and https://github.com/basho/DevOps/wiki/How-to-test-and-deploy-Giddyup


Ouais je sais, mais c'est pas vraiment une URL en fait; juste l'identifiant utilisé pour générer l'URL. Ca va peut être changer de leur côté, si oui je passerai à URLField.


Yep, agreed. I'll take a look at the API - we can probably remove some methods after some of the refactoring.


Ahhh genial, es como el "autoloader" de PHP, ya me queda más claro :D
Gracias por la explicación Rodri!


De nadas!


Nice! Do you think there would be a shared place where you could put `TOP_NUM` in one place rather than in each individual model?


maybe add units here too? sec/min


Aliases are definitely (IMO) a plugin concern. All the aliases logic is done as plugins rather than requiring any changes to the core BRJS and model. It's probably better to look at the finished PR rather than these individual commits as I suggest to Ioana she refactor this and the finished class looks much cleaner - https://github.com/BladeRunnerJS/brjs/blob/app-level-alias-override-1071/plugins/brjs-plugins/src/main/java/org/bladerunnerjs/plugin/bundlers/aliasing/AliasesFile.java.


I'm curious what made you attach this method to the previous function rather than create a new function itself? I haven't seen this style before :smile:


I agree that there's a bit of a disconnect between the guide and the API pages, but the guide imo does a better job at introducing the core scope concepts. And the API shows actual examples, right where the methods are, which also do a better job at explaining because they have context, which the introductory example did not have.
I'm not really sure what you mean by "Work that people spent hours on, and learning how to do it properly in the system get stepped on". Not every addition to the docs improves them. This particular passage was also gone for two years, and you were the first to even notice / make us aware of it.
That said, I'm open to further changes to the API docs, especially to the introductory paragraph.


:+1: We can.


@magnumripper sorry, I think this was a missed --rebase on a pull.  I can not figure out how to purge this darn thing.


Should this be combined with the above into a `elsif`?


TODO: show notice or flash in view


Hey @staticfloat sorry to bug you, could you delete `repomd.xml` from the cache? I'm about to turn this up to 11 and use it for everyone on WinRPM https://github.com/JuliaLang/WinRPM.jl/pull/46


@cboettig https://packages.ropensci.org/ doesn't resolve for me, just `http`


@fmalpartida, @Wackerbarth, and I had some discussion about ways to preserve the bed leveling information during the Hangout on Saturday. Mostly, we talked about saving it across reboots. I'm not sure how "clever" we want Marlin to be about this. One problem is that if you home with the leveling matrix left on, the homing moves are made with the matrix, not just raw machine coordinates. So, if you `G28 X` to just home a single axis, you will get Z movement also. And, when you `G28 Z` alone, you will get XY movement. Users might find this strange.


Satisfy the Foodcritic, eh?


I believe it was to fix a previous bad commit. The idea behind the overall change was to allow a class to be passed in rather than a classname so `fixture(require('myClass'))` could be used instead of `require('myClass'); fixture('myClass')`.


So the valid way to request a reboot would be to `touch` this file?

or should we say `notifies :request_reboot, 'reboot[catchall_reboot_handler]'`


Sorry for the delay here @pioul, this looks great to me, awesome job with the comments in the code, made it easy to review :+1:

One thought, and this would likely be a super small use case here is what about sites that update the URL with `pushState` or a hashchange and have client side rendering of articles. One example is [mic.com](http://mic.com/articles/123305/obama-just-unveiled-his-clean-power-plan) - scroll past the end of an article and a new article appears and renders on the client with the URL updating as well. Maybe we could check the current url somehow? Potentially we could re-build the URL and compare `iframe.src`, but I'm not sure if that's the best way to do it :wink: What do you think?


@cboettig you could mention we have an API in dev to replace the hand maintained registry, but not crucial


To reverse url to the homepage you might use this snippet:

In [6]: reverse("wagtail_serve", args=[""])
Out[6]: '/'

Use reverse for rest of urls as well


.live() too


delete


It's removed in a later commit (https://github.com/BladeRunnerJS/brjs/blob/convert-libs-commonjs-424/brjs-sdk/sdk/libs/javascript/br-component/src/br/component/testing/ComponentFixture.js).


I can't add `|| !rawHosts.Contains(';')` because `127.0.0.1:4321`, which has no `;`, is also a valid hosts list.


Personally, I find the elif syntax pretty gross.  You also switched to using [[ ]] for just the one there.


del


You show that .DS_Store who's boss!!!