TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7877](https://teamcity.mesosphere.io/viewLog.html?buildId=85594&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:05:16


Thanks Furry this did the job thank you again


My bad, this is missing a ```AND NOT BUILD_WIZARD```.


@rinnko 我API都实现了，你可以在服务里调用了


That worked!  Thanks for the quick fix!


@Ryan-He 我API都实现了，你可以在服务里调用了


I think "waste" is too strong word here. I wouldn't say money is wasted on those things. Maybe we could rephrase it like that:
"However, we don’t want you to have an impression that to make a successful Django Girls event you have to have all these things and buy decorations, flowers, make videos and so on. Yes, we believe they make Django Girls different and special, but we also don’t want you to spend loads of money on things that are expensive and not necessary."
What do you think? :)


Fixed in [the polyfill](https://github.com/mathiasbynens/CSS.escape) as well.


Typo here: You want `->dump-ast`


You can make the changes, any additional commits will all be part of the same PR.

> On Dec 3, 2015, at 8:22 AM, Alex Pletzer <notifications@github.com> wrote:
>
> Hi Greg,
>
> Do you want me to make the changes? Or are you working on this?
>
> --Alex
>
> On Thu, Dec 3, 2015 at 5:46 AM, Greg Von Kuster <notifications@github.com>
> wrote:
>
> > The above file should not be re-added to the code base. Instead, a symlink
> > should be created from this tool's directory to the single file in the
> > directory above (i.e., ln -s ../icqsol_macros.xml .).
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760856>
> > .
> >
>
>
>
> --
> Alexander Pletzer
> +1 609 799 7999 cell
> alexander@gokliya.net
> —
> Reply to this email directly or view it on GitHub <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14761530>.
>


Très bien. J'ai fait 2-3 commits pour améliorer des trucs. Je suis dans le métro, là, je push quand j'arrive à l'Université.


WHEN("Turning autosync on")


assertFalse("<message>", <thing we expect to be false>)?

Not that I care much. Either works but assertThat(something, is_something) is nice wrt. consistency, assertTrue()/assertFalse() is nice for brevity.


s/when/as soon as/


@VegarM da er'e klart for deployment ;))


We can't have the same behavior when the user clicks on himself or on another user anyway, as the default behavior is to open a DM with the other user.


Good catch, @AurelioDeRosa ! If I understand correctly, grouping images in a single figure is ok, but we should add ARIA attributes. Something like this, I think…

```xml
<figure role="group" aria-labelledby="fig1">
    <!-- images -->
    <figcaption id="fig1">Figure 1: …</figcaption>
</figure>
```


please insert before encrypt_then_mac, as the standard extensions are sorted numerically


I adjusted the merlin default algorithm to lowercase 'ak' as this has become our default. Be aware that this might break some older configs!


Great, thanks


I'd make this a require instead of a check.


Since it is a prerequisite, I'd make this a require instead of a check.


same comment as line 99.


nit: unnecessary whitespace


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7878](https://teamcity.mesosphere.io/viewLog.html?buildId=85595&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:07:39


Ok, I can make that change. @AurelioDeRosa does this sound good to you?


You can replace comments with sections, so that they still "comment" the code and will also appear in test description/result.


this actually is a recursion that actually looks for the first child that's a service, whether it's in the parent or futher up in the tree of chilren.


the attribute should describe the object type, in this case it will simply be `bytearray`, you can describe what should it by by using `@param size: <description>`


I can't write, I can't sing, the only thing I know is how to ruin this thing...

https://youtu.be/qOyF4hR5GoE


Just another minor nit. I know I suggested above message but want to improve it further. Can we do following 'devmapper: Mount count of device went negative. Put() called without matching Get(). Resetting mount to 0'


Hm, this looks quite weird now, with the app screen sliding down *behind* icons on the home screen.


The Tip is titled "Provide alternative text for images", but the new description talks about sound and video. Either the tip title needs to be broader, or the description should be more focused. I mildly think it is useful to make it broader, since when designing, one might need to provide links to alternatives.

So maybe do include something like "Design a way for users to access transcripts and video with audio descriptions, as needed." (Probably not include captions – since that's usually specific to a media player. For audio descriptions, sometimes there is an alternative video with them, so would need to include a link to it.)

The wording is too formal and convoluted. For example, I think nowhere on the WAI website do we use "imagery". I suggest that it's not worth the extra complexity of info processing (especially for non-native speakers), and that we just use "images". I think the examples may add more complexity than they help.

"Consider how imagery, sound, or video will be accessed by users unable to perceive visual or audio content" -- we don't include this type of info in any of the other tips, afaik, so I don't think we want to do it here.
I'm not sure about "Communicate with writers and developers…"

(I won't spend time drafting specific suggested text until we decide whether this will just be images, or also other alternatives.)


Indeed, I was wrong while reading my own code :) sorry.


esto te lo he borrado porque ya estaba. Buen trabajo


I'm struggling to differentiate between code and formatting changes here...


Blank docstring


please add test cases in `unit_tests/test_tlslite_extensions.py` for this code


"and not spend" ? :)


We can exclude results parameters, it is used for temporary results gathering while validation is running. But since we have now confirmed that the raw_get() on line 161 gets us a complete and correct validation result, we no longer need it in the save_validation_results() function.


Don't just remove this test. Find an object that dill doesn't pickle and replace non_picklable.


These function names (setProjectDetails, setVolunteerAssignments) make me think you're setting class members rather than template vars. The code comments make it clear what the function is for, so maybe it's fine. I think I'd have done something like setProjectTplData, for what it's worth.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7880](https://teamcity.mesosphere.io/viewLog.html?buildId=85601&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:09:43


nit: please use one line to separete


You reading it almost correctly :-), even though it is not documented, `maxErrors` option is disabled when `autoConfigure` option is passed (see name of the test too).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7879](https://teamcity.mesosphere.io/viewLog.html?buildId=85600&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:10:12


Сейчас нет метода Authorize который принимает набор параметров, все текущие пользьватели библиотеки отвалятся после обновления. Верни его обратно пожалуйста и пометь как Obsolete.


this should be scenario_id instead of scenerio_id. No we don't need the name anymore, so kill it.


@borsch наш ресурс поки що тільки український


Sim neste caso nada será apresentado


ok


:facepunch:


Why are there two spaces before the comment?


I think the pipeline store should only deal with the persistence and lookups of pipelines. Adding this to pipeline store would only convolute the responsibilities of that class.


I like to keep this here, because those settings relate to the pipeline execution service.


That refactoring is part of this pr: #15203
(in this pr the only external dependencies are the client and Environment)


Nope. Doesn't work in IE


I think the 3 ```.gitignore``` files under ```features/hu.elte.txtuml.project/bin/hu/elte/txtuml/project/``` were created unintendedly. I do not see their point.


comment that we want to add extensions even when using just SSLv3 would probably be a good idea


Suggestion: "instantiated by the flow" -> "instantiated by (but separate from) the flow"


nit: line longer than 80 columns


Calling methods of model objects is unsafe from outside the model. This must be done with signals.


Suggestion: "flows only contain routing logic" -> "flows only contain routing (`next_node`) logic"


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


Yeah I noticed that as well, it's too small of a change to warrant a PR so I'll commit directly


It looks like there's a bug here. At line 147, the `AttrIndx` is set to `ATTR_IWBWA_OWBWA_NTR_INDEX` and the shareability is set to `ISH`. Here at line 151, you want to override these `AttrIndx` and shareability attributes in the descriptor but a bitwise OR won't do the job as far as I can see. It looks to me the correct code should be:
```
	if (attr & MT_MEMORY) {
		if (attr & MT_MEMORY_NC)
			desc |= LOWER_ATTRS(ATTR_MEMORY_NC_INDEX | OSH);
		else
			desc |= LOWER_ATTRS(ATTR_IWBWA_OWBWA_NTR_INDEX | ISH);
		if (attr & MT_RW)
			desc |= UPPER_ATTRS(XN);
	} else {
```

Also, as an improvement, it would be neat to print the cacheable/non-cacheable attribute in the debug traces below, i.e. adding a call to `debug_print(attr & MT_NORMAL_NC ? "-NC" : "-C")` at line 160.


How to choose the ID of GPU when using the "network.set_handler(PyCudaHandler())" when I have multi GPUs (i.e., in theano, we are using device = gpu2)? Thanks for your reply.


that's a very good point @agagata! I will add it! :sparkles:


Remove the x_labels line 152.


please add unit test for that, see `unit_tests/test_tlslite_tlsconnection.py` for example. You will need to create a MockSock that replies with an Alert message to a Client Hello and expect the connection attempt to fail


Should be done with valid data attributes, src-full is not valid HTML.


r+, thanks!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7881](https://teamcity.mesosphere.io/viewLog.html?buildId=85602&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7882](https://teamcity.mesosphere.io/viewLog.html?buildId=85603&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:52


Just to be sure that everybody sees this commit: I did a small update in the xsec values assigned to some samples (Higgs signals and single top samples). I also *removed* the temporary scaling of the Higgs signal done at plotting level in samples_run2. This is especially significant if you are doing plots where you look at the Higgs signal contribution, using Artus outputs produced before this commit.


@ricardoruas88 Excelente!


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


@sirmordred we dont need to add other libs at least not libseccameraadaptor.so it uses binder function but binder is loaded via libcamera which also loads libseccameraadaptor.so. I have looked at the libs and thats the only one that even uses binder functions other than libcamera.


This variable name is confusing. Anyway, it's probably better to have a class member to store information about all the needs associated with this project, and I think this lookup is probably unnecessary if your `setProjectDetails` method, instead of using the API, uses the BAO method `CRM_Volunteer_BAO_Project::retrieveByID()`, which returns a project object which has a property `->needs`.


goooo


yes we can use iterkeys


ddd


Can you change this line to:
        data_sets[set_name]["title"] = set_title


I think the option should be --no-contiainer-volume, since bind mount volumes should be allowed.

Should docker build with a VOL instruction fail also?


Humm ... ok.


The bootanimation zips are exact identical copy of each others (byte to byte)
Boot-animation may not be properly shown on some devices
So please provide different zips for different resolutions


You don't really need that package, just for converting. Why not just use the official stand-alone package?

http://commonmark.thephpleague.com/basic-usage/

```php
use League\CommonMark\CommonMarkConverter;

$converter = new CommonMarkConverter();
echo $converter->convertToHtml('# Hello World!');
```


@klali, I think you chose too simplistic template for CCC, which may or may not work. I think I and the other gentleman provided you with proven working templates, and would much prefer you to use one of those instead.


This test is wrong. It copy-paste from adguard code for windows. You need to change this test.


We need to delete antiblind, ,this does everything it does.


If this is an example, you want to use http://example.com; See RFC-2606


Not sure. I may want to remove defaultProps in the future?


change from program_run_name to set_name.


Move this try..except block to the top of the file, below the 'import' statements


i presume these colours came from the designs, but should the designs themselves use colours from the GDS palette?


You can set the environment variable `CUDA_DEVICE=2` for that.


## Why for this change?

This PR decouples the creation of `google.maps.Map


Lets add a test to use a bind mount volume  -v /run:/run or something like that to make sure we don't break this functionality.


Two empty lines between functions


Sorry, misunderstood DM :) .


You always stripped blanks from `$archs` with `${archs// /}`, but not in this case?


Two empty lines before if __name__ == '__main__'


I gather that calling the parent's `run()` method here short-circuits further processing in this class. I think it's worth noting in a comment or the function description that this is essentially a bail-out method.


Thanks @matthaias :+1:


You can comment out line 73, and duplicate it, and change to just:

test_config.start(config['id'])


Hi guys. Maybe BoardConfig.mk L39 "android.selinux-permissive" should change in "android.selinux=permissive" ? Thanks @Mazda--


Yes this looks nice. Continue work on this.

Re getting replica set details: I think that it is better to create a new API action to return a list of pods suited for this dialog. The reason is that replica set detail and logs menu are now a litte bit different and will diverge over time. Adding a new API method is cheap and will result in cleaner code.


Should we use the new error structures for this error.
Mayve simplified error

Image volumes are not allowed.


:+1: I've cherry-picked (I wasn't able to merge) this commit into `donations-demo`, and it's working great.


## Why for this change?

This PR decouples the creation logic of `google.maps.Map` instance and the mounting logic for `<GoogleMap>` component. As a result, we'll have:

### The correct lifecycle for `<GoogleMap>` component

Because when it's mounted (`ref` callback invoked with the component instance), it's __ready__ for use: invoking public APIs, reading values with getters ...etc

### Consistent API with `ScriptjsLoader`

See more in the example below.


@maciejmyslinski thanks for the feedback!


Har fiksa selv, pusher snart :)


I had a conversation with David about this, They're indeed not frmo the GDS palette, so he's going to check with the design team. In the meantime we're sticking to Eddie's colours.


Camilla oil?


it can be removed


Perhaps --no-image-volume is the correct option.


Man, I am still having trouble with testing in php. Why does this return 500? I have spent way too long trying to fix it.


The only required columns in the clinical file (at least for us) are SID and PID. The rest can be pretty much anything. The documentation is definitely a little behind on this - it is because we allow multiple clinical files for a study.


Two major things with this code right now. It needs to be liberated from the encompassing comment. Adjust for actual comments within the code. Also, per the style guide, it needs to be adjust to the 2 space tab setting from 4.


remove this line


1st commit だと何したかわかんないから、
何したかわかるコミット名にしよう。


As discussed, to be renamed "ClobColumnDef"


TListView columns are set in designer for "referenced by" list.


The tests on nullability are missing.


favorite color


Fixed


keep them for now, because they might be needed in future adaptations


I wonder why these +1, and +1?
Is that for correcting the days with multiple months?
If so:
-1 month should be 30
-2 month should be 90+1
-6 month should be 180+3
- 1 year should be 365
-3 years should be 3*365+1


AFAIK, `scriptjs` will try to load the same url just once. It has internal caching for this. But if there's different url……hmmm. I think there's nothing I can do for this.


Yes you can remove it.


Oops, thanks for the fix.


Fixed in 0b70b50. Same for Toasts.

So much prettier.


I think I prefer having all the extension's permission handling managed in `CRM_Volunteer_Permission`. This method duplicates some of that functionality. Providing information to the user about what permissions they need for access is not consistent with CiviCRM's general approach and may be giving away more information about the system than advisable.


Please document what problem (and in what software) you think this CCC might cause and suggest an alternative, preferably as a patch.


Done


Great comments!


The scan failed.
Included commits

Commit ID	Owner

2d69727498bf905d5d9abd48b311b014d392725b	@dudug


correct some code


That is not true: I found hard-coded references in the cBioPortal code to handle (and display) these attributes in a special way (the survival graphs are pretty significant), as well as AGE and either SEX or GENDER. This should all be documented on https://github.com/cBioPortal/cbioportal/wiki/File-Formats#clinical-data, together with an explanation of what the allowed values actually are.


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


@D-Alex, physics::Base::FIXED_JOINT doesn't exist.
It seems Gazebo doesn't have fixed joints.
They implement it with revolute joints with zero limits.
http://answers.gazebosim.org/question/1321/a-correct-way-to-implement-fixed-joint/


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7883](https://teamcity.mesosphere.io/viewLog.html?buildId=85608&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:28


Graba el archivo xml.


@aapav01 Does this change break wifi with the corresponding kernel changes?


"doesn't exist" without "s" :-)


Test


Please consider hard type hint here (Title) and removing the instanceof check.
At the moment there is no difference between template being recognized and attempt to classify a non-existent titile.
Logging is an alternative, but I'd lean on the side of fail fast


does that comment make sense anymore with your change?


Please remove stray keystroke.


- %s to be replaced by {}
- argument is missing !


userのavaterも保存しよう


My general understanding of the driver is that is actually relies on coulomb counting (integration of current over time). Any error in coulomb counting leads to a drift which is difficult to compensate without proper calibration. The voltage measurements are just for calibration. I will test the driver for a few more days.


and be careful to display the table name too.


This is not likely to work right out of the gate. Tonight will likely be spent fixing it.


quotes?


ほんとにこれだけで項目足りる？
userのprofileとかもう一回見てみて
http://proto-space.com/


Debug query builder shouldn't need a db connection. Just make `push_identifier` call `push_sql`. Don't need to carry binds around either.


Рахмонов Хайём M3237
https://raw.githubusercontent.com/Xayom/homework4/master/device-2015-11-12-023018.png


test 但上v啥地方v


@t2y Is it a good idea to add the `id` parameter here. In contrast to `index` and `typ` it is optional according to the REST spec. Can we agree to have only `index` and `typ` here. This would change the TV code from

```go
client.TermVector("twitter", "tweet", "1")
```

to

```go
client.TermVector("twitter", "tweet").Id("1")
```


Cool, thanks!


In this commit I adapted the validator to check whether the files conform to the format documented there. I reasoned that anyone maintaining the loading code itself would also assume that valid data files follow this format, and, say, look for the sample ID in the second column.


今回tagはacts_as_taggable_onっていうgemを使って実装するので、このアソシエーションは不要です。
DB設計の段階ではあんま考えなくていいけど、かるーく見といて
http://ruby-rails.hatenadiary.com/entry/20150225/1424858414


And `lsb_release` is not required any more on Debian? Maybe I am missing the bigger plan here.


@t2y Can you also fix the documentation here as well as in the other places? It should read as an English sentence.


d


Also, is this file removed upon uninstall?

How do you update this file if the user does a dist upgrade? If we cannot guarantee that then I would prefer to keep the old way. Better than having users who installed ReaR on Debian 7, upgraded to Debian 8 and ReaR never noticed.


ここも下記同様


Use relative anchor link to avoid redirect and reload of document.


@sirmordred ok so the only libs that load libbinder are :
libcamera
libril
libseccameraadaptor (which if you look in it youll notice it only uses binder function because like i said it and libbinder get loaded from libcamera)
libsec-ril
libsecril-client

so of those only 2 are for camera(i check every lib in my vendor) and im guessing the ril wont need any thing from the shim?


ほんとにこんだけで足りる？？？？？？
http://proto-space.com/prototypes/2
もう一回見てみよう


You should update the documentation for `checkProjectPerms()` to indicate that the operation constants may come from either CRM_Core_Action or this class.


würde es nicht sinn machen die commits in einen eigenen branch zu packen bzw gleich in den master. der branch hier war ja mal die von benno bzw github angelegte webseite...obwohl die wohl auch weg kann ^^


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7884](https://teamcity.mesosphere.io/viewLog.html?buildId=85613&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:05:03


I think I sorted it out with some git voodoo...
On Dec 3, 2015 1:22 AM, "Juan Shishido" <notifications@github.com> wrote:

> @matarhaller <https://github.com/matarhaller> Ah! Sorry about the merge
> conflict with summarize_essays.ipynb. It looks like that's the only file
> that was causing problems. We can checkout a previous version and commit it
> or do something else. I can help with that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/juanshishido/okcupid/commit/fe9155a1bcfd4ebcce1e3fd5687ccceaa59b2f5d#commitcomment-14756584>
> .
>


Please remove blank line.


@t2y Watch out: This does not work if `id` is missing! TV has [two endpoints](https://github.com/elastic/elasticsearch/blob/1.7/rest-api-spec/api/termvector.json). One with `index` and `type`, and another with `index`, `type`, and `id`. Can you please also add a test for `buildURL` and the two endpoints (see [other tests](https://github.com/olivere/elastic/blob/release-branch.v2/delete_mapping_test.go))?


My change is about using the right original context so yes it's still the same general logic.


prototypeに対してついてるlikeの数、commentの数　を保存するカラムも用意してください。

ちなみに上記の処理はcounter_cacheという機能を使うので、これもご一読。
http://railsdoc.com/references/counter_cache


prototypeにbelongs_to っていうことは、
どのprototypeに紐付いてるかを判断するカラムが必要だよね。


die wichtigsten und spassigsten beiden gems fehlen hier noch: rubocop und rspec ;-)


I'm not 100% sure if the reload logic will work as expected.  In the PostConstruct code, executed on each node, we call reload().  That in turn will generate UPDATE notifications for all triggers, so all triggers will be notified from all nodes.  Is that correct?   It seems more like reload() in a distributed env should just call reload(t) for the triggers relevant to that node.  I'm probably confused...


I found the former code block quite funny :wink::
```php
do {
    // ...
} while (false)
```


That makes quite a difference in performance, and it is not what is happening now. I'm open to discussing.

We can also make an API request when serving from the cache to get the latest content asynchronously but still render ultra fast to the user.

Future discussion/improvements?


For me calling velocityManager.getVelocityContext() means getting the current execution context (not the original one!). The javadoc of getVelocityContext even mentions this: "@return the current Velocity Context retrieved from the Execution Context"

This is why I asked.


This output is not appropriate for users to see.


I find it a bit difficult to correctly describe what this function does in the function name, which is a signal that it's probably doing too much.

I'm proposing to remove L11 and L12 from this method, they are not adding much value IMO.

```js
function createAppWithRest() {
  var app = loopback();
  app.set('host', '127.0.0.1');
  app.use(loopback.rest());
  return app;
}
```


Yes. The last few battles tend to get all the people, and knowing their punishment is reset, some of them tend to be bad. I believe this was added after a particularly bad reset where there were tons of spammers and the like in the last battle.


@barryvdh good point :) I was following along with what was in the project prior to the upgrade. This seems like it will do the trick though.


これはカラムとしては不適切。
likesテーブルはついたlikeを保存していくテーブルだから、
それに対して「like_or_not? 」っていう概念は存在しなくて、
そもそも全てliked っていう前提なので。

ただ「likeしたのどうなの？」っていうのを判定するっていう考え自体は大正解で、
これを使うのは後々メソッドとしてやね。


Done.

On Thu, Dec 3, 2015 at 5:46 AM, Greg Von Kuster <notifications@github.com>
wrote:

> The above file should not be re-added to the code base. Instead, a symlink
> should be created from this tool's directory to the single file in the
> directory above (i.e., ln -s ../icqsol_macros.xml .).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760856>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


Done

On Thu, Dec 3, 2015 at 5:50 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Each of these code lines should be a single line. It will not pass flake8,
> but will be more readable. The only exceptions we allow for flake8 is "Line
> too long" exceptions. Flake8 still assumes terminal windows are 80
> characters, which is no longer true.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760920>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


ACK (per https://github.com/ybznek/scap-security-guide/commit/06c12e622cf8bd6ea881c35e9cfc7c44951df376#commitcomment-13452135 10.2.4 might not be the most appropriate rule since it mentions login attempts, and this SSG rule is about invalid access attempts to files in general, but I couldn't find better mapping for this rule).

Maybe in future can be enhanced to add ID also to ```10.2.1``` (but good enough for now).

Thanks.


This check prevents my hooks from being executed because `$results` is `null`.
What exactly is this check for? Looks like a regression to me.


ReportGenerator also generate word reports, not only powerpoint


Done

On Thu, Dec 3, 2015 at 5:51 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Make the above a single line for better readability.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760942>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


tagは上記の通りgemを使用するので、テーブルは自動生成されます。


Can you use your new function `listenWithRemote` here too please?


これも上記同様


Was failing with packageInstance: `new null()`


Perhaps `listenAndSetupRemoteDS` would be more descriptive?


ん？　写真はどこに保存する？？


ACK.


The ampersand should be URL encoded: \&

Anyway, for this view let's open the report in a modal rather than load the snippet directly into the browser.


Okay, thanks for the clarification!


@burtonageo If you are new to Github then this can be tricky! :+1: Please respond to confirm that that you receive these messages.


Sure. It make more clear~. What about "'devmapper: Mount count of device went negative. Put() called without matching Get(). Resetting count to 0"?


nice changes dude!!!


Operator-pending modeでやっちゃいけないことがあってE523が出るのでnoを足せば回避できるのは確認したけれど、normal!コマンド起因か確認してませんでした…


Actually, they do have fixed joints. https://bitbucket.org/osrf/gazebo/src/6f7cfa4697e8abd7c7af87306cda269acc85512c/gazebo/physics/Base.hh?at=default&fileviewer=file-view-default#Base.hh-119

I guess you are still on gazebo 5. The current version is 6.1


Hi Greg,

I'm not seeing that line...

$ grep python *

icqsol_rotate_shape.py:#!/usr/bin/env python

icqsol_rotate_shape.xml:    python
$__tool_directory__/icqsol_rotate_shape.py

?

On Thu, Dec 3, 2015 at 5:52 AM, Greg Von Kuster <notifications@github.com>
wrote:

> python $*tool_directory*/icqsol_refine_shape.py? Should this be python $
> *tool_directory*/icqsol_rotate_shape.py?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14760966>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7886](https://teamcity.mesosphere.io/viewLog.html?buildId=85615&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 205 Build time: 00:04:53
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Yea, when I downloaded the latest Terraform binaries for my pairing user, terraform was throwing exceptions because it couldn't parse terraform.tfvars due to the "." in the key name. Then I found a GH issue someone had filed about it and they had just quoted their key names in those cases. It appeared to fix my error, but I don't thing it makes sense that this should have just started becoming a problems. Unless they broke something with an update recently.


On second thought, using `link_directories()` is probably not necessary. I'll check this...


写真用カラムはcarriewaveというgemを使うからあとからでいいと別の方に伺っていたのですが、
追加した方が良かった追加します！


This operation should be secured for ROLE_ADMIN


Seems my concern was confirmed here :) Could you please add a unit-test to prevent regressions in the future?


Why editing something in console should mean always by default doing reload? The edited fields should be propagated to model even without reload. You don't need the values being used by default, am I missing something?


#include<stdio.h>
#include<stdlib.h>
int main{agrc, *agrv}
{

}


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7887](https://teamcity.mesosphere.io/viewLog.html?buildId=85616&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:57


ACK.


If you duplicate R10/R11 and offset it by 8 bytes, you can potentially

```
 MOVQ    R10, R12
 SHRQ    $8, R12
```
And shift by R10/R12 16 bits for each value. That way you break the dependency. It may give an additional 1-2% speedup.


Do you think we could store this information on the rollupsummary record instead of hardcoding what time it should run? To be very clear, I'm in no way expecting we build any UI around how scheduling works (for now). I'd be happy with the ability to just store the string '0 0 2 * * ?' on the record so we could edit it to run at a different time. Let's say I have integrations that run all night and I want to schedule them to run at 10AM and not 2AM?


@Chiel92 Yes and no. The reason this particular error is thrown is not because the file isn't found, but because it has to be found.
If it was a fileNotFound error, I'd suppose someone might try to catch it (no pun intended), but this one is not supposed to be caught. It has to succeed, hence the assertion.


It is possible to use an AsyncTask for this purpose. You have to be careful with locks though.


Let's set up your IDE to have better behavior re whitespace and [code style](http://wiki.civicrm.org/confluence/display/CRMDOC/IDE+Settings+to+Meet+Coding+Standards).


@pletzer Your local disk file must be different that what you have committed to github.


You are bored.


It is. I think the github folding was hiding the line above.


It's looks like String Template suitable  . I put a example here [StringTemplateExample](https://gist.github.com/darionyaphet/47add85fa3e790981858)


Pull requests need to have proper comments and descriptions. "Kappa" is not really sufficient. @nabbhacker


@pecsi nice - wondering if you are interest in sending a PR to our AeroGear server

Some other community members showed interest in that too

So, I think we, as AeroGear team, are open to that, as an addition


Docker build with VOLUME will fail as per the current implementation

And yes I like --no-image-volume more thanks


+1
I left that by mistake. Good catch.
I will fix it on the next feedback commit.


Sure I'll add one and change the text


The scan failed.
Included commits

Commit ID	Owner

2d69727498bf905d5d9abd48b311b014d392725b	@dudug


Then the message did not made sense before either. The previous code was getting the Velocity context from the XWikiContext which is supposed to be the current one so it is the same logic. The change is just about making sure we always get one (and an up to date one). I would not have written the comment like this but since the previous code was also about current context I tough whoever coded that used "original" as the "official current one".


+1
Yeap, isDebugEnabled() on catch() blocks that are not frequent are too much.


Right. Totally missed it.


we should add a tooltip that explains that from should be less than to.


Ok


Sure, the current implementation won't break that because here I'm explicitly looking for Volumes in image config


`cause` is of type `String`. Are you sure this change is correct?


@dmethvin Are we using the `#`-type references for GitHub issues/PRs now?


Actually, I fixed this anyway as part of my commit, but I'll enable the tests as well.


You could've gone the `display: inline-block` `vertical-align: middle` route here. I'm looking at the [flex compatibility table](https://developer.mozilla.org/en/docs/Web/CSS/flex#Browser_compatibility)…


Why tis the square of the radii used here? Isn't p.dist the linear distance in meters?


ACK.


OK, you were right, but only because I misunderstood `collection`. It doesn’t contain all the selected objects. It only contains those that you’ve hit comma on. So select `A`, hit comma, select `B`, execute and `collection` will only contain `A`. So the `objectValue` was missing `B`. I’ve reverted this commit, but I’m waiting to push until I fix the other small issues.


## thirtyyuan的个人博客

## 关于我

Android developer ,and who want to make a better place

## 联系我

* Github: [thirtyyuan](https://github.com/thirtyyuan/)

* Blogger: [thirtyyuan](thirtyyuan.blogspot.jp)


@jacmarjorie - I see what you mean; the use of 'required' is not consistent thoughout the file, but this field is required.
@mbaudis - this update pulled in changes which had happened in the master branch - the discussion is on #253 .There will be another such synchronisation update shortly as the documentation structure has recently gone into master.


Only when I forget. :smile: This is the only repo where I use them.


the user gets asked two yes/no questions, which is confusing. Suggest " - you may have another prototype running.\n"


ACK for ```10.5.5.``` (monitoring changes on critical system files).


next time dont use "Kappa" for PR reason. I can merge then.


@higebu It seems in the latest version of 'netev_stdlib' module type naming convection is changed as compared to its older version ie. 'netdev_interface' is changed to 'network_interface' and uploaded [here](https://forge.puppetlabs.com/puppetlabs/netdev_stdlib/readme)
.
'netdev-stdlib-junos' module still uses old [naming convention](https://github.com/Juniper/puppet-netdev-stdlib-junos/tree/master/lib/puppet/provider) and is dependent on module uploaded [here](https://forge.puppetlabs.com/netdevops/netdev_stdlib).

Since 'netdevops/netdev_stdlib' module still is available for download on Puppet forge, is this pull request required?


@miked-mellanox @alex-mikheev i think we should not use external oshmem api inside. At least to provide correct profiling info. What is your opinion?


We could but isn't that one of the advantages of CSS - not having to change class names or markup when making a style change? I see your point in this example but not sure if variable names should be tied to class names. Can make the change if that's how you've been doing things.


we don't terminateOngoingHanshake anymore (by sending an alert)


Awesomeness.
Very nice scaling.
It seems these weights may be tuned further with longer time control.


Good catch, will change back.


thx a lot


I'll fix it. Thanks @Nettsentrisk :+1:


Sounds good.


Why does we need a ConnectionProvider ?


ACK.


@akbertram we always want to be able upload/replace image, isn't it? Here if upload is successful placeholder.visible=false. Means browse/replace link becomes invisible. I guess you had some other idea here.


@ldoktor yep.


Is this working in the sparse case? I thought `np.max(X.dot(X.T)[range(n), range(n)])` returned the diagonal part already. I could not figure out how to use `np.diag` on sparse arrays when experimenting and had to fall back to this range(n) dirty trick.


In linux the group is called proactive not proactiveGroup. It should be the same group as the ProActive Agent is using.


Done.

On Thu, Dec 3, 2015 at 6:00 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Change the above to use the standard call used in all other CSG tools:
> output_format, output_file_type =
> icqsol_utils.get_format_and_type(args.output_vtk_type)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14761124>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


"and memory leaks". I think possibly is too weak, since it will happen inevitably


Looks like we have to steal the mw-ui-button mixins… Not now though, obvs.


maybe add a note that you can turn the pooling off on the environment


It's rsync, man! _Remote_ Synchronization! Please, remove this check ;-)


Here again, it is confusing to have proactive for linux and proactiveGroup for windows


:poop:


Rephrase this sentence, it is not clear


I take maximal possible numbers (31 days in month and possible leap-year). It's better to have one day plus than minus in my opinion.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7888](https://teamcity.mesosphere.io/viewLog.html?buildId=85621&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:52


technically if something is "ReferenceCounted", right? ByteBuf implements ReferenceCounted.. or we can leave that off since its a nuance


@alex-mikheev what is an idea in using `(me + i) % nproc` instead of `i`?


To move on to the next one, I think you want to add `continue;` into this if-block.

Are you sure this check is adequate? Needs come in at least two flavors:

1. Fixed date: Start time and duration are set. Activity is expected to start at *start time* and last *duration* minutes.
2. Fuzzy date: Start time, end time, and duration are set. Activity needs to be completed between *start time* and *end time* and take *duration* minutes. Example: I need 5 hours of filing completed between December 1 and December 31.

Seems that to properly account for the former you'd need to take the start date into account.


And right - I overlooked 366 in 3years..


Why do we install the ProActive Agent? The ProActive Node should be enough. The run as me section should cover as less other parts as possible.


Оказывается всё так просто было. Лисоплеер реагировал только на событие `volumechange`, а на выставление громкости при помощи `<foo>.volume = ...` ему было насрать.


you can add a note here that if they need to enable it in prod, setting it to ADVANCED is not as bad as paranoid and be a good middle ground.


Hi Greg,

Do we need those? The shape manager knows how to instantiate the correct
reader depending on the file's suffix. In the case where the suffix cannot
be used to distinguish between I could modify the code to try several
readers until the right one is found (there is a good chance that this
would work but needs to be tested).

--Alex

On Thu, Dec 3, 2015 at 6:02 AM, Greg Von Kuster <notifications@github.com>
wrote:

> You'll need these:
> --input_file_format_and_type $input.ext
> --input_dataset_type $input.metadata.dataset_type
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14761153>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


Table should be an option.


lol


Nice one. Another suggestion would be to fragment the dependencies into commands like seletion_pipeline ihs selection_pipeline fst etc. selection_pipeline full_run etc. Would need to create a bunch more code to determine the dependencies for each of the outputs though. This could be implemented in another class that searches for prerequisite files, skipping over steps when they are present, this is always something I wish I had had time to do.


I agree, @lpellegr has started to work on that after lunch


:hankey:


I think:
`currentIndex  - width == params.text.endPos()`
will never be true because you end the loop on currentIndex == params.text.endPos(), correct?


* agree re API
* this makes connections more balanced (not many-to-one pattern)


you might want to fix the versions!

2.2.2 and 1.4.10


Please address this TODO.


It seems wrong to tell the user that the retention is, for example "3 months" and then actually implement that as a fixed number of days, which may or may not be equal to 3 months.

As a developer who has just worked on a project involving dates ands intervals, I would suggest to keep it simple, and don't let the user specify numbers of months or years, especially if your internal representation is just days.  As a user, I would be just as happy to say "52 weeks" as "1 year", or "4 weeks" instead of 1 month etc


I think this is backwards. It makes sense to me that the near-future shifts would appear at the top of the screen and the far-future shifts at the bottom.


thanks so much for the quick response.  Are you aware of potential issues with `_` underscore in `CSS`?  Here is a link https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Underscores_in_class_and_ID_Names. I think `-` is usually considered better, but I am far, far, far from an expert here :)


In addition to the inline comments, I think it would make sense to add a paragraph or two about:

- That we bundle the bytebuf, which means they need to make sure to use the one we package, otherwise their code won't compile. This is especially important if they use netty somewhere else too (as a direct dependency)


This was actually a bug.

Consider the client being connecting to `test` keyspace. This will result in the table name being `test.`.


Remove excessive empty line.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7889](https://teamcity.mesosphere.io/viewLog.html?buildId=85626&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:31


nice


Greg,

What do you see at

https://github.com/pletzer/galaxy-csg/blob/master/tools/icqsol_rotate_shape/icqsol_rotate_shape.xml
?

I have

<![CDATA[ python $__tool_directory__/icqsol_rotate_shape.py

--Alex

On Thu, Dec 3, 2015 at 7:14 AM, Greg Von Kuster <notifications@github.com>
wrote:

> @pletzer <https://github.com/pletzer> Your local disk file must be
> different that what you have committed to github.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14762778>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


@pauljeff if you click through to the PR link below the commit message (#33), you'll see a reference to an issue on the docker repo (https://github.com/docker/docker/issues/16602). On that issue there's a link to a google groups post I made which should explain the issue.

In short: without `-t` socat jumps in and closes half-closed connections.


Done.


This appears to work:

[root@schoolserver xsce]# ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 16436 qdisc noqueue state UNKNOWN
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet 127.0.0.1/8 scope host lo
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: eth1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP qlen 1000
    link/ether 00:14:d1:da:35:3f brd ff:ff:ff:ff:ff:ff
    inet 192.168.0.19/24 brd 192.168.0.255 scope global eth1
    inet6 fe80::214:d1ff:feda:353f/64 scope link
       valid_lft forever preferred_lft forever
3: wlan0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq master br0 state UP qlen 1000
    link/ether 40:a5:ef:05:5c:87 brd ff:ff:ff:ff:ff:ff
    inet6 fe80::42a5:efff:fe05:5c87/64 scope link
       valid_lft forever preferred_lft forever
4: eth0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc pfifo_fast state DOWN qlen 1000
    link/ether 00:17:c4:a7:2e:81 brd ff:ff:ff:ff:ff:ff
5: br0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP
    link/ether 40:a5:ef:05:5c:87 brd ff:ff:ff:ff:ff:ff
    inet 172.18.96.1/19 brd 172.18.127.255 scope global br0
    inet6 fe80::2cdd:3eff:fe42:ee92/64 scope link
       valid_lft forever preferred_lft forever
6: tun0: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UNKNOWN qlen 100
    link/none
    inet 10.8.0.18 peer 10.8.0.17/32 scope global tun0

[root@schoolserver xsce]# systemctl status hostapd
hostapd.service - Hostapd IEEE 802.11 AP, IEEE 802.1X/WPA/WPA2/EAP/RADIUS Authenticator
   Loaded: loaded (/etc/systemd/system/hostapd.service; enabled)
   Active: active (running) since Thu 2015-12-03 01:59:24 GMT; 11h ago
 Main PID: 3392 (hostapd)
   CGroup: name=systemd:/system/hostapd.service
           └─3392 /usr/sbin/hostapd -P /run/hostapd.pid /etc/hostapd/hostapd.conf

Dec 03 01:59:24 schoolserver.lan systemd[1]: Started Hostapd IEEE 802.11 AP, IEEE 802.1X/WPA/WPA2/EAP/RADIUS Authenticator.
[root@schoolserver xsce]# cat cat /etc/hostapd/hostapd.conf
cat: cat: No such file or directory

# Basic configuration

interface=wlan0
ssid=Internet in a Box
channel=6
bridge=br0

# Hardware configuration
driver=nl80211
ieee80211n=1
hw_mode=g

But xo-1.5, win7, and android phone all fail to connect.


I know `"with-fortran"` is suspect on this line... It has the effect of triggering a re-build of OpenMPI. This is the only way that I can figure out to get OpenMPI setup correctly. This way, it will pass the Jenkins tests and one of the Travis-CI tests, although, [the other two time out](https://travis-ci.org/Homebrew/homebrew/builds/94518198) for reasons that are not clear to me. (I know Travis-CI was having some [issues with its Mac infrastructure](https://www.traviscistatus.com/incidents/0rgmbqxybdg3)... maybe these aren't completely fixed?)

Without triggering this rebuild of OpenMPI, it seems that the [mpif90 wrappers are broken](https://travis-ci.org/Homebrew/homebrew/jobs/94463437#L532) on the test-bot infrastructure.

This is one of the errors that I keep getting:

```
CMake Error at /usr/local/Cellar/cmake/3.4.0/share/cmake/Modules/CMakeTestFortranCompiler.cmake:54 (message):
  The Fortran compiler "/usr/local/bin/mpif90" is not able to compile a
  simple test program.
  It fails with the following output:
   Change Dir: /tmp/opencoarrays20151202-5216-1s3y50t/opencoarrays/build/CMakeFiles/CMakeTmp

  Run Build Command:"/usr/local/Library/ENV/4.3/gmake" "cmTC_6a8d2/fast"
  /Applications/Xcode.app/Contents/Developer/usr/bin/make -f
  CMakeFiles/cmTC_6a8d2.dir/build.make CMakeFiles/cmTC_6a8d2.dir/build
  Building Fortran object CMakeFiles/cmTC_6a8d2.dir/testFortranCompiler.f.o
  /usr/local/bin/mpif90 -c
  /tmp/opencoarrays20151202-5216-1s3y50t/opencoarrays/build/CMakeFiles/CMakeTmp/testFortranCompiler.f
  -o CMakeFiles/cmTC_6a8d2.dir/testFortranCompiler.f.o
  --------------------------------------------------------------------------
  No underlying compiler was specified in the wrapper compiler data file
  (e.g., mpicc-wrapper-data.txt)
  --------------------------------------------------------------------------
```

I'm at a loss for how to proceed, and could really use some ideas/help.


```10.7.a``` makes sense (can't find better mapping). ACK.


:ok:


Use the supplied client or create a new one.


example.com -> https://tools.ietf.org/html/rfc2606


Yes, I thought that ` terminateConnection(peerAddress)` would also cancel any ongoing handshake by means of an alert message. However, this is not the case.
Do you think that we should handle an unexpected runtime exception differently depending on whether it occurred while processing a message in the context of the established session or the ongoing handshake?
Or should we simply terminate both an established session as well as an ongoing handshake?


Typo: "or".


List the set of variables needed for Project Tiger for Roposo


Yes, we need these.  Galaxy stores all files with a .dat extension, so looking at the file system to determine the file format is not possible.  This Cheetah template is not calling the shape manager.  It is calling the underlying Python script with calls the shape manager.  So unless the underlying Python script has this information, it cannot pass it on to the shape manger.

Also, writing code that depends on subdirectory names or file names and extensions on a file system is non-optimal, and generally results in brittle code.  I continually deal with this issue in the labs.  Our code should not depend on any file system path names or extensions if at all possible.


The real options are in `this.client.options`.


Why is this code duplicated from above? Can we not just treat these domains the same?


wait, are you saying using `-v /something:/sometihng` to override the `VOLUME` instruction in the image should be valid?


Connect only if not connected already.


good call


I agree only partially. There can be external reasons why the file is not there (or could not be read). IMHO an AssertionError should only be raised if there is a bug in the code.


@mitchmindtree I've set PistonCollaborator access to `Read` temporarily.


I think that what you may want to do is [here](http://docutils.sourceforge.net/docs/user/rst/quickref.html#external-hyperlink-targets), 2nd example table, meaning that the first ` character should be put before the word _install_.
Sorry again if I wasn't really clear in my previous messages. Hoping this one is...


It is in the style guide to have at least two spaces between code and comments on a line.


Я б на вашем месте сделал метод в дао для выборки только юзеров с рассылкой


disconnect upstream dongle and rerun xsce-network succeeds

However, unplug upstream dongle and rerun xsce-network fails:

device_gw2 undefined

(can't paste log as no connection)


That line also gets a warning on Mac OS X:
/j/julia/src/gf.c:1538:34: warning: comparison of distinct pointer types ('jl_value_t *' (aka 'struct _jl_value_t *') and 'jl_datatype_t *' (aka 'struct _jl_datatype_t *'))
      [-Wcompare-distinct-pointer-types]
        jl_cellset(vi, 1, tv->ub == jl_any_type ? jl_any_type : jl_wrap_Type((jl_value_t*)tv));


Better error handling is needed here. Not being able to connect should not allow going any further. Return / throw an error.


task was Add location section to config file


A reload() operations means that all triggers from the backend should be reloaded into the AlertsEngine memory.
In a distributed operation, each trigger will load only the triggers that are defined by the internal partition.
AlertsEngineImpl.onTriggerChange() is only invoked if that node holds the trigger.
So I think as working as expected.
The only side effect if I have two nodes, we will have two reloads but that is an expected behaviour.


Agree on api. I think that we probably need to fix this in separate PR because preconnect may be not the only code were
external api is used

From: Mike Dubman [mailto:notifications@github.com]
Sent: Thursday, December 03, 2015 4:28 PM
To: mellanox-hpc/ompi-release <ompi-release@noreply.github.com>
Cc: Alexander Mikheev <alexm@mellanox.com>
Subject: Re: [ompi-release] OSHMEM: add skew to the oshmem_preconnect_all (7a1d70f)

  *   agree re API
  *   this makes connections more balanced (not many-to-one pattern)

—
Reply to this email directly or view it on GitHub<https://github.com/mellanox-hpc/ompi-release/commit/7a1d70f69de32f98d7e31fabe3b009e2c9555519#commitcomment-14763114>.


递归使用模板， 相当于二维数组    int twodee[10][5]


parasite_dump_pages_seized from which you has removed dirty bits clearing is called in two places:
./cr-dump.c:
1119     ret = parasite_dump_pages_seized(parasite_ctl, &vmas, &parasite_ctl->mem_pp);
1275     ret = parasite_dump_pages_seized(parasite_ctl, &vmas, NULL);

but dump_task_mm() is called only once in cr-dump.c:
1327     ret = dump_task_mm(pid, &pps_buf, &misc, &vmas, cr_imgset);

So one task_reset_dirty_track() call is lost after this patch.


opensubtitles


Please remove the suffix `_PREDICATE` from constant names for this and other constants.


It might be a little safer to use a value we know no one else will use instead of the 3. A string like 'volunteer_view_listings' perhaps.


@ybznek
I would map this one to ```8.7.c```. If you want to keep also ```Req-8.2.1```, could you also add ```Red-8.7.c``` ?

Thanks, Jan.


Should I use .ConfigureAwait(false) for here or not?
(currently,  .CA is removed for preventing unintended jumping among threads.)


Done.


Please rename the variable to `paramCountIsCorrect`.


Perhaps I was looking at an older commit in your PR stack?  A better practice is to submit 1 commit for a PR.  You should not be committing many small changes and creating a single PR out of them as this make the PR much more difficult to review and manage.


Thanks! :+1:


Well, removing `-DCMAKE_BUILD_TYPE=Release` seems to have fixed the issue, which was probably caused by a bug relating to an optimization flag in the compiler toolchain (not in OCA). The `Release` build type, added as part of Homebrew's `*std_cmake_args` was mucking with the optimization flags set in the `CMakeLists.txt`. Removing this resolved the issue.


Yes, well basically Openshift wants to be able to have a user run something like

docker run -v $(HOMEDIR):/var myimage

But if myimage includes a volume other then /var it should fail.

Openshift can control $HOME for the user via quota but can not control VOL inside of image.


Again. Would add ```8.7.c``` (```/etc/shadow``` is a database in plain-text file format / form storing passwords information). Since this rule is checking of groupowner of /etc/shadow is root, we are ensuring here the database is accessible directly only to the database administrator (as written in the requirement) => ```8.7.c``` IMHO is better fit here (but we can keep 8.2.1 too).


$.map ?


Indeed it is... thanks--committing update.


Didn't thought that would even matter. It's w/e tho.


Again. ```8.7.c``` IMHO.


It doesn't look like a utility class any more.


@sirmordred i updated mine here https://github.com/doadin/android_device_samsung_ancora_tmo/commit/6964089de549812e44c846da4b841264ce70ec43


The scan failed.
Included commits

Commit ID	Owner

2d69727498bf905d5d9abd48b311b014d392725b	@dudug


Je donne un carambar pour celui qui resout les inclusions, je pourrais le faire, mais trop la flemme


ACK (```8.2.1``` makes sense here).


Then it became:
if (scenario_name in sets[set_name]) and (scenario_name in _set):

as _set is sets[set_name] then the condition checks the same thing twice. So I have reduce to:

if scenario_name in _set


Yep, thanks for spotting this one, I messed it up with this change. I'll look into it.


i didnt mean to comment on cometo's PI code.  this branch already had it updated.


The whole point is a bit that the code is depending on certain files (default setting file, default colors file, all source code files, images) being there and being accessible. If they cannot be accessed, the code is not working.

Of course, a user can go to the source folders and remove (or change) the settings file. He can do the same with the images or other code parts. If they fail an assertion error should fit well right?

The only other option I see is to shut down the program with a message: Default settings file could not be read, but that's kind of what I'm doing now right?


ACK.


Good point, we probably don't anymore. During the initial re-factoring it seemed like a good idea to defer the (expensive) lookup of the `Connection` object to the point where the connection was actually needed. However, now it seems that we do the lookup quite early on and thus should simply pass along the 'Connection' instance itself. I will remove it ...


Ok.


Am I misreading/misunderstanding https://github.com/joakin/loot-ui/blob/0b70b50b1f8723d0e17a0e892e123ecd718f2013/lib/service-worker.js#L40?


Done.


Please inline the variable and avoid using class name in the call.


Greg,

Ok, no worries. I added these options. I think the changes you required
have been made. Let me know if something else comes up.

Thanks,

--Alex

On Thu, Dec 3, 2015 at 7:33 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Yes, we need these. Galaxy stores all files with a .dat extension, so
> looking at the file system to determine the file format is not possible.
> This Cheetah template is not calling the shape manager. It is calling the
> underlying Python script with calls the shape manager. So unless the
> underlying Python script has this information, it cannot pass it on to the
> shape manger.
>
> Also, writing code that depends on subdirectory names or file names and
> extensions on a file system is non-optimal, and generally results in
> brittle code. I continually deal with this issue in the labs. Our code
> should not depend on any file system path names or extensions if at all
> possible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763258>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


`node_modules` is now in this file twice.


nice! looking forward to pull request


Please shorten `Parameters` to `Params`.


ACK (```10.7``` makes sense here).


Build 16 test results:  26 passed, 0 failed, 0 errored, 26 total Merge pull request #32 from buzzfeed/feature/use-nsurlsession

Use NSURLSession instead of NSURLConnection


@sarahhunt thanks for the clarification. I was getting all worked up stumbling upon the ```Dataset``` abuse through the updated documentation in ```Metadata.rst``` - which I thought I should be aware of. This is a bigger issue it seems ...


Greg,

Ok, will keep this in mind...

--Alex

On Thu, Dec 3, 2015 at 7:37 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Perhaps I was looking at an older commit in your PR stack? A better
> practice is to submit 1 commit for a PR. You should not be committing many
> small changes and creating a single PR out of them as this make the PR much
> more difficult to review and manage.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763401>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


SMART_ACTION_COMBAT_STOP = 27,
The wiki must be updated and probably your core.


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


NB: I've split the README.md into multiple files, to view them rendered (for now), visit my repo :

- https://github.com/b-long/packagebuilder/blob/feature/fault-tolerance/README.md
- https://github.com/b-long/packagebuilder/blob/feature/fault-tolerance/documentation/Developing.md
- https://github.com/b-long/packagebuilder/blob/feature/fault-tolerance/documentation/Troubleshooting.md
- https://github.com/b-long/packagebuilder/blob/feature/fault-tolerance/documentation/Installation%20Procedures.md


NACK for this one. ```10.5.2``` suites better for this one IMHO (the SSG rule is ensuring auditd configuration isn't modifiable without reboot).


How about this:

```
if (Message.class.isAssignableFrom(returnType)) {
   return true;
}

if (List.class.equals(returnType)) {
    return true;
}

   return false;
}
````


Ah, don't worry about commit messages. It's always better than https://github.com/sferik/active_emoji/commits/master


ACK.


Done.


This was not intended to reach master, sorry. Fixed in 49358274d7145788f2e99471dbb7c7c4301a3d3d.


И класс вынеси в отдельный файл. Хотя может и я это сделаю вечером


@kkrampa what is the desired effect of ```table.alias('x')```?


Ok.


@klauspost I've tried this. It appears to be slower with independent registers. I don't have an explanation for this. I've also tried two other versions where I shift R12 by 32 and use the lower and the upper halfs separately. They're also slower on my machine. (I benchmarked the function itself, of course.)


Please rename the constant to `COMMAND_HANDLER_PARAM_COUNT`.


Also ```10.2.5.b``` (audit elevation of privileges) could be added here.

But ACK in general.


Added retry logic that we talked about at meeting on 12/1.  The two errors that I've seen are "502 Bad Gateway" and "503 Service Unavailable".  Being conservative, the retry is only attempted for those two errors. And to keep it simple, the wait time between retries is 1 second. We could get fancy and make the timeout variable if needed.


Done


Good question. I can't see what different behavior we could implement, so I suppose we can just terminate both established session as well as an ongoing handshake.

I see another difference here, with the previous version its seems that now we terminate connection even on non "Fatal" HandshakeException.


Once the preference adapter is available again, it will refresh it.


update


i do have a gg, please correct it


There is no `<state key>`; this doesn't really make sense.

I would try rewriting this paragraph:

"State events can be sent using this endpoint. This endpoint is equivalent to calling `/rooms/{roomId}/state/{eventType}/{stateKey}` with an empty `stateKey`. Previous state events with matching `<roomId>` and `<eventType>`, and empty `<stateKey>`, will be overwritten."


В большинстве случаев правильнее было бы использовать string.IsNullOrWhiteSpace() потому что доверять пользовательским данным нельзя ведь можно передать пробел и твоя проверка станет бессмысленной


Have you seen this one?

https://github.com/gregvonkuster/galaxy-csg/pull/18/files# <https://github.com/gregvonkuster/galaxy-csg/pull/18/files#>

> On Dec 3, 2015, at 9:45 AM, Alex Pletzer <notifications@github.com> wrote:
>
> Greg,
>
> Ok, no worries. I added these options. I think the changes you required
> have been made. Let me know if something else comes up.
>
> Thanks,
>
> --Alex
>
> On Thu, Dec 3, 2015 at 7:33 AM, Greg Von Kuster <notifications@github.com>
> wrote:
>
> > Yes, we need these. Galaxy stores all files with a .dat extension, so
> > looking at the file system to determine the file format is not possible.
> > This Cheetah template is not calling the shape manager. It is calling the
> > underlying Python script with calls the shape manager. So unless the
> > underlying Python script has this information, it cannot pass it on to the
> > shape manger.
> >
> > Also, writing code that depends on subdirectory names or file names and
> > extensions on a file system is non-optimal, and generally results in
> > brittle code. I continually deal with this issue in the labs. Our code
> > should not depend on any file system path names or extensions if at all
> > possible.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763258>
> > .
> >
>
>
>
> --
> Alexander Pletzer
> +1 609 799 7999 cell
> alexander@gokliya.net
> —
> Reply to this email directly or view it on GitHub <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763628>.
>


@jwaterfield should this be:
```python
name = spectra_name + "_" + par_name
```
I can change it.


Math.random()が０から１までの乱数を生成するみたいなので、１０をかけてMath.floor()で小数点を消して、１を足すという周りくどいことやってるんですが、Javaみたいな便利な生成はできないのでしょうか。


Well, in `for-loop`, of course... I don't see / understand the necessity of all this, instead of
```lua
for i = 1, T do table.insert(clones, model:clone(params, gradParams) end
```
Did you write this chunk of code, or did you borrow it from somewhere else?


Outstanding contribution. Isn't it, @sulewicz?


header* :anger:


should be at the end of file


@beanstown106 device/lge/bullhead :)


```
docker build -t overrideVolumes - <<EOF
FROM busybox
VOLUME /test
EOF

docker run -v /somethingonhost:/test overrideVolumes # this won't error right?
```
but this will error out:
```
docker build -t overrideVolumes - <<EOF
FROM busybox
VOLUME /test
VOLUME /another
EOF

docker run -v /somethingonhost:/test overrideVolumes # this will error out because 2 VOLUME are defined and one only is overridden
```


Done.


use the existing #defines in byteorder.h


What is the problem with search?  And the etc?


No, lots of libraries have semantically problematic APIs so it isn't Hamcrest in particular.


right, that was my comment, for N nodes we will have every node posting an UPDATE for all loadable triggers, so each trigger N times.  If that is the design then it is OK, although it may be worth a comment in the code similar to what you just wrote here.


you're right.  If it's at the end of the text it gets found in the post loop section.  I'll change it.


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


Oh yes - the link to the PR is right at the top of the page. Should look more closely. Thanks @aidanhs.


Deze lijkt me niet echt nodig, toch? We gaan nooit oudere versies ondersteunen...


@friism Thanks! I've already made a PR.


Not sure if it makes sense to move this to the end. Because the caller of the behavior will continue anyway.


I think I've just implemented that manually ^^


Use ExecuteAsync


Readme update


so much more readable


Require user to `USE <keyspace>`;


Was there any particular reason for this change? We serialize generated classes with Newtonsoft Json.NET and having the ByteBuffer as a public property on every generated class really fucks that up.

Commenting the line doesn't break anything so I see no reason for the change being in there?


OnEditWorkspaceExecuteAsync


Done


@enricosada Just to mention that the inclusion of project files with so many files in them have caused major performance problems when editing in Visual Studio 2013 or 2015 with src\fsharp.sln.  Switching between files in that solution now takes about 8 seconds because something is traversing the entire solution file set on each switch (I don't know what - it's not the Visual F# Power Tools but might be either the Visual F# Tools or some other standard Visual Studio component that's active)

I can no longer edit using src\fsharp.sln because of this unless I unload the "tests" projects, so it might be best to more these to a separate solution


line duplication


Indeed, badly moved by Intellij


Done.


Tnx Mero :)  Im happy to see you here!

Im trying to add this to the webinterface and let'u know.

Regards


OOC, is there a reason to avoid `hasattr`?

Also, at that point, isn't
```
assertRaisesRegexp = getattr(self, 'assertRaisesRegex', self.assertRaisesRegexp)
```
equally efficient?


心好累，这个bug有点二。。。。


Private setter?


That was quick :)


I am fine with this since it is simplest.  And the definition of this flag, is it will not run any container images with image volumes defined.


ACK.


I think we should consider this, probably with WCAG WG. Maybe we don't need to have that sentence before every list?

This is related to the important info from the existing intro, that is missing in this draft:
"
About the Techniques
For important information about the techniques, please see the Understanding Techniques for WCAG Success Criteria section of Understanding WCAG 2.0.
Note: The basis for determining conformance to WCAG 2.0 is the success criteria, not the techniques. (The success criteria have 3-level numbering (0.0.0) and in this page they are followed by a link "Understanding Success Criterion".) All techniques are informative; that means they are not required. There may be other techniques besides the ones listed here.
"

What is the least cluttering, yet clear, way to get this message across?

Straw proposal 1:
* Add to the About section:  All techniques are informative; they are not required. Please read important information in the Understanding Techniques for WCAG Success Criteria section of Understanding WCAG 2.0.
* Not include "Note: Other techniques may also be sufficient if they meet the success criterion." with every listing.
Con: If people don't open the About section, they won't see it.
Pro: Cuts clutter of the note all over.

Straw proposal 2:
* Before all techniques (under or after "Hide techniques and…" button): Put "All techniques are informative; they are not required. See Understanding Techniques for WCAG Success Criteria.
Pro: It's all over so can't miss it. It links to the important info.
Con: It's all over so cluttering.
Possible con: It doesn't specifically say sufficient.
Straw proposal 2a:
* Nothing in the About section.
Straw proposal 2b:
* Add to the About section:  All techniques are informative; they are not required. Please read important information in the Understanding Techniques for WCAG Success Criteria section of Understanding WCAG 2.0.


Thanks!


Remove `IF NOT EXISTS`. In case the user already has a `sessions` table, he should handle the error of not being able to create another table with the same name.


I was looking at this to understand #925, and I really don't understand why this revert is so big. The one commit it is supposed to revert just changed two lines, but this revert is massive, touching lots of other bits of code, reverting among other things 0c3248fa6844ae2181ab2a39a64848526c00e65c.

I thought I kind of knew `git`, but I am not too familiar with reverts. This altogether looks like it bit off a bit more than intended.


That token doesn't work anymore, don't bother.


This is missing a closing ] after line 63, and the comma on line 63 is unnecessary


dafuk


We can confirm the record with ``leo_s3_bucket:find_all/0``
```erlang
leo_s3_bucket:find_all().
{ok,[{bucket_2,<<"test">>,<<"05236">>,
               [{bucket_acl_info,<<"05236">>,[full_control]}],
               leofs_1,erasure_code,undefined,undefined,0,63616373723,
               63616373736,false}]}
```


Fixed


Я ж не растягиваю исходный мокап, а только сужаю скриншот с логином и фоном.
Вот мои шаги:
1. Беру оригинал найденного мокапа
2. Подрезаю его немного для того, чтобы он подходил под требуемые размеры odoo (чтоб влезал в тень)
3. Беру слой, а точнее скриншот со страницей авторизации и фоном.
Подгоняю его под размеры экрана мокапа, к примеру wmctrl -r " - Mozilla Firefox" -e 0,0,0,1440,1037
Беру слой в полтора раза больше размера экрана мокапа, чтобы сохранить пропорции.
4. Затем сужаю этот слой до размера экрана мокапа
И получаю картинку, которая показана выше.

"А ты берешь картинку шириной 555 и растягиваешь до 627."
Я беру изначально мокап с размером 627, а картинка с шириной 555 (слева) - это ж другой мокап
Или я чего то не допонял)


@jollopre actually it should be the other way around, adapt the code to the style not the style to the code.

With the other engineers of the FieldtripGB we have an agreement to follow a slightly modified version of the airbnb [coding styling guidelines](https://github.com/airbnb/javascript) for javascript with two modifications tab size in spaces of 4 and else statements starting in a new line.

It's my fault not have documented this here but my though was that the defaults generated for the wizard (with tab size 2) were convenient for this project, and still I think that, I wouldn't change the max length of the line to other than 80 as is a the most common (but for most of the scripting languages).


@lelit hasattr has awful exception-swallowing behavior until Python 3.2, including all Python 2 versions.

What you suggested seems reasonable to me though.


Yes, I suspected that this case does not allow for unsorted children of different types, its just connections on the first level and services on the second. Can it be that the first connection has 0 services? Then there will be no default service. Could it make sense to look for the first connection that has at least one service? If not, and the logic is as planned, maybe it is better to code it unambiguously with
...
ObservableTreeItem item = items.get(0);
...
instead of formal iteration that is not there.


Komt rechtstreeks uit dat hashicorp script, kan geen kwaad dat het er in staat want wget wordt wel gebruikt.


I'm not sure where this note ended up, but this effectively makes this story dependent on the first story from the `android/compile-dependencies/built-from-java-sources` feature. https://github.com/gradle/features/blame/master/android/compile-dependencies/built-from-java-sources/README.md#L7


nauka Noda.js i MySQL - losowanie świąteczne


On Thu, Dec 03, 2015 at 06:58:01AM -0800, Emanuele Gaifas wrote:
> OOC, is there a reason to avoid `hasattr`?
hasattr() catches all exceptions, not just AttributeError.


yeah, done...


Hi Shawn, I will just put the comment to issue #76, it might get overlooked here on the commit.


Thanks, good to know.


Could we add a json_result.values["status"] = "" explanation on why status 400 is returned ?


@zugaldia The target is a good idea, but it should be initially set to DC so that the Map starts there.  The thinking is to be overly obvious that the `animateCamera()` API has worked as it'll move from one city to another.


I guess it's a bug isCvtOpCode method.
  return ((unsigned)OpCode >= OpConvertFToU &&
      (unsigned)OpCode <= OpBitcast) ||
      OpCode == OpSatConvertSToU ||
      OpCode == OpSatConvertUToS;

OpBitcast doesn't convert types.


Could you look at this - is the exit function supposed to be here?


Ok, thanks.


Thats a big chunk of commented out code 0_o


Mwah, liever niet, als het niet echt nodig is... is weer minder code om te onderhouden


Yes lsb_release could become obsolete, if it proofs to work fine of course.
The postinst will run every time you (re-)install rear package, and
therefore, the os.conf would get recreated at every installation or upgrade.

On Thu, Dec 3, 2015 at 3:02 PM, Schlomo Schapiro <notifications@github.com>
wrote:

> Also, is this file removed upon uninstall?
>
> How do you update this file if the user does a dist upgrade? If we cannot
> guarantee that then I would prefer to keep the old way. Better than having
> users who installed ReaR on Debian 7, upgraded to Debian 8 and ReaR never
> noticed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rear/rear/commit/613ba6f379e0ad93e47f707f1ae351d781008732#commitcomment-14762481>
> .
>


non mi piace molto


Am I right in thinking that this is meant to fix #24?


Since argc will always be at least 1, you can simplify this to a ```argv++```. The comparison is irrelevant.


What's the difference between a `u_char` and an `unsigned char`? Seems like they're the same thing.


No problem, thanks for the fix.


remove debug fmt.printlns


these should be
let(:new_user) { build(:user) }


without alias sqlalchemy is creating name in that kind of way: ```table_name.column_name``` but it causes syntax error because we have used alias before and we aren't allowed to mix calling column by alias and by table name.


Imaginei o dev se perguntando "O que é essa querystring workspace=sb_email?"


I agree that you should raise an error, I'm just nitpicking about the kind of exception :)
I also agree that FileNotFound is maybe too mild as indeed this may be caught too easily.
I would probably go for a `RuntimeError` or an `EnvironmentError` (https://docs.python.org/3/library/exceptions.html) with a descriptive error message.


ACK.


are all of your tests passing?


dit maakt de .bash_aliases file echt leeg hè - is het misschien nog te doen om een specifieke line met een bepaalde string ('alias puppet-update') te vervangen door de nieuwe versie?


I found this caused an error when I next tried to boot my VM:

```
$ sudo apachectl restart
Syntax error on line 17 of /data/twfy/conf/httpd.conf:
Invalid command 'ExpiresActive', perhaps misspelled or defined by a module not included in the server configuration
Action 'restart' failed.
The Apache error log may have more information.
```

The fix was to enable the `expires` module by hand:

```
ln -s /etc/apache2/mods-available/expires.load /etc/apache2/mods-enabled/
```

@dracos – Maybe my VM is really out of date, but we should check whether we need to enable this module as part of the Vagrant provisioning step, or whether it's already enabled.


The ARM platforms changes for GICv3 changes have a dependency that TSP should be able to handle GICv3 interrupt model. Hence the TSP/TSPD changes need to be present prior to the platform changes.


Quando for introduzir o conceito de ambiente de desenvolvimento é importante enfatizar que **o ambiente de desenvolvimento não afeta NUNCA produção**.


makes sense.  It might be clearer to pass an alias like ```'renamed_table'``` than ```'x'```


Думав про це, але вагових аргументів не знайшов, тому й в дао не додав.


waarom is deze nu later gedaan? dit stond eerst bovenaan...


Acho que ficou confusa essa parte.
Os conceitos de **Área** e **Modo de Edição** que são muito importantes.
Poderia ter uma sessão sobre Plug&Play para falar sobre áreas e temas mais flexíveis. Talvez outra só sobre o modo de exibição.


weet jij wat dit betekent?


No underscore


Try editing classes/Form/Section.class.php line 84 so that $bodyclass contains collapse out (instead of in) and you will see what I mean. Something of a mystery ATM


(3) reminds me: I changed `err` to `error` because I don't think it makes sense to abbreviate the (rare) `error` but not the more common but equally long `debug`.


Dat er verder geen config mogelijkheden zijn na deze comment


便利ナビの設定方法
http://qiita.com/sa-wa-m/items/debca89be67aaf86bc0b


Gostei dessas imagens! Acho que vai ajudar bastante os desenvolvedores a compreenderem o conceito.


合并


my solution is not slow, but i have really few extensions on this vs instance
I'll send a pr to remove from project the bat/script nodes
I'll add the folders inside the project, to make it easier to open the directory


Don't we already have an inordinate amount of Energy resists?


ACK.


yep but then I need to change this in a couple other places.


this could fail


I have copied it from deepmind repo.
On Thu, Dec 3, 2015 at 9:53 AM Alfredo Canziani <notifications@github.com>
wrote:

> Well, in for-loop, of course... I don't see / understand the necessity of
> all this, instead of
>
> for i = 1, T do table.insert(clones, model:clone(params, gradParams) end
>
> Did you write this chunk of code, or did you borrow it from somewhere else?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wojciechz/learning_to_execute/commit/04a1659e471233ddb81280d9d98113286c1c459d#commitcomment-14763880>
> .
>


Laat het liever wel staan, maakt de diff met het hashicorp script overzichtelijker.


Omdat dit ons custom stukje is, alle code boven de "Leukeleu custom bits comment" komt rechtstreeks uit het hashicorp script. Heb dit dus verplaatst om de diff met dat script overzichtelijk te houden.


@mjeffe98 you accidentally committed some local changes to VagrantProvision.sh.  Can you revert them?


Shouldn't this be?:

    %%GCM%%@sample %%ETCDIR%%.d/charon/gcm.conf.sample


Thanks :) I am grateful to have a positive response from you all and kind support.

@VoyagerOne yes true, there is a room for further tuning of Weight[Threat] more, because by looking at the SPSA graph, one can predict that the slope is still there and increasing, rest of them are quite on the line and stagnant.


Чем потребность в этом методе - не весомый аргумент? А вот getAllUsers на самом деле вроде как  не нужен. Он создавался как и остальные такого плана за компанию.


OK. I fixed it.


Shouldn't this be subclassing `WMFModalImageGalleryViewController`?


This works fine on 2.8


Right, I will take care of this in the next 12h.


For upstream changes at: https://github.com/bitcoin/secp256k1/commit/9234391ed4d5fae38921a8bb4ca36f9f33736fd7


Dat is wel lastig. Op zich zetten we zelf geen andere aliases in voor de leukeleu user, zo ver ik kan nagaan.


nope.

u need to fix one failing test.

UsersController Post create logs the user in after sign up
     Failure/Error: expect(session[:user_id]).to eq assigns(:user).id

       expected: 1
            got: nil

       (compared using ==)


Then you will get an error.


On 03-12-15 14:53, Peter Frühberger wrote:
> via @opdenkamp <https://github.com/opdenkamp> :-)
and via trent and scott ;-)


I just replaced **..** to **.**, is it make sense?


ACK.


Takk


@brianhatchl I don't know how that happened since it wasn't added to my commit. I'll revert.


:D


Fixed.

--Alex

On Thu, Dec 3, 2015 at 7:53 AM, Greg Von Kuster <notifications@github.com>
wrote:

> Have you seen this one?
>
> https://github.com/gregvonkuster/galaxy-csg/pull/18/files# <
> https://github.com/gregvonkuster/galaxy-csg/pull/18/files#>
>
>
> > On Dec 3, 2015, at 9:45 AM, Alex Pletzer <notifications@github.com>
> wrote:
> >
> > Greg,
> >
> > Ok, no worries. I added these options. I think the changes you required
> > have been made. Let me know if something else comes up.
> >
> > Thanks,
> >
> > --Alex
> >
> > On Thu, Dec 3, 2015 at 7:33 AM, Greg Von Kuster <
> notifications@github.com>
> > wrote:
> >
> > > Yes, we need these. Galaxy stores all files with a .dat extension, so
> > > looking at the file system to determine the file format is not
> possible.
> > > This Cheetah template is not calling the shape manager. It is calling
> the
> > > underlying Python script with calls the shape manager. So unless the
> > > underlying Python script has this information, it cannot pass it on to
> the
> > > shape manger.
> > >
> > > Also, writing code that depends on subdirectory names or file names and
> > > extensions on a file system is non-optimal, and generally results in
> > > brittle code. I continually deal with this issue in the labs. Our code
> > > should not depend on any file system path names or extensions if at all
> > > possible.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763258
> >
> > > .
> > >
> >
> >
> >
> > --
> > Alexander Pletzer
> > +1 609 799 7999 cell
> > alexander@gokliya.net
> > —
> > Reply to this email directly or view it on GitHub <
> https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763628
> >.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pletzer/galaxy-csg/commit/e3e8cb57f882b036138d63ac8201b7fe01d1c853#commitcomment-14763863>
> .
>

--
Alexander Pletzer
+1 609 799 7999 cell
alexander@gokliya.net


re evaluate


The scan failed.
Included commits

Commit ID	Owner

2d69727498bf905d5d9abd48b311b014d392725b	@dudug


:-) No. I guess I wasn't clear. I also wanted to get rid of the "... is documented as: ..." everywhere. Don't worry, I can also fix this before I review and merge the PR.


This commit breaks the testsuite, because the testsuite is wrong and needs to be updated.  Please, fold the changes to the testsuite in this commit so that it passes.


Yes, I think we should celebrate it by listening to a story:
https://www.youtube.com/watch?v=LS-ErOKpO4E


The example isn't in lingo, so it seemed reasonable. I'm slowly migrating
headers to use <> with local source files using "".

Andrew Sutton

On Wed, Dec 2, 2015 at 10:52 PM, Jennifer Yao <notifications@github.com>
wrote:

> Sigh. I thought you said you *didn't* want angle-bracketed #include paths
> in source files.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/asutton/lingo/commit/c5620259f8e3c437be7ceb5fb3cdd3afb12ba619#commitcomment-14752693>
> .
>


Ah, I wasn't familiar with `select` so at a glance it appeared like you had to type out the whole thing. Nevermind :stuck_out_tongue:


If it's not intended to be a new inactive text style ( layout/text.less -> `.text-inactive` ) you should use a different variable name; something like `filter-text-muted` as it's a filter specific version of text-muted.


I like this change, but I think you should add one more unit test to cover the case mentioned in your PR description.  [This test](https://github.com/geome/bottlejs/commit/778692b6664360ecae05e547a7366b24c4f95557#diff-b74f1889f15a5468e71ec64cda487086R72) only executes the provider's factory once; you should also test that the thrower still throws in a subsequent service lookup before describing the positive case.

Something like

```js
it('throws the error', function() {
  // ...
});
it('continues to throw the error for subsequent requests', function() {
  // ...
});
```


Add a comment here referring to the example in interval_sum at least.  That way, when we look at this thing in the future we won't scratch our heads thinking "why 8 and 8?"


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


If you still want to work on it, also make sure the godoc wants the name of the func as the first word in the documentation header. E.g. if the function is called `Index`, then the documentation should start with `Index`. See [here](http://blog.golang.org/godoc-documenting-go-code).


ACK.


The compiler is not tricked; its checker boarding (score carding) will correctly recognize that the variable is unused and not waste registers or anything else for it.  Declaring (void)x is one of several ways to tell the compiler that "yes, I know it is an unused variable".  And that method works ever so slightly better than a couple of other of those techniques with the static analysis tool I use, Parfait.  I've not removed them entirely since either they are someone else's code which I want them to consider (e.g., auto-leveling), or I'm still considering the matter.

And yes, there's times you simply leave them rather than remove them such as callbacks or generic routines which may be called via pointers from other routines who do not not the particulars about the routines being called (i.e., don't know that in a specific instance a call argument isn't used).


왜 2번째인가요?


무슨 스터디인가요?


Ось `getAllUsers` знадобився, а створювати новий метод в дао задял використання в одному методі, не знаю. Хоча звісно це спірне питання )


Versão Beta: 1.0.0


Por enquanto, isso aqui morreu né?


You know, I actually had that in there and removed it :)

I'll add it back.


Где кроме этого места он понадобился еще?


You should unwrap `retweet` safely like this:
`if let retweet = tweet.retweet {`
Then, just use `retweet` instead of `tweet.retweet!`. It's unsafe when unwrapping like that.


I'm confused. The SO question/answer is about the terminating newline at the end of a file. These are blank lines in the middle of the file. We have no rules in any style guides about whitespace between elements in HTML or XML files.


Just have a try!


전반적인 내용은 여기에서


Yes,I do


We lost a *lot* of stuff here @sethetter - we probably want most of that in here unless there's a reason to remove it.


@bqpd, please confirm accuracy


What is going on here with $earlierVersion? Why is it needed? Currently (at least for me), it is failing a test where if a migration date is before all the migrations, all the migrations are ran. Just as if no date or a future date is used.

I think this would be better if the loop only worked out the version to migrate to and then call the migrate() method once we know we've got one.

```php
    public function migrateToDateTime($environment, \DateTime $dateTime)
    {
        $versions   = array_keys($this->getMigrations());
        $dateString = $dateTime->format('Ymdhis');

        $outstandingMigrations = array_filter($versions, function($version) use($dateString){
            return $version < $dateString;
        });

        if (count($outstandingMigrations) > 0) {
            $this->getOutput()->writeln('Migrating to version ' . max($outstandingMigrations));
            $this->migrate($environment, max($outstandingMigrations));
        }
    }
```


Do you definitely want to kill this section entirely?


ACK.


Indeed. Additionally, for the line below, we should just truncate the text to 180 chars or something, rather than replacing it entirely.


Also, this isn't a new line that @kswedberg created. His editor simply fixed (removed) the trailing whitespace on the existing line.


this is nonsense, sorry


@rpudie: what triggers calls to the next() method of a scheduler?

I noticed that it seems to get called regularly, even if no task has terminated, and I was not sure whether that was normal or a result of returning None when it wasn't expected by the caller.


Fixed in 9b25bbc8ee0f4654ba580224b074a8affb59dfe3


:+1: for this, I noticed it yesterday when trying to figure out how to fix #9.


SHould this be merged into the volumes patch?


From line 53 to 59, you can write only 1 line like this:
```Swift
retweetButton.setImage(UIImage(named: tweet.isRetweeted! ? "ActiveRetweet" "RetweetIcon"), forState: .Normal)
```


Yup!


Uploaded!


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


Cenno


Also, I just realised I made the PR with the wrong branch (the one I made with `/dist` on it). I'll close and reopen.


It should, something changed in the latest master docs and cache wasn't working anymore in docker build -v


Another option might be to have these as READMEs with the samples themselves.


should this be --no-image-volume?


If your code is not in closure or something like that, please remove unnecessary `self`.


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


We basically check the same thing twice here, but they are two different code configurations - one for concurrent mark, the other for the finishing pause.  Please add a comment to that effect.


ACK.


Tu pourrais expliquer la différence entre les 2 types d'héritage si c'est possible de le faire brièvement. Sinon, juste mettre un lien serait un début!


right, thanks for catching this @rhvgoyal


Just using simplenn then instead of the one we built?


(Oh, I didn't -- you just commented in this branch. Never mind!).


Well,I noticed you are finding the way to solve wifi problem.A commit in Linux 3.10.72 causes that.(https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/fs/proc?h=linux-3.10.y&id=cf6c05a77c5ef41bd72f45a4a008724ffd393668).You should revert it to get samsung's wifi module working.BTW,I will send my Hangouts to you later in XDA-Developers Forums.


updated


Will it make sense to call it NoImageVolume. We are only blocking image volumes and not rest of the volume types.


為什麼你要打這個東西


What I meant was that you don't have to call the same function twice. You can initialize m_user based on the result from the first call. So if m_forwardedFor.empty() then m_user is initialized empty, otherwise identity.second.


it does, in an older commit. maybe i should've done the [renaming](https://github.com/wikimedia/wikipedia-ios/commit/ca650ebbd9e153de3e3a14cd2327796ee25361b0) along w/ this change, but decided to separate them out at the time


Concern: You don't *do* anything with ```h_odata```. I'm thus worried that the compiler will declare that cudaMemcpy dead code, and eliminate it. And then eliminate the kernel call too. You should actually check that ```h_odata``` is correct and print out an error if not. Then it will not be dead code. I am not *sure* it will be eliminated, but best to make sure it is not.


Please follow coding convention. `[String: String]`, not `[String : String]`. Don't put white space before `:`.


right, leftover from the previous patch version, thanks!


DENIED


ありがとうございます。
酷いバグになってます。

さらに、今のロジックだと機能OFFの変更を受け付けないですね・・。
直します。


Please make an inline function for this condition, `evacuate_block_obj_sizes [size_index] && !block->has_pinned && !block->is_to_space`.


OK, that would have been another option, provided `attr_free()` is used consistently. At the time you replied, however, I had already implemented the extra struct members. They provide pretty much the same mechanism we've had up to now and are least regression-prone.

However, I've noticed that c04b9f5 has another issue: the location methods are inaccessible from the vehicle modules. On Linux the library for the plugin fails to load due to an unsatisfied reference, and as a consequence `navit.vehicle` is null and no location is ever updated. On Android I didn't see the error message but `navit.vehicle` is null and the location on the map never updates. However, I do seed the location update callback getting called. On Windows the build already fails due to unsatisfied dependencies (see CircleCI at https://circleci.com/gh/mvglasow/navit/7)

What do I need to change so that the vehicle modules can call location methods?


你還好意思說= =


Yes, I think.  The `--developer` option to the install script is meant to subsume this.


搞什麼東西!!!!!!


XD


rename to template_dir to keep it in sync with other names


Looks like unnecessary blank line. Don't put a blank lines after `{` and before `}`, the prefered format should be
```Swift
someBlock {
  someOtherBlock { // no blank line here
  }
  // blank line here is okay
  someVar = someMoreCode
  ...
  // blank line here is okay
  someFinalCleanup() // no blank line at end of block either
}
```
In short, in a block, no blank line right after the start `{` and right before the `}`. Blank lines are welcome anywhere else in the body though. Don't use double lines to separate your code.


I meant the `firedrake-clean` part


Nope, this was a mistake. The old `.htaccess` must not have been pulled over when I copied over everything.

Fixed in b89827006334a81f6835e3c299cf76f90bfcd60a


fixed


teste


Delete


you can do this in one line download_template(check_loch_axs())


Oh, I guess that appears in the `firedrake-install` section but could still be repeated here.


How did you manage to get activate_consoles back in? There were none when I left you


Pas besoin des guillemets.


Thanks @stanistan
https://github.com/etsy/phan/commit/f065be33f0f21b1b834a7b6fab2a5ef4a00441df


Please add a comment explaining why the `sgen_concurrent_collection_in_progress ()` is here.  Basically we don't want to evacuate objects allocated during concurrent collections.


Be sure to test on a system with IPv6 disabled. In the no-IPv6 case it would be safe to issue warnings and then ignore listeners/connectors configured for IPv6. In the self test case you must detect if IPv6 is enabled and run IPv6 tests only if it is enabled.

It is a safe bet that IPv4 is always enabled. Down the road there may be some IPv6-only system but I've never seen one. Current Linux has issues when IPv4 is missing.


Thanks for good information. I haven't used godoc by myself yet, so I would like you to fix the documentation. After that, I will learn how to do it. :smile:


Why?


Ah, I hadn't noticed; probably no need to repeat...


Problema try-catch risolto


*Now player can only get 1 meter each hit


I'd add an additional check to see if VPOOL_NAME is not empty otherwise you'll end up wiping the complete /mnt/* directory ...


Okay. Will do. No worries.


It is wierd, that one has to do this *(line 18 - 23)*... find out why it is working the way it does!


no actually i have to re-test .. i saw that my change got unmerged may br that's y


Please unwrap variable like this:
```Swift
if let dictionary = dictionary["retweeted_status"] as? NSDictionary {
  retweet = Tweet(dictionary: dictionary)
} else {
  retweet = nil
}
```


Better :+1:


Oh... Yes that was a silly misstake ;-)
I'll generate a new version tomorrow.

Thanks!


This should be calculated as the number of corpus profiles.


I'm still curious why this is not needed on my system. I'm going to merge just in case other people might have this same issue.