Is this still alive?

Hey there. I've got a few google wifi points, plus one of the new nest APs. I'm interested in using GF for my devices to try and add more functionality, but looking at the discussions and comments, I don't see anything recent that confirms this still works.
Could you please confirm that this is still useable in 2021, or has some issue caused it to not work anymore?
Either way, your work is greatly appreciated. Hope to hear back.

It would appear that maybe it works judging from https://github.com/marcosscriven/galeforce/pull/47#issuecomment-704491729

It would appear that maybe it works judging from https://github.com/marcosscriven/galeforce/pull/47#issuecomment-704491729

News?

Any confirmation yet? I can see that the discontinued message has been taken down.

Maybe I'll bite the bullet and try this on one of my points...
On Thu, Sep 16, 2021, 10:09 AM Dean2706 @.***> wrote:

Any confirmation yet? I can see that the discontinued message has been
taken down.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marcosscriven/galeforce/issues/52#issuecomment-920987944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMO4NG6CRBV4ZRJRWOBLI3UCICBBANCNFSM4WQDVLOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I can see someone selling a single puck 2nd hand near me fairly cheap so I might give it a go this weekend

OK, so after playing around with this over the weekend and I can say that this project is defiantly a bust, but not completely without hope.
TL;DR: No.
It does work once you flash the image to a puck and you can SSH into the device. But as soon as you expose the puck to the internet an auto update kicks in and ruins the SSH access and I think even soft bricks the device. This happens when you plug an internet connection into the WAN port or if you try and adopt the device using Google Home. Luckily it only causes a soft brick and you can flash the recovery image to bring it back.
So if you are after a WIFI router without access to the internet, then technically this project works, but this is obviously not practical.
Looking into this further I can see that Galeforce essentially patches the latest OnHub recovery image that Google provides, which is 9334.41.3, this is not current and will be updated as soon as internet is given to the device. So unless someone knows where to download the most up to date version (13729.57.27 at the time of this post) of the ChromeOS image for Gale so we can patch that version and the device will not need to be updated or if anyone knows how to maintain root/SSH even after it receives an update.

OK, so after playing around with this over the weekend and I can say that this project is defiantly a bust, but not completely without hope.
TL;DR: No.
It does work once you flash the image to a puck and you can SSH into the device. But as soon as you expose the puck to the internet an auto update kicks in and ruins the SSH access and I think even soft bricks the device. This happens when you plug an internet connection into the WAN port or if you try and adopt the device using Google Home. Luckily it only causes a soft brick and you can flash the recovery image to bring it back. So if you are after a WIFI router without access to the internet, then technically this project works, but this is obviously not practical.
Looking into this further I can see that Galeforce essentially patches the latest OnHub recovery image that Google provides, which is 9334.41.3, this is not current and will be updated as soon as internet is given to the device. So unless someone knows where to download the most up to date version (13729.57.27 at the time of this post) of the ChromeOS image for Gale so we can patch that version and the device will not need to be updated or if anyone knows how to maintain root/SSH even after it receives an update.

Sorry, been busy as of late with another project.
BUT, I think I can help you help me help you? :P
Looks like the only way to create a recovery image for Google Wifi boxes is using the "Onhub Recovery Utility" extension for Chrome.
After some unsuccessful wiresharking, I had the bright idea to just download the extension source, which was much more productive.
Looks like, for wifi devices, the extension is downloading an "onhub_recovery.json" file from the following URL, which is likely to change in the future...but who knows?
https://dl.google.com/dl/edgedl/chromeos/recovery/onhub_recovery.json
Looking at the JSON, I see the URL for the "old" image is here:
https://dl.google.com/dl/edgedl/chromeos/recovery/chromeos_9334.41.3_gale_recovery_stable-channel_mp.bin.zip
I tried updating the URL to include the version number you specified, but I'm getting a file not found error. Gonna try playing with stuff a bit more, but I feel like this is at least a start into finding the latest image. ;)

OK, so after playing around with this over the weekend and I can say that this project is defiantly a bust, but not completely without hope.
TL;DR: No.
It does work once you flash the image to a puck and you can SSH into the device. But as soon as you expose the puck to the internet an auto update kicks in and ruins the SSH access and I think even soft bricks the device. This happens when you plug an internet connection into the WAN port or if you try and adopt the device using Google Home. Luckily it only causes a soft brick and you can flash the recovery image to bring it back. So if you are after a WIFI router without access to the internet, then technically this project works, but this is obviously not practical.
Looking into this further I can see that Galeforce essentially patches the latest OnHub recovery image that Google provides, which is 9334.41.3, this is not current and will be updated as soon as internet is given to the device. So unless someone knows where to download the most up to date version (13729.57.27 at the time of this post) of the ChromeOS image for Gale so we can patch that version and the device will not need to be updated or if anyone knows how to maintain root/SSH even after it receives an update.

I wonder if it'd be possible to SSH into the device pre-update, and then watch logs to determine the new update URL...

I went down the same track and tried to guess the download URL based on the onhub_recovery.json and on the ChromeOS recovery.conf file found here: https://dl.google.com/dl/edgedl/chromeos/recovery/recovery.conf
I think they deliberately do not advertise the latest image to stop us from messing with it, but I like your idea of watching some logs and try to capture the URL link of the update. I am not super familiar with ChromeOS so I am not sure where to begin to find these logs.

95% chance any relevant data will be dumped to some file in /var/log, most likely "/var/log/syslog". Looking at the scripts that galeforce runs for patching, it's just doing "echo" commands, which means that's where it should be going to.
You could probably start ssh and then tail that file to see what's happening as it updates...and if there are any errors or any other useful information in it.

https://www.googlenestcommunity.com/t5/Nest-Wifi/Several-Questions

The page you are trying to access was not found. Please check your URL for typos and try again.
:D

I swear, I tried the URL before I pasted it. This should work:
https://www.googlenestcommunity.com/t5/Nest-Wifi/Several-Questions/m-p/20693?search-action-id=2922573082&search-result-uid=20693

Google plans to abandon Onhubs, cut them off from cloud management access, in December.
https://support.google.com/googlenest/answer/11257354
Even if Google doesn't officially open up them up for tinkering, their active undoing of firmare mods should at least stop. And for a lot of people not being able to change a router's settings will make it garbage. There will soon be more of these routers out there for next to nothing, and more incentive for someone who wants to make their router work again to contribute to this project.

China can't access google's download server,
So this will be work in china,
or iptables -t nat -I POSTROUTING 1 -d "dl.google.com" -j DROP?

OpenWRT is suppoted on this device now: https://openwrt.org/toh/google/wifi

OpenWRT is suppoted on this device now: https://openwrt.org/toh/google/wifi

LOL, I definitely should have circled back to this issue before. This is the solution I went with, and it works pretty well, all things considered. I even wrote the "SlowLED" module for it so we can get some nice light animations. :D


Secret references test

Compile a list of all secrets references in any template, and verify that all of the referenced secret definitions are also present, per namespace. Test should fail if there is a missing secret.

hmm gonna have to think about this one a bit, requires some aggressive recursive looping.

hmm gonna have to think about this one a bit, requires some aggressive recursive looping.


Feature: Allow skipping downloads of engines by setting env vars

Closes #395
It adds support for operating systems that Prisma does not compile binaries for like NixOS by setting the needed env variables for the engines.
It should still support the previous way of working but it's worth noting that I haven't tested that yet.
I currently only have a NixOS machine so I haven't been able to test it on something like Darwin.

Works for me on NixOS. Looking forward to the merge :+1:

Idk why I called fetch_native before doing the env checks but thanks for fixing it 😂


Remove the margin-bottom of form to not expand the height of navbar

this doesn't merge - could you reopen with a merging request - thanks!


framework: acpilight should be used

Yesterday, I reported https://github.com/NixOS/nixpkgs/issues/171093 about how nixos-generate-config does not correctly detect the need for acpilight. I don't have the background or time to fix this in nixos-generate-config, but a friend who saw the bug suggested I could submit a workaround here. That seems worthwhile to me, as a way to capture the knowledge so nobody else has to rediscover it. I hope this PR is helpful!

Rebased in https://github.com/NixOS/nixos-hardware/pull/415

Merged in https://github.com/NixOS/nixos-hardware/pull/415 thanks!

My pleasure, glad to help!


Ancient-Greek-BERT POS tagger

It seems that they have removed the POS tagger from the HuggingFace repository, Flair told me it was unavailable, and I can't find it in their repo either.

Actually I'm just dumb

Problem is is that even though the model is included in their Github repo, it is invalid, and they have no records of which version of Python or Flair they used.

Okay, predictions are working now.

Flair uses this weird format: "v3ppia--"


failed to amplify env checkout prod with failure retrieving In-App messaging and Email Channel

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
20.10.0
Amplify CLI Version
12.10.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
when I initially did a amplify env checkout prod, I was prompted for a serverKey. By the time I checked for the serverKey, my Mac hung and I had to reboot it. Re-running amplify env checkout prod, this time all resources made it, the command was successfull, but I was not prompted for a serverKey. I looked at console, and did see that FCM channel was not configured in PinPoint, so I manually enabled FCM and assigned the serverKey from pinpoint console.
Then amplify env checkout dev, then came back to amplify env checkout prod and getting complain.
Describe the bug
when I initially did a amplify env checkout prod, I was prompted for a serverKey. By the time I checked for the serverKey, my Mac hung and I had to reboot it. Re-running amplify env checkout prod, this time all resources made it, the command was successfull, but I was not prompted for a serverKey. I looked at console, and did see that FCM channel was not configured in PinPoint, so I manually enabled FCM and assigned the serverKey from pinpoint console.
Then amplify env checkout dev, then came back to amplify env checkout prod and getting complain.
this is what I see when I do amplify env checkout dev:
✔ Initialized provider successfully.
✔ Successfully retrieved channel information for FCM.
✔ Channel information retrieved for In-App Messaging
Deployment state saved successfully.
✅ Initialized your environment successfully.
but when I do amplify env checkout prod:
✔ Initialized provider successfully.
✔ Successfully retrieved channel information for FCM.
⠋ Retrieving channel information for In-App Messaging.⠋ Retrieving channel information for Email.✖ There was an error initializing your environment.
🛑 Could not initialize categories for 'prod': Cannot read properties of undefined (reading 'Id')
Resolution: Review the error message and stack trace for additional information.
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
⠙ Retrieving channel information for Email.
Session Identifier: 403cf9dd-5f13-4002-a614-c1964291d918
✅ Report saved: /var/folders/qd/g556jpdn0r114qsj2prqqrqc0000gn/T/pyxiscontribute/report-1710525021753.zip
✔ Done
Expected behavior
should succeed amplify env checkout prod
Reproduction steps
I don't know, I am in this stuck state and don't know what caused it in the first place.
Project Identifier
c9dd6cbc5e466a59de093a48bd94c543
Log output

# Put your logs below this line
2024-03-15T15:02:50.384Z|error : Could not initialize categories for 'prod': Cannot read properties of undefined (reading 'Id')
ProjectInitFault: Could not initialize categories for 'prod': Cannot read properties of undefined (reading 'Id')

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @ericowhadi, thank you for reaching out. could you add --debug to the env checkout and provide us the output.
Could you try pulling the backend into a new empty directory outside your existing project directory, does this throw a similar error message and re-prompt for serverKey?

✔ Initialized provider successfully.
✔ Successfully retrieved channel information for FCM.
⠋ Retrieving channel information for In-App Messaging.⠋ Retrieving channel information for Email.✖ There was an error initializing your environment.
🛑 Could not initialize categories for 'prod': Cannot read properties of undefined (reading 'Id')
Resolution: Review the error message and stack trace for additional information.
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
ProjectInitFault: Could not initialize categories for 'prod': Cannot read properties of undefined (reading 'Id')
at initializeEnv (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/initialize-env.js:122:19)

at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

at async Object.run (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/commands/env/checkout.js:48:5)

at async Object.run (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/commands/env.js:52:13)

at async Object.executeAmplifyCommand (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/index.js:194:9)

at async executePluginModuleCommand (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:139:5)

at async executeCommand (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:37:9)

at async Object.run (/snapshot/amplify-cli/build/node_modules/@aws-amplify/cli-internal/lib/index.js:121:5)

Cannot read properties of undefined (reading 'Id')
TypeError: Cannot read properties of undefined (reading 'Id')
at getNotificationsAppListMeta (/snapshot/amplify-cli/build/node_modules/@aws-amplify/amplify-category-notifications/lib/notifications-amplify-meta-api.js:148:58)

at getNotificationsAppMeta (/snapshot/amplify-cli/build/node_modules/@aws-amplify/amplify-category-notifications/lib/notifications-amplify-meta-api.js:47:44)

at Object.pull (/snapshot/amplify-cli/build/node_modules/@aws-amplify/amplify-category-notifications/lib/channel-in-app-msg.js:104:98)

at /snapshot/amplify-cli/build/node_modules/@aws-amplify/amplify-category-notifications/lib/notifications-manager.js:112:51

at /snapshot/amplify-cli/build/node_modules/promise-sequential/index.js:16:18

at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

⠙ Retrieving channel information for Email.
Session Identifier: dc6bac4c-6b6e-47cf-bb64-97ce9da3a966
✅ Report saved: /var/folders/qd/g556jpdn0r114qsj2prqqrqc0000gn/T/pyxiscontribute/report-1710537220958.zip
✔ Done
Project Identifier: c9dd6cbc5e466a59de093a48bd94c543

"Could you try pulling the backend into a new empty directory outside your existing project directory, does this throw a similar error message and re-prompt for serverKey?"
I am not sure what I need to do? amplify pull? from dev env or prod env (from an empty directory located where (if I do it outside my project, how is amplify knows what project to pull from?)? since I cannot successfully amplify env checkout prod, not sure how I can amplify pull prod?
BTW thanks you very much for your very fast response, much appreciated.

hey, thank you for the information. we can pull the backend by running amplify pull --appId <app-id> --envName prod as I wanted to ensure if this was due to the failed prompt which may have not hydrated your files from the reboot that occurred on your PC.
additionally, in your team-provider-info.json do you observe analytics and notifications block with an id variable? if no could you try adding the block under the prod env as follows then run amplify env checkout prod to hydrate the meta file.
"analytics": {
        "nextapphost": {},
        "Pinpoint": {
          "Name": "xxxx",
          "Id": "xxxx",
          "Region": "us-east-1"
        }
      },
      "notifications": {
        "Pinpoint": {
          "Name": "xxxx",
          "Id": "xxxx",
          "Region": "us-east-1"
        }
      }

OK, so the amplify pull in an new directory fails with a similar error: Cannot read properties of undefined (reading "Id"). And it did not prompt me for a serverKey. (note that the serverKey was manually added to pinpoint, so I guess it is not surprising that it does not ask for it again?).
As for team-provider-info.json, it already had the correct entries for the prod section. Both prod and dev have correct entries, with no missing id.

@ericowhadi thank you for the information. To ensure could you check the amplify-meta.json present under the amplify/backend. the file should contain the analytics and notifications blocks with the structure similar to as follows
"analytics": {
    "nextapphost": {
      "providerPlugin": "awscloudformation",
      "service": "Pinpoint",
      "output": {
        "appName": "xxxx",
        "Region": "us-east-1",
        "Id": "xxxx"
      },
      "lastPushTimeStamp": "",
      "providerMetadata": {
        "s3TemplateURL": "xxxx",
        "logicalId": "xxxx"
      }
    }
  },
"notifications": {
    "nextapphost": {
      "Id": "xxxx",
      "ResourceName": "xxxx",
      "Name": "xxxx",
      "service": "Pinpoint",
      "output": {
        "Id": "xxxx",
        "Name": "xxxx",
        "Region": "us-east-1",
        "FCM": {
          "ApplicationId": "xxxx",
          "CreationDate": "2024-03-21T14:58:43.798Z",
          "DefaultAuthenticationMethod": "KEY",
          "Enabled": false,
          "HasCredential": true,
          "HasFcmServiceCredentials": false,
          "Id": "gcm",
          "IsArchived": false,
          "LastModifiedDate": "2024-03-21T14:59:20.885Z",
          "Platform": "GCM",
          "Version": 2
        },
        "InAppMessaging": {
          "Enabled": true,
          "ApplicationId": "xxxx",
          "Name": "xxxx"
        },
        "regulatedResourceName": "xxxx"
      },
      "lastPushTimeStamp": "2024-03-21T14:59:12.561Z"
    }

could you ensure if the id's are filled in and matched the pinpoint resource. if not run amplify env checkout prod
to hydrate the meta file. do note the meta file may be hidden in Vscode, under the .vscode/settings.json file.
additionally, could you try downloading the #current-cloud-backend.zip from the s3 deployment bucket amplify-<app-name>-<env-name>-<random-number>-deployment and check the amplify-meta.json.

@yketha after doing amplify env checkout prod (that fails obviously, since this is the bug I am having), the amplify-meta.json is indeed bogus for the notifications section but correct on analytics:
"notifications": {
"pyxiscontribute": {
"channels": [
"FCM",
"InAppMessaging"
],
"service": "Pinpoint"
}
}
while the dev environment have all data correctly populated for both analytics and pinpoint.
Reading your comment, I am under the impression that you suggest doing an amplify env checkout prod to hydrate the file and fix the problem. However, it does hydrate the file but with the problem in it.
as for the amplify-meta.json included in the #current-cloud-backend.zip inside the prod deployment bucket, it is also bogus and contain:
"notifications": {
"pyxiscontribute": {
"channels": [
"FCM",
"InAppMessaging"
],
"service": "Pinpoint"
}
}
instead of a fully populated notifications section.

@ericowhadi could you provide us the team-provider-info.json via email to<EMAIL_ADDRESS>

@ykethan just sent it.

@ericowhadi thank you for the information. Did not notice any inconsistencies with the team-provider-info.json.
could you try running amplify update notifications and reselecting FCM which should prompt you to enter the Server key, then run amplify push this should update the amplify-meta.json.

@ykethan I did amplify env checkout prod
then from this bogus state, I did amplify update notifications
But this triggered a similar error "Cannot read properties of undefined (reading 'Id')". So it did not prompt me for anything and just failed the command.

@ericowhadi interesting, could you try removing the notifications resource by amplify remove notification and then re-add it?
if the above does not mitigate the issue, would you be open for quick chat on a discord DM. my discord handle is ykethan

If I do amplify remove notification, I am not sure I understand if this will remove it from both dev and prod. If I first do a amplify env checkout prod, then amplify remove notification, it will not remove it from dev, or will it?

@ericowhadi could you zip and provide us the amplify folder at<EMAIL_ADDRESS>this would enable us to dive into the resources before making changes to backend?

@ericowhadi thank you for sending the files. I was able to recreate the issue by modifying the amplify-meta.json in my project.
To mitigate the issue:

download the #current-cloud-backend.zip from your project deployment bucket from AWS S3 console. The bucket name should have a pattern amplify-<project-name>-prod-<random-number>-deployment.
extract and open amplify-meta.json.
in file remove the notifications block and save.
re-zip the contents of current-cloud-backend folder. Note, when re-zipping do ensure you are not zipping it as nested folder but rather only the contents.
upload the zip to the S3 bucket.
run amplify env checkout prod, this should prompt you for the server key and re-generate the notifications block in amplify-meta.json

awesome, let me try

great, it worked. It did not ask me for server key, I guess because I already had manually populated it. It coorectly hydrated the notifications section, and successfully checkout prod. However when pushing, I got a graphql updating issue, but that is another story and I will open a new bug on it. Thanks a million for the great support @ykethan

for the story, I found a workaround to the graphql issue that was having hard time updating a table where I changed pk, and had an index on. The solution was to first delete the table, push, then add the modified table. Amplify did not like doing it all at once.
Closing this issue any, and thanks again @ykethan, it is such a great experience to be supported, even on a corner case issue that was caused by a crash of my PC in the middle of a amplify update notifications...


[CLI] add support for cluster management

What does this PR do?
This PR adds support to manage Bring-Your-Own-Credentials lightning compute clusters.
The initial implementation exposes cluster creation, cluster listing and cluster deletion.
The feature is currently only available as closed preview and requires allow listing by a Lightning AI employees.
I've added two end-to-end tests and a couple of unit tests to ensure the CLI works as expected.
The general flow is aligned with how the lighting CLI works today:
Create clusters
This command creates a BYOC compute cluster. The provider arguments are specific to the only cloud provider we support today, AWS.
lightning create cluster <cluster-name> <cloud-provider-parameters>

list clusters
This command does not take any arguments. It lists clusters available to the current user
lightning list clusters

Delete cluster
The command deletes a BYOC compute cluster.
lightning delete cluster <cluster-name>

Does your PR introduce any breaking changes? If yes, please list them.
No breaking changes
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

LGTM!

@Felonious-Spellfire, I see you have requested changes that seemed to be resolved; mind re-review?

Curious if the failing cloud tests are related... @manskx?

Seems the tests are failing because it couldn't access the secrets. @Borda

LGTM!


[F000457] Sen. Al Franken Failing

Relates to #496, tests seem to be failing after a possible redesign of Mr. Franken's website.

YAML file lost. I am working on it.


MetalHammer as new release source

I added metal-hammer.de as a new release source.
MetalHammer does some weird things regarding the artist names, capitalizes the album title completely and has sometimes a date like 2020. I tried to fix these as simple as possible
We still have to figure out how to get the covers. Or for this matter more information in general, because Metal Hammer only has a table with 3 columns: Artist, Album title and (estimated) release date.
But I think just to have one more source this is a good start.

 [Butler] Add Metal Hammer as further release source


script/vagrant: cp the build target to usr bin

Before this change, when I run sudo rkt trust it fails. I hope
we can also install rkt under usr/local/bin by default. So our
users can have a better experience when following our rkt basic
tutorial.
This issue was introduced by a recent script rewrote (#652)

lgtm, thanks!

Should rkt be in /usr/bin or /usr/sbin? Either way this needed to be fixed. I've been running sudo su - after vagrant ssh in order to maintain the user's environment, but execute rkt as root.

@jzelinskie maybe, I don't feel strongly about this because it's an Ubuntu image. If you want to change it, feel free ;D


Add chomp specs for IO#gets, IO#readline, IO#each_line, IO#readlines …

…and IO.foreach
Addresses part of #473.

Ok I will change this as soon as I get home from work. Sorry about that.

@eregon Made the requested changes. Let me know if there's anything else that needs done.

It's all good, thank you for the new specs!


Yikes! Ask timed out on [ActorSelection[Anchor(akka://kafka-manager-system/), Path(/user/kafka-manager)]] after [5000 ms]

Yikes! Ask timed out on [ActorSelection[Anchor(akka://kafka-manager-system/), Path(/user/kafka-manager)]] after [5000 ms]
Getting this issue very often. Almost tried all the answers here.
Every time this error comes, have to restart the server. Any permanent fix for this ?

so am i

I have upped my api timeout to 10 seconds, and checked 'Enable Active OffsetCache'. Neither of these helped. I am running version <IP_ADDRESS> for Kafka & Zookeeper.
My current setup includes a cluser of 3 Zookeepers along with 3 Kafka brokers. We have about 8000 topics.

so am i.
I try to change the config:
kafka-manager.api-timeout-millis from default(5000) to 20000.
But it does not effect.
Yikes! Ask timed out on [ActorSelection[Anchor(akka://kafka-manager-system/), Path(/user/kafka-manager)]] after [20000 ms]

I'm having the same issues with a lightly-used, pre-prod cluster. I recently updated to the latest yahoo-manager release (<IP_ADDRESS>) in the hopes that it would fix this issue. It did not.
kafka version: <IP_ADDRESS>
Topics: 19
Brokers: 10
Partitions: 546
ZK nodes: 3 (v.3.4.5)

you should edit the configuration in “conf/application.conf” as follows:
kafka-manager.zkhosts="my.zookeeper.host.com:2181,other.zookeeper.host.com:2181"
Then restart kafka-manager.
see README.md

Disabling Poll consumer information (Not recommended for large # of consumers) fixes it for me

Fixed as suggested by @tastewind

kafka cluster or manager is broken, just restart would fix it(at least for me)

Cluster Zookeeper Hosts is required ,you can set value like this : "zookeeper:2181"  ,  "zookeeper" is the container name or service name  in docker-compose.yml ;  good luck


with scope-function's side effect tracking.

Expected Rule behavior

when using with scope - side effect that targeting the context should be called (suggested) in apply block.// wrong example
with(CarObj) {
   color = getNewColor(..)
}

// correct example
with(CarObj) {
   val newColor = getNewColor(..)

   apply {
     color = newColor
   }

}

apply block inside the with scope should be only 1 (suggested not more than 1).
when using with scope - side effect that targeting something outside the function should be called in also scope.
var currentCarColor = SOME_COLOR

with(CarObj) {
   val newColor = getNewColor(..)

   apply {
     color = newColor
   }

   also {
     currentCarColor = newColor
   }
}

also scope inside with scope should be only 1 (suggested not more than 1).

Those rule, improve the understanding of what side effects is happening in the code for new developer. Also forcing the developer to pay attentions to the side effects.
Since i can't find any source that suggesting this method, i cannot give any references for this. Since this feature is not fundamental of how the language is working, i don't recommend this to become a basic feature (activated by default), but as an optional feature. And i believe people gonna love this, espescialy I am. Big thanks.

i think this feature is too much 😅

Tnx for opening and closing this issue. This would indeed be to far out of scope for ktlint.


Correct rolling (nightly) CI build

The rolling CI build is failing due to changes made necessary in recent PRs.
Correct the build definition so it will run green again. Do not switch it to YAML on this issue.

PR-Validation CI build was converted to YAML, which was then immediately used during a PR to alter how we build/package/test the extension.
Unfortunately the rolling-ci build was not updated to YAML at the same time, and was not updated to reflect the needs of the new feature.

Fixed the issue in Azure devops tasks for now. YAML conversion is tracked on a separate issue.


chore!: Update aes-gcm to 0.10.3 and aes to 0.8.4

Description
This PR updates aes-gcm and is related to #260
Notes & open questions

Change checklist

[ ] Self-review
[ ] Documentation updates if relevant
[ ] Tests if relevant

The clippy errors are orthogonal to this PR -- have fixed them in: https://github.com/sigp/discv5/pull/262


Support for OCaml 4.03

Seems like the ppx needs to be fixed to work on 4.03.

Now working with 4.03.


(suggestion) Convert sdl.Rect Fields from int32 to int

Would you mind converting the sdl.Rect field types from int32 to int?
This would help code readability when defining new rectangles.

Hi @Priff13, it was done to keep compatibility with SDL2’s native C integers. Would you happen to have a suggestion on how to make the fields int and keep them compatible?

Thanks @Priff13. I think I will have to postpone it due to the amount of effort that it would require for me to maintain by myself but if you would like to create a pull request, it would be greatly appreciated!

Hi,
According to W3schools, on 64-bit architectures, golang's int is 64 bits wide, whereas, int32 is always 32 bits.
If the Golang SDL API accepts int (in sdl.Rect, sdl.Point, and others), what will the API do when it receives a value for an int that is too large for the int32 type required by the underlying SDL implementation?
Coercing the value from int to int32 will change the actual value and this is going to create subtle bugs that will be hard to debug.
Having int32 at the API level makes callers of this SDL wrapper library clearly aware of the size constraints of the underlying SDL library implemented in C, and it forces game devs to think rigorously about the types and values in their game. It prevents bugs.
Using int32 will also consume less memory on 64 bit architectures.
IMHO I feel like the benefits of compatibility (keeping int32 instead of int) outweigh the benefits of readability.
Best regards,
Jose


Feature Request: Protect Individual Account from deletion

Feature Request: Protect Individual Account from deletion.  Many catastrophes have occurred due to this user mistake.

backwards data model support - this is a wontfix from a priority perspective


dr_alg is set to None in attack_wrapper function.

Does this mean we're using a sub-optimal attack for all defenses?
Need to make sure we're calling the gradient_transform function properly.

this is an old question, I don't think I was understanding the code properly. This was probably the first stage where it was running the undefended model.


The provided master VM size 'Standard_D4s_v3' is invalid

[Enter feedback here]
VM size 'Standard_D4s_v3' is supported for master node, but I failed to create the azure openshift cluster with the command:
az aro create
--resource-group $RESOURCEGROUP
--name $CLUSTER
--vnet zhongtao-openshift-vnet
--master-subnet master-subnet
--worker-subnet worker-subnet
--master-vm-size Standard_D4s_v3
--worker-vm-size Standard_D4s_v3
--worker-count 3
--pull-secret @pull-secret.txt
with error:
The provided master VM size 'Standard_D4s_v3' is invalid
any idea?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cae8cd22-9a8c-b92e-6352-ba3d20fcf413
Version Independent ID: 6b0b03ce-5f8f-9ab7-2870-926a99da554f
Content: Supported resources for Azure Red Hat OpenShift 3.11
Content Source: articles/openshift/supported-resources.md
Service: container-service
GitHub Login: @jimzim
Microsoft Alias: jzim

@spartacusX Thanks for posting your question.
Can you please try that in a different region and check once, just wanted to rule out any capacity issues that might have caused this failure when you tried the deployment.

@spartacusX Any update on the issue?
Can you also share the region detail where it failed.

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion. :)

I am running into the same issue as well.  The provided master VM size 'Standard_D4s_v3' is invalid  for a ARO 4.x deployment

Tried in Both Eastus2 and westus2 location. same thing

Hi, we're having the same issue when using az aro. Trying to create D4s_v3 masters but failing with the same error. Is this issue already known? @VikasPullagura-MSFT

@jimzim can you please check and add your comments on this.

D8sv3 is the minimum supported deployment for aro for the masters. Docs are updated a month ago to reflect this

Link to the documentation on supported master sizes: https://docs.microsoft.com/en-us/azure/openshift/support-policies-v4#master-nodes


prevent layout from closing when input submitted

I'm trying to use the input box to send commands, which will send the inputs to an external program updating other popups

but when I submit, all of the boxes in the layout just close.
I can call a function in on submit to re-open the layout, but it causes the screen to flicker as it closes and re-opens which is a little annoying. Do you have advice on the best way to just have the input box clear the input text without closing when the on_submit function runs?

You can just do something like this:
  local prompt = "> "

  local input = Input({
    relative = "cursor",
    position = { row = 1, col = 0 },
    size = { width = 20 },
    border = "single",
  }, {
    prompt = prompt,
  })

  input:mount()

  vim.keymap.set({ "n", "i" }, "<Enter>", function()
    local value = string.sub(vim.api.nvim_get_current_line(), vim.fn.strwidth(prompt) + 1)
    print(value)
  end, { buffer = input.bufnr })


Installation

I have a vps running debian 7.I just installed lamp env and nothong configured.
Just put files in the /var/www/ folder, I navigate to xxx.com/install,nothing shows.
If navigate to xx.com, it shows the xxx.com/#/login page ,I can view the login page.
But I didn't finish the installation,ofcource I can't.
what happend?PHP scripts permisson or something else?

If you follow these instructions which are shown in this video everything works well.

If you follow these instructions which are shown in this video everything works well.

can you upload the gist.github.com file in the video?
I found why it shows nothing when navi to /install/
the apache error log shows
[Fri Jan 02 14:01:16 2015] [error] [client ] PHP Warning:  require(vendor/autoload.php): failed to open stream: No such file or directory in /var/www/c..com/public_html/install/index.php on line 3
[Fri Jan 02 14:01:16 2015] [error] [client ] PHP Fatal error:  require(): Failed opening required 'vendor/autoload.php' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/c..com/public_html/install/index.php on line 3

If you have installed CacoCloud using the archive file: http://cacodaimon.github.io/CacoCloud/download/CacoCloud.tar.gz the autoloader should be available, but if you have cloned the source via git you have to run the composer be executing the build.sh script.

Which HTTP Server does your webhost use?
If they use an apache, is the mod_rewrite enabled?

It's Apache (a LAMP stack), and mod_rewrite does seem to be enabled. To test, I added the code in post 9 of this thread: https://wordpress.org/support/topic/how-do-i-find-out-if-i-have-the-mod_rewrite-apache-module to my domain's .htaccess. The test succeeded in redirecting to www.google.com.

Its redirecting to the wrong location, the installer is located here CacoCloud/ but it redirects to /, for fixing this you have to modify the RewriteBase in the .htaccess located here: https://github.com/Cacodaimon/CacoCloud/blob/master/public/install/.htaccess and here: https://github.com/Cacodaimon/CacoCloud/blob/master/public/api/1/.htaccess .

The file /CacoCloud/public/install/.htaccess now reads:
RewriteEngine On
RewriteBase /CacoCloud/public/install/
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule ^ index.php [QSA,L]
The file /CacoCloud/public/api/1/.htaccess now reads:
RewriteEngine On
RewriteBase /CacoCloud/public/api/1/
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule ^ index.php [QSA,L]
But when I go to https://mydomain.com/CacoCloud/public/install/index.php, it still redirects to https://mydomain.com/install/database and fails.

The database is now created, but I have not been able to create the user account. It redirects to https://mydomain.com/install/user

I have fixed the installer, please download this version: https://github.com/Cacodaimon/CacoCloud/releases/tag/0.1.0 and fix your both .htaccess files after updating.

Almost done.
An error occured when browser https://IP/install/
Apache error log:
[Tue Feb 10 09:02:10 2015] [error] [client *] PHP Warning:  require(vendor/autoload.php): failed to open stream: No such file or directory in /var/www/CacoCloud/public/install/index.php on line 3
[Tue Feb 10 09:02:10 2015] [error] [client *] PHP Fatal error:  require(): Failed opening required 'vendor/autoload.php' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/CacoCloud/public/install/index.php on line 3
[Tue Feb 10 09:03:11 2015] [error] [client *] PHP Warning:  require(vendor/autoload.php): failed to open stream: No such file or directory in /var/www/CacoCloud/public/install/index.php on line 3
[Tue Feb 10 09:03:11 2015] [error] [client *] PHP Fatal error:  require(): Failed opening required 'vendor/autoload.php' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/CacoCloud/public/install/index.php on line 3

i am using the above version but i did not install the database

after i was deleted .htaccess file in then i get these following errors

PHP imap
Needed only by the mail client.
HTTPS
It is suggested to run CacoCloud only with HTTPS!

after i clicked the proceed with warning button i get empty page


[hud] Fix font rendering

Some characters (like i,l, |) should be close to the "middle", not to prev. character.

Bonus: Terminus (TTF)

The "Advance" value should be less than '20', but was set to 20 because of "font shadows".
0001-hud-Use-Terminus-TTF-font.patch.txt

This doesn't really look like a correct fix to me to be perfectly honest. Inverting x would mean that the horizontal origin is on the right side, which isn't the case.

Generated by font-texture-generator for Terminus (TTF): {'|', 36, 80, 6, 25, -5, 23}, , originX mostly have negative values for mentioned characters.

Hm, looks like you're right and it does indeed generate negative lower numbers for those, but this is counter-intuitive and goes against what e.g. Freetype does. I think we should fix the data instead.

There is the PR in original repo to " Export advance parameter in C metadata format", maybe this value should be used.  Fixed repo : https://kevinmoran.github.io/font-texture-generator/

Maybe next time I'll try to use different tool to generate glyphs and data. Currently I have no idea how to fix this properly. Inverting the sign of originX value, even if looks better, may still be wrong.


AZ 2019: Added new folks, retired old ones

Moving #38 forward. Lint summary:
==== az ====
Steve-Smith-915b21ca-5581-4bd4-9dec-be9d6f041676.yml
 1 active roles on retired person
Maria-Syms-88766a62-d2da-497f-b525-0e81f215599d.yml
 1 active roles on retired person
Olivia-Cajero-Bedford-0ecd1f4d-eac5-41fc-9e18-e6c3329333ef.yml
 1 active roles on retired person
Steve-Farley-1f65df83-bbe7-4d3e-a5d3-9f3684e5891c.yml
 1 active roles on retired person
Paul-Mosley-763925d8-df0e-4848-94fd-8a21a62fa07f.yml
 1 active roles on retired person
Ken-Clark-a4d3ecd4-8bfc-4e6c-8290-a557bc347680.yml
 1 active roles on retired person
Macario-Saldate-98f5d164-1419-4f8f-a0ff-70a976a59e12.yml
 1 active roles on retired person
Timothy-Dunn-0b8c9973-1ae5-4af3-8a73-713c8365eccd.yml
 1 active roles on retired person
Ray-D-Martinez-4d4c88ae-1e5c-424b-b674-f44f54e7676e.yml
 1 active roles on retired person
Reginald-Bolding-97d71f2f-9518-43fa-8076-5a78809563ff.yml
 1 active roles on retired person
Drew-John-aa72902f-0c4f-4f01-8d50-98ba33a8bb64.yml
 1 active roles on retired person
Brenda-Barton-df90b258-b808-46bc-bd70-0cc63a0bb619.yml
 1 active roles on retired person
Darin-Mitchell-a16f262c-5889-4341-80f4-2d8901ccc200.yml
 1 active roles on retired person
Katie-Hobbs-cd668109-5466-4c61-96cf-f5448ea118bd.yml
 1 active roles on retired person
Michelle-B-Ugenti-Rita-f496d728-bb97-40f0-a610-31b6a08f78cf.yml
 1 active roles on retired person
Todd-A-Clodfelter-e38a9853-4f63-43c2-8cb2-0075e25f4b54.yml
 1 active roles on retired person
Mark-A-Cardenas-8db9c77c-aeb5-4ec0-b64b-58503b0d56e6.yml
 1 active roles on retired person
Javan-D-Mesnard-06adeea1-c37c-4dd1-8be5-92e9b5d3556e.yml
 1 active roles on retired person
Judy-Burges-e467055b-a818-4f9a-b2ea-3c6502f4ba22.yml
 1 active roles on retired person
John-M-Allen-37ea1ccb-7cba-40ef-ae0b-4840f2f5a3ed.yml
 1 active roles on retired person
Daniel-Hernandez-bd6b8ff6-3e5d-47bf-be70-1a7df7f5caa6.yml
 1 active roles on retired person
Thomas-R-Shope-bde26535-68ee-4eef-bbc8-bfa8a4380015.yml
 1 active roles on retired person
Russell-Bowers-b9343c4d-9f9b-4bbd-ba02-06fbb1b09a22.yml
 1 active roles on retired person
Steve-Yarbrough-90bb7c86-23cd-465b-b88b-1f5e195be5f3.yml
 1 active roles on retired person
Eric-Descheenie-5626f4de-d194-4364-9f62-fac4f84ab046.yml
 1 active roles on retired person
Bob-Worsley-3b18b45f-6766-451e-91eb-9c0fb46e5e3e.yml
 1 active roles on retired person
Catherine-Miranda-cab32a4c-9f0f-445b-b8d2-a80a937ec87a.yml
 1 active roles on retired person
Douglas-Coleman-5559b51c-2244-4434-aa1f-71477c921d61.yml
 1 active roles on retired person
Kimberly-Yee-ff55f877-a410-4d9d-a22f-6b10ef911359.yml
 1 active roles on retired person
Wenona-Benally-370ec420-c6a3-4436-8b5d-fd43d73cb3d0.yml
 1 active roles on retired person
Jill-Norgaard-8eeecd80-362a-4ff5-b09d-159c47128f8f.yml
 1 active roles on retired person
Noel-W-Campbell-4624f496-3256-41d1-b13d-577f1aa0381f.yml
 1 active roles on retired person
processed 90 active people, 37 retired & 29 organizations
  60 lower
  30 upper
Parties
  48 Republican
  42 Democratic
Contact Info
  90 address
  89 voice
  90 email
Identifiers
  81 legacy_openstates
Additional Info
  90 sources
  58 family_name
  68 image
  58 given_name
  90 links
Extras - none
Committees
  11 upper
  18 lower
 179 member roles
  27 vice chair roles
  29 chair roles
   0 roles missing ID 0.0%
exiting with 32 errors

As far as I can tell, the 32 errors are the 32 retired reps who have end dates of 2019-01-14 (the first day of the new AZ session), which is in the future.
I used @csnardi's update script to update dates and retire reps. I spot checked a few and it seems like it's good, but this is my first shot at this so please let me know if I'm doing something wrong.
There were changes to committee files as well but it appears that the scraper and data on those have not been updated recently, so it didn't seem fit address in this commit.

Pull Request Test Coverage Report for Build 215

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

Totals

Change from base Build 212:
0.0%

Covered Lines:
925

Relevant Lines:
1391

💛  - Coveralls

thanks, looking good
we have branch protection set up to not let us merge this until it lints- I think that leaves us with two options: waiting until we reach the end date to merge or alter the dates for now (I'd been using 2018-12-31 as a sort of catch-all end_date, assuming in practice the extra week or two typically won't matter, but your dates are more accurate so I'd hate to say that the fix is to make them incorrect)
let me think over this a bit, maybe the linter logic can be adjusted without introducing too much risk

Thanks! Will get started on another one shortly

The Arizona update was a bit messy, in that anyone whose name did not match perfectly ended up with retiring the old legislator and adding a new one. Things that caused this included adding an initial (i.e. Timothy [M. ] Dunn) or a suffix (i.e. Daniel Hernandez [Jr.]).


coredns pod is in crashloop state on an arm64 node cluster

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy a 1 node arm64 cluster with k8s version - 1.17.13-rancher1-1 (KDM pointing to dev-v2.5)
When the cluster comes up active --> System project --> Coredns workload
Coredns pod is in crashloop state - rancher/coredns-coredns:1.6.5
Logs of coredns Container on the node:

docker logs 02ede23a8e8d
standard_init_linux.go:211: exec user process caused "exec format error"

Expected Result:
coredns should not be in crashloop state but in Active state
Other details that may be helpful:
On 1.16.15-rancher 1-2, core dns version: rancher/coredns-coredns:1.6.2 has the same issue
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5.1
Installation option (single install/HA): single node

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): custom
Kubernetes version (use kubectl version):

1.17.13-rancher1-1, 1.16.15-rancher1-2

Rancher v2.3.9 KDM dev-v2.3
v1.15.12 crashloop on ARM64 in Rancher v2.3.9  with rancher/coredns-coredns:1.6.2

v1.16.15 crashloop on ARM64 in Rancher v2.3.9  with rancher/coredns-coredns:1.6.2

v1.17.13 crashloop on ARM64 in Rancher v2.3.9  with rancher/coredns-coredns:1.6.5

rancher/rancher:v2.48
KDM dev-v2.4
v1.17.13 crashloop on ARM64 with rancher/coredns-coredns:1.6.5

ARM64 hasn't been fully validated on RKE1 so this is a known issue.

This issue is being caused by certain versions of the rancher/coredns-coredns manifests not having arm64 images, so images for amd arch are pulled and encounter an error when the entrypoint binary built for a different arch tries to run on the arm instance.
I see two straightforward options to resolve this:

We can rebuild manifests to include arm64 images
We can upgrade coredns images in our k8s versions to point to manifests that include the arm64 image

I'll do some more investigation to evaluate these approaches and potential impacts.


Styled the account page

Resolves : #23
Solution
I added some CSS properties that will make the account page somewhat presentable.
Testing

yarn dev
Observe the change

Resources
Before adding styles

After adding styles

This PR doesn't address the change indicated in the title, but makes changes to the homepage instead. This PR conflicts with PR#43 and we have a couple of options we can take to resolve the issue:

Merge the base branch of PR #43 into this branch
Close this PR and merge changes into the base branch of PR #43

I don't there's a way of avoiding merge conflicts, so I think option one would be best. Also, the testing steps don't work for anyone trying to run this. Refer to PR #43 for one good testing steps look like.

I changed the base branch of this PR (the conflicts need to be taken care of). Please update your PR title and description to highlight that these updates are for the login page, not the account page.

Since this PR conflicts with PR #43 I merged the base branch of PR #43 into this one as you suggested.


fix: [REGAPIC] Fix repeated fields handling for query parameters

if a field is declared as repeated in proto then its getter's name is get<FieldName>List not get<FieldName> instead.

if a field is declared as repeated in proto then its getter's name is get<FieldName>List not get<FieldName> instead.

Why do we want to change this? Looks cosmetic to me. If it is indeed required change, do we want to port this over to GRPC client as well, like here? Also in the same class, there is another similar logic as well.

@blakeli0 This is not cosmetic, this is required for the generated code to compile, as get<FieldName>List() getter is generted by protobuf for repeated fields and get<FieldName>() for scalar fields.
As for porting it to grpc code - maybe, but it is not in scope of this PR as I don't want to risk breaking grpc code while working on regapic part (it is going to be a big rollout and I would like keep regapic changes separate to simplify debugging and minimize risk of breaking what is already working).

As for porting it to grpc code - maybe, but it is not in scope of this PR as I don't want to risk breaking grpc code while working on regapic part (it is going to be a big rollout and I would like keep regapic changes separate to simplify debugging and minimize risk of breaking what is already working).

That makes sense and I think it's OK not porting over too. Because on gRPC side, the request extractor is only used for routing headers and the field can not be repeated AFAIK.
However, this is another example that we have long similar logics in multiple places which we could benefit a lot if extracted to common utility methods.


JUnit and tap reporter use method, that is not available in node 14

Describe the bug
The newly introduced JUnit reporter (#534 ) uses the String.prototype.replaceAll method. This is only available from node 15 onwards according to MDN, so the vitest fails with the following exception on node 14, which is, according to the docs, a supported node version:
(node:69289) UnhandledPromiseRejectionWarning: TypeError: String(...).replaceAll is not a function
    at escapeXML (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8173:24)
    at JUnitReporter.writeElement (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8198:29)
    at JUnitReporter.<anonymous> (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8279:20)
    at JUnitReporter.writeElement (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8202:20)
    at async JUnitReporter.onFinished (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8277:5)
    at async Promise.all (index 0)
    at async Vitest.report (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8864:5)
    at async file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8724:7
    at async Vitest.runFiles (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8728:12)
    at async Vitest.start (file:///PATH-REDACTED/node_modules/vitest/dist/create-3bc3fb6a.js:8672:5)

Reproduction
Reproduction of this issue is, at least for me, not possible using StackBlitz. The terminal in the StackBlitz environment shows Node version 14.16.0, but the replaceAll method is still available, when I run node there, thus the reproduction is not failing there. On the other hand, when I run the same on a local node installation with the exact same node version, the method is not present, which matches the availability information for this method on MDN. So I created a dedicated minimal reproduction repository
System Info
System:
    OS: Linux 5.11 Ubuntu 20.04.3 LTS (Focal Fossa)
    CPU: (8) x64 Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz
    Memory: 202.82 MB / 15.26 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 14.16.0 - ~/.nvm/versions/node/v14.16.0/bin/node
    npm: 6.14.11 - ~/.nvm/versions/node/v14.16.0/bin/npm
  Browsers:
    Chrome: 97.0.4692.71
    Firefox: 96.0
  npmPackages:
    vite: ^2.7.12 => 2.7.12
    vitest: ^0.1.20 => 0.1.20

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

My bad. Will fix in a moment.

Wow, that was fast! Thank you very much :-)


solved Page not found error #5

solved Page not found error #5
Here is preview working link: https://deploy-preview-6--thirsty-allen-a8c417.netlify.com/home

You are awesome @nitishk72 . Thanks a lot.


Parallax support

Hi! I can't seem to get the parallax background working without changing your index.js file.
According to this the parallax background needs to be inserted between the "swiper-container" and the "swiper-wrapper". There is no option for that right now (as far as I can see).
Perhaps you can consider supporting parallax in this plugin?
Thanks in advance!

Could you open a PR with your changes to the index.js? I'm interested to see.

I'm updating demo page now. I will try to add parallax demo and see if we need to change core file or not.

Can use parallax with v1.5.5. Check demo page for more detail


Add Replicache comparison

It would be nice to see how https://replicache.dev/ compares

Make a PR :)


[Question] Is it possible to add tags intead of structures?

i want to add structure tags instead of structures, will it work?

Do you mean in the config? If so, then no it's not currently possible, but will maybe be in a future update.


Training on custom dataset index error

Hi!
I am trying to train on a custom dataset and I get this error:
python train.py --config=yolact_base_config --resume=weights/resnet101_reducedfc.pth --start_iter=-1  --validation_size 21 --validation_epoch 10
loading annotations into memory...
Done (t=0.02s)
creating index...
index created!
loading annotations into memory...
Done (t=0.00s)
creating index...
index created!
/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/jit/_recursive.py:165: UserWarning: 'lat_layers' was found in ScriptModule constants,  but it is a non-constant submodule. Consider removing it.
" but it is a non-constant {}. Consider removing it.".format(name, hint))
/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/jit/_recursive.py:165: UserWarning: 'pred_layers' was found in ScriptModule constants,  but it is a non-constant submodule. Consider removing it.
" but it is a non-constant {}. Consider removing it.".format(name, hint))
/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/jit/recursive.py:165: UserWarning: 'downsample_layers' was found in ScriptModule constants,  but it is a non-constant submodule. Consider removing it.
" but it is a non-constant {}. Consider removing it.".format(name, hint))
Traceback (most recent call last):
File "train.py", line 504, in
train()
File "train.py", line 187, in train
yolact_net = Yolact()
File "/mnt/meganeura/domeemod/yolact/yolact.py", line 467, in init
self.semantic_seg_conv = nn.Conv2d(src_channels[0], cfg.num_classes-1, kernel_size=1)
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 408, in init
False, pair(0), groups, bias, padding_mode)
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 83, in init
self.reset_parameters()
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 86, in reset_parameters
init.kaiming_uniform(self.weight, a=math.sqrt(5))
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/init.py", line 381, in kaiming_uniform
fan = _calculate_correct_fan(tensor, mode)
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/init.py", line 350, in _calculate_correct_fan
fan_in, fan_out = _calculate_fan_in_and_fan_out(tensor)
File "/scratch/domeemod/anaconda3/envs/yolact-env/lib/python3.7/site-packages/torch/nn/init.py", line 282, in _calculate_fan_in_and_fan_out
receptive_field_size = tensor[0][0].numel()
IndexError: index 0 is out of bounds for dimension 0 with size 0
I have a dataset with which I have already trained a detectron2 mask-rcnn model, thus I am not really suspicious about the correctness of my dataset and annotations.
my custom config.py file:
dataset_base = Config({
'name': 'Grapes Dataset',
# Training images and annotations
'train_images': './grapeNation/train/',
'train_info':   './grapeNation/train.json',

# Validation images and annotations.
'valid_images': './grapeNation/test/',
'valid_info':   './grapeNation/test.json',

# Whether or not to load GT. If this is False, eval.py quantitative evaluation won't work.
'has_gt': True,

# A list of names for each of you classes.
'class_names': custom_classes ,

# COCO class ids aren't sequential, so this is a bandage fix. If your ids aren't sequential,
# provide a map from category_id -> index in class_names + 1 (the +1 is there because it's 1-indexed).
# If not specified, this just assumes category ids start at 1 and increase sequentially.
'label_map': None

})
grapes_dataset = dataset_base.copy({
'name': 'Grapes Dataset',
# Training images and annotations
'train_images': './grapeNation/train/',
'train_info':   './grapeNation/train.json',

# Validation images and annotations.
'valid_images': './grapeNation/test/',
'valid_info':   './grapeNation/test.json',

# Whether or not to load GT. If this is False, eval.py quantitative evaluation won't work.
'has_gt': True,

# A list of names for each of you classes.
'class_names': custom_classes ,

# COCO class ids aren't sequential, so this is a bandage fix. If your ids aren't sequential,
# provide a map from category_id -> index in class_names + 1 (the +1 is there because it's 1-indexed).
# If not specified, this just assumes category ids start at 1 and increase sequentially.
'label_map': None

})
Thank you in advance!

Ok I figured it out: num_classes in config.py has to be at least 2 (including background)


Sort out selecting a frame by one of its clauses

e.g. selecting an else clause on an if.
This might involve using the :has() pseudo class, see https://stackoverflow.com/questions/1014861/is-there-a-css-parent-selector

Decided against this. Clicking on an else or else if should just select that line, as it is implemented as a statement.  (However, selecting on the parent if-then frame will highlight all the else lines (without selecting them).


Camera 0.9.8+1 breaks build with Flutter 3.0.2

Steps to Reproduce

Create new flutter app (flutter create test && cd test)
Add deps<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Paste the demo page for camera module in main.dart (https://pub.dev/packages/camera/example)
Build and Launch (iOS Sim: iPhone 13 [iOS 15.5] in my case -- also replicated an error on Android 4a physical device)

You should see the exception: MissingPluginException (MissingPluginException(No implementation found for method availableCameras on channel plugins.flutter.io/camera))
Logs

Logs
[✓] Flutter (Channel stable, 3.0.2, on macOS 12.4 21F79 darwin-arm, locale en-US)
• Flutter version 3.0.2 at /opt/homebrew/Caskroom/flutter/2.8.1/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision cd41fdd495 (10 days ago), 2022-06-08 09:52:13 -0700
• Engine revision f15f824b57
• Dart version 2.17.3
• DevTools version 2.12.2
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/mike/Library/Android/sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
[✓] VS Code (version 1.68.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.42.0
[✓] Connected device (4 available)
• Pixel 4a (mobile)  • 1634991JEC234233                       • android-arm64  • Android 12 (API 32)
• iPhone 13 (mobile) • 66394C9F-16D2-4C44-96B1-22C626106387 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
• macOS (desktop)    • macos                                • darwin-arm64   • macOS 12.4 21F79 darwin-arm
• Chrome (web)       • chrome                               • web-javascript • Google Chrome 102.0.5005.115
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

Duplicate of #89500 https://github.com/flutter/flutter/issues/89500


August storage costs are not there

August storage cost on AWS for 2017 do not appear on the invoicing page. Is this an AWS anomaly or a bug?

Fixed says Abraham


Issue with UITableViewCell layoutMargins depending on device

Thank you for your work on this library and the awesome video series!
I am snapshotting a UITableViewController using assertSnapshot(matching: sut, as: .image(on: .iPhoneX)). When running on a iPhone XS Max, the test fails.
What seems to be the cause of this is that the cell's (and contentView's) layout margins vary depending on the device:

You can reproduce the issue by running the following project's tests in iPhone XS Max:
SnapshotMarginsIIssue.zip
Any ideas on how I can make the tests device agnostic?

Hi @kremizask! While you can use .iPhoneX, etc., configurations to emulate the traits and layout of specific devices, you still must use a consistent simulator. From the README:

⚠ Warning: Snapshots must be compared using a simulator with the same OS, device gamut, and scale as the simulator that originally took the reference to avoid discrepancies between images.

It's unfortunately just not possible to render screen shots of UIKit across simulators that differ in any of these three ways. We reported the discrepancy to Apple but they closed as "won't fix." As long as you run things on a consistent device or encode the device in the screen shot name, you should be fine, though!
#183 should eventually make this a lil easier.

Hi @stephencelis! If I understand correctly, these 2 Simulators I am using for recording the snapshot (iPhone X) and actually run the tests on (iPhone XS Max) should be compatible. They both have gamut: P3, scale: 3 and I recorded/ran the tests on iOS 12.2 Simulators (I had the same results with 12.1). Am I missing something?


Game flow

closes #50

add total score container and display on menu load
write a hideGame function in the Dom class
rip out stuff from game class and put into index.js

remove Dom class import and all Dom-related logic
remove run loop

do lots of setup in index.js
implement TimeBar that counts down until end of game
implement all logic involved in storing total games won to local
storage
add styling

for total-score container
for buttons

Looks good. I like the refactoring for the game flow moving into index.js


pipeline/extract_effects.ipynb file not found

Hi, I am trying to run workflow "A multivariate EBNM approach for mixture multivariate distribution estimate" with Minimal working example approach, but at initial step in the following command I am getting an error "ERROR: Failed to locate pipeline/extract_effects.ipynb.sos".
sos run pipeline/extract_effects.ipynb extract_effects
--name protocol_example.mashr_input
--analysis-units <(cat protocol_example.mashr.list | cut -f 2 )
--need-genename TRUE
--sum-stat protocol_example.mashr.list
On further checking that particular file "extract_effects.ipynb" is not present in the pipeline folder.
Can you please suggest from where I can get this notebook?
Thanks!

For your purpose which is multi-trait GWAS different molecular QTL
analysis, you don't need to run that pipeline designed for QTL. But the
format of data should be the same at the end. You can see the format of
data from this example:
https://stephenslab.github.io/mashr/articles/intro_mash_dd.html the
simdata data-set. It should contain "strong", "random" and "null" summary
statistics:

"strong" can be obtained when you do LD clumping, ideally jointly on all
traits, but also okay if you do it per trait and put together a list of
index variants thus clumped, then create such a table with rows being the
union of the index variants and columns being the traits themselves
"null" can be a random set of variants, after LD pruning and have
absolute value of z-score < 2
"random" can be a random set of variants after LD pruning.

For the null and random set, I believe with about 10K variants in each
category is sufficient. For "strong" we should hope to get as many as
possible depending on signals in the data.
when these are extracted please format to the same as shown in the vignette
on mashr pacakge website (the beginning of my response) and continue from
there.
On Fri, Aug 2, 2024 at 12:11 AM neeteshpandey @.***>
wrote:

Hi, I am trying to run workflow "A multivariate EBNM approach for mixture
multivariate distribution estimate" with Minimal working example approach,
but at initial step in the following command I am getting an error "ERROR:
Failed to locate pipeline/extract_effects.ipynb.sos".
sos run pipeline/extract_effects.ipynb extract_effects
--name protocol_example.mashr_input
--analysis-units <(cat protocol_example.mashr.list | cut -f 2 )
--need-genename TRUE
--sum-stat protocol_example.mashr.list
On further checking that particular file "extract_effects.ipynb" is not
present in the pipeline folder.
Can you please suggest from where I can get this notebook?
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/cumc/xqtl-protocol/issues/1024, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHADYPUJRS4P6OMYVP7OWDZPMBHBAVCNFSM6AAAAABL3Y5IEKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DGOJXHEZDINQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Remove "volatile" list - not appropriate to the fracking situation

[x] remove from external_dataset_tools
[x] remove from Chemical_index
[x] remove from DataDictionary

Removed this list from the external data set tools

Completed


[BUG] ts-plugin v0.2.1 throws with the 'src' property passed from grunt is a string

While migrating from grunt-ts we encounted an issue where our grunt file contained a 'string' rather than an array for the 'src' property.
Actual:
The compile fails with
 tsFiles.forEach is not a function Use --force to continue.
Expected:
This should not throw and ideally it should continue.

Release in v0.2.2


fix event subscribe

Codecov Report

Merging #65 into dev will decrease coverage by 0.08%.
The diff coverage is 0.00%.

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


Update README.md install command for development

I have seen in other repos that the README is included as the first page of the documentation. Maybe we could also do this here? So we avoid to have "duplicated" documentation?

Right now the two do not have the same structure. We might do this in the future but this is not a priority right now.


Bug in email list generation

In workshops/templates/workshops/instructors_by_date.html:25, we may wind up with a dangling comma if the last person doesn't have an email address. And we should test that we're allowed to contact them. See #765 for a better solution.

Fixed by @chrismedrela in #772.


Implement bump mechanism (mostly ported from rye)

Summary
Implement bump mechanism following major.minor.patch format.
fix #6298
I chose the bump name since version is already used though I would consider removing the current version handler for this

Test Plan
Simple idiomatic tests modifying the test pyprojec and checking the output statically

I'll review this. We'll need to consider what version management interface we want in the long run.

PEP440 https://peps.python.org/pep-0440/
and commitizen config https://commitizen-tools.github.io/commitizen/commands/bump/#version_scheme
could be good reference points

I still haven't had time to decide if we want to replace uv version for this.

Is this PR wanted? should I resolve conflicts?

I would find this useful, but I see there has been no action on this for a while. Currently I use bump-my-version to do this, but it would be great to have it built into uv.

yeah. the maintainers said they want to inspect other project \ alternative interfaces to have better context on this feature.
if someone is interested doing the research it would be cool.

This is great feature and removes the dependency on bump version. This one along with change-log generation makes this tool complete.

I think about this pretty often, but I'm still not quite sure what to do, e.g., cargo version is not used for this purpose.


Cost functions from pycolmap is not available

Thanks for the nice project! I am trying to follow examples/test_BA.ipynb, but it seems that cost_functions is not pycolmap attribute anymore (I install pycolmap through pip install pycolmap directly, in a clean python 3.10 conda environment).
I tried to look around in the pycolmap repo, and I didn't see cost_functions in it. I'd appreciate it if you could help me to find those pre-defined cost functions. Thanks in advance!

As mentioned in the pycolmap README, the development has moved to the colmap repo. The cost functions are in src/pycolmap/estimators/cost_functions.h but haven't made it into the previous release, so you need to build pycolmap from source rather than from PyPI. We will create a new release soon.

Thanks for the prompt response and the pointer! I was able to install pycolmap from source (from the main branch of the official colmap mentioned above).
I still encountered an error when adding the cost function:
    prob = pyceres.Problem()
    loss = pyceres.TrivialLoss()
    for im in rec.images.values():
        cam = rec.cameras[im.camera_id]
        for p in im.points2D:
---->         cost = pycolmap.cost_functions.ReprojErrorCost(
                cam.model, im.cam_from_world, p.xy
               )

The error message is
TypeError: Unable to convert function return value to a Python type! The signature was
	(camera_model_id: pycolmap.CameraModelId, cam_from_world: pycolmap.Rigid3d, point2D: numpy.ndarray[numpy.float64[2, 1]]) -> ceres::CostFunction

I may not install the pycolmap properly, as I copied colmap src folder to pycolmap subfolder and changed the CMakeLists.txt in pycolmap folder slightly for the path, to be able to run python -m pip install . successfully to install the library.
I'd appreciate it if you could take a look into this issue.

You likely need to install pyceres from source as well.

I may not install the pycolmap properly, as I copied colmap src folder to pycolmap subfolder and changed the CMakeLists.txt in pycolmap folder slightly for the path, to be able to run python -m pip install . successfully to install the library.

Why is this needed? python -m pip install . should work fine when when the source files are in src/.

Thanks for the help. I tried to install pyceres from sources. Now there seems to be a conflict between two libraries pycolmap and pyceres. When they are imported together, the following error shows up:
Check failed: !IsGoogleLoggingInitialized() You called InitGoogleLogging() twice!

Why is this needed? python -m pip install . should work fine when when the source files are in src/.
When I tried to run the command in the directory colmap/pycolmap, the pypi will complain it can't find source files. If I copy source files over to the same directory, it will work fine. I am not sure why. One guess is that pypi will copy things to one temporary build location, but only the current directory is included. Sorry I don't know much about how it works internally.

Thanks for the help. I tried to install pyceres from sources. Now there seems to be a conflict between two libraries pycolmap and pyceres. When they are imported together, the following error shows up:
Check failed: !IsGoogleLoggingInitialized() You called InitGoogleLogging() twice!

Apologies for the issues. Are you building with glog < 0.6.0? These versions have an issue that has been fixed in 0.6.0. We will add a min version requirement to CMake.

If I copy source files over to the same directory, it will work fine. I am not sure why. One guess is that pypi will copy things to one temporary build location, but only the current directory is included. Sorry I don't know much about how it works internally.

We have never experienced this in any of our build environments. What version of pip are you using?

Apologies for the issues. Are you building with glog < 0.6.0? These versions have an issue that has been fixed in 0.6.0. We will add a min version requirement to CMake.

No worries at all, thanks for the tip! I will try again later when I have time (on the weekend most likely).

We have never experienced this in any of our build environments. What versions of pip and Python are you using?

I am on Ubuntu 20.04, with system Python and pip via apt (Python 3.8.10, pip 20.0.2). I will try a newer version of Python and pip later.

Did you solve this? After installing the pycolmap and pyceres from source I got this
ImportError                               Traceback (most recent call last)
Cell In[4], [line 3](vscode-notebook-cell:?execution_count=4&line=3)
      [1](vscode-notebook-cell:?execution_count=4&line=1) get_ipython().run_line_magic('load_ext', 'autoreload')
      [2](vscode-notebook-cell:?execution_count=4&line=2) get_ipython().run_line_magic('autoreload', '2')
----> [3](vscode-notebook-cell:?execution_count=4&line=3) import pyceres
      [4](vscode-notebook-cell:?execution_count=4&line=4) import pycolmap
      [5](vscode-notebook-cell:?execution_count=4&line=5) import numpy as np

ImportError: libglog.so.1: cannot open shared object file: No such file or directory```

Were pyceres and pycolmap both installed from source? This this should not occur. Did it regardless of the order in which you import them?

Sorry for the late reply. I tried to install both pyceres and pycolmap from source today (basically go to the source code folder, and run pip install .). I didn't encounter the problem as @colin-de (yet). But when I imported pycolmap, there is an error:
xxx/site-packages/pycolmap.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZN5ceres6Solver7SummaryC1Ev
In the meantime, I can import pyceres, and run colmap itself successfully. My Ceres version from apt is 1.14.0.  I guess pycolmap might expect a different version.

now it works when I install the colmap again from source after updating the glog to 0.6.0
My ceres=2.1 colmap=3.9.1

Sorry for the late reply. After building both pycolmap and pyceres from source, I am able to run the notebook without problem. For reference, I am using ceres=2.2, colmap=3.10, glog=0.6.0.
Thanks for the help and I am closing this issue.


improve flags and usage

Codecov Report

Merging #36 (0ee3d93) into master (c4b63e2) will decrease coverage by 0.97%.
The diff coverage is 30.00%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   68.92%   67.95%   -0.98%
==========================================
  Files           4        4
  Lines         280      284       +4
==========================================
  Hits          193      193
- Misses         70       74       +4
  Partials       17       17

Impacted Files
Coverage Δ

cmd/root.go
25.35% <30.00%> (-1.52%)
:arrow_down:

Continue to review full report at Codecov.

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


docs(es): May 16 - May 23

Commits translated

[X] https://github.com/vuejs/docs/commit/64bae646c6b5c3427ac6c0e87fc07c6df9ecadf7
[X] https://github.com/vuejs/docs/commit/adbb5a3d272c12cace73b11f7e7a8c89fa218054 // This commit has two specific files which changes I cannot replicate for now
[X] https://github.com/vuejs/docs/commit/804620d2cb3e1c01525a552c0c63075f870758cd
[X] https://github.com/vuejs/docs/commit/3de891cd0c225b21698503e35f1203f598b970c4
[X] https://github.com/vuejs/docs/commit/7493b263972c078e39db3b3e4b41ac3e01f5f539
[X] https://github.com/vuejs/docs/commit/4447a5e1b78ff308ed1f426cff21109ad42de412
[X] https://github.com/vuejs/docs/commit/40d770b16423ccca245d99289fc9b546097fec02
[X] https://github.com/vuejs/docs/commit/11a07afc7e6d1b34aa378c468988c4a8f6b592a0

Two files from this commit https://github.com/vuejs/docs/commit/adbb5a3d272c12cace73b11f7e7a8c89fa218054, specifically src/tutorial/TutorialRepl.vue and src/examples/ExampleRepl.vue have some changes where code was deleted but in my current fork I can't do that because the deleted code is used in both files.


fix typo in startup topic

What issue does this PR address?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit follows our guidelines
[ ] Unit Tests for the changes have been added (for bug fixes / features)

Other information:

Thanks


How do I run it, doesn't find keyboard module

Hi, I know this is an old project but I still have to ask... How do I run this?
It keeps saying ModuleNotFoundError: No module named 'keyboard'
Sorry for the trouble.

PS: I installed all of the requirements, I made sure that keyboard exists on the system Requirement already satisfied: keyboard in c:\path

Hi SlitlRL,
Thank you for your attention to this project. You can try reinstalling the keyboard module from the official website: https://pypi.org/project/keyboard/
If it still doesn't work, you might want to check if you have multiple Python versions installed, and ensure that you installed the module for the correct version.


Receiving Unhandled Exception when using BlazorApplicationInsights with Azure AD SSO Integration on login callback

I have a small application using .Net 5 and Blazor Webassembly and using Azure AD SSO for authentication.   When the login authentication completes and issues the callback to the app it is generating the following exception.   It appears that a response from the login callback is not being handled in the BAI well.   The authentication process is still successful but generates a client messages that needs to be dismissed, so this is not a blocking issue but is a distraction in the user flow.   Also the authentication process did not generate this unhandled message prior to adding in the BAI client side logging.
I am attaching the logs from the client side and an image from the developer tools console.
unhandled_exception.log

Any thoughts on resolving this?

Hey,
I'm using this library in a .Net 5 Blazor wasm app with no issues. I'm thinking that your username is null or you are sending some illegal characters (spaces or ,;=|).
Can you please try to updated your Authentication.razor to look like this?
@page "/authentication/{action}"

<RemoteAuthenticatorView Action="@Action" OnLogInSucceeded="OnLogInSucceeded" OnLogOutSucceeded="OnLogOutSucceeded">
</RemoteAuthenticatorView>

@code{
    [Parameter] public string Action { get; set; }
    [CascadingParameter] public Task<AuthenticationState> AuthenticationState { get; set; }
    [Inject] private IApplicationInsights AppInsights { get; set; }

    public async Task OnLogInSucceeded()
    {
        var user = (await AuthenticationState).User;
        var newName = user.Identity.Name;

        if(!string.IsNullOrEmpty(newName))
        {
            // Replace illegal characters
            newName = new System.Text.RegularExpressions.Regex("[,;=| ]+").Replace(newName, "_");

            await AppInsights.SetAuthenticatedUserContext(newName, null, true);
        }
    }

    public async Task OnLogOutSucceeded()
    {
        await AppInsights.ClearAuthenticatedUserContext();
    }
}

Hey,
I'm using this library in a .Net 5 Blazor wasm app with no issues. I'm thinking that your username is null or you are sending some illegal characters (spaces or ,;=|).
Can you please try to updated your Authentication.razor to look like this?
@page "/authentication/{action}"

<RemoteAuthenticatorView Action="@Action" OnLogInSucceeded="OnLogInSucceeded" OnLogOutSucceeded="OnLogOutSucceeded">
</RemoteAuthenticatorView>

@code{
    [Parameter] public string Action { get; set; }
    [CascadingParameter] public Task<AuthenticationState> AuthenticationState { get; set; }
    [Inject] private IApplicationInsights AppInsights { get; set; }

    public async Task OnLogInSucceeded()
    {
        var user = (await AuthenticationState).User;
        var newName = user.Identity.Name;

        if(!string.IsNullOrEmpty(newName))
        {
            // Replace illegal characters
            newName = new System.Text.RegularExpressions.Regex("[,;=| ]+").Replace(newName, "_");

            await AppInsights.SetAuthenticatedUserContext(newName, null, true);
        }
    }

    public async Task OnLogOutSucceeded()
    {
        await AppInsights.ClearAuthenticatedUserContext();
    }
}

@IvanJosipovic that was it, I had wrapped the OnLogInSucceeded in a try catch so I didn't inspect closely what it was pushing back into the User Name context!!
Thanks for the quick response

@IvanJosipovic that was it, I had wrapped the OnLogInSucceeded in a try catch so I didn't inspect closely what it was pushing back into the User Name context!!
Thanks for the quick response


Log message length overflowing golang buf in special cases

The Problem
The discord has several mentions of users having multiple plugins like timestamp.trade throw with "error 120" in some cases, especially with timestamp.trades "create movie" option enabled.
The Issue
"error 120" and the immediate throw hints to a log message that overflows the golang buf i.e larger than what is defined by LOG_PAYLOAD_MAX_SZ at https://github.com/stg-annon/stashapi/blob/9e2e4a673f35a6e30b89c923c6857a9a0b6d7133/log.py#L38C1-L38C19
The truncation expects all characters to be of the same byte length, which is not true, especially for non-latin utf8 stuff and emojis and other encodings for example.
This is causing issues with max-length or close to max-length log output when it contains said characters.
Example
For example creating a new movie/scene/marker where the data is close to or above 64k characters and contains some special characters results in termination with exit code 120. Especially with user generated content like timestamp.trade, this can happen quite often and cannot always be handled by the plugin itself, because public methods of stashapp-tools themselves generate log-messages based on the requests data like update_scene.
A quick-and-dirty fix is to set to set LOG_PAYLOAD_MAX_SZ to something smaller like 64000 exactly to give some "wiggle-room" for larger-byte-length chars.

resolved with e133a96a30d294b1ce1efd10be75c60cb77466ed


Sodoku Enhancements

Adds darkened borders around the 3x3 squares
Adds related number highlighting
Adds completed number highlighting

These look great! Thank you again Trevor for your help!


Remove big/little endiant distinction in representation of DoubleWidth

This doesn't really add anything that we need; it changes the in-memory layout, but that's effectively an implementation detail. We might as well simply pick one and simplify things.
This does mean that DoubleWidth types have a "weird" layout on big endian systems, but that doesn't really cause any problems for anything. This would be a problem if we sunk these types into the standard library and wanted to bridge them to/from __int128_t and friends, because the representation wouldn't match. It's also trivial to revert this change, however, since it doesn't actually intrude into any operations, so let's keep things simple for now.

@swift-ci please test

@swift-ci please test

Fiiine. I was really pleased with my cleverness here.
A point in favor of simplification here is that even on little endian systems there can still be an alignment mismatch between this and the non-DoubleWidth types of the same width.


Fail to call neobundle#config#bundle

Fail to call neobundle#config#bundle('vim-powerline') ( 'vim-powerline' is an example )

Error detected while processing function neobundle#config#bundle:
line    1:
E121: Undefined variable: vim

Your using neocomplcache#config#bundle is wrong.
You must call like this.
neobundle#config#bundle(string('vim-powerline'))
I don't recommend call neobundle#config#bundle directly.


CallExpression is connected to wrong MethodDeclaration

In case the implementation of a method is not given, but a method with same name and same signature (but from a different class) exists, the CallExpression is connected to the wrong MethodDeclaration (via INVOKES edge).
Example:
class A {
public:
        void foo(int a);
};

class B {
public:
        void foo(int a);
};

void B::foo(int a) {
}

int main() {
        A a;
        B b;

        a.foo(1);
        b.foo(2);

        foo();

        C c;
        c.foo(3);
}

When loading the example c++ code into the cpg, all three calls of a method foo (a.foo, b.foo, c.foo) are connected to the MethodDeclaration of B::foo.

Yeah, it seems like this is completely broken :( Maybe it's finally time for a rewrite of the CallResolver :(


BUG: Changes in Heading for better look

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Description
Require changes in the heading for the better look and feel
"It needs some padding in the heading to fill the gap from the navigation bar."

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

pls let me do it under gssoc 24

Please assign me this issue. I would change the fonts and make it look attractive


Add prerequisites check and delete flag support for the longhorn package

Description
Adds the Longhorn Prerequisite Check to the Longhorn Package Example per https://longhorn.io/docs/1.4.0/deploy/install/#using-the-environment-check-script
Adds the Longhorn Delete Flag Update Per https://longhorn.io/docs/1.4.0/deploy/uninstall/#prerequisite
Environment Check

Delete Flag Change

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Other (security config, docs update, etc)

Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

@jeff-mccoy @Racer159 Items are resolved. Can you give this another look, please?

Thanks @Racer159 I like the changes you made,  They've been committed and  I updated the README.md accordingly.


[esco-content-grid] Focus indicators on Google Chrome

Is your feature request related to a problem? Please describe.
With the following, our accessibility team noted that there are no focus indicators on the favorite button and on the categories dropdown.
<esco-content-grid
  portlet-card-size="gridPortletCardSize"
  portlet-api-url="/uPortal/api/v4-3/dlm/portletRegistry.json?category=Online Services"
  hide-action="false"
>

</esco-content-grid>

We're currently suppressing the filtering functionality, but we'd like to have this functionality available for users to filter the portlet definitions.
Describe the solution you'd like
I'd like to see the appropriate CSS added such that there are focus indicators on the favorite button and on the categories dropdown.

The filter part is a "difficult" element (due to his way of UI representation, natural html elements ) and I'm not surprised that doesn't match all the accessibility requirements.
After if it's only the focus indicator that is missing maybe it could be fixed with some js/css inside ? have someone some ideas ?
Maybe the good html tag that we should use would be an input with a datalist, but I don't know if browsers bug are fixed as when I developed it it was not working well inside web-components for firefox (and more with IE 11), after I wasn't able to customize it easily with css on all browser, chrome didn't accepted css on.


You must add at least 1 account - Cannot use app for datamining historic data without actually scanning

TL;DR: Is there a way to configure the app, so I can use the full stats page but not the scanner?
Motivation:
I'm trying to get some people into locating nests from historic data. This map is much easier to install than PokemonGoMap and is aimed to be newbie friendly, so it would be an ideal choice for people unable or unwilling to install and configure stuff like python, environment variables, etc. I would recommend them to install this, copy my pogom.db over theirs and view the stats, but this map cannot be started without authenticating to niantic and actually start scanning.
Possible Solution:
Let the app start without any accounts. Just like when you use the -os (only server) flag in PokemonGoMap. This can be either done by removing the check for accounts, or by adding a "Start server, but do not scan" button, which ignores all account and authentication stuff and starts up in only-server mode.
I know that since noone develops this app any more this issue will get ignored, but I'll try anyway.

You could try editing the main.js file once it is installed and add the -os flag to see what happens when you click 'start server'.
The file to edit would be this one - https://github.com/mchristopher/PokemonGo-DesktopMap/blob/master/app/main.js
Which on Windows is user\AppData\Local\Pokemon\app-0.3.4\resources\app\main.js
You'd need to add a line after line number 211 with
'--os',
save then run the program. May not actually work but you could check.

Actually editing login.html and commenting out the check if (userAccounts.length <= 0) return false; helps getting further, but now I'm stuck at "Loading, please wait...".
Adding '--os' does not seem to have any effect.
I would like to see a console output / log from the python background processes to find the error but I wasn't able to dig them out yet..


Title: Fix Typo in README.md for GPT-4o

This pull request corrects a typo in the README.md file, changing "gpt 40" to "gpt 4o" to accurately reflect the project's way. This ensures clarity and consistency in the documentation, helping users understand the correct version of the model being referenced.

Fixed, thanks


fix codecov workflow

set files and fail_ci_if_error
NOTE: The default coverage data file .coverage is not supported by codecov, see here.

Codecov Report

Merging #45 (c7d1dff) into main (bde0831) will increase coverage by 90.46%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##           main      #45       +/-   ##
=========================================
+ Coverage      0   90.46%   +90.46%
=========================================
  Files         0        6        +6
  Lines         0      671      +671
=========================================
+ Hits          0      607      +607
- Misses        0       64       +64

see 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


could the logo be changed?

can we change the logo picture  of our app to a self_defined picture?

For shiny webapps you can set a favicon.
See: https://stackoverflow.com/questions/30096187/favicon-in-shiny

For shiny webapps you can set a favicon.
See: https://stackoverflow.com/questions/30096187/favicon-in-shiny
@wleepang
thank you for your suggestion
I  want to change the logo of the batch file,not on the web page.
Do you have any suggestion about this?

You can create a shortcut to the batch file, then change the icon of the shortcut (properties > change icon).

You can create a shortcut to the batch file, then change the icon of the shortcut (properties > change icon).

good idea
thank you @Ljupch0


go mod tidy warning

Hey I am just testing this for awesome-templ. When I run go mod tidy I get the following warning. Just thought I would let you know:
go: finding module for package github.com/axzilla/goilerplate-quickstart/ui/pages
go: github.com/axzilla/goilerplate-quickstart imports
        github.com/axzilla/goilerplate-quickstart/ui/pages: no matching versions for query "latest"

edit: to be clear not very important as the site works anyway

Also goilerplate is awesome. My thoughts on making it a shadcn port or something else are that I think shadcn port has merit in that people will be able to migrate their configs without much trouble. So if it matters at all my vote would go to keeping it as close to shadcn as possible while diverging only if you feel like there is a strong reason to

Hey @Oudwins,
Thanks for testing and for the feedback! The warning you're seeing with go mod tidy is actually normal behavior for a template project like this. It occurs because Go tries to find the local packages online, but can't (and shouldn't) find them since they're part of the local project structure.
This warning doesn't affect the functionality and will disappear once you rename the module to your own project name.
Thanks also for the feedback regarding shadcn - really appreciate your thoughts on this!


BLD: Install texlive packages in Travis CI

Closes #12543
texlive-latex-base needed for latex
texlive-latex-extra needed for utf8x.def

@evectant Thanks!
I am going to merge for now, but would want to test the mathjax extension in the future (only drawback of this that this will not work when using the docs offline)

@jorisvandenbossche Sure, just wanted to fix this for now so I could verify the documentation changes in #12492.

And it is looking good now!


Fix #127: Generate maven compiler settings according to selected Java version

Thanks again. As a follow-on I propose adding support up to Java 14 and dropping support for 9 and 10 since both are EOL.
This would require building the project with Java 14, which is available on both Travis and AppVeyor.