[WIP] Support RecordBatch

This pull request addresses https://github.com/Parsely/pykafka/issues/831 by adding support for RecordBatch message formatting to pykafka. This includes supporting the encoding and decoding of varints via the struct_helpers interface.

Codecov Report

Merging #844 into master will decrease coverage by 0.8%.
The diff coverage is 26.47%.

@@            Coverage Diff             @@
##           master     #844      +/-   ##
==========================================
- Coverage   83.99%   83.19%   -0.81%
==========================================
  Files          36       36
  Lines        3780     3814      +34
  Branches      560      568       +8
==========================================
- Hits         3175     3173       -2
- Misses        465      497      +32
- Partials      140      144       +4

Impacted Files
Coverage Δ

pykafka/utils/struct_helpers.py
73.52% <23.8%> (-22.22%)
:arrow_down:

pykafka/protocol/message.py
83.1% <30.76%> (-5.05%)
:arrow_down:

pykafka/utils/socket.py
81.25% <0%> (-18.75%)
:arrow_down:

pykafka/partition.py
72.72% <0%> (-12.13%)
:arrow_down:

pykafka/connection.py
86.13% <0%> (-1%)
:arrow_down:

pykafka/managedbalancedconsumer.py
83.44% <0%> (-0.67%)
:arrow_down:

pykafka/broker.py
93.06% <0%> (-0.58%)
:arrow_down:

pykafka/balancedconsumer.py
89.18% <0%> (-0.3%)
:arrow_down:

pykafka/simpleconsumer.py
85.68% <0%> (ø)
:arrow_up:

pykafka/handlers.py
94.11% <0%> (+0.73%)
:arrow_up:

... and 1 more

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 1ba51f6...37b6afb. Read the comment docs.

The protocol logic for RecordBatch is implemented and untested. The next step is to figure out how to get Kafka to start speaking in RecordBatch instead of MessageSet so I can run some tests to work out the protocol implementation bugs. Once that's done, the next step will be to make changes to the producer and consumer to have them use RecordBatch when appropriate.


added Readme7.txt

Summary

Test plan

@vinay72 Is this for #6957? If so, it should be part of the docs directory.
@rickhanlonii thoughts on where this sort of docs fits?

Is there anything else which I need to do with this issue?

@vinay72 this needs to be part of the documentation, not a standalone text document.
Now, I'm not sure which document, though... I wonder if it makes sense to create a FAQ document?

@vinay72 this needs to be in a markdown file, not odt. You can do yarn start inside the website directory to run the website locally.


Working with Raspberry pi 0?

Hi,
I'm trying your work with an rapberry pi 0 with hyprit, and unfortunatly it does'nt work.
Could you help me? I'm new with docker solution
thanks in advance
Alex

This docker image is for x86 processors, so it won't work with the Raspberry Pi's ARM processor. This is typical for docker images -- I know there is a vibrant Raspberry PI Docker community, but unless an image mentions the pi or ARM processors, it probably won't work.
You might be able to modify this image's Dockerfile to build from an arm-compatible base image, like easypi/alpine-arm. I have not tried this.

I will try, thanks for your reply


gofabric8 0.4.40, templates 2.3.9 fails with "Cannot yet process kind List, kind for social"

Using the aforementioned versions, installation fails:
Loading fabric8 releases from maven repository:https://repo1.maven.org/maven2/
Starting fabric8 console deployment using 2.2.171...
SecurityContextConstraints restricted is configured correctly
SecurityContextConstraints restricted.........................................✔
SecurityContextConstraints fabric8-fabric8 is setup correctly
SecurityContextConstraints fabric8............................................✔
SecurityContextConstraints fabric8-sa-group is setup correctly
SecurityContextConstraints fabric8-sa-group...................................✔
addClusterRoleToUser cluster-admin system:serviceaccount:fabric8:fabric8......✔
addClusterRoleToUser cluster-admin system:serviceaccount:fabric8:jenkins......✔
addClusterRoleToUser cluster-reader system:serviceaccount:fabric8:metrics.....✔
addClusterRoleToUser cluster-reader system:serviceaccount:fabric8:fluentd.....✔
addServiceAccount fluentd.....................................................✔
addServiceAccount registry....................................................✔
addServiceAccount router......................................................✔
Creating fabric8 console template resources from 4 objects
fabric8 console...............................................................✔
Adding OAuthClient redirectURL: http://openshift-master.home:31804
Created fabric8 console
Installing templates!
Downloading apps from: https://repo1.maven.org/maven2/io/fabric8/forge/distro/distro/2.3.9/distro-2.3.9-templates.zip
Loading template openshift/main/chat-letschat.yml
Loading template openshift/microservices/hubot-notifier.yml
Loading template openshift/microservices/hubot-letschat.yml
Loading template openshift/microservices/fabric8-docker-registry.yml
Loading template openshift/microservices/ingress-nginx.yml
Loading template openshift/microservices/elasticsearch.yml
Loading template openshift/microservices/prometheus-blackbox-exporter.yml
Loading template openshift/microservices/artifactory.yml
Loading template openshift/microservices/gogs.yml
Loading template openshift/microservices/fluentd.yml
Loading template openshift/microservices/grafana.yml
Loading template openshift/microservices/jenkins.yml
Loading template openshift/main/social.yml
Processing resource of kind: List version: v1
Cannot yet process kind List, kind for social
gofabric8, version '0.4.39' (branch: 'master', revision: 'b4cb62d')
build date:       '20160818-05:49:06'
go version:       '1.6.3'
It seems that the templates were bumped on their minor version since yesterday, and I failed to find a way to tell gofabric8 to use an older version of them.

Release 0.4.40 switched the "Cannot yet process kind List, kind for social" into a warning, deployment proceeded up to cd-pipeline installation.
It then fails with:

Installing: cd-pipeline
Creating cd-pipeline template resources from 25 objects
Processing resoucre Secret
Failed to create Secret: secrets "gogs-https-admin" already existsProcessing resoucre Secret
Processing resoucre Secret
Processing resoucre Secret
Processing resoucre Secret
Processing resoucre Secret
Processing resoucre Secret
Processing resoucre Secret
Processing resoucre ServiceAccount
Processing resoucre ServiceAccount
Processing resoucre Service
Processing resoucre Service
Processing resoucre Service
Processing resoucre Service
Processing resoucre Service
Processing resoucre Service
Processing resoucre Service
Processing resoucre ConfigMap
Processing resoucre ConfigMap
Processing resoucre ConfigMap
Processing resoucre DeploymentConfig
Failed to create DeploymentConfig: DeploymentConfig "fabric8-docker-registry" is invalid: metadata.namespace: Required valueProcessing resoucre DeploymentConfig
Failed to create DeploymentConfig: DeploymentConfig "fabric8-forge" is invalid: metadata.namespace: Required valueProcessing resoucre DeploymentConfig
Failed to create DeploymentConfig: DeploymentConfig "gogs" is invalid: metadata.namespace: Required valueProcessing resoucre DeploymentConfig
Failed to create DeploymentConfig: DeploymentConfig "jenkins" is invalid: metadata.namespace: Required valueProcessing resoucre DeploymentConfig
Failed to create DeploymentConfig: DeploymentConfig "nexus" is invalid: metadata.namespace: Required valuecd-pipeline...................................................................✘ DeploymentConfig "nexus" is invalid: metadata.namespace: Required value
No router DeploymentConfig so not creating routes

Sorry about that! I just hit these issues this morning too! Wanna try 0.4.41 - thats working here
https://github.com/fabric8io/gofabric8/releases
I just sent a mail to the mailing list on this btw:  https://groups.google.com/forum/#!topic/fabric8/EaS7PjdVsCM
BTW to install an old version of devops/forge you can use:
 gofabric8 deploy -y --version-devops=2.2.234 ...

I'm gonna mark this as closed for now - but please let us know if you hit any more issues. We've hit a few bumps recently with the upgrade to the new YAML / Deployment/DeploymentConfig / fabric8-maven-plugin (see more detail on the changes here- but hopefully we're over the hurdle now

Confirmed: 0.4.41 fixes the issue. Thanks a lot! 👍

Phew @clueware so sorry about that!


Publish plugin to the portal

On a previous attempt, the plugin failed to be published to the Gradle plugin portal due to a memory deficiency. This is presumably caused by the Kotlin runtime requiring more memory than Groovy runtime.
In an attempt to publish the plugin, we increase the memory limit for the Gradle JVM twofold.

@armiol, PTAL.


Mustache(5) example not working

Template:
{{#person?}}
Hi {{name}}!
{{/person?}}
Dictinary:
{
"person?": { "name": "Jon" }
}
Crashes -> this also does not work in the javascript version, I will submit a bug there also.

It looks like this is a duplicate of issue #15, which has been closed.  See the comments there for more information and the resolution.


[HOLD for payment 2024-12-03] [$125] Share Code - Inconsistent profile pic shape in admin room when accessed via #Admin hyperlink

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.59-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: NA
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Create workspace
Go to the #admin room (if you couldn't find it in LHN go to account setting> workspace > click on the three dots then click on #admin room)
Click on the header > share code then see the shape of the profile picture
Go to workspace editor > profile > share code > click on #admins hyper link
click on the header > share code and see the shape of the profile picture.

Expected Result:
The profile picture should consistently appear as a circle
Actual Result:
When navigating to the admin room share code through the #admin hyperlink, the profile picture appears as a rectangle
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/b05f5cab-6736-47f6-a86d-bf7deef46a28
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021854973788673857466
Upwork Job ID:<PHONE_NUMBER>673857466
Last Price Increase: 2024-11-08
Automatic offers:

ikevin127 | Reviewer | 104896217

Issue OwnerCurrent Issue Owner: @joekaufmanexpensify

Reproduced.
https://github.com/user-attachments/assets/775d9473-5bed-41b6-984c-73cd68b3b07e

Seems like a pretty minor bug, so bumping down.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The profile picture in QR code sharing displays as a circle when accessed from the admin room, but shows as a rectangle when accessed via the "#admin" hyperlink in workspace editor.
What is the root cause of that problem?
The logoBorderRadius in the QRCode component is incorrectly set to the full QR code size (size) instead of being calculated based on the logo size. To create a circle, border radius should be half of the logo's width/height
https://github.com/Expensify/App/blob/533aed65143e5bb47b9692952995be81368eed7b/src/components/QRCode.tsx#L86
What changes do you think we should make in order to solve the problem?
Update the QRCode component to properly calculate the border radius:
size * logoRatio
logoBorderRadius={(size * logoRatio)/2}

Proposal
Please re-state the problem that we are trying to solve in this issue.
When users view shared QR codes in the Expensify app, the profile picture shape is inconsistent depending on the navigation path taken. Specifically:

When accessing the share code through the normal admin room path, the profile picture appears as a circle (correct)
When accessing the share code through the "#admin" hyperlink in the workspace editor, the profile picture appears as a rectangle (incorrect)

What is the root cause of that problem?
The issue occurs because when the user clicks on the hyperlink here:
WorkspaceProfileSharePage.tsx#L89,
the current QR code displayed in WorkspaceProfileSharePage is not being properly closed. As a result, when navigating to the new page where the QR code is shown again, two QRShare components are mounted simultaneously. You can verify this by checking the React dev tools for the two QRShare components. This is causing the SVG inside the QR code to lose the border-radius styling, making the profile picture appear as a rectangle.
What changes do you think we should make in order to solve the problem?
When the user clicks on the hyperlink, close the current modal before navigating to the new page. This can be done using the Navigation.dismissModal() method.
Navigation.dismissModal();
Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(adminRoom.reportID ?? ''));

Thanks @Shahidullah-Muffakir for the proposal, the RCA checks out and the proposed solution fixes the issue in accordance with the Expected result.
Suggestion
In order to DRY the code down to one line you might as well write it like this:
Navigation.dismissModal(adminRoom.reportID);

since dismissModal has support for reportID which if passed calls:
https://github.com/Expensify/App/blob/c75b1bcc4c2a23a3c1d5ed84c844f82ed3996293/src/libs/Navigation/dismissModalWithReport.ts#L24-L29
🎀👀🎀 C+ reviewed

Suggestion
In order to DRY the code down to one line you might as well write it like this:
Navigation.dismissModal(adminRoom.reportID);

since dismissModal has support for reportID which if passed calls:
https://github.com/Expensify/App/blob/c75b1bcc4c2a23a3c1d5ed84c844f82ed3996293/src/libs/Navigation/dismissModalWithReport.ts#L24-L29
🎀👀🎀 C+ reviewed

@ikevin127, Thank you for pointing that out! We’ll use that.

CME doesn't seem to be OOO, not sure what's the hold up here.
cc @grgia @joekaufmanexpensify for visibility.

@grgia mind signing off on this one when you have a sec?

@joekaufmanexpensify Do you think it's possible to re-assign CME for this issue ? Must be caught-up with some higher priority tasks and we wouldn't want deter from that. There's another issue reported on Slack with the same situation here.

PR in review

Awaiting final PR review from @grgia

♻ Status updaate: PR was merged & deployed to staging and is now awaiting production deploy.

Sounds good. TY!

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on both staging and production
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Great. I will handle next steps!

@Shahidullah-Muffakir offer sent for $125!

This is a pretty niche bug, I don't think we need a regression test everytime here.

All set to pay:

@Shahidullah-Muffakir $125 for PR via Upwork.
@ikevin127 $125 for C+ via Upwork.

@ikevin127 $125 sent and contract ended

@Shahidullah-Muffakir $125 sent and contract ended

Upwork job closed.

All set, thanks everyone!


Update readme title

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests


Unable to run CsApplicationAccessPolicy from PWSH [MacBook]

[Unable to run CsApplicationAccessPolicy from PWSH [MacBook]]
When trying to initiate any command related to CsApplicationAccessPolicy
New-CsApplicationAccessPolicy
Get-CsApplicationAccessPolicy
The tenant nowdomain.onmicrosoft.com throws Session error. Even after valid login (as Admin).

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

ID: b6533a7d-ddc4-eef1-cca6-0ee4786163ef
Version Independent ID: 8958282d-6816-c8c3-cd0b-07b50681aacf
Content: Get-CsApplicationAccessPolicy (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Get-CsApplicationAccessPolicy.md
Service: skypeforbusiness-powershell
GitHub Login: @frankpeng7
Microsoft Alias: frpeng

Versions and Modules available.

Hi @raguravindran,
Can you please try with the latest Teams PowerShell Module version.  Older versions like 2.5.0 are deprecated, ref - https://docs.microsoft.com/en-us/microsoftteams/teams-powershell-supported-versions.
Thanks,

@pbafna03 Thank you, after update it worked. closing this issue


JJJ: Watch coffeeabout.com

JJJ requests the watch of the watch_keyword coffeeabout\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

coffeeabout\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

https://chat.stackexchange.com/transcript/message/62134950#62134950

Approved by tripleee in Charcoal HQ


Clear cache when you change database connections

Feature Description
When experimenting with Evidence, it's common to swap between databases. For example you may start using duckdb or csv files, and then wish to connect to a cloud data warehouse.
However, after changing the db, the cached queries and data is preserved until you edit the queries, and displayed in the web page, which is confusing, as it can make you think that the database change was not successful.
Current Solution
Delete files manually from .evidence/template/.evidence-queries
Or just change query SQL

I think this would be reasonably well addressed by #730, and easier to implement afterwards

With multiple data source support this is no longer relevant


VS Code Extension redirect is buggy

Report

[ ] If go to https://dvc.org/doc and click on the "Visual Studio Code" link (or any other instance of that link throughout docs), you may notice an intermediate page rendered with a 404-looking error.
[ ] Same if you load https://dvc.org/doc/vs-code-extension directly.
[ ] Once you're in the VSC Marketplace page (after being redirected), if you hit BACK in the browser, you get to stay at the Not Found page).

Cc @iterative/websites

Weird interaction with redirects that look like local pages, which are handled differently by the Gatsby's SPA via the Link component. From what I can see, we'll have to:

Find a way to ensure that links to local page redirects are recognized as such and treated as external links (either automatically or manually, with pros/cons to each)
Possibly fight with gatsby-plugin-catch-links because it's kind of kludgy and invasive.

It's a bit late for me, but I can get a patch in early tomorrow if it's not already done.

Sounds good. I don't think it's critical or anything, but it sure looks like a bug.

I remember fixing a similar issue mentioned for cml.dev.

https://github.com/iterative/cml.dev/issues/346#issuecomment-1283932384

But, I don't see that issue with dvc.org. Could it be a cache issue?

Firefox

Chrome

Also working fine with Safari

Works OK after force-reload... 🤦🏼 Sorry for false alarm!


Updates are too quick

When I'm writing markdown on an empty line, I keep getting a linting warning about trailing-spaces whenever I press 'space'. I think it would be more convenient to only update when the file is saved or maybe after a configurable delay.

Thanks for the feedback. This is a great suggestion, so I will implement a configurable delay. I will consider if it's also possible to implement update only on save, but I'll have to work out how that might work. But I see the problem, so I'm sure I can figure out a way to improve this.

Isn't this something that should be done in the problem matcher of VSCode itself? I mean, your (awesome) error-lens is another style to display problems and it shouldn't contain any delay logic, IMHO.
As soon as the code gets those squiggly lines, error-lens should apply its style, too.
Am I missing anything?

@gersilex Makes a good point and I would have to agree with him. As I've continued to use the extension, I've come to realize that an artificial delay wouldn't be very beneficial. I'm closing this issue because I don't think it's worth implementing.

Thank you for the feedback, it's appreciated!
I am trying to make this configurable and improve it, so hopefully the next release will feature improvements in this area.


docker-langserver not found

Hey, I have just installed your extension, thanks for it!
When I start up nvim Dockerfile I get [coc.nvim]Server languageserver.dockerfile failed to start: Command "docker-langserver" of languageserver.dockerfile is not executable: Error: not found: docker-langserver
But it seems to work, also server starts, I can see that process running:
43416   ??  S      0:00.18 /usr/local/Cellar/node/13.11.0/bin/node /Users/FOO/.config/coc/extensions/node_modules/coc-docker/node_modules/dockerfile-language-server-nodejs/lib/server.js --node-ipc --node-ipc --clientProcessId=43415z
Do you have any idea why nvim is printing out that error message?

Please ignore it, I had some overhanging configuration that caused it.


Peer conected, bad argument L2255

Hey @jbyuki awesome project. I'm looking to experiment with and learn from it so thanks.
I've setup heroku server then executed InstantJoinSession damp-tundra-81540.herokuapp.com
I get the following error:
Error executing vim.schedule lua callback:
...e/nvim/site/pack/packer/opt/instant.nvim/lua/instant.lua:2255:
bad argument #1 to 'insert' (table expected, got nil)

table.insert(events, "ERROR: Tried to join an empty server") Somehow there is no events table and thats what causing part of the issue

Hey, thanks for the kind words.
Thanks for the bug feedback as well. This shouldn't be too difficult to resolve.
This is caused by some debug code that I left (all table.insert(events,... ), they could be probably removed but I will do a quick commit to fix that.
Also I see you use InstantJoinSession [host] to connect, is it the first client?
If not you should use InstantStartSession [host] and the error should be gone.
I will do a quick fix.

e10a6791547bf15cf0c7ad54a4c535260af0a3e1 should do it. Thanks again.

Oh nice thanks @jbyuki , So First I start session then let my friend join right. We are getting the following error
Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:1583: bad argument #1 to 'insert' (table expected, got nil)

Error executing vim.schedule lua callback: ...e/nvim/site/pack/packer/opt/instant.nvim/lua/instant.lua:719: Vim:E474: Trailing characters: [7,117,"conni"]

Yes, first start the session and then connect. Maybe it should be more clear in the README.
Ok, I see, I will get back to you in a moment.

Error executing vim.schedule lua callback: ...e/nvim/site/pack/packer/opt/instant.nvim/lua/instant.lua:976: bad argument #1 to 'ipairs' (table expected, got nil)
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:1236: bad argument #1 to 'insert' (table expected, got nil)
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:719: Vim:E474: Trailing characters: [6, "", [123, 2], [0, 699274, 1048576], []]
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:719: Vim:E474: Trailing characters: [6, "", [123, 2], [0, 699274, 1048576], []][6, "", [123, 9], [0, 425234, 1048576], []]
Error executing vim.schedule lua callback: ...e/nvim/site/pack/packer/opt/instant.nvim/lua/instant.lua:721: Vim:E474: Trailing characters: [8,124]
So far instantJoinSingle worked for us the other is err resulting from session

Error executing lua callback: ...e/nvim/site/pack/packer/opt/instant.nvim/lua/instant.lua:2069: attempt to index upvalue 'prev' (a nil value)
When trying  InstantFollow
|| Connected!
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:1879: bad argument #1 to 'insert' (table expected, got nil)
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:1879: bad argument #1 to 'insert' (table expected, got nil)
|| Connected. 0 other client(s)
|| Connected. 0 other client(s)
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:719: Vim:E474: Trailing characters: [7,126,"tami5"]
|| Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:719: Vim:E474: Trailing characters: [7,126,"tami5"][3]

After some time of expermenting on Single

The trailing character issue seems to be linked to network fragmentation.
I should have fixed that in a previous commit.
Did you guys update instant.nvim to the latest commit?
The error line numbers don't seem to match to the current files as well.
Error executing vim.schedule lua callback: .../conni/.config/nvim/plugged/instant.nvim/lua/instant.lua:1583: bad argument #1 to 'insert' (table expected, got nil)

The current file doesn't contain this line at https://github.com/jbyuki/instant.nvim/blob/e10a6791547bf15cf0c7ad54a4c535260af0a3e1/lua/instant.lua#L1583
If that was it, try updating both clients.
If not, I will continue investigating the issue.

Also make sure, the server files are up to date.

@jbyuki me and @conni2461 had an amazing time testing and experimenting with instant.nvim. So far InstantJoin/Start Single worked well for us, as well as InstantFollow. Although some point in our instant the connection broke.
We will do further testing some time this week and report back here.
Also, I'll remember to update server files along with the plugin.
Thanks a lot, again amazing work :smile:

Great, glad I could provide you both with some fun experimenting.
The plugin is still rather unstable so really any help will be very helpful.
Thanks again, and see you soon then.

Closing this for now as the initial bug was fixed.


blank icons/ enchants

it does that with both versions and doest let me change anything

I'm assuming, since you closed this issue yourself, you found the path to your game files for it to use and everything worked after that?

I kinda found a way too it with all the old enchants but the new ones are
bugged out for me from what I can tell
On Mon, Dec 14, 2020, 23:33 Mike Holt<EMAIL_ADDRESS>wrote:

I'm assuming, since you closed this issue yourself, you found the path to
your game files for it to use and everything worked after that?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CutFlame/MCDSaveEdit/issues/15#issuecomment-745045499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQKDWOGJE65OPIA6VB4YGLSU3RIDANCNFSM4UV3Q3EQ
.

Try the latest version. It asks where to find your game content if it's not in the default install location.

I tried that but it gives me an error ill send screenshots later im
probably doing something wrong
On Thu, Dec 17, 2020, 23:10 Mike Holt<EMAIL_ADDRESS>wrote:

Try the latest version. It asks where to find your game content if it's
not in the default install location.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CutFlame/MCDSaveEdit/issues/15#issuecomment-747855329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQKDWJBHKWIGUSKSHMOZEDSVLI3BANCNFSM4UV3Q3EQ
.


[Question] Adding custom fields to request logs

Is it possible to add a custom field to all request logs?
We are running behind Cloudflare and would like to include the value of cf-connecting-ip in our logs.
We could do this in our middleware by manually logging the value, but I'm wondering if there's a way to get the IP included in the logs that are automatically output by Next/Vercel on request.

hi @b0o , you can create a child of a logger with the args you want, and the args will be attached to every log record. for example:
func handler(req) {
  const log = req.log.with({cloudflareIP: 'someValue'})

  log.info('something')
  ...
}

export default withAxiom(handler)

the output of this will be something like:
{'message': 'something', 'fields': {'cloudflareIP': 'someValue'}}

@b0o Another option would be to wrap withAxiom in a custom function that adds these fields to the logger. Please re-open if your issue remains unsolved 😌


Status dashboard updates

Follow up action from incident retro

[ ] add status bar description
[ ] make status bar description updateable
[x] number of ECS instances in a cluster
[ ] better display of stability

Closing as out of date.


Analysis running message should have 3 states

When being waiting on the Batch API to be run we should say: "Analysis enqueued"
When the analysis is running we should say: "Running analysis"
When done: "Analysis done"

Unifying on the other issue


[Question]: Ask about new features

Describe your problem

Can I use this component (invoke_0) to realize the function of ASR and nest agent ? thanks you ！

ASR is not feasible, which needs audio input.
What do you mean by nested agent? Could you make an example?

ASR is not feasible, which needs audio input. What do you mean by nested agent? Could you make an example?

thanks for your reply , it means I invoke another agent in curent agent

Sorry, not feasible for now.

Sorry, not feasible for now.

thanks


KAFKA-7299: Batch LeaderAndIsr requests for AutoLeaderRebalance

Currently, in KafkaController.checkAndTriggerAutoLeaderRebalance(), we call onPreferredReplicaElection() one partition at a time. This means that the controller will be sending LeaderAndIsrRequest one partition at a time. It would be more efficient to call onPreferredReplicaElection() for a batch of partitions to reduce the number of LeaderAndIsrRequests.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@junrao Please review this minor patch. Thanks.


Convert string UUIDs to CBUUID for advertisement

This is to fix https://github.com/notjosh/bleno-mac/issues/4, a bug which causes the advertising data to not contain the service UUIDs. This prevents some centrals from being able to properly discover the peripheral while scanning.

@notjosh Essential fix for advertising service UUIDs correctly. If you want to use e.g. background scanning on either iOS or Android, this is a pre-requisite, as advertised service UUIDs are the only allowed "subscription filter" you can determine. Quote from Apple's documentation:

Apps that have specified the bluetooth-central background mode are allowed to scan while in the background. That said, they must explicitly scan for one or more services by specifying them in the serviceUUIDs parameter

@notjosh Can confirm this PR solves problem of service id not being advertised on osx mojave.


Reviews Tab in Items Should be Reverse Chronological

Self-explanatory.

Contracts and ratings don't have any time data recorded in them. That would have to be added on the server before ratings can be sorted using that data.

@cpacia and @jjeffryes Any updates on this? Would need to be implemented into the new reviews tab at the store level as well.

I don't believe we've added the time data to the reviews.


[IOTDB-709] Create schema automatically

This pull request implements schema automatic creation in distributed version.
If an InsertPlan is executed but an exception is encountered because the storage group is not set yet, or the timeseries is not created, the MetaGroupMember will first execute a SetStorageGroupPlan, or CreateTimeseriesPlan, and then execute the InsertPlan again.
These plans will be routed among the cluster, so that the corresponding nodes will be notified.

Good job. But I have a question: if you enable AutoCreation, the receiver of the Insertion plan will create schemas locally and the sender will not sense a schema missing. How do you resolve it?

A new property ENABLE_AUTO_CREATE_SCHEMA is added in cluster configurations, which will replace the one in standalone version. In other word, the ENABLE_AUTO_CREATE_SCHEMA in iotdb-engine.properties is alwayes treated as FALSE. Therefore, the auto schema creation on storage engine level will never be triggered.


Copy (ctrl+c) copies only the first character of selection

In 0.145 on linux, after some encoding trouble, I also noticed the problem that i can't copy selected text anymore. Also the file saved as UTF8 is unreadable in eg. gedit or vim (might be an additional issue)

I can confirm that copying will sometimes fail and only copy one character.  Re-copying does not help.

Not sure if you care, but this bug still happens in 2017.
Atom version 1.18.0 x64.


ScriptResult should reflect status of RVRT in addition to panics

Currently, the ScriptResult receipt only reflects errors caught by the VM (panic), but if a contract developer uses RVRT to indicate some failure state, the ScriptResult considers this as a success.
While users should look for the revert or panic receipt, I think it would be a better UX if the script result also reflected the status of user-defined errors (RVRT) in addition to panic receipts.

I feel like a revert should be presented to the user/dev as a failure, not a success.
The only exception (and not really) would be a test that is expected to revert with a specific message or error code.

In the case of a negative test, it would need to look for the revert receipt to see if a specific revert code was given. ScriptResult is currently specified in the VM as more of a boolean success/failure and isn't intended to propagate the actual panic reasons or error codes (yet?).

Smart Contracts should do one thing well and revert otherwise. I think it makes sense for revert to be allowed to be considered a success state as with things like negative tests for something like a test to ensure permissions are properly revoked, etc

Typically when writing a negative test, you're expecting a specific failure mode. For example in Rust, if you wanted to ensure some kind of validation code worked you'd verify the error returned using something like:
#[test]
fn test {
    let result = some_operation(bad_input);
    assert!(matches!(result, Err(BadInputError)));
}

It seems strange to be writing negative tests where the transaction result is successful. While not exactly equivalent since we're talking about receipts, but in Rust you wouldn't wrap an error response inside of an Ok variant to make it easier to write negative tests.

Was confused in regards to where success is interpreted, reread though and definitely agree it makes more sense to consider it a failure and return the RVRT error message if anything was included so the test can handle it as needed


Design proposal for resource management policy and resource reference

New features:

Allow user to define resource management policy to instruct the provider how to manage Kubernetes resources in a fine-grained manner.
Allow user to define resource references for an Object as dependencies to retrieve values from dependent resources at runtime and guarantee the resource rendering in a specified order.

I finally got around to skimming this - thanks @morningspace! I think a key question that will come up in this proposal are which parts belong in this provider specifically, vs which would make sense as 'part of the XRM' (i.e. something we'd adopt for all providers. As you probably know many of these features have been requested 'upstream' as part of Crossplane in general, for example:

https://github.com/crossplane/crossplane/issues/1722
https://github.com/crossplane/crossplane/issues/2439
https://github.com/crossplane/crossplane/issues/2072

My hunch is that Observe-only resources (e.g. the resource management policy you propose) is probably something we want to adopt everywhere, whereas we've historically strongly preferred to optimise for eventual consistency over ordered create/deletes.
Perhaps we could consider using provider-kubernetes as a proving ground for some of these ideas? Start here then adopt them elsewhere if they seem to be working well.

@negz really appreciate your time to read through this lengthy doc :-) And, I totally agree with you!

Observe-only resources is probably something we want to adopt everywhere, whereas we've historically strongly preferred to optimise for eventual consistency over ordered create/deletes.
Perhaps we could consider using provider-kubernetes as a proving ground for some of these ideas? Start here then adopt them elsewhere if they seem to be working well.

That is why I started with provider-kubernetes for this proposal, as I understand some of these ideas may need solid proof based on real usage data from community before it can go into upstream. And, I found provider-kubernetes is a great proving ground to accomodate such things. By enhancing it with proposed features, we can experiment and gain solid feedback from real projects. Ultimately, maybe some of them will still remain in this provider and others will be lifted up. A potential great example of how the project can be driven by the community w/ real cases!

@turkenh Thanks for your detailed review. :+1:

really great to see that provider-kubernetes enabling such interesting use cases.

I would be really happy to see when people need something that hasn't been supported in Crossplane, we can recommend them to use provider-kubernetes. I would expect that provider-kubernetes will become a very popular provider given its uniqueness among other providers :)
Actually, I found a very interesting role of provider-kubernetes where we could take it as "extension" of Crossplane core and a great place to try some features before it gathers sufficient usage feedback from community and can go into core.

One thing I noticed in your demo is (which seems to have affected the design), you're giving cluster-admin permissions to provider-kubernetes which is not the standard way of using a Crossplane provider.

I did not remember if I added any special permission to provider-kubernetes in order to make the demo work. To double confirm that, this is what I grabbed from a demo env, the rules section for the ClusterRole for provider-kubernetes. Did I miss something here?
    "rules": [
        {
            "apiGroups": [
                "kubernetes.crossplane.io"
            ],
            "resources": [
                "objects",
                "objects/status",
                "providerconfigs",
                "providerconfigs/status",
                "providerconfigusages",
                "providerconfigusages/status"
            ],
            "verbs": [
                "get",
                "list",
                "watch",
                "update",
                "patch",
                "create"
            ]
        },
        {
            "apiGroups": [
                "",
                "coordination.k8s.io"
            ],
            "resources": [
                "secrets",
                "configmaps",
                "events",
                "leases"
            ],
            "verbs": [
                "*"
            ]
        }
    ]
}

I would be really happy to see when people need something that hasn't been supported in Crossplane, we can recommend them to use provider-kubernetes. I would expect that provider-kubernetes will become a very popular provider given its uniqueness among other providers :)
Actually, I found a very interesting role of provider-kubernetes where we could take it as "extension" of Crossplane core and a great place to try some features before it gathers sufficient usage feedback from community and can go into core.

Good points, yeah let's see. Thanks a lot for kicking this off!

I did not remember if I added any special permission to provider-kubernetes in order to make the demo work. To double confirm that, this is what I grabbed from a demo env, the rules section for the ClusterRole for provider-kubernetes. Did I miss something here?

Oh. I mean by giving the kubeconfig of the cluster as a provider secret, you effectively gave admin perms.

@turkenh Sorry for the late response! I've updated the doc according to your review comments. Now it looks pretty much better than ever :-) Thanks for the comments especially the suggested policy names. :+1:


stable_12Dec2018

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Unsatisfiable dependencies for platform linux-64: {'toolchain_c_linux-64', 'toolchain_cxx_linux-64', 'toolchain==2.4.0=0'}

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hint: Exit code 137 typically means the process is killed because it was running out of memory
Hint: Check if you can optimize the memory usage in your app
Hint: Max memory usage of this container is 0

Only the tests fail
Traceback (most recent call last):
  File "/opt/conda/bin/conda-build", line 11, in <module>
    sys.exit(main())
  File "/opt/conda/lib/python3.6/site-packages/conda_build/cli/main_build.py", line 456, in main
    execute(sys.argv[1:])
  File "/opt/conda/lib/python3.6/site-packages/conda_build/cli/main_build.py", line 447, in execute
    verify=args.verify, variants=args.variants)
  File "/opt/conda/lib/python3.6/site-packages/conda_build/api.py", line 208, in build
    notest=notest, need_source_download=need_source_download, variants=variants)
  File "/opt/conda/lib/python3.6/site-packages/conda_build/build.py", line 2314, in build_tree
    notest=notest,
  File "/opt/conda/lib/python3.6/site-packages/conda_build/build.py", line 1628, in build
    newly_built_packages = bundlers[pkg_type](output_d, m, env, stats)
  File "/opt/conda/lib/python3.6/site-packages/conda_build/build.py", line 967, in bundle_conda
    output['checksums'] = create_info_files(metadata, files, prefix=metadata.config.host_prefix)
  File "/opt/conda/lib/python3.6/site-packages/conda_build/build.py", line 665, in create_info_files
    create_all_test_files(m, test_dir=join(m.config.info_dir, 'test'))
  File "/opt/conda/lib/python3.6/site-packages/conda_build/create_test.py", line 264, in create_all_test_files
    shell_files = create_shell_files(m, test_dir)
  File "/opt/conda/lib/python3.6/site-packages/conda_build/create_test.py", line 84, in create_shell_files
    f.write(cmd)
TypeError: write() argument must be str, not None
Exited with code 1

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Add ENPO domain

ENPO is a french-style Engneering school, it has a Mathematics and Computer Science Department where they teach Computer Science

@senadir http://www.enp-oran.dz does not contain any information on the courses offered by the school and student emails. Could you please give us an official website URL?

Hi, http://www.enp-oran.dz is indeed the official link, you're redirected to the english version of the website, but that version is not translated and therefor empty, you should view the french version and use google translation
this is the computer science department page
http://www.enp-oran.dz/departement-de-mathematiques-et-informatique/
one has to be an official establishment to have a .dz domain name in algeria so this is the official url
since the school is new and there were no demand on emails, no students email exists yet but they will be soon
emails will be @enp-oran.dz as you can see here
http://www.enp-oran.dz/contact-maths/
courses range from OOP and JAVA and networks and C++

@senadir Thank you for the clarification! Pull request merged. Thank you!


integrate component react with markdown

What would you like to share or ask?
I made a component datetime-local in react and added it to a markdown of taipygui, but it doesn't grab the value of the component, what should I do?
taipy_gui:
<|{btn_from_date_destinations}|example.date_input|>
button react:
<input
type="datetime-local"
className="custom-datetime-input css-1weoyhk css-1weoyhk"
onChange={handleChange}
// value={dateTime}
/>
Code of Conduct

[X] I have checked the existing issues.
[X] I am willing to work on this issue (optional)

Have you read this?

yes, but i have a problem, I see the component in markdown but taipy does not take the date I choose

@FabienLelaquais @FredLL-Avaiga Would you be able to help?

We'll need a bit more context.
How is the component declared in the library?
How is it declared in react (props) ?

I am closing the issue since it is covered by the discussion.


API server with exchange rates

Manual historical updates. If exchange rates for requested date are missing (for any reason) then service asks data providers to update these values and return them immediately.
API endpoints

/rate?from=X&to=Y&date=YYYY-MM-DD
/range?from=X&to=Y&start_date=YYYY-MM-DD&end_date=YYYY-MM-DD

@business-factory/pythonistas please review

@Artimi @miso-belica @paetrik please review


pipeline test

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: wangfeichen sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Added CI pipeline with OpenVINO.

Environment: https://github.com/opencv-infrastructure/opencv-gha-dockerfile/pull/19

Blocker: https://github.com/opencv/opencv/issues/24051

[  FAILED  ] 3 tests, listed below:
[  FAILED  ] Test_Darknet_layers.region/0, where GetParam() = NGRAPH/CPU
[  FAILED  ] DNNTestHighLevelAPI.predict/0, where GetParam() = CPU
[  FAILED  ] Test_ONNX_conformance.Layer_Test/test_dropout_default_ratio_NGRAPH_CPU, where GetParam() = (test_dropout_default_ratio, NGRAPH/CPU)

@asmorkalov, remaining problem is about missed age-gender-recognition-retail-0013.xml file:
C++ exception with description "OpenCV(4.8.0-dev) /home/openvino/opencv/modules/ts/src/ts.cpp:1071: error: (-2:Unspecified error) OpenCV tests: Can't find required data file: intel/age-gender-recognition-retail-0013/FP32/age-gender-recognition-retail-0013.xml in function 'findData'

Yes, @TolyaTalamanov works on it.

There are two options: make file optional or download to CI system. Seems like it's just missed:
https://github.com/opencv/opencv/blob/0245c0cd10387f2de7c7bd076585a285d977e53c/modules/dnn/test/test_ie_models.cpp#L104-L106

Any progress here? Just for reference, direct links are https://storage.openvinotoolkit.org/repositories/open_model_zoo/2023.0/models_bin/1/age-gender-recognition-retail-0013/FP32/. I can add them as a part of https://github.com/opencv/opencv_extra/pull/1080.

@dkurt you are welcome. @TolyaTalamanov promised to do it, but he is on vacation now.

@asmorkalov, please re-run action to check fixes on latest 4.x branch.

@asmorkalov, reminder. Now tests expect to pass. I made some changes to 4.x branch.

@asmorkalov, checks passed :tada:

@dkurt @mshabunin please review and approve.


Support generating multiple files

Feature description
Add support for generating multiple files.
For example, separate files for the refit interfaces, contracts, and dependency injection setup.
Discussed in https://github.com/christianhelle/refitter/discussions/385

This will be a great feature, because clean architecture enforce you to separate external service implementation in infrastructure layer and their models in domain layer. similiar approach is used by ef core scaffold command, maybe we can take this as an example for our approach.

Scaffold-DbContext "Data Source=.;User ID=SA;Password=Passw0rd;Database=Sample;Connect Timeout=30;Trust Server Certificate=True;Application Intent=ReadWrite;Multi Subnet Failover=False" Microsoft.EntityFrameworkCore.SqlServer -OutputDir Entities -Force -NoOnConfiguring -Context SampleDbContext -ContextDir ..\Sample.Infrastructure\Persistence\ -ContextNamespace Sample.Infrastructure.Persistence -Project Sample.Domain

I think we could use --contracts-output, --contracts-namespace to generate contracts and the contracts namespace should be added to generated client file as a using statement

@berkslv Sorry for the late reply, it took longer than expected to get out of vacation mode 😄

I think we could use --contracts-output, --contracts-namespace to generate contracts and the contracts namespace should be added to generated client file as a using statement.
For now, --contract-only option can be used as refitter ./openapi.json --output ./GeneratedContracts.cs --contract-only. But this approach require you to execute two seperate refitter command to generate clients and generate contracts

I can see the value in splitting the contracts and interfaces into multiple files in multiple locations on disk across multiple .NET projects when the system with these files also "owns" the contracts. I wouldn't care too much about splitting files or types across multiple .NET projects when the system that generates them uses another system's OpenAPI specifications as that would be the equivalent of adding a package reference to that other system's client library (like an SDK)
That said, it's not technically hard to implement, so I'll think about it. I have already made quite some progress with splitting the generated code into multiple files. Once I get those changes in maybe I'll re-visit the idea of having options to specify the output file/location for each generated file


1.9

This question requires that you evaluate the underlined text to determine if it is correct.
After you create a virtual machine, you need to modify the network security group (NSG) to allow
connections from TCP port 8080.
Instructions: Review the underlined text. If it makes the statement correct, select "No change is
needed". If the statement is incorrect, select the answer choice that makes the statement correct.
A. No change is needed
B. virtual network gateway
C. virtual network
D. route table

D

A*


Opening exported .csvs in excel displays them as read-only

The files can be opened in Google sheets, but only after google makes a copy of the file first.

Not all CSVs get exported as read-only.

I can't reproduce this. @rahulbot @kanarinka are you two running into this problem?


Craftsim does not include Fresh Fillet in the crafting costs of cooking recipes.

Craftsim does not include Fresh Fillet in the crafting costs of cooking recipes.

thats the concentration value
I just checked, it does actually
recent change also removed the concentration values if concentration toggled off


New command: sign-object

go run ./cmd/sign-object --help

It has a different take on how to generate the signature from what we
currently have.
The current solution uses github.com/itchyny/gojq (a Go-library that
implements jq). I've found that operations like we're performing are
very difficult with jq. The queries are brittle and hard to read (even
harder to modify).
I started looking to see if we could use Kustomize to patch in a
signature. However, Kustomize isn't flexible enough (this is a long
story).
So for this new bit of functionality, I've gone to the library that
underlies Kustomize, jsonpatch (github.com/evanphx/json-patch). I feel
like this is more resilient, though it's definitely more lines of
code. I think this will be more maintainable and composable in the long
run. Those familiar with Kustomize should have an easier time wrapping
their heads around how this code works. Also, if, in the future, the
limitations that prevent us from using Kustomize directly are removed,
we might be able to simply do that, which would simplify a chunk of
this.
I've not functionally modified the existing signature package, though I
did perform a few tweaks that were either sanitary (extracting
constants, etc) or just made the existing code a little easier to
read/use (moving from [32]byte to []byte).
If this approach is deemed worthwhile, I would like to expand to using
its approach in the signature package.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Here's an example of it in action:
$ sign-object -s pkg/signature/testdata/private.ec.key api/testdata/bundle_one.yaml
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: PackageBundle
metadata:
  annotations:
    eksa.aws.com/signature: MEUCIDBLkkCKmlXfEe/Eidzfz8d+dWIlwrqrInJFcioXxwixAiEAsIhv9eT+2TynhKcOVGa3xhXaNJKbFI3q71Vr6pK0uIw=
  name: v1-21-1001
  namespace: eksa-packages
spec:
  kubeVersion: v1.21
  packages:
  - name: Test
    source:
      registry: public.ecr.aws/l0g8r8j6
      repository: eks-anywhere-test
      versions:
      - digest: sha256:64ea03b119d2421f9206252ff4af4bf7cdc2823c343420763e0e6fc20bf03b68
        name: v0.1.1-8b3810e1514b7432e032794842425accc837757a-helm
  - name: Flux
    source:
      registry: public.ecr.aws/j0a1m4z9
      repository: fluxcd/flux2
      versions:
      - digest: sha256:5fb61d0b650483851ce3892e30f1d2c4c1372aafd51be24f37bbb0056db04012
        name: v0.23.0-0b585d5986a1cfcee67b7a956eeef687f00ae468-helm
  - name: Harbor
    source:
      registry: public.ecr.aws/y8n6a2y0
      repository: harbor/harbor-helm
      versions:
      - digest: sha256:423eb28b0586376f4d1d30b492370a4e215f2b0210a587fff6f358efc41dda4c
        name: v2.4.1-f47374093e8a9c48aca8c7a7f06ae185eb7506f3-helm

You can easily sign any k8s object that we've defined in api/v1alpha1:
$ sign-object -s pkg/signature/testdata/private.ec.key api/testdata/harbor.yaml
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: Package
metadata:
  annotations:
    eksa.aws.com/signature: MEUCIQDKQdyFX7iamRzEuZ/ZmRFbEfbCh6cvfBE+x7BRPuGv5QIgOtUZt3NYHDBXjShg7sj8HHF4ayQzYOd/cpAbLwHKNco=
  name: my-harbor
  namespace: eksa-packages
spec:
  packageName: Harbor
  packageVersion: v2.4.1-f47374093e8a9c48aca8c7a7f06ae185eb7506f3-helm

$ sign-object -s pkg/signature/testdata/private.ec.key api/testdata/test.yaml
apiVersion: packages.eks.amazonaws.com/v1alpha1
kind: Package
metadata:
  annotations:
    eksa.aws.com/signature: MEUCIBZyfkEyWn+EdnLk55DppT+VwwNnBrtKg3H134JQ+2GyAiEAvbJcQ8xL4dHCvU4I2As5sg66JSDbKPUfIW97uVj5cU0=
  name: my-test
  namespace: eksa-packages
spec:
  config: |
    config:
      - foo: windom
      - baz: baz
    secret:
      bar: eolus
    pvc:
      storage: 1Mi
  packageName: Test
  packageVersion: v0.1.0

I like this, I'll change the signing job to use this once it merges.

Would it be possible to add a Makefile with build commands for this so it can be used in the pipelines?

/hold

stale


Added markers show conflicts of the sessions in My Schedule.

Implemented #101
Changes:

Add a conflict tag on a CardView
Add some resources for conflict tags
Refresh contents of RecyclerView when the statuses of sessions are changed

:eyes:

Could you merge or rebase master branch?

All of these reviews were reflected.

Use Enum instead of IntDef
Instantiate SessionsTabFragment with using Override
Remove conflict tags immediately without removing unliked sessions
Changed from spacing_xsmall to spacing_small
Removed unreasonable comments
Merged master-branch and reordered commits

And, I changed how to check whether conflicts happen or not.

:eyes:

Sorry for late review.
Because I think about getRangeOfConflictCandidates() method. I want to think more and review it. So please wait until tomorrow 1pm.

@jmatsu
Hi, I am sorry for the delay in my comments.
Maybe it might sound silly to you, I think it seems not straightforward that we have SessionsTabFragment with a mode - ALL_SESSIONS and MY_SCHEDULE.
I feel like we can prepare two Adapters for All Sessions and My Schedule screen respectively because the information shown on the screen is different.
If you have any concerns about that, please let me know.  And if you don't mind, I will put some changes to your branch to verify how my idea goes well.
Another comment is that I am wondering you might not need to compute the possibly affected items on the list precisely. I mean, there wouldn't be so many items (right? It is just a schedule for a user) and we can just render all the items instead of introducing a complicated computation just for simplicity.
Hope it will be of your help. Thanks.

Most ideally SessionsTabFragment should be separated into 2 fragments for each interests, but it's going to be a huge modification so I think separating adapter is the way to go.
Along with this, I think it's great to have Model for those 2 interests(all sessions, my schedules) so that we can easily decouple and test business logics.

Thank you for your reviews! I agree with your ideas.
I will try to handle two current roles (all sessions and my schedule) by using each adapter.
@hkurokawa
You are right. There is no problem with rendering all items in RecyclerView.
In this case, recomputing is needless. So, I will reuse the computation results.
I think that it is better than the current logic and rendering all items.

@jmatsu Thank you for your clarification and appreciate your effective support.
Just to confirm, could you let me know what "Rerendering all items causes misaligning from the current displayed position"? I guess you are saying about the current position is reset to another place when the user removes a session from My Schedule. Is my understanding correct?

@hkurokawa I didn't mean that. Sorry.
The meaning of the current displayed position is the position of the first visible item and/or the offset from the top of the display.
If any conflict tags are added/removed, the current displayed position will be changed. This is expected and correct behavior.
However, even though any conflict tags are not added/removed, it sometimes happens that RecyclerView fails to keep the current displayed position.
Note
Don't disappear any session for the usability if it is unliked. cf. https://github.com/konifar/droidkaigi2016/pull/180#discussion_r52826572

@jmatsu Thank you for your clarification. I got it and agree with you on avoiding the current displayed position is changed even when any conflict tags are not added/removed. We should avoid such weird behaviour.
Let me know when you need my review.
Again, really appreciate your efficient works.  Thank you.

I have one question.
First, the states in master-branch are followings:

The current adapter for sessions is an inner class of SessionsTabFragment.
SessionsFragment and MyScheduleFragment must handle the adapter via SessionsTabFragment.

Which way do you think is better? :smile:
One way is using Enum. Its changes would not be huge.
Create new adapter in current SessionsTabFragment, then instantiate an appropriate adapter based on given Enum
Another way is not using Enum but its changes would be huge.
Create new fragment class (SessionsTabFragment for My Schedule) and declare new adapter in its scope.

My preference is the latter - creating another Fragment for My Schedule. I would just like to avoid switching the behaviours on the mode as it would introduce a bug and some amount of cost to maintain in future.
@konifar How do you think? I would like to respect the repository owner's preference.

@jmatsu Hi, do you have any estimation on how it will take? We are just considering whether we can include this in the next release.

@hkurokawa It won't be long. I'm going to prepare it soon.

@jmatsu Thanks!

Now pushed with rebasing.

:eyes:

@jmatsu Seems good to me. Appreciate your efficient support. I added a comment on an English label. Can you update it?

@konifar @hkurokawa @hotchemi
Updated it. And I also updated an Arabic label with using google translation.

@jmatsu Thanks! I am merging it.

Thank you so much for the polite reviews! :+1:


x/tools/gopls: space in package path leads to invalid hover markdown

gopls version
$ gopls -v version
Build info
----------
golang.org/x/tools/gopls v0.16.0-pre.1
<EMAIL_ADDRESS>h1:EAWgsepx4FJqIUuDtZ+wSeHUQ/oIqYwvNag1DjE50b4=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:5+9lSbEzPSdWkH32vYPBwEpX8KwDbM52Ud9xBUvNlb0=
<EMAIL_ADDRESS>h1:YsImfSBoP9QPYL0xyKJPq0gcaJdG3rInoqxTWbfQu9M=
<EMAIL_ADDRESS>h1:rF+pYz3DAGSQAxAu1CbC7catZg4ebC4UIeIhKxBZvws=
<EMAIL_ADDRESS>h1:3Wt8mZlbFwG8llny+t18kh7AXxyWePFycXMuVdHxnyM=
<EMAIL_ADDRESS>h1:a94ExnEXNtEwYLGJSIUxnWoxoRz/ZcCsV63ROupILh4=
<EMAIL_ADDRESS>h1:2RQpNDoYkPUoN8eTlSyKYJ6pchoM6n6fwsYevP+k4eY=
<EMAIL_ADDRESS>h1:SP0mPeg2PmGCu03V+61EcQiOjmpri2XijexKdzv8Z1I=
<EMAIL_ADDRESS>h1:9MDAWxMoSnB6QoSqiVr7P5mtkT9pOc1kSxchzPCnqJs=
<EMAIL_ADDRESS>h1:G3QvahNDmpD+Aek/bNOLrFR2XC6ZAdo62dZu65gmwGo=
<EMAIL_ADDRESS>h1:lyBNOm8Wo71UknhUs4QTFUNNMyxy2JEIaKKo0RWOh+8=
go: go1.22.4

go env
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\user\AppData\Local\go-build
set GOENV=C:\Users\user\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\user\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\user\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLCHAIN=auto
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.22.4
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=0
set GOMOD=C:\h0llyw00dz\unnammed-project\go.mod
set GOWORK=
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=C:\Users\user\AppData\Local\Temp\go-build2907263936=/tmp/go-build -gno-record-gcc-switches

What did you do?
Reference documentation is broken

What did you see happen?
Today
What did you expect to see?
The link to the reference documentation is broken. It should be:
builtin.IntegerType on pkg.go.dev
Editor and settings
No response
Logs
No response

Similar Issues

x/tools/gopls: references doesn't work properly. #60676
x/tools/gopls:  #52425
x/tools/gopls: missing semantic tokens for function parameters in block #56257
x/tools/gopls: Error for version 0.14.0 with "gopls version" command #63803
x/tools/gopls: <DESCRIBE THE PROBLEM> #67966

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I did not test this on windows but I was unable to reproduce it on Linux. Perhaps the space in the url is causing the markdown to not be rendered. If so seems to be a vscode-go issue.
Either way, these pkg.go.dev urls mixed with the local Go toolchain path will result in a 404.
cc @hyangah

I did not test this on windows but I was unable to reproduce it on Linux. Perhaps the space in the url is causing the markdown to not be rendered. If so seems to be a vscode-go issue.
Either way, these pkg.go.dev urls mixed with the local Go toolchain path will result in a 404.
Edit: I was wrong, gopls formats the markdown links https://github.com/golang/tools/blob/019da39/gopls/internal/golang/hover.go#L1156. We should probably escape urls here.
cc @hyangah

Yes, the other one is correct.

this broken

Thanks for trying the prerelease, and for the report. We will investigate.
Assigning to @adonovan, who has been working in this area.

As @mauri870 points out, the problem is indeed that if the package path has a space in it, the markdown is invalid. It's trivial to reproduce this by creating a package in a directory with a pathological  name such as foo bar. The bug report shows a more realistic case where the pathological path segment is the operating system's own Program Files directory, which leaks into package paths when gopls synthesizes command-line-arguments packages.
Still thinking about a fix.

One possibility is to emit angle brackets around the link. Compare these two, while looking at the markdown source:

angle brackets
[no angle brackets](https://a b/)

But that requires adjusting every place in gopls that puts package paths in markdown links, which is potentially a lot; and it's an unfamiliar syntax. It also seems like a hard invariant to maintain.
Another approach is to URL-encode the URLs, since spaces aren't really safe. But it also seems like a hard invariant to maintain since we're so used to not thinking about URL encoding.
A third approach is to choose paths that don't contain spaces when synthesizing packages. But that only solves the Program Files problem; genuine (pathological) package paths such as my foo bar example may contain spaces.
Option 2 seems best to me.

Thanks. This isn't a regression then, moving out of the v0.16.0 milestone (that should have been obvious to me -- sorry for missing it in my haste).

So, to summarize, there are two bugs here:

gopls is not properly url-escaping package paths.
gopls is not providing accurate hover information for builtins, when viewing the builtin file.

(1) should be fixed by a relatively straighforward application of url escaping. Assigning to our new team member @h9jiang as this is a good starter exercise for bug fixing and writing a test.
On the other hand, (2) might require a more sweeping refactoring of how we handle the builtin file and unsafe packages, and is just about the polar opposite of a good starter bug: requires familiarity with nitty-gritty details of the codebase. I will do this part.

Using this issue to track the broken builtin hover, and opened #70226 for the general problem of broken links.


Don't show 'searching for NAME...' with -m

fixes: #112
Signed-off-by: Olivier Mehani<EMAIL_ADDRESS>

As per the discussion in #112, this is a corner case due to my configuration.


How to improve bist model accuracy? :Bistmodel

I'm using spacybistparser with my own spacy and bist model trained using universal dependencies v2.0 dataset. My training follows the doc. I want to improve the accuracy of bistmodel. Currently, with default parameters, I train the bist model for 20iterations. The evaluation uas score is 87.8 percent. How can I improve it to 93.8 as mentioned in the docs? Is it possible with this english ud 2.0 dataset? Please help.

Any answer?

@danielkorat, do you might have an answer?

@peteriz, thank you for requesting @danielkorat. This is very important to my project. I would like little lesser errors in parsing job descriptions. I guess improving the accuracy would result in fewer parsing errors. Please clarify this one too. I can always give it a try after improving the accuracy.

Hi @ravishpankar, sorry for the delayed response.
93.8 UAS is the score reported in the original paper and original code implementation. As mentioned there, this result is reported on the Penn TreeBank Dataset (Stanford Dependencies).
Training on a different dataset might yield different results.

Hi @danielkorat, Thanks a lot for answering. I understand that universal dependencies 2.0 en dataset is different from Penn tree bank dataset which is based upon Stanford dependency relations. Now the question is how did you train the bistmodel with Penn tree bank dataset to create the pretrained model for spacybistparser. Is it with the default parameters or something else? Any help will be greatly appreciated. Thank you.

Hi @ravishpankar
From what I recall, I used the default parameters.


UWP: logging a new tree point doesn't persist Species related record information

When capturing a new tree point, I've run into several cases where selecting a related tree species does not save this information so that it's viewable outside of the edit session.
The same is also true of ArcGIS Online - the tree record doesn't appear to have a related Species record.
Here's a GIF showing the process of specifying a species, but which isn't available once the edit the saved:

@mikewilburn I've got an idea about what's causing this and the fix likely involves some changes to how we do data binding. Those changes would likely have many conflicts with the open PRs, so I'll wait until all open PRs are closed before tackling this.

@mikewilburn LMK if you have some time on Monday to verify that the species behavior is indeed fixed by #58.

I tried to vet this pretty thoroughly and am no longer seeing any instances - either when adding a new tree point, or editing an existing one - where providing related Species information isn't being persisted outside of the edit session.
Appears fixed to me!


⚠ PrivateHD has degraded performance

In 3d419a6, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3386 ms

Resolved: PrivateHD performance has improved in 1d50bdd.


rename with-segment to with-open

release 0.6.0

Codecov Report

Merging #10 (3ad80dc) into main (17ef89e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   84.95%   84.95%
=======================================
  Files           4        4
  Lines         113      113
  Branches       12       12
=======================================
  Hits           96       96
  Misses          5        5
  Partials       12       12

Impacted Files
Coverage Δ

src/aws_xray_sdk_clj/promise.clj
100.00% <100.00%> (ø)

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 17ef89e...3ad80dc. Read the comment docs.


TextField does not scroll when selecting text

Steps to Reproduce

Execute the code sample
Enter a large text to the text field (e.g. copy-paste code sample to it)
Try to select the entire text

Expected results:
Area inside TextField should scroll when mouse rich to the edge
Actual results:
Area inside TextField is not scrolling

Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const EditorPage(title: 'Editor Demo'),
    );
  }
}

class EditorPage extends StatefulWidget {
  const EditorPage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<EditorPage> createState() => _EditorPageState();
}

class _EditorPageState extends State<EditorPage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text(widget.title)),
      body: SingleChildScrollView(
        child: Column(
          children: const [
            TextField(
              style: TextStyle(fontSize: 24),
              decoration: InputDecoration(
                border: InputBorder.none,
                hintText: 'Please enter title',
                contentPadding: EdgeInsets.symmetric(horizontal: 16),
              ),
            ),
            TextField(
              maxLines: null,
              style: TextStyle(fontSize: 18, fontWeight: FontWeight.normal),
              decoration: InputDecoration(
                border: InputBorder.none,
                hintText: 'Please enter content',
                isCollapsed: true,
                contentPadding: EdgeInsets.symmetric(horizontal: 16),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Logs

flutter doctor -v

[✓] Flutter (Channel beta, 2.13.0-0.2.pre, on macOS 12.3.1 21E258 darwin-arm, locale zh-Hans-CN)
    • Flutter version 2.13.0-0.2.pre at /Users/feimeng/develop/flutter/flutter-beta
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8662e22bac (5 days ago), 2022-04-20 08:21:52 -0700
    • Engine revision 24a02fa5ee
    • Dart version 2.17.0 (build 2.17.0-266.5.beta)
    • DevTools version 2.12.2
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/feimeng/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • 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.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] 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)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • 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

[✓] VS Code (version 1.66.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.3.1 21E258 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 100.0.4896.88

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

https://user-images.githubusercontent.com/12608693/165046328-8d311910-9c53-4062-ac4f-12952d87ad37.mp4

Hi @feimenggo, Thanks for filing the issue. I am able to reproduce the issue on stable and the master channel. The scrolling is buggy on the stable channel, While on the master channel the textSelection is lost when the text goes out of the viewport.
Stable
https://user-images.githubusercontent.com/31410839/165075138-0082405e-6667-4f94-9a42-9ed276dc0373.mov
master
https://user-images.githubusercontent.com/31410839/165075146-6f62fdd9-e3cf-4218-ba41-cf1e3135d729.mov
This issue is mainly on the Desktop, On mobile platforms the scrolling however works fine.
This looks like a related PR https://github.com/flutter/flutter/issues/32344 probably intended for mobile devices.

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 2.10.5, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 2.10.5 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5464c5bac7 (12 hours ago), 2022-04-18 09:55:37 -0700
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • 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.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] 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)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.65.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.39.20220405

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:55555                    • android-arm64  • Android 11 (API 30)
    • iPhone 12 Pro (mobile) • 535317A4-921C-488F-B234-C2CC6D4AE5A1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 100.0.4896.88

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 2.13.0-0.0.pre.712, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 2.13.0-0.0.pre.712 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8c22f5648a (77 minutes ago), 2022-04-25 05:34:07 -0400
    • Engine revision 10efa195bf
    • Dart version 2.18.0 (build 2.18.0-44.0.dev)
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • 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.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that
        responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] 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)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.66.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.39.20220405

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5533 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

cc: @LongCatIsLooong

cc: @LongCatIsLooong

Mobile has the same problem.

CC @justinmc

The thing that causes this is that it's not the TextField that's scrolling, it's the SingleChildScrollView.  You may be able to work around this by changing your widget tree so the TextField is a fixed height and fits on screen.
The real problem is that the selection position should include that scrolling in its calculation.  I'll take a look.


systemd fails to start innernet-sever

I am unable to get the innernet-server systemd service to successfully start. I have followed the install guide on Ubuntu 18.04.5 LTS, 20.04, Debian 10, and Kali Linux, and each time I get the same log messages:
Mar 30 18:20:30 ubuntu-test systemd[1]: Started innernet server for sprawl.
Mar 30 18:20:30 ubuntu-test innernet-server[2493]:  INFO  innernet_server > bringing up interface.
Mar 30 18:20:30 ubuntu-test innernet-server[2493]: Error: Os { code: 2, kind: NotFound, message: "No such file or directory" }
Mar 30 18:20:30 ubuntu-test systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
Mar 30 18:20:30 ubuntu-test systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Mar 30 18:20:30 ubuntu-test systemd[1]<EMAIL_ADDRESS>Scheduled restart job, restart counter is at 4.

It tries to restart a few times then fails. I am using the latest 1.0.0 release .deb.

Fixing this required running modprobe wireguard.


android: update engine to take context

Signed-off-by: Alan Chiu<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: android: update engine to take context
Risk Level: low
Testing: ci
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

I sort of vaguely recall that c-ares wanted Application, not Context. Is that incorrect?

@goaway I believe we went with Application originally because of the lifecycle handlers registration via that object. c-ares just needs the context because in our load method we pass the base context object down rather than the application


Add authentication for the protected routes

Description

Add auth for protected routes
Add logout feature
Add custom page after login

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


"Șinca Veche", "Bucium", "Ohaba", "Perșani", "Șercăița" and "Vâlcea" villages are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 2
Type "Bucium" in the search bar.

Expected results
The location is listed in the drop-down menu as "Bucium, Brașov(Șinca)"

Step 4
Type "Perșani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Perșani, Brașov(Șinca)"

Step 5
Type "Șercăița" in the search bar.

Expected results
The location is listed in the drop-down menu as "Șercăița, Brașov(Șinca)"

Step 6
Type "Vâlcea" in the search bar.

Expected results
The location is listed in the drop-down menu as "Vâlcea, Brașov(Șinca)"


feat(torii-client): add entity change listener

This would allow us to have in the JS binding, something like this
client.onEntityChange({ model: "Position", keys: [ .. ] }, () => {
  // do something here
})

This is very useful


fix(doc): missing documentation on types

Add missing documentation to struct definitions.
Updated tests.
Resolves  #290

Mind having a look at this @dunnock? Thanks
Btw also noticed that Body type parameters don't have a description (it's set to None). Should they?

If this one and 289 go in, do you think we can have a new release? Would be nice to pick these up without patching the toml to point to github.


SoS hang with concurrent nested workflows

Script:
[C]
print('c')
sleep(10)

[D]
print('d')
sleep(10)

[default]
input: for_each={'k': ['C', 'D']}, concurrent=True

sos_run(k)

Command
$ sos run test -v3 -j5

This is one of the reasons why I said that the current executor is too complicated and might fail in complex cases like this. #899

The problem is that without concurrent=True, the step would wait for a workflow, continue, and wait for a workflow, finish. With concurrent=True, the step is waiting for the completion of two workflows and the completion of a workflow leads to different logic.

Fixed by executing input groups with nested workflow sequentially. Not a very good solution but at least it prevents SoS from hanging.


bug: Fail to start up -  Invalid 'ns_id': 0

Did you check docs and existing issues?

[X] I have read all the plugin docs
[X] I have searched the existing issues
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.5
Operating system/version
MacOS 14.1
Describe the bug
Attemping to run the command Fugit2 yields this error:

Error executing Lua callback: .../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: Vim:Error executing Lua callback: ...vu/.local/share/nvim/lazy/nui.nvim/lua/nui/text/init.lua:73: Invalid 'ns_id': 0
stack traceback:
	[C]: in function 'nvim_buf_set_extmark'
	...vu/.local/share/nvim/lazy/nui.nvim/lua/nui/text/init.lua:73: in function 'highlight'
	...vu/.local/share/nvim/lazy/nui.nvim/lua/nui/line/init.lua:58: in function 'highlight'
	...vu/.local/share/nvim/lazy/nui.nvim/lua/nui/line/init.lua:73: in function 'render'
	...im/lazy/fugit2.nvim/lua/fugit2/view/components/menus.lua:98: in function 'set_text'
	...im/lazy/fugit2.nvim/lua/fugit2/view/components/menus.lua:53: in function 'init'
	.../.local/share/nvim/lazy/nui.nvim/lua/nui/object/init.lua:132: in function 'Confirm'
	...are/nvim/lazy/fugit2.nvim/lua/fugit2/view/git_status.lua:236: in function 'init'
	.../.local/share/nvim/lazy/nui.nvim/lua/nui/object/init.lua:132: in function 'GitStatus'
	...local/share/nvim/lazy/fugit2.nvim/lua/fugit2/view/ui.lua:24: in function 'new_fugit2_status_window'
	...u/.local/share/nvim/lazy/fugit2.nvim/lua/fugit2/init.lua:64: in function <...u/.local/share/nvim/lazy/fugit2.nvim/lua/fugit2/init.lua:61>
	[C]: in function 'cmd'
	.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: in function <.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:16>
stack traceback:
	[C]: in function 'cmd'
	.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:48: in function <.../share/nvim/lazy/lazy.nvim/lua/lazy/core/handler/cmd.lua:16>

Steps To Reproduce

Run Fugit2

Expected Behavior
Successfully launch command

Hi, will inspect this. this works normally on my macbook. Have the same version as your.

Same problem.

Found the problem: require('fugit2').setup{} must be called before running.

@lrangell, or you can add opts = {} to lazy config. This has the same effect as config = true.
I will close this issue. Have a nice day @lrangell


Allow model extension with arbitrary likelihood

Here's my take on this.

Add base Likehood class
Allow users to pass arbitrary likelihood terms as instances of Likelihood
Define Gaussian likelihood and use it as the default
Add StudentT likelihood. We might remove this one and expect the users to extend as they please. Or we can create a pool of likelihoods (i.e. negative binomial).

I'm not sure about the variable naming.
Closes #30

I'm not sure whether this is going to be the final API for this feature, but I'm open to merging it for now as it is a useful feature. The main reason why I'm not sure about this API is that I feel the likelihood should be a part of the model, not a part of the fitting parameters, but we can solve that later


[$500] Enlarged View of avatar/profile picture is of low resolution (probably thumbnail) reported by @sobitneupane

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on any user’s avatar from chat.
Again click on avatar from the Details page.

Expected Result:
The picture should have higher resolution.
Actual Result:
The picture has lower resolution.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.9-0
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @sobitneupane
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1662476149722849
View all open jobs on GitHub

@kavimuru
Screen Recording demonstrating this issue.
https://user-images.githubusercontent.com/25876548/192866133-e1fc655d-7009-495b-aff5-eccbd29878b2.mov

Reproducible with our own Chronos image -

@rushatgabhane Doesn't the app have access to the high res image url as well?

Hmmm, it's possible we have the original, but still will probably not be much larger and stretching it out will make it look bad like Rushat said.

I remember having high resolution image before. It must be a regression.

I remember having high resolution image before. It must be a regression.

I am assuming this changed after we added the Image editor/cropper on profile picture upload?
I am in for closing this one out. What do you think @iwiznia ?

I am honestly not sure. If the image we save is large and we have it, then I agree we can show the large image. On the other hand if we only save the small image, then we should close.

Seems like the consensus here is that this should move forward - that the photo uploaded is high resolution but the one shown when selecting the avatar is not?

Doubled price!

@slafortune , @iwiznia
RCA - Seems the large(possibly original size) images are also present and accessible.
Example -
The avatar URL currently being used - https://d1wpcgnaa73g0y.cloudfront.net/aae19548a867de39bf3a74394fdc376331da8cff_128.jpeg
The original image (can be fetched by removing _128 from the above URL) -
https://d1wpcgnaa73g0y.cloudfront.net/aae19548a867de39bf3a74394fdc376331da8cff.jpeg
I can dig deeper into how to gracefully use the original image, otherwise, I guess removing _128 from the URL shall also work

Another example -
Avatar URL -
https://d1wpcgnaa73g0y.cloudfront.net/f0808018b91d553747aa87529366713e8139f44a_128.jpeg
Original Image URL -
https://d1wpcgnaa73g0y.cloudfront.net/f0808018b91d553747aa87529366713e8139f44a.jpeg

@mmjindal2555 Thanks for the findings. The enlarged image is available if you don't crop with the image editor we have on the App.
Please check one such example.
https://d1wpcgnaa73g0y.cloudfront.net/07224468a845eaae17169b21165b2ae3e073db38_128.jpeg
I don't see backend sending high res image to the API. @iwiznia Are you aware about it? and are we okay to show it for some images like @mmjindal2555's example?

I am not sure of what the next actions should be here? @iwiznia Can we get a confirmation if we have the high resolution image stored in the backend which can be sent to the FE?

Looking at the code, we have the original image, in JPEG format saved. If you look for example at the personalDetails. You'll see that it has 2 properties: avatar and avatarHighResolution. The latter is the original one (or rather looking at the code, I think is the original one in jpeg).

I checked that. The difference between avatarHighResolution and avatar URLs is _128 appended in the URL for low res. But for the edited images the resolution is the same.

What do you mean by "edited images"?

Images cropped from the Image editor before uploading.
For instance, check my avatar. Both of them are same.
avatar: https://d1wpcgnaa73g0y.cloudfront.net/07224468a845eaae17169b21165b2ae3e073db38_128.jpeg
avatarHighResolution : https://d1wpcgnaa73g0y.cloudfront.net/07224468a845eaae17169b21165b2ae3e073db38.jpeg

Ah yes, we don't save the original original 😄

Ah yes, we don't save the original original 😄

Yeah! I guess we don't edit/zoom, it stores a large image.

@iwiznia then are we confirmed that we don't have the high res image?

We have the image you uploaded if you did not edit it and the edited image if you did.

So, what do we want to do here? Close?

We can do that unless we want to handle the subset of the cases?

I don't think it's worth it, given it is probably a very small amount of users. Going to close. Reopen if you disagree.


icon for *.nims  file

*.nims is a nimscript file. It can use a same icon as *.nim file.

Hey @doongjohn . Thanks for the suggestion. I think this would be better handled upstream in the main VSCode extension where we pull icons/mappings from.
I could be mistaken but I think the necessary change would be here. I'd suggest opening a PR upstream adding the .nims extension support or opening an issue in that repo for discussion.
thanks!


Run your first app: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-emulator?authuser=1&continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-2%3Fauthuser%3D1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-emulator#1
In which task and step of the codelab can this issue be found?
in video Run your first app on the Android Emulator code-along At 1:06
(See Screenshots)
Describe the problem
Devise manager is not located in the menu shown on the tutorial on newer versions install. its on the right side of the panel (this took me a while to figure it out)
Screenshots
Tutorial shows:

My fresh install:

Correct location:

Hi @emanvidmaker
Thank you for writing in with your concern.
You have the "New UI (Beta)" setting enabled which is causing this discrepancy.
If you turn off this option, it will be as described in the lesson.
Hope you are enjoying the course!
Regards.