I'll try it this evening :+1:


TeamCity Ignite :: Ignite Cache 3 [Build 262](http://204.14.53.152/viewLog.html?buildId=60052&buildTypeId=Ignite_IgniteCache3&guest=1) is now running


Just curious, why is this branch moved here? It can be argued that an empty code block should still trigger the `complete` hook.


fixed through computed properties as suggested


@AlbertoSONIC this evening I could try it by myself :+1:


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


tri


generally there is no drv_init and drv_exit required.

I think, the dualprocshm initialization and exit (which is currently done in drv_initDualProcDrv and drv_exitDualProcDrv) can be moved to these functions.


I want to try it too! I'm not at home for the whole day :(


why do we need to copy the packet? could we use the original packet?


Checkmarx scan results 11 Aug 2015 12:56 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
6
139
1575
0

Fixed
0
0
4
0

 Open Viewer
Included commits

Commit ID	Owner

7ce65d734ba524e8975537d83eaebe206d477290	@dortal

477b1c379e90c60c38445cea8d3de8b67b725739	@dortal

0f7dafdd90c9efa8f9177b121b8b4bb222bc7e4b	@dortal


Yes I did - I felt it was necessary to separate the input- from the output mapping. What do you think?
Clearly you're right - If you agree with the new format (or any modification that should be made) then I will provide an update process (possibly fully automated).


Also append is quite expensive, how about:

```python
offset = len(input_dir)
index_list = [int(path[offset:-4]) for path in file_list]
# Or in case the file name can contain prefix
# index_list = [int(re.finditer(r"(\d{4})\.jpg", path[offset:]).next().group()) for path in file_list]
index_list.sort()
```


The double-click interval would preferably be defined in an external configuration file instead of in being a constant within the bulk of code.


Documentation wasn't updated with these changes.


The return value will not be used by the driver.

We pass the return value directly to the application using the  `pCtrlCmd_p->retVal`.

I will add the null check.


Made my day. Really.


thnx. I will update it


actually we want the `.jpg` to be directly after the number. So:

```
index_list = [int(path.rstrip()[-8:-4]) for path in file_list]
```

should do


its gives crash


You can even get rid of the underscore


fast-forwarding 2.0 to borshop-integration-765-bugfix/jrd/727 = dce213c7


Word


Probably not a big issue,  but shouldn't you use getLevel() for consistency?


Ah!  completely missed this and i didn't know why it was doing this.  Check the post#show page:

![image](https://cloud.githubusercontent.com/assets/7947900/9213744/7a6c68ce-405b-11e5-95cf-7d0fce002f51.png)


Please split this class
Consider adding an OpportuniyForm and OpportunityDetail page


Take a look to AccountObject and OpportunityObject. both classes have commons methods like save and delete methods
Consider using an abstract class for that


You can improve your code until Wednesday


I think we'll run into an overlapping issue on posts#index as well with relative positioning.  With multiple tags, it could overflow and overlap on the `view comments` link.  I think we should bring the partial back into the `posts#preview` partial so that the elements flow properly.


It is not necessary to commit the .idea and binary files *.class folders


Layer buffer is maxed out so I had to resort to this hack. But good point, let me tweak this...


@yfdyh000 mathtable我不知道输出结果是什么样的，不好确定
base似乎用基数更合适
overscript按语义应该指标在字符正上方的标识，但不知道怎么翻译好
其他的就按你的意见来吧


Mhh, looking at the code, it seems that we _have to_ reverse the `arr` here, because of the bit mask / the order of the bits. Is this the case? (Note to myself: read the paper first)


For this case it would be better to use @FindBy(id = "acc2") instead of a css
Review all you css selectors


Will actually test on correct environment with fix


Please try to implement a Singleton class for that


Remove this unnecessary try catch


It is not necessary to use try catch in this scenario. Remove it!


Same as above


How can `N` be zero here?


Please split this class
It is not a good practice to have Opportunity and Account web elements in the same class


Sounds good.

I should have explained the scaling issue. Pango uses font hinting, so the outline of a letter depends on the display size - or what Pango believes the display size to be.  In general this makes text look better, but of course it doesn't look so good if I ask Pango to lay out text 1 pt tall and then scale it up 72x.


`Facter::Core::Execution.exec`


This block of code should be in a @After method


This block of code should be in @Before method


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - i386-Windows-7-VisualStudio-12: OK (1658 of 1795 tests passed)

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


ah okay! thanks for the fix


It is not necessary to commit the .idea folder and binary files *.class


Good idea, I'll change this to use emailFormat, thanks.


technically we could just do
```ruby
Facter.add(:r10k_path) do
  confine :kernel => :linux
  setcode Facter::Util::Resolution.exec("which r10k 2> /dev/null")
end
```
but I can't remember when that showed up as a thing and we still support 2.7 in the module until v4


This one's my favorite :trollface: in this commit.

cc: @drbrain @evanphx


\o\ /o/ \o/
WOOOO!


@SubuSS, there is still a limit on maximum page size (10MB by default).  This change keeps that default setting even if the cache pool reserve size is 20MB -- without this change, the maximum size in memory of a page was constrained to 5MB, possibly leading to more / unnecessary splits.

This upper bound on `memory_page_max` was put in place to deal with very small caches -- we regularly stress test with a 1MB cache to make sure eviction doesn't get stuck.

You're right that we could check for tiny cache pools: I didn't think that was a realistic use case.  The huge root page problem was largely independent of this setting: there was code that prevented splitting the root page under some conditions (even if it grew huge), and that code has now gone away.


n.b. - added random multiple tags in seeds


嗯，其实主要原因是这些字符串今天到Aurora阶段了，需要在官方库那边翻译了，然而很多很难判断，想双方校对一下，取得最佳值……
http://l10n.mozest.org/zh_CN/firefox-aurora/dom/chrome/accessibility/


Take at look to AccountPage and OpportunityPage. both classes have the newButton similar
Could you create an abstract class in order to reuse the web element


Take at look to AccountInfoPage and OpportunityInfoPage. both classes have the deleteButton similar
Could you create an abstract class in order to reuse the web element


:+1: Now this is where I step in and help :)


I just wanted to do something quick and working, clearly more could (and should) be done to make this more customizable. Ideal for me would be to make it configurable enough to allow both using CSS and not using it because there are some email clients that can show (simple) HTML, but don't support CSS at all. Unfortunately I don't know how to do it and, to be honest, I don't think I have time for it, plain ugly HTML should be good enough for my own needs for now.


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


develop (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1273 of 1273 tests passed)

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


Try to use Page Factory


Referente a issue #165


I think that this class should not extend from BasicPage.
Because this class doesn't need to know how switchToSecondBrowser, confirmAlert and deniedAlert


its 32 on 480 and 64 on 481. Is that ok?


I don't think it's actually doing anything anyways, as the results get clipped to the buffered extents when making the vector tile...


This method is hard coded


Do you know why this transactional attribute is here?


This should reverted at some point. It hinders readability and there must be a better solution, especially since xrange is deprecated and we need to get rid of it anyway.


It is not necessary to commit the .idea folder and binary files *.class


Is this empty on purpose?


The package structure here doesn't match the directory structure under grails-app/domain/


@Vilsol


What?


A number of classes are marked with a commented out `@Typed`.  Are you using that in some circumstances?


If you check the latest commit, I fixed it an hour ago


So just to confirm in "%how" section of the  global_attributes*.txt files there should be (will be?) a "S, credit =" entry. And this is the line to add other collaborators or funding agencies that contributed eg Darwin Port Corp for Darwin NRS, or WA gov for PIL/KIM mooring arrays.


It is missing a precondition
Have created an account


98318570_RemoveRadianceCheckbox (macumber) - Win64-Windows-7-VisualStudio-12: OK (2197 of 2215 tests passed)

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


You can improve your code until Wednesday


I can't find where I can resolve this dependency.


/cc @zenorocha


It is not necessary to commit the .idea folder


This allows autocomplete to abort unfinished requests, e.g.

```js
var pendingRequest;

input.addEventListener('keyup', function() {
  if (pendingRequest) {
    pendingRequest.cancel();
  }
  pendingRequest = Launchpad.url('/foo').get().then(...);
});
```


Oops, I think you've accidentally added this file!

I would like to ask during the scrum about purging a file like this from the commit history.


Save in a variable the following page after saving the Opportunity in order to have a readable test cases.


Thanks @wkeese!


Add the delete Opportunity


`defaultAction` should be static


Notice that the AccountAddForm and OpportunityAddForm have the same saveButton
Please create an abstract class in order to reuse it


Thanks for submitting this. I've now released parampy 2.0.0 which has all sorts of improvements including distributed parameter iteration. The README has since been updated, making this pull request unnecessary. Thanks again for submitting it though! Patches welcome.


Notice that the AccountDetail and OpportunityDetail have the same deleteButton
Please create an abstract class in order to reuse it


Please create a Singleton class for that


I like this way of drying code. Does it make sense to add callbacks that you use only in one action? Like this one before_action :require_friendship, only: [:show] ?


Hello. I believe those calls are indeed mine. Commit 1bff8f2ba7ac22dc1a1f62f24ff1b7405181cef2 has them. As for why I used rsync, I can't remember exactly. I think it had something to do with permissions issues when using cp, and with cp being a bit "dumb" when copying files over. All I know for certain is that I needed the call to get the files copied over that allowed the OS X .app to function properly.

If you'd like to use cp to just copy over a particular set of files, that's fine.


Remove unnecessary try catch


Yeah, vectors are diferent, but we also need to make raster tiles work. And it noticeably improves the situation for them, albeit not fixing the problem completely.


Same as above


Hi Michel,
I did pull it and test it out. I first was made aware of your fork through Tango Honeypot Intelligence but I like all the features that you have added.

I am trying to make splunk, Tango Honeypot Intelligence and your kippo all work for a training that I am teaching.

Thanks

Rezuma


Are you not rendering your rasters from this tm2source, i.e. rendering them from vectors?


Any reason why you don't have `extends Component`? Does it matter?

Also, here you define `propTypes` *inside* the class via `static`, but in `List.js` you set them on the class at the bottom of the file (outside the declaration). Is there any reason for the inconsistency?


Omar & Neal,

I just have to tell you again how much I love lurking these threads.  I
know not of what you are speaking, not even a little. And I'm reading it
just through the email notifications, so I have no context at all for the
comments.  Yet still, all of your energy, your pace, your smartness, your
camaraderie, your ping-pong style of collaboration leading to innovation
... it just all shines through somehow. So even though the actual content
is just incomprehensible gibberish to me, as I am reading it I can feel my
own inspiration levels ratcheting up, and it makes me even more excited to
be a part of this amazing project!  Thanks again for doing your thing!  You
are appreciated!  :-)

-TARA

Tara M. Harwood
Manager of Analytics & Data Science
SumOfUs.org

tara@sumofus.org
(510) 725-2979

On Tue, Aug 11, 2015 at 4:12 PM, NealJMD <notifications@github.com> wrote:

> yeah this should be self.class.name. I was going to add it, I couldn't
> decide whether to make a Plugin concern (which I know how to do) or to try
> to add an instance method to all classes in the Plugins module by adding it
> to the module (which I couldn't figure out how to do)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SumOfUs/Champaign/commit/fa6f1f731e8f26370841aa45bf0dd97aeaa0c068#commitcomment-12653299>
> .
>


Why user belongs to a photo?


It is not necessary to commit the .idea folder and binary files *.class


No, 'credit' in the metadata records refers to the GeoNetwork metadata record. Ex.: https://catalogue-123.aodn.org.au/geonetwork/srv/eng/metadata.show?id=22900&currTab=simple .

So you could add other collaborators or funding agencies in the global attribute ```acknowledgement```, augmenting the existing model sentence. Ex. :

"Data was sourced from the Integrated Marine Observing System (IMOS) - IMOS is a national collaborative research infrastructure, supported by the Australian Government. Agency X also participated in collecting/funding the data."


97204612_MultiSpeedFan (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1803 of 1804 tests passed)

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


Just realized you forked this project. Thanks for finding this bug!


This hasn't been colaborated with the pervious version, ie buddy handle is still null. also a few bugs, 2nd window needs to go back and forth once before it works and the select file button only loads the filename into the window I was short on time and just tried to copy a code in without even really looking at it. anyways, if either of you guys can look at the dumpToWindow method and write something out, that would be swell and should work! I'm off.


These attributes are no longer necessary because they will be inherited. Remove them!


@Victorious3 I guess you wanted to put something here, not to disable code that goes next.
As of b5148896f00e2aa0bfdaa63d04534346f4bcbd00, this sole line prevents visual updates of non-bundled IOs of IC gates (i.e. ones that glow red). That's one thing I noticed, something else might have broken too.


logic in view


Good thought whitelisting the params. I have a nasty if/else statement


Did you try to use Page Factory for that?


Sure, I've been super busy today, but I'll mark this as a todo for tomorrow when I track down the exact commit.


I like this way to introduce a method to a controller.


Consider simplifying this by adding a check_friendship model method


is the first template parameter the row or the column. Because if the parameters go like MatLab where it is M rows and N columns, you have transposed the output. should be apf::Matrix<M,N>


I like how you use helper methods here to pull some logic out of the views


I like how clean this index view is!


Renomeei em outro commit, quando percibi. Hahahhah


I swear there was a reason I did all of this formatting stuff twice.  It seems like it only applied once or something, but I could be totally wrong.  If you've verified that it works the same by just applying it once, I'm good with it.


Why use a bit-wise operator here?


Could you explain why do you use VCR.use_cassette here?


첫번째 커밋


Very helpful comments


Can we do something about this?


docstring please.


no don't merge this


It is not necessary to commit the .idea folder and binary files *.class


What should this fix?


Might it be a good idea to indicate whether the transaction is confirmed? I'm not sure what is the intended use case here but mixing confirmed with unconfirmed (and possibly unconfirmable) transactions seems dangerous.


I think this will work without creating the list,
`all(isinstance(s, Posynomial) for s in self.signomials)`


Remove unnecessary try catch


Please create a Singleton class for that


there is no `else` for this conditional - should it return `NotImplemented` or raise a `NotImplementedError`?


Looks good.


rename gp here, maybe `cfp_gp`?


Do u have the log at hand?


Take at look to Account and Opportunity page. both classes have the same deleteBtn method
Please create an Abstract class in order to re-use it


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 300](https://teamcity.labkey.org/viewLog.html?buildId=265842&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0 Build time: 00:02:14
### Failed tests
```
org.labkey.test.Runner.BeforeFirstTest: <no details avaliable>

```


Take at look to AccountsHome and OpportunitiesName page. both classes have the same clickNewBtn method
Please create an Abstract class in order to re-use it


You can improve your code until Wednesday


@marcoT89 nesse caso, eu não estou verificando se o usuário está logado. Realmente o objetivo é verificar se este perfil é o do usuário logado, porque outros também podem ver o seu perfil. Esse available_events deve aparecer somente quando visualizar o seu próprio perfil. Pode fazer o teste aí, quando você visualiza outro perfil, só aparecem as informações dele e não os eventos.


It can't - could just as well be `return false`.


setup your git username and email :)


/cc @shobak101


I agree, this should live in the controller and/or utilize `posts.without(@tag)`, but i couldn't figure out how to eager load it and prevent an N+1 condition


DANG!


See my line comment, but this all looks good to me.


It is missing assertions


Shouldn't this be implemented in terms of `SHAMap::find`, which should return `const_iterator`, followed by `operator++` on the returned iterator?


please break this long line


These strings are pointless


This is really really cool!


Remove unnecessary try catch


Same as above


Take at look to AccountsHome and OpportunitiesHome classes. both classes have the same new button
Please create an abstract class in order to re-use it


extra white space


I, urm, expected that `write(transformed)` would be compared directly to `expected` without building the `expectedTree` then writing it again.  Is there a reason that does not work?

Thanks for this, looks close.


Hi Guillaume, thanks for explanation. Thats what I currently do so all good.

Simon.

On 12 August 2015 at 08:46, Guillaume Galibert <notifications@github.com>
wrote:

> No, 'credit' in the metadata records refers to the GeoNetwork metadata
> record. Ex.:
> https://catalogue-123.aodn.org.au/geonetwork/srv/eng/metadata.show?id=22900&currTab=simple
> .
>
> So you could add other collaborators or funding agencies in the global
> attribute acknowledgement, augmenting the existing model sentence. Ex. :
>
> "Data was sourced from the Integrated Marine Observing System (IMOS) -
> IMOS is a national collaborative research infrastructure, supported by the
> Australian Government. Agency X also participated in collecting/funding the
> data."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aodn/imos-toolbox/commit/71afb004fff39891407308fe2d47aeb7434bc028#commitcomment-12654545>
> .
>


indentation is off


This empties out the tags.  My objective was to get a post with the following tags:

`Angular` `JavaScript` `Ruby on Rails`

When shown under `tag#show` of `Angular`, to only show:

`JavaScript` `Ruby on Rails`.

If we want to just hide all the tags, then we should just strip all the machinery to specify post tags, as done in one of the earlier commits...


That's exactly what I did on my first attempt.  It failed unit tests. After some head-scratching I re-read this comment from the function this replaces:  `peekNextItem`:
````
// Get a const_iterator to the next item in the tree after a given item
// item need not be in tree
````
I was failing the case where `id` wasn't in the tree because I was returning `end()` in that case, instead of the item after `id`.


added public folder


Can you tell me the steps you are following to reproduce the issue?


Well, what should we do for Signomials?


It is not necessary to commit the .idea folder and binary files .*class


Also I don't think it makes sense to show any tags at all in `tags#show`


Lazy copy paste, I should fix that. We don't use ES next proposals in this repo.

Extending Component isn't currently required unless you need setState but I should add that too because people ask about it all the time. :-)


Renamed it to `max_gp`


also fixed raise-argument-error misuse here


I'm confused...if it says "store it in out" then you don't want out to be a pointer to a const REBVAL.

Besides that, with these changes there are problems with:

    ffi_type *ftype = struct_to_ffi(out, VAL_STRUCT_FIELDS(elem));

and

    Copy_Struct_Val(elem, to); //for callback and return value

While it's a warning in C, it's an error in C++.


Notice that the AddAccount and Add Opportunity classes have the same saveBtn
Please create an abstract class in order to re-use it


Does it make sense to also change the other calls for get_detail_mapping here?


Please create a Singleton class for that


Why expire this photo?


@matugm Nitpick: I was just thinking if we word the above statement as `doesn't create a new tag if it already exists`, it in a way complements what the spec says on [line number 14](https://github.com/matugm/ruby-monday-blog/commit/48bbd4d1b0f46ec898d13e266678d6ec62843be4#diff-15604d1eca87a54437afd5465d439890R14). One more reason why I've used the above wording is because the spec on line number 23-24 below tests for the tag count increase and not exactly for the reusable aspect of a tag in the context of a post. What do you think ?


Did you try @FindBy(name="save")?


LGTM


You need to delete the created account and opportunity


Care for a pull request for fixing that?


是不是写反了？应该是 (typeof me != "undefined")


You can improve your code until Wednesday


ok


Não seria tão interessante, porque usamos o user também (mostrando a ala da pessoa), então mesmo que pudéssemos puxar somente os accounts, ainda precisaríamos acessar o usuário.


You renamed my log to BattleNet? qq


Last 2 tests are great checks to have


@kalyankondapally While gardening for M45 I am having hard time understanding this change. Both SetWindowShape and SetOpacity seems to be overriding from PlatformWindow but those functions don't exist in PlatformWindow giving me build errors. Am I missing something?


TODO - Delete user trigger, more basic pingbot features like unsubscribe etc.


Sounds good to me. Upon further inspection, I don't think I was actually following any convention consistently.


great job checking the validations on photos


The relevant change is the inclusion of this line only. The diff show everything as modified because the file was changed from `CRLF` to `LF` upon commit.


The information on SAI is not comprehensive. They do not have a "read this in it's entirety and you'll be able to code/write scripts to our liking" page on their wiki. I have a book titled "Sams teach yourself C++" and it does that for C++, although it is a brain buster. They will have to create that documentation, or I cannot know a thing.

(What they do have is a minimal explanation which fails to help me in any way)


this is crazy


The cancer is gone?


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1658 of 1795 tests passed)

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


RoomAirflowNetwork (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (748 of 748 tests passed)

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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.901.247853d72a16b318d91b8b812aa89c0f09366265](http://build.palaso.org/viewLog.html?buildId=60954&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:03:32


This is an interesting solution. you will have only a few classes with this solution :)
but, the main problem with this solution is that this only works with xpath. Maybe your execution will be too slow instead of using id or name
another problem is when you write the test case is not easy to set the field name (string) instead of using the auto-complete method of IDE


Nice work!


This is configurable via the handlebarjs template now


The comment should be: A Plugin used to compile Haxe sources to Javascript sources, not Python sources


Fixes? https://github.com/CovertJaguar/Railcraft/issues/672


Oh, right. So this is `lower_bound(uint256 const&)`


This is an interesting solution using static methods :)
but you will need to explain what advantages you will have with solution


should move out of "if (value instanceof String)".


Công ty In Bình Minh Pat địa chỉ in thùng carton, hộp carton , in túi giấy các loại uy tín, chất lượng tại Tp. Hồ Chí Minh
Công ty chuyên sản xuất : in thùng Carton 3, 5,7 lớp in offset nhiều màu chất lượng cao, in túi giấy ,in hộp giấy ,in bao bì chất lượng ,đặc biệt in thùng carton khổ lớn với khổ in offset lên tới ( 1.1 x 1.6 m) khổ lớn xuất khẩu hoặc kích cỡ tùy thuộc vào yêu cầu từng khách hàng có thể thiết kế in thùng carton,in túi giấy,in hộp giấy theo đơn hàng có sẵn. Sản phẩm thùng carton in offset của chúng tôi đã được thị trường biết đến hơn 10 năm kinh nghiệm sản xuất và chúng tôi luôn không ngừng phát triển và cải tiến, đầu tư thiết bị máy móc hiện đại nhất nhằm thỏa mãn đầy đủ tất cả các nhu cầu ngày càng cao của khách hàng trong nước và đặc biệt là thị trường xuất khẩu.
[IMG]

Liên hệ ngay: 0916 660 853 hoặc qua Email: inantuigiay@gmail.com


Nice Work!


for Opened list, I think, no default value, should return the value directly.


Interesting name suggestion.


Ruby2.0.0からマジックコメントにるよる文字コード指定がいらなくなったので不必要かも


Shouldn't this line go away now ?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.902.c5e6faa7651a5f63687dc2827ebd1a2d14ba9570](http://build.palaso.org/viewLog.html?buildId=60955&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:02:53


what does the `(?: ` mean?


This is a comment


@dcsan non-capturing group. See [this StackOverflow QA](http://stackoverflow.com/questions/3512471/non-capturing-group)


I'll work on this soon! (That indent IS scary. Whoops.)


there seems to be a typo here...


Me gusta la idea aunque de este commit puedo ver que tamaño no está haciendo mucho, y está violando la regla, recuerda 1 propiedad por clase, qué te parece:

```css
.mitad-width{ width:50%; }
.small-height{ height:56px; }
```

De nuevo, me gusta el propósito de la página


https://github.com/randombit/botan/pull/246
--
Uri Blumenthal
uri@mit.edu<mailto:uri@mit.edu>


Sí se ve que estás tratando con lo de CSS, pero... siento que puedes ser más descriptivo con tus clases...

radio, padding no dicen mucho. Pero small-radius, big-padding etc sí... no sé si me explico.


如果用户传的page=0.5  或者 page=fdjaksfda   你的$page是不是就等于0了，这显然不对
这里应该这么写   $page = (isset($_GET['page']) && intval($_GET['page'])) ? intval($_GET['page']) : 1;


TeamCity pull requests :: netty [Build 4](https://garage.netty.io/teamcity/viewLog.html?buildId=229&buildTypeId=pulls_netty&guest=1) is now running


这里一般不这么写
$sql = "select count(*) as total from blog";  通过这条sql语句就能查出一共有多少条记录


Square root is among the slowest computation there is. You do not want it in a loop, ever.


Trata de hacer más pequeña la ventana y se rompe el diseño, sobre todo por la parte de:
```css
.section-size{  height:30% }
```

Normalmente no me gusta usar `height` por lo mismo, intenta cambiarlo por `min-height, así:

```css
.section-size{  min-height:30% }
```

Así si el div necesita más del 30% en alguna otra resolución, se va a adaptar.

Por otro lado, tu idea está también muy bien y estás siguiendo muy bien las reglas con el nombre de las clases.

Lo que tal vez sería bueno además de agregarle javaScript es intentar hacer un layout más complejo, es decir colocar cosas al lado de otras, etc. no que todo queda abajo, nada más por cuestiones de practicar.


"epsiloM"? Pretty sure it's meant to be "epsilon", but it does work.


@dereksmart Noticing that the toggle appears active while the verbiage says inactive after I reset my settings + jumpstart: https://cloudup.com/cDzuWSFXVH2

I'm sure you can fix this up quick. Just make sure the activated classes are applied / removed accordingly like the others. cheers!


I think I might be missing something -- what's the current behavior for signomials?

I assumed we haven't yet thought about how to handle feasibility reports on them.


Two pipelines
* SQL -> tm2source -> vector tiles (PBFs) storage
* PBFs -> tm2 (style) -> PNGs


lost the mark_delayed call here


Just add `/rubocop` to this line and it'll work again.. :)


mysql_fetch_array 和while的配合使用这里有点问题
这里有demo  http://www.jb51.net/article/37911.htm


@chrisndodge Is there a slash missing before the last n"?


In that case, the buffered extents clipping when creating the vector tile will impact both pipelines, since the raster pipeline is the vector pipeline with an extra stage added.


TeamCity pull requests :: netty [Build 4](https://garage.netty.io/teamcity/viewLog.html?buildId=229&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 1145, ignored: 10 Build time: 00:08:05
### Failed tests
```
io.netty.handler.codec.http2.DefaultHttp2FrameIOTest.binaryHeadersWithoutPriorityShouldRoundtrip: <no details avaliable>

```


Yes, labelgrid and extending box are not related.


should make sure that the admin socket command to dump requests will show this waiting state somehow ...


Sounds good. Feel free to send a PR if you want and we will change this :)


RadianceRefactor (rpg777) - i386-Windows-7-VisualStudio-12: OK (2195 of 2215 tests passed)

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


need to be careful about the interaction between the op and the session lists.  i haven't reviewed that carefully.


Final project - Portfolio site/Suzanne Portfolio/css/style.css

This code is just to help highlight the structure of your mark-up. It is not required.


The same problem happened. I think there is problem with  CMakeLists.txt which came with this Gethube file. It requires 2.8.7 or higher, but in side the folder is 2.8.2.


TeamCity pull requests :: netty [Build 5](https://garage.netty.io/teamcity/viewLog.html?buildId=230&buildTypeId=pulls_netty&guest=1) is now running


You do notice that REBVAL is a struct holding some pointers, right? this function doesn't modify those pointers themselves, but the content they point to, so it can be const.

As for warning from Copy_Struct_Val, I added const to its argument as well, so the new commit should have this fixed.


What I was going for it to hide the all the tags in `tag#show` since we didn't have a clear specification (I think a feature test may help next time) and that seemed the most logical thing to me. As we talked on slack we will discuss this on the next hangout to gather everyone's opinion on the matter.


It does indeed fall into small changes. But if you notice multiple things, feel free to create a new bundled PR.


This argument is not used?!?


I see that this is the first version of the file


I use it in Store.js, before creating the listener in queryStoreAndInitItems.


not needed


This will be corrected in version 3.1.2 that I'll be releasing shortly


@pynicolas is another identifier kind really required ?


It was there before so I let it, but yes it is not very useful.


DNS-fix - Will fix faulty dns-server prioritisation


This revision is clean.


Approved


selamat datang telepe


08-12 12:05:07.119 E/AndroidRuntime(4846): FATAL EXCEPTION: main
08-12 12:05:07.119 E/AndroidRuntime(4846): Process: com.ota.updates, PID: 4846
08-12 12:05:07.119 E/AndroidRuntime(4846): java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.String.length()' on a null object reference
08-12 12:05:07.119 E/AndroidRuntime(4846):      at java.text.SimpleDateFormat.parse(SimpleDateFormat.java:1035)
08-12 12:05:07.119 E/AndroidRuntime(4846):      at java.text.DateFormat.parse(DateFormat.java:577)
08-12 12:05:07.119 E/AndroidRuntime(4846):      at com.ota.updates.activities.AddonActivity$AddonsArrayAdapter.getView(AddonActivity.java:286)
More http://pastebin.com/xbC638uv :smiley:


Do you know why returning the code adds another url parameter in the final link?
i.e accessing /patches/gnubg/latest/ redirects to /patches/gnubg/1.02.000-2/?code=301
I dropped the code in the next commit because of that but if there's a way to return it without changing the url i d like to fix it


noken adalah tas khas papua


@SubJunk @Optimusprimeums To repeat, in both [5.2.2 (Java 7)] (https://github.com/UniversalMediaServer/UniversalMediaServer/releases/tag/5.2.2-Java7) and [5.2.2-Java8] (https://github.com/UniversalMediaServer/UniversalMediaServer/releases/tag/5.2.2-Java8) logging is broken unless 9896cfae5868e85302a252ee26600060193932b2 is included.


I wouldn't take clues from SproutCore :) Some of the worst parts of the UI are ideas we've taken from it.
It's not a big change, just moving a piece of code within the same file. To give another reason, it's nicer to pass down things from the parent to the child than is to reach up in the view hierarchy from the child. It makes the code easier to follow which makes debugging easier.


Telepe ( welcome)


please dont: we agreed that the correct path is /var/run.
it is guaranteed to exist and created during preinit. so we also dont need 'mkdir'


LGTM well done! Maybe the commit message could have been a little more explicit about its content.


I checked it again and it seems that the image not look good, since in my code, before the usage of:
```
page.getImage(width, height, rect ...)
```
I'm using the follow code:
```
		int degrees = page.getRotation();
		if (degrees == 90 || degrees == 270){
			int temp = 1;
			temp = width;
			width = height;
			height = temp;
		}
```
After I removed the Rotation manipulation form my code, the image look as expected.
So, it seems that the usage of ``` page.getRotation() ``` is not needed anymore.


@Flavius12 I can type quicker on my phone, so I use that.


telepe berasal dari bahasa suku ketengban artinya selamat mengunjungi sebua arena


Oleh Masyarakat Ketengban


tenpu


Yes. Will implement it later.


Would it not be more beneficial to have a `GamePlayer winner` which has a `Setter` and a `Getter`? Would allow us to have an event be fired - `GamePlayerWonEvent` which could have a `setWinner` and `getWinner`


dont merge! please dont use seq, its not POSIX 8-) also this does not work with IPv6 disabled also create a hotplug script which calls alfred if the interface is really and read up.


调整修改


supaya memperkenalkan bahasa suku yang ada di negara kita indonesia ini


> On second thought, instead of DstoreToStoreAdapter extending Evented and then forwarding events from this to this.data

Not sure what you mean. For me the events comes from this.data to this and the on is defined on the collection (=this).


Instead or using
```java
for(Player p : getOnlinePlayers){
p.sendMessage(message);
}
```
, isint there a way to broadcast a chat packet or something so it would display to console aswell? It would also make it visible to RCON


telepe , selamat berkunjung di arena ku ini


Do the go get commands work?


@juristr I actually had something similar to that earlier, but was hesitant to add the isDefinedAndNotNull function only to use it in two spots. I think you are right though, I will update the pull request.

Just for clarification when extending the function to check for null were you thinking of changing it across the entire file or just using isDefinedAndNotNull just in these two spots?

-- Thanks


siapakah anda , anda ketngban


botepe telepe


ucapan terimakasihku kepadamu


No! you're right! other.entries.size() is always equals to 1


@mpushkin OK. But it's kind of a problem that this fork is a dead end...


Hey Up! As you know I asked you for permission some time ago. We didn't delete the first line from the action but we'll add it. Regards, Paf.


pls merge ;)


"name": "eslint-config-rambler"  -  название важно


@eburi Where is this contrary to the following that I copied from https://github.com/persvr/rql?

> limit(count,start,maxCount) - Returns the given range of objects from the result set


This change make the tables doesn't hide at all, as find() doesn't find anything


remove unnecessary lines


We're getting "Uncaught ReferenceError: showCartIcon is not defined" in YourPensionNumber client here.


this is to ensure the collar gets rlv locked.
Many complains came that the collar "is insecure" and got detached though it was locked and with this we ensure it gets rlv locked even if rlv is turned off in the collar (but of course on in the viewer) or not yet discovered to be there which causes the RLV_CMD to "fail"


Does it work for You and stick max freq? :smiley:


You should disable ramdisk's mpdecision service initialization too.


fix bug if out with invalid utf
``` ruby
out = "avconv version 9.18-6:9.18-0ubuntu0.14.04.1, Copyright (c) 2000-2014 the Libav developers\n  built on Mar 16 2015 13:19:10 with gcc 4.8 (Ubuntu 4.8.2-19ubuntu1)\nGuessed Channel Layout for  Input Stream #0.0 : stereo\nInput #0, wav, from '/tmp/2c2c480c3218b971a9d7b936a98db93c20150812-1993-1r84jvw.wav':\n  Metadata:\n    date            : 2014-08-28\n    IENG            : \xCF\xEE\xEB\xFC\xE7\xEE\xE2\xE0\xF2\xE5\xEB\xFC Windows\n    encoder         : Sound Forge Pro 10.0\n  Duration: 00:00:01.48, bitrate: 1537 kb/s\n    Stream #0.0: Audio: pcm_s16le, 48000 Hz, stereo, s16, 1536 kb/s\nAt least one output file must be specified\n"
out.split("\n")
```


should be VARIETY


self.assertTrue(list(flatten([[1,2],[3]])) == [1,2,3])
self.assertEqual(list(flatten([[1,2],[3]])), [1,2,3])
self.assertSequenceEqual(flatten([[1,2],[3]]), [1,2,3])

would also work


see https://github.com/piccolo2d/piccolo2d.github.io/issues/3


название переменной смущает


This has already been addressed in a follow-up commit.


I have a pretty strong preference for it being mandatory.


название важно. оно используется в extends


it's not clear from your commit message why you've included this in the frontend links definition. Are you planning that the content store would include this value in the expanded links?

If so, you should explain this in the commit message, and probably include it in the example as well.


"fixed .gitignore" - wrong name, use something like minor changes or various update


could you please review this variable? ->  public static final By localWildfyServerLink = By.xpath("//a[@href='/hawkular-ui/app/app-details/haw-alpha3~Local/jvm']")

It won't work for alpha3


@hairmare Standards... tststs. Now what "standard" are we supporting?
https://doc.apsstandard.org/2.1/spec/rql/#functions
>  limit (<start>,<number>)


Please test if pickling and unpickling works.


Use `logIn` insted of `list-in`


Hi Vasile,

I'm not able to find a commit in the master branch fixing the problem I reported below. Have you can fix it ?

Regards,
Christophe


Updated the commit message and it's [already in the example](https://github.com/alphagov/govuk-content-schemas/commit/7101dbfdf90dbcbd11ad4b57281fb1eec01132a8#diff-429a1c63502984ab4d01dc6cda6d50c7R10).


And probably disable mpdecision from chaning freq's (if it would wake up) e.g. https://github.com/dorimanx/DORIMANX_LG_STOCK_LP_KERNEL/commit/ef5ad4acdff081e222d398c26a7d4a57c22895b7? :smiley:


I would use `switch` statement besade on value returned by `match` here.
This will reduce number of `match` function calls.


Should we keep instance ids from the first table?


I didn't verify whether it would make sense in other places within the file as well.
Otherwise I'd add it only at the mentioned line. You can also simply use the `isDefined` and add a `!== null` condition.

I'll merge it in then. Thx for your effort and contributons!


Test this path as well.


Agree with you :-)


Use `git commit --amend`


Should we raise ValueError? Numpy's concatenate does.


（感想）devise.ja.ymlは追加しないのでしょうか？

https://github.com/plataformatec/devise/wiki/I18n#japanese-devisejayml


Argggg...LOL :D

Bah......I'll remove it


Concatenation of multiple tables returns a new table. Should this case do the same?


Please add a newline at the end of file.


I'm not not sure cause encoding explicitly specified by protocol.


Sure, I could monkey around with `dynamic` typing a bit. I've been stumbling around getting the Expression tree to build correctly so I haven't made as quick of progress as I'd have hoped.

As an aside, the way I've been handling types in my own code involves an additional set of functions that maps the object arguments array into typed parameters on a function. See [ArgApplier.cs](https://gist.github.com/freakingawesome/5d94d14c1110aa174343) and an [example](https://gist.github.com/freakingawesome/7df5dfb9af46f3a463e2).


Here is where I deleted stuff


dsf


Can you test this?


カッコ前後にスペースが有ったり無かったり、安定していないような気がします。

expect(page)

ですかね、、、


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 2](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=56&buildTypeId=Xbim_XbimScripting_XbimScripting) is now running


Just use a method reference:

.map(pathSegment::getMatrixParameters)


Buat direktori userconf di dalam folder /etc/vsftpd untuk meletakkan config file masing-masing user supaya bisa melakukan koneksi FTP ke directory yang disebutkan dalam konfigurasi.
1.Bikin file baru sesuai nama user di/etc/vsftpd/userconf/(NAMAUSER),
isi ini

local_root=FOLDERUSER,

2.Tambahkan NAMAUSER di /etc/vsftpd/userlist,pada connnfig bagian
 pasv_min_port=35000
pasv_max_port=35010
pasv_address=ip.ad.dr.es

Sesuaikan dengan IP Address pada instance / physical server Anda, pasv_address dan pasv config tersebut diatas akan berfungsi sempurna apabila server Anda menggunakan firewall atau virtual env (cloud/openstack/kvm/qemu/ovz/dllsb). Silakan allow port 35000-35010 dari firewall Anda berguna untuk menerima koneksi passive mode.


TeamCity Xbim :: Xbim Scripting :: Xbim Scripting [Build 2](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=56&buildTypeId=Xbim_XbimScripting_XbimScripting) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:28


That somewhat defeats the use map / Optional / ...

Just avoid null by chaining further.


MySql是不是就不能用了？


what do you think man @codeschool-kiddo


But why?  Looks like you reimplemented  matrixParameters.get()???


Fprint can be used in another country. May be take encoding from device?


能提供一下 关于这段代码(这次 commit)的解释么?    遇到了同样的问题 谢谢


Я думаю этот файл уже можно удалить.

git rm newfile а потом опять закоммитить и запушить


I agree, but I was planning on doing that in one of the subsequent commits (and pull-requests) that will be needed to complete the xenprep feature.


Ah my bad. I took a look at the compiler code and I see that they also have visit methods that take method bindings. That's a bit strange to me, but looking at their code, it doesn't really look like they are "visiting" such bindings; they are just processing them in some way.


Я ксати не помню, возможно git rm удалит только в репозитории файл, а локально он останется


Fair point. I'll do that.
I don't think Db has get_by_name_label but I can use get_records_where.


My apologies, as I mentioned earlier, I see that this is from the compiler code.


I'm not sure that I agree with this naming. What is the purpose of raising the probe between probe attempts?
I assume that it serves two purposes; 1) It avoids scraping the probe across the bed and 2) It provides some "throw distance" within which we expect the probe to contact the bed. All of this action takes place while the probe is in its operating position.

Similarly, I see two functions being addressed at the beginning and end of the sequence.
1) Provide clearance to allow the probe to move between its in-service and out-of-service positions. This can be either to account for the extra overhang of a rotating probe arm or to avoid some obstacle that is above the bed level, or
2) To "park" the probe in a position appropriate for manual probe deployment.

In both scenarios, it is unclear whether the distance specified is the total height of the nozzle or is in addition to the difference in the height of the nozzle and the height of the probe.

Before we go about changing everything, yet again, we really SHOULD define the terminology that we intend to use to describe our measurement spaces.


Why was this removed?


Using static method allows to avoid the boiler plate of having to
instantiate objects, the main idea was to keep the framework as clear and
easy to understand as possible.
I in fact used the Page Factory in some areas. Please take into account
that it is not a final implementation, i couldn't enhance it because of
time constraints. I had an medical appointment and i wanted to deliver the
exam as soon as possible

On Tue, Aug 11, 2015 at 9:32 PM, Carledriss Zeiss <notifications@github.com>
wrote:

> This is an interesting solution using static methods :)
> but you will need to explain what advantages you will have with solution
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xaindeneb/SFDC-Ariel-Mattos/commit/2011f07bc7058ab772d3b7b84a50fd3364cf5bb3#commitcomment-12654991>
> .
>


Thanks :)


If I understand the document it use only 866 encoding:

> ККТ принимает текстовые строки в кодовой странице 866 MS-DOS, но хранит и
> возвращает их в собственной внутренней кодировке (см. Приложение 6, стр. 233).
> Только символы собственной кодировки ККТ можно передать на печать или
> запрограммировать в «строковых» ячейках таблиц настроек ККТ. Остальные символы
> кодовой страницы 866 MS-DOS не используются – при приеме от хоста ККТ заменяет
> их на пробелы.


What I'm understanding they are also getting try catch and throw exception node, how they are determining there If 9/0 inside the for each loop. I haven't seen that, also haven't seen any particular  code that getting old the exceptions,


Hm, Why do you want to model exceptions as `Statement`s?


Kris, what would you say to rename CaseMgmtUtil to CaseMgmtServiceImpl (like we do ProcessServiceImpl, UserTaskServiceImpl, and others)? For users it would be easier to determine the implementation of CaseMgmtService.


Looks important, no?


ここの変更って、コミットの内容と関係無いような…？


Hi Andrew,
* The search form already works with searching on tags and comments. I have added sql file so that to combine searching on post titles, tag titles and comments body.
* We made some corrections on the css files, so that the Responsive design is OK now, I think. Please look at it.
* We have written the tests for the search form
Please look at the corrections and comment on them. If you think it`s OK, you can merge it.


thx!


Lol, Np :)


> Do we know how portable and reliable this method is (across JVMs, for example) ?

Quite reliable :), but as you says in the other comment, for our use case, it might be better to check if the classes are available.

> Would it be useful to cache the value in a static final field?

I don't expect huge differences since it's used mainly when registering the providers and when using the QueryBuilder but I will change it.

> What if it fails? I'd try to catch and log exceptions, and decide on a fallback strategy ( 7?

I think it will only fail if we don't have permission to read  the properties, but if an error occurs I would just log and return false. If the user needs some of the feature we provide it will have errors that should point to the problem, anyway.


:smile:


create user、、ですか？

ユーザーが既存の場合は、createしないような？


I was thinking to get all the statement add into an list and iterate through them to find exception,


こちらも create no userでは無いような？


it needs to be `"Windows"` (with a big letter)


Can we change the name to 'Upgrade-Satellite6' or something else.. ?


（感想）ユーザーが存在する場合のテストと同じように見えるのに、なんで通るんだろう…？？？


This is going recurse forever. We're inside operator << for pkt.


This does not compile on other platforms. Would you mind trying if you have a Mac (or Linux) around?


Yep, one of those simple fixes but a bit cumbersome to track down.


I think we should show only two flavours 'm1.large' and 'm1.xlarge'. Because if user selects tiny/small/medium then installer may fail due to insufficient resources.


That might be a better approach for our use case, do you think it might happen that we will have false positives?

Anyway, I'll write a POC


@ctrueden Was there a reason why the version of scijava-ui-swing was overriden?


script name should be like upgrade.sh or satellite_upgrade.sh ?? Just a suggestion.


$no is unused i think :)


No.


Is this necessarily a hard failure? Could we simply take the first in the list?


ok, please stay with `removeFirst(Predicate)`

Using a null predicate should throw as part of the method contract and `HashSet<Predicate<Xyz>>` would work: `remove(Predicate<? super Predicate<Xyz>>)`. But `remove((Xyz) null)` is a no-go!


Previous indentation was the right one.


Sure ?


@ebdrup I agree, but I don't know how to handle it. At the moment in official mongojs there is no support for mongos topology that we need for our mongodb 3.0 on compose.


Some recent work that might be relevant is: https://hackage.haskell.org/package/promises.


how do i update


LGTM


@ctrueden I am not entierly sure about this one. Are a` synchronized` method and `synchronized(this)` statements exchangeable in this fashion?


I guess so. Are those unique across tables? How does one make them kept?


And maybe you should extend the if with an else clause, you execute the file-set on windows twice otherwise:
```python
if platform.system() == "Windows":
    self.file = os.getenv("APPDATA") + "\\coala\\.coala"
else:
    self.file = os.path.expanduser("~/.local/.coala)
```


It looks like this commit is broken since it contains generics in types which aren't supported in Typescript as far as I can tell.  For example, the first line of `rx.all.d.ts` says:

```typescript
// Type alias for observables and promises
export type ObservableOrPromise<T> = IObservable<T> | Promise<T>;
```

which gives me an error of:

`node_modules/rx/ts/rx.all.d.ts(4,36): Error TS1005: '=' expected.`

If there is something I should be doing to support generics let me know, otherwise I will open this as an issue.


We use a custom test setup to measure the vhost performance. The full procedure and the results for `v2015.07.1` and `v2015.08` are presented in #592.

This patch is an attempt to identify the major change that affects the packet processing and bumps the test results slightly up to `9.8 Mpps`.

The patch descritpion will be updated to refer to the issue and with the new results.


This commit broke my page, I can no longer use `ng-class` on an accordion group


@kenathe


I could be wrong in my approach--it's not done yet either. Hopefully I get it done by the end of the week. I'm at a conference the rest of the week now but I'm trying to take time for this.


A newline too much :)


The commit message should have been: Attenuation fixes. The results are not completely off, but it does not converge yet.


Well, I agree they should work for the GP approximation of the SP at a given point. But I think we need to be careful about reporting those feasibility stats out, since they might be at a local, not global optimum. The problem might for example be feasible, but our SP heuristic might fail to find a feasible point.

Two options:
1. Similar to solve/localsolve, we could require the user to call `local_feasibility_report` for Models involving Signomials.
2. We could require the user to call the feasibility on the GP approximation of the feasibility SP at the last iteration.


This is potentially something the user can solve rather than an internal error. Could we raise a proper XenAPI error?


Please use more meaningful parameter names.


You use pytest right? Then you don't need this statement (I think)


Here also an API error would be better.


This is the kind of refactor I like ;)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.449.3d202c2c2ee78c947c9df336a6c6ba1cd377f134](http://build.palaso.org/viewLog.html?buildId=60970&buildTypeId=bt223) outcome was **FAILURE**
Summary: Compilation error: Palaso.Tests\Palaso.Tests.csproj (new) Build time: 00:02:26


commit message


no \n at EOF


I wonder about period at the end of comment sentence.
Should we add it everywhere or just after the first sentence so that Javadoc counts it as description sentence?


Could just inline the `bank_info_for` method here - I don't think there's any reason for it to be a class method instead of an instance one


I think this is also not needed from pytest


Maybe choose another name, since you test both, `add_project` and `delete_project`


The idea of that new rule is to detect the absence of `PM.stamp` and convince `make` to run the respective rule in the top-level `Makefile`.


Surely you need a change such as this to actually use the new surveys? :
https://github.com/zooniverse/Galaxy-Zoo/commit/0c445435b391c4b690e972f98abd52f5aa5fb194


okay thanks will take a look at this. I don't checked everything twice because of the large amount. But that's no problem IMO, we can just find and fix it now ;)


I'd call it `static` or `static_text`


"Static text"


urlcolor没有使用，我就放在最下面了


If you want to support parallel models (now that support for jags is dropped) you can take a look at runjags https://gist.github.com/joelkuiper/d00d7d8bc548d54ba593


```
def config_form(self, *args, **kwargs):
    form = StaticTextConfigForm(*args, **kwargs)
    del form.required
    return form
```

(the "required" option makes no sense here)


why not `is_active`?


This was not doing anything in the code,


I'd use `{..}` instead of `(..)`


Hello
I am trying to calculate the kappa by the means of your blog.
I should be thankful if you tell me which part of log file we have the " ave of total per-atom in/out energy" in langevin method,
and also average of tdidd in heat method.
Many thanks in advance


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.423.3d202c2c2ee78c947c9df336a6c6ba1cd377f134](http://build.palaso.org/viewLog.html?buildId=60971&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3331, ignored: 131 Build time: 00:09:58
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.reporting.ErrorReportTests.GetStandardProperties_ContainDesktopEnvironment: <no details avaliable>

```


Oy vey :dizzy_face:


I do like to thank you very much for your helpful website too


I tested this with single particles from the particle gun in my setup
which went fine, If I use the default macro it does crash. There is an
issue with 32-<64 bits in the cells which then crashes the backtracing
in the evaluator. There are also some types in the evaluator itself
which need changing. I am working on introducing typedefs in the include
files which then can be used to make this consistent.

On 8/12/2015 1:01 AM, Jin Huang wrote:
> @pinkenburg @mccumbermike , is it to me alone or also happen to others: It appears the Fun4All run still crashes silently, unless do_svtx_eval is set to false. With a test run using high Fun4All verbosity, the problem seems point to PHG4Evaluator ...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sPHENIX-Collaboration/coresoftware/commit/4d14bb1ad55b092b54bbacf2cdd59ba09b126638#commitcomment-12656529

--
*************************************************************

Christopher H. Pinkenburg	;    pinkenburg@bnl.gov
				;    http://www.phenix.bnl.gov/~pinkenbu

Brookhaven National Laboratory	;    phone: (631) 344-5692
Physics Department Bldg 510 C	;    fax:   (631) 344-3253
Upton, NY 11973-5000

*************************************************************


Зачем нужен этот класс, если можно переиспользовать VKUploadPhotoBase, немного его изменив?


Maybe you want to supply `file` as an argument and by default it's None. When it's None you auto-detect it like you already do.
If you do so don't forget documentation


documentation


here also docs are missing :)


Zwliew MTP and led battery full is fix ?


One note here - nested def's don't work as you might expect in Clojure - their scope is always the whole file, and re-def'ing the same name multiple times can make things confusing. I'd suggest for this case either a top-level function that returns the mock socket, or using speclj's `with` feature. Similar for any other nested `def`s.


This configuration is also present in the MongoDB storage. I've updated that as well.


Where does this property come from?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.450.fb1806e4539eb9dc26d744175d4e1febd0376259](http://build.palaso.org/viewLog.html?buildId=60972&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 132 Build time: 00:06:56


since every function does not keep a shelf opened, I would suggest to make it not a "public" member variable.
I would remove this line and make in `add_project` and `delete_project` `self.shelf` not a member --> just call it `shelf`


Why is the redirection code is added as a parameter?
The problem here is with with the redirections it gets added as a url parameter i.e when accessing /latest it redirects to /version/?code=301
In the next commit i dropped the code to fix this but it might be better to actually solve this. I don't know how though. Any ideas?


Did you mean is_required?


Put the readme file in


I must say (now, as of  4b11c841) that I right now never get a popup telling me about the missing files in the session. No matter how I proceed I only get the popups when a file is to be displayed, one popup per file.
I (deliberately) haven't looked into the new code yet but wanted to report that. What I would suggest now is that when the session is loaded and missing file(s) are detected the popup should be displayed (before a file is displayed. This should present a list with the missing files (as we had it earlier) and provide the option to remove all the files or not. If the user removes the missing files *now* we won't even try to open them afterwards.

I like the approach with the tooltip, but found it missing the full filename. It can happen quite often that files with the same name are present in different folders. Consider

```
/path/to/manuscript1/page01.pdf
/path/to/manuscript2/page01.pdf
```
I added this in f9d20844


it's usually just a class attribute, but since it's always accessed via `self` nothing stops you from turning it into a method


This line of code looks suspicious to me. Shouldn't the word be byte-swapped? @magnumripper


thanks


eh yeah, whatever the name of the required field is


missing newline


:red_circle: There were too many files in this commit (not including the third-party libraries like bootstrap). The commit message does not adequately describe what was done in this commit. Also, the commit message does not follow the convention as described in the `CONTRIBUTING.md` document.


not convinced on the method name -- I would expect that a method called `feasible` would return a `bool`.


I think div is better than span+br .
br is hard to update in CSS, div os more easily updatable. ANd better here :
A Note On Privacy + This survey is anonymous. are not in same "topic" but more title/subtitle of explanation .

:)


.profile contains more instructions than just our. With this we will wipe /root/.profile and only add our stuff. The arguments must be in the end of the .profile file, that's why I used sed -i.


maybe you want to open a custom file in the current directory to not leave the tester behind with an empty file on his filesystem.


Нету закрывающегося тэга </head>


:red_circle: There were too many files in this commit (including the third-party libraries). Also, the commit message does not conform to the conventions as described in the `CONTRIBUTING.md` file.


Our parser does the following:

```bash
$ ./vendor/bin/rql rql:parser 'limit(1,10)'
query
└─ limit
   ├─ limit: 1
   └─ offset: 10
```

It doesn't support the following though:

```bash
$ ./vendor/bin/rql rql:parser 'limit(1,10,100)'

  [Xiag\Rql\Parser\Exception\SyntaxErrorException]
  Unexpected token "," (T_COMMA) (expected T_CLOSE_PARENTHESIS)

rql:parser <rql>

[13:23:20][hairmare@jenny ~/Documents/git.repos/php-rql-parser](master)$
```

This leads me to say we support the following call signature:

> limit(start,count) - Return the given number of objects from the start position.

I'll update these docs to reflect that.


I pushed a bump to 8.1.1.

(Let's take further conversation on this topic, if there is any, to an issue. I'll lose track of these code comments.)


This is activated in self-signed-ssl.sh instead.


they were related toTypeBinding visit, that why remove them, I am putting them back, would require your instruction as forward


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.424.ce2e9089674008f6718ab47c68e297e3f7d0cef5](http://build.palaso.org/viewLog.html?buildId=60973&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3332, ignored: 131 Build time: 00:09:20


Thank you very much for this commit. It has served as the base to add preEdit, preSave and preView checks and to add the Global Variable validation in a way that it doesn't require base code changes anymore.

Related commits:
c48534117c4a941fb1e60112442deaceb8f53da5
01e84ed4c22b9fd629c0bbb1f95b38ddd88591b1
72b4acad25d345c15a3b2399dd2a18bc858f0ffc
b6b6acd66c269d3a9188bcc74eacbcf92fab9fa1

I will delete the branch as the functionality has been included in the master branch.


Класс header__logo-container нигде не объявлен.
Можно ли эту разметку свести к  следующему виду?

Site Name


I'd like to see this test split up a bit more so that each test checks just one thing. Perhaps we could split it by user, or by permission type.


So when will welcome.sh be activated? Should we put that in rc.local instead?


Students/Farcas Silviu Vlad/Assignment 6.2/Washington univ/src/questions.java

did you post this now? or are they older questions. Let me look through them


:yellow_heart: It's recommended that you remove code that is no longer in use. This includes code that has been commented out.

Removing redundant code helps prevent confusion. Here's an exciting explanation of this: [Remove Zombie code](http://www.bitnative.com/2012/10/22/kill-the-zombies-in-your-code/)


I find these examples confusing -- `feas = m.feasible` creates a pointer to the method? And it can get called as if it's a dict in `feas["overall"]`?? I think it would be better to stick with calling the method.


A reminder, we also need to add a flag which says that a SourceFileType can be streamed if it comes from the dunya website


This might be better structured as more standard prototypal setup:

```javascript
function EventWalker(registry) {
  this.registry = registry;
}
EventWalker.prototype.inRegistry = function EventWalker_inRegistry(id) {
  return !!this.registry[id];
};

EventWalker.prototype.closest = function EventWalker_closest(closest) {
  do {
    if (closest.id && this.inRegistry(closest.id)) {
      return ['id', closest];
    }
    if (closest.hasAttribute('data-ember-action')) {
      return ['action', closest];
    }
  } while (closest = closest.parentNode);
  return null;
};
```


Thanks, merged!


I also don't really like mutating `m.constraints`...


Hmm aku coba pendekin aja kalo gitu
Expku gak terlalu banyak jadi gak keliatan kalo kelewat batas atau kepotong
Jadi tadinya pikirku aman aman aja

Dan ya indentnya lumayan behamburan
Ntar aku rapiin


Agree. Need to write ostream operator for asio::const_buffer..


super psyched about the mega docstring!!


Класс featured-content__feature-container объявлен в сss?


newline


The first test use git


Класс reviews__inner-container объявлен в css?


delete


Should also be setup-when-root.sh in the /root/.profile, not welcome


Can we return early here (move to line 234)


I see no return True


cat >> appends to the end. Do not confuse with cat >.


fixed in https://github.com/storebrand/storebrand.ny/commit/3df659f6d8d4e6399574f6ee279bcb9ce194cb63


Hum, I propose this:
mozregression 0.41 fixes [bug 1185756], which caused the following warning on Windows:

    WindowsError: [Error 32] The process cannot access the file
    because it is being used by another process

Also, mozregression is now more tolerant when facing failures like a build not installable
or a crash from the tested application (see [bug 1192485]).

And finally, when using mozregression with a custom profile,
it won't be modified anymore (see [bug 999009]).
This bug was there for a long time. Thanks Jonathan Pigree for your help in fixing it.

[bug 1185756]: https://bugzilla.mozilla.org/show_bug.cgi?id=1185756
[bug 1192485]: https://bugzilla.mozilla.org/show_bug.cgi?id=1192485
[bug 999009]: https://bugzilla.mozilla.org/show_bug.cgi?id=999009


Can the documentation include that file_type is a slug, not a SourceFileType object, or id. Perhaps a different name


"Lorem ipsum laro dolore sit amet vivamus" по сути является заголовком.
По семантике правильнее использовать тэг "h"


Cool, thanks.


yes,  I'm test


"Lorem ipsum dolor sit amet" также является заголовком


Correct. However, your `MyString` class (even though it's used for the sake of exemplifying) indicates that you'd think about extending the native `String` class maybe? Why is that not possible? And furthermore, why did they choose to make it like this?


On second thoughts - perhaps we can use the `rate_limit` flag. This flag is specifically for regular users who want to stream from the website anyway. Maybe we need a better name.


Тоже нужно использовать тэг h


Nothing to object to. Well done.


Hopefully that satisfies the plural rules once and for all so the translation contributors can do their magic.


"Integer vitae libero" также является заголовком. Но если для случаем, перечисленных выше используются заголовки h1-h2, то для этого больше подойдет h3


Looks ok.


У всех картинок должен присутствовать обязательный атрибут "alt"


GitHub not Github.


Hi @andreasnoack, Thanks for the ping - removed it.


It isnt. Notice the "L" vs "l"


I'm still not sure how one convinces a dry-run to become a wet-run when in mid flow. (also the profusion of "PM.stamp" references as targets and files may also be part of the confusion)


Открывающаяся фигурная скобка должна быть на одной строке с селектором: .banner {


TeamCity Subsonic :: Debug [Build 32](http://109.103.226.38:8512/viewLog.html?buildId=34977&buildTypeId=Subsonic_Debug) is now running


Можно ли в этом случае объединить эти стили, перечислив  селекторы через запятую?


TeamCity Subsonic :: Debug [Build 32](http://109.103.226.38:8512/viewLog.html?buildId=34977&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj (new); compilation error: Client.Common\Client.Common.csproj (new); compilation error: Client.Common.Tests\Client.Common.Tests.csproj (new); compilation error: Client.Tests\Client.Tests.csproj (new); compilatio... Build time: 00:01:09


These settings need documentation, preferably directly in the docs => `docs/configuration/config_yaml.rst`


TeamCity Subsonic :: Debug [Build 33](http://109.103.226.38:8512/viewLog.html?buildId=34979&buildTypeId=Subsonic_Debug) is now running


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.451.ce2e9089674008f6718ab47c68e297e3f7d0cef5](http://build.palaso.org/viewLog.html?buildId=60974&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 3304, ignored: 132 Build time: 00:27:34
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_CallbackOption_Get10LinesAsynchronously: <no details avaliable>

Palaso.Tests.dll: Palaso.Tests.Progress.ConsoleProgressTests.LongRunningMethodUsingConsoleHandler_ProducesLog: <no details avaliable>

```


build -> delivery


Working on the translation as we speak! Great job, thx for sharing this extension! :)


TeamCity Subsonic :: Debug [Build 33](http://109.103.226.38:8512/viewLog.html?buildId=34979&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj; compilation error: Client.Common\Client.Common.csproj; compilation error: Client.Common.Tests\Client.Common.Tests.csproj; compilation error: Client.Tests\Client.Tests.csproj; compilation error: Common\Common.c... Build time: 00:00:19


should be ESH-INF/automation/moduletypes


Имеет ли смысл дублировать значения для font-weight?


Just discussion whether this is really necessary...


aah good call!


good point, don't actually need recursion, the while loop is fine.

Therefore, can't we drop the `zeros=True` kwarg and just put that initialization inside the method?


do we need this Activator at all?


TeamCity Subsonic :: Debug [Build 34](http://109.103.226.38:8512/viewLog.html?buildId=34981&buildTypeId=Subsonic_Debug) is now running


Please insert your full name here


Doesn't setting this option cause issues with not returning a caret position? I don't recall which browsers, but some won't return a caret position unless the element has focus. I'm pretty sure that list includes IE.


Дублирование стилей для body. все стили можно объединить в один блок. Лишний знак ";" после "color: #778f92"


TeamCity Subsonic :: Debug [Build 34](http://109.103.226.38:8512/viewLog.html?buildId=34981&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client\Client.csproj; compilation error: Client.Common\Client.Common.csproj; compilation error: Client.Common.Tests\Client.Common.Tests.csproj; compilation error: Client.Tests\Client.Tests.csproj; compilation error: Common\Common.c... Build time: 00:00:15


So as I said before I am really happy about the idea of you maintaining a long lived branch. This worked out really well when you were in charge of the straightline code integration and merge that was very complex indeed :).

Once your branch is added to `branches.md` it is going to be automatically mirrored on the SnabbCo repository. That means that everybody who clones the main repo should have a copy of your branch and they can simply write `git checkout virtualopensystems-contributions` to test it and `git merge --no-ff virtualopensystems-contributions` to merge the changes into their own tree.

Note: you would not be able to rebase this branch once it is published in branches.md. The script that does the mirroring is doing a `git fetch` from your tree and `git push` to the SnabbCo tree (without `--force`). This means that code pushed has to stay pushed, which is the same rules as other long-lived branches like `master`, `fixes`, and `next`.

The ideal from my perspective is for this branch to contain only changes that are already reviewed and tested and should be ready to apply directly. This would make you responsible for making sure the code is reviewed and tested before it gets pushed to the branch i.e. pushing on this branch is a way of publishing code that is finished. This is the same model as when people maintain a long lived branch of Linux or QEMU.

This means that review and testing would have to happen separately. One way to do this would be to simply send the individual changes as Pull Requests to the SnabbCo/snabbswitch repo as always but to mark them as being targeted for the `virtualopensystems-contributions` branch as the next step. (Could set the Target Branch on the Pull Request this way for example.)

You could also do the review and testing in some completely other way e.g. as Pull Requests on the virtualopensystems/snabbswitch repo instead or with Gerrit or ... anything you like. The important part for me in terms of scaling up development is that when multiple changes are PR'd together on a branch that they are already in good shape so it is easy for me to merge them onto `next` for further testing.

Does that make sense?

I could maybe suggest `vosys-nfv` as a better name for the branch. That would be easier to type (`git merge vosys-nfv`) and also quite descriptive: it's the branch containing NFV related changes that the virtualopensystems team have tested and approved. This is a bit like the way Redhat, etc, have their own branches of Linux that are not identical to Linus's tree.


shimata, dat avatar!!


лишний пробел перед ";"


Is there an ordering on endpoints such that taking the first is meaningful?

We probably want to add `__meta_kubernetes_port_NAME` as a label with the value being the port number. Then if someone has a convention around port numbers names (e.g. one called `prometheus_metrics`) they can relabel things in accordingly.


Hrm, good point! Thanks, I guess going through RefCell makes most sense then


work good with 11


why .5 two times ?


Can you put some inline comments around this section of code?


Remember that this [problem] is spawned from the main git/makefile https://github.com/PhilipOakley/git/blob/f6f8da0e1c4f72ffb766a0c908ff9043238cf243/Makefile#L1740-1748 where the dry-run fails at L1745 because of the non-existance cascade. This is an alternate to using NO_PERL as a fake way to disable it.

I have to admit that it's all very convoluted around that auto-update code and ensuring that it doesn't error out when being dry-run.


dd


TeamCity Subsonic :: Debug [Build 35](http://109.103.226.38:8512/viewLog.html?buildId=34983&buildTypeId=Subsonic_Debug) is now running


In other words, what are you trying to do in saveStates()


ok indeed there is only one selection of lepton-b pairs for single top, i forgot that.
For ttbar we have ~6 alternative selections to compare


Fixed in 2a68ea956197. Thanks.


Fixed in 2a68ea956197.


Kadarkoli spremenijaš tole, je treba spremeniti še pojavitve istih nizov v drugih delih widgeta (sicer crknejo signali).


TeamCity Subsonic :: Debug [Build 35](http://109.103.226.38:8512/viewLog.html?buildId=34983&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Common.Mocks\Common.Mocks.csproj (new); compilation error: SubLastFmTests\SubLastFmTests.csproj (new) Build time: 00:00:45


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1796 of 1802 tests passed)

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


Cool, thanks a lot! I took mostly everything. :) I Will ask :wlach for a final review still, but this is far better now!


Why might this occur?


I wonder why this is necessary in `:compile`..


if you want to ceil, you can use the `//` operator, so you don't need to convert to an int again.
btw why you just don't do
`if total_secs < 60`
?


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


the same here (at least with the ceiling operator)


이 type 정보는 common 쪽에 있는 타입 정보와 맞추는 것이 좋을 것 같습니다. 향후 binary reader/writer와의 연동에도 그 편이 훨씬 편할 것 같네요..


and here also


You might also want to check our RFC 2544 tests which also do a binary search to find the maximum loss-free rate:

https://github.com/werpat/MoonGen/tree/rfc2544/scripts

This was done by one of my students and he should be turning in his thesis in a few weeks, so there will even be some documentation soon.


Style guide pls. Tabs indents pls.


yes


the outer braces are superfluos:
`timediff = time.mktime(reference_time) - time.mktime(timestamp)`


Chimie


Can this become "arguments.join(" ")"? Some users have spaces in their name ;)


as a reminder i've done the merge in my video branch
maybe have a peak at it ...

also i've created a lot of new files, corresponding to all the needed subclasses, maybe only one big file with all subclasses is a better approach ...


Fix _ALL_ the indents.


Missing documentation


fxgfcjgv


Also, would you please provide a code example in the wiki pages of using this option along with `openOn` and the `onfocus` handler?


Same semantics as in [_processTemperatures](https://github.com/spapadim/OctoPrint/blob/94dee723f17106ec2476236ca975df1e39e72635/src/octoprint/util/comm.py#L775) really. The temperature reporting is sadly not consistent across firmwares, so a `T` might mean the first extruder, or the currently selected extruder, or the only extruder, depending on what else is present in the temperature line.


Some inline comments would be nice to explain the steps :)


Sure, forgot about that. Will fix


Hello Herman,

Thank you very much for the very detailed benchmarks! It seems that unless we can use PHP 5.6+ and array unpacking we are better off with the ugly switch as it makes a measurable difference on shared hosting.

I did expect HHVM to be very fast since it does some good internal optimisations. Too bad we can't run Joomla! on it reliably due to some nonstandard behaviors it displays when compared to official PHP :(


@enghqii parser 쪽 테스트 코드도 make test 시에 자동으로 테스트 될 수 있도록 CMake 연동해 주세요..


Sorry, I didn't actually activate the "session pop-up" because I wanted to discuss it first. If you want to try it out or have a soulution in mind, call [`checkMissingFiles`](https://github.com/wbsoft/frescobaldi/blob/manuscript-viewer/frescobaldi_app/viewers/__init__.py#L686). It will check which files is missing and emit the signal as before.

The change of the tooltip looks good!


So what?


@visualcookie Oh oh. Please remove and submit a pull request. :warning:


Hm... I think you need to also reset `self._temperatureWIthinLimitStartTime[tool]` here. If for whatever reason the temperature once hit the set point, than overshot, then hits it again after `time > threshold` but then undershoots forever, you'll trigger the even although the temperature isn't really reached.


Overall structure (at least one empty line between each of the following):

1. Licencse Header
2. System-wide imports in alphabetical order.
3. Third-party imports.
4. Project imports


```
session.setdefault('submitted_suveys', set()).add(survey.id)
session.modified = True
```


Just FYI, when I said "it works fine", I had tested it on a pristine vimrc with only this plugin.

My intention was to spend some time later, figuring out what the conflict was with the rest of my vimrc and the myriad plugins I have.

I tried to do that, but was unable to come to any conclusion in reasonable time.  At the moment my solution is to revert e098491906c4e6d8a2a4d10ab60b91570eb81113 in my copy.

I'm just adding this as an FYI; I am not asking you to fix what is at best an interaction with something else (and without saying what that something else is!)


TeamCity Subsonic :: Debug [Build 36](http://109.103.226.38:8512/viewLog.html?buildId=34986&buildTypeId=Subsonic_Debug) is now running


Please use [] instead of list:
```python
 packages = []
```


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Please use {} instead of `dict`:

```
data = {}
data2 = {}
```


TeamCity Subsonic :: Debug [Build 36](http://109.103.226.38:8512/viewLog.html?buildId=34986&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:46


After revisiting the matter, I've realized the error here is mine.

I'll be reverting this particular change in the next commit.


And I don't like the function name, it just tells nothing what is performed inside.
Also:
I would redesign the function to only accept a `datetime.timedelta` and just translates this into a string, since this is the core of this function. Maybe calling it... okay I can't come up with a good name for it.
And if it's problematic to create timedeltas easily, provide a helper function that creates one from start and end time like you do here.


"Check whether the user has submitted a survey" - more concise.

Also check if there's a survey submission associated with the user.

    session.user and survey in session.user.surveys

If that doesn't work:

    session.user and session.user.surveys.filter_by(survey=survey).count()


why the () around it? (same in getCSSFiles)


This also applies for all other if's:
Even the control flow is the same, I think `elif` is more appropriate here.


@jdsn thanks for the info and raising the limit! Please can you emphasise the importance of marking builds to keep on cloud-devel@?


We have no reason to suppose that the first will be the right one.
There will be one correct VDI supplied as standard with XenServer.
If a user has added another VDI with the same name, in an SR with the same name, then is there any way for us to find out which is the right one? (We could check `description` as well as `name-label`, but the same principle would still apply.)


trailing slash missing. `methods` is not needed here.

I'd go for `display_survey_list` to make it clear that it's not management


How do you be a speed hacker
Or a hacker247


imho the `/survey-form` is not really needed in the URL. we don't have anything else there after all. for the endpoint i'd go for `display_survey_form`


I'm not the gettext expert, but don't you use translations from coala only?
So coala may embed translations that only belong to your project. (as far as I understand)


Did you mean 'constructed' or 'structured' in the commit message instead of 'structed'?


"It will be interesting to see..."


>What I would suggest now is that when the session is loaded and missing file(s) are detected the popup should be displayed (before a file is displayed. This should present a list with the missing files (as we had it earlier) and provide the option to remove all the files or not. If the user removes the missing files now we won't even try to open them afterwards.

Ok, so you suggest we should add the option to remove all or keep all in that popup? If the user decides to keep the files do you mean the extra popup is then ok?


Possibly, but see the subsequent commit in which I changed so that we don't add a new VBD anyway.
(The design page says we should just use an existing empty CD-drive VBD, or raise a suitable error if there isn't one. That's how it works in the next commit.)


This is not equivalent with the Python 2 version. self.task() creates a new generator with each call. Fixed with commit https://github.com/OndrejSlamecka/setroubleshoot/commit/159f5f30e2a69eaa904cf556a1b6d1b59d999524


you could do that right in `_process` - no need to store it in on the object


    normalize_url_spec = {
        'locators': {
            lambda self: self.survey
        }
    }


Really when I see this bugfix in this screen I'm tempted to shoot myself for this act of stupidity.


Please don't place code in the file itself. EVERY piece of code should be in an appropriate function (or method).

All of these should be called from the `main()`:

```
def main(enc_input, package_input, output):
   encryption_data = read_encryption_data(filename=enc_input)
   package_data = read_package_data(filename=package_input)
   merged_data = merge_package_encyrption_data(encryption_data, package_data)
   write_out_data(merged_data, filename=output)

if __name__ == "__main__":
   main(sys.argv[1], sys.argv[2], sys.argv[3])
```

This helps to improve code re-use and readability


Help


Please **DO NOT** use `params` ever. It is legacy and should not be used for  anything besides passing ig to legacy `checkParams` methods.

        self.survey = Survey.find_one(id=request.view_args['survey_id'], is_deleted=False)


If I remember correctly, that depends on the firmware sadly. Some report only the temperature for which the heatup was triggered (btw: external heatup = heatup-and-wait triggered through LCD OR printed SD file, not just a new target temperature), others report all temps (with only one of them really changing). Some include the target temp, others don't.


"I hope you like it." or "I hope you'll like it." Whichever fits the original Korean.


"Your answers have been saved" or something like that sounds better imho


L7 to L22 is basically implementation of `read_encryption_data`.


no here are two, well should be two.


This line is not required since you are using context manager.


you can assign `submission.user = session.user` directly


one for each news item and one for global


Gah, IDE was set to automatically "clean" file (for our indentation)


L23-L33 is basically `read_package_data` (or better name it `read_license_data`)


why not `for name, value in form.data.iteritems()`?


TeamCity pull requests :: netty [Build 8](https://garage.netty.io/teamcity/viewLog.html?buildId=248&buildTypeId=pulls_netty&guest=1) is now running


Only import packages at the beginning of the file.


Yes ...Tested


"Poata at Grynas Hills is well known.."

I'm also confused as to what "they" is referring to here.


test 디렉터리 밑에 다른 파일들이 많이 추가될 예정이니 이들을 모두 포함하여 테스트 코드를 빌드하고 실행할 수 있도록 CMake 파일을 작성했으면 좋겠습니다. 가능하면 나중에 테스트를 추가할 때 더 이상 CMake 관련 파일을 수정하지 않고도 단순히 테스트 파일 만 추가해서 실행할 수 있도록 만들어 주세요..


a lit mistake


I did not mean that they were removed, just that storing pairs `(environment key, path)` in a list that is called search_paths looks odd/nonidiomatic. But I suppose it's fine.


fixed with inheritance of common components form GenericMenuView


Maybe TODO or BUG or FIXME is appropriate.


L41-L76 is basically `write_out_data` and `merge_package_encryption_data` combined.

Please split them.


不懂哪里？


1: M100 needs more reviewers
1: M101 signature is missing
1: M190 line is too long (52 > 50)


1: M111 unrecognized component name: cds
1: M190 line is too long (52 > 50)


1: M111 unrecognized component name: cds
1: M190 line is too long (51 > 50)


이러한 변수 선언/정의 부분은 최대한 최상위 CMake 파일로 올려서 관리했으면 합니다.


you can use the `simple_form` macro here (or do you really not want a cancel/back button?)


3: L102 copyright year is outdated, expected 2015 but got 2014


1: D100 Missing docstring in public module
1: L101 copyright is missing


1: L101 copyright is missing


1: M111 unrecognized component name: cds


nice ...........


Why ? :'(


Thanks, i'll have a look over the weekend, if ok, i'll merge it just before I release the Isengard version of Amber


Ok, perhaps I should review the squashed view instead :)


The initial commit was June 1, 2009 over at Google Code, which is going away. https://code.google.com/p/roundhouse/source/detail?r=1


TeamCity pull requests :: netty [Build 8](https://garage.netty.io/teamcity/viewLog.html?buildId=248&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 3291, ignored: 6; compilation error: Compiler (new) Build time: 00:02:57


Maybe move that whole if/else block in a separate method, called something like _getVersionData() and make sure the doc block explains this is done for BC reasons?

All the old stuff will go in the future. But I really prefer a slow and smooth transistion for the users of the plugin.


"Symbol of the Regwyn Family"

It's just a minor gripe but, "I will help you advance to a higher level," is what I'd put here. Though the first "to" is grammatically correct but it makes the sentence sound stiff and a lot of lines in the translation are like this. Your choice though.


Yes, I can think of the following constellations when multiple files are missing:

* User wants to restore the files (e.g. a removed/renamed folder) -> click "keep them"
* User wants to discard them and either open them from their new location or completely work with different files -> click "remove them"
* User wants to restore some files -> decide what's more work: removing all now and manually reopen them or keeping them now, restore the missing files and click away the popups for the remaining files.

Only in the latter case the user will be confronted with the individual pop-ups, and I think then they are OK.
The important point is that the check is done *before* the active file is actually displayed.


we had this duplicate test as well


Maybe add new buddy flag = BUDDY_TEMPORARY_SUMMON? For control script summoned npc?


I am wondering if we should make `ExDoc.build_config/3`  and  `ExDoc.normalize/1` public?


Actually #247.


Thank you that you drew attention to that problem, fixed in 3be07547afb97e3de37027b7fe5e93ed416bba77


TeamCity Subsonic :: Debug [Build 37](http://109.103.226.38:8512/viewLog.html?buildId=34988&buildTypeId=Subsonic_Debug) is now running


Hello


Looks like bug. Should it allocate space for AnyExecutable? Because now it is allocate one byte with default bounds.
Also why not `calloc()`?


TeamCity Subsonic :: Debug [Build 37](http://109.103.226.38:8512/viewLog.html?buildId=34988&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:54


TeamCity Subsonic :: Debug [Build 38](http://109.103.226.38:8512/viewLog.html?buildId=34989&buildTypeId=Subsonic_Debug) is now running


3 files added.


TeamCity Subsonic :: Debug [Build 38](http://109.103.226.38:8512/viewLog.html?buildId=34989&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:48


TeamCity Subsonic :: Debug [Build 39](http://109.103.226.38:8512/viewLog.html?buildId=34990&buildTypeId=Subsonic_Debug) is now running


TeamCity Subsonic :: Debug [Build 39](http://109.103.226.38:8512/viewLog.html?buildId=34990&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:51


This commit fixes building on Linux (tested on an Ubuntu-based distro), but breaks it on Windows.  The solution will be to use a null error redirect on the custom build steps to avoid errors, and to have two calls; "bash" for Linux support and "call" for Windows support.  The cross-platform batch files work like a charm otherwise.

Additionally, I can't seem to get NuGet to work for Linux, despite having added the plugin for it.  So this commit also changes OpenTK to a binary in libs, rather than a NuGet package.  Going forward, once cross platform build support is operational, I'll look into a better dependency management solution for working across platforms.

Any testers should note that error messages in the Linux version due to missing runtime files are rather cryptic - be sure you are specifying your RTP path manually and correctly, if your RM project depends on it.


sorry to bug you again ;)
typo: provider

also: not sure about whether *W*orld or *w*orld


>The important point is that the check is done before the active file is actually displayed.

That can be arranged (I did test it before). If the active document isn't removed the second popup will immediately follow. That can be annoying. But again I think we have to consider that these cases presumably is rather rare.


class name of the bootstrap grid system was incorrect.
FIX class name "col-md-8" to "col-sm-8".


Thanks, I'll do that at some point... But there are some problems to solve first, and I think I'll have to get rid of my hack for the DIC.


I would very much welcome, if you could extract a o.e.s.automation.module.BaseModuleHandlerFactory from this, similarly to the BaseThingHandlerFactory. All code that can be reused by other factories should be moved there.
In theory, the SampleHandlerFactory should then also use it, but that won't work because of DS vs. ServiceTrackers. But I guess that the Sample bundles will anyhow go away once the rest is working.


For Cata push wait 3 month?


@ctrueden Was there a reason why the version was overridden?


TeamCity Subsonic :: Debug [Build 40](http://109.103.226.38:8512/viewLog.html?buildId=34992&buildTypeId=Subsonic_Debug) is now running


maybe you can improve a bit the performance by calculating minutes,hours etc. only when needed. for example:

```python
total_seconds = 111
if total_seconds < 60:
  pass
elif total_seconds < 3600:
  pass

total_minutes = total_seconds / 60
if total_minutes == 5:
 pass
```


TeamCity Subsonic :: Debug [Build 40](http://109.103.226.38:8512/viewLog.html?buildId=34992&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:50


TeamCity Subsonic :: Debug [Build 41](http://109.103.226.38:8512/viewLog.html?buildId=34993&buildTypeId=Subsonic_Debug) is now running


@Mayank009  dfsfds


TeamCity Subsonic :: Debug [Build 41](http://109.103.226.38:8512/viewLog.html?buildId=34993&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:51


Its not terribly laborious. You don't need to restart the app to stop the read-only feature. You do need to close the .py script, change the pref, and re-open the .py script. I am thinking that this feature is mainly intended to be used on testing-room computers, where things should stay read-only basically all the time. I don't think a menu item is needed for this, but happy to add one if you want it.


Actually, there's a reason I didn't make the default directory the color palette directory --- My plan with these `.clr` files is to add them to the git repo as a local file, then symlink to `~/Library/Colors`. I tried to look into programmatically making the symlinks but I got stuck :/

In any case, please remove the default location - I think it's a good idea, it's just not quite how I had intended to use this tool.


Ups


Better to use findDOMNode, as `getDOMNode` is deprecated since react 0.13.x.

Also, check out this one slider: https://github.com/mpowaga/react-slider


96: D102 Missing docstring in public method
99:5: F999 redefinition of unused 'visit' from line 95
100: D102 Missing docstring in public method
103:5: F999 redefinition of unused 'visit' from line 99
104: D102 Missing docstring in public method
107:5: F999 redefinition of unused 'visit' from line 103
108: D102 Missing docstring in public method
116:5: F999 redefinition of unused 'visit' from line 107
117: D102 Missing docstring in public method
120:5: F999 redefinition of unused 'visit' from line 116
121: D102 Missing docstring in public method
124:5: F999 redefinition of unused 'visit' from line 120
125: D102 Missing docstring in public method
128:5: F999 redefinition of unused 'visit' from line 124
129: D102 Missing docstring in public method
132:5: F999 redefinition of unused 'visit' from line 128
133: D102 Missing docstring in public method
136:5: F999 redefinition of unused 'visit' from line 132
137: D102 Missing docstring in public method
140:5: F999 redefinition of unused 'visit' from line 136
141: D102 Missing docstring in public method
144:5: F999 redefinition of unused 'visit' from line 140
145: D102 Missing docstring in public method


This repeated material can just go up into a shared grade - either in the harness grade itself or a separate utility. I suggest something of the form:

var component = gpii.tests.client.harness( etc.)

and then at the base of the file

module.exports = component.destructionPromise;

As well as eliminating repetition this will allow you to de-indent the whole file one level


I should write some tests for this.


1: D400 First line should end with a period (not 'I')
4: L102 copyright year is outdated, expected 2015 but got 2014
26:1: I100 Imports statements are in the wrong order. from tempfile should be before import pkg_resources
26:1: I201 Missing newline before sections or imports.
30:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser
31:1: I101 Imported names are in the wrong order. Should be CFG_ETCDIR, CFG_TMPSHAREDDIR
31:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.webpage
31:1: F401 'CFG_ETCDIR' imported but unused
32:1: I101 Imported names are in the wrong order. Should be extract_references_from_file_xml, extract_references_from_string_xml, extract_references_from_url_xml
33:36: E126 continuation line over-indented for hanging indent
37:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio_formatter.engine
41: D400 First line should end with a period (not 'n')
50: D400 First line should end with a period (not 'h')
53:12: E127 continuation line over-indented for visual indent
54:12: E127 continuation line over-indented for visual indent
57: D209 Multi-line docstring closing quotes should be on a separate line
57: D400 First line should end with a period (not 'g')
77: D400 First line should end with a period (not 'v')
86: D203 1 blank line required before class docstring (found 0)
86: D204 1 blank line required after class docstring (found 0)
86: D400 First line should end with a period (not 'I')
94: D400 First line should end with a period (not 'f')
103: D400 First line should end with a period (not 'l')
117: D400 First line should end with a period (not 't')
129: D203 1 blank line required before class docstring (found 0)
129: D204 1 blank line required after class docstring (found 0)
129: D400 First line should end with a period (not 'I')
132:9: E128 continuation line under-indented for visual indent
138: D400 First line should end with a period (not 's')
148: D401 First line should be in imperative mood ('Refrence', not 'Refrences')
148: D209 Multi-line docstring closing quotes should be on a separate line
148: D400 First line should end with a period (not 'e')
177:44: E128 continuation line under-indented for visual indent