Add webhook _id and api_version to webhook handler

Description
Fixes #1247
Shopify recommends as a best practice to check the x-shopify-webhook-id to verify you are not processing duplicate webhooks. We currently are not passing this information into the webhook handler. This PR will add the webhook_id and api_version to the webhook handler to bring the ruby library in line with the Javascript library in terms of information the handler has access to.
This could be resolved in a couple of approaches outlined below
Add webhook_id and api_version to handle function
This is what is currently implemented in this PR.
The webhook_id and api_version are added to the webhook request object from the headers. They are then explicitly passed to the handle function.
This would be a breaking change, but would be a straightforward fix. Developers would need to add webhook_id and api_version to their handle implementation.
Add metadata
If we are concerned, that in the future we may need to add even more information to the handle function and would require more breaking changes we could instead change the method signature to include the body and metadata. Where metadata could be a catch all of all the additional information. This would make any future additions easier but would require slight more initial refactoring for developers on the initial change.
      sig do
        abstract.params(body: T::Hash[String, T.untyped], metadata: T::Hash[String, T.untyped]).void
      end
      def handle(body:, metadata); end
    end

Add new handle function
If we are concerned about introducing a breaking change, we could introduce a new handle function. (eg. handle_with_webhook_and_version then developers would be able to implement the new function if they wanted access to the webhook_id and api_version. This could get complicated if we need to add more information in the future. (We would need to add another new function?)
How has this been tested?
Please, describe the tests that you ran to verify your changes.
Checklist:

[ ] My commit message follow the pattern described in here
[ ] I have performed a self-review of my own code.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have updated the project documentation.
[ ] I have added a changelog line.

I think we can make this a minor change by adding a static value to the webhook handler - if we add
module Handler
  FLAG_TO_USE_ALL_FIELDS = T.let(false, T::Boolean)

and then do
module TestHelpers
  class FakeWebhookHandler
    include ShopifyAPI::Webhooks::Handler
    FLAG_TO_USE_ALL_FIELDS = true

we can then use if handler.class::USE_ALL_FIELDS in the registry to call the appropriate version of the function.
We can add deprecation logs when using the old format to give folks time to fix it before a major version. We'd need to declare the abstract class in a way that could deal with both cases, but I think that should be possible with some creative typing?
Do you think that would work?
I agree that there is a bit of a deeper problem here. I see two ways that apps can make this future-proof:

declare their handlers with a **metadata argument
change the function to take in a PORO

I'd lean towards the second option because the first one is somewhat easy to forget and can trip people up - the second option is more reliable IMO.

Tested this and verified that:

The deprecation warning is shown when you use the old handler
When using the new handler you have access to the additional fields.


Kali AWS Error During Docker Build

Any clue on kali in aws error for a docker build of attack range? (edited)
Figured it out.
need to updated filter to use 2022 instead of 2021
filter {
name   = "name"
values = ["kali-linux-2022*"]
}
/attack_range/terraform/aws/modules/kali_machine

https://github.com/splunk/attack_range/issues/622
wrong home. moved.


Formatting Errors in README.md

I know what you mean but it helps to provide a description in the future.


Bump Travis osx_image to xcode9.4

Checking if this fixes Travis CI on macOS/iOS (xcode9.4 is the default now)

bors try

bors r+


[BUG]The red card is displayed for the receiver of the payment after opening the payment link

fixed getTxType
if handled receipt is older than current receipt dont update feed event
Description
[BUG]The red card is displayed for the receiver of the payment after opening the payment link
About # (2788)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist:

[ ] PR title matches follow: (Feature|Bug|Chore) Task Name
[ ] My code follows the style guidelines of this project
[ ] I have followed all the instructions described in the initial task (check Definitions of Done)
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have added reference to a related issue in the repository
[ ] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[ ] @mentions of the person or team responsible for reviewing proposed changes

@omerzam i've refactored some code. please review
@serdiukov-o-nordwhale


Memory Leak

Using you library in our project (thanks a lot for it) in which we use CanaryLeak I've found two leaks in your code:

There are some anonymous  AnimatorUpdateListener defined that holds a reference to the context.
There are some anonymous TypeEvaluator implementations that because they holds references to their parent class and this one to the context leak memory too.

PR: https://github.com/tajchert/WaitingDots/pull/11

How about PR: #6 ? Isn't it better?
I didn't have enough time to check all commits related to #6

looks great! you have any forecast or should we just download the code and work with it instead using the one in jcenter?

I just checked as it works fine, didn't have time to find leaks and verify that those commits indeed solves particular leaks. If you can you can test it for leaks you have found, and ping me here with verdict if it works fine - I will merge it at that point if everything is fine. Just need somebody else to confirm it :)

checked https://github.com/tajchert/WaitingDots/pull/6:, leaks memory too because of the AnimatorUpdateListeners on the animators, the are never being removed
This is the screenshot I get from LeakCanary(which usually helps me more than the log)

I've merged your #11 PR, as #6 is still getting some leaks and (sorry to say) I currently don't have time to go through it - anybody willing do fix other leaks (if there are any) are more than welcome.


[Tracking] Cloud Native Sustainability Week 2024 Amsterdam

Description

Location/Region: Amsterdam, Notherlands
Organizers: @ams0 @JulieChenadec @wilco @pavel
Sponsors: tbd
Date: 10th October 2024
Event link: https://www.meetup.com/dutch-kubernetes-meetup/events/301418005/

Details
We'll have a full half-day of programs, workshops, talks, and plenty of time and space to connect and brainstorm on how to get better sustainability out of cloud native software.
To-Do

[X] Review the Meetup Organizer Guide for onboarding.
[X] Check the website.
[ ] Create a PR to update the Meetup's table on the website. Example PR.
[X] Join one of the TAG ENV Sustainability Week Syncs to talk about your event.
[ ] Find a sponsor for the event.
[X] Find a location for the event.
[X] Set a date for the event.
[X] Announce the event.
[ ] Reach out to the Cloud Native Sustainability Week Organizer team to promote the event.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Comments
No response

@JulieChenadec getting a 404 on the linked event page.

It's a draft at the moment, will publish and make one event on community.cncf.io as well this week!

Thanks for jumping in, Alessandro! Indeed it will be published soon @mijndert :)

Hello @JulieChenadec @ams0
I hope you're doing well. With just a few weeks left, I wanted to check in and see if you need any help with anything.

Hi! Just a reminder to submit all the speakers and organizers' information for Credly badges using this form https://forms.gle/KHizxY47h3ynEBUL9

Closing this since all the main activities were completed. Thanks to everyone who made this local event possible and contributed! 🔝


Add more text options for email alerts

This commit adds fields to most finders to define a separate email alert list title text which will be used when the normally generated string is too long (> 255 characters). It will be appended with a string of the form “x items”, where “x” is the number of selected topics and “items” is either the singular or plural string defined as email_filter_name.
Also reverts https://github.com/alphagov/specialist-publisher/pull/984 since this is a more comprehensive solution.
Trello: https://trello.com/c/KTHRa4a0/422-fix-email-alert-api-errors-when-the-email-alert-short-name-is-longer-than-255-characters
Deployment checklist:

[ ] Deploy specialist-publisher
[ ] Run the rake task publishing_api:publish_finders to re-publish all the finders

I'm not sure if this approach will work with GovDelivery, because it sounds like we would generate identical names for multiple topics. I've just tried to manually create a topic with the same name as an existing one (in our Integration account) and it errors, saying the name needs to be unique 😢  The API docs don't mention that constraint though.
We use the generated title as both name and short-name - it looks like only name needs to be unique, whereas the length error was on short-name, so there might be a way around those constraints. It would be simpler for our future plans to keep those two fields identical if possible, though. I can help you test on integration this afternoon if you like.

Discussed with @jennyd: following this PR I will raise another PR for finder-frontend that splits the logic for title and short_name to be separate so that we can continue to set title as we do now and use new logic for short_name that limits it to 255 characters. This PR is ready for review.


Add ko build strategy sample

Changes
Based on this inquiry, I am adding the ko sample build strategy that I recently created. I added a big note to the documentation about the creative interpretation of the contextDir property.
Submitter Checklist

[X] Includes tests if functionality changed/was added
[X] Includes docs if changes are user-facing
[X] Release notes block has been filled in, or marked NONE

Release Notes
- Adding a sample build strategy for [ko](https://github.com/google/ko)

The Travis build uses something like <IP_ADDRESS>:5000/shipwright-io/build-e2e:ko (with more than one :) as image during e2e which breaks my logic to extract the tag name. Let me fix that. :-)

Hi @ImJasonH ! Could you please review ?

/lgtm
Thanks for adding this! 👍

Updated to ko v0.8.1.

/lgtm
/approve


update typescript to ~2.1.4

Type: feature
Description:
Update typescript dependency to ~2.1.4. Also changes to support Intern ~3.4.1
Reference Issue: #https://github.com/dojo/meta/issues/124

Approve the changes once the ci is fixed.

Current coverage is 100% (diff: 100%)

Merging #28 into master will not change coverage

@@           master   #28   diff @@
===================================
  Files           2     2
  Lines          34    34
  Methods         1     1
  Messages        0     0
  Branches        8     8
===================================
  Hits           34    34
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 2967e88...1451b43


feat: Faker.ts initial

This is the very first code of the project

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4633310). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   99.03%
=========================================
  Files             ?       13
  Lines             ?      207
  Branches          ?       25
=========================================
  Hits              ?      205
  Misses            ?        0
  Partials          ?        2

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 4633310...125c70a. Read the comment docs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4633310). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   99.03%
=========================================
  Files             ?       13
  Lines             ?      207
  Branches          ?       25
=========================================
  Hits              ?      205
  Misses            ?        0
  Partials          ?        2

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 4633310...125c70a. Read the comment docs.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on:

npm package (@pre/rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rest: add swagger doc for tenants

rest: add swagger doc for tenants

@jiazhai
[pulsar_precommit_java8] $ /bin/bash -xe /tmp/jenkins4167227195723580361.sh
+ src/check-binary-license ./distribution/server/target/apache-pulsar-2.4.0-SNAPSHOT-bin.tar.gz
swagger-annotations-1.5.21.jar unaccounted for in lib/presto/LICENSE

Thanks @merlimat for the comments.
updated the license file.
regarding ApiModelProperty, it is able to work for rest.

run java8 tests

@merlimat would you please help take a look again?

regarding ApiModelProperty, it is able to work for rest.

I mean the description itself. I'm not sure of the If empty the current set of roles won't be modified part. If that it's on the REST handler or just implemented in the CLI tool.

Thanks @merlimat. You are right, in rest, we should provide both adminRoles and allowedClusters.

run java8 tests


Set separateMultipleMinor for python to "true" in Renovate configs

Is your feature request related to a problem? Please describe.
I am frustrated that Renovate PRs for Python version updates are never automatically merged because it is very rare to be able to update to the latest minor version of Python at once (e.g. when relying on libraries such as numpy and numba).
Describe the solution you'd like
Change the settings after this issue (feature request) is resolved.
https://github.com/renovatebot/renovate/issues/6576
https://github.com/renovatebot/renovate/pull/24538
Additional context

It might be better to configure each repository individually.
Related: #597
Sorry for opening so many issues.

That sounds sensible, yes. Would you mind sending the PR to my base configs? https://github.com/browniebroke/renovate-configs/blob/main/cpython.json


Add piff and dependencies

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

You only need to add piff. The conda package will know to install its deps.

You only need to add piff. The conda package will know to install its deps.

Wasn't sure what the convention was.  Fixed.


feat: add govukTaskList() component

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

137 of 137 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 98.503%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3539

Relevant Lines:
3567

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

137 of 137 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 98.503%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
3539

Relevant Lines:
3567

💛  - Coveralls


CVE

Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

Please explain what you are trying to achieve with this PR so we can properly review it.


var and std

Hi! Thank you very much for sharing the great work.
Why you insert variation in the std parameter?
https://github.com/cuiziteng/ICCV_MAET/blob/5fe75784ab058e22a8f595d0a26bdfc6d4cdd5c5/mmdet/models/detectors/MAET_YOLO/MAET_YOLO.py#L217
I read original imprementation of 'unprocessing' and it's different.
This repo seems to estimate noise level smaller than original repo.
Also, why quantization noise can be bellow? https://github.com/cuiziteng/ICCV_MAET/blob/5fe75784ab058e22a8f595d0a26bdfc6d4cdd5c5/mmdet/models/detectors/MAET_YOLO/MAET_YOLO.py#L225
I think quantization noise is related to the value of 2**bits.
Thank you in advance!

Hi !

This is same. Because the work "Un-process" they use a specific cameras' settings, and there remain difference between low-light and normal-light condition, so here we don't need to obey them.

We take a uniform distribution for simulate quantisation noise (link), the quantization in real ISPs is 16-bit to 8-bit, and. And you can see this line.  here we take 3 number between 8~16.

Thank you for quick modification!
I can understand the (link) but it saids that, quantizaiton noise is
uniform(-Δ/2, Δ/2)
and
Δ~2^(-bits).
So from the link, I think
uniform(-1/2^(bits+1), 1/2^(bits+1))
is correct although this difference might not affect the accuracy.
if bits=12,
1/3060(=25512) and 1/8292(=2^(12+1))
if bits=16,
1/4080(=25516) and 1/131072(=2^(16+1))

Also, did you use this function?
https://github.com/cuiziteng/ICCV_MAET/blob/a3f74a54c95316c9f8d98bedb918c47f55f41802/mmdet/datasets/pipelines/dark_transforms.py#L34
`
bits = random.choice(self.quantization)
img_quan1 = img* 255.0
img_quan_bit = img_quan1/ bits
img_quan2 = np.around(img_quan_bit)* bits
#quan_noise = (img_quan2 - img_quan1)/ bits
img = img_quan2/ 255.0
`
This quantization noise is also diffenrent from the implementation in MAET_YOLO.py and it is too big.
1/bits of domain is occupyed with noise and darkened image has almost no info.

Thank you so much, I'll correct it. And also very happy to see your modeling's results!


Can you create a GUI for batch files render ?

There is a GUI program named SpleeterGui in developing, since July 2022.
But it still has some details to be completed, and the code has not been released.
Currently, it looks like this:

The Spleeter GUI program has been released.
See https://github.com/wudicgi/SpleeterGui/releases


[FLINK-34969][cdc-cli]Add support for both new and old Flink config files in Flink…

This commit introduces a simple logic check within the Flink CDC codebase to enable support for both the new (config.yaml) and old (flink-conf.yaml) Flink configuration file standards. By utilizing the Files.exists method, Flink CDC now first checks for the presence of config.yaml in the system. If config.yaml is found, it is used as the configuration file. Otherwise, Flink CDC falls back to using flink-conf.yaml. This enhancement ensures that Flink CDC can seamlessly transition between different versions of Flink, maintaining backward compatibility and facilitating easier upgrades for users.

@skymilong Thanks for the PR! According to the code contribution rule of Apache Flink, could you create an issue on Jira for this and include the Jira ID in the PR title and commit message? You can take #3160 as an example.
And the PR has code formatting issue. Please run mvn spotless:apply before pushing your commits.
Thanks!

@skymilong Thanks for the PR! According to the code contribution rule of Apache Flink, could you create an issue on Jira for this and include the Jira ID in the PR title and commit message? You can take #3160 as an example.
And the PR has code formatting issue. Please run mvn spotless:apply before pushing your commits.
Thanks!

Thanks a bunch for your help. This is my first rodeo contributing code on GitHub, so I'm still trying to get the hang of things. I'll whip up a Jira issue like you suggested and pop the ID into the PR and commit messages. Also, I'll iron out those code formatting kinks. I'll get on it pronto and get the PR updated.
Appreciate your patience and all your help.

@skymilong Welcome to the community! Feel free to ask any questions

@skymilong Thanks for the update! Could you rebase the latest master branch? I just merged a commit for fixing unstable CI tests


Fixed TODO in FrameMain.

Changed catch exception to catch specific exceptions.

Coverage decreased (-0.02%) to 14.128% when pulling b00624f5d907dc2a81881a21a5cf5e62bec43b24 on FridaTveit:FixTodoExceptionInFrameMain into b759460bc01728ba0ae7c0dbd82bed8c9e8b7c32 on oskopek:master.

Coverage decreased (-0.02%) to 14.128% when pulling b00624f5d907dc2a81881a21a5cf5e62bec43b24 on FridaTveit:FixTodoExceptionInFrameMain into b759460bc01728ba0ae7c0dbd82bed8c9e8b7c32 on oskopek:master.

Coverage decreased (-0.02%) to 14.128% when pulling b3dd3fa0b6f5e453e82fc4e9f70a4b8b98059615 on FridaTveit:FixTodoExceptionInFrameMain into b759460bc01728ba0ae7c0dbd82bed8c9e8b7c32 on oskopek:master.

Coverage decreased (-0.03%) to 14.118% when pulling 0d1e46e3c02dab73c4f77a05915c368725cfbb31 on FridaTveit:FixTodoExceptionInFrameMain into 45b7c34d5c9a6d3ad69b8221a17419e68bf4b1ff on oskopek:master.


Streaming_mean along specific dimension

Currently the streaming_mean function computes the mean for all tensor values. It would be interesting to have the possibility to specify the axis along with we want to compute the mean.

Marking as contributions welcome since it is in contrib not core TensorFlow.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Support a maximum depth to protect against malicious payloads

Bison supports YYMAXDEPTH (docs), we should do something similar so that we can provide a better experience than a SystemStackError or equivalent.

I have couple of question about this:

Is parser_t a good place to keep the level of recursion?
Is testing whether we hit the limit in parse_expression enough, or do we need to guard against this in any other functions?
What should the parser do when the limit is hit?


tritonast2arybo: unsupported node type 79 (BVSMOD)

Trying to go from Triton AST to LLVM IR by running some examples but getting unsupported node type 79 for BVSMOD
Is it possible to support this in arybo or is there a workaround?

I copied some code from TritonASTLLVMIRTranslator to arybo to glue-fix this:
def glue_for_bvsmod(lhs, rhs):
    srem = lhs.srem(rhs)
    add = srem.__add__(rhs)
    return add.srem(rhs)

I wanted to submit a PR but the testsuite for the triton part is very outdated so bumping it to work the most recent Triton takes a bit more time


"Too few public methods" false positives in PyLint when inheriting from GenericModel

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
I have a class that inherits from GenericModel and adds x public methods to the resulting model. If I then inherit from my created class but don't specify any additional public methods, I receive a "Too few public methods" warning from PyLint (despite the resulting class having the public methods inherited from the base class).
Example below;
RootDictModel inherits from GenericModel and adds 3 public methods.
Tags inherits from RootDictModel (specifying Types)
Shortcuts inherits from RootDictModel (specifying Types)
Both Tags and Shortcuts give a "Too few public methods" warning in Pylint. However, if I switch RootDictModel to inherit from BaseModel instead of GenericModel - the Pylint errors go away.
Example Code
from typing import Generic, Iterator, Tuple, TypeVar
from pydantic import BaseModel, ConstrainedStr, Field
from pydantic.generics import GenericModel

_KT = TypeVar('_KT', str, ConstrainedStr)
_VT = TypeVar('_VT')

class RootDictModel(GenericModel, Generic[_KT, _VT]):
    __root__: dict[_KT, _VT] = {}
    def __getitem__(self, key: _KT) -> _VT:
        return self.__root__[key]
    def __iter__(self) -> Iterator[_KT]:  # type: ignore
        return iter(self.__root__)
    def keys(self) -> list[_KT]:
        return list(self.__root__.keys())
    def values(self) -> list[_VT]:
        return list(self.__root__.values())
    def items(self) -> list[Tuple[_KT, _VT]]:
        return list(self.__root__.items())

class TagKey(ConstrainedStr):
    regex: re.Pattern[str] = re.compile(r'^\d*$')

class Tags(RootDictModel[TagKey, str]):
    ...

class Shortcut(BaseModel, extra='allow'):
    id: int = Field(alias="appid")
    name: str = Field(alias="appname")
    tags: Tags = Tags()

class Shortcuts(RootDictModel[str, Shortcut]):
    ...

Python, Pydantic & OS Version
pydantic version: 1.10.2
pydantic compiled: False
install path: /home/deck/Desktop/PythonProjects/projects/distiller/.venv/lib/python3.10/site-packages/pydantic
python version: 3.10.2 (main, Jan 15 2022, 19:56:27) [GCC 11.1.0]
platform: Linux-5.18.1-arch1_testHoloISO_20220606.1811-x86_64-with-glibc2.33
optional deps. installed: ['typing-extensions']

Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .dict() and .json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[X] Generic Models
[ ] Other Model behaviour - construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

PyLint doesn't play well with pydantic, see #568 and a few other issues.
Generics will change somewhat in pydantic V2, but I can't guarantee it'll play nicely with pylint.


Formatting of Code Examples

The formatting of these samples leaves some to be desired, with terminating semicolon on a separate line, and large gaps the leading commas. The same is true for other Azure DW SQL samples. The other TSQL MSDN documentation is quite different.

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

ID: 068afb53-f222-d2d8-64e0-e84eba9aebb7
Version Independent ID: 16668ecc-69e0-6428-75ab-105a9fe379cc
Content: CREATE TABLE AS SELECT (CTAS) in Azure SQL Data Warehouse
Content Source: articles/sql-data-warehouse/sql-data-warehouse-develop-ctas.md
Service: sql-data-warehouse
Sub-service: development
GitHub Login: @XiaoyuL-Preview
Microsoft Alias: xiaoyul

@aceptra , Thanks for raising the issue. We will investigate and update you.

This issue is being assigned to the content author for updating the formatting as appropriate.

#please-close


Rewrite of outgoing email handling

Also known as: "Iris pokes the email queue with a very pointy stick"

[x] Per-message Sidekiq message send job
[ ] System admin dashboard section for inspecting queued emails
[ ] Maybe make the email sending just use the SendGrid HTTP API instead of SMTP?

Merging as-is, will follow up with admin dashboard later


Test Qt android SDK installation on CI

I didn't find your fork until I had made my own but I made this script to generate an azure-pipelines.yml for my fork with all known QT options:
https://github.com/nelsonjchen/qli-installer/blob/master/.azure-pipelines/generate_azure_pipelines_yml.py
Output: https://github.com/nelsonjchen/qli-installer/blob/master/azure-pipelines.yml
Maybe this might be a good reference.

Ooh, you are looking at my script! Cool!
You should know, it does this and makes a ton of jobs:
https://dev.azure.com/nelsonjchen/QLI-Installer/_build/results?buildId=329

Because generator script is written with some python 3.7 features, I've modified to be able to run by older one.

😅 ah yeah, I made it for 3.7 because I didn't intend for the script to be run by end-users but just developers to regenerate azure-pipelines.yml in rare instances.

It is interesting to see that the Windows SDK jobs aren't completing due to some connectivity issue with a QT mirror in China.

"根据相关法律法规，本站不对欧盟用户提供服务。"
Google Translated: "According to relevant laws and regulations, this site does not provide services to EU users."
This is taken from the legal text stuff at the bottom of the HTTP directory listing at https://mirrors.tuna.tsinghua.edu.cn/qt/archive/qt/5.9/5.9.8/single/.
I wonder if those jobs are running from the EU or something. Azure Pipelines has an overflow system where if a region has too many jobs running, it'll use capacity from other regions. However, this only documented for the Brazil South or West Europe regions. That said, maybe they might not have documented Southeast Asia's overflow, if any.
If you want to "trust, but verify" that, perhaps you can throw in a call to some GeoIP thing and log to STDOUT where the VM running the job.

Actually, nevermind that. If it was in the EU, the chances of the mirror system giving that mirror's address should be not much, if at all. But still, maybe worth verifying.

Hmm, I hit that same issue just recently even in the US region. Maybe a retry option should be added.

Thank you for investigation. We can take workaround for check geoip and handling redirects through python http subsystem. here is a mirror list: https://download.qt.io/static/mirrorlist/

feature is committed


Move to new organisation

Any opposition to moving this repository to github.com/build-a-binder? That way the webpage can be build-a-binder.github.io
This is your last chance to suggest a less clunky name :)

I like the name. :) It is a call to action, so it calls the reader in.

Agreed! :)

Welcome to https://github.com/Build-a-binder/build-a-binder


Replace Arena with MemPool

After replacing Arena with MemPool, we can achieve one copy for string
value read from segment v2. We can exchange MemPool's chunk between
RowBlockV2 and RowBlock. This change only replace Arena, this work will
be done in other change list.
#2011

LGTM


Distractor-aware incremental learning

Hello together,
I am currently investigating the DaSiamRPN architecture, which I thought is implemented in the scope of pysot. I can find the long-term module which implements the long-term strategy described in the official paper and also the distractor-aware training.
However, I cannot find the distractor-aware incremental learning part (section 3.3 in the paper). Did I maybe overread it somewhere or is it not implemented?
Thanks for your help!

It seems there is not an official implement. There is an unofficial implement at:https://github.com/JachinShen/DaSiamRPNAT


Google Authentication

Is your proposal related to a problem?

Currently, users can only sign up using an email and password, which might not be the most convenient option. Many users prefer social logins for a faster, seamless experience, and adding Google Authentication would simplify the registration process for them.
Describe the solution you'd like

Integrating Google OAuth 2.0 to allow users to sign up and log in using their Google account. This would reduce the friction during the sign-up process and improve the overall user experience.
Describe alternatives you've considered

Other social logins such as GitHub or Facebook, but Google Authentication is the most commonly used and widely trusted option, making it a better fit for this project’s first implementation of social login.
Additional context

The Google Authentication button could be placed beneath or alongside the current sign-up form, similar to what other platforms use for OAuth-based logins.
Requirement/Document

A basic implementation of Google OAuth 2.0 can be found in the Google Developers documentation which could guide the integration process. This will require backend handling of OAuth tokens and secure user data storage.

@ayushku634
This issue is already there and someone is already working on this.
Refer issue #9


Changing the Memory Limit

Hello ivanfratric
How would one go about changing the memory limit of the WinAFL tool.
Would changing the memory limit in config.h work?
/* Default memory limit for child process (MB): */
55
56  #ifndef __x86_64__
57  #  define MEM_LIMIT         25
58  #else
59  #  define MEM_LIMIT         50
60  #endif /* ^!__x86_64__ */

WinAFL can't change the memory limit of the target process. If you need to enforce a memory limit you need to build a custom solution for your target.

Hmm OK. Well, the main reason why I raised this issue is because, all the test cases generated by WinAFL in the crash folder (the ones that caused the target to crash during the WinAFL run). When I actually individually ran those test cases straight to the target, none of them crash the target.
Basically I can't reproduce the crashes found by WinAFL. The AFL readme file suggested that changing memory limit might resolve this issue.

I can't reproduce the crashes found by WinAFL. Running the individual test cases (in the crashes folder) against my target binary does not result in a crash or segmentation fault.
Can this happen, or am I doing something wrong?

There are several reasons this could be happening. For reference, take a look at the onexception() function inside winafl.c to see how WinAFL catches crashes. Some possibilities:

The exception is caught by your target so it doesn't crash. In this case you should still see an exception if you run the target with a debugger. You can also run the "crashing" sample in the WinAFL debug mode and the most recent version is going to report encountered exceptions in the debug log.
Since WinAFL uses persistent mode, it is possible that some of the previous samples put your target in a state where a subsequent state caused a crash. In the simple case it could be that previous samples exhausted the memory of the target process, a subsequent sample tried to allocate memory and failed which caused a crash. But it could also mean more serious incorrect states. You can run WinAFL with lesser number of fuzz_iterations. In the extreme case you can set fuzz_iterations to 1 which will restart a process for every sample (needless to say that's going to run much slower). You can also take a corpus provided by WinAFL and run it through some other, non-coverage-guided fuzzer in order to try to rediscover a crash.

I'm closing this issue since memory limit support was added in 1.11


Latest Server Update

What version of Music Assistant has the issue?
2.8
What version of the Home Assistant Integration have you got installed?
2024.8.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
With the latest music assistant server update we need to authenticate Spotify again.
The process will open in a new popup window it says , but nothing happens , after 30 sec. I get a window with 999 in it , when you click ok nothing happens and Spotify is not authenticated.
When I open up Spotify during the process , music assistant doesn't show up either.
I don't have popup blockers installed.
How to reproduce
I'ff installed the previous backup to make it work again.
Music Providers
Spotify
Player Providers
All of my media players because of Spotify not authenticated.
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.8.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

The MA version you have listed is not correct. Please look in MA SETTINGS>>CORE and update the report.

I assume you are trying this from an iOS device ?
The next beta has a small workaround for that (as it blocks the whole tab)

Hello.
Yes indeed.
But also tried it on my Mac.
Same problem…
When I update with the 2.2.0rc1 server update I need to authenticate
Spotify again and that is not working…
Nothing happens…
Regards.
JS.
Le mer. 21 août 2024 à 02:03, Marcel van der Veldt @.***>
a écrit :

I assume you are trying this from an iOS device ?
The next beta has a small workaround for that (as it blocks the whole tab)
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2299961803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FSD4ALPL64YU53NUP3ZSPKM7AVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJZHE3DCOBQGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Then the pop up must be getting blocked. What browser are you using on your Mac?

Hello.
I’m using Safari as default browser.
Regards.
JS.
Le mer. 21 août 2024 à 06:33, OzGav @.***> a écrit :

Then the pop up must be getting blocked. What browser are you using on
your Mac?
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2300901364,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FWCCHNTV64DYH64FCLZSQKBZAVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBQHEYDCMZWGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah we have seen issues with Safari. Please try Chrome

Ok…
Thanks for the reply and input…
Le mer. 21 août 2024 à 07:13, OzGav @.***> a écrit :

Yeah we have seen issues with Safari. Please try Chrome
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2301145062,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FWZXJ5Y426MIE6EQH3ZSQOZHAVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGE2DKMBWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I just tried using Chrome , Edge and Safari on my iPhone.
But still nothing happens.
I click on Authentificate the blue wheel turns and after a while I get as
shown on the foto…
And Authetification with Spotify fails.
Regards.
JS.
Le mer. 21 août 2024 à 07:13, OzGav @.***> a écrit :

Yeah we have seen issues with Safari. Please try Chrome
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2301145062,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FWZXJ5Y426MIE6EQH3ZSQOZHAVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGE2DKMBWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

The trouble is iOS uses the same underlying browser engine for all browsers. That's why I mentioned using your Mac above. To rule out all these variables please try Chrome on your Mac and let us know how you go.

Ok…
As soon as I get home this will be done.
I will keep you informed…
Many thanks for the help and reply.
Regards.
JS.
Le mer. 21 août 2024 à 08:51, OzGav @.***> a écrit :

The trouble is iOS uses the same underlying browser engine for all
browsers. That's why I mentioned using your Mac above. To rule out all
these variables please try Chrome on your Mac and let us know how you go.
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2301267708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FTCEIBBQSMFNV5JQNLZSQ2G3AVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGI3DONZQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

In the 2.2.0 RC1 version I added a workaround for iOS where you get an additional button to click to continue

I’f got the 2.2.rC1 installed.
I’m sorry but I don’t see an additional button to click on…
I can only click on , « Authenticate with Spotify »
And when I do nothing happens when using iPhone.
Will try this evening using Chrome in my Mac.
Le mer. 21 août 2024 à 10:20, Marcel van der Veldt @.***>
a écrit :

In the 2.2.0 RC1 version I added a workaround for iOS where you get an
additional button to click to continue
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2301446934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FRQQOSXFBAZUDISWQDZSRETPAVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGQ2DMOJTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

This is what you should see from an iPhone (tested with safari)

argh... I now see it doesn't work with the HA companion app, sigh..
I'll add some more bandaid

@marcelveldt - sorry to jump on this, but found the thread as I was looking for help on a similar issue.
I have what seems to be the same issue: click on authenticate, and the new window does not launch , but I use windows and chrome, no popup blockers.
Server v2.1.4
It is made more complicated by the fact that I had changed my spotify password, so I thought this was the cause of re-authentication required.
Is there any information I can get to help ? (relative newbie here, so may need a bit of guidance...)
What version of Music Assistant has the issue?
2.1.4
What version of the Home Assistant Integration have you got installed?
Core
2024.8.2
Supervisor
2024.08.0
Operating System
13.0
Frontend
20240809.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?
Yes
The problem
With the latest music assistant server update we need to authenticate Spotify again.
The process will open in a new popup window it says , but nothing happens , after 30 sec. I get a window with 999 in it , when you click ok nothing happens and Spotify is not authenticated.
When I open up Spotify during the process , music assistant doesn't show up either.
I don't have popup blockers installed.
How to reproduce
re-start HA and try to add spotify as provider
Music Providers
Spotify
Player Providers
N/A
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
Core
2024.8.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi 4

@dblanc42 wait for the next 2.2.0 release please. You are still running the stable version which uses spotify connect auth (there is no popup)

Thank you, and sorry for the interruption... I will check with 2.2.0 indeed. I followed the info in the config page (FYI)

Ah, yeah we have been a bit too fanatic with updating the docs. That info is already for the 2.2 release (some folks are testing the beta). If you are still on 2.1.x stable, then clicking the button results in a Spotify connect auth session: you need to open the Spotify app and select music assistant as speaker to play something.
But 2.2 will be released later today so you can also just wait a little bit ;-)

Don’t worry Marcel I’ll wait….
Le mer. 21 août 2024 à 12:10, Marcel van der Veldt @.***>
a écrit :

Ah, yeah we have been a bit too fanatic with updating the docs. That info
is already for the 2.2 release (some folks are testing the beta). If you
are still on 2.1.x stable, then clicking the button results in a Spotify
connect auth session: you need to open the Spotify app and select music
assistant as speaker to play something.
But 2.2 will be released later today so you can also just wait a little
bit ;-)
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2301674085,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FWGPTB2RFOTG3LGNFTZSRRSZAVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGY3TIMBYGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

same here, no rush... thanks for all the great work.

The new spotify auth has worked for me, but broke in either RC1 or RC2, at least now on RC2, no popup appears

The new spotify auth has worked for me, but broke in either RC1 or RC2, at least now on RC2, no popup appears (android, chrome)

2.2.0 stable has landed (with the -final- new spotify auth flow) - we also released 2.3.0 beta 0 with these fixes included

I just upgraded to version: 2.2.0.  I cannot get the authentication to do anything but give me a 999 error after a few minutes.
No pop-up ever appears.  I have tried on Android, PC using Chrome, Edge, and Firefox.  None of them work.
I'm running HA Ver: 13.1.

the same here... 2.2.0 and getting the 999 error on a Mac with Safari, Firefox and Chrome :(

Same here. On HAOS 13.1, MA 2.2.0. Tried on Chrome (Windows) and iOS, Authenticate times out with 999.
Logs show
2024-08-21 15:03:48.035 WARNING (MainThread) [music_assistant] Error loading provider(instance) Spotify: Configuration is invalid (will be retried later)
**Fix: ** Delete the Spotify music provider and re-add it. Authentication popup works as expected after re-adding.

@1joneselliot thanks for identifying a fix. We will see if we can work out why that happened in the first place. Only thing I would say is wait until the log says the provider has been removed before adding it again

I had the same problems as others here.
If you delete spotify as a music provider then re-add it, however, it worked just fine.

I believe this can be resolved with re-addon the provider. If there are any other problems with logging in to Spotify add to this other issue https://github.com/music-assistant/hass-music-assistant/issues/2754

A new update will be available in the next 30mins which will mean you no longer need to remove the provider to get the authentication to work.

Hello.
It works again…
Many thanks for all the work…
Respect….
Regards.
JS.
Le jeu. 22 août 2024 à 10:38, OzGav @.***> a écrit :

A new update will be available in the next 30mins which will mean you no
longer need to remove the provider to get the authentication to work.
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/2784#issuecomment-2304100573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGEH4FX2RJDS7562UX3LUYLZSWPR5AVCNFSM6AAAAABM2OD5U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBUGEYDANJXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update InfluxEnterprise Third-party Libraries for 1.6 Release

URL for relevant page?
http://docs.influxdata.com/enterprise_influxdb/v1.6/about-the-project/
New third party libraries to be referenced
https://github.com/mark-rushakoff/ldapserver
Basic LDAP v3 functionality for the GO programming language.
https://github.com/mark-rushakoff/ldapserver/blob/master/LICENSE
License: BSD-3-Clause
https://github.com/go-asn1-ber/asn1-ber
ASN1 BER Encoding / Decoding Library for the GO programming language.
https://github.com/go-asn1-ber/asn1-ber/blob/master/LICENSE
License: MIT
https://github.com/go-ldap/ldap
Basic LDAP v3 functionality for the GO programming language.
https://github.com/go-ldap/ldap/blob/master/LICENSE
License: MIT

Updated in master. Appearing after next build.


Modifications needed for multi-label classification

If I want to apply this model to a multi-class multi-label classification, what changes would be required for me to make on the Mask() layer? Can you give me any hint?
Will there be any other changes that I would have to make?
Thank You.

@shan18
By default, during training, the mask is provided by y, so you needn't change the line#56.
If the accuracy does not improve, you can track the training loss to see a NaN?

Okay, understood. Thank you for your help.


[] != ["x"] evaluates to false

What version of CUE are you using (cue version)?
playground v0.4.3
Does this issue reproduce with the latest stable release?
yes
What did you do?
a: [] != ["x"]
https://cuelang.org/play/?id=33VAqhnqmrA#cue@export@cue
What did you expect to see?
a: true
What did you see instead?
a: false
PS I believe https://github.com/cue-lang/cue/blob/27b6e87efbde0d14b31eeb59943ff79b90fb92ec/internal/core/adt/binop.go line 113 should return c.newBool(true)

Nicely spotted. Duplicate of https://github.com/cue-lang/cue/issues/2042, which was filed first and has a bit more context from other users. I will get your PR merged fixing that issue instead.


Why FirebaseUser?

Seems it'd be better to expose the actual User class from pkg/firebase
No?

/shrug. I wanted to hide users from a changing firebase SDK API as much as
possible - but maybe that's just because I don't like it. Not that
opinionated myself about changing it
On Tue, Jul 4, 2017, 1:02 PM Kevin Moore<EMAIL_ADDRESS>wrote:

Seems it'd be better to expose the actual User class from pkg/firebase
No?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/matanlurey/angular_fire/issues/6, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKQ7pYebw_3Sym2aZt4ubPzBKAbJuIoks5sKppWgaJpZM4ONp7d
.


Create database from share fails

Terraform CLI and Provider Versions
Provider: 0.88.0
CLI: 1.8.0
Terraform Configuration
resource "snowflake_database" "UAT_INBOUND" {
  comment                     = "db shared by ABC"
  data_retention_time_in_days = 0
  name                        = "UAT_INBOUND"
  from_share                  = {
    provider = "ORGNAME.AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1"
    share    = "SANDBOX"
  }
}

Expected Behavior
Create the database
Actual Behavior
fails with this weeror:
Share '"ORGNAME.AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1.SANDBOX"' does not exist or not authorized.
but when I run the sql command I can create the share, the problem is when terraform is generating the SQL statement, the statement is incorrect. I pull it from query history and it is like this:
CREATE DATABASE UAT__INBOUND FROM SHARE "ORGNAME.AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1"."SANDBOX";

while it should be like this:
CREATE DATABASE UAT__INBOUND FROM SHARE "ORGNAME"."AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1"."SANDBOX";

if you see, the quote marks around (.) in the share name after the orgname is missing.
Steps to Reproduce

terraform apply

How much impact is this issue causing?
High
Logs
No response
Additional Information
No response

Hey 👋
Please, take a look here. We expect <account_locator> in the provider field instead of <org_name>.<accout_name>. Try, replacing the provider field value with the account locator, and let me know if that works.

Hi @sfc-gh-jcieslak ,
Account locator is consisting of: <org_name> and <accout_name> seperated by a dot.
If I copy paste the account locator from interface same thing gets copied.
Conclusion: I am putting account locator in the provider.

Hi @sfc-gh-jcieslak ,
If you look at this article:
https://docs.snowflake.com/en/sql-reference/sql/create-database#:~:text=Shared Database (from a Share)
it shows that the SQL should be like this:
CREATE DATABASE <name> FROM SHARE <provider_account>.<share_name>;

Since this is a cross cloud share, snowflake did an auto fulfilment and replicated data to the destination account's cloud provider.
I did extract the sql that terraform provider is generating and put it up there, if you run it, it will fail.
If you change it like the second one, it will work.
And this used to work before, because I just removed that DB from state file and added it again and it is not running, the only change was that I my provider version got updated to the latest, I am not sure when I ran it first which version I was on.
I checked my history of executions, and realized this behavior has been changed, because the last time I created a share DB, Terraform created an script like the second one (which works).
One way to make it work always is to change the generated SQL to look like this:
CREATE DATABASE identifier('UAT__INBOUND') FROM SHARE identifier('ORGNAME.AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1.SANDBOX');

this will always work.

Hey 👋 ,
I know that the documentation is proposing this format <provider_account>.<share_name>, but <account_locator> should also be acceptable as it's a deprecated format we're supporting right now (check the difference here). In the migration notes I linked previously, you can see the format <provider_account>.<share_name> won't work, because previously the provider was quoting the identifier in the wrong way (with the right way the new format is not possible as it worked previously "by accident"). We will be migrating to the new format, but for the time being, please use the account locator that can be retrieved with the CURRENT_ACCOUNT query.

Hi 👋,
I have tried what you suggested but it fails. I think the reason is overlooked, The shared database I am creating is a special kind, it is from a share in a different cloud provider, and I cannot just use the <provider_account>. To give you more details:
select current_account() returns:  the_abc_account
Share Name is: PL_TEST
We are using is new feature that Snowflake will replicate your data to the destination cloud provider without you needing to create an account and replicate your data (it is call create listing for specific customers).
For example, our snowflake is in AWS_US_EAST_1, but data that is being shared with us is in Azure, and snowflake automatically created an area and replicated data in AWS and the address to access it is: the_abc_account.AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1
if I use the_abc_account, it will fail, because I cannot see that account in the current cloud provider.
what I have to use is the_abc_account.AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1 which AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1 is the replicated copy of data in AWS that I can use.
When the provider is generating the SQL, puts a quote around the_abc_account.AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1 which makes it like this:
CREATE DATABASE SHARED_DB FROM SHARE "the_abc_account.AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1".PL_TEST

but it should be like this:
CREATE DATABASE SHARED_DB FROM SHARE "the_abc_account"."AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1".PL_TEST

or it should be like this:
CREATE DATABASE SHARED_DB FROM SHARE  identifier('the_abc_account.AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1.PL_TEST')

your suggestion is not working for reason I explained, and I couldn't find any work around for it.

Ok, now I can see the issue, thanks for the detailed explanation. I'll try to reproduce the issue and see if we can propose anything as a workaround for now.

Ok, I dug deeper into the codebase behind creating a database from share and I think the workaround that may work would be to quote only inside the provider field like this:
resource "snowflake_database" "UAT_INBOUND" {
  comment                     = "db shared by ABC"
  data_retention_time_in_days = 0
  name                        = "UAT_INBOUND"
  from_share                  = {
    provider = "ORGNAME\".\"AUTO_FULFILLMENT$PUBLIC_AWS_US_EAST_1"
    share    = "SANDBOX"
  }
}

Then it should result in SQL similar to this:
CREATE DATABASE SHARED_DB FROM SHARE "the_abc_account"."AUTO_FULFILLMENT_ABC$PUBLIC_AWS_US_EAST_1"."PL_TEST"

It's a workaround for now, but later on, we would like to fix this issue and make it work for every type of identifier (it's on our roadmap; identifiers redesign).

awesome, this is a great workaround, do you think you can change the label of this ticket to bug too so it would be in the radar? Right now, it is general-usage.

Hey 👋
Yesterday we published a new provider version with the new shared_database resource that should not have this issue anymore. Closing as this is something strongly connected how the old database resource was working underneath. Right now, the recommended account identifier is a must in the new resource and this issue shouldn't occur. Please migrate whenever you can and let us know (create another issue) if the issue still persists with your case.
Thank You for reporting the issue.


VA variable {INT:Ship cargo carried} has not worked for a long time (EDDI 2.3.0)

Main issues for VA variables:
{INT:Ship cargo carried}
Is there any possibility to check/update this variable? It would help alot.
For comparison, {INT:Ship limpets carried} has been working most of the time quite well. Fortunately :)
Additional related questions:
{INT:Ship cargo capacity}
This has been reported as an issue in other thread, and obviously it has some restrictions "when" it can give results, which is again understandable > Frontier
Anyways, it would also help alot if this could be changed,- and/or updated to work all the time if/when triggered via any VA command, cheers? :)
Is there any way to have a VA variable checking both main cargo types carried in the ship >>> Is the cargo carried a commodity,-or a passenger (boolean illegal/legal true/false)? Even a VA variable which cargo rack this type of cargo is carried?
Commodity Purchase and collection commands + variables etc. are already working great, but these additions would be super cool to have.

Daydreaming:
The absolute best thing would be similar to the currently working and fabulous -material monitor, which includes also data lists in the EDDI app. Meaning the possibility to also check directly from the EDDI app how much cargo space is available in the current ship, how much cargo is the maximum (cargo capacity), how much is currently carried (cargo carried), and which type cargo is carried (commodity or passenger) -even a VA variable to check if it is "illegal/legal" -similar VA variable than {INT:Ship limpets carried} is working,- to have the number/count + type
For example:
{BOOLEAN:EDDI Ship passenger cargo carried legal} True/False [illegal] and [Not set] no passengers, zero. {INT:Ship passenger cargo carried} = How many passengers currently in cargo?
{BOOLEAN:EDDI Ship commodity cargo carried legal} True/False (illegal) and [Not set] no commodities, zero. {INT:Ship commodity cargo carried} = How many commodities currently in cargo
These would be perfect additions into the VA ((Mission accepted)) command. I don't have any idea what/how it should look like, for the commodity types to get variable listed? Also no idea what/how it could look like for the commodities/passenger variable to get the cargo rack number (EDDI ship variables -module occupied... etc.. maybe???)
Idea (don't know if these have been suggested before?) Additionally the exact way we now have to monitor material/data, would be the ability to monitor all commodities available in game and especially for "efficient fast" mining operations with collector limpet setups, would be absolutely killer thing to have this type of monitor automatically informing when you have the desired amount of needed commodities on board. Also telling if/when you have commodities that you do not need/want with having a workin VA variables/commands to trigger commands with VA
For example, ((material refined)) -is already working really well, but again, the main problem is the {INT:Ship cargo carried} not working, and the other just partially working {INT:Ship cargo capacity} not being able to tell you when these basic info can be triggered correctly and automatically independently,-and/or inside/with another VA command such as the mentioned ((Material refined)) command, etc. considering commodities.
It would also be even greater to have additional and exact VA Material and data -variables to trigger commands when the EDDI Material monitor is giving a speech info about it's status.
For example:
((Material maximum exceeded)) + similar variable to have the number/count + type
((Material desired amount reached)) + similar variable to have the number/count + type
((Data maximum exceeded)) + similar variable to have the number/count + type
((Data desired amount reached)) + similar variable to have the number/count + type

if possible to do the commodity monitor:
((Commodity maximum exceeded)) + similar variable to have the number/count + type
((Commodity desired exceeded)) + similar variable to have the number/count + type

For the different commodities variable list, would pay real money to get them variable listed for example into the EDDI Information category, -similar way we have the Station variables, Ship variables, etc. :)
I hope you guys understand what i mean, as hard to get it all short and understandable form... huh

PS: I have read in some e-mails that cmdrmcdonald has quit developing this thing. I hope this is just a well deserved break and we'll see the master back on the saddle asap.
Standing in attention and saluting o7

Unfortunately cargo (and specifically cargo flags) are a little borked at the moment for both EDDI and VoiceAttack.
See here: https://github.com/cmdrmcdonald/EliteDangerousDataProvider/issues/258
The latest comments from the developer on the status of the cargo bug are here:
https://forums.frontier.co.uk/showthread.php/294579-EDDI-Windows-app-for-immersion-and-more?p=5726100&viewfull=1#post5726100
Hopefully either the master returns or someone else is able to pick up the mantle of supporting this wonderful program.

Thx for the fast reply.
I can easily agree keeping it only "issues" -related here. Just shooting "blanks" with daydreaming lol
Anyways, would hope we could have a somekinda suggestions tab here too. Many of us find the forums really annoying and heavy to read in general... Adding to that i scrolled around the "event" lists and actually many of my "ideas" have already implemented so i can do a lot wit existing variables already. Still i would hope for some simple words -are my suggestind "daydreams" absolutely impossible bad, or good enough for a separate thread, thx ;)
And addistionally, i sincerely hope the master is ok, and possibly even considering to do a mind blowing comeback,- or even if just one update a week, schedule....  Less is just fine too. Completely quitting is a terrible thought :/ I can only imagine the workload.
I would pay real money if only accepted, and if it would help, -without any demands for any new updates. This thing is already worth of a serious donation for the work done, meaning at least the money for the game account for sure. Without EDDI, i would've quit long time ago.
It really makes the game worth playing and not even mentioning about VA. Virtual Reality gaming is exactly what this is all about. Both Oculus and Frontier should hire the master!!!! And pay a cool million for a licence at least!
Some arrsholes have been demending stuff here quite offensively and too often, so first thing we could do is to set some serious rules for this kind of messaging. Just like everywhere else. One warning. Second offence >>> out of EDDI!! Period!
FireSammy o7

OMG moment with this issue!!!! ???? HOW and what LOLOL OMG!!! WOHOO!!!

I hope this works. Never uploaded a screenpic before ...


ImportError: cannot import name 'Self' from 'typing_extensions'

Trying to use python-docx and getting this error when I try to import it:
Traceback (most recent call last):
  File "/home/damiano/Documents/scriptbuilder/scriptbuilderui.py", line 7, in <module>
    from docx import Document
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/__init__.py", line 11, in <module>
    from docx.api import Document
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/api.py", line 12, in <module>
    from docx.package import Package
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/package.py", line 7, in <module>
    from docx.image.image import Image
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/image/__init__.py", line 7, in <module>
    from docx.image.bmp import Bmp
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/image/bmp.py", line 3, in <module>
    from .image import BaseImageHeader
  File "/home/damiano/.local/lib/python3.10/site-packages/docx/image/image.py", line 14, in <module>
    from typing_extensions import Self
ImportError: cannot import name 'Self' from 'typing_extensions' (/usr/lib/python3/dist-packages/typing_extensions.py)

relevant code looks like this:
from docx import Document

doc = Document()

and yes, i installed the correct module, via pip install python-docx
I'm on Linux Mint 21.3 and Python version 3.10.12. sudo apt-get update and sudo apt-get upgrade do not turn up any new versions of python.

I had the same problem when switching from Windows to Linux.
When I tried to install python-docx with pip, python-docx 1.1.0 and typing-extensions 4.8.0 were installed. After updating to typing-extensions 4.9.0, python-docx worked again under Linux.
I hope this procedure can also solve your problem.

Yeah, apologies for that. typing_extensions really belongs in the install requirements.
In the meantime, pip install typing_extensions should get it working.

Thanks, I was able to get it working by upgrading typing-extensions as Cronexas said, via pip install typing-extensions --upgrade

Fixed in v1.1.1 circa Apr 30, 2024.


deps: turn ts-node and tsconfig-paths into peer dependencies fix #6

fix #6

@RebeccaStevens could you take a look if it fix #6 ?


Support doc comments in definition files

Blocked on https://github.com/Roblox/luau/issues/817

We need to provide a way to specify definition files names for this to work correctly.

We are still missing location information. Blocked on https://github.com/luau-lang/luau/issues/1137


Hello, I used opecv=4.6 to compile pydbow3 can pass, generated but when importing pydbow3 as bow, I get an error import pydbow3 Traceback (most recent call last):   File "", line 1, in  ImportError: /home/xxx/miniconda3/envs/test_env_python39/lib/python3.9/site-packages/pydbow3.cpython-39-x86_64-linux-gnu.so: undefined symbol: ZN2cv8internal18WriteStructContextC1ERNS_11FileStorageERKSsiS5

Hi,
Very sorry for my late reply. It is not so easy to understand what the issue is here. Did you use cmake between 3.4...3.18 to compile pydbow3?


Incorrect rendering in Anno 2070 (DX10 mode)

Anno 2070 is rendering incorrectly when launched in DX10 mode in wine-vulkan with DXVK
Screenshots: https://imgur.com/a/shtjU
When map is zoomed out, whole screen is covered with textures, ocean is black unless fully zoomed in.
Software information
Anno 2070 with Deep Ocean addon and all DLCs.
Screenshots were made with Very High preset, post processing was set to lowest. Game was also tested with all settings set to low, with the same results.
System information

GPU: GTX 660 (2GB RAM)
Driver: 390.25
Wine version: wine-vulkan 3.3, built from source with --without-opengl option
DXVK version: latest from git.

Apitrace file(s)
https://drive.google.com/open?id=1yr85HINg9XtCX8-yoiXRKvhEx_s0rMYB
Log files

d3d11.log: https://pastebin.com/BMqhmgCz
dxgi.log: https://pastebin.com/FQewxBiB

The apitrace you attached renders fine on my RX 480. Can anyone reproduce this (preferably on Maxwell, Pascal)?

The Apitrace you linked runs perfectly on my pascal GTX 1050.  The issue is probably with the limited vulkan support on the 600 series.

@pingubot  @Guy1524  thanks for testing. Low performance when replaying traces is normal and does not reflect actual ingame performance. AFAIK apitrace decompresses the trace file on the fly, which costs quite a lot of CPU power.
@Shished Can you replay your trace with DXVK_DEBUG_LAYERS=1 set? I'm getting some complaints that a fragment shader doesn't write to all render targets, but that's it. If you see more errors, please let us know.

https://pastebin.com/raw/qfmuv6gN

It seems to also happen on Maxwell 1.0

@pdaniell-nv Can you help with this narrowing this glitch in any way?

Nothing immediately comes to mind. Note that the "bufferImageGranularity" physical device limit is quite different between Maxwell 1.0 and Maxwell 2.0/Pascal. Any chance something is violating that limit?

Hi I'm getting the same glitches when replaying your dump... also have glitches in witcher3 and in skyrim
I`m having a geforce gtx750ti

@pdaniell-nv That might indeed be a problem since my allocator does not maintain separate pools for images and buffers. Commit 42d49d7603401d662153b9fc4d94917f62c2bd92 should fix these violations.
Basically, that workaround enforces the value of bufferImageGranularity as the alignment for all images with VK_IMAGE_TILING_OPTIMAL. I'll see if I can alter the allocation strategy in the future since this will lead to more fragmentation and memory consumption on the affected GPUs. More recent GPUs should be practically unaffected by this change.
@nairaner @Shished please test with the aforementioned commit.

tested the replay with the commit - great work! no glitches so far!
Will test my games now!

well also tested the witcher3 and hat not a single glitch! in all versions before somtimes glitches occured. The downside is that now every second or so I have the feeling that the picture "jumps" some frames kind of its stuck for an just barely feelable moment and then jumps on.

Looks like the original issue is fixed.

Can confirm. Rendering glitches gone.
https://i.imgur.com/b0onE4E.jpg


Add native-base

docs: https://docs.nativebase.io/

It might make sense to replace at least one component w/ a native-base one, just to demo?  (rather than a PR dedicated to just adding the dependency)


brunch watch does not check if stdin was closed

I would like to automatically run brunch watch as part of my application but, when my application shuts down, the brunch process persists and continue running. I have observed that my application does close STDIN, however, brunch watch is not checking if STDIN was closed or not. Those two lines solve the issue:
process.stdin.on('end', function() { process.exit(0) });
process.stdin.resume();

I would love to send a pull request for this, the question is: should we always run this or should we make it opt-in?

You're running it as a child process? http://nodejs.org/api/child_process.html#child_process_child_kill_signal

I am not running it using node. I am running on the Erlang VM which does not expose the OS Process API as node.js does (in fact, neither does the Java VM last time I checked. It is common for VMs to not expose those operations as it is hard to provide an unified API across multiple OS and hardware).
Closing the stdin as a signal is a very common mechanism in Unix utilities. Given how widespread Unix ideas are in node.js, I am surprised this behaviour isn't more common. :) Here is one example from our beloved cat:
$ cat -
hello
CTRL-D

node supports it too:
$ node
> 1 + 1
2
> CTRL-D

After this patch, brunch watch would support it too. :)

Another reason for listening to stdin is because the OS will eventually close it if the parent process dies, guaranteeing safe termination of the child process. Relying on kill signals can be very dangerous because, if the parent process dies due to an unexpected reason (segfault, no memory, etc), it may leave orphan child processes.

Sending Unix signals is pretty standard too. Like any other node program, Brunch does what it is supposed to do when sent a SIGTERM or SIGINT.
That said, if this remains problematic for you, go ahead and open the PR. I'm not sure it needs to be behind an option - can't think of where else this may affect someone (that they close stdin and want it to keep running)

PR incoming!


Allow for “root-only” router apps

resolves #970
resolves #968
resolves #979

@rajinder-yadav Can you review this one quickly please? I want to make the new release as soon as possible.


Redesign

A small patch (#27) for the design was added recently, but there's still a lot of work we could do. For example, the author, paper and search result pages are not pretty as is, so they could be worked on.

Front-end is Oliver-end. :)


remove package-lock and gitignore it

It still does not work as expected, better not to use it still.

Done


Fix Telemetry Fail

Fixes a problem where launching the Change Image UI from the All Courses view would fail the telemetry request.

Code LGTM, I'll give it a quick spin for test porpoises.

https://www.youtube.com/watch?v=Lj0ehBeCu0c


How to know Graph A is correct or not?

Code version (Git Hash) and PyTorch version
Dataset used
Expected behavior
Actual behavior
Steps to reproduce the behavior
Other comments
When running Graph.py independently to analyze A value, I got some values with dimension [3,18,18] only in diagonal. So, how to know these values are correct or not?

@JasOlean,
Matrix A is transformed from a graph composed of some edges defined on graph.py.
Some third-party packages such as networkx may help you to visualize these edges.
If you implement this function, welcome to submit a pull-request.


python3Packages.sqlalchemy: 1.4.40 -> 1.4.41

https://github.com/sqlalchemy/sqlalchemy/releases/tag/rel_1_4_41
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build python3Packages.sqlalchemy
@ofborg build pythonPackages.sqlalchemy

@ofborg eval

@ofborg build python3.pkgs.sqlalchemy


Challenge http://www.freecodecamp.com/challenges/waypoint-link-to-external-pages-with-anchor-elements has an issue

Please tell us in detail here how we can make this challenge better.
This challenge accepted :  cat photos >
as an answer.

This is not what I wrote in the FreeCodeCamp bug dialog box, I have since fixed it but I believe it was
"cat photos >" that it allowed.

Thanks for the report. This issue has been raised before and addressed in a commit, so closing it.


feat:  Update connectionManager to  allow configuration of incoming and outgoing connection limits

Closes https://github.com/libp2p/js-libp2p/issues/1508

Would we also want an Outbound connection threshold  as well or would that be left up to the discretion of the user?

I've updated the docs should be ready to review now.

I also decided to include https://github.com/libp2p/js-libp2p/issues/1515 in this PR as well here

It may not actually be useful to have a minimum inbound/outbound connection limit. In the case of a min inbound connection limit, this could force the connectionManager into a endless loop of dialing connections where that inbound limit is never met since each new connection would be an outbound connection. We should just have a minimum connection limit.

It may not actually be useful to have a minimum inbound/outbound connection limit. In the case of a min inbound connection limit, this could force the connectionManager into a endless loop of dialing connections where that inbound limit is never met since each new connection would be an outbound connection. We should just have a minimum connection limit.

This has been refactored. Ready for review @achingbrain

I also decided to include #1515 in this PR as well here

@maschad any reason for the force-pushes? I can't see what that commit references.
Also this is ready for review right, @mpetrunic would you be able to help review (requesting out of caution since not sure if achingbrain will get to it with Helia work)

@maschad any reason for the force-pushes? I can't see what that commit references.
no particular reason than a bad habit, will avoid in the future.

I decided to move #1515 to https://github.com/libp2p/js-libp2p/pull/1564 since we are deliberating how to approach this regarding eclipse attacks

Please can you revert all of the formatting changes, there are many and they make it easy to miss the actual changes here. I've marked the ones I've spotted, there may be more.

I've reverted those changes, I believe we can still go ahead with the changes as explained here so should be ready for review now.

Thanks for pointing out those issues @mpetrunic I've made the updates. I would be interested in @achingbrain 's response to https://github.com/libp2p/js-libp2p/pull/1511#discussion_r1074008311 otherwise we should be good to go.

Closing this as discussed with @achingbrain and @wemeetagain as consumers may have an idea of the maxConnections they want to maintain ahead of time but may only have a notion of a ratio of inbound to outbound as opposed to a fixed number. This could lead to issues with unwanted pruning of connections. For consumers who want this level of granularity they wish to pursue a custom ConnectionManager


anchor tag is not working

Challenge Link to External Pages with Anchor Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 If you get stuck with a challenge, please ask questions to the Help Room. Hint: read the instructions carefully again and make sure you're following them. Happy coding!


Remove token header after validation

This PR removes API tokens from HTTP/gRPC header and metadata values respectively after the validation has occured.
This remedies the fact that today these secure values get sent over the write to tracing backends as part of the request.

#2974


fixed gitlab error typo: gilabClient -> gitlabClient

Problem Statement
What is the problem you're trying to solve?
Just a small PR to fix a typo in an error message.
Proposed Changes
fixed a typo in an error message
How do you like to solve the issue and why?
fixed the typo because the typo bothered me when i'm already bothered troubleshooting :)
Checklist

[x ] I have read the contribution guidelines
[ x] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

Thanks!


No query error information is returned beyond 500 using elasticsearch.net and NEST

NEST/Elasticsearch.Net version: 2.4.6
Elasticsearch version: 2.4.1
Description of the problem including expected versus actual behavior:  Sending in a query with an error returns a 500 response with no additional information.  When using a simple browser post the full detail is returned allowing the problem to be solved.
Steps to reproduce:

Submit an aggregation date range query without providing the date field and only range data

Elasticsearch will return:
{
  "error" : {
    "root_cause" : [ {
      "type" : "aggregation_execution_exception",
      "reason" : "could not find the appropriate value context to perform aggregation [startDates]"
    } ],
    "type" : "search_phase_execution_exception",
    "reason" : "all shards failed",
    "phase" : "query",
    "grouped" : true,
    "failed_shards" : [ {
      "shard" : 0,
      "index" : "project",
      "node" : "dBN8eDolTL69Juu95v7Ugg",
      "reason" : {
        "type" : "aggregation_execution_exception",
        "reason" : "could not find the appropriate value context to perform aggregation [startDates]"
      }
    } ]
  },
  "status" : 500
}

And this is available on ServerError
response.IsValid.Should().BeFalse();
var serverError = response.ServerError;
serverError.Should().NotBeNull();
serverError.Status.Should().Be(500);
serverError.Error.Reason.Should().Be("all shards failed");
serverError.Error.RootCause.First().Reason.Should().Contain("[startDates]");

I've attached a commit to this issue that showcases this.


Founding KubeVirt Common Language

Inconsistency in our terminology is making the latest round of proposals difficult to process (at least for me). For example, I think I understand what's being proposed, but there is enough ambiguity that it makes me uncertain I'm really following the the full intent.
Lets use this issue to solidify a set of common terms we can all agree on.
I tried to get us started below. I'll update the glossary.md with the results.
VM Spec:  ... something to do with libvirt domain represented in kubevirt objects.
VirtualMachineConfig: A cluster object that represents a static VM spec. Configs can be used to launch a VM Instance.
VM Instance: A running virtual machine
VM: A cluster object that represents a running VM Instance. This objectcContains the VM Spec used to launch the VM Instance.
VMPreset: A cluster object that ...
VM Spec Completion: The action of filling in a sparse VM specification through VM Spec defaulting, VMPreset injection, and libvirt domain defaulting
VM Flavor:
VM Spec Defaulting: The action of ...
Libvirt Domain Defaulting: The action of ...
VMPreset Injection: The action of ...
.
.
.

A few additional notes:

VirtualMachineConfig is a static VM representation. It is called static, because it does not contain any runtime (or you could say dynamic)  data. The config contains a temaplte which is used to create VM instances.
VM (soon eventually VirtualMachineInstance) represents a running Virtual Machine. It is the most detailed object when it comes to representing current runtime VM configuratoin. On the one hand it shows the current specification which was used to create the VM, on the other hand (in .state) it will show the actual state, including computed values like device addresses.
VMPreset (not yet defined) are objects which hold partial (more or less) VM configurations, which are applied to VM up on creation. This mechanism allows a user to apply certain partial configurations to all VMs matching specific labels, this can be used to achieve something like flavours. The exact semantics need to be defined.
Expansion (not yet defined) is the process of adding (device or attribute) dependencies to a VM definition to generate a consistent spec. This could be performed on the API level but is also performed by libvirt.
Defaulting (not yet defined) is the process to decide which values to use during the expansion step. Defaulting is an inherent part of the expansion process. But we should view it as a separate step.
VM Template (not yet defined) are representing a VM definition and also referencing the used resources (i.e. disks). Instantiating a VM from a template would lead to cloning the definition as well as the referenced resources. (Definition from oVirt)
Instance type or flavour (not yet defined) define specific VM defintions (only covering the definition, not the data). An existing disk can then be started with a certain instance type. I.e. You got an nginx disk, which you first started with a small instance type, but then switched to huge because of a large amount of sessions. The disk did not change, just the vm definition. (Definition from oVirt.

/cc @michalskrivanek @berrange @rmohr

Can we move to a google sheet? I don't think it makes sense that we all start posting all our slightly different opinions here :)
https://docs.google.com/a/redhat.com/document/d/19qgN8izk9w227T9S5tRCPZvaB1Xngy718FzdFfeY3g0/edit?usp=sharing

Created a public accessible doc: https://docs.google.com/document/d/12X3bwBb48GIx52g9E4DK9MiDEMH2IJKTdzxXMlG6Lng/edit?usp=sharing

Additionally, there is a great RFC that defines keywords such as MUST, SHOULD, MAY - https://www.ietf.org/rfc/rfc2119.txt. It would be helpful to follow it instead of using ambiguous "can".

… the issue with the outsourced doc is that it's hard to follow any development.

@rmohr @davidvossel where do we stand?

@fabiand @rmohr I'll follow up by sorting through everything we have so far and formalizing it into a patch for glossary.md. That will centralize the discussion and hopefully give us a good foundation for future (as far as definitions/terms go)


Add an example for az keyvault set-policy

There is example for some of the other commands, but not az keyvault set-policy, and it's not clear what the format should be, and I had to go to another page to find an example.
for example:
az keyvault set-policy --name MyKeyVault --object-id $MyPrincipalId --secret-permissions list get

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

ID: 226e5769-f6c8-a480-db78-727fb96348ec
Version Independent ID: 8ee06a2a-6eda-11be-3c8e-6ad948be207e
Content: az keyvault
Content Source: src/azure-cli/azure/cli/command_modules/keyvault/_help.py
Service: key-vault
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

hi @bim-msft could you pls add the example? thanks.


Fix location of ** in README

This PR fixes the README to contain the bold within the link.

Closing for lack of contribution context


Segmentation fault

System Information (please complete the following information):

Model Builder or CLI Version:
Visual Studio Version (if applicable):

Describe the bug

On which page of the tutorial did you run into an issue (URL): https://dotnet.microsoft.com/en-us/learn/ml-dotnet/get-started-tutorial/install
Clear description of the problem: I followed the instructions and the Installation was succesful. Then everytime i tried to use 'mlnet' command or '~/.dotnet/tools/mlnet' I get segmenation fault.

To Reproduce
Steps to reproduce the behavior:
Install mlnet following the manual
use the two commands I mentioned earlier
Expected behavior
I expected to
Screenshots

Additional context
Add any other context about the problem here.

@LittleLittleCloud Can you look at this?

What is mldonkey server? Is it possible this is related to this thread - https://mldonkey.sourceforge.net/forum/viewtopic.php?t=5156?


Latest HP BIOS Updates not downloading

Looks like 22H2 case is missing at line 14843 and 21H2 is set for 20H2.
Be advised, the "h" has to be lower case!
				switch -Wildcard ($OS) { 					"*22H2"	{ 						$OS = "10.0.22h2" 					} 					"*21H2"	{ 						$OS = "10.0.21h2" 					}

Which file are you refering to?

He is talking about "DriverAutomationTool.ps1", under the Find-HPBIOS function.
				# Windows Build Driver Switch 				switch -Wildcard ($OS) { 					"*21H2"	{ 						$OS = "10.0.2009" 					} 					"*21H1"	{ 						$OS = "10.0.2009" 					} 					"*20H2"	{ 						$OS = "10.0.2009" 					}
Should be:
				# Windows Build Driver Switch 				switch -Wildcard ($OS) { 					"*21h2"	{ 						$OS = "10.0.2009" 					} 					"*21h1"	{ 						$OS = "10.0.2009" 					} 					"*20h2"	{ 						$OS = "10.0.2009" 					}

Coulden't find bios'es for newer HP models like HP ProBook 445 14 inch G11 Notebook PC using $OS = "10.0.2009"
Find				# Windows Build Driver Switch
switch -Wildcard ($OS) {
Then add below:
"*23h2"	{
$OS = "11.0.23h2"
}


DataTable: Adding class to  assigns it to child element.

Detailed description

Is this issue related to a specific component?

DataTable

What did you expect to happen? What happened instead? What would you like to see changed?

When adding a class to TableHeader, I expected it to add the class to <th>, not the child element.
Additional information

Issue moved to carbon-design-system/carbon-components-react #1026 via ZenHub


Fix VSS Vintorez having a flash

📝 Description
Adds the changes by MCT32 in PR #308. Everything is copy-pasted form there PR
🎯 Goals
This change simply makes the gun behave more realistically.
❌ Non Goals
Fix eveything
🚦 Testing
https://private-user-images.githubusercontent.com/32090502/284242502-13e4796b-0fc1-4528-9657-91eb5d74b45e.mp4?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDQ0OTU4MjQsIm5iZiI6MTcwNDQ5NTUyNCwicGF0aCI6Ii8zMjA5MDUwMi8yODQyNDI1MDItMTNlNDc5NmItMGZjMS00NTI4LTk2NTctOTFlYjVkNzRiNDVlLm1wND9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAxMDUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMTA1VDIyNTg0NFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZiMjFjOGQwNjUwOWFjMTA2YjhiMzA1ZGRhOWZlOWMzNWRiNmNhNzFjOWM2YzViN2RjNDYxOThlNDFmN2FkYmQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.TttuWhpe-WYhJAzrlf-lzLy8zDdFC4SjM7GH4DWCYbA
⏮ Backwards Compatibility
1.12.2
📚 Related Issues & Documents
Fixes https://github.com/Cubed-Development/Modern-Warfare-Cubed/issues/284
📖 Added to documentation?

[ ] 📜 README.md
[ ] 📑 Documentation
[ ] 📓 Javadoc
[ ] 🍕 Comments
[x] 🙅 No documentation needed

Summary by CodeRabbit

Bug Fixes

Fixed custom skin system and ResourceLocation errors.
Resolved fog rendering and world post-processing issues.
Addressed missing language entries and animation glitches.
Corrected tooltip formatting and inventory positioning.
Fixed exposure decrease during gameplay.
Resolved issues with spectator weapon use.
Corrected shell and bullet spawning calculations.

Improvements

Removed flash effect from VSS Vintorez for better realism.
Implemented various performance optimizations.

Documentation

Updated the changelog with recent fixes and improvements.

It is already included in https://github.com/Cubed-Development/Modern-Warfare-Cubed/pull/312


NodeJS build

Build only supports web browsers, need to use the wasm-pack bundler option to build for both node and browsers

will need to support a different nats client library (nats.ws is used for browser, nats.js is used for nodejs). there is a shim to use the nats.ws library for node for the websocket but will require webpack compiling....looking for cleaner way


Cannot run exe from wsl2 after enabled systemd support

Windows Version
Microsoft Windows [Version 10.0.22621.1265]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Enable systemd support on WSL2. Following instruction here

Run any Windows applications on WSL2. e.g. type:

explorer.exe .

Expected Behavior
Expected to open explorer.exe on Windows
Actual Behavior
There is an error message:
run-detectors: unable to find an interpreter for /mnt/c/Windows/explorer.exe

Diagnostic Logs
No response

Hello Sir, maybe this is a duplicate from my previous error. Take a look at this Issue

Your binfmt_misc registration for interop have been overwritten. You can use sudo update-binfmts --disable [NAME-HERE] to disable whatever is overwriting this. Example of a problematic interpreter here:


Identifiers validation failed for dataset DFO Maritimes Region Cetacean Sightings

Identifier validation failed for the dataset DFO Maritimes Region Cetacean Sightings:

Crawler attempt: 321
Publishing organization: Fisheries and Oceans Canada
Publishing country: CANADA
Installation: OBIS Canada
Number of occurrences indexed for this dataset at the time of the issue: 49619
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 65169 total records; 65169 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3877883/step

New IDs sample:

Old IDs sample:

DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216675
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216676
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216677
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212033
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213932
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213933
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213934
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213935
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212021
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212035

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the DFO Maritimes Region Cetacean Sightings : https://doi.org/10.25607/xjo4dj.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset DFO Maritimes Region Cetacean Sightings:

Crawler attempt: 322
Publishing organization: Fisheries and Oceans Canada
Publishing country: CANADA
Installation: OBIS Canada
Number of occurrences indexed for this dataset at the time of the issue: 49619
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 65169 total records; 65169 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3878397/step

New IDs sample:

Old IDs sample:

DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216675
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216676
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_216677
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212033
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213932
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213933
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213934
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_213935
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212021
DFO-Maritimes-Region-Cetacean-Sightings_occurrenceID_212035

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the DFO Maritimes Region Cetacean Sightings : https://doi.org/10.25607/xjo4dj.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Module revolt.py not found in repl.it

https://media.discordapp.net/attachments/731118648126603274/948144352146898954/unknown.png?width=501&height=417

make a .replit file in your repl, and paste this to it:
language="python3"
run = """
pip install revolt.py
python main.py
"""

[packager]
ignoredPackages=["revolt.py", "revolt"]

should work.

Repl.it uses Python 3.8 while revolt.py requires 3.9+.


Prettier Added to Lint and Fix on Existing Files

This is a rather large change to existing files. Could look into another setup which might offer more customization as the current package for prettier doesn't offer really any options from the default settings. Looks like the more recent code has 2 space indentation which matches prettier but some of the older stuff was done with 4 space indentation. The other thing that I found unusual is the semi colons at the end of nested functions. @shiffman let me know what you think. I'm happy to look int other options for formatting.

@shiffman what are your thoughts? May we merge this PR?

@violetcraze are you able to resolve those conflicts?

Opened a new pull request because it was easier than trying to rebase and then run fixes again.


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 1.15.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: