I'm not 100% fluent in Elm. Can you help me understand this line?
Does it means that whenever the component updates with a NewGif action, it will issue a NewGif action as an effect into the context? So there are actually 2 types of NewGif actions in your app right?


FIND_PATH(LIBUV_INCLUDE_DIR NAMES uv.h)
FIND_LIBRARY(LIBUV_LIBRARIES NAMES uv libuv)

INCLUDE(FindPackageHandleStandardArgs)
FIND_PACKAGE_HANDLE_STANDARD_ARGS(LIBUV DEFAULT_MSG LIBUV_LIBRARIES LIBUV_INCLUDE_DIR)
Status API Training Shop Blog About Pricing


Chain currently is a mix of storage and logic for traversing the block
tree. I don't know whether the right solution is encapsulating it entirely.
There could be a API for building a block tree inside libconsensus, for
example.

That's not something I think we should decide now.


This could definitely become an issue. Currently most code expects a list like object returned but `RawQuerySet` doesn't support `len()` and `bool()` operations. Perhaps a better approach would be to wrap the `RawQuerySet` to add support for those.
https://code.djangoproject.com/ticket/22349


I checked and using `.replaceWith()` is definitely faster than using `.load()`
But `replaceWith()` only works on the first replace attempt. Subsequent attempts fail. Not sure why


Since those properties are static is it really necessary to have them in the schema?


Otherwise perhaps we can filter by tag/


Same here as previous comment


I agree. If you want to disable a test you can do so using the test navigator. Find the test in the test navigator, right click and select `disable <test name>`. This will ensure it is not run. Much better than commenting things out.


Same as previous


1 commit


Unfortunately this produces loads of notices about missing keys in the element array.
Probably because the defaults are not pre-loaded as in the beginning of form_builder.


Was the extra indenting on purpose?


Another Indenting that I do not understand


I think those init arrays are correct. The old ones (0x10, ... and 0x8a, ...) do not work.


I have no problems with this, it can be used to ensure the same output format no matter what the input looks like. :+1:


I wonder how the /blob/ URL can have worked for you?  See https://github.com/dimitri/el-get/pull/2338


Updated css file


`Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined index: #autocomplete_path form_process_autocomplete()...
Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined index: #parents form_process_radios()...
Warning: array_merge(): Argument #1 is not an array form_process_radios()...
Notice: Undefined index: #attributes form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: implode(): Invalid arguments passed form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: array_merge(): Argument #1 is not an array form_process_radios()...
Notice: Undefined index: #attributes form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: implode(): Invalid arguments passed form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: array_merge(): Argument #1 is not an array form_process_radios()...
Notice: Undefined index: #attributes form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: implode(): Invalid arguments passed form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: array_merge(): Argument #1 is not an array form_process_radios()...
Notice: Undefined index: #attributes form_process_radios()...
Notice: Undefined index: #parents form_process_radios()...
Warning: implode(): Invalid arguments passed form_process_radios()...
Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined variable: form_state _required_by_role_set_property()...
Notice: Undefined index: #autocomplete_path form_process_autocomplete()...`


Closes #95


Nice :+1:


Jtimon/libconsensus-p3 doesn't need chain.o. If we don't  need to decide now, please don't move it to the consensus package (and neither versionbits as long as it depends on it, like pow.o is not part of the package yet [see #7563]).


wtf just use OKTW


Fair enough, I can leave it outside of consensus.


here you overlooked something :-)


No way to do a real move in a inMemory implementation? Damn


@vilchik-elena It seems to me that "TypeTree" could refer both to a type name (or a type reference) or a type declaration (e.g. `ClassDeclarationTree`).


@Approvers: Better have a look at the commit history to follow the changes.


@groue

I'am surprise to see your comment. I just switch to swift from oc for nearly one month. Your GRDB do a great job for sql database operation.

And thanks for your suggestion.

In this case `.select(PointOfInterest.sortItmes.title)`  is not my intent.

you are right to use `Int.fetchOne(db, PointOfInterest.select(count(PointOfInterest.sortItmes.title))!` to do my count-tilte job.

Thanks again.


don't we have other cabal dependencies?


Hi, sorry, I created the pull request by accident. The patch is basically working for me but I haven’t had to the time to do a regression test yet. Will check that asap and create a new pull request. Cheers, Jörn

> On 03 Mar 2016, at 10:49, Martin Kiesel <notifications@github.com> wrote:
>
> Hello there, is this working as expected? Can you write some documentation as well? Is blade (extending) limited by this change. Thanks
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/joernh/column-sortable/commit/ee19e936e024817f57f461032627e006c877b824#commitcomment-16468502>.
>


Can't seem to find what exactly has changed with this commit and why.
Could you tell me?


I guess you could just use 'gi' with regex.


I think a warning should be added that the force push must never be used on master and it's ok here because you push to your own branch on the fork to resync the PR


Same question as previous commit. What exactly was changed and why?


I guess the date of the document would read 2016.2.24?


:thumbsup:


Wow ... that's horrible ... let me have a look to the code and I'll fix it asap (anyway PR's are welcome :D )


@sils1297 You mean haskell lint? Previously haskell lint was being installed using cabal but now it's being installed using debian package manager.


For some reason git is still seeing this as a binary file... @peter-b: Any thoughts?


Ensure package is not zipped etc? What exactly was changed  in transcrypt.bat and how does that ensure that package isn't zipped?


Why is such a problem to have this in versionbits? Maybe it makes sense to expose it in libconsensus (pretty cheap to have it there, anyway.
Could the cache be a C API friendly struct instead of a c++ class as in my implementation?
Why re-re-re-use cs_main instead of having a dedicated critical section for the versionbits cache?


Actually I think we could avoid having this function. The input could just `setState({searchTerm})` and the filtering could happen in `render`. But let's wait to see what @sapegin says.


As this is priority & never fails other than unversioned.LabelSelectorAsSelector func failure which uses input podAffinityTerm, so if one element is wrong other elements from an array may be correct , so we may get low priority value if we return on failure.so i think just positive validation is fine. what do you say?


@senthilec566  We have removed dataSource field now, please pull the latest code. Thanks


Because every caller already uses cs_main.


Converted Trancrypt to transcrypt. Why is that better in your opinion?


Can we encapsulate under a more generic and to-be-completed getflags?


Môžete mi dať podmienku do oboch súborov, aby dávalo od roku 2016 a vyššie.


> some options shows label name instead of what stored in db(like with scope it should show only Restricted in mail but it shows full represent value Restricted - to users with a known operational requirement (described in restriction). How can I resolve this?

Configure the field as a represent = None in the s3_rest_controller when format=msg


why do you need libclang for coala-quickstart? Why do you ned any of those anyway? You need coala!


ah right.


Is your intention here to skip generating a crash log?


Django etc. are in the metadata because I think Djangists would like to use Python in the browser.


Can't just take the first one,  I think I made this comment before but I can't find it in github or mail :(

Might need to make some special case for um (user mode) vs km (kernal mode?)

In my win 8.1 sdk   winv6.3 contains um (and km), there are also win8, win7 and wdf  which only contain km
In my win 8.0 sdk  win8 contains um  and I don't have any other installed

if building a kernal mode driver maybe you need to match the specific target os which needs to be specified.

for user mode I found search for the um folder rather than taking the first was sufficient to solve for me
      for( final File libSubDir : libDir.listFiles()){
        final File um = new File(libSubDir,"um");
        if( um.exists())
          libDir = libSubDir;
      }

with this change I'm able to build against Win 8.1 sdk  and Win 10 sdk


No log.


I prefer using array functions. What do you think about
~~~
$prefixes = array_values(
    array_filter(
        array_map('trim', $prefixes)
    )
);
~~~


It is flow error.
California failed to revert.
I'll revert for now but I'll try to find a quick fix.


maybe instead of returning an array, each provider should return a well defined ValueObject?


There can be a more generic wrapper around this elsewhere later for sure.
But this is code in the versionbits module. It shouldn't do more than
versionbits.


If we keep `handleChange` function, maybe let's move it here, inline, as arrow function? I'd like to avoid the `bind`.


Wouldn't simply have EXPECT_ASSERT and ACTUAL_ASSERT do ?


What exactly was changed?


Why do you need the try/catch ? Shouldn't the `order` always contained the subfield `2` (based on the values from `field_map`) ?


better to remove full  Spec: api.PodSpec{
 					NodeName: "machine1",
 			},
instead of just NodeName: "machine1". what do you say?


"find" is a verb. You're styling the search input.


A small note, Instead of commit with
`improve blame syntax`
commit with
```
Improve: Blame syntax
^        ^ Capital letter
       ^ have a :
```

It will make the generation of change log easier


What's a PR?


spaceId is no longer used below; unless there is a hidden dependency upon creating it, which ought to be captured somehow.


I added these methods so that we do not break encapsulation:
- state is not accessible from data_logging
- data_logging does not have to know mavlink (that param1==1 means start and param1==0 means stop).


Well... Not withou re-designing in memory mappers.
UID provider and MODseq provider I need for this operation can not be accessed directely. It is only available in the AbstactMessageMapper.
The best I can do is avoid a useless MailboxMessage.copy (see fixup) but not the call to MessageMapper::copy
Note that the call to MessageMapper::copy is very close to MessageMapper::add


hey, nice job
I think your length of codes should be more than 80 characters .


This change makes the old dynamic collection tests pass again


Wanted to write the same ;-)


AFAIK we have all our font sizes in `rem`... Let's keep it consistent.


dddddd


I think what we need here is something understandable, that works for test.
Not a killer design, optimal implementation. If you want so, it will require far more work...


.base from colors.css.


Now the variable memory_usage is used before it is initialized.


.base-bg from colors.css.


No, I mean instructions with 32-bit operands in general. From the Intel manual:

> When in 64-bit mode, operand size determines the number of valid bits in the destination general-purpose register: [...] 32-bit operands generate a 32-bit result, zero-extended to a 64-bit result in the destination general-purpose register.

Reading http://www.felixcloutier.com/x86/MOVZX.html, maybe the practical difference in these encodings is that (due the the REX prefix) `movzx r64 r/m8` cannot access sources AH, BH, CH, DH, but can use R8-R15 dests?

I'm an asm newbie, so I might be way off-base here. Feel free to ignore :)


Cool yes Uili's one is nicer!


Seeing this again, I'm wondering about the name. `RoutingServiceType` is the interface and `RoutingService` is the implementation. I think it's the word 'type' in this context that seems unclear.

Would it make more sense to call the protocol `RoutingService`, and the concrete implementation something more specific, like `OrdnanceSurveyRoutingService`?

An end-user would only interface with the RoutingService protocol, so the implementation type name doesn't matter much to the outside.


Fixes (across the board) available in https://github.com/guozhangwang/examples/pull/8


Hmm typo or too many version numbers in my head.


I forgot to mention, I am not using the whole dev version, I just modified this bit....should there be some magic elsewhere in the code that prevents these notices.


> _vague_
;)


Sorry, I don't know that.


create new theme for mawusar


hi，Fykec，你提交了重复的内容（gotgithub已经收录），以及react-native-gitfeed超链接错误，我刚刚修复了一下


> tried this <a href> but doesnot show in email.

settings.msg.notify_email_format == "html"

You will also need to rename the file as:
notify_email_html.html


@scott1702 - doesn't this remove the logout-icon in 3.3.x. I seams the logout-icon is in the silverstripe-font ('z') but I think the style for it is not there yet. Atm I also just wonder, what happened to the button.sticky-toggle to make the cms-menu sticky?


I this required for something?


for Mac, it is -lfftw3f-3 I guess.
`if os.name == 'nt': extra_link_args = ['-lfftw3f-3'] else: extra_link_args = ['-lfftw3f']`


Why should in your view the license be in the package?
Please don't change overall use of whitespace and () if you intend this to be merged with the master branch.

You state:

+# Inject our custom heirarchy before other paths.
 +# Now loaded dynamically from wherever we happen to be installed.
 +# Import time side-effects are bad, but not sure how to avoid this.

Is this meant to be merged with the master branch or is it just a private fork for use in specific (custom) circumstances?


Hi, please can anyone make update on https://packagist.org/packages/prestashop/prestashop-webservice-lib to the last commit?


The fixup is better.
No need to fix this, that was just to know.

In fact all our MOVE implementations are using copy then delete. What would you think of a default implementation for every backend? Is it conceivable?


This can be simplified a bit:
1. Extract `e.target.value` to  a variable.
2. Use arrow function instead of `bind`.
3. I agree that `searchFilter` can be inlined here: it’ll be simple enough after `toLowerCase ` removal.


Close [wordig message for the top](https://quire.io/w/ZVhx9ntJYYaHOCxTJXantbF9/wordig_message_for_t...): Test10 quire github
via [Quire](https://quire.io/u/quire)


@batikanu @f-higashi @floreks @maciaszczykm It would be great if someone of you guys can do a small review :) thanks


What do you mean @nbarrett ?


delete this lol


Well without IntellIJ then I am nothing @oliyh!


Close [wordig message for the top](https://quire.io/w/ZVhx9ntJYYaHOCxTJXantbF9/wordig_message_for_t...): Test11 Quire github
via [Quire](https://quire.io/u/quire)


Taking inspiration from Swift stdlib really. Things like `ErrorType`, `SequenceType`, `GeneratorType` etc. Bearing in mind I've only really created the protocol for mocking purposes, forcing people to type `OrdnanceSurveyRoutingService` feels a bit mean.


nnnnn


simlink feels more natural here.


`bb b`


Looks clever, could you explain in some detail how this works? And does it work both for Windows and Linux?


nice


YES! it's mandatory now, see the PR description


I've actually fixed the clear private data tests. You can find a PR here https://github.com/mozilla/firefox-ios/pull/1596


Oops. I'm finishing something with PPP and fix this.


https://swift.org/documentation/api-design-guidelines/#general-conventions. Maybe something like `Routable`? Occurs to me I don't think I actually need the `init` method in the protocol. Will experiment with that.


This script doesn't restart Cassandra anymore. Is the argument here that if the cassandra-env is set up incorrectly Cassandra won't start and monit will retry for us? If so could you comment to that effect?


This used to have `text-overflow: ellipsis`. Does it need that still?


Why did you decide to do it this way rather than amending the fastlane script with whatever you needed?


For Cassandra : doing so would require not using reference to the mailbox in the primary key. This is a major redisign, and changes every thing. I does not worth it IMO.

For im memory, it requires a direct implementation of MessageMapper, without AstractMessageMapper. It is not hard, but takes time.

Finally, note that reading the message partially is required (at least metadata) for sending expung notifications.


Any news about merging this patch? It seems to be needed by Libressl too.


Not sure what you mean?

My example works fine as is.

In @jwgrenning's example, we'd then have

ACTUAL_ASSERT == EXIT_TEST
EXPECTED_ASSERT == FAIL_TEST("An expected assert didn't trigger....")

so I don't see much of an advantage there. I think EXIT_TEST is more straightforward. I do believe that we need to put his example in the documentation so it is clear how this is meant to be used.


good job!


This should not be pushed to github :)


great job bro !


Looks good now, ack.


This is an implementation of a "heapster sink", which is a component that just takes heapster metrics and stores them into some target (e.g. other sinks are influxdb, gcm, hawkular, etc.)

On this note: some useful links to get an idea of what's happening here.
* https://github.com/kubernetes/heapster/blob/master/docs/sink-configuration.md <--- how sinks are configured on heapster start-up
*  https://github.com/openstack/monasca-api/blob/master/docs/monasca-api-spec.md <--- the monasca API spec (the one I'm talking to)
* https://github.com/stretchr/testify <--- library used for testing


Thanks for sorting this! Just saved me a lot of pain i guess :)


It would be a lot more descriptive. You would have EXCEPT_ASSERT first (like a mock) and have ACTUAL_ASSERT in the assert mock implementation


Lel xD


If you could help me, I will appreciate. As `portHandle` for serial port can participate in `select()` operation, is it possible to implement Java `java.nio.channels.SelectableChannel` interface based on existing JNI API? What I want to achieve is the following:
```
import java.nio.ByteBuffer;
import java.nio.channels.Selector;
import java.nio.channels.ReadableByteChannel;
import java.nio.channels.SelectionKey;
import java.nio.channels.Selector;
import jssc.SerialPort;

SerialPort port = new SerialPort("COM1");
Selector selector = Selector.open();

port.configureBlocking(false);
port.register(selector, SelectionKey.OP_READ);

// poll cycle:
for (;;) {
  if (selector.select() == 0) {
    continue;
  }
  Set<SelectionKey> selectedKeys = selector.selectedKeys();
  Iterator<SelectionKey> iterator = selectedKeys.iterator();
  while (iterator.hasNext()) {
    SelectionKey key = iterator.next();
    if (key.isReadable()) {
      ByteBuffer readBuffer = ByteBuffer.allocate(BUFFER_SIZE);
      ((ReadableByteChannel) key.channel()).read(buffer);
      ...
    }
    iterator.remove();
}
```
Thanks.


why would u delete the kid?


Goodbye cruel world


Still you are right that we should implement the interface. Will change this.


Now that's a good commit message ;)


myo pls


Thank you very much for digging up that reference; I looked for it but couldn't find it. For the record, it's in section 3.4.1.1 (General-Purpose Registers in 64-Bit Mode). You have educated me—I thought it worked like 8- and 16-bit destinations:

> 8-bit and 16-bit operands generate an 8-bit or 16-bit result. The upper 56 bits or 48 bits (respectively) of the
destination general-purpose register are not modified by the operation. If the result of an 8-bit or 16-bit
operation is intended for 64-bit address calculation, explicitly sign-extend the register to the full 64-bits.

Weird that it's different for 32-bit destinations. I think then that this code is correct, but it might be worth using `MOVBQZX` just to avoid the question altogether.


Yes, that's more readable to me, if you really want to use it I don't mind. I'm pretty sure this ```checkMapMaybe``` could also be written in a better/more general way, but this was enough for me.


aaa


more love


Change it to $scope.refItems and pass the item to the already existing "end-fetch" event.
This code does not handle empty arrays.


See Readme.md / "Releasing a new version" for updating the other files


I don't think so, the Date/Time doesn't overflow, and I've set the closed stocks to do that


В редких случаях вылетает именно эта ошибка при отмене сетевых операций, я ловил на этом проблемы на проектах (последний раз летом)


Is working with my 30.3200 Dostman as well. In order to detect such conditions i have added a CRC flag to the configuration.
I was pretty sure that Bit 35 is a low battery indicator (at least that bit got reliably set when i replaced the batteries with two AAA batteries and nominal voltage of 0,9V each), need to get back to the lab for testing that one out.


beide Dateien bringen
```Parse error: syntax error, unexpected end of file in C:\...\system\modules\metamodels\templates\metamodel_unrendered.text on line 22```

Windows, PHP 5.6

auch ein schließendes ```?>``` ändert nichts - BOM falsch?


Remove it. Its lag. :/


nice


Why not create some test and send PR?


в последнем AFNetworking нет вообще символа NSUserCancelledError
скорее всего ты это было на проекте с RestKit (гори он в аду)


我要报警了！


Shouldn't you check explicitly if the item on  tile is a splash?
In the current state it lets you open the pickhole even if there are items on that tile, doesn't it?


You can also write those 3 lines like so:

`validates :body, :user, :product, presence: true`


Please can you add some comments to these files explaining how they work. Things that I would like to know are:

* What do these files do and how do they do it (in particular the rules for how the displace file works and how it derives the suffix seem really hokey to me)?
* When they are run.

Also I have checked and clearwater-cassandra's postinst does not seem to stop cassandra - it probably should given clearwater-cassandra is modifying its config.


@sacdallago yes i noticed that, too and added it


Close [wordig message for the top](https://quire.io/w/ZVhx9ntJYYaHOCxTJXantbF9/wordig_message_for_t...): Test12
via [Quire](https://quire.io/u/quire)


            好的。
            On 03/03/2016 18:38, coderyi wrote:hi，Fykec，你提交了重复的内容（gotgithub已经收录），以及react-native-gitfeed超链接错误，我刚刚修复了一下

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


Yeah `Routable` probably makes more sense. `...Type` strikes me as a data type.


@dkayiwa No assert has been removed. Git got confused with the changes and showing them differently.


typo: structrure


Hello, there was no PR I just saw new fork and checked it out :)

If you do some tests (manual :D), feel free to make a PR.

Martin Kiesel

Sent from iPhone

> On Mar 3, 2016, at 11:24 AM, Joern Hartmann <notifications@github.com> wrote:
>
> Hi, sorry, I created the pull request by accident. The patch is basically working for me but I haven’t had to the time to do a regression test yet. Will check that asap and create a new pull request. Cheers, Jörn
>
> > On 03 Mar 2016, at 10:49, Martin Kiesel <notifications@github.com> wrote:
> >
> > Hello there, is this working as expected? Can you write some documentation as well? Is blade (extending) limited by this change. Thanks
> >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/joernh/column-sortable/commit/ee19e936e024817f57f461032627e006c877b824#commitcomment-16468502>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


Nice commit name :+1:


Better to use iris.log where necessary. console.log shouldn't be in code.


    +#!/opt/bin/python2.7
    +# -*- coding: utf-8 -*-

Is it needed? )


how to download and import external/bho


see above


I just moved the plugin to the top.


Reason was I thought xfce would use dconf for gtk3 apps, and placeholder for xfce going gtk3, whenever that is.


why?


Here's how it might look to be more accurate:

![peg_diagram](https://cloud.githubusercontent.com/assets/453297/13492356/39b5a6d4-e140-11e5-9b09-aeacf71467ae.png)


Eugenio ha fatto qualcosa


I could rename it to base.


How can I test after replacing preg_replace_callback?


processesed - > processed


can we redefine abort() and std::abort() as static_assert("Never call this function, use ASSERT(\"Reason\") instead") ?


yes https://github.com/zdubzdab/schoolTest/blob/ae297005abb4d5e756e1a691c035a0c8282794c8/app/controllers/teachers/test_settings_controller.rb#L29
https://github.com/zdubzdab/schoolTest/blob/ae297005abb4d5e756e1a691c035a0c8282794c8/app/controllers/students/tests_controller.rb#L22
I shouldn't use them?


@davidstrauss mentioned that we might want to drop the term "SSL" in favor of *"TLS"* in our public docs. He might want to chime in here.


Typo: objecsts


I assume you mean if protocol_name == "udp" here?


typo: indentifier


a ono to chodí sringově z flexibee? :D


this is missing


Do we feedback fixes to the forked projects?


you only want to escape the start path not the rest!


typo: fllowing


typo: requenst


il


typo: Sucessfully


Thanks, it works now!


this is .lock .unlock, not CSingleLock ()

and goes away anyhow in next PR


First test merge


Right, will do.


I think it should be "like any other". "As" can only be used in a sentence like "I'll be just as good as anyone else".
Not trying to be difficult here ^^


I know what it is, but if used as a SingleLock you could avoid calling unlock, as the scope would do it for you by calling the dtor.

Okay - if the next PR is a bigger refactor, perhaps add this one as commit, then we perhaps get the overall picture.


OK. I read the tutorial on this as saying `ezconf` should be used, will see if that can be improved too.


No, you should be able to open the pickhole even though there are items on top of it.


Let's standardize $full_name / $config_ids to $config_name / $config_names, that matches how core calls it internally.


I think your length of codes shouldn't be more than 80 characters


The entity type manager.


ja


Unit test for infoproviders: Sure, once we're clear on what an infoprovider should and shouldn't do.


It feels weird for this class to statically cache all config entity type definitions, especially since other methods just want a definition by id, which they can get directly from the entityTypeManager.


Nice job, :-)


jo jo :D


req.getParameter("adminact") - лучше вынести в отдельную переменную


How about adding ClusterConfig object to the constructor instead of entire MesosCluster object?


When it is assignment in `if` statement, it is more readable to have parenthesis. Do you agree?


This smells like a protected method, not something that should be on the interface.


этот блок try catch - лучше вынести в отдельный метод


Well, the main purpose is to produce a json-encoded array, so a ValueObject is a detour. But might be good for template rendering.


I think it's still safe to invert `!(a==b)` to `(a!=b)` here!


Doesn't feel like a part of a public API. Feels more like a protected method. Not something that should be on the interface.


for equality checks on strings and all that


Feel free to do that.


It makes sense to me


韩慕麟    I am new to here


lgtm :+1:


Could you point me to the place where fonts are in `rem` all the ones I can find are `px`

thankyou


Looks great.


It feels really weird that core doesn't give us a one-line way of importing / deleting config.


Close [dddddd fdsm](https://quire.io/w/TfKbcQ0waVwPaXkmVNKT0PAc/dddddd_fdsm): Test13
via [Quire](http://1722f982.ngrok.io/u/quire)


Hi!
You are missing terminating " character :)


##  스트래티지 패턴 정의 - Strategy pattern

> 알고리즘군을 정의하고 각각을 캡슐화하여 교환해서 사용할 수 있도록 만든다.
> 스트래티지를 활용하면 알고리즘을 사용하는 클라이언트와는 독립적으로 알고리즘을 변경할 수 있다.
>

## set함수를 이용
이번 커밋에서는 MiniDuckSimulator.java 의  main에서 set함수를 이용하여
오리의 날기 행동을 바꿨다.

`model.setFlyBehavior(new FlyRocketPowered()); 	`

이것에 초점맞추어 기억하기!


Cool, agree with 2nd point. Will replace 'variables' with context

Re. 1st point. What I mean is that the context-variables can itself contain expressions so it feels `_expression` suffix is justified. See example [here](https://github.com/dimagi/commcare-hq/pull/10586/files#diff-0ede03ea138027f30e000f57b582d5deR749)


Add a TODO entry marking it for removal once we don't support <1.0 libtorrents anymore.


I'll let Jeff comment further, but, my initial feedback, is that I don't think we should store the smbmount_present in the properties.  The reason being is that if you move to a new linux system, etc, then this value could be set correctly, and and would never be retested.  Personally, I would track it as a static variable in IOUtils or Sage.  ie, something like...

`
if (IOUtils.HAS_SMBMOUNT==null) {
    // check for SMBMOUNT
    ....
    IOUtils.HAS_SMBMOUNT=Boolean.TRUE;
}
...
`

There are a few ways to do this, ie, using a Boolean object as a trinary (null/true/false) or using an int/enum with defines states (SMBMOUNT_UNKNOWN,  SMBMOUNT_YES, SMBMOUNT_NO).

Since this is Jeff's repo, I'll let him decide if he's OK with using the properties vs using a static variable that's evaulated once per server instance.


Small typo: "inserted" is double


**Code Quality: Awesome**

Great use of doc style comments to explain exactly what your code does.


This method returns the entity type id, not the entity type object.

getConfigEntityTypeId()?


PR: Pull Request

Not sure what you mean with

> I just modified this bit....should there be some magic elsewhere in the code that prevents these notices.

could you explain a bit more?


You’re right, they are in pixels.


so I was proposing the name `context_variables`. the value would be a `DictProperty` of template keys to expressions. You could name it `context_variable_expressions` though I don't think that's necessary (and possibly slightly misleading) since it is not a list of expressions, but its own data structure


I know it is ruby... ;)


Close [dddddd fdsm](https://quire.io/w/TfKbcQ0waVwPaXkmVNKT0PAc/dddddd_fdsm): Test14
via [Quire](http://1722f982.ngrok.io/u/quire)


Follow the build and readme instructions.

You must use:

git submodules --init --recursive

Otherwise, see our new gradle/Android studio version at github.com/zom

On Thu, Mar 3, 2016, at 06:01 AM, Ali Aqdas Shah wrote:
> how to download and import external/bho
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/guardianproject/ChatSecureAndroid/commit/a7c1b13a1456cd699ef54705492392075d8c98d0#commitcomment-16470091


OK, ported to elexis-server!


You ended up with `ignoresysdns`, right?
I'm not sure it helped for reloads, I see at least 3 here with `forcerestart` (and I guess it's the same with handover).


hanks for comment. Blog explains everything. Hint for [your implementation](https://github.com/sanzinger/java-simple-serial-connector/blob/dd30c301ef5fc0d061044a76f613e16466340309/src/cpp/_nix_based/setCustomBaudrate.cpp#L32): you could use `termios2` structure on the stack (`termios2 info;`), not on the heap. In this case there are less changes one forgets to free the memory.


I have to agree, we should try to keep backward compatibility where it doesn't cost us anything, and leaving the old methods as overloads doesn't seem to hurt.


You should mention the benefit and outcome in GIT commit messages!


Use the consistent coding style. Please check `README.md` carefully.


Do you want me to match it to the TOC list below at 15px?


I know the getFromActive() / getFromExtension() naming comes from config_update, but loadFromActive() / loadFromExtension() probably makes more sense?


Don't put the generated data in GIT!


`selectEntity` fails mainly when the item is not in the collection.  here the item is guaranteed to be in the collection.

btw i'm aware the code at this point is a complete mess but i wanted to do the minimum to make it work with plan to replace with angular rather than clean it up.


Write down meaningful GIT commit messages.


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


**Code Quality: Nitpick**

According to the loose coupling principle of OOP, we shouldn't embed this sort of functionality within this class and/or if we wanted to do this, a better way would have to assume that the file has already been read.

```python
def sort(self, role='fellows'):
    if len(self.data) == 0:
        return False # File has not been read

    # Some sorting algorithm based on role
```
This way we have not only made our code much more elegant and less cluttered, we've also cut down the need to have redundant functions for sorting staff and sorting fellows seeing as the only difference is the comparison that happens on line 44.


**Code Quality: Nitpick**

This code is repetitive and not really required. We can rewrite these two functions as a single function. See above comments.


I understand the pain. I am going around in circles too trying to sort out a reliable build process.

I occasionally after a clean build, see what looks to be a similar problem where I get 10 warnings and an error for one of the Tasks dll. (sorry deleted the log file with the specific dll name to confirm)

A workaround appears to be to set FLAVOR_WIN=Release, then run the build_solution.cmd.
The error promptly stopped, even though prior to the set, it remained for each run of the build_solution.cmd. As the error state is not 100% reproducible, I can prove this is a fix.

If FLAVOR_WIN is not defined prior to build_sdk.cmd in a clean cmd process, the following Environment Variables have invalid values with double \\\\ characters:  BUILD_TREE_SERVER, BHL_EXE, DEVPATH, MDP_EXE, and PATH which might be causing trouble if they are used ahead of the MSBuild properties with the same names which have the correct values.

An example invalid variable from my dev environment
```
BUILD_TREE_SERVER=C:\Projects\BPplus\netmf\netmf-interpreter\BuildOutput\Public\\Server
```

I am playing with the changes I proposed where BuildEnv.props has a task to export a selection of its properties to a .cmd file for execution in the global context and define them outside MSBuild. This avoids the double slash errors, and the variables and properties are defined to equivalent values as the are define in one place with one set of rules inside BuildEnv.props.


Did you code this with your GA code? I'm not seeing anything in my reports


Wouldn't it be better to use the query builder just to make sure you also get installations where the table prefix isn't GDN?
`Gdn::sql()->update('Discussion', array('Prefix' => null), array('Prefix' => "));


A downside of this approach is we lose the information of what selector we used. Is there a way to get the selector string into the Region by another means?


We should make `JavadocAssertionTestBuilder` understand `@` and `{@code ...}` for that matter. Then most of these tests could be made executable via the normal `groovyTestCase` mechanism. I'll create a separate issue for that.


Может быть лучше Ansible Playbook'ом?


No particular reason. When I started writing the code I was not sure if I need to do something to the promise before returning it. So I stored it to a variable.

Now that the code and related tests are done, I should probably refactor the code and lose the unnecessary variable and return the Promise directly when it's created.


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


The bug and the dirty fix I copied are already in their tracker
https://github.com/google/google-api-php-client/issues/169
...since 2014. Now v2 is on its way, I doubt they care.


Looks good!


Also modified the pom to comment out the gpg plugin and changed references to distribution management.


Hi ebay
Script now belongs to community. Feel free to do any changes you think are necesarry. I made it while learning sqf, nowdays woul most likely rewrite the whole thing if i had the time...which i dont unfortunately.
Cheers

-----Original Message-----
From: "ebaydayz" <notifications@github.com>
Sent: 03/03/2016 03:17
To: "EpochModTeam/DayZ-Epoch" <DayZ-Epoch@noreply.github.com>
Cc: "Raimonds Virtoss" <raimonds.zigurs@gmail.com>
Subject: [DayZ-Epoch] Fix snap build errors (585841a)

Fix snap build errors

@raymix in most of your helper functions you are using exitWith {}; in
the same way the main player_build uses it. The problem is exitWith
inside of a called function only exits that function, not the parent
script it was called from.

When a helper function exits the main script continues on with undefined
variables and other problems. The easiest solution was to just remove
these helper functions for now. I just directly substituted their code
back into the main modular_build file and removed the passes to and from
them.

We should consolidate player_build and modular_build into one file first
since they are about 80% identical. After that we need to integrate
plot4life. Dividing player_build up into smaller sub functions like this
can be left for last.

I tested this thoroughly and can confirm the undefined errors are fixed
now.

---
Reply to this email directly or view it on GitHub:
https://github.com/EpochModTeam/DayZ-Epoch/commit/585841a3ad290033163bce27a89800cafadb6075


Test


In order to install an updated version of a plugin, the correct procedure would be to disable it, copy the new version and enable that, correct?
If it is done that way, the best place for the null/'' fixing would be setup(). By now the fixing is checked whenever the plugin is instantiated, which influences performance.
Although it might be a "hard" decision to force forum admins to disable and re-enable the plugin, I think it would be the cleanest solution.


Fix order: `public abstract class`
BTW extract interface: Identifiable with `UUID getId()` method - IdentifiableEntity should implement it.  Identifiable we will use whenever we can.


OK, I will review that in my scripts and others.


Initial file


So, you think source code comments should be in German instead of English?


unclear - did you by hand change the name of each replicate file? Will it not be easier to use the script from the previous missing data script to read the folder and write.table with the new file name


Super Epic man, having goosebumps!!! :D :D :D :dancer:


I mean for that JSON response. Because I am not sure that it is used somewhere in JavaScript function.


This is the study repository.


you could use `pathlib` to be even fancier


It will not override player's information if no user entity exists.


I am aware of this, however I was seeing cases where someone would join, immediately change server, and then have their health etc set to the default database values.


Is the Else-Case really as it should be? If you running nuget on a buildserver and all concurrent builds uses the same user, then a build can delete temporary files from another build.


已改成直接用json字符串


返回是一串，没想到测试用例怎么写，，


Update this flag so that the include guard does not get confused between the cut and non-cut headers.


نسخه ۲-بتا-رفع اشکال ۴


```c++
auto const polylines{_geoObjects->getPolylineVec(listName)};
```
or
```c++
auto const polylines =_geoObjects->getPolylineVec(listName);
```
would be even more c++ :wink:


just change to 1.6.1.4 and use, it works


My idea is just return the error when err != nil, just for simplicity. Otherwise the code will have too much implied things.


Cool, done. (PR updated)


ok i agree


One question... the release/10 branch of OpenVPN-GUI will be bundled with OpenVPN 2.3.x, which will not have the Interactive service. Do we need this patch for other reasons? If yes, will the reference to the named pipe cause issues?


Can this be derived from InversePowerStillinger_interaction to save some duplication, or would that be too slow?


Yes, removing the full Spec looks good for me.


I'd keep the HTML to a minimum simply to reduce the amount of code here


What is this required for?


This feels like a hack


Why not simply `"%s=1" % IS_POPUP_VAR`? This has a 100% no-bug guarantee and if needed we can account for the general case later


Thanks for this! It was realy annoying having to tap twice on slow mobile connection.


`mesh_dim` could propably `const`.


no I don't think so, the potentials are completely different

On Thu, Mar 3, 2016 at 11:46 AM, Julian Schrenk <notifications@github.com>
wrote:

> Can this be derived from InversePowerStillinger_interaction to save some
> duplication, or would that be too slow?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smcantab/pele/commit/08f5485776b81e77300f8ac2424c58478484b08c#commitcomment-16470930>
> .
>


It looks much tidier with the new paths, doesn't it?


Double.valueOf(this.getBalance()).toString() - крайне странная и излишняя конструкция.
1. Примитивы могут в автобоксинг (автоматическое приведение к class варианту). Так что double -> Double приведётся автоматически.
2. Конкатенация (сложение) строк умеет вызывать toString по умолчанию.

Т.е. вместо
+ Double.valueOf(this.getBalance()).toString();
можно просто написать
getBalance()


Is `sed` the reason that you don't collapse this line with the next one?


Why no Python 3.5?


hgj


Good job man! I love the approach. I have integrated this in my last release Puigcerber/angular-country-picker@a33031a0eedf32d042207dee5281f73bc235ea2c


AKA `distutils.dir_util.mkpath`


nice


Ok, after the second thought, try/except makes more sense here.


In the current form this method feels like a protected helper, not a public method.

compareConfigHash() as a name describes an implementation detail, not what the purpose of it is, which is checking if the config object with the given name has been modified by the user. So maybe something like isModified()?
D7 used the word "overriden", but D8 has the concept of config overrides, so that would conflict.


The power functions are the same.


Let's have a $skip_modified second param that defaults to TRUE?


No newline at the end of file.


Возможно, удобнее будет использовать for (JSONObject subscriber : slistObj)


So is it ok for the next release to be `3.3.0`?


this is first commit


* It turns out that all the crpix needs to be obtained by (double)naxes / 2.0 + 1.0.
* The line #1922 might need to be broken into two lines?


Yes some other guys told me this issue but still can not track this. :-/


А что будет, если в объекте будет отсутствовать поле "s_surname" ?


@labrys


Public API всегда должен проверять входные данные, иначе код может упасть (в данном случае, на 9 строке)


It feels like we should not have process() at all. Instead, the update hook should call import/revert/delete with arrays of ids. That makes the update hooks more explicit, cause it tells me what the intention was (otherwise if I go back and read an old update hook I'll have no clue what was new and what was removed).


no pls change it to Jeff ok? :P


It no longer occurs with this change


Should we just cut all this commented out stuff?


That's what I get for hacking in the early hours of the morning.


yes that's true, we could write a base class that implements only the power
functions. It could be a base class for all power potentials

On Thu, Mar 3, 2016 at 11:51 AM, Julian Schrenk <notifications@github.com>
wrote:

> The power functions are the same.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smcantab/pele/commit/08f5485776b81e77300f8ac2424c58478484b08c#commitcomment-16471011>
> .
>


https://goo.gl/Kd9lHM https://goo.gl/YVcuhS http://goo.gl/BwXo69 https://goo.gl/95M5Vr goo.gl/626U3.info https://goo.gl/6k4ZDQ https://goo.gl/6hkE3I
https://goo.gl/e53Kbh http://goo.gl/f3VQHM


if I put: url = '#{$(".test-form").attr("action")}/finish_test' rails will redirect me into http://localhost:3000/ua/students/test_settings/194/tests/new?#{$(".test-form").attr("action")}/finish_test


This doesn't sound like the best workflow ever, we should try and fix this.


Could you explain why a case statement is favourable to the BT instruction? I've never read that the latter is slower than the former - perhaps you can point to an explanation/research for this?


![simulator screen shot mar 3 2016 5 15 12 pm](https://cloud.githubusercontent.com/assets/17614692/13493690/865f6cac-e165-11e5-95a4-c285d7e809d1.png)


As both the phase banner and tag are now using `display:block`, the `vertical-align: top` here and on line 57 can be removed.


We should add docs to the interface, making sure to mention that the config updater is meant to be called from hook_post_update_NAME() and not hook_update_N(), since it updates entities.


Preperation steps for #14


We need to replace the logger with the Result object that has getProcessed(), getSkipped().


maybe we do this in other PR? since Ed needs a working implementation in
the meantime? inversepower has also its own implementations of power
template metaprograms, so writing a base class for the power potentials
would be a good idea to clean things up (or even make those functions
members directly of SimplePairwisePotential since taking powers is so
common for this kind of potentials)

On Thu, Mar 3, 2016 at 11:56 AM, Stefano Martiniani <
stefano.martiniani@gmail.com> wrote:

> yes that's true, we could write a base class that implements only the
> power functions. It could be a base class for all power potentials
>
> On Thu, Mar 3, 2016 at 11:51 AM, Julian Schrenk <notifications@github.com>
> wrote:
>
>> The power functions are the same.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/smcantab/pele/commit/08f5485776b81e77300f8ac2424c58478484b08c#commitcomment-16471011>
>> .
>>
>
>


Why not just remove the method definition here?


If you want to prevent the text from wrapping underneath the phase tag, padding-left: 4em / 70px would work here too.


` +Path field: "path/to/file" `

Should probably be:

` +Path field: "/path/to/file" `


Yes, there's a need to get all members from the parent during assertion of attribute and function overrides.


@piton Пока не понадобился, поэтому пока тут. Вполне возможно более хорошим решением будет доставать регулярку из devise. Пусть они думают какая она должна быть.


It just looked to me like `enterThread` always registers the current thread as a Ruby thread. I wasn't sure if that was intentional or not.


$('#time_tab').countdown({until: time, expiryUrl: url, onExpiry: liftOff, format: 'MS'})


The code should be a bit more short.... meh lazy.


Yes, I guess.


你吃饭木


How does this make any difference?


Yeah it is not important to change that.


Remove.


s/timee/time/


for two cases? :S (4s and 5s)


True, much better this way!


spelling mistake


rs=me but spelling mistake


Demonstrating the sample project from eclipse to git.


Empty line.


This is bad from performance perceptive - this type of getter cannot be observed, unless evaluated. You will also get a warning in console:
![image](https://cloud.githubusercontent.com/assets/1775602/13493786/fef96a72-e148-11e5-9bd6-51b869e9a893.png)


can you rename the variable like `labels1` to `labelRgChina`, `labels2` to `labelAzAz1`?


Exactly.

In fact, what we should do is have firedrake-install dump an actual configuration file and use that. However that is for another branch.


Think that better store path to module to some variable and dont call twice this function.


Ok I'll review it.


Is that syntaxicaly correct? The else should not replace the previous (36) line?


@piton Лучше смотреть финальный код. этого там нету. Я если честно не помню зачем это делал.


```
player.commands.add({
  base: 'save',
  info: 'save the world',
  usage: '/save',
  action() {
    serv.world.save();
    return serv.color.green + 'The world has been saved';
  }
});
```


At line
47  variable_set('ting_carousel_search_queries_hidden', array());
you setted it to array,  i am little bit confused.


@piton в финальном коде регулятора убрана


Daniel,

It contains all the PT rules, thus the "MISC".

When I have the time I will to sort them according to type.

For now it will do I believe.

:)

Kind regards,
     >Marco A.G.Pinto
       -----------------------

On 03/03/2016 08:03, Daniel Naber wrote:
>
> If |Gramática| means grammar and this category contains only grammar
> rules, then the id should be |GRAMMAR|.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/f178b7c4c04cf2697746efaa4b6a61c52ef53a69#commitcomment-16466659>.
>

--


I would move "Calculate element quality..." after "Extract surface" because the element quality is a non mesh changing operation. Maybe it is possible to insert a separator (separating line) between the groups?


@piton Это да, просто на скорую руку сделано было.


thanks for this pull request


Horizontal spacing ;)


Change .text here to .val


fixed now, thx


"Rename `filter` -> `filters`" since there can be multiple "filters", though I don't think this one is necessary.


It's working fine for Ubuntu 14.04 LTS version along with NGINX server configuration


In TTK there are different checkers for LibreOffice and OpenOffice.org https://github.com/translate/translate/blob/master/translate/filters/checks.py#L2315-L2316


Ah yes, thank you for the suggestions. I just whacked everything together and pushed it to my fork, without thinking too much.


@mauritsvanrees what are the implications of implementing to support for CSRF protection on a package if `plone.csrffixes` is not installed? for instance, we have been working on a [branch of collective.cover to achieve that](https://github.com/collective/collective.cover/pull/594) (CC @rodfersou).


I don't think this works with "aabaa" - as it would return aaaa?


sdfsdf


Hace referencia en github a  #21 , #18 y #22


Thats now #728.


Se referiere a la incdencia del #21


@SubJunk OK I will do it. Maybe there could be a check box "Stream subtitles not transcode if possible" at the UMS subtitles settings page to give users a choice to decide.


why would you post your issue in the comments of a totally unrelated commit?


ahh. but if that were the case, then how to implement it? With plain assert(), you can only fail after it did NOT trigger. With a sophisticated ASSERT() like I described above, ordinary mocks will do just fine. I mean, I really can't imagine how what you propose would work. It sounds mock-like to me. But it's (simple) macros?


Sure, just copied it from another project


Enforce assignments : typo caused invalid iptables syntax.
I tested all cases and it now works as intended:
Known IP, wrong MAC : BLOCK
Known MAC, wrong IP : BLOCK


Curiosity: is not possible to have `UINT16_MAX` ports? I know it should not be likely, but do we have restrictions on that? What if someone creates a node that has?


@piton Помоем они и тут локализованные должны быть.
simple_form - можно, но не всегда форма однородно составляется. То есть какие то одинаковые элементы должны быть чуточку по разному оформлены.. В общем просто не стал юзать в данном случае. А так да, можно.


Your json file does not exist


 it works now :-D

On Thu, 3 Mar 2016 at 12:09 George Thomas <notifications@github.com> wrote:

> Change .text here to .val
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/suneilbasu/masterplan/commit/04ae97d44b938f1f4e000d626d5e8c4c83130a9d#commitcomment-16471307>
> .
>
--
Suneil Basu
Linden House
155 High Street
Yatton
Bristol BS49 4DB
01934 835 249
suneil@futureprogress.com


No i added the jhipster ga code look at the events section
On 3 Mar 2016 19:30, "Julien Dubois" <notifications@github.com> wrote:

> Did you code this with your GA code? I'm not seeing anything in my reports
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16470680>
> .
>


The caption is almost unreadable now on wxGTK 3.1.0:
http://i.imgur.com/cRBmqpL.png


It is but it doesn't look like because it's not indented. If you look at the full code, from  line 34 to 42 it will make more sense


Fixed in latest commit.


@dmuhamedagic Have you had a chance to investigate this?


I would like to do `X[j, :].reshape((1,-1))` to avoid depreciation warning but it does not work with sparse matrix. Any idea?


```
save() {
  var obj  = mca.readSync('../world/level.dat');
  obj.$.blocks.$ = this.dump();
  var buffer = mca.writeSync(obj);
  fs.writeFileSync('../world/level.dat', buffer); // not tested
}
```


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


need a white line before this comment, or some description that the following test cases are for anti affinity?


Is it possible to integrate the yii2-authclient support into an older version of humhub (current stable)?


Yup! Just realized it a second ago... But without the sorting, it should be correct.


The async Receive was called by mistake here because PipeTo returns a Task.
It prevented the EndpointManager from handling messages while the Listens is not completed, it this case it wasn't that bad because it was not a lengthy operation.


Thanks George.
Could you tell me where I could find out what alternatives I could have
used? text, val, etc...

On Thu, 3 Mar 2016 at 12:12 Suneil Basu <suneilbasu@gmail.com> wrote:

>  it works now :-D
>
> On Thu, 3 Mar 2016 at 12:09 George Thomas <notifications@github.com>
> wrote:
>
>> Change .text here to .val
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/suneilbasu/masterplan/commit/04ae97d44b938f1f4e000d626d5e8c4c83130a9d#commitcomment-16471307>
>> .
>>
> --
> Suneil Basu
> Linden House
> 155 High Street
> Yatton
> Bristol BS49 4DB
> 01934 835 249
> suneil@futureprogress.com
>
--
Suneil Basu
Linden House
155 High Street
Yatton
Bristol BS49 4DB
01934 835 249
suneil@futureprogress.com


Will this introduce any performance penalties? If this is needed only for the cyclic group, will it make sense to call `IsCyclic` and ask for `SmallGeneratingSet` only in case it's cyclic?


Are you sure you should add ProtocolHandler and MediaManager libraries twice?


What if one of the current nodes fail.


You can get this data  easier with a little trick - you have [#parents] in clicked button, if you will use drupal_array_get_nested_value($form_state, $form_state['clicking_button']['#parents']), it will loops and will find needed part of formstate.


Delicia :+1:


@piton эмм, вот тут не понял... Вы имеете ввиду что то вроде:

task = Task.find(params[:task_id])
attachments = task.attachments
attachment = attachments.find(params[:id])
attachment.destroy

Как то строк многовато мне что то не очень ..
Тот кто будет дебажить, сам разнесёт это на строки во время дебага, а в репизитории всё должно быть аккуратно и отдебажено :). Тут всё норм и выглядит тоже норм )


t()


I guess there was no trigger earlier, i just did some download check now

Thanks & Regards,
Deepu

On Thu, Mar 3, 2016 at 8:13 PM, Deepu K Sasidharan <d4udts@gmail.com> wrote:

> No i added the jhipster ga code look at the events section
> On 3 Mar 2016 19:30, "Julien Dubois" <notifications@github.com> wrote:
>
>> Did you code this with your GA code? I'm not seeing anything in my reports
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16470680>
>> .
>>
>


i am working in windows and try to mex thisfile (mex -largeArrayDims viterbi_liteC.c) but the matlab give
viterbi_liteC.c(37) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(40) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(42) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(43) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(45) : error C2275: 'mxArray' : illegal use of this type as an expression
        c:\program files\matlab\r2013b\extern\include\matrix.h(257) : see declaration of 'mxArray'
viterbi_liteC.c(45) : error C2065: 'delta_mat' : undeclared identifier
viterbi_liteC.c(46) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(50) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(51) : error C2143: syntax error : missing ';' before 'type'
viterbi_liteC.c(54) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(54) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(55) : error C2065: 'phi_old' : undeclared identifier
viterbi_liteC.c(55) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(58) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(58) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(64) : error C2065: 'phi_old' : undeclared identifier
viterbi_liteC.c(64) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(64) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(64) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(70) : error C2065: 'p' : undeclared identifier
viterbi_liteC.c(70) : warning C4244: '=' : conversion from 'double' to 'int', possible loss of data
viterbi_liteC.c(71) : error C2065: 'tmp' : undeclared identifier
viterbi_liteC.c(71) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(71) : error C2065: 'phi_old' : undeclared identifier
viterbi_liteC.c(71) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(71) : error C2065: 'p' : undeclared identifier
viterbi_liteC.c(74) : error C2065: 'tmp' : undeclared identifier
viterbi_liteC.c(74) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(77) : error C2065: 'tmp' : undeclared identifier
viterbi_liteC.c(77) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(78) : error C2065: 'tmp' : undeclared identifier
viterbi_liteC.c(78) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(83) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(83) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(84) : error C2065: 'delta' : undeclared identifier
viterbi_liteC.c(84) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(90) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(90) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(93) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(93) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(94) : error C2065: 'phi' : undeclared identifier
viterbi_liteC.c(94) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(98) : error C2065: 'loglik' : undeclared identifier
viterbi_liteC.c(98) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(100) : error C2065: 'path' : undeclared identifier
viterbi_liteC.c(100) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(102) : error C2065: 'path' : undeclared identifier
viterbi_liteC.c(102) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(103) : error C2065: 'path' : undeclared identifier
viterbi_liteC.c(103) : error C2109: subscript requires array or pointer type
viterbi_liteC.c(103) : error C2065: 'delta' : undeclared identifier
viterbi_liteC.c(103) : error C2109: subscript requires array or pointer type
  C:\PROGRA~1\MATLAB\R2013B\BIN\MEX.PL: Error: Compile of 'viterbi_liteC.c' failed.
any advise to work


Just so that I understand, what's a use case where you'd want to specify the ID rather than just letting one be generated?


Так точно!


Does the root also need setting to make sure this only operates on this instance of the component?


It may be not very good to change the assertion level in the stable branch - maybe it would be safer to leave this line unchanged?


i can not run by mex file in matalb for windows


Was looking only at that commit, but the branch is ok.


Come to think of it, should the selectors include the unique ID to ensure no clashes? Not that you'd expect the children to contain any dropdowns themselves, I suppose.


Nice one! And a really efficient solution too


How much do you charge for sending me only the template imc, I can not edit
the position of the map! Please!
http://www.sobreviveremsantos.com/


This the old approach, i don't wanted to delete it completely, so i will keep it for a while.
See the branch MultiCamSupport instead please, that's the approach we discussed ;)


And what is wrong with this?
```c++
#include "CppUTest/CommandLineTestRunner.h"
#include "CppUTest/TestHarness.h"
#include "CppUTestExt/MockSupport.h"

// Test double for assert()
void assert(bool condition) {
    if(!condition) mock().actualCall("assert");
}

// Production code
void my_thing(void * other_thing) {
    assert(other_thing);
    // Do useful stuff with non null other_thing
}

TEST_GROUP(useful) {};

TEST(useful, null_pointer_should_trigger_assert) {
    mock().expectOneCall("assert");
    my_thing(0);
    mock().checkExpectations();
    mock().clear();
}

int main(int argc, char** argv) {
    return RUN_ALL_TESTS(argc, argv);
}
```
That's basically exactly @jwgrenning's example done with a mock.


I'd suggest to update the comment: Skipping the test for `Select` if it's enclosed in a `TypeApply` is generally a good idea and not necessarily related to SI-9546. Maybe explain why we actually need to do the rewrite already at the `TypeApply` level in the generic case.


I'm not comfortable with the assumption that if resolved is false it means that the blood type is not determined. It could be that the resolution is not currently known but it will be resolved again later.

This solution does work well with our frontend (which doesn't send requests if the user doesn't tick resolved or indeterminate), but on its own I think it could cause confusion.


There is a slight problem with bl_info: it is in conflict with PEP8-E402. How should we proceed?


newline needed


Thx :)


DO we want to force set the shell type to `sh` ?
Shouldn't we atleast the user set this if not autodetect ?


@aleos89 wrong " ) "


> hm, ça me paraît un peu lourd les 515 lignes de Simple_nstream +
> redéfinition des tokens, au regard de la taille du code qui
> l'utilise. On ne peut pas se baser sur les nouveaux tokens et juste
> sauter les EOL ?

La difficultés, ce n'était pas le EOL, mais de réaggréger les tokens
STRING, COMMENT... Je viens mettre à jour avec un patch beaucoup plus
simple (qui remonte la redéfinition des tokens dans ocp-indent).


It would be better to cache the whole `/.cabal` directory ? Later if a package has `libs` also next time ?


This code needs to be copied to https://github.com/OpenRCT2/OpenRCT2/blob/develop/src/ride/track.c#L3345 to fix #3075


I also can't connect the dots here why this change fixes the SI-9546 crash - i guess `isSimpleCaseApply` actually went true? maybe that should be fixed too?


You can import multiple classes from one file like in `.root` in the beginning of this file.


Can't you write the message part just `(.+)` ?


There are a bug that is when $path is empty string, all ftp items in the account will be removed


I suggest in this file, it is better to use a valid test case, because it will also be used as an example.


Introducing a dependency between the two checks is really not great here. If the method in unclosedResourceCheck change it will be really unexpected to see the behaviour of the rule template change!
This has to change from a design perspective to at least make explicit for future maintainers the impact of modifying the close resource method.


http://mokevnin.github.io/blog/2012/06/03/razdelenie-polucheniya-i-ispolzovaniya/ - прочитай


Is it desired to log the check in this case?


Hi, thanks for using angular-filemanager.

Maybe you want to wait some days to develop the connector for the new API (that support multiple file interaction)


## can we automate this in r instead of doing it by hand?


allright, got confused ;)
keep on the great work, looking forward to merge it ...


Would the regex for `file` work if I have brackets in the file name ? I think `\w` doesnt allow it ?


The same one where you'd wish to add any html ID I suppose! I hadn't originally included it but we had been asked to enable it in the buttons.


It needs to be mentioned that this only applies to intel-core2-32, intel-corei7-64 and intel-quark.


And please run `hack/verify-all.sh` and the whole e2e tests, IIRC, there will be some failure because you add this file, and you need to refresh the tests up to date.
You can also refer to the node affinity PR to find the files to refresh.


Hey,baby,what should I do


why does coala not pick this up? Does it not traverse hidden directories by default.


Why so strange index of array - sort[] ?


I dont' think this import is needed.


The selectors currently get the selector value (localId or ID)  from the trigger instance and only operate on the targets with that same value. Is that similar to what you're were saying?


"...Тот кто будет дебажить, сам разнесёт это на строки во время дебага..." - зачем создавать лишнюю работу другому разработчику?

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

"...а в репизитории всё должно быть аккуратно и отдебажено..." - причём здесь репозиторий, если код находится в контроллере?


Check the globs in the coafile - no glob will capture this file I think


Agree with @valib . Some renderers doesn't support streamed subs for every container (e.g. PanTV) so it can support AVI (DivX/XviD), MKV (h264,...), MP4 but not MPG (mpeg2,...) so burning subs when video is already transcoded is safer. Also then there is better support for positioning, colors, fonts etc. when subs are burned so maybe by default subs should be burned in?
And for others there will be tick for always stream subs if supported.
Maybe it will be the most user friendly and renderer compatible solution. :+1:


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 171](https://teamcity.labkey.org/viewLog.html?buildId=307445&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 5 (1 new), passed: 42, muted: 4 Build time: 01:24:44
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSTest.verifyLearnAboutStudyDetails: <no details avaliable>

```


search.htmlのリンク先を
railsの「search_path」に修正！


@lightmare - yes, I forgot to swap `src/renderer_common/render_markers_symbolizer.cpp and `src/marker_helpers.cpp` as heavy source, thanks!


Good day!
I like your feedback form, but on my site it does not work.
Error in contact-form.js _Uncaught TypeError: Cannot read property 'message' of undefined_
line 78
Can you help me?
Thanks!


Would passing in the root require a `ref`attribute in the React component?


t()


@rajmani1995  Whom are you planning to hack? XD


We really need to start modularizing these components. The use of addins will become too big for production use.


Changed in 447bf3693474a35033b3387ecda163be5cac228b.


better to indicate the namespaces field also, to give a full example.


Like I said in the commit message, ride index was being used instead of the ride entry id for get_ride_entry.
I have changed it to get_ride_entry_by_ride which will use the entry id when given a ride.


In `DonationController` the form parameter in the method `addDonation` needs to be renamed to `donationBackingForm` so it will be validated correctly


Okay, but then `Gramática` should maybe be renamed?


I think using type switch is a bit simple.

```
switch i := val.(type) {
case int:
	return float64(i), nil
case int64:
	return float64(i), nil
case bool:
	if i {
		return 1.0, nil
	}
	return 0.0, nil
...
```


Not sure about this one, but should it be a validation error if you are intending to mark the Donation with NO_TYPE_DETERMINED and try specify the bloodAbo and/or bloodRh?


Is there a reason to not use `for (const auto &genome : reference)`?


Using *_raw functions is indeed preferred as they avoid the overhead
of translating the context to/from the readable format which is not even useful for the code.


:+1: :clap:


You get a pointer here to nowhere. Temporary std::string object returned by portStr.str() is destroyed right after this line.


Вариант с MultiLevelCollector работал без ошибок на малых метриках с данным вариантом решения.


Sure. About time to finally do this.
There you are https://github.com/jenkinsci/ircbot-plugin/commit/ff51b2d875c52117ba075fc1987e7d88885df2c1


id="logo"重複


Oh yeah i'm being a twit. For some reason i failed to realise that rideIndex should be rideType.


Great.

Can you also add a test to check the following situations:
 - no errors
 - bloodAbo set but not bloodRh
 - bloodAbo not set but bloodRh is
 - resolved is not set

That'd get you full coverage on the validator (as it is now) I believe.


Why the const here is a problem?


Personal Resources overlaps the chat history. a value like 7.5f would fix this.


We also trigger USER_ROLE_CHANGED when other user became moderator.
Not sure if its possible to revoke moderator rights at all, so we may rename that event, but for me it seems to be good enough to leave it as it is :)
As for guarding "if" clause, there is already on in the `conference.setStartMutedPolicy` method, which ensures that local user is moderator.


画像がない場合、<img src="nil" >とかになってしまうかな
まあでも一旦OK


Et si on met directement
````
virtual IntrinsicBase * clone() const = 0 ;
````
dans IntrinsicBase  ça te convient ?


> Dependence on shiny package should go into Imports section of the DESCRIPTION file.

I am wondering whether this is required if we deploy application to standalone shiny server and do not run it from command line with incorporated shiny server as described on:
http://www.r-bloggers.com/supplementing-your-r-package-with-a-shiny-app-2/

Our package does not require shiny to operate, it only contains shiny resources to be understood by standalone shiny server.

What do you think?


Can we reduce the HTML and still generate valid HTML or should we rather not try to provide valid HTML ?


Its not. At least the draw sprite version of this I removed these assignments after i had my own variables.


No, problem. Let it be


Perhaps this should rather be in WidgetWrapperMixin ?


shell вообще не круто, не идемпотентно получается, не так долго плейбук написать, хотя бы с теми же командами


So you do not wish that `PointOfInterest.select(PointOfInterest.sortItmes.title).fetchCount(db)` would perform `SELECT COUNT("title") FROM "pointOfInterests"`.


Correct, this was inherited from RelatedWidgeWrapper and doesn't make sense anymore.


Thank you!


Parce qu'après dans le make_pair, il y a un const et un non const du coup il râle. C'est du quick fix, et je soupçonne un soucis avec la libc++ de clang 3.8. Ca compile mais ça me parait suspect également.


sonarComponents.checks() cannot Be Null no need to check for it.


@philipobenito exactly as @GrahamCampbell pointed out. Semver would advice caution on `0.*`, so there _is_ an argument on keeping `^0.6`. But it also depends on how `nikic/fast-route` is tagging right now.
If it was expected to keep up with min releases as well, then it should be changed back to `~0.6`.


buuut - I think the deadlock issue was only resolved in PircBotX 2.1 : https://github.com/TheLQ/pircbotx/wiki/ChangeLog#21---january-24-2016


We should add it, could you enable travis on your repo so that we'll see them run on the next PR please ?


Yes, we will modularize this by setting each inherits inside the module
app.xml. e.g. <inherits name="gwt.material.design.addins.Autocomplete" />

On Thu, Mar 3, 2016 at 8:32 PM Ben Dol <notifications@github.com> wrote:

> We really need to start modularizing these components. The use of addins
> will become too big for production use.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GwtMaterialDesign/gwt-material-addins/commit/eacf956a822b3206f5ab36558fa507c2df3b5ee3#commitcomment-16471738>
> .
>


ok, how about get_case_property() - the couch model just does getattr() on the case but the sql version will only return dynamic case properties. i think that might cause some incompatibilities


어... 뭐지... merge가 이상해요


Ah c'est pareil dans l'utilisation que tu veux en faire, l'idée c'est de se
la jouer génie logiciel et ça permet de tester plus tard si ta classe
supporte le clonage (à la compilation, avec du SFINAE).

2016-03-03 13:39 GMT+01:00 Pierre Moulon <notifications@github.com>:

> Et si on met directement
>
> virtual IntrinsicBase * clone() const = 0 ;
>
> dans IntrinsicBase ça te convient ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rperrot/openMVG/commit/26860f1bbed085e0f72cd81944e3bf8fdc2f49f5#commitcomment-16471875>
> .
>


I will rename it on my next update.

I wanted soon to add more "comma" rules :) knowing I will have to
message the mailing list since I don't have a clue on how to check for
commas before words.

Kind regards,

On 03/03/2016 12:34, Daniel Naber wrote:
>
> Okay, but then |Gramática| should maybe be renamed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/f178b7c4c04cf2697746efaa4b6a61c52ef53a69#commitcomment-16471760>.
>

--


OK, our changes didn't conflict.


never mind, see you added a commit for that


以前行った修正が戻ってるので下記に変更
book.status.nil? → book.status == 'not_yet'
book.status == 1 → book.status == 'in_progress'
book.status == 2 → book.status == 'on_loan'


eng-hin.tei is a generated file.  Changes to it are lost once it is regenerated.


@PaulWoitaschek Do you think this will fix #7?

So far, not a single affected user was able to give me any information what these incorrect new sample value and channel count could be, so this is mostly guessing...


I can certainly push the condition into `isSimpleCaseApply`. It will always be false when called from transformation of `TypeApply`. That's the reason I moved it here: I thought it was easier to explain what it was doing right here looking at the `Select`.

The crash came because:

  - We refused to transform `CaseClass.apply[Int]`  to `new CaseClass[Int]`, because the constructor was less accessible than the `apply` method, so some call sites might fail to typecheck.
  - Upon recursion, we then came to `CaseClass.apply`. The existing code seemed to assume that we could only get here from a monomorphic case class (for which this tree would have a non-HK type). However, in cases when we *did* get here with a polymorphic case class, we tended to get the same result from `isSimpleCaseApply` (formerly called `doTransform`), which was `false`.
  - As you surmised, this crash was because `isSimpleCaseApply` gave `false` for the `TypeApply`, but `true` for the `Select`. This led to us creating `new ClassClass[TParam][Int]`.

Now, we have two lines of defence: `isSimpleCaseApply` gives `false` in both cases (now that we use `finalResultType`), and we don't ask the same question a second time anymore (which guards against unknown / future bugs).


t()


awesome


@AbdealiJK You mean something like ` ../hello(name)regex/hey.sh `? I think it will. I have also included `\W` , so any non-word characters can be matched.


I think there's a negation bug here. This line will throw a NullPointerException when nickname is null.


Agree.
Why don’t i see logs in my console?

From: Jenya Y. [mailto:notifications@github.com]
Sent: Thursday, March 03, 2016 2:06 PM
To: Enttoi/enttoi-ui-web <enttoi-ui-web@noreply.github.com>
Cc: AmirSasson <Amirsas@gmail.com>
Subject: Re: [enttoi-ui-web] rejecting subscribe to available restrooms (b5a0c73)

This is bad from performance perceptive - this type of getter cannot be observed, unless evaluated. You will also get a warning in console:
[image]<https://cloud.githubusercontent.com/assets/1775602/13493786/fef96a72-e148-11e5-9bd6-51b869e9a893.png>

—
Reply to this email directly or view it on GitHub<https://github.com/Enttoi/enttoi-ui-web/commit/b5a0c737de1c3bef5761190d328f8aac4a9534dc#commitcomment-16471252>.

This email message and its attachments are for the sole use of the intended recipient(s) and may not be shared with any other party. They may contain confidential information of 888 Holdings plc or its direct and indirect subsidiaries (together, the “888 Group”) and are to be regarded as confidential information under any non-disclosure agreement. Any review, use, disclosure or distribution by persons or entities other than the intended recipient(s) is prohibited. Nothing in this message is capable of or intended to create any legally binding obligation. The 888 Group will only ever assume a legally binding obligation where recorded in a written agreement duly executed by the authorized signatories of the relevant 888 Group company. The 888 Group accepts no liability for any personal views expressed in this message. If you are not the intended recipient, please contact the sender by return and destroy all copies of the original message and its attachments. Thank you


does the framework prevent overlap between case_json and the native fields?

for example, what happens if someone submits a form that updates a field like owner_id - will the framework set the owner_id property or will it update the case_json?


I can't do that, because I'm using as vendor library. It's not good practice do change something in vendor...


You can do it in field definition  - line 240


This removal allows people who make forks use it own repository in bower


Ah, in the Makefile. Thanks, I missed that.


script should be executed as root no?, sudo is not useful then


The benefit of `strip_heredoc` is that you can indent the heredoc and it will strip the leading whitespace :)


This one should be completely removed.


Wait, then why not just `.+`?


Why it is needed? when drupal render a part of form, it will be not rendered second time.


Thank you.


As a consequence of removal of constructor with sonarcomponents, this if-else should also be simplified.


please update your it/sources repo to latest commit, thos modification should not appear.


soya not good


At all or only this message?


Relying implicitely on the order of check is not so great. It would be better to have something explicit here.


I think this is a bad idea. Larger acc vectors are by definition not gravity, and should be ignored. All this is accomplishing is reducing the accuracy (resolution) of the acc data within the useful range....

/Z


at all..


LGTM, please rebase and we can :shipit:


I just realized that using the conditional here for visible_in_eng? prevents secenario: 'switches to Español in cigarette counter' from discriminating between one language and the other. Thoughts?


Je ne sais pas quoi faire des deux gares suivantes :
23443 Brennero-Brenner  brennero-brenner  118100332
23438 Bichlbach-Berwang bichlbach-berwang 118102247


for the two custom ones - it looks like those are only compatible with the couch model.  is that just because the code for those hasn't been migrated yet for sql compatibility?


No Problem. - btw any reason for removing -ftree-vectorize ??


i can set the "IsAvailable" like you set the "isOnline", would this suffice?


Comma


Du tør ikke.


This solvers second task on #1


:chicken:


If "valid" means "valid according to HTML standard", then I'd not try to provide valid HTML. What matters is if browsers can cope with what we give them. For any sane browser (maybe even IE6?) this suffices: `html = "<script>opener....</script>"`


Почему ManyToMany?


This not only relies on the function `mcrypt_create_iv`, but also on existence of `/dev/urandom` on the system. So it would make sense to add the some constraint as for the `/dev/urandom` fallback.


Komentarz do lini


51行目
<%= link_to "貸出OK", borrow_apply_confirm_book_path(book) %>


@MasterZap
Well gyros can go up to 2000deg/sec, which is the highest possible value. So why not use same for acc. Horizon modes can really build up some G's
I was really not involved in the full IMU rework a while ago. So not much I can say about it.
I don't really see a big deal with released resolution. But lets test with it and see how it goes. That's what betaflight project is for.


Why is this even needed? Is it so that you can `choices` from outside in a transparent way? Then I'd go for a "proxy" property.


Methods that return AutoDriver should 'return this;'


Exactly!


56行目
 <%= link_to "取り消し", borrow_cancel_confirm_book_path(book) if current_user.id == book.borrow_user_id %>


OK. These are great resources. I will consider write some scripts to crawl them later.


**Code Quality: Nitpick**

Class names should be written using `PascalCase`. This class should have been called `FileParser`.


Je ne sais pas quoi faire de ces deux gares:
23443 Brennero-Brenner  brennero-brenner  118100332
23438 Bichlbach-Berwang bichlbach-berwang 118102247


+1 for the courageous attempt


To nowe repozytorium `New Brave World`


61行目
<%= link_to "返却する", borrow_return_confirm_book_path(book) if current_user.id == book.borrow_user_id %>


Do we need the migration that is mentioned in the comments of `common_passwords.rb`?

> If the password file is changed, you need to add a migration that deletes the list from redis so it gets re-populated


Ah, sorry, forgot that you expected more input from me. Not sure
where to start investigating or what is the problem :-/

	# rpm -q --provides glib2-devel
	glib2-devel = 2.22.5-0.8.14.1

Yes, there's a .pc file and it seems fine:

	# rpm -ql glib2-devel | grep pc$
	/usr/lib64/pkgconfig/gio-2.0.pc
	/usr/lib64/pkgconfig/gio-unix-2.0.pc
	/usr/lib64/pkgconfig/glib-2.0.pc
	...
	# pkg-config --cflags --libs glib-2.0
	-I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -lglib-2.0


@fixedmachine


Yes. We aren't planning to migrate any existing domains to SQL yet. Also those should really be changed to form submissions anyway.


72行目
<% if current_user?(book.user) && book.status == 'not_yet' %>


Done!


Added a "Quotebox" with red bars left and right.


Provide default value in case when variable was not initialised.


Why? We have more bits than INT_MAX on 64-bit Linux...


yes


@pynicolas done c89b6cb47f8d8eb875158464d81e3f5f03f8f5a2


This is correct


@pynicolas fixed 47a59f021b19c38185b788eecc8c0dbad3d9324b


Note that you can also import singular methods if you're not using all of `fs`.

```js
import {readFile as read} from 'fs';

// ...
read(fileName, (err, data) => {
// ...
```


tak @MarekDoligalski


@pynicolas fixed 9494352672bbe14898d022444705527c6e6fa48a


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


**Code Quality: Nitpick**

There only needs to be a single instance of the `Fileparser` object here hence you would do this

```python
fileparser = Fileparser(filename)
fileparser.read_file()
fileparser.sort_fellows()
...
```
It is also important to note that for minimal errors, the file reading should take place within the constructor.


Maybe use `auto const*`, so we do not forget that we are dealing with pointers. ^^


统一格式?


Mentioned `rpm -q --provides glib2-devel` should be a good start.
Or `repoquery --provides glib2-devel`.


It will update the `owner_id` property. `case_json` should only contain 'custom' case properties.


@SanketDG  I found `[\W|\w]+.sh`  intuitive. Is there any particular corner case?


```delete nullptr``` is valid by the standard.


把一堆`    `和`  `改成了一个空格…………会出现问题吗


I agree this will be a problem, but can we do something less verbose?

How about the classic `(void)sink;` statement? We can declare a macro to make it more obvious what's happening. Say, `TLOG_UNUSED (_x) ((void)_x)` (in misc.h). If you don't have time for this, just say so and I'll do it.

I'm not quite sure about the effect of __unused__. Does it warn you if you use the __unused__ variable? If not, then there's not much value to it. Please correct me if I'm wrong.


How about we go with a community-style system for this: we ask for feedback from the team or just the reviewers and if nobody objects, it gets accepted. If there's no unanimity, then we do a vote around reviewers and majority wins? I took this idea from NodeJS governance: https://medium.com/the-javascript-collection/healthy-open-source-967fa8be7951#.k4v8a8n7r


Stalker! Nice idea though.


Yeah, it seems like that is trying to match everything anyway

On Thu, Mar 3, 2016 at 6:19 PM, Sanket Dasgupta <notifications@github.com>
wrote:

> Wait, then why not just .+?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tushar-rishav/coala-bears/commit/9e2ec6e50b4131379bb73b771a3a11af21114759#commitcomment-16472084>
> .
>


Still not sure why you do a full git checkout?


9行目はジャンルだから下記になるかな
<% if book.genres.any? %>
    <% book.genres.each do |genre| %>
   　<a href="/books/genre/<%=genre.id %>"><%= genre.name %>
　<% end %>
<% end %>


hej


**Code Quality: Nitpick**

When using `for` loops in Python, except when the index is explicitly required, make use of the regular notation .i.e

```python
for member in members:
    designation = member[-1]
...
```
This makes your code a whole lot easier to read and understand.


Why use Qt's foreach instead of C++'s one?


source code comments should not exist...


nej


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


Casting to (void*) removes warning:
```
        zc->seqStore.matchTable = (ZSTD_match_t*)((void*)(zc->seqStore.offCodeFreq + (1<<Offbits)));
        zc->seqStore.priceTable = (ZSTD_optimal_t*)((void*)(zc->seqStore.matchTable + ZSTD_OPT_NUM+1));
```


Hum... what happened here, no tests, no header updates?


hello


bläää


@tushar-rishav well, it's a confusing regex,
What you're trying to match is `.+` but youre just saying do \w and \W ...
I.e. it's code obfuscation.

Also, the `.` in `.sh` needs to be escaped I think ?

Also, what if the file is named with `.bash` ? or `.bashrc` and so on ?

On Thu, Mar 3, 2016 at 6:33 PM, Abdeali Kothari <abdealikothari@gmail.com>
wrote:

> Yeah, it seems like that is trying to match everything anyway
>
> On Thu, Mar 3, 2016 at 6:19 PM, Sanket Dasgupta <notifications@github.com>
> wrote:
>
>> Wait, then why not just .+?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/tushar-rishav/coala-bears/commit/9e2ec6e50b4131379bb73b771a3a11af21114759#commitcomment-16472084>
>> .
>>
>
>


I think this is for when the wrapper's .choices has been changed after `__init__`, note that this is present in django: https://github.com/django/django/blob/master/django/contrib/admin/widgets.py#L298

Also, they have implemented `__deepcopy__` in there and we don't: https://github.com/django/django/blob/master/django/contrib/admin/widgets.py#L275

I'm wondering, should deepcopy and self.widget.choices=self.choices be in one implementation and not another, or should we try to removing both from django, run their tests, see what breaks, see if it's still relevant, if not submit a patch to django to remove it, otherwise backport those and include a similar test in our app ?


Thanks--*that's* what I was looking for. I tried `make-symbol` with no luck.


17〜40行目は、不要？
リンクなしの「貸出可」が正？


halooo


Very hard understand in which case we should add this class. May be better add to form element some specific attribute?


testing


今天才看到你的blog，已经2.65了，麻烦更新下


popraw sobie cos w lini 5


Yeah, this is a bit cheating, but well, since most of these warnings are false positive anyways, let's do it.


Why case insensitive?


@AbdealiJK Only the filename ending with `.sh`. What's the issue here?


@AlessandroA as it's a local variable now I would drop the g_*


+1, but I'd make the argument a `Hash` option rather than a `Boolean`.

Sorry, its a bit of a pain since you've already written it like this, but changing method signatures is really difficult to do while trying to maintain backwards compatibility.

If we wanted to e.g. add an option to not reindex (lets say we're going to do this as a big batch later on in some other method) then we'd end up with a `Hash` argument, and reusers who've called `#expire` with `false` will have to update code.

I know this sounds a bit like premature optimisation, but IMO its worth getting it right now because we _know_ that method signature changes make work for reusers.


Oh yeah. I see now.


**Code Quality: Critical**

Lots of repetition here. This code could have been written to be much shorter.

```python
if member[value][-1] != 'STAFF':
    if members[value][2] == 'M':
        self.male_method(male, office, members[value][0], members[value][1])
    if members[value][2] == 'F':
        self.female_method(male, office, members[value][0], members[value][1])
else:
    self.staff_method(office, members[value][0], members[value][1])
count += 1
return self.all
```
This way your code is a whole lot shorter and much more concise.


I’d make this little more generic and make `IdentifierNameWithPos` produce a node of type `Identifier`. This would be simpler (adding one line in the `IdentifierNameWithPos` rule + one line in `ASTVisitor`) and it would be possible to reuse the node in other similar situations in the future (think `tables` in `show postgresql tables`) or even in some situations today (e.g. if we decide that a function name in `function foo() { ... }` needs a location for some reason).


> ```
> # rpm -q --provides glib2-devel
> glib2-devel = 2.22.5-0.8.14.1
> ```

This explains it.  RPM pkgconfig generator is either suppressed or broken
on the respective build system.  You should see something like:

```
pkgconfig(gio-2.0) = 2.28.8
pkgconfig(gio-unix-2.0) = 2.28.8
pkgconfig(glib-2.0) = 2.28.8
pkgconfig(gmodule-2.0) = 2.28.8
pkgconfig(gmodule-export-2.0) = 2.28.8
pkgconfig(gmodule-no-export-2.0) = 2.28.8
pkgconfig(gobject-2.0) = 2.28.8
pkgconfig(gthread-2.0) = 2.28.8
glib2-devel = 2.28.8-5.el6
glib2-devel(x86-64) = 2.28.8-5.el6
```


@vilchik-elena Naming: it seems that "coalesce" more or less means "merge". I think it would be better to name this kind "NULL_COALESCING_EXPRESSION".


@mattwigway : Changing -1 to negative edge indexes breakes Bike rentals and P+R searches. Since in Bike rentals in StreetSegment edges with negative ids aren't found in EdgeStore. If I skip over edges with negative IDs then negative IDs become a problem in StreetRouter:360 in `for (State state : bestStatesAtEdge.get(s0.backEdge)) `.

For P+R it is the same problem with seeking negative edge number in StreetRouter:426..


Приймається


## Tourni
Il faut ajouter le déplacement après avoir tourner sans quoi on risque de tourner sur soit à l'infini.


متشکرم آلن عزیز


fgdfg


ここに変数宣言いれちゃうとまずいのって分かります？


...or RPM not new enough, see the RPM commit referenced in the commit message.


@Patater Why the #define of __UVISOR_DECLARE_BUFFER_OVERLAPS_BOX_CONTEXT? - Why can't that be just a plain inline?


Done


google.pl


I dont understand why so strange index? it is to hard for reading.


> I think this is for when the wrapper's .choices has been changed after __init__, note that this is present in django:

That's horrible.

> or should we try to removing both from django, run their tests, see what breaks, see if it's still relevant, if not submit a patch to django to remove it, otherwise backport those and include a similar test in our app ?

Yes!


Small dynamic bootstrap implementation


I thought I had fixed all the places where we had a seek to have a check if the back edge is negative.

The problem is that the edge index has to always be unique or the states will be considered to be at the same place.


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


Won't this run every [_Sunday_](http://crontab.guru/#54_2_*_*_0) morning? I think you want [`54 2 * * 1`](http://crontab.guru/#54_2_*_*_1).


essaye avec `nb.toString()`ligne 9


Last commit fixes according to your comments https://github.com/practiceprovider/openemr/commit/292f5395eec64a68e5b58d1f5a44f30caca1e669

I haven't removed old code just commented out so let me know if it looks fine then I will remove old code


Why not? Qt foreach works perfectly with Qt containers within both immutable and mutable contexts.


This is Bluetooth low-energy, which our device supports


May be better use textarea instead such big textfield?


Committed this for Fork testing.


As far as I can see this just duplicates functionality of `checkNode` which is hidden behind the `DEBUG` flag.

(I’m open to improving debugging messages produced by `checkNode` and maybe changing the enabling mechanism to something more high-level than flipping a constant. However I’m not so sure about making the debug mode enabled by default.)


Anyway, per wikipidia, 11.4 was EOL'd 2015-07, I infer it's not vital
to support it in the upsteam spec file for an upcoming release.


@pynicolas Agree, 933bd9e844250c5078d3ffa73a3a923a6c42bbb3


why adding brackets arounds hosts in all DBI connection strings?
it will cause an error:
[debug] Got MySQL error when connecting somehost(10.100.200.1:3306) :2005:Unknown MySQL server host '[10.100.200.1]' (1)


I don't know. This code moved from Preferences class (AFAIK, it developed by @sledgehammer999).


OK, I will amend it :D


Ah, that was OpenSUSE, sorry again. What's `rpm --version` there?


Det skulle du aldri ha gjort


It looks like fedora based systems use /usr/include/php for php5 so we will need to allow for both


Thank you. Is still holding!


`std::uniqe_ptr` would be in place here and below, in SettingsStorage::SettingsStorage()


Google Could Message


Great, thanks! I'll let the pull be open until tomorrow if somebody should have further comments


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


iluxa_c0m comment


typo: @shapeshed  bplace => place


No, these aren't needed. It was late. Removed.


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


Ditto.


Off-topic
Can you change your XDA username to non-diacritical marks? So i can make you as a contributor and easily edit the thread


Removed.


this is still necessary on Ubuntu `15.10` (at least)
```
Distributor ID: Ubuntu
Description:    Ubuntu 15.10
Release:        15.10
Codename:       wily
```


I believe there is another method other than call compile, right?


更新评审计划的界面


@guiwoda See differences in effect: [`^0.6`](http://semver.mwl.be/#?package=nikic%2Ffast-route&version=%5E0.6) vs [`~0.6`](http://semver.mwl.be/#?package=nikic%2Ffast-route&version=~0.6)


It feels to me that if you're building the component in React, you wouldn't need to locate the element outside the component, so it's simpler just to always generate a unique ID. Or have I misunderstood?


How to do it :D


@mikemccracken I think this module is missing in the upstream branch as I'm getting import errors


Where is the xml used?


who am i?


> The selectors currently get the selector value (localId or ID) from the trigger instance and only operate on the targets with that same value. Is that similar to what you're were saying?

Ah, right. In that case, as-is should be fine if IDs are unique.


Jest super


good


I admit that I completely forget the story of this very database. What I recall is that at some point, I fully consciously took dictionaries like this one out of the conversion pipeline and converted it to P5, at the same time hoping that I was opening it to corrections such as this one.

However, I sort of recall that my idea about freeing dictionaries from legacy formats did not meet with enthusiasm ;-) Or at least that was I think true of the Hungarian dictionary. So I'm rather lost now, and the answer lies probably in the Perl script and Michael's memory -- it's possible that the script now derives TEI P5 (it used to derive P4).

But why derive it if we can keep it as TEI, open to corrections and enhancements? :-)


very good


Mais justement, ocp-index ignore complètement ces tokens non ?


Or: if `0.7` has no breaking changes (to `0.6`) that have an effect on this package: [`^0.6|^0.7`](http://semver.mwl.be/#?package=nikic%2Ffast-route&version=%5E0.6%7C%5E0.7)


Oh, I didn't know that binary rewriting itself in-place should cause an error (on UNIX-like systems).


tab


This won't work. header.html is loaded *using* javascript. If you disable javascript, header.html is never loaded. The way to do this to splatter the <noscript> fragment all over the HTML files.


space


R6写吗？