how about some informed network protection?

Because every device WILL have vulnerabilities, it might be useful to have a declarative model for consumer devices that can be used to configure access at the home router.  I've worked on just such a thing at the IETF, known as manufacturer usage descriptions (draft-ietf-opsawg-mud) that has recently been approved as a proposed standard, that can be used to identify communication patterns that the device is intended to general.  The point is that a thermostat should probably not be trying to access Twitter (by way of example).  Having manufacturers have such descriptions can help identify when the device is misbehaving because it has been broken into.

Thanks @elear! It sounds like this would fall neatly into our Security section. Would you like to propose some language to help us get it into the standard? If I were to frame this from the consumer's perspective, I might say something like, "the device connects only to things that it needs to" or maybe, "there are systems in place to help identify when a device connects to something it shouldn't."
What do you think?

I might even suggest we do a couple of things to include this kind of thinking:

Rename "Known Exploit Resistance" to "Exploit Resistance", and,
Add an indicator that says, "Where applicable, the communication patterns of a device/service are published to help spot suspicious activity." and,
Add another indicator that says, "Where applicable, connected devices with known communication patterns are monitored for suspicious activity."

I think having (2) and (3) separately makes sense, since I think there are two distinct roles involved in the process of monitoring. But, we could also say something less specific.
By the way, what is the standard you worked on? I would love to have a read through it.

Hey @elear--does what Bobby recommended on Aug. 15 2018 accord with your proposed improvement to the Standard? if not, lets continue discussing this :)

thanks for the poke.  i'll take a look in the next day or two

Hi and my apologies for my delay.  I like @secretrobotron's suggested text (August 15, 2018), and might further suggest a procedure, that devices make use of RFC 8520 to declare what network access they need (and don't need).


Exclude water from outline

I'd like the water to not have outlines

The areas without ground underneath show without outline but I need the ground there.
I tried looking in the documentation but it only specifies the Stylized Surface Shader and not the Water Shader.

You can set the Water to transparent rendering mode as well.

Then you need to set the outlines to be rendered before transparents (see 'Render Event' in the doc: https://flatkit.dustyroom.com/outlines/#advanced-parameters)

That worked. I did have to run the scene in order for the outline to update in the scene view though.


Getting both iframes and mailto links to work

Bug description:
mailto links will not work with the following originWhitelist but iFrames will.
<WebView
originWhitelist={['*']}
....
/>
I can get mailto links to work with the following, but IFrames will not (error: can't open url: about:srcdoc)
<WebView
originWhitelist={['http://', 'https://', 'mailto://*',  ]}
....
/>
If I try to add the wildcard, iFrames work, mailto does not.
<WebView
originWhitelist={['http://', 'https://', 'mailto://', '' ]}
....
/>
How can I get both an iframe and mailto working together?
Please note:  the iframe and mailto are separate.  In other words, the mailto link is not within the iframe.  It's on its own page.
Thank you in advance for any help.

I figured it out using onShouldStartLoadWithRequest.  I got the idea from this post:
https://stackoverflow.com/questions/60493866/onshouldstartloadwithrequest-automatically-calling-in-ios-react-native-on-loadin
Here's a working example should anyone else run into this issue.  You can swap out "mailto" for anything, make sure to adjust the slice though.  Also, test on a real device, this will fail on an emulator.
<WebView
    originWhitelist={[ '*']}
    onShouldStartLoadWithRequest={event => {
      if (event.url.slice(0, 6) === 'mailto') {

            Linking.openURL(event.url);
            return false
      }
      return true;
      }}
    source={{ uri: 'https://stackoverflow.com', }}
/>


MINOR: Fix some flaky behavior in DeleteConsumerGroupsTest and clean up

Create a common waitForGroupState function to avoid duplicated code.  Also print out the difference between the actual and expected state in that function.
In a few tests, wait until the groups are in Stable state before proceeding.  If we don't do this, sometimes we try to tear the group down before it ever becomes stable, and get stuck waiting for a rebalance timeout (which is too long to allow the test to pass)

create a common waitForGroupState function to avoid duplicated code.  Also print out the difference between the actual and expected state in that function.

In a few tests, wait until the groups are in Stable state before proceeding.  If we don't do this, sometimes we try to tear the group down before it ever becomes stable, and get stuck waiting for a rebalance timeout (which is too long to allow the test to pass)

test failure is not related

the cleanup LGTM
for another, a part of this fix was included by #8032. Could you rebase the code?


Error when adding ajax complete

When adding complete property in ajax callbacks, the function added as a callback is not called

This issue is going to be fixed on the next release 2.0.0 and with others new features that is coming soon.

This issue was resolved in v1.1.0 already released.


CSHARP-1888: ServerMonitor should use HeartbeatTimeout for socket read/write timeouts.

Evergreen patch build:
https://evergreen.mongodb.com/version/5bdb378fe3c33106fbed6d4f

New Evergreen patch build after latest changes:
https://evergreen.mongodb.com/version/5bdc983ce3c33162379d331d

Squashed, rebased and merged.


Apple support for ign command line tool

Original report (archived issue) by Nate Koenig (Bitbucket: Nathan Koenig).

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked the Q&A board for common solutions: http://answers.gazebosim.org
Checked that your issue isn't already filed.
Checked that there is not already an Ignition package that provides the described functionality: https://ignitionrobotics.org/libs

Description
The UNIT_ign_TEST fails on Apple, through homebrew, when using the ign gazebo command line tool. In general, the ign gazebo command line tool fails to load the libignition-gazebo2-ign.dylib. Other packages, such as ign-transport and ign-msgs work fine. There might be a problem associated with loading a component library.
Steps to Reproduce

Change the src/ign_TEST.cc to use ign gazebo instead of ign-gazebo.
Install gazebo on homebrew
Run the /bin/UNIT_ign_TEST test.

Expected behavior:
The test should pass.
Actual behavior:
The test fails
Reproduces how often:
All the time.
Versions
Gazebo 2+
Additional Information

Pull request #296
Pull request #336

I have been successful with ign gazebo -s when invoked with the homebrew version of ruby and have added this as a test command to the ignition-gazebo* formulae in https://github.com/osrf/homebrew-simulation/pull/1221

I am debugging the UNIT_ign_TEST on macOS to see if we can enable at least part of it

I've enabled UNIT_ign_TEST on macOS in https://github.com/ignitionrobotics/ign-gazebo/pull/477 by making use of the homebrew ruby formula. It also should provide more helpful error messages if a library fails to load

Closed by #477


Migrate to ForkJoinPool for master RPCs

@bf8086 @calvinjia PTAL.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2986/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2987/
Test PASSed.

Some numbers before/after this change.
Overall it's all within our expectations.
Operation         Before       After     Change
Create              10671        9281      - %13
Create(HDFS)  9694        10182     + %5
List-File            26586      45569   + %71
List-Dir             3482        4419      + %26
Delete              26604      23675    - %11
Delete(HDFS)  22415       23794    + %6
List-Missing     17950       21692    + %20

@calvinjia @bf8086 Please check if this is enough https://github.com/Alluxio/alluxio/pull/8827/commits/ace358d37f87eb83fc294e48e73b222da6eec02e

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3021/
Test PASSed.

alluxio-bot, merge this please.


sound file duration

Hi, If I want to extract a music file with duration of 2s.
Can I use compact_cnn to extract the music sound features? I see the duration in your code is 29.s.

In short, no, but probably you can use https://github.com/keunwoochoi/transfer_learning_music, especially https://github.com/keunwoochoi/transfer_learning_music/tree/master/keras2_model this one. Check out the ipython notebook file, where the model is defined with varying input temporal length.


Could I use dynamic schema?

import {Mongo} from 'meteor/mongo';
import {SimpleSchema} from 'meteor/aldeed:simple-schema';

export const Client = new Mongo.Collection("client");

// Optional is ture
Schema1 = new SimpleSchema({
    name: {
        type: String,
        label: 'Name',
        optional: true,
    }
......
});
---------
// Required
Schema2 = new SimpleSchema({
    name: {
        type: String,
        label: 'Name',
    }
......
});

// Check setting collection
.....
if(getSetting.optional === true){
   Client.attachSchema(Schema1)
}else{
   Client.attachSchema(Schema2)
}

Please help me.

Once a schema attached to a collection, the schema will be a property of that collection. If you need to put it dynamically, then you need to wrap the collection instance inside conditional statement you defined above. Ex:
if(getSetting.optional === true){
client = new Collection('client');
client.attachSchema(Schema1);
};
Or if putting it that way bothers you, you should consider using the schema on validation step, without the need to attach it to the schema.
Hope it helps :)


Prisma Schema conventions

Hello!
I suggest we follow a few conventions with the Prisma schema with Postgres and, generally, a SQL database/dialect.
@map attribute
Using the @map attribute, any fields named in the schema using camelCase should be mapped to their respective column name using snake_case.
For example, with #37, there's a plan to change the field <User>.name to <User>.displayName.
We can add the attribute @map("display_name") to commit this change to the database.
As most of us with Prisma knowledge know, we still use .displayName in the schema and Prisma Client. But display_name would be used in the database instead.
@@map
This is very similar to the @map attribute. However, with @@map, we map the model name to its respective database table name to follow the snake_case standard.
We currently have a User model defined. Following this standard, we can place @@map("users") at the end of the model to prepare to commit these changes.
These are the conventions I have in mind right now. As the repo progresses, I'll watch for changes to the schema!

Reference: Prisma schema reference

Right, I forgot about the names having to be snake_case in Prisma, that's my bad. I'll keep that in mind!

I just got this done. Working on pulling.

This has been merged and can be closed.


CON-4735 Use addCategory instead of setCategories

In some cases we got error: QLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '639-7' for key 'articleID' in
With addCategory method this is not possible, because it will check if category is already added.

Coverage increased (+0.03%) to 34.396% when pulling aac4ec717372f8173dd623e665ac36f8214c95c0 on CON-4735 into a48e7d64bdf6786df42b63fdc7403d5c1adab3a1 on master.

Coverage increased (+0.04%) to 34.408% when pulling f303ad7d0192fadd2afc6a2d2198888e8b6c935e on CON-4735 into a48e7d64bdf6786df42b63fdc7403d5c1adab3a1 on master.


AccessTokenValidation broken after System.IdentityModel.Tokens.Jwt 5.0.0 (6/27/2016) update

After an upgrade of  System.IdentityModel.Tokens.Jwt 5.0.0 (6/27/2016) due to another project dependency; a project using accesstokenvalidation fails to complete it's owin startup. I went looking and it seems this new 5.0.0 version has moved TokenValidationParameters to the Microsoft namespace (was previously under System)
Stacktrace:
System.TypeLoadException: Could not load type 'System.IdentityModel.Tokens.TokenValidationParameters' from assembly 'System.IdentityModel.Tokens.Jwt, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
at Owin.IdentityServerBearerTokenValidationAppBuilderExtensions.<>c__DisplayClass9.b__8()
at System.Lazy1.CreateValue()    at System.Lazy1.LazyInitValue()
at Owin.IdentityServerBearerTokenValidationAppBuilderExtensions.UseIdentityServerBearerTokenAuthentication(IAppBuilder app, IdentityServerBearerTokenAuthenticationOptions options) in c:\local\identity\server3\AccessTokenValidation\source\AccessTokenValidation\IdentityServerBearerTokenValidationAppBuilderExtensions.cs:line 46
at projectabc.Startup.Configuration(IAppBuilder app) in C:\dev\projectabc\Startup.cs:line 39

There is not a lot we can do. We are only wrapping the MS JWT middleware.
They don't seem to be compatible with the new System.IdentityModel version. Open an issue in Katana.

Let us know if Microsoft wants to update their MW - so we can use their new version. closing for now.

@ceastwood What is the project dependency that needs 5.0?


Add failsafe to Resolve-DbaNetworkName

fixes #4680 - @SQLDBAWithABeard give this a whirl, please and let me know if it works for multiples being passed in.

Only just seen this, I can test tomorrow bit it looks ok at first glance

np, i tested it against a couple things (tho not your situation) and it worked so merged

Thank you :-) You is Amazing


Wrong domain name in repository url

I've run my own gitea instance for a long time and it works fine all the time until today, I git pulled the latest master branch from here. And the URL of every repo becomes something like git@localhost:USER/REPO.git, which should be git@MYDOMAIN:USER/REPO.git.
In custom/conf/app.ini , I've set DOMAIN to my domain, and it worked before. What's wrong? Is there something changes in the config file?

Gitea version (or commit ref): b7e1bccc501c5246e6f8b9bed4853b90e9a447c5
Git version: 2.10
Operating system: Ubuntu 16.04 X64
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Thanks

Same here, it seems only "ssh" link is affected.

Same here, it seems only "ssh" link is affected.

OK, it works after adding SSH_DOMAIN to my custom/conf/app.ini, in the server section.
Thanks

Would be nice to see this added in the config cheat sheer (https://docs.gitea.io/en-us/config-cheat-sheet/). I ran into this issue and was able to find it here, but would have found it more quickly if it was documented.

@guyspronck This PR https://github.com/go-gitea/docs/pull/75
cc @lunny @tboerger


chore(main): release 0.4.4

:robot: I have created a release beep boop
0.4.4 (2024-12-14)
Bug Fixes

wg-sudoers path to iptables fixed (b3926e3)
wg-sudoers path to iptables fixed (#102) (adcd06e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

0.4.4
:sunflower:


Fix plate scale units - [merged]

In GitLab by @wtbarnes on Feb 26, 2020, 21:39
Merges fix-plate-scale-units -> master
The units on the instrument plate scale were pixel / steradian when they should be the other way around.

In GitLab by @wtbarnes on Feb 26, 2020, 21:39
enabled an automatic merge when the pipeline for 3874bf289b5f6c20b7c0baa1c461eef26ac2ee7c succeeds

In GitLab by @wtbarnes on Feb 26, 2020, 21:45
merged

In GitLab by @wtbarnes on Mar 23, 2020, 13:30
mentioned in commit 56376e226203ae3cf2ab6ae76512a0dda090d4d0

In GitLab by @wtbarnes on Feb 26, 2020, 21:45
mentioned in commit c2d5ca8e11af73812944be44401d9b6fa9adae60


Improved Homepage to Allow Users to View Oldcarts

Main changes:

gen.py - made minor modifications to how OldCartContent data was generated
oldCartModel.py and oldCartContentModel.py - include additional methods to allow retrieval of the three recent oldCarts and their respective contents
home.py and home.html - made changes such that users can see their three recent oldCarts and click to see their respective contents.

Change PR name to Add Old Carts to Home Page


File upload not working with absolute URI

I think this issue might be the same one listed in #612.  Since it hasn't moved in about a month I wanted to double check if my issue was the same.
I have a file that exists on the same machine that hosts a 3rd party site I depend on.  I want to upload that file to the 3rd party site so it gets deployed.
var path = 'file:///C:/Program%20Files/App/bin/myFile.txt';
element(by.id('myFileInput')).sendKeys(path);

When I run this I get:
'Failed: unknown error: path is not absolute: file:///C:/Program%20Files/App/bin/myFile.txt'
The site is non angular, but I wouldn't think that would cause this.  I have checked pasting the URI straight into chrome and that seems to open the file just fine.

What happens if you get rid of file:///
var path = 'C:/Program%20Files/App/bin/myFile.txt';
element(by.id('myFileInput')).sendKeys(path);

Thanks! it looks like that makes the error go away.
Clicking the submit button doesn't seem to want to upload the file for some reason. On further investigation I think it's because the 3rd party upload form is trying to do some fancy validation that doesn't play nicely with Protractor.  Anyway, Thanks for your help!


Throw on upload limit reached

I don't think reaching the upload limit should be a simple log, we should probably throw a documented exception after closing the browser
https://github.com/fawazahmed0/youtube-uploader/blob/db8c5b14f655b105d7ab89b67d8b186fd1822b9e/src/upload.ts#L171-L174

I've also noticed the 0.5 seconds timeout is too short on some occasions. 2 seconds did the trick for me but we should probably handle this some other way. Unsure if waiting indefinitely for 'Upload complete' is what we want
Maybe we could Promise.race the selectors for both the 'Daily upload limit' and 'Upload complete' elements. Will have to look further into this
https://github.com/fawazahmed0/youtube-uploader/blob/c34d632d852c4210855a7fdc014413eb40301021/src/upload.ts#L176

I guess this should be closed with this PR


serving install via olm fails

Describe the bug
serving install via olm fails with the following error:
      Install failed with message: clusterrolebindings.rbac.authorization.k8s.io "knative-serving-controller-admin" is forbidden: user "system:serviceaccount:operators:knative-operator" (groups=["system:serviceaccounts" "system:serviceaccounts:operators" "system:authenticated"]) is attempting to grant RBAC permissions not currently held:
      {APIGroups:[""], Resources:["configmaps"], Verbs:["update" "patch"]}
      {APIGroups:[""], Resources:["endpoints"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:[""], Resources:["endpoints/restricted"], Verbs:["create"]}
      {APIGroups:[""], Resources:["events"], Verbs:["get" "list" "delete" "watch"]}
      {APIGroups:[""], Resources:["namespaces"], Verbs:["list" "create" "delete" "patch"]}
      {APIGroups:[""], Resources:["namespaces/finalizers"], Verbs:["update"]}
      {APIGroups:[""], Resources:["pods"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:[""], Resources:["secrets"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:[""], Resources:["serviceaccounts"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:[""], Resources:["services"], Verbs:["update" "patch"]}
      {APIGroups:["admissionregistration.k8s.io"], Resources:["mutatingwebhookconfigurations"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:["admissionregistration.k8s.io"], Resources:["validatingwebhookconfigurations"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:["apiextensions.k8s.io"], Resources:["customresourcedefinitions/status"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:["apps"], Resources:["deployments"], Verbs:["patch"]}
      {APIGroups:["apps"], Resources:["deployments/finalizers"], Verbs:["get" "list" "create" "update" "delete" "patch" "watch"]}
      {APIGroups:["autoscaling"], Resources:["horizontalpodautoscalers"], Verbs:["patch" "watch"]}
      {APIGroups:["autoscaling.internal.knative.dev"], Resources:["*"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["autoscaling.internal.knative.dev"], Resources:["*/finalizers"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["autoscaling.internal.knative.dev"], Resources:["*/status"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["networking.internal.knative.dev"], Resources:["*"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["networking.internal.knative.dev"], Resources:["*/finalizers"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["networking.internal.knative.dev"], Resources:["*/status"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["serving.knative.dev"], Resources:["*"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["serving.knative.dev"], Resources:["*/finalizers"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}
      {APIGroups:["serving.knative.dev"], Resources:["*/status"], Verbs:["get" "list" "create" "update" "delete" "deletecollection" "patch" "watch"]}

Expected behavior
serving install via olm succeeds
To Reproduce
minikube start
operator-sdk olm install
kubectl create -f https://operatorhub.io/install/knative-operator.yaml
kubectl apply --filename - <<EOF
apiVersion: v1
kind: Namespace
metadata:
  name: knative-serving
---
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeServing
metadata:
  name: knative-serving
  namespace: knative-serving
spec:
  ingress:
    kourier:
      enabled: true
  config:
    network:
      ingress.class: "kourier.ingress.networking.knative.dev"
EOF

Knative release version
v0.25.0
Additional context
I attempted with the following versions:
olm v0.18.3
kubernetes v1.22.1, v1.21.4
Note that install succeeds if operator is installed directly rather than via olm.

/assign

@mjgallag have you given the 1.0 a try recently?

@houshengbo just tried now with 1.1, same error.
k3d cluster create
operator-sdk olm install
kubectl create -f https://operatorhub.io/install/knative-operator.yaml
kubectl apply --filename - <<EOF
apiVersion: v1
kind: Namespace
metadata:
  name: knative-serving
---
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeServing
metadata:
  name: knative-serving
  namespace: knative-serving
spec:
  ingress:
    kourier:
      enabled: true
  config:
    network:
      ingress.class: kourier.ingress.networking.knative.dev
EOF

It has been a long time since this bug was opened.
Operator of knative has evolved into 1.3 in the operatorhub.
If similar issues still exist, welcome to reopen it.

Hi @mathianasj we also encountered this problem on our 1.7 installation. Did you figure out a solution?

same problem with 1.8.1

@houshengbo

1.9.0 still occuring

/reopen

/remove-lifecycle stale

@houshengbo it seems this is still happening

Seems this is still an issue. Anyone find a workaround?

Certainly not ideal but after ending up in a manual adding of RBAC permissions loop (and having tried to add cluster-admin privileges) and nothing working, the below seemed to work
kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
metadata:
 name: knative-operator-missing-rbacs
rules:
- apiGroups: ["*"]
  resources: ["*"]
  verbs: ["*"]
---
kind: ClusterRoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
 name: knative-operator-missing-rbacs-binding
subjects:
- kind: ServiceAccount
  namespace: [INSERT NAMESPACE]
  name: knative-operator
roleRef:
 kind: ClusterRole
 name: knative-operator-missing-rbacs
 apiGroup: rbac.authorization.k8s.io

Obviously wouldn't be appropriate for production but seems to work for testing at least.


display page title instead of breadcrumbs

This MR to display page title instead of breadcrumbs for example if page title now rendered as Cortex  >> Members it'll be Members only

And why do we need this? Having the website title as part of the page title is always a good SEO practice, unless you've a better reasoning.. 🤔


ariaDownload.py checks for .netrc or takes input for username and password from command line

Is your feature request related to a problem? Please describe.
One can't pass a username and password but needs to enter manually when prompted.
Describe the solution you'd like
Have an option to pass username and password in the command line and/or go check the .netrc file

Yes, the ISCE package gets the EarthData login information from $HOME/.netrc, which is a lot easier to use.

Currently, we are leveraging ASF's bulk download script, which does a couple nice things like handling partial downloads. I'm quite sure there is no way to pass credentials through the ariaDownload.py into the underlying bulk download script. However, there should be a way to have it read the .netrc. I spent some time recently working on it but was stumped and received radio silence from a query to ASF. Perhaps @davidbekaert has a contact I can try?
Alternatively, it is relatively simple to change ariaDownload.py to directly use wget or curl (rather than wrapping around the bulk download script). By default, these read a .netrc and it would be trivial to implement manual credentials.

Look at the closed PR on ISCE for staging earthdata products - was a variant of ASF's old bulk downloader. Uses netrc module to load info from .netrc and authenticate

@bbuzz31 I would like to ensure we keep on using the ful DAAC api capabilities for checksums etc. We jsut need to make sure the cookie is made in advance. the PR  @piyushrpt is referring too is this one: https://github.com/isce-framework/isce2/pull/66

Would it be possible to have two options? The interactive username and password entry is a big hassle for Jupyter notebooks and prevents any batch processing.

@piyushrpt @dbekaert thanks for pointing me to the ISCE PR. That was very helpful, and I was able to implement it into our ariaDownload wrapper. I will implement credential passing so both options are available and issue a pull request in the next couple days.

Would it be possible to have two options? The interactive username and password entry is a big hassle for Jupyter notebooks and prevents any batch processing.

That is the plan.


Remove proto nesting so that the Swift protos get generated correctly

I dont think we need to remove the nesting to properly generate wire for swift. Here's the patch I drafted.
patch.txt


Adding custom models removes albedo texture and changes color to white

I've added some custom hand models to my controllers--in addition to the default vive controllers not disappearing, it also changes the albedo textures to white on the hands. Clicking the grip buttons alters their translucency, but they stay white.

I've worked around this by changing line 698 in NVRHand.cs from:
Color transparentcolor = Color.white;
to:
Color transparentcolor = CustomModel.GetComponent<Renderer>().sharedMaterial.color;

Closing, as this will be corrected and expanded as an added feature in NewtonVR, see Trello card (https://trello.com/c/lHaOStSX)


Migrate off of npm:debug

It's not on JSR and I don't want to rely on npm anymore

ref https://github.com/debug-js/debug/issues/972

ref https://jsr.io/@grammyjs/debug
/cc @wojpawlik


Run validation on a test set

Hello, I trained 5 folds of the LUNA16 task, now I have another dataset, with images and mask, and I want to run evaluation on this dataset using the trained models. I ran nndet_predict, and obtain the folder test_predictions on those images, now I try to run nndet_eval and the code ask for masks processed in preprocessed/labelsTs.
How can I make this folder?
Thanks,
Van KHoa

Hi @vankhoa21991 ,
sorry for the late response. labelsTs is generated in the preprocessing stage by nndet_prep. I'll add an additional entry point so it is possible to generate them afterwards.
Best,
Michael


drivers/pcie: Change pcie_get_mbar() to return size and flags

currently pcie_get_mbar only returns the physical address.
This changes the function to return the size of the mbar and
the flags (IO Bar vs MEM BAR).
This implements #26491. I did decide to change the existing function instead of adding another function to retrieve the size for the following reasons::

in most of the cases both size and physical address are required anyways
The flags might are interesting aswell e.g. to select IO Bar vs Mem Bar

This fixes the macros for the use with IO Bars

Actually, about differentiating IO and MEM. I don't know if that's relevant anymore since no platform so far make use IO stuff. We only needed it for galileo board back in the days.

Actually, about differentiating IO and MEM. I don't know if that's relevant anymore since no platform so far make use of IO stuff. We only needed it for galileo board back in the days.

I personally only care about this info, since e.g. The Realtek 8111G Controller has the following bars

address=0xE000 size=256 type=IO
address=0x91404000 size=4k type=MEM (64 bit)
address=0x91400000 size=16k type=MEM (64 bit)

I want to use the first MEM Bar. So I either need this info, or since as you said nobody uses the IO stuff, we could skip all IO Bars aswell.

I want to use the first MEM Bar. So I either need this info, or since as you said nobody uses the IO stuff, we could skip all IO Bars aswell.
Yes somes devices expose both types for legacy support, but the MEM is the way to go in zephyr so far, so skipping IO bars would be fine

Yes somes devices expose both types for legacy support, but the MEM is the way to go in zephyr so far, so skipping IO bars would be fine

Ok then I will simply skip I/O Bars

@tbursztyka I applied the changes, but I do not really understand why the doc build is failing.

@jhedberg Can you merge this one? I need that function for msi-x and ivshmem

Maybe proposing a function in parallel to the existing one would have been more aligned with https://docs.zephyrproject.org/latest/development_process/api_lifecycle.html


Use Github apps

The github provider uses personal access tokens, deploy keys and webhooks.  We also support github apps.  We should use the Github Apps provider once that API settles down.  Perhaps this could be a good opportunity to coordinate with API team to get these flows integrated behind an API.

Dupe of https://github.com/netlify/cli/issues/193


No valid convolution algorithms available in CuDNN

Dear Hans,
First of all, I would like to congratulate you on your work, I'm impressed by the methodology and results!
I'm happily adapting your neural network to another pathology imaging modality with the purpose of classifying patients to clinical subject-level labels. Due to the chosen imaging modality, I cannot use the pretrained ResNet that is chosen as the default configuration in your GitHub. Alternatively, I have to train all neural network layers, or, in other words, I need to set train_streaming_layer=True and train_all_layers=True.
When these parameters are set to True, unfortunately, the running process stops and an exception is raised at cpp_functions.backward function (line 221 from scnn.py):
Exception trace:
no valid convolution algorithms available in CuDNN
File "/gpu-data/djsanchez/pathology-streaming-pipeline/torch_utils/checkpointed_trainer.py", line 174, in backward_batch
if self.mixedprecision: self.grad_scaler.scale(loss).backward()
File "/gpu-data/djsanchez/pathology-streaming-pipeline/torch_utils/checkpointed_trainer.py", line 124, in train_checkpointed_batch
self.backward_batch(loss, fmap)
File "/gpu-data/djsanchez/pathology-streaming-pipeline/torch_utils/checkpointed_trainer.py", line 114, in train_batch_if_needed
self.train_checkpointed_batch()
File "/gpu-data/djsanchez/pathology-streaming-pipeline/torch_utils/checkpointed_trainer.py", line 77, in train_full_dataloader
self.train_batch_if_needed()
File "/gpu-data/djsanchez/pathology-streaming-pipeline/torch_utils/trainer.py", line 171, in train_epoch
self.train_full_dataloader(batch_callback)
File "/gpu-data/djsanchez/pathology-streaming-pipeline/train_plex.py", line 145, in train_epoch
preds, gt = trainer.train_epoch(self._train_batch_callback)
File "/gpu-data/djsanchez/pathology-streaming-pipeline/train_plex.py", line 139, in train_and_eval_epochs
self.train_epoch(e, self.trainer)
File "/gpu-data/djsanchez/pathology-streaming-pipeline/train_plex.py", line 89, in run_experiment
else: self.train_and_eval_epochs()
File "/gpu-data/djsanchez/pathology-streaming-pipeline/train_plex.py", line 533, in
exp.run_experiment()
I understand that something is happening with the cpp_functions, but I cannot tell what, and I don't know how to continue. For this reason, I would really appreciate your help solving this issue and helping me out to run your neural network to train all layers.
Thanks in advance,
Daniel Jiménez-Sánchez.

Hi Daniel,
Can you try setting the env variable CUDA_LAUNCH_BLOCKING=1?
Also, it seems to happen in the second stage of your neural network (the not streaming part). How does that part of the network look like?
It can also happen with streaming when your tile size is not large enough, but then the error should not happen at this stage but earlier.
Cheers,
Hans

Hello Hans,
I set the env variable CUDA_LAUNCH_BLOCKING=1 as suggested, but unfortunately, the network is not working and the 'no valid convolution algorithms available in CuDNN' error is still raised. I created a fresh start with a new Conda environment, but still, I'm getting the same error.
I am using the default network with the only exception of changing train_all_layers=True, and pretrained=False. I tried different patch sizes, one of them being as big as 1800, but got the same error.
Do you have any further suggestions?
Thanks!
Daniel Jiménez-Sánchez.

Hi Daniel,
I’m not working today so can’t check the code myself. Can take a look
tomorrow. Which platform are you? Does normal pytorch code train without
problems?
Thanks,
Hans
On Thursday, September 23, 2021, Daniel Jimenez-Sanchez <
@.***> wrote:

Hello Hans,
I set the env variable CUDA_LAUNCH_BLOCKING=1 as suggested, but
unfortunately, the network is not working and the 'no valid convolution
algorithms available in CuDNN' error is still raised. I created a fresh
start with a new Conda environment, but still, I'm getting the same error.
I am using the default network with the only exception of changing
train_all_layers=True, and pretrained=False. I tried different patch sizes,
one of them being as big as 1800, but got the same error.
Do you have any further suggestions?
Thanks!
Daniel Jiménez-Sánchez.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DIAGNijmegen/pathology-streaming-pipeline/issues/6#issuecomment-925559775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARIWZPTZVUTVGH2A4MYW3UDLHZLANCNFSM5ERVE6OA
.

Hi,
Yes, I will give you all the information concerning my setup. It worked with several PyTorch projects, but I never used custom .cpp PyTorch functions.
Working environment:
PyTorch version: 1.6.0
CUDA used to build PyTorch: 10.1
ROCM used to build PyTorch: N/A
OS: Ubuntu 16.04.6 LTS (x86_64)
GCC version: (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
Libc version: glibc-2.17
Python version: 3.6.13 |Anaconda, Inc.| (default, Jun  4 2021, 14:25:59)  [GCC 7.5.0] (64-bit runtime)
Python platform: Linux-4.4.0-150-generic-x86_64-with-debian-stretch-sid
Is CUDA available: True
CUDA runtime version: 10.1.168
GPU models and configuration:
GPU 0: GeForce RTX 2080 Ti
GPU 1: GeForce RTX 2080 Ti
GPU 2: GeForce RTX 2080 Ti
GPU 3: GeForce RTX 2080 Ti
Nvidia driver version: 418.67
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.0.5
Versions of relevant libraries:
[pip3] torch==1.6.0
[pip3] torchvision==0.7.0
[conda] blas                      1.0                         mkl
[conda] cudatoolkit               10.1.243             h6bb024c_0
[conda] mkl                       2020.2                      256
[conda] mkl-service               2.3.0            py36he8ac12f_0
[conda] mkl_fft                   1.3.0            py36h54f3939_0
[conda] mkl_random                1.1.1            py36h0573a6f_0
[conda] numpy                     1.19.2           py36h54aff64_0
[conda] numpy-base                1.19.2           py36hfa32c7d_0
[conda] pytorch                   1.6.0           py3.6_cuda10.1.243_cudnn7.6.3_0    pytorch
[conda] torchvision               0.7.0                py36_cu101    pytorch
I hope this helps!

I only see a discrepancy in cudnn versions between pytorch built and
installed. However, not sure if this would be a problem. Does regular
Pytorch code work in the same environment?
Thanks,
Hans
On Thursday, September 23, 2021, Daniel Jimenez-Sanchez <
@.***> wrote:

Hi,
Yes, I will give you all the information concerning my setup. It worked
with several PyTorch projects, but I never used custom .cpp PyTorch
functions.
Working environment:
PyTorch version: 1.6.0
CUDA used to build PyTorch: 10.1
ROCM used to build PyTorch: N/A
OS: Ubuntu 16.04.6 LTS (x86_64)
GCC version: (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
Libc version: glibc-2.17
Python version: 3.6.13 |Anaconda, Inc.| (default, Jun 4 2021, 14:25:59)
[GCC 7.5.0] (64-bit runtime)
Python platform: Linux-4.4.0-150-generic-x86_64-with-debian-stretch-sid
Is CUDA available: True
CUDA runtime version: 10.1.168
GPU models and configuration:
GPU 0: GeForce RTX 2080 Ti
GPU 1: GeForce RTX 2080 Ti
GPU 2: GeForce RTX 2080 Ti
GPU 3: GeForce RTX 2080 Ti
Nvidia driver version: 418.67
cuDNN version: /usr/lib/x86_64-linux-gnu/libcudnn.so.7.0.5
Versions of relevant libraries:
[pip3] torch==1.6.0
[pip3] torchvision==0.7.0
[conda] blas 1.0 mkl
[conda] cudatoolkit 10.1.243 h6bb024c_0
[conda] mkl 2020.2 256
[conda] mkl-service 2.3.0 py36he8ac12f_0
[conda] mkl_fft 1.3.0 py36h54f3939_0
[conda] mkl_random 1.1.1 py36h0573a6f_0
[conda] numpy 1.19.2 py36h54aff64_0
[conda] numpy-base 1.19.2 py36hfa32c7d_0
[conda] pytorch 1.6.0 py3.6_cuda10.1.243_cudnn7.6.3_0 pytorch
[conda] torchvision 0.7.0 py36_cu101 pytorch
I hope this helps!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DIAGNijmegen/pathology-streaming-pipeline/issues/6#issuecomment-925629736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARIW6RNQXQAWS5WNE4ODTUDLUVFANCNFSM5ERVE6OA
.

Hi Daniel,
Sorry, I didn't have time anymore, I will go on vacation on Monday, so will not be able to help hands-on. CUDA_LAUNCH_BLOCKING=1 sometimes makes the error message more clear.
You could maybe try a more recent PyTorch version.
Cheers,
Hans

Hi Hans,
I would like to reply to your previous suggestions:

Does regular Pytorch code work in the same environment?

Yes regular PyTorch code is working. Indeed, if the model is trained on the last layers it works perfectly.

You could maybe try a more recent PyTorch version.

To do so I need to update all drivers. I would need some extra time to do so because I'm using our GPU server to carry out other experiments.

If you have any other suggestions, please write.
Thank you very much for your efforts!
Daniel Jiménez.


Update jena-shex to 4.9.0

About this PR
📦 Updates org.apache.jena:jena-shex from 4.3.2 to 4.9.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (4.3.2).
You might want to review and update them manually.
build.sbt

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.jena", artifactId = "jena-shex" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.jena", artifactId = "jena-shex" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #526.


Anger_level, incorrect documentation and missing properties

Thanks to: https://github.com/Blockception/Minecraft-bedrock-json-schemas/pull/175
But default_annoyingness can be number but is specified as string
default_projectile_annoyingness is missing but can be found in the warden

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

ID: 22b035f0-de4b-0cb1-a27a-2fb2d8209447
Version Independent ID: 22b035f0-de4b-0cb1-a27a-2fb2d8209447
Content: Entity Documentation - minecraft:anger_level
Content Source: creator/Reference/Content/EntityReference/Examples/EntityComponents/minecraftComponent_anger_level.md
Product: gaming
GitHub Login: @mammerla
Microsoft Alias: v-jimseaman

@DaanV2 Good catch! Changes are made and should be visible after the next release. Thanks!


License of dependent package is unknown

Please update the crypto-browserify version of the dependent packages to the latest.
License of<EMAIL_ADDRESS>of dependent packages of<EMAIL_ADDRESS>is unknown.
It will be solved by the crypto-browserify to the latest version.

+1

This has been resolved (for Webpack 2) at some point. crypto-browserify now includes<EMAIL_ADDRESS>which defines the license as the 3-Clause BSD.

We will bump node-libs-browser to ^1.1.0 in webpack-1, see https://github.com/webpack/webpack/issues/3355

There was a hidden breaking change with crypto-browserify so that we couldn't upgrade it to the most recent version. However, this PR introduces the most recent versions without breaking change to webpack-1.


Added cpp solution of rotting oranges problem

This PR addresses the issue #48.

It contains a well structured solution of the Rotting oranges problem by making use of a breadth-first search (BFS) approach.
It runs on all edge cases too.
The use of a queue and grid traversal ensures that the code runs efficiently for typical grid sizes.

@Saloni6111 removed. Kindly verify.


Doubts on how Caffe transform_param are applied in Digits

[Latest version of Digits in use]
I'm using a default Alexnet where i changed the transform params for the data (train and test) layers, by adding:
transform_param {
    ...
   mean_value: 128
   scale:  0.5
}

Two questions:

the scale parameter seems to be ignored, at least during Classify One Image, where the image histogram in data activation is not changed by the scale factor. Is this normal?

If I click on "Subtract Mean" during the model creation, there seems no way to make the network use the mean values I manually defined in the network definition. My values are preserved only if I never click on the dropdown. If I click, there seems to be no way to tell digits to just use the network defined values. Am I missing something? Maybe an option like 'Use network defaults' would be nice.

Furthermore, even when the user defined mean value is accepted by the network (and correctly reported in train_val.prototxt) it is still ignored by the classify one image method.

Indeed data layers are ignored by Caffe during inference. If you wish to perform transformations on your input data you can use a Power layer, see this example.

Hi, thanks for the answer.
I think this behavior is a bit confusing, as the mean values selected by drop-down at model creation is, on the contrary, applied to the inference.
So, If the mean is defined in the prototxt then it is ignored by the inference.
But if the mean is defined by the Digits gui, then it is used during inference.
Correct?

So, If the mean is defined in the prototxt then it is ignored by the inference.

I don't think it's entirely correct. Data layers are used to load data from LMDB or LevelDB databases and they are not used during inference (when input fields are being used instead).

Ok, thanks for the clarification.
Still, If I select a specific Subtract Mean in the Data Transformations panel of the Digits GUI this will apply both to the Data layer AND to the inference.
Anyway, the safest bet seems to do all transforms in the power layer.

Thanks. It sounds like this answers your question so I'll close the issue. Feel free to post a question on https://groups.google.com/forum/#!forum/digits-users if you need any more information.


Downloaded xlsx file is opening in libreOffice but shows corrupt in excel 2007

This is my code. It creates a xlsx file which is opening in libreoffice but shows corrupt in excel-2007
also workbook.xlsx.write(response) downloads .zip file
Please help.
var Excel = require("exceljs");
var express = require("express");
var bodyParser = require("body-parser");
var app = express();
app.use(bodyParser.urlencoded({ extended: false }));
app.use(bodyParser.json());
app.all("/getExcel", function(req, res) {
var workbook = new Excel.Workbook();
var worksheet = workbook.addWorksheet("Discography");
worksheet.columns = [{ header: "Album", key: "album" }, { header: "Year", key: "year" }];
worksheet.addRow({ album: "Taylor Swift", year: 2006 });
worksheet.addRow(["Fearless", 2008]);
var rows = [["Speak Now", 2010], { album: "Red", year: 2012 }];
worksheet.addRows(rows);
worksheet.getCell("A6").value = "1989";
worksheet.getCell("B6").value = 2014;
workbook.xlsx.writeFile('taylor_swift.xlsx').then(function() {
console.log("saved");
});
});

I am also facing the same issue in Excel 2007. Anyone find a fix for this?

same issue. One User use Excel 2016, say it is corrupt; but my Excel 2016 can open the same excel. strange

Hi,
Is it possible that there is an issue with fonts on one computer installed and on the other not ?
I noticed that on my machine i got errors related to that. It was not in that context, but that could explain the behavior ?

Am 25.10.2018 um 13:16 schrieb Yuiffy<EMAIL_ADDRESS>same issue. One User use Excel 2016, say it is corrupt; but my Excel 2016 can open the same excel. strange
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/guyonroche/exceljs/issues/631#issuecomment-433011612, or mute the thread https://github.com/notifications/unsubscribe-auth/ACLYMXP2jV5kIKM1EBNLYCzsiwihbROmks5uoZ19gaJpZM4WJnQO.

Same issue here on Excel 2007, quite annoying since saving files is a critical function. Opens perfectly fine with LibreOffice as well (<IP_ADDRESS>), first sheet only is shown just like @xurei
https://gist.github.com/YassienW/936a1c4bd9114d483808080af0f08199

I'm pretty sure it was originally created via Microsoft Excel. Right now i switched to using another library for saving files since this was sadly too inconvenient

Same issue for me. Here is the fully-generated file.
report(1).xlsx
For information :

It does open in Zoho Sheet and Excel 2016 (for Mac) without any problem,
It does open on LibreOffice <IP_ADDRESS>, but the first sheet only is shown, the other is not,
Excel 2007 refuses to open and complains about a corrupt file. I tried to generate only one sheet of each, refuses as well.

I'm using the latest version of exceljs (1.6.2)
Here is a dirty copy/paste of the code : https://gist.github.com/xurei/03aa1d73b7f4b483cdc0b3e0038e3081

hi, the link was expired, can you show me the code ? I am facing this problem ,thx

@a5006 I just made the gist public. You should have access now.

Maybe this could not related to the given code fragment's issue, but I had the same issue where libre shows correctly but excel mark it as a corrupted file. With some analysis of my code I found that using pageSetup and margins caused the issue.
My code was like below
pageSetup: {
  margins: {
        left: 0.1,
        right: 0.1,
  }
}

exceljs this section of code
https://github.com/exceljs/exceljs/blob/master/lib/doc/worksheet.js#L67
Uses Object.assign but margin is also another javascript object where it would just remove the other margin defaults.
As a solution, I had to define all the margin attributes like below.
pageSetup: {
  margins: {
        left: 0.1,
        right: 0.1,
        top: 1,
        bottom: 1,
        header: 1,
        footer: 1,
  }
}

Hope this would help someone!


SonarCloud correctly ignore code

SonarCloud is analyzing the Kociemba code, which it shouldn't do.
That code should be ignored using the build.gradle file.

Don't know if the pull request fixed the issue
Also added global settings to SonarCloud on their page, so I should be set, but keeping this issue open until I can confirm that the issue is 100% solved.


Add support for Django 3

Hi all, this should add support for Django 3, this PR is based on https://github.com/groveco/django-sql-explorer/pull/377 that it was missing the travis config
@chrisclark could you approve it ?

I just tested in my Django 3.0.5 project with poetry and it works:
If any of you needs this before it's merge and you are using poetry just do:
poetry add git+https://github.com/robertofd1995/django-sql-explorer.git#update-django-3


fix(pointcloud): Listen to currentData within useEffect to init 3DView

:tada: This PR is included in version 1.8.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Do we need to set -YdisableFlatCpCaching for Scala 3?

PR #826 sets -YdisableFlatCpCaching when outputting Jar on classpath and compiler version is 2.12/2.13.
Do we need to do it for Scala 3? Or do we no longer need it since https://github.com/scala/scala/pull/6064 is merged?

Probably want to go back to the good old approach of... experimentally test original reproduction in https://github.com/sbt/zinc/issues/282 and see if the issue is reproducible without -YdisableFlatCpCaching.


Line 77: deprecated (protocol_confirmation): Remove in v5.1. Use analyzer_confirmation

🐛 Summary
Seen in reporter.log:
2022-09-15T18:42:31.998498979Z worker-09: warning in [REDACTED]/zeek/site/packages/icsnpp-genisys/main.zeek, line 77: deprecated (protocol_confirmation): Remove in v5.1. Use analyzer_confirmation.

Fixed with cisagov/icsnpp-genisys@1a78fb40d392ef1fabed96b3eb61226428c197a6


Add support for multi-threaded encoding

Implements #20

Coverage decreased (-0.2%) to 94.728% when pulling a4957a9a89e8d0d09e2b1cff043478e9c9e7eec3 on mt-support into f66c2bfe1ef81b2a961ab723e51be38e7024f658 on master.


[BREAKING] Pass the Pipeline to Stage through props instead of parent

This commit changes the way we pass a Pipeline to a newly created Stage.
Instead of passing it as a parent, in the first argument of the constructor,
we now pass it through a separate props object, in the pipeline property.
This is in order to make Stage more consistent with other Constructs,
and with the changes made to the Actions API in #459.

I based the PR on #492 - the base should be changed to master once #492 is pushed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Rebased and changed the base branch to master after #459 and #492 were pushed.


Remove assets

This PR removes /app/assets.py, which is leftover from the Flask Assets era. No longer needed!

Bleh this is carrying over way more than I wanted :)


Bug in 0.17 - value function isn't setting inputs value as it did in 0.16

In 0.16 if you called value on an input the actual dom value always matched what was passed in.
For example the below input would only ever show the value of model.age
input [ type' "text", value model.age, onInput InputAge
  , on "input" targetValue (\s -> Signal.message address (InputAge s)) ] []

Now in 0.17 the initial call to value set's the dom value, however subsequently the input shows whatever was keyed rather than the model's value.
Here's an example where the model is updated when an int is keyed, but stays the same if a non Int is keyed.  As you can see from the gif, when I keyed "jkl" in 0.16, the value of the input stayed empty.  When I keyed "jkl" in 0.17 the input showed the value "jkl"
Here's the code for each version:
Example in 0.16
Example in 0.17

Am I right that it's only happening if the input is focused and not readonly?
Here is a modified example: https://gist.github.com/gdotdesign/f2dcb32b5b2843452876f89d5d303094

I missed the conversation about this on #elm-dev yesterday.  The most likely reason I would need this (or some other equivalent) is to clear the value of an input after a form submit, for example If I had a chat app, and I had an input used to key the message, I'd need some way of clearing the input value after the message was submitted.
I think @rtfeldman suggestion would work good:

tl;dr value is innately buggy and should probably be replaced by defaultValue + setValue

I'm not sure I like removing the value function (attribute), as it reduces elm-html parity with html.  Maybe keep value, with bold guidance to consider the alternative (whatever that might be).
From a perf stand point: I see how this might make perf testing look worse but I'm having a hard time seeing how this would result in perf issues in most real scenarios..  Might be short sighted on my part.

I did a fix for this in https://github.com/elm-lang/virtual-dom/commit/4be12eb1ff81d0b97e745aedf089aaf148212eb0


added make_dict, make_list, make_tuple to unflatten_func in util.flatten

I added the primitive type constructors make_tuple, make_list and make_dict to the unflatten_func returned by util.flatten so that the flattening/unflattening process can be differentiated through in cases like #198 where these constructors are required.

Awesome, thanks!


Distribute our custom dispatch/unistd.h header

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

https://github.com/conda-forge/cctools-and-ld64-feedstock/pull/70 is failing because the dispatch/unistd.h header, added by a patch in this feedstock, wasn't getting installed with the package.

LGTM, thanks a lot! :)

With @h-vetinari's thumbs-up I'll go ahead and merge this to try to unlock progress on https://github.com/conda-forge/cctools-and-ld64-feedstock/pull/70 .


Focus states & keyboard accessibility

First of all, thank you so much for building this! I've been trying it out in a few projects and it's honestly been a breath of fresh air to not have to constantly fight with default user agent styles to get things looking exactly the way I want.
That said, I do have some concerns around accessibility of the reset in its default configuration, specifically because it removes the default browser focus outlines. This is the reset working as intended, but as it grows in popularity I fear that people who use it might not realize this and will build apps with really poor accessibility for keyboard users by not providing any custom focus styles on top of the reset.
Some potential ideas to work around this:

We can make an exception for the default focus styles and not override them. This way usages will continue to have a reasonable default level of accessibility for keyboard users.
We can add a note in the readme, ideally very high up, mentioning that the reset removes focus styles, the consequences around that, and strongly recommend users to provide their own.

I would personally vastly prefer 1 as a safer default, but 2 is probably better than nothing if we want to err on the purist side and override as much styling as possible. Definitely open to other ideas as well, and happy to send a PR once we settle on the approach!

I vote for option 1, as well.

I know this is unpopular, but the main idea is to remove all the styles, and then the user/developer will add focus styles.
Think that this is the reset project, and on top of it, you can add any new level of styles.
If you check checkboxes and radio buttons, you will see that you can't see them after resetting them. This is because the main idea is to reset, and after that, web developers will add their styles.

This article is a little dated now, but Eric Meyer reflects on – in his words – the "blunder" of removing visible focus styles from his original CSS reset.
That said, this isn't an exact parallel and @elad2412's position isn't unreasonable. But I fear that developers are much more likely to verify that a checkbox is visible than that the focus styles are appropriate.

@elad2412 would you be open to option 2, i.e. add something prominent to the readme to recommend providing focus styles?
While I completely agree with @SeanMcP that it's probably not going to be enough, and a vast majority of users that adopt this reset will end up ignoring focus states completely, we'll at least be in a slightly better situation than the status quo where it's not mentioned at all.

Hi @SeanMcP,
This CSS reset's primary goal is to take the CSS reset to the next level of resetting the basic styles which aren't under shadow DOM. I understand your opinion, but this misses this CSS reset's primary goal.

Hi @lewisl9029, I added the "Accessibility Recommendation" section on the READ ME.
Issue closed.

Just wanted to add that this seems to bring back the browser focus styles. There may be some missing use cases but this seems to work:
:focus {
   outline: revert;
}

Putting that below the-new-css-reset seems to do the trick.


Ruby group capstone: set up your Kanban board

Kanban project
Team members
✔ @uno36 (Team member #1)
✔ @rahimialijan (Team member #2)
✔ @Nahid1911 (Team member #3)

[[# STATUS: APPROVED :green_circle: :clap:
Hello @TEAM     ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:  .
Congratulations! 🎉  Keep up the good work 🥂
HIGHLIGHTS:

[x]  Your repo is professional  😎
[x] Descriptive P.R 💯
[x] Kanban setup correctly 🥇

Now you can merge this branch and move on :1st_place_medal:
Happy Coding cheers 🥇 💯

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.]]


[BUG]there should be a button for shooting

Description
by clicking spacebar again and again not making game responsive there should be a button for shooting ...
kindly assign this issue to me @YadavAkhileshh so i fix it
Screenshots
No response
Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[ ] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@YadavAkhileshh if this issue is open can i work on it


Callback is not invoked in MediaSession.onSessionTerminate

I'm wondering if the intention was to call super.onSessionTerminate instead of super.end here:
https://github.com/legastero/stanza.io/blob/0283e6e02790580ba3937069d533da61b1d282b9/src/jingle/MediaSession.js#L234
It seems that super.onSessionTerminate has correct general flow, including call to cb:
https://github.com/legastero/stanza.io/blob/0283e6e02790580ba3937069d533da61b1d282b9/src/jingle/ICESession.js#L153
Currently, cb is passed into super.end instead of silent parameter, so the callback is never invoked in processing of session-terminate.

Yeah, it seems that with the following workaround the client now sends result in response to session-terminate, something that was missing before.
MediaSession.prototype.onSessionTerminate = function(changes, cb) {
  for (const receiver of this.pc.getReceivers()) {
    this.onRemoveTrack(receiver.track);
  }
  ICESession.prototype.onSessionTerminate.call(this, changes, cb);
};

Fixed in 10.2.5.
I looked through and didn't find any other super calls that could cause further issues like this.


change newline character to LF from CRLF

I think that newline character should be unified.
The other files use LF for newline character.
This PR rewrites newline character to LF from CRLF in k8s201.md.

/lgtm
/approve


sparkctl is simple. Does  it support blocking mode?

I want create a spark application and wait logging, but sparkctl didn't support it

If all you want is to stream the driver logs after your app gets submitted, you can use sparkctl log <app name> --f.

I suspect the OP is interested in something like https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/732


SaveAs / LoadFrom / Compression

Added saveManger for Trie.
Add loadFromFile and saveAs (closes #31 ).
Used XWRT to compress XML file (closes #30).
Added LoadThread.
Added open program from command line with another history file.
Notes:

LoadFromFile clears the log first, I'm thinking of merging the tries instead of clearing.
.gitignore don't allow .exe files, you should include xwert32.exe 3.4.
We're using static dicitionary compression. wrt-eng contains nearly all english words. We should include only words may exist in the XML file for better compression.
ext.reg is supposed to be the registry command to include our program in the file context menu (change our path to the path of CalculatorUI.exe).

closes #29


Cost Analysis

[x] What are the average inpatient charges across different states or regions?
[ ] Are there any trends or patterns in inpatient charges over the years?

done q1 and plotted a map for the same as well

[ ] inpatient vs outpatient average charges yearly trend


Add support for thin.

Add support for thin webserver.

@macournoyer just thought you might find this interesting. This is a small server conformance test suite. Thin is passing.


Change kubernetes dependencies to use release-1.11 branch as constraint

Updating apiserver dependency to pull from release-1.11 to bring in a fix for running metrics server in clusters with client authentication mechanisms other cert based.
See https://github.com/kubernetes/kubernetes/issues/65724 for more details.

@DirectXMan12 can you take a look? Thanks!

please change the PR comment to have a) an actual issue link (e.g. kubernetes/kubernetes#65724), and b) describe what that issue actually was.  Otherwise, looks good.

@DirectXMan12 Fixed the commit message and the PR description!

I didn't notice this before, but I think we need to pin all the deps to that branch, otherwise we might get incompatibilities next time we try to update.

@DirectXMan12 I have updated all kubernetes depdencies to release-1.11 branch and client-go to release-8.0 branch.
Testing

Built the metrics-server based on these changes
Deployed to EKS cluster (v1.10.3) which uses webhook based authentication
Metrics server came up successfully
Tested out HPA functionality

@serathius we'll probably want to pull this in for the release.  Can you test with it in place, if possible?

@serathius I'm more than happy to help out with the testing, let me know.

I'm mainly focusing on scalability testing. I don't know we want to rerun those for every PR.


BUG: rv_discrete fails when support is unbounded below

Describe your issue.
I am trying to create a discrete distribution on the integers (both positive and negative). The documentation states that rv_discrete takes an "optional" lower bound parameter a. It does not, however, seem to allow for an unbounded distribution.
The rv_discrete class should support unbounded distributions. I recognize this might be a big project, but in the meantime the documentation should be updated to reflect that

The a parameter isn't really optional (an optional in Python allows for the value None). Rather, it has a pre-set default.
a must be bounded, even though b can be unbounded

Reproducing Code Example
from typing import Any

import scipy
import numpy as np

class RV(scipy.stats.rv_discrete):

    def __init__(self) -> None:
        # NOTE: np.nan, None, -np.inf, and float("-inf") all fail
        super().__init__(a=np.nan, b=0)

    def _pmf(self, k: np.ndarray[int], *_: Any) -> np.ndarray[float]:
        raise NotImplementedError

RV().expect(lb=-5, ub=0)

Error message
Depends on what you pass for `a`, but generally a `ValueError`

SciPy/NumPy/Python version and system information
1.10.1 1.24.3 sys.version_info(major=3, minor=11, micro=3, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: cmake
    found: true
    include directory: unknown
    lib directory: unknown
    name: OpenBLAS
    openblas configuration: unknown
    pc file directory: unknown
    version: 0.3.18
  lapack:
    detection method: cmake
    found: true
    include directory: unknown
    lib directory: unknown
    name: OpenBLAS
    openblas configuration: unknown
    pc file directory: unknown
    version: 0.3.18
Compilers:
  c:
    commands: cc
    linker: ld64
    name: clang
    version: 13.1.6
  c++:
    commands: c++
    linker: ld64
    name: clang
    version: 13.1.6
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 0.29.33
  fortran:
    commands: gfortran
    linker: ld64
    name: gcc
    version: 12.1.0
  pythran:
    include directory: /private/var/folders/_f/lyvxf0v13gs7984d7sf7j83c0000gn/T/pip-build-env-c1_tgrdc/overlay/lib/python3.11/site-packages/pythran
    version: 0.12.1
Machine Information:
  build:
    cpu: aarch64
    endian: little
    family: aarch64
    system: darwin
  cross-compiled: false
  host:
    cpu: aarch64
    endian: little
    family: aarch64
    system: darwin
Python Information:
  path: /private/var/folders/_f/lyvxf0v13gs7984d7sf7j83c0000gn/T/cibw-run-s3_k_ke5/cp311-macosx_arm64/build/venv/bin/python
  version: '3.11'

I'll keep this in mind when I'm working on the infrastructure this summer. This is now tracked in gh-15928.

There is already one distribution with unbounded support (all integers)
don't remember name, it's difference of two Poisson distributions

aside: docstring for dlaplace is confusing because formula does not correspond to the literature or R packages
based on PR https://github.com/scipy/scipy/pull/2562
this is just a rewriting of the standard formula, but that's not obvious

Agreed with https://github.com/scipy/scipy/issues/18564#issuecomment-1565396580'; there's not an inherent problem with unbounded support here.
import numpy as np
from scipy import stats

class RV(stats.rv_discrete):

    def _pmf(self, k):
        return np.tanh(1/2) * np.exp(-abs(k))

rv = RV(a=-np.inf, name='RV', longname='An unbounded discrete random variable')
k = np.arange(-100, 110, 10)
rv.pmf(k)
# array([1.71911093e-44, 3.78659382e-40, 8.34052793e-36, 1.83712353e-31,
#        4.04653386e-27, 8.91308397e-23, 1.96323739e-18, 4.32431813e-14,
#        9.52494453e-10, 2.09800865e-05, 4.62117157e-01, 2.09800865e-05,
#        9.52494453e-10, 4.32431813e-14, 1.96323739e-18, 8.91308397e-23,
#        4.04653386e-27, 1.83712353e-31, 8.34052793e-36, 3.78659382e-40,
#        1.71911093e-44])

The problem is with certain methods, like cdf, that try to use the lower end of the support.
https://github.com/scipy/scipy/blob/c1ed5ece8ffbf05356a22a8106affcd11bd3aee0/scipy/stats/_distn_infrastructure.py#L3316-L3319
The immediate error is that _a, -np.inf, can't be converted into an int. And of course, it wouldn't be possible to create an array with all integers from negative infinity to k.
The infrastructure would need a more sophisticated algorithm for handling infinite sums (e.g. like mpmath.nsum). I understand that this can be tricky with limited precision arithmetic (e.g. some of the techniques suffer from catastrophic cancellation), but we can probably do better than erroring out due to an undocumented limitation.

cdf could also add negative numbers, as in expect, AFAIR. However, that still has problems because it's difficult to get a measure or error for the lower tail. (prob of an interval is easier but cdf needs lower tail prob)
same with stats, mean, moments.


Update Cargo.toml

This makes it compile on ubuntu (worked fine in arch)

This didnt fix it, but it got farther in the install


Weighted mean reports - also output total "N"

Currently the weighted mean report outputs "n" the number of analyses included in the mean.  It would be helpful if it also output the total number of analyses (included those excluded) so at a glance it would be possible to tell how selective the calculation is.  This is currently shown on the WM plot as e.g. "n=13 of 15", so it would merely be a function of adding an additional column with for the "15", with the header "N (total)"  - I am open to other label suggestions here.

This request for including n, and N as indices for weighted means should apply to unknown WM, RM WM (as per issue #606) AND RM Calib constant

Feature provided in v1.7.6


Backport WebView2 to WinUI2

This is a backport of the WinUI3 control and tests into WinUI2. Notable differences from the experience you get in WinUI3 via WinAppSDK are:

This change adds a nuspec dependency from Microsoft.UI.Xaml onto WebView2's nupkg. In WinAppSDK the winrt implementation dlls are carried in the WinAppSDK runtime. We may change to do that as a follow-up.
In WinUI3 the Focus related methods were moved so that WebView2 can derive from FrameworkElement. We don't have that in OS XAML so WebView2 control derives from Control instead. 🤷

WinUI2 also compiles and supports Arm32, but WebView2 package does not because there is no WebView2 runtime for Arm32. I've asked the WebView2 folks to consider adding at least the type information (with stub implementation), but I don't know if/when that will happen. As such I have made the build conditional so that the Arm32 version does not include WebView2 control. Consumers will need to write conditional code as well if they support Arm32 and want to use WebView2.
Fixes #5610

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

                string installerArgs_WV2Runtime = "/silent /install";

remove comments since we aren't useing mini_installer here

Refers to: dev/WebView2/InteractionTests/WebView2Tests.cs:147 in 1ffea83. [](commit_id = 1ffea83ef278157ebf7c06a055f472ed0e822c09, deletion_comment = False)

Per discussion with @jevansaks and @asklar , some of the comments made by @asklar and myself are existing issues in WinUI3 codebase and that are lower priority. I will capture these for fixing in WinUi3 codebase.

/azp run

/azp run

Congratulations!

How's the extra dependency going to work for an app that doesn't use WebView2? Will it be optimized out or increase the footprint of all apps using WinUI 2?

How's the extra dependency going to work for an app that doesn't use WebView2? Will it be optimized out or increase the footprint of all apps using WinUI 2?

The dlls are relatively small (~500KB). For now it's getting copied into the consuming apps. We are discussing alternatives, since WinUI3 is already doing this slightly differently by including the WebView2 client-side dlls in the WinAppSDK framework package.


Optional capture groups?

Not super important but is it normal that capture groups cannot be optional?
This works:
$ echo ab | ./src/main.py --no-debug '(?P<agroup>a)(?P<bgroup>b)'
0: [⊢agroup]a[agroup⊣][⊢bgroup]b[bgroup⊣]

But this fails:
$ echo ab | ./src/main.py --no-debug '(?P<agroup>a)(?P<bgroup>b)?'
Traceback (most recent call last):
  File "./src/main.py", line 65, in <module>
    for count, match in enumerate(enum_matches(pattern, document)):
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/__init__.py", line 20, in enum_matches
    for mapping in enum_mappings(va, text):
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/explore_dag.py", line 107, in enum_dag_mappings
    jump = Jump(dag)
  File "/home/a3nm/git/enum-spanner/src/benchmark.py", line 44, in wrapper
    ret = function(*args, **kwargs)
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/precompute_dag.py", line 102, in __init__
    self.levelset = LevelSet(dag)
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/precompute_dag.py", line 61, in __init__
    self.__init_levels__()
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/precompute_dag.py", line 85, in __init_levels__
    self.vertex_index[vertex] = len(self.vertices[level])
IndexError: list index out of range

In perl, however, it works:
$ echo a | perl -p -e 's/^(?P<agroup>a)(?P<bgroup>b)?$/agroup:$+{agroup} bgroup:$+{bgroup}/'
agroup:a bgroup:
$ echo ab | perl -p -e 's/^(?P<agroup>a)(?P<bgroup>b)?$/agroup:$+{agroup} bgroup:$+{bgroup}/'
agroup:a bgroup:b

Not quite ;)
$ echo ab | ./src/main.py --no-debug '(?P<agroup>a)(?P<bgroup>b*)?'
Traceback (most recent call last):
  File "./src/main.py", line 87, in <module>
    for count, match in enumerate(enum_matches(pattern, document)):
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/__init__.py", line 20, in enum_matches
    for mapping in enum_mappings(va, text):
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/explore_dag.py", line 119, in enum_dag_mappings
    for Sp, new_gamma in next_level(dag, gamma):
  File "/home/a3nm/git/enum-spanner/src/benchmark.py", line 57, in generator_wrapper
    val = next(ret)
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/explore_dag.py", line 86, in next_level
    gamma2 = follow_epsilons(dag, follow_SpSm(dag, gamma, Sp, Sm))
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/explore_dag.py", line 40, in follow_SpSm
    return [vertex for vertex, ps in path_set.items() if len(ps) == len(Sp)]
  File "/home/a3nm/git/enum-spanner/src/enum_mappings/explore_dag.py", line 40, in <listcomp>
    return [vertex for vertex, ps in path_set.items() if len(ps) == len(Sp)]
TypeError: object of type 'NoneType' has no len()

Looks like this latter problem is fixed. :) However there still seems to be a problem:
$ echo aa | ./src/main.py -b --no-debug '(?P<agroup>a)?'
0,0 agroup=None,None: aa
1,1 agroup=None,None: aa
0,1 agroup=0,1: [⊢agroup]a[agroup⊣]a
2,2 agroup=None,None: aa
1,2 agroup=1,2: a[⊢agroup]a[agroup⊣]

Looks like we're getting duplicate matches, no? Either the group is matched to one of the a's, or it's not matched: why would there be three different ways of not matching it?

The reason is it always try to match the "entire" expression. Your example will be rewritten to something like ^.*(?P<match>(?P<agroup>a)?.*)$ before computing anything. It creates a new group match which is used to identify where the expression is matched. Here this group matches all empty subwords.
I implemented this behaviour because I think it is closer to the usual regexp semantic where you want to match an expression and you can extract parts of it using groups, but what you except might be more convenient to use.
It is possible to override the main match group: (?P<match>a)? or to force it to match the whole input : ^.*(?P<agroup>a)?.*$.
PS: '^' and '$' symbols are not implemented (yet?) and only work when placed at the begining and end of the expression thanks to a dirty hack.
unrelated PS: the very last commit breaks the memory saving behaviour


Wiki: Improved tmux session switcher

I'd like to offer an improved tmux session switcher for the wiki:

Display at top of screen like default switcher
Put current session as first entry (hence dismissing with "enter" becomes a noop)
Show number of attached clients
Show session id
Switch by unique session id
Opens in dedicated window, not pane (because hardly usable from small panes)
Easy switching by id (typing number after pre-populated '$' query gives high chance to select it)

Works fine on OSX, have not been able to test on anything else.
# Use FZF to switch Tmux sessions:
# bind-key s run "tmux new-window 'bash -ci fs'"
fs() {
	local -r fmt='#{session_id}:|#S|(#{session_attached} attached)'
	{ tmux display-message -p -F "$fmt" && tmux list-sessions -F "$fmt"; } \
		| awk '!seen[$1]++' \
		| column -t -s'|' \
		| fzf -q '$' --reverse --prompt 'switch session: ' -1 \
		| cut -d':' -f1 \
		| xargs tmux switch-client -t
}

Edit: Hold on, there's an issue with switching to a session that has a space in it's name. Fixed

Looks good to me. Write permission to the wiki page is open to everyone, so feel free to update the page. Thanks!


test failure: TestResumeFromCheckpointE2EWithManualSentinel

Sometimes the test TestResumeFromCheckpointE2EWithManualSentinel errors with:
    runner_test.go:2666:
        	Error Trace:	/home/runner/work/spirit/spirit/pkg/migration/runner_test.go:2666
        	            				/opt/hostedtoolcache/go/1.22.4/x64/src/runtime/asm_amd64.s:1695
        	Error:      	Error "sql: database is closed" does not contain "context canceled"
        	Test:       	TestResumeFromCheckpointE2EWithManualSentinel

Duplicate of https://github.com/cashapp/spirit/issues/315


Unable to deploy azure function

Does this occur consistently? Yes
Repro steps:

Unable to deploy in both VS Code or Visual Studio 2019 Enterprise version
I have Latest .NET Core SDK 3.1 but still im facing issue to deploy sample hello function app

Action: azureFunctions.deploy
Error type: 500
Error Message: An error has occurred.
Version: 0.22.1
OS: win32
OS Release: 10.0.18363
Product: Visual Studio Code
Product Version: 1.41.1
Language: en

Call Stack
12:12:32 AM indaosubscriberfnapp: Creating zip package...
12:12:33 AM indaosubscriberfnapp: Starting deployment...
12:14:43 AM: Error: An error has occurred.
```
extension.bundle.js:309:399345extension.bundle.js:309:399345
retryCallback extension.bundle.js:225:114550
retryCallback extension.bundle.js:157:31063
extension.bundle.js:157:32676extension.bundle.js:157:32676
handleRedirect extension.bundle.js:171:7752
extension.bundle.js:157:29848extension.bundle.js:157:29848
Request._callback extension.bundle.js:1:670871
Request.n._callback.n.callback.n.callback extension.bundle.js:201:4483
Request.emit events.js:200:13
Request. extension.bundle.js:201:17592
Request.emit events.js:205:15
IncomingMessage. extension.bundle.js:201:16382
Object.onceWrapper events.js:288:20
IncomingMessage.emit events.js:205:15
endReadableNT _stream_readable.js:1154:12
processTicksAndRejections task_queues.js:84:9
```

@gigsvinoth Sometimes Azure just has temporary outages/hiccups, so you can try deploying again and it might just work. 😄
Otherwise please check the VS Code output window for any logs/errors that you could show us.

Hi Team,
I tried to deploy azure function through visual studio 2019 version 16.6.1. i getting below error
1>THSubscriber -> C:\Users\v.a.vijayakumar\source\repos\THSubscriber\THSubscriber\bin\Release\netcoreapp3.1\bin\THSubscriber.dll
1>Done building project "THSubscriber.csproj".
2>------ Publish started: Project: THSubscriber, Configuration: Release Any CPU ------
2>THSubscriber -> C:\Users\v.a.vijayakumar\source\repos\THSubscriber\THSubscriber\bin\Release\netcoreapp3.1\bin\THSubscriber.dll
2>THSubscriber -> C:\Users\v.a.vijayakumar\source\repos\THSubscriber\THSubscriber\obj\Release\netcoreapp3.1\PubTmp\Out
2>Publishing C:\Users\v.a.vijayakumar\source\repos\THSubscriber\THSubscriber\obj\Release\netcoreapp3.1\PubTmp\THSubscriber -<PHONE_NUMBER>1755471.zip to https://thsubscriberpoc.scm.azurewebsites.net/api/zipdeploy...
2>The attempt to publish the ZIP file through https://thsubscriberpoc.scm.azurewebsites.net/api/zipdeploy failed with HTTP status code InternalServerError.
========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========
========== Publish: 0 succeeded, 1 failed, 0 skipped ==========
Thanks

@gigsvinoth Based off your errors, it seems like this is an Azure error.  VS Code and Visual Studio can't control server errors unfortunately.
Usually these problems are transient though.  If you're still experiencing issues, I recommend creating a new Function App and trying to deploy to that instead.


WIP: Running in vscode container

Proposed changes
This PR is for implementing DevContainer from VSCode to be able to be in a pristine environment.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
This is in WIP since this is not stable now.

Closing due to problems.


remote error: handshake failure Mutual TLS to RabbitMQ Server

So I am having a problem authenticating my client certs on my RabbitMQ Server using Go-lang. I can however do so with a python script and the same client certs. I have provided the code I am using below and would like to know if I am doing something wrong any help would be appreciated.
func main() {
    cfg := new(tls.Config)

    cfg.MinVersion = tls.VersionTLS12
    cfg.MaxVersion = tls.VersionTLS12

    cfg.CipherSuites = []uint16{
    tls.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 }

    cfg.RootCAs = x509.NewCertPool()

    if ca, err := ioutil.ReadFile("certs/cacert.pem"); err == nil {
      cfg.RootCAs.AppendCertsFromPEM(ca)
    }

    cfg.ServerName = "dev"
    cfg.PreferServerCipherSuites = true

    if cert, err := tls.LoadX509KeyPair("certs/cert.pem", "certs/key.pem"); err == nil {
      cfg.Certificates = append(cfg.Certificates, cert)
    }

    connection, connectionError := amqp.DialTLS("amqps://dev:5671/", cfg)
    if connectionError != nil {
      log.Printf("%v", connectionError)
    } else
    {
       // Create channel, queue etc.
    }
}

I was able to successfully establish a Mutual SSL connection to the RabbitMQ Server. The process I took was to create the secure TCP connection using the OpenSSL library and then tunnelling a amqp connection through that. The code I used to do this is bellow:
ctx, error := openssl.NewCtxFromFiles("cert.pem", "key.pem")
  if error != nil {
    log.Fatal(error)
  }

  error = ctx.LoadVerifyLocations("certs/cacert.pem", "")

  if error != nil {
    log.Fatal(error)
  }

  sslConnection, error := openssl.Dial("tcp", "dev:5671", ctx, 0)

  if error != nil {
    log.Fatal(error)
  }else
  {
    config :=  amqp.Config{
      Vhost:     "/",
      SASL:      []amqp.Authentication{&amqp.PlainAuth{"guest", "guest"}},
    }

    amqpConnection, amqpConnectionError := amqp.Open(sslConnection, config)
  }

Hope this is useful to anyone else who comes across the same or similar problem when mutual SSL is required.

Hi Roger,
I am running into the same issue with RabbitMQ 3.6.9 and Erlang/OTP 19. Basically I am able to successfully connect to the rabbitMQ server using the openssl commands listed in  the troubleshooting guide( https://www.rabbitmq.com/troubleshooting-ssl.html), but when I try to programmatically connect it from a go service, I get an "unknown CA" error. Which openssl library did you use? I tried to issue a go get on https://github.com/spacemonkeygo/openssl but got errors. Thanks!


fix: fix enum array annotation elements

Codecov Report
Base: 87.21% // Head: 86.96% // Decreases project coverage by -0.25% :warning:

Coverage data is based on head (6d0f43c) compared to base (9f9f3a2).
Patch coverage: 77.77% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #37      +/-   ##
============================================
- Coverage     87.21%   86.96%   -0.26%
- Complexity      173      174       +1
============================================
  Files            16       16
  Lines           485      491       +6
  Branches         66       68       +2
============================================
+ Hits            423      427       +4
- Misses           45       46       +1
- Partials         17       18       +1

Impacted Files
Coverage Δ

src/main/java/net/jailgens/mirror/Reflections.java
83.33% <77.77%> (-5.56%)
:arrow_down:

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

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


coredns build failed with kubernetai

I tried using master/v1.7.0/v1.6.9/v1.6.8 version of coredns/coredns with kubernetai enable, make failed with all these version.
master and v1.7.0 's output of make
../../../../pkg/mod/github.com/coredns/kubernetai@v0.0.0-20200616005541-3c3a37378785/plugin/kubernetai/setup.go:24:24: not enough arguments in call to k.InitKubeCache
	have ()
	want (context.Context)

while 1.6.9 and 1.6.8 's output of make
../../../../pkg/mod/github.com/!azure/azure-sdk-for-go@v32.6.0+incompatible/services/dns/mgmt/2018-05-01/dns/client.go:24:2: ambiguous import: found package github.com/Azure/go-autorest/autorest in multiple modules:
	github.com/Azure/go-autorest v11.1.2+incompatible<EMAIL_ADDRESS>	github.com/Azure/go-autorest/autorest v0.9.4<EMAIL_ADDRESS>../../../../pkg/mod/github.com/!azure/azure-sdk-for-go@v32.6.0+incompatible/services/dns/mgmt/2018-05-01/dns/models.go:24:2: ambiguous import: found package github.com/Azure/go-autorest/autorest/azure in multiple modules:
	github.com/Azure/go-autorest v11.1.2+incompatible<EMAIL_ADDRESS>	github.com/Azure/go-autorest/autorest v0.9.4<EMAIL_ADDRESS>plugin/azure/setup.go:14:2: ambiguous import: found package github.com/Azure/go-autorest/autorest/azure/auth in multiple modules:
	github.com/Azure/go-autorest v11.1.2+incompatible<EMAIL_ADDRESS>	github.com/Azure/go-autorest/autorest/azure/auth v0.4.2<EMAIL_ADDRESS>../../../../pkg/mod/github.com/!azure/azure-sdk-for-go@v32.6.0+incompatible/services/dns/mgmt/2018-05-01/dns/models.go:25:2: ambiguous import: found package github.com/Azure/go-autorest/autorest/to in multiple modules:
	github.com/Azure/go-autorest v11.1.2+incompatible<EMAIL_ADDRESS>	github.com/Azure/go-autorest/autorest/to v0.2.0<EMAIL_ADDRESS>

kubernetai currently requires coredns v1.6.5 (per go.mod).

thanks, now the coredns images build success.
But seems kubernetai doesn't work correctly.
My corefile is
.:53 {
    errors
    health
    kubernetai cluster.local in-addr.arpa ip6.arpa {
        pods insecure
        upstream
        fallthrough in-addr.arpa ip6.arpa
        ttl 30
    }
    prometheus :9153
    forward . /etc/coredns/resolv.conf
    cache 30
    loop
    reload
    loadbalance
}

the coredns can't resolve the service name, if using kubernetes plugin, it works fine.

my build step:
Clone https://github.com/coredns/coredns into $GOPATH/src/github.com/coredns
Add this plugin to plugin.cfg per instructions therein.
make
make -f Makefile.release DOCKER=your-docker-repo release
make -f Makefile.release DOCKER=your-docker-repo docker
make -f Makefile.release DOCKER=your-docker-repo docker-push

it works. thanks


WIP: Synchronously unlink IPC file in TrioEndpoint

What was wrong?
It seems wrong to me to use an async API when cleaning up the IPC file as that will simply not run when the current trio context is cancelled. As a concrete example, that is what happened in https://github.com/ethereum/trinity/pull/1325, which was always leaving behind the IPC file for the discovery service
How was it fixed?

Had to update to mypy 0.740 because of https://circleci.com/gh/ethereum/lahja/2754?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link but then now we're getting that other failure

If the mypy error is about a trio API that is missing in trio maintained stub files, then an type: ignore + comment seems like an appropriate solution for now. Also opening an issue upstream would be nice then.

The mypy error seems legit as it's only MemorySendChannel that has a send_nowait()

The mypy error seems legit as it's only MemorySendChannel that has a send_nowait()

I ran into this in the trinity codebase.  AFAIK there is no good way around it.  I'd use a type: ignore, comment as to why, I'll open an issue in the trio codebase to see what they can do to fix this.

https://github.com/python-trio/trio/issues/1327

@gsalgado FYI the remaining lint error is because this repo uses black formatting so you need to rerun black <dir> to have it reformat the file for you.

@cburgdorf any suggestions on how to fix https://circleci.com/gh/ethereum/lahja/2897?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link?


Add docker and update readme [resolves #13]

resolves #13

hi @Badhan-abhishek! appreciate all the effort put into this PR. in favour of this issue and some recent development, i will be making some fundamental changes into how code is built and deployed for this project. the changes in this PR will therefore stop aligning. hence, i am closing this PR.


Run tests with OpenJDK 7 to verify runtime compatibility

watches travis

Looks like you need to skip test compilation as well. Oh, Maven. A better build system would know it already compiled them...

Like gradle?

The problem is the tools.jar depends on the $JAVA_HOME, so it's different between invocations and Maven then correctly recompiles as dependencies have changed.
Invoking surefire:test won't run the compile plugin.


add types for escape and unescape methods #18813

Although the issue is marked working as expected, it is important to mention that most major browsers maintain support for escape and unescape, and some javascript codebases moving to typescript may have escape and unescape in them. They are valid JavaScript, and thus they should be included in the global definition.

Fixes #18813

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: wbhob sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution. but the omission of these functions from the library is intentional. please see https://github.com/Microsoft/TypeScript/issues/18813#issuecomment-335220922

@mhegazy - In the current ES Draft spec, as well as in MDN I see no reference to escape / unescape being deprecated anymore. What is your reference for it still being deprecated?

u are right. there does not seem to be a deprecation notice any longer.

@wbhob mind refreshing the PR and addressing the failing tests?

I'll look into fixing those tests soon

🎉 Yay! Thank you