[Next 9] Internal server error when api request ends with /

Internal server error when api url request ends with /
Describe the bug
When you add / at the end of url, you get a internal server error - it cant find index page even when index page exists.
BTW I have only tested this with api routes, not the normal ones.
To Reproduce

Go to https://github.com/sadmone/next-issue and clone the repo
Install all dependencies with npm install
Start the server with npm run dev
Go to localhost:3000/api/post - nothing happens
Change localhost:3000/api/post to localhost:3000/api/post/
See internal server error

Expected behavior
It should find the index page, even when url ends with /, not return a internal server error
System information

OS: Windows
Browser: chrome
Version of Next.js: 9.0.0

Hi, API routes support have been updated in the latest canary you can try it with yarn add next@canary or wait for v9.0.1. Also, routes are currently expected to return a 404 when they contain a trailing slash. This behavior is being discussed.

The internal server error is not gone but reappears after it exhausts all of the memory of my instance


checkcp: can't stat '/config/home-assistant_v2.db-shm': No such file or directory

When I try to run the Check Home Assistant configuration addon, to test before updating to Core v0.111.0 it fails, with the following output:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] udev.sh: executing...
starting version 3.2.9
[13:41:54] INFO: Update udev information
[cont-init.d] udev.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[13:41:54] INFO: Setup udev devices

[13:41:55] INFO: Don't worry, this temporary installation is not overwriting your current one.
[13:41:55] INFO: Installing Home Assistant: latest...
[13:41:55] INFO: Please be patient, this might take a few minutes...
[13:42:50] INFO: Installed Home Assistant 0.111.0
[13:42:50] INFO: Making a copy of your configuration for checking...
cp: can't stat '/config/home-assistant_v2.db-shm': No such file or directory
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

Not sure what happened the last time.. but I re-ran it today, and it worked. Proceeded with the update to 0.111.0.


Incorrect .spectral.yml

Problem description
The current .spectral.yml in the repository is not a YAML file, rather it is the HTML for the Github page that represents the YAML file (mistake when downloading)
Expected behavior
Replace with the actual .spectral.yml
Alternative solution

Additional context

Apologies @javierlozallu , I had made the same error with all the linter downloads. Fixed in #13


[Arc] Failed to Fetch Data Error

Extension
https://www.raycast.com/the-browser-company/arc
Description
The "Search Tabs" feature just totally stopped working.
Steps To Reproduce
Search tabs, you'll get an error:

Current Behaviour
It just errors out
Expected Behaviour
It used to actually give the tab results

Looks like you have a tab with an unusual character open. Could you check if any of the tabs has a special title and/or url? That would help with debugging.

Hmm, super strange! I don't see any tabs open with odd characters. Even
closing out of Arc completely and reopening it, the same issue happens.
If you're talking bookmarks (pinned/favorites tabs) then oh gosh...
The only characters I use a lot are:
—
–
And maybe some:
→
Any chance any of those could be problematic?
On Mon, Mar 6, 2023, 2:34 AM Thomas Paul Mann @.***>
wrote:

Looks like you have a tab with an unusual character open. Could you check
if any of the tabs has a special title and/or url? That would help with
debugging.
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/issues/5165#issuecomment-1455703587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKAHSUPF2TW5MYPETCHEDTW2WOR7ANCNFSM6AAAAAAVQOUOHI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah that could be it... they might need some additional decoding. I wonder what's the best way to debug this is. Maybe we can add some way to log the tab titles in the error message?

It should stay open, it's still broken 😢
--
Alex Bass
Founder & Head of Product
Efficient App ( https://efficient.app/?ref=email-signature ) | Efficient VC ( https://efficientvc.com/?ref=email-signature )
Connect with me:
read.cv ( https://alexbass.xyz/?ref=email-signature ) | LinkedIn ( https://linkedin.com/in/alexhbass ) | Twitter ( https://twitter.com/alexhbass ) | YouTube ( https://efficient.link/youtube ) | Subscribe to my Journey ( https://digest.efficientvc.com/?ref=email-signature )
Looking for Inspo? Check out other CRM Integrations ( https://efficient.app/app-category/crm?ref=email-signature ) that Efficient App customers are using.
Know a company that can use our help? Send them a Free Efficiency Audit ( https://efficient.app/audit?ref=email-signature ) 🚀
Typed joyfully with Canary ( https://efficient.link/canary )
Sent via Superhuman ( https://efficient.link/r/superhuman )
On Wed, Apr 26, 2023 at 6:03 PM, raycastbot < @.*** > wrote:

This issue has been automatically marked as stale because it did not have
any recent activity.
It will be closed if no further activity occurs in the next 10 days to
keep our backlog clean 😊
—
Reply to this email directly, view it on GitHub (
https://github.com/raycast/extensions/issues/5165#issuecomment-1524144512
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABKAHSQJ4NXEAHZCJ63LWBLXDGSUVANCNFSM6AAAAAAVQOUOHI
).
You are receiving this because you authored the thread. Message ID: <raycast/extensions/issues/5165/1524144512
@ github. com>

Bummer, it's still not fixed :/
--
Alex Bass
Founder & Head of Product
Efficient App ( https://efficient.app/?ref=email-signature ) | Efficient VC ( https://efficientvc.com/?ref=email-signature )
Connect with me:
read.cv ( https://alexbass.xyz/?ref=email-signature ) | LinkedIn ( https://linkedin.com/in/alexhbass ) | Twitter ( https://twitter.com/alexhbass ) | YouTube ( https://efficient.link/youtube ) | Subscribe to my Journey ( https://digest.efficientvc.com/?ref=email-signature )
Looking for Inspo? Check out other CRM Integrations ( https://efficient.app/app-category/crm?ref=email-signature ) that Efficient App customers are using.
Know a company that can use our help? Send them a Free Efficiency Audit ( https://efficient.app/audit?ref=email-signature ) 🚀
Typed joyfully with Canary ( https://efficient.link/canary )
Sent via Superhuman ( https://efficient.link/r/superhuman ) (get 2 free months)
On Thu, Jun 15 2023 at 7:13 PM, raycastbot < @.*** > wrote:

This issue has been automatically marked as stale because it did not have
any recent activity.
It will be closed if no further activity occurs in the next 10 days to
keep our backlog clean 😊
—
Reply to this email directly, view it on GitHub (
https://github.com/raycast/extensions/issues/5165#issuecomment-1593873524
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABKAHSTSSVGQZAWCT5Q3QC3XLOQJRANCNFSM6AAAAAAVQOUOHI
).
You are receiving this because you authored the thread. Message ID: <raycast/extensions/issues/5165/1593873524
@ github. com>

Still not fixed 🫣
--
Alex Bass
Founder & Head of Product
Efficient App ( https://efficient.app/?ref=email-signature ) | Efficient VC ( https://efficientvc.com/?ref=email-signature )
Connect with me:
read.cv ( https://alexbass.xyz/?ref=email-signature ) | LinkedIn ( https://linkedin.com/in/alexhbass ) | Twitter ( https://twitter.com/alexhbass ) | YouTube ( https://efficient.link/youtube ) | VC Journey ( https://efficientvc.com/?ref=email-signature )
Looking for software inspiration? Check out our new recommended software stacks ( https://efficient.app/stacks?ref=email-signature ) and top picks ( https://efficient.app/top-picks?ref=email-signature ) !
Know a company that can use our help? Send them a Free Efficiency Audit ( https://efficient.app/audit?ref=email-signature ) 🚀
Typed joyfully with Canary ( https://efficient.link/canary )
Sent via Superhuman ( https://efficient.link/r/superhuman ) (get 1 free month)
On Mon, Aug 14 2023 at 11:04 AM, raycastbot < @.*** > wrote:

This issue has been automatically marked as stale because it did not have
any recent activity.
It will be closed if no further activity occurs in the next 10 days to
keep our backlog clean 😊
—
Reply to this email directly, view it on GitHub (
https://github.com/raycast/extensions/issues/5165#issuecomment-1677619737
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABKAHSQSWZHAY3DCWS745YDXVJD7FANCNFSM6AAAAAAVQOUOHI
).
You are receiving this because you authored the thread. Message ID: <raycast/extensions/issues/5165/1677619737
@ github. com>

Get same issue many times!

Hi @thomaspaulmann , I find an testing URL that it failed to parse its title. Anthropic \ Home
title: Anthropic \ Home
url: https://www.anthropic.com/

+1, seeing similar issue with sites with \ in the title


Issue#53

Fix for when the link is not in the DOM.

@Jacse

One could arguably also hide the info box if the link is not present.


Govee Scene ?

Hello,
Is it possible to recover the scenes of the govee application in Home assistant?
We can change the colors, but I would have liked to be able to put scenes like lightning etc...
is this possible?
Thanks...

Not via the LAN API, but it is technically possible via BLE, provided that someone has sniffed the BLE packets sent by the app for the device and we've figured out what the packets mean.
I have some uncommitted work in progress for this for some of my devices.
It is compatible with hass bluetooth proxies.
See also: https://github.com/egold555/Govee-Reverse-Engineering

Thank you for the info. And what a pity...
I looked at the list but none are there. Here are mine:

I think this must be possible in some way. Currently it is possible to trigger scenes via Alexa. In the Govee-App you can define "Tap-to-run" Commands. Those will become available in the Alexa App as a scene. At this point you can trigger those scene

There isn't a documented or supported way to do this via the LAN control API.
The author of https://github.com/bwp91/homebridge-govee has invested time in reverse engineering the undocumented and unsupported commands used by the Alexa integration, but they are high effort to maintain and I don't personally have the need for that, and certainly don't have the time for that here.

I think this must be possible in some way. Currently it is possible to trigger scenes via Alexa. In the Govee-App you can define "Tap-to-run" Commands. Those will become available in the Alexa App as a scene. At this point you can trigger those scene

Does this mean in home assistant we can just setup a switch that will silently fire the command off to Alexa and trigger the scene?

Yes. This is what i am doing at the moment. Unfortunately there is some delay, because you are telling homeassistant to tell alexa to trigger a scene and alexa is telling the govee application to set the scene. I think you got the point. This process takes some time, but is the only way to set a scene via HA.

The scene codes in fact do work via the Govee LAN API (just don't seem to be documented)

+1  THis would be great!

I assume making the calls to Govee is the easy part, but getting this to show up on Home Assistant in a way that's usable is the trickier part?
Really good to see this in their HTTP API, finally. Hopefully that means it'll show up in their LAN API at some point in the near future, too.

I've been working on the successor to govee-lan-hass which is govee2mqtt, a service that bridges your govee devices into home assistant via MQTT.  It also happens to support the new scene API.
Learn more about it here: https://github.com/wez/govee2mqtt#govee-to-mqtt-bridge-for-home-assistant

very good news.
When could we test this?

If you are running HAOS or Supervised HASS, then follow the steps here: https://github.com/wez/govee-lan-hass/issues/50#issuecomment-1876104679
If you have a machine on your network that runs docker, then: https://github.com/wez/govee2mqtt/blob/main/DOCKER.md

I'm going to close this in favor of my new Govee2MQTT AddOn, which supports this feature.


json help option undocumented?

I'm seeing a command -line option for the mkdocs command called "json". I don't see it documented anywhere. Is it unimplemented? Undocumented?

Well, any help would be useful ... this output isn't very useful:
$ mkdocs help
mkdocs [help|new|build|serve|gh-deploy|json] {options}

Yeah, so, it really isn't useful, I agree. This comes under #259, we need to rework the command line to add help output etc.
The JSON command works and is implemented. Rather than outputting html files in the build directory it creates .json files containing the contents of the page. It's a slightly odd addition, and may change before we hit 1.0 but it was added for ReadTheDocs (so they can more easily index MkDocs projects in their search backend).

Thanks @d0ugal. Actually, this is a feature I'm excited about--the ability to output json, to (for example) create endpoints for devs to (contextually) link to help content. Hope it stays in and gets some attention.

@thinkhuman I don't understand your use case, can you explain it further? What attention do you think it needs?
The only use case we have is for search and since we are adding search to MkDocs (slowly in #222) we would have two JSON formats. The plan/idea was to combine these and potentially drop the json command.


Replace QPDF by PDFio as PDF manipulation library in libcupsfilters

As the libcupsfilters library is principally written in regular C and not C++ due to the issues which would occur due to use of C++. But libcupsfilters still depend on one library which is written in C++, QPDF, a library for manipulation of PDF files. We need to replace this dependency on C++, with another library PDFio. This is a PDF handling and manipulation library, as QPDF is, but it is fully written in standard, regular C, not in C++.

There is still one single C++ source file: cupsfilters/pdftoraster.cxx It is not from a QPDF dependency, it is for the use of Poppler (from the C++ library libpoppler). It can be skipped via the ./configure call with appropriate command line parameters and so we should be able to get a pure C libcupsfilters now. We will still need pdftoraster as there are some printers with buggy PostScript interpreters which do not work with Ghostscript's PostScript output (ps2write output device). A way to eliminate C++ here would be to call a command line tool of Poppler which does the PostScript rendering.
Another point is the naming of the files you have created. Could you all rename them removing the C-
in the beginning? We do not need it, as they have the .c extension and not .cxx any more.

By the way, your work has passed the included build and CI tests.
Could you add further test cases to the file cupsfilters/test-filter-cases.txt to exercise your changes? Note that if you need extra input files for that you need to add them to the cupsfilters source repository, and that they need to be under a free software license.

Sure, letme go through the poppler Library as I haven't studied anything about it. Then I would port pdftorastor.cxx into C as well. I will remove the c- suffix as well.
I will also start with test cases as well

Your last commit just adds the new files but named without C-. You also need to remove the C- files (actually you should git mv ... from the old name to the new name to preserve history) and you also need to update Makefile.am to use the new file names without C-.


[INVESTIGATE] Cannot read property 'includes' of undefined

Description:
The Android version (Beta 3) not able to read the channels properly (#random and #general channel). It throws Cannot read property 'includes' of undefined.
Server Setup Information:
Rocket.Chat Server: 0.63.0
Operating System: Linux
Mobile device model and OS version:

Nexus 5 - Android 6.0.1

Steps to reproduce:
I'm not able to reproduce the steps since the exception seems to be related to the content of the respective channels. The same works with open.rocket.chat server.

Reported again from another user:

I'm having the same issue, I get this error when I connect to any public channel on our self hosted rocket chat server. Interestingly private channels seem to work just fine. I'm willing to help debug or test if needed.

@JackNWeems can you confirm the server version and the app version please.

Server version 0.63.1 App version 2.0.1 Build#2001

Guys, we just discovered that this is a bug in our back-end and this is going to be addressed very soon! Thank you all for the feedback!


Make network connect idempotent

This changes the network connect command to be more idempotent by silently ignoring if the container was already connected to the network in stead of producing an error.
closes #17371

@thaJeztah @mavenugo where we here?

I'll close this one for now, but will see if I can play with the suggestions @mavenugo gave


Develop/game feature

This MR introduces basic evaluation logic associated with rps rules.

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


⚠ Bofrost.es has degraded performance

In a56b8d1, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 2273 ms

Resolved: Bofrost.es performance has improved in 5e3b6bd after 7 minutes.


Implement auto publishing to PyPI

[ ] #119
[ ] Deploy to Test PyPI on all pushes to master
[ ] Mark as pre-release when appropriate. Beta version for Test PyPI
[ ] Validate version match between release tag and version in code

Refs

https://packaging.python.org/en/latest/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/
https://github.com/marketplace/actions/pypi-publish
https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions

Publishing to test pypi:

      - name: Publish distribution to PyPI
        uses: pypa/gh-action-pypi-publish@release/v1
        with:
          repository-url: https://test.pypi.org/legacy/


App Service with docker container

Actually, app service support docker container (single container, docker-compose and k8s).
But I cannot make it works with Pulumi. Is is possible to support it?
I can see terraform manage it.

@spike008t  I've been able to do this by setting the DOCKER_CUSTOM_IMAGE_NAME application setting on the azure.appservice.AppService. If it's behind a custom registry then you also need DOCKER_REGISTRY_SERVER_URL, DOCKER_REGISTRY_SERVER_USERNAME, DOCKER_REGISTRY_SERVER_PASSWORD.
For multi-container WEBSITES_WEB_CONTAINER_NAME controls which container is exposed.

@johlrich thanks for your response, I'll give a try and give a feedback

@johlrich Thanks for jumping in! This seems to be a common ask so I'd love to make a new example for this scenario. Do you happen to have some code that I could take as a base? That would be awesome :)

@mikhailshilkov  Those settings get added to the appSettings, so something like:
const dockerAppService = new azure.appservice.AppService(name, {
  location: appservicePlan.location,
  resourceGroupName: appservicePlan.resourceGroupName,
  appServicePlanId: appservicePlan.id,
  httpsOnly: true,
  siteConfig: {
    alwaysOn: true,
  },
  appSettings: {
    APPINSIGHTS_INSTRUMENTATIONKEY: appinsights.instrumentationKey,
    DOCKER_CUSTOM_IMAGE_NAME: registryArgs.DOCKER_CUSTOM_IMAGE_NAME,
    DOCKER_REGISTRY_SERVER_URL: registryArgs.DOCKER_REGISTRY_SERVER_URL,
    DOCKER_REGISTRY_SERVER_USERNAME: registryArgs.DOCKER_REGISTRY_SERVER_USERNAME,
    DOCKER_REGISTRY_SERVER_PASSWORD: registryArgs.DOCKER_REGISTRY_SERVER_PASSWORD,
    ...appSettings
  },
  tags,
})

This bookmark helped me figured this stuff out: https://docs.microsoft.com/en-us/azure/app-service/containers/app-service-linux-faq#custom-containers It also describes other app settings to control ports, change startup timeouts, attach storage.
This is optional, but if you make your own containers and want to support SSH via the azure portal and kudu then there some container config needed too. It looks like they added a good doc for this now: https://docs.microsoft.com/en-us/azure/app-service/containers/configure-custom-container#enable-ssh
One bump I hit when figuring this out: I originally tried adding these settings to an existing app service that was deployed via zip and I couldn't get things to work until I deleted and redeployed the app. Not sure if it's a reproducible issue, an unsupported scenario or just something I was doing wrong when figuring this stuff out. I haven't had any issues when using these settings across dozens of fresh deployments though.

I want to add some comments about that on what I've seen at the moment when create a new stack.

When creating the linux AppPlan, if I do not set preserve: true on the linux App Plan, all the AppServices belong to this app plan do not have containers settings on the web portal.
On AppService,

I have to add on siteConfig the field linuxFxVersion: 'DOCKER|<registry>.azurecr.io/image'
Also need to add on appSettings WEBSITES_ENABLE_APP_SERVICE_STORAGE: 'false'

Meet theses conditions, I can see the container settings on the azure web portal but the app services is not working... I try to figure out why.
I do have something like this:

const appPlan = new azure.appservice.Plan(appPlanName, {
    location: resourceGroup.location,
    resourceGroupName: resourceGroup.name,
    kind: `Linux`,
    sku: {
        size: appPlanSize,
        tier: appPlanTier,
    },
    tags,
    reserved: true,
});

const appService = new azure.appservice.AppService(appServiceName, {
    appServicePlanId: appPlan.id,
    location: appPlan.location,
    resourceGroupName: appPlan.resourceGroupName,
    appSettings: {
      DOCKER_REGISTRY_SERVER_PASSWORD: ``,
      DOCKER_REGISTRY_SERVER_URL: `https://${registry}.azurecr.io`,
      DOCKER_REGISTRY_SERVER_USERNAME: registry,
      WEBSITES_ENABLE_APP_SERVICE_STORAGE: 'false',
      DOCKER_CUSTOM_IMAGE_NAME: `${registry}.azurecr.io/${dockerImage}`,
    },
    siteConfig: {
        alwaysOn: true,
        linuxFxVersion: `DOCKER|${registry}.azurecr.io/${dockerImage}`,
    },
    httpsOnly: true,
    tags,
});

@spike008t Good call on the plan, I forgot to mention that I also have kind: Linux / reserved: true
Best way I found to troubleshoot startup issues was via the appservice's portal blade by enabling logging in the Diagnostics log tab and then look at the Log stream tab. You should see logs for for it pulling the image, the docker run call and polling for startup before your applications logs appear.

Oh in fact thats works.... The issue was by my side, I didn't configure correctly my container... All seems ok for me

I also struggled with getting a Linux Docker container to run on App Service. My goal was to write a Pulumi program that ran a custom container by following this guide. Using the CLI or console UI as a guide was not helpful -- each one names parameters differently and does not align with the actual REST API calls. After many hours of trial and error, here's the working code I came up with. Note that it uploads an image to ACR.
import * as pulumi from "@pulumi/pulumi";
import * as azure from "@pulumi/azure";
import * as docker from "@pulumi/docker";

const stackName = pulumi.getStack();

const resourceGroup = new azure.core.ResourceGroup("resource-group", {
    name: stackName,
    location: "West US 2"
});

const registry = new azure.containerservice.Registry("registry", {
    name: stackName.replace(/-/g, ""),
    resourceGroupName: resourceGroup.name,
    sku: "Basic",
    adminEnabled: true
});

const myImage = new docker.Image("image", {
    imageName: pulumi.interpolate`${registry.loginServer}/${stackName}`,
    build: {
        context: "../my-app"
    },
    registry: {
        server: registry.loginServer,
        username: registry.adminUsername,
        password: registry.adminPassword
    }
});

const appPlan = new azure.appservice.Plan("app-plan", {
    resourceGroupName: resourceGroup.name,
    kind: "Linux",
    name: stackName,
    sku: {
        size: "S1",
        tier: "Standard",
        capacity: 2
    },
    reserved: true
});

const myApp = new azure.appservice.AppService("app-service", {
    name: stackName,
    resourceGroupName: resourceGroup.name,
    appServicePlanId: appPlan.id,
    appSettings: {
        WEBSITES_ENABLE_APP_SERVICE_STORAGE: false,
        WEBSITES_PORT: 8080,
        DOCKER_REGISTRY_SERVER_URL: registry.loginServer,
        DOCKER_REGISTRY_SERVER_USERNAME: registry.adminUsername,
        DOCKER_REGISTRY_SERVER_PASSWORD: registry.adminPassword
    },
    siteConfig: {
        linuxFxVersion: pulumi.interpolate`DOCKER|${myImage.imageName}`,
        alwaysOn: true,
        cors: {
            allowedOrigins: ["*"]
        }
    },
});

Notes:

As said in a comment above, you must set reserved: true on the Plan for the container to be deployed as a Linux Docker app.

I'd still like to know how to do a few other things:

Can I expose multiple ports? (I kinda remember reading that App Service does not support this but not sure)
How do I get the URL of my app?
How would I run two different AppServices, one with 2 replicas and the other with 1 replica?

Some answers:

About multiple ports on an AppService, this is not possible, only 80 and 443 are open.
To get the URL of your app you can retreive the url on the defaultSiteHostname attribute of AppService instance. You can exports the generated values (eg: exports.myAppUrl = myApp.defaultSiteHostname.apply(n => `https://${n}`)), doing so, after deploying, pulumi will display the output exported once deployed.

How would I run two different AppServices, one with 2 replicas and the other with 1 replica?

You can set perSiteScaling: true for the azure.appservice.Plan to setup the plan for this.
For the app itself, there's unfortunately no way to set the siteconfig's numberOfWorkers via the upstream terraform provider yet AFAIK. Calling the ARM api to update the configuration should allow you to fill that gap for now http://docs.microsoft.com/en-us/rest/api/appservice/webapps/createorupdateconfiguration

One thing with this that I'm still struggling with is the fact that in the portal when applying this configuration the "Container Settings" show "Private Registry" and not "Azure Container Registry".
It obviously works, just wondering what setting is responsible for this, as I can't find anything in the config or resources.azure.com .

@davidobrien1985 It seems that the portal expects "Azure Container Registry" URL to start with https. If I do this in my Pulumi program:
DOCKER_REGISTRY_SERVER_URL: pulumi.interpolate`https://${registry.loginServer}`,

then the portal shows the first tab.

Closed via https://github.com/pulumi/examples/pull/302


Building issue on Fedora bazil.org/fuse".AllowRoot

Build command is long, but this is how rpm does it:
/usr/lib/golang/pkg/tool/linux_amd64/compile -o $WORK/b001/_pkg_.a -trimpath "$WORK/b001=>" -shared -p main -complete -installsuffix shared -buildid m_yMPH6vwMA6DHUTghoY/m_yMPH6vwMA6DHUTghoY -goversion go1.15 -D "" -importcfg $WORK/b001/importcfg -pack ./acl.go ./cleanup.go ./cmd_backup.go ./cmd_cache.go ./cmd_cat.go ./cmd_check.go ./cmd_diff.go ./cmd_dump.go ./cmd_find.go ./cmd_forget.go ./cmd_generate.go ./cmd_init.go ./cmd_key.go ./cmd_list.go ./cmd_ls.go ./cmd_migrate.go ./cmd_mount.go ./cmd_options.go ./cmd_prune.go ./cmd_rebuild_index.go ./cmd_recover.go ./cmd_restore.go ./cmd_self_update.go ./cmd_snapshots.go ./cmd_stats.go ./cmd_tag.go ./cmd_unlock.go ./cmd_version.go ./doc.go ./exclude.go ./find.go ./format.go ./global.go ./global_release.go ./lock.go ./main.go  github.com/restic/restic/cmd/restic
This is the error:
_build/src/github.com/restic/restic/cmd/restic/cmd_mount.go:118:39: undefined: "bazil.org/fuse".AllowRoot
Do you have any idea what may have caused this?
No, however, I did notice that Fedora is trying to use as the bazil dependency:
golang-bazil-fuse-devel                                                   noarch               0-0.15.20200722gitfb710f7.fc33
Do you have an idea how to solve the issue?
It looks like this is fixed in master, https://github.com/restic/restic/pull/2652. So the question is, can this patch be back ported to 0.9.6 safely
Did restic help you today? Did it make you happy in any way?
Always makes me happy :)
Other info
I am the maintainer of restic on Fedora

Please edit your post so it's clear exactly what commands you ran to build this, and please also wrap the commands and output in code blocks (see toolbar in message editor).

Cleaned it up a lot, sorry about the formatting. It was really bad.

I'm sorry, but this is something you'll have to ask the package maintainer for that RPM/distribution. We don't maintain the packages out there, so how they build and not on different distributions is something they have to deal with. The restic source code should build properly including the dependencies as they are versioned, so I'm surprised it doesn't in this case.

@copart I see you have already backported the PR. The changes in the PR are simple enough to work without problems (it's mostly just removing an optional function call).
The way I understand https://src.fedoraproject.org/rpms/restic/blob/master/f/restic.spec#_37 , it simply uses the latest version of each dependency?

@copart Is there anything left to do for this issue, or can we close it?

Closing this issue as apparently the problem was resolved and the outdated dependencies shouldn't be problem with restic 0.10.0


keyload update

Adding comments to the code is note the way to recommend code changes.
Also, I don't agree with the recommendation.
This PR will not be applied.


manage-bde -pause: Add missing dash before pause

Description:
As reported in issue ticket #5037 (dash missing in command-line command for manage-bde pause), the second part of the command ("pause") has to be prefaced with a dash to be recognized as part of the command.
Thanks to Neil Matin (@neil007m) for noticing and reporting this typo issue.
Change proposed:

add a preceding dash to all appearances of the command parameter "pause"
add the "meta" syntax highlighting name "Output" to the code block

Ticket closure or reference:
Closes #5037

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

Please note: You may want to disagree with my suggested change for the meta/non-effective syntax highlighting word "Output", as well as discussing if it is 100% correct to add the dash in the page title as well as the metadata title & description cells.


Pixbuf support for loading from bytestring

gdk_pixbuf_new_from_bytes
This is critical for single binaries that store data in memory e.g. using https://github.com/jteeuwen/go-bindata
Perhaps even replace all GIO-based Pixbuf interaction (C.gdk_pixbuf_new_from_file) with ioutil.ReadFile + gdk.PixbufNewFromBytes ?

Do you mean the API at https://developer.gnome.org/gdk-pixbuf/unstable/gdk-pixbuf-Image-Data-in-Memory.html? Or rather https://developer.gnome.org/gdk-pixbuf/unstable/gdk-pixbuf-Inline-data.html?
I can do some first steps in porting...

FYE: https://github.com/gotk3/gotk3/commit/5c02e875bfa5e15df2d2bc0e2161934ff2e38b4d contains inmem pixbuf handling. While playing around with transparency/alpha, I realize that some image conversion is needed to handle it correctly. The image.Decoder does some things too much...
Not sure where to discuss this.

at the moment is the best way, we don't add unstable functions.
the actual plan is to add the gtk and gdk stable functions.


@v2.0.0 - reduce package size

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.476%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
53

Relevant Lines:
57

💛  - Coveralls


no matching function for call to isnan

Probably AOMP developers have already been aware of this issue. Error since 11.11-0 in miniQMC when compiling source code with OpenMP offload compiler options although the source has no target region.
clang++ -DADD_ -DH5_USE_16_API -DHAVE_CONFIG_H -Drestrict=__restrict__ -I/home/yeluo/opt/miniqmc/src -I/home/yeluo/opt/miniqmc/build_ryzen_aomp_MP/src -I/home/yeluo/opt/miniqmc/external_codes/catch -fopenmp -fomit-frame-pointer -fstrict-aliasing -D__forceinline=inline -march=native -fopenmp-targets=amdgcn-amd-amdhsa -Xopenmp-target=amdgcn-amd-amdhsa -march=gfx906 -O3 -DNDEBUG -ffast-math -std=c++11 -o CMakeFiles/catch_main.dir/catch_main.cpp.o -c /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8591:9: error: no matching function for call to 'isnan'
    if (std::isnan(lhs) || std::isnan(rhs)) {
        ^~~~~~~~~~
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8643:20: note: in instantiation of function template specialization '(anonymous namespace)::almostEqualUlps<float>' requested here
            return almostEqualUlps<float>(static_cast<float>(matchee), static_cast<float>(m_target), m_ulps);
                   ^
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = float]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8591:28: error: no matching function for call to 'isnan'
    if (std::isnan(lhs) || std::isnan(rhs)) {
                           ^~~~~~~~~~
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = float]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8591:9: error: no matching function for call to 'isnan'
    if (std::isnan(lhs) || std::isnan(rhs)) {
        ^~~~~~~~~~
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8645:20: note: in instantiation of function template specialization '(anonymous namespace)::almostEqualUlps<double>' requested here
            return almostEqualUlps<double>(matchee, m_target, m_ulps);
                   ^
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = double]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:8591:28: error: no matching function for call to 'isnan'
    if (std::isnan(lhs) || std::isnan(rhs)) {
                           ^~~~~~~~~~
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = double]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:11585:9: error: no matching function for call to 'isnan'
    if (std::isnan(value)) {
        ^~~~~~~~~~
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:11748:12: note: in instantiation of function template specialization 'Catch::fpToString<float>' requested here
    return fpToString(value, 5) + 'f';
           ^
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = float]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
In file included from /home/yeluo/opt/miniqmc/src/Utilities/catch_main.cpp:3:
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:11585:9: error: no matching function for call to 'isnan'
    if (std::isnan(value)) {
        ^~~~~~~~~~
/home/yeluo/opt/miniqmc/external_codes/catch/catch.hpp:11751:12: note: in instantiation of function template specialization 'Catch::fpToString<double>' requested here
    return fpToString(value, 10);
           ^
/usr/lib/gcc/x86_64-linux-gnu/9/../../../../include/c++/9/cmath:632:5: note: candidate template ignored: substitution failure [with _Tp = double]: no type named '__type' in '__gnu_cxx::__enable_if<false, bool>'
    isnan(_Tp __x)
    ^
6 errors generated.

sorry about this regression.  The  clang math headers was a very big rip-up.   This will be fixed in 11.11-3 this week.

11.11-1 is available for you to try.

It is all good now.


Overwriting a specific event

Can't understand why you doing that? Why I can't bind to ctrl+a two callbacks?

Any updates on this, this is mainly useful when you try handle numeric keypad's buttons like (Enter, Plus, Minus, ...) as you can differentiate using the location.


feat: expose content of static files via endpoint

fix #4
Serve the content of given .json files via the rest /moderationList API endpoint
Files are located inside the /data directory, and split in 3 different files for easier history tracking:

flaggedLinks.json
flaggedProposalIds.json
verifiedSpaces.json

The endpoint also takes a ?fields= query params, in order to select only the chosen list (default to all if not set) for a lighter request, e.g. http://localhost:3000/moderationList?fields=verifiedSpaces,flaggedLinks,flaggedProposalIds

I bit hard to review since it includes changes from other PRs

Yes, this PR is waiting for the first commit of the repo, installing express and other basic packages, to be pulled first.
Will ping you when it'll be ready.
For now, are the features listed in the description what you're expecting ? Meaning we still need to commit changes to files in the repo to make changes, instead of using a DB ?

@samuveth Hey, PR is now clean of unrelated changes. There's only 2 main files for this feature, all the other are related to the tests.
For the .json files themselves, should we move them to another repo, only dedicated to serve json files (as I get that we may have more json files, with changes requested by user, and not team), so we could separate issues/tickets related to json files, apart from code ? (we still will use this repo to read the json file from here).

@wa0x6e please add the latest data from the original files so we can test/merge

@wa0x6e please add the latest data from the original files so we can test/merge

Done

Everything else looks good, I think we should actually add data :) so UI can test, and we can update the data later once everything is ready in UI and stop updating snapshot-spaces

What's the relation with snapshots-spaces ?

Please add the flagged proposal ID we added today as well https://github.com/snapshot-labs/snapshot/commit/106650215d9340e27d2acf93ed16a12315ac50c0
Is this deployed now, where can we test?

Please add the flagged proposal IDs we added today as well snapshot-labs/snapshot@1066502
Is this deployed now, where can we test?

Let's just go with that, until deployed and ready, and you will be free to submit PR to update these files


Make it work with non-numeric PKs

Thanks for creating this useful tool and sharing it publicly!
I ran into an issue when comparing CSVs with string-valued primary keys:
TypeError: '<' not supported between instances of 'float' and 'str'

If you are interested, here is a small patch that replaces math.inf with a custom class that can be compared against anyting. :smile:

Thanks for contributing!


Update warnAboutDeprecatedCJSRequire.js

修正ie下语法问题

Checklist

[ ] npm test passes
[ ] tests are included
[ ] documentation is changed or added
[ ] commit message follows commit guidelines

Description of change

any feature?
close https://github.com/dvajs/dva/ISSUE_URL

Coverage remained the same at 87.282% when pulling 2642ab767c2fffd9a6f06f47940831c839df71ba on cjdxhjj:master into 89e338e775d6a45ace494ae27a149517151f6003 on dvajs:master.


[Bug]: insecure download warning when downloading from pdf generator

Minimal, reproducible example
In ruby/rails app i get that warning prompt when trying to download pdf in chrome. I know my tls is fine because i dont get this error when using Prawn.

Error string
user warning prompt
Bug behavior

[ ] Flaky
[X] PDF

Background
No response
Expectation
to download without any errors prompting
Reality
warning prompting me to accept the download
Puppeteer configuration file (if used)
No response
Puppeteer version
20.9.0
Node version
v20.5.1
Package manager
npm
Package manager version
9.8.0
Operating system
Linux

Could you please add a reproducible example and describe why do you think it is an issue in Puppeteer?

Could you please add a reproducible example and describe why do you think it is an issue in Puppeteer?

  grover = Grover.new(html, format: Params.paper_size)
  pdf = grover.to_pdf
  return send_data pdf, filename: 'download.pdf', type: 'application/pdf'

Please report your issue to https://github.com/Studiosity/grover or provide a reproducible example using JavaScript, thanks!

Please report your issue to https://github.com/Studiosity/grover or provide a reproducible example using JavaScript, thanks!

you are right thats where i should have reported it, but are you also experiencing these new chrome download warnings everywhere recently? i do no like it.


Add "blank" to docs

I used the same format as in the entry for "required".

thanks for fixing this pull request


QT Wallet menu under the Windows & Linux

Black text on purple background is not good. Default colors look ugly under the Windows.
I try Windows 10 with default settings.

Confirmed with Jim on Linux and Windows. Looks better.


[BUG]  UI is accessible but websocket is returning 404 error

Rancher Server Setup

Rancher version: v2.6.7-rc9
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): v1.24.2+k3s2

Proxy/Cert Details: rancher/cert-manager

Information about the Cluster

Kubernetes version: v1.24.2
Cluster Type (Local/Downstream): Local

User Information

What is the role of the user logged in? Admin/Cluster

Describe the bug
The deployment is successful and we are able to reach the ui but in the browser console we get 404 when rancher tries to establish any websocket connections:
* /v3/subscribe
* /k8s/clusters/local/v1/subscribe
To Reproduce
Install rancher helm chart version v2.6.7-rc9 with k3s + kubernetes v1.24.2:
Result
Websocket connections failed with 404
Expected Result
Successfully establish websocket connects
Screenshots

Additional context
We are able to connect to the websocket inside the rancher pod with curl cmd.
The logs for the pod does not print anything helpful for debugging this issue.
We have the following setup in our environment and made sure to add the headers to upgrade the connection with nginx :
nginx -> traefik ingress controller -> pod

It was an nginx misconfiguration.

Can you please describe what did you do to solve the issue?

It was an nginx misconfiguration.

Can you please describe what did you do to solve the issue?


Reward System

As a user, I want to earn rewards for completing lessons to stay motivated.

How d

Given a user launches the app, When they successfully complete a task, Then the reward earned for that task is securely saved in the user's profile, And the reward becomes visible within the "Rewards" section of the app interface, The user can access the "Rewards" section to view their accumulated rewards and track their progress.

Feedback

How do the rewards look like? Do you just want to build in special badges or something else?
In this context how should the reward page look like?


Support for Always Encrypted - Microsoft.Data.SqlClient

It seems this Serilog sink depends on System.Data.SqlClient, which is not compatible with Always Encrypted as described in https://github.com/dotnet/SqlClient/issues/11 .
Are there plans to migrate to Microsoft.Data.SqlClient in the near future, since that seems to have support for Always Encrypted (https://github.com/dotnet/SqlClient/issues/11#issuecomment-493091283)?

No real plans, but open for a PR of course!

Looks like this is the reason for exception:
Exception thrown: 'System.PlatformNotSupportedException' in Microsoft.Data.SqlClient.dll
2020-07-07T16:01:05.1680936Z Unable to write 1 log events to the database due to following error: Microsoft.Data.SqlClient is not supported on this platform.
My .net standard 2.0 lib + serilog used as reference for web api with configuration like this:
<system.web>

</system.web>

Thanks for the feedback. Which framework is your executable assembly targeting?

Updated my prev comment by removing html braces, so it is visible now.

Thanks! Can you supply a minimal version of your solution to reproduce the problem?

https://github.com/Slavuta851109/SerilogBugOn452
To reproduce simply run web host.
Thank you for checking. Currently I decided to use working v.5.3.0 in prod.

@Slavuta851109 Thanks for the sample. I was able to reproduce your problem but have not yet had enough time to investigate the reason. I will continue looking into it as soon as I can and keep you posted.

@Slavuta851109 It seems due to an open issue in Microsoft.Data.SqlClient (see my mention above).
Currently there is no information about when or if this will be fixed in the future. I asked in their github issue for an update but the last comments were from late 2019, which does not give me much hope to see a fix soon.
Therefore I would like to revert the sink back to System.Data.SqlClient for .NET Standard. This means that the specific features of Microsoft.Data.SqlClient (like Always Encrypted) can only be used when targeting .NET Core or .NET Framework but not when using the sink in your .NET Standard library. But to me this seems to be less of a problem than to make the sink unusable from a .NET Standard library at all due to the exception.

After some further testing I found that this problem seems to only occurr when using the sink in a .NET Standard lib that is called from a .NET Framework app. It seems not to occur when the .NET Standard lib is called from a .NET Core app (see sample/NetStandardDemo).
Sink users with .NET Standard scenarios, can you please state if my overvations are correct?
I believe there are not as many users using the sink in the affected scenario while on the other hand there are more users using the sink in the working .NET Standard + .NET Core scenario. If we would now revert the sink for .NET Standard back to System.Data.SqlClient, this would probably take features away from many while solving a problem only for a few. So I would rather leave the code that way and hope the SqlClient team comes up with a fix soon. In the meantime, if you are on .NET Framework and using this sink, do not call it from a .NET Standard library.

@Slavuta851109 Good news! I could now understand and fix the problem using the sample app you provided. I was then able to generate log data with the sample app. Here is the proof:

The problem seems to be the same as described here: https://github.com/serilog/serilog-sinks-mssqlserver/issues/283#issuecomment-664397489
There seems to be a problem with Microsoft.Data.SqlClient when it is used in a classlib (e.g. Serilog MSSqlServer sink) and this classlib is then used in a .NET Framework application.
So, to fix the issue I had to add a reference to the NuGet package Microsoft.Data.SqlClient to the .NET Framework app (WebApplication1 in your sample). I'm afraid there is not much we can do about this in the sink but at least we have a workaround.

I'm not expecting anything out of this, just throwing it out there for other users to google and hopefully save a few hours.
Issue we had Serilog fails "silently" and nothing is logged in our db.
Workaround is roll back to v5.4 before Microsoft.Data.SqlClient was implemented or fix our server certificates.
details:
We've integrated serilog with our .net framework 4.72 application.
Issue is when using an IP address in our connectionstring against a server without a certificate (I know).. serilog doesn't write.

Pulled in source (thanks serilog open source 👍, also oops we didn't know about SelfLog.Enable()) and found it failing with this error:
A connection was successfully established with the server, but then an error occurred during the login process. (provider: SSL Provider, error: 0 - The certificate chain was issued by an authority that is not trusted.)
serilog-sinks-mssqlserver\src\Serilog.Sinks.MSSqlServer\Sinks\MSSqlServer\Platform\SqlBulkBatchWriter.cs:line 43
Here:

Thank you for the research and documentation of this.


Incorrect formatting on Unit Testing and Test Driven Development tutorial page

The Unit Testing and Test Driven Development page (https://p5js.org/learn/tdd.html) is formatted incorrectly and is exposing code that shouldn't be visible (such as GA snippet). This is probably due to an incorrectly closed HTML tag
I'm happy to work on this and raise a PR when done if you would like.

that would be great!


Meaning of frequency when set to "After data refresh"

In the document, it is stated that:
Subscriptions that have the frequency set to After data refresh will only send an email after the first scheduled refresh on that day.
Is "that day" based on the UTC time zone?
Thanks!

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

ID: 2caf5705-b9c8-3818-4524-3c4bf9b112a2
Version Independent ID: 98ede3b9-f4f6-88a6-83fc-6afab863bf4a
Content: Email subscriptions for reports and dashboards in the Power BI service - Power BI
Content Source: powerbi-docs/collaborate-share/end-user-subscribe.md
Service: powerbi
Sub-service: pbi-explore
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @asocorro. This issue has been assigned to the doc author for review.

Hi @asocorro ,
Thank you for your suggestion.  We've made the update and set the article to republish.
All the best,
The DOCS team


RuntimeError: checkpoint url or path is invalid

00             checkpoint = torch.load(model_path, map_location="cpu")
101         else:
--> 102             raise RuntimeError("checkpoint url or path is invalid")
103
104         state_dict = checkpoint["model"]
RuntimeError: checkpoint url or path is invalid

Could you provide the detailed file?


rescue_from :all  &  Grape::Exceptions::ValidationErrors

I was writing an integration test of our API and was having trouble understanding why I was getting a 500 response code on validation tests instead of the 400 response I was expecting.  Eventually I tracked it to our use of rescue_from :all.
It would be cool if we could still use rescue_from :all but for situations where the error is a validation error the returning response code would be a 400?
Thanks!!

Upon further debugging today, we discovered that error.error_response doesn't allow the status code to be passed thru from the source error when you do a rescue_from :all, and do not specify a block for it.  So a validation error that would otherwise return a 400 status returns the default 500 status code.
https://github.com/intridea/grape/blob/873f801440ba0c08e4627d784f2865ffc0de0302/lib/grape/middleware/error.rb#L65

I am going to close this by design, added some explanation in #939. For context here, rescue_all's purpose is exactly to trap all exceptions and handle them in a uniform way, shortcutting any other decisions about status codes and such. It's a catch all.
If you want to return a 400 validation error with a catch-all, handle those explicitly via a rescue_from Grape::Exceptions::Validations.
I can see the opposite argument here, but I also want to mention that this would be a pretty major behavior change for existing APIs and you can already achieve what you want with existing code.
Please do let me know if I missed something!

@johnsinco
If you want to return a 400 validation error with a catch-all, handle those explicitly via a
rescue_from Grape::Exceptions::ValidationErrors do |e|
error_response(message: e.message, status: 406)
end


Can't filter properly when using auto flip

Expected Behavior
When using automatic page turning, search results can be filtered normally
Actual Behavior
When using automatic paging, the search results are not properly filtered
Steps to Reproduce the Problem
1.Install this tampermonkey script(https://greasyfork.org/scripts/419215-自动无缝翻页/code/自动无缝翻页.user.js)
1.Do a google search (choose keywords that are easy to intercept)
1.Successfully reproduced
When the page is not turned automatically (normal search situation, everything is ok)

There was a problem after the automatic page turning, and the results of subsequent loading were not successfully filtered (guessing that such a situation was not considered)

Specifications

Browser:Chrome 113.0.5672.63
Other extensions/add-ons:tampermonkey uBlacklist

Search engine:google
Language:中文
Region:台湾

DUPLICATED
https://github.com/iorate/ublacklist/issues/266#issuecomment-1229362937


time format should include ss.ffffffff

when looking at the scheduleEnqueuetime and equeue time's both are in minutes.... would be nice to see a more precise time that includes seconds and milliseconds.

Out of curiosity, what's the use case for such precision that I have a feeling cannot be guaranteed by the broker.

I don't understand what you are referring to about precision of the broker. Displaying the seconds would already be very helpful for me. The milliseconds/microseconds I guess can live without.
The reason I would like to include seconds is that I've some messages in the DLQ and I'm trying to correlated messages in the logfile or app-insight and this might help pinpoint the cause. Especially with lock tokens expiring after seconds not minutes and I use the scheduled enqueue time to handle deferred messages where the schedule enqueue time is exponential in seconds.

@qsdfplkj when I implemented this feature I had in mind scenarios where message consumption is scheduled after minutes, not seconds. We could simply replace the actual textbox with a smaller textbox + dropdownlist where you can select the unit of time:

seconds
minutes
hours
The solution should be relatively simple to implement. @qsdfplkj let us know if you can implement this feature and submit a PR, thanks.

https://github.com/paolosalvatori/ServiceBusExplorer/pull/535

@qsdfplkj, please post one or more screenshots of the GUI changes.


auth: nil check AuthInfo when checking admin permissions

If the context does not include auth information, get authinfo will
return a nil auth info and a nil error. This is then passed to
IsAdminPermitted, which would dereference the nil auth info.
via #7471
/cc @mitake

@heyitsanthony lgtm, thanks! CI failures seem to be unrelated to this change. I agree with merging this PR.


oh-my-posh: 21.9.1 -> 21.17.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: Prompt theme engine for any shell
meta.homepage for oh-my-posh is: https://ohmyposh.dev
meta.changelog for oh-my-posh is: https://github.com/JanDeDobbeleer/oh-my-posh/releases/tag/v21.16.0
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 21.16.0 with grep in /nix/store/rbj9v7yvnmhj49flv5fbzjn5h2rdy6b8-oh-my-posh-21.16.0
found 21.16.0 in filename of file in /nix/store/rbj9v7yvnmhj49flv5fbzjn5h2rdy6b8-oh-my-posh-21.16.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rbj9v7yvnmhj49flv5fbzjn5h2rdy6b8-oh-my-posh-21.16.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oh-my-posh https://github.com/r-ryantm/nixpkgs/archive/131f74da45838611c63e533d310a25aa98872201.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/131f74da45838611c63e533d310a25aa98872201#oh-my-posh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rbj9v7yvnmhj49flv5fbzjn5h2rdy6b8-oh-my-posh-21.16.0
ls -la /nix/store/rbj9v7yvnmhj49flv5fbzjn5h2rdy6b8-oh-my-posh-21.16.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins @urandom2 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: Prompt theme engine for any shell
meta.homepage for oh-my-posh is: https://ohmyposh.dev
meta.changelog for oh-my-posh is: https://github.com/JanDeDobbeleer/oh-my-posh/releases/tag/v21.17.0
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 21.17.0 with grep in /nix/store/3s0jijzlk1mysf7pk75qis7ldm3wa97g-oh-my-posh-21.17.0
found 21.17.0 in filename of file in /nix/store/3s0jijzlk1mysf7pk75qis7ldm3wa97g-oh-my-posh-21.17.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3s0jijzlk1mysf7pk75qis7ldm3wa97g-oh-my-posh-21.17.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oh-my-posh https://github.com/r-ryantm/nixpkgs/archive/9d4091ebe1fc89354cbe6531dc3c7e2c9a0712c9.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9d4091ebe1fc89354cbe6531dc3c7e2c9a0712c9#oh-my-posh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3s0jijzlk1mysf7pk75qis7ldm3wa97g-oh-my-posh-21.17.0
ls -la /nix/store/3s0jijzlk1mysf7pk75qis7ldm3wa97g-oh-my-posh-21.17.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins @urandom2 for testing.

Done in #323647.


Replace string pointer with plain string

There is no reason to use a string pointer here.
It makes the calling code weird since string pointers are pretty much never used in Go.

Something that is 99% ready and only needs to update the deps, so we can
merge it within few minutes of each other.
On Tue, Aug 27, 2019 at 1:24 PM chaodaiG<EMAIL_ADDRESS>wrote:

I think @vagababov https://github.com/vagababov meant a test PR in
serving with changes to vendor directly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/knative/pkg/pull/607?email_source=notifications&email_token=AAF2WX5CEILTU6OW2NNYKO3QGWEQ3A5CNFSM4IQJILH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5JAD7Q#issuecomment-525468158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAF2WX5X3G7VS43KOZ6AY5LQGWEQ3ANCNFSM4IQJILHQ
.

/hold cancel


properties should support concurrency

#288

:shipit:

It works, I sure hope we don't forget about it and have someone accidentally change it in future.
Can you add a unit test which asserts that the type of the IDictionary provided in key types is ConcurrentDictionary as expected to be safe?
:shipit:


[incomplete] Start splitting code into session connection classes

@DavideGalilei you don’t need the [incomplete] in the title if you’ve marked it as draft but OK


Revert "TGP-2558/table-overflow-handled-properly"

Reverts hmrc/trader-goods-profiles-frontend#451

reverting for now as it overflow without scrollbar at times


opening the Developer Tools pane prevents the Command Palette from taking focus

If I use the keyboard to open the developer tool panel (using the command palette) and then try to run another command using the command palette, I end up typing into the console. In detail:

make sure the developer tool panel is closed
type cmd-shift-p to open the command palette
type "toggle dev" followed by a "return" to open the developer tool pane
type cmd-shift-p to open the command palette again
type anything; the text goes into the console (developer tool pane) at the prompt, instead of into the command palette. This is scary. I expected the text to go into the command palette.

By contrast, if I type cmd-t (to open the file palette) after I open the developer tool pane, nothing seems to happen. I don't get the file palette, but at least I don't end up typing somewhere unexpected and possibly dangerous.
Personally I wish cmd-t and cmd-ctrl-p both worked normally at that point. However, I admit that I am not an atom developer; I only open the developer panel when I want to see error messages. If I was a heavy atom developer I might prefer the cmd-t behavior. I can't imagine ever wanting the cmd-ctrl-p behavior.
I am on MacOS 10.9.5 with Atom 0.177.0

When you open the dev tools it takes focus. While it has focus it will intercept it's own shortcut combinations. Simply clicking into the Atom editor again will let Atom continue to accept its commands.


Add refresh button for xDS, stats, and clusters

Description
Add refresh button for xDS, stats, and clusters

This would be extremely useful in a case where grafana dashboards are not available.


x509: certificate signed by unknown authority

Looks like this is coming from inside the container, could be our corporate proxy security firewall, Idk.
docker run --rm --network host \
  -e AKITA_API_KEY_ID=apk_5...F \
  -e AKITA_API_KEY_SECRET=1...3 \
  akitasoftware/cli:latest apidump \
  --project RTTA

Unable to find image 'akitasoftware/cli:latest' locally
latest: Pulling from akitasoftware/cli
72cfd02ff4d0: Pull complete
4ac5a229ebba: Pull complete
829d9da225a6: Pull complete
Digest: sha256:7e20d3b81b28a35669e2eb25cef0b9275fc8e746dbabe30ecee967e9574ad6d2
Status: Downloaded newer image for akitasoftware/cli:latest
[INFO] Akita Agent 0.23.0
[ERROR]  request failed [error Get "https://api.akita.software/v1/user": x509: certificate signed by unknown authority method GET url https://api.akita.software/v1/user]
[INFO] Telemetry using temporary ID; /v1/user API call failed: Get "https://api.akita.software/v1/user": x509: certificate signed by unknown authority
[INFO] This error may indicate a problem communicating with the Akita servers,
[INFO] but the agent will still attempt to send telemetry Akita support.
[ERROR]  request failed [error Get "https://api.akita.software/v1/services": x509: certificate signed by unknown authority method GET url https://api.akita.software/v1/services]
[ERROR] failed to get list of services associated with the account: Get "https://api.akita.software/v1/services": x509: certificate signed by unknown authority

I troubleshoot the problem to our firewall.
openssl s_client -proxy webproxy:8080 -connect api.akita.software:443
It's a forbidden from the proxy - and it's certs are signed.

Workaround.. for the client.
I unset all proxies in docker and on the system.
got the certificate chain and put the last certificate into the file.
mounted the certificate into the container
docker run -it --name akita -v D:\transport-ca.pem:/etc/ssl/certs/t.pem akitasoftware/cli:0.23.1
can login and apidump now
Still can't do this on the docker extension as it's a single /etc/ssl/certs/ca-certs.crt file


Documentation for adding normal variables to group is incorrect

The documentation claims: Variables in format key=value space separated pairs. Secret variables should be managed using az pipelines variable-group variable commands.
However, when doing so, those variables are created under one variable, they are not parsed and created separately.

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

ID: a067cc50-3fa1-3f19-1c8f-d0077be070d8
Version Independent ID: 6a0e745f-e9ee-644a-85ce-db299b10551d
Content: az pipelines variable-group
Content Source: latest/docs-ref-autogen/pipelines/variable-group.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Routing to service team.


Incompatible with airflow 2.7.x due to jsonschema version conflict

Describe the bug
jsonschema library version conflict when building a poerty project with airflow:2.7.1 and datahub:0.11
is it possible to loosen version dependencies
To Reproduce

pyproject.toml

[tool.poetry]
name = "airflow-datahub"
version = "0.1.0"

[tool.poetry.dependencies]
python = "~3.10"

apache-airflow = { version = "2.7.1", extras = [ "statsd" ] }
acryl-datahub = { version = "^0.11", extras = [
    "datahub-rest", "postgres", "kafka", "kafka-connect", "ldap", "tableau", "superset",
    "athena", "glue", "mysql"
] }

[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

poetry update --lock

Screenshots
Because apache-airflow (2.7.1) depends on jsonschema (>=4.18.0)
   and acryl-datahub (0.11.0) depends on jsonschema (<=4.17.3), apache-airflow (2.7.1) is incompatible with acryl-datahub (0.11.0).
  And because no versions of acryl-datahub match >0.11.0,<0.12.0, apache-airflow (2.7.1) is incompatible with acryl-datahub (>=0.11.0,<0.12.0).
  So, because airflow-datahub depends on both apache-airflow (2.7.1) and acryl-datahub (^0.11.0), version solving failed.

Additional context
poetry:1.4.2
python: 3.10
os: MacOS

@KulykDmytro we did this a few days ago: https://github.com/datahub-project/datahub/pull/8842. It will be in a release soon.
Closing this issue for now, but let us know if you continue to have issues with it.

Hi @hsheth2, do you know when the next release is coming?

It's available in acryl-datahub v<IP_ADDRESS>


Implement sliders properly

Currently it just delegates in a combo. We might want to use something like this.

Found this one as well.


Include printer profiles being prototyped for python-escpos

As a python-escpos dev
I want the profiles prototyped at python-escpos/python-escpos#149 to be included in this database
So that I can fetch a single file with everybody's profile-related work in it

[ ] TM-T90
[ ] TM-P20 2
[ ] TM-P60II 2
[x] TM-P80
[x] TM-T88III
[x] TM-T88II

YAML snippet for things left to add. TM-P80 and TM-T88II added with current values at #9
    TM-P60II 2 (58mm):
        inherits: default
        manufacturer: "Epson"
        media:
            width:
              mm: 58
        fonts:
            a: {columns: 35}
            b: {columns: 42}
            c: {columns: 52}

    TM-P60II 2 (60mm):
        inherits: default
        manufacturer: "Epson"
        media:
            width:
              mm: 60
        fonts:
            a: {columns: 36}
            b: {columns: 43}
            c: {columns: 54}

    TM-P20 2:
        inherits: default
        manufacturer: "Epson"
        fonts: [a, b c, d, e, f]

    TM-T90:
        inherits: default
        manufacturer: "Epson"
        colors:
            - black
            - red

(code pages will need a check-over if bugs crop up)


bug(streaming): Timestamp unordered in Materialized View with ORDER BY

Describe the bug
Timestamp not ordered in materialized view with ORDER BY.
To Reproduce
create table t(v timestamp);
 insert into t values
    (timestamp '2022-01-01 10:04:00'),
    (timestamp '2022-01-01 10:03:00'),
    (timestamp '2022-01-01 10:02:00');
create materialized view m as select * from t order by v;
select * from m;

Result changes:
dev=> select * from m;
          v
---------------------
 2022-01-01 10:04:00
 2022-01-01 10:03:00
 2022-01-01 10:02:00
(3 rows)

dev=> select * from m;
          v
---------------------
 2022-01-01 10:03:00
 2022-01-01 10:04:00
 2022-01-01 10:02:00
(3 rows)

Expected behavior
same as batch:
create table t(v timestamp);
 insert into t values
    (timestamp '2022-01-01 10:04:00'),
    (timestamp '2022-01-01 10:03:00'),
    (timestamp '2022-01-01 10:02:00');
select * from t order by v;

dev=> select * from t order by v;
          v
---------------------
 2022-01-01 10:02:00
 2022-01-01 10:03:00
 2022-01-01 10:04:00
(3 rows)

dev=> select * from t order by v;
          v
---------------------
 2022-01-01 10:02:00
 2022-01-01 10:03:00
 2022-01-01 10:04:00
(3 rows)

Additional context
No response

https://www.risingwave.dev/docs/current/release-notes/#sql-features-3:~:text=An ORDER BY,scans. %233670

I feel we probably need to give a warning that links to the doc page when users do Materialized View with ORDER BY


FAQ: How come you are using global, element level resets on getbem.com when you recommend against it?

Hi.
While trying to square our real world lives with BEM best practices we came to the question about CSS reset. I do like your approach from the FAQ, but since we have done some CSS diving on your site - to see how you do things - I also know that you use Milligram as the CSS base for that site and it does CSS reset for you ...
How does that play in to the overall BEM philosophy? As an example, your .panel h2 doesn't reset it's line-height, but relies on the global h2 line-height reset from Milligram, and will break if you change the global h2 line-height.
Is this an oversight, pragmatism or am I just trying too hard to stick to the rules here?

Milligram will be removed in next site iteration. I don’t think css resets are forbidden by BEM though.


Added convenience methods

Make SamplingMethod a Generator for use in for-loops
Make SurrogateModel callable for intuitive use as function

The CI failing should not be related to the changes

The CI failing should not be related to the changes

Sampling methods generate points all at once, there is no meaning in calling compute for one point.
Regarding the surrogate model callable API, unless there is a strong argument for it, I prefer the explicit predict_values call and do not want to have several ways for doing the same thing.

Sampling methods generate points all at once, there is no meaning in calling compute for one point.
Regarding the surrogate model callable API, unless there is a strong argument for it, I prefer the explicit predict_values call and do not want to have several ways for doing the same thing.

* Sampling methods generate points all at once, there is no meaning in calling compute for one point.

* Regarding the surrogate model callable API, unless there is a strong argument for it, I prefer the explicit `predict_values` call and do not want to have several ways for doing the same thing.

The send method does provide special features to the class in python. See the docs here
The reason is not to call compute for one point, instead it enables code like this:
...
sm = Random(xlimits=xlimits)
# Iterator
for sample in sm:
    ...
# or
transformed_samples = [transform(sample) for sample in sm]
# or use as generator
transformed_sample_generator = (transform(sample) for sample in sm)

Also, it enables coroutines see https://www.python.org/dev/peps/pep-0342/
Using generators is considered as good practice (as described here).
The Sampling Methods could also overwrite the send method and provide a more memory/computationally efficient generator. Also it enables Sampling Methods to receive the parameter value, allowing them to generate context dependent samples. This is part of the coroutine pattern.

* Sampling methods generate points all at once, there is no meaning in calling compute for one point.

* Regarding the surrogate model callable API, unless there is a strong argument for it, I prefer the explicit `predict_values` call and do not want to have several ways for doing the same thing.

The send method does provide special features to the class in python. See the docs here
The reason is not to call compute for one point, instead it enables code like this:
...
sm = Random(xlimits=xlimits)
# Iterator
for sample in sm:
    ...
# or
transformed_samples = [transform(sample) for sample in sm]
# or use as generator
transformed_sample_generator = (transform(sample) for sample in sm)

Also, it enables coroutines see https://www.python.org/dev/peps/pep-0342/
Using generators is considered as good practice (as described here).
The Sampling Methods could also overwrite the send method and provide a more memory/computationally efficient generator. Also it enables Sampling Methods to receive the parameter value, allowing them to generate context dependent samples. This is part of the coroutine pattern.

I do not contest the generator pattern, but taking into account your implementation, the code below (your example) just loops for ever.
import numpy as np
from smt.sampling_methods import Random

xlimits = np.array([[0.0, 1.0]])
sm = Random(xlimits=xlimits)
for x in sm:
    print(x)

I do not contest the generator pattern, but taking into account your implementation, the code below (your example) just loops for ever.
import numpy as np
from smt.sampling_methods import Random

xlimits = np.array([[0.0, 1.0]])
sm = Random(xlimits=xlimits)
for x in sm:
    print(x)

I do not contest the generator pattern, but taking into account your implementation, the code below (your example) just loops for ever.
import numpy as np
from smt.sampling_methods import Random

xlimits = np.array([[0.0, 1.0]])
sm = Random(xlimits=xlimits)
for x in sm:
    print(x)

This is expected, as this represents a stream (you can find something like this in Java as Streams or in Haskell as infinite lists). You would instead use the sample to for example update your model and when your model is good enough or fulfills a certain criteria you would break the loop.

I do not contest the generator pattern, but taking into account your implementation, the code below (your example) just loops for ever.
import numpy as np
from smt.sampling_methods import Random

xlimits = np.array([[0.0, 1.0]])
sm = Random(xlimits=xlimits)
for x in sm:
    print(x)

This is expected, as this represents a stream (you can find something like this in Java as Streams or in Haskell as infinite lists). You would instead use the sample to for example update your model and when your model is good enough or fulfills a certain criteria you would break the loop.


ServiceNow CI/CD Install Application -  Application Version Issue

Hi Chiarng,
When I have App 1.0.0 on my DEV instance (dev branch) and I use the dev branch to update the delta code to my test instance using ServiceNow CI/CD Install Application (ServiceNow-CICD-SC-Apply@1) extension, it installs 1.0.1 version.
When I choose to pick the application version from source (which I manually update in Application Settings prior to App Publish), why would I want the ServiceNow CI/CD Install Application step to increment the version for me? Is this an expected behavior?

Hi Ramyaramdasan,
If you do not want the increment use the versionformat as detect_without_autoincrement (versionFormat: 'detect_without_autoincrement')
This was updated in the latest release

Hi Vinay,
I believe the version format is for Publish to App Repo. In the ServiceNow CI/CD Install Application extension to install application from source control repository (GitHub in my case), there is no version format that we can mention. Or is my understanding wrong? Can we use detect_without_autoincrement in this extension too?

Hi Ramya,
How are you doing the publish.? If you are making in the pipeline, then use the version for app publish in pipeline and that should solve the issue.
Ideally for install you just give sys_id as input and that installs the latest vesion.

Hi Vinay,
The App Publish task (ServiceNow-CICD-App-Publish@1) is working as expected. I am referring to Install Application task(ServiceNow-CICD-SC-Apply@1) which does not have a versionformat input

Hi Ramya,
I am still in doubt how it will increase version without having that version in the app repository.
Try to send input publishVersion with value.
I believe that should work.

The version we set under Application Settings (sys_app table) is also loaded into GitHub Repository as an application file when we commit the code into the repo, may be from there? I don't see any publishVersion input for this task.

task: ServiceNow-CICD-SC-Apply@1
inputs:
connectedServiceName:
appSysId:
branchName:
appScope:

add that publishVersion : '{your version value}' under the appScope and give a try

Sorry, I am not sure I understand it. I am new to Azure Pipelines. What is this syntax for -  '{your version value}' how do I pass the latest version in here? The pipeline is auto-triggered when I commit code into the source control, so how can I pass this value?

publishVersion : '1.0.0' (this 1.0.0 is version you wanted to install)

Hi @ramyaramdasan and @vinay2099,
Just to clarify, the AppInstall task has a version parameter that is optional. If it's filled in, it'll try to install that version of the app from app repo. If it's NOT filled in, it will default to whichever version was published by the AppPublish task (or try to).
https://github.com/ServiceNow/servicenow-cicd-azure-extension/blob/master/src/lib/AppInstall.js#L19
So @ramyaramdasan the behavior you are seeing is because the AppPublish task is first auto-incrementing the version to x.y.(z+1), that's why the AppInstall task is trying to install x.y.(z+1). Originally, this was intended to help automate pipelines by not requiring the developer to have to keep bumping up versioning, but in practice I'll admit it's been tough to use because the auto-incremented version isn't committed back to Source... I would actually love for somebody to fix that haha.
@vinay2099 is correct that we recently added a new versionFormat value as an option to NOT auto-increment:
https://github.com/ServiceNow/servicenow-cicd-azure-extension/releases/tag/1.7.0
So you can try to publish with the new "detect_without_autoincrement" option and then your AppInstall will do x.y.z.
Hope that helps! :)
Thanks,
Chiarng

Hi @chiarnglin ,
My Pipeline starts with a commit to GIT from DEV Studio. Pipeline updates TEST instance using AppInstall extension. Then ATF is run on TEST. If successful, we do AppPublish to AppRepo and update ACC using AppRepo. Now:
Step 1) I commit 1.0.0 from DEV Studio
Step 2) Pipeline triggers. AppInstall updates TEST instance with 1.0.1
I still haven't published my app. The AppInstall is picking code from GIT, which has 1.0.0, but when it updates TEST, its incrementing the version.
Step 3) On successful ATF, applicaton is published from TEST, so App Repo gets version 1.0.1 as I have set it to 'detect'.
Going through AppInstall code, it looks like it is taking version details from AppPublish? But how would that work in a pipeline where AppInstall happens before AppPublish?

@ramyaramdasan Hmm. Granted I haven't tried installing without publishing, but theoretically it should error out since there's no versions in the app repo to install from. The incrementing of versions should only happen on AppPublish. Let's take a look at your pipeline in closer detail on the Monday call. I'm wondering if somehow you have multiple stages of your pipeline simultaneously triggering on parallel, so AppPublish runs before AppInstall, somehow. Either that, or there's some random bug we have to go hunt down.

Sure @chiarnglin . I am not talking about App Install from Repo. ServiceNow-CICD-SC-Apply@1 is the extension for  ServiceNow CI/CD Apply Changes (code is picked from GIT Repository).


source-repo-sync: don't clone submodules

If one branch of a repository has a submodule that another does not, the
submodule may get added to the commit.

I think this is a better fix


self.navigationcontroller

Project   LeftViewController has a  login_button,  when  click the button ,  the  print  self.navigationcontroller  is nil  whit an error , how to fix it ?

I believe LeftViewController should not have a self.navigationController.. instead, you can try using self.SlideMenu (Dont remember the name of the function), then get the main controller, and ask that one for the navigationController (asumming that one has it).

I get the .manViewController  not null but  print(self.slideMenuController()?.mainViewController?.navigationController)   always nil

Can you post an example?


Update README.md

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
[ ] Filename in the lowercase format
[ ] Image format: png

[ ] Image size: 256 by 256 px

Add BIK


feat: Type parser for complex trino types

SUMMARY
Initial work to support a rewrite of the PRESTO_EXPAND_DATA feature for trino. This allows parsing complex structural types described by trino, e.g.
row(foo varchar, bar int)

or arbitrarily deeply nested types like
row(foo row(bar array(int)))

This parses them into a structural format we can then inspect in order to discover all fields and structures available within them, which is needed in order to expand the results of a SELECT * query out into additional rows and columns, or to help autocomplete dotted.paths available.
Note this doesn't yet fully implement expand_data for trino, I've tried to break it down into reviewable pieces and I've started with the low-level logic of parsing out the types into something structured. I'll start trying to integrate this into the trino spec and implement expand_data next, which will involve analysing this structure to separate the data out into multiple columns like it's being done for presto with PRESTO_EXPAND_DATA.
TESTING INSTRUCTIONS

Unit tests. Currently there's no impact on implementation, as the next steps will be using this type parsing logic to write expand_data logic for the TrinoEngineSpec.
ADDITIONAL INFORMATION

[x] Has associated ~issue~ discussion: https://github.com/apache/superset/discussions/25713
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

Part one, see associated discussion for some notes about this functionality: https://github.com/apache/superset/discussions/25713
Not sure if we need the entire feature in one PR, happy to do that if required but I think reviewing the code will be much easier with it broken down so I've raised this first part now.

CC @sujiplr @cccs-tom for additional 👀

Thanks @giftig for the change. One overarching question I have is whether we should be adopting the "shift left" mentality and add said logic to https://github.com/trinodb/trino-python-client, given that it already provides a Trino to Python type mapping, albeit maybe being limited for the advanced types.

That's a fair point @john-bodley; as I'm looking into this my gut feeling is that PRESTO_EXPAND_DATA built a feature into superset to account for it being missing in trino -- I think actually trino itself, even, rather than the python client.
We're essentially working around two possible missing features in trino:

We can't ask trino to summarise the types of even the names of each field in a ROW for us
We can't ask trino to expand ROWs out for us itself

which both feel like necessary features to have given it does let us descend into the row with a dotted.path.syntax.
And, as I documented on my class here, there are edge cases where trino will break the format of its own type annotation by failing to quote column names containing spaces or brackets.
I think I'm probably suffering from the same problem that the original author of the PRESTO_EXPAND_DATA feature did, which is that it's easier to contribute code to superset than to trino.

What I'll do in parallel to this is try to raise this as the an issue with the trino community I am in touch with a couple of relevant people on the trino side so I may be able to get at least a bit of feedback on whether these features would be considered. Unfortunately I'm not a java developer, though, so it'd be difficult for me to make a fix in trino if it comes to it.

PR to complete the work I described above: https://github.com/apache/superset/pull/25809
based on existing type-parsing available in the client instead of this PR.


Heatmap of predicted bboxes confidence and its IoU score with GTs

Hi, This is a very good piece of work, and some of the methods of analyzing problems are very good. I would like to ask how Figure 4 in the paper was drawn, and is it convenient to provide the code? It is the heat map of confidence and IOU scores.

"That seems to be code for plotting a scatter plot. How about the code for drawing a heatmap? Thank you!"

Should be quite similar. Just use the main code to get the score_list and iou_list, and plot using plt.hist2d
score_list= np.array(score_list)
iou_list = np.array(iou_list)
grid_x, grid_y = 50,50
h, x, y, _ = plt.hist2d(score_list, iou_list, bins=(grid_x, grid_y), norm=matplotlib.colors.PowerNorm(0.2), density=True)

MINX = np.min(score_list)
MAXX = np.max(score_list)

plt.xlabel('Confidence Score', fontsize=20)
plt.ylabel('IOU with the GT', fontsize=20)
plt.xlim(MINX,MAXX)
plt.ylim(0.0,1.)
plt.colorbar()
plt.tight_layout()

plt.show()

Many thanks!


Add refund button

Add a refund button to the payment overview screen

This needs more thought around users and permissions

This can now be done thanks to #79


M-01 redeem Lacks Slippage Protection

The redeem function in Escrow.sol contains the logic to determine the amount of GRT rewards an indexer will receive as the minimum of the currently available escrow balance and the aggregated amount in the voucher. This allows slippage to occur against the indexer (i.e., the indexer may get less than what is deemed acceptable).
The redeem function does not allow the indexer to specify an expected amount of rewards to receive. Therefore, it is possible that the indexer receives fewer rewards than expected, at which point the voucher that was worth more has been used and cannot be re-used. This could surprise the indexer, who may have otherwise expected the call to revert if the full reward amount could not be given to them.
Consider adding a parameter to the redeem function that allows the caller to specify an expected reward amount. If the calculated amount is not greater than or equal to the expected amount, the call should revert.

Does adding slippage protection open up a somewhat niche but still real front-run attack. Specifically when a RAV is submitted for an EscrowAccount with low funds.
It would look something like this:

Escrow account has some arbitrarily low amount (lets say 0)
Indexer that is relying on slippage protection blindly submits RAV
Malicious actor takes all info from mempool and front runs the function call, but replaces the slippage value with 0 (or permitSlippage == true if we use a bool instead)
Because the new slippage value is zero the transaction goes through and the allocation ID is claimed for that sender
Indexer is now unable to claim their rewards

Discussion from slack about solutions and trade-offs:
Me:

So right now possible solutions would be:

Revert if account is below redeemed amount
Allow underfunded redeem and leave it to the indexer to only call redeem if they are ok with the amount available in the account
Allow underfunded redeem and include a min_redeem amount with a singed proof

Pablo V.:

I think the slippage without protection (#2), even with off-chain checking, could open unexpected edge cases, e.g. a redeem is called while some amount is thawing, and the transaction gets delayed until the thaw is completed... Whereas the revert (#1) produces more predictable behavior but less flexibility, and adding the signed parameter (#3) gives us the "best of both" worlds at the cost of uglier code and potential dead code in a future upgrade. So I don't see a clear winner, my instinct just pulls me towards #1 as it's the simplest, or #3 as it's the most "complete", but I think it's fine if you make the call for #2.

Also for reference here is a previous discussion on this topic #5

One more possible solution that was considered:
Split the redeem API into two functions:

redeem() which would implement #1
redeemWithSlippage() which would implement #3

This would allow indexer code to either choose either slippage protection with a tradeoff of extra gas or to choose all or nothing redeem (reverts if there is not enough to completely redeem RAV).

Final decision is to leave the code as is (option #2). This allows flexibility while maintaining a low gas cost. The documentation will be updated to reflect the expectation of the indexer.
Some context for the decision:

Whenever a thaw commences and it threatens to reduce the escrow account below the owed amount, the indexer software should promptly act to close any pending RAVs with that gateway. Otherwise, the indexer should act as if anything being thawed has already been withdrawn.
Complete transparency exists regarding an account: indexers can verify values anytime, and numerous events (like deposit, thaw, withdraw, redeem, and other account interactions) are signaled.
The only methods to withdraw money from an account are either under the indexer's control (redeem()) or are subject to a time-lock (withdraw()).

Pasting below a comment from OpenZeppelin reviewer from Slack, Aug/22/2023, 6:20AM PST:
Thanks for the great discussion @ColePBryan and Pablo and laying out the ideas clearly. I see a few considerations. Allow me to summarise to form an opinion.
On (#1): The purpose of setting a slippage is to protect an honest indexer from a rogue sender/signer pair. For instance, a rogue sender pairs with a rogue signer could sign away more value than they plan to pay to honest indexers. The rogue sender could either (a) deposit nothing/little for the indexer; or (b) pre-thaw and revoke the signer to invalidate a voucher. In both cases, the indexer gets nothing/little when redeeming. In (a), it can’t redeem again and the voucher is considered used; In (b), the voucher can’t be used unless the same signer is again authorised by the same/different sender who could deposit into the escrow account for the indexer. This may create further problems/confusion when different gateways re-issue (or issue more than once) RAVs against one allocationID.  In (a), by allowing the indexer to drain however much is available is a better protection than reverting. In (b), there isn’t much the indexer can do at this stage. This argument would favour (#2) and (#3) over (#1). However the outcome of (#2) and (#3) may be the same.
On (#2): Due to the thawing=>withdraw mechanism, it seems that one can check off-chain the escrow balance and send a redeem transaction based on that information. However it is potentially possible this balance may change in that period of time. For instance, a rogue sender could start thawing the balance earlier and leave it there until there is a redeem transaction from the indexer account in the mempool. Then the sender could put a withdraw transaction right before the redeem. Thus the sender can trick indexer to redeem for nothing, in theory if timed well. This argument then favours (#3) over (#2). At least for (#3), there is a chance to retry.
On (#3): One argument against (#3) is that of the separation of concerns, i.e. the min_amount needs to interact with Escrow.sol and this variable is not useful to keep in AllocationIDTracker.sol. Indeed this interaction must happen to validate if the transaction needs to be reverted. That could involve an external call from the allocationIDTracker to the Escrow contract. But it can be done elegantly like what’s already there in verifyProof using msg.sender to represent the Escrow contract.  It seems one does not really need to keep min_amount in the storage but merely use it in verifyProof, unless it could be useful later (idk, for refund? just thinking out loud).
So the preference here is (#3) despite that the outcome to the indexer may be the same under the unfortunate scenario of a rogue sender. By whitelisting senders, the protocol has already reduced the chance of rogue senders.


Module 3 exercises [Amanda Selasie Doagbodzie]

Please "drag & drop" exercise 3 file
WS2_M3E1_Template.docx
below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 3 and make sure to take
the post-training survey.

Well done, Amanda. You clearly grasped the material outlined in Module 3 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


Add exception class to raise

An exception class was missing here so I added a ValueError, otherwise we get a "TypeError: exceptions must derive from BaseException".

Looks good, thanks Ben!


Only first character of suffix is modified

While trying to run different configurations for the run_gcg_individual.sh launch script, I can see that only the first character of the suffix is being modified. See screenshot below for an example.

This is happening with the following configurations:
bash run_gcg_individual.sh vicuna strings
bash run_gcg_individual.sh vicuna behaviors
bash run_gcg_individual.sh llama2 strings
bash run_gcg_individual.sh llama2 behaviors
I have tried Python 3.8 and 3.10, both with the most recent version of torch.
I got model weights from hugging via the default method, e.g. git clone git@hf.co:lmsys/vicuna-7b-v1.3
I am using a GPU with 24 GB of memory, which prompted me to reduce the batch size to 256. I doubt this is related but worth mentioning.
I tried this with low_cpu_mem_usage set to true and set to false for every launch script configuration but it didn't seem to make a difference in suffix optimization.
I have found that in the GCGMultiPromptAttack.step function, when the prompt is sent to the model worker and the gradient is returned as new_grad, the resulting gradient is always a tensor of zeroes. A few lines later, new_grad = new_grad / new_grad.norm(dim=-1, keepdim=True) causes new_grad to be all NaN. I suspect this has something to do with the fact that no meaningful changes are made to the suffix, but I think its a symptom and I don't know the cause.

I think I've narrowed this down to some side effect of multiprocessing. When a model is instantiated while a ModelWorker is being instantiated, the model's embed weights are nonzero (self.model.model.embed_tokens.weight produces an interesting tensor instead of just zeroes).
After that point, embed weights for a ModelWorker's model are all zero (get_embedding_matrix(worker.model) or worker.model.model.embed_tokens.weights)
Any ideas what is going wrong?

Sorry for the late reply. We have never seen this issue before. It looks like you have an zero embedding matrix so that gradients can not propagate to the token ids.
@andyzoujm do you know what might be going on here?

Hm strange. The only suggestion I have is to verify the model and fschat versions. Often errors come from the fact that the slices in this function aren't corrected defined so the code is taking the loss over some fixed tokens.

Have you solved the same problem