Api - DefinitionUri - Bucket in a different region

Hello,
If I try to deploy a stack with a s3 bucket in another region than the one in which I deploy the stack then I get this error:
"The bucket you are attempting to access must be addressed using the specified endpoint. Please send all future requests to this endpoint."
I guess the limitation is in the URI style s3://bucket/key in which we can't precise the region endpoint.

Can you provide the relevant snip-it of what you are trying to deploy.
I can't seem to find the documentation for it (found this forum post though) but the S3 bucket will need to be in the same region as the Lambda Function.

It's not about the codeUri attribute for a lambda function (because as you stated, it's an AWS limitation, the bucket has to be in the same region as the function) but the DefinitionUri which aim to a S3 location containing the Swagger document for an API.
I've fixed this by using the solution provided by dinvlad here: https://github.com/awslabs/serverless-application-model/issues/8#issuecomment-290212883
It fixed my bucket location problem and allowed me to use the Fn::Sub in my Swagger file for the function arn.
However, I think it's still an issue not be able to have the Swagger file in a different region than the one in which we deploy the stack?

Why should swagger be any different and have a different limitation than your Lambda function code? They are both stored in S3. To my knowledge, this is expected behavior.
I don't have a ton of context on why this was done this way. I think this is done for some kind of safety and reduce blast radius, but let me talk with some folks and get back to you.
From digging through some of the SAM code, we don't look to be doing any validation on the bucket in terms of what region it is in. So CloudFormation must be complaining. This would mean that AWS::Gateway::RestApi CloudFormation resource is not allowing cross-region, not AWS::Serverless::Api.
Have you looked into S3 cross-region replication? Wouldn't this give you what you want but still allow the content of the bucket be in the other regions you need?

Hello,
Sorry for late answer.
It's just I can't find documentation on AWS stipulating the Swagger definition must in the same region as the API Gateway.
But as you said, it's a limitation from AWS::Gateway::RestApi Cloudformation resource.
Thank you!
Alexandre


feat: support container restart

Removed yarn cache.
Will check how many users has been registered and send hello message if === 0.

We assume the database has been reseted whenever the user count is null.

Removed default hello message
Updated dependencies

🙏🙏🙏🙏🙏🙏🙏


wrong path in '[OK] Cleanup custom locations' :

[OK] Cleanup custom locations:
/Users/.choosenim/toolchains/nim-0.19.0/lib/system.nim: No such file or directory
/Users/.choosenim/toolchains/nim-0.19.0/lib/system.nim: No such file or directory
/Users/.choosenim/toolchains/nim-0.19.0/lib/system.nim: No such file or directory
should be:
/Users/username/.choosenim/toolchains/nim-0.19.0/lib/system.nim
etc

I have no idea where this comes from. I would need some help here as I'm new to Nim.
By the way, the "Cleanup custom locations" is a simple unit test on the proc reset_coverage. It creates folders and a .info file then call reset_coverage with some parameters.

I have no idea where this comes from. I would need some help here as I'm new to Nim.

not sure either on quick inspection; but using doAssert instead of silently ignoring error codes via discard (see https://github.com/samuelroy/coco/issues/11)  would give a stacktrace and should reveal where it gets generated

(btw, hope you don't mind the issues I'm reporting; I think it's an awesome tool, I'm only trying to help by pointing out usability issues)

Not at all, thanks for your feedback, it's really appreciated. I'm exploring Nim through this project and I want to see if it's worth investing my time and my team in it.

I want to see if it's worth investing my time and my team in it.

I really hope it'll turn out positive ... happy to help, and the forum is usually a friendly and efficient place to ask questions (so is nim irc/gitter channel)


Question about patch-wise correlation map (sigma) in "Shadow-Interaction Attetnion"

Nice work to solve shadow degradation problem
I look around the code to understand how patch-wise correlation map $\Sigma$ is obtained
As far as i investigated, in the SIA($\boldsymbol(X), \Sigma$) is obtained by attention operation * [ $\sigma$ $\Sigma$ + (1- $\sigma$) 1] (eq 8 in the paper)
https://github.com/GuoLanqing/ShadowFormer/blob/a55cf042a1f9371d0e56a1299c245604d6663b6f/model.py#L426C24-L426C33
The code reference above (in the class WindowAttention) is about operating attention operation. However, i cannot find where
[ $\sigma$ $\Sigma$ + (1- $\sigma$) 1]  is operated and $\Sigma$ is obtained.
Can you explain more detail about patch-wise correlation map and where the rest of the SIA operation ( [ $\sigma$ $\Sigma$ + (1- $\sigma$) 1] ) is done?
Thank you

You can refer to Lines 398-405 in model.py.

Thank you,
To check my understanding is right, i have some question.

variable 'mm' stands for $\Sigma^{i,j}$ (in eq (7)) ?
The XOR operations is done by torch.where operation ?
$\sigma$ which is the weight of shadodw-shadow and nonshadow-nonshadow pairs  is 0.8 (in eq (8))??

def forward(self, x, xm, attn_kv=None, mask=None):
        B_, N, C = x.shape
        # x = self.se_layer(x)
        one = torch.ones_like(xm)
        zero = torch.zeros_like(xm)
        xm = torch.where(xm < 0.1, one, one*2)
        mm = xm @ xm.transpose(-2, -1)
        one = torch.ones_like(mm)
        mm = torch.where(mm==2, one, one*0.2)
        mm = torch.unsqueeze(mm, dim=1)
        q, k, v = self.qkv(x, attn_kv)
        q = q * self.scale
        attn = (q @ k.transpose(-2, -1)) * mm

after xm = torch.where(xm < 0.1, one, one*2), the value in xm are 1 and 2, and after xm @ xm.transpose(-2, -1), you can hava a variable 'mm' which just has three value 1， 2， 4。 And you can see 1 * 1 = 1, 2 * 2 = 4, 1 * 2 = 2, this can be seen as XOR operation。

@BlackJoke76 Thank you.
I'm confused because the output of "mm" consists of 1 and 0.2, which is different from what i expect (1 and 0).
The value 0.2 comes from [ $\sigma$ $\Sigma$ + (1- $\sigma$) 1] where $\sigma$ is 0.8.
The operation of mm = torch.where(mm==2, one, one*0.2) is combination of XOR and [ $\sigma$ $\Sigma$ + (1- $\sigma$) 1] operations.
Thank you for helping me understand


Feature/location breadcrumbs

http://ci-deploys.cid-labs.com/cid-harvard/atlas-subnational-frontend/feature/location-breadcrumbs/111/

@katybeck I'm not sure where the breadcrumbs should be, could you take a look?

Surely! Taking a look this morning.

On hold, will add with nav redo


publications of private communications

in your definitions of harassment it is written, that "Publication of non-harassing private communication" is seen as harassment. Even though it is written that harassment is not limited to those definitions of harassment, i see a problem here: It implies that there are situations and circumstances that justify publications of private communications, meanwhile it is not said how the person who makes the communication public gets access to this communication.

Take a look at #84, this repository is currently being paused and rewritten.

i won't read all that stuff in #84, but i can wait until that rewrite is done

Thanks for your feedback. See #84.


⚠ Samsung has degraded performance

In 20892fe, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2634 ms

Resolved: Samsung performance has improved in 4035d7b.


use datafile for list of templates

Utilizes Panini by adding a yml file with the list of templates.  This
data file make its easier to update the list of templates on the home page.  It also
provides users with an example of how to use yml data files with Panini.

This is similar to #20, but adds a working example of how to use the data file


Task/add fields to db allow reupload

We should really ship this feature already :(
Please merge and deploy ASAP, fix the problems in my CR afterwards.


Add use_websocket

Highly inspired by use_websocket of yew-hooks
Content

[x] use_websocket
[x] use_websocket_with_options
[x] auto-reconnect in case of Close or Error events.
[x] Example
[x]  Book: New chapter websocket
[x] Bump default-struct-builder

Demo
use_websocket_demo.webm

@maccesch Preview of chapter use_websocket. All in 12eb953 , incl. changes to address your review suggestions.

Hehe the checks are failing because of my changes, not yours… let me fix real quick

Great work, thank you!

To be more in line with VueUse / SolidJS-USE the use_websocket function now lives in the root module and is in the "Network" book category. Just FYI.


Scripts signed with an external CA aren't automatically trusted

This could be the expected behavior, but if it is, it doesn't really make sense to me, so if someone could clarify, I'd really appreciate it.
Basically, I'm trying to make sure that scripts signed with my company's code signing certificate are trusted on remote systems that have their execution policy set to AllSigned. From what I've seen in my tests and looking at the code, PowerShell ultimately requires that the certificate be included in the Trusted Publishers store. Below is the code that seems to confirm it:
    // Checks that a publisher is trusted by the system or is one of
    // the signed product binaries
    private bool IsTrustedPublisher(Signature signature, string file)
    {
        // Get the thumbprint of the current signature
        X509Certificate2 signerCertificate = signature.SignerCertificate;
        string thumbprint = signerCertificate.Thumbprint;

        // See if it matches any in the list of trusted publishers
        X509Store trustedPublishers = new X509Store(StoreName.TrustedPublisher, StoreLocation.CurrentUser);
        trustedPublishers.Open(OpenFlags.ReadOnly);

        foreach (X509Certificate2 trustedCertificate in trustedPublishers.Certificates)
        {
            if (String.Equals(trustedCertificate.Thumbprint, thumbprint, StringComparison.OrdinalIgnoreCase))
                if (!IsUntrustedPublisher(signature, file)) return true;
        }

        return false;
    }

Is this the expected behavior? If so, then why shouldn't it check other certificates in the chain? What would be the purpose of purchasing a certificate if I would have to import the certificate on the target machines anyway?

Is this the expected behavior? Yes
why shouldn't it check other certificates in the chain? CAPI do. (You can enable CAPI event log and see there)
What would be the purpose of purchasing a certificate if I would have to import the certificate on the target machines anyway? By design 😊 Do you want to make a request to the Internet each time you start the script?


Delete guides error

Pull request guidance
Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.
We also need your help in order to process your request as soon as possible:

We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

#sign-off

#sign-off

PR#5210 is created in private repo for this article.

Hi @v-wendysmith, this article is published via private repo. The link will be: https://learn.microsoft.com/en-us/troubleshoot/dynamics-365/guides/application-functionality/cant-delete-guide.


Syntax error or access violation

Expected behavior:
Check the DB-COLLATION async after set the credentials
Actual behavior:
Error on the last step of installation if the DB-COLLATION is not utf8mb4_unicode_ci:
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '1.0.0'' COLLATE utf8mb4_unicode_ci' at line 1 (SQL: ALTER TABLE ti_extensions CHANGE version version VARCHAR(32) DEFAULT ''1.0.0'' COLLATE utf8mb4_unicode_ci)
Reproduce steps:
Set the DB-COLLATION on other than utf8mb4_unicode_ci
Version:
Beta 3
EDIT
Now i set the DB-Collation to utf8mb4_unicode_ci, but the installation breaks on the last Step.. with the error above

MariaDB 10.2
PHP-Version 7.2

#224 related


Create Comparison with Other Software page

Page added to Getting Started section to serve as an introduction to the software, especially to new users.
I didn't add a Performance section because I thought without proper profiling it could be subjective to a certain degree, even if the results are evident by simple observation. I've shared my thoughts here: https://www.reddit.com/r/playnite/comments/1hicu3h/working_on_a_comparison_table_for_game_library/m32e73z/


Indent multi line error message on state contract error.

Add multiline error so we could check text wrap indent was applied to the error message.
Dictionary field validator together with List would be great targets to test this.
Context validation returns just a string.
We should remove tests with fancy input data and just test for failure with single line and multiline error message.
Dictionary and List validators are necessary to test reference transparency only.

Superseded by #627


Fix trans showing  element

Nice, thanks! I appreciate the practical use of the  element, good stuff!


Deprecate ActiveAdmin Theme

When I originally wrote ExAdmin, I used the layout from the Rails ActiveAdmin project to avoid having to write my own UX. Later, when I added the AdminLTE2 theme, I kept the ActiveAdmin theme in for backwards compatibility.
I don't use the ActiveAdmin theme in any of my projects now. Also, supporting both is a royal pain. I would like to deprecate the ActiveAdmin theme. It has fallen behind, isn't bootstrap based, and a pain to maintain.
Please add a reaction to this issue if you agree/disagee. I would like to get community feedback before proceeding.

Agree!

It would be a pity to deprecate it since it provides a way for migration without UX shocks to users of existing interfaces written in ActiveAdmin. Its actually quite a strong selling point IMO ...

@os6sense Thanks for your input. I added the issue to get this type of feedback. Can you add a thumbs down reaction to the issue description. Thanks...


as require not working

const queryString = require('query-string')
(error: Uncaught TypeError: queryString.parse is not a function )
now if you use it... it's broken the application.
It's working fine as import/

https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c


Add Actix example

What is the motivation?
Requested on Discord
What does this change do?
Adds an example of how to use the static client with Actix Web.
What is your testing strategy?
Tested the endpoints manually.
Is this related to any issues?
No

don't we need something lika a connection pool?

don't we need something lika a connection pool?

No. As you can see in the README, one of the features of the connections in this library is

Clonable connections with auto-reconnect capabilities, no need for a connection pool

The library is specifically designed to make connection pools unnecessary.


Handle GameLoader loading failure

From @demiurghg on June 5, 2016 9:6
An error in GameLoader may occur while loading.
GameClient should handle such case, disconnect and show appropriate message.
Possible solutions:

Add IsFailed and ErrorMessage in GameLoader.
Use async/await?

Workaround:

Catch exception in loading task.
Send disconnect command (bad: server is still running).

Copied from original issue: demiurghg/FusionEngine#69

250dfc532892a25dca9366b8a198d7d084e26ebf
Precacher failure not fatal by definition.


Email Verification

Hey,
The bot is working fine :) However, you can not do anything with the tokens, because you need an email verification directly... Would be great if you could add that

Hi, Thank you for the suggestion but unfourtanetly discord has extremely advanced bot detection/prevention features. So even if I add Email Verification the tokens will still be flagged/locked/disabled this repo was made for offline member boosting since that's the only thing that they don't monitor as of now.


SPARK-4044 [CORE] Thriftserver fails to start when JAVA_HOME points to JRE instead of JDK

Don't use JAR_CMD unless present in archive check. Add datanucleus always if present, to avoid needing a check involving JAR_CMD.
Follow up to https://github.com/apache/spark/pull/4873 for branch 1.3.

Test build #28474 has started for   PR 4981 at commit 3aafc76.

This patch merges cleanly.

Test build #28474 has finished for   PR 4981 at commit 3aafc76.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28474/
Test PASSed.

LGTM.


Fix errors in Kuzu example notebook for latest version

Description
Updates the documentation notebook example to clear errors after thew version bump.
Fixes # (issue)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[x] Yes
[ ] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No

Type of Change
Fix errors in notebook and ensure it runs end-to-end.

[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[x] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@logan-markewich fair point :). I removed it and it's a lot less verbose now.


[mlvm] adding cudnn to mlvm documentation

Adding cuDNN to mlvm documentation, updating libraries.
Running into the same HDFS issue as in https://github.com/GoogleCloudDataproc/initialization-actions/pull/858#issuecomment-768583503

/gcbrun

/gcbrun

@mengdong do you know when RAPIDS 0.18 will be released? There's currently a dependency conflict with cuSpatial 0.17 and the libraries on our base image. The conflict disappeared when I tried the nightly build of 0.18

Please, rebase this branch on master HEAD

@medb rebased


Code blocks, interfaces

Consider following markdown note content:
A line containing `ISomething` interface name.

```csharp
interface ISomething : IBase
{
  // code containing interface names
}
```

Plugin tries to autocorrect them, and this is not something I need.
Adding any possible names to exceptions is not viable solution.
Perhaps most reasonable solution here - (an option for) not autocorrecting inside code blocks, both inline and multiline.
Other solution that might work in different situations, including when I'm not opening a code block before typing in the interface name - ability to define exceptions as regular expressions.

Being able to switch off the correction in code blocks is a good idea. I'll take care of that soon

i disabled auto correction in multiline code blocks and inline code blocks now

Thank you


Remove listeners when closing websocket

Thank's @marcus-j-davies for the suggestion. I've noticed "extra messages" at times with web sockets. Suggestion is to be sure listeners are destroyed properly before opening a new connection.
Please review, @Shaquu

@crxporter Can you also bump version in package.json to 0.5.2


Modal window background color

hey, how I can change the background color for the modal window?

As in change the background of the snipe menu itself ?

yes, I was able to do it with:
vim.api.nvim_set_hl(0, "NormalFloat", { fg = "none", bg = "none" })
vim.api.nvim_set_hl(0, "FloatBorder", { fg = "none", bg = "none" })


A way to represent new lines

I don't know if I should open a new issue, but enter wouldn't add a slash or something like because when you get a text instead of a quote it's get all mess up
https://9o8x7rwor4.codesandbox.io/
Originally posted by @evansantos in https://github.com/gabrielfurini/morsee/issues/1#issuecomment-450444746

@gabrielfurini thank you for your answer, later I wrote this comment I was figuring out code morse wasn't supposed to be more than a few words, but thanks anyway I'll add this to my UI


Recheck sdcard expansion of fresh install (not recovery)

see https://github.com/rootzoll/raspiblitz/issues/214#issuecomment-723595602

FYI, I had this same problem on a fresh install of v1.6.1. I solved it by resizing the partition with gnome-disks. It's nice to know there is a script I could have run.

I think I found why its not happening on fresh install - not on final testing.
BTW with a new update this will autofix itself for older setups.

OK tested on fresh setup looks good. Closing issue for v1.6.2 release.


SVG Scaling

Issue Summary
There seem to be some problems in tex2svg() with the scaling option.
Steps to Reproduce:
MathJax.tex2svg('x = p', { scale: 1 })
MathJax.tex2svg('x = p', { scale: 2 })

These two lines produce SVG with exactly the same width/height and viewBox. Where is the scaling going?
Technical details:

MathJax Version: 3
Client OS: Windows 10
Browser: Chrome

Normally, MathJax's SVG output is places inside a <mjx-container> element, and that element has a scaling factor that is designed to match the font size in the math to the surrounding font.  The SVG (or CHTML) renderer needs to know that scaling factor in order to compensate for it when dimensions are given in absolute units (e.g., \hskip 1cm) or when switching back to native fonts for characters that aren't in the MathJax fonts (e.g., for \unicode{x0416}, or when mtextInheritFont is true, or for things like \style{font-family: Arial}{x}).  The scale parameter tells the renderer what factor was used to scale the math to match the outside fonts so that that can be compensated for in those situations.  It is not a scaling factor for the final result.
Note that SVG images are, by nature, scalable (SVG is Scalable Vector Graphics), so you can scale the SVG element after the fact.  For example, if you want it to be twice as big, add font-size: 200% to the style attribute of the surrounding <mjx-container> element.
If you are discarding the container, then you would have to scale the width, height and vertical-align values yourself by hand.  You could, of course, make your own wrapper for MathJax.tex2svg() that accepts a scaling factor and scales the width, height, and vertical-align of the result fo you.


[swift-inspect] Fix taskToThread dictionary on watchOS.

watchOS's 32-bit address types continue to cause trouble.
rdar://90489466

@swift-ci please smoke test and merge


created a component - icon after text download button

added a component : 'download button with icon'
stored in:  buttons/icon_button_secondary.html

Closing this PR as you are not assigned to it. As well we just need the main TML file. Not with the boiler plate of HTML. @yashsarnaik22


Upgrade mitol-django-authentication, common, pygithub

Pre-Flight checklist

[ ] Testing

[ ] Changes have been manually tested

What are the relevant tickets?
Part of #23
What's this PR do?
Upgrades some packages as a prerequisite to installing mitodl-django-mail. These changes should be simple upgrades without a change of functionality so it should hopefully be easy to spot and fix if this breaks something.
How should this be manually tested?
TBD

Codecov Report

Merging #373 (2911281) into master (2e2ee53) will increase coverage by 5.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
+ Coverage   89.02%   94.26%   +5.23%
==========================================
  Files         123       60      -63
  Lines        3883     1378    -2505
  Branches      596      317     -279
==========================================
- Hits         3457     1299    -2158
+ Misses        384       78     -306
+ Partials       42        1      -41

Impacted Files
Coverage Δ

main/s3_utils.py

main/celery.py

users/factories.py

websites/factories.py

content_sync/tasks.py

content_sync/decorators.py

websites/constants.py

content_sync/apps.py

main/__init__.py

websites/signals.py

... and 53 more

Continue to review full report at Codecov.

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


Wrong certificates in k8s cluster

After syncing today I get the following errors in kubelet logs:
E0515 10:42:54.160745    3462 reflector.go:130] Failed to list *api.Pod: Get https://<IP_ADDRESS>/api/v1beta3/pods?fieldSelector=spec.host%3De2e-test-filipg-master: x509: certificate is valid for <IP_ADDRESS>, not <IP_ADDRESS>
I see a similar error for (almost) all requests.
@lavalamp This could be related to #8071

@wojtek-t

@jayunit100

Did you kube-push? I admit that I didn't test that path but only the cluster create/delete flow....

As an aside, if we expect kube-push to always work correctly, then we need to add an e2e test for it to verify that it isn't breaking from PRs.

AFAIR first I only pushed, but then I did down & up.
Note that the certificate itself could be ok, it's just that we are using internal IP for connection, while certificate is generated for the external one. Has something changed with regard to this?

The certificate on the master is signed with $MASTER_NAME and has the external IP as a SAN (subject alternate name). Before this change we were using salt to construct a fqdn for the master name but the fqdn isn't included in the cert. So the kubelet will be connecting to something like https://kubernetes-master.c.project and the cert is signed for kubernetes-master.

I double checked where this happens, and it's only a problem in kubelet running on master machine.

I wonder if this was actually broken as a combination of my change and #8009. In that change it looks like we caused the master to no longer write any auth data for the kubelet running on the master node.
/cc @mbforbes

This looks like the same issue I saw in #8207?

I wonder why this happens only sometimes?

If it happens always, then me seeing it in #8207 was a red herring for the issue I was having.
Does this have any symptoms besides the errors in the kubelet log?

Is the kubelet on the master a first-order citizen?

I thought it wasn't actually managing any pods, and the only pods that ran on the master are from startup. Please correct me if I'm wrong.
If I'm not, why is it running there?
If I am, we should probably put the kubelet token on the master too.

@mforbes the kubelet on the master is responsible for setting up the (static) pods the master components run in

@bprashanth Please address @mbforbes instead.

I opened #8444 to (hopefully) address this. (Pending e2e runs.)

@roberthbailey's #6949 was supposed to fix this--anyone able to repro at head?

@roberthbailey The certificate problem is solved, but now I get the following errors:
E0520 00:50:40.178050    3473 kubelet.go:1523] error getting node: node e2e-test-filipg-master not found
E0520 00:50:40.191870    3473 kubelet.go:2097] Cannot get host IP: cannot get node: node e2e-test-filipg-master not found

Is that on the master? The kubelet on the master shouldn't be trying to talk to the apiserver any longer.

Is this still relevant/open?  It was marked P0 and there have been no updates in 5 days.

This should have been fixed by https://github.com/GoogleCloudPlatform/kubernetes/pull/8164

@fgrzadkowski is this still a problem or can it be closed?

Closing.  re-open if it pops up

Sorry for not responding. I missed this issue. I don't see this error anymore. The only error message is:
E0608 14:08:16.941655    5968 kubelet.go:2111] Cannot get host IP: cannot get node: node e2e-test-filipg-master not found


[ci] Test CI - remove deprecated topic selector

Do not merge - just test PR to verify Oracle CI flow from PR #4325

Oracle tests were started and even succeeded, so everything seems to work as expected - closing.


Add support for downloading a list of URLs in Skyplane for pangeo

Add a new command skyplane download-pangeo s3://... that downloads all pangeo files (8187 URLs, each is about 30 MB) to the S3 bucket.
Not fully tested yet and has some hard-coded statements.

Hey @phi-line, we are collaborating with some folks working with Fernando Perez to see if we can make Pangeo Forge (data loaders for the pangeo system) faster using Skyplane. To support that, we would need to support an HTTP object store connector (i.e. download a list of files via URLs). This PR is an initial exploration to get a benchmark of how much faster Skyplane would be.
I agree that if successful, we would extract this logic into an external repo. More broadly, I'd actually rather see support for this use case via the API where integrations would be upstreamed to Pangeo Forge. The dataplane components would stay in Skyplane (e.g. HTTPInferface). But given the absolutely massive stock of data in NOAA/NASA/ESA data sources, I would like to help academics access big datasets more easily.

@parasj What are your thoughts on making an examples file that could contain pangeo_util.py? This would keep the new code outside of the cli while still providing all the code to your users in a single repo.

@xutingl Let's hold off on merging this to main until we have a chance to clean this PR up further.

@xutingl I'm going to close this for now. We should add support for this once @sarahwooders lands the UDF support into the gateways. We can then simply write a UDF to bulk download files.


'Help > Send Feedback' option mentioned in docs but non-existent in app

In looking for a way to opt-out of VSCode's telemetry, I found step-by-step instructions for doing this in the VS Code Resources. However, Step 1 isn't possible as the location described doesn't appear to exist in the app, specifically the Send Feedback link under the Help menu in the application's menu bar.
Here is a screencap of the instructions from the docs:

Here is the help menu as seen in VS Code 1.39.1 running on MacOS Catalina 10.15.1 Beta. The 'Send Code Feedback to Apple' link triggers Apple's own Feedback Assistant, so it's assumed this isn't related to the 'Send Feedback' mentioned in the resources:

It may be that these instructions are specific to Windows.

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

ID: d08893df-9565-a578-3dda-9fb5c7d3528c
Version Independent ID: 23d30ca6-71e7-aece-d032-9b0ea5c81f58
Content: Customer Experience Improvement Program - Visual Studio
Content Source: docs/ide/visual-studio-experience-improvement-program.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @PoulChapman
Microsoft Alias: tglee

My mistake, I was in the wrong docs 😄


search for upgraded versions

Hello,
Can it get the upgraded version prices of the itens, Pri duo tri tet?
if yes what is the configuration

This is now a feature in the latest release. https://github.com/kookehs/bdo-marketplace/commit/fc15e8084871dbe9786c8e136c5b71e216bb8ead
The tool will grab all enhancement levels that can be listed on the market for each item.
It may skip some enhancement levels based on how BDO groups some levels.


Add support for the message_content intent

Rationale
Privileged intents are listed in the root command, and this adds support for detection of the "message_content" intent, if available. Contrary to #142, this uses the specific message content intent (unlike the messages intent it only applies to content, hence the name - and is also privileged). This intent is not supported by discord.py, but as it's a name declared by the API, it can be considered a "standard" name and should be consistent across forks (A "Smaller feature-detection based change" as defined here). If this feature does not detect support for the message_content intent, it will revert to the old functionality (simply not displaying info on this intent).
I can personally verify that this works with Pycord (already on API v10), and should work with disnake when their v10 support PR is merged. The same should be true for edpy, when their v10 PR is merged, and nextcord as well - when this PR is merged.
Summary of changes made
Pretty straightforward, simply adds the message_content intent to the root command. At first I thought this would be useless, but bots can still receive message content without the intent in some cases, so the command can be invoked even if the intent is off.
Checklist

[x] This PR changes the jishaku module/cog codebase

[x] These changes add new functionality to the module/cog
[ ] These changes fix an issue or bug in the module/cog
[x] I have tested that these changes work on a production bot codebase
[x] I have tested these changes against the CI/CD test suite
[ ] I have updated the documentation to reflect these changes

[ ] This PR changes the CI/CD test suite

[ ] I have tested my suite changes are well-formed (all tests can be discovered)
[ ] These changes adjust existing test cases
[ ] These changes add new test cases

[ ] This PR changes prose (such as the documentation, README or other Markdown/RST documents)

[ ] I have proofread my changes for grammar and spelling issues
[ ] I have tested that any changes regarding Markdown/RST syntax result in a well formed document

Unsure why the AppVeyor CI failed. It says invalid Slack webhook, so I assume it's unrelated to my code.

Would be nice if we got a comment on that


Incompatibilities with PHP 7.2

If you run a PHP version compatibility analysis tool such as PHPCompatibility on this plugin, you'll see that there are a few dependencies that are outdated and are no longer compatible with PHP 7.2 or 7.3. For example, here is the output you get:
FILE: wp-stateless/lib/classes/class-logger.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 257 | WARNING | Since PHP 7.0, functions inspecting arguments, like debug_backtrace(), no longer report the original value as passed to a parameter, but will instead
     |         | provide the current value. The parameter "$args" was used, and possibly changed (by reference), on line 244.
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/lib/Google/vendor/phpseclib/phpseclib/phpseclib/Net/SFTP/Stream.php
---------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------
 182 | WARNING | Global with anything other than bare variables is discouraged since PHP 7.0. Found ${$host}
---------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/lib/Google/vendor/phpseclib/phpseclib/phpseclib/Crypt/Random.php
-------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 2 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------------
  69 | ERROR | Function mcrypt_create_iv() is deprecated since PHP 7.1 and removed since PHP 7.2; Use random_bytes() or OpenSSL instead
  69 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 108 | ERROR | Function mcrypt_create_iv() is deprecated since PHP 7.1 and removed since PHP 7.2; Use random_bytes() or OpenSSL instead
 108 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 108 | ERROR | The constant "MCRYPT_DEV_URANDOM" is deprecated since PHP 7.1 and removed since PHP 7.2
-------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/lib/Google/vendor/phpseclib/phpseclib/phpseclib/Crypt/Base.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 45 ERRORS AND 2 WARNINGS AFFECTING 27 LINES
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  610 | WARNING | Since PHP 7.0, functions inspecting arguments, like func_get_args(), no longer report the original value as passed to a parameter, but will instead
      |         | provide the current value. The parameter "$method" was used, and possibly changed (by reference), on line 608.
  792 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  792 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  825 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  825 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  828 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  828 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  833 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  833 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  842 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  842 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  852 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  852 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
  855 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
  855 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1132 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1132 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1160 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1160 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1165 | ERROR   | Function mcrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1165 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1174 | ERROR   | Function mdecrypt_generic() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1177 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1177 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1659 | ERROR   | Function mcrypt_list_algorithms() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1659 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1738 | ERROR   | Function mcrypt_module_close() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1738 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1739 | ERROR   | Function mcrypt_module_close() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1739 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1744 | ERROR   | Function mcrypt_module_close() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1744 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1851 | ERROR   | The constant "MCRYPT_MODE_ECB" is deprecated since PHP 7.1 and removed since PHP 7.2
 1852 | ERROR   | The constant "MCRYPT_MODE_CBC" is deprecated since PHP 7.1 and removed since PHP 7.2
 1854 | ERROR   | The constant "MCRYPT_MODE_CFB" is deprecated since PHP 7.1 and removed since PHP 7.2
 1855 | ERROR   | The constant "MCRYPT_MODE_NOFB" is deprecated since PHP 7.1 and removed since PHP 7.2
 1856 | ERROR   | The constant "MCRYPT_MODE_STREAM" is deprecated since PHP 7.1 and removed since PHP 7.2
 1859 | ERROR   | Function mcrypt_module_open() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1859 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1860 | ERROR   | Function mcrypt_module_open() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1860 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1866 | ERROR   | Function mcrypt_module_open() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1866 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 1866 | ERROR   | The constant "MCRYPT_MODE_ECB" is deprecated since PHP 7.1 and removed since PHP 7.2
 1871 | ERROR   | Function mcrypt_generic_init() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 1871 | ERROR   | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 2610 | WARNING | Function create_function() is deprecated since PHP 7.2; Use an anonymous function instead
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/lib/Google/vendor/phpseclib/phpseclib/phpseclib/bootstrap.php
---------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------
 10 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP 7.2
---------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/lib/Google/vendor/guzzlehttp/promises/src/functions.php
---------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------------------------
 222 | WARNING | Function each() is deprecated since PHP 7.2; Use a foreach loop instead
 257 | WARNING | Function each() is deprecated since PHP 7.2; Use a foreach loop instead
 313 | WARNING | Function each() is deprecated since PHP 7.2; Use a foreach loop instead
---------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/vendor/udx/lib-utility/lib/class-utility.php
--------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------------------------------------------------
  511 | ERROR | preg_replace() - /e modifier is deprecated since PHP 5.5 and removed since PHP 7.0
 2321 | ERROR | Function mcrypt_encrypt() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 2321 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 2321 | ERROR | The constant "MCRYPT_RIJNDAEL_256" is deprecated since PHP 7.1 and removed since PHP 7.2
 2321 | ERROR | The constant "MCRYPT_MODE_CBC" is deprecated since PHP 7.1 and removed since PHP 7.2
 2348 | ERROR | Function mcrypt_decrypt() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
 2348 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 2348 | ERROR | The constant "MCRYPT_RIJNDAEL_256" is deprecated since PHP 7.1 and removed since PHP 7.2
 2348 | ERROR | The constant "MCRYPT_MODE_CBC" is deprecated since PHP 7.1 and removed since PHP 7.2
--------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/vendor/udx/lib-wp-bootstrap/lib/classes/class-scaffold.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 126 | WARNING | Since PHP 7.0, functions inspecting arguments, like debug_backtrace(), no longer report the original value as passed to a parameter, but will instead
     |         | provide the current value. The parameter "$args" was used, and possibly changed (by reference), on line 101.
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: wp-stateless/vendor/ccampbell/chromephp/ChromePhp.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 266 | WARNING | Since PHP 7.0, functions inspecting arguments, like debug_backtrace(), no longer report the original value as passed to a parameter, but will instead
     |         | provide the current value. The parameter "$args" was used, and possibly changed (by reference), on line 253.
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------

I updated related libraries to new versions, and checked compatibility with PHP since 7.0. They still use deprecated functions.
Sorry, but it does not depend on us.

Backlogging for now.

@planv is this still relevant?

Closing the ticket


Fix an issue where no output for 60 seconds would throw an error

This builds on #24 by fixing an issue where winrm would error if the command being run didn't output any data to standard output or standard error for over 60 seconds.

This looks good to me, but it won't merge on top of #24 since it contains the same commit. Would you agree if I only merge this one, and close the previous one?

Yeah that's fine (although I'm surprised GitHub wouldn't merge it given its a strict fast forward and should auto merge fine).

Any plans on getting this merged soon?

Oops, I completely forgot, thanks for pinging me @hach-que. I'll merge the patch now.


Microphone Guardian version doesn't work

I'll describe the problem - microphone guardian correctly recognizes all sound devices on the computer, but when the sound crosses the threashold no sound occurs (although supposedly there should be a monkey sound like in Scymia.exe), and the GUI looks different from the one shown on the Readme page.

Hi, the GUI looks different because I've updated it and didn't take another screenshot to update the Readme.
is the output correctly set? moreoever, what happens if you click "debug" ? it should make a bip

Hi, the GUI looks different because I've updated it and didn't take another screenshot to update the Readme. is the output correctly set? moreoever, what happens if you click "debug" ? it should make a bip

You were right, at first there was no sound when I pressed the “debug” button, but then I looked more closely at the output devices and for some reason there are many more than I have on my computer (for some reason they are duplicated) and on one of the copies the sound appeared, thank you very much. (But it would be nice if the program could remember the last selected I/O devices)

yes, it would be better.
I will implement it thankyou


add support for secrets files

this introduces new env variables for user name, password and uri. more importantly user name and password can also be read from files - this is needed to securely pass secrets to the docker container by just mounting these files.
new variables:
DATA_SOURCE_USER
DATA_SOURCE_USER_FILE
DATA_SOURCE_PASS
DATA_SOURCE_PASS_FILE
DATA_SOURCE_URI

Coverage decreased (-2.2%) to 49.605% when pulling fe42532b5eb1fba176fa189ab747a9e9b8efe2da on matthias-kloeckner:master into c4f701e817e3d6ea49bcabfd1a7effb8d0f4bea8 on wrouesnel:master.

Coverage decreased (-2.2%) to 49.605% when pulling fe42532b5eb1fba176fa189ab747a9e9b8efe2da on matthias-kloeckner:master into c4f701e817e3d6ea49bcabfd1a7effb8d0f4bea8 on wrouesnel:master.

Coverage decreased (-2.2%) to 49.605% when pulling b3027183aaeb4087d0ab32983d88793e67dcaa13 on matthias-kloeckner:master into 1fa394d4888ed8893e275b1686507fe4c3ff1172 on wrouesnel:master.

Coverage decreased (-2.1%) to 48.694% when pulling 8a7daebfd512eb1e7d2c6fd0c427fd2fa6cbf11f on matthias-kloeckner:master into 81194c92194402d5a79f87a11247d80039d6ed66 on wrouesnel:master.

Coverage decreased (-2.1%) to 48.694% when pulling ac60ce92649390576d95b952be361d1f03a6776a on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

+1 would like to see secrets supported, too!

Coverage decreased (-2.0%) to 48.844% when pulling abc018e2f12b19fe22b9d7577fbe6a027af0f41e on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

Coverage decreased (-2.0%) to 48.844% when pulling 551af02f752542179de08b0b9546f8e9b23aa93f on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

Coverage decreased (-2.0%) to 48.844% when pulling 97518a257d762489e51a8347ac80e7adc2b409f7 on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

Coverage decreased (-2.0%) to 48.844% when pulling 9328047ba85286c3e428901a729b96720160b126 on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

Coverage decreased (-2.0%) to 48.844% when pulling c3ff253e27224dff362659b09efd3872ec0291d3 on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

Coverage increased (+1.6%) to 52.388% when pulling 908912d8a1f17b290a5682321cde32a3932a6d46 on matthias-kloeckner:master into 1afbd62ab194c045a88488d77de2f116400dedb7 on wrouesnel:master.

added some tests and fixed the makefile. sorry for the amount of commits - I needed some test runs on travis to understand the real problem.

Coverage increased (+1.8%) to 54.135% when pulling 05f70bb6d78d91463541a2baf191b2204389452a on matthias-kloeckner:master into e701b80fd52821b3fab1dbab99138902e581eeac on wrouesnel:master.

@wrouesnel
anything still missing here?

No this is looking good. I'll give it a closer inspection tonight and merge it hopefully.


Add support for exact quantisation of images with <= 256 colors

We detect if an image has <= 256 colors, and if so bypass NeuQuant and generate a palette directly from those colors.
As a relative Rust newbie, I would particularly appreciate any suggestions on fitting in with the style of the package, and general Rust idioms. This is a first cut that (I believe) does the job, but any feedback appreciated.
Side issue
One concern I have is around handlng of "transparent". For NeuQuant, it looks to me like if there's multiple colours that have 0 alpha, only one gets selected as transparent, but independently NeuQuant will build an RGBA palette for all the given colours, and of multiple colours with 0 alpha, only one will be mapped to transparent. The others will become visible.
I haven't had time to build a test case, so this only comes from code reading, but it seems suboptimal. I suggest the pre-pass converts all transparent pixels to explicitly the same RGBA value to avoid this issue. I'll be happy to raise another issue and another PR if you agree this is an issue.
For this PR, I've tried to remain consistent with the NeuQuant transparency-handling.
Cheers,
Simon.

Thanks for this PR! The current logic looks workable to me, but I think could be significantly simplified by eliminating the enum and simply handling both cases in the from_rgba_speed function.

I'll have a go at that and see what you think of it...


Jesse: Watch telkomuniversity.ac.id

Jesse requests the watch of the watch_keyword telkomuniversity\.ac\.id. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

telkomuniversity\.ac\.id has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Just an additional info: telkomuniversity.ac.id is the official site of a prestige university in Indonesia. While watching this domain might be okay for further monitoring, I have a doubt if they really want to spam it.

Rejected by cigien in Charcoal HQ. Reason: 'There are several fp for this on the network. Watching a more specific URL instead.'


Apply order to Inventory#offer

This commit changed the order of the GridInventory to be before the Hotbar in the Main PlayerInventory. As a result, offering an item to the player's inventory now tries to send it to their storage first rather than the hotbar. This has been reported as a bug a couple of times, as even though it's correct by implementation it's not what players expect (and not what Vanilla does either).
One solution to this is the following:
public InventoryTransactionResult offer(ItemStack item, Class<? extends Inventory> types) {
    Inventory inventory = //some empty inventory. Would be nice to have Inventory.empty() in the API
    for (Class<? extends Inventory> type : types) {
        inventory = inventory.union(this.query(QueryOperationTypes.INVENTORY_TYPE.of(type)));
    }
    inventory = inventory.union(this);
    return inventory.offer(item);
}

This method constructs a new inventory that reorders the slots based on the given types. This means that a call of inventory.offer(item, Hotbar.class), will construct a new inventory containing all of the slots in the hotbar followed by any remaining slots in the inventory (as per Inventory#union).
This does not provide an easy way to include the hand of a player first, and I don't have a solution for this in the API (though it is easy enough to do as a utility method if a developer chooses to).

As it is right now, you currently need to use work-around code to get this ordering functionality that was incidentally present API 6 and before. Major props to whoever is able to take this up.

You can now get an inventory with different ordering using Inventory#transform.


[BUG] Incapable of speaking abbreviations

Feel free to ask any kind of questions in the issues page, but please use English since other users may find your questions valuable.
Describe the bug
The model can not speak SSH nor S.S.H.. How can I prompt it to do so?
(also, the model often stops talking right before the S.S.H. occurence, somehow the EOS token prob is high there)

A 100 steps of training already solved that, astonishing how great this model is!


How to use PIXI.Application() when there is already a canvas element in HTML?

Good evening,
my issue is not a bug, but more my difficulty to understand the documentation.
I am using pixi.js version 5.2.0 to create a word app at Facebook and your library is great, thank you all very much for developing it.
However recently I have migrated my code from pixi.js 4 to 5 and I am not sure how to change my code:
        var ratio = BOARD_WIDTH / (BOARD_HEIGHT + SmallTile.HEIGHT);
        var renderer = new PIXI.Renderer({
                width: BOARD_WIDTH,
                height: BOARD_HEIGHT + SmallTile.HEIGHT,
                view: document.getElementById('board')
        });

        window.onresize = function(event) {
            var w = Math.min(window.innerWidth, screen.width) - 2 * MENU_WIDTH;
            var h = Math.min(window.innerHeight, Math.round(.7 * screen.height));

            if (w / h >= ratio) {
                w = Math.round(h * ratio);
            } else {
                h = Math.round(w / ratio);
            }

            renderer.view.style.width = (w - 2 * PADDING) + 'px';
            renderer.view.style.height = (h - 2 * PADDING) + 'px';
        };

        window.onresize();

My problem is that the v5 doc suggests:
const app = new PIXI.Application();

// The application will create a canvas element for you that you
// can then insert into the DOM
document.body.appendChild(app.view);

But I already have a canvas element in my HTML file:

   <CANVAS ID="board" STYLE="border: 1px dotted white;"></CANVAS>

How can I please use the new PIXI.Application, but attach it to the existing canvas element?
As you see probably see from my code I am not very proficient in the latest Javascript syntax, but I think my issue is very simple, I am just missing some guidance. Thank you.
Environment

pixi.js version:  5.2.0
Browser & Version:  Opera
OS & Version:  Win 10
Running Example: https://apps.facebook.com/slova-farbera/

Here the error reported by some of my game users (using Opera browser or maybe Yandex browser - the communication with the user is challenging, she is an elderly lady):

Do something like you were doing in v4, pass the view option to the Application constructor:
const app = new Application({ view: document.getElementById(“board”) })

Thank you, I will try this right away and report back.

Thank you, the following has worked for me and I will report later if that elderly lady with the weird browser replies. My new code is:
        var ratio = BOARD_WIDTH / (BOARD_HEIGHT + SmallTile.HEIGHT);
        var app = new PIXI.Application({
                width: BOARD_WIDTH,
                height: BOARD_HEIGHT + SmallTile.HEIGHT,
                view: document.getElementById('board'),
                antialias: true,
                backgroundColor: 0xFFFFFF
        });

        window.onresize = function(event) {
            var w = Math.min(window.innerWidth, screen.width) - 2 * MENU_WIDTH;
            var h = Math.min(window.innerHeight, Math.round(.7 * screen.height));

            if (w / h >= ratio) {
                w = Math.round(h * ratio);
            } else {
                h = Math.round(w / ratio);
            }

            app.renderer.view.style.width = (w - 2 * PADDING) + 'px';
            app.renderer.view.style.height = (h - 2 * PADDING) + 'px';

I reopen issues because this is another problem.
Pixi v5 can't include canvas renderer in main package, you must use pixi.js-legacy instead.
У вас другая проблема, как я вижу по скриншоту.
Пикси не поддерживает canvas2D в основном пакете, вы должны использовать его legacy версию - pixi.js-legacy

Thank you, the following has worked for me and I will report later if that elderly lady with the weird Yandex browser replies. My new code is:
        var ratio = BOARD_WIDTH / (BOARD_HEIGHT + SmallTile.HEIGHT);
        var app = new PIXI.Application({
                width: BOARD_WIDTH,
                height: BOARD_HEIGHT + SmallTile.HEIGHT,
                view: document.getElementById('board'),
                antialias: true,
                backgroundColor: 0xFFFFFF
        });

        window.onresize = function(event) {
            var w = Math.min(window.innerWidth, screen.width) - 2 * MENU_WIDTH;
            var h = Math.min(window.innerHeight, Math.round(.7 * screen.height));

            if (w / h >= ratio) {
                w = Math.round(h * ratio);
            } else {
                h = Math.round(w / ratio);
            }

            app.renderer.view.style.width = (w - 2 * PADDING) + 'px';
            app.renderer.view.style.height = (h - 2 * PADDING) + 'px';
        };

        window.onresize();

Thanks for your comment Konstantin and let's stick to English lanuage :-)
Do you mean I should keep the

and add another line with pixi.js-legacy.js? And how to get it activated?

Legacy build is same as pixi.js build, but include depricated renderer.
I don't know why it doesn't exist on cdn, but you can download it by this link:
http://pixijs.download/v5.2.0/pixi-legacy.min.js

Yes, the lady just reported back, that her issue "This browser does not support WebGL" is not solved - so switching to PIXI.Application was not enough there.
But I think I will better ignore her problem, because I don't want to downgrade the experience of all the rest users by switching to legacy and forceCanvas:true
Thank you

Pixi (pixi legacy) is must automatically detect supported renderer when you use Application.

This seems answered now. Can we close this @eXponenta?


Error after updating to RN 0.47.2?

Hi, i was this library working fine but some day (i think after upgrade React Native) this broke.
XCode build okey but when i share something from safari, camera roll or somewhere y have this error in console.
(myapp.debug.share) <Warning>: Unknown key for Boolean: EnableTransactions

(myapp.debug.share) <Warning>: Unknown key for Boolean: EnablePressuredExit

(com.apple.DictionaryServiceHelper) <Warning>: Unknown key for Boolean: EnableTransactions

(com.apple.DictionaryServiceHelper) <Warning>: Unknown key for Boolean: EnablePressuredExit

(com.apple.StreamingUnzipService) <Warning>: Unknown key for Boolean: EnableTransactions

(com.apple.StreamingUnzipService) <Warning>: Unknown key for Boolean: EnablePressuredExit

(com.apple.uifoundation-bundle-helper) <Warning>: Unknown key for Boolean: EnableTransactions

(com.apple.uifoundation-bundle-helper) <Warning>: Unknown key for Boolean: EnablePressuredExit

(myapp..debug.share[36327]) <Error>: assertion failed: 16G29: launchd_sim + 63487 [4F755FB1-4156-3577-BEB8-3FD0A54C6323]: 0x16

Have you got some clue?
regards!
Marcelo.

@mferranti Have you able to resolve this issue?

We are experiencing the same. RN 0.57.1.

Same error on my side.. Any clue how to resolve that ?


check if docker command is available before running pipeline run

Close #1442

@droot @Shell32-Natsu please take a look


[IoT] Remove IoT PNP preview commands from core CLI

Description

These preview commands have changed significantly, (as has the structure of the service) and we will be moving their implementation into the IoT Extension.
Testing Guide

All preview commands under az iot pnp have been removed.
History Notes

[IoT] BREAKING CHANGE: az iot pnp a: Preview az iot pnp commands will require the IoT CLI Extension

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

IoT

@yonzhan  / @zhoxing-ms - just curious if there's any update on this? Would like to avoid any last minute changes to this PR

@c-ryan-k Hi, may I ask why not delete the relevant tests and just delete the record files??

@c-ryan-k Hi, may I ask why not delete the relevant tests but just delete the record files?
@zhoxing-ms:  Not quite sure I understand - I've deleted both the PNP test file and the PNP recorded test file, and only out of "latest" because we did not include the PNP commands in our previous profile tests.

Is there another file (or other files) I should have removed?

@c-ryan-k OK, there are no other files need to be deleted.
By the way, the deletion of the command will cause a breaking change to the customers. Has the impact been evaluated and is it acceptable?

The deletion of these out-of-date preview commands align with IoT goals and future plays. A different implementation of these commands will exist in the IoT extension for Azure CLI. Users should expect preview (as everything removed here is under preview) labeled commands have no guarantee of stability as preview/concept technology can rapidly change.
Is there anything else pending for the merge of this PR? We would like to get this in the scheduled 7/14 Azure CLI release.

Hi @c-ryan-k  and @digimaun , through telemetry, az iot pnp is used by 1 subscription in the past 30 days. I will recommend you to add deprecation information for one sprint and hide the commands. And then we can remove them.

Hi @c-ryan-k and @digimaun , through telemetry, az iot pnp is used by 1 subscription in the past 30 days. I will recommend you to add deprecation information for one sprint and hide the commands. And then we can remove them.

@Juliehzl, from our CLI guideline  Items marked preview can be changed, broken or removed at any time without following the deprecation process., I think it's OK to delete these preview commands directly.


Rasa takes all available GPU memory (inference)

Rasa version: 0.14.6
Python version: 3.6.3
Operating system: Ubuntu 18.04
Issue:
When we start an agent like so:
from rasa_core.agent import Agent
from rasa_core.interpreter import RasaNLUInterpreter
from rasa_core.utils import EndpointConfig

interpreter = RasaNLUInterpreter('models/current/nlu')
agent = Agent.load('models/dialogue', interpreter=interpreter, action_endpoint=EndpointConfig(url="http://localhost:5055/webhook"))

It loads up TensorFlow, which goes on (by default) to hog all available GPU memory:
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 418.56       Driver Version: 418.56       CUDA Version: 10.1     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce RTX 208...  Off  | 00000000:01:00.0  On |                  N/A |
| 41%   37C    P2    64W / 260W |  10729MiB / 10986MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      1423      G   /usr/lib/xorg/Xorg                            18MiB |
|    0      1553      G   /usr/bin/gnome-shell                          58MiB |
|    0      2127      G   /usr/lib/xorg/Xorg                           131MiB |
|    0      2272      G   /usr/bin/gnome-shell                         108MiB |
|    0      8550      G   ...quest-channel-token=9868978255989529958    44MiB |
|    0      9277      C   python                                     10363MiB |
+-----------------------------------------------------------------------------+

This prevents other ML models from being able to load on the same system. The solution to this problem is to enable allow_growth (see here)
config = tf.ConfigProto()
config.gpu_options.allow_growth = True
session = tf.Session(config=config, ...)

However, how can we get access to the TensorFlow session being used?
Content of configuration file (config.yml):
language: en

pipeline:
- name: "nlp_spacy"
- name: "tokenizer_spacy"
- name: "intent_entity_featurizer_regex"
- name: "intent_featurizer_spacy"
- name: "ner_crf"
- name: "ner_duckling_http"
  url: "http://localhost:8000"
  dimensions: ["time", "number", "amount-of-money", "distance", "duration", "temperature", "volume", "phone-number"]
- name: "ner_synonyms"
- name: "intent_classifier_sklearn"

@neil-119 i wouldn't suggest using a GPU with that, it's unnecessary. The pipeline you're using doesn't use tensorflow at all btw (i also won't suggest it even if you switch to the tensorflow pipeline)

Hi, yes as you can see, I am not using TensorFlow in my pipeline, but I assumed that the underlying LSTM model being trained with the Keras policy was using TensorFlow. Here's my policies.yml:
policies:
  - name: KerasPolicy
    epochs: 151
    max_history: 5
  - name: FallbackPolicy
    fallback_action_name: 'action_default_fallback'
  - name: MemoizationPolicy
    max_history: 5
  - name: FormPolicy

Now, when the trained model is loaded by Rasa for inference / production use, it appears to initialize a TensorFlow session without the allow_growth flag. With TensorFlow, a session initialized that way uses all available GPU memory by default.

ah right, didn't realise you were talking about core. But yeah, here I wouldn't recommend using a GPU either because an LSTM isn't optimised for running on a GPU (unless you use the gpu specific implementation with tensorflow, which we're playing around with right now)
To make it run on a CPU i suppose you just need to install the regular old tensorflow rather than tensorflow-gpu.
I'm gonna close this issue now as there's not much we can do about that at the moment

For anyone coming to this, a solution is to add the environment variable:
        - name: "TF_FORCE_GPU_ALLOW_GROWTH"
          value: "true"

e.g. here: https://github.com/RasaHQ/rasa-x-helm/blob/99f8f5789948a350f8585527c92c3e767268e77c/charts/rasa-x/templates/rasa-deployments.yaml#L96
@akelad  I would be interested to know if Rasa provides any support for GPU these days? I can't find any official image for it, but I believe it seems up training of large models significantly, so seems like an important feature for e.g. training in the Rasa X GUI.


Create custom language model with SRILM

Hello, I have been reading the tutorial documentation about creating custom language models using SRILM toolkit. The documentation also mentions that:
"A language model can be stored and loaded in three different formats: text ARPA format, binary BIN format and binary DMP format. The DMP format is obsolete and not recommended."
However, the demo uses only .dmp format. Can I train a .lm Language Model with SRILM and use it with the demo? E.g.:
// Create language model search
File languageModel = new File(assetsDir, "custom.lm");
recognizer.addNgramSearch(CUSTOM_SEARCH, languageModel);

I think #36 might be related but the answer link is broken. Thanks in advance!

Hello, here are some of my findings:
Things that didn't work for me

Modeling with SRILM was easy, it outputs a  .lm model, in my case, in a few seconds, which I found weird. Note that, I was using capitalized text for the corpus in test and training. The resulting .lm model didn't work in Android
The .lm and .dmp extensions work with Android, one can use this available small-size LM  (not pruned LM give OOM error). The .lm.bin didn't work with Android.  Also, using the eng-70k-0.2-pruned.lm gives a WER of ~ 90% (dict and AM same as demo)

Solution

I downloaded the CMUCLMTK from Sourceforge, installing it from other sources (homebrew, or CMU webpage didn't work). . It does work! But the commands in the tutorial may need an update. Note that in this case the utterances in the corpus.txt are between  tags, and the third command needs an extra .ascii extension (source1, source2).

Also one needs to add the following path to access the commands (tested on ubuntu:latest Docker image, gcc g++ make required):
echo 'export LD_LIBRARY_PATH="/lib:/usr/lib:/usr/local/lib"' >> ~/.bashrc
. ~/.bashrc

text2wfreq < corpus.txt | wfreq2vocab > vocabulary.vocab
cat corpus.txt | text2idngram -vocab vocabulary.vocab -idngram languagemodel.idngram.ascii -write_ascii
idngram2lm -vocab_type 0 -idngram languagemodel.idngram**.ascii** -vocab vocabulary.vocab -arpa languagemodel.lm

The resulting LM with .lm extension works, I suppose that the .dmp extension also does. To convert one can use e.g.
sphinx_lm_convert -i languagemodel.lm -o languagemodel.dmp


python3.9 无法运行

报错提示：
Traceback (most recent call last):
File "/home/nxb/tools/WeblogicScan/WeblogicScan.py", line 13, in
from config.config_console import Weblogic_Console
File "/home/nxb/tools/WeblogicScan/config/config_console.py", line 14, in
from poc.index import *
File "/home/nxb/tools/WeblogicScan/poc/index.py", line 10, in
from poc import Console
File "/home/nxb/tools/WeblogicScan/poc/Console.py", line 13, in
from config.config_requests import headers
File "/home/nxb/tools/WeblogicScan/config/config_requests.py", line 13, in
ua = UserAgent(verify_ssl=False)
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/fake.py", line 69, in init
self.load()
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/fake.py", line 75, in load
self.data = load_cached(
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/utils.py", line 250, in load_cached
update(path, use_cache_server=use_cache_server, verify_ssl=verify_ssl)
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/utils.py", line 245, in update
write(path, load(use_cache_server=use_cache_server, verify_ssl=verify_ssl))
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/utils.py", line 187, in load
ret = json.loads(get(
File "/home/nxb/.local/lib/python3.9/site-packages/fake_useragent/utils.py", line 84, in get
raise FakeUserAgentError('Maximum amount of retries reached')
fake_useragent.errors.FakeUserAgentError: Maximum amount of retries reached

同样的报错：
Traceback (most recent call last):
File "D:\AAA INTERNET SECURE\web leak\serial\WeblogicScan-master\WeblogicScan.py", line 13, in
from config.config_console import Weblogic_Console
File "D:\AAA INTERNET SECURE\web leak\serial\WeblogicScan-master\config\config_console.py", line 14, in
from poc.index import *
File "D:\AAA INTERNET SECURE\web leak\serial\WeblogicScan-master\poc\index.py", line 10, in
from poc import Console
File "D:\AAA INTERNET SECURE\web leak\serial\WeblogicScan-master\poc\Console.py", line 13, in
from config.config_requests import headers
File "D:\AAA INTERNET SECURE\web leak\serial\WeblogicScan-master\config\config_requests.py", line 10, in
from fake_useragent import UserAgent
ModuleNotFoundError: No module named 'fake_useragent'


on_change event for cells

Hi there!
I would like to render a table where the user is able to edit the content of the cells. I already noticed the on_click event on the RowRenderer which is basically what I am looking for, but on change in the cell renderer.
Something like:
fn CustomCellEditRenderer(...) -> impl IntoView {
    view!{
        cx,
        <td>
            <select on:change=move |_| on_change(/* parameters that contain information like row key, column and updated value */)>
                <option value="1">Something</option>
                <option value="2">Else</option>
            </select>
        </td>
    }
}

Is there a simple way to do this?

Editing cells is definitely on the roadmap but right now it’s not implemented yet. We’re open to a PR though

Sounds good, I would be happy to give it a try. Can you give me a short guideline what needs to be changed or where to look at first, so I have a starting point?

Thanks for your explanations, they have been very helpful for me to get started!
I prepared a PR for each of the repositories:
https://github.com/Synphonyte/leptos-struct-table/pull/4
https://github.com/Synphonyte/leptos-struct-table-macro/pull/1
The first one contains explanations about the changes and my thoughts on it.

In addition to the updates in #4 I already have some other things added like support for DateTime<Utc> and double click event on rows. You can have a look here. For the macro crate the same branch exists.
Still the server_function example is not updated yet, but if you are happy with the additions I did we can replace #4 with the feature/opus-rs branch.

I like your changes in the feature/opus-rs branch. Let's go back to the TableDataProvider as it was with the synchronous set_row method and I'll merge everything. We can figure out the async later.

Implemented in 34db429


Fix incremental logic in segment_web_sessions__initial (option B)

Description & motivation

Fix https://github.com/fishtown-analytics/segment/issues/62
Add logic to include all page views from a session if any page view of that session overlaps with segment_sessionization_trailing_window during incremental runs.
(Alternate approach here: https://github.com/fishtown-analytics/segment/pull/64/)
Checklist

[x] I have verified that these changes work locally
[ ] I have updated the README.md (if applicable)
[ ] I have added tests & descriptions to my models (and macros if applicable)

Closing in favour of the alternate option (#64)


mod-main /supportRoles text not fitting in UI

https://maintemplate.ci.getcouragenow.org/#/supportRoles/orgs/003

ui is cleaned up, paddings aligned, more space for text


Switch to sysregistriesv2

Switch from using github.com/containers/image/pkg/sysregistries to using github.com/containers/image/pkg/sysregistriesv2 to complete unqualified image names.  Keep v1 around because it'll tell us which configuration file to name in an error message if things don't work right.

FYI @vrothberg (not sure if you watch Buildah).

bot, retest this please

bot, retest this please.

bot, retest this please

@rh-atomic-bot r+


Local parchment hangs during loading in Chrome

Parchment hangs loading files locally in Chrome while it has no issues doing so with Firefox. This is the same with Phonegap Builds and NW.js.

Do you mean trying to load something like file:///.../parchment/index.html?story=game.z8? Chrome has restrictions that means that it can only load .js files. If you need to load from file: I'd recommend you use Inform 7's method of producing a custom Parchment site.


dev-cmd/update-maintainers: If there are changes, auto-update man pages

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

I noticed this in a PR recently, maybe the maintainer didn't think they would have to run brew man for this? In that case there's an extra step after the PR is raised, red CI checks on the PR (which is demoralising) and potentially requires a reviewer to point this out.
This runs brew man if the README file has a diff as part of this command. Then the user has to git add . and their PR is (probably) good.
I thought at first to remind users to run brew man after the success message, but it was just as easy to run the update command automatically.

Review period will end on 2021-02-10 at 21:31:40 UTC.

Love it, thanks @issyl0!

Review period ended.


[MRG] Remove ipython dependency

Remove ipython as a dependency, Closes #192

While I think this works it would be good if someone else checked the quickstart tutorial works via a docker install before merging.


[Feature] Text Animation Effect

Description
The Text will be animated having Typewriter effect but different than the existing one. For reference you can check screenshot.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

please assign this issue to me under gssoc 24

@Aashka19 I have been waiting for days to assign it to me. Please find another issue.

Go ahead @jaydeep099

@Rakesh9100 Please give this issue a label

@Rakesh9100 Please do it asap so i can make a PR

Pls provide your PR @jaydeep099
Label will be added once the PR is merged!!

@Rakesh9100 PR has been made please review it.


chainerx.ndarray.__setitem__ raises AttributeError if CuPy is unavailable.

>>> x = chainerx.array([3])
>>> x[0] = 4
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/tos/GitHub/chainer/chainerx/_fallback_workarounds.py", line 134, in __setitem__
    if cupy is not None and isinstance(self, cupy.ndarray):
AttributeError: module 'cupy' has no attribute 'ndarray'
>>> chainer.print_runtime_info()
Platform: Darwin-18.2.0-x86_64-i386-64bit
Chainer: 6.0.0b1
NumPy: 1.15.4
CuPy: Not Available
iDeep: Not Available
>>> chainerx.is_available()
True

If CuPy is available, it raises with the expected message (while the type could be NotImplementedError)
>>> x = chainerx.array([3])
>>> x[0] = 4
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'chainerx.ndarray' object does not support item assignment

Which commit are you using?
I can't reproduce the reported behavior (I'm using current master 2e1c2838dcd14dc182766525f8511ce3e33617d9).

Without CuPy

>>> x = chainerx.array([3])
>>> x[0] = 4
>>> x = chainerx.array([3], device='cuda')
>>> x[0] = 4
...
RuntimeError: ChainerX fallback implementation for cuda backend requires cupy to be installed.

With CuPy

>>> x = chainerx.array([3], device='cuda')
>>> x[0] = 4


[#1277] Improvement(column): Add column attribute autoIncrement of the increment interface.

What changes were proposed in this pull request?
Add column attribute autoIncrement of the increment interface.
Why are the changes needed?
Fix: #1277
Does this PR introduce any user-facing change?

Change in user-facing APIs.

How was this patch tested?
UT

Should you add an exception thrown in the server side implementation since the catalog has not been implemented yet?

It is not easy to determine whether it is set by the user in the catalog layer, as it is not a wrapper Boolean object

we can only throw an exception when autoIncrement()==true, because autoIncrement()==false is naturally supported.

@mchades can you please take care of this PR? Thanks.

Overall, LGTM, but some comments left and IT issue seem to be addressed.

If this PR change user-facing interface, we should also add the related documents.


Unable to download the model

Hi there,
thanks for providing the code and model for use.
I'm having trouble replicating the example:
`import ablang
heavy_ablang = ablang.pretrained("heavy")
heavy_ablang.freeze()`
prints "Downloading model ..."
Which hangs there with no further logging for many hours.
I find I can navigate via web-browser to the url specified in the code:
"http://opig.stats.ox.ac.uk/website/data/downloads/" and download the tarball of the model there.
Best,
Michael

Hi Michael,
Thanks for your interest in using AbLang.
Since you are able to download the model directly using the url, I don't think it is something on our side.
On possible reason the download hangs could be, since the weights of AbLang are downloaded and stored inside the AbLang package, that your environment is in a directory with not enough disk space?
Please let me know if this is the case.
Best,
Tobias

Thanks Tobias! A possible enhancement would be to add a simple download progress bar.
In any case for folks who may have the same issue, my workaround was this:

download the model of interest here (heavy in my case)

http://opig.stats.ox.ac.uk/website/data/downloads/ablang-heavy.tar.gz

create a folder called model-weights-heavy in AbLang/ablang/
untar the model in that folder


CLOUDSTACK-10118 Domain level settings

The intentions with domain level settings is to make the ldap functionality configurable on a per domain basis. This will allow for business units or customers to maintain their own users.
Config enum unused methods removed

@borisstoyanov I added a jira ticket. This is in itself not usefull, though. It requires use by some opther feature. I think it can be tested but no tests are available. CLOUDSTACK-10117 is the reason it is needed.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1186

@DaanHoogland can you rebase this and advise if this is ready for review/testing?

@rhtyd i will rebase but this change is structural for the ldap changes I am doing and does not have any merrit on it's own. It is here to keep an eye on integration. Marking it with WIP

closing in favour of #2369


CCSS and W3C

I'm really sorry if this has already been asked previously (pretty sure it did) but I couldn't find a direct answer to it.
I was really impressed by Constraint CSS and VFL and even more impressed that there was a proposal for this since 1999.
How far has this group or other people have pushed and influenced W3C to actually make it natively supported?
Could we say that concepts of CCSS have been proposed/introduced already but always in the form of conventional CSS? I believe the following specs were developed to address some of the same problems CCSS tries to solve:

CSS Grid Layout
CSS Flexible Box Layout
CSS Box Align

But they obviously lack a lot of features that CCSS enables due to cyclic references.
Can someone elaborate a little bit more or point me to references of these discussions that are close to how W3C is handling this topic?

I think GSS would be a good language to base standards on, as it's pretty well contained and we've worked hard to make solving process determenistic and (relatively) easy to reason about. But it takes a lot of work to write a proper spec, and there's no incentive from the browser dept.
In our projects, we take a way of giving up dynamicity for the sake of portability (by exporting pages to css). But it's obviously not suitable for regular websites.


docs: add steps to add Dart SDK to PATH

⬇ Generated builds by Codemagic for commit 007374af710090f1580328704790c59cb86593cc ⬇

iOS
Android

  Download link
  Download link


Fix typos in the chart values

Hello, I noticed some typos in the values file.

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 increased (+0.5%) to 92.681%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
1583

Relevant Lines:
1708

💛  - Coveralls

Thank you for your contribution


Minor doc update

Continue previous updates which updated doc. Changed javascriptmvc reference to jquerymx, to match the following commands.

thanks!


fix: allow websockets in url rewrite

What type of PR is this?

fix: allow websockets in url rewrite
What this PR does / why we need it:
This will add websocket UpgradeConfig to urlrewrite case.
Which issue(s) this PR fixes:

Fixes #2989
cc @arkodg

/retest

add label to verify github action

We also need @Xunzhuo to approve then

We also need @Xunzhuo to approve then

just want to make sure bot will create the PR.

/retest

/retest

wait https://github.com/envoyproxy/gateway/pull/3039


BUG: build graph non-determinism

Describe your issue.
It seems that some Cython code needs more dependencies specified. Namely, some Cython generation requires from scipy.special cimport cython_special to work. While trying to ninja -k0 in order to ninja -j1 to isolate the error message, the problem went away. This tells me that dependencies probably just need better ordered.
Reproducing Code Example
N/A

Error message
[674/845] Generating 'scipy/special/_ellip_harm_2.cpython-39-darwin.so.p/_ellip_harm_2.c'
FAILED: scipy/special/_ellip_harm_2.cpython-39-darwin.so.p/_ellip_harm_2.c
/Users/ben.boeckel/code/superbuild/build/install/bin/cython -3 --fast-fail --output-file scipy/special/_ellip_harm_2.cpython-39-darwin.so.p/_ellip_harm_2.c --include-dir . ../../scipy/special/_ellip_harm_2.pyx

Error compiling Cython file:
------------------------------------------------------------
...
from scipy.special cimport cython_special
^
------------------------------------------------------------

SciPy/NumPy/Python version and system information
SciPy 1.11.2
Meson 0.13.2

Thanks for the report. 0.13.2 is the meson-python version. Can you add the meson and Cython versions? This should be automatic, but it's very well possible that we're missing an explicit dependency here and that triggering the resulting problem is difficult.

So I tracked it down to these two rules which should reproduce anywhere (may need a ninja -t clean first depending on how you get your builddir and hacking mesonpy (see Bartpab/meson-py#4) to get access to such a build directory):
% <generate meson build>
% ninja scipy/special/_ellip_harm_2.cpython-39-darwin.so.p/_ellip_harm_2.c
<fails as above>
% ninja scipy/special/cython_special.cpython-39-darwin.so
<works>
% ninja scipy/special/_ellip_harm_2.cpython-39-darwin.so.p/_ellip_harm_2.c
<now works>

Thanks! Yes, this is a clear bug. I've had on my wish/idea list for a long time to build every target separately to smoke out issues like these.
Here's how I reproduce it, from a clean repo:
$ meson setup build  # configure the build dir
$ # find an exact target name:
$ ninja -C build -t targets depth 0 | rg ".*ellip_harm.*.so"
$ # try to build only that target:
$ ninja -C build scipy/special/_ellip_harm_2.cpython-310-x86_64-linux-gnu.so
ninja: Entering directory `build'
[1/3] Generating 'scipy/special/_ellip_harm_2.cpython-310-x86_64-linux-gnu.so.p/_ellip_harm_2.c'.
FAILED: scipy/special/_ellip_harm_2.cpython-310-x86_64-linux-gnu.so.p/_ellip_harm_2.c
/home/rgommers/mambaforge/envs/scipy-dev/bin/cython -3 --fast-fail --output-file scipy/special/_ellip_harm_2.cpython-310-x86_64-linux-gnu.so.p/_ellip_harm_2.c --include-dir . ../scipy/special/_ellip_harm_2.pyx

Error compiling Cython file:
------------------------------------------------------------
...
from scipy.special cimport cython_special
^
------------------------------------------------------------

/home/rgommers/code/scipy/scipy/special.pxd:1:0: 'scipy/special/cython_special.pxd' not found
ninja: build stopped: subcommand failed.

Meson is good at automatically adding dependencies, but it will miss the ones that come through a custom_target where we use a Python script to generate Cython source files which are then needed for a next target.

I'm unsure of how many places need this added, but I suppose grepping pyx files for scipy imports would be a good guide.

I think it's only the places that use the output of a _generate_pyx.py script - that's easy to grep for.