[Feature]: support to all Jira events

Describe the feature
We would need that the agent handles all the possible events triggered by Jira.
Here is the full list: https://developer.atlassian.com/server/jira/platform/webhooks/#configuring-a-webhook
A possible option could also make it configurable.
Code of Conduct

[X] I agree to follow this project’s Code of Conduct

At the moment, the supported events are listed here.
It is not simple to add support to each events since it is not documented the event payload.
Do you need any other event types specifically?
Probably in the future it would be configurable, but it is quite simple add new events for jira (once tested the event payload) in this page.


InfluxDB 2.0: The "/documents/templates/{templateID}/labels" returns null instead of an empty array [area/api]

The GET /documents/templates/{templateID}/labels is defined as:
'/documents/templates/{templateID}/labels':
    get:
      tags:
        - Templates
      summary: list all labels for a template
      parameters:
        - $ref: '#/components/parameters/TraceSpan'
        - in: path
          name: templateID
          schema:
            type: string
          required: true
          description: ID of template
      responses:
        '200':
          description: a list of all labels for a template
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/LabelsResponse"
        default:
          description: unexpected error
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Error"

When I trying get the list of labels for template without labels than the response should be an empty array as is common in rest of API not the null:
-> GET
-> http://<IP_ADDRESS>:9999/api/v2/documents/templates/03aee1db49909000/labels

<- 200
<- OK
<- {"links":{"self":"/api/v2/labels"},"labels":null}

introduced by #13208

Currently it's not works. I tested it over latest docker build and also over the locally builded binary.
cc @kelwang


[RN] Error: Uncaught TypeError: e is not a function

Precondition:
The "My profile" page is opened (editing avatar page)
The avatar is empty (placeholder-avatar is displayed)
Steps to reproduce:

Click the "Upload photo" button
Pay attention to the error in the console

Actual result: The error message is displayed on the console after clicking the "Upload" button
Expected result: The error message shouldn't be displayed on the console after clicking the "Upload" button

Checked on  native 1.3.1-1
LG V30+ // Android 8.0 // Google Chrome 86.0.4240.75
Desktop: Windows 10 // Google Chrome  86.0.4240.75

@WolfVV it's already hotfixed on react-native. Please re-deploy & re-check

@mauriguz In some comment i described the rules of useOnPress:

it should be used with native onPress event produced by the some bottom level RN component (Button, TouchableOpacity etc)
do not stack useOnPress. If some our custom button already have useOnPress, no not add useOnPress to components using this button
check if some callback prop provided an event. For example dialog actions aren't providing events so useOnPress isn't needed. just useCallback in that cases

Do you or  @MathiasMoralesDixtra have checked the whole project regarding correct useOnPress usage ?

I'm running a check for the whole project to make sure usage of useOnPress follows the rules of the hook

@mauriguz now works without error but the button isn't clickable inside the photo camera It means that the center of the icon isn't clickable. The upload window will be opened if the user clicks closer to the edge of the button
Video: https://www.screencast.com/t/P52HV8mryEzG
Checked on native 1.3.1-1
Desktop: Windows 10 // Google Chrome  86.0.4240.75

@WolfVV if there's a new issue then open a separate one. and close this one.
did you encounter this issue with other buttons also?

@sirpy ok I'll open another one
I didn't encounter like that but if I meet, I would add a comment to a new issue that I'm going to create

@mauriguz left a comment there #2653


Add first rule for sulu rector

This pull request should contain the first rule which should upgrade the deprecated methods on the https://github.com/sulu/sulu/blob/2.5/src/Sulu/Component/Localization/Localization.php object.

Seems like I'm currently missing something in the test setup. I run into:
InvalidArgumentException: Could not resolve argument "Rector\Config\RectorConfig $rectorConfig" for
"/home/runner/work/sulu-rector/sulu-rector/vendor/rector/rector-src/src/Kernel/../../confi/config.php".
...
Caused by
LogicException: You cannot use the ConfigBuilders without providing a class implementing ConfigBuilderGeneratorInterface.

/cc @tomasvotruba

Quick Setup Script if you want try locally
git clone git@github.com:alexander-schranz/sulu-rector.git
cd sulu-rector/
git checkout feature/first-rule
composer update
composer test

My rought guess would be:
composer install

Why? When you use "rector/rector-src": "dev-main", in composer.json, the patches are not applied yet. So you have to "compile" Rector yourself.
I recommend using "rector/rector" as standard and downgraded to PHP 7.1. The "rector/rector-src" is only for core Rector packages and require PHP 8.1

The best inspiration of 3rd party using Rector for upgrades would be:
https://github.com/craftcms/rector/blob/main/composer.json

@TomasVotruba Thx for the quick response and thx for the reference project. Was using rector-symfony as reference, but seems like there it works differently. The tests are running now :) Now I need to checkout why the ruleset is not yet applied correctly.

:+1:
I'd like to run it locally myself, but fork and PR does not work quote easily :/
Could you merge this one to master or give me contributors access here?


How do I set up a loop?

I'm using control_box, I carefully read the instruction interface, and github's documentation, I tried it for 2 hours yesterday and I don't know how to set it up, a code I want,
For example, when port B has an input signal, then port Y will output 20 tick true, then 20 tick false, and then 20 tick true. And so on and so forth.

Hi,
the RLC works signal based, so you don't implement like in normal imperative languages. It's actually simpler.
Means, you don't have loops or code that can hang up. Instead, the whole code is executed in one tick, and the values of the variables remain, so you can re-use them in the next tick. Then you use timers, counters, edge detectors etc. This also gives the mod a possibility to optimize performance - if you have something changed every 20 ticks, you don't need to waste server CPU for 19 ticks, except an input port changes ;)
So, the simplest solution is to generate a pulse every 20 ticks. The Timed Intervals TIVx objects can do that for you, I use TIV1 here. Then you toggle Y with that one tick pulse (here P), and force it off with B (if B not on/true).
# Pulse every 20 ticks
# (1 tck on, 19 off)
P = TIV1(20)

# XOR toggles Y if pulse is on.
# AND forces Y off if B is off.
Y = B and (Y xor P)

Thanks, but I have a question, why don't I see the introduction of TIV on the official github documentation?

Thanks, I just tried it and it works very well.
https://github.com/stfwi/rsgauges
As you can see, I just wanted to implement the interval timer feature in the above mod, e.g. interval Xtick, true, interval Ytick false, this kind of function, but I haven't known how to do it.

Aye, G&S had grown big, but was eventually too much maintenance - hopefully someone will take over or re-use the features in that mod. I'll try to keep the Pen mod small an maintainable - although interfacing Redstone is a challenge of a kind ;).
I added the TIVx documentation in the RLC readme. Cheers and ty again for reaching out that this was missing.


Update microsoft-teams to <IP_ADDRESS>51

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask's name and version.

Thank you for the contribution. It was merged directly as d836c572a96d77189b23b14ebb8ee9059664a7e8 to keep commit history cleaner. Your contribution is still credited to you.


AsyncDisplayKit

I saw in some issue in Facebooks ASDK that someone wanted to use Neon with ASDK. They told him it would be difficult to implement but I think its as easy as
extension ASDisplayNode : Frameable, Anchorable, Alignable, Groupable {
    public var superFrame: CGRect {
        guard let supernode = supernode else {
            return CGRectZero
        }

        return supernode.frame
    }
}

Isn't it?

So Neon doesn't use autolayout?

@wieseljonas No, it doesn't. Feel free to dig into the code, it's fairly simple.

Boy @mamaral  I'm in love with Neon :+1:

Thanks @wieseljonas!! :) Hopefully will be making improvements and updates soon!


[ERROR] Docusaurus server-side rendering could not render static page with path /site/.

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I run import MonacoEditor from' react-monaco-editor' in src/pages/site; then npm run build, reporting an error: [ERROR] Docusaurus server-side rendering could not render static page with path /site/.
Reproducible demo
No response
Steps to reproduce

run import MonacoEditor from' react-monaco-editor' in src/pages/site;
npm run build

reporting an error: [ERROR] Docusaurus server-side rendering could not render static page with path /site/.
Expected behavior

Actual behavior

Your environment

Public source code:
Public site URL:
Docusaurus version used:
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

it's ok to use

how do you fix it


Importing YCSB into Eclipse and starting a new project

Hello,
How can I import YCSB into Eclipse and start a new project? I want to make some additions for my project.

Closing as nonresponsive. Please reopen if you have more details


Please document case of P4=0 for bugcheck C1

[Enter feedback here]
People report cases of  bugcheck C1 with P4=0, but this is not documented. If possible, please add the case of P4=0.
More general, please provide official documentation on "KernelLiveEvent" codes. Are these same as bugcheck codes?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9052f9b0-2c6c-6b4b-653a-c42dbe98bd17
Version Independent ID: 6e6b5056-3ea0-2428-3052-c000052d79fe
Content: Bug Check 0xC1 SPECIAL_POOL_DETECTED_MEMORY_CORRUPTION - Windows drivers
Content Source: windows-driver-docs-pr/debugger/bug-check-0xc1--special-pool-detected-memory-corruption.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@pavel-a - Thanks for providing this input. I think in this case the live kernel event is not a blue screen, but rather a live dump.  Perhaps a Bug Check 0x193: VIDEO_DXGKRNL_LIVEDUMP, but I would have to look further. I'm working to better breakout the live dump stop codes and talk more about them as they are a bit different from traditional blue screen bug checks. I don't think that  0xC1 SPECIAL_POOL_DETECTED_MEMORY_CORRUPTION would have a param 4 =0. Thanks for passing on what you are seeing.


Add run on repl.it badge to README

This pull request adds a Run on Repl.it badge to the README. This will allow users to easily run this repository in their browser, without having to set up an environment. You can learn more about Repl.it here.

Hey, If you clean it up just a little bit by putting the badge on its own line, i'll happily merge.

Okay that is fine


ackermann_msgs: 2.0.2-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository ackermann_msgs to 2.0.2-1:

upstream repository: https://github.com/ros-drivers/ackermann_msgs.git
release repository: https://github.com/ros-drivers-gbp/ackermann_msgs-release.git
distro file: rolling/distribution.yaml
bloom version: 0.10.7
previous version for package: null

ackermann_msgs
* missing std msg dependency
* Contributors: Rousseau Vincent

Please add a LICENSE file in the root of the repository for best practices.


codegen: generate stable names for tests functions.

r? @heycam

Thanks! This is an improvement on the current situation, but I think the numbers will still often change depending on what types we (Gecko) happen to use in templates across the codebase.  It would be nice if we could generate names based on the class name and its template arguments, though that might be a bit tricky.  r=me on this, though.

Yeah, that has its own sets of dangers/trickiness, in the sense that it's pretty hard to avoid conflicts (plus it's more inefficient). I see what you mean though. This is an improvement over the current situation I think.
@bors-servo r=heycam

:pushpin: Commit dd80c27 has been approved by heycam

:hourglass: Testing commit dd80c27 with merge 9b78e09...

:sunny: Test successful - status-travis


allow dead code for generated api

Most users would probably only use either from_env or from_vec,
leaving the other method unused.
Put allow(dead_code) on both methods so neither would ever generate a
warning.

Souldn't this be allow_unused? Please rebase onto master and re-run the tests
bors d+

hmmm... I'm getting
warning: associated function is never used: `from_vec`
  --> src/flags.rs:36:12
   |
36 |     pub fn from_vec(args: Vec<std::ffi::OsString>) -> xflags::Result<Self> {
   |            ^^^^^^^^
   |
   = note: `#[warn(dead_code)]` on by default

and I'm not sure which unused lint would be appropriate from the list . Sorry if I'm missing something obvious.

Yeah, I don’t remember myself which is which
bors r+

Looked a little more closely, and seems like #[allow(unused)] will work because it's the name of the lint group. Let me know if you'd prefer that.

Yeah, I think dead-code is better here. The tests are failing though, this perhaps needs a rebase


ECDSA Plain support

I've added support for the Plain ECDSA signature algorithms. The main difference for this is that the signatures for this are not DER encoded.
It's not a big difference, but wasn't supported yet and therefore I lost a lot of time to research for this.
Therefore, I've decided to share my work to improve the library :)
I tested it with verifying signatures. Also, because of the default value in the constructor parameters, existing code using the DsaDigestSigner will not break.
If there are any improvements I can make (or maybe did something wrong), I'm open to all feedback :)

We've finally added plain ECDSA support, along with the BSI OIDs. Code was based on porting from our Java build rather than merge this PR. Actually there are some issues with the PR's PlainEncode and PlainDecode methods, so I would advise switching to the updated BC once 1.8.4 is out with our changes.
Thanks for the suggestion and the sample code. Please let us know if you find any problems.


vercel integration bug: "code" is not allowed to be empty

I'm not sure if this is the right place to open the issue, but hopefully it can be forwarded to the right people if not.
I get the following error when I try to set up the checkly integration on vercel: "code" is not allowed to be empty

@Snailedlt thanks for reporting! This is not a bug in our CLI, but certainly is in our Vercel integration wizard. We are looking into it!

I figured, but couldn't find any repo for the integration (is it closed source?).
Anyways, let me know if you need more info

I figured, but couldn't find any repo for the integration (is it closed source?). Anyways, let me know if you need more info

@Snailedlt the integration wizard is indeed closed source, as it is part of our webapp. But we found the bug and shipped a fixed. I just tested it on production and I can integrate Vercel with Checkly again. Let me know if this works for you.

It works for me too now. Thank you for the quick fix!
Also, please consider making a repository for issue tracking your closed source projects. Would make it easier for us to find where we can report issues :)


Exclude KeePassJava2 when using azure-identity

Fix #22012

Closing this PR. Let's wait for azure-identity team's fix.


overpass parsing error

I'm submitting a bug report
Brief Description
Change cannot be displayed. I can't tell whether it's due to #349 or not, but I guess osmcha should not generate syntactically invalid requests anyway.
Description of the problem
Request:
https://overpass.maptime.in/api/interpreter?data=[out:xml][adiff:%222019-06-03T13:14:59Z,%22,%222019-06-03T13:15:00Z%22];(node(bbox)(changed);way(bbox)(changed);relation(bbox)(changed));out%20meta%20geom(bbox);&bbox=18.9357333,47.3271267,18.9357333,47.3271267

Response:
Error: line 1: parse error: ';' expected - ')' found.

Error: line 1: parse error: Unexpected end of input.

Error: line 1: static error: Element "print" cannot be subelement of element "union".

When does this occur ?
Clicking on a change, before anything is displayed about it.
Please tell us about your environment:
Debian/sid, Firefox nightly.

This was already fixed here: https://github.com/osmlab/changeset-map/pull/179#issuecomment-481691303
For unknown reasons, the fix hasn't made it into production yet.

It seems wishful thinking is needed. Can't help it from here.


Issue 146 enhance statistics UI for each source

Issue: https://github.com/dolbyio-samples/rts-app-react-publisher-viewer/issues/146
Description
Provide each source with the statistics UI popover

Please also fix the CI-CD error before merge.


Game crashing when i joining casual.

Used latest build of Fedoraware (used fedoraloader)
https://files.catbox.moe/7ltc10.fed this config i used
When I log into the casual, my game crashes without any error messages

I'm able to connect to game perfectly using this config, please attach PC Specs, and if you are using any configs (matercomfig/vusaline(or vusalo)/chrismaxframes etc.)
Does this issue happen uninjected & does it occur with other cheats.

thanks for not responding


Build on 00FAD5654C00 is not supported

Checklist

[X] I have read through the manual page (man fzf)
[X] I have searched through the existing issues
[ ] For bug reports, I have checked if the bug is reproducible in the latest version of fzf

Output of fzf --version
0.55
OS

[ ] Linux
[ ] macOS
[ ] Windows
[X] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Some time ago I successfully compiled fzf v.0.40 for AIX.
When I try to build v.0.55 I get the message:
Makefile:74: *** Build on 00FAD5654C00 is not supported, yet... Stop
Is there any hope that fzf can be compiled for AIX?

Related #2845
But you don't have to use the Makefile. Just try go build from the project root.

I'm sorry, I didn't think that the problem could be only in the Makefile.  ;-)
fzfz compiled, issue can be closed.


lxc/list: Reduce number of API calls

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Cool!

Yeah, I can't remember why we did it the way it was, might have been because the old API client wasn't safe to use from multiple goroutines but the new one should be, if not we'll fix it.


[BUG] test_GUNW.py: duplicated test names

There are two tests in test_GUNW.py named test_check_weather_model_availability. The one that comes later in the file is overriding the earlier one and preventing it from being run. They and their related tests seem to be doing the same/similar things.
https://github.com/dbekaert/RAiDER/blob/a826e077ca29dfc70cab87081559853f4d373e69/test/test_GUNW.py#L340-L369
https://github.com/dbekaert/RAiDER/blob/a826e077ca29dfc70cab87081559853f4d373e69/test/test_GUNW.py#L658-L672

This was fixed in #672.


startingDateValue and endingDateValue error

When you use the startingDateValue prop, and reload the page a javascript error is reported:
"TypeError: Right-hand side of 'instanceof' is not an object"

Can you provide us with an working example of the error?

Can you provide us with an working example of the error?

I closed due to inactivity


Code clean up added new functions

Other:

Improved command registering.
Added chat messages string list. - To list plugin commands.
Removed with large first-name placeholders.
Added README.md.

Thanks for the pull request. Most of that code cleanup probably will be removed as my Intellij's code cleanup function will probably change most of those spaces back to empty lines.

Any news on this pull ?

yep

Other:

Fix NullPointerExceptions
Added 1.13 support

Other:

Updated to 1.13.2 version.
Fix version checking problems.


feat: use class-is for type checks

This is an ongoing effort to fix the ipfs/js-ipfs#938 issue.
The PR libp2p/js-libp2p-websockets/pull/72 must be merged, released and the deps updated here.

Just did a big release party for multiformats and libp2p. Updated the deps here. Let's see if it passes CI :)

js-libp2p-webrtc-star and js-libp2p-utp didn't get released, only merged to master, so they haven't been updated in the libp2p dependencies!


More accurate file format detection

Checking that the file extension exists anywhere in the filename will easily lead to choosing the wrong format, especially .gam (game.z8 is categorized as a TADS file.) Added . in front of all extensions and changed the I7 template to check the end of the file as Inform generates always the same extensions.

Oops, I should've thought of that. Is there any reason not to also anchor the tests in formats.js too?

I was thinking you could have a URL that doesn't end in a file extension like example.com/game.z8?version=2 or something similar so you'd need to add at least & and ? to the end. Optimally there would be a setting to override the automatic detection e.g. if you have a URL that doesn't have a file ending (I need to do default_story: [ "data:text/javascript,processBase64Zcode('" + data + "'); // .ulx .js" ] to make it detect Glulx and use the correct loader)

Ahh, yeah that makes sense.
Do you need to use a data URL, or if you could simply pass a typed array to a launcher function would that be okay? I'd have thought data URLs are a last resort option, so happy to work out something else.

Yeah, if I could just pass in a typed array that would work best for me.

I saw in the snippets that you're doing this:
borogove.loader( function( data ) {
  const extension = ( window.location.search.indexOf( "vm=quixe" ) > -1 ) ? "ulx" : "z8";

  window.parchment_options = {
  default_story: [ `data:text/javascript,processBase64Zcode('${data}'); // ${extension} .js` ],
  lib_path: 'https://assets.borogove.app/interpreters/parchment/2/',
  lock_options: 0,
  lock_story: 0
  };
  //			<script src="main.js" type="module"></s cript>

  $.getScript( "https://assets.borogove.app/interpreters/parchment/2/main.js" );
});

Which is quite hacky.
So I'm thinking of the follow changes:

Add a setting like auto_launch: false to parchment_options to not auto launch
Expose a function window.launch_parchment you could call to launch it. I'm wondering what parameters would be best. I'm thinking it would take an object of options to again add on top of the other layers of options, and that I'll add a data option to provide an Uint8array, and either format or vm option to specify the format and specific VM implementation.

Does that sound like it would meet your needs?

That would indeed sound like the best solution, and that's close to how Quixe works (different launchers for different data inputs.)


Upgrade Spark 3.5.1

Description
Merge feature bracnh spark-3.5.1.
Test
sbt integtest/awsIntegration

[info] Run completed in 3 minutes, 13 seconds.
[info] Total number of tests run: 2
[info] Suites: completed 2, aborted 0
[info] Tests: succeeded 2, failed 0, canceled 0, ignored 0, pending 0

Issues Resolved
https://github.com/opensearch-project/opensearch-spark/issues/352
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Leave this PR as is for author to decide whether to squash or not.


Fix BinaryXent got NaN and also formula comment

Fix formula in comment
Fix log(1-prob) with something like log(1.000001-prob), if not that the loss func will got NaN in some rare case. I have test that in my code https://github.com/auxten/edgeRec/blob/main/model/din/model.go#L58

Hey nice fix.


THE BIT.TRIP [PCSE00735]

App summary

App name: THE BIT.TRIP
App serial: PCSE00735
App version: 1.00

Vita3K summary

Version: v0.1.4
Build number: 2424
Commit hash: https://github.com/vita3k/vita3k/commit/775aced727fb51edbac5ea912155b73a1ef464bd
CPU backend:   Dynarmic

Test environment summary

Tested by: Krvavi Abadas
OS: Windows 7
CPU: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz, 4001 Mhz, 4 Core(s), 8 Logical Processor(s)
GPU: NVIDIA GeForce GTX 1080
RAM: 16 GB

Issues
Game crashes immediately on launch, i think the issue is down to the game using multiple .self files. and the included "eboot.bin" appears to be a duplicate of "BitTrip_MENU_VITA.self", which might also be confusing the emulator.
Screenshots
None.
Log
vita3k - Copy.log
Recommended labels

Nothing
Crash

Vita3K v0.1.5 2704-f37e09b0
comes up with the one screen
vita3k.log

Vita3K summary

Version: v0.1.6
Build number: 2858
Commit hash: https://github.com/vita3k/vita3k/commit/ad2e287f
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game can't go ingame
Screenshots:

Log
vita3k.log
Recommended labels

Menu

as of v0.1.8 we can only play core, runner and fate. beat requires the use of tilt to goto menus void menu wont show up at all and same thing for flux requires tilt


The overwatch license is no longer enforcable

Seeing as overwatch 1's servers were shut down, the original overwatch license no longer holds any value.
Whether or not that makes it more valuable for this repo is left as an exercise for the reader.

Fixed by #15


Added Netstandard support to Nancy.ViewEngines.Markdown

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[ ] I have provided test coverage for my change (where applicable)

Description

Use Markdown package which supports netstandard and is actively mantained.

Hi @Sphiecoh, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

I reran the build. Still fails. Could you take a look, @Sphiecoh ?

will rebase this weekend and see
On Thu, Mar 9, 2017 at 12:25 AM, Christian Horsdal<EMAIL_ADDRESS>
wrote:

I reran the build. Still fails. Could you take a look, @Sphiecoh
https://github.com/Sphiecoh ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/pull/2630#issuecomment-285189593, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEPFooDZxfKCJwi5br5E_djgyg_3Ldd4ks5rjyq8gaJpZM4K5ckX
.

Re-ran the tests and made them pass ;) Needs rebasing though

@Sphiecoh I think we both can agree that this re-base was less than successful 😮 Please let us know if you need any help resolving it. This issue https://github.com/NancyFx/Nancy/pull/2697#issuecomment-284184051 has a similar problem and I detailed steps to take

This is what I have done:

git reset
git merge --no-commit master
fixed the conflict
git commit

Now I think am ready to push with -f

@Sphiecoh not exactly how I would've solved it (I would have used the approach I linked to), so it is impossible for me to say what you commits will look like. Git a the swiss army knife of VCS, so your approach might as well be as reliable as mine 😄 push it and see what it looks like

Any luck?

I would say it was success 👍

LGTM 👍


Vendoring libnetwork to remove stale xfrm states

When using encrypted vxlan network, some of the xfrm states were left stale and this fix will clean them up.
This fix also filters out self advertise-addr rules.
Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

LGTM
Is this fixing an open issue?

@cpuguy83 no. we observed these stale entries during our testing.

LGTM

Janky is hitting that weird docker-py bug


No error on no-escape patterns in strings

A string containing an "invalid" escape is ok since it isn't an escape sequence, for example:
"\x1" or "\a\b" are ok.
For reference:
https://docs.python.org/3/reference/lexical_analysis.html#string-and-bytes-literals
"...Unlike Standard C, all unrecognized escape sequences are left in the string unchanged, i.e., the backslash is left in the result..."
And I fixed an issue with octal escapement: \123 is the correct way to have an octal (and not "\o123")

@maxbrunsfeld could you review the 4 PRs here ?

Ah yes, sorry for neglecting your work @calixteman; I guess I wasn't watching this repo. Taking a look.

No problem, you're welcome (I was working on an other project)


Field documentation

This pull request includes help_text for undocumented fields in both the core and elections models.
There are two fields left undocumented because I do not know their purpose. They are Division.redirect and OCDBase.locked_fields. It would be great to get those added as well.

Coverage remained the same at 98.818% when pulling 50748b188d44ef01024e275faf3f286d17ea6bf2 on california-civic-data-coalition:field-documentation into 3da4ab8956c272b4b4ec03b84980785e792d5cb3 on opencivicdata:master.

Coverage remained the same at 98.818% when pulling d9389cc797931ca3d3f4653fe8f2a49537cef215 on california-civic-data-coalition:field-documentation into 3da4ab8956c272b4b4ec03b84980785e792d5cb3 on opencivicdata:master.

Coverage remained the same at 98.818% when pulling ae36c93e4936490d7f5eaff52dc8a0266bdeae4b on california-civic-data-coalition:field-documentation into 3da4ab8956c272b4b4ec03b84980785e792d5cb3 on opencivicdata:master.

@jamesturk, I believe I have resolved the merge conflicts in this pull request. The tests are now passing and I think it's ready for you again.

Coverage decreased (-18.9%) to 79.927% when pulling 6b6469f14b2d3f2718ffedc1278dda80eb7d00cc on california-civic-data-coalition:field-documentation into a68a9073783873a198823e884fd33e008c6bd1b2 on opencivicdata:master.


AnimatedTexture: ext_resource, invalid UID

Godot version
v4.0.beta16.official [518b9e580]
System information
Windows 10
Issue description
I am using an AnimatedTexture to create a spinner icon, it loads 8 spinner textures and combines them into one animation.
When I close the project and delete the *.import files, then open them again, the *import files are created but the UID is invalid and I get the following warnings.
W 0:00:00:0803   Test.gd:5 @ _ready(): res://assets/spinner.tres:3 - ext_resource, invalid UID: uid://crhotvwkqevtn - using text path instead: res://assets/spinner/Progress7.svg
  <C++ Source>   scene/resources/resource_format_text.cpp:448 @ load()
  <Stacktrace>   Test.gd:5 @ _ready()

I used this texture in my plugin and typically the images are imported when the project is opened or a plugin is installed.
But it does not work for AnimatedTexture.
Steps to reproduce
Open the attached project and run the test scene.
You will receive 8 warnings.
Minimal reproduction project
AnimatedTexture.zip

Godot 4.0 RC1
scene/resources/resource_format_text.cpp:448 Also spams when using version controls systems.
If a colleague does some changes to scenes his uids are put into scene files.

*.import files are not supposed to be deleted.

*.import files are not supposed to be deleted. It makes them re-assign new UID and to fix that you need to save the dependent scenes again.

but it makes no sense to push *.import files to github, so this is a problem when you pull from github.
E.g. AssetLib is pulling fom GitHib.

Yes, *.import files (like character.png.import) should be pushed to your Git repository. Those files contain information on how to import the file. If you delete them, it will use default values, which may no be what you want.

sounds like a regression, this was not need on Godot3

sounds like a regression, this was not need on Godot3

This was the same on Godot 3. You're confusing the *.import files and the res://.import/ folder (now res://.godot/ in Godot 4). The folder, you shouldn't commit, but the *.import files, you should. It's been like this since Godot 3.0.

hm but why i don't need them for all the other images i have?
https://github.com/MikeSchulze/gdUnit4/tree/master/addons/gdUnit4/src/ui/assets
For all this images i get no warnings, this occures only for the AnimatedTexture

@akien-mga , @KoBeWi i fixed my issue now by update the uid's and push the *.image files to github.
But it still open the question:
Why this only happens on AnimatedTexture?
All other images had no issues.

It might have something to do with the fact that AnimatedTexture is not a "real" texture (i.e. it's not imported etc.).

I can confirm opening addons/gdUnit4/src/ui/assets/spinner.tres explicit then save makes the error go away.
I hope this downstream PR fixes it https://github.com/MikeSchulze/gdUnit4/pull/193 ... then we can close this.

I can confirm opening addons/gdUnit4/src/ui/assets/spinner.tres explicit then save makes the error go away.
I hope this downstream PR fixes it MikeSchulze/gdUnit4#193 ... then we can close this.

My problem is already solved as described above. See comment from March 4
The load or preload has no affect to this scenario, the root cause is the way how Godot4 now loads resources.
On Godot 3 there was no need to store the  *.image files to github.

So what exactly is the problem here? Texture UUID is stored inside the *.import files, which don't exist in your MRP. Once you import the project and re-save the texture, the warnings will be gone and will not re-appear when you delete .godot folder. Everything functions as intended.
Also not sure what you mean by "AnimatedTexture" being special. It's not even related to the animated texture. You will get the same warning when you add a sprite to your scene with icon.svg and then delete icon.svg.import.

@KoBeWi
The main problem "was" that a plugin spams this kind of errors when no *.import files are included.
This was not necessary in Godot3 is only mandatory with Godot4.
At the time of bug creation I could see a different behavior of AnimatedTexture and standard textures when loading (no *.import files included). The error was only spam for AnimatedTexture.
Since I deliver the *.import files with the plugin there are no more error messages.
Be that as it may, I find it a pity that you are now forced to deliver the *.import files with the plugin, although they are newly created during the first import. Thus, in the end, the resources are found and re-imported, the displayed error is just very misleading.
For me, this is done so far, the ticket can therefore be closed gladly.

Having a special error for when a file doesn't have an *.import will spam users with errors each time they add a new file
But it doesn't make any sense not to include those files, how would you get import settings?

In Godot 4 the external dependencies are tracked by UID and path, and UID is checked first. Thanks to this you can have a file anywhere in the project and copy it to anywhere in another project and all dependencies will just work if you copy all necessary files. UIDs are stored inside resources. In case of TRES/RES files they are directly inside, in case of PNGs and other imported files, they obviously can't be inside the file, so they are stored inside .import. If you delete the .import file, the UID is re-assigned, so all dependencies using this UID will complain and try to use path.
While deleting these files "worked" in 3.x, it doesn't mean it was correct to do so. If you used any import settings like repeat of filter, they'd be lost.
Closing as it's not a bug nor a regression of any kind.


Started burning at 18V

See title. I'm assuming the module that started burning is the step-down converter. Had 3x4 battery packs with 1,5V batteries connected in series.
Board gets quite hot overall even when running with 12V - probably due to the WiFi module and the small board not being able to dissipate heat.
Just wanted to bring it up, as I think it should be mentioned in the README.md - in case someone plans to build this to carry in e.g. their backpack.

I just saw this issue and I am so sorry this happened to you.  It could have been due to the batteries that you chose when fully charged are actually higher than the nominal voltage.  Lets say that they actually read 1.65v fully charged, well that additional .15v multiplied by your 12 batteries would mean an extra 1.8v input and you may have actually been running the voltage regulator with an input voltage of 19.8 volts and the 18v was an ABSOLUTE MAXIMUM.  My prototype I designed could handle up to 24v but there was no benefit in range that I could notice when running the reader at those higher voltages.  In the future go as low as possible.
Regardless what the issue may have been thank you very much for bringing this to my attention and I have added a safety warning and downgraded the maximum voltage rating to 12v to hopefully prevent this from happening again with notes that adding a heatsink to the voltage regulator would be a good idea and to aim for even lower voltages.  But hackers are going to hack and abuse the specs just like this device actually should technically use level shifters but it works without them so it was created to use as few components as possible in that spirit.

SAFETY: It has come to my attention that the voltage regulator used on the commercially available units that you may find for sale is different than the voltage regulator I originally chose for this project. Also due to global chip shortages and the possibility of manufacturers substituting components I am downgrading the absolute maximum voltage rating to 12v as in you may be stressing the unit to its limits at this voltage and extra care should be taken. Most of my smaller portable prototypes ran at around 9v(x6 AA Batteries) and I found this to be more than adequate, remember that you can always power your reader and the RFID-Tool unit separately if needed and I suggest supplying your project with the lowest possible voltage that you can get away with. In fact, I have found multiple readers that run just fine at less than the recommended voltage. Note that I did not notice a significant range increase in my original testing between supplying 12v and 24v to my reader(I do not recommend this as I tend to abuse the specs for what I am testing, and I was using a different voltage regulator in my prototype). Also be advised that most commonly used batteries produce more than the commonly stated nominal voltage at a full charge. It may also be a good idea to apply a heatsink to your voltage regulator especially if you notice that the unit runs hot at your chosen voltage. It is also never recommended to leave your device unattended. Please be safe and take all necessary safety precautions when testing your setup.

Also, I would be interested in replacing the voltage regulator on my device and seeing if it still works. Can you point me in the direction of the correct component to purchase?
Thanks

I hate to hear that and am sorry that it happened to you. Was it ever attached to higher voltage?
Below is the part for the voltage regulator that was used in the prototype.
Voltage Regulator LM1117 voltage 3.3V out; current 800mA; package sot223; Mouser Part # 926-LM1117MPX3.3NOPB
Also using a low amperage fuse on at minimum the positive Vin line is highly recommended, using a fuse on every line including data shouldn't hurt anything and could help protect connected equipment. I will try to update the schematics soon to include the addition of an inline fuse as well as go through the pull requests.

Thanks for the update.
Re the schematic - I feel like I looked thru the repo here for said document but I could not find it. Am I blind haha?
AFAIK not attached to a higher voltage. At one time it was mistakenly attached to the data lines hooked up to an REX motion sensor. I will make a note to check the voltage on those lines.
Another question for you - I have been searching around for answers and not finding anyone who knows enough. I have been playing with the device on an HID ProxPro and an ICT Vario Prox reader. The HID puts out understandable data. However the same device hooked up to the Vario is spitting out what appears to be random data. Eg one card from the HID reader puts out
26 bit card,18 bit preamble,Binary:000000100000000001<PHONE_NUMBER>0001001001000101,HEX:2005B1245
and this was confirmed with a Proxmark3
However the same card, run 2x with the Vario puts out
30 bit card,14 bit preamble,Binary:00000010000001<PHONE_NUMBER>10001001000100000101,HEX:2050D89105
33 bit card,11 bit preamble,Binary:00000010001<PHONE_NUMBER>01110000100010010000101,HEX:22C0B84485
and I can't seem to make sense of either scan. Same deal with various other cards, puts out understandable data from the HID confirmed with the Proxmark and then gibberish with the Vario. Any direction you can point me in?

Same here. 13,8V input has something done with the voltage regulator. At the moment all pins of the regulator are shorted. I desolder it and check if the ESP is booting when I direct connect 3.3V to the Vin of the ESP. I hope he is still alive. If he is alive I will try to change it with this part: https://www.amazon.de/gp/product/B079Q4CH6N/ref=ox_sc_act_title_1?smid=A19GSHTT314PS3&psc=1 otherwise I have to change and refresh the whole ESP too ... :-/

Same here. 13,8V input has something done with the voltage regulator. At the moment all pins of the regulator are shorted. I desolder it and check if the ESP is booting when I direct connect 3.3V to the Vin of the ESP. I hope he is still alive. If he is alive I will try to change it with this part: https://www.amazon.de/gp/product/B079Q4CH6N/ref=ox_sc_act_title_1?smid=A19GSHTT314PS3&psc=1 otherwise I have to change and reflash the whole ESP too ... :-/

Update: ESP is death too :-/


Fix gemspec warning

WARNING:  open-ended dependency on rack (>= 0) is not recommended
  if rack is semantically versioned, use:
    add_runtime_dependency 'rack', '~> 0'
WARNING:  open-ended dependency on rake (>= 0, development) is not recommended
  if rake is semantically versioned, use:
    add_development_dependency 'rake', '~> 0'
WARNING:  open-ended dependency on contest (>= 0, development) is not recommended
  if contest is semantically versioned, use:
    add_development_dependency 'contest', '~> 0'
WARNING:  open-ended dependency on test-unit (>= 0, development) is not recommended
  if test-unit is semantically versioned, use:
    add_development_dependency 'test-unit', '~> 0'
WARNING:  See http://guides.rubygems.org/specification-reference/ for help
```

to see the warnings, gem build *.gemspec


2270 filter extracts

Codecov Report

Merging #2806 (3263387) into development (7ecf078) will decrease coverage by 23.12%.
The diff coverage is 100.00%.

@@               Coverage Diff                @@
##           development    #2806       +/-   ##
================================================
- Coverage        87.14%   64.01%   -23.13%
================================================
  Files             1877     1386      -491
  Lines            65956    39019    -26937
================================================
- Hits             57479    24979    -32500
- Misses            8477    14040     +5563

Impacted Files
Coverage Δ

app/controllers/extracts_controller.rb
85.71% <ø> (ø)

app/models/otu.rb
44.84% <ø> (-44.85%)
:arrow_down:

...pp/controllers/tasks/extracts/filter_controller.rb
100.00% <100.00%> (ø)

app/models/extract.rb
50.90% <100.00%> (-5.46%)
:arrow_down:

spec/support/concerns/random_token_methods.rb
2.63% <0.00%> (-97.37%)
:arrow_down:

spec/support/concerns/request/api.rb
11.11% <0.00%> (-88.89%)
:arrow_down:

app/models/identifier/global/issn.rb
14.28% <0.00%> (-85.72%)
:arrow_down:

app/models/identifier/global/isbn.rb
8.33% <0.00%> (-85.42%)
:arrow_down:

app/models/protonym/becomes.rb
16.43% <0.00%> (-79.46%)
:arrow_down:

lib/catalog/description_from_observation_matrix.rb
13.22% <0.00%> (-78.52%)
:arrow_down:

... and 1024 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7ecf078...3263387. Read the comment docs.


Implement storm_control for ethernet and port-channel interfaces

Change Summary
Add storm-control support to ethernet-interfaces.j2 and port-channel-interfaces.j2.
The port-profile data model can look like:
storm_control:
  <broadcast | multicast | all>:
    level: <storm-control-level>
    unit: <percent | pps>

Types of changes

[x] New feature (non-breaking change which adds functionality)

Related Issue(s)
Fixes #154
Component(s) name
eos_cli_config_gen
Proposed changes
Update template to support following data-model:
  <Ethernet_interface_2 >:
    storm_control:
      all:
        level: < Configure maximum storm-control level >
        unit: < percent | pps >
      broadcast:
        level: < Configure maximum storm-control level >
        unit: < percent | pps >
      multicast:
        level: < Configure maximum storm-control level >
        unit: < percent | pps >
      'unknown-unicast':
        level: < Configure maximum storm-control level >
        unit: < percent | pps >

How to test
Update your host-vars to use the data-model described above.
Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have updated molecule CI testing accordingly
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

Ready for review


Ionic v2 - Searchbar - Skips / Deletes characters while typing

Short description of the problem:
While typing into an ion-searchbar it skips or deletes some of the characters i'm trying to type. It even does that on the demo site: http://ionicframework.com/docs/v2/api/components/searchbar/Searchbar/
What behavior are you expecting?
Expected normal typing into input without skipping.
Steps to reproduce:

Just add the following tag to one of the pages

<ion-toolbar>
    <ion-searchbar placeholder="What are you looking for ?" (search)="addToSearchList($event)"
        [(ngModel)]="searchQuery" hideCancelButton="true">
    </ion-searchbar>
</ion-toolbar>

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
None.
Which Ionic Version? 2.x
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Version: 2.0.0-beta.3
Ionic CLI Version: 2.0.0-beta.19
Ionic App Lib Version: 2.0.0-beta.9
ios-deploy version: 1.8.2
ios-sim version: 5.0.3
OS: Mac OS X El Capitan
Node Version: v0.12.7
Xcode version: Xcode 7.3 Build version 7D175

Screenshot: (Attempting to write cars in Google Chrome session in an ionic2 2 page with  a searchbar)

Hi @tonyawad88,
I had the same problem. Resolved it by specifying the [debounce]="500" property on the search bar as follow:
<ion-searchbar light [(ngModel)]="searchQuery" [debounce]="500" placeholder="Search Question" (input)="search($event)" (cancel)="cancel($event)" (clear)="cancel($event)"></ion-searchbar>

Giving the search bar enough time for users to type before it starts

It helps a bit thanks. @AdrianTeh
But the problem still exists where you have to wait for the debounce to occur before you can hit the enter key. I basically type in a keyword in the searchbar and hit enter which will create a chip. I can't hit the enter before the debounce occurs. As in it will ignore my enter key until the debounce occurs.

+1

+1

I have not able been to reproduce the problem. I'm hopeful that some other fix already fixed this, but it's probably that I'm not replicating this issue well. Would this group be able to test out the nightly version and see if it's still there? https://github.com/driftyco/ionic/tree/2.0/scripts#installing-nightly-version Thanks

I can't replicate either... I had the debounce adjusted to 400 to minimize the effect but now taking that off, I can't replicate it. It must have slipped in somehow :) .... I'll close for now and we'll re-open if it occurs again.


feature: allow MarkdownContent component to render HTML

Signed-off-by: djamaile<EMAIL_ADDRESS>Hey, I just made a Pull Request!
To give some context I have started out reading personal README's from the user repo at work. Some write their README with html. So, it would be cool if the MarkdownContent can parse HTML content. Obviously, this might not align with maintainers vision so feel free to close this if so.
:heavy_check_mark: Checklist

[x ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ x] Tests for new functionality and regression tests for bug fixes
[ x] Screenshots attached (for UI changes)
[ x] All your commits have a Signed-off-by line in the message. (more info)

oops seems my prettier changed more files, let me fix that


Dubbo service grouping registered with nacos, the grouping is not valid

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.3
Operating System version: windows7
Java version: 1.8

Steps to reproduce this issue

Register services into two different groups
<dubbo:registry  address="nacos://localhost:8848" group="group-dev-1"/>
<dubbo:registry  address="nacos://localhost:8848" group="group-dev-2"/>
Groups are invalid and services are transferred to each other.

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
I want the same service to be registered in different groupings, and service invocations are isolated, rather than invoking each other.
Actual Result
Different groupings, Dubbo consumption call or the same service, did not achieve service grouping isolation

Even a dubbo service is registered to different group of the same registry, the service is still the identical one.

Even a dubbo service is registered to different group of the same registry, the service is still the identical one. So the client will invoke the same one.

If I use nacos, I want to register the same service to separate calls in different groups. How can I do that?

用zookeeper可以生效，nacos不生效

<dubbo:application name="demo-provider"/>

<!--<dubbo:registry  id="group1" address="nacos://localhost:8848" group="group1"/>
<dubbo:registry  id="group2" address="nacos://localhost:8848" group="group2"/>-->
<dubbo:registry  id="group1" address="zookeeper://<IP_ADDRESS>:2181" group="group1"/>
<dubbo:registry  id="group2" address="zookeeper://<IP_ADDRESS>:2181" group="group2"/>

<dubbo:protocol name="dubbo" port="20887"/>

<bean id="demoService1" class="com.mark.i4970.providor.DemoServiceImpl">
    <property name="id" value="demo1"/>
</bean>
<bean id="demoService2" class="com.mark.i4970.providor.DemoServiceImpl">
    <property name="id" value="demo2"/>
</bean>
<dubbo:service interface="com.mark.i4970.api.DemoService"  registry="group1" ref="demoService1"/>
<dubbo:service interface="com.mark.i4970.api.DemoService" registry="group2" ref="demoService2"/>

``
result: Hello world, response from provider: <IP_ADDRESS>:20887demo1
result: Hello world, response from provider: <IP_ADDRESS>:20887demo2
`

<dubbo:application name="demo-consumer"/>

<dubbo:registry id="group1" address="zookeeper://<IP_ADDRESS>:2181" group="group1"/>
<dubbo:registry  id="group2" address="zookeeper://<IP_ADDRESS>:2181" group="group2"/>

<dubbo:reference id="demoService1" check="false" interface="com.mark.i4970.api.DemoService" registry="group1"/>
<dubbo:reference id="demoService2" check="false" interface="com.mark.i4970.api.DemoService" registry="group2"/>

`

@13567436138 I ran with your configuration, it is OK.


Example on android receives shared item but does not render/display until screen is touched

Running the example app on Android works, but the shared url (from browser) only shows up when I touch the screen. I can see the item being received via console.log in the useEffect listener hook, which means the item is received by the example app. However, it does not appear in a View until I touch the screen on physical device. How do I have it display as soon as the app starts up?
"dependencies": {
"react": "16.13.1",
"react-native": "0.63.2",
"react-native-share-menu": "link:.."
},

More info -  the issue appears when running in debug mode on physical device when attached to my Mac, and goes away when disconnected from my Mac. Seems to be just an issue during development.


After adding new coin admin/withdraw/coins doesnt work

Hi,
/admin/withdraws/coins peage gives me error "We're sorry, but something went wrong."
Bitcoin, bank and other altcoins /admin/withdraws working perfect.

Hello @senisende
Can you please share the logs? Better to run app in dev env to get more output information. I am sure you have probably missed some file.

I solved problem. Actually i made mistake that coins.rb folder.


Error TypeError: Cannot destructure property 'width' of 'image.bitmap' as it is undefined

What are you trying to achieve?
I was trying to resize images with the options that get passed through my API. The options are width and height.
Have you searched for similar questions?

No.

Are you able to provide a minimal, standalone code sample that demonstrates this question?
Example code
// imageProcessor is `sharp` being injected as a dependency
const imageToResize = await imageProcessor(filepath)
      .resize({ width: modifiedWidth, height: modifiedHeight })
      .toFile(outputFile)

modifiedWidth and modifiedHeight are the width and height values that I convert to numbers using the Number() object wrapper.
Are you able to provide a sample image that helps explain the question?

No.

Please can you provide a complete, minimal, standalone code sample that demonstrates this question? I suspect the error is occurring in code not shown here.

Please can you provide a complete, minimal, standalone code sample that demonstrates this question? I suspect the error is occurring in code not shown here.

Okay, you were right, I did a little more digging and found the error. Closing this issue.

Okay, you were right, I did a little more digging and found the error. Closing this issue.


Use without sublevel

Currently I have a leveldb created without using sublevel. Is it possible to use level-search on that?

no, this needs sublevel, but you can add sublevel just for this.
LevelSearch(SubLevel(db), 'search')

however, you are now responsible for making sure that the other stuff you add does not collide with sublevel's keyspace.

Thanks @dominictarr. I thought so too, but for some reason I wasn't able to perform the search. Perhaps I am using level-search incorrectly?
Here's the code
var index = search(sub(db), 'transactions');

db.createReadStream({
	gte: 'transaction!',
	limit: 1
}).on('data', console.log);

//{ key: 'transaction!daily!1418839200000',
//  value:
//  { amount: 1000,
//     date: '2014-12-17T18:00:00.000Z',
//     description: 'T card',
//     merchant: 'T',
//    status: 'POSTED',
//     category: 'default' } }

index.createSearchStream(['status', 'POSTED'])
	.on('data', console.log);
// no output

About collision with sublevel keyspace - the sublevel page mentions that "Sublevel prefixes each subsection", but how does it do that exactly?
Right now, I'm prefixing my db entries with namespace such as transaction!. If there's a db.sublevel('transaction'), would that have the same prefix?

there are several different prefixing strategies, by default it uses this:
https://github.com/dominictarr/level-sublevel/blob/master/codec/index.js
prefixes with \x00 (null) and separates with \xffff so you'll be alright.
did you add level-search before you added the data? because level-search creates indexes on write,
so you have to add it before writing anything.
which is realize is actually pretty awkward, and is why I have a new project https://github.com/flumedb/flumedb that adopts a very different architecture for a modular database, which solves that problem!

@dominictarr awesome. thanks.
No, the data was not added before level-search. In that case, should I re-index level search somehow? That should just be a one-time operation?
I'll take a look at flumedb. Does that build on top of leveldb?

flumedb builds beside leveldb. It's basically built around a log - where the data is stored - and then many views which are generated from the data in the log. Level is pretty good for an index/view, but not great at being a log.

If you just write the data back into the the database, you'll get the indexes generated again.

flumedb sounds great. what kind of value can be stored with flume? Im assuming anything that can be serialized through JSON like level?
what do you mean when you say

If you just write the data back into the the database
Would that not create duplicate entries?

well, it's still very new, but any type of value that can serialize to a buffer.
I meant if you do db.createReadStream().pipe(writeStream(db))

What is writeStream in this case?

say,
https://www.npmjs.com/package/level-write-stream
originally level had a built in write stream but we took it out because there where too many ways to implement it

I see. Thanks for the context. So my understanding is that that would not create duplicate entries, because it's put-ing with the same key and value, but would still trigger the search index? And this only needs to be done once?

yeah, except then I realized that it needs to call put where the sublevel can intercept it, which would duplicate the values, or better, copy everything currently in there to another db that uses sublevel.

So the search index needs to be at the same sublevel as the db in order for it to work?

yeah, basically:
var db = sublevel(level(path))
search(db, 'search')
db.put(...)


Breakpoints and data-srcset no longer working on Chrome

Hi,
I just realized that Blazy features for Breakpoint options and data-srcset is having issues with Google Chrome Windows 7 Version 60.0.3112.113 (Official Build) (64-bit) lately.
This is a screendump of the demos/examples page of Blazy, as you can see the "data-srcset" on  is not working anymore... even if you reload the page with different size of the window.

screendump-issue-data-srcset
Curiously though, the <picture> element is working still:

screendump-picture-data-srcset

Now the implementation on our project is using the breakpoint option:

screendump-breakpoint
HTML

JS
var imgLazy = new Blazy({
      selector: '.rbn--image .lazy.image',
      successClass: 'lazy--loaded',
      breakpoints: [{
        width: 768,
        src: 'data-src-small'
      }],
      offset:0
    });

Our setup was working well last I checked some 3 weeks ago, and then I remember several days later Google Chrome had an update, then later I found that the blazy is not working fine with the breakpoints and data-srcset for img element.
On Firefox data-srcset on img element is still working there.

screendump-firefox-data-srcset
however Firefox also has some issue with the breakpoint options
I hope you could look into this issues as we are a fan of this plugin and have used it a lot on our recent projects.
Regards,
Teej

Hi @teej043, i recently used the bLazy plugin and it's really excellent work.
However i am also having some problems with srcset, though  seems to work fine.
my example code is like:

this means that above 993px the images display in a 3-column layout and therefore the browser should pick the 480px image.
But it never does that in Chrome. Firefox seems to work fine.
Have you finally found any workaround or have you anything to suggest me in order to make it work?

Hi @teej043, do you solved? Blazy is not updated since 2016..


Disallow mutation of the container from a lazy configuration action

The lazy task configuration approach works reliably only when task configuration actions mutate the task only. We can help plugin and build script authors with this by failing when their build logic does this. A simple place to start is with the task container itself.
We should make it an error to change the state of the task container from a lazy task configuration action. For example:
tasks.createLater("task") {
    tasks.createLater("another")  // should fail
    tasks.configureEach { ... }  // should fail
    someTaskProvider.configure { ... }  // should fail
}

The implementation should be general and not task specific so that it disallows the mutation of the owning container from any lazy configuration action. For example, this should be an error:
configurations.matching { ... }.configure {
    configurations.create("${it.name}Implementation")
}

We should also consider deprecating mutation of the container from an eager configuration action.

This needs to work for all containers.

WIP PR: https://github.com/gradle/gradle/pull/6441

Splitting PR https://github.com/gradle/gradle/pull/6441 into https://github.com/gradle/gradle/pull/6576

Splitting PR https://github.com/gradle/gradle/pull/6441 into https://github.com/gradle/gradle/pull/6617
This change contains a fix for getting added external provider with NamedDomainObjectCollection#named.

I'm stuck behind a performance regression due to disallowing the execution. I may have a fix in the pipeline to see if I can move forward with this work. I guess we can give a review of the code so if the fix works, we can just merge.

Disallowing existing and creating NamedDomainObjectProvider#configure PR can be found here: https://github.com/gradle/gradle/pull/6734

PR
Configuration
Mutation
Query

~https://github.com/gradle/gradle/pull/6647~
configureEach
disallow
allow

~https://github.com/gradle/gradle/pull/6734~
existing NamedDomainObjectProvider.configure
disallow
allow

~https://github.com/gradle/gradle/pull/6734~
creating NamedDomainObjectProvider.configure
disallow
allow

register
disallow
allow

named
disallow
allow

all
allow
allow

getByName(String)
allow
allow

create
allow
allow

withType(Class)
allow
allow

withType(Class).named
disallow
allow

withType(Class).all
allow
allow

withType(Class).getByName(String)
allow
allow

~https://github.com/gradle/gradle/pull/6647~
withType(Class).configureEach
disallow
allow

matching(Spec/Closure).named
disallow
allow

matching(Spec/Closure).all
allow
allow

matching(Spec/Closure).getByName(String)
allow
allow

matching(Spec/Closure).configureEach
disallow
allow

We also need to grow the mutation and quering list of method to test with (such as PR https://github.com/gradle/gradle/pull/6768 which adds create and register to the mutating methods). The integration test doesn't need to be as extensif but should be present for at least TaskContainer and Project.container.


Add a copy button to a few fields in the admin-side edit contact format

Copy to: (Father|Mother) Name, Day Phone, Employer, Home Phone

Name, Email - STUDENTS
Day+Home Phones, Employer - STUDENTCOREFIELDS


KARAF-5124 Fix location logging in Karaf console

See https://issues.apache.org/jira/browse/KARAF-5124

@gnodet @cschneider Could you take a look at this pr?

R: @jbonofre

Can you please base the pull  request on  master ? We cherry pick on other branches by ourselves.

I can do it for you if you prefer.

Created master pr https://github.com/apache/karaf/pull/303. Closing this one.

Thanks !


Convert NGeoNames to .Net Core 1.1 ?

Hi,
Please, can you convert your package NGeoNames to .Net Core 1.1

I can, but not now nor anytime soon since I'm very busy at the moment. But feel free to make a PR 😉

.Net Core 2.0 support was added recently; does that help?

NGeoNames is working fine on my ASP.NET Core website


Bump kind to v0.4.0

In e2e tests

bump kind to v0.4.0
add kube 1.15.0 in list of clusters in e2e tests

Codecov Report

Merging #117 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   64.73%   64.65%   -0.09%
==========================================
  Files          90       90
  Lines       10933    10933
==========================================
- Hits         7078     7069       -9
- Misses       3551     3564      +13
+ Partials      304      300       -4

Impacted Files
Coverage Δ

internal/controller/resourceupdater.go
42.59% <0%> (-7.41%)
:arrow_down:

internal/controller/stackreconciler.go
73.4% <0%> (-3.2%)
:arrow_down:

internal/registry/logstreamer.go
75.18% <0%> (+1.45%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41d5cbe...13fd2df. Read the comment docs.

@thaJeztah Yes, here the "official" images built for kind 0.4.0 are used. We can use custom built images with a different version of kube but I think it should be done in another PR.


feat(core): introduce FieldArrayType to simplify managing repeating fields.

What kind of change does this PR introduce? feature
Repeating Section example has been updated, we higly recomand to update your code if you use
RepeatTypeComponent in order to take advantage of that new feature.

This change is

Codecov Report

Merging #749 into master will decrease coverage by 1.58%.
The diff coverage is 48.27%.

@@           Coverage Diff            @@
##           master   #749      +/-   ##
========================================
- Coverage   94.59%    93%   -1.59%
========================================
  Files          17     18       +1
  Lines         758    786      +28
  Branches      224    228       +4
========================================
+ Hits          717    731      +14
- Misses         41     55      +14

Impacted Files
Coverage Δ

src/core/src/utils.ts
96.22% <100%> (+0.14%)
:arrow_up:

src/core/src/core.ts
100% <100%> (ø)
:arrow_up:

src/core/src/templates/field-array.type.ts
34.78% <34.78%> (ø)

src/core/src/components/formly.form.ts
94.36% <0%> (+1.4%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76938a8...beccba6. Read the comment docs.


Create a front page for documentation

Alter the /docs/index.rst such that the front page and nav bar looks more like num.pyro.ai

This has been done by @Pluttodk. So I close the issue.


Using with identityuser type int

hi there
i am trying to use IndividualAccountsAuthentication
which in my case class named AppUser inheriting from identity user
class AppUser : IdentityUser<int>
and i get an error
/*
Error	CS0311	The type 'RivaWeb.Models.AppUser' cannot be used as type parameter 'TCustomIdentityUser' in the generic type or method 'SetupExtensions.IndividualAccountsAuthentication(AuthSetupData)'. There is no implicit reference conversion from 'RivaWeb.Models.AppUser' to 'Microsoft.AspNetCore.Identity.IdentityUser'
*/
is there something i am missing or there is no way i can use an int for the user id with this library.

Hi @Kiwiz777
This library relies on a userId that is a string, so you can't have a IdentityUser<int>. Sorry, but that is limitation of the library


Forms with errors

It would be great to see some advice / examples of forms with errors.
Seems that quite a difficult part of making forms look good and work well is how the user interacts with validation.
For example:

http://alistapart.com/article/inline-validation-in-web-forms
http://www.smashingmagazine.com/tag/forms/

Hi ,you can use http://getbootstrap.com/css/#forms-control-validation

I agree with @lenamtl. This is available in the Bootstrap documentation. Examples are also available in the forms -> general elements section in the AdminLTE preview.
For more advanced usage of form validation with Bootstrap, you can checkout this plugin, which will work out of the box with AdminLTE.

Thanks guys.  That link is interesting.
I might be being picky but I find the Bootstrap approach is difficult to
use without the result looking garish.  Perhaps it's a matter of taste.
On 10 March 2015 at 07:43, Abdullah Almsaeed<EMAIL_ADDRESS>wrote:

I agree with @lenamtl https://github.com/lenamtl. This is available in
the Bootstrap documentation. Examples are also available in the forms ->
general elements
http://almsaeedstudio.com/themes/AdminLTE/pages/forms/general.html
section in the AdminLTE preview.
For more advanced usage of form validation with Bootstrap, you can
checkout this plugin
http://reactiveraven.github.io/jqBootstrapValidation/, which will work
out of the box with AdminLTE.
—
Reply to this email directly or view it on GitHub
https://github.com/almasaeed2010/AdminLTE/issues/283#issuecomment-77938048
.

I see what you mean. I'd use the tooltip as an alternative. Show the error on focus using the tooltip, but I still would change the color of the input to red to indicate an error. Just a thought :)

Yeah, that's about where I end up i think.  Mind you, tooltips don't help
for smart phones and tablets.
Anyway, just a thought that there's opportunity to have a better theme if
it addresses those types of issues.
On 10 March 2015 at 12:40, Abdullah Almsaeed<EMAIL_ADDRESS>wrote:

I see what you mean. I'd use the tooltip as an alternative. Show the error
on focus using the tooltip, but I still would change the color of the input
to red to indicate an error. Just a thought :)
—
Reply to this email directly or view it on GitHub
https://github.com/almasaeed2010/AdminLTE/issues/283#issuecomment-77980906
.

Hi,
For form validation this one seems great
http://formvalidation.io

Is there a way to use jQuery's validate utility to show/hide "Input with error" and add/remove the "has-error" class when doing form validation?  Or, is there code already in the template that can show me how?  (Thank you for such a professional template, btw!)

Well it seems there is no jquery validation able to show different input colors when errors happen. This is really disappointing .. I hoped on that part when firstly downloading this theme which is more than great .. till now ..

$.validator.setDefaults({
  highlight: function(element) {
    $(element).closest('.form-group').addClass('has-error');
    $(element).closest('.form-group').removeClass(
      'has-success has-feedback').addClass('has-error has-feedback');
    $(element).closest('.form-group').find('i.fa').remove();
    $(element).closest('.form-group').append(
      '');

  },
  unhighlight: function(element) {
    $(element).closest('.form-group').removeClass('has-error');
    $(element).closest('.form-group').addClass('has-success');
    $(element).closest('.form-group').find('i.fa').remove();
    $(element).closest('.control-label').append(
      '');
  },
  errorElement: 'span',
  errorClass: 'help-block',
  errorPlacement: function(error, element) {
    if (element.parent('.input-group').length) {
      error.insertAfter(element.parent());
    } else {
      error.insertAfter(element);
    }
  }
});

will make jquery validate plugin show the errors n stuff

#1840 i would like to put these validation for the form
https://github.com/cooldeeparmar/Bootstrap-Validator.git
please help to configure it correctly, and theme should have the validation


ARROW-3246: [C++][Python][Parquet] Direct writing of DictionaryArray to Parquet columns, automatic decoding to Arrow

There's a lot going of interconnected pieces in this patch, so let me try to unpack:

Refactor TypedColumnWriterImpl::WriteBatch/WriteBatchSpaced to utilize more common code and be more readable
Add TypedEncoder<T>::Put(const arrow::Array&) and implement for BYTE_ARRAY so avoid having to first create ByteArray* as before. This should improve write performance for regular binary data -- I will do some benchmarks to measure by how much
Add TypedStatistics<T>::Update(const arrow::Array&) and implement for BYTE_ARRAY. This is necessary to be able to update the statistics given directly-inserted Arrow data without serialization
Implement PutDictionary and PutIndices methods on DictEncoder. PutDictionary is only implemented for BYTE_ARRAY but can be easily generalized to more types (we should open a follow up JIRA for this)
Implement internal TypedColumnWriterImpl::WriteArrowDictionary that writes dictionary values and indices directly into a DictEncoder. This circumvents the dictionary page size checks so that we will continue to call PutIndices until a new dictionary is encountered or some non-dictionary data is written. Note that in master, dictionary encoding is turned off as soon as a threshold in dictionary size is reached, which is by default 1MB. So if you want to preserve exactly the original dictionary values (e.g. if you are roundtripping DictionaryArray, R factor, or pandas.Categorical), then we have to step around this threshold check in this narrow case.
Add ArrowWriterProperties::store_schema() option which stores the Arrow schema used to create a Parquet file in a special ARROW:schema key in the metadata, so that we can detect that a column was originally DictionaryArray. This option is off by default, but enabled in the Python bindings. We can always make it the default in the future

I think that's most things. One end result of this is that arrow::DictionaryArray types from C++ and pandas.Categorical types coerced from pandas with string dictionary values will be accurately preserved end-to-end. With a little more work (which I think can be done in a follow up PR) we can support the rest of the Parquet physical types.
This was a fairly ugly project and I've doubtlessly left some messes around that we should clean up, but perhaps in follow up patches.
I'll post some benchmarks later to assess improvements in read and write performance. In the case of writing dictionary-encoded strings the boost should be significant.

Looks like I have an ASAN failure and some compiler warnings on Windows. Will investigate tomorrow

OK I think I've fixed the issues that popped up. I know there's a lot to review here, so we may have to leave many of the refinements to follow up work

See before and after benchmarks:
https://gist.github.com/wesm/b9e917b46536676d2ed829bc4458ba3d
Summary, for this synthetic example

Write performance of DictionaryArray improved 4x from 800ms to 200ms
Peak memory use when writing drops from ~700MB to ~50MB because a cast to dense is no longer required
Read performance (comparing direct dictionary read vs. read-as-dense) improved 5x from ~500ms to ~100ms
Writing of regular BinaryArray as BYTE_ARRAY improved by ~10% (see "dense_data" write at the bottom of each benchmark)

The Appveyor failure in the R build seems legitimate, this is annoying:
C:/projects/arrow/cpp/src/parquet/arrow/reader_internal.cc: In function 'arrow::Status parquet::arrow::GetOriginSchema(const std::shared_ptr<const arrow::KeyValueMetadata>&, std::shared_ptr<const arrow::KeyValueMetadata>*, std::shared_ptr<arrow::Schema>*)':
C:/projects/arrow/cpp/src/parquet/arrow/reader_internal.cc:567:55: error: converting to 'const std::unordered_map<std::basic_string<char>, std::basic_string<char> >' from initializer list would use explicit constructor 'std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::unordered_map(std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::size_type, const hasher&, const key_equal&, const allocator_type&) [with _Key = std::basic_string<char>; _Tp = std::basic_string<char>; _Hash = std::hash<std::basic_string<char> >; _Pred = std::equal_to<std::basic_string<char> >; _Alloc = std::allocator<std::pair<const std::basic_string<char>, std::basic_string<char> > >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::size_type = long long unsigned int; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::hasher = std::hash<std::basic_string<char> >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::key_equal = std::equal_to<std::basic_string<char> >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::allocator_type = std::allocator<std::pair<const std::basic_string<char>, std::basic_string<char> > >]'
     auto new_metadata = ::arrow::key_value_metadata({});
                                                       ^
make[2]: *** [src/parquet/CMakeFiles/parquet_static.dir/build.make:90: src/parquet/CMakeFiles/parquet_static.dir/arrow/reader_internal.cc.obj] Error 1
make[2]: *** Waiting for unfinished jobs....
C:/projects/arrow/cpp/src/parquet/arrow/writer.cc: In function 'arrow::Status parquet::arrow::GetSchemaMetadata(const arrow::Schema&, arrow::MemoryPool*, const parquet::ArrowWriterProperties&, std::shared_ptr<const arrow::KeyValueMetadata>*)':
C:/projects/arrow/cpp/src/parquet/arrow/writer.cc:569:44: error: converting to 'const std::unordered_map<std::basic_string<char>, std::basic_string<char> >' from initializer list would use explicit constructor 'std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::unordered_map(std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::size_type, const hasher&, const key_equal&, const allocator_type&) [with _Key = std::basic_string<char>; _Tp = std::basic_string<char>; _Hash = std::hash<std::basic_string<char> >; _Pred = std::equal_to<std::basic_string<char> >; _Alloc = std::allocator<std::pair<const std::basic_string<char>, std::basic_string<char> > >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::size_type = long long unsigned int; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::hasher = std::hash<std::basic_string<char> >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::key_equal = std::equal_to<std::basic_string<char> >; std::unordered_map<_Key, _Tp, _Hash, _Pred, _Alloc>::allocator_type = std::allocator<std::pair<const std::basic_string<char>, std::basic_string<char> > >]'
     result = ::arrow::key_value_metadata({});
                                            ^
make[2]: *** [src/parquet/CMakeFiles/parquet_static.dir/build.make:116: src/parquet/CMakeFiles/parquet_static.dir/arrow/writer.cc.obj] Error 1
make[1]: *** [CMakeFiles/Makefile2:1265: src/parquet/CMakeFiles/parquet_static.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
==> ERROR: A failure occurred in build().
    Aborting...

https://github.com/apache/arrow/pull/5077/files#diff-66bf5c14b0dc83dceaebb3fe0ddbb55cR567
I'd recommend building new_metadata as an unordered_map then calling key_value_metadata to wrap it after it's built.

Thank you for the code review!

Will merge this as soon as the build is passing...

Travis CI: https://travis-ci.org/wesm/arrow/builds/572728109
Appveyor: https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/26746111


CLI push with --opt-tree or --opt-path not working correctly on windows

When working on Windows with the CLI and doing a push with flags --opt-tree or --opt-path,
the optional paths are not created correctly. The optional folders are merged to one folder name.
1.  Using tm-catalog-cli.exe push <source-folder> --opt-tree
source-folder structure:
source-folder\a\b\my-thing-model.json
current tm-catalog folder structure after push:
tm-catalog\author\manufacturer\mpn\ab\v0.0.0-20231205112438-00c2335c9a75.tm.json
(notice the \ab\)
expected tm-catalog folder structure after push:
tm-catalog\author\manufacturer\mpn\a\b\v0.0.0-20231205112438-00c2335c9a75.tm.json
2.  Using tm-catalog-cli.exe push <source-folder> --opt-path a\b
source-folder structure:
source-folder\my-thing-model.json
current tm-catalog folder structure after push:
tm-catalog\author\manufacturer\mpn\ab\v0.0.0-20231205112438-00c2335c9a75.tm.json
(notice the \ab\)
expected tm-catalog folder structure after push:
tm-catalog\author\manufacturer\mpn\a\b\v0.0.0-20231205112438-00c2335c9a75.tm.json
3.  use-description of push does not match the flag names
The current use description of push
Use: push file-or-dirname [--remote=remote-name] [--with-path=optional/path] [--copy-tree]
is not correct according to the flag names.

--with-path  should be --opt-path
--copy-tree should be --opt-tree

@enricoschuetz, please confirm this is fixed


Ensure FeatureHubRepository#find_interceptor returns an instance of InterceptorValue

Hello 👋
After registering an environment interceptor and reading a feature value, I encountered an exception #<NoMethodError: undefined method `cast' for #<FeatureHub::Sdk::EnvironmentInterceptor:0x000000010ac9c088 @enabled=true>  at this line: https://github.com/featurehub-io/featurehub-ruby-sdk/blob/b7f818659d4e8617f45db7eb25124cff63f2ec20/featurehub-sdk/lib/feature_hub/sdk/feature_state.rb#L113 .
The bug is inFeatureHubRepository#find_interceptor as it is supposed to return an instance of InterceptorValue, not ValueInterceptor. I have updated existing spec to test real behaviour.
Gemfile.lock was outdated so I included here too.

and released!

and released!

awesome, thanks @rvowles


_ModalScope has local copies of route state

Now we just use the state from the Route.
Fixes #744

@Hixie

lgtm


Not necessary to logout and log back

File: install/linux/linux-postinstall.md
Instead of advising to logout and log back in, maybe we could just advise to execute:
newgrp docker ?
But I'm not sure, maybe it hasn't the same range as the logout/login technique.

I've tested this on a few versions of Linux, and it's worked on every version. I also see that 7510 was approved but never merged, so I propose adding the following:
You may be able to issue a newgrp docker - command instead of logging out and back in. The - option forces a new environment, and you will be given a new shell (losing any command history etc).
========
Randomly, does anyone have a clue what this comment is referring to "If testing on a virtual machine, it may be necessary to restart the virtual machine for changes to take effect." ??
Ping @sudo-bmitch


JBPM-5114 - Nested Map<String, Map<String, Object>[]> objects can not be de/serialized correctly

https://issues.jboss.org/browse/JBPM-5114
This use case is particular important for migration. (See other PR for more info).
List of PRs:

https://github.com/droolsjbpm/drools/pull/782
https://github.com/droolsjbpm/droolsjbpm-integration/pull/469 (contains test)

+1 to merge


Reuse code

Codecov Report

Merging #303 (74192c2) into master (82fa4b0) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #303      +/-   ##
============================================
+ Coverage     46.46%   46.48%   +0.01%
  Complexity      487      487
============================================
  Files           117      117
  Lines          2615     2614       -1
  Branches        183      184       +1
============================================
  Hits           1215     1215
+ Misses         1322     1321       -1
  Partials         78       78

Impacted Files
Coverage Δ
Complexity Δ

...u/address/logic/commands/ShowStatisticCommand.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

src/main/java/seedu/address/ui/StatisticTable.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7eda3db...74192c2. Read the comment docs.


Specifically name Ubuntu 16.04 in build status table.

This change is

@digitalinfinity review please?

:shipit:


Fail reading font from CDN due to cross-site

I'm getting this error in Chrome on Mac:
Font from origin 'http://cdn.clappr.io' has been blocked from loading by Cross-Origin Resource Sharing policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8881' is therefore not allowed access.
VM439:1 FlasHLS Clappr (version: 0.1, id: o5)
I think you should add the cross-domain headers when serving the fonts
many thanks
-Itay

@ishahak I'm not sure why you're having this issue. The CDN is already configured to return CORS headers correctly. I tried accessing it using curl and the result was this:
curl -I -H 'Origin: http://localhost:8881' 'http://cdn.clappr.io/latest/assets/Player-Regular.ttf'
HTTP/1.1 200 OK
x-amz-id-2: lRFx2lMP1pxoUbtNj3Vej+V+V3AuB17Y/e3M2RykOa6b79gFJMIPgtZ0X3g8bVGtVkVoBeOm2Vk=
x-amz-request-id: BD092FEA2F9EEC04
Date: Thu, 30 Apr 2015 21:16:18 GMT
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: PUT, GET, POST, DELETE, HEAD
Access-Control-Max-Age: 3000
Vary: Origin, Access-Control-Request-Headers, Access-Control-Request-Method
Last-Modified: Thu, 23 Oct 2014 20:22:09 GMT
ETag: "3e43a5d764f841e7e78896de82cd6c50"
Content-Type: application/x-font-ttf
Content-Length: 5081
Server: AmazonS3

Could you please check whether this command returns the same from your location?

Hi Bruno,
many thanks for your response.
Chrome is reporting this:

Remote Address:
<IP_ADDRESS>:80
2. Request URL:
http://cdn.clappr.io/latest/assets/Player-Regular.ttf
3. Request Method:
GET
4. Status Code:
200 OK (from cache)

Response Headers
Content-Length:
5081
Content-Type:
application/x-font-ttf
Date:
Tue, 28 Apr 2015 06:25:56 GMT
ETag:
"3e43a5d764f841e7e78896de82cd6c50"
Last-Modified:
Thu, 23 Oct 2014 20:22:09 GMT
Server:
AmazonS3
Via:
1.1 Application and Content Networking System Software 5.5.37
x-amz-id-2:
SE50NB7HL9afMpebgf1vRTFH9Y1Q59Ln1+xB7LGWmD0r5XL0+dN1WLiAWSsV+ZuT
x-amz-request-id:
059232200AFFF7E6
Request Headers
Provisional headers are shown
Origin:
http://localhost:8881
Referer:
http://localhost:8881/
User-Agent:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/537.36
(KHTML, like Gecko) Chrome/42.0.2311.90 Safari/537.36

--> so I do not see the headers which you see.
besides,here is my curl response:
>curl -vv http://cdn.clappr.io/latest/assets/Player-Regular.ttf
http://cdn.clappr.io/latest/assets/Player-Regular.ttf

Adding handle: conn: 0x7f9b58804000

Adding handle: send: 0

Adding handle: recv: 0

Curl_addHandleToPipeline: length: 1

Conn 0 (0x7f9b58804000) send_pipe: 1, recv_pipe: 0

About to connect() to cdn.clappr.io port 80 (#0)

Trying <IP_ADDRESS>...

Connected to cdn.clappr.io (<IP_ADDRESS>) port 80 (#0)

GET /latest/assets/Player-Regular.ttf HTTP/1.1

User-Agent: curl/7.30.0

Host: cdn.clappr.io

Accept: /

< HTTP/1.1 200 OK
< x-amz-id-2:
ZKh05nsEjGBJxc1Evspgy8zd+GjLbhc8KOlF2Y4xIFtf57/6Et+po16RyYPDL2qgjrA1OYB3+F0=
< x-amz-request-id: 5A853D952A5FE993
< Date: Fri, 01 May 2015 07:04:32 GMT
< ETag: "3e43a5d764f841e7e78896de82cd6c50"

Server AmazonS3 is not blacklisted

< Server: AmazonS3
< Content-Length: 5081
< Content-Type: application/x-font-ttf
< Last-Modified: Thu, 23 Oct 2014 20:22:09 GMT
< Via: 1.1 ntn01-dmz-wsa-1-mgmt.cisco.com:80 (Cisco-WSA/8.5.0-497), 1.1
Application and Content Networking System Software 5.5.37
< Connection: Close
<
--> so it is possible that the Flash plug-in on Mac is not using the -I
option which I'm getting just like you:
>curl -I -H 'Origin: http://localhost:8881 http://localhost:8881'
'http://cdn.clappr.io/latest/assets/Player-Regular.ttf
http://cdn.clappr.io/latest/assets/Player-Regular.ttf'
HTTP/1.1 200 OK
x-amz-id-2: DXQUkIw5W9+RyVwkY6d6pAedqPYGajvd6gC2Zrj9cAZgoEatxB19Qn78xNqMdmu5
x-amz-request-id: C2BA2A8DAD60ABAE
Date: Fri, 01 May 2015 07:11:47 GMT
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: PUT, GET, POST, DELETE, HEAD
Access-Control-Max-Age: 3000
Vary: Origin, Access-Control-Request-Headers, Access-Control-Request-Method
ETag: "3e43a5d764f841e7e78896de82cd6c50"
Server: AmazonS3
Content-Length: 5081
Content-Type: application/x-font-ttf
Last-Modified: Thu, 23 Oct 2014 20:22:09 GMT
Via: 1.1 ntn01-dmz-wsa-1-mgmt.cisco.com:80 (Cisco-WSA/8.5.0-497), 1.1
Application and Content Networking System Software 5.5.37
Connection: Close
2015-05-01 0:21 GMT+03:00 Bruno Torres<EMAIL_ADDRESS>
@ishahak https://github.com/ishahak I'm not sure why you're having this
issue. The CDN is already configured to return CORS headers correctly. I
tried accessing it using curl and the result was this:
curl -I -H 'Origin: http://localhost:8881' 'http://cdn.clappr.io/latest/assets/Player-Regular.ttf'
HTTP/1.1 200 OK
x-amz-id-2: lRFx2lMP1pxoUbtNj3Vej+V+V3AuB17Y/e3M2RykOa6b79gFJMIPgtZ0X3g8bVGtVkVoBeOm2Vk=
x-amz-request-id: BD092FEA2F9EEC04
Date: Thu, 30 Apr 2015 21:16:18 GMT
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: PUT, GET, POST, DELETE, HEAD
Access-Control-Max-Age: 3000
Vary: Origin, Access-Control-Request-Headers, Access-Control-Request-Method
Last-Modified: Thu, 23 Oct 2014 20:22:09 GMT
ETag: "3e43a5d764f841e7e78896de82cd6c50"
Content-Type: application/x-font-ttf
Content-Length: 5081
Server: AmazonS3
Could you please check whether this command returns the same from your
location?
—
Reply to this email directly or view it on GitHub
https://github.com/clappr/clappr/issues/429#issuecomment-97972328.

@ishahak I noticed you didn't add the Origin header to your curl request. Could you please try it again, using the parameter -H 'Origin: http://localhost:8881'?
I suspect this Cisco ACNS may not set the response headers correcly, even when the Origin header is set. Your browser request/response made me think this way. Maybe testing through a VPN may show whether this is true.

Hi Bruno,
I can confirm that with VPN I do not see this error.
so it means that without "Origin" header, loading of fonts will fail. this
leads to a problem for some corporate networks.
I think you can close this bug.
many thanks!
-Itay
2015-05-02 1:28 GMT+03:00 Bruno Torres<EMAIL_ADDRESS>
@ishahak https://github.com/ishahak I noticed you didn't add the Origin
header to your curl request. Could you please try it again, using the
parameter -H 'Origin: http://localhost:8881'?
I suspect this Cisco ACNS may not set the response headers correcly, even
when the Origin header is set. Your browser request/response made me think
this way. Maybe testing through a VPN may show whether this is true.
—
Reply to this email directly or view it on GitHub
https://github.com/clappr/clappr/issues/429#issuecomment-98255039.

@ishahak Unfortunately this is not easy to solve, but you can work around this issue by hosting the fonts and other assets on the same domain as the page that will load the player. This way, the cross origin policy does not apply and fonts will load correctly.


Added prompt to Sandcastle navigation.

Resolves issue #2178 . Storing html and js in variables whenever any demo loads. If the user tries to navigate:

If the codes are edited, a prompt shows up.
If the codes are unedited, navigation successful.

Thanks @ayk115.
Can you please make this pass JSHint.  It currently fails with:
[runJsHint] Apps/Sandcastle/CesiumSandcastle.js: line 492, col 6, Missing semicolon.

See here.
@emackey do you want to review this?

Done.
How can I run this test locally?

JSHint will run automatically in Eclipse.  You can also run it from the command line (along with the actual Cesium tests) using Ant.  See the Contributor's Guide.

We should also make sure we set the demoJs and demoHtml to the content of the editors when the "Save html file" button is used, this way we don't prompt after a save.

@ayk115 In order to merge this we need you to fill out and email us a Contributor License Agreement. The details for this are in Contributing.md You most likely want the individual CLA

@ayk115 Are you still planning on working on this?  I don't believe we've received your CLA yet and you haven't pushed any new commits in the last 2 weeks.  You can just push new commits directly to this branch to address our review comments.  Please let us know either way.  Thanks.

@mramato I just signed the CLA and emailed it to<EMAIL_ADDRESS> I have a slight doubt regarding the "Save html file" button. When the user clicks the button, a save file dialog appears. The user can choose to save the file or close the dialog. In the latter case, the file isn't actually saved and so, the demoHtml and demoJs variables shouldn't be set. The problem I now face is that how do I track whether the user has actually saved the file or closed the file dialog without saving? I couldn't figure out a way to access the save file dialog.
The easier solution is to set the variables when the user clicks the "Save html file"  button. In that case, we aren't actually checking the download action. Which solution should I implement?

@ayk115 Thanks, we have your CLA.  You make a good point about not being able to detect if save was canceled or not. It might be okay to just always assume they saved the file, since the alternative is to always prompt (which itself could be confusing) @emackey do you have any thoughts on this?
Assuming we just decide to always prompt, the only thing left (I think) is to perform the minor cleanup I mentioned above.

@ayk115 I think you're right, there's no way to tell if they saved it, so the safest thing is to always prompt.

In addition to @mramato's suggested cleanup, @ayk115 can you change the prompt text to this please?  I've tried to re-word it to be acceptable even if they've already saved their edits unknown to us.
Be sure to save a copy of any important edits before leaving this page.
Are you sure you want to navigate away from this demo?

Also we don't get a warning message if the user clicks on a different demo.  This won't trigger the window.onbeforeunload event, because we're not unloading Sandcastle, just loading a different demo.  I think you'll need to use window.confirm to get confirmation before loading the next demo.

@mramato @emackey I did the changes you guys suggested. Please have a look.

Looks great, thanks!

Minor nitpick, there were some tabs instead of spaces in there, I fixed them before merging.  Thanks again!


asynchronous validationValue

Question.
For example: what if you want to check username availability, can you make an asynchronous call to the server to check the availability inside the 'validationValue' element?
The code below doesn't work because Meteor.call is asynchronous. What is best practice here?
ReactiveForms.createElement({
    template: 'UserNameInput',
    validationEvent: 'keyup',
    validationValue: function(el, clean, template) {
        var v = $(el).val();
        console.log(v);
        Meteor.call('isUsernameAvailable', v, function(err, result) {
            if (err) {
                Meteor.Error('could not check username availability.');
            }
            return result;
        });
    }
});

This example might help:
https://github.com/meteortemplates/forms/blob/release/2.0.0/docs/reference/examples/form-elements/advanced/FileUploader.md
Basically you have to return this.stop and then call this.validate(/* value */).


Upgrade to pylexibank 2.x

Done in https://github.com/lexibank/leejaponic/commit/5bffac85c5a812474bdc41b03dfaae81b0948e05