Implement CBOR as an optional alternative transport format

The standard HTTP Accept header mechanism is used to negotiate the transport format between the client and server.
JSON is assumed to always be an option, while CBOR is used only if available. The cbor2 module is used for CBOR support, since it seems to be actively maintained and has an interface nicely matching that of the built-in JSON module.
A major benefit of allowing CBOR is to efficiently handle binary data, but uniform operation with JSON is important as well. To that end, a simple representation is added for the python bytes type when encoding to JSON, replacing it with a dictionary containing a key "bytes" which maps to a list of integers containing the byte values. This is chosen to match the encoding used by nlohmann/json. The client it also extended to recognize this structure in JSON data and synthesize it back into a bytes object (also permitting, but currently ignoring the "subtype" key in case this might be used for compatibility with formats which have binary subtypes).

So, you'll see a failing test case here:

tuber_call = <function tuber_call.<locals>.tuber_call at 0x7f6e3d2618a0>

    @pytest.mark.orjson
    def test_numpy_types(tuber_call):
>       assert tuber_call(object="NumPy", method="returns_numpy_array") == Succeeded([0, 1, 2, 3])
E       assert {'error': {'m...y.ndarray'>"}} == {'result': [0, 1, 2, 3]}
E         Left contains 1 more item:
E         {'error': {'message': 'CBOREncodeTypeError: cannot serialize type <class '
E                               "'numpy.ndarray'>"}}
E         Right contains 1 more item:
E         {'result': [0, 1, 2, 3]}
E         Full diff:
E           {...
E
E         ...Full output truncated (8 lines hidden), use '-vv' to show

tests/test.py:267: AssertionError
=========================== short test summary info ============================
FAILED tests/test.py::test_numpy_types[cbor] - assert {'error': {'m...y.ndarr...
============== 1 failed, 86 passed, 1 skipped, 6 xfailed in 2.54s ==============

This is the short-circuit specialization for Numpy arrays, which is one of orjson's biggest selling points. I think we'll have to figure out a similar shortcut for orjson.

Sorry I missed that test failing. Adding a shim for that should be fairly simple, but should I include it all the time, or only when orjson is in use? Either way things are a bit strange with numpy arrays being only conditionally supported. I guess I could truly fix that by also adding the shim for them to JSON when using other JSON modules.

We really do want CBOR to be able to serialize Numpy ndarrays as directly as possible. This test is conditional because we control the server-side environment, and can guarantee orjson is present whenever we care about this style of c++ code.

It probably does make sense to add the ndarray hooks to both CBOR and JSON (put another way, I think you'll get JSON "for free" once you've done CBOR.)
The bytes situation is a little different: the CBOR and JSON implementations will return slightly different objects that will have to be reconciled in client-side code. I suspect it would be a mistake for tuber.py (or equivalent) to deeply inspect the object, in an attempt to reverse these transformations automatically. In other words, for code like this, we'd accept that client code would probably only work with JSON or CBOR, or would be explicitly coded for compatibility.
That's fine: I suspect only a small subset of server-side code would produce binary data, and that you'd care so much about performance that JSON wouldn't be appropriate anyways.

The latest addition to the patch should fix the failing test by adding specific support for numpy arrays when using CBOR. So far, only arrays of numbers which have corresponding RFC 8746 Typed Array tags defined, as these are particularly efficient to support (no conversion of the array data is required). This could be extended to other data types by using Homogeneous Arrays, but that has not yet been done. Likewise, only arrays backed by contiguous memory regions are currently supported, since their data can be handled in a single operation.

Great - this now passes tests on my machine (github actions seem to be refreshing their apt repos, so the CI runner fails.)
Long term, I would love it for cbor2 to take over this functionality, but this now appears mergeable to me. Are you content to merge?

Merging is fine with me.

Thanks again, Chris, really nice work - I've created an v0.10 release tag to match.


Addition of break and continue statement

@sumn2u  Could you please assign me this task?

asssign it to me please


Vulnerability roundup 112: consul-1.10.3: 1 advisory [6.5]

search, files

[ ] CVE-2022-24687 CVSSv3=6.5 (nixos-21.11)

CVE details
CVE-2022-24687
HashiCorp Consul and Consul Enterprise 1.8.0 through 1.9.14, 1.10.7, and 1.11.2 has Uncontrolled Resource Consumption.

Scanned versions: nixos-21.11: efea022d6fe.
Cc @kalbasit
Cc @nh2
Cc @pradeepchhetri
Cc @vdemeester

PR #172618


feat: support network proxy from environment

In current implementation, rust-releases-io will just ignore HTTP_PROXY/HTTPS_PROXY
When users cannot access github without a proxy, this results an error like:
HTTP error: https://raw.githubusercontent.com/rust-lang/rust/master/RELEASES.md: Connection Failed: Connect error: Connection refused (os error 111)

With ureq:Agent, rust-releases-io will try to read proxy from environment
This avoids requirement of hook-based/cgroup-based/tun-based e.g. proxy tools

Could you please rebase your branch? I've fixed the pipelines in #246. Thanks, also for the PR!

Could you please rebase your branch? I've fixed the pipelines in #246. Thanks, also for the PR!

Rebased. :+1:

Thanks!


Cucumber Pro was renamed to Cucumber Jam

Rename in the docs:

Reporting page
Other locations?

Maybe we can remove it from the docs? It's kinda weird to have all this open source documentation for a feature that is not available.

Valid point. Afaik Jam has it's own docs? We could still link to it though?

Sure. We have this section were we link to other tools right?

Yes, we have the Related Tools page: https://docs.cucumber.io/tools/related-tools/
But also the Reporting page, which lists several built-in plugins, Cucumber pro (=> renamed to Cucumber jam), and third-party plugins: https://docs.cucumber.io/cucumber/reporting/

We should really check the third-party plugins also for how up to date they are.

As discussed with @tooky at Cukenfest the Cucumber Pro section can be removed.

Closed by #329


Update data-classification-activity-explorer.md

@chrfox customers are asking why we have not documented the fact that Activity Explorer only reports on maximum 30 days worth of data. Hence adding it to the documentation

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@chrfox Please merge this PR. Thanks


Incorrect number of autocomplete choices

https://github.com/Discord-Net-Labs/Discord.Net-Labs/blob/c5b8d07378c7a8f277d5d7df1035809d6b2603e8/src/Discord.Net.WebSocket/Entities/Interaction/SlashCommands/SocketAutocompleteInteraction.cs#L83
25 choices are supported. autocomplete options follows the Application Command Option Structure

Adding to the report: exact bug location
InteractionHelper.cs
public static Task SendAutocompleteResultAsync
Preconditions.AtMost(result.Count(), 20, nameof (result), "A maximum of 20 choices are allowed!");


fix: Allow traces inside jobs while avoiding Redis noise

Previously, I attempted to removed the patch for Sidekiq::Processor#process_one altogether in an attempt to allow for traces inside jobs without having :trace_processor_process_one enabled. Doing this brought up an issue with the Redis instrumentation, which produced a new trace everytime a Sidekiq worker polled for a job, creating a lot of low-value traces. Additionally, some people expressed concerns about removing the :trace_processor_process_one option.
This allows for traces inside jobs by default while keeping all other behaviors the same. The :trace_processor_process_one is still there and Redis won't produce noise when polling for jobs.
See #545 for more info.
Screenshots
With propagation_style: :link

With propagation_style: :child

@open-telemetry/ruby-contrib-approvers  It looks like we have some unrelated CI builds failing due to issues with Actions. Do these errors look familiar to y'all?
@kevinnio Thank you again for your contribution. I really appreciate your flexibility here and your willingness to continue submitting PRs.
I would like to request that you add some automated test coverage for this change before merging this PR.

I believe the test failures will be addressed by https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/581/files, once it's merged into master.

Thanks, y'all. Now that specs are fixed I'll add some of my own to cover the new patch.

@arielvalentin Added some specs to test the new patch.

Specs failed with a segmentation fault. Probably a re-run would fix them.

curses! rdkafka 0.12 and your blasted segfaults!!!


List all Inventory Entries by Condition

As an Inventory Manager
I need to list entries based on condition (new/open box/used)
So that I can list all entries based on their condition
Details and Assumptions
We will store the inventory details in a database
Base URL should be /inventory/:condition
Acceptance Criteria:
Given the service is running
When I call the '/inventory/:condition' with an HTTP GET method
Then I should be able to see the list of entries corresponding to the condition
If the condition does not exist then it should return an appropriate message to the user.

Could be duplicate.

Note: this issue is duplicated to issue #24


SQL Managed Instance diagram misspelled "feature"

[Enter feedback here]
In the SQL Managed Instance diagram, under the "Fully managed PaaS", the word "freatures" should be corrected to "feature".

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

ID: 28ca1f7f-5a34-0f23-10ce-3ff24c76db97
Version Independent ID: 5c4a22c4-78fc-911d-9fa0-8dfab1653b6e
Content: What is Azure SQL Managed Instance? - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/sql-managed-instance-paas-overview.md
Service: sql-managed-instance
Sub-service: service-overview
GitHub Login: @niko-neugebauer
Microsoft Alias: nneugebauer

Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will update the document as appropriate.

@Drkjr92 Thank you for the feedback.
This is being assigned to the content author to evaluate and update the image accordingly.

Thanks @Drkjr92, nice catch! We'll have this image updated right away.
#reassign:williamdassafMSFT
#please-close


⚠ Bofrost.de has degraded performance

In d568784, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1142 ms

Resolved: Bofrost.de performance has improved in 03c1e43.


Support image registry path rewriting in containerd

Is your feature request related to a problem? Please describe.
I am attempting to mirror a repo to a specific namespace within an image registry. ie docker.io/ ---> git.companyx.com/Group/Project/ .
Describe the solution you'd like
This feature has been solved on under rancher rke2 (https://github.com/rancher/rke2/issues/741) by exposing the rewrite option in the registries.yaml config file.
Example:
---
mirrors:
  docker.io:
    endpoint:
      - "https://my.registry.com"
    rewrite:
      "^rancher/(.*)": "testing/maxtest/$1"
configs:
  "my.registry.com":
    auth:
      username: fakeuser
      password: fakepass
    tls:
      ca_file: /etc/docker/certs.d/my.registry.com/ca.crt

Describe alternatives you've considered
There are some alternatives that involve using a nginx proxy to rewrite the path on request but increase complexity and adds more external requirements.
Additional context
If you run crictl info the rewrite flag exists but is null.
    "registry": {
      "mirrors": {
        "docker.io": {
          "endpoint": [
            "http://gitlab.local:5050/local/docker"
          ],
          "rewrite": null
        },
        "gitlab.local:5050": {
          "endpoint": [
            "http://gitlab.local:5050"
          ],
          "rewrite": null
        }
      },

Backporting
No back porting is required as current versions ignore the rewrite key in registries.yaml.

This feature is also available on K3s, and has been since the initial release in the 1.21 series. Have you restarted K3s to apply the changes to docker's config.toml? The mirror and auth settings in your crictl info output don't match the config you provided either. It's also possible that crictl doesn't understand the rewrite field, since it is unique to our fork of containerd.

Thanks for the prompt response @brandond. I tried searching issues for it. I'm currently using 1.19. Will upgrade.

I don't see rewrite usage in document.

I don't see rewrite usage in document.

I wasn't able to find it in the documents either but was able to confirm it is in there by looking at pull requests. Also had an example of how it works.
https://github.com/k3s-io/k3s/pull/3064


bugfix: remove texts

Description
Remove the texts and instead use <CoreTypographyBody1> for the texts.

moved to wrappid-app


[Bug] Crash when tapping the scrollview while refresh is happening

When the scrollview is loading, if you tap on a cell or attempt to scroll, you will get a fatal error with because of an index that's out-of-bounds

Taking this issue


Image Classification using cnn

Related Image Classification using cnn

I have created Image classification using CNN

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
The idea behind this is that our model will be able to classify the image of male and female with accuracy of above 91 % . As it is a Automatic attendance system the system should also be capable to classify the students by sending the images to our neural network and by that NN will classify the images .
->Added Image-Classification.py file the CNN file
->Added network_image,json file of the CNN model such that it can be used either on website or an application in  short words it is ready for deployment
->Added the Dataset that has been used for training as well as testing
->Added the weights.hdf5 the weight of the CNN
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

@ParthSingh0506 can you please create this pr in a separate branch instead of master directly ,  make a branch with your name and then create a pr for this

@Swarnimashukla I have created this PR on separate branch not directly on the main branch.


PS5.3.2 don’t sum the singular values

In other words I'll fix it later today by pushing that PR with additional improvements.


Review account docs comments

Fixes https://github.com/ChainSafe/lodestar/issues/1229

Fix account docs directory name holding validator keystores
Don't use $rootDir in docs, but its default value
Clarify the role of --name option in 'lodestar account wallet create'

It would still be nice to use custom directory instead of default

It would still be nice to use custom directory instead of default

Right, we should definitely explain the --root-dir and other options in the docs.
I think we could expand the docs to include something like "Advanced configuration", where we cover all of these options not directly related to running the lodestar instance.
This would be beyond a "Testnet configuration" page, which would likely also mention --root-dir as a way to partition data per-testnet.
In general tho, I think the docs should be geared to mainnet users so we should keep it simpler where we can. We might even mention that explicitly somewhere. "These docs assume mainnet usage. For testnet use, see 'Testnet configuration'" (or something to that effect).


Windows Networking Support

My understanding is that the current implementation works well for standard POSIX systems, OS/X, and the HaLVM via HaNS. However, I believe I remember seeing an email or issue fly past that it didn't work so well on Windows. I also remember, at the time, glancing at Google and finding out that getting the network library to work on Windows was an adventure.
We should see if there's an alternate networking library to use and link against that, or see if there's anything we can do to simplify getting haskell-tor running on Windows.

https://www.haskell.org/hoogle/?hoogle=withSocketsDo


QoS granted on SUBACK is ignored

When a client SUBSCRIBE to a topic, it can specify a QoS, and the gateway replies with a SUBACK specifying the granted QoS. However, the granted QoS for a subscribed topic is ignored by the gateway afterwards. The QoS of the subscribed topic at any given time seems to be determined by the QoS set on the PUBLISH message by e.g. another client.

What's your SUBSCRIBE Qos? and what is PUBLISH Qos? The min Qos should be picked.
MQTT v3.1.1 says

The Server might grant a lower 983 maximum QoS than the subscriber requested. The QoS of Payload Messages sent in response to a 984 Subscription MUST be the minimum of the QoS of the originally published message and the maximum 985 QoS granted by the Server.

Ok, got it. It works as you describe it. The subscribing client was using QoS=2 by default.


Wrong preferred_color_scheme color scheme

After 664af3446ce13740130c9d62ab97fdccf5584a16, the preferred_color_scheme looks incorrect:

It seems that this is caused by https://github.com/giscus/giscus/blob/664af3446ce13740130c9d62ab97fdccf5584a16/client.ts#L118
My browser version is Mozilla Firefox 106.0.5.

I think this should be set to color-scheme: normal

Can't replicate this problem on Chromium 107

Also not replicate on Firefox 102.4 ESR. Looks like this is not a bug but simply reveals a regression in Firefox

Also not replicable on Firefox 102.4 ESR. Looks like this is not a bug but simply reveals a regression in Firefox

Also not replicable on Firefox 102.4 ESR. Looks like this is not a bug but simply reveals a regression in Firefox

Looks like a feature of Firefox 105: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/105#css

Also not replicable on Firefox 102.4 ESR. Looks like this is not a bug but simply reveals a regression in Firefox

Looks like a feature of Firefox 105: https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/105#css

Interesting, recent W3C standard makes embedded content inherit color-scheme from the embedding element, rather than resetting to browser defaults.
So its actually not a bug in Firefox but in other browsers, and in this issue, which set color-scheme: light for whatever reason? Because according to recent W3C standard, the html window in  should inherit color-scheme from the  element

Did #785 fix it for you?

MDN - color-scheme
normal: Use browser default. Not necesarily related to @media (prefers-color-scheme) ! This is idential to light in most browser settings
light: Use light theme
dark: Use dark theme
light dark: Use either of light or dark theme. The correct way to transparently pick up @media (prefers-color-scheme)
only light, only dark: Disable "Auto Dark Theme" (Currently introduced in Chrome on Android, see Chrome Developers)


kubectl_build: build docker images directly in the k8s cluster

@victorwuky please review :)

ooh very excited for this! will try to review later this afternoon


Fix flaky tests

Right now, the tests pass locally, but not in CI.
However, even local tests can stop working if I hit the rate limit due to not using a GH token in the api request.

Looks like the test is failing because the PR numbers aren't getting resolved correctly:
https://github.com/purescript-contrib/purescript-up-changelog/runs/5224282336?check_suite_focus=true#step:9:27

I got the output to finally print in CI. https://github.com/purescript-contrib/purescript-up-changelog/runs/5237256639?check_suite_focus=true#step:9:72
Looks like the test is failing because the checkout action merges the PR into the base branch as a merge commit and fetches only that single commit. Thus, commits for the changelog entry files don't have any PR numbers included in them.
Per their docs, we can change it to fetch everything.


Load JVM Cleanup and Fix for 0.4

A fix for the macros @windows and @osx that fail on 0.4.x preventing use by Julia 0.4.
Removed deprecations
Removed spurious load error messages.

thanks!


Events not firing for local API

I have only looked at this API for a couple of hours but can't seem to get events to work for Console, I can pause, mute, skip, etc but cannot get events. Is this project up to date or am I doing something stupidly wrong in the example below?
class Program
{
    static void Main(string[] args) => new Program().Start(args);

    public SpotifyLocalAPI spotify;
    public Track currentTrack;

    public void Start(string[] args)
    {
        spotify = new SpotifyLocalAPI();

        spotify.OnTrackChange += (s, e) =>
        {
            Console.WriteLine("Track Change");
            currentTrack = e.NewTrack;
        };

        Console.ReadKey(true);
    }
}

Did you have a look at the example?

Thank you so much, overlooked spotify.Connect(); and spotify.ListenForEvents = true;

No problem, I will try to add the local api to the docs within this week

That would be awesome.

Just added the local API docs @koya-io .
It's not very detailed but should give you some overview how things work.


bin -> lib

I'm not sure if I need to use this crate but you've specified bin on this line:
https://github.com/WilliamVenner/prefer-dynamic/blob/master/build.rs#L37
but the libstd is under the lib directory (in Linux NixOS), not bin. For example:
/home/rbozan/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/libstd-977ece543e1e3d2f.so

That's an interesting difference between Windows and Linux. Windows also has a very complex lib/ structure.

Another difference is that on this line:
https://github.com/WilliamVenner/prefer-dynamic/blob/master/build.rs#L66

it states it starts with "std-" but in Linux it's libstd (as you can see in the full path above)

Thank you! Fixed.


Dead link in 'Creating your own custom alerts'

In 'Creating your own custom alerts' (https://user-guide.cloud-platform.service.justice.gov.uk/documentation/monitoring-an-app/how-to-create-alarms.html), and specifically in the PrometheusRule examples section, there is a link out to https://github.com/ministryofjustice/cloud-platform-infrastructure/blob/main/terraform/aws-accounts/cloud-platform-aws/vpc/kops/components/resources/prometheusrule-examples/application-alerts.yaml which 404s.

Link has now been replaced. Thank you for your feedback.


1st6thbooklistnavbar

Description
Adjusted Navigation Bar on website to have adjusted booklists
What
Pages for Junior Cycle and Senior Cycle booklists have been separated into different classes.
Why
Parents can access booklists easier
Verification Steps
http-server, navigate to Booklists and make sure all content is added to all booklists.
Checklist:

[x] This pr is linked to the related trello card
[x] My code follows the style guidelines of this project
[x] I have requested a code review from (at least) two team members
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] This change has been verified by another team member

@denispop9 please review when you have the time.

@denispop9 thanks. merging now.


How would I create a selector for more than 5 components?

Hi, I'm trying to find out whether to use elm-ecs for an application we are developing at work. It's not so much a game as it is an application with highly composable types, so a kind of ecs seems to make sense. It could very well be though, that we will have more than 5 of these different components, so it would be nice to know whether it's theoretically possible (and ergonomic) to select more than 5 components in a selector.
Of course I'm hoping we won't need selectors that big, but you never know^^
Side note: I'll totally be fine with an answer of "no, you shouldn't be doing that" ^^

Definitely possible. 5 was just an initial guess on how many would be needed, and it saved me some timing from adding more 😁. However I think in general it's better to keep the number of components in a selector low. For instance by merging two components together of by splitting up system logic. But if you really need more than 5 they could be added. Also if there would be selector composion (#2) you'd have some more options 😄


AntennaPod: Unable to connect to Gpodder.net (code 404: Not Found)

Describe the bug
Version: v4.0.1-16173cd2 (HEAD)
Whenever I try to connect AntennaPod to my PodFetch instance, it keeps getting an error:
Unable to connect to Gpodder.net (code 404: Not Found)
Here is my docker compose:
version: '3'
services:
  podfetch:
    image: samuel19982/podfetch:postgres
    user: ${UID:-1000}:${GID:-1000}
    ports:
      - "80:8000"
    volumes:
      - /var/lib/docker/volumes/podfetch/podcasts:/app/podcasts
    environment:
      - POLLING_INTERVAL=300
      - SERVER_URL=https://podcasts.mydomain.com # Adjust to your server url
      - DATABASE_URL=postgresql://postgres:<redacted>@postgres/podfetch
      - DB_CONNECTIONS=10 # optional
      - GPODDER_INTEGRATION_ENABLED:true
      - BASIC_AUTH=true
      - USERNAME=<redacted>
      - PASSWORD=<redacted>
  postgres:
    image: postgres
    environment:
      POSTGRES_USER: ${POSTGRES_USER:-postgres}
      POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:-<redacted>}
      PGDATA: /data/postgres
      POSTGRES_DB: ${POSTGRES_DB:-podfetch}
    volumes:
      - postgres:/data/postgres
    restart: unless-stopped

volumes:
  postgres:

In Nginx Proxy Manager, I do have Websockets Support turned on. I also went into the CLI and tried creating another user and another admin to see if it worked, but same issue. I can access the website in a web browser at https://podcasts.mydomain.com without any issues.
In AntennaPod, I've tried setting the server address to these, but none of them worked:

https://podcasts.mydomain.com
http://podcasts.mydomain.com
podcasts.mydomain.com

I have also tried the docker compose version with image: samuel19982/podfetch:latest to see if the Sqlite version would work, but had the same issue.
Maybe I am missing something?
Reproduction
Create container via docker compose w/ postgres example with gpodder and basic auth on true
System Info
Proxmox -> LXC Container (Debian Template) -> Docker version 24.0.5, build ced0996
Proxmox -> VM (Debian) -> Docker version 24.0.5, build ced0996

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I had the same problem. See here: https://github.com/SamTV12345/PodFetch/issues/219

Disregard... I somehow managed to type GPODDER_INTEGRATION_ENABLED:true instead of GPODDER_INTEGRATION_ENABLED=true each time...


Align exercise with the english version

[ ] check preview

Job d2l-zh/PR-1036/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-1036/

Job d2l-zh/PR-1036/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-1036/


remove fixed compute capabilities list

Removed fixed compute_capabilities = {80, 86, 89, 90} from setup.py when installing from source code.
It has been got by torch.cuda.get_device_capability() at runtime.

When I installed it from source code on my single A100 GPUs with cuda 11.7, it only support compute_capabilities = 80, 86,
the 89 & 90 is not supported.
The issue is caused by a lower version of CUDA ?

Unsupported gpu architecture

This error is most likely related to your installation because NVCC does not support compute capability 89 and 90 before CUDA 11.8


Type '"multi"' is not assignable to type 'SortType'.

I'm submitting a ... (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
When building app for production, it fails with the following message:
error TS2322: Type '"multi"' is not assignable to type 'SortType'.
The error happens too if using [sortType]="'single'".
Expected behavior
No build errors related to ngx-datatable.
Reproduction of the problem
Add sortType attribute to your ngx-datatable component.

What is the motivation / use case for changing the behavior?
N/A

Please tell us about your environment:

Table version: 20.0.0

Angular version: 12.1.1

Browser: All

Language: All

i have this error also.... any solution for this?

I have a solution, you have to change the html to:
[sortType]="SortType.multi"
And you have to declare a variabele in the typescript file (above the constructor) like ( and dont forget to import SelectionType):
SortType = SortType;
Import: import { ColumnMode, SortType } from 'projects/swimlane/ngx-datatable/src/public-api';
I found this answer in the official docs:
https://github.com/swimlane/ngx-datatable/blob/master/src/app/selection/selection-single.component.ts
http://swimlane.github.io/ngx-datatable/#single-selection

I was able to workaround the issue by setting this in my html:
[sortType]="sort"
And this in my component:
sort: any = 'multi';


"https:/" to "https://"

Description

when I use guzzle, I write url "http:/xxx" by mistake, and magically everything works well. I tried http:/some-website on postman, and it failed. Then I guess guzzle automatically correct the url to "http://". But it seems guzzle can not deal with "https:/".
Example

here is my code:
$req = new Request(
$method,
$url,
["Authorization" => $authType . ' ' . $authToken, "Content-Type" => "application/json;charset=utf-8"],
$body
);
if $url = "http:/some-website", everything is ok.
but if $url = "https:/some-website", error.
Additional context

of course, this is my mistake of typing incorrect url. however I think if guzzle accepts "http:/" why not "https:/"?
Or all my guess is wrong?

of course, this is my mistake of typing incorrect url. however I think if guzzle accepts "http:/" why not "https:/"?

I just tried http:/example.com and https:/example.com and both worked.

>>> GrahamCampbell\GuzzleFactory\GuzzleFactory::make()->get('https:/example.com')->getStatusCode();
=> 200
>>> GrahamCampbell\GuzzleFactory\GuzzleFactory::make()->get('http:/example.com')->getStatusCode();
=> 200

Closing due inactivity.


lodash missing _.merge after loading optimizely

This is an issue for me with the client sdk but the problem seems to stem from here.
Not sure quite what is going on, but after loading the client sdk, lodash is missing the merge function which is required in our app.
I have setup a basic enough test where I:

load the exact same version of lodash this project does into a page
try use merge and it works fine.
load the optimizely client sdk
_.merge is not a function

index.js
<html>
    <head>
        <script src="/node_modules/lodash/lodash.js"></script>
        <script>
            // works as expected
            _.merge(1, 2);
        </script>
        <script src="/node_modules/optimizely-client-sdk/dist/optimizely.min.js"></script>
        <script>
            // TypeError: _.merge is not a function
            _.merge(1, 2);
        </script>
    </head>
</html>

package.json
{
  "name": "lodash-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "lodash": "^4.13.1",
    "optimizely-client-sdk": "^1.3.4"
  }
}

@dpix thanks for raising this issue. Will take a look get back to you when we get the chance!

Hi @dpix -- I looked into it and I can't seem to reproduce your error using that code. We'll be bumping the version to 1.4.1 very soon, try it out with that version and see if the error persists!

Thanks @caitlinrubin-optimizely, looks like it's been fixed up in the last release 👍


[Snyk] Upgrade @navikt/ds-icons from 0.8.17 to 0.8.18

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @navikt/ds-icons from 0.8.17 to 0.8.18.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2022-06-30.

Release notes

Package name: @navikt/ds-icons

0.8.18 - 2022-06-30<EMAIL_ADDRESS>

0.8.17 - 2022-06-10

    from @navikt/ds-icons GitHub release notes

Commit messages

Package name: @navikt/ds-icons

c1baf84 Lerna versionbump [ci skip]
031a8dc [FIX] Ikonsynk Figma (#1588)
31a37c5 Lerna versionbump [ci skip]
a353175 [FIX] Linkpanel spacing justering (#1587)
1800ef9 Storybook html-plugin
1419955 yarn lock
ffdb9e1 flex: 'start' and 'end' have mixed support (#1585)
0826fcf Lerna versionbump [ci skip]
b53acac [FIX] Stepper postcss-warning hotfix (#1583)
c47048c Lerna versionbump [ci skip]
1fb51c1 [FIX] Search-komponent figma-synk (#1582)
7596f3f Lerna versionbump [ci skip]
21179b0 [FIX] Tokens sync Figma (#1581)
4f437d4 Lerna versionbump [ci skip]
4aa0312 [FIX] Stepper horizontal (#1507)
1563689 Lerna versionbump [ci skip]
431864c [FIX] Search darkmode-token hotfix (#1579)
31098d6 [FIX] Select style hotfix (#1580)
8a15698 Lerna versionbump [ci skip]
df704ee [FIX] justify-self stretch for confirmation-panel (#1578)
17f4c35 Lerna versionbump [ci skip]
3a112a3 [FIX] Oppdatert loader styling basert på figma (#1576)
3d12dba Update README.md
7a75d45 Tabs codemod (#1575)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

oppdatert i siste release


BuildConfig is undefined

I have done everything but I am getting BuildConfig is undefined.

Hi @itsmasud, maybe you have proguard enabled? Check issue #1 for more details.


Remove current item?

This is not an issue!
Question: How do i remove the current item (song) that is about to play?
I'm just curious.

Hey! Hope I understand you correctly here.
Calling stop() on the player will set the current item to nil and stop the player.
If you are using a QueuedAudioPlayer and want to remove all upcoming items you can call removeUpcomingItems().
I see now that there is no convenient way to remove the next item in the queue, so will look into that 👍

Made a PR that makes this more convenient: #47


Add support for .diff and .patch

Add the ability to get text-diff and format-patch by adding .diff or
.patch in the end of a commit url. Issue #2641
With the current behavior with the wildcard in the route, this is the only solution i see. There are probably more elegant ways to solve the problem, but that will require some rethinking. What is your opinion @Unknwon?
And "maxlines", that should not have any impact on this..?

@odinuge
I've added a route for you: https://github.com/gogits/gogs/commit/60ae8ac3d2995d46156ead8ad93004801daad4ce#diff-da541ef36e39344f2e12bddd91d363e2R526

Hi!
I did think about that, but that breaks the ability to use refs instead of sha1sum. Here is an example. Using regex instead of the wildcard is a good idea! I'll keep working on it, but I don't have more time today or tomorrow.

That use case was never part of the design.. maybe somehow supported, but not intended to.

I've tested this after rebasing it to the latest develop branch and it works very nicely. Thanks!

PR has conflict...

I've rebased the patch to the latest develop tree. You should be able to merge the PR without conflict.

If you think it is ready, i can rebase it to the latest develop.

There are some changes on https://github.com/gogits/gogs/pull/3300 as well, you would have to update those at the same time.

Rebased the PR.
New PR, see #3300..


a question on the Kind field

i see you added kind: "Team" and more to the type with partial Db entity. but the actual entity has no kind when save. when do you inject that field ? i assume after retrieve from db but could not see that line

It is just used for authorisation, so e.g. you'd do something like
const ability = await defineAbilityFor({ userId: user.id, teamId: user.activeTeamId });
const canUpdateTeam = ability.can("update", { kind: "Team" })

and it will return true or false depending on if the user is allowed to

i read this like so:  update if the condition (user has kind in the object is true). the missing part for is there is no kind in the schema which i get it as it bloats the db... and when extracted from the db, i could no see where you enrich the user/team data with an extra field of kind: user/team so the ability can read...

It isn't needed to have a kind field in the database. I'd recommend reading the documentation I linked to above, as it should make it more clear. This code
function defineMemberRules({ can }: AbilityBuilder<AppAbility>, user: NonNullable<Member>) {
  // Team
  can(["create"], "Team");

  // User
  can("update", "User", { id: { $eq: user.id } });
}

defines what a normal member can do, for example.
Since there is no can(["update"], "Team"); canUpdateTeam in my comment above will return false (for more information see authorisation.ts) if the user for the given userId has the role "member"

i've read the docs. i understood since we're not using classes we need a way to mark an object as "team" or "user"
we have 2 options:

push a kind field to the object we want to test ability
use subject helper

my thinking: once you select a user from db - you enrich it with a casl kind field or use subject. i only saw the kind  in the type but not on the user object. so i thought "what am i missing here"


Some things do not work when rhel8cis_crypto_policy is FUTURE

https://github.com/ansible-lockdown/RHEL8-CIS/blob/49117ef490f6d6894804407a5b10ed43f8a68a50/defaults/main.yml#L425
Lot's of repositories and mirrors are signed by Let's Encrypt or GlobalSign, which use 2048 bits keys.
FIPS is a bit more secure than DEFAULT.

Hi @bbaassssiiee
This is very true but fips is not without its own challenges.
Like all benchmarks it does require the user to understand what impact each control has on their system and the way that it works for them.
While DEFAULT is not ideal it is still more restrictive than LEGACY. Ideally though it should be greater as you have pointed out.
Thanks as always
uk-bolly

There's a workaround for that (that's been sitting in my mailbox for over a year now, as I haven't needed it yet) that might be a good fit for in this role somewhere:
echo 'key_exchange = ECDHE RSA DHE DHE-RSA PSK DHE-PSK ECDHE-PSK ECDHE-GSS DHE-GSS' > /etc/crypto-policies/policies/modules/RSA-2048.pmod
echo 'min_rsa_size = 2048' >> /etc/crypto-policies/policies/modules/RSA-2048.pmod

update-crypto-policies --set FUTURE:RSA-2048

This will allow RSA keys with a length of 2048 bits, but still keep all the other policies in place.
However, I have no clue how useful that is (I don't know the details of the crypto policies, just that setting FUTURE seems to break a lot of things :-) )

Nice, I used a custom policy:
# https://access.redhat.com/articles/3642912

mac = AEAD HMAC-SHA2-384

group = X25519 X448 SECP256R1 SECP384R1 SECP521R1 \
    FFDHE-2048 FFDHE-3072 FFDHE-4096 FFDHE-6144 FFDHE-8192

hash = SHA2-256 SHA2-384 SHA2-512 SHA3-256 SHA3-384 SHA3-512 SHA2-224

# SHA1 signatures needed for DNSSec
sign = EDDSA-ED25519 \
       RSA-PSS-SHA2-256 RSA-PSS-SHA2-384 RSA-PSS-SHA2-512 \
       RSA-SHA3-256 RSA-SHA2-256 \
       RSA-SHA3-384 RSA-SHA2-384 \
       RSA-SHA3-512 RSA-SHA2-512 \
       RSA-PSS-SHA2-224 RSA-SHA2-224 \
       ECDSA-SHA1 RSA-PSS-SHA1 RSA-SHA1

# tls_cipher is deprecated
cipher@TLS = AES-256-GCM AES-256-CCM \
    AES-128-GCM AES-128-CCM

cipher = AES-256-GCM AES-256-CCM CHACHA20-POLY1305 \
    CAMELLIA-256-GCM AES-256-CTR AES-128-GCM AES-128-CCM AES-128-CTR

# Yum mirrors need ECDHE (using weak elliptic curves like ecdh-sha2-nistp521)
key_exchange = ECDHE DHE-PSK ECDHE-PSK DHE DHE-RSA PSK DHE-PSK

# protocol is deprecated
protocol@TLS = TLS1.3 TLS1.2 DTLS1.2
# ike_protocol is deprecated
protocol@IKE = IKEv2

min_tls_version = TLS1.2
min_dtls_version = DTLS1.2

# Parameter sizes not ideal, but needed for some repositories
min_dh_size = 2048
min_dsa_size = 2048
min_rsa_size = 2048

# GnuTLS only for now
sha1_in_certs = 0

arbitrary_dh_groups = 1
ssh_certs = 1
ssh_etm = 1

There's a workaround for that (that's been sitting in my mailbox for over a year now, as I haven't needed it yet) that might be a good fit for in this role somewhere:
echo 'key_exchange = ECDHE RSA DHE DHE-RSA PSK DHE-PSK ECDHE-PSK ECDHE-GSS DHE-GSS' > /etc/crypto-policies/policies/modules/RSA-2048.pmod
echo 'min_rsa_size = 2048' >> /etc/crypto-policies/policies/modules/RSA-2048.pmod

update-crypto-policies --set FUTURE:RSA-2048

This will allow RSA keys with a length of 2048 bits, but still keep all the other policies in place.
However, I have no clue how useful that is (I don't know the details of the crypto policies, just that setting FUTURE seems to break a lot of things :-) )

This can be implemented since https://github.com/ansible-lockdown/RHEL8-CIS/pull/297 was merged.

The 3.0.0 version of the benchmark states:
Over-riding or opting out of the system-wide crypto policy could allow for
 the use of less secure Ciphers, MACs, KexAlgorithms and GSSAPIKexAlgorithm

Note: If changes to the system-wide crypto policy are required to meet local
site policy for the openSSH server, these changes should be done with a sub-policy
assigned to the system-wide crypto policy. For additional information see the
CRYPTO-POLICIES(7) man page

hi @bbaassssiiee
Im hoping that this has now been addressed?
Please feel free to close if this is the case.
Many thanks as always
uk-bolly

hi @bbaassssiiee
Im hoping that this has now been addressed? Please feel free to close if this is the case.

I think we can close this if you move rhel8cis_allowed_crypto_policies_modules from vars to defaults, then sites can define their local policy modifications.

A PMOD for better SSH security
# message authentication code algorithms
mac@SSH = -*-64* -UMAC-128 -HMAC-SHA1
# Encrypt then MAC. Prioritize the etm over the non-etm macs
ssh_etm = 1

# key exchange algorithms
# disables weak DH groups and NIST curves
group@SSH = +X25519 +X448 -FFDHE-2048 -SECP256R1 -SECP384R1 -SECP521R1
# Disable diffie-hellman-group-exchange-sha256
arbitrary_dh_groups = 0

hash@SSH = -SHA1
sign@SSH = -*-SHA1 +EDDSA-ED25519
min_rsa_size = 2048
min_dh_size = {{ min_dh_size }}

# encryption algorithms (ciphers)
cipher@SSH = -*-CBC -CHACHA20-POLY1305 -AES-128-*
ssh_certs = 1


appveyor failing

https://github.com/conan-io/conan/issues/5780

https://ci.appveyor.com/project/Dekken/maiken#L436


auto load subtitle, subtitle language same with Netflix profile language

Feature request
I'm submitting a proposal to ...

[Y ] new feature
[ ] change an existing feature

Describe your request

the subtitles for each program need select seperately.
Describe the use case

is't possible to auto load subtitle and subtitle language same with Netflix profile languge. reduce remote buttons touch for subtitle language select.
Describe an alternative

Development tips

Additional details or screenshots (if appropriate)
Specific details

Screenshots

the management and loading of subtitles is done by Kodi core not the add-on
before you have to ask to Kodi devs (for external files)
you means external file?

Netflix original subtitles auto match.

the management and loading of subtitles is done by Kodi core not the add-on
before you have to ask to Kodi devs

@CastagnaIT i mean Netflix original subtitle, auto match profile language.

In Kodi (not in Netflix Addon) go to: Settings - Player - Language - Prefered Subtitle Language

@notoco It's work. Thanks a lot!

In Kodi (not in Netflix Addon) go to: Settings - Player - Language - Prefered Subtitle Language

Not work on Kodi 19 beta 2 anyone?  Netflix Adds-on version 1.12.0.


Update mgmt_ide-setup.asciidoc

Updated IDE setup steps with new workarounds because of problems with latest devonfw version.
@devonfw/cobigen

Spellcheck fails:
 <context> documentation/mgmt_ide-setup.asciidoc
--------------------------------------------------------------------------------
cobigen
eclipsepde
freemarker
longpaths
swtbot
url
--------------------------------------------------------------------------------

!!!Spelling check failed!!!

Please comply to the typography of the words as represented in our dict: https://github.com/devonfw/.github/blob/master/en-devonfw.dict

I think you updated the docs into even more faulty typo. Have a look what I wrote ;)

spellcheck bugged?


Fix bug upload image from clipboard

This fixes 2 issues

The pluginId in server API is editorjs only but in admin it's recognized as react-editorjs so it causes some issues with upload and link preview as 43, 38, 36
The user token could be retrieved from sessionStorage or localStorage (see auth.getToken from @strapi/helper-plugin) (depends on user browser?) that also causes the bug 36

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add University Service and Tests

In this PR, I added an endpoint /api/university/get that can be used to get information
about universities that match a given name.

LGTM


[#5 Software Updates] Add Update Actions

Reviewer: @orlandohohmeier

is this rebased? all the changes are from this PR?

merging after tests pass and the one :abc:


Namespace prefix media on content is not defined.

I have a similar issue (https://github.com/cantino/huginn/issues/960) with iTunes style namespaces (e.g. itunes:summary).
I've tried adding xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd" and created a PR for that (https://github.com/cantino/huginn/pull/1196), but your test routines all failed. Don't really understand what happened, so would you kindly check it out?
I've tried it on my herokuapp and it seems to be working, now I don't get error messages when I try to open the RSS XML output of the data output agent and instead have <rss xmlns:atom="http://www.w3.org/2005/Atom" xmlns:media="http://search.yahoo.com/mrss/" xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd" version="2.0"> displayed as in a proper .
Or should it be a stub declaration like xmlns:itunes="http://search.yahoo.com/mrss/" instead of the real one?

Your PR looks good, I think.  We left a few comments on it.

Merged!


onset and offset？

Why detection means classification? the output is not contain the onset and offset of event...

@hegc The output contains framewise prediction of sound events. So users could design their own algorithm to detect onset and offset.


[Backport] Use daemon thread pool for AsyncHttpClient in emitters

Backport of #5057 to 0.11.0.

@gianm thanks for creating those backports

@leventov no problem; I have a script to do it so it's pretty easy :)

@gianm could you please fix compilation

@leventov should be fixed now

@gianm can you share your script ?

@himanshug it's https://gist.github.com/gianm/68145b09a55e6f6460a11ae9adfc25a4
For some reason the "milestone" and "labels" never get set, I'm not sure what's wrong there. If you ever figure it out let me know how to fix it :)


Add color customizations

Background colors:

Primary: header, subscribe box, footer
Secondary: About box
https://getbootstrap.com/docs/5.1/utilities/background/

--
Buttons:

Primary (default yellow):
Secondary (default black):
https://getbootstrap.com/docs/5.1/components/buttons/

--
Font color:

Subscribe box
About box
https://getbootstrap.com/docs/5.1/utilities/colors/

Coming in 2022.1.7


Bug report: AttnBlock3D reshape disorder

There is a bug when reshaping the spatial-temporal input for computing spatial attention, here
        b, c, t, h, w = q.shape
        q = q.reshape(b * t, c, h * w)

Correct one is:
        b, c, t, h, w = q.shape
        q = q.permute(0, 2, 1, 3, 4)  # (b t c h w)
        q = q.reshape(b * t, c, h * w)

Without transposing T-axis , the values may not be grouped correctly in reshaping.
Another reference is  the MagViT-v2 torch impl.
https://github.com/lucidrains/magvit2-pytorch/blob/main/magvit2_pytorch/magvit2_pytorch.py#L443-L444

Sorry for that. We merge that to fix this bug.

Sorry for that. We merge that to fix this bug.

thanks. btw, since the computation logic is changed, the model may require re-training.


doc: Fix typo in ansible readme content

Typo found in: https://github.com/tungbq/devops-basic/tree/main/topics/ansible#ansible-helloword-

NOTE: This issue is opened for first timer contributor only. Thanks

Hi - this is my first time trying to contribute to an open source project and I was wondering if this issue can still be worked on? Is there anything more I need to know about the project before I get started? Thanks!

Hello @KatieChilds thanks for your interest to this repository contribution!
Yes you can work on this issue.
To get start you could refer to https://github.com/tungbq/devops-basic/blob/main/CONTRIBUTING.md


fix: Non stop pending of run command document

https://github.com/trendmicro/cloudone-community/tree/main/Workload-Security/Deployment/aws-python-agent-deploy-automation-using-lambda
Describe the bug
Run Command document never leaves pending state. Instances are being tagged correctly but no agent is deployed.
No outputs were generated in the session.
To reproduce
Just deploy it and it should replicate itself. Found during WS immersion day.

So I've tested with @fernandostc and it looks like if you're having issues it might be intermittent or even something in the account, could you double-check?

Yeah, it is working again. Odd, I was not the only one having this issue. Maybe it was AWS issue.


Handle root URL's with trailing slash

Fixes an issue where requests to retrieve the metadata, such as localhost:8000/saml/metadata,
would fail with a 404 when URL option had a trailing slash, such as
rootURL, err := url.Parse("http://localhost:8000/")
.
.
.
samlsp.Options{
	URL: *rootUrl,

Rather than textually appending the middlware paths to the given root URL's path, this uses
ResolveReference to reliably create
a URL relative to the root URL given by the user.
Previously, when textually appended, the metadataURL would end up with a double slash, such as
http://localhost:8000//saml/metadata and therefore the path matching in the middleware's
ServeHTTP
would never match since the incoming r.URL.Path would be just /saml/metadata.

IntelliJ created this PR the wrong way.


[bugfix] Fixed performance issue with git prompt

Problem
The git prompt would could bash to use 100% CPU for large amounts of time when dealing with long git status outputs.
Repro:

clone a repo with a large number of files (e.g. the linux kernel)
delete all files in the working tree

cd'ing in the directory should now be painfully slow for users of the git prompt.
The issue is noticeable starting from ~100 modified/deleted files on my machine.
It becomes unmanageable for large working tree changes - after removing 100k+ files from a working tree for instance, bash used 100%CPU on one core for a few hours before I killed it.

Solution
The idea is to use awk to pre-process the output of git status, condensing the needed information into a 4-line summary, rather than processing the raw output in multiple places as was done before.
I timed the proposed fix on my machine - it took ~250ms to process a 65k lines status.

Thanks for this contribution, is a good improvement.
On what platform you tested it?

Tested on Debian testing, x86-64, bash 4.3.42

I have the same environment, @nwinkler can you test if this PR works on Mac?

I just tried this. It doesn't work on the default Bash (v3.2), which does not ship with the readarray command.
-bash: readarray: command not found

More details can be found here: http://stackoverflow.com/q/23842261/1228454 - maybe there's an alternate way of handling the output of the command without using readarray, the SO questions has quite a few answers.

Hi, thank you for the feedback.
I replaced readarray with a combination of awk and read.
I successfully tested the new version on a hand-built Bash 3.1 rc2 on Debian.

Works fine on a stock Mac now! Thanks!


bug(mat.theme-overrides): Overridden CSS properties not displaying correctly in IDE hints

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
This is simply a quality of life improvement request.

I've overridden on-surface and surface in the recommended manner, but the IDE (IntelliJ in this screenshot) won't pick those modifications up.
The rendered page in the browser shows the correct colour, this is strictly an issue within the IDE.
The concern is that deeper into the project, when a developer is using Material CSS variables, the colour hints will not match what they are expecting (and needing) which will cause confusion and potentially result in additional variables being created by the developer that aren't actually needed, just so the IDE hints match their expectations.
Reproduction
Steps to reproduce:

add Material to an Angular project
override a theme variable such as surface
use the matching CSS property in a rule (--mat-sys-on-surface in this example)
observe that the IDE colour hint does not match the overridden value

Expected Behavior
The colour hints in the sidebar should show the overridden colour(s) instead of the defaults.
Actual Behavior
What I see in the sidebar are the defaults for the current theme.
Environment

Angular: 19.0.3
CDK/Material: 19.0.2
Browser(s): N/A
Operating System (e.g. Windows, macOS, Ubuntu): macOS
IDE: IntelliJ IDEA 2024.3 (Ultimate Edition)

Closing since we can't do much about this. You'll have to bring it up with the IDE developers.


Cannot even login since update to 2.0.68

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
'str' object has no attribute 'get'
Traceback (most recent call last):
python3.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 575, in execute
    raise ex
cli/core/commands/__init__.py, ln 633, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 626, in _run_job
    six.reraise(*sys.exc_info())
...
azure/cli/core/_profile.py, ln 946, in adal_token_cache
    return self.load_adal_token_cache()
azure/cli/core/_profile.py, ln 952, in load_adal_token_cache
    self._load_service_principal_creds(all_entries)
azure/cli/core/_profile.py, ln 979, in _load_service_principal_creds
    if c.get(_SERVICE_PRINCIPAL_ID):
AttributeError: 'str' object has no attribute 'get'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --use-device-code

Expected Behavior
Environment Summary
Darwin-18.6.0-x86_64-i386-64bit
Python 3.7.4
Shell: bash

azure-cli 2.0.68

Extensions:
aks-preview 0.4.5
resource-graph 0.1.11

Additional Context

Sorry you're running into trouble, @caoh. Can you try running an az account clear and trying again?

No command works anymore. I was trying to use az aks, but az login stopped working altogether. I've already tried uninstall and reinstalling both Python and azure-cli.

I had to go and delete the ~/.azure/accessTokens.json to get things to work again. Regardless, it isn't a problem that occurs in my opinion.

It seems it was your cache getting invalidated that caused this issue. We're working on re-writing the way our caching mechanism works all together, so I'm going to close this issue out. Sorry you ran into this trouble!


Update solidusio references

Summary
We think that would be better to move this repo under solidusio
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:

[ ] I have written a thorough PR description.
[ ] I have kept my commits small and atomic.
[ ] I have used clear, explanatory commit messages.

The following are not always needed:

📖 I have updated the README to account for my changes.
📑 I have documented new code with YARD.
🛣 I have opened a PR to update the guides.
✅ I have added automated tests to cover my changes.
📸 I have attached screenshots to demo visual changes.

I have fixed the checks

There are some other nebulab references remaining:

https://github.com/solidusio/rubocop-solidus/blob/a0c1f7d9c745cbe888f00f7d8ba474e814ac5bf8/CHANGELOG.md?plain=1#L7
https://github.com/solidusio/rubocop-solidus/blob/a0c1f7d9c745cbe888f00f7d8ba474e814ac5bf8/relnotes/v0.1.2.md?plain=1#L3
https://github.com/solidusio/rubocop-solidus/blob/a0c1f7d9c745cbe888f00f7d8ba474e814ac5bf8/changelog/fix_typos_in_readme_and_add_instructions.md?plain=1#L1

Additionally, checks are not passing, and I don't know why 🤷🏼

@MassimilianoLattanzio these are references on changelog or release notes, I think we should keep them for historical data.
About the checks I have fixed them, FYI the problem was related to the fact that I have committed the *.gem files

@MassimilianoLattanzio these are references on changelog or release notes, I think we should keep them for historical data.

Since the old link will be redirected to the new one, and the new link contains the same thing, why not move these links too? Anyway, it's not crucial, so you can skip them.


fix: change ovbserver container

Currently observer is listening for #appContainer https://github.com/usoonees/logseq-plugin-link-unlink/blob/master/public/index.ts#L45 and it is too wide for plugin tasks.
Suggesting to listen for unlinked page references block only - .page>div:nth-last-child(1) .references to improve performance ⚡

PS: hope soon this selectror can be replaced to better one -  .page-unlinked (https://github.com/logseq/logseq/pull/7066)

PS: hope soon this selectror can be replaced to better one - .page-unlinked (logseq/logseq#7066)

latest Logseq 0.8.10 has my PR, so you can update )


[release-1.11] fix issue that is Helm chart creates invalid manifest when given boolean or numeric value for environment variables

Manual cherrypick required.
#36946 failed to apply on top of branch "release-1.11":
Applying: add quote to env's value
Using index info to reconstruct a base tree...
A	manifests/charts/gateway/templates/deployment.yaml
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): manifests/charts/gateway/templates/deployment.yaml deleted in HEAD and modified in add quote to env's value. Version add quote to env's value of manifests/charts/gateway/templates/deployment.yaml left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 add quote to env's value
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Not applicable.


Downgrade web3

Rollback to avoid https://github.com/ethereum/web3.js/issues/1916

I did the update on web3 to beta-36 because of the following error WebSocket connection to 'wss://rinkeby.infura.io/ws' failed: Error during WebSocket handshake: Sent non-empty 'Sec-WebSocket-Protocol' header but no response was received

I did the update on web3 to beta-36 because of the following error WebSocket connection to 'wss://rinkeby.infura.io/ws' failed: Error during WebSocket handshake: Sent non-empty 'Sec-WebSocket-Protocol' header but no response was received

Yeah my bad forgot about that. I opened another PR which rolls it back to 33, and should have neither of these bugs, please take a look.


feat: support releases with semantic versioning

Fix: https://github.com/casdoor/casdoor-helm/issues/9

[x] Admin Installs the app from the GitHub Marketplace

@sp71 plz fix: https://github.com/casdoor/casdoor-helm/actions/runs/7515929174/job/20460434532


Just checking if this API works with a Site configured with AWS Asset Sync discovery connection

Issues and Contributing

This is just a general inquiry and to check if there is a limitation in the API.

Expected Behavior

Actual Behavior

I have a script to check the scan status but looks like its not returning the status. I tested the script using a AWS connection (Legacy) and this works fine.

Possible Solution

Steps to Reproduce (for bugs)

Python code that reproduces the issue:
scanstatus = api_instance.get_scan(scansessionid)
print(scanstatus)
scanstate = scanstatus.status
print(scanstate)
while scanstate != 'finished':
scanstatus = api_instance.get_scan(scansessionid)
scanstate = scanstatus.status
print("still scanning")
time.sleep(300)
else:
print("done scanning")
print(scanstate)
Your Environment

Library version or repository branch:
Python version: 3.8
Operating System and version: Windows Server 2012
InsightVM/Nexpose product version: NExpose 6.6.8

Other relevant information

Hi @dirtcrosser, I'm not aware of any inherent limitations. I'd recommend raising this as an issue with support so that further investigation can be done: https://www.rapid7.com/for-customers/


Don't open print links in a new window

Target _blank has a vulnerability:
https://mathiasbynens.github.io/rel-noopener/ and
https://github.com/alphagov/whitehall/issues/2942
Links don’t need to be in a new window

So I know some of our print pages do funky stuff on a new page.
For example:

Visit https://www.gov.uk/become-lorry-bus-driver
Scroll to the bottom
Click 'Print entire guide'
New page opens, with print dialogue.

Would this break that functionality?
It's also worth noting the print links should not go to an external domain.

@nickcolley Nothing breaks. The link would act like a normal in-page link. The auto-print uses a window.onload function and is unaffected.

Visit https://www.gov.uk/become-lorry-bus-driver
Scroll to the bottom
Click 'Print entire guide'
New page opens, with print dialogue.

I don't feel so comfortable with this change because I don't have lots of devices to test this against, printing behaves very different depending on the operating system so I worry this would introduce an odd behaviour.
That being said, I've not seen an evidence of that testing to make this change in the first place.
Will let someone else also have a look. 👍

My strength of opinion on this is low, 3/10 for context.

Is it still a security issue if the opened link is controlled by us? Or is this about consistently avoiding target="_blank"?

Is there anyone with context of the original change? Or anyone who's observed user research around print since then?

My reckon would be that it opens in a new page to reduce the chance of users getting stuck on the print page - as we know some users have difficulty navigating via the back button.
I'm not sure  the reasons given for the change are convincing enough to make it.

Target _blank has a vulnerability: https://mathiasbynens.github.io/rel-noopener/ and alphagov/whitehall#2942
Links don’t need to be in a new window

There might be other benefits/context i'm not aware of though.

Links don’t need to be in a new window

I get the whole 'give users control over whether things open in a new window or not', but when it comes to print links specifically I think there's a pretty common pattern for doing so. I'm not convinced that we should remove it without further research.

Main focus of this PR should have been:

Print links should use target attribute not JS if they want to open in a new window

Closing this until we have PRs to add attributes where needed.


ServerCertificateValidationCallback being hijacked in web application

Hello,
We have a web api application that connects to internal services using a customized ServerCertificateValidationCallback in which we validate our own certificates.
We have started to implement some Skype for Business APIs, but the ServerCertificateValidationCallback appears to be getting hijacked when we instantiate Microsoft.SfB.PlatformService.SDK.ClientModel.ApplicationEndpoint.
This is the new ServerCertificateValidationCallback target after doing so:
{Microsoft.SfB.PlatformService.SDK.ClientModel.OauthEvoRestfulClient.<>c}
Can this be avoided in some way?
Thank you

And a follow up here, if I'm reading the code correctly, this new callback returns true for a certificate name mismatch.  Since the callback only returns the result of the last delegate, this would cause a security issue in the rest of the application would it not?
https://stackoverflow.com/questions/7559283/how-are-multiple-servicepointmanager-servercertificatevalidationcallback-methods


Next version

Hi,
When do you guys planning (approximately of course) to release next version?
Very good job btw.!

As I'm too busy with my own business, I cannot give you any ETA. Sorry. But any help on new version is welcome. Like auto posting of position over HTTP...

@mauron85  i implemented a custom async task from handleLocation which works pretty well for my requirement. and i somehow managed to make it work in background without notification in the drawer. did nothing rocket science but i just stopped it from starting in foreground. lol. which is ok for my requirement.
any drawback of stopping it to start in foreground ?( except the callback not reaching cordova plugin)

@uzumakinaruto123 it would be great if you can send PR here. You'll be recognised as contributor with all the fame and glory :-). The code doesn't have to be perfect (I'm not fascist) We can discuss what are the drawbacks or ask users of this plugin. Actually there is some discussion regarding this in #20.

@mauron85  i never did a pull request. till now i have been making changes in my app so it includes all raw coding(noob :p) . i mean all properties like url,post data,server api key are not coming from javascript. i simply hard coded those.
my changes are not worth a PR .   i will try adding new plugin object properties if i could so that i can make a PR.
I followed you on twitter few days back to discuss and show my code. but i couldn't message since you dont follow me. :p
anyways thanks for your work :) appreciate.

@uzumakinaruto123 @mauron85 Wow! You guys are doing great job. Thanks a lot. Really appreciate.

@uzumakinaruto123 can you please post your code here, may be someone can improvise on it if needed. I for one would like to utilize the raw code itself.

@uzumakinaruto123 even if not as a PR, but please share your raw code. it will be a lot of help to the community.

I've just pushed update to next version branch. It has very basic http posting of location, currently for Android only. There are two option attributes currently (url and httpHeaders). Read docs of next branch, specially configure and HTTP location posting.
Install via:
cordova plugin add cordova-plugin-mauron85-background-geolocation@next

And tell me what you think/need.

@shyamal890  @btdevn  Ok here its but it might not be perfect. and understand  i have nothing to do with javascript i only configure and toggle background service from javascript.
That means no kind of callback would return to your javascript file. it has to be like this because callbacks in javascript are of no use when app is terminated.
In your added android platform , you can catch all location updates here .
platforms/android/src/com/tenforwardconsulting/cordova/bgloc/AbstractLocationService.java

> i assume we all use some editor and not android studio so its difficult to know the packages to import. these are the packages needed. I am just posting all i used you can figure which one you need.
import java.util.Random;
import org.json.JSONException;
import org.json.JSONArray;
import org.json.JSONObject;
import java.net.HttpURLConnection;
import java.net.MalformedURLException;
import java.net.URL;
import java.net.*;
import java.net.URLConnection;
import java.io.*;
import android.os.AsyncTask;
import java.util.*;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import com.your.package.NotificationHelper; // <-- Custom sql database helper i needed for the app
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase.CursorFactory;
import android.database.sqlite.SQLiteDatabase;

find   protected void handleLocation (Location location) { }

 protected void handleLocation (Location location) {
      final LocationProxy bgLocation = LocationProxy.fromAndroidLocation(location);
      bgLocation.setServiceProvider(config.getServiceProvider());

       if (config.isDebugging()) {
            bgLocation.setDebug(true);
            persistLocation(bgLocation);
        }

       //here we have location in bgLocation. we need to convert it to string to pass it to async task.
       // we want to make http calls in background mode so we need do it in async task.

        try {
            String locStr = bgLocation.toJSONObject().toString();
            Intent intent = new Intent(Constant.ACTION_FILTER);

             CustomfunctionTask task = new AbstractLocationService.CustomfunctionTask();
              task.execute(locStr);  // THIS INITIATES OUR ASYNC

           //REST OF try catch block
          }

        //REST OF THE handle location() CODE BELOW

}

Create this Async task in he same file

private class CustomfunctionTask extends AsyncTask<String, Void, String> {

       @Override
        protected String doInBackground(String... locStr) {

                 try {
                          Log.i(TAG,"Entered Background Posting"+locStr[0]);
                          JSONObject mylocation = new JSONObject(locStr[0]);
                          Log.i(TAG,"Custom FunctionLatiitude:"+mylocation.getString("latitude"));

                    //here i fetch some data from my custom database. like user preference for radius etc
                   // create a string object to post it to the URL. please ignore the additional properties from below object.

                   String testdata =  "{\"radius\":"+userradii+",\"usercity\":\""+cityman+"\",\"latitude\": "+mylocation.getString("latitude")+",\"longitude\":"+mylocation.getString("longitude")+"}";

                     // before posting you can add your logic here whether to post data or not.

                      // example i connect  to two urls in this async. once to perform some user checks on the server.
                      // based on that response i decide whether to post notification or not.

                       HttpURLConnection conn=null;
              try{
                URL url = new URL("http://www.xyz.in/your/check.php");
                conn = (HttpURLConnection) url.openConnection();
                conn.setRequestMethod("POST");
                conn.setDoInput(true);
                conn.setDoOutput(true);
                conn.setRequestProperty("Content-Type", "application/json");
                conn.setRequestProperty("Accept", "application/json");
                conn.setUseCaches(false);

                DataOutputStream writer = new DataOutputStream(conn.getOutputStream());
               writer.writeBytes(testdata);
               writer.flush();
               writer.close();

               //Get Response

                InputStream is = conn.getInputStream();
                BufferedReader rd = new BufferedReader(new InputStreamReader(is));
                String line="";
                StringBuffer response = new StringBuffer();
                while((line = rd.readLine()) != null) {
                  response.append(line);
                  response.append('\r');
                }
                rd.close();

                JSONArray temp1 = new JSONArray(response.toString());

                //here temp1 array has response from the server. similarly i post notification here using calling rest api endpoint of my push service.

                  if(send){
                               http connection to push service rest api. same way we did for the first link above
                   }

                         }catch (MalformedURLException e) {
                                 Log.d(TAG,Log.getStackTraceString(e));
                         }

                  } catch (Throwable e) {
                           Log.e(TAG,"Exception posting data: " + e);
                   }

         }

         @Override
        protected void onPostExecute(String result) {

        }

}

the code might be confusing but i tried my best.
@mauron85 has updated the plugin with next branch if you simply want to post data.  :)

@uzumakinaruto123  your implementation is very close to mine in next branch. 👍  The next branch is missing some sync mechanism, when there is no network and iOS part.

yes :) Thanks for the  update. am planning on updating to the next branch. only the thing is i need to connect to two links.where second is dependent on first ones response. but i guess i can handle that once i update.
just to confirm ANDROID_FUSED_LOCATION is renamed to ANDROID_ACTIVITY_PROVIDER right?
working is same ? Thanks.

@uzumakinaruto123 yes it's renamed to express more that it uses activities. Also Intent communication bgservice -> app main activity was replaced with IPC messenger pattern, but it should behave same.
Also I've start implementing other methods for iOS (showLocationSettings, ....) so maybe checkout from git instead of npm.

is opening settings in ios possible ? i found no plugin for that. even cordova diagnostic doesn't have that option.
The only issue now that bothers me is restart of the service. even when notification is in the panel the service is taking a lot of time to restart.  I have been trying to resolve this.
if user kills the app, service tries to restart again . If by any chance user locks the device before its restarted then it would take forever to restart again. It wont even try to restart in locked state.
After reading some stackoverflow posts what i found was START_STICKY resumes immediately. i am trying to find that post again. But since we have intents to restart with, app crashes if we use START_STICKY.  I have seen some apps which behave like our service they try to restart. but they resume immediately.
Thanks ! :)

is opening settings in ios possible ?

Yes, it is. Code is already in git.

START_STICKY resumes immediately

I've experimented a lot with those values. Currently START_REDELIVER_INTENT is used, so

if this service's process is killed while it is started (after returning from onStartCommand(Intent, int, int)), then it will be scheduled for a restart and the last delivered Intent re-delivered to it again via onStartCommand(Intent, int, int)

which assures, it will also restart service always with same config options. You can replace it with START_STICKY and reload config from db (config is also persisted). But it doubt it would help. Anyway, let know your results.
The main problem with service crash (and slow restart) is Android bug as I wrote in issue #9.

ya ! i saw the next branch i think you have pushed one of the workarounds  from that android issue tracker.  DummyActivity ones. So  i think before trying anything weird i should try next branch as it is :p
Thanks for the great work. appreciate.

@mauron85  I updated the plugin ,it works good. service resumes quickly. that was great Thanks ! appreciate your work. I need to check on other android versions and other RAM eating ROM's :D


Re-architecture of the Godot Android plugin. [3.2]

This PR revisits the Godot Android plugins' architecture.
This is done by leveraging the Android library file format. A Godot Android plugin is now defined as follow:

A Godot Android plugin is a regular Android library packaged as an aar archive file with the following caveats:

The library must have a dependency on the Godot Android library (godot-lib.aar). A stable version is made available for each release.

The library must include a <meta-data> tag in its manifest file setup as follow:
<meta-data android:name="org.godotengine.plugin.v1.[PluginName]" android:value="[plugin.init.ClassFullName]" />
Where:

PluginName is the name of the plugin.

plugin.init.ClassFullName is the full name (package + class name) of the plugin class extending GodotPlugin.

At runtime, Godot (via GodotPluginRegistry) will parse the Android manifest, and retrieve all components with the specified <meta-data> tag. It'll then use reflection to instantiate and load the plugin class defined by plugin.init.ClassFullName. From there, the plugins follow the same lifecycle as with the previous architecture.
A nice side-effect of this new architecture is that we can remove all the gradle build and android manifest custom update logic, since we can just rely on the Android library build and manifest merge mechanism.
Documentation update: https://github.com/godotengine/godot-docs/pull/2979
Note:

This is a breaking / non-backward compatible change with the previous (v0) Godot Android plugin architecture. For backward compatibility in the 3.2.x releases, the new system will work alongside the previous one. The previous one will be fully removed in Godot 4.0.
Just like the previous architecture, this requires the use of custom builds.
This new architecture may be helpful for PR #26221, as the ARCore build dependencies no longer need to be included in Godot core. This would however limit ARCore to custom builds only.
The updated architecture simplifies the process of bundling Android gdnative libraries with Android java libraries. See the updated documentation for details.

Example implementation:

Hello World reference plugin: https://github.com/m4gr3d/Godot-Android-Plugins/tree/master/hello_world
Custom branch of the Godot Oculus mobile plugin: https://github.com/m4gr3d/godot_oculus_mobile/tree/2.0/plugin

I gave the PR a try rebased on the current 3.2 branch.
The non-custom build workflow still works as usual, so all good on this front.
When trying a custom build with no plugin and default settings, the build fails:

I don't see new entries in the preset that I might have omitted to fill, so I assume it's an issue with this PR? I haven't tried a custom build in a while without this PR though, so it might be something that affects 3.2-stable custom builds too.

I gave the PR a try rebased on the current 3.2 branch.
The non-custom build workflow still works as usual, so all good on this front.
When trying a custom build with no plugin and default settings, the build fails:

I don't see new entries in the preset that I might have omitted to fill, so I assume it's an issue with this PR? I haven't tried a custom build in a while without this PR though, so it might be something that affects 3.2-stable custom builds too.

Looks like only the release portion of the build failed. Can you validate that the android_source.zip contains both debug and release artifacts?
It may be that generation of the release artifacts was omitted.

Thanks, I had omitted to build the release template as usually I do my tests with custom_template/debug only when using the non-custom build workflow, which works fine (unless I explicitly export a release build).
After building both debug and release templates and reinstalling the source archive, it works fine.
I'll now try the Hello World plugin to validate the workflow.

Thanks a lot!
Once the 3.2 docs are updated and maybe one or two plugins ported, it might be worth writing a blog post about the changes, WDYT?

Thanks a lot!
Once the 3.2 docs are updated and maybe one or two plugins ported, it might be worth writing a blog post about the changes, WDYT?

Yep, that sounds good!

Hello @m4gr3d
I'm porting my plugins to use the v1 system, I was able to make my share plugin to work as it has no additional dependencies, but I'm having some trouble with AdMob and I think you can help me.

Where can I declare the dependencies? I've tried to add:

[dependencies]
remote=["com.google.android.gms:play-services-ads:19.1.0"]

on the .gdap file, but it's not working.

Where can I create the plugin AndroidManifest? The final user needs to add their APP ID using metadata on the manifest file.

Thank you very much for your help.

EDIT 3: Well, after having a look at the exporter code, I've created an AndroidManifest.conf file inside the res://android/plugins directory and worked, is that the correct way to do?

@Shin-NiL The AndroidManifest.conf file is a remnant of the old system. While it'll work for 3.2.2+ (we're maintaining backward-compatibility), it's not recommended to mix the two, since it'll be removed in 4.0.
So I can better understand your issue, prior to the new system, how did you instruct your users to add their APP ID?

@m4gr3d For Godot 3.2.1 the user must edit the AndroidManifest.conf distributed with the plugin. Before that I was using a very old SDK where the APP ID was not required.

@Shin-NiL For this scenario, it's probably simpler to ask the user to directly edit the res://android/build/AndroidManifest.xml file and add their app id as directed in the setup instructions.
You can provide a link to these setup instructions in your own plugin guide.

@m4gr3d It's an option, the only drawback is that this file is overridden when installing a new Android build template from editor.

@Shin-NiL yes, although a typical user would rarely have the need to install new Android build templates often unlike us developers, so I see that as less of an issue.
Let's try that approach as a starter and we can iterate on the process based on user feedback.

@m4gr3d I got, let's try it, thank you very much.

yes, although a typical user would rarely have the need to install new Android build templates often unlike us developers, so I see that as less of an issue.

In the future we do need to figure out a nice system for users to be able to upgrade their build templates versions without losing their customization. Upgrading from 3.2.2 to 3.2.3, etc. will always be a pain if you have any custom changes to the build folder.

Godot 3.2.2 RC1:
Is it on purpose that after installing the Android build template (Menu Project > Install Android Build Template) the created android/ directory is "visible" by Godot? Because for my plugin's res/ directory Godot creates .import files for the icons (.png) and gradle complains: "Resource and asset merger: The file name must end with .xml or .png". That's because of the "nature" of res/drawable/: they should contain only .xml and .png files.
If I remember correctly in previous versions the android/ directory was hidden (correctly).

@yalcin-ata yes it's on purpose as the res://android/plugins directory can be used to install the new Android v1 plugins and thus need to be accessible by users.
I took a look a some of your instructions, and yes these practices are now disrecommended and highlight why the previous system was an issue.
I'd strongly recommend to move over to the new plugin architecture, but you can always throw a .gdignore file in your repo (since you're asking the user to clone the whole thing) as a workaround.
Bear in mind that the old architecture is fully removed for Godot 4.0, and so your plugin as is will no longer work in that version.

I am porting the current plugin (old version) to the new v1 (not public yet), so far it is working fine, but there are still some issues with the build process.
Ideally I would compile the .aar file and provide it as is. As the plugin is for Firebase, to compile the .aar file it needs a project specific google-services.json from the Firebase console.
I will dig in deeper, thank you for your answer.

@yalcin-ata can you point to the firebase instructions about the json console file. It's possible you may just need to instruct the user to add that file in their res://android/build directory.

Thanks again for your help. I solved my problem.
It was a "special" case in my case because of the google-services the plugin depends on. The key was to remove the google-services.json from the library module (.aar), and also the lines regarding "...google-services..." from the build.gradle, and later add the missing parts to [GODOT-PROJECT]/android/build.
Adding all the library module's dependencies from build.gradle to the module's .gdap was also mandatory.
Hopefully very soon I can publish the Godot Firebase module v1 to the public!
Thank you!

@m4gr3d Is there a possibility - like in the previous version via gradle.conf and AndroidManifest.conf - to add some chunks to build.gradle and AndroidManifest.xml? When I add some stuff to those files manually the build system removes them again.
This is i.e. need for AdMob <meta-data .../> for custom AdMob App Id, or for "apply plugin: 'com...google-services'


Webhooks besides discord

Describe the feature
So  basically i would like to receive webhooks that are not automagically converted to Discord. Just a straight JSON webhook with the data for when those particular objects are created.
Additional Context
No response

Where and how would this be used @dexslab?

So like a ticket is made for a person speeding. The data from the ticket would be sent to the webhook in json. Then I could take the fine data and create an esx billing request for the fine amount so the user can then pay it after it's paid I can post back to the CAD to say it was properly paid

So an option for owner* to provide a Webhook URL for each webhook type, this would simply send the respective data, right?
Would this send the raw data, or somewhat already formatted data such as units, etc?

It would basically send the raw data as json like hitting the API request. Then owner would be able to parse and use what info needed. Like fine amount and character being fined.


[Feature fix] File notifications & Logging

Cherrypicked the relevant commits d5151de...64c67c1 onto feature/aggregate-emails


Make NavBar responsive for Mobile View

Description
Please include a summary of the change and which issue is fixed.
Fixes #487
List any dependencies that are required for this change
Type of change

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

How Has This Been Tested?
UI /UX changes
Attach gif or screenshot for changes.

Before :

- After :

Checklist:

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

Finally its resolved. You can merge the PR now. Thanks


FakeHttpMessageHandler ctor - Wiki update or do I miss something?

I'm using the last recent version 5.0 and tried to follow the Wiki, but it seems the FakeHttpMessageHandler ctor has changed and the wiki is not up to date.
From the wiki I should do something like this:
        var messageHandler = new FakeHttpMessageHandler(requestUrl, messageResponse);

But there is only one parameter:

I did this in my code and it seems to work:
        var messageResponse = FakeHttpMessageHandler.GetStringHttpResponseMessage(responseData, HttpStatusCode.BadRequest);

        var fakeMessageHandler = new FakeHttpMessageHandler(new HttpMessageOptions() { HttpResponseMessage = messageResponse });

Is this the right approach?

Hi @robertmuehsig
yep - that is correct. I bit the bullet and did some breaking changes from 4->5. I was hoping the semver bump would be a hint to people that there's been 1+ breaking change applied.

I more or less nuked the Factory pattern because I felt it was too hard to easy figure out what is going on, if a new person had a look at some code and saw 'magic' happening. I also felt like it was a bit like a service locator pattern. So I decided that DI/IoC was the safest and most maintainable way forward.
I also replaced all the parameters for DI/IoC with a new HttpMessageOptions class. This makes it easier (IMO) to set up your various permutations for endpoint calls.

Secondly, I thought I updated all the wiki items :/ Looks like I didn't at all!
/me goes and updates it.
Ok - all updated.

To answer your question directly:
YES.
that's the correct way to do it now, if you don't care what URL is passed in.
If you want to make sure that the fake response is returned ONLY if you're calling a specific URL, then change your code to:
var messageResponse = FakeHttpMessageHandler.GetStringHttpResponseMessage(responseData, HttpStatusCode.BadRequest);

var options = new HttpMessageOptions
{
    RequestUri = "http://www.my.website.com/some/endpoint",
    HttpResponseMessage = messageResponse
};
var fakeMessageHandler = new FakeHttpMessageHandler(options);

I "expected" a breaking change because of the major version bump, but I wasn't sure if my way is the correct one - so it was more or less a wiki related issue.  Thank you for your work!

Pleasure @robertmuehsig 😄 I hope you (and others) find this helpful.
It also highlights that I really need to start having release notes for my open source projects 😊


cmap to GNNExplainer.visualize_subgraph appears to fail

🐛 Bug
Passing cmap as a kwargs to GNNExplainer.visualize_subgraph leads to a
TypeError: draw_networkx_labels() got an unexpected keyword argument 'cmap'

To Reproduce
Steps to reproduce the behavior:
Add an explicit cmap to line 46 of the gnn_explainer.py example:
ax, G = explainer.visualize_subgraph(node_idx, edge_index, edge_mask, y=data.y, cmap="cool")

Additional context
I have already found a fix and will submit a PR shortly.

Actually, this may be a wider spread issue with the kwargs based implementation for GNNExplainer.visualize_subgraph located at
https://github.com/rusty1s/pytorch_geometric/blob/master/torch_geometric/nn/models/gnn_explainer.py#L324-L329.

Thanks for letting me know. This is now fixed in master, see here.


Allow developer to pick which output datasets should be archived by default

Currently, we default all output datasets to be archived by default for rule submission. We should let App developer pick which ones should be archived by default to indicate which output are more/less important.

This has been implemented.


We can use port 80 - Need to update the document https://docs.microsoft.com/en-us/azure/app-service/containers/configure-custom-container#configure-port-number

[Enter feedback here]
The document here says we can use any other port than 80. Actually we can use port 80 as well now.
https://docs.microsoft.com/en-us/azure/app-service/containers/configure-custom-container#configure-port-number
Document needs an update
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6489b871-9cf8-7c99-9834-076ec646d0e6
Version Independent ID: fa5d2995-5833-c04b-2530-c100bcd1e76b
Content: Configure a custom Linux container - Azure App Service
Content Source: articles/app-service/containers/configure-custom-container.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@abhilashmk - Thanks for brining this to our attention. We will investigate it further and update you shortly.

@abhilashmk, Apologies for the delay!
In the beginning of the same document that you’re referring to, yes it does already mention the point ‘the web server in your custom image may use a port other than 80’. The sample referenced (on GitHub page) in the document, uses Port 8080 and hence this document provides this as an example : set WEBSITES_PORT to 8000.
We strive to make our Azure docs more discoverable and the docs aims at demonstrating key concepts with step-step instructions of a particular process /deployment and emphasis clearly with a ‘Note’ for any difference/distinguishes during the course. For example, here you learn how to ‘Use persistent shared storage’, ‘Enable SSH’, ‘Access diagnostic logs’ , ‘Configure multi-container apps’, ‘Configure environment variables’ and access them. The document just uses Alpine Linux distribution to demonstrate the concepts, but similar steps are good for other flavours of Linux as well and the beginning of the doc points to the tutorial first and calls out the prerequisites.
You may checkout this document for more details Azure App Service on Linux FAQ.
Hope this explanation helps! We will close this out, but if you feel you need more information please just let us know.
Thanks again for taking time to share your valuable feedback.

@AjayKumar-MSFT  There is no update in the documentation. We take time out to provide feedback, do respond and pass it on to the concerned team please. Again to be specific  'The documentation says -- We cannot use port 80'.. Actually we can and that was the feedback. To update the doc.

Hi @abhilashmk, we appreciate you taking the time to share your feedback with us.  We reopened the issue since the documentation was not updated as you said. Can you please share with us your environment setup and the steps you took so we can repro your scenario on our end before update the doc? Thanks

@abhilashmk, We value your time and feedback. Sorry for any confusion, the documentation does mention that Port 80 is supported "The web server in your custom image may use a port other than 80." However, under the section 'Docker Compose options' > under sub-section "Unsupported options" Docker Compose configuration options, where it mentions - "ports other than 80 and 8080 (ignored)" - 'ports' is supported docker Compose configuration options.
Kindly let us know the exact section in the documentation and 'lines' you are referring to and share more details about your scenario.

@AjayKumar-MSFT I am not sure why you mentioned " Sorry for any confusion, the documentation does mention that Port 80 is supported ".... The sentence here means  "The web server in your custom image may use a port other than 80." we cannot use port 80 for single container ( which is wrong ). We can, all the image samples provided by Azure as samples use port 80 in custom images -- ex:
https://github.com/Azure/app-service-quickstart-docker-images/blob/master/wordpress-alpine-php/0.8/Dockerfile

@Grace-MacJones-MSFT here is your repro -- Its Azure Sample -- uses port 80
https://github.com/Azure/app-service-quickstart-docker-images/blob/master/wordpress-alpine-php/0.8/Dockerfile

@abhilashmk Thank you for taking the time and raising up this issue. We feel your issue would be best addressed by the doc author. As a result, we have assigned the issue to the doc author for them to review their doc further.
To help expedite the process, we have also emailed the doc author internally to raise awareness.
We thank you for your patience on this matter.

@abhilashmk we appreciate your patience on this issue. I have submitted a PR to update the doc with a clarifying sentence about port 80, the change is merged internally and will go-live in the public docs shortly. Thanks again for your feedback!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[add]action.group.dispatcher

Codecov Report

Merging #5 into master will increase coverage by 0.59%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage      57%   57.59%   +0.59%
==========================================
  Files          30       32       +2
  Lines         521      533      +12
  Branches       37       36       -1
==========================================
+ Hits          297      307      +10
- Misses        222      224       +2
  Partials        2        2

Impacted Files
Coverage Δ

rplugin/python3/curstr/action/group/file.py
52.63% <ø> (+2.63%)
:arrow_up:

rplugin/python3/curstr/action/group/base.py
75% <ø> (+2.27%)
:arrow_up:

rplugin/python3/curstr/exception.py
66.66% <ø> (-3.93%)
:arrow_down:

...lugin/python3/curstr/action/group/file_position.py
44% <ø> (+3.25%)
:arrow_up:

rplugin/python3/curstr/action/group/directory.py
71.42% <ø> (+1.42%)
:arrow_up:

rplugin/python3/curstr/importer.py
30.43% <0%> (-0.91%)
:arrow_down:

rplugin/python3/curstr/action/group/__init__.py
100% <100%> (ø)
:arrow_up:

...gin/python3/curstr/action/group/dispatcher/base.py
33.33% <33.33%> (ø)

rplugin/python3/curstr/action/source/base.py
70.58% <75%> (+25.42%)
:arrow_up:

...gin/python3/curstr/action/group/dispatcher/file.py
76.92% <76.92%> (ø)

... and 2 more

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 c8f0f26...3e1539f. Read the comment docs.


Added metrics for prometheus/client

I have added support for sending metric using the prometheus/client gem. More or less a copy of the statsd support.

Can you add a test for it? I'm afraid I'll break it when adding new metrics, for instance.

Ok, I write some unit test and make a new commit next week.
Regards
Peter
-------- Originalmeddelande --------
Från: Daniel Schierbeck<EMAIL_ADDRESS>Datum: 2019-05-31 19:57 (GMT+01:00)
Till: zendesk/ruby-kafka<EMAIL_ADDRESS>Kopia: Peter Mustel<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Rubrik: Re: [zendesk/ruby-kafka] Added metrics for prometheus/client (#739)
Can you add a test for it? I'm afraid I'll break it when adding new metrics, for instance.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/zendesk/ruby-kafka/pull/739?email_source=notifications&email_token=AMGJEMU26O2RYX4DR4WVXPDPYFRI7A5CNFSM4HQK7Q42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWV6HVI#issuecomment-497804245, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMGJEMVK2JG5NCX2TVC5EIDPYFRI7ANCNFSM4HQK7Q4Q.

Thanks!