optional_new_state is dead... delete it


@mrunalp Ok, will work on integration test.


@rhatdan Who is writing docker patch


no pointer here


Same as abover: don't start error strings with a capital letter, and don't fix unneeded things in an unrelated commit ;)


The syntax is the same.

python import_products_to_bigcommerce.py categoryname partsnumberfile.txt


and for the User object I don't want the entire user object dumped, it will quickly overflow the 255 VARCHAR field storing AUD_RESOURCE for the log


Please remove this obviously forgotten debug function call.


@dukesrg I'm going to remove all tools because all of them have bugs and it can be barrier for the further development. The code should be easier to read than font_tool.


> We can't use shaders from 2.1 with 3.2 context.

Can anybody actually confirm this?

> And 3.2 is required for RenderDoc, which is used for renderer debugging

Is this so critical now? Does the project decisions depend on a specific product which is not even required to run the executable? I don't believe that this is a technical reason and it wasn't even discussed before making such crucial decisions.


Is this retrying thing really useful for people? I always disliked it, but kept it for the sake of those that may need it...


"Seen by"

We aren't (*yet*) chopping people!


@173210 human unfriengly code will have an explicit values in place of long calculations with defines. Well, font_tool was really unreadable if you don't know what it is goint to make.


Double close?


It has to be @synchronized(self) or implement _IPAddress (and other contained objects) as atomic properties (Objective C would then use spinlocks to ensure atomic access).

The problem with @synchronized(_IPAddress) occurs between 0x100034b70 where _IPAddress is in rax and objc_retain actually incrementing the retain count.  It is possible for the packet capture thread to release and free the original _IPAddress (the one that is in rax in the UI thread) before objc_retain in the packet capture thread increments the retain count.  objc_retain then tries to retain an object that has been freed.

```
- (NSString *)getIPAddress{
    if (_IPAddress == nil) return @"unknown";
    @synchronized(_IPAddress) {
        return _IPAddress;
    }
}
```

```
KisMac2`-[WaveClient getIPAddress]:
    0x100034b20 <+0>:   pushq  %rbp
    0x100034b21 <+1>:   movq   %rsp, %rbp
    0x100034b24 <+4>:   subq   $0x40, %rsp
    0x100034b28 <+8>:   movq   %rdi, -0x10(%rbp)
    0x100034b2c <+12>:  movq   %rsi, -0x18(%rbp)
->  0x100034b30 <+16>:  movq   -0x10(%rbp), %rsi
    0x100034b34 <+20>:  movq   0x101e2d(%rip), %rdi      ; WaveClient._IPAddress
    0x100034b3b <+27>:  cmpq   $0x0, (%rsi,%rdi)
    0x100034b43 <+35>:  jne    0x100034b61               ; <+65> at WaveClient.m:265
    0x100034b49 <+41>:  leaq   0xe2958(%rip), %rax       ; @"unknown"
    0x100034b50 <+48>:  movq   %rax, %rdi
    0x100034b53 <+51>:  callq  0x1000db866               ; symbol stub for: objc_retain
    0x100034b58 <+56>:  movq   %rax, -0x8(%rbp)
    0x100034b5c <+60>:  jmp    0x100034bc8               ; <+168> at WaveClient.m:268
    0x100034b61 <+65>:  movq   -0x10(%rbp), %rax
    0x100034b65 <+69>:  movq   0x101dfc(%rip), %rcx      ; WaveClient._IPAddress
    0x100034b6c <+76>:  movq   (%rax,%rcx), %rax
    0x100034b70 <+80>:  movq   %rax, %rdi
    0x100034b73 <+83>:  callq  0x1000db866               ; symbol stub for: objc_retain
    0x100034b78 <+88>:  movq   %rax, %rcx
    0x100034b7b <+91>:  movq   %rcx, %rdi
    0x100034b7e <+94>:  movq   %rax, -0x28(%rbp)
    0x100034b82 <+98>:  movq   %rcx, -0x30(%rbp)
    0x100034b86 <+102>: callq  0x1000db884               ; symbol stub for: objc_sync_enter
    0x100034b8b <+107>: movq   -0x10(%rbp), %rcx
    0x100034b8f <+111>: movq   0x101dd2(%rip), %rdi      ; WaveClient._IPAddress
    0x100034b96 <+118>: movq   (%rcx,%rdi), %rcx
    0x100034b9a <+122>: movq   %rcx, %rdi
    0x100034b9d <+125>: movl   %eax, -0x34(%rbp)
    0x100034ba0 <+128>: callq  0x1000db866               ; symbol stub for: objc_retain
    0x100034ba5 <+133>: movq   %rax, -0x8(%rbp)
    0x100034ba9 <+137>: movl   $0x1, -0x1c(%rbp)
    0x100034bb0 <+144>: movq   -0x30(%rbp), %rdi
    0x100034bb4 <+148>: callq  0x1000db88a               ; symbol stub for: objc_sync_exit
    0x100034bb9 <+153>: movq   -0x28(%rbp), %rcx
    0x100034bbd <+157>: movq   %rcx, %rdi
    0x100034bc0 <+160>: movl   %eax, -0x38(%rbp)
    0x100034bc3 <+163>: callq  0x1000db860               ; symbol stub for: objc_release
    0x100034bc8 <+168>: movq   -0x8(%rbp), %rdi
    0x100034bcc <+172>: addq   $0x40, %rsp
    0x100034bd0 <+176>: popq   %rbp
    0x100034bd1 <+177>: jmp    0x1000db7ee               ; symbol stub for: objc_autoreleaseReturnValue
```


hehe :-) it's a comment it's fine


Oop!  Watch out for the [mutable default argument](http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments) gotcha here.


I don't see it here, but what is the value of TIRE_CHANGE_STR_MOD?
 Will I be able to change the tires on my twenty by twenty square cube of steel plating and engines with twenty strength?


@173210 then stary with the python


@rhvgoyal You can look at tests in libcontainer/integration/exec_test.go


s/downloadig/downloading/


That makes sence. Warning = unusable in production application and warning is also harder to test. Why not go for an exception. You will get immediately clear response what can be wrong with integers. There can be event short explanation in exception. It will be super clear then, why it accepts just and only strings.

BTW: integer with leading zeroes => it is converted to octal


I think not using the `SOL_FLOW_NODE_TYPE...` prefix here was intentional, since this sample pretends to be application code and we use prefix for stuff that is provided by us.


Most of this code can probably refactored into a working REST server, see my comment at https://github.com/restic/restic/pull/253#issuecomment-128130604


@mrunalp Will do. thanks for the pointer.


I can as soon as we have the runc patch.


Is it possible just to use `user.toString()` here (I don't believe there is a password in the returned value).


This is a very nice contribution. I think that we should split it into a separate file. I would certainly like to beat it a bit more in other projects, so that we can expose it to more widespread usage.


I don't want the entire object serialized, I only want relevant portions


Вот эти все find, all, лучше убрать в отдельный модуль, чтобы разделить ответственности.


`!is.na(seniors)` ?


How does this work? `fmt.Fprintf` writes `Blob already uploaded` to w, and the additional parameter `404` is ignored... I suspect that replacing `fmt.Fprintf` with `http.Error` could work.


Try instead something like:

```python
def jsonp(body, headers=None):
    headers = headers or {}
    ...
```


Same as above, probably meant `http.Error` instead of `fmt.Fprintf`.


resons -> reasons


Please tell me what you think.


Oops. I messed up the code. Will fix


@dukesrg I just mean the less code is the easier to understand. Anyway, it's also difficult to read. I couldn't understand this line, so I simply removed the line and fixed the output to know what it does.
https://github.com/roxas75/rxTools/blob/a8d84dd80e51bb60e3bf510d8fa64c369942f035/tools/toolsrc/font_tool/main.c#L62


BTW warnings are harder to debug (no stacktrace)


Yeah, that looks good, nice stored proc work.

I should share using a binary(16) column before going this varchar(32) route, it's even more compact.  I'd imagine people with large instances would really like it.

My tiny install with about 64k rows.

-rw-rw---- 1 mysql mysql 29360128 Aug  5 14:44 binaries.ibd      -- varchar(255)
-rw-rw---- 1 mysql mysql 18874368 Aug  5 14:41 binaries.ibd      -- varchar(64)
-rw-rw---- 1 mysql mysql 16777216 Aug  5 14:41 binaries_tmp.ibd  -- binary(16)

11% reduction when using varchar(64)
42% reduction when using binary(16)


@mSourire ИМХО, имеет смысл вот это коммуникатор сделать классом и его использовать явно там, где ты его подключаешь как модуль. Композиция лучше миксинов.


*Three weeks later...*
Oh no ! What have I done? WHAT HAVE I DONE?


Already within the MathUtils namespace. Makes linux very unhappy.


You're right. I forgot to commit that change


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

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


comprehensive awareness? To właśnie chciałeś napisać?


В общем, миленько.


Nice line. :-) We guarantee, I like it. Minor: did want - wanted?


@dukesrg I'm removing code easy to do so.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.417.064371a46ce9ea95a376bd632d0a19ef51074189](http://build.palaso.org/viewLog.html?buildId=60640&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:19:28


Hey, I wouldnt do this, because it breaks compability where we don;'t have GNU makefile (which I cannot think of, but anyway, there is no that much pain in using POSIX makefile)


@173210 That was a transposing 16x16 glyphs with line byte-swapping


Doh!  This should be ``headers=headers or {}`` I think.


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2214 of 2234 tests passed)

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


@Zren
Where is/was your first intended use for this in the existing code? I think I might have an intended use for it in another issue but I'd like to know what your thoughts were at the time.


Well, in general returning objects (non POD, non pointers) from functions is more inefficient than passing them in and filling them. The performance depends on how the compiler implements it and sometimes it involves obscure operations. In the QList case the performance penalty would be very low since QList is implicitly shared, so it's more
a matter of personal taste. However note that unless the compiler optimizes it heavily (and in a non-obvious way) you still manage two objects (the one inside the function and the returned one) and call at least one function for the assignment (either assignment operator or copy constructor).


:D wasn't here when I first looked at setting an app up... you got me!


Yeah sorry about the whitespaces. Not sure how that happened.


Maybe this should be called `matchPathSegmentString`.


If the decision is to reduce the capability of debugging the application or reducing the required OpenGL version just to make some users happy that have dated hardware, I'm out. This just annoys me.


Thanks, Mr. Java :)


Обновите для Prestashop хотя бы после 1.6.0.14


Welcome back @tux-mind 😀


MILLISECONDS is 64-bit, don't you think it's enough? I can use 60i64 if you prefer.


:P


Creating getters and setters for the AWS operation that chained up errors in the callbacks.


Good point. Chciałem napisać że po 12 sesjach będą wiedzieć co jest grane. Może po prostu (simplest possible)

Cover it in such a way, that after these 12 sessions all attendees will know _what's the fuzz about those microservices_.


> Can anybody actually confirm this?

This is true on OS X at the least. It is probably true on all system where a true Core Profile context is used (as opposed to a Compatibility Profile one).


Is they a way I can remake this without all the hex binary crap?
On Aug 5, 2015 4:50 AM, "idgrepthat" <notifications@github.com> wrote:

> Search
>
> "\x52\x65\x67\x69\x6F\x6E\x54\x68\x72\x65\x65\x2E\x64\x61\x74"
>
> in index.html. That's RegionThree.dat in hex binary. Replace it with
> whatever.
>
> the full decoded html of that variable is:
> RegionThree.dat
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/idgrepthat/3DS-MultiLauncher/commit/b202aaebb848905c78c69f48eb8d97a2a7791c8b#commitcomment-12537550>
> .
>


@justinedelson on surface looks good; ill have to pull locally to try it out (not very familiar w/ coral2)


upside / downside, together, not separated. Separated would dark and light sides. Up- and darksides perhaps?


@dukesrg Ok, so, (probably) only the author can answer the meaning immediately. I'll remove such a code.


We didn't want to just praise, we went a step further... we wanted to go a step further?


Good to have you back. Let me know how I can help.


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/9b828126d7c69d4b69a6fd3a6559912b4655d0ac


merging basho/riak_kv/bugfix/incorrect-2i-pagination-finish-2.0 = 9b828126 into borshop-integration-1155-bugfix/incorrect-2i-pagination-finish-2.0


basho/riak_kv/bugfix/incorrect-2i-pagination-finish-2.0 = 9b828126 merged ok, testing candidate = fe293e80


Bring the context here means?


I solved the problem regarding having no selected at all. After trying the code you suggested, it still does not center the selected item in the menu bar.


Re minor: w założeniu to miała być taka figura stylistyczna: http://english.stackexchange.com/questions/125139/i-did-want-to-versus-i-wanted-to
Ale może trochę na wyrost


font tool purpose is to convert array of 256x256 characters into array 1x65k characters with transposed (x-y switched) glyphs. Since bitmap is not LSB ordered, also byte-switching for each character line is needed. This allows to make font pointer linearly increase with screen buffer pointer. Yes I'm an optimisation geek, for me it is obvious because I made such  sings many times with x86 assembly.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.421.bff28eee27698923ad10a800862fd7628c077bee](http://build.palaso.org/viewLog.html?buildId=60647&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 130 Build time: 00:07:26


@iwarapter I'll let you review my enhancements and merge if it's ok with you.


Made our best what? Or did you mean "did our best"?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.462.c484d79bb5dc000660c49b766e470c6c57d4da1a](http://build.palaso.org/viewLog.html?buildId=60639&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:35:07


Just agenda. Since it's a title.


Let's move to #252, it's really not a good place to discuss such important things.


when running "composer install" this one gives me this error:

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - authorizenet/authorizenet dev-master requires jms/serializer xsd2php-dev as 0.18.0 -> no matching package found.
    - authorizenet/authorizenet dev-master requires jms/serializer xsd2php-dev as 0.18.0 -> no matching package found.
    - Installation request for authorizenet/authorizenet dev-master -> satisfiable by authorizenet/authorizenet[dev-master].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion> for more details.

Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

How can this be resolved?


We want to bring *up* the context where it worked....

Ukazać szerszy kontekst gdzie zadziałało gdzie niet? Jak myślisz?


Why did you split up this line? Every other CMakeLists.txt puts them on the same line.


Shouldn't there be a `OSSpinLockUnlock` here?


991757360-DocIssues-4956-4997 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 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-69.96%25-green.svg)


Porque comentou o import se você apagou o arquivo?


Ashish,

This is a Github README. I didn't touch the README.  Take it or leave it. Or maybe even improve it.


"three sic keynotes", unless you want to underlined keynotes more than three. Bold suggests otherwise.


Do not forget
```
    "repositories": [{
        "type": "vcs",
        "url": "https://github.com/goetas/serializer.git"
    }]
```
on your root package


Ah, if `MILLISECONDS` is 64-bit it's fine.


Argh! My bad, fixed


The hadoop-2.6 compatible Tachyon binaries should be available now for 0.6.4 and 0.7.0.


And I discovered we still use this during `Loading ...` pagination messages, so adding it back in.


Porque não fui eu que alterei esse arquivo e adicionei esse import, entao nao sabia se seria necessario. Achei melhor deixar comentado, porque nao estava sendo usado, em vez de apagá-lo.


Linha duplicada nas dependências


The main problems are this line and the next line.


ok, nao reparei. Fixarei.


I can see that this fix didn't took effect on the website


This should be Closeable (it has stronger requirements that AutoCloseable, such as requiring close() to be idempotent)


The discrepancy stems from the fact that previously the same error was reported twice. After lifting, the error is only reported once (as for the other 4 cases).


One-evening assignment, or assignment for one evening.

slack -> Slack? or chat. Minor.

Missing `it`, replace `for` with `to`: could have made for the conference  -> could have made it to the conference


@dukesrg Our lives are limited and we don't have time to write assembly. Take advantage of C language. If you are "an optimisation geek", you'll find it's also optimization to write human-friendly code.


One? Which of the three? :-)


TIRE_CHANGE_STR_MOD = 40 defined in veh_interact.h and equal 40
20 strength allow work with cars less then 840 kg (mass / 40 rounded down). I think you car much heavier.


if it's schedule, link directly to schedule for reader's convenience


@ds1239 Sorry, I'm not sure without seeing your entire code. Try this demo: https://github.com/paulsfds/PageMenuTest. I copied and pasted your exact page menu params in (minus the colors).


Should probably be called setParams


for now I have found an alternative to uploading photos from smartphone via
ftp connection with the app es file explorer, not from common user but for
an operator who makes a survey there may be, even good loading times


and it's gone :-)


Looks good, just move this block to a function and use it here and on max31855_process() and merge it to master.


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

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


Anything that helps is gravy.  Pretty sure we would need to change some code in PHP for that though, since we have to either convert it in PHP or use the special MySQL function to turn it back to string (best choice).

Submitted another PR for parts.  Merge them both in with yours and check the index names you made so that they line up with our convention.  I think you named one binaryid without ix_tablename_blahblah or something.


Gilt SELLS fashion, sales is a noun, not a verb. Awesome paragraph, I am kinda leaning to pushing it higher, or adding a mention about it in opening sentence. This is really good info for potential people to come.


Thanks, fixed.


done


Use text please. This feels like code commented out. Actually, if it isn't obvious here, you might want to think of restructuring the code so that the logic reads better.


Sam! Come out of hiding =P


Good section. Deserves a mention in opening for certain.


(cc @lethalman.)


Weren't we going to use http://unexpected.js.org/ as the canonical url? I've started putting that everywhere.


Fixed. ce01997


@173210 I've read tons of shitty code on different languages. And actually don't take complains about readability just after one case when I was tol to have a bad formatting, while it was just fine and there was no convention about it prior to starting the work.
Readability is different for all, unless all the code is not in one line. I'ts a bad approach to throw out a working code instead to increasing it readability, at your opinion.


Great work!


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.461.44559b532951aa7350d95dcb6d723461730d4f9f](http://build.palaso.org/viewLog.html?buildId=60638&buildTypeId=bt2) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 729, ignored: 45 Build time: 00:56:15


@Zren
So I assume that it's first use outside of any issue ticket could be [here](https://github.com/OpenUserJs/OpenUserJS.org/blob/c928907e9c88b9562970d5d79c8f4547b8533c3f/controllers/scriptStorage.js#L116-L123) using the *async* package? ... or am I missing the point?


These access keys have been deleted.


I dont have a problem with this at all, but I thought you weren't supposed to be able to watch movies until you have already encountered them in the game. If we want to change that then we should remove the code (if it exists) that is updating that now. I may just be remembering incorrectly.


The closing here can be read as "now I'm closing this post", or about "closing the conference" but it's kinda really about "after the conference I hope you guys will", isn't it?


Fixed. 8af65fa


What's the relationship between shouldFlush and "boundary seek"? I guess I don't really understand what all these concepts mean and I'm having a hard time reverse-engineering them from the code.


Hi!
Is this the expected behavior ? I find it odd that the body is being applied when its size is not set / null ; I imagine there is a reason, but couldn't figure it out from the code.
(I'm debugging an issue I have with PHP-VCR that fails when curl has the CURLOPT_READFUNCTION set without the CURLOPT_INFILESIZE)
Thanks in advance for any help/explanation.


Just released 0.2.5 to the forge.


I think we need to provide an option to use this, no? I think we can just drop it for simplicity.


Drop extra whitespace here.


yup! I didn't write the original testChecker, but it's been on my TODO list to fix at some point. the test => testChecker is totally redundant.


@coding46 glad to see your contribution - could you please contact me by email?


Why is this lock here? I don't understand why we need a lock if a deferred is returned. The lock makes callers block on `advance()` but because it returns a `Deferred` this already is an async operation. I don't think that is a good combination? Why not execute the body of `advance` on the global queue that you get with `dispatch_get_global_queue()` That will guarantee that invocations are handled serially?


So many speakers instead of so such speakers.

Not just short coffee breaks instead of not so. Breaks to get most of the conversations sounds... stiff with not so, with not just will sound better.

:bulb: Idea aside:  do we perhaps offer people something to go away to the beach? Except towels? ;-)


This is dependent on how I am implementing "keeping focus stationary". Because we can do better and this is only thing different in other platforms. I am holding of doing the same thing for other platforms.

Possible will not have to do anything for other platforms, if we find a good solution for the mentioned issue.


Don't the overlay models ("controllers") have the pointer to the overlays that they manage? If not, they really should. They main idea of those objects was really to control the overlays so this container controller doesn't need to touch them directly.

```
[self.overlayControllers enumerate…{
    [overlayController removeOverlay];
}];
```


Maybe call this just `Processor`?


Do we need a `try ... except ...` block? We only want to grab the picture once.


Actually, my response is in #533 but just to clarify, I agree that it's possible to rewrite makefiles to be portable, but since GNU Makefile is the main tool from GNU/Linux toolchain and the only reasonable make-tool for MS Windows I wouldn't worry that much about it. There is no priority in R Check not *Writing R Extensions* guidelines to write in POSIX, so IMHO that's not the issue.


اكشن للانتاج الفني


Just location. Previous section could have made some remark leading to this more explicitly. Like:

Long breaks that we offer are dictated by one more thing.

**Location**


This must be a boolean flag otherwise argparse expects an argument: 0a86e8a6f7a4d23177eccfd24d3ae94db4529e0c


Here I welcome "A". Fits well! :-)


@CurlyMoo oh I see. Yes it will be useful if you can create an account there for me (or should I create one and you will give permission to it later?)


That’s 72 characters!


Hmm I only truly localized `%c` `%X` and `%x` since that's what php and other strftimes localize. I suppose I could make `%D` be the standard date and then every `%x` would just use that. Looking now, that is what they did for 'nl_NL' at https://github.com/samsonjs/strftime/issues/54 so that is probably the correct answer.

i'm pretty sure that the `%x` I defined as `%d.%m.%Y` is correct. I've been getting all my locale information from [here] (http://lh.2xlibre.net/locale/de_DE/).


I quite liked your post. It's really eye-opening for average dev and I think it should go up. How do you want to proceed? When to publish? I'd say it might be good to publish a day or two aside from each other. :-)

Also - add few remarks about some of the content in the opening (like, I'll reveal some insider's knowledge about building the agenda and some of the talks).


They're not exactly identical. Some of the formats are different in the DefaultLocale.


:+1: :ship: it


Try using `os.path.join` to combine the folder and filename together. Also, use `os.extsep` instead of explicitly using the period.


Slim


This should be added to vendor_slim


Drop unused import.


I think the `destroy_iceberg()` inside `delete_iceberg_from_list()` will deallocate the memory that "moving_berg" is pointing to. Did this work?

If not, try replacing the call to `delete_iceberg_from_list()` with:
```fortran
  if (associated(moving_berg%prev)) then
    moving_berg%prev%next=>moving_berg%next
  else
    bergs%list(grdi,grdj)%first=>moving_berg%next
  endif
  if (associated(moving_berg%next)) moving_berg%next%prev=>moving_berg%prev
```


Going with this line of thinking… the method `updateOverlaysPositions` should also be treated the same way. This controller shouldn't manage the individual constraints or update them. It should tell the Overlay controllers to update them. Something like:
```
[overlayController updateOverlayPostiionWithYOffset:offset]
```

Basically all the logic in that method should really be in the individual overlay controllers

```
if (m.anchor && m.topConstraint) {
                NSNumber* topOffset = headingsTopOffsets[i];
                CGFloat topFloat    = topOffset.floatValue + self.webView.scrollView.contentOffset.y;
                [m.topConstraint setOffset:topFloat];
                m.yOffset = topFloat;
            }
```


Add a `-` or `_` between "background' and the timestamp.


This commit breaks the deployment of a project with sprockets 2.12.4 and sprockets-rails 2.3.2.
It is trying to use `.sprockets-manifest` instead of `manifest`.
When using capistrano 2.15.5 it works.


@dukesrg > Readability is different for all, unless all the code is not in one line. I'ts a bad approach to throw out a working code instead to increasing it readability, at your opinion.

But the new code is easier to read, isn't it? (Please remove the author's bias. Everyone thinks his or her own code shouldn't be removed by others, but it's not permitted unless they are writing a proprietary software.)


Comprehensive understanding brzmi lepiej niż awareness.


Publishing two days after you makes sense. Extensive tweeting afterwards.

Thx for comments, I think I would be afraid of your code review


A! Ładnie! :-) :+1:


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.418.ade39ff36fea59893e7274969d8a2cdbb05fbe09](http://build.palaso.org/viewLog.html?buildId=60650&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:21:56


Share? Z dwukropkiem do tego. Share the context: where it worked...


Beach party with beers sponsored


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2259


@andrepuschmann, thanks for the hint, I think this won't work -- on LFRX boards, 0.0 might actually be intended. However, I'll post something very soon!


Definitely needs wordsmithing.


@johnpaulbalagolan FYI - think this was why pages weren't refreshing, or at least why only 10 were refreshing. And we need better debug - doesn't seem to work in all functions/classes.


Are `-lf` switches intentionally missing?


So maybe we should add to the wiki that the former should be dropped (if there)


Not sure


Helllo


@173210 I can only see 4 addition/subtractions in my inner loop with no dependance with compiler optimisation vs. addition and multiplication in yours, in case compiler makes optimisations right. :)
If you didn't touch the result font format or rewrite all for common font format I didn't say a word against.


This doesn't work from me. The selected item moves from the left of the screen to the right as I swipe, but I would like the selected item to snap to the center of the view (when possible)


On Wednesday 05 of August 2015 13:49:32 Brad Allred wrote:
> I dont have a problem with this at all, but I thought you weren't supposed
> to be able to watch movies until you have already encountered them in the
> game. If we want to change that then we should remove the code (if it
> exists) that is updating that now. I may just be remembering incorrectly.
That's exactly what's going on — only the intro movies are set to 1 and
PlayMovie will set the appropriate variable once a movie is played. They were
just not getting saved basically.

LP
--
To err is humour
www.gemrb.org


Well, I think we should here.


@mardanlin Can you open an issue for this? :)


You wouldn't have to Kuba. :-)


Huggy Il a pas vu mon ancien commit ! :o
Tu as mis en forme [ça](https://github.com/hverlin/site_integration/commit/3ea671783f413ce4ab980ef3f59c1083a8edc279#diff-eacf331f0ffc35d4b482f1d15a887d3bR76).
Alors qu'il ne faut pas ! Je l'ai expliqué dans mon ancien commit [ici](https://github.com/hverlin/site_integration/commit/1ba267e6ef346f879f629206a71e1dcd2f2114ac) dans le 3eme point. Bravo ! :P


Yeah. Turns out i got the job! :)


TeamCity xBIM :: XbimAnalysis :: Build [Build 7](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2217&buildTypeId=XBIM_XbimAnalysis_Build) is now running


Double delegate is probably a bad sign here. The web view should definitely be the delegate of this class.

However, this class doesn't need to be a delegate of the overlay controller… they are pretty close friends. Delegates are for things that are more loosely coupled.  May be easier to explain what i mean on hangout.


Good spot :+1:


TeamCity xBIM :: XbimAnalysis :: Build [Build 7](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2217&buildTypeId=XBIM_XbimAnalysis_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:37


We already have copies of ahem.ttf in `/tests/ref/fonts/ahem` and `./tests/wpt/mozilla/tests/css/fonts/ahem`.  Maybe we can remove those now and let those tests use the same UA stylesheet (in a follow-up issue?).


Thanks @jdavis!


@jendrikseipp please understand that everybody agree that it's not "normal", but that the way it is until the 2.7 release, where the "old one file txt2tags" should succeed all the tests for its last release, and the new "modular txt2tags" should succeed all the tests for a smooth transition.


Delegate methods should always pass the sender as the first argument.

Also, they should be in the form of a notification, not an "directive". like `didTapEditButtonForSection`


Oh, I see, that makes sense :) My mistake.


The various work-in-progress stages are here: https://github.com/fredemmott/definitions-finder/tree/wip-scopeconsumer - but they're not isolated changes, so made sense to squash before merging.


oups! j'ai activé le réalignement des lignes sur les commits


This was being cast to an int which does not seem like desired behaviour.
The if statement below would select "brightly" for a light value of 1.5 for example.


it fixed the link


it removed all of the methods!!!


This isn't included anywhere.


have you managed to get this to build correctly for android?
if so, what command did you use?


fast-forwarding 2.0 to borshop-integration-1155-bugfix/incorrect-2i-pagination-finish-2.0 = fe293e80


test-s3-fixes (lefticus) - i386-Windows-7-VisualStudio-12: OK (1795 of 1795 tests passed)

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


awesome! I am so glad to see that this bug is finally fixed! :+1:


Why are we shadowing the normal inventory file w/ a local text copy?


\o/


This requirement is no longer necessary: the semantics of `Movable` are now strong enough to prove that implementation has the effect stated in the effects clause. (Also, `reference_wrapper` is not a great counter example because copy construction creates aliases and copy assignment rebinds the reference - that implementation actually works for `reference_wrapper`.)


Would it be worth factoring out the duplicate code from here and `handle_add_stylesheet`?  It could move into a `LayoutTask::on_stylesheet_added` method that is called from both `create_layout_task` and `handle_add_stylesheet`.


done


It isn't necessary. Removed.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.463.ade39ff36fea59893e7274969d8a2cdbb05fbe09](http://build.palaso.org/viewLog.html?buildId=60641&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:37:18


```
# NOTE: We also have a custom mapping for items in the stdlib not
#       included in https://docs.python.org/objects.inv
```


@jgraham notes that the harness changes should be submitted upstream.


`files` and `scripts` stuff is auto-ordered by npm when `npm i --save`


That is strange:

This is already in place: 'userdom_use_inherited_user_ttys(telepathy_domain)"
since both telepath_logger_t, as well as telepathy_missions_control_t are associated with "telepathy_domain"  via "telepathy_domain_template()", it should have already been covered.

what does this say: seinfo -xatelepathy_domain
and this: sesearch -A -s telepathy_domain -t user_tty_device_t


why not use "wrapped instanceof BaseAdapter"


is it necessary?


Where are we on this? Should we merge this patch?


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.419.ade39ff36fea59893e7274969d8a2cdbb05fbe09](http://build.palaso.org/viewLog.html?buildId=60654&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:20:30


fuck off


This could be merged with the policy from ReconnectionTest, will do that on branch where i'm doing some more test refactoring.


Sorry, forgot about this...

And ignoring that I get as antsy about new dependencies as you get about new command line switches (is File::Slurp in core? or pretty universally included in repos?)...

Overall it looks like the right kind of thing, but needs more, I think, to deal with the different kinds of data that can be expected. For your usecase, you're expecting xml, presumably UTF8, but other image formats will by binary.  I saw that File::Slurp has an option for binmode, but didn't notice what the default is, or whether it tries to be smart.  But most images should stay binary, and *probably* you're going to want SVG to be read in unicode mode so that you get a "string of text" rather than "string of bytes"; otherwise it's likely to run afoul of other streams along the way...  And it also makes me wonder that in a server mode, it's probably going to want to have a mime-type as well.


Можете конкретезировать вопрос?


I prefer the term padawan learner to neophyte.


Done.


Please follow the same coding style as i do, no spaces


[gm](cc|++) would work, even though AFAICT the MetaWare C++ compiler is called mcc.


Resolves 10


I was referring to the comment `// passed.`


On Wednesday 05 of August 2015 14:09:40 Brad Allred wrote:
> Oh, I see, that makes sense :) My mistake.
No problem, thanks for reviewing!


991757360-DocIssues-4956-4997 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1769 of 1772 tests passed)

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


Nice, richtig viel Code abgekürzt!


@jxnblk yop You need to do a big cleanup of all dependencies in package.json file that have a :
"git://" or "github:" repo adress. These are only accessible to you since they are ssh reliant.
Please replace by https address everywhere :)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.447.bff28eee27698923ad10a800862fd7628c077bee](http://build.palaso.org/viewLog.html?buildId=60646&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:12:25


This is a good start but not sufficient (as you know). So, we need to expand this. What you should do is check either if:

1. This type **is** `java.util.Collection` (which you have already started to do here, or
2. This type **implements** `java.util.Collection` (in the case that the type is a class), or
3. This type **extends** `java.util.Collection` (in the case that the type is an interface).

So, (1) you have just about done (I filled #21). For (2) and (3), I recommend using the following methods in the following order:

1. http://help.eclipse.org/mars/topic/org.eclipse.jdt.doc.isv/reference/api/org/eclipse/jdt/core/dom/IBinding.html#getJavaElement-- (which you can call on the `ITypeBinding` you have to get the corresponding `IJavaElement`, which should be an instance of `IType`),
1. http://help.eclipse.org/mars/topic/org.eclipse.jdt.doc.isv/reference/api/org/eclipse/jdt/core/IType.html#newSupertypeHierarchy-org.eclipse.core.runtime.IProgressMonitor- (to get the supertype hierarchy), and
1. Either:
  - http://help.eclipse.org/mars/topic/org.eclipse.jdt.doc.isv/reference/api/org/eclipse/jdt/core/ITypeHierarchy.html#getAllInterfaces-- or
  - http://help.eclipse.org/mars/topic/org.eclipse.jdt.doc.isv/reference/api/org/eclipse/jdt/core/ITypeHierarchy.html#getAllSuperInterfaces-org.eclipse.jdt.core.IType-

For this last one, I'm not quite sure. Try it out and see what it returns. You'll need some good test cases here. Three I can think of are:

1. Iterate over an instance of `java.util.Collection` (which you have).
2. Iterate over an instance of `java.util.List`.
2. Iterate over an instance of `java.util.ArrayList`.

All three of these should pass.


Xcode is probably showing tabs as 4 spaces instead of 8, so you can still fix if you want by backspacing and then reindenting.


@jxnblk same here :)


I did not, I'll do, but if we are doing good job in optimising ruby it should be same, shouldn't it?


why this removal?


Just to clarify: I was stupid and broke lots of stuff with this commit. In case anyone was wondering. I immediately fixed in the next commit.


IMO, You should just use a normal `if` instead the `ternary` operator here.


I think this change could be split into a [bug] commit, what do others think?


Is this comparing a single segment with an entire path value?


Maybe call it "shouldSeekBoundary"


you can either set LD_PRELOAD to an absolute path, or libname.so which ld.so will do its best to resolve. ld.so will then look in the arch-specific search path, depending on the type of executable being run.

This commit will probably break e.g. debian which installs libear.so into /usr/lib/<arch>/bear, which isn't in the default search path, and I'll rewrite the commit message to reflect this.

Of course you can get the same behavior with bear -l'libear.so', but this seems a more reasonable default instead of hardcoding the absolute path.

The upside is that with this commit you can install architecture-specific libear.so in e.g. /usr/lib/x86_64-linux-gnu and /usr/lib/i386-linux-gnu/, and a simple:

bear make

where make calls both 32 and 64-bit executables will "just work"


Is there a reason why we can't have a single URL be constructed from a concatenated path?


When the size is null, it means that we don't know what the size is, therefore we don't know if it is 0 or > 0. This could happen for example when using a body that is streamed from another source dynamically. We still want to send the body in this case.


Thanks.


Its is better to use this code, since it unwraps until it finds the SlideExpandableAdapter. There is no reason your code should now about HeaderViewListAdapter. This solution is more generic

while (! listAdapter instanceof SlideExpandleListAdapter && listAdapter instanceof WrappedListAdapter) {
  listAdapter = ((WrappedListAdapter)listAdapter).getWrappedAdapter()
}


Hm ... thanks for translation. Now how to internationalize?


No worries. Was a tab :smile:


missin 'l' :)


Why "lastPage"?


You can depend on `irmin-unix` directly if needed.


PlantOperationSchemes (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


You still do not get any summary if you do this (but with that in .gitconfig every pull was doing this)

git pull --rebase

It is the rebase process that scrubs the summary.  But with the merge=ff we get a normal pull, but that pull will fail with a warning message telling you to rebase if the pull is not able to be done.  But most pulls will get a nice summary.


Ah, that sounds cleaner, thanks!

On Wed, 5 Aug 2015 22:34 Thomas Gazagnaire <notifications@github.com> wrote:

> You can depend on irmin-unix directly if needed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/djs55/ocaml-xenstore-server/commit/758a6daa2a25a04b2dcf9df0b3023fcf10866936#commitcomment-12552128>
> .
>


Can we please localize it then? Usually I write all english, but don't know how to localize shell script.


many thanks

On Wed, Aug 05, 2015 at 12:01:24PM -0700, Ed Maste wrote:
>    Clause 3 cleaned up in r286327, you can re-copy this file
>
>    --
>    Reply to this email directly or [1]view it on GitHub.
>
>    Reverse link: [2]unknown
>
> References
>
>    Visible links
>    1. https://github.com/freebsd-riscv/freebsd/commit/2d3e92a8bf69edb1b0555413d12b40ae9b442bbb#commitcomment-12548935
>    2. https://github.com/freebsd-riscv/freebsd/commit/2d3e92a8bf69edb1b0555413d12b40ae9b442bbb#commitcomment-12548935


Ops, sorry forget that we only passed wascally during Lapin object creation. Yeah you're right it doesn't make any difference.


Vi por aqui que ele criou os gets e sets automaticamente. Hj não consigo
mexer pois estou em rc e não tenho PC aqui. Amanhã eu trato isso se não
houver urgência.

Em qua, 5 de ago de 2015 14:08, Fábio Franco Uechi <notifications@github.com>
escreveu:

> @gmoneda <https://github.com/gmoneda> , não faz sentido algum essa classe
> que vc criou dentro do Card.
> Favor removê-la.
>
> valeu!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smartcanvas/java-sdk/commit/045cdd8c33a34dea78aadbca060a46a70d101c6f#commitcomment-12546609>
> .
>
--
Guilherme Moneda • Product Developer CI&T •
Collaborate. Innovate. Transform. +55 19 98122.4542 • gmoneda@ciandt.com •
ciandt.com


we're cleaning up this node types dep, since it's an inner config to FLOW we don't need to explicitly depends on FLOW.


@stephenc maybe you can expose your kick provisioner hack into plugin (if it not CB strategic)? :)


Any reason preventDefault is the last one instead of first?


Actually, not using getAdapter solves the problem, you should use the member "adapter". I just commited and fixed that


CBC_Encryption handles the padding itself and (currently at least) requires the padding mode expand out to a full block size. And all defined padding modes are valid with empty inputs, and pad out to a full blocksize is response.

However your point is valid for XTS which is undefined if the input size is less than 1 full block due to ciphertext stealing.


If you **need** a rebase, there is no way to get a summary unless you rebase yourself first, something like this:
```
$ git fetch
$ git rebase origin/bleeding-jumbo
$ git pull
```


I like this :-)


No wait, that will probably not give you a summary either. I'm not sure it can be done.

Anyway, this new setting is much better than the old one. I'm quite happy with it.


Nice catch! I wonder if it would be better to put this fix in `removeTab` rather than `selectTab` since that's where things are actually getting out of sync?


why not simply declare opts as ```struct sol_flow_node_type_temperature_max31855_options *opts``` and avoid this ```const ...``` cast?


I think it's better keep the `dragNode.show()` at `line 113` and remove duplicated call from `resize_` function. Because always after any drag ends the handle must be showed.


Is snowbound a set of games are you referring to the Snowblind engine games?


Done: https://github.com/w3c/wptrunner/pull/137


Are exceptions the only recommended way to deal with erroneous program states? In other words, does it ever make sense to just output to cerr and exit()?


fixed Mar 30, 2015 via commit 66761d998b15288430ba5ebcd8936f1dbc8ce7c5 memcg: rename mem variable to memcg


If I haven't mentioned this before, this is the ideal place to use a `current_user` helper method.


:+1: :+1:


That's why I asked to ready the commit msg ;-). But the fix is right.


Actually, it should probably be `\Exception|\Throwable`.


This functionality doesn't exist anymore, the url is now a string in the http target


I take it you're doing this specifically to trigger the code above? That seems like another good reason to move the fix here -- then we don't need to call `selectTab` unnecessarily just to trigger its side effects. It also feels a bit dirty firing `didSelectedTabChange` in this case since the selected tab itself doesn't actually change.


The point is not to block on `advance()` but to mutex reads and writes to the advance deferred.  See the long comment above explaining this.

If I wanted to encapsulate the entirety of `advance()` in a queue, how would I prevent internal operations from themselves delegating work onto a different queue?


@Orvid - Can you please check this against MSVC and let me know if this helps?


Implemented in https://github.com/amphp/amp/commit/39d3212fb0421e89c7943d8accd6f7a178a6828f.


The issues are not related. Coverity shows all branch decisions that uses to create an issue and this returned wasn't present.


Does this mean that every RSA cipher is ECB mode and should trigger the EcbModeDetector?


Hey @IonicaBizau. I wanted to say a huge thank you for setting up the graph hack a few months ago. Folks have had consistent fun looking at it and I think we've had a bagillion people say they love it. Everyone admires the work you've done here.

I considered whether it makes sense to ask for an update to the graph hack, but I think (for now) it makes sense to retire it on my profile. I used to use the default graph for roughly tracking my work on GitHub and it'd be neat to go back to doing that for a while. What would that involve from this repo's side of things?

I DO think we should revisit the graph hack again at some point. Will have to try thinking of something new and fun we could do :)


Is it not possible to add classes to the markup and use those as css selectors? this does not feel like a future proof approach


Add schema.rb not seed.rb and you delete it in commit 23e1dc8fd53b8be93f532958e3d4cddd44ff8ca3.
Don't work on master anymore please!


Please document the plugin API :) Have a blog post on how to write an Aurelia plugin :)


I'd recommend putting this on multiple lines, just to make it easier to read. A newline in your file won't show up as a newline in the browser, which is why `<br>` exists.


Oops yes you were right for the date!


I didn't create or import any records in the 'states' table. Since the table is empty, `states.features` was null which was causing an unhandled exception in openlayers. This is the workaround.


@fabien I talked it over with @raymondfeng this morning. We agree that we should normalize the queries that don't explicitly state 'regexp' as an operator. ie) `{where: {name: /^a/i}}` should automatically be recognized and normalized into a RegExp object. Will get this update prioritized tomorrow during sprint planning.


Fixed in cf1ce7397f3ac850734569d8b9a540c2a09ef7bc

Thanks


You read my mind, I think!
I was going to file a feature request for this in the next week or so. :+1:


It isn't necessary to have ListView.


Revert as storage here is needed


If you look at what was line 592 it was the last instruction originally as well, so I just didn't change the position when refactoring it.


Yes, but only 9 minutes left in my time zone :smile:


I would this is is an unit, I fixed the functionality and the tests covering it. It might be that the tag on this should be [bug] though


Storage is need revert


Why are you specifying the registry URL for the `ansi-color` dependency instead of the version?  This will cause issues for developers using private registry mirrors without access to the official registry.


worth erroring if both these branches fail to find a `spark-shell`? Or do we error in that case somewhere upstream?


One dozen days later...


Why do these tests get removed? Shouldn't they be changed instead to be like the if-tag-not02 test in the django test suite?


Noted, thanks.


This is a bug in Appium. WebDriver expects the IDs for remote WebElements to be stable across multiple calls to findElement. Closing this bug.


See details in https://github.com/borisyankov/DefinitelyTyped/pull/3851


Oops, not a bug.. nevermind on the closing bit :)


Dammit, no comment due to difficulties checking that in:

Changed the framwork to 4.0 client profile, we already had problems with servers that aren't running 4.5.2.
Threw away the confusing app.config port definitions - now we're reading directly from the server_config.ini, admins just need to link the acServer directory correctly.


+1 for making this commit a hot-fix and bumping the PATCH number.


This addresses #32


Without ternary it's impossible to get 100% coverage in this case (a line with closing bracket to be precise). Test specifics.


@raelgc this has the potential of creating a ripple effect through the UI, please take a look and let me know if you think it was a mistake. It made everything on /promise line up properly finally (it was driving me crazy :)


I did not know there was a Django test suite, but there there is no if-tag-not 03, 04, 05, also they were wrong, "not" can't be a variable name (I tested in Django itself once I removed them)


I'm sorry, I don't think I understand the functionality that you want. These menu options just let you center the menu items and also size them so that the width of the menu item fits the text exactly. It sounds like what you want is different from what I implemented.


implement __reduce__/__setstate__ for Period pickle support


This might not work on Windows especially for mk_mvc.mak because the shell is cmd.exe not sh.


My mistake, sorry. I created a new test.


I ended renaming the the tests, if you prefer I let the 3, 4, 5 gap as Django has


this `:db` crap is so that my local machine time vs DB time don't get mixed up


I thought toggle-quotes was broken for a long time because of this keybind change :confounded:

Debugging sucked because it looked like the linux/win32 binds were somehow taking precedence over the darwin one. It took me way too long to noticed that the darwin one said `cmd`, not `ctrl`. My fault, I guess, but I wonder how many others were tripped up by this.


Okay no problem, never ending kernel optimizations ;-)


They are suppose to be used anywhere and everywhere we fetch or set the script source code for a script so we don't have to copy the AWS code anywhere else that we might need to get the script source code, and would instead call those functions. The function you linked requires an express request object which should not be needed to fetch the source code of a script.


Hi Brady. changes tested at : select patient- select encounter – Medical Problems – Edit – Click Enc to the right -> Click Enc -> Issues an Encounters pop up -> link issue with encounter


should be `=true`

or just `--disable-content-trust`


Updated.


Done.


-added first user created markdown file *HelloWorld.md*


I'm working on compatability with fastcraft.


Why was this change made? This always gives me errors like this:

```
[17:04:05] Failed to load external module babel-core/register
[17:04:05] Requiring external module babel/register
```

I didn't get these errors with `babel-core`.


Thanks @msurguy, just out of curiosity do you find this easy to extend for your own overrides?


I'd be especially enamored with the idea of separate definitions if we could make the customization points in `ranges_v1` function objects a la N4381.


Maybe the `--out-dir` should be a `build` folder or some other place?

If you decide to keep it this way, you need to add the generated `.js` files to `.gitignore`, but I'd rather you use the `build` dir, because these generates `.js` files really pollute the source code folder.


Thanks for clarification.

Could you add a zero-length plaintext test for all those modes? When I added those assertions, nothing was failing.


Also, there's not point in using the async package there since you're only going to be doing one thing in that function (fetch the source code from AWS) and the AWS library is probably already asynchronous (which is why there's a callback instead of a returned value).


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.464.ade39ff36fea59893e7274969d8a2cdbb05fbe09](http://build.palaso.org/viewLog.html?buildId=60649&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:35:04


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

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


@dominika widać, że się lepiej czujesz .. ;)


That wouldn't eliminate the need to constrain `std::swap` somehow.


Try to break this to keep lines at most 120 char long.


learn to code Kappa


test-s3-fixes (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1779 of 1795 tests passed)

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


My bad, I should have caught this also.


No it wouldn't, but separate definitions allow us to have both minimal impact on `std::swap` for backwards compatibility and a `ranges_v1::swap` with automagic ADL.


Update


@anytitleyoulike Fiz as alterações mano. Pode testar


What do you mean?


can this be tested?


How can this ever be less than 0?


because NPM is a piece of shit

https://github.com/npm/npm/issues/9168


Even without QList you can rely on Return Value Optimization if you return a local variable, nothing non-obvious here... Well, unless your compiler is from 1900s.
If that return value is used to construct a new object, copy won't happen either, the object will be constructed in place.
```c++
Obj x = f();
```
If that return value is used in assignment, move semantics of C++11 can kick in, and there will be no copy either. Move is more expensive than noop, but it's generally cheaper than copy.
```c++
Obj x;
x = f();
```
So, these days returning by value is usually(*) not more efficient, but is not less efficient either. But IMHO it's less cumbersome.

(*) It can be more efficient e.g. in this case, because constructor of Obj is called twice, and with returning it by value it can be called only once:
```c++
void f(Obj& arg) {
arg = Obj();
}
Obj x;
f(x);
```


I don't believe the Logo should be on every JSTransformer. Just the project itself.


Hi Luke, I recently discovered your yahoo fantasy node apps and they are great. I had a question and didn't know any other way to contact you. For this app in particular, when creating the application on YDN, what did you set the callback domain as? I'm wanting to run something like this locally, but I don't think you can use localhost as the callback domain.


It might be clearer to signal "not found" by either 1) returning an Optional 2) returning something like -1.


LGTM


:+1: Very nice

Could you create a PR considering this slightly revised version ?

```
+  QAbstractButton *acceptButton = d->button(QMessageBox::AcceptRole);
+  if (acceptButton && !acceptButton->text().isEmpty())
+    {
+    QString dontShowAgainText =
+      this->tr("Don't show this message again and always %1").arg(acceptButton->text());
+    d->DontShowAgainCheckBox->setText(dontShowAgainText);
+    }
```

For the info about ``tr()`` and dynamic text. See `Use QString::arg() for Dynamic Text` on http://doc.qt.io/qt-4.8/internationalization.html


Yes I am late :(


What's going on here?


Ah, sorry about that @dmnd! :bow: It's hard to make non-breaking changes to keybindings, if only because the options for usable keybindings are so limited - I could've (and maybe should've) left the old keybinding for awhile, but then this package would be taking up an extra keybinding that another package could potentially be using instead.

Hopefully https://github.com/atom/atom/issues/8240 will help make this kind of change less jarring and surprising for users.


:warning: :)


Thank you. I suspected, that something is fishy :)


name is a string


Why is this a special case? Couldn't it be handled in the same way we handle the "else" case below?


:+1: Minimum should never be set.


pure identity function as a method on a data type?


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.16](http://pixelated.design/build/viewLog.html?buildId=85&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 292 Build time: 00:00:10


I think it looks better. And would look "more official" for users. It clear that it is official transformer if he is in the org, but yea. Why not?

Most of the PostCSS plugins have it and that trend looks great.


pure function as a method on a prototype that doesn't do anything with the instance state. how is this used?


5058-fix-readvars-cli (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1790 of 1795 tests passed)

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


Also we can add notice to the JSTransformer that says "if you want you can add our logo to your transformer".


Ohhh, I thought you were assigning to `name`, not `{ name }`. I take back my :stuck_out_tongue:.


Neat!  I didn't know that spelling.


keep the update coming (y)


it might be more costly, but I think it makes things more explicit and easier to deal with. Without it, we'd 100% have to rely on the receiver implementing the default value correctly. i.e. not supporting increases the chance of failures.


need a ajax call here.


Yup. Found it on http://sphinx-doc.org/markup/toctree.html after doing a scan and giving up on the page (then changing my mind and reading more closely).


Maybe a better name than `what`? Perhaps something like `target`, `payload`, `body` or `data`


5058-fix-readvars-cli (Myoldmopar) - 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)


Data Science Coursera


Can you make the variable a little more specific to KFS? EXTERNAL_CONFIG_DIR is a pretty common name to be used. Maybe UA_KFS_EXTERNAL_CONFIG_DIR


This isn't yet a win. The 0.9f in the multiplication promotes the whole operation to a float then converts back. To avoid the implicit conversion you can instead do:
((g.min_gndspeed_cm*9)/10)

If that is to irksome then the implicit conversion can be left in. (IE we aren't yet to the point of triyng to save on every floating point op.) I suspect that an extra multiplication is faster then the conversion in and out of float though.


No worries @mnquintana. atom/atom#8240 is a great idea btw :+1:

For keybindings specifically, another idea could be to introduce a "deprecated" keybind. When a deprecated bind is triggered, it could pop a notification that explains what the new keybind is. The notification should of course have a checkbox that says "don't show this again". There's no reason a deprecated keybind needs to "use up" a binding, either — the notification could pop but not block any new use of the keybind.

Perhaps that's too elaborate a scheme, though, since hopefully keybinds aren't moved around that much.


why a loop you need to break from and why a for loop instead of a while(true)` loop?


I really do not understand the issue here. What does this section mean? https://docs.npmjs.com/files/package.json#github-urls


Please, consider also extending the associated test.
See https://github.com/commontk/CTK/blob/master/Libs/Widgets/Testing/Cpp/ctkMessageBoxDontShowAgainTest.cpp

Thanks
Jc


fixes #524


Don't forget to bump `ceph-deploy.spec` and `debian/changelog` too :) and we should probably fast-forward merge this PR, if you want to do the bump in this PR.


@danilowzn Só trocar o `/ops` pela url que tu quer quando o IE for muito antigo.


Ah, thank you


Done.


broken indentation


Maybe this will help clarify:
Current:
![screen shot 2015-08-05 at 6 31 43 pm](https://cloud.githubusercontent.com/assets/11168127/9099642/6755bb9c-3ba0-11e5-9cc4-f34146f1bfd1.png)

Intention:
![screen shot 2015-08-05 at 6 32 00 pm](https://cloud.githubusercontent.com/assets/11168127/9099649/73517f12-3ba0-11e5-8510-c5d210e16d13.png)


Yeah, I just forget ~50% of the time. Thanks for the reminder!

On Tue, Aug 4, 2015 at 4:24 PM, Hadley Wickham <notifications@github.com>
wrote:

> Do you know you can put closes #4 in the commit message?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/COSTDataExpo2013/AmeliaMN/commit/908e13000b1285a8eca54817c2726b732bbe63e4#commitcomment-12529337>
> .
>

--
Amelia McNamara
University of California, Los Angeles Statistics Department. Ph.D. expected
June 2015.
Twitter: @AmeliaMN <http://twitter.com/AmeliaMN> Website:
http://www.stat.ucla.edu/~amelia.mcnamara/


Python classes should be in camel case


Ok. Nevertheless let's keep minimal discrepancy from POSIX, so maybe just use the wildcards


There is an ansible playbook that does those parts for me (updates `spec`, `debian/changelog`) and tags (with a FF merge).  See https://github.com/ceph/ceph-deploy/commit/384ea40bcd74c4bb0c0401325da4f42e8d89540e as an example.


rename to `withInput`?


Reverting is not going to help anyone. The previous way has been extremely contentious in that it doesn't future proof the primitives in WebDriver. We have seen numerous people disagree with the previous approach and what is currently in the specification allows the previous approach to be emulated on the client side and allows us to move the specification forward.

As was pointed on in [Bug 26147](https://www.w3.org/Bugs/Public/show_bug.cgi?id=26147#c3) we need to make sure that specification can still be useful in 2 years time with new and interesting specifications coming up.


I did this three step, I also comment in the code like Step 1, can you take a look above code and lets me if its proper approach, and the following


Oops, this is actually for #20!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.464](http://teamcity.monogame.net/viewLog.html?buildId=9439&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


OK. I've put off respecifying the customization points as N4381 suggests because I don't really know how it should be specified. Shall we define "customization point" in the library introduction, describe the properties, and leave implementers to figure it out? Or specify in detail the implementation (the games with class statics, the free functions and function object type in a hidden namespace, etc. etc.). It's thorny.


> They are suppose to be used anywhere and everywhere we fetch or set the script source code for a script so we don't have to copy the AWS code anywhere else that we might need to get the script source code, and would instead call those functions.

I understand that... in order to possibly include this into the existing source the "'ish" part I added in would need to be modified before any of these could go in... in place migration routines is where I could use possibly use at least one of these as I wouldn't need to wait on sizzle to do certain things with the database and it could keep the source a bit cleaner. I presume the GH import routine could also use this without the request object as well. On a miscellaneous note the only place `exports.getSource()` is used right now is in `./controllers/user.js` ... so are you preferring/proposing to use these routines *as a replacement of the existing calls* without the request object?

> since you're only going to be doing one thing in that function

I'm thinking ahead which is why *async* was mentioned. I understand that it isn't currently needed with the one thing but if we start chaining with other asynchronous routines there it would be useful to have stock.

Do you see what I am looking for here?


'%s' % command
is equals to
command


Is the idea to poll for metric names because the metric names can change (increase in quantity)?
Or should the list of metrics be cached once requested, and only the callback of this request should be scheduled as a `poll`?


Thanks, this works really nice with `nested_object()` and `nested_array()`, but it doesn't cope with `default_value_writer` specialisations. I need to concoct some twisted template machinery for that, because unfortunately I didn't plan for pretty-printing while thinking `default_value_writer<T>::operator()` - and I regret this. Now I can't break the interface, so probably I'll try and detect a three-argument overload of `default_value_writer<T>::operator()` and call it rather than the two-argument one, if available. I'll keep you posted.


Yes, I will make the change and do another pull request.


Hello YouTube!


So when you reference "git:" repos its only for ssh based access repo. You can push to them because you have a ssh connection with github and you are the owner of the repos.
If you want other to be able to clone them properly use the https version.


`selector` is not defined, replace with `'.subject'`.
`'.subject'` applies to event, email, and twitter interactions, but we may need to move things around if we ever run into an incompatible interaction type.


I am too tired, it will happen tomorrow


Shouldn't None be used here as "no code yet" mark?


This handles rollback in case `container(transactor)` throws an error.
ex:
```
function container(transaction) {
  throw new Error()
}
```
If this is the case, then the transaction will never be rolled back and all future actions on this connection will be execute as part of the transaction that may never be commited.
Look at this [comment](https://github.com/tgriesser/knex/pull/928#issue-98852258).  It explains the reproducible scenario.


This is sketchy


I suggest to put
stdout=subprocess.PIPE, stderr=subprocess.PIPE, close_fds=True
into __init__ params with this default values


Ah - that's annoying. Looks like that issue is now resolved, however. Can this work around be removed?


hey @danlin we can merge this back into master if you want :)


Don't need to convert to 'str' explicitly, %s is equals str(x)
Use self.__class__.__name__ instead of 'checked_Popen'


Braces unnecessary here
x,y = some_func()
is ok in python


Good point

......................................
Uri Laserson
+1 917 742 8019
uri.laserson@gmail.com

On Wed, Aug 5, 2015 at 2:54 PM, Ryan Williams <notifications@github.com>
wrote:

> worth erroring if both these branches fail to find a spark-shell? Or do
> we error in that case somewhere upstream?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laserson/adam/commit/aa5535474adaca849f586948b63d435b310b58a1#commitcomment-12552556>
> .
>


i hope this is good js code :)


&end?


Don't need 'loglevel', this will be handled by logging gracefully


yes, the code in tests/pollfd.jl already works with WindowsRawSocket objects, it is just calling `poll_fd` instead of this method to implement the call to wait


This suppose to be a global setting?


Maybe it's better to replace that with an A* algorithm (https://en.wikipedia.org/wiki/A*_search_algorithm), because that doesn't include infinite loops or recursion.


hi harrison


ESPはlinux環境下で手動で作る方法を紹介された方がいいかと思います。あと、EFIでしたらgrubを使うよりはKernelのEFISTUBを使った方がいいと思います。EFI->grub->{Linux, Windows}とする意味はあまりなく、適当にrEFIndかGummibootかなんかで各OSのEFI用ブートローダを呼び出した方が速いと思います。


if i already have runing iodine on android and i'm connected to my tunnel, what is the next step?


Didn't found any issues in logic


Looks good!


squash back?


Swift..这么屌。。


I think this is fine, I can't think of a reason why export would care about the hobject sort order.  @dzafman : if import gets objects out of order, would it cause trouble?


> replacement of the existing calls without the request object?

Yes, if it's the same code reused everywhere. That way all AWS interaction is in one place and if we even changed the `Script => SourceCode` interaction (eg: changing the key from `Script.installName` to `Script.id`) we would only need to change a couple of functions.


asdfasd


It looks like maybe this `--out-dir` is a mistake? Why not just run `gulp build:lib` for prepublish instead of having two ways to build?


Confirmed this works! Thank you!


Also, these were probably meant to be virtual functions in the Script model.


I think it's a close second to the Oxford comma addition and collision detection.


It does not return the deleted value.


Chance "canols" with "cannola


This way, you're absolutely sure the `package.json` is built too.


shure that it was removed with 2.x?

http://www.wowhead.com/npc=5981/portal-seeker#comments
By MissNoel (123 – 0) on 2009/03/24 (Patch 3.0.9)
He still is around, I found him and killed him on accident. Ashame I did not have the quest at the time :(

would be nice if we could see him in 3.x too :D


Do you know why we have to call this explicitly? Curious what scenario necessitated this change.


r+


5058-fix-readvars-cli (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (747 of 747 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-7.18%25-green.svg)


1: M101 signature is missing
1: M110 missing component name
1: M190 line is too long (104 > 50)


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/ae17aa1cd276db5662c7db8212961d8613218183


merging graydon/stellar-core/windows-process-fix = ae17aa1c into auto


graydon/stellar-core/windows-process-fix = ae17aa1c merged ok, testing candidate = dee311f6


This change breaks the rel URLs: https://webfinger.net/rel/avatar


Remove debugging


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.464](http://teamcity.monogame.net/viewLog.html?buildId=9439&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:24:11


Is something comparable not necessary on creation of user?


Great idea! This will help to keep everything clean when more suggestions will be made.


the reason it for not using a hex or a resource, is that value linked to the theme which is will be variable once i insert the theme selector, if you check style.xml, you'd see i've already predefined few themes, and if you look 2 lines up from altering code, youd see how is the color retrived to begin with.


Hey Sri,

I think these changes are fine, perhaps we can extrapolate what is in the parentheses as the long hint?

In terms of defining iterations, how about - "The number of iterations determines how many times the k-means algorithm is run on the dataset until convergence is achieved." Does that make sense?


hmm, I had this like this for a reason.  I think it was because one of the commands (I think it was `list`) didn't need permission to work, and the bash completion script would fail if `checkPermission` was run.

I'll do some tests to verify..


Hi Alyssa
What is this


OH nvm


Accepted


98812358-IDF-SwimmingPoolSurface-4844 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1789 of 1789 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)


it would be good to note what the default poll interval is and what it was changed to.


Also the message could just be "polling interval adjusted"


En del stavfel och lite gramatik problem här annars så tror jag att det är ganska bra.
Sen i dem sista två meningarna så skulle jag ändra uttrycket lite för att få det att låta lite mer som att undersökningens resultat stödjer den thesen, men det kan vi gå över senare när vi grovläser den.


> these were probably meant to be virtual functions in the Script model.

Have a possibility of a future problem with that and `.toObject`... there currently isn't a way that I could find to include "methods" like I did with virtual properties in [`b755ddb`R423](https://github.com/Martii/OpenUserJS.org/commit/c928907e9c88b9562970d5d79c8f4547b8533c3f#diff-b755ddbc6b2edf964c5586d742e3afdaR423) ... plus I think we aren't always passing the *mongoose* object always... e.g. I think that's why `scriptData` indicates that there isn't any methods attached even though I just recently attached the virtual properties for `_since`. Otherwise a lot of code could go into the models.

> Yes, if it's the same code reused everywhere.

Okay that's what I was thinking... if one of us puts something together to actually use those routines I'd be willing to look at it and hopefully the reverse... should have some time in about 2 weeks to readdress this as I still am a proponent of doing what you initially proposed but with the premise of putting it in place.

Thanks.


From looking at TRS3, I'd say that "overlaps" should instead be called "edges", as they connect boxes (aka "vertices"). Zones seem to be some sort of clustering of the graphs built by overlaps and boxes, to reduce CPU consumption, as the amount of vertices and edges is reduced. This is like finding a driving route: If you know you stay in the same city, you don't need to consider other cities for route calculation.

Boxes itself are basically attributes of that vertex, and they seem to be used together with NPC attributes; I think different NPCs can go up/down different height differences.


You don't need to comment what the flag does here, the documentation in doc/ is enough.

You *may* document here *why* the item has the flag or why it has a specific value, *if* that needs to be commented (e.g. sometimes the price of items is commented to explain why the specific value has been chosen).


Isn't `!(CGSizeEqualToSize(forcedMaximumSize,CGSizeZero)` superfluous with the subsequent return?
If `forcedMaximumSize == CGSizeZero` this will return `CGSizeZero` either way.


Will this be removed later?


Deleting the Mac specific files may make merging--to or from this branch--difficult later.


Unsure of demonstrating with `console.log()`. Distracting from what is actually important.


Ser bra ut, lite jätte små saker som kan touchas upp men än en gång kan vi ta det när vi grovläser det.


LGTM


Isn't it better to fail here when we know what happened, rather than have mysterious build failures later on?


@PabstMirror

I tested isTouchingGround with air units on roof tops with much success.

However, It is so strange with ground vehicles.
What I should have said, isTouchingGround is unreliable in dedicated MP situations. For example check this out: http://gyazo.com/d610d1cfdbf88d3cb17538d779549b01


It was auto-correct on my mobile. Thought it is clear what I meant.


Bra tillägg för att förklara mer om hur git skiljer sig från andra system, tror conny ville ha ännumer till och med, typ skillnader mellan git och SVG samt git och HG.


@andrejro Please submit this to coreos/portage-stable.


left-over debugging?


Prefer INVALID_OPERATION.


You no longer need to change `computeProperties()`/`refreshRendering()` in make them handle all widget properties at initialization. In other words, code like [this line](https://github.com/wkeese/delite/commit/14b1d57a22c976f65f600cab4b73c413c79fed47#diff-7b912bb0da9df7e075867a28509d2a50R121) will no longer be necessary.

> In that case, is `refreshRendering()` called again with the widget instance as the argument?

Yes.


98318570_RemoveRadianceCheckbox (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


For new spec citations, please use the newest version of the spec. In this case, ES 3.0.4.


98318570_RemoveRadianceCheckbox (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2176 of 2215 tests passed)

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


debugging code


this is `fullname(LIghtGraphs)[1]`. -1 to this variable name!


When reading, just use this.orientation.  Will be faster.


This type and default is wrong


And here.


It's faster but doesn't work with computed properties.


@andrejro I asked mjg59 to look into fixing sbsign for arm64.  I have a work-around in my repo, so this should not be needed.


@SilentSpike
I have not tested a ground vehicle on a dock yet.
Good thoughts.


Hiya!

The way I've handled it is by simply creating an entry in my hosts file.

So in /etc/hosts I have:

127.0.0.1   www.fantasy.dev

and I just use www.fantasy.dev on Yahoo!

Hope this helps!

--d


Hey @jonmak08,

Thanks for your input, I have rewritten the message and have separated the invalid words from the invalid characters. Can you help me take a look to see if this format is okay?

**Sample Message**
```
The folder name cannot be blank or a reserved word such as null, con, prn, aux, nul, com1, com2, com3, com4, com5, com6, com7, com8, com9, lpt1, lpt2, lpt3, lpt4, lpt5,
lpt6, lpt7, lpt8, lpt9.
The folder name cannot end with the following characters: .
The folder name cannot contain the following invalid characters: \ // : * ? " < > | ../ /..
```

Reference to previous conversation: 405be8264bbdbfb314e61371e78042abaa0d2726

Thanks!


it's killing me. what is `.atw`??! (and man I gotta stop getting Github notifications pushed to my computer)


Claro, la idea es que las funciones de ciaaLibs_Matrix llamen a funciones de la capa drirver (/modules/platform...) y estas ultimas llamen a las funciones correspondientes de la DSPLib (para el caso de lpc43.., correspondiente a la EDU-CIAA-NXP que poseo).

La DSPLib la pretendo alojar en /externals/ similar a "lwip". Es correcta la idea?.

Si me estoy yendo por las ramas lo continuamos por la lista.


Bra, helt rätt tänkt där!


*hides in the new WIP caves*


s/Specifically, i/I/


doh!  reverting.


No "specifically" here either.


It's not. At its most simple, it's string(fullname(LightGraphs))[1], which is 1) awkward and 2) dependent on the import at the top of the file. There needs to be a better way of doing this, I agree, especially since `current_module()` doesn't apply here.


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

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


This is the only **protected** displayAjax method of all existing *displayAjax* definition in frontController. in AdminController **protected** is used…


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/74326783


Prefer INVALID_OPERATION. We only add complexity when we do things like this. Just disallow the operation and spew to the console about what's wrong.


why not just make this file `404.html` and use an `<h2>` tag below instead of the markdown syntax?


@173210 You should consider adding "-transpose" here and making fontY loop inner in drawCharacter, font addr row/column will also change. In that case frame buffer access won't be scattered and will be accessed with linear increasing address stripes. This won't affect readability, but may increase performance, especially with direct writes to active framebuffer, also inner loop can be even rolled-out to make integer writes.
Also filling backgound for half-width characters is a dirty trick


fixed. thanks


for creation everything about the user is brand new and not modifying an account in place. So, it's pretty obvious what is being done to the account. for an update it wasn't listing out what was being updated for an account, so the audit log listed accounts that were updated, who updated them (in the case of the admin UI), but not what changed on those accounts. So, it made it hard to know when looking at the log what an admin actually did. That coupled with the fact that we are adding an admin password reset box for accounts (in order to give our help desk tools to address the unable to reset password for corporate domains and forgot SQ/SA issue). It is important to distinguish when admin A is reseting a user's password vs just fixing a email address typo.


Setting aside for the moment binding method as an implementation detail...

So, I think the solution is actually have to do what you said at first: having a plugin that talks through JNI/other binding method back up to Cardboard (or conversely have a JNI binding for PluginKit with a small stub to load plugins written in Java, but I don't know how this would interact with dynamic loading of plugins). This is based on the architecture of OSVR, where plugins have a very small, simple (and fairly stable) API (PluginKit) that they build against, and they remain ignorant of the rest of the system, while the server is responsible for loading these plugins (typically dynamically), providing them with the opaque context objects they interact with, and running a main server loop that calls the various callbacks registered by plugins as needed. (There's a diagram early in the "Path Tree" slides on http://osvr.github.io that might give an idea of this architecture better than this wall of text)

The cardboard sensor data plugin is something that's pretty interesting to me and others in terms of being able to provide an easy-start Android experience that doesn't require root or an external tracker, and at the same time you have expertise in this area that I don't have. Is there anything you need from me or one of my connections that would help you out here? (some documentation, voip or IM Q&A/architecture discussion, something else?)

If you haven't already, consider joining the OSVR dev mailing list - http://osvr.github.io/mailing-lists/ - it's probably the best place to continue the technical discussion (there or on a github issue), because I'll eventually forget which commit has all the GitHub comments on it and lose the email that I followed here :D (and there are others on the mailing list who might be able to help, and it's archived for future reflection, etc - the typical reasons why open source projects have mailing lists, you know.)


Yes. See [here](https://github.com/SummerWish/privly-applications/blob/embed-posting/shared/javascripts/options.js#L146) and [here](https://github.com/SummerWish/privly-applications/blob/embed-posting/shared/javascripts/glyph.js#L71).


I wanted to be consistent. The 404 is only a placeholder, let's discuss what to put on that page ;)


@matbalez this solves it; will flag on any future instances I see `gets` not stubbed out


Hmm, It makes sense.


(fyi @loganhasson!)


Ici on essai juste de vérifier si la valeur d'un paramètre de route a été définie lors de l'appel dans la fonction url, sinon on utilise sa valeur par défaut a la place si elle a été définit grâce a defaults();


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 267](https://teamcity.labkey.org/viewLog.html?buildId=264808&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:55
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


Fixed in ea14805
<https://github.com/Day8/re-com/commit/ea14805ad545934b5056185152769dba3f6b411f>

;-)

On 6 August 2015 at 03:21, Mike Thompson <notifications@github.com> wrote:

> perhaps instead:
>
> however we are informed by @frozenlock <https://github.com/frozenlock>
> that it is still not as fast as Chrome if you are using *deeply* nested
> flexbox layouts (much more nested than our demo app).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Day8/re-com/commit/aa7d1e2109f9cdcbbd941bdf28308d9a70e258a8#commitcomment-12546816>
> .
>

--
Gregg Ramsey
Day 8 Technology <http://www.day8.com.au>
0450 001 082
gregg.ramsey@day8.com.au


@Ninikitos DWMR. Good work. Press on.


@edysegura Após uma pesquisa, seguimos o tutorial:

http://www.angularcode.com/demo-of-a-simple-crud-restful-php-service-used-with-angularjs-and-mysql/

onde traz um exemplo de aplicação funcionando, o qual nos fez ganhar tempo, após termos gasto mais horas que o esperado no desenvolvimento do carrinho de compras.


@Ninikitos DWMR. Looks good.


You are welcome to comment on the upstream source, but there is no need to behave like a stalker by going through a fork to leave your illegible comments.


Apache commons lib has a utility method for this already.


Side note: and its better :)


Looks good


:+1: :+1:


Oops.


991757360-DocIssues-4956-4997 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1775 of 1778 tests passed)

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


new line after functions!!!


991757360-DocIssues-4956-4997 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1775 of 1778 tests passed)

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


Daca ai importat service_instance, de ce mai importi inca odata o clasa din service_instance?


new line after functions


Wondering if this is a valid use case for the lambda expression... or abuse.  I looked at using a different interface so that ```this.responseObject``` in line 46 could be replaced with dynamic data.  Certainly doable, and something like this could be packed into a utility class for null and generic exception catching.

Another thought, I suppose just catching and swallowing the exception is probably not always the best idea. So maybe not best intended for a utility class, but an inner utility method.  Hoping someone has thoughts here.


new line at the end needed


@memolipd note the `vm.$location.url()`. it's in a bunch of directives now.


You may want to add a `<div class="row">` around here


Fixed in latest Readme.


I don't like this. We should call Receive recursively instead - in case there are messages that are not encoded etc that are received.


the initramfs does it like this since it uses dash (to keep the initramfs small).
in bash, which we should use here, there are much more compact ways of extracting the last character of a string.


why don't u change this typo too? xDD `Rouch`


Done (pushed https://github.com/grpc/grpc/commit/0607bae87e0f3417972bc221a5c4f8dc6bec5fef to this pull request).


It should just have stayed readonly! Fixed here: https://github.com/grpc/grpc/commit/56c6574652ce261370582fec7decd4a8e6ce1cb3


At this point udev should have seen that sfdisk closed the block device which was opened for writing, so it will be reprobing. Add `udevadm settle` here to avoid races.


At this point udev should have seen that e2fsck closed the block device which was opened for writing, so it will be reprobing. Add `udevadm settle` here to avoid races.


http://i.imgur.com/mfp7ts8.gif

Single node-only like the default in-memory implementation but slower. Can save memory, but is it really necessary?


Do you think we can get rid of the `gh-pages` branch and just commit to `master` since there's a `CNAME` file and we have a custom domain?


looks good to me


Converter for BuildInformationBean to Build would be good here.


I goofed on the logic here. It should be:

```
if (bson_empty0(options) ||
    (
        !bson_iter_init_find_case(...) &&
        !bson_iter_init_find_case(...) &&
        !bson_iter_init_find_case(...)
    )) {
	return true;
}
```


1: M101 signature is missing
1: M110 missing component name
1: M190 line is too long (104 > 50)


I didn't realize these triggered exceptions. I suppose just warning and above will get converted, and the rest will end up being sent to whatever the log handler is (like debugging and info messages)?


Using `minlenght` and `maxlenght` for validating input data is a good starting point ,but as you may anticipate there could be some special cases where the validation could not work, as in the case of short names (Jo Lee).

Usually there is an extra validation on the backend side before saving this data, this to verify if its "valid" in the current context (paranoia !).

Input validation is fun :gem:


Resists restarts, can be spread using duplication in a farm, and cached versions can even be deployed. I see it as a sort of Warmup, but explicitly designed for output caching. Hint: I do it in Decent, to cache the JsDoc-generated Markdown documentation pages, which are quite expensive to render, and it's awesome.


Excellent.


fixed


how hard would it be to inject this functionality into util/exec?


zoomlevel is not working. The keys are captured, but the screen zoom levels is not responding to any of the three commands `ctrl +,ctrl -,ctrl 0`.


Can you Haddock the public function?


Sorry, that was a mistake to commit. Can somebody tell me if that's ok, or should it be reverted?


oh, never mind. I see that comes later.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.465](http://teamcity.monogame.net/viewLog.html?buildId=9443&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


done, but you forgot to finish up the feature ಠ_ಠ (fixed in https://github.com/LaserQueue/LaserQueue/commit/0c8155e68ed91cc7c0f66a02d9d2e44826ae441f)


Is this surprising?  Should it have been the empty string?


I think fairly trivial. It appears to just wrap `os/exec` to limit what can be used, so that mocking is easier to implement.

I think the real problem would be that everywhere else that we use `os/exec` and mock it we would have to implement the updated interface... Not sure how trivial that would be yet.


Det ser mycket bättre ut! Och det tar inte så förbannat mycket plats ;)


Maybe some kind of better error here, like `maybe (error "Eureka application instance does not provide a version") lookupVersion i` or something.


@cherrydev - did this get merged back in? Found via your comment http://stackoverflow.com/questions/25515129/angularjs-show-errors-directive-with-dynamic-input

I'm having some dynamic problems myself


Jag fattar ingenting xD


so in this case, `readP_to_S parse v :: [(Version, String)]`. What does it mean for the `String` to be non-empty? Does that mean there are leftover characters, and therefore this is not really a valid parse?


5058-fix-readvars-cli (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1272 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-69.97%25-green.svg)


Also, I think `Distribution.Text.simpleParse` might do it?


no more notes?


@miigotu we should remove skipped too. only WANTED should be downloaded, isn't? if user has default episode status to skipped he does not want to download it


Fixed some mistakes and rerun the CI.


If the guide takes them through this, you can also link to it.


Good! 개발 막바지에 우리가 이 프로젝트를 하면서 어떤 오픈소스들을 썼나 리스트를 한번 정리하는게 필요하긴 하겠군요.

물론 build.gradle만 보면 얼추 필요한것들은 다 나오겠지만요~


Last one should be `production:stop`, I think?


For additional parallelization, please take a look at this package I've started for running StARS using a cluster queuing system (via BatchJobs) https://github.com/zdk123/BatchStARS

On a torque cluster, I've gotten significant speedups on large projects (>1000 OTUs) compared to mclapply. The Map/Reduce logic also significantly reduces memory requirements since you don't need to hold all those graphical model paths in memory.

I'm attempting to make the package general (for all sorts of lasso problems) but it would be great if you want to test and/or contribute.

The other function is a binary search along the lambda path, since graphical models run slower for lower value of lambda so we'd like to reduce search along the whole path (by reintroducing some serialization). I've wasted a lot of time doing this in an ad-hoc way.


A-ha! but it is not so much of an error but rather it indicates that you missed something.
Because you throw an exception from the `getSpeciesFactory` method, you are supposed to 'catch' it somewhere. Either you throw 'up', towards another entity that will call your `main` method and then it is its responsibility to in turn catch it or throw it, or, you could catch it yourself here. So you have two choices as suggested by the Eclipse IDE when you press `Ctrl` + `1` over this line. Play around with these two and see what happens.


Everything else looks good, however, you should add more constructors to your non-abstract animal classes


Why are you using ; at the end of each line?
Try to not use it and will see that code shoudl also works without it,


Tests pass at https://travis-ci.org/jayvdb/pymemento/builds/74169278

No doubt there are some other issues not yet found by these tests.
We'd need to check code coverage of the tests to confirm all code is Python 3 compatible.


if you don't define the specific format, for example age.to_i if I call your function using :
person_age "Jason","22"
will it still working?


If I use
person_age "Alex","20"
your method will still working?


이 부분은 참고한 예제에서 네트워크 통신(httprequest)를 보내기위해 사용한 쓰레드(AsyncTask)인가요?

물론 현재 짜는 코드에서는 네트워킹에 관련한 준비가 안되어있기 때문에 아래와 같은 예제로 4초동안 불러오는 로딩 시간을 주다가 예시로 몇개의 데이터를 더하는것은 참 코드를 잘짠것 같아요.

다만 GetDataTask()라는 이름보다는 GetFriendsListTask()라는 형식으로 했으면 어떨까하는 제안을 해봅니다~

아마 실제로 이 부분에 네트워킹 코드가 들어가게 된다고하면 volleyer()를 이용한 httprequest 코드가 들어가게 될것이고..

그때는 알아서 Volley가 쓰레드를 만들어주니 GetDataTask 클래스가 없어지는 쪽으로 정리 될것같아요~

일단 매우 수고 많으셨습니다~ 만족스러워요!


@technobly, can the RGB still be connected while using serial2 but just not being used? Does it matter if the G and B leds are connected? Or do the resistors for the g and b need to be removed?

Thanks,


you can also try to call
person_age "Walter","32"
Like you are handling the correct format in your code, it should works even if you are recieving a string.


코딩 컨벤션에 맞춘다면 m_List보다는 mFriendsList가 어떨까 제안해봅니다~


I'm no expert on CSRF so I can't thoroughly audit the implementation, but it looks very reasonable.

The API also looks very well thought through and extremely well documented, the examples are good too. The only thing I can think of that might be handy to include is a full form template in doc.go that would submit to `/signup/post` just so it's really clear how the whole thing should be implemented.

Other than that, it LGTM :)


nice guys


you could also try sending value for the second argument and will see that our value inserted is recognized over the default vakue


리스트 내부의 버튼과 리스트 그자체를 누르는것을 실험해보기 위한 코드인가요?


Nice Job!!


xml에서 id값은 가장 최상단에 놓여지는걸 추천드려요~


수고 많으셨습니다~

제가 코멘트를 남긴 몇몇 군데에 대해서 답변을 듣고 merge하는걸로 하면 좋겠습니다~


if I call to the method using
person_age 'ana', '20'
Your method will still working?


you don't need to resubmit, but Python does floor division with integers.
So `30/60` = 0 in Python land.
In order to get around that we need to change at least one integer to a float or do a fancy thing that I'll mention in class tomorrow (`from __future__ import division`) (this works b/c in Python 3 division for integers is as you'd expect)
The simplest thing for now is: `30.0/60`
Try the difference in your interpreter


There's some debug still in this @colemanw - see error_log() at line 301 at least.


thanks for apply teh correcttions!


did you try printing without parenthesis? it also shdu works
puts "hi #{name}"


did you try calling your funtcion
person_age_in_hours "Paola", "23"
like you are defining the correct format it should give you the correcct answer even if you send a string value.


谢谢提醒，在windows上没报错，我下次提交是修复


Linaro recommands not to combine ftree-vectorize and unrolling loops.

Since O3 add tree vectorizing, are you sure it is a good idea to combine it with loop unrolling?


JSコードにてhtmlを組み立てていますが、テンプレートを使用するように修正してください。


May I see that statement?
Afaik, loop unrolling just increases the code size with a bit improvements
in performance.
On Aug 5, 2015 12:22 PM, "Maxime Poulain" <notifications@github.com> wrote:

> Linaro recommands not to combine ftree-vectorize and unrolling loops.
>
> Since O3 add tree vectorizing, are you sure it is a good idea to combine
> it with loop unrolling?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2b32ac8627030749f47dd228d1bc584d84885a1c#commitcomment-12531648>
> .
>


I wasn't asking a question.  I'm telling you that the code is broken.  The code starting at line 89 (85 before your change) needs to run on widget startup, but it no longer does.   You presumably need to change `computeProperties()` to use that new `justCreated` boolean flag (or whatever it's called).


@motivatedHamster This line is needed for Heroku to get the secret_key_base from an environment variable.


@motivatedHamster The app is available on Heroku at the following URL:

morning-beach-2764.herokuapp.com