Splits apart blockchain integration tests

I will hold off on merging this until the conflicting PR is merged and then sort out the diff.

Resolves #44


fix some of the documentation errors

@Martin-P tnx for your comments. I 'm working in an other branch to clean this up. There are several methods in the zf-code component that where not covered by tests and methods like these tostring that are useless. I didn't want to mix those changes, so this one only contains docblock changes.

The to string methods are possibly added for the same reason why this was done in the FunctionReflection class There are no tests for this method and I could reproduce it by adding a test?

Required due to bug in php

@ralphschindler You wrote this comment. Can you elaborate on this and provide a link to the bug you were referring to?


Hi, how to add custom AuthClient icon in standart component yii2-authclient?

Hello, please tell me how to add custom AuthClient icon in standart component yii2-authclient?
After the last update Yii2 was gone one client of a regional network of customers icon, how to add another icon to my custom client?

@denoll, please use Official Yii 2 forum to ask a questions - GitHub is a bug tracking system.


Issues with Passing Custom Credentials to sql_table Function

Source name
30 + SQL databases
Slack thread:  here.
Description:
I am encountering a problem when trying to pass custom credentials to the sql_table function. By default, it uses the credentials stored in my secrets.toml file, even when I attempt to pass a different set of credentials directly to the function. Additionally, when I tried passing an Engine object directly to the function, I received the following error:
dlt.common.configuration.exceptions.InvalidNativeValue: ConnectionStringCredentials cannot parse the configuration value provided. The value is of type Engine and comes from the ('credentials',) section(s). The expected representation for ConnectionStringCredentials is a standard database connection string with the format: driver://username:password@host:port/database.

Describe the bug
This error suggests that the sql_table function might not currently support passing custom credentials directly.
To Reproduce

Attempt to pass custom credentials to the sql_table function.
Alternatively, try passing an Engine object directly to the sql_table function.

Temporary Solution:
I was able to bypass this issue by changing the sql_table decorator from @dlt.common.configuration.with_config(sections=("sources", "sql_database"), spec=SqlTableResourceConfiguration) to @dlt.source. Additionally, I included spec=SqlTableResourceConfiguration in the arguments of the returned dlt.resource, modifying sql_table to resemble the structure of sql_database.
Expected behavior
The sql_table function should allow users to pass custom credentials directly without defaulting to the credentials stored in the secrets.toml file. Furthermore, it should be possible to pass an Engine object without encountering configuration parsing errors.
Actual Behavior:
The function defaults to the credentials in secrets.toml and throws a configuration parsing error when an Engine object is passed.
Suggested Fix:
There might be a need to review and potentially update the implementation of credential handling in the sql_table function to ensure it supports custom credentials and the direct passing of Engine objects.

+1 here, Just faced the same issue.
In fact when using mssql source, it is required to set up connectionString as
connectionString = f'DRIVER={{ODBC Driver 18 for SQL Server}};SERVER={SERVER};DATABASE={DATABASE};UID={USERNAME};PWD={PASSWORD}'

If using only secrets.toml, it is not parsing correctly.
So I have to pass custom credentials to the sql_table function. Tried also the Engine object directly, there is the same error message as mentioned by @dat-a-man

+1, faced the same issue and spent way too long debugging this

Possibly related slack thread.
Even if you pass custom credentials inside sql_table, dlt still expects to find the section [sources.sql_database.credentials] inside secrets.toml. Otherwise it throws the error:

<class 'dlt.common.configuration.exceptions.ConfigFieldMissingException'>
Following fields are missing: ['drivername', 'database', 'username'] in configuration with spec ConnectionStringCredentials
        for field "drivername" config providers and keys were tried in following order:
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__CREDENTIALS__DRIVERNAME was not found.
                In config.toml key credentials_test.sources.sql_database.sample_analytics_data1.credentials.drivername was not found.
                In config.toml key credentials_test.sources.sql_database.credentials.drivername was not found.
                In config.toml key credentials_test.sources.credentials.drivername was not found.
                In config.toml key credentials_test.credentials.drivername was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key SOURCES__CREDENTIALS__DRIVERNAME was not found.
                In Environment Variables key CREDENTIALS__DRIVERNAME was not found.
                In config.toml key sources.sql_database.sample_analytics_data1.credentials.drivername was not found.
                In config.toml key sources.sql_database.credentials.drivername was not found.
                In config.toml key sources.credentials.drivername was not found.
                In config.toml key credentials.drivername was not found.
        for field "database" config providers and keys were tried in following order:
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__DATABASE was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__CREDENTIALS__DATABASE was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__CREDENTIALS__DATABASE was not found.
                In Environment Variables key CREDENTIALS_TEST__CREDENTIALS__DATABASE was not found.
                In config.toml key credentials_test.sources.sql_database.sample_analytics_data1.credentials.database was not found.
                In config.toml key credentials_test.sources.sql_database.credentials.database was not found.
                In config.toml key credentials_test.sources.credentials.database was not found.
                In config.toml key credentials_test.credentials.database was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__DATABASE was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__CREDENTIALS__DATABASE was not found.
                In Environment Variables key SOURCES__CREDENTIALS__DATABASE was not found.
                In Environment Variables key CREDENTIALS__DATABASE was not found.
                In config.toml key sources.sql_database.sample_analytics_data1.credentials.database was not found.
                In config.toml key sources.sql_database.credentials.database was not found.
                In config.toml key sources.credentials.database was not found.
                In config.toml key credentials.database was not found.
        for field "username" config providers and keys were tried in following order:
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__USERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__SQL_DATABASE__CREDENTIALS__USERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__SOURCES__CREDENTIALS__USERNAME was not found.
                In Environment Variables key CREDENTIALS_TEST__CREDENTIALS__USERNAME was not found.
                In config.toml key credentials_test.sources.sql_database.sample_analytics_data1.credentials.username was not found.
                In config.toml key credentials_test.sources.sql_database.credentials.username was not found.
                In config.toml key credentials_test.sources.credentials.username was not found.
                In config.toml key credentials_test.credentials.username was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__SAMPLE_ANALYTICS_DATA1__CREDENTIALS__USERNAME was not found.
                In Environment Variables key SOURCES__SQL_DATABASE__CREDENTIALS__USERNAME was not found.
                In Environment Variables key SOURCES__CREDENTIALS__USERNAME was not found.
                In Environment Variables key CREDENTIALS__USERNAME was not found.
                In config.toml key sources.sql_database.sample_analytics_data1.credentials.username was not found.
                In config.toml key sources.sql_database.credentials.username was not found.
                In config.toml key sources.credentials.username was not found.
                In config.toml key credentials.username was not found.
Please refer to https://dlthub.com/docs/general-usage/credentials for more information```

+1 happens during an onboarding


[validation-gen] Fix IsValidIP usage

What happened?
The IsValidIP interface has changed after https://github.com/jpbetz/kubernetes/commit/98940366ddba1c90f38d2811bbb3d49f04f29db9
Need to update the call site accordingly e..g
https://github.com/jpbetz/kubernetes/blob/e0a142dc5cca935c85f66068125aabf09bd7e2d1/pkg/apis/core/validation/validation.go#L4042
What did you expect to happen?
Code is able to compile.
How can we reproduce it (as minimally and precisely as possible)?
N/A
Anything else we need to know?
No response
Kubernetes version

$ kubectl version
# paste output here

Cloud provider

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

Fixed by https://github.com/jpbetz/kubernetes/pull/60


[WIP] Fix validation in dialog model to only happen if visible and not read…

Content validations should only happen if field is visible and not read only.

@miq-bot add_label wip
@miq-bot add_label refactoring
@miq-bot assign @gmcculloug

@miq-bot remore_label wip

👍  Tested this is working.  Ran into some other issues with hiding fields through automate while testing, but they are unrelated to this PR change.


Can we use more than one lines?

I want to show more than one lines in the graph. Is it possible to show add two or more lines in the same graph? Please provide a solution.

"Please provide the solution ASAP." seems quite rude.

@gkaimakas does please means "Rude"? Removed it.

Is it possible to add more than one line at all, with this current version?

This was added in version 4. 👍


feat: categorical sheaf condition for presieves of arrows

[x] depends on: #7855

This is looking more and more like the presieve library we already have. I wonder if it is possible to generalize most of the library to a family of arrows and specialize to presieves when necessary?

You mean define something like IsSheafFor a family of arrows, etc?

Maybe. And FamilyOfElements stuff as well. I might be plainly wrong but the API you are adding looks suspiciously like what we already have at first glance.

Let's start by fixing the merge conflict :)

That should be easy, I'll do it now

Can you please add a docstring explaining the difference with the Presieve version? Thanks!

This is looking more and more like the presieve library we already have. I wonder if it is possible to generalize most of the library to a family of arrows and specialize to presieves when necessary?

I would agree that the definition of the sheaf condition for a presieve should be made a special case of the sheaf condition for a family of arrows. Ultimately, I would think that the API should focus more on families of arrows and the sieves they generate, rather than on presieves which seem to be a mathlib-specific notion, which may be convenient only in the formalization of pretopologies.

I'll try to refactor the parts of the Presieve API that I can, using this, in a separate PR.

Can you please add a docstring explaining the difference with the Presieve version? Thanks!

Done

Let's merge this as it is, we will have time to do a refactor later.
Maybe we can even wait to have the characterization of condensed spaces (that is not that far), to test that the refactor doesn't make things more complicated on a a "concrete" application.
Thanks!
bors merge


Verify/cleanup the File monkey patch in lib/metadata/util/win32/versioninfo.rb

Is it still needed with ruby 2.0 and beyond?
https://github.com/ManageIQ/manageiq/blob/386e43720ac4f156757e2784d4b4b3449dc4efd5/lib/metadata/util/win32/versioninfo.rb#L4

@lfu I think this can be closed, as the lib/metadata stuff was moved over to the smartstate repo. It's their issue now if they want to clean it up.


Should be `h=img.size(-2), w=img.size(-1)'?

https://github.com/ZhugeKongan/torch-template-for-deep-learning/blob/e2a4ab70a00b45709da6ec5f1ae5c089b95b92da/autoaug/cutout.py#L22

Thank you for your reminder, it has been corrected.


'wake-detect-path' error

Just wonder 'wake-detect-path' should be the conv_actions_labels.txt file? It seems causing error below, i suppose 'wake-filter-path' should be the tflite file while 'wake-detect-path' should be the label txt file. Appreciate your advice

After putting the 'wake-detect-path' and 'wake-filter-path' with the same path to the conv_actions_frozen.tflite, it seems to fix the error above, however another error appear as below:

i suppose 'wake-filter-path' should be the tflite file
correct
while 'wake-detect-path' should be the label txt file
wake-detect-path should be the detector tflite file. See https://github.com/pylon/spokestack-android#wakeword-detection for additional documentation on the two model stages of the wakeword detection pipeline


All PRs with dependency updates fail in CI

Title: All PRs with dependency updates fail in CI
Examples:
#19125, #19124, #19122
Logs:
github_release:  GitHubRelease(organization='census-instrumentation', project='opencensus-proto', version='v0.3.0', tagged=True)
Traceback (most recent call last):
  File "/source/tools/dependency/release_dates.py", line 288, in <module>
    verify_and_print_release_dates(
  File "/source/tools/dependency/release_dates.py", line 258, in verify_and_print_release_dates
    release_date = get_tagged_release_date(repo, metadata['version'], github_release)
  File "/source/tools/dependency/release_dates.py", line 215, in get_tagged_release_date
    release = repo.get_release(github_release.version)
  File "/source/build_tools/release_dates/venv/lib/python3.8/site-packages/github/Repository.py", line 3053, in get_release
    headers, data = self._requester.requestJsonAndCheck(
  File "/source/build_tools/release_dates/venv/lib/python3.8/site-packages/github/Requester.py", line 353, in requestJsonAndCheck
    return self.__check(
  File "/source/build_tools/release_dates/venv/lib/python3.8/site-packages/github/Requester.py", line 378, in __check
    raise self.__createException(status, responseHeaders, output)
github.GithubException.UnknownObjectException: 404 {"message": "Not Found", "documentation_url": "https://docs.github.com/rest/reference/repos#get-a-release-by-tag-name"}

cc @phlax @kfaseela

cc @ashankar-fiserv

https://github.com/envoyproxy/envoy/pull/19111 should fix this. Waiting for review after incorporating comments.

i think it may already be fixed, or was in some way transient
neither pre or post submit queues trigger this and running the offending script against current HEAD doesnt fail
i retested one of the above PRs and it passed, ill kick the others now
it would be nice to know what caused this - not sure if the fact that it always fails on the opencensus dep is relevant. It may be just that a release was temporarily deleted, and our error handling is not great.
As the script is already a bit flaky, its hard to tell immediately if the change that i made yesterday has caused it, but mho is that its unrelated
lets keep this open for now - we can close after a day or two if the issue has gone away

another failing PR check:
https://github.com/envoyproxy/envoy/pull/19097/checks?check_run_id=4359014533
https://dev.azure.com/cncf/envoy/_build/results?buildId=95492&view=logs&jobId=1c04237c-5f52-5959-5f83-5522b3556331&j=1c04237c-5f52-5959-5f83-5522b3556331&t=db3f3889-c922-566b-fa2e-4bc4948bc467

It may well be a transient issue. The failing repo is unusual in that its releases page doesn't list the v0.3.0 release. But the tag is there.
I've just re-run #19082 a couple of times with no success.

yep - it doesnt appear to have fixed #19097 either
ill check out one of these branches and try to repro locally...

@phlax could you please check my review mentioned above, I was also hitting this issue intermittently, and hence made sure that get_release call is guarded properly in the PR what I was working on

made sure that get_release call is guarded properly in the PR what I was working on

ha, k, thanks, will do
i have managed to repro locally

cool, thanks @kfaseela it works perfectly


Round the corners of all boxes

In the search results and home pages, the content boxes should have slightly rounded corners as they currently do in production. This should be achieved through a combination of CSS and React properties.
How it current looks:

How it should look:

The difference is subtle but important.

Hi, I'm interested in taking on this issue. Could you please clarify the task for me? I just want to make sure I understand everything correctly: should I only focus on rounding the corners of the content boxes and home pages? Also, could you provide me with the class names and/or file names for the content boxes and home pages that need to be changed? If it's all content boxes, then just the home page names would be enough. Thank you in advance for your help.

Hi @EidanGar, sorry for the mix up, I actually flagged this as a good first issue for a new intern we have starting in the new year. I really appreciate your interest, though!

@nienna73 no problem at all! Thank you for letting me know about the mix up. I understand that the intern will be taking on this issue, and I wish them the best of luck.


Add checks for number of arguments in the transaction

Errors otherwise are unintelligible.
Is there a better way to do this?

I wonder if underlying issue is that we don't have proper errors in Borsh serializer. We know all the field types there, so can verify them generally and report field path with wrong type.

also make sure this doesn't fail in ci env https://travis-ci.com/nearprotocol/nearlib/jobs/268369102

You might be right, the errors in borsh should be improved as well.
It's sad that TS actually doesn't have a way to automatically enforce types when used from JS.
I think you have a better idea how to do this, so I'm going to close this.


Cocoapods integration broken in td-swift-2 branch, Runes.swift not included

I'm integrating Argo in my project Swift 2.0 project via Cocoapods.
The relevant line in my podfile looks like this (to get this https://github.com/thoughtbot/Argo/commit/b2be8edc080d937bbdc6c43b29a9d1e5084c312c commit)
pod 'Argo', :git => 'https://github.com/thoughtbot/Argo', :commit => 'b2be8edc080d937bbdc6c43b29a9d1e5084c312c'
After pod install, the Argo framework will not build in my project because the Runes.swift file is not included. This file is explicitly included in the Argo.podspec, yet it does not get included during install by Cocoapods 0.38.2.
Any ideas?

The same problem (i think) can be seen by running pod spec lintor pod lib lint on the Argo repo.

adding :submodules => true in the podfile resolves this issue. as discussed here #207


More UG improvements

Status
COMPLETED
Description
Added explanation of what Entries are in the Before using.. section.
Added colours to the UI Diagram table as below.
Screenshots

LGTM!


Film VFI node not found within ComfyUI

Film VFI node not found within ComfyUI, im unsure why the installation is failing, cuda supported, problems with cupy, tried install.bat

same issue here


Make performance counters work locally

This might be difficult to achieve or might require significant effort. Currently all performance counters are components that are registered with the symbolic namespace in AGAS. In order to make them available with the local-only runtime we will have to make them non-components and create a new registry for instantiated performance counters. There might be more....

Yeah, this is me optimistically just documenting what could be done. I imagined that it may be possible to wrap a local-only performance counter to turn it into a distributed one (with support e.g. for the locality syntax) but this is without having looked at the implementation at all. This may indeed be difficult.


chore: Replacing KubernetesServicePatch with ops.set_ports

Description
This PR replaces KubernetesServicePatch with ops.set_ports for setting the ports exposed by the workload.
It's worth noting, that with ops.set_ports, we can't remove the default port 65535 created by Juju and also we're not able to set custom name for the port. ops will automatically set the name of the port to juju-{PORT_NUMBER}-tcp
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 made corresponding changes to the documentation
[x] I have added tests that validate the behaviour of the software
[x] I validated that new and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have bumped the version of the library

The placeholder port thing was fixed a couple of days ago so should land in the next release :) https://github.com/juju/juju/pull/16617

The placeholder port thing was fixed a couple of days ago so should land in the next release :) juju/juju#16617

It's been 4 minutes since I created this PR... :O How...? :rofl:

@Gmerold Have we tested this properly? I'm somewhat worried about making such changes right before promoting our SD-Core charms to beta.

@gruyaume Just finished the test and it seems just fine: bundle with updated charm deploys without any issues, simulation with gnbsim  works as expected.


Project manager scan should run in a thread to keep responsive UI

Operating system or device - Godot version:
Windows 10, Godot 2.1rc1
Issue description (what happened, and what was expected):
Using the scan function at the project manager crashes Godot when I click 'open'.
Steps to reproduce:
Open Godot, click Scan, select a large folder, click Open. Shows the Windows Close/Debug error box.

I cannot reproduce this. The project manager stops responding while it is searching for files but if I leave it running it manages to fill the list.
Maybe moving the search to a separate thread might be good so the editor keeps responsive while searching.

Try one of the folders in C:\Users on Windows. Other folders on my Desktop seem to work fine.

For me it just hangs for a long time scanning the folders. On the Downloads folder (which has just a few subfolders) it finishes pretty much instantly.

Any chance this is related to stack overflows?

AFAIK it does not "crash" it just stops responding. If you click on the window then Windows will show the problem and asks if you want to force close it, which can be mistaken by a crash.
I still think it's just a matter of moving the scan to a new thread to show some progress and keep the editor responsive.

I still think it's just a matter of moving the scan to a new thread to show some progress and keep the editor responsive.

Edited issue title accordingly.

Note that https://github.com/godotengine/godot/pull/30735 doesn't implement this.
If we want a simple solution to this, we can dim the window while projects are being scanned (like when quitting the editor or project manager). This will at least provide some kind of visual feedback without complexifying the code too much.

This doesn't need to run in thread actually. We could just utilize _process() method and e.g. scan 20 directories each frame. Easy to do and even allows cancelling, so it's more UX problem.

If this is done in process, then it should be staggered (i.e keep scanning until 10 ms elapsed then continue next frame) because filesystem access can vary wildly across computers, hardware and OSes. On the other hand, with sliiightly more work, using a thread may actually be a more efficient approach, and it can be cancelled even just with a boolean.
I did the staggered approach for Find in Files as well but I'd like to switch to a thread one day for larger projects.

Still valid as of 4.0 for the record. This is only a problem on huge folders, even a somewhat big tree of projects like https://github.com/godotengine/godot-demo-projects is scanned in milliseconds.


Deploy the hash-server under the 3Box domain

@c-castillo what credentials do you need here for the 3box.io domain?

We need access to the domain. Is registered on namecheap? Under the Consensys account?

@michaelsena access to the domain. We need the domain records (basically, credentials to log in to manage the domain )

Ah, I see. I will provide access to credentials offline

New URL is https://beta.3box.io
I'll put this in review. Need for the DNS to propagate and verify on the JS library.

@c-castillo I'm having problems with CORS errors. Getting a 502 on GET. Probably an easy fix when you get back :)

@oed Can you provide detailed info about the error? The CORS checks are in place.

@oed I reviewed one error. It wasn't CORS.
Now it's working properly. Tried with
https://beta.3box.io/hashserver/hash/0x12345
and got the expected response


TS test refactor - issue #1

Updated test match to only allow .spec tests in TS starter and removed the tests directory.
This will help retain consistency between the Node and TS starters.

Thank you @rob212


Ability to use a secret for the password for authentication

Currently, in order to enable authentication on cmak with this chart, you can do the following:
ui:
  # enable basic auth on the UI
  extraArgs:
    - "-DbasicAuthentication.enabled=true"
    - "-DbasicAuthentication.username=<username>"
    - "-DbasicAuthentication.password=<password>"

This results in the password being stored on the manifest within the cluster, which enables anyone with read access to get the manifest AND the password for cmak. It would be ideal if something similar to the below could be allowed such that the chart was extended in capabilities like so:
---
# Values file
authentication:
  enabled: false
  username: ""
  password: ""

I'll open a PR to show this as I really want to avoid exposing a secret via a manifest

Hello
I belive you can achieve your goal with existing chart functionality

create a secret with basic auth credentials.
useextraEnv in values.yaml to expose secret data as environmental variables.
use extraArgs to setup basic auth config using exposed env vars.
Could you confirm this ?

Also, passing secret data via values file is insecure, since the values of installed helm chart can be inspected vi helm get values

Was hoping to be able to create everything needed using the chart rather than having to create a larger chart which creates the secret and has this chart as a dependency (I'm creating the namespace which this belongs in via helm upgrade -i --create-namespace, but I can see why you might not want to include the basic-auth piece if it is already possible.
helm get values reads secrets which means you have more than read permissions on the cluster which is the main thing I am safeguarding against as users might have a reader role but nothing else, so they can see things in the cluster without being able to read secrets (which helm get values relies upon).

Dear @jack1902
I've asked smth different, repeating my question once again

Hello
I belive you can achieve your goal with existing chart functionality

create a secret with basic auth credentials.
useextraEnv in values.yaml to expose secret data as environmental variables.
use extraArgs to setup basic auth config using exposed env vars.

Could you confirm this ?

Dear @eshepelyuk
Yes, that does appear possible, which I alluded to at the end of my first response around why you might not wish to include basic-auth within the chart because CMAK itself has many configuration options and this pins the chart to one such auth mechanism which others might wish to extend upon in the future.
Would it be better suited to potentially extend the secret manifest in the chart so that it would be easier to make use of extra* settings without needing to pre-create a secret or wrap this chart up into a larger chart? I'm merely trying to make this chart more usable without needing to wrap it up into a larger chart or fork it.

Again
I've asked you to confirm this, i.e. go and test and ensure it works.
Can you please do it since you raised this issue ??
If you confirm that it is posaible to achieve in proposed way - we can continue on designing the simplification.

So having tried to use env vars, it appears I would need to set the entrypoint (Dockerfile) or command (k8s-manifest) via the values since the current one is /cmak/bin/cmak, no shell env var substitution takes place meaning the following:
cmak-operator:
  ui:
    # enable basic auth on the UI
    extraArgs:
      - -DbasicAuthentication.enabled=true
      - -DbasicAuthentication.username=admin
      - -DbasicAuthentication.password=${CMAK_PASSWORD}
    extraEnv:
      - name: CMAK_PASSWORD
        valueFrom:
          secretKeyRef:
            name: cmak-password
            key: password

sets the password as ${CMAK_PASSWORD} rather than the value defined within the secret.
Unfortunately, this means that even though extraEnv and extraArgs exist, it appears combining the two currently isn't possible.
I believe this is the Dockerfile used to power the UI container https://github.com/eshepelyuk/cmak-docker/blob/master/cmak/Dockerfile

Hello
But, look https://github.com/yahoo/CMAK/blob/master/conf/application.conf#L68
CMAK by default read basic auth from env vars.
You should just use proper names for that.


compatible with laravel >= 5.7

change helper methods to compatible with laravel >= 5.7
https://laravel.com/docs/5.7/helpers

@ndinhbang Thank you very much for your work!

@franzose will these changes be released in a tagged version anytime soon?


Hide single property values

We want to automatically hide properties of objects that only have a single property inside of them.  We do this making the original property internal and then exposing a wrapper to the inner property.  This allows us to not have to worry about the serialization / deserialization since each class in the hierarchy still exists, but at the same time customers won't have to new an object just to set a single property.
        /// <summary> The AddressSpace that contains an array of IP address ranges that can be used by subnets. </summary>
        internal AddressSpace AddressSpace { get; set; }
        /// <summary> A list of address blocks reserved for this virtual network in CIDR notation. </summary>
        public IList<string> AddressPrefixes
        {
            get
            {
                if (AddressSpace is null)
                    AddressSpace = new AddressSpace();
                return AddressSpace.AddressPrefixes;
            }
        }

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Resources and Sql looks good to me.


FunctionApp storage costing

According to the documentation, Azure Storage is billed out of the scope of the function but is related to the number of execution of it. I would like to know how the storage is requested by the SDK to be able to make an estimation for my project.
I would like to estimate the cost for hundreds of calls, maybe millions, in details.
I do some investigations to know the number of REST call, but it appears that the storage is used even when the function is not called

As we can see on the screenshot, when the FunctionApp is not used, we have some calls to the blob API. It's seems to be at 0 only when the function is stopped
The FunctionApp is on a Consumption Plan and use only HTTP Triggers

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

ID: a540325f-491a-5be3-7ffd-6724139f965d
Version Independent ID: 15d9f76f-8bf0-dab6-19fb-cb6e0e897dfe
Content: Estimating Consumption plan costs in Azure Functions
Content Source: articles/azure-functions/functions-consumption-costs.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@AlexisMtr Thank you for your question! We will review and provide an update as appropriate.

@AlexisMtr Please check Consumption Plan Cost Billing FAQ for details. If you are seeing any discrepancy in the monitoring and documentation, I would request you to create a support request as it would require a deeper analysis.
If you do not have access to a support plan, please reach out AZCommunity[at]microsoft[dot]com with a link to this Doc/Issue as well as your subscription ID and we can help get the support ticket opened for this issue.
Since there is no doc update required at the moment, 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 and document expected structure and logic when lambda is involved

For new comers in CDK willing to test solutions construct, it would be great to have more info and what needs to be setup in addition to the construct instantiation code.
Use Case
When starting to use any construct involving lambda such as https://docs.aws.amazon.com/solutions/latest/constructs/aws-lambda-sqs.html we need, in addition to adding the documented code,

create a lambda folder,
create an index.js file in it,
figure out how to write to a queue (https://docs.aws.amazon.com/sdk-for-javascript/v2/developer-guide/sqs-examples-send-receive-messages.html)
dig into to the construct code to figure out what environment variable will be set for the SQS queue ...

Proposed Solution
Add documentation in addition to code snippet on:

folder structure
Env variables names set for the lambda
Snippet of lambda code

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

We will add a comment explaining the assumption of file structure in the minimal code.
Sample implementations are on our backlog, but not on the roadmap at this point.

Released in 1.94.0


bug: Install cuda again in the image

#13 62.52 Installing collected packages: tokenizers, sentencepiece, msgpack, mpmath, lit, cpm_kernels, cmake, zipp, typing-extensions, sympy, regex, pyyaml, psutil, Pillow, packaging, nvidia-nvtx-cu11, nvidia-nccl-cu11, nvidia-cusparse-cu11, nvidia-curand-cu11, nvidia-cufft-cu11, nvidia-cuda-runtime-cu11, nvidia-cuda-nvrtc-cu11, nvidia-cuda-cupti-cu11, nvidia-cublas-cu11, numpy, networkx, msgspec, mosec, MarkupSafe, h11, fsspec, filelock, falcon, click, uvicorn, nvidia-cusolver-cu11, nvidia-cudnn-cu11, llmspec, jinja2, importlib-metadata, huggingface-hub, transformers, diffusers, triton, torch, accelerate
The image already has the cuda.

I guess it's related to the version of cuda image?

I think it is related to the conda package.
It's in low priority. Please take a look at #62 first. Now we cannot run chatglm model.


(Somewhat OT) Issues with viewing/downloading from YouTube (Other sites work fine)

So, I wrote this script to access, view, and, optionally, d/l vids from YouTube and other sites (i.e., anything site that youtube-dl supports).  It runs under OSX, and is built around the following tools:

youtube-dl
jq
VLC (Mac OSX version)
wget

Basically, I use "youtube-dl -j" to translate the short URL into the long URL, jq to parse the JSON output to get the "long URL", VLC to view it, and finally, wget to d/l it (if I like it).
This all works fine on other sites, but on YouTube itself, as of sometime in the past few months, two problems arise:

VLC always fails to display the vid, with a typically unhelpful error message like "Failed to access: https://..."
wget tries to download it, but fails with a message like:

Resolving r3---sn-hp57knly.googlevideo.com... <IP_ADDRESS>, 2607:f8b0:4008:3::9
Connecting to r3---sn-hp57knly.googlevideo.com|<IP_ADDRESS>|:443... connected.
ERROR: cannot verify r3---sn-hp57knly.googlevideo.com's certificate, issued by /C=US/O=Google Inc/CN=Google Internet Authority G2':   Unable to locally verify the issuer's authority. ERROR: certificate common name *.c.docs.google.com' doesn't match requested host name r3---sn-hp57knly.googlevideo.com'. To connect to r3---sn-hp57knly.googlevideo.com insecurely, use --no-check-certificate'.
Note: If I manually re-try this, supplying "--no-check-certificate", then it does, indeed, work.
So, it boils down to: VLC won't play it, wget will download it, but only with --no-check-certificate.
What can I do?
P.S.  The exact youtube-dl command I am using is:
youtube-dl.py --prefer-insecure --skip-download --write-info-json --write-description -o tmptmp "$url"
which creates a file called tmptmp.info.json, which I then parse with "jq".

This a problem with the certificates (we are getting some similar reports), can those videos be downloaded with youtube-dl itself?
(Note that --prefer-insecure doesn't do anythin for youtube videos, they only use https on their website now)

Yes, direct downloading via youtube-dl works.
So, the real question remaining is: Why does VLC fail?
P.S.  The URL I am using for testing is:
https://www.youtube.com/watch?v=UJkxFhFRFDA

It works with VLC and wget on OSX for me, maybe the SSL support is not properly configured (I don't know how to fix it).

It's probably version-of-VLC related.  VLC has a nasty habit of changing stuff (removing features) from version to version, so I try to avoid "upgrading" unless it is absolutely necessary.
Therefore, I'm running a fairly old version of VLC -> VLC media player 2.0.7 Twoflower (revision 2.0.7-3-g51f7bb4).  When I find something that works, I stick with it.  But I would imagine that version might not support the latest https: foo.

Since youtube-dl works fine, I'm closing the issue.


Added RefreshModulePagePath

This is related to https://github.com/PowerShell/platyPS/issues/387
At the moment -RefreshModulePage creates a TOC but it places it under certain name in certain folder which with a lot of other docs simply vanishes and it's hard to fine. With this addition you can define it like this:
Update-MarkdownHelpModule $PSScriptRoot\..\docs -RefreshModulePage -RefreshModulePagePath "$PSScriptRoot\..\docs\Readme.md"

And your TOC will become Readme.md for your Docs folder. Or your main folder if you wish.

Sorry for the "reverted" but I'm quite new to working with PRs and branches. Usually working alone. Hope that with the revert you have only what is needed in this PR and other PR is having other part.

It seems it fails because of changed params order. Please let me know if I should put it to end of each function or leave as is.

I would gladly do it... however I am not that experienced in git. I did type git rebase and tried to read what it does but not really sure against what I am supposed to rebase it. I'm really sorry. I do hope you can help me out here so I can know more in future.

Hey, no problem! I can help you out with it.
To get you started the process should look like this:

git fetch origin master:master - will update your local master to be up-to-date with the origin master (note that origin may point to your fork, depending on your setup, use git remote -v to check it)
git rebase master from your Fix branch will put your work on top of the updated master
I you likely would need to resolve the conflicts, they are marked with ==== >>>>> <<<<< markers (VS Code provides nice one button-expirience for resolve), once conflicts are resolved, add the files git add file-name.
When all conflicted files added, do git rebase --continue
If at any point during the rebase, you want to start again, run git rebase --abort

When everything is done, check the history with git log and check the final diff with git diff master.
Then push the updated version to your fork:
git push --force-with-lease your-remote-name Fix

Hm, ok I tried to use this GitHub web-based UI to resolve conflicts and it seems to work. :)
I will rebase and squash the PR before merging, should be all good! Thank you for working on it!
If you ever decide to master git, here is my favorite gamified way https://github.com/Gazler/githug :)

I will do so. I need to. So far I've been working 99% alone for all my projects.


Added 3 Components

Draft PR
Added the following components:

Search bar
Progress bar
Generic Card list

The search bar input box can be hidden to allow for an onSubmit function that only gets suggestions, rather than making a query for a specific thing.
The progress bar displays progress and a label for the current progress stage.
The card list at the minute has not been tested with any card component, since I'm unsure what the best way for converting objects (User, Match, etc) to cards would be.

Nice, but uhhhhhhhh why are there 68 commits... 😳

I have rebased your branch on main and got rid of all the commits that weren't yours.
Please check that all your changes exist before pulling

@ros40 is this ready to be merged?

This PR is ready to be merged


Relative wind

Changed hycom_opt and mom_momtum.F to include a parameter "relative_wnd_coef" that multiplies usur and vsur to take into account the relative wind speed to compute the momentum.
Parameter added to the hycom_opt namelist file.

I tested again if I got the same results as before when relative_wnd_coef = 1.0, but I got some differences. I am trying to understand them.

Is it round off or more significant?

Hi Till,
‘Squashing’ when merging seems like a good idea to me: I see there are two options, rebasing the branch before merging or squashing when merging.  I did the second one when merging some changes yesterday, buy I don’t know git well enough to know if one is better than the other.
Annette
Annette Samuelsen
forskningsleder | Research Leader
@.*** | +47 932 38 202
Stiftelsen Nansen Senter for Miljø og Fjernmåling
Nansen Environmental and Remote Sensing Center
Jahnebakken 3 | N-5007 Bergen | Norway
@.*** | nersc.nohttps://nersc.no | +47 55 20 58 00
[cid:logo_de1c6baf-5da6-4f53-b1e5-34827df1144c.jpg]
Konfidensialitet. Informasjonen i denne e-posten er utelukkende ment for bruk av adressaten(e),
og kan inneholde konfidensiell informasjon som ikke kan deles uten avsenders skriftlige samtykke.
Notice of confidentiality. The information contained in this e-mail is intended solely for the use of
the addressee(s) and may contain confidential or legally privileged information.
From: TRasmussen @.>
Date: Monday, 11 September 2023 at 10:41
To: nansencenter/NERSC-HYCOM-CICE @.>
Cc: Annette Samuelsen @.>, Mention @.>
Subject: Re: [nansencenter/NERSC-HYCOM-CICE] Relative wind (PR #190)
@TillRasmussen commented on this pull request.
I think that it is ok now. One thing to consider is that this is a relatively small add on to the code that includes many commits. This is a more general thing but maybe it is more obvious to follow the history in git if some of these were "sqaushed". See e.g. https://www.git-tower.com/learn/git/faq/git-squash. This is a more general thing thus what is your thoughts @lbertinohttps://github.com/lbertino @lilleannettehttps://github.com/lilleannette @Jipingnerschttps://github.com/Jipingnersc (more people could probably be added to the list)
—
Reply to this email directly, view it on GitHubhttps://github.com/nansencenter/NERSC-HYCOM-CICE/pull/190#pullrequestreview-1619443771, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQZHB5KRMOQZUMT27ARSMLXZ3FETANCNFSM6AAAAAAX7IQWNQ.
You are receiving this because you were mentioned.Message ID: @.***>

I am closing the pull request to make a new one clearer with merged commits


Use browser back navigation

Instead of hardcoding back links, rely on window.history.back() to step backwards in browser history (same behavior as using browser back button). This allows scroll to be preserved and fixes some existing flows like:

/research

inline link to FAQ ->

/research/forest-offsets-explainer-faq

hardcoded meta.back link ->

/research/forest-offsets-explainer

nice! this is an even better solution than what we discussed!


How to use a Vimscript function as a maker on any filetype

In my init.vim I have:
function! CapsWarn()
  if (system('xset -q | grep -oE "Caps Lock:   on"') =~ "on")
    echohl WarningMsg | echo "Caps Lock is still on" | echohl None
  endif
endfunction
autocmd InsertLeave * call CapsWarn()

It works great, except when xset is not immediately available, so I'd like to do something like this:
function! CapsWarn()
  if (system('xset -q | grep -oE "Caps Lock:   on"') =~ "on")
    echohl WarningMsg | echo "Caps Lock is still on" | echohl None
  endif
endfunction
let CapsFunc = function("CapsWarn")
let g:neomake_caps_maker = { "exe": "CapsFunc" }
autocmd InsertLeave * Neomake caps

However, that is not displaying my warning. Do you know why? Thanks.

Funny idea - although it sounds like you might want to use jobstart / job_start directly, or there should be some more lightweight interface in Neomake to achieve this.
This works (replace if 1 with your condition again).
function! CapsWarn(...)
  if 1
    echohl WarningMsg | echo "Caps Lock is still on" | echohl None
  endif
  return []
endfunction

let g:neomake_caps_maker = {'get_list_entries': function('CapsWarn')}

augroup my_capslock_warn
  au!
  autocmd InsertLeave * Neomake caps
augroup END

HTH.
Closing this for now.


Empty Taglist doens't return any articles

Hello, i was using the API and i found an possible bug, when i doesn't pass an list of tags sometimes doesn't return any article on the api/articles route.

An possible to this is implement an condition like you already did in the the favoritedFilter function:

With this, i was able to solve the problem.
Empty Tag returning all the articles.

With Tag return just the DACIOK BRABO article.

Good catch @joaooliveiradev!


grpcgateway with restapi function

*Added grpc-gateway
*Added Api function to proto files
*Updated gateway.go file

Closed in favor of #79
New issue opened for GetStorage at #98


Dev/tsweeney/uninfo2

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>- What I did
Updated man page for dockerd to indicate that subuid and subgid files need to be created for --userns-remap and then added a little more verbiage to the errors to make it more user friendly.  This is the docker-1.13.1-rhel variant to go along with the docker-1.13.1 variant at #301
- How I did it
vi is my friend.
- How to verify it

Verify that both /etc/subuid and /etc/subgid are present and empty.

Add --userns-remap=default to OPTIONS in /etc/sysconfig/docker

restart docker service 'systemctl restart docker'
Should see:
dockerd-current[*]: Can't create ID mappings: No subuid ranges found for user "dockremap" in /etc/subuid

enter values into /etc/subuid via 'echo dockremap:808080:1000 >> /etc/subuid'

restart docker service 'systemctl restart docker'
Should see:
dockerd-current[*]: Can't create ID mappings: No subgid ranges found for gid "dockremap" in /etc/subgid

enter values into /etc/subgid via 'echo dockremap:808080:1000 >> /etc/subgid'

restart docker service 'systemctl restart docker'

Should not see an error.
Verify 'man dockerd' has new verbiage as noted below.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1546870
- Description for the changelog
Add better error for subuid/subgid with usernamespace

@rhatdan @edsantiago @cevich 1-13.1-rhel variant.  I'm not planning to submit this to 12.6 or any other Docker variants, please holler if you think I should.

LGTM

LGTM.  No need for fixing this in earlier versions (IIUC) since it's not supported by those kernels anyway.

I did not change the original two files, but added changes to docker login's man page to address https://bugzilla.redhat.com/show_bug.cgi?id=1556956

LGTM
Merging.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:51:29 on 2023.05.16
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 20745
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20745

Problem has been resolved in 9m 4s at 17:00:33 on 2023.05.16
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20745


Add conda installations to setup.cfg

Create a .yml file and pass it to setup.cfg (see https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#create-env-file-manually)

Right now, it's not possible to include conda packages in the setup.cfg installation (see 1 and 2 as examples) and thus in the PyPI installation. Since for now we are not releasing on conda, I'll close the issue as not planned. The user can install the conda packages before installing via pip deeprankcore, as explained in the readme.


Refactor middleware constants

Summary
An attempt to help make status codes maintainable among all the endpoints in addition to providing some helpful middlewares.

Assuming no further changes occur on #8, then this will be ready for review.


See demo of bug:
https://numbrojs.com/try-me.html?lang=en-US&value=-0.00490&format={"mantissa"%3A2%2C"trimMantissa"%3Afalse%2C"thousandSeparated"%3Atrue}

in numbro 2.2.0

Merci!


fix(cli) prevent start from stopping already-running Kong

add a check ensuring Kong (Nginx) is not running even before executing
the prefix_handler, which would maybe have consequences such as
re-generating SSL certs, or running migrations, etc...
add a test for this behavior
error now says "Kong is already running" instead of "Nginx is already
running", which is also probably more elegant.

lgtm, pending the tests...

Hi guys and thanks for fixing this. Any idea when this will see release? I'm running 0.9.5 and the problem is still present there.

@lsh-0 It's scheduled for 0.10 for which a first release candidate already is overdue...


tls13_meth.c: Check for negative return from EVP_CIPHER_CTX_get_iv_length()

Fixes Coverity 1598052

This pull request is ready to merge

Merged to master. Thank you!


GL3W Build_failed

Hi, i'm trying to install gl3w
Package: gl3w:x64-windows
Vcpkg version: 0.0.110-2018-04-30-5288e9e7029aa6b9b491bc2197b87a09778825b4
but i've got the following error :
File does not have expected hash:
File path: [{path_to_vcpkg}/vcpkg/downloads/temp/glcorearb-2018-02-27.h ]
Expected hash: [ 02c3672606e6360f2e1e8335fe581bc2d2b3d518b0f24b0c327006a70de07261dace4b53b13e93029a8eb2af43bcba904c4392f2c35ac512c7f278534ef8eb5d ]
Actual hash: [ dfb4c54e7a8b0f2fd5d1e2f41e9d49057765b52e17989c115ec92d673a9d84732f9b990babdd3aab624eb4f613b88c2b02aced46ec99c5341836133bcf22fdab ]
The file may have been corrupted in transit.  This can be caused by
proxies.  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY
environment variables to
"https://user:password@your-proxy-ip-address:port/".
Call Stack (most recent call first):
scripts/cmake/vcpkg_download_distfile.cmake:162 (test_hash)
ports/gl3w/portfile.cmake:13 (vcpkg_download_distfile)
scripts/ports.cmake:72 (include)

I can confirm this and the hash of source has changed.
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:87 (message):
File does not have expected hash:
File path: [ .../vcpkg/downloads/temp/glcorearb-2018-02-27.h ]
Expected hash: [ 02c3672606e6360f2e1e8335fe581bc2d2b3d518b0f24b0c327006a70de07261dace4b53b13e93029a8eb2af43bcba904c4392f2c35ac512c7f278534ef8eb5d ]
Actual hash: [ 2d8ff9263cdb317b08af9dc7759c61d162dcc560eb83c0de5aa26af9898244dbc475c799f81bd237615f8369bf923932534c30a69b8cc977eb9e3a4ce6a4b714 ]

Just as a temporal solution is to replace expected hash to actual hash inside ./ports/gl3w/portfile.cmake file.

While using the new hash above does allow attempted installation, the compilation will still fail due to a missing header required by the new file. This is fixed in the gl3w repository but that hasn't yet filtered down to vcpkg. (see here for the relevant gl3w commit)

Thanks for posting this issue.
Please open a new issue if this is still a problem for you.


Had to submit twice to effectively execute the request

Expected Behavior
When I click on the "send" button on a request with a prompt funtion in the URI, a popup appear in order to provide the information missing.
After setting the desired value, a single click on the button "submit" should send the request.
Actual Behavior
When I click on the "submit" button, instead of sending the request, the popup disappear, et reappear. I have to click again, a second time, on the button "submit", and now the request is sent.
Reproduction Steps

select a request with a prompt functionin the uri
click on send button
provide the desired value and click on submit button

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnium Version
0.2.1-b
What operating system are you using?
Windows
Operating System Version
Windows 10 Entreprise 22H2
Installation method
portable
Last Known Working Insomnium version
No response

hey thanks for the report; I will need to investigate the prompt function more for this

Hello, many thanks.
I found out this behaviour is not related specifically to Insomnium. It is prefesent also in the version 2023.5.8 of Insomnia.
And not only with the prompt function. We can reproduce it in the preferences, in the plugins tab :

in Insomnium or Insomnia 2023.5.8, we have the click twice on the check boxe of a plugin to toggle it.
In Insomnia v2023.4.0 no issues with prompt function, of in the preferences/plugin tab. One click is enough to do the action

Something changed between version 2023.4.0 -- 2023.5.8 of Insomnia. I will try to take some time to see which version on Insomnia introduce this issue.

Finally, it was fast to find the release of Insomnia which introduce this issue !

2023.4.0 : no issue !
2023.5.0 and after : issue present !


Added bottle

Link
https://github.com/crystal-data/bottle
Checklist

[x] - Shard is at least 30 days old.
[x] - Shard has CI implemented.
[x] - Shard has daily/weekly periodic builds (ideally with Crystal latest and nightly).

Does it require installing third party C libs? LAPACK/ BLAS? If yes, would be nice to have such a section in Readme.
Why do you need this? https://github.com/crystal-data/bottle/blob/0ca31ecf6e8f5a802b67070fbdd5dbf39073e93a/src/bottle.cr#L10-L11

@veelenga I updated the Readme with prerequisite requirements.  I also removed the code snippet you referenced, that was old debug code :P

@christopherzimmerman Might be worth looking at https://github.com/crystal-lang/shards/blob/master/SPEC.md#libraries if it requires external libs.


[WIP] Use more prebuilt Docker images for CI tasks

What is this PR trying to achieve?
Continuing the build work from Friday; I’m pushing more code into the dockerfiles repo and calling it from here.
See https://github.com/wellcometrust/dockerfiles/pull/7 for where these are all going.
Who is this change for?
🚤

That shaves another 4 minutes off the total build time (hard to tell cumulative, as I rearranged the build tasks between the last point of comparison). 🎉

I’d already pushed local builds so the tests on this branch would pass; merging.


Doxy

I believe that the better fix for this would be to update the CI images to use Doxygen 1.8.19, where this Doxygen bug seems to be fixed.
We could also mention this in the "FAQ" document I mentioned in #222 until Doxygen 1.8.19 lands in more locations

Update:
Doxygen 1.8.18 appears to generate the correct documentation with the following changes necessary:
diff --git a/doxygen/CMakeLists.txt b/doxygen/CMakeLists.txt
index edce0af..20c3150 100644
--- a/doxygen/CMakeLists.txt
+++ b/doxygen/CMakeLists.txt
@@ -19,8 +19,12 @@ set(DOXYGEN_INTERACTIVE_SVG YES)

 set(DOXYGEN_USE_MDFILE_AS_MAINPAGE README.md)

+set(DOXYGEN_STRIP_FROM_PATH ${CMAKE_SOURCE_DIR})
+
 doxygen_add_docs(Doxygen ALL
   COMMENT "Generating documentation"
   ${CMAKE_SOURCE_DIR}/README.md
+  ${CMAKE_SOURCE_DIR}/LICENCE.md
+  ${CMAKE_SOURCE_DIR}/CODE_OF_CONDUCT.md
   ${CMAKE_SOURCE_DIR}/include
 )

Then can you post that summary here? Discord is a great way to lose things

Comment pulled from Discord (15/08/2020)
In short latest has relative path support, but many older versions dont.
And the pre shipped one on Ubuntu doesnt
So either we can ignore the Ubuntu LTS users and go for latest.
Or provide backwards support

We're already ignoring LTS users older than 20.04 since the CI environment only targets 20.04 - for end users there's no real reason to be on anything older than this. However, the diff I mentioned earlier does not work in 1.8.17, which is what 20.04 uses.
Because of this, I'm going to tentatively approve this as a workaround, with a heavy note that as soon as we can, we should revert back to the patch I mentioned in https://github.com/novelrt/NovelRT/pull/221#issuecomment-674341322.


Crashing on Opening

Microsoft PowerToys version
0.66.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
General
Steps to reproduce

✔ Expected Behavior
Clicking on system tray Icon to open app.
❌ Actual Behavior
Crash dumps trying to open the settings or app.  Has created a 60 mb log file full of
[2023-01-06 18:20:45.3173] [INFO] [C:\a_work\1\s\src\modules\launcher\PowerLauncher\MainWindow.xaml.cs::67]
Send Run settings telemetry
[2023-01-06 18:20:45.3173] [INFO] [C:\a_work\1\s\src\modules\launcher\PowerLauncher\MainWindow.xaml.cs::67]
Send Run settings telemetry
[2023-01-06 18:20:45.3173] [INFO] [C:\a_work\1\s\src\modules\launcher\PowerLauncher\MainWindow.xaml.cs::67]
Send Run settings telemetry
[2023-01-06 18:20:45.3173] [INFO] [C:\a_work\1\s\src\modules\launcher\PowerLauncher\MainWindow.xaml.cs::67]
Send Run settings telemetry
[2023-01-06 18:20:45.3173] [INFO] [C:\a_work\1\s\src\modules\launcher\PowerLauncher\MainWindow.xaml.cs::67]
Send Run settings telemetry
Other Software
No response

/bugreport
To allow to compare install path contents.

Checking installationFolderStructure.txt, it seems PowerToys.Interop.dll on dll/Interop/ is 0.66 but almost everywhere else it's still 0.65.
Something seems to have failed on hard linking (or removing the previous links).
@badgerbadgerdroid
Would you mind:
1 - Uninstall PowerToys.
2 - See if the PowerToys install directory (default: C:\Program Files\PowerToys) is still there. If it is, delete it manually.
3 - Install PowerToys.
Please let us know if this works for you.

Reinstall was successful thanks.


Gmsh: Various fixes for mesh format v4.1

Oh, you where too fast! I'll make a new one to fix some weird issue with numpy dtypes

Never mind that. This is what happens if the tests are missing. :)
Do you have a file with periodic data handy? You could add it to the gh-pages branch in a separate PR and use it for testing as in test_vtk.py.


customElement support?

It will be really nice if we can use customElement syntax, e.g.,
<code-mirror mode="javascript" theme="dracula">
function thisIsSomeJavaScript() {
  console.log('hello!')
}
</code-mirror>

There is a separate package for this, https://github.com/hughsk/element-codemirror , though since customElement is standard and also a nicer API, I think it makes sense to incorporate to the main repository.  Also the implementation is small and pretty straightforward as far as I can see.

No, this is out of scope for the core library. Even if all major browsers supported custom elements (they don't), it's not clear at all yet that they are the right way to go in, so this library is going to remain agnostic to them.


No sudo, how to install external dependencies

Hello,
I'm trying to use thematrix/tox image on a private project build on CircleCI. One of the dependencies requires the system package libmagic to be installed.
But apparently, sudo isn't installed on this Docker image, so as a consequence I don't know how to install any Debian package.
Do you have any hint on how to make it work?

nevermind, found it. using apt-get install directly, without sudo, it works.
Sorry for the noise. At least the issue will stay there to eventually help others who would ask the question.


Fix regression when trying to bind an EpollDatagramChannel with port

only
Motivation:
4b7dba1 introduced a change which was not 100 % complete and so
introduce a regression when a user specified to use
InetProtocolFamily.IPv4 and trying to bind to a port (without specify
the ip).
Modifications:

Fix regression by respect the InetProtocolFamily
Add unit test

Result:
Fix regression when binding to port explicit

@hyperxpro you can workaround this by using bind(InetSocketAddress) and not bind(port)


[stable/metrics-server]

What this PR does / why we need it:
Fix cluster roles for metrics-server. In this way metrics-server is able to list all namespaces.
Metrics-server version bump
Special notes for your reviewer:
Ref: https://github.com/kubernetes-incubator/metrics-server/issues/293
Ref: https://github.com/kubernetes-incubator/metrics-server/pull/296
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @olemarkus

/lgtm

/retest

Please merge this pull request

/retest

/ok-to-test

/retest

@olemarkus and now?

Looks like metrics-server is not stopping properly. Not sure if that is related to the version bump. You could try splitting this PR in two

We have a concurrent PR here: https://github.com/helm/charts/pull/15919/files

We have a concurrent PR here: #15919

This one has some unresolved comments

@olemarkus I did it https://github.com/helm/charts/pull/16606

I will close this because is useless. Ref: https://github.com/helm/charts/pull/16618 or https://github.com/helm/charts/pull/16606

https://github.com/helm/charts/pull/16654


Fix quick-pick pill vertical alignment issue

This PR fixes #89049
Since the pill itself has an explicit line-height: 11px we need to sent the same line-height for the container element so that the container element has the same heigh (18px) as the pill. Below is a screenshot of the quick-pick dialog with the proposed change.

Abandoning the PR since my fork got into a funky state. Will re-create it from a clean fork.


Fix auth README links

PR summary
Updates missed links from https://github.com/IBM/cloudant-node-sdk/pull/76
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the
Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Yes, this should be fixed in the README template that gets generated for all SDKs.  Closing issue.

Yes, this should be fixed in the README template that gets generated for all SDKs.  Closing issue.


Retry operator argument semantics fix

Change the parameter of retry to be counted one more time.
The parameter 'times' in retry operator actually means the total number of tries, not the number of retries. This is some misunderstanding due to operator's naming and is explicitly documented in some Rx implementation descriptions:

RxNet: http://www.introtorx.com/uat/content/v1.0.10621.0/11_FlowControl.html#Retry
RxJS: https://github.com/Reactive-Extensions/RxJS/blob/master/doc/api/core/operators/retry.md

@elelel,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft Open Technologies, Inc.. We will now review your pull request.
Thanks,
Microsoft Open Technologies, Inc. Pull Request Bot

Thank You!


[local_auth] IOS useErrorDialogs= false not working if locked by the system

I'm doing and custom UI for error popup of FaceID. However, even I had set useErrorDialogs = false for get the error code instead default popup, it's still not working of lockout case.
my code was as same as the README.md of local_auth but adding bioTmetricOnly = true in option,
final bool didAuthenticate = await auth.authenticate(
          localizedReason: 'Please authenticate to show account balance',
          options: const AuthenticationOptions(useErrorDialogs: false,  bioTmetricOnly = true));
      // ···
    } on PlatformException catch (e) {
      if (e.code == auth_error.notAvailable) {
        // Add handling of no hardware here.
      } else if (e.code == auth_error.notEnrolled) {
        // ...
      } else {
        // ...
      }
    }

Hi @felixfok-dh, please provide the version of local_auth and local_auth_ios you're using.
Please also provide the complete code sample that you use.
Also, what do you see when you are supposed to be locked out? Can you share some screenshots/recordings?
Thank you

Hi @felixfok-dh, please provide the version of local_auth and local_auth_ios you're using.
Please also provide the complete code sample that you use.
Also, what do you see when you are supposed to be locked out? Can you share some screenshots/recordings?
Thank you

This is my first time opening a issue in GitHub, sorry for missing those details:
local_auth: 2.1.0
local_auth_ios: 1.0.9
Complete code sample that I use:
Future<bool> getFaceID() {
final LocalAuthentication _localAuth = LocalAuthentication();
bool authenticated = false;
    try {
      authenticated = await _localAuth.authenticate(
          localizedReason: 'Please authenticate to show account balance',
          options: const AuthenticationOptions(
              useErrorDialogs: false, biometricOnly: true));

    } on PlatformException catch (e) {
      authenticated = false;
    }
    return authenticated;
)

As the Readme.md of local_auth, I supposed will get the exception when useErrorDialogs: false so I can do my customise. However, it still return the system dialog.

Hi @felixfok-dh, is the dialog below what you see when you get locked out?

Hi @felixfok-dh, is the dialog below what you see when you get locked out?

Hi @danagbemava-nc , yes, but I want to show custom dialogs so I set useErrorDialogs: false  for getting the exception as the readme.md said. However, it still show the above dialog.

Hi @felixfok-dh, I just wanted to confirm that that dialog was what you were seeing.
Steps to reproduce

Run the sample below
Fail biometric authentication several times to get the system to lock you out
Click the Authenticate: biometrics only button
You should see the dialog in https://github.com/flutter/flutter/issues/117810#issuecomment-1369788199

Observation
It looks like in the handleErrors method of the plugin, the case for locked out doesn't take into account the useErrorDialogs parameter
See https://github.com/flutter/plugins/blob/main/packages/local_auth/local_auth_ios/ios/Classes/FLTLocalAuthPlugin.m#L270-L276 unlike the case for not enrolled
See https://github.com/flutter/plugins/blob/main/packages/local_auth/local_auth_ios/ios/Classes/FLTLocalAuthPlugin.m#L255-L265
Labeling for further insight from the team.

minimal sample
// Copyright 2013 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

// ignore_for_file: public_member_api_docs, avoid_print

import 'dart:async';

import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:local_auth/local_auth.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatefulWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  State<MyApp> createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  final LocalAuthentication auth = LocalAuthentication();
  _SupportState _supportState = _SupportState.unknown;
  bool? _canCheckBiometrics;
  List<BiometricType>? _availableBiometrics;
  String _authorized = 'Not Authorized';
  bool _isAuthenticating = false;

  @override
  void initState() {
    super.initState();
    auth.isDeviceSupported().then(
          (bool isSupported) =>
              setState(() => _supportState = isSupported ? _SupportState.supported : _SupportState.unsupported),
        );
  }

  Future<void> _checkBiometrics() async {
    late bool canCheckBiometrics;
    try {
      canCheckBiometrics = await auth.canCheckBiometrics;
    } on PlatformException catch (e) {
      canCheckBiometrics = false;
      print(e);
    }
    if (!mounted) {
      return;
    }

    setState(() {
      _canCheckBiometrics = canCheckBiometrics;
    });
  }

  Future<void> _getAvailableBiometrics() async {
    late List<BiometricType> availableBiometrics;
    try {
      availableBiometrics = await auth.getAvailableBiometrics();
    } on PlatformException catch (e) {
      availableBiometrics = <BiometricType>[];
      print(e);
    }
    if (!mounted) {
      return;
    }

    setState(() {
      _availableBiometrics = availableBiometrics;
    });
  }

  Future<void> _authenticate() async {
    bool authenticated = false;
    try {
      setState(() {
        _isAuthenticating = true;
        _authorized = 'Authenticating';
      });
      authenticated = await auth.authenticate(
        localizedReason: 'Let OS determine authentication method',
        options: const AuthenticationOptions(
          stickyAuth: true,
        ),
      );
      setState(() {
        _isAuthenticating = false;
      });
    } on PlatformException catch (e) {
      print(e);
      setState(() {
        _isAuthenticating = false;
        _authorized = 'Error - ${e.message}';
      });
      return;
    }
    if (!mounted) {
      return;
    }

    setState(() => _authorized = authenticated ? 'Authorized' : 'Not Authorized');
  }

  Future<void> _authenticateWithBiometrics() async {
    bool authenticated = false;
    try {
      setState(() {
        _isAuthenticating = true;
        _authorized = 'Authenticating';
      });
      authenticated = await auth.authenticate(
        localizedReason: 'Scan your fingerprint (or face or whatever) to authenticate',
        options: const AuthenticationOptions(
          stickyAuth: true,
          biometricOnly: true,
          useErrorDialogs: false,
        ),
      );
      setState(() {
        _isAuthenticating = false;
        _authorized = 'Authenticating';
      });
    } on PlatformException catch (e) {
      print(e);
      setState(() {
        _isAuthenticating = false;
        _authorized = 'Error - ${e.message}';
      });
      return;
    }
    if (!mounted) {
      return;
    }

    final String message = authenticated ? 'Authorized' : 'Not Authorized';
    setState(() {
      _authorized = message;
    });
  }

  Future<void> _cancelAuthentication() async {
    await auth.stopAuthentication();
    setState(() => _isAuthenticating = false);
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: const Text('Plugin example app'),
        ),
        body: ListView(
          padding: const EdgeInsets.only(top: 30),
          children: <Widget>[
            Column(
              mainAxisAlignment: MainAxisAlignment.center,
              children: <Widget>[
                if (_supportState == _SupportState.unknown)
                  const CircularProgressIndicator()
                else if (_supportState == _SupportState.supported)
                  const Text('This device is supported')
                else
                  const Text('This device is not supported'),
                const Divider(height: 100),
                Text('Can check biometrics: $_canCheckBiometrics\n'),
                ElevatedButton(
                  onPressed: _checkBiometrics,
                  child: const Text('Check biometrics'),
                ),
                const Divider(height: 100),
                Text('Available biometrics: $_availableBiometrics\n'),
                ElevatedButton(
                  onPressed: _getAvailableBiometrics,
                  child: const Text('Get available biometrics'),
                ),
                const Divider(height: 100),
                Text('Current State: $_authorized\n'),
                if (_isAuthenticating)
                  ElevatedButton(
                    onPressed: _cancelAuthentication,
                    child: Row(
                      mainAxisSize: MainAxisSize.min,
                      children: const <Widget>[
                        Text('Cancel Authentication'),
                        Icon(Icons.cancel),
                      ],
                    ),
                  )
                else
                  Column(
                    children: <Widget>[
                      ElevatedButton(
                        onPressed: _authenticate,
                        child: Row(
                          mainAxisSize: MainAxisSize.min,
                          children: const <Widget>[
                            Text('Authenticate'),
                            Icon(Icons.perm_device_information),
                          ],
                        ),
                      ),
                      ElevatedButton(
                        onPressed: _authenticateWithBiometrics,
                        child: Row(
                          mainAxisSize: MainAxisSize.min,
                          children: <Widget>[
                            Text(_isAuthenticating ? 'Cancel' : 'Authenticate: biometrics only'),
                            const Icon(Icons.fingerprint),
                          ],
                        ),
                      ),
                    ],
                  ),
              ],
            ),
          ],
        ),
      ),
    );
  }
}

enum _SupportState {
  unknown,
  supported,
  unsupported,
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.10, on macOS 13.1 22C65 darwin-arm, locale en-GB)
    • Flutter version 3.3.10 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 135454af32 (3 weeks ago), 2022-12-15 07:36:55 -0800
    • Engine revision 3316dd8728
    • Dart version 2.18.6
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.74.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.1 22C65 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 108.0.5359.124

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[!] Flutter (Channel master, 3.7.0-14.0.pre.30, on macOS 13.1 22C65 darwin-arm64, locale en-GB)
    • Flutter version 3.7.0-14.0.pre.30 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c53501d835 (2 hours ago), 2023-01-03 23:00:08 -0500
    • Engine revision 32c468507b
    • Dart version 3.0.0 (build 3.0.0-77.0.dev)
    • DevTools version 2.20.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.74.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.1 22C65 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 108.0.5359.124

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

This behavior has been in place from the beginning: https://github.com/flutter/plugins/pull/151
@stuartmorgan do you think fixing this would be a breaking change, since perviously the code would handle a popping up the dialog, but if we fix this then the developer would have to handle that error?

Since the behavior is obviously incorrect, and contrary to the documentation, I would just treat it as a bug fix.

So confirmed this is a bug of auth_local?
Then I'll wait for the bug fix.
Thanks all.


Harshal0902/gosw32

Issue Number
ISSUE #58

Describe the changes you've made
Added a scroll to top button.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

@nanda-mik please let me know, if any changes required

Yeah its good. Try to resolve the merge conflicts.

@AVS18 I am not getting how to resolve conflicts.

@AVS18 can I work on another issue?

Hey @Harshal0902  Please do one thing, remove the package-lock.json from the Pull Request. I will update the same manually, I have copied your package-lock.json file code. Please update me once done so that I may merge

@manav014 done

@manav014 now all tests are failing, should I add it again?

Yeah just add it as checks are to be done and also now I will tell you the line which is conflicting within a few minutes after adding it again.

@manav014 Done

So @manav014 can i work on other issue now?

Yeah! I am working on it and will update you with the solution.

So right now can I work on new issue or not?

Yes you may.

But please keep the solution, i would be proposing, on priority in future for this issue.
Thank You

@Harshal0902 Please use git pull upstream main in the GOSW32 branch and update it with project main as it is not updated with the main branch. Please do the same on priority basis.

@manav014 done. Bu it is showin
$ git pull upstream main
From https://github.com/GameofSource-GFG/Web-Development

branch            main       -> FETCH_HEAD
Already up to date.

@manav014 can you please assign issue GOSW25 to me?

I pushed your changes in the main branch manually as the reason of conflict was unknown. So this PR would be graded and considered as merge. I am closing it.

Thank a lot @manav014 for helping me out. Cay you please assign issue GOSW25 to me?


Update docs: initializeCommand execution frequency

See https://github.com/microsoft/vscode-remote-release/issues/8179, https://github.com/microsoft/vscode-remote-release/issues/9278 which report behavior that initalizeCommand is executed more frequently than is indicated in the documentation.

Is there any update with respect to this? I'm experiencing similar issues whenever I have to relaunch my window for my debug extensions to launch correctly

i will work on update today


[Bug]: Compare Feature Missing

Contact Details
No response
What happened?
I am sure I have seen it there before, but I can't seem to find the Compare function from the preview. The one where I can check the local vs online diff.

I am currently running the preview ON.
What version of the tool are you using?
205
What version of Visual Studio Code are you running?
1.66.0
Relevant log output
No response

Not sure what happened, but suddenly it's back.. I didn't even restart the app, but the button is there now:)


refactor: embed mermaid within markdown as now supported on github and in intellij which is good enough for me

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

GitHub release
crate.io packages (1 packages published)

Your semantic-release bot :package::rocket:


Add tests

Hard to draw inspiration for unit tests on an analysis package. I guess checking types, dimensions, etc. is useful but also comparing the direct output of functions gets tricky with expensive (order several minutes) functions

Closing as tests have been added in #23.


StopIteration in GET /clan/PYPCV2CQ/2RV9P089C.json

Error in ClashLeaders
StopIteration in GET /clan/PYPCV2CQ/2RV9P089C.json
View on Bugsnag
Stacktrace
clashleaders/views/clan_player.py:13 - clan_player_json

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
StopIteration in GET /clan/PYPCV2CQ/2RV9P089C.json


docker build instructions (Makefile:51 recipe for target 'tippecanoe' failed)

the docker build instructions yields this for me on Amazon LINUX:
docker build -t tippecanoe:latest .
... {lots of activity, then}
Makefile:51: recipe for target 'tippecanoe' failed
The command '/bin/sh -c make   && make install' returned a non-zero code: 2

ah, found the Centos version of the Dockfile (Dockerfile.centos7) which succeeds.
closing.


Won't stop

When I try to stop the T.O.P. in the example (I clicked twice), I get:
err.txt

this worked for me (line 41 in example's main.dart):
 if ((await UtopicTorOnionProxy.stopTor()) != null) {


next build not updating content in out folder

Not sure if it's a next or nextra thing, but when I do bun run build, the built content doesn't contain the new content.
I have to remove .next folder before it picks up the new content.
Below is the next.config.js that I have.
const withNextra = require('nextra')({
  theme: 'nextra-theme-docs',
  themeConfig: './theme.config.tsx',
})

const nextConfig = {
  output: 'export',
  images: {
    unoptimized: true
  }
}

module.exports = withNextra(nextConfig)

And package.json
{
  "scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start"
  },
  "dependencies": {
    "next": "^14.2.4",
    "nextra": "^2.13.4",
    "nextra-theme-docs": "^2.13.4",
    "react": "^18.3.1",
    "react-dom": "^18.3.1"
  }
}

It will produce an out folder but the content is not the latest.

https://github.com/orgs/vercel/discussions/2079
Few workarounds mentioned here

https://github.com/orgs/vercel/discussions/2079
Few workarounds mentioned here, unless you refer to a local build?

Thanks. Yes, I'm not using vercel to host. Am building and moving the 'out' folder with azure cicd pipeline.


curl 8.7.1: --dns-ipv6-addr and --dns-ipv4-addr options

Hi
Using curl 7.76.1 from Alma Linux 9.4 I'm able to set the IPv4 and IPv6 DNS, but with curl 8.7.1 from this repo, it shows the following results:
curl --dns-ipv4-addr <IP_ADDRESS> --dns-ipv6-addr 2606:4700:4700::1111 --compressed -k -m 10 -X GET https://example.com
curl: (6) Could not resolve host: example.com
Thanks

I have reproduced it here as well.
The results are still different on different servers, some resolve DNS properly using only --dns-ipv4-addr, some need to use --dns-ipv6-addr.
But using --dns-servers are all fine.
curl --dns-servers <IP_ADDRESS> \
--compressed -m 10 \
-v -s -o /dev/null \
-X GET https://example.com

I'll check what --dns-ipv4-addr and --dns-ipv6-addr are about

Hi
My example is not good since I used CF dns servers, but --dns-ipv4-addr and --dns-ipv6-addr binds the request to a specific ip address.
Thanks

Yeah, I checked the docs and it says that --dns-ipv4-addr and --dns-ipv6-addr are used to bind the local ip addresses.
You can check whether these commands were executed correctly.
ping -c 4 -I [local IPv4 address] [your dns server IPv4]
ping -c 4 -I [local IPv6 address] [your dns server IPv6]

dig -b [local IPv4 address] @[your dns server IPv4] example.com
dig -b [local IPv6 address] @[your dns server IPv6] example.com


[AutoRelease] t2-servicenetworking-2023-11-07-08107(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4718
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=5828

issue link:https://github.com/Azure/sdk-release-request/issues/4718


Git settings missing

Issue Type: Bug
Git settings are missing from User Settings.

Only gitlens settings are displayed and giving getlens.advanced.git a path to git.exe did not resolve the issue.
Also, the Source Control tab displays "There are no active source control providers."
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:51:32.889Z)
OS version: Windows_NT x64 10.0.16299
Extensions (5)

Extension
Author (truncated)
Version

Bookmarks
ale
0.19.1

gitlens
eam
8.4.1

vsc-material-theme
Equ
2.1.0

cpptools
ms-
0.17.4

vim
vsc
0.13.1

(1 theme extensions excluded)

If you open F1, Developer: Toggle Developer Tools, is there any error in the console?
And does F1, Developer: Show Running Extensions show the Git extension there?

There aren't any errors in the console and the only Git related extension running is GetLens.

Interesting. If you download our ZIP release and run Code from there... does Git work?

Still no Git options after downloading that release.

Doesn't make much sense 🤔
How about if you run with a fresh new user app data? code --user-data-dir SOMEFOLDER

That did the trick! There are Git settings and the Source Control tab populates.

Can you ZIP up your usual user data dir and send me that to jomo @ microsoft dot com?


Change "ctrl" in help text to COMMAND to show correct text in apple.

Hi!
I was trying to add some new features in a personal fork when I noticed that some help text is using a RAW "CTRL" string instead of the defined COMMAND variable that uses different strings taking into account the OS, so this is just a small change to fix that.

Hi there, the use of "Ctrl" is intentional, since those particular shortcuts indeed use ctrl, even on macOS. Or did you spot a shortcut that responds to Cmd instead of Ctrl without being labeled as such?

Hi.
No, I just assumed it was supposed to work the other way and it was expected that those "Ctrl" keys changed to Cmd in macOS. I forgot to try all of them to be sure since I had no macOS computer available when I found that.
Sorry!


Scroll Functionality

Is your feature request related to a problem? Please describe.
No issues
Describe the solution you'd like
It would be great to have scrolling functionality with a modifier/hyper key and some letter buttons or even better a trackpad move. This might help help to emulate trackpoint by some degree
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

Mouse keys ambidextrous rule for Karabiner
Scroll 2 by Ryan Hanson

Well It do have scrolling functionality according to Doc-Navigation 😉
If you hold ✱⇧⌥ (Capslock + Shift + Control) all together.  The navigation cursor will turn into mouse wheel (horizontal & vertical) and works just like trackpad.


one-way-select with prompt option auto-selects first element, so you never see the prompt

This example with "prompt=..."
https://github.com/DockYard/ember-one-way-controls/blob/master/docs/one-way-select.md#adding-a-blank-or-prompt-option
auto-selects the first option from start. So the prompt never shows up.
You can use promptIsSelectable=true to work around this issue. But then the user can select the prompt again after he already made a choice. So you also need to validate for presence.

Hi @RobIsHere, i tried to reproduce your issue but failed - please see this twiddle!

I have this error again in a new app that's on current release versions all over.
{{one-way-select fileChangeSet.customGalleryPicture options=pictureFiles prompt="Please select one:" optionValuePath="id" optionLabelPath="filename" update=(action (mut fileChangeSet.customGalleryPicture))}}

with fileChangeSet beeing an ember-changeset object based on an ember-data model.


Use all --testPathPattern and  args in testPathPattern

Summary
This PR changes:

the type of the --testPathPattern arg to match the multiple allowed values
the way multiple --testPathPattern args are joined to form a valid regular expression
the way --testPathPattern args are joined with the default <regexForTestFiles> args

This closes #5037.

Test plan
Tests have been added to check the way the args are joined.
Evaluating the two test cases from #5037 also produce expected patterns:
> jest --testPathPattern=aa --testPathPattern=bb
...
WAS: Pattern: aa,bb
NOW: Pattern: aa|bb

> jest aa --testPathPattern=bb
...
WAS: Pattern: bb
NOW: Pattern: aa|bb

Codecov Report

Merging #5066 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5066      +/-   ##
==========================================
- Coverage   60.76%   60.75%   -0.01%
==========================================
  Files         198      198
  Lines        6603     6602       -1
  Branches        4        4
==========================================
- Hits         4012     4011       -1
  Misses       2591     2591

Impacted Files
Coverage Δ

packages/jest-config/src/normalize.js
91.95% <ø> (ø)
:arrow_up:

packages/jest-config/src/validate_pattern.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for contributing to Jest <3

Happy to see things move so quickly!
Any suggestions on the next issue? I'm looking at Help Wanted. :grin:

@seanpoulter help with getting open PRs over the line would be great 🙂 As well as hardening the codeframe merged yesterday (I'll create an issue for it)


Problem while compiling on Ubuntu 12.04 LTS

I have given up trying to get Fluxengine to work on my windows 7 box, and have moved on to Ubuntu 12.04 LTS.  But since there are no binaries, I am trying to compile it.
So far, I have found that gcc/g++ 6 gets the farthest along, and only errors out about 1/3 of the way through the ~278 files, when trying to compile through the Makefile with 'CXX lib/usb/usb.cc', and (among other things) says "lib/usb/usb.cc:37:44: error: 'libusb_strerror' was not declared in this scope".
I did install the dependencies listed as: ninja-build, libusb-1.0-0-dev, libsqlite3-dev, and they seem to be fine.
Sorry, this may be a noob error, but I have done a fair bit of coding in several languages, including c and many variants.  But compiling on Ubuntu is new to me.  Any ideas?  Thank you!

Ubuntu 12 is pretty old --- it was end-of-life'd three years ago... probably your version of libusb is too old. Is there any way you can try on a newer distribution?


[Bug] JavaDoc (generated?) has Chinese text in it (as appears in Intellij)

Search before asking

[X] I searched in the issues and found nothing similar.

Version
2.10.3, didn't check others
Minimal reproduce step
Intellij, mouse over SinkContext to see context JavaDoc
What did you expect to see?
English JavaDoc
What did you see instead?

Anything else?
Does the language of the generated JavaDoc depend on Release manager's locale?
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Cannot reproduce. It seems an IDE settings issue, especially the Chinese content is not API content but the display template.

You may check https://www.jetbrains.com/help/idea/working-with-code-documentation.html#generate-javadoc about setting locales.

Closing...
Invalid.


[Badge] Unread count behavior

With https://github.com/BeatPlus/Protonmail/commit/3a62e2c5a71de492b4534484256e4a10ff0f9a25 the badge looks better but some concerns araise.
From 1 to 4 digits badge:

The badge works well on 2 digits, some 3 digits look good too but >100 gets ugly. We can simply replace with 99+ or something similar so it does not break.
In addition, the badge does not appear when logged in even if you have unread messages. I do like this behavior because it will only appear once a new message comes. Another way is to check for unread messages when the tab loads. Thoughts?

"99+" is great!
The reason that the unread emails don't show up immediately is because of Protonmail. When you login to your account the title doesn't display unread emails until you

Get a new email
Navigate from Inbox and navigate back to Inbox

From my perspective it's a bug. I think the best would that we (and Protonmail) always display total unread count.
I can contact them regarding this and hear their thoughts.


Add command line options to unity-app-generator to optionally push docker image into a registry

Add a command line option that enables pushing in a command line specified Docker image repository.

Implemented at 38b86ffcfc3c64e3df35a21914c153c0c7b9a657


[2.x] Update to Node 18 Runtime and update used Chromium Version

This PR updates the package to use the Node 18 Runtime and updates the used Chromium Version / Layer to the latest version. (fixes #83)

Note
v33 of shelfio/chrome-aws-lambda-layer doesn't seem to work with the package.
I've created my own layer based on Sparticuz/chromium which I've used in local development. I plan to create a sidecar-browsershot-layer which contains all required dependencies and I can keep up to date automatically.
Still need some time to set up the respective AWS account and automation to deploy the layer to all AWS regions

TODOs

[ ] Update Layer definition to use the right layer
[ ] Move aws-sdk dependency to a separate layer or use suitable layer.

Just noticed that shelfi/chrome-aws-lambda-layer has an open PR to keep the layer up to date automatically. https://github.com/shelfio/chrome-aws-lambda-layer/pull/59
Might put the idea of "making my own layer" on hold.
(Not keen on maintaining another project or creating a very similar layer that already exists)

Quick update here:

https://github.com/shelfio/chrome-aws-lambda-layer/pull/59 has been merged and the newest version can be used with sidecar-browsershot. Going forward, we will be using this layer embed Chromium.
puppeteer-core is now required to boot the Chromium instance. As I would like to make this package as simple as possible to use, I created a new layer that contains a compatible version of puppeteer-core. The layer will be called sidecar-browsershot-layer (will make the repo public as soon as it hits v1).

Alternative would be to put the burden of installing the correct version of puppeteer-core to the consumers of the package and update BrowsershotFunction to include userland node_modules-folders.

Currently struggling with the automated scripts and pipeline. My previous manual attempts create a layer that works without problems. My automated scripts – which just zip up a folder and push it to AWS – creat a layer that now throws Cannot find module 'puppeteer-core'-errors. 🤷