disableAjaxTracking flag is not working

As reported here disableAjaxTracking: true is not working as expected. Right now the only mitigation I could suggest is to use maxAjaxCallsPerView:0

Can't repro it for some reason

I can't repro either. The disableAjaxTracking seems to be working fine. Configuration is stored in the global var: window["appInsights"].config.disableAjaxTracking , and if it's false than the AjaxMonitor is not initialized.
@AlexBulankou, you said that you can repro it. Could you provide more details?

I definitely had a repro, but now I no longer do :( Will ask on the forum if they can help with a repro and reactivate if needed.


Provide examples of FASTA + GFF (decribing 2 transcripts with overlapping ORFs) for bacterial genome.

As discussed on the 18 March call, John, can you please give us some such examples in this ticket?
Thanks.

Here is an E coli. genomic fasta and a gff3 that has only two genes where the last base of the stop codon of the first gene is the first base of the start codon of the next gene. The .txt file is the gff, I had to give it a different extension to upload it here.
e_coli_genome.fasta.gz
e_coli_overlapping_genes.txt

We can close this now, as @john-s-f  has provided an example. Next, #159 completion includes making transcript- and/or ORF-centric gffs using this example.


Missing grab_frame event from ImageCapture

Describe the Bug
When i'm trying to capture the webcam image via ImageCapture, the grab_frame method is missing and also the take_photo method seems to be a void.

I wonder if this is a case where our WebIDL we're using is out of date? The source for these APIs doesn't list grab_frame and has a different signature of take_photo than what's documented on MDN. Would you be interested in looking to see if there's an updated version of the WebIDL we can use?

Yes i would love to.  But to be very honest am completely new to the term WebIDL. So where should i start?

@fitzgen where was it again you pulled all the original WebIDL from?

https://searchfox.org/mozilla-central/source/dom/webidl

Seems like they have the old one too.

@mahadeer13 maybe you could check other browsers? Is this feature implemented in Firefox?

@alexcrichton My bad.  These are an experimantal features only works in chrome for now.

No worries!
In that case for now I'm going to close this since we're only adding standardized APIs to js-sys/web-sys for now.
Note though that you can always bind the API locally in your crate without the assistance of web-sys, it'd look like:
#[wasm_bindgen]
extern "C" {
    type ImageCapture;
    #[wasm_bindgen(method, js_name = grabFrame)]
    fn grab_frame(this: &ImageCapture);
}

(etc)

Yes thanks for the suggestion, but i'm trying to create some benchmark on drivers using WebSys and Native JavaScript. So i prefer the Rusty way, with just two lines of JS Code.
Or maybe from what you say, every web_sys is going to call the native javascript method again?

@mahadeer13 Yes, the APIs only exist in JS (not wasm), so every time you call a web_sys (or js_sys or wasm_bindgen) API it will make a call into JS.

@Pauan Thanks.


Make watcher's path absolute for notify's MacOS fsevent implementation

This is a proposed fix for #154
I have chosen to replicate the logic present in notify's inotify implementation. In the notify crate, these checks make it so that there is no error when presenting a watcher with an empty path ("") on Linux, but they are missing from MacOS's fsevent implementation, and therefore I propose to always offer the watcher an absolute path.
Another possible solution is to handle the error returned from this call:
watcher.watch(parent_path, RecursiveMode::NonRecursive)?;

using something like
if let Err(e) = watcher.watch(parent_path, RecursiveMode::NonRecursive) {
  // handle...
}

but this means that one could not run rathole on MacOS using this syntax: rathole config.toml

Nice catch. Thanks for your contribution!


Duration Scalar

This is just a quick draft of a timedelta/Duration scalar. Please let me know what you think of the input structure. If you like it I will work on the tests and changelog stuff.

Duration is a good idea yes.
Please, don't forget to add docs & test :smile:

I like it very much, once #27 and #24 are merged, we'll release a 0.2.0, don't forget to add this feature to the next.md.
Thanks again for your contributions.


WIP: More specs/docs/tests for iter

This PR will deal with https://github.com/mirage/irmin/issues/1134 (more tests) but I will also take the opportunity to improve the documentation.
I need your inputs first.

It's indeed a good idea to specify and test the iter order for the commit history graph. The current one seems wrong. On both case (rev=true or rev=false) we want all the predecessors to be visited before a node can be processed.

@Ngoguey42 Can you open an issue to track this (and close this PR)? The order seems indeed quite wrong to me.


#1256 Build the app in a Docker container

This PR ...
Changes
./prime-router

This adds a prerequisite that docker is installed on the developer's machine (due to the use of <IP_ADDRESS> as the bridge gateway to provide connectivity across docker-compose and local builds and test runs). Instead of using localhost, code now uses this gateway address (which ends you at the same 'location' in the end)

Adds a docker-compose environment (docker-compose.build.yml) which provides a standardized build environment for the product. This environment builds the builder as defined in Dockerfile.build.

Removes the docker-infrastructure.yml docker-compose environment (as it is now supplanted by docker-compose.build.yml); note that the new build docker-compose does NOT include adminer to keep it more 'lightweight'. Anyone who wants to put an admin UI on top of the (exposed) PGSQL can just point at their own localhost:54342 with the tool of their choice.

Adds a ./build.sh script to do the build in the container with the ability to run different build targets; this script can be used on a developer's box to perform a build in a similar environment that the GitHub pipeline performs a build in. This script should be the replacement for gradlew.

./.github/workflows

Updates the GitHub workflows build_hub.yml ("Router") and release.yml ("Release to azure") to use the new container-based build mechanism

Checklist
Outstanding question

Should we use host.docker.internal instead of <IP_ADDRESS>? This would require modification of hosts files but might make life a tiny bit easier down the line?.

What/How to validate
$ cd prime-router # all runs from prime-router
# Stop everything docker (note: this will get rid of all docker containers and their configurations)
docker stop -t 1 <all_containers>  # or the equivalent necessary docker-compose downs if you have compose environments running
docker system prune --volumes -f  # Prune your system, including your volumes
docker-compose --file docker-compose.yml up  # Bring up the docker-compose development environment pre-requisites
#
# Actual validation; if you have a lot of time, reset your environment in between each using the steps above
#
./build.sh -- gradle clean
./build.sh  # just a simple build (equivalent to -- gradle package)
./build.sh -- gradle clean package -x fatjar # Generate the large jar
./build.sh -- gradle generatedocs  # generate documentation
./build.sh -- gradle compile
./build.sh -- gradle migrate
./build.sh -- gradle test
# Bring up a proper environment to run E2E tests
docker-compose --file docker-compose.yml up -d # or without the -d in a different terminal to see the logs scroll by (but then wait for it to come up properly)
./build.sh -- gradle testEnd2End
./build.sh -- gradle testIntegration
./build.sh -- gradle primeCLI

Testing

[x] Tested locally?
[ ] Ran quickTest all?
[ ] Ran ./prime test against local Docker ReportStream container?
[ ] Downloaded a file from http://localhost:7071/api/download?
[ ] Added tests?

Security

[x] Did you check for sensitive data, and remove any?
[x] Does logging contain sensitive data?
[ ] Are additional approvals needed for this change?
[x] Are there potential vulnerabilities or licensing issues with any new dependencies introduced?

Process

[x] Includes a summary of what a code reviewer should verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[x] DevOps team has been notified if PR requires ops support?

Security
List potential security threats and mitigations in the PR
Fixes
List GitHub issues this PR fixes

#1256 : Build the app in a Docker container

To Be Done

#1012: Integrate Docker-Compose plugin into Gradle

Can you provide a bit more info on this?  What is this for and for whom?  Use cases?

Can you provide a bit more info on this? What is this for and for whom? Use cases?

The purpose of this is to provide a standardized, reproducible environment in which we build. This is not about producing containers as output of the build and instead is about using containers during the build. This will ensure consistency in the produced artifacts (and make it easier to have an environment in which you can perform a build).
Secondly, as per #1012, it'd be great if all of this is neatly integrated with our current build system (Gradle) in such a way that any of the dependant (as opposed to dependent) container is spun up automatically when you initiate a build.

Can you provide a bit more info on this? What is this for and for whom? Use cases?

#1256: The purpose of this is to provide a standardized, reproducible environment in which we build. This is not about producing containers as output of the build and instead is about using containers during the build. This will ensure consistency in the produced artifacts (and make it easier to have an environment in which you can perform a build).
Secondly, as per #1012, it'd be great if all of this is neatly integrated with our current build system (Gradle) in such a way that any of the dependant (as opposed to dependent) container is spun up automatically when you initiate a build.

I guess, my confusion is that Gradle is supposed to

Can you provide a bit more info on this? What is this for and for whom? Use cases?

#1256: The purpose of this is to provide a standardized, reproducible environment in which we build. This is not about producing containers as output of the build and instead is about using containers during the build. This will ensure consistency in the produced artifacts (and make it easier to have an environment in which you can perform a build).
Secondly, as per #1012, it'd be great if all of this is neatly integrated with our current build system (Gradle) in such a way that any of the dependant (as opposed to dependent) container is spun up automatically when you initiate a build.

I am not sure I am following this approach.  I wanted to run it locally, but I get an error on the script.  Can you add usage details on how to run this and what to expect?

@RickHawesUSDS - it would be good to get your feedback on this PR

I completed a screenshare code review with @td-usds and have reviewed the code. I like this approach to running through the Docker, and we still have some flexibility with running Gradle outside Docker.
My only concern is the IP <IP_ADDRESS> and if it's cross platform across Docker environments (Windows, Mac, Linux). If it it's good on the three major platforms, and there aren't any typical reasons why a user would have a different default, then I'm good with it.
I'm approving, but this is something we should get a few more eyes on for an approval before we merge.

I understand the goal, but I think this will break locally running of the code.  Let's talk about this implementation before we merge it in.

@carlosfelix2 as long as you have docker installed, this should not make a difference and running locally should continue to work the way it did before.
<IP_ADDRESS> is the IP address associated with the Docker Bridge network Gateway. It is in the private IP range <IP_ADDRESS>/12 and created by docker itself. Any container is automatically connected to the bridge network as well which means that any container can talk to it (unless explicitly NOT connected to it, but by default, you are connected to it). This IP address is also accessible from your local machine because that's what docker makes available locally.
n: docker0: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc noqueue state DOWN group default
    link/ether <snip> brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/16 brd <IP_ADDRESS> scope global docker0
       valid_lft forever preferred_lft forever
    inet6 <snip>/64 scope link
       valid_lft forever preferred_lft forever

$ docker network inspect bridge
[
    {
        "Name": "bridge",
        "Id": "<snip>",
        "Created": "<snip>",
        "Scope": "local",
        "Driver": "bridge",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/16",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        // snip
    }
]

In this case, we're using it to reach the docker host machine (i.e. your 'real' machine) on which all other development environment containers are exposing their own ports as well.
Are there particular cases that you are concerned about that would not run locally anymore? Let me know and then I can put mitigations/fixes in for those.

@carlosfelix2 may be on to something, I think we are hitting this issue: https://docs.docker.com/docker-for-windows/networking/

I'm reverting this back to draft until we have a solution for non-Linux platforms as well

Thomas, let's have a presentation of this at a dev sync session before you merge it in, with a focus on why we're making this change, and what developers need to do differently than before. Thanks.

Yes, I've moved this PR back to draft because of the issue that @carlosfelix2 highlighted as well. Right now this works on Linux but not on Windows (and I haven't tried mac yet) due to some networking issues related to the docker bridge. So I have some work to do on this.

I get this error on the 3rd step:
$ docker-compose --file docker-compose.yml up
Creating network "prime-router_prime-router_build" with the default driver
Creating volume "prime-router_vault" with default driver
Creating prime-router_sftp_1    ... done
Creating prime-router_redox_1   ... done
Creating prime-router_vault_1     ... done
Creating prime-router_azurite_1 ... done
Creating prime-router_prime_dev_1 ... error

ERROR: for prime-router_prime_dev_1  Cannot start service prime_dev: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "/prime-data-hub-router/start_func.sh": stat /prime-data-hub-router/start_func.sh: no such file or directory: unknown

ERROR: for prime_dev  Cannot start service prime_dev: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "/prime-data-hub-router/start_func.sh": stat /prime-data-hub-router/start_func.sh: no such file or directory: unknown
ERROR: Encountered errors while bringing up the project.

I think this is related to the networking issue as well. Just for my information, can you share your configuration with me (i.e. OS, environment, ...) where you ran this? To the best of my current understanding, I think this is related to trying to bring something up that couldn't build.

./build.sh seems to require a password? And if you ^C out of it, it seems to continue building anyway.

Correct, this is to be able to deal with having different UIDs within and outside of the docker container in which the build itself runs and make it as transparent as possible.
Basically, when you build in a clean repo and thus you don't have build/, .gradle/, nor docs/, then the build creates those for you as part of the process in the bind-mounted directory prime-router. But this creation happens inside the container. Thus the owner of those directories is the user in which that process inside the container runs, which is root (within the container's namespace of course) and that user has UID 0. Thus anything created by that user has its owner set to 0 as well. These directories (and their content is created inside the container but really live outside of the container and are bind-mounted in (or in reverse: it's a local directory mounted inside your container). The result of this is that you now have local directories inside your repo (created by the build) that has the user with UID 0 as their owner (which is root - and not/should not be the user under which you typically operate) and thus your 'outside-of-container-user' would not have permissions to those directories outside of the container. If you then run a gradle build outside of the container (which you still can), you'd get weird permission errors because the build cannot write or even access to those directories anymore.
To accommodate for this, there's a function that chown's those directories back to the 'outside-of-container-you'. That is where the password prompt is coming from. Ctrl+C just prevents that chowning from happening and building can happily continue within the container because there the process is running as 'inside-the-container-root' anyway. It's there as a safeguard and convenience for those who wish to build in a 'heterogeneous' way (i.e. both inside and outside of the container):
function ensure_build_dir() {
  mkdir -p "${HERE?}/build"
  chmod 777 "${HERE?}/build"
  echo "Making sure you own 'build/', 'docs/' and '.gradle/'... (may require elevation)"
  sudo chown -R "$(id --user --name):$(id --group --name)" \
    build \
    docs \
    .gradle
}

Just ran this and it looks the script itself succeeded, but still got some test failures:
Test passed: for ignore.CSV action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.CSV action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.CSV action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_NULL action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_NULL action batch:  Expecting 5 item lineage records and got 5
*** TEST FAILED*** for ignore.OTC_PROCTORED_YYY action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.OTC_PROCTORED_NYY action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.OTC_PROCTORED_NUNKUNK action receive:  Expecting 5 item lineage records but got 0
*** Tests FAILED:  end2end ***

Just ran this and it looks the script itself succeeded, but still got some test failures:
Test passed: for ignore.CSV action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.CSV action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.CSV action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7 action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_BATCH action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action batch:  Expecting 5 item lineage records and got 5
Test passed: for ignore.REDOX action send:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_NULL action receive:  Expecting 5 item lineage records and got 5
Test passed: for ignore.HL7_NULL action batch:  Expecting 5 item lineage records and got 5
*** TEST FAILED*** for ignore.OTC_PROCTORED_YYY action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.OTC_PROCTORED_NYY action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.OTC_PROCTORED_NUNKUNK action receive:  Expecting 5 item lineage records but got 0
*** Tests FAILED:  end2end ***

That is due to #1583

@td-usds, this needed review and tested by someone doing active development and QA.  Our ReportStream tests are not working now.
Our ./quick-test.sh script now asks for a password (???)
Also my local ./prime test smoktests are failing:
*** TEST FAILED*** for ignore.CSV action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.CSV action batch:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.CSV action send:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7 action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7 action batch:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7 action send:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7_BATCH action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7_BATCH action batch:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7_BATCH action send:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.REDOX action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.REDOX action batch:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.REDOX action send:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7_NULL action receive:  Expecting 5 item lineage records but got 0
*** TEST FAILED*** for ignore.HL7_NULL action batch:  Expecting 5 item lineage records but got 0
I think you need to demo at a DevSync before merging.


Avoid ClassCastException when loading LogService from Scope

Impact

[x] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR fixes a bug where a ClassCastException occurs when alternative logger implementations are on the classpath when the CLI is used. This was previously fixed on the older legacy Main class but the newer LiquibaseCommandLine still has the issue.
Motivation is to avoid issues like this from being created.
Things to be aware of
Things to worry about
None that I know of. This change is backwards compatible.
Additional Context

Initial/Pre-Review Thoughts
Change makes sense to me, we don't want to be assuming a specific logger when the CLI code is running.
Questions I have:

None

Potential risks:

None

What could make the full review difficult:

Nothing


Phoebemirman/ch14891/adjust number of decimals shown for eth and

https://app.clubhouse.io/augur/story/14891/adjust-number-of-decimals-shown-for-eth-and-rep-in-the-portfolio-reporting-claim-section

Coverage remained the same at 66.654% when pulling 8399fa60669311a8f0ffbd986d8e4983aba6a073 on phoebemirman/ch14891/adjust-number-of-decimals-shown-for-eth-and into bec405865129998e7ffcd172b8d8e1b2988e7d78 on master.

Coverage remained the same at 66.654% when pulling 8399fa60669311a8f0ffbd986d8e4983aba6a073 on phoebemirman/ch14891/adjust-number-of-decimals-shown-for-eth-and into bec405865129998e7ffcd172b8d8e1b2988e7d78 on master.


Task parameter input fields not scrollable in macOS 12.4

Seems like a TextField related issue in macOS 12.4, works fine in macOS 13 though. Need a better solution for macOS 11 and above.

Fixed in https://github.com/v2ex/launcher/pull/33


pods are not being scheduled on open shift clusters for the aks periscope namespace

Describe the bug
When we try to deploy the periscope daemonset on to the open shift cluster, you end up with this error: Error creating: pods "aks-periscope-" is forbidden: unable to validate against any security context constraint: [provider restricted: .spec.securityContext.hostPID: Invalid value: true: Host PID is not allowed to be used spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed spec.containers[0].securityContext.hostPID: Invalid value: true: Host PID is not allowed to be used]
To Reproduce

Spin up an open shift cluster: https://docs.microsoft.com/en-us/azure/openshift/howto-create-private-cluster-4x#:~:text=Create an Azure Red Hat OpenShift 4 private,the OpenShift CLI. ... 6 Next steps.
Then follow the instructions in the periscope appendix and try to deploy the daemon set onto the openshift cluster.
Wait 3 minutes and to view the error, run kubectl get events -n aks-periscope. You can also run kubectl get daemonsets -n aks-periscope and you will see that the daemonset will not be in ready state.

Expected behavior
Pods should be in running state after deploying the periscope yaml onto the open shift cluster.
Desktop:

OS: WSL or unix 20.04

📓 Thanks for opening this @sophsoph321 , periscope currently support AKS cluster only and not support other kind clusters.
Was it working before? Thanks 🙏

Hi Sophie, thanks for testing this.
It looks like ARO (Azure Redhat Openshift) clusters have different permissions by default (more restrictive), from your post there are 3 different errors reported for default periscope config:

Privileged containers are not allowed
Host PID is not allowed to be used
hostPath volumes are not allowed to be used

This matches what I would expect based on what is reported at the following link under the section: "Restricted SCC: The Most Secure Standard Choice": https://www.openshift.com/blog/managing-sccs-in-openshift
_When a pod is created without explicitly using the PodSecurityContext field or the SecurityContext field under the container specifications it will use the Restricted SCC by default. _
I believe we will need to configure a Security Context Constraint (SCC) to deploy to ARO. The instructions at that link above show ways of creating it via the "oc" command line tool, assume there are equivalent definitions for baking it into yaml.
Feel free to ping me on teams if this doesn't make sense or you get stuck, but I imagine there should be documented ways of permitting (1) (2) and (3) using an SCC in the ARO docs. Or if you prefer we could try to reach out to someone in the ARO team directly?

Oh to get started - maybe its as simple as: oc apply -f aks-periscope.yaml --as=Privileged

📓 Thanks for opening this @sophsoph321 , please note periscope currently support AKS cluster only and not support other kind clusters.
Was it working before? Thanks 🙏
update:
Idea / thought share: this is a good issue where if this tool decide supporting the wider kind cluster we should then include or atleast shope for open shift kind cluster.

@Tatsinnit, totally understand that periscope supports AKS cluster only. However, I discussed internally with the arc team and we need periscope to work for the MVP of our troubleshooting tool. The reason is that OpenShift is one of the distros most used amongst our customers and that's the distro on which more things are likely to go wrong. Thus, making our troubleshooting tool more necessary there.
Anyways, I was able to get the periscope pods to schedule.

Oh to get started - maybe its as simple as: oc apply -f aks-periscope.yaml --as=Privileged

Thank you @davidkydd.  I was able to get the pods to schedule by running oc adm policy add-scc-to-user privileged system:serviceaccount::, just FYI. Will close this issue for now and will communicate over mail/Teams if there's any code changes needed for periscope to work with OpenShift.

That's great Sophie! Thanks for confirming the exact steps too.
Worth mentioning that I am very grateful for the excellent R&D work you have been doing to test and develop Periscope across new distros and platforms: you have contributed greatly to improving the capabilities and coverage of the tool and have been a joy to collaborate with 😃 💯 🥇

📓 Thanks for opening this @sophsoph321 , please note periscope currently support AKS cluster only and not support other kind clusters.
Was it working before? Thanks 🙏
update:
Idea / thought share: this is a good issue where if this tool decide supporting the wider kind cluster we should then include or atleast shope for open shift kind cluster.

@Tatsinnit, totally understand that periscope supports AKS cluster only. However, I discussed internally with the arc team and we need periscope to work for the MVP of our troubleshooting tool. The reason is that OpenShift is one of the distros most used amongst our customers and that's the distro on which more things are likely to go wrong. Thus, making our troubleshooting tool more necessary there.
Anyways, I was able to get the periscope pods to schedule.

Thanks Sophie, yeah sounds like a plan, and you could either add PR or if you have any thoughts feel free to add workitem for the project created here https://github.com/Azure/aks-periscope/projects/2 , thank you 🙏


[DOCFIX] Clean up Eng Doc - remove glusterfs

What changes are proposed in this pull request?
Removing document for GlusterFS
Why are the changes needed?
As part of effort of Eng Doc clean up, we have decided to remove the document of experimental / deprecated features to avoid confusion.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

Eng document

@Xenorith for review. thanks!

build failure shows a link to one of the deleted docs:
2022/09/28 22:50:00 Errors found in documentation markdown
  /tmp/docsCheck2308189996/docs/cn/overview/Getting-Started.md:
    306: relative link pointed to unknown file /en/ufs/GlusterFS.html
Error: Process completed with exit code 1.

i also found other files mentioning glusterfs; see the matches when doing a search for Gluster or gluster

cn/Overview.md
cn/ufs/Ufs-Extensions.md
cn/contributor/Contributor-Getting-Started.md
cn/overview/FAQ.md
cn/overview/Getting-Started.md
en/Overview.md
en/Ufs-Extensions.md
en/contributor/Contributor-Getting-Started.md

alluxio-bot, merge this please


AppVeyor builds fail for 4.0.5

The AppVeyor builds are failing, by default.
It seems like it is an authentication issue, which is out of my control.
https://help.appveyor.com/discussions/problems/11287-the-build-phase-is-set-to-msbuild-mode-default-but-no-visual-studio-project-or-solution-files-were-found
Nothing changed in the commits that should cause this failure, you can check the history here.
https://ci.appveyor.com/project/mapbox/node-sqlite3/history
This means that there are no windows builds for the 4.0.5 release.
@springmeyer @mapsam

I've unpublished 4.0.5 in the mean time.


Add function getUserInputSteps

Expose function getUserInputSteps so we could get only user input steps.
In my case, I need to use step from aggregateDataByDate and minus it with user input steps.

Any estimates on the npm release for this merge ?
Is there an existing flag to ignore the user entered steps fetched from getDailyStepCountSamples() ?

Any estimates on the npm release for this merge ?

@Mudit18 oh dear. Sure I'll do this right now. I thought it's already there. Sorry


not issue, just question

Firstly I want to say your module is fantastic and works perfectly, I have a few questions if that's ok?
currently I am making one initial session with .create_scraper(sess=sess) then using .cfscrape.get(someurl).content and saving the cookie to a file from .cookies using pickle.dump(requests_cookiejar, file) so every request/session I make from any other .py file I can just do requests.get(url, cookies=load_cookies(cookiefile)).content which just loads the cookie from the file using pickle and this works very well and doesn't require the 5 second delay on any requests made from any of my .py file's because it was done initially from the original .create_scraper
However.. I have two problems and hoping make you could shed some light on this?

how would I turn the cookie that's returned to a readable HTTP cookie like this example: #HttpOnly_.somesite    TRUE    /   FALSE<PHONE_NUMBER>  __cfduid    de8ab1c4e3e11c676b4edfdbae7dc1c245374942278
how would I do all this but setting a user_agent so I can use that in all my requests, because right now it's just using python-requests/version which works with that cookie I initially generated.. but I would like to make it something different.

I know this isn't an issue but I hope you wouldn't mind me asking this here?
Once again, fantastic job on the module it works absolutely wonderful! and thank you for your time reading this.

You can save the contents of get_cookie_string(). Otherwise, just write some Python to convert the CookieJar object to whatever format you need.

When calling get_cookie_string(), you can pass a user_agent argument. Otherwise, create_scraper() creates a Requests session, so you'll want to consult the Requests documentation for setting a session's user-agent string.


Update for Datasette 0.14a0

Refs:

https://github.com/simonw/datasette/issues/2381

I'm going to switch this plugin to work directly against the new tables using these new methods: https://docs.datasette.io/en/1.0a14/internals.html#getting-and-setting-metadata

Starting Datasette like this ensures even logged-out users can edit metadata
datasette --internal internal.db -s permissions.datasette-metadata-editable-edit true

This plugin won't need its own datasette_metadata_editable_entries table any more - it can use the Datasette core tables instead.
But... I might add an audit log table of some sort in order to let it record WHO edited the metadata and when.

I should probably add a migration which looks for an existing datasette_metadata_editable_entries table and copies values from it into the new metadata_* tables.

I landed the PR.


Enable building for Raspberry Pi

@y0sher I don't mind changing the name of the make command, but I think that this is the main usecase which I thought makes it clearer.
Now that I think about it again, maybe you're right, I'll change it.


Release v0.7.1 breaks LOVE key events

Hi, just pulled the latest release and this is now not working for some reason:
local Slab = require 'Slab'
local SlabTest = require 'SlabTest'

function love.load(args)
	love.graphics.setBackgroundColor(0.4, 0.88, 1.0)
	Slab.Initialize(args)
end

function love.update(dt)
	Slab.Update(dt)
	SlabTest.Begin()
end

function love.draw()
	Slab.Draw()
end

function love.keypressed(key)
	print(key) --DOES NOTHING
end

function love.mousepressed(mx, my, mb)
	print(mx, my, mb) --STILL PRINTS
end

Hi @flamendless, thanks for reporting this. Unfortunately, there were some callback forwarding missed with the Keyboard refactor. I plan on pushing out a fix soon, but wanted to give a little more time to see if other issues arise. In the meantime, this can be fixed with the following lines of code modified in Keyboard.lua:
local function OnKeyPressed(Key, Scancode, IsRepeat)
	PushEvent(Common.Event.Pressed, Key, Scancode, IsRepeat)

	if KeyPressedFn ~= nil then
		KeyPressedFn(Key, Scancode, IsRepeat)
	end
end

local function OnKeyReleased(Key, Scancode)
	PushEvent(Common.Event.Released, Key, Scancode, false)

	if KeyReleasedFn ~= nil then
		KeyReleasedFn(Key, Scancode)
	end
end

Apologies for this inconvenience. Hopefully this fix will unblock you from your work.

No worries. I'll wait for the fix


Make SignContent struct or Signable trait public

Description of feature:
We would like to use the SignWithLabel and VerifyWithLabel functionality from Section 5.1.2 with our data.
Implementation discussion (Optional)
I did this internally with this simple code:
#[derive(Clone, MlsSize, MlsEncode)]
struct SignContent<'a> {
    #[mls_codec(with = "mls_rs_codec::byte_vec")]
    label: Vec<u8>,
    #[mls_codec(with = "mls_rs_codec::byte_vec")]
    content: &'a [u8],
}

impl fmt::Debug for SignContent<'_> {
    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
        f.debug_struct("SignContent")
            .field("label", &mls_rs_core::debug::pretty_bytes(&self.label))
            .field("content", &mls_rs_core::debug::pretty_bytes(self.content))
            .finish()
    }
}

impl SignContent<'_> {
    pub fn new<'a>(label: &'a str, content: &'a [u8]) -> SignContent<'a> {
        SignContent { label: [b"MLS 1.0 ", label.as_bytes()].concat(), content }
    }
}

This is a trimmed down version of the functionality used by the Signable trait.
Do you think we could make either the full trait public, or make a SignContent struct available?
Thanks for considering!

I made SignContent borrow the label since I found that I only need the struct to stay around for a split second in my use case: I create it and then immediately use it with a sign or a verify method.

Maybe expose let s = sign_with_label(label, bytes) and verify_with_label(label, bytes, s) would make a better API? Signable is a bit cumbersome and exposing SignContent means the application has to deal with signatures.
(The reason content is owned in SignContent is that Signable::signable_content typically runs mls_encode which creates a vec, an you can't return a reference to locally created variable which would be dropped.)

Maybe expose let s = sign_with_label(label, bytes) and verify_with_label(label, bytes, s) would make a better API?

Yes, that would be a nice and very usable API for me!

Signable is a bit cumbersome and exposing SignContent means the application has to deal with signatures.
(The reason content is owned in SignContent is that Signable::signable_content typically runs mls_encode which creates a vec, an you can't return a reference to locally created variable which would be dropped.)

I saw the Signable trait and I must confess that I'm not sure why this functionality needs so much machinery :smile: But if we can provide two simple functions for others, then I'm more than happy.
I'll put up a PR for this next.

@mulmarta, would you be up for having a new method on CipherSuiteProvider, with a default implementation:
    /// Sign `data` using `secret_key` and `label`.
    ///
    /// This implements [SignWithLabel][1]. The `label` can be used to
    /// distinguish between signatures on the same `data` made in
    /// different contexts.
    ///
    /// [1]: https://www.rfc-editor.org/rfc/rfc9420.html#name-signing
    async fn sign_with_label(
        &self,
        secret_key: &SignatureSecretKey,
        label: &str,
        data: &[u8],
    ) -> Result<Vec<u8>, Self::Error> {
        let sign_content = SignContent::new(label, data).mls_encode_to_vec()?;
        self.sign(&secret_key, &sign_content)
    }

The whole signer module could then be moved to mls-rs-core if we go this way.
Alternatively, I can expose a new top-level function in signer:
    /// Sign `data` using `secret_key` and `label`.
    ///
    /// This implements [SignWithLabel][1]. The `label` can be used to
    /// distinguish between signatures on the same `data` made in
    /// different contexts.
    ///
    /// [1]: https://www.rfc-editor.org/rfc/rfc9420.html#name-signing
    async fn sign_with_label<P: CipherSuiteProvider>(
        signature_provider: &P,
        secret_key: &SignatureSecretKey,
        label: &str,
        data: &[u8],
    ) -> Result<Vec<u8>, MlsError> {

Which approach do you like better? I think the second means moving less code around, so it might fit better with the current layering of mls-rs vs mls-rs-core.


Search files to include field cannot contain square brackets

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.75.0 (Universal)
OS Version: Darwin arm64 22.2.0

Steps to Reproduce:

Create a file whose filename is [username].tsx.
add text test into it.
search test and set the files to include to [username].tsx.

Here are 2 screenshots.
Without files to include.

With files to include.

/duplicate
See https://github.com/microsoft/vscode/issues/168715#issuecomment-1345409463 for the syntax you need to use.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


[Bug]: Git Auto commit issues on Git lab and bit bucket

Is there an existing issue for this?

[X] I have searched the existing issues

Description
issues observed in Git lab and bit bucket

The auto commit bar is too fast for a medium sized app
The uncommitted changes is displayed in the app when there are any edits done in the original app and committed it
The Git lock error is displayed when the app is imported
The protected branch is not retained and the auto commit runs on that branch

Steps To Reproduce

go to https://ee-4305.dp.appsmith.com/
import a app from git lab
the app should have a protected branch the default branch other than main and some newly added commits
When the app is imported
The user doesn't see auto commit bar
The protected branch no more a protected branch
Git file lock error is displayed on app import
Some uncommitted changed related to query is displayed

Note : Import the below json app connect it to git and try the above steps or use this git hub link for importing git@gitlab.com:raksha7/all-templates.git
Untitled application 1 (22).json
Public Sample App
No response
Environment
Deploy Preview
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
cloud-dp

Not seen any of these issues on release. Hence closing this out.


publicPath don't work in my project

publicPath don't work in my project, and no matter how I set up the publicPath, it always requests at the current origin

Closing due to inactivity.
If this bug still applies, please create a new issue. Make sure to select the form for bugs and include a minimal reproducible example. Thanks!


chore: update config files (.jshintrc, karma.conf.js, protractor.conf.js)

This brings the config files on par with the tutorial (see angular/angular-phonecat#326).
Related to #329.

Superceeded by #341.


Game crashed at start with new version of forgery

https://gist.github.com/Tw1L1ght12/93d26309bac3b951f3ae3107101fe8c8
All works fine in previous version

Duplicate of #566
should be fixed in 3.1.3


enrich document for azapi vscode extension

This PR adds some images and descriptions for the azapi vscode extension

@TomArcherMsft
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @ms-henglu - internal contributors should submit PRs to the internal, private azure-dev-docs-pr repo. Could you please close this PR and recreate it in the internal repo? Thanks!

Got it! I'll close it


flannel-v0.6.1 install error

I0920 17:34:26.813001 07716 main.go:126] Installing signal handlers
I0920 17:34:26.813431 07716 manager.go:133] Determining IP address of default interface
E0920 17:34:26.813737 07716 main.go:146] Failed to create NetworkManager: failed to get default interface: Unable to find default route

@gl328518397 Thanks for the report. Have you tried this with a previous flannel release? Can you confirm that it was working before?

Closing as there's no repro.

i am sorry ,i forget reply.  i have fix it ,thank you .

@gl328518397 How did you fix it? Was the fix something that other people would benefit from?

HI @gl328518397 ,I met the same issue  , what was your solution?

hello ,  -iface="eth0"  just try。


Merge mesh before decimation

The decimation algorithm here is quadritic error. The vertices near each other will be merged even if there's a gap between. Decimation should be done after meshes are merged so that the final vertex number will be less.

Cannot find a proper way to do decimation after merge. Closing this issue now.


Can't edit template

Rancher Version: master 11/01
Steps to Reproduce:

Create a cattle environment
Edit the environment and change something
Click on save

Results: Stuck in saving, and won't finish.  Error in console - Uncaught TypeError: Cannot read property 'match' of undefined(…)
Expected:

Version - master 11/06


Does not conform to the syntax reference

At the beginning of the syntax reference, there are a number of examples of how precedence works in djot. https://htmlpreview.github.io/?https://github.com/jgm/djot/blob/master/doc/syntax.html#precedence. This is how the parser parses these examples currently
Here are some examples that don't parse correctly.

(no smart punctuation)

(First quotation mark parsed, but not the second)

(Quotation marks are not escaped)

Causes errors to be produced in the syntax tree
> This is a block quote.
>
> 1. with a
> 2. list in it.

That's all the examples I could find. All this being said though, this is some incredible work that you've done here! Having an incremental parser available for this relatively obscure language has been so useful to me, and I'd like to thank you for making this! :smile:

Excellent, thank you. I'll take a look when I have time left over.

Found a few things:

The examples for quotation marks have a very easy fix

diff --git a/grammar.js b/grammar.js
index b16ef99..0465167 100644
--- a/grammar.js
+++ b/grammar.js
@@ -573,7 +573,7 @@ module.exports = grammar({

     _smart_punctuation: ($) =>
       choice($.quotation_marks, $.ellipsis, $.em_dash, $.en_dash),
-    quotation_marks: (_) => token(choice('{"', '"}', "{'", "'}", '\\"', "\\'")),
+    quotation_marks: (_) => token(choice('{"', '"}', "{'", "'}", '"', "'")),
     ellipsis: (_) => "...",
     em_dash: (_) => "---",
     en_dash: (_) => "--",

The block quote example does not produce errors in the syntax tree, it's just when there's not a final newline at the end of the document that it doesn't like that. I also found this to be true with footnotes.

I've broken out problems into standalone issues, see: #49, #50, #51, #52.
As for the quotation marks I don't think it's that simple as we don't want to mark ' in it's for example, and we also need to have matching pairs I guess? It's more annoying to implement and I don't currently have a plan for how to do it correctly.
The emphasis/strong issues are... Hard I think. If we need to match the minimal substring then we may not be able to rely on the tree-sitter grammar specification as I'd hoped and we may need to handle it in the scanner, but I don't see how I'd implement that in a way that works with everything else...
That quotation marks aren't escaped in 5\'11\' and are parsed as quotation_marks is a conscious decision I made as Djot treats them as "straight quotes". I don't differ between the different types of quotes though, I don't know what's the best approach.
The list in blockquote errors is fixed in https://github.com/treeman/tree-sitter-djot/pull/48, although the second marker isn't properly identified (tracked in #47).


Convert fhir bundles from xml to fsh

I need to convert a bunch of XMLs to FSH for the implementation guide.
All our full medication examples are currently in xml, but to handle the version changes and some attribute or code system name changes, it would be much better to have them as fsh.
Advice/help appreciated.

I can help you with that. Can you please share some examples of what you would like to convert?

I think the idea is to document this so that Rutt as well as others can do
this.
Generally, the options are:
0. Ensure the XML is valid (if needed, using some XMLLint).

Convert XML to JSON (using the FHIR validator , needs to be documented
more )
java -jar validator_cli.jar "XXXX.xml" -version 4.3.0 -output "XXXX.json"
Convert JSON to FSH using GoFSH (need to document how, esp, the GoFSH
options)
Clean up FSH

Alternatively to 1, you can POST to a server, and get the JSON
On Fri, Nov 18, 2022 at 12:51 PM Francesco Galisi @.***>
wrote:

I can help you with that. Can you please share some examples of what you
would like to convert?
—
Reply to this email directly, view it on GitHub
https://github.com/hl7-eu/unicom-ig/issues/48#issuecomment-1319897942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3HUUAXWXRSBEG62ABQN4TWI5UULANCNFSM6AAAAAASEJFD64
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think the idea is to document this so that Rutt as well as others can do this.

"The idea"... Whose idea?
My idea was to post the problem and get someone do stuff for me.

I correct myself:
Let's take this opportunity not to give Rutt a FSH, but to give her the
tools so that she can FSH.
On Fri, Nov 18, 2022 at 1:07 PM Rutt Lindström @.***>
wrote:

I think the idea is to document this so that Rutt as well as others can do
this.
"The idea"... Whose idea?
My idea was to post the problem and get someone do stuff for me.
—
Reply to this email directly, view it on GitHub
https://github.com/hl7-eu/unicom-ig/issues/48#issuecomment-1319913222,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3HUUH7ZSM7FQ35PDFHKN3WI5WPNANCNFSM6AAAAAASEJFD64
.
You are receiving this because you commented.Message ID:
@.***>

@costateixeira my original intention was (and still is) to create a tool able to do that automatically

I have already done similar work with a small script that I can share. Uses the java validator as Jose mentioned

Hi @joofio, pleased to meet you. I think if you shared that script it could be helpful

I will change the code to suit this case (the original is the other way around) and post it ASAP

this snippet should be enough for this purpose with goFSH:
https://gist.github.com/joofio/8eeb8586005c4eed8082e630af02d559
for the opposite (fsh to XML) this code over here with sushi:
https://gist.github.com/joofio/c9b04b6e8dfa9a0a876526f724933cce

@joofio how do you think we can document these auxilliary tools? I think they are useful and should not be lost in a backlog.
@rlindstrm did this help? Can we close this issue if it works and is documented?

@costateixeira we can add a folder of something like "utils"? and add the scripts? properly documented of course, like the branch allocation discussed early today

@costateixeira we can add a folder of something like "utils"? and add the scripts? properly documented of course, like the branch allocation discussed early today

I agree, we should create a wiki in this repo to use as our knowledge-base.

add to https://github.com/hl7-eu/unicom-ig/tree/master/tooling
?
(Suggestion: when Rutt confirms this is useful/working, we can make a PR,
quoting this issue #48, and adding a readme in that folder for
documentation)
On Fri, Nov 25, 2022 at 5:55 PM João Almeida @.***>
wrote:

@costateixeira https://github.com/costateixeira we can add a folder of
something like "utils"? and add the scripts? properly documented of course,
like the branch allocation discussed early today
—
Reply to this email directly, view it on GitHub
https://github.com/hl7-eu/unicom-ig/issues/48#issuecomment-1327709016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3HUUANSXUJFGVIUGUB7F3WKDVRXANCNFSM6AAAAAASEJFD64
.
You are receiving this because you were mentioned.Message ID:
@.***>

Wiki is fine, linking to the different files in the repo. (I'd still have a
readme)
On Fri, Nov 25, 2022 at 5:57 PM Francesco Galisi @.***>
wrote:

@costateixeira https://github.com/costateixeira we can add a folder of
something like "utils"? and add the scripts? properly documented of course,
like the branch allocation discussed early today
I agree, we should create a wiki in this repo to use as our knowledge-base.
—
Reply to this email directly, view it on GitHub
https://github.com/hl7-eu/unicom-ig/issues/48#issuecomment-1327711757,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3HUUFO7LWDDPHBMRBLJ73WKDVWPANCNFSM6AAAAAASEJFD64
.
You are receiving this because you were mentioned.Message ID:
@.***>

not needed anymore. reopen if needed


fix(ui): client side error when passing labels as functions for custom view tabs

@paulpopus it seems that this is not working anymore in version v3.0.0-beta.129
Could you maybe have a look ?
Thank you in Advanced


Database type not created in default config

Currently the default config leaves the type: keyword out of the database section. This can cause confusion for those not using MySQL as there won't be a clear way to actually use Plsm with other database vendors

Merged to master


Maisy wanted her profile updated. again

👷 Deploy Preview for virtualrealitylab processing.

Name
Link

🔨 Latest commit
18c50a2a916ee737bee9e867b283a1e68c4278a0

🔍 Latest deploy log
https://app.netlify.com/sites/virtualrealitylab/deploys/675703981165fb00083a3406


Skip tests searching security signals

Skip the tests to improve CI reliability.

/azp run

/azp run


Make the temp directory configurable

Enable users to specify an alternate temp directory in the chocolatey configuration.
If  is not present, then $env:TEMP is used as a default.
This enables companies with IT policies that prevent executing files
in the %TEMP% directory to continue to use Chocolatey (see discussion at https://groups.google.com/forum/#!topic/chocolatey/V8aU7VnB5eM)
Sample Configuration File with value specified:
<?xml version="1.0"?>
<chocolatey>

    <tempDir>%ProgramData%\Chocolatey\Temp</tempDir>

    <useNuGetForSources>false</useNuGetForSources>
    <checksumFiles>true</checksumFiles>
    <virusCheck>false</virusCheck>
    <sources>
        <source id="chocolatey" value="https://chocolatey.org/api/v2/" />
    </sources>
</chocolatey>

This implements #307

@carolynvs I went with a bit of an easier fix for now. It helps with when package maintainers are also not using our defined functions and using $env:TEMP in their scripts.

I just pushed <IP_ADDRESS>-beta1 with the functionality in it.


Add adopter for kueue

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

/release-note-none

/ok-to-test

/lgtm
/approve
/label tide/merge-method-merge
/assign @alculquicondor

/cherry-pick website


The Connect4 game no longer works :(

Looks like you have a more polished version here: https://kevinshannon.dev/connect4
Error:
Uncaught ReferenceError: Peer is not defined
    at setUpOnline (connect4.js:1027)

Love this app, btw. Exactly what I was looking for. Having fun making some altercations.

Thanks! 😄
@kevin-shannon and I removed the api that Lansite uses to interface with connect4 in a redesign. I had a fork of the connect4 version at tannerkrewson/connect4lansite, but i think i deleted that. We've since implemented a new system (https://github.com/kevin-shannon/connect4/issues/25) for connect4 multiplayer that might work. It's been awhile anyways so I'll take a look.

fixed with 37745ca01a9070aebce2fd298c4f45d545e7cc75
lmk if it works :)

fixed with 37745ca
lmk if it works :)

wow, you are the best! works great! thanks for taking the time to dig into this!
On another note, been playing around with a few things:

auto assigning colors to users
adding steam profile pics and timestamps to messages
adding option to allow all users to be admin by default
allowing the Match Finder, Connect4 player, and an added Introductory TextBox to be pinned to the top by default
adding another info section for the Discord link and WiFi info. If you're interested I can look into creating a pull request for one or more of these features.

Wow those are amazing ideas. i would be honored to merge them into the main repo! 😄


Submit Workflow: Go to Here

Preflight

[X] The Alfred team has invited me to submit this workflow.

The submitted workflow:

[X] Was authored by me. ⓘ
[X] Uses Workflow Configuration or has no user-facing settings. ⓘ
[X] Does not have an auto-updater. ⓘ
[X] Does not contain unsigned and non-notarised binaries. ⓘ
[X] Does not download extra software. ⓘ
[X] Uses keywords with more than one character. ⓘ
[X] Has a descriptive About section. ⓘ
[X] Has an icon of at least 256x256 px. ⓘ

Homepage
https://github.com/paulrudy/alfred-go-to-here
Alfred Forum page
https://www.alfredforum.com/topic/6013-go-to-here
Gallery page content
Usage
Quickly switch the selected path in a MacOS Open/Save/Export/Move dialog based on an Alfred selection.
With an Open/Save/Export/Move dialog active, invoke the workflow with one of these methods for selecting a file or folder path:
Search for a file or folder and apply to the dialog via the Universal Action.

Alternatively, get paths for open Finder windows and tabs with keyword gtf, or select a file or folder in Alfred with keyword gth. ↩ to apply to the dialog.

Advanced users can also use the External Trigger from another workflow (not shown, see below).
The workflow looks for English-language button titles in the dialog. Non-English users can change these strings in the Workflow Configuration, along with adjusting the speed of execution in case of slower machines.
Credits
Thanks for help and suggestions: ahmadt, sepulchra, vitor, jwrc, tombenz
Icon credit: Stash Icons. MIT License
Tags
universal-actions, file-management, automation
Homebrew dependencies
No response
Bundled libraries
No response

Forgot to close this, but already live.


[opencv4:x86-windows] build failure

Host Environment

OS: Windows_zhCN
Compiler: MSVC 2017

To Reproduce
Steps to reproduce the behavior:
./vcpkg.exe install opencv4[ade,contrib,core,dnn,eigen,ffmpeg,gdcm,halide,ipp,jasper,jpeg,nonfree,openexr,opengl,openmp,ovis,png,sfm,tbb,tiff,vtk,webp,world,qt]
Failure logs
The following packages will be built and installed:
    opencv4[ade,contrib,core,dnn,eigen,ffmpeg,gdcm,halide,ipp,jasper,jpeg,nonfree,openexr,opengl,openmp,ovis,png,qt,sfm,tbb,tiff,vtk,webp,world]:x86-windows
Starting package 1/1: opencv4:x86-windows
Building package opencv4[ade,contrib,core,dnn,eigen,ffmpeg,gdcm,halide,ipp,jasper,jpeg,nonfree,openexr,opengl,openmp,ovis,png,qt,sfm,tbb,tiff,vtk,webp,world]:x86-windows...
-- Using cached C:/vcpkg/downloads/opencv-opencv-4.1.1.tar.gz
-- Using source at C:/vcpkg/buildtrees/opencv4/src/4.1.1-d4651a36c2
-- Using cached C:/vcpkg/downloads/opencv-opencv_contrib-4.1.1.tar.gz
-- Using source at C:/vcpkg/buildtrees/opencv4/src/4.1.1-521dd00bd2
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/0ea90e7a8f3f7876d450e4149c97c74f-boostdesc_bgm.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/232c966b13651bd0e46a1497b0852191-boostdesc_bgm_bi.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/324426a24fa56ad9c5b8e3e0b3e5303e-boostdesc_bgm_hd.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/202e1b3e9fec871b04da31f7f016679f-boostdesc_binboost_064.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/98ea99d399965c03d555cef3ea502a0b-boostdesc_binboost_128.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/e6dcfa9f647779eb1ce446a8d759b6ea-boostdesc_binboost_256.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/boostdesc/0ae0675534aa318d9668f2a179c2a052-boostdesc_lbgm.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/e8d0dcd54d1bcfdc29203d011a797179-vgg_generated_48.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/7126a5d9a8884ebca5aea5d63d677225-vgg_generated_64.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/7cd47228edec52b6d82f46511af325c5-vgg_generated_80.i
-- Using cached C:/vcpkg/downloads/opencv-cache/xfeatures2d/vgg/151805e03568c9f490a5e3a872777b75-vgg_generated_120.i
-- Using cached C:/vcpkg/downloads/opencv-cache/data/7505c44ca4eb54b4ab1e4777cb96ac05-face_landmark_model.dat
-- Using cached C:/vcpkg/downloads/opencv-cache/tiny_dnn/adb1c512e09ca2c7a6faef36f9c53e59-v1.0.0a3.tar.gz
-- Using cached C:/vcpkg/downloads/opencv-cache/ippicv/0157251a2eb9cd63a3ebc7eed0f3e59e-ippicv_2019_win_ia32_20180723_general.zip
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
CMake Error at scripts/cmake/vcpkg_fixup_cmake_targets.cmake:98 (message):
  'C:/vcpkg/packages/opencv4_x86-windows/debug/share/opencv' does not exist.
Call Stack (most recent call first):
  ports/opencv4/portfile.cmake:318 (vcpkg_fixup_cmake_targets)
  scripts/ports.cmake:85 (include)

Error: Building package opencv4:x86-windows failed with: BUILD_FAILED

Additional context
Retry with clean directories doesn't work either.

@tan-wei, this issue should fix by vcpkg latest commit 9e68729b, could you get latest vcpkg source and execute bootstap-vcpkg.bat, and build opencv4 again? I verified, everything build successfully for me with latest commit.

@PhoebeHui OK, I'll try it now. If fixed I'll close the issue.

@PhoebeHui Oops, something else happens, maybe related with the fearture qt:
install-x86-windows-dbg-out.log
Actully, I've met this issue several days before, so I update vcpkg and rebuild qt5. I can confirm that without feature qt, it will success. However, this time the same issue occurs.
I can see lots of link errors with qt in the log file. So what can I do to fix this? Thanks!

Qt is problematic at the moment, there are many PR trying to fix it. Look for #7667
I hope that the qt feature will be fixed when that will land

@cenit OK, maybe I should wait for some days after qt is update to 5.12.4

hmm i see that link line:
-LIBPATH:C:\vcpkg\installed\x86-windows\lib opengl32.lib glu32.lib 3rdparty\ippicv\ippicv_win\icv\lib\ia32\ippicvmt.lib C:\vcpkg\installed\x86-windows\debug\lib\Halide.lib C:\vcpkg\installed\x86-windows\debug\lib\aded.lib C:\vcpkg\installed\x86-windows\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpdecoderd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpdemuxd.lib C:\vcpkg\installed\x86-windows\debug\lib\libwebpmuxd.lib C:\vcpkg\installed\x86-windows\debug\lib\libpng16d.lib C:\vcpkg\installed\x86-windows\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmMSFF.lib C:\vcpkg\installed\x86-windows\debug\lib\tiffd.lib C:\vcpkg\installed\x86-windows\debug\lib\lzmad.lib C:\vcpkg\installed\x86-windows\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\debug\lib\jasperd.lib C:\vcpkg\installed\x86-windows\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\debug\lib\Half-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\Iex-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\Imath-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\IlmImf-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\IlmThread-2_3_d.lib comctl32.lib gdi32.lib ole32.lib setupapi.lib ws2_32.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5Testd.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5Concurrentd.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5OpenGLd.lib C:\vcpkg\installed\x86-windows\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\debug\lib\Halide.lib C:\vcpkg\installed\x86-windows\lib\hdf5.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\glew32d.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkInteractionStyle-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersExtraction-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersStatistics-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkImagingFourier-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingLOD-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersModeling-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOPLY-8.2.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\double-conversion.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\lz4.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\lz4d.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\lzma.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\zlib.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersTexture-8.2.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\freetype.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\freetyped.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOExport-8.2.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\jpeg.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\libpng16.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\libpng16d.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\tiff.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\tiffd.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOXML-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOXMLParser-8.2.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\expat.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingContext2D-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingGL2PSOpenGL2-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkgl2ps-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOGeometry-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOLegacy-8.2.lib 3rdparty\lib\quircd.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreBites_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreHLMS_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreMeshLodGenerator_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreOverlay_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreProperty_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreRTShaderSystem_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreTerrain_d.lib C:\vcpkg\installed\x86-windows\debug\lib\OgreVolume_d.lib C:\vcpkg\installed\x86-windows\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpdecoderd.lib C:\vcpkg\installed\x86-windows\debug\lib\webpdemuxd.lib C:\vcpkg\installed\x86-windows\debug\lib\libwebpmuxd.lib C:\vcpkg\installed\x86-windows\debug\lib\tiffd.lib C:\vcpkg\installed\x86-windows\debug\lib\lzmad.lib C:\vcpkg\installed\x86-windows\debug\lib\jasperd.lib C:\vcpkg\installed\x86-windows\debug\lib\Half-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\Iex-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\Imath-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\IlmImf-2_3_d.lib C:\vcpkg\installed\x86-windows\debug\lib\IlmThread-2_3_d.lib gdi32.lib ole32.lib setupapi.lib C:\vcpkg\installed\x86-windows\lib\hdf5.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\glew32d.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\double-conversion.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\lz4.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\lz4d.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\lzma.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\zlib.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\zlibd.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\freetype.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\freetyped.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\jpeg.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\jpegd.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\libpng16.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\libpng16d.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\tiff.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\debug\lib\tiffd.lib C:\vcpkg\installed\x86-windows\share\vtk\..\..\lib\expat.lib C:\vcpkg\installed\x86-windows\debug\lib\tbb_debug.lib C:\vcpkg\installed\x86-windows\debug\lib\libprotobufd.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmDICT.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmIOD.lib C:\vcpkg\installed\x86-windows\lib\expat.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmDSED.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmCommon.lib crypt32.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmjpeg8.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmjpeg12.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmjpeg16.lib C:\vcpkg\installed\x86-windows\debug\lib\openjp2.lib C:\vcpkg\installed\x86-windows\debug\lib\gdcmcharls.lib rpcrt4.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5Widgetsd.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5Guid.lib C:\vcpkg\installed\x86-windows\debug\lib\Qt5Cored.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkImagingCore-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingFreeType-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\freetyped.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingOpenGL2-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkRenderingCore-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonColor-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersGeometry-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersSources-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersGeneral-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonComputationalGeometry-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkFiltersCore-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\libpng16d.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOImage-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkDICOMParser-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkmetaio-8.2.lib comctl32.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkIOCore-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonExecutionModel-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonDataModel-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonTransforms-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonMisc-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonMath-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonSystem-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtkCommonCore-8.2.lib C:\vcpkg\installed\x86-windows\debug\lib\vtksys-8.2.lib Psapi.lib C:\vcpkg\installed\x86-windows\debug\lib\avformat.lib C:\vcpkg\installed\x86-windows\debug\lib\avdevice.lib C:\vcpkg\installed\x86-windows\debug\lib\avcodec.lib C:\vcpkg\installed\x86-windows\debug\lib\avutil.lib C:\vcpkg\installed\x86-windows\debug\lib\swscale.lib C:\vcpkg\installed\x86-windows\debug\lib\avfilter.lib C:\vcpkg\installed\x86-windows\debug\lib\swresample.lib C:\vcpkg\installed\x86-windows\debug\lib\zlibd.lib ws2_32.lib wsock32.lib Secur32.lib bcrypt.lib C:\vcpkg\installed\x86-windows\debug\lib\OgrePaging_d.lib C:\vcpkg\installed\x86-windows\debug\lib\manual-link\OgreMain_d.lib kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib
and see that  VTK will link a lot of libraries wrongly in a debug build( just search for share\vtk\..\..\lib in that line)

Vtk and qt were not deeply tested features mostly because those port themselves had many problems. I fixed so many others along the road, that those were just feeling like black holes with no come back. How is your work going on with qt? I hope to be able to have qt feature working for free after you finished that. For vtk, the problem is that its dependencies are not correctly stored and for any downstream project it will forward links to both debug and release libs......

@cenit Sorry for the delay. I will try it again today.

my experience with qt is very low. I can't answer your questions as of now (but i have to learn it soon to be able to manage some projects with it). I would be glad if you could look at it in the meantime.

@tan-wei:
Just add -DCMAKE_AUTOMOC=ON to the  vcpkg_configure_cmake call in the portfile of opencv4. Should solve your problem. To be save you could also add  -DCMAKE_AUTORCC=ON or even -DCMAKE_AUTOUIC=ON depending upon if the opencv4 source tree has .qrc or .uic files.
Installing package opencv4[ade,contrib,core,dnn,eigen,ffmpeg,gdcm,halide,ipp,jasper,jpeg,nonfree,openexr,opengl,openmp,png,qt,sfm,tbb,tiff,vtk,world]:x86-windows...
Installing package opencv4[ade,contrib,core,dnn,eigen,ffmpeg,gdcm,halide,ipp,jasper,jpeg,nonfree,openexr,opengl,openmp,png,qt,sfm,tbb,tiff,vtk,world]:x86-windows... done
Elapsed time for package opencv4:x86-windows: 24.61 min

Total elapsed time: 24.61 min

The package opencv4:x86-windows provides CMake targets:

    find_package(OpenCV CONFIG REQUIRED)
    # Note: 4 target(s) were omitted.
    target_link_libraries(main PRIVATE numeric multiview opencv_sfm opencv_world)

(i removed sdl1 ports since i cannot currently build those due to a windows SDK bug)
I let @cenit figure out when he has to add those flags to the port ;)

Wonderful. Thanks

@Neumann-A Thanks a lot!

@Neumann-A I confirm that it sucess after adding the -DCMAKE_AUTOMOC=ON. The issue should be closed after the flag is added when feature qt is enabled.
By the way, for SDL1, there's a tiny solution described in #7535. The issue is related to the bug of the SDK indeed.

Thanks @Neumann-A  @cenit


Having issues with converting PerSave settings to PerCampaign

Hi, I tried converting my PerSave settings to PerCampaing, but I just can't get them to show up in game  when I do.
This is my current implementation of PerSaveSettings: BannerlordCheatsPerSaveSettings.cs
Is there something I must do in addition to deriving from AttributePerCampaignSettings<T>?
Cheers!

Should be fixed with v5.5.3!

Thanks! The cheats show up now, unfortunately they seem to not be initialized correctly when you start a new campaign, only after saving once and loading back in the game.

Roger, will investigate

What's the issue exactly with the initialization?
Also, I see there's some logic in the constructor. Because the Settings are used as a model by MCM, the class is instantiated multiple times, which might be a slowdown

Sorry, my explanation was bad. When you start a new campaign and immediately go into the mod options after character creation, the per campaign settings do not show up. The JSON file for the per campaign settings is also not created. Only by saving, exiting to menu and loading the save again does the file get created and the per campaign settings show up in the mod options.

Found the issue! I was getting the UniqueId of the Campaign on game start, not when it was loaded!

Should be fixed with v5.5.4

You're the best! I will check in the next days and close the issue.

Works now, thanks!


"Connection terminated unexpectedly"

I'm getting the above exception intermittently from a long-running script (via setInterval()), usually within a couple of minutes and sometimes immediately.
My Kysely setup seems pretty vanilla:
const dialect = new PostgresDialect({
  pool: new Pool({
    database: process.env.DATABASE_NAME,
    host: process.env.DATABASE_HOST,
    password: process.env.DATABASE_PASSWORD,
    user: process.env.DATABASE_USER,
    port: process.env.DATABASE_PORT ? parseInt(process.env.DATABASE_PORT) : 5432,
    })
 })

It works intermittently so there's no problem with the credentials.  The same config works without exceptions in a web server process as well.
Here's the stack trace I'm seeing:
{PATH}/node_modules/pg/lib/client.js:132
      const error = this._ending ? new Error('Connection terminated') : new Error('Connection terminated unexpectedly')
                                                                        ^
Error: Connection terminated unexpectedly
    at Connection.<anonymous> ({PATH}/node_modules/pg/lib/client.js:132:73)
    at Object.onceWrapper (node:events:628:28)
    at Connection.emit (node:events:514:28)
    at Connection.emit (node:domain:489:12)
    at Socket.<anonymous> ({PATH}/node_modules/pg/lib/connection.js:63:12)
    at Socket.emit (node:events:514:28)
    at Socket.emit (node:domain:489:12)
    at TCP.<anonymous> (node:net:337:12)
make: *** [Makefile:6: refresh-cache] Error 1

Thanks for any guidance!

Kysely delegates all communication to to underlying driver. The same thing would happen if you didn't use kysely at all but the pool instance directly.


sbt loads sbt-onejar but rejects one-jar command

After adding plugin to build.sbt, sbt loaded sbt-onejar but then complained "one-jar" is not a valid command???
Java HotSpot(TM) Client VM warning: ignoring option MaxPermSize=256m; support wa
s removed in 8.0
[info] Loading project definition from C:\scala\db\project
[info] Updating {file:/C:/scala/db/project/}db-build...
[info] Resolving org.scala-sbt.ivy#ivy;2.3.0-sbt-2cf13e211b2cb31f0d3b317289dca70
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.s
cala-sbt.plugins/sbt-onejar/scala_2.10/sbt_0.13/0.8/jars/sbt-onejar.jar ...
[info]  [SUCCESSFUL ] org.scala-sbt.plugins#sbt-onejar;0.8!sbt-onejar.jar (2216m
s)
[info] Done updating.
[info] Set current project to db (in build file:/C:/scala/db/)
> one-jar
[error] Not a valid command: one-jar
[error] Not a valid key: one-jar (similar: on-load, unmanaged-jars, export-jars)

[error] one-jar
[error]        ^

This should probably be closed, with apologies.  It was actually an sbt config issue caused by unwise action on my part.  I ended up going in a different direction so did not pursue it further, but subsequently discovered I had done something in the project that prevented sbt from properly finding plugin commands.

For those that are having the same problem a simple solution is to do the following
$ sbt

reload plugins
clean
reload return

This should reload the plugin and put the plugin tasks into scope proper. Worked for me at any rate.


Error 500 when searching for keyword

Internal server error is thrown everytime I try to search for keyword, e.g. curl cheat.sh/~redirect. Same thing happens on website and when using via curl.

Try https://cheat.sh/curl+redirect instead?

I was looking for keyword redirect specifically, i.e. results for either curl or redirecting standard error and output.

Then the URL that gives 500 error should be https://cheat.sh/~redirect because curl.sh/~redirect is a different site.

Sorry, I’ve switched curl with cheat. Updated original post.

yeah, I get a 500 error from most requests it can't quite parse it seems

Should be fixed now, please test.

@chubin it works now, but queries like /scala/~closure or /~snapshot/r still don’t work.

Please check it again

It works now, at least it doesn’t throw 500; for example /scala/~closure doesn’t return anything.

I suppose that it is correct in this case, but maybe we should return some "nothing found" message explicitly.
This query searches for closure in the scala cheat sheets, not on external resources, and so there are no closure occurrences there. Compare it with /scala/~loop or /scala/~function, for example

Yeah, that should probably make more sense. Closing this issue since original problem has been solved!


osm-init Back-off restarting failed container

modprobe: can't change directory to '/lib/modules': No such file or directory
iptables-restore v1.8.7 (legacy): iptables-restore: unable to initialize table 'nat'
Error occurred at line: 2
Try `iptables-restore -h' or 'iptables-restore --help' for more information.

Hi @shanghai636848 which env you are using and which version of OSM?
Also, can you give the steps you followed before encountering the error?


php 7.4 and fix   str_repeat(): Second argument has to be greater than or equal to 0

Thank you for your PR. However, the changes got merged via other PRs.


python3 int <-> num_bigint support

See also #124

I implemented this for grins a while ago: https://gitlab.com/Askaholic/pyrusty.
However, it was my experience that the rust extension module using BigInt was noticeably slower than the python big integer implementation. I was testing this using the baby step giant step algorithm for finding a discrete logarithm.

I implemented this for grins a while ago: https://gitlab.com/Askaholic/pyrusty.
However, it was my experience that the rust extension module using BigInt was noticeably slower than the python big integer implementation. I was testing this using the baby step giant step algorithm for finding a discrete logarithm.

I would guess python is faster because it optimises for single-word numbers, whereas BigInt doesn't really (it uses Vec internally instead of something like SmallVec)

See also: https://github.com/rust-num/num-bigint/issues/36


Add kolektor dataset

Description

Work in progress

Added the dataset classes, dataloaders and other necessary functions to load Kolektor Dataset

Due to the nature of the dataset, used unconventional methods which needs to be discussed

Requires ImageIO and Numpy libraries

Fixes #964

Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which refactors the code base)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the pre-commit style and check guidelines of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing tests pass locally with my changes
[x] I have added a summary of my changes to the CHANGELOG (not for minor changes, docs and tests).

Hi @ashwinvaidya17
Thank you very much for your feedback.
In the latest commits I resolved three of the issues you raised.
Regarding the prepare_dataset there's an issue. The download link for the dataset doesn't provide an extension for the downloaded file. Instead it sends the extension via the response header. This works well with a browser but the urlretrieve method used in the download_and_extract method downloads the file without extension and therefore cannot extract.
We have two options to resolve this

Instead of download_and_extract use a custom function to download and set the extension.
Update the download_and_extract function to receive the response header and set the extension. If we choose this we'll have to update other make_dataset methods accordingly.

Please let me know your thoughts on this.
Also as I have informed @samet-akcay , I had to pause the work on this until I get a green light on the current logical implementation ( As you might have seen, I had to classify images on the runtime instead of loading from directories because of the nature of the dataset). Please let me know whether the logic is good to go so that I can finish up.

Hey guys
I've gone ahead and completed writing docstrings and implemented a custom function to download and extract the dataset.
Since I've tested it locally I removed the WIP tag. Please have a look.
Could you refer me a documentation on the test convention used here. Once I get an insight on this I can finish it as well.
Thanks

Thank you very much @samet-akcay  for the review. I resolved one requested change and answered your questions as well. Please let me know your thoughts and any further changes.

Thanks for the review guys.
The Codacy Static Code Analysis fails because  the init function in the DataModule class has too many arguments. I see no way around this as other implementations have the same issue.
Not sure why the Tox checks are failing. When I ran all pre-merge checks locally they all passed. Any ide on this?

Thanks for the review guys. The Codacy Static Code Analysis fails because the init function in the DataModule class has too many arguments. I see no way around this as other implementations have the same issue. Not sure why the Tox checks are failing. When I ran all pre-merge checks locally they all passed. Any ide on this?

It's a problem with our CI that we need to fix. You don't need to do anything else at this point. Thank again for your contribution

Thanks for the review guys. The Codacy Static Code Analysis fails because the init function in the DataModule class has too many arguments. I see no way around this as other implementations have the same issue. Not sure why the Tox checks are failing. When I ran all pre-merge checks locally they all passed. Any ide on this?

It's a problem with our CI that we need to fix. You don't need to do anything else at this point. Thank again for your contribution

Noted with thanks. Hope to contribute more in the future.


boolean error

plugin version :- latest release
pmmp verion:- PocketMine-MP 1.6.2dev+poggit.1537
pmmp link:- https://poggit.pmmp.io/r/10779/PocketMine-MP.phar
Error:-
[13:27:19] [Server thread/CRITICAL]: Unhandled exception executing command 'rankup' in rankup: Return value of rankup\command\RankUpCommand::execute() must be of the type boolean, none returned
[13:27:19] [Server thread/CRITICAL]: TypeError: "Return value of rankup\command\RankUpCommand::execute() must be of the type boolean, none returned" (EXCEPTION) in "237-RankUp-1d3bf/src/rankup/command/RankUpCommand" at line 56

@Falkirks this bug has been removed in the latest commits.


config folder not exist after doing a git. So configuration never saved then setup on each launch

Description
Jarvis.sh doesn't save the configuration because he cannot find the folder config.
So on the first launch, Jarvis should create a folder config otherwise everytime we launch jarvis.sh we need to do the installation setup all the time.
Workaround until next update... mkdir config on jarvis folder

I'm so chocked I never noticed or heard about this before!! Will just create an empty file for now in the repository so that it comes with the clone.


Model "file type"?

Disclaimer: I'm not so much into ML models, but have a use case ;-)
So we will have several providers that aim to implement this extension for models they generate with specific software in their infrastructure (e.g. random forest). Other providers may then read these results. I've been told that these models depending on which software they may have been generated with, may generate different types of model files so that only some software may be able to read it and some others may not, for example:

Software A does RF and can read/write it in model type X
Software B does RF and can read/write it in model type X and Y
Software C does RF and can read/write it in model type Y

How can I know from the model metadata whether I can read the exposed model file with my software? @duckontheweb
Maybe this is easy to answer and may just be reading a different media type or so, but want to ensure this is considered. :-)
Related issue: https://github.com/Open-EO/openeo-processes/issues/300

Thoughts @duckontheweb ?

Sorry I missed this the first time around @m-mohr!
I think the easiest way to handle this is through some combination of media types and roles or relation types (depending on whether we are dealing with an Asset or a Link). In some ways, using media types would be preferable because it would work for both Assets and Links. However, it seems like most model artifacts do not have an official IANA media type, so we would have to define our own within the spec.
I recently added the "ml-model:checkpoint" role to handle the case of PyTorch checkpoint files as assets, but it seems like if we continue to take this approach we would need define a new role for each type of model file, which could be cumbersome.
I will put some more thought into this, but I'm curious if others have any insight into a better approach.

I just had the idea to use "processing:software" (on assets?) to specify the software writing it, but it could also be additional roles or media types. I guess we need to investigate this a bit more, we will probably also experiment with it in openEO Platform and see what works for us and propose that as a potential solution.


Adding new contributor

Comment:
Hi everyone,
could any of you add me as a contributor to this repository? I have just release a new version of napari-crop on pipy (0.1.8) and I would like to track the updates here :)
Best,
Marcelo

Closed via #6


Leaks needs to be solved before release

Summary:
Here are leak reports needs to be solved before release (means this week)
Steps to reproduce:
Play with app, in general.
Add System logs:

This issue can be solved by several people. Please use comments efficiently to assign it yourselves by using letters (A/B/C/D/E/F).
Besides, this issue needs to be done in a week so help is appreciated :)

I'll kick things off by taking up the the 'A' leak. I can try to fix all of them too if that's okay. 👍 @nicolas-raoul @neslihanturan

Firstly you can start with 'A', since this issue is urgent and requires parallel working. If you are done with 'A' don't hesitate to get another one, after you wrote comment here @yashshah7197

Thanks for volunteering to do "A", @yashshah7197 . :) If anyone else would like to take the others, please feel free to mention it here.

@neslihanturan can you tell me how I could reproduce these leaks? LeakCanary doesn't seem to be detecting anything for me yet.
On a side note, sorry for the late updates. I'm down with a bit of fever and cough so can't really work on anything properly right now. 🙈

@yashshah7197 just playing with app, I don't know either.

I am working on B and C

@yashshah7197 this issues needs to be done today, will you be able to make it up? If not I will give a try.

It seems like E and F is something we can ignore: https://medium.com/@amitshekhar/android-memory-leaks-inputmethodmanager-solved-a6f2fe1d1348

Closing, we can create a new issue if there are further leaks. :)


tests/e2e: add more buckets and reorganize test bucketing

Description:
Adds 2 more buckets to parallelize e2e tests and reorganizes
them roughly based on their areas.
Signed-off-by: Shashank Ram<EMAIL_ADDRESS>
Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [X]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

[bucketting intensifies]


leshow.humidifier.jsq1 (Xiaomi Mijia Pure Pro)

hello
no water level
no mode control
screenshot https://ibb.co/CJ5pn1z
firmware <IP_ADDRESS>2

Not sure what you mean. What is the bug here?

Not sure what you mean. What is the bug here?
Sorry, I spelled it wrong. the device has modes and water level in the mi home app, but when using this plugin, these functions are not present. For example, the water level always shows 0

Then you need to post a homebridge debug log so i can see what is happening. I will not be able to tell you what is wrong without a log.

Then you need to post a homebridge debug log so i can see what is happening. I will not be able to tell you what is wrong without a log.

[2/22/2022, 12:13:02 AM] [miot] [Увлажнитель спальня] (Protocol) Call <IP_ADDRESS>: get_properties - [{"did":"408075888","siid":8,"piid":13},{"did":"408075888","siid":8,"piid":14}] - {}
[2/22/2022, 12:13:03 AM] [miot] [Увлажнитель спальня] Error while parsing response from device for property humidifier:water-level. Response: {"did":"408075888","siid":2,"piid":7,"code":-1}
[2/22/2022, 12:13:03 AM] [miot] [Увлажнитель спальня] Error while parsing response from device for property device:turn-ovp. Response: {"did":"408075888","siid":8,"piid":11,"code":-1}
[2/22/2022, 12:13:03 AM] [miot] [Увлажнитель спальня] Error while parsing response from device for property device:ntc-over-push. Response: {"did":"408075888","siid":8,"piid":12,"code":-1}
[2/22/2022, 12:13:03 AM] [miot] [Увлажнитель спальня] Error while parsing response from device for property device:dry-turn. Response: {"did":"408075888","siid":8,"piid":13,"code":-1}
[2/22/2022, 12:13:03 AM] [miot] [Увлажнитель спальня] Error while parsing response from device for property device:overwetting. Response: {"did":"408075888","siid":8,"piid":14,"code":-1}
is this enough or do you need more?

[Увлажнитель спальня] Splitting properties into chunks. Number of chunks: 2. Chunk size: 14
[2/22/2022, 12:14:52 AM] [miot] [Увлажнитель спальня] (Protocol) Call <IP_ADDRESS>: get_properties - [{"did":"408075888","siid":2,"piid":1},{"did":"408075888","siid":2,"piid":2},{"did":"408075888","siid":2,"piid":3},{"did":"408075888","siid":2,"piid":6},{"did":"408075888","siid":2,"piid":7},{"did":"408075888","siid":3,"piid":1},{"did":"408075888","siid":5,"piid":1},{"did":"408075888","siid":8,"piid":1},{"did":"408075888","siid":8,"piid":2},{"did":"408075888","siid":8,"piid":3},{"did":"408075888","siid":8,"piid":9},{"did":"408075888","siid":8,"piid":10},{"did":"408075888","siid":8,"piid":11},{"did":"408075888","siid":8,"piid":12}] - {}

Your device does not want to report the data... not sure why. I would recommend that you try to reset your device and see if that helps.

unfortunately it didn't help

Yeah well, I cannot do anything from my side about this. The device refuses to report the values so not sure what else can be done here... You could try and set the forceMiCloud property to true and specify micloud credentials and check if maybe over the cloud the device reports the data correctly.

@merdok In addition to the water indicator. there is another problem in the mods, they are not

@merdok Yes, sure. No mods

I also have after applying all the tips - no mod management and no water level

As mentioned, the device beahves very strange and i am not sure why it is like that...

Same here. Water level is always 0%

Yeah, i mean you can post some homebridge debug log here but i guess also in your case the device will simply deliver wrong numbers...


Make objects invisible that are obviously outside of the plane

Problem
We want some objects to be shown only when they are on the associated panels in immersive mode. Otherwise we want them to be hidden. (If I understand correctly) we use masking and clipping systems to control them. But unfortunately those systems seems to be costly.
Suggestion
We add a new system that checks the positions of objects and associated panels and sets .visible to false of the entirely objects that are obviously outside of the panels. Perhaps this additional cost in CPU would be much cheaper compared to clipping or masking in GPU.

It may be easily doable with IntersectionObserver


Support for abbreviated ruby version in .ruby-version file

I think rbenv should process each .ruby-version through whatis command.
For many projects I really don't care which revision the ruby is, unless it's 1.9.3. Developers working on one project may have different revisions of ruby 1.9.3 installed, so currently adding .ruby-version to repository is not the best idea, as probably each developer is going to change it in order to not install slightly newer ruby version (like 1.9.3-p362 to 1.9.3-p374).
If rbenv supported abbreviated versions, I could add .ruby-version file set to 1.9.3 to repository and that way make sure other developers wont get problems when starting working on project.

Old issue, but I want to comment that this is a pain point in 2024.  I'm often packaging ruby code that uses rbenv for early/local dev, but actual deployments don't use rbenv.  Examples are deployments in Docker and AWS Lambda.  In these deployments it's generally undesirable to pin to patch, as that would inhibit the build process from automatically pulling in trivial and security updates, if the upstream source supports pinning at the patch level at all.  I just wrote some Terraform code like the following to deploy a lambda:
locals {
 # .ruby-version / rbenv requires major.minor.patch, and lambda just deploys major.minor
  lambda_ruby_version_major_minor = join(".", slice(split(".", file("${path.module}/lambda_source/.ruby-version"), 0, 2))
}

resource "aws_lambda_function" "lambda" {
  runtime = "ruby${local.lambda_ruby_version_major_minor}"
}

In this kind of use case rbenv isn't part of the version enforcement between dev and prod.  It would be nice if the major.minor.patch requirement could be re-evaluated and softened for use cases such as this one.


Update tracing-subscriber to v0.3.17

Simply bump tracing-subscriber to latest v0.3.17.
Fixes #534, resolved by tokio-rs/tracing#2566.
Not sure if any of the checkpoints apply for such a minor change?

I would prefer that you actually just do cargo update, and then verify that a sufficient version has been pinned in the lock file. Let's not update the Cargo.toml itself in this case.

Interesting, why is that exactly? I assume cargo update changes the locked version to the latest? But I just ran it as you said and amended my branch.

I forgot to push the changes beck then, but I rebased it on master and pushed it just now. Thought I'm not sure how relevant it is at this point.. @thedodd

I don't think that this PR is needed any longer.