Stale request

When I add a "login_required" decoration to my view, Shibboleth tells me that my request is stale. Anyone ever dealt with this?

@sternj please give more details. Can you post the full traceback?

The docker-compose is located here. Upon navigating to localhost/web (specified in the httpd-proxy container routing to the sp container), I get the following screen:

The text of the log starting from the request is this. Note that the configuration for that specific container is located in /sp/etc-httpd/conf.d/sp.conf
web_1            | [13/Aug/2018 19:49:51] "GET / HTTP/1.1" 302 0
sp_1             | sp-native 2018-08-13 19:49:51 WARN Shibboleth.Config : DEPRECATED: legacy 2.0 configuration, support will be removed from a future version of the software
sp_1             | sp-native 2018-08-13 19:49:51 WARN Shibboleth.RequestMapper : DEPRECATED: legacy 2.0 configuration, support will be removed from a future version of the software
sp_1             | sp-native 2018-08-13 19:49:51 WARN Shibboleth.Application : handlerSSL should be enabled for SSL/TLS-enabled web sites
idp_1            | 2018-08-13 19:49:52,083 - ERROR [org.opensaml.profile.action.impl.DecodeMessage:73] - Profile Action DecodeMessage: Unable to decode incoming request
idp_1            | org.opensaml.messaging.decoder.MessageDecodingException: No SAMLRequest or SAMLResponse query path parameter, invalid SAML 2 HTTP Redirect message
idp_1            | 	at org.opensaml.saml.saml2.binding.decoding.impl.HTTPRedirectDeflateDecoder.doDecode(HTTPRedirectDeflateDecoder.java:108)
idp_1            | 2018-08-13 19:49:52,103 - WARN [org.opensaml.profile.action.impl.LogEvent:105] - A non-proceed event occurred while processing the request: UnableToDecode
ldap_1           | tail: '/var/log/dirsrv/slapd-dir/access' has been replaced;  following end of new file
ldap_1           | 	389-Directory/<IP_ADDRESS> B2018.178.1311
ldap_1           | 	localhost:389 (/etc/dirsrv/slapd-dir)
ldap_1           |
ldap_1           | [13/Aug/2018:19:49:31.833164994 +0000] conn=1 fd=64 slot=64 connection from <IP_ADDRESS> to <IP_ADDRESS>
ldap_1           | [13/Aug/2018:19:49:31.833386548 +0000] conn=1 op=0 BIND dn="cn=admin,dc=idptestbed" method=128 version=3
ldap_1           | [13/Aug/2018:19:49:31.833821186 +0000] conn=1 op=0 RESULT err=0 tag=97 nentries=0 etime=0.0000581104 dn="cn=admin,dc=idptestbed"
ldap_1           | [13/Aug/2018:19:49:31.834867868 +0000] conn=1 op=1 UNBIND
ldap_1           | [13/Aug/2018:19:49:31.834889716 +0000] conn=1 op=1 fd=64 closed - U1
idp_1            | 2018-08-13 19:50:34,852 - ERROR [org.opensaml.profile.action.impl.DecodeMessage:73] - Profile Action DecodeMessage: Unable to decode incoming request
idp_1            | org.opensaml.messaging.decoder.MessageDecodingException: No SAMLRequest or SAMLResponse query path parameter, invalid SAML 2 HTTP Redirect message
idp_1            | 	at org.opensaml.saml.saml2.binding.decoding.impl.HTTPRedirectDeflateDecoder.doDecode(HTTPRedirectDeflateDecoder.java:108)
idp_1            | 2018-08-13 19:50:34,854 - WARN [org.opensaml.profile.action.impl.LogEvent:105] - A non-proceed event occurred while processing the request: UnableToDecode

Try taking this package, and even django itself, out of the picture completely, and just make sure you can secure a URL with Shib. Once that's working, try django and this package again.

I can secure a URL with shib, navigating to / and clicking on the first link guarantees that

Yup, the issue could be with ProxyPass. For this package to work, you've got to be able to see REMOTE_USER (or some other header or environment variable you've configured) in request.META. When you secure a session with Shib, the REMOTE_USER env variable in Apache will be set (unless your shib is configured differently), and then Django can see it, and then this package can act on it.
You can remove this package completely, and just make sure that that REMOTE_USER header or environment variable is visible in request.META. If it's not visible, then the issue isn't anything in this package. And REMOTE_USER might not be set because of the ProxyPass setup, like you mention. We don't use this package with ProxyPass. You would need to pass through or set some header or environment variable in the request that actually gets to django - this might help: https://httpd.apache.org/docs/trunk/mod/mod_headers.html.
Here's how you would test it: set up your django app at /django_app. In your shib config, require a session for all /django_app URLs. When you visit it, you'll have to log in to shib. Then, you can see what's in your request.META information in the django view - there needs to be some information about the user that's logged in through shib. And this test doesn't require this package at all - it's just shib and django.
This page might be helpful: https://wiki.shibboleth.net/confluence/display/SHIB2/NativeSPAttributeAccess

I moved to a Flask implementation and ran into the same issue, I resolved it.
For future people with the same problem-- I resolved the issue by putting a specific subpath of my Django site behind Shibboleth protection (in Apache) and having the login link be to there instead of to the actual SSO link (which is what caused the stale request.
@bcail thank you for your help!!!

I moved to a Flask implementation and ran into the same issue, I resolved it. For future people with the same problem-- I resolved the issue by putting a specific subpath of my Django site behind Shibboleth protection (in Apache) and having the login link be to there instead of to the actual SSO link (which is what caused the stale request.
@bcail thank you for your help!!!

Hello. Sorry to ask this question six years later, and it’s okay if you can’t answer my question here, but I am running into this issue in a project at work, and it is a bit unclear to me what exactly you are saying you did?
When you say you changed the login link to a subpath under Shibboleth protection, do you mean the shib_view login link, or the in the project settings? The latter doesn’t seem to me like a setting that should be changed as such (it should be pointing to the local SP login link); but changing the former still isn’t working for me. (Granted, I am using an Nginx setup instead of Apache for this project, so maybe there’s an important subtlety there that I have missed?)

Oh jeez, that was several lifetimes ago for me. I think we might not have used nginx at that point specifically because the Apache ecosystem was simply better documented with Shibboleth. What it looks to me like I was saying is that I both set the SSO login URL on the Django side of things and also configured that route in the Apache side of things, I believe this commit that I made is most likely to be close to what I did. I can't speak to if this is the version where I finally got things working. I believe that my configuration likely is still live in my old department-- if you reach out to me (a few layers of indirection through my github profile and website), I could likely put you in touch with my old boss, who has likely touched the live code more recently.


Fix openvino export model from ONNX

Fixes https://github.com/huggingface/optimum-intel/issues/553

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Integration Test - Release - 2019/12/03 17:11:42

Auto create for integration test. IssueCommandFlow

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


urgent to fixed the delay search bug.

fixed the delay search table bug, it not real to wait for typing nothing before, and I fixed it this PR.

urgent to fixed the delay search bug for #330

@madeinfree, release on v2.1.1


Fix sklearn test that calls a removed field

Seeing this error in the CI:
E       AttributeError: '_CalibratedClassifier' object has no attribute 'base_estimator'
20852

Looks like it's been removed in the sklearn version.

@trivialfis @hcho3

Thanks!


Exporting spot lights angles and colors

The calculation used to determine inner and outer angles of the exported spot are completely wrong and have no meaning compared to what the user is setting up (blending in from full 180 deg instead of what the user is setting as the outter angle? why?)
Light colors again, they were fixed to be in the right range (0-255) but are exported in the wrong colorspace and that gives no real control to the user over the color of the lights, and i'm guessing the import doesn't translate them into linear color for blender to use the color properly (haven't tested though but i will)

You should use CW light editor for now.

I'll look into this but yeah I would recommend using CW light editor. Lights in Sollumz are pretty bad right now


Update image-right.vue

!!!!!! Does not work !!!!!!!
It does scale
But layout is wrong (url left instead of right)

replaced with #783


Added better exceptions when verifying the CSRF state

This PR makes it easier to debug to find out what is wrong with the CSRF state.
Ping @Donovan-Ensicaen

So, with this PR, I have this exception :
Type: Happyr\LinkedIn\Exceptions\LinkedInApiException

Message: Could not find a stored CSRF state token.

Filename: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php

Line Number: 267

Backtrace:

File: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 313
Function: getCode

File: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 296
Function: fetchNewAccessToken

File: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 216
Function: getAccessToken

File: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 200
Function: getUserFromAvailableData

File: .../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 427
Function: getUser

File:.../vendor/happyr/linkedin-api-client/src/Happyr/LinkedIn/LinkedIn.php
Line: 114
Function: getUserId

File: .../controllers/C_user.php
Line: 460
Function: isAuthenticated

It seems the issue comes from the storage. The storage is empty when I'm redirected on my website.
With a var_dump at the beginning of the getCode method :
var_dump($storage);
--> object(Happyr\LinkedIn\Storage\SessionStorage)#42 (0) { }

var_dump($this->getState());
--> NULL

Okey, Lets figure out why your storage is empty.
Can you debug the LinkedIn::establishCSRFTokenState() function. It sets a state to the storage. Is it executing as expected?
Which storage are you using?

Ok, there is a problem with the storage. I don't use the IlluminateSessionStorage, I have a SessionStorage object.
 protected function establishCSRFTokenState()
    {
        if ($this->getState() === null) {
            $this->setState(md5(uniqid(mt_rand(), true)));
            $this->getStorage()->set('state', $this->getState());
        }
        var_dump($this->getState()); --> string(32) "b67da5014f79b1fbd4729f89287c8cdf"
        var_dump($this->getStorage()); --> object(Happyr\LinkedIn\Storage\SessionStorage)#47 (0) { }
    }

The storage is set but still empty.

Okey, interesting.
Do you ever run session_start()?
https://github.com/Happyr/LinkedIn-API-client/blob/master/src/Happyr/LinkedIn/Storage/SessionStorage.php#L24

The session is started in the SessionStorage constructor
public function __construct()
    {
        //start the session if it not already been started
         if(php_sapi_name() !== 'cli') {
            if (session_id() === '') {
                session_start();
                echo "Session started";
            }
        }
    }

--> Session started

I use codeigniter but I think it couldn't create this issue. I have no issues when I don't use another redirect-uri.

So;
$urlGivesYouError = $linkedIn->getLoginUrl(array('redirect_uri' => 'http://www.mywebsite.com/index.php/C_utilisateur/storeLinkedin'));
$urlIsJustFine = $linkedIn->getLoginUrl();

That is really weird. That should not effect the storage at all.
Is the code where you generate the login URL on the same domain as where you would like to store the accessToken?

It could be a problem on the login url.
I call the login url from http://www.mywebsite.com/index.php/LoadPanel/edit_profile
When I use the method getLoginUrl() like that :
$url = $linkedIn->getLoginUrl();

The redirect url generated is http://draafter.com/index.php/LoadPanel/edit_profile without the 'www'. In the linkedin app, I have to registered this url http://mywebsite.com/index.php/LoadPanel/edit_profile because this one doesn't work with the 'www' http://www.mywebsite.com/index.php/LoadPanel/edit_profile ...
If I use this method with the redirect-url parameter :
$url = $linkedIn->getLoginUrl(array('redirect_uri' => 'http://mywebsite.com/index.php/LoadPanel/edit_profile'));
--> it works

$url = $linkedIn->getLoginUrl(array('redirect_uri' => 'http://www.mywebsite.com/index.php/LoadPanel/edit_profile'));
--> it doesn't work

If I redirect in another controller, it doesn't work
$url = $linkedIn->getLoginUrl(array('redirect_uri' =>'http://www.mywebsite.com/index.php/C_utilisateur/storeLinkedin'));

because the generated url doesn't contain the 'www'
If I use this one
$url = $linkedIn->getLoginUrl(array('redirect_uri' =>'http://mywebsite.com/index.php/C_utilisateur/storeLinkedin'));

IT WORKS !!

Okey. Thank you.
The problem is that the $_SESSION is bound to a domain. www.mywebsite.com and mywebsite.com are two different domains and they do not share the $_SESSION.
You could create an other storage (like file storage or memcache) in order to be decoupled from the issue with different domains.
I will merge this PR because it gives more hints when debugging.

Yes you're right, www is like a subdomain of mywebsite.com. Thank you for your help! A better way should be to take the current url in the browser. My current url is http://www.mywebsite.com but the generated url us http://mywebsite.com.

I thought it did take the current URL.. https://github.com/Happyr/LinkedIn-API-client/blob/master/src/Happyr/LinkedIn/Http/UrlGenerator.php#L75
If there is an error here would you mind creating a PR?

I could be in this line
$protocol = $this->getHttpProtocol() . '://www';

instead of
$protocol = $this->getHttpProtocol() . '://';

I'm not sure if it will fix the issue for all people or just in my case.

I think it will just cause problems if you assume that every domain will start with www.

I have a feeling this has something to do with deployment on multiple servers. i.e. You cannot use file based sessions there. You need to use memcache/redis or mysql based sessions to keep it consistent across instances.


[msal-node] SNI auth for confidential client flows

Coverage remained the same at 82.901% when pulling 658319c95deb4c017c79585e1660fcfc356c1610 on sni into b3f1422a07c98b4ad34b8ce53059dff33092f7c2 on dev.


Screenshot generation fails in 2.10.x pre-release

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[ ] Bug is not already reported in another issue
Mentioned in https://github.com/corona-warn-app/cwa-app-android/pull/4036#issuecomment-915021258

Technical details

Device name: Samsung Galaxy A52 5G SM-A526B/DS
Android version: 11
App version: release/2.10.x at commit https://github.com/corona-warn-app/cwa-app-android/commit/82aa547dc3450f4a650235179fbd7cb9b7cee7d6

Describe the bug
Screenshot generation fails and generates only a small number of the expected screenshot files.
Steps to reproduce the issue
In Android Studio:

Run de.rki.coronawarnapp (androidTest)
Note that test run fails and stops at RecoveryCertificateDetailFragmentTest.kt showing 1 failed, 28 passed 197 tests
Note that in /data/data/de.rki.coronawarnapp.test/screenshots 14 png screenshot files have been saved.

Expected behavior
Compare to running de.rki.coronawarnapp (androidTest) on release/2.9.x which runs to completion showing 6 failed, 191 passed 197 tests. In /data/data/de.rki.coronawarnapp.test/screenshots 119 png screenshot files have been saved.
Possible Fix
Review and correct RecoveryCertificateDetailFragmentTest.kt. Other test files may also cause issues once RecoveryCertificateDetailFragmentTest.kt has been fixed.
Additional context
Before PR https://github.com/corona-warn-app/cwa-app-android/pull/3988 "Booster Vaccination (EXPOSUREAPP-8718)" was merged into  release/2.10.x screenshots could be generated correctly.

See PR https://github.com/corona-warn-app/cwa-app-android/pull/4043.

This issue was resolved by PR #4043.
Running de.rki.coronawarnapp (androidTest) on PR #4043 runs to completion showing 6 failed, 191 passed 197 tests. In /data/data/de.rki.coronawarnapp.test/screenshots 120 png screenshot files have been saved.
PR #4043 is included in v2.10.0-RC0.
Closing, since the issue is resolved and it only occurred in a pre-release version.


Add pixi update --json

Problem description
https://github.com/prefix-dev/pixi/issues/73#issuecomment-1926999625
I would like to create an internal dependabot replacement that calls pixi update and generates some markdown diffs. I planned to make the json-to-md renderer open source after that :)
I don't think it would make sense to let pixi create markdown directly since eventual dependabot/renovate config will probably do their own implementation.

What would you like to output of this command to look like?

{
  "py38": {
    "linux-64": {
      "python": {
        "before": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "after": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "changed": "no" // one of build, major, minor, patch
      },
      "polars": {
        "before": {
          "version": "0.10.0",
          "build": "py38h1f7f2b9_0"
        },
        "after": {
          "version": "0.10.1",
          "build": "py38h1faeeb9_1"
        },
        "changed": "patch"
      }
...

how about something like this?

Seems simple enough!
Just the changed is a bit more complex. Not all versions are semver based and both the version and the build number can be changed at the same time. Do you really need this field?

I would also be fine with no, added, removed, newer, older or no, added, removed, changed and i would do the version magic with a custom tool.

But wouldnt this already be present from the fact that "before" and "after" exist or not?

added - only "after"
removed - only "before"
changed - "after" and "before"
no - package not present? Or neither and "after" or a "before"

yes, that's a bit smarter than my idea 😅 then let's not include changed at all

we probably also need to add pypi dependencies there:
{
  "py38": {
    "linux-64": {
      "python": {
        "before": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "after": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "type": "conda"
      },
      "polars": {
        "before": {
          "version": "0.10.0"
        },
        "after": {
          "version": "0.10.1"
        },
        "type": "pypi"
      }
...

how about this?

Sure thing!

Almost done with the markdown generator 🤩
One other thing I would love to have: an indicator whether the dependency is explicit (i.e. defined in our pixi.toml) or not.

{
  "py38": {
    "linux-64": {
      "python": {
        "before": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "after": {
          "version": "3.9.0",
          "build": "h12debd9_0"
        },
        "type": "conda",
        "explicit": true
      },
      "polars": {
        "before": {
          "version": "0.10.0"
        },
        "after": {
          "version": "0.10.1"
        },
        "type": "pypi",
        "explicit": false
      }
...


implement squeezing

Adds symbolic squeeze operator SqueezeOp and squeezed coherent state SqueezedCoherentState, with supported numerical translations to QuantumOptics.jl and symbolic simplification.

@krastanov I thought about this a little more after developing the Gaussian quantum info package and would like to alter this PR slightly as follows:

Change SqueezedCoherentState to SqueezedState, the latter being a squeezed vacuum state. If someone wants to create a squeezed coherent state, then they can simply apply the displacement operator DisplaceOp to a SqueezedState object. On the user end, this makes it more obvious what is being done symbolically. Plus, currently in the Gaussian quantum info package I have a squeezedstate function that creates a squeezed vacuum state in the phase representation, so making these changes would make a symbolic translation to that package slightly more straightforward. In the future once we register the Gaussian info package I can submit a PR to QSymbolics that enables easy translations with express.


Failed to start local server

When trying to install a local threebot server I get the following error:
    ---------------------------------
    COULD not load
        source:/sandbox/code/github/threefoldtech/jumpscaleX_threebot/ThreeBotPackages/zerobot/base/actors/gdrive.py
    ERROR WAS:No module named 'googleapiclient'
--TRACEBACK------------------
<stdin> in <module>
    1
/sandbox/lib/jumpscale/Jumpscale/servers/threebot/ThreeBotServersFactory.py in local_start_default
    91    client = self.default.start(background=background, packages=packages)
/sandbox/lib/jumpscale/Jumpscale/servers/threebot/ThreebotServer.py in start
    265    self._packages_core_init()
/sandbox/lib/jumpscale/Jumpscale/servers/threebot/ThreebotServer.py in _packages_core_init
    386    p.actors_reload()
/sandbox/lib/jumpscale/Jumpscale/tools/threebot_package/ThreeBotPackage.py in actors_reload
    183    raise e
/sandbox/lib/jumpscale/Jumpscale/tools/threebot_package/ThreeBotPackage.py in actors_reload
    178    cl = j.tools.codeloader.load(obj_key=None, path=fpath, reload=False, md5=None)
/sandbox/lib/jumpscale/Jumpscale/tools/codeloader/CodeLoader.py in load
    85    raise j.exceptions.Base(msg)
-----------------------------

I think you've an old docker image because googleapiclient is bundled with the current image?

@xmonader correct, updating the image fixed the problem.


FRONTEND WITH FRAMEWORK: VueJS-4.5 => HTML rendering (v-html)

Description
Contributors are required to explain abou HTML rendering (v-html) in Vue.js, which should cover questions like:-
What is the HTML rendering (v-html) in Vue.js?
How does this work?
Why there is a need for this?
What are the advantages to use it and when to use it?
How to implement this with code (also give theory explanation of code)?
You can take 2-3 examples with codes of different types to explain this.
etc....
Note:
Plagiarism is strictly prohibited.
Changes should be made inside the Frontend_Web_Development_React_Angular_Vue/ dir inside Frontend_Web_Development_React_Angular_Vue branch.
Files need to be added into the Week4 folder only.
Domain
Frontend Dev React/Angular/Vue
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@SourabhSaraswat-191939 assign me

@Saurabh-crypto16 any updates

@anandhu720 Sorry I couldn't complete it due to medical reasons

@Saurabh-crypto16 can you do it within one week or need to re assign this.


How to transform a igraph or networkx object into hypergraph?

How to transform a igraph or networkx object into hypergraph object used in DHG?

Hi, you can extract the edge list and the number of vertices from G of networkx. Then, you can construct a DHG's Graph with the default construction function https://deephypergraph.readthedocs.io/en/0.9.3/api/dhg.html#dhg.Graph .
We will add the transform function of igraph and networkx in the next version. ^^

Thank you for your timely reply. Could you kindly suggest how to set node attribute?

If you don't have the node attribute, you can try the following ways:

initialize the vertex feature with the normal distribution and set it as trainable parameters. (Common in recommender system)
initialize the vertex feature with the normal distribution.
initialize the vertex feature with a one-hot encoded vector. (you can encode the degree of each vertex or directly use a identity matrix as the vertex feature)
train a auto encoder: the input is the adjacency matrix A and the output is the same. initialize the vertex feature with the hidden embeddings.

Thanks. If I want to set a numeric vector (degree) as node attribute. How to assign the numbers to nodes?

You can try to transform the degree to a one-hot vector. Refer to this https://github.com/weihua916/powerful-gnns/blob/master/util.py#L118

Hi @yifanfeng97 I wonder what encoding of Hypergraphs you would use in NetworkX transformations. I'm implementing such encoding and was hoping to use this one:

https://ncatlab.org/nlab/show/hypergraph#hypergraphs_as_2colored_graphs

What do you think? Thanks!

Hi again! I would like to invite you to join the HIF effort, allowing hypergraphs to be shared across many libraries. This can complement the pickle strategy that is currently implemented with cross library and language compatibility.
I hope you would like to take a look at our repository where we provide the specification of a JSON format. It is an incidence-based encoding that looks to be compatible with the structure of your library. This format has been implemented in HypergraphX, HyperNetX and XGI.
https://github.com/pszufe/HIF-standard
Let me know what you think and any way I can help make this happen!
Cheers and happy new year.


ISPN-9225 ANTLR 3.5.2

https://issues.jboss.org/browse/ISPN-9225

Compilation failed with 3.5.2, but I found out the cause. Seems that the 3.4 we were using before did not generate code using generics. An then for whatever reason 3.5.2 generates code with List instead of List. I diffed the parser generated with 3.4 and 3.5.2 and it seems these + some exception handling are the only differences so I hope we are not breaking anything (more than it already is) while upgrading. This upgrade seems logical also because we previoulsly processed the grammar using 3.4 but used the 3.5.2 runtime :) Not sure how that happened but I think it's safer to have both 3.5.2.

Updated commit message and rebased. Ready now.

Merged, thanks


AKCORE-3: Add ShareGroupHeartbeat RPC which is marked as an unstable RPC , supports authorization and group.share.enable config

About
This PR implements ShareGroupHeartbeat RPC which is marked as an unstable RPC , supports authorization and group.share.enable config
Reference - AKCORE-3
The implementation to support ShareGroupHeartbeat RPC is not done yet in the Group Coordinator so I have mentioned it as a TODO which needs to be followed once the implementation of Group Coordinator is complete. Right now we give an INVALID_CONFIG exception for that code scenario.
Build
The build is passing as shown in the screenshot

The check styles is passing as shown in the screenshot

Testing
Added 3 unit tests to demonstrate the added code functionality -

testShareGroupHeartbeatReturnsUnsupportedVersion

testShareGroupHeartbeatRequest  - The current test checks for INVALID_CONFIG exception, however once the RPC is supported in Group Coordinator, this TODO should be followed (I have tested the commented out test by using Andrew's prototype code for ShareGroupHeartbeat API support in Group Coordinator)

testShareGroupHeartbeatRequestAuthorizationFailed

Note - I have run the tests for approx. 10 times to make sure they are not flaky.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the PR @adixitconfluent. Left some comments, also can you please add tests for ....Request|Response.java classes as well.

Hi @apoorvmittal10, Regarding adding comments for ShareGroupHeartbeatRequest | ShareGroupHeartbeatResponse.java, these classes are tested through the 3 tests mentioned in KafkaApisTest.scala. I see ConsumerGroupHeartbeatRequestTest.java but most of those tests use group coordinator functionality except using UNSUPPORTED_VERSION, so I can add 1 for that. I will add a test similar to that for share groups.


Can not build Trapdoors

when i want to build a trapdoor on the craftingtable I only can build the Wall with the same Wood. (with the receipebook or per hand)

Oh. I didn't remember it was the same recipe as trapdoor. I will definitely fix it in nex update.

It is not exactly the same recipe. The walls want the upper 6 fields and the trapdoors the lower 6 ones. But now it becomes walls in both variants.

I changed the recipes for plank walls to be like :
 #
###
###

where # is a plank.
And that give 7 walls.

Thanks for ur fast response and fixing it

Same with glass_pane (colored, too)

Oh yeah it's impossible to craft glass pane now

Done, like plank walls


Backstage owner

:tada: This PR is included in version 0.11.21 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implement Serialize and Deserealize traits for resources

What problem does this solve or what need does it fill?
The following SystemParam doesn't compile:
#[derive(SystemParam, Serialize, Deserialize)]
struct MySystemParam<'w, 's> {
    control: ResMut<'w, MyResource>,
    #[serde(skip)]
    #[system_param(ignore)]
    marker: PhantomData<&'s usize>
}

Because ResMut doesn't implement Serialize and Deserialize.
What solution would you like?
I would like to have this traits implemented for ResMut and Res if Bevy's serialize feature is enabled.
Additional context
Serde have a similar implementation for WeakPointers.

Won't work, Res and ResMut are just pointers.


Fix path traversal vulnerability

There is a vulnerability in upload.py that allows users to upload files outside of the expected user.scan_directory + uploads folder.

Thanks! Looks good to me :)


pipelineX remove source state from operator functions

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


May I ask where is the Patch Embedding or PatchEmbed in your code?Do you divide the window first and then divide the patch?

https://github.com/rayleizhu/BiFormer/blob/1697bbbeafb8680524898f1dcaac10defd0604be/models/biformer.py#L183


[Feature Request] Restore old share url

For about a month or so now, the share button sends an x.com url, however this is annoying as Twitter's embeds on other (commonly, Discord) aren't the best. To work around this, we often use sites like vxtwitter to fix the embeds on those sites.
I was wondering if we could have an option to restore the old url so adding the vx isn't as much of a pain, alternatively a way to add our own url adjustments?

This request has already been declined https://github.com/insin/control-panel-for-twitter/issues/413


Question: where can I find a list of valid targets

I can't find pkg's changelog, so it's hard to know which Node versions are supported by pkg.
Is there a way to either have a definitive list, that gets updated whenever a new version is added, or better yet: an API call that will show all supported targets?

The README seems to answer to your question: https://github.com/zeit/pkg#targets
You'll find the same releases that the existing ones here: https://nodejs.org/en/download/releases/. Just respect the naming pattern node${release}-${platform}-${arch}

The README only explains how to specify a target. It does not (as far as I've seen) specify WHICH targets are supported. The amount of people who hit issues with Node 3 should be evidence of that.
The list of available releases you pointed to contains many releases that are actually not supported by pkg.
Again, either a changelog specifying which releases are supported, or a CLI command listing supported releases is needed, IMHO.

@TravelingTechGuy  I think you can refer to the patches file here to see which version are used: https://github.com/zeit/pkg-fetch/blob/master/patches/patches.json

@allardy Thanks, that does clearly show the supported version. Now, is there a way to include those in a changelog, or include a link to this file in the README?
Also, what is the process by which support for a new Node version is added? E.g I'm currently using v13.9.0. When will that be supported? What is required to add support for that? Can I help?

In the pkg-fetch repository, you have all the tools required to build the node binary from its source code, so the best I can tell is to clone it yourself and try building one of the existing version.
It seems that there's only one patch to be executed for a specific version, so I guess you could just see what pkg is changing and try to do the same in the newest version.
I also guess the process is community-driven, so when someone takes the time to do it, it appears :)


Error in reading initial charges - coordinates

From the input file obtained with the new version of the script, the one with the "restart-from-iter" option, the initial charges seem not to be correctly read by the program, even in the option ReadInitialCharges = Yes is specified. The output does not show the typical step 0 where old charges are read, which should appear as follows due to the zero difference in the energies:
***  Geometry step: 0
iSCC Total electronic   Diff electronic      SCC error
1    0.00000000E+00    0.00000000E+00    0.21845085E-05
Differently, the program tries to optimize the charges for the step 0 and fails in the optimization. Since the output says that the charges are correctly read from the charges.bin file, this should mean that the coordinates do not correspond to the last step of the previous run. This probably derives from some error in the production of the Geometry with the current format (the "explicit geometry specification", as indicated in the DFTB+ manual) instead of the GenFormat, which was employed in the previous version of the python script.

Does this issue occur only when you restart from a step excepting the last step? or always occur?
If you change the current description from current format to GenFormat, does the charges are read correctly?

As you correctly said in the other conversation, the charges stored in the 4th column of the geo_end.xyz file are not suitable for restart purposes, otherwise the charge optimization will crash in step 0 as I could observe. Therefore the only option to restart from a given step which is different from the last one is to deactivate the ReadInitialCharges option and not to provide to the program any partial charge at all. In this way the program will optimize them in step 0, which is only a small lost of time.
The other issue remains: you say that it is mandatory to provide initial charges when the "explicit geometry specification" is used. Could you indicate where this is written in the manual? I can not figure out this section. Also, I tried restart with this kind of geometry and no initial charges, and it looks ok.


BLD: Update numexpr to latest to satisfy tables deps.

tables 3.3.0 requires numexpr >= 2.5.2

If this passes CI, then we should remove our numexpr conda recipe.

Coverage remained the same at 87.216% when pulling 55e3f9f8de93f73b784689cbdafbeb32842e3550 on update-numexpr into f143a83a44bca08d7199fa7727ed2079d90e63ca on master.

Just pushed removal of the numexpr recipe

Coverage remained the same at 87.216% when pulling bfe4500412cf8934c2f4119b311f6e9e3b015585 on update-numexpr into f143a83a44bca08d7199fa7727ed2079d90e63ca on master.


allOf within schema properties of the response generate inline records

Description:
When trying to generate the connector for Pipedrive API, the records in the type.bal are not generated properly. The allOf within schema properties of the response in operations of the following openapi.yml generate inline records.
OS, DB, other environment details and versions:
JDK 11
Ballerina Beta 2
Ubuntu 20.04
Steps to reproduce:
Download the following openapi.yml & try to generate the connector using the OpenAPI tool.
You could see the following record is generated
public type GetActivitiesResponse200 record {
    boolean success?;
    record {# Due date of the Activity. Format: YYYY-MM-DD
        string due_date?; # Due time of the Activity in UTC. Format: HH:MM
        string due_time?; # Duration of the Activity. Format: HH:MM
        string duration?; # The ID of the Deal this Activity is associated with
        int deal_id?; # The ID of the Person this Activity is associated with
        ........

Related to the following lines in the openapi.yml
      responses:
        '200':
          description: A list of Activities
          content:
            application/json:
              schema:
                title: getActivitiesResponse200
                type: object
                properties:
                  success:
                    type: boolean
                  data:
                    type: array
                    items:
                      title: activityResponseObject
                      allOf:
                        - type: object
                          properties:
                            due_date:
                              description: 'Due date of the Activity. Format: YYYY-MM-DD'
                              type: string
                              format: date
                            due_time:
                              description: 'Due time of the Activity in UTC. Format: HH:MM'
                              type: string
                            duration:
                              description: 'Duration of the Activity. Format: HH:MM'
                              type: string
                            deal_id:
                              description: The ID of the Deal this Activity is associated with
                              type: integer

Please move any inline object schema given under a composed schema to a separate schema as below
responses:
        '200':
          description: A list of Activities
          content:
            application/json:
              schema:
                title: getActivitiesResponse200
                type: object
                properties:
                  success:
                    type: boolean
                  data:
                    type: array
                    items:
                      title: activityResponseObject
                      allOf:
                        - $ref: "#/components/schemas/ActivityResponseObject"
components:
  schemas:
    ActivityResponseObject:
      properties:
        due_date:
          description: 'Due date of the Activity. Format: YYYY-MM-DD'
          type: string
          format: date
        due_time:
          description: 'Due time of the Activity in UTC. Format: HH:MM'
          type: string
        duration:
          description: 'Duration of the Activity. Format: HH:MM'
          type: string
        deal_id:
          description: The ID of the Deal this Activity is associated with
          type: integer

Type for the response will be generated as below.
public type GetActivitiesResponse200 record {
    boolean success?;
    record{*ActivityResponseObject;}[] data?;
};

public type ActivityResponseObject record {
    # Due date of the Activity. Format: YYYY-MM-DD
    string due_date?;
    # Due time of the Activity in UTC. Format: HH:MM
    string due_time?;
    # Duration of the Activity. Format: HH:MM
    string duration?;
    # The ID of the Deal this Activity is associated with
    int deal_id?;
};

It is further advised not to configure an AllOf schema if only one object given.


requirements.txt and runtime.txt are not followed if setup.py is missing

In https://github.com/simonsfoundation/regulatory_network_examples the requirements.txt
and runtime.txt were not being observed.  In https://github.com/simonsfoundation/jp_svg_canvas they are observed.  The only difference I can see is that https://github.com/simonsfoundation/jp_svg_canvas has a setup.py script.
I could work around by adding python=2 to environment.yml and adding
pip install -r requirements.txt to the postBuild script.
In my view these work arounds should not be needed.

one option to help you debug is to use the --debug and --no-build flags with repo2docker locally...this will print the dockerfile that r2d builds before creating an image. See the syntax for that here:
https://github.com/jupyter/repo2docker#usage
That said, IMO if you're using environment.yml, you should be using that file for pretty much all python dependencies. I could see how mixing env.yml and reqs.txt could create confusions and edge cases

I tried to use the --debug and --no-build flags.  The output of the two builds were identical.

then...that is super weird :-)
If you're still debugging, feel free to paste the --debug output in this issue so we can take a look, though it sounds like the pip: field in environment.yaml fixed it?

Here is the output (stderr and stdout)  the only difference between the two builds
is the last line currently.
[Repo2Docker] Looking for repo2docker_config in /Users/awatters/tmp/repo2docker

FROM buildpack-deps:zesty

 # Set up locales properly
RUN apt-get update && \
    apt-get install --yes --no-install-recommends locales && \
    apt-get purge && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

RUN echo "en_US.UTF-8 UTF-8" > /etc/locale.gen && \
    locale-gen

ENV LC_ALL en_US.UTF-8
ENV LANG en_US.UTF-8
ENV LANGUAGE en_US.UTF-8

 # Use bash as default shell, rather than sh
ENV SHELL /bin/bash

 # Set up user
ENV NB_USER jovyan
ENV NB_UID 1000
ENV HOME /home/${NB_USER}

RUN adduser --disabled-password \
    --gecos "Default user" \
    --uid ${NB_UID} \
    ${NB_USER}
WORKDIR ${HOME}

RUN apt-get update && \
    apt-get install --yes --no-install-recommends \
       less \
       nodejs-legacy \
       npm \
       && apt-get purge && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

 # make JUPYTERHUB_VERSION a build argument
ARG JUPYTERHUB_VERSION=0.7.2

EXPOSE 8888

 # Almost all environment variables
ENV APP_BASE /srv
ENV CONDA_DIR ${APP_BASE}/conda
ENV NB_PYTHON_PREFIX ${CONDA_DIR}
# Special case PATH
ENV PATH ${CONDA_DIR}/bin:${PATH}
 # If scripts required during build are present, copy them

COPY conda/install-miniconda.bash /tmp/install-miniconda.bash

COPY conda/environment.yml /tmp/environment.yml
RUN bash /tmp/install-miniconda.bash && \
rm /tmp/install-miniconda.bash /tmp/environment.yml

 # Copy and chown stuff. This doubles the size of the repo, because
 # you can't actually copy as USER, only as root! Thanks, Docker!
USER root
COPY src/ ${HOME}
RUN chown -R ${NB_USER}:${NB_USER} ${HOME}

 # Run assemble scripts! These will actually build the specification
 # in the repository into the image.
USER ${NB_USER}
RUN conda env update -n root -f "environment.yml" && \
conda clean -tipsy

 # Container image Labels!
 # Put these at the end, since we don't want to rebuild everything
 # when these change!

 # We always want containers to run as non-root
USER ${NB_USER}

RUN ./postBuild

You have an environement.yml file, thus we use conda. If you need extra requirement not available with conda I believe you need to use the pip: section of environment.yml
I think it looks like the requiremetn.txt of svg_canavas looks to be observed, is that they are all dependencies of ipywidget, which is not the case for regulatory_network_examples.
I may be wrong.

Ok, I'm fine with not using requirements.txt if you have environment.yml as long as the restriction is documented.  I find it confusing to have runtime.txt not work sometimes (even though there is an easy work around).

👍 on documenting this. The docs make a point of mentioning composability which made me think that you can use environment.yml and requirements.txt and apt-get all together (with some well defined order of execution). I think we should add two sections to the docs:

which build packs can be combined and which can't
in which order build packs are applied

FWIW I feel like it wouldn't be that hard to use both environment.yml and requirements.txt, no? You'd just need to source activate the conda environment before you ran the pip install commands.

FWIW I feel like it wouldn't be that hard to use both environment.yml and requirements.txt, no? You'd just need to source activate the conda environment before you ran the pip install commands.

I think it would make sens though There should be one-- and preferably only one --obvious way to do it.. And from experience using conda + pip without the pip section of env.yml lead to trouble when installing-uninstalling packages. I'd like to not encourage behavior in binder that would lead to trouble when used  locally.
If we do a reproducibility badge we  could add a lint rule that detect requirements.txt and env.yml and display a warning.

or we could error on detection of both environment.yml and reqs.txt. I foresee people getting confused over this since it's a bit hard to catch in the logs (esp since they don't persist after the binder is built). If we had an informative error message telling people to use the pip: field maybe that'd help

@choldgraf Do you think #165 closes this issue?

yep!


Refactor vision tasks(image classification, object detection)

Description of changes:

Refactor vision tasks(image classification, object detection to align with new public APIs)
split jenkins unittests for each module, let them run on separate nodes to speed up unittests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-733-30 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/30/index.html

Cool stuff regarding splitting the jenkins unittests to be parallel. Do you know if this is possible for the doc build?

Cool stuff regarding splitting the jenkins unittests to be parallel. Do you know if this is possible for the doc build?

For doc it's not possible yet, as the built results must be collected by sphinx and generate html after all tutorials are complete. I heard there are some advanced artifacts sharing techniques across different nodes, but aren't quite sure about that.

Job PR-733-40 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/40/index.html

Job PR-733-45 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/45/index.html

Job PR-733-47 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/47/index.html

Job PR-733-48 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/48/index.html

Ready for review, with docs(http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/48/index.html)
@Innixma @jwmueller @sxjscience @gradientsky Appreciated!

Job PR-733-49 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/49/index.html

Job PR-733-56 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/56/index.html

Job PR-733-58 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/58/index.html

Job PR-733-59 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/59/index.html

Job PR-733-65 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/65/index.html

@Innixma Could you kindly give it another round of review? Doc is here: http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-733/65/index.html

Merging this to unblock other PRs, will address docs in a follow up PR.


Keyboard text navigation/selections

I often use the following keyboard shortcuts for navigation:

Home to navigate to start of line (in VSCode pressing it once navigates until end of indentation, pressing it again will navigate to actual start of line)
End to navigate to end of line
Shift+Home to select until start of line (similar behavior to Home with regards to indentation)
Shift+End to select until end of line
Ctrl+Shift+LeftArrow, Ctrl+Shift+RightArrow
PgUp, PgDn, Shift+PgUp, Shift+PgDn
Ctrl+Home to navigate to start of file
Ctrl+End to navigate to end of file
Ctrl+Shift+Home to select until start of file
Ctrl+Shift+End to select until end of file
Ctrl+A to select entire file
Delete to remove character to the right
Ctrl+Backspace and Ctrl+Delete to remove until word boundary

None of these seem to be working by default at the moment on Linux at least, which seriously impedes usability and productivity

I tried to close file via cmd+w it doesn't work either.

cc https://github.com/lapce/lapce/issues/186 as one more case, similar to Ctrl+Shift+LeftArrow, Ctrl+Shift+RightArrow

To add to this, a lot of what is expected is OS specific. I would like to see macOS's standard, option+arrow keys moves by whole words. option+arrow+shift selects whole words.

I'd also expect Shift-Delete to cut the selection (or entire line) to clipboard, and Ctrl-Insert to paste.


Make uproot-written files appendable in ROOT

Files written in uproot can be read in ROOT, but so far they can't be added to in ROOT (i.e. opening the file as a "UPDATE" TFile and putting new objects in it). The solution could be as simple as setting nfree to 1, rather than 0; we simply haven't investigated it, but it's an important feature to have.

We needed several new concepts:

The directory info has a UUID and 12 empty bytes beyond that. This is important because ROOT overwrites the directory info when it adds objects to the directory.
We were completely misunderstanding the fSeekFree, fNbytesInfo, and nfree fields; see issue #135 (block management) for an explanation. This is how we can inform ROOT of empty blocks in the file, something that will become important when we do block management.

Tested (1) writing an empty file in uproot and appending to it in ROOT, and (2) writing a non-empty file in uproot and appending to it in ROOT. As per #139, we need unit tests!


Last line of code blocks is truncated

Pictured:

Turns out this is an issue with the latest version of logseq. It is present in all themes.


Redirect stdout

Coverage decreased (-1.6%) to 79.279% when pulling 8624fdfd7baf5a34405ef499a5d10d2cc49ca505 on feature/pipe-stdout into 8e7253ade8064db8b742f4fae594282edcb9eb37 on master.

Coverage increased (+0.6%) to 81.513% when pulling f6a8a5dc9317cf1eceda90f8c7bf05eeb94d5985 on feature/pipe-stdout into 8e7253ade8064db8b742f4fae594282edcb9eb37 on master.

Coverage increased (+0.6%) to 81.513% when pulling 610d40ff051272306d794200dd18cc1853b82048 on feature/pipe-stdout into 8e7253ade8064db8b742f4fae594282edcb9eb37 on master.

Coverage increased (+0.6%) to 81.513% when pulling 156abb5b2d08f130e6c3ad927eec142ae759e33c on feature/pipe-stdout into 8e7253ade8064db8b742f4fae594282edcb9eb37 on master.


Edited README and FAQ

Some pretty lightweight fixes.

Thanks a lot @LappleApple. Do you want to sign the CLA so we can merge this? Otherwise, we would have to commit these changes with our accounts.

"I signed it!"

LGTM, thanks a lot @LappleApple  :) just a minor fix mentioned by @linki above and can be merged

@LappleApple I'll change it myself. @ideahitme I think we can do small nitpicks directly outselves in the browser.

@ideahitme Let's don't merge it just yet. I'll have some minor tweaks but no time right now.

Made some minor changes to your changes @LappleApple. /cc @ideahitme
LGTM, thanks @LappleApple

👍


Stanched undefined-property warnings in site model

Good enough! +1

Next time I'd use get(attribute), but this is all kinda a mess anyways for now.

Closed #364


Link inmediato con la búsqueda.

The Issue
Steps to Reproduce

Expected Behaviour
Tell us what should happen.
Actual Behaviour
Tell us what happens instead.
Suggested Solutions
Tell us your suggested solutions if you have any.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

the link appears immediately at the time of the search. so that the address is already known in advance.

BSk.7j

This does not look like a valid issue. Please be more mindful when opening one next time.


Policy not applied

When I create a lambda with a statement like the following the policy is not applied but no error is thrown either. The result is a policy with only the AWSLambdaBasicExecutionRole without any Role policies.
  SignAttachmentUploadFunction:
    Type: 'AWS::Serverless::Function'
    Properties:
      Handler: 'index.handler'
      Runtime: 'nodejs4.3'
      CodeUri: 'lambdas/sign-attachment-upload'
      Description: !Sub '${Project} API endpoint for signing a new file upload with a random filename hash.'
      Policies:
        - PolicyName: 'AllowAttachmentUploads'
          PolicyDocument:
            Version: '2012-10-17'
            Statement:
              - Effect: 'Allow'
                Action:
                  - 's3:PutObject'
                  - 's3:PutObjectACL'
                Resource: !Sub 'arn:aws:s3:::${AttachmentsBucket}/*'
      Environment:
        Variables:
          AttachmentUploadBucket: !Ref 'AttachmentsBucket'
      Events:
        SignAttachmentUpload:
          Type: 'Api'
          Properties:
            Path: '/sign-attachment-upload'
            Method: 'post'
    DependsOn:
      - 'AttachmentsBucket'

I solved it by only including the PolicyDocument instead of the entire policy.
  SignAttachmentUploadFunction:
    Type: 'AWS::Serverless::Function'
    Properties:
      Handler: 'index.handler'
      Runtime: 'nodejs4.3'
      CodeUri: 'lambdas/sign-attachment-upload'
      Description: !Sub '${Project} API endpoint for signing a new file upload with a random filename hash.'
      Policies:
        - Version: '2012-10-17'
          Statement:
            - Effect: 'Allow'
              Action:
                - 's3:PutObject'
                - 's3:PutObjectACL'
              Resource: !Sub 'arn:aws:s3:::${AttachmentsBucket}/*'
      Environment:
        Variables:
          AttachmentUploadBucket: !Ref 'AttachmentsBucket'
      Events:
        SignAttachmentUpload:
          Type: 'Api'
          Properties:
            Path: '/sign-attachment-upload'
            Method: 'post'
    DependsOn:
      - 'AttachmentsBucket'

Thanks!

That is the expected behavior. You are supposed to have the PolicyDocument data in there instead of the actual policy. We should update the documentation to make it more clear


Prepare to publish to npm as @ampproject/amp-closure-compiler

This PR makes a few changes to how the various sub-repos in @ampproject/amp-closure-compiler are structured. The primary goal is to be able to publish it to npm. The reason these changes were necessary is because only public repos can be published, while only private repos can use workspaces. Therefore, we have to replace all code that relies on workspaces with direct references between the different parts.
With this PR, we should be able to publish the following repos:
@ampproject/amp-closure-compiler
@ampproject/google-closure-compiler-java
@ampproject/google-closure-compiler-linux
@ampproject/google-closure-compiler-osx
@ampproject/google-closure-compiler-windows

Coming up: Further clean up.

@kristoferbaxter I had to make a bunch of changes (outlined in the description) to get this PR to work. I will now merge and attempt to publish before testing with amphtml.


Reduction of empty elements

As you might have guessed I'm currently dealing with a lot of empty elements and I'm trying to find the best way to handle them. While trying few things I stumbled across this behaviour, probably nan would be more appropriate in this case
>>> import awkward
>>> a = awkward.fromiter([[1,2], [3,4], [5,], [6,7]])
>>> b = awkward.fromiter([[1],[],[7],[8]])
>>> l, r = a.cross(b, nested=True).unzip()
>>> import numpy as np
>>> diff = np.abs(l - r)
>>> diff
<JaggedArray [[[0] [1]] [[] []] [[2]] [[2] [1]]] at 0x7fe9dab4d290>
>>> diff.min.__doc__
>>> diff.min()
<JaggedArray [[0 1] [9223372036854775807<PHONE_NUMBER>854775807] [2] [2 1]] at 0x7fe9e6dfa1d0>

Actually, the identity of min is inf, but inf, -inf, and nan are only possible values for floating point, and you have integers. Therefore, we have to use the maximum possible integer for this type.
It's ugly, and I'm changing it. In the future, min and max will be "semigroup reducers," without an identity element, and these will show up as None. (It will be a masked array with None covering up the extreme value, whether that's inf or the largest possible integer.)

Yeah, makes sense, thanks!


prompt pure 1.0.0 requires async

Hi,
sindresorhus/pure was updated to 1.0.0 with the commit https://github.com/sorin-ionescu/prezto/commit/f7ea78078f81d2d6fbe71d73274926a3f756a586. Problem is that there was a breaking change, and pure now depends on two files, not just one. async.zsh is now a dependency.
Any thoughts on how to solve the problem? Would adding the mafredri/zsh-async plugin be a plausible solution?

ln -s modules/prompt/external/pure/async.zsh modules/prompt/functions/async fixes it.
I would rather add zsh-async as an external module since it's useful in other places. It will have to be loaded like all other modules by adding it in zpreztorc since pure does not know to use pmodload async.

Your async is quite a lot of code. If I were to add it as another module, why should I change https://github.com/sorin-ionescu/prezto/blob/master/modules/prompt/functions/prompt_sorin_setup#L62-L92 to use it, my theme would know to pmodload 'async'?
Can you comment on #796's rendering issues and #888's freezing issues? Thank you.

If this can help u, I've solved the problem installing the pure-prompt with npm. But I think the best solution is to add zsh-async as an external module.

Is there a good way of detecting that pure is being used within prezto? If so, I could add that check to pure, and instead of doing autoload -Uz async && async, it would do pmodload 'async'. Does this sound reasonable?
@sorin-ionescu if you were to use it, one benefit of my async library is that it does not rely on files for communication, it all happens within the shell. So no race-conditions should happen and there is nothing that needs to be cleaned up. Other than that it can manage unique jobs (no multiple jobs) and you can tell it to kill of all current jobs if they are no longer relevant. The notifying via WINCH is also optional, you can resort to polling for results if you please.

@mafredri maybe, instead of add that check to pure, we can use a conditional prompt -c inside .zpreztorc to check if the selected theme is pure?

@Stelgard mm, I agree that it would be preferable to keep framework specific stuff out of pure, I guess your idea would work just fine as long as the async module get's initialized before the pure precmd.

A check could be added to modules/prompt/init.zsh. It would have to keep track of all a sync themes in an array that will require manual updating. It's dirty and error prone.

Cheers :beers:!

Woohoo! :tada: Thanks!

I would still like async as an external module though.

You could replace the symlink to prompt_pure_setup with a file, with this contents:
local current_dir
current_dir="$(dirname "${(%):-%x}")"

prompt_pure_setup() {
    pmodload 'async'
    source "$current_dir/../external/pure/pure.zsh"
    prompt_pure_setup "$@"
}

prompt_pure_setup "$@"

Why didn't I think of that?
pmodload 'async'
source "${0:h}/../external/pure/pure.zsh" || return 1

Wouldn't your example get evaluated at run time, even if pure is selected
or not? I wrote mine like I did because I thought it would get initialized
by the command prompt pure. I could be wrong and in that case go ahead
with your terser version :-).
On May 31, 2015 7:25 PM, "Sorin Ionescu"<EMAIL_ADDRESS>wrote:

Why didn't I think of that?
pmodload 'async'source "${0:h}/../external/pure/pure.zsh" || return 1
—
Reply to this email directly or view it on GitHub
https://github.com/sorin-ionescu/prezto/issues/892#issuecomment-107217224
.

It should not. The contents of the file is wrapped in an auto-loadable function with the name of the file whose body is loaded on first run. You execute it manually with prompt_pure_setup "$@" because you are redefining it.


Update config.toml

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference

@msfussell I think it's doing what it's supposed to, now.


Folder picker connectors produce error [Parameter "undefined" cannot be found]

Describe the Bug with repro steps

Open a workflow (any).
Add a new action.
Select "List files in folder" from OneDrive for Business.
Observe dropdown content.

I would expect a folder picker here, but instead an error appears.
Workflow JSON
N/A

Screenshots or Videos

Browser
Edge x64 111.0.1661.54, Windows 11.
Additional context
No response

Thanks for the reference. I searched "folder" and didn't find one.

Will have the undefined error fixed in: https://github.com/Azure/LogicAppsUX/pull/2020
however, still working on another PR to get the data for the picker. Use this for tracking:
https://github.com/Azure/LogicAppsUX/issues/1622


Enable a better launch screen on cold boot on android

ref: https://developers.google.com/web/updates/2015/10/splashscreen
The old icon was just a wee bit too small :)

https://developers.google.com/web/updates/2015/10/splashscreen

Oh, my bad! I dunno where I got the idea it was supposed to be 192. Thanks!

Huh, weird, it says here that the size for xxxhdpi is indeed 192. Are you sure the issue isn't that we should add a 512 icon so that it shows that for the splash screen?

Oh I see, web icons vs Android native icons. Got it! Thanks for the link. :)

heh :hocho: :cry:
yeah the manifest story right now is not as strong as I want it to be... but I am going to work on fixing that :dancer:

b1d7a9fe836f84f44172e5f7bb343bf765bc563e and 013576afd5c0d2a0ae0470db8435be9fa51aa988

@nolanlawson let me know when this lands :)

Awesome. Thanks again!
BTW it might be good to confirm in that post that it's 196, not 192. I saw some folks getting confused. cc @paulkinlan

cc @surma @mounirlamouri

Landed! Service worker bumped to 1.0.9 as well.

welp, this still is not working on a nexus 5 or nexus 5x :crying_cat_face: dang
I guess we could push some higher res options and see if that does it here

Hm, I think I noticed that a few days ago as well. The icon was bizarrely tiny.  It’s known (and annoying!) issue.

word, I just did the entire dance ^ on a 5x and no luck.. I will try on some other devices though

ref: https://developers.google.com/web/updates/2015/10/splashscreen
that it's 196, not 192

@samccone, @nolanlawson If nothing changed in Android, it should be 192x192px, see:

https://github.com/h5bp/html5-boilerplate/pull/1587#issuecomment-53695204
https://www.youtube.com/watch?v=MyQ8mtR9WxI&t=21m12s

(Cc: @paulkinlan, @addyosmani)

ok I just tested on chrome 47 and 48 and it works as expected... this feature has just not landed in chrome 46 like the blog post said :)

Oh interesting @alrra !

To be clear @alrra this is not the touch app icon, rather the splash screen loading icon, which is different as far as I know.

@samccone I was referring to both (sorry for the confusion).

the splash screen loading icon

@samccone See the second link, namely, https://www.youtube.com/watch?v=MyQ8mtR9WxI&t=21m12s.

welp dang (I was even in the audience for that talk :tomato: )
Going to try and wrangle the authorities on this to the thread :postal_horn: so we can get the "official" answer

The authority on this one would be @mounirlamouri. I've seen different dimensions (192) used more recently for PWAs, though Mounir has previously suggested opting for 384x384 (what we use in Polymer Starter Kit). Would be good to get a confirm :)

The issue is that a blog post describing a feature can go so far into details. There is unfortunately one size fit all icon size. The size Chrome expects for the splash screen is 128dp. We will use a smaller size if available. Below 80dp, in latest Chrome versions, we will change slightly the splashscreen layout. Below 48dp, we will no longer show an image on the splashscreen.
Anything below 200px is going to be a very poor match. I recommended 384px to @addyosmani because it is 128dp for devices with a pixel density of 3. It's a fairly high pixel density (Nexus 5 is lower than that). I believe some devices have a slightly higher pixel density. If you have 512px image, it would go be a 128dp for a pixel density of 4 (I don't know any device with a pixel density that high).
What I would highly recommend you to do is having multiple icons listed in your Manifest. Have a 128, 256, 384 and 512 icons. That way, you can let Chrome make the right decision and download the icon that has the size matching the one needed. The issue if you only have a 512px (or 384px) icon is that users with a low end device on a slow network will still end up downloading that bigger image and Chrome will resize it to fit the required dimensions.


Use generic crop icon when specific crop icon does not exist

@simonv3 has started to display the crop icons across the frontend. See the first peek under "Companion Plants" for Kale!

But its pretty ugly when there is no icon available:

So we can use this generic crop icon whenever this is the case.

I am getting started in the open source community and would like to help out on this issue.

Awesome @jeffcore! Let me know here or on slack if you need a hand.

Do you want me to store the generic-plant.svg in the public/img folder?

@jeffcore Yeah, that seems like a sensible place.


[Bug] A memory allocation bug is occurred on video super-reselution demo when use *.mp4 file as input

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (1.x).

Task
I have modified the scripts/configs, or I'm working on my own tasks/models/datasets.
Branch
master branch https://github.com/open-mmlab/mmediting
Environment
sys.platform: linux
Python: 3.7.16 (default, Jan 17 2023, 22:20:44) [GCC 11.2.0]
CUDA available: True
GPU 0,1,2,3: TITAN Xp
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.1, V10.1.24
GCC: gcc (Ubuntu 5.5.0-12ubuntu1~16.04) 5.5.0 20171010
PyTorch: 1.7.0
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) oneAPI Math Kernel Library Version 2021.4-Product Build 20210904 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v1.6.0 (Git Hash 5ef631a030a6f73131c77892041042805a06064f)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 10.1
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.3
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_VULKAN_WRAPPER -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.8.0
OpenCV: 4.7.0
MMCV: 1.6.0
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 10.1
MMEditing: 0.16.0+8de2721
Reproduces the problem - code sample
    if file_extension in VIDEO_EXTENSIONS:
        video_reader = mmcv.VideoReader(img_dir)
        data = dict(lq=[], lq_path=None, key=img_dir)
        for frame in video_reader:
            data['lq'].append(np.flip(frame, axis=2))
        tmp_pipeline = []
        for pipeline in test_pipeline:
            if pipeline['type'] not in [
                    'GenerateSegmentIndices', 'LoadImageFromFileList'
            ]:
                tmp_pipeline.append(pipeline)
        test_pipeline = tmp_pipeline
else:
         ...

Reproduces the problem - command or script
python demo/restoration_video_demo.py
./configs/restorers/edvr/edvrm_wotsa_x4_g8_600k_reds.py \ https://download.openmmlab.com/mmediting/restorers/edvr/edvrm_wotsa_x4_8x4_600k_reds_20200522-0570e567.pth \ dataset/test.mp4
./output
--window-size=5 \
Reproduces the problem - error message
Traceback (most recent call last):
File "demo/restoration_video_demo.py", line 90, in
main()
File "demo/restoration_video_demo.py", line 68, in main
args.filename_tmpl, args.max_seq_len)
File "/home/xxx/Project/mmediting/mmedit/apis/restoration_video_inference.py", line 102, in restoration_video_inference
data = test_pipeline(data)
File "/home/xxx/Project/mmediting/mmedit/datasets/pipelines/compose.py", line 42, in call
data = t(data)
File "/home/xxx/Project/mmediting/mmedit/datasets/pipelines/formating.py", line 135, in call
results[key] = torch.stack(results[key], dim=0)
RuntimeError: [enforce fail at CPUAllocator.cpp:65] . DefaultCPUAllocator: can't allocate memory: you tried to allocate<PHONE_NUMBER> bytes. Error code 12 (Cannot allocate memory)
Additional information
I want to do super-resolution processing on a video (*.mp4), but when I run it with the command line shown above, an error occurs. I didn't modify any code, and use the provided demo model for testing
PS：the *.mp4 file is a 8.08 MB, 512x288, Duration 03:24 video,
However, it can be used correctly when executing on the case that input files are images (*.png).
python demo/restoration_video_demo.py ./configs/restorers/edvr/edvrm_wotsa_x4_g8_600k_reds.py https://download.openmmlab.com/mmediting/restorers/edvr/edvrm_wotsa_x4_8x4_600k_reds_20200522-0570e567.pth demo_files/lq_sequences/calendar ./output --window-size=5 load checkpoint from http path: https://download.openmmlab.com/mmediting/restorers/edvr/edvrm_wotsa_x4_8x4_600k_reds_20200522-0570e567.pth

Can someone help me solve this problem？

Since the input video memory is too large, can not allocate enough memory for it. I think this should be pointed out in the documentation.


[REQ] Quick Image Share

I use this to browse memes, comic strips etc. Was wondering if there could be a feature to just have either a share option when viewing a full picture or like hold down for a send prompt. Just some kind of fast share image.
Good example could be if you've ever seen Sync for Reddit. That has a great quick sharing image feature.

and videos and links, not just images.


How to use CDN？

This theme doesn't support CDN natively, but it should not be very hard to implement this yourself. There are a couple of plugins available that could help you, for example hexo-cdnify.


Make available with pip

The database  should be available on pip for easy addition to one’s Project, makes it easier for users.
Thanks as you look into it.

Noted!
It will be made available on pip soon, need to fix some core issues first with the package. Can't let users be running buggy code


Make dark tables readable for print

Fixes #25453.
The text of dark tables/theads is white and invisible if the page is printed (some browsers don't print backgrounds). Wrapping the theming for dark tables in a @media not print media query will disable theming for dark tables for print.
Also I removed the lines which forced the tds and ths to have a white background. If a table/row/cell has an accented background, you would probably also want this when you print the page (if the browser supports coloured backgrounds).

I think I'd rather see us override these styles in our _print.scss file. Thoughts?


Add terminal creation target setting

This PR fixes #126249

still testing while I work on reconnect in order to catch bugs


Auto backup doesn't stage files?

Describe the bug
I have auto backup after stopping file edits enabled, and set the vault backup interval to 1 minute.
But still it doesn't commit and push the changes. Is it because the files are not staged automatically?
Relevant errors (if available) from notifications or console (CTRL+SHIFT+I)
No response
Steps to reproduce

Expected Behavior
No response
Addition context
No response
Operating system
macOS
Installation Method
None
Plugin version
latest

Hi, I'm experiencing the same, but only on Linux. On Windows it works fine. How did you solve it?


rubygem-nokogiri is not available in any RHEL7 repos

I am having trouble installing Nokogiri on a RHEL 7 server because it is only available from an older RHEL 6 repo 'rhel-6-server-sam-rpms'
I installed it by downloading it from that repo and then did a yum localinstall, but obviously that is not ideal.

@GeraldScott Did you see any error message when you failed to install on RHEL 7? If so, would you paste the error here?

yum -y install opennebula-server opennebula-sunstone opennebula-ruby opennebula-gate opennebula-flow
Loaded plugins: langpacks, product-id, search-disabled-repos, subscription-manager
Resolving Dependencies
--> Running transaction check
---> Package opennebula-flow.x86_64 0:5.4.1-1 will be installed
---> Package opennebula-gate.x86_64 0:5.4.1-1 will be installed
---> Package opennebula-ruby.x86_64 0:5.4.1-1 will be installed
--> Processing Dependency: rubygem-nokogiri for package: opennebula-ruby-5.4.1-1.x86_64
---> Package opennebula-server.x86_64 0:5.4.1-1 will be installed
--> Processing Dependency: opennebula = 5.4.1 for package: opennebula-server-5.4.1-1.x86_64
---> Package opennebula-sunstone.x86_64 0:5.4.1-1 will be installed
--> Running transaction check
---> Package opennebula.x86_64 0:5.4.1-1 will be installed
---> Package opennebula-ruby.x86_64 0:5.4.1-1 will be installed
--> Processing Dependency: rubygem-nokogiri for package: opennebula-ruby-5.4.1-1.x86_64
--> Finished Dependency Resolution
Error: Package: opennebula-ruby-5.4.1-1.x86_64 (opennebula)
Requires: rubygem-nokogiri

yum can be configured to try to resolve such errors by temporarily enabling
disabled repos and searching for missing dependencies.
To enable this functionality please set 'notify_only=0' in /etc/yum/pluginconf.d/search-disabled-repos.conf

Error: Package: opennebula-ruby-5.4.1-1.x86_64 (opennebula)
Requires: rubygem-nokogiri
You could try using --skip-broken to work around the problem
You could try running: rpm -Va --nofiles --nodigest

yum repolist
Loaded plugins: langpacks, product-id, search-disabled-repos, subscription-manager
repo id                                                                              repo name                                                                                status
Webmin                                                                               Webmin Distribution Neutral                                                                                279
cf-me-5.8-for-rhel-7-rpms/x86_64                                                     Red Hat CloudForms Management Engine 5.8 (RPMs)                                                                            114
opennebula                                                                           opennebula                                                                                11
pgdg96/7Server/x86_64                                                                PostgreSQL 9.6 7Server - x86_64                                                                                464
rhel-7-server-extras-rpms/x86_64                                                     Red Hat Enterprise Linux 7 Server - Extras (RPMs)                                                                          710
rhel-7-server-optional-rpms/7Server/x86_64                                           Red Hat Enterprise Linux 7 Server - Optional (RPMs)                                                                     13,030
rhel-7-server-rpms/7Server/x86_64                                                    Red Hat Enterprise Linux 7 Server (RPMs)                                                                                17,720
rhel-7-server-supplementary-rpms/7Server/x86_64                                      Red Hat Enterprise Linux 7 Server - Supplementary (RPMs)                                                                   238
rhel-server-rhscl-7-eus-rpms/7Server/x86_64                                          Red Hat Software Collections RPMs for Red Hat Enterprise Linux 7 RHEL 7 Server EUS                                       7,117
repolist: 39,683

@GeraldScott I see rh-ruby23-rubygem-nokogiri package in cf-me-5.8-for-rhel-7-rpms  repo. This repo is listed in your repolist. So, I believe you can use this version.
I installed rh-ruby23 and rh-ruby23-rubygem-nologiri on RHEL 7.4 and confirmed Nokogiri worked. The hard part would be Nokogiri is in a dependency, which sees another version.

Thanks @yokolet , yes, I tried that, but it still failed. I also tried installing the gem nokogiri manually.
Installing the RHEL 6 version from the 'rhel-6-server-sam-rpms' repo works, so I guess the solution is to get rubygem-nokogiri into the rhel-7-server-optional-rpms repo (the equivalent of EPEL).

@GeraldScott I got this info from my co-worker. The rubygem-nokogiri is already in EPEL repo. It installs ruby 2.0.0 and nokogiri 1.6.1, older versions, but you can install the rubygem-nokogiri package on RHEL 7.
// enable EPEL repo
$ sudo yum install http://dl.fedoraproject.org/pub/epel/7/x86_64/Packages/e/epel-release-7-11.noarch.rpm

// temporary disable it
$ sudo sed -i -e "s/^enabled=1/enabled=0/" /etc/yum.repos.d/epel.repo

// install nokogiri
$ sudo yum -y --enablerepo=epel install rubygem-nokogiri

  ...
Installed:
  rubygem-nokogiri.x86_64 0:1.6.1-1.el7.1
  ...

$ rpm -qa |grep nokogiri
rubygem-nokogiri-1.6.1-1.el7.1.x86_64

$ cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.4 (Maipo)

$ /usr/bin/irb
irb(main):001:0> require 'nokogiri'
=> true
irb(main):002:0> Nokogiri
=> Nokogiri

Does above work for you?

It worked perfectly and is a more elegant solution. Thank you @yokolet.
Who is responsible to get rubygem-nokogiri into the RHEL equivalent of the EPEL repo instead of using the Fedora repo? Will you raise an issue with Redhat?

@GeraldScott That sounds nice.
I'm not sure how the package gets in. I suspect even @flavorjones doesn't know. I'll research a bit. If I get some, I'll post an update.
If you have a valid RHEL subscription, you'd better to file a ticket at the Red Hat customer portal. It may work better.

Yes, I have a RHEL Developer's Subscription, so I will file a ticket with them..
Thanks again for your assistance!

@GeraldScott thanks for following up.


Make bevy_app and reflect opt-out for bevy_hierarchy.

Objective
Bevy_hierarchy is very useful for ECS only usages of bevy_ecs, but it currently pulls in bevy_reflect, bevy_app and bevy_core with no way to opt out.
Solution
This PR provides features bevy_app and reflect that are enabled by default. If disabled, they should remove these dependencies from bevy_hierarchy.

Changelog
Added features bevy_app and reflect to bevy_hierarchy.

It's complaining about the formatting of your toml files: you'll have to clean that up before we can merge :)


Use mimic-response

Resolves #1

Coverage remained the same at 100.0% when pulling c51e5f8e5ebf837dce809d94c399af859da24d6e on use-mimic-response into 39d82f7c8c56ea524b01d9c4ca985ae482f174b7 on master.


schematic(add): does not respect workspace's defaultProject

Reproduction
Steps to reproduce:

ng new tmp and respond to prompts
cd tmp/
ng g app web-app and respond to prompts
Edit workspace (angular.json) to change "defaultProject": "tmp" to "defaultProject": "web-app"
ng add @angular/material and respond to prompts

Expected Behavior
The ng add docs state

Adds the npm package for a published library to your workspace, and configures your default app project to use that library,

The default app project is the value of defaultProject in angular.json.

Thus it is expected to add Angular Material to the default project, web-app.
Actual Behavior
Skipping installation: Package already installed
? Choose a prebuilt theme name, or "custom" for a custom theme: Custom
? Set up HammerJS for gesture recognition? Yes
? Set up browser animations for Angular Material? Yes
UPDATE package.json (1473 bytes)
UPDATE src/main.ts (391 bytes)
UPDATE src/app/app.module.ts (423 bytes)
UPDATE src/styles.scss (1418 bytes)
UPDATE src/index.html (484 bytes)

As you can see, it configured the initial app project in the workspace ("tmp") for Angular Material instead of the default project ("web-app").
Environment
Angular CLI: 8.2.1
Node: 10.15.3
OS: darwin x64
Angular: 8.2.1
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.802.1
@angular-devkit/build-angular      0.802.1
@angular-devkit/build-ng-packagr   0.802.1
@angular-devkit/build-optimizer    0.802.1
@angular-devkit/build-webpack      0.802.1
@angular-devkit/core               8.2.1
@angular-devkit/schematics         8.2.1
@angular/cdk                       8.1.2
@angular/material                  8.1.2
@ngtools/webpack                   8.2.1
@schematics/angular                8.2.1
@schematics/update                 0.802.1
ng-packagr                         5.4.3
rxjs                               6.4.0
typescript                         3.5.3
webpack                            4.38.0

Ran into this while reviewing PR https://github.com/angular/components/pull/16654.

@alan-agius4 thank you for the clarification!

@jbogarthyde the following, from the ng add docs needs to be changed

Adds the npm package for a published library to your workspace, and configures your default app project to use that library,

to something like:

Adds the npm package for a published library to your workspace, and configures the project related to your current directory, or your default project, to use that library,


How to find position (x,y,z) and orientation (Euler angles) of the measurement frame for D435i ?

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Platform
PC

Language
MATLAB, LabView, C#, C++ or C

Segment
Robot

Issue Description
Hello all,
I apologise in advance if my problem is already covered in another issue. I have not been able to find a straightforward solution so far.
I need to manipulate a D435i camera through a robot to reconstruct the geometry of an object, by bringing the device to different locations around the object. I have been able to develop the software required to acquire a point cloud from D435i at every pose. However I need an accurate measure of the position of the device reference system (origin coordinates and Euler angles), in the robot absolute reference system, in order to translate and rotate each acquired point cloud. In other words, I need to find the fixed offset and rotation between the robot flange and the device reference system. I assume this should be done using a predefined object (e.g. a chessboard pattern in a fixed position) and a calibration routine to find the camera extrinsic parameters (??). Can anyone point me towards a documented well documented algorithm applicable to D435i?
Thanks,
Carmelo

Hi @CarMineo   It sounds as thouh you already know how to position and rotate the individual clouds to align them, using an affie transform.  Instead, you want to be able to calculate the transform between the camera device and the flange.  Is this correct please?
If you do need details of how to perform an affine transform, an example of one in librealsense is the instruction rs2_transform_point_to_point
https://github.com/IntelRealSense/librealsense/issues/5583
In regard to the flange: this reminds me of a tool that a RealSense team member wrote to work out the extrinsic calibration between a 400 Series camera and a T265 Tracking Camera on a custom mount.  I wonder if it may be applicable to this situation if the flange represented the position on the mount where the T265 would have been.
https://github.com/IntelRealSense/librealsense/pull/4355
The use of gimballing with RealSense calibration for the 400 Series cameras was also recently discussed.
https://support.intelrealsense.com/hc/en-us/community/posts/360052504573-Gimbal-for-Realsense-cameras

Hello MartyG,
thanks for your reply.
Yes, I would like to use an automatic procedure to calculate the transform between the reference system of the camera, centred at the depth measurement origin, and the reference system of my robot flange. This transform is also known as "tool parameters" for the robot. I could estimate the tool parameters using the geometry CAD information of the D435i and the CAD model of the support I designed to mount the camera to the robot. However that would not be accurate, because the support and the real dimensions of the camera are affected by the manufacturing and assembly tolerances. What I am after is a procedure that enables me to calibrate the position of the camera measurement origin and the orientation of the camera, when it is mounted onto the robot flange by mean of the support I am using.
I am going to see the links you shared.
Is there anything else that springs to your mind?
Many thanks,
Carmelo

I carefully considered the information that you kindly provided.  This is not one of my specialist areas of knowledge, but I wonder if what you are describing is hand-eye calibration, which can be used to get the transformation between the end-effector of a robot arm and the camera.
If that is the case, the links below have information about hand-eye calibration in regard to RealSense:
https://github.com/IntelRealSense/librealsense/issues/3569#issuecomment-475621533
https://support.intelrealsense.com/hc/en-us/community/posts/360051325334/comments/360013640454

Hi @CarMineo  Do you require further assistance with this case, please?  Thanks!

Hello Marty,
thank you for the additional information. I am going through implementing the procedure for the hand-eye calibration, as you suggested. It should work to meet my requirements.
Best regards,
Carmelo

Okay, thanks very much for the update @CarMineo

Hi @CarMineo   Do you require further assistance with this case, please?   Thanks!

Hello @MartyG-RealSense,
thanks for checking how this is going. I have found a quite straightforward procedure to calibrate the centre of the RGB sensor of D435i. This is based on the MATLAB Camera Calibrator app (https://it.mathworks.com/help/vision/ref/cameracalibrator-app.html). I manipulate the D435i with a robotic arm to take 10 pictures of a chessboard pattern (https://github.com/opencv/opencv/blob/master/doc/pattern.png) from different positions. I record the colour frame and the robot positional feedback (the position of the robot flange on which the D435i is mounted) at every pose. The MATLAB Camera Calibrator allows me to compute the camera intrinsics, extrinsics, and lens distortion parameters. Thus it provides also the coordinates of the location where each colour frame was acquired. Then I can compare the transform between the locations of the RGB sensor and the robot flange, which gives the robot tool parameters I need.
The only think it is still not clear to me is if the origin of the depth measured by the D435i coincides with the centre of the RGB sensor. If that is not the case, what is the link (the transform or the offset) between the centre of the RGB sensor and the depth origin?
Best regards,
Carmelo

Hi @CarMineo  Great news that you were successful.  Thanks so much for sharing the details of your solution for the benefit of RealSense community members.    :)
The origin point of the 400 Series camera is always the center of the left IR sensor.  The link below explains the coordinate system relative to this origin point.
https://github.com/IntelRealSense/librealsense/issues/7279#issuecomment-689031488

Hi @CarMineo  Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


bump istio version to 1.16.1

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets

License
Apache 2.0

What's in this PR?
Istio 1.16.1 update

Probably the supported Kubernetes version should be updated in the readme and helm chart.

Could you please check this diff regarding the manifests folder: https://github.com/istio/istio/compare/1.15.3...1.16.1? I think the Istio CRDs were not updated accordingly in this PR. Also, there might be some other helm chart changes that might worth incorporating into our operator. Could you pls check those as well to see what's worth porting here?


How do I install the latest version of Gridsome

Description
I installed gridsome using npm on my windows PC with npm install --global @gridsome/cli, but when I run gridsome --version I got @gridsome/cli v0.3.4
Steps to reproduce
Run npm install --global @gridsome/cli on the terminal
Expected result
I should see the latest version of gridsome<EMAIL_ADDRESS>Actual result
I got an old version @gridsome/cli v0.3.4
Environment
System:
    OS: Windows 10 10.0.19042
    CPU: (4) x64 Intel(R) Core(TM) i7-7600U CPU @ 2.80GHz
  Binaries:
    Node: 12.22.1 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.12 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 94.0.4606.71
    Edge: Spartan (44.19041.1023.0), Chromium (94.0.992.31)

The gridsome CLI (@gridsome/cli) is at version 0.3.4.
The gridsome library (gridsome) is at version 0.7.23.
If you run gridsome --version outside of a gridsome project (with CLI installed globally) you would see:
@gridsome/cli v0.3.4
If you run gridsome --version inside of a gridsome project you would see both:
@gridsome/cli v0.3.4
gridsome v0.7.23
Hope this helps.


logger.wran is not a function

@baixiaodu now you can upgrade your weex-debugger to latest v1.0.2 version to fix this issue.


comb filter M = 3，and  PITCH_MAX_PERIOD 768, how to meet the 40ms look-ahead requirement?

In paper:
"To achieve 40 ms look-ahead including the 10-ms overlap, we use M = 3"
40 ms look ahead, it means have only 30ms data to shift for comb filter in time domain.
30ms data = 3 * 480 = 1440
And M = 3,  PITCH_MAX_PERIOD 768, the maxim shift will be 3 * 768 = 2340
Now in code,
FRAME_LOOKAHEAD is set to 5,  5 * 480 > 3 * 768, but it will lead to 60ms look ahead.
In paper:"Low-Complexity, Real-Time Joint Neural Echo Control and Speech Enhancement Based On PercepNet",
M is changed to 2. But even if M is 2,  2 * 768 > 3 * 480.
So what is the correct parameter M, PITCH_MAX_PERIOD to meet 40ms look ahead requirement?

Frame Lookahead is needed for internal reason have to care about overflow in pitch filter buffer
I draw scratch about pitch buffer in case FRAME LOOKAHEAD is 3(current FRAME_LOOKAHEAD is 5)
frame_size is 480 and window size is 960.
assume max pitch period is 768 according to rapt module from Rnnoise.
and we need 3 delayed stft window for comb filtering(COMB_M=3) which is yellow windows describe below
7683+960-960 > 4803 so it occur overflow

we can meet 40ms look-ahead if we can change PITCH_MAX_PERIOD 768 -> 512  and FRAME_LOOKAHEAD 5->3.


[QUESTION] How can i bind dynamic row within specific column

Versions

PHP version: 7.1
Laravel version: 5.6
Package version: ^3.1

Description
Here i attached screenshot: I want to inset multiple comment for one user same as screenshot

Please fill in the issue template.

@rijan0101 show us the code you already have.

@rijan0101 hello sir...
i think i also have same issue like you...
if you dont mind, can you share to me the code, if you has solve the issue ?

Closing due to inactivity.
@mandrysn If you need support, please create a new ticket with the issue template filled. The issue template is required for us to be able to help you and is automatically available for you when opening a new issue, please do not remove the contents of the issue template.


.refresh() doesnt work after i used filter on ng-repeat

When i try to filter ng-repeat like this
<select class="sel1" ><option value="1">option1</option>

Im using on select change like this

var wall = new freewall($(".results"));
wall.reset({
     selector: '.item',
     draggable: true,
     animate: true,
     cellW: 185,
     cellH: 180,
     fixSize: 0,
     gutterY: 7,
     onResize: function () {
                wall.fitWidth();
  }
});

$(".sel1").change(function(){
      wall.refresh();
});

It doesn't works.
Items filtered stay on his position with opacity 0 but not reordered

:+1:
Got the same issue. When testing in Firefox browser console I can create a freewall instance:
var wall = new freewall("#metro-ui");

But calling refresh() method return:
wall.refresh()
TypeError: can't convert null to object

The method exist in the wall object with following description:
refresh: n/<.refresh()
  arguments: null
  caller: null
  length: 0
  name: ""

Here is how I solved it:
The refresh() method should be called by an angular event. Here is my ng-repeat block:

    …

First, add a ng-change attribute to call the refresh() in the element that update your filter.
<input ng-model="search" ng-change="ctrl.wall.refresh()" type="text" >

Then your controller should instanciate the wall and expose it to the view:
function dashboardController(apiService) {
    var ctrl = this;
    ctrl.wall = null;  // expose wall to the view
    …
    initFluidTiling()

    function initFluidTiling() {
        ctrl.wall = new freewall("#metro-ui");  // instanciation
        ctrl.wall.reset({ … });
    }
}

})();


revert_to_snapshot(vm_name, snapshot_name) function should be implemented

Hi There
revert_to_snapshot(vm_name, snapshot_name) function should be implemented
something like this:
def revert_to_snapshot(self, vm_name, snapshot_name):

    virtual_machine = self.get_virtual_machine(self.vm_name)
    snapshots = virtual_machine.get_snapshots()
    for snapshot in snapshots:
        if snapshot.name == self.snapshot_name:
            snapshot.revert()

BR,
George

hey buddy, it's already implemented
https://github.com/jossef/vmwc/blob/master/vmwc/__init__.py#L651
Thanks!

Looks Great. @GeorgeFischhof Could you make a PR about this suggestion?

Thank you. PR: on my workplace I will tell this to my colleagues ;-)


add unit test ApiDocControllerTest & OperationRecordLogControllerTest

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4121 (c37758c) into master (0453b48) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4121      +/-   ##
============================================
+ Coverage     70.23%   70.34%   +0.11%
- Complexity     6892     6910      +18
============================================
  Files           944      944
  Lines         25913    25913
  Branches       2329     2327       -2
============================================
+ Hits          18199    18228      +29
+ Misses         6296     6259      -37
- Partials       1418     1426       +8

Impacted Files
Coverage Δ

...ava/org/apache/shenyu/admin/utils/SessionUtil.java
78.94% <0.00%> (-15.79%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
68.96% <0.00%> (-13.80%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...enyu/admin/service/impl/PermissionServiceImpl.java
34.61% <0.00%> (-6.42%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
66.66% <0.00%> (-5.56%)
:arrow_down:

...va/org/apache/shenyu/common/timer/TimingWheel.java
85.71% <0.00%> (-4.77%)
:arrow_down:

... and 41 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update stimulus

Update to Stimulus 3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
273

Relevant Lines:
273

💛  - Coveralls


Smartling ePROMs upload

Move Smartling project_id to each persistence file
Upload translations for each project

Looks like we may need an API token per project in Smartling- waiting to confirm with Tyler