It's not clear to me why this is needed


Nor why we loose this


Well this is pretty stupid, I just thought about this solution, this put all the files we don't want produced during node compilation and not just .css (I could edit extractTextPlugin instance but people will probably report the same behavior for others assets someday).


And one more comment:
After looking at it again I think the current behavior is OK because we only add the Query Parameters if the *serverside* added them.
Compare
```
<f:form action="filter" method="get">
  <f:form.hidden name="term" value="foo" />
</f:form>
```

With:

```
<f:link.action action="filter" arguments="{term: 'foo'}" />
```

In the first example a route is resolved to the `filter` action. The "term" query param will be added by the browser.

In the second example though, a route is resolved to `{"@action": "filter", "term": "foo"}`, creating that cache entry.
We should maybe make this distinction more explicit by adding a `queryParameters` argument to the Uri/Link ViewHelpers that allow to append query arguments *after* the routing


Oh, right!


Nice! Before this patch, I could use also "null" string to bypass the "@password@" validation in configuration action.


Shouldn't everything that's part of this assignment be indented?


why not use `/boot/config/installer/rcS`?
That way, when/if we have more customizations for the installer, they'll be nicely grouped together


Why do we only increment this number?


2nd parameter should be const std::string& as well, see 9dc7dd50d7538e819ad0baacac2e5ea6501d8e97


to open API for Event : please redirect your browser to 'localhost:8000/reports/api'


It's staying in, but who puts the license in the README when there's a LICENSE file also?


please remove commented code


I'm not sure which exactly asset is the problem. I check that two slices have the same length and column indexes are valid.


Excellent :)


unneeded parentheses


insert_before and insert_after variants? I guess we might consider then later.


Hi,

I hope you received the notifications. Please fix two small problems - with parsing the default value and change noting to nothing, and I'll merge your request.

In general, good job!

Thank you,
Andrey.


I don't think you need to conjugate `eig.second`.
The unitary matrix is made of **columns**, which are the right eigenvectors of `h_matrix`. `eigenelements()` returns these eigenvectors as **rows**. That's why we additionally transpose the result.


To further improve this test, one is likely to add

```ruby
expect(book.entries.size).to eq(1)
```

right before calling `remove_entry`.

One may also use RSpec's [change matchers](https://www.relishapp.com/rspec/rspec-expectations/v/2-0/docs/matchers/expect-change):

```ruby
expect { book.remove_entry(…) }.to change { book.entries.size }.from(1).to(0)
```


It is worth noting that, after this change, xed will accept both ctrl-shift-z and ctrl-y for redo.


wouldn't we want some of those to be true? Are those the defaults from JSHint?


if current_user.nil?


`return if current_user.present? && token_valid?(token)`


:+1: good job


Add **Gitter**


ci:retest


Fix: https://github.com/lateminer/DopeCoinV3/commit/c2483518f014002d94cf0258283e3c9ae0785185


libio was probably copy & past from some btrfs code


I think it would allow creating props via api, should propser_id be taken from current_user?


Thank you. I added your commit to my `rc1` branch. Please give it a try.


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


we want to redirect api/users to api/v1/users.
same with api/rankings and api/props


Where did the red line go?


Code style: The rest of ScriptRunner has else on a newline (https://en.wikipedia.org/wiki/Indent_style#Variant:_Stroustrup)


For now it serves only for constraints for long variables.
So I renamed this to `LongVariable`.
But maybe you are right.
And it is better to have single annotation which eventually is used for both long an type variables.
I will change it.


@DavidLemayian Thanks!


[R/get_average_nltt_matrix.R:13:3:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt_matrix.R#L13) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:51:28:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt_matrix.R#L51) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
    nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                           ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:57:23:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt_matrix.R#L57) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
    stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                      ^~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:79:9:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt_matrix.R#L79) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  xy <- get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:70:30:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt.R#L70) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
      nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                             ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:76:25:](https://github.com/richelbilderbeek/ribir/blob/73312f5e5d91e7497690575ffacb1f23ee84d145/R/get_average_nltt.R#L76) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
      stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                        ^~~~~~~~~~~~~~~~~~~
```


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


We will want to highlight sequences throughout the gene as restriction sites can be useful even if they are not in the 3' and 5' regions. e.g. for checking your cloning process has worked by doing a restriction digest that would only cut if the gene you inserted in your plasmid contains a cut site for a restriction enzyme.

however we will want to to check for both orientations of the restriction site e.g.
5'-ATGCATGC-3' and 5'-GCATGCAT-3' which its reverse transcription (each character is swapped for it pairing partner e.g. A->T and the order of the characters is reversed) as we only that the sequence of the 5'->3' strand.

because in the 2 DNA strands  it looks like this:
5'-ATGCATGC-3'
3'-TACGTACG-5'


This was done to support the following commit, but on further testing, I think this code is not quite right. This commit has been removed, and the needed code added to the following commit.


Hello git


Is this related to this PR? (the commit message "further tweaks to z-index" isn't very enlightening)


A TabIndex should not be necessary here. The browser will determine the proper tab order based on the order of the markup.


This is really a good catch.

Kindly,

Runhang

> On Mar 8, 2016, at 20:15, pascal-cuoq <notifications@github.com> wrote:
>
> Int64.bits_of_float f must be masked with 0x800fffffffffffff here, or instead, reconstruct_nan should have been Int64.logor 0x7ff0000000000000 in place of Int64.logand…
>
> —
> Reply to this email directly or view it on GitHub.
>


you can curry them. Since the current version of the production code does not do that anymore, I can change that.


V cool - stickers needed :)


done


db/schema.rb , la suppression de 'name', vient de Moul , cela posait un pb lors du lancement de rails s.


Duplicate test case name?


I think we need an extender version of user show endpoint.
Currently it only returns only props received and given, it should also return a user name - it will be needed in frontend.


Difficult one. More choice = less consistency.  Personally, I hate having a ton of windows with their own styling rules.

OSX, you create a window with a custom titlebar and it sticks out like a sore thumb.  Plus the window controls were on the wrong side.

IMO, the happy medium would be:

- Choice between native & custom titlebars
- Option to use native colouring, or match titlebar colour with window colour (I know that's supported on OSX, not sure about Windows/Linux).

Not sure how much of that Electron supports though.


The problem is: it hurt my eyes. Now, seriously, besides that, it's completely non standard IMHO, even for tools that use colored output, syntax highlighting, etc.

Let's leave it out for now, and come back to it later.


trims it's values ---> trims **its** values ;)


`Returns`


should be SIGN


You should avoid using inline styles. A new class in the CSS is almost always the better option. The only time inline styles are appropriate is when you are building a Javascript component and you need to override the style inside the logic of the component (i.e. via Javascript)


you are right. Current decomposition is a refactoring artifact.


@grdryn I'm not sure on that one. This was based on templates in this repo https://github.com/openshift/origin-aggregated-logging/blob/master/deployment/deployer.yaml#L10
It could help as an extra selector to filter things when running 'oc' commands.


Perhaps just make the options, well, optional. Adding `options = options || {}` before the line 47 should do it.


same here


`object.propser_id != current_user.id && !user_has_upvoted?`


changed


## 2nd heading

###3rd heading


Nope, it just has to be in `PYTHONPATH`.

To validate this create a file `foo.py` with the following content:

```python
def load_ipython_extension(ip):
    print("loading")
```

then launch IPython in the directory containing `foo.py`

```
$ PYTHONPATH=. ipython
```

and then do

```
In [1]: %load_ext foo
loading
```


### 3rd heading


insert_float is replaced by insert_float_in_bounds, which is indeed dangerous to use. I made the error you said when writing code. I need to make sure again on each call site that abstract float is not freshly allocated.

Kindly,

Runhang

> On Mar 8, 2016, at 20:12, pascal-cuoq <notifications@github.com> wrote:
>
> I don't think you are using this function. But I think it's a dangerous function. If the programmer wants an abstract_float that contains f, the programmer can request that from the start with inject_float, not make an uninitialized abstract_float and then change it. And insert_float is dangerous to use on any abstract_float that is not currently uninitialized.
>
> —
> Reply to this email directly or view it on GitHub.
>


[R/get_average_nltt.R:76:1:](https://github.com/richelbilderbeek/ribir/blob/6bb3d32f97a5eda7b0e6fcde17049ed4c5712552/R/get_average_nltt.R#L76) *style:* **lines should not be more than 80 characters.**
```r
      stretch_matrix <- ribir::stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


needless empty line


[R/get_average_nltt_matrix.R:57:1:](https://github.com/richelbilderbeek/ribir/blob/35d429dd16613110010b46e80ce33a911da4ca1f/R/get_average_nltt_matrix.R#L57) *style:* **lines should not be more than 80 characters.**
```r
    stretch_matrix <- ribir::stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:79:1:](https://github.com/richelbilderbeek/ribir/blob/35d429dd16613110010b46e80ce33a911da4ca1f/R/get_average_nltt_matrix.R#L79) *style:* **lines should not be more than 80 characters.**
```r
  xy <- ribir::get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:76:1:](https://github.com/richelbilderbeek/ribir/blob/35d429dd16613110010b46e80ce33a911da4ca1f/R/get_average_nltt.R#L76) *style:* **lines should not be more than 80 characters.**
```r
      stretch_matrix <- ribir::stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


@kostecky Can you explain why ignoring the root volume is of use (and also if you find the check generally useful in itself)


Shouldn't this return an array of start positions and an array of end positions in the gene sequence to be given to @DolapoA 's layer where he adds the HTML?


*1
*2
*3


You should generally avoid id selectors in CSS (i.e. #foo) and use a class instead. ID selectors have a very high precedence value (the cascading part of CSS) and so are very difficult to override when that is necessary.


Removing getName() in 2.8 causes the error

> The type name specified for the service "sc_datetimepicker.form.type.datetime" does not match the actual name. Expected "collot_datetime", given "SC\DatetimepickerBundle\Form\Type\DatetimeType"

As a quick patch on our system I've include;
`    public function getName()
    {
        return $this->getBlockPrefix();
    }
`


It shouldn't be a set-word.


More inline styles.


The list hasn't been updated for a long time. But yes , its available.
You can see my build log for confirmation : https://s3.amazonaws.com/archive.travis-ci.org/jobs/112403527/log.txt
Or you can see build history of my forked master branch at : https://travis-ci.org/opticod/BuildmLearn-Toolkit-Android/builds/112403526

I have set the configuration which is most suitable for error-less builds...As Travis -CI is still in beta phase.


Both comments make perfect sense. Not sure why I wrote it in such weird way :)


This is a test for commit.


There's a critical problem with this default: it won't parse as a number!

Can you fix this and remove the property override from our testing, so that the defaults actually get tested?


[R/get_average_nltt_matrix.R:57:1:](https://github.com/richelbilderbeek/ribir/blob/43474395140e0ac3e3422160b1028719483b85a5/R/get_average_nltt_matrix.R#L57) *style:* **lines should not be more than 80 characters.**
```r
    stretch_matrix <- ribir::stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:79:1:](https://github.com/richelbilderbeek/ribir/blob/43474395140e0ac3e3422160b1028719483b85a5/R/get_average_nltt_matrix.R#L79) *style:* **lines should not be more than 80 characters.**
```r
  xy <- ribir::get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```


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


How should we store pyplayers in memory? Suggestions?


Interesting and surprising!


Nice change, now it's just perfect for TimeLapse, thanks a lot!


Thanks Martin, nice that you managed to fix it.

I have jus had similar case in Operation/Ticket/TicketCreata.t test, and I thought $SysConfigObject->ConfigItemUpdate wasn't necessary, however without that it doesn't work.
You will see soon, I work on this tests now.

Regards
Zoran


inlined


done


This is not a race, this is caused by the fact that hash tables on Ruby 1.8 are not ordered.


yeah - squashing


User must have admin permissions to access these.


Use lowercase, like all the other plugin names


:D Wouldn't that be easier by creating a separate (extension) list and then inserting a slice once at the end?

Like in:

    x = [1,2,3,4] # the base list

   extension = [] # the extension list

    # Build the extension list
   extension.append(5)
   extension.append(6)
   extension.append(7)

    # Finally, insert the extension
   x[1:1] = extension


do not MERGE when you are doing a pullreq


This is my first git repository.


this can be perfectly squashed


squash this. a commit by itself should be atomic and consistent


As for Unity, pretty sure it still has the controls on left - but it's configurable.

There's almost no consistency across DEs, and even across different installations - which is why one-size-fits-all won't work here.

No such thing as "native colouring" on Linux, afaik - you can sort of do something in GTK3 with their Client Side Decoration, but it looks absolutely awful outside Gnome.

I'd vote for choice - if it's possible. Apart for the CSS, it should be straightforward. That could be worked around with some added class maybe?


It might be easier to use the wysiwyd-technical mailing list .. :)


Maybe - we've never used that pattern before and this one that we have used elsewhere works...I have moved on & don't feel like coming back to change this (& the other instances of the pattern)


yes, it is not about removing/replacing the maps, just about better encapsulating them and thus being able to better control access to them by means of a dedicated interface.


`anchor` as param is kind of String, that's ok. But inside class it's store as String and used as Time. Could we use `_anchor_at` method like we do in `StrictPeriod`?


Tamamdır


It doesn't need to be a number, that's the syntax expected by ES. You also could do something like "10s".

I didn't expose this syntax and rather am expecting the value in ms there to be inline with other timeout settings we expose (see below how the value is assembled if it's not the default).


How can I do that?


Like I showed previously:

```Rust
columns = &[2, 3];
values = &[&1, &2];

for &column in columns {
    assert!(Value::type_transformable(values[column as usize].to_value_type(), type_));
    // so here, you try to access to values[2], which doesn't exist
}
```

And in the next lap, you'll try to get `values[3]` which doesn't exist either.


r = ((1<<(b+1))-1) - ((1<<(a+1))-1);
It might be possible to simplify it further.


ci:retest


@kkrampa i was referring to: https://github.com/lwyszomi/commcare-hq/commit/9081000dd0b0e6d6b39a5c75f206a6b4ab2ad6b7#diff-08cb4149c07d6c6f5a5ac4d3c703baefR462. isn't that date range going to include the 1st of a different month other than the current one?


Oh yeah, I hit this too. I think there's a big with the Yeomen template.


How do i see wat i sent


If the IWAD picker doesn't work in fullscreen, couldn't it be disabled only when running in full screen?


squashed


Yes apparently Dragon API searches for the KCauldron class


Never freed.


@pcraciunoiu "When changing this, remember to change it in __init__.py"
So why didn't you?


更正 toj 266


Very interesting; I was awaiting when you'd tackle memory mapped files. This would also be a very good way to read translations, as the portions of the translation file not currently on display would not need to be paged in from disk -- though that's assuming you'll use something like FlatBuffers as I believe you read and parse the whole XML file and convert to in-memory structures ATM.


I did but I didn't realize that back ports would also be 1614 and not their respective forge numbers.


LGTM


Yeah - never too late to improve one's style ;) inserting slices is less error-prone than counting indices.

Just a tip.


Nice =)


you're right, sorry I tested by removing the lines from the properties file and had lots of failures but it was just wrong build

WDYT of still simply using the defaults for testing? I'd like the defaults to be tested in most cases, or at least one.


I thought both boards use linux-boundary now. Is it not the case?


LGTM (note: org.springframework.jms.IllegalStateException; was not the right package, but it's been updated to the right one already: java.lang.IllegalStateException)


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


Hi Frank,

Christian and I looked into the pull request together and merged it to a new branch (https://github.com/camunda/camunda-bpm-platform/tree/wildfly10).

We did the following change: we tried to use the wildfly 10 subsystem code you added also on wildfly 8. This seems to work. Our aim is currently:
- use the same subsystem for 8 and 10
- on wildfly 8, users upgrading to Camunda 7.5 from 7.4 will have to configure threads in camunda subsystem (and remove the threading subsystem configuration). We will explain this in the upgrade guide.
- The subsystem will be compiled with JDK 7. In our first smoketest we can then run it in wildfly 10 with jdk 8. (we have not yet executeted the integration test suite).

Further, we have broken down the rest of the work into the follwing issues:
https://app.camunda.com/jira/issues/?jql=labels%20%3D%20Wildfly

As a next step, @hawky-4s-  will look into the integration testsuites and make then run with wildfly 10. This will tell us whether we are on the right track.

Question for you: are you interested in contributing more towards wildfly 10 support.
What is currently missing the the possibiltity to configure the job executor thread pool in the subsystem.
Would you be interested in working on this? This would be the following jira issue:  https://app.camunda.com/jira/browse/CAM-5596

Best Regards,
Daniel


In a commonjs environment, jQuery will not be defined, which is probably what Borginator is running into.


I think this should be:

```ruby
main_text += ('"' + extract + '"').gsub(/\n+$/, "") + "\n"
```


@gabizou ... and breaks every plugin using optional dependencies.


Great! That looks much better now.
Btw, I'm your Udacity reviewer, I came here since you left the link on your submission :)


@Saori-M

> 例) 引数 [1, 3, 2, 3, 8, 3, 2] => 返り値 { 2 => 2, 3 => 3 }
問題ではこうなってたけど、動作させたら
```
overlap([1, 3, 2, 3, 8, 3, 2])
{1=>1, 3=>3, 2=>2, 8=>1}
```
こうなったな


You should give credit to Steven.


Great job, when redesign the hardware will add this code part to the official version.


0_0 im done XD


@david-martin ok, thanks.


:smile:


ok


jep


`let` is een ES6 keyword en werkt daarom niet in alle browsers (zoals de meest recente versie van Safari bijvoorbeeld). Als je dat alleen nog aanpast naar `var` ben ik helemaal tevreden en ga ik deze request mergen.


Nice. Would it take time importing 390k files from SimpleAuth?


Seems like you commit this statement by accident


(i suppose it might not matter if it's comparing against datetimes, but it looks like it's comparing against dates so it will include an extra day)


testing


@fabianhoward this *still* doesn't fix the bug we were getting. We end up with a sensible-looking apparmor profile...

```
/etc/mysql-default/*.pem r,
/etc/mysql-default/conf.d/ r,
/etc/mysql-default/conf.d/* r,
/etc/mysql-default/my.cnf r,
/var/log/mysql-default/ r,
/var/log/mysql-default/* rw,
/mnt/took/mysql// r,
/mnt/took/mysql//** rwk,
/run/mysql-default/** rw,
/run/mysql-default/mysqld.pid rw,
/run/mysql-default/mysqld.sock rw,
/tmp/mysql-default/ r,
/tmp/mysql-default/my.sql r,
/mnt/took/mysql_tmp/ rw,
/mnt/took/mysql_tmp/my.sql rw,
```

but still the same error as before. I'm increasingly thinking it isn't actually to do with apparmor.


@reginakim I did not see line 862.  Previously looking only at the patch diff, I did not see where teh "onlyT1" variable was used.


@cerha `HTTT...` -> `HTTP...`?


Also removed two unit test assertions that were presumptive. More absolute tests to come.


Just wondering, isn't the C at the start of CompletionHandler?() supposed to be lower case? I'm not sure how this is compiling since the local variable for the completion handler is `completionHandler: PrefetchCompletionBlock?` o_O


This script works perfectly fine for Team. I had created a test team and then created two repo in it. Running this script on the test team updated the webhooks for the repos. Now I am going to run the script on our primary team, which has 64 repo ;)
Thanks a ton @WillPresley.


An issue is here!


I'm not convinced that setting the donorNumber if it is not set should be done in the Validator. It is done (again) in the DonorController and when we import a Donor.


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


Should hit `state_retry` like in voice_registration.  You'll need to add state_retry.


jj já ten grunt celý smažu, je tu navíc


LGTM


@Valtis please git commit --amend -s -c HEAD and rewrite the commit message to what you had in the PR notes.


Yes, it should. I am working on this now, and will release an update for it when everything prepared. Although it was merged into master, it is not fully functional and there is a lot to improve.


test comment (sorry for spamming you but your commit showed up a little special case in GitMate)


remove space


musí je to kvůli ide, původně to nebylo nutné, ale je to tak lepší


role: "menu"に変えよう！


This is the problem with the CDatabaseMgr : if the database hasn't been connected before then it won't be connected here. And we're gonna have problems everywhere later on (unit tests, other classes and so on). That's why I was thinking of using a factory to create the database instead. I'll look into it.


jo je to blbé, co naděláme. Takto to dělá i six, není jiná možnost


Good!


Update: I changed the thread context for my Dokan.Mount and it now mounts and finds the drive instantly again. Not an issue.


Hi Tristan,

With this update it seems I'm no longer able to visualize daily stats using the MySQL engine type. Is this something that can be corrected, e.g. by evaluating the engine type ?


Make sense.


This line (`_rpc_reply_dict['test'] = 1`) is not needed, remaining debug code only.


I think we could use the old `${source_files}` (without `${CMAKE_SOURCE_DIR}`) and create a library:
```
add_library(tiny-process-library ${source_files})
add_executable(examples examples.cpp)

target_link_libraries(tiny-process-library ${CMAKE_THREAD_LIBS_INIT})
target_link_libraries(examples tiny-process-library)
```
Or in other case, we should replace `${CMAKE_SOURCE_DIR}` with `${CMAKE_CURRENT_SOURCE_DIR}`.


An issue is here!


Oh, this'll end up below!!


Oh, this'll too end up below!!


ano tento už to umí v is-core neměl přetíženou metodu str, tento už jí má :)


wondeful


Why you pass Configuration instance as a reference. In that case, class that inherit Configurable is allowed to modify configuration. Do we really need that?


See #23


Good refactoring - much cleaner. Thanks.


meow.


Yay! Thank you, that's amazing :blush:.

Do you know how Travis CI built successfully with this typo?


I did a once-off test "by hand" to make sure the default and the value given do work, which I considered good enough. I can add a regular test for both if you think we should have it.

Personally I tend to think those don't pull their weight. This code for accessing props tends to be very stable once written, so the risk of regressions is very low.


Here in line 4, there's a spelling mistake!


the mocked device will support close as well if you mock the close method:
```
      dev._conn.rpc = MagicMock(side_effect=mock_manager)
+    dev.close = MagicMock()
      return dev
```


Again, should this method be public? Why do we need to have non-const reference to Configuration instance, that allows us to modify it?


Jasně, díky.

> 8. 3. 2016 v 13:21, Boris Dušek <notifications@github.com>:
>
> @cerha <https://github.com/cerha> HTTT... -> HTTP...?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/brailcom/wiking/commit/9c032d7a23ecca76fc57e57712ce9fde29157fa1#commitcomment-16560078>.
>


checking comments


ok


嗯嗯好像没啥问题


Add `creator_opts.retry` to the helper_metadata args and apply the `use_text` like in voice_registration.


An issue is here!


Oh, this'll end up below!!


Oh, this'll too end up below!!


LGTM


@js69 changed context :O could you give an example ?


Hmm - does this get the notes or the cards?


njn co naděláš :)


If my Hamiltonian has complex elements, then the eigenvectors could be complex, no?


test


Great commit ;D :+1:


@stevejgordon, adding these two lines to ActivityViewModel:
```
OrganizationId = activity.Campaign.ManagingOrganization.Id;
OrganizationName = activity.Campaign.ManagingOrganization.Name;
```
means when newing up Activity if you don't provide an instance for the Activity's Campaign as well as the Campaign's Organization, you get a object null reference exception when calling the ActivityViewModel constructor that takes an Activity.

I've discovered this when continuing to work on my unit tests for ActivityApiController.

Can we add a null reference check in the ActivityViewModel's constructor for an Activity's Campaign's Organization?

If not, then I guess two things:
1. is this something we can make a "first class citizen" by requiring instances in constructors for things like Activity, Campaign and Organization? This would be more work than adding a null check in the ActivityViewModel's constructor
2. Or, do we enforce the creation of an Activity with an instance of Campaign,and subsequently, a Campaign with an instance of Organization someone else in the system? Maybe a repository?

Thanks


I'm uncomfortable silently catching and handling this exception. Can you add a Logger (see BaseValidator) and the log a warning message. There's no need to log the entire stacktrace, but perhaps just the Exception message and something else helpful about the invalid Gender value.

Technically this is an error that should be handled by the validator, but with the way the BackingForms have been set up (wrapping the entity) I think it's fine to handle it here. Just make it clearer to the user.


I'm puzzled by this code overall... It iterated over a list, but it stopped after the first element. This behaviour was changed in this patch, so it will process other elements as long as they were 'unchanged', is this ok? Or, was normal behaviour ok? I mean, why not `if (r == 0) break;`? If this is the case, why the loop altogether?


Great to have added the defaults. Looking great.


Ah. I see this code has been removed in a later commit. Please ignore.


You might want to use the squash feature of git to reduce the commit noise by combining similar changes.


It was called from main thread before, now it's called via Task.Factory.StartNew.


An issue is here!


Oh, this'll end up below!!


Oh, this'll too end up below!!


fd2 does not get used.


oops


projectsコントローラーに変えたからディレクトリの名前もviews/projects/....やね


Missing empty line


I am thinking about a proper solution later. For now the important thing is to avoid some hard crashes.


Here in line 4, there's a spelling mistake!


Why rename this index ?
You follow a standard ?


projectsに関する部分テンプレートやしprojectsディレクトリの中にあったほうが良さそう


je travailLE


What happens when we have a published version uploaded that has a long (say 5 year) embargo on it, and an AAM that has a 6 month embargo period?

Would something like 'get AAMs and VoRs, order them by embargo release date, return the earliest released version' be better?

I'm grappling with some similar things for #26. Having a config method to 'get the best document' is a really good idea!


Nice work, especially the comprehensive tests! :) Some comments below:

I'm not sure if this part is entirely right:

```
+            # Crash address
+            if line.startswith("Instruction Address:"):
+                # Example:
+                #     Instruction Address: 0x000007fef86c13e4
+                address = line.split(": ")[1]
+                self.crashAddress = long(address, 16)
```

Instruction Address in your case is the pc which happens to be also the crash address for your example because all of your examples are `int 3` (interrupt) examples. I would recommend using a regular crash as an example, not an `int 3` (which doesn't even have a crash address in the actual meaning) . Maybe go for a near-null deref (e.g. accessing 0x8) to ensure you're getting the right address here. If you are having trouble locating it, I can help if you show me trace output of a real crash.

Does moz_abort always use `int 3` on windows to crash?

One more thing: Can you exact the crashing instruction and registers as well? We have that information for GDB available. It's not super crucial but it would be nice to have and it seems to be that you have it already here:

```
 +Faulting Instruction:01206fbd int 3
````

The `int 3` should go into `self.crashInstruction`.

For registers it would be nice if we could parse them from here:

```
+eax=00000000 ebx=00000000 ecx=12630a6d edx=00000012 esi=00c6b180 edi=00000000
+eip=01206fbd esp=0020d5b4 ebp=0020d5bc iopl=0         nv up ei pl nz ac pe nc
```

For 32-bit, just the `eax, ebx, ecx, edx, esi, edi, eip, esp` and `ebp`. For 64 bit, the r-equivalents + the additional common purpose registers with numbers (`r8` to `r15`). You can parse and assign their values to `self.registers`.

You can also take a look at GDBCrashInfo to see examples for instruction and register saving. ASan also stores register information, but only has limited registers available.


Ja ik ben nog bezig met de contribs generaten, er ging wat mis met branches hier.


Are you serious?


Pourquoi pas MongoDB ?


Jajamän. I julas.


Should be moved to a place where self.gui is inited already


seems a few unithexagons for the capturepointHUD got into this .xml file. is this intended?


Hi all,

I'm noob interested in MVC and .NET... Heeeeelp


Est-ce vraiment Meteor qui va encourager cela ?


How should this work? Now `force` is always `false`.
Shouldn't it be read from the config?


Yep agree. Probably need to similar to some of the menu stuff too.


I like these higher level test cases! Thanks!


Added check that number is 128bit


top/indexとprojects/indexってどう使い分けてる？


An issue is here!


Oh, this'll end up below!!


Oh, this'll too end up below!!


in case this is the second time (or more) this method is called, you must remove the references as they are strong references meaning the previous view controller will never be released.
```
[self.scrollViewController.view removeFromSuperview];
[self.scrollViewController removeFromParentViewController]


It would be better separating concerns once again. The controller handles the prompt logic, and then passes it over to the address book to actually destroy the records:

```ruby
def destroy
  puts "Would you like to delete all entries, Y or N?"
   answer = gets.chomp
   if answer == "Y" || answer == "y"
     @address_book.destroy!
    #end # weird end here
   else
     system "clear"
     main_menu
   end
 end
```

```ruby
# models/address_book.rb
def destroy!
  @entries = []
end
```


Currently there are no plans to develop Atos for EE4.  This is not the place to make these requests.

Please use the forums at eventespresso.com/support/forums.


Well, buy native I just mean like letting the OS decide what to do - like it has in your screenshot. Though tbf it looks fugly with the blue outline on focus, so I can see why you'd want to be able to customize it ;-)

I reckon my previous suggestion is the best of both worlds - choice between native and the old custom layout, and if native is selected (and it's supported), choice between native colours and matching it with the window colour.  I added a similar option to Radiant a while ago, though tbf in our case that was more of a performance fix than a styling decision. :disappointed:


TeamCity Todo :: Build [Build 51](http://109.103.226.38:8512/viewLog.html?buildId=38182&buildTypeId=Todo_Build) is now running


Here in line 4, there's a spelling mistake!


Can I get to the actual Bonobo server resources from the test dll easily, or should we just import the bonobo resources into the test project?


Good suggestion! I'll do that.


TeamCity Todo :: Build [Build 51](http://109.103.226.38:8512/viewLog.html?buildId=38182&buildTypeId=Todo_Build) outcome was **FAILURE**
Summary: Exit code 127 (new) Build time: 00:00:30


although this isn't documented, [UIImage imageWithData:] should also be called from the main thread.


Just to have it public why this happened. On February 27, 2016 around 2am EST the top 3 files on Bitbucket downloads (Apktool 2.0.1, Apktool 2.0.2, Apktool 2.0.3) were serving 0kb files.

I submitted a help request to BitBucket.

Response:
> We're currently investigating this issue with a high priority, and we'll update you when we have any further updates. Thanks for your patience.

Later Response:
> Over the weekend, while making upgrades to the Bitbucket platform, non-source code binary files were permanently overwritten for a small number of users who use the downloads feature. Unfortunately it appears that some of your files were impacted. You should have received an email by now, outlining the files affected.

>We recognize the trust you place in us with your data and take that responsibility seriously. We have measures in place to protect the integrity of your data, but those failed us in this instance. We sincerely apologize, and are improving our operational procedures in order to prevent something like this from ever happening again.

( I never received an email )

Files deleted and reuploaded.

* Apktool 2.0.1 - 148,771 downloads
* Apktool 2.0.2 - 154,940 downloads
* Apktool 2.0.3 - 106,387 downloads


空行


Is it correct? the parameter for Promise.all will be [undefined]. typeof [1,2,3].forEach(() => {}) === undefined; `Promise.all(users.hashes().map(user => userService.create(user)));`


Thanks mate, appreciate the clean up!


`&& FLOW_NODE_TYPE_PERSISTENCE`?


Breaks JUnit tests, as there is no jar file in this phase yet.


call one of the valid initializers of UIVIew (either initWithFrame or initWithCoder).
in case you don't know the frame yet, use [initWithFrame:CGRectZero];


implement functionality here


same


add new setting here


Questão 1


Unfortunately no, https://github.com/Freescale/meta-fsl-arm-extra/blob/master/conf/machine/cubox-i.conf#L17


there is really no way except that I make sure I don't call this function on freshly allocated abstract float. I just made sure again I didn't make this mistake.


This is a dead end, used to find the MPI bug with a stripped down version of the code. Can be deleted.


Coulb be incorrect, I didn't check semantics.


Meant to do Pull Request... Just reversing to previous commit.


avoid excessive use of spies. a good example why not to use them like this is this case: the functionality does not work because `input` is not defined inside `addItems` function, but because testing stubbed function calls, the tests will pass


that looks really odd now if asking me. I get why sniffer suggested to make less indentation here but I'd argue that the tool is wrong here. How about re-indenting the whole return statement like below:
```php
		return array_map(function(Task $task)
		{
			return $task['id'];
		}, array_filter(
				$this->tasks,
				function($task)
				{
					return $task['closed'];
				}
			)
		);
```
Or maybe even (I find it even better)
```php
		return array_map(
			function(Task $task)
			{
				return $task['id'];
			},
			array_filter(
				$this->tasks,
				function($task)
				{
					return $task['closed'];
				}
			)
		);
```


since you're getting this in the initializer now, this better be set to readonly.


> If my Hamiltonian has complex elements, then the eigenvectors could be complex, no?

This is exactly my point.
Regardless of being real or complex, the eigenvectors must be columns of `unitary_matrix`, while `eigenelements()` returns them as rows. This call to `transpose()` has nothing to do with linear algebra; it is simply a switch from one notation to another.


wpcore에 많은 기능을 넣었군요~ 좋습니다 제점수는요


LGTM


I will wait on adding the barcode to it. Need to think it out.


这是啥


Note that this breaks z-ordering among segments and relative to all other projectiles.


Amélioration D.


There's no need for changes to this class.


Ah okay, second Tx queue comes with next commit...


This should take you back to the same state - if this is a check for poorly formatted msisdn.  There should be another test checking for a valid msisdn that is not recognised.


I was reading this article https://www.viget.com/articles/gulp-browserify-starter-faq and ended up here. browserify + coffee S2


done


Sadly git diff didn't seem to work too well here. But it looks like another good cleanup. Thanks.


@dlazerka I remember copying it from the Fabric Android docs. The above URL no longer works.
Maybe there is an updated proguard config in the new API docs: https://docs.fabric.io/android/index.html


n not used.
If doing it this way, it would be more efficient to read the entire reg value, manipulate all the bits necessary, and then write.


added back


You are correct about the video. I am however still getting some `/*` (I installed from master). If I replace by a `#`, it works already better.


removed


Reviewed. Please merge


It would spend quite a lot of time. However, it is run in a separate thread, so don't worry.


I believe this property can be removed from `ConsumerProperties` class?


Repro steps please.


<read meat>


see #6655 for a change on the module names


LGTM


added
#This step was left blank after removing support for sql-ledger.


I have doubt about the name because you don't use it only in the case of inference. Please get back to the root definition of what inferring means.


#This step was left blank after removing support for sql-ledger.


@slitazcn 嗯你说的对……我最开始没注意标签那里╮(╯▽╰)╭
#1599


default value to `1` ?


We already introduced semvar. The annotation is there since I was told once that it needs to be there for a "proper" Modelica library. I will remove it.


`CompletionHandler?` is a type, it is `Optional<CompletionHandler>` under the hood. And when you calling `CompletionHandler?()`, it means `Optional<CompletionHandler>()`, which will leads to a `nil`.

It could compile since there is no syntax error here, and the unit test did not cover it. (since it is only a starting step and not be the final implementation)

Although the result of this statement is not used, but since there is no "result not used" warning for it in Swift standard library, the compiler fails to catch it up and give any message.


Should we assign the default value to `1


2nd contributor is added in setup. Changed this first assert after reload was to check to make sure that there were still two contributors. And then the second one is to doublecheck that the admin contributor is still around.


I agree, doesn't make much sense to add this here. From what I see the best place should be when persisting the donor for the first time. There even is a method called updateDonorAutomaticFields that sets the donor hash in DonorRepository. Should we just do it there and remove setting the donor number everywhere else?


Can you please document this step here. Why do you check the launchTime for zero and assign the current mac time if so?


@jglick actually from the analysis of https://github.com/jenkinsci/jenkins/pull/2092 its actually 29 out of 662 so 95%+ of usages **in core** can assume non-null and 100% of usages in plugins (after the fix in https://github.com/jenkinsci/jenkins/pull/2091 and https://github.com/jenkinsci/jenkins/pull/2093) can assume non-null


LGTM ;)
但是 migrition 文件哪里去了？这个要加上，rebase 一下再提交一次。


I'm not at my home pc now, but I think running the following commands (in a new branch, naturally) will fix things:
```
git rm -r --cached .
git add .
git commit -m "cleared gitignore files"
```
Once again, not at my home pc but I'll try to clone it on a cloud IDE.


Dude, not cool to release this before you release 16.02! :cactus: Blindly trusting parties will run into issues. You might consider switching to https://github.com/freight-team/freight, so users of the jabber.at packages are at least able to downgrade. Freight is able to hold multiple versions of packages in a repository.


:+1: Looks good to me


This works as long as you don't mix up type var from the method (current substitution) with bounds from the class :

class A<T> {

<S extends T> void foo(S s) {}
}
class Bar extends Qix {}

new A<Bar>.foo(new Qix());
new A<Qix>.foo(new Bar());


Done


doing this will allocate a new MouseComponent struct on every frame and ask the GC to collect the no more referenced one ... Do you not fear a performance impact?


I'm so happy that  I will begin  to learn here!


No it doesn't matter here. I guess you may think ```start_date``` should be first day of some month whereas ```end_date``` should be last day of some month. It's not the case here. All reporting data have dates converted to the first day of particular month. Basically I can leave end date not touched but there is no reason to get data from range e.g. 2015-01-01 - 2015-04-20 so we convert 2015-04-20 to 2015-04-01.


for reference: fixes #669


I have no idea why you introduce yet another pair of macros that do the same as MIN() and MAX() ? I believe MIN() and MAX() are already a _de facto_ standard on their behaviors, and introducing another pair doesn't make sense.

If you fear name collision, you may #undef the old one. The code would be like this:

    #ifdef MAX
    # undef MAX
    #endif
    #define MAX(x, y) ((x) > (y) ? (x) : (y))

Any reason not to do this way?


I wonder if there is any allocation? Since we're only doing value-assignment? (that's why I used that syntax instead of `mc = &MouseComponent`)


TeamCity jonas-requests :: WindowsBuild [Build 18](https://build.neohq.net/viewLog.html?buildId=568573&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 31857, ignored: 117 Build time: 1:54:34
### Failed tests
```
org.neo4j.test.ha.ClusterTest.testClusterWithWildcardIP: <no details avaliable>

```


bleh


:+1: looks good to me, but what out of interest, how significant is the order of the arguments in the assert?


Does this actually belong to `producer` properties as well? I thought `minPartitionCount` applies only to consumer.


The only possible excuse for having `getInstance()` not throw an exception is the case of plugins with bugs that are sending Callables to remote JVMs and those callables incorrectly call `Jenkins.getInstance()` in order to determine if they are running on a remote node... such cases are bugs in those plugins... if you are significantly concerned about that risk then I will consider

```
--- a/core/src/main/java/jenkins/model/Jenkins.java
+++ b/core/src/main/java/jenkins/model/Jenkins.java
@@ -719,17 +719,18 @@ public class Jenkins extends AbstractCIBase implements DirectlyModifiableTopLeve
     /**
      * Gets the {@link Jenkins} singleton. In certain rare cases you may have code that is intended to run before
      * Jenkins starts or while Jenkins is being shut-down. For those rare cases use {@link #getInstanceOrNull()}.
+     * In other cases you may have code that might end up running on a remote JVM and not on the Jenkins master,
+     * for those cases you really should rewrite your code so that when the {@link Callable} is sent over the remoting
+     * channel it uses a {@code writeReplace} method or similar to ensure that the {@link Jenkins} class is not being
+     * loaded into the remote class loader
      * @return The instance.
      * @throws IllegalStateException {@link Jenkins} has not been started, or was already shut down
      */
     @CLIResolver
-    @Nonnull
+    @Nullable // TODO replace with non-null once Jenkins 2.0+
     public static Jenkins getInstance() {
-        Jenkins instance = HOLDER.getInstance();
-        if (instance == null) {
-            throw new IllegalStateException("Jenkins has not been started, or was already shut down");
-        }
-        return instance;
+        // TODO throw an IllegalStateException in Jenkins 2.0+
+        return HOLDER.getInstance();
     }

     /**
```

As a band-aid for such rogue plugins... but I know that there are plans to ban loading `Jenkins` on remote agents, so my view is that plugins should find and fix the bug before they get the hard to find `LinkageError` when remoting has the instantiation guard added


So you need to specify the /b both in the modparam as well as the script? An error is thrown if the profile name doesnt match the profile name in the modparam...so if a profile named "calls" was previously in the modparam, then it needs to be changed to calls/b and the script also needs to be changed to calls/b? Ie you cannot have calls/b in the modparam and just calls in the script.

If that is the case, then it can break all kinds of external scripts that rely on consistent profile names, and also requires that you update an entire cluster of servers at a time to effect this change.


Danny, nos ouvindo :)


No we don't, good catch.


I think it'd be better to look at the string encoding header (and assume `UTF-8`) and use the UTF8/16/32 structs for proper parsing.


:+1: pity you can't do a diff on the binary spreadsheet, but looks good anyway.


It's supposed to be
`It's not tax evasion, it's legal.`


Fuck this commit name,


ok maybe you avoid the garbage collection because of the value assignment but you still allocate a local variable for the MouseComponent no?


Added blood samples previously annotated in RNAseq.

Changes from old version:
- deleted:
  - GTEX-OXRL-0003-SM-4X611 : WXS not RNAseq
  - GTEX-PLZ6-0006-SM-2I5FQ : subject deleted for drug abuse
  - GTEX-PLZ6-0006-SM-33HBZ: subject deleted for drug abuse
  - GTEX-WYVS-0002-SM-3USR9: : WXS not RNAseq


Lets try and minimize unnecessary style changes.


When the asset fails then the error message will be incorrect. For example "Preferred Language matches. Expected: null Was: English" when it should be "Preferred Languages matches. Expected: English Was:null".

Plus I think it's easier to read this way around.


Should this be removed instead of commented?


close #235


Why not URL of the `via` permalink (if exists)... That way, if `via` made commentary / contextualization we link there.


good


This is because we want to persist only the original `observe` request and don't want to persist block-wise requests. (I mean request for asking next block).
Block-wise state is not persisted/clusterable.


哈哈 景涛，现在在实习还是回北京了？


в каждом файле должна быть новая строка....


Sorry, forget it. :)


This seems to have fixed the problem for my project also. Should a pull request be made to merge it into the source?


This one was dropped accidentially I believe?


You can target id's in jquery with `$('#id')`


Lets try and keep the indentation consistent.


What is a valid launch time? Is zero also a valid launch time (e.g. in case of openMAC it is)? Isn't it possible to validate the launch time just with the fTimeTrig flag (or call it fLaunchTimeValid)?


:+1: cool :)


Should not be the case? `mc` is located in the heap, so I think the compiler is smart enough to (instead of allocating it locally, and then moving it), to place it in the heap immediately.

(this was more done for correctness; I don't think it'll impact performance that much)


Yeah, I should've used `enumerate`


May be we can remove both the `RedisProducer/ConsumerProperties` classes and have `Producer/ConsumerProperties` classes serve the purpose?


Shouldn't there be an asterisk in there

Sent from my phone

> On 8 Mar 2016, at 13:21, ExplodingCabbage <notifications@github.com> wrote:
>
> @fabianhoward this still doesn't fix the bug we were getting. We end up with a sensible-looking apparmor profile...
>
> /etc/mysql-default/*.pem r,
> /etc/mysql-default/conf.d/ r,
> /etc/mysql-default/conf.d/* r,
> /etc/mysql-default/my.cnf r,
> /var/log/mysql-default/ r,
> /var/log/mysql-default/* rw,
> /mnt/took/mysql// r,
> /mnt/took/mysql//** rwk,
> /run/mysql-default/** rw,
> /run/mysql-default/mysqld.pid rw,
> /run/mysql-default/mysqld.sock rw,
> /tmp/mysql-default/ r,
> /tmp/mysql-default/my.sql r,
> /mnt/took/mysql_tmp/ rw,
> /mnt/took/mysql_tmp/my.sql rw,
> but still the same error as before. I'm increasingly thinking it isn't actually to do with apparmor.
>
> —
> Reply to this email directly or view it on GitHub.
>


строка..


Also, why change this CLAMP macro? That increases no readability at all.
By the way, this macro is introduced by me in commit 6dae8108f8e6a8d3f4ccf3d92f8bb19d3933861a , in case you didn't `git blame` this line.


For the future, you can test a variable (class) "type" using `.class`
i.e.:

```ruby
>> sp = StrictPeriod.new
=> #<StrictPeriods::StrictPeriod:0x007fd49cc32b48 @anchor=2016-03-08 13:10:59 UTC, @past_only=true, @week_picker=#<StrictPeriods::PeriodPickers::WeekPicker:0x007fd49cc32aa8 @steps=0, @anchor=2016-03-08 13:10:59 UTC, @past_only=true, @offset=432000>>
>> sp.anchor.class
=> Time
```

(can't access `WeekPicker` from the outside, but if you add an `attr_reader` on it, you will find the same result)


Corrected.


Maybe this conflicts with https://github.com/openPOWERLINK/openPOWERLINK_V2/pull/102#commitcomment-16554808 ...


роутеры должны быть отдельным файлом - routes.js


a word about simulating distributions:

1) most importantly - be very careful about thinking about boundary conditions. Pseudorandom number generators hold no interest in things like 'biological plausibility'. If you actually look at the distrubtion you have assigned:

![image](https://cloud.githubusercontent.com/assets/3196313/13602619/b669f4e6-e505-11e5-9188-aa7c27a73998.png)

You see that if you actually did CTS or anything with a meaningful number of individuals generated (or random 'bad' luck) you will get individuals with even negative values.

At minimum apply truncated bounds, regardless of distribution. The easiest rule of thumb is pretend you have inclusion/exclusion criteria for for the 'trial' you are simulating and apply those. Things like > 18 years old, or CRCL > 70 as many trials will not enroll people with moderate/severe CRCL impairment unless it is a specific focus of the trial, etc.


大きな変更はsampleの画像を消したことです。残りが作業中に出たよくわからない（改行とか）の変更です


Ah yes right. If only Modelica (and tools) itself would support semver properly then such dummy updates because of a new MSL would not be necessary ;-)


Or `zip`. Nevermind, I'll fix it this evening.


I used `instance_variable_get` method to access `steps` variable without adding `attr_accessor`. Tell me if that's wrong.


слишком большие отступы, табуляция должна быть в два пробела


That's what it is yes, sorry for not mentioning that in the commit message.


The class `DefaultBindingPropertiesAccessor` and `BinderPropertyKeys` can now be removed.


Je crois que oui.
Meteor complète bien Running Lean, je trouve.


This would be clearer as:
```python
while True:
  ...
  if time.clock() >=  bail_out_time:
    break
```


I see what you're doing with this below, but do we need the concept of explicitly stating which transactions should be unique? I think that should be the standard probably, as for most cases I see that being desired. Maybe, we could simply add a flag parameter in the createTransaction function for stating whether or not to overwrite (abort) any existing transaction or whether to duplicate it somehow.


@mattstauffer redoing this ended up flushing out this bug that was replacing the query instead of appending, hooray!


I did some additional unscientific testing myself, running the cross flow tutorial with the different models. For simulation times of about 5 minutes, the differences is within the region of 3 seconds for the total simulation time. Hence, it does not seem to be important for the performance at that particular case at least.
However, from a consistency point of view, `interpPropsMultiRe()` and `interpCoeffLists()` should be implemented in the same way.

Note also that the two functions `normalCoefficientList` and `chordwiseCoefficientList` can be cleaned up in the same way as subList, considering the use of `interpCoeffLists()`


нет необходимости в определении пустого массива...


Yeah it would, see the TODO comment a line above...


Nitpicky:  `App` instead of `Module`


`zip` is definitely better!


Yeah I think this is a good change here


call to undefined function here... switch to `_e`


`user.quickpay_hidden` is a boolean, so you can just type `!user.quickpay_hidden`


Oh, also, I added userinfo storing. This will succeed separate username, nickname, lastfm, etc, arrays.


Agreed, let's do it per transection.


Yeah I saw that, I just wasn't sure whether you were going to change to using the native Swift `UTF*` structs. All good then :blush:


Put a space before false

Don't know if this works, but still


i'd prefer 4.x for lodash, and pull the entire thing. should help with deduping with other things that need lodash.


> There's a bug on the mixxx bugtracker. Let's wait until they fix it.

you mean do not merge this until this is fixed ?
I don't think it is blocking, it just needs to be properly done

> Can we add libav support later if somebody really needs it?

if you tried it and it doesn't work, of course, but the please add a comment stating so; if it's just about not wanting to test, then no :)


else не нужно указывать если сделать return $scope.err = true
а ниже определить
$scope.err = false;
$location.path("/");


At the moment It explodes if the cable's max transfur is greater than the blocks max input. This needs to be changed. @gigabit101 any ideas on how we should do it? High/Medium/Low?


`post.credits` は配列(的なもの)が返ってくるので、常に真として評価されたりしてませんか？


lol


Could you run `bundle exec annotate`, to update the annotations in the user model


you want to extract out 'all' constants, for clarity and documentation, as often as possible, so for example you might instead write this equation as

```
# scaling factor for CRCL based on publication ...
CRCL_SF = 0.67
# normalizing factor based on rounded median CRCL of trial
CRCL_NORM = 70
....

mutate(CL = (CRCL_SF*(CRCL/CRCL_NORM)....))

```


Should be solved with https://github.com/odoo/odoo/commit/04f7858159231f74c6efb19a3284387ccf5fd9b9


This statement is pretty much looking as if it is doing nothing anyway...

There is no sideeffect involved, and we either return ``nil`` or the value of ``node`` but discard it anyway. Perhaps we should take a closer look at this function in generall.


`create_post` に `published_at: nil` を与えるのだとダメでしたか？
(初期データを用意したあとで、さらにデータを更新したくなるのは怪しいサインです)


In each transaction, there is a transactionError function that will (should) get called (at the end) when any request in a transaction gets any form of error (such as canceling/aborting) we should be able to leave it to the transaction to handle calling that method and the data that function should get should include a number of parameters, one of which should be a reason like `Francis.REASON.ABORT` or something like that. Does that make sense??


Since we only need the `environment`, I believe we can use `EnvironmentAware` instead.


A disease_id and a casebook_id are the same thing, see my comment in the pull request, and benchx's explanation. For some reason (preexisting this patch) every diagnostic step has a 'disease.id' which is named after the room that the step takes place in.
In some cases (presumably the ones where getCasebookId are called) we want the last thing the patient would have been asked to pay for rather than the patient's disease. This needs to be better documented both in the commit message and in code.


can be simplified at top of this for loop..

```
if (v->forced_update) {
   chck_iter-pool_push_front(visible, &v);
   v->forced_update = false;
   continue;
}
```


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 24](https://teamcity.labkey.org/viewLog.html?buildId=308855&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 39, muted: 4 Build time: 01:11:52
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

```


view で使われている credits や participants を一緒に取得していないですが、 N+1 が発生していたりしませんか?


You may want to move this inside the condition check above after the call to msleep


I like this, but ideally I think it would be good for a transaction to clean up after itself, once it has been `ended` by the caller and when it's done handling it's actions (so basically the finalize() method should (and will be) wrapped by the transaction and should include a self-destruction function so to speak). What do you think?


Doesn't actually schedule anything.


I assume this part is a WIP.


I don't get why the view isn't painted?


Okay cool I'll fix it. Thanks.


I'll revert this and make it better once I start working on the worklets.


記事が stop …
モデルの語彙に合わせると、 unpublish でしょうか？


Back online. Nice find `PyObjCMethodSignature_WithMetaData`.


Yeah I saw that , just didn't know how it would be handled.


OHHHH, I see, this is a flag, like an option... I see now, can probably ignore my comment above then, my bad


Apart from this I like the idea of cleanly managing the value reset this way.


Sweet!  Commit the files (including the deleted bootstrap file)

Then push them to master and then merge master down to dev.


There's a test, on persistence-fs-1.fbp, that breaks with this change. `validator_boolean_def` expects a `TT` sequence that it's not possible anymore. That test must be changed, then.


Agreed, just left this here for flexibility. The inner private method is the important part.


Cambios en la estructura


Either way really, we can discuss further


> you mean do not merge this until this is fixed ?

Nope, just merge it and fix it in the next version if the version supports ffmpeg 3.

> if you tried it and it doesn't work

It's kinda painful to replace ffmpeg with libav, I'd like to avoid doing so in my system.


Should probably set the individual transaction's status to CLEANUP, not the entire Francis object's


I am a nice comment


<img src>


Thanks!! :D


:+1: now much clearer code for me with emphasize Left for size, which is not so visible in old code.


@ranp yep, it seems a regression. I've just pushed a fix to 5.6+. Please check your app.

Thanks.


Hmm, I was going to suggest just doing a `delete self.transactions[transactionName]`, but I guess you're thinking of the case where we have non-unique transactions. However, in that case I would point out that you'll still just have the one object on that map, since it's the same property name. Will probably need to figure that out somehow anyway.


Hi Richard,
I still didn t understand the restrictrion sites part from a biological point of view.
The coursework instructions say only to highlight these sites which only cuts in 5 or 3 regions and not in between. We will talk later; also the part where you say to check the complementary strand, isn't that redundant? If i find a motif in a string like "acca" isn't it obvious that its complemetary is "tggt" and therefore I am gonna find a tggt in the complem strand?; i mean  if i search a motif in a sequence, and then I search its complementary motif in a complemetary sequence, it should give the same results isn't it?....

Also, guys basically if the sql is not in the middle layer there is not much for me to do.The only real calculation is the codon frequency, which is done, i just need to transform it into a module.The restriction sites calculation, highlighting exons in the sequence and displaying the coding sequence next to the translated sequence are calculations/task that go hands to hands with the presentation.It would just make it harder for dolapo if he has to use my data (eg. an hash with  start and end of sites or exson) to then highlight and divide the sequence in the cgi; he would have to basically redo the same amount of calculations i did to make the hash (or arrays) with postisions; this is the problem of my middle layer guys, i don t know what u give me and i don t know what u prefer having for me; i always need to think wether what i return can be used in the front end, and it is tooo hard, that is why i need to go a bit into the presentation side for some tasks to understand what is needed in the front end;. Like i could make 2 arrays with start and end postion of sites but then i don't know if dolpao can work with those or he would prefer these info passed to him differently (like an hash); it would be very challenging for DOLApo too use and understand all the functions i write and then use them; I need to take into account presentation for this calculations at all times; anyway the big chung of the presentation would still be in the front end, in the middle layer i just add meaningless tags which take a meaning once are printed in html by dolapo.
 i was looking for a way in which the front end could use my hash or arrays with start and end postion to highlight a sequence, but it does not make sense to do it in 2 steps in my opinion; when u can chop the sequence by adding tags straight away in one script in the middle layer.
This is also the reason why i wanted the sql queries in my layer, so the data from the database is passed to me in a way which i know i will be able to use:everyone has his own coding style; so like if i do my calculations i know what i need to start them. Like for my codons calculations, i need specific things passed to me from the db in a specific manner to be used in my functions. (like an array of sequence etc.). If richard for instance write a query that pass me the sequence in a different way, like an hash or whatever, that could not be ideal for me and the way i like to work in the middle layer (hope this  makes sense). Plus I need to wait for richard to write the queries to make sure my middle layer connects, which is another obstacle which could be avoided by me doing queries. Same on the other side, if just return an array with positions then i need to wait for dolapo to get there to see wether he can use it to highlight thigns etc.

 If i just do a script to make an hash (or 2 arrays) with start and and end values of exons (and restrictions sites for the other calculation), my work will be literally not much; also if i do not do any sql, and if i do not include a bit of html my work is too abstract cos:1. I don t know how data is passed to me from richard.2. I dont know how dolapo prefer is data passed to him.
So the option is either you leave the queries  in the middle layer with me and I will be more abstract in the front end side trying not to include presentations stuff, but I will have to contsantly communicate with dolapo and he will have to tell me asap how he wants data from me... or if sql stays in the bottom layer I need to do a little bit more than just create an array of positions in the middle layer (otherwise my cw will be 3 lines of code) and i need to do the full calculations to highlight things, otherwise my work is literally 1/3 of yours.....Also it s too messy and I basically need to constantly ask you how you giving me data and how you are gonna use my data .

Richards you have databse, parser and sql.Dolapo has HTML page plus at least 2-3 cgi script.me basically just codons and restriction sites calculations.
If somebody has a better idea how to do the middle layer in an abstract manner, happy to take any of the other 2 sections.

gabriele
Date: Tue, 8 Mar 2016 03:51:17 -0800
From: notifications@github.com
To: Biocomp2Project@noreply.github.com
CC: g.nocchi@hotmail.com
Subject: Re: [Biocomp2Project] Removed another unused line. (e0a99ef)

We will want to highlight sequences throughout the gene as restriction sites can be useful even if they are not in the 3' and 5' regions. e.g. for checking your cloning process has worked by doing a restriction digest that would only cut if the gene you inserted in your plasmid contains a cut site for a restriction enzyme.

however we will want to to check for both orientations of the restriction site e.g.

5'-ATGCATGC-3' and 5'-GCATGCAT-3' which its reverse transcription (each character is swapped for it pairing partner e.g. A->T and the order of the characters is reversed) as we only that the sequence of the 5'->3' strand.

because in the 2 DNA strands  it looks like this:

5'-ATGCATGC-3'

3'-TACGTACG-5'

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


There is a lot of attributes in an exchange:
```java
	/** The endpoint that processes this exchange */
	private Endpoint endpoint;

	/** An observer to be called when a request is complete */
	private ExchangeObserver observer;

	/** Indicates if the exchange is complete */
	private boolean complete = false;

	/** The timestamp when this exchange has been created */
	private final long timestamp;

	/**
	 * The actual request that caused this exchange. Layers below the
	 * {@link BlockwiseLayer} should only work with the {@link #currentRequest}
	 * while layers above should work with the {@link #request}.
	 */
	private Request request; // the initial request we have to exchange

	/**
	 * The current block of the request that is being processed. This is a single
	 * block in case of a blockwise transfer or the same as {@link #request} in
	 * case of a normal transfer.
	 */
	private Request currentRequest; // Matching needs to know for what we expect a response

	/** The status of the blockwise transfer. null in case of a normal transfer */
	private BlockwiseStatus requestBlockStatus;

	/**
	 * The actual response that is supposed to be sent to the client. Layers
	 * below the {@link BlockwiseLayer} should only work with the
	 * {@link #currentResponse} while layers above should work with the
	 * {@link #response}.
	 */
	private Response response;

	/** The current block of the response that is being transferred. */
	private Response currentResponse; // Matching needs to know when receiving duplicate

	/** The status of the blockwise transfer. null in case of a normal transfer */
	private BlockwiseStatus responseBlockStatus;

	// indicates where the request of this exchange has been initiated.
	// (as suggested by effective Java, item 40.)
	private final Origin origin;

	// true if the exchange has failed due to a timeout
	private boolean timedOut;

	// the timeout of the current request or response set by reliability layer
	private int currentTimeout;

	// the amount of attempted transmissions that have not succeeded yet
	private int failedTransmissionCount = 0;

	// handle to cancel retransmission
	private ScheduledFuture<?> retransmissionHandle = null;

	// handle to extend blockwise status lifetime
	private ScheduledFuture<?> blockCleanupHandle = null;

	// If the request was sent with a block1 option the response has to send its
	// first block piggy-backed with the Block1 option of the last request block
	private BlockOption block1ToAck;

	// The relation that the target resource has established with the source
	private ObserveRelation relation;

	// When the request is handled by an executor different than the protocol stage set to true.
	// The endpoint will hand sending responses over to the protocol stage executor
	private boolean customExecutor = false;
```
Currently we only identify the `request` attribute as mandatory to handle notification.
That's why we go for request persistence.

I suppose we will also need some security data to do the context correlation.
we probably also need the observe sequence number.
About deduplication, the current approach don't handle it. That's means that after a reboot or between to instance we could have duplicate message. But the idea is to mainly load balance traffic by IP/port, so close messages should be handled by the same instance in the cluster. Does it seems acceptable ?


Change*


@Nevtep   Chromes latest update seemingly broke js interpretation of function declarations.  The parseScript function was being seen as undeclared where it was being called. We had to move the function to fix signing/sending on production.   I know staging has refactored a lot of it so this change may not be necessary but was still necessary on production until we finalize/merge the rebrand


@dpastoor Thank you for the teaching point.  I'm learning a lot from your comments...

This was just a quick and dirty, but I definitely wouldn't have caught that.  Thanks!


+1 to this. Fixed the issue for me. I also had success with this solution but also just calling $el.mode(false) at the end of refresh.init(), but your solution probably covers more bases.


Hmm.. I'm wondering if we should wait to reset the status until the callbacks for error and stuff like that have been called. That way we're sure that the state of the transaction is really clean. Also, I still think this would end the transaction and that it should be removed afterwards, but I suppose that's a separate thought.


Will this help https://fossies.org/dox/itdb-1.22/PDF__Label_8php_source.html

Terry


WTF.


FRIGGIN BLOCKED IT


Any change this can make it into a new version of the rest-client-builder plugin for Grails 2 ?

In 2.5.3 this causes problems, and one can't (or: I did not manage to) just use the latest datastore-rest-client jar with the 2.1.1 plugin. As  dustindclark also reported  in https://github.com/grails-plugins/grails-rest-client-builder/issues/40 that results in a ClassNotFoundException: org.grails.web.converters.configuration.ConvertersConfigurationHolder


Testing


Yeah, I wasn't sure about this either. If it's possible for HadoopIO to produce data that are *not* Writable, then we should definitely remove it. I added the bound to avoid some really ugly casting around lines 97-100 below. Beam now throws an exception if the coder is not set, and we had to cast to Writable classes to set the coder there. But removing it from the API and just doing the ugly casting might be a better path until we land on a long-term approach.


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


Hmmmm.... I guess that's a special "Phil Webb" App. :wink:


@prashantchoudhari @roundrobin99 In the future, try not to mix a merge from master with your changes. It's now very difficult to work out what's actually been changed in this commit


move


agreed, yeah I think that's fine, just wanted to make sure I got that idea out there.


cool, sounds good


There's also a test that fails with this patch, `http-boolean.fbp`. But this one is tricky: as default value of server node is `true`, and first packet on its port `IN` is also `true`, this first packet is never sent out. Only the `false` one sent after a `POST` from client. Maybe these nodes should sent initial packet? Or only the test changed? Not sure about this case...


Master is stupid


I am a nice comment


TeamCity Todo :: Build [Build 52](http://109.103.226.38:8512/viewLog.html?buildId=38183&buildTypeId=Todo_Build) is now running


Because I want to update the view while drawing my effect(in my case, the Expo effect where you see all of your workspaces on the screen in the same time). But, during this operation, views are rendered with custom geometry, so WLC shouldn't render them at all, but they should be updated(mostly frame callbacks should be called).


TeamCity Todo :: Build [Build 52](http://109.103.226.38:8512/viewLog.html?buildId=38183&buildTypeId=Todo_Build) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:00:05


Did you get a warning from Google?


Yeah.


deleted:
- Tech Dev - Large Insert RNA Sequencing - High Coverage (50M pairs)
  - GTEX-S32W-0006-SM-5S2SW
  - GTEX-XLM4-0005-SM-5S2RY
  - GTEX-XXEK-0005-SM-5S2TG
- Tech Dev
  - GTEX-N7MS-0007-SM-26GME
  - GTEX-N7MS-0007-SM-26GMV
  - GTEX-NPJ8-0007-SM-26GMF
  - GTEX-NPJ8-0007-SM-26GMW


Ah, nice. Thanks!


主页面


sure - as I'm all about 'rules of thumb', here is my philosophy on extracting variables. If in 6 months you think you could pull back that equation and explain exactly what those variable numbers corresponded to, then you can get away with hard coding (eg an allometric scaling factor of 0.75 on weight based clearance), however if it is pulled from a paper, another calculation, etc, its always better to document. It also makes it much easier to re-read the equation when you're coming back to it rather than trying to sort out all the numbers.

For example, in the beginning you did a great job on line 142 to extract out the patient demographics when calculating - which also makes it very easy to read that equation, imagine instead trying to re-read the equation of `(0.67+87/70 + 61.1^0.24)*70*0.06` and trying to recall what meant what


No, but I figured that if ad blocking is not allowed, port forwarding would not be allowed as well.


Is this number expected to be fixed, and be a characteristic of the audio sub-system + drivers + hardware, or is it a function of time as well, in the sense that if you ask for this value right before a callback fires from the audio sub-system, the value returned is very small ?

Is it more like   `snd_pcm_delay` from Alsa, or more like `IAudioClient::GetStreamLatency` from WASAPI ? It think the latter is better.


а зачем генерик?


Wouldn't this be better in the Shopify ship - responsibilities?


how do you raise PR without merging master?


придумай названия получше


`<a><code></code></a>` and not the opposite, please.


This is code is ..... not its not....


Good point, let me update docs


@bhousel I'm not quite sure where the "unmaintained" comes from, there are track variants (via tracktype key) that imply unmaintained, but at least tracktype grade1 to grade3 will typically be maintained (grade2 and grade3 typically at least annually because they are unpaved).


不建议采用这种方式，可读性比较差，直接通过id获取即可。


@prashantchoudhari In a separate commit to any changes. The merge and your changes are in this one commit. Makes it difficult to see what has been changed by you


for you ladies :+1:
you've delayed due to this reason, it should be a gift :smile:


I think a better approach would be to put the default into `compilerOptions` in `compile` and `compileSync` and omit this logic. Or is there something that prevents that?


You're right @pschichtel. README file updated: https://github.com/maestrodev/puppet-sonarqube/commit/0d289aead8e746341554d863b8f774d0f8359531. Thanks for your feedback!


Is the indentation like this on purpose?


Definitely addresses the issue with `settings.PAGE` in the function definition. Just a quick illustration of the potential problem:

```
In [1]: class Settings(object):
   ...:     PAGE = 200
   ...:

In [2]: Settings.PAGE
Out[2]: 200

In [3]: def path_to_object(page_size=Settings.PAGE):
   ...:     print 'Page size: %d' % page_size
   ...:

In [4]: path_to_object()
Page size: 200

In [5]: path_to_object(page_size=100)
Page size: 100

In [6]: Settings.PAGE = 300

In [7]: path_to_object()
Page size: 200
```

/cc @aleksandra-tarkowska


tring -> string


I don't think this link will work, it's unfortunate by you'll have to manually put in the `href`.


deleted:
- eGTEx - mmPCR:
  - GTEX-T6MN-0011-R3A-SM-5CHS4_rep1


Now that these actually are used as defaults, I think the original name (`DEFAULT_PROCESSORS`) fits better.


индент поехал


Derselbe Header muss auch für PR-Preview gesetzt sein, wenn das auch dort funktionieren soll.


E c'hai ragione :)


this line brake compilation when EEPROM is disabled...


In theory it is allowed to add the language qualiifer to any value:
http://dublincore.org/documents/dc-xml-guidelines/
- Each value is a literal string.
- Each literal string value may have an associated language (e.g. en-GB).
But does it make sense to use them on dates, author names, uri etc.
There indicating a schema like xsi:type="dcterms:URI" or xsi:type="dcterms:W3CDTF" might be more useful.
As far as I remember DSpace has not been consistent applying the default language qualifier, sometimes it was applied to all metadata and sometimes only to a subset. At the moment it is a subset leaving out author, dates, uri's, identifiers ... But those running DSpace for a while will have the odd date with a language qualifier.


отступа не должно быть у логаута


Callum wanted a PR to review, as I had requested yesterday for it.


на самом деле это classPath


(copia e incolla frettoloso fece il post sbagliato)


определить сверху а не здесь


may be next time.


переделать все функции как я говорил
$scope.login = login;
function login() {
};


to correct this


Yeah, so the idea behind this function (which I should make more clear), is to identify "admin" users by their user ids. My understanding is that there are some forms with the username "admin" and user ids that are generated by devices, but the ids aren't uuids and don't correspond to any document in couch. (Danny said this is probably only a J2ME thing?).

I'm replicating the logic that the filter currently uses because I don't really know anything about "admin" users, but there might be a pattern to them. For instance, the user ids I've found for admin users seem to be 15 digit numbers. Is this always the case? Can "admin" users own cases? (if not we can get rid of this function entirely).


Thanks. But as previous pull requests formatting should be avoided


cc @zenitraM


finals


removed during refactor


и тут поехал


introducing those spaces is against the code style :)


между ними должен быть div .row


nah -- just stars fell that way naturally ;)  thanks for thumbsups


As I have seen this I took a quick look at the source of the module and realized, that we can't construct the empty-tree...


Négatif, retirer `jquery` change les inputs en simple case à cocher.


strange o.O I did not touch that


отступы ненужны


After a few years this functionality seems to be ignored in the official versions of the plugin. Is there any chance to add it there?


> Nope, just merge it and fix it in the next version if the version supports ffmpeg 3.

then this applies:

> Hmm, I tend to prefer to leave it fail (that is, no < dep) and have an open bug blocking the tracker: that way, ffmpeg-3 wont be unmasked until it is fixed

> It's kinda painful to replace ffmpeg with libav, I'd like to avoid doing so in my system.

you dont need to rebuild everything; replace it temporarily and use emerge --nodeps,


@licaon-kter maybe this was the cause of your port forwarding problems, so maybe you want to try v2.5-2016030801
https://github.com/M66B/NetGuard/releases/tag/2.5


This will want to use the `crate_salt` instead after #31539 has landed.


Done: added len(lru.cache) and PushFront inside lock


I believe this comes from rescaling done in division algorithm. We could probably make it 18 sacrifacint speed of division a bit or by implementing different version of division for operands up to scale 17 and different for scale 18.


We just import. Don't worry. I did it for you in the verify-as-you-type PR.


Could perhaps use a similar trick to 2e5895e3ba1eecdf7b5b2b3ddd3feafe43d9ee92 ?


обертка велл


[**@sarangvyas**](https://github.com/sarangvyas) commented on [Mar 8, 2016, 7:21 AM EST](https://github.com/WillPresley/bitbucket-bulk-add-webhook/commit/884c110fed097d8b4577802c334996ee88ad7767#commitcomment-16560089 "2016-03-08T12:21:56Z - Replied by Github Comment Enhancer"):
> This script works perfectly fine for Team. I had created a test team and then created two repo in it. Running this script on the test team updated the webhooks for the repos. Now I am going to run the script on our primary team, which has 64 repo ;)
> Thanks a ton @WillPresley.

Awesome! I am very glad to hear that it works for someone else haha. I use it for my personal and our company's repos, so I was hopeful that it was working well for teams in general. Let me know if you have any suggestions or improvements, or if BitBucket changes something in their API that breaks this script (it has happened numerous times already). I will be using it as well and updating with anything else I happen to include locally.


ng-if="!registeredUser"


Zero is a valid launch time though we cannot specify Zero in the descriptor as the controller needs a value in +0.5 sec to -0.5 sec range from the current mac time to send the packet with time triggered send successfully.


I had not thought about the other meaning of `close`. Changed to `checkNear`.


Why have a fallback at all?


不建议同步执行，这个是layout同步执行代码块，不建议同步回调出去执行非layout相关内容。


why?


Yeah, I will do that thanks.


I agree that my wording is vague.  On reflection, I don't think this change
should be accepted.

However, to explain what I intended with this commit (and other commits in
the same pull request): I wanted to leave as little discretion as possible
to the project maintainers.  In my view, the forbidden behaviours should be
listed in full in the the Code of Conduct.  Thus, "diminish[ing] the role"
is not a bug, but a feature.


The command name should not be "force-reconnect" (as it is a config reload). Rather "force-reload" or "reload-config" ?


hold -> held


I sent in a pull request with some justification on why it should change.


ejabberd requires the packages for building, so it's a chicken-and-egg issue. `apt-get upgrade` will keep this package back until I can build ejabberd itself.


Well its always going to be connected here because the 2nd thing the server does in main is connect. CLoginServer isn't even created until the database connection is established and if it doesn't the server will just crash anyway with an exception.


建议post出去。


lol


The question is: How is the Python cache generated on Debian then?
Is it remaining after uninstall or not?


ng-show="reviews.length"


The spacing's a bit funky here.


r - what is this?


Thank you :)


So `crossing_1_out` is always equal to `crossing_1_in_or_mid`? Is that just for naming/semantics, or was that not intended?


This was stated in the issue explicitly


Hey, i didn't even share my blog, nice to see someone reading it! About libgtkflow, well, the last commit touching the code vs tests or build system code is d08c77fa6047e4c1a53d0b353bc3491bb217271a from August, sorry if i misjudged it, i'll change it on the next update (today).


What you actually want to do is function that tells wlc to add surface's frame callback to the output's next frame's callback list. You can remove all this, and simply do this kind of function and use wlc_view_set_mask(0);


hey man you could just change a line in until.h
define loop for(;;) to define loop while (true)
then no need to change loop in each file.... well this is also OK.


I wonder if we can check with @dimagi/product whether we can lump in admin users with unknown users and then not worry about it. I suspect that there is very little demand for being able only see specifically admin users.


add return here and do not indent the rest


Fixed.


Good question.
I have my modified package installed in a VM, this evening I'll check.


I did already


Made a minor edit to the comment


слишком много тегов


Maybe I need to rebase this on top


Why are you using `POST`? You are only reading?


обертка


Fixed


I added the TabIndex because the default tab order jumped from the second textbox to the help phone number instead of the sign in button. Gray had pointed it out to me yesterday. It may have to do with front end validation logic.

From:  Jay Hogan <notifications@github.com>
Reply-To:  singlestone/pra-payments-prototype <reply@reply.github.com>
Date:  Tuesday, March 8, 2016 at 6:52 AM
To:  singlestone/pra-payments-prototype <pra-payments-prototype@noreply.github.com>
Cc:  Senghor Kamguia <skamguia@gmail.com>
Subject:  Re: [pra-payments-prototype] Added TabedIndex for ussability; clean up (fc1a39f)

A TabIndex should not be necessary here. The browser will determine the proper tab order based on the order of the markup.

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


The merge conflict is removed.  Just have to wait for CircleCI to complete.

NOTE, why does travis not get called on this fork?  Was there something I should have done to get that testing done also?


отступы


I will rewrite it, after we make some patterns(blinks, color) for led indicator, for different tasks and events of the robot ( initial, move, performing tasks, errors)


`ListMultiMap` instead of `ArrayListMultiMap`


check if this can be put inside load method


Here the idea became simple to understand, but the time consupmtion increased significantly due to several nested iterations over the jobs.


Added `private` keyword to make intended inheritance explicit.


Finally!


If I'm not mistaken you could directly update `angular2`, `rxjs` and `zone.js` versions too, matching the changes made for `ionic-angular`: https://github.com/driftyco/ionic/commit/06d77a26e48391f04f3f9222d0e13f53925b2ed4


Sure. I know vault stores these JSON objects, and I'm not sure if they're allowed to have depth or not, but assuming they do, that's what this test is for.


去去去去


That's what I get for coding at the [Ballmer peak](https://xkcd.com/323/)..


This parameterized version of DepNode already shows up in the previous commit...


The segments FS and GS sizes have been changed recently and shouldn't have been. Those segments are supposed to be able to contain far pointers, thus must be 64 bits sized so it can support UC_MODE_64.

Faulty commits :
https://github.com/unicorn-engine/unicorn/commit/f3dc2522a090e1f8467f5a67571dc3e1bef49786
https://github.com/unicorn-engine/unicorn/commit/4cb43be5bf5a7dee090a6c0c337e1ae0573a679f


@jimis Looks good to me. Does this need to be picked back through 3.7.x?


Yes, obviously that would be better, but testthr is not available in MMQA yet.


The last-written values of local variables of previously-called functions may often be visible on the stack, because nothing ever clears the stack and functions often don't initialize all the memory in their stack frames (for instance, space may be left for alignment, or for variables which are local to untaken branches).  If f calls g, which returns, and then f calls h calls i calls j, then a hexdump of the stack may well reveal local variables of g in the stack frame of h or i or j (on occasion this is very useful during post-mortem).  In other words, we can't rely on function return to lose local variables.  Whereas 2e5895e is much more likely to lose them (not guaranteed, of course), but at the same time a much more costly mechanism.


Did you check with @jtorresfabra if it will not break existing code ? We could use a new version and deprecate the old version with a message in the meantime.


建议改为 `String.format` ， 并且打log就好，不要发toast，


Thanks for the mention @svaksha, much appreciated! Your repo is such a great benefit to the Julia community.


I did consider using the method from 2e5895e, but testthr is not available in MMQA yet.


Indeed !


Why do we have to set `comment` if we could simply make use of `subject.comment`?


这里描述好注释，比如(用英文描述): 提供键盘监听功能，如果需要监听键盘的变化，可以通过以下方式进行监听。


Wow, I didn't realise you could call a closure from its associated type name like that! Jeeeeezzzz!


тегам <a> нужно добавить атрибут href и собственно ссылки проставить. тут просто страницы ссылаются друг на друга + ссылка на html-academy. на главной, кстати, в меню должны быть те же самые ссылки, это не твоя ошибка, это косяк в макете.


Não seria clojure, ao invés de closure?


You can pass a variant constructor as a function? Neat :)


1. I know, I have a plan
2. fair point
3. How does one handle VAULT_ENV? ENV vars can be interpolated into secret paths using the syntax `secrets/db/@VAULT_ENV@/etc:url`
4. good point, I didn't know about this behavior w/ dynamic secrets


Break lines after, say, 100 or 120 characters. Infinite lines are not so great.


I'm going to remove this, we don't use FCKEditor anymore.


Capital `In`


I am thinking of mixed approach between those two proposed by you.
Apart from method list and extraArguments function there caller can also specify predicate function.
With both arguments and predicate function being optional the second approach from proposed by you becomes cumbersome as you have to support multiple cases when one of above or both arguments are provided.

In first approach I do not like the fact that you cannot define it function as single statements.

What do you think about sth like that:

```java
Builder builder = SqlScalarFunction.builder(DecimalToDecimalCasts.class)
    .signature(SIGNATURE)
    .implementation(implementation()
                          .methods("shortToShortCast")
                          .withPredicate(...)
                          .withExtraParameters(...))
    .implementation(implementation()
                          .methods("shortToLongCast", ...)
                          .withPredicate(...)
                          .withExtraParameters(...))
    .build
```

Or with lambda where builder is lambda parameter

```java
Builder builder = SqlScalarFunction.builder(DecimalToDecimalCasts.class)
    .signature(SIGNATURE)
    .implementation(b -> b.
                          .methods("shortToShortCast")
                          .withPredicate(...)
                          .withExtraParameters(...))
    .implementation(b -> b.
                          .methods("shortToLongCast", ...)
                          .withPredicate(...)
                          .withExtraParameters(...))
    .build
```

I will wait with changing that until I verify that `extraParameters` functionality is needed due to performance reasons.


Pois é, parece que [não](
https://developer.apple.com/library/ios/documentation/Swift/Conceptual/Swift_Programming_Language/Closures.html
)

Diogo

On Tue, Mar 8, 2016 at 11:08 AM, Vinicius Carvalho Marques <
notifications@github.com> wrote:

> Não seria clojure, ao invés de closure?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/diogot/equinociOS/commit/64785592f29dc6801b99d967b9255c7e9bfdf095#commitcomment-16562303>
> .
>


Yeah, translation files are so small (~400kb) that there's no advantage to memory-mapping them.  It's far less work (and far more predictable performance-wise) to persistently cache the entire source file.

Writing large files like Undo data are a different story, since bare WriteFile calls are synchronous, and I've had trouble getting asynchronous I/O to play nicely with VB.  I got my old 2005 Celeron laptop out of storage and found that the "write an Undo file" step was taking more time than just about anything else in the program, and I wouldn't be surprised if the same thing was responsible for performance woes on your little netbook.

Writing the files via memory-mapping should help, as the only synchronous part now is writing the files to a RAM buffer, but the actual flush out to disk is now handled asynchronously at the discretion of the virtual memory manager.


Does `dtype_` automatically link to a term reference?


wow questa modifica da' tutto un altro taglio al progetto, bravi bene yee


page-header__main-block и container - разнести нужно эти классы на разные обертки, т. к. сама верхняя полоска хедера тянется на 100% ширины (page-header__main-block), а контейнер центрует контент внутри, и он явно уже. и не будет миксования, которого ты боишься


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


I think this idea can work. Two things come to mind:

1. We should probably specify ```pre``` and ```post``` somehow so we execute commands before or after the general process.
2. Would this make more sense to pass in to the ansible logic instead? Example: if RHEL enable specific repos.


**SUCCESS** - TeamCity HotRod C# Client :: Pull requests monitor - WIN_S2012 [Build 96](http://ci.infinispan.org/viewLog.html?buildId=37156&buildTypeId=HotRodCSharp_PullRequestsMonitorWinS2012&guest=1) for 9ddcf6b389ecdc8144348182d4faf966b2c9cee1
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:15:10


cool, will try it, thanks


It's not something that needs to be checked in freeimage. I'm just saying that it's better to use these functions https://github.com/arrayfire/arrayfire/blob/devel/src/api/c/imageio2.cpp in case we ever plan to support load/save of higher bpp images.


Sure


seems to be an indent problem here too


I'm not sure whether you will need to add a new function. I'm just saying a 3-channel hex should also be a valid input.


Start a new sentence at "To find [**out**] what the issue is, ...".
Also at "For example[**,**] in the above [**case**] ..."


ek olarak tablolar düzenlendi


wait, self.transactions isn't an array, it's an object (map), so you can't actually iterate over it. We really should just need to iterate over all items in workingRequests and call the abortTransaction based on the transactionName of the workItem. Obviously in the abortTransaction method, it should take into account whether or not it's already being aborted and return early as needed.


bien


Its not rendering, its reordering. I'm implying that the enums for types should be reordered to make it more uniform.


:smile:


to remove?


I mentioned this in my e-mail where I discussed the problem:

> The test case should use ephemeral threads to do the allocation and manipulation of the objects, so that there is no reference from the stack of the main thread.

and then I explained why I didn't do it:

> (but not by using ephemeral threads, there isn’t any thread support in MMQA)


mkdir for python


New sentence at "The most simple"


Also the `astype` method does not exist yet.