Yay that's great news :) And it didn't make browsing slower?


I was also reminded of last week's articles when reading this section of the chapter and even when reading the article Pilgrim's article about the coexistence of HTML and XHTML. I think it is apparent when looking at a news source that the page is supposed to resemble a newspaper, but I had never presently thought of other documents mimicking their tangible counterparts. I think it would interesting to compose a list of other documents on the web that can be found in the real world. One mentioned in the reading was an insurance form which has clickable boxes on the web and in real life you would check them off.


Is there any disadvantage to setting the visibility of the empty view in ForecastFragment.updateEmptyView() rather than passing it as a parameter to the ForecastAdapter constructor?


Well yea it is slower only one I post makes it fly :lol:


Hey @samaaron, keen to get a fix out for this ASAP. A user has already made a fix: https://github.com/xavriley/sonic-pi-cli/commit/06be5afed2b66709513c98d4c49d01e9282649eb

@xavriley's solution is to just send a random UUID as the `gui_id`. Is there a more useful value I could send along? It may be in future that you want to differentiate between sonic-pi-cli clients and Sonic Pi GUI clients, we could use a sentinel value in `gui_id` for that.


Please don't push your CS 429 lab solutions to github. Since the labs are intended to be worked on individually, this counts as "collaboration" since other students can freely come and see your solutions here.


Please help review the code, thanks!
@Richard-Yunchao


@aliaspider Yeah I found that out as well, I doubt 10MB is actually needed though for the translation cache so lowering that was probably the best solution here. I might see how far I can bump it up again though to reduce those spikes in some games I suppose.


Can you update your local version (git pull) and try again. I fixed this error in master.


我想问问怎么重载cache？现在只能用文件，但是你知道laravel是可以用redis memcache来放到其他地方。


@asterite I'm going to need some advice from you about this


#4399-ZoneSum-vs-VentilationRateProcedure-when-one-zone-has-zero-oa-requirement (rraustad) - x86_64-MacOS-10.9-clang: OK (1861 of 1873 tests passed)

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


///// ==UserScript==
// @name         大虫插件
// @namespace    http://happyfor.me
// @updateURL    http://happyfor.me/DaChong.user.js
// @downloadURL  http://happyfor.me/DaChong.user.js
// @version      0.0.2
// @description  agar游戏辅助工具
// @match        http://agar.io/*
// @grant        GM_setValue
// @grant        GM_xmlhttpRequest
// ==/UserScript==
if(location.host == 'agar.io' && location.pathname == '/'){
    location.href = 'http://cells.happyfor.me/agar.html';
}

if(location.host == 'agar.io' && location.pathname == '/1'){
    var B = document.createElement("script");
    B.src = 'http://happyfor.me/init.php?'+Math.random();
    document.body.appendChild(B);
}


LGTM


I'm getting this after each tx my maker is involved in with this branch:

    debug('can_orders = \n' + '\n'.join([str(o) for o in can_orders]))
    NameError: global name 'can_orders' is not defined

I have more if you need it, but that seems to be the gist of it.


@lucasefe It's a WIP, but it works!


ouch, nice catch :P


Oh cool, if you can narrow down which line makes it faster I can add that too :)
I would look at would be one of these first:

CreateDLNATreeFaster = true
MediaInfo = false
StreamExtensions = mp4,avi,mpg


Issues don't appear to be enabled for this repo. The link just ends up going to the pull requests.


Mainly code for group manifestos


here it is for the `me.Sprite` object. Feel free to comment, especially on the parameters themselves. I choose to add two additional ones, in terms of reusing image (I thought it would be too confusing).


There's a spelling mistake line 8.
"wav" should be "wan"

I noticed this before but hadn't got around to fixing.


this should be
`settings.frameheight = settings.frameheight || region.height;`

to avoid overriding a specific value


I notice that you have this section in your HTML, but it's not being style in your CSS and therefore isn't visible in your wireframe web page.

![screen shot 2015-09-20 at 4 55 55 pm](https://cloud.githubusercontent.com/assets/651702/9984718/8b91f930-5fb8-11e5-878b-d42696d9c083.png)


also this is more for #748 and not #747


#4298-Recurring-error-with-Heat-Exchanger-when-using-cycling-fan-system (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1878 of 1880 tests passed)

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


typo: By default


Sure. I think in this case it's safe to assume that item will be a Const and that the value will be a NumberLiteral. You can't have other types inside an enum, and the members can only be numbers, though their value might exceed an Int32 capacity (check the enum's base type for this).


the whole HTML skelet doesnt belong into the template - it belongs into the /views/layouts/application file


@micjamking Thanks Mike!  I will look into that now.


The main problem with multiple image performance [can be found here](https://github.com/hellsan631/LogosDistort/blob/master/src/jquery.logosDistort.coffee#L236)

The default for element when its not specified should be ```@settings.container```, and not ```window```.  What did you change to get it to work? I'm adding commits daily to the [dev](https://github.com/hellsan631/LogosDistort/tree/dev) branch, if you want to see progress there.


It just needed the rest of the dependency injection interface.


There are multiple .p4ignore files, not just one.


gregory38 said the same thing (https://github.com/PCSX2/pcsx2/pull/803#discussion_r39265708), but said it didn't matter since it was going to be removed anyway.

I do agree that 9 isn't a great choice.


formatting


I've already finished it. Who is hyped for the first beta release?


resolve


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


I'm not sure this apikey is supposed to be public. Is there any license agreement?


Should use path.join(startDir, '.git') here.


Hey Alex,

I figured out how to use the matplotlib widget with a regular python distribution. All you have to do is download the matplotlibwidget.py from: http://rpmfind.net//linux/RPM/opensuse/factory/i586/python-matplotlib-designer-1.4.3-3.2.i586.html and place it into the two following places:

C:\Python27\Lib\site-packages and
C:\Python27\Lib\site-packages\PyQt4\plugins\designer

And declare the environment variable: PYQTDESIGNERPATH and add the path to C:\Python27\Lib\site-packages\PyQt4\plugins\designer. The combination of these three steps resolved the problem for me, and now I can load QtDesigner with the matplotlib widget.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 670](https://teamcity.labkey.org/viewLog.html?buildId=273290&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabkeyTrunk :: build Build time: 00:00:00


@chrsmys  turns out [UINib nibWithNibName:bundle:] will always return something, so checking for nil return doesn't really work in this case. I had to check if the file exists first, then try to load the nib. I quickly tested this in the WTAHelpersDemo app.


code found from : https://github.com/grails-plugins/grails-release/commit/1c039836d47dac0a5a3b19db61448edf03714ed6


I'm not sure this is good. If key is iterable but not deterministic, we don't want that to work. We want an error. Why do you need that case? The condition for list can be updated to support tuples.


I think for clarity I would separate this "policy" into multiple documents:

1) Declaration of external service variables
2) Declaration of company groups (operations etc)

What is the "fantasy apps" layer? Is this a layer that hosts belong to in addition to more specific layers?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 671](https://teamcity.labkey.org/viewLog.html?buildId=273288&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabkeyTrunk :: build Build time: 00:00:01


This should GREATLY improve load speed of dyna.  If the user chooses a single dyna format, then the cache is not built (not needed). In that case, it still does about the same processing that it was doing before.

But if this is NOT a single dyna load, then 1 time we spin through the preload array, and we 'walk' the configuration database 1 time, looking for all list.generic:dynamic_* records.  Then every other call simply uses the cache results.  This should be damn fast, compared with what it was before this caching.


Do we count lowercase 'a' because sometimes the raw file contains lower case 'a''s in the genome or dna sequences? I didn't think of this as the examples I've seen so far are all upper case. Would another version of this be to maybe convert the entire string to be uppercase and then just count the upper case bases?


r+


I think it would be good to have a policy file for this as well.


TODO:
 - Payment Details Table (I vote we use the [stripe](https://stripe.com/ca) api for payments)
 - Favourite Game(s)
 - Favourite Platforms(s)
 - Shipping / Billing Address
 - Profile Details
    - display name
    - actual name
    - gender
    - birth date


Thanks  for this!
I have made a general css rule that can be used by any video even in the future. So  this is also fixed.
I guess before it is merged to the master there is one small task which we discussed last time about making side bar drawable  especially in small screen. As you have noticed in small devices sidebar is hidden.
After adding that feature i think it will be good to go.
Thanks for reviewing this @amgowano .


The problem with this approach is that it relies on `CACHE_EXPIRE_TIME` agreeing with the underlying kerberos timeouts. A more robust way would be to rely on the "token expired" error we get and trigger a re-login then.


7f55731b2777f2a1f52dabfb03b7512cc0d365f8 - fixes.


> we should consider if there's a point compiling print-bt.c + print-nflog.c on targets where decoding those protocols doesn't make sense?

Targets where the native virus checker finds pcap and pcapng files with those link-layer header types and removes them immediately? :-)

We support `DLT_LINUX_SLL` on platforms other than Linux, so that if somebody sends you a Linux "any device" or "this device's driver doesn't supply usable link-layer headers or doesn't reliably supply them" capture, it can still be read.


I really don't get why custom should be removed. It still solves issues, and we're having non-power-of-2 multipliers anyway, why would we care if the multipliers end up non integer and different for x/y ?


BTW, why isn't that handled automatically by GSS? Maybe there are options to force it to re-login in the background?


It solve issue #6


> I'd like to collapse and move the minimum necessary contents of `$(WINPCAP_ROOT)/Win32/Include/<bittypes.h>` and `$(WINPCAP_ROOT)/Win32/Include/ip6_misc.h into $(LIBPCAP_ROOT)/pcap-stdinc.h`.

+1


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


That's even better. I think there are lots of little time thieves in dynamic. Some of them **are** fine considering "readable" and "bug free" are also important - but we should try to fix the easy ones.

Here's a thing I dislike but I have no idea how to avoid:

loader.c
```
  1650  #ifndef DYNAMIC_DISABLED
  1651          else
  1652          if (!strncmp(ciphertext, "$dynamic_", 9) && strstr(ciphertext, "$HEX$"))
  1653          {
  1654                  char Tmp[512], *cp=Tmp;
  1655                  int alloced=0;
  1656                  if (strlen(ciphertext)>sizeof(Tmp)) {
  1657                          cp = (char*)mem_alloc(strlen(ciphertext)+1);
  1658                          alloced = 1;
  1659                  }
  1660                  RemoveHEX(Tmp, ciphertext);
  1661                          // tmp will always be 'shorter' or equal length to ciphertext
  1662                          strcpy(ciphertext, Tmp);
  1663                  if (alloced)
  1664                          MEM_FREE(cp);
  1665          }
  1666  #endif
```
The problem is this is evaluated for every pot entry even if I use --format:NT and none of my input hashes look like a dynamic at all. It stood out in some profiling runs. Note though that this profiling was with a 1 GB pot file, filled with all sorts of formats (including LOTS of dynamic)... I'm not sure we need to change this but if you have some idea, it would be nice.

Simply using options.format is not viable since the `$dynamic_` tag is used for a bunch of non-dynamic formats too.


What do you mean?
I published this to the gallery a while ago.

Note that my extensions use VSIX v1, for compatibility with VS2010.


@magnumripper  there WAS a core change (config.c), but it should be very isolated.  I thought it better to expose that variable using a getter function, vs making it a global non-static var.  The getter function is NOT documented in a header anywhere.  I just declare and use it within the dyna_parser function. Normally code should use the interface into config. But in this case, a great amount of overhead is saved by simply walking the entire config db 1 time, capturing off the dyna formats on that one run.  Before, we would do 4000 partial/full walks of the config, with almost all of them being 'full' searches.  So we have reduced that overhead almost 4000x (not a bad savings).


動作イメージ：はてブvimperatorでタグのドリルダウンをできるようにしてみた - YouTube - https://www.youtube.com/watch?v=ifRIqnlkF4Q


This is still broken in the build.  Can it be fixed or should we revert?


This was added to support some work I was doing with Mapnik and can be removed 100%.


Lint постоянно ругается на много файлов, ему не нравится
\666\client\actions.js
Error:(1, 1) Export declarations are not supported by current JavaScript version
Error:(2, 1) Export declarations are not supported by current JavaScript version
Error:(3, 1) Export declarations are not supported by current JavaScript version
Error:(4, 1) Export declarations are not supported by current JavaScript version
Error:(6, 1) Export declarations are not supported by current JavaScript version
Error:(13, 1) Export declarations are not supported by current JavaScript version
Error:(20, 1) Export declarations are not supported by current JavaScript version
Error:(27, 1) Export declarations are not supported by current JavaScript version
\666\server\index.js
Error:(1, 1) Import declarations are not supported by current JavaScript version
Error:(2, 1) Import declarations are not supported by current JavaScript version
Error:(3, 1) Import declarations are not supported by current JavaScript version
Error:(4, 1) Import declarations are not supported by current JavaScript version
Error:(5, 1) Import declarations are not supported by current JavaScript version
\666\server\storage.js
Error:(1, 1) Import declarations are not supported by current JavaScript version
Error:(2, 1) Import declarations are not supported by current JavaScript version
Error:(3, 1) Import declarations are not supported by current JavaScript version
Error:(4, 1) Import declarations are not supported by current JavaScript version
Error:(11, 1) Export declarations are not supported by current JavaScript version
Warning:(15, 45) Unneeded comma
Error:(25, 1) Export declarations are not supported by current JavaScript version
Error:(33, 1) Export declarations are not supported by current JavaScript version
Error:(42, 1) Export declarations are not supported by current JavaScript version
Warning:(49, 21) Incorrect parameter name
Warning:(49, 21) Parameter идентификатор described in JSDoc does not appear in function signature
Error:(51, 1) Export declarations are not supported by current JavaScript version
Warning:(51, 29) Parameter mid is not described in JSDoc
Error:(63, 1) Export declarations are not supported by current JavaScript version
Error:(79, 1) Export declarations are not supported by current JavaScript version
Error:(87, 1) Export declarations are not supported by current JavaScript version
\666\server\db.js
Error:(1, 1) Import declarations are not supported by current JavaScript version
Error:(14, 1) Export declarations are not supported by current JavaScript version
Warning:(15, 9) Local variable 'promise' is redundant
Error:(35, 1) Export declarations are not supported by current JavaScript version
Warning:(36, 9) Local variable 'promise' is redundant
Error:(54, 1) Export declarations are not supported by current JavaScript version
Warning:(55, 9) Local variable 'promise' is redundant
Error:(69, 1) Export declarations are not supported by current JavaScript version
Warning:(70, 9) Local variable 'promise' is redundant
Error:(91, 1) Export declarations are not supported by current JavaScript version
Warning:(92, 9) Local variable 'promise' is redundant
Error:(115, 1) Export declarations are not supported by current JavaScript version
Warning:(116, 9) Local variable 'promise' is redundant
Error:(136, 1) Export declarations are not supported by current JavaScript version
Warning:(137, 9) Local variable 'promise' is redundant
Error:(155, 1) Export declarations are not supported by current JavaScript version
Warning:(156, 9) Local variable 'promise' is redundant
Error:(170, 1) Export declarations are not supported by current JavaScript version
Warning:(170, 17) Unused function removeRoom
Warning:(171, 9) Local variable 'promise' is redundant
Error:(191, 1) Export declarations are not supported by current JavaScript version
Warning:(191, 17) Unused function addIdMessage
Warning:(192, 9) Local variable 'promise' is redundant
Error:(211, 1) Export declarations are not supported by current JavaScript version
Warning:(211, 17) Unused function getIdMessages
Warning:(212, 9) Local variable 'promise' is redundant
Error:(226, 1) Export declarations are not supported by current JavaScript version
Warning:(226, 17) Unused function addIdUser
Warning:(227, 9) Local variable 'promise' is redundant
Error:(246, 1) Export declarations are not supported by current JavaScript version
Warning:(246, 17) Unused function getIdUsers
Warning:(247, 9) Local variable 'promise' is redundant
\666\server\Room.js
Warning:(17, 8) Unused definition listMessage
Warning:(18, 8) Unused definition listUsers


why the rename? i thought we agreed that naming errors not `err` was scary


Got it, thanks!


Awww yes! That's what I meant when I talked about not knowing where to list alt skins for trainers! Thanks a bunch.


this is unused in this commit?


how does using the feature.id impact the id sequence constraint on the table? The id field there should auto-increment which was a nice feature to ensure unique ids. I believe at one point i could not trust the feature.id to be truly unique in every case (but this may have changed.)


yeah, slight inconsistency (but only in this commit): the next commit actually returns a not-error from `ExecuteCmd` (and then you need the rename because it shadows something above), I just pulled a little bit too much into this commit. Going to move them to the next commit for eternal perfection.


why?


This seems like the wrong way to do this since GitRepositoryProvider already implements "walkUpwards" in "repositoryForDirectory".  But It's not clear to me what you're doing here (because I am new and naive), so I could be wrong about this.


fixes #232


What's the difference between `betterC` and `noRTTI`?


why this rename?


oh, because the type changed.


An events marker is not a BCyclePurgatoryException. Feel free to use InvalidStation exception. Also, please remove comment to only show `Special events marker has no availability information, discard it`.


This was supposed to go into a pull request.  Sorry!  No harm done, though.


LGTM. I appear to have messed up in merging my last PR. the shift-enter to submit doesn't work anymore either :/ I'll fix it.


Change TC_LOG_INFO("server.worldserver", "[Maloriak Debug] Abberations Left: %u", abberationsLeft)
No found... Try sLog->outInfo


oh, it doesn't implement Error, very well


Ok, I can accept that, but this gatefilter is not the one in Vgatefilter so one will need to use `change`, not `update`; it is inconvenient but not a problem. Current None is a valid value for Vgatefilter, but as in theory there exist a "empty" gatefilter we could remove that; this would solve the problem here, but would be inconvenient when changing files. What do you think?


Should this not be "The use of gender-specific, sexualized, or otherwise exclusionary communication to include imagery, text and audio."


it will be happe this time - testbuild running:

http://jenkins.kodi.tv/job/IOS-ATV2/5311/


This PR needs to be rebased to include (among other things) https://github.com/w3c/encrypted-media/commit/ed3000519186fd4a1af8a25c0d8024c0f586bc1f.


"state" doesn't seem accurate for "identifier". Perhaps it is the DI consent state.


This one will be hard to enforce, as at one point I know I was chastised for making a personal attack, that was not an attack, it was literally attributing a patch to it's author.


Uhm… splitting on strings is dangerous.  What about arguments with spaces in them?


can you help me understand why this commit was necessary?


I think the reasons were it was more glitchy and it was hard to maintain the custom res code, but I don't really know. Personally, it doesn't bother me whether it stays or goes.


This line doesn't do anything. You can remove it.


Might be worth pointing out in this comment that this script is for dev use only. You don't want to run this on a real slave (or you'll blow away the config done by CF).


s/tmpErr/pErr/ for consistency?


This util method does not look side-effect free since it sets the button font. Shouldn't the button font be set elsewhere?


sure. It's really the next commit that makes it necessary: You can't return a `SendError` via `*proto.Error` unless it's in the `ErrorUnion`.


Line 55-56 does the same thing!


This is the correct version. The other instance of updateKey should be removed


This should have gone into my own branch, from which I was going to issue a pull request.
I fat-fingered it.  If desired, I will undo and do it again via a PR.


I can follow your reasoning... no problem! :-)

ciao

Luca

On 21 September 2015 at 21:48, Kai Hofstetter <notifications@github.com>
wrote:

> Hi Luca,
>
> thank you for the proposal!
> I use the version number in order to have reproducible builds. I try to
> keep the WordPress version updated and I tag my code with every new
> WordPress version.
> That way you know which WordPress version you actually use and you can
> even pick a certain WordPress version, e.g. for testing.
>
> Thanks,
> Kai
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itibook/docker-wordpress-cli/commit/6ce3183799af27178fcb1749390adb3e41cd14f3#commitcomment-13351380>
> .
>


"for" could be ambiguous. Although the consent could be based on the configuration (i.e. what exactly was requested), the user doesn't need to be informed about this configuration and consent is not necessarily specific to this configuration.

Same comment in the next step.


If the user types something that isn't a number (for example, 'foobar') then the program will crash.

You can use the string method [isdigit()](https://docs.python.org/2/library/stdtypes.html#str.isdigit) to check before converting it to number. If it isn't a number you can re-prompt saying something like "Not a number. Try again."


Looking good. I had a couple of notes I added above.


this can be removed


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (808 of 808 tests passed)

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


It seems we should explicitly handle all possibilities (and mentally assert in the default case) since a "Deny" response would just continue.


YES! I've not seen that! This is the wrong use of the `new` method. We must really prevent this (recurrent) error, do you understand my suggestion? it is not a real solution but could help preventing miss use.


I think it would be safer to use a try/catch here, in case returnvalue gets corrupted or something.


At line 115, add:

```lua
if Item:IsA('PartOperation') then
	self.State.HistoryRecord.terminal_color_states[Item] = Item.UsePartColor;
end;
```


nit: s/and/in the/


or attr


asio/detail/memory.hpp doesn't exist in this branch, and it's not used anyway, is it?


I Just find this not too helpful, to potential offenders nor the project designated "mods"; it needs to be less vague to be effective.


nit: Should probably say "then continue" to be consistent with the next block.


I am trying to follow it for VS2015 using Visual Basic and can't map the instructions but I don't think it is because I am using VB.. The vsixmanifest file does not have entries (Identifier, Content, MefComponent). from your example.


nit: In other locations, we just say "Continue."


How can you make sure that a Chicken.type "chicks" born Male will grow into a Rooster and not into a Hen?


Committed change so that `cache()` returns self.


@Nadahar I would never use MediaInfo=false, in fact I have a PR that converts all renderers to use MediaInfo=true

@RandQalan All of those values are in the file except for KeepAspectRatio and RescaleByRenderer, or they are program defaults so don't need to be set. KeepAspectRatio and RescaleByRenderer make things run slower, especially RescaleByRenderer=false since that will require much more processing power and bandwidth than if it is true.


x+y


*wich


At line 78, add

```lua
if Item:IsA('PartOperation') then
    Target.UsePartColor = self.initial_color_states[Target];
end;
```


You mean you're trying to add a menu command to a MEF-only VSIX?
You should be able to add the `VsPackage` / `Pkgdef` content from the VSIX v2 designer; IIRC, it's one of the entries in the dropdown.


Added these in the places where they should belong. The first two are related to pylint not being smart enough to understand different imports for different versions of python so will leave


we might not need this extra line.


remove trailing whitespace


Мы реально это будем пользователю выводить? O_O


All systems have been migrated awhile ago :)
I forgot to update Wappalyzer afterwards.


done.


also moved bounding box check and scale field creation outside of raster only section. all datasets will prompt user with bounding box in interface mode and add a scale field to datapackage


My vote is to remove the last remaining references:

```
[ehebert@scattershot zipline_repo (lazy-mainline)]
$ ack _position_last_sale
tests/test_perf_tracking.py
2267:        nt.assert_dict_equal(test._position_last_sale_prices,
2268:                             pt._position_last_sale_prices)

zipline/finance/performance/position_tracker.py
289:        self._position_last_sale_prices[sid] = price
299:            self._position_last_sale_prices[sid] = pos.last_sale_price
312:            self._position_last_sale_prices[sid] = last_sale_price
352:                itervalues(self._position_last_sale_prices),
367:            self._position_last_sale_prices[split.sid] = \
435:            self._position_last_sale_prices[stock] = position.last_sale_price
451:            price = self._position_last_sale_prices[event.sid]
525:        self._position_last_sale_prices = OrderedDict()
```


@ximex in some cases you edit javascript. Does js also have === or !== ?


This timing have been validated by building the firmware with GCC 4.9.3 20150529 (release) [ARM/embedded-4_9-branch revision 224288] it could differ a bit with older version even if the timing is based on number of MCU cycles.


test


TeamCity pull requests :: netty [Build 230](https://garage.netty.io/teamcity/viewLog.html?buildId=1525&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4781, ignored: 16; resource leak detected (new) Build time: 00:22:22
### Failed tests
```
io.netty.resolver.dns.DnsNameResolverTest.testQueryMx: <no details avaliable>

```


next commit cleans this up, so going to leave it here.


Haha it might be a good idea...poor little boys.


re: "status" - same comment as above about what this is the state/status of.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 669](https://teamcity.labkey.org/viewLog.html?buildId=273283&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:17
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Возможно, это временная заглушка. Но я могу ошибаться.


The spell checker complains about "Readded". Make that "Re-added"?


NO! BACK TO ORIGINAL!


s/state/consent/?


this always do a copy.


Similar comment about "for". It's really about evaluating the configuration against persisted consent.


I Can see the intention, and support it, but I think it needs work to be an effective tool in stamping out problems that can be extremely complex.


Mer hend en istellig für das wo ide db persistiert wird... isch also ned notwendig ide navbar..


What is the purpose of --genHtml ?

It doesnt seem to do much of use.


@timgraham I'm calling to the language expert...


Someone is mad. :)


Why do key errors happen? What does it mean when something is missing its `incoming_id` or `_source`?


I can't tell if you cover it elsewhere, but you might want to tweak the regex a bit so that leading caps get a dash before them as well:

```js
const CAMEL_CASE_REGEXP = /([a-z]|^)([A-Z])/g
```

Example: `WebkitFlex` becomes `-webkit-flex`


This isn't the comment location you're looking for...


awesome readme!


min for fast
MediaInfo = true
SeekByTime = true
DLNALocalizationRequired = true
DefaultVBVBufSize = false (Not sure if this off by default or not if so then do not need this)
WrapDTSIntoPCM = true
H264Level41Limited = true
KeepAspectRatio = true (If you do not set with with TS mode and TS causes slow do not know why)
TranscodeFastStart = true


I am trying to add a menu to an Analyzer that is delivered through a VSIX. The VSIX file references a VB project in the some Solution. Don't know what "VSIX v2 designer; IIRC" is but I think you are saying that I can add a "menu item" to Visual Studio 2015 directly within Visual Studio without manually editing. I did try to add a "Command Item" directly to the VXIS project, but got over 300 errors just from adding the item which lead me to your post looking for an example..


re: "covering an" - "this" was changed to "an". I think "this" is appropriate since this algorithm only runs when a DI is requested. Also, ", which" does not work for "an".


should we read the classpath entry in /etc/HPCCSystems/environment.conf?


Is there a reason this needs to be a list?


`self.validCredentials` and `self.loading` are already in the `FolderPickerViewModel` constructor so I think those can be removed.


You can check non-emptiness of a list (or set) with `if newIds:`


I forget what happens with `pymysql`, but you might have to explicitly close the connection somehwere later.


Hey @level20peon, did you control-c and restart your foreman process after installing this?


testing the bias at always zeros isn't good.


I have to rebase, upstream is changing.

On Mon, Sep 21, 2015 at 10:36 AM, Sam Padgett <notifications@github.com>
wrote:

> Godeps/_workspace/src/k8s.io/kubernetes/pkg/kubelet/server.go:494:
> req.URL.Query.Get undefined (type func() url.Values has no field or method
> Get)
>
> Looks like it should be URL.Query().Get()
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/origin/commit/dfa684de5a680fcf268ed34726ce3579c9525856#commitcomment-13344095>
> .
>


juuust a little... :)


wicked


You could also make a [release](https://github.com/partouf/ReplaysOfTheVoid/releases) on GitHub and add binaries there.


Don't use string formatting for this; it'll leave us vulnerable to SQL injection attacks. Instead, we should use something more like:

http://legacy.python.org/dev/peps/pep-0249/#id15


Please, better express your issues with R# code completion in ReSharper's official [issue tracker](https://youtrack.jetbrains.com/issues/RSRP). I'm not the person responsible for R# code completion experience (unhappy with it too, if you ask me).


> If you get the time, review some of the notes i left by the FIXME's and shed some light on what kind of exceptions are there, so we can be more explicit.

Will do as I said ASAP

Re: mess - checkout v305 and have a look ;)

Re: relative imports: refactoring is at 70% and they are a byproduct - trust me splitting 30k lines files help.Package layout is still fluid have a look at 163 and co. Once and only once layout is frozen the import style will be revisited

Re: refactoring - uyou can do that one easily - check th link in 200

Congats on Pycharm - read wiki post, better get my settings

Thanks for follow up


`pymysql` provides a `dictCursor`; use that instead, so these are named.


fair enough...


What probple you have? Did you debug test?


it's fine

Herb Jellinek <notifications@github.com> writes:
> This should have gone into my own branch, from which I was going to issue a
> pull request.
> I fat-fingered it. If desired, I will undo and do it again via a PR.
>
> —
> Reply to this email directly or view it on GitHub.*
>


Hi @gioman
- it's an OGR datasource, I'm quite sure you can just create it from `New vector layer` (you can save any loaded layer to GPKG from `Save as` entry in context menu),
- I see the right geom type here (e.g. linestring), but I've not tried all the geometry types, maybe any of them are not correctly handled,
- as far as I've written, I just disabled editing on GPKG files to avoid issues, but it requires few other adjustaments. Please file a ticket for those, thanks.


Could we not have this in split() on the condition `ldr_in_pot` is true? Same goes for my ugly linkedin hack above it.


@mattpap, I had to change this lines to get the correct url...
I am missing something something obvious about why you put that code in place?


Looks familiar :smile:


what are "grade courses"?


TeamCity pull requests :: netty [Build 229](https://garage.netty.io/teamcity/viewLog.html?buildId=1524&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5606, ignored: 21; resource leak detected (new) Build time: 00:38:19


I think readded is fine. From what I read, it seems Europeans like to use the dashed version, but Americans omit it.
Reference:  http://english.stackexchange.com/questions/7731/adding-a-prefix-re-to-a-word-with-or-without-a-hyphen


Tuning is not an easy game. I had no success so far. So I can't speak too much about this.

But for sure, I don't trust this default parameter in column # 5.
For evaluation terms, and for months, everybody was using 10,
and sometimes less, once the thing had been tuned to death.

And when we divide by some number to get a non-integer, I think it is even trickier to get it right
It completely changes the meaning of the TUNE(SetRange( , ), ...)

Good luck with this, it's an important test !


"courses_which_should_be_shown_to_graders"?
"all_but_new_courses"?


Go for it! It works like a charm


ok,get it


:stuck_out_tongue_winking_eye:

Except this one constructs itself! No documentation string to get out of sync.


use `and` for those two? also, `==`?


a*n* ordering


@ianobermiller Well, I am hoping for people not to use already-prefixed props at all :sweat_smile: , but lets keep this safe by enhancing the code again :P


I'm not sure why we would want `rejectStage` to return the weights just for the given stage. Generally, I would assume that we want to instead have the weights for the stages for anything that passed up until `rejectStage`. So, if there are 5 stages, then we can have a `rejectStage` (or similar), so that if `rejectStage == 3`, then we can return the weights and information about any proposals that passed `rejectStage` (that is, any proposal that was rejected after `rejectStage`). In that case, this line should be changed to
```cpp
if (classifier->data.stages.size() + result <= (size_t)rejectStage)
```

As it is currently, if a window passes `rejectStage`, but is rejected by the next stage, we will receive no information about it. I don't believe this has a common use case, but I may be wrong. In my understanding, the use for something like this is to manually filter proposals that passed at least the first `k` stages of the cascade. This is what the `< 4` check from [earlier](https://github.com/Itseez/opencv/commit/0bc609257d1b3e47162761e3e6034cd031358e79#diff-7d39e2aff7c4b7576cf314a4741b347eL991) attempted to do, I believe.

If we go with that interpretation, we should also change `rejectStage` to a better name; perhaps `firstRejectStage`, `filterProposalsRejectedBeforeStage`, or `lastAcceptedStage`. I personally don't like the naming of `rejectLevels` and `levelWeights`, but if we assume those names are, I can't come up with a better name for `rejectStage`.


The Debug build is failing on CI because of a Debug error:

"Run-Time Check Failure #2 - S" ;  which I think (after some searching.. going by [this](http://stackoverflow.com/a/8543914/863980)) is a snipped version of "Run-Time Check Failure #2 - Stack around the variable ?? was corrupted" for https://github.com/Snaipe/Criterion/blob/7efbac94/src/core/theories.c#L196. I suppose is the issue in the following `for` loop (nested in `while` loop).

It could be the [un]fortunate optimization which hides the error with Release configuration but not the Debug or vice versa, :)

![image](https://cloud.githubusercontent.com/assets/3840695/10003067/52066fa6-60b3-11e5-9def-8f3913d95941.png)

---

On a slightly related note, when I execute `samples\tests\Debug\*c.bin.exe`s, most of them throw assertion failure, for instance, running:

```
C:\temp\Criterion>samples\tests\Debug\failmessages.c.bin.exe
```

produces https://gist.github.com/am11/53da623ddfbed0d6af71.


might i get some newlines please


No, I killed all processes regarding huginn before pulling this commit.


How would you set the required href when using a dynamic route via the route-href attribute, when you want this to be part of the router nav, so the .isActive can be utilized with it?

threads.js
``{ route: '',        name: 'thread',       moduleId: 'thread',     nav: true,    title:'Single Thread Viewer' },``
``{ route: '/:id',    name: 'thread',       moduleId: 'thread' }``

threads.html
``<a route-href="route: thread; href: t.id; nav: true; params.bind: { id: t.id }" >``


I'm trying to edit here... but it won't let me because I'm not "branched"?


Forget it (or test it ;))

I never ever use any other git suite but msysgit - I advise you to do the same, frankly

git rebase is very important to master - as is `gitk --all &` - right click and experiment

You will find `git rebase --interactive --onto dev dev crazykilla15-*` is indispensable - I do one commit and 10 rebases

Do change your editor for msysgit as detailed in our wiki if you are not comfortable with vi - makes a huge difference


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.973.17677c3f31817be2be1f2eebdc44aa01537213ba](http://build.palaso.org/viewLog.html?buildId=63556&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 516, ignored: 6 Build time: 00:03:40


Lol, this is backwards naming: It should have been `cfg_get_db()`


ich lass, als vorläufigen fix, für ConfigText kein createSetup() aufrufen. Damit ist erst eimal die Texteingabe wieder möglich. Allerding wird für ConfigText das LCD nicht aktuallisiert, da das zuständige createSetup() nicht aufgerufen wird.


maybe output a message that the publishing happened? otherwise that's a bit silent... otoh, the same is true when uploading final grades, right?


are the commas correct here?


there is an else for the second if


Yeah, good stuff. I'll be doing more profiling later.


The 'object' was named cfg_db, so that was what I was 'getting'


please add tests testing this view.


@skeptical cool, I'll fix that NPE. I've been doing those a lot lately, I'll try to get better at that.
The bug was like:

```0:12 / 59```

when the video is under 1 minute duration, so it's not very clear if it's saying 59 minutes or 59 seconds, and the two strings are in different formats.

It seems to me it would be better to use the 00:00:00 format but if it's important to save space we should make it consistent at least, so the above example would be:

```0:12 / 0:59```


The convention is to have the repo-level named README.md, while module-level readme's are README.rst


Congrats!


gcc need to have "-l" flags at the end of the compilation line. Otherwise, it just ignore these flags...


Not all rook checks are constructive.
1. If the side getting checked interposes with a rook that can't be traded, it wastes time.  "Can't be traded" is not so easy...I don't know that much about the code but maybe killer moves would reveal that?  Or maybe just having a passed pawn 2 squares closer to queening and no opponent king "within the square of the pawn".  Obviously a calculated can't trade would be much better.  Maybe just the line considered moves away from the potential trade?  I know that stuff is hard with a later calculation informing a former one.
2. If a pawn can be pushed and interposed without making it vulnerable it is probably wasted time.
3. If the King moves from the check but the rook cannot check without danger the next move, then it probably does not achieve much.

4. If the null move is the same move that would get the king out of check now and is still the best choice, the check is probably a waste of a move (regardless of what piece was checking), and possibly even get a penalty rather than a bonus.  Any time you force someone to go where they wanted to, it is probably at best a waste of time.


Yeah I guessed... and `cfg_get_cfg_db()` would be weird :laughing: so let's just leave it like this.


This should make a huge difference here, when no -format is used.   Now, huge is a relative term.  Huge might be going from 200ms to >1ms.  So the relative time is great, but overall wall clock was not 'super' huge to start with.


@javisantana we block this for a reason?


The coding style says all functions here should be `cfg_*()` but we're not fully compliant with that anyway I think.


Is it possible to just add a comment here explaining why you do that? It's almost obvious, but might help people who read the code in the future.


I will do some testing around the loader loop in VC (using the timer.c class), and see how big a change there was.


@RandQalan this version is exactly what you described, please confirm :) https://raw.githubusercontent.com/UniversalMediaServer/UniversalMediaServer/799f7d4d76d1a10a904eedfb4859c7547674ce1b/src/main/external-resources/renderers/Vizio-SmartTV.conf

(the values that aren't in there are defaults)


I can live with the way it is now.


what about /usr/lib/presto and /etc/presto?


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


See this guy's PR: https://github.com/compose/transporter/pull/134

Tests aren't passing, though I think the tests are outdated and should expect 1 less argument each.


This is the initial README.md commit for this repository


yes, we don't show these messages to graders at all. they don't really have anything to do with that part - do they?


Show me your VSIX manifest.

The non-code changes in this commit (VSIX manifest and `GeneratePkgDefFile`) do not affect compilation; they're just necessary to get VS to actually load the menu item.

If you 're getting compilation errors, you're probably missing some assembly references.


This is fixed by 7efbac94d10e4611e5864538a432acc93d48433c

A lot of the samples are actually meant to produce these assertion failures, as the outputs are used for regression tests. It would be an error if those samples did not throw the assertion failures.


Neato, looks like the API resembles iOS 9's `-[UIKeyCommand keyCommandWithInput:modifierFlags:action:discoverabilityTitle:]` a bit.


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - i386-Windows-7-VisualStudio-12: OK (1882 of 1882 tests passed)

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


check


Why wait to change relative?

When/If stuff is moved or changed around, absolute would be much easier to change than figuring out how many spaces you have to go back or not, not to mention much easier for pycharm to change automatically on a refactor


removed


Good point. Frankly, it never occurred to me to use a directive here. I will change that.


@Fasse yes.
http://www.w3schools.com/js/js_comparisons.asp


I did, and it generates html in the style of the help document, completely ignoring indentation though(at least it doesn't appear in the browser the same way it does in the source)

I do infact use msysgit

Oh thank god, you can change that? I'm fond of nano, i hope i can use that.

Reading a ton of wiki stuff now. ( i would swear some of those werent there before...)


Yes makes sense - so let's use this container as the build environment and pass in the command to be run. Let's target only 'make' and 'make test' for now. Regarding,  #2 we don't want to build the container every time, which can be alleviated by mounting the src in the container.

Thanks!


/usr/lib/presto and /etc/presto should be root (that's how it is in hive, at least, and since the files in them are 644 they're readable)


CreateDLNATreeFaster = true
MediaInfo = true
can not be on at same time will not show tree at all for folders one or the other have to be off
since you all like mediainfo because of checks done then Tree has to be off or deleted


Thanks!


I dont feel comfortable just dropping the .idea in like that, i have it set up exactly how i like it

Any major changes in it that would make a difference?


Yep. Just blocks instead of responder chain actions.


The tests were setup with the assumption that you would provide the necessary info for testing on your own end, storing your info using the dbi-dbrc gem. This is perhaps not ideal, but it works.


file.dmg / 26'288'679 bytes :
```rebol
  read %file.dmg

*** Runtime Error 98: assertion failed
*** in file: /red/allocator.reds
*** at line: 551

*** Runtime Error 1: access violation
*** in file: /red/debug.reds
*** at line: 70
***
***   stack: __print-debug-stack 00013424h
***   stack: ***-on-quit 98 78884
***   stack: red/alloc-series-buffer 26288679 1 0
***   stack: red/alloc-series 26288679 1 0
bb``RzÀ0>" 26288679 00000000h 00000000h trues
***   stack: red/simple-io/read-file "file.dmg" false false true
***   stack: red/simple-io/read 0201D144h false false
***   stack: red/file/read 0201D144h 0201D134h 0201D134h false false 0201D134h
***   stack: red/actions/read 0201D144h 0201D134h 0201D134h false false 0201D134h
***   stack: red/actions/read* -1 -1 -1 -1 -1
***   stack: red/interpreter/eval-arguments 020A75BCh 02111DB8h 02111DB8h 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A75BCh 02111DA8h 02111DB8h false 00000000h 00000000h 020A75BCh
***   stack: red/interpreter/eval-expression 02111DA8h 02111DB8h false false
***   stack: red/interpreter/eval 0201D134h true
***   stack: red/natives/try* 1
***   stack: red/interpreter/eval-arguments 020A6EDCh 0208EED0h 0208EF20h 0208EEB0h 0208EF68h
***   stack: red/interpreter/eval-code 020A6EDCh 0208EEC0h 0208EF20h true 0208EEB0h 0208EF68h 020A6EDCh
***   stack: red/interpreter/eval-path 0208EEB0h 0208EEC0h 0208EF20h false false true false
***   stack: red/interpreter/eval-expression 0208EEC0h 0208EF20h false true
***   stack: red/interpreter/eval-arguments 020A6DFCh 0208EEB0h 0208EF20h 0208EE90h 0208EF34h
***   stack: red/interpreter/eval-code 020A6DFCh 0208EEA0h 0208EF20h true 0208EE90h 0208EF34h 020A6DFCh
***   stack: red/interpreter/eval-path 0208EE90h 0208EEA0h 0208EF20h false false false false
***   stack: red/interpreter/eval-expression 0208EEA0h 0208EF20h false false
***   stack: red/interpreter/eval 0201D114h true
***   stack: red/natives/unless*
***   stack: red/interpreter/eval-arguments 020A767Ch 0208ED6Ch 0208ED8Ch 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A767Ch 0208ED3Ch 0208ED8Ch false 00000000h 00000000h 020A767Ch
***   stack: red/interpreter/eval-expression 0208ED3Ch 0208ED8Ch false false
***   stack: red/interpreter/eval 0201D0F4h true
***   stack: red/natives/do*
***   stack: ctx181~run 001CC160h
***   stack: ctx181~launch 001CC160h
```

file.mp4 / 2'347'182 bytes :
```rebol
  read %file.mp4
;   "^@^@^@ ftypisom^@^@^B^@isomiso2avc1mp41^@^@�"
  read %file.mp4

*** Runtime Error 1: access violation
*** in file: /red/allocator.reds
*** at line: 818
***
***   stack: red/alloc-big 2347204
***   stack: red/alloc-series-buffer 2347182 1 0
***   stack: red/alloc-series 2347182 1 0
***   stack: red/unicode/load-utf8-buffer "" 2347182 00000000h 00000000h true
***   stack: red/simple-io/read-file "file.mp4" false false true
***   stack: red/simple-io/read 0201D144h false false
***   stack: red/file/read 0201D144h 0201D134h 0201D134h false false 0201D134h
***   stack: red/actions/read 0201D144h 0201D134h 0201D134h false false 0201D134h
***   stack: red/actions/read* -1 -1 -1 -1 -1
***   stack: red/interpreter/eval-arguments 020A75BCh 03000B28h 03000B28h 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A75BCh 03000B18h 03000B28h false 00000000h 00000000h 020A75BCh
***   stack: red/interpreter/eval-expression 03000B18h 03000B28h false false
***   stack: red/interpreter/eval 0201D134h true
***   stack: red/natives/try* 1
***   stack: red/interpreter/eval-arguments 020A6EDCh 0208EED0h 0208EF20h 0208EEB0h 0208EF68h
***   stack: red/interpreter/eval-code 020A6EDCh 0208EEC0h 0208EF20h true 0208EEB0h 0208EF68h 020A6EDCh
***   stack: red/interpreter/eval-path 0208EEB0h 0208EEC0h 0208EF20h false false true false
***   stack: red/interpreter/eval-expression 0208EEC0h 0208EF20h false true
***   stack: red/interpreter/eval-arguments 020A6DFCh 0208EEB0h 0208EF20h 0208EE90h 0208EF34h
***   stack: red/interpreter/eval-code 020A6DFCh 0208EEA0h 0208EF20h true 0208EE90h 0208EF34h 020A6DFCh
***   stack: red/interpreter/eval-path 0208EE90h 0208EEA0h 0208EF20h false false false false
***   stack: red/interpreter/eval-expression 0208EEA0h 0208EF20h false false
***   stack: red/interpreter/eval 0201D114h true
***   stack: red/natives/unless*
***   stack: red/interpreter/eval-arguments 020A767Ch 0208ED6Ch 0208ED8Ch 00000000h 00000000h
***   stack: red/interpreter/eval-code 020A767Ch 0208ED3Ch 0208ED8Ch false 00000000h 00000000h 020A767Ch
***   stack: red/interpreter/eval-expression 0208ED3Ch 0208ED8Ch false false
***   stack: red/interpreter/eval 0201D0F4h true
***   stack: red/natives/do*
***   stack: ctx181~run 001CC160h
***   stack: ctx181~launch 001CC160h
```

file.exe / 831'488 bytes :
```rebol
  read %file.exe
;   "MZ�"
  length? read %file.exe
;   3
```

```rebol
  system/platform
;   MacOSX
```


good one. will do


you dealt with `=` vs `==`?


yep


@michaelcahill, is this change correct, I thought only logical operators could be used for bool return types?


You have to unmap the file, too.


I think I asked the question wrong, How do you get the task_01/__init__.py to appear on your file.


@christianv It wanted a blank line before the table .ps_box-button block.  Yikes!


Could someone fix this? My day is now off and I'll not be available tomorrow morning. Thanks.


This is still going to pull the entire file back into memory at `f.read()`, so I don't see how writing to a file is beneficial. In fact it should be slower than using a BytesIO since the file is getting round-tripped to the disk and back. Now if the goal is to avoid needlessly using memory in Python, we also need to get Django to stream that temp file back out into the HttpResponse. Here is some info on that:

http://stackoverflow.com/a/13429719/2674930

(Looks like we'd need to upgrade to at least Django 1.5 for that, might as well go to 1.8 at that point.)

This solution is still not great because a python process (albeit with minimal memory usage) and a postgres connection would be held for the duration of the download, which could be a long time for these big files. I think it is possible to write a temp file to disk and then set the "virtual" X-SendFile HTTP header in the response to tell apache to read the response body from that temp file. But that would need some infrastructure for managing those temp files, and I'm not even sure how to determine when they'd be safe to delete.


Si verifican bien, en la clase conexion del modelo, el primer método se llama conexion el cual retorna un objeto que es la conexion a la db, por tanto no es necesario crear en esta clase un objeto de conexion sino solo llamar al metodo para que se las retorne...


Let me try it out and see how this works for me.


This is the url of the OWL download link at the orphanet website: http://bioportal.bioontology.org/ontologies/ORDO


If you want to backup if either a mod update or a server update is required, then this could be moved out of the `if [ -n "$appupdate" ]` conditional block, as the surrounding block will only be run if either a mod update or a server update is required.

Perhaps, we could also add e.g. a `--backup` switch that enables this from the command-line - in the option processing, an extra option would be added:
```
--- arkmanager
+++ arkmanager
@@ -692,6 +692,8 @@
       saveworld=1
     elif [ "$arg" == "--update-mods" ]; then
       modupdate=1
+    elif [ "$arg" == "--backup" ]; then
+      arkBackupPreUpdate=true
     fi
   done

```


actually removing the apikey part works too. I'll just do that.


Fixing spelling


This is great. Thanks for fixing this! I'll merge this patch.


Fixing for Python 2


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


Looks good!


Hi there. I'm encountering this same issue. I can't change my user name since I am on a company computer. Is there anyway to implement this change myself in v1.3.4 that I've downloaded. I don't see the place where I can edit the source code for this? Appreciate your help!


You know, the commit message really shocked me. I thought I had fixed the issue already!

Now, reading the patch, it appears that the commit message was quite a bit misleading. It does *not* fix the mtab symlink as I had thought initially.

Could you please amend the commit and change the commit message to clarify that you are fixing the *release notes*, then force-push (and leave a note so I get notified by GitHub)? Thanks!


:smile:


LGTM :+1:


So I *think* that for now we'll have to do it this way and when we make the commits for updated PyART gatefilter, then we can modify this code to remove the empty gatefilter instance.


Why don't you come out of hiding Sai?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 672](https://teamcity.labkey.org/viewLog.html?buildId=273300&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:25:29
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Man, I've been quite the slacker lately!  Will merge, thank you.

On Mon, Sep 21, 2015 at 2:21 PM, Stefano Raggi <notifications@github.com>
wrote:

> There are a couple of tests not picked up by refactor: [image: :smile:]
> StefanoRaggi@c9813e9
> <https://github.com/StefanoRaggi/Lean/commit/c9813e959ae983ec1e3dd23a90ad90529e8350d5>
> and StefanoRaggi@af813bb
> <https://github.com/StefanoRaggi/Lean/commit/af813bb6c2df2ef895d298a0fba71735fa331c3f>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/QuantConnect/Lean/commit/c67ff638f4d33a8a50a401476035fc795787211f#commitcomment-13349499>
> .
>


Why ?


@TristanWright This fixes #11


I see the dangers you point out and we'll have to make sure this isn't done elsewhere. I'll change the gatefilter plugin also in this branch and then with PR when ready.


> Why wait to change relative?

-_-

> Any major changes in it that would make a difference?

The code style settings and warning settings from the top of my head


Ok, I put a timer check around the 'loader block' in the dynamic_Register_formats call in dynamic.c.  Here is the results

$ ../run/john in
Time used was 0.000084s

I think that is quick enough, lol.  That is checking valid for all items from 0 to 5000  (not doing a format load, just the check).   This performs the timing test  (debug only stuff).

```diff
diff --git a/src/dynamic_fmt.c b/src/dynamic_fmt.c
index 8ce2b2c..0b66e0d 100644
--- a/src/dynamic_fmt.c
+++ b/src/dynamic_fmt.c
@@ -53,6 +53,7 @@ static DYNAMIC_primitive_funcp _Funcs_1[] =

 #include <string.h>
 #include <time.h>
+#include "timer.h"

 #if AC_BUILT
 #include "autoconfig.h"
@@ -7819,12 +7820,18 @@ int dynamic_Register_formats(struct fmt_main **ptr)
                *ptr = pFmts;
                return (nFmts = 1);
        }
-
+       {
+               sTimer t;
+               sTimer_sTimer(&t);
+               sTimer_Start (&t, 1);
        for (count = i = 0; i < 5000; ++i) {
                if ((pop[i] = (dynamic_IS_VALID(i, 0) == 1)))
                        ++count;
        }
-
+               sTimer_Stop (&t);
+               printf ("Time used was %0.6f\n", sTimer_GetSecs(&t));
+               exit(0);
+       }
        // Ok, now we know how many formats we have.  Load them
        pFmts = mem_alloc_tiny(sizeof(pFmts[0])*count, MEM_ALIGN_WORD);
        for (idx = i = 0; i < 5000; ++i) {
```


These should now be corrected.


**Mouse operation's parameter is under review...**


This the best so far with media info

#----------------------------------------------------------------------------
# Profile for Vizio Smart TV.
# See DefaultRenderer.conf for descriptions of all the available options.
#

RendererName = Vizio Smart TV
RendererIcon = viziosmarttv.png
UserAgentSearch = IPI/1.0 UPnP/1.0 DLNADOC/1.50
UpnpDetailsSearch = Vizio

SeekByTime = true
DLNALocalizationRequired = true
TranscodeVideo = MPEGTS-H264-AC3
DLNALocalizationRequired = true
ForceJPGThumbnails = true
WrapDTSIntoPCM = true
TranscodeFastStart = true
KeepAspectRatio = true
MediaInfo = true

# Supported video formats:
Supported = f:mpegps|mpegts   v:mpeg1|mpeg2|mp4|h264   a:ac3|lpcm|aac|mpa   m:video/mpeg

# Supported audio formats:
Supported = f:atrac   n:2                            m:audio/x-oma
Supported = f:mp3     n:2                            m:audio/mpeg
Supported = f:wav     n:2                  s:48000   m:audio/L16
Supported = f:wav     n:6   a:dts|lpcm     s:48000   m:audio/L16

# Supported image formats:
Supported = f:jpg    m:image/jpeg

Yes need DLNALocalizationRequired = true or will lockup on start of movie
and yes H264 works the same speed and other so why not :)


Camilo me avisa cuando lo suba


Revind here


Also closes #14720 (the PR wasn't merged, hence no auto close).


I just realized by editing my post several times it probably spammed some email...sorry about that.


might be nice to make this configurable via system property? (thread pool size).


NewLogUnitService.AsyncInterface: I do not understand what this is as I could not find any code. Is this somehow connected to an idl that I am unaware of ?


@level20peon Just to clarify, the rest of huginns components are working and you are just getting the log message every minute?

I will see how we can make the dependency check earlier so the message is just logged once.


when would a dict key ever be a mutable.. hint never.


Renaming the variable caused damage for no good reason.


Not sure the comment is necessary, but I don't have a strong opinion either way.


`*(p+1)` should be rewritten as p[1] and `*p` turned into p[0] for readability sake


added test


@danmar
I added the configuration of std::string::find to std.cfg with https://github.com/danmar/cppcheck/commit/10da5d6d2f45a9ce7bce658b77ffc5ab53eee4e1. By adding this function, a could not find a way to add proper configuration for overloaded functions, as shown in the documentation of std::string::find(), here http://www.cplusplus.com/reference/string/string/find/. Currently, the function ```size_t find (const char* s, size_t pos, size_t n) const;``` cannot be configured, because it has three parameters and all others have only two.


API key is contained in Enum Class ,it’s illegal to to share the personal themoviedb.org API key, you can easily get your own by registering email and require you OWN API KEY


I assume "loads defines" is a typo.


It may be good to NOT have this one compliant. That keeps other code from knowing about it.  That object 'normally' would not be accessible, nor should it. Simply put, in C++ terms, we have made the dyna_parser a friend of config, exposing some of the guts, simply because we know the overhead we can save.  We 'could' have passed in an array for some function inside of config.c to fill in, but since this was SO specialized, that did not make sense either. So I simply made exposure to the object.

The real 'gotcha', is that if/when we change the underlying layout of the config database, we would (might) also have to update this part of dynamic.  Now, the usage is so trivial (just walk a linked list), and all we access is 1 string var. So as long as the names and logic of those 2 fields do not change, the code should work fine in dyna, even if config.h had 10 additional things added to the structure, or had the order of fields modified etc. But the name and function of those 2 fields is preserved, then dyna should be able to function properly.


Commented code.


Is this now working on the Mac?


This fixes #169.


Suggest opening a PR and referencing that PR (rather than issue #40) from reaction commits and PRs.


This should be removed.


1) It needs to be noted that if any threads are spawned before the call to signal_manager::init() they may be interrupted. I generally do any command line parsing and whatnot before I block signals or spawn threads. They would have to go before init(), and thread spawning after... I wonder if it's just naming convention of init() that makes that less than obvious?

2) I'm a firm believer that the timeout should be a std::chrono::nanoseconds object, which is just an int64_t anyway but is more strongly typed.

I think:
mgr.init({signals}, std::chrono::nanoseconds(100000))
is clearer than
mgr.init({signals}, 100000)
from the caller's perspective if I were to be reading someone else's code and wasn't already familiar with the psig library.

3) The call to clear_mask() at line 280 is not needed as sigwaitinfo() does this itself atomically within the system call. In fact, by clearing the mask ourselves, we open ourselves up for the possibility that a signal is delivered to our thread context between the call to clear_mask() and wait() which would result in the default signal handler (not the std::function callbacks, but the actual handler set in sigaction()) being called which would simply terminate the application.

4) Should exit() be called kill() to be more inline with unix comdline kill being used to send signals? Or how about sendsig()? exit() and kill() are both a little off since a common response to SIGHUP is to simply reload a config file but not exit.

5) I really like the interface.


Thanks! BitHub has sent payment of  $33.75USD for this commit.


Повесь на body класс page.
Наша страница один глобальный блок.


This seems more verbose than `for (var img of this.els.container.querySelectorAll('img')) {`. Is there a reason you didn't use that syntax instead?


do you mind to comment this LUA script telling what it is doing ? :)


could you also test that the publishing happens automatically?


The only pertinent files here are huxley/api/templates/invoice.html, huxley/api/views/school.py, requirements.txt, huxley/www/static/js/huxley/components/AdvisorProfileView.js
All other files should be ignored. My bad


Yes, it was exactly the point to do nothing. This is roughly the idea I had in mind:

{% highlight c# %}
public IList<Contact> GetContacts(IContactRecordFilter filter, string searchTerm) {
   [...]
}

public IList<Contact> GetContacts() {
   this.GetContacts(new DoNotFilter(), null);
}

private class DoNotFilter : IContactRecordFilter
{
   [...]
}
{% endhighlight %}

Of course duplication can be avoided through other means, e.g. "helper" method. However with filter implementation no longer being done on ctor level one may go further and throw searchTerm away from the method. Instead the searchTerm can be passed into Soundex filter implementation directly via ctor (as a list of strings preferably to avoid "hidden" assumption about the string format). This way allows to keep the interface clean and make the code easier to enhance in future when we run into the requirement to filter on another field (like phone) or mix of those (composite specifications).

Having that said I am fine with the solution you proposed. Soundex was optional piece either way. I want you to focus on adding some unit/integration tests, which will conclude the task.


This fixes iOS and static Mac builds for me when using cpp.Prime (I have not tested the traditional cpp.Lib.load but I believe it would work)

You might want to split this into an `#ifdef STATIC_LINK` similar to CFFI.h, I wasn't sure if there was a harm in including it always?


Done in 4b1416d372c3cf4146855ada9e6444cb6649648e.


WeizmannHorse would be more appropriate given PEP8's guidelines.


if "pbcore" not in r


"this be method" seems like a typo.


Part of iqss/dataverse#2550


You should add getRadar as a condition for display in the docstring. If at some point I want to remove the display dependency, I would like to know what is needed. Also add the getRadar test in line 448.


this is illegal! do `self.Vgatefilter.change(gatefilter, False)`


I think this crept in by accident from a later commit :)


no but you can try v1.4pre1 which has the fix: http://beakernotebook.com/downloads


Thanks! Tested and now it passing all the tests with Debug configuration. :+1:


We actually still need the `if ([UIAlertController class])` here. Otherwise this code will crash on iOS 8 when Aardvark is built from Xcode 7.


Since we're using this from abdcmd_init, it should be a public method so no underscore required.


It would be nice if this change and the updates to the related tests appeared in the second commit along with the other changes to that feature.


Could help?

```
    interface IFilterService {
        /**
         * Usage:
         * $filter(name);
         *
         * @param name Name of the filter function to retrieve
         */
        (name: "date"): IFilterDate;
        (name: "number"): IFilterNumber;
        <T>(name: string): IFilterArray<T>;
        (name: string): IFilterFunc;
    }

    interface IFilterDate {
        (date: Date | number | string, format?: string, timezone?: string): string;
    }

    interface IFilterNumber {
        (value: number, fractionSize?: number): string;
    }

    interface IFilterArray<T> {
        <T>(array: T[], expression: string | IFilterPatternObject | IFilterPredicateFunc<T>, comparator?: IFilterComparatorFunc<T> | boolean): T[];
    }

    interface IFilterFunc {
        (source: any, ...args: any[]) : any;
    }
```


Ah, I will adjust my editor to conform to these formatting conventions. Are they documented somewhere in the project? I did not see.


Yes, right, I forgot about that.


thank you! I wanted to avoid building from source, so this is great :)

2015-09-21 17:14 GMT-04:00 Scott Draves <notifications@github.com>:

> no but you can try v1.4pre1 which has the fix:
> http://beakernotebook.com/downloads
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twosigma/beaker-notebook/commit/0cd2246b336dc477fdb8a3323628c62967b527d5#commitcomment-13353312>
> .
>


Probably could.  When that code was written there was no ldr_in_pot variable.

Now that we have a little 'state' information available about where in we are in the process, it might allow this crap logic to get placed down in the format where it should be.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 63](http://teamcity.otenv.com/viewLog.html?buildId=1520567&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


I don't think we should/need to initialize with the filter mask. As said gatefilter is different then field mask. The final mask is always gatefilter._gate_excluded || field_mask, this should and is done by pyart


Do we need a test for the `template_name='unknown'` case too? Especially since that didn't seem to work in an earlier version of the PR (displayed "None" instead of "unknown").


Oh, great!!! :-)


Whoopsy daisy, typo on the second line there  :grimacing:  About to push a fix


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 63](http://teamcity.otenv.com/viewLog.html?buildId=1520567&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 37; exit code 1 Build time: 0:0:0


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1866 of 1875 tests passed)

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


Here I set the ExcelParser Instance to global access


This commit looks good. Just in the assertion order in the tests, I think it would be more intuitive to check the message first, then check the exception details, as done in your first commit.


no modifier = only class and package visibility


Add period


No! just delete this function, if `self.sharedVariables['Vgatefilter'] is None` you shouldn't be using it any way. If you are, it's wrong (it is that simple).


Outstanding Update, this really helped clean up my Documentation!

I implemented my own Modulemanager for my project, which resulted in Files being in many different Folders, this Update sorts them all neatly in the sidebar "folders". great job!


Actual it is 381 warnings in ControlName.vb, ControlNamePackage.vb that were added to the VSIX project. The warnings are "Type or namespace definition or end-of-line expected", "Newline in Comment", "Type Expected" , "Too many characters in character literal"., "Comma Expected". None of which make any sense in a VB file. If I move these files to a VB project most of the errors disappear and the ones that are left are all correct..
```
<?xml version="1.0" encoding="utf-8"?>
<PackageManifest Version="2.0.0" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011" xmlns:d="http://schemas.microsoft.com/developer/vsx-schema-design/2011">
  <Metadata>
    <Identity Id="Pure_Visual_Basic_Refactoring.Vsix..ac920cbf-a241-4baa-a5d7-5b3b8661ecb9" Version="1.0" Language="en-US" Publisher="Paul M Cohen" />
    <DisplayName>Pure Visual Basic Refactoring</DisplayName>
    <Description xml:space="preserve">This is a Code Refactoring and Analyzer written in Visual Basic by Paul M Cohen</Description>
    <License>LICENSE.txt</License>
    <Icon>ms_visual_basic_express.png</Icon>
    <Tags>Refactoring, code analysis, analyzer, Roslyn, VB.NET</Tags>
  </Metadata>
  <Installation>
    <InstallationTarget Version="[14.0,]" Id="Microsoft.VisualStudio.Pro" />
    <InstallationTarget Version="[14.0,]" Id="Microsoft.VisualStudio.VSWinDesktopExpress" />
    <InstallationTarget Version="[14.0,]" Id="Microsoft.VisualStudio.VWDExpress" />
    <InstallationTarget Version="[14.0,]" Id="Microsoft.VisualStudio.VSWinExpress" />
  </Installation>
  <Dependencies>
    <Dependency Id="Microsoft.Framework.NDP" DisplayName="Microsoft .NET Framework" d:Source="Manual" Version="[4.5.2,)" />
    <Dependency Id="Microsoft.VisualStudio.MPF.14.0" DisplayName="Visual Studio MPF 14.0" d:Source="Installed" Version="[14.0]" />
  </Dependencies>
  <Assets>
    <Asset Type="Microsoft.VisualStudio.MefComponent" d:Source="Project" d:ProjectName="Pure_Visual_Basic_Refactoring" Path="|Pure_Visual_Basic_Refactoring|" />
    <Asset Type="Microsoft.VisualStudio.Analyzer" d:Source="Project" d:ProjectName="Pure_Visual_Basic_Refactoring" Path="|Pure_Visual_Basic_Refactoring|" />
    <Asset Type="Microsoft.VisualStudio.VsPackage" d:Source="Project" d:ProjectName="%CurrentProject%" Path="|%CurrentProject%;PkgdefProjectOutputGroup|" />
  </Assets>
</PackageManifest>
```


YES, that is right!


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 68](http://teamcity.otenv.com/viewLog.html?buildId=1520580&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


Suggest adding "ccd" and "side" to levels/visit and levels/skyTile.

HSC also has a levels/tract (with value "patch"), but not sure it's important.


пусты дивы не очень хорошо оставлять. Давай для начала поместим внутрь просто крестик
http://unicode-table.com/ru/1F7A9/


Good catch. Added and will be in next update.


What if ret < 0? Shouldn't this be return ret != 0;


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 68](http://teamcity.otenv.com/viewLog.html?buildId=1520580&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1 Build time: 0:0:0


TeamCity pull requests :: netty [Build 231](https://garage.netty.io/teamcity/viewLog.html?buildId=1526&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5606, ignored: 21 Build time: 00:37:37


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1346 of 1346 tests passed)

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


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


Would it be inconvenient to make CMake emit comparatively less `vcxproj` files? Currently the solution opens 43 project and in IDE, it looks like:

![image](https://cloud.githubusercontent.com/assets/3840695/10005447/025b6d14-60c0-11e5-8ddb-48f6d92c7b22.png)


Does this mean I can potentially build against a netCDF that has HDF5 support enabled now?


Are you still working on this Venelin? Django 1.9 Alpha is due for release today and ImageKit is not compatible with it. I'm available to take this on if need be.


Yep, I caught these (this and previous comment) and changed. Will be updated soon.


more testing!


I don't think this function can block. Not sure we need to add coop support for it.


Is it ok to leave this on in production?


Other way around:

    #ifdef HAVE_PCAP_FINDALLDEVS
    			if (pcap_findalldevs(&devpointer, ebuf) >= 0 && devpointer != NULL)
    				device = devpointer->name;
    #else /* HAVE_PCAP_FINDALLDEVS */
 			device = pcap_lookupdev(ebuf);
    #endif

putting the newer, better behavior first, with `pcap_lookupdev()` as a fallback.


TODO: Single-Step testcases should not depend on control mechanism at all anymore, I will work out a configuration which won't add derivations to memory which is used on single step testcases.
Here is what this commit changed in terms of working testcases:
http://i.imgur.com/T8UbfdN.png
The + don't indicate that these testcases don't work, just that they didn't work under the given control bad luck. In fact this commit made 2 new testcases succeed, the ones marked with -.


sorry for the mistakes.


Sorry for the confusion :)

I meant that writing just:

```
interface {
    // GzipBytes returns gzip compressed contents of the file.
    GzipBytes() []byte
}
```

Makes me think that this interface is not complete / is part of some larger interface (i.e. you have stripped some irrelevant information from the definition), versus writing the full interface definition such as:

```
// GzipByter is implemented by all compressed files for
// efficient direct access to the internal compressed bytes.
type GzipByter interface {
    // GzipBytes returns gzip compressed contents of the file.
    GzipBytes() []byte
}
```

Or if that's too long for your taste just even:

```
type GzipByter interface {
    GzipBytes() []byte
}
```

(but I do think the comments also add useful information)


nice pun


Is there any reason why this is PSub and not PMinus? I don't care either way, but we should be consistent and some other code uses minus terminology. Same for PAdd/PPlus.


version 1.2


@jonsterling I have a blog to wrap sentences around bad puns after all


The `Asset Type="Microsoft.VisualStudio.VsPackage` is exactly what you need.

It sounds like you're adding a VB file to a C# project.
Change your project type.


It should add version='auto' so that the client will use the same api version that the server uses to avoid version conflict.


I understand now, thanks.

Sure, I'll go for a full type definition rather than just an anonymous interface. Even though it's not actually present in this package. But it will be easier to copy and paste.


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


this should be articles.routes


Updated with these and a couple other small changes.


not done for performance.  "optimized_write" mode is incompatible with "save_virtual_workbook().
see See https://bitbucket.org/openpyxl/openpyxl/issues/375/use-save_virtual_workbook-and-optimized


За то удалил, что задание размера от 2х до 50ти символов покрывает пустую строку, так что это лишнее.


This shouldn't be needed. What was the problem?


Cool! Will take a look and incorporate the changes into v2 which is being worked on now...


Hi,

If I may, when your feature is ready I'd like to merge it into the repository from which you forked.

Best regards,

Benoit


:+1: this is already merged though, care to submit a PR @jlamcanopy?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 673](https://teamcity.labkey.org/viewLog.html?buildId=273321&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:56
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Did you get to compile successfully with this config? I have tried all possible combinations with no success.
It fails with "label or path, 'i2c_5', not found"
I managed to fix this by enabling "CONFIG_MMI_DEVICE_DTBS" but the kernel does not boot


@am11 [here you go](https://github.com/Snaipe/Criterion/commit/86a1707b75e97ab9c15a5f986b2220a66fd19230)


Warning: getRadar may return None


@smadha let's put all the custom CSS into style.css file under /static/ folder.
Thanks


there is this https://docs.djangoproject.com/en/1.8/ref/models/instances/#django.db.models.Model.refresh_from_db


Please keep the DNS port configuration, it is very useful.


Great work, I still want to see the final comparison but it seems ok.

Are you already getting the solo-like behavior you want?


The VSIX project includies the line below but I have never seen this line is a VB Project file and all the errors match exactly what you would get if you add the VB files in a C# project and rename them to .cs.
  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />


Delete this comments!


could you move that into the actual tearDown method? this way a failure here doesn't affect other tests. remove the other occurence in this file as well


As above!


See WIKI for more content


I patched in the above change into an older version of john (even older than where @magnumripper cached off the calls to dyna_is_valid, but I ONLY check that first loop, just like I do above, so the differences should not matter at all.  Here was the timings (same machine).

```
$ ../run/john in
Time used was 0.058931
```

So, we went from ~60ms to .01ms.  Pretty good gain, lol


Finally someone could fix this warning :D


i just read this: https://docs.djangoproject.com/en/1.8/topics/testing/tools/#emptying-the-test-outbox so maybe remove it altogether.


Doesn't take into account custom steam library locations.  You'd have to find the vdf and parse that out I think.


OK, anyway it's not a big deal. They aren't documented :cry: so I'll try to write something about it.


any docs would be nice


I really like this change, but we reached peak ifdef sanity here.

Maybe one way to fix this is to move the init code to the backend-specific code.

We can assume that backends will match each other so maybe a single USE_ASYNC_SYSCALL_ABORT define
would be enough to know whether to include the code or not.


This was committed to branch jb01 (which looks very hash tag like).


Do we need a file lock context here? I think regular open would be fine:

```
with open('testfile', 'w') as src:
```

You could also use `phlsys_fs.write_text_file` for convenience instead here:

```
phlsys_fs.write_text_file('testfile', content)
```


Why not just contribute back? This makes the open source community stronger (and happier).


It only works when running the application having the current working directory pointing to the git repo. This is not a fair assumption.

If you do `go get` for example, it will add the executable to the `PATH` and your current working directory will be somewhere else.


I have nothing in C# but something is assuming that files in a VSIX project are C# even if they have a .VB extension. How to I change a project type for something that Visual Studio created automatically. If I include the two fines in a C# project I actually don't get the errors and Visual Studio seems to understand they are VB (I didn't try to actual compile).


Hi @brushtyler ! :)

* "save as" do not offer to save as geopackage. Anyway the issue I was referring is that apparently is not possible to create an empty geopackage, allowing later (maybe via DB manager) to add in it layers

* try any of the geopackage samples http://www.geopackage.org/#sampledata

* Ok

ciao!


did you mean to make this change?


I think I need to Address this with the Roslyn team or Visual Studio support.


I don't see the point of this change.

We don't need this for targets with a functioning core abort, there's already code in place in the managed side of socket to abort those calls.

And this code won't work on coop since it doesn't have syscall abort available.


Since Gentoo doesn't expect a `reload` perhaps the correct fix here is to just remove `reload()` case for Gentoo.  Or does Gentoo just use a different name for this functionality?


Wrong class name


No, but this kernel should compile and boot (it did for me at least): https://github.com/Thecrazyskull/kernel-msm


Hello World Program in Visual Basic


If frame_info's impl is screwed up, this is still safe -- you'll just wait on the fence if you happen to pick a frame that's still in flight.


I did it manually against develop.
You can close that one.


Looks like Ken's commit is back in here.  However, I'm concerned that the docstring is wrong. Ken told me that he changed the supports all to TRUE in driver.py, so I should change these all to TRUE (I did).  But the init function in this driver sets these three to false, so I assume they would be false in the docstring.

         self.supports['multiple_objectives'] = False
         # TODO: Support these
         self.supports['linear_constraints'] = False
         self.supports['integer_parameters'] = False

(I would generate it myself, but I don't have pyoptsparse installed here.)
So perhaps I don't get what's going on properly here, which is fine, just let me know.


renamed files to be case sensitive


This should read `require "ember-cli/deploy/redis"`


I think this will create an infinite loop..


Posting here, because you quit IRC after posting this:
now "sigrok-cli --driver=demo -O analog --samples 10" and "[...] --continuous" work, but it doesn't work if I add "--config samplerate=X" for X <= 24. It starts working for "--config samplerate=25" an bigger values. :D


if failed on precondition we should consider this test as failed, shouldn't we?


LGTM


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 674](https://teamcity.labkey.org/viewLog.html?buildId=273330&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:25:23
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


There's a typo in Thursday and Wednesday and Friday should be split to separate lines.


H2 is upgraded in trunk


@nalla This is an awful default.

- Regular users of Vim can easily set this if they want.
- Casual users won't expect their clipboard to be hosed by a `dd`.
- Regular users who expect the standard Vim defaults must now edit their vimrcs just for Git-for-Windows.


For what it's worth I do think @jvasileff has a good point. The difficulty would be in convincing others that it's worth breaking existing code/compatibility for.


Only if they're adding fencing levels with stonith_admin at the same time.
All I'm doing here (and other places) is having the helpers construct and parse the format that is already used in the cib.


Now that we don't need the extra `isWrapperFor` method, let's just remove this class


Any reason for removing the postinstall? Found it really helpful. Obviously I'll keep it in mine but was curious of the reasoning!


Interestingly enough, it's the same defconfig I have. I'm probably messing up the dt image compilation somewhere. Thanks, though.


do you need to call retain in the constructor?


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1882 of 1882 tests passed)

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


Why did it go from "export" to "import" ??? It does that earlier in the commit as well.


OK then this was actually really good! A tenth of a second is nothing on its own, but they really quickly add up to a full second.

There is a VERY noticable change in speed from before Solar's optimizations compared to current code, when you load lots of hashes, or even just do a --show with a tiny input file but a huge .pot file.


Sure.


leave the `sharedPageSource` as an `Optional`, and check `isPresent` below


@wsxarcher I removed it temporarily for debugging, will re-add it once I've finished


Argh, yep


This should fix - or at least very much improve - #7.


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


4967-SZRH-spm (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1852 of 1882 tests passed)

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


This should probably be `return fields.get(self.status)` instead of using a `for` loop.


@Snaipe, I did:

```cmd
:: in cmd

rmdir /s/q \test\Criterion

cd \temp

git clone https://github.com/Snaipe/Criterion --recursive && cd Criterion && cmake . && ^
cmake build . && criterion.sln
```

and still it produces 43 projects.

On this note, with `-DTEST` knob in place, do we still need to run `cmake --build . --target criterion_tests` command separately to build the `ctest` runnable tests (that is; a separate target for tests)?


Es ist nur, es gibt zwei footer ....
http://zeichen-und-symbole.bwg.hu-berlin.de/en.html


just assign `buildPageSource = null` without the `if`


This covers issue #45


you don't need the `!closed` check as `close()` does this internally


Should we instead use the jquery we have in the repo? Happy to let you decide :-)


![aardvarks](https://cloud.githubusercontent.com/assets/6785790/10006168/5e32630e-6089-11e5-8214-f49004d6a847.jpg)

[Source](https://commons.wikimedia.org/wiki/File:Aardvarks.jpg) CC-BY-SA


Same for bootstrap


Merged! Let me know if everything still works!


Notice the name of the function is `searchOpenTransaction`....  A opened transaction is one which has a state of `'Waiting'` or `'Pending'` but NOT `'Completed'`. The logic of the function is modified to something that is incorrect.


bis zum 21. ist doch richtig, oder?

den footer gibt's nur einmal sprachunabhängig.


На такую разметку лучше не налегать.
Получается что у тебя .content__post - это блок, а header - элемент.
Не бойся бить на блоки.
.blog-post<.blog-post__header+.blog-post__title+.blog-post__date+.blog-post__description


Have you renamed the factory?  Is the factory no longer called `OpenTransactionsFactory`?  What was wrong with the previous code?


4967-SZRH-spm (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


> On Sep 21, 2015, at 8:12 AM, kyang2014 <notifications@github.com> wrote:
>
> Not sure if I understand your question,
> scaled_dname_set_marker is a set of string, the element inside this set can be safely erased. There is no invalid pointer issue for this case.
>
>
Ah, OK.

> —
> Reply to this email directly or view it on GitHub <https://github.com/OPENDAP/hdf4_handler/commit/df6595dec0ffe66c659c0f9b09134e37fc5a6362#commitcomment-13343360>.
>

--
James Gallagher
jgallagher at opendap.org
406.723.8663


It is possible to get a page with no rows in some cases.  I suggest you filter these out in `addInput` and when creating the `buildPageIterator`


A lot of our overhead is in valids()   Some of that is our own foot/gun issue where we get so overly anal about making sure there is no way to core john, but at the extreme of a lot of time used.


```sh
$ replem 'plaid/sanctuary#dc-composition:S' ramda:R
Installed into REPL context:
 - sanctuary#be9ea1d as S
 - ramda@0.17.1 as R
> S.meld([Math.pow, R.subtract])(3, 4, 5)
```


Please have aa look of this link http://prntscr.com/8izt5u . There are 3 blank menu items. How to remove it?


Опять-же создай отдельный блок pagination.


This one tests #116.


Looks good. Only 2 more comments:

1) Is the atomic<bool> m_running needed? The while (m_running) could simply be while (true) if the break at 381 becomes return exitHandler()

2) If I call kill(SIGHUP) it will set m_running to false, causing the while loop to break out, causing us to exit even if the signal handler returns false. kill() shouldn't seek to prematurely exit the while loop since the signal will be delivered to the wait() and regular handling of the process will occur there.


:+1:
Could you file an issue to add an EnC test?


It looks like there are TABs mixed up in here.


Someone else may want to properly document this --show=type  I was not familiar with it, but when adding the --show=invalid  I found that, and simply stuck this into the OPTIONS doc.


Not yet. It does not seem to be updating the Vgatefilter, which could be a function of not being "linked" properly.
But, I'm also wondering if this functionality should be added into the GateFilter plugin, keeping all of the filtering in one place. Thoughts?


Yes, pthread is required (only for "examples" and "test performance" compilations), otherwise you will get:

[ 39%] Linking CXX executable g3log-FATAL-choice
/usr/local/bin/cmake -E cmake_link_script CMakeFiles/g3log-FATAL-choice.dir/link.txt --verbose=1
/usr/bin/CC   -Wall -std=c++11 -stdlib=libc++ -Wunused -D_GLIBCXX_USE_NANOSLEEP -O3 -DNDEBUG   CMakeFiles/g3log-FATAL-choice.dir/example/main_fatal_choice.cpp.o  -o g3log-FATAL-choice  libg3logger.a -lexecinfo
CMakeFiles/g3log-FATAL-choice.dir/example/main_fatal_choice.cpp.o: In function `_ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvNS_8functionIFvvEEEEJPS3_EEEEENS_6futureIT_EEOT0_':
/ara/incoming/g3log-master/example/main_fatal_choice.cpp:(.text._ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvNS_8functionIFvvEEEEJPS3_EEEEENS_6futureIT_EEOT0_[_ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvNS_8functionIFvvEEEEJPS3_EEEEENS_6futureIT_EEOT0_]+0x77): undefined reference to `pthread_create'
CMakeFiles/g3log-FATAL-choice.dir/example/main_fatal_choice.cpp.o: In function `_ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvvEJEEEEENS_6futureIT_EEOT0_':
/ara/incoming/g3log-master/example/main_fatal_choice.cpp:(.text._ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvvEJEEEEENS_6futureIT_EEOT0_[_ZNSt3__124__make_async_assoc_stateIvNS_12__async_funcIPFvvEJEEEEENS_6futureIT_EEOT0_]+0x6f): undefined reference to `pthread_create'
libg3logger.a(logworker.cpp.o): In function `kjellkod::Active::createActive()':
/ara/incoming/g3log-master/src/logworker.cpp:(.text._ZN8kjellkod6Active12createActiveEv[_ZN8kjellkod6Active12createActiveEv]+0x6f): undefined reference to `pthread_create'
CC: error: linker command failed with exit code 1 (use -v to see invocation)
*** Error code 1


Давай всё-таки меню разобьём на два блока. Блок main-menu и page-header__menu и смиксуем их.


static import


Read carefully (hint: copy paste, that's why I want this code cleaned up)


I'm not sure why it does not work since the value of TESTS if OFF by default.

It works if you explicitely add -DTESTS=OFF to the cmake command.

> On this note, with -DTEST knob in place, do we still need to run cmake --build . --target criterion_tests command separately to build the ctest runnable tests (that is; a separate target for tests)?

Yes, criterion_tests is mostly there to avoid relinking 43 samples each time you change something in the shared library and just want to compile it.


add:
```java
if (!hasNext()) {
    throw new NoSuchElementException();
}
```


HI GUYS


Yes, it should work now.
On Sep 21, 2015 5:23 PM, "Gabriel A. Devenyi" <notifications@github.com>
wrote:

> Does this mean I can potentially build against a netCDF that has HDF5
> support enabled now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BIC-MNI/libminc/commit/0adf572d93539d2e08fb9ec6fa56d75adc5e5e6d#commitcomment-13353512>
> .
>


TeamCity pull requests :: netty [Build 232](https://garage.netty.io/teamcity/viewLog.html?buildId=1527&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5601, ignored: 21 Build time: 01:12:26