Вот сюда нужно **явно** указать самую свежую версию `Flask-Testing`, иначе тесты падают из-за jarus/flask-testing#68. Она в `master` закрыта, но в `0.4.2` ещё присутствует.


97902934_ZoneMixing (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Even in the compat layer, I don't really feel comfortable with panics. If that place is unreachable, why do we even return ok from End?


We don't need this variable, lane never changes - only x and y change, so you can check if the player is in its lange using self.lane.


@rahmanusta We need to find a generic way to load the template (or at least a way to override this code because in  other environment afx will not be available).

A simple way would be to use [File.read](https://github.com/asciidocfx/asciidoctor/blob/master/lib/asciidoctor/opal_ext/file.rb#L113) ?


Hopefully everyone is happy with the resolution to this.  We have published
a new version.

On Fri, Jul 31, 2015 at 9:39 AM, Marcos Caceres <notifications@github.com>
wrote:

> Okay, if I can figure out why a test in branch p466 is failing I will fix
> it and put in a pull request.
>
> I'm on IRC in #specs if you need any help with anything.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/85e2f950d69028a7300033fa05aee3281a64c0db#commitcomment-12467831>
> .
>

--
Shane McCarron
halindrome@gmail.com


Forgot to optimize the svg.


This is not needed, we already have in_lane(), returning True if player in its lane.


Presently core_file is always None, need to activate and test for exception


sure....doc-strings!


Use:

    if player.in_lane():
        ...
    else:
        ...


Same as above, use player.in_lane()


This idea is to have a module manager that brings a default module. So if someone wants to implement a ui-redis-logs module, or a ui-sql-logs module, it's possible.


@dbotha So it wasn't iOS that had the UK using metric, we had set an override for it.


better: "Removing a category also means to delete all child categories of this category:"


I think we should resolve the injection recursively to maximize the convenience of this concept


These lines are not needed and not related to your change.


`lftNormalFee` - `lft` means what?

`d` means what?  This isn't academic - I'm trying to figure out what happens in the next one or two dozen lines _in terms of business logic_ and I'm drawing a complete blank.

Oh, I know the number theory fine - you're essentially reducing three fractions to lowest terms... but as to why, I'm baffled.   What are `baseFee` and `fee`?  Why do you keep conditionally swapping them?


maybe: "changing the reference to the parent category"


Kein README.md? :-1:


That is sure. I think we can use File.read for default implementation.

Another question is shall we precompile jade templates to its function ? If yes, where to do that ? in Asciidoctor.js grunt file ?


Why is ensureAKE unnecessary here?


This is a  simple AngulaerJs base form.
At the opening windows you can choose between login or registration.
Try to choose the registration option and try to enter a user name.
When you try to enter the username 'elad' the input border change is color to red.
Go to the api/user.php. You can see it is return true if the user name exist.
You can create your own php file file that return True is a user name is exist


謝謝你的關注，ubuntu系統下我沒有去實作喔


Right I did not reread the ifs too closely I thought it skips some args which would not make sense :) I'll fix it and push.


Piter pi el folder    "register" es el nombre del modulo,  pon tu js en una carpeta con el mismo nombre de tu modulo o controller para identificarlo

por ejemplo news/formNews.js  @pabloilopez


You mean you cannot just unset a parent category and so making it a root category? But you can create a new category with no parent set?


I'm not quite following you here.  If you do it through a query you will just have another DAO method that says public ArrayList<Book> findBooksByAuthor(String author);   This should build a query that retrieves any book where that author is attached to it in the database.  At the end of the day you will be given an ArrayList of Book objects all of which will have that author attached to the book.  The only way what you have now makes sense is if you returned an Author object that is filled with an ArrayList of Book objects, which is completely contrary to everything else you have coded to this point.


97902934_ZoneMixing (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)


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

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

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

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

```


FAZENDO CAGADA NE MURILO AEJAEAEKAKEKAKEKAE @muriloaraujo


Why?


>I'm curious why my usb speed is so slow?

A good question. After fixing this function, let's see what others report and then try to find a commonality. Can only be the client side.

> And as a last defense,

No offense was intended.

> And do you mean that the CMD_DOWNLOADED_RAW_ADC_SAMPLES_125K should not be "storedCommand(UC) in UsbCommandReceived() ?

Exactly.


POSSIVEL TRETA ???? isso pode virtuozi ? @guivirtuoso  @caibar @hudsonssrosa  @paulocesarmsf


Cara, colocasse uma classe sem classe


maybe better: "In order to fetch products that belong to a category or to a category tree you specify the category IDs in the query predicate:"


TeamCity DNN JavaScript Libraries :: CI [Build 58](http://teamcity.engagesoftware.com/viewLog.html?buildId=326&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 58](http://teamcity.engagesoftware.com/viewLog.html?buildId=326&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:9


Isn't it wasteful to do the message parsing several times? Why don't we just do this at the same place as parseMessageHeader would be called in the main flow?


One last issue here. `_tgengraph_getWeight` still returns a pointer and it may be NULL. If its NULL, you may not dereference it or you will get a segfault. So, return this instead:

```
return (weight != NULL) ? *weight : 0.0;
```


That's not exactly true. We are allowed to give this information to the user.


TeamCity DNN JavaScript Libraries :: CI [Build 59](http://teamcity.engagesoftware.com/viewLog.html?buildId=327&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


Done! Very short, but there's not much to say about it :-)


TeamCity DNN JavaScript Libraries :: CI [Build 59](http://teamcity.engagesoftware.com/viewLog.html?buildId=327&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:6


TeamCity DNN JavaScript Libraries :: CI [Build 60](http://teamcity.engagesoftware.com/viewLog.html?buildId=328&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 60](http://teamcity.engagesoftware.com/viewLog.html?buildId=328&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:6


Where does it say instance tag verification should happen before version check?


The submodules seem to make problems with travis. Can you have a look?


@dorkster Can you review this? Please check if I have converted correctly:
* missing chances
* present chances when power is missing
* 0 chances
* beacon without chance
* anything you will find


does hogan.compile memoize the results? If not we want it because many elements will reuse the same template. And we do not want the template compilation to be done for each element each time we do the rendering.

BUT one thing the should be done is to avoid storing that in the state. Actually the memoization contains a state in itself so we do need to rely on another layer of statefulness.


Ah, sorry about that. I've updated the script to fix that.


Also, I'm open for all suggestions instead of using Slide class.


We are looking at the relationship between sudden oak death disease, biodiversity of woody plants, and wildfire disturbance. This repository is to store the data files and the R code used in the analysis.


hi @zmartine, to filter using a different quality score metric you just have to change this line


/js/lib/summernote.min.js  agregalo tambien al gulpfile.js en el array de libs   ,  y solo corres en consola en tu directorio raiz    gulp    para regenerar los archivos lib.js   modules.js   all.css    asi ya no sera necesario estar cargando cada js  en cada plantilla,  lo mismo con los css


@slarson Yes, will do next time.


:8ball:  Your're right you your localization port not work for me :-(
Why not direct in the Makefile, like textproc/aspell/dicts/$BAD_LICENSE_LANG/Makefile


Note, secret was changed. This one is now invalid.


Because I can have two conversations with the same account using two different versions happening at the same time. We should ignore messages not sent to our conversation (by checking instance tags) and it includes ignoring errors due versions for this messages that should be ignored.


No {} for single-line if blocks please, see CODING_STYLE


the problem is that usually you have distutils installed and updated by yum :wink:


This is still incorrect, better to simply say `Downloading wkhtmltopdf`


Date.today is only available if you `require 'date'`. The rake install will fail with `undefined method 'today' for Date:Class`


javascript is self hoisting so this is unneeded i guess but #yoloshipit


But I want to be able to return NULL so that I know there's not a valid key there, see its usage in _tgengraph_parseChooseVertex. Would you prefer as a safer alternative I add a boolean-returning function that checks if a key is valid, separating functionality?


lol...


Oh yeah, good point!


This is working fine


why does this need a try/except?

showPolyData returns the obj, so you can just say "objScene = vis.showPolyData(...)" and skip the findObjectByName() call.  Also, you can pass a default color array to showPolyData:

vis.showPolyData(... colorByName='rgb_colors')

Also, if you are calling this many times, perhaps you want vis.updatePolyData instead of show?  The show call always creates a new object.  Maybe that's what you intended.


This isn't an improvement, because `realloc()` copies the old buffer contents. It's also a semantic change because it doesn't clear the memory anymore.

Either stick with the `free(s)` as it was before, or drop the entire patch. TBH, I don't see much of a benefit using auto-cleanup features to avoid two `free()` calls.


Approved


Yeah, good idea, done.
Although I will keep a check that the hash string literal does actually correspond with the test data.


#1 make sure to define the variable every run... make this mandatory


5058-fix-readvars-cli (nealkruis) - x86_64-MacOS-10.9-clang: OK (1772 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-100.0%25-green.svg)


Nope, it won't work at all:

```pawn
MyFunc_()
{
    return 0;
}
#define MyFunc(%0) (I@=MyFunc_(%0)),I@

main()
{
    new
        I@ = 1,
        FALSE = 0;
    if (FALSE && MyFunc())
    {
        printf("hi");
    }
}
```

That code will print "hi".


Would it make sense to use your Github handle in the TODO?


We should also display the bio here.


Also, I'd prefer if we'd use strv_isempty() to check if `match_path`, `match_driver`, `match_type`, `match_name`.

Also, there a checks missing for `match_host`, `match_virt`, `match_kernel`, `match_arch`.

Given how complex that check will then be I think I'd prefer a new helper call `network_match_empty()` being added for that.


AddCalcCoilSurfTempSource (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


@Stefano80
Do you ever plan to try out my one idea?

When tuning just initialize the array as the default value:
pieceType[6]={200,200,200,200,200,200}

The results should be interesting.


Hi Marc,

I agree that they should be removed. After reviewing it I intended but forgot to remove these params in https://github.com/phpmyadmin/phpmyadmin/commit/b47b4e3a5e629e5180cff593615421457f5255c0.
@gzzhanghao please confirm.


This looks good to me, but please add some tests for this new function in src/test/test-util.c


No naked except clauses please.


Interesting. I'm assuming this also deletes the associated measurement.


No, keep `_tgengraph_getWeight` how it is. I suggested changing line 807 to what I had in my last comment. This is not changing what `_tgengraph_getWeight` returns, rather checking that if `_tgengraph_getWeight` returns NULL we don't segfault. Does this make sense?


the changes are badly fine


Got it :+1:


These debug messages are not coming across when -VV is passed.
After running `export KNIFE_DEBUG=true`, the manifest debug logs came across. As a nice to have change, I would like -VV to be enough to trigger debug logging across the entire knife code base.


``self.getInputCloud`` can be used in conjunction with LIDAR so it would raise an exception if I tried to set a property that is invalid.

I will change to avoid the findObjectByName(). It's not called often, right now once per run through, will use updatePolyData nonetheless as we might call it more often down the line.


Who is this maxsechz guy that seems to just invade all the computers here?


- Beacons should have an explicit 100% chance. Leaving it blank treats it as 0%
- The power that already have an explicit 0% chance is id 136, which is Disenchant. This is used by the player to untransform, but shouldn't be used by the AI. So it is safe to leave that as-is.

Everything else looks good.


it was here ;) - should be aligned with sources

As my colleague carla says "As a dog can hear noises we cannot, YAML can see malformatting that is not at all visible to the human eye"


So its just ie8?
On Jul 31, 2015 3:23 AM, "Michael Naumov" <notifications@github.com> wrote:

> This comment can be removed. As the issue is not happening in IE7. I
> tested it like
>
> var d = document.createElement("div");
> d.customProperty = { toString: function() { throw "toString failed"; } };
> d.attributes[0].value === "null"; // no errors
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/megawac/MutationObserver.js/commit/a10b63108c28d765eccc46bb55d316e8ee2b3142#commitcomment-12460902>
> .
>


This was cleaned up via https://www.pivotaltracker.com/story/show/99470044, see commit 4a2c934


That's correct. `parse_size()` has

```c
        assert(base == 1000 || base == 1024);
```


typo


Apologies, I see what you mean now. New commit shortly.


![Uploading P10200372.jpg…]()
madrugabot


Ok, no problem.  You can also do:

    if 'rgb_colors' in objScene.getArrayNames():
        ...set property...

Another option, if you pass the array name directly to showPolyData but the array does not exist then it will print a warning but not throw an exception.


Correct -- otherwise it broke the views :)


This is a model method -- see the other commit


Yes, `abortStream()` looks to be the way to go, but `abortStream()` isn't thread-safe so make sure you call it from within `NettyServerHandler` (or anything else running within the Netty-Channel's event-loop). That does mean that the status needs to be included in `CancelServerStreamCommand` as well.


Since only one `EventPump` can exist at any given time, I'd rather not hold directly onto the pump within `Sdl2Window`. The event pump is retrieved only during event polling. I'm not sure of the performance implications there, but it allows the calling code to do extra stuff outside Piston event polling _just in case_. More flexibility, more options.

IIRC my reasoning for holding onto the event subsystem was just to make sure it's initialized during the lifetime of the window, because it's not possible to obtain a copy of the subsystem after `Sdl`'s been initialized externally, and it would be needed during `poll_event`. Maybe `Sdl` could hold `Option<Subsystem>`s for each subsystem so they can be retrieved easily if they're already initialized? And the functions for getting the subsystems could instead return the value contained in the `Option` if it exists.


dea Kartufel


priwejt statig Stringi deaHitler;


@Overdrajw


publik wojd onDisejbl()


Thanks for the hint - incorporated here


WindowsToGówno.ołt.printełIn(costam);


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

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

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

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

```


importer orginalny.diorite.pluginy.Plagin


deaHitler.toStringi()


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.414.75bc86ae621c953934f902819443b90c6ea05dd9](http://build.palaso.org/viewLog.html?buildId=60417&buildTypeId=bt322) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:01:55


Some games do things like call it 3-4 times on startup before they first call WSAStartup.


I'm assuming I made my git info global on one of the AA computers. Just figure out where the commit came from and please make anything but me the global account


I think a better fix is to just free the memory on bitmap_clear(), not just decrease the counter.


I hope I didn't make a pre-emptive safety measure here, but if the subsystem isn't `Copy`, then it can't be used in a non-`move` closure, correct? By cloning it, I just move and drop the clone so `video` can be used further down.


Ah, what a silly mistake on my part.


I guess that kernel doesn't write to the buffer in the ERANGE path, so first 64 bytes were still clear.  And it's ok since isempty() only checks first byte.

Anyway, this patch is just for cleanup and I think it'd help readability of the code a little bit.  If you think it's not needed I can drop this patch then.  Thanks for your review.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.442.75bc86ae621c953934f902819443b90c6ea05dd9](http://build.palaso.org/viewLog.html?buildId=60419&buildTypeId=bt223) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 62, ignored: 1; exit code 1 (new)) Build time: 00:03:47


Ok, I'll take a look at the test code too.


http://www.ecma-international.org/ecma-262/6.0/index.html#sec-parsemodule, Step 12, sets `[[Realm]]` to `undefined`, we just need to fill the blank.


```c
if (r != j->get<1>()) {
  // length, i.e. j->get<1>() is the multiple of stripe_width, and r should be equal to length. if the returned shard is the expected piece.
  // probably we can take this case as an EIO.
}
```


I agree.  I think that is out of scope for this PR.  Happy to work on it though.


Commits look ok to me.


Nice work.  and nice find on the shinyTable


Doppelter Commit, sorry!


I will just then take that original commit out of the history


Causes tests to fail on Windows using MinGW-4.9.1.
The Error: The procedure entry point vsprintf_s could not be located in the dynamic link library msvcrt.dll.


![Uploading puppy-345334_640.jpg…]()


@ajklein, @dherman I'm not sure if we will need to set the backpointer before resolving `_entry_.[[Instantiate]]`, in case someone have a hook that relies on `_instance_.[[RegistryEntry]]`.


I think, this commit can be squashed. ;)


Better add sol_flow_node to struct max31855_data, this way you don't need to do any allocation.
Nitpick: always name the variable with the node data to mdata, all other nodes keep this way.


@czchen indent_size 設到 8 也太大了吧？還是你習慣 Tab 間隔那麼開？


You are right @Simage and @maethor and I thought it was a good thing to make the WebUI have some default modules. The main idea is to avoid that WebUI users have to install and configure many modules before the WebUI is operationnal ... **out of the box** is the concept :-)


Please check for errors here.
if (status < 0)
....


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.443.61da6d6dd4f45bbfa7782d73c984a9f10a4cd930](http://build.palaso.org/viewLog.html?buildId=60421&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:08:43


@XhmikosR  Why would you revert this?

**Unnecessary languages need not to be installed and take up disk space.**
Users don't use 99% of this, so why not let them choose their own language and keep it that way?


@mfonville I believe this is covered by the api19 arm.neon version, or is it important to support a "arm only" version too?

This also reminds me how best to handle reverting commits like this and 87bbb272e1819a2652286347ee40a386233ca5c9 were a higher api already provides support.  For 87bbb272e1819a2652286347ee40a386233ca5c9, I just deleted it in {just went looking and saw you got it already :) }


Does MinGW have ```sprintf_s``` but not ```vsprintf_s```?
If so, we need to change the tests to check ```vsprintf_s``` directly.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.415.61da6d6dd4f45bbfa7782d73c984a9f10a4cd930](http://build.palaso.org/viewLog.html?buildId=60420&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 130 Build time: 00:09:02


I think is because we're calling startAKE inside of dhCommitMessage


Ooops. Thanks.


I'm not sure quite how, but Head had an experimental version rather than the correct one.

While the change in this pull request helps, there are numerous other issues with the version.  I've put the correct one in after completing the tests.


I'm going to need some help to implement this. Can't get this to work this way


On 31/07/15 16:20, scls19fr wrote:
> I don't say that "it's not right". I say that's a bit ambiguous for
> someone who had never seen URI:

I reworked the README.
Let me know if it's better.


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

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


If a call to max31855_process() happen and the previous sol_spi_transfer did not had finish? It would fail and user would only receive one flow packet. I don't know if this is a problem but if it is you should keep a flag to know that there is a pending transfer and when this one finish you start a new one, even better keep a 'unsigned pending_packets' inside of max31855_data.


Yes, this one is already covered by the api 19 version.
Good for spotting that. I will then also pull this commit out of our history.


Approved


meta.yaml needs updating too.


The commit message is inaccurate I guess. I am _hoisting_ but it's because we declared `var message` multiple times. So I'm not hoisting to be a hoister. I'm hoisting to quiet jscs.


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


Also this one is already superseded by the api19 version, so I will also pull this one.


I suppose you it can be changed to const REBVAL *. But the content pointed to by func->info could be changed evey time it's invoked. Not sure what it will buys us.


93469234-DemandResponse2 (Myoldmopar) - i386-Windows-7-VisualStudio-12: 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)


Thanks!


@ckaznocha @allanliu Please review and I will send a PR to the maintianer


What's this about?


@ntustison - so is the majority of the speed up from checking non-zero pixels?


Thanks again!


This README looks great.  Would you mind wrapping the lines to 79 characters?


Also this one is already superseded by the api19 version.


Note to self: delete `.idea`.


``"".format()``


You could install vim-go and add
`let g:go_fmt_command = 'goimports'` to your vimrc :wink:
This will format automatically when you save a file.


@diomed: that is just stupid, what you say. It's just a few KB.

And again, once and for all, bloating the ONE installer page just to be hip, it's wrong. Make a proper patch to add a new clean page for that in the installer.


@ALShum: What - no Wendys?!?


I figure streq_ptr() could be reworked to use strcmp_ptr() then, too?


I also cleaned up the other <19 commits if they were already superseded by a <=19 version.


should read `--size=SIZE`, not just `--size`.


@berrye - Have you tried Super Burrito on Johnson and Edgewood?


@Kevin-C-Bell I moved to DC from Tampa...there was a spot in Ybor called El Porto I would eat at with my Kforce coworkers


哦，请问你在windows下是使用14565b的软件去记录电流数据吗？


@ankita2986 I love naan bread


Yes. This is what allows grouped buttons to keep their border when they no longer have spotlight.


Surprised by 149 chr line width ;)


@jzaefferer: can you push the npm package too?


Yeah, that seems reasonable.


This function seems like it has a kind of weird contract.  It creates the directory, but only in one case and not the other.  That is, in the `Nothing` case, if `~/.stack` is missing, shouldn't it create it?  And then the comment for this function can mention the side effect and the invariant that the dir exists after this function returns?

(I came across this because I was looking for the approved code path to create `~/.stack` if it doesn't exist.)


Submitted pull request #9 to get this fixed.


Still working on this btw, just a bit busy.


Mainpaths in this part for:

![Bild](http://i.imgur.com/RlvDlnb.jpg)
![Bild](http://i.imgur.com/UPUMswE.jpg)


I want title 'Enable Keybindings'.
Its clear that making checkbox off disabling keybindings.
So I don' think 'setting-description' section.


@Dwaynekj it looks good to me. Before you submit a PR, to be good citizens, can you squash your commits and add some comments explaining what the properties of the Marathon struct are for?


What's the benefit of using the `eos` lib over `stream.on('end', ...)`?


Ah, então a suposição é que essa variável será instanciada ao iniciar um
contêiner..

Mas no caso de rodar o script diretamente terrenos problema pois a variável
não estará instanciada certo?

--

Atenciosamente,

*Luciano Panepucci*

Cofundador e Diretor de Tecnologia

+55 (16) 3307-2068 | +55 (16) 991 728 681 | skype: lpanebr

*Editora Cubo*

soluções para o universo acadêmico

Rua Marechal Deodoro 2709

13560-201 São Carlos - SP

www.editoracubo.com.br


Hmm.. right.  Like this?
```
bool streq_ptr(a, b) {
    return strcmp_ptr(a, b) == 0;
}
````


Nitpicking time! :) Sorry :) If there is no need for the actual string (even then you can get it from the enum), using an enum is cleaner, less errors and easier to understand


@jmcphers I missed this `tryCatch` in the `rsconnect` patch as well.


Thanks a lot!


Good catch, thanks! Maybe we should start using `lintr` via travis to spot these style issues -- what makes me sad anyway that we could not find a way to run public tests on the package yet.


The code you removed (making exp an int64, checking if it is < MinInt32 or > MaxInt32) was not unnecessary. For example, adding this test: `"123.456e" + strconv.FormatInt(math.MinInt32, 10),` should generate an error, but no longer will.


OK, will fix!


The naming of these methods is a little confusing to me - "StorageFromImage" sounds like it creates a storage object from an image, and the method that it calls, storage.ContainerCreateFromImage sounds like it is creating a container from an image, but what it is really doing is initializing the storage for a container.

I think a minimal improvement would be to rename these methods to be a little clearer, maybe just StorageFromImage -> InitializeStorageFromImage or something.

However I wonder if it's possible to avoid this split initialization in the client code - do we need to do containerLXDCreate and then a storage initialization there, or can we instead move that here and provide containerLXDCreateFromImage and containerLXDCreateFromNone , which might simplify the client code.


It will be created later no master what. We have to create it before canonicalizing it because otherwise that will throw an exception


Yes, exactly.


? Isn't the bind something that the Nifty calls when it binds itself to the screen controller? What does this do?


I replaced the last three lines of the patch with these:
```
     doppler:
       id: doppler
       secret: dopplersecret
```


Great! I will merge it into the develop branch then.


Is a issue if user call am2315_read_temperature() and before the _read_data() run, user call this function again or am2315_read_humidity()?


drop mdr arg


Connected to #165


use caller_uid, setattr_uid args here


and here


The old do not alloc/free each time. It do what I suggested. So not doing this is a regression in the new back-end.

We can just tell in the docstring of that buffer the semantic and how to use it. We don't want to sync twice when only once is needed.


jsut return false.. we don't have mdr


`unsigned`, not `unsigned int` please... not htat it would matter much


just return false, skip the respond_to_request here too


Not bad


Why do you implement `broadcast` here? The comment says that it's not possible.


Sorry - TRON caught that - but thanks to JeffR anyway!


`function (str) {`


which flags are you talking about? This isn't theano flags. nvcc/opencl compilation flags?


I had the same mistake, the problem was that I manually changed gamebuild.

You have to have gamebuild version 20182 Table realmlist.


97902934_ZoneMixing (macumber) - i386-Windows-7-VisualStudio-12: OK (2214 of 2236 tests passed)

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


Ok, I added missing uk/ru translations in https://github.com/symfony/symfony-demo/pull/128


then, change the original check_access() so that it calls the new function (_check_access instead of _check-access_async()?), passing in mdr->client_request->args.setattr.uid, gid, and mdr->client_request->get_caller_uid() and gid(). and if _check_access() returns false, the check_access() caller will respond_to_request(mdr, -EACCES).

that way there logic is only in one place, and the outer function is just feedling args in from mdr and providing callers with a more convenient interface


93469234-DemandResponse2 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1783 of 1783 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)


Do it reuse memory? If it cached memory, it could be faster.

On Fri, Jul 31, 2015 at 11:09 AM, abergeron <notifications@github.com>
wrote:

> No, but h_malloc() is malloc() + some other work so I can't see it being
> slower.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abergeron/libgpuarray/commit/abf32a113ff141cc31498a6cdf20d9df1295ef2c#commitcomment-12468587>
> .
>


To me, this is a feature of any multiple-choice-type question, in the sense that even if you have no idea will get some amount of partial credit based on pure luck. In the case of this multiple-choice question with partial credit, this is particularly bad in that you will, on average, receive 50% credit even if you have no clue. (That's just because there's a 50% chance of getting each question right, because there are only two choices for each question, and in partial credit mode each question is essentially scored independently.)


I already got it ;) Saw it right after I pushed. @sindresorhus does your node module boilerplate have JSCS configuration/npm scripts in it?


These log message seems overly complex.  Also, we may want to parse these messages with logstash, and the simpler to parse the better.  How about just:

```javascript
console.log('%d sources(s) still fresh, %d source(s) need updates', retrieved.length, errs.length)
```


You're right. It's not required. I can move this to RuntimeUDFContext instead, which isn't abstract.


No, but we're planning to replace JSHint by ESLint.


isEmpty() should be 0.000000000001 cycles faster :)


Discovery is always asynchronous... if a discovery operation is blocking in nature, it must run in a background thread.


Streams can end in a couple different ways depending on what type of stream they are or if they encounter an error and eos abstracts that away.


Okay, there  you go... http://issues.joomla.org/tracker/joomla-cms/7587


On 07/31/15 16:59, Rafael Sadowski wrote:
> Why?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jasperla/openbsd-wip/commit/6ca74724e4e8d915dac9d4e0834af99d355c3c25#commitcomment-12469508>.
>
There was a request from the porters to add -i to sed, see tech@ mail
list, this is now supported in the base sed version. The main reason was
to avoid the use of gsed / changing scripts from sed to gsed.

As a result many ports have been switched from using perl for inline
edits to sed -i. This is just following that lead. see port-changes@
mail list.

CVSROOT:        /cvs
Module name:    ports
Changes by:     jasper@cvs.openbsd.org  2015/07/18 15:11:18

Modified files:
        databases/directoryassistant: Makefile
        databases/evolution-data-server: Makefile
        databases/freetds: Makefile
        databases/gdbm : Makefile
        databases/jxplorer: Makefile
        databases/mysql-zrm: Makefile
        databases/openldap23: Makefile
        databases/pgfouine: Makefile
        databases/pgloader: Makefile
        databases/puppetdb: Makefile
        databases/py-ldap: Makefile
        databases/py-sqlite2: Makefile
        databases/ruby-bdb: Makefile
        databases/ruby-hiera: Makefile
        databases/ruby-hiera3: Makefile
        databases/skytools: Makefile

Log message:
use sed -i

sleuthkit and nikto are the two that I have changed.


I'm not sure - is there a benefit to this notation or is it just personal preference?


Approved
Chain up .replace statements


It's possible that, as `_update_readings` will ignore the second request and return cached data, cached data is invalid - especially on first execution. Suggestions?


This won't necessarily be the complete set of TMs which are involved in the execution of the job. Imagine that shortly after the submitJob method has finished a new TM registered at the JM, then this new TM can also be used to execute the job.


You should be using go-ldap now as that seems to be the most standard and maintained.


I just reverted the commit. Thanks!


htobe16() is much nicer than htons() really, much less crufty and explains what it really does, because it actually indicates it works on uint16_t and doesn't pretend to work on "short"s, but then does not.


@bsongis The option to set "Serial port" could be navigated to, but the value could not be changed (tested on 9XE). It now works, but I am not sure this is the right solution! Please verify!!


@leycec You removed the `_MEI` directory level. Is this intended? If so, we need to adopt `libraries/test_onefile_tkinter.py`.


shouldn't you compare == CALLBACK_RETURN_UNDEF rather than == 0 here?


So would it not be good to add `liftIO $ createDirectoryIfMissing True x` directly under the `getAppUserDataDirectory` line?

What I'm fixing atm is that I found a bug in the current #670 that I just patched up.  If I ran `stack haddock` as the very first thing on a fresh machine, it tried to grab the lock but `~/.stack` didn't exist.  This happens *after* `loadConfigWithOpts`, so I thought that it would have been created already, but I guess not.

Anyway, for now I fixed it by playing it safe and doing a `createDirectoryIfMissing` right at the point where we reach for the lock.


I will rename this commit. The title doesn't really resemble what got changed.


Fixes Issue #1


@leycec Okay, I found the commit-message for the related change 3ec8fed41a. I'll update the test-case.


Are you running the specs in opal also?  (And the browser specs?)
```BROWSER=firefox be rake```


Yes. I let this be for now. I'm going to add another message of type {{InstanceListMessage}} which will notify the task managers of any new registrations.


you create a new object here, and return EINVAL on error? sounds suprising, no? should be return -errno, or return -ENOMEM or so?


Hello people


It's my understanding that it's generally better to avoid `Array.Copy` if the src/dest arrays are `T[]`s (as opposed to `Arrays`).


пинг


paso


`<build-ckeditor-plugins-wsc /> `and `<build-ckeditor-plugins-scayt />` are already included in `<macrodef name="build-ckeditor">`


`for (;;)` sounds nicer than `while (1)`. If at all it should be `while(true)`...

THIS IS SUPER IMPORTANT, DAMNIT!


A slot can contain more than one task. A slot contains a parallel pipeline which can have an arbitrary length and thus arbitrarily many tasks.


Use only single quotes.


Sick.
I've started making an editor gamestate that's going to contain all the widgets we need, and I made a super simple combo-box to change the current drawing mode (now that we have two!) Not going to commit yet because the entire Placer code needs re-jigging to fit with the new structure so having a black screen really isn't useful for testing purposes. I was trying to decide whether the editor should manually grab data from the widgets to change its settings, or if all tools need to just reference the editor for direct changes. My instinct tells that me the latter, while kinda messy and un-OOP, is going to cause a lot less of a headache, especially with Placer.  The editor should be storing stuff like shapes too if we want undo/redo functionality, which means Placer is pretty much just a means of manipulating data stored in Editor and visualizing the whole thing. So I'll be migrating a lot of the data from Placer into Editor and having it take Editor as a constructor argument, similar to how my combo box works. I'm getting bugged into going drinking though so lol noprogress
#sociallife


The reason they are different is because the loader works differently if there is no bucket and directory passed (it just checks for fresh sources). When a place to put the data is specified, it gets loaded there. I think the difference is meaningful, but I will definitely update the logging mechanism to use winston over plain console.log()s (and it should also respect the --quiet flag which it currently doesn't


added to http://collab.kpsn.org/display/tc/smart_scripts#smart_scripts-action_type


Can you point me to the relevant documentation for `Hashtable` that describes the reader/writer behavior? I can't find anything that indicates that a non-synchronized `Hashtable` makes the guarantees described by the comment.


It's fine to create the directory at that point, it's just never been
necessary and therefore was unnecessary overhead

On Fri, Jul 31, 2015, 10:05 AM Ryan Newton <notifications@github.com> wrote:

> So would it not be good to add liftIO $ createDirectoryIfMissing True x
> directly under the getAppUserDataDirectory line?
>
> What I'm fixing atm is that I found a bug in the current #670
> <https://github.com/commercialhaskell/stack/pull/670> that I just patched
> up. If I ran stack haddock as the very first thing on a fresh machine, it
> tried to grab the lock but ~/.stack didn't exist. This happens *after*
> loadConfigWithOpts, so I thought that it would have been created already,
> but I guess not.
>
> Anyway, for now I fixed it by playing it safe and doing a
> createDirectoryIfMissing right at the point where we reach for the lock.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/commercialhaskell/stack/commit/5e895eaa3ed3fdfcfc6e48ed1f5ca27eab415192#commitcomment-12470680>
> .
>


Oops.. Don't run sudo here


This should be thread-safe since `enqueue` and `dequeue` is accessed by multiple threads. Or simply use thread-safe queues.


FWIW, all operations on `_domainTable` (other than the replacement on line 379) are performed under the same lock. So even if the guarantees made by the type differ, the locking behavior should be the same.


Done.


Is that one a descendant of the other two? Hologram isn't my project, but I can try proposing a migration to the owners.


The `InstanceManager` should know when new `TaskManagers` are registered and unregistered.


I was under the impression that a task manager has several slots(akin to cpu cores), and each slot runs one Task. What is the parameter to find out how many parallel tasks are being run? At one worker node. Will getNumParallelTasks work?


"knife bootstrap IP..." worked after this commit. Thanks


@RyanBalfanz this is the function I found on SO to determine if a class exists on an ancestor element. Here's the SO post: http://stackoverflow.com/questions/16863917/check-if-class-exists-somewhere-in-parent-vanilla-js

I assign this function to a variable on line 117 below `checkAncestorClass`, and then use that variable to return a boolean value on a conditional statement before running the loop.

I think this works. If I console log `checkAncestorClass` in the conditional statement on line 119, it returns `true` as it should.

I think this implementation solves the assignment "Shows a different alert when it fails to find a parent with the given class name ("No parent found with that class name")", but I want to double check with you.

Thoughts on this approach?


@kidwm 其他值我也 ok。我大部分 project 都是用 space，然後把 tab 設定成 8。


oh, ok


I was kicking around an idea like that before--took a slightly different form, but similar purpose.  I think that will have to wait though (and won't really help resolve this issue I don't think).

I'm working on a refactoring of the polynomial classes that will make them easier to deal with and subclass anyways (without adding the more complicated notion of a multi-parameter).


@ldoktor ok, let me see if I need to update inspektor on pypi.


package com.tzvip.testone;

import java.io.BufferedReader;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.ArrayList;

import android.app.Activity;
import android.content.Intent;
import android.net.Uri;
import android.os.Bundle;
import android.telephony.SmsManager;
import android.util.Log;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.EditText;
import android.widget.Toast;

public class MainActivity extends Activity implements OnClickListener {
	private Button bt_save;
	private Button bt_call;
	private Button bt_sendmessage;

	private EditText et_phonenumber;
	private EditText et_messages;

	private String number ;
	private String messages ;

	@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.activity_main);

		Log.i("MY_INFO", "我现在已进入到acitivity中来了！");
		MyLog.d("MY_INFO", "这是我自定义的日志信息");
		MyLog.write("这是我自定义的日志信息");
		/**
		 * adb logcat - help 查看logcat命令 adb logcat - s INFO 过滤tag日志
		 */
		et_phonenumber = (EditText) findViewById(R.id.et_phonenumber);
		et_messages = (EditText) findViewById(R.id.et_messages);

		bt_save = (Button) findViewById(R.id.bt_savelogs);
		bt_save.setOnClickListener(this);
		bt_call = (Button) findViewById(R.id.bt_call);
		bt_sendmessage = (Button) findViewById(R.id.bt_sendmessage);

		bt_call.setOnClickListener(this);
		bt_sendmessage.setOnClickListener(this);

	}

	@Override
	public void onClick(View v) {
		Toast.makeText(this, v.getTag()+"", 1).show();
		switch (v.getId()) {
		case R.id.bt_savelogs:
			// 控制cmd命令保存的日志
			// 模拟cmd命令采集日志
			ArrayList<String> cmdline = new ArrayList<String>();

			cmdline.add("logcat");// 日志采集命令
			cmdline.add("-d");// 只采集一次日志
			cmdline.add("-s"); // 过滤
			cmdline.add("MY_INFO"); // 过滤tag

			String[] strs = new String[cmdline.size()];
			try {
				// 得到这个进程
				Process process = Runtime.getRuntime().exec(
						cmdline.toArray(strs));
				// 通过进程，取得输入流
				InputStream is = process.getInputStream();

				BufferedReader br = new BufferedReader(
						new InputStreamReader(is));

				String str = null;
				StringBuffer sb = new StringBuffer();
				while ((str = br.readLine()) != null) {
					sb.append(str);
					sb.append("\n");
				}

				Toast.makeText(MainActivity.this, sb.toString().trim(),
						Toast.LENGTH_LONG).show();
			} catch (Exception e) {
				e.printStackTrace();
			}
			break;

		case R.id.bt_call:
			number = et_phonenumber.getText().toString();
			Log.i("com.tzvip.testone", "打电话-->"+number);
			Intent intent = new Intent();
			intent.setAction("android.intent.action.CALL");
			intent.addCategory("android.intent.category.DEFAULT");
			intent.setData(Uri.parse("tel:"+number));
			startActivity(intent);
			break;

		case R.id.bt_sendmessage:
			number = et_phonenumber.getText().toString();
			messages = et_messages.getText().toString();
			Log.i("com.tzvip.testone", "发送信息-->"+messages);
			//产生一个短信管理器
			SmsManager manager = SmsManager.getDefault();
			ArrayList<String> contents = new ArrayList<String>();
			contents.add(messages);
			manager.sendMultipartTextMessage(
					number,
					null,
					contents,
					null,
					null);
			break;

		}

	}
}


typo here:
multiple


What if multiple tasks (different types) access the `RuntimeMessageManager` and want to send/receive different message types?


Will do so now.


This needed to be moved inside the scope of the promise - see e636ac6 for the deets


I'm still 50/50 is we should ban `lt` outright because it's non-conforming. @halindrome, @tobie, @darobin?


This won't scale, since Akka has a fixed maximum message size limit. Once a message gets bigger than this limit, the message will be dropped.


Huh. First time I see `target_include_directories()` being used other than in "best CMake practices" blog posts :) It's weird that Eclipse can't handle the former, it's been there for ages.


whoops I'll remove these...


Vestige from where I procured the file originally. I'll get rid of it.


issue #43


Oops, eh eh, I feel very stupid now. :ghost:


All messages from any task will be broadcasted to everyone. If the message is not something the user is interested in, they can always drop it.


This can easily kill your TM if the messages get to big or the stored number of messages to high. The problem is that they are stored on the heap which does not allow to estimate the proper size. Thus, you never know when to discard some messages.


@halindrome Thank you for being so responsive!


All messages will be `RuntimeMessage` type.


Hi,
Parameter `$db` and `$table` are used internally, so if it's no longer needed it can absolutely be removed.


Check the gam example file. In sandbox example


That's very clever.  Never thought of command line options that way.  Tend to think of them as an assertion.  Have I missed examples of other commands that treat their options in that manner?  Show me a man page?


Done.  That was copied from old code of mine before we had auto. :-)


These are only called from inside the RuntimeMessageManager, which by itself follows synchronized calls. Wouldn't that be enough?


Looks good.


Thanks!


Sind das komplette Spiele oder nur einzelne Zustände von Spielen, die in read_file geladen werden sollen?


Hmm... ok.  I can't get worked up about this either way.


I assumed there would be a situation where you would want all the plugins to be built.  But if that's not the case I can change this target to just build-ckeditor-bbcode.


Yes, this is non-deterministic so it shouldn't be a test.


These changes are to be used with "seeds.rb" to create a database with different "members", "projects", "skills", etc.
@aailpl
@patriciamadureira87


93469234-DemandResponse2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


It would be nice to avoid adding a scipy dependency to do a small discrete random draw.


I don't know either.  I'll try to come up with a better name.


I can't think of a reason we'd need to update these two plugins, if the CKEditor version/ZIP didn't change. I've never needed to run them directly.


No burning reason.  I'll change it to a scoped `lock_guard`.


Changing memory interleave enable from system level to proc level attribute


TeamCity YaccConstructor :: YC.Core General Build [Build 479](http://teamcity.codebetter.com/viewLog.html?buildId=196360&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


TeamCity YaccConstructor :: YC.Core General Build [Build 480](http://teamcity.codebetter.com/viewLog.html?buildId=196361&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


No, there are not... if it exceeds it (which can happen, I've made this 2X bigger in a subsequent commit), the emulator will segfault. I'm not sure how to add the appropriate checks, or if the emiiter even supports that... for now I was just going to make it sufficiently big that no known cases exceed it. @hrydgard - any comment on this?


*Nice* find on the `DIFFERENT` command. Before comparing the contents of the files, I should `string(REPLACE ...)` the arch-specific install directory, with the final install destination directory, as there are files, like *.la created by `libtool`, that have in their contents the absolute path of the directory there were originally installed in.

Ideally I would search/replace only non binary files, but I found no way to determine if a file is binary or not, so I guess it would use `all_except_library_files` and let's see if this will break anything in the future. I'll give it a try, thanks for the input, much appreciated.


Didn't work here. I'm using Windows 10. :pensive:


Oh, the reason was that I now need to pre-declare `feeFactor` and `uRemFee` to live outside the protected scope.  But it isn't a big deal either way.


Yes. But the idea is, there won't be too many messages. Most of the times everyone will be working, and after doing some work [assuming everyone finishes more or less the same time, let's even say someone finishes half of their next work while another is just finishing the first], everyone exchanges messages and that's it.
As soon as you use the messages, they're gone forever.
Even if the parallelism of an operator is set to, say 20, we will likely have only ~40-50 messages to deal with at one time, which shouldn't be a problem.

The messageQueue right now manages explicit queue for every task. This can be easily made more efficient by maintaining head pointers for every task.


I don't know.  The `4` pre-existed my forage into this dark place. :-)


What are you testing here? It tests if the program runs, right? But this might also be true if the broadcasting does not work at all. It does not verify that the all tasks will have seen all elements either as input or as broadcast value. A more fine-grained test, also with more than one task broadcasting different types, would be helpful.


Correction: I'm not sure of a nice way to add the appropriate checks, obviously it's easy to see if the code pointer is approaching the end of the memory space allocated for the code block


@dveeden it's stale compared to their HEAD now, and we ended up not using Morgue, so I don't have the capacity to bring it up to date right now. Please take up it if you're interested :smiley:


In future you can extract all these strings [@"title", @"subtitle", ...] into constants.


Another benefit...  currently, with all access being via JavaScript, any generated bindings code is more JS that has to be downloaded with the app. If that can be pushed to WebAssembly APIs provided by the host (generated from WebIDL), then there's no download cost for the bindings. (This maybe falls under 1 above).


The `static` isn't needed, but also won't hurt.  In a std-c++11 world this should be `constexpr` instead of `const`.  gcc and clang will compile it down to the same machine code either way.  Not sure if VS-2015 will handle this or not.  So far I'm not impressed with VS-2015 `constexpr` support.  It appears it may be sub-C+11.


AddCalcCoilSurfTempSource (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1772 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-100.0%25-green.svg)


they should be.


This revision is clean.


you could make this a `descriptionFunc: => String` to get the lazyness without having to always pass a function, and you cache the result for free


What kind of exception can you get here?


www


the problem is however, that the `indexOfSubtask` is global and not local. Thus you might end up with all the tasks which produce `index % numTasks == 0`.


On Fri, Jul 31, 2015 at 8:13 AM Dieter Adriaenssens <
notifications@github.com> wrote:

> Hi Alex,
>
> Thanks for the feedback.
>
>    - SHA, I assume you mean the commit-hash? Definitely a possibility to
>    group by commit-hash, or by build ID? The chart might get overcrowded for
>    longer timeframes (month, year), though, but should be OK for week or day :
>    buildtimetrend/python-lib#102
>    <https://github.com/buildtimetrend/python-lib/issues/102>
>
> Yeah, I'd like a zoomed-out view at first, but ability to zoom into
individual items from the 'Build History' tab in Travis. It's a 'travisty'
that they don't show this (instead they show cumulative time including
sitting in the queue)

>
>    -
>    - kiloseconds : it's on the todo-list to display in 'xxh yym zzs'
>    format, just like the badge in the top right corner :
>    buildtimetrend/python-lib#39
>    <https://github.com/buildtimetrend/python-lib/issues/39>
>    - travis buildstage labels (script.1) : as far as I know, the Travis
>    stage names can't be customized. But there are other possibilities. The
>    chart could show the executed command (fe. './scripts/ci/build_and_test.sh
>    ${MODE}' in case of 'script.1'), is that something that could work for you?
>    Another possibility, is to generate custom timestamps with labels during
>    the build (that's how Buildtimetrend recorded timing data before Travis
>    started adding timestamps to their logs), but you'll have to add those to
>    your .travis.yml file and the script that parses the logfile has to pick
>    them up (currently not supported).
>
> I think I'd like to just emit lines into the build log in some format you
recognize, and have these shown instead/in addition to the times that
travis writes. I imagine I would need to write startYYY/stopYYY pairs so
you know the duration, you can't assume that a particular build step has
finished the same moment the next one starts.

>
>
> Currently the charts show averages of builds during a timeslot (per hour
> on the 'day' timeframe, per day on the week and month timeframe), so
> digging into specific builds is not possible at the moment, but definitely
> something I'd like to add.
> Would a chart with maximum times be useful?
>
Actually the easier thing for me is, I know the dart stable build is the
slowest, so if I could just select a single matrix build dimension at the
top, and all the graphs narrowed to show just that build, it would be good.

> BTW : auto-refreshing the dashboard is almost finished.
>
> I hope Buildtime Trend, with some improvements, can be useful for you to
> find the culprit for the increased build time. Feedback and new ideas like
> your use case, will make Buildtime Trend a better tool.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/30bd2a3fc72f8f1cdb0ffc4b9d11b4ca42e9ece7#commitcomment-12468653>
> .
>


At this point `d` just stands for denominator of the expression I'm computing.  I can grudgingly spare a few more characters... ;-)


i would have made these `def descriptionFunc: String = buildConditionString(...`


and what about future seraphim coop mission?


It's 10 MB right? Again, this is just an interface for different tasks to report small updates they have from doing their job. For example,
Consider an iteration of kmeans where everyone is working on finding updates for their part of the data. They'll end up sending one vector to everyone. With 10 MB, we can easily deliver a ~1 million dimensional vector.
IMO This interface won't be used for something like broadcast variables, which can be whole data sets, even large ones.


Here you can enqueue an unlimited number of messages.


This comment:
````
// fee = fee * baseFee * feeFactor / (referenceFeeUnits * lftNormalFee);
````
was intended to tell the entire story of what is happening.  Obviously I failed...


Fixes #72


Good catch


Would WebAssembly get any kind of "weak handle" primitive? I.e., I want to use this object so long as it's available, but not pin it in memory? I'm guessing the answer is "no" because JS doesn't have it. But it sure seems handy...  other languages that might target WebAssembly have it (e.g. Java weak references), and it would be hard to emulate.


we have to include the svg in the csproj, otherwise it won't be in the build output


so you are not redefining the variable in the same block then :)


@mxck @okcoker Can you make a PR?


Yes. This wasn't meant as a unit test. I only verify the result by looking at them. It can easily be converted into a unit test.


you are a bad link


Looks like some of the default reactive table styles are not being applied to the table navigation. It's strange, I'm not sure why.


I'm not sure of a nice way to do this without adding several extra instructions here (I don't think there is something you can just mask `RBX` with that will work in all cases). Furthermore, doing this will probably just cover up bugs in other places (for the same reason, we have not done this in the interpreter, either).


thought


After I implement my own version of a queue to handle several consumers, there will be a restriction on the number of messages in the queue.


I wasn't sure how to check for the error type, so I ended up committing this part like that so I could continue working on it from home. From what I've gathered, the idiom to do this kind of assertion should be something like this:

```
if e, ok := err.(*strconv.NumError); ok && e.Err == strconv.ErrRange {
    return Decimal{}, fmt.Errorf("can't convert %s to decimal: fractional part too long", value)
}
```


@czchen 我只是很意外有人會設到 8 這麼大…


Ceilometer has grown a lot since we started it 3 years ago. It has evolved...


If you're going the scipy route, this seems more parsimonious: http://stackoverflow.com/a/26938166

Is this what you're trying to do?

It would be very useful to add a couple unit tests for expected behavior of these functions...


Controlling the transmission stuff is giving me a lot of trouble...
If the r0 is set to 5...the population is stable at around n= 500....anything below drives to extinction...
Not only that, but the "rebound" phases always seem to overshoot and the decline phase seems over drastic....


we have too many files like tests files that don't respect flake8, so we
have a list of file not to tests. We have a test that will teste exactly
what we want. But this test need Theano to be installed... If I fail to
make it install, I'll change the stript to not import theano.

On Fri, Jul 31, 2015 at 10:49 AM, Bart van Merriënboer <
notifications@github.com> wrote:

> @nouiz <https://github.com/nouiz>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nouiz/Theano/commit/9937e0b7538808a772c5bfb618c1fff9bc80f4e2#commitcomment-12468009>
> .
>


s/asked/asked for/


Do you discuss anywhere the mapping from, say, a POSIX thread to a WebAssembly context? (I'm assuming a WebAssembly context is a lot like a JS context). My naive assumption is that it's a one-to-one mapping, which I think implies that an opaque reference would only be usable in a single POSIX thread in some language that's using those. Also, most APIs would only be usable in one POSIX thread.

This is probably manageable and maybe even the best choice, but it would be nice to point out this limitation...  it may be surprising to some developers.


Ah. Yes. This is a problem. Again, this will be solved by writing a queue from scratch. We'll be storing head pointers for all tasks as a HashMap(index of task, head pointer in the queue). This won't be an issue.


How do I remove these locally? When I:
```
git checkout master && git pull
```
on sources/arm, I get a:
```
Entering 'sources/arm'
Already on 'master'
Your branch is up-to-date with 'origin/master'.
```

but all of the removed apks still exist...


The above paragraph is grammatically a bit off but I'm not sure of the right way to fix it. It's not bad enough to be a significant problem.


Thinking of defaults, it seems like {1:1} might be a less random choice... COI stabilizes quite quickly during burnin I think anyways, right?


I'd have to check to be sure.  But my understanding is that dmtcp_checkpoint() causes a 'c' request to be sent to the coordinator.  Possibly, it's a 'bc' request (blocking checkpoint).  But either way, the request is sent to the coordinator, and the function then returns.  To make it block while waiting for the checkpoint to happen would require extra coding.  This is another motivation for fixing the bug in the function `dmtcp_get_generation()` in pull request #155.


Will do.


What if multiple tasks of the same type are running on the same TM. Then only one task will receive a specific broadcast message, because the message is directly dequeued from the queue.


This revision is clean.


It looks like the voting links are dead, or at least they are not working for me right now.


I Can't get Mirage Jubbah in inventory or somenthing, when i farming in Dynamis and the item was dropped it said i can't get because no ones can get more then one some item...


As a first prototype, this commit seems to be a good start. I think, you will get some problems when different tasks want to broadcast messages and when the messages have different types. Also, I think there is a problem when multiple tasks want to receive a message. Then only one task will get it.

I'm not so sure about the future scalability, because akka is actually not intended for large data transfers. Furthermore, there is the problem of the heap memory. By keeping all messages in the `MessageQueue`, sooner or later you'll get an `OutOfMemoryException`.


Thank you!

I'm just the fuzz hanging on the end of the string around here, However I think it would behoove the TagLib community if pull requests were made and allowed some time to be reviewed before committed.


Red Mage


Which is why, messages are right now duplicated for every running task. They're only dequeued from their respective queues.


ssdfdsfsfdsf


Blue mage


Sand'oria


TeamCity YaccConstructor :: YC.Core General Build [Build 479](http://teamcity.codebetter.com/viewLog.html?buildId=196360&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Building ..\src\YC.SDK.sln failed with exitcode 1. Build time: 00:13:12


Dimma


Hola, estimado amigo, espero que te encuentres bien, estuve viendo tu sistema, pero tengo problemas para poder correrlo, quisiera que me ayudes o digas con que version estable de PHP puedo ejecutarlo?, estoy usando la ultima version de WampServer, por favor si podrias actualizar para poder verlo, y me pasas la clave del Webmaster para ver si puedo hacerlo correr, espero tu respuesta y agradezco la labor y tiempo que tomaste en este proyecto.


s/biggest/bigger/


> Das merken die NIE!1!


Just initialize a population of monogenomics? Yea, that sounds less random then setting an arbitrary cut-off somewhere...

I think COI stabilizes pretty quickly ( or at least...the fraction of polygenomic infections takes off immediately and then stabilizes, with very little influence from the initial COI)


Yep, just realized that. #d9312849dd38cf25c58ff7d1f7cec5d183266305


This has the output file from the GenomeReport listener.  It will only be in that directory if one has run for example run_simulation.py from the examples folder.


I agree.  But let's do that as a separate pull request.  I'd like this commit to be for comments only.  This way, if anything breaks on this commit, we will know that it was simply a mistake in trying to move declarations into a different order.


Strings are not Errors, please do new Error('There is already an existing serverBridge instance').  Also nit, use ' instead of "


Yes but then you always have to type cast right? And how do you distinguish messages of the same type but which are intended for a different task?


l really like this. I think the lessons learned stuff is huge and I hope other projects will catch on.


Yes, that was basically what i wanted to do. I just chose alpha/beta parameters so that it resembled the COI distribution that Dan N's paper reported.


Yup, that's why I created the issue :-).


@karya0:  I've replied to the inline comments now.  I propose that I handle the "reformat please" request, and we defer other issues (involving the code itself) for a different issue/pull request.  Do you agree that I can do the reformat, and then push this in?  Thanks.


Why prefer ' to "?


I think this assumptions holds hardly true in reality. Especially if you have skewed data, you easily end up with stragglers.


@jeremyeder, is the --debug switch supported on all versions we have in the field now?  Or do we need to modify pbench to require a certain version?


TeamCity YaccConstructor :: YC.Core General Build [Build 480](http://teamcity.codebetter.com/viewLog.html?buildId=196361&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (9).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\work\4138ce7c6332a78c\tools\Build.Tools\Core.fsx:line 1... Build time: 00:17:03
### Failed tests
```
RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Calc test - simple for translator: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Counter test - simple for translator: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.If Then Else: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Parse empty string: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Resolvers: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Translate with Attributes: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.Calc no operand in the end: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.Primitive translate: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.PrintErrorInfo: <no details avaliable>

```


True, sorry I missed that.


Yes. It's not a perfect solution but it'll do the trick for small stuff. I started out thinking of the async iterations application but ended up making this about essentially implementing a runtime communication system between task managers. Right now, they can only talk to the Job Manager.
Do you think this can lead to better job metrics and statistics? I mean, these are available at runtime, to the user, from everyone. User can access how many records every parallel instance is processing, individual times for each instance, etc. Effectively, these can function as Accumulators which are available right when you want them. [Maybe I'm getting too excited.]


I don't understand what this is attempting to do.  This seems to violate the interface of the vectorial_capacity_fn as an arbitrary function of one variable, time.


Side note:
I wrote up some thoughts a few months ago before I moved on from working on the Pepper plugin APIs in Chromium:
https://docs.google.com/document/d/1JxaOM4wMuCZwZygVSYdMojn3xVGrsJj4i0p2v4k3wHw/view
Not a formal doc, but I think I was thinking along similar lines to you (except I'm ignorant and hand-wavy ;-) ); you may find it interesting.


>
> @karya0 <https://github.com/karya0>: I've replied to the inline comments
> now. I propose that I handle the "reformat please" request, and we defer
> other issues (involving the code itself) for a different issue/pull
> request. Do you agree that I can do the reformat, and then push this in?
> Thanks.
>
Sound good.


yes, it's supported on all the versions we've shipped, no matter rhel6 or 7.  i have a bz filed to revert the upstream patch that did this, and an email thread on linux-pm about it too...that's mostly for customers though.  --debug actually captures a lot of useful info that we've been missing, so for pbench and us internally, it makes sense to get used to --debug.


Convention that I always follow from the Google JavaScript coding standards http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml#Strings


I propose to use na editor which do it automatic.


Ah, alright. But this will easily blow up your memory consumption and thus is a problem for scalability.


Catogories


Yes, type cast.
Every message a task has received will be available through fetch, one by one. If something is not of interest, say because another task of a different type sent it, we can just drop it. [I'm assuming it will fail to cast as any different type. Will that hold true for data transmitted over network?]


Addresses #14 and #15


issue #43


Done.


Ah ok. I guess for the future this should  be automated.


Seem reasonable enough. Please consider this PR as review comments so take or leave as necessary. Right now I'm trying to pick up install off the floor. :)


Hmm. I added that because the original model's reproductive rate calculation had incorporated the current COI.

reproduction_rate = initial_r0 * r0_params['expire_rate'] * max(0, 1.0 - mean_COI/5.0) * seasonal_forcing(t)


Updated.


hile `memcpy()` entered the codebase in a few places, the ZFS codebase largely uses `bcopy()`. We should use `bcopy()` for consistency.


Yes. Sorry.


There is no functionality at the moment. It can also change over time depending on how fast each task produces results for the succeeding tasks. Once a task has produces consumable data, its consumer is started.


@Gaffey 你看一看，弄了一个最基本的接口，控制算法如果不在 python 里的话到时候在 Strategy 外面包一层就好了

车子的参数是直接弄了一个字典 = =

都很粗糙，你先看看这个结构可不可以↖(￣▽￣")


A slot is more like a logical placeholder for a parallel pipeline.


Yes. I'll work out a solution for removing the redundancy here. I already figured out how [I hope so.]. Just need to implement it. Will be finished by tomorrow.


Ahh... ok, so I think we will probably want to address that rather in something like https://github.com/InstituteforDiseaseModeling/genepi/blob/master/genepi/infection.py#L135 where we limit the total number of strains someone can have in them at once... or introduce a more explicit concept of immunity as we have in the DTK.


welcome ^^


Thanks. I would like it to be correct, so I fixed it. Please start by working thru this totorial on your own:
https://guides.github.com/activities/hello-world/
------------------------------
then read this:
https://guides.github.com/introduction/flow/index.html
-------------------------------
you can always look here:
https://help.github.com/

I will post the original code, then, please create your own fork.


Ah. I guess we could send messages keyed with task names then. RuntimeContext will provide that.


well...since i found spell that links player and veh.. .. whole quest is completed and works perfectly fine...
also...spell 45877 has diff target for each of all 3 effects...
and for us is no diff... who will be set in spell_script_target becouse of buddy system in DBScripts ;)
pls add ACID part and all will work perfect ;)


@ldoktor OK, I've updated inspektor, indeed it was my mistake of not updating the version on pypi. I'll replace your PR with a PR updating inspektor.


Apparently Atom strips trailing whitespace by default, which explains all the non-obvious changes. Sorry about that.


93469234-DemandResponse2 (Myoldmopar) - i386-Windows-7-VisualStudio-12: 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)


Yes. But I don't think passing the instanceManager to TaskManager is a good solution.
What if we were to send a response of HeartBeats back to TMs with current active managers?


Use errors.New when there's no format string involved.


If it's not the type you casted it to, then you'll get a `ClassCastException`.


One minor nit but LGTM.


I think that fixed the issue. Not able to reproduce it now.


I don't understand why this is here, isn't the default behavior to have no expected exception on tests? Can you make a comment about how this changes the behavior?


Very minor; I might make these three lines an inline function, since it's duplicated exactly just above. It really makes no difference; it's just for clarity. It also _might_ help prevent a subtle bug in the future if someone touches one and forgets the other.


Nice touch :lipstick:


Estimado, podrías habilitarme la DB, para poder probar por favor, espero tu respuesta.


Yes, or we can make this explicit where dhCommitMessage is called.


which is very well the clear indicator of that message not being meant for you.
[In another of my comment, I mentioned it would make sense to include the task name in the message. Then this problem will be solved. Only messages where task names match will be returned.]


For the future, it would be great if we could generalize it.


Why is there a bare `except` here?


changed your mind on this one :)


the task name does not necessarily be distinct. Some kind of ID would be.


This is a temp version tries to solve the issue mentioned in https://github.com/GiraffaFS/giraffa/issues/184
There are three types of failures:
  1. ```Assert Null failure``` from ```testLeaseRecovery```
  2. ```Assert CLOSED failure``` from ```testLeaseRecovery```
  3. ```Lease migration failure```

This patch can
  1. Solve case 1 completely (Pass more than 20000 times simulation)
  2. Solve case 2 when we run ```testLeaseRecovery``` only (Pass more than 20000 times simulation)

But can not
  1. Solve case 3
  2. Solve case 2 if we runs ```testLeaseRecovery``` and ```testLeaseMigration``` together


That's good.


What happens when the task is finished but has not consumed all of its messages? Then they will be simply kept, which means they are blocking memory.


It's not silencing anything. If there's any exception, delete the tempdir and reraise. If we're throwing an exception the caller won't get the chance to delete or access the tempdir


First xd


Recently, a PR has been merged which added support for live accumulators. Do you mean something like that?


this isnt working #2


Ah. One task means one Vertex in the Job Graph, right? [I maybe terribly wrong.] We could use that as an id maybe?


Why?


@eahrold - Are these quotes necessary? Does it matter whether they're `"` or `"`?


C'est beau ?


@mjackson done. :)


This logic seems strange to me. It seems to assume that any `required_by_rlib` crate is an allocator. If that's the case, can you rename `required_by_rlib` to something about allocators?


Maybe using `logger.debug` would be better than using `System.out.println()`.
What do you think? ( and sorry for not mentioned before )


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-umd/dev/611cb3f60ae2c7a9210db8a95859e592d21ba253/app.js)


Part of my issue is that with this implementation, should I choose not to install pymongo, the module will raise an ImportError and entirely prevent the webui from loading, I would suggest the default implementation just return an empty list from both, or just list itself as unimplemented and hide the ui components


No need for apologies. I know that this is WIP and that you're actively working on it. Having good tests allows you to spot early errors in your code which would be later hard and costly to repair.


 lewischeng-ms 你好，首先很感谢你贡献的源码，
我尝试在ubuntu下编译你的源码，运行生成的uhttpd文件时，出现如下
Sat Aug  1 01:51:31 2015
Info: main(): About Server:
Name: uHttpd
PID: 13742
Address: 0.0.0.0
Port: 80

Sat Aug  1 01:51:31 2015
Fatal: start_server(): Fail to bind server socket

这是什么原因呢？谢谢


Indeed, I hadn't considered using it as a function name, I'll have to live with verzion I guess :)


Oh, I didn't notice that you already know about that, so I created #4454


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-umd/2.1.0/app.js)


Yes, in terms of the literal meaning. I didn't actually find out what new functionality has been added. Only the test AccumulatorLiveITCase, which really wasn't quite clear. What I understood was that now the accumulators of one task will be made available in the next task. This was also somewhat I figured out by the relevant part in RegularPactTask code where Accumulators are collected in the next chained driver.


Hmmm.... I want to see what will happen if we don't limit it...
I'll remove the limitation and see how it behaves.


Yeah. :)


A task corresponds to a `ExecutionVertex`. And n `ExecutionVertices` correspond to the parallel representation of a `JobVertex`.


I think if you make the selector return `INumberType` it can be shortened to `AddProperty()` :boom:


Why does `C -prefer-dynamic` mean 'Rust dylib'? I'm worried that this is a brittle assumption.


Ah. Umm, yes. I'll think of how to deal with this. Perhaps some kind of lease or something. Or maybe on receiving a change in job execution state, the task manager can clear that out itself.


why I can't see your code?


So, a JobVertex id and the index of subtask should suffice to uniquely determine every running instance.


trop <3


xd gg lel


Yes but it puts a lot of constraints on the user to get things right. From the user perspective it should be as easy as possible and you shouldn't be able to shoot yourself in the foot. Of course this is not always possible but type casting is usually a good source for problems.


Great suggestion.  I'll make the change as I implement the rest of them.


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


No {} around single-line if blocks, please!


Yes, you are totally right. I created a pull request #640. Please check it if you can.


They basically allow you to get immediate results of your accumulators not only after the jobs has been finished. There are also some metrics implemented which use them.


In the context of API, what is 1 and 2? Node IDs?


the use of this.menu_margin in this case accounts for a discrepancy I was noticing. Without it the menu default behaviour would still get partially hidden off the bottom of the screen before triggering the if statement to correct it.


Здесь всё хорошо, только одно замечание:
```
if((...)==(...)) {
   return true;
} else {
   return false;
}```
это всё можно записать одной строкой ))


What was the scenario where the 2nd ckpt was getting triggered?


Jumping in this new: so I'm assuming this is an example of how a developer would build a custom plugin, yes? (Are we calling these plugins?)


Yeah. I actually looked into Apache Ignite. Reading the documentation, it does appear that they do have a good solution, which ensures consistency, and since it is distributed, there'll be not too many worries of running out of space. They also offer a solution where there is no need for explicitly specifying what all task managers are. It can automatically discover all ignite servers and clients.
After finishing this work up a bit to address your comments, I'll try integrating that too.


Was this meant to make it?


Yup! The requirement for an API trigger function is to return a node id


@kyuupichan correct me ifI'm wrong here


Hola, estimado amiga, podría facilitarme la BD por favor para poder ver su sistema, esta interesante por favor, espero que pueda enviarme a jpomaca@gmail.com gracias, espero su respuesta


а здесь тоже все ифы можно свернуть в одно выражение...


yea, atom removes it when it's a default value and i was tired of skipping it :)


jekyll comment in wrong place


Olrite. I'll look into how I can get rid of typecasting. At least on the user end.


reorder?


the =x is redundant


Yep, this is good.


Здесь всё хорошо, молодец


@stepanstipl I am trying to get a 3.0 release of the module out today, can you submit a ticket on why this commit was needed for puppet 4?


You mean, merged accumulators from all tasks? While working in the same task? I figured it would be in the next task, which still is an improvement from only making them available after all tasks, i.e. job finish.


"one"


Excellent :)


Hi!
Instead of iterate over all characters and use condition check on every character, I think is better to use a simple copy to move whole line to the return variable, and test only the last 1/2 "character" of the string.


@Dimmabr **This is the wrong place to try and tell anyone about it. **
https://github.com/LegionDark/Issues/issues/new
is where you would report a bug.

Please make sure its not simply a matter of the sort bug happening (sort bug = where the game acts as if you have a full inventory when your client shows you have plenty of free space, till you hit sort and then suddenly it goes normal). The sort bug is also responsible for "invisible items" and "missing items" that pear in your inventory after you sort as well. Since the item you mention has been working for years it is most likely you have an invisible one in your inventory already.


I did that because I needed to log the value to console and just left it in if we need to do that again. If not, I'll just take it out.


> This commit disallows that.

:+1:

> You can still set unknown keys inside of object schemas though.

I remember we needed this because many packages still werent using schemas yet, so they needed to be able to set schema-less options.


:thumbsup:


честно говоря, я удивился, что это вообще заработало...


@eddelbuettel Great to use read.dcf(). Note that I had commented out the strsplit() in my earlier commit, as it was no longer needed to do the simple grepl on line 226/227. So, you could probably eliminate the new line 215 strsplit and revert line 227 back to work on built_string directly, just for simplicity.  The only reason I had split them previously was to simplify regex parsing of the version number, which I removed anyways.


commit comment


Made for handling new CSS classes until they can be put in a proper home


why take it from shinano?


With the iteration modification:

if (from == NULL){
return L"";
} else {
std::wstring to(from);
}


1. по заданию массив уже отсортирован, сортировать его не надо
2. поиск есть, но он не оптимален


Yeah. But one can hope. :'). You're right though. With the user unaware of the limits, it'd be a disaster to drop messages. Throwing an exception at least lets them know something is wrong, and maybe heap size can be increased.


Всё остальное - хорошо, молодец


5058-fix-readvars-cli (nealkruis) - i386-Windows-7-VisualStudio-12: OK (1717 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-96.57%25-yellow.svg)


Fixed in 1819817620f88af47aa8d70eec7627400fb87198.


I would have that was a major...


90946736_SHRgt1_4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1723 of 1779 tests passed)

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


@gonsfx I write my own project and asked @gaearon how reduce a lot of `switch/cases`. He advised me [redux-actions](https://github.com/acdlite/redux-actions) and I showed him this diff with gratitude.

And `handleActions()` is really very handy!


Ahh, just saw `api.html` in its full context. :+1: If I'm a new dev picking this up for the first time (not a stretch) then that's a great way to understand quickly how an external API works.

Stupid content nit pick, rename `api.html` to like `sample-api.html` or make reference to the string "custom plugin" so developers (internal or otherwise) can quickly pick up how it works.


Y'all are freaking geniuses. Genii.


Is it missing a comma sign?


> I remember we needed this because many packages still werent using schemas yet, so they needed to be able to set schema-less options.

Yeah, I figured I shouldn't mess with that. It also matches the default behavior of JSON schemas, where unknown object keys are allowed by default (you have to set `additionalProperties: false` if you want to disallow them).


Hi @acidprime, cause Puppet 4 open source also comes with own ruby and related in '/opt/puppetlabs' (https://docs.puppetlabs.com/puppet/4.0/reference/release_notes.html#all-in-one-packaging). So in my case I don't even have system ruby & gem installed and trying to use this module would fail in complain about unknown provider 'gem'. Also the paths changed. This is just a quick fix and I didn't really have chance to test it properly. I can try to verify the functionality and add some spec tests.


I wasn't thinking about the fact you can access an AR object's attributes with object[:attribute], but realistically we are build a hash of attributes coming from a form submission, and we are not passing an AR object to the Setup model.


> Before comparing the contents of the files, I should string(REPLACE ...) the arch-specific install directory, with the final install destination directory, as there are files, like *.la created by libtool, that have in their contents the absolute path of the directory there were originally installed in.

Where this files are used? Can this value be empty or relative path (relative to what?). It's important for caching since same cache can be reused in different locations.


I looked into that this morning before rushing to work, and I think we need to do more work.   Eg the R version can be parsed, and I want to double-check the differences between binary packages and all-R packages like drat itself.  Field two empty for the .tgz I tried.


5058-fix-readvars-cli (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1778 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-100.0%25-green.svg)


end of sentence missing


p.s. this was gd stupid of you. Just so you know.


perfect.


no, i had tested in my app.
but i did now (with phantom) and no difference


Whoops. Should be "consider adding oneofs to unittest_proto3.proto instead".
Given the number of PRs in flight, it might be easier to fix that up later, but I can do it now if we want.


I did the whole setup for all 14 devices at once... And heh, sorry, this was a mistake!


It extracts filename from full path and returns it as std::tr2::sys::path object.
string() method then used to convert it to std::string representation.


@technobly, would it be better to initialize the pins as input_pulldown?

See below:
http://electronics.stackexchange.com/questions/50539/unused-mcu-pins-better-to-tie-to-gnd-or-use-software-to-prevent-floating


I agree with @Kakifrucht's suggestion for implementation, that seems like the best way to go about it.


Fine to add later.


Sorry, I edited that last post after creating it, so you probably didn't see the up date.

I meant [further up](https://github.com/mitodl/lore/pull/427#discussion-diff-36000991).


Спасибо.
Я забыла только остальные задания свои залить:)

пятница, 31 июля 2015 г. пользователь brus-fabrika написал:

> Здесь всё хорошо, молодец
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OlgaGrygoryan/homework/commit/0ea99569ac0bd0184da96ac859eb55a1d1266834#commitcomment-12472074>
> .
>


97902934_ZoneMixing (macumber) - Win64-Windows-7-VisualStudio-12: OK (2214 of 2236 tests passed)

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


Consider making this an 'else if'


It's actually [the other way around](https://github.com/alexcrichton/rust/commit/5dbf9bebb9cff08393b80079424e411eaec36e8c#diff-40c1e07fe1fc5308791be61edc1c18c8R64) in the sense that crates which are indicated as `required_by_rlib` are *not* implicit allocators. It basically means that the crate was linked in explicitly via `extern crate` at some point.

This flag is false for injected allocator (hence the [`false` below](https://github.com/alexcrichton/rust/commit/5dbf9bebb9cff08393b80079424e411eaec36e8c#diff-e5a82d974db2b4391946b2f73eb68613R667)).


Sorry, i've forgotten to remove it.


Yes, or least log something when hitting the limit. I had no way of knowing what was the issue.

When switching to 0.2.1 projects will have to regenerate the log from the beginning, so a largish `maxBuffer` will be useful.


This is [already the current interpretation](https://github.com/rust-lang/rust/blob/0919f4ad862bc3ae771fc56a8fe9905bca299fe2/src/librustc/middle/dependency_format.rs#L126-L128) of this flag, so this is just an extension of that.


@paramat


@buddlflink
> Problem zur Konsistenzerhaltung des Modells in den beiden Ansichten
> sachlich beschreiben. Z. B. keine Fragen an den Leser stellen, sondern
> das Problem schildern und wie damit umgegangen werden kann:
> Z. B.: "Bei der Implementierung muss sichergestellt werden, dass selbst
> nach einem Syncfehler zwischen TreeView und 3D View Änderungen eines
> Datenmodells in das jeweils andere übertragen werden und so wieder eine
> konsistente Datenbasis hergestellt wird."

Also soll ich das unter requirements dazu nehmen und den refactor weg rausnehmen und unten nur das ergebnis hinpacken?


Ok, sorry 2 times, i will double check it the next time.


:P sorry for this ad-hoc comment on a commit some 2 months ago. I pressed comment before re-thinking it. But you have some experience with that from me hehe

Was looking for a different custom then aosp / cm / omni and it came to my mind I liked liquid ages ago on my first samsung. This is nice, instead of doing this from scratch I can take it from here :) thanx


```knowRecipes = typesof(/datum/recipe) - /datum/recipe/foodprep/datum/recipe```


I had it that way originally, and it made the spacing a bit odd. But I guess I can move the comments to inside the if blocks and it should look fine.


@dcbw
For (1) 'sameMinionRestriction' is not an existing kind in the node policy. sameMinionRestriction term was used as an example in the documentation.
(2) & (3) vagrant is intended for development purpose and I don't think we need to maintain backward compatibility here. Existing vagrant setup/cluster won't work and we need to recommend the user to recreate their vagrant cluster.
I was planning to send an email to openshift-dev about this change once this PR is in merge queue.


are we generating an actual secret somehow and is this tied into `kato op regenerate` and firstboot?


any reason these tests were removed?


It seems you didn't change mkmake.lua. The changes to the platform-specific makefiles you be overridden the next time mkmake.lua is run.


Now it seems to work. Proper logic at least (if respondr set to true, then load normal theme, otherwise, load mobile). Same view everywhere.


@dcbw
Yes, running openshift-sdn in standalone mode will not solve the current problem without this change. I was not sure what was neeeded here, I got inputs from Rajat today. I will update the PR with the needed change.


It's worth noting that this doesn't actually work, at least not how it's supposed to work, which is why I didn't add it to the test. dnf (and yum) take includepkgs to mean that everything else from the repo should be excluded, while we want it to just prefer the repo for that particular list of packages. But things have been that way for a good while so I figured for now it's good enough to just put things back into the same broken state we had with yumpayload.


mds->server->_check_access(in, MAY_WRITE, 0, 0, 0, 0);


these 2 lines should be one line, } else {


just 'in', not 'CInode *in'

there is no mdr, use the arguments!


No testLedger.info().seq < validIndex case?


nice concurrent tests


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1259 of 1259 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)


imho that change is a bit too specific to your usecase. no apache, no .htaccess for example. and the favicon path is rather arbitrary. not hating, just saying. ;)


r+


sounds good

On Fri, Jul 31, 2015 at 11:13 AM Aaron Jensen <notifications@github.com>
wrote:

> yea, atom removes it when it's a default value and i was tired of skipping
> it :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/substantial/atomfiles/commit/2ecd1b03ec2509fab91adf6d5d463b766e7e0cd6#commitcomment-12472030>
> .
>


@buddlflink
> keine Fragen an den Leser stellen

Sind rhetorische Fragen verpönt?

Zugegeben 3 is bissel heftig, werde das definitiv umschreiben. Aber ich glaube, dass einzelne rhetorische Fragen den lesefluss positiv beeinflussen können, den Leser mehr einspannen und einzelne wichtige Aspekte betonen können. Unter gewissen Vorraussetzungen natürlich, zum Beispiel, dass die Antwort offensichtlich ist.
Ziel ist es dann nur, dass der Leser kurz inne hält und sich fragt ob dem denn wirklich so ist? Stimmt die Aussage des Autors? :arrow_left: Oh da sind'se schon wieder, die bösen rhetorischen Fragen :smirk:
Aneinanderreihungen schierer Fakten fehlt es an Akzentuierungen und man sieht schnell den Wald vor lauter Bäumen nicht mehr.

Ich kann die auch weglassen, aber ne Erklärung die weiter geht als "Das macht man nicht" würde ich mir dann schon wünschen :smiley:

Ich finde eine langweilige und trockene Sprache sollte nicht teil der Definition eins Papers sein.


this should have been the reverse.

this will require changing if you go further than 1.7.


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


merging graydon/stellar-core/inhibit-loadgen-when-desynced = 6963f91e into auto


graydon/stellar-core/inhibit-loadgen-when-desynced = 6963f91e merged ok, testing candidate = 906e5443


> Can you point me to the relevant documentation for  Hashtable  that describes the reader/writer behavior?

See the Thread Safety section of:
https://msdn.microsoft.com/en-us/library/system.collections.hashtable(v=vs.110).aspx

> all operations on  _domainTable  (other than the replacement on line 379) are performed under the same lock

Great, that's what it seemed from a quick perusal. Dictionary is good, then.


here _check_access will run respond_to_request(mdr, -EACCES)

how to make check_access run it??


It's more meant as a workaround for not having `make depend` (read: for OSX). Almost all OS-es have ` make depend`  and therefore will not need `which`.


That makes sense.  I'll fix it.


if prevHash is zero, wouldn't we want to invalidate it?


It still requires FML though. And it didn't fully adopt gradle until 1.7.


I don't believe this is the case (or at least not any more)... @jkotas?


What's this?


Pushed fixes with commit 67b7d6fa31f4426f18d223faa06c7b1f3c59fa97


I agree it makes sense to have the two different log statements, I was just referring to the pluralization logic within each of them.  It seems like unnecessary complexity, but if you find value in it, I'm fine with it.


oh i take it back, we do have mdr here, nevermind


oh - I thought they employed gradle, for 1.6


全角スペース　一括置換で
 src/common/CommRoutine.cpp
を変更している


Ah, it should be legoPort. Thanks for catching!


Optional style suggestion:
````
mLastValidLedger = {hash, seq};
````


OK, that makes more sense :wink:


tak, jesteś gejem


@kevinushey Could you review this? Just want to make sure I'm preserving all of the pre-existing semantics timing that you had in DocumentOutlineWidget


ack


+1 for being paranoid!


Yeah, and I'm actually going to revert this to the original behavior.

The unsigned shift right is used fairly commonly to [convert to a 32-bit integer](http://www.2ality.com/2012/02/js-integers.html), and is equivalent to the [ToInt32 type conversion](http://ecma-international.org/ecma-262/6.0/#sec-toint32) in the spec. But in this case, the spec calls specifically for the [ToLength](http://ecma-international.org/ecma-262/6.0/#sec-tolength) operation, which is slightly different. This is simpler, but less correct than the [original toLength() implementation](https://github.com/studio-b12/array-from/commit/deebb108d31dec63e21285d9a30b479ba0e91611#diff-168726dbe96b3ce427e7fedce31bb0bcR16).


Agreed. I'll change to prefer `Symbol.iterator` if it exists.


I've forced-pushed the fix.


93469234-DemandResponse2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1672 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-93.46%25-yellow.svg)


Either work, I've never figured out why it bounces back and forth between "" and &quote; It's Xcode evil minds.


I said fully. They had builds for it, but they didn't officially make the switch until 1.7.

That's what I know. I didn't start my modding career until 1.7