Old backup images not being cleaned up

Im having a problem with oVirtBackup not cleaning up old backups as specified by the backup_keep_count option in the config file.  The old backups are not removed, nor do I see a "Backup deletion started for backup:" message in the log..
With commit 1e3b29e1a17b44d829692d6cbce3c0767ffcfa48 (March24) this functionality works as described - but with the latest release (a4a74d56a618d6c7a5b4f278397414f6eb4305d4) it does not.
Based on some manual testing with old commits, I believe this broken functionality was introduced with commit 6563e9236ed08bba1088f7c374cc02e29e30e3cf (March30).
I fixed this for me by changing lines 144,145 of vmtools.py (method delete_old_backups) to be like in this diff:
https://github.com/wefixit-AT/oVirtBackup/compare/master...MrFishFinger:master
I believe the problem stems from the fact that api.storagedomains.myStorageDomain.vms.list(query=vm_search_regexp) does not seem to work correctly and returns an empty list.
However api.vms.list(query=vm_search_regexp) does seem to work (as seen in the method delete_vm).
I am using an older version of oVirt (3.6) - so maybe that is causing the problem? I am also not an expert with Python by anymeans...
ovirt-engine-sdk-python.noarch - <IP_ADDRESS>-1.el7
ovirt-release36.noarch - 1:3.6.7-1
let me know if I should submit this patch as a PR or if this is just an issue localised to me :)

We are suffering from this problem as well under oVirt 4.0, so this does
not seem to be related to the oVirt version.
A patch would be very much appreciated.
On 11.04.2017 13:23, MrFishFinger wrote:

Im having a problem with oVirtBackup not cleaning up old backups as
specified by the |backup_keep_count| option in the config file. The old
backups are not removed, nor do I see a "Backup deletion started for
backup:" message in the log..
With commit 1e3b29e
https://github.com/wefixit-AT/oVirtBackup/commit/1e3b29e1a17b44d829692d6cbce3c0767ffcfa48
(March24) this functionality works as described - but with the latest
release (a4a74d5
https://github.com/wefixit-AT/oVirtBackup/commit/a4a74d56a618d6c7a5b4f278397414f6eb4305d4)
it does not.
Based on some manual testing with old commits, I believe this broken
functionality was introduced with commit 6563e92
https://github.com/wefixit-AT/oVirtBackup/commit/6563e9236ed08bba1088f7c374cc02e29e30e3cf
(March30).
I found that by changing lines 144,145 of vmtools.py (method
|delete_old_backups|) to be like in this diff:
master...MrFishFinger:master
https://github.com/wefixit-AT/oVirtBackup/compare/master...MrFishFinger:master
I believe the problem stems from the fact that
|api.storagedomains.myStorageDomain.vms.list(query=vm_search_regexp)|
does not seem to work correctly and returns an empty list.
However |api.vms.list(query=vm_search_regexp)| does seem to work (as
seen in the method |delete_vm|).
I am using an older version of oVirt (3.6) - so maybe that is causing
the problem? I am also not an expert with Python by anymeans...
|ovirt-engine-sdk-python.noarch - <IP_ADDRESS>-1.el7|
|ovirt-release36.noarch - 1:3.6.7-1|
let me know if I should submit this patch as a PR or if this is just an
issue localised to me :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wefixit-AT/oVirtBackup/issues/39, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAf6vKSbx3JLYr94-5em-KHzVW96QQACks5ru2KagaJpZM4M59hV.

--
Torsten Stolpmann
Geschäftsführender Gesellschafter
verit Informationssysteme GmbH
Europaallee 10
67657 Kaiserslautern
E-Mail<EMAIL_ADDRESS>Telefon: +49 631 520 840 00
Fax: +49 631 520 840 01
Web: http://www.verit.de/
Registergericht: Amtsgericht Kaiserslautern
Registernummer: HRB 3751
Geschäftsleitung: Claudia Könnecke, Torsten Stolpmann

ok a PR has been submitted for review. Keep in mind that I have not tested this very robustly - but at least it fixed my issues...
https://github.com/wefixit-AT/oVirtBackup/pull/40

We should consider to write some tests for all the new idea's. I will create a separate issue.
Can somebody test the merge and report if it works?

I tested the current version and it now works as expected. Thanks people!

Thank you all.
btw, It was a good idea to publish this as open source ;-)


Accessing Armada Pods externally

There are a few different methods for doing this:
Port Forwarding
This is the simplest method, forwarding the service port to a target port on <IP_ADDRESS>
kubectl port-forward service/lookout-sample --address <IP_ADDRESS> 8080:8080

NodePorts
This exposes the service automatically on a port. Nodeports need to be in the range 30000-32767
Setup
Any ports that you want to use need to be included in the kind config
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
nodes:
- role: control-plane
  kubeadmConfigPatches:
  - |
    kind: InitConfiguration
    nodeRegistration:
      kubeletExtraArgs:
        node-labels: "ingress-ready=true"
  extraPortMappings:
  - containerPort: 80
    hostPort: 80
    protocol: TCP
  - containerPort: 443
    hostPort: 443
    protocol: TCP

    # Any Other Ports
  - containerPort: 30051
    hostPort: 30051
    protocol: TCP

The service type then needs to be set to NodePort in https://github.com/armadaproject/armada-operator/blob/main/controllers/builders/service.go
	// Create the above but with type "NodePort"
	service = corev1.Service{
		ObjectMeta: metav1.ObjectMeta{Name: name, Namespace: namespace, Labels: labels},
		Spec: corev1.ServiceSpec{
			Selector: identityLabel,
			Ports:    ports,
			Type:     corev1.ServiceTypeNodePort,
		},
	}

Then, you can optionally specify the nodeports in https://github.com/armadaproject/armada-operator/blob/main/controllers/install/
	service := builders.Service(as.Name, as.Namespace, AllLabels(as.Name, as.Labels), IdentityLabel(as.Name), []corev1.ServicePort{
		{
			Name:     "grpc",
			Port:     50051,
			NodePort: 30051,
		},
		{
			Name:     "rest",
			Port:     8080,
			NodePort: 30080,
		},
		{
			Name:     "metrics",
			Port:     9000,
			NodePort: 30900,
		},
	})

NOTE: If they are not directly assigned they will be randomly picked from the valid range
Typing in kubectl get services -n armada should then show the services as nodeports with their ports
Ingress
This is the solution for production. If you want to test locally, you will need to add hostnames to /etc/hosts. See https://techhub.hpe.com/eginfolib/networking/docs/IMC/v7_3/5200-2694/content/466972523.htm
Otherwise, you can points your domains towards the ingress-cluster and it should behave as expected.

great writeup -- should this just be converted to a developer doc?

note that Service and Container ports are coming from applicationConfig!


fix(API): Frame ID validation to support 11 bit (not 10)

[x] This contribution adheres to CONTRIBUTING.md.
[x] New tests have been created for any new features or regression tests for bugfixes.
[x] tox successfully runs, including unit tests and style checks (see CONTRIBUTING.md).

Coverage remained the same at 64.23% when pulling 9b187a0a9bc09283f9c7146ff96b56aa34a19706 on epage:bug into b8e8f09ebde81698e8db7d1e701375e9e3b4c25e on ni:master.

Travis had a spurious pypy failure, merging anyways.


Fix aws-sdk-cpp and OpenSSL cross-platform builds

This PR borrows from https://github.com/microsoft/vcpkg/pull/20277
1- Do not build tests for any of the SDKs' dependencies. This makes the build faster and avoids some errors related to binaries that do not exist when cross-compiling.
2- Remove unused flags
Tested against Windows, Android, MacOS, and iOS

Which triplets are supported/not supported?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@NancyLi1013 I had to revert the quotation-mark changes as it broke the build on OSX

@NancyLi1013 I had to revert the quotation-mark changes as it broke the build on OSX

Yes, now it is right: In CMake language, quote whole string or list item. It is different from sh or bash.

@NancyLi1013我不得不恢复引号更改，因为它破坏了 OSX 上的构建

Yean, you can address the review suggestions later.

aws-sdk-cpp can currently be built on all platforms BUT it's not usable do to some hardcoded paths here https://github.com/aws/aws-sdk-cpp/blob/main/cmake/AWSSDKConfig.cmake#L67

aws-crt-cpp failed on x64-windows-static:
Invalid crt linkage. Expected Debug,Static, but the following libs had:

    D:\packages\aws-crt-cpp_x64-windows-static\debug/lib\aws-crt-cpp.lib: Debug,Dynamic

To inspect the lib files, use:
    dumpbin.exe /directives mylibfile.lib
Invalid crt linkage. Expected Release,Static, but the following libs had:

    D:\packages\aws-crt-cpp_x64-windows-static\lib\aws-crt-cpp.lib: Release,Dynamic

python3 failed on x64-osx:
install: mkdir /Users/vagrant/Data/packages/python3_x64-osx/Users/vagrant/Data/installed/x64-osx/debug/lib: File exists
make: *** [altbininstall] Error 71
make: *** Waiting for unfinished jobs....
renaming build/scripts-3.10/pydoc3 to build/scripts-3.10/pydoc3.10
renaming build/scripts-3.10/idle3 to build/scripts-3.10/idle3.10
renaming build/scripts-3.10/2to3 to build/scripts-3.10/2to3-3.10

aws-crt-cpp failed on x64-windows-static:
Invalid crt linkage. Expected Debug,Static, but the following libs had:

    D:\packages\aws-crt-cpp_x64-windows-static\debug/lib\aws-crt-cpp.lib: Debug,Dynamic

To inspect the lib files, use:
    dumpbin.exe /directives mylibfile.lib
Invalid crt linkage. Expected Release,Static, but the following libs had:

    D:\packages\aws-crt-cpp_x64-windows-static\lib\aws-crt-cpp.lib: Release,Dynamic

python3 failed on x64-osx:
install: mkdir /Users/vagrant/Data/packages/python3_x64-osx/Users/vagrant/Data/installed/x64-osx/debug/lib: File exists
make: *** [altbininstall] Error 71
make: *** Waiting for unfinished jobs....
renaming build/scripts-3.10/pydoc3 to build/scripts-3.10/pydoc3.10
renaming build/scripts-3.10/idle3 to build/scripts-3.10/idle3.10
renaming build/scripts-3.10/2to3 to build/scripts-3.10/2to3-3.10

That's not the only problem. On my Windows machine I can build the SDK but its configuration file is not usable. I'll fix that next week once I have enough time.

Fixed and ready for review.

On aws-sdk-cpp: I run into debug+release issues while make use of this port in netcdf-c (#21239), and I developed another build check to help with the problem (#21415). This PR solves the problem insufficiently, according to #21415. I would add a different PR, or perhaps make the changes part of #21239, so that CI doesn't only test build but also actual usage.
On openssl: This is entirely unrelated to the aws sdk and should be put in a different PR for separate review. It just increases the CI build time for this PR, due to its broad usage.

Thank you for your review. I did make all the requested fixes so far. I understand your concern with OpenSSL but the change has already been done and tested so I just changed the title of the PR to include OpenSSL.
As for the guards, I didn't really face any problem when building or consuming the library. I do understand the problem but I'm not sure how it can affect linking the library to apps. I'll keep a close eye on your PRs.

/azp run

For some reason the Commit Suggestions button is not appearing!!

/azp run

LGTM, thanks for your PR @ahmedyarub

Need to update version files, otherwise LGTM.

Thanks for the fixes!


Add shadow dex-ee mj description.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 97.524%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
906

Relevant Lines:
929

💛  - Coveralls

LGTM, thank you for this contribution :+1:


Missing 4-star character in weapon event wish

In the wish weapon event chart, the 4-star character is currently missing. Can you re-add it? Thanks!

Is it possible that you don’t have a 4-star character in your record?

ah, I got it :))


Respect new line breaks in cells

The following code:
    final V2_AsciiTable at = new V2_AsciiTable();
    at.addRule();
    at.addRow("request", "status", "causes");
    at.addRule();
    at.addRow("1", "invalid", "* size limit exceeded\n* invalid characters found");
    at.addRule();
    at.addRow("2", "valid", "* all requirements satisfied");
    at.addRule();
    final V2_AsciiTableRenderer rend = new V2_AsciiTableRenderer();
    rend.setTheme(V2_E_TableThemes.PLAIN_7BIT.get());
    rend.setWidth(new WidthFixedColumns().add(10).add(10).add(40));
    final RenderedTable rt = rend.render(at);
    System.out.println(rt);

results in:
+----------+----------+----------------------------------------+
| request  | status   | causes                                 |
+----------+----------+----------------------------------------+
| 1        | invalid  | * size limit exceeded
* invalid        |
|          |          | characters found                       |
+----------+----------+----------------------------------------+
| 2        | valid    | * all requirements satisfied           |
+----------+----------+----------------------------------------+

but should result in:
+----------+----------+----------------------------------------+
| request  | status   | causes                                 |
+----------+----------+----------------------------------------+
| 1        | invalid  | * size limit exceeded                  |
|          |          | * invalid characters found             |
+----------+----------+----------------------------------------+
| 2        | valid    | * all requirements satisfied           |
+----------+----------+----------------------------------------+

Yes, lists and paragraphs are not handled (yet) by the renderer. I have an idea how to do that, but didn't get time to try it out.
For paragraphs it should be easy, for lists it might be tricky if the line needs to be wrapped later, since the list introduces indentation.
I'll have that on the todo list now.
thanks
/sven

Fix for line break added, now in the dev branch. Your WidthLongestLine works nicely now. Also added the test of this issue to the examples in test test code, will add it to the main documentation later (for next release).
Thanks!
/sven

Nachtrag / After math
I have created a new repo for AsciiList with all sorts of lists initially supported. Those lists can wrap themselves against a given width (and have all sorts of other features, like nested lists and styles). I will add those lists to the AsciiTable in the next table's release, then once can use either manual lists (as in your example, which now works) or the new lists.
Thanks very much for your contribution!
/sven


Why are the file signatures in the project folder, and bin folder different?

md5sum
Project folder:   f8243a3f2473bfb1b13a3db9ea2dca66 *hashtwm.exe
bin subfoloder: 0ebf9b43e4a3685a1a391daded34c348 *hashtwm.exe
They should be the same, right?

Yup, I'd missed that sorry, it looks like the one from the bin subfolder is a few days older, so the project folder one is the latest.

Thanks! I am closing the issue now.


Question : About Security in helm3

I am using GKE , and i am using jenkins as cicd tools
I have a question about helm3 security, while as i know the one key point migrate from v2 to v3 is about security improvement
while helm 2 ---->tiller --->service account----> target actions
while helm 3 --- > your account /service account ---> target actions
if it is local user deployment via local machine trigger helm3 cmd
then it should record the user information action from GCP/GKE audit level
but when we are using jenkins as our cicd tools
somehow i think it still need a service account to access object inside the cluster
then will still go back to a situation
from GCP side will it still cannot record whom via the jenkins account did what in my cluster

In Helm 3, security is on a per user basis. It is delegates to K8s user cluster security. It will work similar to using kubectl authentication and authorization..
A good reference is a talk at Helm summit by @bacongobbler on securing Helm:  https://sched.co/S8tr

We use Jenkins as well. You are right that Jenkins needs some account to connect to your GKE cluster. If you use a service account for Jenkins then as you say the audit will only know about the Jenkins account, not the user who triggered Jenkins.
You could define a way to use a specific user kubeconfig file for their Jenkins build.
Or you could add annotations to your deployments indicating the user that triggered the build. I'm sure there are other tricks to achieve this too.

Closing as answered


How to read secret env vars from bash?

Both $FOO and $(FOO) syntax resolve as an empty string in bash

The only relevant section seems to be this one.
From this I understand that the only way to make this work is to explicitly make TFS resolve the variables by using bash my_script.sh $(SECRET1) $(SECRET2) ... $(SECRETN) otherwise they will never ever be available.

Np. I can wait for now. Worst-case scenario is passing the 15 secret env vars as an argument to the bash script I run 😕

Yes, https://docs.microsoft.com/en-us/vsts/build-release/concepts/definitions/build/variables?view=vsts&tabs=shell#set-in-script is how you do it.

That link is the one I posted 16 days ago. So, again, not helpful.
It'd be clearer to state somewhere in that same page that this approach is the only way to go instead of a code snippet that doesn't even say where's its supposed to go.
Example: In order to use secret env bars in a bash task you actually do need to explicitly send them as arguments otherwise they'll resolve to empty strings.

Secret variables are documented for both YAML and Designer now.


refgenie build does not include subassets in seek_keys

refgnie build does not include the subassets under the seek_keys key. It still treats them as separate assets:
refgenie build -c genomes.yaml rCRSd/fasta --fasta rCRSd/rCRSd.fa.gz
cat genomes.yaml

config_version: 0.3
genome_folder: /Users/mstolarczyk/Desktop/testing/test_genomes
genome_server: http://refgenomes.databio.org
genomes:
  rCRSd:
    asset_packages:
      fasta:
        default:
          asset_path: fasta/rCRSd.fa
          asset_description: Sequences in the FASTA format
      fai:
        default:
          asset_path: fasta/rCRSd.fa.fai
          asset_description: Indexed fasta file, produced with samtools faidx
      chrom_sizes:
        default:
          asset_path: fasta/rCRSd.chrom.sizes
          asset_description: Chromosome sizes file
    genome_checksum: 32a37a52a377d95bfd4b3d66763e1396a4480f34ab5c318a

right... build needs to be tweaked to use the new conf format...


cpg.local.dump outputs empty strings

Currently, cpg.local.dump returns empty strings, at least for the C frontends, while cpg.local.method.dump works fine. My guess is that locations for locals are not returned correctly or that line number information is missing.

Looks like line numbers are missing for locals: cpg.local.lineNumber.l returns an empty list.

Added the line number, but that alone seems to not yet fix it. Investigating further...


⚠ Processors has degraded performance

In e8694df, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: Processors performance has improved in fd76bef.


eth p2p types abstraction

once we support custom p2p types, we also need to make the eth-wire-types generic over txs/blocks.
ideally this solution also makes it possible to incorporate nuances e.g bnb
https://github.com/bnb-chain/reth/tree/develop/crates/net/eth-wire-types/src
we should start this with the BlockBodies type

@yutianwu could you perhaps point me to the bsc differences on the eth rlpx protocol level
iirc blocks are different and some message ids?

There's an extra filed pub sidecars: Option<BlobSidecars> after withdrawals and no requests on bsc. The BlobSidecars is like:
pub struct BlobSidecars(Vec<BlobSidecar>);

pub struct BlobSidecar {
    pub blob_transaction_sidecar: alloy_eips::eip4844::BlobTransactionSidecar,
    pub block_number: U256,
    pub block_hash: B256,
    pub tx_index: u64,
    pub tx_hash: B256,
}

@mattsse

There's an extra filed pub sidecars: Option<BlobSidecars> after withdrawals and no requests on bsc. The BlobSidecars is like:
pub struct BlobSidecars(Vec<BlobSidecar>);

pub struct BlobSidecar {
    pub blob_transaction_sidecar: alloy_eips::eip4844::BlobTransactionSidecar,
    pub block_number: U256,
    pub block_hash: B256,
    pub tx_index: u64,
    pub tx_hash: B256,
}

@mattsse

This impact the NewBlockPacket, BlockBody packet regarding p2p.

I guess the typed_transaction is an good abstract example? Ethereum can extend new transaction type without breaking anything.  Maybe introduce typed block body, typed block so anyone depends on Reth can custom it.

marking this as completed, any further improvements will be tracked separately


use go 1.16

all our build and tests should run using go 1.16
Few things that needs to be updated

[ ] prow jobs
[ ] psi jobs
[ ] release build scripts

/priority high

all our build and tests should run using go 1.16

Instead of calling it with versions, how about when we just say, run test on latest go?

Instead of calling it with versions, how about when we just say, run test on latest go?

sure, we can do that. But latest go needs to mean latest stable version as it is on https://golang.org/dl/ not latest version that is awailable for a given system.
Test run on different systems, Linux (fedora, centos ...), win, mac ... We need to make sure that at least we use the same minor (1.16.x) version everywhere.  We should not have one system running 1.14.x and the other 1.16.x

sure, we can do that. But latest go needs to mean latest stable version as it is on https://golang.org/dl/ not latest version that is awailable for a given system.

+1

The release builds currently has a go 1.15 for latest build target, I think we should do that instead going for 1.16 for now.
We can create a separate ticket to add a newer version of golang(1.16).

OCP uses go 1.16 everywhere. They have their own build targets; ours are decided by a different team.

We could open a request with the build team to use 1.16 for odo's build targets if we want to move to 1.16 in odo repo.
Or, figure out how OCP always uses latest go version in their build targets and do that for odo as well. (preferred approach)

Else, we bump to 1.15 (current latest in build systems) for now.

As 1.16 is not available should we create a ticket and spend time on figuring out if we can also have a custom build target that is latest?
or move forward with 1.15 for now and figure out how can we use latest on build target?
what do you think?
cc_ @dharmit @kadel

As 1.16 is not available should we create a ticket and spend time on figuring out if we can also have a custom build target that is latest?

Where is it not available? On the infra used to build odo artefacts?

or move forward with 1.15 for now and figure out how can we use latest on build target?

If answer to my question about artefacts is yes, then I think this is our only option, no?

Where is it not available? On the infra used to build odo artefacts?

yes, In our brew build target.

If answer to my question about artefacts is yes, then I think this is our only option, no?

I was a little confused about what should we do now, do we want to use 1.16 directly or invest time to figure out the way to use the latest.
And also I was thinking about all the possibility that we can go through, we can also create a ticket for 1.16 and use 1.15 for now, and  figure out how can we use latest on build target?
For my perspective I think we should " create a ticket for 1.16 and use 1.15 for now, and  figure out how can we use latest on build target "

If answer to my question about artefacts is yes, then I think this is our only option, no?

I was a little confused about what should we do now, do we want to use 1.16 directly or invest time to figure out the way to use the latest.
And also I was thinking about all the possibility that we can go through, we can also create a ticket for 1.16 and use 1.15 for now, and figure out how can we use latest on build target?
For my perspective I think we should " create a ticket for 1.16 and use 1.15 for now, and figure out how can we use latest on build target "

My understanding of the build infra we use is very limited. But using the currently available latest (1.15 at the time of writing this), and figuring out how we can decide/control the version we want to use (1.16 at the time of writing this) sounds like a good idea.
@kadel @mohammedzee1000 wdyt?

Scope of the task

[x] onboard to go 1.15
[x] create issue getting go 1.16 into cloud build
[ ] automate cloud build process to use go latest
[ ] Update built-in dockerfile in odo repo

Should the odo-init image also be updated to use go 1.15?
https://github.com/openshift/odo-init-image/blob/27dc3d4193ca8f6d34e781907ceb584a2d8f5ae4/Dockerfile#L7-L9
Should it be a part of this issue or should we have a separate issue for it?

Should the odo-init image also be updated to use go 1.15?

yeah, I completely missed to change in odo-init-image repo, as it was not mentioned on the doc i was following. I will update both the doc and image tag in odo-init-image repo.

Should it be a part of this issue or should we have a separate issue for it?

Both works fine for me. I can add this to scope of the issue which is mentioned above https://github.com/openshift/odo/issues/4778#issuecomment-872935635

@anandrkskd there's already a PR for it - https://github.com/openshift/odo-init-image/pull/88/

Understood! I guess we don't need to create a new issue then as a PR is already created.

reducing the priority to medium.
/priority medium


Consider providing ability to override volume mount default behavior

For larger projects or mono-repos containing many directories that have self-contained functionality, it would be nice to have a way to specify which director(y|ies) to load. This would potentially decrease the amount of files that must be mounted into the container, required or imported by interpreted languages such as ruby or python, and therefore decrease the startup-time of the container. Perhaps something like nib run-dir <list/of, paths/to/load> <service-name>

I like the idea of improving developer workflow (e.g. performance) in projects like you describe. Before we try to implement a feature here though, I wonder if it's worth considering an approach like what's described in this post.
https://iwpnd.pw/articles/2021-05/docker-compose-in-monorepos
tl;dr — have a top-level docker-compose.yml that defines the shared services (e.g. databases) and a docker-compose.yml in each component directory that extends the parent to pull in the db's.


index out of range [0]

E:\rplacebot\sexrplace727> ./redditplacebot -minX=851 -minY=-192
Waiting for login to finish...
2023-07-24T00:41:03.863+0700    INFO    reddit-placebot-2023-master/client.go:81        Login successful        {"username": "Direct_Log_9991"}
2023-07-24T00:41:04.333+0700    INFO    reddit-placebot-2023-master/board.go:50 Controller changed      {"username": "Direct_Log_9991"}
Login finished!
Waiting for board data
Board data received!
2023-07-24T00:41:06.129+0700    INFO    reddit-placebot-2023-master/worker.go:49        Placing client  {"username": "Direct_Log_9991"}
Placing pixel {866 -174} {0 163 104} 2 {2366 -174}
panic: runtime error: index out of range [0] with length 0
any help ?

Outdated error, please use the latest commit

Outdated error, please use the latest commit

PS E:\rplacebot\sexrplace727> ./redditplacebot -minX=840 -minY=-192
Waiting for login to finish...
2023-07-24T13:29:42.393+0700    INFO    reddit-placebot-2023/client.go:84       Login successful        {"username": "Popular-Homework-259 "}
Login finished!
Waiting for board data
2023-07-24T13:29:43.925+0700    INFO    reddit-placebot-2023/board.go:50        Controller changed      {"username": "Popular-Homework-259 "}
Board data received!
2023-07-24T13:29:45.809+0700    INFO    reddit-placebot-2023/worker.go:49       Placing client  {"username": "Popular-Homework-259 "}
Placing pixel {868 -172} {255 153 170} 2 {2368 -172}
panic: runtime error: index out of range [0] with length 0
goroutine 1 [running]:
main.(*Client).Place(0xc000222510, 0x103e1ca?)
D:/a/reddit-placebot-2023/reddit-placebot-2023/client.go:307 +0x142f
main.(*Worker).Run(0xc00022c180)
D:/a/reddit-placebot-2023/reddit-placebot-2023/worker.go:50 +0x58f
main.main()
D:/a/reddit-placebot-2023/reddit-placebot-2023/main.go:51 +0x67a
same error, even with latest commit


Vivaldi vs Chrome | unvisible root

Don't know why, but extensions do not show root files in Vivaldi(chromium) browser.
I will post bug in Vivaldi forum too.

I have never used Vivaldi before, I suppose there may be some differences concerning the extensions API.

I'm having the same error in Vivaldi and Chrome, both on their latest version. After a few seconds, the Root appears.
Cheers.

I'm in the same boat - I can never get anything to display in the Vivalid devtools window. The extension recognises Vue on the page (icon lights up and says 'Vue.js is detected on this page.'), but the devtools panel is just empty. But for me on Chrome (55.0.2883.87) it works fine. It would be really great if this could be supported on Vivaldi. @adridev did you post this to the Vivaldi forum?
I also looked at the background page for Vue-Devtools, and nothing is shown in the console. I would expect something if there were errors - is there any debugging info produced by the extension?

@dsl101 Yes I did post this on Vivaldi formu, but not even after 2min Root is not loaded.

@AlennGK Sorry - I got the comments mixed up :) For me, on Chrome everything works as it should. On Vivaldi, I never see anything in the Vue panel. I think that's the same for you?
@adridev perhaps you have a different issue. Are you saying that Vue-devtools does work in Vivaldi, but only after a few seconds? My Vivaldi version is "1.8.770.44 (Official Build) (64-bit)", and I've never been able to get anything to show

Hi,
No, in Vivaldi sometimes it appears after a few time and sometimes it doesn't. In Chrome It has a little delay sometimes but less frequently.
I'm running Manajaro Linux with the latest LTS kernel, Chrome  57.0.2987.110 (64-bit) and Vivaldi 1.8.770.50 (Stable channel) (64-bit).
Chrome

Vivaldi

Both screenshots are from the same App and URL.

@dsl101 I fact, after today's update of Vivaldi It doesn't work at all.

Seems to be a general issue. Doesn't work under Windows 10 and Vivaldi 1.8.770.56 (Stable channel) (32-Bit) either.

Funny thing. The newest vivaldi snapshot that allows docked devtools, the vue devtools work as in chrome. Using vue 2.3.0.

@hultberg Lets wait until that version of Vivaldi is officially released. BTW, just a side question, does the latest snapshot have mobile emuiation too? I haven't seen it.

@raniesantos Yes, there is a button for it. However, it only works when the devtools is docked.

From what i can see, any of extensions to devtool from chrome is not working under this browser, eg. react/angular 🗡

Based on what @hultberg said, it seems like this issue is related to the fact that the devtools can't be docked.

Confirmed working in Vivaldi 1.10 (docked devtools)

Just FYI, Vue Devtools has been working for me for the past few weeks non-docled. I vaguely remember that I had to dock the devtools window at first to get it to work, but since then, it's been fine. I'm using Vivaldi 1.10.862.6 (Official Build) (64-bit) on Windows 10 Pro

Just tried this and it seems to work great docked or undocked on:
Vivaldi: 1.10.867.38
vue-devtools: 3.1.4
Vue: 2.3.4
OS: Fedora Linux 26 x86_64


[cnats] Repair installation path

Fixes https://github.com/microsoft/vcpkg/issues/35421
Repair the installation path of the dynamic library from lib to the bin directory.

[X] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Usage test pass with following triplets:
x64-windows
x64-windows-static

Thanks :)

Still see this issue with x64-windows and x64-windows-release.
Only x64-windows-static with cnats::nats_static used in target_link_library works for me


Load events with react query

Hi,
I'm using react query and I want to pass the events through the state by the events prop, after fetching and setting the new state.
It seems that the calendar keeps the initial empty array state instead of updating with the new events from the state.
Appreciate any help.

yes this is intentional.
To update the events you can:

use getRemoteEvents to get remote events.
use useScheduler hook to progmatically update state, there is setEvents function.

Let me know if this solves your issue.
Also you could look into: https://github.com/aldabil21/react-scheduler/issues/39

I solved the issue like that:
const queryClient = useQueryClient();

const fetchRemote = async (query) => {
    const data = await queryClient.fetchQuery("events", getEvents)
    return data;
  };

  // ...

  return (
    <div>
      <Scheduler
         // ...
        remoteEvents={fetchRemote}
      />
    </div>
  );

Before I tried to use useQuery hook, which probably requires me to use setEvents function in useScheduler hook.
I prefer the first choice.

Cool. All roads lead to rome


Add data loader to database

Here, the data loading to the database which reads the parquet files is added.

I have corrected the issues mentioned. can you recheck this.

@pasan04 Thanks so much for making the latest commits about the documentation. I went through things a bit myself and committed some changes.
Before going into those changes, a couple of things:

No matter what, I think that we can keep the changes I made in the commit whose message is "Clean up README files".
However, please check the changes I've made (summarized below) in the commit whose message is "Update script documentation, formatting (black), etc.". If for some reason I've messed things up so bad that you want to just throw away the changes I've made, you can pull down the branch that I've made and reset --hard the log so that only the readme changes have been made. To do this, you would run:

git reset --hard 32aa63ac24a87343d495711989a3584583672e14

32aa63ac24a87343d495711989a3584583672e14 can be found by running git log and looking at the commit code above the "Clean up README files" message.

After that, you just would push the changes to the branch.

Apologies for the length. If anything isn't clear, please let me know if you'd like to go through it together.
My changes

I updated the README.md files so that they were a bit more uniform in their structure across the different directories. I believe now they all have a Contents section and then basically list the files and what they do. Other necessary information is included where necessary. You'll also see that I added a couple of new README files that were missing.
I cleaned up the docstrings for the scripts and functions where necessary. Previously you had the inputs for functions listed in the inputs section for the script, and did not have docstrings for most of the functions. I have added them with the standard triple quotes format ("""docstring example""") under where the function is defined. Aside from the clarity that they add to the code, other systems are built expecting docstrings to be there and they know how to work with that format. For example, in VS code, if I am in script example.py and I hover my mouse over a function in that script which is saved in another file called functions.py, it will show me tooltip of that functions docstring — so you have less flipping back and forth between scripts to understand what everything does. Also, if start a Python session in your terminal you can run help(some_function) and it will print the docstring for some_function, again, saving you the hassle of having to look through code.

Please make sure to go through these docstrings and make sure that I did not mess anything up

Generally, all files that I edited were updated to use the black formatting style because my VS code automatically formats python scripts this way. Most of these things update very long lines to use multiple lines instead.
I made some changes to code that I felt made things a bit cleaner:

server.py: I removed the two functions you had load_current_month_data() and load_past_months_data(). The code only ran the load_past_months_data() function which was actually set up to iterate through LOAD_PAST_MONTH_DATA = ['2022_01'] which is confusing because you can pass anything to that list (not just the past month). Now there is only one function called update_database() (I think this name is more specific) which iterates through a list called MONTHS. You'll see that by default it runs the current month but you can comment in other options for specific months or all months.

Instead of using the existing get_logger function in top_fibers_pkg.utils, you had copied the function over into a new utils.py script. There is no need for this so I loaded the function from top_fibers_pkg.utils. However, I just realized that I did not delete your duplicate utils.py script. Can you please delete it?

fib_indices.py: there is a function in this script (get_all_posts()) that looks like it should be deleted. Can you confirm and then please delete it? Given the other database_functions code, it also seems like perhaps you would like to add a function that retrieves all of the data from the FIB indices table? If not, please ignore.
controller.py: I cleaned up the comments and some of the logging messages in this one. For example, the first line in the read_files() function (now called add_data() as this seems more specific) logged a message that said "Found the data directory on that day!" before that had actually happened in the code. Another piece of code that I think can be improved has to do with the reports.check_report_already_added() function (however, there is a chance I am misunderstanding it, which is why I did not change it directly). I am wondering if you can flip the boolean value that is returned. Right now, it seems like it returns False if a report exists and True if it does not. So when you actually use the function like reports.check_report_already_added(file_date, selected_month, platform) it seems like it is asking the question "has a report been added for selected_month?" and returning False when it exists is sort of backward. If my assumption about the output is correct — i.e., it outputs False when a report exists — I would suggest that you change the function name to report_already_added() and have it return True if the report exists and False if not. Then the if block for the controller.py script will be changed to...if not reports.report_already_added(file_date, selected_month, platform):
    # add the new data

... which then reads like "if this report does not exist, we then add the data" which is much more natural.

There were some packages there were imported here and there which were not used, these imports were removed.


README contains paths to incorrect custom scripts folder

Summary
README contains incorrect path for custom scripts: see pre-backup and post-backup sections.
Steps to reproduce
/assets/custom/pre and /assets/custom resp.
What is the expected correct behavior?
/assets/custom-scripts/pre and /assets/custom-scripts resp.
Possible fixes
#155

Ahh thanks for this! I gapped out early in the morning putting this together.

No problem 😉


Kernel running well on Python 3.9 for Spyder 4.2 but the title of app still showing 3.7?

Problem Description
unsure if there will be any issues as the app doesn't look "pretty" with the wording on the top left as shown with 3.7 instead of the default.

What steps reproduce the problem?

installed kernels using pip install on my current path C:\Users\<myusername>\AppData\Local\Programs\Python\Python39\python.exe

Previously, GitBash, navigate with cd into the intended path directory as above, and typed pip install spyder-kernels==1.10.00

after installation have managed to see that the installer worked well with my Spyder IDE running on the interpreter meant for Python 3.9 but....

What is the expected output? What do you see instead?
Can I ask with my interpreter set to the path as mentioned for the python version 3.9 is this the right way of doing so? please take note that I am not using Anaconda as I have moved away from it after several strange encounters with package managements.
 Quick question for you: did you use our new Windows installer to install Spyder?

Quick question for you: did you use our new Windows installer to install Spyder?

Based on the dependency versions all matching precisely, that seems almost certain.
In any case, the title bar in any Spyder install method has always shown the version of Spyder's own Python runtime, so everything is behaving as expected here. Given the user can have multiple consoles open in different environments with different Python versions and freely switch among them, and they are fully independent of Spyder itself, I'm not sure what if anything else it would make sense to show in the title bar than Spyder's own runtime version.
However, this is really a relic of the days before we recommended and fully supported users separating their Spyder and working envs (and before the status bar widget), especially given that with our new installers, users may not even know that Spyder itself is written in Python and be confused by the Python version shown in the title bar, I would suggest just removing it. For the vast majority of users Spyder's runtime version shouldn't make a difference and its still displayed in the console and the status bar if they are actually using it to run their code, so the only purpose it serves there is to cause confusion (as it did here). If users need that information, they can easily get it from the About dialog.

Therefore, I would suggest just removing it

I agree, but only for the Windows and Mac apps. It's still useful if you install Spyder in multiple envs.
@dalthviz, @mrclary, what do you think?

Therefore, I would suggest just removing it

I agree, but only for the Windows and Mac apps. It's still useful if you install Spyder in multiple envs.
@dalthviz, @mrclary, what do you think?

I agree with @ccordoba12, sort-of. If one launches Spyder from different environments, it is useful to know from which environment it was launched, but maybe the Python version is not needed and can be known from the About dialog.
Would it be best to display the title bar as "Spyder ()", where  is the environment type and name, e.g. "conda: my-env"? And, as @ccordoba12 suggests, for Windows and Mac apps the title bar would read "Spyder".

Would it be best to display the title bar as "Spyder ()", where is the environment type and name, e.g. "conda: my-env"? And, as @ccordoba12 suggests, for Windows and Mac apps the title bar would read "Spyder".

I do recall this being proposed, but we evidently never got around to implementing it. As of now, it only shows the Python runtime version, so unless users have a different version of Python in every env they have Spyder in, and remember which is which, it is not helpful in this regard. If we did implement it, that would be much more helpful than the Python version, at least for the case where it matters.
For developers like us who use multiple Spyders at once it makes sense, but I worry that for the average user, if they are following our recommended approach of keeping one copy of Spyder (either via the installers, Anaconda or one of our many other install methods) and using separate working environments, the information isn't needed and I worry they might be confused why the Python version or environment shown doesn't match that in their IPython console. And if they are not and instead using one Spyder for each working env, their working env name, version and type is now right in the statusbar as well as the IPython console, so not sure how much they need it in the window title unless they've disabled single instance and have several Spyders open at once...not sure how common that is; I've never seen it myself.

dear @ccordoba12 @CAM-Gerlach @mrclary , yes I used the standalone installer and had always wanted to run on Python 3.9 and no ulterior motive but its also because I'd wanted to move away from the Anaconda environment and do the pip installs whenever required. So this works but is reassuring to note that its mainly a "display" - well yes having the display set as showing the "root" dir or the env as an additional indicator would be useful. Then again, I hope that setting the path is the right things as Am still new to the venv practice myself.

@vernhow Hey, not sure how much you understood from our discussion since it was pretty technical, but to avoid any confusion, let me explain what you're seeing here (which I should have before, sorry):
Spyder itself is written in Python, and runs on its own Python interpreter, which is the the version shown in the title bar. This is currently Python 3.7 as bundled with the standalone installers, or is whatever version of Python you have in the environment you install Spyder in, if you use it through Anaconda, a regular pip install, or one of our other distribution method. By default, if you haven't added the path to your own environment under the Python interpreters preference pane, new consoles will open in this environment.
If you do configure paths to your own Python interpreters/environments (venv/virtualenv, conda env, pyvenv, system Python, etc), then new IPython consoles (that run your code and you can interact with) will open in the environment(s) of your choosing, but Spyder itself still runs in its own environment, so you only have to keep one copy installed and changes you make in your environments won't break Spyder. Since the version in the title bar shows the environment that the Spyder application is running in, it won't change if you launch a Console in a different one (and it doesn't really make sense to, since you can have multiple consoles open in many different environments and switch between them at will).
What we're proposing here is, at least for the standalone installer versions of Spyder, to no longer show this Python version in the window title bar, since the version of Spyder is using doesn't make any difference for the user running their own code and just causes confusion, as it did here, and the environment in which your code runs is shown in the IPython console and in the status bar. For more advanced users who install Spyder in one (or more) of their existing Python environments, we're proposing changing the title bar to show the environment name instead of/in addition to the interpreter version to make it easier for users to keep track where the current copy of Spyder their using is installed (especially true for advanced users and developers).
Does that make sense now?

Dear @CAM-Gerlach , it does make sense and I believe the scenario as you described is the one where the path I have defined is a system Python which is where the Python 3.9 is installed. I believe the the scenario for me is that I have an interpreter that runs on Python 3.9 and Spyder's internal software runtime is on Python3.7 but will not interfere with my editor and console style being operated in Python 3.9?

for Windows and Mac apps the title bar would read "Spyder".

+1 about this option @ccordoba12

Thanks for chiming in @dalthviz! I'll implement that right away.


Unknown properties in MQTT for MIR-HE200-TY

Description of the issue
Hi
i use this device: https://www.zigbee2mqtt.io/devices/MIR-HE200-TY.html .
It's recognized by z2mqtt and working properly. The only issue is, that Occupancy and Presence is not published to Mqtt correctly, it's shown as "Unkown" in Home Assistant but when i look it up in Z2Mqtt i see true/false .
Addon version
e56dff4
Platform
Home Assistant 2023.2.5
Supervisor 2023.01.1
Operating System 9.5
Frontend 20230202.0 - latest
Logs of the issue (if applicable)
No response

This is fixed after updating to latest version (commit 73d6039)


add golang programming resource bangla

What does this PR do?
Add resource(s)
For resources
I have added Golang programming Bangla research
Why is this valuable (or not)?
This is the first course of Golang in the Bengali language
How do we know it's really free?
This course has been published on YouTube
For book lists, is it a book? For course lists, is it a course? etc.
This is a free video course
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Duplicate of #6609

Duplicate of #6617


ViewPager's selectedIndex prop does not change children

Issue type
I'm submitting a ...  (check one with "x")

[x] bug report
[ ] feature request

Issue description
Current behavior:
ViewPager's View does not change when selectedIndex props change.
Swiping changes selectedIndex properly.
(Please check the gif below)
Expected behavior:
When change selectedIndex props(with Hooks/setState), ViewPager's View changes
Steps to reproduce:

set ViewPager's selectedIndex prop as state
invoke state change(with Hooks/setState) the selectedIndex prop's state
ViewPager's View does not chage

Related code:
import React, { useState } from 'react';
import { View } from 'react-native';
import { Text, ViewPager, Button } from 'react-native-ui-kitten';

export default function MainPage(){
  const [pageIndex, setIndex] = useState(0);
  const tabNames = ['a', 'b', 'c', 'd', 'e'];
  return(
    <View style={{flex: 1}}>
      <View style={{flex: 0}}>
        <Button onPress={()=>{setIndex(pageIndex - 1)}}> - </Button>
        <Button onPress={()=>{setIndex(pageIndex + 1)}}> + </Button>
      </View>
      <ViewPager style={{flex: 1}}
        selectedIndex={pageIndex}
        onSelect={(number)=>{setIndex(number);}}>
          {tabNames.map((item, index) => (
            <View key={index}>
              <Text>{item}, {index}</Text>
            </View>
          ))}
      </ViewPager>
      <View style={{flex: 0}}>
        <Text>{pageIndex}</Text>
      </View>
    </View>
  )
};

Swiping works well, though.
I tried setState instead of hooks, the problem was same.
Other information:
OS, device, package version
Android 9.0
React native 0.60.4
react-native-ui-kitten 4.1.0

Thanks for reporting this. Will take a look👍

@cereme we've got a working example in master branch. Will be published in 4.2

Still have exactly the same issue on V5.1.1, on both platforms (iOS, Android).
Swiping works fine. External change of selectedIndex does not change the view.


Update how-to-use-parallel-job-in-pipeline.md

New data division method 'partition_keys' release.
Update example links to the new location.
Update important box when user use mltable as the input of their parallel job

Hi @alainli0928
This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead?
Thank you! We'll close this PR.


Should ninja prevent build.ninja from depending on the build?

We had a bug in LLVM's build recently where the build.ninja rule ended up depending on part of the build. This was effectively a circular dependency, and meant that incremental builds would spuriously fail on CMake changes, because the build file regeneration step would fail to build before it could perform the regeneration.
https://github.com/smeenai/ninja-regeneration is a simple repro to demonstrate what I mean:
git clone https://github.com/smeenai/ninja-regeneration
cd ninja-regeneration
git checkout HEAD^
mkdir build
cd build
cmake -G Ninja ..
git checkout main
ninja generate

You'll get a spurious link error, because build file regeneration is itself attempting to build generate, which needs the CMake changes to be reflected in the build system in order to work. Manually regenerating the build system (cmake -S .. -B .) will make the build work.
Should ninja error out for this sort of scenario? It never seems desirable and will always lead to broken incremental builds; it's basically a circular dependency in disguise (since all other targets implicitly depend on the build system regeneration). If this seems more like a CMake concern than a ninja concern, let me know and I'll ask there instead.

Ninja already detects circular dependencies that are found in its build plan (and will error correctly in this case).
It looks like the generated build plan is simply missing the right set of dependencies, and there is no special logic to handle build plan regeneration in the implementation itself. So I would conjecture that this is an issue with what CMake generates here.


Cut v2.33.5

Note that this bugfix release also includes the update of the
containerd/containerd dependency, with the only purpose of shutting up
security scanner spam. We usually do not decument dependency updates
in the CHANGELOG. In this case, it isn't even doing anything. I plan
to mention the update in the announcement mail.
Signed-off-by: beorn7<EMAIL_ADDRESS>

Similarly, I'll mention in the announcement mail that Go 1.17.8 is used for the pre-built binaries.

Just to confirm: Go 1.17.8 should be picked up automatically by the build pipeline. Right?


Windows Live Essentials Erroring out upon starting install in XP

Upon downloading said program from Legacy Update, it errors out with the error of OnCatalogResult 0x80090326 and the next steps of An unknown error occurred. clicking get help brings me to a "this page cannot be displayed" by IE.

Legacy Update should replace the exe that is downloaded with the offline installer, since that one works.

Here are all the offline installers for all versions and all languages of Windows Live Essentials:
http://web.archive.org/web/20150313192405/http://windows.microsoft.com/en-GB/windows-live/essentials-install-offline-faq
You can even change "en-GB" in the URL to your own language and probably get the information on the web page in your own language.
Please be aware of this: you may have to change the URL for the offline installer after having followed the appropriate link (that is, deleting some numbers and letters in the end of the URL). Unfortunately, there may be some minutes of further working with the URLs and the Wayback Machine before the download of the offline installer can begin.

A problem with replacing the installer with an offline one would be that Windows would probably throw a fit over a checksum/certificate being invalid.


update deps, pytest8, fix whitespace

closes #42
Description:

update to pytest 8
Important Notes:

.
Screenshots:

.
Checklist:

[ ] Code Quality
[ ] Tests (if applicable)
[ ] Success Criteria Met
[ ] Inline Documentation

@itsthejoker is this right? Or should we add a range of pytest versions?


Add missing closing string quote to Layout example in doc

Type of changes

[ ] Bug fix
[ ] New feature
[x] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Added a missing closing quote to an example in the Layout docs.

Thanks for the project, looking forward to trying out the new Layout!

Thanks


Emojis are scaled down & Emojis adding rules added to CONTRIBUTING.md

Closes #1217

The Dogehouse emojis are now correctly scaled
How to add emojis was added to CONTRIBUTING.md

did you mean 1213?

@jamesql Yes, Thank you


Output of fake-provider should be flushed after every request

Sometimes, when using the fake-provider service, a consumer does an invalid request, the client library ends the request and shuts down the FakeProvider. If this happens, then important log output is missing.

This is automatically fixed now where we're using the development logger of wai-extra package.


doc: Add a SECURITY.md

doc: Add SECURITY.md
doc: Expand the Security section
doc: Remove the Security section

I removed the Security section from CONTRIBUTING.md and added it to the Security section in README.md.


ScheduleDarkMode

Description of the new feature / enhancement
Scheduling the switch between Light and Dark mode depending on particular time very similar to toggling Night Light Schedule.
Scenario when this would be used?
This would be useful for me who work through hours day and night and seamlessly move from light to dark.
Supporting information
No response

Related to #1331

Thanks for your suggestion! This issue is a duplicate.
I'm linking this issue to #1331 to centralize the discussion thread, please go to that thread for discussion.
Thanks for your help to make PowerToys a better piece of software!


[Scaffolder] Support adding scaffolded components from Azure DevOps and GitLab to catalog

Feature Suggestion

Right now the scaffolder is hard-coded to construct a GitHub url for newly scaffolded components and try to add them to the catalog. This should also work for Azure DevOps and GitLab. I haven't checked checked GitLab but Azure DevOps does not work right now.
Possible Implementation
Move logic for adding scaffolded components from the frontend to scaffolder-backend. Also let the different publishers construct the final url to catalog-info.yaml since they differ between the different providers.

Context
We want scaffolded components in Azure DevOps to be available in the catalog.

@mfrinnstrom wanna pick this up yourself after we've got #2881 merged?

@benjdlambert we need it so why not

Would be great to also have this feature for Bitbucket Server!

@regicsolutions There doesn't seem to be any support for Bitbucket Server at all in Backstage right now. I'm trying to make that easier to add through #3313 &  #3351. If there had been initial support I could probably have added this functionality for that as well. I don't know Bitbucket Server well enough to add all of it though.

@benjdlambert do you want to keep this issue around or should new ones be created for the things I didn't include in the mentioned PRs?

Think we can close this and create issues for new stuff as and when it's needed.

Closing this then. @regicsolutions you might want to open a new issue for Bitbucket Server instead.


"Same day" distribution type for Exams

It will be nice to add a "Same day" distribution type for Exams similar to what currently exist for courses.
Best Regards

This feature has been added to CPSOLVER
https://github.com/UniTime/cpsolver/pull/14


Enhancement: Request: Add support for RDS Proxy Target Groups

Request
It would be great if Moto added support for db proxy target groups. According to the RDS feature list it is missing both the describe and modify action: [ ] describe_db_proxy_target_groups & [] modify_db_proxy_target_group. Using the following code, Terraform eventually times out after a long while and gives the following error:
# Error
│ Error: updating RDS DB Proxy Default Target Group (testdb): operation error RDS: ModifyDBProxyTargetGroup, exceeded maximum number of attempts, 25, https response error StatusCode: 500, RequestID: , error while deserializing xml error response: XML syntax error on line 2: unquoted or missing attribute value in element
│
│   with aws_db_proxy_default_target_group.main,
│   on rds.tf line 30, in resource "aws_db_proxy_default_target_group" "main":
│   30: resource "aws_db_proxy_default_target_group" "main" {

Thank you!
Reproduction
providers.tf
terraform {
  required_providers {
    aws = {
      source = "hashicorp/aws"
    }
  }
}

provider "aws" {
  region                      = "us-east-1"
  s3_use_path_style           = true
  skip_credentials_validation = true
  skip_metadata_api_check     = true
  skip_requesting_account_id  = true

  endpoints {
    ec2            = "http://localhost:5000"
    iam            = "http://localhost:5000"
    rds            = "http://localhost:5000"
    secretsmanager = "http://localhost:5000"
  }

  access_key = "my-access-key"
  secret_key = "my-secret-key"
}

rds-prereqs.tf
data "aws_vpc" "default" {
  default = true
}

data "aws_subnets" "all" {
  filter {
    name   = "vpc-id"
    values = [data.aws_vpc.default.id]
  }
}

resource "random_string" "this" {
  length  = 4
  special = false
  upper   = false
}

resource "aws_secretsmanager_secret" "test" {
  name = "rds/test/test_admin"
}

resource "aws_secretsmanager_secret_version" "test" {
  secret_id = aws_secretsmanager_secret.test.id
  secret_string = jsonencode({
    username = "test_admin"
    password = "${random_string.this.result}"
  })
}

data "aws_iam_policy_document" "rds_trust_policy" {
  statement {
    effect = "Allow"

    principals {
      type        = "Service"
      identifiers = ["rds.amazonaws.com"]
    }

    actions = [
      "sts:AssumeRole",
    ]
  }
}

resource "aws_iam_role" "proxy" {
  assume_role_policy = data.aws_iam_policy_document.rds_trust_policy.json
  name               = "rds-proxy-${aws_db_instance.test.db_name}"
}

rds.tf
resource "aws_db_instance" "test" {
  allocated_storage   = 10
  db_name             = "testdb"
  engine              = "postgres"
  engine_version      = "15"
  instance_class      = "db.t3.micro"
  username            = "test"
  password            = random_string.this.result
  skip_final_snapshot = true
}

resource "aws_db_proxy" "main" {
  name                   = aws_db_instance.test.db_name
  debug_logging          = false
  engine_family          = "POSTGRESQL"
  idle_client_timeout    = 1800
  require_tls            = true
  role_arn               = aws_iam_role.proxy.arn
  vpc_security_group_ids = []
  vpc_subnet_ids         = data.aws_subnets.all.ids

  auth {
    auth_scheme = "SECRETS"
    description = "Secrets through IAM Authentication"
    iam_auth    = "REQUIRED"
    secret_arn  = aws_secretsmanager_secret.test.arn
  }
}

resource "aws_db_proxy_default_target_group" "main" {
  db_proxy_name = aws_db_proxy.main.name

  connection_pool_config {
    connection_borrow_timeout    = 120
    max_connections_percent      = 100
    max_idle_connections_percent = 50
    session_pinning_filters      = ["EXCLUDE_VARIABLE_SETS"]
  }
}

resource "aws_db_proxy_target" "main" {
  db_instance_identifier = aws_db_instance.test.identifier
  db_proxy_name          = aws_db_proxy.main.name
  target_group_name      = aws_db_proxy_default_target_group.main.name
}

Expectation
It would be exceptional if I could create an aws_db_proxy_default_target_group and aws_db_proxy_target with Terraform using Moto to mock AWS for testing purposes for code utilizing these resources. Thanks again!
Current Setup
A very basic docker-compose.yml file:
  moto-server:
    image: motoserver/moto:latest
    ports:
      - 5000:5000
    environment:
      - MOTO_IAM_LOAD_MANAGED_POLICIES=true

Hi @jrindy-iterable, thanks for raising this - I've marked it as an enhancement to add this.
@bpandola Is this feature part of your RDS fork, by any chance? If not I'm happy to implement it, but I just wanted to check. If it is already implemented, there's no point in me doing the work again, and it will only make the merge of your work more difficult.

@bblommers Not in my fork.  Have at it. 😄


Seemingly random crash when jumping around

Jumping around in Player Playground screen causes a crash randomly. Extremely hard to reproduce.

 Bug


Type 'string' is not assignable to type 'GoogleVizEventName'.

I am using React ^16.8.6 with TypeScript 3.8.0-dev.20200109 and react-google-charts ^3.0.15.
I have followed verbatim the events example Listen to chart events in the README.
I receive the error
Type 'string' is not assignable to type 'GoogleVizEventName'.

and also the error
Type '{ eventName: string; callback(...}' is not assignmable to type 'ReactGoogleChartEvent'.

It is my understanding that react-google-charts now supports TypeScript out-of-the-box, so I'm not sure what is the correct solution for my issue.

Shouldn't happen on latest.


Oper login issue

I tried to configure the oper for my inspircd server except it keeps on stating that the fields login, password and hosts did not match. This happens when using the default opers.conf.example (renamed to opers.conf and moved to /conf) as well including the load of the module in my inspircd.conf and with the docker run comment with the variables:
Docker run command:

docker run -d
--name inspircd
--memory=128M
--memory-swap=256M
-p 6697:6697
-p 7001:7001
-e INSP_OPER_NAME=test
-e INSP_OPER_PASSWORD=test
-e INSP_OPER_HOST=@
-e INSP_OPER_HASH=hmac-sha256
-e INSP_OPER_SSONLY=yes
-v /volume4/Drive4/Apps/inspircd:/inspircd/conf/
inspircd/inspircd-docker

Noticed I used old conf files as well so did remove everything and let inspircd create everything automatic again and same issue exist but this time it only says that the password doesn't match

Oh it is possible a misunderstanding.
You have to provide the hashed password. Not the clear text one.
To create one, connect to the InspIRCd and use /mkpasswd hmac-sha256 yourpassword.

After a check back I noticed that the documentation is not clear about it. Well extend it :)

Ah ok thanks for the information got it to work now with ''.
Might as well be a suggestion that it automatic converts to hashed password? Dunno if it's even possible or requires to much work.
Still odd that it didn't accept the password for plain text and hashed from the opers.conf the first time. But w/e. The variables work

This container only contains InspIRCd. But @adam- and me also work on an anope container.
We build the whole setup following most of the best practices of Docker official images and provide one process per container. Current ETA for a basically usable Anope container is 2 weeks.

@DracoMilesX can you check the possible fix for this issue:
https://github.com/Adam-/inspircd-docker/blob/fix/issue-49/README.md
Would you say it's easier to find out, what you have to enter there?

@SISheogorath Yeah that looks better. Personally I would change the description to:

Hashed oper password

The wording kinda feels odd with the "for usage with /oper"
and modify the "/oper  " to "/oper  "
But this might just be me going full retard.

Thanks for the feedback :) I think the example is a better solution than providing longer example values to replace :)


Dockerfile support

Is it possible to configure drone so that if your project already has a Dockerfile, you can omit the script part of .drone.yml and rely on the Dockerfile to build the image?

this can be considered closed based on work in the 0.4 branch (upcoming release). We have a Docker plugin that can run in privileged mode (Docker in Docker) to build and push images. We also have the ability to mount volumes on the host machines (such as the Docker unix socket).
the ability to build and publish images is much stronger in 0.4

Hi bradrydzewski,
i used follow code to build image:
        - build -f Dockerfile_local .
i got error like:
unable to evaluate symlinks in Dockerfile path: lstat /python3/src/xxxx-xxxxx.xxxxxxxxx.com/xxxx/Pubhub_Test_Drone/Dockerfile:
so it looks it still find the Dockerfile rather than Dockerfile_local
could you please tell me if i can use specify dockerfile?
thank you very much


Local omnibar nav and routes rename

Added support for specifying local navigation in skyuxconfig.json

Refactored publicRoutes property to routes with public and referenced properties in preparation for publishing to the nav service

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #152   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          34     34
  Lines         694    694
  Branches       85     85
=====================================
  Hits          694    694

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

runtime/config.ts
100% <ø> (ø)
: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 c527e66...20c1c62. Read the comment docs.


EventEmitter warnings in RN 0.65

In React Native 0.65 removeListener was deprecated and users of react-native-agora will get a warning every time Agora calls this function. Unfortunately I don't know if there is a backwards compatible way of fixing that yet. Similar PR https://github.com/react-native-modal/react-native-modal/pull/594

duplicate of #409

Hi, this is not related. The warnings are still there when using the code from my PR.


Add a pure solidity verify (non)membership implementaton

Read #84 for more details.

An implementation can be found in TendermintTreeVerifier.sol.
Another implementation of this can be found in union's repository

Closing due to https://github.com/cosmos/sp1-ics07-tendermint/issues/84#issuecomment-2400451198

Reopening this due to #114


Projects#show page

As contribulator is getting merged in here (see https://github.com/24pullrequests/24pullrequests/discussions/3514), we should have a project show page that shows the details of the project and it's score breakdown as well as contributors, maintainers and pull requests.

🤔 I could poke at this one over the next few days. Any particular mockup for the page or just use my best judgment to come up with a v1 that is similar to the rest of the site styling?

I don't have any mockups yet, I was imaging something like the user contribution page https://24pullrequests.com/users/emclaughlin1215 (excluding the calendar view).
If you want to put some basic details together I can help with suggestions on styling, maybe attach some screenshots to your pull request for easy reviewing.
Thanks for all the contributions this year, you've had a huge impact!

Hey @emclaughlin1215 Have you PR for this issue?

can i work on this issue ? is it still active ?

@sarvo123 yep


Bump 0.7.0-rc1

Signed-off-by: Nana Essilfie-Conduah<EMAIL_ADDRESS>Description:
Bump 0.7.0-rc1
Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #479 (857ba36) into release/0.7 (4c3191c) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/0.7     #479   +/-   ##
============================================
  Coverage        76.38%   76.38%
============================================
  Files               12       12
  Lines              923      923
  Branches           144      144
============================================
  Hits               705      705
  Misses             165      165
  Partials            53       53

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


groups and reference in sc.tl.rank_genes_groups

I'm afraid, I do not fully understand the documentation of sc.tl.rank_genes_groups when using the groups and reference arguments. Suppose that I have the categories '1', '2', '3' in my observation grouping louvain. Now, I run
sc.tl.rank_genes_groups(adata, 'louvain', groups=['1', '2'], reference='rest')

Would the reference now include cells from category '3'?

What worries me is that the behaviour you describe might not so for method='logreg'. Let's have a look at the following example:
import scanpy.api as sc
adata = sc.datasets.blobs()
sc.pp.scale(adata)

sc.pp.neighbors(adata)
sc.tl.louvain(adata)

for method in ['t-test', 'logreg']:
    # first call, without groups
    sc.tl.rank_genes_groups(adata, 'louvain', method=method)
    sc.pl.rank_genes_groups_matrixplot(adata, n_genes=1)

    # first call, with groups
    sc.tl.rank_genes_groups(adata, 'louvain', method=method, groups=['0', '1', '2'])
    sc.pl.rank_genes_groups_matrixplot(adata, n_genes=1)

As setting groups to ['0', '1', '2'] should not change the reference dataset, exactly the same marker genes should be detected for the first and the second call of sc.tl.rank_genes_groups. This is indeed true if I set the method to t-test. However, when setting method to logreg, I get other marker genes. Visually it appears to me that only the groups ['0', '1', '2'] are used a the reference set when using the method logreg.

Yes, right, for logreg you always regress on data that includes the respective group as this is not a differential test, but a regression. It's a shame that this is not evident from the documentation. It's sort of a recent alternative way of approaching the definition of marker genes and we should give it a bit more of a thought.
Regarding the behavior that is inconsistent up there, there is the following in the code by the person who extended this not long ago:
        # if reference is not set, then the groups listed will be compared to the rest
        # if reference is set, then the groups listed will be compared only to the other groups listed
        from sklearn.linear_model import LogisticRegression
        if len(groups) == 1:
            raise Exception('Cannot perform logistic regression on a single cluster.')
        adata_copy = adata[adata.obs[groupby].isin(groups_order)]
        adata_comp = adata_copy
        if adata.raw is not None and use_raw:
            adata_comp = adata_copy.raw
        X = adata_comp.X

        clf = LogisticRegression(**kwds)
        clf.fit(X, adata_copy.obs[groupby].cat.codes)

You're right that logreg only includes the passed groups, if groups are passed. This should not be the case. I wonder why it's a problem in your specific case as I'd expect that 0, 1, 2 make up the whole data; but maybe the resolution in Louvain is somewhat set to a high value.
In any case, I'll change the implementation so that irrespective of whether groups is passed or not, one gets the same result.

Hi @falexwolf, thanks a lot for the clarifications. This helps me a lot. In the example I provided yesterday, louvain found 5 clusters, so 0, 1, 2 made up only part of the data. I should have provided the output as well to make this clear right away.
Concerning a PR for the documentation, I think I would wait until you will update the behaviour of logreg.


Add Category "Frameworks" with UI libraries and frameworks

This PR will add new category "Frameworks" which will consist UI Libraries and Frameworks, I tried to add category "UIFrameworks" but there was some problem.
I got rate limit exceeded error so only added 15 for now, atlaskit from Atlassian is hosted in bitbucket so am not sure if it works.
Resource: https://hackernoon.com/23-best-react-ui-component-libraries-and-frameworks-250a81b2ac42
Icon: flaticon.com

Hey thanks @samundrak !
Looks like a few things are failing, I will take a look

Sure, Great App

Thanks for the contribution @samundrak , all merged :)


Can't install the font-manager through npm

Hey,
I get this error when i install the font-manager, am i missing something?

219 info install<EMAIL_ADDRESS>220 verbose unsafe-perm in lifecycle true
221 info<EMAIL_ADDRESS>Failed to exec install script
222 verbose unlock done using C:\Users\Alexander\AppData\Roaming\npm-cache_locks\font-manager-6db06e008ef16939.lock for C:\Electron\font\node_modules\font-manager
223 verbose stack Error<EMAIL_ADDRESS>install: node-gyp rebuild
223 verbose stack Exit status 1
223 verbose stack     at EventEmitter. (C:\Users\Alexander\AppData\Roaming\npm\node_modules\npm\lib\utils\lifecycle.js:212:16)
223 verbose stack     at emitTwo (events.js:100:13)
223 verbose stack     at EventEmitter.emit (events.js:185:7)
223 verbose stack     at ChildProcess. (C:\Users\Alexander\AppData\Roaming\npm\node_modules\npm\lib\utils\spawn.js:14:12)
223 verbose stack     at emitTwo (events.js:100:13)
223 verbose stack     at ChildProcess.emit (events.js:185:7)
223 verbose stack     at maybeClose (internal/child_process.js:821:16)
223 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
224 verbose pkgid<EMAIL_ADDRESS>225 verbose cwd C:\Electron\font
226 error Windows_NT 10.0.10586
227 error argv "C:\Program Files (x86)\nodejs\node.exe" "C:\Users\Alexander\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "font-manager"
228 error node v5.5.0
229 error npm  v2.1.18
230 error code ELIFECYCLE
231 error<EMAIL_ADDRESS>install: node-gyp rebuild
231 error Exit status 1
232 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
232 error This is most likely a problem with the font-manager package,
232 error not with npm itself.
232 error Tell the author that this fails on your system:
232 error     node-gyp rebuild
232 error You can get their info via:
232 error     npm owner ls font-manager
232 error There is likely additional logging output above.

@AlexanderYW - Do you have node-gyp installed globally on your system?

I had, but i figured it out after alot of work
I'm on windows so i had to set the settings for node vsbuild.exe to use the 2015 edition instead of 2012 or what default was, then i had to install the old python 2.7.x with the PATH setup correct, after all that i still got errors then i did a npm update -g which fixed everything
But thanks anyway! :)


Cost 300_Optimization_Data_Collection :  Overuse of sts assume role in organization accounts

On code : OrganizationData from child accounts
Theses files :

300_Optimization_Data_Collection/Code/rds_util_template.yaml
300_Optimization_Data_Collection/Code/source/ecs/ecs.py
300_Optimization_Data_Collection/Code/source/fof/ami.py
300_Optimization_Data_Collection/Code/source/fof/ebs.py
300_Optimization_Data_Collection/Code/source/fof/snapshot.py

Issue :
The sts assume role is currently called for each service and each region. A quick improvement could be to use a sts session and reuse it for each region.
Doing this sts calls will be reduced to only 1 for each service instead of one per region and per service.

Hey, thank you for this PR I will review and get back to you ASAP :D

Merged, thank you for your contribution


Improvement limit the column size on tablet schema

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


License

Hello.
I would like to add your tool into ArchStrike, but I can't without a license file.
Is it possible for you to add in a license file?
Thanks.

I'll get on it and get back to you within a few hours. Happy hunting!

License has been added. Thanks for working on ArchStrike!

Just added in backdoorme-git to the ArchStrike repositories. Thanks for the license!


Windows v0.17.0.beta-1 not loading check extensions

A couple odd things happening with the latest windows install:

The client starts, but only puts "warn" level messages in c:/opt/sensu/sensu-client.log, no "info" or "debug" messages with debugging on
There are no "loading extension" entries in the log as there should be with a "extension_dir" attribute in the config
I attempted to change the logging with "log_file" and "log_level" to "debug" but neither of these take affect and change the log output or the file path, as they should
If the extension is being loaded and that's holding it up, it should at least tell me it's in the process of loading the extension. That snippet of logging code happens before the extension file is opened
To confound things further, the client sends standalone check responses to the server (it has no subscriptions)

Below is the log file in its entirety, plus the list of installed gems:
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config files from directory","directory":"C:\\etc\\sensu\\conf.d"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/checks/check_disk_windows.json"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/checks/check_windows_cpu_load.json"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/checks/check_windows_cpu_load.json","changes":{"checks":{"check_windows_cpu_load":[null,{"command":"/opt/sensu/embedded/bin/ruby /etc/sensu/plugins/windows/check-windows-cpu-load.rb","interval":60,"handler":"notification","standalone":true,"subscribers":[]}]}}}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/checks/disk_usage_metrics.json"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/checks/disk_usage_metrics.json","changes":{"checks":{"disk_usage_metrics":[null,{"command":"/opt/sensu/embedded/bin/ruby /etc/sensu/plugins/windows/disk-usage-metrics.rb -s REDACTED","interval":60,"handler":"metrics","type":"metric","standalone":true,"subscribers":[]}]}}}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/checks/ram_usage_windows.json"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/checks/ram_usage_windows.json","changes":{"checks":{"ram_usage_windows":[null,{"command":"/opt/sensu/embedded/bin/ruby /etc/sensu/plugins/windows/ram-usage-windows.rb -s REDACTED","interval":60,"handler":"metrics","type":"metric","standalone":true,"subscribers":[]}]}}}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/checks/windows_cpu_load_metrics.json"}
{"timestamp":"2015-03-10T20:34:02.888184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/checks/windows_cpu_load_metrics.json","changes":{"checks":{"windows_cpu_load_metrics":[null,{"command":"/opt/sensu/embedded/bin/ruby /etc/sensu/plugins/windows/windows-cpu-load-metrics.rb -s REDACTED","interval":60,"handler":"metrics","type":"metric","standalone":true,"subscribers":[]}]}}}
{"timestamp":"2015-03-10T20:34:02.889184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/client.json"}
{"timestamp":"2015-03-10T20:34:02.889184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/client.json","changes":{"client":[null,{"name":"REDACTED","address":"REDACTED","keepalive":{"thresholds":{"warning":120,"critical":180},"handler":"notification"},"subscriptions":[]}]}}
{"timestamp":"2015-03-10T20:34:02.889184+0000","level":"warn","message":"loading config file","file":"C:/etc/sensu/conf.d/common.json"}
{"timestamp":"2015-03-10T20:34:02.889184+0000","level":"warn","message":"config file applied changes","file":"C:/etc/sensu/conf.d/common.json","changes":{"extension_dir":[null,"/etc/sensu/extensions/checks"],"rabbitmq":[null,{"host":"REDACTED","port":"5672","user":"sensu","password":"REDACTED","vhost":"/sensu"}]}}

    amq-protocol (1.9.2)
    amqp (1.5.0)
    async_sinatra (1.0.0)
    bigdecimal (1.2.0)
    childprocess (0.5.3)
    daemons (1.2.0)
    em-redis-unified (0.5.0)
    em-worker (0.0.2)
    eventmachine (1.0.7)
    ffi (1.9.6 x86-mingw32)
    io-console (0.4.2)
    json (1.7.7)
    minitest (4.3.2)
    mixlib-cli (1.5.0)
    multi_json (1.10.1)
    psych (2.0.0)
    rack (1.6.0)
    rack-protection (1.5.3)
    rake (0.9.6)
    rdoc (4.0.0)
    redphone (0.0.6)
    sensu (0.17.0.beta)
    sensu-em (2.4.1)
    sensu-extension (1.0.0)
    sensu-extensions (1.0.0)
    sensu-logger (1.0.0)
    sensu-plugin (1.1.0)
    sensu-settings (1.2.0)
    sensu-spawn (1.1.0)
    sensu-transport (2.4.0)
    sinatra (1.3.5)
    test-unit (<IP_ADDRESS>)
    thin (1.5.0)
    tilt (1.4.1)
    uuidtools (2.1.4)

Did you specify an extension directory in the service XML config? eg. -e C:\opt\sensu\extensions\

As @portertech suggested, a change had to be made to C:\opt\sensu\bin\sensu-client.xml to include -e C:\opt\sensu\extensions\. No changes are necessary on linux to load extensions because the init scripts by default set -e /etc/sensu/extensions.
To all who come across this issue in the future:
https://github.com/sensu/sensu/blob/master/lib/sensu/cli.rb#L29
(Note: The extension_dirs and extention_file options are not yet exposed through the CLI. https://github.com/sensu/sensu-extensions/blob/master/lib/sensu/extensions.rb)


[XLA] Fix a bug.

The Span returned point to an initializer list that is removed. This isn't allowed.
This is triggered in Ubuntu 20.03 that have GCC 9.3.
@sanjoy @nluehr

LGTM, but can you please use a more specific subject :) maybe "Fix a dangling reference bug"?

Edited. I also added a little bit more information in the description.

@nouiz  Can you please check @sanjoy's comments and keep us posted ? Thanks!

I amended the commit. I had understood that @sanjoy would do the fix internally.

Any update? This bug is in the TF2.4 release. Should it be backported?

I'm not surprised. I just too a chance by asking.


Pylance from time to time crash (when ssh to remove repo).

 Environment data

Language Server version: 2024.7.101
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

XXX

 Repro Steps

XXX

 Expected behavior
XXX
 Actual behavior
XXX
 Logs
2024-07-24 23:26:04.022 [info] (Client) Pylance async client (2024.7.101) started with python extension (2024.10.0)
2024-07-24 23:26:04.533 [info] [Info  - 11:26:04 PM] (3999797) Pylance language server 2024.7.101 (pyright version 1.1.372, commit d6372d45) starting
2024-07-24 23:26:04.533 [info] [Info  - 11:26:04 PM] (3999797) Server root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:04.533 [info] [Info  - 11:26:04 PM] (3999797) Starting service instance "aigc_inpaint_controlnet_2"
2024-07-24 23:26:04.534 [info] [Info  - 11:26:04 PM] (3999797) Starting service instance "samples_parallel__15000_9Unet_depthv4_24_8_iter_all_updated_mask"
2024-07-24 23:26:04.534 [info] [Info  - 11:26:04 PM] (3999797) Starting service instance "video_inpaint"
2024-07-24 23:26:04.534 [info] [Info  - 11:26:04 PM] (3999797) Starting service instance "samples_parallel__40000_13Unet_depthv4_48_interval_2_8_iter_None_all_v3"
2024-07-24 23:26:04.534 [info] [Info  - 11:26:04 PM] (3999797) Starting service instance "WebVid_validation"
2024-07-24 23:26:04.594 [info] [Info  - 11:26:04 PM] (3999797) Setting pythonPath for service "aigc_inpaint_controlnet_2": "/nfs/flash/VideoAlg/personal/us000110/miniconda3/envs/controlnet2/bin/python"
2024-07-24 23:26:04.595 [info] [Info  - 11:26:04 PM] (3999797) Setting environmentName for service "aigc_inpaint_controlnet_2": "3.9.19 (controlnet2 conda)"
2024-07-24 23:26:04.602 [info] [Info  - 11:26:04 PM] (3999797) No include entries specified; assuming /nfs.auto/flash_VideoAlg/personal/us000110/codebases/aigc_inpaint_controlnet_2
2024-07-24 23:26:04.602 [info] [Info  - 11:26:04 PM] (3999797) Auto-excluding **/node_modules
2024-07-24 23:26:04.602 [info] [Info  - 11:26:04 PM] (3999797) Auto-excluding **/__pycache__
2024-07-24 23:26:04.602 [info] [Info  - 11:26:04 PM] (3999797) Auto-excluding **/.*
2024-07-24 23:26:04.775 [info] [Info  - 11:26:04 PM] (3999797) Assuming Python version 3.9.19.final.0
2024-07-24 23:26:08.497 [info] [Info  - 11:26:08 PM] (3999797) Found 427 source files
2024-07-24 23:26:08.524 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(1) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:08.524 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(1) started
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(2) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(2) started
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(3) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(3) started
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(4) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:08.525 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(4) started
2024-07-24 23:26:08.526 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(5) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-24 23:26:08.526 [info] [Info  - 11:26:08 PM] (3999797) Background analysis(5) started
2024-07-24 23:26:08.530 [info] [Info  - 11:26:08 PM] (3999797) Setting pythonPath for service "samples_parallel__15000_9Unet_depthv4_24_8_iter_all_updated_mask": "/usr/bin/python3"
2024-07-24 23:26:08.530 [info] [Info  - 11:26:08 PM] (3999797) Setting environmentName for service "samples_parallel__15000_9Unet_depthv4_24_8_iter_all_updated_mask": "3.8.10 (global)"
2024-07-24 23:26:08.530 [info] [Info  - 11:26:08 PM] (3999797) No include entries specified; assuming /nfs.auto/flash_VideoAlg/personal/u0403683/Video_Inpainting/animatediff/outputs/samples_parallel__15000_9Unet_depthv4_24_8_iter_all_updated_mask
2024-07-24 23:26:08.531 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/node_modules
2024-07-24 23:26:08.531 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/__pycache__
2024-07-24 23:26:08.531 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/.*
2024-07-24 23:26:08.560 [info] [Info  - 11:26:08 PM] (3999797) Assuming Python version 3.8.10.final.0
2024-07-24 23:26:08.728 [info] [Info  - 11:26:08 PM] (3999797) No source files found.
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) Setting pythonPath for service "video_inpaint": "/usr/bin/python3"
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) Setting environmentName for service "video_inpaint": "3.8.10 (global)"
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) No include entries specified; assuming /nfs.auto/flash_VideoAlg/personal/us000110/proj_misc/video_inpaint
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/node_modules
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/__pycache__
2024-07-24 23:26:08.732 [info] [Info  - 11:26:08 PM] (3999797) Auto-excluding **/.*
2024-07-24 23:26:08.766 [info] [Info  - 11:26:08 PM] (3999797) Assuming Python version 3.8.10.final.0
2024-07-24 23:26:18.878 [info] [Error - 11:26:18 PM] (3999797) Enumeration of workspace source files is taking longer than 10 seconds.
This may be because:
* You have opened your home directory or entire hard drive as a workspace
* Your workspace contains a very large number of directories and files
* Your workspace contains a symlink to a directory with many files
* Your workspace is remote, and file enumeration is slow
To reduce this time, open a workspace directory with fewer files or add a pyrightconfig.json configuration file with an "exclude" section to exclude subdirectories from your workspace. For more details, refer to https://github.com/microsoft/pyright/blob/main/docs/configuration.md.
2024-07-24 23:26:22.482 [info] [Info  - 11:26:22 PM] (3999797) No source files found.
2024-07-24 23:26:22.490 [info] [Info  - 11:26:22 PM] (3999797) Setting pythonPath for service "samples_parallel__40000_13Unet_depthv4_48_interval_2_8_iter_None_all_v3": "/usr/bin/python3"
2024-07-24 23:26:22.490 [info] [Info  - 11:26:22 PM] (3999797) Setting environmentName for service "samples_parallel__40000_13Unet_depthv4_48_interval_2_8_iter_None_all_v3": "3.8.10 (global)"
2024-07-24 23:26:22.490 [info] [Info  - 11:26:22 PM] (3999797) No include entries specified; assuming /nfs/flash/VideoAlg/personal/u0403683/Video_Inpainting/animatediff/outputs/samples_parallel__40000_13Unet_depthv4_48_interval_2_8_iter_None_all_v3
2024-07-24 23:26:22.491 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/node_modules
2024-07-24 23:26:22.491 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/__pycache__
2024-07-24 23:26:22.491 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/.*
2024-07-24 23:26:22.523 [info] [Info  - 11:26:22 PM] (3999797) Assuming Python version 3.8.10.final.0
2024-07-24 23:26:22.641 [info] [Info  - 11:26:22 PM] (3999797) No source files found.
2024-07-24 23:26:22.644 [info] [Info  - 11:26:22 PM] (3999797) Setting pythonPath for service "WebVid_validation": "/usr/bin/python3"
2024-07-24 23:26:22.644 [info] [Info  - 11:26:22 PM] (3999797) Setting environmentName for service "WebVid_validation": "3.8.10 (global)"
2024-07-24 23:26:22.644 [info] [Info  - 11:26:22 PM] (3999797) No include entries specified; assuming /nfs/flash/VideoAlg/AIGC/datasets/SHARE/WebVid/WebVid_validation
2024-07-24 23:26:22.645 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/node_modules
2024-07-24 23:26:22.645 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/__pycache__
2024-07-24 23:26:22.645 [info] [Info  - 11:26:22 PM] (3999797) Auto-excluding **/.*
2024-07-24 23:26:22.677 [info] [Info  - 11:26:22 PM] (3999797) Assuming Python version 3.8.10.final.0
2024-07-24 23:26:22.831 [info] [Info  - 11:26:22 PM] (3999797) No source files found.
2024-07-24 23:26:23.249 [info] [Info  - 11:26:23 PM] (3999797) Indexer background runner(6) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist (index)
2024-07-24 23:26:23.250 [info] [Info  - 11:26:23 PM] (3999797) Indexing(6) started
2024-07-24 23:26:25.721 [info] [Info  - 11:26:25 PM] (3999797) [IDX(6)] Long operation: scan packages file:///nfs.auto/flash_VideoAlg/personal/us000110/codebases/aigc_inpaint_controlnet_2 (2318ms)
2024-07-24 23:26:25.721 [info] [Info  - 11:26:25 PM] (3999797) scanned(6) 351 files over 1 exec env
2024-07-24 23:26:26.387 [info] [Info  - 11:26:26 PM] (3999797) indexed(6) 265 files over 1 exec env
2024-07-24 23:26:26.552 [info] [Info  - 11:26:26 PM] (3999797) Indexing finished(6).
2024-07-25 00:16:35.386 [info] [Info  - 12:16:35 AM] (3999797) Starting service instance ""
2024-07-25 00:16:35.435 [info] [Info  - 12:16:35 AM] (3999797) Setting pythonPath for service "": "/nfs/flash/VideoAlg/personal/us000110/miniconda3/envs/controlnet2/bin/python"
2024-07-25 00:16:35.436 [info] [Info  - 12:16:35 AM] (3999797) Setting environmentName for service "": "3.9.19 (controlnet2 conda)"
2024-07-25 00:16:35.437 [info] [Info  - 12:16:35 AM] (3999797) No include entries specified; assuming /
2024-07-25 00:16:35.437 [info] [Info  - 12:16:35 AM] (3999797) Auto-excluding **/node_modules
2024-07-25 00:16:35.437 [info] [Info  - 12:16:35 AM] (3999797) Auto-excluding **/__pycache__
2024-07-25 00:16:35.437 [info] [Info  - 12:16:35 AM] (3999797) Auto-excluding **/.*
2024-07-25 00:16:35.549 [info] [Info  - 12:16:35 AM] (3999797) Assuming Python version 3.9.19.final.0
2024-07-25 00:16:35.838 [info] [Error - 12:16:35 AM] (3999797) File or directory "/" does not exist.
2024-07-25 00:16:35.839 [info] [Info  - 12:16:35 AM] (3999797) No source files found.
2024-07-25 00:16:35.880 [info] [Info  - 12:16:35 AM] (3999797) Background analysis(7) root directory: file:///nfs.auto/home/us000110/.vscode-server/extensions/ms-python.vscode-pylance-2024.7.101/dist
2024-07-25 00:16:35.880 [info] [Info  - 12:16:35 AM] (3999797) Background analysis(7) started

XXX

Enumeration of workspace source files is taking longer than 10 seconds. This may be because:
* You have opened your home directory or entire hard drive as a workspace
* Your workspace contains a very large number of directories and files
* Your workspace contains a symlink to a directory with many files
* Your workspace is remote, and file enumeration is slow
To reduce this time, open a workspace directory with fewer files or add a pyrightconfig.json configuration file with an "exclude" section to exclude subdirectories from your workspace. For more details, refer to https://github.com/microsoft/pyright/blob/main/docs/configuration.md. 2024-07-24

This messsage in your logs indicates that the enumerations of files in your workspace is taking longer than 10 seconds. Could you please check the root of your workspace to see if it matches any of the scenarios described in the error message?

From log, we see No source files found. multiple times, but yet we get these message Enumeration of workspace source files is taking longer than 10 seconds.
I think your repo setup is not a typical set up our heuristic based auto setup can handle. it might be better for you to manually setup your workspace. see this link for detail/help - https://github.com/microsoft/pylance-release/wiki/Opening-Large-Workspaces-in-VS-Code#manually-configure-your-workspace