CanopyLayers module is empty

As of now the package for CanopyLayers.jl is empty in main Land.jl, as compared to v0.1. It would be nice if an updated example.jl is pushed to main. Or what would be the recommended way to start working with CanopyLayers?

It is in the README of this repo: https://github.com/CliMA/Land#examples. The example folder contains a Manifest.toml file where deps reside (depends on branch v0.1).
v0.2 is a major release change, and thus is not ready to use yet. To use v0.1, follow the instructions at https://github.com/CliMA/Land/tree/v0.1.
FYI, in v0.2, CanopyLayers is renamed to CanopyRadiativeTransfer to be more specific.


[Python][CI] Add a Crossbow test build with a debug-enabled CPython

Describe the enhancement requested
We should add a Crossbow build running the PyArrow test suite with a debug-enabled CPython interpreter, to better test the sanity of our low-level C++/Python interface code.
One possibility is to install CPython using conda and the conda-forge/label/python_debug::python package reference.
Related to #43557
Component(s)
Continuous Integration, Python

Issue resolved by pull request 43565
https://github.com/apache/arrow/pull/43565


Draft: Separate Helm Charts

Breaks the helm charts out into individual charts with a superchart pattern.
generally by pulling this branch down it can be used in the same manner it always was via:
helm template k8s-mediaserver ./

When changes are made to the sub charts the changes can be pulled in via:
helm dep update

and in the future we can add a GitHub pages helm index for the charts and reference them via that.

@kubealex I'm leaving this in draft state atm, but I'd love if you could do a brief review to make sure you like the direction and to get your overall feedback.

@kubealex been on hold for the Jellyfin PR https://github.com/kubealex/k8s-mediaserver-operator/pull/72 for a month now.
Want me to just toss this all together?
Think with quick PR reviews I could get this PR done with Jellyfin within a week.
I'll start the work either way. 🙂 Probably have an update by tomorrow.

Got a few minutes not buried under kiddos. Had to modify my current fork slightly, but back into business with Jellyfin.
Will try to clean this up further, and prove that it gens the same stuff. Hopefully then we can just merge this PR all at once to get Jellyfin and individual helm charts.

Jellyfin has been merged @reedjosh, can you double check if this in any way creates any clash?

I will begin cleanup on this branch. Hopefully I'll have a non-draft PR within the next week. : )

@reedjosh Any idea how much cleanup you still want to do? Anywhere I can assist?

@NigelVanHattum I'll try to get back to this sometime soon.
Unfortunately, it will need quite a bit still as the upstream changes will need incorporating and my PR was still not polished even before that.
I do still want this done though, so I'll try to get to it in the next week or so.


Remove some consumer settings

Hey, I did some refactoring on the consumer and removed some settings to make the consumer simpler.

Removed duplicated default values

Reduce some global state

The consumer no longer tries to catch up on the buy timings. The old behavior caused irritating bursts sales.

Removed probability_of_buy. The buying behaviors are responsible whether an item is bought or not. If the total amount of sales should be reduced, the consumers per minute setting can be reduced.

Removed timeout_if_no_offers_available. This is not necessary since there is a waiting time between two consumers.

Remove setting for number of consumers (= number of threads started). Currently one thread is used. In future, if we want to bombard the marketplace with consumers and one thread cannot generate enough requests, we should find a solution that respects the consumers_per_minute setting.

I tried to remove the consumer settings from the global scope, but couldn't find a way.

The consumer no longer tries to catch up on the buy timings. The old behavior caused irritating bursts sales.

Why not having a short randomized wait time for catching up? Similar to merchant.

The problem with the previous implementation was that the consumer waited on rate limit but tried to catch up this waiting time afterwards. Thus, the rate limit timeout was not effective.
I'll try to find a way to catch up on buy timings, but also respect the rate limit timeout.


verdictdbblock question

Is it normal for the verdictdbblock value to always be 0?

It could happen if your dataset contains less than one million tuples.

OK thank you. Is the data stored in some table with the samples, or does this just happen when the queries are being executed through the DAG?

A sample (or equivalently, scramble) table is created when you issue a CREATE SCRAMBLE ... query.
The scramble table is materialized as a regular table in your database along with materialized verdictdbblock. This means "always 0" is stored in the table (not happen at query time).

And verdictdbtemp is used for what? When I create a scramble the verdictdbtemp has nothing ...

The schema is used for storing temporary tables (and removed when you exit VerdictDB). For simple queries, no temporary tables are created, so what you see is expected.

Ok, thank you.


Disable source map

Disable source map to remove long line. Fix https://github.com/graphql/graphiql/issues/439.

Thanks!


Update openlabcmd

Add switch_status for node to control HA master/slave switch work.
Add status filter for list_services

rec


WIP - Expose headings in search for pages in the Styles section

The pages in Styles are very long and are broken into sections that correspond to things that users are likely to be looking for, like 'headings, 'lists' etc.
Enhance search so that it matches against headings in pages in the Styles section.
We already have a plugin that extracts page headings and attaches them to
the metadata.
This work grabs that data and creates an entry in the Lunr store for
the search index.
At the moment each page heading is a separate store document.
Current size of the search-index file on Netlify: 5.1KB (gzip)
Design
@dashouse and @BrainJammed
I've put a chevron between section and page for now before we explore a better presentation

Trello ticket: https://trello.com/c/6MwA9cMf/1423-expose-headings-in-search-for-pages-in-the-styles-section-5

You can preview this change here:
Built with commit da7f0ee4a1f8de7d6bd9c877181c8c534999e855
https://deploy-preview-546--govuk-design-system-preview.netlify.com

comments from @dashouse
I would consider increasing the spacing slightly around the > but I can see “Styles > Typography” is a single string so may not be worth splitting it out
I was thinking ~10px either side, so it looks similar to the breadcrumbs spacing
But as the font size is smaller 5px might be enough

LGTM 👍


next.js - set up routes

Need desciption or proper proposal for this.  What needs to be done beyond what's already in place?  Is this a refactor, or does it imply change in behavior?


The type or namespace name 'Counter' could not be found

Description
error CS0246: The type or namespace name 'Counter' could not be found (are you missing a using directive or an assembly reference?)
1>Done building project "WinFormsBlazor.csproj" -- FAILED.
** Counter.razor in root of solution
Form.cs
var services = new ServiceCollection();
services.AddWindowsFormsBlazorWebView();
blazorWebView1.HostPage = "wwwroot\index.html";
blazorWebView1.Services = services.BuildServiceProvider();
blazorWebView1.RootComponents.Add("#app");
Page URL
https://learn.microsoft.com/en-us/aspnet/core/blazor/hybrid/tutorials/windows-forms?view=aspnetcore-7.0#create-a-windows-forms-blazor-project
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/hybrid/tutorials/windows-forms.md
Document ID
b0b88d48-74df-8dc4-446d-121e65b552ab
Article author
guardrex

Hello @behnamasaei ... Can you check and confirm that you used a BlazorWebView control?
There's a step that calls out making sure that you're placing the correct control. It's where the article says ...

Be careful not to accidentally drag a WebView2 control into the form.

If you accidentally use a plain WebView2, I think it throws that error.


error previewing parquet file: invalid schema for type: DECIMAL, can not handle precision over 18

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.31.1
Regression From
No response
Architecture
x64
Storage Explorer Build Number
20230822.1
Platform
All
OS Version
No response
Bug Description
when trying to preview a parquet. I encounter this error:
{
"name": null,
"message": ""invalid schema for type: DECIMAL, for Column: MyColumn, can not handle precision over 18""
}
the column MyColumn data type is money in the underlying sql server database. the parquet file was generated from the ADF Copy activity the parquet file seems valid.
As a side note while there is an error shown in the activities tab. the preview u/i shows the message "No Data" this is potentially misleading and I feel like it should say something like "Error Encountered" as no data implies at least to me that the file was empty.
here is a screenshot for context:

Steps to Reproduce
See Above
Actual Experience
see Above
Expected Experience
No response
Additional Context
No response

This is currently not supported. We will semi-support it in 1.32.0 by unblocking parsing when there are decimal fields whose underlying representation cannot be easily parsed (e.g. fixed length byte array, non-fixed length byte array) whose precision may be beyond 18.

@JasonYeMSFT not sure how I missed this issue last night... I did look. I do think the error messaging visibility in the preview window could be improved. Hope you will consider this for a future build. the preview features especially for parquet and Avro have been very useful.


Automated cherry pick of #1676: Wait for webhooks server using probes

Cherry pick of #1676 on release-0.5.
#1676: Wait for webhooks server using probes
For details on the cherry pick process, see the cherry pick requests page.
Kueue replicas are advertised as Ready only once the webhooks are functional.

This allows users to wait with the first requests until the Kueue deployment is available, so that the
early requests don't fail.

I needed to resolve simple conflicts, because due to the test code being in different places in release-0.5 and main. On main we now have e2e-common.sh, and we have singlecluster and multicluster subfolders for tests.

/release-note-edit
Kueue replicas are advertised as Ready only once the webhooks are functional.

This allows users to wait with the first requests until the Kueue deployment is available, so that the
early requests don't fail.

/kind bug


Handle File renames

Sidebar is still showing old name when a notebook gets renamed

sdidebar doesnt show notebook name any longer


Cannot resize columns in IE when Overlay is displayed

This issue occurs in IE 9+. if there are no rows to show, the overlay blocks a user's ability to resize columns, the icon for resizing does not show. This issue is not present in Chrome, (have not checked any other browsers).

This is because IE 9 doesn't implement the CSS spec correctly. It works in other browsers and later versions of IE.
As it's fixed in later IE (and Edge) I am slow to fix.
To get around this, you can use the property suppressNoRowsOverlay, and then if you want you can provide your own message to the user somewhere that doesn't block the grid.


Feature Request: Use "made skill by" in OtF

There are various optional or house rules that say you can do something to a certain degree or level dependant upon how much you make your skill by.
For example, dodging bullets.  It seems common to use the method of on Dodge = dodge the first round from a ROF>1 and then for every point you make your dodge by you can dodge another round from the total ROF.
So much like when you fire a weapon that has a ROF>1 you get a report on how many rounds would hit based on ROF and Bulk, it would be handy to have a variable that shows the most recent margin of success so you could add some text something like "You can Dodge X number of rounds", or whatever might be applicable in the situation you want, where X is a variable you can insert into the chat text you're outputting in the OtF.
Also happy if this has to be script and not chat.

Answered in #1094


How to use NVTX.jl with CUDA-aware MPI?

Hi, I used command nsys profile --trace=nvtx,mpi --mpi-impl=openmpi mpirun -n 2 julia -O3 --color=yes t10.jl to run the code:
# t10.jl
function main()
    MPI.Init()
    comm = MPI.COMM_WORLD
    rank = MPI.Comm_rank(comm)
    n = 10000000

    CUDA.device!(rank)

    if rank == 0
        sendmsg = CUDA.rand(n, 30)
    elseif rank == 1
        recvmsg = CUDA.zeros(n, 30)
    end

    CUDA.synchronize()
    MPI.Barrier(comm)

    if rank == 0
        NVTX.@range "send data" begin
            MPI.Isend(sendmsg, comm; dest=1, tag=1)
        end
    elseif rank == 1
        NVTX.@range "receive data" begin
            rreq = MPI.Irecv!(recvmsg, comm; source=0, tag=1)
            MPI.Wait(rreq)
            recvmsg .+= recvmsg .^ 2
        end
    end

    CUDA.synchronize()
    MPI.Barrier(comm)
end

main()

But I cannot see GPU utilization in Nsight-system:

Did I do something wrong?😅

You may need to trace cuda as well, i.e. --trace=nvtx,mpi,cuda?
If that doesn't work, try using the profiler inside mpirun: I had better results with that.

If I was to hazard a guess:

the gap at the start is compilation.
the gap at the end is the GPU finishing work that was launched.


Double clicking on any of the items in 'Help and Additional Resources' should just open the external link

Feature Request
Double clicking on any of the items in 'Help and Additional Resources' should just open the external link. Currently, the Open External link sphere symbol is displayed and it is not obvious that this symbol needs to be clicked on. Though, clicking on it does indeed open the external link.
Describe the solution you'd like.
Double clicking on any of the items in 'Help and Additional Resources' should just open the external link. Users would normally expect to be able to double click on an item.
Actually, this applies to items under all categories.

Complete. Closing


Attach AVPlayer to PlayerView earlier

Fixes #229
After some investigation, I found out that attaching the AVPlayer instance to AVPlayerLayer not early enough causes side effects as described in #229.
Currently, it is attached very late, sometimes after play() is called on the AVPlayer instance.
With this PR, I am moving the attachment of AVPlayer on AVPlayerLayer as early as possible.

thanks @MartinMoizard ! ✌


Permission Request

We would like to use the d3.v3.js on commercial products. but according to our enginering regulation, 'BSD-3-Clause licensed javascript should be include the license description'.
So, your source code does not include it.
Can you please add BSD-3-Clause License to your source code, or comment here for permission?
Thank you for coopration.

Duplicate of #2741. The LICENSE is here: https://github.com/mbostock/d3/blob/master/LICENSE


[Syntax]: get rid of net keyword

Let there be nodes {} and everything else be network by default. So what's inside component {} is either nodes block or connection.
There probably should be only one nodes block but question is should that be enforced by analyzer or just linter.
Before
component Main(start) (stop) {
	nodes { Range, PrintLine, Match }
	net {
		:start -> [
        	(1 -> range:from),
			(101 -> range:to)
		]
		range.data -> printLine -> match:data
		100 -> match:case[0] -> :stop
	}
}

After
component Main(start) (stop) {
	nodes { Range, PrintLine, Match }
	:start -> [
		(1 -> range:from),
		(101 -> range:to)
	]
	range.data -> printLine -> match:data
	100 -> match:case[0] -> :stop
}

Motivation
While being less explicit and also less consistent with the rest of the language this has some benefits

Saves us about 2 lines of code and one nesting level
Looks more like a function with var on top of it and list of instructions in a body

In other words it probably improves readability.
What about downsides? Well, let's discuss them.

It's less explicit

Yeah that's for sure. However, you can't program in Nevalang anyway without reading documentation. You learn it once and you read code thousands of times then. So what we should optimize here is the reading part.

It's less consistent.

Yeah but the thing is - who said that everything should always be consistent? Consistency is a nice to have thing but it's just of the things we need. Sometimes consistency has some downsides. What we need to achieve when we're talking about the syntax is "easy to read". Sometimes consistency contributes to that. Maybe this isn't that time?

Done


FastCGI and FastCGIServer doc updates

Updated the doc for the FastCGI and FastCGIServer classes, as FastCGI had very small changes

Codecov Report

Merging #253 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #253      +/-   ##
==========================================
- Coverage   74.81%   74.74%   -0.08%
==========================================
  Files          31       31
  Lines        4181     4181
==========================================
- Hits         3128     3125       -3
- Misses       1053     1056       +3

Flag
Coverage Δ

#CHTTPParser
57.46% <ø> (-0.07%)
:arrow_down:

#KituraNet
74.74% <ø> (-0.08%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/KituraNet/FastCGI/FastCGI.swift
100% <ø> (ø)
:arrow_up:

Sources/KituraNet/FastCGI/FastCGIServer.swift
64.61% <ø> (ø)
:arrow_up:

...ces/KituraNet/Server/ServerLifecycleListener.swift
79.16% <0%> (-4.17%)
:arrow_down:

Sources/KituraNet/HTTP/HTTPServer.swift
80.44% <0%> (-0.56%)
:arrow_down:

Sources/CHTTPParser/http_parser.c
57.27% <0%> (-0.07%)
:arrow_down:

Continue to review full report at Codecov.

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


Master

small README.md edits (emphasizing the button and that you need a Heroku account)

The README is definitely more clear. Thanks! :smiley:


Power Rename

Microsoft PowerToys version
0.53.3
Running as admin

[X] Yes

Area(s) with issue?
PowerRename
Steps to reproduce
Open the power rename menu, then rename something, close it, then reopen
✔ Expected Behavior
When I reopen would be nice if it remembered the window size and location.
❌ Actual Behavior
When it opens, the window is too small, requires extra steps to resize.
Other Software
No response

cc: @stefansjfw

Is there any chance that Power Rename will start to save/restore its size & position?
Also, the default size/position is not always good, either:  I have three monitors 1080x1920, 3840x2160, and 1280x1024. If File Explorer is on the first (tall, narrow) screen, then Power Rename opens centered (I believe) on that screen but with a default size that is HUGE: roughly 60% or so the size of the (4K) center screen.
If Power Rename is always going to open in a "default" size/location, it could at least be sized appropriately for the screen where it is appearing.

Is there any chance that Power Rename will start to save/restore its size & position?

Yes, this will be added soon. Thanks for the report.

/dup https://github.com/microsoft/PowerToys/issues/2152


std.array.extend

bearophile_hugs reported this on 2014-11-29T13:44:38Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=13797
CC List

greensunny12

Description
I suggest to add to Phobos a simple function like this, mainly useful to extend dynamic arrays with a given lazy iterable (but this works with std.array.array too):

void extend(A, R)(ref A arr, R iterable)
if (__traits(compiles, { foreach (item; iterable) arr ~= item; })) {
    import std.range: hasLength;

    static if (hasLength!R && hasLength!A)
        arr.reserve(arr.length + iterable.length);

    foreach (item; iterable)
        arr ~= item;
}

void main() {
    import std.stdio, std.range;

    int[] arr;
    arr.extend(only(1, 2, 3, 4));
    writeln(arr);

    static struct Gen5 {
        int opApply(int delegate(ref int) dg) {
            int result;
            foreach (i; 0 .. 5) {
                result = dg(i);
                if (result)
                    break;
            }
            return result;
        }
    }

    arr.extend(Gen5());
    writeln(arr);

    import std.container: Array;
    Array!int arr2;
    writeln(arr2[]);
    arr2.extend(only(1, 2, 3, 4));
    writeln(arr2[]);
}

greensunny12 commented on 2018-03-31T15:07:09Z
Hmm. Why can't we handle this on a language level?

---
int[] arr;
arr ~= [0, 1, 2];
---

works fine. So I don't see any reason why the compiler couldn't be improved to accept this too:

---
int[] arr;
arr ~= 3.iota;
---

Custom data types can already do this with operator overloading, but built-in arrays can't.

I submitted an enhancement request: https://issues.dlang.org/show_bug.cgi?id=18699


cargo multivers not working properly

Compiling 39 versions compressed into a runner
Error: Cargo command failed: The cargo subcommand returned an error.
error: current package believes it's in a workspace when it's not:
current:   /home/admin/tmp/surrealdb/target/cargo-multivers/package-runner/Cargo.toml
workspace: /home/admin/tmp/surrealdb/Cargo.toml
this may be fixable by adding target/cargo-multivers/package-runner to the workspace.members array of the manifest located at: /home/admin/tmp/surrealdb/Cargo.toml
Alternatively, to keep it out of the workspace, add the package to the workspace.exclude array, or add an empty [workspace] table to the package's manifest.
I got this error 2 times, adding target/cargo-multivers/package-runner to the workspace.members in /home/admin/tmp/surrealdb/Cargo.toml didn't fix the problem. How can I fix this?

You are right something was missing to make it build a binary from a workspace. It is now fixed in the version 0.7.0.

You are right something was missing to make it build a binary from a workspace. It is now fixed in the version 0.7.0.

The issue was however somehow fixed by adding exclude = ["target/cargo-multivers/package-runner/Cargo.toml"]
That's why I closed the issue, it is awesome if it now works without this "workaround", I cannot test it yet however since I'm building something with the fixed workaround and it will take more than a few hours, I would gladly test if after though.


add new subdirectory to boot image to avoid glob expansion issue

Adds a new directory inside boot-image that can be entirely deleted & recreated by cmake. This avoids this glob expansion issue and still allows boot-image/ to be bind-mounted outside of a docker container.
After this update, devs should run rm -rf /root/psibase/rust/psibase/boot-image/* inside their environments to manually clear out the boot-image.

Why not bind-mount higher up the tree?

Why not bind-mount higher up the tree?

I don't feel strongly about it, but I did it to keep the bind mount smaller / more focused. Didn't seem useful to expose src/ or tests/ to the host.


Security group setup with changing public IP

Flintrock version: 1.0.0-dev0
Python version: 3.6
OS: Ubuntu 18.04

Hi, thanks for the great work on this tool. I've been having some trouble using Flintrock if my client's public IP changes as I lose SSH access to the cluster. Is there an existing way to push IP updates to all the relevant security groups using Flintrock?

Hi @pratiksha. I'm glad that Flintrock is useful to you.
Flintrock does not have a way to push security rule updates to an existing cluster.
You can specify additional security groups on launch with flintrock launch --ec2-security-group. You can configure one of those additional groups with a more permissive rule based on a CIDR range that will tolerate the changes in your public IP address, as opposed to a single, specific IP address rule that will block you the moment your address changes. That may work well enough for your needs.


In bundled js, automatically handle affixed sub-navigation components

Also, scrollspying

oh, I thought it is already there. can you point out the script that handles this part in documentation?


 when i use 'import Vue from vue'

how can i make the code work?

that's because import from vue doesn't contain comipiler
https://github.com/skypackjs/skypack-cdn/issues/78

please check https://esm.sh/?test, vue2/vue3 can work fine.


C API test using google test

First part adressing #73

Coverage is rising :smile:
I do not know how to provoke this one:
https://github.com/trixi-framework/libtrixi/blob/bb9582c8d720e15a6940bb9edacbd00f0683290b/src/auxiliary.c#L116-L120
If a non-existing function pointer name is given, the execution of the julia command will already abort with an error message.

If a non-existing function pointer name is given, the execution of the julia command will already abort with an error message.

Yeah, this path you cannot easily check. Just leave it be for now.

95% coverage 🥳


Detection error: First station works well second station shows error.

Hi, when running detections (from seeds directly) the first station in my seeds directory works well then I get the following error (I am using a very similar code as the one in the example detections.ipynb).

02-26 08:43 [INFO] [EQTransformer] Finished the prediction in: 0 hours and 5 minutes and 49.27 seconds.
02-26 08:43 [INFO] [EQTransformer] *** Detected: 364 events.
02-26 08:43 [INFO] [EQTransformer]  *** Wrote the results into --> " /Users/acosta/EQTransformer-master/examples/detections2/B057_outputs "

02-26 08:43 [INFO] [EQTransformer] Started working on B921, 2 out of 53 ...
02-26 08:43 [INFO] [EQTransformer] 20190901T000000Z__20190902T000000Z.mseed

KeyError                                  Traceback (most recent call last)
/Users/acosta/anaconda3/envs/eqt/lib/python3.7/site-packages/EQTransformer/core/mseed_predictor.py in mseed2nparry(args, matching, time_slots, comp_types, st_name)
447         meta["instrument_type"]=st[0].stats.channel[:2]
--> 448         meta["network_code"]=stations[st[0].stats.station]['network']
449         meta["receiver_latitude"]=stations_[st[0].stats.station]['coords'][0]
KeyError: 'B921'
During handling of the above exception, another exception occurred:
KeyError                                  Traceback (most recent call last)
 in ()
14          overlap=0.3,
15          gpuid=None,
---> 16          gpu_limit=None)
/Users/acosta/anaconda3/envs/eqt/lib/python3.7/site-packages/EQTransformer/core/mseed_predictor.py in mseed_predictor(input_dir, input_model, stations_json, output_dir, detection_threshold, P_threshold, S_threshold, number_of_plots, plot_mode, loss_weights, loss_types, normalization_mode, batch_size, overlap, gpuid, gpu_limit, overwrite)
302             eqt_logger.info(f"{month}")
303             matching = [s for s in file_list if month in s]
--> 304             meta, time_slots, comp_types, data_set = _mseed2nparry(args, matching, time_slots, comp_types, st)
305
306             params_pred = {'batch_size': args['batch_size'],
/Users/acosta/anaconda3/envs/eqt/lib/python3.7/site-packages/EQTransformer/core/mseed_predictor.py in mseed2nparry(args, matching, time_slots, comp_types, st_name)
452     except Exception:
453         meta["receiver_code"]=st_name
--> 454         meta["instrument_type"]=stations[st_name]['channels'][0][:2]
455         meta["network_code"]=stations_[st_name]['network']
456         meta["receiver_latitude"]=stations_[st_name]['coords'][0]
KeyError: 'B921'

I don't understand where this comes from since the two station messed files were downloaded in the same way and I have the exact same channels for both of them.

@MAcosta-GitHub it seems there is an inconsistency in the miniseed files. Could you check the miniseed file (for statin B921) using Obspy to see if all the fields for stream are written out?


Disable Italian in v1.13

This PR disables the Italian localization until it meets minimum standards of quality.
Context: sig-docs Slack
/sig docs
/assign @rlenferink

/approve


Provide a convenient way of producing a Provider<Directory/RegularFile> from a file collection with only one element

For a file collection there is the method FileCollection.singleFile(). Though this API is not lazy and does not inherit task dependencies of the file collection. It would be for there to be a convenient way to obtain a Provider<Directory>/Provider<RegularFile> from a FileCollection by methods which could be called FileCollection.getSingleDirectory()/FileCollection.getSingleRegularFile().
There is a workaround which you can do already to obtain a Provider<FileSystemLocation>:
FileCollection.getElements().map { it.iterator.next() }

Split out of #4532.

This is extremely useful when, for example, you want need to go from a Configuration to a RegularFileProperty.
Thanks for providing the workaround, @wolfs, I will wrap it in a kotlin extension method so it will be less verbose but having something native would be even better.


docs: min sdk version mention in the docs somewhere

Background
Dev eng stand up Sep 1 at 8AM Pacific-
We discussed addin the minSdkVersion 1.5.32 ? (I think that's right) into the documentation
People who use at_onboarding_flutter might not know to do this.
@yahu1031 @CynthiaPeter is this the correct description of what we discussed at stand?

Just like what Minnu said, it's more of trying to ensure some level of consistency across applications: https://fvm.app/

Also discussed potentially adding FVM to the documentation https://fvm.app/

I recommend against this, it adds yet another barrier of entry to developing on our platform. Or at most, this should be optional since the usual expectation is that developers are using stable channel.


ARROW-6437: [R] Add AWS SDK to Homebrew formulae

@ursabot crossbow submit homebrew-cpp homebrew-cpp-autobrew

@ursabot crossbow submit homebrew-cpp-autobrew


Translation to German and small enhancement

Hi,
I changed the "save as Image" Button to be translatable and added a German translation for this awesome theme. Please Double-check the translation for Chinese and Japanese - I had to use Google translate on those two languages ;) I am native German speaker, so, I assume, German should be ok.
Cheers
Peter

Thx Peter! LGTM.


Multi-master deployment

Currently multiple masters can be deployed if masterCount and masterZones flags are set appropriately. But this multi-master deployment is not really usable or tested. This issue is for:

Creating a proper multi-master deployment with load balancer.
Testing this multi-master deployment to see how the failure of a master will affect the cluster and end user.

I understand this is stretch goal for v1 but if we could get this to work with S/W load balancer with little effort then it's a big plus.

This issue was moved to kubernetes/kops#2734


CI: Npgsql test matrix is incorrect

Problem
It is unexpected that GH-169 is green, because Npgsql 8.0 does not support .NET 6 any longer. So, why doesn't it fail?
Observation
This test matrix slot, which should invoke .NET 5.0.x, apparently also uses .NET 8, which is wrong.
dotnet test --framework=net8.0 --collect:"XPlat Code Coverage"

-- https://github.com/crate/cratedb-examples/actions/runs/7033351740/job/19138983476?pr=169#step:7:136
Conclusion
Test matrix slot value propagation is flawed somewhere and needs to be fixed.

Those patches will remedy the issue.

https://github.com/pyveci/pueblo/pull/35
GH-178


Execution Workflow: APAlyzer

WHAT
Write execution workflow for APAlyzer. Use the provided small files for testing (running the workflow on real data is a different issue).
CHECKLIST

[x] Use snakemake template or nextflow template to create your workflow.
[ ] Comment your code
[x] Run individual rules/processes in either conda envs or docker/singularity containers for reproducibility
[x] Input:  .bam or .fastq from test_data
[ ] Give feedback about the method

OUTPUTS (see specification):

[ ] ~Output: Adhere to output specification for Identification challenge~ (No identification)

[ ] Output: Adhere to output specification for quantification challenge

This BED file contains positions of unique poly(A) sites with TPM values for each identified site in the score column.

chrom - the name of the chromosome
chromStart - the starting position of the feature in the chromosome
chromEnd - the ending position of the feature in the chromosome; as identified PAS are single-nucleotide, the ending position is the same as starting position
name - defines the name of the identified poly(A) site
score - TPM value for the identified site
strand - defines the strand; either "." (=no strand) or "+" or "-".

[ ] Output: Adhere to output specification for differential usage challenge

This TSV file contains two columns:

gene ID
significance of differential PAS usage

Column names should not be added to the file.

I have created the following branch https://github.com/iRNA-COSI/APAeval/tree/apalyzer
APAlyzer implementation is currently stuck since the creation of a reference genome based on a given gtf is not working for this tool. You currently have to rely on predefined PAS atlas, which is not reliable for the benchmark.
I opened an issue in the git repo of the tool (https://github.com/RJWANGbioinfo/APAlyzer/issues/7) and await further info.

@mkatsanto looks like issue you've reported has been fixed. Are you still able to carry on with completing this workflow (looks like you've done a lot already!) or should we consider this workflow open for other contributors?
Thanks :)

Hey @mkatsanto, do you think you'll find the time to finish this off over the next couple of weeks or should I reassign the workflow to someone else?

Hi! Sorry for the late reply! I will try in the next week to resolve it . If someone wants to also take a try or work also on this, it is fine by me.

Hey @mkatsanto, just a gentle bump on this workflow. We've got a deadline of 30th September for getting the execution workflows merged, do you think you'll have time to finish it off?


return axes object in plotting functions

Currently, dependence_plot(), summary_plot() finishes with
if show:
    pl.show()
return

which displays the plots, but does not expose the axes object to outside of the function.
Returning the axes object will allow the plot to be further adjusted (changing axes labels, x- and y-limits, change to log_scale, aspect ratio, etc.
One way might be something like this:

# added "ax=None" to function arguments
def dependence_plot(ind, shap_values, features, feature_names=None, display_features=None,
                    interaction_index="auto", color="#1E88E5", axis_color="#333333",
dot_size=16, alpha=1, title=None, show=True, ax=None)

...

if ax is None:
    fig, ax = plt.subplots(1, 1)

# change "pl." to "ax." afterward

if interaction_index is not None:
        ax.scatter(xv, s, s=dot_size, linewidth=0, c=features[:, interaction_index], cmap=red_blue,
                   alpha=alpha, vmin=clow, vmax=chigh, norm=color_norm, rasterized=len(xv) > 500)
    else:
        ax.scatter(xv, s, s=dot_size, linewidth=0, color="#1E88E5",
alpha=alpha, rasterized=len(xv) > 500)

# and finally

return ax

I will try to submit PR if the idea is sound.

Thanks for the suggestion! If you just want to adjust the axes and such, you can pass show=False and then just call pl.xlim(0,1) after the plot, finishing with your own pl.show(). You could also call pl.gca() if you want the axis shap was using.
There could be other cases where passing back the axis is better. If you have some in mind let me know.

@slundberg that works like a charm. Thank you!

Hi, I am trying to place summary plots on a subplots grid. something like: 4 models, a normal summary and a bar one for each model on each line.
I tried using ax = plt.gca() after producing the plot and then placing the ax on a grid generated by fig,axes = plt.subplots(4,2) but it's not working at all, the new grid shows empty even if the axes variable contains the generated axes
Also, it seems plt.gca() catches the axes only if I place it before generating the plot.
I believe adding an ax parameter to the summary_plot function would work better in this use case, as I did a similar thing with dependence plots and it worked like a charm.


Do not re-create text views on every text draw in CTCatalog test app

Fixes #1046

@aballway is added to the review. #Closed

:shipit:

:shipit:

:shipit:

:shipit:

Wouldn't you know there's a better way of updating views than murdering and replacing them


Retrieves Port Group

Response to issue #595
Retrieves port groups for both VSS and DVS
Script either succeeds or fails with error checking built-in

@pgbidkar Can you review please?

@pgbidkar Can you review please?

Sorry for the delay. It got out of pipeline

It works. Good to merge.

@pgbidkar LOL 😊 Sorry about that. I committed the thing I didn't want to commit and left my actual changes uncommitted... 😆


[Docs] SMI Conformance functionality page causing build errors

Description
The recently created SMI Conformance functionality page is causing Jekyll build errors.
Expected Behavior

The front matter of this page needs to have a colon (:) after each key, like so:

---
layout: page
title SMI: Conformance Capability
permalink: functionality/smi-conformance-capability
type: functionality
---

A missing / has the permalink for the page showing in the wrong place. A forward slash is needed after "functionality".

Rename filename of this page to smi-conformance.md and update the permalink to functionality/smi-conformance

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

I want to start with this.

@kalradev, excellent!


Re-enable ONNX tests on ROCm.

These might be less flaky now.
ci-exactly: build_packages,regression_test_amdgpu_rocm

Yeah, this runner has been a lot more stable. hopefully can handle these now


[linen] allows multiple compact methods

What does this PR do?
WIP

Hi there @cgarciae, how far off is this from getting merged & released. I'd love to be able to mark multiple methods as compact. Cheers!


arch/tiva: Serial TIOCxBRK BSD-compatible BREAK support

Summary
In the lower half UART driver for Tiva architecture (TM4C12x), adding the TIOCxBRK ioctl calls, which allow an application to transmit a BSD compatible line BREAK. TIOCSBRK starts the BREAK and TIOCCBRK ends it.
This architecture supports BSD-style BREAK in hardware. We write to the BRK bit (bit 0) of the UART Line Control register (UARTLCRH) to start the BREAK, which begins after the UART finishes shifting out the current character in progress, if any, including its stop bit(s), and continues indefinitely until we write to the BRK bit again to stop the BREAK.
Impact
When new Kconfig option TIVA_UART_BREAKS is activated, TIOCSBRK and TIOCCBRK ioctl calls are supported on Tiva architectures. When not activated, no effect.
Testing
Custom board.

LGTM

Your input on this (previously on the mailing list) has been tremendously helpful. Thanks for all your help!


Request for New Release to Include Types

Hello @claustres ,
I hope this message finds you well. Could you please consider publishing a new release of the library? I’ve added new types recently, and incorporating them in an official release would significantly improve type safety and consistency for users.
Thank you very much for your support!
Best regards,
Andriy Mriy

@MrAndriy finally this is done.
Thanks for your contribution :)


FIX Only change constraints for recipes

Bad logic in cow resulted in many non-recipes having their constraints updated automatically when the beta was released.
Only recipes should have their constraints updated.
Issue

https://github.com/silverstripe/.github/issues/33

I am confident it'll work. But I can't do a dry run with it until the PRs are merged that set the correct constraints to start with.
Once those are merged I'll do a dry run.

This has been approved and I've confirmed in a dry-run that it works as expected.
Merging.


feat: ability to configure self-agent component's node selector and tolerations

Signed-off-by: Soumya Ghosh Dastidar<EMAIL_ADDRESS>
Proposed changes
This PR adds env var to configure self-agent component's node selector and tolerations. Related to #3526
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[x] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the commit for DCO to be passed.
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Special notes for your reviewer:
@rajdas98 @Jonsy13 will need to expose the env vars in helm after which the issue can be closed

@gdsoumya can we rename SELF_CLUSTER to SELF_AGENT with this PR?

@rajdas98 done


Use GNU main server for vTPM autoconf

The mirror server is not reliable and happens to be down frequently.

@shjala Can you also do a PR to the 12.4 branch so I can get that one to build as well?
Do we refer to this in stable releases? Don't know if we need to fix this proactively in such branches or only do it if we run into build failures when building a patch in those branches.


增加数据集生成功能

自定义数据集生成工具，配置好dataset_util 下的db_config.yaml， 然后执行dataset_util 下 main.py，即可生成 spider 格式的数据集表元数据

lol anytime On Oct 28, 2023, at 11:24 PM, wangzaistone @.***> wrote:
@wangzaistone commented on this pull request.
great work!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Page is empty

After logging in (solidcommunity.net), I get an empty gray page.
No errors are shown in the console.

Oh yes, if it does not find any data it displays an empty page (with no means to add that data, either). I will try to fix it soon. Thank you!

This has been fixed now.


[python]: enable crashtracker tests

Motivation
Enable crashtracking tests for dd-trace-py==2.11.2+ for the SSI GA effort.
Changes
Enables crashtracking tests for dd-trace-py==2.11.2+
Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Merge waiting on 2.11.2 release. Will merge this after that's published to PyPI


Fix test hooks for CommandLineParameterParser

PR Summary
xUnit tests fail when running with Debug configuration after #11482. The failed tests are those related to WindowsStyle_With_Right_Value because ConsoleControl.SetConsoleMode actually gets called during testing but there is no console window, so the assert fails.
This PR adds a constructor for testing code, which marks the created instance as isForTesting == true.
When isForTesting == true, we skip the actual operations like SetConsoleMode for -WindowStyle and SetSystemConfigFilePath for -SettingsFile.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

My idea is to make the parser self-contained as much as possible -- its properties shouldn't be changed from outside the parser except for test hooks.
Specifically, ConfigurationName should be properly set within the parser, so after calling _parser.Parse(args), _parser should have the correct ConfigurationName value without needing to do the following:
_parser.ConfigurationName = CommandLineParameterParser.GetConfigurationNameFromGroupPolicy();

ConsoleHost is owner of the parser (s_cpp), it defines how use results of command line parsing so I believe all logic should be in ConsoleHost, not in the parser.
It would be surprising if the PowerShell parser started compiling or interpreting or making WMI queries.

I don't think it really matters to make CommandLineParser a pure argument parser that doesn't do anything else, and I can imagine that future changes to it for additional flags may likely add simple side effect operations to it. I don't see that's a problem.

I don't see that's a problem

Oh, I spent many time to debug new tests because the parser has side effects. If you catch the issue again it confirms that the design is not so good and the worst thing is that this again provokes the addition of workarounds not only in tests but also in the main code. This is what I tried to avoid in my PR. Sorry for long discussion.

I spent many time to debug new tests because the parser has side effects.

It reveals the fact that the code was not written in a testable way, and hence I added the constructor for testing purpose and the _isForTesting field.
Again, I personally don't care that much to make the argument parser a pure parser without side effect, and it's likely we cannot hold that true in future changes, unless you can catch and review all subsequent changes to the argument parser :).
For now, I care more about fixing the test failure in debug build, so I will just go with your approach for this PR.

It reveals the fact that the code was not written in a testable way, and hence I added the constructor for testing purpose and the _isForTesting field.
Again, I personally don't care that much to make the argument parser a pure parser without side effect, and it's likely we cannot hold that true in future changes, unless you can catch and review all subsequent changes to the argument parser :).

To express a personal preference, I think:

Testability and purity are closely related, and both also help code quality and robustness
Using flags to change side-effects into testable state means testing code paths and production code paths aren't the same
My ideal would be for the argument parser to process the string[] args input and spit out an object that we could either test or send to the consuming ConsoleHost component. Similar to this .NET library or the prevailing Rust library. (System.CommandLine operates at a different level, so doesn't really compare here)

With that said, I know we ultimately want to minimise code churn and maximise performance, so an "ideal" design may not be the right move here

I know we ultimately want to minimise code churn and maximise performance

As side value, after decoupling the parser we can easy measure its performances with dotnet test.
As side thought, we do xUnit tests based exclusively on SMA and we could test decoupling (independent from SMA) code in separate xUnit tests to check the algorithms used.

The Windows test fails
[-] Pre-Requisistes link for 'WMF 4.0' is reachable: https://www.microsoft.com/download/details.aspx?id=40855

The link does not work https://www.microsoft.com/download/details.aspx?id=40855
I think it is important for Windows Installer.
/cc @TravisEz13 @adityapatwardhan  @SteveL-MSFT

The Windows test fails

We're working on this. Unclear why it's failing

That test was disabled by https://github.com/PowerShell/PowerShell/pull/13479. The failure was tracked by #13478


fix parallel test execution

When tests are executed in parallel on the same machine (which happens due scylla oss and enterprise pipelines may run simultaneously) one of it fails with scylla port being busy error.
Add simple semaphore using socket to make one test wait for other finishes.

@soyacz this solution is gonna be a bit problematic once there are multiple releases running those. it might start timeout the whole job.

@soyacz this solution is gonna be a bit problematic once there are multiple releases running those. it might start timeout the whole job.

Yes, I can enhance this solution to make parallel jobs possible.

I also updated commit message


Feedly sign in issue on M1

OS: macOS Monterey 12.1
Machine: M1 apple silicon
Software version: 6.0.3 (6035)
I tried today to install NetNewsWire on my new machine. Everything seems to work fine except for the Feedly integration in the Accounts tab: looks like no browser window is opening after the “NetNewsWire.app” wants to use “feedly.com” to sign in dialog. I clicked on Continue with no luck.
I also tried to quit the app and change the default system browser (tested with Safari and Brave) but nothing changed.

This is a Monterey bug. You can reboot your computer to resolve it.

You were right! It worked.


Python : Logs are not coming in App Insight for async function using eventloop

I am using azure function for one of our python function. We are using eventloop as described in document.
Below is sample of __init__.py
from typing import List
import logging
import json
import asyncio

import azure.functions as func

from .somepackage import SomeClass

async def main(events: List[func.EventHubEvent]):  # pragma: no cover
    """
    Starter function for the azure functions
    @param events:
    @return:
    """
    loop = asyncio.get_event_loop()
    tasks = []
    logging.info("Got %s events" % len(events))
    for idx, event in enumerate(events):
        event_body = json.loads(event.get_body().decode('utf8'))
        properties = event.metadata['PropertiesArray'][idx]
        logging.info("Got Event : %s with properties %s" % (event_body, properties))
        try:

            tasks.append(
                asyncio.create_task(wrapper_function(loop, event_body, properties)))
        except Exception as e:
            logging.exception("Error occurred while processing surface event trigger")
    if tasks:
        result, _ = await asyncio.wait(tasks)
        logging.info("Function : event processing done")

async def wrapper_function(event_loop, event_body, properties):
    some_obj = SomeClass()
    result = await event_loop.run_in_executor(None, some_obj.run, event_body, properties)
    return result

and this is content of somepackage.py file,
import logging

class SomeClass(object):

    def __init__(self):
        logging.info("Object created")

    def run(self, event_body, props):
        logging.info("event body is %s" % event_body)
        logging.info("Properties are %s" % props)

The logging statements from run function are not coming to app insight which is connected to our azure functions.
If I use normal function instead of async eventloop it push the logs to app insight.
Please let me know if more details required on this.
Thanks

Hi @amitnanaware, are you still facing this issue?

Hi @shreyabatra4 yes this issue is still exists.
On Mon, 7 Feb 2022 at 11:40 PM, Shreya Batra @.***>
wrote:

Hi @amitnanaware https://github.com/amitnanaware, are you still facing
this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-functions-python-worker/issues/898#issuecomment-1031765419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHBOPPETM4FILEWK2KYSX5DU2ADJ7ANCNFSM5F3GAJPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@amitnanaware thanks for raising this issue! We have investigated and root caused the problem. Essentially the host checks if invocation_id field (id generated by the host to identify each call to your function in function app.) is present in generated LogRecord object which wraps your log message, if yes then only it will log messages to AppInsights. For the missing logs you can't find in AppInsights is because invocation_id field is missing in LogRecord objects of your log messages.
Currently for the case if a function is a single threaded blocking function, invocation id is only set to thread local variable(_invocation_id_local) in worker managed executor's threads. (https://github.com/Azure/azure-functions-python-worker/blob/dev/azure_functions_worker/dispatcher.py#L644-L648). We do not set invocation_id to customer managed executor's threads, and even trying to set it, that _invocation_id_local variable is not accessible outside worker module and is not considered good practice.
(more for details at https://github.com/Azure/azure-functions-python-worker/blob/dev/azure_functions_worker/dispatcher.py#L742-L754 and https://github.com/Azure/azure-functions-host/blob/dev/src/WebJobs.Script.Grpc/Channel/GrpcWorkerChannel.cs#L696-L732).
Unfortunately sorry, there is no quick mitigation at current moment. We are tracking it internally and will continue investigating/working on good solutions as soon as possible.

Tracking internally for investigation/fixing, closing this issue

@YunchuWang do you know if this issue has been fixed or is still outstanding? I'm seeing similar symptoms (logs from an async function in an Azure Function have no effect, logs on the main thread show up in Application Insights as expected)

Same question.  @YunchuWang - we closed the issue, but is it fixed?
If I create an azure function in python that spawns an async task, does not await it, and immediately returns a 202 to the caller, do I have any way to pipe that async tasks logs to application insights?

Is this updated and fixed? Or still being tracked internally and someday appear suddenly in App Insights?

sorry for late reply, the root cause of the issue is identified as event_loop.run_in_executor is executing the functions in eventloop's default threadpoolexecutor (https://docs.python.org/3/library/asyncio-eventloop.html#executing-code-in-thread-or-process-pools) which does not set the invocation id in thread local variable(_invocation_id_local). please try the following example which executes run function in worker event loop asynchronously and logs are sent to appinsights:
let me know if you have any questions after trying
import threading
import logging
import json
import asyncio
import azure.functions as func

class SomeClass(object):

    def __init__(self):
        logging.info("Object created")

    def run(self, event_body, props):
        logging.info("Event body is %s" % event_body)
        logging.info("Properties are %s" % props)

async def main(req: func.HttpRequest, context) -> func.HttpResponse:
    loop = asyncio.get_event_loop()
    result, _ = await asyncio.wait([asyncio.create_task(wrapper_function(context.invocation_id, {"hello_azure_bw_2022": "sss"}, {"ee": "ss"}))])
    logging.info("Function : event processing done")
    print("Task assigned to thread: {}".format(
        threading.current_thread().name))
    return func.HttpResponse(
        json.dumps({'result': 'ok'}),
        status_code=200,
        mimetype="application/json"
    )

async def wrapper_function(event_body, properties):
    some_obj = SomeClass()
    return some_obj.run(event_body, properties)

I am trying to apply the above work around but have encountered 2 issues:

I am using the Azure storage queue trigger rather than http, so the context object is not available. How do I get the invocation_id?
Where you call wrapper_function, you pass 3 arguments to a function that accepts 2 arguments. How does this work?

@wallflower762 updated the example, can you try again? thanks for feedback

I've just got back from holidays. I'll try and test this out in the next few weeks.


Add support for add-list-attribute to render expressions

When using expressions using the following example:
<put-list-attribute name="breadcrumbs" cascade="true" inherit="true">
  <add-list-attribute>
    <add-attribute expression="${customer.id}"/>
    <add-attribute expression="/customers/${customer.id}/info"/>
  </add-list-attribute>
  <add-list-attribute>
    <add-attribute value="Orders"/>
    <add-attribute expression="/customers/${customer.id}/orders"/>
  </add-list-attribute>
</put-list-attribute>

We first encounter an NPE when attempting to access this object. This is due to Attribute#toString returning the value null which with the container attempting to turn this into a binary array cases an NPE. This has been resolved by returning the string "null".
For the list attributes I have chosen to recursively process any lists encountered within [AbstractAttributeEvaluator#evaluate](https://tiles.apache.org/framework/apidocs/org/apache/tiles/evaluator/AbstractAttributeEvaluator.html#evaluate(org.apache.tiles.Attribute, org.apache.tiles.request.Request) to clone each attribute to a new instance with the value set to the evaluated expression.
NOTE: This solution comes with one caveat that is semi-intentional and you may choose to reject the pull request on this point. Previously a ListAttribute would be exposed to the view requiring consumers to make references to the attributes value property before accessing the index to the list.

can you take commit be97c75 out of the pull request please.
pull requests need to be atomic to just the required change at hand.
(extra stuff, like code formatting and such, is welcome in a separate PR whom's goal is explicitly such)

Will do. You can cherry-pick commit https://github.com/brettryan/tiles/commit/41759e82bbc1b4ec15878caba6bf6b659d256886 if it's easier, I think I'll have to create a separate pull request though Github doesn't give me the option when I specify a commit range. I'll see what I can' see and let you know.

Will do. You can cherry-pick commit brettryan@41759e8 if it's easier, I think I'll have to create a separate pull request though Github doesn't give me the option when I specify a commit range. I'll see what I can' see and let you know.

I'm reviewing just the individual commits for now, so i'm not waiting on you to pull out that first commit. But it'll need to be done before we accept+push any final patch.
I agree that github isn't the best for such things. I find it's easier to keep github branches as atomic as possible. I also don't have any problem with your third commit here being folded into the second, just include the NPE as well in the commit msg. (At the end of the day we'll be accepting and pushing just one patch so to work with juts one commit message makes sense already).

No prob, thanks. I've never had to rebase before, just trying to squash my commit messages and not having the best of luck, but, I'm tired and unwell :)
I'll see what I can do throughout the week.

No prob, thanks. I've never had to rebase before, just trying to squash my commit messages and not having the best of luck, but, I'm tired and unwell :)

Take it easy and no hurry.
For my own part i've just started a new job and am in the middle of an international move and the process of selling two houses, so i'm really struggling to give you the within-24hr responses that you deserve. But with a little patience I hope we'll get this across the line, it's an improvement that does make sense i believe.

On 12 Jul 2015, at 22:56, mck<EMAIL_ADDRESS>wrote:
No prob, thanks. I've never had to rebase before, just trying to squash my commit messages and not having the best of luck, but, I'm tired and unwell :)
Take it easy and no hurry.
For my own part i've just started a new job and am in the middle of an international move and the process of selling two houses, so i'm really struggling to give you the within-24hr responses that you deserve.

Hell, don't worry about me, I understand how it is. We all offer what little time we have across many projects.

But with a little patience I hope we'll get this across the line, it's an improvement that does make sense i believe.

No rush, I'm using a custom build for my purpose at the moment.

Well, I think we should at least have a test case for that. See this commit cc1195563686cc36271b03521b21e0b3e6daeb06 as an example. The tests can be run with mvn -Prun-selenium clean verify (sorry, we still depend on a servlet container)
As for the incompatibility... well it seems a bit larger than exposed here. If people are using insert-attribute like our test case, this change would break their code. Moreover, it may break the possibility of lists of cascaded definitions like:

Basically this change may drop support for any list attribute that is not a list of strings.
That being said, I like the idea of a special treatment for lists of strings, that would map them directly to List available in EL. Good job!

It appears there is no regression after all. The hack of cloning the Attribute does it, although it probably slows things down slightly.
Could you please add the test case and I'll +1 this.

@ brettryan have you had any luck with writing a testcase?
speak up if you need help with it.

Sorry, been out of action, wife's in hospital. I'll try get onto it when I can but it won't be this week unfortunately.

On 3 Aug 2015, at 18:41, mck<EMAIL_ADDRESS>wrote:
@ brettryan have you had any luck with writing a testcase?
speak up if you need help with it.
—
Reply to this email directly or view it on GitHub.


Update help : UTF-8 orthographical variants

This PR fix UTF-8 orthographical variants in help.
Checked all helpsfiles, almost these files used UTF-8
Remain orthographical variants in eval.txt
Fix below variants to UTF-8:

UTF8
utf-8

And keep specific variants in eval.txt

arguments name {utf8}
encoding name utf-8
other function need specific style

Thanks, I'll include it.  I found some more in mbyte.txt


[Feature] Matmul for CPU

Some of the most popular models provide weights in bfloat16, which unfortunately can not load on CPU because Matmul::eval_cpu only supports float32.
I know CPU support is not on priority, but it would be great if my code can run on other platforms than mac arm64 even being very slow.

maybe this can be also interesting to look at https://github.com/microsoft/BitNet

Are there plans for supporting integer tensors in tensordot/matmul?

We're not opposed to having integer support for matmul, but it's not an active priority at the moment.


Weird behavior chaining multiple commands using &&

I'm trying to port an AppVeyor script for wxFormBuilder to GitHub Actions. I've noticed that chaining a cd command with another one won't actually change the working directory. Only if I also add ls &&  at the front: https://github.com/wxFormBuilder/wxFormBuilder/pull/568/commits/c05b987ef473522dbcb5025a98abbc2aeef9bef5
I'm not really sure what's going on there, do you have an idea?

msys2do does already include bash -c, you don't need to specify it:
https://github.com/numworks/setup-msys2/blob/a59619f8cc2c4c08e51bb0d8f2dc41d8e2dae095/index.js#L37-L39
So, your commands are expanded to:
%~dp0\\msys64\\usr\\bin\\bash.exe -ilc "cd $OLDPWD && bash -c cd build/3.0/gmake && make config=release"

%~dp0\\msys64\\usr\\bin\\bash.exe -ilc "cd $OLDPWD && bash -c ls && cd build/3.0/gmake && make config=release"

As you see, bash -c cd build/3.0/gmake and bash -c ls are executed in children shells and have no effect on the commands before or after.

I see. How do I chain those commands using && then?

msys2do "cd build/3.0/gmake && make config=release" should work.

I thought so, too, but make is executed in the wrong directory then: https://github.com/wxFormBuilder/wxFormBuilder/runs/327916925
Maybe some extra quoting is required?

I think that there's something else going on there. I just tried:
    - run: msys2do "pwd && cd .github && pwd"
    - run: msys2do "pwd; cd .github; pwd"
    - run: |
        cd .github
        msys2do pwd

https://github.com/1138-4EB/setup-msys2/runs/327926283#step:5:14
https://github.com/1138-4EB/setup-msys2/runs/327926283#step:6:14
https://github.com/1138-4EB/setup-msys2/runs/327926283#step:7:15

So, either the first or the last syntaxes work. The second one does only execute the first pwd, which is expected.

Could path-type: inherit be to blame?

I suggest to replace make config=release with pwd && ls -la, just to be sure that you are where you expect to be and that a Makefile exists.

Could path-type: inherit be to blame?

Note that neither path-type nor update are supported in the current v1 tag. Those features were contributed to master recently, and the release has not been updated (see https://github.com/numworks/setup-msys2/pull/14#issuecomment-559025007). I wonder why it did not complain when you used options that are not defined.

I'm in the base directory as if cd build/3.0/gmake wasn't executed: https://github.com/wxFormBuilder/wxFormBuilder/runs/327939048

So, the issue seems to be related to the slashes? https://github.com/1138-4EB/wxFormBuilder/runs/327961332#step:12:11

Which slashes do you mean? I think that your example works because of the pwd &&  in the front.

Yes, you are correct. It is currently not supported to use &&. Some specific examples do work, but I believe that those are false positives. For your specific use case, I suggest an approach such as the following:

https://github.com/1138-4EB/wxFormBuilder/blob/master/.github/build.sh
https://github.com/1138-4EB/wxFormBuilder/blob/master/.github/workflows/windows.yml
https://github.com/1138-4EB/wxFormBuilder/commit/9f4d132edd22d28aa6c0895a7f48a21e61bc0411/checks?check_suite_id=336612832#step:6:36

Thank you so much for the support :) I'm using the build script approach now.

 I hope that actions/toolkit#232 will eventually get some attention from engineers at GitHub, so that we can use #21. Otherwise (if args need to be processed twice, once by a cmd file and then in bash), we are likely to hit multiple issues such as this one.


Validation Views Update

Validation Module View and Work View updated to common format.  Also updates Travis configuration following Python 2 -> Python 3 and PyGTK -> PyGObject upgrades.

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

In the file src/ramstk/gui/gtk/workviews/Validation.py:

Line 768:80: E501 line too long (80 > 79 characters)
Line 793:80: E501 line too long (81 > 79 characters)
Line 1032:9: E265 block comment should start with '# '


Retrying for too long while sending build scan

We have some developers for which sending build scan is hanging for too long.
In our case, sending build scan should have very short timeout (~20 sec) and no retries. After that, sending build scan should be just skipped.
Expected Behavior
Do not retry sending build scan, and shorter timeouts. Ideally - allow to configure both (number of retries and timeout in second).
Current Behavior
3 retries and very long timeout.
Context
Because of that gradle command hands for too long.
Publishing build scan...
Publishing build scan failed due to network error 'java.io.IOException: Error writing request body to server' (2 retries remaining)...

Publishing build scan failed due to network error 'java.io.IOException: Error writing request body to server' (1 retry remaining)...
A network error occurred.
The hostname '________________________' could not be resolved.
Please check that you have specified the correct server address.
You may also receive this error if you are offline from the network.
If you require assistance with this problem, please report it to your Gradle Enterprise administrator and include the following information via copy/paste.
----------
Gradle version: 5.6.4
Plugin version: 3.1
Request URL: _____________________________________
Request ID: _________________________________
Exception: java.net.UnknownHostException: ___________________________
----------

Steps to Reproduce
Introduce domain resolving problem.

One more example:
BUILD SUCCESSFUL in 26s
329 actionable tasks: 329 up-to-date
Publishing build scan...
<=============> 100% EXECUTING [1m 48s]
> IDLE
> IDLE
> IDLE
> IDLE

So sending took here >1 min and still hanging. I saw cases 5min+.

Hi @mwos-sl
This one got lost in the stream when it came in. Since version 3.3 (May 2020), the upload now happens in the background and the initial request has a short timeout of 3s-connect/5s-read.


I get an error when execute the compiled file

I compliled the demo in the documentation and got an executable file named 'fresh-demo'
deno compile main.ts

but when I execute this file, I got an error:
./fresh-demo
error: Relative import path "$fresh/server.ts" not prefixed with / or ./ or ../ from "file:///xxxx/fresh-demo/main.ts"

Did I miss something, or fresh haven't support this feature yet?
deno compile command doc

Fresh does not work with deno compile.

OK，thanks for replying. It's better to mention that in the documentation


[Spanish] First character should be Uppercase

Title says it all, comments and suggestions are welcome 😉

@iksaku if it's a title/sentence beginning, it's obviously capitalized but then you should do it manually with ucfirst for each of those specific cases. Google translate also give lowercase:
https://translate.google.com/#fr/es/Rendez-vous en avril
When it does uppercase for english for example:
https://translate.google.com/#fr/en/Rendez-vous en avril
So I guess it's like in French: lowercase.


NTR - Queuosine nucleosidase activity

Please provide as much information as you can:

Suggested term label:

Queuosine nucleosidase activity

Definition (free text)

A queuosine nucleosidase is a purine nucleosidase that cleaves the N-glycosidic bond of queuosine to yield the nucleobase queuine and D-ribose.

substance metabolic process: The chemical reactions and pathways involving substance, [description of substance].

purine ribonucleoside catabolic process

Reference, in format PMID:#######

PMID:35940128
Queuosine salvage in fission yeast by Qng1-mediated hydrolysis to queuine.
Patel BI, Heiss M, Samel-Pommerencke A, Carell T, Ehrenhofer-Murray AE

Gene product name and ID to be annotated to this term

https://www.pombase.org/gene/SPAC589.05c

Parent term(s)

GO:0008477 | purine nucleosidase activity

Any other information

requested by Ann Ehrenhofer-Murray

FOR ONTOLOGY EDITOR

Logical definition

substance metabolic process:
'metabolic process'
and ('has primary input or output' some [substance]

substance biosynthetic process:
'biosynthetic process'
and ('has primary output' some [substance].

substance catabolic process:
'catabolic process'
and ('has primary output' some [substance]

Standard synonyms

substance metabolic process:

EXACT: substance metabolism

substance biosynthetic process:

EXACT: substance anabolism

EXACT: substance biosynthesis

EXACT: substance formation

EXACT: substance synthesis

substance catabolic process:

EXACT: substance breakdown

EXACT: substance catabolism

EXACT: substance degradation

synonym , exact
Queuosine Nucleoside Glycosylase
tRNA Queuosine Nucleoside Glycosylase

A queuosine nucleosidase is a purine nucleosidase that cleaves the N-glycosidic bond of queuosine to yield the nucleobase queuine and D-ribose.
Catalysis of the reaction: a N-D-queuosine + H2O = queuine + D-ribose.

+[Term]
+id: GO:0106432
+name: queuosine nucleosidase activity
+namespace: molecular_function
+def: "Catalysis of the reaction: a N-D-queuosine + H2O = queuine + D-ribose." [PMID:35940128]
+synonym: "queuosine nucleoside glycosylase" EXACT []
+synonym: "tRNA queuosine nucleoside glycosylase" EXACT []
+is_a: GO:0008477 ! purine nucleosidase activity
+property_value: creation_date 2022-08-17T17:39:10Z xsd:dateTime
+created_by: hjd

wow speedy, thanks Harold!

Not yet val; having issues with the commit (new token). hope I dont' have to redo ; I'll let you know when

Ok this term is good to GO


ligatures/pragmata: Update ligature table for pragmata 0.829

Fixes #5358
Updating ligature table with new unicode codes for ligatures. This is drawn from https://github.com/lumiknit/emacs-pragmatapro-ligatures/blob/master/pragmatapro-lig.el.

This does not work for me. I just replaced original file with file from this PR and ligatures still broken but in a different way.

Hey, thanks for trying this out.
Just to check, do you have PragmataPro 0.829 ? Can you check the sha256sum of the zip you downloaded from fsd.it just to be sure that we have the same version ?
~ sha256sum PragmataPro0.829.zip
553c490067c2bc655d517c7648a099304cc84b580b703bbea1d65924cff33461  PragmataPro0.829.zip

@Tomahna you are right I did have wrong version for some reason. I tested archive and it's sha didn't match. I download it again and it shown correct sha now:
❯ sha2 -256 ~/Downloads/PragmataPro0.829
SHA-256 (PragmataPro0.829) = e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855

❯ sha2 -256 ~/Downloads/PragmataPro0.829\(1\).zip
SHA-256 (PragmataPro0.829(1).zip) = 553c490067c2bc655d517c7648a099304cc84b580b703bbea1d65924cff33461

I reinstalled font and restarted emacs, but some(most) ligatures are not correct anyway.

C-u C-x = shows me that wrong ligatures are from all-the-icons font. Not sure why it is messing with my ligatures. I thought it should be used only for icons when specifically requested.

Not sure it is linked to your issue. But I have indeed messed up ligatures when I go to doom-big-font-mode (even while having pragmata pro defined as doom-big-font).

Thanks for your help!


Chrome 91: position: sticky on thead & tr fixed 🎉

The issue from note 4:

Status: Fixed (was: Started)
I believe with the above patch and tables-ng being enabled [1] this issue is now resolved.
[1] https://bugs.chromium.org/p/chromium/issues/detail?id=958381#c248

https://bugs.chromium.org/p/chromium/issues/detail?id=958381#c248

[TablesNG] turn to stable
Goal is to ship in M91

The codepen from https://tests.caniuse.com/css-sticky is already green in Chrome 91 (unlike 89):

And also seems to behave like Firefox (86.0.1), so not exactly sure what was missing until today, but at the latest tomorrows Chrome Canary appears to have it all 🎉

Awesome, thanks!


Fails with non-default setting of redis

Hello,
I ran redis on port number 9000, and tried to use redlock-py then it failed. It reproduces with the following code:
import redlock

rl = redlock.Redlock({"host": "localhost", "port": 9000})
lock = rl.lock("test", 100)
rl.unlock(lock)

and the following is the error message I get:
ERROR:root:Error unlocking resource test in server StrictRedis<ConnectionPool<Connection<host=None,port=6379,db=0>>>
Traceback (most recent call last):
  File "/home/kato/.pyenv/versions/3.5.1/lib/python3.5/site-packages/redis/connection.py", line 439, in connect
    sock = self._connect()
  File "/home/kato/.pyenv/versions/3.5.1/lib/python3.5/site-packages/redis/connection.py", line 494, in _connect
    raise err
  File "/home/kato/.pyenv/versions/3.5.1/lib/python3.5/site-packages/redis/connection.py", line 482, in _connect
    sock.connect(socket_address)
ConnectionRefusedError: [Errno 111] Connection refused

It looks it is looking at the default port when it tries to lock.

I solved it by myself looking at the source code. The argument should be specified as list of dict, not a dict. Changing the third line with
rl = redlock.Redlock([{"host": "localhost", "port": 9000}])

then it works.


Disable autocomplete-light

Hello!
Is there approach for disabling element?
For example: i need to disable auto-complete field before something event.
How i can do it?
Help me please, i broke my brain.

Did you try this ?
https://stackoverflow.com/questions/17680946/select2-will-not-disable-or-clear

Closing for now. If this didn't resolve your issue, feel free to re-open.


Are there any accessibility considerations we should list for Breadcrumbs?

Relates to https://github.com/github/primer/issues/261
The description for the component in PVC is barebones right now but will be consolidated to match Primer CSS Breadcrumbs and Primer React Breadcrumbs.
Aside from the general description, are there any accessibility considerations we should note specifically for PVC usage?

Should we mention that we automatically append an aria-label and aria-current?
What sort of information is helpful for consumers to know about?

cc: @alliethu

I don't think there's anything specific the consumer needs to handle. If the component is already taking care of accessibility attributes, is it something we should note anyways, or is it "expected"? 🤔

As long as we are covering the Interface Guidelines in the design hub, I don't think there's a need for more info in the PVC docs.

I don't think there's anything specific the consumer needs to handle. If the component is already taking care of accessibility attributes, is it something we should note anyways, or is it "expected"?

@khiga8 good question.  If all of our docs (PVC, PRC, PCSS, IG) were consolidated into one place, I'd say there wouldn't be a need to mention the attributes. However, because they aren't, I feel it's worth mentioning the ARIA usage from an education/awareness standpoint to better explain what's going on under the hood.

because they aren't, I feel it's worth mentioning the ARIA usage from an education/awareness standpoint to better explain what's going on under the hood.

@alliethu Good point! I'll do that! :-)


update README.MD

Since we moved to android project structure build steps are bit different.

Solves problem like #64


🛂 SPIKE: Branch Protection Rules

The purpose of this spike is to establish which Branch Protection rules we would like to be in place.
If changes are desired:

[ ] try and figure out a way to manage the repos en masse
[ ] push these out to all of our repos
[ ] create an epic for this with a task for each repo
[ ] have everyone in the team setup to be able to push (i.e. if we desire pushes to be signed everyone will have to setup their GPG key in GitHub.
[ ] document the above

Cloud Platform Information:
Hi there
The default branch protection setting called administrators require review is not enabled for this repository
See repository settings/Branches/Branch protection rules
Either add a new Branch protection rule or edit the existing branch protection rule and select the Require a pull request before merging option
See the repository standards: https://github.com/ministryofjustice/github-repository-standards
See the report: https://operations-engineering-reports.cloud-platform.service.justice.gov.uk/github_repositories
Please contact Operations Engineering on Slack #ask-operations-engineering, if you need any assistance

Agreed max one hour timebox.


Problem with string array in influx auth create

This came in as a docs issue, but it looks like a bug to me!
Looks like this should in fact be a list of strings: https://github.com/influxdata/influx-cli/blob/9076a65d1e429fd3a84faeea5efcbf9ff9f2c2b7/cmd/influx/auth.go#L155

The InfluxData cloud CLI , token create: https://docs.influxdata.com/influxdb/cloud/reference/cli/influx/auth/create/
specifies the arguments --read-bucket and --write-bucket as string arrays. The example shows them as string values. Further, testing shows that only the first element in an array would be used, so it is a string value, not a string array.

This isn't a bug. The flag is working as designed, but , in the documentation, we may need to link the parameter types in the flag tables to the type descriptions provided on the influx auth create command.


Decide on approach: different versions for different language str 1 version with different skins?

closed by popular demand :)


Cluster credentials: duplicate names possible but not working

Internal reference: SURE-6920
Reported in 2.7.5
Issue description:
It is possible in the Rancher GUI to create a cluster credential with an identical name to an already existing one. Then it is listed twice in the selection menu. It can even be selected but it seems never to be used.
Use case:
The user wanted to replace a cluster credential for a Harvester cluster which was not working anymore (reason unclear and not subject to this case).
They created a second one with the same name and selected it to be the new credential, but cluster access was still not working. Obviously, the old one was still used.
The only way to replace the old cloud credential was to create a new one with a different name and assign it to the existing cluster.
This should either not be possible to create or it should work with both credentials.
Business impact:
There is a UI bug where someone can create a cluster credential with the name of an already existing credential. It seems the older one is then used anyway. There should be a validation that prevents re-use of the same name or both credentials should work when selected.
Troubleshooting steps:
On a cluster with nonworking cloud credentials, a new one has been created with the same name. However, the new one was never used. It looks like the name is used for finding the object and the first one found (in this case older) is always used.
Repro steps:
In Rancher UI create a cloud credential with the same name as the existing one
Select a new credential when starting the cluster
Workaround:
None
Actual behavior:
It is possible to create a cloud credential with the name of the existing one. But then once selected during cluster creation it is not used
Expected behavior:
It should either not be possible to create a cloud credential with the name of an existing credential or it should be possible the one that was selected.

If the API allows it (i.e. someone creates creds externally with matching names), should we instead of blocking it, make it obvious that the creds are different (for example change display name to include id)?

Had a quick look and was not able to easily reproduce this - at least for RKE2, seems like we are correctly using the credential ID and not the name.

This may be Harvester specific, or at least that area would provide the easiest way to understand what's going on.

It's worth noting that we cannot validate the credential name is truly unique. It's optional and also the user may not be able to see all credentials. This has been taken into account in the solution in the associated PR

I did a manual pass to understand the coverage and it looks good tome. cc @yonasberhe23

Thank you @izaac! moving to Done


Possible to add custom attributes in a PKCS10 csr?

Hello,
Is it possible to add a custom attribute named "CertificateTemplate" in a csr before signing?
It would be so that a Microsoft PKI would agree to create a certificate..
If it is possible, could you show me how?
Thanks in advance for your answer,

You can append any attribute you want. Here is source code of example. Here is live example.

Yes,
But but the value of the field 'type' has to be an OID right?
You can't just pass the 'name' of the attribute as a parameter.
Am I wrong?

Yes, CSR standard (PKCS#10) and PKIjs supports OIDs only. You must find neccessary OID for your specific attribute, then find which type has a value for this attribute, then make PKIjs attribute and then append it to PKIjs CSR before signing.

Thank you for your answer,
Sorry, I'm pretty new to this, so I don't fully understand..
Let's say I have a custom OID '<IP_ADDRESS>.1.311.21.7'. (Certificate template)
How do I find which type has a value for this attribute?

Find this info in that document where you found this OID :) Seems that this OID is not "well-known" and probably it has a "private formatiing".

OK, thanks for all the information. :)

Sorry,
Last question, is it possible to set an extension for a csr?

Yes, everything is possible. In PKCS#10 PKIjs examples there are no extensions, but in fact you can use code from other example. There you will find how to append any extensions.

Thanks again.

@YuryStrozhevsky added extensions to the sample for p10 - https://pkijs.org/examples/PKCS10_complex_example.html

Yes,
Thank you, Yury already notified me.
Brice Nandi
*Deployment Engineer @ *

revevol*
Revevol France SAS
mobile: +33 <IP_ADDRESS>.01

2015-05-18 21:25 GMT+02:00 Ryan Hurst<EMAIL_ADDRESS>
@YuryStrozhevsky https://github.com/YuryStrozhevsky added extensions to
the sample for p10 -
https://pkijs.org/examples/PKCS10_complex_example.html
—
Reply to this email directly or view it on GitHub
https://github.com/GlobalSign/PKI.js/issues/35#issuecomment-103182006.


Automatic throttling to conform to Twitch rate limits

Add a mechanism to enable automatic throttling in case the message rate exceeds the limits imposed by Twitch. Should support known and verified bots.

What is the expected response if a connection is getting rate limited? At fast join rates with a lot of channels (testing with about 2k), it's pretty common for me to get a fair bit of these errors:
[2021-01-17T01:53:49Z INFO  twitch_irc::connection::event_loop] Closing connection, cause: Error received from incoming stream of messages: WebSocket protocol error: Connection reset without closing handshake

Is this indicative of rate limits, and twitch forcibly closing the websocket connection in response, or simply the nature of opening that many websockets at once? My guess would be the former, considering it should only be like 21 websocket connections.
The described twitch rate limits are "20 join attempts per 10 seconds per user", so the default join rate is obviously far faster than that. That said, even though there's an elevated rate of connection resets, it seems like the client climbs up to peak message throughput far faster than if I slow down the join rate settings.
I'm using the anonymous connection, and my current join rate settings (and the default ones I think):
max_channels_per_connection: 90,
connection_rate_limiter: Arc::new(Semaphore::new(1)),
new_connection_every: Duration::from_secs(2),

What is the expected response if a connection is getting rate limited? At fast join rates with a lot of channels (testing with about 2k), it's pretty common for me to get a fair bit of these errors:
[2021-01-17T01:53:49Z INFO  twitch_irc::connection::event_loop] Closing connection, cause: Error received from incoming stream of messages: WebSocket protocol error: Connection reset without closing handshake

Is this indicative of rate limits, and twitch forcibly closing the websocket connection in response, or simply the nature of opening that many websockets at once? My guess would be the former, considering it should only be like 21 websocket connections.
The described twitch rate limits are "20 join attempts per 10 seconds per user", so the default join rate is obviously far faster than that. That said, even though there's an elevated rate of connection resets, it seems like the client climbs up to peak message throughput far faster than if I slow down the join rate settings.
I'm using the anonymous connection, and my current join rate settings (and the default ones I think):
max_channels_per_connection: 90,
connection_rate_limiter: Arc::new(Semaphore::new(1)),
new_connection_every: Duration::from_secs(2),

Sadly the whole rate-limiting on Twitch's end is not a very exact science, and Twitch has been very bad at documenting what's actually going on. So the fact that opening connections is rate-limited in my library is based entirely on my own empiric evidence. (Opening tons of connections at once means they all time out, I guess it's some kind of firewall, not even an application-level rate limit). The limit of 2 seconds waiting between connections is also just a "has worked well for me" value.
I personally run the recent-messages2 service, on which I use TLS connections (not WebSocket), my server is in Germany, and I'm able to connect and hold 720 parallel connections open - with the exact same settings that you posted above. (Joined to roughly 65 thousand channels).
Which is why I'm also kind of skeptical about the documented JOIN limits. My bot works fine even though I definitely go over the limit. But perhaps that's because the connections are anonymous. Yet again something that's not documented and means more guesswork for me implementing the library.
However I'm very open to more experimentation to find out whether WebSocket connection opening works different and perhaps needs different rate limit parameters or a different rate limiting strategy altogether. Could you do me a favour and try out some other values for the three parameters, (especially lowering max_channels_per_connection to the documented limit of 20, just to find out whether you're hitting that rate limit, and then also try increasing the new_connection_every duration to several seconds)

Sadly the whole rate-limiting on Twitch's end is not a very exact science, and Twitch has been very bad at documenting what's actually going on. So the fact that opening connections is rate-limited in my library is based entirely on my own empiric evidence. (Opening tons of connections at once means they all time out, I guess it's some kind of firewall, not even an application-level rate limit). The limit of 2 seconds waiting between connections is also just a "has worked well for me" value.
I personally run the recent-messages2 service, on which I use TLS connections (not WebSocket), my server is in Germany, and I'm able to connect and hold 720 parallel connections open - with the exact same settings that you posted above. (Joined to roughly 65 thousand channels).
Which is why I'm also kind of skeptical about the documented JOIN limits. My bot works fine even though I definitely go over the limit. But perhaps that's because the connections are anonymous. Yet again something that's not documented and means more guesswork for me implementing the library.
However I'm very open to more experimentation to find out whether WebSocket connection opening works different and perhaps needs different rate limit parameters or a different rate limiting strategy altogether. Could you do me a favour and try out some other values for the three parameters, (especially lowering max_channels_per_connection to the documented limit of 20, just to find out whether you're hitting that rate limit, and then also try increasing the new_connection_every duration to several seconds)

Ah, sorry about posting in the wrong issue, since there's another one for the JOIN command, must've just clicked the wrong one.
It seems that after doing multiple runs on the same settings, significant amounts of timeouts don't actually happen that often, so the default settings might work fine. I remember attempting to join 50,000 channels and after ~30 seconds a massive cascade of failures occurred, but notably this was using TLS rather than the websocket. It seems right now that for my current amount of channels the connection failures are rather limited to the point that it isn't a big issue. I am interested to stress test further though, so I'll check with different settings at some point and report back with my findings.

Ah, sorry about posting in the wrong issue, since there's another one for the JOIN command, must've just clicked the wrong one.
It seems that after doing multiple runs on the same settings, significant amounts of timeouts don't actually happen that often, so the default settings might work fine. I remember attempting to join 50,000 channels and after ~30 seconds a massive cascade of failures occurred, but notably this was using TLS rather than the websocket. It seems right now that for my current amount of channels the connection failures are rather limited to the point that it isn't a big issue. I am interested to stress test further though, so I'll check with different settings at some point and report back with my findings.

Please do report back. I'm definitely trying to find out possible improvements to the rate limiting/connection strategy.

Please do report back. I'm definitely trying to find out possible improvements to the rate limiting/connection strategy.

I added rate limiting to my bot with the governor crate, just for the global limits. I've got it set to 200 messages per minute right now, but I could take a crack at implementing it in the library if you think it'd be a good approach; I'd take a better approach with the governor (instead of Quota::per_minute(nonzero!(200u32) I'd build a quota manually to replenish one cell per 300ms for non-verified bots and one per 4000us for verified bots, with maximum bursts of 100 and 7500 messages respectively) and I'd put it behind a feature flag so only people who actually care would need to have it enabled. There's a convenient .until_ready() future that would be my initial approach to using it here, but it also provides a .check() function if returning an Error would be preferable to waiting for an open cell.
My main concern would be addressing the per-channel rate limits for verified bots; I don't think it's as serious of a concern as the global rate limits, but I'd prefer to address both if possible. The governor crate includes support for keyed rate limiters, and I imagine that would be the way to go: one direct rate limiter for the global limit, then one keyed rate limiter for all the channels.
Shouldn't take me a ton of work to get something working, though I'd like to make sure this sounds like a good approach to you first before I go poking around your code and make a PR.


Selecting large form-data file crashes app

I have a 250 MB IPA file -- selecting it with the file picker consistently crashes the packaged app.

having large sized response like ( audio/image ~ 2mb) also crashes the app.

just another memory related issue

@senjaliya what version of Postman are you using?

Version 2.0.25 (packaged)
OS: mac
Architecture: x86-64
Native client architecture: x86-64

Closing this as a duplicate of #555. Will be tracked there.


Fixing a typo in README

Short description
I fixed a typo of Xcode in README.
I read the document to work with social-symbols in my project, and I found the typo.
I'm a big fan of SF Symbols and thank you for this amazing package 🚀
Reference
https://developer.apple.com/documentation/xcode/

Thanks 🙏🏼