Plugin: add support for [XYZ]MODEM file transfers

refer to the title

I'm not sure how this is related to the terminal. This sounds like a request for adding a commandline application to Windows, not a feature to the terminal.
You might be able to use WSL for this. If not, then someone would probably need to port lszrz to Windows, though this issue tracker is not the place for that feature.

I mean it needs to support sz rz like xshell.

@zadjii-msft
I mean it needs to support sz rz like xshell.

@zadjii-msft
I mean it needs to support sz rz like xshell.

@zadjii-msft windows terminal should support sz/rz protocol and popout dialog box, I can't reopen this issue, please reopen it, it is very useful feature.

@zadjii-msft  I think rz sz is a feature for terminal not shell (e.g. WSL).
rz sz is a program running on server, and also require terminal to support it.
In macOS, iTerm2  implement it instead of bash or zsh.
Please tell us where to request that feature.

@zadjii-msft  I think rz sz is a feature for terminal not shell (e.g. WSL).
rz sz is a program running on server, and also require terminal to support it.
In macOS, iTerm2  implement it instead of bash or zsh.
Please tell us where to request that feature.

So I still don't know why the terminal would need to add support for lszrz directly. Seems to me like the lszrz package should be the one to display the file picker on windows itself, and it's not the responsibility of the terminal to display that picker. I'm not sure why a terminal window needs to get involved in the implementation of a commandline application like this.
Maybe I don't understand how sz rz works? If someone can explain why I'm wrong, I'll be happy to re-open and investigate. But this just sounds like an application that needs to be ported to Windows on it's own, and the terminal doesn't need to ship with itself.

@zadjii-msft
As a user, I don't really know what happens.
If I want to send a file from server to laptop,
I just run sz -be filename on linux (via Terminal, via shell, via ssh). This means I need to install lszrz on server.
And something should let me select where to put it, then receive and save it on my laptop.
Other Terminal do this, so I think Windows Terminal should do it too.
e.g. iTerm2 on macOS,  SecureCRT / XShell on Windows

@zadjii-msft
lszrz don't display the file picker on window.
Instead, people are required to run sz/rz on both sides.
This post show what should people do without using a terminal. And this really hard to do for users.
Terminal should support lszrz (and lszrz is required to be installed on PC).
Terminal is ONLY a UI of lszrz.

@zadjii-msft

Seems to me like the lszrz package should be the one to display the file picker on windows itself

lszrz is backgroud program, it can't dispaly the file picker.  Something(e.g. Terminal) should find a place and run sz/rz command on laptop.

and it's not the responsibility of the terminal to display that picker.

So do you think user should be responsibility to display that picker?
ALL OTHER TERMINALS DO! (out of box or need some configure.)

I'm reopening this one and putting it on the infinite backlog.
@qin-nz: We're not going to do this ourselves, but it is a valuable feature and it would be good to think about it when plugin support is written (#555)
@zadjii-msft: This is a weird/interesting thing. lszrz is a package that implements the [XYZ]MODEM set of protocols, and is typically used for transferring files over serial connections to remote machines. The deal, though, is that that data must either be multiplexed-with or totally replace the tty stream. It's the terminal emulator's job to stop sending user input and waiting for connection output and instead send encoded file data packets and wait for ACKs (or the other way around.)
lszrz only includes the client portions that "send a file to the terminal" or "receive a file from the terminal," and the terminal needs to be complicit.

 Well yea of course I didn't understand that, that's a totally different style of application than anything I've ever heard of before. I'd think there would be a better way of doing this, such that the Terminal application didn't have to be directly involved in this process (maybe using a pty or something) but hey if that's how it works, that's how it works.
Sounds like a perfect extension scenario.

I need it

As https://github.com/microsoft/terminal/issues/1602 referred, Zmodem needs a plugin mechanism to implement.

Hey,
Do we have a plan to have this feature?

This is on the backlog and we think it would be good to revisit when we support plugins. I can’t commit engineering time to it right now :)

This is on the backlog and we think it would be good to revisit when we support plugins. I can’t commit engineering time to it right now :)

terminus has already have this features including plugin-supporting. So for now I will use terminus.

any update？

If there had been an update, you would have received an e-mail.

I need it!

I need it!
If you approve it , please give me a thumb.

Please do not unnecessarily bump this thread. There's a perfectly good button for showing your support for this feature request that doesn't send an email to everyone on the thread:

Please support ZModem. We need it.

So I still don't know why the terminal would need to add support for lszrz directly. Seems to me like the lszrz package should be the one to display the file picker on windows itself, and it's not the responsibility of the terminal to display that picker. I'm not sure why a terminal window needs to get involved in the implementation of a commandline application like this.
Maybe I don't understand how sz rz works? If someone can explain why I'm wrong, I'll be happy to re-open and investigate. But this just sounds like an application that needs to be ported to Windows on it's own, and the terminal doesn't need to ship with itself.

Please pay attention to this detaill:
sz/rz use zmodem to trans files between current terminal connecttion,
Don't care about the connection method (ssh, or telent,  tty, serial Or something can trans terminal sequence)
it need to run sz & rz on each side of the connection.
not like scp create new connect, librzsz only use current connecttion, and it no ability to run sz/rz on each side,
without terminal help it can't be done, not they don't want to do it.
it just show some special char sequence to terminal (or connection layer program), some thing like this:
$ rz | hd
rz waiting to receive.00000000  2a 2a 18 42 30 31 30 30  30 30 30 30 32 33 62 65  |**.B0100000023be|

also there are some cli tool do the sz/rz support:

zssh: a ssh client, (so only work with ssh connection), use special shortcut key to switch to peer side of current connection.
screen: screen manager with VT100/ANSI terminal emulation

but cli tool user experience is not good.

i need this

when i ssh a linux terminal , the sz / rz will failed. already fix it ??

when i ssh a linux terminal , the sz / rz will failed. already fix it ??

not yet,  checked their road map to 2.0. the terminal will not support the sz/rz at least until summer in 2021.
in theirs projects the command rz/sz belong to the thrid party tools xyz-modem. and  they do not have a timeline for when they will support which one third party tool.

i need it

我需要它

I also hope to have this feature.

I'm not sure whether the rs&zs function should belong to the terminal or the ssh client program in your computer?

I'm not sure whether the rs&zs function should belong to the terminal or the ssh client program in your computer?
rz & sz , a command, quickly download and upload。

I'm not sure whether the rs&zs function should belong to the terminal or the ssh client program in your computer?
rz & sz , a command, quickly download and upload。

I know what it is, but should it be a function of the terminal, rather than that of the ssh program?

@permissionx Yea, I was confused about this too originally. From earlier in the thread:

This is a weird/interesting thing. lszrz is a package that implements the [XYZ]MODEM set of protocols, and is typically used for transferring files over serial connections to remote machines. The deal, though, is that that data must either be multiplexed-with or totally replace the tty stream. It's the terminal emulator's job to stop sending user input and waiting for connection output and instead send encoded file data packets and wait for ACKs (or the other way around.)
lszrz only includes the client portions that "send a file to the terminal" or "receive a file from the terminal," and the terminal needs to be complicit.

@ everyone else: Please stop bumping the thread with "I need this". There's a perfectly good button for showing your support for this feature request that doesn't send an email to everyone on the thread:

lrszs is basically ZModem in a terminal.   Here's the use case.
SCP is great.  but rz and sz can be used when inside a nested ssh session.
In secure environments its often the case that one needs to ssh into a bastion/jumphost.   from there, ssh into the next layer and finally to the box you need to work on.   rz and sz can be initiated straight to/from that 3rd layer terminal direct from the users terminal, avoiding the need to scp/ssh/scp/ssh/scp....
aka: no direct access from box A to box D, just a chain of connections from one box to another.
A -> B -> C -> D

This is on the backlog and we think it would be good to revisit when we support plugins. I can’t commit engineering time to it right now :)

terminus has already have these features including plugin-supporting. So for now I will use terminus.

really?Why can't I use it on terminus?

This is on the backlog and we think it would be good to revisit when we support plugins. I can’t commit engineering time to it right now :)

terminus has already have these features including plugin-supporting. So for now I will use terminus.

really?Why can't I use it on terminus?

Any update? This feature will indeed accelerate human civilization development.

Nope. We'll make sure to update this thread when there is. In the meantime, might I recommend the Subscribe button?

That way you'll be notified of any updates to this thread, without needlessly pinging everyone on this thread ☺

I wanna this feature , It is useful for me.

I need it !!! I need it !!! I need it !!!

I need it, plz

rz | hd

How to kill this rz | hd ???

Is there another way to transfer files?

I need it

I developed a ssh command line tool to support sz/rz for Windows Terminal in WSL.
When rz/sz command is found, it will call OpenFileDialog/FolderBrowserDialog through a powershell script, and then pass the file path to rz/sz to finish upload or download.
I hope it helps somehow.
https://github.com/esonic/sshz

I developed a ssh command line tool to support sz/rz for Windows Terminal in WSL.
When rz/sz command is found, it will call OpenFileDialog/FolderBrowserDialog through a powershell script, and then pass the file path to rz/sz to finish upload or download.
I hope it helps somehow.
https://github.com/esonic/sshz

not work in wsl2 with Ubuntu-20.04
[king@pc-home king]$ rz
� waiting to receive.**B0100000023be50

版本	Windows 10 教育版
版本号	21H2
安装日期	2021/11/13
操作系统内部版本	19044.1415
体验	Windows Feature Experience Pack 120.2212.3920.0

Hello, I made a ssh wrapper to support rz and sz. I hope you find it useful.
https://github.com/qingyunha/zssh

Hello, I made a ssh wrapper to support rz and sz. I hope you find it useful.
https://github.com/qingyunha/zssh

hi， How to use？

Hello, I made a ssh wrapper to support rz and sz. I hope you find it useful.
https://github.com/qingyunha/zssh

hi， How to use？

I upload a asciinema to show its usage in the readme.

If u need.
Linux Terminal Use lrzsz
Or just
Using rz/sz to transfer files

三年之后又三年，你知道这三年我怎么过的吗。

@Monzale
two years later:
"五年!五年!!你知道这五年我怎么过的吗?"

Hello, I made a ssh wrapper to support rz and sz. I hope you find it useful.
https://github.com/qingyunha/zssh

hi， How to use？

I upload a asciinema to show its usage in the readme.

hi, how to install it?

If u need. Using rz/sz to transfer files

Security Fortress servers don't seem to work very well when they exist

Yeah, the feature is too useful to transfer my file! Hope to see it soon in WT!

+1

The reason that the request for zmodem support comes for the Terminal application, is because the Zmodem protocol is designed to use a very characteristic character sequence for handshaking between the two ends, which is designed to be easy to recognize for the terminal emulator and start the zmodem client locally on the character stream handled by the terminal app.
It is very useful for situations where you cannot create a direct TCP/IP connection from/to a given network host but you have a serial/telnet connection to the host.

This problem has not been resolved for so long, I think it is not a technical problem, but a management problem, who can come forward to coordinate and resolve this matter?

Whether it's "management" or "technical" isn't really up for debate. This feature request is the perfect combination of (1) not important enough to prioritize ahead of other work and (2) already handled better by other software. WT isn't yet intended to be the best--say--serial terminal. It works for that, sure, but it's missing some important things that would make it the best... such as x/y/zmodem support. But it's also missing more basic things like actually opening a serial port as an endpoint.
This is a popular community request, but it is by no means the most popular one. Once we're through with the more important foundational ones, we'll be better equipped to tackle this one.
For now, I'm going to lock the thread. It's been open for a while and the new replies haven't really cleared up the specific requirements for data exchange protocols here, so it'll serve the community better as an upvote aggregator rather than an open-ended discussion about how we just don't have it yet. 😄

For anyone stuck on using lrzsz ( rz / sz ) in Windows Terminal #1999. You can try trzsz ( trz / tsz ).
It is recommended to use trzsz-ssh ( tssh ):
# Install trzsz-ssh ( tssh ) in Windows Terminal
scoop install tssh / winget install tssh / choco install tssh

# Log in to the remote server using tssh. It works similar to ssh.
tssh remote_server

# Install trzsz-go on the remote server. Please check https://trzsz.github.io/go
yay -S trzsz / sudo apt install trzsz / sudo yum install trzsz / etc...

# Upload files like rz
trz

# Upload directories
trz -r

# Download files like sz
tsz file
tsz /path/to/file

# Download directories
tsz -r dir
tsz -r /path/to/dir


we can add prefix for container

we can add prefix for container like
container-lg , container-md, container-xl
when screen get small the container will be fluid container. it just idea 👍
because sometimes we need the margins  just for big screen .

If I understand correctly then I am against this because this #22008

Punting on this for now.

Any update on this ?? ,, All the best.


[X] Yes
[ ] No

I would like to make this project
I'm a GSSOC22 contributor


Feature: Omnibar

What feature or improvement do you think would benefit Files?
TBA
Requirements

Touch friendly
Proper keyboard navigation #7077 #7086
Responsive design #5513

Files Version
v3.6
Windows Version
Windows 11
Comments
No response

FYI, I made BreadcrumbBarEx but I feel we should create the control from scratch.
https://github.com/0x5bfa/BreadcrumbBarEx You can't customize much.

Whatever works best


Event InsertLeavePre, FocusLostPre is needed.

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
Sometimes we need to save the state of the Insert Mode before leaving it, for example the keyboard layout. Now we have InsertLeave and FocusLost, but once we leave the insert mode, the keyboard layout has already been changed. Currently  no way to save this kind of information.
Describe the solution you'd like
Support Event InsertLeavePre and FocusLostPre which means before leaving insert mode. The event names are in line with the others, BufReadPre, FileReadPre, FilterReadPre, etc.
Note FocusLost is also a must. Some input methods have one global state, so losing the focus (switching to other app) will also change the input method state, so the state also needs to be saved before losing focus.
Describe alternatives you've considered
No other way to achieve this purpose.
Additional context

Not sure why you closed this.  Adding InsertLeavePre is quite simple and it might be useful.


enh(haskell) BinaryLiterals, NumericUnderscores, HexFloatLiterals

Changes
Added support for the following Haskell language extensions:

https://ghc.gitlab.haskell.org/ghc/doc/users_guide/exts/binary_literals.html
https://ghc.gitlab.haskell.org/ghc/doc/users_guide/exts/numeric_underscores.html

It slightly over-approximates. For example, it would highlight 10_. If that's an issue, I'll put more effort in the PR. IMO this isn't really a problem though, as it would be a syntax error anyway.
Edit: sorry for all the force pushes, should be done now :)
Checklist

[X] Added markup tests
[X] Updated the changelog at CHANGES.md

@martijnbastiaan Thanks for all the effort!

@joshgoebel Thanks for all the patience! This is what makes me love OSS :heart:


Databricks

Initial PR includes ReadMe and /lib.  Workbooks to follow.

One thing i just noticed - this is set up as a PR for release branch.  I think our practice is featureBranch -> dev -> release.  We can finish the review here, but probably best to reset this as PR into dev?

Closing to resubmit pull request to Dev


Create answers.html

Adding answers.html as the solution of the exercise.
Microsoft Reviewers: Open in CodeFlow

nice


roadmap and/or production readiness?

I was actually looking to move a Vue 2 project to React since i loved working with SWR in my day job, quick google brought me here. Few questions around the roadmap and production readiness.

what is the plan for Vue 3 support? or is the plan to keep this Vue 2 + @vue/composition-api or a bit longer? I saw there was a "beta" branch but that seems weird since this is still not a 1.0 release
would you add this to a production app in it's current state?

Thanks for all the hard work, looking forward to pulling this into my future projects.

vue 3 support is "beta" mainly because it's a bit of a labor of love to support it. We don't use Vue 3 at Kong yet, but FWIW the test suite runs across both composition-api and Vue 3. So while it is not field tested by us, there is some confidence when we make changes. Support for @vue/composition-api isn't going anywhere anytime soon. The thought behind 0.x for swrv + composition-api is that vercel/swr itself is still <1.x and swrv relies on a beta peerDep (@vue/composition-api). semver is a little hard sometimes... The thought behind the beta was just to have a tag in npm without creating a new package like swrv-vue3 or something. I imagine long-term Vue 3 will be mainstream enough that swrv will have a major release for Vue 3 and composition-api will live on in whatever major release it is in.
With respect to production readiness, we (Kong) are actively building several applications with @vue/composition-api/swrv (composition api is still in 1.x-beta). Therefore, we're pretty committed to this library being battle tested alongside all our applications.

Disclaimer: this is open source project, so "production readiness" is maybe subjective here since there is no (legally binding) obligation to fix bugs with any SLA. That being said, hopefully this helps you decide to use swrv, and I am personally proud to support it.

Hey Darren, thanks for the response, this is what i wanted to hear.

The thought behind the beta was just to have a tag in npm without creating a new package like swrv-vue3 or something

Perfect, this is what i figured but wanted clarification

we (Kong) are actively building several applications with @vue/composition-api/swrv

This is what i was looking for, this paired with my tests should be enough "piece of mind" i need.
All in all this is a perfect fit for what i was looking for, and makes it easy for me to upgrade to vue3 when i'm ready to deal with that. Thanks again for taking the time to write this up, looking forward to playing around with this.
Cheers!


Added hasIndex and HasField functions

That's possible using something like:
// check whether f1 exists and is an int field
var f1IsIntField bool
if index.HasField("f1") {
  f1 := index.Field("f1")
  if f1.Opt().Type() == OptFieldTypeInt {
    f1IsIntField = true
  }
}

Yeah. What I sort of want, but don't see an easy way to do client-side, is check whether a field matches a given set of field options. This is hard to do client-side, because so far as I can tell, the actual default values and combinations and error-checking is done server-side. But I didn't realize that we had the .Opt() thing -- I should probably read the code more carefully. So that looks fine to me for now.


lmb-949: add extra check for serial part of rrn

Description
We need to check if the serial part of rrn is between 001 and 998
How to test

Fill in this rrn with serial part 000 and it should not be a valid rrn. 06.02.02-000.80
When running the test suit of with flag `-f="RRN helpers" it should pass instead of fail now

Links to other PR's

https://github.com/lblod/frontend-lokaal-mandatenbeheer/pull/379

We will not be doing this as this will lead us to far and we also do not want to be blocking input from the user e.g. a non belgian RRN.


Update http4s-circe, http4s-ember-client to 0.22.5

Updates

org.http4s:http4s-circe
org.http4s:http4s-ember-client

from 0.22.4 to 0.22.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.22.4).
You might want to review and update them manually.
website/jvm/src/hugo/content/changelog.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch, old-version-remains

I think this is failing on a pre-existing condition, but it's just used in a plugin.  We can get it on the next one.


Add clubstep (low pitched) by Dj nate

Song Name
clubstep (low pitched)
Artist Name
Dj nate
Source
Youtube
Youtube Link
9jH1RvgL54c
Direct File Link
No response
Song ID
13
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 4125776

Hope you like it!


feat(overlay): Add new components and adjust current ones

Description (required)

This change is for next, so don't merge until 4.0!

Docs for https://github.com/withastro/astro/pull/9118
New components were added and some of the current ones were slightly reworked to be more generic.

Hi @Princesseuh Just wanted to check whether you were OK with these changes! I know you have limited internet right now, so I'm going to go ahead and commit these so you don't have to do it. Then you can comment with any changes needed!  :raised_hands:

Comparing this to Matthew's newer PR that (re)names everything!

Closing as these changes were brought in and merged with #5541


[CRASH] FancyMenu Crashes on Controller Configurations

My game crashes when I open the Controller Configurations menu.
Minecraft: 1.20.1
Forge: 47.2.30
CTC: 1.2.4
FancyMenu: 3.2.3

crash-2024-05-22_19.01.21-client.txt

The mod works fine for me, or at least seems to work like it should
Make sure all the dependencies are updated
Also the crash report shows that you have a lot of other mods too, so try to reproduce the crash only with CTC and FancyMenu (and their dependencies)

Do you have anything new on this ? Without any updates from your side, I will have to close this


add the source of the sippets

Hello,
Could you be so kind and add the source of the snippets to your read me file.
https://github.com/Rapptz/cpp-sublime-snippet
Thank you it advance! ;)

done!
Thanks for pointing it out!


termsnap: init at 0.3.0

Description of changes
Added termsnap package which is used to create SVGs from terminal output.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This change is missing an addition to all-packages.nix, which is now discouraged. Please migrate to by-name

This change is missing an addition to all-packages.nix, which is now discouraged. Please migrate to by-name

done!

Thanks for addressing the review feedback so promptly, @Yash-Garg, much appreciated.
A few more things:

Please add yourself to the maintainers/maintainer-list.nix in a separate single commit. For an example see ac189de1e50472903ab82a43880953130b3d500b.
For more details please refer to Commit conventions in CONTRIBUTING.md and the Nixpkgs maintainers README
Squash all existing commits into a single one with the message termsnap: init at 0.3.0
Make sure that the commit adding you to the maintainers list comes before the commit that adds the new termsnap package

Hopefully I've been able to explain well what is needed to follow nixpkgs' best practices, if not please ask me to clarify 🙂

ac189de

Thanks, made the required changes.

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

1 package built:

termsnap

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1840

welcome to nixpkgs!

Nice thorough review and checklist, @pbsds, is this part of your personal review best practices or newer versions of nixpgs-review?

I use this list at times to make sure i don't start glossing over important things.

That's very helpful, thanks, @pbsds 🙂 👍


After applying an operator the boundinx boxes of the inputs are still visible

After applying an operator the boundinx boxes of the inputs are still visible:
https://github.com/user-attachments/assets/2126212d-9a65-4c9f-8d76-d7a8e9e10edc
I expected this not behave that way after putting the bounding box as part of the mesh group in https://github.com/jupytercad/JupyterCAD/pull/521, but it does.
Maybe we're not setting the visible property on the whole group.

Oh wait I guess it is the combination of two issues.

I see the two inputs are still selected on the left panel: the generated output should probably be selected after the operator is done instead
We should not create/show a bounding box for an object that's not visible

thanks for opening this @martinRenou, the explanation helps already and will look more into it


Configure basic github ci

it is not test but dart run --enable-vm-service test/e2e_test.dart

now wait for it to pass


Merge b006c0491082fc12903d1b9eeeedea7282e8b6d6 from upstream

Updates to the latest commit from kubernetes-sigs/apiserver-network-proxy to bump all dependencies to Kubernetes 1.27

/retest-required

/test unit

/retest-required

/close
in favor of https://github.com/openshift/apiserver-network-proxy/pull/35


Adds school_id field

What's this PR do?
Adds a school_id text field to be used by the School Finder component in Phoenix (https://github.com/DoSomething/phoenix-next/pull/1668)
How should this be reviewed?
👀
Relevant Tickets
https://www.pivotaltracker.com/n/projects/2401401/stories/169294661/
Checklist

[ ] Documentation added for changed endpoints.
[ ] Tests added for new features/bug fixes.
[ ] Post a message in #api if this includes something that causes a rebuild!

That's correct! We'll save the Great Schools gsid as the user school_id if they select a school, and save a string like not-available if they are unable to find their school in the dropdown.

Here's a database view:

I've been assuming we should save gs-id, as we've historically done. Universal ID is a new player in the mix, do you think we're ok with sticking with gs-id @mshmsh5000 ?

In the Great Schools data dictionary, gs-id is

Unique within each state; also included in GreatSchools' school/district overview page URL string

whereas universal-id seems to be unique within the GS data set:

ID for data matching between feed products

The full data dictionary is on the Shared Drive: Data -> Vendors -> Great Schools -> 2019-10


Make sure repos are updated before installing valgrind in CI

Valgrind run: https://github.com/quickjs-ng/quickjs/actions/runs/8782108910/job/24095543243 The previous run had failed due to an error installing a dependency because the repos were up to date.


Clearer step and terraform command for SSH Private Key in AWS Config Page.

During Configuring Ops Manager on AWS,  the step on SSH Private Key
Should match what we have in PCF for AWS and SSH Private Key

CC: @outofmem0ry

hey @kroberts7 -- this has been fixed here https://docs.pivotal.io/runtimes/pks/1-2/aws-om-config.html#aws-config


UnboundLocalError: cannot access local variable '_TEMPLATE' where it is not associated with a value

Command to reproduce the error
Error occured at the moment of execution 'nbdev_new' command (https://nbdev.fast.ai/tutorials/tutorial.html#initialise-your-repo-with-nbdev) inside python venv with installed nbdev library.
Detail description
Full error message looks like
Traceback (most recent call last):   File "/home/alex/PythonProjects/nbdev_env/bin/nbdev_new", line 8, in <module>     sys.exit(nbdev_new())              ^^^^^^^^^^^   File "/home/alex/PythonProjects/nbdev_env/lib/python3.12/site-packages/fastcore/script.py", line 119, in _f     return tfunc(**merge(args, args_from_prog(func, xtra)))            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/home/alex/PythonProjects/nbdev_env/lib/python3.12/site-packages/nbdev/cli.py", line 91, in nbdev_new     _TEMPLATE = f'{_ORG_OR_USR}/{_TEMPLATE}'                                  ^^^^^^^^^ UnboundLocalError: cannot access local variable '_TEMPLATE' where it is not associated with a value

OS and Python versions
OS: Ubuntu 24.04 LTS
Python version: Python 3.12.3
nbdev version: nbdev 2.3.19
nbdev library was installed with help of pip:
pip 24.0 from /home/alex/PythonProjects/nbdev_env/lib/python3.12/site-packages/pip (python 3.12)
Possible reason of the error
It is possible that the source of error is the incorrect argument to f - string.
https://github.com/fastai/nbdev/blob/master/nbs/api/13_cli.ipynb, function nbdev_new.
    _TEMPLATE = f'{_ORG_OR_USR}/{_TEMPLATE}'
In this statement variable _TEMPLATE is used before assignment
Probably this is misprint and '_REPOSITORY' should be used instead of '_TEMPLATE'
How to fix locally
nbdev_new - initial command when creating project with nbdev, so it is not possible to use functionality of the library without it.
Steps to fix locally:

Create python venv and activate it.
Install nbdev and jupyter lab according to the tutorial mentioned above.
Get the source https://github.com/fastai/nbdev.git and go to created directory.
Replace '_TEMPLATE' with '_REPOSITORY' inside f'{_ORG_OR_USR}/{_TEMPLATE}' in the function nbdev_new
Execute command 'nbdev_export' to update exported py - files
Uninstall old nbdev library ( pip uninstall nbdev)
Install new nbdev library: pip install .
Go to project directory and execute nbdev_new

Hi @stepetal, would you like to create a PR for this?

Hi!
Thanks. Yes, I would. I will include changes mentioned in issue


Add mask and layer to powercell items

Fix for #562. Not sure why the first line has a diff, but this is pretty simple. Looks like the mask/layer didn't get set, so it was just flinging through walls.


Issue with ReactOnRails

I've followed the steps for installation of Sharetribe from github. I'm running sharetribe in dev environment and I get this error at the browser console when I try to post a new list but the form is not shown.
[SERVER] RENDERED OnboardingTopBar to dom node with id: OnboardingTopBar-react-component-6d1f9179-0460-46bd-897d-10dcbd686226 with railsContext: {"inMailer":false,"i18nLocale":"en","i18nDefaultLocale":"en","href":"http://localhost:5000/en/listings/new?ref=add_listing_link","location":"/en/listings/new?ref=add_listing_link","scheme":"http","host":"localhost","port":5000,"pathname":"/en/listings/new","search":"ref=add_listing_link","httpAcceptLanguage":"es-US,es;q=0.8,es-419;q=0.6,en;q=0.4,en-US;q=0.2,gd;q=0.2","marketplaceId":1,"loggedInUsername":"oscarl","marketplace_color1":"#4a90e2","marketplace_color2":"#2ab865","marketplace_slogan_color":"#ffffff","marketplace_description_color":"#ffffff","serverSide":true}
[Violation] Forced reflow while executing JavaScript took 34ms
styles.scss?11f9:4 Uncaught Error: Cannot find module "!!./../../css-loader/index.js!./../../sass-loader/index.js!./styles.scss"
    at webpackMissingModule (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4335), <anonymous>:4:79)
    at eval (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4335), <anonymous>:4:228)
    at Object.<anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4335)
    at __webpack_require__ (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73)
    at eval (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4329), <anonymous>:19:1)
    at Object.<anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4329)
    at __webpack_require__ (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73)
    at eval (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4269), <anonymous>:37:35)
    at Object.<anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4269)
    at __webpack_require__ (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73)
webpackMissingModule @ styles.scss?11f9:4
(anonymous) @ styles.scss?11f9:4
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4335
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ ManageAvailabilityCalendar.js?0032:8
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4329
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ ManageAvailability.js?b37e:9
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4269
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ ManageAvailabilityContainer.js?0b81:5
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4263
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ ManageAvailabilityApp.js?6853:9
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:4257
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ clientRegistration.js?9c39:9
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:186
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:179
__webpack_require__ @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:73
webpackJsonpCallback @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:44
(anonymous) @ application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:175
new?ref=add_listing_link:240 Uncaught TypeError: Cannot set property 'jsonTranslations' of undefined
    at new?ref=add_listing_link:240
(anonymous) @ new?ref=add_listing_link:240
new?ref=add_listing_link:272 Uncaught ReferenceError: $ is not defined
    at new?ref=add_listing_link:272
(anonymous) @ new?ref=add_listing_link:272
new?ref=add_listing_link:281 Uncaught ReferenceError: $ is not defined
    at new?ref=add_listing_link:281
(anonymous) @ new?ref=add_listing_link:281
new?ref=add_listing_link:288 Uncaught ReferenceError: $ is not defined
    at new?ref=add_listing_link:288
(anonymous) @ new?ref=add_listing_link:288
clientStartup.js?0ac5:155 Uncaught Error: ReactOnRails encountered an error while rendering component: OnboardingTopBar.
Original message: Could not find component registered with name OnboardingTopBar. Registered component names include [  ]. Maybe you forgot to register the component?
    at Object.get (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1525), <anonymous>:65:11)
    at Object.getComponent (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:192), <anonymous>:235:40)
    at render (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1471), <anonymous>:134:47)
    at forEach (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1471), <anonymous>:64:5)
    at forEachComponent (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1471), <anonymous>:76:3)
    at reactOnRailsPageLoaded (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1471), <anonymous>:173:3)
    at HTMLDocument.eval (eval at <anonymous> (application-b48f054646d231d5ccab2d331c67c298f375ce8e6ee67cdbe0c95059c65ccf9c.js:1471), <anonymous>:225:7)

I run the server with this command
nash@Nash-L:~/sharetribe$ foreman start -f Procfile.static
00:38:46 rails.1               | started with pid 28291
00:38:46 rails-client-assets.1 | started with pid 28292
00:38:46 rails-server-assets.1 | started with pid 28294
00:38:46 rails.1               | Waiting for app/assets/webpack/server-bundle.js...
00:38:46 rails.1               | app/assets/webpack/server-bundle.js found
00:38:46 rails-client-assets.1 | Skipping routes export task. Routes file ./client/app/routes/routes.js exists already.
00:38:46 rails-server-assets.1 | Skipping translation export task. Translation bundle ./client/app/i18n/all.js exists already.
00:38:47 rails-server-assets.1 | Waiting for client/app/routes/routes.js...
00:38:47 rails-server-assets.1 | client/app/routes/routes.js found
00:38:53 rails-server-assets.1 |
00:38:53 rails-server-assets.1 | > @ build:dev:server /home/nash/sharetribe/client
00:38:53 rails-client-assets.1 |
00:38:53 rails-server-assets.1 | > npm run ensure-node-version && webpack -w --config webpack.server.config.js
00:38:53 rails-client-assets.1 | > @ build:dev:client /home/nash/sharetribe/client
00:38:53 rails-server-assets.1 |
00:38:53 rails-client-assets.1 | > npm run ensure-node-version && webpack -w --config webpack.client.config.js
00:38:53 rails-client-assets.1 |
00:38:53 rails-client-assets.1 |
00:38:53 rails-client-assets.1 | > @ ensure-node-version /home/nash/sharetribe/client
00:38:53 rails-client-assets.1 | > check-node-version --package
00:38:53 rails-client-assets.1 |
00:38:53 rails-server-assets.1 |
00:38:53 rails-server-assets.1 | > @ ensure-node-version /home/nash/sharetribe/client
00:38:53 rails-server-assets.1 | > check-node-version --package
00:38:53 rails-server-assets.1 |
00:38:54 rails-server-assets.1 | node: v7.8.0
00:38:54 rails-server-assets.1 | npm: v4.2.0
00:38:54 rails-client-assets.1 | node: v7.8.0
00:38:54 rails-client-assets.1 | npm: v4.2.0
00:39:14 rails-client-assets.1 | Webpack dev build for Rails

Also this is what I get after tun npm install
nash@Nash-L:~/sharetribe$ npm install

> undefined postinstall /home/nash/sharetribe
> cd client && npm install

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

I'm using Ubuntu 16.04. If it helps I just let you know that a previous version of sharetribe works well

@NashL I took a peek at https://github.com/sharetribe/sharetribe/blob/master/Gemfile#L90
They are on v6.9.0.
Please create a branch there and try to upgrade to v9, and create a PR and let me know how that goes.

@justin808 do you mean change my node version to 6.9 ?. If I do that I can run the server with foreman start -f Procfile.static because it asks me to run the command with node 7.8. I'm sorry for my inexperience.
I also checked my Gemfile at my clone repository and It has the v6.9.0 of react_on_rails too.

I suggest you open an issue in the shartribe project.

Okay, Thanks

HI,
Have your resolved this issue? I have the same problem and not able to find any solution to this.


Iterating over an opened file should be strings/bytes

with open("something.txt") as f:
    for line in f:
        print(line)

line should be some text type. Iteration just needs to be specialized.

Since open is specialized to return TextIOWrapper, BufferedRandom, BufferedReader or BufferedWriter from io and that's in turn, exists in typeshed as
def __iter__(self) -> Iterator[str]: ...
def __next__(self) -> str: ...

We need to properly invoke __next__ and specialize Iterator in typing.


Unable to use Local DB in Ubuntu

Steps to reproduce

Create a Sample ASPNET core application with LocalDB in windows 10 .
Ensure it works
Make the code to work with kestrel
Now run the code in Unix Ubuntu OS (14.0) the below exception is thrown

The current problem I have is, I use a local SQL server file(.mdf) file for persistence in my app. I have also added the database to the as App_Data. My connection string looks as below{
"ConnectionStrings": {
"DefaultConnection": "Data Source=.\\MSSQLLocalDB;AttachDbFilename=%CONTENTROOTPATH%\\App_Data\\ASPNETCoreToDo.mdf;"
}
}
This works perfectly fine in local. This does not work when pushed to Ubuntu (14.0)
If you are seeing an exception, include the full exceptions details (message and stack trace).

Exception message:
Stack trace:
System.Data.SqlClient.SqlException: A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: TCP Provider, error: 35 - An internal exception was caught) ---> System.AggregateException: One or more errors occurred. (No such device or address) ---> System.Net.Internals.SocketExceptionFactory+ExtendedSocketException: No such device or address          at System.Net.Dns.HostResolutionEndHelper(IAsyncResult asyncResult)          at System.Net.Dns.EndGetHostAddresses(IAsyncResult asyncResult)          at System.Net.Dns.<>c.<GetHostAddressesAsync>b__14_1(IAsyncResult asyncResult)          at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic(IAsyncResult iar, Func2 endFunction, Action1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Data.SqlClient.SNI.SNITCPHandle.d__22.MoveNext()
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at System.Threading.Tasks.Task.Wait(TimeSpan timeout)
at System.Data.SqlClient.SNI.SNITCPHandle..ctor(String serverName, Int32 port, Int64 timerExpire, Object callbackObject, Boolean parallel)
`
Further technical details
EF Core version: 1.0.0
Operating system: windows 10 anniversary ypdate
Visual Studio version: Visual studio 2015
Other details about my project setup:

Localdb is a Windows only product, and only supports connections from the local machine

As @ErikEJ said, LocalDB is Windows only. On Linux, you can use EF Core with a remote SQL Server (running on Windows), the preview version of https://sqlserverforlinux.com, or SQL Azure. Alternatively, you can use the Postgres and SQLite providers.


TxBuilder does not work as documented

https://github.com/bpfkorea/agora/blob/4c7f32d9a84a8ea666bdf362f742a5a931ec058c/source/agora/utils/Test.d#L283
It's documented like this, and then there's a test for the exact opposite behavior:
https://github.com/bpfkorea/agora/blob/4c7f32d9a84a8ea666bdf362f742a5a931ec058c/source/agora/utils/Test.d#L746-L747

Fixed by #1664


Support uploading large files

https://github.com/googledrive/PyDrive/issues/27
https://github.com/googledrive/PyDrive/issues/55
Reading the code for apiclient.http.MediaIoBaseUpload at
https://github.com/google/google-api-python-client/blob/master
/googleapiclient/http.py , it would seem the simply setting
resumable=True should be enough let it select a chunk size for
itself, thus precluding loading the entire file into memory.
Additionally, I just tested this with a 4Gb file, which is way larger than what has failed for me before

Signed

Any news on a time frame for accepting the PR?

Thank you for this !
This project seems abandoned. Someone should fork it !
I have to use this patch to my scripts :
from pydrive.files import GoogleDriveFile
from pydrive.files import MediaIoBaseUpload

def DeleteFile(self):
    try:
        self.auth.service.files().delete(fileId=self['id']).execute()
    except errors.HttpError, error:
        print('An error occurred: %s' % error)
GoogleDriveFile.Delete = DeleteFile

def _BuildMediaBody(self):
    """Build MediaIoBaseUpload to get prepared to upload content of the file.
    Sets mimeType as 'application/octet-stream' if not specified.
    :returns: MediaIoBaseUpload -- instance that will be used to upload content.
    """
    if self.get('mimeType') is None:
      self['mimeType'] = 'application/octet-stream'
    return MediaIoBaseUpload(self.content, self['mimeType'], resumable=True)

GoogleDriveFile._BuildMediaBody = _BuildMediaBody

So sad :-(

@aliafshar : Ping?

Looking into this, will test and merge in the next week.


Fula: choose one consensus library and replace RAFT implementation with it

1- https://github.com/aleph-zero-foundation/consensus-go
2- https://github.com/nspcc-dev/dbft
3- https://github.com/SmartBFT-Go/consensus
4- https://github.com/dedis/cothority/tree/main/byzcoinx
5- tendermint without the blockchain part

@ehsan6sha I've looked into this, got some thoughts:

How are we going to sync nodes that have been offline or freshly onboarded without storing all the blocks/decisions made (basically blockchain)? This is the first thing we need to solve, it would be nice to have an understanding about that.
Once we have solved that - we will be able to choose the consensus.
Wrapping this into lib-p2p-consensus interface is going to be tricky, take a look at this to see how some of the interfaces look.

https://github.com/nspcc-dev/dbft - a protocol that also produces blocks
https://github.com/aleph-zero-foundation/consensus-go - no commits for 1.5+ years
tendermint - block generation is part of the protocol, ABCI interface is based on that and it might be too complicated and chunky.
https://github.com/SmartBFT-Go/consensus/blob/v2/test/test_app.go - an example of how smartBFT is implemented. We still need to maintain the sequence (block number), otherwise there is no way to perform the checks.
https://github.com/dedis/cothority/tree/main/byzcoinx - still needs a research

We are storing all the event list (It is a merkle DAG).

@ehsan6sha I've looked into this, got some thoughts:

How are we going to sync nodes that have been offline or freshly onboarded without storing all the blocks/decisions made (basically blockchain)? This is the first thing we need to solve, it would be nice to have an understanding about that.
Once we have solved that - we will be able to choose the consensus.
Wrapping this into lib-p2p-consensus interface is going to be tricky, take a look at this to see how some of the interfaces look.

https://github.com/nspcc-dev/dbft - a protocol that also produces blocks
https://github.com/aleph-zero-foundation/consensus-go - no commits for 1.5+ years
tendermint - block generation is part of the protocol, ABCI interface is based on that and it might be too complicated and chunky.
https://github.com/SmartBFT-Go/consensus/blob/v2/test/test_app.go - an example of how smartBFT is implemented. We still need to maintain the sequence (block number), otherwise there is no way to perform the checks.
https://github.com/dedis/cothority/tree/main/byzcoinx - still needs a research


Sample Rules Definition JSON has too many curly brackets

In trying to create a credential using the provided Rules Definition json I get an error (rules.vc.type is a required value. Please enter a value and try again.
Error code: missingRequiredParameter)
After entering the code line by line in the provided text box, I noticed the extra bracket (maybe set of brackets)
So copying and pasting did not work as expected

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

ID: e85be67f-8375-6229-18aa-d687463991f9
Version Independent ID: c494773d-9fcc-d733-6ddb-935a8e545abd
Content: Tutorial - Issue Microsoft Entra Verified ID credentials from an application (preview) - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-issuer.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

It is NOT about too many curly brackets, see the correct JSON below
{
"attestations": {
"idTokenHints": [
{
"mapping": [
{
"outputClaim": "firstName",
"required": true,
"inputClaim": "given_name",
"indexed": false
},
{
"outputClaim": "lastName",
"required": true,
"inputClaim": "family_name",
"indexed": false
}
],
"required": false
}
]
},
"validityInterval": 2592000,
"vc": {
"type": [
"VerifiedCredentialExpert"
]
}
}

@matthewjlevy thank you for your feedback. We are making updates to the articles and the JSON provided will get updated as part of those changes. Content should be updated in the next couple of days.

#please-close


refactor: Use Snappier library instead of Snappy.Standard

My organization has started using Snyk to detect vulnerabilities in repositories. It is complaining that Snappy.Standard, an 8 year old project, is using an ancient version of System.Net.Http with vulnerabilities.
This PR replaces the Snappy.Standard library with the more modern Snappier library.

Hi, thank you for the PR! It looks like you've removed array pools, so memory traffic will be high, can you try returning it back for performance purpose? It looks like Snappier does support work with buffers

Nevermind, returned it back myself

Thank you so much!!


Camera Control

Hi,
we´re using BMD ATEMS with BMD Cameras  - the camera control works via Atem and its received via the sdi in on the camera. we are currently using a skaarhoj rcp v2 to control the camera.
what i am looking for here is a way to trigger the button "autofocus " + some of the other features there ( mainly need focus)
having also some volume control over the audio mixer would be great

It is possible to control some audio levels in the 2.1.1 betas.
Camera control is not implemented, and is quite a large task so unlikely to happen soon

Just wanted to +1 on this, would be very keen on having camera control implemented in Companion. I’d love to be able to set a button to trigger a certain gain value or whitebalance value as well as open / close the IRIS in small steps.
If there would be an option to first select a camera as a variable and the have the other actions use that camera number as a variable would be great. Is there a chance this is going to get implemented?

You can use the Middle Control  Companion plugin. It does exactly that, you can control all the Blackmagic camera related settings https://www.middlethings.co/documentation/docs/Usage/bitfocus-companion/

With the new Stream Deck+ having rotary knobs it would awesome to do camera control. I would be willing to contribute to a bounty on this.

@Julusian, does the addition of CameraControlCommand in atem-connection (thanks for that contribution!) unblock enabling basic camera control in this plugin?  Camera control is the last piece that requires me to keep ATEM Software Control open in addition to Companion.  Thanks!

Maybe. Ideally more should be added to atem-connection to help with the intepreting and generating of the CameraControlCommand, so far it is the bare minimum needed to allow someone to do all that logic themselves.
The main problem is that the values used by that command are not really documented. Not even in the official SDK. I suspect that the values documented in https://documents.blackmagicdesign.com/UserManuals/ShieldForArduinoManual.pdf?_v=1581580811000 will be what we should use, but that disagrees with a PR that someone opened for atem-connection, so I don't know which to trust.
I don't have a blackmagic camera to test with, so this is very hard for me to motivate looking at doing myself.
I did do an implementation in another library once, but getting tests to work reliably when using newer ATEM firmware was a pain, a few times they broke with no clear way to know which to expect. Thats a challenge when wanting a stable api and to support as many atem firmware versions as we do.

Sep 2023, Blackmagic released API (local REST) documentation for controlling cameras directly. https://documents.blackmagicdesign.com/DeveloperManuals/BlackmagicCameraControl.pdf
This could be separate from the ATEM module, but this module would still be helpful for controlling cameras outside of the supported list or HDMI/SDI controlled devices through the ATEM.
This is mentioned in this request: https://github.com/bitfocus/companion-module-requests/issues/719

Another use case for the Blackmagic Camera Control would be to control one of the supported cameras connected to the network with an ethernet adapter. The documentation for the camera suggests you can control it directly over the network without an ATEM.

The following Blackmagic cameras are compatible with Camera Control REST API:

Blackmagic Cinema Camera 6K
Blackmagic Micro Studio Camera 4K G2
Blackmagic Studio Camera 4K Plus
Blackmagic Studio Camera 4K Pro
Blackmagic Studio Camera 4K Pro G2
Blackmagic Studio Camera 4K Plus G2
Blackmagic Studio Camera 6K Pro

Downloading API’s from your Camera
You can download REST API YAML documentation from your camera by adding /control/documentation.html to the end of the camera’s Web media manager URL or IP address. For example, https://micro-studio-camera-4k-g2.local/control/documentation.html

@GarthDB lets keep this issue about doing camera control via the atem.
While REST is an option for some cameras, that would be implemented as a new module. I've created https://github.com/bitfocus/companion-module-requests/issues/1383 as a replacement for https://github.com/bitfocus/companion-module-requests/issues/719 to describe the REST api and a potential new module.

A first pass of this is done and will be in the betas shortly.
If anything doesn't work or you think is missing, please open new issues

I would like to test the beta for BMD camera control discussed in this thread.
I'm not that familar with Github, so I'm not sure how to accomplish getting the beta module.
Thanks,


Mouseover event

How to implement mouseover on a day event and to show the tooltip.
Thanks.

Refer to https://github.com/intljusticemission/react-big-calendar/blob/master/examples/demos/rendering.js

I am using the DnD addon. When I try to use the event component, it cannot capture the mouse events underneath the resize handles, the time range line and a few pixels off each side. I tried to use an event wrapper, but wrapping props.children in a div results in a zero-height div stacked at the top of the day column.
I am thinking that maybe instead of wrapping the children in the event wrapper, I could use React.cloneElement and assign the mouse handlers there, that way I should be able to get on rbc-event which is the whole area of the event. I will report back if that worked or not. Seems subpart though.
It would be cool if the event wrapper could be fixed so that is actually wraps the event element area, but the fix is probably not going to be trivial since rbc-event is absolutely positioned so it jumps out of the wrapper happily.

So turns out the above will break completely when actually dragging or resizing the event. I decided to prototype the cloneElement way and it works even when dragging and resizing, but my ultimate goal - to preselect slots on hover for the DnD actions to happen immediately and not after an event click, that I did not meet. I guess the DnD addon keeps its own state about what event is selected and just won;t start dragging/resizing even if the event is already selected without it being clicked once before the action.
function BoundEventWrapperComponent(app: App) {
  return class EventWrapperComponent extends React.Component<EventWrapperProps, never> {
    public render() {
      const stockEventWrapper = React.Children.only(this.props.children) as React.ReactElement;
      const oldEventDiv = React.Children.only(stockEventWrapper.props.children) as React.ReactElement;
      const newEventDiv = React.cloneElement(oldEventDiv, { onMouseOver: this.onEventDivMouseOver });
      return React.cloneElement(stockEventWrapper, { children: newEventDiv });
    }

    private readonly onEventDivMouseOver: React.MouseEventHandler<HTMLDivElement> = _event => {
      app.setState({ selected: this.props.event });
    };
  };
}

sds


Driver interface changes

Cleanup
As mentioned in #224 I believe the Driver interface needs a little bit of cleanup.
Currently it's not really an abstraction, and part of the problem is this also reaches the Queue abstraction layer.
Talking about three methods in particular:

count (also in Queue)
peek (also in Queue)
info

These are never used in the consumer-producer context (which I think is the primary goal of Bernard), I believe they are there for the sake of Juno.

Count: it's the least problematic of all, but still not supported by some drivers. I would either move it to a CountableDriver interface or implement Countable instead and check for those interfaces in Juno.
Peek: Most of the drivers doesn't even support this. I would do the same with peek as with count. I think it makes the Driver interface more stable and other drivers easier to implement.
Info: This is probably the most problematic one, as it returns totally unstructured data. I don't really know what the best solution would be here, but at least some structuring would be nice. Eg. return an array of key value pairs at least.

Other changes
I also suggest the following changes:

Implement rejectMessage as suggested in #134
Implement some kind of RawMessage object which exposes the raw message and the receipt instead of how currently pop returns messages. It's too easy this way to return invalid data.

If you are comfortable with these changes I'm more than happy to do the changes and move Bernard towards a stable version.

I like the idea of moving stuff the a CountableDriverInterface and other specific interfaces so each driver can decided on what it want to support of "extra features". This way we also keep the general interface clean

That's the idea.

I would say go for it! 💪

I've done some research (previous assumptions were purely based on the driver interface and it's implementations): looks like it would be quite hard to make peek and count removed from the interface, but at the same time kept as an optional feature.
I'm not really happy to say this, but without major refactor on the queue level as well removing those methods is not possible.
While I think it would be great if we could remove those things from the interface, I believe this part of the library is mature enough to be ready for a first stable.
As @henrikbjorn said: the driver concept is an internal thing, end users (devs) should not meet with the interface at all.
I will do some more research, but it looks like it would be easier for us (at least for now) to leave them be.
As for the other changes: I still think those should be done.


author information missing

While using observation/show endpoint only authorId is coming authorName and authorImage keys are missing

Has this not been tested with swagger pojo classes?
Prabkakar
On Mon, Oct 21, 2019, 2:27 PM Harsh Zalavadiya<EMAIL_ADDRESS>wrote:

While using observation/show endpoint only authorId is coming authorName
and authorImage keys are missing
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/strandls/biodiv-observation/issues/2?email_source=notifications&email_token=AAAQZGE2VCGW3EM4KXP3DETQPWNZNA5CNFSM4JC4SK72YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HTE5DQQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQZGF3BHMPDY7ZVA2UY4TQPWNZNANCNFSM4JC4SK7Q
.

Fixed on https://github.com/strandls/biodiv-observation/commit/ce08a438a03f7b105d26ac51bf1e2b4f82b497b0


Updated DATASHEET.md with relevant information

Hello @vukosim.
The following needs to be addressed:

I am not sure what information to provide on the distribution section of the Datasheet, I have left that section blank to avoid providing misinformation.
The uses section, questions: 1, 3, and 5. I believe these questions would best be answered by you or anyone else on the team. These are questions on what the dataset will be used for and what is has already been used for.
At the very top, the author, must I have written my name? I left that empty and I put the Data Science for Social Impact Research Group (DSFSI) on the organization, I'm not sure if this is fine.

If there is anything else that you may need, please do communicate and I will avail myself.


Fix incorrect assertion on map test

KeyError is raised instead of an ArgumentError when trying to use
pattern matching to update a map key that doesn't exist.

Thanks!


Java 21 green-thread support might be very useful to save infra

Given we already have the spark-11 commit that supports virtual threads.
This will extremely important into all proxy related calls which are primarily IO heavy.
https://inside.java/2021/05/10/networking-io-with-virtual-threads/

This is the branch that works it out.
https://github.com/nmondal/cowj/tree/vthread
Here is the PR.
https://github.com/nmondal/cowj/pull/79

This is done and doc is updated via this:
https://github.com/nmondal/cowj/commit/e63d3c531a2130bbd6dc1e69c649b14599a9e25d


Node.js 12 actions are deprecated

Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout, actions/checkout

https://github.com/sanity-io/github-action-sanity/tree/v0.4

https://github.com/sanity-io/github-action-sanity/tree/v0.4

What's the reason this isn't what's in master?

Does anyone of a simple example of v4 working?   For install, dependencies, a simple cli Sanity command like a COPY or EXPORT...any relative sanity cli command example will do.
@geball,  what should be considered current stable version?
Cheers!


feat: add Platform NetworkStatus get and set

Add wrappers for the following Platform APIs:

EOS_Platform_GetNetworkStatus
EOS_Platform_SetNetworkStatus

Thanks!


alignmentCheck test fails for netlrts-darwin-x86_64-smp build

Original issue: https://charm.cs.illinois.edu/redmine/issues/670

I could not reproduce this error, but the alignmentCheck test in tests/charm++/alignment/ failed in the nightly autobuild for netlrts-darwin-x86_64-smp:
https://charm.cs.illinois.edu/autobuild/cur/netlrts-darwin-x86_64-smp.txt
To test run: ./charmrun +p2 ./alignmentCheck ++local

Original author: Nikhil Jain
Original date: 2015-02-19 22:24:52

This is being caused by clang doing nasty thing for the following comparison:
startHdr < startEnv
Even with sane values, its inverting the expected output.

Original author: Nikhil Jain
Original date: 2015-02-20 05:45:22

A deep dive into reading assembly code confirms that clang is producing incorrect assembly.

Original date: 2015-02-20 05:50:39

Unexpected, certainly. We need to be much more thorough before we can say that it's wrong to produce it.

Original author: Nikhil Jain
Original date: 2015-02-20 05:52:38

Agreed, but it seems that way. I will see if we get time to briefly discuss this tomorrow.

Original date: 2015-02-20 17:08:19

Happens with Apple's Clang derived from upstream version 3.2. Doesn't happen with MacPorts clang-mp-3.3, and I can't install the clang-3.2 port on my Mac laptop due to version incompatibilities. Also doesn't happen with clang-3.5 or clang-3.6 on other systems. The code also doesn't trigger the Undefined Behavior Sanitizer at all. I'm now much more inclined to agree that this is a wrong-code generation bug in the older compiler.
To address this on our machines, we'll need Xcode upgraded to at least 5.0 from the current 4.6.x (per http://en.wikipedia.org/wiki/Xcode). I'll ask Yanhua to do that on Wit.

Original date: 2015-02-20 20:18:34

My earlier attempts at reproduction were flawed. It happens even on later compiler versions, with either distribution.
We concluded that the pointer arithmetic in question was potentially undefined behavior, and replaced it with integer arithmetic:
http://charm.cs.uiuc.edu/gerrit/#/c/559/


Shadow IPFS server

There should be a "shadow" IPFS server where the only functionality is pinning uploaded content. This server should not be externally accessible.

After all Origin content is moved to the new servers, the former gateway can be repurposed as the shadow server

As automated provisioning is being set up with Ansible, the current thought is to deploy a shadow server(s) with the same configuration as the active servers, but to disable NGINX as to disallow client access. In the event a shadow server has to be activated, the service can be turned on and the server's IP added to DNS rotation.
With pinning, shadow servers could pin on a slower cadence than the active servers. In the event of an issue arising with the set of Origin listing hashes, that issue would not immediately propagate to the shadow servers.


source_id registration IPSL-CM7A-ATM-LR

label = IPSL-CM7A-ATM-LR
label_extended = IPSL-CM7A-ATM-LR
source_id = IPSL-CM7A-ATM-LR
institution_id = IPSL
release_year = 2019
activity_participation = [HighResMip]
aerosol:
description = none
nominal_resolution = none
atmos:
description = DYNAMICO-LMDZ (NPv6; 16000-point icosahedral-hexagonal; 79 levels; top level 40000 m)
nominal_resolution = 250 km
atmosChem:
description = none
nominal_resolution = none
land:
description = ORCHIDEE (v2.2, Water/Carbon/Energy mode)
nominal_resolution = 250 km
landIce:
description = none
nominal_resolution = none
ocean:
description = none
nominal_resolution = none
ocnBgchem:
description = NEMO-PISCES
nominal_resolution = none
seaIce:
description = none
nominal_resolution = none

@SebastienDenvil this model is now registered. Note like with IPSL-CM7A-ATM-HR (#763) I removed the NEMO-PISCES reference for ocnBgchem as you're not planning on running this in an ocean-atmosphere coupled mode. Please take a peek at CMIP6_source_id.html to double check the final registration


Version 0.8.1

Added localization for German and English to front-end
Improved UI widgets
Back-End fixes and improvements

Still some minor UI issues on Report Configuration page (see screen shots):

element justification
different row heights

empty fields on top of weekday and monthly/weekly choice lists

width overflows in German language version

There could be a better (less difficult to understand) reason on wrong time entries than just "406"

The last commits on develop branch should fix the issues mentioned above.

Fixed a typo and 24h time format. Row heights still not perfect, but this is more of a flutter issue -> close.


[DOC] Authors page should reflect more than git commit contributions

Brief Description of Fix
Currently, the docs only link to code contributions that involve a git commit. This does not reflect the contributions made by others to the project, such as talks, helping with sprints, social media, and more.
I would like to propose a change, such that now the docs includes at least code sprint contributions. For example, @jk3587 and @sallyhong helped with issues and guiding others at the last sprint at SciPy 2019, @loganthomas and @jekwatt helped others with git at SciPy and PyCon respectively.
Relevant Context

Link to documentation page: Contributors
File to be edited: https://github.com/ericmjl/pyjanitor/blob/dev/AUTHORS.rst

If the link to File to be edited does not work, try this:
https://github.com/ericmjl/pyjanitor/blob/dev/AUTHORS.rst

How can they be properly recognized on Authors.rst?
Including when and how they contributed and guided others?
A tree graph of guidance?

I think bullet points indicating non-coding contributions would be a way out.

Is this now closed by #491 ?

Yes, indeed, thanks for pointing this out, @hectormz!


running a micc2 command in a non-project directory raises an exception instead of producing an error message

fixed


Account templates for recurrent tasks

Create a template account that can be cloned to an actual or real account linked to an entity, so we can use it as recurrent tasks like monthly payments, etc
Relationship:
Entity -> account(s) template [New|Edit|Delete|Clone] Clone: copy to a new account ready to be added.
Add a modal panel with the account template list with the above options and from that can clone to create a new account.

Project pluto-template created to handle operations by entity. Need to save accounts in this project to not duplicate code, will try to use AWS SQS or G queue


Plaid Integration

Description
Little exploration has been done on this, but the overall goal is to make is so that someone who is using Sungmanito can link their payment accounts with Plaid, and then map any outgoing transactions to payments for bills. This way we could auto mark bills as paid, updating any household members when the notice is received
Concerns
While we want the user who has hooked up the account to have management over that account, we also do not want anyone who does not need transaction access to said account to see it.
Exit Criteria
Stage 1

[ ] POC for Plaid integration + transaction mapping
[ ] POC for account permissions to households

Stage 2

[ ] Closed testing for Plaid features

Stage 3
Assuming everything goes well

[ ] Open Beta

It has occurred to me that I'm going to have look into what exactly SOC2 compliance is because I'm not going to want to deal with that bag of shenanigans should this somehow get hacked in a production environment.


⚠ Bofrost.es has degraded performance

In e235e10, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1397 ms

Resolved: Bofrost.es performance has improved in be16bc1.


[Bug]: Start Ams in IDEA Error

What happened?
I wanted to contribute for amoro, so I tried to run the AMS in IDEA, but encountered an error.
Before starting, I had successfully executed ' mvn clean install -pl '!trino'  -B -ntp  -DskipTests  l' in the command terminal.

Affects Versions
0.5.x branch
What engines are you seeing the problem on?
AMS
How to reproduce
No response
Relevant log output
No response
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Did you select the profile for Hadoop 2 or Hadoop 3?
Alternatively, you can try "Maven reimport".


Revisions to readme clarity

As requested by adbar on January 9, conducted a copy-edit aimed at bringing README.rst into compliance with standard English idiom usage. Also corrected some typographical issues.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   91.59%   91.59%
=======================================
  Files          15       15
  Lines        1654     1654
=======================================
  Hits         1515     1515
  Misses        139      139

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 b05e9fd...bfc279c. Read the comment docs.

Thank you very much!


Add done and undone order

closes https://github.com/AY2122S1-CS2103T-W16-2/tp/issues/87 and https://github.com/AY2122S1-CS2103T-W16-2/tp/issues/88

Codecov Report

Merging #93 (ced07ca) into master (b66fc50) will increase coverage by 0.78%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master      #93      +/-   ##
============================================
+ Coverage     76.09%   76.88%   +0.78%
- Complexity      768      812      +44
============================================
  Files           104      109       +5
  Lines          2138     2267     +129
  Branches        248      262      +14
============================================
+ Hits           1627     1743     +116
- Misses          445      456      +11
- Partials         66       68       +2

Impacted Files
Coverage Δ

...22s1_cs2103t_w16_2/btbb/commons/core/Messages.java
0.00% <ø> (ø)

..._2/btbb/logic/commands/order/FindOrderCommand.java
100.00% <ø> (ø)

.../java/ay2122s1_cs2103t_w16_2/btbb/model/Model.java
100.00% <ø> (ø)

..._cs2103t_w16_2/btbb/model/util/SampleDataUtil.java
0.00% <0.00%> (ø)

...y2122s1_cs2103t_w16_2/btbb/ui/order/OrderCard.java
0.00% <0.00%> (ø)

...s2103t_w16_2/btbb/model/order/UniqueOrderList.java
73.91% <66.66%> (-2.56%)
:arrow_down:

...y2122s1_cs2103t_w16_2/btbb/model/order/IsDone.java
81.25% <81.25%> (ø)

..._w16_2/btbb/logic/descriptors/OrderDescriptor.java
85.13% <83.33%> (+1.07%)
:arrow_up:

..._2/btbb/logic/commands/order/DoneOrderCommand.java
90.00% <90.00%> (ø)

.../btbb/logic/commands/order/UndoneOrderCommand.java
90.00% <90.00%> (ø)

... and 11 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 b66fc50...ced07ca. Read the comment docs.


cmd/keygen: add script for generating high-quality gocloud.dev key URIs

Codecov Report

Merging #94 into master will increase coverage by 0.15%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
+ Coverage   60.88%   61.03%   +0.15%
==========================================
  Files          31       32       +1
  Lines        1828     1848      +20
==========================================
+ Hits         1113     1128      +15
- Misses        508      511       +3
- Partials      207      209       +2

This key is used for temporary encryption, but I'll clarify that point.


Maybe replace Stubble by Handlebars.Net

Just an idea : replace Stubble by Handlebars.Net ?
(disclaimer : I'm using Handlebars.Net & Scriban and I don't know Stubble at all...)

I’ll have a look, but so far Stubble has been really easy to use. Thank you for the suggestion, though.

@StefH I've looked into this and it seems like both libraries essentially do the same thing. Since Stubble is a reasonably active project, and I'm really only using its string rendering capability for replacing path parameter templates, I think it's safe to leave it like it is.
I appreciate the suggestion, though, gives me an alternative to check out in case Stubble doesn't live up to the expectations anymore.
Are you OK with me closing this?

Ok

Closed with thanks for the suggestion.


[WIP]Showing proper error message when quantization is used with CPU context

Description
When we use quantization with just CPU context w/o MKLDNN then MXNet SEG FAULTS. This is not ideal behaviour. Therefore, adding proper error messaging to show that currently CPU w/o MKLDNN doesn't support quantization. https://github.com/apache/incubator-mxnet/issues/11846
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-label-bot add [pr-work-in-progress]

Also need to add the error for quantized_flatten and quantized_concat.

Hmmm ....  Good point, makes sense

Thanks @access2rohit

@access2rohit could you rebase the code? thanks

@pengzhao-intel Sure! ..... I haven't finished it yet. LOG(FATAL) is not throwing correct stack trace. Still running into Core Dump (Aborted)

@access2rohit Were you able to figure out the root cause ?

@anirudh2290 : can you review ?

@mxnet-label-bot add [pr-awaiting-review]

@anirudh2290 Could you please review this PR?
@access2rohit Please look at the CI failure and also the conflicts. Thanks!


rosdep install robaka error

hi alex
my os version:18.04, but i cant run rosdep install robaka. please help check this. thanks~

fixed

how was this fixed? ```seb@tangerine-rover:~/catkin_ws$ rosdep install robako
ERROR: Rosdep cannot find all required resources to answer your query
Missing resource robako
ROS path [0]=/opt/ros/melodic/share/ros
ROS path [1]=/opt/ros/melodic/share```


mapperpyramid.cpp does not compile with opencv 2.4.9

Hi,
if i compile the library with visual c++ 2010, i got these message:
mapperpyramid.cpp
......\contrib\modules\reg\src\mapperpyramid.cpp(93): error C2039: 'get': Ist kein Element von 'cv::Ptr<_Tp>'
with
[
_Tp=cv::reg::Map
]
Thank you,
Steff

Hmm you are trying to mix the contrib module with OpenCV2.4.9 which is not possible. It only works with OpenCV3.0 and higher. There was no seperate module before OpenCV3.0. @mshabunin this can be closed (running through PRs to clean them up a bit)

@alalek should I address you instead for all the others I am checking?

@StevenPuttemans no, mention is not required. OpenCV team usually receives all notifications.

Okay thank you!


Program is now a console application. Would be nice if it is a true Windows program with a proper GUI, that also can be minimized to the notification area and run in the background..

Solved


flutter: [error] Failed to save envelope:

Dear Sentry Flutter team,
First, thanks for sharing this package !
I'm only getting started and receiving messages in sentry, yet I see an error that keeps popping up in the log :

flutter: [error] Failed to save envelope: MissingPluginException(No implementation found for method captureEnvelope on channel sentry_flutter)

i'm on sentry_flutter: ^4.0.6
Could it be the cause for this 'error' ?

@926 any news?

Hi @marandaneto,
Thanks for the reply and sorry for the delay due to holidays,
Unfortunately I cannot updated to the latest version yet,
The app is still trailing back before null-safety, I will retry once it is upgraded and closes this.


Double arcane stone slabs don't drop when broken

When you stack arcane stone slabs so that they form a full block, the block just disappears when broken instead of dropping 2 slab blocks.

Duplicate of #677. Please search before submitting issues.


aggregate chain fail

use sinon-mongoose to mock aggregate method with chain lookup, show err:
var sinon = require('sinon');
require('sinon-as-promised');
require('sinon-mongoose');

   describe("getNewTask", function () {
        var taskQueueMock;
        it("应当返回加密后字符串", function (done) {
            co(function*() {
                try {
                    taskQueueMock = sinon.mock(model.taskQueue);
                    taskQueueMock.expects('aggregate')
                        .chain('lookup')
                        .resolves({
                            "_id" : "57b5fea8bcd0fd2f0df06320"
                            }
                        );
                    var result = yield model.taskQueue.aggregate();
                    console.log(result);
                    done();
                } catch (err) {
                    console.log(err);
                    done(err);
                }
            })
        });
    });

error:
 TypeError: Attempted to wrap undefined property lookup as function
      at Object.wrapMethod (node_modules/sinon/lib/sinon/util/core.js:106:29)
      at Object.expects (node_modules/sinon/lib/sinon/mock.js:88:27)
      at Object.mock.expects (node_modules/sinon-mongoose/dist/index.js:19:37)
      at Function.chain (node_modules/sinon-mongoose/dist/index.js:12:20)
      at test/taskManager.test.js:54:26
      at next (native)
      at onFulfilled (node_modules/co/index.js:65:19)
      at node_modules/co/index.js:54:5
      at co (node_modules/co/index.js:50:10)
      at Context.<anonymous> (test/taskManager.test.js:50:13)

just mock aggregate without chain work

Hi @stableShip, thanks for reporting the issue.
It's fixed in new release 1.3.0

:+1:


Fix 'pokemon_data' referenced before assignment in incubate_eggs_worker (#1823)

Short Description:
'pokemon_data' referenced before assignment in incubate_eggs_worker
Fixes:

'pokemon_data' referenced before assignment in incubate_eggs_worker #1823

already fixed in #1777


Spinner for resolving ENS text avatar/address overlaps briefly with avatar

In the send screen

Type or paste ENS name which has an avatar text record configured
Little spinner appears
Wait

Observed

ENS avatar appear
Spinner still around, but doesn't look nice because it is overlapping the ENS avatar
Spinner disappears and address appears

Expected
Probably similar to observed, but reposition the spinner maybe?

Thanks for spotting this.


Memory leak in forcedphotrunner.py

I had to restart the forcedphotrunner.py process.  It has a severe memory leak.  psweb had run out of memory.
2021-01-31 12:10:29.471522  Waiting for tasks...
2021-01-31 12:31:29.034016  Unfinished jobs in queue: 1
2021-01-31 12:31:29.045658  job07743: Starting job for ken.smith (who has run 0 tasks in this pass so far):
2021-01-31 12:31:29.045763  job07743: {'id': 7743, 'timestamp': datetime.datetime(2021, 1, 31, 12, 31, 28, 794188), 'ra': 149.25245, 'dec': -10.34936, 'mjd_min': 59215.0, 'mjd_max': None, 'use
r_id': 1, 'use_reduced': 0, 'comment': '', 'finishtimestamp': None, 'send_email': 0, 'starttimestamp': None, 'email'<EMAIL_ADDRESS>'username': 'ken.smith'}
2021-01-31 12:31:29.046075  job07743: Executing on atlas: nice -n 19 /atlas/bin/force.sh 149.25245 -10.34936 m0=59215.0 dodb=1 parallel=16 | sort -n | tee ~/atlasserver/results/job07743.txt
Traceback (most recent call last):
  File "forcephotrunner.py", line 447, in <module>
    main()
  File "forcephotrunner.py", line 436, in main
    queuedtaskcount = do_taskloop()
  File "forcephotrunner.py", line 340, in do_taskloop
    localresultfile = runforced(conn=conn, logprefix=logprefix, task=task)
  File "forcephotrunner.py", line 106, in runforced
    encoding='utf-8', bufsize=1, universal_newlines=True)
  File "/usr/local/swtools/python/atls/anaconda3/envs/forced37/lib/python3.7/subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "/usr/local/swtools/python/atls/anaconda3/envs/forced37/lib/python3.7/subprocess.py", line 1482, in _execute_child
    restore_signals, start_new_session, preexec_fn)
OSError: [Errno 12] Cannot allocate memory

In the meantime, while investigating, we should restart the process at least weekly.

Thanks, Ken. I would have expected the start_forcedphotrunner.sh script to restart the Python script after an error like this. Was that not the case?
I've reverted to running the plotting script in a separate process. Let's see if that keeps the memory usage from growing.

Thanks, Ken. I would have expected the start_forcedphotrunner.sh script to restart the Python script after an error like this. Was that not the case?
I've reverted to running the plotting script in a separate process. Let's see if that keeps the memory usage from growing.

@genghisken could please check if this issue is now solved?

@genghisken looks like this is now solved, so I'll close this issue soon. Let me know if this is not the case.


google-storage can be configured with inline keyFile

1. Why is this change necessary?
So you can do this in your configuration keyFile: '%env(json:GCP_KEY_FILE)%' and have the configuration in a string and must not deal with a config file.
2. What does this change do, exactly?
The Google StorageClient accepts either a keyFilePath or the content of the service-account.json file as json-decoded array.
This change extends the GoogleStorageFactory to support both options of the underlying StorageClient.
3. Describe each step to reproduce the issue or behavior
Up until now, only this was possible:
shopware:
  filesystem:
    public:
      type: 'google-storage'
      config:
        projectId: '%env(GCP_PROJECT_ID)%'
        keyFile: '/path/to/service-account.json'

Now it's possible to have this:
export GCP_KEY_FILE={"type":"service_account","project_id":"...

shopware:
  filesystem:
    public:
      type: 'google-storage'
      config:
        projectId: '%env(GCP_PROJECT_ID)%'
        keyFile: '%env(json:GCP_KEY_FILE)%'
        bucket: '%env(GCP_PUBLIC_BUCKET)%'
        root: '%env(GCP_PUBLIC_ROOT)%'

4. Please link to the relevant issues (if any).
5. Checklist

[ ]  I have written tests and verified that they fail without my change
[x]  I have squashed any insignificant commits
[ ]  I have written or adjusted the documentation according to my changes
[ ]  This change has comments for package types, values, functions, and non-obvious lines of code
[x]  I have read the contribution requirements and fulfill them.

hey @hhoechtl
thanks for your contribution :+1:
please fill out the pull request template completely. How could we test this? Must some documentation to be changed? Is it possible to write some tests?
In addition to that, please squash your commits.
Thank you! :blush:

@mitelg better now? The storage adapters are not tested or documented yet.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-5541
Please use this issue to track the state of your pull request.

@hhoechtl
if you rebase your branch we'll look into this again :)
Sunny regards
Pantrtxp

@pantrtxp done

@hhoechtl
sorry for the delay. Can you please sign the CLA, so we can import this.
thank you for your contribution 👍

@ssltg How? That's just a markdown page without any "sign here" field or information where to send it to.

https://cla-assistant.shopware.com/shopware/platform?pullRequest=290
see here :)

@pantrtxp done

I just added a small change to also allow options like in the S3 adapter which is necessary to do this:
        options:
          visibility: 'public'

This is necesary.

Thank you for your contribution :blue_heart: :tada:

Thank you for your contribution 👍
THis was merged with: https://github.com/shopware/platform/commit/56c2d7698afd18a8acd0c220b92266f3f9217cde


API stops answering requests and "locks" until "Enter" or "Ctrl+C" is sent

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI, Query, Body, Depends
import pandas as pd
import numpy as np
import pyodbc
from typing import Any, List
from pydantic import BaseModel

# Load Config
config = {
    "db": {
        "DSN": "DSN",
        "UID": "UID",
        "PWD": "PWD"
    },
    "n_conn": 4,
    "port": 5000
}

DB = config["db"]
N_CONN = config.get("n_conn", 4)
PORT = config.get("port", 5000)

# Build Connection Pool
class Pool:
    def __init__(self, connect, n_conn=N_CONN):
        self.conns = [connect() for _ in range(n_conn)]
        self.current = 0
        self.n_conn = n_conn

    def connect(self):
        conn = self.conns[self.current]
        self.current = (self.current + 1) % self.n_conn
        return conn

def connect():
    dsn = ";".join([f"{key}={value}" for key, value in config["db"].items()])
    return pyodbc.connect(dsn)

pool = Pool(connect)

@Depends
def get_conn():
    return pool.connect()

# Build App
app = FastAPI()

# Build Responses
class ResponseModel(BaseModel):
    columns: List[str]
    rows: List[List[Any]]

def serialize_df(df):
    df = df.replace({np.nan: None})
    return {
        "columns": df.columns.tolist(),
        "rows": df.values.tolist(),
    }

# Routes
@app.get("/")
def root():
    print("Health Check.")
    return {"message": "All Working!"}

@app.post("/aggfunc", response_model=ResponseModel)
def aggfunc(
    start_time: str = Query(...),
    end_time: str = Query(...),
    tags: List[str] = Body(...),
    conn=get_conn,
):
    first_tag = tags[0]
    tags = ",".join(tags)
    start_time = pd.Timestamp(start_time)
    end_time = pd.Timestamp(end_time)

    sql = f"""
        SELECT Name, Timestamp, Quality, doubleval(Value) AS Value
        FROM EpmQueryAggregateFunction(
            -3, '{start_time}', '{end_time}',
            300000, 'TimeAverage',
            '{tags}'
        )
        ORDER BY Timestamp, Name;
    """

    print(f"First Tag: {first_tag} - Início")

    t0 = pd.Timestamp("now")
    df = pd.read_sql(sql, conn)
    t1 = pd.Timestamp("now")

    print(f"First Tag: {first_tag} - Fim - {(t1-t0).total_seconds():.2f}s")

    return serialize_df(df)

if __name__ == "__main__":
    import uvicorn

    uvicorn.run("main:app", host="<IP_ADDRESS>", port=PORT)

Description
Hi all,
I am having some problems using fastAPI in Windows.
Sometimes, the program stops answering requests, and the terminal "locks" (no more logs).
If I click on the terminal window and press Enter or Ctrl + C, it starts working again and answers the requests.
Have you ever experienced something like this? I don't even know how to start debugging it
More Details:
The requests are all directed to the same route. The requests are based on using pre-estabilished connections to a database (using pyodbc.connect()), fetching data with pd.read_sql(), parsing it to json and returning.
Operating System
Windows
Operating System Details
No response
FastAPI Version
0.65.2
Python Version
3.9.2
Additional Context
No response

I can't reproduce the example.
Can you show us the logs until the point it freezes?

I also have the same problem in windows, did you manage to solve it somehow?

It's probably that you accidentally selected some text on the terminal, I think that freezes anything happening there until you hit enter (enter copies the selected text).
Maybe, because Windows... 😅


0.14 Release

Is a 0.14 release planned? I'm keen to use the lock handling features recently added.
Thanks!

I would also like to know. The Spring 2.x PR would be very useful for me

The maintainers of mongobeeJ have deprecated it as of August 2020, with a suggestion to use an actively maintained library such as Mongock instead.

Deprecation Notice
mongobeeJ is no longer actively developed. It will be supported (bug fixes and dependency upgrades) for the sake of legacy projects that use mongobeeJ or mongobee.
However, it's recommended to migrate active projects from mongobeeJ to other tools like Mongock or Sherlock.


Kubernetes: debug(Launch) failed with Nodejs project

Repro steps:

create a project following this doc : https://nodejs.org/en/docs/guides/nodejs-docker-webapp/
set proper vsdocker.imageUser in user setting and docker login
execute "Kubernetes: debug(Launch)"  and input debug command "node debug server.js"

expect: the debugger will start
actually:

Then we execute "Kubernetes: Remove debug" an error occured.

This should be fixed by #756 and it is now part of the new release 1.2.1. Closing it.
You can find some more infos in the documentation.
If you still face any issues with it let us know and/or feel free to reopen it. Thank you :)


[FEATURE] middleware implementation

add middleware for endpoint protection etc..

yes good idea if you got some time please add that very important feature

understood! ill work on it

we created a method Router::setAuthenticationRequired(label) to support authentication


Confusing deprecation notice for "info" system adapter

Bevy version
main / 12.0
What you did
From Discord, yesterday:

i'm a bit confused why info!() is now deprecated, don't people use info!("some log"); to log stuff? wasn't mentioned in the migration guide, should I not rely on bevy to give me a stable logging API?
I fixed it by adding a dependency to tracing and import from  there instead from bevy prelude

I and others were confused about this and unable to answer. But I later stumbled on the same deprecation notice and looked into it.
This code compiles, runs, and even works. (presumable in Bevy 0.11 as well)
use bevy::prelude::{info, App, DefaultPlugins, Startup};

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_systems(Startup, sys)
        .run();
}

fn sys() {
    info!("Hello, world!");
}

What went wrong
But it emits a deprecation notice:
   Compiling bevy-test v0.1.0 (/Users/robparrett/src/bevy-test)
warning: use of deprecated function `bevy::prelude::info`: use `.map(bevy_utils::info)` instead
 --> src/main.rs:1:21
  |
1 | use bevy::prelude::{info, App, DefaultPlugins, Startup};
  |                     ^^^^
  |
  = note: `#[warn(deprecated)]` on by default

Which is very confusing.
It seems that the info "system adapter" was/is somehow importable as bevy::prelude::info?

"Fixed" by #10693


feat(lib): remove extern Url type usage

Closes #1089
BREAKING CHANGE:
The Url type is no longer used. Any instance in the
Client API has had it replaced with hyper::Uri.
This also means Error::Uri has changed types to
hyper::error::UriError.
The type hyper::header::parsing::HTTP_VALUE has been made private,
as an implementation detail. The function http_percent_encoding
should be used instead.

Just as a usecase/question: I used to have a base url stored as a hyper::Url and then appending path components to it via .join(). This PR seems to have removed this kind of usage, as there are no more methods to mutate an Uri in place once built. Am I right that this is on purpose and the recommended usage is as described by @alexcrichton in https://github.com/hyperium/hyper/issues/1089#issuecomment-286252505?

Yes, this is on purpose for now. Uri could possibly gain mutators in the future.
hyper is going through what most libraries go through as they mature: removal of extras, providing a strong core, and allowing extras to be built on top with whatever opinions the builder has.