New Feature: template for GCP tile configuration

This adds a template file and resource for rendering out a file
that contains all of the relevent settings for configuring the
GCP tile in Operations Manager.

example rendered file contents:
# Operations Manager GCP Tile Config

## Google Config
Project ID: XXXXX
Default Deployment Tag: pivlab-vms
AuthJSON: { XXXXX }

## Director Config
NTP Servers: metadata.google.internal
[x] Enable VM Resurrector Plugin
[x] Enable Post Deploy Scripts
Blobstore Location: Internal
Database Location: Internal

## Create Availability Zones
us-central1-a,us-central1-b,us-central1-c

## Create Networks

### Management Subnet:
Name: pivlab-management-subnet
Google Network Name: pivlab-pcf-network/pivlab-management-subnet/us-central1
CIDR: <IP_ADDRESS>/26
Reserved IP Ranges: <IP_ADDRESS>-<IP_ADDRESS>
DNS: <IP_ADDRESS>
Gateway: <IP_ADDRESS>
Availability Zones: us-central1-a,us-central1-b,us-central1-c

### PAS Subnet:
Name: pivlab-pas-subnet
Google Network Name: pivlab-pcf-network/pivlab-pas-subnet/us-central1
CIDR: <IP_ADDRESS>/24
Reserved IP Ranges: <IP_ADDRESS>-<IP_ADDRESS>
DNS: <IP_ADDRESS>
Gateway: <IP_ADDRESS>
Availability Zones: us-central1-a,us-central1-b,us-central1-c

### PAS Services Subnet:
Name: pivlab-services-subnet
Google Network Name: pivlab-pcf-network/pivlab-services-subnet/us-central1
CIDR: <IP_ADDRESS>/24
Reserved IP Ranges: <IP_ADDRESS>-<IP_ADDRESS>
DNS: <IP_ADDRESS>
Gateway: <IP_ADDRESS>
Availability Zones: us-central1-a,us-central1-b,us-central1-c

### PKS Subnet:
Name: pivlab-pks-subnet
Google Network Name: pivlab-pcf-network/pivlab-pks-subnet/us-central1
CIDR: <IP_ADDRESS>/24
Reserved IP Ranges: <IP_ADDRESS>-<IP_ADDRESS>
DNS: <IP_ADDRESS>
Gateway: <IP_ADDRESS>
Availability Zones: us-central1-a,us-central1-b,us-central1-c

### PKS Services Subnet:
Name: pivlab-pks-services-subnet
Google Network Name: pivlab-pcf-network/pivlab-pks-services-subnet/us-central1
CIDR: <IP_ADDRESS>/24
Reserved IP Ranges: <IP_ADDRESS>-<IP_ADDRESS>
DNS: <IP_ADDRESS>
Gateway: <IP_ADDRESS>
Availability Zones: us-central1-a,us-central1-b,us-central1-c

## Assing AZs and Networks
Singleton Availability Zone: us-central1-a
Network: pivlab-management-subnet

## Security

Download the CA cert from `Operations Manager -> Settings -> Advanced` and paste the contents into here.

## Resource Config

Ensure all resources are "internet connected"

This seems like a very specific checklist or document for a workflow. For example, Ensure all resources are "internet connected".
What information are you trying to get out of terraform that the outputs don't provide?

@jtarchie I'm not looking for any new information, just re-organizing the output to match defaults from the install docs, and the configuration forms in operations manager so that you don't have to hunt for specific values in a randomly ordered list of keys/values in terraform output.

Hi @paulczar, sorry for the delayed response. I think this is a really interesting idea, and is definitely something we need to work on. I think perhaps a better option for the template is something that om could use to configure the tile directly. For an example check out our pipeline, where we essentially create this template in a concourse task.
Here we have a go template file which expects a transformed version of the terraform output as an input, and fills in the templated out fields using the texplate tool, which is a light wrapper around go templating. This is where we fill in the template https://github.com/cloudfoundry/infrastructure-ci/blob/f2389b77a6d26b0c5acbb02fc76573e3ac4554d7/scripts/terraforming/gcp/upload-configure-stage/task#L19.  The outputted file is ready for om configure-product.
We like this solution because om can read the file directly to configure the tile, so we can fully automate our pcf install.
Would something like this make sense for your use case?

@joshzarrabi om configure-product can now interpolate variables itself, so you don't even need to use texplate. It also understands ops-files...

@davewalter Wow!

 It provides a nice cli with syntactic sugar around templating.

My goal was to iterate on this until it was good enough to fully automate opsman and then do something along the lines of what you're suggesting to fully automate the whole thing.  However even then I don't think its a bad thing to have a configuration document that shows exactly how the tile should be configured.

I keep seeing people unsure of settings to use in Ops Manager, especially the trickier stuff like the network names ...  I really think this PR will be helpful to people.  Given its not really affecting functionality and just increasing outputs I'd like to see it get moving.

@paulczar We would really like to see it fully automatable before we merge it. I think it makes a lot of sense for it to output a template that om knows how to fill in and apply. We would be happy to discuss doing this as a team, as I do think it would make the user experience a lot better.

@joshzarrabi how goes the automation effort to have OM configure-product word with the terraform output?  If its done can you point to documentation on using ?

hey @paulczar - unsure what the status of this is. Maybe we can convene a little discussion with us and @pivotal-cf/pcf-platform-automation to discuss how to work this into om as mentioned earlier.


Corrections in the text

What does this PR do?
Some changes have been made to make the text clearer.
For resources
Description
Text changes for the Spanish language
Checklist:

[X] Read our contributing guidelines.
[X] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[X] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Chore/update dotnet deps sep 2024

upgrade dotnet dependencies. This includes some upgrades that were enabled by new package versions:

use system.text.json for quartz serialization instead of Newtonsoft.
add Mailkit smtp OTEL trace source

I love that the monthly deps updates just "happens".  Kudos!


fix: add peer dependencies

📝 Description

I encountered an error starting a new Next 13 project. This was a problem with yarn PnP and there was no similar case. As a result of analyzing the error, it was an error that the react-dom package could not be found in the react-aria/overlay package.
Looking for a little more, the peer dependency of the package used by next-ui was not declared as the peer dependency of next-ui.
The following solutions are being used temporarily.
Change peer dependency forced injection or pnpFallbackMode: "all" using packageExtensions option in yarnrc.yml
It is enough for me already, but I made a PR because I thought it would be nice if a more fundamental solution could be prepared in next-ui.
⛳ Current behavior (updates)
Module not found because peer dependency does not have react-dom and framer-motion in nextui packages. (only yarn PnP , not node_modules)
🚀 New behavior
Add as peer dependency to packages requiring react-dom or framer-motion.
💣 Is this a breaking change (Yes/No):
No
📝 Additional Information

I didn't update Yarnlock. Sorry. Update within 30 minutes


Grade item becomes gradebook item

Rename 'Grade Item' to 'Gradebook Item'
Add '+' to the Add Gradebook Item button
Move the Section filter to the Student column header

Hi @steveswinsburg - I've tacked on a fix for the clipping you noticed on the bottom row's menu.  Like you mentioned on the call, I've simply added some padding down there so there's always room - turns out there's enough room when there's pagination and the pagination summary.. but not enough when there's no pagination!  Thanks.


Added fix for missing thumbnailImageData

This fixes issue with crash due the fact that imageDataAvailable could be True even if thumbnailImageData is nil.

Good catch! 👍


Pubsub sample

So you'll notice there are commits in this PR from your previous PR, #26. I fixed this by running git revert e4900e3. You created pubsub-sample from the edit-emojify branch instead of from the master branch. Next time, when you create a new branch, just remember to run the following:
git checkout master
git pull origin master
git checkout -b new-branch-name

Ok I lied. Reverting is dicey because when we merge the other PR it could end up reverting those changes as well. So I performed a git rebase -i e4900e3~2 and then I deleted both e4900e3 and the revert commit.

I see. This makes sense! Took notes.


Create initial RFmx NR service with Init/Close.

What does this Pull Request accomplish?
Adds a simple RFmx NR service to the generation project with handcoded metadata for Initialize and Close.
Closes #AB1788691
Why should this Pull Request be merged?
We want to get in an initial version of the RFmx NR service with Init/Close to ensure the framework is set up correctly in grpc-device.
What testing has been done?
Ran the added SystemTests locally with RFmx NR installed.

I wanted to run these on a Windows VM but couldn't get that quite set up so I just resolved to install the driver locally on my dev machine. I'll get the Windows VM set up, though, so future system tests and manual testing later on with Examples can be done with it as the server.


Changed API HOST for retrieving account for evolutiondex

This PR

Old API HOST was throwing the following error:
Fail to retrieve account for evolutiondex

Replace API HOST with:
jungle3.cryptolions.io

Attaching the error image for reference:

Hi @adriexnet
Any update on this PR? I have just changed the endpoint


Fixes

Sorry, some fixes are needed. I had to change all X

did you try to compile it? because files in the res/* folders must be indexed palette png's with the fixed palette order

nope, i only changed the PNG's


[v1.15] Replaced declare_tailcall_if with logic in the loader

[x] #30467 (@dylandreimerink)
[x] #31419 (@mhofstetter)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 30467 31419

/test

/test-backport-1.15

We might also want to include https://github.com/cilium/cilium/pull/31419 where the CI contrib script got fixed which included an issue in my initial PR

/test-backport-1.15

/test-backport-1.15


Exclude a particular file/files?

I have a couple of default image files in my media folder, 'default_m.png' and 'default_f.png' which are assigned to the user as soon as the user is created.
When the user edits his/her profile and uploads a new profile image, the uploaded image is stored in media/profile/ and the default images, (media/default_m.png and media/default_f.png) are deleted.
I would like to exclude these two files from being deleted.

The code only supports one default image. I would advise that the profile image field be nullable as the default and you simply show one of the default images when the profile image is null. With that change, and if desirable, you could move the default images out of the media root and into the static files.


Restructure code base slightly

Move mor.js to a new lib directory.
Move minified file and source map to a new dist directory.

This has now been done and will be included in v1.2.0.


Avoid inheriting from an internal base class for KeyVault PagedResponse Options and expose fields directly

Can we use a trick with the using statement + private inheritance?:

I will take a look at that later especially if we actually want to do this type of inheritance in the future, but we discussed it offline with the team, and we don't need that workaround, since the "right" approach is to not have inheritance here.


[WIP] Moving collector's common from ingress-api-client-ruby

to providers-common gem, because ingress-api-client is built from generator

[x] depends on https://github.com/ManageIQ/topological_inventory-providers-common/pull/1

[x]  Note: Based on https://github.com/ManageIQ/topological_inventory-azure/pull/1, needs rebase

@slemrmartin can you rebase?

@agrare rebased


CrossEntropy --> NLLLoss?

logits is logsoftmax

Thanks for the comment.
Actually CrossEntropyLoss equals LogSoftmax combined with NLLLoss, so we think using CrossEntropyLoss is fine.
If I misunderstand your point, pleaes kindly let us know.
Thanks.


Feedback for "OData Services"

https://help.sap.com/docs/BTP/65de2977205c403bbc107264b8eccf4b/811b5231f77e4987abe8e985e32f70a1.html?locale=en-US
Hi SAP,
in the following Link the Documentation for Interaction between SAP BTP, ABAP Environment and OData is incomplete. The Site of the Link itself is incomplete, aswell as most of the Chapters under this Section are missing/incomplete.
best Regards,
Xaver

Thank you for your feedback! We’ll look into it and come back to you if we have any questions.

Hi @xavergr, the documentation is currently being reworked and will be updated as soon as possible. Thanks again for your contribution!


sfinktah - use addChild to bringToFont without overhead [time 08:25]

Well, there was no 'next' branch, so it's ending up here.

pixi-what-now?  i'll check it out.  i did read a discussion about the proposed introduction of a z-index, and i figured if manipulating the order of children was going to become a more frequently performed operation, that there ought to be some specific code to optimize such things

@sfinktah Thanks for the PR and I appreciate all the inline comments. Two requests: 1). Could you resolve conflicts with dev? and 2). Could you make sure that these cases are covered by the unit tests? (if not please add a tests or two).

@sfinktah I'm closing this PR because of inactivity. If you would like it to be reconsidered, please feel free to re-open and respond to my comments above. Thanks.


Add str and repr for Task, Project, and Label

This is just a convenience feature to get helpful string versions of (what I view as) the core three classes.

Thank you very much for the contribution. I will review this soon.


x/pkgsite: render examples using syscall/js with referenced symbols

What is the URL of the page with the issue?
N/A
What is your user agent?
N/A
Screenshot
No response
What did you do?
Wrote an example function using syscall/js and rendered it with pkgsite.
Currently there is an explicit check to skip syscall/js imports, as code using syscall/js can't be run in the browser. This seems to cause the example to be rendered without any referenced symbols.
While understandable, this is unfortunate as it makes the examples a lot less helpful to the reader, as they miss out on a lot of context. If the author wants the context to be included in the rendered examples, they have to include all referenced types in the function body itself, which means they can't be reused between examples, and also makes the code itself less readable.
It would be helpful if examples using syscall/js were still rendered as standalone programs, including any referenced symbols, even if the run button is disabled/doesn't work.
What did you see happen?
Only the function body was rendered:

What did you expect to see?
The example function to be rendered as a standalone example, with the function body in a function called main and any referenced symbols included, just like examples that don't make use of syscall/js:

Note that from go/doc.Examples https://pkg.go.dev/go/doc#Examples

The example function is self-contained: the function references only identifiers from other packages (or predeclared identifiers, such as "int") and the test file does not include a dot import.
The entire test file is the example: the file contains exactly one example function, zero test, fuzz test, or benchmark function, and at least one top-level function, type, variable, or constant declaration other than the example function.

In the entire test file case, there is no such restriction on syscall/js.

cc @griesemer @agnivade per dev.go.dev/owners


Nested Devices/assets for Map

At this moment, Device search is supported only for one level.
We need to provide support for nested search for devices/assets

Upto 3 level hierarchy is now supported with version 2.1


unable to build kubernetes objects from release manifest: [resource mapping not found for name: "kubescape-scheduler" namespace: "kubescape" from "": no matches for kind "CronJob" in version "batch/v1"

uccessfully ran following 2 commands -
helm repo add kubescape https://kubescape.github.io/helm-charts/
helm repo update
However, I'm receiving following error, while installing kubescape. K8 version I'm using is 1.20.7.
Command I'm executing via PowerSHell in Azure Cluster -
helm upgrade --install kubescape kubescape/kubescape-cloud-operator -n kubescape --create-namespace --set account=myaccountid --set clusterName=kubectl config current-context
Release "kubescape" does not exist. Installing it now.
Error: unable to build kubernetes objects from release manifest: [resource mapping not found for name: "kubescape-scheduler" namespace: "kubescape" from "": no matches for kind "CronJob" in version "batch/v1"
ensure CRDs are installed first, resource mapping not found for name: "kubevuln-scheduler" namespace: "kubescape" from "": no matches for kind "CronJob" in version "batch/v1"
ensure CRDs are installed first]

The Helm chart is trying to create a CronJob object.  Those went GA, i.e. batch/v1 API types were created, in Kubernetes v1.21.  You said you're running v1.20.x.  That will be your problem!
I'll add an issue suggesting that the chart should tell you what the required version is, and give a more obvious error if you're running something older than that.


vue-loader issues since v2.13.0 - compiler options changed?!

Version
v2.13.0
Reproduction link
https://codesandbox.io/s/green-waterfall-yyldo?file=/package.json
Steps to reproduce

write vue component with custom-blocks like <docs>
start nuxt
get errors

What is expected ?
render a simple vue component
What is actually happening?
The vue-loader trys to parse my custom  block which contains markdown.
Additional comments?
I upgraded our software UI from v2.12.2 to v2.13.0 and suddenly every dev-start failed because the vue-loader tried to parse our custom  blocks in our UI-Components.
After a wild hour of debugging I realized that the compiler options for the vue loader plugin changed from v2.12.2 to v2.13.0 but I couldn't see in the release commits which part you guys changed.
My question is: How do I configure nuxt to NOT parse the  custom block?
For now I switch back to v2.12.2 but I really would like to know what changed in the build process.
The error can be seen in this codesandbox: https://yyldo-3000.sse.codesandbox.io/

This bug report is available on Nuxt community (#c10832)

Hi @MrParano1d. It seems issue is from @nuxt/components. Would you please try components: false from nuxt.config? We will meanwhile work on the fix on components module.

I didn't install that module in the codesandbox or is it installed by default now?
Anyway, I added components: false to nuxt.config and still get the error in the codesandbox.

I've encountered a bug similar to this one.
It happened after I upgraded my version from v2.12.2 to v2.13.1.
I'm getting the following errors in the terminal
ERROR in /Users/tanchiky/test_prj/node_modules/@nuxt/types/config/build.d.ts(16,34):
16:34 Cannot find module 'vue-loader'.
    14 | import { Options as PugOptions } from 'pug'
    15 | import { TerserPluginOptions } from 'terser-webpack-plugin'
  > 16 | import { VueLoaderOptions } from 'vue-loader'
       |                                  ^
    17 | import {
    18 |   Configuration as WebpackConfiguration,
    19 |   Loader as WebpackLoader,

What do you know about this?
I hope this information helps someone else.
thanks:)

@Tanchiky I got the same problem. My temporary fix was to execute yarn add<EMAIL_ADDRESS>--dev.

Would you please try recreating yarn.lock? It may both fix vue-loader issue and also components: false (fixed with @nuxt/components v1.0.5)

I updated the sandbox to nuxt.js v2.13.1 and reinstalled all packages. Everything is back to normal now. Thank you :)
@nuxt/components v1.0.5 is fixing it.

It worked @pi0, many thanks. Keep up the good work! :)


Update index.d.ts

Hoist type defs from the logform module.
Currently there isn't a way to write correct type definitions for
when e.g creating a format since the types for such objects are
declared in logform.
This patch move logform type defs so that they can be used by
user code - For example by doing:
import { Logform } from 'winston'; const f: Logform.Format;

Good to see this has been merged, but when can we expect a release to include this?! @DABH

Up to @indexzero ...


Feature request: log-log scale for tensorboard

Would there be interest in adding a log-log display mode to tensorboard? That is, to enable plotting the x-axis in log scale for step count (and maybe relative time as well).
I find log-log to be very useful for inspecting learning curves (loss vs. iteration). Given a constant learning rate, standard optimization algorithms usually give a straight line in log-log scale. It's really easy to visually spot when the straight line behavior ends, which is generally a sign that the model has almost converged (for its current learning rate). On the other hand, in lin-lin you can't distinguish the two conditions because everything looks vaguely like exponential decay, and log-lin isn't ideal either.

@danmane, could you consider this feature?

Hi @nikitakit, thanks for the suggestion!
@bdilday sent a pull request a few months ago (#3876) which added y-axis log scales on a per chart basis. I think it shouldn't be too hard to add x-axis log scales either. The main question in my mind is how to incorporate it into the UI: should it be a per-chart switch (like the y-axis log scales) or a global switch? Right now we coordinate all the x-axes through the global switch on the left, so it might be best to add it through another global setting. (Although I'm reluctant to keep adding checkboxes as a UI paradigm, that might be the right way to accommodate this request.)
The core TensorBoard team has other priorities right now, so we won't likely add this ourselves, but given bdilday's PR as a template, it should be feasible for someone in the community to put this in. I'm marking the issue contributions welcome and un-assigning myself.

I can take this on if no one else is working on it.

Go for it :)

I'm closing this due to inactivity, but if you have a PR, please feel free to submit it to our new repository at https://github.com/tensorflow/tensorboard/pulls! Thanks :-)

I would love an horizontal log scale, too.

This would be awesome to check convergence rates.


Connecting to Google Fit Consent Screen Denying one of multi permissions will not connect

Step 1: Are you in the right place?
Appears to be a feature issue, not an app issue.
Step 2: Describe your environment

Device: Pixel3a, Samsung S8+, and more
OS version: reproduced on Android 9 and Android 10, might be more
SDK version: 28
[Android] Google Play Services version: Gradle uses 16.0.1 for fitness and auth

Step 3: Describe the problem:
My app is trying to connect to Google Fit. The user is prompted to pick a Google Account and once picked, the "Consent Screen" shows up requiring users to accept or deny each permission. There are separate screens for each permission. Once the user has gone through each permission pop up to accept or deny, a main screen appears that shows a list of all the permissions with check marks next to the ones that the user had allowed. If the user denies one or more of the permissions, it usually closes the pop up, shows a spinner, and then restarts the consent screen pop ups to force the user to go through each permission pop up and then continues to the main pop up of the whole list of permissions.
Steps to reproduce:

Have an app call GoogleSignIn.requestPermissions() with the fitnessOptions seen below which prompts the user to pick a Google Account.
Pick a Google Account to use. This then prompts the user to accept specific permissions.
Go through all 7 permission screen pop ups by allowing some but denying some.
After all 7 permissions pop up screens are completed, the user is taken to the last screen which shows a list of all the permissions allowed and denied via check boxes. Have at least one option denied. Hit Accept.

Observed Results:
The pop up disappears, a spinner appears, and it goes back to step #3 where the permissions pop up (1 out of 7) shows with the option to accept or deny.
Expected Results:
The pop up disappears and Google Fit should be connected and have only the allowed permissions accepted.
Relevant Code:
public final static FitnessOptions fitnessOptions = FitnessOptions.builder()
.addDataType(DataType.TYPE_STEP_COUNT_DELTA, FitnessOptions.ACCESS_READ)
.addDataType(DataType.AGGREGATE_STEP_COUNT_DELTA, FitnessOptions.ACCESS_READ)
.addDataType(DataType.TYPE_HEART_RATE_BPM, FitnessOptions.ACCESS_READ)
.addDataType(DataType.TYPE_NUTRITION, FitnessOptions.ACCESS_READ)
.addDataType(HealthDataTypes.TYPE_BLOOD_PRESSURE, FitnessOptions.ACCESS_READ)
.addDataType(HealthDataTypes.TYPE_OXYGEN_SATURATION, FitnessOptions.ACCESS_READ)
.addDataType(HealthDataTypes.TYPE_BLOOD_GLUCOSE, FitnessOptions.ACCESS_READ)
.addDataType(HealthDataTypes.TYPE_MENSTRUATION, FitnessOptions.ACCESS_READ)
.build();
if (!GoogleSignIn.hasPermissions(getLastSignedInAccount(getActivityBase()), fitnessOptions)) {
GoogleSignIn.requestPermissions(
this,
GOOGLE_FIT_PERMISSIONS_REQUEST_CODE,
getLastSignedInAccount(getActivityBase()),
fitnessOptions);
}

@Karringgton thanks for the report.  This seems like a really specific Google Fit issue and I don't think I can help you.
I would suggest filing an issue here:
https://github.com/android/fit-samples
And mention @codingjeremy in the issue, hopefully that will get this in front of the right people

@Karringgton We are also facing the same issue you mentioned. Please let us know if you were able to find a solution for that. Eagerly waiting for your response.

@Karringgton me too facing same issue in google fit consent screen in ionic cordova application, Im using cordova-plugin-health to fetch health data from google fit.
Please let us know, if you need more information.

@Karringgton I am also facing the same issue in google fit consent screen.
Please let me know, if you need more information or did you find any solution to it?
@codingjeremy any updates on this?


User has to provide offset to get time local time

Description:
If the user knows his/her timezone "Asia/Colombo" ballerina can keep track of default offsets. There are default timezone data used in other languages.
eg: IANA Time Zone Database (TZDB)
Currently user has to provide all of these.
Describe your problem(s)
Describe your solution(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Time zones are not yet supported in Ballerina. We are currently working on the design.

Closing this since we have support for Zone APIs


Incongruent direction indentation

(Also added a missing check in the congruent-directions case.)
Note that bidirectional text currently doesn't wrap correctly -- see pop-os/cosmic-text#252 for details. This PR only ensures that the first line accounts for indentation.

Yes, unfortunately. I have the tests running on the Warp side to show this working though.


Remove built dependency

This was pulling in git2 and writing a lot of data that was never used
in the crate. This just changes build.rs to manually read the HEAD
commit information and set an env var.
AFAICT this is actually superior in every way to built since we no
longer have the extra dependencies, and we actually get up to date
information when the commit changes, which AFAICT was not the case with
built since it was not registering file changes.

These tests weren't flaky until today 😄

These tests weren't flaky until today 😄

Yeah! It's being really grumpy! I wonder what's up with that.
If you see a flake, please chuck a comment in the PR of a c/p paste of the flake in the PR, and I'll do the same - that makes it easier to search for common flakes through GitHub search and eventually resolve (also we delete old build failures, so it's harder to see the history).

If you see a flake, please chuck a comment in the PR of a c/p paste of the flake in the PR, and I'll do the same - that makes it easier to search for common flakes through GitHub search and eventually resolve (also we delete old build failures, so it's harder to see the history).

It wasn't even the same test though which is odd, it was different tests.

It wasn't even the same test though which is odd, it was different tests.

Could be Cloud Build is having a grumpy day.


[Next.Ink] Bridge rewrite (#3863)

Reimplement NexINpact bridge, now named Next.Ink bridge. Fixes #3863.
Note that due to major changes on the website, I had to change some parameters so the new bridge is not compatible with existing feeds and they need to be recreated. I took this opportunity to rename the file.

There is an issue with the diff.

I have an issue with my fork and will reopen a new pull request once my other pull requests get merged.


[release-1.4] Update to common-files to implement docker.push (#207)

Manual cherry-pick of aece88de0ff43ff319434a52d7d70dbc4f328a56.
Conflicts:
common/.commonfiles.sha
common/Makefile.common.mk

There are a few problems with this PR:

The common-files SHA is a _master_branch SHA.
The UPDATE branch in the Makefile is master which will force a new update-common to pull from master.

Based on earlier discussion, I think we want to cherry-pick https://github.com/istio/common-files/pull/111 pack to release-1.4 and then we'll do the make update-common to get the changes. Please correct me if I misunderstood @howardjohn .

Yes that is correct, we should be using 1.4 specific common files.
Do we really need this in 1.4 anyways?
On Fri, Oct 25, 2019 at 6:16 AM Eric Van Norman<EMAIL_ADDRESS>wrote:

There are a few problems with this PR:

The common-files SHA is a _master_branch SHA.
The UPDATE branch in the Makefile is master which will force a
new update-common to pull from master.

Based on earlier discussion, I think we want to cherry-pick
istio/common-files#111 https://github.com/istio/common-files/pull/111
pack to release-1.4 and then we'll do the make update-common to get the
changes. Please correct me if I misunderstood @howardjohn
https://github.com/howardjohn .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/cni/pull/208?email_source=notifications&email_token=AAEYGXOVK76VRA6TYXSA44TQQLWR5A5CNFSM4JEU7GTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECIKC7I#issuecomment-546349437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNSEOVX73HPADCSBSTQQLWR5ANCNFSM4JEU7GTA
.

I don't mind following a different process for common-files then the rest of Istio even though it is inconsistent.  @ericvn what you propose makes more sense.  I'm not sure we have done a good job of backporting common-files fixes to the 1.4 branch.
We don't need docker push, but we do need other parts of this common-files PR (the stuff that sets the TARGET_OUT dir, as well as the latest images) so that the architecture is set correctly.
@geeknoid may have some thoughts on the rollout of common-files to the master branches.  He has undertaken this for master.  I am unclear on his recommended approach for 1.4.
Cheers
-steve

spoke with @geeknoid in general about our 1.4 rollout plans are for the repos.  He will sort out cni, so closing this PR.
Cheers
-steve


Feature: Added filetypes as arguments to font mixin.

The font mixin will request every type defined in src even if the file doesn't exist. To prevent wasted HTTP request I've added each font type as an argument with a default value of false. Within the mixin contents each filetype argument is checked for a value of true before outputting the src property, meaning only required filetypes are requested.
Example usage;
@include font('My font', 'myfonturl', $eot:true, $woff:$true);

@jtuds is this one still relevant?

It's still relevant but confident in the code. I've since changed this mixin in other production code. Need to review it

merged into solution, merged via another pull request because I am a genius


Dep ensure -add fails when github username changes on required package

When running dep to get the certstrap project, it relies on a package from a user once named 'codegangsta'. That repo now points to https://github.com/urfave/cli (was
https://github.com/codegangsta/cli). Github correctly redirects, but dep does not.
What version of dep are you using (dep version)?

v0.4.1-115-g7d5cd19
What dep command did you run?
dep ensure -add github.com/square/certstrap

What did you expect to see?
Success
What did you see instead?
Solving failure: No versions of github.com/square/certstrap met constraints:
v1.1.1: "github.com/square/certstrap" imports "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli", which contains malformed code: no package exists at "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli"
v1.1.0: "github.com/square/certstrap" imports "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli", which contains malformed code: no package exists at "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli"
v1.0.1: "github.com/square/certstrap" imports "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli", which contains malformed code: no package exists at "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli"
v1.0.0: "github.com/square/certstrap" imports "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli", which contains malformed code: no package exists at "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli"
master: "github.com/square/certstrap" imports "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli", which contains malformed code: no package exists at "github.com/square/certstrap/Godeps/_workspace/src/github.com/codegangsta/cli"

I have same problem with private repo.
I fixed by this way but it's not a good solution.
Step

Pull your target project by manual. (In this case is "codegangsta")
add this line to "Gopkg.toml" of your work project
ignored = ["git.private-repo.com/path/your-target-project/*"]
refer form https://github.com/golang/dep/blob/master/docs/Gopkg.toml.md#ignored
run "dep ensure" again

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Update play-json to 2.9.1

Updates com.typesafe.play:play-json from 2.8.1 to 2.9.1.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.play", artifactId = "play-json" } ]

labels: library-update, semver-minor

Superseded by #116.

Superseded by #116.


Get number of keys down

Allow developers to get the number of keys down at one time

Dropped for the same reason as #46

it's already possible
it makes more sense to do it with the functions already provided

In all the function would just be syntax sugar for this code
local total_pressed = 0
for k = -1, BongoInput.KeyCount - 1, 1 do
    if isPressed(k) then
        total_pressed += 1
    end
end


[Discussion/Question] Impact of changes on HIBP API

Hi!
Today, Troy Hunt announced via Twitter that there will be some changes on how the API of HIBP will be working on 4 weeks from today (that is, Aug. 18), and these changes are detailed on a post in his blog.
Basically, the TL;DR is that (although I recommend to read the post, as it explains the "why" in a detailed way):

There will be an authentication process.
There will be a cost/subscription model.

So, well, although I have a few ideas about how that will impact to several services/applications that consume the provided data, I wanted to ask if you:

Have already seen the post.
Have though about what that will mean to the project itself[1].

Thanks in advance.
Kind regards.

[1] : I Know that it will be very early/premature to have something to say about this one, considering that it has been less than a day since the news broke, but nonetheless I wanted to ask (before I forget to ask, tbh)...

With the new API, this plugin isn;t working as it used to. Is this still under active development? If I invest in an API key for haveibeenpwned.com, can I utilize it with this plugin?

I actualy just changed it so you could use it with your own api key. But it is not yet tested...
For now I decided to obtain the API Key by searching for an entry with the title "hibp-apikey" in the database and use the password of that entry as api key.
As it is my first work on an Keepass Plugin I used this mechanism so i don't have to worry with UI and persistency.
Maybe you could provide some input to this question?

By the way the two options "check by domain" and "check by password" still work fine as those API's were not affected by the above mentioned changes.

Thanks to @jakob-ledermann the latest release of the plugin now fully supports the v3 authentication mechanism :)


Form submission sample

Status
READY
JIRA Ticket
SPACON-562
Description
Added an endpoint that the sdks can use to generate sample form submissions.  This will allow us to write integration tests for submitting forms.

Also fixed a bug adding/removing users to/from teams.

Hey man, great job! 👍


Reduce Release build leaking (due to Swift 2.x bug) (#433)

Replace some instances of String interpolation, to reduce leaking in Release build on Swift 2.2.
See: #433
As well as this comment from an XCGLogger issue:

So I spent a tonne of time digging into the memory leaks. The leaks are actually in the Swift core library and our code just happened to trigger them. As you probably noticed, judging by the PR, the leak is triggered in some cases using string interpolation, so changing affected code from: "[(owner.identifier)] " to "[" + owner.identifier + "] " avoids the leak.

Worth noting the leaks appear to be fixed in Swift 3.

Hmm, unit tests failures:
[16:18:56]: ▸ WillFinishObserverTests
[16:18:56]: ▸ ✓ test__did_attach_block_is_called (0.000 seconds)
[16:18:56]: ▸ ✓ test__will_finish_block_is_called (0.001 seconds)
[16:18:56]: ▸ ✓ test__will_finish_block_with_errors (0.005 seconds)
[16:18:56]: ▸ OS_X_OperationsTests.GroupOperationTests
[16:18:56]: ▸ test__group_operation_cancellation__queue_is_empty_when_finished, XCTAssertTrue failed -
[16:18:56]: ▸ /usr/local/var/buildkite-agent/builds/Meera Reed/blindingskies/procedurekit/Tests/Core/GroupOperationTests.swift:399
[16:18:56]: ▸ ```
[16:18:56]: ▸             XCTAssertEqual(group.queue.operations.count, 0)
[16:18:56]: ▸             XCTAssertTrue(group.queue.suspended)
[16:18:56]: ▸         }
[16:18:56]: ▸   ```
[16:18:56]: ▸ test__group_operation_cancellation__queue_is_empty_when_finished, XCTAssertTrue failed -
[16:18:56]: ▸ /usr/local/var/buildkite-agent/builds/Meera Reed/blindingskies/procedurekit/Tests/Core/GroupOperationTests.swift:399
[16:18:56]: ▸ ```
[16:18:56]: ▸             XCTAssertEqual(group.queue.operations.count, 0)
[16:18:56]: ▸             XCTAssertTrue(group.queue.suspended)
[16:18:56]: ▸         }
[16:18:56]: ▸   ```
[16:18:56]: ▸ 	 Executed 558 tests, with 2 failures (0 unexpected) in 24.354 (24.795) seconds
[16:18:56]: ▸
[16:18:56]: ▸ Failing tests:
[16:18:56]: ▸ -[GroupOperationTests test__group_operation_cancellation__queue_is_empty_when_finished()]
[16:18:56]: ▸ ** TEST FAILED **

Will just retry this...

@swiftlyfalling - I'm going to merge this, and then pull it into the 3.2.0 RC, which I'm going to release in a bit. 🚀


updating the readme to add clarity to cert docs and developer testing

Updating readme to point to official certification docs for clarity
Updating the testing section so parters know these are not tests they run, but tests for developers.
JIRA

@taylorjordanNC Please take a look at let us know if you have more suggestions. Thanks.

lgtm!

Coverage: 81.792%. Remained the same when pulling baf8a60ed942b681cf02849f126530afff7c46ff on acornett21:update_readme into be3f9b6644331c7f6f615a0bb2440dc8d0c2a485 on redhat-openshift-ecosystem:main.

/retest

This looks like it does more than just update for the official docs? Do we want to split this into two PRs?
@bcrochet Both of these asks were in the JIRA (which I added above). I've updated the title of this PR for clarity, since it only pulled the first half of the commit message.


Alternate Wasm Import for running without webserver

Since WASM uses fetch to load, running the html file does not work and results in a cors error, the only way around this is by loading the wasm as a data uri that is in base64 since the inline wasm limit is 4kb. Is this something that has been done or can be done? I tried seeing if I could do it but I'm somewhat new to JS and don't know how to use WASM. (I know you can get around CORS using a command line argument when launching chrome but I cant do that on a chromebook)

@Tarnadas Take a look at this

Are you really sure that you want to run this without a web server?
In theory something like this might work:
https://webassembly.org/getting-started/js-api/
But even if you bundle the wasm, there are plenty of other resources that the html file would want to load. You can really just use the most basic http server. I personally use this sometimes:
https://github.com/brson/basic-http-server
But plenty of other options available.
If you want to bundle everything in the html file I would try a build tool like Vitejs, but I would rather want to avoid you to go through the painful process of making this work.

 In theory something like this might work: https://webassembly.org/getting-started/js-api/
But even if you bundle the wasm, there are plenty of other resources that the html file would want to load. You can really just use the most basic http server. I personally use this sometimes: https://github.com/brson/basic-http-server But plenty of other options available.
If you want to bundle everything in the html file I would try a build tool like Vitejs, but I would rather want to avoid you to go through the painful process of making this work.

Random question tho how would you use https://github.com/sm64js/sm64js-mmo-server on a web server like: https://example.com or sub like https://test.example.com as basic-http-server says in the readme: Note that basic-http-server is not production-ready and should not be exposed to the internet. It is a learning and development tool. so it wouldnt work on the actual internet like something: example.org

@Tarnadas Unless there is a way to somehow use the basic-http-server for production besides sm64js-mmo-server (likely not)

Well, then you use some web server that is production ready 😄
We provide Docker builds with the server that is used in prod, but it bundles the API server and everything into one image. It uses actix-web as web server.
https://github.com/sm64js/sm64js-mmo-server/blob/main/Dockerfile

@Tarnadas Could you provide to the link to these(s) or a guide for the ones that literally cant be downloaded directly like the web server (web server guide by any chance)

@Tarnadas also one last thing (sorry for pinging you so much here) tho im using namecheap as my hosting provider and domain provider

Well, then you use some web server that is production ready 😄 We provide Docker builds with the server that is used in prod, but it bundles the API server and everything into one image.  Im trying to make sm64js-mmo-server work on my web server https://mmo.sm64jsarchive.com for testing my own features i could add into sm64js tho i cant get the actual website/rust server for it to actually load like assisting me via My Email (btw its something for fun entirely)

@Tarnadas Im sorry if i sound like a total to complete idiot here, tho could you assist me? Im trying to make sm64js-mmo-server work on my web server https://mmo.sm64jsarchive.com/ for testing my own features i could add into sm64js tho i cant get the actual website/rust server for it to actually load like assisting me via My Email (btw its something for fun entirely) Also if this is a waste of your time I am actually very very Sorry if this wasted your time at all.

@Tarnadas I am genuinely confused as for some reason I cant find out how to properly change localhost:3060 to https://mmo.sm64jsarchive.com (or just mmo.sm64jsarchive.com) also I havent found out how to setup the docker-compose to My Web Server like get the docker-compose to the web server using Docker. Instead of a development one Im basically trying to get one for production fully up and running.

@Tarnadas Also I have everything setup in folders (from https://sm64js.com/apidoc/ before it got removed) like /api/ /api/login/ /apispec/ /apidoc/ i got it all i got everything neded in /api/ like /api/account etc

Can you run everything locally? Before deploying it make sure that it runs locally:
https://github.com/sm64js/sm64js-mmo-server?tab=readme-ov-file#prerequisites
Deploying it for production comes later :smile:
It involves setting up an Nginx reverse proxy or similar which auto regenerates certificates for HTTPS and then pointing your subdomain to your webserver.

Can you run everything locally? Before deploying it make sure that it runs locally: https://github.com/sm64js/sm64js-mmo-server?tab=readme-ov-file#prerequisites
Deploying it for production comes later 😄 It involves setting up an Nginx reverse proxy or similar which auto regenerates certificates for HTTPS and then pointing your subdomain to your webserver.

How do I set up nginx? I've tried setting up nginx before tho it never worked. Also Tarnadas I'll attempt to run it locally.

@Tarnadas Also sadly I only have Stellar Plus (Shared Hosting) on Namecheap. So I dont know what to do like Is there any simpler method? like Modify the code myself to meet my requirements or How would Production work like that without modifying (Unless you have to modify the code If you do I could use some assistance.)

Or @Tarnadas Since you said nginx or **similar could I use Apache? (Stellar Plus on namecheap.)

The HTML file can access the other js files, but cannot make fetch request due to CORS (even though wasm is a file). The only thing that needs to be replaced is the fetch request to wasm and it should work. I can try disabling CORS when I get home on my personal computer to see if the rest runs fine, if it does, then we know the only thing that needs to be replaced is the fetch request when locally loading.

Can you run everything locally? Before deploying it make sure that it runs locally: https://github.com/sm64js/sm64js-mmo-server?tab=readme-ov-file#prerequisites
Deploying it for production comes later 😄 It involves setting up an Nginx reverse proxy or similar which auto regenerates certificates for HTTPS and then pointing your subdomain to your webserver.

New update: I found out how to set it up for production on a web server after a while!  @Tarnadas :D

@Tarnadas Btw tarnadas for web server production (using sm64js-mmo-server repo) how can I create a postgres database? with both internal and external databaseurl (connection string so i can put it in .env database url thing in sm64js-mmo-server repo)

@Tarnadas Nvm i thought i had it for production dangit i didnt i'll post a seperate issue in sm64js-mmo-server repo instead

@Tarnadas Yo these are my docker logs: 2024-03-24 12:02:51 thread 'main' panicked at 'called Result::unwrap() on an Err value: NotPresent', crates/sm64js-env/src/lib.rs:56:51
2024-03-24 12:02:51 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
2024-03-24 12:02:51 Panic in Arbiter thread.
If you can please deploy a fix for this issue

@Tarnadas I learned how to setup everythign like my own docker images / servers, postgres, etc (everything needed) You have been a big help on my development journey I followed all your tips and I now know how to understand and do it all! As I know use Docker more often

Can you run everything locally? Before deploying it make sure that it runs locally: https://github.com/sm64js/sm64js-mmo-server?tab=readme-ov-file#prerequisites
Deploying it for production comes later 😄 It involves setting up an Nginx reverse proxy or similar which auto regenerates certificates for HTTPS and then pointing your subdomain to your webserver.

I learned how to point it to my own subdomain, thank you Tarnadas your steps worked for me!


not find compatible versions for pod "Bytedance-UnionAD"

flutter编译运行的时候pods报错：
[!] CocoaPods could not find compatible versions for pod "Bytedance-UnionAD":
In Podfile:
pangle_flutter (from .symlinks/plugins/pangle_flutter/ios) was resolved to 0.1.11, which depends on
Bytedance-UnionAD (~> <IP_ADDRESS>)
Specs satisfying the Bytedance-UnionAD (~> <IP_ADDRESS>) dependency were found, but they required a higher minimum deployment target.

Bytedance-UnionAD最低支持9.0
根据提示，在iOS项目下的Podfile开头增加这一句
# Uncomment this line to define a global platform for your project
platform :ios, '9.0'


fix: correctly accept JIMP instances

Passing JIMP instances in the first argument doesn't work as docs suggest.
#10

Thanks!


Add source to steckbrief shortdescription

This PR adds source information below the short description for every rsv steckbrief.
Closes  #52

Nice, looks good! I'm gonna merge this.


new pull request will be sent after the current one is closed.

Hi Makk,
I made few changes on our proposal. But because I have been having troubles opening new pull request (please see the first issue i sent), I will wait till you accept the existing pull request and I will send another one if it allows me. Please let me know when you are done.
Thanks.
Fan

I guess the reason that I could not open a new pull request is because it has already taken all the changes/commits I made on my github.com repo and "secretly" added them on the open pull request. So I don't need to open another one. Learn something new everyday!


RN Screens Patch

The Patch for RN Screens is failing. My current version is:     "react-native-screens": "^3.10.2",

I'll take a quick look, can you use version 3.4.0 in the meantime?

I pushed a commit that fixes it. Looks like you can remove the patch for RNScreens if you are using 3.10.2.

i just tried this and im now getting the error:   Invariant Violation: requireNativeComponent: "RNCViewPager" was not found in the UIManager. I think @react-navigation/material-top-tabs installs this as a needed dependency

Is this happening on Android or tvOS?

TVOS

I'm not seeing this issue.  Be sure to do
cd TVReanimated
rm -r node_modules ios/Pods ios/Podfile.lock
yarn
cd ios
pod install
open TVReanimated.xcworkspace

Kill any running packagers. Then build and run the TV target.

is there any way possible this can work with updated patches/dependency versions and the new 68.2-2 release?

@mikevocalz :  Yes it can. See for example the dependencies in https://github.com/douglowder/TVInputDemo .
I'll spend some time this week updating this repo for 68.2-2.

Actually the update went pretty quick, give it a try 😃

Thanks so much. it actually works great on androidtv but doesnt load on tvOS


Don't work as  expected After modify timezone.

Version: 0.8.27
If  I modify time zone to "Asia/Shanghai".
data range picker will display as expected, but pick a time range and query, can't return expected result.e.g if selected interval is 14:00~15:00,but return the data of 6:00 ~ 7:00.
I found that  the interval that was posted to pivot server from browser  use UTC timezone

"actions":[{"action":"in","expression":{"op":"literal","value":{"start":"2016-05-12T06:44:00.000Z","end":"2016-05-12T06:54:00.000Z"},"type":"TIME_RANGE"}}

Plywood will translate this interval use method dateToIntervalPart
function dateToIntervalPart(date) {
        return date.toISOString()
            .replace('Z', '')
            .replace('.000', '')
            .replace(/:00$/, '')
            .replace(/:00$/, '')
            .replace(/T00$/, '');
    }

Just drop the last letter "Z",the date will use local timezone "Asia/shanghai",but the result is wrong.
the query json like this
{"queryType":"timeseries","dataSource":"dsp_analysis","intervals":"2016-05-12T06:44/2016-05-12T06:54","granularity":{"type":"period","period":"PT1M","timeZone":"Asia/Shanghai"},"context":{"timeout":30000,"skipEmptyBuckets":"true"},"filter":{"type":"and","fields":[{"type":"selector","dimension":"cast","value":"15748"},{"type":"or","fields":[{"type":"selector","dimension":"idea","value":"22195"},{"type":"selector","dimension":"idea","value":"22204"}]}]},"aggregations":[{"name":"pv_sum","type":"doubleSum","fieldName":"pv_sum"},{"name":"bid_sum","type":"doubleSum","fieldName":"bid_sum"},{"name":"consume_sum","type":"doubleSum","fieldName":"consume_sum"}]}

related to #161 #41

I have found the cause. Because My druid use local timezone 'Asia/shanghai'.
But I don't why drop the last letter 'Z'

Hi. Plywood assumes that Druid is running in Etc/UTC that is why it drops the 'Z'. It probably should not do it. I have never encountered a Druid cluster not running in UTC which is why we never detected it in the test. Will fix.

Hi, just consulted some Druid committers. I will modify Plywood to explicitly send 'Z' over in the interval and it may fix your issue but you should change the timezone of your Druid server to UTC. There are a lot of things in Druid that assume timezones being evaluated in UTC.

Hi Pivot will send Z in version >= 0.8.28. Please still consider switching your Druid to be in UTC timezone as many other things (Druid internal) might not work right for you.

@vogievetsky Thanks
My Druid using  Asia/shanghai timezone has been running for over half year,hadn't found any Time Zone problem so far.
Druid always  transform the interval of query to the given time zone by java environment. e.g.
-Duser.timezone="Asia/Shanghai",the interval "2016-05-13T12Z/2016-05-13T14Z"  will be transformed to "2016-05-13T20:00:00.000+08:00/2016-05-13T21:00:00.000+08:00".
So I don't need set timezone to UTC ,
The interval with Timezone is a good idea.No matter which time zone druid use,it always work as expected.


chore: added prettier import order sorting to the monorepo

Closes #
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
In #1392 we may be adding prettier import sorting to generated cta apps. this pr adds the same but to the monorepo instead.
looking over the format changes i think they all look fine. i couldn't find any of the problem ones that we don't in #1392

Screenshots
no lol
💯

Wondering if i should revert the formats on the template files... probbaly could be handled in #1413. then again that would cause format linting to fail...


"sam package" uploads file to s3 even if the Lambda function code hasn't changed

Description:
When CodeUri is set to a local directory where the Lambda function code resides, sam package uploads the package to the s3 bucket even if there hasn't been any changes to the local directory. This causes a file to be uploaded to the s3 bucket every time sam package is run, even if there isn't any change to the Lambda function code.
Is there any way for sam to only upload the file when there are changes?
This is how I use the sam package command:
sam build --use-container --template FunctionV1.yaml
sam package --s3-bucket ${my_s3_bucket} --output-template-file FunctionV1-packaged.yaml \
    --region ${AWS_REGION}
sam deploy ...

Please see below for the function definition:
  FunctionV1:
    Type: AWS::Serverless::Function
    Properties:
      FunctionName: MyFunctionV1
      Description: MyFunctionV1
      CodeUri: ../src/v1/
      Handler: lambda_function.lambda_handler
      Runtime: python3.7
      Timeout: 120
      MemorySize: 128
      AutoPublishAlias: live

I have this same problem and I wish there was a native builtin function to calculate the SHA of a file (e.g. Terraform has filesha256()). What I have been doing for my workaround is in my deploy script, after I run my SAM package, I then loop through the binaries/code that need to be deployed and I run the following command to get the SHA256:
SHA256=$(openssl dgst -sha256 -binary cmd/$function/release/$function | openssl enc -base64 | tr -d "=+/")

$function is the name of the function in the SAM as well as the name of the binary (GoLang for use, but you could hash any code file). After I get the SHA256 I then insert the "AutoPublishCodeSha256" property with the SHA value using yq. This looks like this:
yq w -i packaged.yaml "Resources.$function.Properties.AutoPublishCodeSha256" "$SHA256"

This will cause SAM to not create a new version if the SHA is the same. It will still upload the code unfortunately and deploy a $LATEST Version, but your alias will still point to the latest "versioned" change (e.g 1,2,3 etc.)

This should no longer happen with version of SAM CLI after 0.42.0. We now sha the contents of the file: https://github.com/awslabs/aws-sam-cli/pull/1789
Closing


Multiplex addon: Add open and close tokens when highlighting delimiters

This simple change adds extra open and close token styles to multiplexing delimiters when delimStyle is set. This makes it easy to style open and close delimiter tokens as in this example:

For example, if delimStyle is set to delim, then the opening delimiter gets the following token styles: cm-delim and cm-delim-open and the closing delimiter gets cm-delim and cm-delim-close.

Looks useful. Merged.


Add Custom props to tree node

What problem does this feature solve?
Can we have feature that node can contains more information other than title, key, isLeaf etc
What does the proposed API look like?
have different icon based on json,
can display more information instead only title,
can have html template

Hi @hsbalar
please read the doc
https://ng.ant.design/components/tree/en#components-tree-demo-customized-icon
you can change all template if you want
https://stackblitz.com/edit/angular-d3ury1?file=app/app.component.ts

@hsbalar it is very easy to custom your own tree using template, it can transform any json to the template you want.

I'm not able to provide custom JSON, when Initialising tree.
nodes = [
    new NzTreeNode({
      title   : 'root1',
      key     : '1001',
      type    : 'a',
      status  :''
  })
];

@simplejason

@hsbalar https://stackblitz.com/edit/angular-d3ury1?file=app/app.component.ts
you can use this as temp solution, there are some typescript definition lost in our interface,
thanks for your feedback, we will fix it in next version.

Your welcome @vthinkxie.

#1584


perf(UpdatePrompt): only check for updates every 6 hours

The update prompt was checking for new updates to VRTK every time Unity
was started or recompiled the project. This change makes sure to only
check every 6 hours.

Rebased on latest master and resolved conflicts.


undefined is not an object (evaluating n.reactCurrentOwner

After adding all my necessary npm packages on my app and try to run, I'm en counteracting  this bug. I have tried the following and still same bug
watchman watch-del-all
rm -rf ./node_modules
yarn cache clean
rm -rf $TMPDIR/react-*
yarn install
npm start -- --reset-cache

I've also tried to update everything to latest version and still same bug. i.e
react-native: 0.45
react: 16.0.0-alpha.12

Anybody else with similar issue?
Environment
react-native-cli: 1.0.0
react-native: 0.44.0
react: 16.0.0-alpha.6
node: v6.10.0
npm: 4.5.0
yarn: 0.27.5

Platform: Android
OS: Windows
Android Studio
Steps to Reproduce
react-native run-android
Expected Behavior
It's supposed to show app main page
parsing template literals
Actual Behavior
red error page with
undefined is not an object (evaluating n.reactCurrentOwner

Hi there! Is this happening for you on a blank React Native project? react-native init SomeProject and then react-native run-android causes the problem for you? If not, can you share an example project with this happening? Thanks!

I'm stuck on this issue too,
I'm currently using those versions:
react: 16.0.0-alpha.12
react-native: 0.45.1
npm: 5.0.3
node: v8.1.3
yarn: v0.24.6

hi there,
same problem here.
app.js
import React from 'react';
import {Text, View, AppRegistry} from "react-native";

export default class App extends React.Component {
    constructor() {
        super();
    }
    render() {
        return (
            <View style={{
                margin: 50
            }}>
                <Text>Hallo 1.0.1</Text>
            </View>
        );
    }
}
AppRegistry.registerComponent('main', () => App);

package.json
{
	"name": "nacked-app",
	"version": "1.0.0",
	"private": true,
	"scripts": {
		"exp-start": "exp start --clear",
		"exp-build-ios": "exp build:ios",
		"exp-build-android": "exp build:android",
		"exp-build-status": "exp build:status",
		"reset1": "rimraf \"C:/Users/xxxxxxxxx/AppData/Roaming/npm-cache/*\"",
		"reset2": "rimraf \"C:/Users/xxxxxxxxx/AppData/Local/Temp/react-*\"",
		"reset3": "rimraf \"C:/Users/xxxxxxxxx/AppData/Local/Temp/npm-*\"",
		"reset4": "rimraf node_modules",
		"reset5": "npm cache clean",
		"reset6": "npm i",
		"reset7": "node node_modules/react-native/local-cli/cli.js start --reset-cache",
		"log1": "node node_modules/react-native/local-cli/cli.js log-android"
	},
	"dependencies": {
		"expo": "18.0.0",
		"react": "16.0.0-alpha.6",
		"react-native": "github:exponent/react-native#sdk-18.0.0"
	},
	"devDependencies": {
		"cross-env": "5.0.1",
		"exp": "42.1.0",
		"npm-check-updates": "2.12.1",
		"rimraf": "2.6.1"
	}
}

exp.json
{
	"version": "1.0.0",
	"name": "nackedapp",
	"slug": "nackedapp",
	"privacy": "unlisted",
	"entryPoint": "app.js",
	"icon": "expo.png",
	"orientation": "portrait",
	"sdkVersion": "18.0.0",
	"packagerOpts": {
		"assetExts": [
			"ttf"
		]
	},
	"loading": {
		"icon": "src/icons/expo.png"
	},
	"ios": {
		"bundleIdentifier": "com.nackedapp.exp.ios",
		"config": {
			"usesNonExemptEncryption": false
		}
	},
	"android": {
		"package": "com.nackedapp.exp.android"
	}
}

thanks!
greetings crazyx13th

works again with:
"expo": "18.0.9",
"react-native": "0.46.1"

sorry,
"react": "16.0.0-alpha.6" i got this error
"react": "16.0.0-alpha.7" i got another error
"react": "16.0.0-alpha.8" seems to works

blank project from react-native init somename works fine. The issue is actually when I build a debug or release apk and run it. running react-native run-android works fine. Not sure what's being broken on building apk and installing it on device/emulator. here is my package.json
{
"name": "my app",
"version": "1.0.0",
"description": "My coolest app",
"author": "Aroniez",
"private": true,
"scripts": {
"start": "node node_modules/react-native/local-cli/cli.js start",
"test": "jest"
},
"dependencies": {
"firebase": "^3.6.4",
"html-entities": "^1.2.1",
"lodash": "^4.17.2",
"moment": "^2.17.1",
"react": "16.0.0-alpha.6",
"react-native": "0.44.2",
"react-native-admob": "^1.3.0",
"react-native-animatable": "^1.1.1",
"react-native-blur": "^3.0.0-alpha",
"react-native-drawer": "2.",
"react-native-dropdownalert": "^2.3.0",
"react-native-fbsdk": "^0.6.0",
"react-native-fence-html": "^1.0.6",
"react-native-google-analytics-bridge": "^5.0.1",
"react-native-google-signin": "^0.10.0",
"react-native-invertible-scroll-view": "^1.0.0",
"react-native-linear-gradient": "^2.0.0",
"react-native-modalbox": "^1.3.9",
"react-native-onesignal": "^3.0.4",
"react-native-render-html": "^2.0.0",
"react-native-scrollable-tab-view": "0.",
"react-native-snap-carousel": "2.1.1",
"react-native-swiper": "1.",
"react-native-timeago": "0.",
"react-native-vector-icons": "^4.2.0",
"react-navigation": "^1.0.0-beta.11",
"react-redux": "^5.0.4",
"react-timer-mixin": "^0.13.3",
"react-tween-state": "^0.1.5",
"redux": "^3.6.0",
"redux-persist": "^4.8.0",
"redux-thunk": "^2.2.0",
"sanitize-html": "^1.13.0",
"striptags": "^3.0.1",
"urijs": "^1.18.4",
"url": "^0.11.0",
"util": "^0.10.3",
"wpapi": "^1.1.1"
},
"devDependencies": {
"babel-jest": "20.0.3",
"babel-preset-react-native": "1.9.2",
"jest": "20.0.4",
"react-test-renderer": "16.0.0-alpha.6"
},
"jest": {
"preset": "react-native"
}
}

@crazyx13th "react": "16.0.0-alpha.8" works fine?

yupp, inside my minimal test project

Let me try that

I'm experiencing a similar issue issue when trying to add react-native-web to an existing react-native project: Cannot read property 'ReactCurrentOwner' of undefined. react-native-web currently requires react v15.4.x which is a different version than the one installed with the current version of react-native. To fix the issue I had to downgrade react-native to 0.42.3 and set react, react-dom, and react-test-renderer to 15.4.1.
tl;dr make sure the versions of react in your dependencies are matching.


Cannot read properties of undefined (reading 'filter')  @nuxt/kit

Environment
Local environment
System:
    OS: Windows 10 10.0.19045
    CPU: (8) x64 AMD FX(tm)-8350 Eight-Core Processor
    Memory: 6.82 GB / 19.96 GB
  Binaries:
    Node: 18.18.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.5 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 9.8.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.3636.0), Chromium (119.0.2151.93)
    Internet Explorer: 11.0.19041.3636

Reproduction

Describe the bug
I'm trying to deploy the SSR application on Heroku.
I do not know what else to do. I've been trying for 3 days...
The application works perfectly in the local environment.
Additional context
No response
Logs
Heroku Logs

-----> Building on the Heroku-22 stack
-----> Using buildpack: heroku/nodejs
-----> Node.js app detected

-----> Creating runtime environment

       NPM_CONFIG_LOGLEVEL=error
       NODE_VERBOSE=false
       NODE_ENV=development
       NODE_MODULES_CACHE=true

-----> Installing binaries
       engines.node (package.json):  18.x
       engines.npm (package.json):   unspecified (use default)
       engines.yarn (package.json):  1.x

       Resolving node version 18.x...
       Downloading and installing node 18.18.2...
       Using default npm version: 9.8.1
       Resolving yarn version 1.x...
       Downloading and installing yarn (1.22.21)
       Installed yarn 1.22.21

-----> Installing dependencies
       Installing node modules (package.json)

       added 2793 packages, and audited 2795 packages in 1m

       482 packages are looking for funding
         run `npm fund` for details

       40 vulnerabilities (1 low, 20 moderate, 17 high, 2 critical)

       To address all issues possible (including breaking changes), run:
         npm audit fix --force

       Some issues need review, and may require choosing
       a different dependency.

       Run `npm audit` for details.
       npm notice
       npm notice New major version of npm available! 9.8.1 -> 10.2.4
       npm notice Changelog: <https://github.com/npm/cli/releases/tag/v10.2.4>
       npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
       npm notice

-----> Build
       Detected both "build" and "heroku-postbuild" scripts
       Running heroku-postbuild

       ><EMAIL_ADDRESS>heroku-postbuild
       > npm run build

       ><EMAIL_ADDRESS>build
       > nuxt build

 WARN  Changing NODE_ENV from development to production, to avoid unintended behavior.
       Nuxt 2.17.2 with Nitro 2.8.0
 WARN  No .env file found in /tmp/build_334a19c4.
 WARN  Module @nuxt/content is disabled due to incompatibility issues:
 - [nuxt] Nuxt version ^3.0.0-rc.3 is required but currently using 2.17.2
 WARN  Module @nuxtjs/robots is disabled due to incompatibility issues:
 - [bridge] Nuxt bridge is required
 ERROR  Cannot read properties of undefined (reading 'filter')
  at Module.writeTypes (node_modules/@nuxt/kit/dist/index.mjs:2821:36)
  at Object.run (node_modules/nuxi/dist/chunks/build.mjs:93:15)
  at async runCommand$1 (node_modules/nuxi/dist/shared/nuxi.4fde776c.mjs:1648:16)
  at async runCommand$1 (node_modules/nuxi/dist/shared/nuxi.4fde776c.mjs:1639:11)
  at async runMain$1 (node_modules/nuxi/dist/shared/nuxi.4fde776c.mjs:1773:7)
 ERROR  Cannot read properties of undefined (reading 'filter')
-----> Build failed

       We're sorry this build is failing! You can troubleshoot common issues here:
       https://devcenter.heroku.com/articles/troubleshooting-node-deploys

       If you're stuck, please submit a ticket so we can help:
       https://help.heroku.com/

       Love,
       Heroku

 !     Push rejected, failed to compile Node.js app.
 !     Push failed

Reading the error message, it looks like you have versions of the robots and content module that are incompatible with Nuxt 2.

@Gabrielmoraesp did you eventually find the issue?

I'm facing this error now, did anyone find a workaround?


What exactly does get pushed?

I just setup capistrano-wpcli to push from local to stage.
Staging.rb
set :stage, :staging
server 'project.labs.domain.com', user: 'deploy', roles: %w{web app db}

fetch(:default_env).merge!(wp_env: :staging)

set :wpcli_remote_url, 'http://project.labs.domain.com/'
set :wpcli_local_url, 'http://project.dev/'
set :wpcli_args, 'staging'

When I push to stage I get no error messages
INFO[7386ee4b] Running /usr/bin/env wp db export - | gzip > /tmp/wpcli_database.sql.gz on localhost
INFO[7386ee4b] Finished in 0.008 seconds with exit status 0 (successful).
INFOUploading /tmp/wpcli_database.sql.gz 100.0%
INFO[4aad959f] Running /usr/bin/env gunzip < /tmp/wpcli_database.sql.gz | wp db import - on project.labs.domain.com
INFO[4aad959f] Finished in 0.153 seconds with exit status 0 (successful).
INFO[20c0aa98] Running /usr/bin/env rm /tmp/wpcli_database.sql.gz on project.labs.domain.com
INFO[20c0aa98] Finished in 0.056 seconds with exit status 0 (successful).
INFO[ba98b0a6] Running /usr/bin/env wp search-replace http://project.dev/ http://project.labs.domain.com/ staging on project.labs.domain.com
INFO[ba98b0a6] Finished in 0.785 seconds with exit status 0 (successful).
INFO[7b41a576] Running /usr/bin/env rm /tmp/wpcli_database.sql.gz on localhost
INFO[7b41a576] Finished in 0.006 seconds with exit status 0 (successful).

But if I check the staging site I cannot see any changes? The pages + settings on my local environment does not appear on the stage . What exactly should get pushed?
Could you please provide an example on how to set up the upload settings, what sort of path should be provided?

+1 @intelligence

+1 @intelligence - Same here. I feel like I'm missing a big piece of the puzzle here.

Hey Guys.
I will respond on this one shortly.
Cheers

Same here. :) Should I import the database to my PHPMyAdmin (or something alike) manually?

Okay all.
Could you try doing it without setting the http:// in front of the url?
So just
set :wpcli_remote_url, 'http://project.labs.domain.com/'
set :wpcli_local_url, 'http://project.dev/'
set :wpcli_args, 'staging'

Maybe it has to do with the search and replace operation.

And: can you provide me with the whole output of the log please?

Also try it without the trailing slash!

I faced a similar issue today where nothing was being pushed, it was due to the machine not having mysqldump installed. Technically nothing got pushed so the import didn't overwrite anything. I would suggest changing the capistrano log level to info to see more information about what it being pushed.

Just had someone else with the same issue, notice the error only being displayed as a debug message.
INFO [9ff0116e] Running /usr/bin/env wp db export - | gzip > /tmp/wpcli_database.sql.gz as Christian@localhost
DEBUG [9ff0116e] Command: ( WP_ENV=staging /usr/bin/env wp db export - | gzip > /tmp/wpcli_database.sql.gz )
DEBUG [9ff0116e] 	sh: mysqldump: command not found
INFO [9ff0116e] Finished in 0.293 seconds with exit status 0 (successful).

I'm having the same issue only everything shows as successful with log level info. Still no change to staging db.
$ bundle exec cap staging wpcli:db:push                                                                                [17:07:51]
INFO[f6a2cdda] Running /usr/bin/env wp db export - | gzip > /tmp/wpcli_database.sql.gz on localhost
INFO[f6a2cdda] Finished in 0.110 seconds with exit status 0 (successful).
/Users/christianmagill/.rvm/gems/ruby-2.3.0/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:67:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
/Users/christianmagill/.rvm/gems/ruby-2.3.0/gems/net-ssh-2.9.1/lib/net/ssh/transport/session.rb:84:in `initialize': Object#timeout is deprecated, use Timeout.timeout instead.
INFOUploading /tmp/wpcli_database.sql.gz 100.0%
INFO[7af53ac8] Running /usr/bin/env gunzip < /home/influe01/capistrano_tmp/wpcli_database.sql.gz | wp db import - on us17.siteground.us
INFO[7af53ac8] Finished in 0.334 seconds with exit status 0 (successful).
INFO[b596ffd6] Running /usr/bin/env rm /home/influe01/capistrano_tmp/wpcli_database.sql.gz on us17.siteground.us
INFO[b596ffd6] Finished in 0.141 seconds with exit status 0 (successful).
INFO[0661e08c] Running /usr/bin/env wp search-replace influencebydesign.dev staging.influencebydesign.com --skip-columns=guid on us17.siteground.us
INFO[0661e08c] Finished in 0.610 seconds with exit status 0 (successful).
INFO[ad2ea563] Running /usr/bin/env rm /tmp/wpcli_database.sql.gz on localhost
INFO[ad2ea563] Finished in 0.005 seconds with exit status 0 (successful).

See my previous comment, in that case it seemed that the missing mysqldump wasn't being picked up as an error or suggesting any issues with info level logging.
Try running /usr/bin/env wp db export - standalone and see if that throws any errors.

Mysqldump is available both locally and on the server. When I run "wp db export" everything works as intended.

Are you able to check the database on the staging server to see if the updated data is being transmitted at all? I've had issues previously with php-fpm having to be restarted before changes are being seen.
Is this something which was previously working and has now stopped or has it never worked? Have you tried with debug level logging to see if that points anything out?

Ah, it looks like gunzip is not available on the server.

My hosting company has recommended using gzip -d which is equivalent. @davestevens Any suggestions on how to best implement this change?

I'm not entirely sure. You are able to use SSHKit.config.command_map to change commands but there is an issue of arguments here too.
Maybe you could setup an alias on the staging server of alias gzip-decompress="gzip -d" and then add SSHKit.config.command_map[:gunzip] = 'gzip-decompress' to your deploy/staging.rb file.


Add wget and curl to remote_file_copy_binaries

What type of PR is this?
/kind rule-update

Any specific area of the project related to this PR?
/area rules

What this PR does / why we need it:
Added additional binaries that are capable of remote file copy. These binaries are commonly used as a prerequisite to command and control.
Which issue(s) this PR fixes:

N/A
Special notes for your reviewer:
N/A
Does this PR introduce a user-facing change?:

rule(Launch Remote File Copy Tools in Container): add additional binaries: curl and wget.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ec4n6
To complete the pull request process, please assign kaizhe after the PR has been reviewed.
You can assign the PR to them by writing /assign @kaizhe in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ec4n6
To complete the pull request process, please ask for approval from kaizhe after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hey @ec4n6
Tests are failing :point_down:
Tue Nov 16 09:46:11 2021: Runtime error: Could not load rules file /etc/falco/falco_rules.yaml: 1 errors:
Compilation error when compiling "proc.name = curl and (proc.cmdline contains (" > ") or proc.cmdline contains (" >> ") or proc.cmdline contains (" | "))": 45: syntax error, unexpected '(', expecting 'BareString', 'String', 'Number'
---
-  macro: curl_download
   condition: proc.name = curl and (proc.cmdline contains (" > ") or proc.cmdline contains (" >> ") or proc.cmdline contains (" | "))
---
. Exiting.

I guess the round brackets are not needed around strings.

LGTM label has been added.  Git tree hash: e012ad4708ea6ca230b33bcc2d575d6d6f99f1af

Closing and reopening to trigger the CI
/close

@leogr: Closed this PR.

In response to this:

Closing and reopening to trigger the CI
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

@leogr: Reopened this PR.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/milestone 0.31.0

@leogr FYI I just added 1 minor commit per @Kaizhe's suggestion here

@leogr FYI I just added 1 minor commit per @Kaizhe's suggestion here

Hey @ec4n6
The PR looks good to me, but there's still a failing test likely (it's not your fault anyway). IIRC, this issue was fixed by https://github.com/falcosecurity/falco/pull/1764. Could you rebase your PR on master? That should fix the test.
Sorry for the inconvenience and thanks again for your patience :pray:

@ec4n6: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

New changes are detected. LGTM label has been removed.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

200231c Merge branch 'add-wget-curl-to-remote-file-copy' of https://github.com/ec4n6/falco into add-wget-curl-to-remote-file-copy

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ec4n6
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

200231c Merge branch 'add-wget-curl-to-remote-file-copy' of https://github.com/ec4n6/falco into add-wget-curl-to-remote-file-copy

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@leogr Hi, it looks like my merge is missing the DCO signoff. Is this necessary, and if so, how do I add signoff to an existing merge?

/check-dco

@leogr Hi, it looks like my merge is missing the DCO signoff. Is this necessary, and if so, how do I add signoff to an existing merge?

Merge commits are not allowed, but it seems to me you already removed it. Indeed, after checking for DCO again, now it's all fine. Thank you!

LGTM label has been added.  Git tree hash: 788fefea0a448c7688afcf4b2905e05092b73dc9

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ec4n6, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Bump @Kaizhe


regex changes, fixes for abcd-hcp-pipeline + nhp-abcd-bids-pipeline

fix issues related to changes in get_fmriname function in helpers.py of DCAN pipeline BIDS-Apps (ideally get_fmriname in macaque and abcd-hcp pipeline will be updated to be consistent w/ infant pipeline going forward)
session/task regexes changed to be consistent with get_fmriname in infant pipeline
add --legacy-tasknames flag to parse input tasknames consistent with versions <= 4.0.4; may be needed for older HCP-style input e.g. "task-myTask01"(as opposed to BIDS-style "task-myTask_run-01")
use -legacy-mode flag with -cifti-parcellate to keep behavior consistent with wb_command < 1.4.0 / avoid error when concatenated timeseries has empty grayordinates
fix unfiltered motion tsv writeout
fix sorter.m error on tasks without run index
may also resolve #5  , but has not been tested on actual fMRIPrep output yet
compiled with R2017a / v9.2 MCC (same as current release)

@kathy-snider I agree with your suggestion, I will get it added! The one drawback is that it's going to take extra work to accommodate both run indices of arbitrary length and task names ending in digits; e.g. does  "ses-mySes-task-T0001.dtseries.nii" mean:
task=T, run=0001
task=T00 run=01
task=T0001, run=n/a
?
so for now I'll just add your suggested fix, and disclaimer that task names ending in digits won't be supported unless you have two-digit run indices and use the --legacy-tasknames flag.

This would only affect the BIDS version, i.e., non-legacy. So you could have task-rest_run-1 or task-rest_run-000001 and the "+" would get all of the digits whereas the {2,} only gets the digits on the second one (run-000001), but would not match the first one (run-1).

From: Thomas Madison @.***>
Sent: Wednesday, August 4, 2021 11:03 AM
To: DCAN-Labs/dcan_bold_processing
Cc: Kathy Snider; Mention
Subject: [EXTERNAL] Re: [DCAN-Labs/dcan_bold_processing] regex changes, fixes for abcd-hcp-pipeline + nhp-abcd-bids-pipeline (#15)
@kathy-sniderhttps://github.com/kathy-snider I agree with your suggestion, I will get it added! The one drawback is that it's going to take extra work to accommodate both run indices of arbitrary length and task names ending in digits; e.g. does "ses-mySes-task-T0001.dtseries.nii" mean:
task=T, run=0001
task=T00 run=01
task=T0001, run=n/a
?
so for now I'll just add your suggested fix, and disclaimer that task names ending in digits won't be supported unless you have two-digit run indices and use the --legacy-tasknames flag.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/DCAN-Labs/dcan_bold_processing/pull/15#issuecomment-892860839, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJAQP4HJM2PA5ZQE6H4KBDLT3F6GJANCNFSM5BGQUJGQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

Ahh that makes sense now that you explained it, I'm getting confused by my own code. Merging now.


test(nexus): run all pytest system tests with nexus

Fixes

Fixes WB-NNNNN
Fixes #NNNN

Description
What does the PR do?

🤖 Generated by Copilot at 5f0f13c
This pull request adds the nexus_failure label to various test modules that fail on Nexus due to feature incompatibilities or issues, and splits the pytest job into system and smoke tests with Nexus. It also fixes some minor bugs and linting errors in the Nexus backend code and the test fixtures.
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at 5f0f13c

We're trapped in a Nexus of doom
Where artifacts and files can't bloom
We mark our tests with labels of pain
To escape the wrath of the incompatible domain

/fix-title


[locale,xkbfile] improve mapping

Add missing keycodes to mapping
Use sorted list to create mapping table

@sasha0552 update the xkbfile mapping, please check if your keys are now properly mapped.

@akallabeth Hi, thank you for your work.
Unfortunately it doesn't work for 2 of 5 keys.
Updated table:

rdp code
scancode
keysym
tested/works

16
173
XF86AudioPrev
+

25
171
XF86AudioNext
+

32
121
XF86AudioMute
+

34
172
XF86AudioPlay
+

36
174
XF86AudioStop
+

46
122
XF86AudioLowerVolume
+

48
123
XF86AudioRaiseVolume
+

50
180
XF86HomePage
+

101
225
XF86Search
+

102
372
XF86Favorites
+

103
181
XF86Refresh
Unknown key with X keycode 0x08

104
136
XF86Stop
Unknown key with X keycode 0x08

105
167
XF86Forward
+

106
166
XF86Back
+

108
155
XF86Mail
+

109
234
XF86AudioMedia
+

110
156
XF86Launch1
+

111
157
XF86Launch2
+


[SILGen] Use the signature of the real UIApplicationMain

...instead of trying to guess it ourselves.
My previous attempt at this (part of the optional pointers work, bc83940) made a critical mistake because our only test case also referenced UIApplicationMain directly. I've made the test case test several more situations, and also added what would be an execution test if our simulator testing handled UI-based tests.
rdar://problem/25712303

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test
@jckarter or @slavapestov, mind reviewing?

@swift-ci Please test

I don't see anything obviously wrong here, but I don't have enough knowledge of the underlying issue.

Looks reasonable. The _stdlib_UIApplicationMain stub approach didn't pan out?

I couldn't get it to reliably be force-inlined, although this morning I realized that was probably because the "inline me" flag needs to be set at the use site, not just on the function. It also would have required an extra NSString-to-String-to-NSString conversion anyway given how UIApplicationMain normally comes in, something we'd peephole but still making extra work for the optimizer.
Crossed fingers re: future SDK changes.


test: Cypress - Automated tests for Firestore datasource

Description

This PR includes tests for Firestore ds - with querying & usage of various fields in the query page

Type of change

New script

How Has This Been Tested?

Cypress CI runs

Checklist:
QA activity:

[ ] Added Test Plan Approved label after reviewing all changes

/ok-to-test sha=ed77bc3

/ok-to-test sha=40ef7a0

/ok-to-test sha=1fc8002

/ok-to-test sha=f4f87c3


point_latitude and point_longitude should be guessed as lat / lon columns

And I think we should make the list of tokens that could be automatically interpreted a bit bigger, right?

this shouldbe gdal


new default help command

Summary
title
What is the feature request for?
The core library
The Problem
the right-now help command is not really good, its made when buttons isnt released
The Ideal Solution
design a new default help command. maybe using an embed or button/dropdown?
The Current Solution
No response
Additional Context
No response

IMO, the help command isn't meant to be pretty or anything. As people can, and already customize them to whatever they want, you might find that people may like not something like an embed too much. Remember that nextcord is not meant to be doing everything for you, therefore, if you want a nicer help command, you can do just that, but that's just my opinion.
And also, if for instance, we did something like a command in each embed field, it wouldn't work if you have more than 25 commands, as that is the embed field limit due to discord's limitations.

Given the fact, that example help commands will be an own repo, I think this issue can be closed.


A new option added to restrict search based on Country

A new option added to restrict search based on Country
It can be set by adding value to Intent through "COUNTRY_ISO_CODE" flag.
Contributors.md file added.

Country can be restricted in this way
intent.putExtra(MapUtility.COUNTRY_ISO_CODE, "AE");

Thanks for working and improving the library and giving your valuable time to it. I will soon look upon it and merge the improvements to the library


Trotterization - evaluate observables at intermediate time steps.

In the current version of Trotterization, if the user provides observables/auxiliary operators to be evaluated on a quantum state, it only happens at the end of the evolution. It was suggested by @Cryoris to also have an optional setting to evaluate them at intermediate time steps.

This issue has been fixed in the time_evolvers module.


Tick formatting depending on x range

Use case: Timeseries with zooming/unknown X range.
Want to show only dates when I have data of multiple days (but the full timestamps in tooltips), and more fine grained x values if I have data only of a few hours, or I have zoomed.
Easiest solution would be have the axis.x.tick.format function take three arguments value, minValue, maxValue, with the latter specifying the currently shown X range.

I was wondering if it's possible now out of box or should I use hack?


Remove the check api infrastructure

Turns out the check api stuff did not perform as we had hoped, remove the script and all the output files.

@clarkmoody can we have a concept ACK on this before merging?

Merge when you get a chance please @apoelstra.


api: add gRPC endpoints for creating, updating and deleting passwords

closes #639
cc @ericchiang

Changed storage package doc to have min cost value of 10 instead of 14.

All the unit tests passed:
[rjohn@e4-b3-18-d3-91-0e server]$ go test -v .
=== RUN   TestCreatePassword
--- PASS: TestCreatePassword (0.08s)
=== RUN   TestUpdatePassword
--- PASS: TestUpdatePassword (0.00s)
=== RUN   TestDeletePassword
--- PASS: TestDeletePassword (0.00s)

Unit test passed:
$ go test -v .
=== RUN   TestPassword
--- PASS: TestPassword (0.00s)

one last nit, other than that lgtm


[bug] cannot set num_features

When I try:
model = torch.hub.load('rwightman/gen-efficientnet-pytorch', 'efficientnet_b0', in_chans=1, num_features=16)
or
model = torch.hub.load('rwightman/gen-efficientnet-pytorch', 'efficientnet_b0', in_chans=1, num_features=16, pretrained=False)
I get this error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\t-maserr\AppData\Local\Continuum\anaconda3\envs\python37\lib\site-packages\torch\hub.py", line 359, in load
    model = entry(*args, **kwargs)
  File "C:\Users\t-maserr/.cache\torch\hub\rwightman_gen-efficientnet-pytorch_master\geffnet\gen_efficientnet.py", line 636, in efficientnet_b0
    'efficientnet_b0', channel_multiplier=1.0, depth_multiplier=1.0, pretrained=pretrained, **kwargs)
  File "C:\Users\t-maserr/.cache\torch\hub\rwightman_gen-efficientnet-pytorch_master\geffnet\gen_efficientnet.py", line 430, in _gen_efficientnet
    **kwargs,
TypeError: type object got multiple values for keyword argument 'num_features

@mattans Changing num_features from the top level instatiation isn't currently intended. Did you mean to change the number of classes for the classifer (num_classes)?
If you want to change num_features, I'd recommend creating new a new, or editing existing _gen_XXX() and model entrypoint function and add the necessary overrides for the num_features calculations. Note it'll currently render the pretrained weights invalid without adding addition support for ignoring the weights for the impacted convolution.


How are we going to get the 'latest post' link in the footer to work on this site?

If this remains a problem when we've worked out the final site architecture, I vote for killing the feature.


Repo sync for protected CLA branch

The pull request is created from master637652700827241972 to master to fix git push error for protected CLA branch

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


Overlapping Windows feature request

📝 Overlapping windows in the Fancy Zone Setting
At work & at home, multiple windows are used, some in foreground, some in back. At work, a minimum of 13 windows opened, at home between 5-7.
I have to manually resize & replace the windows so that they all "peek"out from behind each other but yet, are still viewable with whichever may be on top without having to click up in the taskbar. Might there be an addition to the fancy zones which could help this?
I've attached a screenshot of my desktop as an example just in case. (Sorry for the busy desktop, what can I say? I get it in 😎💪🏼)

If you'd like to see this feature implemented, add a 👍 reaction to this post.

are you looking more at "work" and "home" mode?

Preferably both 💪🏼
On Fri, Dec 4, 2020, 13:30 Clint Rutkas<EMAIL_ADDRESS>wrote:

are you looking more at "work" and "home" mode?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/8398#issuecomment-738945747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALY67Q7A5336ECV5OWBSXTTSTETFDANCNFSM4UNRIDTA
.

d

If we could hotkey FZ layouts, would that solve this?  Like you hit key combo 1 and it does one layout, key combo 2 and shifts to a second?

If we could hotkey FZ layouts, would that solve this? Like you hit key combo 1 and it does one layout, key combo 2 and shifts to a second?

 Exciting notion :D

tracked in https://github.com/microsoft/PowerToys/issues/1411


Bad render header/tags

Header 1 is a # followed by a space.
When #tag1 is rendered as a header and should not.
Very thanks for your work.

I haven't been using tags but to fix this, change the first * on line 18 of syntax/headeratx.php .
I can send a PR after #14 gets fully merged. I didn't see this before, and that PR changes the same line to fix another more data-destructive bug.


Remove unnecessary build dependency on std_msgs.

Instead, make it a test_depend, which is much more appropriate. We also make it so that we don't export the 'std_msgs' dependency to downstream packages, as only our tests depend on it.
While we were in here cleaning things up, make a couple of other minor fixes:

Remove an unnecessary #include to rcutils.
Remove a duplicated call to ament_cmake; that is already a transitive dependency via ament_cmake_ros.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>Note that this depends on https://github.com/ros2/rosidl/pull/708 , and hence is going to fail the Rpr job until that one is merged and released.
Fixes #144

Full CI, with this PR and https://github.com/ros2/rosidl/pull/708:

Linux
Linux-aarch64
Windows

CI is green, and while I could merge this one in now, I'm actually going to wait for https://github.com/ros2/rosidl/pull/708 to be approved before doing that.

All right, the rosidl PR has been merged, so I'll go ahead and merge this one as well.


SyncDictionary OnChange - key represents old value

It took me a while to understand that in OP_SET and OP_REMOVE operations, the key actually represented the old value.
The action description shows it but the documentation does not. Could be helpful.

Closing this - the doc has been updated.


Fix mypy errors

Optuna provides the type hints for its application since v2.1.0.
It causes a new mypy error as follows:
$ mypy .
xfeat/optuna_selector/_kbest_explorer.py:69: error: Incompatible types in assignment (expression has type "FrozenTrial", variable has type "None")

This PR adds type annotation to KBestThresholdExplorer.trial to avoid the error.
In addition, CupyArray in xfeat/cat_encoder/_target_encoder.py causes another mypy error, and I also fix it using the string annotation.
xfeat/cat_encoder/_target_encoder.py:31: error: Variable "xfeat.cat_encoder._target_encoder.CupyArray" is not valid as a type
xfeat/cat_encoder/_target_encoder.py:31: note: See https://mypy.readthedocs.io/en/latest/common_issues.html#variables-vs-type-aliases

Thank you for your contribution! LGTM.