Cannot upgrade to 2.x: off-sync package.json and package-lock.json

Stems from an nx problem: https://github.com/nrwl/nx/issues/17653
Happens anytime I deploy functions (functions build ok).
Any known workarounds? Or just need to wait for upstream fix before upgrading to 2.x?

@fedenusy Are you seeing this problem when running npm ci on the workspace ? or something else?

Also, are you using v1 or v2 cloud functions?

@simondotm npm ci works fine, as does npx nx build firebase-functions. The problem comes up when I run npx nx deploy firebase-functions.
I am not modifying dist/ in any way, just letting the plugin do its thing.
Using v2 cloud functions.

Ok - some things that would be helpful to see what the problem could be:

Just to confirm - you are migrating a workspace from v1.x of the nx-firebase plugin to v2.x?
What is your nx version?
Could you share the firebase deploy errors that you are seeing?

Thanks

Correct, migrating from v1.x of the plugin to v2.x. Doing so by generating a new firebase v2 application and shifting a single function into the new app. (Also migrating that function from cloud functions v1 to cloud functions v2 in the process.)
Using nx 16.10.0.
Logs:
npx nx deploy firebase-functions                                                                                ✘ 1 f/cloud-functions-v2 ✱

   ✔    6/6 dependent project tasks succeeded [5 read from cache]

   Hint: you can run the command with --verbose to see the full dependent project outputs

 ————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

> nx run firebase-functions:build  [local cache]

> nx run firebase-functions:deploy

 >  NX   Running target deploy for project firebase and 9 tasks it depends on:

> nx run email:handlebars-precompile
> nx run firebase:build  [local cache]
Build succeeded.
> nx run firebase:upload-sentry-sourcemaps
Done
> nx run firebase:deploy --only functions:firebase-functions
> nx run firebase:firebase deploy --only functions:firebase-functions
=== Deploying to 'my-firebase-project'...
i  deploying functions
i  functions: preparing codebase firebase-functions for deployment
i  functions: ensuring required API cloudfunctions.googleapis.com is enabled...
i  functions: ensuring required API cloudbuild.googleapis.com is enabled...
i  artifactregistry: ensuring required API artifactregistry.googleapis.com is enabled...
✔  artifactregistry: required API artifactregistry.googleapis.com is enabled
✔  functions: required API cloudfunctions.googleapis.com is enabled
✔  functions: required API cloudbuild.googleapis.com is enabled
i  functions: Loading and analyzing source code for codebase firebase-functions to determine what to deploy
Serving at port 8206
shutdown requested via /__/quitquitquit
i  functions: Loaded environment variables from .env, .env.my-firebase-project.
i  functions: preparing dist/packages/firebase-functions directory for uploading...
i  functions: packaged /Users/fedenusy/dev/farmpro/dist/packages/firebase-functions (163.71 KB) for uploading
i  functions: ensuring required API run.googleapis.com is enabled...
i  functions: ensuring required API eventarc.googleapis.com is enabled...
i  functions: ensuring required API pubsub.googleapis.com is enabled...
i  functions: ensuring required API storage.googleapis.com is enabled...
✔  functions: required API eventarc.googleapis.com is enabled
✔  functions: required API run.googleapis.com is enabled
✔  functions: required API pubsub.googleapis.com is enabled
✔  functions: required API storage.googleapis.com is enabled
i  functions: generating the service identity for pubsub.googleapis.com...
i  functions: generating the service identity for eventarc.googleapis.com...
✔  functions: dist/packages/firebase-functions folder uploaded successfully
i  functions: creating Node.js 18 (2nd Gen) function firebase-functions:myTrigger(us-central1)...
Build failed with status: FAILURE and message: npm ERR! code EUSAGE
npm ERR!
npm ERR! `npm ci` can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with `npm install` before continuing.
npm ERR!
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Missing<EMAIL_ADDRESS>from lock file
npm ERR! Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>npm ERR! Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>npm ERR!
npm ERR! Clean install a project
npm ERR!
npm ERR! Usage:
npm ERR! npm ci
npm ERR!
npm ERR! Options:
npm ERR! [--install-strategy <hoisted|nested|shallow|linked>] [--legacy-bundling]
npm ERR! [--global-style] [--omit <dev|optional|peer> [--omit <dev|optional|peer> ...]]
npm ERR! [--strict-peer-deps] [--foreground-scripts] [--ignore-scripts] [--no-audit]
npm ERR! [--no-bin-links] [--no-fund] [--dry-run]
npm ERR! [-w|--workspace <workspace-name> [-w|--workspace <workspace-name> ...]]
npm ERR! [-ws|--workspaces] [--include-workspace-root] [--install-links]
npm ERR!
npm ERR! aliases: clean-install, ic, install-clean, isntall-clean
npm ERR!
npm ERR! Run "npm help ci" for more info
npm ERR!  For more details see the logs at https://console.cloud.google.com/cloud-build/builds;region=us-central1/1ba8ffc7-9fb9-41e1-b66d-65868af6ef4c?project=195657483410.
Functions deploy had errors with the following functions:
	firebase-functions:myTrigger(us-central1)
i  functions: cleaning up build files...
Error: There was an error deploying functions
Warning: run-commands command "firebase --config=firebase.json deploy --only functions:firebase-functions" exited with non-zero status code

 >  NX   Running target firebase for project firebase failed
   Failed tasks:

   - firebase:firebase

   Hint: run the command with --verbose for more details.
Warning: run-commands command "nx run firebase:firebase deploy --only functions:firebase-functions" exited with non-zero status code

 >  NX   Running target deploy for project firebase and 9 tasks it depends on failed
   Failed tasks:

   - firebase:deploy

   Hint: run the command with --verbose for more details.
Warning: run-commands command "nx run firebase:deploy --only functions:firebase-functions" exited with non-zero status code

 ————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Ran target deploy for project firebase-functions and 7 task(s) they depend on (1m)

    ✖    1/8 failed
    ✔    7/8 succeeded [6 read from cache]

Ok thats great thanks. I understand the problem now and the relevance of the link you shared in your first post.
So the issue can be described as:

We have a firebase function that uses various dependencies from the workspace root package.json
When we build this function with @nx/esbuild, Nx emits a package.json to the dist dir listing these dependencies, but it does not generate a package-lock.json
When firebase deploy runs, it determines dependencies by running npm ci, however due to the missing package-lock.json, peer dependencies may vary from the workspace

I see that Nx have implemented generation of a pruned package-lock.json file when generatePackageJson build option is true, but it does not yet appear to be present for the @nx/esbuild plugin.
@fedenusy - could you try something for me?
in your functions project.json edit the deploy target from:
"deploy": {
      "executor": "nx:run-commands",
      "options": {
        "command": "nx run firebase:deploy --only functions:your-function"
      },
      "dependsOn": ["build"]
    }

to:
    "deploy": {
      "executor": "nx:run-commands",
      "options": {
        "commands": [
          "cp package-lock.json dist/apps/functions/package-lock.json",
          "nx run firebase:deploy --only functions:your-function"
        ]
      },
      "dependsOn": ["build"]
    }

The idea here is that by copying the root package-lock.json to the dist folder before firebase deploys it should use the same package versions for npm ci.

Also worth noting this isnt a migration v1.x -> v2.x issue. Its a v2.x / esbuild issue.

could you try something for me? ...

Thank you, the suggestion worked!

Also worth noting this isnt a migration v1.x -> v2.x issue. Its a v2.x / esbuild issue.

Agreed

Raised a new ticket for this: https://github.com/simondotm/nx-firebase/issues/161

Closing this issue, Nx 17+ seems to be generating package-lock.json files in dist correctly now.


Fix's implemeneted

This Code fixes all this issues that i have found to do with the SCA. SCA should now complete with a green tick

Hi @markgov - the most recent run still appears to have a number of failed checks in the Static Code Analysis step. Did you still have changes to make before this PR gets reviewed / approved?


Don't process multiple times

Description

Type of change

[ ] Docs change / dependency upgrade
[ ] Configuration / tooling changes
[ ] Refactoring
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires changes in customer code

High-level change(s) description - from the user's perspective

Related Issue(s)
Fixes
Related pull request(s)

@preethamr pls re-review there have been a few changes.

@rhlsthrm I put in a fix with:
https://github.com/connext/monorepo/commit/6dc0a118591223b8ad5c84f922f5222296a5195a
Pls merge if good.

@rhlsthrm I put in a fix with: 6dc0a11
Pls merge if good.

Ah wow! Thank you for this.


[Code health] Clean up SurveyConverter.toSurvey()

@gino-m Can you please add more details to this bug?

[@gino-m](https://github.com/gino-m) Can you please add more details to this bug?

Sorry I didn't provide more details when I wrote this, but I think @2939 likely addresses what I had in mind.


Add FastCGI backend support

You should be able to proxy to a FastCGI destination.
This destination could be over TCP, or a local socket.

+1


iOS: The app crashes after Deleting a Workspaces and going offline and coming back online

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to settings> Workspace
Create a workspace
Delete the workspace
Be offline
Notice the deleted workspaces appear offline
Be Online

Expected Result:
Deleted Workspaces don't appear offline and the App doesn't crash
Actual Result:
Deleted Workspaces appear offline and the App crashes
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:
Reproducible in staging?:  needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/49141ec4-5c68-46dd-b591-4f8c6c19a885
Expensify/Expensify Issue URL:
Issue reported by: @daveSeife
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689213503592969
View all open jobs on GitHub

I'm not able to reproduce. The workspace is deleted as expected and I don't see it reappear in the workspace list after I go offline.


Should CallHierarchyItemProvider have overloads with CallHierarchyItem

Testing https://github.com/microsoft/vscode/issues/81655
Currently CallHierarchyItemProvider has position based provider functions. If we have a function resolveCallHierarchyItem or prepareCallHierarchy shouldn't there be an overload for provideCallHierarchyIncomingCalls and provideCallHierarchyOutgoingCalls that take a CallHierarchyItem instead of a document and a position.

Unsure, today we use the uri/position of an item to make the next request. That helps us to prevent sending these items around and also aligns nicely with other language APIs. I feel like adding an overloads doesn't help but makes things more complicated. Either have uri/position or have a prepare-method and items as args but not both

I agree on the not both argument.

/duplicate of https://github.com/microsoft/vscode/issues/81753


add text to the article

[x ] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I do not think that this change is appropriate for the article.

We are marking this PR invalid for Hactoberfest as it does not follow the recommended guidelines below:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove white space / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


meta-lxatac-bsp: base-files: add fstab with explicit mount for /srv

We have previously relied on systemd-gpt-auto-generator to automatically generate a srv.mount unit based on the partition type of the data partiton. The data partition is created on first boot by systemd-repart.
This results in /srv not being mounted on first boot as systemd-generators run very early in the boot process, long before systemd-repart had a chance to create the partition.
Add a fstab (mostly the default fstab from poky) that includes an explicit /srv mount.

PR #42 is a cleaner attempt to solve the same issue.
I will close this PR.


No UI in certain Clusters

Describe the bug
I have one of the clusters that I don't get a UI for after connecting to it:

To Reproduce
Steps to reproduce the behavior:
Nobody can reproduce this
Expected behavior
The UI should load.
Screenshots
At the top.
Environment (please complete the following information):

Installed via deb on Ubuntu 21.10
Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
Your logs go here...

Kubeconfig:
N/A - It works for other team members with older Lens (5.0.x)

Duplicate of https://github.com/lensapp/lens/issues/4018


essage: invalid character '<' looking for beginning of value

Summarize the commit message use gpt-3.5-turbo model
We are trying to summarize a git diff
Error: error, status code: 403, message: invalid character '<' looking for beginning of value

类似的错误

@liangsj, @lqshow   openAI  Model?

@liangsj, @lqshow openAI Model?

gpt-4

@lqshow Show your config by codegpt conflig list

@lqshow Show your config by codegpt conflig list

@appleboy 有没什么我可以在本地尝试做的方向吗？

I tried the gpt-4, gpt-4-1106-preview and gpt-3.5-turbo versions and all worked.

I can't reproduce the problem. Please update the codegpt version to v0.7.0

I have the same issue. The problem may due to the custom openai.base_url.
@appleboy can you help with it?

Show your config

Key                       Value
git.diff_unified          5
git.exclude_list
git.template_file
git.template_string
openai.api_key            ****************
openai.api_version
openai.base_url           https://api.gptapi.us
openai.frequency_penalty  <nil>
openai.headers
openai.max_tokens         1000
openai.model              gpt-4
openai.model_name         gpt-3.5-turbo
openai.org_id
openai.presence_penalty   <nil>
openai.provider           openai
openai.proxy
openai.skip_verify        false
openai.socks
openai.temperature        1
openai.timeout            30s
openai.top_p              <nil>
output.lang               zh-cn

add v1 for your custom openai base_url


[Update] [JobMode.java] JobMode instance Serialized

Purpose of this pull request
To make JobMode instance Serializable.
Serialization is needed if you want to store enum files in a database and retrieve them later.
Does this PR introduce any user-facing change?
Yes

hi~ maven run : ./mvnw spotless:apply

Could you please elaborate?

Hey @Hisoka-X,
Why don't we mark sensitive fields as temporary private ones?

Hey @Hisoka-X, Why don't we mark sensitive fields as temporary private ones?

Which one? Could you share some detail?

Hey @Hisoka-X, Why don't we mark sensitive fields as temporary private ones?

Which one? Could you share some detail?

I'm wondering if it would be able to mark the sensitive fields as transient in explicit files related to this JobMode.java file.

Hey @Hisoka-X, Why don't we mark sensitive fields as temporary private ones?

Which one? Could you share some detail?

I'm wondering if it would be able to mark the sensitive fields as transient in explicit files related to this JobMode.java file.

Sorry, I don't get it. Could you provide some demo for this? Thanks.

public enum JobMode implements Serializable {
BATCH,
STREAMING;
// Private and transient field to store sensitive information
private transient String sensitiveField = "This is sensitive information";

// Public method to access the sensitive field
public String getSensitiveField() {
    return sensitiveField;
}

}
Above code is just an example to make sensitive field as "transient".
So similarly, Is it possible to make sensitive field as "transient" in other files of this repository which are associated with JobMode.java.

Hey @Hisoka-X, Why don't we mark sensitive fields as temporary private ones?

Which one? Could you share some detail?

I'm wondering if it would be able to mark the sensitive fields as transient in explicit files related to this JobMode.java file.

Sorry, I don't get it. Could you provide some demo for this? Thanks.

public enum JobMode implements Serializable {
     BATCH,
     STREAMING;

  // Private and transient field to store sensitive information
     private transient String sensitiveField = "This is sensitive information";

  // Public method to access the sensitive field
     public String getSensitiveField() {
          return sensitiveField;
     }
}

Above code is just an example to make sensitive field as "transient".
So similarly, Is it possible to make sensitive field as "transient" in other files of this repository which are associated with JobMode.java.

But JobMode just a simple enum. Does not need any sensitive field

But JobMode just a simple enum. Does not need any sensitive field

So I asked is it possible to make sensitive field as "transient" in other files of this repository which are "associated" with JobMode.java.

But JobMode just a simple enum. Does not need any sensitive field

So I asked is it possible to make sensitive field as "transient" in other files of this repository which are "associated" with JobMode.java.

Just keep it as now.

But JobMode just a simple enum. Does not need any sensitive field

So I asked is it possible to make sensitive field as "transient" in other files of this repository which are "associated" with JobMode.java.

Just keep it as now.

Yah, I understand.


Dependency: pyxpcconnection

Reading https://pypi.python.org/simple/pyxpcconnection/
Couldn't find index page for 'pyxpcconnection' (maybe misspelled?)

I get this error even when I've installed pyxpcconnection manually, through python setup.py install. What do you think is my problem?

I've "resolved" the issue. Might not be very elegant, but hopefully works just without breaking anything.
I installed pygattosx and pyxpcconnection libraries independently. In pygattosx's setup.py, I've removed the dependency (pyxpcconnection) because I know that I've installed it manually.
With this, the installation was successful. The discovery works fine as well, but connect fails with
>>> d[0].connect()
No scheduled event for kCBMsgId53
No scheduled event for kCBMsgId81
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "bleep/device.py", line 172, in connect
    serv = GATTService(self, uuid, start, end)
  File "bleep/gatt/service.py", line 55, in __init__
    self._characteristics = self._discover_characteristics()
  File "bleep/gatt/service.py", line 78, in _discover_characteristics
    characteristic = GATTCharacteristic(self.device, handle, value_handle, end_handle, BLEUUID(uuid), properties)
  File "bleep/gatt/characteristic.py", line 147, in __init__
    self._descriptors = self._discover_descriptors()
  File "bleep/gatt/characteristic.py", line 160, in _discover_descriptors
    discovered = self.device.requester.discover_descriptors(self.value_handle + 1, self.end_handle)
TypeError: discover_descriptors() takes exactly 2 arguments (3 given)

I'm guessing this has nothing to do with the jugaad that I've done! :P


[OTP-21] Deprecation of erlang:get_stacktrace() call

Hi, the new Erlang 21 is deprecating erlang:get_stacktrace() in order to replace with a new feature to retrieve the stacktrace only in the catch clause. Currently I can see this call being used in xdb_lib.erl.
For now it is possible to silence the warning and make the project compilable. Another option would be to pass exception across all the calls between a catch and xdb_lib:raise, or overall exception strategy should be changed.
What do you think should be the best way to go about it?

I talked to @cabol , we agreed on replacing the erlang:raise/3 calls to error/1


a-collada-model - model does not appear

I'm trying to run the "Anime UI" example from aframe on aframe-react, but I'm unable to load engine.dae. It just does not appear.
My Scene contains the following code:
<a-assets>
  <a-asset-item id="engine" src="models/engine.dae" />
</a-assets>

<Entity primitive="a-entity" position="1.75 0 1.2" rotation="0 28 0">
  <Entity primitive="a-camera" near="0.1" user-height="0" />
</Entity>

<Entity primitive="a-entity" position="0 0 -3">
  <Entity primitive="a-collada-model" src="#engine" rotation="90 0 0" scale="18 18 18" />
</Entity>

Can anyone please tell me what I'm doing wrong and why the model is not appearing.
Thanks!

Does <Entity collada-model={{src: '#engine'}} rotation="90 0 0" scale="18 18 18"/> work?

Sorry still did not work.
This is my project structure right now:

This is what a-collada-model looks like when I inspect it:

And when I run the example on aframe I get this:

Does the model load OK? 200, 404?

Yes the model seems to load OK, 200.
Although, the model appears to load twice for some reason.
This is the log and network tab when page loads:

Hmm, can you inspect what the engine.dae response is? That nodes parse error seems to be it, makes it sound like it got something weird in return,

Yes, I think you are absolutely correct.
The response does not look right.
This is the response:

Any idea why this might be?

Hm, maybe something to do with the dev server? If you open the file, it looks correct, right?

Ye, the file looks okay and comes up as expected in the preview window. I copied it over directly from the aframe example.

By the way I'm using the aframe-react-boilerplate for this.
I simply did npm i and then npm start.
Did I have to configure anything else in the boilerplate for this example to work?

Try moving the model folder relative to the index.html file?

Thanks for pointing me in the right direction. Turns out I just had to put the models folder into public folder.

This post just save me. Had the same issue.


Add feed button never enables itself

I also noticed that auto capitalization and auto correct is on for iOS when trying to enter a URL.

what is the url you are using? I've not been able to reproduce either with typed entries or urls from pasteboard

Something without http:, https:, or feed:?

Yeah.  See in slack.  We don't require the scheme when adding a URL to the Add Feed screen.


add gdb

Add GDB to the list of packages to be installed.

whoops. Apparently, this pull request also adds libcurl-dev now. :-(


Out of date . The templates are from version 2015 , we have 2019,2021 in place. Powershell and CLI are pertaining to 2015

[Enter feedback here]

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

ID: 1f2bd9d0-af11-d684-051b-7ac8e611cca9
Version Independent ID: f83d6e85-4d52-6b7f-a79e-f3ed3f3d0a95
Content: Create a newer metric alert with Azure Resource Manager template - Azure Monitor
Content Source: articles/azure-monitor/alerts/alerts-metric-create-templates.md
Service: azure-monitor
Sub-service: alerts
GitHub Login: @harelbr
Microsoft Alias: harelbr

@sukirti88 Thanks for your feedback. We will review the issue and get back to you shortly.

Thanks @sukirti88 for the feedback. I just issued an update to the schema version in this article, bumping the version to 2019-04-01.

Thanks @harelbr for taking care of this issue.
@sukirti88 As there are no further documentation changes, we will close the issue for now. If there are further questions, please revert and we will be glad to assist you.


adapt to setUpdateListener promise

see https://github.com/deltachat/deltachat-core-rust/issues/3203

nice addition, also works as expected :)


add documentation for neovim

Thanks for the pull request @WhoIsSethDaniel ! There's an in-person hackathon tomorrow in Houston: https://github.com/perlconference/tprc-2022-hou/wiki/Hackathons . A great task for someone would be reviewing this pull request by following the documentation. In addition to getting this reviewed, that person will also have Perl Navigator installed on their machine.


Snapshot images as a different files

Snapshot images as a different files.
Hi is the way to get separate images, actual, expected, diff  - instead of one .diff image which contains all of this three images?
So I want to have base image .snap and  diff image as a separate file (only with diffs), and image which was made during test "actual".
Now each of this 3 images(snapshots) are combined into one image

This plugin is just a wrapper around jest-image-snapshot. You could take a look at the options available there:
https://github.com/americanexpress/jest-image-snapshot/tree/main?tab=readme-ov-file#-api
This looks the closet to what you are asking at first glance:

onlyDiff: (default: false) Either only include the difference between the baseline and the received image in the diff image, or include the 3 images (following the direction set by diffDirection).


Fix path and filenames for the sap managed fiori sample

Minor config and readme change to fix the mtar filename and directory path for the sap managed fiori sample.

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

Good catch, thanks for your contribution!


Fix typo in F.convolution_2d

Thanks, LGTM!

Jenkins, test this please.

Jenkins CI test (for commit bee47ae8ecac25861131dd9661b89f92216c2921, target branch master) failed with status FAILURE.

LGTM!


Simplify has-role

The function could be simplified to

;; Checks if an account has the specified role
(define-read-only (has-role (role-to-check uint) (principal-to-check principal))
    (default-to false (get allowed (map-get? roles {role: role-to-check, account: principal-to-check}))))

Thanks for the feedback.  This was incorporated.


When are you planning to run pytorch support

When are you planning to run pytorch support
How long will it take?)

Congratulations! It is a strong contribution.
I think everyone will appreciate a Pytorch version of the code. There are any plans for that?

Hi, Thanks.
A PyTorch implementation of LTCs is included in our keras-ncp project, here:
https://github.com/mlech26l/keras-ncp


Creating an action didn't set it as the selected action in the sayings input

Probably related in some way to #607

@wrathagom The issue with this is that every tag of keyword or action requires a full re render of the all syings being shown in the page. If you change the page size to 5, the response time is much less. Also the number of keywords tagged on each saying affects the result.
I will try to improve this by only re rendering the saying that was updated by the user. I will let you know how this goes.

@dcalvom did you post that on wrong issue?

Teo! Removing...

closed by https://github.com/samtecspg/articulate/commit/098a80771119e9f0d868b5f15364575342ec5a15


docs(integrations): integrations section

[ ] Move Help to End User Guides
[ ] Move Integration Guides to integration
[ ] Terraform + Import/Export zusammen?
[ ] Python is more an example for accessing the ZITADEL API
[ ] Overall introduction
[ ] Develop > Examples
[ ] Develop > SDKs
[ ] ZITADEL APIs > Overview

Remove boxes, since constantly outdated

[ ] Remove Boxes from Start
[ ] Remove Boxes from Concepts
[ ] Remove Boxes from Legal

@mffap what is the state on this?

replaced by #5271

replaced by https://github.com/zitadel/zitadel/pull/5318


i18n support

Hello, is there any way you could add i18n to the error/info messages that are hardcoded into the lib? For example having a different language per server the same way prefixes work.
Thank you :)

i18n is something I've actually been planning to do, but so far I didn't find time to implement it. I hopefully get managed to add it to the next release. For the moment you can already change the embeds, and thus the language, by modifying the embed factory. https://github.com/Kaktushose/jda-commands/wiki/Advanced-Configuration

Lovely, I'll look into it, get my star sir

i18n is now implemented. I'll write the wiki for it within the next days.  let me know if you have any questions :)


IntentService runs outside application process ?

Hi,
Thanks for sharing beautiful knowledge of Android on your blog.
I have a query under Section "Services"-->"Starting Background Service" :
https://guides.codepath.com/android/Starting-Background-Services
Here, its mentioned that "IntentService runs outside the application in a background process,.....",
whereas according to the Android documentation, IntentService runs in a background thread outside main UI thread that is in the same application process.
We can see in the source code of IntentService.java, inside onCreate() callback, HandlerThread is being initiated and started a new thread.
Kindly, correct me if i am wrong.
Thank You.

+1
Good pick-up, the wording should be updated in this article to reflect the official api documentation.
My interpretation of what the guide is saying is that it executes on a non-UI thread, in the context of the application, so that even though the user may be interacting with the application UI, the service itself isn't executed on the UI. It could be triggered by the UI, but runs separately, such as alarm managers, data services, push notifications, wake ups etc.


Optimize range encoding

Ranges are currently encoded with a custom format, using 51 bits for the base and 33 bits for the delta. This is more compact than using a u64 for both, but still might not be optimal.
Ideas for things to try:

Varints. Many base and delta values are small, so they might compress will using varints. Can we exploit the fact we only need 51 + 33 bits?
Delta encoding. If ordinals bases and offsets tend to be close, maybe we could delta encode them. It is highly speculative that this would work.
Compress ranges with a general purpose compression algorithm, like deflate

This is blocked on #456.

Looks like compressing the existing packed compression get's around 15% savings on a fully synced database:
raw:<PHONE_NUMBER>8
packed:<PHONE_NUMBER>8
compression1:<PHONE_NUMBER>3
compression6:<PHONE_NUMBER>2
compression9:<PHONE_NUMBER>4
compression10:<PHONE_NUMBER>9
packed_compression1:<PHONE_NUMBER>3
packed_compression6:<PHONE_NUMBER>4
packed_compression9:<PHONE_NUMBER>0
packed_compression10:<PHONE_NUMBER>8
vint64:<PHONE_NUMBER>3
bitcoin_varint:<PHONE_NUMBER>6
packed_start_vint64_end:<PHONE_NUMBER>6

Unclear if it's worth it, would require further benchmarking, and will increase CPU usage.

Could also consider changing packed encoding from base, offset, base, offset… to base, base, …, offset, offset… which might be more compressible.

Eh. Let's do this as a last resort.


bump to v0.5.0

[ ] This patch includes any updated documentation of the proposed changes
[ ] This patch includes tests that verify the proposed changes work as expected

Codecov Report

Merging #1026 into master will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1026      +/-   ##
==========================================
+ Coverage   77.06%   77.15%   +0.08%
==========================================
  Files          52       52
  Lines        5616     5616
==========================================
+ Hits         4328     4333       +5
+ Misses       1288     1283       -5

Impacted Files
Coverage Δ

setup.py
0% <ø> (ø)
:arrow_up:

vent/api/plugins.py
79.13% <0%> (+0.86%)
:arrow_up:

Continue to review full report at Codecov.

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


Support force: section to force all steps to run.

Add an option to run to the last step even if the intermediate step fails.
If even one step fails, it is considered a failure as a runbook.

Hi @k2tzumi.
What do you think of force: as a section name?


moloch parse http filename to be a unreadable code

when I use moloch parse http that filename or url contains of chinese, filename or url to be unreadable code

Can you attach a sample pcap or DM me one on slack.

Can you attach a sample pcap or DM me one on slack.

here is the pcap and download file that I test, you can see the filename/uri is unreadable.
and from
https://github.com/aol/moloch/issues/1037#issue-426368376
package.zip
I know moloch can not parse filename from content-disposition in multipart header

Do you have any pcap with a full tcp session (SYN/SYN-ACK)?

Do you have any pcap with a full tcp session (SYN/SYN-ACK)?

sure, here is the pcap,and the filename is nmap扫描.txt
downloadFile.zip

So are you talking about the url display?
<IP_ADDRESS>/nmap%E6%89%AB%E6%8F%8F.txt
We display what is on the wire.  Are you suggesting that we decode this?  I don't think we would change the current display, but maybe we could add a decoded display also?  What is the use case?

So are you talking about the url display?
<IP_ADDRESS>/nmap%E6%89%AB%E6%8F%8F.txt
We display what is on the wire. Are you suggesting that we decode this? I don't think we would change the current display, but maybe we could add a decoded display also? What is the use case?

For a user, he enters a URL in his browser, and the browser sends it as a URLEncode to the network, and moloch parses it out and of course parses it into the one that the user understands, which is the URL that he enters,  not the URL that the browser does the URLEncode for

It is not clear to me how to know we should decode the item and display it in the decoded format.  It could also just be random data in the url.
We might be able to do the reverse, where if you say http.uri.path == nmap扫描.txt we could encode it.

It is not clear to me how to know we should decode the item and display it in the decoded format. It could also just be random data in the url.
We might be able to do the reverse, where if you say http.uri.path == nmap扫描.txt we could encode it.

yes,it's also a solution


BUG: differentials dataframe needs valid index name to load as metadata

otherwise the q2 multimodal action raises an error on completion because it is failing to load differentials as metadata.

Hi @nbokulich thanks for reminding me about this.  There is another PR under #64 . Is this the problem that you are referring to?

Hi @mortonjt sorry I pushed without seeing that PR, was running into this bug with the current release of songbird in qiime 2 version 2019.4 + 2019.7. It does not look like #64 is related to this bug; however it looks like #60 solves this problem already (once it is merged). Please feel free to close this PR.

oh, I meant #60.  That one should be ready to merge.


Make package usable as a library

In order to leave the main package available as a command, I created a subpackage figletlib to house the main machinery of this package. I also added print.go to handle easily printing, particularly to non-stdout streams (eg. http ResponseWriter).
I moved most of the files there unchanged, but had to export some types and getters by making the names uppercase.
The biggest change is probably in new file fonts_loader.go, which may or may not do as good a job as the original code at finding the fonts directory. I added the -d option to the command line version so you can specify wherever the fonts dir lives rather than relying on this being correct.
Let me know what you think / what needs fixing / etc.; happy to make additional changes.

Wow, that looks great! It's been a while since I looked at this code. Thanks for giving it some attention.


Stunprod

About the PR

Improved stun prod for PR #17467.
The stun prod is now more useful.
Why / Balance

The stun prod is not used by players in the game (Corvax).
A stun prod is different from a stun baton. It is useless to throw a stun prod at the enemy.
Uses up a lot of space in your backpack (Huge).
Uncomfortable sound of blows.
Syndicate agents and revolutionaries will use this weapon more often. It will be easier for the agent to kidnap his target.
Technical details

Code copied from the stun baton.
Resources/Prototypes/Entities/Objects/Weapons/security.yml
Media

https://github.com/space-wizards/space-station-14/assets/154531074/ced60c5c-614d-42b0-820f-456c0203a37d

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

The stunprod.yml prototype file has been changed.
Changelog

:cl: IlyaElDunaev

add: Added StaminaDamageOnCollide for the stunprod. Stun prod can be thrown at enemies.

Your changelog should be changed to be more player-understandable. Just "The stun prod can now be thrown at enemies"  is probably fine.

Your changelog should be changed to be more player-understandable. Just "The stun prod can now be thrown at enemies" is probably fine.

Thanks :))


Hero Selection Box Never Dies (and covers stuff up)

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Clicking on a Hero web part link shows a highlighted border to indicate selection. Clicking anywhere else should remove this border. Even when the border is showing, it should be beneath the search suggestions drop down.
Observed Behavior
Clicking on a Hero web part link shows the highlighted border to indicate selection. However, it only goes away when you select another hero web part link.
Additionally, this highlighted border shows above the search suggestions dropdown and also the Feedback buttons on the bottom-right. This is ugly.

Seems like a z-index problem with the Hero web part and also an issue that the selection never goes away when the web part loses focus.
Steps to Reproduce

Create a communication site using the Topic design (anything with the hero webpart).
Click on a link.
Close the new window.
Notice the highlighted border.
Try and click anywhere else to have it go away.
Open the search suggestions and see that the highlighted border covers the drop down as well as the feedback buttons.
Shed a tear.

The main cause should be selection never reset after loses focus.

@Seiyasu that is the main problem, but the z-index is also an issue since it covers up the Feedback buttons (even if it went away once it lost focus).

As this is not a developer issue, we cannot route this properly using this process. This is clearly inconvenient UI issue, but it would need to get reported either using UserVoice or normal support channels.


Add simple example of polymorphism in C

Add last example to compare OOP with procedural programming.
I use "images/intro-polymorphism.png" to create the example.

Nice! Thank you


Add BufferVec, an higher-performance alternative to StorageBuffer, and make GpuArrayBuffer use it.

This is an adoption of #12670 plus some documentation fixes. See that PR for more details.

Changelog

Renamed BufferVec to RawBufferVec and added a new BufferVec type.

Migration Guide
BufferVec has been renamed to RawBufferVec and a new similar type has taken the BufferVec name.

Thank you to everyone involved with the authoring or reviewing of this PR! This work is relatively important and needs release notes! Head over to https://github.com/bevyengine/bevy-website/issues/1310 if you'd like to help out.


Remove inefficiencies created by introducing specific C++11

As SBE generates C++ code, it's doing more than it needs to do and in some cases, actually hurting performance.
First, SBE is generating a move constructor and move assignment operator for the generated class.  However, the implementation of these methods are the exact same as their copy counter parts (see note below).  Since T&& will be promoted to const T&, the copying methods are sufficient.
Next, the C++03 version of the class member template function forEach takes a single template parameter which is meant to be a functor.  The signature has changed in C++11 version to take in a std::function.  This is where the inefficiency is introduced.  The implementation of std::function uses type-erasure to achieve its functionality.  However, that means it's performing an allocation on the heap, copy the data of the functor, and invokes the functor indirectly which loses the ability to inline the method call.  By leaving it as a template parameter, the compiler has the opportunity to inline the invocation altogether and allows the compiler to inline the invocation.
NOTE:  We can take this a step further and declare that the copy constructor and copy assignment operator methods are not necessary since it's the exact same implementation that the compiler would generate for us anyway.

The copy constructor and copy assignment operator are required since the positionPtr isn't handled correctly by the compiler. So, we can't quite go that far.
The move constructor and assignment operator was from a specific (private) user request. Let me check with them to see if a copy instead is enough.

I think that having a move constructor and assignment operator is a good idea as long as it is implemented correctly.  In the current implementation, it simply copies the data members.  We should be setting the "right side" of the operation to a state in such a way that it was default constructed or some other empty but valid state.

@skizzay I would rather keep the move constructor/assignment and set the arg (move from) to a default constructor state (m_buffer nullptr, m_bufferLength 0, and m-offset 0). If you make that change, I'll merge.

@tmontgomery I apologize for the late response.  I've incorporated the changes you've suggested.  For moved objects, I call reset on the 'moved to' object with the 'moved from' object as its parameter.  I then call reset on the 'moved from' object with a default constructed object as its parameter.  This effectively zeroes out the data members of the 'moved from' object while populating the data members of the 'moved to' object.

No worries. Thanks for the PR!

These changes broke the build. Specifically, I think the constructors for fixed flyweights aren't being handled appropriately.


Network issues in Drupal VM after installation of Docker & WSL 2

Issue Type
Support Request
Your Environment

Vagrant 2.2.9
VirtualBox 6.1

Your OS
Windows 10 Pro
Full console output
Checksums do not match, Checksum check failed!
error Received malformed response from registry for ...
Summary
I am using Drupal VM for many years.
2 weeks ago I installed Docker & WSL 2 on my Windows 10 (to use it independent from Drupal VM).
Since then I have network issues within my Drupal VM (Ubuntu) still when Docker and WSL 2 have not been started or have been stopped.
Now I can not install Ubuntu packages anymore. I get errors like
"error Received malformed response from registry for ..."
I can not run 'nvm install node' , can not run 'yarn install' .... always when packages are downloaded the files/repos are not downloaded correctly (checsum error) or the downloads break with error like "Received malformed response from registry ..".
I think there is an ip conflict or something similar between Docker/WSL2 and Drupal VM.
Something affects the Drupal VM ... I can not work within VM anymore.
How can I solve this?
I want to have installed Docker/WSL2 and Drupal VM on my Windows 10 without affecting each another.
How can I find out what the problem is?
Has Hyper-V to be installed still if it is not used by Docker/WSL2?
How can I find out what is causing the problem? Need solution.
Thank you very much.

I could solve this problem by updating Virtualbox to the latest version (6.1.30).


fix(color-palette-options): add warm red and ability to omit palettes…

Add the Warm Red palette to the global defs.
Convert the defs to an object so we can create component specific subset palettes (e.g. remove warm red option).
Add utility method to remove option(s).

:tada: This PR is included in version 1.0.0-beta.13 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


IDEA: Masking to control stitchlines

I might have completely missed it, but is there a way to control the stitchlines by masking? That would be an essential feature to get rid of ghosting and optimize the stitch.
Importing masks from PTGui would, of course, also be awesome, just don't know if it is possible at all.

There is no UI for masking or editing the control points for calibration directly in VideoStitch Studio. Fine-tuning like that should be done in PTGui.
Masks that mask out parts of the image, like a tripod (negative/red masks?) done in PTGui should apply properly when importing the calibration in VideoStitch Studio. Other types of masks are not supported.

Can confirm red masks work - makes it possible to create masks for a whole scene, just need to make sure to not rely on green masks.
I'll close this & make sure to implement this information in the documentation.


Add self-referencing canonical to language routes

Background:
As mentioned in #915 , I am working on making i18n module work with i18n rewrites. Basically, this allows to serve user content from CDN in their language, without a redirect.
The functional part is resolved, but there is an issue with SEO, which doesn't do what is desired in URL rewrite case.
Currently, with prefix_and_default strategy and seo enabled, the rel=canonical tag will be added on default language routes. This means, that when default language is "en",
/ <- this one is canonical
/en <- this one will have link rel=canonical href="/"

This is great and exactly how it should work without rewrites. With rewrites, however, the desired behavior would be reversed: the canonical tag should point to language route, not the default:
/ <-default route never canonical, but have link rel=canonical href="/{{defaultLocale}}"
/en <- should have self-referencing link rel=canonical href="/en"
/fr <- should have self-referencing link rel=canonical href="/fr"

Additionally, the default route should have hreflang="x-default", signaling that no other language/region matches the user's browser setting.
With this setup, we can expect the crawler to scan all language routes, but not the default routes, because the content of default routes depends users' browser settings and should not be indexed.
What is the best way to introduce such change? Any guidance would be appreciated.
Here is what I am thinking regarding config changes:
Current behavior (marks default route as canonical)
i18n: {
  seo: true
  // or
  seo: {
    defaultCanonical: true
  }
}

Desired behavior for i18n rewrites (marks language routes as canonical, plus sets hreflang="x-default" for default route)
i18n: {
  seo: {
    defaultCanonical: false
  }
}

default route should have hreflang="x-default"

Well, I think this is another feature request which is great if added.

With rewrites, however, the desired behavior would be reversed: the canonical tag should point to language route, not the default:

What you're looking for is a prefixed strategy, basically I've already created a feature request here
Regarding x-default there is a PR for that too.
Thanks!

I think you are right. Prefixed strategy with canonical links is what I need.

I think you are right. Prefixed strategy with canonical links is what I need.

Then this feature request might be closed.
Thanks!

x-default was added through https://github.com/nuxt-community/i18n-module/pull/922.
As for your case, it's a bit special since you are rewriting the URLs. That way of doing things feels like a hack to me, sorry. Due to that, I'm not really willing to dive into it and investigate if and how this should work.

It may be a special case because of the rewrite. Yet it would be nice if it was possible to choose, whether the default or the prefixed route is considered canonical. My understanding is that such a case is going to be covered in #912

Fixed for #912 are released in https://github.com/nuxt-community/i18n-module/releases/tag/v6.15.2 but not sure if it's not gonna fully help your case.

Hey @rchl and @divine. Thank you for your PRs! Huge improvement for SEO.
Forget my special case with URL rewrite for a minute. Here is what essentially I am trying to achieve:
<link rel="alternate" href="http://example.com/en" hreflang="en" />
<link rel="alternate" href="http://example.com/es" hreflang="es" />
<link rel="alternate" href="http://example.com/fr" hreflang="fr" />
<link rel="alternate" href="http://example.com/" hreflang="x-default" />

It's an example from Google Webmasters blog you have probably seen, except I am using es, en, and fr for language codes for simplicity.
It appears that there is no way to generate the above result.
With prefix_and_default strategy we get this:
<link rel="alternate" href="http://example.com/" hreflang="en" /> <!-- difference here -->
<link rel="alternate" href="http://example.com/es" hreflang="es" />
<link rel="alternate" href="http://example.com/fr" hreflang="fr" />
<link rel="alternate" href="http://example.com/" hreflang="x-default" />

With prefix strategy we get this (as expected):
<link rel="alternate" href="http://example.com/en" hreflang="en" />
<link rel="alternate" href="http://example.com/es" hreflang="es" />
<link rel="alternate" href="http://example.com/fr" hreflang="fr" />
<link rel="alternate" href="http://example.com/en" hreflang="x-default" />

And regarding to canonicals, I still can't make prefixed route canonical.
Here is what I would like to see:

/index.html

<link rel="canonical" href="http://example.com/en" />

/en/index.html

<link rel="canonical" href="http://example.com/en" />

@divine suggested that I need to use prefix strategy, which at first I thought was the right direction to go, but it turns out prefix strategy generates meaningless /index.html meant to redirect, which is not what I want.

It appears that there is no way to generate the above result.

Have you tried setting up defaultLocale: '' with prefixed strategy?

Here is what I would like to see:

/index.html

<link rel="canonical" href="http://example.com/en" />

/en/index.html

<link rel="canonical" href="http://example.com/en" />

Why? Doesn't make sense at all. You don't have content at root path and why it should be canonical?
I can't really understand what you're trying to achieve (do you have some weird errors in Google search console that your pages can't be indexed or any other SEO tool suggest to change that? Maybe some articles that suggest using root path as canonical?).
Let me remind, canonical used to mark page as unique, it doesn't do anything else. Making your root path canonical without content wouldn't help you at all.
Thanks!

Have you tried setting up defaultLocale: '' with prefixed strategy?

Just tried. It doesn't generate x-default link

Why? Doesn't make sense at all. You don't have content at root path and why it should be canonical?

Thanks for your response @divine. I think there is a bit of misunderstanding.
I am building a multi-language statically generated site, and I want to take advantage of i18n rewrites provided by Firebase Hosting. The rewrite allows me to host the website on CDN, yet provide content in user's language immediately without redirect.
For example, when opening root of my website, a spanish-speaking visitor will get content from /es, whereas a russian-speaking visitor will get content from /ru. A visitor from unsupported locale will get content from /. (root path has content in default locale)
This is why I am trying to avoid marking default path as canonical, and instead make language routes /en canonical, so that when the website is indexed, only the prefixed routes are respected.
Right now, with prefix_and_default strategy, the default route gets to be canonical. And with prefix strategy, root path doesn't get any content.

Just tried. It doesn't generate x-default link

Sorry, I forgot that you're using static generation, currently there is a bug https://github.com/nuxt-community/i18n-module/issues/911 that doesn't render anything which is solely used as redirection.

I am building a multi-language statically generated site, and I want to take advantage of i18n rewrites provided by Firebase Hosting. The rewrite allows me to host the website on CDN, yet provide content in user's language immediately without redirect.

Still doesn't make sense to me. What exactly does give i18n rewrites in terms of SEO? No redirection?
You're trying to make things more complicated. I've just read that documentation, giving content without redirection based on user IP-address. Basically I don't know what search engines bots might think about your site? 🤔

This is why I am trying to avoid marking default path as canonical, and instead make language routes /en canonical, so that when the website is indexed, only the prefixed routes are respected.

You're looking for a prefixed strategy again which is currently has a bug. Even after it will be fixed I don't know it wouldn't help you since it will redirect.
Sorry, I'm out from this, too much complexity for doing some weird i18n-rewrites (weird because this library already does i18n and you're on top of it trying to do another layer of i18n logic.)
You should check for sites hosted with firebase and see how they perform with i18n-rewrites in terms of SEO.

What exactly does give i18n rewrites in terms of SEO? No redirection?

You are correct, the purpose of url rewrite is to avoid redirection. It complicates things in terms of SEO, but doesn't make it impossible.

I would like to stick to serving a static site on CDN an avoid adding back-end language detection and redirect
I client-based redirect is bad in terms of user experience: the browser first downloads the document, then parses and executes all JS (Nuxt+i18n+content is more than 350KB), then redirects and does the parsing and execution of JS again. To experience this, try the prefix strategy with redirect slow 3G throttling.

Yet, without the URL rewrite, the not-so-good case of client-based redirect is the only option (if you want to serve content in client's language).
This issue, however, can be closed, as my original request was to add self-referencing canonical to prefixed routes, which is now possible with prefix strategy.
To resolve my particular case, I generate website with defaultLocale: '', and then cp -a ./dist/en/. ./dist. Not perfect, but it works.

To resolve my particular case, I generate website with defaultLocale: '', and then cp -a ./dist/en/. ./dist. Not perfect, and doesn't generate x-default, but it works.

 Then you can use your copy hack with that.

Also, I wonder if you have some more complex options when it comes to rewrites. Maybe it allows you to set the cookie in which case you could use the no_prefix strategy and control the language with a cookie).


Ignore vendor directory

The vendor directory is where the app dependencies are stored, after running composer install or composer update. It shouldn't be added to version control, since it is enough to have the composer.json file to resolve dependencies

Closing in favour of #1672


ActivationValue for RabbitMQ scaler

Signed-off-by: Alejandro Dominguez<EMAIL_ADDRESS>
Adding activation threshold (minMetricValue) for Rabbitmq.
Feature PR: #2831
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

I think we should have a generic section on activation threshold in the docs. Where we describe the behavior (current and the new). And here reference this doc. WDYT? @kedacore/keda-contributors

I like it, since, is going to be a generic concept for all scalers.

+1 to a specific section

I think we should have a generic section on activation threshold in the docs. Where we describe the behavior (current and the new). And here reference this doc. WDYT? @kedacore/keda-contributors

What do you think about add the section in a different PR/ticket. Since this is only affecting a single scaler. So we don't block this PR and also because not sure where this new section should be placed.

A new PR is fine, but this general section should be added before this feature lands.
@kedacore/keda-contributors ideas on the docs section?

Maybe we can explain it under Scaling Deployments, StatefulSets & Custom Resources due to this only applies to ScaledObjects.
Something like "Differences between activation and scaling"


Add email sent verification. Related #126.

The user will now be presented with a message stating whether their email has been successfully sent or not. If the email has sent the session storage will be cleared.
Fixes #126
Fixes #149

Codecov Report

Merging #150 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #150   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines           9      9
  Branches        4      4
=====================================
  Hits            9      9

Continue to review full report at Codecov.

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

Codecov Report

Merging #150 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #150   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines           9      9
  Branches        4      4
=====================================
  Hits            9      9

Continue to review full report at Codecov.

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

Thanks @des-des. I have made the changes.

@RhodesPeter wikid


New "unban" feature and bug fixes

Hello @rapejim! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/apidb.py:

Line 19:80: E501 line too long (80 > 79 characters)

In the file src/app.py:

Line 67:80: E501 line too long (92 > 79 characters)
Line 145:80: E501 line too long (84 > 79 characters)
Line 163:47: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 166:1: W293 blank line contains whitespace

In the file src/user.py:

Line 79:80: E501 line too long (85 > 79 characters)


Track global cx limit in overload manager

Deprecate runtime key overload.global_downstream_max_connections and track global active downstream connections limit in overload manager instead. The runtime key is still usable but using it yields deprecation warning. If both mechanisms are configured (overload resource monitor and runtime key), overload manager config will be preferred.
Commit Message:
Additional Description:
Risk Level: High (change affects request hot path)
Testing: Done
Docs Changes: Done
Release Notes: TBD
Platform Specific Features: NA
Fixes #12419
Deprecated: Added deprecation note for runtime key overload.global_downstream_max_connections

There are still few TODOs and tests left, but implementation and all the main tests are ready.  Since proactive resource monitors do not make use of overload actions (and entire overload manager mechanism of "subscribe for updates for this action"), we should either make it clear in user docs that action configuration is not needed for such monitor or introduce separate monitor type in api.

Thanks for review @KBaichoo, I will get back to it in 10 days after vacation.

one thing worth clarifying in the user docs is that this monitor is not integrated with OverloadActions, since ProactiveResource was introduced as parallel type in class hierarchy to Resource and it does not implement ResourceUpdateCallbacks.

Thanks for review @KBaichoo. I will add release notes once CI checks all pass. Current known left TODOs:

release notes
update user docs that is not possible to reconfigure monitor via runtime
small optimisation to not recompute boolean value for each instance of AcceptedSocketImpl

cc @phlax maybe you could help, mobile checks started failing for this PR, even though patch does not touch mobile code

Generally looks good, though some of the CI failures are real.
/wait

Apologies for delay, had to deal with a high priority task. Will wait if there are any other review comments left and then add the changelog.

ping @RyanTheOptimist
I think this is waiting for your input.

ping @RyanTheOptimist
I think this is waiting for your input.

Oh, gosh, sorry. I thought it was waiting on you. Sorry about that! Added my comments.

/wait

@RyanTheOptimist addressed review comments+added changelog, did not add assert statement yet, waiting for input.

Looks like this needs a main merge.
/wait

/retest

@RyanTheOptimist looks like all the checks are passing now


swagger.json cannot be loaded behind reverse proxy with rewrite

I have an application deployed behind a reverse proxy (ingress-nginx) with path rewriting. Swagger's index.html is loaded successfully but then it fails to fetch swagger.json.
https://sample-proxy.com/service-a/ is rewrited to / so the backend app sees / as a base request path.
Now when I request https://sample-proxy.com/service-a/swagger the index.html is fetched successfully but there is a problem with swagger.json. When I look at the index.html content I can see that url for the swagger.json is /swagger/api/swagger.json but it tries to fetch it using https://sample-proxy.com/swagger/api/swagger.json and not https://sample-proxy.com/service-a/swagger/api/swagger.json.
I can manually fetch swagger.json using full address https://sample-proxy.com/service-a/swagger/api/swagger.json.
ASP.NET Core setup:
app.UseForwardedHeaders();
app.UseSwagger();
app.UseSwaggerUI(c =>
{
    var name = GetType().Assembly.GetName().Name;
    c.SwaggerEndpoint($"/swagger/{name}/swagger.json", name);
});

<PackageReference Include="Swashbuckle.AspNetCore" Version="5.6.3" />
Forwarded headers:
[X-Forwarded-For] = <ip-address>
[X-Forwarded-Proto] = https
[X-Forwarded-Host] = sample-proxy.com
[X-Forwarded-Port] = 443
[X-Forwarded-Prefix] = /service-a

Nevermind, just found out that I should use relative path.
c.SwaggerEndpoint($"{name}/swagger.json", name);

I do not understand you add the name of the assembly. For me that did not work at all.


Problem with prometheus function : "rate"

Hello World,
I use vmware exporter to generate dashboard for billing for our clients. In my enterprise we have to charge VMs to the number of vCPU, number of Memory and size of hard disk drives.
But, I have a problem.
I want to use "rate function of prometheus" with this metric for example : vmware_vm_num_cpu, and result is always : 0
My use case :
vmware_vm_num_cpu{vm_name="DVOTEST0000"}

vmware_vm_num_cpu{cluster_name="Cpt-Cluster", dc_name="DevopsCannes", host_name="cadvovmwp54.aes.alcatel.fr", instance="vmware_exporter:9272", job="vmware_exporter", vm_name="DVOTEST0000"}	8

With rate function :
rate(vmware_vm_num_cpu{vm_name="DVOTEST0014"}[5m])

{cluster_name="Cpt-Cluster", dc_name="DevopsCannes", host_name="cadvovmwp54.aes.alcatel.fr", instance="vmware_exporter:9272", job="vmware_exporter", vm_name="DVOTEST0000"}	0

But i have time serie metric ...
vmware_vm_num_cpu{vm_name="DVOTEST0014"}[5m]

vmware_vm_num_cpu{cluster_name="Cpt-Cluster", dc_name="DevopsCannes", host_name="cadvovmwp55.aes.alcatel.fr", instance="vmware_exporter:9272", job="vmware_exporter", vm_name="DVOTEST0014"}	1 @1583144008.979
1 @1583144023.979
1 @1583144038.979
1 @1583144053.979
1 @1583144068.979
1 @1583144083.979
1 @1583144098.979
1 @1583144113.979
1 @1583144128.979
1 @1583144143.979
1 @1583144158.979
1 @1583144173.979
1 @1583144188.979
1 @1583144203.979
1 @1583144218.979
1 @1583144233.98
1 @1583144248.979
1 @1583144263.979
1 @1583144278.979
1 @1583144293.979

I don't understand why I can't use "rate" function. But when I use "avg_over_time" function, it's works! But I don't want to use this function :)
avg_over_time(vmware_vm_num_cpu{vm_name="DVOTEST00000}[5m])

{cluster_name="Cpt-Cluster", dc_name="DevopsCannes", host_name="cadvovmwp54.aes.alcatel.fr", instance="vmware_exporter:9272", job="vmware_exporter", vm_name="DVOTEST00000"}	8

Could someone help me understand why it is not working and how to make it work?
Regard,

vmware_vm_num_cpu is a gauge and indicates that amount of vCPUs allocated to a VM, this value doesn't change over the runtime of the VM and you can't use a rate() function on gauge metrics, only on counters.

rate(v range-vector) calculates the per-second average rate of increase of the time series in the range vector.

rate should only be used with counters. It is best suited for alerting, and for graphing of slow-moving counters.

https://prometheus.io/docs/prometheus/latest/querying/functions/#rate
vmware_vm_num_cpu's value doesn't change over time, all the samples will return the same value, since the number of vCPUs allocated to the VM doesn't change during VM runtime, so given that all samples in a range vector are the same the rate will be 0 - since there is no increase in value the rate of increase is 0.
avg_over_time calculates the average of all samples in a range vector ([5m] in your case), so an average of n-samples of 8 is... 8.
Given your requirement: "charge VMs to the number of vCPU, number of Memory and size of hard disk drives" - why do you need to use rate() or any other aggregating function? I'm not sure what you are trying to achieve with this query...
You should clarify to yourself what is that you expect your query to return and what that value should reflect, once you defined your requirement use the appropriate metrics and/or functions.

@SckyzO I'm closing this, if there's any issue with exporter you can open another issue.


TypeError: DB is not a constructor

In the line client.db = await new DB().init(process.env.MONGODB_URI, 'saves');
I get the error  "TypeError: DB is not a constructor". The line is copied from the readme.

Please show the actual code of how you import DB.

das englisch war btw ausversehen, mein kopf hing da fest ig
const { DB } = require('@mephisto5558/mongoose-db'); am anfang
try {
client.db = await new DB().init(process.env.MONGODB_URI, 'saves');
console.log('Verbunden');
...

Are you sure that that part of your code throws the error? In the most recent version of this library, your code is correct.
Please look a your stack trace again.
Also please the output of npm list @mephisto5558/mongoose-db.

Error: TypeError: DB is not a constructor
at main ...\source\index.js:27:23)

Also please show the output of npm list @mephisto5558/mongoose-db.
I am unable to reproduce your issue, see my screenshot:

solved it alone, still thank you for helping


[BUG] No policy report generate

Bug description
Hi,
When we generate documentation for VDOM firewall, the policy chapter is empty.
Chapter 1.5.5 is empty, 1.5.4 is ok.
I tried on two firewalls (with vdoms) and the issue is the same.
If i generate one on normal firewall (without vdom), it's ok.
Could you take a look please ?
Best regards
Renaud HUMBERT
Command-line input

Steps to reproduce
vdom firewall
Expected behaviour

Screenshots

Operating System
2022 server
PowerShell Version
$PSVersionTable
Name                           Value

PSVersion                      5.1.20348.2400
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.20348.2400
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
PowerShell Modules
Modules :
Script     1.4.0      AsBuiltReport.Core
Script     0.3.0      AsBuiltReport.Fortinet.FortiGate
Script     0.8.1      PowerFGT
Script     0.10.0     PScribo
Additional Context
No response
Before submitting

[X] I have read and followed the bug reporting guidelines.
[X] I have read the documentation, and referred to the known issues before submitting this bug report.
[X] I have checked for previously opened & closed issues before submitting this bug report.

Hi @rhumbert54
Can you check you have a PolicyLayout Settings on  AsBuiltReport.Fortinet.FortiGate.json ? (new feature of 0.3.0)

Yes, you're right.
I used always the same .json file that i edit for. So this options wasn't in my file.
I tried to add it and generate a new vdom report and it works.
My bad, Sorry about this mistake.
I update my documentation.
Best regards

Yes, you're right.
I used always the same .json file that i edit for. So this options wasn't in my file.
I tried to add it and generate a new vdom report and it works.
My bad, Sorry about this mistake.
I update my documentation.
Best regards

de rien :)


Please add $select to GET auditLogs/signIns

Is your feature request related to a problem? Please describe.
Currently it is not possible to use $select with the auditLogs/signIns API. This leads to massive requests each time when attempting to pull the Sign In Logs through the REST API: https://learn.microsoft.com/en-us/graph/api/signin-list?view=graph-rest-beta&tabs=http
Describe the solution you'd like

We would like to be able to select specific fields e.g. userPrincipalName, ipAddress, deviceDetail, etc as the current output for each log item is huge and filled with tons of unused fields. Currently it results in the following: Query option 'Select' is not allowed. To allow it, set the 'AllowedQueryOptions' property on EnableQueryAttribute or QueryValidationSettings.

Describe alternatives you've considered
Right now I have to filter the data I want to select on my side. This is do-able but pulling the logs we need results in over 3MB of data - it's not efficient at all to have to fetch this each time.
Additional context
Add any other context or screenshots about the feature request here.

I must echo joeldeteves. I have the same problem.  Any idea when this functionality is going to be present?

Hi @joeldeteves,
Thank you for reporting your concern. Unfortunately, we do not have any ownership of the APIs that are causing you issues. If you report this issue in the Microsoft Q&A forum, it will get routed to the appropriate team for them to triage.
https://aka.ms/askgraph


Add Font Family Extension for #64

Adding Font Family Extension as the request #64

After some thinking I decided to not supporting all styling nodes like text alignment, font family or font color in this core packages. Maybe you can publish this as a npm package.


[backport] Fix check_availablity for cupy.cusolver

Backport of #5207 by @kmaehashi

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 3fa2ec77bd6868c55f7230411b7478352c0b1361, target branch v9) succeeded!


Allow device_tracker and sensor entity for google travel times

Description:
This one was a bit tricky. It allows the google travel time component to use an entity_id as the origin or destination. During update and init it checks if it's an entity and if so looks for the longitude and latitude attributes if it's a device_tracker or the state if it's a sensor. If found it returns them in the proper format for the google api.
Allowed Entities

device_tracker
sensor
zone

Gotchas

During home assistant start if the entity state updates after it will report a warning log that it can't find the attributes.

BUT there is logic to make the platform wait until the event HOMEASSISTANT_START event is sent out. This allows it to make sure the entity is valid even if it's state or attributes are not accurate at the time.

If the entity is not found it will log an error and not continue loading that instance of the component.

If the entity is found but no attributes it will continue to load and show no state for the travel time entity. This is to allow the next polling update to check for the entity state after everything has had a chance to load and be populated.
Yes this could cause people that ignore their logs to never get travel times if they give it a bad entity with invalid location data. But this is already the case if you give it a bad address so no difference.

Behaviors

device_trackers

If state is not a zone it will look for longitude and latitude attributes.
If the state is a defined zone entity it will use the longitude and latitude of the zone.

sensors

If state is not a zone it will just send the state as is to the google api.
This is the same behavior as the current setup.
If the state is a zone it will load the longitude and latitude of the zone.

zones

Just loads the longitude and latitude from the attributes.

Useful for only defining a location once in home assistant

Backward Compatibility

All existing methods of import for origin and destination are still working exactly the same and will show no differences.

This can be used for origin or destination or both if you want to get travel times between two devices.
Example entry for configuration.yaml (if applicable):
- platform: google_travel_time
  name: Work Travel Time
  api_key: !env_var GOOGLE_API
  origin: device_tracker.owntracks_device
  destination: 123 abc street

Checklist:
If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

How about a googlemaps upgrade (#2481) as well?

Awesome idea!
Zone support as a trackable_domain would also be great.
With the correct template sensor you can implement smart destinations:
If device at home then destination='Work address' else destination='Home address'
Grouped together, the sensor can show you where it recommends you to go with the travel time.
To make this smart destination concept even more user friendly it would be good to iterate through all zone names to determine actual destination to be passed to google. So the sensor can simply return 'Home' or 'Work'. Loop through the Zone names, if there return the coordinates, else pass it to google
One caveat of smart sensors is halfway to work it will recommend you to go home :smile:

I didn't even think about zones. I will get these added and as you suggested allow a state == zone also work.

With the resolve_zone function I proposed, we can do "smart" destinations, like this
sensor:
  - platform: google_travel_time
    name: Google Travel Time
    api_key: !secret gmaps_api
    origin: device_tracker.johanniphone
    destination: sensor.destination_johann
  - platform: template
    sensors:
        destination_johann:
          value_template: >
            {%- if is_state("device_tracker.johanniphone", "home")  -%}
              J Work
            {%- else -%}
              Home
            {%- endif %}

zone:
  - name: J Work
    latitude: -25
    longitude: 28
    radius: 100

# And to group it nicely in a state card
group:
  travel_johann:
    name: Johann Travel Time
    entities:
      - sensor.google_travel_time
      - sensor.destination_johann

So using the human friendly string rather than the j_work entity hass sets up. I can make the change, but it still has the issue of someone like myself that overrides these friendly names in customize and contain duplicates.
I typically try to stick with programmatically making these decisions using ids or strings generated by the system and leaving the user definable and changeable strings to being interface side only.
This does bring up one thing I should probably change. Providing the sensor data with  just j_work could be ambiguous. So I would change it to look for "zone.j_work" and if nothing is find then check "j_work" and prefixing it with "zone." automatically. Somebody looking at the sensor data may not know what "J Work" or "j_work" really is, but if it's "zone.j_work" there is no question about it.

I hear your concerns with, but we are working with a relatively 'free' string here if google resolves addresses etc. Unfortunately with duplicate Friendly Names for zones, only the first will be returned.
The reason why I like Friendly Names is that you can display it in the interface. In the past the Travel time sensor could be named with the destination, but if you change the destination with a template you need to display what the destination is (or do an extra click to see it).
Currently doing it here you only resolve zones for TRACABLE_ENTITIES. Where if you move it to just before the sensor it applies everywhere, so you can also do static config like this
  from: Home
  to: J Work

Not sure what the state returns currently, but lets wait and see what the others think?

Yeah I think it may be set to the friendly name. I've pushed the changes to get it form the entity_id or friendly_name. I will make sure to put a note of it in the documentation that if there is duplicates it will just assume the first one it finds with that friendly_name on domain zone.

@bah2830 nice work, I was thinking about doing this at some point next weekend - but it looks like you've beat me to it!

Looks good! :dolphin:

Great work @bah2830! :tada:


"CSRF token missing or incorrect" when trying to upload annotations (COCO Format)

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

Be able to upload annotations to a task on browser. The images from the task is loaded from cloud storage (AWS S3).
Current Behaviour

When uploading annotations in COCO 1.0 format, the following error occurs:
Error: tus: unexpected response while creating upload, originated from request (method: POST, url: https://cvat.org/api/jobs/372892/annotations/, response code: 403, response text: {"detail":"CSRF Failed: CSRF token missing or incorrect."}, request id: n/a).
Steps to Reproduce (for bugs)

Open a task
Menu -> Upload annotations -> COCO 1.0
Select file corresponding to annotations in COCO Format

Your Environment

Git hash commit (git log -1): NA
Docker version docker version (e.g. Docker 17.0.05): NA
Are you using Docker Swarm or Kubernetes? No
Operating System and version (e.g. Linux, Windows, MacOS): cvat.org
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Hi @lubin-liu , try to relogin in CVAT

Hi @bsekachev,
I'm having the same error as @lubin-liu.
Also, I've tried to relogin in CVAT, and now the error is the following:
Error: Request failed with status code 400. "Could not match item id: 'f927a8c2-a521-11e7-93c7-e55f29b1fcdc_PGB_7_PGB_7_PGB7' with any task frame\n".
Environment:
Docker local environment from windows 10.
Thanks.

Hi, @nskostas
It says that in your annotation file you have image named f927a8c2-a521-11e7-93c7-e55f29b1fcdc_PGB_7_PGB_7_PGB7, but your task does not have images with this name.

Hi @bsekachev ,
Thanks for your quick reply. That's true the file is missing, is there a way to still import the annotations when some files are missing (ie you want to revise/reannotate part of your dataset)?
Thanks

Hi @bsekachev ,
Thanks for your quick reply. That's true the file is missing, is there a way to still import the annotations when some files are missing (ie you want to revise/reannotate part of your dataset)?
Thanks

@nskostas
That is probably question for @zhiltsov-max. Maxim, could you please comment on that?

Hi. Currently, you will need to remove extra images and all associated annotations. It can be done

with Datumaro (install with pip install datumaro) in a python script like this:

import datumaro as dm

dataset = dm.Dataset.import_from('path/', 'coco')
dataset.remove('<your image name here>')
dataset.export('path2/', 'coco_instances')

Annotations in the path/ directory need to be placed like described here.

manually by modifying the json file with annotations

Hi @zhiltsov-max ,
Thanks for the suggestions. We can already remove missing images from the annotations , so no problem with that.
It's just that I think it would be useful for CVAT to be able to 'ignore' the missing files and continue annotating the rest of the existing images.
Is there any plan of introducing this functionality in the future maybe?
Thanks

@zhiltsov-max Could you please answer/submit corresponding issue?

@nskostas, hi. Currently, there is no such feature in our plans, but it sounds reasonable and interesting to me. Thank you, I'll create a feature request issue for this idea.

Occurred Environments

Windows 10, docker
Server version: 2.3
Core version: 7.3.0
Canvas version: 2.16.1
UI version: 1.45.0

Reproduction Procedure

I haven't operated CVAT for a long time
Export a CVAT XML as edited tags
Import edited a CVAT XML
Occurred an error like the issue title

Patch Solution

Sign-out user account
Sign-in user account
Upload XML for a Task

Hypothesis
Maybe, Users can continue to action for the session has expired.
Forced reload didn't update the CSRF token.
This may be due to session reuse for a long time.
Fundamental Solution

try-catch CSRF error as shows a dialog "please sign-out, sign-in"
Don't reuse settions, All users are sign-out after long periods of inactivity (don't forget to save a job)
Another solution is to refresh the CSRF token at regular intervals, Ajax can do anything, maybe :P
Don't delete 'django.middleware.csrf.CsrfViewMiddleware'

Hi @lubin-liu , try to relogin in CVAT

This is very useful


create AssetManager base path dir when init

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
no

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

Would this overwrite $this->basePath is that happens to be a file or a link..?

Unfortunately, enhancements aren't accepted to Yii 2 anymore. That could be fixed in Yii 3 though.

Thanks for contributing anyway.


Update oeedger8r to run formatcode

Signed-off-by: Radhika Jandhyala<EMAIL_ADDRESS>

bors r+

bors r+

@CyanDevs There seems to be some issue with Code QL

bors r+


removeFavBrewery()

Back end:
Make mutation to remove fav brewery
Make resolver to remove fav brewery
Front end:
Make mutation in mutation.js to remove fav brewery

Back end:
Mutation to remove fav brewery
Resolver to remove far brewery
Front End:
mutation in mutation.js to remove fav brewery


Every TTS provider can have specific configuration fields

Refactored the structure of text to speech system to support multiple custom provider and customizable configuration. Separated the TextToSpeechConfiguration of different provider and the TextToSpeechSettings in the project settings. The selected provider loads it's own SettingsObject<MicrosoftTextToSpeechConfiguration> now.
To Implement your own TextToSpeech provider

Implement YourConfigurationClass: SettingsObject<YourConfigurationClass>, ITextToSpeechConfiguration as your new configuration
Implement ITextToSpeechProvider as your new provider and link your new configuration in LoadConfig()

I added the documentation and a better view in the project settings.


Breaking changes in PostgreSQL 12's libpq

The stack images currently install the PostrgreSQL client, library and headers from the upstream PostgreSQL APT repository (apt.postgresql.org) rather than the less frequently updated packages in Ubuntu's APT repository.
This means that as new major Postgres versions are released, the next stack image release will include updated major versions of libpq5 and libpq-dev (the clients themselves use versioned package names such as postgresql-client-11 so have to be explicitly updated).
New major versions of libpq are generally backwards compatible, however libpq v12 includes the following breaking change that makes the validation of connection parameters more strict:
https://github.com/postgres/postgres/commit/e7a2217978d9cbb2149bfcb4ef1e45716cfcbefb
Unfortunately this breaking change wasn't listed in the PostgreSQL 12 release notes, so we weren't able to factor it into the stack release review process.
This meant the latest stack image update caused database connection errors at runtime for a couple of customers, since their Rails apps were using invalid values for connection parameters (such as connect_timeout having trailing non-numeric characters). (Previously with libpq v11 these invalid values were being silently ignored.)
As such we:

rolled back that stack image release
reported the missing release notes breaking change entry to the Postgres project
filed ged/ruby-pg/issues/302 to have the pg gem report the more helpful validation related error message instead of PG::UnableToSend: no connection to the server

Determining how to best roll out the PG 12 client libraries will be more involved, so in the short term we should pin to v11 so that stack image releases are unblocked. After that someone will need to investigate further as to how widespread invalid app configurations are and work on a rollout plan.
CC @heroku/dod-infra since they will need to be aware of this if wanting to upgrade to PG12 in the stack image in the future. For more info see the investigation notes here.

The libpq5 and libpq-dev packages have now been pinned to the PostgreSQL 11 release in #148.
Leaving this issue open to track resolving the PG12 compatibility issues at some point in the future.

@edmorley so these are clearly integer values, what do customers put there?:)

There were two variations seen (both for Rails apps), one like:
    connect_timeout: <%= ENV['SOME_VAR'] || 5 %># some comment

...the lack of whitespace before the comment means connect_timeout ends up having a value like '5# some comment' (if a space is added, the interpolation works fine).
And for the other:
    connect_timeout: 15s

What made this hard for the customer to debug was the Ruby pg gem swallowing the real error. I've filed ged/ruby-pg/issues/302 for this, but even if it's fixed it's going to be some time before customers update to the new version.

So there's some good news about the confusing Ruby pg gem error message (that makes debugging harder) - it turns out it's a bug in libpq, for which the author of the Ruby pg gem has kindly already created a patch. Most importantly this means we can pick up the fix ourselves (once released), rather than needing to encourage/wait for customers to update their pg gem.
Whilst this doesn't make the libpq 12 changes any less breaking (for customers with invalid configs), it at least gives customers an actionable reason that they can self-serve without having to wait for support ticket assistance.

The libpq fix that results in an improved error message in ruby-pg (and likely others) landed in:
https://github.com/postgres/postgres/commit/ed5109a616cf98d3b3b2491d043099a6e0a966a2
There hasn't yet been a new libpq release since then, from what I can tell.

@edmorley Thanks for the update here. We're walking through other breaking changes in 12.0 and we'll continue to track this. It will likely will be in the next patch release but we'll stay tuned as well

@edmorley can we have a separate stack images for pg12 specifically until we clean this up, it can take a bit and we would like customers to start using them?

@edmorley we started something here - https://salesforce.quip.com/azvDAzWyI3NO

@lyubo1 Hi! Just checking you'd seen @schneems and @genslein's notes here?
https://salesforce.quip.com/8Ik7Abj1T0sK#YeAACAFqMJR

@edmorley thanks, have not!


add include for config

I was wondering if an include option could be added to allow more then 1 file to be used as the config file as it would make my dotfiles easier to maintain.

You can bundle a default_config inside of sworkstyle by modifying default_config.toml and building/installing the project. Only for version 1.2 and up


feat: add canPruneAtTime

A minor change to help on #9308. Instead of the archiver using the current time, it will use the time of the next ethereum block.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#9751  👈
#9728
#9725
#9430
#9399
#9398
#9328
#9297
#9296
#9247
#9246
#9084
#8942
#8941
#8940
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @LHerskind and the rest of your teammates on  Graphite

I thought about making this change here when I added it to the quote validation stuff, but had a concern:
Suppose you're in the beginning of slot 12 of an epoch, and no proof quote has been claimed. Doesn't this change make it so that if a proof does land after I look, I've already unwound?
But considering further, even if that does happen, it should just rebuild the chain, so maybe it is fine?
Regardless, it almost seems like the current behavior is the correct one?

I thought about making this change here when I added it to the quote validation stuff, but had a concern:
Suppose you're in the beginning of slot 12 of an epoch, and no proof quote has been claimed. Doesn't this change make it so that if a proof does land after I look, I've already unwound?
But considering further, even if that does happen, it should just rebuild the chain, so maybe it is fine?
Regardless, it almost seems like the current behavior is the correct one?

I don't get what you say? If it lands after you looked, it landed after it should be pruned. There should be no time for a proof to land in between you looking and it pruning?
If we can do that, it is an inconsistency in the flow, e.g., a bug. Looking at submitEpochRootProof looks like it do not check at all if it should be pruned, which just seems plain wrong to me. If it is to "automatically" prune, it should always do it, any action that moves the chain any way should perform the check.


Change if statement

https://github.com/qresgg/space-shooter/blob/1328a55a14ff817a18a984d03b66b6912c024eee/Assets/Scripts/Laser.cs#L14
Remove redundant ternary operator:
if (transform.position.y > 8f) Destroy(this.gameObject)

fixed


Adding Node v6.10.3

This is the companion PR to the kudu change to add Node v6.10.3 support. Note that the 6.10.3 Node image hasn't been published to DockerHub yet, so let's not merge this change until it has. I'll update this as soon as it's available.

@naziml This is ready to go, along with the corresponding Kudu change.

Thanks for the contrib!

@nickwalkmsft Thanks!


Function summary

Docstring and skeleton only

Inputs should be:

cluster center locations
cluster assignments for points in training data

Outputs:

pandas data frame with summary stats on each cluster (locations, number of points assigned, total within-cluster distances)


Mockup: NodeBalancer: Add Config

follow up to https://github.com/linode/manager/issues/1146

see #1146 for screens


HfHubHTTPError: 500 Server Error: Internal Server Error for url:

Describe the bug
newly uploaded datasets, since yesterday, yields an error.
old datasets, works fine.
Seems like the datasets api server returns a 500
I'm getting the same error, when I invoke load_dataset with my dataset.
Long discussion about it here, but I'm not sure anyone from huggingface have seen it.
https://discuss.huggingface.co/t/hfhubhttperror-500-server-error-internal-server-error-for-url/99580/1
Steps to reproduce the bug
this api url:
https://huggingface.co/api/datasets/neoneye/simon-arc-shape-v4-rev3
respond with:
{"error":"Internal Error - We're working hard to fix this as soon as possible!"}

Expected behavior
return no error with newer datasets.
With older datasets I can load the datasets fine.
Environment info
Browser
When I access the api in the browser:
https://huggingface.co/api/datasets/neoneye/simon-arc-shape-v4-rev3
{"error":"Internal Error - We're working hard to fix this as soon as possible!"}

Request headers
Accept	text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,*/*;q=0.8
Accept-Encoding	gzip, deflate, br, zstd
Accept-Language	en-US,en;q=0.5
Connection	keep-alive
Host	huggingface.co
Priority	u=1
Sec-Fetch-Dest	document
Sec-Fetch-Mode	navigate
Sec-Fetch-Site	cross-site
Upgrade-Insecure-Requests	1
User-Agent	Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:127.0) Gecko/20100101 Firefox/127.0

Response headers
X-Firefox-Spdy	h2
access-control-allow-origin	https://huggingface.co
access-control-expose-headers	X-Repo-Commit,X-Request-Id,X-Error-Code,X-Error-Message,X-Total-Count,ETag,Link,Accept-Ranges,Content-Range
content-length	80
content-type	application/json; charset=utf-8
cross-origin-opener-policy	same-origin
date	Fri, 26 Jul 2024 19:09:45 GMT
etag	W/"50-9qrwU+BNI4SD0Fe32p/nofkmv0c"
referrer-policy	strict-origin-when-cross-origin
vary	Origin
via	1.1 1624c79cd07e6098196697a6a7907e4a.cloudfront.net (CloudFront)
x-amz-cf-id	SP8E7n5qRaP6i9c9G83dNAiOzJBU4GXSrDRAcVNTomY895K35H0nJQ==
x-amz-cf-pop	CPH50-C1
x-cache	Error from cloudfront
x-error-message	Internal Error - We're working hard to fix this as soon as possible!
x-powered-by	huggingface-moon
x-request-id	Root=1-66a3f479-026417465ef42f49349fdca1

same issue here.

Also impacted by this issue in many of my datasets (though not all) - this seems to affect datasets that have been updated recently:

https://huggingface.co/api/datasets/acmc/cheat_reduced
https://huggingface.co/api/datasets/acmc/ghostbuster_reuter_reduced
https://huggingface.co/api/datasets/acmc/ghostbuster_wp_reduced
https://huggingface.co/api/datasets/acmc/ghostbuster_essay_reduced

Hey how to download these datasets using git cloning?

Also reported here
https://github.com/huggingface/huggingface_hub/issues/2425

I have been getting the same error for the past 8 hours as well

Same error since yesterday, fails on any new dataset created

Same here. I cannot download the HelpSteer2 dataset: https://huggingface.co/datasets/nvidia/HelpSteer2 which has been uploaded about a month ago

Hey how to download these datasets using git cloning?

You'll find a guide here 👍🏻

Same here for imdb dataset

It also happens with this dataset: https://huggingface.co/datasets/ylacombe/jenny-tts-6h-tagged

same here for all datsets in the sentence-tramsformers repo and related collections.
same issue with dataset that i recently uploaded on my repo.
seems that the upload date of the datset is not relevat (getting this issue with both old datasets and newer ones)
for some reason, i was able to get the dataset by turning it private and accessing it with the id token (accessing it as public while providing the token doesn not work)..... but i can say if that is just a random coincidence.
seems not much deterministic, for a specific dataset (sentence-transformer nq ) , that was "down" since some hours , worked for like 5-10 minutes, then stopped again

we're working on a fix !

We fixed the issue, you can load datasets again, sorry for the inconvenience !

I can confirm, it's working now. I can load the dataset, yay. Thank you @lhoestq

@lhoestq thank you so much!

Thanks for reporting, we are investigating !


Move Dev UI to v2

This PR removes the old Dev UI and add the new one.
This is an exact copy of the old Dev UI, with the exception that the class names are clickable and will open in an IDE.
Some ideas that might further improve the experience:

Display the wsdl file (see https://quarkus.io/guides/dev-ui-v2#code-block)
Dedicated log, especially if org.apache.cxf.ext.logging.LoggingFeature is enabled. see https://quarkus.io/guides/dev-ui-v2#add-a-log-file. This can be similar to what has been done for the test log. see https://github.com/quarkusio/quarkus/blob/main/extensions/vertx-http/dev-ui-resources/src/main/resources/dev-ui/qwc/qwc-test-log.js

fix #814

Yeah, thanks a lot @phillip-kruger!