No examples for xWebApplication #190

Added a sample for adding a sample web application on the Default Web Site. It's configured to use Windows Authentication and will use a copy of C:\inetpub\wwwroot as content. As a fix for #190.

Hi @remcoeissing, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@remcoeissing, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Examples/Sample_xWebApplication_NewWebApplication.ps1, line 21 at r1 (raw file):

    # Import the module that defines custom resources
    Import-DscResource -Module xWebAdministration, PSDesiredStateConfiguration

Import each resource separately.

Comments from Reviewable

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

Examples/Sample_xWebApplication_NewWebApplication.ps1, line 21 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> Import each resource separately.

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Structured logging and move to logrus

We should make our logging better and adopt structured logging.
As virtcontainers recently moved to logrus, we can just follow the trend.

Closing that one, the proxy is now using logrus.


Low (<3.0) FW versions with no browser

First of all, thanks for the guide.
As for the question:
https://github.com/Plailect/Guide/wiki/3DS-Stuck-(2.1.0-Without-Functional-Browser)
It does not appear to be clear how to proceed in the case of versions < 3.0 that already show the browser update nag screen. It says:
"If you downgraded to 2.1.0, on Old 3DS, 2DS, or New 3DS but for some reason you have a nonfunctional browser, this guide is for you."
Then right after, in step 6:
"Open the browser and go to http://www.reboot.ms/3ds/load.html?Launcher.dat"
In this case, what is the definition of "non-functional"? As in, it can't be opened? Nag screen?
Also, what is "a valid NAND backup of some kind"? As I understand, If you don't have gateway mode or a functional browser, and can't exploit MSET, under which conditions if any would you be able to dump NAND?
EDIT: I mean, besides going the 10.7.0-32U -> downgrade route.

This is for if you did Part 4 and got stuck due to screwing up somewhere.
Non-functional means exactly what it sounds like, not working. The card update will allow browser to open once more even if it partially downgraded, letting you fix your system by restoring the NAND backup you made at the beginning of Part 4.

I see... Let me check if I got things right:

If you've got a non-trivially exploitable system version (i.e. 2.2.0-0U);
AND your browser/network version is zero, as in, you've never done an online update before;
AND this is the first time you're following the guide (i.e. haven't started step 1 yet);
Then your only option is going the cubic ninja route? Or are there any procedures you can use to upgrade to a browser-enabled 9.0/9.2 from versions < 3.0 (since cart updates wont give you browser)?

I hope you don't mind me asking, but a lot of the stuff on gbatemp is outdated and conflicting information.
Thanks for your time.

Your cheapest option is to buy Ocarina of Time and a Powersaves, then updating to 10.7 (quickly before 10.8 is released), then downgrading to 9.2.


Remove acrylic sidebar from settings | Default to on

I believe that allowing the user to choose weather they want the acrylic sidebar turned on will cause confusion and therefore will increase the likelihood of false issue reports.
The acrylic sidebar (when turned on) respects the windows global setting for transparency (Windows settings app -> Personalisation -> Colours -> Transparency effects).
Even when the files app has acrylic sidebar turned on, it won't work unless transparency effects are enabled through windows settings. The two solutions to this are either to make it not respect the windows settings and allow the acrylic effects no matter what, or alternatively remove the option all together and have it respect the windows setting for transparency by default; which is exactly what I think should be the case.
Example of what currently happens:

@RandomAJL We added this as an option specific to the app, because certain users expect acrylic in the Windows Shell, but not on the sidebar pane. We'll keep this suggestion in mind, but there is a low likelihood we'll act on it considering how background acrylic is technically poor design.


website generation is local - so move metalsmith packages to install …

…section.
Without npm install @qooxdoo/Compiler is not runable

merging, thank you @hkollmann


gmpy2 installation in the travis-ci build is broken?

In my project I used literally your code snippet for building gmpy2 deps:
        # Install gmpy2 dependencies
        - mkdir -p $HOME/.local
        - wget ftp://ftp.gnu.org/gnu/mpc/mpc-1.0.2.tar.gz
        - tar xzvf mpc-1.0.2.tar.gz
        - pushd mpc-1.0.2
        - ./configure --prefix=$HOME/.local
        - make
        - make install
        - popd
        - export CPATH=$HOME/.local/include
        - export LIBRARY_PATH=$HOME/.local/lib

Unfortunately, it seems this doesn't work, see build log: line "ground types: python" clearly shows that builtin python integers are used.
Probably, you should set LD_LIBRARY_PATH=$HOME/.local/lib, this fixes problem in my case, see log.
I can prepare this trivial fix, but part of the problem is that you don't check that mpmath's uses gmpy.  At least you could print mpmath.libmp.BACKEND.

This would be welcome --- both a diagnostic print (like we do with numpy relaxed stride checking) and a fix. Thanks!


GCP Pub/sub HOW TO

Hello @berndverst @ItalyPaleAle @yaron2  ;)
I noticed you worked on the GCP pub/sub feature so i would have liked to ask you a little help please ;)
I'm using dapr with redis pubsub for a while now and i wanted to use GCP pub/sub in order to have easy interface communications between GCP service and some of my pods on k8s , so i did as follow:
Create a pubsub binding for k8s such as:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: gcp-pubsub
spec:
  type: pubsub.gcp.pubsub
  version: v1
  metadata:
  - name: type
    value: service_account
  - name: projectId
    value: super-project-gcp-dev
  - name: identityProjectId
    value: "super-project-gcp-dev"
  - name: privateKeyId
    value: "8..................................."
  - name: clientEmail
    value<EMAIL_ADDRESS>  - name: clientId
    value: 11...............................
  - name: authUri
    value: https://accounts.google.com/o/oauth2/auth
  - name: tokenUri
    value: https://oauth2.googleapis.com/token
  - name: authProviderX509CertUrl
    value: https://www.googleapis.com/oauth2/v1/certs
  - name: clientX509CertUrl
    value: "https://www.googleapis.com/robot/v1/metadata/x509/dapr-pubsub%40super-project-gcp-dev.iam.gserviceaccount.com"
  - name: privateKey
    value: "-----BEGIN PRIVATE KEY-----\nMIIEvQI........mvRfbs=\n-----END PRIVATE KEY-----\n"
  - name: disableEntityManagement
    value: "false"
  - name: enableMessageOrdering
    value: "false"
  - name: maxReconnectionAttempts
    value: 30
  - name: connectionRecoveryInSec
    value: 2

I made a little quick python app for a simple test:
@gcppubsub_client.route('/dapr/subscribe', methods=['GET'])
def subscribe():
    subscriptions = [{'pubsubname': 'gcp-pubsub', 'topic': 'gcppubsubtopic', 'route': "pubsubroute"}]
    return jsonify(subscriptions)

@gcppubsub_client.route('/pubsubroute', methods=["POST"])
def scanning():
    print("event triggered", flush=True)
    print(request.json, flush=True)
    return json.dumps({'success':True}), 200, {'ContentType':'application/json'}

The deployment file look as follow:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: gcppusub-client
  labels:
    app: gcppusub-client
spec:
  replicas: 1
  selector:
    matchLabels:
      app: gcppusub-client
  template:
    metadata:
      labels:
        app: gcppusub-client2
      annotations:
        dapr.io/enabled: "true"
        dapr.io/app-id: "gcppusub-client"
        dapr.io/app-port: "9000"
        dapr.io/http-max-request-size: "20"
    spec:
      containers:
      - name: gcppusub-client
        image: gcr.io/super-project-gcp-dev/gcppubsub-client-dev:latest
        imagePullPolicy: Always
        ports:
        - containerPort: 9000

When i deploy this app in my k8s cluster, that automatically create a GCP pubsub with topicID gcppubsubtopic and subscription ID gcppusub-client-gcppubsubtopic
To try, i went to GCP console >> Topic, chose gcppubsubtopic Topic, and clicked Public Message
I Just filled in the message Body with Hello World, hoping my app would receive it and display it, but i never manage to
receive anything, and i couldn't find a tutorial showing end to end scenario, so that's why i'm writing you hoping you could help me understand what am i missing :)
Thank you so much in advance
Dapr version 1.8.4
PS; super-project-gcp-dev is a random name for the sake of this post ;)

@trondhindenes @msfussell @daixiang0 @AaronCrawfis 🙏

Overall is looking good to me. Sounds like the component is registering properly as you can see the topic in GCP. Sounds like it might be an issue either with the subscription endpoint or the pubsubroute route.
I'd log that the subscription is complete within the subscribe() function, and check your Dapr sidecar logs to see if Dapr is receiving the message.
This guide covers how to run locally and gather logs: https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-publish-subscribe/#subscribe-to-topics

In this scenario, the pubsub name is the one i declare in the metadata > name field of the Component right ?

Yep, that should be the case.

And in the GCP console, i don't need to format the payload, a simple hello world should work ?

The issue might be that you're not publish a Cloud Event in the GCP console. Dapr pub/sub defaults to Cloud Events. Guide here: https://docs.dapr.io/developing-applications/building-blocks/pubsub/pubsub-cloudevents/
And guide on bypassing Cloud Events and using raw data here: https://docs.dapr.io/developing-applications/building-blocks/pubsub/pubsub-raw/
You can update your subscription with:
@gcppubsub_client.route('/dapr/subscribe', methods=['GET'])
def subscribe():
    subscriptions = [{'pubsubname': 'gcp-pubsub',
                      'topic': 'gcppubsubtopic',
                      'route': "pubsubroute",
                      'metadata': {
                        'rawPayload': 'true'
                      } }]
    return jsonify(subscriptions)

Hello @AaronCrawfis
Thanks a lot for checking this out with me, before reading your post, like 5 min ago, i just tried moving from dapr 1.8.4 to dapr 1.9.5 , and suddenly everything works out of the box , i publish a little json using the gcp console pub/sub and my little python app receive it now,  i should never under estimate the power of upgrades it seems :)
Nevertheless i had a question about this part of the doc:
You can use either "explicit" or "implicit" credentials to configure access to your GCP pubsub instance. If using explicit, most fields are required. Implicit relies on dapr running under a Kubernetes service account (KSA) mapped to a Google service account (GSA) which has the necessary permissions to access pubsub. In implicit mode, only the projectId attribute is needed, all other are optional.

So far im using explicit, meaning static creds, but implicit would be much better especially security wise, which dapr pod should I configure with workload identity ?? Here is all the dapr pod i have on my namespace
dapr-dashboard-6b4b6f8579-c96s7          1/1     Running   0               3h12m
dapr-operator-868c664c8f-b7n8h           1/1     Running   0               3h12m
dapr-placement-server-0                  1/1     Running   0               3h12m
dapr-sentry-847c666bcf-pp2rj             1/1     Running   0               3h12m
dapr-sidecar-injector-7969fd98c9-j2mfv   1/1     Running   0               3h17m

Thanks a lot for your help ;)

which dapr pod should I configure with workload identity

I believe this would be the Dapr sidecar that is within your application's Pod, so therefore your Deployment. But it could also be the operator service as well. @berndverst any ideas?


Update terraform + aws dependencies; build on go 1.14

I'm doing this mostly to work with terraform 0.13, so I'm not certain the aws bump is necessary. If there's a known way to use version 0.4.1 with terraform 0.13 I'll happily use it :grin:

Thanks for taking care of this. I'll merge and create a new release.

I released a new version: https://github.com/sspinc/terraform-provider-credstash/releases/tag/v0.5.0

If there's a known way to use version 0.4.1 with terraform 0.13 I'll happily use it

Go binaries are statically linked so I think there is no way to use 0.4.1 with terraform 0.13 if terraform broke the protocol that they use for communication between providers and the core.

@tmichel Belated thank you so much!


Simplified the xhDbConnectionMonitor query to work with more SQL dialects

The existing query, SELECT * FROM xh_monitor WHERE code = 'xhDbConnectionMonitor' LIMIT 1, does not work in SQL Server (Microsoft SQL).
The equivalent statement in SQL Server would be SELECT TOP(1) * FROM xh_monitor WHERE code = 'xhDbConnectionMonitor'
I propose we simplify it to the SQL statement SELECT * FROM xh_monitor WHERE code = 'xhDbConnectionMonitor', which should work in all dialects.

We could also potentially do SELECT 1 FROM xh_monitor WHERE code = 'xhDbConnectionMonitor' - if we are worried about * fetching too much data.

Not sure why we had that LIMIT 1 in there given that code has a unique constraint - thanks for fixing!

Wish we had just made this configurable via params - found it failing on a deployment, went to adjust the monitor config then remembered this is hard-coded.

I don't feel too bad about that -- I think this is more appropriate than config driven sql would have been


Compound endpoint not working

Hi!
Last week I went through the examples provided here on Github, everything worked perfectly.
Today, the Compound endpoint seems not to work anymore:
 Exception: There was an error getting the data from TheGraph{"errors": [{"message": "deployment graphprotocol/compound-v2 does not exist", "locations": [{"line": 2, "column": 17}], "path": ["borrowEvents"], "extensions": {"code": "INTERNAL_SERVER_ERROR", "exception": {"errors": [{"message": "deployment graphprotocol/compound-v2 does not exist", "locations": [], "path": ["borrowEvents"]}]}}}], "data": null}
`
I use version 0.4.0.

Hi,
Looks like the compound subgraph is not deployed anymore https://thegraph.com/hosted-service/subgraph/graphprotocol/compound-v2
Will try to find an alternative.

It is working again. Seemed to be a temporary issue.
Thanks!


Project defaults

Currently, tanka has spec.json to control it's behavior per environment. Some of that might be general enough to be shared (diffStrategy, pruning, etc).
I propose to introduce another file, project.json in the projects root, with the following format:
{
  "apiVersion": "tanka.dev/v1alpha1",
  "kind": "Project",
  "metadata": {
    "name": "<project name>"
  },
  "defaults": "<spec.json here>"
}

This file would also replace tkrc.yaml as the optional root indicator, because having a mix of yaml and json for config makes no sense

This can be fully implemented using Inline Environments, no need for support on the Go side of things

This can be fully implemented using Inline Environments, no need for support on the Go side of things


Add docker-compose-local.yml

Add docker-compose-local.yml

JIRA Ticket: LTSVIEWER-218
What does this Pull Request do?
This adds a docker-compose-local.yml file so that you can run this using our normal build process. It also sets it to use port 23024, which is the port that will be used on LTS Cloud servers.
How should this be tested?
A description of what steps someone could take to:

Pull in the latest code form LTSVIEWER-203 branch
Copy the env-example.txt to .env (You don't have to make any changes to the file)
Build the container using the following command: docker-compose -f docker-compose-local.yml up -d --build --force-recreate
Confirm that you can get to the application at: http://localhost:23024
Confirm that you can generate a PDF using curl:
curl -X POST 'http://localhost:23024/api/coverpage' \  -H 'Content-Type: application/json' \  -d '{"title":"Lee, Joseph. Hot dog in the manger. [Boston : Massachusetts Civic League, 1928?].","manifestUrl":"https://iiif.lib.harvard.edu/manifests/drs:4997399","pdiiifVersion":"0.1.0git","thumbnail":{"url":"https://ids.lib.harvard.edu/ids/iiif/4997393/full/200,/0/default.jpg"},"provider":{"label":"","logo":"https://iiif.lib.harvard.edu/static/manifests/harvard_logo.jpg"},"metadata":[["Rights/License","http://nrs.harvard.edu/urn-3:hul.ois:hlviewerterms"]]}' \ --compressed \ -o test.pdf

Test coverage
Yes/No: Are changes in this pull-request covered by:

unit tests? No
integration tests? No

Interested parties
@enriquediaz

FYI I spoke with Anthony about --no-sandbox vs. cap_add and for the short term we can stick with --no-sandbox. long term: we should refactor the cover page to not use Puppeteer. I will create a JIRA task for this effort.


Fixed #25 and #26

Refactor of async calls.
Prevented duplicate callback errors in _uploadSTK500v2()

Also switched to graceful-fs for nicer file handling :)

Fixes #25 and #26

@noopkat Any chance that you could merge this in today? This is a dependency for a multi-device project, and it's hard to deploy without this being in the actual release.
Thanks!!

absolutely! Gimme a few hours and I'll have it sorted.

Thanks! :+1:

published in 1.4.10 c4971577798df2aef


Changelog: references to commits are broken

The references to commits in the changelog are broken:
https://github.com/angular-ui/bootstrap/blob/master/CHANGELOG.md
For example, it links to:
https://github.com/angular-ui/bootstrap/blob/master/angular-ui/bootstrap/commit/992b23297cd100ab4e236fba469e3a70566a4163

instead of
https://github.com/angular-ui/bootstrap/commit/992b23297cd100ab4e236fba469e3a70566a4163

@chrisirhc Are you able to take a look at this?

Perhaps updating grunt-conventional-changelog to v1.1.0 might help instead of sticking to v0.1.2

Actually, it seems that this issue is due to me mistakenly using v1.1.0 of grunt-contentional-changelog.   v0.1.2 seems to generate the correct URLs. I haven't looked into why exactly this is happening.

Looks like it's because the new version of grunt-conventional-changelog expects a repo url in the package.json .


Change name to avoid trademark issues with Moog Music Inc.

Hello,
I want to make an openSUSE package for this program but the "moog" name might cause trademark issues and I don't want users to be annoyed by it, could you please change it to something different? Thank you :smiley_cat:

Hey there!
That's a cool idea! Would it be enough just to change the repository name? We would rename it to "RaffoSynth"

That's fine :smile_cat:

Done!


Add Loading.app Latest

https://github.com/BonzaiThePenguin/Loading

Seeing as url doesn’t point to a specific version, this cask should use version :latest and sha256 :no_check. See an example.

Whoops, should be all fixed now.

Thank you. Could you please squash your commits?

Done!


installer: Write out PostgreSQL version even on frontend hosts.

Because frontend hosts (including docker hosts) need the PostgreSQL
command-line client, they too need the postgresql version to be
defined in zulip.conf.
Testing Plan: Inspection.  Tested the regex on dpkg --get-selections by hand.

I think I'd prefer to only install postgresql-client without version specified on frontend hosts, if we can do that in a way that doesn't become a giant mess (I'm not sure how apt as invoked by puppet will handle a case where you have postgresql-11-server installed and try to install postgresql-client that depends postgresql-client-12).  If needed, I suppose we could do it conditionally: If $postgres_version is set, we install postgresql-client-{version}.
@andersk thoughts?

If you request postgres 10, and the puppet config for the frontend just specifies postresql-client, you get both the 10 and 12 clients installed, with the command-line giving you 12:
# dpkg --get-selections | grep postgresql
postgresql-10					install
postgresql-client				install
postgresql-client-10				install
postgresql-client-12				install
postgresql-client-common			install
postgresql-common				install
# psql --version
psql (PostgreSQL) 12.3 (Ubuntu 12.3-1.pgdg18.04+1)

Which feels fine to me.

Great, merged, thanks @alexmv!


hg branch command is sometimes slow, run it just once

Ehh, nope :D What was I thinking, the second command is hg status. Gotta think a bit more, going to a mercurial dir over sshfs gives me a 1-2 second delay before every prompt. Could things be cached a bit, perhaps


Unable to host match after Season 3 update

BakkesMod Version:  141
RocketPlugin Version:  0.6.5

Describe the bug
hosting server fails after using host button with correct settings
To Reproduce
Steps to reproduce the behavior:

open plugin GUI
type in hamachi ipv4 address and try to host

Expected behavior
map begins to open and becomes joinable
Screenshots
https://user-images.githubusercontent.com/82294968/114288258-e9fa6980-9a22-11eb-844d-7d0dfac32143.mp4
Additional context
the host ability seems to be disabled? worked before season 3 update with same settings
not sure how github works, but if replying to this isnt a thing, direct message me on discord, bradwest#3719

i am aware that, that isnt a proper ip address, but i didnt want to post that to a public form, the proper ip address was used while attempting to host

@Stanbroek you tagged as fixed, where can i find an update?

This issue is still open, which means that it is still an active issue and the description of the tag says:

Fix will be pushed next update

which means that this specific issue was been addressed and will be pushed with the next update.

I really do not want to bother you @Stanbroek, but do you have an idea, how long it will take to fix this?


[Tab]Add tab component in row/ resizable, and boarder

Is your feature request related to a problem? Please describe.
Pain 1 I'm always frustrated when Tab component takes up the whole row when added in the dashboard. As a user, I would like the Dashboard layout to be more flexible, and/or to be able to insert tab in Row.
Pain 2. I can not visually distinguish dashboard and Tabgroup easily as tab use white background as charts
Task: 1. The goal is for Tab not to take up the entire row, whether to make resizable(preferred) or Tab inside Row, doesn't really matter

Task 2. add ability to name the second level tab component
Task 3. add grey background/boarder/padding to sub-level tab chart

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please open a new Issue or a PR to address the problem.


Docker-compose doesn't work

Version: 0.62.3
Platform: Ubuntu 20.04
Docker Version: 20.10.19
Docker-compose Version: 1.29.1

docker-compose doesn't work out of the box.
Running the command sudo docker-compose -f dockerfiles/docker-compose.yml up the jackal service fails to start with this log message
jackal_1  | exec ./wait-for-it.sh: exec format error

I have tried to understand better the problem, more details about:

If I try to change command in docker-compose.yml to run a simple command like echo:

command:
     - echo
     - "Hi Jackal!"

I've got the same error exec format error

If I try to run directly jackal output is ok

command:
     - ./jackal

The problem seems specific of the jackal docker image. I've tried to run wait-for-it using directly the debian:stable-slim image (the one used by jackal image) and wait-for-it script works correctly
jackal:
    image: debian:stable-slim
    depends_on:
      - etcd
      - pgsql
    volumes:
      - ../scripts:/scripts
    command:
      - /scripts/wait-for-it.sh
      - pgsql:5432
      - --
      - echo
      - "Hi Jackal!"

Output
jackal_1  | wait-for-it.sh: waiting 15 seconds for pgsql:5432
jackal_1  | wait-for-it.sh: pgsql:5432 is available after 0 seconds
jackal_1  | Hi Jackal!
dockerfiles_jackal_1 exited with code 0

More tests about the problem.
I think I have isolated the problem, it is something about the --platform=$BUILDPLATFORM and GOARCH=$TARGETARCH defined into the Dockerfile
To test this I have created a Dockerfile.local without these data, so something like this
FROM golang:1.19.1-bullseye as stage

ADD . /src
WORKDIR /src

RUN CGO_ENABLED=0 GOOS=linux go build -a -tags netgo -ldflags "-s -w" "github.com/ortuman/jackal/cmd/jackal"
RUN CGO_ENABLED=0 GOOS=linux go build -a -tags netgo -ldflags "-s -w" "github.com/ortuman/jackal/cmd/jackalctl"
RUN chmod +x jackal jackalctl

FROM debian:stable-slim

RUN apt-get update && apt-get install -y ca-certificates
RUN update-ca-certificates

WORKDIR /jackal

COPY --from=stage /src/jackal /jackal
COPY --from=stage /src/jackalctl /jackal

EXPOSE 5222

ENV PATH $PATH:/jackal

CMD ["./jackal"]

and have used this Dockerfile.local directly into docker-compose, so with a jackal service definition like this
 jackal:
    build:
      context: ..
      dockerfile: dockerfiles/Dockerfile.local
    ports:
      - 5222:5222
      - 15280:15280
    environment:
      - JACKAL_LOG_STANZAS=on
    depends_on:
      - etcd
      - pgsql
    working_dir: /jackal
    volumes:
      - ../scripts/wait-for-it.sh:/jackal/wait-for-it.sh
      - ../config/docker-compose.config.yaml:/jackal/config.yaml
    command:
      - ./wait-for-it.sh
      - pgsql:5432
      - --
      - ./wait-for-it.sh
      - etcd:2379
      - --
      - ./jackal

then I have executed the command
sudo docker-compose -f dockerfiles/docker-compose.yml up --force-recreate
and wait-for-it.sh script works as expected
The output:
jackal_1  | wait-for-it.sh: waiting 15 seconds for pgsql:5432
jackal_1  | wait-for-it.sh: pgsql:5432 is available after 0 seconds
jackal_1  | wait-for-it.sh: waiting 15 seconds for etcd:2379
jackal_1  | wait-for-it.sh: etcd:2379 is available after 0 seconds
jackal_1  | level=info ts=2022-10-28T16:55:13.685507956Z caller=jackal.go:190 msg="jackal is starting...
...
...


Prevent LayoutBuilder from rebuilding more than once

Fixes https://github.com/flutter/flutter/issues/146379: introduces Element.buildScope which BuildOwner.buildScope uses to identify subtrees that need skipping (those with different BuildScopes). If Element.update calls updateChild then dirty children will still be rebuilt regardless of their build scopes.
This also introduces LayoutBuilder.applyDoubleRebuildFix migration flag which should only live for a week or less.
Caveats:
LayoutBuilder's render object calls markNeedsLayout if a descendant Element is dirty. Since markNeedsLayout also implies markNeedsPaint, the render object is going to be very repaint/relayout-happy.
Tests:
Presubmits with the migration flag set to true: https://github.com/flutter/flutter/pull/147856/checks?check_run_id=24629865893
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Reason for revert: tree is closed with errors like:
test/integration.shard/break_on_framework_exceptions_test.dart: breaks when rebuilding dirty elements throws [E]
  Expected: <45>
    Actual: <2756>

  package:matcher                                                       expect
  test\integration.shard\break_on_framework_exceptions_test.dart 56:5   main.expectException
  ===== asynchronous gap ===========================
  test\integration.shard\break_on_framework_exceptions_test.dart 604:5  main.<fn>

I suspect this pull request is the cause @LongCatIsLooong


snowflake_stage_grant READ and WRITE dependency, change privilege argument to be an array?

Provider Version
v0.56.5
Terraform Version
v1.3.1
Describe the bug
When configuring snowflake_stage_grant resources, if you want to grant the WRITE privilege, it is mandatory to first grant READ. As of the error returned:

Error: 003511 (23001): SQL compilation error: Privilege order violation for the future grants on stage. READ should be granted before/simultaneously with WRITE.

This could be solved by making one resource dependent on the other one, but in my case, I use a Terraform module which can not check which privileges are being set to generate the dependencies between resources.
Expected behaviour
As with the regular command (check examples at https://docs.snowflake.com/en/sql-reference/sql/grant-privilege#examples), I understand that multiple privileges could be assigned with a single grant resource so that it is an array of strings, or just accepts multiple privileges separated by commas.

I have the same issue. I don't see a work around for this in the Snowflake provider. Stages are very finicky from a privilege perspective anyway. It's sort of not surprising there is issues in the provider around stages. I noticed other issues that are open related to stages like #1544.
I agree with the expected behavior as well.

+1 for expected behaviour. For anyone else who ends up here, I have an ugly but workable solution for my use-case, hope it helps yours.
locals {
  stage_product = {
    for pair in setproduct(
      local.database_schema_pairs,
      var.grants_stage
    ) :
    "${pair[0].database}-${pair[0].schema}-${pair[1]}" => {
      database  = pair[0].database,
      schema    = pair[0].schema,
      privilege = pair[1]
    }
  }
}

resource "snowflake_stage_grant" "read_grant" {
  for_each      = { for key, value in local.stage_product : key => value if lower(value.privilege) == "read" }
  database_name = each.value.database
  schema_name   = each.value.schema

  privilege              = each.value.privilege
  roles                  = [data.snowflake_role.role.name]
  enable_multiple_grants = var.allow_external_grants
  on_future              = true
  with_grant_option      = false
}

resource "snowflake_stage_grant" "grant" {
  depends_on = [
    snowflake_stage_grant.read_grant
  ]
  for_each      = { for key, value in local.stage_product : key => value if lower(value.privilege) != "read" }
  database_name = each.value.database
  schema_name   = each.value.schema

  privilege              = each.value.privilege
  roles                  = [data.snowflake_role.role.name]
  enable_multiple_grants = var.allow_external_grants
  on_future              = true
  with_grant_option      = false
}

Hey 👋
Closing the issue as it's referring to the deprecated resource and we're not supporting them. Please, try to use the latest provider version with the non-deprecated resources (e.g. snowflake_grant_privileges_to_account_role). If the issue persists on the non-deprecated resource (or the feature is present in Snowflake, but not in the provider), please create a new GitHub issue. The migration guide may help during migration to the latest resources. Here's a list of the latest, non-deprecated, grant resources:
Resources
snowflake_grant_privileges_to_database_role
snowflake_grant_privileges_to_account_role
snowflake_grant_account_role
snowflake_grant_database_role
snowflake_grant_application_role (coming soon)
snowflake_grant_privileges_to_share
snowflake_grant_ownership (coming soon)
Data sources
snowflake_grants


docs: update curl to follow redirects

Currently users are being redirected when using curl to download the operator sdk.
By adding the -L flag to the curl command, it will be forced to follow them and get to the correct download

/ok-to-test


Minimize lock contention on collecting metrics

Double-checked locking was introduced in order to accelerate avoidance
of cache invalidation. Added benchmarks shows improvement in relative
numbers. At my machine it gives following results:
before
Benchmark                             Mode  Cnt   Score   Error  Units
ProcfsStatusBenchmark.collectAverage  avgt    5  48.140 ± 4.332  ns/op
after
Benchmark                             Mode  Cnt   Score   Error  Units
ProcfsStatusBenchmark.collectAverage  avgt    5  31.209 ± 0.689  ns/op
before
Benchmark                                     Mode  Cnt    Score   Error
ProcfsStatusBenchmark.collectAverageContended avgt    5  351.977 ± 9.126
                                                                   Units
                                                                   ns/op
after
Benchmark                                      Mode  Cnt   Score   Error
ProcfsStatusBenchmark.collectAverageContended  avgt    5  33.422 ± 2.751
                                                                   Units
                                                                   ns/op

Thank you very much @wyhasany!
Allthough - if I am assessing the situation right - contention should only happen if multiple threads are querying the metrics from a MeterRegistry. For push-based ones, this should not really happen. For the pull-based PrometheusMeterRegistry this should only happen if multiple scrape jobs would be accessing the actuator endpoints concurrently.

Allthough - if I am assessing the situation right - contention should only happen if multiple threads are querying the metrics from a MeterRegistry. For push-based ones, this should not really happen. For the pull-based PrometheusMeterRegistry this should only happen if multiple scrape jobs would be accessing the actuator endpoints concurrently.

@mweirauch that's correct, even without contention there is no point to enter critical section more than once per one minute. Additional CPU cycles are saved in such situation.


Add Zed Editor Support

Hi! I use Zed, please add support.

Heard!
We anticipate at some point making it possible to spin up a local version of https://promptfiddle.com using baml-cli, but backed by your local filesystem: would that work for you?


I'd like to be able to block video and streaming sites.

Hello,
I'm addicted to youtube. It's a full time job at this point. I've watched 48 hours of youtube in the past week.
I'd like to be able to block platforms like that in an attempt to get off of them.
Maybe a bulk file would be nice - yt, twitch, vimeo, tiktok etc...

Hello Ivan @ivandimitrov8080 thank you for your question.
You can manually add domains to your copy of the hosts file. It's very easy to do, simply with a text editor.
Closing.


helm secrets dec not working and deleting secrets file

I have a simple setup just using pgp
.sops.yml
---
creation_rules:
  - pgp: B6EE067A65308CA16B0D39FE27FAE19092947959

I can run enc and view just fine
$ helm secrets enc ./secrets.yml
Encrypting ./secrets.yml
Encrypted secrets.yml

When I try to decrypt I'm getting the following
$ helm secrets dec ./secrets.yml
Decrypting ./secrets.yml
sops metadata not found
Error: plugin "secrets" exited with error
$ ls secrets.yml
ls: secrets.yml: No such file or directory

$ sops -v
sops 3.2.0 (latest)

of course, I figure it out once I post this but the difftool was looking for yaml and mine was yml
secrets.yaml diff=sopsdiffer
secrets.*.yaml diff=sopsdiffer

Hi,
The files must be .yaml not .yml. You have to change the file name to *.yaml and try again.

But I feel it's a bit strange. My .gitattributes is
secrets.yml diff=sopsdiffer
secrets.*.yml diff=sopsdiffer

and if I try to view the secrets, its working fine
helm secrets view secrets.yml
# Giving proper result

whereas if I try to decrypt it, its failing with the sops metadata not found and the file is removed. But if I rename the file to .yaml, decryption is working fine.
Any reason why this behaviour for "yml" files and not "yaml" files.


🌐 Add Chinese translation for docs/zh/docs/deployment/index.md

as title

📝 Docs preview for commit 02fb134fd40aa3c9f5c7fd420c46a8fcd30235d1 at: https://6499e28679b9ff05581fc231--fastapi.netlify.app

It seems this was covered in another PR, so I'll now close this one, but thanks for the effort! :rocket: :coffee:


vocload synonyms encased in apostrophes[

These synonyms have an extra apostrophe which are giving grief to our loader at MGI.
Can you please fix? Thanks
synonym: "Langerhans' cell" EXACT []
synonym: "Langhans' cell" EXACT []
synonym: "Bm2' B lymphocyte" EXACT []
synonym: "Bm2' B-cell" EXACT []
synonym: "Fr. C'" BROAD []
synonym: "'T-shaped' Martinotti neuron" EXACT []
synonym: "'fanning-out' Martinotti neuron" EXACT []
synonym: "'CAP1'" BROAD []
synonym: "'capillary 1 cell'" BROAD []
synonym: "'gCAP'" BROAD [PMID:33057196]
synonym: "'general capillary cell'" BROAD [PMID:33057196]

Thank you for this ticket, @hdrabkin.
My proposed edits are below.
synonym: "Langerhans' cell" EXACT []
[ ] Remove apostrophe. Use of the apostrophe is not correct. I only have found it used here: https://www.gosh.nhs.uk/conditions-and-treatments/conditions-we-treat/langerhans-cell-histiocytosis/
synonym: "Langhans' cell" EXACT []
[ ] Remove synonym. Improper use of the apostrophe aside, I do not think this term refers to the cell type described in the definition.
See also https://www.thelancet.com/journals/lancet/article/PIIS0140-6736(03)14323-1/fulltext regarding the first two cells.
synonym: "Bm2' B lymphocyte" EXACT []
synonym: "Bm2' B-cell" EXACT []
[ ] Keep apostrophe. These appear to be valid uses of the apostrophe (to represent "prime").
https://link.springer.com/article/10.1007/s12016-007-8000-y
synonym: "Fr. C'" BROAD []
[ ] Keep apostrophe. Although I don’t see this form of the synonym (with apostrophe) used in the dbxref of the term, an apostrophe is sometimes used to stand for “prime”.
synonym: "'T-shaped' Martinotti neuron" EXACT []
synonym: "'fanning-out' Martinotti neuron" EXACT []
[ ] Remove apostrophes. These seem redundant and presence or absence would not impact the meaning of the term.
Thoughts @shawntanzk ?
synonym: "'CAP1'" BROAD []
synonym: "'capillary 1 cell'" BROAD []
synonym: "'gCAP'" BROAD [PMID:33057196]
synonym: "'general capillary cell'" BROAD [PMID:33057196]
[ ] Remove apostrophes. These seem redundant and presence or absence would not impact the meaning of the term.

I'm ok with having my terms apostrophes removed - added them as it was probably how the paper used it, don't think removal would affect much on our side. Thanks

Commenting late, but looks good.  @hdrabkin  - we can't promise to always keep apostrophes out of synonyms where these are valid/important.  I think MGI needs to look at fixing their loader.

@dosumis, I think it was the double quotes around the synonym that is the problem.


Better handling of splat func arguments

According to Petka, calling slice$ on arguments kills v8 optimization. He suggested to do a naive indexed for loop on arguments instead.
Currently:
f = (a, ...b, c) ->

compiles to:
f = function(a){
  var i$, b, c;
  b = 1 < (i$ = arguments.length - 1) ? slice$.call(arguments, 1, i$) : (i$ = 1, []), c = arguments[i$];
};

I did a patch (summivox/LiveScript@e6591cf) but it currently relies on #859 being merged. It can be done without #859 but I'm too lazy :wink:
This patch itself does not change any semantics -- the only change is inlining slicing of arguments. Therefore I did not add/change any tests.

Ping @gkz (Babel and TypeScript do this by default, and CoffeeScript has been annoying people with the slow progress.

@isiahmeadows : currently @vendethiel  is in charge (and yes he did comment on that issue too).
BTW: #859 has been merged, so I can actually make a pull request now.

@summivox Okay. I didn't know this.
(I don't keep up with this as much as I used to. LS 2.0 does seem pretty interesting, though.)


Archive mode does not match when pure = off

What version of rules_go are you using?
0.51.0
What version of gazelle are you using?
0.40.0
What version of Bazel are you using?
Bazel 8.0.0
Does this issue reproduce with the latest releases of all the above?
Yes
What operating system and processor architecture are you using?
Darwin/ARM
Any other potentially useful information about your toolchain?

What did you do?
When crosscompiling a go_binary with goos, goarch, and pure = "off" I'm getting a mode mismatch error.
This error happens on both our CI runner (linux/amd) and locally on a M1 macbook.
    go_binary(
        name = "target",
        embed = embed,
        goarch = "arm64",
        goos = "darwin",
        pure = "off",
    )

What did you expect to see?
The target should build without error. This worked in 0.50.1.
What did you see instead?
Error in fail: Archive mode does not match @@//:target is darwin_arm64_debug expected darwin_arm64_debug

If I print a.source.mode and go.mode from this line, I get the same struct values.
// Debugged from CI hence different arch/os/linkmode

// a.source.mode
struct(amd64 = "", arm = "", cover_format = "lcov", debug = False, gc_goopts = [], gc_linkopts = [], goarch = "amd64", goos = "linux", linkmode = "pie", msan = False, pgoprofile = None, pure = False, race = False, stamp = False, static = False, strip = True, tags = [])

// go.mode
struct(amd64 = "", arm = "", cover_format = "lcov", debug = False, gc_goopts = [], gc_linkopts = [], goarch = "amd64", goos = "linux", linkmode = "pie", msan = False, pgoprofile = None, pure = False, race = False, stamp = False, static = False, strip = True, tags = [])

Have the same issue. Trying to update rules_go from 0.50.1 -> 0.51.0 and getting a build failure:
(12:06:10) ERROR: /builds/gitlab-org/cluster-integration/gitlab-agent/cmd/kas/BUILD.bazel:16:23: in go_binary rule //cmd/kas:kas_linux_amd64_debug:
Traceback (most recent call last):
	File "/home/ci/.cache/bazel/_bazel_ci/9ca8268df963885560ea55034139bea5/external/rules_go~/go/private/rules/binary.bzl", line 145, column 46, in _go_binary_impl
		archive, executable, runfiles = go.binary(
	File "/home/ci/.cache/bazel/_bazel_ci/9ca8268df963885560ea55034139bea5/external/rules_go~/go/private/actions/binary.bzl", line 43, column 25, in emit_binary
		archive = go.archive(go, source)
	File "/home/ci/.cache/bazel/_bazel_ci/9ca8268df963885560ea55034139bea5/external/rules_go~/go/private/actions/archive.bzl", line 78, column 17, in emit_archive
		fail("Archive mode does not match {} is {} expected {}".format(a.data.label, mode_string(a.source.mode), mode_string(go.mode)))
Error in fail: Archive mode does not match @@//cmd:cmd is linux_amd64_race_stripped expected linux_amd64_race_stripped
(12:06:10) ERROR: /builds/gitlab-org/cluster-integration/gitlab-agent/cmd/kas/BUILD.bazel:16:23: Analysis of target '//cmd/kas:kas_linux_amd64_debug' failed

The target is defined like this (via a layer of macros):
    go_binary(
        name = binary_name,
        embed = binary_embed,
        goarch = "amd64",
        goos = "linux",
        race = "on",
        tags = ["manual"],
        visibility = ["//visibility:public"],
        x_defs = _x_defs,
    )

Defined in https://gitlab.com/gitlab-org/cluster-integration/gitlab-agent/-/blob/ed1aac14a057630d36a4a7145e246c01b6c3f031/cmd/kas/BUILD.bazel#L16-21
Macros are here https://gitlab.com/gitlab-org/cluster-integration/gitlab-agent/-/blob/ed1aac14a057630d36a4a7145e246c01b6c3f031/cmd/cmd.bzl

I think that this broke in https://github.com/bazel-contrib/rules_go/commit/5933f8727950a7544352105bf5ca4e01732c8f99 since struct and named providers such as GoConfigInfo are considered distinct even if they have the same fields. I don't know why our tests didn't catch this.
We could try to always use GoConfigInfo. @dzbarsky
I will also send a Bazel PR to add the provider type to the debug representation.

Sorry for the breakage, sounds like we may be missing some coverage :) I can probably take a look in the next few days unless @fmeum you get to it first?

I couldn't figure out a way to repro (if yall have an easy repro to add to rules_go, let's do it) but hopefully https://github.com/bazel-contrib/rules_go/pull/4203 should fix it


TopKV2 Op incorrectly requiring scalar input for k

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS Mojave 10.14.2
TensorFlow installed from (source or binary): binary (pip)
TensorFlow version (use command below): ('v1.12.0-rc2-3-ga6d8ffae09', '1.12.0')
Python version: 2.7.10

Describe the current behavior
I want to be able to compute the top k results for different values of k in a batched evaluation metric. (e.g. for sample 1, compute top 2, sample 2 compute top 1, etc.)
tf.nn.top_k is throwing a ValueError despite clearly using the TopKV2 op under the hood.
the following toy example illustrates the problem:
import tensorflow as tf

y_true = tf.constant([[0., 1., 1., 0], [0., 1., 0., 0.], [0., 0., 0., 1]])
y_pred = tf.constant([[.1, 0.4, 0.4, .1], [.1, .7, .1, .1], [.1, .1, .1, .7]])

(confidences, indices) = tf.nn.top_k(y_pred, tf.constant([2, 1, 2]))

I get the following error:
ValueError: This Input must be scalar but has rank 1 for 'TopKV2_20' (op: 'TopKV2') with input shapes: [3,4], [3] and with computed input tensors: input[1] = <2 1 2>
Diving into the source code, it would seem that the TopKV2 op is precisely intended for varying values of k, however, something at a higher level is preventing my code from utilizing this functionality. Additionally, there is a top_kv2 function in gen_nn_ops.py that is not exported for use.
Describe the expected behavior
That the TopKV2 op functions as specified, or that the top_kv2 python binding is exposed.

@wickstopher the tf.nn.top_k op is expecting a scalar and you are providing a tensor. Please check an implementation here and here is the TF resource.
This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!

Are you satisfied with the resolution of your issue?
Yes
No


Moon set and rise times?

First thank you for this amazing script
second: Can you add moon set and rise times?

Glad you find it useful! I'll look into adding those, thought it may take a while as I'm working on a few other things right now.

Thank you very much @janrg  . it is very useful !
I need moon rise & set times if you can, Because I didn't find any accurate script like yours

Do you know I'm waiting you !


place wlstrap cells every 8 sram bitcells

Codecov Report

Merging #32 (6f106c5) into master (fac2c02) will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
+ Coverage   60.91%   60.95%   +0.04%
==========================================
  Files          28       28
  Lines        2635     2633       -2
==========================================
  Hits         1605     1605
+ Misses       1030     1028       -2

Impacted Files
Coverage Δ

sram22/src/lib.rs
11.26% <0.00%> (+0.15%)
:arrow_up:

Continue to review full report at Codecov.

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


Not for submission: First pass at SessionPool (and PooledSession)

This is unused as yet.
This PR has no unit tests yet, and is to allow for early review of the API and any immediate problems, while I write unit tests.

Closing this PR for now, as we're going to split this up further. Not just updating it with new commits, as we want all the conversation.


Refactor Tests to use Pre-generated Data

Currently, the unit test suite relies on simulated data, which is then processed through celda(). This makes the test set annoying to run, because it takes ~30 second or so before the actual test starts.
I'm going to run celda_CG on the extdata/cmatp data for various K/L combinations for many chains, and store the parsed results in either extdata or the test directory itself, so we can use it to run the tests immediately.

Redundant with #192


[Blazor] Errors in the startup project

Error in the startup project
I have errors, but still, I can compile and run the code without any issues.
To Reproduce
Steps to reproduce the behavior:

Install ASP.NET core version, 3.1.0-preview1.19508.20
Install the last update of Visual Studio, my version 16.4.0 Preview 2.0
Create a new Blazor WebAssembly project with the latest Blazor project template in the empty solution
Open the SurveyPrompt.razor page in the Shared folder of the new application
You will see three types of errors

CS0111	Type 'SurveyPrompt' already defines a member called 'BuildRenderTree' with the same parameter types, Line 1

CS0102 The type 'SurveyPrompt' already contains a definition for 'Title' Line 15

CS0229 Ambiguity between 'SurveyPrompt.Title' and 'SurveyPrompt.Title'

Expected behavior
Open and see razor files without any issues and clear "Error List" tab of the Visual Studio
Screenshots
See the screenshot.

Here is the list of all nuget packages.

Additional context
If I open the compiled file in the background of the razor file and add a partial class keyword in the name of the class, errors will disappear.
This could be related to this feature, about partial class, https://devblogs.microsoft.com/aspnet/asp-net-core-updates-in-net-core-3-1-preview-1/
Thanks,
Ratomir

More details, I did changes on the Counter page and made a new C# class, Couter.razor.cs. Two errors disappeared, but still, I have a one about BuildRenderTree. Screenshot below:

Huh, I found what was the issue. I had a global.json file with the specified sdk version, for some reason, that file has been copied when I create a new project. Anyway, I deleted this file and everything working properly. I will close the issue.


Send e-mail to admins on internal server errors

We should be able to handle "Internal server errors" proactively.

This was probably be fixed with #666. Closing. Reopen if not.


Add performance-test for UserInterface::build with Criterion

Relates #7
Runnable by cargo bench

No problem, probably can look at the weekend or so 😊

Could this be merged as-is for a start?

@erlend-sh I will definitely merge this once we start working on benchmarks.
For now, the API is currently changing and we don't want to maintain an empty benchmark.

Oh the API has changed a lot already 🙀
I'll close this and maybe give it another try after this month or so 🐱


chore: Removed comment related codes from RTS

Description
As part of the removing comment related codes, this PR removes the unused codes from RTS.
Fixes #20637
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=c6bce29bba78d66fa0eb508327ebdb1a72f12c53

/ok-to-test sha=28e68a5f53c5e459ed6f922b3ad17477f5128419

/ok-to-test sha=7c26c4d9ddba5a2450cfb20e28355451ab627c8f

/ok-to-test sha=3480207b250391f16d50a57f3e2e35594e22371a


unexpected config reader behaviour

Behaviour of the Config class:
If it is initialized with a filename deviating from the default "supersid.cfg", then exactly this single file is read. This is an expected behaviour.
If it is initialized without a file name or with the default filename "supersid.cfg", then three files are read in this order:
1) ../Config/supersid.cfg
2) ./supersid.cfg
3) ~/supersid.cfg or similar, the users home folder
The behaviour for the default name "supersid.cfg" may be surprising for the user. It is surprising for me. And I know of one more user who had issues with two supersid.cfg files, one in the Config/ and the second in the supersid/ folder. I personally would expect if I use "supersid.cfg" as parameter and this file exists in the current folder, then this file is read and nothing else.
An example for the actual behaviour. Let's assume there are thre supersid.cfg files in all three searched locations.

First ../Config/supersid.cfg is read. It doesn't configure paper_size.
This will result in the default paper_size = A4.
Second ./supersid.cfg is read. It configures [PARAMETERS] paper_size = Legal.
This will overloaded the paper_size = Legal.
Third ~/supersid.cfg is read. It configures [PARAMETERS] paper_size = Letter.
This will overloaded the paper_size = Letter.

If one really wants to have an automatic path detection, then I'd not go for a likely filename as "supersid.cfg" to trigger this cascaded reading, but I'd either use Pythons None or a special keyword like "CASCADE".
But I doubt this complexity is really needed. I'd recommend to remove it entirely. If one wants to have the configuration file in his home folder (i.e. because there are her/his mail credentials), then she/he can also put the entire configuration parameters into the hom e folder.
Proposed behaviour:

If a configuration file name is given, then read this configuration file,
else read ../Config/supersid.cfg.
If none of both is present, stop with an error.

I like the idea of removing the "search path" behavior. I think the .cfg file location should be either in the default location, or explicitly provided on the command line with the -c option. All that other trying different locations will only cause confusion.
Also, I think the default location (../Config/supersid.cfg) should be relative to the script directory, and not to the current working directory of the shell that is invoking the command.

The the default location ../Config/supersid.cfg is relative to the script directory.
All scripts using the configuration file have a unique method to pass the configuration file:

-c/--config path/to/supersid.cfg
if omitted, the default location is used
no cascaded config file reading anymore


Stop telling people to mess up their Podfile in firebase_messaging docs

The firebase_messaging docs instruct the user to remove the use_frameworks! line from ios/Podfile.  This is no longer necessary and it breaks other flutter plugins, such as geolocator.
I wasted an hour figuring out why all of my other modules stopped compiling after installing firebase_messaging.  The cause was the change to Podfile.  How about saving other engineers' time by removing the offending instruction from the docs?

duplicate of https://github.com/flutter/flutter/issues/25172 ?

Duplicate of #25172


IntegerLookup layer performance issue when inited with vocabulary

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
binary
TensorFlow version
2.14.0
Custom code
No
OS platform and distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I ran into a performance issue when calling an IntegerLookup layer initialized with a Python list vocabulary. Please see code snippet.
Note that passing a numpy array as a vocabulary does not cause performance issues, but hangs during serialization since only the list in code is checked.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np

vocab = np.arange(0,10_000,1)
data = np.array([[10,20,30,40], [50,60,70,20_000]])

layer_1 = tf.keras.layers.IntegerLookup(vocabulary=vocab.tolist())
%timeit layer_1(data)
# 6.21 ms ± 349 µs per loop (mean ± std. dev. of 7 runs, 100 loops each) # <- that is issue

layer_2 = tf.keras.layers.IntegerLookup(vocabulary=vocab) # pass as numpy
timeit layer_2(data)
# 285 µs ± 6.04 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)

layer_3 = tf.keras.layers.IntegerLookup()
layer_3.adapt(vocab.tolist())
%timeit layer_3(data)
# 286 µs ± 1.31 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)

### Relevant log output

_No response_

Hi, @jamm1985!
Could you please have a look at this gist where I could reproduce this output ? Kindly confirm the results?
8.22 ms ± 363 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)
509 µs ± 91.1 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)
421 µs ± 7.34 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

Thank you!

Hi @sushreebarsa!
This is the same code snippet from the bug report. My timings are slightly different as it depends on system load, but I confirm that tf.keras.layers.IntegerLookup(vocabulary=vocab.tolist()) is an order of magnitude slower.

@sushreebarsa from your gist:
print(tf.__version__)
# 2.15.0

this bug also affects version 2.15.

Hi @jamm1985 ,
I have checked with tf-nightly and there seems no performance issue. Please check the attached gist and confirm.

@SuryanarayanaY
I confirm that this bug does not affect tf-nightly.

Hi @jamm1985 ,
Thanks for confirmation. Since the issue resolved in nightly can we mark this as closed? Please feel free to close the issue. Thanks!

Hi @SuryanarayanaY :
I hope this bug can be fixed in versions 2.14 and 1.15 as these are the most heavily used versions in production right now.

@jamm1985,
From the gist I can see that the issue is fixed in nightly which will be added in the upcoming release might be Tensorflow 2.17 in coming days. For adding the fix in the Tensorflow 2.14 might get the delay of the other branch cuts to be added.
I request to wait for the other Tensorflow release and check if you are facing the same issue. Please feel free to re-open if the fix is not added in upcoming releases. Thank you!


feat(tokens): Add Brreg theme

Resolves: #706

Added theme for Brreg
Fixed a mistake with compact size
Moved primary, secondary, tertiary surface and border colors into brand sets.
Removed default set.

Altinn secondary color is missing. This need to be fixed.

Added Brreg to token transformation
Added Brreg to Storybook theme/brand selector
Generated new tokens. Looks like most changes were stuff just moved around


Error: Can't resolve 'net' in node_modules/@supercharge/request-ip/dist

Can't run the package, keep getting 'module not found' error for net. Observed with 1.1.2

@dmitrizzle Hey, this package uses the Node.js net module to determine whether a given IP is a v4 or v6 address. Does this package have access to the Node.js environment? Or are you bundling this package with Webpack?

Hey @marcuspoehls I'm using it on the server side with Next.js - I've installed net and that fixed the error.

@dmitrizzle Sweet! Thank you for sharing your solution!


Plotter: multiple mesh

Changes that allow to manipulate a DpfPlotter object to enable the "multi-mesh plotting"
MeshesContainer.plot(fields_container: optional) is now possible.
Plot a meshes container
from ansys.dpf import core
from ansys.dpf.core import examples
multishells = examples.multishells_rst

multishells = examples.multishells_rst
model = core.Model(multishells)
mesh = model.metadata.meshed_region
split_mesh_op = core.Operator("split_mesh")
split_mesh_op.connect(7, mesh)
split_mesh_op.connect(13, "mat")
meshes_cont = split_mesh_op.outputs.mesh_controller()

meshes_cont.plot()

Plot a meshes container with fields container data
from ansys.dpf import core
from ansys.dpf.core import examples
multishells = examples.multishells_rst

multishells = examples.multishells_rst
model = core.Model(multishells)
mesh = model.metadata.meshed_region
split_mesh_op = core.Operator("split_mesh")
split_mesh_op.connect(7, mesh)
split_mesh_op.connect(13, "mat")
meshes_cont = split_mesh_op.outputs.mesh_controller()

disp_op = core.Operator("U")
disp_op.connect(7, meshes_cont)
ds = core.DataSources(multishells)
disp_op.connect(4, ds)
disp_fc = disp_op.outputs.fields_container()

meshes_cont.plot(disp_fc)

ISSUE
Plotting on a path
Plotting values on a path is now possible. With field, mesh, field_m and mesh_m defined, the following code can be used to plot proceed:
from ansys.dpf.core.plotter import Plotter as DpfPlotter
pl = DpfPlotter()
pl.add_field(field_m, mesh_m)
pl.add_mesh(mesh, style="surface", show_edges=True, opacity=0.3)
pl.show_figure()

This answers the following requests: dpf-post issue 36, dpf-post issue 45.

Looks good. Please wait to merge until I can review tomorrow.

@anslpa please could you add examples of usage into the doc?
one which uses the path, and an other which allows to compare two results?

@anslpa please could you add examples of usage into the doc? one which uses the path, and an other which allows to compare two results?

Hi @rlagha, that's done (under examples/03-advanced/06-multi_mesh_plotting.py).

Approved with various suggestions.

Thank you for the review !


Support mercurius 9.x

Mercurius 9 requires graphql@16, while @nestjs/graphql (which this library heavily relies) still uses graphql@15, so this upgrade is not possible.
Said that, something is moving in the official @nestjs/graphql repo, precisely this: https://github.com/nestjs/graphql/pull/1893
When the PR will be merged and released, this library won't have any reason to exist since mercurius support will be native

@Davide-Gheri https://github.com/nestjs/graphql/pull/1893 has been merged


Update Event Hubs and Service Bus validation

Event Hubs and Service Bus entities (event hubs, consumer groups, topics, subscriptions, queues) have the same validation rules. This PR fixes some issues with the validation and uses the same regex for Event Hubs and Service Bus.

Upper case letters are allowed
1 and 2 character entity names are allowed (currently Service Bus validation does not allow this)

Corresponding validation messages (and translations) may need to be updated.

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

This looks good, I am going to update the corresponding messages


Run some config server maintainers without maintanence job lock

Some maintainers only operate on local data and can be run
simultaneously across servers (ApplicationPackageMaintainer and
FileDistributionMaintainer changed to do this in this PR) without
distributed locking

ping @baldersheim


[EPM] Adds on-chain update mechanism for untrusted minimum score

Currently, we calculate and update the MinimumUntrustedScore manually by calling Call::set_untrusted_min_score. Historically, the MinimumUntrustedScore value has calculated by fetching the last 30 election scores and halving their average. This approach requires developers to keep an eye on how the election scores are evolving with time and updating the MinimumUnstrustedScore requires considerable overhead with governance proposals and manual work. In addition to that, values that are lagging behind the previous may open attack vectors for attacks.
This PR implements an on-chain mechanism that dynamically updates MinimumUntrustedScore based on a rolling average of previous N election scores.
This PR adds a new storage item (MinimumUntrustedScoreAvg) which is updated every time a new election finalises successfully. If at least MinimumUntrustedScoreUpdateInterval elections (eras) have passed since the last update, the MinimumUntrustedScore is updated with the latest MinimumUntrustedScoreAvg. The final set value of the untrusted minimum score is a fraction (MinimumUntrustedScoreMargin) of the calculated rolling average to give a safety margin for the subsequent elections. The MinimumUntrustedScoreBackstop enforces a lower bound on the minimum untrusted score set.
New Dispatchables

set_minimum_untrusted_score_backstop(maybe_score_backstop): sets the MinimumUntrustedScore; can be called by T::ForceOrigin.

New Events

NewMinimumUntrustedScore: emitted when a new minimum untrusted score is set through the on-chain update mechanism.

Task list

[x] improve test coverage
[x] companion PR
[x] update bin/node/runtime

polkadot companion: https://github.com/paritytech/polkadot/pull/7357

Closes https://github.com/paritytech/substrate/issues/11965
More info https://hackmd.io/IwJiYgMwRKer0e7khbLhUQ

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate-macos
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2960506

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate-macos
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2968466

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate-macos
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2969152

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2972617

bot rebase

I'm still worried about the possibility of black swan events making the best possible score below the minimum set here. Particularly worrying is one where most of the validator set are chilled for some reason and leaving us with a small set. This will cause the score of possible solutions to drop a lot, because any solution would have to include a lot of waiting validators. It still might be better for the chain to expand the set and include a few less backed validators, because then nominators are incentived to help the recovery by backing the now highly profitable less backed validators or waiting validators who now have a lower barrier to entry.
With the assurance that we can still lower this by governance in case such an event happens, this PR is a strict improvement on what came before.


Plotly doesn't update after re-compiling

I just tried a simple plotly example, and after changing the code and re-compiling, the output plot does not change, I guess it's related to the JavaScript code being executed somehow, will take a look.

Somehow, I cannot reproduce this, what are the steps that are you following?

@NickSeagull

Clone this repository: https://github.com/mdibaiee/haskelldo-issue-103
Build and run the code
Change one of the points in either sp or l
Run again
The plot does not change

@NickSeagull I re-tried this, it seems to work if I compile a second time, and I'm a little suspicious of the highlighting code, would mind to explain how it works? When is updateDisplays triggered and when is the code re-highlighted? :grin:
https://github.com/theam/haskell-do/blob/master/src/common/HaskellDo/Compilation/View.hs#L43

Yes, it looks like it is there, I might have introduced a bug 😅
Basically, it gets a variable from the JS side called alreadyHighlighted. This flag is set to true after each highlighting, and set to false with each compilation.
The problem is in line 46 we render the outputDisplay, if not, we don't do it, that's why it doesn't render by clicking compile in the toolbar either.
This is the main reason why the render doesn't get updated the first time 🤔

@NickSeagull I see, seems like it means "already compiled" rather than "already highlighted" really, and I guess we do need that state, but why in the JavaScript? Can't we achieve the same thing by having a key in our state? :grin:

Umm, probably, it's just that it was really-ultra-quick-rapid-hotfix that I did because a lot of people were complaining 😅

@NickSeagull :joy: I will see what I can do :grin:

Great, thanks! :D

@NickSeagull addressed in #106


test/conftest.py missing from pypi package

pynvim-0.3.1.tar.gz as available at https://pypi.org/project/pynvim/#files, doesn't match the github tarball (https://github.com/neovim/pynvim/archive/0.3.1.tar.gz) - test/conftest.py is missing.
As a result, all tests are failing (fixture 'vim' not found).
For some reason python setup.py sdist doesn't include test/conftest.py in the distribution tarball. This is the same issue as #348, which was not fixed in 0.3.1.

I've stumbled upon the same problem.


Artisan console colors have gone

Laravel Version: 5.4.21
PHP Version: 7.1
Database Driver & Version:

Description:
Colors are no longer showing up in console
Using php artisan http://i.imgur.com/xbgan5A.png
Running composer http://i.imgur.com/p77iAmx.png
I am using Git Bash on Win10
Steps To Reproduce:
Open Git Bash on Windows 10
Run php artisan on a Laravel install.

Doesn't look like an issue in the core, please ask on the forums.


Warn user before doing SCB restore

SCB restore closes all open channels, which may result in significant on-chain fees.
The current implementation of lncli create does not warn users.
We need to warn users that this is what will happen if they try to do an SCB restore, and get their affirmative consent.

Please be gentle. This is my first ever golang work.
I tested it on the commandline (lncli create --multi_file ...) and it works with both "y" and "n" on the warning response

Took 7 commits to conform to the go linter expectations. "Better late than never". I don't yet know how to replicate the travis-ci linter rules on my local system, so that I can preview the warnings, so...

would be nice if the warning would only show if there actually are channels (i.e. a warning about closing channels when none exist could also be confusing)

would be nice if the warning would only show if there actually are channels (i.e. a warning about closing channels when none exist could also be confusing)

I agree, we should suppress this warning if there are no open channels. Furthermore, we should show the number of open channels in the warning.
I am not sure if at that stage of lnd's boot process, lncli can access the current open channels. I'll have to see if that is possible. Let me dig around. I'm new to this codebase

From what I can tell, the channel arbiter is not running during the "create" phase of lnd's boot cycle and therefore I can't check open channels. Let me know if I missed something

Fixed comment formatting, addressed all your review issues, squashed and rebased!

Cleaned up comments some more (still getting used to the style conventions of Go and the lnd team). Squashed, rebased and pushed anew.

Great suggestion. Please hold while I revise it to exit the "create"
command.. As you noticed, it doesn't make sense to continue with the wallet
recovery/creation if the user has said "no"

Merging as is so this can make it into 0.8.2, will add on a commit addressing Johan's final comment.


Fix drag-select bug

Fix for issue#6 I just opened
Also since dragStart is not updated at all while using modes other than Select, I think being able to drag-select while in other mode is not intended so I change that as well in this PR.
If it's intended then just need to disable the ImGui::IsWindowFocused() condition for this.

👍


FormBuilder Inline Repeater doesn't allow for single deletions

Description

When using the Form Builder and Inline Repeaters per this guide https://twillcms.com/docs/relations/belongs-to-many.html it seems that you cannot delete a single repeater item when you have more than 1. Deleting all does work properly, and new additions work, but if you have 3 and delete one, save, and refresh it persists.
Digging in it looks like it is trying to run detach in HandleRepeaters.php on the id column of the pivot table, where it should be your related ID (for example, category_id)?
I ended up pulling the method up to my ProductRepository.php and applied a fix that maps the ID field just with a simple array, but then noticed that new additions also run the detach() call, so any new adds would instantly be deleted.
$object->{$relation}()->detach($existingRelation->pivot->id);

Adjacent to $currentIdList I had to create $currentIdList which stores (int)$newRelation['id'];, then added another conditional before the detach to check if the incoming ID was newly created. This appears to work for now but would appreciate input as it's just a quick patch and not a real fix.
Since this is my second ID mismatch bug report I figured I had set something up wrong but I have looked over it a few times and didn't see any glaring issues. Will probably try to create a new Twill repo to recreate this when I get a little free time and will link that if I do!
Steps to reproduce

Add 3 items to a repeater and save, then delete one, save, and refresh the page.
Expected result

The deleted item should be deleted.
Actual result

It remains attached.
Versions

3.0.2

10.25.2

8.1.18

mariadb:10.6

Fast follow, I noticed something interesting.
On one of my tables, the Pivot ID matches the related ID, however on 2 of my other ones this is not the case
Matching:

Mismatched:

The one that matches works without patching, but the other does not. Is the expected behavior to have the ID column match the Related ID? Curious why 2 of my tables do not function this way but one does. Any input is appreciated!


[1320] Add courses view to providers sub menu

Context

https://trello.com/c/0Yz9ZE0n/1320-s-add-course-list-to-provider-pages-on-api-ui

Changes proposed in this pull request

Add the provider's courses under a courses view

Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 [S] Add course list to provider pages on API UI


Responsivness Problem

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
The problem is all across the Github website, the right sidebar interferes with some content when resizing browser, blocks some links and buttons.
Browser: Firefox
Device: Macbook Pro 14
What changes are you suggesting?
Return everything back to normal, the responsiveness was perfect like a week ago, now it gets messed up sometimes.
Additional information
No response

@Emad-Eldin-G Thanks for opening an issue. Are the problems you are experiecing within the Docs site or the GitHub UI? We maintain the Docs site and the documentation within. For bugs or feedback related to GitHub site or products, please leave your feedback here


NTR: normal mitotic G2 DNA damage checkpoint

parent of
FYPO:0005299 | normal mitotic G2 DNA damage checkpoint during cellular response to UV
PMID:9950674

normal mitotic G2 DNA damage checkpoint FYPO:0006578
edit file: 40fba236182fa955ab21d3c0212ef93056fb31b3
release: 6a623d18fa658d9addc4479b145f8c08c6938aba


only add widget if slide actually exists

Fix #190

Coverage increased (+0.06%) to 64.05% when pulling 3af18113f2ce74bb0c4cec8f90d5ef97d65668a1 on prevent_slide_unloading_race_crash into 2fc1e9446bb2adc649a75062c7e2b9469ca001e4 on dev.

Coverage increased (+0.06%) to 64.05% when pulling 3af18113f2ce74bb0c4cec8f90d5ef97d65668a1 on prevent_slide_unloading_race_crash into 2fc1e9446bb2adc649a75062c7e2b9469ca001e4 on dev.


[BUG] some type errors in pipeline.ts

causes:

PS: I don't know why our Webpack is trying to read your TS file... that's probably an issue on our end. But the fact that the type errors are there might be able to be fixed in your source code.

I've fixed the type issue in v1.2.3. It looks like have something to do with TypeScript 4.5.
With that said, ideally, the library's type issue shouldn't be your project concern to begin with.  If you use vue-tsc to check the types, there is a "--skipLibCheck" option you can set to skip type checking on libraries.


refactored logout-service

Switch to private properties.
Use async/await consistently instead of mixing with .catch.
Use a constant for the maximum check count instead of a magic number.

Thanks for this @kevinlaiGH. Appreciate the contribution!
That said, can you please remove all the semicolons. This project somewhat follows 'standardjs' styling (though I need to get it added as part of the build/linting steps).
Also, can you remove the private functions. I'm not a fan of them in typescript/js code since JavaScript doesn't have a true concept of private functions. Also this is an open source library so I err on keeping functions public. (I personally hate when I need to do a simple monkey patch or method overload and can't because the library writer protected functions)
Otherwise, much better removing the .catch.

Also, if you are interested in cleaning up style/linting, we really need a contribution to add standardjs, but for typescript. Would love that if you are interested in contributing more! https://standardjs.com/

Thanks for the suggestions! I have changed it accordingly.
Could you review it again please?


Replace Show More buttons with carets w/ counts

Little issues:

Clicking on the minified load numbers next to the sparklines doesn't expand them
If there is no cpu/mem in a table the whole row doesn't highlight on mouseover.
With the +42 v off to the side the line becomes more prominent and feels "at-a-glance" to be more of a divider between sections, but a very unbalanced one (no margin-top and too much margin-bottom). But i've been playing w/ the diff combos. I think lhs/center/rhs all have strengths and weaknesses. I'm leaning a tiny bit more towards centered again but I you can arg for any of them really.

LGTM after 1 & 2!

If there is no cpu/mem in a table the whole row doesn't highlight on mouseover.

If there are no metrics, no td elements are generated, which the browser seem to need to make up a full row.

  ftw. I'm not that fussed. Still LGTM i think.

I concur, LGTM

The styled the show moar line to be dotted. Looks more subtle, and a bit less like a divider.


Plus sign not supported in combo keys

I want to implement "ctrl+plus" in my controller, but it doesn't work. Looks like Mousetrap is supporting it.

Try combo: 'ctrl+='
That should resolve the issue

The latest hotkeys should have the updated Mousetrap, so this issue should be solved

Mousetrap only partially supports it (on numpad, not as "normal" key), but still you are right, this issue is not an angular-hotkeys specific issue.


COBOL program wrapped in JCL

Development environment used

Z Open Editor version: v3.1.1
Editor Platform

[x] Visual Studio Code
[ ] Red Hat CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version: 1.76.1
Operating System (on which VS Code runs such as Windows 10 2004, otherwise name and version of platform such as OpenShift v4.3): Windows 10 21H2
Java Version (when using VS Code or Theia, execute java -version and paste the details here):
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version: 7.12.0
Node.js version: v18.14.2

Logs attached (see here how to get them): no

Problem Description
Detailed steps for reproducing the problem:

First step wrap cobol source program in JCL compile procedure
manually select cobol as language

Observed behavior

In my company, we have cobol programs wrapped in JCL procedure, which creates problem when you select cobol as source in VS code with IBM Z Open Editor. If you manually select cobol as source and its wrapped in JCL, it tries to see the JCL code as cobol and underlines all of it in red (obviously, but thats not the issue, we dont really care about that). After that everything seems ok until few tens of lines down, then it starts to red underline most of the code. Saying either "Misplaced constructor parsing" or "ERROR_TOKEN unexpected tokens ignored parsing". Everything seems to be colored properly tho. As for logs, theres no difference between having jcl included or not.

Expected behavior

not have red lines everywhere

HI @LukasJaks,
I understand that your request concerns the support of several language modes in the same document.
VS Code allows this, but I think your request is too specific to your context and you will have to implement your own solution: you want to manage documents that use both the JCL language mode and the COBOL language mode, but the number of combinations of language mode is infinite, and the rules for switching from one language mode to another language mode can also be context-specific.
I invite you to read an article on Medium and the VS Code documentation:

Medium: Multi-Language Support in VSCode Text Documents
VS Code reference documentation:  Syntax Highlight Guide , Embedded Programming Languages
VS Code sample extension: LSP Example for Embedded Language using Language Service

There are two mechanisms that are implemented for processing a language mode:

syntax highlighting: it is based on textmate files describing the syntax of a language.
grammar recognition: it is based on a language server.

To achieve your goal, you would have to develop a proprietary extension associated with a new "JCL + COBOL" language mode.
This extension should:

define a syntax combining the syntaxes (textmate)  of the JCL and COBOL language modes, and define the patterns that allow switching from one mode to another, (and these patterns may be specific to your context).
create a language server filtering JCL regions and COBOL regions and addressing the COBOL language server when needed, (ZOE does not implement a language server for JCL).

In your case, it may not be necessary to implement the COBOL language server, the combination of JCL and COBOL syntax highlighting may be sufficient for your needs.
Regarding the appearance of red squiggle lines, to avoid them, the COBOL language server should be disabled to keep only the COBOL syntax highlighting.
Another solution is to make them less intrusive, see:

#337

Hi @LukasJaks, while we support embedded CICS and SQL through the EXEC statement in COBOL, as @FALLAI-Denis mentioned, we do not support JCL wrapping in Z Open Editor or the JCL language with a language server. We do have a standalone JCL textmate, so JCL files should have syntax highlighting when opened, but no language functionality.
While not ideal, if you want to get rid of all red errors there is a setting to disable all reported problems in a program. To enable that, go to VS Code settings, the setting falls under Z Open Editor -> Cobol -> Disable problems. If you'd like to search for it, the setting ID is zopeneditor.cobol.disableProblems. The drawback of this is that real errors won't be reported, although I'm not sure in your case if any "real" errors were being reported anyway as the document almost certainly wasn't able to be parsed correctly with the JCL present.

Hello,
@FALLAI-Denis thank you, for us, it will probably be solved by disabling the COBOL language server. As we dont need to have the JCL syntax highlighted. or if someone will complain, we will use @sdaimwood.
I guess we can now close this issue as resolved, even though, I still wonder why it's not being parsed correctly, one would think that just the JCL part would be in red lines, and the rest would be fine.
Thank you for your help though!

@LukasJaks thanks for submitting this. We keep it in our radar, but I had not heard about this practice before. Our cobol language mode right now is strict as the language is very sensitive about every character and column a character is placed etc.


add SocketInfo Metrics on process/thread

Fixed by https://github.com/skydive-project/skydive/pull/425

It the idea to add SocketInfo metrics (like cpu load, memory usage) in flows aim to chart the cpu load from flows


*: remove future worker

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
This FutureWorker use LocalSet as an event scheduler, which will cause panic when Runnable call a sync request of grpc.  No only grpc, any method which call block_on for a Future will cause panic because of conflict with LocalSet. I do not think it is a good design that use LocalSet to implement a thread worker.
So I replace all FutureWorker with Worker to avoid the problem.
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Side effects

Breaking backward compatibility

Release note

No release note

The normal Worker can not change deadline of Delay, so I try to solve this problem with another PR: https://github.com/tikv/tikv/pull/8683

@Little-Wallace
Do you mean https://github.com/tikv/tikv/pull/8686


ARTEMIS-1797 Auto-create-address flag shouldn't block temp destination creation

When creating a temp destination and auto-create-address set to false, the
broker throws an error and refuse to create it. This doesn't conform to
normal use-case (like amqp dynamic flag) where the temp destination should
be allowed even if the auto-create-address is false.

I will merge it .. but I will rename the commit message.


DNS test cannot resolve name

Tests run: 37, Failures: 0, Errors: 1, Skipped: 2, Time elapsed: 70.75 sec <<< FAILURE! - in io.ipfs.api.APITest
dnsTest(io.ipfs.api.APITest)  Time elapsed: 0.166 sec  <<< ERROR!
java.lang.RuntimeException:
IOException contacting IPFS daemon.
Trailer: [X-Stream-Error] {"Message":"could not resolve name (recursion limit exceeded)","Code":0,"Type":"error"}

	at io.ipfs.api.APITest.dnsTest(APITest.java:623)
Caused by: java.io.IOException: Server returned HTTP response code: 500 for URL: http://<IP_ADDRESS>:5001/api/v0/dns?arg=ipfs.io
	at io.ipfs.api.APITest.dnsTest(APITest.java:623)

This is caused by infra change which broke similar test in js-ipfs-api - https://github.com/ipfs/js-ipfs-api/issues/839. TL;DR is that you need to call resolve with -r to resolve ipfs.io now.

This is fixed in master now


Server returned 5XX Server Error reply

I installed everything as expected.  This is on Catalina, latest version.
The 1st attempt came back with the error 'Server returned 5XX Server Error reply'.  The 2nd attempt I got. 'Invalid data found when processing input [FATAL ERROR] Unknown FFmpeg error'.

The first error it's nothing we can do something about.
For the second one I suggest you to try and download the most recent ffmpeg (static if possible) and try again

Hey,
Thanks for responding quickly.
I actually do have the latest ffmpeg.  I ran it a 3rd time after opening this issue and it just worked.  3rd times the charm I guess.  Weird.

Yeah...that ffmpeg "bug" it's something that we couldn't figure out since it's so random and unusual.
Don't worry, I'm (slowly cause no free time) working on v3.0 that will rely on ffmpeg only locally so no more weird bugs

Closing as solved


[Modules] Update test folder structure

Description
Closes #4182
Pipeline references

Pipeline

[

Type of Change

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update to documentation

Checklist

[ ] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (readme)
[ ] I did format my code

The Authorization namespace has diverse errors (as folder paths were not updated). For example

PolicyAssignment rg.commmon

The same is true for the PolicyInsights namespace

Fixed 🪛


Query: Supporting postgres going ahead ?

Are there plans to support postgres database going ahead.

Hi @o1lab I think you may have opened this issue in the wrong repository--this is a MySQL driver module, so this module has no plans to support anything other than MySQL.


Show dialog to enable bluetooth and location in main screen

Priority level of this issue should lifted. We get maybe not daily but every week feedback messages asking that why Ruuvi stopped working. Re-enabling location permission has so far helped all of the users that had these problems.

duplicate #272


update Contributors

We forgot to put @rmsrosa into the contributors list. He provided the vortex example: #389

Documenter gets run now on GitHub I meant for the section at the end of the readme.

Oh, ok. So maybe it just needs some time to build. I checked the example section and the vortex one was not there yet.

Now in the dev version:
https://wikunia.github.io/Javis.jl/dev/examples/
We'll tag a new release after #390 gets merged and then it will be in stable :wink:


added JK for worker with build job

added JK for worker app
currently it only has a working spec for build job. using mock steps.

this is fine


Cannot launch.

This is a brilliant work. But some small issues stops it from launching...

Annotation here raises an error.
I think you need to add from typing import Tuple at the beginning, and change the annotation to ->Tuple[float, int, int].

After remove the annotation, some errors still occur about the args. My launch script is the same as the one in readme.md, which is python taichi_ngp.py --gui --scene lego

Traceback (most recent call last):
  File "taichi_ngp.py", line 771, in <module>
    main(args)
  File "taichi_ngp.py", line 739, in main
    if args.ckpt_path:
AttributeError: 'Namespace' object has no attribute 'ckpt_path'

Hope you can fix these soon :)

Hi, @identxxy. Thanks report this. It was a mistake that I changed ckpt_path  to model_path and forgot to update line 739. Please git pull and try it again!


Group types alphabetically inside consolidated namespaces

Assigned to myself as it was actually a change from Rex - I just created the PR.

Assigned to Rex as a reviewer just so we can get this done between us.
It feels to me like we want more blank lines - and in some cases we can unwrap and improve other formatting. But I'm happy to do that afterwards.
Rex: if you approve this, I'll then merge it and send another one back to you with whitespace-only changes.


Error 0.0.7 in grafana 5

Error:
Plugin Error
HTTP Error (404) Not Found

More information please?  Where are you looking?
What do you seen in the chrome debug tools?

I had the same error after upgrade.  Opening a dashboard with a discrete panel gave a 404 error, and everything displayed EXCEPT the discrete panels.
I copied the .html files from data\plugins\natel-discrete-panel\dist\partials to data\plugins\natel-discrete-panel\dist and restarted Grafana.  That seemed to fix it.

Oh, the message in the logs was
2018-03-29T07:55:34-0400 lvl=info msg="Request Completed" logger=context userId=1 orgId=1 uname=milhouse method=GET path=/public/plugins/natel-discrete-panel/module.html status=404 remote_addr=<IP_ADDRESS> time_ms=15 size=23334 referer="http://xxx.xxx.xxx.xxx:3000/d/000000002/smartthings-power-usage?refresh=5m&orgId=1"

Weird.  I wonder if it is a caching issue?
Yes, the html file moved from:
/public/plugins/natel-discrete-panel/module.html
to
/public/plugins/natel-discrete-panel/partials/module.html
But the reference to it also did:
https://github.com/NatelEnergy/grafana-discrete-panel/blob/master/dist/module.js#L841
@ncmilhouse -- if you shift+F5 (clear cache) and delete your copied files, does it still get an error?

I deleted the 5 .html files from dist, leaving them in partials as original.  Re-started Grafana.  Re-opened my Grafana dashboard and it still worked correctly.  Shift-F5 to clear cache.  Still worked correctly.
I believe you are correct in that it's just a caching issue.  Fix for anyone else hitting this error is simply Shift+F5.

I'll close this since the only real solution is in grafana.  https://github.com/grafana/grafana/issues/11446


Allow multiple instances of the Compositor

As a user
I want to be able to control multiple instances of the compositor independently.
So that it's easier to iteratively enhance my composition.
acceptance:

instance variables are indexed by node_id and leak no internals in the global scope

the branch has now the capability, the safe area seems to be breaking the controls, or it can be the newly introduced zoom hiding disable


Please release new version on Maven Central Repository

Version 4.0.0 has an outdated database.

I was mistaken.


Implement non-retriable errors

ManageConnection::should_retry can now be implemented to prevent bb8
from attempting to retry connection attempts. This serves two purposes:

The caller of pool.get will now receive the underlying exception if
a "catastrophic" error is hit. Previously, the caller would get a
TimedOut with no cause.

The error can be surfaced quickly, rather than going through several
connection attempts.

This is particularly useful for interactive use of bb8, such as allowing
a user to directly input a connection to a program such a CLI. If the
connection is incorrect, it is desirable to fail fast with a useful
error message.

The tests will fail in the postgres/redis libraries because I haven't propagated the changes to make Error implement Clone. I thought we should discuss that first.
It seems like there are two paths forward:

If you get a catastrophic error, the Sink does not get a copy of the actual error. Instead, it could get a CatastrophicError or something else. This seems reasonable to me.
Make Error: Clone as I've shown. This is not backwards compatible and is a fairly annoying bound to have.

Yes, I think your understanding of (1) is correct, and I do think it might solve your specific problem. I had forgotten about this exact feature, or I would have mentioned it sooner as potentially helpful to your problem. The initial spawning of connections can be executed asynchronously while returning the first error directly. Note, to make that work you'll need to configure min_idle to something other than 0 (the default).

Given the lack of follow-up, going to close this for now. Feel free to reopen if you think it's still relevant.

My bad for losing track of this. It didn't solve my problem actually. I landed up just making an initial health check call myself, outside of bb8. I passed the client into bb8 so the initial (http) connection goes into the pool.


Error: Type parameter (T) of Optype (Where) bound to different types (tensor(float) and tensor(double) in node (/model/decoder/Where).

Star RTDETR
请先在RTDETR主页点击star以支持本项目
Star RTDETR to help more people discover this project.

Describe the bug
Trying to run inference on the onnx file gives an error:
Fail: [ONNXRuntimeError] : 1 : FAIL : Load model from /content/RT-DETR/rtdetr_pytorch/model.onnx failed:Type Error: Type parameter (T) of Optype (Where) bound to different types (tensor(float) and tensor(double) in node (/model/decoder/Where).
To Reproduce
https://colab.research.google.com/drive/1Ievc9FkjJALoWBjOo4Q6f0wu5buKIZiy?usp=sharing

I tried it in a different colab and the error disappeared. So it might be an issue related to the environment?
https://colab.research.google.com/drive/1Gh2YH_ojOoLOI6ExGupoeR8fRaNoHiBW?usp=sharing
I discovered however that Yolov5 runs approx 3 times (!) faster (via onnxrunner - cpu). I'll try it with scaling and reducing the number of decoders. Do you think that will have that much impact?

Do you use latest code?  please check these two lines that using where op

https://github.com/lyuwenyu/RT-DETR/blob/main/rtdetr_pytorch/src/zoo/rtdetr/rtdetr_decoder.py#L484
https://github.com/lyuwenyu/RT-DETR/blob/main/rtdetr_pytorch/src/zoo/rtdetr/rtdetr_decoder.py#L466

Thanks for your reply!
Yes, I'm using the latest code

I have fixed it by replacing line 466
from
anchors = torch.where(valid_mask, anchors, torch.inf)
to
anchors = torch.where(valid_mask, anchors, torch.tensor(float('inf'), dtype=torch.float32, device=device))

About the speed issue:
Yolov5 runs in about 54ms on my machine (CPU)
RTDETR scaled 18 with 2 decoders runs in about 121 on my machine (CPU)
I'm using these configs:
PResNet:
depth: 18
freeze_at: -1
freeze_norm: False
pretrained: True
HybridEncoder:
in_channels: [128, 256, 512]
hidden_dim: 256
expansion: 0.5
depth_mult: 0.33
RTDETRTransformer:
eval_idx: 2
num_decoder_layers: 3
num_denoising: 100

We test end-to-end speed on GPU with tensorrt-fp16.

@lyuwenyu Thanks for your answers.
Just one final check. It is correct that the exported onnx file is about 80MB?