Android NullReference at GetContext

UNHANDLED EXCEPTION:
System.NullReferenceException: Object reference not set to an instance of an object.
at ZXing.Mobile.MobileBarcodeScanner.GetContext (Android.Content.Context context)
at ZXing.Mobile.MobileBarcodeScanner.Scan (Android.Content.Context context, ZXing.Mobile.MobileBarcodeScanningOptions options)
at ZXing.Mobile.MobileBarcodeScanner.Scan (ZXing.Mobile.MobileBarcodeScanningOptions options)
at ZXing.Mobile.MobileBarcodeScannerBase.Scan ()
at App4.MainPage+d__1.MoveNext ()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw ()
at System.Runtime.CompilerServices.AsyncMethodBuilderCore+<>c.b__6_0 (System.Object state)
at Android.App.SyncContext+<>c__DisplayClass2_0.b__0 ()
at Java.Lang.Thread+RunnableImplementor.Run ()
at Java.Lang.IRunnableInvoker.n_Run (System.IntPtr jnienv, System.IntPtr native__this)
at (wrapper dynamic-method) System.Object:7b87ce6b-d56f-4d47-a91e-06914b52665c (intptr,intptr)

Reproduction steps:

Create new blank PCL app, Xamarin.Forms v<IP_ADDRESS>0
Install ZXing.Net.Mobile.Forms v2.3.2
Add a button with Clicked handler ( basically sample code from readme):

public async void OnButtonClicked(object sender, EventArgs args)
{
    #if __ANDROID__
        // Initialize the scanner first so it can track the current context
        MobileBarcodeScanner.Initialize (Application);
    #endif

    var scanner = new ZXing.Mobile.MobileBarcodeScanner();
    var result = await scanner.Scan();
    if (result != null)
    {
        await DisplayAlert("Result", result.Text, "OK");
    }
}

When it comes to platform-specific initialization steps the only difference is that I didn't add the flashlight permission.

Yes, I had the same problem.
Fell back to using
var scanPage = new ZXingScannerPage();
which seems to work

Not sure if this is a workaround or "as designed" - looks like the sample from the Readme is for a Shared Xamarin.Forms project, rather than the platform specific. I achieved the desired behavior by moving the pieces wrapped in #if __ANDROID__ to MainActivity.cs in my Android project, like so:
    protected override void OnCreate(Bundle bundle)
    {
        TabLayoutResource = Resource.Layout.Tabbar;
        ToolbarResource = Resource.Layout.Toolbar;

        base.OnCreate(bundle);

        global::Xamarin.Forms.Forms.Init(this, bundle);
        global::ZXing.Net.Mobile.Forms.Android.Platform.Init();

        demoApp = new App();

        // This line is leveraging the android-specific implementation
        ZXing.Mobile.MobileBarcodeScanner.Initialize(Application);

        LoadApplication(demoApp);
    }

@JoeM-RP Just had the same problem, using the latest version of everything, and your solution still works for this problem nearly a year later. Well spotted. Cheers

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Working with submodules

While getting started in this repo I forget to update/initialize the submodules. I suspect that might be a common issue for folks so I was thinking we should consider adding a git submodule update --init to the beginning of the build scripts but given I'm pretty new to submodules I don't know if that will start to cause issue if there are changes to the modules or if there are patches applied to them. Anyone have enough experience to advise for or against always doing an update/init from the build scripts?

I would be fine with doing this if a project submodule dir that's in the build lineup doesn't exist, but I think we shouldn't do this blindly on each build: it would update away from dev commits on the submodules.


Route Isn't Displayed with Local Clone

If I clone the project, opening index.html in Firefox shows the map as expected, but as soon as I select two points the route never shows up. It works properly on the demo server. Steps:
git clone -b gh-pages https://github.com/Project-OSRM/osrm-frontend-v2.git
cd osrm-frontend-v2
firefox index.html
Select two points, the pins display but that is all.
Also "npm install" completes successful on the master branch, but that doesn't seem to fix the routing.

You need to load it from a local http server. For that occasions I have a small script:
https://gist.github.com/TheMarex/b5a44fa78a036a42f1a2
That starts up a http server in the current directory which listens on https://<IP_ADDRESS>:8337

Thanks for sharing the script, that works great to get the frontend working locally. Do you know what part of the frontend requires a httpd? I am interested in using it for a mobile routing application using cordova, but loading a httpd on the phone probably isn't a good solution.

One of the primary issues is probably that the OSRM server is referred by a protocol relative URL (//router.project-osrm.org/viaroute rather than for example http://router.project-osrm.org/viaroute) which causes problem when the page is loaded from file://. You can fix this by overriding the serviceUrl option to the router instance (L.Routing.OSRM).
However, there are other things that break when you use file://, such as not being able to load things with XHR even locally without running into cross-site issues in normal browsers. I have no idea how Cordova deals with this, but I guess there must be a way to load local assets with XHR.

Thanks perliedman! Changing from a relative URL to a complete URL allows it to work properly without a httpd locally. Now I need to work on getting it working with Cordova.

@TheMarex Is this something we should fix, too?

@DennisOSRM nothing to fix on our side. Running from file:// is just a bad idea in general.

Alright, fair enough. We should add a section to the readme explaining how to run locally and what dependencies that carries.


Update best-practices.md

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will not fix.

Closing the pull request


Custom search engine appends #gsc.tab=0 in every URL

Describe the bug
Custom search engine appends #gsc.tab=0 in every URL
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.sack.edu.lk/'
See the URL after loading the page
The URL will be changed to 'https://www.sack.edu.lk/#gsc.tab=0'

Expected behaviour
The page URLs should not change after loading
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS] All
Browser [e.g. chrome, safari] All

Additional context
This article will be helpful: https://techjourney.net/google-custom-search-engine-appends-gsc-tab0-in-url/
This issue can only be fixed by a person who has the login details of acf mail(or the account which was used when creating the custom search).
Are you working on it?
No

@gajindubandara which email did you use when creating the custom search?

We can remove it but I think if we remove that we will loss the our google search bar.

Shall I work on this?

@gajindubandara which email did you use when creating the custom search?

@kumuditha-udayanga gave me the required links for the custom search. I have no idea about the email thing

@kumuditha-udayanga gave me the required links for the custom search. I have no idea about the email thing

Ah right. Probably it was created with the ACF mail.

Yes, it was created using<EMAIL_ADDRESS>

Then, @kumuditha-udayanga could you please fix the issue by login into the admin account.
Steps to fix the issue: https://techjourney.net/google-custom-search-engine-appends-gsc-tab0-in-url/


[Feature] Introduce flink to dataproxy module

Description
Apache inlong contains four module, includes Ingest, Converge, Cache, Consume. After looking into the project, we
can find that there is a similar structure in several modules, we can call it "source - channel - sink"(SCS in short).

Ingest module, there is a customized "source - channel - sink" structure.
DataProxy module, we can see this module depends on flume project, which is a typical SCS structure.
Consume module, we can also think flink is also a SCS structure.

Several disadvantages:
In Ingest module, we may need to implement many sources ourselves, inculdes streaming and batch ingest, like cdc, file, socket...
In DataProxy module, as we known, the development of flume project is getting slower.
We can see that it's hard for us to maintain them, we need to learn the flume api and implements many connectors. The apache flink already contains rich connectors, and support sql grammar, batch and streaming source connectors, also
contains many other excellent features, we can benefit from it. So, here we can merge some functions of the Ingest, Converge module and introduce flink to dataproxy module.
Architecture
Here the new architecture diagram compared to origin.
Origin

New

@leo65535 @gosonzhang Thank you for the great thinking of InLong. I try to understand this question from another side.
we agree that apache flink has rich and mature connectors,  maybe we could add a plugin implement like DataProxy on Flink in the future. But at present, we will keep the flume implement for DataProxy, and will not replace it with Flink in the recent versions for some reasons:
1, InLong has an internal version that serving lots of Tencent businesses for years, we found that there are no MQ services that could process the super-large scale data, that why we add DataProxy(using flume) and develop TubeMQ.
2, we will focus on adding more data flow for the following versions,  like ClickHouse、IceBerg、HBase, and DataProxy support Pulsar, we will not replace it with Flink immediately.

Hi @dockerzhang @gosonzhang

But at present, we will keep the flume implement for DataProxy, and will not replace it with Flink in the recent versions

+1, you are right.

maybe we could add a plugin implement like DataProxy on Flink in the future.

Yeah, this is what I want to talk about, not to replace flume right now.

The issue has not been dealt with for a long time, so I closed it first


Error in E2E test setup: Internal error occurred: failed calling webhook "webhook.cert-manager.io": Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": x509: certificate signed by unknown authority

In #3815 @RinkiyaKeDad made it so that we would be able to easily see errors in the ./devel/addon/*/install.sh scripts and now I think I've found one of these errors:
 INFO: Analyzed target //devel/addon/samplewebhook:bundle (0 packages loaded, 246 targets configured).
INFO: Found 1 target...
[1 / 15] [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (3 actions, 0 running)
[72 / 75] [Prepa] SHA256 devel/addon/samplewebhook/sample/image-layer.tar.sha256
Target //devel/addon/samplewebhook:bundle up-to-date (nothing to build)
INFO: Elapsed time: 2.342s, Critical Path: 1.73s
INFO: 9 processes: 4 remote cache hit, 5 internal.
INFO: Build completed successfully, 9 total actions
INFO: Running command line: bazel-bin/devel/addon/samplewebhook/bundle
INFO: Build completed successfully, 9 total actions
Loaded image ID: sha256:3f24f1ea8d0f38a966d684946873500456e9b73db891fd1df0f90e26dbabd508
Tagging 3f24f1ea8d0f38a966d684946873500456e9b73db891fd1df0f90e26dbabd508 as sample-webhook:bazel
Image: "sample-webhook:bazel" with ID "sha256:3f24f1ea8d0f38a966d684946873500456e9b73db891fd1df0f90e26dbabd508" not yet present on node "kind-control-plane", loading...
Error from server (NotFound): namespaces "sample-webhook" not found
namespace/sample-webhook created
Release "sample-webhook" does not exist. Installing it now.

Error: Internal error occurred: failed calling webhook "webhook.cert-manager.io": Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": x509: certificate signed by unknown authority

from: https://prow.build-infra.jetstack.net/view/gcs/jetstack-logs/pr-logs/pull/jetstack_cert-manager/3838/pull-cert-manager-e2e-v1-20/1377592354835271681#1:build-log.txt%3A369
/area testing
/kind bug

Oops probably a false alarm.
I'd broken cainjector in the PR where this error occurred: https://github.com/jetstack/cert-manager/pull/3838

2021-04-01T14:12:39.196297988Z stderr F I0401 14:12:39.196054       1 start.go:107] "starting" version="1617286276" revision="e71433f58f29fdf3f03e0cf6bbad4a7656e4e395"
2021-04-01T14:12:39.801584887Z stderr F I0401 14:12:39.801274       1 leaderelection.go:243] attempting to acquire leader lease  kube-system/cert-manager-cainjector-leader-election...
2021-04-01T14:12:55.25026156Z stderr F I0401 14:12:55.250082       1 leaderelection.go:253] successfully acquired lease kube-system/cert-manager-cainjector-leader-election
2021-04-01T14:12:55.253661133Z stderr F I0401 14:12:55.251344       1 recorder.go:52] cert-manager/controller-runtime/manager/events "msg"="Normal"  "message"="cert-manager-cainjector-78fb9879cc-sqzcz_bfd6eedb-beda-408d-9814-d2e26ea57b86 became leader" "object"={"kind":"ConfigMap","namespace":"kube-system","name":"cert-manager-cainjector-leader-election","uid":"be01af3a-36ad-4a7c-a552-08dcd90907b4","apiVersion":"v1","resourceVersion":"757"} "reason"="LeaderElection"
2021-04-01T14:12:55.253692465Z stderr F E0401 14:12:55.251809       1 start.go:170] cert-manager/ca-injector "msg"="Error registering certificate based controllers. Retrying after 5 seconds." "error"="no kind is registered for the type v1.APIService in scheme \"pkg/runtime/scheme.go:101\""
2021-04-01T14:12:55.254638363Z stderr F E0401 14:12:55.254516       1 start.go:138] cert-manager/ca-injector "msg"="manager goroutine exited" "error"=null
2021-04-01T14:12:55.254893238Z stderr F E0401 14:12:55.254783       1 leaderelection.go:357] Failed to update lock: context canceled
2021-04-01T14:12:55.254907811Z stderr F I0401 14:12:55.254832       1 leaderelection.go:278] failed to renew lease kube-system/cert-manager-cainjector-leader-election: timed out waiting for the condition
2021-04-01T14:12:55.255034558Z stderr F I0401 14:12:55.254923       1 recorder.go:52] cert-manager/controller-runtime/manager/events "msg"="Normal"  "message"="cert-manager-cainjector-78fb9879cc-sqzcz_bfd6eedb-beda-408d-9814-d2e26ea57b86 stopped leading" "object"={"kind":"ConfigMap","apiVersion":"v1"} "reason"="LeaderElection"
2021-04-01T14:12:55.26344268Z stderr F E0401 14:12:55.255123       1 internal.go:521] cert-manager/controller-runtime/manager "msg"="error received after stop sequence was engaged" "error"="leader election lost"
2021-04-01T14:12:55.263890339Z stderr F Error: error registering secret controller: no kind is registered for the type v1.APIService in scheme "pkg/runtime/scheme.go:101"
2021-04-01T14:12:55.264884449Z stderr F Usage:
2021-04-01T14:12:55.264905255Z stderr F   ca-injector [flags]
2021-04-01T14:12:55.264912898Z stderr F
2021-04-01T14:12:55.264919045Z stderr F Flags:
2021-04-01T14:12:55.264924546Z stderr F       --add_dir_header                            If true, adds the file directory to the header of the log messages
2021-04-01T14:12:55.264931881Z stderr F       --alsologtostderr                           log to standard error as well as files
2021-04-01T14:12:55.264938138Z stderr F   -h, --help                                      help for ca-injector
2021-04-01T14:12:55.264945306Z stderr F       --kubeconfig string                         Paths to a kubeconfig. Only required if out-of-cluster.
2021-04-01T14:12:55.264951665Z stderr F       --leader-elect                              If true, cainjector will perform leader election between instances to ensure no more than one instance of cainjector operates at a time (default true)
2021-04-01T14:12:55.264959005Z stderr F       --leader-election-lease-duration duration   The duration that non-leader candidates will wait after observing a leadership renewal until attempting to acquire leadership of a led but unrenewed leader slot. This is effectively the maximum duration that a leader can be stopped before it is replaced by another candidate. This is only applicable if leader election is enabled. (default 15s)
2021-04-01T14:12:55.264973667Z stderr F       --leader-election-namespace string          Namespace used to perform leader election (defaults to controller's namespace). Only used if leader election is enabled
2021-04-01T14:12:55.264992935Z stderr F       --leader-election-renew-deadline duration   The interval between attempts by the acting master to renew a leadership slot before it stops leading. This must be less than or equal to the lease duration. This is only applicable if leader election is enabled. (default 10s)
2021-04-01T14:12:55.264999469Z stderr F       --leader-election-retry-period duration     The duration the clients should wait between attempting acquisition and renewal of a leadership. This is only applicable if leader election is enabled. (default 2s)
2021-04-01T14:12:55.265005614Z stderr F       --log-flush-frequency duration              Maximum number of seconds between log flushes (default 5s)
2021-04-01T14:12:55.265011568Z stderr F       --log_backtrace_at traceLocation            when logging hits line file:N, emit a stack trace (default :0)
2021-04-01T14:12:55.265017208Z stderr F       --log_dir string                            If non-empty, write log files in this directory
2021-04-01T14:12:55.265023906Z stderr F       --log_file string                           If non-empty, use this log file
2021-04-01T14:12:55.265030251Z stderr F       --log_file_max_size uint                    Defines the maximum size a log file can grow to. Unit is megabytes. If the value is 0, the maximum file size is unlimited. (default 1800)
2021-04-01T14:12:55.265035851Z stderr F       --logtostderr                               log to standard error instead of files (default true)
2021-04-01T14:12:55.265041147Z stderr F       --master --kubeconfig                       (Deprecated: switch to --kubeconfig) The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
2021-04-01T14:12:55.265046622Z stderr F       --namespace string                          If set, this limits the scope of cainjector to a single namespace. If set, cainjector will not update resources with certificates outside of the configured namespace.
2021-04-01T14:12:55.265051873Z stderr F       --skip_headers                              If true, avoid header prefixes in the log messages
2021-04-01T14:12:55.26505728Z stderr F       --skip_log_headers                          If true, avoid headers when opening log files
2021-04-01T14:12:55.265062801Z stderr F       --stderrthreshold severity                  logs at or above this threshold go to stderr (default 2)
2021-04-01T14:12:55.26506851Z stderr F   -v, --v Level                                   number for the log level verbosity (default 0)
2021-04-01T14:12:55.2650739Z stderr F       --vmodule moduleSpec                        comma-separated list of pattern=N settings for file-filtered logging
2021-04-01T14:12:55.265079329Z stderr F
2021-04-01T14:12:55.265084962Z stderr F error registering secret controller: no kind is registered for the type v1.APIService in scheme "pkg/runtime/scheme.go:101"

https://storage.googleapis.com/jetstack-logs/pr-logs/pull/jetstack_cert-manager/3838/pull-cert-manager-e2e-v1-16/1377623812052357120/artifacts/cert-manager-e2e-logs/kind-control-plane/pods/cert-manager_cert-manager-cainjector-78fb9879cc-sqzcz_2548f19d-98bd-466c-9ef8-1dfb7dab2c12/cert-manager/1.log


Add aarch64_sme_accessible to newlib patch

To check for SME support, compiler-rt makes a call to __aarch64_sme_accessible(). It expects this symbol to be defined in newlib, which is currently incorrect. This patch extends newlib.patch to add the definition of __aarch64_sme_accessible.

Could you add some more explanation in the commit message, please?
Who is expecting to call this function? What currently isn't working because it's missing?
Does it have a specification anywhere? Or does another C library implement it, that you're aiming to match the API of?

I've changed the commit message to add more explanation.


Request option/documentation on controlling the maximum length latex string

Describe the bug:
When a piece of latex is too long, it is not typeset.  I could not find documentation on the limit.  Ideally, I would want to know how to change it for our installation.
To Reproduce:
Go to http://beta.lmfdb.org/NumberField/47.47.6807739517051283990370106856968124192474229146184900948675688481754810058334971205734548194436392534997735758198660848138706901604387352879006731328411841.1
This page uses katex for typesetting all of the math.
If you scroll down, you will some things are typeset, and others are raw latex.  On other pages on the site, the whole page is processed correctly.  I think the difference is in the length of the latex strings.
Expected behavior:
I expected everything to be latex'ed correctly
Screenshots:
Environment (please complete the following information):

KaTeX Version: 0.10.2
Device: Desktop
OS: Ios
Browser: Brave and chrome at least

It seems Chrome optimizes by splitting a long Text node into several Text nodes:

You can call normalize() on the element to merge them before passing to KaTeX. I'm not sure it'll be worth adding it to auto-render, as it may increase processing time, but we should definitely add it to the documentation.

@ylemkimon Nice find.  Weird behavior!
Instead of normalizing, we could do some work in auto-render.js's renderElem/renderMathInText to detect consecutive text fields where the first one has an open delimiter but no closing delimiter, and treat them as if they were merged... though it definitely would make the code messier. ☹


Avoid static global by using user_data to pass state instead.

This only updates the cube example to avoid a global static. If you're generally happy with this change, I can update the other examples to be similar.

Great fix thanks :) I think it makes sense to apply this to all samples, and that way we are at least safe from the deprecation warning becoming an error at some point.
Eventually I'd like to get rid of the unsafe in the samples entirely, maybe by adding some sort of "SokolApp" wrapper/trait/superclass thingie which somehow would provide a "safe-wrapper" for this unsafe user data handling (internally it would probably do the same thing as your fix).
My Rust-foo isn't good enough to know what a proper idiomatic Rust solution would look like though.

Here you go, examples and readme updated. The basic idea of a wrapper with a trait sounds good as a future solution.

Giving this a whirl now...

...and merged. Many thanks, it's nice to see a warning-free build again :)


fix(typings): inherit Radio props from Checkbox

After #1155 the Radio component did not inherit the checkbox props. This might help with #1169

This looks correct to me.  @layershifter any comments on this direction?  I know you're heading up the TS refactor efforts.

Thanks much fellas!

Released in<EMAIL_ADDRESS>


Add zyro to the list of companies

Adding zyro.com to the list of companies that are using express.js

Hey there, we are no longer accepting additions here. We actually might remove this page entirely in the near future. Thank you all the same!


Added custom sensor

Added a custom sensor.

Rounding must be done when filling the data in Domoticz, as the rounding is dependent on the type of data being used.

Thanks:)


A problem about parameter passing in train_model.py

In train_model.py , we create a object named model using "mx.model.FeedForward ", and we pass these parameters : learning_rate , momentum , wd . But in model.py , there is no initialization of these 3 parameters in "def init "of class FeedForward. And I cannot find the parameters anywhere in model.py. So how do these parameters work in model.py?

@wanderingpj , they are used in the optimizer, please see https://github.com/dmlc/mxnet/blob/master/python/mxnet/model.py#L793

@zlmzju ，since we have passed the parameter **model_args which include the learning_rate , why do we pass learning_rate again?

@wanderingpj so this is another question, right? And please tell me the evidence or the url.

@zlmzju  https://github.com/dmlc/mxnet/blob/master/example/image-classification/train_model.py#L84

we use model_args in line 84 , but never use learning_rate in line 80.

Ok, I know what you mean.  I suggest that you can first study to understand what **kwargs means in Python.

@zlmzju Fine , I will learn it first . I have one more question . We use
"        model_args['lr_scheduler'] = mx.lr_scheduler.FactorScheduler(
step = max(int(epoch_size * args.lr_factor_epoch), 1),
factor = args.lr_factor)
"
to update lr , but we didn't pass the parameter of lr to FactorScheduler , how does it get the base_lr and change it?
https://github.com/dmlc/mxnet/blob/master/example/image-classification/train_model.py#L64

@zlmzju  After reading what you suggest , I know the answer of my second question . Thank you very much !


questions about using deepspeed config

I have two machines with 8 graphics cards each.
when I use the following config, the accelerator.num_processes printed in the main process is 16:
compute_environment: LOCAL_MACHINE
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: all
machine_rank: 0
main_process_ip: xx
main_process_port: xx
main_training_function: main
mixed_precision: fp16
num_machines: 2
num_processes: 16
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false
when I use deepspeed config, the accelerator.num_processes printed in the main process is 8,  and after calling the prepare(dataloader) function, the data is also divided into 8 parts instead of 16 parts.
the config is following:
compute_environment: LOCAL_MACHINE
deepspeed_config:
deepspeed_config_file: ds_config.json
deepspeed_multinode_launcher: pdsh
zero3_init_flag: true
distributed_type: DEEPSPEED
downcast_bf16: 'no'
machine_rank: 0
main_process_ip: xx
main_process_port: xx
main_training_function: main
num_machines: 2
num_processes: 16
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false
ds_config.json:
{     "fp16": {         "enabled": true     },     "zero_optimization": {         "stage": 1,         "reduce_bucket_size": 5e8     },     "gradient_clipping": "auto",     "train_batch_size": "auto",     "train_micro_batch_size_per_gpu": "auto",     "gradient_accumulation_steps": 10,     "steps_per_print": 10 }
I understand that the accelerator.num_processes here should also be equal to 16, right?
It seems that there is no communication between the two machines, but each uses 8 graphics cards for training.
Then I added a hostfile config now:
deepspeed_hostfile: hostfile
hostfile:
ip1 slots=8
ip2 slots=8
and encountering the following error "subprocess.CalledProcessError: Command 'ssh -o PasswordAuthentication=no ip1 hostname' returned non-zero exit status 255."

cc @pacman100

Hello, you need to enable passwordless ssh between the 2 nodes to use pdsh. Read more about it in the official DeepSpeed documentation here: https://www.deepspeed.ai/getting-started/#resource-configuration-multi-node
I had to do the below steps when using pdsh. Hope this helps.

What is using passwordless ssh is not possible? Is there a way to have accelerate pass the no_ssh_check flag to deepspeed easily?
https://github.com/microsoft/DeepSpeed/blob/master/deepspeed/launcher/runner.py#L148
This works using pure neox megatron w/ deepspeed, but I've been trying to figure out how to enable it via accelerate and haven't had success

CC @pacman100 @muellerzr


Disappearing instructions

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-same-element-with-multiple-jquery-selectors has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
Code below "You shouldn't need to modify code below this line" initially missing, then once it appears, instructions disappear.  Reloading page sometimes causes scrollbar next to instructions to disappear as well.  Requires reloading page 4-5 times for everything to return, so maybe it's just me?

No, it is not just you, but the issue I created was also closed. Irresponsible, in my opinion, to close an issue just because the person at the other end doesn't experience the same issue.

And I see that for you it is in Chrome. For me it is in IE 11. So it is not a specific browser problem.

@JoeCool1971 Please close this issue if this is resolved.

Why is this issue closed? I am still having this problem.
My code:

Will not go through.


Late MacBook and GBU521 dongle

I've tried the 2.0 and the 2.0.1 beta.
When I do the system diagnostics, every check passes except detect BT hardware. I get a warning.
Unplugging and replugging the dongle doesn't help. Any ideas?
Handoff option is there, just it doesn't work.

Same here except Mac Pro 2010. Diagnostics and install messages from current public beta
--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as inactive
Verifying Mac model reference...        ioreg: error: can't obtain properties.
OK. Known compatible Mac Model detected: MacPro5,1
Verifying Mac board-id...               ioreg: error: can't obtain properties.
OK. Short board id detected: Mac-F221BEC8
Verifying OS X version...               OK. Mac OS X 10.10.2 (14C1510) detected
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and uses the legacy Brcm4331 kext. This tool can fix this.
Verifying AWDL status...                OK. No AWDL is active, but the hardware seems to be able to support it. This tool can try to fix this.
Verifying Bluetooth hardware...         OK. 3rd party Bluetooth hardware detected
Verifying Bluetooth version...          ioreg: error: can't obtain properties.
NOT OK. No active Bluetooth hardware detected.
Verifying Bluetooth features...         ioreg: error: can't obtain properties.
NOT OK. No Bluetooth features could be detected
Verifying Bluetooth firmware...         ioreg: error: can't obtain properties.
WARNING. No Bluetooth Firmware version could be found
--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is not active. This tool can fix this.
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Bluetooth blacklist status... ioreg: error: can't obtain properties.
OK. Your Mac model is not blacklisted
Verifying Wi-Fi whitelist status...     ioreg: error: can't obtain properties.
OK. Your board-id is not yet whitelisted. This tool can fix this.
Verifying old Wi-Fi kext presence...    OK. Legacy Brcm4331 Wi-Fi driver is present. This tool can fix this.
Verifying legacy Wi-Fi card patch...    OK. The legacy Wi-Fi patch is not present. This tool can fix this.
Verifying BT4 dongles compatibility...  OK. Compatibility with BT4 USB dongles is not enabled, this tool can fix this if a dongle is plugged in
--- OS X Continuity Activation Tool 2.0.1 beta ---
by dokterdok
--- Initiating system compatibility check ---
Verifying Mac model reference...        ioreg: error: can't obtain properties.
OK
Verifying Mac board-id...               ioreg: error: can't obtain properties.
OK
Verifying OS X version...               OK
Verifying Wi-Fi hardware...             OK
Verifying Bluetooth version...          ioreg: error: can't obtain properties.
NOT OK. No active Bluetooth hardware detected. Aborting.

Has this been abandoned?
My diag. Even when installed, handoff doesn't work. Im using beta 2.0.1 and the IO gear 521 dongle.
--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as inactive
Verifying Mac model reference...        OK. Known compatible Mac Model detected: MacBook5,1
Verifying Mac board-id...               OK. Short board id detected: Mac-F42D89C8
Verifying OS X version...               OK. Mac OS X 10.10.3 (14D131) detected
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and uses the legacy Brcm4331 kext. This tool can fix this.
Verifying AWDL status...                OK. No AWDL is active, but the hardware seems to be able to support it. This tool can try to fix this.
Verifying Bluetooth hardware...         WARNING. No Bluetooth controller references were found in the PRAM, dongles can't be detected.
Verifying Bluetooth version...          OK. Bluetooth 4.0 detected
Verifying Bluetooth features...         OK. Bluetooth features are currently not compatible with Continuity. This tool can try to fix this.
Verifying Bluetooth firmware...         OK. Bluetooth firmware version: v14 c4096
--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is not active. This tool can fix this.
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Bluetooth blacklist status... OK. Your Mac model is not blacklisted
Verifying Wi-Fi whitelist status...     OK. Your board-id is not yet whitelisted. This tool can fix this.
Verifying old Wi-Fi kext presence...    OK. Legacy Brcm4331 Wi-Fi driver is present. This tool can fix this.
Verifying legacy Wi-Fi card patch...    OK. The legacy Wi-Fi patch is not present. This tool can fix this.
Verifying BT4 dongles compatibility...  OK. Compatibility with BT4 USB dongles is not enabled, this tool can fix this if a dongle is plugged in
any help

after installation
--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as active
Verifying Mac model reference...        OK. Known compatible Mac Model detected: MacBook5,1
Verifying Mac board-id...               OK. Short board id detected: Mac-F42D89C8
Verifying OS X version...               OK. Mac OS X 10.10.3 (14D131) detected
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and is using the Continuity compatible Brcm4360 kext
Verifying AWDL status...                OK. An AWDL interface is up, Wi-Fi is ready for Continuity
Verifying Bluetooth hardware...         WARNING. No Bluetooth controller references were found in the PRAM, dongles can't be detected.
Verifying Bluetooth version...          OK. Bluetooth 4.0 detected
Verifying Bluetooth features...         OK. Bluetooth features are Continuity compliant
Verifying Bluetooth firmware...         OK. Bluetooth firmware version: v14 c4096
--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is already active
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Bluetooth blacklist status... OK. Your Mac model is not blacklisted
Verifying Wi-Fi whitelist status...     OK. The whitelist is correctly patched with your board-id
Verifying old Wi-Fi kext presence...    OK. Legacy Wi-Fi driver Brcm4331 was already removed
Verifying legacy Wi-Fi card patch...    OK. The patch is already done. Old Broadcom Wi-Fi cards may work.
Verifying BT4 dongles compatibility...  OK. The patch that enables BT4 USB dongles compatibility has already been applied

this fixed it
https://github.com/dokterdok/Continuity-Activation-Tool/issues/167


Migrated _workspace to vendor and updated CI

1.Updated docker image using GO1.7
2.Migrated _workspace to vendor
3.Updated README.md

Hi dr.max @maximilien,
This is ready for your review. Thanks.

Cool, merging. Thx


add attr docstring

The generated docstring merely points to the docs elsewhere.  I started with "{} <{}>".format, but I'm new to attrs and wasn't sure if that meets the requirement for all supported Python versions.

Current coverage is 100% (diff: 100%)

Merging #74 into master will not change coverage

@@           master   #74   diff @@
===================================
  Files           8     8
  Lines         395   396     +1
  Methods         0     0
  Messages        0     0
  Branches       87    87
===================================
+ Hits          395   396     +1
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update f09cc7f...d0e59fe

Are supported versions clearly written anywhere? I see "rigorously tested on Python 2.7, 3.4+, and PyPy" in the readme -- does this imply 2.6/3.3 are not supported, or might work but not tested (with unannounced breakage, ...)?

Correct, 2.6 and 3.3 are not supported: https://github.com/hynek/attrs/blob/master/CHANGELOG.rst#1600-2016-05-23
We don't test on them so I have no idea if they will work or not (3.3 will probably work), and if they don't work we basically don't care.

Heh I can stress as much as I want that small contributions are welcome and people will still apologize for them. :)  Thank you!


The exception when the jsx and mss files have the same name

Hello again :)
When I compose something from the "both" folders, I get this:
This was wrong. The exception occurs when the jsx and mss files have the same name. For example: "app.jsx" and "app.mss" or "app.m.css".
=> Errors prevented startup:

   While processing files with nathantreid:css-modules (for target web.browser):
   packages/mss/css-modules-build-plugin.js:226:10: Cannot read property 'length' of undefined
   at CssModulesBuildPlugin.compileResultSize (packages/mss/css-modules-build-plugin.js:226:10)
   at LRUCache.length (packages/mss/css-modules-build-plugin.js:41:28)
   at LRUCache.set (.meteor/packages/nathantreid_css-modules/.<IP_ADDRESS>j6dxyq++os.osx.x86_64+web.browser+web.cordova/plugin.mss.os.osx.x86_64/npm/node_modules/meteor/mss/node_modules/lru-cache/lib/lru-cache.js:166:18)
   at packages/mss/css-modules-build-plugin.js:175:19
   at .meteor/packages/static-html/.<IP_ADDRESS>caorm3++os+web.browser+web.cordova/plugin.compileStaticHtmlBatch.os/npm/node_modules/meteor/promise/node_modules/meteor-promise/fiber_pool.js:33:40

meteor 1.3.1
<EMAIL_ADDRESS><EMAIL_ADDRESS>mac os 10.11.2

Can you post a repo for this one? I can't seem to reproduce it.

Sure!) It's hard to reproduce because the problem occurs only when you compose something.
https://github.com/proxiper/meteor-css-modules-same-name-bug
If you rename imports/ui/app/app.mss to something else and change the importing in the app.jsx, then app will work.
Also in this app you can see this issue too https://github.com/nathantreid/meteor-css-modules/issues/18 -- text has classes for font-size and color, but color styles don't exists and text remains black.
Also please notice that styles generated by a scripts despite the fact that the SSR is disabled.

This should be resolved in 1.1.1 - are you still experiencing issues?


Almost whole response is truncated

I cannot see the response, for some reason pluto prints it on one line (even if its the long response) and even message is truncated. I'd like to see whole response like Gander can do it

this is fixed with commit 4402aee5f41ed13c8815fa29fc5fda1d4b83ca24


feat(protocol): apply a min value for gas excess

This guarantees a min base fee value.
Based on the current config, the min base fee is 0.099627953 wei or 0.99gwei.
At the current moment, the Ethereum base fee is 40gwei:

Based on the current config, the min base fee is 0.099627953 wei or 0.99gwei.

One question : i think the 99627953 is 0.099 gwei, not 0.99 gwei.
Is it ok and intended ? Beside that looks good to me.


Relax extension_update_paths argument to text

Issue #, if available: #126
Description of changes:
All of the other extension functions use text as name, so this appears to be an arbitrary restriction. Hence, relaxing the argument from name to text.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

To update the extension version you'll need to bump EXTVERSION in the Makefile as well. Not sure if that's causing these regression test failures though

Thanks @adamguo0, fixed it! 😄

Sorry I haven't gotten to this in a while -- the change looks good to me. I did a manual sanity check to make sure that after updating pg_tle from 1.4.0 to 1.4.1, the function works as expected

Hi, do you mind elaborating what are the restrictions that you're facing before this change?

Hi @anth0nyleung 😃,
The name data type can hold 63 characters string only (ref #define NAMEDATALEN 64). Whereas text has no limit at all!
Note that install_extension already uses text instead of name data type for the extension-name's argument. So a user can create an extension with names that are >63 characters, but the operation will fail when trying to find its update paths using extension_update_paths() function because it's using a data type that limits the extension name to 63 chars!
This PR addresses a long time pending issue, #126 raised by @jkatz. 😄

As a side-note there's another bug preventing users from creating extensions with almost-63 chars. We'll need to address that separately
https://github.com/aws/pg_tle/issues/100


support correct hex color

it's that basic
resolves #125

thanks for the contribution @tunnckoCore. i agree that this would be useful. would you mind also including a test in https://github.com/probot/settings/blob/master/test/lib/plugins/labels.test.js to verify the new behavior (adjusting the one that is there so that it handles both cases would probably be enough too)? with that, i think merging this makes sense.

It be good if so.eone can do that, cuz I'm on phone these days. Otherwise, I may add test when get to pc in the weekend or the next week.

I don't know what's with that bot, or is it slow, or it needs time... but I don't see problems in this config? It probably would be better and faster to type all the 21 labels manually once and for all :D

I don't know what's with that bot, or is it slow, or it needs time... but I don't see problems in this config?

I don't understand what this has to do with your PR

That's why I want some config validation / reporting on problems (#126), we currently only can assume, right?

i understand the desire for improved feedback when an error occurs. thankfully, someone has offered to contribute a PR, which you have found and commented on. again, i don't understand how this comment is useful to this PR.

regarding the tests, they are not that clear to me what, where and how

i'm happy to help point you in the right direction so we can get this over the finish line. i think it would be enough to update one of the colors from the simulated config file to contain a # and then ensure that the corresponding stubbed api does not contain the #.

Also, it's so natural and intuitive and basic that I don't see the point in adding a test for that. Yea guarantees, but how we can break this anyway...

please keep in mind that this project is now maintained by volunteer labor. personally, i am still learning this codebase, so my confidence is not high, even for small changes. having a test that exercises behavior changes helps give me better confidence that the change won't introduce another case where updates fail in ways that aren't reported to users of the app. there is also regression value so that other changes that happen in the future are safer to introduce without risk of breaking this behavior.

could you also include a brief note in the README that mentions the option of including the # so that others can decide if including it would be valuable to them?

Oh, thanks for pointing about tests. Got it.

could you also include a brief note in the README

For sure.

Done.

Cool. Thanks. :)

thanks!


Can not render Unicode Greek lowercase in $$...$$ or $...$.

Can not render Unicode Greek lowercase in $$...$$ or $...$.

Besides, the \tag{} is can not be rendered in $$...$$, either. Please support it.

Can not render Unicode Greek lowercase in $$...$$ or $...$.

@integzz  What letters are you having trouble rendering? Please provide an example.
All Greek lowercase letters can be rendered using tex commands or unicode

Besides, the \tag{} is can not be rendered in $$...$$, either. Please support it.

@integzz It works. Please provide an example of a formula that is not rendered correctly.

@OlgaRedozubova Thanks for your reply. It's just like this. However, I just found that it's something wrong with my VSCode. So you can close this issue. Sorry to bother you.
This is a brilliant extension.


fix: insert-at-front (issue #68)

fix: insert-at-front (issue #68)
Fixes #68
misc.

Current dependencies on/for this PR:

main

PR #73   👈

This stack of pull requests is managed by Graphite.


Add LICENCE

Example: GNU General Public License v3.0

BSD-3 clause. It was in another branch. I merged everything.


Cannot install openai package due to SSL error

Describe the bug
Description
I'm trying to install the openai package using pip, but I'm encountering an SSL error.
To Reproduce
Steps to reproduce

Run pip install openai

Expected behavior
The openai package should be installed without any issues.
Actual behavior
The following SSL error occurs:
以下是一个用于在 GitHub 上报告此问题的模板。请确保在提交 issue 之前填写所有必要的详细信息，以便维护者更好地理解问题。
markdown
Copy code
Title: Cannot install openai package due to SSL error
Description
I'm trying to install the openai package using pip, but I'm encountering an SSL error.
Steps to reproduce

Run pip install openai

Expected behavior
The openai package should be installed without any issues.
Actual behavior
The following SSL error occurs:
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))': /simple/openai/
WARNING: Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))': /simple/openai/
WARNING: Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))': /simple/openai/
WARNING: Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))': /simple/openai/
WARNING: Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))': /simple/openai/
Could not fetch URL https://pypi.org/simple/openai/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/openai/ (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))) - skipping
ERROR: Could not find a version that satisfies the requirement openai (from versions: none)
ERROR: No matching distribution found for openai
Could not fetch URL https://pypi.org/simple/pip/: There was a problem confirming the ssl certificate: HTTPSConnectionPool(host='pypi.org', port=443): Max retries exceeded with url: /simple/pip/ (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:997)'))) - skipping
Code snippets
No response
OS
windows
Python version
python3.10
Library version
openai

Have you tried installing any other version of openai ?

no

have u checked, is it working?

you're trying to install the OpenAI package using pip and encountering an SSL error, you can try the following steps to resolve the issue:
Upgrade pip: Ensure that you have the latest version of pip installed by running the command

pip install --upgrade pip

. This will update your pip installation to the most recent version, which may include fixes for SSL-related issues.
Use the --trusted-host flag: When running the pip install command, add the --trusted-host flag followed by the package repository URL. For example, if you're installing from PyPI, you can try running pip install --trusted-host pypi.python.org openai. This flag tells pip to trust the provided host explicitly.
Disable SSL verification (not recommended): As a last resort, you can disable SSL verification, but this is not recommended for security reasons. You can use the --trusted-host flag along with the --insecure flag. For example, run pip install --trusted-host pypi.python.org -- you're trying to install the OpenAI package using pip and encountering an SSL error, you can try the following steps to resolve the issue:
Upgrade pip: Ensure that you have the latest version of pip installed by running the command pip install --upgrade pip. This will update your pip installation to the most recent version, which may include fixes for SSL-related issues.
Use the --trusted-host flag: When running the pip install command, add the --trusted-host flag followed by the package repository URL. For example, if you're installing from PyPI, you can try running pip install --trusted-host pypi.python.org openai. This flag tells pip to trust the provided host explicitly.
Disable SSL verification (not recommended): As a last resort, you can disable SSL verification, but this is not recommended for security reasons. You can use the --trusted-host flag along with the --insecure flag. For example, run pip install --trusted-host pypi.python.org --insecure openai. Keep in mind that this option bypasses SSL certificate verification and may expose you to potential security risks.
Check your system's SSL configuration: Ensure that your system has the necessary SSL certificates and they are up to date. Refer to your operating system's documentation for instructions on updating SSL certificates.
Verify your network connection: Make sure you have a stable internet connection without any network interruptions. If you're behind a proxy or firewall, ensure that they are properly configured to allow the necessary network connections.. Keep in mind that this option bypasses SSL certificate verification and may expose you to potential security risks.
Check your system's SSL configuration: Ensure that your system has the necessary SSL certificates and they are up to date. Refer to your operating system's documentation for instructions on updating SSL certificates.
Verify your network connection: Make sure you have a stable internet connection without any network interruptions. If you're behind a proxy or firewall, ensure that they are properly configured to allow the necessary network connections.


python37Packages.identify: 1.5.10 -> 1.5.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.identify is: "File identification library for Python"
meta.homepage for python37Packages.identify is: "https://github.com/chriskuehl/identify"
meta.changelog for python37Packages.identify is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.5.11 with grep in /nix/store/byrkzv88xxl9k2gyia2dm569kh5z1c8a-python3.7-identify-1.5.11

directory tree listing: https://gist.github.com/380fb04df0fc960e24af708dec697aa8

du listing: https://gist.github.com/c5c2acb7944a69e8d34e3c3736b0f8ce

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

32 total rebuild path(s)

8 package rebuild(s)

8 x86_64-linux rebuild(s)
8 i686-linux rebuild(s)
8 x86_64-darwin rebuild(s)
8 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gitAndTools.pre-commit
pre-commit
python37Packages.identify
python37Packages.pre-commit
python38Packages.identify
python38Packages.pre-commit
python39Packages.identify
python39Packages.pre-commit

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/byrkzv88xxl9k2gyia2dm569kh5z1c8a-python3.7-identify-1.5.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/byrkzv88xxl9k2gyia2dm569kh5z1c8a-python3.7-identify-1.5.11
ls -la /nix/store/byrkzv88xxl9k2gyia2dm569kh5z1c8a-python3.7-identify-1.5.11/bin

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

6 packages built:

gitAndTools.pre-commit (python38Packages.pre-commit)
python37Packages.identify
python37Packages.pre-commit
python38Packages.identify
python39Packages.identify
python39Packages.pre-commit

Maintainer pings

Result of nixpkgs-review pr 108358 1

6 packages built:

pre-commit (gitAndTools.pre-commit ,python38Packages.pre-commit)
python37Packages.identify
python37Packages.pre-commit
python38Packages.identify
python39Packages.identify
python39Packages.pre-commit

Result of nixpkgs-review pr 108358 1

6 packages built:

pre-commit (gitAndTools.pre-commit ,python38Packages.pre-commit)
python37Packages.identify
python37Packages.pre-commit
python38Packages.identify
python39Packages.identify
python39Packages.pre-commit

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/324

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/324


fn layer registers lctrl & lalt separately but not together

Hi, I'm not sure if this is a bug or user error but thought I'd raise it to see if anyone else has come across the same thing. Basically, I have an fn layer mapped, and yesterday I ran into an issue where I messed up some configuration on my computer and nuked my graphical tty, and I couldn't swap into a different tty to fix it with the inbuilt keyboard.
Issue:
My other layers that have both a ctrl and alt key mapped seem to have no problem registering both being held at the same time to modify a key on that layer (E.g. lctr+lalt+1 on my numpad layer). However, on the fn layer, they will register and modify a key individually (E.g. lalt+F1, or lctl+F1), but they will not register at the same time (E.g. lalt+lctl+F1).
Is this known behaviour that I am missing, a bug or am I doing something wrong?
gist of my config

This sounds like a roll over issue between the keys q, j and k.
Take a look at this FAQ Entry.
t.l.d.r.:
Try pressing q, j and k on some rollover testing website while kmonad is deactivated.
If it doesn't register all three key presses, it's probably the fault of your keyboard.

Ah seems you are bang on with the roll-over issue. It's not something I've come across before but for any interested readers of this thread in the future, I'm on an xps15 laptop.  You are right that I am able to trigger the correct combo by starting with the lctl + lalt keys from my base later, then pressing the fn layer toggle, and then the desired Fn key. I'm sure I tried that yesterday but likely in a less intentional way and so didn't get the ordering correct. Will consider whether it's worth putting in a 'rescue' layer for my sins, although - as you say - it's pretty rare that I actually need another tty and can just plug in another keyboard temporarily. Thanks for the help!


warnings.go:70 metadata.finalizers: "trident.netapp.io": prefer a domain-qualified finalizer name to avoid accidental conflicts with other finalizer writers

Describe the bug
A clear and concise description of what the bug is.
https://github.com/NetApp/trident/blob/4513299c34b045d1d4685a60ecc8429a2b49de73/cli/k8s_client/k8s_client.go#L62 provides an error message that is also discussed in https://github.com/kubernetes/kubernetes/issues/127287
Environment
Provide accurate information about the environment to help us reproduce the issue.

Trident version: [e.g. 19.10]
Trident installation flags used: [e.g. -d -n trident --use-custom-yaml]
Container runtime: [e.g. Docker 19.03.1-CE]
Kubernetes version: [e.g. 1.15.1]
Kubernetes orchestrator: [e.g. OpenShift v3.11, Rancher v2.3.3]
Kubernetes enabled feature gates: [e.g. CSINodeInfo]
OS: [e.g. RHEL 7.6, Ubuntu 16.04]
NetApp backend types: [e.g. CVS for AWS, ONTAP AFF 9.5, HCI 1.7]
Other:

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

@Dexus can you please provide more details on what the bug is ? Are you suggesting a different value for the finalizer ?

One customer mentioned that every time he is deploying Trident, a warning is issued. And that is annoying and unnecessarily unsettling.


Compiler error with underapplied projections in TC instances

instance : Coe { p : Nat → Prop // p 42 } (Nat → Prop) where
  coe := Subtype.val

fails with:
error: failed to compile definition, consider marking it as 'noncomputable' because it depends on 'Subtype.val', and it does not have executable code
https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/noncomputable/near/326249302

As a workaround, you can write coe x := x.val for the time being.


Can't get .env.development && .env.production variable

Describe the bug
(Write your answer here.)
Did you try recovering your dependencies?

(Write your answer here.)
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

(paste the output of the command here.)
Steps to reproduce

(Write your steps here:)

add .env.developent in root path
add content into the .env.developent

REACT_APP_CHAIN_ID = "42"

# 10+ nodes balanced, US/EU
REACT_APP_NODE_1 = "https://kovan.infura.io"

# 10+ nodes balanced, US/EU
REACT_APP_NODE_2 = "https://kovan.infura.io"

# 10+ nodes balanced in each region, global
REACT_APP_NODE_3 = "https://kovan.infura.io"

in the App.js to process.env.REACT_APP_NODE_1 can't get the data, is undefined

Expected behavior

(Write what you thought would happen.)
Actual behavior

(Write what happened. Please add screenshots!)
Reproducible demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

From the documentation:

The environment variables are embedded during the build time. Since Create React App produces a static HTML/CSS/JS bundle, it can’t possibly read them at runtime.

Is process.env.REACT_APP_NODE_1 undefined at runtime or after you've created a build?


release 3.0.41 test regression openapi

Hi,
during build of the 3.0.41 release tag I get test failures related to openapi:
panic: read json from products.json failed &{[]} [recovered] 	panic: read json from products.json failed &{[]}
Previously I was able to successfully run go test
ok  	github.com/aliyun/aliyun-cli/cli	0.022s
ok  	github.com/aliyun/aliyun-cli/command	0.011s
ok  	github.com/aliyun/aliyun-cli/config	63.576s
ok  	github.com/aliyun/aliyun-cli/i18n	0.010s
?   	github.com/aliyun/aliyun-cli/main	[no test files]
--- FAIL: Test_main (0.00s)
panic: read json from products.json failed &{[]} [recovered]
	panic: read json from products.json failed &{[]}

goroutine 6 [running]:
testing.tRunner.func1.1(0x55bab7065f20, 0xc0003151f0)
	testing/testing.go:940 +0x2f7
testing.tRunner.func1(0xc00019b680)
	testing/testing.go:943 +0x3fb
panic(0x55bab7065f20, 0xc0003151f0)
	runtime/panic.go:969 +0x16a
github.com/aliyun/aliyun-cli/meta.LoadRepository(0x55bab7126fc0, 0x55bab777c220, 0x55bab66d109a)
	github.com/aliyun/aliyun-cli/meta/repository.go:35 +0x613
github.com/aliyun/aliyun-cli/openapi.NewLibrary(...)
	github.com/aliyun/aliyun-cli/openapi/library.go:38
github.com/aliyun/aliyun-cli/openapi.NewCommando(0x55bab7126a60, 0xc000317e30, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	github.com/aliyun/aliyun-cli/openapi/commando.go:45 +0xa4
github.com/aliyun/aliyun-cli/openapi.Test_main(0xc00019b680)
	github.com/aliyun/aliyun-cli/openapi/commando_test.go:37 +0x1cb
testing.tRunner(0xc00019b680, 0x55bab70ef418)
	testing/testing.go:991 +0xde
created by testing.(*T).Run
	testing/testing.go:1042 +0x359
FAIL	github.com/aliyun/aliyun-cli/openapi	0.024s
?   	github.com/aliyun/aliyun-cli/resource	[no test files]
FAIL

This may be because we changed the version of go-bindata, we changed from https: // github.com / shuLhan / go-bindata to https: // github.com / jteeuwen / go-bindata. And the packaging command has been changed to go-bindata -o resource / metas.go -pkg resource ../ aliyun-openapi-meta / ...

oh, thats totally awesome :heart_eyes_cat:
PS: i'm sure you mean you changed from jteeuwen to shuLhan :wink:
I guess you can close #262 as well with this nice change. Package now built successfully


Stop Ink Friendly snippet making text unselectable in output PDF

This PR resolves #3563.
This PR changes the Ink Friendly snippet to use box-shadow instead of filter. This should achieve the same visual effect while not rendering the output PDF as an image.

Is there a Brew we can compare before and after results with?

Is there a Brew we can compare before and after results with?

Example 1: Current
HB link: https://homebrewery.naturalcrit.com/share/wjdLwCIMiGf7
PDF: Ink Friendly Example 1_ Current - The Homebrewery.pdf
Example 2: PR Changes
HB link: https://homebrewery.naturalcrit.com/share/oU5XmiPzlhMg
PDF: Ink Friendly Example 2_ Proposed - The Homebrewery.pdf
You should see that the "current" brew contains only a single framed monster block and the Ink Friendly snippet. However, the PDF output is that of an image; the text is not able to be selected or interacted with.
However, you should also see that the PR changes contains the exact same framed monster block, and the only difference is in the style, where the Ink Friendly snippet has been replaced with the version from this PR. The resultant PDF has text that can be highlighted and interacted with as expected for text.


Now with Grs Failed instead of Not supported

Description of Work carried through
Brief description of what/why/how.
Check list

[ ] ./precheck was executed (Integration/Component/Unit tests)
[ ] sbt scalafmt test:scalafmt was executed
[ ] Required Environment Config has been amended/added
[ ] Links to dependencies have been included (BE/FE work)
[ ] User Acceptance Tests (UAT) were run locally and have passed.
[ ] No Accessibility errors/warnings reported by Wave

Either this Or https://github.com/hmrc/plastic-packaging-tax-registration-frontend/pull/493


Why does the user need to provide keys?

Why don't I just provide keys myself if I know what widgets are going to be what in the AnimatedTransition? That should be easier than relying on users or doing that asserts everywhere
This shouldn't be a breaking change

Wrapping
KeyedSubtree(child: child)

around each and having a unique id for each will solve this


YCM extension not loaded but is installed and working

environment

vim: 0.4.3
vim-airline: 0ca9576331ba845e7bf29f5e092f5da04f4f3580
OS: Ubuntu 20.04.1 LTS
Have you reproduced with a minimal vimrc: yes
What is your airline configuration:

call plug#begin()
Plug 'https://github.com/ycm-core/YouCompleteMe'
Plug 'https://github.com/vim-airline/vim-airline'
call plug#end()

if you are using terminal:

terminal: ubuntu terminal
$TERM variable: screen-256color
color configuration (:set t_Co?): vim says keycode not set
if you are using Neovim:
does it happen in Vim: not sure

actual behavior
YCM extension says not loaded and I don't get the count of ycm errors in the extension area of the airline.
:YcmDiags works and shows the location list filled of errors.
:AirlineExtensions shows a bunch of extensions loaded automatically (keymap, quickfix, etc.) but shows ycm as not loaded.
expected behavior
YCM exntension is loaded and I do get the count of ycm errors in the extension area of the airline

vim: 0.4.3

Okay, so please mention that this is neovim and not vim.
It looks like you need to explicitly set :let airline#extensions#ycm#enabled = 1 to have the integration loaded. Does that work?

any update?

vim: 0.4.3

Okay, so please mention that this is neovim and not vim.
It looks like you need to explicitly set :let airline#extensions#ycm#enabled = 1 to have the integration loaded. Does that work?

Yes this worked, thank you. Was away from
work computer, and I recently swapped from ycm to coc. Sorry for the late reply.


Avoid NSInternalStateInconsistencyException by setting rows inside performBatchUpdates

This solves the issue I was having related to #21. This change replaces the didSet observer on rows with a setter. Modifying rows now works like this:

rows is assigned to, which calls its set setter
The setter computes a diff from the previous value of rows.
If the diff indicates any changes, the setter calls performBatchUpdates. Inside the block, it modifies the backing array for rows and triggers any necessary animations.

Previously, CollectionViewDiffCalculator changed its backing values by modifying rows first, then invoking a didSet observer. Like I documented for table views in #23, the modification to rows needs to happen after the collection view has been informed a block of changes is incoming.
I also updated the test for CollectionViewDiffCalculator to invoke UIKit so that it catches these kinds of errors in testing.

Thanks, @elliottwilliams - I added a couple small changes of my own to make TableViewDiffCalculator behave the same way.

Great, thanks for that!


MMS2021, MMS2022 -> windows-2022

Updating used pool to compliant image, v. 2022:

https://github.com/Azure/azure-sdk-tools/issues/3407

Note that searching for MMS2021 returned no results.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Moving to .NET Standard 2.0

Is there any plan to move this library to .NET Standard 2.0?

Hi @vicancy,
Unfortunately Nustache is primarily only getting maintenance updates as I've been focusing on my .NET Standard compliant mustache renderer called Stubble.
It's a .NET Standard 1.3 project with support for net 45 also. Stubble is currently in Alpha as i'm working on getting the extensibility interfaces right for plugins and other renderers to hook in.
Hopefully docfx would be interested in migrating over, I think you'll find the interfaces very similar and i'd be happy to help.
Thanks,

Thanks @Romanx for your quick response. Will you be willing to merge the PR for migrating Nustache to .NET standard? One of our contributor already ported one, https://github.com/dotnet/docfx/issues/1963#issuecomment-338318944 It would be best that Nustache embraces the change and publish it to its official nuget feed.

@vicancy I'd certainly accept a PR for Nustache to move to .NET standard. Please feel free to open a request and i'll take a look.

@Romanx Is Stubble in alpha because it has certain features not ready or may contain some potential issues? In general docfx would prefer a stabilized version....

@vicancy It's currently in alpha since i'm stabilizing the external interface for plugins to add features to the parsing and rendering pipeline. It passes all the mustache spec tests currently and (as far as i'm aware) has no bugs.
It should have a stable 1.0 version in the next few weeks but I understand docfx wouldn't want to use something unstable i'd be happy to accept a PR into Nustache with the .NET standard changes and we can talk about Stubble when it stabilizes 👍

@Romanx Sounds great! Thanks for your understanding

Would the intention be to provide compatibility for .NET frameworks in the nuget package?
I actually came here in the hope of finding something that could be used on .NET4 or .NET4.5.
"Moving" this library into .NET Standard would rule me out of using it.

Hi there, the plan here was to additionally target netstandard to allow the library to be used cross platform. I'm not sure that's entirely viable now since i've finished my other project Stubble which was designed to be cross platform from the start. It also targets net45 as it's lowest target.
Would you mind me asking why you're still targeting net4/net45 when microsoft no longer support it?

Thanks, I didn't even realize net45 was out of support. Fair enough in that case!
FYI it's just one use-case I have in a WinForms-based project where catering for the lowest common-denominator has some appeal to some small number of stragglers (supported or not)

@PandaWood I can understand that entirely. The in place upgrades to net451 are usually very stable if thats an option for you which is covered by netstandard.


decode-uri-component <= 0.2.0 DoS

https://github.com/advisories/GHSA-w573-4hg7-7wgq

https://www.npmjs.com/package/snapdragon depends on this

This package is deprecated.
snapdragon is unmaintained.
Here’s how to get rid of it (most likely): https://github.com/lydell/source-map-resolve#deprecated


ipywidgets test added [WIP]

Describe your changes
Issue ticket number and link
Closes #x
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added thorough tests (when necessary).
[ ] I have added the right documentation in the docstring and changelog (when needed)

:books: Documentation preview :books:: https://jupysql--141.org.readthedocs.build/en/141/

taking a different approach since it's not working on jupyter-book

taking a different approach since it's not working on jupyter-book

@yafimvo: it's not mandatory that this works on jupyter-book (it's good if it's possible, but if it's not, it's probably a problem on jupyter-books end), please ask the jupyter book team so we know if it's a limitation on their end, since AFAIK the comms feature in Jupyter is the official way to communicate backend and frontend (and that seems to be what you tried this time)


v1.8.1 does not build on skypack

Describe the bug
skypack fails to build v1.8.1: https://cdn.skypack.dev/solid-js/web
/*
 * [Package Error]<EMAIL_ADDRESS>could not be built.
 *
 *   [1/5] Verifying package is valid…
 *   [2/5] Installing dependencies from npm…
 *   No matching version found for<EMAIL_ADDRESS> *
 * How to fix:
 *   - If you believe this to be an error in Skypack, file an issue here: https://github.com/skypackjs/skypack-cdn/issues
 *   - If you believe this to be an issue in the package, share this URL with the package authors to help them debug & fix.
 *   - Use https://skypack.dev/ to find a web-friendly alternative to find another package.
 */

console.warn("[Package Error]<EMAIL_ADDRESS>could not be built. \n[1/5] Verifying package is valid…\n[2/5] Installing dependencies from npm…\nNo matching version found for solid-js@1.8.1.");
throw new Error("[Package Error]<EMAIL_ADDRESS>could not be built. ");
export default null;

Your Example Website or App
https://cdn.skypack.dev/solid-js/web
Steps to Reproduce the Bug or Issue

Just navigate to https://cdn.skypack.dev/solid-js/web

Expected behavior
v1.8 should build
Screenshots or Videos
No response
Platform
Not sure what skypack uses.
Additional context
it also previously failed to build in v1.7, but was fixed: https://github.com/solidjs/solid/issues/1660
jsdelivr works for the main library but I cannot find a way to get it to work for /web or /html.
As a result, I do not know a way to run solidjs from a CDN. All solidjs pens on codepen seem to be broken (except one that imports an older version).

Yeah this one is mystifying as the code looks like valid JS. In 1.7 it was a very non-sensical fix that just by chance happened to fix it. Although it is pretty clear the issue is with /web so there is something here.

I asked Fred and he told me Skypack is all but dead. So no help there. The recommended CDN now is<EMAIL_ADDRESS>It supports submodules via URLs like<EMAIL_ADDRESS>I will leave this around for a while if people want to try to speculate what is causing this but it probably isn't the best use of time.

@vezwork  -  https://status.skypack.dev/

Thank you @ryansolid! I was successfully able to get the starter counter app working with esm.sh using its submodule support.
I was attempting to use skypack because it is what the getting started buildless options part of the solid js docs uses. I'm going to go put up a PR to change skypack to esm.sh there and I'll link it here once I have.

Here is a PR to replace skypack usage with esm.sh usage in the getting started -- buildless options part of the docs: https://github.com/solidjs/solid-docs/pull/275

With skypack down, with no prospect of getting up again, I guess this can be closed.

Docs update has been merged. I'm going to close this. Thank you everyone involved.


litedb userProjects support

https://github.com/mbdavid/litedb

not required since project management is not under vrc-get


JavaScript error on launch

Error on almost every launch

Error while running, after uploading two packs successfully

Multiple App instances are now prevented (as of 0.3.3) I assume this would prevent this issue.


Networking/App Delivery - Reliability and Operational Excellence pages are identical?

When comparing these two pages, they're basically identical:

https://docs.microsoft.com/en-us/azure/architecture/framework/services/networking/application-delivery/operational-excellence
https://docs.microsoft.com/en-us/azure/architecture/framework/services/networking/application-delivery/reliability

I don't understand - what's the rationale here?

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

ID: a22a84c6-204e-93e8-d8f3-9909701b7b37
Version Independent ID: a22a84c6-204e-93e8-d8f3-9909701b7b37
Content: Application Delivery (General) and operational excellence - Microsoft Azure Well-Architected Framework
Content Source: well-architected/services/networking/application-delivery/operational-excellence.md
Service: architecture-center
Sub-service: well-architected
GitHub Login: @v-stacywray
Microsoft Alias: robbymillsap

Hi @MarcosMMartinez,
Thanks for pointing this out. WAF was refreshed recently, and this issue has been addressed.
Be sure check out the latest guidance.


➕ [RESOURCE] - Adding the project for Useful Extensions for Web Developers

Idea Contribution

[X] I have read all the resource request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Where do you want to add the resource? Specify the category and section.
I want to add important useful Chrome Extesions for webdeveloper like:
1.WhatFonts : For finding the style and all information related to fonts for styling by just hovering on the text
2.Pesticides : For finding out the layout for websites with easiness by hovering on design provided.
Provide the URLs of the resources you are adding
No response
Any alternative approaches/features
https://github.com/chengyin/WhatFont-Bookmarklet
https://github.com/paulmolluzzo/pesticide_for_chrome
Additional Context
No response
Code of Conduct

[X] I follow the CONTRIBUTING GUIDELINE of this project.

@jfmartinz please assign this task to me to add useful extension for web developer


Support for midi QOL rolling system

DnD5e 1.2.2, Midi QOL  0.3.50 and DnD5e Helpers 1.8.6 (no other modules enabled)
With roll automation support enabled in midi, the open wounds table roll triggers and works fine for death saves and 0 HP, but not for critical hits.
Disabling roll automation support the table roll on crits works. I would be nice to have support for modules like midi QOL that change the rolling system.

Thank you for the suggestion. I will have to leave the decision to support Midi-QOL to @kandashi as I do not use any rolling modules, personally.

Thank you for the suggestion. I will have to leave the decision to support Midi-QOL to @kandashi as I do not use any rolling modules, personally.

I will look at it but make no promises. Its not a high priority issue in the end

I will look at it but make no promises. Its not a high priority issue in the end


Add tests to main pipeline

Description of change
Add test suite into merge to main pipeline
Question:
Do we want to build and push the ECR image immediately (faster) or only AFTER tests pass (will add a few minutes to pipeline)
Screenshots of changes (if applicable)
Before changes:
Current main branch flow

After changes:
❌ Proposed main branch flow see alternative flow below
NOTE: ignore the hold-dev step as it is only there to prevent me deploying to dev on this branch

🏆 Alternative main branch flow -only build AFTER test - winning flow

LGTM I see no required tests before building on open branch but that is fine as speeds up development.

cheers, yes current flow does not require it either on a branch - auto deploys branch name version in dev namespace


How to output strack trace to file async signal safely?

I have seen example online such as:
google::InstallFailureWriter([](cosnt char* data, int size){
    std::string str = std::string(data,size);
    LOG(ERROR) << str;
});

which I think it's not async signal safe. This function seems to be called inside signal handling function. Is it possible to implement it in a asynal signal safe way?

What would be a signal safe approach?

I'm closing the issue due to lack of response. Please let us know once you have more information.

I think bothstd::string str = std::string(data,size); and LOG(ERROR) << str; are not signal safe because they will call functions like malloc/mmap, according to the signal-safety linux manpage.
Indeed my question is if my program breaks, and I would like to see the strace/stack/coredump content in my log file, how should I write this function properly? What would be a signal safe approach?


Fix desired size of eks node group

Motivation
Encountering error "invalid desiredSize" while applying with below input variables:
node_pool_min_size = 1
node_pool_max_size = 12
node_pool_desired_size = 1
node_pool_instance_types = ["c6i.2xlarge"]

That's caused by the computed desired size via node_pool_instance_types could be 0., which  violates the rule "Desired number of nodes should not be lower than the minimum number of nodes that the group can scale out to."
Actually the desired_size has already been assigned in its above code block node_group_defaults: https://github.com/streamnative/terraform-aws-cloud/blob/e0e7315f9a49a04b18fe7264aa8f0cb6c7a92ea9/main.tf#L82
We should just respect it.
Modifications

Remove the duplicated desired_size assignment line

Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Documentation

[x] no-need-doc

Can node_pool_min_size be 0? New terraform code create one pool per instance type and subnet. Will it cause waste?

Can node_pool_min_size be 0?

Yes. The scaling config rules for a node group:

minSize $\ge$ 0
maxSize $\ge$ 1
desiredSize $\ge$ 0
minSize $\le$ desiredSize $\le$ maxSize

Refer https://docs.aws.amazon.com/eks/latest/APIReference/API_NodegroupScalingConfig.html


Fix internal macvlan network to work in swarm

Using dummy interface allows communication beween containers only if
they are running on the same node in swam.
Signed-off-by: Pavel Matěja<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "2418-fix" git@github.com:lemrouch/libnetwork.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842357810328
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

can you sign the commit with above instruction?

can you sign the commit with above instruction?

Hi, my impression was _dco-signed — All commits are signed _ and All checks have passed means it's already done.
BTW: those instructions dont work.

you need to amend commit with -s option.   I think that is what expected from the above message.  if you have already done it, not sure why we received  'sing your commit' messages.

you need to amend commit with -s option. I think that is what expected from the above message. if you have already done it, not sure why we received 'sing your commit' messages.

There was unsigned merge fixed by:
"lemrouch force-pushed the lemrouch:2418-fix branch from c7203f1 to da39847 5 days ago"
It should be all OK now.

What can I do to get this commit merged?

@lemrouch if the intent of this PR is to separate the notion of empty parent to create a dummy interface vs making sure internal networks can communicate in swarm, there might be more plumbing involved
https://github.com/docker/libnetwork/blob/da39847dba735007c79e104d53d4133616d5b00b/drivers/macvlan/macvlan_network.go#L65
and
https://github.com/docker/libnetwork/blob/da39847dba735007c79e104d53d4133616d5b00b/drivers/macvlan/macvlan_network.go#L93

@arkodg intent is described in https://github.com/docker/libnetwork/issues/2418 therefore I didn't really care about empty parent in other code paths as I have non-empty one filled by config-only network and all my MACVLAN networks are internal.
What do you suggest in other cases?

I see so both the above code flows are redundant for your case ( config-from networks ), have you tested these changes out ? Can you please share a working example

Yes, I have tested them in combination with https://github.com/docker/libnetwork/pull/2407
but I don't have access to the system right now. I hope I can provide example in a week or so.

Thanks, please share a snippet of the working commands when possible

Example is in https://github.com/docker/libnetwork/issues/2418
It assumes fix for default gateway as well.

About lines 65 and 93 mentioned above:
I think someone can try the same thing with internal MACVLAN IPv6 only network.
As he would have same parent interfaces on all nodes and no need for IPv4 network split he would not need config-only networks.
But those lines are for nonexistent and empty parent so this fix should be enough. Am I correct?

Thanks!
Can you check https://github.com/docker/libnetwork/pull/2407 for me as well as those are both required this to work?

also do we need a similar change for ipvlan ?

Frankly I don't know. I have not read nor tried that code. I can check it on Monday.

@lemrouch can you please make similar changes in https://github.com/docker/libnetwork/blob/6659f7f4d8c1ec7a412a33f8973423ad38d22982/drivers/ipvlan/ipvlan_network.go#L197

thanks, will merge on green

Do you know what went wrong? I'm not familiar with your tests.

can you please rebase to the latest master, we fixed a bunch of flaky tests a while back

git remote add upstream git@github.com:docker/libnetwork.git
git checkout master
git merge upstream/master
git checkout 2418-fix
git rebase master

Sorry, bad merge.
I'll try to fix this mess.


Separate terminal history per editor/folder/project

In many of my assignments there are at least 2, usually more, different code bases we are working on. I mostly work with React but there is always a back-end of some sort and sometimes micro-services int he form av separate APIs.
This leads me to organize myself to use several editors for each code-base. This works perfectly in VS Code and it even highlights old "folders" I have had open before in the MacOS context menu.
My biggest issue in terms of developer experience is that when I close down my computer the terminal history (I believe it is zsh) is merged into one of the (either from VS Code or separate terminal window) and it slows down my workflow a lot having to re-write commonly used commands and the ability to find back to what branches etc I have switched between.
Would be great to have a separate history per terminal. Is this something that could be accomplished in VS Code you think? It would take it from awesome ty hyper-awesome.
MacOS, Catalina, evergreen VS Code.
Thanks for an awesome software that is helping me become a better coder each day.

I'm not sure, but I heard about this zsh plugin, you can use it with oh-my-zsh. Shell keeps history, so this is a solution probably.

I don't think this is possible. Things you could try:

A shell plugin like @sh7dm suggests
Set up a bunch of aliases with simple and/or unique names (ctrl+r) so you're less reliant on history


How to make the ellipse rotate with mouse move ?

I want to realize a function the Ellipse can be created when the  mouse down and move ,thus I get the Ellipse.cs in ScottPlot5 ,but there is a deviation when i used canvas.RotateDegrees to rotate, i don't know why?
The following is the code in Ellipse.cs
 public virtual void Render(RenderPack rp)
    {
        if (!IsVisible || RadiusIsNotFinite)
            return;

        using var paint = new SKPaint();

        rp.Canvas.Translate(Axes.GetPixel(Center).ToSKPoint());
        rp.Canvas.RotateDegrees((float)Rotation);

        float rx = Axes.GetPixelX(RadiusX) - Axes.GetPixelX(0);
        float ry = Axes.GetPixelY(RadiusY) - Axes.GetPixelY(0);

        PixelRect rect = new(-rx, rx, ry, -ry);
        Drawing.FillOval(rp.Canvas, paint, FillStyle, rect);
        Drawing.DrawOval(rp.Canvas, paint, LineStyle, rect);
    }

The following  is the code that i realised:
this function is used when mouse down and move ,the Ellipse Size is changed by UpdataEllipse()
public void UpdataEllipse(Coordinates fixPoint, Coordinates mousePoint)
{
    RadiusY = mousePoint.Distance(fixPoint) / 2;
    RadiusX = RadiusY * Math.Sqrt(1 - E * E);
    Center = new Coordinates((fixPoint.X + mousePoint.X) / 2, (fixPoint.Y + mousePoint.Y) / 2);
    double angleDegrees = Math.Atan2(mousePoint.Y - fixPoint.Y, mousePoint.X - fixPoint.X) * (180.0 / Math.PI);
    Rotation = angleDegrees;

    Debug.WriteLine(Rotation);
    FixPoint = fixPoint;//this is the mouse down position
    MousePoint = mousePoint; this is the mouse move position
}

protected override void OnRender(DrawingContext drawingContext)
{
    if (WPFPlot == null)
    {
        throw new InvalidOperationException("ScottPlot loading error");
    }

    var canvasWidth = (int)WPFPlot.OriginRect.Width;
    var canvasHeight = (int)WPFPlot.OriginRect.Height;

    using SKBitmap bitmap = new SKBitmap(canvasWidth, canvasHeight);
    using SKCanvas canvas = new SKCanvas(bitmap);
    using var paint = new SKPaint
    {
        StrokeWidth = 2,
        IsAntialias = true //
    };

   // i draw a line between mouse down point  FixPoint and mouse move point  MousePoint to verify the accuracy of the rotation angle
    PixelRect rect1 = new(GetPixel(FixPoint).X, GetPixel(MousePoint).X, GetPixel(FixPoint).Y, GetPixel(MousePoint).Y);
    Drawing.DrawRectangle(canvas, rect1, paint, LineStyle);
    PixelLine pixelLine = new PixelLine(GetPixel(FixPoint), GetPixel(MousePoint));
    Drawing.DrawLine(canvas, paint, pixelLine, LineStyle);

    // it's not easy for me to understand the function of canvas.Translate and   canvas.RotateDegrees
   // the  RotateDegrees  is  not the same as the line
    canvas.Translate(GetPixel(FixPoint).ToSKPoint());
    canvas.RotateDegrees((float)Rotation);

    float rx = GetPixelX(RadiusX) - GetPixelX(0);
    float ry = GetPixelY(RadiusY) - GetPixelY(0);
    PixelRect rect = new(-rx, rx, -ry, ry);

    Drawing.DrawOval(canvas, paint, LineStyle, rect);
    Drawing.DrawRectangle(canvas, rect, paint, LineStyle);
     // this is a new Layer I rendered over the Scottplot
    ImageHelper.DrawImage(drawingContext, bitmap, WPFPlot.Plot.LastRender.DataRect);

}

in the picture the red line is the deviation  , i want the red can coincide with yellow line

Hi @luyingru555, thank you for this question!
Unfortunately, there isn't enough information here for me to fully understand your question. I also cannot run the sample code you shared above because it is incomplete.
It sounds like you suspect that ellipse rotation is not accurate. I don't understand much more about your question though 😅
I will try to create a sample app that lets us evaluate whether ellipse rotation is accurate or not and will share what I find in a follow-up comment here.

This seems all right to me! I'll close this issue for now, but let us know if the code below can be modified to demonstrate the issue you're trying to describe and I'll open it back up and we can take a closer look together 👍
using ScottPlot;

namespace Sandbox.WinForms;

public partial class Form1 : Form
{
    ScottPlot.Plottables.Ellipse MyEllipse;
    public Form1()
    {
        InitializeComponent();

        // place an ellipse
        Coordinates center = new(3, 3);
        MyEllipse = formsPlot1.Plot.Add.Ellipse(center, radiusX: 3, radiusY: 0.5);
        formsPlot1.Plot.Add.Marker(center);

        // setup initial axis limits
        formsPlot1.Plot.Axes.SquareUnits();
        formsPlot1.Plot.Axes.AutoScale();
        formsPlot1.Plot.Axes.ZoomOut(3);

        // change ellipse rotation using the mouse
        trackBar1.ValueChanged += (s, e) =>
        {
            MyEllipse.Rotation = trackBar1.Value;
            Text = $"Rotation: {trackBar1.Value}º";
            formsPlot1.Refresh();
        };
    }
}

This line may be the key:
formsPlot1.Plot.Axes.SquareUnits();

Without it, the plot could be "stretched" so that a angle in Cartesian space is no longer the expected angle in pixel space. For example, consider this code which makes a line you'd expect to be at a 45º angle. If you "stretch" the plot, it is no longer 45º on the screen, but it is on the plot

axis with square units
axis with non-square units

Is there any way that the plot don't need to stretched without using formsPlot1.Plot.Axes.SquareUnits(), because i draw a ellipse which is not rendered with scottplot . it's diffcult for me to know where SquareUnits() work

Thank you for your answer!!  this is really the key,  i have resolved the problem


net/local: Add an option to specify the prefix of named pipe

Summary
to avoid the user create socket under the real filesystem
which isn't supported by nuttx VFS yet.
Impact
path like this "/tmp/test" don't fail anymore.
Testing

Should the default be "" so this is not a breaking change?

How to generate the pipe name is implementation detail(the path is already different from the user provided because the code appends the suffix(CS/SC/HD) to the path), the caller should never access the pipe by open/close pipe directly. And since all local socket api change to use the new path, the prefix addition is transparent to the caller.

Should the default be "" so this is not a breaking change?

How to generate the pipe name is implementation detail(the path is already different from the user provided because the code appends the suffix(CS/SC/HD) to the path), the caller should never access the pipe by open/close pipe directly. And since all local socket api change to use the new path, the prefix addition is transparent to the caller.

@xiaoxiang781216 - Thank you for the clarification.


Remove dead katacoda links

Signed-off-by: Yonathan Negussie Mengesha<EMAIL_ADDRESS>
What this PR does / why we need it:
Katacoda has been shuttered. This change removes the links in the documents.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Signed the commit for DCO to be passed
[ ] Labelled this PR & related issue with documentation tag

@divya-mohan0209 can I get a review on this one? The netlify preview also seems to be failing.


Add version tag to qhub header

Description
It would be helpful for end users to have insight into when changes have been made on the deployment.
Suggestion
Given that deployment is automated via CI upon tagging release (at least that’s mostly the case?), I’d like to request that the gh tag be added to the jupyterlab header. We did this in one of the qadmin deployments. Its just helpful for end users to know when a change has been made.
Context
An example - I asked John to update our env for me this morning (since I have no access). He had other things he was dealing with so he did it later in the day. If I had a version tag on the header, I’d be able to see when a change was implemented.

Question for the team:

Can we configure the "JupyterLab" name?

Question for the team:

Can we configure the "JupyterLab" name? Change to "QHub"?

Changing Browser Tab Title for JupyterLab jupyterlab/jupyterlab#4422

Update the QHub theme as well?

As per the latest discussion on this, there is two addition we can do to benefit the end-user:

Add the current qhub version to the Jupyterhub theme pages, this would be similar to Qadmin (as @kcpevey  commented)
The second one is adding another piece of info in the jupyterhub lab session tab, as suggested in the link above. To do so we would need to update the initialization command for the lab user jupyter lab --name='n-th lab' (I think we are currently adding the user name to that...)

For now, I will submit a PR to address the first point (which is the enchantment requested here) and will open another issue to discuss the second one.


Update specs2-core to 4.12.4-js-ec

Updates org.specs2:specs2-core from 4.12.3 to 4.12.4-js-ec.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2", artifactId = "specs2-core" } ]

labels: test-library-update, semver-pre-release

Superseded by #233.


add additional resources page

@gwleclerc is it ok if I add your talk video directly in the Resources page?

@gwleclerc is it ok if I add your talk video directly in the Resources page?

Yes you can use this one which is the best so far https://youtu.be/Y1CnSDbOWLA?si=YbQyfBiVNlOgUuTj but specify that it's in French


NOAA Tides

While running home assistant in docker, if I have NOAA_tides integration enabled I get the following error.
https://paste.ubuntu.com/p/P3ZSFBzfG5/

Wrong repository. Register your component in our portal to build wheels for it.

This is not my component it is found at https://www.home-assistant.io/components/noaa_tides/

Maybe you mean this. https://github.com/home-assistant/home-assistant/commit/4d08e73e3eb5db0391ca86488648b8acce95521d
Look like the maintainer of noaa did not want to run this on docker. I give it free now for you.

In around 4 hours it should work


Handle Encrypted DDOs

if encrypted, reach out to provider/node that was used to encrypt and fetch decrypted ddo

Hey team! Please add your planning poker estimate with Zenhub @alexcos20 @an7e @bogdanfazakas @jamiehewitt15 @mariacarmina @paulo-ocean


[WIP] Update logstash to latest version

Updated logstash to latest version for mitaka work.
Connects rcbops/u-suk-dev#176

I have one question, but this looks fine to me.

looks ok to me
:+1:

@d34dh0r53 is this actually a WIP or can we merge it?

@sigmavirus24 this is currently a WIP as the service doesn't start due to config issues.

Is this still a WIP?

RPC-AIO-Liberty-PR: Fail

recheck_swift

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

:+1:

Has anyone actually tested this, or just tested with their eyes?

So I have an AIO with this, #1245 and #1246 running and it seems to be working. Logstash is shipping things to elasticsearch and kibana is accessible and showing the items in queries out of elasticsearch.
So a 👍 after having tested this.

👍  I replicated what @sigmavirus24 saw, and was able to get a working ELK stack, in that logs are being passed all the way through to elasticsearch and are visible by kibana.


Can't stream to Zencoder

Hello
I'm trying to stream to zencoder.
When I get the rtmp link from Zencoder I try to create a stream but the app gets stuck with this output
ClientState: 1
ClientState: 2
ClientState: 3
ClientState: 4
ClientState: 5
ClientState: 6
received server window size: 10000000
received peer bandwidth limit: 10000000 type: 2
Received invoke
pktId:<PHONE_NUMBER>
received invoke onBWCheck
received peer bandwidth limit: 0 type: 0
received unknown packet type: 0x02
received ping, sending pong.
received ping, sending pong.
received ping, sending pong.
received ping, sending pong.
received ping, sending pong.
received ping, sending pong.
...
it keeps writing  "received ping, sending pong." forever and nothing changes
Could you please help me?

I also occured problem like you. The problem is the RTMP package size large than 128, and server seperate the package and start with a 0xC? byte in next package.

How did you resolve the issue? Any Clue>?

The last version shoult be fixed.


Don't raise exception if prop_name is not found

https://ocd.appscale.com:8080/job/Daily Build/497/


DashboardSkin databar gradient does not work if min value not 0

A gauge with dashboardskin and min value not zero, does not set properly  databar gradient because it consideres starts at zero.
The issue can be reproduced easily setting a minValue different to zero in OverviewDemo class.

Fix was included in last commit when added threshold indicator for DashboardSkin