Does this work in case of plugin timeouts? Consider this scenario:

* Kill Bill API call takes the lock then calls the plugin API
* The plugin calls back a Kill Bill API which attempts to take the lock. A dummy lock is returned
* The plugin API call times-out (e.g. too short timeout configured) and Kill Bill releases the real lock. At this point, both the database lock and the ThreadLocal reentrantLocks are cleared
* The plugin thread continues however (its thread is still in Kill Bill core), and is expected to have the lock, but another operation (different requestId) can take it

Do we need to do refcounting instead?


Likely requires AI adjustments, but there are already a few similar AI-unaware buildings in the tree, and this should be far from a release, and these can easily be ignored by testers who don't want to have related AI complications.


test comments


I think prints are not being shown, only logger output from the application. will double check though


Should we also check this in the `isFree` call?


I agree that you can equally toss "Login details" - it is equally superfluous. Just "Set your password" or so is sufficient. It does not matter if this is a login form, password-reset, or whatever - this example should illustrate an instruction proceeding a form control, not an entire form. Please keep it simple...


@githubjeka окей, если мы переносим управление связными данныи в пхп, каким образом обрабатывать ситуацию, когда я хочу сделать `Project::deleteAll(['stats'=>'bad'])`
Я должен вспомнить про все связанные данные и удалить их вручную?
Или просто не удалять записи из базы? Оставляя мусор.


When you create a new file in the project and put the license, please review that there isn't any extra copyright line and the file author is really the author.
Maybe it is collateral effect when you copy and paste the license information.


With `i = lumphash[hash]` this could never work!


In the following code mentioned in this chapter
```js
function foo() {
	console.log(this.a);
}

var a = 2;

foo(); // 2
```
why does `foo()` print `undefined` instead of `2`? I've tested it in node.js with `nodejs ch1.js`.


Thsi won't fix the defect:

```
large_shift: In expression 1 << bgtype, left shifting by more than 31 bits has undefined behavior. The shift amount, bgtype, is as much as 32.
```

The actual issue here is that mask is shifted 32 bits (greatest value of bgtype) to the left which causes an overflow of 1 bit.
This isn't solved through increasing bgtype to 64 bit size.

![screenshot_1](https://cloud.githubusercontent.com/assets/1146834/9381891/f984cd0e-473b-11e5-9fff-897da0528ad1.png)


Please adhere to the commit message conventions.
e.g. `chore: added metafiles`


ok!


remove doc string


The synthax remains the same as before

https://github.com/temasek/android_bootable_recovery/blob/cm-12.1/updater/install.c#L282
format(fs_type, partition_type, location, fs_size, mount_point)

In fact, the algorithm now tries to check the underlying filesytem instead of formating to the given parameter. Which is usually ext4.


Updating pluginhook can be avoided if core plugins live in their own dedicated plugins folder, with each core plugin symlinked into the regular `/var/lib/dokku/plugins` directory. It kinda solves the original issue at hand, but also I think it might just be a sane thing to do to prevent any buggy custom plugins from interfering with Dokku's installation process.

I'm still decently unfamiliar with Dokku's codebase, but I'd be willing to volunteer to implement the symlink approach if you guys think it's worthwhile.

As for the chef thing, I'm personally using the [dokku-simple](https://github.com/rlister/chef-dokku-simple) cookbook which basically just executes the `bootstrap.sh` script. I have a feeling the guy who added the dependencies hook was doing some more custom and fine-grained stuff via chef. Separating general setup from dependencies is pretty logical, but I'd argue it gets annoying for users if they need to go through multiple steps rather than one to ensure new plugins will actually work.


Is there a normal scenario under which `currentHolder.equals(requestData.getRequestId()` is false (one case I'm thinking about is if a previous dummy lock wasn't released properly for this thread)?

Should we add a `WARN` log line in that case?


@sidonath ne razumijem logiku ovog buga, jer radi unutar samog (novog) if bloka, a unutar postojeceg else if ne radi.
Treba ispod postojeceg if bloka dodati novi, tako da ce biti podrzan userov input..
 if (option == 4) {
       getAllUsers();
}


Here it tries to overrides the given fs_type with the detected one


сейчас Project::deleteAll(['stats'=>'bad']) появится ошибка связанная с `fk-image-project_id-project-id`.
В данной реализации удаление Project не предусмотрено, поэтому и обработки этой ошибки нету. А чтобы её решить нужно на beforeDelete в Project повешать
```php
ProjectImages::deleteAll(['project_id'=>$this->id)
```


heh.


Fall back to the given parameter if there is any error.


Extra imports.


Yep, you are right, happens due to copying the original file and than substituting the content, thus no (c) 2012. Fixed in comment https://github.com/owncloud/android/commit/a544a0cfb6ab119670c6f24cab096dc0c8235b39


Please, apply the naming convention for all  the attributes of the class: start with m
menuItemId --> mMenuItemId

(Same comment for the rest of the classes)


Yes, but you are already linking/using it through qt etc.


 Nice example!

One minor comment - it's more idiomatic to write

    let spin rng mapToBox mapToValue : Spin =
        ...

cheers!
don


Oh, okay. Thanks :)


Also here, it's more idiomatic to write

    let x : decimal = ...

best
don


This is how Fraggle has it in https://github.com/chocolate-doom/chocolate-doom/commit/a28f2f4333102961668354d75850c7ffebbdb194 though. Am I missing something?


I would keep the `segmentsIntersect(X1, X2, X3, X4)` syntax since it is more intuitive to https://en.wikipedia.org/wiki/Line%E2%80%93line_intersection#Intersection_of_two_lines

You can take the `dataDiff`, inside `segmentsIntersect()`


Np, I based this patch on this commit : https://github.com/temasek/android_bootable_recovery/commit/d5d7b84be81457718a2889e475afd583ca936ffe


Could you give me an example so that I can learn? Big thanks


HOLY SH*T !... If you wrote this as "extern void *end", its value will be 0!


Nice


At least this is the behaviour expected by Daiquiri's UWS implementation, tested with CosmoSim. Is there any reason why the + '/phase'  should be kept?


@FWAJL @bsaiken
Yes, certainly, in fact I was hiding the matrix popup and showing the location popup again as that is where the user opened the matrix popup. I added the redirect as it was mentioned particularly in the issue description. Let me know how you want it finally and I shall change the code.


I prefer this explicit dumbed down code which is also likely to be faster when JITed.


`ddoc_rev` here is better since docs have been using underscores not camel case?


where sholu we iclude the directive when i included this directive the grid diappears


画像ファイルの読み込み方を変更する（いつか）


The default without setting anything is already STEAM_RUNTIME=1


I was not able to see any error in ```set_default_paths``` on MRI !? example please ! I will see for the ```add_file``` part.

but it is OK to revert those patches. yes, I just closed it on sonatype to be able to use it with jruby-1_7 as if it comes from rubygems.org. so 0.9.10 is not released !!! just drop it on sonatype


I'm sure the coloured full-stop is a feature of another company's logo but I can't remember it and hopefully nor will others. Or we could just take it out.


If you're removing these two lines of commented code then the 3 lines of comments above should also be removed


This is a fantastic update to the codebase, it should be implemented immediately! :D


Need at least -webkit-transform to make this work in Safari
http://caniuse.com/#feat=transforms2d


How about [this](https://github.com/Homebrew/homebrew/issues/43112)? @mikemcquaid


:smile:


some cells have no pathgrids and hence will have no points, and this assert will trigger


Firstly, manually add the missing files is working?

https://github.com/TheMuppets/proprietary_vendor_oneplus/blob/cm-12.1/bacon/bacon-vendor-blobs.mk#L17

It is easy your just have to add a line like this

PRODUCT_COPY_FILES += \
<path_to_the_file_you_wanna_copy_from_root_dir>:<path_where_it_should_be_copied>

Look at the file I linked


Please add an additional exception that checks the Location headers content.


Please squash this commit into the one before.


Could you elaborate, because i dont really know.

Do you like the new code or the old one?

In any case, the new code is identical to the old one except that all points are compared simulataneously


test


startNode may be -1 here


mPathgrid may be empty here


What sense does `minsize 2048` and `notifempty` make together?


I explicitly wanted to avoid that.

For larger datasets, this becomes costly, as the diff would have to be calculated 4 times instead of one. Which goes up to 12 times instead of 3 for D


`detA = -(X2(1,:)-X1(1,:))*(X4(2,:)-X3(2,:)) + (X2(2,:)-X1(2,:))*(X4(1,:)-X3(1,:))` becomes the vectorized version


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1796 of 1796 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)


Curious why use the medic prefix here for the doc id vs something more generic like app_settings?  Also underscores are more consistent here too?


Please do not indent the breaking change section in your commit message and start the message with a lowercase character.


This are attributes, it is not a comment. Name it comment_attributes, for example


It shouldn't be. Sorry my mistake.


article_id: article.id or artice: article


@jordens so you think we should allow one to use an output range (min/max pair) which does not correspond to the sample values?


fine :P


@pksunkara pls, replace `struct SerializeKey` by `namespace`


If I understand correctly, PLAT_SYS_SUSPEND_STATE_ID need to be specified only for AFFLVL2. So that check could be added here. Also, the req_state[] parameter is supposed to be populated with the local state ID for each level. So req_state[2] can be assigned to PLAT_SYS_SUSPEND_STATE_ID when specified by the power state parameter.


I suspect that with this change, standby state will always be overwritten as power down within this loop.


Will the suspend hook be only ever invoked for the system suspend API ? It seems that at line number 124, there is a hard coded dependency for CPU 1. So I infer that the CPU 0 will always be the last CPU to enter system suspend. Please confirm if that is the case.

If the target state is encoded correctly as mentioned in comments for the validate_power_state() handler, the target_state[2] will have the required state-ID thus removing the need for the platform layer to keep track using tegra_pm_system_suspend_entry() and tegra_pm_system_suspend_exit() functions. You would need to ensure that PLAT_MAX_OFF_STATE is suitably modified for including the PLAT_SYS_SUSPEND_STATE_ID within the OFF state-ID range. Suggest changing PLAT_MAX_OFF_STATE to 0xD for T132 SoC


Why not a `struct`?


Nit: would be helpful to describe that the topology description is for a single cluster with 2 core system.


nit: suggest number of cluster.


nit: Number of CPUs in cluster.


How exactly do you ensure there are no hash collisions..? This is very weird imo.


These state - IDs must correspond to power down at certain power levels. Hence these need to be correlated with the corresponding level and the req_state[] should be suitably populated.


If the req_state[] is populated properly, platforms need not store the state-id within the platform layer.


As mentioned for the T132 platform, this loop will overwrite the standby state.


No need to shorten `user` to `usr` here.


I was not able to understand whether state-ID correspond to retention (CPU context preserved) or power down hence I am not able to give suggestion on the suitable values for PLAT_MAX_OFF_STATE and PLAT_MAX_RET_STATE.

But as I mentioned in my previous comments, if the psci_power_state_t parameter is populated correctly in the validate_power_state() handler, you should get the appropriate local state-ID through the target_state[].


I am not very sure about the OS view of the Nvidia platform. But if this needs to be viewed as a dual cluster , quad core/cluster system, then the topology description need to be modified. But I also see that in the definition of 'plat_core_pos_by_mpidr()' only the CPU id is considered and the cluster ID is ignored in the core-pos calculation. Also PLATFORM_CORE_COUNT is also defined as cluster count * num of cpus per cluster.

It would be useful if you can provide some clarification regarding this.


98812296-SkyTemperatureShiftsSchedules-4866 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (750 of 750 tests passed)

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


@ vwadekar, some initial comments from me. Please let me know if you need any clarifications.


s/sqladmin/sqlAdmin


headset is working with poweramp player i try with this http://m.hiapphere.com/down/poweramp-full-version-unlocker


Is `order_line.product_id.name` well translated in the customer language without any context alteration ? Shouldn't you do something like:
```
order_line.product_id.with_context(lang=order.partner_id.lang).name
```
?

I guess in your test case, the customer and user had the same language, but the supplier add a different ? You should probably test with three language:
 - User: English
 - Supplier: French
 - Customer: Spanish

And see if your fix alone solves the issue.


Fixed and pushed for this commit (https://github.com/owncloud/android/commit/ba80646e4e31a6c19d4ef8b60ea0e30cf3bd00cc)


Now there is error:

```
DB ERROR:

Array
(
    [code] => 1054
    [message] => Unknown column 'DBSIMPLE_ERROR_VALUE_NOT_ARRAY' in 'where clause'
    [query] => INSERT INTO aowow_source (`type`, typeId, src10, moreType, moreTypeId) SELECT 1, spellId, 1, NULL AS m, NULL AS mt FROM dbc_skilllineability WHERE  (skillLineId IN (DBSIMPLE_ERROR_VALUE_NOT_ARRAY) AND acquireMethod = 2) OR  (acquireMethod = 1 AND (reqSkillLevel = 1 OR skillLineId = 129)) GROUP BY spellId         ON DUPLICATE KEY UPDATE moreType = NULL, moreTypeId = NULL, src10 = VALUES(src10)
    [context] => ***/aowow/setup/tools/sqlgen/source.func.php line 1160
)
```


You should instead do:

    with ssh.sftp(node).open('/etc/astute.yaml') as f:
        data = f.read()


Hyperlink is set on wring file:
` <a href="print-extent.js">print-v3.js</a>`
instead of
`<a href="print-v3.js">print-v3.js</a>`


@chelmuth, @nfeske: please add to staging


Use `sh` instead.

Don't break command string into lines, pass it as one argument (bash just passes other arguments to 'script' in the one after `-c`


How about  a rename to

function out = segmentsIntersect(X1, diffX2X1, X3, X4)

And then calculate the determinant by:
 detA = -diffX2X1(:,1) .* (X4(2)-X3(2)) + diffX2X1(:,2) .* (X4(1)-X3(1));

As a side note one could check (X4-X3)==0 (which is fulfilled for one of the two) and only calculate only the appropriate part but that would probably take it too far


Don't do awk here. Use smth like https://github.com/Mirantis/octane/blob/master/octane/commands/upgrade_node.py#L30-L36 instead


Same issues as above, also use {}-formatting instead of %


Same as above.


ditto


tudazhe


nutiponyal


By ref plz


Use `next(i)` instead of `i.next()`.


@flangofas You added qobo-wp-brand twice
@flangofas You added qobo-wp-brand twice


Purpose of 'struct' is declare data structures. There is struct used just for separate const stings from other code. Namespace is better for this purpose.

It is just "cosmetic", but it is "more C++ way".

Similar case are anonymous namespaces in cc files for avoid function name collision. In "older days" was used static functions for same purpose.


Should be injected as param.


Is it ok to change that? I didn't dare to. That would also simplify the implementation a bit.


It actually is added in the latest commit


Regarding speed, I have some experience porting MATLAB code to C and I have to say, that it is extremely difficult to even get on par with MATLAB when dealing with matrices operations.

Lopps are still another story ;)


Forgot `"` around `glance`?


Я пропустил или потерян коммит `version up`?


Please make sure that this is not creating any warning (halRet is set but not read...)


jshint doesn't like `''` here :-(


It's the downstream bridge. As in "download". The Spigot->Bungee packets were visible to the client.


I fixed anything I could find now but I can't guaranty I found all variables to be prefixed though.


and here :-(


This very commit was in fact the source of the problem. See https://github.com/ornicar/Stockfish/commit/6dd43dca5aa179f1c5beed7c271ba48adc04d4f5


This is just awesome. It's worth to code in this language only to finally see this :-)


Generally comment to this file: Is the file still needed in include/oplk? I guess it would be sufficient being in include/kernel since only the kernel (+driver) needs that info, right?


Thanks @neophytosconst . Will fix it.


hello I want to have a digital voice?


@bsaiken On this one, using jQuery is better than a redirect, especially in your field testing since we deal with the Mobile UI here. Since both options are doable in one line of code and none is complex to code, I suggest we use jQuery.


Also, `%s` should be `{}`.


for dsd 100%.


... probably cause there's not enough tests for all these no where ;( - will drop and test another 0.9.10, thx!

pls try to come up with test-cases - also a hint for the debug helpers on `OpenSSL` there's a reason they're not used outside the "frontend" Ruby classes - this way we get stack-traces always come from the direct caller (we're also avoid potential double-logging), the sub-packages usually do not know much about JRuby's classes - Lookout being (already) an "unfortunate" exception to this since it needs to resolve paths the JRuby-way. I think it's ripe for some refactoring - meaningful/useful tests might show us the path to follow ...


I guess you simply copied this, but for the comments please make sure using the same notation for "LED" as well as "on" and "off".

e.g. for the comment ///< **LED** **on** for 50ms and **off** for 50ms.


:+1:

I was able to verify this behavior using a payment control plugin calling back `addPaymentMethod`:

```
2015-08-20 05:00:46,676 [qtp745835029-17] INFO  o.k.b.payment.api.DefaultApiBase - PaymentApi : AUTHORIZE, account = XXX, paymentMethodId = YYY, amount = 5, currency = EUR
2015-08-20 05:00:46,928 [Plugin-th-96] INFO  o.k.commons.locker.GlobalLockerBase - Global Lock %s already taken for request %s
```


DO NOT VOTE IN NUNERIQUE


Please remove this. Should be doable with rebase ;-)


and this


Due to (intentional) lack of CFooMessage::operator=(CMessage), it was a bit clumsy:
```
CMessage Copy = Message;
CJoinMessage& JoinMsg = static_cast<CJoinMessage&>(Copy);
// ...
```
vs.
```
CJoinMessge JoinMsg;
JoinMsg.Clone(Message);
// ...
```
Alternatively we could add copy ctor(CMessage) and operator=(CMessage) etc. to all CMessage subclasses. I've been trying to avoid that, to make these operations very explicit. It has helped a lot so far by preventing accidental conversion/copy mistakes.


It looks like you accidentally included a bunch of unrelated files in this commit?  I would expect it only includes `seeds.rb`


LGTM


Test


Looks like `this` won't be defined when the returned function is run?


Why this line has been commented? When I enable cexample_init() the simulation crashes when toggling a DAGRoot. When cexample_init() is disabled, all seems to be ok. Any idea?


This is the last commit :)


**Cleanup** kernel LED module


The function **cleans up** the kernel LED module.


## This is a markdown file


.. status **LED** mode ...


Should this be container.dup?


> Is it to avoid useless passes on the two first if ?

Oh, it was just to be technically correct about what API we are supporting.   I guess also because the function name `_testItemProperty()` doesn't really make sense anymore.   I was thinking of having a top level `_compileQuery()` method (with the two if() statements) that calls a recursive `_compileDstoreQuery()` method.


Why do you have two `if` statements here?  This probably is not doing what you think.   For example, if you want to check for a to be true AND b to true, it would look like

    if a && b
      # do something only when both a and b are true
    end


Having two `end`s at the same indentation level is an immediate indication that either something is wrong or you didn't indent properly


@EugeneButusov, from my understanding it just truncates the value. Are you sure we'll get ms here?


test case for today...
upgraded virtuemart to V3.09....joomla remain the same :V2..5.11

Payment method successfully added, but remained unopen when revisited. (Blank screen)
kenapa ya bro....tq


is it working


Why do you forward `FALSE` and als `kLedModeOff`?


Thanks, that looks better.  Now just to get rid of the `normalizedQuery` code.


Very minor comment, but I'm working on implementing more formats for sparse matrices in Julia (right now we just have csc) and I'm looking here for inspiration. It appears that this algorithm computes `Y = A*X` instead of `Y += A*X` given that at the end of each loop we have

```C++
Yx[i] = sum;
```

Instead of

```C++
Yx[i] += sum;
```


Same here with `TRUE` and `kLedModeOn`..


Minha antiga senha... hahaha


To apply possible modifications, by module hooks, back to the original message. I was actually wondering if it's intentional that CTCP replies get buffered a few lines below, and was tempted to change it, but I didn't want to hide any behavioral changes into this big diff. :)


Please add newline at end of file


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


I'm not sure what the ($obj->qty <= 0) test does, think to avoid useless 0 stock mouvement.
But it should not be ($obj->edbqty <= 0) because edbqty is null on non batch expeditions, so stock mouvement of non batch will not be registred.

The test for expeditiondet_batch is on if (empty($obj->edbrowid))


This feature sounds good :)

Keep up good work ;)

Tom


Besides the double `if`, what you are doing here is checking if a post exists where the title is 'Jophinsposts', then separately checking if a post exists where the body is 'Thisisthefirstposts'.

Consider if before running the seed, you had done something like

    Post.create(title: 'Jophinsposts', body: 'This is not the post from the seed')
    Post.create(title: 'A different post', body: 'Thisisthefirstposts')

Will running `rake db:seed` create the new post or not?

How can you fix this?
HINT: [find_by](http://devdocs.io/rails/activerecord/findermethods#method-i-find_by) can take multiple parameters to search for


You can merge the two `if`s here.


fixed


Great work here!  No requirement to output anything though.  Really this failing to create a comment is a good thing.. it means it already exists so you are all set.


Please update the descriptions (it's just a copy of ledk.c...)!


I'm doing a review of value initializations *(because there is something that needs to be "done" to a series when it is put into a block as a value now, namely shared out to the garbage collector to say "you can see this now, and are responsible for freeing it"...)*

I'm assuming the INDEX field of this (and below) are supposed to be initialized to zero?  If so, no need to submit a patch to Ren/C.


I will prefer these changes to be part of a separate commit say:
```
[TASK] Use atomic modify ipcore for PCIe designs on Windows

Windows systems with PCIe designs should use the atomic IP
core for the atomic exchange.
```


**U**pdate **LED** state


If the changes in the windows.h are removed then the commit message is okay.


Why?


PS: I'm saying that the current `_testItemProperty()` is really a method to compile a dstore query, so it's better to call it something like `_compileDstoreQuery()`.  And probably changed back to take the query Object rather than the individual `prop`, `type`, and `value` parameters.

The only question is really whether the recursion in `_testItemProperty()` should call `_compileQuery()` or just call `_testItemProperty()`.


Looks good. Thanks for your contribution.


@flexiondotorg: oh no, this will add unnecessary dep to GTK+3 build... and actually, we have to build pluma w/o python support here because of pygtk/GI differences.


previously, the query would always return an error, now the query executes and returns null if the id is not in the followers table @ptsteadman


This function updates the status LED state. The blinking is achieved by using the system timer ticks.


Better add `@context_provider.grandparents_for(...)` and similar methods.


Teste de comentário


test comment


Teste 2


Hmmm, not sure this is ideal. I'm using RedisCache and it's printing out the warning about lockfile. Using warnings this would at least be possible to silence, as it may confuse developers.


В спешке с утра делал. Тэг прокрутил, а это забыл.


@flexiondotorg: not needed... we're building with ```--disable-python``` as I explained above.


If this commit safe to use for production data?


One could also do this BTW:

    pushd .\third_party\python_26
    set PATH=%CD%;%PATH%
    popd

http://stackoverflow.com/a/4488734/2044940


**Set** the LED mode


:+1: from me on lowering number, though I think no-one will be surprised that is my opinion :)


The commit message is OK


The function **sets** the LED mode.


But the error message will never happen because the uuid part is longer than 32 chars, which will move the : to the right, which will trigger the wrong error message. Also there will never be dashes on a copied session id.


hm. I wonder whether it'd be smarter to let it write to stdout, rather than into a file.  Then we have one leak, the filesystem, less. Does that make sense?


Thanks Pardo, as the issue appears for ionic with header and placement of walkthrough outside ion-view I will incorporate the change into the original code, great fix.


Yes. Testbuilds for RPi1 and RPi2 were last night sucessfully


This `break` has no effect. Please makre sure if the `goto Exit` is really needed!


no newrelic APM on these machines :(


The symbol + is in the 859 and 860 lines of source code


... and containing Kodi 15


Обновил


@djstrong
copy+paste is the worst programming technique.
The whole idea of delegation is the fact, that you can configure the other players in the initial context and then usually you don't have to modify methods such as support_for_category... since the other players will provide the necessary data. Other way around is a option for the method, that will select the depth of parent/child tree. But methods such as this one are root of all evil imho. What about distance_3? support_for_category_candidate_set_distance_3???


I'm not sure what you mean exactly...
You mean to initialize variable ```rewrite_rules``` in method ```escape_description``` as static local? If so, local variable even static would have been initialized for each call of ```escape_description```.

To avoid repeated initialization and regexp creation variable, ```rewrite_rules``` should to be initialized before first call of method  ```escape_description```. It can be also initialized in constructor.


Look at the top of the loop. Sum is initialized by yx[i], line 848.


Great, but ... these files need to be site-specific!


Ahh ok will do that. Just need to get myself on Wi-Fi. :p you know I'm on vacation but I love to play with programing. Always learning. You are beeing helpful. Big thanks. :)


Where does the `http://search.ebscohost.com/` come from? Another kind of junk data which shouldn't be there. Can't see it in the source:

```xml
<datafield ind2="1" ind1="e" tag="655">
            <subfield code="u">
              http://www.bibliothek.uni-regensburg.de/ezeit/?1472746
            </subfield>
            <subfield code="x">EZB</subfield>
            <subfield code="A">0</subfield>
          </datafield>
          <datafield ind2="1" ind1="e" tag="655">
            <subfield code="u">http://scitation.aip.org/content/aip/journal/pop</subfield>
            <subfield code="x">Verlag; 1.1994 -</subfield>
            <subfield code="A">0</subfield>
          </datafield>
          <datafield ind2="1" ind1="e" tag="655">
            <subfield code="u">http://scitation.aip.org/content/aip/journal/pop</subfield>
            <subfield code="x">Verlag; 1.1994 - 20.2013</subfield>
            <subfield code="z">Deutschlandweit zugÃ¤nglich</subfield>
            <subfield code="A">0</subfield>
          </datafield>
```


Do you mean array element observers? I tried to use that feature of ember and found the following in this case:

There are two types of options in the menu with multiple selection - simple items- like 1, 2, 3, etc. or array items - such as [1, 0] or [6, 5, 4, 3, 2], etc. When an array-menu item is set to value, the whole value is overriden by the value of the item. If the user decides to uncheck , or check, some additional items later (like 1 ie. when current value is [1, 0], which was set from clicking the menu item with value [1, 0]), this value will be pushed to the array, which will modify the multiple-value item! So, the multiple item menu option value must be sliced (to avoid the modification of the original item value) or the array must be iterated over and all the values must be added by hand, which is a bad solution because it involves an unnecessary loop. Secondly, the defaultValue of the model is an array. if it is modified, then on all subsequent "new model" creations, a modified value will be shown (user chose [6, 5, 4], and on the new job creation, the value will be [6, 5, 4], and not [6, 5, 4, 3, 2, 1, 0]). Probably it can be fixed somehow, but it will involve non-standard procedures (without defining default-value on the model). Regarding all the facts above, I thought it is counterproductive (and probably will not work as intended) to add array observers. This feature is used to simplify the work with array properties, and here it will only complicate things.


Causes regression as per https://bugzilla.redhat.com/show_bug.cgi?id=1202355#c17


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


「:型」は必要ないんで消してください


Hi Graham, thanks for chiming in.
While I understand your rationale, I believe this commit is a decent stopgap measure for the users of the cards and drivers I mentioned; it is better to have a stalled compiler than a non-working program.

That said, should it be the case that the more polished approach can be done quickly, I look forward to seeing it!


I'm searching for the files but they doesn't seem to exist. Is there a way to create or are we completely dependent on firmware files?


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1802 of 1802 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)


これらはくさっちのミスですが、リストに変えてください


This repo is so old, I don't even remember why I had to build binaries myself...

Please head over to upstream instead: https://github.com/brianmcd/contextify/issues/63


@kablamo I'd be interested in your thoughts on this - pulling related stuff for a given result class out to a role.

I like it since, while I like "fat model", I dislike these multi-thousand line classes. I'm just not clear on the... idiomaticness of having highly specialised (i.e. non-reusable) roles.


Announcement on xbian.org?


Hmm, don't think so, my assumption is that container is frozen and nothing will be touching it


Are wasps not included in this?


So this commit is still on branch and not even PR-ed to the release branch coming up. I would at least wait for that.


Chale


Done: 520f98f6c8d838a4e39cfbe755d87fd92de5eadd. Also cherry picked ef63be705e295719ce3bed02abba26d6b3c05fac: 3214a7be85ac1d41a9efb878c18bdae01da75e8c.


Same as [above](https://github.com/lobid/lodmill/commit/9c15dd4290c9afe5471433aa1e4baee07348def4#commitcomment-12805867).


Tests are passing with me. Did you got an issue ?


result: 0.60786


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


I guess that's very very wrong!
Either the reference should point to version 14.0.0.0 which is the version of vs2015 or should include all version 12.0.0.0 assemblies.
In my case, I don't have the version 12 assemblies on my machine and the extension is not able to start because they're needed. (I use vs2015)


ah right :)  Thanks!


Isn't OBJECT_BEE for wasps?


アソシエーションで使う型はreferences型を使います


in process


Oui chef, ok chef!


FourPipeBeamAirTerminal (EnergyArchmage) - 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)


urlはtextのほうがよさそうです


"assigned back to its previous value" -- not sure "usual" captures what you're after here.


Not quite. `wrap` only sets `needSort` equal true if sorting is needed. `needSort` might start out be false for a `sortNode` if `len(sortNode.columns) < len(plan.Columns())`.


Yes, would be good.
But what still has to be done is Kodi 15 for iMX.6 and updating the buildserver, so the Kodi builds can be done there


Ok, but there's no dev branch, and the continuation of 3.0.4 ended in the unaligned branch that also changed data in a semi-corrupting manner. I tried running the unaligned branch which is directly downstream to the 3.0.4 tag that I built off of, but then I get `Could not get free mem on linux, received memory info raw string: []` which you have filed an issue of with the resolution *"fixed"* but with no link to where/when it was fixed. And no dev branch. Is master the new dev branch and does that contain competing consumers?


Tell me when you need access to the wordpress environment to write the blogpost.


I think this is it:
https://www.dropbox.com/s/qcjn3m1rj9o52s4/bluetooth.zip?dl=0


これはどんなテーブルですか？第３者でもわかる名前にしてください


I found there is no need to checking presence of parameters :)


You don't need to hack the gem to modify partials. Just run `rake datagrid:copy_partials` under rails root to get them under your control


You're right. Thanks.


Perfect, it seems to work fine on my side, authentication is working great now ! Thanks !


Very nice!


@haf
https://github.com/EventStore/EventStore/tree/competing-etc/ is the old "dev" branch.


Looks like new download way + old server for seving the fs is stable.
Statuslog in the bg too.


 is that your device tree?
https://github.com/jgcaap/android_device_oneplus_bacon


:) tu croyais que j'avais oublié notre "projet" hein?? Avoue!!


We are currently reorganizing a bunch a stuff and moving much (but not all
to release). As part of this we have also redone all of our internal CI
stuff and are ramping up our release cadence. You may have also seen things
like packaging etc coming in. There will be much more discussion about this
in September (as usual).

On Thu, Aug 20, 2015 at 3:33 PM, Pieter Germishuys <notifications@github.com
> wrote:

> @haf <https://github.com/haf>
> https://github.com/EventStore/EventStore/tree/competing-etc/ is the old
> "dev" branch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EventStore/EventStore/commit/58ea28017fd412be8b4b7a07784e99fa30b754c6#commitcomment-12806061>
> .
>

--
Studying for the Turing test


Payment method buttonnya keluar ga bro pas mau checkout/bayar? Coba cek
error log php juga bro, kalo ada error copas di mari, kita bantu investigasi
On Aug 20, 2015 7:02 PM, "bascovon" <notifications@github.com> wrote:

> test case for today...
> upgraded virtuemart to V3.09....joomla remain the same :V2..5.11
>
> Payment method successfully added, but remained unopen when revisited.
> (Blank screen)
> kenapa ya bro....tq
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rizdaprasetya/vtweb-virtuemart3/commit/9a7512bae54f4ef02875e39e1648bd65d27b0b66#commitcomment-12805551>
> .
>


Very good idea :)


I believe it should. I'm not 100% sure, as I tested on the lighthouse using an attribute for style in the reader.tpl. In any case, I highly doubt there are harmful side effects of that code, so when we deploy for the website, I can check that.

Thanks,
Rohin

Sent from my iPhone

On Aug 20, 2015, at 3:33 AM, Branko Vukelic <notifications@github.com<mailto:notifications@github.com>> wrote:

Does this fix work for desktop browsers?

—
Reply to this email directly or view it on GitHub<https://github.com/RohinBhargava/librarian/commit/169d819f67c94698dc93f8d104d4a606322fe6e3#commitcomment-12802302>.


Labe dependency! "You may have seen us"


Yes. We are searching the ble_conf correct?


Thanks for your help. I can't understate how much I look forward to your CI environment and increased release cadence. :+1:


Why have we been able to remove the values from the `CURRENT_KEYS` object?


Nice big thanks.


It maps to \Rset, not \Rset^d


@bparees would it be reasonable to pre-pull the S2I images we are going to test here? It will speed up tests as k8s don't need to pull them.


Can you link me your devices tree ?


delete


You are correct. I didn't see that earlier.


Could you deploy this to pypi please @jirikuncar?


Yes, that was poor testing on my side. Should be fixed now.


Wieso?


I wonder why you create FileInputStreams here. How can you know the resources are files in the local file system since, in a NetBeans installation, all modules are JARs.

You are better off using the Java New IO API, e.g. get rid of FileInputStream by using java.nio.file.Files.newInputStream(path). In addition, you would use
   final Path path = basePath.resolve(filename);
instead of
   final String pathStr = basePath.toString()+filename;

Regards


@Martii
Thanks for the the tip. I will fix that


Btw. congrats @SharkHunter :+1:


done


they know what gems and bundler is right?


So, it's annoying but we should use a website besides ours. I really think we need to create a custom website because every time we change our website, this lab will break. Another idea is to just fork the current website to another repo, and host it on github pages.


Erstmal damit es kompiliert. Die Version hat dann Fehler beim Build gegeben.


It would seem to me that this code would be far simpler written:

```java
// The first `resultRows-1` results are always returned to the user.
for (int i = 0; i < resultRows-1; i++) {
    pageList.add(...);
}

// If we are not on the last page, we need to use the last
// result as the start key for the next page and therefore
// we don't return it to the user.
// If we are on the last page, the final row should be returned
// to the user.
int lastIndex = resultRows - 1;
if (!isLastPage) {
    /* ... */
    page.setHasNext(true);
    nextPageMetadata.startKey = rows.get(lastIndex).getKey();
} else {
    page.setHasNext(false);
    pageList.add(rows.get(i).getDoc());
}
```


This is not prefix, it's template on which kernel bases interface name creation.


Yes let's make sure to fully support gdal 2.0.


That method and the following should be private. They should not be called from outside.


So what I did here was add a hard copy of the v12 assembly.
My issue it that v14 is not in the Gac as far as I can tell. I have to play around a bit more to try to get it to use v14.


How r u


@eparejatobes WDYT?
I think it's useful for long types. But then should we format all messages like this or not?


this is my  test repo, for my data science course...


missing closing back-tick


This is part of our test set which is generated from mab xml sources we loaded last year or so. Apparently this data has changed. Now we _could_ update that test data, I am unsure if its worth it, though. What do you think?


Maybe %d we should change to first part of POD hash as it's done with iptable rule names?
But remember that device name length is limited (AIR to 16 characters).


FourPipeBeamAirTerminal (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1802 of 1802 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)


Please prefix with `Thrift` or `ThriftRW` to easy heap snapshot introspection


Yes, mind that we have _two_  test sets: _hbz.nt_ and _hbz.es.nt_.


you are mixing old and new API. `from openerp import api, models`, `class website(models.Model)`


98812296-SkyTemperatureShiftsSchedules-4866 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1275 of 1275 tests passed)

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


create student, create teacher, create course, assign adviser, offer course


Sorry, wrong commit name again. I will be careful from now on.


And use CamelCase for class name


Commit message:
[FEATURE] Add user CAL modules for Windows PCIe interface

Windows PCIe user CAL uses a kernel driver as the communication
interface to the external PCIe running openPOWERLINK kernel stack.

The CAL interacts with the driver using IOCTLs. The IOCTL buffer are
used to pass additional information required to process the IOCTL.
Driver parses the IOCTLs and forwards the command, event or  data
to the openPOWERLINK kernel stack through shared memory.


@klaavo What do you mean by ‘lasso selections’?


> the uuid part is longer than 32 chars, which will move the : to the right, which will trigger the wrong error message.

The UUID comes after the `:`, so it will show the correct error message.

> Also there will never be dashes on a copied session id.

Sometimes there are. It happened when I tested it with my own session ID.


The commit message is okay


Ha! Should have looked more carefully. Thanks


Nether Ores have only two block ids for ores (and one for hellfish egg) and uses metadata for each ore. So this won't work anyway, because in `cr0s/warpdrive/world/SpaceWorldGenerator.java` it places blocks with metadata 0.


Shouldn't this be something like this?

```js

self.get().split("\n").each(function (c) {
    return c.trimRight();
}).join("\n");
```


1. Не совсем понимаю, а почему `2.3.1`, а не `2.3.0`?
2. В описании релиза опечатка:
```
Исправлена и улучшено последовательность
                   ^^
```


It's usually not a good idea to add features to the C++ wrappers, which are meant to be minimal wrappers around the C library.  For example, it would break if a program mixed the C++ interface with the C-level lo_server_add_method function.  The _handlers variable is only there to facilitate automatic clean-up -- I'd have preferred not needing it at all.  I'd suggest tracking this in your own program's data structures instead.


llilkl


@danielweinmann Hm.. my idea was to encapsulate the menu logic in one single component, and reuse it in those pages that renders it.


Hey @mtdowling, just out of curiosity -- could you explain what `static` does here?
It brakes PHP Storm parser (it says there's a syntax error while actual PHP parser accepts it as a valid PHP) and, to be honest, my brain is broken, too :)


:+1:  - fixed now per your report. Thanks for that.


But previously there was declared here SchemaRepository<Schema> type which basically means AvroSchemaRepo, I went further and created AvroSchemRepositoryMetadataAware, because only SchemaRepository<Schema> need to be here metadata aware.


We use two this:https://github.com/jgcaaprom/android_device_oneplus_bacon

And :https://github.com/jgcaaprom/android_device_oppo_msm8974-common


Hi @gkibilov, because the original developer removed it:

https://code.google.com/p/javapns/issues/detail?id=214
https://code.google.com/p/javapns/source/detail?r=387
https://code.google.com/p/javapns/source/detail?r=388

Anyway, I've been testing it and did not have any issues so far. Let me know if you encounter any issues.


Teste de comentário. Duas linhas foram removidas (uma linha vazia), e essa foi inserida.
![jn2i0oq](https://cloud.githubusercontent.com/assets/4261743/9383674/40019d68-4721-11e5-84c6-c5f8aa81e309.png)


TBH, this only moves the limit further - what if we run on an ever more constrained host?
Wouldn't it be better to receive all _large heap_ messages from the watchdog and for each:
- if it's our designated process the test passes,
- if it's some other process receive next _large heap_ message,
- timeout and fail after X seconds.

Of course, if this already works in a repeatable fashion, then maybe we don't need this level of detail in the test...


Fixed in  35dd990


@waitfor1t would you add the text you propose? In that case make it a separate description on the README rather than replace it.


An interesting URL that discusses the problem:

http://www.daemonology.net/blog/2014-09-04-how-to-zero-a-buffer.html


Point 2 is more perhaps an objective for the entire workshop, rathe than the intro section.

Points 1 and 3 feel more like learning checks due to their specificity.


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
163: D102 Missing docstring in public method
167:5: E303 too many blank lines (2)
170: D210 No whitespaces allowed surrounding docstring text
170: D209 Multi-line docstring closing quotes should be on a separate line
170: D203 1 blank line required before class docstring (found 0)
170: D400 First line should end with a period (not 'l')
170: D205 1 blank line required between summary line and description (found 0)
195: D102 Missing docstring in public method
216: D102 Missing docstring in public method


Thanks @edwinvanrijkom !

I want to help fix this, but I'm not able to replicate the error you show in the example. I have tried with the Framer versions of the last 5 commits in master and all of them work properly.

I also took a look at the framer.js file of the [counter example you shared](http://share.framerjs.com/rb75v8suurbm/) and the `animatingKeys` function reads as follows:

```
LayerStates.prototype.animatingKeys=function(){
	var keys,ref,state,stateName;
	keys=[],
	ref=this._states;
	for(stateName in ref)
		state=ref[stateName],"default"!==stateName&&(keys=_.union(keys,_.keys(state)));
	return keys
}
```

As you can see, it includes the "default"!==stateName exception, so my guess the version of Framer that you used to create this example is not the latest one?


This changes the `dist/` file, but not the CoffeeScript source in `src/` so as a result the tests fail.


I think it is important to retain the actual time period `dt` as part of the reported values.  For me with a large machine this number is often a good indicator that something is going wrong.


Ew, go Egyptian or go home :P


Minor typo (common/comma)?


Comentario sobre a terceira alteracao


Please stop sending me this unsolicited junk.

I am 'gh' on github.

--gh

On 20/08/15 08:07 AM, Marc Chantreux wrote:
> removed promise. see eiro/rcfiles @gh
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/eiro/uze/commit/f301b032b1ac6d2ae9465f8c77429fac7333cb74


Selector should have depth of applicability no greater than 3, but was 4


something more


Thomas m'a dit avant de s'en aller que tu avais pris des congés, je te croyais juste a la plage, entrain de jouer avec des "depots de sable" :) De mon cote, j'ai pensé (sans l'avoir encore fait) reogarniser l'arborescence de ce projet en faisant comme [eux](https://github.com/ulule/ipfix)


could this same behavior be achieved by not defaulting the `targets` to type `markup`?

```javascript
var targets = className ? className.split('language-') : [''];
```


Why `[1:]`? That wasn't in the original code.


qw


why not `Result.Ok(NaN);`


Did you check out the code snippet I gave in a previous comment (I guess it got lost when you repushed your branch)? It showed that doing it the Ember way was actually less complex than this solution.


Interesting idea, please go on !

If it works, you can achieve the same effect at a cheaper cost by changing the psqt for the queen.
In which case this would be a parameter tweak test.


Why not

```javascript
if (s === 'Infinity')
{
    return Result.Ok(Infinity);
}
if (s === '-Infinity')
{
    return Result.Ok(-Infinity);
}

```


You have to use double quotes for substitution of $t_mantis_bug_file_table


Denne commit'en er ment å cherry-pick'es inn når klientene er klare for å bruke URI'er for språk.


This change is not valid any more.
Sorry I missed it before.


It's not necessary to explicitly set `descending` to false since it's the default value.


this also


Now This "patch" works...
But there is a huge problem with RT Pipeline...We need to work in Dcrop.cc with full Image...
Don't expect good result...
I think it will be very difficult to solve..

1) try to equalize "histogram" (curve, black)
2) after use Dehaze
3) corect with CC curve, L curve, and also Wavelet "edge sharpness"


FourPipeBeamAirTerminal (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1802 of 1802 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)


PHPCS own coding standard doesn't deal with short array syntax very well, that's why I use the long syntax.


Yes it should :)


would be good, if possible. Not a priority.


Formerly the code carried two keys around where `startKey()` was changed based on the paging direction. `CURRENT_KEYS` was used as the start key value when paging backwards. This was redundant because we already carry the paging direction. Now the correct start key is assigned in the `PagingMetadata` based on the direction of that link from the current page when it is created. So it assigns a `startKey` of `rows.get(lastIndex).getKey()` for the next case and `rows.get(0).getKey()` (via `thisPageStartKey`) for the previous case. Does that make sense?


Unfortunately, this is was PHPCS Coding Standard expects to be a correct multi line array declaration.


@ExSport - thx. Ask me I'm the answer :)
ps. tomorrow/tonight I WILL release this. promise no fingers ds.


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (752 of 752 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-16.76%25-green.svg)


 [FEATURE] Add Windows driver installer and uinstaller applications

- Add installer and uninstaller application for PCIe driver.
- Add NSIS script to prepare openPOWERLINK installer.

Remove
- **Update Windows driver project to add install path.**


Still missing corrections:
1) Matrix is now a real positive
2) input vectors s and t are from R^n not R^d

Proposition:

We consider :math:`X: \Omega \times\cD \mapsto \Rset` a
stochastic process of dimension 1, where :math:`\omega \in \Omega`
is an event, :math:`\cD` is a domain of :math:`\Rset^n`, :math:`\vect{t}\in \cD` is a multivariate index and :math:`X(\omega, \vect{t}) \in \Rset`.

The covariance function :math:`C : \cD \times \cD \mapsto  \Rset_{+}^{*}` is defined as:

.. math::
    C(\vect{s}, \vect{t})=\prod_{k=1}^K  C_k(\vect{s}_k, \vect{t}_k)

where :math:`C_k : \cD_k \times \cD_k \mapsto  \cM_{d \times d}(\Rset)`  with :math:`\cD_k \in \Rset^{d_k}`, :math:`\vect{s} = (\vect{s}_1, \dots, \vect{s}_k)` and :math:`d_1+\dots+d_k=n`.


I remember, it was only a property part comment. Are you sure you sent it - i remember you commented it, that you had not tested it yourself but it could be done with array observers like .property('...@each') - but it was just a hint, there was no code. Sry, i must have deleted it or sth. Can you send it here once again? I have no idea how to make this snippet simpler with array observers - it requires a lot of changes and I didnt make it work at the end - as i described in my comment.


`0.4` should be written without a leading zero as `.4`


Merge rule `.primary-toggle + label` with rule on line 1


Merge rule `.primary-toggle + label:after` with rule on line 1


Merge rule `.primary-toggle + label:before` with rule on line 1


Selector should have depth of applicability no greater than 3, but was 4


Assignment Branch Condition size for set_users_in_gon is too high. [32/15]<br>Method has too many lines. [19/10]


the commit message is okay


Assignment Branch Condition size for find_missing_users is too high. [20.02/15]


Align the elements of an array literal if they span more than one line.


Align the operands of an expression in an assignment spanning multiple lines.


Use empty lines between defs.


@ptsteadman I ran the migrations but didn't restart the app... now I gotta go :-1:


We should probably release the allocated NetID here when the WriteNetNamespace fails?  We should check and verify that WriteNetNamespace() definitely doesn't write the netid anywhere when it returns an error, and then release the allocated NetID when there's an error here.


Passing for me too, since you are passing in `this` as the context in `this.source.filter(this._isQueried, this)`.    I guess that's OK.


Great, this looks good.


Hmm, would be useful. But I must warn you - I have no experiences in using wordpress :worried:


Maybe we should use `CharT`, `IntT` etc. for the C aliases to better disambiguate them?


I'm most likely missing it, but what adds a namespace to the oc.adminNamespaces map?


Whats the difference between:

```python
NotSet = object()
```
and
```python
class ESettings(object):
    pass
```
 Can we use all `class foo(object): pass` :grey_question: looks more Pythonic than the other way :grey_question:


(You do have a few jshint errors though.)


				Property cdaProperty = cdaSourceClass.getOwnedAttribute(
-					null, getCDAClass((Classifier) property.getType()));
+					property.getName(), getCDAClass((Classifier) property.getType()));
 				if (cdaProperty != null && cdaProperty.getName() != null) {
 					String modelPrefix = getExtensionNamespace(cdaProperty.getType());
 					elementName = !StringUtils.isEmpty(modelPrefix)

The issue i was trying to address was when a template has two of the same types it will return one or the other can you give me an example of when this is not working


That would probably, as long as you're okey with the default changing to
not highlight as markup?

On Thursday, August 20, 2015, robert mcguinness <notifications@github.com>
wrote:

> could this same behavior be achieved by not defaulting the targets to
> type markup?
>
> var targets = className ? className.split('language-') : [''];
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/phillipj/metalsmith-prism/commit/6ecbd1dec591c84789bcf3acbeb1ed98f537b7bb#commitcomment-12806529>
> .
>


actually it's down to maxcomplexity 12 now


I like the idea, but I would suggest you to try your idea together with lowering the Queen value.

In other words, since we all know that stockfish Queen value in endgame is already high, we first decrease the Queen value in endgame, and then apply bonus if the queen is in enemy half . Or in other words we can keep Queen value the same and apply penalty if queen is in the defensive half.


Add at least 1 line descriptive DocString if you can.


1. потому что 2.3.0 как tag уже был заведен
2. спс


Cool! Glad you were able to find & follow the developments.

Currently I'm using [a custom Babel plugin](https://github.com/meteor/babel/blob/master/plugins/async-await.js) to compile `async` functions. Because Meteor has fibers, the translation is much simpler than the default Babel transform (based on Regenerator, which I also maintain). Basically it comes down to returning a wrapper function that calls [`Promise.asyncApply`](https://github.com/meteor/promise/blob/master/promise_server.js#L137).

I'm on my phone right now, or I would go into more detail. Hope that demystifies things a bit.


You did'nt delete metadata completely. do CTRL + F and type metadata


I think this can work just fine and allow SCHMO(E) into BTN cards.  I will test this later today.


Oh yes sorry, I will fix it.
If it's ok with you I'll squash everything in one commit again to be merged soon ?


There is no such literal:
```
> NaN
## ERRORS in repl-temp-000.elm #################################################

-- NAMING ERROR ---------------------------------------------- repl-temp-000.elm

Cannot find variable `NaN`

4|   NaN
     ^^^
Maybe you want one of the following?

    isNaN
    max
    tan
    Basics.isNaN
```


Fix #1299


The parameters listed in the jsdoc (a few lines above this line) need to be updated.


Hi Wi,

seems to be a good solution. I'm just afraid for the performance aspect because you are calling "gsub" many times, one time for each escape character. As this is going to be done for every string literal I think it can really slow down a lot the code generation.

I suggest to keep the "escape" function you made but to replace it with a function that use format "%q" and just replace in the replacing string the eventual offending newlines with:

string.gsub(<output of format "%q">, "\\\n", "\\n")

In this way we keep use "%q" to perform the bulk of the work. This seems to be better in term of performance since the format function is written in C.


So within this commit memory barrier is okay?

````diff
 // Target memory barrier function
-#define OPLK_MEMBAR()               ((void)0)
+#define OPLK_MEMBAR()               MemoryBarrier()
````


i would rather not make this change.

the better way i to properly test the simple SST strategy -- manually create the SST using the login resource and then use it in the strategy


![comet-hyakutake-mountain-nature-night-favim com-350091](https://cloud.githubusercontent.com/assets/204693/9384091/1c3f1d88-474e-11e5-9845-0dd9e31ace0a.jpg)


This comment could explain why, rather than repeat the content of the if condition? "Result needs reversing because to implement backward paging the view reading order is reversed" (or something more concise)


I didn't want to remove metadata, just this unused parameter in the item templates.


Same as above.


I'm not sure I understand why we are doing this. Shouldn't we just leave the DNS search list intact?


Would you terribly mind [splitting](http://stackoverflow.com/questions/1440050/how-to-split-last-commit-into-two-in-git) the whitespace changes from the actual changes affection behavior? It is really awkward to review when that is smooshed together.

Also, please note that it would be a very good thing if you could install the new [Git SDK](http://git-for-windows.github.io/#download-sdk) and port this Pull Request to [Git for Windows 2.x](http://git-for-windows.github.io/) because Git for Windows 1.x is now kicking back on a beach.


@ichrispa the typo is here ;)


There existed already signatures for taskmgr and regedit.  I'm not sure if I want to replace those two with this or simply add two more to have more descriptive names.


And in the separated commit the ifdef changes, right?


I prefer to have HasProperties (or something equivalent) in `records`. Importing `ops` shouldn't be needed


Modified https://github.com/openmrs/openmrs-module-legacyui/commit/cd2fb109d37b107cb999d06e9b4c4a17008e3881


Modified https://github.com/openmrs/openmrs-module-legacyui/commit/0e35bf2ad8d8c6c29b41543bd584607b13bce72c


@Rocky640 but psq is applied to all phases of the game no? Can't check this just on a mobile at the moment.

Interesting idea fauzi


No `print()` can we use `logging.debug` at least :grey_question:
`print()` may fail if the printed stuff is unicode *(arab, japanese, ñ, etc)* on Python2
or comment out all together...


I was only introducing self in methods that needed to use callbacks.


Can you please elaborate why we need to loop through the 2 nameservers' lists? Maybe, we'd loop through them with the `lineinfile` Ansible module? With that module we won't be duplicating nameservers on each run.


I hope it will work :o:


This actually won't work, because it's a callback and the function would lose context, you either need the anonymous wrapper function or use `self.render.bind(self)` which I wanted to avoid.


No, it isn’t just for GUI apps. Many users prefer to install everything through us and we may one day separate completely from homebrew, so it is perfectly OK to have software in common.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.920.25c26595a82f8b727fb54391ef7084fc8bbd4e8b](http://build.palaso.org/viewLog.html?buildId=61605&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:04:54


Possibly expand on this to give context to the below "the start of the page to retrieve":

"Each call to queryPage() calculates the parameters required to get the pages preceding and following it. This class is used to persist the values for either page as tokens a caller can pass back to queryPage to retrieve them at a later point."


Right that one is what I wanted to fix, I just messed up!


Specific learning goals will make it easier for participants to know exactly what they should know at the end of each section. If the goals are related to the assessments, it will make them easy to tie together and unambiguous. The problem with asking participants to understand an issue is that it is hard to verify, and most folks will quickly feel that they reach some level of understanding.

For #2, it may be worthwhile creating 3 large learning goals for the entire workshop, and then 1-3 goals for each section.


Fine with me, but then why did you move it too the ops? I just removed this because I thought you forgot to do it :laughing:


Are there any reasons why we can't stay with a single dnsmasq installation without setting up the networkmanager service?


j'imagine qu'il faut aussi rajouter un champ `wakeup_port` dans `servers` ? Ou alors `wakeup_url` accepte une syntaxe avec un port ? tu mets quoi en local ?

J'essaye de réfléchir à comment ça pourrait se passer avec une grappe de serveurs à la demande derrière un NAT (du type aws beanstalk, c'est pas ce qu'on va faire tout de suite donc c'est pas urgent)... J'ai l'impression que ce serait difficile dans l'état actuel non ?


@David-JaeHyuk
Have you tried Intel's MPL3115A2 driver? https://software.intel.com/en-us/node/559209

Here is some example code:
https://github.com/intel-iot-devkit/upm/blob/master/examples/c%2B%2B/mpl3115a2.cxx
https://github.com/intel-iot-devkit/upm/tree/master/src/mpl3115a2


101280704_lowTempRad (Nigusse) - x86_64-MacOS-10.9-clang: OK (1779 of 1794 tests passed)

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


Whats a `do_dp()` :grey_question: what it does :grey_question:
`set_env_key()` is more easy, this looks like a getter/setter pair of functions somehow...


Het ziet goed uit


Awesome :octocat:


That's what I thought when I wrote the code and indeed [java docs](https://docs.oracle.com/javase/tutorial/java/nutsandbolts/datatypes.html) state the default is `false` but also say
>Relying on such default values, however, is generally considered bad programming style.
All that said I was surprised to get a NPE running the tests at the line `uriBuilder.query("descending", !descending);`
setting the value to `false` prevents the NPE.
This might be some kind of JVM issue, but given that best practice is to declare a value anyway it seemed easier to set false!


Could you instead of just consider the problematic '+' caracter sanitize the file name ?


solved that


:ok_hand: :stars:


May I say "that goes any faster"?


@tdhock The opacity for legend entries didn't kick in until `update_selector()` was called.  I think I fixed it here, but could you +1 to confirm?  [Example](http://bl.ocks.org/kferris10/raw/084ca1abf565530a3cc1/)


Couple notes:
* All code inside the event manager, should be inside either an initializer, or a method. I'd put the `puts` inside an initializer specifically.


These variables are only used once; inline them in the uses on lines 380,381.


No, we don't ever want to allow SSH from the internet, but always only from our office ranges and other specific cases.


The literal is in JavaScript, `0 / 0` evaluates to NaN


**SUCCESS** - TeamCity Quickstarts :: Pull requests monitor [Build 94/mgencur:cameljdg_nosnapshots/jdg-6.6.x](http://ci.infinispan.org/viewLog.html?buildId=29389&buildTypeId=quickstarts_PullRequestsMonitor&guest=1) for 306c457ad7ec3453888e65655ee6de5cedf34172
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:08


How come we use `i = lumphash[hash]` as an array index here, is this alright?


`- 1 / 0` evaluates to `-Infinity` `1 / 0` to `Infinity`


Here is the Queen PSQT, for columns ABCD, and rank 1 to 8
{ // Queen
   { S( 0,-70), S(-3,-57), S(-4,-41), S(-1,-29) },
   { S(-4,-58), S( 6,-30), S( 9,-21), S( 8, -4) },
   { S(-2,-39), S( 6,-17), S( 9, -7), S( 9,  5) },
   { S(-1,-29), S( 8, -5), S(10,  9), S( 7, 17) },
   { S(-3,-27), S( 9, -5), S( 8, 10), S( 7, 23) },
   { S(-2,-40), S( 6,-16), S( 8,-11), S(10,  3) },
   { S(-2,-54), S( 7,-30), S( 7,-21), S( 6, -7) },
   { S(-1,-75), S(-4,-54), S(-1,-44), S( 0,-30) }
  },

For the Rook, the eg value is all 0 !


Shouldn't those "hack"s be "FIXME"s instead, so that anyone could see them explicitly as issues on the level of IDEs?


It's not clear here that `limit` is `rowsPerPage + 1`. (Also given limit is `+1` is this off-by-one?)


Maybe @thewatts will disagree, but i'd probably put this spec file inside a 'specs' directory in the root, and not inside the lib folder. Other than that, nice job man. (nathaniel may have some other input on these specs).


yes


Could've renamed `pageList` rather than adding a comment?


                                                                                I made a guide on the wiki, as it seemed more appropriate for the kind of content.                                                                                http://wiki.openwrt.org/doc/howto/openconnect-setup                                                                                Sent from my BlackBerry Passport                                                                                From: Nikos MavrogiannopoulosSent: Thursday, August 20, 2015 6:00 AMTo: openwrt/packagesReply To: openwrt/packagesCc: waitfor1tSubject: Re: [packages] openconnect: corrected port descriptions in README (0313444)@waitfor1t would you add the text you propose? In that case make it a separate description on the README rather than replace it.

—Reply to this email directly or view it on GitHub.


oh wait.. the new arbNum hasn't been added yet


delete this comment?


final


"requires a _temporary_ reversal"?


Any chance for a new release .zip?


gitignoreわからないんで託します！
![LGTM](http://lgtm.in/p/USR9rpK0q)


@nhoizey it is (sorry for the late reply) and I've even noticed a library for Jekyll that can do this already so I might try to get a-hold of that library author and see if they want to work together and have them provide that functionality through a plugin into this one as a filter.


All great points. The updated version will be pushed in a wip-master branch.


if ... if --> it else if


Thanks to @Yomguithereal for noticing:  https://github.com/jacomyal/sigma.js/pull/639#discussion_r37513527


Some base64 encoders in the wild incorrectly pad data before the end of
data. For instance, the string "testtesttest" might be encoded as
"dGVzdA==dGVzdA==dGVzdA==" instead of "dGVzdHRlc3R0ZXN0". This is,
technically speaking, a bug.

This patch makes the decoder tolerant to this kind of nonsense.
Intermediary padding is processed and then decoding carries on as
though a new string were provided.


node doesn't behave by the same rules as the browser with respect to its global object. all code by default is running in the scope of a module, even your main file, so a `var` at the top level of your file isn't actually making a global like it does in a browser. if you were to say `global.a = 2` instead of `var a = 2`, and then try it, you'd see the expected behavior.


Or is the MemoryBarrier() only needed in PCIe (`CONFIG_PCIE`)?


This should read
```bash
if [ ! -e %{_sysconfdir}/origin/master/master-config.yaml ] && [ ! -e %{_sysconfdir}/origin/master/ca.crt ]; then
```


I'm sure there's a joke in here somewhere about re-writing this in [Java](https://en.wikipedia.org/wiki/Java_coffee)…


Instead of a registry approach, a single overwriteable function would probably suffice. I can't imagine, that many mods want to fiddle with this at the same time. If they want to, they still could chain each other.

Since this is a potentially often called abm with nested search loop, you want to keep this lean; Which means avoiding lookups and looping through tables every second for every entity.

Besides: it would get rid of the ambiguity between "doing nothing" and "not damaging player", which is only a structural quirk of giving up decision responsibility to the registry.


fg


Please, put in blocktrans for translate it in Transifex,


I see -- setNextParam and setPrevParam only require either the details of the last row and first row of the page respectively, so need not have both.


while i do understand why using a CDN for common libraries on public websites may speed up load times on first visits, i disagree with this being an issue with the stereotypical JT installation (ie: kiosk mode).
the same argument (multiple users, same machine) invalidates my usual privacy concerns when it comes to involving CDNs in web projects, so i'm not entirely against this change.

however, there are bigger performance issues with JT (ie: server side caching of journalTOC/CrossRef API answers)


You'll need to move the `mate_stock.js` file as well for this to work.


The function call gets cached and the diff() is actually taken once. I tested on your previous example and get same speed results.


Done in  d4bbb2a


Minor: commit message "Update govuk_frontend_toolkit to 4.2.1" would be clearer :)


+NUKEVIET 4.0.23 (No release)


@rowanmiller Does this mean that I should be using this for "daily builds" going forward?


prefix your default function with "_"(make similar change in this branch for all default functions)


## This is a markdown file


@SalahAdDin It is in blocktrans.


Yes, I can change that, although I'm not sure that there's any real fix we can do without changes from RESTlet.


Is this right?  This says to me.....use curl if s3:// is in the $url line.


OPLK_MEMBAR is not used in the code anywhere but for future uses MemoryBarrier() is valid for both cases


Sweet!!


Thanks for the response!
I tried something similar, but the model I was loading required inn. My workaround was to recompile Caffe without CUDA (there is a CPU_ONLY flag) then use the [torch-caffe-binding](https://github.com/szagoruyko/torch-caffe-binding) module instead of loadcaffe. It works, but you don't get access to the model's internals..

(And I agree that a CPU implementation of inn would be ideal, but it's probably more work than just finding a machine with CUDA installed)


There is a go client lib! https://github.com/dustin/go-nma


You should have not refactored that first if condition. Now it can't exit early and will check all conditions on all iterations.


@zihbot Is <windows.h> really needed for this program?


How could it be breaking anything anyway?


It is.


@zihbot Good start.


Ah, thank you man :D


Very interesting - particularly that it led to a NPE. I'd always understood that initialising fields to the default value used to be considered bad practise because it was inefficient as the value would be set by default and then set again because of the explicit initialisation. I'd hope these days the compiler optimises such things so that it's not so inefficient.  Also, I believe it's one of the default checks in `checkstyle`. Looks like I'll now have to update virtually every Java class I've ever written to improve my coding style :)


W Pythonie istnieje wygodna funkcja do enumeracji obiektów na liście razem z indeksami:

for i, result in enumerate(results):
    ...


This commit makes it possible for users to enter assumed register values on procedures. That is, partially addresses issue #25 .


"ACROPOLIS BUS MANAGEMENT SYSTEM"

HOME

About_Us

Institute

Staff

Contacts

Admin_Login

ERP LOGIN

<input type="button" value="Internet">

ERP LOGIN

<input type="button" value="Internet">

copyright@AAG


Github is very good! :smiley:

but it is *definitely* better with git especially on Linux.


Reviewed PR and good with merge.


Oh, yeah. I mixed up JavaScript and Elm. Thanks!


Sicher, dass das daran liegt?

Also der Script für Travis (.travis.yml) kann derzeit gar nicht erfolgreich sein, weil es davon ausgeht, dass ein Emulator gestartet wird, was nicht der Fall ist.


Added an option for new kml menu option


What I mean is that maybe the `State.next()` problem was related to the framer.js version shipped with the latest Framer Studio, and not with commit [f6400e6](https://github.com/koenbok/Framer/commit/f6400e66bc816a283a57327b5df05de18831e116). Does that make any sense?


I take it this doesn't get populated by the firmware (like /axi/dma/broadcom,channels). Should it be?


101064176-AddCO2DemandControlledVentilation (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1797 of 1798 tests passed)

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


Does this doing anything? Isn't `@shape.object=self` what gives access in collisions callbacks.


'ufo' in this context is a shape.


Will add an option for picking a url from a file


ss


I think we are calling the wrong methods on the wrong files but the port of that code has c-style returns.  I would argue catching the exceptions is still appropriate OR we change these to be exception throwing vs c-returns.  So I think the code needs to change but if we keep c-returns then lets also catch exceptions like this patch does.


This is actually changing `LINUX_STORAGE_USE_FRAM` for the other linux boards.


> "engine" invokes BQP which in turn either applies the changes locally via the IM or sends them remotely

where exactly is the code which makes this decision?


My old map. If KMLMap works mabe this will be deleted


Oh, I don't think it's confusing, at least when I'm inside `LibC` I know `Char` is `LibC::Char`. But this broke because we introduced these aliases **and** because `printf` had a wrong type.


Okay, thanks for the clarification!


DD


Ooops. Thanks for catching that.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.921.b9ac73bb7a51be3819b6d1270a704927a104a4dc](http://build.palaso.org/viewLog.html?buildId=61606&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:03:06


Notice you return exe_file instead of exec_file so the compilation will fail. Also the is_osx_bundle() check is unnecessary - exec_file will be NULL if it's not a bundle.

Otherwise +1 for simplifying the patch, I wasn't very fond of the _NSGetExecutablePath() call.


tested and working


Sorry.

You mean start the bit after the colon lowercase, right?

Also, is it OK if the breaking change wraps multiple lines?

Can I git commit --amend and git push -f, to fix the message, or is that bad?


just to trigger hook


I've linked the wrong Commit..
https://github.com/StiviiK/lua_utils/commit/fd3609c241fa4d5e26facb9809845403ef32150c


Maybe a FileNotFoundException would be more accurate than RuntimeException ? Although the message is clear.


101064176-AddCO2DemandControlledVentilation (lgu1234) - 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)


here it is ! I did however kept the event registration, but just published a event through minpubsub (it case it can be handy for anyone)


``` python
if __name__ == '__main__':
    main()
```


The IOCTL buffer**s** are used...
or
The IOCTL buffer **is** used...


Sorry, but I've now decided to change the API of this function again. The third `flags` argument is now gone, as it wasn't commonly used and it was annoying having to always set it.

If you want to maintain backward compatibility with all recent LALSuite versions, you probably need to do something like this:
```
try:
   iterator = lalpulsar.CreateLatticeTilingIterator(tiling, 3)
except:
   iterator = lalpulsar.CreateLatticeTilingIterator(tiling, 3, 0)   # 0: default flags
```
or else use some other way to determine how many arguments the function expects.

In future, I'll try very hard not to alter the API of the functions used by this code, to minimize these kinds of issues coming up.


Yeah right, the message bit after the colon :)

Sure the breaking change can wrap multiple lines, just no indentation, because that might mess with how `conventional-changelog` formats that.

As long as things are happening on branches/PRs and not on master squashing commits, rewriting commit messages and force pushing is totally cool, and more than that: appreciated :)

Thank you!


A few notes/observations while I was looking up my changes from way back when:

- I had made a few allocation path changes in the past, but didn't touch this part (see 119e16c)
- we've never used the global seed in CODES (ie we always pass NULL into `tw_define_lps`). As discussed before, it's been discouraged to do our own manual clcg4 seeding and instead rely on the ROSS defaults.
- LP RNGs don't interact with g_tw_rng_seed at all in the current setup, AFAICT - upon a call to `tw_rand_init_streams->tw_rand_initial_seed->rng_init_generator`, the SeedType argument is fixed at InitialSeed, which doesn't use the global rng that is initialized as a result of `tw_pe_init->tw_rand_init->rng_init`. I should note that none of the other seed types make an appearance in the ROSS codebase, so unless people are hand-crafting LP RNG structures, I have the feeling that this code hasn't been executed in a long time.


@emeraldvision you could probably switch out the `remove_from_game` to using this destroyable module on the lasers. However at the very least, the `space.add_post_step_callback` seems like a safer way to go to make sure it's not in the middle of step, especially when you add the laser beam to be destroyed on collide.


Yes, sounds good.  I think we are finished, or almost finished.


@djstrong
Move to parent and use in other classes.


It probably should (it was and still is on the command line), but at the moment the firmware gets it wrong (sort of). The firmware is built with 0x7f35 hardcoded, which is accurate in the sense that the firmware won't be using those channels, but the ARM can't use the top 3 channels (12, 13 & 14) because of the interrupt sharing with channel 11.


If there are many documents being uploaded at once, could it be a problem to hold them in memory until all of them are uploaded?


I should qualify my comment a bit more.  We should not be trying to load a .pem file as a java keystore (wrong methods on wrong files).


This line could live here without problems. Methods returning composite, not-much-related results aren't good.


@mchaver Do you know if this package fails when trying to use mtl 2.2.1 and some of the other changed versions?


Nie rozumiem tej struktury danych. Po co przechowujesz tutaj wyniki pośrednie w data[-1][-1][-1], skoro później nigdzie z nich nie korzystasz? Wystarczy chyba przypisać je do zmiennej lokalnej... Poza tym możesz też od razu uśrednić te wyniki po 20 symulacjach, skoro i tak nie zamierzasz ich zwracać. Wg mnie efektem działania tej pętli powinna być tabela DataFrame z kolumnami: "CSA_mean", "CSA_var", ... oraz wierszami "max_avg_bet", "max_avg_clust"...


addressed.


copy many to one? o_O


It's exceptional case, but in case of propagation of a command which is neither 'rename' nor 'restore' (it seems to be handled by argparse module) - we must exit with non-zero code point from the script. sys.exit() without params returns 0 exit code, which is minor drawback assumint the statement above.


Why no more `#undef`?


```php
$msg = '"%1$s" requires WordPress %2$s or higher, and has been deactivated! Please upgrade WordPress and try again.<br /><br />Back to <a href="%3$s">WordPress admin</a>.';
$url = esc_url( admin_url() );
$msg = sprintf(
	$msg;
	$plugin_data[ 'Name' ],
	WPVERSION_CONST
	$url
);
```


yep i totally agree. this reading though doesn't come with any code for them to try––its just a reading for them to get a general idea of how scraping works. next lab is meant to be a code along so maybe we can leave this as is and set up something on gh pages for that?


fixed.


yes they do.


無いコントローラを呼ぶというのはどういうことでしょうか。
自分に関係がありそうな感じもしますが、あまり覚えがありません、、、


@vahidreza yes, com.medlei.pushplugin version 0.1.2. Btw I will not keep updating that plugin anymore and instead will work on medlei-parse-push-plugin on npm repository. To use this version simple change your plugin tag to:

```xml
<gap:plugin name="medlei-parse-push-plugin" version="~0.1"  source="npm">
        <param name="APP_ID" value="PARSE_APP_ID" />
        <param name="CLIENT_KEY" value="PARSE_CLIENT_KEY" />
</gap:plugin>
```
I use parse a lot so I will be adding more features as I need them, or as they are requested. Also I don't like the API that much, I just based my work on a previous plugin and added what I needed. So expect the API to change a bit on future releases to make it easier to use.


status should be boolean in this case


Ta pętla jest praktycznie identyczna do poprzedniej. Wniosek: powinna zostać zamieniona w funkcję, która przyjmuje networks jako argument :)


BrowserWindow*


and maybe rename to
done: true/false


Looks good to me.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 17](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=116&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


By default it will create 'restore.info' in the current dir which cannot be non-existing.
Here is repeatable code which contradicts to DRY principle:
```python
output = args.output if args.output else 'restore.info'
shuffler.rename(args.dirname, output)
```


101064176-AddCO2DemandControlledVentilation (lgu1234) - x86_64-MacOS-10.9-clang: OK (1791 of 1792 tests passed)

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


wrong, you should change checkbox status (checked/unckeched)


Good point, it is very important to make that clear, added in de27c20


Done in de27c20


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 17](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=116&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:57


Ok I could start find out some thing from sources code.

https://github.com/temasek/android_external_bluetooth_bluedroid/blob/cm-12.1/main/bte_main.c#L51
https://github.com/temasek/android_external_bluetooth_bluedroid/blob/cm-12.1/main/bte_main.c#L170


The actual default here is `sys.stdout`, so we could just have `out=sys.stdout`, but I somehow don't like this because the function signature is ugly (though admittedly more informative):  `f(a=<open file '<stdout>', mode 'w' at 0x10028e150>)`.  In this function `out=None` means something different, which is to return a dict of the info.

I was never totally happy with this API so I'm game for suggestions.


index = range(0, 2010, 10)


ok


missed checkbox


I guess yes. Sorry, I don't have much idea of desugaring so i don't use it much... Will add desugaring if it is required.


Correct, this is out by one and it is not clear, fixed in de27c20


It's better to use uuid module for random id generations. Something like:
```python
import uuid
random_id = uuid.uuid4()
hashname = hashlib.md5(str(random_id)).hexdigest() + '.mp3'
```


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1277 of 1278 tests passed)

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


Added in de27c20


Hm, thinking again about that I would prefer not changing it, otherwise we need to add a comment on this to the commit message. So if it is not used and needed then we don't change it. What is your view?


I think it got lost when you changed your repo to be forked from flamejs. It was something like this:
```javascript
    selectHighlightedItem: function() {
        var chosenValue = Ember.get(this.get('_highlightedItem'), this.get('itemValueKey'));
        var currentValue = this.get('value');
        if (Ember.isArray(chosenValue)) {
            currentValue.replace(0, currentValue.get('length'), chosenValue);
        } else {
            if (currentValue.contains(chosenValue)) {
                if (currentValue.get('length') > 1) currentValue.removeObject(chosenValue);
            } else {
                currentValue.pushObject(chosenValue);
            }
        }
        // Don't really see the point of having action support for this class, since the menu
        // stays open after selecting an item.
    }
```
If you say that the default value of the attribute gets modified this way (I haven't checked it and we haven't used array for record attributes before), then this piece of code is not at fault.
My best guess would be that the datastore does not handle arrays in a special way, which means that it just grabs the same reference from the prototype for each instance. This is also true for normal Ember objects:
```javascript
var Foo = Ember.Object.extend({
    bar: [1, 2, 3]
});

var baz = Foo.create();
var quux = Foo.create();
quux.get('bar').pushObject(4);
alert(baz.get('bar'));
```


Should be easy to test if it's required. Just try to create an array with a typedef'd element type.


To avoid platform dependent concetns the optimal way is to concatenate with:
```python
os.path.join(...)
```


Since PostCSS 5 is a breaking release it might have been better to bump to `1.0.0`.


@emeraldvision, never mind on destroyable module, because Ricky tied that to explosion animation, but definitely `add_post_step_callback`


Built-in function file gets rebound.
In my taste the clearer way is:
```python
for _file in files:
  if _file.endswith('.mp3')
    ...
```


Personally I'd say the firmware is correct in reporting 12, 13, 14 as available.
It's the linux driver than doesn't handle interrupt sharing that should mask those out. In theory an improved driver would make those available again.


Hi Franko,

Sorry I don't really understand your suggestion.
Do you mean this?
```
str = string.format("%q", val)
str = escape(str)
```

Another reason I removed format "%q" is because it has different [implementation](https://github.com/keplerproject/lua-compat-5.2/issues/13) in Lua 5.1 and 5.2 standard lib, and I am not sure if it will cause us surprise later.


Spend 3 hours on fixing my jenkins built. Don't know what changed here but I fixed the issue...

Commit: you can't send a null object to request.defaults

Before:
```
const authRequest = request.defaults();
```

After:
```
const authRequest = request.defaults({});
```


test comment


I guess we could add a kwarg and make things more explicit / consistent:
```
def info(self, option='attributes', out=sys.stdout, return_attributes=False):
```
Then `out=None` would mean no output, and this would typically be paired with `return_attributes=True`.  That would make calls to info in this mode more clear.  You still have the problem of an ugly function signature.  I guess another option is a default of `out=None` which would mean `sys.stdout`, but then if you set `return_attributes=True` then that would also imply no output.  So a typical call would be `attrs = t.info(return_attributes=True)`.


directories variable is not used here... idiomatic way to declare such variable is the following:
```python
for root, _, files in os.walk(dirname):
  ...
```


Using `sys.stdout` as an argument default is unfortunately problematic, since it will capture `sys.stdout` at the time the function was *defined*, which may be different from `sys.stdout` when the function is called.

There are a few other functions / methods floating around astropy like this, where the approach is `out=None` or the default (`sys.stdout`), and `out=False` for *no* output (but return a dict / list / etc.)


Updated the readme


I don't know about pcre, but couldn't you just call `pcre_exec` with `NULL` extra if `pcre_study` fails?


Extra note. The Frame fields in the editor were never used and due to a more advanced system on this department Kthura communicates with, they are no longer needed either. As a result these fields are now deprecated and will soon be removed. Fields with some other functionality are more likely to take their place.


tested and working


Yes. In this implementation, Make_Block clears memory, so it's already zero.


Esse exercício tem que ser sem OO, apenas funções. Mais duas coisas:
* Seguir a [PSR-2](http://www.php-fig.org/psr/psr-2/pt-br/);
* E encapsular a função ```echo``` na função ```$aula1->assert```.


This actually should be lowercase. https://developer.mozilla.org/en-US/docs/Glossary/Chrome


We're being a bit silly here it is defaulting to `null` not `false` because it is a `Boolean` not a `boolean`


I get more and more confident that we should include all the necessary dlls...
Could you add all the referenced dlls as hardcopy and I'll try your solution?
```
Microsoft.TeamFoundation
Microsoft.TeamFoundation.Client
Microsoft.TeamFoundation.Common
Microsoft.TeamFoundation.Diff
Microsoft.TeamFoundation.VersionControl.Client
Microsoft.TeamFoundation.VersionControl.Common
Microsoft.VisualStudio.CoreUtility
Microsoft.VisualStudio.Services.Common
Microsoft.VisualStudio.Shell.Immutable.10.0
Microsoft.VisualStudio.TeamFoundation
Microsoft.VisualStudio.Text.Data
Microsoft.VisualStudio.Text.Logic
Microsoft.VisualStudio.Text.UI
Microsoft.VisualStudio.Text.UI.Wpf
```


```python
with open(dirname, 'r') as f:
  ...
```


101064176-AddCO2DemandControlledVentilation (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1797 of 1798 tests passed)

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


Of course, I forgot about that more fundamental problem with `sys.stdout` in the function def.


```python
f = open(output, 'w')
```


`s/plugin/extension/`? Just to separate the error clearly from the old plugin model.


I'm not sure, I think Scott might have changed a few things because he was running 7.8.3 and I was running 7.8.4, so need to confirm it works on both.


`false` while you're at it


Reminder: need to add `initial_allocation` and `allocation_quantum` args here.


When the driver can use those top bits we'll let the firmware patch the DTB, but until then I'd rather use a restricted mask in the DTB than patch the driver to ignore them (which will lead to questions about why those channels aren't being used).


Shall we use `Extension` instead of `Plugin`? It might make it clear which of the two "plugin" interfaces are being used.


Would it make sense to move `script/smart_answer_flow_template.erb` to `script/templates/smart_answer_flow.erb` as it is not a runnable script?


I mean selecting points by drawing a rectangle with mouse down. Not the correct term probably... :)


Both methods should be split into at least 2 methods to satisfy SOLID and DRY principle. Both methods do too many things in one place. Moreover there is a code duplication in case of getting files from directory.


It would be great if Python had a built-in mechanism to capture defaults that should be evaluated at call time rather than definition time :/  I believe this has been brought up countless times on python-ideas but nobody has ever come up with a workable solution for some reason...


Better us dependency Injection


You need to check options before trying to get .pool

It took me hours to find the issue. See this https://github.com/request/request/commit/58e8f53f6272f36aaa3e5915bab027e6da5da22d#commitcomment-12808032


Looks fine for me.


of*


use the same template used for searchResultSetTemp, without custom handlebars templates


Can you submit an *issue* about it, just point out which version you are using.


Thank you sooo much for your help!  After using a function for the checklist-model as you suggested, it all worked GREAT!

[Beryl Small]

Beryl Small
IT / Programmer
985.629.5934 (D)
800.892.8179 (O)
985.892.8114 (F)
http://www.pecsafety.com/
This e-mail is confidential and is intended solely for the use of the recipient.  If you are not the intended recipient, be advised that you have received this e-mail in error and that any use, dissemination, forwarding, printing, or copying of this e-mail is strictly prohibited.

From: Adrian Ber [mailto:notifications@github.com]
Sent: Thursday, August 20, 2015 3:12 AM
To: vitalets/checklist-model <checklist-model@noreply.github.com>
Cc: Beryl Small <Beryl@pecsafety.com>
Subject: Re: [checklist-model] function as model fix and test case (52e4be0)

But you're not using the checklist-model as a function in that example. Btw, are you fine with moving our conversation to the issue list? Just submit a new issue with the plunkr example.

—
Reply to this email directly or view it on GitHub<https://github.com/vitalets/checklist-model/commit/52e4be0801f180a41c37725b25a554f8c22278a2#commitcomment-12801946>.


I am fine to not make the change. we can include in future when it is needed.


Supper picky: Logging is not consistent with the other functions here... some spaces around the `=` and a `:` after the function name.


A mess from moving functions around, I guess. Thanks.


ah, i see. Next() is a passthrough to the scanNode if sorting isn't required. cool.


extract to const in the test class so that it's not a magic number but an explicit test const


Don't all the `return v` above leak `compiled_re` and `pcre_extra`?


Sounds awkward. I prefer: "but it has a visual state indicating it will not respond..."


Hmm, this looks rather noisy to me. I know we spoke about using `next_node{}` everywhere, but I believe the `option value: :next_node_name` syntax is worth using as it significantly reduces the noise in the file and is easier to read. What's your thinking?


@stefanceriu I couldn't find `self.pageViewController setBounce` method, maybe 'self.pageViewController.shouldBounce'


This is cool! I've never really looked at the inline OCaml generator before now :)


The IOCTL buffer is used...


`fsockopen` et donc `wakeup_url` accepte une syntaxe avec un port; j'utilise `udp://localhost:20000` et ça fonctionne (cf. la [doc PHP](http://php.net/manual/fr/transports.inet.php)).

Derrière un NAT, je ne vois pas de solution simple; je pense que le moins coûteux en ressources serait d'avoir un mini-proxy auquel les graderserver se connecteraient pour "écouter", ce mini-proxy servant de relais pour transmettre le signal de réveil de la graderqueue. Après, je ne connais pas trop AWS, mais il me semble que tous les serveurs sont sur le même réseau virtuel ? Donc si la graderqueue est aussi sur un serveur virtuel, elle pourra quand même contacter les autres... On verra quand on mettra ça en place je suppose.


It's a totally valid case. A null pcre_extra means there is no possible optimization for the RE. Error cases are reported on the error str var.


add a default template in unbxdSearch.js for `viewTypeContainerTemp` like `paginationTemp`


@ide Shouldn't all changes be committed as pull requests so we can land them in the internal fb repo first and then export back to github?

I'm just doing an export and this is getting overwritten back to the old version because we have "facebook/jest#0.5.x" internally at fb.


Will need the `*allocation*` params here too.


@simov https://github.com/aamirafridi/request/pull/1/


Please could you change the variable name to pageTypes here, since it's always an array? (The page_type URL parameter can stay)


Oh, I just realised t's a `Boolean` not a `boolean`, so the default value would be null and initialising would be necessary!  Oops!


sounds reasonable to me.  i wonder if this is considering a breaking change or a bug fix? :smile:


nice


Thanks, I'll check.


", an example is" => ". For example:" like above


this one totally threw me off, but looks right once I realized this was duplicating the one below  also


Useful test case. It does actually reveal a bug in the bytecode implementation.


I use ```"request": "^2.55.0"```


please consider extracting an inner static class with those methods and passing that class to the builder


This looks a bit funny


How did you determine the 2:1 ratio here? If we're modeling relative throughput, the page below shows that SELECTS vs. INSERTS/UPDATES are 10.8 : 7.9

https://openedx.atlassian.net/wiki/display/PLAT/Courseware+Student+Module+%28CSM%29+Replacement


Oh, just saw your comment after I'd written mine, we've both realised we made the same mistake, not realising it's an object.


heh


Should this not be `title: |-` as it's a multi-line block?


Likewise - could `self.target_content_type` be changed to `self.target_content_types`, since we're standardising on it being a list?


:8ball: from me, would still appreciate someone else give it a look


`.decode` will raise `UnicodeDecodeError` that you're not catching on invalid input


Remove "that are"? Sorry, I find "that" clauses generally superfluous :-P


Maybe this could be moved into its own function?