Increase code coverage of libunwind tests

Per VVDN investigation, there are many tests from the libunwind repo itself that we cannot get to run in the enclave environment which we would like to run. Specifically the back trace and concurrency tests at least seem pretty fundamental.
A summary of which tests are unsupported and the reasons for that:

Tests_Name
Test_Details
Supported/Unsupported
Comments

Gtest-bt&Ltest-bt
The objective of this program was to perform normal backtrace from an invoked function, backtrace from a handler invoked from signal generation and backtrace from a handler with an alternate stack that is allocated manually using the "sigalstack" structure and using the "SA_ONSTACK" flag for the sa_flags variable. An explicit backtrace using "unw_step" and "unw_backtrace" are done and both traces back through the same set of instruction pointers
Unsupported
undefined reference to  'kill', 'sigaltstack', 'sigaction', 'signal'.

Gtest-concurrent&Ltest-concurrent
This test verifies that multi-threaded concurrent unwinding is working as expected
Unsupported
undefined reference to 'signal', 'pthread_kill', 'pthread_attr_init', 'pthread_attr_setstacksize'.

Gtest-dyn1&Ltest-dyn1
This test aims to perform dynamic code generation by function cloning. It dynamically creates a function and performs recursive calls to that function which eventually leads to a segmentation fault that generates the "SIGSEGV" signal. Control will then switch to the signal handler and unwinding is done
Unsupported
undefined reference to 'mprotect', `signal'

Gtest-exc&Ltest-exc
These tests illustrates the basics of using unwind interface for exception handling
Supported
-

Gtest-init&Ltest-init
This program tests unwinding from a function invoked through a classes constructor and from the handle of an "atexit" function.Successfully reached beginning of unwind-chain
Supported
-

Gtest-resume-sig&Ltest-resume-sig
This test is used to verify that unw_resume() sucessfully restores the signal mask at proper time.In this test a signal(SIGUSER1) was generated from the main function, and from the handler function, another signal (SIGUSER2) was generated and kept pending. And on invoking unw_resume () signal mask was sucessfully restored and corresponding signal was handled properly
Unsupported
undefined reference to 'sigemptyset', 'sigaddset', 'sigprocmask', 'signal', 'kill', 'sigaction'.

Gtest-resume-sig-rt&Ltest-resume-sig-rt
The purpose of this test is to invoke different code paths in libunwind (on some architectures), that are executed when the SA_SIGINFO sigaction() flag is used. In this test it verify that unw_resume() sucessfully restores the signal mask at proper time by setting the flag TEST_WITH_SIGINFO = 1
Unsupported
undefined reference to 'sigemptyset', 'sigaddset', 'sigprocmask', 'signal', 'kill', 'sigaction'.

Gtest-trace&Ltest-trace
This program does stack trace using normal trace using unw_step and backtrace using unw_backtrace on a normal function, signal handler and signal handler with an alternate stack similar to Gtest-bt.c
Unsupported
undefined reference to 'kill', 'sigaltstack', 'sigaction', 'signal'.

Ltest-nocalloc
Create (and leak) 100 thread keys, then call backtrace() and check that it doesn't call malloc()/calloc(). The do_backtrace call right after 100 thread key creation makes a call to calloc but error count variable isforced to zero right before the second do_backtrace call. The second do_backtrace call on the other hand does not invoke either malloc/calloc. On commenting the TSD part, the first do_backtrace also do not call the calloc/malloc function
Unsupported
undefined reference to `dlsym'.

Ltest-nomalloc
Program tests whether normal backtrace call makes a call to malloc(). The corresponding backtrace call does not invoke malloc
Unsupported
undefined reference to `dlsym'.

Lrs-race
In this test "unw_backtrace" is done from on 129 different functions (foo_n()) call from two different threads under all the three caching policies. The "unw_backtrace" called from the final function checks whether all functions traces back the same number of levels.
Unsupported
test calls pthread_create() and gets aborted

Ltest-varargs
This program checks whether functions which are "NOINLINE" are not inlined by the compiler at a higher optimisation level. The check is done using the number of addresses given by "unw_backtrace" if the compiler had inlined everything, the check would have broken
Supported
-

run-coredump-unwind
The test itself is a shell script, which runs the program. The script then tries to determine the virtual addresses based on output of 'ldd' and 'readelf' and then runs test-coredump-unwind to check whether the stack trace obtained from the dump roughly corresponds to what it should look like
Unsupported
execinfo.h is not supportedundefined reference to 'syslog', 'write', 'dprintf', 'backtrace_symbols_fd', 'sigaction_UCD_accessors', '_UCD_create', '_UCD_add_backing_file_at_vaddr', '_UCD_destroy'

run-check-namespace
This test will check whether some specific symbols are there in the generated libraries or not
Unsupported

run-ptrace-mapper
This test does tracing of a child process. The child process is the program "mapper.c". The parent process is signalled from the child process. On reception of signal "SIGUSER1", the parent process starts backtrace at each instruction of the child process. On reception of signal "SIGUSER2", the parent process stops the backtrace
Unsupported
depends on test-ptrace

run-ptrace-misc
This is similar to run-test-ptrace-mapper only difference being, the child process does regular function calls instead of "mmap" in the case of "mapper.c"
Unsupported
depends on test-ptrace

test-async-sig
This test checks whether basic unwinding truly is async-signal safe. The function do_backtrace,that is unwinding was worked properly when called asynchronously
Unsupported
undefined reference to 'setitimer', 'sigaction'.

test-flush-cache
This test performs backtrace before and after all information cached on behalf of address space is flushed. Backtrace works identical on both cases
Supported
-

test-init-remote
This test checks unw_init_remote() can be used in instead of unw_init_local(). The unw_init_remote() routine initializes the unwind cursor pointed to by "c" for unwinding in the address space identified by "as". The "as" argument can either be set to unw_local_addr_space (local address space) or to an arbitrary address space created with unw_create_addr_space(). The unw_init_local() routine can be used only for unwinding in the address space of the current process.It was verified that by using unw_init_remote() in lieu of unw_init_local() function "do_backtrace" will work identically
Supported
-

test-mem
In this test, unwinding is tested by setting soft limit as 0 and hard limit as infinite for the maximum size of the process's virtual memory and process's data segment
Supported
-

test-proc-info
This test program checks whether proc_info lookup failures are cached. They must NOT be cached because it could otherwise turn temporary failures into permanent ones
Supported
-

test-ptrace
This program tests backtrace at each syscall of a child process. A child process is spawned and it is traced by the parent process. The child process spawned is "/bin/ls". Each syscall of the child process is backtraced. The backtracing stops when the child process terminates.
Unsupported
undefined reference to  'kill',  'fork', 'open', 'dup2', 'ptrace', 'execve', 'wait4'.

test-setjmp
This test tests setjmp(), longjmp(), sigsetjmp(), siglongjmp() and it also checks whether:*  _setjmp() changes the signal mask* sigsetjmp(jbuf, 1) preserve signal mask with and without sighandler- sigsetjmp(jbuf,0) does not preserve signal mask with and without sighandler
Unsupported
undefined reference to 'sigsetjmp', 'sigaddset', 'sigprocmask', 'siglongjmp' 'sigemptyset', 'kill', 'sigaction'.

test-static-link
The purpose of this program is simply to link in all libunwind-API functions both in their local-only and generic variants and to make sure that the final result can be linked statically
Supported
-

test-strerror
In this test the function the unw_strerror() routine maps the (negative) err_code to a corresponding text message and returns it
Supported
-

Related to #1068


feat: return to correct number of record page after having queried

Is your feature request related to a problem? Please describe.
I was labelling some data and was on a certain page (lets say 7). Then decide to do a quick filter to label some specific data. Then I removed the filter and I started over at page 1.
Describe the solution you'd like
I would expect to return to page 7.
Describe alternatives you've considered
N.A.
Additional context
https://user-images.githubusercontent.com/25269220/202690260-a5e41434-c5b1-4065-a9a1-cd661938464c.mp4

Pagination results are directly dependent on the filter configuration. If you change a filter, we cannot sure that records will remain on the same page.
For the use case that you propose maybe a page-level filter could be applied. Maybe discuss this with the team
cc @Amelie-V @dvsrepo

Perhaps, but I would expect to return to the same page after filtering. Maybe these kind of things can be cached per filter/query config?

This could be some kind of "Back" feature? @davidberenstein1957

I would expect the pagination to be cached against the query.


WIP: Add Anchore scanning for container in CI

I'm actually going to close this.
Realizing that this container should only really be used as a local dev container. It's not optimized in any way for prod, and therefore I also don't think we need -- at least for now -- to keep up with package versions etc. in this strict of a way.
Happy to reconsider in the future if anyone raises a need for it. 👍


issue: 2587

Please ignore this pull request. I accidentally committed changes that were not meant to be committed. Is there a way to withdraw this??
To address the issue: 2587 made existing fields private. and exposed utility functions to access the fields. Platform specific information is handled in these utility functions.

No worries. It can be closed if you didn't mean to file it.


Add rfe/rfep/cmmb

Add 3 snippets for React Functional Component and Comments big block.
Inspired by Our Best Practices for Writing React Components.

rfe -> React functional export component
rfep ->  React functional export component with propTypes
cmmb -> comment big block
modify code style: delete semi and add space before function parentheses inspired by JavaScript Standard Style
change mapDispatchToProps as object. See react-redux/mapDispatchToProps

[mapDispatchToProps(dispatch, [ownProps]): dispatchProps] (Object or Function): If an object is passed, each function inside it is assumed to be a Redux action creator. An object with the same function names, but with every action creator wrapped into a dispatch call so they may be invoked directly, will be merged into the component’s props.

Thank you for these great snippets!

@dsznajder  I fix some thing

Ok :) @YutHelloWorld Nice work :) Thanks for contribution

the stateless functional components is not in readme, maybe you need add them


Extend Main from GATK4 for simplify code

After https://github.com/broadinstitute/gatk/pull/2204 is accepted, our Main could extend the GATK one and override the methods to include only our package and few useful tools for managing VCFs and required inputs for our software.

Although some changes will be included in #52, this should wait til https://github.com/broadinstitute/gatk/pull/2246 is accepted

Closed by #52


[Regression] IOS is generating a background color not consisten with the one in pubspec.yml

This code seems to be bugged since is not generate the color correctly:
https://github.com/henriquearthur/flutter_native_splash/blob/ecfa8fefe108fd9ac57e596a80c3bcf394114e7e/lib/ios.dart#L207
Example:
#FF0024 gets translated into the following image

Thanks for the heads up!  Should be fixed now.

Thanks for the heads up!  Should be fixed now.


Where clause seems to cause issues with parameter bag

We are using the .where() method to fetch nodes like so:
existingDataQuery.match([     node('a', Attribute.getLabel()),     InstanceValue.getAttributeRelationship(false),     InstanceValue.getEmptyNode('iv') ]) .where({ 'ID(a)': attribute.id }) .return(['iv']).limit(1);
We were noticing that there were no results returned for this query (even though we should see something).  Running .toString() seems to return the correct query:
MATCH (a:Attribute)<-[:IS_VALUE_OF]-(iv:InstanceValue) WHERE ID(a) = $idA RETURN iv LIMIT 1;
When we run .interpolate() to see the parameters dropped in we see a problem:
MATCH (a:Attribute)<-[:IS_VALUE_OF]-(iv:InstanceValue) WHERE ID(a) = 9882 RETURN iv LIMIT 1;
The number value on the end of the parameter bag property name seems to be left at the end of the id parameter.  See the attached images for an example.

Hi, thanks for reporting this, I'm sorry for the delayed reply.
I've confirmed a bug in the .interpolate() function and there is a fix that will be merged shortly. However this actual issue is caused by a bigger bug with the WHERE clause which is that multiple calls to .build() will repeatedly add clauses to the parameter bag, hence why you have 3 different idA parameters in your second screenshot. I'm currently looking into this and should have a fix out at the end of the week 🤞 .

:tada: This issue has been resolved in version 3.5.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Thanks @jamesfer !  We got around it by using .raw() - but now we can use where like normal human beings 😄.  I appreciate your time into this project.  It has worked well for us!

Ok, will do.  Thanks again!


fix: ignore lockfiles when loading keys for voluntary exit

Motivation
The voluntary exit command needs to load the keystore in order to sign the message. If this command is executed on the same keystore used by the currently running validator client it will fail due to existing lockfiles. There is a workaround to pass --force flag but this seems unnecessary as there is no reason to use lockfiles for the voluntary exit command.
Description
Ignore lockfiles when loading keys for voluntary exit

:tada: This PR is included in v1.12.0 :tada:


WIP: Fail e2e tests when there are unexpected errors in the logs

Filter the logs for errors that are not expected.
e.g. Errors caused by the controller attempting to list navigator API resources before the navigator API has been registered.
Release note:
NONE

/unassign @munnerz

Closed in favour of https://github.com/jetstack/navigator/pull/100


fix einsum decomposition bug

ArrayRef was used after the underlying array changed, fixed it by creating a copy of the underlying array and make the ArrayRef point to the copy
Signed-off-by: Soren Lassen<EMAIL_ADDRESS>

Jenkins Linux amd64 Build #7733 [push] fix einsum decomposition... started at 01:00

Jenkins Linux ppc64le Build #6797 [push] fix einsum decomposition... started at 02:02

Jenkins Linux amd64 Build #7733 [push] fix einsum decomposition... passed after 1 hr 1 min

Jenkins Linux s390x Build #7749 [push] fix einsum decomposition... passed after 1 hr 22 min

Jenkins Linux ppc64le Build #6797 [push] fix einsum decomposition... passed after 1 hr 27 min


feat: add OTEL_SAMPLING_RATE env var

User can now control the trace sampling rate by means of env var.
Require assistance in setting up a test, any suggestions?

Same as https://github.com/open-telemetry/opentelemetry-js/pull/974#issuecomment-615899822

Hello, I cannot seem to get OTEL_SAMPLING_RATE to work for the BasicTracerProvider test, any suggestions?

Codecov Report

Merging #975 into master will increase coverage by 1.46%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #975      +/-   ##
==========================================
+ Coverage   95.02%   96.49%   +1.46%
==========================================
  Files         209      189      -20
  Lines        8541     7757     -784
  Branches      766      627     -139
==========================================
- Hits         8116     7485     -631
+ Misses        425      272     -153

Impacted Files
Coverage Δ

packages/opentelemetry-tracing/src/Tracer.ts
100.00% <100.00%> (ø)

...telemetry-tracing/test/BasicTracerProvider.test.ts
100.00% <100.00%> (ø)

...ckages/opentelemetry-core/src/common/NoopLogger.ts
60.00% <0.00%> (-20.00%)
:arrow_down:

...metry-core/src/trace/instrumentation/BasePlugin.ts
81.57% <0.00%> (-5.27%)
:arrow_down:

...es/opentelemetry-exporter-collector/src/version.ts

...es/opentelemetry-exporter-collector/test/helper.ts

...entelemetry-plugin-xml-http-request/src/version.ts

packages/opentelemetry-web/src/utils.ts

...ry-context-zone-peer-dep/src/ZoneContextManager.ts

...y-exporter-collector/test/browser/index-webpack.ts

... and 14 more

Not sure how https://github.com/open-telemetry/opentelemetry-js/pull/975/commits/637f982b7e7b00f05ee9e2ba8f329ba344fcfb7b wound up in this branch. I tried rebasing from master.

Anyone know why isn't CI triggering?

Closing and opening a new PR due to git complications.


[FEATURE] invaders game

Description

Screenshots
No response
Additional information
No response

@panwar8279 please assign it to me

Hi @abansal15 , I have assigned this task to you. Kindly complete your work within 10 days. Make sure you will make your game in react. Good luck!


added prithivi narayan campus

@senny789 First stage of verification passed, official domain is pncampus.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://pncampus.edu.np/governing.php

@senny789 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Fix bug with empty DataTemplate

Summary
Fixed a bug where an ItemRepeater with an empty DataTemplate as ItemTemplate would result in a crash.
Motivation
Fixes #776
Testing methodology
Added a test where we create an ItemRepeater with an empty DataTemplate and assert the size being zero. Also an ItemRepeater with empty DataTemplate was added to the test UI.

/azp run

Thank you for fixing this!

Happy to help :)

/azp run

DataTemplateSelector can also return empty DataTemplate. This PR doesn't consider that.

DataTemplateSelector can also return empty DataTemplate. This PR doesn't consider that.

Oh sorry, I did not think about that. I have looked into that problem and the situation is a quite difficult, so here everything I found out:
Problem with empty DataTemplate
The problem is, the way the ItemRepeater works internally, is that it requests item at position x and then the internal StackPanel adds the UI element. Since you want to render "nothing" this results in the problem that there is no such thing as the "nothing" element which we could return :/ .
As far as I can see, correctly ignoring items that should render nothing would end in having to rework large parts of the internal logic of the ItemRepeater. The problem is how the StackPanel used for layout gets its height and width and its elements.
Possible Solutions
We would either:

Have to return an element with zero height and width OR
Rework every piece of code that touches the element generated from template and change it to be able to deal with null. This would not only affect code inside the ItemRepeater but also inside the StackPanel as the internal StackPanel requests the items one at a time for size calculations.

Why we could fix an empty DataTemplate that is not inside TemplateSelector
When we don't use a TemplateSelector every item gets treated the same, so if the DataTemplate renders null/nothing, we can just detect that and say: No content we want to render, just skip problematic measuring and set height and width to zero.
Summary
Because of how the ItemRepeater renders, rendering a "null element" (which essentially happens when specifying DataTemplate with no content) crashes, as a lot of code relies on the element existing for measuring different values such as height and width.
So my question is, what would be the best way to handle this problem, provided that I did not miss something crucial regarding that topic.

/azp run

/azp run


I would love to have a profile, thanks

I'd like to have my profile on ghuser.io, thanks!

here you go :) https://ghuser.io/sarthak268


Powercord Support

With the BDCompat plugin, everything works except prefixes and actually sending the messages: nothing is sent. I can't find any alternatives and this is a useful plugin.

Sorry, I don't use or know anything about Powercord so I can't help you with this. Feel free to make a pull request if you can figure it out.


refactor(website, flat select): update demo layout

Proposed Changes
Added JsDoc on FlatSelect props
Updated FlatSelect demo layout
Potential Breaking Changes
None
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Grouped AND big? :laughing:

Grouped AND big? 😆

Is this expected? I wasn't sure what you meant until I played around with it 😄

Yes, the size changes when you put the group props, I'm not sure where it is used so I haven't touched it


DRILL-7405: Avoiding download of TPC-H data

Unit tests as well as Functional & Advanced Regression tests from [1] are successful.
[1] https://github.com/mapr/drill-test-framework

I disagree that we should add so large files to the project sources.
I have experimented some time ago with alternative solutions for this problem and one of the ideas was to create a new GitHub repository with these files, use JitPack for publishing archive and use maven-dependency-plugin in Drill to obtain and unpack files when the project is built.
Here is a link to the repo with files: https://github.com/vvysotskyi/tpch-parquet and commit with changes in Drill: https://github.com/vvysotskyi/drill/commit/0635133bbd22945e7648791cfd6e2d146730b219.
What do you think about this? You may use these changes if we will choose this approach.

@vvysotskyi , I understand but I am not for using personal GitHub repositories. I don't think it's the best approach for an Apache project. The files are small enough ~3 MB. I'm not sure I agree that this would constitute as large. I think what we have here would be more straightforward.

Looks like these files were packaged as jar in Drill class path as an example data for users to run some exploratory queries. I think putting these files as part of source repo should be fine.
@vvysotskyi : I think your main concern is related to the unit tests data files which are merged with the source files. I guess that was done to keep the test execution time lower otherwise ideally unit tests should use the in-memory data generator for it's use. May be we should come up with some policies which can dictate when is it fine to check in the test data file and when one should use in-memory data generator.
Also how does moving data files to a separate git repo will help here ?

In general, it is a good idea to have all the files either in the project, or via a Maven dependency. We have another set of data, the "FoodMart" files, which we pull as a jar file from a repo maintained by Julian Hyde.
So, one possibility is to create another Drill repo that just handles supplementary files. In fact, already have one: the test framework. We could add another.
But, I agree with @sohami, the files are small and simple enough that we might as well add them to Drill directly.

How are these files used within Drill? If they are used for unit tests, then we have to consider the module hierarchy. If the files are in contrib, then only modules "later" than contrib can access them. Is this a problem?

+1


concurrency issue

Hi, I'm experiencing a problem when I set up two or more workers ( processors ), only process the first job and never again take a job from the queue, leaving all others jobs in inactive queue.
If I set a single worker, working properly and all the jobs are processed.
I try to check if an error is generated, but no error is emitted, apparently this callback is never called
 client.blpop(client.getKey(self.type + ':jobs'), 0, function( err ) {
Also i made a network capturing and only see this between server and redis server
*3
$7
evalsha
$40
54b5f3c93bc373eb500cdb6e80ccca0220b89330
$1
0
:0
*3
$7
evalsha
$40
54b5f3c93bc373eb500cdb6e80ccca0220b89330
$1
0
:0
*3
$7
evalsha
$40
54b5f3c93bc373eb500cdb6e80ccca0220b89330
$1
0
:0

Any idea what could be happening?
Thx!

kue version: 0.10.4
redis_version: 2.8.12

please send me the code, so I can run it locally to debug whats happening

Hi thanks for the answer. I'm creating the queue in this way:
var queue = kue.createQueue( {
	redis : {
		host : host,
		port : port,
		auth : auth
	}
} );

Add jobs:
var job = queue.create( "populate", {
	jid : jid
} ).priority( "high" ).save();

job.on( "complete", function() {
	callback();
} ).on( "failed", function() {
	callback( new Error( "Failed" ) );
} );

And the code that run is something like this. But I'm sure that the done callback is called because i see the complete event is fired.
	queue.process( "populate", 2, function( job, done ) {
		var jid = job.data.jid;

		myLib.populate( jid, function( err ) {
			if( err ) console.log( err );
			done( err );
		} );
	} );

Thx!

Hi @behrad could you reproduce this issue? Let me know if you need more info.
Thx!

Sorry I hadn't time to test your code, will do this tomorrow and let you know...
Note: this is a very basic Kue functionality to dispatch jobs to concurrent workers and this is already working. There should be problem in your own code. You can however drop myLib.populate and test Kue separately on that.

@pepoviola did you find something? We have a similar issue here: https://github.com/Automattic/kue/issues/815


panic when loading home page and 404 error

Description
Gitea panics when loading home page in /avatars/ route (and therefore the avatar image doesn't load) and when open the home page, it shows a 404 error.
Gitea Version
1.20.4 and git main branch
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/sryze/68fb927ff8b95121597942a3fa118f3e
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Behind Apache web server as described at https://docs.gitea.com/next/administration/reverse-proxies#apache-httpd-with-a-sub-path
Database
SQLite

It seems that next is nil here: https://github.com/go-gitea/gitea/blob/08507e2760638124d75774c29ef37e692a88c02d/routers/web/base.go#L65

Found the root cause - I had a trailing / after the port in my ProxyPass directive in Apache config, and because of this all requests began with //. It would be nice if the server didn't panic because of this IMO.


Replaced alert contact us form submission with toast notification method

Closes: #1099
Summary
Replaced alert contact us form submission with toast notification method

Types of Changes
Please check the boxes that apply

[ ] Bugfix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Please check the boxes that apply

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

@ananyag309 thank you for your contribution to ML Fusion Lab.
your contribution gives a value to us.
now please do work on other projects.


Issue with k8s.io/docs/tasks/administer-cluster/dns-debugging-resolution/

This is a Bug Report

Problem:
k8s-app=coreos    isn't the label for coreos, to preserve compatibility  with kube-dns coreos didn't change the label
Proposed Solution:
change the document,  replace k8s-app=coreos with k8s-app=kube-dns
Page to Update:
https://kubernetes.io/...

Thanks for the report
/language en
#14816 should fix this

/close
Fixed via #14816


expectAssignable and expectNotAssignable not behaving correctly on arrays

Hey
I noticed that the behaviour for expectAssignable and expectNotAssignable seems wrong for arrays
Here are a few examples;
✅ Works as expected
type myCustomType = {
  value: number[]
}

expectAssignable<myCustomType>({value: [1]})      // ✅ it passes
expectNotAssignable<myCustomType>({value: [1]})   // ✅ it fails as expected

❌ Fails
type myCustomType = {
  value: 1[] // Array of only 1's
}

expectAssignable<myCustomType>({value: [1]})      // ✅ it passes
expectNotAssignable<myCustomType>({value: [1]})   // ❌ it passes when it should fail

I saw that in #190 a suggested fix would be to use as const but this doesn't really work with arrays
type myCustomType = {
  value: 1[] // Array of only 1's
}

expectAssignable<myCustomType>({value: [1]} as const)      // ❌ it fails with error
expectNotAssignable<myCustomType>({value: [1]} as const)   // ❌ it passes when it should fail

Any suggestions?

Did you try:
expectNotAssignable<myCustomType2>({ value: [1 as const] });

I tried that and that approach works, but it kind of looks a bit meeh to do something like this:
expectNotAssignable<myCustomType>({ value: [1 as const, 1 as const, 1 as const] });

For a slightly longer type, I feel there are way too many as const needed to test and it isn't intuitive that as const should be used
type myCustomType = {
  simple: 1;
  optional: 2 | null;
  list: 3[];
}

expectAssignable<myCustomType>({
  simple: 1,
  optional: 2,
  list: [3, 3, 3]
});  // ✅ it passes, but should this have 'as const' as well?

expectNotAssignable<myCustomType>({
  simple: 1 as const,
  optional: 2 as const,
  list: [3 as const, 3 as const, 3 as const]
}); // ✅ only way to make it fails as expected

expectNotAssignable<myCustomType>({
  simple: 1, // not made 'as const'
  optional: 2 as const,
  list: [3 as const, 3 as const, 3 as const]
}); // ❌ it passes when it should fail

expectNotAssignable<myCustomType>({
  simple: 1 as const,
  optional: 2,  // not made 'as const'
  list: [3 as const, 3 as const, 3 as const]
}); // ❌ it passes when it should fail

expectNotAssignable<myCustomType>({
  simple: 1,  // not made 'as const'
  optional: 2,  // not made 'as const'
  list: [3 as const, 3 as const, 3 as const]
}); // ❌ it passes when it should fail

expectNotAssignable<myCustomType>({
  // make the full object 'as const'
  simple: 1,
  optional: 2,
  list: [3, 3, 3]
} as const); // ❌ it passes when it should fail

Thoughts?

Thanks for asking. It is always interesting to talk about type testing.
Please note that I am not associated with tsd. It was used to test typings in Jest repo and I was contributing a lot of type tests. Limitations and issues of the library pushed me to create tsd-lite fork. It does not have the issue you are wrestling with, but at this moment I do not recommend migrating to tsd-lite.
It is going to be replaced by brand new type testing tool, which I will publish in about a week. It is written from scratch, the architecture is different and all that helps to solve limitations which tsd-lite inherited. For instance, it is enough to pass a CLI flag to test on several versions of TypeScript (see: https://github.com/jest-community/jest-runner-tsd/issues/32#issuecomment-1228738812).

Back to your problem. What do you mean by “slightly longer type”? In this case { value: [1 as const] } is not a type, this is an expression. tsd is using TypeScript’s APIs to  infer type of expression and compares it with the one you pass as type argument.
To check how type information is inferred, you can use hover information:

But:

That is just how TypeScript works. Use as const:

All this is simple and obvious, of course. (Might be even intuitive, but that’s rather subjective.)

The problem is that tsds  expectAssignable infers type of an expression like this:

But expectNotAssignable infers type of the same expression like this:

In a way, expectAssignable already knows what is expected. In my opinion, the behaviour of expectAssignable is broken. (By the way, it is the cause of #142 as well.) This is not a problem for tsd-lite. It was easy to fix.
Hope this explains the inconsistency of behaviour between expectAssignable and expectNotAssignable.

Gotcha, thanks for the explanation 😄
By “slightly longer type”, I meant this:
type myCustomType = {
  simple: 1;
  optional: 2 | null;
  list: 3[];
}

In the initial version where myCustomType only had a single key value, the idea to mark the array items as const wouldn't be too much of a hassle (i.e. { value: [1 as const] }), buuuut for the longer objects with more keys and at least one array, it becomes more of a hassle to declare every single thing as const since you can't just declare the full object as const as that would make the array readonly
// I'd prefer to do something like this (which doesn't work)
expectNotAssignable<myCustomType>({
  simple: 1,
  optional: 2,
  list: [3, 3, 3]
} as const);

// Instead of marking everything as const
expectNotAssignable<myCustomType>({
  simple: 1 as const,
  optional: 2 as const,
  list: [3 as const, 3 as const, 3 as const]
});

I kind of expected expectNotAssignable to behave the same way as expectAssignable since we pass the type to each of them - instead of inferring the type as number, use the type that I gave you (if my type is number use that, if it is 1 use that instead)
Looking forward to tsd-lite


Does not produce a Block on All Connected Servers & LocalHost

[ Subject of the issue ]
Description
can the version for 0.7.0 alpha1 be used for testing? i tried that version as reference material. on trying it produces no block at all the port is already enabled and my configuration is correct. but when trying to localhost it produces a block but in explorer it doesn't show any blocks at all.
polygon-edge monitor --grpc-address <IP_ADDRESS>:9632
Block is produced but in explorer it is not visible only genesis-0 block is visible.

Hi @avenbreaks
We can't provide you with support for alpha release, and please use the latest 0.6.2 for testing purposes.
For blockexplorer issues you will have to post a query under their official Github repo.


Provide ability to filter FreqMap by regex

Allow for transforming of Frequency Maps using a Regex to extract the elements of interest, leaving only those that match the regular expression.

I am a beginner. I want to start with contributing in this.

Merged into develop.


Optimize update facility seen range

Update facility seen range to optimize it
def update_facility_seen_range

likely needs a trading_interval bound on the select query

[production] update_facility_seen_range in 1s

just a bit better ...


[SPARK-26603][K8s] Update minikube backend

What changes were proposed in this pull request?

Covers latest minikube versions.
keeps the older version support

Note: While I was facing disk pressure issues locally on machine, I noticed minikube status command would report that everything was working fine even if some kube-system pods were not up. I don't think the output is 100% reliable but it is good enough for most cases.
How was this patch tested?
Run it against latest version of minikube (v0.32.0).

@felixcheung could you merge ?

merged to master


MUMMNG-2414 : remove myuw specific stuff

Now that angularjs-portal has a docs page we don't need this here.

:hocho: :+1:

linking https://github.com/UW-Madison-DoIT/angularjs-portal/pull/440


[Platform GLFW] Fix warning on windows in MSVC

There is an error with the forward declaration of the winMM functions when building on MSVC. Inconsistent DLL linkage.
This is not a critical warning, so this PR disables it.
This PR also removes the timeBeginPeriod declaration as it is no longer needed.

AFAIK timeBeginPeriod may still be necessary, as its operation changed on Windows 10, version 2004 (see Remarks section, third paragraph, of the documentation), which was on May 2020, not that long ago. There's a lot of hardware still running older versions of Windows (e.g.: automation, industry, etc).

AFAIK timeBeginPeriod may still be necessary, as its operation changed on Windows 10, version 2004 (see Remarks section, third paragraph, of the documentation), which was on May 2020, not that long ago. There's a lot of hardware still running older versions of Windows (e.g.: automation, industry, etc).

It's not called in that file, thus it does not need to be forward declared, it'll still be linked by the lib if some other internal file uses it, like GLFW itself.

I see, thanks.


align vllm hpu version to latest vllm-fork

Description
align vllm hpu version to latest vllm-fork
related GenAIComps PR https://github.com/opea-project/GenAIComps/pull/847
Issues
List the issue or RFC link this PR is working on. If there is no such link, please mark it as n/a.
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[ ] Others (enhancement, documentation, validation, etc.)

Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.

waiting for UT machine upgrade to 1.18


Create Plan for Implementation

Create a Plan for Implementation
User Story

As a user
I need an application that requests and analyzes tokens provided using OpenID
So that they can automatically be provided with an analysis of the tokens

Acceptance Criteria

A plan on how to develop a program to analyze OpenID tokens
Every developer knows the structure of the planned project
The planned structure is documented
The general project is created

The basic project-structure was set up and a basic plan for the further implementation was created.


[Feature Request]: option to turn on checkboxes at the corners of images

Hello!
I would like to request a feature that allows users to turn on checkboxes at the corner of each image as an alternative to the "add to download list" button, which currently appears when we hover over an image. I believe this would be more convenient than the existing "add to download list" option because it provides a visual indication for images that have already been checked or not been checked yet, and it is easier to click on the corner than on a button in the bottom middle. Here is an example:

Thank you!

@Aosneg, sorry for the late reply. This feature has been added in v0.32.3.

Oh, cool! Thank you!


Version Number Error

The update version number says Version <IP_ADDRESS>.4(behind) instead of  <IP_ADDRESS>.2. This causes updates notice in plugin page. Thanks

Resolved. Downloaded wrong package

No problem!  ☺


Secrets with Hyphen in Name Fail

Describe the bug
Secret names that contain - fail. Underscores and camelcase work. There is no warning in the UI when creating secrets with hyphens and the UI allows it.
To Reproduce
Steps to reproduce the behavior:

Create a secret called foo-bar
Create a simple workflow as shown below
Run the workflow

Expected behavior
The workflow either runs with secret names that contain hyphens OR secret names containing hyphens are blocked from ever being created.
[ 11:42:23.229 ]
Preparing workflow triggered by API.
[ 11:42:23.244 ]
Running state logic (step:1) -- get-details
[ 11:42:23.246 ]
Decrypting secrets.
[ 11:42:23.257 ]
Instance failed with uncatchable error: failed to evaluate jq: error in 'headers': error in 'x-token': error running jq query beginning at offset 0: function not defined: bar/0

** Workflow **
description: My first workflow
functions:
- id: get-metadata
  type: reusable
  image: direktiv/request:v1
states:
- id: get-details
  type: action
  action:
    secrets: ["foo-bar"]
    function: get-metadata
    input:
      method: "GET"
      url: "https://mysite.com/someAPIEndpoint"
      headers:
        "x-token": "jq(.secrets.foo-bar)"

Thanks for raising that issue but this is jq behavior. In this case jq sees that as foo -bar = X.
If you want to use hyphens in jq they have to be quoted. In this case jq(.secrets."foo-bar").


Detect a bare GitLab platform

It's annoying to maintain a file to know the kind of platform behind a URL.
Try to at least detect GitLab platforms, maybe using an API endpoint returning a GitLab specific version.
The file in charge of this is https://github.com/etalab/data-codes-sources-fr/blob/master/platforms.py
cc @bzg, I'm sure you would be happy to stop doing shitty commits updating the CSV

cc @bzg, I'm sure you would be happy to stop doing shitty commits updating the CSV

Yes, I would, but not a high priority, the inconvenience is really bearable.

Do you think you can fix this issue in a reasonable time?

Not sure it's important to work on this now. The CSV file for platforms has not been updated a lot: https://github.com/etalab/data-codes-sources-fr/commits/master/platforms.csv

Agreed, feel free to close this one.


Added adiabatic taper version

Added adiabatic taper functionalities to taper out the abrupt change if the tip of the strip_to_slot_SiN PCell. Now the gap between the tip of the 120nm pillar and the main waveguide is 300nm instead of 120nm and can be adjusted to other values accordingly.

Thanks!


[ISSUE] Identity fails to start because of the Connectors var

Describe the issue:
Identity fails to start because the Connectors secret has a typo, and it uses Operate key. It fails when the Operate existingSecret is used so the secret is not generated and the Keycloak client seed of the Connectors.
Also, I've found another issue where it's not possible to use existingSecret.name to refer to the actual existing secret key for Connectors. Right now, it's only possible to use existingSecret to provide the text password (it's used in the upgrade).
Actual behavior:
Expected behavior:
The var KEYCLOAK_CLIENTS_0_SECRET should always be generated to make sure Identity doesn't fail (in the future, when we migrate to a flat chart structure, we can disable the KEYCLOAK_CLIENTS_0_* vars completely when Connectors is disabled).
How to reproduce:
Use existing secret for Operate:
global:
  identity:
    auth:
      operate:
        existingSecret:
          name: custom-secrets

Logs:
When KEYCLOAK_CLIENTS_0_SECRET is empty, Identity fails with the following error:
2023-04-13 15:19:02.209 ERROR 1 --- [           main] o.s.b.SpringApplication                  : Application run failed

java.lang.IllegalStateException: Failed to execute ApplicationRunner
	at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:763) ~[spring-boot-3.0.5.jar!/:3.0.5]
	at org.springframework.boot.SpringApplication.callRunners(SpringApplication.java:750) ~[spring-boot-3.0.5.jar!/:3.0.5]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:317) ~[spring-boot-3.0.5.jar!/:3.0.5]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1304) ~[spring-boot-3.0.5.jar!/:3.0.5]
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1293) ~[spring-boot-3.0.5.jar!/:3.0.5]
	at io.camunda.identity.Application.main(Application.java:18) ~[classes!/:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[?:?]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[?:?]
	at org.springframework.boot.loader.MainMethodRunner.run(MainMethodRunner.java:49) ~[identity.jar:?]
	at org.springframework.boot.loader.Launcher.launch(Launcher.java:95) ~[identity.jar:?]
	at org.springframework.boot.loader.Launcher.launch(Launcher.java:58) ~[identity.jar:?]
	at org.springframework.boot.loader.JarLauncher.main(JarLauncher.java:65) ~[identity.jar:?]
Caused by: java.lang.NullPointerException: Cannot invoke "String.isBlank()" because the return value of "io.camunda.identity.impl.keycloak.config.record.KeycloakClient.secret()" is null
	at io.camunda.identity.impl.keycloak.initializer.service.ClientInitializationService.validateClientSecret(ClientInitializationService.java:183) ~[classes!/:?]
	at io.camunda.identity.impl.keycloak.initializer.service.ClientInitializationService.processClient(ClientInitializationService.java:63) ~[classes!/:?]
	at java.util.ArrayList.forEach(Unknown Source) ~[?:?]
	at io.camunda.identity.impl.keycloak.initializer.service.ClientInitializationService.process(ClientInitializationService.java:59) ~[classes!/:?]
	at io.camunda.identity.impl.keycloak.initializer.KeycloakClientInitializer.run(KeycloakClientInitializer.java:34) ~[classes!/:?]
	at org.springframework.boot.SpringApplication.callRunner(SpringApplication.java:760) ~[spring-boot-3.0.5.jar!/:3.0.5]
	... 13 more

Environment:

Platform: Any (tested on GKE).
Helm CLI version: v3.9.3
Chart version: 8.2.0
Custom Values: added above.

Related to:
SUPPORT-16602

Camunda Platform Helm chart v8.2.1 has been released which contains the fix for this issue.


Use fullfile for path construction.

Hi ellingsonj,
Thank you very much for this wonderful library! It worked like a charm on my Windows workstation. However, it would not run on Ubuntu. It turned out some paths were not constructed with fullfile to make sure they work for different platforms.
This pull request contains a fix for this issue. And it seems my editor also automatically removed all trailing white space in the file.
Thanks,
Yaguang

I also had a problem using the original kmz2struct on MacOS (using Matlab_R202b). I followed the changes suggested in this pull request, and now it works perfectly.

Thanks so much for the pull request!!


hwloc: 2.10.0 -> 2.11.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/hwloc/versions.
meta.description for hwloc is: Portable abstraction of hierarchical architectures for high-performance computing
meta.homepage for hwloc is: https://www.open-mpi.org/projects/hwloc/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.11.0 with grep in /nix/store/1j0pr021mlydip2gn48npz948zga8nqy-hwloc-2.11.0
found 2.11.0 in filename of file in /nix/store/1j0pr021mlydip2gn48npz948zga8nqy-hwloc-2.11.0

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

351 total rebuild path(s)

351 package rebuild(s)

First fifty rebuilds by attrpath

ants
apacheHttpdPackages.mod_tile
apacheHttpdPackages_2_4.mod_tile
bicgl
bicpl
c3d
cdo
cloudcompare
conglomerate
costa
cp2k
dbcsr
dl-poly-classic-mpi
dtcmp
ebtks
eccodes
elastix
elmerfem
elpa
entwine
ezminc
febio
fftwMpi
freecad
frigate
gdal
getdp
globalarrays
gmt
gnudatalanguage
gplates
gpt2tc
grass
gromacs
gromacsCudaMpi
gromacsDouble
gromacsDoubleMpi
gromacsMpi
gromacsPlumed
haskellPackages.haskell-mpi
haskellPackages.hgdal
haskellPackages.mpi-hs-binary
haskellPackages.mpi-hs-cereal
haskellPackages.mpi-hs-store
hdf5-mpi
highfive-mpi
hp2p
hpcg
hpl

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/955e04a283aa8c03643d4d39e03b53550fc79622#hwloc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1j0pr021mlydip2gn48npz948zga8nqy-hwloc-2.11.0
ls -la /nix/store/1j0pr021mlydip2gn48npz948zga8nqy-hwloc-2.11.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 120m and timed out

Maintainer pings
cc @fpletz @markuskowa for testing.

@GrahamcOfBorg test slurm


Load the tenants from database provider

There is a need to add a new option to load the tenants from the database
Initially lets use MSSQL server as a db provider and add the others as needed

nobody?


ZarrMerge producing empty counts while .zarr files used to merge have associated counts?

Thank you for producing this pipeline - it's quite helpful when dealing with huge sc datasets! I'm running into an issue when I try to merge two zarr objects.
I'm actually not throwing an error until I get to the mark_hvgs() step, but it's related to the ZarrMerge step not outputting cell counts by feature (i.e., the .zarr directory has no data files in test.zarr -> RNA -> counts).
Expected behavior
I would have expected that when I merge the two .zarr objects, that the DataStore object would not have "RNA assay has 0 (22032) features". All nCells are equal to 0. It's clear that the merge somehow did not summarize the counts. I played with the overwrite option and adjusting the update_key based on the vignette, but I can't seem to get the zarr files merged correctly. I am attempting to merge 115 .zarr dirs, but this seems to be problematic with only two .zarr.
The code I used is below (apologies if anything reported incorrectly, I haven't reported a ton of issues on git):
import re
import os
import scarf

a = scarf.DataStore(
         'S5464Nr1.zarr',
        nthreads = 4,
        min_features_per_cell = 1)
b = scarf.DataStore(
         'S5464Nr2.zarr',
        nthreads = 4,
        min_features_per_cell = 1)
scarf.ZarrMerge(
    zarr_path='test.zarr',
    assays=[a.RNA,b.RNA],
    names=['a','b'],
    merge_assay_name='RNA',
    overwrite=True
)
ds = scarf.DataStore(
    'test.zarr',
    nthreads = 4
)
ds

(RNA) Computing nCells and dropOuts: 100%|
1150/1150 [00:00]
(RNA) Computing nCounts: 100%|
1155/1155 [00:00]
WARNING: Minimum cell count (0) is lower than size factor multiplier (1000)
(RNA) Computing nFeatures: 100%|
1155/1155 [00:00]
(RNA) Computing RNA_percentMito: 100%|
1113/1113 [00:00]
WARNING: Percentage feature RNA_percentMito not added because not detected in any cell
(RNA) Computing RNA_percentRibo: 100%|
1113/1113 [00:00]
WARNING: Percentage feature RNA_percentRibo not added because not detected in any cell
WARNING: More than of half of the less have less than 10 features for assay: RNA. Will not remove low quality cells automatically.
DataStore has 20369 (20369) cells with 1 assays: RNA
   Cell metadata:
            'I', 'ids', 'names', 'RNA_nCounts', 'RNA_nFeatures',
            'orig_RNA_nCounts', 'orig_RNA_nFeatures', 'orig_RNA_percentMito', 'orig_RNA_percentRibo'
   RNA assay has 0 (22032) features and following metadata:
            'I', 'ids', 'names', 'dropOuts', 'nCells',

ds.mark_hvgs(
    min_cells=20,
    top_n=2000,
    min_mean=-3,
    max_mean=2,
    max_var=6
)

/opt/anaconda3/lib/python3.9/site-packages/dask/array/slicing.py:647: RuntimeWarning: divide by zero encountered in long_scalars
  maxsize = math.ceil(nbytes / (other_numel * itemsize))

OverflowError                             Traceback (most recent call last)
/var/folders/b4/14kj85s94x52tlh_ptt0vf6r0000gn/T/ipykernel_63026/3363267796.py in <module>
----> 1 ds.mark_hvgs(
      2     min_cells=20,
      3     top_n=2000,
      4     min_mean=-3,
      5     max_mean=2,

/opt/anaconda3/lib/python3.9/site-packages/scarf/datastore.py in mark_hvgs(self, from_assay, cell_key, min_cells, top_n, min_var, max_var, min_mean, max_mean, n_bins, lowess_frac, blacklist, show_plot, hvg_key_name, **plot_kwargs)
   3604                 f"of cells will be considered HVGs."
   3605             )
-> 3606         assay.mark_hvgs(
   3607             cell_key,
   3608             min_cells,

/opt/anaconda3/lib/python3.9/site-packages/scarf/assay.py in mark_hvgs(self, cell_key, min_cells, top_n, min_var, max_var, min_mean, max_mean, n_bins, lowess_frac, blacklist, hvg_key_name, show_plot, **plot_kwargs)
    938             return f"{identifier}_{x}"
    939
--> 940         self.set_feature_stats(cell_key, min_cells)
    941         identifier = self._load_stats_loc(cell_key)
    942         c_var_col = f"c_var__{n_bins}__{lowess_frac}"

/opt/anaconda3/lib/python3.9/site-packages/scarf/assay.py in set_feature_stats(self, cell_key, min_cells)
    830             return None
    831         n_cells = show_dask_progress(
--> 832             (self.normed(cell_idx, feat_idx) > 0).sum(axis=0),
    833             f"({self.name}) Computing nCells",
    834             self.nthreads,

/opt/anaconda3/lib/python3.9/site-packages/scarf/assay.py in normed(self, cell_idx, feat_idx, renormalize_subset, log_transform, **kwargs)
    792         if feat_idx is None:
    793             feat_idx = self.feats.active_index("I")
--> 794         counts = self.rawData[:, feat_idx][cell_idx, :]
    795         norm_method_cache = self.normMethod
    796         if log_transform:

/opt/anaconda3/lib/python3.9/site-packages/dask/array/core.py in __getitem__(self, index)
   1798
   1799         out = "getitem-" + tokenize(self, index2)
-> 1800         dsk, chunks = slice_array(out, self.name, self.chunks, index2, self.itemsize)
   1801
   1802         graph = HighLevelGraph.from_collections(out, dsk, dependencies=[self])

/opt/anaconda3/lib/python3.9/site-packages/dask/array/slicing.py in slice_array(out_name, in_name, blockdims, index, itemsize)
    172
    173     # Pass down to next function
--> 174     dsk_out, bd_out = slice_with_newaxes(out_name, in_name, blockdims, index, itemsize)
    175
    176     bd_out = tuple(map(tuple, bd_out))

/opt/anaconda3/lib/python3.9/site-packages/dask/array/slicing.py in slice_with_newaxes(out_name, in_name, blockdims, index, itemsize)
    194
    195     # Pass down and do work
--> 196     dsk, blockdims2 = slice_wrap_lists(out_name, in_name, blockdims, index2, itemsize)
    197
    198     if where_none:

/opt/anaconda3/lib/python3.9/site-packages/dask/array/slicing.py in slice_wrap_lists(out_name, in_name, blockdims, index, itemsize)
    260     if all(is_arraylike(i) or i == slice(None, None, None) for i in index):
    261         axis = where_list[0]
--> 262         blockdims2, dsk3 = take(
    263             out_name, in_name, blockdims, index[where_list[0]], itemsize, axis=axis
    264         )

/opt/anaconda3/lib/python3.9/site-packages/dask/array/slicing.py in take(outname, inname, chunks, index, itemsize, axis)
    645         warnsize = maxsize = math.inf
    646     else:
--> 647         maxsize = math.ceil(nbytes / (other_numel * itemsize))
    648         warnsize = maxsize * 5
    649

OverflowError: cannot convert float infinity to integer

Scarf and Python version
Python: 3.9.7
Scarf: 0.19.6

Hi @marhirst!
Thank you for your appreciation and I hope that you find Scarf useful in your work.
The design philosophy in Scarf is to write out the data in two steps. In the first step, the Zarr structure is created and in the second the step the count data is written out. This is true for all the classes in the scarf.writer module.
The second part is missing in your script:  dump function of ZarrMerge. The code snippet for the same is below:
writer = scarf.ZarrMerge(
    zarr_path='test.zarr',
    assays=[a.RNA, b.RNA],
    names=['a', 'b'],
    merge_assay_name='RNA',
    overwrite=True
)
writer.dump()

Just to be on the safe side, I suggest that you delete test.zarr beforehand.
Hope this solves the issue. I'm happy to help if needed. :)
/PD

Thanks for the quick response! I actually did dump the zarr object, I just forgot to paste it over in my issue.
Is there anything else that could be happening to cause the problem?
Thanks,
Marissa

On Aug 13, 2022, at 4:22 AM, Parashar @.***> wrote:

Hi @marhirst!
Thank you for your appreciation and I hope that you find Scarf useful in your work.
The design philosophy in Scarf is to write out the data in two steps. In the first step, the Zarr structure is created and in the second the step the count data is written out. This is true for all the classes in the scarf.writer module.
The second part is missing in your script: dump function of ZarrMerge. The code snippet for the same is below:
writer = scarf.ZarrMerge(
zarr_path='test.zarr',
assays=[a.RNA, b.RNA],
names=['a', 'b'],
merge_assay_name='RNA',
overwrite=True
)
writer.dump()
Just to be on the safe side, I suggest that you delete test.zarr beforehand.
Hope this solves the issue. I'm happy to help if needed. :)
/PD
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Hi again,
I have released a new version for Scarf (0.20.1).  Please update your Scarf version:
pip install -U scarf
In this updated version, I have included certain internal check in ZarrMerge that might help us identify the issue and also changed the overwrite behaviour (which I think might be the underlying issue). I tried running ZarrMerge on my end with example files and it works fine. But I'm quite interested in understanding why it doesn't work on your data.
/PD

Hi Parashar,
Your original fix worked. It was absolutely adding the .dump to the zarr
call that made the pipeline work downstream. I'm not sure why that would be
the case, but I was able to process through the remainder of the workflow.
Thank you so much for your help and quick response!
Marissa
On Mon, Aug 15, 2022 at 4:02 AM Parashar @.***> wrote:

Hi again,
I have released a new version for Scarf (0.20.1). Please update your Scarf
version:
pip install -U scarf
In this updated version, I have included certain internal check in
ZarrMerge that might help us identify the issue and also changed the
overwrite behaviour (which I think might be the underlying issue). I tried
running ZarrMerge on my end with example files and it works fine. But I'm
quite interested in understanding why it doesn't work on your data.
/PD
—
Reply to this email directly, view it on GitHub
https://github.com/parashardhapola/scarf/issues/71#issuecomment-1214888489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOCN52LV5EW4INFF5EX6OTVZIPU7ANCNFSM56NNMXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Glad to hear that! I will close this issue then.

Hi Parashar,
Sorry for my last email, it actually was appending the .dump command after
merging the zarr object. Thank you for the quick response and sorry to clog
your issues log with such a simple, dumb mistake on my end.
Thank you again!
Marissa
On Sat, Aug 13, 2022 at 9:43 AM Marissa Hirst @.***> wrote:

Thanks for the quick response! I actually did dump the zarr object, I just
forgot to paste it over in my issue.
Is there anything else that could be happening to cause the problem?
Thanks,
Marissa
On Aug 13, 2022, at 4:22 AM, Parashar @.***> wrote:

Hi @marhirst https://github.com/marhirst!
Thank you for your appreciation and I hope that you find Scarf useful
in your work.
The design philosophy in Scarf is to write out the data in two steps. In
the first step, the Zarr structure is created and in the second the step
the count data is written out. This is true for all the classes in the
scarf.writer module.
The second part is missing in your script: dump function of ZarrMerge.
The code snippet for the same is below:
writer = scarf.ZarrMerge(
zarr_path='test.zarr',
assays=[a.RNA, b.RNA],
names=['a', 'b'],
merge_assay_name='RNA',
overwrite=True
)writer.dump()
Just to be on the safe side, I suggest that you delete test.zarr
beforehand.
Hope this solves the issue. I'm happy to help if needed. :)
/PD
—
Reply to this email directly, view it on GitHub
https://github.com/parashardhapola/scarf/issues/71#issuecomment-1214141069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKOCN5ZW3SGB2ZGJWEMMPSDVY6APNANCNFSM56NNMXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Authentication

How can we validate AD access token with isolated function app

@AkashDoke, you can add reference to Microsoft.Identity.Web and instead of calling AddJwtFunctionsBearer you can call any overload of AddMicrosoftIdentityWebApi(... which has been designed to work with Microsoft authentication providers (AAD, B2C, Live)
Also, make sure you don't use the default Bearer scheme. You can use JwtFunctionsBearerDefaults.AuthenticationScheme constant.
Please let me know

Alternatively, you can use AddOpenIdConnect(... and configure it based on the values found in your Entra ID app registration

Thank you for the response @artmasa
I have below creds can you please help with how can i configure it for function app to do the access token based authentication ?
"AzureAd": {
"Authority": "https://login.microsoftonline.com/<>",
"ClientId": "<>",
"ValidateAuthority": true
}

The magic on how aspnet core validates token is already built-in. If you look into the source you will notice that you just need set the Authority and Audience properties for a successful token validation.
The first assumption of the framework is that they are standards compliant. In you inspect the headers section of a token coming from AAD you will notice the kid and x5t elements. These are public key identifiers.
In order for the signature of a token to be verified you need access to these keys. When you implement your own token generation you have access to these keys within your application.
Microsoft authentication providers follow the OIDC standards. When you provide the value for the Authority property, for example https://login.microsoftonline.com/<your-tenant>, the framework will concatenate this value with the standard based metadata path: /.well-known/openid-configuration.
You can grab your authority value and append the string above: https://login.microsoftonline.com/<your-tenant>/.well-known/openid-configuration and paste it in your browser. You will receive a json payload with metadata. This json payload has the jwks_uri, also standard, with a URL to access the public keys used by the IdP to validate the signature on the tokens. One of these keys should match identifiers of the kid and x5t elements referenced above.
For ASPNET Core JWT Bearer auth, if you don't specify a custom MetadataAddress values and the URL referenced above is not resolved, or you don't provide a custom token validator; you would receive an error stating keys to validate signature were not found. Validating the signature is the default behavior for JWT Bearer handler.
So, the concept of validating authority is embedded in the process of validating the signature as the framework relies on the keys provided by the authority to validate the signature of your token.
All this functionality is available when you use the generic AddJwtFunctionsBearer(... method to configure your application. AddMicrosoftIdentityWebApi(... gives additional configuration for features only available on Microsoft identity providers, but all this is not required to validate the token (which validates the authority)
Conclusion: Just by providing the Authority property you are ensuring your application is verifying the received tokens come from such Authority.
I hope this explanation helps. Try the well known metadata address referenced above on your own and all these elements will start making sense pretty fast.

The magic on how aspnet core validates token is already built-in. If you look into the source you will notice that you just need set the Authority and Audience properties for a successful token validation.
The first assumption of the framework is that they are standards compliant. In you inspect the headers section of a token coming from AAD you will notice the kid and x5t elements. These are public key identifiers.
In order for the signature of a token to be verified you need access to these keys. When you implement your own token generation you have access to these keys within your application.
Microsoft authentication providers follow the OIDC standards. When you provide the value for the Authority property, for example https://login.microsoftonline.com/<your-tenant>, the framework will concatenate this value with the standard based metadata path: /.well-known/openid-configuration.
You can grab your authority value and append the string above: https://login.microsoftonline.com/<your-tenant>/.well-known/openid-configuration and paste it in your browser. You will receive a json payload with metadata. This json payload has the jwks_uri, also standard, with a URL to access the public keys used by the IdP to validate the signature on the tokens. One of these keys should match identifiers of the kid and x5t elements referenced above.
For ASPNET Core JWT Bearer auth, if you don't specify a custom MetadataAddress value and the URL referenced above is not resolved, or you don't provide a custom token validator; you would receive an error stating keys to validate signature were not found. Validating the signature is the default behavior for JWT Bearer handler.
So, the concept of validating authority is embedded in the process of validating the signature as the framework relies on the keys provided by the authority to validate the signature of your token.
All this functionality is available when you use the generic AddJwtFunctionsBearer(... method to configure your application. AddMicrosoftIdentityWebApi(... gives additional configuration for features only available on Microsoft identity providers, but all this is not required to validate the token (which validates the authority)
Conclusion: Just by providing the Authority property you are ensuring your application is verifying the received tokens come from such Authority.

I am using above code still it is not working i am not really sure what went wrong. But your guidance here will be really helpful.
I am using functionV4 isolated mode.

Getting this error

First, you are leaving you AddMicrosoftIdentityWebApi with the default Bearer scheme. It needs to be changes to FunctionsBearer or any other custom scheme name.
Second the audience needs to match your token's aud claim. That might be the value it contains, just verify. Commonly the audience is in URI format and can be retrieved from the Expose API of your app registration in Entra ID as API ID URI
It should look like:
services
    .AddFunctionsAuthentication(JwtFunctionsBearerDeafults.AuthenticationScheme)
    .AddMicrosoftIdentityWebApi(
        options =>
        {
            options.Authority = "https://login.microsoftonline.com/<your-tenant-id>";
            options.Audience = "<the-aud-claim-in-a-token-for-api-you-know-comes-from-aad>";
        },
        JwtFunctionsBearerDefaults.AuthenticationScheme);

If you need to decode a token to extract the correct aud claim, use https://jwt.ms. It is ensured by this Microsoft site the token never leaves your browser.

Thanks for the response @artmasa , i also tried above provided solutions but while working with azure c# function app getting below error, it will be really helpful if you can guide me on how can implement it in standard way ?

I guess for that method there's another function parameter that needs to be specified.
Try just using AddJwtFunctionsBearer(options => { ... }) method, as what you are pursuing is signature validation with authority keys.

After implementing AddJwtFunctionsBearer getting above warnings :)

You're on the right track! Now you know what key is being identified to validate the signature. Troubleshooting further at this point would requiring exposing your configuration (not advisable). You can send me a DM @artmasa at X with some way to connect. Would be glad to help.

@artmasa  Was trying to ping you but getting warning as cannot be messaged
Can you please assist it with what are the probable missing points that I may need to try to get it to work?

@AkashDoke, sorry about that. I have changed my settings in X.
We need to look into why signature cannot ve verified with the provided key ID in the token. I don't think this is not related to the framework as we can see in the console, the aspnet framework is doing the expected to validate signature.

Hi @AkashDoke, were you able to identify the problem with the token validation?

Sorry @artmasa  I wont be able to fix the issue yet. Is there any chance so we can connect over any communication source and look into it together ? That will be really helpful

Hey @artmasa  sorry to bother you, but its kind of urgent as we are going live in next 2days and we have to do this change on priority. Your help will be really appreciate.

How can I contact you?

@artmasa its kind of urgent we have last day today to get ready for PROD , you have some free time today connect that will be really helpful

@AkashDoke, I have tried to reach you on Meet and Chat, but you don't seem to be available.

Sorry @artmasa  i didn't receive any notification
Can you please join here once you are available ?
To join the meeting on Google Meet, click this link: https://meet.google.com/xer-ysud-vew
Or open Meet and enter this code: xer-ysud-vew

@artmasa
I have completed all the R&D task and function is working as expected still while using darkloop and authenticating function locally getting above error

@AkashDoke it seems you have not configured the right audience in your function app. The token references a different value in the aud claim. You might want to compare these values

@AkashDoke, Hope by now you have been able to solve the challenge. I will go ahead and close the issue.


Wishlist API for BigCommerce fails to add an item.

Adding an item to wishlist using BigCommerce framework results in:
BigcommerceApiError: Big Commerce API error (422):
{
  "status": 422,
  "title": "JSON data is missing or invalid",
  "type": "https://developer.bigcommerce.com/api-docs/getting-started/api-status-codes",
  "errors": {
    "customer_id": "error.expected.jsnumber"
  }
}

A quick hacky fix is to make sure the customer_id is an integer not a string on this line.
customer_id: Number(customerId)

this fix looks good to me, why you donth submit an PR @Miladroigari ?

Looks like this has been fixed!


Aurora 1.31 Crashes playing Elite Dangerous

Hello guys, I believe I found a bug. After playing Elite Dangerous for a couple minutes, Aurora 1.31 Crashes with the error attached.
Expected Behaviour
Aurora should be working
Actual Behaviour
Aurora crash after playing the game for a while.
Reproduction steps
I flew from a system to another, them went to any space station and docked.
Frequency
Cant figure, it seems like it crashes trying to read some game files.
Aurora Version:
1,31
Previously an Issue?
I don't know

2023-04-11 00.04.14.log
Log:
[INFO] 00:04:14:
Operation System: Windows 10 Pro
System Architecture: 64 bit
Environment OS Version: Microsoft Windows NT 10.0.19045.0
System Directory: C:\WINDOWS\system32
Executing Directory: C:\Program Files\Aurora
Launch Directory: C:\Program Files\Aurora
Processor Count: 4
SystemPageSize: 4096
Environment Version: 6.0.14
Is Elevated: True
Aurora Assembly Version: <IP_ADDRESS>
Aurora File Version: v131
[INFO] 00:04:14: Loading devices from assembly...
[INFO] 00:04:15: Loading Configuration
[INFO] 00:04:16: Loading Input Hooking
[INFO] 00:04:16: Loaded Input Hooking
[INFO] 00:04:16: Fetching latest pointers
[INFO] 00:04:17: Loading Plugins
[INFO] 00:04:17: Loaded Plugins
[INFO] 00:04:17: Starting IpcListener
[INFO] 00:04:17: [IPCServer] Pipe created 41583755
[INFO] 00:04:17: Listening for wrapper integration calls...
[INFO] 00:04:17: Loading Applications
[INFO] 00:04:19: NVAPI
[INFO] 00:04:19: Version: NVidia Complete Version 1.10
[INFO] 00:04:19: Number of GPUs: 1
[INFO] 00:04:19: AMD Display Library
[INFO] 00:04:19: Status: -1
[INFO] 00:04:22: Loading RazerSdkManager
[INFO] 00:04:22: RazerSdkManager loaded successfully!
[INFO] 00:04:22: Loading KB Layouts
[INFO] 00:04:22: Loaded KB Layouts
[INFO] 00:04:22: Loading ResourceDictionaries...
[INFO] 00:04:22: Loaded ResourceDictionaries
[INFO] 00:04:24: Loading Device Manager
[INFO] 00:04:24: Loading ConfigUI...
[INFO] 00:04:24: [Device][Logitech] Initialized Successfully.
[INFO] 00:04:24: [Device][LightFX] Initialized Successfully.
[ERROR] 00:04:24: SteelSeries GameSense SDK could not be initialized: System.IO.FileNotFoundException: Core Props file could not be found at "C:\ProgramData/SteelSeries/SteelSeries Engine 3/coreProps.json"
at SteelSeries.GameSenseSDK.GameSenseSDK.init(String sseGameName, String sseGameDisplayname, Byte iconColorID) in D:\a\Aurora\Aurora\Project-Aurora\Project-Aurora\Devices\SteelSeries\GameSenseSDK.cs:line 58
at Aurora.Devices.SteelSeries.SteelSeriesDevice.Initialize() in D:\a\Aurora\Aurora\Project-Aurora\Project-Aurora\Devices\SteelSeries\SteelSeriesDevice.cs:line 49
[INFO] 00:04:24: [Device][SteelSeries] Initialized Successfully.
[INFO] 00:04:24: [Device][Corsair] Initialized Successfully.
[INFO] 00:04:24: [Device][Bloody (RGB.NET)] Initialized Successfully.
[WARN] 00:04:24: Could not find an AutoNode getter method for field '_ParsedData' of type 'JObject'. It will not be automatically populated.
[INFO] 00:04:24: [Device][CoolerMaster (RGB.NET)] Initialized Successfully.
[ERROR] 00:04:25: Current bind file: C:\Users\Rafael\AppData\Local\Frontier Developments\Elite Dangerous\Options\Bindings\Custom.4.0.binds
[INFO] 00:04:29: [Device][OpenRGB] Initialized Successfully.
 Path '', line 2, position 0.
at Newtonsoft.Json.JsonTextReader.Read()
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
at Aurora.Profiles.EliteDangerous.GameEvent_EliteDangerous.StatusReadCallback(Int32 lineNumber, String lineValue) in D:\a\Aurora\Aurora\Project-Aurora\Project-Aurora\Profiles\EliteDangerous\GameEvent_EliteDangerous.cs:line 41
at Aurora.Profiles.EliteDangerous.Helpers.FileWatcher.TailFile() in D:\a\Aurora\Aurora\Project-Aurora\Project-Aurora\Profiles\EliteDangerous\Helpers\FileWatcher.cs:line 53
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
[FATAL] 00:25:00: Runtime terminating: True

Seems like that file has unnecessary lines or wrong characters there. Can you also share the log file?
Please just send the file, posting file content ruins thread readability

Thanks man, gonna try it later!


Remove static var one from PointwiseMultiplicative.

Friend PR: https://github.com/apple/swift/pull/26120

Closing stale PR for now. To be revisited later.


Replaced std::unordered_{set,map} with robin_{set,map}.

Hi, after comparing recent benchmarks of popular hash map implementations such as
https://tessil.github.io/2016/08/29/benchmark-hopscotch-map.html and https://martin.ankerl.com/2019/04/01/hashmap-benchmarks-03-01-result-InsertHugeInt/ the winner for me is robin_map https://github.com/Tessil/robin-map due to general performance, being header-only, providing a close-to-std::unordered_map-API, and released under the MIT license. Google previously provided
https://github.com/sparsehash/sparsehash that is dated, a newer version is named absl::flat_hash_map https://github.com/abseil/abseil-cpp/blob/master/absl/container/flat_hash_map.h that is part of the absl library https://abseil.io/docs/cpp/guides/container and would make installing the entire library necessary to compile KaHIP which did not look ideal to me.
What i basically did in this PR is type-aliasing extlib::unordered_map as robin_map and replaced std::unordered_map with extlib::unordered_map in each and every project file except the occurrences in misc/pymodule. (The same applies to occasional occurrences of std::unordered_set).
This is just a starting point and must be seen as a first suggestion. What do you think about it? Wouldn't it be nice to write some unit tests to verify the correctness of the changes?
Best,
Daniel

Thanks for this! I will have a look at it very soon.
Just out of curiosity, did you already test if this yields improved overall performance?
Best
Christian

Thanks for this! I will have a look at it very soon.
Just out of curiosity, did you already test if this yields improved overall performance?
Best
Christian

I yet had some time for tests on desktop hardware and ran
./kaffpa ~/Downloads/great-britain.osm.graph --k=10 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=50 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=100 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=500 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=10 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=50 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=100 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=500 --preconfiguration=eco

multiple times but see almost equal time spent for partitioning for the STL-version and the 3rd-party-library. Played around a bit with other available cmd-line arguments and two other street graphs from https://www.cc.gatech.edu/dimacs10/archive/streets.shtml without any noticeable effect.
Do you have an idea what i oversee? Do you have suggestions for further tests?
Best,
Daniel

I yet had some time for tests on desktop hardware and ran
./kaffpa ~/Downloads/great-britain.osm.graph --k=10 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=50 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=100 --preconfiguration=eco
./kaffpa ~/Downloads/great-britain.osm.graph --k=500 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=10 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=50 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=100 --preconfiguration=eco
./edge_partitioning ~/Downloads/great-britain.osm.graph --k=500 --preconfiguration=eco

multiple times but see almost equal time spent for partitioning for the STL-version and the 3rd-party-library. Played around a bit with other available cmd-line arguments and two other street graphs from https://www.cc.gatech.edu/dimacs10/archive/streets.shtml without any noticeable effect.
Do you have an idea what i oversee? Do you have suggestions for further tests?
Best,
Daniel

Hi Daniel,
thanks! I will run some more extensive tests on a wide range of graphs as soon as there is time.
I hope this is fine.
Best
Christian

Hi Daniel,
thanks! I will run some more extensive tests on a wide range of graphs as soon as there is time.
I hope this is fine.
Best
Christian

Hi Daniel,
I performed more experiments, but in all cases the new hash maps seem to slow down the program. Closing the PR for now, but I will investigate this with the next release again.  Again thanks for you input!
Best
Christian

Hi Daniel,
I performed more experiments, but in all cases the new hash maps seem to slow down the program. Closing the PR for now, but I will investigate this with the next release again.  Again thanks for you input!
Best
Christian


Using shadow blocks

I think most microbit blocks don't have a default value for missing inputs, so we could instead use shadow blocks on the toolbox to ensure there is always a valid input.

This is a good idea. I think I've worked out how to configure it.... let's see what I can cobble together on the train this week.

Fixed by 78b3a80a63b42efb88938bebd01484aa9005d1bf


Cover Creating Language

Motivation
Currenlty test Kentico.Kontent.Management.Tests.ManagementClientTests.CreateLanguage_CreatesLanguage is skipped, because the Languages can't be deleted, only archived.
Proposed solution
Make this test completely Unit and do not rely on the real endpoint.
Additional context
related to #78

Fixed on #91


Link to an code sample on how to integrate Key Vault with App Config for Spring Boot app

The links given to the code samples on how to integrate Key Vault with App Config in the other feedback issues to this tutorial are all broken (e.g. https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/spring/azure-spring-boot-samples/azure-spring-cloud-sample-appconfiguration-conversion-initial).
Could you please provide me with an updated link to an code sample on how to integrate Key Vault with App Config for Spring Boot app?
Thanks, Michael

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

ID: afc14028-8f4c-05bf-8a01-3ce82041b0b2
Version Independent ID: 05e48444-1d81-07de-5ca5-1efcc9241d00
Content: Tutorial for using Azure App Configuration Key Vault references in a Java Spring Boot app
Content Source: articles/azure-app-configuration/use-key-vault-references-spring-boot.md
Service: azure-app-configuration
GitHub Login: @AlexandraKemperMS
Microsoft Alias: alkemper

@michaelegner Thank you for the feedback. We are actively investigating and will get back to you soon.

Hi @michaelegner, yes this link is currently broken. I have looked into it and the doc is temporarily down because it is being updated. It should be available again within the next week.

This issue is being assigned to the content author to evaluate and update the documentation as appropriate.

#reassign:maud-lv

Hi, and thank you for your interest in our documentation. I have looked at this doc and haven't found any broken link. This may be due to the fact that this doc has been updated since this issue was last commented on. I will close this issue for now. Please open a new issue if you think this article still needs to be updated. Thank you #please-close


Add support for TLS settings

The rubygem gelf library supports ssl since it's version 3.0.0. This PR adds new SSL options to logstash-output-gelf. This is pretty much a reimplementation of #28.
Parameters

ssl (boolean)
ssl_certificate_authorities (string)
ssl_certificate (string)
ssl_key (string)
ssl_verify_mode ([none, peer, force_peer])

Currently there is no option for ssl_ciphers as the gelf library dosen't have any kind of cipherlist option.
Please review and let me know if any changes are necessary.

Hi, i just signed the CLA. Let me know if any further changes are necessary.


Add support for relationship direction.

When using the Structurizr PlantUML exporter with autolayout, the result is not always satisfying.
PlantUML component diagrams support hints for the layout by means of arrow directions, see Changing Arrow Direction in https://plantuml.com/component-diagram.
This pull request is a proof of concept that use tags associated to relationship for setting up those hints:
foo -> bar: "name" "" "right"

Will generate
foo .[#707070,thickness=2]right.> bar : "<color:#707070>Uses"

It's code I use in a custom exporter but I think it would be useful as an upstream feature. I understand the real solution would involve adding a direction attribute to relationship styles instead of using tag (maybe plantumlDirection to make it clear it's just a plantUML thing). I thought it would be clearer to propose this new feature in this way then just creating an issue.
Thanks for considering this and for all your work on Structurizr!

Thanks for the PR. I'm not going to merge this into the repo, but it's great that it's working for you. You may also be interested in https://github.com/matdurand/structurizr-c4plantuml-layout-exporter that is attempting to do something a little more sophisticated (i.e. provide a way for the direction of relationships on specific views to be customised).


feat: get capacitor range

Porting get capacitor range function
This function can be used to get an estimate of how large the capacitance is. It doesn't yield anything pF. Instead it outputs a numerical value. This could be used to compare two objects wrt to their capacitance.
The following figure shows how this numerical output varies wrt to charging time. As charging time increases, the range seems to converge to a flat value.

In the python driver, this function was originally used to choose an appropriate charge time for MULTIMETER_GetCapacitance. It is still used for that purpose in pslab-android: https://github.com/fossasia/pslab-android/blob/d46e5d8c5deafffb5f878ab703dda577cb80d6bf/app/src/main/java/io/pslab/communication/ScienceLab.java#L2473
The python driver now uses an alternative method to estimate the capacitor range which takes longer but is more accurate. Since the android app depends on this function, we need to keep it.


[dkr] Try to fix corrupted kubectl in images

It looks like busybox wget doesn't exit non-zero if it fails while
downloading the file. Replace with wget to try address this.

/land


Feature/extensions

Add extensions menu
Add a recurring keysend helper extension

https://user-images.githubusercontent.com/80651189/124531526-8c787d80-ddc3-11eb-8473-616dc7c02ac4.mp4
A short explainer on what's going on this demo vid:

User navigates to extensions menu (this menu is generated automatically based off scripts in /home/admin/extensions
blitz.recurringpayment.sh is selected, script can be previewed and edited before committing to run it
recurring payment script is run, new menus ask user for node pubkey, how many sats they want to send, and at what frequency
script spits out a new keysend script and a crontab entry to paste into crontab.

Thank you, loving this!
One possible improvement I can think of is to store the extensions on the disk so they are preserved during major updates with and SDcard (OS) change.
The usual place is /mnt/hdd/app-data where there could be an extensions directory owned by admin.
One problem with that is the disk is currently mounted with noexec in the fstab so nothing on it can be executable.
For now we need to store the extensions on the SDcard and redownload them after SDcard changes unless this changes #2405

Could we automatically back up and restore the extensions folder when doing updates and manual backups? Unfortunately this wouldn't do anything for catastrophic failures of the SD, since the extensions would be stuck there.. but I envision extensions being fairly replaceable.
Alternatively, we could watch the extensions folder for changes and copy them over to the hdd when needed, a-la channel.backup

I also like the idea of extensions ... but might delay it it bit to a v1.8.1, because we are changing quite a lot now on the menus and that will settle down after v1.8.0 release to make further modifications.


Update to TypeScript 5

#54
#53
#52
#51
#50
#49
#48  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @steveluscher and the rest of your teammates on  Graphite


unable to connect to bitcoind code 401

Background
New install of bitcoind 17 and LND v0.5.2-beta. Upon trying to unlock wallet the following error occurs.
lnd --bitcoin.active --bitcoin.mainnet --debuglevel=debug --bitcoin.node=bitcoind --autopilot.active
2019-02-21 20:43:36.391 [INF] LTND: Version: 0.5.2-beta commit=v0.5.2-beta, build=production, logging=default
2019-02-21 20:43:36.391 [INF] LTND: Active chain: Bitcoin (network=mainnet)
2019-02-21 20:43:36.392 [INF] CHDB: Checking for schema update: latest_version=7, db_version=7
2019-02-21 20:43:36.426 [INF] RPCS: password gRPC proxy started at <IP_ADDRESS>:8080
2019-02-21 20:43:36.426 [INF] RPCS: password RPC server listening on <IP_ADDRESS>:10009
2019-02-21 20:43:36.427 [INF] LTND: Waiting for wallet encryption password. Use lncli create to create a wallet, lncli unlock to unlock an existing wallet, or lncli changepassword to change the password of an existing wallet and unlock it.
2019-02-21 20:43:50.089 [INF] LNWL: Opened wallet
2019-02-21 20:43:50.268 [INF] LTND: Primary chain is set to: bitcoin
unable to create chain control: unable to connect to bitcoind: status code: 401, response: ""
2019-02-21 20:43:50.523 [INF] LTND: Shutdown complete
unable to connect to bitcoind: status code: 401, response: ""
Your environment
LND v0.5.2-beta
Bitcoind
linux 18.04.
lnd.conf
debuglevel=trace
maxpendingchannels=9
bitcoin.active=1
bitcoin.node=bitcoind
bitcoind.rpchost=localhost
bitcoind.rpcuser=browning
bitcoind.rpcpass=****
bitcoind.zmqpubrawblock=tcp://<IP_ADDRESS>:28332
bitcoind.zmqpubrawtx=tcp://<IP_ADDRESS>:28333
autopilot.active=1
autopilot.maxchannels=8
autopilot.allocation=.75
bitcoin.conf
mainnet=1
server=1
daemon=1
rpcuser=browning
rpcpass=****
zmqpubrawblock=tcp://<IP_ADDRESS>:28332
zmqpubrawtx=tcp://<IP_ADDRESS>:28333
Steps to reproduce
Expected behaviour
lncli should unlock wallet.
Actual behaviour
No connection established.

Wrong authentication information.

Thanks Roasbeef. Any chance you could suggest where to look to fix this?

Wrong authentication information.

Thanks Roasbeef. Any chance you could suggest where to look to fix this? Does this mean I am not passing the correct password or rpcuser to LND?

Yes, check you bitcoins conf file

Come across to the same problem, @bruceleeroy18 have you solved it?

@jasonzhouu Yea, check your bitcoin.conf file and lnd.conf file carefully. I had rpcpass instead of rpcpassword in bitcoin.conf.

@bruceleeroy18 Thank you, problem solved. I launched lnd with --bitcoind.rpcuser=REPLACEME --bitcoind.rpcpass=REPLACEME, forget to replace this two argument with real user and password.

@bruceleeroy18 Are rpcuser and rpcpass necessary? Does lnd not support ~/.bitcoin/.cookie?

It came back online. Idk if it was my end or not. I restarted my node and it was active again.
Sent from ProtonMail mobile
-------- Original Message --------
On Feb 27, 2021, 17:19, Geremia wrote:

@bruceleeroy18 Are rpcuser and rpcpass necessary? Does lnd not support ~/.bitcoin/.cookie?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


[DC-831] CR remove_operational_pii_fields to Base Class

Ran into issues while trying to rebase. Closing this PR opening a new one with branch bg/dc_831_a


add --timeout-smt to check and simulate

In our experiments, it sometimes happens that apalache-mc simulate gets stuck in a randomly selected schedule, without progressing for many hours. Since we would like to get quicker feedback with simulate, this PR add the CLI option --timeout-smt that limits the time spent on a single SMT query. Also, we are removing the old tuning option search.smt.timeout, which is hard to discover.

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[x] Documentation added for any new functionality
[x] Entries added to ./unreleased/ for any new functionality

As a question unrelated to the review, should we be supporting timeout-smt with check? Simulation is probabilistic anyway, so no real harm there, but setting a timeout may incorrectly give the impression that check succeeded, since the behavior is to assume invariants hold whenever timeouts happen.

As a question unrelated to the review, should we be supporting timeout-smt with check? Simulation is probabilistic anyway, so no real harm there, but setting a timeout may incorrectly give the impression that check succeeded, since the behavior is to assume invariants hold whenever timeouts happen.

check --timeout-smt=n finishes with a runtime error on timeout. Also, in case of simulate, the user gets a message that there were timeouts.

check --timeout-smt=n finishes with a runtime error on timeout. Also, in case of simulate, the user gets a message that there were timeouts.
I see a lot of searchState.onResult(RuntimeError()) replaced with searchState.onResult(SmtTimeout(...)).
Is it this bit? https://github.com/apalache-mc/apalache/blob/f7aeb232442052916bca82b1708b8eb9bc594fa7/tla-bmcmt/src/main/scala/at/forsyte/apalache/tla/bmcmt/SeqModelChecker.scala#L269-L274

check --timeout-smt=n finishes with a runtime error on timeout. Also, in case of simulate, the user gets a message that there were timeouts.

I see a lot of searchState.onResult(RuntimeError()) replaced with searchState.onResult(SmtTimeout(...)). Is it this bit?
https://github.com/apalache-mc/apalache/blob/f7aeb232442052916bca82b1708b8eb9bc594fa7/tla-bmcmt/src/main/scala/at/forsyte/apalache/tla/bmcmt/SeqModelChecker.scala#L269-L274

Yeah. This is the clear distinction here. There are other cases where I am less certain. In any case, you will get a message that there were timeouts. I think we should play with this UX a bit and figure out what works the best.


Save photos when clicking save and close

Can you add to the other place we save, i.e. the "save and next" button press when there is an annual portion to the report? (realizing that the save logic is a bit scattered and could perhaps be consolidated. Not a request for this PR, but something we can do in a followup perhaps)

Sure, I wasn't really sure if I needed to add it there!


update documentation for contributors to be able to add new resources and remote

Q
A

🐛 Bug fix?
no

🚀 New feature?
no

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#867

❓ Documentation
yes

Description
update documentation for contributors to be able to add new resources and remote

@wbeuil I added some fixes, when you'll get back from holidays this one need your legendary doc review pass 🙏🏻


Unable to construct generic type using more than one argument

Describe the bug
If the static constructor uses more than one argument, the execution fails with a CLR exception.
Reproducible Example
public class  TestClass<T> {
    public TestClass(int a, int b) { }
}
public interface IFactory {
    [ConstructorShim(typeof(TestClass<>))]
    TestClass<T> New<T>(int a, int b);
}

ShimBuilder.Create<IFactory>().New<string>(1, 2);

Expected behavior
A new instance of the target is created, using the supplied arguments.
Screenshots / Outputs

System.InvalidProgramException: 'Common Language Runtime detected an invalid program.'

Environment (please complete the following information):

Version: 1.6.5
Platform: .NET 5

Fixed in v1.6.6


[BUG] deleted clusters still visible under rancher

Describe the bug
I can see (2) as the cluster count in Rancher, but I only have "local" - also when creating Cloud Credentials, I can see a list of previously deleted clusters
To Reproduce
Add more than one Harvester Cluster, then delete it. Go to Cloud Credentials, it's still listed there.
Expected behavior
Clusters should be deleted from Rancher when deleted from Virtualization Management
Support bundle
rancher 2.6.1-rc11
Harvester ISO version: master-c4a76706-head
Environment:

Harvester ISO version: master-c4a76706-head
Underlying Infrastructure: Dell PowerEdge r620

Additional context
Add any other context about the problem here

.

.

this is identical to https://github.com/harvester/harvester/issues/1311, close as it has already been verified on the other issue, feel free to re-open it if u encounter a different issue, thanks.


feat: add option php.files.exclude

Codecov Report

Merging #45 (1cdb253) into main (0efb96a) will decrease coverage by 0.37%.
The diff coverage is 55.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main      #45      +/-   ##
============================================
- Coverage     82.73%   82.36%   -0.37%
- Complexity      970      977       +7
============================================
  Files            45       45
  Lines          2444     2467      +23
============================================
+ Hits           2022     2032      +10
- Misses          422      435      +13

Impacted Files
Coverage Δ

src/Client/Workspace.php
58.82% <0.00%> (-41.18%)
:arrow_down:

src/Indexer.php
62.37% <72.72%> (+0.83%)
:arrow_up:

src/LanguageServer.php
78.62% <77.77%> (-0.58%)
:arrow_down:

src/utils.php
83.72% <0.00%> (-2.00%)
:arrow_down:

src/DefinitionResolver.php
87.00% <0.00%> (-0.02%)
:arrow_down:


Does not work on current Debian

transcode-video "VIDEO_TS"
HandBrakeCLI: unrecognized option '--auto-anamorphic'

@hagabla As near as I can tell from the information you provided, the version of HandBrakeCLI you have installed is older than 1.0.0, which is currently the minimum requirement for using transcode-video. You need to upgrade to HandBrakeCLI  version 1.0.0 or later. I assume that's possible on a Debian Linux distribution?

https://packages.debian.org/stretch/handbrake-cli
that's what up to date stable Debian (9) ships with (0.10.5+ds1-2 ). No newer versions around on the official (or unofficial Debian) repository. There is an unofficial ubuntu repository that has uptodate packages, xenial packages seem to work, for now, but that's not really a solution.

@hagabla Ugh. Being unable to use up-to-date software packages is a known "feature" of Debian. Seriously, HandBrakeCLI version 1.0.0 is not some bleeding-edge experiment. It was released on December 24, 2016, over 18 months ago.
If you can't install a newer HandBrakeCLI, then my advice is to downgrade your video_transcoding Gem to version 0.19.0.

I have used the ubuntu package and transcoded the movie successfully already,  just though that is something you should be aware of. After all Debian is one of the bigger linux Distros out there.
Did not think of downgrading video_transcoding, thanks for that.

@hagabla Thanks for letting me know! My Linux testers didn't mention that when we were evaluating the change for 0.20.0. It happens. :)
I'm glad that downgrading is an option for you, too.
I'll close this now but feel free to comment further.