This file has not been uploaded to the update site. Why merge the PR when the file is not uploaded, i.e. the problem still persists that the PR wanted to solve?


Ok Thanks


I will.


Removing the datetime_to_str option here makes that it doesn't work with standard json.dumps() serializer anymore.

Should probably be retained.


This section seems like implementation notes that would be better as comments in the code.


Note that Row.as_dict doesn't propagate datetime_to_str through the recursion - which breaks it again even if you fix it here. See: https://github.com/web2py/pydal/issues/141


This should return nil in case ep.sadboxInfo is nil


WebCrawlerImp gives an impression that its an implementaiton.
it's good to have documentation for public api


Hello,

Why are those files named ending with `.f`?

Thanks!


The idea is to find all the potential alternative data sources that we could add in addition to NIST. If you have good ideas, feel free.


That's a good point! And I'll have to admit, that I currently don't know how this is handled in general. One thing that comes to mind would be wrapping the error messages generated by `Moo`. Either that or localizing `Moo` in the same way...


Typo: missing "of" before utilities.


This will be named Open Event


Add thằng này nhom tui vào nha
Vào 20-04-2015 16:41, "Dao Ngoc Duy Tan" <notifications@github.com> đã viết:
>
> ???
>
> —
> Reply to this email directly or view it on GitHub.


@rwd
OK, they should all be considerably slimmer. sorry about that.
:)


@slr yes, the exception is there (in twister-core): you will receive at most 1 promoted post every 8 hours :-)

check this discussion here about disabling promoted posts:

https://groups.google.com/forum/#!topic/twister-users/dpaxAwFYUJ0

@shaman already done! (see "switch to promoted posts" from options menu)


Yes.


CSS


OK thanks.


I don't understand the point of exposing a new method getInstance, while we already have access to configuration directly using getConfiguration method. Moreover, the addition brings issues if instance is not created.


I think I wrote this because this action will throw `NoSuchUser` if the uid doesn't exist, while `checkPassword` returns `BadCredentials` in that case, which we probably want here as well.


haddock comment needs to be updated.  explain that it is the responsibility of the caller to make sure there is no valid session that should be used instead of creating a new one.


These look like todos that ought to live on the wiki (and perhaps some of them should turn into issues when ipam is merged). As with comments in code, it's ok to talk about potential future work where it helps to understand the current state of the code (e.g. to define the scope, or to say "an ideal approach would be to do X, but we currently do the more modest Y"), but otherwise it should be tracked in a more appropriate place.


Not sure I like this random brace; can we just move the const gchar *env_shell up to the start of the function pls.


No need for {} for single line if's.


51104461.51100378
Vào 20-04-2015 16:41, "Dao Ngoc Duy Tan" <notifications@github.com> đã viết:

> ???
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/51103061/2015_De09/commit/e30c538ab894608d50762fca1264c7d12976913f#commitcomment-10799365>
> .
>


Thanks


Nguyễn Văn Bảo Chinh sẽ là người đại diện nhóm Front-end upload code nha....


Fixed (I hope) in 700ae684cb0af8847fbd5e0cdcb836ee8a40b8c4.

I also got rid of some more occurrences of `'v':` in 3b2b6bb707d49d799d0626073f434248d5f8fa5f. One difference is that `freshVar` always does `touchVar` on the fresh variable, which the previous code didn't always do.


Maybe `localhost` instead of `locahost` :)


Fixed in 643d6ade4d9399b26a06af6d06d1c775bc719eb7 :)


How about letting this be a parameter, and reading it in the do_execute()? You can default to User::createPassword() if no password is provided.


This should throw an exception if the user is not found, so it doesn't try and continue execution and fail.


ok, in that case we want three functions:

1. `verifyPass`
2. `verifyPass'`
3. `checkPass`

(or pass*word* for all three.)

2. does what `verifyPass` does now.
1. does the same thing, but turns `NoSuchUser` into `BadCredentials` to avoid user info leakage.
3. does what `checkPassword` does now (but calls 1. differently now to align with new exception semantics).


createPassword() is a static method, it should be called like this:
```php
User::createPassword()
```


You can use the provided argument and default to something else if it's not provided:
```php
$password = $this->getProvidedArgument('user_pass', User::createPassword());
```


That could go into the front page, I don't think it's that useful on every page.


Not sure why performance would decrease.  By default, babel does not look in node modules. Using ignore:false makes it look everywhere. So if anything, removing it should help performance.


Nice ton, nanti enemy gw tambahin knockback deh


Hey, Fridemann. I saw something about this in the paper but didn't quite understand it. Can you please explain it to me a little? Is there no use of e_thr here then?


Should this be "a library and collection of utilities"? I think we should try to write most modules so they are independently executable or easily called from an entry_point. That would help make them easy to understand, test, and re-use. "utilities" describes that somewhat, but we also plan to use this as a library from other projects (a CLI and a server).


Those "Import Symbol" objects are usually created when importing .SAT files.


Muhammad,
I did not receive the last attachment. Please email the poap.log file to joddavis@cisco.com.

Thanks,

Jody

From: ashfaqch99 <notifications@github.com<mailto:notifications@github.com>>
Reply-To: joddavis/nexus9000 <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tuesday, April 14, 2015 at 1:22 PM
To: joddavis/nexus9000 <nexus9000@noreply.github.com<mailto:nexus9000@noreply.github.com>>
Cc: Jody Davis <joddavis@cisco.com<mailto:joddavis@cisco.com>>
Subject: Re: [nexus9000] Update poap.py (32b7bf9)

Hi Jody,Thanks for your prompt reply, i tried to figure out exectly where is the issue, can you please direct what could be the possible reason or how i can resolve. if you want we can have a webex session also on your availability. please find the attached script and log file attached.
RegardsMuhammad Ashfaq

On Monday, April 13, 2015 8:16 PM, joddavis <notifications@github.com<mailto:notifications@github.com>> wrote:

Hello,

Break out of the POAP process on the switch in question and login to it. Once you are in the switch, issue a “dir” command. Locate the relevant “poap.log” file in the bootflash:, and then execute the “show file bootflash:poap.log.xxxxx” command, and this should tell you exactly why the script is failing to execute. See the example below.

[cid:AC4224A9-2562-475B-AEC1-E2D4AA2F25A1]

Regards,

Jody

From: ashfaqch99 <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>
Reply-To: joddavis/nexus9000 <reply@reply.github.com<mailto:reply@reply.github.com><mailto:reply@reply.github.com>>
Date: Monday, April 13, 2015 at 4:46 PM
To: joddavis/nexus9000 <nexus9000@noreply.github.com<mailto:nexus9000@noreply.github.com><mailto:nexus9000@noreply.github.com>>
Cc: Jody Davis <joddavis@cisco.com<mailto:joddavis@cisco.com><mailto:joddavis@cisco.com>>
Subject: Re: [nexus9000] Update poap.py (32b7bf9)

I am using above mentioned code, but due to some reason it poap script download successfully but not executing giving me error "POAP Script execution failed".
Any idea how i can resolve this issue???
downloaded POAP script file
2015 Apr 13 21:38:17 switch %$ VDC-1 %$ %POAP-2-POAP_DHCP_DISCOVER_START: POAP DHCP Discover phase started
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Using DHCP, information received over mgmt0 from 192.168.175.120
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Assigned IP address: 192.168.175.180
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Netmask: 255.255.255.0
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: DNS Server: 192.168.175.1
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Default Gateway: 192.168.175.1
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Server: 192.168.175.120
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Name: poap.py
2015 Apr 13 21:38:49 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script download has started
2015 Apr 13 21:38:49 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script is being downloaded from [copy tftp://192.168.175.120/poap.py bootflash:scripts/script.sh vrf management ]
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_DOWNLOADED: Successfully downloaded POAP script file
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script file size 20404, MD5 checksum f4427f42bc28b0e6d1bdb28d286e78d7
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_STARTED_MD5_NOT_VALIDATED: POAP script execution started(MD5 not validated)
2015 Apr 13 21:38:51 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed

—
Reply to this email directly or view it on GitHub<https://github.com/joddavis/nexus9000/commit/32b7bf997654e35e4d7a45c5e566ad0bae2ac70c#commitcomment-10701810>.
—
Reply to this email directly or view it on GitHub.

switch(config)# reload
!!!WARNING! there is unsaved configuration!!!
This command will reboot the system. (y/n)? [n] y
2015 Apr 13 23:13:31 switch %$ VDC-1 %$ %PLATFORM-2-PFM_SYSTEM_RESET: Manual system restart from Command Line Interface
[23508.036085] [1428966817] writing reset reason 9,
[2▒
CISCO SWITCH Ver7.17
Device detected on 0:6:0 after 0 msecs
Device detected on 0:1:1 after 0 msecs
Device detected on 0:1:0 after 0 msecs
MCFrequency 1600Mhz
Relocated to memory
Detected CISCO IOFPGA
Donner Present
MIFPGA Present
Code Signing Results: 0x0
Using Upgrade FPGA
Booting from Primary Bios
FPGA Revison : 0x5
FPGA ID : 0x1345727
FPGA Date : 0x20140828
Power Debug Register: 0x0
Reset Cause Register: 0x4
Boot Ctrl Register : 0xe0ff
FPGA Update Status : 0x20
Detected CISCO MIFPGA
Version 2.16.1240. Copyright (C) 2013 American Megatrends, Inc.
Board type 2
IOFPGA @ 0xc8000000
SLOT_ID @ 0xf
Trying to read config file /boot/grub/menu.lst.local from (hd0,4)
Filesystem type is ext2fs, partition type 0x83

Booting bootflash:/n9000-dk9.7.0.3.I1.1a.bin ...
Booting bootflash:/n9000-dk9.7.0.3.I1.1a.bin
Trying diskboot
Filesystem type is ext2fs, partition type 0x83
Image valid

Image Signature verification was Successful.

INIT: version 2Loading IGB driver ...
Installing SSE module ... done
Creating the sse device node ... done
Installing CCTRL driver for card_type 19 ...
Checking SSD firmware ...
Model=Micron_M550_MTFDDAT064MAY, FwRev=MU01, SerialNo=MSA182800LD

Checking all filesystems.......
Installing SPROM driver ...
Installing default sprom values ...
done.Configuring network ...
Installing veobc ...
Installing OBFL driver ...
blogger: nothing to do.
..done Mon Apr 13 23:14:27 UTC 2015
tune2fs 1.35 (28-Feb-2004)
Setting reserved blocks percentage to 0 (0 blocks)
Starting portmap daemon...
creating NFS state directory: done
starting 8 nfsd kernel threads: done
starting mountd: done
starting statd: done
Saving image for img-sync ...
Loading system software
Uncompressing system image: package:/isanboot/bin/images/sys Mon Apr 13 23:14:34 UTC 2015
blogger: nothing to do.

..done Mon Apr 13 23:14:37 UTC 2015
Load plugins that defined in image conf: /isan/plugin_img/img.conf
Initialize Patching Repository during load
Loading plugin 0: core_plugin...
load_plugin: pboot compatibility check for swid-core-inseor (plugin_id 0) num_local:1 num_remote:1
num srgs 1
0: swid-core-inseor, swid-core-inseor
!!!my_swid:2075106359
num srgs 1
0: swid-inseor-ks, swid-inseor-ks
Creating /dev/mcelog
Starting mcelog daemon
INIT: Entering runlevel: 3

Populating conf files for hybrid sysmgr ...
Starting hybrid sysmgr ...
2015 Apr 13 23:14:52 %$ VDC-1 %$ %USER-2-SYSTEM_MSG: <<%USBHSD-2-MOUNT>> logflash: online - usbhsd
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-PS_FAIL: Power supply 1 failed or shut down (Serial number DCB1837Y2CG)
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-PS_OK: Power supply 2 ok (Serial number DCB1837Y2CK)
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-PS_FANOK: Fan in Power supply 2 ok
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-PS_ABSENT: Power supply 1 is absent/shutdown, ps-redundancy might be affected
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-PS_RED_MODE_CHG: Power supply operational redundancy mode changed to non-redundant
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-FANMOD_FAN_OK: Fan module 1 (Fan1(sys_fan1) fan) ok
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-FANMOD_FAN_OK: Fan module 2 (Fan2(sys_fan2) fan) ok
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-FANMOD_FAN_OK: Fan module 3 (Fan3(sys_fan3) fan) ok
2015 Apr 13 23:14:55 %$ VDC-1 %$ %PLATFORM-2-FANMOD_FAN_OK: Fan module 4 (Fan4(sys_fan4) fan) ok
2015 Apr 13 23:15:08 %$ VDC-1 %$ %COPP-2-COPP_NO_POLICY: Control-plane is unprotected.
2015 Apr 13 23:15:17 %$ VDC-1 %$ %CARDCLIENT-2-FPGA_BOOT_PRIMARY: IOFPGA booted from Primary
2015 Apr 13 23:15:17 %$ VDC-1 %$ %CARDCLIENT-2-FPGA_BOOT_PRIMARY: MIFPGA booted from Primary
2015 Apr 13 23:15:18 %$ VDC-1 %$ %VDC_MGR-2-VDC_ONLINE: vdc 1 has come online
Waiting for system online status before starting POAP ...
Waiting for system online status before starting POAP ...
Starting Auto Provisioning ...
Done

Abort Auto Provisioning and continue with normal setup ?(yes/no)[n]: 2015 Apr 13 23:16:19 switch %$ VDC-1 %$ %POAP-2-POAP_INITED: POAP process initialized

Abort Auto Provisioning and continue with normal setup ?(yes/no)[n]: 2015 Apr 13 23:16:38 switch %$ VDC-1 %$ %POAP-2-POAP_DHCP_DISCOVER_START: POAP DHCP Discover phase started
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Using DHCP, information received over mgmt0 from 192.168.175.120
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Assigned IP address: 192.168.175.180
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Netmask: 255.255.255.0
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: DNS Server: 192.168.175.1
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Default Gateway: 192.168.175.1
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Server: 192.168.175.120
2015 Apr 13 23:16:47 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Name: poap.py
2015 Apr 13 23:16:58 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script download has started
2015 Apr 13 23:16:58 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script is being downloaded from [copy tftp://192.168.175.120/poap.py bootflash:scripts/script.sh vrf management ]
2015 Apr 13 23:16:59 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_DOWNLOADED: Successfully downloaded POAP script file
2015 Apr 13 23:16:59 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script file size 19505, MD5 checksum 16b4667c5d4eb6fb7952c86a764a5c41
2015 Apr 13 23:16:59 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_STARTED_MD5_NOT_VALIDATED: POAP script execution started(MD5 not validated)
2015 Apr 13 23:17:00 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed
2015 Apr 13 23:17:03 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed, restarting the DHCP Discover phase
2015 Apr 13 23:17:03 switch %$ VDC-1 %$ %POAP-2-POAP_DHCP_DISCOVER_START: POAP DHCP Discover phase started
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Using DHCP, information received over mgmt0 from 192.168.175.120
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Assigned IP address: 192.168.175.180
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Netmask: 255.255.255.0
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: DNS Server: 192.168.175.1
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Default Gateway: 192.168.175.1
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Server: 192.168.175.120
2015 Apr 13 23:17:25 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Name: poap.py
2015 Apr 13 23:17:35 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script download has started
2015 Apr 13 23:17:35 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script is being downloaded from [copy tftp://192.168.175.120/poap.py bootflash:scripts/script.sh vrf management ]
2015 Apr 13 23:17:36 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_DOWNLOADED: Successfully downloaded POAP script file
2015 Apr 13 23:17:36 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script file size 19505, MD5 checksum 16b4667c5d4eb6fb7952c86a764a5c41
2015 Apr 13 23:17:36 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_STARTED_MD5_NOT_VALIDATED: POAP script execution started(MD5 not validated)
2015 Apr 13 23:17:37 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed
2015 Apr 13 23:17:40 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed, restarting the DHCP Discover phase
2015 Apr 13 23:17:40 switch %$ VDC-1 %$ %POAP-2-POAP_DHCP_DISCOVER_START: POAP DHCP Discover phase started

Abort Auto Provisioning and continue with normal setup ?(yes/no)[n]: yes

---- System Admin Account Setup ----

Do you want to enforce secure password standard (yes/no) [y]: no

Enter the password for "admin":
Confirm the password for "admin":

---- Basic System Configuration Dialog VDC: 1 ----

This setup utility will guide you through the basic configuration of
the system. Setup configures only enough connectivity for management
of the system.

Please register Cisco Nexus9000 Family devices promptly with your
supplier. Failure to register may affect response times for initial
service calls. Nexus9000 devices must be registered to receive
entitled support services.

Press Enter at anytime to skip a dialog. Use ctrl-c at anytime
to skip the remaining dialogs.

Would you like to enter the basic configuration dialog (yes/no): no

2015 Apr 13 23:18:20 switch %$ VDC-1 %$ %COPP-2-COPP_POLICY: Control-Plane is protected with policy copp-system-p-policy-strict.

User Access Verification
switch login: admin
Password:
Bad terminal type: "linux". Will assume vt100.
Cisco Nexus Operating System (NX-OS) Software
TAC support: http://www.cisco.com/tac
Copyright (C) 2002-2015, Cisco and/or its affiliates.
All rights reserved.
The copyrights to certain works contained in this software are
owned by other third parties and used and distributed under their own
licenses, such as open source. This software is provided "as is," and unless
otherwise stated, there is no warranty, express or implied, including but not
limited to warranties of merchantability and fitness for a particular purpose.
Certain components of this software are licensed under
the GNU General Public License (GPL) version 2.0 or
GNU General Public License (GPL) version 3.0 or the GNU
Lesser General Public License (LGPL) Version 2.1 or
Lesser General Public License (LGPL) Version 2.0.
A copy of each such license is available at
http://www.opensource.org/licenses/gpl-2.0.php and
http://opensource.org/licenses/gpl-3.0.html and
http://www.opensource.org/licenses/lgpl-2.1.php and
http://www.gnu.org/licenses/old-licenses/library.txt.
switch# feature bash
^
% Invalid command at '^' marker.
switch# config t
Enter configuration commands, one per line. End with CNTL/Z.
switch(config)# feature bash
switch(config)# run bash
bash-4.2$ cd /bootflash/
bash-4.2$ dir
20150413_231619_poap_6244_init.log lost+found scripts
home n9000-dk9.7.0.3.I1.1a.bin virtual-instance
bash-4.2$ ls -l
total 434800
-rw-rw-rw- 1 root root 4605 Apr 13 23:17 20150413_231619_poap_6244_init.log
drwxrwxrwx 3 admin network-admin 4096 Apr 9 16:48 home
drwxrwxrwx 2 root root 4096 Apr 9 22:17 lost+found
-rw-rw-rw- 1 admin network-admin 444765696 Apr 9 22:12 n9000-dk9.7.0.3.I1.1a.bin
drwxrwxrwx 2 root root 4096 Apr 13 23:17 scripts
drwxrwxrwx 2 root root 4096 Dec 11 20:19 virtual-instance
bash-4.2$ cat 20150413_231619_poap_6244_init.log
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=1
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=2
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=3
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=4
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=5
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=6
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=7
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=8
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=9
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=10
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=11
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=12
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=13
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=14
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=15
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=16
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=17
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=18
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=19
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=20
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=21
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=22
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=23
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=24
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=25
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=26
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=27
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=28
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=29
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=30
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=31
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=32
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=33
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=34
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=35
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=36
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=37
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=38
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=39
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=40
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=41
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=42
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=43
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=44
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=45
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=46
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=47
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=48
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=49
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=50
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=51
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=52
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=53
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=54
Mon Apr 13 23:16:19 2015:poap_powerup_port_add: port_cnt=55
Mon Apr 13 23:16:27 2015:poap_vsh_all_ports_cmd_exec: Command execution failed 4096
Mon Apr 13 23:16:40 2015:poap_is_offer_valid: valid option_flag=a6f
Mon Apr 13 23:16:40 2015:poap_is_offer_valid: valid option_flag=a6f
Mon Apr 13 23:16:59 2015:poap_verify_downloaded_script:Could not retrieve MD5 info from script file.
Mon Apr 13 23:17:00 2015:poap_phase_kill_script:kill failed: errno:3
Mon Apr 13 23:17:00 2015:poap_phase_kill_script:waitpid failed: errno:10
Mon Apr 13 23:17:00 2015:poap_vsh_rollback_dhcp_cfg: Command execution failed 64512
Mon Apr 13 23:17:00 2015:poap_vsh_rollback_boot_vars: Command execution failed 4096
Mon Apr 13 23:17:04 2015:poap_is_offer_valid: valid option_flag=a6f
Mon Apr 13 23:17:05 2015:poap_is_offer_valid: valid option_flag=a6f
Mon Apr 13 23:17:36 2015:poap_verify_downloaded_script:Could not retrieve MD5 info from script file.
Mon Apr 13 23:17:37 2015:poap_phase_kill_script:kill failed: errno:3
Mon Apr 13 23:17:37 2015:poap_phase_kill_script:waitpid failed: errno:10
Mon Apr 13 23:17:37 2015:poap_vsh_rollback_dhcp_cfg: Command execution failed 64512
Mon Apr 13 23:17:37 2015:poap_vsh_rollback_boot_vars: Command execution failed 4096
Mon Apr 13 23:17:41 2015:poap_is_offer_valid: valid option_flag=a6f
Mon Apr 13 23:17:42 2015:poap_is_offer_valid: valid option_flag=a6f
bash-4.2$

—
Reply to this email directly or view it on GitHub<https://github.com/joddavis/nexus9000/commit/32b7bf997654e35e4d7a45c5e566ad0bae2ac70c#commitcomment-10720361>.


I tried to follow your instruction in order to get my cooliris wall working again, but it doesn't work. I checked the mentioned step's over again but could not find out why it doesn't work yet.
please can I ask for some help?


Test


@brettwooldridge note that the partial merge introduced a bug here - elapsed time in milliseconds is compared to nanosecods. Here is the correct version https://github.com/brettwooldridge/HikariCP/commit/3dc3f619b4831f39e53bf86b54d7c0c9a8477774#diff-644ae04f83606ef1da22346fde9bff64R259


Du och dina instanser. Har tagit bort referenser till placeService i e4d150b1


Added check as I was getting console errors


Indent still messed up.


We've had a tough time in the past with getting labelled as doing only _green field_ deployments when we say only "deploying applications", and the label seems to stick. Our intent is to handle also ongoing application lifecycle actions (migrate, scale out, scale up, audit, clone, etc...). Maybe "deploying and managing"?


yes , i changed name of this interface in final commits.


We want providers to be pluggable - I think that's a key concept. We can list our intent to support Vagrant and Docker, but let's only list them as planned until we actually get them done.


Yeah, that's what I was thinking. Just the front page would be better.


`'Fixture path ...' -> 'Data path ...'`


Ohay.


It's strange to have a virtual destructor without any virtual method


This loop is to keep on the next files busy , while the previous file is running


Couldn't those be implemented as constructor/destructor?


I've discovered a problem with this action, in that for multiple machine configurations this will always result in chef-zero being torn down when you destroy any of the machines. I don't know that the previous approach was any better since I suspect the action still fired on all destroys, but it does mean that I need to modify the action to check if any machines will still be active.


I see canvas is upgraded to newer version of rails and discarding the `vendor/plugin` folder and hence wiziq.Now I want to integrate back the wiziq plugin compliant to all standards.Will it go into core or it has to be a plugin(`lib/wiziq`) or something else?.Would be nice if you can share some getting started docs


I haven't tried building the sources, though it seems this constructor isn't defined (in this import at least, but I might have missed something)


__API Blueprint syntax error on line 3212__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


replaced sss mysql container with layers mysql user and database creation for sss


Some test like ServerTest needs to setup server with a configuration. but MDMConfiguration only allow one instance to be created and no method to retrieve the current instance. that's why I added the public method to get an instance .


测试完整性上讲
要覆盖到buffer的长度对4取模后余0/1/2/3个字节的情况
并且这些尾数要分正负数情况
ps:我本地验证过了, 哈哈


Same comment here about virtual destructor & lack of other virtual methods


good, thanks!


And now also available as 3.5...


inline is implicit here since you're defining the method as part of the class


I'd add an operator== for convenience


This has security implications, you are storing credentials in plaintext.


TeamCity ss13 :: RuBayStation :: Check_PRs [Build 11](http://terrariatc.noip.me:8111/viewLog.html?buildId=48&buildTypeId=Ss13_RuBayStation_CheckPRs&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:43


move nvme-lightnvm to here


This class is lacking a virtual destructor


i am not sure here since i don't know whats the expected outcome from jump


can we remove sysouts.


Another approach could be create new instance in getConfiguration() if it's not created yet.


Can be removed


should this be crawledUrls  ?


It feels incomplete to list those without explaining why we're doing this and how we differ.

Some points:
- The ability to "describe my application once" and re-use that description everywhere. The translation to specific providers (AWS, OpenStack, Docker) is done elsewhere (in the provider code, or in separate, provider-specific blueprint flavors).
- Provider-agnostic. This speaks to a major differentiator from docker-compose, as well as what appears to be missing from Terraform.
- Python, not Ruby or Go (it matters to many)


う〜ん、XX2とかは好きでは無いけどしょうがないですかね。
ここで一回分けておけばあまり互換性を気にしなくても良さそうだし。

2015年4月20日 19:32 Kei Okada <notifications@github.com>:

> #590 <https://github.com/fkanehiro/hrpsys-base/pull/590>
> に解決策を書きました．
>
> https://github.com/start-jsk/rtmros_gazebo/tree/master/hrpsys_gazebo_general/iob
> からも悲鳴があがっているみたいですが，このPRでもダメですかね．
> 一応，コメントは入れておきましたが．．．．
> k-okada@01b72a2#diff-b30cc06105f07d38ed353b57449d72c4R586
> <https://github.com/k-okada/hrpsys-base/commit/01b72a221c889a1bc906e3bfc214311f2f49dbc7#diff-b30cc06105f07d38ed353b57449d72c4R586>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fkanehiro/hrpsys-base/commit/47cf3605ad409585aee23a9e7aa1e290471247fd#commitcomment-10800070>
> .
>


In minetest, identation is with tabs.


Lambda capture part, PR: filipw/AspNetWebApi-OutputCache#127


But I think this would be possible as well when the plugin specified it's name. correct? @lukemelia


can have a different name "sngle"


ETag creation part, PR: filipw/AspNetWebApi-OutputCache#127


This is initial


This could be repacked (and, nitpicking, m_bAutoRelease has a weird intendation :) )


Read this for details: http://dimaip.github.io/2015/04/18/caching-typoscript/


Just tried to build from scratch yesterday before import.  Yes you are right at current revision there are just two static methods used to initialize and finalize application, and yes both constructor and destructor are not implemented but should not get any error if you use it as for examples.
I think at current revision both can be removed and may be reintroduced when they will have a scope.


Final home page.
problem in testing.js


лучше так: .replace(/\u00a0{2,}|\u00a0\u0020|\u0020\u00a0/g, function(str){


I doubt it will make much of a performance difference. But wouldn't avoiding the addition be better? So
```sqf
if (diag_tickTime > GVAR(nextActionSearchTime)) then {
```
With the GVAR being set to `diag_tickTime + 0.20`.


can it just be crawl


Sorry, commit shouldn't be on develop branch. But should anyways work just fine


Looks good


Readme.md was here intentionally and in my opinion it was really helpful for newcomers and me as well.


аналогично. `k` нет, и ничего мы не знаем о том что было.
вообще говоря, тут можно было без массива обойтись, а просто читать файл, парсить значения и выводить на экран.


для чего отдельный индекс в массиве нам тут? почему не используется `l`?


вообще говоря, у нас здесь не должно быть `k`. т.е. при чтении файла мы ничего не знаем о том, что было до.


Аналогично, если не смогли считать - будем дальше пытаться что-то сделать.


Если что-то пошло не так при записи, мы просто печатаем стэктрейс и продолжаем работу программы?


Не нужная операция, при записи в wr система должна будет сама создать файл.


1. `z++` можно делать в самой конструкции фора, рядом с `ind++`
2. почему бы не использовать утилитные методы копирования массивов?


Немного странная и не очевидное условие на итерацию. Если идём по массиву, то почему бы не использовать ограничение вида: `m < myArr.length` ?


Надо проверять, а вдруг не число будет введено.


Зачем этот массив?


Если пользователь ошибся и запустил программу без аргументов вообще, то получим инициализацию массива отрицательной длины


Зачем? О_о


Вообще, читать сразу весь файл не очень хорошо. А вдруг система не даст тебе его полностью прочитать за одну операцию?
Но в этой задаче, в принципе, можно оставить


Пустой пакет - не очень хорошо


initialization and finalization can be in different scopes, so better to let programmer to do it.


There's a lot of code like this in ripple-client, we probably want to avoid the use of caching until it's been given due consideration:

```
        keychain.requestSecret(id.account, id.username, function (err, secret) {
          // XXX Error handling
          if (err) {
            $scope.loading = false;
            console.log(err);
            return;
          }

          $scope.mode = 'granting';

          tx.secret(secret);
```


__API Blueprint syntax error on line 3279__: Undefined resource model 1039 (Autogenerated message from apiary.io)


it will be easier.
any idea if some kind of weight on traffic light nodes would be doable for urban car.
something like multiply by 2 the node traversal if real time datas says it.


Oh


I have mention package name of views.


inline keyword will force compiler to do it, note that both general setting for compiler and also a different compilers don't produce same result. eg. if you declare a complex method in the header file you have no assurance this will be expanded inline, most of compiler will manage it as normal function like you declared it in the cpp. Some time even you have inline keyword compiler ignore it and you must force if by general settings one of that is msvc.


agreed can be removed.


The shim could just take a block and pass it I suppose.
The general issue is to fake the parameters to match the ones of the original methods, which is not possible in Ruby, so it might be needed to define it in Java.

How can we solve the problem of being able to call non-self? this breaks some assertions if not fixed!


есть мнение, что эту штуку никто не удаляет. да и вообще, зачем она на куче?


How could frame be null there?


Hi @dimven You can explode the element, but unfortunately it's not editable afterward.


what if the initial value is out of bounds already?


@torkelo Please forgive my ignorance. Is this means that grafana 2.0.0 released? I am asking this because I can not see any git tag in repo. It would be nice to tag each milestone as well.


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


имеет место некоторый беспорядок с пробелами перед скобками


This should be node-retro not my local file.


а зачем эта штука хранится по указателю?


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


(sorry about the late reply btw)
redis_key_length isn't in scope anywhere? Is this a constant you import from somewhere?


эта штука не используется


а эта штука используется, но называется так, будто она класс из Qt :)


How about a simple
"if secret != None:" ? :)


I guess we should add a doctype here, just to play nicely.


строки надо передавать по константной ссылке, иначе они копируются при вызове метода


fntef 的话，有没有可能也取消了？


No need for the dpi > 150 check.


кстати, этот метод можно было сделать static-ом, он же не использует никак состояние объекта


This is 95% cosmetic. I switched using a md5 to generate the name for a dump file to using the appropriate library and did some other cleaning up of error handling.


Looks good. Could you solve to two issues i mentioned?


Learn how to use git and merge my commit next time, ok?


Regardless of why the function is declared as _inline_, it's only a hint for the compiler. If you absolutely want that function to be inline you can, as you mentionned, use some MSVC settings (you can also use __forceinline specifier) or other compiler specific methods.
Anyway, inline is redundant, but that was pretty much a pointless nitpicking :)


also - formatting


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


Methods don't have declaration frames - only blocks do. This patch is about turning methods into procs, and so we get procs without declaration frames.


it's a PUT (POST) method so the parameters are sent into the `body`


Done in 7461fed1 .


@ajinabraham i have to ask you one thing--- How to run this script. i wanna check vulnerability in My Mblaze Ultra Wifi. BUt when i run it it shows ImportError: no module named requests. Can u Help


@Ayysir AFAIK the original project is maybe dead, but I'm still maintaining my fork. I just update the dependencies in the bower.json file and I think this still works as always. If you detected issues with polymer 0.8.0, please report them.

Thanks!


по указателю обычно передаются только out-параметры --- методу отдают указатель на место в памяти, куда надо положить результат. Просто принимать лучше по ссылке или константной ссылке, если это возможно.


change typo on strings to


This is why `add()` returns its first param. This proc could be shortened to `newNimNode(kind).add(children)`


а чего оно на куче?


yea, I am in the processes of releasing, a tag will be added shortly


а его кто-нибудь удалит?


Instead of indenting preprocessor directives like this, the most common way to do that is:
```
#ifdef ...
#  include <...>
#  define ...
#endif
```

I'm not sure if it's included in our Style Guide, but it definitely must be there.


а его?


Yeah I think the parameter handling thing may also be important - maybe you can query the arity of a block - and in this case it would tell you it was anything, and maybe it should tell you the arity of the original method.

Rubinius uses a primitive rather than Ruby, so it seemed to make sense to do the same.

Not sure what you mean by being able to call on non-self?


WordClearer::clear(wordToAdd) мне кажется разумным вызвать один раз перед всем этим кодом, чтобы не вызывать его снова и снова


yo.
`pacman -S` `lightdm` `lightdm-gtk3-greeter` till
`pacman -S` `lightdm` `lightdm-gtk-greeter`


It should be PLATFORM_STM32F429_EXTI_H_ not PLATFORM_STM32F4_EXTI_H_


I think I tried with `let` earlier but it didn't work.


вместо operator [] можно было написать просто (*frequencyDictionary)[wordToAdd], а если бы frequencyDictionary была ещё и на стеке, было бы совсем хорошо


This is a matter of preferences, but IMO the following would be much more readable:
```
    while (...)
        ;
```


Would creating a mostly empty frame be a good alternative?


can you check if crawledUrl is a synchronizedCollection after this statement ?


#include <platform/stm32f429/registers.h>


вообще, лучше !translation.isEmpty()


JFS?


> Not sure what you mean by being able to call on non-self?
The failing spec.


Good commit, mate


Same here


Did you send a pull request with this?

Without this change it is impossible to implement the simplest lock using zookeeper.


  -f  整合性チェックで失敗しても処理を続行する
optionをつけて実行しましたが、結果が先と変わりませんでした。
同じ警告文が出てしまうようです。


@jkvis I see you have quite nice structured comments in the header file. Do you think it is possible to generate API documentation from that, for example in HTML format? E.g., using something like [Doxygen](http://www.stack.nl/~dimitri/doxygen/)?


All I did here was copy and paste the frmMain.cs file from master.


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


"catalog"? A directory may be?


also changed in cocoa , needs testing !
only ask, alert and confirm dialogs


Why should we remove this parameter ?


renamed method


Jacek chyba z nami tego nie robi :)?


I guess now it will also fail for python2 :smile:
`rmsd_pred` should be `np.sqrt(831.7)` if I calculated correctly


Also settings and logging capabilities added


лучше QVector, чем сырые массивы. сырые массивы вообще почти никогда не нужны


I guess this commit doesn't introduce logical changes, does it? Can I skip it without reviewing?


Нужно указать методы в `only`, сейчас вызывается перед всеми (и по идее тесты для неавторизованного пользователя сейчас должны падать).


this happens when you mix editor and vi :D  btw tab removed.


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


Because i can access object with key $result->$field_key, but now I want to access a value by key $result->$field_key[$i]['rendered']['#markup']. e.g.(that obviously does not work) so i clone result object and assign some like this {$clone_key_array[$i]['rendered']['#markup']}


annot1 et annot2 take a string as input, not an array. so it should be

    `"annot1": "ch.epfl.bbp.uima.types.BrainRegion",`


> Rubinius uses a primitive rather than Ruby, so it seemed to make sense to do the same.

But it does explicitly check if the proc is associated with a method, which is not ideal IMHO, while our native conversion should be able to copy parameters metadata between proc and block. So this commit s good for that.

  def parameters
    if @ruby_method
      return @ruby_method.parameters
    elsif @bound_method
      return @bound_method.parameters
    end


Hum, destroy a windows object to recreate a new one...
That take a long time.

Couldn't you just update the position ?


this is not the case??


Not sure I follow...

Say your plugin in called 'ember-cli-deploy-s3'

In the config I might want to use that same plugin, twice, for uploading
the assets and the index....As per the example above....

Therefore, in my config, I'm giving the plugin instance (for lack of a
better way to describe it) the name of 's3:assets' and then another one
's3:index'.

I might be confused about your question but I'm not sure how you would
define that in the plugin.....The plugin only knows it's an s3
plugin....The other part is how the user wants to use the plugin.

On Mon, 20 Apr 2015 at 12:08 Michael Klein <notifications@github.com> wrote:

> But I think this would be possible as well when the plugin specified it's
> name. correct? @lukemelia <https://github.com/lukemelia>
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/ember-cli/ember-cli-deploy/commit/e7cbe709cfef9d1173830b6ec2fad41d2ead8892#commitcomment-10800589>
> .
>


Great @jessertaylor. BTW, there's a "isDefined" function declared in leafletHelpers which checks that a variable is defined and i'ts not null, so it's better:

```
    if (compileMessage && isDefined(marker))) {
```


@nolanlawson was it intended to have two instances of the same db replicating to each other?


Hi Erwin,

Though it's already a while ago since I did this, I see this difference.

Old:
session = Core.initializeSession(user, null);

New:
session = Core.initializeSession(user, session != null ? session.getId().toString() : null);

Because of this, existing session contents will get transferred from the anonymous to the named session.
It was required in a project and thought it might be useful for the masses.

Kind regards,
Arjen Lammers


orz 把学长的用户名写错了。。罪过。。


Or should this be
```js
var db = new PouchDB(dbs.name);
```


side-navが閉じちゃうのはこのせいだった。
ローカルで発生しなかったのはわからん。


He añadido mi nombre


not needed. This class can't have any instance.


Please, go ahead !

In case it might help I tried some tuning as well since this patch was submitted. (10M @ 2.0+0.6)
The SPSA model used was as follow.
QCC,91,0,200,8,0.0020,0

I tuned the contact checks, the safe checks, the RookOnOpen and Semiopen, and finally RookOnPawn,
because I suspected that they were penalized by the [QUEEN] bug that I removed.
See new values at the left, the old ones at the right
int QueenContactCheck = 91; //89;
int RookContactCheck  = 65; //71;
int QueenCheck        = 46; //50;
int RookCheck         = 43; //37;
Score RookOnPawn         = S(14, 21); //S( 7, 27);
Score RookOnOpenFile     = S(47, 25); //S(43, 21);
Score RookOnSemiOpenFile = S(26,  5); //S(19, 10);

I also included the only the rook weight from the following array
It dropped very quickly to 2 and stayed between 1.4 and 2 for most of the tuning session.
int KingAttackWeights[PIECE_TYPE_NB] = { 0, 0, 7, 5, 2, 1 }; //{ 0, 0, 7, 5, 4, 1 };

Finally, to do a really good job, I was planning to redo the tuning experiment, but include the
rook mobility stuff too.


It's just an idea.
Two way do made it:
- accessors for dedicated types
- get all, optionally filter by parameter


Thanks will do
On Apr 20, 2015 7:38 AM, "Lucas Matías Ciruzzi" <notifications@github.com>
wrote:

> @Ayysir <https://github.com/Ayysir> AFAIK the original project is maybe
> dead, but I'm still maintaining my fork. I just update the dependencies in
> the bower.json file and I think this still works as always. If you detected
> issues with polymer 0.8.0, please report them.
>
> Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vangware/font-awesome-polymer-icons/commit/13da889f3abd5d01afc2690958839534075c2b2d#commitcomment-10801058>
> .
>


Well, you can merge that. Instead to have 2 functions enable / Disable

you can one which is call "set"  with key, mark, and position as arguments.

And if mark is nil, that means you want to remove it.

Example:

```lua

Set(MyKey, "A", position1)
Set(MyKey, nil)
```

A function can be called with less arguments than expected. it's allowed in lua.


Beste Martijn,

De bedoeling was van wel. Ik heb de stijl van Jeroen aangehouden wat betreft het commentaar. Hij zei dat hij een tool had om daar automatisch documentatie van te maken.

VrGr,

Jonathan

________________________________
From: Martijn Vermaat [notifications@github.com]
Sent: Monday, April 20, 2015 1:44 PM
To: LUMC/extractor
Cc: Vis, J.K.
Subject: Re: [extractor] Start on examples (85c91b9)

@jkvis<https://github.com/jkvis> I see you have quite nice structured comments in the header file. Do you think it is possible to generate API documentation from that, for example in HTML format? E.g., using something like Doxygen<http://www.stack.nl/%7Edimitri/doxygen/>?

—
Reply to this email directly or view it on GitHub<https://github.com/LUMC/extractor/commit/85c91b9e19220cf60148dec6157ff359470a5a21#commitcomment-10801164>.


You need to install the python module name requests


could you shortly tell me what is not working. Note that columns are 0-indexed in this version ( I can change that back to a 1-index if you like)


yeah sorry it will still work in python2 because of wrong division in rmsd implementation, see my other comment


This was the function of the post-process logic (that you implemented...)


s/childs/children/g


nitpick: no parenthesis needed.


Please do not bother yourself on squashing when there is only one author in PR.
In that cases the squashing is done automatically during merging.


LGTM


Изначально собирался привязывать Device к Platform через контроллер Platform.

Но после решил, что лучше это делать через Device.
@RequestMapping(value = "/{serial}/platforms/{platformId}", method = RequestMethod.PUT)

Не стал добавление (привязку) делать как POST, а удаление как DELETE. По скольку реально в базе сущность остается и ее состояние не меняется. Получается, что в обоих случаях мы делаем update, чему больше отвечает PUT. Однако у нас может быть только 1 PUT на этом @RequestMapping. Пока сделал другой как DELETE. Просто не знаю как лучше, хотя варианты есть.


Thank you!


Thanks for noticing that. Actually, this file was [generalized beyond JavaScript in the subsequent commit](ee49579de55eca4dfa5b2392f42795926f393a26). But regardless, it was indeed missing from the update site, so I uploaded it now. @ehrenfeu, let us know if any problems.

This sort of skew will become much less likely after imagej/imagej-maven-plugin#11 is addressed, along with fiji/fiji#37 and fiji/fiji#39.


@gabrielwalt this commit and 73ae5905a9dd62f329cf5934d208491926ce83f1 pull in JSP stuff that's not currently suppported by the ide tooling

![archetype-errors](https://cloud.githubusercontent.com/assets/21237/7229519/4e3585c0-e76d-11e4-9739-72324fbb598d.png)

Is there a way to rewrite this functionality with sightly or use another way which is better supported in the IDE tooling?


This line really confused me. It is an error code, right?


I said 'sorry twiz'. Do you want me to say 'Sorry Kevin also known as twizmwazin' time? I'll go revert it.


I just pushed a test with new values for safe checks using the Texel tuning method. They are quite different from the old ones.
In a quick local test they did quite well, so most likely they will fail. ;-)


ok corrected


this still returns an empty string. the prinf function returns a variety of things depending on how i set it up from ".E" to ".F"


hello


What do you mean by "stopped completely"? This parameter was in loader.conf for years, the only version that had it changed was 2.2.2.


hhh


True, it can't be instantiated, but the fact that you declare an interface make me think you want to have a pointer to it, with any implementation underneath.
Not having a virtual destructor will cause the derived class destructor not to be called if that happens.


There was a constructor parameter at some point then it's got removed. So no need for the getInstance(). Will remove it.


Nie zwróciłem uwagi, dopisałem siebie po prostu :D

2015-04-20 13:45 GMT+02:00 Kuba Borowski <notifications@github.com>:

> Jacek chyba z nami tego nie robi :)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mc-suchecki/APSI/commit/b8150816ceedf5198da9f5b0f59f8cac637269df#commitcomment-10801178>
> .
>


Thanks!


This parameter in pfsense.util.inc Now after make change pfsense can not boot

        Android için Outlook tarafından gönderildi

On Mon, Apr 20, 2015 at 4:57 AM -0700, "Renato Botelho" <notifications@github.com> wrote:
What do you mean by "stopped completely"? This parameter was in loader.conf for years, the only version that had it changed was 2.2.2.

---
Reply to this email directly or view it on GitHub:
https://github.com/pfsense/pfsense/commit/cc1f655f8e88ccda837e21d0646bbc71781198b9#commitcomment-10801391


Исправлено в 4fbc789d2931015292e46e199ba92342945c1bb3


@gr2m shouldn't this line left unchanged:
```js
result.errors = result.errors.concat(errors);
```


Will this comment be seen after the PR?


He añadido DAW


AFAIK, `ERANGE` is for math overflow errors, etc.


If you passed `false` through the third route parameter, you'd get a relative url.

```
route('name', [], false);
```


Thanks for fixing this bug!


Ah nice @Lavoaster is that an L5 thing?


Nice simplification of the `_this` and `utils`


@dentmaged It is in pretty plain English, just read the license.


empty?


@oloflarsson, I searched some on the internet, but I couldn't find a better way to do this casting. I would really appreciate a neater way, because my knowledge lacks here.

The important thing for me was to get the .name() function out of the object and this was the only way I saw so far (besides reflection).

Thanks in advance! :)


do you need to test `shot_open_tag` here as well


This is never triggered, `panic` exits immediately.


Just checked, it isn't.


Oh, awesome.


@twizmwazin It says in the license '..the rights to use, copy, modify...' so why can't I copy your version and paste it in here?


Hmm I cant' recall it ... should have commented. Maybe try to change it and run tests?


Interesting. I'm annoyed that I missed this parameter because it'd have been really nice.


@valib @SubJunk Yay, already reported but not taken as a serious behavior which produces "bug feeling" from user side:-)
Maybe creating dummy file with text that subs file is generated and when finished, file is closed also on renderer side. After that user must start file again...but user will be blocked for about 4mins for every HD file with embedded subs:( https://github.com/UniversalMediaServer/UniversalMediaServer/issues/532


Ah sure enough, I see it now.


empty


@dentmaged You can copy it, but just apply the commit, don't copy and paste it.


Maybe not, do you want me to remove it?


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

VSO State: Done


You should only test what is needed.


please link HHVM bug instead.


Nice one! :-)


I will just raise my usual complaint here. It's true we still miss a proper way of testing UI, but while we "wait" for it, I'm not sure if there is a point in writing tests like this (mirroring the implementation), just for the sake of having the class covered.


I don't think that was the whole story, but unfortunately now I can't easily find any references to where I more clearly expressed the outstanding questions!


I'm not sure how I feel about doing things this way. Seems rather unconventional. @fadils what do you think?


A fix is now committed to Platform.sh's API, but not deployed


Need use X9180?


Then I'll remove it.


Is it the same one as in `short_open_tag`?


@twizmwazin How would I know that you would threaten to DMCA if I copied it? If I did know that, then I would have used a patch.


Quite original values indeed. Who knows ?

I don't believe in quick local tests too !
When you look at individual workers results for neutral patches,
the range at the end of 1000M is +-40.


I agree, but purpose of this class is to be implements a pattern. You should provide an implementation of IAVDecoderEvents to CAVDecoder in order to retrieve events from decoder but IAVDecoderEvents isn't intended to a base class for a hierarchy of classes.


Practising commit, etc.


`var NEGTIVE_MAP`长期驻留，可以用外部存储，避免GC。


Reading this I vaguely remember a discussion a year ago whether any static or dynamic library may depend on _base_. Does anybody recall exactly what the discussion was about?


Can't remember, sorry


Hmm, would it be possible to log timestamps for all repos into dnf.log? Not flooding terminal but still have detailed information logged.


Two huge ```it``` with more than 10 ```expect``` each is by no mean a meaningful test. If I get it correctly, this is not UI, so there should be a way to make it more readable or behavior-oriented.


cc @whyrusleeping :)


already mentioned it here: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1097#issuecomment-87074817


Why version 0.0.1? Shouldn't it relate to the Highcharts version?


Readme.md still says that Rails 3.2 is supported.


As discussed with @rsalevsky: the shebang line can be removed from all  `*.py` files except for `docmanager/docmanager.py`.


Certainly possible, it's just personally I am not a fan of writing Android layout xml files :( . It's not a very complicated UI anyway so for now let's leave as it is.


type names should not be that qualified...


Last time I checked ` xcode61 ` was still their default, which is why I bumped it to ` 611 ` manually. Might not be the case these days though.


Fixed


Knowing the ` $PATH ` is still semi-useful with Travis at times.


Hey @brodavi, how do I email you?


I didn't do this before because I had some issues with the Cask's ` basictex ` compared to the ` cURL `ed version. If you made it work though, :+1: :heart:


not specific enough


Actually, that doesn't work either. It looks you had a similar issue last month or so. https://gist.github.com/renaud/a025fb10d66575412355 Do you remember how you fixed it?


According to @ssumpf only the driver or rather the binary should be linked against _base_.


Hey @jjg how do I email you?


NP: I would say "returns" (or nothing at all) instead of "initialize", since the latter seems to imply that the method should only be called once.


NP: same as above


I left this in previously because Travis comes preinstalled with a few ` brew ` utilities and having them be outdated could be troublesome. If it works okay to ditch it though, *shrug*.


Fixed.


Hum no I don't. Did you try to clean (`mvn clean`)?


Fixed too.
Yeah, I agree with your concern, but not sure what's the best way to do here, unless we use some DI library? But given the size of the SDK, would it be a little bit overkill?


@BambooIV 嗯，我暑假再试试折腾


According to our internal version numbers, we have the 3rd incarnation of DocManager. I would prefer to use 3 as the major number for that reason.


Well.. tab.


We made same rephrasement and updated it for the current changes.


Exactly.


The `let initrd =` stuff is also unnecessary now.


The order of the params matters, while this way as it was originally in the README the mongo shell will look for local 27017 anyway. This small change helps


better if you name it KEY_IRON_VERSION ;)


Doesn't this ack the IRQ before the handler is called? Shouldn't we in general ack after the handler was executed only? The early ack makes no difference for edge-triggered interrupts but will cause in instant reassertion of the IRQ in case of level-triggered interrupts.


and here obv. :)


Thanks, removed as well.


changes done by dev 20 for success message


I guess GREL has been redefined as "Gently Rejected Evaporated Language".

Quick! Remove the `$(vte_CFLAGS)` and `$(vte_LIBS)` before it boards an alien spacecraft and spawns a new process..


Looks like a better place for `_irq_conn.ack_irq()` but you may prove me wrong ;-)


Yes, all the includes of `UintTypes.h` refer to it that way. We'd get a compile error on linux if we used different capitalization.


Is there a reason why not to use six.PY3 constant to check python version?


Yep, didn't change a thing. :-/

Does it work on your end by any chance?


This comment should be probably updated too.


change this to a match?


Agree an image would look good but also agree that there is no easy picture.

The 2 pictures at the following url could be used:
http://docs.open-hydrology.org/projects/oh-auto-statistical/en/stable/run.html

Unless it was a photo taken by one of us I am reluctant to include.

On 20 April 2015 at 11:00, Florenz A. P. Hollebrandse <
notifications@github.com> wrote:

> Results example (clipped) is in the docs. Not sure if that works, though.
>
> What about simply a nice photo of a gauging station, say a CC-licensed
> images like this: https://flic.kr/p/bbWTXv ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenHydrology/publications/commit/a3bda91574c4d0cfa69be933de51d51dfae185ee#commitcomment-10799647>
> .
>


üyelik girişi eklendi.


Thanks @dscho & @ctrueden !
Tested it, works fine. I therefore removed the file from this PR from our update site.


Yes. X9180 and ZTE, upper case.
With lower case some applications in google play (such as real racing 3) may be unavailable.
Look into build.prop from stock firmware: http://pastebin.com/R0zDWaBp


@matthewvita, you're right. Since infinity scroll prevents the user by
design from ever hitting the footer,  that was a hack to play around with
the idea of an absolutely positioned footer. Can do it in flexbox, but as
you've found, it's a less than ideal implementation otherwise.

That should not have made it into the repo, though—I apologize. Will hop in
today and clean it up.

On Sun, Apr 19, 2015 at 10:00 PM, Matthew Vita <notifications@github.com>
wrote:

> @cameronscott137 <https://github.com/cameronscott137> this is a hack to
> get the footer to look right... basically, there's 10ish pixels of space
> between the bottom of the footer and the bottom of the available browsing
> area. I noticed this with the HTML/CSS you handed off and with how it's
> implemented in this repo ATM.
>
> thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saxifrage/CaaC_Explore/commit/a246e83868d6227761da4dce8ee9a959dff3a6ba#commitcomment-10795111>
> .
>

--
Cameron
(610) 883-7342


I've no interest in being first author.  Also I've decided I will put MML down as my organisation in which case it will be easier to defend if I am last author.


Wouldn't help, same LoC and cast still needed due to FJT being invariant?


Lower case 'L' in `highLight`? :trollface:


Never mind. I understood it. Just had to re-read the paper once ;)


Hi @nylen. I don't get completely what's the main purpose of `_publicAPI` property.\Can you explain it please ?


AipoLocalization_ja.properties
では
USER_BROWSER_ADD_ACCOUNT_EXPORT_VM
になってるので、どっちかに合わせる感じかと思います


From the background of today's discussion, I wonder if we could replace _wait_and_dispatch_one_signal_ by busy checking a kind of interrupt-request bit during initialization or any other non-special handling.


Ah yes, _wait_and_dispatch_one_signal_ again...


@bparees any ideas here?


@bparees and here?


@jreback Is there a reason you deleted this?


Maybe makes sense to add `primary_key_value`?


Ah, I see, because it was in there twice .. OK!


Curious. Let's look at it at lab today.


This looks complicated, the patter above seems much nicer to me:
```
	printed = snprintf(ebp, env_sz, "%s=%s", ce_d->name, val);
	if (env_sz <= printed)
		...
```


ditto


Stop doing that.


I don't think MSM8x10 family devices are working yet. No big surprise, there would probably be some more significant work needed to backport the kernel. I'm not sure if I'll actually bother, or just use a 3.4 series kernel.


yep a couple of issue number typos

btw not sure how hard would be to have the doc build process spit out duplicate issue numbers (so we can look at them at least)?


@sbe plz review. @brylie unfortunately, i used "ger" instead of "de". i tried to change all "ger" instances to "de" in mfAll.js but it didn't work.


Then why are you doing it.
On 20 Apr 2015 13:36, "BrainMaestro" <notifications@github.com> wrote:

> Stop doing that.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BrainMaestro/FCE/commit/218a683c290787fbc84df0b97ee2460f9f9d6f52#commitcomment-10801975>
> .
>


Move star


lnvm->nvm


same


Which ?


to można ładniej zrobić enumem imho.


Alright. Would you consider starting from scratch? The reason for this is twofold:
* 'de' would be better to use, as it conforms to ISO standards
* I changed all of the translation keys last sprint, and these translations are using the original keys


Remove this branch. Discard is handled by lightnvm


Where the hell did that come from? It's not in my local commit :X


Remove this branch as well


Make copy and define it statically


Looks fine so far. I would stick to either 'Schlagwörter' or 'Schlüsselwörter'
I think 'Schlagwörter' is more appropriate.


It would be a good strategy to copy all of the translations to a second document, start the translation process anew, and paste them in to the empty 'de' translation. This way the duplication of work is reduced.


Well... It is my fault. At first I thought the value parameter should be able to customize. (Seems I need to review everything about tfc anvil.)
Another thing is that there are some constructors which extends from the same constructor. What I wanted to do is to add the base one so that a modpack creator can customize the property of their recipe.
Whatever, I'm going to handle it asap.
@Kittychanley


Heh, looks like attractive solution to circumvent _wait_and_dispatch_one_signal_ ,-)


Ah ! That's what I was thinking ^^ otus, falcon, peregrine, condor and thea can't get an 3.10 MMi  support using this as base for now. MDSS need patching, dts needs adaptation, and many other things.


hmm... przekombinowałeś imho.
```
std::string intToString(int n) {
    std::ostringstream oss;
    oss << n;
    return oss.str();
}
```


oh ya, I forgot to commit that. Thanks. Will try again.


Oh, missed that indeed, thx for the fix!


:heart:


:see_no_evil: :)


@ciaran its wrong file to add mixpanel track for `sign-in` and `create-account` because this file `init` when user open user-account page.


Why not just `subscribe(subscriber, event_types)` ?


Jeff is not setting the host in the routes for the web console. Since we have to do with multiple ports now I guess this should be set, right? @smarterclayton @bparees If so, what's a host to a route here?


Yay!


talk to @jcantril for both of these, he's done route generation in the webconsole already.


I wonder how the custom headers we set for the OPTIONS requests (Authorization, Content-Type) will be handled by rack-cors. I expect you'll need to specify them again, using [their `headers` option](https://github.com/cyu/rack-cors#resource):

> headers (string or array or :any): The HTTP headers that will be allowed in the CORS resource request. Use :any to allow for any headers in the actual request.


This should probably be `Seller.name`, rather than `Seller.seller`


aaa


Better, "ap" aliased to "master" -- as in man pages:
  Master  (the  node  is  the  synchronisation  master  or  acts as an Access Point)


test


@snd1 No worries, as long as you don't touch master :wink:


Ahah, you actually did use `headers => :any`. So tiny I overlooked that.


fajne rozwiązanie.
w praktyce twarze, których nie złapie przy obrocie rzędu 30-45 stopni już nas nie interesują, więc można to jeszcze ograniczyć.


This is going to always raise an error - I think this was testing code that was incorrectly committed.

The structure should be:
```
Profile.transaction do
   profile = Profile.create!(....
   if associated_user?
     User.transaction do
       profile.create_user( ... ) || raise ActiveRecord::Rollback
     end
   end
end
```

This way:
* we will raise an error that rolls back the Profile if the user save fails
* we won't start a transaction on User if there's no associated user


You *could* have re-used the code from 3 lines above... ;)


This variable is deleted since 2.1.0-alpha1 #1222


(slowly coming back to this code) This is for children added to _another_ container. Do we have something here that I could use instead?


Hi, this removed the last use of 'tr' in this file. Please check for unused import #pylintpolice.


You don't need these fields since you can store what you need in the `name` field, and then look it up using the record type, eg: get the `name` field in all records whose type is `seller`.


@karimFarid leh $rt men 20:16 w $rd men 15:11 howa elmafroud el3aks !!
y3ni elformat Opcode,$rs,$rt,$rd,Shamt,Functioncode


ok


I would have created two separate commits: one for rack-cors, another for the `users#create` stuff.


imho bardziej elegancko by było to zrobić ranged for'em. nie potrzebujesz w tej pętli i.

```
for (auto const& angle : angles) {
    ...
}
```


heh heh,


Bar.baseClass is `undefined`.  I'm not sure how to get a hand on baseClass for a pointer to Bar.


Using [`Find`](http://ruby-doc.org/stdlib-2.2.2/libdoc/find/rdoc/Find.html) for this is more convenient.


I have attached a DTrace script I used when debugging the NFS mount work.
If you could run this it should help to narrow down what is going wrong
during your mount and then we can try to do more investigation from there.

You would run this like so:
/native/usr/sbin/dtrace -q -s nmnt.d -c '/sbin/mount.nfs
ub64:/jjelinek/public /home/jerry/mnt'

Substitute your server:path and mountpoint appropriately. In the command
line above we want to trace the NFS mount command itself and not the
generic mount command wrapper.

Thanks,
Jerry

On Sun, Apr 19, 2015 at 1:47 PM, Jorge Schrauwen <notifications@github.com>
wrote:

> Any doc on how to do this? I have tried and failed to mount nfs and cifs
> shares in lx branded zones.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joyent/illumos-joyent/commit/7652b10523b05b4d89e5deb010e8b0ea590ceb7d#commitcomment-10793757>
> .
>


Oh, I get it. It's like `Array(event_types)` :)


@buzzy Consider passing `, env: env`.

Odds are this doesn't matter here because everything was consumed during `./configure`, but unless we validate that assumption, we shouldn't rely on it.


@cameronscott137 my understanding is that the infinity will only be on a few pages and the footer on all pages.

I like your idea of using flexbox... a great tool for the job!


Un `div` no puede ir dentro de un `p`.

No te parece la idea del `panel-footer`?


This works for me, as a basic level of completetion (though argcomplete shouldn't be in the standard library imports section)


Host should not be needed - you automatically get a name on the router.

> On Apr 20, 2015, at 8:45 AM, Michail Kargakis <notifications@github.com> wrote:
>
> Jeff is not setting the host in the routes for the web console. Since we have to do with multiple ports now I guess this should be set, right? @smarterclayton @bparees If so, what's a host to a route here?
>
> —
> Reply to this email directly or view it on GitHub.
>


No need for this


@buzzy ,

This will actually build and install all of OpenSSH (install is dependent on most other targets). It'll also generate a host key.

From `Makefile.in`:

    install: $(CONFIGFILES) $(MANPAGES) $(TARGETS) install-files install-sysconf host-key check-config

Ideally, you'll want to install `ssh-keygen` manually, so that we don't pull everything in. In `Makefile.in`, this is done with:

    	$(INSTALL) -m 0755 $(STRIP_OPT) ssh-keygen$(EXEEXT) $(DESTDIR)$(bindir)/ssh-keygen$(EXEEXT)


agree, but there are places where `libraries` is used already.  i've updated docs to say only `brooklyn.libraries`, but am leaving the check here to allow the older syntax (with a TODO).


这个表述我没什么意见。就是需要统一一下scheme的译法，@yarray 你觉得色系更合适，还是配色方案合适？


> I looked over this quickly. I'm wondering about the bulk of the code in Bar.js, which is to wrap each child in a <div>. Is that needed to support TabBar, and if so, why?

That is needed for the table layout to work as expected in both vertical and horizontal mode. Another solution might be to use flex layout but that would be more complex in terms of CSS and not working on all browsers.


GCC on our servers is too old to compile libsass, so I fear it will have to wait until post redesign


@buzzy

You'll want to use `#{install_dir}` here.


Do we need to exclude the picture dump here or is it only generated to be included in the full dump anyway?


I blame gnome-terminal. (and the fact I wrote this like 4 years ago)


Utilisez les Issues pour communiquer plutôt que de créer un fichier et de commenter les commits. C'est difficile de retrouver les discussions après.


@buzzy

I'd rather we don't pull in all of OpenSSH considering it's rather easy to just get `ssh-keygen`, which is the only piece we need.

Can you rename this to `ssh-keygen`.


I applied the patch @twizmwazin, but it says that I commited it:

![](http://i.imgur.com/ew5Ghok.png)


@buzzy

You can probably remove this one — we didn't get anything from Chef here.

Cheers,


I think it's
root "home#index"


@karimFarid aiwa tamam (Y)


Done


zaczynam się zastanawiać czy nie poszedłeś za daleko z tymi wyjątkami.
"nie ma twarzy = wyjątek" to już trochę hardcore. czy to rzeczywiście jest sytuacja, która nie powinna wystąpić i powinna wygenerować wyjątek? zauważ, że w przypadku działania programu z kamery, ta sytuacja może występować bardzo często.


@buzzy

Do we really need the above and this one?


also AGENT_TOKEN and AGENT_NAME for using [Agent Custom Token Authorize Plugin](https://confluence.jetbrains.com/display/TW/Agent+Custom+Token+Authorize+Plugin)


Or even `Dir[Rails.root.join('public/data/zip/*.zip')]`.


Hola

It looks like some tests are failing (a flag rose in my office)


谢谢哈


@wburns remove conflicts above


splitted the changes


Checking for file age shouldn't be neccessary.


rather use 'isinstance' than 'type is'. #pylintpolice


You have not imported that right? I think this actually breaks things.

I know they are defined in wrappers but in most places they are imported through ``galaxy.tools`` - which is valid.


@buzzy ,

You don't have this included in the project, so it's not going to be included in the build.

Also, once you've included it in the project, let me know: I'll need to upload this to the S3 bucket we use for git caching before you can use it in builds.


Nice one.


Nevermind - I hadn't noticed 27424139a01d80df0ba85a795844d920dd75470e.


I think so, thanks Andres


Ik hou je in de gaten hoor


oh crap, I forgot that flexbox doesn't exist in IE9 http://caniuse.com/#feat=flexbox and the existing Polyfills are a misnomer, so we may have to implement a lowest common denominator solution.

![](http://media.giphy.com/media/TZBED1pP5m8N2/giphy.gif)


to be precise,  366 is only partly fixed


Edited browser link to become a browser that doesn't fill up with frigging ads


Putting the new code into a separate method would also be nice.


Changed text to be a bit more accurate


Still file description is missing.


test


Tak, ale ten wyjątek nie powoduje zatrzymania programu, tylko main go obsługuje w sposób: "Wypisz, że w danej klatce nie znaleziono twarzy. Pobierz następną klatkę". Metoda jest typu FaceData, więc chyba bardziej elegancko jest rzucić wyjątek, że nie znaleziono twarzy niż zwracać jakieś puste FaceData i potem to ifować w mainie?


No need to mention this comment "Allow to set a user premission" because hook are already define by drupal core so everyone know what it does.


Wrong indentation.


No tak... Nie przywykłem zbytnio do używania c++11 (choć staram się przestawić), więc zapominam czasem o takich ułatwiających życie możliwościach.


I meant that the caller should've sorted it out already - putting the concatenated libraries in "brooklyn.libraries" in meta. Not that important though.


hook function description always ended with parenthesis


Następny commit zmienił kąty na 35 i 70. Zauważyłem, że w praktyce OpenCV ma problem z wykryciem twarzy odchylonych o więcej niż 30 stopni w którąś stronę. Stąd pięć kątów zamiast trzech.


Why the lower case "aborting"?


@yehudasa switched here from _ERR_PRECONDITION_FAILED_ to _EINVAL_.


No need of blank line.


Mozilla/5.0 (PLAYSTATION 3 4.66) AppleWebKit/531.22.8 (KHTML,like Gecko)


~REVERT CHANGES~


Did you confirm the result of the following command?

```zsh
% rake reference:generate
% firefox doc/reference/en/file.tutorial.html
```


我感觉还是不必忠于原文吧，可以意译。其实我已经在多个地方大踏步的采用意译并适当的扩展内容了。“叙事美学”这么高大上的词出来。。读者还是不太容易接受吧，不如就直接说：

> 色彩搭配方案多的浩如烟海，它们可以帮你做出更明智的选择。

你觉得如何？（我怎么这么喜欢“配色方案”这个词呢。。）


pahahaha get reverting kayjay


They are fixed in a subsequent commit.

https://github.com/infinispan/infinispan/commit/7837d3bdf76fa9b4777a3da455b2075354ac9f85#diff-93ec943e00794c26478270e3ad763d46R322


Wrong indentation in whole function.


A to miałem zmienić. Ba, jestem prawie pewien, że zmieniłem. Gdzieś mi musiało uciec przy cofaniu commitów...


我觉得都行，略倾向于色系因为字少…… @tumluliu


@andyjmaclean, @Luthien-in-edhil: please note the stack upgrades in this merge.

After pulling changes to your dev environment, you will need to install Ruby 2.2.1 (with `rvm install 2.2.1` if using rvm) and run `bundle install`


You don't need the square brackets here and in many cases below. Also in the case right above this one, I think.


@semdoc ping me on hangouts.
I'll delete this later :p


Now using the original `composer.json`; currently at 0.4.*. :smile:


ah sa7. @AmrEmara


OK.  Is the `body` parameter optional?


Poor space, to find out what GREL is about ..... ;)


The only one reason is my knowledge gap ;)

So yes, you can use six.PY3 to perform the check


Ground-troops already tracked and traced the remaining resistance and took the necessary measurements to eliminate every trace of its existence ;).


now that the workshop is over, can you provide your answer to this final exercise for final comparison?


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


ahi forgot to delete the swing i was trying something out and when i finished i forgot to remove the swing for the button and label


@hzsweers @fadils Some examples for referencing icons via attributes:
- The android settings app: https://github.com/android/platform_packages_apps_settings/blob/master/res/values/themes.xml
- https://github.com/sreichholf/dreamDroid/blob/master/app/res/values/attrs.xml
- https://github.com/cjbrooks12/LogicPlayground/blob/b85b47b6b9ab864bf73c57489b066f565c67f0e7/app/src/main/res/values/attrs.xml
- more examples: https://github.com/search?l=xml&q=attr+name+ic&ref=searchresults&type=Code&utf8=%E2%9C%93

This has the advantage that the icon depends on a theme, and can be changed via theme (white icon on dark background and dark icon on light background).
So the menu/*.xml files are independendent from knowing thier (background-)color.


Can't you use empty function here ?


This classifies more to ``log.warning`` I think.


I am not sure about it because I don't know if the case is due to "Python2 multiple exceptions in traceback" (I don't know if it supports this feature, if that's the case, the comment should be updated), or to "locat exceptions array contains more than one exception"


It will throw an error, if $label is empty


Do you have a plan how to handle these dropped dependencies in the installation images? With this they would no longer be included in the installation images. A patch for lorax, maybe?


Is it legal to map a string over a seq? I'd expect this to be something like:

```
(map :name people)
(map #(% "name") people)
(map #(get % "name") people)
```


Yes now that 10.8 is the deployment target I'll change all unsafe references to weak. Thanks for pointing it out!


What's a Cortex A6? For the arm 6 Architecture, [Wikipedia](https://github.com/roboterclubaachen/xpcc/commit/58c934637c5e74a1e212b1defee2c77631b7db17) only lists cortex M cores.


zamiast if'a masz try-catch, więc moim zdaniem na jedno wychodzi.
ale jak spojrzysz na to z punktu widzenia kogoś kto ten kod widzi po raz pierwszy to ciężko się domyśleć dlaczego ta funkcja rzuca wyjątek. a zawsze powinieneś zakładać, że ktoś inny będzie kiedyś pisał coś w oparciu o Twój kod.

alternatywnie można zrobić to na kilka sposobów np użyć optional http://en.cppreference.com/w/cpp/experimental/optional albo wektora FaceData


This should be applied to UserMailForm aswell ;)


可以用 es6 ...

()=>{
}


Is this `(main)` call necessary? If so, it might be good to include this in the first core.cljs snippet.


Fixed.


Thanks for the tips! See d4d1072


good idea


Is there a reason to use `.-response1` instead of `:response1` here? I believe both work, but the keyword is more typical ClojureScript.


Unused import. #pylintpolice.


(368c887) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


Yes is mandatory, and you are right about the need of add it on the first snippet.


Wouldn't it be safer to use <= here?    What if, by some scheduling glitch, this thread never sees the case where the counters are exactly equal?


coś na miarę
enum half
{
    left = 0,
   right = 1
};
?
Czy to nie trochę przerost formy nad treścią?


The `:closure-warnings` option is introduced later in the tutorial. Is it reasonable to remove it from this example to keep devel and prop configs as similar as possible?


Clearly is a mystake! ;) Your proposed solution is the right way.


I'd say that linear algebra is the best place for the package to go for now.  Thanks!


test commit


@mseaton did you update this via Transifex as well? LMK if you need a Transifex account.


I agree with your proposal ;)


If this is not supposed to be inherited from the ``Method`` class, then I don't know why and it should be clarified in the comments/docstring.


The lack of masking wasn't hiding the overflow bug, the overflow bug was hiding the lack of masking.  Since a "normal" entry has both bits set, the entries were always 0xcxxxxxxx and since we were doing 32-bit math, the 0xc part was getting dropped by the 32-bit arithmetic.  As soon as I aded the cast to convert the math to 64-bit math, our unit tests failed because the calculated offset was out of range due to those high bits being set.  Fixing the masking issue corrected the math and our unit tests pass again.

The code you reference at lines 1393/1389 will indeed return if *one* of the bits is set, but the normal case is encoded by having *both* bits set, which is the case my change addresses.


需要支持  ie8


Sorry about the delay. This looks fine to me.


How about actually using `golang.org/x/sys/unix`? I heard that `syscall` package is deprecated. I should be a simple matter of `s/syscall/unix/g` I think. Godocs - https://godoc.org/golang.org/x/sys/unix


I'm concerned about using the same output path and filename for `devel` and `prod` builds. I believe it's best practice to give them unique names.


It generates no graph at all with the files in test/golden_buffers if I do -c all, and generates a bunch of horizontal lines if I try to do -c 0,1.


Function description is wrong.


Wybieram optional. Fajna sprawa!


Wrong indentation.


Hmm, I'm not clearly remember this, in cljs something like this are legar (with difference with clojure)...


bower should be global just like gulp-devtools


I'm not clearly understand you.


Yes, I also receive the Exception from shadowForm.


@vbatts

Ok, i got what you are saying. So instead of checking for library version, presence of a certain symbol should probably be sufficient and if that symbol is present that implies that library supports deferred removal.

This will be broken only if library had a bug where proper supports came in later versions while symbol was introduced in previous versions. As I am not aware of any such thing, I think what you are suggesting makes sense.

I will make changes and get rid of code doing library version checks.


mousemove 没模拟？


I don't know why the patch didn't work, but this commit was created by @twizmwazin and belongs to him.


Oh, look what we have here! Didn't this work? :D


Indeed, if you know the values of those constants then you know you can just OR in BTT_MAP_ENTRY_NORMAL.  One could argue the code is written the way it is so that it works without knowing the values of the constants, that is "clear off the E and X conditions and set the NORMAL condition".  But I do see your point.  I didn't write that line but when I read it I understood what it was doing pretty clearly so this patch leaves it alone.


This is only included in the full dump iirc. At least I couldn't find any way to download only the pictures for a single phenotype or for all picture phenotypes :D


Esperanto? :)


Gone!


Will give it another shot later, just wanted to make sure I needed native-mount, I tried way earlier and back then it was broken. Will let you know once I tried it.


I just did. Thanks.

However, this baseline hasn't changed in some time, so I wouldn't hold your breath.


Two minutes timeout sounds excessive, now that you don't need to download anything...


Python has the ``all()`` function that can be used here instead of ``not any()`` to make this much better readable.


last week we had some problem adding new users in accessed.php. It seemed like the database stored both first- and last name in one column ("last name column") and a NULL value in the "first name column". It was because we added users in an incorrect way. In this branch I've tried to make it more clear to people how to correctly add users.


Can't we use empty function here.


(ignore this, this commit was part of a different PR - will be all nice when squashed/rebased)


Oh, no I didn't.  I probably do need an account


You forgot one...


Some leftover from resolving conflicts, I guess?


Moved from the "Code Generation" to "Interpolation".


Schemas are now located in schemas/iso193139 module.

Please do not commit schema plugin files from the datadir. Dropped https://github.com/geosource-catalogue/core-geonetwork/commit/b0535e71f39b164856a4ca787e1fe27cce914d6c


I think there's a syntax error.


While testing the SQL queries I got:
```
ERROR 1146 (42S02): Table 'x5dat.email_virtual_domain' doesn't exist
```
... the table is email_virtual_domains, with an s.


This should be `@($(SUPPORTED_KERNELS))` instead of `@(SUPPORTED_KERNELS)`
It's my bad ; sorry for misleading you.


Same here


move up 2 lines to make sure it is done even if the tryOn things throws


same here


Note: The ```dispatcher.fromGameController(board)``` sends object to ```gameViewController``` which invokes ```postInvalidate``` method. I wondered if it is ok to call this method when the view is not attached to window because of what the documentation says: "This method can be invoked from outside of the UI thread **only when this View is attached to a window**.". I inspected the source code of this method and it does nothing when the view is not attached.


Perhaps the exception can say what node/value it failed serializing?


Fixed, thanks.


This means you limiting plugins names to Bipbip prefixed?


that's strange since chocobo has 100% nothing to do with onzonein method

maybe it's related to loading the inventory? doesn't make sense regardless


Well it's unneeded now :)


No need for exporting `SleepCmd` member here. `sleepCmd` should be fine.


It is supposed to inherit from Method.


Yes they will.   Do you see any alternative?


What is the reason of this re-instantiation?


Yes, I forgot to delete files after cherry-pick


That's right. But it is checked both ways:

1. trying to set `verify_mode = CERT_NONE` checks for `check_hostname == False`.
2. trying to set `check_hostname = True` checks for `verify_mode != CERT_NONE`.

Where by default `check_hostname = False` and `verify_mode = CERT_NONE`.

Case 1. is prevented by making sure that function param `check_hostname` is always False when cert_reqs is CERT_NONE (it's down below, at :150).
To prevent case 2. we have to set `verify_mode` before `check_hostname`, otherwise this happens:

```python
In [1]: import websocket

In [2]: websocket.create_connection("wss://echo.websocket.org").status
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-2-f86ccfe5ac08> in <module>()
----> 1 websocket.create_connection("wss://echo.websocket.org").status

/home/kenji/.virtualenvs/snipy3/lib/python3.4/site-packages/websocket/_core.py in create_connection(url, timeout, **options)
    102                         skip_utf8_validation=skip_utf8_validation)
    103     websock.settimeout(timeout if timeout is not None else getdefaulttimeout())
--> 104     websock.connect(url, **options)
    105     return websock
    106

/home/kenji/.virtualenvs/snipy3/lib/python3.4/site-packages/websocket/_core.py in connect(self, url, **options)

--> 253         self.sock, addrs = connect(url, self.sock_opt, proxy_info(**options))
    254
    255         try:

/home/kenji/.virtualenvs/snipy3/lib/python3.4/site-packages/websocket/_http.py in connect(url, options, proxy)
     67         if is_secure:
     68             if HAVE_SSL:
---> 69                 sock = _ssl_socket(sock, options.sslopt, hostname)
     70             else:
     71                 raise WebSocketException("SSL not available.")

/home/kenji/.virtualenvs/snipy3/lib/python3.4/site-packages/websocket/_http.py in _ssl_socket(sock, user_sslopt, hostname)
    151
    152     if _can_use_sni():
--> 153         sock = _wrap_sni_socket(sock, sslopt, hostname, check_hostname)
    154     else:
    155         sslopt.pop('check_hostname', True)

/home/kenji/.virtualenvs/snipy3/lib/python3.4/site-packages/websocket/_http.py in _wrap_sni_socket(sock, sslopt, hostname, check_hostname)
    127
    128     if HAVE_CONTEXT_CHECK_HOSTNAME:
--> 129         context.check_hostname = check_hostname
    130     context.load_verify_locations(cafile=sslopt.get('ca_certs', None))
    131     context.verify_mode = sslopt['cert_reqs']

ValueError: check_hostname needs a SSL context with either CERT_OPTIONAL or CERT_REQUIRED
```

IMHO it is badly thought out in the stdlib, but we have to live with this.


@wburns yep thanks I already checked


Removing the last use of Qt here. Please check unused imports. #pylintpolice.


I really need a nap...


I figured that. I was just wondering whether we have a problem with our scheduled jobs. ;)


Gone!


Yes!


Oh yes, thanks. Forgot that these throw.


Maybe with a method `values.map.addBackgroundLayers(...)` which adds the unparsed layers to `values.map.layers`, parses the new layers and then directly adds the parsed layers to `values.map.mapLayers`?


It seems like an extremely easy change on resin-vcs lib to add it a getApplicationName()

It's basically getApplicationId without the last line. Maybe make the change there, use vcs.getApplicationName here, and remove the TODO?

Otherwise you'll have to do another PR here for the same issue. And too many callbacks.

Unless it's more difficult than I think


Ok, I will change it to 1 minute.


Acc1 ..
Mantap, jangan lupa buat laporannya yh ...


Where did you hear that `syscall` is deprecated? Using `golang.org/x/sys/unix` is annoying because it would need to be vendored...


This is the last use of get_postprocessor_human_name. Please check unused imports. #pylintpolice.


Afternoon Ed - hope you're good?

Quick question: I've tried moving this into a module in Models folder so I can use methods like this across models, though it's giving a no method error at present. You'll see this in the commented out lines of task.rb and named_scopes.rb. Is there something simple I'm missing that will get this working?


Can you update the pip package with the last commit? The install by cloning the repository is not working for me...


I think it's just info. The client may have deliberately chosen not to install, say, lvm, so it is not an error to them. And blivet is supposed to be able to handle the missing plugin.


Whoops yeah, this was a mistake. It does repro the "URL too long" bug, but it's clearly wrong.


Will this still be implemented as a before_save filter? If so, I'll get working on refactoring.


地图汇上面的这个说法靠谱啊，不错，就用它吧


if the current call to super.aroundReceive (user code) called `cancel` we must not `request`
this worked previously because of the `canceled` flag was checked in `request`


Gotcha - will work on this now.


@al-martinez done with my review. checkpoint complete.


I'll work on moving this across to the controller.


I have no idea whether they ever worked :P


Can you sign up for one at www.transifex.com and let me know your username and I'll add you to the project.


Should be `});` as now a function call and not an assignment


I'm very sorry to waste your time but there is an error on this commit.
Translations are already managed by Tools::displayError, please revert back.


but this must be delayed until `OnSubscribe` if `subscription.isEmpty`
I don't think you can remove the `canceled` flag.


我倾向于意译，而且那应该是32种颜色，不知道写原文的是怎么数的。。


Fixed with the latest commit: https://github.com/svaksha/Julia.jl/commit/12601212597a369e722d1a9344c5ce40e248a55a


rzeczywiście, trochę overkill.
ale z drugiej strony takie defines też nie wyglądają szczególnie elegancko.
może da się to rozwiązać jeszcze inaczej?
np w detector.cpp pętlę 92-119 przenieść do osobnej metody i wywołać ją dwa razy z parametrami.


@zamar: I vote for commiting this if it passes. It's not so much the code simplification, but the myth debunking aspect of it that I like. People have been believing for years that MVV/LVA is the right way, just because they have been told so by talkchess folks, or because they saw other engines doing it. If we prove that the LVA part is useless, it's good to get rid of it. It creates a vacuum, and the vacuum is what forces people to think of other solutions to fill it.


@laouji yes, please set it to `tile_foot--2` so I can see the problem -- if it's happening foryou, it will likely happen to others, so I'd love to fix it!


I logged in with my github account.  mseaton


Thanks @nolanlawson!


@yjun2 done with my review. checkpoint complete.


This gets succesfully compiled as ```@import url(http://fonts.googleapis.com/css?family=Roboto:100,400,300,700|Sintony:400,700&subset=latin,cyrillic-ext,cyrillic,latin-ext);``` on my end.
Do you get an error on compile? Which less compiler are you using?


[This](https://github.com/mperham/sidekiq/wiki/Scheduled-Jobs#recurring-jobs) also looks interesting:

> If you are worried about the overhead of booting your Rails app to push jobs, you can create a stripped down job type that just knows how to push a job to Redis without booting Rails:


Never.


thank you


Yes, you're right - did not think through all interactions. I'll polish up and add some tests.


Deprecated Facebook FQL API


thanks


@lxnay additional specifications in which groups portage must be in order to use mknod?


original version


Please watch white space before ':'. #pylintpolice.


groupId and version can be inherited from parent POM. These changes simplify version management as we'll need to update the swagger-core/pom.xml only when switching to another version.


20.04.2015 geupdated


Yes, I'm getting and error there:
```
NameError: variable @font-subsets is undefined
in theme.less on line 19, column 9:

18 // @import url(http://fonts.googleapis.com/css?family=Dosis:200,300,400,600|Signika+Negative:300,400,600|Exo:300,400,600|Roboto:100,400,300,700|Sintony:400,700);
19 @import ~"url(http://fonts.googleapis.com/css?family=Roboto:100,400,300,700|Sintony:400,700&subset=@{font-subsets})";
20
```

Compiler is `lessc 2.4.0 (Less Compiler)`.


@bnolan jason dot gullickson at gmail dot com :)


Attention, ici il faut revoir la moyenne. != 80


ok kak..jdi asistensix via github ti d print ji jg laporanx kak gitu,,,ok mkasih kak


[@RequestMapping](http://happyourlife.tistory.com/65)에 대해 알아보니, 'headers = ' 방식으로 content-type 을 설정할 수 있더라구요. "/json"처럼 패쓰가 아니라 이렇게 요청한 헤더에 따라 응답 데이터를 다르게 한다면, 굳이 컨트롤러를 따로 두지 않아도 될 것 같아요.


@cal101 why?


On devrait rajouter un validation set pour faire une rotation pour trouver les paramêtres.


So did this also fix #394?


@kimmobrunfeldt


Phil,
do you think the second added line (line 26: RewriteRule ^/?(.*) https://%{SERVER_NAME}/$1 [R,L]) works for the environment of vagrant that uses the port 8888?


TESTQA


ignore => disable


'issueStatesInProgress' could be made configurable as array over `application.properties`. Reasonable defaults can be set in dash-core version of config file.


trust me guys. we don't need it


Looks good to me. Could you you add a test case for this? Thanks!


I'm happy whatever, Just had a read through. It's good. Submit when you're
ready - four days early!

On 20 April 2015 at 13:29, Neil Nutt <notifications@github.com> wrote:

> I've no interest in being first author. Also I've decided I will put MML
> down as my organisation in which case it will be easier to defend if I am
> last author.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenHydrology/publications/commit/f348636f15da0372ed9a796b968df336c802a6c0#commitcomment-10801879>
> .
>


Je like, joliment fait.


No problem! I have reverted it back.


C'est pas fini je pense ? (pas de return et le code semble pas fini)


Hi,
app/code/core/Mage/Core/Controller/Request/Http.php is missing:

diff --git app/code/core/Mage/Core/Controller/Request/Http.php app/code/core/Mage/Core/Controller/Request/Http.php
index 6513db9..31eb6d6 100644
--- app/code/core/Mage/Core/Controller/Request/Http.php
+++ app/code/core/Mage/Core/Controller/Request/Http.php
@@ -76,6 +76,13 @@ class Mage_Core_Controller_Request_Http extends Zend_Controller_Request_Http
     protected $_beforeForwardInfo = array();

     /**
+     * Flag for recognizing if request internally forwarded
+     *
+     * @var bool
+     */
+    protected $_internallyForwarded = false;
+
+    /**
      * Returns ORIGINAL_PATH_INFO.
      * This value is calculated instead of reading PATH_INFO
      * directly from $_SERVER due to cross-platform differences.
@@ -534,4 +541,26 @@ class Mage_Core_Controller_Request_Http extends Zend_Controller_Request_Http
         }
         return false;
     }
+
+    /**
+     * Define that request was forwarded internally
+     *
+     * @param boolean $flag
+     * @return Mage_Core_Controller_Request_Http
+     */
+    public function setInternallyForwarded($flag = true)
+    {
+        $this->_internallyForwarded = (bool)$flag;
+        return $this;
+    }
+
+    /**
+     * Checks if request was forwarded internally
+     *
+     * @return bool
+     */
+    public function getInternallyForwarded()
+    {
+        return $this->_internallyForwarded;
+    }
 }


If the thunk has never been called, then it seems like the "key" wouldn't be cached yet.


Please note the url you gave,   you are pointing to my fork,


I am not sure if this is actually needed; drivers are bound to interfaces and interface numbers are sequential anyway; this way you are visiting each interface more than once, once per altsetting.


Same here,  you don't want to point to my fork


Importing an unused function 'get_string'. #pylintpolice.


commiting


This kind of assumptions keeps coming up, see for instance `forall_intro` in
https://github.com/FStarLang/FStar/blob/master/lib/classical.fst
or
https://github.com/FStarLang/FStar/blob/master/examples/metatheory/norm.fst


I reverted to the older syntax, until I find out why this isn't working on lessc.
Should work now.


Thanks.


I still think that it would make sense to use the automatic detach and reattach mechanism here, so that the kernel driver gets reattached when libhackrf closes the device.


Checking the return value here actually makes sense, thanks. I'll add this check also to my projects.


Static Constructor := "Array"


```detach_kernel_driver``` can be called ```detach_all_kernel_drivers``` because it possibly detaches different drivers bound to different interfaces (see http://sourceforge.net/p/libusb/mailman/libusb-devel/thread/20150410104847.de274ef48853d611aba23da1%40ao2.it/#msg33764727)


OK. My approach will be to narrow down the list to the major,
forward-looking points. It's not worth bickering over things that are
annoyances of implementation.

# [Dan Gunter](http://acs.lbl.gov/~dang)

On Sun, Apr 19, 2015 at 9:58 PM, cshenry <notifications@github.com> wrote:

> Thanks Shane. I agree. Ideally the document should be a product unto
> itself and not require an investigation of the PR discussion to understand
> the content.
>
> I don't mean to be stickler, and usually I'm not. I certainly leave plenty
> of holes in my own docs. But this document seems really important to me,
> because other design documents for next gen point to this one to motivate
> various design decisions.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/kbase/nextgen/commit/3d00a1de85179da2e65aa264ab7ce65b0704ba12#commitcomment-10796057>
> .
>