format->params.test is supposed to be a null terminated array, so unless there is no elements at all (which would mean just a NULL param), then there is no chance of overflow.  I have checked all formats currently using -1000 and there are none that have less than 2 test vectors anyway.


It still redirects to https even when not available. In what sense is the call to `fsockopen` an indication that https is available?


Because we're called Reevoo, I think


I'm actually not completely sure what the context should be, it was more that I wanted a well-known context defined.


ack in general.

have you thought about making the difference value dependent on the size of the function? For me it sounds like you keep shifting the focus around and maybe this could yield better results across the board.


\o/


chinh sua


@hoholee12 please make sure none of the variables i changed will break the loops, thanks! :)


I changed the file name in my folder to "ReadMe.txt" but when I pushed, It still remain "Readme.txt"


is it that relevant? So I just want to proove with it that if I take
arbitrary implementations without modification it'll just work. (Made me
very happy)

2015-06-24 14:45 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> For the non-clones, you link to german wikipedia articles and at least one
> example has german comments and german variable names.
>
> It might be better to find an english reference or at least refactor the
> variable names?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/a82871ce60c4ea0f80be540ff890f7a138736946#commitcomment-11837579>
> .
>


No, this patch is really not OK. This is how systemd works: we load units lazily, and after we loaded a unit we will not go back to disk for them until the user issued "systemctl daemon-reload" or the GC logic unloaded the unit.

Can you give a usecase example, what behaviour a user experiences without this patch that you think should be different?


> making the difference value dependent on the size of the function

isn't this what I'm doing here?

2015-06-24 14:49 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> ack in general.
>
> have you thought about making the difference value dependent on the size
> of the function? For me it sounds like you keep shifting the focus around
> and maybe this could yield better results across the board.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/657cf7836edb4549e829db0abe146836114a0e21#commitcomment-11837663>
> .
>


The licence has another different name on it...


If you build the service(s) you need directly using the [base manual repository](https://github.com/alphagov/specialist-publisher/blob/master/app/repositories/repository_registry.rb#L31-L33) then you wouldn't need an organisation slug.


Ooh, so it might be that I send some requests before analyzer ready? I had that on a TODO note, but never got around, since it still worked. I'll try to fix up atom so that it awaits this message before doing anything else.


ack


I would prefer to use `std::unique_ptr` since I think the users of this class should do so, too. See the commit https://github.com/TomFischer/ogs/commit/3bb712084bb27cc877293df8e352ac87d99bae25


createPropertyClone api should be common to all the Property Tax services. Why is this service a part of TransferOwnerService?


Hnuh, how come this wasn't here previously?  It appears in Algorithm 1 of W&S2014...


Ik denk dat een student af vinkje inderdaad wel goed is, niet iedereen studeert natuurlijk af.

Ik denk dat je de `date_to` niet voor LvV'ers wilt invullen, ten eerste omdat sommige checks alleen kijken naar of dat veld is ingevuld of niet, maar misschien nog wel belangrijker: het is niet correct, een LvV'er is gewoon nog steeds lid, dus een datum tot zou raar zijn


@unho, I used different way to update cache for regular items. Here I cleared cache for the self only. And I update parent cache from this place just one time https://github.com/translate/pootle/blob/bbf14fed9a63bf690d98a8ca63bc6b1fac8d9039/pootle/apps/pootle_app/project_tree.py#L159.


Ik zal hem sowieso even omschrijven naar een functie die met `manage.py` aan te roepen is.


can you do an npm list --depth=0? it's a pretty odd error


Sure


There are more than 1000 lines in this file.. Let's review and discuss this.


Not necessarily.  I thought that 'doc' was under construction yet so that patch would just fix in the meanwhile.

Of course it would be even better if there is something ready to be installed.


You're right that the use of organisation slug is to handle permission checking and tagging. I think we'd have a more appropriate interface for our services if we made the `ManualDocumentServiceRegistry` use the base manual repository, and add an `OrganisationScopedManualDocumentServiceRegistry` which requires an organisation slug in its constructor.


Hi. Currently interaction of pygraphviz with libgraphviz is done via `agwrite` call and it needs bare file handler which `io.StringIO` doesn't provide. Why don't you use `graph.to_string()` instead?


simplified this commit a lot, will have v2 ready in 10 mins or so

2015-06-24 14:34 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> You work a lot with the length of these dictionaries, I don't know whether
> it adds any overheat to calculate it ever time?. You might as well store it
> I guess?
>
> Also, line 68 can be simplified the same way you did in another commit
> here, with if 0 in (X ,Y)
>
> this is the only function that scrutinizer rates D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/9a94b98b80b7bcee9addf8ce757462f635b97a96#commitcomment-11837368>
> .
>


I will fix that.
Thanks


and I miss alx, repository seems pretty dead... :(


ack

Implicit continuation is also actually preferable according to PEP8


@radar do you remember what the problem here was?  ci.spreecommerce.com seems to be gone.


ack


Can this be simplified to `is_callable`?


makes sense - updated.


@unho, this doesn't work when I remove the whole directory and then run `update_stores`.
vfolders stats stay without changes when the total stats are changed properly.


yes, ack


По базе. Если там будут поля - Название элемента, последовательность, структура, организм.
Есть ли смысл ставить ограничения на такие поля? И делать ли название как primary key? Или это не всегда обязательно?


I think we should use `np.asarray` instead of `np.array` here, otherwise the data buffer get's copied:

```python
>>> _ = np.memmap('/tmp/a', shape=10, mode='w+')  # create a mmap data file in write mode
>>> a = np.memmap('/tmp/a', shape=10, mode='r')  # reopen in read-only mode
>>> a
memmap([0, 0, 0, 0, 0, 0, 0, 0, 0, 0], dtype=uint8)
>>> a.mode
'r'
>>> a.flags
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : False
  ALIGNED : True
  UPDATEIFCOPY : False
```

with `np.array`:

```python
>>> b = np.array(a)
>>> b
array([0, 0, 0, 0, 0, 0, 0, 0, 0, 0], dtype=uint8)
>>> b.flags
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  UPDATEIFCOPY : False
>>> b.base
```

with `np.asarray`:

```python
>>> c = np.asarray(a)
>>> c
array([0, 0, 0, 0, 0, 0, 0, 0, 0, 0], dtype=uint8)
>>> c.flags
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : False
  ALIGNED : True
  UPDATEIFCOPY : False
>>> c.base
memmap([0, 0, 0, 0, 0, 0, 0, 0, 0, 0], dtype=uint8)
```

`c` is still a an array based on a readonly memory buffer.


Well okey if don't mind typing it by hand.


别用内网地址呀！


This creates this column:

`org_status_rv_fk` bigint(20) NOT NULL

org_status_rv_fk gets filled with 0, the 0 produces this hbm2ddl.SchemaUpdate error messages:

Unsuccessful: alter table org add index FK1AEE4BDA6253F (org_status_rv_fk), add constraint FK1AEE4BDA6253F foreign key (org_status_rv_fk) references refdata_value (rdv_id)

Cannot add or update a child row: a foreign key constraint fails (`KBPlus`.`#sql-5e0_a7`, CONSTRAINT `FK1AEE4BDA6253F` FOREIGN KEY (`org_status_rv_fk`) REFERENCES `refdata_value` (`rdv_id`))


Hi @jmchilton @kellrot, sorry about that.  Draft 1 and draft 2 code had diverged sufficiently that supporting the draft 1 code was (to me) imposing a maintenance burden with no upside.  It might be possible to do an automated conversion.  I should also note that the draft 1 spec has gaps, and we didn't go through the process of conformance testing on draft 1 that we're planning for draft 2, so I'm not even sure if tools written for SBG/Rabix would actually work with cwltool.


you mean escaping newline character?


ah nope, conflict. Let's just leave it that way, ok?

2015-06-24 14:45 GMT+02:00 Lasse Schuirmann <lasse.schuirmann@gmail.com>:

> ah yeah
>
> 2015-06-24 14:42 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:
>
>> you can probably just squash this commit and the previous one?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/coala-analyzer/coala/commit/36fb8ae988bcb963112a684e7c24de48cafc55a9#commitcomment-11837510>
>> .
>>
>
>


Did this ever work for you? I get a syntax error that this is not a valid try-catch block. I've wrapped it in a `(do)` now but it still doesn't actually create directories for me on puredata.info


Somehow, this doesn't seem right. Now, in a multi-module project, only the root project uses all repositories to resolve the latest versions, the subprojects don't use any repositories at all and all dependencies are listed as unresolved.


looks weird to me, codestyle wise. I see where it's comming from, but isn't it much more readable to just put them in a line?


you only changed very minor parts, looks like it will work.


no, I mean I introduced a regression. I have a fix in CI, you might want to use that branch :wink:


same thing


other places, too


Okay Cool


The database is small and very often is less than 50kbytes.  Even for huge and complex setups it shouldn't be more than 500kb~1Mb.

This is my current database with 2 ethernet devices:

Bridge table
_uuid                                controller datapath_id        datapath_type external_ids fail_mode flood_vlans flow_tables ipfix mirrors name     netflow other_config ports                                                                                protocols sflow status stp_enable
------------------------------------ ---------- ------------------ ------------- ------------ --------- ----------- ----------- ----- ------- -------- ------- ------------ ------------------------------------------------------------------------------------------------------------------ --------- ----- ------ ----------
556c7745-6478-4a17-ad3b-3aea3f9eb201 []         "0000ecf4bbd0b848" ""            {}           []        []          {}          []    []      "ovsbr0" []      {}           [07d140ab-902e-4bdb-bdb5-f5983d9edc0a, 256339ab-f7ae-4266-9b0e-f24e2888f988, 9c1c2e43-28f7-4663-ab53-729ec8c10f13] []        []    {}     false

Controller table
_uuid connection_mode controller_burst_limit controller_rate_limit enable_async_messages external_ids inactivity_probe is_connected local_gateway local_ip local_netmask max_backoff other_config role status target
----- --------------- ---------------------- --------------------- --------------------- ------------ ---------------- ------------ ------------- -------- ------------- ----------- ------------ ---- ------ ------

Flow_Sample_Collector_Set table
_uuid bridge external_ids id ipfix
----- ------ ------------ -- -----

Flow_Table table
_uuid external_ids flow_limit groups name overflow_policy prefixes
----- ------------ ---------- ------ ---- --------------- --------

IPFIX table
_uuid cache_active_timeout cache_max_flows external_ids obs_domain_id obs_point_id sampling targets
----- -------------------- --------------- ------------ ------------- ------------ -------- -------

Interface table
_uuid                                admin_state bfd bfd_status cfm_fault cfm_fault_status cfm_flap_count cfm_health cfm_mpid cfm_remote_mpids cfm_remote_opstate duplex external_ids ifindex ingress_policing_burst ingress_policing_rate lacp_current link_resets link_speed  link_state mac mac_in_use          mtu  name     ofport ofport_request options other_config statistics                                                                                status                                                                             type
------------------------------------ ----------- --- ---------- --------- ---------------- -------------- ---------- -------- ---------------- ------------------ ------ ------------ ------- ---------------------- --------------------- ------------ ----------- ----------- ---------- --- ------------------- ---- -------- ------ -------------- ------- ------------ -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- ---------------------------------------------------------------------------------- --------
7579cbde-3522-423b-b6ee-9f8fd554d5d4 down        {}  {}         []        []               []             []         []       []               []                 []     {}           10      0                      0                     []           0           []          down       []  "ec:f4:bb:d0:b8:48" 1500 "ovsbr0" 65534  []             {}      {}           {collisions=0, rx_bytes=0, rx_crc_err=0, rx_dropped=0, rx_errors=0, rx_frame_err=0, rx_over_err=0, rx_packets=0, tx_bytes=0, tx_dropped=0, tx_errors=0, tx_packets=0}                  {driver_name=openvswitch}                                                          internal
ee44e8d6-c59e-4086-915d-9be04dfc04ac up          {}  {}         []        []               []             []         []       []               []                 full   {}           4       0                      0                     []           0           10000000000 up         []  "ec:f4:bb:d0:b8:48" 1500 "em1"    10     10             {}      {}           {collisions=0, rx_bytes=2774024248, rx_crc_err=0, rx_dropped=0, rx_errors=0, rx_frame_err=0, rx_over_err=0, rx_packets=189399314, tx_bytes=0, tx_dropped=0, tx_errors=0, tx_packets=0} {driver_name=ixgbe, driver_version="4.0.1-k-rh7.1", firmware_version="0x80000647"} ""

Manager table
_uuid connection_mode external_ids inactivity_probe is_connected max_backoff other_config status target
----- --------------- ------------ ---------------- ------------ ----------- ------------ ------ ------

Mirror table
_uuid external_ids name output_port output_vlan select_all select_dst_port select_src_port select_vlan statistics
----- ------------ ---- ----------- ----------- ---------- --------------- --------------- ----------- ----------

NetFlow table
_uuid active_timeout add_id_to_interface engine_id engine_type external_ids targets
----- -------------- ------------------- --------- ----------- ------------ -------

Open_vSwitch table
_uuid                                bridges                                cur_cfg db_version external_ids                                       manager_options next_cfg other_config                            ovs_version        ssl statistics system_type system_version
------------------------------------ -------------------------------------- ------- ---------- -------------------------------------------------- --------------- -------- --------------------------------------- ------------------ --- ---------- ----------- --------------
24b1487b-36c4-44ad-82bc-929e28d10645 [556c7745-6478-4a17-ad3b-3aea3f9eb201] 378     "7.6.2"    {system-id="1473e4cd-71dc-4a6b-8f28-97e7fdbd3a9a"} []              378      {n-dpdk-rxqs="1", pmd-cpu-mask="0x100"} "2.3.1-git3282e51" []  {}         unknown     unknown

Port table
_uuid                                bond_active_slave bond_downdelay bond_fake_iface bond_mode bond_updelay external_ids fake_bridge interfaces                             lacp mac name     other_config qos statistics status tag trunks vlan_mode
------------------------------------ ----------------- -------------- --------------- --------- ------------ ------------ ----------- -------------------------------------- ---- --- -------- ------------ --- ---------- ------ --- ------ ---------
256339ab-f7ae-4266-9b0e-f24e2888f988 []                0              false           []        0            {}           false       [ee44e8d6-c59e-4086-915d-9be04dfc04ac] []   []  "em1"    {}           []  {}         {}     []  []     []
9c1c2e43-28f7-4663-ab53-729ec8c10f13 []                0              false           []        0            {}           false       [96045eec-453c-4692-8225-9491f2cbdcb6] []   []  "em2"    {}           []  {}         {}     []  []     []
07d140ab-902e-4bdb-bdb5-f5983d9edc0a []                0              false           []        0            {}           false       [7579cbde-3522-423b-b6ee-9f8fd554d5d4] []   []  "ovsbr0" {}           []  {}         {}     []  []     []

QoS table
_uuid external_ids other_config queues type
----- ------------ ------------ ------ ----

Queue table
_uuid dscp external_ids other_config
----- ---- ------------ ------------

SSL table
_uuid bootstrap_ca_cert ca_cert certificate external_ids private_key
----- ----------------- ------- ----------- ------------ -----------

sFlow table
_uuid agent external_ids header polling sampling targets
----- ----- ------------ ------ ------- -------- -------


looks good besides and seems to work


You can get around this with
```
GRUB_GFXPAYLOAD_LINUX="text"
```

Depending on the size of your lish console, you may get output like this:
![image](https://cloud.githubusercontent.com/assets/317653/8329880/4838f858-1a4b-11e5-9eb4-b4e15e27eb62.png)

The only workaround I've found for this display artifiact is to use the hidden menu option, which requires you to press escape to get the grub menu:
GRUB_TIMEOUT_STYLE="hidden"


Indeed this would be setting value instead of comparsion. @dukesrg @roxas75.
Cppcheck told me it tried to set a pointer with a boolean value. Orz.


Ok - it sounds like what you had is fine - we don't want a context passed in!


The report was done on Skype, and seemed pretty logical : service is not a known command for sh
```
sh-4.2$ service
sh: service: command not found
```

While, using vzctl:
```
vzctl exec 251 service
Usage: service < option > | --status-all | [ service_name [ command | --full-restart ] ]
```

vzctl exec uses the shell for user root of the container


Нужно поле id записи. Оно и должно быть primary key. Лучше ещё и auto_increment. Ограничения остальных лучше подробно обдумать, потому что, вероятно, не потребуется их устанавливать


I think this would be more idiomatic using `#any?` – `manual.documents.any? { |doc| doc.id == section_id }`


As above


Кстати, при желании можно вынести название организма в отдельную таблицу


Too many php this day =)


@kkirsche Thanks for the link!


I think it doesn't fit then


Please push such changes to the preview


Hi thanks for your comments. We have done as you have suggested and all is working well. Sorry for the delay in replying i totally forgot. Thankyou again!!


We want to start calling it authorizationStrategy, since it's not really a callback


Put a space between 'private' and methods.
Well done with the identations!
Dont forget to add the 'article' method


имаш if, в който ако влезеш, това е последното нещо дете ще се случи. Защо ти е else?
Освен това, ако завъртиш if-а на обратно, ще е по-добре.


We may want to extract out the strings here ("search=" and "&count=classification.exact") to constants


@FranDepascuali
Well done with the identations.
Dont forget to add the 'article' method.


Put a space between 'private' and the methods


For the record. Calling article as i showed to you is called "Memorization"


I think there's a missing comma here


I agree it would be better. However, this is not how your serialization functions work right now, thus the tests will be FAILED. Is that OK?


And now that I've read the rest of this PR, I see you've done something like that already!


and here...


```class="panel panel-top panel-top_nav"```


Duly noted.


There is indeed.  Will add.


well, *technically* not I guess, because there is nothing to stop iteration just here aside from the string ending. If it doesn't break a test to remove it, I might swap it for a comment


@Bipsy I like that idea for the promises.


changed in v2


```<div class=" panel-top_nav-list ">``` => ```<div class="panel-top-nav">```


Ако няма такъв User, връщаме false?

User, ако не се лъжа, има за @Id името си? Тогава и да искаш, няма да можеш да добавиш 2 с еднакво име в базата.


I am going to reimplement that stuff on top of the current state of the code and check if the tests still pass. Theoretically it should not affect them in any way.


Couldn't these two be chained?


reack


tested and working


simplified in v2


was acked, conflict during rebase, rereview needed.


```<a  class="panel-top_list active_panel-top">``` => ```<a  class="panel-top-nav panel-top-nav-link_link">```


@koying anyway to add a path for android?


reacjk


Тези коментари трябват ли ни?


你好，请问一下关于这个APN的怎么用，我现在有个项目正好用到这个，是接入一个移动的专有网络。里面的描述文件怎么生成呢？希望赐教


Wow that's minimal :D


I think you're right


@hoholee12 yes minor refactoring, just to make sure it will work, ill change more variables later lol, thanks for conformation


they do becase they can escape I guessed.


ack


@hoholee12 i use irc all the time, would be way easier, ill look into setting up a freenode server :)


Now that is an interesting tidbit.


huh? What? Wildcards can escape something? *thinking...*


ㅎㅇ


I am md chobo


OK, I'll remove it.


Does the new commit you made with removing the $(gettext) work better for you?


Oh, you're about that part. Yes too many casts and screen reassignment in the next operation looks a weird.


@Tobion While you may disagree with the patch, the patch is in response to an issue reported by the Drupal Security Team. We have detailed the vulnerability as [ZF2015-05](http://framework.zend.com/security/advisory/ZF2015-05), and the solution we provided has been vetted by multiple members of both our own security team and the Drupal Security Team.

I do not disagree about the correctness of your remarks about multiple slashes. I do disagree, however, that multiple slashes have any valid, semantic meaning in a path. An empty path segment (i.e., `//`, one or more slashes with no characters between) is meaningless, and most browsers, servers, and routers will either normalize them or treat them as a mistake. (Apache, because it treats the path as a filesystem path, inherits behavior from the OS, and every OS I've used treats multiple slashes as a single slash.) Having a path _start_ with multiple slashes has potential security implications when emitted without the scheme and authority, as already demonstrated. We can debate all day about whether or not a path *can* include them; the point is that having them has no practical applications.

If you feel strongly that the solution in this patch is incorrect, then please submit a pull request with your suggested changes that continues to address the reported vulnerability.


The capacitor rate was deliberately low, its intended to be a long term storage method. 10RF/tick was probably too low in hindsight though


It is done


Thanks :)


falta para las otras acciones


Sure, it works also. But then you have to cast the result to String.
You can also get the item direct from the ArrayAdapter:

```
String forecast = (String)adapterView.getAdapter().getItem(position);
```
But you still need to cast the result in String.
The original version looks optimal for me in this case.


Added appropriate mapping - N.B. this commit should also contain a change to disable auto updating of schema?


The 'getIsRepoFailedAttribute()' method which returns a boolean should be named 'is...()' or 'has...()'


PHPCS: Line exceeds 120 characters; contains 121 characters


That does make more sense. Can you open an issue?


Увере, у тебя получится избавиться от span.


Because Python failed to offer me the "one, obvious way to do it"


Yes, they would be formatted and copied, but I think they would add an unecessary complexity to the code, the hard coded 'SOL_FLOW_NODE_TYPE_BUILDER' and 'sol_flow_node_type_builder_options' sounds to fit for me since we only use these strings on 'libsoletta.so-gdb.py'. What do you think?


ack


Fixed, sorry for the embarassing typing-error


performance, I will try to make it simpler according to your suggestions


Это конструктор класса.


@DiamondBond that'd be gr8!


I take my hat off to you, @clebergnu. That's some resourceful thinking right there.


`function (next) {
    this.listen(channel, next);
}.bind(this)`

This causes tests to fail.


uhm does everything in `[]` except circumflex loose its special meaning in a regex automatically? I mean: don't you need re.excape anyways?


no. They result solely from the differences between the old and new glob.

A commit of these without the glob function would break test, as well as committing the glob function without these changes would.


This didn't quite fix it.  The merge! command still references 'config' which is undefined.


@behlendorf I had to add the following patch to allow unloading without a hang:
```
diff --git a/module/spl/spl-taskq.c b/module/spl/spl-taskq.c
index 427284d..59731c9 100644
--- a/module/spl/spl-taskq.c
+++ b/module/spl/spl-taskq.c
@@ -1016,7 +1016,8 @@ taskq_destroy(taskq_t *tq)
         * When TASKQ_ACTIVE is clear new tasks may not be added nor may
         * new worker threads be spawned for dynamic taskq.
         */
-       taskq_wait_outstanding(dynamic_taskq, 0);
+       if (dynamic_taskq != NULL)
+               taskq_wait_outstanding(dynamic_taskq, 0);
        taskq_wait(tq);

        spin_lock_irqsave(&tq->tq_lock, tq->tq_lock_flags);
@@ -1086,5 +1087,6 @@ void
 spl_taskq_fini(void)
 {
        taskq_destroy(dynamic_taskq);
+       dynamic_taskq = NULL;
        taskq_destroy(system_taskq);
 }
```


I guess, the problem is related to the `project.repositories.clear()` in line 61.


Nice... that works for me.


@calumhalcrow So this "CSL App" is already its own repo. Its a mirrored version of Explorer. The front-end styles are coming from the Export.gov libraries. Also there are many layout changes requested by Stuart. One of those requests was to label the total number of results with the text "Complete Consolidated Screening List". So I found a temporary solution that gets this done here: app/components/total-results.js but its probably not the best practice, when the total results change.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 81](https://teamcity.labkey.org/viewLog.html?buildId=256905&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:44
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


Still, i don't really understand your purpose.
* (screen==(TOP_SCREEN&&TOP_SCREEN2))?
* (screen==TOP_SCREEN)?

Which one would you like?


I don't think so, because of the try: except...


I understand, but 10RF/t is VERY low and not useful for anything :)


I'm not sure how making it "done" works - I haven't noticed that feature really... So I'm afraid I have now quick solution for you...


still don't see it...


That for the symbols, but all these others you are using the pointer you receive from the outside, which may not necessarily be a constant string, so you do need to copy those.

As for SOL_FLOW_NODE_TYPE_BUILDER, if we have a name there, I would also add it to the type, so it's clear when debugging which "type" it is.


```
<form class="panel-top-search">
  <button type="submit"><i class="i i_search"></i></button>
  <input type="text">
  <button><i class="i i_reload"></i></button>
</form>
```


OpenStreetMaps provides the map.
Also, these aren't all open source (CircleCI, OpenStreetMaps, Waffle.io). Do we need to list the open source items we use somewhere?


hahaha Stevie G!


aaaaaaaaaaaaa


@jtsoi take a look.


This may change to the Release branch. I'll reflect that once I set that up.


Hello from FULL_TEST_CASE :p


hey, U can try this link to get your avatar  :)
src="https://secure.gravatar.com/avatar/{{ settings.email_md5 }}?r=x&s=440"


you might be right, doesn't look like it would break the case sensitive tests


lgtm


This tag name isn't good. The name means all child elements are "action".


Whoops, meant IPv4; IPv6 will be supported in another function.


Eerst stond er enkel dat de uri werd doorgegeven aan de references. Het is voor de applicatie zeer handig om ook de request parameter door te krijgen. Deze heb ik eerst toegevoegd aan de view en om geen al te grote veranderingen in de code zelf aan te brengen heb ik hier \*\*kwarg gebruikt.
Na overleg met Koen was het beter om hier geen **kwarg te gebruiken, maar wel de request parameter zelf:
* de module maakt steeds deel uit van een pyramid project en de request is een bron van informatie om referenties van de uri op te zoeken
* hiermee maken we de parameter verplicht ipv optioneel


Would it make sense to include `javax.servlet.Servlet` in this list, rather than relying on just the specific `Sling*MethodsServlet` classes? While most AEM servlets extend one of these base classes, other base classes are used from time to time.


It means that if we are writing in the top screen, and a second screen is specified, we write on that second too.
The pointer of the second screen is set to zero if we do not have it, but i usually set it in the entrypoints loader, since i cannot always know the active framebuffer, and writing on two fb isn't that speed-spending.


When I run this command on Mac...

```
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
```

... I get...

```
-bash: command substitution: line 1: syntax error near unexpected token `newline'
-bash: command substitution: line 1: `curl -fsSL <https://raw.githubusercontent.com/Homebrew/install/master/install>'
```


perfect :ok_hand:


As you are using a custom theme, it is important to define it. The BT won't pick up what you didn't configure.
As an example on how to do this, see the following example (taken from https://github.com/sproutcore/sproutcore/blob/team/mauritslamers/newbt/sc_config#L27)

```
BT.addTheme(BT.Theme.extend({
  ref: "sproutcore:ace",
  cssName: "ace",
  path: dirname() + "/themes/ace",
  dependencies: ["sproutcore:empty_theme"]
}));
```

Define this above the project, then also don't forget to set the theme to the app:

```
var my_app = BT.AppBuilder.create({
  name: 'my_app',
  theme: 'my_theme',
  title: 'My App',
  path: 'apps/my_app'
});
```


Also, do macs come with ruby by default?


Hi, if you think this is needed, should probably include the other dependencies (from the dependencies section) in the note: json.el (part of emacs as of version 23) and popup.el for inplace popup menus.

Can you update the PR?  Thanks


Happy to do that, but I opted to do it this way to mimic the way it is done in WrappedObject.php:
https://github.com/phpspec/phpspec/blob/master/src/PhpSpec/Wrapper/Subject/WrappedObject.php#L111

```
    public function beConstructedThrough($factoryMethod, array $arguments = array())
    {
        if (is_string($factoryMethod) &&
            false === strpos($factoryMethod, '::') &&
            method_exists($this->classname, $factoryMethod)
        ) {
            $factoryMethod = array($this->classname, $factoryMethod);
        }
```

What is preferred?


Isn't `fca_number` already an integer data type?


Possibly. I'm just playing with this for now. I see managed orders do a lot
more than OCA. When it's more mature, we can discuss whether and how you'll
want to pull it. BTW, do you plan sending the Tag on the order to brokers,
and getting it back? This is required for reconciliation. I see scenarios
when algorithm would need to recover from interruption. For that it would
also need recently filled and currently open orders. Since internal ids are
not persisted, need a tag to reconciliate.

On Tue, Jun 23, 2015 at 11:43 PM, Jared <notifications@github.com> wrote:

> Hey! Recommend creating a new class here (
> https://github.com/QuantConnect/Lean/tree/master/Common/Orders) with
> OneCancelsAllOrder(). This is definitely something the community would love
> as well!
>
> You could store the high and low order id's for the OCA in special class
> properties.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpavlenkov/Lean/commit/d772016df95710eee1b0f3b80572c43f6e7cd153#commitcomment-11830155>
> .
>


It looks like this could leak memory.

Are you sure you need to strdup() and then free() in this function?


The log message should be more details. How can you know what this revision does after some months?


"and built by Tutum Stack on a host at DigitalOcean"
This needs rewritten but I'm not exactly sure how. The Tutum stack describes the container configuration, layout, and relationships, and automatically downloads/starts the Docker containers in that configuration on the provided nodes (which can be a combination of VMs from any source - DigitalOcean in our case).


yes, it's the same line. don't know who wrote this originally.


Is this a good idea?


Can you tell me why you are setting the artifact type to the distribute field of the modules to install please ?
https://github.com/Talend/tcommon-studio-se/commit/25e17d291a11a766db1f4c3e5d9469af9771ad98#diff-230c685137adaf6ca7f0057b11d67db3R196


Thanks for figuring this out!  You describe it as excessive in the pull request, but it seems like it just has to be done, and it's too bad that we have such a cascade of dependencies.

I'll let @no-reply weigh in, but otherwise this looks good.


I think it is better to provide right now tests that test current behavior. And in the future adjust the behavior and the tests so a roundtrip test works (json->properties->json).

But perhaps @jleclanche thinks otherwise. This is his codebase and he knows better about where this should go.


@roxas75 Thanks for your explanation, i've know your plan.
However TOP_SCREEN and TOP_SCREEN2 are macros defined already, and these would be replaced by actual number while compiling, not running.. So the comparsion is wrong.
BTW is there any feature/typo tracker?


Like I say, yeah, very very low XD Whoops


Meteor's config files are in .meteor, and CircleCI's config file is circle.yml. Confidential information (passwords, etc.) are stored as environmental variables in CircleCI and aren't available in GitHub.


It is in the schema, but the `fca_number` variable here comes from the params, so it's a string — and Postgres will complain if you compare an integer column to a string.


Sure that this doesnt have to be

```js
$interval($rootScope.$applyAsync, 1000);
```

or
```js
$interval($rootScope.$applyAsync.bind(this), 1000);
```

Does $rootScope.$applyAsync() return a function? Or is  $rootScope.$applyAsync the function we want to schedule?


I have no idea about which styleguide this codebase follows, but previous TTK follows http://docs.translatehouse.org/projects/translate-toolkit/en/latest/developers/styleguide.html

@jleclanche Anything to add re styleguide?


Hello from FULL_TEST_CASE :p


Move the ssl thing to row 7.


Is this needed for this commit?


Containers. The actual Tutum stack consists of 1 haproxy container doing round robin load balancing to 2 foodwatch containers. The foodwatch containers share a mongo container. A New Relic container monitors all of the running containers.


Uh, 0.7.5 you mean ;)


in that case change the search as well :)


Sounds good, yeah


Yeah, but this is silly :) we should update the readme, and not just replace company names.


> Doesn't Lua panic if it runs out of memory? It might not be able allocate
> space for the string message.

No; it throws a lua error.

lua has the "not enough memory" error statically allocated so that it can't fail :)


Again this needed for the form? It looks like some of my initial mail stuff has been pulled in


What about the rest of stuff that is in nginx.conf? Like workers, worker processes, etc?


(The round robin load balancing is primarily so the automated redeploy doesn't cause any downtime)


No Firefucks given this day.


And run using the Meteor Velocity test runner.


@n9 what shouldn't be there?


Please replace the hardcoded path with ``config_get_config_directory()``.


```
'(Glueck|Reichtum[)]'
                             ^
```


Maybe update license not to mention "Shadab Ahmed".


I need this plugin for videojs player. If you have the sample code please share.


Hi @gioman

I have tested with empty layers parameter and it works. I have all the fixed legend with an empty map. And it is the purpose of this fixe, it's to always have the legend even if the layer are not in the layers parameter.


Maybe we should ``file_prepare_directory`` instead. This could be done outside of the foreach. Because ``$content_type`` won´t change.


ok. And maybe there are other places where it should be changed


Тыц


Should be calling from utils/blazed_orange(). :D


he is the original author https://github.com/shadabahmed/logstasher


We don't seem to change the rest, according to cloud_v2, right @hostmaster?


Yep, i know i messed up many comparisions in the code...
Feel free to improve the drawing library!

Nope, haven't had time to write anyting yet :P


Well, "after 0.7.4" == 0.7.5 ;-)


Ah right on


As long as lan_addr->str never can be NULL here, is that guaranteed? (I just stumbled upon this so haven't looked at any surrounding code.)


Well, I guess I count the development cycle to the next version and you to the previous one :P


The patch is at the wrong place. As with any XSS, the potential problem lies in the output generation. It's just a different case of encoding. If you just do `<a href="<?php echo $uri->getPath() ?>">link</a>`, then there is of course a security risk. So to avoid that you have to create a helper that actually returns a valid URI reference or use a router as symfony does.

The way, it has been fixed makes Diactoros unusable as a valid PSR-7 implementation. How would I now be able to redirect `http://example.org///foo` to `http://example.org/foo`? With this change people would be able to create as many duplicate content sites as they want by just prepending slashes to any page.
All those pages would resolve to the same page instead of either returning a 404 or be able to redirect. Good luck with your page rank:

- http://example.org//foo
- http://example.org///foo
- http://example.org////foo
- http://example.org/////foo


Maybe we can do a pull request?


Hi @rldhont ok, let me check it, I may have overlooked something. cheers!


Should we add new parameter here?
If yes, what should be the default value here?


This new list is very similar to `three_day_residency_requirement_applies`, but the latter has Spain in it. I'm not sure if this is coincidental or not.


Ok, I'll dup using the arena, but for the symbols I'll not be able to use arena since I use asprintf() to format the strings


Can the PGP signatures also be posted here:
 - https://qa.php.net/
 - https://downloads.php.net/~ab/
 - http://windows.php.net/qa/ *(no HTTPS if that makes a difference with PGP, but it does for hashes)*


auto_increment - я правильно понимаю, что он будет в результате сам создавать уникальные id?

Ну, наверно, на название элемента нужно ставить unique. Больше я пока ничего не придумала.

Да. Я уже подумала, что можно создать отдельные таблицы по Бактериям, Вирусам и Эукариотам с информацией по организмам, и эти таблицы будут уже связаны с общей таблицей со структурами


Hello from FULL_TEST_CASE :p


almost everything loses it's special meaning, yes.

Therefore I am also not a hundred percent sure that re.escaping would not break things by introducing backslashes where they don't belong


This fix could not ensure a FIFO waiting list


I'm rather sure it would break things ;)


Optional: It may be a nice idea to break a line before printing strerror msg. I hit some cases where the log msg was incredible long and look terrible because we already print a very long message.

+1 anyway.


The function `redirectToRoomLoc`, it contains application logic.


@hoholee12 & @DeicPro on freenode servers i have a channel for us:
#hybriddev

:) hop on!


@roxas75 Thanks for your permission. Would first try to read the code.


slack test


That is NOT the worst thing you can find inside the *presentation* folder ;)


@ralferlebach I take into account all input for version 1.2. I will work on new features at the beginning of september.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 14](http://build.prxm.it:8811/viewLog.html?buildId=3168&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Zie #64


@stringparser why was this done? I'm not seeing any corresponding discussion for this change. I'm curious because we were suppressing default gulp output that was prefixed with a timestamp by proxying console.log, which we can obviously no longer do since this change.


oh sry, was thinking about the stuff below.

What's this obsession with using filter() at once?

i can do it though if it makes you happy


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 14](http://build.prxm.it:8811/viewLog.html?buildId=3168&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 108 Build time: 00:01:29


Is there a reason you wrote it so that an option named `AbUsing` would match as well?
Why not simply `if ($key eq 'using') ?

Please clarify or simply adjust the PR


Native ESP8266 projects


You should also fix the Country name: Portgual -> Portugal


h


More tests from the beloved user:

*Installed as recommended* to the **C:\ drive and kept everything as default - Everything worked fine.**

**Changed saved download folder to E:\ - Items showed in Transfer list but gave error and would not download.**

*Changed saved download folder* to **C:\Users\Downloads - Everything worked fine.**


Update


I left a comment regarding this in the pull request.
I now think this change is good, however it doesn't fix it completely fix everything because, if one uses the hidden field it re-adds the record to the indexque in the RecordMonitor but I actually think this is a seperate issue with the RecordMonitor.


I am afraid I do not understand why do we need it. IFAIK a common practice is using something like
```
docker run --name vs-nginx -v /var/nginx/conf:/etc/nginx:ro -P -d nginx
```
and an "official" image


I'll try to fix it properly in the code.


If you're talking about the whitespace/tab-issue that this issue tried to fix, then no. It was this commit 0a24e3358a6c0a67434fe5731e7fb82a138889b7 that you made, that made it work for me.

The removing of ```gettext``` (a4557b87cd3c5801cd2133061c46dd0aa1077000) was only to get cleaner and more readable code, since it didn't do anything. The whitespaces are exactly the same as in the commit you did. (though I noted that github colorized the changes in that commit as if the whitespaces where changed.. but they're the same :)


thanks I'm slowly learning


ah I thought * == wildcard...


ok leave it as it is if you prefer it


I think the FILE_APPEND flag makes no sense because we write a single entity?


@ilmari Can you review the rest of this in terms of featureset sanity? E.g. it seems to me WHERE needs to be able to take an actual expression, instead of just `predicate`...?


Oops!


ah ok

2015-06-24 15:44 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> I'm rather sure it would break things ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/e1ab796383bc2c7293a668a10eb4e955c00eccff#commitcomment-11838852>
> .
>


Hello,

I have a suggestion for you. The export data to csv(with delimiter ; (UTF-8 format)) didn't work very well. I had to put this "\uFEFF " (BOM) as first line of data.


this indentation is simply too much and filter would reduce it


We usually write this as `lib/galaxy/datatypes/{data,triples}.py` .


Godoc


Fixed, thanks!


Yes - but that will conflict with Ira's change :).


I decided I was too tired and went to sleep. I guess we'll have to fix this and do another release?


dont look at surrounding code, look at this :
https://github.com/miniupnp/miniupnp/blob/master/miniupnpd/miniupnpdtypes.h#L22


We install OneTimeInitializer in /system/priv-app/OneTimeInitializer so we must remove /system/app/OneTimeInitializer


This should really be three separate commits:

* The removal of the `Client` modules
* The removal of the lwt.syntax stuff
* The API changes for `connect` and `connect_by_name`


from code review it seems all ok to me, are you still getting this issue? if yes do not care about it, i will take care of completing the unit test while integrating


Do you want to keep this commented?


Looks good to me!


Best kind of correctness. Though I think the correct term is extinguish, no?


This could do with splitting down to - it seems to be several unrelated changes.


Added PR for this:
https://github.com/TYPO3-Solr/ext-solr/pull/22


Hola Jesús,

ya he añadido el enlace a Heroku. Si hay alguna otra cosa con la que pueda ayudarte no dudes en ponerte en contacto.

Saludos.


Check


You can add this to `ApplicationController` which will be shared to other controllers to avoid duplication.


I have pushed a commit on the dev branch to solve this awkward issue. Please verify it using the 2014 Matlab version


@hostmaster
We are discussing different options.

Lets talk about this in a hangout. I'll ping you when AB is free.


Should pass `path`?


@justinedelson I was thinking about it. It's for sure worth consideration. I thought it may to broad and by looking into couple of projects, I did not find such case.


Let's use something like this instead of just 'sleep':

while [ "`ping -c 3 node-1 1>/dev/null && echo $?`" -eq "1" ]
do
    sleep 10
done


Should pass `filename`?


in this i create a php page and css page


> What is `'A'`? Oh, it is the initial file. That makes sense!


SRSLY?


@borisbstyle this copies the data used for the delta argument -  @ledvinap was talking about using it in place (use a pointer).


Ok, was just aesthetic.


Could you ever have 2 predecessors that both contain the same node?


realize insert uploading state into mysql


Alright, fair enough.
Mine was a bit of a ghetto fix anyway, so it's best if we improve the code.


`NUMBER_OF_TYPES` is a bit vague.  can you think of a better name?


podNodeName?


I begun enthusiastically. On the still to do list is a separate PR to fix all of these.


@eseidelGoogle might be nice if the story is 'pub run sky:init'. Until this tool is ported from Python to Dart, we could make sky:init launch this script.


The default to CEVNI is historical. When I started rendering notice marks, it was in response to the WSV import, so all were in the CEVNI region. Only later, when I added the BNIWR+PPWBC, did the system tag become necessary. We need to leave it that way as there are 5000+ CEVNI notice marks without the system tag. This is because the import was from S57 files, which code the region as global meta-data and not on each object.

Our editing tools should always include the system tag so that over time, these will propagate through the data.


We will PEP-8 the whole module at the GCC and just eliminate both lines :).


Hi @benogle,

do you mind sharing the rationale? I just spent an hour trying to figure out why my snippet with an explicit string selector wasn't working :smiley_cat: and language-coffee-script has a snippet for `"#{interpolation}"` which doesn't work so it had to hack around it somehow (I'm not quite sure how, couldn't find it in the source).


Due to the lossy saving of the old chunk, the next chunk wouldn't always properly be concatenated with the old one.  Consider the following chunks:

    "{...}\r"
    "\n{...}"

With the previous algorithm, these would be concatenated to `"{...}\r\n{...}"` and then sent straight to json_parse, but having `"\r\n"` between to objects in a json string is not valid json.

The new algorithm splits *after* concatenation by saving the excess from the previous chunk in its original form.


or to check ssh availability:
while [ " ` nc -w 1 node-1 22 | grep SSH 1>/dev/null && echo $? ` " -eq "1" ]
do
  sleep 10
done


In case that we successfully put `nx1` but failed to put `nx2`, `ba[][]` should not be updated.


Yes - I have rebased this to fix that. @jessevdam this does raise the point though that I have not re-tested these sniffer - if you could do that and let me know if they still work that would be great!


or to check ssh availability:
while [ "\` nc -w 1 node-1 22 | grep SSH 1>/dev/null && echo $?\`" -eq "1" ]
do
  sleep 10
done


Need to fix 'sleeps'


@drbassett The call to `unwrap()` would have unwrapped a `Result<()>` or a `Result<std::io:Error>`.

Surrounding the call with `try!(...)` will automatically `panic!` if the callee returned a `Result<std::io::Error>`.

Am I on the right track?


To avoid copying the `PTerm` data you want this

`PTerm = pt1_element(axis, &PTerm, pidProfile->pterm_cut_hz, PTERM);`

See http://www.csee.umbc.edu/~chang/cs313.s02/stack.shtml


:+1: :)


check the coding conventions, but generally speaking everything in bsp should be lowercase (except macros).


NIT: How about doing return `1` (meaning `true`)  if this method successfully set puyos?


a macro to help initialize this would be nice


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Launch Environment [Build 2](https://bsr.ict.je-labs.com/viewLog.html?buildId=29604&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_LaunchEnvironment) is now running


Hi @braintreeps, it would be great if you could say whether this change resolves a security vulnerability.

I'm applying security updates to the libraries of an application that I work on and I'd like to know whether I should apply this one. If there's no security content then I won't apply the update to reduce the risk of regressions and reduce the amount of testing I need to do.


I liked this name :D. NUMBER_OF_FILTERS is better name


Linda excusa para sumar commits, no?


By removing the pidof call you have introduced another problem because the fifo file still exists after the splash daemon has exit. This causes a ssh login or normal login on the console to stall because in there you call splash_exit() again which then will send an echo command to the fifo file again but will never return.


should consider useful error codes, and to distinguish the two cases of !send_read_buffer, and !buffer_succeeded


vector_3f it may be the ugliest name ever given to a type. :-)

Triple
A group of three elements, also called a triad. A triple is therefore a 3-tuple. Or whatever suggestion to s/vector_3f/<something nice>/ that one may have.


Which version of CMSSW is this in?


I would like to drop this exception from the documentation.  (Yes, I know.. :)


JavaScriptの新仕様（EcmaScript6）では、
```
var hoge = "aaa";
hoge + "bbb"
```
は
```
var hoge = "aaa";
`${hoge}bbb`
```
という感じで書けるみたい！


Improve comments about parameters


Is a lock going to go here?


improve comments about parameters


Thanks. I did not test, but from reading the code, this should do.


The issue with the wrong classification node prefix being appended to class nodes when converting to CitLab models is now fixed also.


I can't find anywhere in the standard that says this but isn't the space here non-portable?


Just as an FYI I have the markdown option turned on for jsdoc so you can use backticks around the dunders names instead of having to insert backslashes before every underscore.


nice!


this redirect also works with api/json endpoints?


I think local is a bashism?


Great, thanks!


This should `exit 1` right?


"$python" here


@hydra: Problem is when you want to filter integers - passing input by value and returning filtered value handles this automatically. With pointer you need function for float, int32_t and int16_t ...


I would write this:

```
If the database state type changes, http://hackage.haskell.org/package/safecopy is used to migrate deprecated existing serializations from disk transparently.

There should be a tests that make sure nobody breaks this rule, but currently there aren't.  Instead, the current "migration process" is simple: ...
```


There are three options in menu, but for now only the `Play vs AI` is implemented.


Oh, sleep sounds great, too bad it's only 4pm here...
I found out that this happens only for a project where I apply the plugin to the root project only. If I apply it to the subprojects, everything's fine.


-flto will mostly take care of this .. IMO readable code is important. It is possible to optimize it later if neccesary


can you also edit README.md?  there is some duplication in the documentation now that should be replaced by references to your document, which is more thorough and up-to-date than the REAMDE.


No need for explicit cast here, compiler must be clever enough to insert it automatically.


That is exactly why it was complicated first time as we have different pid controllers all using different types of PID variables :(.
So painfull sometimes.


who did write unbreabkable helm? @_@ should be garment.
and extra 0 for bFlee2.


This affects everything, just like `ensure_org_url_if_org_user` does, for example. I think the conditionals (`return unless current_user` and `unless current_user.enable_account_token.nil?`) should make it safe, since redirection can only happen if there's a current user who has a token. Only organization signups have those tokens at the moment.


should we move the installation instructions here?

or everything to the README?  READMEs tend to be quite extensive in some github projects.


ownerName? (here and below)


@mikepetersonccv We ran the new build this morning and couldn't access some block properties.  Just keeping this migration on your radar.


yeah.. its just past 7am here. I of course forgot to run tests before I pushed these changes so this is a mess. I'll have a fix shortly.


:+1: on this.


"Take a look at the configuration file `devel.config` if you want to change this."


HaHaHa still blowing bilbo chump


some nice irc client for android?


Sounds like more trouble than it's worth for a harmless warning that can be ignored.


Consider using a single call to appendf instead of all the calls to append


I was thinking more on if it will return a 400/403 for an api endpoint, assuming it renders some error page for admin endpoints.

I think is safe, indeed, is just about the error "presentation".


Thank you for providing the resource! :heart:


can't ... SuccessorEdges?


TestGit is a repository


isn't this a regression if you look at the reasons for #7?


pushing right to `develop` @matthewplunkett ?!  Don't you know how important first impressions are? :disappointed:


...  and maximum version: my rule is to set the lower bound to whatever is most recent at the time of creating the dependency, and the upper bound to `<a.(b+1)`, where `a.b.c...` is the lower bound.


Gollum wants you to take one in the butt.


It wouldn't surprise me if this is one we grabbed from the header of a page on interactivepython in the midst of a debugging session.
I'll update this today.


"check `git diff cabal.config` to see if everything went as you expected."


Do you need a space after the initial tab?


Okay, pushed a version that actually passes the tests =)

I'll be on the train with slow wifi, but be available.


@bbrodt could this be related to the build failures? We are still building with JDK 6.


@jfoug  Yes. Currently there are none that have less than 2 test vectors. I think **it would be better to have a check whether the size is less than 2**.


@Syphurith those macros contain dereference which can't be precalculated while expanding a macro.
TOP_SCREENx contains no scrren buffer address itself, but the pointer to the address. Expanded macro will contain the precalculated address at which different rxTools entry point will populate the actual buffer address, so this should work fine.


コメントの形式が//になっていますが、Javadocコメントとして扱われないので/** */にした方がいいですね。


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Launch Environment [Build 2](https://bsr.ict.je-labs.com/viewLog.html?buildId=29604&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_LaunchEnvironment) outcome was **SUCCESS**
Summary: Running Build time: 00:14:15


@DiamondBond the problem is... how to unzip from shell? xD


No problem.


:+1:

Looking at the API, it seems the function `SetViewNodeIDs` is already doing this ? May a convenience function named `SetViewNodeID` could be added ?


Can you PR this to Joel's branch?


Musicインスタンスの名前はlistじゃないほうがいいかも、musicかmでもいいかもね。


падать не будет
https://drive.google.com/file/d/0B-humNPiH1w8OEg1X0ZoUEhtaE0/view?usp=sharing


too the links on Releases section I can create a goo.gl link to short it...


Listが大文字になっているとクラスと間違う可能性があるので注意


Is containerNode guaranteed to be a DeploymentConfigNode?


Correct Working For me!!! Thanks


Good idea.


Why all these std::fabs?! That way a change from -1 to 1 is detected as a 0% change. That does not seem right to me.


Good catch, thanks


@zilkey How can I call filter and reduce on an HTMLCollection object?  I've run into this problem before and I always have to revert back to a for loop.  This is what I've been trying:
```
var queryString = Array.prototype.filter.call(checkmarks, function(e){
    if (e.checked === true){
      return e.name+'=true';
    }
  }).reduce(function(prev, curr){
    return prev+'&'+curr
  });
```


if according to Aegis, 18575 should be it
````
// ID: 18575
item YinYang_Earring
	event OnStartEquip:
		//...
	return
	event OnFinishEquip:
		//...
	return
````
then I'm worried since it's first implementation on 069a311e38b44cbc7c0be12e88442f9be53d8c0d.
well, I just need to revert it.


I tested this and it doesn't work. Probably because I am new at xml or csl, so maybe someone could help me fix this?


这个类库我初次使用了之后，查询正常插入中文字段到数据库一直出现乱码的问题，是不是就被你在这里解决了？


引数が４つのコンストラクタをうまく使うことで、引数３つのコンストラクタを楽に書くことができますよ。
Music(title,artistName,playTimes,null)　のような形でnullをうまく使うのもポイントです。


Why does this return a ServiceNode?


Woops, flipped the order of log severities in the enum but forgot to change this logic to match.


特に問題なさそう。強いて言えばJavadocコメントにするくらいかな


useless, set later in the code


You're right, opened #4189 for it since it's not critical for this PR.


> SuccessorEdges

Its a helper function that turns out to have very narrow utility.  I can't use it here, but the code I have looks very similar.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 15](http://build.prxm.it:8811/viewLog.html?buildId=3170&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


@DeicPro you dont need to with the direct files in the repo, then create a goo.gl like you said then just wget it lol :) or create a zip if a file dosnt download and use tar, maybe we have to add tar to required binaries ;-;


Noticed something funny today: if you are on a public hotspot that has a landing-page (to login or accept the terms), the whole HTML of the hotspot landing page will be returned as the 'IP address'.

Sidenote: Will update Glitter to handle the -1


This change is for unit test. Although atom value is harmless on generating app.config and vm.args, cuttlefish_unit fails on a config validation phase like this:

```
% ./rebar eunit skip_deps=true suites=stanchion_config_test
==> rel (eunit)
==> stanchion (eunit)
stanchion_config_test: default_config_test...*failed*
in function cuttlefish_unit:assert_valid_config/1 (src/cuttlefish_unit.erl, line 60)
in call from cuttlefish_unit:assert_config/3 (src/cuttlefish_unit.erl, line 69)
in call from stanchion_config_test:default_config_test/0 (test/stanchion_config_test.erl, line 8)
**exit:{assert_valid_config_failed,
    [{phase,transform_datatypes},
     {errorlist,
         [{error,{transform_type,"erlang.schedulers.compaction_of_load"}},
          {error,{type,{false,string}}}]}]}
```


Rabix included an extension of draft1 anyway. Luckily, most of it was semantically same as draft2 so we'll be able to make a converter to draft2.

+1 on dropping support for draft1.


`surfaces.uniq {|surf| surf}` => `surfaces.uniq`


assertThatでもいいし、assertNullでもOKです。楽な方で。


well, you *could* have noticed that the layout is pretty f*cked up here :D


So the comment should say "can't **use** SuccessorEdges"?


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 15](http://build.prxm.it:8811/viewLog.html?buildId=3170&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 108 Build time: 00:01:19


nothing obviously wrong. What's not working? How are you testing?


This is some complex routine for capturing the remote output. I was able to follow the logic, but I bet this was tricky to get right.


In general should the vars be called fooID or fooName?


@shellscape see gulpjs/gulp-util/pull/92, it was done to be able to use formatting with gulp-util. i.e. `console.log('%s', 'something')`. You can still bypass the timestamp bypassing `process.stdout.write`. Although for me its ok if this is reverted.


could this test be combined with line 123


This is a major update, do you think it's time for increasing the release number?
On the mailing list are already reports about incompatibilities. You could say: "Since 2.15 you may need --all-cdecl".


typo pythong


I tried to import it into Zotero and it told me that it was not a valid style file.


Rock.Rest doesn't compile, familyGroupTypeRoles on L411 should be "familyGroupType.Roles"

https://github.com/SparkDevNetwork/Rock/blob/ae1c0a421b19b6d4aa25a18c87f68317edc0c5e7/Rock.Rest/Controllers/PeopleController.Partial.cs#L411


Visual studio tells me that socket_IO is excluded from build.

virtual_mac gets used for Bridged mode (which doesn't seem to work for me either), socket_IO defined it but from the looks of the code, never used it.


OK, a safe restore of the original stdout/stderr streams.


Can you point to a commit that does that? I think it's the other way around: The try would catch exceptions from newInstanceWithoutConstructor(). The reason for the try/catch was that until PHP5.6 newInstanceWithoutConstructor() threw an exception for internal classes (see http://php.net/manual/en/reflectionclass.newinstancewithoutconstructor.php  )


Sloppy. What about the heading? definition in the table above? What about czech version?


It seems this and previous function could be written more dense.
Of course this is a minor observation, so the final decision is up to you.
```
fun getPackagePart(callable: ProtoBuf.Callable): Name? =
    callable.getExtension(JvmProtoBuf.implClassName)?.let { packageData.getNameResolver().getName(it) }
```


為什麼Nickel要翻譯成亞鐵....


Alright, anything left for 0.11.3?


OK with dropping all the `quiet` handling.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 16](http://build.prxm.it:8811/viewLog.html?buildId=3172&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


the problem is that files uploaded to repo with text can't be downloaded (or not know the parameters of curl to can)... for that i upload in Releases...


Do you want to test that the nodes and edges you were expecting were created?


yes. i cloned your branch arduino-upload and tried.

Initially i tried with ubuntu 14.10. Having issues with arduino path. I will check and get back.

It works in windows after couple of changes.
i have updated the actual path where i have installed my arduino in the py file.
Arduino 1.6.5 is not working. Its look like arduino issue and i tried with 1.6.1 works. Here the twist is i need to switch to arduino code editor mode to upload otherwise the code flow to the webserver as empty both in firefox and chrome.


I guess someone is a German speaker here, right? :P


ここって、Objectクラスを使うように習ったの？
もしそうだとしてもObjectクラスで各インスタンスに処理をさせるのはバグの温床になるのでやめたほうがいいかも。
継承でもinterfaceでも良いので使用することを検討してください。


你在亞鐵什麼啦…


OK.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 16](http://build.prxm.it:8811/viewLog.html?buildId=3172&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Tests passed: 108; exit code 1 (new) Build time: 00:01:36


Ok, fair.


Gracias por el enlace.

Por mi parte iré preparando un README.md para el mío para el proximo Push que haga :octocat:

A ver si saco un rato para seguir avanzando.


https://zh.wikipedia.org/wiki/Nickel
原本的翻譯是對的...


Maybe mention that it also adds the RCSpec and down?


<html>
<head>
</head>
<body>
<b>欢迎</b>
</body>
</html>


I think we should revert this change (my mistake) because this is a protected field in a base class; if someone has extended this class and used methods from the ArrayList class (like get(i)) that code will no longer work after this change.


Yes it is. Actually it is closed in the destructor when building against python 2.7. However, when bulding against python 3+ the file was not properly closed and I assume closeit flag was added to fix this issue, but the way it's done now introduces another issue (double file closing) when building against python 2.7.


Not in the long run - but fixing it needs some thought about the class structure and I'm about to leave for Europe.


This is actually used in the previous commit, right? Nitpick: revert the order of the commits.


Yes, there's definitely a bug in BlocklyDuino - it only generates the Arduino code when you switch to that tab. I was intending to look into that next.

You said "it works in Windows with a couple of changes" - what changes? And what happens when you try with 1.6.5?


deployapi


Seems to work smoothly on 2014.

One question: You've only made the default settings for the gantry and
couch angles now. Not for the bixel width and the SAD. Why? Do you think it
is not necessary because it is just one number not an array?

On Wed, Jun 24, 2015 at 3:59 PM, becker89 <notifications@github.com> wrote:

> I have pushed a commit on the dev branch to solve this awkward issue.
> Please verify it using the 2014 Matlab version
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/e0404/matRad/commit/9e33c77e2e74549da3d497a250b3f57aa850eb6f#commitcomment-11839204>
> .
>


Credit to NightSpark


I think it'd be better to set up a DC with a real RC


well it's not nice but it's ok.


reack


@frankoid this is to protect against a potential issue with forged webhooks. If your application does not use webhooks then there is no need to apply it. You could also add the regex to your own code and skip updating the library if you do use webhooks.


well upload in fileserver as a .zip xD or remove the extension and then download then rename it by running mv oldname newname lol


What did this break? I mean doesn't this just make it so that you get `libwebsockets.so" instead of `libwebsockets_shared.so`?


the name of the variable is ```publisher-place```
Using the validator would give you this:
http://validator.citationstyles.org/?url=https%3A%2F%2Fraw.githubusercontent.com%2Fcitation-style-language%2Fstyles%2F2fafeab5b99419107f727b5bd391a6d2826cad39%2Fus-geological-survey.csl&version=1.0.1
which may have been helpful?


GUYS!! this is braking change... this should change major version :(


@shellscape you could also change the default gulp output removing it's listeners and using yours which would be less hacky (see here the original logging code [here](https://github.com/gulpjs/gulp/blob/master/bin/gulp.js#L173))

```js
var gulp = require('gulp');
gulp.removeAllListeners(); // now there is no listeners => no logging

gulp.on('task_start', function(ev){
   // do something
});
```

As far as I'm concerned gulp only uses events for logging.


See http://validator.citationstyles.org/?url=https%3A%2F%2Fraw.githubusercontent.com%2Fcitation-style-language%2Fstyles%2F2fafeab5b99419107f727b5bd391a6d2826cad39%2Fus-geological-survey.csl&version=1.0.1

("place" is not a CSL variable)


TeamCity Jarvis Framework :: CI [Build 83](http://build.prxm.it:8811/viewLog.html?buildId=3173&buildTypeId=CqrsFramework_Ci) is now running


TeamCity Jarvis Framework :: CI [Build 83](http://build.prxm.it:8811/viewLog.html?buildId=3173&buildTypeId=CqrsFramework_Ci) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


For future reference, you can use a triple single quote here (''') to construct a multi-line string without having to escape double quotes


ack


See: https://github.com/TomBZombie/Dice/pull/64#issuecomment-114479781

Daniel's fix works, but there's no real reason for the elseif here. By removing the elseif it ensures that count($parameters) == count($paramInfo). This means that dice will always supply an argument (even if it's the default) for every single constructor argument. Optional parameters were the cause of the related argument swapping bugs, by ensuring each argument gets assigned a value it prevents these sort of issues.

I don't think having the elseif there will cause a problem but there may still be an edge case where the behaviour is undesirable (If there's a default value that's not null if allowsNull() returns false and there is a default value set, perhaps). I'd rather err on the side of caution and just play it safe by ensuring all arguments get a value in all cases as it's less likely to cause these edge-case bugs.


Looks great, but I can't see if I'm already connected with a channel. Before these changes I could see that, cause there was no connect button.


1) i have updated the actual path where i have installed my arduino in the py file. this can be avoid when we have env variable and providing doc to set the arduino path to that.
2)There is no issues with the py server and it works exactly same as the version 1.6.1. Once uploaded the change is not reflected in my device.


See #221 ...


Hi @gordonwoodhull , will this change override custom output formats? Since upgrading all rmarkdown cells seem to be run as html_fragment (c.f. rcloud.iip)


As far as I understand it, everything is a folder, from XNAT point of view ;)
If you want to add a new folder via the webinterface, you have to

1. specify the level, e.g. resource, which is a folder called "Resources" == ctkXnatResourceFolder
2. and the name of the new folder you want to add, e.g. "data" == ctkXnatResource

The second one contains the actual data files.
So adding a new resource folder means that you also have to create a folder called "Resources" as well.
This is done automatically by XNAT, but in order to display this in the treeview as well, we need the class ctkXnatResourceFolder


removing extension not help xD...


Nope, lets cross our fingers and release =)


OK.


Awesome!  I should have known.  I was looking at this for help: http://aurimasv.github.io/z2csl/typeMap.xml#map-thesis

It lists 'publisher-place' as the variable.


dfdf


He didn't accept any pull requests for couple of months https://github.com/shadabahmed/logstasher/pulls.


No it doesn't...
I will look at that.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 17](http://build.prxm.it:8811/viewLog.html?buildId=3175&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 17](http://build.prxm.it:8811/viewLog.html?buildId=3175&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:02


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 18](http://build.prxm.it:8811/viewLog.html?buildId=3177&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 18](http://build.prxm.it:8811/viewLog.html?buildId=3177&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:02


No Problem! I did a pull request for the repository. Hope it is ok like
this!

Am Mit, 24. Jun, 2015 um 1:03 schrieb Luca Ciavatta
<notifications@github.com>:
> THX.
> Could you add it? Because now I'm on holiday and away from PC.
> On 23 Jun 2015 8:16 pm, "Steindi01" <notifications@github.com> wrote:
>
> > Screenshot for generating Gnome desktop entry with Android Studio
> > [image: menu_001]
> >
> <https://cloud.githubusercontent.com/assets/803596/8313659/808371a0-19e4-11e5-8d83-f43220d22648.png>
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/lucaciavatta/android-studio-fedora/commit/810e4cbec1c3d63e3a664d0e0a99530ec7241305#commitcomment-11822672>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


new


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 19](http://build.prxm.it:8811/viewLog.html?buildId=3179&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


No worries, I'll fix up the comment.


Typo `as a`


yea, also from profile it looks like he's very passive https://github.com/shadabahmed


totally agree @visnup !


ack


Oh, good point.  I'll go ahead and do that.


Mr Rockers can you email me @ kylesnyder31@gmail.com


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 19](http://build.prxm.it:8811/viewLog.html?buildId=3179&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Tests passed: 108; exit code 1 Build time: 00:01:19


Aren't we doing 50 on these indexes?


Thank you for the report. I will try to fix the bug.


AndroIRC :) connect to freenode then do /join #hybriddev


@DeicPro look at new note commit :P


Also, i have noticed one chrome app / extension "BlocklyDuino Editor 1.4.3" is it from github same user ?

It has more features "https://chrome.google.com/webstore/detail/blocklyduino-editor/ohncgafccgdbigbbikgkfbkiebahihmb?utm_source=chrome-app-launcher-info-dialog"


in your deploy directory on the VM::
(just to further emphasize the location)


Right. CSL styles don't know anything about Zotero, so you'll need to use the CSL fields, not the Zotero fields.


ACK


I don't think there's any need for an environment variable - users can either add the Arduino directory to the path, or specify the binary using the --command argument.

I've noticed some issues with uploading too - the first time I upload, I have to reset the device before it starts responding. Have you tried the same?

I don't know whose the Chrome app is.


I was going to suggest that this could be left to the `remoter`, with no action from the `Test` class, *but*, this requires the `remoter` to have knowledge about the current `Test`, which is the kind of difficult situation we felt on the GDB code.

This is definitely a cleaner approach.


OK.


Thanks @dougmet, you're right, it should not override custom formats. Opened #1553


ack in spite of german stuff


ack


Include all the authors https://github.com/reevoo/logstasher/pull/3/files


Thanks for the feedback. Will fix it later.


I'm not sure how this fixes it, but it apparently does.


so one could just use `collet_dirs` here, right? Despite there's no `i` version of it... so I'm ok with this, just mentioning


reack


unneeded change here :)


@catajoe ooooo eliminaa sto cazzo di file


Sorry, shouldn't have looked at this at 3 am >:(


You can use constraints to validate things like email address automatically http://grails.github.io/grails-doc/2.4.x/ref/Constraints/Usage.html


that was needed before because you indented that stuff further but isn't now


its quite slow when uploading first time but i didnt face issues with that. I have tested with a cheep arduino and the original. It works. It could be due to crystal oscillator timer circuit and i am not sure.


LGTM


ok


It seems `toList()` could be omitted here.


as @dangra says, because we have a non-standard Elasticsearch setup (and not using docker links), we need to have configurable Elasticsearch URL (IP address and TCP port)


@stringparser ah, thanks for the additional info. very helpful. we originally went the route of removing all of the listeners, but found that there were situations (debugging, etc) that we wanted to show the gulp output along side our custom output.

she 'aint pretty, but this is how we're suppressing at the moment:

```javascript
  if (swig.env === 'development' && !swig.argv.poolparty) {
    var oldLog = console.log,
      oldWrite = process.stdout.write,
      suppress = false;

    // since this commit (https://github.com/gulpjs/gulp-util/commit/6f6e7c2947ccb59d71d3a680e3a76683289a0548)
    // gulp uses stdout for the time, and console.log for the rest.
    process.stdout.write = function () {
      var args = Array.prototype.slice.call(arguments);

      suppress = false;

      if (/^\[\d\d\:\d\d:\d\d\]/.test(strip(args[0]))) {
        suppress = true;
      }
      else {
        oldWrite.apply(process.stdout, args);
      }
    }

    // gulp-util.log always starts each log with [00:00:00]
    // we're suppressing that output if on a dev machine.
    console.log = function () {
      var args = Array.prototype.slice.call(arguments);

      if (suppress) {

        // if gulp is reporting an error, let that through.
        _.each(args, function (arg) {
          if (arg && (!_.isString(arg) ||
              arg.toLowerCase().indexOf('error') > -1) ||
              /Task \'(.+)\' is not in your gulpfile/.test(arg)) {
            suppress = false;
          }
        });

        if (suppress) {
          suppress = false;
          return;
        }
      }

      oldLog.apply(console, args);
    }
  }
```


tbh. I think it is better to change this, just in case of further changes to globs or collectors or so


Now a attempt to instantiate class Person will result in this:
![abstract](https://cloud.githubusercontent.com/assets/3900509/8332939/99915b42-1a5e-11e5-94a3-c80478bf699e.png)


Do these methods need to be virtual ?   Does anything derive from this class?


uhm no, actually you want to simpliofy this: `if fnmatch(self.path, abs_file_pattern)` and remove this whole is_applicable thing. That was only needed because fnmatch was missing


Why don't you override  QWidget::wheelEvent() instead of eventFilter and install? It'll make the code simpler.


Seems Ok to me.


Sorry, I grabbed 2.2.1.  2.2.2 looks great. Thanks for the quick response!


I totally agree – we should turn this into a style guide...


We shouldn't mix K&R style bracket placement with non-K&R


I don't understand where you see duplication. Coveralls and Scrutinizer don't provide the same service.
They just use the in part the same data to provide a part of their service.


Ребята (Майя, Егор) настораживает большой удаленный блок,
в связи с нашей пока не опытностью, проверьте все ли изменения попали в расширения


Outdated comment.


I'd check the return value here


Credits to @bakaxyf @IamAchang @UUUii (so far.)


this commit is weird. And this function seems similar to a similarly named one above...


no ok, not doing it for the `i` ;)


You probably want to set your email address properly in git so that your commits get annotated with correct author information. https://git-scm.com/book/en/v2/Getting-Started-First-Time-Git-Setup


>1 addition and 22 deletions

 :+1:


@nicolasThal


well in this case you don't need that pattern as a parameter at all do you?


Other than that, and if check_style is ok, LGTM.


弾の動くスピード


interesting, oh well see if i can use the code to randomize the MAC in TAP if i can (im not even sure how to enable TAP to check it) unless TAP has a function to get your real MAC address. Ill get it to update the virtual mac for bridged mode as well then.


commit message is bad. Really bad.


This is checked during the deserialization process, wouldn't this change break backward compatibility when deserializing bitmap generated from previous versions?


Always a great support ... Thanks a lot guys. :+1:


Fine, but I'd like to emphasize the double type. triple-f? f+?


Makman2 dislikes backslashes and I see how your and trick is useful. We need to decide wether we want to allow this and thing globally or not.


Apparently yes it does but it may not be updated? That's my take based on
this thread I found (that I only quickly skimmed):

http://stackoverflow.com/questions/3474875/why-does-mac-os-x-come-with-ruby-rails

On Wed, Jun 24, 2015 at 9:29 AM, Susan Buck <notifications@github.com>
wrote:

> Also, do macs come with ruby by default?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/susanBuck/notes/commit/08aa74b95e523d70f107c1dcaf78f1686e0a0833#commitcomment-11838510>
> .
>


弾が誰の弾か判断するためのフラグ
trueの場合，敵
falseの場合，自分


Two explode are needed??

```php
$aux = explode('/', $track->getFramerate());
$num_frames = intval($track->getDuration() * intval($aux[0]) / intval($aux[1]));
```


New line here.


Many BibCheck rules modify a subfield.
They have various boundary conditions.
Either we copy-and-paste code to several plugins or use a very generic interface like this to build on.

Should amend_field be able to change the subfield code?
This could be used instead of replace record_modify_subfield.


Nice Tests!


` (If user is an admin)` is not exactly correct, stdout and stderr are added also for non-admin users. Remove?


Yes, you're right; I will modify it for MRAA and compare the execution time with both approaches. Good point there!


Gandalf Loves Bilbo's bag ins his face


Configure your QtCreator to automatically add new lines at the end of files. There's a settings for that.


1.0.2 was never published to npm.


Sorry haven't got to South etc yet but what changes would have to be copied back in order to ensure migration.
Also again I think.. generate the migration in the deployment directory "on the VM" and an additional clarification at the end ie back to the development directory "on you laptop. This will ensure that those migration details are included in your git repo ready for submission."


can't you add all your little new functions in one commit and use them in another to split this one up a bit?


Make this self and it will be instance-by-instance configurable.


Don't need a new line if there's nothing to go in the code block.


Thank you, now it works.


agree: Presets and tools should add the system tag.
but: Having a JOSM style-file, displaying a different view than online-map makes no sense to me.


Hope this is not inappropriate. This is a definite improvement.


"continue" should be indented on a new line


a


I believe tap has its own MAC address.


aaaaa


Author last name actually doesn't have to be exact, but it matches backwards, so ```dicke``` doesn't match ```dickens``` but ```ckens``` does:
```ckens```: https://librivox.org/api/feed/audiobooks/author/%5Eckens?&format=json
```dicke```: https://librivox.org/api/feed/audiobooks/author/%5Edicke?&format=json


geomach


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 20](http://build.prxm.it:8811/viewLog.html?buildId=3182&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Here you need to clamp to [0.0f,1.0f] for luma and [-0.5f,0.5f] for chroma.


This needs to be higher than the maximum valid value. 100.0f should be fine.


And here you check for that 100.0f mentioned above.


Again, the clamping.


Plain double is probably enough. Depending on how you compile it, I think "long double" is sometimes just double anyway.


コンストラクタ
x，弾のx座標
y，弾のy座標
size，弾の大きさ（半径）
enemyFlag，誰が発射した弾か判断するためのフラグ


Good suggestion, done.  The `ArrayBufferView` fix-up, too.


         <item name="menu_trash_icon">@drawable/ic_menu_trash_holo_light</item>
         <item name="menu_selectall_icon">@drawable/ic_menu_selectall_holo_light</item>
         <item name="menu_split_icon">@drawable/ic_menu_split_holo_light</item>
 @@ -134,7 +134,7 @@
         <item name="conversation_send_button_push">@drawable/ic_send_push</item>
         <item name="conversation_send_button_sms_secure">@drawable/ic_send_sms_secure</item>
         <item name="conversation_send_button_sms_insecure">@drawable/ic_send_sms_insecure_dark</item>
-        <item name="conversation_delivered_indicator">@drawable/ic_sms_mms_delivered_dark</item>
+        <item name="conversation_delivered_indicator">@drawable/ic_check_white_18dp</item>
         <item name="conversation_emoji_toggle">@drawable/ic_emoji_light</item>
         <item name="conversation_keyboard_toggle">@drawable/ic_ime_light</item>

 @@ -145,7 +145,7 @@
         <item name="menu_call_icon">@drawable/ic_menu_call_holo_dark</item>
         <item name="menu_unlock_icon">@drawable/ic_menu_unlock_holo_dark</item>
         <item name="menu_lock_icon">@drawable/ic_menu_lock_holo_dark</item>
-        <item name="menu_lock_icon_small">@drawable/ic_menu_lock_small_holo_dark</item>


This is only needed for integer formats with 9..16 bits per sample. The limits for float formats only depend on which channel you're processing. That's all the information you need to pass to the functions.


yea, i'm getting the error now as well :-(
probably the result of some late-night refactoring.

anyhow, once it's fixed (using `(do)`), it *does* create directories for me as expected.


Added the lint comment back.


親クラス（moveObject）のコンストラクタに引数（x，y，size）を渡す


@HmHmmHm 글쎄 그다지 보고 싶지는 않은데요


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 20](http://build.prxm.it:8811/viewLog.html?buildId=3182&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 108 Build time: 00:01:39


I understand this as a question of formal correctness, not as an inevitable requirement.
Shouldn't OSM (and such JOSM) be as easy as possible, thus not hide information which could be rendered, just because a tag is missing, which could be assumed properly?
So everything what can be displayed in JOSM should not be hidden because of formal aspects.


@Thog Hopefully the modid has been reverted properly in 70256de0bcec7414e2e4892a6fcbfa3fc278f94b


FYI, rather than using the old Java Date, Java 8 introduced the new LocalDate object.  Here's a good article discussing: http://www.journaldev.com/2800/java-8-date-time-api-example-tutorial-localdate-instant-localdatetime-parse-and-format.


Do you mean something like this ?
void TabWidget::wheelEvent(QWheelEvent * event)
 {
     if(event->type() == QEvent::Wheel)
         event->ignore();
 }
It doesn't do what we want. My tests only worked when we ignore the event from the QTabBar itself. I guess making a new TabBar class and overriding TabBar::wheelevent(QWheelEvent*) would work. We'll have to override QTabWidget to call protected in old qt version method setTablBar(XXX). Not sure that would make the code simpler.


弾の一連の処理を行う関数
弾を動かし，描画して，
画面外に消えたかの情報を返す


Генерира ги автоматично, не ми пречат, ако треба ще ги махна


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 2](https://bsr.ict.je-labs.com/viewLog.html?buildId=29612&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) is now running


This was how we got the app to stop showing that annoying Rails welcome message.


EL cambio en la linea 125 a la nueva 97 se hace porque se incluira el archivo .htaccess para quitar el index.php de la url


void ли да го направа като другите.


I suggest to define and use everywhere our common notation for template arguments naming, since it's becoming harder and harder to read the code :( Even in this file, for the implementation of one class and its alias used `E`, `ENTRY` and `T_ENTRY`.


弾を動かす関数
敵の弾の場合，y軸に+方向で動く
自機の弾の場合，y軸に-方向で動く


Don't even know what these are...


We can remove 'ChiliProject' from the list if it is no longer being maintained.

Can we also keep the Demos/Source Code links at the end of the line.

Other than that this pull request adds a lot, so thanks!


弾を描画する関数
敵の弾の場合，敵の弾の色に
自機の弾の場合，自機の弾の色に
塗りつぶすようfillの設定を行う


I'm working on improving the architecture (like the name of the branch is already telling you).
I'll start to wrap what I can (and makes sense) in constructor and determining what's *public* and *private* for each.

But it seems it doesn't behave like we talk about this morning (see in-line commit).


この弾が画面内に存在する場合にはtrueを
画面外に存在する場合にはfalseを返す


ако нивото на достъп му отговаря просто връща. Иначе искаш role.equals(user.getRole()) .


Seems this line is causing problems: whenever it's included, ending up with a ```neko(9488,0x7fff71e8a310) malloc: *** error for object 0x102814200: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug``` error.


削除忘れ
削除して構わない


no need to call getViewType twice


Because of the closure, this function (and the next ones) is private (or kind of). That's fine.


FILTER_COUNT is better and more consistent with the codebase.


just leaving this here.. http://blog.linuxmint.com/?p=2703#comment-112806


The only case I can think of which is relatively common is if you are creating a custom OSGi Web Console plugin. For example https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/bundle/src/main/java/com/adobe/acs/commons/wcm/impl/WCMInboxWebConsolePlugin.java which extends `HttpServlet`


This could be `BOOTSTRAP_CFGFILE` from client_utilities


I would do that, but I'd really like to move away from using the global state pointer in general. I had it in there originally because `emscripten` was giving me trouble with passing around the pointer.

Ideally, the API would be consumed by using the `newGameState`, `drop`, `bestMoveForState`, etc.


TeamCity Jarvis Framework :: CI [Build 84](http://build.prxm.it:8811/viewLog.html?buildId=3183&buildTypeId=CqrsFramework_Ci) is now running


lollz!


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 2](https://bsr.ict.je-labs.com/viewLog.html?buildId=29612&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) outcome was **SUCCESS**
Summary: Running Build time: 00:05:54


Great, all good. Sry.


TeamCity Jarvis Framework :: CI [Build 84](http://build.prxm.it:8811/viewLog.html?buildId=3183&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 119 Build time: 00:01:04


Hi! Good work, could you help me adapting the library in Android? See my Github.


Und Â, Á, À , usw? ^^


This line is in bad style imo. It should be something like
`return std::abs((_postprocessor - _postprocessor_old)/_postprocessor);`


Thanks for your reply, I'll try to work something out myself :)


w


@jheer, why have all the log.debug statements been removed? Those have been crucial for debugging, so far.


We accept pull requests but arguably we may not even want std::abs at all
per Daniel's comment.

On Wed, Jun 24, 2015 at 10:14 AM Yaqi <notifications@github.com> wrote:

> This line is in bad style imo. It should be something like
> return std::abs((_postprocessor - _postprocessor_old)/_postprocessor);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/permcody/moose/commit/1d9e9623a522b526254ade260c1298dcf1ed6ec6#commitcomment-11840963>
> .
>


Spelling issue.


Thanks! Actually I wasn't sure about the number here; confirmed and changed!


type**s**


Links


Может здесь в названии написать: "Узнайте о потоке выполнения" или "Изучите поток выполнения"


I think this one calls for a little more care. It seems to work as designed (although I would've preferred an impementation based on vector<>.) However, I'd really like to see multidimensional arrays, and then it becomes more tricky. I'll think about it and get back to you.


Can I get the name changes separate from the content changes? All of the content changes seem difficult to follow and I don't know why all the sizes are changing.


well, `relative_recursive_glob` is one choice for `relative_glob_function`, which always gets these two parameters. (see new line 350)


It's a pity that the background image doesn't tile... a task for an artist to look into, perhaps?


yeah, this commit is just unreviewable :(

2015-06-24 17:19 GMT+02:00 Fabian Neuschmidt <notifications@github.com>:

> well, relative_recursive_glob is one choice for relative_glob_function,
> which always gets these two parameters. (see new line 350)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/eaddb5e0c8495bf8eade6da574e630a9427d6fc7#commitcomment-11841080>
> .
>


I guess it isn't clear that I used the actual board dimensions and kept a nice strict simple ratio between inches of the physical board and number of pixels. Perhaps I should add that to the comments?


@64FF00 이미 해결됐네요


And this one is public because of the `this`.
But it seems I can't wrap it in a `return {}` (I have an error with grunt when doing so) to make it *more obvious* that's a public method.
It only works when returning the anonymous function alone
``` javascript
return {
  function() {
    animate(step);
  }
};
```
With this I can use the function, but don't seem optimal to use when returning a list of function.


@sharpnk  Correct. But since we changed the file format and removed support from the old format, it is better to have an early failure that you can diagnosed rapidly instead of loading up corrupted data... This way you do not get garbage, you just get an exception.

As I explained in my email, we made this change at the time expecting that nobody was using Roaring in production yet...

This has changed now and we would definitively not break backward compatibility in such a manner.


As written, you must run build_app twice. Either that or we create the maintenance problem of having the same list of files in two places.

Note that the manufacturing extension has _three_ manifest files:

- `foundation-database/frozen_manifest.js`
- `foundation-database/manifest.js`
- `database/manifest.js`

and it's unclear (at least to me) what some of the tables will look like in various scenarios. For example, is there a way to generate a database with xtmfg that's missing the[ booitem_opntype_id](https://github.com/xtuple/private-extensions/blob/4_10_x/source/manufacturing/foundation-database/xtmfg/tables/booitem.sql#L2) column?


wow. thank you!


Mistake, the code is actually working 100%. I just accidentally made an edit that fixed it before I saved it.


Add '\n' after `types:` pls.


is_array will throw a warning if $display_settings is not set :( i'd be nice if they were consistent, right?


:+1: These look fine. Let me test them in a bit and then I'll merge this in.


@Amamatthew 这个要看的你文件和数据库编码的配合。我的PHP文件是UTF-8无BOM编码，数据库的校对规则是utf8_general_ci，字符集是UTF8。

考虑到许多含有中文的PHP文件已经是UTF-8无BOM编码，这里就不能再经历一次encode了。


"Большую часть человеческих рассуждений можно классифицировать по правилам Булевой алгебры. К счастью для нас, компьютеры \"думают\" так же! Давайте научимся инструктировать компьютеры с помощью \"If\" и \"Else\" выражений."


The difference between `BufEnter` and `BufWinEnter` is subtle. When would you expect one to be fired but not the other? My understanding is that `BufEnter` will fire whenever `BufWinEnter` fires, but at a different time (ie. `BufWinEnter` will fire after any modeline has been read).

Given the subtlety here, it would be nice if the commit message explained the rationale and effect of the change.


Set the mantis main menu link to codevtt to the codevtt-URL given in config.ini instead of the hardcoded relative path


A humble compilation of day to day bash scripts.
Use at your own risk.


@apsallid Hi unfortunately I could not find your mail address to contact you, so I try it through this comment.
Currently I'm working on this file, do you know what the original idea behind this FIXME was, I would then re-write the class.

Thanks


don't we have the `path_list` function for that in `Setting.py` I think?


`s/GSCARendererDelegate/GSCARendererDelegate`


`s/:self/: self/`


Normally I am all for minimal text and maximal images and I also agree that we don't need to go over each bullet point.  However, I disagree that removing bullet points will help. Since these slides are also online, we need them to be fairly self contained and contain some text. Its just that when we talk through the slides, we can skip a lot of stuff (but making sure that the students know why!)


We'll need to verify that I made the correct suggestion here.


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


FIXME: xfree (msig.signature);


missing a space


I meant `copy(self.configuration)`, I don't think this runs


@BakaRei 嗯我造，我之前就用 gravatar 的，但是因为被墙的缘故，每次 blog 打开了头像还卡在那里加载不出来，所以我就保存到本地了


agreed, I ended up using another solution, and I only use bestMoveForState function, the rest is already in the game play anyway :)


hi there


@quantumpacket These aren't actual releases though. Although we probably should eventually post the signatures for those too, my priority is to get the actual releases in a good state.


Then that's why I haven't updated TAP and probably don't need to :P


Okay, my bad, it was just a matter of syntax. Had to test some before finding the right one.
And when in front of the right one it's so obvious I'm thinking "why didn't I see that before?".


+1 I agree!! I felt the same last week too!


Поменял концовку: "До этого момента вы работали в основном с числами и строками. Теперь мы перейдем к более сложным структурам данных: массивам - \"Arrays\" и объектам - \"Objects\"."


Nevermind. I didn't notive that it had a default in the function params. That takes care of that.


Possibly. However, this quickly becomes complicated (in time and effort) to manage two slide decks for *each* workshop...


Added? :)


Lambda


Может тут лучше сделать "Откройте для себя Chrome DevTools", по типу сделайте открытие)


Are you sure we don't need a loop with timeout ?  Just make sure there are no races.   If it's ok I can merge it later today


"Теперь настало время изучить самый мощный инструмент в вашем браузере - Chrome DevTools!"


@HmHmmHm :P


So much wow commit message.


No, it's not inherited yet. It will be once #8515 etc. have been merged, but until that happens this is needed.


more like pimp @sudosurootdev but close enough LMAO


Yeah, though there are only 4...it wouldn't be TOO painful :P But you're right, we could stick to one deck and not go into detail on text.


True. And because emtpy is validatef first as not empty it can check the value.