Ah I see. Is it feasible to infer the heading name from the path of the
`TableConfiguration`, or in some other way? If its just a presentation
detail, I think it is preferable not to store this field on the model.
Another thought is that `name` could also be a property instead of a model
field, which might make it more clear how it's calculated and enforce its
read-only nature.

On Thu, Feb 4, 2016 at 3:49 PM, Ben <notifications@github.com> wrote:

> the tables in the UI have basically two names. you see the header for the
> main table is always Forms, even if you change the sheet name (which
> defaults to Forms). so there had to be a way to track both
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/a301c9bd8ee039854907dc4735181fa1b08a1335#commitcomment-15897371>
> .
>


ok, ingo said that this is the new way to do it without npm warning, i'm good.


Would it make more sense to keep the styling in the template? This looks like an easy mistake to make again.


lol


@espinosa Yeah, I know. But it's better than the `1.8` default. >_<


Your base tag needs to be "master" with the correct bench.


fix


so do you mean i can just use text-blend-order and have alpha? i thought i needed blend:overlay  in there


@pestOO, does it help to see some address in logs?


Ay bud. These changes are a no go. 420 blaze lyfe fix ur code scrub l0rd. Tri ch4ang1ng the gr8er than values.


@espinosa Oh snap. Can you create an issue so I can remember to fix this later?


you should PR this.


barion efficiency header shouldn't be added to 'monsters' csv. only visit and runes


its for offline charger


ops


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.492.0](http://build01:8888/viewLog.html?buildId=14100&buildTypeId=ConfigInjector_CI) is now running


hah nvm u did :/ , https://github.com/thlorenz/flamegraph/pull/12


sorry about that, there you go


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.492.0](http://build01:8888/viewLog.html?buildId=14100&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


Capitalize `presto`.


Just my personal choice, but I prefer to have functions defined before their use (so, first define rune_efficiency first, then the BarionEfficiencyPlugin which uses the function)


@suhailshergill The only case, I aware of, when `onRecoveryCompleted` won't be called is when the recovery is failed. When recovery fails the actor stops. In my opinion, the strategy of dealing with actor failures has a wider scope than the current bugfix. Therefore, I suggest to open another issue to continue this discussion.


The following doc(s) have been deployed to the [wpsolr](http://wpsolr-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/drupal/solr)
- [docs/articles/sites/apache-solr](http://wpsolr-static-docs.pantheon.io/docs/articles/sites/apache-solr)
- [docs/articles/wordpress/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/wordpress/solr)


Nah, that wasn't disagreement, just a bit of surprise. :) I'm totally fine with this as, especially since we can just give the `view` in each result an appropriate class with a nice font.


I have a question about this.
If I look at the actor creation in `Receptionist`, it seems to be that either `responses` should be filled or `responseprovider` should be filled.
If this is the case maybe we can make them mutually exclusive using an `Either`. This way we can have proper coverage on the exception thrown  in `Receptionist` because we don't need it anymore.


Ignoring errors is better than panicking. People's bosses will get angry at them for using `qq` when a production server crashes because `qq.Println()` call got left in the source and panicked. If think we should suppress all panics that bubble up from below, through something like this:
```
func Println(a ...interface{}) {
    defer func() {
        recover()
    }()
    ...
}
```


I'd say the plugins array should be an argument here because once the parse_visit/parse_login_data moves into its own plugin, it will need access to the plugins to call process_csv_row on them.


Its a more serious issue. I began to work at it but was called away today.

Sent from phone, thus brief.
Am 04.02.2016 19:23 schrieb "Peter Portante" <notifications@github.com>:

> It looks like there are a few paths through this code where tpls is not
> referenced, so a fix would be to only check tpls when for the dynSrchType
> and dynBulkId cases.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rgerhards/rsyslog/commit/05e04ac84f2350dce12306c0d36ef35e1fe2449f#commitcomment-15893896>
> .
>


Why are we removing alsa-plugins.

I'm still using it for custom asound.conf pcm plugins.


- **трансформувати**

- **вставляти їх**


Well i did the junction trick but I ended up with some compilation errors (includes issues). Anyway it looks like with your version and the path of the MS toolchain it's working perfectly!

Thanks for sharing!

Perhaps you could share it to epic so that they might distribute it for everybody?

I've started some tests and currently my distributed version is compiling slower than the non distributed one. As I'm at home with 2 very different pc it might explain why I'm seeing this.

Anyway even out of the box fastbuild is fater! I used a lot when I was working at ubisoft.

Thx again!


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 0 test warnings)

![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)


Yep, you can directly do that. It's the default blending mode for text.


- **з** опрацюванн**Я**


:thumbsup:


I would rather write: Amount of required memory.


The AWS metadata service returns a 404 when your request is for a non-existent public IP, and the AWS SDK hides this in an 'Unknown Error'

https://github.com/kubernetes/kubernetes/blob/master/Godeps/_workspace/src/github.com/aws/aws-sdk-go/aws/corehandlers/handlers.go#L97-L103

I don't think there's any way to work around this, short of either patching the AWS SDK or using a much simpler EC2 metadata client - it is a very simple HTTP based API.


Again, I'm a little confused (playing dumb), what additional configuration is no longer needed? Do I not need to specify any of the other two properties?


Thanks! At first I did choose the test type regression when I set up the test and it changed my base... When I use the standard type everything works ok!


...in the right way


Linear history is dead :disappointed:


Podemos eliminar todo este código comentado, a fin de cuentas nos ahorraría código


> should't this apply to Windows and not UNIX??

?  I think you're misreading the cmake code.  It is in the else of if (UNIX).


@fjvallarino I took this out because the CAP error message was too long for this max height. Everything seemed fine after this change.


```PHP
'attr' => array(
    'class' => 'form-control',
),
```
Ese código ya podemos eliminarlo, desde que fue adicionado el template en symfony para soporte a bootstrap no es necesario.


Hi Chris,

When i try to generate the solr-adapter with ember i get the following error:

`lawsuit@localhost:~/lawsuitember$ ember generate solr-adapter application --url=http://localhost:8983/solr
version: 1.13.15
The "enableRealTimeGet" option's name of the "generate" command contains a capital letter.
Error: The "enableRealTimeGet" option's name of the "generate" command contains a capital letter.
    at Class.Command.validateOption (/home/lawsuit/lawsuitember/node_modules/ember-cli/lib/models/command.js:254:11)
    at Array.map (native)
    at Class.Command.registerOptions (/home/lawsuit/lawsuitember/node_modules/ember-cli/lib/models/command.js:87:25)
    at Class.module.exports.Command.extend.beforeRun (/home/lawsuit/lawsuitember/node_modules/ember-cli/lib/commands/generate.js:39:12)
    at CLI.<anonymous> (/home/lawsuit/lawsuitember/node_modules/ember-cli/lib/cli/cli.js:83:13)
    at lib$rsvp$$internal$$tryCatch (/home/lawsuit/lawsuitember/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:493:16)
    at lib$rsvp$$internal$$invokeCallback (/home/lawsuit/lawsuitember/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:505:17)
    at lib$rsvp$$internal$$publish (/home/lawsuit/lawsuitember/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:476:11)
    at lib$rsvp$asap$$flush (/home/lawsuit/lawsuitember/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1198:9)
    at nextTickCallbackWith0Args (node.js:419:9)
`

Apparently the options can not have capitals... I have this error both on my win8 x64 and Debian x64 machines.

Is this a Bug? :)


How does one own a session? Which setting controls that?


Igual que el comentario anterior.
De ahora en adelante donde quiera que encuentres esa instrucción la puedes eliminar.


Igual que el comentario anterior


@Memphiz the linked code was written by @anssih Anssi can you help here?


what's with the empty print haha


Typo: impoersonate


those decks seem like an ideal candidate for an array or dict


_й надалі_ або _в наступному рядку_


Creo que podemos hacernos de una clase tipo constante, que contenga los tipos de movimiento a mostrar en choices, organizados por una especie de código, e internacionalizarlo, para poder cambiar a gusto del consumidor.


this is the most unexpectedly reused code from the new site


I like that we don't need to run this script as root. If we put an `apt-get` in there we need additional permissions. I can defend not installing those deps. We already have `install_rippled_depends_ubuntu.sh` that installs all the deps for a gcc build. This is the script as only installing the additional deps we need to we point to build rippled with clang.


I do, if I want to avoid a warning: `JLOG(x)` is defined as `if (!x) { } else x` so the compiler tries to be helpful.


You can upgrade to TeaVM 0.4.3 as well


"This is done in the Hadoop configuration file" -> "Impersonation in Hadoop is configured via the file ... "


This commit is very sad :cry: :smirk:


@jef-n any chance we could get these additional dependencies added to the nightly builds?


nice commit, nice python-like improvements :+1:


custom.pro is still not ignored... I guess I don't get it.


"is provided" -> "can be found"


LGTM


O operador NOT ele inverte os bits, ou seja, o que é 1 vai ser 0 e o que é 0 vai ser 1
Exemplo:
O número 12 em binário é 00001100, certo??

Aplicando o operador NOT, teremos:
0000 1100

1111 0011

Viu só, o NOT inverteu todos os bits de 00001100 (12), assim obtendo o número 11110011 (243).


продовження поточної лінії,_**(кома**_) а не початок..


wow, what a weird bug. forgot a comma and `'tag_push_events'\n'build_events' got merged into one attribute.


Tilde en Bitácora


OK, I can fix this.


Trying to make sense of your changes. I have a WD01 and initially found digidem's code for automatically backing up from sdcard to USB. Why did you remove these? To me this was the reason I researched modifying the device.


Yeah. Any magic numbers you find should be considered a bug. Although here we're passing in SQL fragments, so it does make it slightly messy. Still worth it to replace the magic numbers, though.


What do you think about installing clang in the `install_rippled_depends_ubuntu.sh` script instead? We already need to run that script as root. Or am I making too big a deal about keeping `sudo` out of this script?


Isn't there documentation on this? If so, include links to it @lawinslow


Should be using the same filters as the C2 test

@validators = [::Validators::QrdaCat3Validator.new(product_test.expected_results),
                   ::Validators::ExpectedResultsValidator.new(product_test.expected_results)]


Documentation on the availability of source? Not that I know of. They call it an "open source" project, but when I asked about the source, they said it was only available on request.


this is not part of auido will be moved


If we destroy a course, then we destroy the other records that are tied to it essentially.

A course can be soft-deleted by setting 'listed' to false, but if we actually delete the course, we need to clean up all records that index on it.


For now. At some point, we may be able to actually calculate word counts by counting words, instead of guesstimating based on exploratory calculations from dissimilar groups of editors.


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 20](https://teamcity.labkey.org/viewLog.html?buildId=301089&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **FAILURE**
Summary: Tests failed: 10 (6 new), passed: 245 Build time: 00:59:44
### Failed tests
```
org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testPublicationDetail: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCardStudyLinks: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testFilterOnStatus: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOwithCASFromHeaderLink: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testNewUserSSO: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOWithCASFromLoginPage: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testCASUnreachable: <no details avaliable>

```


thanks karim! its working now @karimnaaji
its on the new build


Testing an edit


Cat 3 validators should be the same as the C2 test

`@validators = [::Validators::QrdaCat3Validator.new(product_test.expected_results),
                   ::Validators::ExpectedResultsValidator.new(product_test.expected_results)]`


-s


:zap: Good find @karimnaaji!


Why? I use this in my plugin -,-


Can we hold on this? I have a counter-proposal for all this cross-package public-but-private junk we're adding. I want to unify it all into one turd.


Other than my site, I don't.  I've been meaning to look for non-profits who
would like website assistance but have been focusing on my CAPM exam.  Need
to find a better balance though. If I come across anything in the future,
I'll hit u up so we can both get the experience.
On Feb 4, 2016 3:20 PM, "SollloDolllo" <notifications@github.com> wrote:

> Yo,
>
> Finally got some down time to check out GitHub and it seems pretty slick.
> Decided to comment on your latest commits since GitHub doesn't really have
> private messaging. You have any active plans or ongoing projects?
> Definitely willing to help.
>
> One!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Deceptivesmiles/ProjectRedLetter/commit/ccfea47d2675d789bd600d90e750275ac9bd3f74#commitcomment-15896686>
> .
>


##MarkDown Test
* my first comment
* my second bullet comment


can this commit be amended in the previous one ?


@suhailshergill This class was used to test only in a context of a single run. In this commit I added tests covering multiple runs, and it's more convenient to assert the result reflecting only the stage switches of the current run, not the full history. That's why I put the logic to ignore the stage switches replayed from recovery process.


It might be more trouble than it's worth right now, but it would be nice to not need cloudfront for staging.


:+1: Thanks for updating this! Using it in my dotfiles repo :)


Does it need to be `**/custom.*`?


My feeling is that Kerberos support for Hive connector should be part of (or subpage of) hive.rst. It is not general feature shared between many connectors - but hive specific stuff.
If we extend it to support multiple connectors with common semantics - then separate section would be fine. For now it seems keeping it as part of hive connector doc makes more sense.


Why complicate this? wouldn't str(v) be a noop if v is already a string? or would str(v) cause an error if v is already a  unicode string ?


Because nobody besides you is using it ... and until now it was not even known that it has exactly one user.


...але, по суті,**_(дві коми)_** така поведінк**а** визначається як баг, **тому** більшість розробників радять не користуватися **таким** трюком.


You can do this with `swiftValueDecl->getType()->equals(stringTy)`. No need to go to the canonical type that way.


That's a good point. I agree.


Thanks for the fix. I have to ask: what's the point in pushing pull requests if you haven't installed Haiku yet?


This doesn't make sense. Windows users don't need Anaconda


El número tiene que estar en una variable estática y final (todo en mayúscula y separado por guión bajo)


Actually, online services can no longer be hidden, not stream sources.
(Jemendo, Magnatune, SoundCloud, Podcasts)


Thanks :fearful:


Let's add something about "attempted jQuery selector:..."


Agreed. I'll drop this commit.


AH! okay, figured it was open source, nevermind


Que hace esto?


No worries :)


ah, ok, or just don't have a staging.


@sensescape Looking great with some alpha in the app! :)


Hi


@oliviergaertner I had those same include errors, thankfully they were all simple fixes and there wasn't anything nasty. There's a preprocessor bug that pops up if you force warnings as errors on so let me know if you plan to and I'll track down my fix.

Yeah I'm definitely planning to put in a pull request to Epic. I want to clean it up first since it's still pretty messy and there's still performance improvements I can think of. The way I handle dependencies causes a bit of a slow down I think.

Ah yes, there is a point where distributed will be slower than non-distributed, especially if you're not using the cache since it skilps preprocessing. If you're using the cache, *shrugs*, may need more machines. IIRC, fastbuild supports 15 simultaneous workers. With all 15 connected I got full rebuilds in around 6-7 minutes.

I forgot to update the Wiki post, but you'll be able to enable PCHs now and should see some speed up if you enable them. With my 12 thread machine and a fully populated cache I can do a full rebuild in about 4-5 minutes. Once you have a fully populated cache, the distribution part of it barely gets used.


This comment seems specious to me. What else would DailyScraper.__init__ do?


If this is a bug that eventually will get fixed, it should be a `span_bug`. It it is not a bug (seems to not be a case from the fixme above), this code should be changed to bail out from const-evaluation so the dependents could proceed onto non-const path.


> Given a paymentId, we can only have one purchase (the check is for safety)

But is that true now? If i set `AUTO_PAY_OFF`, then make N partial payments to pay that invoice, what happens?

My confusion comes from the fact that we introduced both partial payments and partial refunds, i intuitively i expect some symmetry between the two (and also a 1 INV -> N PAY/ REF model)


Is this name required for some interface? It might be nice to add a comment about the expectations on `args`.


@tomgarcia Can you put the previous, old code, as a commented section and then add your new code after it? This helps when we need to update this plugin and need to resolve merge conflicts between what we added and what was there originally.


maybe explain what `_HOST` means. That it will be replaced with fqdn at runtime.


@suhailshergill
Even though this class can be used by library users, it was created for `eigenflow` internal use. I don't see a reason to hide it. But I wouldn't encourage using it from outside neither, mainly because it's quite specific to our use case.


LGTM


Thanks ! ^^
I wanted to be more modular on filters, so that you can build any filter chain with basic filters.
I think you can even create time-aware pid (this implementation isn't) with an other chain.


'Spint' ist nahe an 'Splint', 'Spind' und 'spinnt'. Deswegen: ohne ist
besser.

Am 04.02.2016 um 19:45 schrieb f-knorr:
> Spint war eine alte deutsche Maßeinheit. Ich nehm's wieder raus...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/549b51d9d30c3423a879d667bfee53e25d8fa199#commitcomment-15894347>.
>


Was looking though my local code.  This line jammed me.  I tabbed that out one to be inside of "elif '@' in username:" in my copy seemed to fix it without the need of modifing the 'fields' dictionaries.  Also I looked at the master branch and that seems to have been fixed there already.


s/objection/object in the commit description.


s/be able/be allowed/

Also you may reference hive config variable name used to set this up.


In general, many of the method comments/documentation fragments are simply restatements of the names of the methods. I find that they are not useful at all, and may only be there to satisfy pep8. It's not a big deal; I won't hold up the review. But I hate extraneous comments that don't actually say anything.


The scripts still work without the sudo on the file permissions since the script will need to be called with the sudo command either way.
On line 11 the source folder for each student (/home/student1) is changed to allow users in an authorized group to have full permissions in their folders as well (student assistance, missing homework, etc).


@whimboo r+


Sam, plz no prints (couts) in header files... (especially when you break the build by not including <iostream> ....). Test build before you commit plz thx


Explicitly state that this setting does not use Kerberos at all.


I agree generally, but we also have admin/kerberos.rst on our phase2-on-127 which we've never upstreamed. I feel like it would be appropriate to put it in kerberos/cli.rst.

Thoughts?


> Is this solving a circular dependency or something? Why would we not push it upstream?

for some reason they dont' hit it.


Also here - no kerberos is used.


**з** відступами


@karimnaaji thats a nice catch, :+1:


typo on line 165: "jut"

(sorry I cannot get github to show me the line by line code.)


Added ability to add images


*will* be ignored


Why here?


Do *note* grant... should be Do *not* grant....


Technically it uses principal from `hive.hdfs.presto.principal` to connect to HDFS.
The impersonation is just another layer - on top of kerberos. I would say that principal impersonates the user who owns the Presto session. Not state that user who own the session is used as kerberos principal.


You usually put your email address here, too.


@isysd just saw your comment, sounds good. Tried using Calendly, got this message:

"Ira Miller has no openings at the moment."


Die Aussage an sich ist aber ganz schön gedehnt. Der ganze Schutzzauber block ist mehr schlecht als recht angepasst worden.


Oh - here it is - ignore my previous comment on hive config :)


There's one more level of indirection here than seems necessary. How about just having \Idno\Core\Logging implement \Psr\Log\LoggerInterface directly?


You repeat maven integration a few times in this page. Might be better just saying at the top that this feature only works with maven at the moment and then assume maven in the rest of the page.


Is this what spawned https://github.com/SUSE/Enceladus/issues/71 ?


I'm not sure if we should call this the "Liferay public API" any more. I would call it the API of portal-service or something like that.

The reason is that now that Liferay is so modular, each module will have its own public API. While portal-service is not an OSGi module I would still treat it as a module with respect to this.

What do you think?


I like this!

We should probably also have a document somewhere with a set of named test-cases that each client should try.


'false' should be false; there's a few instances throughout


Is this useful?  Filtering on record type when finding duplicates makes sense, but you're using record.schema.record_type for that.  Storing record_type on the RecordDuplicate denormalizes a bit since it's also available through the record.


When using sass you can use nested selectors. this can be written:
```
.carousel-control {
  &:hover {
    background-color: transparent;
   }
}

```


Hmm... I suppose it wouldn't hurt to listen to ``exit`` too, but I wouldn't be surprised if that doesn't catch it either...


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 400.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -397,7 +397,7 @@
                      time in UTC is used
     Added new parameter for width to resolve bug #325
     :param width: either "long" or "short" or "extra"
-
+
     :param locale: the `Locale` object, or a locale string
     """
     datetime = _ensure_datetime_tzinfo(_get_datetime(datetime))
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1339.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -1336,7 +1336,7 @@
         return week_number

 #Added 'X' and 'x' in PATTERN_CHARS for to include datetime format support for them.
-#Added '5' in 'Z' list to include support for format 'ZZZZZ'
+#Added '5' in 'Z' list to include support for format 'ZZZZZ'

 PATTERN_CHARS = {
     'G': [1, 2, 3, 4, 5],                                           # era
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1340.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -1337,7 +1337,7 @@

 #Added 'X' and 'x' in PATTERN_CHARS for to include datetime format support for them.
 #Added '5' in 'Z' list to include support for format 'ZZZZZ'
-
+
 PATTERN_CHARS = {
     'G': [1, 2, 3, 4, 5],                                           # era
     'y': None, 'Y': None, 'u': None,                                # year
```


**Line is longer than allowed. (123 > 120)**
*LineLengthBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 1353 column 121 to line 1353 column 124*.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `default`.* *Related to file `babel/dates.py`, line 394.*

The issue can be fixed by applying the following patch:

```diff
--- a/babel/dates.py
+++ b/babel/dates.py
@@ -391,7 +391,7 @@
     u'GMT-07:00'
     >>> get_timezone_gmt(dt, locale='en',x_format=True,width='short')
     u'-07'
-
+
     The long format depends on the locale, for example in France the acronym
     UTC string is used instead of GMT:

```


These parameters are reversed -- level is first.


Einheiten trenner


WOuld be good if this value can be changed dynamically (pagination)


`xcalib` is not passing compiler flags to `make`, this needs to be addressed.


When I modified `eval_const_expr_partially` to return `Err` in this case, I still got a compile error, so...

But yeah this is a bug that will get fixed at some point in the future.


nit: can you update the docstring to include all three parameters and their types?


So if `wcjsRendererOptions` and `wcjs` are not set, then it will just pass `null` to the renderer's init method and use the default settings (ie: create a `wcjs` object)?


I think this covers the most obvious use-cases. I think it's fair to assume that supporting a jQuery result with multiple elements will be useful and it can be supported pretty easily.

Your first test should be is it a string and turn it into an array.
If it is still not an array, confirm it is a jQuery object. Loop through it and create an array, replace the original variable.
Then loop through the array and if an item is a string, replace it with the jQuery result. If the item is neither, log an error.
You should assign the array to the object property as the last step and do it un-conditionally.

... we should not be throwing exceptions here. Have mercy, logging the error is enough, we don't need to halt everything.


Can we look at generating these lists from the image configuration, dynamically?


i was just trying learning about github.. it's so new for me!  now i,m happy because  you  restarting the conversation ! nice!


The current code in master branch would make 2 request here


Looks good to me.


The current code in master branch would make 27 request here


The level should be a string here for PSR3. I was thinking for backward compatibility, detect the case where message is an int (the signature used to be message,level),  convert it to the PSR3 string level, and then swap message and level


You mean your directory tree was:
```
engine/src/sky/etc
engine/src/myservice
```
...?

If so that seems like asking for trouble. Surely we should encourage this style instead:
```
.../flutter
.../engine/src
.../myservice
```


Yeah, it displays all keycodes and numbers, that's just to be useful. That display can be removed if we want.


It sure does.


restart works fine for me.


Sería bueno poner aquí alertas de  correo existente y de password que no cumpla con complejidad


Yup, there are no sensible defaults for the others.


This is wrong.  That parameter should be EITHER a bool or a string.
Check the readme from puppetboard... https://github.com/voxpupuli/puppetboard/blob/master/README.rst#id16

Further... if you look at the app.py file where it uses the PUPPETDB_SSL_VERIFY it is using the pypuppetdb.connect method which has the following in its docstring:
:param ssl_verify: (optional) Verify PuppetDB server certificate.
:type ssl_verify: :obj:`bool` or :obj:`string` True, False or filesystem             path to CA certificate.

Also... this breaks PuppetDB SSL auth.  That arg needs to point to the CA cert.


It makes me so happy that you moved this 'only' to the correct place.


Looks good in up and coming release.
https://github.com/aspnet/Hosting/blob/dev/src/Microsoft.AspNetCore.Hosting/WebHostBuilderExtensions.cs#L65


You are repeating the `$font-headings` & `$font-stack` definition in different files. I would recommend you to create a file with your generics (colors, fonts, general style rules), let's say 'base.scss';, then in your application scss you can include this file to `@import "base";` before loading other files, so that the varibles/styles defined there get available to other files.


Oo, hört sich nicht nach einem gesunden Effekt an, würd ich nicht auf mich zaubern/anwenden.


Poner aquí alerta de Password o usuario Incorrecto


If a task is not deployed yet the RPC would fail, right? We could detect this and retry to send the stop signal (ie, for job state RUNNING). For job state CREATED a direct state transition to FINISHED might not be too invasive for the state machine (If I understand it correct this would happen only in the JM, ie, no distributed race conditions with the TMs to guard against, right?). On state RESTARTING we could delay until RUNNING is reached.

However, I think it is hard to guard against an error after STOP signal was delivered correctly. If all task are running and all receive STOP signal without an error. It could take some time before all tasks return from `run()`. If an error occurs during this period, the failing task would get restarted and STOP is lost for this task.

I am not sure, how ofter users will use STOP in a client application... To get the code right there might be tricky. (Kind of a re-send STOP with certain delays until if finished maybe). For a user interaction with the WebFrontend or CLI, this should not be too problematic -- the user will observe that the job does not finish and can issue STOP command again.

@tillrohrmann IMHO, this would be a good compromise. What is Stephan's take on this?


I *think* this class can implement \Psr\Log\LoggerInterface directly, and merge it with KnownLogger. Also you can extend /Psr\Log\AbstractLogger and remove all the critical/error/warning/etc. methods.


@bqpd: Prof. Drela reports this is not the correct virtualenv instructions for linux. He says `$DESTINATION_DIR/bin` is a directory that `source` has no idea how to activate; when he `cd`'s into `$DESTINATION_DIR/bin`, he can then run `source activate`


Actually, javascript will set the value of the property to undefined
(because it is undefined) and all of the wcjs-render code will process the
same way as if they have not been set (you can actually do some tests if
you like, or look at their code), which is exactly the behavior we are
looking for

Delian

On Thu, Feb 4, 2016 at 9:40 PM, Alexandru Branza <notifications@github.com>
wrote:

> So if wcjsRendererOptions and wcjs are not set, then it will just pass
> null to the renderer's init method and use the default settings (ie:
> create a wcjs object)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/delian/wcjs-player/commit/9109e7e9fb5819340e5db8fc4ed378299b994d2d#commitcomment-15898702>
> .
>


You've called the same function with the same args twice. Why didn't you cache the result in a var?


I understand the rationale for this change, but FYI it breaks the ability to change attribute name formatting as explained in the `README`. I have this in a serializer:

```ruby
     has_many :load_operations

    # Override this to customize how attribute names are formatted.
    # By default, attribute names are dasherized per the spec naming recommendations but that sucks
    # for deserializing objects
    # @param attribute_name [String] what to underscore
    # @return [String] underscored attribute name
    def format_name(attribute_name)
      attribute_name.to_s.underscore
    end

    # The opposite of format_name. We're supposed to override this if you override format_name.
    # @param attribute_name [String] what to dasherize
    # @return [String] dasherized attribute name
    def unformat_name(attribute_name)
      attribute_name.to_s.dasherize
    end
```

But now I can't do:
```ruby
    message = LoadRequestSerializer.serialize(load_request,{
      include: %w(load_operations)
    })
```

Not sure if there's a way around that (perhaps a more declarative way of controlling dasherized-vs-undescored attribute naming conventions)
PS thanks for this gem, it saved me a lot of time and pointed me in a good direction for a new API I'm building


good job


nit: changing the indent level of every line in the file makes the revision history hard to read (plus it doesn't match the style of the rest of the files anymore)


I thought about this, and I'm almost certain that we can guarantee `len(names) == len(values)`. If they weren't equal, it would mean `go/parser` screwed up parsing the argument list. I think that's sufficiently unlikely that we can skip the error check.


**фактично** вважа**ю**ться


That sounds like another option. We can then have another page, kerberos/server.rst that then redirects you to the kerberos section for each connector.


Yes.  This code used to print 4 analogs, now it prints 5 analogs.  The decoder is expecting five.  I decided to turn off the actual reading of the first 4, because they're not needed.  The 4 zeros are still needed for it to decode properly.


It should me urn:xmpp:mam:1 I guess.


Have you tried loading and using different 'font-weight' values?. This simple trick (using different weight for same font on different places) can add some life to your design.


Not needed, I know the renderer well enough.


Maybe you could have i2c_open_device_file() to receive a SOL_STATIC_ARRAY_SIZE(blah) and do the snprintf in it. Other than that, r+


I took your advice and refactored this. I used an `else` instead of `continue` because I think it's 0.1% more readable that way.


Boolean, no quotes (again, unless addressed in another commit)


Not sure what you were trying to fix here, but this change breaks FALCON_unzip as it tries to match the read names from the subreads.fasta with the 'fakemoviename' prefixed reads from the falcon assembly.


what will 'evidential' do?


Your edits are fantastic! Great work man!


I am pretty sure about that :)

Delian

On Thu, Feb 4, 2016 at 9:45 PM, Alexandru Branza <notifications@github.com>
wrote:

> Not needed, I know the renderer well enough.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/delian/wcjs-player/commit/9109e7e9fb5819340e5db8fc4ed378299b994d2d#commitcomment-15898825>
> .
>


Is that possible? I think if you use the Airlift configuration related classes you're forced to provide a default. I may be wrong.


try writing this using the sass syntax that makes reference to the parent selector:

http://sass-lang.com/documentation/file.SASS_REFERENCE.html#referencing_parent_selectors_


the true read ID needed to be tracked for FALCON-unzip. I guess this change is related to smrtpipe integration,


the "Overlapping impls and specialization" section does not currently have any examples that use the associated-type equivalence constraint `Trait<AssocType=ConcreteType>`.

I *think* such cases may just fall under the same category as the `where` clause examples that are shown here, but it might be nice to actually spell one of these out explicitly here since you've done such a great job of enumerating all these other examples for the reader.


So I did this because I actually encountered a problem where the html build repo had changed but I was on an older version of the html repo. If you remove this line it makes it such that older hashes of your html repo cannot be built if you introduce backwards incompatible changes to the html-build repo. I know it may seem like a pain to update this with new versions if your build scripts change but it will also mean your dockerfile can continue to work for previous versions if you have to return back and see what it looked liked at a point in history.


wait so if we have the inputs then what are the left click and right click for?


The following doc(s) have been deployed to the [wpsolr](http://wpsolr-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/drupal/solr)
- [docs/articles/sites/apache-solr](http://wpsolr-static-docs.pantheon.io/docs/articles/sites/apache-solr)
- [docs/articles/wordpress/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/wordpress/solr)


a comment, when I have multiple movies in one file, I have a script to split them into individual files to be conformed with daligner's input.


Прошу сделать так, чтобы этот коммит ответвлялся от текущей devel. Иначе мне мёржиться, мягко говоря, неприятно - слишком далеко общий предок.
Это, кстати, тот случай, когда проще руками, чем ребейз.


Will do. I cannot stress the pre-alpha nature of this entire workflow enough :)


@rgatkinson Is there any reason why the phone name check was removed?


let's not commit unnecessary scaffolding please.


Thank you!


ayy lmao


No, you aren't forced to provide a default.


Ah, good call. How should we describe package installation in python x-y? I unfortunately don't have a windows computer to test on.


Returning a success or failure value from the add/remove functions would be possible, but I don't think there's anything useful that the caller could do with that information.

They're all called from the GLX dispatch stubs, not from the vendor library proper, and the dispatch stubs generally aren't going to know how to recover from a failed call.

The __GLdispatchPatchCallbacks struct is there as an optional optimization, which can allow a vendor library to skip the overhead of looking up each OpenGL function in libGLdispatch's dispatch table. For example, Mesa could use some variation of the entry_generate functions to overwrite the entrypoints in libGLdispatch with its own dispatch code.


I've started creating PRs for edits that seemed OK, with the intention to install Haiku asap... but for various personal reasons I have not yet started the installation process... (Please accept my apologies for any recipes I might have broken.)


Bloops


Pep8 likes to have a docstring fro public methods, including `__init__()`. Do you have a better idea?


A `interpolation_variable.strip` will be more efficient than using a gsub. If spaces are redundant, then a `strip!` prior using the string in `case interpolation_variable` might be even better (it's the result of a match anyway so mutating the string shouldn't matter).


typo: you wanted (I think) to write: "we both cannot, and *should not*, allow ..." (emphasis on the text you forgot)


Right, `activate` is a binary in `bin`. We could also say `source $DESTINATION_DIR/bin/activate`; which is clearer?


Se fue por error $("#error-msg").hide(); esto se debe eliminar


Would it be better to simply have some text instead of a link but be able to turn on sanitization, or keep it as a clickable link?


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2069 of 2069 tests passed, 3 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

![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)


With this we are ready t

IT'S GONNA BE YUUUUUUUUGEEEEE


Ok, will do. I used version displayed on http://teavm.org/ .


Thats fine


This version doesn't exist...
https://github.com/variadico/noti/releases


Проте у цих всіх обхідних шляхах не вистачало чогось, що було потрібно розробникам.


Eh if we get to it


custom.pro or custom.pri is already ignored by custom*.*
But already committed files like librecad/src/custom.pro are not ignored as they are a part of the repo. You have to remove it from the repo to get it ignored.

Armin


Great catch! I refactored `argNames()`, `argName()`, and `qCall()`. They're so much flatter and more readable now. I must've been high when I wrote this the first time.


What? This is just keycodes, it has nothing to do with mouse clicks.


event.getPlayer().hidePlayer(Bukkit.getPlayer(id));


It doesn't look like we're using the `format.json`, so let's get of the `respond_to` docs please.

Also let's turf the comments please, since we both know the rails routing conventions and how to read `routes.rb`.


Btw: in `cancel()`, if job is in state CREATED, the cancel signal is sent to all nodes. Why? Sound as if this is actually not required (at least if my argument from above holds). Can you explain?


Hi Uronito, thanks again.
Can you please test.
Please don't forget to add:
 "model_year" : "2015"
to your config


That alternative layout code in ALSA sink just allows 1:1 mapping in some appropriate additional situations, it doesn't reverse anything. I.e. without the code more source channels would be mixed between two output channels, but all audio would still come from correct speakers relatively to each other (e.g. on a 7.1 system without the code side inputs would be routed to rear and rear inputs to between rear speakers, i.e. rearer-than-rear, instead of the proper 1:1 mapping).

So if you are seeing swapped channels, it is something else.

If OSX requires a specific channel order, make sure you alter format.m_channelLayout to have that order before returning from sink Initialize().


Any chance of an apk in the releases section here on GitHub? WOuld like to test, but realistically, I don't have time learning to compile Android apps...


Wow, I forgot that. We have compatibility wrappers in JtR :sunglasses:

New version pushed.


@nirarmon this will not solve the issue, since filename is still not in use.
maybe try to use %filename% instead of %file% in File instantiation.
10x


Ahhh... Danke!


then what are they doing?


The stub type is probably unnecessary for x86 and x86-64, since there's probably not going to be any variation within an architecture that the vendor library cares about.

It might be useful for something like ARM, though, where you can have two different instruction sets within a single process. If we ever have reason to add entrypoint stubs using regular ARM instead of Thumb, then the vendor library would need some way to distinguish them at runtime.


You're right, I've been hoping this wouldn't turn into an error at some point but it has. Think we have to import Base.error and overload all the different versions of it in Lumberjack. Filed an issue here: https://github.com/WestleyArgentum/Lumberjack.jl/issues/32


I agree. I like the module-like reference **portal-service**.

I'll make that update and make a similar change for portal-impl. Thanks!


This would extend the generic API, I ment this as example to extend the driver specific API:
https://github.com/db-migrate/mysql/blob/master/index.js#L401-L413

You actually have access to the extension from within the driver.


`evidential` is the attribute on `sc_category` that can be used to define whether ScItems of a certain ScCategory can have Level of Evidence.  This is because for many content items, like Community Services, it makes no sense to show Level of Evidence for.


The "stop/warn" is not mentioned in this spec, only in the original 241 (when it explains about `NET_THRESHOLD`).


This grabs 1-4 keyEvents and assigns according column values to the input fields.


Good!


It also puts the last keyEvent column into the remove field.


Use `unless` instead of `if !` when there's no `else` or `elsif`


Been playing with loading in complete list of samples by just specifying the folder and then playing them at random by varying the index. (This appears to work like a ring so works well with tick). One nice addition would be the ability to return the number of samples in a specified samples folder.
I’ve converted several of my previous programs that used samplepack before and it’s not too bad to do. I think the benefits of the new functions outweigh the agro :-)
> On 4 Feb 2016, at 17:31, Sam Aaron <notifications@github.com> wrote:
>
> Please do have a play and let me know if it's missing anything you'd like (or if you have any issues with it).
>
> It definitely needs a fair bit of road testing before I'm confident enough to release v2.10
>
> —
> Reply to this email directly or view it on GitHub.
>


This is kinda confusing.  I would suggest `parent.present? ? parent.evidential : evidential`, with `evidential` having a default value of `false`


This is just scaffold cruft, but I'm not sure I favour getting rid of the format.json.   If we ever need to build an API it will be really handle to have our controllers designed like this from the get go.  No reason to turf functionality that works if we might need it later and it's not causing problems.


test 1123


とほほのperl入門
http://www.tohoho-web.com/wwwperl1.htm


but we are using clicks to play this game?


El 4/02/16 a les 18:44, EtchedPixels ha escrit:
> On Wed, 03 Feb 2016 13:30:25 -0800
> gustauperez <notifications@github.com> wrote:
>
> > I built a hard disk image (I guess) using the build-filesystem
> script. Then I did the same createdrives does to put the kernel and
> the bootblock on the drivea.cpm.
> >
> > I moved those images to the cpmsim/disk directory and then I tried
> to run cpmsim. I see the bootblock is fine because it loads the kernel
> and asks me for the boot device. Here is where I can't proceed any
> further.
> >
> > So if I tell it to use 0, it complains because the drivea.cpm disk
> is empty (no init there). If I give it 4 (I think) it tells me there's
> a valid filesystem, but there's no init there. I checked drivei.cpm
> with ucp and I can see the init binary in the root.
> >
> > Am I doing something wrong or perhaps I am missing something?
>
> 4 ought to fail that sounds like a bug. Does 256 work

   256 does work, thank you. I can start playing a bit with it and start
looking at the code. Btw, why that value for the bootdev?


Just to be clear: if we instead had written
```rust
impl<'a> Foo for &'a str { fn foo(&self) { println!("Some string slice: {}", self); } }
```
(instead of the `&'static str` impl), then all would be fine and the lint would not fire, right?

(That observation is probably obvious in this context, but I point it out because I wonder whether in the design of the lint we can advise programmers to potentially consider "just generalize your `impl` to be more parametric with respect to the lifetimes involved...")


We came to the design choice of having both functionalities.


**має бути**


>
> > I built a hard disk image (I guess) using the build-filesystem
> script. Then I did the same createdrives does to put the kernel and
> the bootblock on the drivea.cpm.
> >
> > I moved those images to the cpmsim/disk directory and then I tried
> to run cpmsim. I see the bootblock is fine because it loads the kernel
> and asks me for the boot device. Here is where I can't proceed any
> further.
> >
> > So if I tell it to use 0, it complains because the drivea.cpm disk
> is empty (no init there). If I give it 4 (I think) it tells me there's
> a valid filesystem, but there's no init there. I checked drivei.cpm
> with ucp and I can see the init binary in the root.
> >
> > Am I doing something wrong or perhaps I am missing something?
>
> 4 ought to fail that sounds like a bug. Does 256 work

   256 does work, thank you. I can start playing a bit with it and start
looking at the code. Btw, why that value for the bootdev?


So someone can click or use the keys.


Not really. Maybe describe a little more what a daily build is and how it maps to mozilla-central? Otherwise, this is not that important.
I think pep8 is too pedantic here.


@jbgi, if we named this uncons here and added the Haskell uncons function then how will this cause an overloading problem?


haha


this warning is bound to cause confusion.  I'd say take care of it programmatically: when a level of evidence is destroyed, set all of its sc_items' `evidence_id` to `nil`


I see


right?


Well, after some more wikipedia-ing it turned out that my whole calculation of the RMS was wrong, which explains some of my confusion. So ignore everything related to RMS above, it's all disastrously wrong.


This commit is needed.  If I take it out, there is a test failure:

```
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 41.307 sec <<< FAILURE! - in com.cloudbees.plugins.flow.AbortTest
testThatAbortAbortsStartedJobs_IgnoredParallel(com.cloudbees.plugins.flow.AbortTest)  Time elapsed: 10.326 sec  <<< FAILURE!
java.lang.AssertionError: unexpected build status; build log was:
------
Started by upstream project "ha:AAAAux+LCAAAAAAAAP9djTEKwlAQRNeIHsBSbLxAfiM2kkJsVFSE2MuPf0nU5W/YvxG9htfQxkN4HO+gkMpMMTPFY+b5gU4QGBaVC+zjA/vAhPGaHdL8VqLQ0Z83rAi1WhHAVWDQ4P/QcfJ4JffyHUG0gC6hz7VYQbsSUuitTvZiDVmfm1Tl6PPJb67fmJvV2
originally caused by:
 Legacy code started this job.  No cause information is available
Building in workspace /tmp/hudson5900056223045920038test/workspace/job1

------

Expected: is <ABORTED>
     but: was null
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.junit.Assert.assertThat(Assert.java:956)
        at org.jvnet.hudson.test.JenkinsRule.assertBuildStatus(JenkinsRule.java:1078)
        at org.jvnet.hudson.test.JenkinsRule$assertBuildStatus.call(Unknown Source)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:42)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:108)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:120)
        at com.cloudbees.plugins.flow.AbortTest.testThatAbortAbortsStartedJobs_IgnoredParallel(AbortTest.groovy:156)
```


We discussed about it and since we already have shell-popup on `SPC '` we want to make `SPC m '` an equivalent for a REPL popup. We can discuss about what it the best new binding for `ruby-toggle-string-quotes`, is `SPC m "` a good candidate ?


@bgracie  Yeah its more a placeholder to scare anyone from deleting one because realistically they shouldn't be deleting levels of evidence.


I created an issue for further discussion and tracking of this: https://github.com/hoburg/gpkit/issues/510


@suhailshergill we could, but I didn't see need for it so far. Unit tests cover functions level only, it means it's not using akka at all. Thus there is no need to tweak the logging settings for unit tests, at least now.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14440](https://teamcity.mesosphere.io/viewLog.html?buildId=124821&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 128 Build time: 00:01:44


Actually, _HOST is wrong here, right? The metastore runs on a single machine.


That could work too, the idea was to just allow a parent to automatically set rules for children, unless the child wants to overrule the parent.


return new ModuleCollection<>(new TitleRespawn(2, false, false, false, false, null));

it will remove the warning (just add the "<>")


Can we have this called 'removeNull', to fit with the other tests?


```
options = options || {};
extend(command, options);
```

I'll accept the PR but fix this after that.


s/may/must/


This line actually fixes everything... ^^


Oops - my mistake.  Fixed


What's this import for?


...**_літерали_** можуть ~~виглядати~~ **_здаватися_**


The following doc(s) have been deployed to the [wpsolr](http://wpsolr-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/drupal/solr)
- [docs/articles/sites/apache-solr](http://wpsolr-static-docs.pantheon.io/docs/articles/sites/apache-solr)
- [docs/articles/wordpress/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/wordpress/solr)


Are weapons being added? lel ( ͡° ͜ʖ ͡°)


@ljacqu Done ;)


hmm, so:

```
if evidential.nil?
  if parent.present?
    parent.evidential
  else
    false
   end
else
   evidential
end


In almost all of our other autobeans we import the PropertyName so the annotation results in `@PropertyName`.


Yeah, I agree. Will update


hm, I thought it might be, but on the other hand, that can be handled in the serializer if we need it anyways. i'll take it out.


So my understanding of this change is that in the `attached` case there is no `err` and also no `connectionManager.getStateError()` so the callback is called with no error? Does that make sense?


Based off of my install (Windows 10)  I'd say you should really emphasize removing all versions of python and not ignoring the python xy installation warning (the python uninstaller I had was bad and left some remnants which caused problems).

I also do not think that pip install gpkit works. When I tried using pip to install gpkit for some reason couldn't find cvxopt even though it was installed. I had Prof Hoburg trouble shoot my installation and he ran setup.py (I think that's the file name) at which point the install seemed to work. Hopefully that helps.


Yeah I think `default: false` (at least while data entry is ocurring), would be more straightforward.


`$pass` doesn't work here, that's a variable. How do you want to handle this?


It's a distraction/ dead code.  It makes it harder to determine what the intent is (a new dev might inappropriately assume that this controller is in fact responding to json requests).


One quick thing, I'm getting a build error on iOS:
```
/Users/me/Projects/testing-app/client/platforms/ios/testing/Plugins/com.raccoonfink.cordova.plugins.http/CordovaHttpPlugin.h:4:9: fatal error: 'Cordova/CDVJSON.h' file not found
#import <Cordova/CDVJSON.h>
        ^
1 error generated.
```

I comment it out and it works fine.


I'm currently compiling an actual 0.13.1 build, might make it rc2


yay!


isn't this possible to do via webpack config? , this is not correct to do well with in our build.

```
 output: {
    path: path.resolve(__dirname, 'production'),
    filename: 'index_bundle.js'
  },
```


@AzamatB

Was the removal of Base.show on purpose or accidental.

If on purpose whats the reasoning.
If accidental I'll put it back in.


ком**и**


Okay, I implemented that change in 9bcc160

The calling interface changed a little, but I updated the README to reflect this


There's no reason to believe that this will eventually be an api endpoint, and even if it did turn out to be, we might not use the `respond_to` style.  (Could make a separate ApiController)


@SprocketNYC my build fails locally with this snippet - do you have any local variable for this that isn't pushed? (I keep removing this line locally for it to build fine)


lgtm


I read this a few times and thought that it was saying that there was some special relationship between the fact that this is a *blanket* impl and the use of the `default` keyword.

(As in, I was initially inferring a nonsensical interpretation of the text that went something like "is he saying that if its not such a broad blanket impl, I would be able to leave off some items *without* providing the `default` keyword")

Maybe it could just use an extra sentence saying something like "Every `impl` that is not declared as a `default impl` must include a definition for every item in the trait" ?


Кожен наступний аргумент - **_(тире) _**це відтворене значення кожного заміщення.


@ZephireNZ Suggestions?


So: "Please do not delete levels of evidence without consulting IT or a Medical Lead"


For this one lets use - https://www.npmjs.com/package/html-webpack-plugin


Seems that str converts a string to ascii

On Thu, Feb 4, 2016 at 7:27 PM, Youness Alaoui <notifications@github.com>
wrote:

> Why complicate this? wouldn't str(v) be a noop if v is already a string?
> or would str(v) cause an error if v is already a unicode string ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15898402>
> .
>


Which cordova version do you use?
I have 5.4.1.


:smile:


Missing `libmysqlclient-dev`!


alsa-plugins was initially added December 18th, 2015 as part of pulseaudio development. It is now not needed. Did you really use it for this month that it's been included?


Refers to #378


The following doc(s) have been deployed to the [wpsolr](http://wpsolr-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/drupal/solr)
- [docs/articles/sites/apache-solr](http://wpsolr-static-docs.pantheon.io/docs/articles/sites/apache-solr)
- [docs/articles/wordpress/solr](http://wpsolr-static-docs.pantheon.io/docs/articles/wordpress/solr)


Mo and I are getting 810 for this number, can you double-check?

Checksum for validation:

```
$ md5sum cracklib-small
45521452a27ccca0cd547537e3847b7f  cracklib-small
```


you've forgot __builtin_clzll bulilt'in


...можуть бути зв’язані,**_(кома)_** щоб утворити...


we can: https://medium.com/@okonetchnikov/long-term-caching-of-static-assets-with-webpack-1ecb139adb95#.a68up74df

this is how i implemented initially, but it's too complicated and unnecessary.
For all the effort the webpack plugin just adds a hash value to the bundle.js
We will still have to replace the bundle name in index.html externally using build.js

so it is simple to add a unique string (commit id) by ourselves.


Sorry dude, it was accidental


here it should be clz64 and not ctz64

Doest the tests even work ?


thank you! this was bugging me throughout testing.


I don't think `echo -e` is POSIX, consider `printf`?


I dont think so. I mean yes if we were uploading the whole collection at like `PUT /collection/` we should. Which would be the proper RESTful way. But just as we are not doing that to prevent uploading the whole collection, we should also not download the whole thing.


Hmm, we've seen that before on macs. @mayork, before @whoburg debugged it, had you tried running the commands in the third bullet of the [third instruction](http://gpkit.readthedocs.org/en/latest/installation.html#id8)? I expect that would have solved the cvxopt issue. If not, any suggestions on how to make it more useful / prominent?


Great use of the alt attribute! :cookie:


global variables `fragment`, `el`, `model`?


add new line after please


ACK

But next time try to have a shorter message plz


I think that better:
`NPC_GUARD = me->GetEntry() == NPC_HIGH_COMMANDER_HALFORD_WYRMBANE ? ... : ...`


fixed in 712c38a981b001be7ea2157591f4120db12d55f4


`assert_difference` takes a second argument that specifies how much it should change by (as an integer).  It would make sense to add that.


It would be best to split this commit into smaller parts and clearly describe what you are adding or changing in each commit. There is too much going on here.


Lexicon? Is the update getting close?


Clarified in 712c38a981b001be7ea2157591f4120db12d55f4


Added in 712c38a981b001be7ea2157591f4120db12d55f4


мають **_бути_**


> We will still have to replace the bundle name in index.html externally using build.js
>
> so it is simple to add a unique string (commit id) by ourselves.

Not necessarily - https://www.npmjs.com/package/html-webpack-plugin see this can help.


I actually just updated to 6 like an hour ago. haha that might be causing it


Nice catch. Just added quotes and a comment.


Added explanation in 712c38a981b001be7ea2157591f4120db12d55f4. Would appreciate thoughts on using it in hive.metastore.principal.


no need in a separate `if` branch. just put a bit upper next to the `string === 'undefined'` (` || string === ''`)


Very well done!  You have a great grasp of how to structure content with HTML, and your formatting & esp attention to indentation is impeccable!


Fixed in 712c38a981b001be7ea2157591f4120db12d55f4


Fixed 712c38a981b001be7ea2157591f4120db12d55f4


utACK


Yup, looks like the changes that I made as well.  Should probably refactor to replace httpClient with http.Client.  You want that or should I?


Fixed in  712c38a981b001be7ea2157591f4120db12d55f4


Shellcheck tells me that you don't need the sigil in `$RANDOM` inside whatever that math expression is, you can write this:

```
$((RANDOM % 5))
```

Not sure if it's worth changing, though, as it's one more thing that students would have to understand without much added learning value.


LinkedList implementation with Reverse method.


不要用 archive 啦，用 tag 喵。


Makes button flash green while recording is being started


by the way. Looks like you don't need this fragment


no need in model?


The `difference` parameter for `assert_difference` defaults to 1 if you don't set it.
http://apidock.com/rails/ActiveSupport/Testing/Assertions/assert_difference


Good job with the CSS!


Confirm, 810. Thanks.


@ClxS Well it looks like the engine is not linking. Same error that I had with your previous version regarding one of the bff generated:

C:\Dev\UnrealEngine\Engine\Intermediate\Build\fbuild.bff(181,1): FASTBuild Error #1005 - DLL() - Unsupported node type in 'Libraries'. (Node: 'C:\Dev\UnrealEngine\Engine\Intermediate\Build\Win64\UE4Editor\Development\UE4Editor-ReliabilityHandlerComponent.lib', Type: 'Exe')
2>  DLL('DLL-21')
2>  ^
2>  \--here

I guess it related to this lib description:

DLL('DLL-17')
{
	.Linker		            = '$VSBasePath$/bin/amd64/lib.exe'
 	.LinkerOutput		  =  'C:\Dev\UnrealEngine\Engine\Intermediate\Build\Win64\UE4Editor\Development\UE4Editor-ReliabilityHandlerComponent.lib'
	.Libraries		   = 'C:\Dev\UnrealEngine\Engine\Intermediate\Build\Win64\UE4Editor\Development\UE4Editor-ReliabilityHandlerComponent.lib.response'
 	.LinkerOptions		 = ' /NOLOGO /errorReport:prompt /MACHINE:x64 /SUBSYSTEM:WINDOWS /DEF /NAME:"UE4Editor-ReliabilityHandlerComponent.dll" /NODEFAULTLIB:"LIBCMT" /NODEFAULTLIB:"LIBCPMT" /NODEFAULTLIB:"LIBCMTD" /NODEFAULTLIB:"LIBCPMTD" /NODEFAULTLIB:"MSVCRTD" /NODEFAULTLIB:"MSVCPRTD" /NODEFAULTLIB:"LIBC" /NODEFAULTLIB:"LIBCP" /NODEFAULTLIB:"LIBCD" /NODEFAULTLIB:"LIBCPD" @"%1" /OUT:"%2"'
}


Hey Naomi!  Great job on this assignment; you're demonstrating a really strong sense of how to organize and structure your content with code.  Keep it up!


By the way, I still think a lot of people would be happy to see this "load all project namespaces" functionality as a standalone command.


Whoops, didn't use git mv. My bad


So yeah sorry about just pushing this up. I was accidentally working on the master branch on my computer and not my branch. I'm not sure why it just pushed it up instead of using a pull request.


On Thu, 04 Feb 2016 13:59:50 -0800
gustauperez <notifications@github.com> wrote:

> >
> > > I built a hard disk image (I guess) using the build-filesystem
> > script. Then I did the same createdrives does to put the kernel and
> > the bootblock on the drivea.cpm.
> > >
> > > I moved those images to the cpmsim/disk directory and then I tried
> > to run cpmsim. I see the bootblock is fine because it loads the kernel
> > and asks me for the boot device. Here is where I can't proceed any
> > further.
> > >
> > > So if I tell it to use 0, it complains because the drivea.cpm disk
> > is empty (no init there). If I give it 4 (I think) it tells me there's
> > a valid filesystem, but there's no init there. I checked drivei.cpm
> > with ucp and I can see the init binary in the root.
> > >
> > > Am I doing something wrong or perhaps I am missing something?
> >
> > 4 ought to fail that sounds like a bug. Does 256 work
>
>    256 does work, thank you. I can start playing a bit with it and start
> looking at the code. Btw, why that value for the bootdev?

Major 1 minor 0 ( 1 * 256 + 0)


напевно там **слешу**

**за необхідності**


yes I ran those commands with no effect, it still couldn't find cvxopt. That's why I think there was a problem with using pip to install. Once the other setup.py was run all the tests immediately passed


When updating to 3.4.9 I assumed the recipe was OK for 3.4.6. I should not have replaced 3.4.6 by 3.4.9 before making sure everything was OK. I am therefore adding back 3.4.6 with https://github.com/haikuports/haikuports/pull/465. Sorry for the noise.


It isn't, and I have had issues where it doesn't work the same on mac.

At the same, I'm trying to minimize the number of new things we show people. We've used echo a lot, printf would be yet another new thing.

50/50 on this


Thank you, thank you!

On this topic, do you have ideas on how to fix the test for this thing? As you may have gathered, I've been swamped for the past few weeks, but I still really want to fix that test and then backport this to 15.09 (assuming I get unswamped before the next NixOS release)

My big concern is that I don't think we have other tests that do `nix-build` (or `nixos-rebuild`) operations inside the test, so they don't have to deal with a mutable store and that sort of crap.


@zackbloom


Yeah, however, "echo RANDOM" doesn't work and "echo $RANDOM" does. I think it introduces unnecessary confusion to drop it.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.contentlisting/4//violations
```
plone/app/contentlisting/catalog.py:42:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:44:14: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:84:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/catalog.py:185:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/contentlisting.py:130:12: P002 found "hasattr", consider replacing it
plone/app/contentlisting/realobject.py:36:12: P002 found "hasattr", consider replacing it
```
You can run that yourself using buildout.coredev ``experimental/qa.cfg`` config file.


Agreed. From time to time someone asks about this.


- первинн**и**й

- **_значення_** з маленької


Message::setMessage is missing a docblock.


Message::addField is missing a docblock.


Poke @Grz3s
Please add this to DB:

```sql
DELETE FROM creature_template_addon WHERE (`entry`=17312);
INSERT INTO creature_template_addon (entry,mount,bytes1,b2_0_sheath,b2_1_pvp_state,emote,moveflags,auras) VALUES
(17312, 0, 0, 1, 16, 0, 0, '');
```

Also, please add the SQL part of this: https://github.com/scriptdev2/scriptdev2/issues/169#issuecomment-153153023 as I'm going to correct this quest soon.


Did you mean to commit this file?

With a typo no less? :D


Yeah, that does sound like a pip problem. :-/


Can you please replace this with some simpler logic? Maybe something that just checks the first integer to be greater.


:+1: :100:


We should probably discuss this in a proper issue. For instance, even if I exclude the Shift bindings, `C-c , M-,` feels like finger-acrobatics to me.
Also, since we're changing things up, we might as well add more intuitive keys (we can keep the old ones too). Something like `C-c , a` for running all tests and `C-c , c` for running the current test.


Nope! Whoops.


as there should only be one title respawn module, i think this should just:
return new ModuleCollection(new TitleRespawn(delay, auto, blackout, spectate, bed, message));


neat, thanks!


Yes


Can you please put new verification logic here? Something like `isValidVersion` that tests for the `/\d.x/` regex should probably be enough.


oh, how about unicode(v) ? wouldn't that work even better ?

On Thu, Feb 4, 2016 at 5:12 PM Leonardo Stern <notifications@github.com>
wrote:

> Seems that str converts a string to ascii
>
> On Thu, Feb 4, 2016 at 7:27 PM, Youness Alaoui <notifications@github.com>
> wrote:
>
> > Why complicate this? wouldn't str(v) be a noop if v is already a string?
> > or would str(v) cause an error if v is already a unicode string ?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15898402
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15899448>
> .
>


Where is `british-english`?

```
waleed@waleed-VirtualBox:/etc/dictionaries-common$ locate american-english
/usr/share/dict/american-english
/usr/share/man/man5/american-english.5.gz
waleed@waleed-VirtualBox:/etc/dictionaries-common$ locate british-english
waleed@waleed-VirtualBox:/etc/dictionaries-common$
```


After reverting this worked fine https://github.com/temasek/android_art/commit/72cdd24dcc106f52b21372a972cdb72042f4632a

Sorry delay on  replying but i was on exams...Super tired.


you have 2 functions, isDeadUUID(uuid) and isDeadByUUID(uuid), as there should only be 1 module, the loop is not needed, just keep isDeadUUID and remove isDeadByUUID


No argument from me. The current keybindings were simply inherited from the legacy `clojure-test-mode`. I never liked them and I often thought about altering them.


Це означає, (**кома**) що будб-які..

...ста**є** у нагоді,..


tipo error


Please make this a 302 redirect.


@vmx :+1: Thanks!


This fixes #5


Hey Brit!

I would highly suggest looking at [http://learnlayout.com/float-layout.html](url) for float layout example. And this website is also excellent for CSS fundamentals as well :)

You can try to do **just the first layout challenge** and get back to me. We can see if anything changes because currently the first layout didn't render correctly for me and some divs were overlapping with one another.

Let's start small.

-Stephen


_д**и**влячись_
 думаю так краще звучатиме))


Yes, but the problem is two fold:

1.) We are not interested in every framework specific package, or not all packages my be listed. For example python-azure-sdk is a dependency and not explicitly listed in the config.kiwi in SP1, also I just added azurectl to the image builds.

2.) I think we would still depend on a manual step, i.e. someone has to remember to run a tool

That said I think if anyone has a great idea about how to do this in a nice way I am in favor of it.


Reverted back to 5.4.1 didn't fix it. I see in this repo you have it commented out. Where is
```
com.raccoonfink.cordova.plugins.http
```
coming from?


Yea, but I'm trying to use Android's mediaPlayer class, which requires a FileDescriptor. To get that from an InputStream requires that I first copy InputStream into a File, then create a FileInputStream from the File, then get the FD from that FileInputStream. :(


your implementation was about on par; the implementation would be quite similar to this provided the sharedDefaults was lazily instantiated when nil and used a lock
```
_sharedDefaultsLock.lock()
_sharedDefaults?.synchronize()
CFPreferencesFlushCaches()
_ sharedDefaults = nil
_sharedDefaultsLock.unlock()
```


> Ви все ще можете використовувати `\n` та інші екрановані символи.

зробіть це новим реченням


good points. we need that name in the JS so it'd be hard to do `@property` for that without changing how jsonobject serializes it. i think just using the path though is promising. one kind of sticky wicket is that a repeat is some arbitrary path. we can infer from that fact that it starts with `data` that it's a repeat group which should be correct. i'll try and do that


Maybe using HyperText Markup Language ones maybe? But it will be ugly af


i just realized one is static and the other one isn't, keep it like that


You might have noticed that your Typekit script throws an error if you look in the console / your fonts might not be rendering quite the way you expect.  That's because the Typekit service only renders the fonts you want when you've added them to your kit, and your kit lists which domains to serve up those fonts.  That's not to say that you can't use this for local development, but there's a bit of a trick to setting things up for doing that.  Let's chat about this when you stop by for office hours.


I think there might be a better way to implement this. changing it to rand = 1 or rand = 2, you can just change the actual id's of the user's being added to 1,2,3.


Closes issue #20.


LGTM but can you add a quick summary of the changes to the commit log?  Rule of thumb: one should not have to look at the diff to know what changed.


In future can we move these into the plotIt block. I have had trouble in a number of the testing environments because of matplotlib.


Its a fork of https://github.com/RangerRick/cordova-HTTP.

https://github.com/RangerRick/cordova-HTTP/commit/6d4144fe4ff84c5fe161a4d12c40d7a6959d9686

I have no clue why this guy changed the original id.


Could we add the check_exit_status() here too?


You are right, I haven't read the commit message because we tend not to write them. However, a statement like "the work in progress" or "the servers is not finished" does not indicate that no one should touch the code. The best way would be working in a branch (better) or clearly indicate in commit message/a head comment that the code should not be touched by anyone. Finally, there is no way to tell whether it was you or someone else because we so often commit things under other people's accounts.

Yes, this pisses me off because coding experiments should not be the art...

As for the TIMEOUT constants, yes, constants in fact is a good practice. But, it should be defined at the beginning of the file. If it were defined in the beginning of the file, i wouldn't touch it. Now, when there only one time use of a time constant and no constants are defined, for example, for baud rate and other things i think it is better not to use a constant. How timeout is different from other constants in the files that are used only once?

I thought about changing the key, and totally forgot. The flow rate unit conversion should be checked and maybe something else. There are still lots of things that can be improved.


Oh, yes, macros, what fun.


Tests?


>If you define only simple methods here, not entailing the inclusion of additional headers and not containing any complex logic (for example, simple getters and setters), this should speed up compilation.

I get what you're saying, but that has the potential pitfall that someday down the road, through a series of small modifications, those getters end up not so trivial/simple. And each time you edit them you make a bunch of other recompile because they include that particular header.
And I don't see how defining setters/getters in the header will result in faster compilations. Unless that class/struct is the only one in the header and won't need a .cpp. Then I can see the benefit for a "headers only" class.


Changed title


TeamCity DNN JavaScript Libraries :: CI [Build 141](http://teamcity.engagesoftware.com/viewLog.html?buildId=419&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


Just a minor note on indentation here.  Your "photo-caption" div can be aligned with the img tag above.


Style: can you write it as `proxyElements`?


TeamCity DNN JavaScript Libraries :: CI [Build 141](http://teamcity.engagesoftware.com/viewLog.html?buildId=419&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:22


Fixed the issue


Suggestion: use a template string?  Easier to read.


wtf


`args.GetIsolate()` would have been fine, too.  It's infinitesimally faster than looking up the `Environment` first.  Your call.


I made Stream changes in line with this in 64fdac726d65c9dc1f6bd38a2537f16fa65e9e95


lgtm


Line continuations should use 4 space indent.


@Dokman I am on it


missing whitespace


/usr/share/dict

I just checked on a VM that I've never installed any packages on and it's there. Weird that locate didn't find it for you.


"osd_uuid not found under omap, ..."


¿No creo haga falta no? Es un número elevado para indicar que si una skill esta siendo cargada y su tiempo es superior a X, dicho delay termine. Fue obtenido del mismo sistema que en l2j para las olimpiadas.


I love the attention to detail that you applied to this assignment, especially with all of your styling!  There's nothing wrong with the way you've written the declarations for each part of the font styling, but just wanted to let you know that you also have the option of collapsing these into one line with shortcut properties.  Section 3 in [this markdown](https://github.com/ga-students/FEWD_DTLA_7/blob/master/Week_01_Basics/02_css_basics/complete-css-selector-basics-lesson.md) goes over a few shortcut properties.


This makes boost_test compile, but there is an APPCRASH on:

`operator CONTROL_LIB_NAMESPACE::TempSensorBasic*() { return &sensor(); }`


It's not there for me. Did you install updates while installing the VM? I don't think I did; I'm running an update/upgrade now to see if that changes things.


This sucks


very nice!


I think this has bugs


That works.


Looks good to me.


Dispatcherの26行目
ハッシュレファレンス
http://www15.plala.or.jp/h-kihara-home/prog/ref.html
http://perldoc.jp/docs/perl/5.8.8/perlref.pod


testing comments


`parge_id`... this is part of why I don't think our braintree stuff is well tested.


Yes I did some shell tests and unicode(v) have the desired behaviour. It
seems that lot of people have unicode issues with python so I was unlucky
trying to find this command

On Thu, Feb 4, 2016 at 8:28 PM, Youness Alaoui <notifications@github.com>
wrote:

> oh, how about unicode(v) ? wouldn't that work even better ?
>
> On Thu, Feb 4, 2016 at 5:12 PM Leonardo Stern <notifications@github.com>
> wrote:
>
> > Seems that str converts a string to ascii
> >
> > On Thu, Feb 4, 2016 at 7:27 PM, Youness Alaoui <notifications@github.com
> >
> > wrote:
> >
> > > Why complicate this? wouldn't str(v) be a noop if v is already a
> string?
> > > or would str(v) cause an error if v is already a unicode string ?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15898402
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15899448
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15899813>
> .
>


I don't do the aligned thing, but it doesn't bother me -- so I'm cool with this :-)


taken from your `three_column/style.css`
```css
.left {
	width: 250px;
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
	float: left;
	margin-right: 15px;
	display: inline-block;
	box-sizing: border-box;
}
.middle {
	width: 710px;
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
	float: center;
	display: inline-block;
	box-sizing: border-box;
}
.right {
	width: 250px;
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
	float: right;
	margin-left: 15px;
	display: inline-block;
	box-sizing: border-box;
}
```

I would've done something like this.

```css
.left {
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
	box-sizing: border-box;
          /* important part */
        float: left;
        width: 20%;
          /* important part */
}
.middle {
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
	box-sizing: border-box;
        /* important part */
        width: 48%;
        margin: 0 2%;
        /* important part */
}
.right {
	height: 300px;
	border-width: 1px;
	border-style: solid;
	border-color: black;
	background-color: gray;
         /* important part */
        float: right;
        width: 20%;
        /* important part */
}
```

few notes,

- there's no `float: center` ;)
- if you use `float` property, no need to change the `display` property to inline-block, it should already act as if.
- try to switch out your styles with my styles in your `three_column/style.css` file and see how it affects the divs and how they move

Stephen


I tried to replace by $blast.element_identifier as you suggested.
However the variable still return the name of the dataset in the python code output.
see the example in dataset 121 of https://mississippi.snv.jussieu.fr/u/drosofff/h/metavisitor-plos-ntd---with-hg38-subtraction---srr1-10-test-3


status, fan, sata, usb, beep and power don't seem to do anything on the WD01


is this for new tabs opened by cmd+T, clicking a link, or both/other?


I got 2 messages from @thefiddler  on opentk.com just recently, Jan 14, 2016


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14441](https://teamcity.mesosphere.io/viewLog.html?buildId=124872&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 128 (new) Build time: 00:00:19


test comment


I'm considering deleting the ObjC implementation as well, to be honest. CFPreferencesFlushCaches() doesn't exist in the CFPreferences in corelibs, and doesn't do what anyone actually wants in the one in ObjC CF.


This version works correctly.


Kristina, amazing job with this first assignment!  I really appreciated that you went beyond the requirements and really got into the styling parts of this.  Your code is really well-formatted, clean for reading, and well-maintained.  Your use of comments, especially in the CSS, is exceptional and will continue to help you keep your codebase organized as your projects become more complex.  Great job overall and keep it up!


code lgtm but "Switch to new tabs immediately" could be more clear about what that means


Just out of curiosity, why the new class?


oooh... yeah, I did. I told people to check that box week 1, but it's a certainty that some people didn't.

I added a note to install the wbritish package if it's missing


Oh, yes, we talked about it but i have forgotten it again :\


What do you have in mind, something like e3eee06 or 16bf0ea?  I still don't have intuition for what's idiomatic Go.


как вариант, можно использовать гем simple_form, интегрирован с i18n, показывает ошибки


Right I'm just saying that instead of writing code to loop over results and print the content out in json format just pass results to json.dumps() and it will do that for you.  You can tell json.dumps() to sort and indent if you want the output to be a little more human readable.


Actually without this line, preservation file would still be present after onDestroy(). That means (due to the fact that onDestroy() is called only when service is really and intentionally not needed anymore (when system kills the service or user removes the tasks onDestroy() is not called)) that, when monitoring will be started again, even after the complete device reboot the monitoring status would be restored (the restoreMonitoringStatus() method) and can produce false events. For example produce "exit event" after monitoring restart in case of no beacons are visible and monitoring has been stopped when some beacon were visible.
So I think this line is needed as it cleares the state of monitoring when it is supposed to not be preserved, do you agree? :)


Simple chequeo de skills, en olimpiadas se usa por el tema de eliminar las skills de clan. No es encesario, epro nunca viene de más creo yo :P


cool! :) I've never really used/cared for unicode (with or without python),
so not really my domain.

On Thu, Feb 4, 2016 at 5:48 PM Leonardo Stern <notifications@github.com>
wrote:

> Yes I did some shell tests and unicode(v) have the desired behaviour. It
> seems that lot of people have unicode issues with python so I was unlucky
> trying to find this command
>
> On Thu, Feb 4, 2016 at 8:28 PM, Youness Alaoui <notifications@github.com>
> wrote:
>
> > oh, how about unicode(v) ? wouldn't that work even better ?
> >
> > On Thu, Feb 4, 2016 at 5:12 PM Leonardo Stern <notifications@github.com>
> > wrote:
> >
> > > Seems that str converts a string to ascii
> > >
> > > On Thu, Feb 4, 2016 at 7:27 PM, Youness Alaoui <
> notifications@github.com
> > >
> > > wrote:
> > >
> > > > Why complicate this? wouldn't str(v) be a noop if v is already a
> > string?
> > > > or would str(v) cause an error if v is already a unicode string ?
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15898402
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15899448
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15899813
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lstern/SWParser/commit/26fe6bb2b211a25083d98e66a3be1402b493c04c#commitcomment-15900217>
> .
>


You are right, the fix wasn't there...vim to the rescue and it's working.


This line is very wrong. On first synced from SyncMachine it invokes setState with the instance of the collection as a first param. Can you imagine the consequences? At least it should be

@once 'synced', -> @setState {} if @comparator

Or maybe we should look forward of removing this.


oh heavens


second test


Just guessing, but probably because it was redundant as filtering on
properly formatted
names was added to the wifi pairing activity in the SDK.

Craig

On Thu, Feb 4, 2016 at 1:49 PM, Nicholas Rizzio <notifications@github.com>
wrote:

> @rgatkinson <https://github.com/rgatkinson> Is there any reason why the
> phone name check was removed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SwerveRobotics/ftc_app/commit/1bb8baab6bdedb2165b47c46ed7943c9bcf5c68a#commitcomment-15898915>
> .
>


the function makeCacheMatrix will define x as a matrix data type which is required as a input for the function.
 the function further checks whether the value of x was already present in the cache? if yes, it will assign the value through nested function "get" defined else in case of no value in cache, it will value from matrix "y" defined by nested function "set".


This line prevents from fetching collection on third time. Because if comparator is a string, then you reset it into the function in _setComparatorAndSort(), so now comparator is a function, so the condition stops collection from fetching data on server completely.


a


windows -> Windows


cli -> CLI


Does it actually have to be <100 characters? My solution for the first one was 129 characters.


campatibility -> compatibility


Minor indentation note: your <title> and <link> can both be indented one in from <head> to show that they are nested within the <head>


2016-02-04 on 2016-02-04 :+1::sheep::fire:


shouldn't most/all of these be checked by default?


This may be missing a couple of contexts.  When the share token is explicitly not set we expect a value to be generated at random, so that's something we can test.  When the share token is explicitly set, we can be sure that what we set it to is returned.


@changleicn sure, I created https://issues.apache.org/jira/browse/HAWQ-400 to improve our regression test framework, to support expectations on exit codes.

The reason why error's level is a FATAL - as for now all errors, which are used in postinit.c are either WARNING or FATAL, for "hcatalog" database it's not WARNING, that's why it's FATAL.


I understand and agree completely!


`module's` -> `modules`


whoops misread, but it's still missing article (sic): "in _the_ installed module's"


test


I got this in between each time is restarts

>[Console] Restarting server...

C:\Users\Downloads\Ogar-master2\Ogar-master2\src>if errorlevel 1 (if not errorlevel 3 (GOTO END; ) )

C:\Users\Downloads\Ogar-master2\Ogar-master2\src>goto loop


Don't know if it's exactly an error


generating a lot of garbage here


Great use of alt!  Your usage of border here is totally fine, but just a heads up that the more recent HTML5 standards recommend using CSS instead of the border attribute to assign borders.  Read more [here](http://www.w3schools.com/tags/att_img_border.asp).  (We'll be talking more next week about when to style something with an HTML attribute or with CSS next week.)


Actually, what the function does is to convert a camel-case string, like "produtFamilyId" into title/label appropriate string, "Product Family Id".


not handling short reads


Missing the main use case.


assuming that the data being read is utf8 (when it’s probably just binary):


Lol, no, that was a joke (though mine are all < 100). Maybe that should change to "in one line" or something


Please write a statement describing the return value.


/*
Я тут удалил поле id и все с ним связанное,
прикольная штука getClass().getSimpleName() - не знал
 */


@zendesk/vegemite - sorry guys, commited to master by mistake... retro :+1:'s?


@thenickcox (in case you didn't see) one of these last four commits broke the build.


Cool switching up the list style type!  You will want to watch out for inline styles using the style attribute; while it will render correctly on the screen, too many inline styles can lead to code that's hard to read or debug.


 ```go
block := make([]byte, filechunk)
    for {
        n, err := file.Read(block)
        if err == io.EOF {
            break
        } else if err != nil {
            // panic?
        }

        hash.Write(block[:n])
    }
```

[3:57]
note on that last line though: I’m not checking for short writes, but that’s because the crypto hash packages won’t do short writes

[3:57]
usually you’d have to guard against that as well

[3:58]
e.g.:
```go
writeBlock := block[:n]
for len(writeBlock) > 0 {
    m, err := hash.Write(writeBlock)
    if err != nil {
        // handle error
    }
    writeBlock = writeBlock[m:]
}```


if (lcd_id > 1)

Some P5113 use old bootloader, lcd_id = 0 on those devices,


We actually have this in a few other controllers.  I don't mind the scaffold crud very much because it suggests to me I'm dealing with a very straightforward controller.

A good convention might be to drop the comments once a controller gets modified past its original scaffold.  That said, a beginner dev is more likely to benefit from seeing these routing reminders than not.

I personally think controller design should idiomatically move towards always including JSON endpoints in responses anyways as a way to increase flexibility later.  E.g. if we redesign these forms to be more SPA, etc.

From a maintenance perspective the only time that I think it would make sense to remove `respond_to` would be in situations where you are heavily changing the behaviour of redirect points -- at that point there is an argument for removing them.


GBAD-5 Make responsive UI for authorization pages


Should also test that it preserves an already uppercase string and that in cases with multiple words only the first is affected.


I guess we also need to support `@url`, it has the same value in backbone:

```
    // Default URL for the model's representation on the server -- if you're
    // using Backbone's restful methods, override this to change the endpoint
    // that will be called.
    url: function() {
      var base =
        _.result(this, 'urlRoot') ||
        _.result(this.collection, 'url') ||
        urlError();
      if (this.isNew()) return base;
      var id = this.get(this.idAttribute);
      return base.replace(/[^\/]$/, '$&/') + encodeURIComponent(id);
    },

```


Nisa, great job on the first assignment!  You're demonstrating a really strong command of structuring content using HTML and your formatting & indentation is all really well done.  I also really appreciated that you tried a couple of things that weren't explicitly asked of you or talked about class -- keep it up!!


cacheSolve function will check the cache first by getting input from external environment function makeCacheMatrix.
it will assign the inverse value to "s" as defined in makeCacheMatrix by calling "getinverse" function from external environment, else will continue to calculate inverse value for the matrix assigned through "get function"


![screen shot 2016-02-04 at 2 58 34 pm](https://cloud.githubusercontent.com/assets/549654/12832650/224e4e2a-cb50-11e5-887b-f564301440f9.png)

i see an intermittent bug where the color is black or shows up as the previous color for a second. can't repro consistently though so i'm not sure it's related to this commit.


It isnt a error, it is normal


Why not just metadata to print everything? (May need add to string)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 14442](https://teamcity.mesosphere.io/viewLog.html?buildId=124881&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 128 Build time: 00:01:40


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.

![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)


Why is this here?


Yeah, though I might commit the test suite from the scaffold into fspec.  In this case with a CRUD object that's so clean the mockups will probably last a long time.


lgtm


Why isn't this just a regular cross compile target?


Why should composer functions and mapper functions be exported from container module? Under what circumstances would you use them directly without the container?


@robwise Thank you :)


Why do we need this?


hello because the program does not start telling me not to have the main method.
can you give me your email to keep in touch please. thanks


Can I get rid of it? :$


why do you add `start.day` here?  Seems like some debug code to make sure there is something to show...


`AllocCoTaskMem` for COM memory. Please use `AllocHGlobal` and associated `FreeHGlobal` instead.


You're not testing this for success, that's dangerous with `C`, please do not assume exceptions will be thrown if something fails.


This comparison is broken. Both MPICH and Open MPI call their Fortran compiler `mpifort`, which MATCHES `ifort`. This breaks the build when MPI is built using gfortran, for example. In turn, this breaks Elemental. Note that this is a default configuration for many Linuxes. Also note that `mpifort` could well be the Intel compiler, or that might be called `mpiifort`. The executable name is just not reliable. You could run the Fortran executable with `--version` or `-qversion` (IBM XL) if you wish to specialize compiler flags by vendor.


And can I make it stop sending me the listening on server port begining thingy


Before we go and do this, I'd like to understand who we will make this available to and how.  That answers the question about how we build it and distribute it.


awesome! i feel like open in foreground should be default, but that can come later


Hmmm doesn't seem like a good idea just to remove the asserts, they were put there for a reason. They point that something else must be wrong? There should only be one record returned.


Are you doing this twice? Maybe I'm misunderstand something here.


Let me do something!


`String` vs `string`


You're missing a closing } on your img rule -- the reason why your browser let it slide is because it's the last rule; had there been another rule following this missing }, you would have had some bugs with your styling.


No puede haber números mágicos en el código.


Looks good, but in the future, try to branch off on Group 9-2 page and not your own master. That way it's easier to keep track of the branches.


Ok!