I think that this is done by [this test below](https://github.com/gasche/ocamlbuild/commit/01df1248caf8e6f055a67c74cc1bc73f931ee1dc#diff-ff037f302dd9f1921ffa8823dec74985L165), but I'll try to test for it more carefully.

In fact I think I use the current behavior in the [bootstrap test](https://github.com/ocaml/ocamlbuild/blob/master/bootstrap/Makefile) where I do several sequential builds from the same directory, with different build directories (`_build_stage1`, `_build_stage2`, etc.). With the proposed patch, that should raise hygiene violation concerns, so I would need to `-traverse` the other build directories explicitly.


Hmm this seems to have broken some behavior:

```js
export const dist = series(clean, css);
```

```
Task never defined: dist
```


кавычки нужны только у тех шрифтов, в названии которых есть пробел


Should we not have an else here to ensure that the host is reset to the default REST endpoint when `fallback` is false?


#### What's this PR do?
Added unit tests to the pet model for the scopes.

#### Where should the reviewer start?
Pet model specs

#### How should this be manually tested?
RSepc

#### Were the tests updated?
Yes

#### What are the relevant tickets?
https://trello.com/c/mXdnnCJT


perimeterRight.push(totalArray[i][j]); ==>

var tmp = totalArray[0].slice();
perimeterRight = perimeterRight.concat(tmp);


В последующих коммитах - все выровнено)


Here's why: series/parallel specify a displayName.

https://github.com/gulpjs/undertaker/blob/master/lib/series.js#L17

The displayName of `<series>` is being used instead of `dist`


a minor comment: the functions in `utils.py` are no more private and should not start with underscores anymore


This is so exciting! That is maybe the best ```.gitignore``` I’ve ever seen! Looking forward to see where this project goes!


No test for a 500 response?  Given the changes you have made here, it would be nice to know that this works as intended and that we don't have a regression in future


Оно же n-арное??


for(var j = totalArray.length - 1; j < totalArray.length; j++) {
 +      perimeterRight.push(totalArray[i][j]);
 +    }

===>

totalArray[ i ][ totalArray.length - 1 ];


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


Had a look and it looks perfect and makes sense.  Before, I wondered what other files in the DOCS folder were for as it didn't make clear sense to me and (as I am new) wasn't sure of their purpose.


Use a set instead of a dict:
```
uniqueHosts = set()
uniqueHosts.add(host)
```


Do we have a standard for the use of brackets around a single line of code conditioned by an if statement? Sometimes you use them and other times you don't. Like not here...


... but here.


So what's the actual problem/fix?


alarmserver


Alright, I'll find someone with a Windows computer to test it out. I'll report back when I know more.


@sebakerckhof it also lets you control your polling interval - so if someone leaves the tab, you can just stop polling entirely. There's also an opportunity to switch strategies on-the-fly - poll when it's not important to be realtime, and switch to a websocket for latency-critical parts.


Полупрозрачная линия для ссылок в товарах через border-bottom.


oh, someone is trying to use `gulp dist` at the command line and it isn't defined


and what about setting tag value?


This commit is incomplete. It is extended by 250dee121a817c520e1adc8c815db5a2cb5c01ae


Hi Fred, I tried to compile the latest version but could not do without the file "f1f2plot.eps" in the repository.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 151](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=771&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


I think the logic is slightly wrong here, it should be `while (retryCountRemaining >= 0);`
Example:
* Assume max retries is 2
* First run against primary host retryCountRemaining = 2, but after failure it's reduced to 1
* while loop checks condition, is true, so tries again reducing retryCountRemaining to 0
* while loop checks condition and fails.

Therefore the retry count is off by one


This completes the fix in 19f48187a61308e3e4514fa8eac649aa82fbd40d


...оптимизируем названия классов...


@carpodaster : I tried to test against `projects_proposable`:
`expect(subject).to be_projects_proposable`
but the (fake) season did not respond_to projects_proposable. Hence this more verbose solution. Okay?


vetor? supporte?


No need to sort here since it is sorted after.


Looks good so far dude, let me know if you need any help


Good point, fixing...


nit: `getNewHostFromIds` is more accurately `getNewUrlFromIds`. otherwise looks fine, thanks!


This should be changed still, it should be `HTTP_MAX_RETRY_COUNT` i.e. you have to use underscores consistently for word breaks


What do you think about making the signature
Uri UriCreatedDelegate(Uri uri);
?
So instead of the `ref` you just return the new uri.
Feels cleaner to do `return otherUri` than `uri = otherUri`


The first comment is wrong, this is intended to be used with python2.7


Minor thing, the naming of "Pr" feels a bit too short.

That said, would really look to Marty to weigh in on the overall direction of approach here, especially as there's scatter-gather'ing involved and whether the overloading of SearchResult is what he had in mind.


I am talking about npc 18369 first of all. Second, the NPC has a respawn timer during which the quest can not be finished. So second player will not be able to finish quest until the npc respawns and gate closes.
Even if he had already started the cast to unlock gate, an error message is supposed to show up, not dbscript. (i.e. handled in the core)
Lastly, the respawn time of NPC 18369 must be == to autolock of gameobject door to avoid this exact issue.


Vault всегда будет хардкодить другие API экономики/прав, потому что разработчик просто не связывается с другими.
В AuthMe кэш реализован слишком глупо.
В майне хардкод лишь из-за того, что он не предусматривает API в принципе.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1355](http://teamcity.monogame.net/viewLog.html?buildId=14705&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Yes, it's always included in the code, see `ZonedDateTimeToCalendar.ftl`:

```
private Calendar ${name}(ZonedDateTime dateTime) {
    if ( dateTime == null ) {
       return null;
    }

    Calendar instance = Calendar.getInstance( TimeZone.getTimeZone( dateTime.getZone() ) );
    instance.setTimeInMillis( dateTime.toInstant().toEpochMilli() );
    return instance;
}
```


Лого - через фон.


Hi, what was the version you downgraded from please? I'm looking to upgrade to 2.5.2 but wonder whether there is an issue with it?


+ @fbbarros


`size_type` is designed to match the return type of  `n_active_cells()` and
return type of this function is just a local counter of number of data vectors which should not connect to
return type of  `n_active_cells()`.


height или лучше padding-top


The string unknown or NULL?


Блок с картой а макете - шире всего остального содержимого.


не отделяй пробелом единицы измерения, браузер не понимает такое значение


Why not let it be a vector type of some kind (or a generic VectorType template)?


Yes. See #295 for lots of details.

0.18 line introduces new semantics for `socket.send`: it does not attempt to finish partial send. Previously, it would loop like `socket.sendall`. Now we're doing minimal work as per documented API. So it will not break proper applications.

On Python3 it's a problem, though. `GreenFileIO` inherits from stdlib `IOBase` which does no buffering. So if you have code like `sock.makefile('w').writelines(data)`, you loose part of data after first `EAGAIN`, that could not be sent right away.

`eventlet.wsgi` is famous piece of art containing such pattern. 0.18.2 contains crutch fix in wsgi. General fix for any `makefile().writelines()` is coming soon.


Jak chcesz zrobić rzeczywisty odnośnik do innego repo, to dodaj plik `.gitmodules`. Przeczytaj [dokumentację](https://git-scm.com/docs/gitmodules).


Issue #186.


I didn't removed it. I gave it an id so it's easer to place and style him with CSS.


เป้ลองรันโค้ดแค่72 record นะคะ เพราะ3หมื่น รอบ่ไหว รอนานมากๆ


"Please" suggests that the user has the option to ignore the advice. That's not the case here.


Ping @AbdealiJK
is this ok??


Подгрузка всех картинок - внизу файла стилей, в следующих коммитах все обращения сгруппированы в одно.


Please keep canonical case for headers.


Yes, make it anything that allows you to use `operator[]`.


Any chance you can have a test that ensures the token TTL is in fact correct i.e. we normally create a token with a TTL of say 10 seconds, check that a request in 5 seconds works, and at 10 seconds fails, this way we know there are no inconsistencies


Also I just noticed the old version would've had double the whitelist size because I wasn't resetting the count :laughing:


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


I think that this would actually be slower since constructing a `DBSeqRecord` runs two select statements on the database.


createNewEntries there is always ture ?


createNewEntries error ?


unknown.  Null is returned when there are no parameters.


It's an oddity that this actually works because derived classes can still overload the function, even though it's private in the base class. They can even make their overload public if they want.

But it's certainly not good style or design.

There is of course the question of what a derived class might want to do in its overload!?


Huh, strike the updateInterval comment above.

Anyway, see https://github.com/SignalK/signalk-server-node/blob/accumulator-interface/lib/interfaces/accumulator.js for an example of how to add an independent listener that is not dependent on websocket connections.


This commit breaks compilation at least on Linux, due to DebugPrint. It should be something like:

`DebugPrint(
							"BUG: RangeTargetFinder::FillBadGood.Compute out of range. ,"
							"size: %d, pos: %d, "
							"x: %d, xx: %d, y: %d, yy: %d,"
							_C_ size _C_ pos _C_ x _C_ xx _C_ y _C_ yy
						);`

and one more DebugPrint too.


You do now :)


Sure, I'll take care of it. Just wanted to make sure you know about it for the future! :smile_cat:


ah, probably not.  sorry about that


Please do not push to master. use:

'git checkout dev'
once to switch your active branch to the development branch.
then whenever you want to push, use:

'git push origin dev'
after commiting.

Once it is determined that dev has necessary changes and is functional, we will merge them.


huehuehue x) I m writing a "fix" for that => aka:  try/catch


I was just going to do a PR for this. I already pushed fixes for `pollen` and `css-tools` that were needed. `racket-lang-org` now builds fine for me.


Does this have to be a member variable, or could it also be a local variable in one of the functions?


Mind linking to www.ably.io/documentation/rest-api/#message-history instead?


Смотри, для уменьшения количества кода, шрифтовые параметры модно указывать вот так, я считаю так читается более наглядно, когда всё в 1 строке: font: italic bold 14px/24px "PTSans Narrow", sans-serif. И сразу на заметку:когда указываешь шрифт, необходимо после основного указать, заменяющий его шрифт - допустим Arial, а потом семейство шрифтов. В реальных проектах указывают главный шрифт, шрифт в каждой операционной системе который наиболее подходит к основному, семейство шрифтов - может идти более 5 шрифтов


The variable is a vector, not a pointer, so the name is misleading.


I think the "Code for making a config from a table" is actually the "Code for making a table from a config". These 2 headings have to be switched.
And your code for making a config string from a table doesn't work if the value is a table. Here's a fix:
```Lua
function generateConfigString(configTable)
  local s = ""
  local tk, tv
  for k, v in pairs(configTable) do
    tk, tv = type(k), type(v)
    if tk == "string" or tk == "number" then
      if tv ~= "string" and tv ~= "nubmer" and tv ~= "boolean" and tv ~= "table" then
        error("Invalid type for value: Expected string, number, boolean, table; got "..tv)
      end
      if tv == "table" then
        tv = tostring(textutils.serialize(tv)):gsub("\n", "")
      end
      s = s .. tk .. ": " ..tostring(tv) .. "\n"
    else
      error("Invalid type for key: Expected string, number; got "..tk)
    end
  end
  return s
end
```
Also, IMO, it'd look better if you make both codes a function, like I did in my fix above.


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


contributin :)


Ah nice there is a fix for this. Maybe you want to add the  $utf8_encode in there too in case someone needs that for subarrays too.


As above, please use www.ably.io/documentation


Для html - то указывать не надо. Ширины ему не нужны, а margi и padding - у него нет


I filled out the schemas and documentation further in 46aa39e.


Yes, I agree.

But the thing is that you are actually changing the variable. The compiler doesn't notice, but you change the iterator in `++cell`, and you change the object pointed to when you set the user index.


You're probably right, since there are other kinds of requirement that would break `any<P>: P` conformance if added later, such as any `static` or `init` member.


Actually, without the bug fix, I think what would happen is we would call import, it would try to delete_all under the covers, and that would result in an exception before it got to the point where it imported our new data. I think then records.length would actually be 33 (or perhaps even less, as I think the delete_all would possibly delete the first 25 records, and then fail)


done


changes in readme contents


The real issue is that it's still the same renderer process with the same command line args. I created #672 to handle this on the renderer using rules as in https://developer.chrome.com/apps/tags/webview#method-addContentScripts


But it is not like the real agar.io


ha. that and you have to load the library before you can call the function ;)


I have added everything I am pretty sure, but it's still not working? I never remerge? What's happening?


(Explicitly declaring `extension any<P>: P { ... }` would also give a place to fill in any requirements that don't fall out from the type system too…)


Fair enough about 2.2.


I'm viewing on mobile so I'm a bit confused… you deleted whole files?


Can't do that, that would give all the power to delete repos:
https://help.github.com/articles/repository-permission-levels-for-an-organization/


I feel so sorry for @Luka967 , dealing with errors while being tired :/


moving demo files to their own repositories


I think Jordan just reviewed it : )


Thanks for adding some rationale for making the commit. It makes it easier to follow your work. Additionally, I believe the l1-cache-size is for the size of the L1 cache on one CPU.


Will this return an error if `offset` doesn't match the last offset? If so, is this racy for eventually consistent object stores?


Should i try to find out how implement merge? But i dont know how the merge mech works but i can figure it out


@DerrickMalan you can see the changes I made from here. Let me know if you need any other changes in the future. I'll make sure to make them here so they are saved permanently.


Maybe in the future we should put that behind a feature flag (if it comes back)


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1514 of 1515 tests passed, 0 test warnings)

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


Oh, whoops. That's a line-by-line performance profiler's log output


Hi,
any idea why I'm getting this error:

> library(oncoprint)
> data(tcga_brca)
> vert_x <- theme(axis.text.x = element_text(angle = 90, hjust = 1, vjust=.5))
> oncoprint(tcga_brca) + coord_fixed() + vert_x
Error: Unknown parameters: width, height

Thank you


Whoops, Thanks for fixing.


No problemo

> On Feb 8, 2016, at 8:10 PM, Connor Haight <notifications@github.com> wrote:
>
> Whoops, Thanks for fixing.
>
> —
> Reply to this email directly or view it on GitHub.
>


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


yeah, tomorrow I'll change that.


I understand,
see: vte, qconf, bomi...  the team "users" has admin permissions


vlw lipz


I was not sure how the `baseurl` should be set when redirecting from custom url.


we are not real agario >.> we are Ogar! :P


They all retired? Need to move moshpit up and as I know he is map manager now, right? Fix it please


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (963 of 963 tests passed, 0 test warnings)

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


Não conhecia do metodo allMatch
codigo ficou bonito hehe


zip files for unix derivatives? ;)


beep


There's a more succinct way to do this:

`return re.match(r'^[^@\s]+@[^@\s]+\.[^@\s]+$', visitor_email) != None`


HUeuhe, ss, mas ta "errado", me confundi, o método é pra verificar se tem algum item no inventario, ai ele ta verificando se NÃO tem nenhum. Já arrumei e comitei.


Yes, good idea. I'll add it.


not quite possible because of the `if let`, but I reworked it a bit. let me know which you prefer.


this is also an options:

```rust
    return if let Some(write_mode) = matches.opt_str("write-mode") {
        match WriteMode::from_str(&write_mode) {
            Ok(write_mode) => { config.write_mode = write_mode; Ok(()) },
            Err(_) => Err(format!("Invalid write-mode: {}", write_mode)),
        }
    } else {
        Ok(())
    }
```


LGTM :+1:


다이나믹fsync에서 뭔가문제가있는듯합니다. 이거추가후 뭔가 cpu운영에서 비정상적이고 배터리가좀이상하더군요 이건아직시기상조인거같습니다.


I have checked with Chris, he should be giving me all the graphs that we need to include in the GUI. Until then, just sit tight and be familiar with the rest of Telemetry Frame. Also this is just an example to comment others' code.


Could be optimized as

```rust
Stack::allocate(size, false).map(FixedSizeStack(stack)
```


Consistency is king. If you want I can condition it for tarballs if you desire. =)


The same as above

```rust
Stack::allocate(size, false).map(ProtectedFixedSizeStack)
```


For issue #1


@JustAkan 어떻게 비정상적이시죠? 명절 동안 쓰면서 문제는 없었습니다.


Don't use `.unwrap()`, replace it with meaningful error message, such as

```rust
FixedSizeStack::default().unwrap_or_else(|err| panic!("Failed to allocate stack {:?}", err)
```


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1341](http://teamcity.monogame.net/viewLog.html?buildId=14619&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


The same as above


And i finally got it right! I didn't realize that it takes few moments before changes in this file are used. So i tried "fixing it" not knowing, that it already was fixed.


I reverted it because disconnect button is not work.
http://imgur.com/MUfef4x


Haha gotta love Python!


Would I need this if its not compiled in?

https://github.com/inconshreveable/mousetrap


Here is the new outputs method


This protect page can only prevent stack overflows (at least for safe Rust), but the program is still vulnerable to stack underflows. So I suggest to add another function to protect the top of the stack.


It's better to use `git revert` instead of merging another PR.


This is how you make a separator argument


The new taxonomy method


Use the new `content_by_lua_block` form.


The output class


You can get a WorkflowJSON object from the runner, this describes the workflow that is being run


Index of the current step being run


Good catch. Due to the way that data is prepared, the sizes of images `[h,w]` read in are always fixed. Also, I always set `opts.imageSize + opts.border` to be equal to `[h,w]`. So `factor` was always 1 for me.

You edits make total sense to me!


did you intend to strip out the homedir stuff?


Results from previous steps, note that this uses the OSResult class which allows you to get registered attributes using the Attribute class.  If this interface is too clunky we can improve it.


fixes #1102


this is redundant (word.length == 1 || word.length == 2 || word.length == 3) should be
(word.length > 0 && word.length <= 3)


This would tell your measure if the user wants units in SI or IP, note the runner is not currently passed to the arguments or outputs methods, only to the run method


All these function could be restricted by a trait, such as

```rust
trait StackAllocator {
    fn allocate_stack(size: usize) -> io::Result<Stack>;
    fn protect_stack(stack: &Stack) -> io::Result<Stack>;
    fn deallocate_stack(ptr: *mut c_void, size: usize);
    fn page_size() -> usize;
    fn min_stack_size() -> usize;
    fn max_stack_size() -> usize;
    fn default_stack_size() -> usize;
}
```


New class for storing argument values in the WorkflowStep object


just to clarify what's the difference between extras and extra_hour_records?


“In Natron, we also use OCIO to linearize the transfer function when decoding images and videos, and most videos nowadays seem to use Rec.709, Rec.2020, or one of the constructor-specific so-called "log" curves.”


WorkflowStep captures data needed to run a single measure, stored in the OSW


WorkflowJSON is a wrapper around the OSW format


@henryHorsey @nllong @brianlball @asparke2 @DavidGoldwasser @kbenne @rpg777 can you guys take a look at these proposed interface changes for OS 2.0?


-echo "Moving Fastboot"


This line should go away entirely.  When we define a tag here we can't use the .foodcritic files, which is what we should really be doing


output
```
$ go build && ./wercker  b --pipeline test --enable-volumes
--> Executing pipeline
--> Running step: setup environment
Pulling from tcnksm/gox: 1.5.1
Digest: sha256:2f7d5fc7a82f9c3bdcd03cf0eee343d0364020606d1831636fe25d38853bd47c
Status: Image is up to date for tcnksm/gox:1.5.1
$CODE_PATH
ERROR Session finished before sending commands: [mkdir -p "/pipeline"]
ERROR --> Step failed: setup environment 3.36s
context canceled
ERROR context canceled
WARNING Box container has already stopped.
FATAL Exiting.
```


These changes are to implement the proposed changes in https://docs.google.com/document/d/1QfMDboZjw3vA1pYnWeF_Xs44uSeNwHhIn8uaIQnVDjk/edit#heading=h.uqnahv8v5sl9


you have to put your specified oauth redirect url in your todoist developer app in the place of the text that says 'yourredirecrurlhere'


You could look in the Info.plist file inside the application bundle, but
that's not much different.
On Mon, Feb 8, 2016 at 5:32 PM Joe Lencioni <notifications@github.com>
wrote:

> Is there an easy way to find these bundleIDs (com.googlecode.iterm2)?
> I've been opening the Hammerspoon Console and doing something like
> hs.application.find('hint'):bundleID(), but it seems like there must be a
> better way.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wincent/wincent/commit/844f78b9eee9823d6d3c5bd7d1325a8eef980bdf#commitcomment-15967052>
> .
>


I ran into this error when trying to run the current PR under our current package versions:
```
numpy.dtype has the wrong size, try recompiling
Traceback (most recent call last):
  File "WolfSheep.py", line 18, in <module>
    from mesa.datacollection import DataCollector
  File "/Users/fje630/Projects/code/classes/mesa/mesa/mesa/datacollection.py", line 36, in <module>
    import pandas as pd
  File "/Users/fje630/Projects/envs/mesa/lib/python3.5/site-packages/pandas/__init__.py", line 7, in <module>
    from . import hashtable, tslib, lib
  File "pandas/src/numpy.pxd", line 157, in init pandas.hashtable (pandas/hashtable.c:22953)
ValueError: numpy.dtype has the wrong size, try recompiling
```

After research it seems like updating package versions is an easy fix.
@dmasad, there is no action you need to take at this time. Just giving you heads up.


This actually shouldn't be necessary since HTTPS everywhere rules should be applied to the entire URL, not just host. Ex:

```
<rule from="^http://(?:www\.)?politico\.com/(?=favicon\.ico|.+\.(?:gif|jpg|png)(?:$|\?))"
        to="https://a248.e.akamai.net/f/248/1/1/www.politico.com/" />
```

means that `http://www.politico.com/blah.gif?param=1&param2=2` should be rewritten to `https://a248.e.akamai.net/f/248/1/1/www.politico.com/blah.gif?param=1&param2=2`. The `from` field is a regex whose captured value is replaced with the value in `from`.


Is this a typo?


In case you didn't see it in the #build channel in Slack, this line is considered too long by CheckStyle.  You should shorten it.


The reason I have two sizes here `OBJECT_SIZE` and `IMAGE_SIZE` is to leave some space for random shifting during training. In this sense, I will suggest to keep it [224,224] so that we have some room for data augmentation vertically.


Typo?


Sourav Bhowmik, 2/8/2016, Lab 1,


I think we talk about different thinks. On my windows machine the `posix_kill` would be executed (in detached mode ofc) when `killPid` **and** would throw an error, because I dont have this function in windows.


FYI - you need to bump this version from 23805 to 23808. Same for the .cmd file above.

When moving DotNet versions, you need to find and replace project.json files, but these two scripts as well.


aku edit untuk beradaptasi dengan Mxit app.


Shouldn't we just leave the URL alone? It's still valid.


Nevermind - I see you did it in a later commit.


I want this one to be the active one.


I cant seem to find the oauth redirect Url. Where would I find that?

Thanks for your help


Failed to kill technomagi?!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1341](http://teamcity.monogame.net/viewLog.html?buildId=14619&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:19:13


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1342](http://teamcity.monogame.net/viewLog.html?buildId=14623&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


++
and google sponsored links work now right?


Ok, will try tomorrow. I remember we had a problem with news.yahoo.com. As it did not give us https for http://news.yahoo.com but did for http://news.yahoo.com/.... which is wrong. Maybe it is fixed already, will try.


You will have to do to todoist developer web page. Create an app there. And
then you will get things like client id. Client secret from there and also
the oauth redirect url that you specified there -  it can be any url -  the
same you will have to specify in the file - my-message-sidebar.cjsx

On Tue, Feb 9, 2016, 8:11 AM thejewishblt <notifications@github.com> wrote:

> I cant seem to find the oauth redirect Url. Where would I find that?
>
> Thanks for your help
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/anopensourceguy/TodoistN1/commit/46e5097237f31868ccbee252bf7c42311341728e#commitcomment-15967838>
> .
>


Yes, they do.


dfdsafasdfad


asdfasdfasdf


@SergeyZhukovsky that's probably a problem with the upstream rule, not with the implementation. there are tens of thousands of rules so there will always be rule bugs. it's not worth our time to fix them unless the site is visibly broken


Yes, it was a comment that came from the default `namespace.conf`


The indentation gets lost when you scroll the list.


Very interesting. `factor(1)` > 1 for [256, 224] input, so I didn't realize that was set equal before... You mentioned before the reason for [256, 224] was that the ankle keypoints lie outside the bboxs. Then, did you notice a performance difference between [256, 224] and [224, 224] inputs, and do you recommend using one over the other? If the performance is similar then I should probably shift back to [224, 224] since that fits well with you pre-trained models too.

And I guess I confused this [256, 224] vs. [224, 224] input thing with `OBJECT_SIZE` below.


@ilexp has some great ideas!!!!!!!!!!  And I love the enthusiasm from @amulware !!!!!!!!!!!

I agree a lot depends on if we hear from @thefiddler soon.  I've done everything possible to contact him on github, https://launchpad.net/~stapostol & even searching for him on facebook, wo we may hear back from him sometime in the next month or so.

I think it would be better to revitalize OpenTK than to create a spin off project - there is value in name recognition.

I agree 100% we need to be "person-wise redundant"!!!!  No organization can function w/ a single point of failure if 1 person quits.  It does not matter if you're a fortune 500 company or a small open source software project like this!

I think when we hear from @thefiddler  he'll be reasonable & probably give administrative privileges to @andykorth + @mrhelmut and maybe a few others.  I really hope we hear from @thefiddler soon.

I think the most important thing is to use social media, like facebook, twitter, youtube videos to get out the word about this project.  http://www.opentk.com/forum is not a bad forum per-say (although the issues link is broken) - the real problem is almost no one knows about opentk.com.  The way Google works is they create a matrix of every page on the internet based on how many other websites link to your website & use Eigen vectors to rank how popular your website is.  The bottom line is if no other webpages are talking about you, it's almost impossible anyone will find you in a search :(

I know @amulware  is working on a Awgraphics, great new C# platform build on top of OpenTK.  If @thefiddler gets back to us & grants admin privileges to a few other people, Awgraphics could actually be part of OpenTK (If @amulware wants that).  I got a message from @thefiddler last month saying it would be welcome for me to add unit tests & some new math operators to OpenTK (of course that was before @Frassle left).

I love @ilexp's ideas & we could really use @amulware's support, but I think we should give @thefiddler at least a month to write back & we can use that time for planning.  If we don't hear back from @thefiddler for months, then maybe this will take the form of adding OpenTK's + @amulware's Awgraphics + some other features conglomerating into one new project.  But I think keeping the recognized OpenTK project alive & creating new projects connected to OpenTK would be better (Not to mention more webpages linking to eachother increases your Google Eigen vector score LOL :))


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


Gotcha


Time to get a JEI Handler going ;)


Great. Got that setup but nothing clicks or takes place after can delete the name or add todo ist


copy/paste problem.


Progress tracked by #6.


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


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


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


Not quite the same (sorry about not having a place to put some of these comments) but the theme chooser is getting cut off when I tested it on my 5s and it's also not scrollable in landscape when it needs to be (which also affects the 6/6s-sized devices).

![img_1744](https://cloud.githubusercontent.com/assets/68303/12906424/12f38038-ceae-11e5-8bfd-5c0431f4e182.PNG)


You are correct, I don't preserve the element if it contains git-flow defaults. We can change this behavior.


did you run `git rebase master` on your own branch?
It is recommended to follow the convention using command line to merge PR fast forward only (with only one parent), instead of merging through the merge button on the github site.


+1 this breaks content provider for new version of dbflow


> copy/paste problem.

BTW you can get this information from ServerInfo API itself there is no reason to add GetUIVersion.  It is a comprehensive list.

You can add another field

```
// ServerInfo - server info reply.
type ServerInfo struct {
        MinioVersion  string
        MinioUIVersion string
        MinioMemory   string
        MinioPlatform string
        MinioRuntime  string
}
```


Guidance is that you can name EITHER on what is tested or on associated defect number. I actually prefer the descriptive format, if it is accurate and exhaustive.


으흠.. 제가 서스팬디드드라이버를바꿔서그런지모르겠지만. 뭔가배터리가빠르기닳아가고 빅코어가비정상적으로동작합니다
제가 가버너가달라서그런걸지도모르겠지만. 큰문제는 딥슬립이오래작동할시 데드슬립이됩니다. 다이나믹fsync문제일거같아서 한번문의드렸습니다


Mean fixes the content provider that is -)


Wow, I had this change locally, then released v0.9.0, then never pushed this up. How embarrassing.


test


That's quite a wild function. How did you ever figure it out?


Manuscript -- Epigenetic and genetic regulation of tissue-specific gene expression


@jingyanw The way I included more content at the bottom is through `base_size` in the data preparation function. Based on that, choosing `[256,224]` will make it less distorted in terms of the aspect-ratio comparing to `[224,224]`. While, in terms of the content, they should be the same.

In practice, I did not see improvements by making the aspect ratio less distorted. I will suggest you stick with `[224,224]` at least for now.

Yes, the pre-trained models are trained with `OBJECT_SIZE=[224,224]`.


The Update function needs to be able to overwrite the existing manifest.xml file without triggering a re-reading of the file while its in the process. If the property notice is used, it will trigger _Load to be called while we are in the middle of updating the XML file. It became necessary to change notice to use _notice instead to prevent this.


file:read_file_info is more efficient than filelib:file_size to retrieve the size of file when you already have a file handle.


The Update function needs to be able to overwrite the existing manifest.xml file without triggering a re-reading of the file while its in the process. If the property default is used, it will trigger _Load to be called while we are in the middle of updating the XML file. It became necessary to change notice to use _default instead to prevent this.


I found it on the Google, of course. It's a very effective use of list comprehension and recursive calls.


@yosukehara ping


The Update function needs to be able to overwrite the existing manifest.xml file without triggering a re-reading of the file while its in the process. If the property remotes is used, it will trigger _Load to be called while we are in the middle of updating the XML file. It became necessary to change notice to use _remotes instead to prevent this.


What version are the tests run against. I don't think we have Copter 3.4 built yet?


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1022.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=69039&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:18:09


Okay, I suspected that might be the case. I just made these changes so that CircleCI wouldn't break while using the explicitly vendored version.


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.31.0](http://build01:8888/viewLog.html?buildId=14169&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) is now running


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.24.0](http://build01:8888/viewLog.html?buildId=14167&buildTypeId=OpenSourceLibraries_Sourcerer_CI) is now running


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.24.0](http://build01:8888/viewLog.html?buildId=14168&buildTypeId=OpenSourceLibraries_Yaclp_CI) is now running


Для этой операции есть стандартный метод toString()
Стоит переопределять именно его


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.31.0](http://build01:8888/viewLog.html?buildId=14169&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:28


This means that you can't have multiple such objects operating on the same triangulation. Better construct this kind of map internally in this object without setting global state.


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.46.0](http://build01:8888/viewLog.html?buildId=14170&buildTypeId=BlogMonster_CI) is now running


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.24.0](http://build01:8888/viewLog.html?buildId=14168&buildTypeId=OpenSourceLibraries_Yaclp_CI) outcome was **SUCCESS**
Summary: Tests passed: 20 Build time: 00:00:23


oh yeah, let me update it


У body padding и так равен 0, так что это лишнее. min-height не надо. У body min-width: 940px + 10px = 960px


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.29.0](http://build01:8888/viewLog.html?buildId=14171&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) is now running


equals, hashcode an toString сам писав? Мужик!!!

Рекомендую подивитися в сторону сторонніх бібліотек.
https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/builder/package-summary.html
EqualsBuilder, HashCodeBuilder, ToStringBuilder


Переменная присваивается сама к себе - зачем?


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.46.0](http://build01:8888/viewLog.html?buildId=14170&buildTypeId=BlogMonster_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:00:27


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.29.0](http://build01:8888/viewLog.html?buildId=14171&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) outcome was **SUCCESS**
Summary: Tests passed: 21 Build time: 00:00:21


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


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.50.0](http://build01:8888/viewLog.html?buildId=14173&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


Какое отличие класса от структуры?


@mbutterick Is using `sugar/unstable/string` the right repair, or did you have a better one?


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.24.0](http://build01:8888/viewLog.html?buildId=14167&buildTypeId=OpenSourceLibraries_Sourcerer_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:01:17


works


5452-controllerWaterCoil-IP-units (MatthewSteen) - x86_64-MacOS-10.9-clang: OK (2055 of 2055 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


Can you please amend the patch to add your explicit sign-off on the contributor agreement, as described here:

https://github.com/sassoftware/python-keyutils/blob/master/ContributorAgreement.txt

Thank you!


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


We talked about this one IRL.  For now / expediency, I decided to keep the granularity at which this is tested at the same level I already had it.  I was a bit worried about losing some of the etag-level comparison that I was getting previously by moving those tests to a lower-level and wasn't sure I wanted to create more duplication in the tests at this point.  We can revisit this later if it proves to be a problem.


Yes, that is exactly the repair I was going to push.


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.50.0](http://build01:8888/viewLog.html?buildId=14173&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:58


Did someone forget to fix the conflicts??


[^/.]+ is not quite right I don't think unless we can be 100% sure that '.' is not a valid character for these. You could add a second check to see if '../' is present if that would be easier.


Whoops, did not mean to commit that second file.


should also be removed.


ditto.


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (2057 of 2058 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


Hm, when did this bug get here?


This should goes to src/test/src


Typo here. It breaks! Could you fix please?


I reworked this to handle `RubySymbol` directly, falling back to throwing an Exception rather than calling `.toString` if the object can't directly be coerced by Clojure's `keyword`' function into a Clojure keyword.  This was added in 46da66f.


so then grab cm toybox commits or aosp ones ;)


Не найдет картинку! ты сейчас находишься в файле css, тебе сначала надо подняться из файла ../img/main-background.jpg


I think we should wait to put JavaScript in until all of the templates are finished.  Also inline JS is :disappointed_relieved:


missing plurial


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


The three `is` assertions here did not change as part of this commit.  I moved the `etag` function up a level so that I could use it in a separate `testing` block, which affected the indentation on the rest of these assertions.


I fixed it. Forget it.


There are three of these "since the future" tests.  The first had ```+ 1.hour```, and the second and third had ```+ 1.minute```.  I figured I might as well make them consistent, and if we are going to change them might as well choose the one we can be sure there wouldn't be an issue with a request took over a minute to go through, which then invalidates the purpose of this test.  Seemed like an hour in the future was a safer choice.


missing two plurials .. sigh


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


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


```
config MMC_TEST
	tristate "MMC host test driver"
	help
	  Development driver that performs a series of reads and writes
	  to a memory card in order to expose certain well known bugs
	  in host controllers. The tests are executed by writing to the
	  "test" file in debugfs under each card. Note that whatever is
	  on your card will be overwritten by these tests.

	  This driver is only of interest to those developing or
	  testing a host driver. Most people should say N here.
```
I think it's better to keep this disabled...


but for the functions thing, it is not an API, for the pastebin api I used functions.


Unlike the previous ones, this is a new test for this commit.


Re hardcoding> I guess if that is the decision, that is 'ok' for now, but I strongly feel this is going to be a maintenance burden and something we'll want to address before 16.04 is released. I also worry if we continue hardcoding we will be entrenched.

Thank you for explaining the two sides-- I forgot this point in reviewing the merge since it didn't jump out at me that this would happen. Since this is the case and since this is implied but not completely clear unless reviewing more of the code, can you add a test case for the consumer side to make sure it has the specific pin in security policy instead of the glob? That way we can futureproof ourselves against side-effects on this security-sensitive code.

Re more permissions: sure :)


Этому классу в html необходимо добавить класс .clearfix::after {content:""; clear:both; display: table;} - а то он у тебя схлопнулся по высоте, так как все элементы внутри него float-ные http://prntscr.com/a158qv


Sounds good. Maybe even make that another question? Something like this perhaps:

#### Why isn't Wetty working with IE?

[This fix](http://stackoverflow.com/questions/13102116/access-denied-for-localstorage-in-ie10#20848924) has been known to help some users.


Ну а, что я поддерживаю.


@nmn We can't see a way to support this and the example above so I'm going to be backing this change out and writing some regression tests.


> Each resource can have its own set of exception.

I would prefer to define a list of exceptions that can resources package raise so it will be easy to (re)use. That was the purpose.


Did this in 254c1c6.


See comment above.


Thanks for the help. Everything just rebased just fine http://strava.github.io/iD


@AbrarSyed Would it be possible to get setDirty() back? Or is there another way I can do this?


Именно по причине проблем кодировок, комментарии обычно пишут на английском, пусть и с ошибками


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1355](http://teamcity.monogame.net/viewLog.html?buildId=14705&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:19:04


Formatting is off throughout this method


I updated a few of these in 749b271.


No, each provider will have its own responsibilities.


padding -лучше записывать вот так padding: 60px 0 40px;


Done in 749b271.


Did this and the next one in 749b271.


Отступ лучше давать сверху идущему элементу - вниз, чем ниже идущему вверх


I would have moved the parse_visit_data to this file and only kept the rune/monster/etc.. functions in SWParser.


Did this in 749b271.


Same here, parse_login_data would make more sense in the plugin than in the SWParser now that it's been moved to a plugin.


Update работает, но что с чтением и отображением?
Как думаешь, есть ли вариант записи в файл, не читая его каждый раз по новой?


Finished #10


Где идут float-элементы, их родителю необходимо добавлять класс .clearfix


We are designing a package...


Made some changes for this in 254c1c6.


Igore izvini sto ne stigoh ono da dodam, malo me posao ometa trenutno.
Jedno pitanje, video sam na meetup-u da su imena predavaca i teme
linkovani. Isto treba da su linkovani i na .md fajlovima?

2016-02-08 22:45 GMT+01:00 Igor Spasić <notifications@github.com>:

> izbaci x ispred svih imena, kako bi strana bila cela
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zargaroth/website/commit/3658568e9ab2a06f5707c5d755d4348ea2a0e87e#commitcomment-15961661>
> .
>


Yes! For using os.path.join, but why add the '/' at the end of the 'plugins' string ?


Зачем ограничивать ширину блока????


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


are all the `this.`s necessary?  If so, maybe we should rename the local variables.  If not, please remove them.


Зачем давать min-height?


valid recommendation - I suggest we bring this in


In the interest of time, I decided not to pursue this one further.  I'm open to revisiting this later if we think it would still be valuable.


This was a proposal to deal with the design. The names are up to you and they heavily depend on the implementation which is not here....


Ширину давать не надо. Это блок. Блок - тянется на всю ширину родителя, родитель - container - с шириной 940px.


It's almost ok when just having `window.setTimeout(sendChunked, 0)`...  :confused:


Это лишнее!!!


Of course, this was meant to be a design proposal....


Лишнее


ana u suck


Ma hangshi Karin


Is there a reason not to use?
1) rxjava 1.1.0
2) rxandroid 1.1.0
3) rxlifecycle 0.4.0
4) rxlifecycle-components 0.4.0


What sorcery is this? Why not just unittest?


Лучше float: right и тогда в блоке .easy-search можно убрать margin-right: 20px;


s/succesful/successful/


Okay, okay, you were right on this one ;)


Because it is part of settings. You are setting up an object which is encapsulating all the job which is needed to be done. Where do you want to store results? /dev/null? /home? /root?


checkArgument only supports `%s`.  Can you check the all of these commits for uses of `%d`?


-center img
maybe add color to text


Можно убрать если в блоке .right-column - задать float: right


There are several error handling improvements such as these in this commit. These changes seemed to fit in well   with the theme of improving the messaging code.


Wprowadziłem "lekkie" zmiany.. :D


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1525 of 1525 tests passed, 0 test warnings)

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


When you find yourself repeating code that is already somewhere else in the class, that is a sign that you ought to factor that common code out into a private method that is then called both places. Android Studio can make this easier using the "Refactor" menu -- just select the code you want to extract and then from the "Refactor" menu select the "Extract" submenu and then "Method ..." (There is also a keyboard shortcut for this.) I'd be happy to show you this in class if you can't figure it out.


I wouldn't use 'args' as argument to start_proxy_server, it would make more sense for it to :

     def start_proxy_server(port, debug_level="ERROR")


@mattnenterprise ok I changed the sentence and amended the commit.


Начало хорошее, но медленно. Можешь барбершоп не делать, главное сделать личный проект


“Made appropriate changes.” is not a useful commit message. Each commit should be a distinct change that should be easy to briefly describe in the commit message, e.g. “Removing Unused imports”, “Redundant public modifier”, “CountBy & test file references missing from VS projects”, etc.


In the interest of time, I decided not to pursue this one further.  I was still a little on the fence about the idea of losing some of the etag-level test coverage by moving all of these to lower-level tests and wasn't sure about how much duplication among the tests it would make sense to have if we wanted to keep these but add other lower-level ones that cover a lot of the same stuff.

In 46da66f, I did at least add one more test to cover the case of an inability to coerce a key in the source map from the call into JRuby back into a keyword.


Да, как-то странно выглядит... Исправил, заодно расставил правильные пробелы для однострочных правил.


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


@Gi-lo Thanks.


First "commit".


I would set the debug argument as a string instead of a boolean, in case someone wants to do --debug="INFO" for example.


this


Garbage collector? I don't want to use config parser inside a package which encapsulates basic functionality. Do you want to have a config for everything?
https://en.wikipedia.org/wiki/Garbage_collection_(computer_science)


Можно создать файл .gitignore в него записать .idea/ и тогда лишние файлы в репозиторий попадать не будут.


Much better!

You could also do something like:

    if i % 3 == 0 and iif i % 5 == 0:
        print 'FizzBuzz'


not sure why you remove response but not request, if you wanted to move them both into the derived class.


@RSsimranjeetsingh
Hi Simran,
We add an exponentialRetry filter to the client by default. You can disable the default retrypolicy by setting
options.noRetryPolicy = true in the client constructor. Check this [link](https://github.com/Azure/autorest/blob/master/ClientRuntimes/NodeJS/ms-rest/lib/serviceClient.js#L29). All the clients that are generated with "NodeJS" flavor are children of the ServiceClient class in the (ms-rest) clientruntime.

At the same time we offer an option to add your own filters. We will add those filters to the request pipeline.
Check this [link](https://github.com/Azure/autorest/blob/master/ClientRuntimes/NodeJS/ms-rest/lib/serviceClient.js#L24). So you can create your custom retry filter (may be reuse our retry filter with different default values) and add it as a filter in options.filters array to the client constructor.

Hope this helps.

Thanks,
Amar


`TKey` is more appropriate that `TResult`. This is not a projection of the source elements, but the *projected keys*.


Hi Sou-Cheng,

Try running TraubMemorial.m, which I just uploaded.  I want to avoid putting binary files on the repository.

Best regards,
Fred

Fred J. Hickernell, Professor and Chair
Department of Applied Mathematics, Illinois Institute of Technology
RE Bldg Rm 208, 10 West 32nd Street, Chicago, IL 60616
hickernell@iit.edu <mailto:hickernell@iit.edu>, www.iit.edu/~hickernell <http://www.iit.edu/~hickernell>
Office: 1 312 567 8983   Cell: 1 630 696 8124

> On Feb 9, 2016, at 1:48 PM, Sou-Cheng T. Choi <notifications@github.com> wrote:
>
> Hi Fred, I tried to compile the latest version but could not do without the file "f1f2plot.eps" in the repository.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/xtong5/LocalAdaptiveAppxOpt/commit/a124adbf46dd100adbb0109e616b101207cacdcc#commitcomment-15987493>.
>


Do you use this constructir? If not it is not needed


thanks, this works great!


TarballFilesystemStorage knows how are files stored and how to retrieve them.


`keySelector` would be a better fit here than `projection`. It's good to look up existing LINQ methods that work on keys for inspiration & for consistence.


I don't think you want to limit the email to `APPROVER_ROLE_INDICIES`.  At least my mental model of this is that if an entire group has the ability to grant a permission, anyone in that group can action it (that's how Grouper permissions work now, I think), and therefore everyone in the group should get the email, not just the approvers (which are the group owners).  That's potentially a lot of people for large groups that have Grouper grant permissions, but I'm fine with discouraging people from giving that permission to large groups.


Здесь лучше

    self.redirect('/')


By passing in the message as an r-value reference it allows for more flexibility for dealing with the message. For example, in my other commit that has the remainder of the transport abstraction work, the message is passed off to the transport. For in process transports, this allows us to actually pass the message ownership off directly to the application router.


In the feedback email, provide information about:
- User email
- Query options
- Number of queries


This test is to check why this `if` is needed
https://github.com/lgalfaso/angular.js/commit/e75e0ad685e3f7ce2e8b1cd2bae1cb037c50d90a#diff-a732922b631efed1b9f33a24082ae0dbR810

The test is somehow basic, so in theory it can be deleted


should `display:block` just be on the `modal` class?


Необходимо после главного шрифта, указать заменяющий - допустим Arial, а потом sans-serif! serif - это шрифт с засечками - типа Times New Roman


What's the difference between BinaryStorage and DirectoryStorage? How can a file store data without a file?


You ought to use a string resource rather than the hardcoded "Prev". That way if the app is internationalized or other user interface changes are made, all the strings that need changing are in the strings.xml file. Android Studio makes this easy: put your cursor on the string "Prev" and press option+return. From the menu that pops up, select "extract string resource".


This is a test


you for sure should use third party libraries for things like equals, hachCode, toString.
You will see how less code is needed to do what you did.


@cprice404 and I talked about whether or not we should add a test which specifically validates that we can coerce from a `RubySymbol` here.  We were both leery of adding more coupling to concrete classes in JRuby than absolutely necessary, though - even for tests - so I didn't add such a test to this PR.  Instead, we'll just have to live with this being indirectly covered by virtue of the full-path integration tests that would presumably fail if the coercion were to start failing sometime in the future - e.g., on a breaking JRuby upgrade.


change formatting to `&__`-style, like other new stuff:
```
.forgotpass {
  &__header { ... }
```


Я не совсем понимаю разницу, которая должна быть методами initConn() и getConnection()? По сути ведь оба служат для получения Connection"а к БД.
Заодно вопрос по connection, ты ранее мне посоветовал: **"По вопросу 7: Попробуй потестить это. Например используй коннекшн сразу после его закрытия в конкретном методе."**
Я попробовал - когда закрываешь connection внутри метода, который работает с БД - вылетает exception, что connection closed, что логично. Но меня сбивает с толку другое, когда мы делаем
UserDao dao = new UserDaoImpl(factory.getConnection()), и потом вызываем методы у данного ДАО, а после в каждом из методов в секции finally вызываем метод close(когда вся работа метода сделана). По сути у нас же ссылка постоянно на один и тот же connection UserDaoImpl. И не смотря на то, что в каждом методе мы его закрываем, мы его всё равно повторно используем. Почему?


Assuming you took my earlier suggestion to factor out a method for moving to the next question, you could generalize that method so that it takes a parameter of either 1 or -1 to move in either direction. I think that would be an improvement, but I'm less sure of this than I am of the earlier refactoring -- having exactly the same code in two places is definitely bad, whereas having two closely related pieces of code is less bad.


testing @haikarthikssk  mention


Ah sorry, didn't see your commit message... I see now what the sorcery is :)


Это не надо


capitalization on `from 'SquareButton'`


Anyway I don't see how "#675 Shorten CLI identifiers" relates to "Variables/constraints interpreted using ${} notation"


.main-logo a {
display: block;
}


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1537 of 1537 tests passed, 0 test warnings)

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


Why return a dictionary? It should be a sequence of `KeyValuePair<TKey, int>`. See also issue #104 that refers to F#'s [`countBy`](https://msdn.microsoft.com/en-us/library/ee340317.aspx).


Не надо


:no_bell: we should start thinking about autobinding. maybe require events to be named `onSubmitForm`, loop through props, and autobind in the constructor?


Можно и больше чем 400


nuketown


не надо


nuke className


it'd work either way; os.path.join will handle if it needs the trailing / or not. It's left in to better indicate to the programmer that the string "plugins" represents a directory


Ок, понял.
Но хранить эти данные мы всё равно будем в таблице administrator в соответствующих полях или как?


can this be moved to a class?


The null argument checks are simple enough to be collapsed to a single line, as in:

```c#
if (source == null) throw new ArgumentNullException("source");
```


Oh i thought it showed the <footer> was deleted amd <div> (with id) was
added.
I guess i am too tired :s

On Tuesday, February 9, 2016, ChielHackman <notifications@github.com> wrote:

> I didn't removed it. I gave it an id so it's easer to place and style him
> with CSS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/navase/boatconnection/commit/61f85b51c1fea424affc1eaeb771edb1d5ea9b37#commitcomment-15987617>
> .
>

--

Met vriendelijke groet,
Kind regards,

Iris Bune


Можно вместо width, задать max-width: 100px; Тогда текст в блоке будет всегда прижиматься к правому краю и увеличиваться в зависимости от содержимого


int should be enough for us. Please change this for all classes.
Save memory ;)


are these checks for programming errors or corruption.  If corruption, you should use your new corruption exception.  You should also consider adding helper method like `checkArguemment` to the corruption exception (maybe  `validateParquet`)


saw approval from tburghart
at https://github.com/basho/riak_ensemble/commit/a49f3f2e163291e7be1bde18b86d40d96c04b320


merging basho/riak_ensemble/dr/upgrade_to_eleveldb_2.0.13 = a49f3f2e into borshop-integration-92-dr/upgrade_to_eleveldb_2.0.13


basho/riak_ensemble/dr/upgrade_to_eleveldb_2.0.13 = a49f3f2e merged ok, testing candidate = 00151548


man this devonne01 is quite the trouble maker....


Hi Luis-

Most of your functions should be placed before the if __name__ == '__main__'. What goes below that should be specific values you want to pass through the functions. For example, above the main function,  you can define a function called decrypt_this() that takes a string as input. Below the main function, you can pass something like:

    user_input = input("Type a string to decrypt: ")
    decrypt_this(user_input)

Also, this only appears to be encrypting and not decrypting. Try creating a function for each, and passing them values as I showed you above.


The key projection is already provided by `GroupBy` so the `Select` is redundant.


Should these be docker specific commands? Why not use `ahoy drush sqlc` and `ahoy drush sql-dump` ?


Like I said, it may be better to put the Player and NPC classes in the Human module, as they _are_ humans, just classified as Player or NPC. The import line in `Fallout_Shelter.py` would then be `from Human import Human, Player, NPC`.


yes. especially if we are going with bem, ill either make a class that uses the bootstrap classes as mixins or just implement it to move entirely away from the bootstrap based css


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 151](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=771&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:38:04


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 152](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=772&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


display block is added by the bootstrap js when used normally


You are changing **one** character to another.

    $ stat -f %z /usr/bin/sed /usr/bin/tr
    34392
    16136

As you can see from the above command, `sed` is **twice** the size of `tr`.

What's the benefit of using `sed` over `tr` here?


Is hub now a dependency?


similarly,

```
.login {
  &__submit-btn {
```

etc


because I'm leaving the base functionality of onRequest on the base class; the implementation class (SWProxyCallback) is responsible for implementing anything it needs over the base functionality (all it needs to do is store the request for later retrieval; so that's all it does in its one line implementation:73)


I see you are now using a string resource for "Previous". Good!


zanim to ogarnę, to trochę minie.. :3


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


В разметке не должно быть b class="slider-name", b - мы не используем


:)

Don't know what happened. We'll get it next time!


Derp exports


args increases extensibility as new arguments could be added to the parser and we'd want them in start_proxy_server without having to keep adding parameters to the method, args is perhaps a poor name (confusing with *args), so ill rename it to options (because thats what it is; an options dict as setup by argparse)


Again, a dictionary is not what's requested so this should be a simple `Select` to project out the elements that are `KeyValuePair<TKey, int>`.

This is a naïve implementation but a good starting point to get all tests in place. It should then be unrolled into a full implementation that should not rely on `Count()` to do the job. Only the keys need to be kept around for the job of this operation. The source elements can be thrown away.


<footer> changed to <div id="footer">
So in the css you can call it by #footer


Для кнопки .slider-btn - 380px много. Её не надо делать float: left; и margin-right: 80px ей не нужен


hmm that ain't working what i wanted to show....


Feel free to make/merge the change; i'm punting on the loglevel stuff since you said you were working on it also in a separate branch. You could merge those parts into master and then setup the log level


Изображения в слайдере будут фоновыми их в разметке не будет


What about calling this just DocsCronTask since, in the future, it may do more than just refresh/update the markdown files ?


require './lib/string_segmenter'

instead of
require '../lib/string_segmenter'
or instead of
require_relative '../lib/string_segmenter


Since this discussion is getting pretty in-depth, I'm going to open an issue so we can discuss it further there instead of in a random commit comment section: https://github.com/DarwinAwardWinner/ido-ubiquitous/issues/106


extra 'print e' here that should be removed.


This is looking great! I think your user prompts could be a little more helpful, let them know what they should press next. I like the use of the zip function, nicely done!


Имеется ввиду бросать этот экспепшн в
catch (SQLException e) {...}
я правильно понял?


call_plugins already does a try:except, so the try here is useless.


@jcoyne I've updated it to add additional test cases that ensure our current single predicate usage would be preserved: a1fb474d61f4cc840f73d2cf43032a005e0c7d12


Thanks!


Good idea to replace the bash script with php :+1:


Все 3 свойства выше не нужны. У тебя - это блочный элемент. Его ширина будет равна ширине родителя, а родитель шириной 880px.


Missing version. Please add current version of this gem


Good catch, I missed that one in the previous review! :)


`tr` was causing some encoding issues. see https://github.com/stevemao/diff-so-fancy/issues/7


Это можно убрать оно ничего не дает


@sage - Should it read: if ((flags & CINIT_FLAG_DEFER_DROP_PRIVILEGES) **==** 0) to imply the deferred flag has not been set, and we wish to allow global_init() to do the permissions drop for us?


Thank you! I added this line to avoid a warning on gcc. It should be declared in `<functional>, but it seems like it is not everywhere. So I will add an `#if APPLE`.


Если 0 - то px, можно не писать!


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - i386-Windows-7-VisualStudio-12: Tests Failed (1459 of 2058 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


@F3PiX Yeah sure, that works! :)


Ooops this should have been a separate commit :(


I think that "Use cases" would still be better. Sounds more natural to me. Also, MIME type is "MIME type" or "MIME Type", without the dash.


I agree; both should be moved.

I didn't move them because they are/could be used for something backend in a number of other places; **specifically (parse_login_data in generaterunes.py)** and it seems weird to have something like generaterunes.py import from a plugin.

those issues would have to be fixed before they can be moved (would be simple, but is outside my scope, id say take this commit and then re-arrange it quickly yourself as part of the merge, we can probably delete generaterunes.py?)


Два свойства выше не нужны, причина как в предыдущем случае


Hkjaslkdjlañsd


Also your user.name and user.email keep changing between "Programming" and "Anish Kumar"

set them like this.

git config --global user.name "name"
git config --global user.email "email"


formatting... We don't align arguments in Presto.  Use 8 spaces (two indents) instead.


I guess the second answer is in response to my question here:
https://github.com/Ultimaker/Uranium/pull/106#issuecomment-179677708

I understand how these CMakeList.txt files benefit you when you use Qtcreator. But we don't include similar files for any other IDE either, so I asked why would we include them (and have to keep them up to date) in our repository as opposed to you just using them without adding them to the repository. We don't prescribe any single IDE, and would like to keep the repository IDE agnostic (especially if adding support for a certain IDE involves an ongoing maintenance effort).


agree!


Good point. Will change it


What about setting hostname: www.silverstripe.org for all three environments dev, test, and live ? That way, someone can work on docs in a local dev environment.... :)


Right! That was a commit error...


If the url contains connect.facebook.com, don't apply TP
@SergeyZhukovsky does this look for now?


Это не сработает! Эта запись означает что надо найти в блоке .content - блок .left, а такого нету, у тебя вот так `<div class="content left">` - а чтобы записать css для этого случая надо писать так .content.left - без пробела


Та же проблема как и сверху


no need for the `toString`


> This is veri exciting.


Для блока .partners в html - добавить .clearfix


It should be noted that because of the way Print_Master_Reactions_From_Files.pl works, simply updating the direction field will update the equation and definition fields.


1. Fair points.

2. I don't know to be honest. On 1080p, using pretty small font I can display 300ish characters on a single line. Supporting IDE/editor workflow with 3 columns open at the same time, would require to set the line length limit to something like 100 characters. That might be ok... (?)

3. Since I'll be editing the file already, I might as well have look at these but I'm making no promises to write grammatically correct English in the future (especially inside code). And I also won't be spending time checking spelling and grammar in other contributors  pull requests.


width: 100% - можно не задать. Блочный элемент всегда растягивается на 100% - это его поведение по умолчанию


czy ta zmiana dbversion potrzebna? numerki się zmieniają ale nie ma update rekordu. wpis w bazie mamy 2016020503 a versja w bazie 2015020501


@lrowe Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.subrequest/1//violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/__init__.py Imports are incorrectly sorted.
plone/subrequest/__init__.py:0:1: C101 Coding magic comment not found
plone/subrequest/__init__.py:0:1: I001 found unsorted imports
plone/subrequest/__init__.py:17:27: E261 at least two spaces before inline comment
plone/subrequest/__init__.py:37:18: E261 at least two spaces before inline comment
plone/subrequest/__init__.py:59:20: Q000 Remove bad quotes.
plone/subrequest/__init__.py:61:1: C901 'subrequest' is too complex (18)
plone/subrequest/__init__.py:61:1: E302 expected 2 blank lines, found 1
plone/subrequest/__init__.py:62:29: Q000 Remove bad quotes.
plone/subrequest/__init__.py:67:40: Q000 Remove bad quotes.
plone/subrequest/__init__.py:67:80: E501 line too long (109 > 79 characters)
plone/subrequest/__init__.py:80:25: S001 found %s formatter
plone/subrequest/__init__.py:82:22: S001 found %s formatter
plone/subrequest/__init__.py:97:80: E501 line too long (87 > 79 characters)
plone/subrequest/__init__.py:112:31: E261 at least two spaces before inline comment
plone/subrequest/__init__.py:114:32: E261 at least two spaces before inline comment
plone/subrequest/__init__.py:137:9: B901 blind except: statement
plone/subrequest/__init__.py:138:30: Q000 Remove bad quotes.
plone/subrequest/__init__.py:138:60: S001 found %s formatter
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/interfaces.py Imports are incorrectly sorted.
plone/subrequest/interfaces.py:0:1: C101 Coding magic comment not found
plone/subrequest/interfaces.py:0:1: I001 found unsorted imports
plone/subrequest/interfaces.py:3:1: E302 expected 2 blank lines, found 1
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/subresponse.py Imports are incorrectly sorted.
plone/subrequest/subresponse.py:0:1: C101 Coding magic comment not found
plone/subrequest/subresponse.py:0:1: I001 found unsorted imports
plone/subrequest/subresponse.py:9:1: E302 expected 2 blank lines, found 1
plone/subrequest/subresponse.py:17:29: E261 at least two spaces before inline comment
plone/subrequest/subresponse.py:18:12: P002 found "hasattr", consider replacing it
plone/subrequest/subresponse.py:18:80: E501 line too long (86 > 79 characters)
plone/subrequest/subresponse.py:44:16: P002 found "hasattr", consider replacing it
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/testing.py Imports are incorrectly sorted.
plone/subrequest/testing.py:0:1: C101 Coding magic comment not found
plone/subrequest/testing.py:0:1: I001 found unsorted imports
plone/subrequest/testing.py:1:1: F401 'OFS' imported but unused
plone/subrequest/testing.py:6:1: F401 'getRequest' imported but unused
plone/subrequest/testing.py:23:9: F841 local variable 'response' is assigned to but never used
plone/subrequest/testing.py:49:23: S001 found %s formatter
plone/subrequest/testing.py:85:25: Q000 Remove bad quotes.
plone/subrequest/testing.py:86:17: Q000 Remove bad quotes.
plone/subrequest/testing.py:94:1: E302 expected 2 blank lines, found 1
plone/subrequest/testing.py:100:80: E501 line too long (97 > 79 characters)
plone/subrequest/testing.py:104:80: E501 line too long (112 > 79 characters)
plone/subrequest/testing.py:114:80: E501 line too long (81 > 79 characters)
plone/subrequest/testing.py:152:80: E501 line too long (116 > 79 characters)
plone/subrequest/testing.py:152:87: Q000 Remove bad quotes.
plone/subrequest/testing.py:153:80: E501 line too long (110 > 79 characters)
plone/subrequest/testing.py:153:82: Q000 Remove bad quotes.
plone/subrequest/testing.py:154:1: W391 blank line at end of file
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/tests.py Imports are incorrectly sorted.
plone/subrequest/tests.py:0:1: C101 Coding magic comment not found
plone/subrequest/tests.py:0:1: I001 found unsorted imports
plone/subrequest/tests.py:12:1: E302 expected 2 blank lines, found 1
plone/subrequest/tests.py:14:5: F841 local variable 'traversed' is assigned to but never used
plone/subrequest/tests.py:33:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:33:66: S001 found %s formatter
plone/subrequest/tests.py:33:80: E501 line too long (126 > 79 characters)
plone/subrequest/tests.py:34:61: S001 found %s formatter
plone/subrequest/tests.py:41:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:41:66: S001 found %s formatter
plone/subrequest/tests.py:41:80: E501 line too long (126 > 79 characters)
plone/subrequest/tests.py:42:61: S001 found %s formatter
plone/subrequest/tests.py:49:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:49:66: S001 found %s formatter
plone/subrequest/tests.py:49:80: E501 line too long (126 > 79 characters)
plone/subrequest/tests.py:50:61: S001 found %s formatter
plone/subrequest/tests.py:76:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:76:53: S001 found %s formatter
plone/subrequest/tests.py:76:80: E501 line too long (137 > 79 characters)
plone/subrequest/tests.py:78:80: E501 line too long (89 > 79 characters)
plone/subrequest/tests.py:81:54: S001 found %s formatter
plone/subrequest/tests.py:81:80: E501 line too long (138 > 79 characters)
plone/subrequest/tests.py:83:80: E501 line too long (89 > 79 characters)
plone/subrequest/tests.py:86:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:86:53: S001 found %s formatter
plone/subrequest/tests.py:86:80: E501 line too long (150 > 79 characters)
plone/subrequest/tests.py:88:80: E501 line too long (89 > 79 characters)
plone/subrequest/tests.py:96:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:96:53: S001 found %s formatter
plone/subrequest/tests.py:96:80: E501 line too long (113 > 79 characters)
plone/subrequest/tests.py:99:80: E501 line too long (89 > 79 characters)
plone/subrequest/tests.py:103:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:103:53: S001 found %s formatter
plone/subrequest/tests.py:103:80: E501 line too long (113 > 79 characters)
plone/subrequest/tests.py:107:80: E501 line too long (99 > 79 characters)
plone/subrequest/tests.py:111:15: Q000 Remove bad quotes.
plone/subrequest/tests.py:111:53: S001 found %s formatter
plone/subrequest/tests.py:111:80: E501 line too long (82 > 79 characters)
plone/subrequest/tests.py:120:80: E501 line too long (83 > 79 characters)
plone/subrequest/tests.py:123:9: F841 local variable 'app' is assigned to but never used
plone/subrequest/tests.py:145:46: Q000 Remove bad quotes.
plone/subrequest/tests.py:155:46: Q000 Remove bad quotes.
plone/subrequest/tests.py:165:46: Q000 Remove bad quotes.
plone/subrequest/tests.py:182:80: E501 line too long (109 > 79 characters)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


I think that once we know it's type OST_ZFS, we should call into the ZPL code to deal with the zfsvfs_t -- basically moving the code inside this "if" statement into a function in the ZPL.  See zfs_get_zplprop() for a similar example.


Yep correct, only for TP. Here is my fix for it
                // Temporary whitelist until we have an UI to unblock hosts
                List<String> whitelistHosts = Arrays.asList("connect.facebook.net");
                if (whitelistHosts.contains(host)) {
                    return false;
                }


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1356](http://teamcity.monogame.net/viewLog.html?buildId=14709&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


intersection of the PDB names and generated atomic names


У тебя в разметке беда у тебя там все идет в блоках b. Это не правильно http://prntscr.com/a15q68


Aren't static methods supposed to have the @staticmethod decorator otherwise, it doesn't work ?
Anyways, reading this : https://stackoverflow.com/questions/136097/what-is-the-difference-between-staticmethod-and-classmethod-in-python I think it might even be a good idea to make it a @classmethod if you move plugins to be a class variable.


costTracker and *pvChecks seem to have the wrong order here and in main.cpp vs how they're actually used.


:+1: Thanks for the fix!


Got it, thanks for letting me know.


only works for atom names starting with 'Q' or 'M', these will be replaced with 'H'


@shekhardesigner here is the change


Why do we need "originalText" here. The original text should be the covered text for the annotation, no?


Instead of requiring the quotes, you can do: `Given(/^there are (\d+) books in the database$/)`


Where does the name "EDU" come from?


initializing new restraints


Fixed the haversine function to output either miles or kilometers based on a metric boolean parameter.


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


NULL should be for validationTracker, not for pvChecks.


This is awesome!

I'm curious if you're using try/except/finally and passing 'pass' to finally because you are planning for more error catching downstream?


Can you rewrite the commit message to state the bug that is being fixed, instead of saying that is is being updated to follow the ORC code?  Generally, it looks like this code is simply being update to handle missing statistics.


Why is the type realized as a feature and not in the UIMA type hierarchy?


You can remove this line. It's not doing anything relevant to what you're testing here.


I added a folder `test` to see what the changes did when it generated the static website.
Didn't want to add that to the repos.


5382_PoolMakeUpWater (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Functions should be defined in `camelCase`.


This still isn't *quite* doing what the test says. You're checking that the page has the given book, but you're not checking the order of the books. Let me know if you want help with how to do this.


Awesome! :)


@diracdeltas I think I have to revert that commit for now. I've found today that lots of websites broken with that, but all of them worked good before and used https if found a rule. Need time to investigate why for all of them, but we need to release this week, so better to do it after. Let me know your thoughts.


Yeah, that one is tricky. But in any event, it wouldn't belong into the unit test.


The following doc(s) have been deployed to the [envconfig](http://envconfig-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/environment-specific-config](http://envconfig-static-docs.pantheon.io/docs/articles/wordpress/environment-specific-config)


Fixed now :)
https://github.com/Andrettin/Wyrmgus/commit/764b19d459dca54645a113303e0753afa1120d78


you probably meant `model=model` but you can be more specific here and use `project=model` and reference `project` in your template


It feels odd that a "relation" type inherits from "node" type...


wooops! I've merged it in bigmlcom/bigml-java

@leonhwang your repo is forked from an old one (deprecated) the actual Java bindings are in https://github.com/bigmlcom/bigml-java

cc'ing @xalperte @javinp


You can remove these two lines.


This function is indented slightly more than all the other functions.


add a comment on why it is ok to ignore this exception


Same here. You're not checking the actual order of the books, just that this book is appearing on the page.


like in 'QD1'


The following doc(s) have been deployed to the [aeligature](http://aeligature-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/unsupported-modules-plugins](http://aeligature-static-docs.pantheon.io/docs/articles/sites/code/unsupported-modules-plugins)


storing new restraints in a class level list


TeamCity Civic Assembly :: Build backend + frontend [Build 88](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=911&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


TeamCity Civic Assembly :: Build backend + frontend [Build 89](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=913&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


This needs to be a separate PR.


TeamCity Civic Assembly :: Build backend + frontend [Build 88](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=911&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:00:27


Vendor and version should be filled in automatically using Maven. Cf. section on "Type descriptors" in DKPro Core 1.8.0 Developer documentation: https://dkpro.github.io/dkpro-core/documentation/


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


With "rm -rf" on line 93 I don't think you need to check if the file exists on link 92....saves a couple of lines of code.


this big fat dict is history


Put this check first, so we don't mess with the logging if the test will fail anyway


`$soc` could also stand for something like `$socket` or, well, `$sock`. You should use a more descriptive name so that you can easily tell what a variable is supposed to do, so in this example, `$newSocialLink` or something similar.


read resolved restraints from the new method


This shouldn't be necessary.  Are you calling shutdown anywhere on the view controller?


Такого шрифта нет! Есть PTSans Narrow. Bold - это жирность


I ran the migration first time by not following the rails naming convention by mistake. So it didn't work. So I created another migration file that follows the rails naming convention. The first shouldn't have any effect on schema.


Yup. Thx.
I used a crappy editor...


What is the purpose of this ID?


like in 'QD'


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


см. дальнейшие комментарии здесь  http://jsbin.com/xetuto/edit?html,css,output


Why not just exit ${ec}?


Are "discourseConnective1" and "...2" supposed to be used instead of the "arg1" and "arg2" defined in the supertype?


package cherbox;

/**
 * Created by Виталий on 07.02.2016.
 */
public class Readerjavax2 {
    public static void main(String[] args){
        Readerjavax r = new Readerjavax();
        r.Scan();
        r.i = count(r.i);
        r.k = count(r.k);
        r.Scan(r.i, r.k);
    }
    private static int count(int x){
        x = x + 1;
        return x;
    }
}


declare variable `i` at the top of the scope


Gosh, that's a new one. I assume this is an idiom through our codebase, ya?


primeiro coment


Задаем для body, а он будет наследоваться


How are start/end offsets for this type supposed to be set?


Проверил


O man, I always mess up the indentation!


Why use an ID here instead of directly referencing the parent relation feature structure?


It looks like there are two paths in the base view controller.  One goes through clear, which will be triggered by a dealloc.  The other goes through shutdown, which is only triggered if we explicitly call shutdown.

I'll try having the dealloc call shutdown.


Instead of passing 5 arguments, just pass the last one, which is really the only difference, and capture the rest. This will make both the lambda signature simpler and the usage here must more transparent.


Not calling shutdown. It's getting dealloced when the parent view controller gets dealloced.


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


ширина 960


Yes, I was expecting that you used it for that. The current config seems only to be used for ATOM. Thought I would make it easier. Still not sure it's very useful, but here it is...


Make this code even more readable, by passing the string directly as a lambda argument and omitting any if statements here.


This appears to be redundant. We already know which argument it is by the fact whether it is referenced from "arg1" or "arg2", no?


Why does the relation need an ID?


k.


Why model this as two features instead of an array?


Karma tests currently aren't running as part of the build, but you can use them!


r=dmose


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/245


I think accounting against the (potentially) wrong txg is not a great idea.  How about this algorithm:

if the *total* amount of "free_dirty" space (sum of all txg's) is more than the threshold, delay a txg.

Once we assign the txg, then increment this txg's amount of free_dirty space.


Huh. I only find "lazy log" here. Where'd you get this idiom, @Marcus-Rosti? I haven't seen it before. Is this your own invention? :-)


Since `NodeLog.{PROJECT_REGISTERED,PREREG_CHALLENGE}` are both strings rather than computed values, what's the value to hiding them behind a lambda? Before when I've seen lambda wrapping in log context, it's to avoid unnecessary evaluation, but here there's no evaluation since they're just strings.


TeamCity Civic Assembly :: Build backend + frontend [Build 89](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=913&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 148 (1 new), passed: 24, ignored: 4 Build time: 00:07:05
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455050852659: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

##### there are 138 more failed tests, see build details
```


Thanks

-------- Original message --------
From: Darklight-03 <notifications@github.com>
Date: 02/09/2016  2:34 PM  (GMT-06:00)
To: Team5427/robotics-code-2016 <robotics-code-2016@noreply.github.com>
Cc: Robo0603179 <varsha.kumar@outlook.com>
Subject: Re: [robotics-code-2016] made shooter timeout (b4b881b)

Also your user.name and user.email keep changing between "Programming" and "Anish Kumar"

set them like this.

git config --global user.name "name"
git config --global user.email "email"

---
Reply to this email directly or view it on GitHub:
https://github.com/Team5427/robotics-code-2016/commit/b4b881b379f32d75aff31ec02a47dbeb085f851b#commitcomment-15988603


They should also be LocalCache, this is an old piece of code


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


I gave u only one example - why this check is used...

respawntime is sync with gate - u can see that in script.
Im using this checks, koz i dont like any kind of errors in console.
Other ppl also
look here:
http://cmangos.net/thread-6585.html?highlight=buddy

Anyway ...
Noone will force you to use ... scripts that are made *not in your way*


@arikwex, would you mind checking whether Karma will run on your windows setup?


@mauritsvanrees Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.subrequest/2//violations
```
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/__init__.py Imports are incorrectly sorted.
plone/subrequest/__init__.py:0:1: I001 found unsorted imports
plone/subrequest/__init__.py:31:5: E301 expected 1 blank line, found 0
plone/subrequest/__init__.py:66:20: Q000 Remove bad quotes.
plone/subrequest/__init__.py:69:1: C901 'subrequest' is too complex (21)
plone/subrequest/__init__.py:70:29: Q000 Remove bad quotes.
plone/subrequest/__init__.py:76:9: Q000 Remove bad quotes.
plone/subrequest/__init__.py:92:25: S001 found %s formatter
plone/subrequest/__init__.py:94:22: S001 found %s formatter
plone/subrequest/__init__.py:150:9: B901 blind except: statement
plone/subrequest/__init__.py:151:30: Q000 Remove bad quotes.
plone/subrequest/__init__.py:151:60: S001 found %s formatter
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/interfaces.py Imports are incorrectly sorted.
plone/subrequest/interfaces.py:0:1: C101 Coding magic comment not found
plone/subrequest/interfaces.py:0:1: I001 found unsorted imports
plone/subrequest/interfaces.py:3:1: E302 expected 2 blank lines, found 1
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/subresponse.py Imports are incorrectly sorted.
plone/subrequest/subresponse.py:0:1: I001 found unsorted imports
plone/subrequest/subresponse.py:20:12: P002 found "hasattr", consider replacing it
plone/subrequest/subresponse.py:21:16: P002 found "hasattr", consider replacing it
plone/subrequest/subresponse.py:22:12: P002 found "hasattr", consider replacing it
plone/subrequest/subresponse.py:48:16: P002 found "hasattr", consider replacing it
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/testing.py Imports are incorrectly sorted.
plone/subrequest/testing.py:0:1: I001 found unsorted imports
plone/subrequest/testing.py:51:23: S001 found %s formatter
plone/subrequest/testing.py:87:25: Q000 Remove bad quotes.
plone/subrequest/testing.py:88:17: Q000 Remove bad quotes.
plone/subrequest/testing.py:163:10: Q000 Remove bad quotes.
plone/subrequest/testing.py:167:10: Q000 Remove bad quotes.
plone/subrequest/testing.py:169:1: W391 blank line at end of file
ERROR: /home/jenkins/workspace/package-plone.subrequest/plone/subrequest/tests.py Imports are incorrectly sorted.
plone/subrequest/tests.py:0:1: I001 found unsorted imports
plone/subrequest/tests.py:22:5: Q000 Remove bad quotes.
plone/subrequest/tests.py:23:5: Q000 Remove bad quotes.
plone/subrequest/tests.py:42:61: S001 found %s formatter
plone/subrequest/tests.py:50:61: S001 found %s formatter
plone/subrequest/tests.py:58:61: S001 found %s formatter
plone/subrequest/tests.py:147:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:147:51: S001 found %s formatter
plone/subrequest/tests.py:160:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:161:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:172:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:173:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:184:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:185:13: Q000 Remove bad quotes.
plone/subrequest/tests.py:231:46: Q000 Remove bad quotes.
plone/subrequest/tests.py:241:46: Q000 Remove bad quotes.
plone/subrequest/tests.py:251:46: Q000 Remove bad quotes.
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


I just ported it from ipb4, idk if using `RandomString` util could work or not.


fix-missed-date (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2069 of 2069 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


ok, its better now :smile:


Which allergy is returned?


I've made the shutdown call, rather than the clear explicit in the dealloc.  Seems to work.
Let me know if this clears it up.

I'd rather take this path rather than your change.  The lockingShutdown logic in the interaction layer is much more comprehensive and was built for exactly this case.  The app that triggered the problem was calling shutdown, though, which is why you saw it.


Actually, wait.  Where is the "Your Link: " string with a variable to be substituted in?


Which allergy are we getting?


For most stuff that would be nice. I actually like the pattern I am using in loginPage to make the handlers, although I am open to any different pattern that is just as dry


SURE WHY NOT


My commits are not organized: I was probably working towards a custom homepage. And in that case it's not a problem that the `html = '';`


Hi, sorry for late response.
Yes you are right, it would be better to play the first note instead of the last. Didn't realized that.

About the inefficience. Maybe you have a better solution for that, I didn't find any.

Regards
Paul


reextract?


YAY


Why change this? To conform to style elsewhere? To avoid a circular import? ... ?