undefined should probably be undef since that's the actual value it's set to


undef


1: M110 missing component name


Hi, thank you. What browser were you using?


:heart:


Hi Niq,

On Tue, Mar 3, 2015 at 8:23 PM, sublimator <notifications@github.com> wrote:

> Hey, you know we tested that the correct transaction engine result etc was
> returned in the npm tests for tickets, but failed to test that the ledger
> entries were correct. The wrong account's owner count was decremented.
> There's a lesson there.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scottschurr/rippled/commit/a3239a196a089dcef15f1b41e256a177c2b15257#commitcomment-10021975>
> .
>
You make an excellent point.  So it seems, for these to be robust unit
tests, I need to check the ending state of each ledger -- at least on those
accounts I'm focussing on affecting.  That seems like the right thing to
do.  I'll work on figuring out how to do that.

Scott


I tried to include the server factory as part of the project itself but I got the same result.


It seems the rules are:
* If `suite` is included there can be a single. or no, `dir`
* If `case` or `group` are included there must be one `suite`

It would seem we can get around both issues by checking for `suite` and if it is present not including the `dir` option.

Regarding #215 I will send a PR to OTP to fix the `function_clause` error.

Now I have dug into it I can pick this up, thank you for the help.


"a FS listener".

@mitio duh


@djazayeri do you see any issues with this?


I think we should error out here rather than returning 0.


you can probably simplify this via just `plugin.stub(:shell_out).with("whatever").and_return(stuff)`


0x80 - length


Lincomatic.  Please ignore this.  I'm learning GitHub and didn't mean to do this.


Aw, thank you!


@stephanbergen I'm going to take care of this now and get a release out.


A few more:

You kick ass!
 I love what you've done with the code!
Way to go!
You get an A+!


I swear I checked this. Dang. I can't safely do the instanceof check. Can I just not support Safari (just kidding, kinda).


oops, I should have mentioned in the commit message that it adds support for message code as labels as well


+1.   This looks good.  My one suggestion is that you add a check for the validity of the metadata file path before doing anything.  Along the lines of these other checks on the inputs.

The reason is that this script is not very graceful about continuing after a failed attempted that choked in the middle of all of these priming steps...


ok. but for making it as simple as it could be we should't have code been executed over and over again even if it takes no effect to double close the request here. I think what method closes the request may be indicated also by its definition. Maybe readResponse isn't such a good place for this behavior. Perhabs we can have this refactored in the future.


89438532_RadianceBugFix (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


1: M110 missing component name


So now the first colorbar really is getting clipped by the left and right edges of the figure, for all the backends.  Would it be better to reduce the `Axes` width for the test, to keep the entire colorbar on the page, such as the following?:
```python
#ax2 = fig.add_axes([0.05, 0.65, 0.9, 0.1]) # <-- former Axes position coordinates
ax2 = fig.add_axes([0.1, 0.65, 0.8, 0.1])
```


There should probably be an error check to make sure that the displacement variables are provided if you ask for t_stress.


Looking good Greg


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 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-4.06%25-green.svg)


I'll merge it into the master when I get a chance


Maybe we should just do this if ```_t_stress``` is true.


shouldn't this rather be ```_.isError(status)```?


so actually, if you go down that road then `stuff` would need to be a double which responds to stdout, which starts getting very annoying.

i'd setup a test in isolation for `#run_cmd` to make sure that it is calling shell_out correctly and doing the correct stuff with stdout there.

then for the meat of the test i'd just stub `#run_cmd` like `plugin.stub(:run_cmd).with("whatever").and_return("string")` which avoids passing around mixlib-shellout mock objects.


Oops. Hi @performantDave -- for some reason, I thought I was going to see the most recent commits at the top of the PR, so I didn't see these here -- apologies!


Can't we execute on nonlinear rather than linear?


> On Feb 26, 2015, at 1:57 PM, Lloyd Cotten <notifications@github.com> wrote:
>
> Just wondering why this was commented out? This is breaking build for me. Am I missing something else in my environment?
>
>
Lloyd,

Sorry for the later reply - got lost. You need bison 3.x; otherwise, it should build just fine. If you’re still having problems, send me the error along with your build platform info.

James
> —
> Reply to this email directly or view it on GitHub <https://github.com/OPENDAP/libdap4/commit/698cafb81ddf0b360aa3a6317d7d9a307c9bdc3e#commitcomment-9945438>.
>

--
James Gallagher
jgallagher at opendap.org
406.723.8663


Is the ```select``` at the end necessary?  If so, then the test should query for the current filehandle at the start and restore whatever it was at the end
```
my $selected_fh = select;
# tests here
select $selected_fh;
```


> It looks like during this refactor, the # normalize by inverse length to compensate for phase invariance that should've only been applied in the case of the pseudo_cqt is being applied to all versions of the cqt via the refactored __fft_filters function.

Oops, my bad!  I can fix it in develop, or you can fix it in #172 (probably simpler that way).

Maybe add a flag to `__fft_filters(..., pseudo=True)` to trigger the normalization?


I'll tweak the unit test to add one that has a `global` object in the web worker to hit this path.


I've raised https://issues.apache.org/jira/browse/KARAF-3585 to capture the problem.


< serviva per testare il gtkDialog


prefer
```
return _.sortBy(pictures, function(p) {
  Math.abs(p.rating - targetRating);
});
```


Yeah, the whole request/response handling needs a refactoring anyway, it is too far from what HttpClient 4 is expecting, and too fragmented at the moment. For now this should work.


ditto above


Idk if it also affects your hotplug, but it seems neobuddy had some issues with the fb notifier. Have a look here: https://github.com/neobuddy89/shamu/commit/a6564086fbe95c6d480d3b194210ec0b673b2222 and here: https://github.com/neobuddy89/shamu/commit/72795dbeea6372172b6e3c95c813928e5cf7ffa6


Actually, I think it can be a bit simpler -- using `|(p,_)|` as the block parameter works equally well for hashes and arrays (i.e. `[1,2,3].any? { |(p,_)| ...etc... }` yields `p` as `1`, then `2`, then `3` even though there's nothing to be yielded into the second ignored parameter). Strictly speaking `|(p)|` works too, but I think the `,_` makes it clearer that any other yielded values will be ignored...


@jdalton, I know that you would triple test stuff like that. This is actually what I hate most in programming. Hidden global state and unpredictable runtime. Sometimes you have to trust your runtime. E.g. CPU. That he actually computes 2+2 correctly.

IMO `freeGlobal instanceof freeGlobal.WorkerGlobalScope` should be enough


This alters the inputs, this is an absolute no-go. Concatenations must not have side effects on the inputs.

```
// no need to provide a constructor, actually
pointer result = new concatenated_tuple;
auto append = [&](pointer& ptr) {
  auto dptr = dynamic_cast<concatenated_tuple*>(ptr.get());
  if (dptr) {
    auto& vec = dptr->m_messages;
    result->m_messages.insert(result->m_messages.end(), vec.begin(), vec.end());
  } else {
    result->m_messages.push_back(std::move(ptr));
  }
};
append(lhs);
append(rhs);
return result;
```

Btw, why not take an `initializer_list<pointer>`? Would be more flexible.


Looks awesome, thanks for the fix!


The format is definitely solid. I just wonder if, for the kind of throughput-oriented applications we want to support, the overhead of whatever wrapping is going on is a big deal. It'd be good to take a look at benchmarks.

I think so long as we are careful to keep all of the HDF5-specific logic at a level such that it can be swapped out for other, possibly more performant options when warranted or required, we ought to do it.


Correct the comment.

```
# Just in case we get back an ASCII `str` object, convert it to a `unicode` object.
```


Damn, not sure how I missed this PR, sorry =(

Should be fixed in `v0.7.0` now though.


i am not sure if we would like to introduce realtime into etcd's system.


Can you clarify exactly what the above commit does? Does it merely display the first message at the head of the thread (which as Andy says is not needed, since the issue extract is already shown), or does it actually clone the issue as the first message automatically?

The underlying issue we are trying to solve is described at:
https://github.com/cyclestreets/cyclescape/issues/43
namely that users create an issue and assume that the job is done, when they *also* have to create a thread. I think you (Nikolai) suggested that one way to do this is to add the 'Start thread for: ...' & 'With privacy: ...' as two extra fields that get populated dynamically when an issue has been created, or something to prompt this.

I'll copy this message to the issue page, so we can discuss it properly there:
https://github.com/cyclestreets/cyclescape/issues/43


read


```
if (modeMap[r])
```
implies the same thing


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


**Tip**: Use `<string...>` resources to store configuration data. Instead of saving those values as const in the code, save them in the XML's. That will help you in:
- Saving all configuration in one place instead of spread around Java files
- Easily change them between different configurations

For example, this string saves the API address for the network calls:
```xml
<string name="tipz_api_base_url">http://www.tipz.io/api</string>
```
You can then create different build variants to use different URLs (Debug, Test, Production...)


you'll probably next want a block of allows (oops, shouldn't be using rspec 2.x syntax):

```
def setup_stubs
  allow(plugin).to receive(:run_cmd).with("/usr/bin/vmware-toolbox-cmd stat hosttime").and_return("stuff1")
  allow(plugin).to recieve(:run_cmd).with("/usr/bin/vmware-toolbox-cmd stat speed").and_return("stuff2")
  [...]
end
```

you can probably do something with a loop over the different arguments `[ :hosttime, :speed, etc ]` and then have a hash to lookup the return values `retstrings = { hosttime: "stuff1", speed: "stuff2" }`

then in the before block just:

```
  before do
    setup_stubs
  end

  it "sets the right hosttime" do
     expect ...
  end

   it "sets the right speed" do
    expect ...
  end
```


:+ :package:


Should there be spaces in this URL?


This is overly complicated and accumulating a value using side effects is... not very intuitive.

```
auto acc_size = [](size_t tmp, const pointer& val) {
  return tmp + m->size();
};
m_size = std::accumulate(m_messages.begin(), m_messages.end(), size_t{0}, acc_size);
```


Please don't call it str


not a big fan of doing these calculations every time... can't we just construct a modeMap then take the max key instead of checking every time?


s.strip("/").replace("/", ".") is enough.

Or possibly, s.replace("/", ".").strip(".") may be what you are looking for.


Quorum is only used for get request. Put implies quorum.


Is this going to be brittle? and line #43.


> IMO `freeGlobal instanceof freeGlobal.WorkerGlobalScope` should be enough

This will throw an error if `freeGlobal.WorkerGlobalScope` is `undefined` or an object (ignoring Safari's host object oddity). So I'll have to modify the check. Webpack/Browserify are common enough to add a test for it (it's an easy add).


This could be a static member function taking an `initializer_list`. Then one could simply write: `message::concat({a, b, c, d})` instead of `a.concat(b).concat(c).concat(d)`. Also a lot faster.


Yes a bit. If the events fixtures are updated it will no longer pass.


**Tip**: In the Manifest.xml use `${applicationId}` when things might change between build variants.


I just published https://github.com/jonschlinkert/glob-base based on this. We can use some of the [unit tests](https://github.com/jonschlinkert/glob-base/blob/master/test.js) from there if you want.  Those tests show a ton of edge cases that are easy to miss


update your go version?


It's just a nitpick, but the name `m_messages` makes no sense. Probably an oversight during refactoring, but renaming it to something more intuitive would be still a good idea. Even `m_data` would be a better name. :)


1: M110 missing component name


Done.


Is this windows only? What's the code page?
On Mar 4, 2015 2:46 PM, "Marcin Miłkowski" <notifications@github.com> wrote:

> They probably fail due to codepage settings on my machine. But they DO
> fail.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10037286>
> .
>


Hey man, you posted your private key on the internet. You should probably generate a new one and delete this.


LGTM.


don't we need to add io.js too?


Well master is fully updated. No need to use develop unless your a testing a fix


Done. It was left over code from before.


Additional chages in the web application.


**Tip**: Bulk/Transactional inserts are tremendously faster. When inserting multiple values into the DB directly, or when doing that through a ContentProvider, make sure to do it in a transactional manner.


Done. Yes we can add [Service UUIDs](http://developer.android.com/reference/android/bluetooth/le/AdvertiseData.Builder.html#addServiceUuid(android.os.ParcelUuid)) to the packet later.


Does this properly cover the even num of pictures case? ie if my set is 1,1,1,2,2,2 the median is 1.5


Thank you for bringing this awesome feature back.


no


i think this implies type conversion, and thats something that I wanted to avoid

its sort of like 0 == false

js has to type convert to truthy (or falsy) evalues and then compares them.

See http://stackoverflow.com/questions/359494/does-it-matter-which-equals-operator-vs-i-use-in-JavaScript-comparisons


**Tip**: [Avoid Internal Getters/Setters](http://developer.android.com/training/articles/perf-tips.html#GettersSetters). It is discouraged to use Getters/Setters, unless you actually need them. The additional method call in the stack is bad for performance.


Woops! Should come back....


I was thinking about people using error strings in node callbacks:

```javascript
doSomething(function(err) {
  if (err) {
    return context.error(err);
  }
});
```

In know not all things use actual error objects so passing a non-epilogue error or a string will make an EpilogueError with the cause set to whatever was passsed


**Tip**: Clean automated versioning. Polluting the git with versioning info doesn't feel clean for me, that's why we are using [Automatic versioning and increment using Git tags and Gradle](http://blog.android-develop.com/2014/09/automatic-versioning-and-increment.html).


Thanks--please make the commit messages more descriptive though!


Amazing! :)


Probably `finish()` here since the user declined to turn on bluetooth for us, and there's nothing else this app can do.


Checked in to wrong branch.. This closes #213


**Tip**: Use a version numbering scheme - Read the [Using a version code scheme](http://developer.android.com/google/play/publishing/multiple-apks.html#VersionCodes) section.


I think libgcrypt (when included) is in "$(STAGING_DIR)/usr"

I just ran across this in another package.


You will need to add environ-xulrunner to configure xulrunner and geckofx properly.


@Eluinhost gotcha


why do you use glGetUniformLocation instead of glGetProgramResourcesiv


Right. So I've merged this script, but I did change it around. And actually modified ab3.sh to make it work ($ABBLPREFIX).


any need for "and games" here?


Use `@copyright = attributes.fetch(:copyright) { copyright_from_work }`

As I'm thinking about this, we want to use the keyed value if one is given, otherwise default to what we have stored.


Lets go with `:attr_reader`; I'm wanting to revisit forms so that we don't expose the ability to mutate the data.


Also:

Improved: changed some strings

And, at the beginning just after the V 1.5 it should say:
IMPORTANT: A minimum version of Lighting Launcher is required now. It will ask to update otherwise.


`App` is capitalized in this sentence, it isn't in other sentences, not sure which is right.


Switch all instance to use fetch (including abstract); Thanks!


No. I've updated this in #481


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1140 of 1140 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 you know how to trigger this or is this jack a "be safe" fallback?


Does the planned list of mobile websites require one? Its Marketplace Content.


What's the point behind this refactor? I'm failing to see why you thought it necessary.


@calvinatintel ready for cherry-pick


1: M110 missing component name


once #296 gets merged I will be able to remove line 135 and 136 :santa:


This is an interesting solution to this. Although it's essentially telling the caller that the Docker container has gone away, which is not true, so this function no longer does what is implied by its name.


FB notifiers are not friendly if you have doze mode / ambient display in ROM (lollipop). Please check it before release.


Closes #778


Given that discipline can be multi-value, we need a multi-value validator.

Consider the following:

```ruby
object.discipline = []
object.valid? == false

object.discipline = ['']
object.valid? == true
```


Are you kidding me?


Enable or disable ???

here the help text:

CONFIG_SWP_EMULATE:                                                                                ARMv6 architecture deprecates use of the SWP/SWPB instructions.                                                                                │
  │ ARMv7 multiprocessing extensions introduce the ability to disable                                                                                │
  │ these instructions, triggering an undefined instruction exception                                                                                │
  │ when executed. Say Y here to enable software emulation of these                                                                                │
  │ instructions for userspace (not kernel) using LDREX/STREX.                                                                                │
  │ Also creates /proc/cpu/swp_emulation for statistics.
  │ In some older versions of glibc [<=2.8] SWP is used during futex                                                                                │
  │ trylock() operations with the assumption that the code will not                                                                                │
  │ be preempted. This invalid assumption may be more likely to fail                                                                                │
  │ with SWP emulation enabled, leading to deadlock of the user                                                                                │
  │ application.
  │ NOTE: when accessing uncached shared regions, LDREX/STREX rely                                                                                │
  │ on an external transaction monitoring block called a global                                                                                │
  │ monitor to maintain update atomicity. If your system does not                                                                                │
  │ implement a global monitor, this option can cause programs that                                                                                │
  │ perform SWP operations to uncached memory to deadlock.


Ok cool. So long as they're there, glob-parent is effectively being tested for those cases, so I don't feel too compelled to port them all over, but I'll look through and maybe pick out a couple to add to the test script here.


I tried to upgrade from padrino-warden 0.1.0 to 0.20 and need this method. I just need simple logged-in/logged-out authorization and don't want to use CanCan, etc.


the only way i can figure out how to do this is:

var mode = _.reduce(obj, function(mode, value, key) {
  if(value > mode['o']) {
    return {'r': key, 'o': value};
  } else {
    return mode;
  }
}, {'r': 0, 'o': 0});


nice XD


> added close chanel to listener to coplete the wait chanel, also added check for nats subscription

Could this have been done as 2 separate commits?


I think you can reproduce it right here : http://ui-grid.info/docs/#/tutorial/101_intro

On chrome for mac using the trackpad, if I put the mouse on the grid and try to scroll up, it doesn't work


Using an all C# test project, I could get MVC to respond, but I couldn't serve a static file. So ther must be a bug in the StaticFiles middle ware.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I'm not sure that something that was deprecated at 1.2 and now removed in 1.4 belongs in "Additional Breaks" - since the feature was deprecated a full version ago (and 5 years ago).


Thanks for this pull request. I need to have a look at the code generated by recent versions of UiBinder to understand the problem. UiBinder code for SVGImage is generated by org.vectomatic.dev.svg.impl.gen.SVGImageParser. Normally, this class gives a higher priority to the "resource" attribute by immediately calling image.setResource after the constructor has been called.
Once I get a better understanding, I will either merge your fix or tweak SVGImageParser (or both !). Other classes which accept an SVGResource could also suffer from the same problem: SVGPushButton, SVGToggleButton.
I will get back to you once I know more.


**Tip**: Hello CardView - In the material design, the [Cards have a main part](https://www.google.com/design/spec/components/cards.html#). Read about them and use them to create an out-of-the-box material experience. Read more about it [here](https://developer.android.com/training/material/lists-cards.html).

![](https://material-design.storage.googleapis.com/publish/v_2/material_ext_publish/0Bx4BSt6jniD7Qkl1T0YzeHpFNzA/components_cards1.png)

You would probably want to use the [support library version](https://developer.android.com/reference/android/support/v7/widget/CardView.html) -
```xml
<android.support.v7.widget.CardView ....>
```


What's the point of that? With this it fails if there is no mkfifo() into sys/stat.h, without it too.


Whoops - I was confused. These updates point Flume toward a Hive file sink rather than an HDFS file sink.


Thanks Eldy !


I find it peferable to modify the prototype of vglfaker::init (server/faker-sym.h line 34) to include a bool parameter to force reload the environment when true. The parameter will default to false and will only be true at server/faker-x11.cpp:545.
The reason is that init() is also called from the hijacked version of glXGetProcAddressARB and I don't know if a) there are possible side effects  and b) it will slow down initialization time of utilitiy code such as GLEW.


Look at you! Nice work


Probably should have deprecated `authorize!` first, then removed it later in 1.0.0.


I think you wanted to this `if` condition inside the `natsclient.Subscribe`, which should always happen once.


@xfurry:
Pls remove it from sd2:
```sql
UPDATE creature_template SET ScriptName='' WHERE entry = 28912;
```
also pls add ACID part:
http://paste2.org/NwgI9PKD -- text ids need to be changed ;)


Why do you need a friend declaration? `vals()` is a public function, what else do you need access to?


Do we want to hardcode the white_player_id into the factory? I was thinking of a 'create game' test that asserts the user that created the game is given the white_player_id. Not sure that its necessary or not, but it does test the logic that has been implemented for the 'create' action rather than assuming it as hardcoded within the factories.

I've accepted the PR but just wanted to throw this thought out there in case it has any value for keeping the test / logic consistent.


But i believe the general problem for tearings isn't the too low lcd freq .. it's just wrong freq or we've a problem with vsync !!! - I'll try to fix this... problem started with stock JB .......


`uid` for logyard itself and is unique to the node that apptail runs on, so putting this in this map doesn't make much sense. What are we tracking with `state`? Why do we have a time stored in against the logyard uid?


you forget Quantum Cat and Swamp Mirrorer


@dpgeorge : I hope that's indeed typo.


What is the purpose of the instance game variable within this test? I assumed it would be a regular local variable, but I might be missing something...


The https://codereview.chromium.org/876313004 forgot to revert a line that caused compilation error under Debug mode.


The names `state` and `mutex` are too generic unless used in a smaller function where there intention would be obvious. I think [this](https://github.com/ActiveState/logyard-apps/blob/master/apptail/docker/listener.go#L12-L15) is a better way to associate the mutex with the map.


Do you have an example of where this makes a difference? All of the documentation I'm finding suggests that the `ECHOE` and `IEXTEN` flags only take effect if `ICANON` is also set, and that gets turned off here.


Yep : https://youtrack.jetbrains.com/issue/IDEA-85478 :)


Hi


Lol.. You are the one here crying..


**Tip**: Custom attributes should use `res-auto` xmlns.
When you need a custom view attributes, you can do it the ugly way and be strained to package names:
```xml
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
   xmlns:custom="http://schemas.android.com/apk/res/com.example.customviews">
 <com.example.customviews.charting.PieChart
     custom:showText="true"
     custom:labelPosition="left" />
</LinearLayout>
```
*(code from http://developer.android.com/training/custom-views/create-view.html#customattr)*

Or, you can do it automatically with replacing "...schemas.android.com/**apk/res/com.example.customviews**" With "...schemas.android.com/**apk/res-auto**"


Folder structure:

YourName/Assignment1
YourName/Assignment1-extra
...
so basically, just create a new folder for each assignment inside the folder with your name


agreed


double ditto


I've tried the tests with LC_ALL=pl_PL.utf8 and I tried with both java 7
and java 8 and they still pass.

2015-03-04 15:40 GMT-05:00 Andriy Rysin <arysin@gmail.com>:

> Is this windows only? What's the code page?
> On Mar 4, 2015 2:46 PM, "Marcin Miłkowski" <notifications@github.com>
> wrote:
>
>> They probably fail due to codepage settings on my machine. But they DO
>> fail.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10037286>
>> .
>>
>


I added "Towel : " so that it shows up in the General chat tab


I am *VERY* nervous about treating json files as unformatted. There’s no guarantee that characters will each be one byte, or how the file will be encoded and this seems like a fragile approach to me, but admittedly I am no expert on IO.

I suggest you play around with stream access (which works for formatted files too) as well as some of the IO buffering options. (This is off by default for ifort, which has some tunable parameters, on by default with gfortran but can be turned off using environment variables… it’s possible that turning off buffering for gfortran, or turning on buffering and tuning the parameters for ifort might result in a speedup.)

Another idea is to try to read the file in chunks into a character variable and then process the character variable that is already in memory. You could use two character variable buffers and asynchronous IO to perform a double-buffered read: Post the first read, then go into a loop that posts the next read and waits for the current/previous read to finish, processing the newly available data while the next buffer is being read in. Once the processing is over, cycle the loop.


private helper functions should start with \_, so \_avg


what do you think, @davlin93


**Tip**: Create naming conventions for your ID's. Use of naming conventions for your IDs in XML will help you to find them easily from the Java side and be sure you get what you were looking for (and not a NPE).

Don't spare the length of the ID, build them in a hierarchy way,
for example this ID: `item_tips_title` is: `{type of layout}_{main name}_{view name}`


88734010-AFN-horizontal-openings (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1105 of 1105 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.08%25-green.svg)


The `typeof` filter needs to be a little bit more robust as the default operator in JS doesn't quite correctly always do what you expect it to. See https://javascriptweblog.wordpress.com/2011/08/08/fixing-the-javascript-typeof-operator/.


Thanks for this! I again didn't hit this error with PHP 5.4.


> Do you have an example of where this makes a difference?

disregard `ECHOE`,  but OSX users reported ctrl+o not being sent if `IEXTEN` is set.


However, if this approach show promise, fine, but can I request that you wait to merge this feature branch until I’ve figured out the Unicode stuff in a little more detail? The unicode stuff is a somewhat complicated code transformation, and, best case scenario I’m imagining a complicated merge or rebase to rectify the differences in our two feature branches, and worst case, I’m imagining the use of unformatted IO to completely break—and be incompatible with—the unicode/UCS4 branch. Waiting a little bit to at least test this approach with a working and finished unicode support implementation before merging back into master would be greatly appreciated.


Published to http://ui.edev.d2l.com/48_bf5c0aadd63135e18b1b6be9b7e138af59b1bc3f


> If unsure, say Y.

according to what it says, it's better to don't disable it, unless you want problems with some programms that use it :D from what I can understand  -  it'll generate undefined instruction exception, namely, kernel panic, when they are used.


e


change fail-like assertions to rspec matchers


@dstogov this break shared builds of phar since zend_strpprintf() is not an exposed ZEND_API function


Dude...


If X-Forwarded-Port is set to 443, by a SSL nginx proxy for example for the application running on http/8080, the builder's port remains set to 8080?
This check has been removed in the previous version of ServletUriComponentsBuilder


(Also, I think, in the interest of flexibility, *delegating* to an HDF5 handler from the dataset might be more flexible down the road than inheriting from one.)


Hi @abourget,

I'll respond in more depth later, but see here for the design intentions around that:
http://dave.cheney.net/2014/10/17/functional-options-for-friendly-apis


Is there a way we can use whatever GPG's default is?


okk thx.. that means u've an app what uses such (old) instructions xD


Refactored as we discussed. @szemek thanks for idea :)


xD


Petite ligne magiquement moche qui permet de laisser du temps au moteur de jeu de s'initialiser et de se lancer avant dans terminer le programe python. Si la commande rule.stop() est envoyée trop rapidement après la commande rule.start(), le moteur de jeu ne va pas "prendre en compte" qu'on lui a dit de s'arrêter et va démarrer sa boucle infinie. Même si le script python aura "terminé" son exécution, le moteur de jeu tournera toujours indéfiniment en background.
Ce "bug" nous a coûté un ticket chacun de 50$ à @MathGaron et moi :-1:


As reference, I've created a new brach [clang-doc-squashed] (https://github.com/mehw/company-mode/tree/clang-doc-squashed) with one single commit after the upstream commit [a7a4ba681cf99afbd94077554f0161a7e37f0400] (https://github.com/company-mode/company-mode/commit/a7a4ba681cf99afbd94077554f0161a7e37f0400). Now my master branch is in sync with that upstream commit too.

I really do not want to loose the log of the branch [clang-doc] (https://github.com/mehw/company-mode/tree/clang-doc) I've worked on. Is it possible to switch the pull request or to squash the commits without loosing the log? Could the GitHub support be of any help in this case?


https://qtran3.github.io/
https://github.com/qtran3/qtran3.github.io/commit/d012ae19ddd66e8b909b33280a474966c3e1ff78


**Tip**: Don't put dummy text in release builds. While on debug it might come effective to simulate different dummy texts, in release you wouldn't want your users to see those.
Use Android debug/release build variants to smartly manage those texts.


@mingwandroid my plan is to release a Git for Windows 2.3.0 based on this work ASAP and then throw stuff your way. Hopefully you will like most (or all) of what I did, and if not, hopefully we'll find a good solution for everybody involved!


No problem with this. I wonder if we want some generic method you can call on config that performs this kind of cleanup?


@rmcmx


WHY. SO. UN-'APPY?!


@LinuxBozo It's just a little bit of safety to ensure we build the key in the proper order. The order of iteration for a for...in loop is implementation specific. This doesn't matter in the mongo code where order is unimportant, but here we are building a string in a defined order. While it seems most implementations go by the order of declaration, there are some niggles (chrome does numbers before strings for example).


Yes, we probably do, but I haven't given it much thought...


I don't know if it's normal to run migrations every time you test or start the server.


If ambient display / doze mode is disabled, will it still work work fine?


Just a general question here, what are the models and builds created by add_model and add_build doing here ? Are they just temporary models and builds to test on which don't get actually created since UR_DBI_NOCOMMIT is on ?


I would like to inject an ISwaggerMetadataProvider here. Not sure how to achieve this.


79453838-AutomateSomeDocumentation (Myoldmopar) - 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-76.25%25-red.svg)


Ok yeah. I'll bbl in like 2 hours or so I can extract out the migrations to its own test script.


This line throws an exception when there exist two implementations of ISwaggerMetadataProvider in scope


@rizar I was just looking for this, because I wanted to add a few more graphs and wanted to use this as a template, but found it was deleted. Any specific reason you deleted it?


Btw, this is broken for ebola, and easy to test/replicate on the public
devtest server.

Go to:
http://devtest03.openmrs.org:8080/openmrs/ebolaexample/doseManagement.page?patient=b7df92f2-65d6-4749-8bd1-77ffffa5da07

And delete or restore any med administration. You'll see a "redirect://"
instead of actually redirecting.

On Wed, Mar 4, 2015 at 9:02 AM, dkayiwa <notifications@github.com> wrote:

> Yes am looking into that right now.
>
> On Wed, Mar 4, 2015 at 7:58 PM, Mark Goodrich <notifications@github.com>
> wrote:
>
> > Cool, thanks! Unfortunately looks like there is emr-api weirdness going
> on
> > right now that might need to be fixed first.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/openmrs/openmrs-module-uiframework/commit/5c9b32da6666c2aa3f0d338c74b9892d0554b185#commitcomment-10033865
> >
> > .
> >
>
>
>
> --
> If we keep uppermost in our minds the unkind and unjust acts of others, we
> shall find it impossible to love them as Christ has loved us; but if our
> thoughts dwell upon the wondrous love and pity of Christ for us, the same
> spirit will flow out to others.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-uiframework/commit/5c9b32da6666c2aa3f0d338c74b9892d0554b185#commitcomment-10033964>
> .
>


@jdalton, what was I thinking. Will see what I can do on Friday ECT.


Yes.  `add_model` is an unusual way of doing `Genome::Model::ClinSeq->create()`, but it is creating a test model for the newly created test processing profile.


**Tip**: Storing in DB? use `_id` and not `id`. [Android are also doing that](http://developer.android.com/reference/android/provider/BaseColumns.html) in their contracts.
Using `_id` will help you integrate with the CursorAdapter:

`The Cursor must include a column named "_id" or this class will not work. Additionally, using MergeCursor with this class will not work if the merged Cursors have overlapping values in their "_id" columns. `
*(source: http://developer.android.com/reference/android/widget/CursorAdapter.html)*


We're trying to move away from useref as it's not a reliable solution long-term. Was there a reason you wanted to bring this back?


@leonerd can you apply this(and the previous commits in this branch) to libvterm?


awesome, thanks for handling this for me


I see the potential for moving this up to `EmsOpenstackMixin`.  I guess that's what @bzwei is talking about in the comments in the main conversation.

I'm fine for it being in Infra for now, then later finding its way to the Mixin when the cloudy orchestration bits are added.


Can't comment on different devices. On Nexus 6, I experienced random reboots.
Need to test on N5 with ambient display on.
If you are not facing random reboots, it should be fine for you.


Jacob...If Hopkins is closed tomorrow, how about I call you at noon?

On Wed, Mar 4, 2015 at 1:07 PM, Vivek Khatri <khatri1976@gmail.com> wrote:

> Let's say noon on 9th floor. After I work through the current  code with
> you, I can migrate the code into the skeleton package format.
>
>
>
> On Wednesday, March 4, 2015, Jacob Carey <notifications@github.com> wrote:
>
>> I'm free from 12-1:30 tomorrow. Do you want to meet then? Also, I think
>> one helpful thing will be to turn this into a package. there's a function
>> in Rcpp called Rcpp.skeleton.package (i think). Do one of you want to try
>> running that and then migrate the existing code into the skeleton package
>> format? Apparently you can only have multiple C++ files if you make a
>> package, and I think multiple C++ files will make this easier to understand.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Jacobcvt12/bm1_project/commit/53330c151aa257fd625bd594c3f29b6e8256a860#commitcomment-10035096>
>> .
>>
>


EPEL is a sub-project of the Fedora project, so it's under the governance of the Fedora Project's leadership, but the packages from EPEL are not applicable on Fedora. The acronym is "Extra Packages for Enterprise Linux", so the packages are only built for RHEL and its derivatives (eg CentOS or Scientific Linux, etc).

In other words, we only need to enable EPEL on RHEL and CentOS - it doesn't make sense to do it when `lsb_release -si` is `Fedora`

Otherwise this looks good!


TeamCity Akka.NET :: Akka.NET PR Build [Build 101](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=154&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


If you just want to keep the branch, you can create a new pull request based on the new branch.

Or do you really want to have this history in the `company-mode` master branch after merge?


The link here does not work fyi.


why?


2015 2015?


This could be `gs_shutil_rm_rf_at`


>laphabetical order

Haha, sorry, this made me lol.  I half expected it to list in some funny order rabcvso etc.  I read it as 'Laugh-abetical order'


Why removing Opt suffix when the values can be null?


Ah, right, forgot we're basically iterating Object.keys()..  Good call..


why not?  we've been doing that on libraries that are meant to be dependencies of other projects...


Should and can we reuse the already defined module attributes? E.g. `@seconds_per_day 24 * @seconds_per_hour`


it shouldn't work in case of compiled liblwgeom used without installing, or if installed with a different prefix. But I don't know where lwgeomPath method is used.
Can you clarify?


How does [this](https://github.com/tedious/Stash/pull/211/files) look?


**Tip**: Goodbye old List/Grid and Adapters, Hello RecyclerView. [Read more](https://developer.android.com/training/material/lists-cards.html) about this and how it will help you to better handle your data and list/grid transformations in UI.

Also, Have a look at [this code](https://github.com/tipz/tipz-android/commit/3c364b74ce2e6e425f81e4cf10ee2619b58a4779) we've created to help manage the RecyclerView boilerplate code.


we can take this line and line 14 out of the if satinet and put it after the end of the if satement since they are the same.


I fixed that but it isn't showing the error messages


Composer ignores the `composer.lock` file for dependencies already. Committing your `composer.lock` file makes sure developers all use the same dependencies to prevent "works on my machine" syndrome.

Composer even recommends committing the lock file: https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file


I think there's a copy/paste error in this line.


Looks like by mistake you wrapped domain_obj.custom_chat template with enabled_disabled instead of domain_obj.sms_mobile_worker_registration_enabled


See sample code on how to use it: https://github.com/tipz/tipz-android/commit/e1ccac6e6d166fe82d867eea4a02848ded671f3c


And the same copy/paste error here...?


I wouldn't do that. The calculations on that line are too heavy to abandon them on next line. Consider put it in 'else' path


Does the main version of the app I release not work with 120584? I might have to test that again :(


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 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-4.06%25-green.svg)


The only difference I can see between this `save_orchestration_stacks_inventory` and the same method in `save_inventory_cloud` is that the cloud inventory stacks have a `template_id`.

Is there a way to reuse that method?  Maybe move orchestration to `save_inventory_helper` or just `save_inventory` (like `save_vms_inventory`).

Also, do Openstack Infra Stacks not have a template?  Is that specific to cloud orchestration stacks?  @bzwei?


The separation of concerns comment still holds, let's discuss this.


I had issues with it when I started using it a month ago, but probably was some mdss patches missing.
Works normal now.

What he did seems like basicly the same thing lcd notify does, what's the point of doing a driver that already does what LG did?


hey this looks good but i don't see the image work for the posts...i thought we worked on that on our last meeting?  maybe stashed or on a different branch?


jaja yes, was a typo xD


I know that might sound weird, but I think that using yet another domain-specific language increases significantly the cost of maintaining proper documentation.  Making a change in it would require learning DOT language which is far less self-explanatory than our current documentation markup.  I just thought it is an overkill.

But if you really want them, go ahead.


Agreed! No plans to merge this yet. Just doing some experiments. Learning as I go. See also: https://software.intel.com/en-us/forums/topic/542533

It's possible that we can use the ENCODING argument to OPEN, along with the SIZEOF function on the character kind to make it work with unicode...but I haven't looked into this in any detail.


this line can be removed


Same reuse question here as for `save_orchestration_stacks_inventory`.  It looks like we could just move the cloudy method to a common place (`save_inventory_helper`, or whatever) and reuse it.


No need to test, I have random reboots in N5 with doze enabled, just after a notification is received and the screen turns off.
I think FB notifyers are the cause.


88620944-FluidCoolerWithLowSpeedAuto (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1637 of 1647 tests passed)

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


@dscho we'll definitely do what we can to accommodate your changes, I'm sure many are fine, but we need to make sure that the git path mangling alterations don't break other things, so we may need to come up with a way to enable them in that specific situation (e.g. if program is git.exe, or perl.exe called by git.exe  etc, ironically, perl is already one of the toughest programs to please from a path mangling perspective and the sooner git doesn't need it the better); we can discuss this in detail after you get your release done.


Thanks for the feedback. Null checks have been added now.


**Tip**: Fragments are automatically recreated upon Activity re-create. If `savedInstanceState != null` it means the fragments will also be part of the saved instance state. In this case simple find them again (`findFragmentByTag()`) and create the reference, don't recreate them.


TeamCity Akka.NET :: Akka.NET PR Build [Build 101](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=154&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:10:27
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


@WayofTime I have no real code for this ready, but i thought a bit about this:
If you only reflect 'incoming' projectiles that would solve the 'own arrows' problem.

You should be able to determine if a projectile is 'incoming' by checking for:
`DotProduct(delVector,MotionVector) < 0`

[(Some GeoGebra thing i used for thinking)](http://ggbtu.be/mpdnubGDT)


ugh no bueno!  ok lets look at it today.


Windows cp 1250...
4 mar 2015 22:12 "arysin" <notifications@github.com> napisał(a):

> I've tried the tests with LC_ALL=pl_PL.utf8 and I tried with both java 7
> and java 8 and they still pass.
>
>
>
> 2015-03-04 15:40 GMT-05:00 Andriy Rysin <arysin@gmail.com>:
>
> > Is this windows only? What's the code page?
> > On Mar 4, 2015 2:46 PM, "Marcin Miłkowski" <notifications@github.com>
> > wrote:
> >
> >> They probably fail due to codepage settings on my machine. But they DO
> >> fail.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10037286
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10038985>
> .
>


You can use [`ethnicities.uniq()`](http://api.prototypejs.org/language/Array/prototype/uniq/).


Thanks @StefanescuCristian !!


@xbony This would make the thing "steamcraft:steamcraftBlockPlacer".


Clever way to keep the session locked in uncontrolled scopes.


Perhaps `RelWithDbgInfo`?


Whaaa??!?!?


What is the reason for this? Is it the ServiceProvider/publishing or some larasearch class incompatibility with Laravel 5? I would love to use it in my project with is a Laravel 5 ... :-(


Will the log on GitHub be lost after squashing the commits in the current pull request #288 , clang-doc branch? I always can make a backup of the branch to preserve the history for personal use. In this case do not mind the newly created branch.


This is good!

Although the deepest sleep mode that'll save the most power is power-down mode, which is what is implemented. In this mode, all PRRs are already disabled. If we want to implement some other sleep mode apart from power-down, these functions become necessary. turnOffGPIO() and turnOffDigitalInput() might be the most useful ones here, especially if we have a lot of pins in use. If you create a new pull for those two only, I'll merge those.

Thanks


I don't think it would make any difference. It's not like we're gonna use
the debug info on travis.

On Wed, Mar 4, 2015 at 1:40 PM, strager <notifications@github.com> wrote:

> Perhaps RelWithDbgInfo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sas/ds2/commit/e3e130598efddfe0438be7e3d5c06dc0f6756884#commitcomment-10039478>
> .
>

--
Stephane Sezer


"Audios" is not a word.


I love it. Dropping our Hs and singing "God Save The Queen" we are!


I've started a branch with bricked_hotplug, cpu-boost and msm-limiter and I had reboots without these commits after he moved away from lcd notify. Idk why he moved away from lcd notify in the first place, just thought these commits may help you.


**Tip**: DB operations are heavy, use [AsyncQueryHandler](http://developer.android.com/reference/android/content/AsyncQueryHandler.html). AsyncQueryHandler will help you in running those operation in the background (without messing around with AsyncTasks, threads and such)


Please stop submitting files from "bin", "gen" directories.


30414/Bologa M. Marius - Vasile/Assingment1/liste/main.c

You may want to declare a global static variable for the file name if you are to use/open it in every function.
ex:
static const char filename[] = "output.txt";
...
FILE * fout = fopen(filename,"a");


Please use the BlueVisionTec module: https://github.com/bluevisiontec/GoogleShoppingApi

Ours only works with PHP >=5.4


I guess #641 would be an important issue to be fixed, but otherwise it would be wise to do quick releases so bugs arise more quickly and so we shouldn't having to care with old bugs as we do have now.


Shouldn't this be `return super(UnicodeSafetyWrapper, self).__iadd__(other)`?


Why are you putting this on a commit from months ago..?


Well, it's purely optional to use them of course, so I don't think it really adds a burden to anyone who prefers to just explain things with text. I do think that in some cases images can help a lot, and in this particular collection of bricks a few images could clarify very quickly what all the different bricks represent (and reading Dot is far more accessible than reading raw SVG...)

It's not that pressing right now though. The current documentation is barely usable anyway, because it prints all the attributes of `Application` for each `Brick` :/


Hey let me know if you guys are gonna meet tomorrow, and if not what I can help with.

Sent from my iPhone

> On Mar 4, 2015, at 4:27 PM, Vivek Khatri <notifications@github.com> wrote:
>
> Jacob...If Hopkins is closed tomorrow, how about I call you at noon?
>
> On Wed, Mar 4, 2015 at 1:07 PM, Vivek Khatri <khatri1976@gmail.com> wrote:
>
> > Let's say noon on 9th floor. After I work through the current code with
> > you, I can migrate the code into the skeleton package format.
> >
> >
> >
> > On Wednesday, March 4, 2015, Jacob Carey <notifications@github.com> wrote:
> >
> >> I'm free from 12-1:30 tomorrow. Do you want to meet then? Also, I think
> >> one helpful thing will be to turn this into a package. there's a function
> >> in Rcpp called Rcpp.skeleton.package (i think). Do one of you want to try
> >> running that and then migrate the existing code into the skeleton package
> >> format? Apparently you can only have multiple C++ files if you make a
> >> package, and I think multiple C++ files will make this easier to understand.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <https://github.com/Jacobcvt12/bm1_project/commit/53330c151aa257fd625bd594c3f29b6e8256a860#commitcomment-10035096>
> >> .
> >>
> >
> —
> Reply to this email directly or view it on GitHub.
>


Why are there two buttons "play" and "stop" with listeners. It should be just one button that changes the drawable and stores the current state "playing" or "stopped" in the background.


Quid des vendors complexes ? (ex: A\B\C)


**Tip**: Adapter view data is always there, clear it if not already overriding it. An Adapter is a recycler of data, moving the data between already-created views.
When you have a view that was set with some value, the next recycling of that view will come with those values already set. Meaning - if you don't override them you will witness old values raising in your new recycled view.


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1140 of 1140 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.62%25-green.svg)


done


Thanks for squashig.


double done


You might want to write something like this:
You might want to add a "free(head);" there. This way, you will free the memory allocated to that one element as well.

Currently, you are not freeing the memory in the case in which you have only one element in your list. It's not a big deal, don't worry about it too much, but it's nice to 'clean-up' your mess every time ^^


Intentional?


same principle as above


same here


Are you actually trying to delete the file in three different ways? :question:


**Tip**: You can use `"@null"` in attributes values


doesn't the fact that the function exists in a function closure imply that its a private helper?

    my.functionName(...) { ... }
is explicitly public. shouldn't it stand to reason that
    functionName(...) { ... }
is implicitly nonprivate


Good commit overall. Do not take my particular comments in a day way. I was just testing the commenting system!


OMG


Derp.  Thanks :)


done and done


Moved the heart meter over a bit so that it doesn't overlap with Kokona's long name.


I will ask Dmitriy to check icon, it is pending task to update icons on all screens and background. Mostly they are done but not integrated. I hope will be done by the end of this week.

Yes Save button should be displayed once you have something to save that's an issue.


It will. But either way you can keep the current branch history somewhere, in this branch or a copy of it.


I will fix this hardcoded path later.


I would have split it over multiple lines, but that's personal preferences, I'm ok with this :)


This does simplify the code :)


I can copy the hpd code from DrScharps slides,. It is east to repeat the code for each patsmeter in some dataframe,  however I dont know how many parameters the mcmc ultimately spits out, or really how to deal with a variable number of parameters(eg ragged arrays).

Sent from my iPhone

> On Mar 4, 2015, at 12:32 PM, Jacob Carey <notifications@github.com> wrote:
>
> @Tharper84 So I think what you have is a good start. However, we will need to estimate autocorrelation and effective size for each parameter of each chain (i.e chain 1 estimation of theta1). Additionally, we should also probably estimate the 95% HPD (highest posterior density) region for each parameter. I'm not sure if there is a function in R to do this, or if you will have to come up with something manually. Also, try writing these r functions in a .Rmd (R markdown) or .Rnw file, so that we can have reproducible research. I think Rob mentioned that a portion of the grade was doing this.
>
> @KhatriVivek I'm still wrapping up the code to produce the MCMC estimates. Almost done. I think once you are done reading through the code, you can let me know if you see any problems. I will read about this too, but can you read about implementation of reversible jump MCMC in the context of mixture models? I'm hopeful that we can find a good algorithm. I'm not sure what your question is regarding input X.
>
> —
> Reply to this email directly or view it on GitHub.
>


Yes indeed looks like a typo.


Ajouter un message à la fin, invitant à supprimer le répertoire install ?


Local directory


Neat! :ok_hand: (I like emoticons)


`WithCookie` takes a `http.Cookie` object, not a name and a value. It could be changed if you think it’s more convenient to have it as `WithCookie(string, string)`.


Typo: e.g.
I like the flags :+1:


Your code looks nice and clean, good job.

I have only 2 (more or less aesthetic) suggestions
- your functions names should be consistent in the way they are written. For example, you have print_list which suggests you will use the "_" as a delimiter for your words but then you have "addfirst" and other functions which do not follow this 'delimiter rule' (also, DOOM is again, not consistent with any of them). Try your best and follow just one principle. I would suggest camel-casing as it is the most widely used in every programming language. A simple example of camel-casing would be: addFirst, addLast, deleteFirst, deleteLast, aLongerFunctionNameWithCapitalLettersForEachWord etc.

- even though it was a rather small assignment, you might consider modular approaches. feel free to distribute your functions in a modular fashion but careful not to overdo this!


:sweat_smile: kind of true...


commit du yml ?


uncomment ?


30414/Dolineanu A. Vlad Ionut/Assignment 1 - extra/functions.c

it's good that you have this handy method here. I always tend to create one myself


Hmm, UkrainianTagger sets the locale explicitly with
    setLocale(new Locale("uk", "UA"));
and for several files it reads I set explicit encoding of UTF-8

My only theory at this point is that it compiles the java source with wrong
encoding. But maven project specifies the option explicitly:
      <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
Unless the failure happens only in IDE where source file encoding may not
be set to utf-8...

2015-03-04 16:40 GMT-05:00 Marcin Miłkowski <notifications@github.com>:

> Windows cp 1250...
> 4 mar 2015 22:12 "arysin" <notifications@github.com> napisał(a):
>
>
> > I've tried the tests with LC_ALL=pl_PL.utf8 and I tried with both java 7
> > and java 8 and they still pass.
> >
> >
> >
> > 2015-03-04 15:40 GMT-05:00 Andriy Rysin <arysin@gmail.com>:
> >
> > > Is this windows only? What's the code page?
> > > On Mar 4, 2015 2:46 PM, "Marcin Miłkowski" <notifications@github.com>
> > > wrote:
> > >
> > >> They probably fail due to codepage settings on my machine. But they DO
> > >> fail.
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10037286
> > >
> > >> .
> > >>
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10038985
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/languagetool-org/languagetool/commit/b8f10a44b255e50fbd12078054f22e6f88efdba7#commitcomment-10039470>
> .
>


Is this literal concatenation documented somewhere to be a supported feature?   (A quick google search leads me to http://tools.ietf.org/html/rfc4880 which at a really brief scan appears to be packet based, so theroetically one can just concatenate packets?)

Anyways at least worth a comment in some form.


**Tip**: Carefully use platform drawables: `"@android:drawable/...`. There were days when the official documentation discouraged the use of platform drawables.
`Warning: Because these resources can change between platform versions, you should not reference these icons using the Android platform resource IDs (i.e. menu icons under android.R.drawable). If you want to use any icons or other internal drawable resources, you should store a local copy of those icons or drawables in your application resources, then reference the local copy from your application code. In that way, you can maintain control over the appearance of your icons, even if the system's copy changes.`
(This was written, but no longer, in http://developer.android.com/guide/practices/ui_guidelines/icon_design_menu.html)


I'm assuming this code is derived from a reference example.  I don't see anything obviously wrong.  However, comparing with librepo, see e.g.:

https://github.com/Tojaj/librepo/blob/master/librepo/gpg.c#L150

I notice that it's also checking `GPGME_SIGSUM_VALID` and `GPGME_SIGSUM_GREEN`.  Ideally at some point we actually share code with librepo more, as it's another LGPLv2+ GLib library.

Note rpm-ostree master uses librepo too.


Huh? Might want to remove the API key.


Faut penser au favicon


yes, c'est prévu


75127368_DCFSEC (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1644 of 1651 tests passed)

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


@params is not a valid tag, it's [@param](http://documentjs.com/docs/documentjs.tags.param.html)


That's, right now you must style the menu yourself. Maybe paper-list is handy here, but I never tried. You can find inspiration for these styles in the dummy app.

You can revert the changes or commit a fix. It's your fork. :) I would try to revert.


within pedigree algorithm code I'm trying to not use any prototype-specific stuff (as well as stuff not available in older browsers) to keep the core of the code portable. E.g. jquery may conflict with prototype, and some other project (standalone pedigree?) which uses pedigree algorithm may use jquery or whatever other library without .uniq()


well done bro nice fix


At line 32 add a
finally{ do this thing; }
So that program won't just stop there if anything goes wrong with times.
I mean use something which will make our program recognise the times as closed like 00:00:00
(a default thing)


(Though GPGME does look high level enough that having a shared library wouldn't really help much - I think i'd be most comfortable if the logic for "check the signature result" was shared/identical, ensuring we handle things like unknown keys, expired keys, etc. consistently)


May as well remove the commented `/*const*/` if it's not used.


are you sure?


oh, i saw this library in a bigger picture. i use the functions on startup/setup to disable all parts that i don't need for my program. so the atmega not only saves power during power-down, but also while being active.  I would also like to include a delay()-implementation without busywaiting by using the wdt during powerdown


Ah, mince, exact.


please answer my questions?


I think this broke Travis, see, e.g., https://travis-ci.org/JuliaLang/Tk.jl/jobs/52558592. Travis doesn't have a display environment set up, so you need that `xvfb-run` in there.


yes.


Silly question. There's a lot of `self = this` usage seen here. Have we confirmed the cleaner `.bind(this)` isn't able to cater to our needs?


this will fix the problem.


30414/Dolineanu A. Vlad Ionut/Assignment 1 - extra/functions.h

Your structures are well created and divided. Another (not better, nor worse) solution would have been to have only two structures, your st_node and country_elem, then country_elem would contain a head and tail instead of a list_elem. Nevertheless, both ideas are good.

What is not so 'aesthetically' good is the names of your structures.

why "lst" and not "waveList"
why "node" and not "wave"

Make them more meaningful to the problem at hand. This generally helps people who browse your code understand it more quickly!


Added gianfar.c to drivers


@illegalprime was there any reason why you left both "song" and "song2" in the templates? We definitely don't need to have both of them (and I think the new one is better anyway).


Ça n'a pas de sens que ça soit dans la config


30414/Dolineanu A. Vlad Ionut/Assignment 1 - extra/main.c

Even before reading what you did here and just taking a look at the dimension (in # of lines) of this main method, I can tell you that it NEEDS TO be split!
Let's see where..


I somewhat hate this, would rather see a semantic class name line sub-headline or something... but maybe I am just being a baby


Ok, so up until here you have your "reading" part in which you read from your file. That's good, but you can extract this in a different function.


Adding gianfar_driver_netmap support (Make sure CONFIG_GIANFAR is 'm' in config file)


![screen shot 2015-03-04 at 1 59 05 pm](https://cloud.githubusercontent.com/assets/2068077/6494476/ef4cfdba-c276-11e4-81fe-88a6bba0bab0.png)


oh gosh, really? :cry:


This is temporary - to not touch styles yet, just added alias. I'm on markup cleanup up to weekend for Joanne.


Because i can only add comments to Lines in commits and not lines in files, i think.


**Tip**: Silent the logs. Printing to log in release mode might cause a security risk - exposing things you won't want typical users to find out.
You can get rid of them by adding this section to your Proguard:
```
+# Removing the log (needed for security in release builds)
+-assumenosideeffects class android.util.Log {
+    public static boolean isLoggable(java.lang.String, int);
+    public static int v(...);
+    public static int i(...);
+    public static int w(...);
+    public static int d(...);
+    public static int e(...);
+}
```


Thanks, will fix later.


this snippet here computes the best and worse countries. this belongs to a different function as well!


well done!


+1 on @DevL ... makes it easier to reason about imo.


75127368_DCFSEC (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1638 of 1645 tests passed)

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


> I don't see public versions of the wiredtiger_struct_XXX functions that take va_list arguments, though, do they exist? If not, should they?

I'm happy for them not to exist. I think the pack stream interface is more intuitive for creating and decoding structures and covers the functionality. @michaelcahill should have the final say though.


C'est prévu aussi, c'est une version PoC


this snippet here which computes who did the last hit also belongs in a separate function


These tests looks like excellent candidates for row/table-specs.

Is there anything like that in elixir?


@andreanastacio Yup :pensive:


Last but not least, this last part belongs to a separate function as well.


Turns out that making the point release version more dynamic (as in, major and minor defined separately, the way projects like FFMS2, L-SMASH, and FFmpeg do it) will require deeper work, since including internal.h in avisynth.rc causes some weird errors to crop up in main.cpp.  So I went with a much smaller change instead.


@adunofaiur this causes tweetbubble rendered tweets to contain raw html tags.


Well this module is using useref now and "html" task is based on it. So uncss is not working now. Other tool thats based on useref will not work unless we change the build process.


Your code looks OK for now. I'm not looking for efficiency necessarily but I am looking for a good and clean code design. I think we can work on efficiency after, if need be.

Good job and good luck with the refactoring!


Turns out that additions were a bit broken! I fixed them in the next commit. Thanks!


You want to use `put` for updating, not `post`.  Use the `_method` trick to change the form method to `put`.


OK, I agree.


You probably don't want to change `created_at` when you're _updating_ a record. :)


bugfix de issue #62 (https://github.com/RedPanal/redpanal/issues/62)

Como en el template se usa la variable 'user' se mostraban los seguidores del usuario logueado independientemente del <object_id> que recibiera la función followers, decidi cambiar 'actor' por 'user' en la vista (tal como la usa followig también), en vez de cambiar el template por 'actor', esperemos que esto no afecte alguna otra cosa!


Yes we need to update, cause we wanted to provide easier user experience so
the plugin could be bought directly from application. It is not used now,
though it is hard to put it, so we need to make conscious decision to drop
it again.

On Sun, Mar 1, 2015 at 4:07 PM, Boris Kraut <notifications@github.com>
wrote:

> Could you explain for what this permission is needed and maybe update your
> FAQ at https://osmandapp.github.io/faq.html#permission ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/Osmand/commit/275ea7728cac21815c633af4722d0066f0fc273b#commitcomment-9974951>
> .
>


Are you sure this works?  Did you test it?


Adicione uma descricao do projeto


Newline at end of file, please :)


This is a new commit...


Personal code-style preference would be:

```
var dialog2 = document.getElementById('dialog2');
if (dialog2.toggle){
  dialog2.toggle();
}
```

Mostly to avoid re-querying for dialog2 if toggle exists.


Could we make it less hacky and define it in the missing model?


Then why *months* ago.


This will only be tested with `-type full` or `-type ecryptfs`.
Testing and verifying ecrypt format on my quad core system (`i5-4570 @
3.2 GHz`) takes 2m14s.
May be Jim wanted to speed it up by using a shorter word list, but forgot to adjust the number of expected cracks (and, may be, reduce the ecryptfs_tst.in file).


`<button type="submit">`


That's a good idea. Our dataloggers are designed take measurement from a sensor and write data to a microSD card when awake, so the current draw is a lot when awake, starting with 5mA for the chip and going between 40-70mA while writing to SD card (although this happens for about 50ms only). Turning these peripherals off during that time probably won't make a big difference there, but for your application, it might help.

The WDT delay sounds interesting - do you mean replace any delay() in code with power-down and wake up using the WDT? I'm interested to know if it would take the same time as a delay() or slightly longer.


:+1: Why the method name change from ```click!``` to ```render!```?


@razzeee i didnt know it but i was just one step away from submitting the readme pr ive been talking about forever, i just 'rebased' (lol, complete guess, is that the right term?) or recreated it off the current readme but before submitting i wanted to verify that the translation system is 'transifex' vice 'transiflex' as written in readme, thx


Take a look at line 45.


I think we've pretty much switched to recommending vtctlclient over vtctl. @alainjobart can confirm.

The difference is that vtctlclient doesn't talk directly to the cluster (as shown in this diagram). Instead, it talks only to vtctld, and vtctld then talks to the cluster. This has a few advantages like making vtctlclient lightweight, and allowing vtctld to do access control.


I mean, was what made you think that ```render!``` is better than ```click!``` on line 45.


Do you know what type is actually returned from `require('leaflet-draw')`? If not might pay to make this `var draw: {};` so that using it requires casting it to `any` instead of just silently propagating the `any`ness.


kk


fun.html added


This repo is only for knife solo. For that doc you'd want to open an issue
with chef proper.

2015年3月5日木曜日、Nilesh Londhe<notifications@github.com>さんは書きました:

> Yes. doc <https://docs.chef.io/config_rb_knife.html> seems wrong. .first
> in code
> <https://github.com/chef/chef/blob/master/lib/chef/knife/cookbook_create.rb#L70>
> seems right.
>
> Reopen the PR?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lvnilesh/knife-solo/commit/1898857d74ea64019fc1178c3155cf24a702a362#commitcomment-10036718>
> .
>

--

-Mat

about.me/matschaffer


Are you compiling with SM-4.9? I'm trying with 4.8 and so success because of LZ4 errors and SM.


What is this? unknown to me...


using **img_name** here will result in ... manjaro Linux and not Manjaro Linux. If we use lower case also following words should be lower case ...


I dunno. I found it overly verbose, and "everyone" knows there are 24 hours per day, 60 minutes per second, 60 seconds per minute. I don't find it unclear the way it is, and having `24 * @seconds_per_hour` felt like it forced me to actually read and understand the line rather than just glance at it.

But I don't have a strong opinion on the matter; I'm just stating what went through my head when I made the change. I can never remember if it's 86400 or 84600, so that feels like a magic number. 24*60*60 felt lightweight, even though _technically_ they're magic numbers.


The algorithm will 'show' SIMD for both sha1 and sha512, BUT the only SIMD currently is for office-2007.

I will have to compute the 'inc' value the same way I did for 2007, when I move to 2010 and 2013.

NOTE, the encryptKey was reduced from 256 bytes, to only 20. We only use 20, and there is code that will exit jtr is a password is used that somehow requires a longer key.


Do all these events need the `Draw` prefix when they're already under the `Draw` module?


I changed the report model code to search for render events because it was broken otherwise.


This was an horrible idea


I don't see any difference, but you should ignore my previous comment as you're already doing the is_null check above and I just missed that.


We're using it to indicate a method is used by jitted code, even if nothing in the codebase statically refers to it. Hopefully it will help us avoid deleting "dead" code that's not really dead.


I came across this issue as well but agree with @pablojim that the above change doesn't seem quite right.  I'm creating a custom directive which includes highcharts-ng but when I set $scope.chartConfig in my directive, most of the options are ignored.  Commenting out this line seems to fix the issue.
```javascript
if (newOptions === oldOptions) return;
```
Comparing old/new values in the watch statement makes sense to me so I'd rather not remove it.  Adding ng-if="chartConfig" in my directive template did the trick.  By doing so the ng-highcharts directive isn't processed until $scope.chartConfig is set by my directive and everything sees to work as expected.  Yay!


manque le add de AsseticDumper


I haven't dug deep to find anything like it. A cursory search didn't turn up anything. Creating one should be a straightforward macro, though. Want to give it a try?


Should we use on delete cascade instead?


easier to spot, standard practice


On a side note, the dropTable call should also drop the `x_shards_` table


will fix in the next commit!


rax retry tests


don't need to enqueue a style that isn't registered. this line can go.


We need to drop the x_shards table here


Indicator values are synchronized as part of the site synchro region - you only need to sync the schema here.


Nice.


Wstyd przyznać ale tego nawet nie czytałem dokładnie, skopiowałem tekst ze strony http://lms.org.pl/doc_en/intro.html#ABOUT-LMS


these two lines can go, if they're not being used.


@paulwalker , the change is nice, but I've notice you have not bumped up the version number to 0.0.11


Fix a116d1e


Nice!


It's transifex your right. Could have sworn it was flex, my world is crumbling oO


Being honest is difficult because, there are no breaks.


The setup program automatically creates a configuration file named 'builder-release.conf' which is imported before importing this https://github.com/marmarek/qubes-builder/blob/master/example-configs/templates.conf#L15 (and the r2-default https://github.com/marmarek/qubes-builder/blob/master/example-configs/qubes-os-r2.conf).

The builder-release contains the release version the user selected to build against as well as the GIT_PREFIX which allows them to be able to select which branch to build against.

This change to the configuration file will break this functionality since it resets the GIT_VERSION absolutely.  Is it possible to revert this commit, or would it break something else?  If it is not possible to revert this commit, I will need to look at another solution for the user to be able to select another branch to build against.


I have no strong feelings about it. I concede that not everyone have my savant-lite memory when it comes to remembering numbers so changing from the original magic numbers to less magical numbers is certainly a win any way it's done.


I love that you guys are pulling this out of gmail!


.rtl is not used anywhere - all references to this can go. as well as .desaturated


There are 1501 lines in the file (first line is a comment), so I believe it was created to be 1500 lines.

Now, the problem could be several things.

1. problems with pass_gen.pl
2. the input file created was too short  (I take it this is the problem)
3. problem in the format in JtR
4. problem with the TS

My best guess on the likeliness of the problem was the order above.  I just did not have time to check.  It must be that this format has a very short Plaintext length. There are 150 'unique' passwords used.  Likely we should redo this anyway, reducing that number down to 10-15.  This format does not have dupe salt remove logic (it can not be done).  In this case, we should reduce the count to improve speed (some).  It will never be fast, but we should be able to get it down to 30-45s


89438532_RadianceBugFix (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2090 of 2212 tests passed)

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


I wouldn't remove the license file


Ok. I'm trying to squash the commits in the pull request, the result should be the same as clang-doc-squashed.


Je l'avais fixé en avance mais je bloque sur ça (qd je lance assetic:dump)

```
[RuntimeException]
  The source file "/home/web/dev/spear/api/www/../assets/compiled/main.css" does not exist.
```

Source ?


move to slim_dimens.xml


Oh, sure, agreed. Will include it


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


@Mark, were you able to test the followup fix?

On Thu, Mar 5, 2015 at 12:26 AM, Darius Jazayeri <notifications@github.com>
wrote:

> Btw, this is broken for ebola, and easy to test/replicate on the public
> devtest server.
>
> Go to:
>
> http://devtest03.openmrs.org:8080/openmrs/ebolaexample/doseManagement.page?patient=b7df92f2-65d6-4749-8bd1-77ffffa5da07
>
> And delete or restore any med administration. You'll see a "redirect://"
> instead of actually redirecting.
>
> On Wed, Mar 4, 2015 at 9:02 AM, dkayiwa <notifications@github.com> wrote:
>
> > Yes am looking into that right now.
> >
> > On Wed, Mar 4, 2015 at 7:58 PM, Mark Goodrich <notifications@github.com>
> > wrote:
> >
> > > Cool, thanks! Unfortunately looks like there is emr-api weirdness going
> > on
> > > right now that might need to be fixed first.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/openmrs/openmrs-module-uiframework/commit/5c9b32da6666c2aa3f0d338c74b9892d0554b185#commitcomment-10033865
> > >
> > > .
> > >
> >
> >
> >
> > --
> > If we keep uppermost in our minds the unkind and unjust acts of others,
> we
> > shall find it impossible to love them as Christ has loved us; but if our
> > thoughts dwell upon the wondrous love and pity of Christ for us, the same
> > spirit will flow out to others.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/openmrs/openmrs-module-uiframework/commit/5c9b32da6666c2aa3f0d338c74b9892d0554b185#commitcomment-10033964
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-uiframework/commit/5c9b32da6666c2aa3f0d338c74b9892d0554b185#commitcomment-10039217>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


Ya la compilation less a faire avant.

```shell
vendor/bin/lessc assets/less/build.less > assets/compiled/main.css
```


May be there were 1500 passwords in an earlier version, but then you added some random longer passwords at the top of the dictionaries.
Later you wanted to speed it up by reducing the dictionary size, but committed an incomplete change.
(Just guessing here, I didn't check the history.)


#CaseFoLife


i do u were the best communist ever


C'est pas totalement faux ;-)


C'est quoi que t'as pas totalement compris ?

:D


OK appli live (modulo le changement de namespace qui n'a pas marché partout - surement car encore en wip ?)


88734010-AFN-horizontal-openings (lgu1234) - 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-78.46%25-red.svg)


Unless it doesn't make sense to use it.


ha, I was actually considering to remove them .....


C'est fort possible, c'est plus du Poc que du wip en fait.


Je me suis pas posé la question là dessus, j'ai simplement pris le htaccess de la doc de silex.


Actually, it was more a case of not wanting to add `@seconds_per_week 604_800`. Now that's a magic number.


I think this should be a const cast?


79453838-AutomateSomeDocumentation (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-76.31%25-red.svg)


Oops, sorry... we've been having lots of other build issues, so I haven't been able to test... will make sure I update locally tomorrow and test there if nowhere else... thanks!


Uhm... You sure you wanted to wipe out the styles.css completely???


fix 36dbc9d


<string name="battery_level_template" msgid="1609636980292580020">"<xliff:g id="LEVEL">%d</xliff:g>%%"</string>


I was just about to push my modifications, with similar intentions of giving more detailed information.


fix 5a5e47393


Sadly, these are still wrong. man strncat says:
       If src contains n or more characters, strncat() writes n+1 characters to dest (n from src plus the terminating null  byte).   Therefore, the size of dest must be at least strlen(dest)+n+1.
We want n to ignore the size of dest.
I think we might want a different strncat-like routine that does the right thing.


why 3 seonds


Hey ! What is the permission for use a sign [Warp] ? When i do : essentials.sign.use.warp, He tells me : [Error] You can't teleport to that warp :( Plz help me, it's very important ! ( bad english ? I am french xD )
Thanks !


Without this commit, builder.conf was ineffective because of defaults from Makefile. Instead of this change, we can move setting the defaults after loading builder.conf. Care to prepare a patch?


Did this break something? On both 0.3 and 0.4 I get:
```
INFO: Updating cache of Images...
ERROR: Missing package versions (possible metadata misconfiguration):  Images v(v"0.4.31",v"0.4.32") [68126750d07dec407ed0dbbd8ba01be28ce176d1[1:10]]
```


What about when openal is active, you missed that bit.


I think you probably need to check for `if (doc.meta && doc.meta.build)`


(better for something to be missing from a view intentionally than because the view index crashed)


Bei der Zielbestimmung ist tatsächlich die HINOTORI Executive der Kunde, nur wir. Die Änderung in Zeile 3 müssten wir wieder rückgängig machen.


You should attach the permission for the warp too, so if you want to get into a warp you may have the permission:
``essentials.warps.[warp name]``


I've tested this with Cryptol and the SAW tools, and everything looks good.


89438532_RadianceBugFix (rpg777) - 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)


I thought the song went "3 is a magic number". ;)

But yes, that'd be hard to stomach.


Not my fault :P I was coping @Zenith-One 's branch.


Looks like @timholy may need to push 68126750d07dec407ed0dbbd8ba01be28ce176d1


this might be a little bit confusing with couchdbkit's `Meta` class. though i don't feel too strongly.


Remember to update the require as well: https://github.com/unexpectedjs/unexpected/blob/52ce0939df18ad2e548e09d061a55b779677ca91/site/build.js#L81


Yup, that was my old stuff (it was commented since MaterializeCSS)


This seems like a somewhat odd constant value.  Coudl this be a problem if someone actually names something "*default*"  (i.e.with a string literal) somewhere in their code?


Enfin il faut en fournir un par défaut dans www (pour éviter les 404 et le déclenchement de silex à chaque requête)


could consider pulling this out into a utility function


Comme ça on supprime tout mon `responsive` :innocent:  ?

![dogscary](https://cloud.githubusercontent.com/assets/1492516/6495145/94f7ce2a-c2c7-11e4-9af3-e6de6d85d180.gif)


Alright alright. :) Just checking


I try to keep `master` a clean copy of the upstream. That's why there's the separate `dist` branch.


Disregard.


Could you comment what original code this case handles?


88734010-AFN-horizontal-openings (lgu1234) - 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-78.48%25-red.svg)


crap. I'll revert.


This is still incorrect actually.  The correct version would be `info=$($file)`


With just the top 200 passwords, but all the 1500 hashes, we would get 444 guesses.
Test duration (including re-run ot .pot file test) would be 1m59s on the same system, so not much of a gain if we'd just reduce number of expected cracks and reset dictionary size to 200.
(I would have expected a higher speedup for reducing dictionary size from 1555 to 200.)

But apparently, almost nobody runs `./jtrts -type full`, so I don't care too much about the run time here.


was this pushed to maven with 1.0.3?


lol


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

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


88734010-AFN-horizontal-openings (lgu1234) - 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-78.4%25-red.svg)


@bfontaine. You're correct I'll update the documentation accordinglly.


This section isn't correct; you're not outputting the http header


remove commented code.


Yup :+1:


a new baby alone can do that 😜, add a job and a LOT of time coding for the community and im not surprised, ill submit that pr shortly, thx


Also, like we discussed, `$file` will take its parameters from the environment, not stdin


it seems odd to use the name ExportAssignment to refer to "export default".  Could it just be an ExportStatement, with a kind of ExportDefaultStatement or ExportEqualsStatement?


Ideally, you would refactor this section of code so that the same code is used to generate http headers whether the contents is determined statically or dynamically.


This is no longer present in the final code.


Again, not present in the final code.


fix 9c16351


Hi, Magnus


can you comment here that the name of the member is actually intended to be 'default', and that it needs to be that because consuming code can do "e.default" or "e['default']" to access it?


It should probably be ExportDefaultDeclaration.


We have a helper that gets the best child node to report an error on for a given node.  We should just call that here.  That helper should see if the thing has a name, and if so return it, otherwise return something better (or the whole declaration if necessary).  That way all code benefits from the helper.


I'm not sure what this means.  can you clarify?


Curious: why are you saving the profile to the db? Seems unnecessary. Also, you don't need to load any task info from the db in the memory processing module, as the Processing class has access to task information. See the following:

https://github.com/cuckoobox/cuckoo/blob/master/lib/cuckoo/common/abstracts.py#L601


:metal::angry::metal:


Can you comment in the code the distinction here between why we treat an identifier and non-identifier differently?


Dammit!! I'm going to amend the commit to hide my shame.


Not according to [this](https://github.com/iojs/website/blob/template-build-refactor_i18n-footer/content/en/template.json)


We have one already? What's it called and where is it?


obviously :P also it will not work with strings, at least without Java 7.


There are comments in the final code. Would be better for you to start there.


, Image


Can you add comments for the different cases that can flow through here.  Which export are you marking as referenced in each of the cases.  And how does hte code in the body of this function accomplish that for you.


This micro-optimization has little sense, because
1) time.After() is basically implemented as return time.NewTimer().C
2) I believe golang runtime doesn't create a goroutine when tc expires, so tc.Stop() on the line 160 does nothing. See http://golang.org/src/time/sleep.go for details.


@rwjblue mind doing a quick scan before I PR? Want to make sure this matches your views from our conversation today.


This seems odd.  THe function is called markImportSymbolAsReferenced, but we also seem to check things as well as mark them.  Is there a reason we need to check things here?


It does seem odd, but it's also what Csc and Vbc where already doing. I'd like to come back to this later and decide what to do for all clients.


Again, there are comments in the final code.


This must be done only if dbmail.conf allows it. We don't know if it's enabled or not on the PHP side, so it's probably better to just skip it altogether -

It there's no header name matching, just skip it.


The problem is salt related.  To get the first 200 takes half the time.  Then the next 400 takes the next 25% and the final 900 are found in the last 25%

Now those timings are not 'exact',but probably not too far off.   Run using -v -v to get the command line, then simply run JtR.  You will see things just crawl a while, then speed up a bit, then finally, run pretty fast.

We could also speed things up a bunch by only doing 700 or so for these super slow formats. Making it half the size should speed things up AT LEAST by a factor of 4 (possibly more).


I don't think we have any consistent convention about that


Btw, THANKS for finding this simple problem.  I just did not have time today to dig into this one.


Why only for < ES6?


sdfsdff


I know, its why I asked about some variables to possibly remove.
Yet one more var doesn't make sense to me.
I think we need to streamline the branding related vars and sort out the ones unneeded.


Thanks. I'll probably force push sometime soon to get this back into sync with the upstream.


is this condition used elsewhere?  We might want to consider something like "isNonEmittedExpressionContext" or something...


Okay? 1. Why is that relevant, and 2. Literally no one has an excuse to be using Java 6 or earlier.


I'm fine with waiting but can we file an issue to look at this?


, Image


TeamCity Akka.NET :: Akka.NET PR Build [Build 102](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=156&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Why does ExportAssignment not indicate that a module has exported members?


GG @Zenith-One


can you add a comment as to which case this is.  Or, explain why an export decl with an export clause doesn't count as an exporting members from a module sym.


Хэх.


Tror de bare aksepterer PRs mot dev-branch, ikke master! :/


We should be able to retry streaming queries if they haven't started returning results. Can we detect that somehow ?


I think i'd prefer an assert over just returning undefined.  If you have a class/function, it better have a name if it doesn't have the 'default' modifier on it.


Looks like you added some duplicate lines here. This version of the code now gives me a syntax error!


why this better than use openssl/md5?


It was in the original, but I agree that "apps" covers games.  I can update this in the agreement


Ah, I pulled a dumb. I assumed the two following words were part of the iojs/website url for some reason. My bad.


Hey Mike,

I don't have a strong enough understanding of how Glass will be caching and resolving items, so this might be a bit of red herring, but it looks to me like you might also want to maintain the Sitecore Language name when generating a unique key? Otherwise, perhaps you could append that to the unique key in the CacheCheck task?


shouldn't this use `triggerRect.bottom` rather than `.top`?


Thanks, I'll change.


if this is for error reporting, then we should use the helper function we have to pick the best error location for a node.


Let's discuss this one.


Yeah, but normally they're near the aria at the end, as per @mdo's code guide.


We don't actually use this in puppet acceptance


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


In CommonJS and AMD, the default export becomes the actual exported instance, thus hiding any regular named exports. But in ES6, the default export is just a regular named export with the reserved name "default".


:( kk


Could you explain the purpose of this change?
If requestsChan has free slots, then the case on the line 148 will be executed. Otherwise goroutine will block in the select on the line 150, which doesn't differ from the original outer select on the left side.


yes please

woot


Actually it's about working on dejavu recognizer...


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


Don't think it occurs anywhere else.


Hi @tanyalang - looks good.

Few tips:

1. Instead of writing `if ($a > $b){`, I'd suggest `if ($a > $b) {` (with a space between the control bracket and the curly bracket.) It's easier to read. http://framework.zend.com/manual/current/en/ref/coding.standard.html#if-else-elseif

2. It looks like you're using 'tab's for indentation on lines 13 - 39. I'd suggest using "4 spaces" instead of a tab. You can configure Notepad++ and most IDEs to do this by default when you press [TAB]. The advantage of four spaces over tabs.

Links: http://paul-m-jones.com/archives/2312 http://programmers.stackexchange.com/questions/57/tabs-versus-spaces-what-is-the-proper-indentation-character-for-everything-in-e https://github.com/php-fig/fig-standards/issues/175 https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md http://framework.zend.com/manual/1.12/en/coding-standard.php-file-formatting.html

The main advantage is that this way, it will look the same in all operating systems and all IDEs.

3. Indentation with If/ElseIf/Else and all control statements is very important for readability. See below for the same thing reformatted and consistent with modern coding standards.

**Why are coding standards important and a good idea?**

 * By writing your PHP in a 'standard' way, it will be easier to read and other developers will find it easier to read.
 * It will make it easier to work collaboratively with other PHP programmers in the future.
 * Your code will be better respected.
 * Most professional programmers have had the experience of coming back to a project many years after originally creating it. If you don't comment things well and follow a clear standard, it can be very difficult to work out what it used to do and 'fix' or 'maintain' the bugs.
 * It will be consistent with PSR-2 and Zend Framework 2 - which is sometimes necessary as part of automatic testing that happens in some PHP projects (such as with 'TravisCI').

So, to summarize all of the learnings, here is how I would re-write the same code:

````php
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Control Statements</title>
</head>

<body>
<?php
// Initalize two variables: $a and $b
$a = 15;
$b = 10;

// Test whether $a is bigger than $b
if ($a > $b) {
    echo "A is Bigger than B";
} else {
    echo "A is not Bigger than B";
}

echo "<br/><br/>";

// Initalize some variables
$x = 20;
$y = 25;
$z = 15;

// Check whether $x is less than $y or equivalent, or neither.
if ($x < $y){
    echo "Y is Bigger than X";
} elseif ($y == $z) {
    echo "Y is equal to Z";
} else {
    echo "Y is Bigger than Z";
}

echo "<br/><br/>";

// While Loop
$i = 1;

while ($i <= 10) {
    echo $i++;
}
?>
</body>
</html>
````


rax retry tests


Yes, I think we should make a test using a png - the same as used for Facebook and check if it is picked by crawlers.


how do we tie this to issue #526


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


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