add test script which used for ARM

The test is used to verify if kubevirt can successfully
run on ARM cluster and if it can boot vmi successfully
Signed-off-by: Howard Zhang<EMAIL_ADDRESS>Release note:
NONE

@rmohr @fgimenez

Would it be possible to use our normal automation/test.sh script with the external provider and simply add a very narrow focus on the tests, to e.g. just run 2-3 basic VMs? I would hope that we can start this way and over time unblock more and more e2e tests.

In the current scenario, we import ARM server as an external provider rather than import it as a prow cluster. And bootstrap image does not support ARM. The normal tests (like bazel test) can not run on ARM.
And nested virtualization is not supported on ARM server, so most tests have to be run bare metal.
I think it is just a transition period. Once prow and bootstrap image can run on ARM, we can run the same test as x86 and this test script can be abandoned. If you think it is necessary I can integrate the test in this PR into automation/test.sh.

In the current scenario, we import ARM server as an external provider rather than import it as a prow cluster. And bootstrap image does not support ARM. The normal tests (like bazel test) can not run on ARM.
And nested virtualization is not supported on ARM server, so most tests have to be run bare metal.
I think it is just a transition period. Once prow and bootstrap image can run on ARM, we can run the same test as x86 and this test script can be abandoned. If you think it is necessary I can integrate the test in this PR into automation/test.sh.
Correct me if I have any misunderstanding.

Maybe I don't understand what this script is for. I would expect that our CI can do more or less this:
export BUILD_ARCH=aarch64
export KUBEVIRT_PROVIDER=external
export KUBEVIRT_E2E_FOCUS="\\[multiarch]\\"
automation/test.sh

You would mark one or two tests with [multiarch] and probably have to select the right containerdisk image upfront in the tests based on a target architecture (maybe a commandline flag on the test binary).

In the current scenario, we import ARM server as an external provider rather than import it as a prow cluster. And bootstrap image does not support ARM. The normal tests (like bazel test) can not run on ARM.
And nested virtualization is not supported on ARM server, so most tests have to be run bare metal.
I think it is just a transition period. Once prow and bootstrap image can run on ARM, we can run the same test as x86 and this test script can be abandoned. If you think it is necessary I can integrate the test in this PR into automation/test.sh.
Correct me if I have any misunderstanding.

Maybe I don't understand what this script is for. I would expect that our CI can do more or less this:
export BUILD_ARCH=aarch64
export KUBEVIRT_PROVIDER=external
export KUBEVIRT_E2E_FOCUS="\\[multiarch]\\"
automation/test.sh

You would mark one or two tests with [multiarch] and probably have to select the right containerdisk image upfront in the tests based on a target architecture (maybe a commandline flag on the test binary).

I see. I have not gone deep in E2E test, for now, I think most E2E test needs nested virtualization, but it is good to get a direction. I will try to pick some tests which do not need nested virtualization.

I see. I have not gone deep in E2E test, for now, I think most E2E test needs nested virtualization, but it is good to get a direction. I will try to pick some tests which do not need nested virtualization.

I think there is a fundamental misunderstanding. We may have one test which wants nested virt to see if it is successfully passed through to a VM, but the e2e tests only need a cluster with kubevirt and kvm (does not require nesting).


Stream construction not possible with string

[x] I was not able to find an open or closed issue matching what I'm seeing.
[x] This is not a question. (Questions should be asked on chat (Signup here) or our forums.)

Stream constructor says it accepts a string, but it doesn't.
Providing a string results in an InvalidArgumentException.
In my opinion, this is misleading.
/**
 * @param string|resource $stream
 * @param string $mode Mode with which to open stream
 * @throws Exception\InvalidArgumentException
 */
public function __construct($stream, string $mode = 'r')
{
    $this->setStream($stream, $mode);
}

Suggestion
Simply remove string from @param string|resource $stream, because I have a feeling providing a simple string was never intended.
Feel free to correct me ;)

@steffenbrand
Are your sure? Please look at:
https://github.com/zendframework/zend-diactoros/blob/83743b4558265e01e74b9da9987441628915d67a/src/Stream.php#L331-L341
https://github.com/zendframework/zend-diactoros/blob/83743b4558265e01e74b9da9987441628915d67a/test/StreamTest.php#L52
https://github.com/zendframework/zend-diactoros/blob/83743b4558265e01e74b9da9987441628915d67a/test/StreamTest.php#L62-L65

Ah, sure php://memory is a string as well.
Maybe removing the string is not the right suggestion ;) But I think an additional hint would help that the string must actually be a stream identifier, maybe with a link to http://php.net/manual/de/wrappers.php.php.
To be honest, it actually says must be a string stream identifier or stream resource in the exception, but I didn't get the meaning of string stream identifier at first.
My expectation was, that I could use it like new Stream('my string that this class is going to wrap in some kind of I/O stream by itself.');

@steffenbrand If you are able to upgrade to version 2 (which, unless you are using the emitters or on pre-7.1 versions of PHP, is fully backwards compatible), you can use Zend\Diactoros\StreamFactory::createString() to do what you suggest.
We'll definitely accept a patch that would clarify the constructor argument, however.

Thank you for the answer!
I guess we can update to ZE 3.2 / Diactoros 2 and use the StreamFactory, as you suggested. :v:
If I come up with a clarifying sentence, I'll provide a PR.


Can't download Yowamushi Pedal movies

Describe the bug
This is actually two bugs that may be related. The first one is that trying to download Yowamushi Pedal, which is listed as having 5 seasons in Crunchyroll, only shows two seasons when the CLI tries to download the episodes. This may not be the fault of the CLI, but I wanted to call it out. The next odd behavior is that trying to use the CLI to search for Yowamushi Pedal never returns any results. I have verified that other searches work fine, but crunchy-cli search "yowamushi pedal" and several other variations (including crunchy-cli search https://www.crunchyroll.com/series/GRGGVKP4R/yowamushi-pedal) product no data.
To Reproduce
Steps / command to reproduce the behavior:
$ crunchy-cli archive --ffmpeg-preset h265-lossless -a ja-JP -a en-US -s en-US -o '{series_name} - S{season_number}E{episode_number} - {title} [WEBRip-1080p].mkv' --skip-specials --skip-existing --include-fonts https://www.crunchyroll.com/series/GRGGVKP4R/yowamushi-pedal

$ crunchy-cli search "yowamushi pedal"

$ crunchy-cli search "yowamushi"

$ crunchy-cli search https://www.crunchyroll.com/series/GRGGVKP4R/yowamushi-pedal

Expected behavior
The archive command should see seasons 3-5 for download. The search command should find anything
Screenshots

Client (please complete the following information):

OS: Ubuntu
Version 3.2.5

This looks like it was on me, at least for the download. I didn't realize the movie was marked as a special so I was filtering it out with --skip-specials. Seems like different shows have different setups as I was able to download movies for another show while --skip-specials was set. As for the searching, my suspicion is that it doesn't see me as logged in for some reason, but that's fine as I do not intend to use that functionality much, if ever


[Enhancement] fix mv on iceberg issues

Fixes #issue
Include PR about str2date partitioned MV:

#32458
#32462
#32638
#32744

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 108 / 123 (87.80%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/common/PartitionDiffer.java
68
83
81.93%
[69, 70, 95, 96, 97, 148, 183, 184, 185, 186, 188, 189, 190, 191, 206]

:large_blue_circle:
com/starrocks/catalog/MaterializedView.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMvRefreshProcessor.java
21
21
100.00%
[]

:large_blue_circle:
com/starrocks/sql/common/SyncPartitionUtils.java
10
10
100.00%
[]

:large_blue_circle:
com/starrocks/connector/PartitionUtil.java
7
7
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.2


Change shell prompts to read "python3" vs "python"

Since this is a tutorial written largely for novices, it is more user-friendly to have the shell prompts be "python3". The tutorial states that "This tutorial is written for Django |version|, which supports Python 3.8 and later." So it is safe to assume the user is using python3 for shell prompts. Any experienced python developer who is using an alias such as python-is-python3 will know to substitute python for python3, but a novice will not necessarily know to do the inverse.

I agree with that change. Can you check if there's other tutorial step where you can replace python with python3  ?

I have updated all txt files in the intro folder by changing "python" to "python3" in shell prompts.

I've approved this PR, we'll wait for other that can merge.

Just FYI I'm not against this change but if you're working in a venv then python & python3 are symlinks to the same binary (and venvs are the recommentation on the install Django page)

Thanks for the contribution.
python3 doesn't work on Windows, however. It launches the store so you can install it instead. python (or maybe just py) are needed.


security.pam: make pam_unix.so required, not sufficient

Having pam_unix set to "sufficient" means early-succeeding account
management group, as soon as pam_unix.so is succeeding.
This is not sufficient. For example, nixos modules might install nss
modules for user lookup, so pam_unix.so succeeds, and we end the stack
successfully, even though other pam account modules might want to do
more extensive checks.
Other distros seem to set pam_unix.so to 'required', so if there are
other pam modules in that management group, they get a chance to do some
validation too.
For SSSD, @PsyanticY already added a workaround knob in
https://github.com/NixOS/nixpkgs/pull/31969, while stating this should
be the default anyway.
I did some thinking in what could break - after this commit, we require
pam_unix to succeed, means we require getent passwd $username to
return something.
This is the case for all local users due to the passwd nss module, and
also the case for all modules installing their nss module to
nsswitch.conf - true for ldap (if not explicitly disabled) and sssd.
I'm not so sure about krb5, cc @eqyiel for opinions. Is there some nss
module loaded? Should the pam account module be placed before pam_unix?
We don't drop the security.pam.services.<name?>.sssdStrictAccess
option, as it's also used some lines below to tweak error behaviour
inside the pam sssd module itself (by changing it's 'control' field).
This is also required to get admin login for Google OS Login working
(#51566), as their pam_oslogin_admin accounts module takes care of sudo
configuration.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I couldn't test the ldap test for example, as nss-pam-ldapd is currently broken.

@flokli  Not sure of this is the right place to bring this up but when installing sssd nixos also keeps nscd which is basically doing the same job as sssd. Having 2 caching daemons can cause various issues.
Here Redhat opinion on this. So what's your opinion on this and how we can address that.

Rebased on master.
Now that nss-pam-ldapd was fixed in unstable (thanks @Mic92 :heart: ), I gave nixosTests.ldap a go, and it also did went through. So I'd say this is good to merge.

@GrahamcOfBorg build nixosTests.ldap

For Kerberos, it's not fully working. Right now I have to choose between unlocking my gnome-keyring or using Kerberos. There is probably a problem with the following lines:
Here:
https://github.com/NixOS/nixpkgs/blob/f9bb39e2940db4634b5d7702411c6ed27d5a31ba/nixos/modules/security/pam.nix#L390
pam_auth is required and here:
https://github.com/NixOS/nixpkgs/blob/f9bb39e2940db4634b5d7702411c6ed27d5a31ba/nixos/modules/security/pam.nix#L406
it is sufficient, that means - as stated in a comment - nothing runs afterwards, but there follow necessary the pam_krb5 lines
https://github.com/NixOS/nixpkgs/blob/f9bb39e2940db4634b5d7702411c6ed27d5a31ba/nixos/modules/security/pam.nix#L413-L415
When I can disable unixAuth, Kerberos works as expected, but the Gnome Keyring here:
https://github.com/NixOS/nixpkgs/blob/f9bb39e2940db4634b5d7702411c6ed27d5a31ba/nixos/modules/security/pam.nix#L398-L399
will not be unlocked. But when I enable unixAuth, the pam_krb5 cannot be reached.


Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:52338

Since 6.1.0 I'm unable to run gatsby develop:
...
info  gatsby-source-wordpress  Watching for WordPress changes
⠹ Building development bundle
/....../node_modules/yoga-layout-prebuilt/yoga-layout/build/Release/nbind.js:53
        throw ex;
        ^

Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:52338
    at Server.setupListenHandle [as _listen2] (net.js:1316:16)
    at listenInCluster (net.js:1364:12)
    at GetAddrInfoReqWrap.doListen (net.js:1501:7)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:68:8)
Emitted 'error' event on Server instance at:
    at emitErrorNT (net.js:1343:8)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  code: 'EADDRINUSE',
  errno: 'EADDRINUSE',
  syscall: 'listen',
  address: '<IP_ADDRESS>',
  port: 52338
}
npm ERR! code ELIFECYCLE
npm ERR! errno 7
npm ERR<EMAIL_ADDRESS>start: `gatsby develop`
npm ERR! Exit status 7

Yes, there is another server running on <IP_ADDRESS> (Local by Flywheel), but it's using a different port.
sudo lsof -i :52338

returns nothing.

you've probably already tried this but, did you try a gatsby clean in between upgrades?
also, have you tried restarting all running instances of the gatsby server on your machine? you said there is another server running on a different port, what kind of server? a gatsby server? try shutting down all gatsby servers first, then run clean, then try the upgrade

you've probably already tried this but, did you try a gatsby clean in between upgrades?
also, have you tried restarting all running instances of the gatsby server on your machine? you said there is another server running on a different port, what kind of server? a gatsby server? try shutting down all gatsby servers first, then run clean, then try the upgrade

Speaking for myself :
No other running server on <IP_ADDRESS>, and lsof is not retuning anything for the port supposedly in use.
Using v6 of this plugin.
It happens when doing gatsby develop, and it happens more frequently when content writers are updating the Wordpress server.
I can't reproduce systematically the issue, but the port in use is not always the same.
Quitting the terminal, or restarting my computer sometimes helps, but sometimes not. gatsby clean sometimes helps, but sometimes not. But it also takes 10-20 minutes for a fresh build so it is my last resort.

Speaking for myself :
No other running server on <IP_ADDRESS>, and lsof is not retuning anything for the port supposedly in use.
Using v6 of this plugin.
It happens when doing gatsby develop, and it happens more frequently when content writers are updating the Wordpress server.
I can't reproduce systematically the issue, but the port in use is not always the same.
Quitting the terminal, or restarting my computer sometimes helps, but sometimes not. gatsby clean sometimes helps, but sometimes not. But it also takes 10-20 minutes for a fresh build so it is my last resort.

@babgyy what version of gatsby core are you running? is it the latest?

@babgyy what version of gatsby core are you running? is it the latest?

I think this issue is related https://github.com/gatsbyjs/gatsby/issues/25859
I ran into this myself once and couldn't figure out what was causing it. Changing my node version made it stop 😬

I think this issue is related https://github.com/gatsbyjs/gatsby/issues/25859
I ran into this myself once and couldn't figure out what was causing it. Changing my node version made it stop 😬

@acao I can say it happens every time. Yes I tried gatsby clean. I also tried to restart my machine.
Here are more details of my setup:

macOS 11.1 (20C69)
node v12.20.0

you said there is another server running on a different port, what kind of server? a gatsby server? try shutting down all gatsby servers first, then try the upgrade after running gatsby clean

It's the Apache of Local by Flywheel, the program I host the Wordpress with. I cannot disable it, because then the GraphQL API is not accessible by gratsby develop. I can ensure that it worked with 6.0.0. I reverted my version and the build works, so it's something that was added in 6.1.0 or its dependencies.
Here is my downgrade patch of the package.json:

Red entries caused this issue, green entries fixed this problem (downgraded versions).
@TylerBarnes Your linked issue looks suspicious, the major version of node matches with mine. I will try to switch to node v10 later, but whatsoever, it would not explain why the issue appeared on minor version steps, now, when it was reported back in July 2020.

@acao I can say it happens every time. Yes I tried gatsby clean. I also tried to restart my machine.
Here are more details of my setup:

macOS 11.1 (20C69)
node v12.20.0

you said there is another server running on a different port, what kind of server? a gatsby server? try shutting down all gatsby servers first, then try the upgrade after running gatsby clean

It's the Apache of Local by Flywheel, the program I host the Wordpress with. I cannot disable it, because then the GraphQL API is not accessible by gratsby develop. I can ensure that it worked with 6.0.0. I reverted my version and the build works, so it's something that was added in 6.1.0 or its dependencies.
Here is my downgrade patch of the package.json:

Red entries caused this issue, green entries fixed this problem (downgraded versions).
@TylerBarnes Your linked issue looks suspicious, the major version of node matches with mine. I will try to switch to node v10 later, but whatsoever, it would not explain why the issue appeared on minor version steps, now, when it was reported back in July 2020.

Yeah, it's hard to say what's causing it. We will be investigating it further but in the meantime switching your node version should fix it 👍

Yeah, it's hard to say what's causing it. We will be investigating it further but in the meantime switching your node version should fix it 👍

I just want to report that I finally downgraded to node v10, updated all my packages so I'm @latest right now. Everything works fine, eventually. Maybe it should be part of the documentation to better use node v10, or is it, already? @TylerBarnes
Anyway, I'm closing this now, it seems to be related to https://github.com/gatsbyjs/gatsby/issues/25859 as you said. If the base of Gatsby supports v12, we could re-investigate and make a new issue if problems come back.
Thanks for your support!

I just want to report that I finally downgraded to node v10, updated all my packages so I'm @latest right now. Everything works fine, eventually. Maybe it should be part of the documentation to better use node v10, or is it, already? @TylerBarnes
Anyway, I'm closing this now, it seems to be related to https://github.com/gatsbyjs/gatsby/issues/25859 as you said. If the base of Gatsby supports v12, we could re-investigate and make a new issue if problems come back.
Thanks for your support!


Template for Linux VMs.

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

Build broke due to existing state in the repo, and was caught when we enabled "go vet."  The fix has been submitted, and this merge is good.

Part of the changes needed for #94


Cannot access brand account's youtube channel

I'm trying to use oauth flow to get access to youtube channels list. Everything works fine when I'm using my main default youtube channel, but when on consent screen I choose another channel that associated with google brand account I'm getting error in catch block (arguments -> {type: "tokenFailed", idpId: "google", error: "server_error"}), even though I gave access to everything asked.
Also, on https://myaccount.google.com/security it says that I gave access to application.
Here are some code snippets, I've tried different combinations of scope, prompt, access_type, but nothing successful
const googleScope = [
    'email',
    'profile',
    'openid',
    'https://www.googleapis.com/auth/youtube'
  ]

const googleOauthConfig = {
  client_id: GOOGLE_CLIENT_ID,
  fetch_basic_profile: true,
  ux_mode: 'popup',
  response_type: 'permission',
  access_type: 'offline',
  prompt: 'consent'
}

const loginGoogle = scope =>
  global.gapi &&
  global.gapi.auth2
    .getAuthInstance()
    .signIn({ ...googleOauthConfig, scope: scope.join(' ') })
    .then(res => {
      const authResponse = res.getAuthResponse()
      const { access_token } = authResponse

      return access_token
    })
    .catch(() => {

      throw new Error(POPUP_CLOSED_MESSAGE)
    })

In network there is failed request too
https://accounts.google.com/o/oauth2/iframerpc?action=issueToken&response_type=token%20id_token&login_hint=AJDLj6JUa8yxXrhHdWRHIV0S13cANzW1y8OtI22VrYqaEC2i73av3HLVph3WpN9U_HlWSNxCsiYTB2bXOl1-DWQmT0zrdSxGa1JR03gPm3hk1GCjY9-EQsDI5aEH73UEvOOCoqynaOnr&client_id=583558123459-oc3k39nd2i4cismo7gcvnsv12v03q6an.apps.googleusercontent.com&origin=http%3A%2F%2Flocalhost%3A4000&scope=openid%20profile%20email&ss_domain=http%3A%2F%2Flocalhost%3A4000
status - 403
response - { "error" : "invalid_request" }

Is there any way to solve this problem?

I'm also facing the same error. Is there any workaround for this?

Same here... any luck getting this working?

Have you solved this issue?

Following the example at https://stackoverflow.com/a/43990479/988591 I made some minor changes to the code and now it works correctly.


Outdated nuget package

Current nuget version is 0.1.0.  It has a bug related to ITestOutputHelper instantiation.

Please update the nuget package with the latest commit as it fixes TestOutputHandler initialization.

The latest version is 0.4.0.


Web notification icon not updated once you read said notifications

Description
Gitea is really good about updating browser tabs when new notifications arrive.  When a notification comes in, all my browser tabs, open to the gitea instance in question, properly update the notification bell (within a very short period of time). However, once you read the notifications the other tabs do not update even after waiting hours.
This is extremely confusing as one typically has several gitea tabs open at once and some will say 0 notifications while the others are saying 2 or more etc.
Screenshots
Example:

Gitea Version
1.19.0+dev-649-g2093ca517
Can you reproduce the bug on the Gitea demo site?
No
Operating System
Win10
Browser Version
Chrome Version 110.0.5481.105

Looks like this was somehow fixed between 1.19 and 1.21 that we're running now. Would have appreciated at least some communication on this issue though :/


Update private-link-security.md

MicrosoftDocs/azure-docs#121616

@johnmarco
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


deprecate params alias for query

Describe the feature
Context: https://github.com/nuxt/nuxt/issues/25208
Additional information

[X] Would you be willing to help implement this feature?

thanks for the issue. I am thinking of planning it for the next major version only.


Fix grammar in agentchat_stream.ipynb

Why are these changes needed?
For readability.

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report

Merging #357 (35465c9) into main (a2aa911) will decrease coverage by 0.30%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #357      +/-   ##
==========================================
- Coverage   42.33%   42.03%   -0.30%
==========================================
  Files          19       19
  Lines        2393     2393
  Branches      539      539
==========================================
- Hits         1013     1006       -7
- Misses       1302     1308       +6
- Partials       78       79       +1

Flag
Coverage Δ

unittests
42.03% <ø> (-0.21%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes


Aqara H1 Switch appears as house icon ("not supported")

Issue
Two of my Aqara H1 Wireless Remote Switch (Double Rocker) accessories were added as a house icon with the text "not supported" and I can't remove them from my favourites. How do I either change them into a switch or remove them from the favourites screen?
homebridge-hue plugin version: v0.13.19
deCONZ version: 2.12.06 / 19/08/2021
Product: Wireless Remote Switch H1
Apple Home app:

Debug Files
homebridge-hue.json.gz
homebridge.log

Please attach the debug dump file and screenshots of the accessory details.

@ebaauw Sorry, I've added it to the original post.

What are the serial numbers for the accessories?

Model: lumi.remote.b28ac1
Serial numbers:

54EF4410001B09D5
54EF441000068CDF

*They are supposed to be the exact same


Performance loss after vid_restart

This is better seen with low graphics profile, so the spent time is probably on CPU side. Here we can see how I jump from 1000fps to 500fps after a vid_restart:
Fresh start:

After vid_restart:

I'm running game code as native exe and not dll, so I doubt that is caused by dll memory leaking.

Also, what does mean snc ms?

Enabling cg_noPredict (which is not engine stuff, but cgame stuff) restores the framerate.

But cg_noPredict is already off before doing vid_restart. Well, this may be a client bug.

On fresh start, g_synchronousClients is 1.
After vid_restart, g_synchronousClients is still 1, but performance are affected, if I set g_synchronousClients to 0 then 1 again, performances are recovered.

Well, this is very likely a bug on game code side, let's move the issue.


Issue with checkpoint

I am trying to access this site for checkpoint and it kept returning this message,
Is there any other way that I can see the checkpoint?
Thank you in advance

The cite is unreachable. Which checkpoint you cannot download? Please specify it in a new issue. This issue will be closed.


Needs elaboration: "there are certain names that have defined semantic values"

What are those names, and what are the corresponding values?

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

ID: bc9e5fb8-5b5f-cfe0-2727-cbbd5a5f59a3
Version Independent ID: ec8d81cd-5a86-582d-e5a9-aed1207692cd
Content: IKeyStore (dbgmodel.h) - Windows drivers
Content Source: wdk-ddi-src/content/dbgmodel/nn-dbgmodel-ikeystore.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@aluhrs13 - Andy can you help with this request? We make a pretty broad statement about existing key pairs, and it would be good to provide a few examples or otherwise clarify what the constraints are or what the existing key pairs would be.
By convention only, the values in a metadata defined key store are restricted to basic values (intrinsics and property accessors) While a key store can have an arbitrary number (and arbitrary naming) of keys, there are certain names that have defined semantic values.
Does this imply that a developer should not duplicate the existing pairs? If so, can we provide a dx command or code snip to introspect the objects to learn these, or provide a similar process to list them?

@jazzdelightsme - I will add this text to provide additional information about the defined semantic values:
While a key store can have an arbitrary number (and arbitrary naming) of keys, there are certain names that have defined semantic values. For a list of those values, see Debugger Data Model Metadata Interfaces.

@jazzdelightsme - The topic updates are live, so closing this issue. Thanks for the issue and your interest in the DbgModel docs.


Build the Virtual Machine on windows failed

Build the Virtual Machine on Windows failed, message as below:
==> drupalvm: TASK [geerlingguy.mysql : Update apt cache if MySQL is not yet ins
talled.] *****
==> drupalvm: fatal: [localhost]: FAILED! => {"changed": false, "failed": true,
"msg": "Could not fetch updated apt files"}
==> drupalvm:   to retry, use: --limit @/vagrant//provisioning/playbook.retry
==> drupalvm:
==> drupalvm: PLAY RECAP *******************************************************

==> drupalvm: localhost                  : ok=51   changed=0    unreachable=0
failed=1
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

Yes, it is, but it is same when I turned off proxy and run "vagrant up" and "vagrant provision" again.

It seems like the apt repos simply can't be reached; can you log in with vagrant ssh then inside the VM manually run sudo apt-get update and see what it says?

I received the same message on OSX. Home network, no proxy. Here is the end of of the output of hte sudo apt-get update command.
Fetched 1,085 kB in 10s (101 kB/s)
W: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/trusty-updates/main/binary-i386/Packages  Hash Sum mismatch

E: Some index files failed to download. They have been ignored, or old ones used instead.

I had this problem also in OSX, but it disappeared when I changed my country in my VPN.

This seems like a network or upstream apt/distro issue... I'm going to close this particular issue as I can't reproduce it, but please leave any more notes/clues/fixes here for the benefit of others who may run into the issue.


add enough functionality for client example

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ab1528b). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   22.10%
=======================================
  Files           ?        3
  Lines           ?      475
  Branches        ?        0
=======================================
  Hits            ?      105
  Misses          ?      370
  Partials        ?        0

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 ab1528b...789df92. Read the comment docs.


Equalizer failing during modal reveal with hover state

It appears that when an equalized div has an active hover state that affects flow (margin, positioning, etc) and is then covered by a modal reveal, the equalized heights revert.
I'm not sure if there's a more simple way to describe this condition, but this fiddle should explain all.
https://jsfiddle.net/4edzcc3p/
To recreate the error in the fiddle, simply click one of the boxes
EDIT: I should also add that the reason I suspect it relates to hover states is that if you place the reveal command inside a timeout and, after clicking the box, remove the cursor from the clicked box before the modal appears, the error does not manifest.

This problem seems in part due to this https://github.com/zurb/foundation/blob/master/js/foundation/foundation.equalizer.js#L49
Potential workarounds could be to set the parent div to a fixed height, to height: inherit would keep the styles consistent, or to use the before_height_change callback http://foundation.zurb.com/docs/components/equalizer.html#callbacks

Thanks for the report and the example. It's not in our roadmap for equalizer to change it for this use case. If you have a solution though, we'd accept a PR on it to help anyone else that comes across it. We'll close this here. Thanks!


Corrupted utf-8 cell value if TotalCellChars reached

Description
Excel has upper limit of character count which is 32767, but excelize checks value length by bytes instead by characters.
If I put utf-8 value (repeated 16000 times two-bytes symbol "Ы", for example) to a cell, excelize won't show any error, but file will be corrupted. Excel will show error on this file. Mac Numbers doesn't show any error, but empty cell and half-empty file.
I've already created pull request: https://github.com/qax-os/excelize/pull/1517
Steps to reproduce the issue:
long_value := strings.Repeat("Ы", 16000)
f.SetCellValue("Sheet1", "A1", long_value) // No errors here
// Save file and so on

String long_value contains 16000 chars, but it will go into if len(value) > TotalCellChars because...Go works so. Let's fix it.
Output of go version:
go version go1.19.1 darwin/arm64
Excelize version or commit ID:
v2.7.1
Environment details (OS, Microsoft Excel™ version, physical, etc.):
It doesn't rely on an environment.

The worst is that without the fix you can not fix this from outside, because if len(value) > TotalCellChars is true for every utf-8 TotalCellChars/2 len string and next line value[:TotalCellChars] will corrupt the string

Thanks for your issue. I have merged the pull request #1517 to fix this.


Edge: menus no longer show drop shadow

Using web selfhost in Edge I see that menus no longer show drop shadows:

I see other issues in Edge, so closing as upstream.

I see other issues in Edge, so closing as upstream.


[todo-mvp] Can cache be cleared by the system?

The situation: we've started app and everything works fine. Cache has tasks.
Then we  hide an app for some time, lets assume an hour.
If we open it again - we can get NPE, because cache is null.
Looks like the system clears repository classes to free some memory.
Is there is a way to reproduce the situation with clearing repositories with developement tools?
I'm sorry if this question belongs to SO instead.

Okay, we can force this situation if we turn on the "Don't keep activities" setting in Developer options.
Repositories been cleared and we must check cache for not null and reload data if it is.
In my project i have few repositories, and repository A is observers repository B.
And i can't just reload data, i need to resubscribe my repositories after they have been killed.
I believe i can store repository instances in Application class, but i'm not sure it is good idea.


Add a configurable path

[ ] Allow user to configure S3 path (beyond bucket)
[ ] Allow user to use date in S3 path configuration
[ ] Modify Lambda so that images are stored in a sub-folder with their date

Discussed with MK, need to go //--.gif


Ground Truth Data Format

Hi, thank you for the great project.
I would like to adapt other datasets to use with this Code but i am facing some issues in the GT generation.
Is there a timeline on when you plan to publish the preprocessing code or could you elaborate on the ground truth data format? Especially on what time indices represents the "current" frame and if the ego_motion_gt/bbox_tsfm represent the transformation between some previous and the current frame or between two adjacent frames.
Thanks in advance

Hi, would you like to release the pre-process code this month?
Are there differences with FastFlow3D?

@plpalmer hi, thank you for your interest in our project. Please excuse me for this super late reply, I will upload some scripts used for GT data generation now, they are not immediately runnable but contain most of the functions I used to create the dataset. As we mentioned in the paper, "We denote the first frame X 1 the target frame, while the remaining frames { X t | t > 1 } are termed source frames.", and ego_motion_gt/bbox_tsfm represent the transformation between some previous and the current frame.
@RaymondByc  Please check the dataset_toolbox folder.

thanks for your reply. I am checking your preprocessing code, but it seems that the code related to the "lidarrec_oracle" folder which obtains the "est_tsfm" label and "inst_labels" is missing.
I appreciate it if you upload related code.

"lidarrec_oracle was used for other tasks and is not relevant to this project. I failed to mention that my preprocessing code is mostly based on the toolboxes provided by waymo and nuscene, I was mostly extracting the flow vectors from the odometry and tracking information and re-organising the dataset.

close due to in-activity.


Address #412 and #413 with an editorial tweak

closes #412 and #413 i hope

discussed on call, merging

there is not much choice the wallet has which alg to use to sign a presentation...

I think we might be talking past each other, because I think this section describes the verifier telling the wallet WHICH VCs to present, not how to sign the VP!  As in, "do you have any VCs I know how to verify? if so, ask the user which of them to show" :D


Feature/local impl of ip feature

Copies the implementation for a number of nightly APIs which are in the process of being stabilized so that this crate can be built with stable. This addresses issue #47.

@GGist any chance we could get this merged?


Unified logic for tracking graph tracing

Summary
Unified function to wrap all graph "tracing" mechanisms. E.g.

Reportify dump
Graph printing (e.g. passes::print_graph)
Unique op tracing (e.g. passes::extract_unique_op_configuration )

This effort should encapsulate flags/configurations for all graph tracing in a single place. Make sure to focus on simplifying configurations and how/where data is stored.

cc @pilkicTT feel free to add more details :))


EmptyStates: add an api empty state

If the API isn't started the user is prompted to start it.

Fixes #1754


How can I create a Parallel Coordinate graph in the react?

I want to use this library to create something like this in React: https://plotly.com/javascript/parallel-coordinates-plot/ Is this possible with this library?
Can you please suggest some steps to work with reactjs?
Thanks.

Hi @sgratzl can you please help?

you mean the static plot or the interactivity.
I don't know a way to implement the axis slider


Controller specs ✅🧪

Hi! 👋
On this pull request we made the following changes:

Added the rails-controller-testing gem.
Created RSpec unit tests for each one of the four existing controller actions. Every unit test checks if:

The response status was correct.
The correct template was rendered.
The response body includes the correct placeholder text.

By doing so, we followed and completed the project requirements specified here

-Here's an image to showcase that all tests are passing 😁🧪

Hi @rmrmrmr,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because linters are not set up correctly for this repository
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Update NH SAS rules and new api-version

:clock1:

@smithab once you have the updates, change back the title. So we can review.

Hi @smithab, I have a question about this PR. Was the removal of the license header intentional (see https://github.com/Azure/azure-sdk-for-net/pull/2341/files#diff-cab03c2f3e0416c0ed4f0599f55bef7fL1)

@tbombach : this was auto generated code. I didn't make any changes manually.


[Feature Request] Self Update Argument

Can you please provide a self update argument, that checks for updates and then automatically installs the update, if one was found for example, sherlock.py -update.
Thanks

Yes, thats great idea. we will have that feature.

If Sherlock is packaged up and placed on PyPi, then you would be able to update it like any other Python module. Would that not be better?

@hoadlck
Both have positives and negatives, perhaps go for the one that's easiest to maintain and update.

Well, the infrastructure to update via pip is already in place.  Although I could envision a mechanism to update Sherlock itself, it would be pretty non-trivial to handle the dependencies of Sherlock.  Imagine that a new Python module was needed in the future: Sherlock would have to handle downloading that content.
Don't forget that there already is the option of fetch the site information from some place on the Internet (instead of using what was shipped with Sherlock).

@hoadlck

Well, the infrastructure to update via pip is already in place. Although I could envision a mechanism to update Sherlock itself, it would be pretty non-trivial to handle the dependencies of Sherlock. Imagine that a new Python module was needed in the future: Sherlock would have to handle downloading that content.

To my knowledge there's no inbuilt way to update all pip packages but, it's still simple to update all pip packages with a couple of commands.
Either way, i personally don't have a preference between a pip or inbuilt update option, i suppose it should be down to whichever is the most stable and easiest to maintain.

Don't forget that there already is the option of fetch the site information from some place on the Internet (instead of using what was shipped with Sherlock).

I wasn't aware of that, is there any documentation on that.

The documentation is in the command line help.  The --json option lets you specify the JSON file location (be it on your filesystem, or via an http request).
To illustrate, I removed the information about the Zhihu site from the data.json file local to Sherlock, and provided a path to get it from the internet.  The below command fetches the latest JSON file from master on GitHub.
python3 -u sherlock.py --site Zhihu blue --json https://raw.githubusercontent.com/TheYahya/sherlock/master/data.json

Of course, you have to be sure that the JSON file data that you are providing is compatible with the version of Sherlock.  But, so far, everything has been backwards compatible.

O, i missed that, Thanks


Define flags doesn't work for a normal armi submit run

This error is never hit when i run armi locally on our cluster. contact me for the directory for full example.

Mark's result here is surprising, since we have significant testing in ARMI for the defineFlags() feature on UserPlugins.
The tests even build an App from scratch and then load plugins.
Should be interesting.


Encoding Relationships

Hi,
Just wondering how relationship endpoints are meant to be encoded? The encoder interface doesn't seem to have a method for encoding relationships, and I can't find any mention in your wiki about it.
The example instance from the JSON-API spec would be:
GET /articles/1/relationships/author HTTP/1.1
Accept: application/vnd.api+json

HTTP/1.1 200 OK
Content-Type: application/vnd.api+json

{
  "links": {
    "self": "/articles/1/relationships/author",
    "related": "/articles/1/author"
  },
  "data": {
    "type": "people",
    "id": "12"
  }
}

Your links belongs not to the resource but to the document (side question for you what do you think it would belong to if data contains array of resources?).
Anyways you can add such links with a second parameter of encode method. For your case it will be
    $links = [
        'self' => new Link('/articles/1/relationships/author'),
        'related' => new Link('/articles/1/author'),
    ];

However most likely you want

to have those links in resource relationships
have them automatically generated
In this case you need to add them in Schema like here.

If you have further questions please don't hesitate to ask.

The problem is the data member, not the links member. There's no way of passing in an object that has a type and id member, because the encoding process tries to look up a SchemaProvider for the object.
Effectively this endpoint is returning a Relationship Object that is in the root document. I've tried providing a Neomerx\JsonApi\Schema\RelationshipObject as the first parameter for encode, but it doesn't work because the encoding process tries to look up a SchemaProvider for it.
As there's an error and errors encoding method on the EncoderInterface, does there not also need to be a relationship method for encoding Relationship Objects directly to the document?
Also, would it not be possible for this to be done using the SchemaProvider? In the example above, effectively the endpoint is returning the author relationship that can be obtained by using the getRelationships method on the AuthorSchema object.
So effectively what you want to be able to do is something like this on the encoder interface:
public function relationship($resource, $relationship, $meta = null, EncodingParametersInterface $parameters = null)

which in the example would be called as:
$encoder->relationship($article, 'author');

I admit I don't understand the question. Reading your original question a few more times I can't understand where it might come from.
My questions in particular

what is a 'relationship endpoint'?
why it should be encoded separately from object?
what is a real life example of it (maybe this help)?

I think it will be easier to discuss in more interactive way with

This post might be completely irrelevant to your question but what I think while reading you second comment

There's no way of passing in an object that has a type and id member, because the encoding process tries to look up a SchemaProvider for the object.

Encoder don't work with input data directly it uses Schema for that. You can program Schema for reading any properties from objects. You can register Schema for any type including object (obviously no more than 1 Schema) but I don't understand why not using classes for that as they have stronger typing.

Effectively this endpoint is returning a Relationship Object that is in the root document.

I've got same set of questions what is this endpoint, what PHP type it has, why it should be encoded separately and how it fits JSON-API spec.

does there not also need to be a relationship method for encoding Relationship Objects directly to the document?

That's likely the same question applied to similar object Error. I don't understand the original idea about endpoint-relationship and how it fits the spec.

Also, would it not be possible for this to be done using the SchemaProvider?

A real example might help to understand what you want to achieve.

@lindyhopchris You have disappeared so suddenly :disappointed:

@neomerx It's been evening here (York, UK) and I've been out tonight!
Will be able to write more tomorrow if needed, but the endpoint I'm talking about is fully documented in the JSON API spec. See http://jsonapi.org/format/#fetching-relationships

A server MUST support fetching relationship data for every relationship URL provided as a self link as part of a relationship's links object.

A server MUST respond to a successful request to fetch a relationship with a 200 OK response.

The primary data in the response document MUST match the appropriate value for resource linkage, as described above for relationship objects.

That section of the spec has examples in it.

I got it.
You have to add handler to your controller that reads author/comments from data storage and returns encoded result.
class ArticlesController
{
    public function getRelationship($articleId, $relationshipName)
    {
        ...
        $author = ...;

        $links = ???

        $this->getResponse($author, 200, $links);
    }
}

The question is what is the best way to create $links?
$author doesn't contain information about response links and that it actually come from a relation to Article. So Encoder can't add such links to output without additional data.
You know you're in Article controller handler for relationship requests so one option is to create such data manually
$links = [
    'self' => new Link("/articles/$articleId/relationships/$relationshipName"),
];

Another option is to automate Link process generation fully or partly. For such link generation we need

Articles JSON-API type (from ArticleSchema)
$articleId and $relationshipName (Controller parameters)

Thus such generation function might look like
createRelationshipLink(Article::class, $articleId, $relationshipName);

What are your thoughts on that?

I really feel this totally has to be automated, because otherwise I'm having to program a lot of logic into a controller that is actually contained within the relevant schema providers. It's a duplication of code, which means I'll have two places to update stuff.
For this request:
GET /articles/1/relationships/author HTTP/1.1
Accept: application/vnd.api+json

The minimum response is:
HTTP/1.1 200 OK
Content-Type: application/vnd.api+json

{
  "data": {
    "type": "people",
    "id": "12"
  }
}

(I've intentionally left out the links, as they are not compulsory whereas the data is.)
In the controller, I get the article from the data store, and then get the author from that. I now have to construct a resource identifier for the response, for which I need to know in the controller that the type of $author is people. To work that out without hard coding it into my controller, I need to get the schema provider out of your container.
Also, if I'm using a single instance of an Encoder across my whole application, the other thing I'll need to do in the controller is return a response that is encoded consistently with every other endpoint in the API. That information is held in the EncoderOptions that the Encoder is created with. So therefore it definitely makes sense to me to use the encoder to generate the JSON response.
So the result of this, is I need the encoder to generate the entire JSON response, both data and links.
Another thing is the spec says that the response at this endpoint needs to match the relationship object contained within the resource object to which it relates.

The primary data in the response document MUST match the appropriate value for resource linkage, as described above for relationship objects.

So if my article resource object looks like this:
{
  "type": "articles",
  "id": "1",
  "attributes": {
    "title": "Rails is Omakase"
  },
  "relationships": {
    "author": {
      "links": {
        "self": "/articles/1/relationships/author",
        "related": "/articles/1/author"
      },
      "data": { "type": "people", "id": "9" }
    }
  }
}

Then the response to the /articles/1/relationships/author endpoint must match the value in the above JSON at /data/relationships/author. That means that the ArticleSchema class already knows how to generate that, because if you call getRelationships on that schema it's the author key in the returned array.
So all I need to do in the controller is get the article based on the id, then asked the encoder to encode the author relationship for the supplied article. The schema provider for article already knows how to get the author from the article because that's written into the getRelationships method, and it knows what the links are because they are also written into that method.

@neomerx if you need any help developing the solution then let me know.

@lindyhopchris I'm doing a prototype to discuss with you

I've got an idea to split this task into 2 parts

first one is adding a helper for adding links to document
the second is adding support for encoding relationship of a given object (which will be using the first part)

I've added a non-working prototype (feature/issue55 branch) for the first part (needs small changes in Encoder). Please have a look if we are on the same page. Possible usage might look like
$encoder = ...;
$encoder->withRelationshipLink(
    DocumentInterface::KEYWORD_SELF,
    Article::class,
    $articleId,
    $relationshipName
);

Yes, I would appreciate your help with with it. I'm working on CORS support at the moment (which might be useful for you as well).

@lindyhopchris I can do first part quickly however second part looks more complicated and I can start working on it in a few days

@lindyhopchris I've added prototype for second part. It might be not as difficult as I thought in the first place.

@neomerx
Yeah, the encodeRelationship looks fine - it makes sense to have a different method on the EncoderInterface that is specifically for this case, as the encoded data is different from encoding resource objects. It would make sense for the public interface to expose this single method rather than a 2-step process.
The only thing I'd question is the $linkName argument. For me it makes more sense for the first two arguments to be $resourceObject and $relationshipName, as they always have to be provided. The link is optional.
In the draft encodeRelationship you've written, will the internal call to encode know to encode $data as a resource identifier, rather than a resource object?
So for the author relationship on an articles resource object, this would be the correct result:
{
	"data": {
		"type": "people",
		"id": 1
	}
}

but this would not be correct:
{
	"data": {
		"type": "people",
		"id": 1,
		"attributes": {
			"firstName": "Foo",
			"surname": "Bar"
		}
}

PS: Working on it in the next few days is not a problem! Not expecting this to be solved straight away. We're currently prototyping a JSON-API in an application that's not due for immediate release, so I'm not working against an immediate deadline.

Yeah, the encodeRelationship looks fine - it makes sense to have a different method on the EncoderInterface that is specifically for this case, as the encoded data is different from encoding resource objects. It would make sense for the public interface to expose this single method rather than a 2-step process.

I think of replacing encode list of parameters with methods. It will allow combining functionality with methods like encodeRelationship and adding new parameters via chain methods
$encoder = ...;
$encoder->withLinks(...)->withMeta(..)->withSomeBeer(...)->encode($data, $options);

The only thing I'd question is the $linkName argument. For me it makes more sense for the first two arguments to be $resourceObject and $relationshipName, as they always have to be provided. The link is optional.

You are right for this particular method it's not needed. The method can send the right value to one of the chaining methods.

In the draft encodeRelationship you've written, will the internal call to encode know to encode $data as a resource identifier, rather than a resource object?

It will be using the Schema used on $encoder creation. So the result depends on the Schema implementation. At the moment I have no definite opinion on whether it should force encoding as resource identifier and why adding attributes will be an error.
Anyways feel free to post a PR to feature/issue55 branch.

@neomerx

At the moment I have no definite opinion on whether it should force encoding as resource identifier and why adding attributes will be an error.

The JSON API spec says that a relationship endpoint returns the resource linkage, not the resource object.
For our article with an author relationship example, there are two endpoints that can be exposed for the relationship. These are the self and related links that are on the relationship object.
This request (the related link in the relationship):
GET /api/article/1/author

Returns the resource object:
{
    "data": {
        "type": "people",
        "id": 1,
        "attributes": {
            "firstName": "Foo",
            "surname": "Bar"
        }
}

From the spec:

Consider, for example, a request to fetch a to-one related resource link. This request would respond with null when the relationship is empty (such that the link is corresponding to no resources) but with the single related resource's resource object otherwise.
http://jsonapi.org/format/#fetching-resources

Whereas this request (the self link in the relationship):
GET /api/article/1/relationships/author

Returns the resource linkage:
{
    "data": {
        "type": "people",
        "id": 1
    }
}

The primary data in the response document MUST match the appropriate value for resource linkage, as described above for relationship objects.
http://jsonapi.org/format/#fetching-relationships

Not meaning to be a pain about this, but there is definitely a distinction between the two! Your library is a great implementation, adding in this ability to encode the relationship resource linkage will increase its coverage of the spec.

@lindyhopchris Ok thanks for your help with the spec.

encodeRelationship that returns resource identifier should be added
encodeRelationship might add 2 links for self and related

@lindyhopchris I've partly solved the issue. How relationship links could be added you can see in this example.
As you can see you can encode relationships as this
$encoder->withRelationshipRelatedLink($post, $relationshipName)->encodeData($post->{$relationshipName});

Are you OK with this approach or like encodeRelationship($post, relationshipName) more?
The only problem encodeData encodes with attributes and relationships. I can see 2 solutions

Use Schemes that return empty attributes and relationships. This approach is very easy and could be used right now.
Add a special method for encoding resources as data identifiers e.g. encodeAsResourceIdentifiers (I would be happy if you suggested better name :wink: ). It's a good and clean solution from design perspective however it wouldn't be easy to add. It will require adding new parser manager that will instruct parser to skip following resources' relationships, new parser result interpreter which will ignore resources' attributes and possible a couple more helper object. As everything is connected with interfaces It looks possible it's just won't be trivial.

Would be nice to have some feedback from you.

@lindyhopchris I've finished this enhancement (some samples from tests here)
That's the code snippet for encoding relationships
$encoder = ...;
$resource = ...;
$relationshipName = '...';

$encoder
    ->withRelationshipRelatedLink($resource, $relationshipName)
    ->encodeIdentifiers($resource->{$relationshipName});

@neomerx
Looks good. I'll be able to try it out tomorrow at work (sorry, didn't get a chance to do it today). From the sample tests, it looks like it'll work nicely.

@lindyhopchris I would be happy to hear your opinion on paging links generation enhancement #61

@neomerx this all works a treat. Thanks for adding it in. :-)


feat: update defuser

Update defusing definition.

Change version if you need.

Added reasons to all security checks.

https://listauthorschat.slack.com/archives/C010PK19W78/p1659953849463639


Changed terminal colors, suggestions colors

Preview:

Can you provide me with the details of your pr

Terminal background color is darker now (#151515).
Also some menus now have darker background (#151515)


apps/cmp.c: Improve safeguard assertion on consistency of cmp_options[] and cmp_vars[]

This does not change the actual app semantics but just makes sure that an assertion
checking the consistency of the lengths of the cmp_options[] and cmp_vars[] arrays is more accurate/helpful.
If the arrays do not match (which may happen by mistake after adding or removing options) the app may crash.

Travis timeout on shown (successful) install.log is unrelated.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master.


Unable to navigate using RouterExtensions and named Page Router Outlets

The navigation works when I use [nsRouterLink]="['detail']" but it breaks when using this.routerExtensions.navigate(["detail"]);
**Error message:**
`[BfcV-SmFp]: ERROR Error: Uncaught (in promise): Error: Cannot match any routes. URL Segment: 'detail'
[BfcV-SmFp]: at MapSubscriber.Subscriber._error (file:///data/data/org.nativescript.preview/files/app/tns_modules/rxjs/internal/Subscriber.js:90:26) [angular]
[BfcV-SmFp]: at MapSubscriber.Subscriber.error (file:///data/data/org.nativescript.preview/files/app/tns_modules/rxjs/internal/Subscriber.js:70:18) [angular]
[BfcV-SmFp]: at MapSubscriber.Subscriber._error (file:///data/data/org.nativescript.preview/files/app/tns_modules/rxjs/internal...
[BfcV-SmFp]: at CatchSubscriber.error (file:///data/data/org.nativescript.preview/files/app/tns_modules/rxjs/internal/operators/catchError.js:44:31) [angular]
[BfcV-SmFp]: Error: Cannot match any routes. URL Segment: 'detail'
[BfcV-SmFp]: at CatchSubscriber.selector (file:///data/data/org.nativescript.preview/files/app/tns_modules/@angular/router/bundles/router.umd.js:1436:33) [angular]
[BfcV-SmFp]: at ApplyRedirects.noMatchError (file:///data/data/org.nativescript.preview/files/app/tns_modules/@angular/router/bundles/router.umd.js:1455:20) [angular]`

Sample:
https://play.nativescript.org/?template=play-ng&id=7ozclI&v=9

@Jai-Krish the issue seems to be related to this known bug in the Angular router (navigating to lazily loaded modules with named outlets)

@Jai-Krish  you need to use ActivatedRoute to get the associated route to navigate in.
home.component.ts
import { Component, OnInit } from "@angular/core";
import { ActivatedRoute } from "@angular/router";
import { RouterExtensions } from "nativescript-angular/router";

@Component({
    selector: "Home",
    moduleId: module.id,
    templateUrl: "./home.component.html",
    styleUrls: ['./home.component.css']
})
export class HomeComponent {

    constructor(private routerExtensions: RouterExtensions, private activeRoute: ActivatedRoute) {
    }

    goToDetail() {
        this.routerExtensions.navigate(["detail"], { relativeTo: this.activeRoute });
    }
}

Here is the working Playground demo

@NickIliev thank you for the solution, I wasn't able to find this anywhere in the documentation.

This doesn't work for sibling routes.. I have a page which should navigate to a sibling page but I can't do that because it just says that the route doesn't exist, which it does because the TabView can navigate to it.

try ../sibling route

How would write unit test for that function goToDetail() using mocha?
I am getting an error that TypeError: Cannot read property 'navigate' of undefined

Maybe related with https://github.com/NativeScript/nativescript-angular/issues/2072


Removing PDB check for Min available replicas

Fixes: #922

This change is

Coverage remained the same at 69.474% when pulling ffaa8def20e88c06272c3d2aa7f7f0613fe191d6 on johnugeorge:pdbsync into 1270aa410d02c1c05fc2f46da03037e3666b9769 on kubeflow:master.

/retest

/retest

/lgtm

/retest

/retest


Paths of Web document root

Do you have a list of default paths for commons document root?
Example:
/var/www/sites/all
/var/www/html/default/
/home/www/
/var/www/public/
...

Ive just very quickly made something and added them... WIP!

Here are some of mine:
https://github.com/ron190/jsql-injection/blob/master/src/main/java/com/jsql/view/swing/resources/list/file.txt

Thanks @ron190 !
Just added your missing values into here: https://github.com/danielmiessler/SecLists/pull/183/commits/3f79d071ce66d68b037ff2b06cc38be0540e91ef#diff-c619d49fc9ed6839095333b72a1d8937


Plugin install fails because .plugin_version() modifies requested version

Cookbook version
2.6.0
Chef-client version
12.12.15 (from chefdk 0.16.28-1)
Platform Details
Ubuntu 14.04 (trusty) on EC2
Scenario:
The conditional-buildstep plugin depends on matrix-project plugin version 1.0-beta-1.
The jenkins_plugin resource fails to install matrix-project because it incorrectly rewrites the version to 1.0.pre.beta.pre.1.
Steps to Reproduce:
jenkins_plugin "conditional-buildstep" do
  action :install
end

Expected Result:
The conditional-buildstep plugin and its dependencies should be installed.
Actual Result:
The plugin install fails with:
* remote_file[/tmp/kitchen/cache/matrix-project-1.0.pre.beta.pre.1.plugin] action create[2016-08-09T06:15:40+00:00] WARN: remote_file[/tmp/kitchen/cache/matrix-project-1.0.pre.beta.pre.1.plugin] cannot be downloaded from http://updates.jenkins-ci.org/download/plugins/matrix-project/1.0.pre.beta.pre.1/matrix-project.hpi: 404 "Not Found"

Chef::Provider::JenkinsPlugin.plugin_version (https://github.com/chef-cookbooks/jenkins/blob/master/libraries/plugin.rb#L435-L439) uses Gem::Version for plugin version parsing, which rewrites the plugin's version to 1.0.pre.beta.pre.1. This causes the Update Center download to fail.
In previous versions of Ruby, Gem::Version would raise an ArgumentError in this case, so .plugin_version() would fall through and return the unmodified version string:
[jwm@hobbes:s006 ~> /usr/bin/ruby --version
ruby 2.0.0p648 (2015-12-16 revision 53162) [universal.x86_64-darwin15]
[jwm@hobbes:s006 ~> /usr/bin/irb
irb(main):001:0> require 'rubygems/version'
=> false
irb(main):002:0> Gem::Version.new('1.0-beta-1')
ArgumentError: Malformed version number string 1.0-beta-1
        from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/version.rb:191:in `initialize'
        from (irb):2:in `new'
        from (irb):2
        from /usr/bin/irb:12:in `<main>'

but more recent versions rewrite the version instead:
[jwm@hobbes:s004 ~> ruby --version
ruby 2.1.8p440 (2015-12-16 revision 53160) [x86_64-darwin13.0]
[jwm@hobbes:s004 ~> irb
irb(main):001:0> require 'rubygems/version'
=> false
irb(main):002:0> Gem::Version.new('1.0-beta-1')
=> #<Gem::Version "1.0.pre.beta.pre.1">

As a workaround, we're monkeypatching this in our local recipe that defines jenkins_plugin resources:
class Chef::Provider::JenkinsPlugin
  alias old_plugin_version plugin_version
  def plugin_version(version)
    return version if version == "1.0-beta-1"

    old_plugin_version(version)
  end
end

+1

We had the same error here, but seems like it was due to plugin installation order :). Changing this order solved the issue.

What a strange behavior. It seems like the cookbook should use strings for representing plugin versions instead of Gem::Version


Remove similarity matricies, images

These are useful but need further development and are not supported by SI. For now, keep this strictly as an SI wrapper and develop additional postprocessing through SI / with further consolation with researchers.

closed by #85


Split out managed resources into separate classes

Having a class structure for K8s resources would make it more obvious where to put other Puppet-managed resources when expanding the module, and it should also make it easier to deploy only loose resources.

i like this approach, the resources class is far to big atm (for me). 👍


Latest Version not running

Environment: RasPI 2, latest ioBroker, node.js 7.10
All I get once the instance is created and configured:
host.csprint01	2017-05-27 14:27:24.393	error	instance system.adapter.tradfri.0 terminated with code 1 () host.csprint01	2017-05-27 14:27:22.907	info	instance system.adapter.tradfri.0 started with pid 9746 host.csprint01	2017-05-27 14:27:21.566	info	Restart adapter system.adapter.tradfri.0 because enabled
In additional Info is needed please let me know.

Thanks for the report. The issue is known. One of the 3rd party libs seems to need additional setup/configuration/whatever, idk. I had it work on a raspi 2 but I can reproduce the issues.
I'm going to implement COAP and DTLS myself but that will take some time, since I'm currently moving.

The problematic 3rd party libs have been rewritten and the adapter is ready to use again.


Error message (4)

In a previous version of the JsonPreprocessor, this code
"testlist" : ["A", "B", "C", "D"],
"param1"   : ${testlist}[1:3]

returned the list ["B", "C"]
In the meantime it has been decided to block slicing - but nevertheless previously slicing worked with hard coded integers.
In current version this changed to:
"param1" : ['A', 'B', 'C', 'D'][1:3]
This kind of output should be avoided.

Retest successful. Issue can be closed.

integrated in RobotFramework AIO 0.11.0


DF-551 website long filter search

Ticket URL: https://national-archives.atlassian.net/browse/DF-551
About these changes
These changes enables See more for Website search and allows interaction with the long filter.
Currently its the topic field of Website search which shows See more
The long filter methods in CatalogueSearchLongFilterView are moved to BaseLongFilteredSearchView so that Catalogue search for long filter and Website search for long filter (new) uses the same.
Fixes show no results for a refine query for a non matching criteria.
How to check these changes
As mentioned in the ticket
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer.
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

Merging a feature or bug fix?

Use the Squash and merge option to keep the target branch's commit history nice and clean.
Where relevant, include the ticket number in commit title, e.g. DF-XXX: Ticket name / short description.

Merging a release branch?

Use the message format release/<major.minor.patch> when merging.

Deployment guidance

Post to the appropriate Slack channel to check that it is okay to continue.
To start a deployment, push the relevant branch to the platform remote.
Update the appropriate Slack channel when the deployment is complete.
Update the status on the corresponding Jira ticket (where relevant).

@TNA-Allan I'm afraid this is going to have massive amounts of conflict with #729. We need to decide which should be merged first :)

@TNA-Allan I'm afraid this is going to have massive amounts of conflict with #729. We need to decide which should be merged first :) It looks like the only changes to website_search.html and catalogue_search.html here might be superficial only, so I would suggest #729 comes first, and the changes here be reverted. Thoughts?

@ababic  converted to Draft

Hi @ababic , following merges (from https://github.com/nationalarchives/ds-wagtail/pull/729) , submitting for your review. Thanks.


Postgresql Support

You probably address guys coming from ruby-on-rails. Will there be Postgresql support? But nice work.

I don't plan on adding PostgreSQL to the project, but I found that there are a couple changes I should make to make it easier to swap out MySQL for PostgreSQL. I should also add documentation on how to add your own database.
For a quick overview, to add PostgreSQL, you'll need to do the following:

Change the https://github.com/blue-jay/blueprint/blob/master/lib/database/database.go file to connect to PostgreSQL database using a package like this: https://github.com/lib/pq
Update the database migration files to use PostgresSQL queries: https://github.com/blue-jay/blueprint/tree/master/database/migration
Update the models so they reference PostgresSQL instead of SQL
If using Jay, add a migration package for PostgreSQL here: https://github.com/blue-jay/jay/tree/master/lib/migration
If using Jay, update the https://github.com/blue-jay/jay/blob/master/lib/database/database.go file so it matches the database file from the first step above with the new PostgresSQL. This is so there are no dependencies between projects.

Does this make sense? Would you be willing to make a few of these changes and I can add them to a Wiki article on how to switch to PostgresSQL?

I added a wrapper for the pq package. I don't have any migrations for PostgreSQL yet, but it's a start. https://github.com/blue-jay/blueprint/commit/84663fe8cfc8a007e12b4048a2220eea1b1c7e4d


Test failure: AddTestTagHelperComponent_FromController

This test fails occasionally with the following error:
System.IO.IOException : The request was aborted or the pipeline has finished
   at Microsoft.AspNetCore.TestHost.ResponseStream.CheckNotComplete()
   at Microsoft.AspNetCore.TestHost.ResponseStream.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.WebUtilities.HttpResponseStreamWriter.FlushInternalAsync(Boolean flushEncoder)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewExecutor.ExecuteAsync(ViewContext viewContext, String contentType, Nullable`1 statusCode) in /_/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ViewExecutor.cs:line 260
   at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewExecutor.ExecuteAsync(ActionContext actionContext, IView view, ViewDataDictionary viewData, ITempDataDictionary tempData, String contentType, Nullable`1 statusCode) in /_/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ViewExecutor.cs:line 193
   at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewResultExecutor.ExecuteAsync(ActionContext context, ViewResult result) in /_/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ViewResultExecutor.cs:line 170
   at Microsoft.AspNetCore.Mvc.ViewResult.ExecuteResultAsync(ActionContext context) in /_/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewResult.cs:line 68
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeResultAsync(IActionResult result) in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 138
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextResultFilterAsync[TFilter,TFilterAsync]() in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 1090
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResultExecutedContext context) in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 1193
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.ResultNext[TFilter,TFilterAsync](State& next, Scope& scope, Object& state, Boolean& isCompleted) in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 1072
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeResultFilters() in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 851
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextResourceFilter() in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 791
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResourceExecutedContext context) in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 1147
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted) in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 753
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeFilterPipelineAsync() in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 123
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeAsync() in /_/src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs:line 81
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Mvc.FunctionalTests.CultureReplacerMiddleware.Invoke(HttpContext context) in /_/test/Microsoft.AspNetCore.Mvc.FunctionalTests/Infrastructure/CultureReplacerMiddleware.cs:line 45
   at Microsoft.AspNetCore.TestHost.HttpContextBuilder.<>c__DisplayClass10_0.<<SendAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.TestHost.ClientHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.AspNetCore.Mvc.FunctionalTests.TagHelperComponentTagHelperTest.AddTestTagHelperComponent_FromController() in /_/test/Microsoft.AspNetCore.Mvc.FunctionalTests/TagHelperComponentTagHelperTest.cs:line 82
--- End of stack trace from previous location where exception was thrown ---

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on master.
CC @mkArtakMSFT
This issue was made automatically. If there is a problem contact ryanbrandenburg.

Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

AppRelativeUrlsAreResolvedAndEncodedCorrectly failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

There are a ton of other tests with the exact same error. Looks like it was an actual bug that has since been resolved.


Browser-policy

Sanjo
When I add the browser-policy package I immediately run into problem with jasmine throwing error on using "eval". I tried really hard to find the correct browser policy to disable, but meteor API for BrowserPolicy is no longer available.
Do you have any idea?
Thanks

See https://github.com/meteor-velocity/velocity/issues/255


scrollview bounces when textinput focused

Code:
<KeyboardAwareScrollView
    style={{backgroundColor: '#f7f7f7'}}
    enableResetScrollToCoords={false}
    keyboardOpeningTime={300}
>{/* ... */}</KeyboardAwareScrollView>

Needed to add enableResetScrollToCoords, otherwise the ScrollView would have been scrolled to top when Keyboard was dismissed interactively. Also added an higher keyboardOpeningTime, so it scrolls smoother when ScrollView actually needs to be pushed upwards.
As seen in gif above, there is no need for scrollview to be adjusted. How may I prevent this behavior?

Any ETA on a fix for this? Just run into a similar issue which is probably related.
When switching focus from one input to another it bounces first

@RSSfeed  I was able to get around this by removing the React Navigation Header on ios. Does that potentially work for you?

Still having this issue.

I have the same issue.

I was having similar issues with the scrollview bouncing when focusing on a TextInput, and changing
<KeyboardAwareScrollView style={styles.scroll}>
to
<KeyboardAwareScrollView contentContainerStyle={styles.scroll}>
seemed to fix it for me.

A temporary workaround is to remove keyboardOpeningTime.

I'll be working on this issue this week, thanks for reporting.

same issue in 0.7.2.

@alvaromb  I found this issue is caused by react-navigation,  I add extraHeight={-64} then it works.

how to resolve this issue plz help

@KryDos  yes, it is.

Is the negative extraHeight solution the proposed one for this? I can confirm ( as per #340) that this issue still remains

I'm using this library + react-navigation and setting extraHeight={90} with a <TextInput /> at the footer of the screen seemed to work for me (smooth scrolling with no jumps)

The same happens with KeyboardAvoidingView but above solution do not work.

Still broken in 2022. A workaround is to set multiline={true} and blurOnSubmit={true}

As @alvaromb mentioned, the final solution to this is:
keyboardOpeningTime={0}


Remove duplication in FakeInitModule

Goal
Clean up FakeInitModule to reduce duplication between it and the real thing.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

master

PR #312

PR #313

PR #314

PR #315

PR #316

PR #317   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 31, 12:18 PM: @bidetofevil started a stack merge that includes this pull request via Graphite.


add onWheel to VList

Updates ViewportComponentAttributes to expose onWheel in VList.
Enables user to differentiate automated scroll behaviour from mouse scroll.
Solves problem from this discussion.

Let me know if I should include a story (I have tested one locally that shows it working).

Released in 0.33.5.


Improve ReceiveMessage performance

Thanks @colprog !

Is .net 5 a project requirement? I have a series of fixes coming but I haven't tested on .net 5.

Is .net 5 a project requirement? I have a series of fixes coming but I haven't tested on .net 5.

Yes.


Huge performance problem with async

We are trying to scale an application that uses ASP.NET Identity (which is all async) and we have huge performance issues with all async work against SqlClient (we believe it has boiled down to this). We have investigated and tried many different things for a couple of weeks, and we now have very little isolated code that performs more or less equally bad all the time. The code has implementation of both sync and async calls with huge differences between them. We have read about the issues for big fields, but we see the problem on all our queries (without any big fields) and the reproduced slow calls below has no large fields.
We run in Azure with an App Service plan of P1V2 scaled to two instances and an SQL Azure Premium P1: 125 DTUs. We don't see any performance problems at all in SQL Azure. We have a VNET with a private link between the App Service and SQL Azure. We use Azure Load test to run the tests (and we can share the tests too if needed). We also get similar results when testing locally.
We use Stackify Retrace for monitoring and have enabled that through their extension.
Sync code performance when running 10 instances with 250 threads:
90%: 4,69 sek
Average: 1,417 sek
Open Connection: <0,001 sek
Select Logins: 0,002 sek

Async code performance when running 10 instances with 250 threads:
90%: 5,38 sek
Average: 2,235 sek
Open Connection: 1,624 sek
Select Logins: 0,298 sek

Sync code performance when running 45 instances with 250 threads, each thread making 10 calls after each other:
90%: 40,21 sek
Average: 17,502 sek
Open Connection: <0,001 sek
Select Logins: 0,006 sek

Async code performance when running 45 instances with 250 threads, each thread making 10 calls after each other:
90%: 80,97 sek
Average: 35,080 sek
Open Connection: 19,634 sek
Select Logins: 7,975 sek

Table script:
`
CREATE TABLE [security].[AspNetUserLogin](
[LoginProvider] [nvarchar] (100) NOT NULL,
[ProviderKey] [nvarchar] (100) NOT NULL,
[TenantId] [uniqueidentifier] NOT NULL,
[ProviderDisplayName] [nvarchar] (450) NULL,
[UserId] [uniqueidentifier] NOT NULL,
CONSTRAINT [PK_AspNetUserLogin] PRIMARY KEY CLUSTERED
(
[LoginProvider] ASC,
[ProviderKey] ASC,
[TenantId] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON) ON [PRIMARY]
) ON [PRIMARY]
GO
ALTER TABLE [security].[AspNetUserLogin]  WITH CHECK ADD  CONSTRAINT [FK_AspNetUserLogin_AspNetUser_UserId] FOREIGN KEY([UserId])
REFERENCES [security].[AspNetUser] ([Id])
ON DELETE CASCADE
GO
ALTER TABLE [security].[AspNetUserLogin] CHECK CONSTRAINT [FK_AspNetUserLogin_AspNetUser_UserId]
GO
`
Data: The table is empty
ASP.NET Code (.NET 6 and Microsoft.Data.SqlClient 4.1.0 (we have tried with version 5 too). We started with a default .NET 6 template, added support for controllers and removed authorization, https redirection and exception handler.
The controller:
`
using Microsoft.AspNetCore.Identity;
using Microsoft.AspNetCore.Mvc;
using Microsoft.Data.SqlClient;
namespace SyncAsyncPerf.Controllers
{
[Route("api/[controller]")]
[ApiController]
public class LoadTestController : ControllerBase
{
private const string _connectionString =
"Server=tcp:ourserver.database.windows.net;Authentication=Active Directory Managed Identity;Initial Catalog=oursqldb;MultipleActiveResultSets=False;Encrypt=True;TrustServerCertificate=False;Connection Timeout=30;Max Pool Size=5000;Min Pool Size=100";
    [HttpGet]
    public IActionResult Get(Guid userId)
    {
        var sql = $"SELECT [LoginProvider], [ProviderKey], [ProviderDisplayName] FROM [security].[AspNetUserLogin]" +
                    $"WHERE [UserId] = @UserId";

        using var connection = new SqlConnection(_connectionString);
        using var command = connection.CreateCommand();

        command.CommandText = sql;
        var parameter = command.CreateParameter();
        parameter.ParameterName = "@UserId";
        parameter.DbType = System.Data.DbType.Guid;
        parameter.Value = userId;
        command.Parameters.Add(parameter);

        connection.Open();

        using var reader = command.ExecuteReader(System.Data.CommandBehavior.SingleResult | System.Data.CommandBehavior.CloseConnection);

        var result = new List<UserLoginInfo>();
        while (reader.Read())
        {
            result.Add(
                new UserLoginInfo(reader.GetString(0), reader.GetString(1), reader.IsDBNull(2) ? null : reader.GetString(2))
                        );
        }

        return Ok(result);
    }

    [HttpGet("async")]
    public async Task<IActionResult> GetAsync(Guid userId)
    {
        var sql = $"SELECT [LoginProvider], [ProviderKey], [ProviderDisplayName] FROM [security].[AspNetUserLogin]" +
                    $"WHERE [UserId] = @UserId";

        using var connection = new SqlConnection(_connectionString);
        using var command = connection.CreateCommand();

        command.CommandText = sql;
        var parameter = command.CreateParameter();
        parameter.ParameterName = "@UserId";
        parameter.DbType = System.Data.DbType.Guid;
        parameter.Value = userId;
        command.Parameters.Add(parameter);

        await connection.OpenAsync(HttpContext.RequestAborted);

        using var reader = await command.ExecuteReaderAsync(System.Data.CommandBehavior.SingleResult | System.Data.CommandBehavior.CloseConnection, HttpContext.RequestAborted);

        var result = new List<UserLoginInfo>();
        while (await reader.ReadAsync())
        {
            result.Add(
                new UserLoginInfo(reader.GetString(0), reader.GetString(1), reader.IsDBNull(2) ? null : reader.GetString(2))
                        );
        }

        return Ok(result);
    }
}

}
`
Finally, some notes on the connection string. We have tried many different configurations without effecting the underlying problem. If we use defaults for connection pooling the number of errors increases rapidly, especially for async code because of connection timeouts.

Having the same issue here... It's an async issue with the SQL client for large fields - Changing the nvarchar(max) to nvarchar(255) for example helped alot.

Having the same issue here... It's an async issue with the SQL client for large fields - Changing the nvarchar(max) to nvarchar(255) for example helped alot.

Large fields is an identified issue since earlier, but we get slow performance for small fields too.

Having the same issue here... It's an async issue with the SQL client for large fields - Changing the nvarchar(max) to nvarchar(255) for example helped alot.

Large fields is an identified issue since earlier, but we get slow performance for small fields too.

I removed all async db calls but can’t in the identity which is causing me issues.
Seems like sqlclient is just broken for async.

When switching to Linux App Service the problems disappeared. We haven't tested the downscaled example used in this issue, but the full application is scaling very well on Linux.

My company is seeing issues with this as well. I've tried everything I could in modifying the connection string and priming the pool on startup.
{address};Database={db};Trusted_Connection=True;MultipleActiveResultSets=true;Min Pool Size=100;Max Pool Size=300;Pooling=true;

Just a screenshot of my log in QA environment trying to login to the application. The user-experience just hangs because we're getting spikes in opening a sqlconnection when there's no load at all. I'm not sure what else we can do - this is unusable at the moment.

Chiming in with, we are seeing this issue as well and can see it reported within our APM tool. Rolling back to sync call cuts the time in half.

Async performance is completely unacceptable. In one particular instance changing from ToListAsync to ToList reduced execution time from 18 minutes to 7 seconds. That is 100,000x speed up. This happens when we are retrieving 3 rows each one having nvarchar(max) column with 25,000,000 characters of data.


MWPW-146031 [MEP] Existing Personalization Tag UPDATE: Photoshop-any

Update prod and stage id's for photoshop-any

Resolves: MWPW-146031
Test URLs:

Before: https://main--milo--adobecom.hlx.page/?martech=off
After: https://photoshop-any--milo--adobecom.hlx.page/?martech=off

verified?

verified

@JackySun9 , sorry, I don't understand the question.

@JackySun9 , sorry, I don't understand the question.

@vivgoodrich according to our contribution guidelines you shouldn't apply the label yourself, but a dedicated person that tests changes (that isn't the PR owner)

Every PR must be tested. QE should apply the 'Ready for Stage' label only after a PR has received approvals, all checks have passed, and testing is complete.


Trying to compile NClone from Github

When trying to compile NClone in VS 2013, pulled from Github, I keep getting:
Warning	1	The referenced project '..\mijay.Utils\mijay.Utils.csproj' does not exist.	NClone.Tests
Warning	2	The referenced project '..\mijay.Utils\mijay.Utils.csproj' does not exist.	NClone.Benchmarks
Warning	3	The referenced project '..\mijay.Utils\mijay.Utils.csproj' does not exist.	NClone
Warning	4	The referenced project '..\mijay.Utils\mijay.Utils.csproj' does not exist.	NClone
Warning	5	The referenced component 'mijay.Utils' could not be found. 	NClone
How can I get around this? What is contained in "mijay.Utils" that is needed inside the NClone project(s) ? Couldn't that be included directly into NClone??

mijay.Utils is a submodule. To get it, you either should clone repository recursively with
git clone --recursive https://github.com/mijay/NClone.git

or update submodules after clone using
cd NClone
git submodule update --init

And how do I get this without using Git myself? I'm not a Git head - can't stand it - just tried to download your code as ZIP from this site here....

Then just also get code from mijay.Utils and put it in corresponding folder.

OK, that works - now I just still have references to the "Jetbrains.Annotations" package which I don't have....

It should come from nuget. Did you restore the packages?


SkeletonAccentSystem has world replacements right in cs file

it should be in prototypes or something
https://github.com/space-wizards/space-station-14/blob/30cdb90ac217c4e599c53eed10e693af098df1f6/Content.Server/Speech/EntitySystems/SkeletonAccentSystem.cs#L14-L31

fixed by #32166


Added publication of $fh.sync notifications for data sets to the mediator

Motivation
This is to allow raincatcher modules to subscribe to notifications that happen from the $fh.sync Client API.
This will allow the consuming modules to subscribe to the topics to alert the user whenever a synchronisation operation has failed.
Changes

Updated the code to add additional comments and more informative variable names
Added a subscribeToSyncEvents function to the DataManager to allow subscribing to sync events.

@evanshortiss, @MikeyBurkman quick review guys?

LGTM 👯♂

Can we get this merged to a feature branch and publish a pre-release?

@paolobueno Already publised as 0.1.0-alpha.1. I will merge to a feature branch

Closed for https://github.com/feedhenry-raincatcher/raincatcher-sync/pull/17

You can actually edit the base branch of a PR, i.e. could've gone from master to the feature branch.


⚠ Turnos 24x7 has degraded performance

In 3436fa6, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 958 ms

Resolved: Turnos 24x7 performance has improved in 68e2d7a.


helm chart

Is your feature request related to a problem?
deploying memos to a kubernetes cluster is inconvenient
Describe the solution you'd like
a hosted helm chart
Additional context
No response

see https://github.com/usememos/memos/pull/2095

Moved to https://github.com/usememos/helm


Remove version: FengHe.FocusNote version 2.12.5-660

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://download.kirakuapp.com/FN_PC/Focus Note Setup 2.12.5-660.exe
Response Error: The remote server returned an error: (403) Forbidden.
(Automated message - build 912)


Community meeting July 20 Agenda

Please add a topic in this thread and add a link to the Github issue associated with the topic.

Please make sure you give folks enough time to review/discuss the topic offline on Github before coming into the meeting :)

Paste the image of an animal :)

Some topics we think are important to discuss:

Introduce https://github.com/HeavyWombat
Support for Kube v1.18
Bumping Tekton to a newest release
Updates on last meeting https://github.com/redhat-developer/build/issues/295#issuecomment-660861587

Meeting notes:

We can update the supported features in primary documentation page, like runtime base-image support;
How do we release the operator? Lets write down our current procedure and look for further automation in next steps;

More meeting notes:

Get a new first tag for the Build. Prerequisite are:

Support for Kube v1.18
Marketing around the first tag should happen on the new organization.
We need to verify that the current Tekton works with Kube v1.18
Ensure backwards compatibility with v1.17 at least, TBD how to do this.
?? (from IBM TBD)

Future automation for building artifacts:

Build operator image for every commit merged into master and for each tag
Maybe store the Kube resources on a public accessible S3 bucket

Bump to tekton newest release:

Move to a more newest version and document why we need this.

Some findings around Tekton TaskRuns not allowing users to define resources for init-containers, tekton containers: see https://github.com/tektoncd/pipeline/issues/2931


Numeracy Programming Challenges

Addresses #247
Looks as though this branch claimed the history of milestone/unplugged-programming :\

@JackMorganNZ Uh oh ignore this for now something went wrong with the last merge because of the history voodoo trickery.

I also fixed all the red scratch blocks within my exercise additions.

Additional Fixes:

purple scratch blocks are now purple
updated all other programming markdown files in unplugged-programming for consistency:

sentences are now separated by newlines
language independence in challenge descriptions
list spacing