Big bugfix! Fixed many bugs with the new Media display,
and refixed old code to match up with the fresh code.
Fixed these (and more): • video play • file deletion • auto play audio/video • add media to media list
• searching & sorting media list • some optimization
Now it adds only the newly downloaded file to list, instead of refreshing the folder.
The "options-settings" now always work as default for the download.
Possibly fixed the slowdown during running the program, which I experienced.


Is this not equivalent to b &= ~ei.pinnedPieces[Us]; ?


Forgot a using... so this is broken. Will be fixing in a future commit


@ns827 `src/pocketmine/utils/BlockIterator.php` is changing :P

Nah, it's so it's more precise to the real implementation. This will throw less exceptions


93467340-AddLCCdataset2014 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1647 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-100.0%25-green.svg)


Already checked this at line 9.


Is the goal here to create something that will be compatible with Kafka 0.8.2+?

Yes (it is actually compatible with 0.8.1.1+), the kafka team would be [deprecating storing offsets in zookeeper](http://kafka.apache.org/documentation.html#distributionimpl) so we have decided against implementing storing offsets in Zookeeper.


I don't think that these test case test what you want to be testing because of how TR subtyping works. I think you are trying to show that the split out cases with logicial information in the filters of `fl<=` and `fl>=` are correct, but these tests will pass even if you remove those cases and only leave the `(comp -Fl)` case (aka `(-Flonum -Flonum -> -Boolean)`.

The best way to test this would be to test the actual filter result instead of trying to test what happens based on an `if` check. But to do that you cannot use `let` to introduce the variables because they will be out of scope in the expected section. The solution is to use a lambda.
```
[tc-e/t
  (lambda: ([x : Flonum-Zero] [y : Flonum]) (fl>= x y))
  (t:-> -FlonumZero -Flonum -Boolean : (-FS (-filter -NonPosFlonum 1) -top))]
```

Or a more complicated form to cover all the comparison operations in one case
```
       [tc-e/t
         (lambda: ([x : Flonum-Zero] [y : Flonum])
           (values (fl>= x y) (fl<= x y) (fl> x y) (fl< x y)))
         (t:-> -FlonumZero -Flonum
               (make-Values (list
                              (-result -Boolean (-FS (-filter -NonPosFlonum 1) -top))
                              (-result -Boolean (-FS (-filter -NonNegFlonum 1) -top))
                              (-result -Boolean (-FS (-filter -NegFlonum 1) -top))
                              (-result -Boolean (-FS (-filter -PosFlonum 1) -top)))))]
```


`docs/api/file-object.md`  has an error at line  `console.log('File you dragged here is', file.path);`
`file.path` will print the path of the current webpage instead of the file you dragged in.
ps: win7 + electron0.25.1


Einrückung


Why not preserve this generic stuff?


```diff
diff torbrowser-31.6.0_p408.ebuild torbrowser-31.6.0_p450.ebuild                                    torbrowser-overlay/git/poncho
--- torbrowser-31.6.0_p408.ebuild	2015-05-02 18:16:50.287481003 +0200
+++ torbrowser-31.6.0_p450.ebuild	2015-05-02 18:16:50.287481003 +0200
@@ -12,9 +12,9 @@ if [[ ${MOZ_ESR} == 1 ]]; then
 	MOZ_PV="${PV/_p*}esr"
 fi

-# see https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/gitian/versions?h=maint-4.0
-TOR_PV="4.0.8"
-GIT_TAG="tor-browser-${MOZ_PV}-4.0-1-build1"
+# see https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/gitian/versions
+TOR_PV="4.5"
+GIT_TAG="tor-browser-${MOZ_PV}-4.5-1-build3"

 # Patch version
 PATCH="${MY_PN}-31.0-patches-0.2"
@@ -108,7 +108,7 @@ src_prepare() {
 	epatch "${WORKDIR}/firefox"

 	# Revert "Change the default Firefox profile directory to be TBB-relative"
-	epatch -R "${FILESDIR}/tor-browser.git-bccc144c21f6069a294038a6559ba46ceb8da2a2.patch"
+	epatch -R "${FILESDIR}/4.5-Change_the_default_Firefox_profile_directory_to_be_TBB-relative.patch"

 	# FIXME: https://trac.torproject.org/projects/tor/ticket/10925
 	# Except lightspark-plugin from blocklist
@@ -177,7 +177,7 @@ src_configure() {
 	mozconfig_annotate 'torbrowser' --libdir="${EPREFIX}"/usr/$(get_libdir)/${PN}
 	mozconfig_annotate 'torbrowser' --with-app-name=torbrowser
 	mozconfig_annotate 'torbrowser' --with-app-basename=torbrowser
-	# see https://gitweb.torproject.org/tor-browser.git/tree/configure.in?h=tor-browser-31.3.0esr-4.0-1#n6401
+	# see https://gitweb.torproject.org/tor-browser.git/tree/configure.in?h=tor-browser-31.6.0esr-4.5-1#n6395
 	mozconfig_annotate 'torbrowser' --disable-tor-browser-update
 	mozconfig_annotate 'torbrowser' --with-tor-browser-version=${TOR_PV}

@@ -220,12 +220,7 @@ src_install() {
 		>> "${S}/${obj_dir}/dist/bin/browser/defaults/preferences/all-gentoo.js" \
 		|| die

-	# Disable adobe-flash by default
-	sed -i -e 's:pref("plugin.state.flash", 1);:pref("plugin.state.flash", 0);:g' \
-		"${S}/${obj_dir}/dist/bin/browser/defaults/preferences/000-tor-browser.js" \
-		|| die
-
-	# see: https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/gitian/descriptors/linux/gitian-bundle.yml?h=maint-4.0#n148
+	# see: https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/gitian/descriptors/linux/gitian-bundle.yml#n150
 	echo "pref(\"general.useragent.locale\", \"en-US\");" \
 		>> "${S}/${obj_dir}/dist/bin/browser/defaults/preferences/000-tor-browser.js" \
 		|| die
@@ -273,17 +268,15 @@ src_install() {
 	dodoc "${profile_dir}/extensions/tor-launcher@torproject.org.xpi"
 	rm "${profile_dir}/extensions/tor-launcher@torproject.org.xpi" || die "Failed to remove torlauncher extension"

-	# Force remote Tor check since the control port might not be available
-	echo "pref(\"extensions.torbutton.local_tor_check\", false);" \
-		>> "${profile_dir}/preferences/extension-overrides.js" \
-		|| die
-
 	insinto ${MOZILLA_FIVE_HOME}/browser/defaults/profile
 	doins -r "${profile_dir}"/{extensions,preferences,bookmarks.html}

-	# see: https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/RelativeLink/RelativeLink.sh?h=maint-4.0#n248
+	# see: https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/RelativeLink/start-tor-browser#n301
 	dodoc "${FILESDIR}/README.tor-launcher"
 	dodoc "${WORKDIR}/tor-browser_en-US/Browser/TorBrowser/Docs/ChangeLog.txt"
+
+	# see: https://trac.torproject.org/projects/tor/ticket/11751#comment:2
+	dodoc "${FILESDIR}/99torbrowser.example"
 }

 pkg_preinst() {
@@ -294,13 +287,18 @@ pkg_postinst() {
 	echo
 	ewarn "This patched firefox build is _NOT_ recommended by Tor upstream but uses"
 	ewarn "the exact same sources. Use this only if you know what you are doing!"
-	echo
+	elog ""
 	elog "Torbrowser uses port 9150 to connect to Tor. You can change the port"
 	elog "in the connection settings to match your setup."
+	elog ""
+	elog "To get the advanced functionality of Torbutton (network information,"
+	elog "new identity), Torbrowser needs to access a control port."
+	elog "See 99torbrowser.example in /usr/share/doc/${PF} and check \"man tor\""
+	elog "for further information."
 	echo

-	if [[ "${REPLACING_VERSIONS}" ]] && [[ "${REPLACING_VERSIONS}" < "31.2.0" ]]; then
-		echo
+	if [[ "${REPLACING_VERSIONS}" ]] && [[ "${REPLACING_VERSIONS}" < "31.6.0_p450" ]]; then
+		ewarn ""
 		ewarn "Since this is a major upgrade, you need to start with a fresh profile."
 		ewarn "Either move or remove your profile in \"~/.mozilla/torbrowser/\""
 		ewarn "and let Torbrowser generate a new one."
```


This gives new warnings on Linux:
```
wordlist.c:15:0: warning: "_POSIX_SOURCE" redefined [enabled by default]
 #define _POSIX_SOURCE /* for fileno(3) */
 ^
<command-line>:0:0: note: this is the location of the previous definition
unique.c:32:0: warning: "_POSIX_SOURCE" redefined [enabled by default]
 #define _POSIX_SOURCE /* for fdopen(3) */
 ^
<command-line>:0:0: note: this is the location of the previous definition

Make process completed.
```


93467340-AddLCCdataset2014 (JasonGlazer) - 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)


reçuu


I've moved if from litescope to uart to be similar with others cores (for example for liteusb, litepcie, liteeth, the wishbone bridge and the software to control it is in the core). But for the uart since it's simple we can eventually provide it with litescope it you prefer and I'll move it to litescope.


BTW: The warnings appear only on my Fedora 20 system with gcc 4.8, not on Fedora 22 beta with gcc 5.1.1.


Not cross-platform filename.
You installing library, not just file. Yous install(TARGETS) or install (LIBRARY).
Read [documentation](http://www.cmake.org/cmake/help/v3.0/command/install.html) and samples before.


Yes thanks, I'll do a review of MiSoC for that point.


Where is installation for header files? Do you plan it?


OK I'll do that.


No, I do get the error on Fedora 22 as well (I did a git fetch earlier, but forgot the pull)


@Glenpl Wyjątek raczej nie.

Pytanie co kto odpala – bo to nie jest aplikacja np na wątkach, żeby wysłać info bezpośrednio do `SapperApp` i ten ubije `Gameplay`.

Ale podpowiada mi to, że robimy chyba błąd, że komunikacja jest bezpośrednio dwustronna: UI odpala coś na Gameplay, a Gameplay odpala coś na UI.

Jeszcze pomyślę.


I don't think it's necessary to keep code that we don't use, it isn't tested and it makes changing the code more difficult. If you want to keep it, feel free to restore it.


awesome ...thanks.


なるほど


Yes. I saw that. But it was confusing because half of the formations had linked units. I put the formation restrictions into a rule so it's clear. I've populated modifiers for minselected awaiting a bug I filed with Battlescribe. Once he updates it, the units will be populated the same way it used to.

Sent from my iPhone

> On May 2, 2015, at 12:58 AM, Amadeusz Sadowski <notifications@github.com> wrote:
>
> I see you've changed formations. Often when I put copies not links of units, I've done that on purpose - those copies had the requirements built-into them. Such as Firebase Cadre requires XV88 teams to be 3-member-full. The copies had min/max appropriately set.
>
> —
> Reply to this email directly or view it on GitHub.
>


This is absolutely gross


Which unicode problems? Do you have a ticker to refer to?


These are declared twice and are causing build problems...


jgjhghgh


This link and the others on this page are no longer working.


It has BUG !

```
  public static void main(String[] args) {
    String html =
      "<ul>" +
        "<li name=hello>hello</li>" +
        "<li>world</li>" +
        "<li>world</li>" +
      "</ul>";

    Document doc = Jsoup.parse(html);
    Element hello = doc.select("li[name]").first();
    System.out.println("hello = " + hello);

    Element nextSibling = hello.nextElementSibling();
    while(nextSibling != null) {
      System.out.println("nextSibling = " + nextSibling);
      nextSibling = nextSibling.nextElementSibling();
    }
  }
```

It will run into infinite loop. (1.8.2)


Sorry to be a nag, but didn't you just made the same statement in line 169? I think this line can be removed completely.


Done


Pilnai įvykdytas #3


That's true, personally I think separating parameters and activations into
separate OOP entities would be cleaner for parameter sharing and RNNs.

Yes, in that code LookupTable is an alias to either the torch one or the
fbcunn optimized LookupTableGPU, which have the same purpose as Google's
Embedding class from the learning to execute code.
On May 2, 2015 5:19 PM, "mszlazak" <notifications@github.com> wrote:

> OK, that is a good reason. However, it does get a bit more complicated in
> a teaching scenario when learning to use Torch 7. To many "new moving
> parts" to understand at first and if simpler or familiar ones work then I
> tend to go that way first.
>
> Also, you used the Embedding package. Does this have the same function as
> the following in create_network() of
> https://github.com/wojzaremba/lstm/blob/master/main.lua ?
>
> LookupTable = nn.LookupTable
> local i = {[0] = LookupTable(params.vocab_size, params.rnn_size)(x)}
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11012447>
> .
>


电话号码的正则应该是类似这样的……
http://blog.csdn.net/yangbobo1992/article/details/8217536


I can't found class Token, I saw you connect lib folder in application.rb, but you not committed her.
could you please commit Token class?


This should be in an "if" block, something like:

```perl
if (!defined $dimobjs->{$_[0]}) {
  my $msg = "FOO can't get SIZE of undefined dimension (RedoDims=$redodims).\nredodims is $redodims\n";
  eval { require PDL::IO::Dumper };
  $msg .= "dimobjs is ". PDL::IO::Dumper::sdump($dimobjs) ."\n" unless $@;
  croak $msg;
}
```


OK it's wordlist.c that has them wrong - it should be `#define _POSIX_SOURCE 1`.

I'm contemplating removing all this from autoconf and just put them in the files instead. But this sometimes lead to problems. For example, `_POSIX_SOURCE` is needed for fileno(3) on Linux but introduces problems on Sparc.


I think exit code is missing^^


The docs probably should have been updated for this change.  This now requires users to set backup on the concat define, since the default value for $backup isn't correct.


Attribute term table & additional cost tables has `addClass('alernate')` so removed and merged together ;)


ack


no, its zero, we only define nonzero exitcodes in place because they have special meaning, look up a few lines!


changement d'adresse


Maybe we could inline 0 beneath all other actions listed. So instead of
```
The following options are applicable to this result:
1: option1
2: ...
Please enter the number of the action you want to execute (0 is no action):
```
we do

```
The following options are applicable to this result:
0: do nothing
1: option1
2: ...
Please enter the number of the action you want to execute:
```
If no options are given besides 0 we could place an if that instantly returns the result (and doesn't print the zero-option), namely 0.


Yep, client hints will be useful. The other polyfill I've seen is cookie-based, which seems like a more viable approach if we were to adopt it for BBC News. I think the problem with client hints -unless they've changed the spec recently- is that the dimension hints are based on the screen size, rather than the document size.

I'm not taking it too far as I'm only supposed to be analysing and measuring at the moment, but I'm trying a couple of experiments by modifying a static copy of /news (see original.html in this repo). I'm using WPT and browser dev tools to get a rough idea of the impact of various things, which should help to give us clear priority when we attack the actual performance work.


Suggesting: Is it possible to get midi_to_hz to reflect the current tuning set by use_tuning/with_tuning ? At present it give the same value regardless of the tuning set.  Would be nice for doing investigations into different tunings. I tried the code below and all the midi_to_hz outputs were the same:
#============================================
#illustrating the different tunings using the beats produced
use_synth :sine
l=[:c,:cs,:d,:ds,:e,:f,:fs,:g,:gs,:a,:as,:b]

l.length.times do |i|
play :g5,release: 0.1

sleep 0.5
use_tuning :equal
puts ":g4 :equal tuning, frequency "+midi_to_hz(:g4).to_s
play :g4,attack: 0.02,sustain: 3,release: 0.02
  puts l[i]
  use_tuning :just, i
puts ":g4 :just tuning, frequency "+midi_to_hz(:g4).to_s
  play :g4,attack: 0.02,sustain: 3,release: 0.02
  sleep 3.04

inc i
end


Thanks for the fixes. Do you use gcc or clang? What compiler flags did you use to get the warnings/errors?


You're right. I didn't see the changes in this file. If they are per user I'll add this directory to the git ignore.


Pars! Et ne reviens jamais ! ;-)


Yes, indeed. I was writing something then I changed it. Thanks! We need to talk about these conventions. Maybe tomorrow.


I think we could but that would need a discussion with pros and cons, I'd rather have this comit for now because it makes it better than before and discuss this later. If you want this discussed please open a bug for it. (Note that that bug is different in purpose than the one adressed here.)


Esta linea no es muy eficiente


Me parece buena idea hacer esto


Related question - does the tuning system work with none-integer MIDI notes such as `50.34`?


There is a Runtime error that happens here (runtime error: Cannot read null.flags)
It spams the logs every time NanoUI is open, it's from custom_state, or lack of in this situation.


SpeakLogに書くべきこと


ack
I don't even understand why python calls a test class a "test case". In my opinion a test case would be a test function inside the class. I'm somehow dragged to call the classes even "test suites" (or slightly less "test (case) group", dunno).


shard{1,2,N}.domain.com all need separate DNS lookups. I guess it's something we can experiment with, but I'm fairly certain most of the recommendations are to have only 2 shards.


I'll put the word abstract in the documentation inside quotes. Although there is no abstract class currently in typescript I want this prefix to remind us that such a class is not to be instantiated.


On a second thought I think you're right. I'll leave it in the places it may conflict with other uses.


93467340-AddLCCdataset2014 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 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-5.16%25-green.svg)


test is shorter and thus better. Yeah naming o unittest lib is strange.


ack, especially the new editor argument handling is nice


I don't know yet. (NullExpression)
I don't see why docs should always remain in the context of a file. In general this seems to be a good idea but I don't think it's a must.


Since we have renamed the record, now it makes sense to rename:

Value -> Key
FuncTextId -> TextId
FuncId -> Id


Rename to Funcs as well


I am not a big fan of the _total as we go approach_, but this is pretty well done. If we rename `updateTotals` to `subtotal` the class works the same. Most people don't approach it that way.


Nit: Should be using a typehint here.


Yes, Meta key FF for mac return keycode 224.

https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyCode

Em sábado, 2 de maio de 2015, Bob Knothe <notifications@github.com>
escreveu:

> Ney,
>
> Thanks for the code contribution.
>
> Is this FF for Mac?
>
> It will be a week before before I can test & merge.
>
> Thanks Bob
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neyestrabelli/autoNumeric/commit/a95ddff9a1fe85ab289e80c400cb2b082bb9bef1#commitcomment-11011295>
> .
>

--
Ney Estrabelli
http://www.neyestrabelli.com
meu blog sobre games...


it should not always remain in context.
but when the context is external and not yet even defined will we remember to modify the docs if and when
they become irrelevant ?


меня подставили :)


LGTM


Įvykdytas #4


I was looking for implications on setting it to an empty string and it seems ok.

Rubygems [tries to load the gemdep file upon initialization](https://github.com/rubygems/rubygems/blob/v2.4.6/lib/rubygems.rb#L1244) then [it fails silently](https://github.com/rubygems/rubygems/blob/v2.4.6/lib/rubygems.rb#L1049)

Skimming through rubygem's reflog it seems behaves like that since the beginning, so it should be compatible with all rubygem versions.


ack


thank you for your timely fix!


We appear to be deprecating the ability to output the interim styles here to disk. If this is the case, are we confident third party tools (grunt-critical and so on) will be able to continue offering an end-to-end workflow without relying on this at all?


I much prefer this interface. More terse and reads better. Nice work.


seems a bit odd, doesn't it?


:+1: for switching away from the interface just using callbacks.


yeah definetly better, ack


Nice! I tend to make aliases for these things, but a Makefile lets everyone on a team leverage it, which is great.


Hierdoor gaat de tekst niet achter die knop om shit weg te halen.. is mooier imo


Much redundant.


I would probably call this `buf` rather than `b` since `b` tends to be `byte`


I think that maybe the comment seems a bit useless because the API might not be quite right.


Actually fixes issue #11 :-/


Is there a reason we're splitting this across two lines, rather than just saying `_.invoke(cssArray, 'toString').join(' ')`?


I think that `e` is typically used when an error is the function receiver (e.g. `func (e *SystemError) DoStuff()`), whereas when errors are being handled I generally see them referred to as `err`.


Okay want to be sure that zero is correct^^


J'adore cette astuce. <3


smallest of nits. We include a newline before each `critical` call in the tests before this one. Could drop one in for consistency.


well "exiting gracefully" implies zero retval


same as above


Test.


This is more typically written as

```
if err == io.EOF {
    return ErrInvalidSpliceFile
}
return err
```

https://golang.org/doc/effective_go.html#if


@diaoul, so I am setting up a testing/beta package for Mono 4.0.1. I set it up as you can see above with a mono-testing ```PKG_NAME```. This works. However, I now realize that it would be preferable to use the same ```PKG_NAME``` (mono) so that users testing Sonarr etc. will not have to install the standard Mono package in addition (as it is a requirement.) Thy would also have to start Sonarr manually each time because Sonarr calls on the standard Mono installation/package.

My question is: Is it possible to use the same ```PKG_NAME``` (mono) with a different folder in SPKSRC (mono-testing) and the ```BETA``` flag set? If I did that would SPKSRC REPO only give the beta version of Mono to users who have flagged that setting? I would think that would be the optimal set up if it is possible. This would also allow the beta package to be updated once the master package version exceeds it. However, I do not think currently SPKSRC allows for this. Does it?


@dr-bean


adding now...

2015-05-02 19:18 GMT+02:00 Makman2 <notifications@github.com>:

> Parameter description missing.
> Especially for intended overrides this is important^^
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/531afae9d08575bddade75a7f995b55fa6611cd2#commitcomment-11012630>
> .
>


あとで修正する


Why did you switch to jQuery one? something wrong with es5 verison?


Oh, and mobile media queries.


What if a `note` had a `String()` method?


当年我发现 Vocaloid 和 M$ 一样弃暗投明，转用 XML 而不是二进制倍感欣慰。之后我发现 `vsqx` 的 XML Schema 是 404，便开始反向研究它的格式，并首次使用百度贴吧希望扩散。等我大概完成 30% 左右的时候，我在 Vocaloid 根目录下发现了完整的带日文注释 XML Schema……然后就没有然后了……

没想到当时完全独立的事情，居然能和我后来认识的 StarBrilliant 出现在同一列 `Credits` 里，为  Rocaloid 我大 FOSS 事业添砖加五，真是感慨万千啊……


These define _POSIX_SOURCE as well, but are used for generic builds only, I guess:
```
bench.c:22:#define _POSIX_SOURCE
best.c:17:#define _POSIX_SOURCE
para-best.c:14:#define _POSIX_SOURCE
```

These define _POSIX_SOURCE as well:
```
signals.c:20:#define _POSIX_SOURCE
status.c:15:#define _POSIX_SOURCE
unique.c:32:#define _POSIX_SOURCE /* for fdopen(3) */
```
Not sure what needs to be fixed in core and what is jumbo specific.


#492
ack then


GitHub is inaccurately displaying the number of spaces for the indentation. In Sublime, it should only be 2 spaces per indent.


I think `golint` would complain, as well.


You mean the `*p`?


Ah, and here it is fixed :) Nevermind then.


To answer each in turn:

  >  Is it possible to get midi_to_hz to reflect the current tuning?

It's not straightforward at the moment as it's implemented as part of the `play` method. I spoke with Sam about this and the next step will be to move that code into the `SonicPi::Note` class so that it applies to all notes, regardless of how they are accessed. As a workaround you can access the note value like this for now:

```ruby
use_tuning :just, :a

foo = play :cs5
puts midi_to_hz(foo.args["note"]) # outputs 549.9999999934003
# should be 550.0 ideally but rounding errors creep in with the midi formulas we have
```

This does mean that you have to sound the notes in order to analyse them so to speak.

  >  does the tuning system work with floats?

Errr... no. That would be a bit hairy to work out I think as you'd have to have some sort of continuous space to map the tunings onto. I would suggest that using a float as a note value would bypass the tuning system. WDYT?


I realize you are probably not done, but the `readPattern` probably should be implemented on the decoder, what do you think?


I would put the two ! - conditions at the end. First, it reads more natrually. Second, these conditions are not very discriminative, so it's more efficient the other way round.


It seems you did not understand the usage request() method. Without understanding it, you should not change the return value of build_request_params() method. When the entire content of $args passed to MCP api as parameter in the url, it may fail the api call at the MCP server. Please analyze the request() method in deep to understand the bug. And if you have a question that is not clear in the code or documentation, you can always fire a question to our team. We will gladly help you to find the best way.

FYI, 'LK' stands for 'LIKE' search for video titles. Please do not try to make such changes in the code without understanding the full implications for the actual use case.

Thanks,


an interesting pattern is to store the error on the decoder


Thanks https://github.com/rails/spring/commit/bda1c66630b9dbce3fb51c23d69d463d6f202153


Does this have any negative consequences? I thought that clefs needed to go in the Attribute list?


Actually, I hadn't thought about this when I merged the pull request. I think that it's essential that it handle non-integer values. You'd therefore need to implement the tuning system as a series of functions rather than a lookup table. We should look into ways of achieving this. I'll revert the commit for now.


add bootstrap.js as an global of app


parse users fetched to build as a collection


Nothing driver work with, "1080 skin".. zgemma have same problem.
Only skin 1280 work


Ohhh :) I didn't realize we used it in two places.


It would be good to have a doc comment on these classes explaining what they are for.

Also, why not out them into one object? That way we do not have to duplicate the doc comment, and it's more maintainable to boot.


Cool!  Can we check the location of each clef after conversion to make sure they're at the right place?  Thanks!  This test can probably go into one of the other files in musicxml/ that already has a test suite.  (I know that it does both directions, but good to know.)  Also good to make sure that this works w/ piano instruments (instruments that have multiple staves.)


So you would reference it with `dec.Err` or something like that?


So is SuperH always 32-bit, or it has some other way to change address model? Shall we issue a warning when address-model is specifies on architectures that don't support it?


removed unnecessary code of bootstrap toggle button group


> Also, why not out them into one object?

Can you elaborate? Do you want to join VCShortPrototype with VCShortCasePrototype?


moved moment date format to template file


used toggle button group of bootstrap defaults


hide head of table on mobile


add moment to use in template file


change to the right path of bootstrap js file


Css


I actually think the distinction between passes and non-pass-items is pretty useful and being able to require having a pass (maybe optionally) is also pretty useful. A dependency system could be good for power users but I don't think it's going to be less confusing.

Admittedly I haven't been fielding most of the support for this, but I suspect what's confusing now is that our categories are neither comprehensive nor do they have obvious behavior implications. I'd like to leave some way to indicate "This is a pass" for the most common cases. A boolean field might be sufficient and unambiguous and we can add help text that explains what it does.


ack


No I meant merge VCIntPrototype, VCFloatPrototype and so on. If they are all on one file it's easier to share docs and its also easier to find and replace.


Ok. I thought it was in the good place because there are here some other commands like whereis, that have the same role : saying a precise information about the game.
So I should put it [here](https://github.com/Ombridride/minetest-minetestforfun-server/blob/master/mods/irc_commands/init.lua#L135) ?


Ok. Agreed.


I don't actually feel strongly about this. If you think removing item categories entirely is the best approach, I trust your judgment. I just had some thoughts.


Nice. Same comment on merging into one object as before applies.


其实继续使用 Vocaloid 2 的 SMF 格式也未曾不可。
SMF 的历史比 XML 早很多，而且现有的音乐软件基本都能开 SMF。

官方文档不够详细，有的用词难以理解，所以才有 Oran 撰写的这份文档啰。

> 没想到当时完全独立的事情，居然能和我后来认识的 @m13253 出现在同一列 Credits 里，为 Rocaloid 我大 FOSS 事业添砖加五，真是感慨万千啊……巧合么。

臭味相投……吧。


@opdenkamp @wsnipex I think those two lines with CXX, CC are wrong and should be removed


I like the generalization, but I think we should change PYBV4 and PYBV10 to use this. Then we wouldn't need any board specific customizations.


What is the reason for "companion extends ClassTag"? It smells a bit like over-using inheritance, But maybe there is a hard to reason to do it this way?


sqf contamination ;)


vg devs have now a info to fix this issue, thanks for feedback


The repository picks the latest available release for each branch (stable and beta). The unicity is given by name, version and architecture.

I think it should work although we have no package using that kind of setup. That would also mean that two distinct users may not share the same installation setup for Sonarr and introduces overhead in case of support.
Downgrade of packages is not supported by DSM so once a user has updated hist Mono package there is no going back without uninstalling all depending packages (Sonarr).


you will not need a `revisionTree` module - just create a route like for the doc-editor


@techhelper1 I had a bit of time this morning and got 10.1-RELEASE up and running.


no need for `JSON.parse` in both cases


Can we use here some go style coding to reduce verbosity ?
Something like:

ports := make([]nat.Port, 0, len(portSpecs))
for p := range portSpecs {
    ports = append(ports, p)
}


missing linebreak


`className="visualizeRevTree"`


intendation is mixed, we use 2 spaces


into the right folder please


you can use `path.reduce` for that

https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce

we don't use for loops at all


closes #5


One thing we might want to think about is style consistency.

For example:
- 2 tabs or 4?
- dashes to separate multiple word file names or underscores?
- single or double quotes?

It doesn't particularly matter to me what we choose, but we should stick with whatever we decide to do throughout the project.


do you mean `circle` ?


The irc_commands mod is better, because this is neither a mandatory nor a very used command, it should come as a complement of the default irc commands, and this is irc_command's purpose.


@Diaoul, thanks for the clarification! Which do you think is the better approach to take?


it is easier to render react components now, see

https://github.com/apache/couchdb-fauxton/commit/02a5ee3f0067432d3abf4fd683397ed6d66137ab


I would vote two tabs, single quotes, don't particularly care about file
names but I prefer dashes. I'm a lousy typer so I avoid hitting shift
whenever possible, ha.
On May 2, 2015 10:53 AM, "Andrew" <notifications@github.com> wrote:

> One thing we might want to think about is style consistency.
>
> For example:
>
>    - 2 tabs or 4?
>    - dashes to separate multiple word file names or underscores?
>    - single or double quotes?
>
> It doesn't particularly matter to me what we choose, but we should stick
> with whatever we decide to do throughout the project.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andyjac/bitmap_transformer/commit/f671095e75b9d21271779e84de120b5c63ab8f6a#commitcomment-11012753>
> .
>


I love this idea. Matt and I had talked about something along these lines early on and I had forgotten.


move that to an action


all async stuff should happen in actions


Because of ignorance. I don’t entirely understand the difference, and I was mostly doing a quick and dirty implementation to use during level testing. When you create a new script, it has only two methods by default, and one of them is Update( ).

If you have a reason to change it, you have my blessing. You won’t break anything.

=====================

Jesse A. Janowiak

Virginia Tech HCD Student

jajanowiak@vt.edu

On Sat, May 2, 2015 at 8:48 AM, Brandon Gilliam <notifications@github.com>
wrote:

> For the changes in CameraMovement.cs, why did you use the Update function instead of the FixedUpdate function?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/compscialien/StudioProject/commit/e000ad7ceeb72a53f6702576beefa6d379a441c2#commitcomment-11011751


please use reduce


Doh. My bad.


I agree, if I have extra time tomorrow I might add a separate commit that defines these for said boards and does away with the board-specific defines here.


You missed login server's classpath


I'm also thinking I should move the `defined(PYB_USART_2_RTS) && defined(PYB_USART_2_CTS)` check here.


After some thought, it obviously didn't make much sense to do what I suggested in the current repo, like you said, so I put things in [purescript-any-db](https://github.com/epost/purescript-any-db). I'd got as far as you did, but that's not quite the API I'm after.


Almost, but not quite. If you enter an illegal command the "game" ends, instead of prompting you again.


unused var


To avoid reallocation of s, you may want to explicitly initialize s with make(type, lenght, capacity) specifying 0 length and appropriate capacity.


should probably be
```
+        except FilenameWithTrailingSlashError, e:
+            self.to_stderr(str(e))
```


Loooong overdue, sorry. Please let me know if you guys think we can substitute this with the one I drew below.

 <https://github.com/w3c-html-ig-zh/clreq-fig/blob/master/ruby/inline-zhuyin.svg>


Hah sounds good to me :)


version 0.5 -all the file types has been change except one -cleaning some unuseful code or comments


Things known to be local absolute paths should be quoted using `quote_local_unicode_path`.


@pysailor The form tag doesn't really have a function on this page, does it? Shouldn't we remove it?


(On Windows this strips the `\\?\` which looks ugly and surprising otherwise.)


What's the point of adding a new test case? Do only ONE THING at a commit.


@danott yup. totally an option.

I like that the shape with the closure matches the shape and mental model of the documentation. But it's true that some prefer not to see or write IIFEs. For me, passing a class definition just feels weird and it makes that opening line a little difficulty to interpret.


Do not dump the version by 0.0.3.


@pysailor Note that the method with the dfn tags is the one that doesn't allow for block level elements such as paragraphs and also no lists. Only inline ones such as strong/emphasised or hyperlinks.


You are missing Game and Monsters part.


More idiomatic way of saying this: "Use ``--enable-css/--disable-css`` options to switch styling"


Why remove this line?


I see you copy&pasted a lot parts of https://github.com/neojski/visualizeRevTree/blob/gh-pages/visualizeRevTree.js

any reasons not to use their code?


Is there any reason why this fix is not yet in the 2.1 branch? I just spend a half an hour trying to figure out why I was getting an error on my freshly updated 2.1.8-pl1 environment when loading the Contenttype translations and was about to make a new pull request identical to above when I found out that master <> 2.1.8-pl1 on this.


"use `~/style.css` instead of builtin stylesheet".


so this should be
```
+                    raise FilenameWithTrailingSlashError(source_spec,
+                                                                                quotefn=quote_local_unicode_path)
```


The arrow character for some reason renders as an asterisk on my computer. Is that just me?


"export README.html embedded with `style.css`"


but this is fine as it is (using `quote_output`).


Again, do ONE THING at a commit.


Otherwise LGTM.


Nope; it's something about how the Unicode char is processed by MonoGame/DirectX. I'll try to figure out what the problem is when I get the chance.


Listening at all interfaces by intention. Pick available port number by intention. Do not change this.


Explain why change this?


New line before a method per coding style.


Like this? Or I have to count all the cases and if all of them fulfill the condition should I output the ERROR message then?


The intention of special route `__moo__` is to avoid shadowing user's own static files. Even if using custom CSS, it should following this same rule.


As we only need 'user_data' and CV_ErrHandler, which is no problem to have initialized before calling CVSetErrorHandler(), I really don't see why the error if  having them in the CV_data. The the remaining functions would simply get initialized as they are processed. Do you have an example of it, or does it happen always just by putting it in the CV_data?
Also, as you mentioned, the 'user_data' should be set earlier to err_data in case an error in processing the remaining CV_Set* initialization functions occurs.
Other question is whether that 'user_data' would be the same for errors as is for the computation. Maybe a new 'err_userdata' option to add, which could default to 'user_data' if not specified?


Very strange issue... why I get 'invalid-email-address' ?


Make each subrule a sentence.


The righthand side before:
![image](https://cloud.githubusercontent.com/assets/243627/7442211/216fb414-f110-11e4-81f6-c3b7dd0dc2d1.png)

after:
![image](https://cloud.githubusercontent.com/assets/243627/7442207/e739979c-f10f-11e4-8686-9d281674afd5.png)


semicolon after this line, full stop after the next.


The original `style.css` is ~700 loc. There is simply no easy way to write a custom CSS from scratch. The most common use case of custom CSS would be overriding some styles of builtin CSS. We should provide a command to print builtin CSS to the user. For example:

~~~bash
$ moo --print-css
~~~


done testig


where the behaviour of Tahoe-LAFS v1.10.1


@dqminh, docker daemon will rely on libnetwork for anything that is network related. The port allocation will be a concern of the network driver libnetwork is using. Docker daemon should not worry about sorting the ports.

Please take a look at https://github.com/docker/libnetwork/pull/102.

I am thinking your fix should go in the network driver code (divers/bridge/port_mapping.go).
Note that libnetwork is not going to rely on nat pkg, so hopefully your sorting job will be easier and look less convoluted.

Maybe your change can go in now in nat pkg, but then it will have to be reworked in bridge once libnetwork is in.

What do you think ?


I reverted this commit in https://github.com/agda/agda/commit/56417342a64b46fe358865034dc09800c2b293ec because `--enable-tests` is required.


Huh, well I'm not sure then. My only ideas (and these are all kinda far fetched) are: check the files in your copy of the plugin to make sure you pulled this (I doubt this is the issue). Restart vim if you have kept it open since you last pulled. Otherwise, I'm not sure.


ditto for all the other version references


Update


should probably just kill this.


This is an .rst file, use `typewriter` style for commands.


Next step?: Alter resource format to include an array of tags, instead of separate lines for "weapon, food", etc. Have resource reader produce the HashSet and store it with the Item Model, so you only create each HashSet once, and also so you don't need all the "tags.add(...)" calls in the item constructor, just pull the whole set from blueprint instead.


(I usually still include double quotes outside the typewriter text if that reads better.)


Make each subrule a sentence here as well.


First point would be the size of the PR this is in, second is there is a large delta between `release/2.1` and `master`, and third it had been there for such a long time it probably didn't occur to me at the time... It was a fairly intense weekend of coding.


Ah! I looked at this for about 30 seconds, but didn't see `eo_get_template_part()`. C'est magique!


А есть юридические препятствия этому?


File arguments should not have trailing slashes; in the case of directory arguments, trailing slashes are ignored.


Made changes to challenge.js file for "Booleans" checkpoint assignment.


(Is that true also for Tahoe-side paths?)


Otherwise LGTM.


Thanks. I'm away for the weekend, but will test when back.
I've cherry-picked this to newclock4.
@MilhouseVH Any chance you could test this?
Try first without this and with dvdplayer/mmal use FF x16 or FF x32. You probably will see seek bar jumps backwards occasionally.
Does it progress monotonically with this commit?


Bump


I also just found the issue. Just installed it via maestro a couple hours ago. docker-py was installed via pip with the buggy code still in it.


Ok I don’t mind the revert, but I would suggest you’re being too ambitious in saying that we need to model tunings as continuous spaces. We’re going from a simple array lookup to a crazy geometry/topology space that I wouldn’t want to think about.

I understand that it’s valuable for the API to remain consistent so I would suggest something like:

```ruby
case
when n.is_a?(Integer)
  # apply tuning as normal
when n.is_a?(Float)
  lbound = apply_tuning(n.floor)

  decimal_part = n.to_i - n
  n = lbound + (lbound * (1.0 + decimal_part))
end
```

I don’t think there’s a practical use case for having *exact* mappings of tunings to continuous spaces that respect all the topologies etc. Even if there is a practical use case, it’s very much a special interest one that would require research which we could always retrofit as a breaking change if it proved to be worthwhile.

I feel quite strongly that, if the aim is to keep a consistent API, we need to do something pragmatic here rather than going into uncharted territory for the sake of being mathematically nice.

I’ve looked at SuperCollider and I don’t think they address this in their Scale and Tuning classes. You can specify cents but that’s as much control as you’re given.

> On 2 May 2015, at 18:30, Sam Aaron <notifications@github.com> wrote:
>
> Actually, I hadn't thought about this when I merged the pull request. I think that it's essential that it handle non-integer values. You'd therefore need to implement the tuning system as a series of functions rather than a lookup table. We should look into ways of achieving this. I'll revert the commit for now.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/samaaron/sonic-pi/commit/516c80058cef05e391f4ffc076cb1719b7e3cc51#commitcomment-11012678>.
>


KEY_FRACTIONALMETRICS is definitely an improvement although with the ``Afterlife`` sample above (and it might just be my :eyes:) I think it looks ever so slightly blurry, especially the ``r`` and the ``l``. To me when the font size is 20 with just the anti-aliasing hint is the clearest. Ultimately I think we are at the limits of Swing text rendering so will need to just accept that the text will not look as good as on the original card image.

> It's even working wonders on small text - Just look at Animate Dead, it's readable!

although the line spacing seems a bit uneven, it seems to group them into lines of two...
![screenshot00135](https://cloud.githubusercontent.com/assets/2443519/7442252/79821728-f101-11e4-9fe5-368c24486534.png)


93467340-AddLCCdataset2014 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1136 of 1136 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.79%25-green.svg)


Not changed by this commit I know, but this shouldn't be a bare assert.


issue #1


Maybe std::swap(pVibration->wLeftMotorSpeed,  pVibration->wRightMotorSpeed) ?


Can this be written
`for (auto& e : findSet(hash))`
@HowardHinnant (or anyone)?


I guess you forgot to remove the `debugger` statement


Missing
`std::sort(times.begin(), times.end())`


Why you change from 'jscs-fixer' to 'jscsfixer'? I get 'undefinet' three times because in settings group has name 'jscs-fixer'. After I change it back your plugin become working.


True, on the other hand it shouldn't be a problem to ignore an extra (unneeded) argument and we wouldn't have to care about it... But that's probably only rhetorical question, anyway...


This looks like a system configuration issue, it should not be required by Builder specifically. For example, when installing into a non-standard system prefix.


TeamCity Colore :: Main Build [Build 53](http://tc.sharpblade.net/viewLog.html?buildId=574&buildTypeId=colore_mainbuild) is now running


TeamCity Colore :: Main Build [Build 53](http://tc.sharpblade.net/viewLog.html?buildId=574&buildTypeId=colore_mainbuild) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 112; number of failed tests is too large: 1 (new) Build time: 0:0:55
### Failed tests
```
Corale.Colore.Tests.dll: Corale.Colore.Tests.Razer.Keyboard.Effects.CustomGridTests.ShouldSetProperColorsWithArrCtor: <no details avaliable>

```


This looks like an old copy of the file (this repo is not as up to date as https://git.gnome.org/browse/gnome-builder). Additionally, it follows the template from other GNOME projects (which may be a good idea to convert them to markdown), but doing so will require build system changes too so that `make distcheck` still passes.

If we, (GNOME in whole), want to switch to markdown, we should probably make a wiki page for it, and the rational, and setup GNOME Love bugs.


And done!

	https://pypi.python.org/pypi/nlup/0.5

> On May 1, 2015, at 6:12 PM, Kyle Gorman <kylebgorman@gmail.com> wrote:
>
> great, do that and i'll merge. i can submit it to pypi sometime weekend unless you want to.
>
> On Fri, May 1, 2015 at 6:09 PM, Hobson Lane <notifications@github.com> wrote:
> Will change that to 3.3 to be accurate, but, FYI, the parts used by
> DetectorMorse work for me on simple tests.
>
> On Fri, May 1, 2015 at 3:07 PM, Kyle Gorman <notifications@github.com>
> wrote:
>
> > I see the :: Python :: 2.7 field...does it matter that this doesn't
> > actually support 2.7? It's 3.3 or better, I think, as I use some of the new
> > features.
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11008710>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>
>


@Memphiz I haven't got time to look into the details but I think the problem is here:
https://github.com/xbmc/xbmc/blob/master/xbmc/cores/VideoRenderers/LinuxRendererGLES.cpp#L2390

it is assumed that the type of cvref is BGRA with is most likely not the case for Hi10


I tried to do this before but failed; Very cool.


Do we need the `id`?


EDIT : All big main classes declared


тут href опять потеряли)


Update


TeamCity Colore :: Main Build [Build 54](http://tc.sharpblade.net/viewLog.html?buildId=575&buildTypeId=colore_mainbuild) is now running


yes `p` is passed then overwritten, looks like there is a better way to do that.


@MinaHany94 comment id should be passed and you should check that comment already exists


TeamCity Colore :: Main Build [Build 54](http://tc.sharpblade.net/viewLog.html?buildId=575&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 113 Build time: 0:0:56


Fixed.


yes, and you could do something like ```dec.Err = readPattern()```


/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

package simuladorautomatos;

/**
 *
 * @author David
 */
public class Main {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        new JFramePrincipal().setVisible( true );
    }

}


nice!


Hi,

Great work! I'm trying to create something similar in c# mostly for some practice as I'm (slowly) learning it. I'm struggling at the first hurdle and can't even get a response for the command {"INFO":0}. I've tried with hercules, putty, c# and crestron but get nothing! There's no issues with the connection on everything I've tested but I just don't get a response from the hub at all. I would really appreciate a few minutes of your time to give me some tips as to where I might be going wrong.

Thanks,
Simon


This section was more to discuss Sabotage's differences from the FHS, /src is a very special case I believed needed to be introduced first.


@junegunn Yet I did, and I replied. I didn't say it touches a specific config file, I said it touches the configuration in general. It adds binaries download off the net with no validation to a directory in my home directory that _should have only config files that I put there_ or things generated on the system. `install --bin` is not something that should be run without prompting the user for approval.


@junegunn Why remove the links? Also, without links lines 22 & 23 are not needed.


That's exactly what I was referring to in the issue. A very small nitpick is that you convert your byte slice into a string to compare when you could have compared the slice of bytes directly avoiding an allocation.


TeamCity Colore :: Main Build [Build 55](http://tc.sharpblade.net/viewLog.html?buildId=576&buildTypeId=colore_mainbuild) is now running


Why not use an enhanced for loop here?


I thought my eyes were going last night as well - but they do seem a little fuzzy, and I did notice the line-spacing as well. I'll have to have a look at what impact it has on the .leading values. (What I'm using for line-spacing)


TeamCity Colore :: Main Build [Build 55](http://tc.sharpblade.net/viewLog.html?buildId=576&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 113 Build time: 0:0:51


Still, hell of a lot better than what it was!


small nitpick, the constants are usually set at the top of the file


I cannot guarantee this doesn't have any negative consequences, but it don't seem to have so far. When converting m21 to mx the attribute tag is included, see 485406c6a9cc8fc400c305d07baa1d2fd6a70e9d.


Made changes to challenge.js file for "Functions" checkpoint assignment.


I would have stored that as a byte slice to avoid conversion (see issue comment)


Hi,

I've just solved it! I was trying to send the null termination as part of the string but after sending it separately it works.

Thanks,
Simon


исправила и закомитила


I will make a pull request for ShadowNinja's repository.


Thanks for the fix.


Maybe we can loop the loop and use `Request::GET` constant instead of strings? Sure, this will never change, but… just saying.


Sweet new additions! =)


yes actually the fos user stuff would provide partial data here so this should be fine


Если честно, до сих пор не понимаю для чего тебе тут класс `changeable`, обсудим голосом `:TALK`


отлично!


Yep. I normally only skip the PR step for initial readme.md creation (which was 3/4 done when your PR came in). There's no particular work in progress going on here apart from rebuilding whenever VS 2015 updates.


Случайно добавился таб?)


Removed, added any notable differences to README.


Можно было, например, просто `mobile-header index`) Ну так тоже норм


J'aime beaucoup ce nom de méthode haha


Do we need `<span class="sr-only">Toggle navigation</span>`? I've tested it [on bootly](http://www.bootply.com/pz6GsGYf8S) and I didn't find any difference.

If we need, can we internationalise `Toggle navigation`?

Typically you need to run `python manage.py makemessages -l pt`, add the translation to `locale/pt/LC_MESSAGES/django.po`, and run `python manage.py compilemessages -l pt`.


Needs a comment explaining what it does.


--enable-tests is required for some new tests. It pulls in some additional dependencies, which seem to trigger the error. I don't think any of the additional packages is broken per se, but I suspect that we run out of memory when installing them. I think travis also uses "cabal install -j ...", so it does parallel installation of multiple packages which probably eats up all memory. But I fear removing "-j" will then run into the 50min build limit.

That's just my best guess based on the logs etc., haven't looked at it in more detail yet.


Je suis pas certain de bien comprendre pourquoi Case et plein d'autres héritent de Tableau.
À première vue, Tableau UTILISE des cases, des pions, et tout ça, mais conceptuellement, la case n'est pas un tableau.


Agreed, I've removed this section. I'll add a commit adding usage documentation where needed.


Thanks, I have already made the changes.


Good reasons :-)

So when will this be merged into a release? Will it go into the next 2.1, 2.2? I have copied the fix above locally now but that'll be lost when I upgrade again


J'ai une préférence pour "Encapsuler" des listes dans des objets.
Par exemple, Cases.
Qu'est-ce que ça te permet de faire ensuite ?
* Changer une liste pour un autre type, genre une collection, sans affecter "Tableau"
* Ajouter des méthodes dans "Cases", du genre "InitialiserCases(int nombreDeCases)", ensuite "bool PionEstSurEchelle(Pion pion)" qui te retourne vrai ou faux, et plein d'autres cas.
* Autres trucs que j'ai pas pensé, mais bon c'est un bon conseil quand même.


It was not.


Les static, on évite.
Ça s'appelle du "Feature Envy" si tu crois que tu as "besoin" d'une propriété statique. Dans 97% des cas, on peut s'en sortir autrement.


Yeah, I only realized it today. :)


Unfortunately, you can't compare byte slices unless they're nil.


Hésite pas à appeler ça "passerAuProchainJoueur" si tu en ressens l'envie :)
Visual Studio complète très bien automatiquement


J'adore ça :+1:


Yes, well `for (auto& e : *findSet(hash))`.  The "container expression" is guaranteed to only be evaluated once.


Please fix the formatting here. Make sure to use tab indentation.


> creando un rápido, fácil de usar, fácil de instalar agregador

This is calquing the English-language sentence structure, and does not make sense in Spanish. Ugh.


This slice is unnecessary, the bot will truncate the content automatically.


Fixed this section up with your advice.


Remove the ellipses (...), the bot will add them if necessary.


Note: this fixes #6


Je peux paraître "rough" mais c'est vraiment pas par méchanceté !
Être rigoureux dans la critique, c'est ça qui fait du meilleur code ;)
J'ai l'impression que tu es vraiment intéressé, et si je peux t'aider à devenir meilleur, alors c'est

![Mission accomplished](http://i1.ytimg.com/vi/sRQTDPei1co/maxresdefault.jpg)


This word: *chequee*. It does not exist.


Fix the indentation here.


J'ai souvent le même problème, j'aimerais que par exemple les cases soient indépendantes du reste, sauf que pour les placer correctement j'ai besoin de la liste des cases. Si j'ai 9 cases, elles se placent en grille de 3x3.

Je voulais reléguer cette tâche au "tableau", mais le tableau devient une sorte de "manager". Je suis rendu là dans la conception et j'ai un peu de misère à trouver la bonne solution.


or can you? ;)


Pestaña is capitalized (wrong, because things like Title Case do not exist in Spanish), and lacks accelerator.


hint: http://golang.org/pkg/bytes/


Added comments.


_el_ is misused here. Should be _un_.


Again, was this proofread at all?


AdaptiveComfort (hongtz68) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1647 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-100.0%25-green.svg)


Si j'avais un autre commentaire d'avis général à faire, ce serait
"Essayer de séparer l'affichage du jeu"

Imagine que ton serpent et échelles pourrait être en mode "console". Ou bien en mode "Graphique".

Donc tu as un "Core" qui contient genre
``` C#
Case {
 avancer();
 recupererPosition();
 couleur;
}
```

Puis tu as un "afficheur" qui fait genre
``` C#
AfficheurCase {
 Initialiser(Form form, Cases cases);
}
```


ok but please comment on PR diff and not on commits, is old :older_man:


In the off chance we do switch to markdown (I don't care either way), you have to dist README.md or it won't make it to the tarball. (The standard README gets distributed automatically.)


I notice you've updated to bootstrap 3.3.4. Good idea. :-) Is there a reason to use maxcdn instead of cdnjs?


We have a word for that, *Complementos*.


you got me!


Então cara, as classes sumiram


Good idea, thank you @Gael-de-Sailly


fucking stupid bro


> instalación de muchas librerías

*Library* is being mistranslated as *librerías*. Its correct translation is *biblioteca*.


Merci, c'est vraiment très apprécié! C'est rare de pouvoir se faire critiquer, même à l'école, je reçois la note du travail, mais jamais de commentaires. Gêne-toi pas pour être le plus critique possible, je pensais pas que tu allais autant m'aider, merci encore. :)


To be honest I hardly ever use FF - I'm more of a jump-seek type of guy!

While testing a current build (#0501, RPi2, mmal), I failed to see the "jump-back" behaviour during FF, but I did see the seek bar "jump-forwards" ever so slightly during REW x16/x32.

With the addition of this PR I didn't see any significant difference in terms of seek bar behaviour - no detectable jump backwards during FF, but I continued to see the occasional jump forward during REW.

And this could be my imagination, but with this PR, are video frames appearing a little more often/frequently, giving the impression FF is a touch smoother than before? Although it still stutters/pauses briefly, every second or two.

I tested HD and SD material, and as far as I can tell this PR has no adverse effect on existing REW/FF functionality, but unfortunately I can't say if it actually fixes the problem it sets out to fix.


*Command* is not *comando*, but *orden*. Another false cognate.


Well done !
We can now update the "u_skins" skins database without problems, thank you @crabman77 !


This line will cause a different behavior comparing to what we have today. Currently r_wt only affect p_wt calculation, but n_wt is kept untouched. Can we revert back to this? It require a very small change in your code:
1. Remove UpdateNwt method
2. Add a second FindPwt method with the following signature:
    void TopicModel::FindPwt(const TokenCollectionWeights& r_wt, TokenCollectionWeights* p_wt) const;
3. Add a second CalcPwt method as follows:
  void TopicModel::CalcPwt(const TokenCollectionWeights& r_wt) { FindPwt(r_wt, &p_wt_); }
4. Use topic_model->CalcPwt(global_r_wt) instead of topic_model->UpdateNwt()
5. Remove "new_ttm->CalcPwt();   // calculate pwt matrix" together with its CuckooWatch
5. Revert changes in repeatable_results_test.cc
6. Update topic_model_test.cc  (it should be also OK to remove the part of the test that you had to update --- back in good old days this test covered a very important logic in TopicModel; I mean the logic that maintained n_t counters dynamically as user calls IncreaseRegularizerWight and IncreaseTokenWeight; since we removed that logic the test became almost useless).


Yes, and D2L also extends the LUIF.  LUIF allows Lua code and BNF rules, but there is not easy way to allow the Lua code to affect the BNF rules.  But since D2L is Lua, D2L calls can be in a LUIF source, and the arguments to D2L calls are affected by preceding Lua code.


bem, vou commitar de novo pois essas classes existem aqui


What is this debugSSHCommand?


not sure how not passing dependency injection to this new thing that's now not like the others in that regard helps... we have yet to start addressing the options shape problem at al in tchannel.

That work was experimented with briefly, but then tabled for the bigger win of gtfo core EventEmitter.  We will need to circle back to that optimization at some point, but we're not there yet.

For now, it seems a worse option to not have dependencies (like logger, stats, etc) available within one of many related objects.


Why are you running cloudinit?  Why is that needed?


updating ...


Je pense que https://github.com/m-afortin/MonSerpentEchelles/commit/45420feb920a5048d17dbb385d50710dd651cfb2#commitcomment-11013038
Pourrait t'aider à régler ton problème.


Nice commit, just testing..


But `findSet` returns a `shared_ptr`, is the lifetime of that object assured to outlive the body of the loop?


Pass topic_model by const reference:
virtual bool RegularizePhi(const ::artm::core::Regularizable& topic_model, ...)
This makes code more readable, because const& is standard in google code style for immutable parameters. On practice it makes no difference, because our Regularizable interface already gives read-only access to topic model. But still code style is important :)


Please remove 6 new files which are copies of example09 .. example14). Reference the existing files from in python_examples.pyproj.


I did it all again until 20, I had a different text and I can not get it to github, so I uploaded it


Ouaip, j'applique ça dès que possible, je suis dans le rush de fin de session, ce projet-là me bouffe pas mal de temps, héhé.


I made the parser not throw, and you're saying that you want it to throw now? I spent considerable time removing the exceptions and returning concise errors that we report to clients...


Yes.  `findSet` returns the `shared_ptr` by value.  That temporary will be bound to a compiler-generated rvalue reference which will extend the lifetime of the temporary through the scope of the for-loop.  Had `findSet` returned by reference, we wouldn't have the guarantee.


What would you suggest? The pointer itself is not overwritten.


That would certainly be fancy! I'll address that since tracks and patterns implement their printing logic.


@JoelKatz Suggested:
`for (auto& e : findSet(hash))`
:-)


Presse toi pas :)

Le 02/05/2015, Marc-André Fortin<notifications@github.com> a écrit :
> Ouaip, j'applique ça dès que possible, je suis dans le rush de fin de
> session, ce projet-là me bouffe pas mal de temps, héhé.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/m-afortin/MonSerpentEchelles/commit/45420feb920a5048d17dbb385d50710dd651cfb2#commitcomment-11013133


This seems like the wrong approach.  Looking at the b2d and ubuntu provisioners it seems like we should do just do sshcommand("sudo system-docker <ACTION> <NAME>").  The only difference is that if name == "docker" we need to turn that into "userdocker".  Honestly this does point out that maybe we should name userdocker as docker.  That is more consitent with other distros.


Why has the split between right and left owners been removed? It's required for the case where assets are encrypted and require the following to be decrypted:
1 delegation from group A and 1 delegation from group B, where groups A and B are not necessarily equal or disjoint.

This case is important, albeit not tested.


topic_model->FindPwt(&global_r_wt);  global_r_wt.Reset();

That's a hack :)
Add a new constructor in TokenCollectionWegiht class that accepts #tokens and #topics, and make it initialize all tokens with zeros.


I think TopicModel class should not expost non-const p_wt_. Please change this method to

const TokenCollectionWeights& GetPwt() const { return p_wt_; }

Is this enought for all current use-cases?


Indeed, golint pointed that out to me.


Also consider this:
```
///Represents the exception for taking the median of an empty list
class median_of_empty_list_exception:public std::exception{
  virtual const char* what() const throw() {
    return "Attempt to take the median of an empty list of numbers.  "
      "The median of an empty list is undefined.";
  }
};

///Return the median of a sequence of numbers defined by the random
///access iterators begin and end.  The sequence must not be empty
///(median is undefined for an empty set).
///
///The numbers must be convertible to double.
template<class RandAccessIter>
double median(RandAccessIter begin, RandAccessIter end)
  throw(median_of_empty_list_exception){
  if(begin == end){ throw median_of_empty_list_exception(); }
  std::size_t size = end - begin;
  std::size_t middleIdx = size/2;
  RandAccessIter target = begin + middleIdx;
  std::nth_element(begin, target, end);

  if(size % 2 != 0){ //Odd number of elements
    return *target;
  }else{            //Even number of elements
    double a = *target;
    RandAccessIter targetNeighbor= target-1;
    std::nth_element(begin, targetNeighbor, end);
    return (a+*targetNeighbor)/2.0;
  }
}
```


Changing my mind.  We need the local `j`.  `for (auto& e : *findSet(hash))` doesn't save the temporary `shared_ptr`.  It forms a reference to the referenced `ValidationSet`, and that reference does not extend the lifetime of the `ValidationSet`.


AdaptiveComfort (hongtz68) - 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)


booooooooooooooooooooo


Consider using one of the following two methods to fix and validate ModelConfig:

void Helpers::Fix(::artm::ModelConfig* message) {
bool Helpers::Validate(const ::artm::ModelConfig& message, bool throw_error) {

The idea is that all ModelConfig messages that circulate withing artm::core must be validated. Currently this happens in c_interface.cc -- and this is enough because there is no other way for ModelConfig to enter our system. For batches this is a bit more complicated because they can be loaded from disk.

http://en.wikipedia.org/wiki/Separation_of_concerns
Validation logic should not spread across the entire BigARTM code :)


Hmm. Node. Interesting. I wonder if React Native will end up using a combination of CocoaPods and npm. (At the time bento was created, it wasn't yet possible to specify a `s.dependency` for React Native. Don't know if that is true yet either.)


Cool


I'd probably not pass the pattern as an argument since you aren't reusing a structure to save allocations. As a matter of fact I might put the pattern on the decoder itself or return it:

```
func (d *Decoder) Decode() error {
	return d.readPattern()
}
```


I'm actually using .ascent and .descent for the line spacing, and can't seem to get rid of the pairing effect.


This is an interesting line. One alternative here would be to use Pwt from the previous topic_model (previous = produced by last model.Synchronize()).
I think we can keep your version, but keep in mind that there are alternatives here :)


Consider placing this logic in
void Helpers::Fix(::artm::ModelConfig* message) {
bool Helpers::Validate(const ::artm::ModelConfig& message, bool throw_error) {


make an array for function onEat


Well that is what PixelArt is about :x


Perhaps it should just say `See ":help nvim-python"` ?


I do not like this behavior. But, I respect the fact that you do like it. Can you make a separate option to govern this? The existence of host and port options should not be sufficient to open a connection on load.


Idiomatic Go prefers the original if statement struction: https://golang.org/doc/effective_go.html#if


#খেলা #জয় #ক্রিকেট

পাকিস্তানের ক্রিকেট দলের বিরুদ্ধে টেস্ট ম্যাচে ৩১২ রানের ওপেনিং জুটি যা, যেকোন উইকেটে বাংলাদেশের সর্বোচ্চ জুটি।


parsing_reorg (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

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


It is only relevant for screen readers. It's good practice to add a description for every input. So I think we could start improving accessibility where we can, and it should be internationalized. What do you think?


No, we don't. It just a leftover from a previous attempt.


maxcdn is what is they recommend on the [website](http://getbootstrap.com/getting-started/#download-cdn). But I leave up to you to decide what to use.


Oh shit, typos...


The option for Anisotropic Filtering in the Windows UI of GSdx-OGL is still grayed out.


৩১২ রানের ইনিংস টেস্টের সেকেন্ড ইনিংসে সর্বোচ্চ ওপেনিং জুটি গড়ে টাগরেরা।


Cool. I was trying to emulate `encoding/json`, but I should've thought about what you really gain by passing the pointer in. Thanks for the suggestion!


Current `master` will be 2.2 beta hopefully early next week, so you're safe there. I'll cherry pick it into `release/2.1` when I get out of bed and that'll guarantee you're safe if there is to be 2.1.9. :-)


Yes- the packaging required for ReactNative is kind of a horror show when you start digging into it. I can confirm that this will get you up and running in a standalone application  with roughly these steps:

$ react-native init AmblyExaple
$ # create and edit cocoapods
$ pod install
$ # edit AppDelegate.m
$ # edit build-phases so ldflags are $(inherited)

After this, you should be able to run the app then connect from the click-clack-joe project. Note that for some reason I had to delete my ~/Library/Developer/Xcode/DerivedData dir the first time I ran the build.


৩১২ রানের ইনিংস টেস্টের সেকেন্ড ইনিংসে সর্বোচ্চ ওপেনিং জুটি গড়ে টাইগেরা।


@Kusold You changed the `habitSync()` `historyPath` argument from using the `.file` to using `.historyPath`. This causes the sync to fail when it tries to find the todoist-habitrpg.json file. Changing the `habitSync()` argument `config.historyPath` to `config.file` is the easiest way to fix this.

The other alternative is to only append `+ '/.todoist-habitrpg.json'` in one location. Either in the argument parsing of main.js, or when checking options in habitSync().


Also the hiding of the mouse cursor is no longer working on FFXIII:
https://github.com/PeterTh/gedosato/issues/278#issuecomment-93829297


There is pip on OSX but the steps for downloading through pip has already been mentioned above. Should I add it here too?


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/45baf6f8e9388b373913931de35250696758660e/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/45baf6f8e9388b373913931de35250696758660e/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/45baf6f8e9388b373913931de35250696758660e/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/45baf6f8e9388b373913931de35250696758660e/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/45baf6f8e9388b373913931de35250696758660e/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


টেস্টের দ্বিতীয় ইনিংসে পাকিস্তানের বিপক্ষে যেকোন দলের এটাই ২০০ রানের উপর ওপেনিং পার্টনারশিপ।


তামিম ইকবাল টেস্টে ৭ টা সেঞ্চুরি করেন, যা বাংলাদেশের পক্ষে সর্বোচ্চ।


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (346 of 346 tests passed)

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


I was so excited to have figured out a sane solution to this that I misspelled support in the commit message.  Prior to writing this, I had come up with a few really really bad ideas for how to do this!


তামিম ইকবালের ব্যক্তিগত ২০৬ রানের ইনিংস যেকোন বাংলাদেশীর সর্বোচ্চ ব্যক্তিগত ইনিংস।


Your patch is obviously whitespace damaged :(.


You're missing the change to actually add the c_time member to bp_key.


প্রথম বাংলাদেশী হিসেবে তামিম ইকবাল টেস্টে টানা তিন ম্যাচে তিন সেঞ্চুরি করেন।


I'm wondering if we should just go ahead and use 64-bit time_t, to avoid future 2038 problems.


zapper1 was made essentially weak for harming someone with full health, it is very good as a mid-range rifle/smg counter and for finishing off enemies for kill steals in FFA or in general. because of this zapper2 needs to be formidable enough to stand on it's own. i really do not want to weaken it anymore than necessary, or we'll be back where we were before.

keep in mind it's not very useful at higher-mid to long range because of the high spread, it truly is spray n' pray and gets more frustrating to use the farther away an enemy is (though i'm not suggesting this change).

the secondary having 4 rays, the value 5 seems like a reasonable ammosub, for 10 shots (smg2 also has 10 shots).


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (346 of 346 tests passed)

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


I agree with this logic @kcristiano - in my opinion, it makes no sense to call this method unless the intention is to access WordPress permissions. And if this cannot be explicitly decided in the calling method, then bootstrapping WordPress should happen, if it hasn't already happened.

The only reason I can see for this being questioned is the tab-indenting.


parsing_reorg (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

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


Yes, I agree. internationalise. Let me know if you need help on it.


Installation instructions for Windows :-

[https://github.com/UKDemocracyOS/app/wiki/Windows-Installation](https://github.com/UKDemocracyOS/app/wiki/Windows-Installation)


What about adding a small explode value to zapper2 (and possibly zapper1) for cosmetics? It adds a nice dynamic light/explosion effect.


@rf- could you take a look at this when you have a chance and see if you can figure out why these specs are failing?


@GawainLynch Isn't the ``isset()`` part redundant now?


I was thinking about that choice too.

We could be blunt and make it `getOptions("blpAutoConnect", FALSE)` so one would have to set it to `TRUE` to get it.  That may protect you from "accidentally" getting, but allows us to have it on by default.


We shouldn't rely on peripheral.find for sending. What if we want to use another interface?


Wi ch


If it is not an issue for you, I would prefer keeping cdnjs for consistency with all other dependencies we already have, and leave this potential change of provider to a new issue, where we can learn which one is more suitable.


Wich freqs ?


if it doesn't cause more particle ruckus from the lag it might be a good idea. suggested variables?


Ah, good catch, careless mistake. Fixed in 3be2b155.


Could it be that this line as a undefined behavior bug?


@tapcio: Thanks for advice. Done.


@feiranwang One of my other feedback about this test case is can we make this test
case a bit stronger? Currently, it seems that you only check whether the weight is positive
or not? However, if your case, even if you do not sample the latent B, I think the weight
is still positive? If so, whats the point of this unit test -- meaning, will this test also passes
without the fixing of the bug that Zifei reports (sampling latent variable)? I might be wrong,
so feel free to correct me.

I think a better way is probably manually calculating the ground truth weight, and in the
unit test checks whether the result fell in an epsilon-interval centered on the ground truth.

@netj does this make sense? Your input here would be really helpful!

Ce


Date: may 3rd 4 am Hawaiian time
1)decommission 1000C
2)enable port 2 on FGVM01
Pls approve


90179052-SetupEMSActuatorMissingDuplicates (JasonGlazer) - 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)


For some reason this commit triggers a failure in the test plan, because this cancel fails:
https://github.com/dexX7/mastercore-rpc-tests/blob/master/test_meta_dex_plan.py#L1021

It's caused by a price mismatch, which was introduced by the trade that happened right before the cancel operation:
https://github.com/dexX7/mastercore-rpc-tests/blob/master/test_meta_dex_plan.py#L994

Here is the log with the relevant parts, where "correct" refers to the test run, which doesn't fail, based on a build of one commit before this one, and "faulty" refers to the test run, which fails, based on this commit:

- http://pastebin.com/raw.php?i=n13S19C0

It's known that there are issues with the numbers, but this is what makes it complicated now. So maybe you have an idea, and it would be great to know, whether you are certain, if this commit is fine?


@kyranstar I just pushed a change that should add a back button -- can you test it out? I was attempting to make it render with a back arrow as well as the text but there's something funky that MonoGame is doing to my Unicode chars.


Yeah, I noticed. I just liked having it, but it was not so important to me that I would ask for bringing it back.


Just out of curiosity, any reason that we are not using the one I manually calculated?
https://github.com/HazyResearch/deepdive/issues/269#issuecomment-97933493

It is totally OK! I just want to understand why (maybe I made some stupid mistake in calculation?)

Ce


Is there any dependency issue here? (Your clone this https://github.com/sstephenson/bats)


will it be possible to set a custom babel config? for example, I need to pass `stage: 0` here for some of my projects.


é isso!


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (346 of 346 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)


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/e64e4b087732e9e570907e5fa08d2b51fe0dc6d1/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/e64e4b087732e9e570907e5fa08d2b51fe0dc6d1/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/e64e4b087732e9e570907e5fa08d2b51fe0dc6d1/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/e64e4b087732e9e570907e5fa08d2b51fe0dc6d1/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/e64e4b087732e9e570907e5fa08d2b51fe0dc6d1/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Why not

    var mat = this.program.match(/[\+\-<>,\.\[\]]/)

?


why `1250` ?


I wonder if we could write it out to a different file descriptor?


@zhangce In the first example (A-B-C), if B is not sampled, the weight will be negative, since Bs are initialized to 0. I also tested this case.

Sorry I didn't quite understand your example in https://github.com/HazyResearch/deepdive/issues/269#issuecomment-97933493. There, you mention B is non-evidence, which means training data should not contain B, but in the example B is in the training data?


parsing_reorg (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (346 of 346 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)


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (346 of 346 tests passed)

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


Thanks Renzo!

On Fri, May 1, 2015 at 7:33 AM, Renzo Frigato <notifications@github.com>
wrote:

> @lmperry <https://github.com/lmperry> Added this little change in the
> branch adminModal
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scitran/sdm/commit/33ca50ec9bf9e7bae5f51988ba5995fee8b4e2a6#commitcomment-11002037>
> .
>

--
Michael Perry
VISTA Lab
Stanford University
420 Jordan Hall, Room 482
lmperry@stanford.edu | 650.724.1419


As far as I can see in the old version `neworder_buyersprice` is inserted, while in the new version `new_mdex.effectivePrice()` is inserted.

In the old version, `neworder_buyersprice` equals the price the offer had before the trade.
In the new version, `new_mdex.effectivePrice()` equals the price the offer has after the trade.

So if I'm not completely mistaken, this is correct and expected behavior, and the failure is caused, because the original unit price isn't honored.


excelente!! :+1:


Quick note: this should be `const CMPMetaDEx& objMetaDEx`, because otherwise a clone/copy of the object is created, which isn't necessary.


This should be Pascal Cased.


Yeah... Cuuudddd. Nose goes!
On May 1, 2015 9:00 PM, "Derek" <notifications@github.com> wrote:

> Make an not default like darwin, and have to opt into intel?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VanirAOSP/platform_manifest/commit/f5a699bc9f8c7b3699b18fd11ddf4e5a96cfb28c#commitcomment-11010204>
> .
>


ResortedSurfaces (amirroth) - 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-71.83%25-red.svg)


Ah thanks. Hm.. so how about reporting only successful restoring of the state, given that this is what we *certainly* know at this point?


@zhangce In the A-B-C example, the training data is (1, 1, 1), (1, null, 1), (1, null, 1), (1, null, 1). If Bs are initialized to 0 and *not* sampled, since majority of training data will be considered as (1, 0, 1), weights will be negative. If Bs are sampled, the learned weights will be positive.


You forgot to remove the install rule - I did so in a2b84678a.

You will need to re-release as 0.7.9 will be broken.


The `range_check_cast` removes the need for the `if (value.asInt () < 0 || value.asInt () > 255) since in that case the cast will `throw`. We lose the name `out_or_range` error, but we can fix that.


parsing_reorg (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (346 of 346 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)


:speedboat: :speedboat: shaved several ms off of response time.

![new_sequence](https://cloud.githubusercontent.com/assets/739782/7439611/29a74e32-f037-11e4-91fd-f0eee867baa6.png)


:+1:  Its OK


Me gustan esos pequenos cambios se ve super :)


@netj Thanks! I'll check it out. For the factor graph, I think we can either modify the binary parser to generate text, or rerun deepdive to get the intermediate graph.


Oops, it's `mempty`, not `empty`. Oh well.


`right` should be `top`.


Can you delete white spaces?


@lem9 Tested the patch it works fine. I suggest to merge the patch.


Make sense. Thanks for catching up this.

On Fri, May 1, 2015 at 7:48 PM, Carsten Brandt <notifications@github.com>
wrote:

> yii2-dev repo has the "type": "yii2-extension", in composer.json to
> trigger the symlink for Yii class to be created in yii2 directory. So it
> has to be added here. Also when only yii2-dev is installed, yii2-composer
> needs to be required to allow installation of extensions.
>
> should all be resolved by f6c72d2
> <https://github.com/yiisoft/yii2/commit/f6c72d25bfc2d112e0c50dd3e5592615aea92a96>
> now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yiisoft/yii2/commit/e07a9d11593ca6165985b4fb3d52f6657767aa62#commitcomment-11009732>
> .
>


parsing_reorg (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (344 of 346 tests passed)

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


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (346 of 346 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)


@djowel and @daminetreg,

I think we should publish a patch for BOOST_FUSION_DEFINE_STRUCT(_INLINE)'s regression.
So, I squashed Damien's PRs into [single file](https://github.com/Flast/website/commit/9e63fddd3766b97ea076d5a1a21170faa724b42d#diff-c57bc6a95c52fb4b2b1ce2d45c49ca50) for user convenience.

Could you review this? If OK to publish, I'll open a PR.

Thanks,
Kohei


oh, probably it will handle `.babelrc` automatically, need to try. sorry :)


Looks good to me!


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (346 of 346 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)


I think Maggo's path is much like the git clone.
The path in source file would confuse user git clone the T-Bone.


90179052-SetupEMSActuatorMissingDuplicates (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (624 of 624 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-5.19%25-green.svg)


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (346 of 346 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)


U're right here cocafe..  in the past i had set as for min. here 400Mhz .. but after checking on CM Settings - Performance it was 200Mhz .. but then i've removed the 200Mhz Step from CPU LiveOPP Table so that the smallest freq is 400Mhz .. and now the Phone is working overall @ min freq 400Mhz ... Checked with ADB ..: Recovery and after boot in CM .. and probably also on lpm (offline-charger)

I'm using min 400Mhz since months and i'm really satisfied with it.. also nobody complain over battery life... I've measured the diff between 200/400 and it's not really much .. just approx 4mA ... measured on IDLE.. that means u'll have 4% more power consumption on idle ... and i didn't measured it on full load because on scrolling the Gov. (interactive) switch the freq @ 600Mhz .. so we can say min freq is running always at max. 4 - 5 % load ..

Conclusion: Big Performance boost, more smooth, no lagg by first scrolling, faster response, at cost for very small more power consumption ;-)


Credits to Mrcl1450


Great.
This will fuck up online updates for all of these machines ...


WHERE IS MY VIM


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (346 of 346 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)


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


cc @dagolden

This isn't an ideal fix, but makes the feature that has been supported for years, stil work.


parsing_reorg (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (344 of 346 tests passed)

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


When the lists are the same the old message would be nicer, with just one list.


> That would result in the fatal internal error being notified.

If `userMessage` is not empty, then the `"Error: A fatal internal ..."` line is not written, but instead `userMessage`.


全部个性化修改放到myuserinfo 模块中，恢复之前修改的全部ps的源文件。


Is that break supposed to be inside the if block above?


Is the Supreme Court step necessary? Courts typically exercise judicial restraint and examine actual disputes.


Looks great, @raykamp !


The current folder structure isn't the structure of an Android project, it can be directly opened in Android Studio


http://review.cyanogenmod.org/#/c/95859/

Although functionally the same for xmm6260, Utkarsh Gupta's solution is better:
http://review.cyanogenmod.org/#/c/95908/


@tdooner I've updated some dependencies that the Heroku stuff that you worked on uses, but I didn't try out the Heroku stuff. When you have a little time, would you please give things a try and see if there is anything broken?


AdaptiveComfort (hongtz68) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 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-5.16%25-green.svg)


@kamino I'm trying to use libselinux Python bindings in setools4 now. The only way I see to do that is to do sitepackages = True in tox.ini, since the bindings aren't available in PyPI. But if I do that, I don't see the use of tox anymore. Moreover, to get TravisCI to work correctly with the bindings, I have to do the above, which also makes it look like tox is rendered useless. What do you think?


This is minor, but I would call the class `ColoredLogTestPrinter`, starting with "Test" irritates that this is actually a real test instead of an helper class^^


parsing_reorg (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (344 of 346 tests passed)

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


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (346 of 346 tests passed)

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


Here's my revised comment.  At the beginning, the NOTE says that this concenrs
only the checkpoint thread (since this comment is inside the routine
threadlist.cpp:checkpointhread()).  How does this comment sound?  Thanks.

    /* NOTE: This code is only for the checkpoint thread.  If you're looking for     *      what the user threads do at checkpoint time, see:  stopthisthread()
     *
     * There are two ways for the checkpoint thread to return from a checkpoint:
     *                 resume and restart
     * If we're here, we just resume'd after checkpoint.  It's the same process.
     * If we chose checkpoint, 'bin/mtcp_restart' created a new process.  The
     *   source code is in 'src/mtcp'.  The program 'bin/mtcp_restart' will map
     *   our memory into the new process, and then meet us back here by calling
     *   the function specified by 'mtcpHdr->post_restart':
     *                                        ThreadList::postRestart().
     *   Actually, postRestart() will start the user threads and then call
     *   restarthread() for the 'motherofall' thread.  Then, restarthread()
     *   will call setcontext(), in order to arrive back at getcontext() here
     *   in this function, just before the 'while(1)' loop.
     * FIXME:  The 'motherofall' thread is the primary thread of the process.
     *   On launch, 'motherofall' was the user thread exeicuting main().
     *   and the checkpoint thread was the second thread.  But now,
     *   motherofall will be the checkpoint thread.  Why do we switch at the
     *   time of restart?  Should we fix this?
     */


space before ==


space before {


space before writeToLog() so that it is inline


Remove merge part


@asergi Yes, i know. I am going to write a work around if the 'C' locale is used.


Remove space


remove space


*status message


done and pushed


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (346 of 346 tests passed)

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


bro, can I clone it now?


ack


I wouldn't count on him solving such bugs soon. And there is no way to make that work for requirement of 3-man strong Broadside teams. Anyway, not that it matters much to me, I've a feeling Tau are coming this year. ;)


Вылетает при обращении к членам класса!


@joone Ya, should have sent it as pull requests for review first.


TeamCity Colore :: Main Build [Build 56](http://tc.sharpblade.net/viewLog.html?buildId=577&buildTypeId=colore_mainbuild) is now running


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (346 of 346 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)


This list is not complete. Please include all html inlineElements.


Adopted, thanks!


We've in defconfig config_hz=100 .. But i readed that if config_nohz is used then it's overriden  . .. So.. Nohz is faster than hz=1000 !

Check again:
If CONFIG_NO_HZ is set, the limit is 10ms fixed
HZ=1000: min=20000us (20ms)
HZ=250: min=80000us (80ms)
HZ=100: min=200000us (200ms)


TeamCity Colore :: Main Build [Build 56](http://tc.sharpblade.net/viewLog.html?buildId=577&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 113 Build time: 0:0:47


a text node *is* an inline element. This looks wrong


This is something I hate python for...
It isn't possible to do that without `\`, is it?


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (346 of 346 tests passed)

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


it is with brackets... you want that?


This is bad code. Each recursive function should call self only once.


This part of the logic could be written in a simpler way, I think.


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Kevin, when you made the formatting changes, did you make them to the .md or the .html file?


fixes #1


we can change the name to `isAtBeginingOfBlock`


Are you sure about this ? is it a list or an object ?


parsing_reorg (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (344 of 346 tests passed)

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


Hola xD


Test comment


pushed new version without backslashes


okay so we do want to fall-back on to the browser default behavior for cases we are not taking care of.

Therefore we shouldn't preventDefault at the very begining of the logic!


Is it inspired by that https://github.com/CyanogenMod/android_kernel_htc_m7/commit/d499cbf27355349b5ad5c7bc8561dd9a9920f343 ?


@guidow, sorry I meant to branch this and open a PR.  I'm happy to roll this back if you object though.


Hey, so I think this part of the code can be rewritten to be more clear. What about sometime along the line of `isAtTheBeginingOfBlock`?


@sayoun wants only one confirmation prompt for all operations, as well as only one progress bar. So here is my proposed implementation for the `detach` operation; let me know if that's the right behavior and I will adapt the other methods as well. cc @kalou


This seems from the future :P


Hi, I've added this to the latest release.


El problema seguramente radica en que la base de la variable es de tipo int, y peta cuando recibe un char


Hi, I've added this in the latest release. Thanks.


Sweet! Really appreciate that.

On Sat, May 2, 2015 at 11:37 AM, Kyle Gorman <notifications@github.com>
wrote:

> And done!
>
> https://pypi.python.org/pypi/nlup/0.5
>
> > On May 1, 2015, at 6:12 PM, Kyle Gorman <kylebgorman@gmail.com> wrote:
> >
> > great, do that and i'll merge. i can submit it to pypi sometime weekend
> unless you want to.
> >
> > On Fri, May 1, 2015 at 6:09 PM, Hobson Lane <notifications@github.com>
> wrote:
> > Will change that to 3.3 to be accurate, but, FYI, the parts used by
> > DetectorMorse work for me on simple tests.
> >
> > On Fri, May 1, 2015 at 3:07 PM, Kyle Gorman <notifications@github.com>
> > wrote:
> >
> > > I see the :: Python :: 2.7 field...does it matter that this doesn't
> > > actually support 2.7? It's 3.3 or better, I think, as I use some of
> the new
> > > features.
> > >
> > > --
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11008710
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11012922>
> .
>


this could fix jumping on rw:
https://github.com/FernetMenta/xbmc/commit/652701948dc9b05a6a2caccc97ba99e8a863572e


@h2o64 Yup, intervigil suggested that commit like week or 2 ago.


This isn't a good idea that'd make it more powerful.