Unwrapping macros to abort() isn't detected, can I give a hint?

I am trying to run infer against PostgreSQL and PostgreSQL extensions.
A standard pattern in PG codebases is something like:
	if (!HeapTupleIsValid(tuple)) // This is a null check
	{
		elog(ERROR, "cache lookup failed for relation %u", relOid); // This will eventually resolve to abort()
	}
	natts = ((Form_pg_class) GETSTRUCT(tuple))->relnatts;  // This line causes a null dereference bug

elog is:
#define elog(elevel, ...)  \
	ereport(elevel, errmsg_internal(__VA_ARGS__))

ereport is:
#define ereport(elevel, ...)	\
	ereport_domain(elevel, TEXTDOMAIN, __VA_ARGS__)

ereport_domain is:
#define ereport_domain(elevel, domain, ...)	\
	do { \
		pg_prevent_errno_in_scope(); \
		if (errstart(elevel, domain)) \
			__VA_ARGS__, errfinish(__FILE__, __LINE__, PG_FUNCNAME_MACRO); \
		if (__builtin_constant_p(elevel) && (elevel) >= ERROR) \
			pg_unreachable(); \
	} while(0)

and pg_unreachable:
#define pg_unreachable() abort()

So the natts = ((Form_pg_class... line will never get hit in the tuple==NULL case but infer does not currently detect this.
Is there a way to tell infer that ereport(ERROR is never going to return?
I don't think it should have an effect but:
$ infer --version
Infer version v1.0.0-d1935941a

and this is running on RHEL 8 currently.

The explore for this is:
lang.c:50: error: Null Dereference
  pointer `tuple` last assigned on line 42 could be null and is dereferenced at line 50, column 32.

lang.c:31:1: start of procedure cpsm_lang_check_perm()
29.  * has the required permissions.
30.  */
31. static void
    ^
32. cpsm_lang_check_perm(Oid langOID, uint32 required)
33. {

lang.c:42:2: Skipping SearchSysCache1(): method has no implementation
40. 	 * Fetch tuple of the Language.
41. 	 */
42. 	tuple = SearchSysCache1(LANGOID, ObjectIdGetDatum(langOID));
     ^
43. 	if (!HeapTupleIsValid(tuple))
44. 	{

lang.c:43:7: Taking true branch
41. 	 */
42. 	tuple = SearchSysCache1(LANGOID, ObjectIdGetDatum(langOID));
43. 	if (!HeapTupleIsValid(tuple))
          ^
44. 	{
45. 		ereport(ERROR,

lang.c:45:3: Taking false branch
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:45:3: Taking false branch
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:45:3: Loop condition is false. Leaving loop
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:50:2:
48. 	}
49.
50. 	langForm = (Form_pg_language) GETSTRUCT(tuple); //INFER_BREAKPOINT
     ^
51.
52. 	cpsm_get_label(LanguageRelationId, cpsm_heap_tuple_get_oid(tuple, LanguageRelationId), InvalidAttrNumber,```

So I think the failure is because it's taking the false path on the last if, but if elevel >= ERROR (which is true) then why does it follow the false path?

The explore for this is:
lang.c:50: error: Null Dereference
  pointer `tuple` last assigned on line 42 could be null and is dereferenced at line 50, column 32.

lang.c:31:1: start of procedure cpsm_lang_check_perm()
29.  * has the required permissions.
30.  */
31. static void
    ^
32. cpsm_lang_check_perm(Oid langOID, uint32 required)
33. {

lang.c:42:2: Skipping SearchSysCache1(): method has no implementation
40. 	 * Fetch tuple of the Language.
41. 	 */
42. 	tuple = SearchSysCache1(LANGOID, ObjectIdGetDatum(langOID));
     ^
43. 	if (!HeapTupleIsValid(tuple))
44. 	{

lang.c:43:7: Taking true branch
41. 	 */
42. 	tuple = SearchSysCache1(LANGOID, ObjectIdGetDatum(langOID));
43. 	if (!HeapTupleIsValid(tuple))
          ^
44. 	{
45. 		ereport(ERROR,

lang.c:45:3: Taking false branch
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:45:3: Taking false branch
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:45:3: Loop condition is false. Leaving loop
43. 	if (!HeapTupleIsValid(tuple))
44. 	{
45. 		ereport(ERROR,
      ^
46. 				(errcode(ERRCODE_INVALID_PARAMETER_VALUE),
47. 				 errmsg("CPSM: cache lookup failed for language %u", langOID)));

lang.c:50:2:
48. 	}
49.
50. 	langForm = (Form_pg_language) GETSTRUCT(tuple); //INFER_BREAKPOINT
     ^
51.
52. 	cpsm_get_label(LanguageRelationId, cpsm_heap_tuple_get_oid(tuple, LanguageRelationId), InvalidAttrNumber,```

So I think the failure is because it's taking the false path on the last if, but if elevel >= ERROR (which is true) then why does it follow the false path?

It looks like it is the __builtin_constant_p part, removing that resolves the errors in this case.
How can this be resolved without removing it? (That is in an upstream postgres header so not something that can be edited from a downstream project).

It looks like it is the __builtin_constant_p part, removing that resolves the errors in this case.
How can this be resolved without removing it? (That is in an upstream postgres header so not something that can be edited from a downstream project).


change logging in registryserver.go

I don't see fmt.printf messages in log with default setting
so I replace thise with glog.Info. Also, correcting
spelling and wording.

the issue #104 turned out as false alarm, but lets have the log improvement in this PR


[Bug] dense.py in matmul  tensor_b.shape mismatch after autoscheduler tuning in ARM CPU

File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 81, in cfun
		    rv = local_pyfunc(*pyargs)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/relay/backend/te_compiler.py", line 311, in lower_call
		    best_impl, outputs = select_implementation(op, call.attrs, inputs, ret_type, target)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/relay/backend/te_compiler.py", line 189, in select_implementation
		    outs = best_plevel_impl.compute(attrs, inputs, out_type)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/relay/op/op.py", line 126, in compute
		    return _OpImplementationCompute(self, attrs, inputs, out_type)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 237, in __call__
		    raise get_last_ffi_error()
		  3: TVMFuncCall
		  2: std::_Function_handler<void (tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*), tvm::relay::__mk_TVM6::{lambda(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)#1}>::_M_invoke(std::_Any_data const&, tvm::runtime::TVMArgs&&, tvm::runtime::TVMRetValue*&&)
		  1: tvm::relay::OpImplementation::Compute(tvm::Attrs const&, tvm::runtime::Array<tvm::te::Tensor, void> const&, tvm::Type const&)
		  0: std::_Function_handler<void (tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*), TVMFuncCreateFromCFunc::{lambda(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)#2}>::_M_invoke(std::_Any_data const&, tvm::runtime::TVMArgs&&, tvm::runtime::TVMRetValue*&&)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 81, in cfun
		    rv = local_pyfunc(*pyargs)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/relay/op/strategy/generic.py", line 833, in _compute_dense
		    return [topi_compute(*args)]
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/topi/nn/dense.py", line 184, in dense
		    return matmul(data, weight, bias, out_dtype, False, True, auto_scheduler_rewritten_layout)
		  File "/host/mlperf_environment_xsk/tvm/tvm_build_armv9/tvm/python/tvm/topi/nn/dense.py", line 81, in matmul
		    out_dim, red_dim = tensor_b.shape
ValueError: too many values to unpack (expected 2)

Expected behavior
tensor_b.shape will be changed after tuning ?
Actual behavior
out_dim, red_dim  != tensor_b.shape
Environment
Any environment details, such as: Operating System, TVM version, etc
Linux, Ubuntu: 21.04, TVM 0.9 dev, ARM CPU,
Model : mobilenet-v3.tflite
Steps to reproduce
This error happens only in ARM CPU after autotuning, but disappears  before autotuning.

This can be solved by changing:  https://github.com/apache/tvm/blob/8f6fa8f2c41406cb54d01647ba8731e4ceb8f4ab/python/tvm/relay/op/strategy/arm_cpu.py#L466
to:
     wrap_compute_dense(topi.nn.dense, need_auto_scheduler_layout=True),

This seems to be missing in all implementations for arm_cpu.


[Channel dev, v0.3.6] I can build flutter_gallery, but the boot will crash after installation.

I can build flutter_gallery ( use flutter build apk ), but the boot will crash after installation.

running flutter doctor -v：
PS D:\flutter\examples\flutter_gallery> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel dev, v0.3.6, on Microsoft Windows [Version 10.0.17134.1], locale zh-CN)
[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    X Android license status unknown.
[X] Android Studio (not installed)
[√] IntelliJ IDEA Community Edition (version 2018.1)
[!] Connected devices
    ! No devices available

! Doctor found issues in 3 categories.
PS D:\flutter\examples\flutter_gallery>

V0.3.6 is not available, using v0.3.5

V0.3.6 is not available, using v0.3.5

@fly512  I upgraded the version with the flutter upgrade command, and I don't know why it was upgraded to an unusable version.
Anyway, thank you for your answer


Add Windows Logon Authentication.

Add Windows Logon Authentication.

Added windows Authentication.
This works in the following way.
only windows.
windows and key file
windows and master key
currently having issues with:
windows, master key and key file.

This has been added in the new connection function.


TestFoundation: Add infrastructure to assert that a test should crash

You can now define tests as follows:
func testCrashes() {
     assertCrashes {
          fatalError()
     }
}

These tests will spawn a child process and will assert that the child process crashes while executing the code in the block. Wrap the entire body of the test this way.

@swift-ci please test

@swift-ci please test

Also +1 Lily! = ). You are awesome!


Update to egui 0.20

Can we get a new release of egui_dock? 🙏

Yeah, it's about time. Would be nice to get #76 addressed first, I'm on it

🙏


refactor TestIngressRequestAuthentication and add vm

Refactor TestIngressRequestAuthentication in integration/security/jwt_test.go to become workload-agnostic and extend test coverage for VM.
Implements a new test pattern for ingress gateway test, which is documented in https://github.com/istio/istio/pull/32024. And adopts the pattern to refactor TestIngressRequestAuthentication.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test release-test_istio

gencheck_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test release-test_istio

gencheck_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test gencheck_istio

lint_istio
9d2f603c09a5ae944932bbbec3cf22d7237ad2d8
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@googlebot I fixed it.

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
0a70c5a5aa034237a533e07f983cbbecb65cd6c0
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster-tests_istio

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
0a70c5a5aa034237a533e07f983cbbecb65cd6c0
link
/test integ-pilot-multicluster-tests_istio

integ-security-multicluster-tests_istio
0a70c5a5aa034237a533e07f983cbbecb65cd6c0
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
0a70c5a5aa034237a533e07f983cbbecb65cd6c0
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
0a70c5a5aa034237a533e07f983cbbecb65cd6c0
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: #32067 failed to apply on top of branch "release-1.9":
Applying: refactor TestIngressRequestAuthentication and add vm
Using index info to reconstruct a base tree...
M	pkg/test/framework/components/echo/echotest/echotest.go
M	pkg/test/framework/components/echo/echotest/run.go
M	pkg/test/framework/components/echo/echotest/setup.go
M	tests/integration/security/jwt_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/security/jwt_test.go
CONFLICT (content): Merge conflict in tests/integration/security/jwt_test.go
Auto-merging pkg/test/framework/components/echo/echotest/setup.go
CONFLICT (content): Merge conflict in pkg/test/framework/components/echo/echotest/setup.go
Auto-merging pkg/test/framework/components/echo/echotest/run.go
CONFLICT (content): Merge conflict in pkg/test/framework/components/echo/echotest/run.go
Auto-merging pkg/test/framework/components/echo/echotest/echotest.go
CONFLICT (content): Merge conflict in pkg/test/framework/components/echo/echotest/echotest.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 refactor TestIngressRequestAuthentication and add vm
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #32236


Lvis checkpoint

Hi I wonder if you have a plan to provide the checkpoint file of LVIS.
Thanks

No.
The impl with LVIS is straight forward and I am happy to assist with any issues along the way.
We simply took Detic from https://github.com/facebookresearch/Detic, changed the third_party/deformable_detr to deta and added some detectron2 configs (assign first, assign second, balance_k) and NMS.


Store#onDispatch receive only dispatched Payload by UseCase

Almin 0.13 beta change the behavior of Store#onDispatch.

Store can not receive un-important payload by Store#onDsipatch.
It means that Store#onDispatch can't receive willExecutePayload.
In other words, Store#onDispatch is same with Store#receivePayload.

We can change this more primitive.

Store#onDispatch receive only dispatched the Payload by UseCase#dispatch.

Store#onDispatch can receive only User defined payload.
Store#receivePayload can receive specific payloads for updating view.

it includes User defined payload.


Fix for logback version upgrade

Description of what I changed
This fixes the build failure issues for the PRs #957 and #895
E2E test
Relied on e2e test cases
TESTED:
Please replace this with a description of how you tested your PR beyond the
automated e2e/unit tests.
Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

@bashir2 Thanks for the review, I had a bit deeper look at this and have provided my comments. Please check the latest commit as well.

The build is failing due to some regression introduced by the latest HAPI release. This is fixed by the PR #970 https://github.com/google/fhir-data-pipes/pull/970. We can rebuild this build, once the other PR is merged.


Ground.methodResume is a little bit confusingly described

I had some hard time understanding how Ground.methodResume works. Here are the issues I've had:

It starts in the Concept paragraph, which states Ground.resumeMethods instead of methodResume (slight detail, but adds on to the confusion)

I don't think the provided example really helps, it didn't help me at all:

// This is how grounddb uses this internally
Ground.methodResume([
  '/' + self.name + '/insert',
  '/' + self.name + '/remove',
  '/' + self.name + '/update'
], self.connection);

What I got out of this and how I'm using it now could be summarised in the following sentence:

When you to have your server-methods automatically run / resumed on the server, after a connection is established you need to tell this to Ground via Ground.methodResume. For instance:

Ground.methodResume('serverMethod');
Ground.methodResume(['serverMethod2', 'serverMethod3']);

This is also, how I'm using it now, and it seems to work fine. If that's the correct way, maybe the documentation can be adapted to be easier to understand?
Thx

Its just about telling grounddb what methods are resumable/should be resumed on reconnect. I take pr's :)

PR is here, hope to your satisfaction :) - in which case, 23 to go (http://24pullrequests.com/)

I have this method which i need to be call offline
Meteor.call('programs', remoteHost, diseaseId,deviceToken, function(err, data){
if(!err) {
// do something
}
how can i do this the Ground.methodResume method?
Thank you.


Add simple test api and use it on unit testings

Pull Request Test Coverage Report for Build 371293723

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.562%

Totals

Change from base Build 369225221:
0.0%

Covered Lines:
1553

Relevant Lines:
1734

💛  - Coveralls


backblaze-b2: 3.6.0 -> 3.7.0

Description of changes
New release from yesterday.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 215443 run on aarch64-darwin 1

5 packages built:

backblaze-b2
duplicity
duply
python310Packages.b2sdk
python311Packages.b2sdk

@ofborg build backblaze-b2 duplicity duply python310Packages.b2sdk python311Packages.b2sdk


Cbs/curvealongz

I wanted to add a toroidal field to a configuration, because I just wanted to jiggle the rotational transform a bit without too much ado. That would be nice with a straight coil along the z-axis, so I coded it up using the JaxCurve.
(Yes upon writing this PR I see that ToroidalBfield exists, but it does not allow one to shift the location of the curve along Z (playing with a 1/1 component), or optimize with it).
Think this might be useful for others, @krystophny has mentioned wanting to do something similar but resorting to using very large radius coils.
Some of the unittests are failing, mostly because the curve is not curved, and the evaluation of Kappa and it's derivatives fail. In that regard, what would be the best fix?

overload the kappa, torsion etc functions and their derivatives in the CurveAlongZ class with a default choice.
adapt the base Curve methods of evaluation of kappa, torsion etc and their derivatives to deal with straight sections.

option 1 would work, but lead to code repitition if other curves with straight sections are implemented
option 2 could lead to unneccesary computational overhead if every field evaluation nans need to be replaced
@andrewgiuliani @florianwechsung, any suggestions as to the best approach?

@smiet I opt for option 1. There will be no code duplication if you extract the required functionality into lower level functions that are used by several classes. Performance and simplicity of class hierarchy are higher priority than implementation details in special cases. Can you add that?


Standardize use of terminology re: UUID vs. EntryID

This relates to the sharding work, ex. #487 . There are many places in the code that refer to a UUID. However, the new, longer UUID is called an "EntryID" and the UUID is only a component of this. Should we update references to uuid to be entryID where it refers now to this longer hex string? Alternatively, should we leave references to uuid alone but have a different term for the old (short) uuid?
There may also be a case for making sure shardID, logID, and treeID are merged into one term, though I haven't checked whether this is already the case. The new sharding package uses the term treeID based on the term in the log index work.

I think this is also complete, feel free to reopen if not


Update coco_metric.py

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
This PR helps Visualize per-category performance of AP, AP_50, AP_75, AP_s, AP_m and AP_l.
Modification
add some elements to the results_per_category
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

@yarkable Thank you for your contribution. Can you explain the effect before and after the change in the pr message?

@hhaAndroid
Hi, before the modification, the per-category output result is like:

After the pr, the per-category output result:

@yarkable pre-commit run --all-files then git commit ...


(+VIRALˇ𝘝𝘐𝘋𝘌𝘖𝘴:.ＳＥＸ) One Girl one Frog 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 MMS On Social Media X Twitter [LINK√FULL]

13 sec ago One Girl one Frog 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 Viral On Social Media. (+VIRALˇ𝘝𝘐𝘋𝘌𝘖𝘴:.ＳＥＸ) One Girl one Frog 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 MMS On Social Media X Twitter [LINK√FULL]

One Girl one Frog's latest video has taken the internet by storm, igniting conversations across various social media platforms. The rising digital star, known for her engaging content—especially her recent Spiderman-themed clip—has experienced a remarkable boost in popularity, with fans eagerly seeking the full version. Viewers are flocking to Instagram, X (formerly Twitter), and Discord to engage with her latest creation.
As excitement grows, fans are passionately sharing their reactions online, emphasizing the crucial role social media plays in propelling creators like One Girl one Frog into the spotlight. With her unique style and innovative approach, she has quickly made a notable impact in the digital landscape, leaving many curious about her next moves.


Map Single Src to Inheritance dest

Hello,
Please help. I don't understand why this don't work :
class Src {
	private String fieldA;
	private String fieldB;
	...
}

class BaseDest {
	private String aField;
	...
}

class Dest extends BaseDest {
	private String bField;
	...
}

TypeMap<Src, BaseDest> baseTypeMap = modelMapper.createTypeMap(Src.class, BaseDest.class)
	.addMapping(Src::getFieldA, BaseDest::setAField);

TypeMap<Src, Dest> typeMap = modelMapper.createTypeMap(Src.class, Dest.class)
	.addMapping(Src::getFieldB, Dest::setBField)
	.includeBase(Src.class, BaseDest.class);

Result : the aField is empty.
includeBase() don't work in this case ?
What's wrong ?

Sorry. You can close. i've made a mistake.


Add support for remote development using ftp/sftp

Hello all. I am thinking to migrate from Jetbrains PhpStorm IDE to VsCode. The one feature I am currently missing is an ability to open and develope projects from a remote FTP servers like it is done there. I tryed to search for some plugins to do this but most of them are controlled only by command line wich is just consums much time just to use them but not actual coding. Thank you in advance. Sorry if it is a duplicate.

Currently, there are many extensions that do this. However, no one guarantees its reliability and safety. If it is made by the owner, it will be much more secure and stable. Thanks
vote +1

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


select by country, region, city

Hello Pierre-Henry
let's make together select by country, region, city and autocomplete when typing,
for one country or several

@excalibur-web Your hope are too high.
Look at the other person who wrote code and made Pull Request. It all went stale...
The only thing you might get is that PH is gonna complain about your code and let you write all by yourself !
There's a bug that happens when you reach 2000 or so user. It's related to the use of unsigned field in the database.  Me (I finally decided to let go and do something else, plus another user who's a good developer. We both wanted to fix it but we asked some guidance as maybe he had a reason to use unsigned. We never got a answer)...
https://github.com/pH7Software/pH7-Social-Dating-CMS/pulls
https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/793
Don't waste your time. You've got better to do ;-)
https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/830


Create a 'link --overwrite' equivalent for 'install'

Build sometimes fail on the link step, prompting me to link and overwrite. I would like to be able to do this straight away, as an option for the install command.
Example scenario:
==> Installing gcc
==> Downloading https://homebrew.bintray.com/bottles/gcc-6.3.0.sierra.bottle.tar
==> Pouring gcc-6.3.0.sierra.bottle.tar.gz
Error: The `brew link` step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink include/c++
Target /usr/local/include/c++
already exists. You may want to remove it:
  rm '/usr/local/include/c++'

To force the link and overwrite all conflicting files:
  brew link --overwrite gcc

To list all files that would be deleted:
  brew link --overwrite --dry-run gcc

I'm not sure I understand what advantage this provides over brew link --overwrite?

I want to be able to automate the process. The alternative might be to run
brew install <formula> || brew link --overwrite <formula>

which seems pretty insecure considering that the install process could fail in a million other ways not actually related to the linking.

Hi, Let me take the freedom to ask a related question, even though the issue is closed:
Getting the same error (https://travis-ci.org/tvatter/vinecopulib/builds/197634424#L445) I wonder if the later error in the installation (https://travis-ci.org/tvatter/vinecopulib/builds/197634424#L540) is not caused by the link issue? Thanks for help, Sylwester

@slayoo We don't provide support for Travis CI builds, sorry.

This is indeed is the same issue. The error output is telling you to run
rm '/usr/local/include/c++'

so you could get around the issue by running that before the failing command.

The obvious problem what this approach is that you have run the job and fail once to figure it what needs to be done. And a script might produce several link errors that have to be handled separately.

Thanks for your replies!

why is this closed?

@luisdavim Because we don't leave open feature requests we're not planning on working on.


i4i support in eks ami

What would you like to be added:
i4i support in eks ami.
i4i instances are not listed in https://github.com/awslabs/amazon-eks-ami/blob/master/files/eni-max-pods.txt
Why is this needed:
it would be great to be able to use i4i as eks node to get all benefits provided by this instance type.

This change has been merged and will be released in the next AMI, later this week. I'll update this issue once it's available.

This change is available in v20220523.


Please fix copyToSpreadsheet method, circular JSON error

Hello,
Please fix the copyToSpreadsheet method.
axios.post gives me a circular JSON error with the Google Spreadsheet object. Like this:
Hello, I get the following error when cpying a spreadsheet to another spreadsheet:
Code:
sheet = await exports.archive.addSheet({ title: "Sheet for month " + new Date().getMonth() + " of " + new Date().getYear() + 100 })
} catch {
let length = exports.archive.sheetsByIndex.length - 1;
sheet = exports.archive.sheetsByIndex[length]
}
console.log("Archive sheet found");
let original = exports.findSheet(0) //returns the spreadsheet to be coppied from.
await original.copyToSpreadsheet(sheet)
console.log("Coppied!")
//        await original.resize({ rowCount: 2, columnCount: 20 });
Result:
Archive loaded
Archive sheet found
Converting circular structure to JSON
--> starting at object with constructor 'GoogleSpreadsheetWorksheet'
|     property '_spreadsheet' -> object with constructor 'GoogleSpreadsheet'
|     property '_rawSheets' -> object with constructor 'Object'
--- property '1204944981' closes the circle
Thank you!

Google's naming with this is certainly confusing... This method doesn't copy a worksheet to another worksheet, it copies the worksheet to a different spreadsheet document.
Look at the method docs - you are supposed to pass in another spreadsheet document ID
You may be able to pass in the same document id if you want to duplicate a worksheet?


benchmarks: reduce size of minor-for-ci

It's a bit too slow to work with. Would be much nicer with a quicker turnaround.

/gcbrun exp -n d-cov-30 -b minor-for-ci -ns 1 -rd

Experiment is good: https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-09-22-640-d-cov-30-minor-for-ci/index.html
Time reduced by 10 min in comparison to the larger: https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-09-22-639-d-cov-24-minor-for-ci/index.html
Nicer to work with


Investigate performance delta with rc_switch_user API endpoint

See the following:

https://github.com/irods/irods/blob/c3c62ad4cfe8064bcc1475839cc79f362d0e1caa/lib/core/include/irods/library_features.h
https://github.com/irods/irods/blob/c3c62ad4cfe8064bcc1475839cc79f362d0e1caa/lib/api/include/irods/switch_user.h

This will not be implemented in this repository.
Please see the new HTTP API:
https://github.com/irods/irods_client_http_api


SlimeVR is requesting that you read this github issue 🥲

Hi @ThePheonixGuy
I'm assuming you turned off notifications for #308. But you are the only person left who still needs to fix their comment. The comment you left says

"Eiren Rain and SlimeVR Contributors".

BUT it should say instead

"Eiren Rain and SlimeVR Contributors" or "SlimeVR Contributors".

We cannot proceed with the relicensing until you do this 😅

the linked got a comment by this person earlier today - closing


Make sure controller-manager image gets pushed

We've been building binaries and images for the new controller-manager in parallel with the old controller... but the push target wasn't pushing them. That's made installation tricky when deploying images built from source. This fixes that.

Merging, shouldn't be controversial.


Exploration of an attempt to improve experience around transforming loggers

Inspired by the tedium of #835

Replaced by https://github.com/typelevel/log4cats/pull/854


Revert dispatcher stats

This reverts:
109d23a6648b1ac7723c4b2f125e913125bb9a84
bad70bf1625ad5809f2b32be62b4aa4c33a6741d
77748b20c5157a26afd63dcf28ceb8d8cf1e3823
@mergeconflict as I mentioned in Slack (https://envoyproxy.slack.com/archives/C78HA81DH/p1555624409062200), for statsd backends this writes raw histograms out to statsd at an extremely high rate which is not optimal. We should probably make this feature opt-in for perf reasons, but we need to either have a giant warning about statsd or we need to use an in-memory histogram for this data and maybe just write out summary data to the stats sinks? WDYT?
Either way I would prefer to revert for now if that is OK with you while we figure out the right solution.

cc @tonya11en


Fix alt-f and alt-b

Thanks for the atom keymap! I noticed one small mistake with alt-f and alt-b. It looks like these commands should begin with editor: instead of core:

Thank you for the contribution! :+1:


Checkpoint Challenge Required

Describe the bug
I want to use watch_user_likers_stories.py
when i login to my account i get
Checkpoint Challenge Required!
I was disable 2fa (sms and app)
but i cant login and i get challenge in instagram but, and when i accept that, it dosent work and python bot cant login and send challenge for second , third and ......
Screenshots

log
2020-01-21 22:15:23,597 - INFO - Instabot Started
2020-01-21 22:15:25,866 - INFO - PRE-LOGIN FLOW!...
2020-01-21 22:15:31,646 - ERROR - Request returns 400 error!
2020-01-21 22:15:31,647 - INFO - Instagram's error message: challenge_required
2020-01-21 22:15:31,647 - INFO - Error type: checkpoint_challenge_required
2020-01-21 22:15:31,648 - INFO - Checkpoint challenge required...
2020-01-21 22:15:31,649 - INFO -
            Going to get your likers and watch their stories (and stories
            of their likers too). You can specify username of another user
            to start (by default we use you as a starting point).

Expected behavior
login to account and watch stories
Additional context
Im using python 3.7.6
Windows 7
Tnx for help

did you manage to solve? I have the same problem


Add support for ControllerConfigs

Also changes the dataurl encoding to more closely match what's done by
the encoder used in MCO
These changes should prevent diffs in the machine config and hopefully
avoid reboot trigger by MCO

/test e2e-aws-ovn-single-node-recert-parallel

/retest

/retest

/test e2e-aws-ovn-single-node-recert-parallel

/test e2e-aws-ovn-single-node-recert-parallel

/retest

/retest

/test e2e-aws-ovn-single-node-recert-serial


Page last modified date incorrect

eg https://cert-manager.io/next-docs/configuration/acme/http01/

Last modified January 1, 0001

To be fair ACME could well have been written by Jesus
/kind bug


"when I try to send a message, the text is reset" from Discord

Discord thread
Help me figure it out, it doesn't work for me, when I try to send a message, the text is reset
I also catch errors "A Python virtual enviroment cannot be activated because running scripts is disabled for this user. In order to use Continue, please enable signed scripts to run with this command in PowerShell: Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser, reload VS Code, and then try again."
and
Continue requires Python3. Please install from https://www.python.org/downloads, reload VS Code, and try again.
I tried to fix it as written in the error but the errors don't go away

there are no active connections on port 65432, the command produces no output
Loading Continue server...
attempts to connect to the server do not give a positive response

@irekgroup Since this comment 8 hours ago, I believe I have resolved this issue.
However, even if not, I've added the ability to view the server logs, which would expose the issue. You can see these by using the "Continue: View Logs" command in the VS Code command palette (cmd+shift+p).
If still not working, it would also be helpful to know operating system. Hoping to get you past this barrier asap—thanks!

a log fragment from the application, Visual Studio Code
2023-07-29 16:05:13.606 [error] [xabikos.JavaScriptSnippets]: Unknown language in `contributes.JavaScriptSnippets.language`. Provided value: vue
2023-07-29 16:05:16.110 [info] [perf] Render performance baseline is 83ms
2023-07-29 16:05:19.136 [info] UNRESPONSIVE extension host: starting to profile NOW
2023-07-29 16:05:20.445 [info] UNRESPONSIVE extension host: received responsive event and cancelling profiling session
2023-07-29 16:05:33.381 [error] cannot open file:///c%3A/Users/allab/.continue/continue.log. Detail: Unable to read file 'c:\Users\allab\.continue\continue.log' (Error: Unable to resolve nonexistent file 'c:\Users\allab\.continue\continue.log'): Error: cannot open file:///c%3A/Users/allab/.continue/continue.log. Detail: Unable to read file 'c:\Users\allab\.continue\continue.log' (Error: Unable to resolve nonexistent file 'c:\Users\allab\.continue\continue.log')
    at e.$g0.$tryOpenDocument (vscode-file://vscode-app/c:/Users/allab/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2306:36550)

@irekgroup thanks for all of this extra information. A quick update: I'm working here on a solution to bundle the python code as an executable, which I expect to solve this issue. Will let you know as soon as this is done

@irekgroup I believe you've already been @'ed in another issue, but let me know about the status of this. A version that packages an executable is now released (should fix this), and I'd like to either close this issue if resolved.

This error https://github.com/continuedev/continue/issues/310#issue-1824787943 now clearly not.
It's different now. It looks like there is a problem with the OpenAI API. The model 'gpt-4' does not exist or you do not have access to it. Please check the OpenAI documentation to learn more.
This is a default error that I got when I first started continue
Traceback (most recent call last):

  File "continuedev\src\continuedev\core\sdk.py", line 46, in create
    config = sdk._load_config_dot_py()

  File "continuedev\src\continuedev\core\sdk.py", line 169, in _load_config_dot_py
    path = getConfigFilePath()

  File "continuedev\src\continuedev\libs\util\paths.py", line 39, in getConfigFilePath
    with open(path, 'r') as f:

FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\allab\\.continue\\config.py'

I got this error when trying to send a question to gpt
Traceback (most recent call last):

  File "continuedev\src\continuedev\core\autopilot.py", line 248, in _run_singular_step
    observation = await step(self.continue_sdk)

  File "continuedev\src\continuedev\core\main.py", line 319, in __call__
    return await self.run(sdk)

  File "continuedev\src\continuedev\plugins\steps\chat.py", line 35, in run
    async for chunk in generator:

  File "continuedev\src\continuedev\libs\llm\maybe_proxy_openai.py", line 49, in stream_chat
    async for item in resp:

  File "continuedev\src\continuedev\libs\llm\openai.py", line 116, in stream_chat
    async for chunk in await openai.ChatCompletion.acreate(

  File "openai\api_resources\chat_completion.py", line 45, in acreate

  File "openai\api_resources\abstract\engine_api_resource.py", line 217, in acreate

  File "openai\api_requestor.py", line 310, in arequest

  File "openai\api_requestor.py", line 646, in _interpret_async_response

  File "openai\api_requestor.py", line 683, in _interpret_response_line

openai.error.InvalidRequestError: The model `gpt-4` does not exist or you do not have access to it. Learn more: https://help.openai.com/en/articles/7102672-how-can-i-access-gpt-4.

@irekgroup The first error is known, and a solution was just released.
As for the latter, it might be possible that your OpenAI key doesn't have access to GPT-4. If you use "gpt-3.5-turbo" might this error disappear?
(you can edit this in config.py by changing the "default=..." line in "Models" to use model="gpt-3.5-turbo" instead of "gpt-4".)

Now everything fell into place, but before that I had to do:
1. Run netstat -ano | find "65432"
2. After you've got the PID, type the following command but replace "<PID>" with the actual PID value: taskkill /F /PID <PID>. 3. For example, if the PID is 1234, you would type: taskkill /F /PID 1234
Then restart VS Code

After that, I deleted .continue, and reinstalled Continue in CAPE, and only then I got a .config file that I was able to edit, hooray!
Can I ask a question,
Is there a VSC hotkey to call this function?

That's great! We'll have to work on making sure all this debugging isn't necessary, but glad you got it working.
You can use command+period when your cursor is over text with a red/yellow underline to bring up quick fixes (VS Code docs).
Other shortcuts for Continue are:

command+m to toggle the text entry box
command+shift+m to toggle text entry with /edit prefixed so that it will edit code instead of just answering
If you drag the Continue icon from the left side of the screen to the right, you can keep the window pinned at the right as in this screenshot below. Then you can use option/alt+command+m to toggle that window


chore: default home directory for single-node.sh

Closes https://github.com/celestiaorg/celestia-app/issues/2404
I frequently need to modify a config and test it out. When using the existing single-node.sh script, it's a bit tedious to find the randomly generated CELESTIA_APP_HOME directory and plumb that into subsequent ad-hoc commands.
This PR modifies the single-node.sh script to use the default CELESTIA_APP_HOME directory (usually ~/.celestia-appd).  That way, if a developer needs to make a modification to a config, they can do that in the expected location (i.e. ~/.celestia-appd/config/config.toml) and then re-run celestia-appd start and test out the modification.
Other improvements include:

Adds a y/n prompt to confirm deleting any existing CELESTIA_APP_HOME directory
Hoists variables to the top
Silences noisy terminal output by redirecting to dev/null

In addition to reviewing the code diff, it may make sense for reviewers to try the script and see if it behaves as expected.

To provide more context: @mojtaba-esk made it use a randomly generated one in https://github.com/celestiaorg/celestia-app/pull/1583
Since we have two scripts that use randomly generated home directories, proposal for one of them to not use a randomly generated one so that a developer can modify configs across runs. More context in https://github.com/celestiaorg/celestia-app/issues/2404


Cross build sbt (1.0.4 and 0.13.16)

Use scala 2.12 for sbt 1

This fixes issue #56 (also supports sbt 0.13.16)

Thanks a lot! I was able to make the companion example work with both sbt 0.13/scala 2.11/play 2.5 and sbt 1/scala 2.12/play 2.6.


Nested dropdown/menu does not expand on hover

Steps to Reproduce
Check out the "Attached" example for dropdown in documentation
Expected Behavior
"New" menu should expand and show additional options on hover
Actual Behavior
Nothing opens
Version
Stardust: master
Browser: Chrome
OS: Mac
Test Case
Documentation

Though we do not currently support nested Dropdown menus, please provide a link for the test case.  Without a test case, there isn't a clear path for a developer to work on it.  Thanks much.

The test case is the documentation: "attached.js"

Sorry, I'll need more specificity please.  There is no "Attached" Dropdown example on our doc site for the Dropdown: http://technologyadvice.github.io/stardust/modules/dropdown
The only two files named Attached.js are:

Perhaps you are referring to the Semantic-UI main repo?

https://github.com/TechnologyAdvice/stardust/blob/master/docs/app/Examples/collections/Menu/Types/Attached.js

Got it, thanks!

I believe this is an issue with the createIcon factory.  It is returning empty icons.  There is a similar issue with the regular Dropdown example's DropdownItem's:
http://technologyadvice.github.io/stardust/modules/dropdown#dropdown
I'll start on this now, thanks again for the report.

Not sure it is related, I don't see anywhere where hover events are wired up. I am about to submit a pull in a few minutes that gets it to work, but it definitely feels like a hack :)

Sure, thing let's split up.  I'll fix the icon factory issue.
I'm not sure if the Dropdown is supposed to open on hover via CSS or JS via a Dropdown setting.  If the latter, then the Dropdown component needs extended as we do no currently support this.

OK, the menu issue is that the augmentation simply needs reversed:
<Dropdown as={Menu.Item} icon='wrench'>

Instead of:
<Menu.Item as={Dropdown} icon='wrench'>

@levithomason - looks like the issue with the dropdown item icon is due to this change:
- {iconName && <Icon name={iconClasses} />}
+ {createIcon(iconName)}

iconName is defined as:
iconName = icon || childrenUtils.someByType(children, 'DropdownMenu') && 'dropdown'

which can be false since childrenUtils.someByType returns a boolean. false isn't handled ignored by factories.
Two options:

update factories to ignore _.isNil(val) || val === false
update iconName= here to be childrenUtils.someByType(children, 'DropdownMenu') ? 'dropdown' : undefined

Thanks for the investigation. I've added return null to factories for boolean values.  I think this is sane.  They accept string, object, or element only.  I'll push shortly.


Fix e.detail.keyboardHeight undefined error in keyboardAttach directive

My background:

newest ionic
newest ionic cordova keyboard plugin installed
iOS 8 + iPhone 5C
debugging through Safari Emulator

What happens:
When focusing on the input in <ion-footer-bar keyboard-attach>, I get a TypeError: undefined is not an object (evaluating 'e.detail.keyboardHeight') error.
This bugfix fixes it.

Bump, I have this same bug.

I've no idea.

Tracking for 1.2, thanks

Err, one liner easy to merge :smile:  :beers:


NullPointerException in ControlTransportPoller

The following code from ControlTransportPoller:
selector.selectNow();

for (final SelectionKey key : selectedKeySet.keys())
{
    bytesReceived += ((SendChannelEndpoint)key.attachment()).pollForData();
}

selectedKeySet.reset();

Fails to use the size of the selecteKeySet as bounds on the loop.  Instead a pre-sized array is return that main contain null values.

Fixed in https://github.com/real-logic/Aeron/commit/c8f21e8d04a90cb1d891fcdab23c6be119dfa5bc


[Bug] All notes are suddenly gone ; Feature Request : Backup Function

I don't think I delete anything. I'm not sure whether prolonged save is a feature to be expected here and all of the notes are only there as a cache. Is it possible to back all th notes that is created as a .md / .txt file? I didn't even turn off my PC and all the notes are gone.
p.s : Any information regarding how the data are stored up would be much appreciated.

ahh yes, i found it.. but do you have any idea why it suddenly got deleted? thankyou for the response

I'm going with similar questions. I tried creating a symlink from my Nextcloud folder where I put the notes folder from Sticky, however it won't recognize the symlink as a real notes folder. Being able to use symlinks would be pretty neat!

do you have any idea why it suddenly got deleted?

Not sure.

Being able to use symlinks would be pretty neat!

I agree!

however it won't recognize the symlink as a real notes folder.

please create a new issue requesting that we follow symlinks to the notes folder!


Add support for overwriting the hostname and/or certificate thumbprint on the listener - Fixes #23

Pull Request (PR) description
This PR adds support for overwriting the hostname and/or certificate thumbprint on an HTTPS listener if a listener already exists with a different hostname and/or certificate thumbprint.
This Pull Request (PR) fixes the following issues

Fixes #23

Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[x] New/changed code adheres to DSC Community Style Guidelines.

This change is

/AzurePipelines run

@corydwood can you rebase this PR with the changes in main so the test pipeline can run? There were fixes merged for the pipeline to work again.

@corydwood forget my last comment, seem it picked up the changes without needing a rebase 🤔
@PlagueHO this is ready for review then.

@PlagueHO all API keys should be up to date, if not ping me and I can fix.

@johlju - sorry mate, looks like the API keys for PS Gallery need an update: https://dev.azure.com/dsccommunity/WsManDsc/_build/results?buildId=8181&view=logs&j=5d0a9c4e-8741-5118-af45-406a30fe661c&t=bda0b158-a237-516f-c1d0-31339587c5b5&l=117

@PlagueHO fixed new API key and restarted the pipeline for the main branch. It should deploy now. 🙂


Handle ad-blocked GitHub star widget

This change adjusts the layout of the main nav to hide the word “Star” and right-align remaining nav elements when the GitHub star-widget script fails to load.
Companion to https://github.com/pulumi/www.pulumi.com/issues/279.
Signed-off-by: Christian Nunciato<EMAIL_ADDRESS>

Merging based on the approval & merge of https://github.com/pulumi/www.pulumi.com/pull/287.


Remove Gmail signature

Hello,
I was searching for a way to use gam tool for removing end user' signature on Gmail, completey:

Unassign on new emails
Unassign on reply/forward
Delete all signatures

Thank you,
Luca

Hello @jay0lee,
excuse me, what did I miss ?
Thank you,
Luca

Hello @jay0lee,
excuse me, what did I miss ?
Thank you,
Luca

Luca,
There is no Google Gmail API support for this, thus GAM can't help.
Ross

Luca,
There is no Google Gmail API support for this, thus GAM can't help.
Ross


Implement commands

PIV standard

[x] SELECT
[x] GET DATA - #5

[x] Mandatory Data elements:

[x] Card Capability Container
[x] Card Holder Unique Identifier

[x] asymmetric Signature Field

[x] X.509 Certificate for PIV Authentication
[x] X.509 Certificate for Card Authentication
[x] Cardholder Fingerprints
[x] Cardholder Facial Image (only up to 1KB)
[x] Security Object

[ ] Conditional Data elements:

[x] X.509 Certificate for Digital Signature
[ ] X.509 Certificate for Key Management

[ ] Optional Data Elements

[x] Printed Information (Not applicable)
[x] Discovery Object
[x] Key History Object
[x] Retired X.509 Certificates for Key Management
[x] Cardholder Iris Images
[x] Biometric Information Templates Group Template
[x] Secure Messaging Certificate Signer
[x] Pairing Code Reference Data Container UUIDs:
[ ] Card UUID

[x] VERIFY

[x] Global Pin
[x] PivPin
[ ] Primary finger OCC (Optional)
[ ] Secondary finger OCC (Optional)
[ ] Pairing Code (for NFC only)

[x] CHANGE REFERENCE DATA

[x] PivPIn
[x] Puk

[x] RESET RETRY COUNTER
[x] GENERAL AUTHENTICATE

[x] Witness
[x] Challenge - #3
[x] Response - #3
[x] Exponentiation

[x] PUT DATA - #5
[x] GENERATE ASYMMETRIC KEYPAIR

Yubico extensions

[x] Set management key
[ ] Import asymmetric key https://github.com/Nitrokey/piv-authenticator/issues/41
[x] Get Version (Not applicable?)
[x] Reset
[ ] Set Pin Retries
[ ] Attest (Not applicable?)
[ ] Get Serial (Not applicable?)
[ ] Get Metadata

New YubiKey extensions as part of their 5.7 firmware update should be probably added to this list:

[ ] Move keys between slots
[ ] Delete keys from slots
[ ] AES management keys
[ ] New algorithms

[ ] RSA 3072
[ ] RSA 4096
[ ] X25519
[ ] Ed25519

See: https://docs.yubico.com/hardware/yubikey/yk-tech-manual/5.7-firmware-specifics.html#piv-enhancements


Manual Install?

hi, great library!
I am trying to manually install the files in my project, but there's no documentation regarding how to do so.
I would assume to copy all the files within the Sources/GradientCircularProgress folder, but I can't seem to import GradientCircularProgress as Xcode can't find it.
Here's what I have in my project:

I am receiving this error in BackgroundTransparentStyle.swift:

Do you happen to know what else I am missing?
Thanks!
PS: I've also copied the framework, but to no avail.

SOLVED!
Just FYI for anyone else who wants to use this library the manual way, make sure to drag the GradientCircularProgress.framework into Embedded Libraries as shown:


Open by src

Load a script given url.

Priority

data-src will load.
data-target will load if data-src is not exist.

Won't fall back, throw an error.
screen: Set src
editor: Set code

Unify data-src
Define a script with fake path. Like this


chore: Support redis cluster

Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

Codecov Report

Merging #6 (de7c2a1) into main (920b0ec) will increase coverage by 1.48%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   78.22%   79.71%   +1.48%
==========================================
  Files           1        1
  Lines         124      138      +14
==========================================
+ Hits           97      110      +13
- Misses         21       22       +1
  Partials        6        6

Impacted Files
Coverage Δ

redis.go
79.71% <92.30%> (+1.48%)
:arrow_up:

Continue to review full report at Codecov.

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


Travis CI test fails with Python 3.9

Travis CI now also tests with Python 3.9.
The output reveals that the XML Parser used by NLTK's corpus.reader is no longer compatible with the newest Python 3.9 version.
So all NLTK builds will fail Travis CI testing with Python 3.9 until this is fixed:
=================================== FAILURES ===================================
___________________________ [doctest] corpus.doctest ___________________________
615
616 semcor
617 ------
618 The Brown Corpus, annotated with WordNet senses.
619
620     >>> from nltk.corpus import semcor
621     >>> semcor.words('brown2/tagfiles/br-n12.xml')  # doctest: +ELLIPSIS
622     ['When', 'several', 'minutes', 'had', 'passed', ...]
623     >>> sent = semcor.xml('brown2/tagfiles/br-n12.xml').findall('context/p/s')[0]
624     >>> for wordform in sent.getchildren():
UNEXPECTED EXCEPTION: AttributeError("'xml.etree.ElementTree.Element' object has no attribute 'getchildren'")
Traceback (most recent call last):
File "/opt/python/3.9.1/lib/python3.9/doctest.py", line 1336, in __run
exec(compile(example.source, filename, "single",
File "<doctest corpus.doctest[135]>", line 1, in
AttributeError: 'xml.etree.ElementTree.Element' object has no attribute 'getchildren'
/home/travis/build/nltk/nltk/nltk/test/corpus.doctest:624: UnexpectedException
920 rte
921 ---
922 The RTE (Recognizing Textual Entailment) corpus was derived from the
923 RTE1, RTE2 and RTE3 datasets (dev and test data), and consists of a
924 list of XML-formatted 'text'/'hypothesis' pairs.
925
926     >>> from nltk.corpus import rte
927     >>> print(rte.fileids()) # doctest: +ELLIPSIS
928     ['rte1_dev.xml', 'rte1_test.xml', 'rte2_dev.xml', ..., 'rte3_test.xml']
929     >>> rtepairs = rte.pairs(['rte2_test.xml', 'rte3_test.xml'])
UNEXPECTED EXCEPTION: AttributeError("'xml.etree.ElementTree.Element' object has no attribute 'getiterator'")

getchildren and getiterator have been deprecated since 2.7 and 3.2 respectively. We need to use list(elem) or iteration instead of getchildren, and iter instead of getiterator.


Jenkins on Azure overview landing page references outdated material

The Jenkins on Azure overview landing page, which is the first page I looked at for learning how to use Jenkins on Azure, references the Azure VM Agents Jenkins plugin which is a plugin that will be deprecated in 2024.
This page should be updated to included the new recommended way to setup Jenkins on Azure.

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

ID: 6a437702-cc4c-86c4-25ba-2a0365737d82
Version Independent ID: 318fc29a-87f6-784f-918b-6ec7ec7ed282
Content: Overview of Jenkins and Azure
Content Source: articles/jenkins/overview.md
Service: jenkins
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@Minreaux Thanks for your feedback. Since the plugin you mention is supported for 2 more years, it is the recommended way. When the plugin gets closer to retirement, we'll have a transition path we'll document.


claim-config.json values are not updating

Updating title fields in claim-config.json files don't update the website pages.
For example, I tried to test by editing the "First Name" title fields as seen in the screenshot below, but after running ./manage build and ./manage start, the website still shows "First Name"

I also tried first removing the old containers in docker in case they were being reused and not updated by calling:
docker rm issuer-kit_issuer-admin_1
docker rm issuer-kit_issuer-web_1
And then rerunning the build and start commands

Your browser might be caching the content, please try clearing the cache and/or force-refreshing the page to see the new config.
The Caddy webserver should be able to detect the file was updated and serve the new one, but sometimes the browser still doesn't refresh the cached content - unfortunately.

Awesome clearing my cache worked perfectly. Thanks for the quick help, and your work on this solution!


Feature to profile Crons commands?

Hi,
I was wondering if there is a feature to profile crons, or if there is a way to do it.
I noticed silk will only work when there is a request generated by a view, so is there a way to override this file so that it will also work on crons?
We love silk but we have some long running crons that we would like to debug in production.
If is not possible, could I get some minor directions on how it could be implemented so that I can create a pull request for it?
I guess after this is implemented, it will not be too hard to do a similar thing for celery, I will be happy to create both pull requests if someone could tell me more and less how you would like it to be done so that the pull request gets approved and merged.
Thank you! Congrats for the library it is really useful!

Duplicate of #593

FYI I found this GitHub gist that you can use to run django-silk against arbitrary functions not wrapped in APIs. This was a good-enough workaround for me to debug some performance challenges, although it would be fantastic to see something supported by django-silk in the future.


feat(auth): Add auth and unauth methods

Part of #4

@jteplitz602 great PR! It brings up some questions of how to best abstract Firebase.
The current implementation essentially proxies to Firebase SDK methods, so any component in the app that would want to log in would need to know the right method and configuration to use to log in.
// This would probably be imported from a config, or injected
const provider:AuthProviders = AuthProviders.Github;

@Component({
  selector: 'my-login'
})
export class MyLogin {
  constructor(private _af:AngularFire) {}
  login () {
    this._af.authWithOAuthPopup(provider);
  }

  logout () {
    this._af.unauth()
  }
}

Could you explore moving some of the options to injectable configuration, and make the auth service just have login and logout? The login method should accept options to override defaults of course. For example:
bootstrap(App, [
  FIREBASE_PROVIDERS,
  defaultFirebase('http://somefirebase'),
  defaultAuthProvider(AuthProviders.Github),
  // We could even have a default if we find that one method is far more common than others
  defaultAuthMethod(AuthMethods.PopUp)
]);

@Component({
  template: `
    <button (click)="af.auth.login()">Login with Github</button>
    <button (click)="twitterLogin()">Login with Twitter</button>
  `
})
class App {
  constructor(public af:AngularFire) {}
  twitterLogin() {
    // Would still use default method of PopUp
    this.af.auth.login({provider: AuthProviders.Twitter});
  }
}

Maybe you've already given this approach of "fewer methods+more configurability" some thought and have some reasons why it provides a worse user experience.

@jeffbcross In general I love the idea of reducing the number of public methods, but I'm not sure it will work here. That design works well for popup auth, redirect auth, and anonymous auth. But it runs into problems with customToken, email, and oauthToken login. These methods all require specific arguments. It feels weird to me that your call to login could suddenly become invalid and throw an error if you change your default auth method in your bootstrap file. Especially because we wouldn't be able to provide static errors for this since all the arguments to login would have to be optional.
We could separate the firebase login methods into two groups: automatic login (popup, redirect, and anonymous) and manual login (customToken, email, and oauthToken). We could provide one generic login method for the first group and then three manual login methods for the second group, but that might be overkill. What do you think?

I've never actually implemented Firebase's login, please forgive my naivety!
The auth service seems to be inherently stateful, and the internal details of providers, credentials, and tokens are nearly constant through the entire life of the app.
Could we possibly have a separate means of providing options (like custom OAuth tokens, passwords/emails, etc.) with some kind of setOptions() method? In a well designed app, those details should be in a single component/service that manages the user's session.
I'm assuming the .login() or .auth() methods are used for e.g. @CanActivate and other conditional behavior. The methods should be as simple as possible; keep the options separate, since they'll rarely change.
Please correct anything I've got backwards!

@TylerEich thanks for the input!
I just chatted with @jeffbcross about this. Here's what we decided on. We're going to have one login method that by default uses options provided through DI. So if your app generally authenticates users by doing Github login with a popup you would provide these options to DI using something like
// Providers to give bootstrap
provide(DefaultFirebaseAuthProvider, AuthProviders.Github),
provide(DefaultFirebaseAuthMethod, AuthMethods.Popup)

// to login
auth.login();

However, all of these options can be overwritten by an optional config object that gets passed into login. So if you actually wanted to use Facebook instead of Github you could do:
// Providers to bootstrap
provide(DefaultFirebaseAuthProvider, AuthProviders.Github),
provide(DefaultFirebaseAuthMethod, AuthMethods.Popup)

// this will login the user with Facebook popup since Facebook will override Github, but nothing overwrote Popup
auth.login({
  provider: AuthProviders.Facebook
});

login will also take a second optional argument for credentials. So if you're using customToken, email, or OAuthToken login  you'll do something like this to login:
auth.login({
  provider: AuthProviders.Password
}, {
  email<EMAIL_ADDRESS>  password: 'password'
})

@jteplitz602 +1 I think we could drop the Default from the provider name.

Talked to @jeffbcross and @hansl a bit more. We're going to create one config provider, and have two signatures for the login method. I'll write up a longer description this afternoon.

@jteplitz602 @jeffbcross yeah, I like that design better. Keeps the API clean and centralized 😎👍

Here's a more complete overview of the design that we settled on:
You can, but are not required to, provide a single set of configuration arguments to DI like this:
bootstrap(app, [
  FIREBASE_PROVIDERS,
  defaultFirebase('https://<some-firebase>.firebaseio.com'),
  firebaseAuthConfig({
    provider: AuthProviders.Facebook,
    method: AuthMethods.Popup,
    remember: 'default',
    scope: ['email']
  })
]);

Once you've done that you can simply call login on the auth object. This will automatically use the options that were configured with DI. You can override those options by providing an optional configuration object to the login method like so:
// This will perform popup auth with google oauth and the scope will be email
auth.login({
  provider: AuthProviders.Google
});

The Auth service will use all values provided by the given configuration object over those provided through DI, and will fall back to the DI provided config object (if available) for any options not provided by the caller.
Additionally, if you're using OAuthToken, CustomToken, or Password auth then you can pass a credentials object to login like so:
bootstrap(app, [
  FIREBASE_PROVIDERS,
  defaultFirebase('https://<some-firebase>.firebaseio.com'),
  firebaseAuthConfig({
    method: AuthMethods.Password,
    remember: 'default'
  })
]);
auth.login({
  email<EMAIL_ADDRESS>  password: 'password'
});

If you want to pass both a credentials object and a configuration object you can do so like this:
auth.login({
  method: AuthMethods.Password,
  remember: 'sessionOnly'
}. {
  email<EMAIL_ADDRESS>  password: 'password'
});

If two arguments are passed to login than the service will assume that the first one is a configuration object and the second one is credentials. If only one is passed then it will assume that argument is a configuration object unless the auth method is set to Password, OAuthToken, or CustomToken in which case it will assume that object is a credentials object since credentials are not optional for any of those three methods and the configuration object is always optional.
@jeffbcross does that sound like a good overview of everything we discussed?

LGTM with comments addressed


Bootstrap unit test gen for nodejs

Add happy case support for simple and paged methods
Add value producer for NodeJS
Reuse NodeJS sample snippets for initialization code
Baseline test

TODO:

Test exceptions
Add support for Grpc streaming methods

Current coverage is 82.15% (diff: 90.99%)

Merging #825 into master will increase coverage by 0.10%

@@             master       #825   diff @@
==========================================
  Files           321        323     +2
  Lines         12384      12493   +109
  Methods           0          0
  Messages          0          0
  Branches       1621       1636    +15
==========================================
+ Hits          10161      10264   +103
- Misses         1801       1803     +2
- Partials        422        426     +4

Powered by Codecov. Last update c6e7123...2599a9d

@bjwatson @garrettjonesgoogle ptal

generator code LGTM

@jmuk I have addressed all comments. PTAL


NVIDIA GPU with Posit Connect apps

Ey up!
We're trying to get GPU to work with Posit Connect. This is of the Kubernetes flavour, with AWS's EKS.
So we've developed an app that uses CUDA (NVIDIA) code through PyTorch.
The machine type with the AWS EKS is g4dn, which has been setup with AMI that supports both EKS and GPU.
Posit documentation isn't 100% clear but my understanding is that the way to ask Posit Connect to use GPU is through this configuration in the Helm chart:
launcher:
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1

However with the app deployment, we're seeing this error in the pod logs:
0/4 nodes are available: 4 Insufficient amd.com/gpu.
which perplexed me, because we've not asked for AMD but NVIDIA, so I checked the manifest for the pod, and saw this:
    Limits:
      amd.com/gpu:     1
      nvidia.com/gpu:  1
    Requests:
      amd.com/gpu:     1
      nvidia.com/gpu:  1

I "SSHed" into the pod and looked in /etc/rstudio-connect/launcher/launcher.kubernetes.profiles.conf:
[*]
default-amd-gpus=0
default-nvidia-gpus=1

I tried explicitly asking that AMD be excluded with:
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1
      default-amd-gpus: 0 (and 0.0)
      max-amd-gpus: 0 (and 0.0)

however they still appeared with 1 and not 0:
    Limits:
      amd.com/gpu:     1
      nvidia.com/gpu:  1
    Requests:
      amd.com/gpu:     1
      nvidia.com/gpu:  1

I tested outside of Posit Connect by creating a pod manifest ourselves:
cat <<EOF | kubectl apply -f -
apiVersion: v1
kind: Pod
metadata:
  name: gpu-pod
spec:
  restartPolicy: Never
  containers:
    - name: cuda-container
      image: benfletcherft/rstudio-content-base:r4.1.0-py3.9.2-ubuntu1804-cuda11.4.3
      resources:
        limits:
          nvidia.com/gpu: 1
  tolerations:
  - key: nvidia.com/gpu
    operator: Exists
    effect: NoSchedule
EOF

and it worked great.
I tried to find where the manifest gets generated, however it seemed to me that this particular piece wasn't open source and therefore I was at a loss as to how to get this working. :( I wonder whether there was a bug somewhere with the manifest generator that accidentally produced the AMD config alongside NVIDIA?
All the best,
Ben

Helm chart: 0.3.18
Posit Connect: v2023.03.0
Build v2023.03.0-0-g927f384
As to your latter question, I believe that is how it works - values from the Helm chart eventually go into /etc/rstudio-connect/launcher/launcher.kubernetes.profiles.conf but with incorrect outcome (AMD added when it shouldn't).

Sorry, we have all of our Kubernetes experts out at the moment. I'm looking into the issue, but it's taking longer than I would like to learn the ropes.
When @atheriel says "are you using the templating feature", I believe he means a section like this in your helm chart values YAML file:
launcher:
  useTemplates: True
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1

The default value is True, so I am pretty sure the answer is yes.
It looks to me like a launcher bug, rather than a problem with the helm charts, although the launcher code in question is much older than the helm charts. The template code that is in use has not changed in roughly 2 years.
Sorry for the delay. Hopefully we can get you more info early next week.

Ey up @AlexMapley! Thanks so much for looking into this.
I've got some more information that may help.
I've upgraded our Helm chart to 0.4.0. The sidebar in Connect UI now says:
Posit Connect v2023.03.0
Build v2023.03.0-0-g927f384

The values.yml launcher block:
launcher:
  DataDirPVCName: rsc-pvc
  enabled: true
  namespace: rsc
  templateValues:
    pod:
      affinity:
        podAntiAffinity:
          preferredDuringSchedulingIgnoredDuringExecution:
          - podAffinityTerm:
              labelSelector:
                matchExpressions:
                - key: connect.rstudio.com/content-guid
                  operator: Exists
              topologyKey: kubernetes.io/hostname
            weight: 100
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1

That is the full launcher block, I've not omitted anything from here. You can refer to support ticket number 88353 regarding the affinity section along with the full values.yml.
I SSHed into the Connect pod:
$ kubectl exec -it -n rsc rstudio-connect-6cb5748f46-v4fmd -- bash
Defaulted container "connect" out of: connect, exporter
root@rstudio-connect-6cb5748f46-v4fmd:/# cat /etc/rstudio-connect/launcher/launcher.kubernetes.profiles.conf

[*]
default-nvidia-gpus=1
max-nvidia-gpus=1

which surprised me - it's good news as I expected to see the AMD lines as I mentioned previously, so looking like this problem is now gone.
HOWEVER...
The API content items are still not starting up. So I checked one of the API pods:
$ kubectl -n rsc describe pod run-python-application-7xhdx-w44mj
Name:           run-python-application-7xhdx-w44mj
Namespace:      rsc
Priority:       0
Node:           <none>
Labels:         app.kubernetes.io/component=k8s-launcher
                app.kubernetes.io/name=rstudio_connect
                app.kubernetes.io/version=2023.03.0
                connect.rstudio.com/bundle-id=199
                connect.rstudio.com/content-guid=17a68aef-1da6-4f52-afba-feb166d1bd4c
                connect.rstudio.com/content-id=3
                connect.rstudio.com/job-key=HPI1K4Jt57SZzxwV
                connect.rstudio.com/job-tag=run_fastapi_app
                connect.rstudio.com/python-version=3.9.2
                controller-uid=d2930c98-a810-45df-ae4b-fc0c8bf23169
                job-name=run-python-application-7xhdx
Annotations:    connect.rstudio.com/server-address: https://connect.in.ft.com/
                kubernetes.io/psp: eks.privileged
                name: run Python application
                service_ports: [{"targetPort":3939,"protocol":"TCP"},{"targetPort":50734,"protocol":"TCP"}]
                stdin:
                  {"Environment":[{"name":"CONNECT_SERVER","value":"hCLF3HVz98xwWzGie0J/ncyIQ+5FLVFVmz1fv0zDthxazI4uoXM5FEcCJdll3bOHSEtirKZMgVYjSxlTvdqa8pcJ...
                user: rstudio-connect
                user_metadata:
                  {"job":{"annotations":{"connect.rstudio.com/server-address":"https://connect.in.ft.com/"},"labels":{"app.kubernetes.io/component":"k8s-lau...
Status:         Pending
IP:
IPs:            <none>
Controlled By:  Job/run-python-application-7xhdx
Init Containers:
  init:
    Image:        ghcr.io/rstudio/rstudio-connect-content-init:bionic-2023.03.0
    Port:         <none>
    Host Port:    <none>
    Environment:  <none>
    Mounts:
      /mnt/rstudio-connect-runtime/ from rsc-volume (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-kgvpr (ro)
Containers:
  rs-launcher-container:
    Image:      ghcr.io/rstudio/content-base:r4.0.5-py3.9.2-bionic
    Port:       <none>
    Host Port:  <none>
    Command:
      /opt/rstudio-connect/ext/env-manager
    Args:
      /opt/rstudio-connect/scripts/with-locale.sh
      /opt/rstudio-connect/ext/rsc-session
      -d
      /opt/rstudio-connect/mnt/job
      -a
      <IP_ADDRESS>:50734
      -i
      600
      -g
      2
      -b
      content-guid:17a68aef-1da6-4f52-afba-feb166d1bd4c
      -b
      content-id:3
      -b
      bundle-id:199
      /opt/python/3.9.2/bin/python3.9
      /opt/rstudio-connect/python/run_app.py
    Limits:
      amd.com/gpu:     1
      nvidia.com/gpu:  1
    Requests:
      amd.com/gpu:     1
      nvidia.com/gpu:  1
    Environment:
      USER:      rstudio-connect
      USERNAME:  rstudio-connect
      LOGNAME:   rstudio-connect
      HOME:      /tmp
      TMPDIR:    /tmp
    Mounts:
      /opt/rstudio-connect from rsc-volume (rw)
      /opt/rstudio-connect/mnt/app from mount0 (rw,path="apps/3/199")
      /opt/rstudio-connect/mnt/job from mount0 (rw,path="jobs/3/HPI1K4Jt57SZzxwV")
      /opt/rstudio-connect/mnt/python-environments from mount0 (ro,path="python-environments")
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-kgvpr (ro)
Conditions:
  Type           Status
  PodScheduled   False
Volumes:
  mount0:
    Type:       PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
    ClaimName:  rsc-pvc
    ReadOnly:   false
  rsc-volume:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  kube-api-access-kgvpr:
    Type:                    Projected (a volume that contains injected data from multiple sources)
    TokenExpirationSeconds:  3607
    ConfigMapName:           kube-root-ca.crt
    ConfigMapOptional:       <nil>
    DownwardAPI:             true
QoS Class:                   BestEffort
Node-Selectors:              <none>
Tolerations:                 amd.com/gpu:NoSchedule op=Exists
                             node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
                             nvidia.com/gpu:NoSchedule op=Exists
Events:
  Type     Reason             Age                  From                Message
  ----     ------             ----                 ----                -------
  Normal   NotTriggerScaleUp  2m48s                cluster-autoscaler  pod didn't trigger scale-up:
  Warning  FailedScheduling   15s (x3 over 2m50s)  default-scheduler   0/4 nodes are available: 4 Insufficient amd.com/gpu.

That tells me a few things.

the events section suggests that, again, it was looking for the AMD nodes when we had only NVIDIA nodes
the Limits, Resources and Tolerations sections suggest that configuration included requirements for AMD nodes

One ting that may be useful is the AMI we're using is:
name: amazon-eks-gpu-node-1.23-v20230304
ID: ami-014452f3ad6f7d021
(it's public image, so you should be able to access it)
The machine type we're using is g4dn.xlarge.
If you & Posit use a different machine type (for GPU) and/or a different AMI, let me know and we'll switch :)

I ran something a similar setup @bjfletcher  and always ended up with:
    Limits:
      nvidia.com/gpu:  1
    Requests:
      nvidia.com/gpu:  1

as my launcher container requests/limits - i haven't been able to reproduce the issue/
I've traced through the helm and lower level generic templating of the code - I couldn't find any bugs there yet. What I suspect is  that connect is somehow submitting a bad manifest to the launcher api, one that includes requests for both amd and nvidia gpus.
It's possible amd gpus are being specified elsewhere, outside our helm values. If you have the json/yaml configuration of that run-python-application-7xhdx-w44mj app that might hold the key, I'm wondering if we're somehow getting an extra amd gpu requested at that layer.

Ey up @AlexMapley! When you said JSON/YAML configuration of the pod, did you mean the output from kubectl -n rsc describe pod run-python-application-7xhdx-w44mj? (The full output is in my last comment) But I think you're after something different right? Do you have the command you'd like me to run? Cheers, Ben

I have tested this in the latest release and can confirm that this is a bug in the job-launcher. Using the following values in Connect's values file:
launcher:
  enabled: true
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1

And running any piece of content produces a pod with the following requests/limits:
    Limits:
      amd.com/gpu:     1
      nvidia.com/gpu:  1
    Requests:
      amd.com/gpu:     1
      nvidia.com/gpu:  1

@dbkegley I see the cause of this Launcher side. Do you want me to ship you a patch build to try?

Yes please, thanks @kfeinauer. I'll go ahead and confirm the fix but it's unlikely this will make it in to the 2023.05 release of Connect. I'll make a note that this is something we could address if there's a patch release for 2023.05.

I have just verified that this will be fixed in the next release of Connect v2023.05.0.
The following config:
launcher:
  launcherKubernetesProfilesConf:
    "*":
      default-nvidia-gpus: 1
      max-nvidia-gpus: 1

yields the expected requests and limits for content pods:
    Limits:
      nvidia.com/gpu:  1
    Requests:
      nvidia.com/gpu:  1

This is fixed by https://github.com/rstudio/helm/releases/tag/rstudio-connect-0.5.0-rc01

I can confirm this has indeed been fixed. Nice one @dbkegley & team 🎉


Get fixed spawn positions from config and assign them to players

In Player constructor added initial_position parameter to specify where to spawn.
Adds initial_positons to the GameConfig structs.

Had to make Position derive from Deserialize macro.

In the add_player method get the corresponding position from the configuration depending on the number of players already in the game, and then pass that position to the constructor of Player as a parameter.

To work thses changes are also needed.

To work thses changes are also needed.

I'm guessing these does't count anymore


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1492 ms

Resolved: Bofrost.de performance has improved in 3907b6d after 22 minutes.


Anaconda punishes people for using Python 3.5

Anaconda developers push people into using latest version of Anaconda by claiming that the
option of working with Python 3.5 is creating an environment. In truth an earlier Anaconada
version can be used, but the option is hidden on the website.
An additional problem here is that some packages
do not work with Python 3.6 yet, like nltk for Natural Language Processing,
and keras developers for Deep Learning only recently claimed that their
library now cooperates with Python 3.6, although in practice is it not a
smooth ride. I have students who are learning and the environments for them
is additional burden. I wish you would have a choice for people what
Anaconda version to load with what Python. By the way people with Windows
10 complain that 5th Anaconda crushes. I directed them to Anaconda site but do
not know if they made it or switched to earlier version.
Sorry for belated response. I was extremely busy.
Best,
Mya
Versions and main components

Spyder Version:
Python Version:
Qt Version:
PyQt Version:
Operating system:

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Sorry, we're not affiliated with Anaconda anymore. Please report this problem there:
https://github.com/ContinuumIO/anaconda-issues/issues/

But you still recommend Anaconda as a first way to install Spyder.

Yes, but all the problems you mentioned are related to Anaconda, not to Spyder, so there's nothing we can do about them here.

So if I what to know learn what are your current recommendations on Spyder installation I'm to open a new ticket, because on the following site not-Anaconda ways are called outdated?
https://pythonhosted.org/spyder/

We're going to still recommend to use Anaconda in the future because
it's the easiest way to install Spyder. But we're not responsible nor we
can fix all Anaconda's shortcomings (such as the ones you listed).
El 27/11/17 a las 14:56, Maiia Bakhova escribió:

So if I what to know learn what are your current recommendations on
Spyder installation I'm to open a new ticket, because on the following
site not-Anaconda ways are called outdated?
https://pythonhosted.org/spyder/
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/5837#issuecomment-347307424,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWS7b1QVQ7SeuDR4ODniIRL4B3ANt_4ks5s6xPxgaJpZM4Qq2L8.

Oh, I see. Thanks anyway.


Add Django 1.9 Compatibility

Some new changes in Django 1.9 prevent the plugin from working as is. Made the required changes and made sure current tests pass.
Had to bump up the CMS version to 3.2 since 3.1 doesn't support Django 1.9.

Specifically, these errors are fixed:
cmsplugin_filer_file.FilerFile.file: (models.E006) The field 'file' clashes with the field 'file' from model 'cms.cmsplugin'.
cmsplugin_filer_link.FilerLinkPlugin.file: (models.E006) The field 'file' clashes with the field 'file' from model 'cms.cmsplugin'.

Thanks @impala2 !
I like source much better than file, but we need to evaluate the implications for existing projects before changing this.

do users who are overriding the template need to change anything?
are there other cases where 3rd party software relies on file being there?

Might it be an option to set related_name ='+' instead of renaming the field?

@stefanfoulis I merely copied the fix over from here: https://github.com/divio/djangocms-file/pull/13
I tried the related_name fix and it doesn't seem to work. Anyone overriding the templates should have no issues as long as they're using the provided methods (get_file_name() and such). A possible solution is to have file and file_id properties that emit deprecation warnings when accessed - I just tested it and and all tests pass.

@impala2 I like the deprecation thing. Changing the field names is always painful: even if a clean API exists, I guess that most code still rely on the field name

I have to admin, I'm still not understand quite why this error occurs in the first place.
So our plugin has a model field called file. But what is the file attribute on cms.cmsplugin?
Is it maybe the automatically generated relation to the plugin model of djangocms_file.file (https://github.com/divio/djangocms-file/blob/master/djangocms_file/models.py#L22) due to multi-table inheritance?
If so, there is a deeper problem in how all the plugins work, which needs to be solved in django-cms itself. That would mean that if someone releases a new cms plugin called Source, things would break again. And we'll be in a constant dance to try to avoid any such clashes.
@yakky do you think we could fix that in django-cms itself somehow? E.g fix the relationship to have a prefix of the appname (cmsplgauto_djangocms_file_file instead of just file).

This looks more related to the ORM than to the CMS itself

@stefanfoulis this is the same issue outlined in http://stackoverflow.com/questions/20334029/django-app-cannot-assign-imagetext-link-must-be-a-link-instance/20346355#20346355 and discussed in https://github.com/divio/django-cms/issues/3204. In the end it's a matter of properly naming the models and the fields.
So far all plugins, I prefix the fields with the model name to avoid any kind of clash.

I've created an issue on django-cms for this: https://github.com/divio/django-cms/issues/5030

@impala2 now I realise, the related_name = '+' can't work, because the one causing the conflict is actually djangocms-file. So I created this pull-request: https://github.com/divio/djangocms-file/pull/17
I'm closing this pull request in favour of fixing the "bug" in djangocms-file, since I suspect many other plugins to have the same problem. After all it is quite likely someone would create a field named file.

An update: keep an eye on divio/django-cms#5032 . We plan to fix this issue in django-cms itself.


ISS fragment:

docked vehicles recycler adapter and individual elements added

Signed-off-by: James Gitonga<EMAIL_ADDRESS>

LGTM


Enable independent use of subspecs

Include .h and .m files in root class folder into Core subspec
Use preprocessor definitions to compile different components
independently
This enables use of standalone subspec, current implementation only
allows use of default 'All' subspec
Improve file patterns for subspecs with use of */ where appropriate
Add subspec for PDF417 components

That's a nice contribution. I will look into it and merge it after review. Thank you.


Update Projects

Automated changes by create-pull-request GitHub action

No idea. The logs don't mean much to me. Do you have permissions to re-trigger the deployment to see if it was just a one-off? I do not.

Do you have permissions to re-trigger the deployment to see if it was just a one-off? I do not.

Same here, I don't have permissions either.
Logs don't mean too much, other than that this can be a one-off like @cjyabraham said.
@geekygirldawn can you close and reopen the PR if you have the permission? That should trigger a rebuild I believe.

That should trigger a rebuild I believe.

No it didn't 🤦
Can you re-trigger the build on Netlify platform then @geekygirldawn ?

I think it re-ran the check (not 100% sure), but it failed again. Is there someone else who can help out with Netlify from the CNCF?

I just triggered a rebuild by editing the README and it looks like it worked this time.

Weird. When I look at the deploy logs here I only see one failure at 7:09AM.

I looked at the preview and everything looked ok, so I went ahead and merged it.


Integer overflow in PowerToys Run when converting hexadecimal to decimal

Microsoft PowerToys version
0.69.0
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

Open PowerToys run by pressing the key bind
Enter a hexadecimal that is greater than the signed 32-bit int upper bound, 0x7FFFFFFF. Such as 0x80000000 and 0xCAFEBABE

✔ Expected Behavior
Convert the numbers to decimal correctly, 0x80000000 →<PHONE_NUMBER>, 0xCAFEBABE →<PHONE_NUMBER>
❌ Actual Behavior
Gives numbers like 0x80000000 →<PHONE_NUMBER>1562100000, and 0xCAFEBABE →<PHONE_NUMBER>2820300000 instead of the correct numbers. Those numbers seems to be the approximation of the unsigned 64-bit int upper bound<PHONE_NUMBER>3709551615.

Other Software
No response

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


Add support for generating IAsyncDisposable implementations.

Issue Details
Currently the generator does not give the ability to generate IAsnycDisposable disposable implementations that work with the async dispose pattern. This issue tracks that so that it can eventually get added.

I think this should be critical to support.


jsr generates faulty import/export urls when using submodules

With deno.json:
{
   "imports": {
       "@hexagon/bundlee": "jsr<EMAIL_ADDRESS>   }
}

Permalink -> https://github.com/Hexagon/pup/blob/7d06bb367dcbc6f3321bbc93cef5bed4b95271db/deno.json#L43
pup/plugins/web-interface/deps.ts
export { Bundlee } from<EMAIL_ADDRESS>
Permalink -> https://github.com/Hexagon/pup/blob/7d06bb367dcbc6f3321bbc93cef5bed4b95271db/plugins/web-interface/deps.ts#L3
Is transpiled to the following on jsr.io (notice the extra slash before @):
export { Bundlee } from "jsr:/<EMAIL_ADDRESS>
Permalink ><EMAIL_ADDRESS>The same thing happened here<EMAIL_ADDRESS>which were later worked around by specifying the full path to base64url submodule in deno.json, like:
{
   "imports": {
       "@std/encoding/base64url": "jsr<EMAIL_ADDRESS>   }
}

That slash is actually fine - jsr: and jsr:/ are equivalent :)
The same also applies to npm: and npm:/.

Ehm, can't reproduce this now. I'll get back if I figure out what was going wrong :)


CloneTemporaryContext fails with code EGL_BAD_CONTEXT

Description
Remote capturing on a Mali G31 powered device (RK3326 SoC) fails to create temporary sandboxed contexts in EGLPlatform::CloneTemporaryContext due to a mismatch in the EGL_CONTEXT_CLIENT_VERSION attribute, causing failures such as garbage data in textures and other adverse effects, resulting in a partially corrupted capture.
For reference - the following workaround was attempted and provides successful captures in the few applications I've tested so far.
Steps to reproduce

Launch remote server
Launch application on said server
Attempt a capture
Sandboxed context creation fails apparently silently
Capture continues regardless
Capture fails to fetch textures and other adverse effects

Environment

RenderDoc version: git-c5480aa622a2
Operating System: Ubuntu 19.10 (aarch64)
Graphics API: OpenGL ES 2.x
Graphics Drivers: Mali-TDVX / v1.r6p0-01rel0.871ae322bd66147fe999503abdaa6f0d

RDOC 000725: [17:50:16]        gl_driver.cpp(1142) - Log     - Core context 011BFFB0 created with attribs, sharing with context 00000000
RDOC 000725: [17:50:16]     egl_platform.cpp(  91) - Warning - Creating cloned context failed. Trying again with queried old EGL client version: 2
RDOC 000725: [17:50:16]        gl_driver.cpp(1155) - Log     - Created new sharegroup 011B2858

Works great, thank you.

RDOC 000725: [17:50:16]        gl_driver.cpp(1142) - Log     - Core context 011BFFB0 created with attribs, sharing with context 00000000
RDOC 000725: [17:50:16]     egl_platform.cpp(  91) - Warning - Creating cloned context failed. Trying again with queried old EGL client version: 2
RDOC 000725: [17:50:16]        gl_driver.cpp(1155) - Log     - Created new sharegroup 011B2858

Works great, thank you.


Edit Datapoint doesn't work

Adding a Datapoint works and gets registered correctly.
When going back to edit such datapoint, it is correctly populated in the form. But when saving it, only part of the data is sent. Many things missing. And that is saved and then next round of "Edit Datapoint" it will be mostly empty form.
Edit Project doesn't have this problem.

I have now tried far too much and possibly breaking more than I am fixing. I think I need to ask @grzegorekb for help.

Please look into this. I have exhausted my understanding of what is happening (and don't have a local system set up) and console.log() or logInfo() don't work when running Grafana on prod (don't know if it works when running locally).
Feel free to charge/invoice me for work done.

Sorry for the timing. Things has changed since when I changed job. Will
take a look this week and let you know.
On Tue, Nov 2, 2021, 2:17 PM Niclas Hedhman @.***>
wrote:

Please look into this. I have exhausted my understanding of what is
happening (and don't have a local system set up) and console.log() or
logInfo() don't work when running Grafana on prod (don't know if it works
when running locally).
Feel free to charge/invoice me for work done.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Sensetif/sensetif-app-plugin/issues/25#issuecomment-957550876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB53AGRGFADWWA5VLFMTS33UJ7QGXANCNFSM5HGMDAHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

As I switched from linux to macos, I experience a similar issue with building backend plugin code:
https://github.com/confluentinc/confluent-kafka-go/issues/523
Need more time to set up dev environment

update: managed to build it in linux-based container; docker ftw

@niclash
I've made 2 commits;
the first one only upgrades grafana's libraries <- should also work without it, didn't check.;
second commit with the actual fix: https://github.com/Sensetif/sensetif-app-plugin/commit/066856102c5d9a8ed89892a43758eedb5a465131
Please verify on your environment if it works; if it does I will propagate the change to other affected places.

btw. it worked before; maybe it's some issue with the newest Grafana / or Form library it uses underneath. That's interesting. I will check on that when have some free time

I don't know when it stopped working, but you are right that it worked at some point in the past. I made the assumption that it happened when you refactored things into more modularity.
I will try to test it today.

Also, once we have some working version I would merge these branches (datasource & app plugins) to master; create some release in Github and delete these branches. I think it will be more convenient to work once a new issue comes in.
I would go with trunk-based / short-lived branches so we always have the latest working version on master.
Additionally, I would suggest writing unit-tests for backend (as I'm more familiar with golang, I could start work on this when having some time). The sooner the better I think because it might be that the code requires some/lot of refactoring to be properly unit-tested.
@niclash