XRDP over XVNC extremely sluggish on RHEL 8

I have XRDP configured on RHEL 7 and 8.  They are configured near identical, and the XRDP was installed in the same manner:

Server with GUI, graphical.target default
xrdp and tigervnc-server installed
xrdp enabled
firewalld ports enabled
connecting over VPN

When running on RHEL 7 there are no noticable delays or sluggishness.  When running on RHEL 8 every single action is delayed by 1-3 seconds in response.  This includes actions such as opening a terminal window, or moving a window to another location.
A console to the VM through VMware does not show the same delays.  Meaning there is some unique element of the RDP connection settings that is causing this sluggishness/delays.
The RHEL 8 box is a fresh install, with no unique elements other than those identified above.

Interesting problem.
I'm assuming you're using EPEL. Both operating systems would seem to have the same version of xrdp available (0.9.14-3 as I write this), so if you've performed no other customisations you'll be running the same xrdp software and configurations in both cases. On both machines, xrdp -v will give you more detailed version information.
1-3 seconds is a big chunk of time. xrdp is acting as little more than a protocol converter in this situation, and there's nothing in the software I'm aware of which could account for this. This suggests you'll need to consider the problem from a more systemic perspective.
I've set up CentOS 7 and 8 boxes here (xrdp from EPEL in both cases), and I've connected to them from a Windows 10 client. Once I've done that, I can use ss to look at the TCP stats for the link from the VM to the Windows 10 box:-

$ ss -it '( sport = 3389 )'
State          Recv-Q          Send-Q                   Local Address:Port                             Peer Address:Port
ESTAB          0               0                        <IP_ADDRESS>:ms-wbt-server                   <IP_ADDRESS>:60124
	 cubic wscale:8,7 rto:238 rtt:37.759/22.026 ato:40 mss:1460 pmtu:1500 rcvmss:691 advmss:1460 cwnd:10 ssthresh:25 bytes_sent:4815887 bytes_acked:4815887 bytes_received:25558 segs_out:4032 segs_in:1052 data_segs_out:3826 data_segs_in:291 send 3.1Mbps lastsnd:424277 lastrcv:132504 lastack:132520 pacing_rate 6.2Mbps delivery_rate 39.0Mbps delivered:3827 app_limited busy:12315ms sndbuf_limited:214ms(1.7%) rcv_space:29200 rcv_ssthresh:84622 minrtt:3.67

I won't pretend I understand what a lot of these fields mean (and they differ between 7 and 8), but I can use the bytes_sent field to see how much data is sent when I move a window, or perform some other kind of operation. On the setup here, with no customisations, moving a terminal windows from one side of the screen to the other is more efficient on CentOS 8 than it is on 7. This may be down to the whacky CentOS 7 backdrop - I haven't investigated any further.
You mention a VPN. Where is this terminated? Is it terminated on the boxes, or is terminated on another appliance?
The VMWare console working is useful and rules out CPU/memory starvation as a cause. may give us a few clues. Here are a few questions which may be useful.

Does the VMWare console route through the VPN?
The VMWare console won't being going through the interface on the VM which the RDP connection is terminating on. CentOS 7 and 8 will be using different driver versions on their interfaces. Are you seeing packet drops on this interface? (use ip -s link show <interface> to see packet stats). Are there differences in the virtual network routing which could be significant?

Comparing bandwidths and latencies of the links between Windows and C7, and Windows and C8 could be useful. Bandwidths are easily measured using iperf3 (available on Windows and in EPEL), and latencies can be inferred from the round-trip-times displayed by ping from Windows.
Sorry about the scattergun approach. I don't know about your architecture at all, and so it's pretty hard to suggest ways forward from here.

Interesting problem.
I'm assuming you're using EPEL. Both operating systems would seem to have the same version of xrdp available (0.9.14-3 as I write this), so if you've performed no other customisations you'll be running the same xrdp software and configurations in both cases. On both machines, xrdp -v will give you more detailed version information.
1-3 seconds is a big chunk of time. xrdp is acting as little more than a protocol converter in this situation, and there's nothing in the software I'm aware of which could account for this. This suggests you'll need to consider the problem from a more systemic perspective.
I've set up CentOS 7 and 8 boxes here (xrdp from EPEL in both cases), and I've connected to them from a Windows 10 client. Once I've done that, I can use ss to look at the TCP stats for the link from the VM to the Windows 10 box:-

$ ss -it '( sport = 3389 )'
State          Recv-Q          Send-Q                   Local Address:Port                             Peer Address:Port
ESTAB          0               0                        <IP_ADDRESS>:ms-wbt-server                   <IP_ADDRESS>:60124
	 cubic wscale:8,7 rto:238 rtt:37.759/22.026 ato:40 mss:1460 pmtu:1500 rcvmss:691 advmss:1460 cwnd:10 ssthresh:25 bytes_sent:4815887 bytes_acked:4815887 bytes_received:25558 segs_out:4032 segs_in:1052 data_segs_out:3826 data_segs_in:291 send 3.1Mbps lastsnd:424277 lastrcv:132504 lastack:132520 pacing_rate 6.2Mbps delivery_rate 39.0Mbps delivered:3827 app_limited busy:12315ms sndbuf_limited:214ms(1.7%) rcv_space:29200 rcv_ssthresh:84622 minrtt:3.67

I won't pretend I understand what a lot of these fields mean (and they differ between 7 and 8), but I can use the bytes_sent field to see how much data is sent when I move a window, or perform some other kind of operation. On the setup here, with no customisations, moving a terminal windows from one side of the screen to the other is more efficient on CentOS 8 than it is on 7. This may be down to the whacky CentOS 7 backdrop - I haven't investigated any further.
You mention a VPN. Where is this terminated? Is it terminated on the boxes, or is terminated on another appliance?
The VMWare console working is useful and rules out CPU/memory starvation as a cause. may give us a few clues. Here are a few questions which may be useful.

Does the VMWare console route through the VPN?
The VMWare console won't being going through the interface on the VM which the RDP connection is terminating on. CentOS 7 and 8 will be using different driver versions on their interfaces. Are you seeing packet drops on this interface? (use ip -s link show <interface> to see packet stats). Are there differences in the virtual network routing which could be significant?

Comparing bandwidths and latencies of the links between Windows and C7, and Windows and C8 could be useful. Bandwidths are easily measured using iperf3 (available on Windows and in EPEL), and latencies can be inferred from the round-trip-times displayed by ping from Windows.
Sorry about the scattergun approach. I don't know about your architecture at all, and so it's pretty hard to suggest ways forward from here.

I have multiple setups with different issues (and clearly some without any issues).  The one I'm most concerned with is clearly the RHEL 8 via xvnc over VPN.
The VPN in this case is connecting to an OpenVPN running on a PfSense Router.  This applies to the working RHEL 7 setup of XRDP over xvnc, as well as the console connection over VMWare.
As far as the virtual networking goes - Those are running over vlans on 10G switches with no significant configurations outside of ACLs.  Most traffic is separated onto several different switches, and no area is remotely saturated.
I'll certainly look into other options, but I'm in the same boat where I'm not sure what's causing the lag.

I have multiple setups with different issues (and clearly some without any issues).  The one I'm most concerned with is clearly the RHEL 8 via xvnc over VPN.
The VPN in this case is connecting to an OpenVPN running on a PfSense Router.  This applies to the working RHEL 7 setup of XRDP over xvnc, as well as the console connection over VMWare.
As far as the virtual networking goes - Those are running over vlans on 10G switches with no significant configurations outside of ACLs.  Most traffic is separated onto several different switches, and no area is remotely saturated.
I'll certainly look into other options, but I'm in the same boat where I'm not sure what's causing the lag.

Checking the bandwidth and latency between the Windows client and the C8 sounds like it's worth doing. It might point to possibilities like a duplicate MAC or a problem with the NIC driver on the VM.

Checking the bandwidth and latency between the Windows client and the C8 sounds like it's worth doing. It might point to possibilities like a duplicate MAC or a problem with the NIC driver on the VM.

IPv6 was the culprit.  Past the VPN, IPv6 is disabled.  The OS was preferring IPv6, but could not route properly, causing a delay for all actions.  Once IPv6 was disable on the OS, everything worked fine to include remotely over the VPN.

IPv6 was the culprit.  Past the VPN, IPv6 is disabled.  The OS was preferring IPv6, but could not route properly, causing a delay for all actions.  Once IPv6 was disable on the OS, everything worked fine to include remotely over the VPN.

Thanks for the update - it might help someone else in the future with a similar issue.

Thanks for the update - it might help someone else in the future with a similar issue.


Develop

@butwerenotthereyet Could you please have a look at commit 2fdae94c033d7a1e4419f2eb082a98d7a6ef8ec6? It changes many of the lines you already refactored. I think it should be enough to use one ID per -t switch since all IDs are unique in one tmux server instance. Or do I miss something here?

@butwerenotthereyet Could you please have a look at commit 2fdae94c033d7a1e4419f2eb082a98d7a6ef8ec6? It changes many of the lines you already refactored. I think it should be enough to use one ID per -t switch since all IDs are unique in one tmux server instance. Or do I miss something here?

@IngoMeyer441  Looking good!   And getting rid of subshells!  🚀

@IngoMeyer441  Looking good!   And getting rid of subshells!  🚀


Multiclass segmenation using MedSAM and custom dataset

@JunMa11
I wanted to do multiclass segmentation using a custom dataset.
The ground truth size here is (256,256). Can I give (256,256,6) size as input for 6 classes? If I do this, I get ValueError: the number of dimensions for input and target should be the same, got shape torch.Size([8, 1, 256, 256]) and torch.Size([8, 1, 256, 256, 6]).
Then what should be the size of the input and where to do the changes/modifications in your code?

Hi @simantika ,
Both SAM and MedSAM are promptable segmentation methods can they don't provide category information. If you want to apply them for multi-class segmentation, you may need to convert the task to multiple binary segmentation tasks.

Thanks @JunMa11
So do you mean that if I have 6 classes to segment, then there will be 6 models??

Hi @simantika ,
Sorry for my late reply. I had a deadline yesterday.
Could you please give me more background on your segmentation task?
If the six classes have no overlap, you can just train one model. During inference, you can obtain the class-wise segmentation results by giving them separate bounding boxes.

The six classes that I have are overlapping.

Hi,
I'm very sorry for the reply.
For the label overlapping case, you can first separate them into multiple binary segmentation datasets followed by merging all of them in one folder to train medsam.

Thanks for the reply


Add --tiebreak proximity=path

This patch adds a new type of tiebreak: proximity=path. It works by ranking equally-scored candidates by their proximity to the given path argument, where proximity is defined as the number of shared leading path segments when split by the OS path separator.
Consider the following simple file hierarchy:
test.txt
bar/test.txt
bar/main.txt
misc/test.txt

Where a user is currently in the context of bar/main.txt. This could be for a number of reasons, such as it being the file that is currently open in their editor (see junegunn/fzf.vim#360 and junegunn/fzf.vim#492). They now want to open bar/test.txt. If they invoke fzf and type, well, any search, the by_length scoring will make the test.txt file in the root the "best" candidate, as would by_start. by_end would propose misc/test.txt. by_score would also likely suggest test.txt in the root. None of these take into account the user's current location.
With this patch, the user can invoke fzf --tiebreak path=bar/main.txt, which will cause it to rank files that share a prefix segment (bar/ in this case) to rank higher. Thus, when the user types, say, t, bar/test.txt will immediately be the top recommendation. In editor context, the user's editor should probably automatically add this flag based on the user's current file.
The flag also allows for more natural search in deeper hierarchies. For example, if the user is in foobar/controller/admin.rb here:
baz/controller/admin.rb
baz/views/admin.rb
foobar/controller/admin.rb
foobar/views/admin.rb

And wants to quickly edit the view for that application (i.e., foobar/views/admin.rb), then
fzf --tiebreak path=foobar/controller/admin.rb

will rank foobar/views/admin.rb higher than baz/views/admin.rb for a search of admin.rb, which is what the user most likely expected.
Note that contrary to other suggestions like only listing files at or below the user's current directory, this patch does not limit the user's ability to search beyond the scope of their current context (e.g., they can get to baz/controller/admin.rb if they so wish).

Interesting idea, though I'm not sure if I want to add options for specific types of input. Once we have an option for file paths, my argument for keeping fzf simple and rejecting options for other types of input – list of processes, URLs, songs and albums, or whatever we can imagine – becomes somewhat moot. And even for file paths, other users may have different opinions on how they should be sorted, by file size, by modification time, by extension, etc, and I don't think we can continue adding options for such different requirements. I agree that this is a common requirement, but I just don't want to make an exception.
As I commented in the linked issue, this particular case can be addressed by reordering the input and using --tiebreak=index.
(fd -p src/util; fd -E src/util) | fzf --tiebreak=index

function! s:list_cmd()
  let base = fnamemodify(expand('%'), ':h:.:S')
  return base == '.' ? 'fd' : printf('fd -p %s; fd -E %s', base, base)
endfunction

command! -bang -nargs=? -complete=dir Files
  \ call fzf#vim#files(<q-args>, {'source': s:list_cmd(),
  \                               'options': '--tiebreak=index'}, <bang>0)

Mmm, that's a fair argument. And thanks for the explicit example config! Sadly the solution you propose doesn't actually solve this issue (at least for me), since it only gives a binary "close/not close" ranking, as opposed to being ranked by proximity. I guess the way to solve this without modifying fzf is to implement a separate program that takes a list of files as input and order them by their proximity to an argument. That way you could do:
$ fd -pFt f path/to/file | proximity-sort | fzf --tiebreak=index

I guess implementing proximity-sort is next on my TODO then :p

Thanks for understanding :)

proximity-sort now exists here: https://github.com/jonhoo/proximity-sort
And the included .vimrc config blurb in the README there fixes junegunn/fzf.vim#360 :tada:

Cool, that was really fast!

What can I say — I really wanted this feature. It's been making me crazy over the past week while working on a paper in a repository with multiple papers that all have files like intro.tex, design.tex, eval.tex, etc.!

proximity-sort now exists here: https://github.com/jonhoo/proximity-sort
And the included .vimrc config blurb in the README there fixes junegunn/fzf.vim#360 🎉

@jonhoo love it! How would I implement this into my .bashrc so fzf always uses this functionality?

It's a little tricky to have it be used on the command-line, because there is no "current" file. The README on https://github.com/jonhoo/proximity-sort has info about how you integrate it with vim though!


Same packet received over and over (ESP32 and 500Kbps vehicle bus)

Tried the basic receive example after trying the OBD keystats example with no success.
Seems to be getting valid data from the bus - but keeps printing the same packet over and over. Same ID, same packet data. Like the buffer is not being cleared?

I have been able to duplicate this - seems to happen when the bus load is high - cangen can0 -g 3 from a raspberry pi to the ESP32 seems to cause the issue. 5 ms packet gap is stable.

I am having the same problem. Have you been able to find a soluction?

I have been able to duplicate this - seems to happen when the bus load is high - cangen can0 -g 3 from a raspberry pi to the ESP32 seems to cause the issue. 5 ms packet gap is stable.

i am also having the same problem. basically my device device was running properly around 8 hrs after that i am keep on getting same msg in can receiver side and it never stops. Did you find the solution?

Never really fixed it - kept my project from being finished. I'll dig into this again maybe over the December break.

Same here. Using a SN65HVD230 module, built in 120ohm termination, Rs=10k sets the chip to slope control. Might have a go at replacing the 10k with a short to ground to put the module in high speed mode. EMI is not a concern for just listening anyway.

Just put a 10+ ms delay in your loop

Did you ever figure this out? I'm having exactly the same problem. Did shorting pin 8 to ground for high speed mode help?

Did you ever figure this out? I'm having exactly the same problem. Did shorting pin 8 to ground for high speed mode help?

I used a 10ms delay in the loop to prevent the loop from reading the same package twice.
It's better to use micros rather than delay, but delay will also work

Tried adding a 10ms delay, it didn't help.

I strongly suggest not trying to "fix" bugs by adding delays.
It's not going to fix anything, it will only make the issue less likely to happen.
The issue will still happen, possibly when you least expect it.
Since we're talking about a library with automotive applications, I don't think that's good enough.

I strongly suggest not trying to "fix" bugs by adding delays. It's not going to fix anything, it will only make the issue less likely to happen. The issue will still happen, possibly when you least expect it. Since we're talking about a library with automotive applications, I don't think that's good enough.

The delay is because the esp32 scan cycle is much faster than the comms update rate , so it reads the same package in multiple scans. Delay is an easy way, best way as I said earlier is to use micros so to chose how often to read.
On the other hand this are diy project mostly that we only read not write to the canbus , so I can't see any harm

Same issue here, sadly.


FINERACT-835: Upgraded Swagger Annotations to Annotations 2.0

FINERACT-835
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[x] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[x] API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm has been updated with details of any API changes.

[x] Integration tests have been created/updated for verifying the changes made.

[x] All Integrations tests are passing with the new commits.

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

Travis failed because:

The job exceeded the maximum log length and has been terminated.

Yes the resolve task of Swagger plugin for some reason ends up with log level DEBUG and produces a lot of output. Need to find a way to set the log level to something more sensible...

Ok - think I found a way to sort out the excess logging from :resolve: and :enhance:
Let's see if this build gets through...

So looks like now it's not failing because of log levels, but because the Swagger resolve task takes so much memory that Docker inside Travis can't handle it and kills the build. Need to see if there's a way to either reduce memory usage by resolve or give Docker more memory...

Right - seem to have fixed both the Travis logging as well as memory issues with the build
@vorburger Here's a PR ready to update all the Swagger Annotations to 2.0 - after that, majority of the content should be in fineract.json. What's still missing is the overview (beginning of the file) that needs to be passed in as a config file to the plugin, and then finetuning of the output (for example, why do we have attribute descriptions as freetext in the summary rather than in the schemas for each attribute). And of course all the other stuff that was planned (i.e. getting "Try this API" to work etc). But I'm happy to leave that for someone else...

I've resolved the conflicts caused by #1083 and will merge this when it passes the build (or another committer can).


Reorder the links.

Every time a new link is added, it goes to the end. It would be quite useful if the add-on allowed to order the links however the user wanted.
Add this function please.

Yeah, please!


Feature: Display item number in list

I think it's useful to quickly see how many items are in the list. For example, if you're trying to narrow down a regexp (watching the number of matches decrease) or fiddling with a test (hopefully making fewer tests fail).
With a quickfix list, since the lines are numbered, you can go to the bottom of the list (with G) and see the total number of items relatively quickly. Also, while working your way through the list, it's easy to see your progress (have I done 10 of the items or 30?).
It would be nice to have a similar feature in trouble.

Development on the main branch is EOL.
Trouble has been rewritten and will be merged in main soon.
This issue/feature either no longer exists or has been implemented on dev.
For more info, see https://github.com/folke/trouble.nvim/tree/dev


added litmus-portal base code for frontend and backend

Added litmus-portal base code for frontend, also modified…the gitignore files of both projects to be a common file
Signed-off-by: arkajyotiMukherjee<EMAIL_ADDRESS>
What this PR does / why we need it: This provides the website (frontnd) code setup for Litmus portal
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1587
Major Changes in Frontend:

React
Typescript
Material UI
React Router
GraphQL
Redux

Checklist

[x] Does this PR have a corresponding GitHub issue?
[ ] Have you included relevant README for the chaoslib/experiment with details?
[ ] Have you added debug messages where necessary?
[ ] Have you added task comments where necessary?
[x] Have you tested the changes for possible failure conditions?
[ ] Have you provided the positive & negative test logs for the litmusbook execution?
[ ] Does the litmusbook ensure idempotency of cluster state?, i.e., is cluster restored to original state?
[ ] Have you used non-shell/command modules for Kubernetes tasks?
[ ] Have you (jinja) templatized custom scripts that is run by the litmusbook, if any?
[ ] Have you (jinja) templatized Kubernetes deployment manifests used by the litmusbook, if any?
[ ] Have you reused/created util functions instead of repeating tasks in the litmusbook?
[ ] Do the artifacts follow the appropriate directory structure?
[ ] Have you isolated storage (eg: OpenEBS) specific implementations, checks?
[ ] Have you isolated platform (eg: baremetal kubeadm/openshift/aws/gcloud) specific implementations, checks?
[ ] Are the ansible facts well defined? Is the scope explicitly set for playbook & included utils?
[ ] Have you ensured minimum/careful usage of shell utilities (awk, grep, sed, cut, xargs etc.,)?
[ ] Can the limtusbook be executed both from within & outside a container (configurable paths, no hardcode)?
[ ] Can you suggest the minimal resource requirements for the litmusbook execution?
[ ] Does the litmusbook job artifact carry comments/default options/range for the ENV tunables?
[ ] Has the litmusbooks been linted?

Special notes for your reviewer:

Backend directory structure is not decide yet can you remove this part?

Backend directory structure is not decide yet can you remove this part?

You mean to say remove the entire backend folder...or just the mention o backend from the PR

I meant for the entire backend. The schemas and all are still dummy, it will do nothing. It will go with the cluster connect PR

I meant for the entire backend. The schemas and all are still dummy, it will do nothing. It will go with the cluster connect PR

I meant for the entire backend. The schemas and all are still dummy, it will do nothing. It will go with the cluster connect PR

So what is the exact thing that you want me to "remove" when you say "remove this"?

I meant for the backend part

I meant for the backend part

Removed the backend folder, I'll edit the PR message too.

Also what do you prefer for the frontend codebase 2 space or 4 space indent?

Let's update the labels accordingly to figure out at what stage we are.

Also, logos like mayadata and openebs should be remove

Also what do you prefer for the frontend codebase 2 space or 4 space indent?

space 2, as there's a lot of nesting it makes for easier viewing. FYI That is already there in the linting.


Fix comparison operators for TypedAllocators

Move the operators into the same namespace as TypedAllocator for ADL instead of leaving them in the global namespace.

I can't press merge, but this is ready to go.


Problem after upgrading to juniper

Hey,
I have a problem with lookAhead after upgrading juniper 0.15.3 ( it works with 0.14.2
Thanks
wrong number of lifetime arguments: expected 2, found 1
expected 2 lifetime argumentsrustc(E0107)
graphql.rs(14, 1): expected 2 lifetime arguments
no method named select_child found for reference &juniper::executor::look_ahead::LookAheadSelection<'_, juniper::value::scalar::DefaultScalarValue> in the current scope
method not found in &juniper::executor::look_ahead::LookAheadSelection<'_, juniper::value::scalar::DefaultScalarValue>
help: items from traits can only be used if the trait is in scope
note: the following trait is implemented but not in scope; perhaps add a use for it:
use juniper::executor::look_ahead::LookAheadMethods;rustc(E0599)
graphql.rs(14, 1): method not found in &juniper::executor::look_ahead::LookAheadSelection<'_, juniper::value::scalar::DefaultScalarValue>

Are you using a git dependency for juniper-from-schema? The version that is on crates.io doesn't support juniper 0.15.
There has been a bunch of breaking changes as well. The changelog has the details.

I only use the crate.io

How can I use the last version ?
Thanks

Using a git dependency:
juniper-from-schema = { git = "https://github.com/davidpdrsn/juniper-from-schema.git", branch = "master" }

I have now conflict of versions with juniper_eager_loading
here is my Cargo.toml file
[dependencies]
async-std = { version = "1.8.0"}
diesel = { version = "1.4.5", features = ["postgres","64-column-tables","r2d2" ,"chrono", "numeric"] }
r2d2-diesel = "1.0.0"
#r2d2 = "0.8.9"
#features = [..., "64-column-tables"]
dotenv = "0.15.0"
diesel_cli_ext = "0.3.6"
#bigdecimal = "0.0.14"
bigdecimal = { version = "0.1.2", features = ["serde"] }
diesel_full_text_search = ""
#diesel_codegen = { version="", features =["postgres"] }
serde = { version = "1.0.116"}
serde_derive  = ""
serde_json = ""
actix-web = "3.3.2"
actix-rt = ""
actix = ""
actix-cors = "*"
chrono = { version = "0.4", features = ["serde"] }
lazy_static = "1.4.0"
#juniper ="0.14.2"
juniper ="0.14.2"
#juniper-from-schema = "0.5.2"
juniper-from-schema = { git = "https://github.com/davidpdrsn/juniper-from-schema.git", branch = "master" }
juniper-eager-loading = "0.5.1"
env_logger = "0.8.1"
futures = "0.3"

juniper-eager-loading doesn’t yet support juniper 0.15. It’s quite tricky updating it to async and I haven’t had time to do it yet. So I’m afraid if you want to use the new juniper you cannot use juniper-eager-loading.

Thanks


opus.Encoder._pushEncodedBuffer

Hello!
I have stream of encoded opus packets.
I know rate, channels number and frameSize.
I need write ogg file.
I wrote something like this
var opusEncoderStream = new opus.Encoder(rate, channels, frameSize);
var oggEncoder = new ogg.Encoder();
var fileStream = fs.createWriteStream('file.ogg');

opusEncoderStream.pipe(oggEncoder.stream()).pipe(fileStream);

while (encodedBuffer) { // read encoded opus data
   opusEncodeStream._pushEncodedBuffer(encodedBuffer);
}

But I get nothing in file.ogg
Tell me what else I need to do?
Thanks.

It works, but seems audio has not time stampes inside.
vlc player for example plays it wrong.
ffplay plays normal.
browser plays normal but doest show full time info.
var fileStream = fs.createWriteStream('file.ogg');
var oggEncoder = new ogg.Encoder();
oggEncoder.pipe(fileStream);

var opusEncoderStream = new opus.Encoder(rate, channels, frameSize);
opusEncoderStream.pipe(oggEncoder.stream())

while (encodedBuffer) { // read encoded opus data
   opusEncodeStream._pushEncodedBuffer(encodedBuffer);
}

How to fix time problem?
Thanks.

I'm archiving the repository for lack of interest in maintaining it. Sorry I never got to deal with this issue. Please see @discordjs/opus for an alternative implementation that is based this project but uses more stable Node.js module technology that should break less between Node.js versions.


Added peekable streams to avoid blocking read calls on pipes with no data

Named pipes are not able to support concurrent read/writes from separate threads. This causes code that does this to get into a deadlock when you call read on an empty pipe because the read blocks and waits for data to come in and no write operations can go through while this is happening which means the read will never see any data. There is a windows system call to peek a pipe that returns immediately even when no data is present. This allows you to be able to peek before calling read so that you can be sure data is present before calling read.

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

Is this likely to be merged in any time soon?

@galvesribeiro - Do you have interest in trying to prune the backlog now that we can take some of these? This one seemed good tome many moons ago :)

Yeah, I guess before we release the package with the current codebase we need to have a task force to review the outstanding PRs and merge/close them.
You know this codebase better then myself so if you see something is good like this, feel free to merge directly. I'll take a look over the week in case you don't have time.


Arbitrary vertical velocity update takeoff threshold used for offboard mode

When flying a drone in offboard mode, the drone will not take off unless it receives a vertical velocity less than -0.3 m/s. I first thought it was being caused by the EKF accuracy, however after digging through the code it seems the "FlightTask.cpp" file specifies at line 195 that:
velocity_triggered_takeoff = _velocity_setpoint(2) < -0.3f;

This arbitrary constant of "-0.3" should be a configurable parameter.
What this means is that if you are attempting to send constant velocity updates in offboard mode, your first velocity has to be less than "-0.3", otherwise your drone will just sit there. And in most cases if you are regulating the drones acceleration, then drone will not take off, as the velocity updates will be too small.
The current work around is to just set your initial velocity beyond the threshold, and then go back to your normal velocity updates.

Thanks for the issue. If you have an idea how to solve it, feel free to create a  PR.
To me this seems similar (maybe related to) #12471.
FYI @MaEtUgR @bresch.


Bar chart issue with time-based X axes

If I use time-based X axes in a bar chart, the first & last bars are cropped.
Expected Behavior
Draw the last & first bars correctly. It works properly with linear X axes.
Current Behavior
Crop the last & first bars.

Possible Solution
I don't know.
Steps to Reproduce (for bugs)
Repro example: https://codepen.io/icebob/pen/LYGjZjx?editors=0010
Context
Environment

Chart.js version: 2.9.3
Browser name and version: Chrome 83
Link to your project: -

Set offset: true on the x-axis


GraphQL Error on /subscriptions (possible flicker)

Describe the bug
GraphQL Error (string)
To Reproduce
Steps to reproduce the behavior:

Visit /subscriptions for first time

Expected behavior
no error
Screenshots

Desktop (please complete the following information):

OS: Arch Linux
Browser Chrome latest-stable

Additional context
Could not reproduce

Thank you for submitting this. I unfortunately can't reproduce at the moment.
I tried on Firefox / macOS:

navigating to /subscriptions with a normal account
force refreshed the /subscriptions page with a normal account
navigating to /subscriptions with an empty account
force refreshed the /subscriptions page with the empty account

K, these types of issues are the worst to debug (not reproducible), but as long as others aren't reporting it then it seems super low priority and I suggest closing this. And it is helpful in case it crops up again with other users.


fix: convert canadian prices from cents per liter to dollars per liter

fixes #74

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.48%. Comparing base (0cad9d4) to head (2eae460).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
+ Coverage   77.21%   77.48%   +0.26%
==========================================
  Files           6        6
  Lines         338      342       +4
==========================================
+ Hits          261      265       +4
  Misses         77       77

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fixed a bug when URLString like "https://www.google.com?"

Thanks for the PR! However, due to its age it will not be merged. If the issue is still occurring feel free to open a new PR.


Bulk update stored procedure source code

Is your feature request related to a problem? Please describe.
After a DB column rename I have to update more than 100 stored procedures to reference the correct DB column name again. How can I achieve this in a semi-automated way, e.g. exporting, search/replace, importing?
Describe the solution you'd like
Most convenient solution would be a search/replace functionality on stored procedure source code.
Describe alternatives you've considered
Exporting DDL to external file, searching/replacing the content as needed. But then I fail how to import the changes again in a single file.
Additional context

Got a solution in T-SQL.
DECLARE @MyCursor CURSOR;
DECLARE	@DDLCode NVARCHAR(max);
BEGIN
    SET @MyCursor = CURSOR FOR
    select top 1000 REPLACE(REPLACE(REPLACE(object_definition(object_id(routine_name)), '.DeleteUser ', '.DeleteUser_Id '), '.ParentCategoryId', '.ParentCategory_Id'), 'CREATE PROCEDURE', 'ALTER PROCEDURE')
    	from INFORMATION_SCHEMA.ROUTINES
        where ROUTINE_NAME like '%ById'

    OPEN @MyCursor
    FETCH NEXT FROM @MyCursor
    INTO @DDLCode

    WHILE @@FETCH_STATUS = 0
    BEGIN
      --print @DDLCode;
      EXECUTE sp_executesql @DDLCode;

      FETCH NEXT FROM @MyCursor
      INTO @DDLCode
    END;

    CLOSE @MyCursor ;
    DEALLOCATE @MyCursor;
END;


GH-2552: Fix Builder Argument Types maxLength etc

Resolves https://github.com/spring-projects/spring-amqp/issues/2552
Variables map to Erlang ints which are 8 bytes, so need to be long in Java.

@garyrussell ,
is this supposed to be back-ported to 3.0.x ?

No; SuperStreamBuilder doesn't exist there.
I suppose we could back port the QueueBuilder part, but nobody has complained about it so we can defer that, I think.

Also, maxLength is messages there (Vs. bytes with streams) so it's probably less of an issue for queues.


Fixed code to "Using an existing managed identity"

Description
This is a bug in the sample code in README.md.
When using an existing User Assigned Managed Id, it uses "subscription().Id" in the Subscription Id specification, which is incorrect.
The correct value is "subscription().subscriptionId".
Updating an existing module

[x] This is a bug fix:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.
[ ] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[ ] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

The CI failure is due to the CI using a newer version of Bicep, which was released a few days ago. Do you mind update Bicep CLI on your machine and run brm generate? Alternatively, you can configure a workflow  in your fork to have it generate files automatically.

build system has been updated so README.md can be updated without regenerating main.bicep.


How to generate a schema file from an existing MySQL database

How to generate a schema file from an existing MySQL database
My project is an old project and already exists in the database
What should I do to migrate to ENT now.

Take a look at:
https://github.com/ariga/entimport


GitHub UI. Period: Dropdown is broken.

I am not sure it is right issue track but.
I try using GitHub UI -->  dropdown to select 1 month, day,.. but it shows only 1 week duration.

It works now.


Updating repository name

As discussed earlier, when merging the reference viewer branch we should rename this repository to something like:

glTF-Sample-Viewer
glTF-Sample-Renderer

I suggested "Viewer" originally, but would also consider "Renderer" - per https://github.com/KhronosGroup/glTF-WebGL-PBR/issues/132 I'm hoping that the API will be fully useable programmatically, and the UI viewer is simply a 'demo' showing use of the sample renderer. Fine either way.
Aside from other connotations of reference vs sample, the word "Sample" also makes it clearer that the project is not meant to be used as a production-grade engine, although use in QA tooling could be quite reasonable.
/cc @emackey @sbtron @UX3D-nopper

I'm fine with either of those names.
Would it be worthwhile to leave the old glTF-WebGL-PBR master branch here, possibly as an archived repo?  It doesn't have many features, but it is an example of possibly the most bare-bones implementation of a functioning PBR shader.  I still see comments on Twitter from folks learning PBR for the first time, who appreciate the minimalism here, for ease of learning the most basic principles.
But if it's better to make everyone learn the newer, more rigorous sample, I'm OK with that too.

Suggest glTF-Sample-Viewer as the main focus is on viewing.


[JENKINS-62290] Make Email Extension Plugin accesible to Jenkins.MANAGE

JENKINS-62290
This is using the new Jenkins.MANAGE permission. It is intended to grant an intermediate access level to global configuration for non admins but more than read users.
The groovy scripts shouldn't be a problem because of the script-security usage.
(I've also updated the the minimum Jenkins core LTS including the Jenkins.MANAGE and updated to a newer parent pom).

Build is failing with a compilation error:
[ERROR] /home/jenkins/workspace/Plugins_email-ext-plugin_PR-208/src/main/java/hudson/plugins/emailext/ExtendedEmailPublisherDescriptor.java:[821,23] cannot find symbol
symbol:   variable MANAGE
location: class jenkins.model.Jenkins

Because it's running with -Djenkins.version=2.164.1, not sure why (a bug in the pipeline?)

Build is failing with a compilation error:
[ERROR] /home/jenkins/workspace/Plugins_email-ext-plugin_PR-208/src/main/java/hudson/plugins/emailext/ExtendedEmailPublisherDescriptor.java:[821,23] cannot find symbol
symbol:   variable MANAGE
location: class jenkins.model.Jenkins

Because it's running with -Djenkins.version=2.164.1, not sure why (a bug in the pipeline?)

That's part of the recommendedConfigurations() in buildPlugin. https://github.com/jenkins-infra/pipeline-library/blob/master/vars/buildPlugin.groovy#L258

Hi @slide I have addressed your requests (new test and restore the three other). Regarding 2.222.1, that LTS is required to build this PR.

Well, we may not be able to merge this for a while then. I need to review the plugin installation stats and see how many people are still on 2.164.x.

Hi @slide I have used reflection to avoid core bump


twilio sms - update external links for azure docs

A few additional updates for this doc. There were 3 external links that were not properly linked. I just updated.
cc: @tonybaloney

@aprilspeight : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @aprilspeight - This pull request was opened in the public repository. The content has a private repo, and PMs usually work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!

#sign-off sorry for the delay.


Release candidate 1.0

This PR prepares the package for release 1.0, to follow immediately upon the publication of the pyam manuscript in Open Research Europe.
The PR implements the following changes:

[x] remove all functions marked as deprecated and update tests accordingly
[x] clean up the release notes to start with a clean slate
[x] update the Readme and docs to reference the new manuscript

The docs with this PR are rendered at https://pyam-iamc.readthedocs.io/en/release-rc_1.0/

Was a bit fiddly to get that all into one line, edited the badges for groups.io and Slack to make it work...

Published successfully on test.pypi.org, see https://test.pypi.org/project/pyam-iamc/1.0.0/


⚠ Bofrost.at has degraded performance

In 32df430, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1726 ms

Resolved: Bofrost.at performance has improved in d7b037e.


drivers: gpio: sam0: Add interrupt support

This adds interrupt support to the SAM0 GPIO driver.  This is heavily
inspired by @nzmichaelh work in #5715.  The primary difference
from that implementation is that here the External Interrupt
Controller (EIC) is separated out into an interrupt controller driver
that is less tightly coupled to the GPIO API.  Instead it implements
more of a conversion from the EIC's own odd multiplexing to a more
traditional port and pin mask IRQ-like callback.  This ends up
simplifying both the EIC driver and the GPIO interface to it.
The EIC driver still makes some assumptions about how it will be used:
mostly it assumes exactly one callback per port.  This should be fine
as the only intended user is the GPIO driver itself.
This has been tested with some simple programs and with
tests/drivers/gpio/gpio_basic_api on a SAM D21 breakout and an
adafruit_trinket_m0 board.  It has only been compile tested for a
SAM D20, since I don't actually have any hardware for it, but the
EIC should be identical on both per the manuals.
Signed-off-by: Derek Hageman<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3522e05). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #14005   +/-   ##
=========================================
  Coverage          ?   52.23%
=========================================
  Files             ?      307
  Lines             ?    45448
  Branches          ?    10515
=========================================
  Hits              ?    23742
  Misses            ?    16912
  Partials          ?     4794

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 3522e05...eb58418. Read the comment docs.

Is there someone specific I should be putting for code owners for the DTS bindings?  It doesn't seem like most of that tree is assigned at all.

I can confirm that all tests from tests/drivers/gpio/gpio_basic_api succeed to run on samr21_xpro. (I've used the adjacent pins PB22 & PB23 and bridged them with a jumper)
However, when I try samples/basic/button I don't get an interrupt when pressing the button on the board. Maybe a configuration issue?

I still think it would be better to keep the file at a standard include location instead of having to introduce a new include directory for every platform.

I agree, it felt rather odd to add that.  However, I'd argue against putting it in the main include/ directory, as this isn't intended to be a user exposed API.  The way I did it was just the result of being "consistently inconsistent" with the other cases where something similar was done (e.g. the STM32 EXTI).  It seems like something to fix that properly would be a tree wide change (e.g. add something to the zephy_library() cmake macro?), since there's probably a need for internal driver includes that cross the normal directory boundary.  Another option would just be changing the include to "../interrupt_controller/sam0_eic.h", but that felt worse.

However, when I try samples/basic/button I don't get an interrupt when pressing the button on the board. Maybe a configuration issue?

I will try and reproduce this, since I didn't try that specific example.  But a quick skim the main difference appears to be setting the majority vote filter (i.e. GPIO_INT_DEBOUNCE) which has some implications for EIC clocking.  So if I don't get to it immediately, you can try setting that in samples/basic/button and see if that changes the outcome.

Updated with a mapping of the PA24-31 weirdness.  If you have a chance, try again now.  Regardless, thanks for the testing!

Thank you, now it works as expected!
It appears to be similar as to what RIOT is doing for different SAM MCUs.

Hmm yeah, looks similar.  Good to know there's weirdness across multiple chip families!
I suppose if there's ever support added for different EIC versions then this would need a similar generalization.  The only odd part would be the reverse mapping of the mask (at least without a couple of big loops).

Huh, neat.  I hadn't really looked beyond their M0+ lines, so I didn't know they where re-using the same peripheral interfaces.  Also glad to know somebody's finding some use for this stuff :smile:
I guess that's some more motivation for me to make some time to work on rebasing/updating #6306 since a quick skim looks like the DMAC is the same.
Either way, yeah I'll separate out the mapping somehow.  I'll probably just do what you suggested and move the two mapping functions to the soc.c.  The SAME54 looks to have it's own new set of weirdness, so a default one doesn't make much sense if that's coming.

Nice! I'll try it tomorrow on the same54, it might just work.
Minor nitpick: I'd put sam0_eic_channels into a separate header like sam0_eic_priv.h, just so it doesn't clutter up that file with generated code.

Yeah, that's a good call, I'll move it out of the actual code.
Unfortunately, I don't think it will work as-is on the SAME54.  Outside of the usual clocking changes: it looks like there they assign one NVIC line per EIC line, while the SAMD21/20 have one NVIC line for the whole thing.  So it'd need to make the IRQ handling conditional on the EIC line selected and connect all 16 of them during init.  If I have some time I'll take a stab that.

I works on the SAME54 with only minor modifications!

@benpicco I guess samples/subsys/usb/hid-mouse should work now?


Limiting length of STRING column

Updated strings.md to list STRING(n) as the primary form for limiting length on a string column.

This change is

LGTM

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

@knz correct me if I'm wrong, but I believe "The precision must be equal to or greater than the scale" for a DECIMAL.

@nvanbenschoten, you're right that precision can be equal to scale, but the behavior seems strange to me:
root@:26257> create table decimals (a DECIMAL(5,5));
CREATE TABLE
root@:26257> show columns from decimals;
+-------+--------------+-------+----------------+
| Field |     Type     | Null  |    Default     |
+-------+--------------+-------+----------------+
| a     | DECIMAL(5,5) | true  | NULL           |
| rowid | INT          | false | unique_rowid() |
+-------+--------------+-------+----------------+
root@:26257> insert into decimals values (DECIMAL '0.12345');
INSERT 1
root@:26257> select * from decimals;
+--------+
|   a    |
+--------+
| 1234.5 |
+--------+

What's happening there? Bug?

Definitely looks like it, I'll investigate.


Add test to perform extraction on patient registration

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes [link to issue]
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

@DebbieArita going to merge this for now, please lmk if my assumption about failing is incorrect, otherwise we can revisit when we get updated content and remove the exception expectation at that point


fix(adding-interactivity): fixed Render and Commit link label

Fixes the label of the link that navigates to Render and Commit.
The label says that React updates the UI in two phases, however the page it links to affirms multiple times that the phases are three (Trigger, Render, Commit).

I'm not super happy with the wording on that phase, but "Trigger" isn't really meant as a phase, it just refers to kicking off the process.

I'm not super happy with the wording on that phase, but "Trigger" isn't really meant as a phase, it just refers to kicking off the process.

Ah, got it. Maybe we could opt for something like "Request" or "Queue".


Adds operachromiumdriver as a Cask.

operachromiumdriver is used for selenium automation of the
Opera browsers.

Thanks for the tip.  I added the checksum, this cask should be good to go!

Thank you for the contribution. It needed some fixes, so they were made in 381eca6f4d0f1a907b30ecdaa41d9aeb8fd093fe. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Thanks!


valaclinter directory path appended twice for file

Hi , i'm developping a linter for vala. Here is the error I get :
SublimeLinter: chdir not set or invalid, using
C:\Users\user\Documents\development\github.com\mparaiso\mongodb-vapi\examples
SublimeLinter: valac: valalint-test.vala ['C:\\msys64\\usr\\local\\bin\\valac.exe', '-C']
SublimeLinter: valac output:
Compilation failed: 1 error(s), 0 warning(s)
error: Unable to map file /c/Users/user/Documents/development/github.com/mparaiso/mongodb-vapi/examples/C:\Users\user\Documents\development\github.com\mparaiso\mongodb-vapi\examples\valalint-test.vala': Failed to open file '/c/Users/user/Documents/development/github.com/mparaiso/mongodb-vapi/examples/C:\Users\user\Documents\development\github.com\mparaiso\mongodb-vapi\examples\valalint-test.vala': open() failed: No such file or directory

(valac:7852): GLib-CRITICAL **: g_mapped_file_get_length: assertion 'file != NULL' failed

It seems the filepath is appended twice : once the linux filepath then the window filepath
 /c/Users/user/Documents/development/github.com/mparaiso/mongodb-vapi/examples/C:\Users\user\Documents\development\github.com\mparaiso\mongodb-vapi\examples\valalint-test.vala

here is how my linter.py file looks like :
from SublimeLinter.lint import Linter, util
class Valac(Linter):
    """Provides an interface to valac."""

    syntax = 'vala'
    cmd = 'valac -C '
    executable='valac'
    version_args = '--version'
    version_re = r'(?P<version>\d+\.\d+\.\d+)'
    version_requirement = '>= 0.0.1'
    regex = r'^.+?:(?P<line>\d+)\.(?P<col>\d+).+?: (?:(?P<error>error)|(?P<warning>warning)): (?P<message>.+)'

    multiline = False
    line_col_base = (1, 1)
    tempfile_suffix = "-"
    error_stream = util.STREAM_BOTH
    selectors = {}
    word_re = None
    defaults = {}
    inline_settings = None
    inline_overrides = None
    comment_re = None

Edit: If I could pass only the filename to the linter I think it would help, instead of the fullpath to the file , how could I do that ? thanks.

Please discuss this in SublimeLinter3 itself, not the individual linters.

Fine, so where do people get help when developing their own plugin ? because it's precisely what I'm doing here.

SublimeLinter-pylint is a linter among others, not the actual framework to develop the linters.
https://github.com/SublimeLinter/SublimeLinter3 is the place to use !

Understood


Improve Veracruz Dockerfile

Improve the Dockerfile to make it build faster and make the produced image lighter.

@basmaelgaabouri I think you had specific suggestions for how the Dockerfile could be improved, right?  Can you spell them out?
@ShaleXIONG I think we have also discussed the size of the resulting Docker image that is being produced in the past.  Would @basmaelgaabouri's suggested changes be beneficial in your opinion?

I would start by using a multistage build, it will for sure help reduce the build time, and more cleanup to reduce the size

I am not an expert of Dockerfile, I think @basmaelgaabouri has more experience than me. It will be nice @basmaelgaabouri reduces the size of the final image.

Reducing the Docker size will also help with the CI.  @basmaelgaabouri, can you help improve the Docker situation?  Both reducing the image size and moving the image so that it is hosted on Docker hub are good ideas.

Resolved by recent changes in the Veracruz Docker image repository.

Resolved by recent changes in the Veracruz Docker image repository.


feat: add the indices to the optional sidebar components

Fix #316
It has been requested a long time ago but better late than sorry ;-)
So I created a small indices.html and the according _indices.scss. It is build to be added in the left_sidebar_end:
"left_sidebar_end": ["indices.html", "sidebar-ethical-ads.html"]

I decided not to add it in our default documentation, maybe we should ? In the meantime here are 2 screenshot from one of my documentation:

@choldgraf I added the requested extra page. There is now a warning in the build (I think it's unrelated to my modification) but I didn't find where the transform member is called. Do you have an idea ?


David integration to keep track of our dependencies

Since we've added Travis integration, we might expand our badge collection with a David badge to keep track of our dependencies.
badge
tracking page

#202


Update ConcavePolygonShape2D.xml

Fix a typo that said ConvexPolygonShape2D instead of ConcavePolygonShape2D. ConcavePolygonShape2D is hollow and both this page and ConvexPolygonShape2D page confirms it.

The same error is present in the 3D shape, can you fix that as well? Please update your code with git commit --amend to not create an additional commit

The same error is present in the 3D shape, can you fix that as well? Please update your code with git commit --amend to not create an additional commit

Please also amend the commit message to be more explicit. See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-messages-with-readability-in-mind. Here it could be "doc: Fix typo in ConcavePolygonShape2D/3D description"

Please squash your commits into one, see here, and use git push --force to not add a merge commit

Sorry for inconvenience. This should be all good now.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.1.4. While the change is minor and will affect translations, this was blatantly and misleadingly wrong before. So I concluded that it was worth it.


Code isuue

if scale >= 2:
output = Subpixel(64, (3,3), r = 2,padding='same',activation='relu')(output)
if scale >= 4:
output = Subpixel(64, (3,3), r = 2,padding='same',activation='relu')(output)
if scale >= 8:
output = Subpixel(64, (3,3), r = 2,padding='same',activation='relu')(output)
All the three lines are same.
Isn't it
output = Subpixel(64, (3, 3), r=scale, padding='same', activation='relu')(output) ?

I think when working on the paper, the paper was not explicitly mentioning what do to when we need to output in scales above 4 or 8. I figured that using the pixel shuffle upscaling a number of times for higher upscales won't hurt. I verified it by changing r = 8 , or using the same operation 3 times. The final results are based on this final variant of code. The code only applies the same operation 3 times if scale is set to 8 , 2 times if 4 and 1 time is 2. I am looking into the paper again , to look at how to do higher level upscaling (progressively or imediately upscale to the scale)


Are some files missing?

I cloned the repository and tried to run tests with python3 -m tests but it couldn't manage to make them pass. Here is the traceback: https://pastebin.com/f1tZvVYP
Am I doing something wrong?

Lark supports converting grammars from Nearley: https://github.com/kach/nearley
It's an optional feature, so it's not there by default. If you want to test against it, you can clone it, or use git submodules to fetch it:
    git submodule sync -q
    git submodule update --init

OK thanks! Just learned a new thing :)


Individual Review

link to blog

I can't get to link on Hanlun's stuff above.

Oops, I linked localhost on accident, it should be fixed now. Sorry for the inconvenience

Regarding video.   I don't like the placement of the Captions, plus they are to much to read while you are click.

Observe Data, aka Collection  - This would be my fist Caption.   I would suggest coming up with your own names like Billy Baker and Mary Master Chef.  This will help get your people in personality.
1a. See saved recipe(s).   Arrow or perhaps Zoom data of interest.
Input new Recipe - This would be my 2nd Caption.  This is where you go to baker and animation.
2a. Observe ingredients in Oven.   Do this before Bake.
See Output - Arrow to the croissant
Review Data Update.

1.0/1 - Meets almost all requirements except two tiny parts, goes above and beyond on parts
For one Hanlun is missing one tiny part, he didn't show one or more parameter. Hanlun showed me this separate for his blog so I should be easy to add a picture of this. Also it would be better to show the output code alone with the output being shown on the website so users can understand which part below with each part of code. Also Hanlun's video is a YouTube video which goes against CPT requirements. I asked Hanlun about this and he said it is a YouTube video for easy access but he has a .mp4 file ready for college board.
Hanlun seems to have everything and went above and beyond on parts like his video where he showed high understanding of his sqlite.db files and how everything is connected. His blog has everything but nothing is above and beyond.
Hanlun seems to have everything but could add a few things to go above and beyond. Hanlun can add his commits and exactly what he did for each part of the project. This could help college board graders and others see what parts he is highly knowledgeable about. Hanlun could also make his page a bit more readable with tables.
Overall Hanlun's blog along with video show high understanding of the cpt requirements and meets requirements but is missing a few things.

Ishan you are paragraphing and generalizing with your comments.   I would like to see them more like what I shared.   This is "real" feedback.  The owner of code can adapt to meet CB objectives.  But we need to be strong in our opinions.  Truly, this is the basis of effective collaboration.

In-depth review of code screenshots:

Hanlun's first screenshot of input shows it on his frontend but needs to show code.
Your second screen shot of your sqlite.db is a different way of showing a list and shows good understand but the sqlite.db along with the code would be better.
Your third screenshot shows a function called Success shows the intended purpose well.
Your fourth screenshot shows sequencing, selection, and iteration using an if and for to go through get the current item and points.
Your fifth screenshot shows it calling success but shows what causes that to be called and the code above is needed.
Your sixth screenshot shows the output well and tells you the name of what you create and gets the image for it.


Package repository version doesn't include gallery

Hinders the ability to judge the package by its README for anyone skimming the package repository

Nor manual, etc.

Due to this issue the new package page also misses the pictures.


Creating azure privatelink for rebalancing service hangs

Running into problems upgrading existing infra with the newly added static IPs and aiven_azure_privatelinkfunctionality.

postgresl service deployed previously without static IPs associated.
tf plan/apply which

creates and associates static IPs to the existing service
creates an aiven_azure_privatelink associated to said service

Proper dependencies are in-place between the aiven_pg and aiven_azure_privatelink resources.
Modification of the aiven_pg service completes.  Actual service enters into a rebalancing state asynchronously in the background.
The aiven_azure_privatelink proceeds to create but never completes, even after the associated service completes its rebalance.
Manually removing the azure privatelink from the service, waiting for it to delete, and re-running terraform successfully creates it.
Not sure this is expected or a bug on the server side. I'd expect the privatelink to eventually complete after rebalancing. If its the case that this needs to be phased across 2 terraform runs (phase1: associate static IPs + rebalance, phase2: create privatelink), then we shoudl call it out somewhere in provider docs.

Should in principle finish in one apply. Do you have an reproducing example?

Ok after talking to the team, this seems to be API related and has to be fixed in the backend. The mitigation for now is to apply updates in 2 phases like you described, but let me quote here for reference:

Not sure this is expected or a bug on the server side. I'd expect the privatelink to eventually complete after rebalancing. If its the case that this needs to be phased across 2 terraform runs (phase1: associate static IPs + rebalance, phase2: create privatelink), then we shoudl call it out somewhere in provider docs.

This is a race condition that is more likely to occur with TF and only present for azure privatelink. We will create an internal issue to track this problem and link it to your support conversation so you can receive updates as and when they happen.

Hey @gandelman-a ,
sorry i forgot to ping, this issue has been closed in the backend for a while now! I'll close this issue now; if you still experience this error please consider it a bug and reopen this ticket!
Thanks


fix: refresh TrainLoc's token when auth_revoked

Per discussion in Slack, TrainLoc needs to refresh its Firebase token when it receives the auth_revoked event. It seems previously that Firebase would close the connection when the token expired, but they stopped doing this at some point yesterday, causing TrainLoc to stop receiving events hourly and require a restart. ServerSentEventStage is injected with TrainLoc.Utilities.FirebaseUrl.url/0, causing it to generate a new token when ServerSentEventStage.refresh/1 is called by TrainLoc.Manager.handle_events/3. Note that BoardingStatus.ProducerConsumer already handles this properly, this code was adapted from that implementation:
https://github.com/mbta/commuter_rail_boarding/blob/01f4308ecbe5298dacff2281dfc9de78161b76bb/apps/commuter_rail_boarding/lib/boarding_status/producer_consumer.ex#L31-L33

Looks like it worked!


Fix: Websocket upgrade when used with Http.collectM

To resolve #623

🚀🚀🚀 Benchmark Results
Concurrency: 256
Requests/sec: 431355.36

Codecov Report

Merging #649 (9c9f2cf) into main (94a9212) will increase coverage by 2.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #649      +/-   ##
==========================================
+ Coverage   58.60%   61.28%   +2.68%
==========================================
  Files          59       59
  Lines        1401     1400       -1
  Branches       55       55
==========================================
+ Hits          821      858      +37
+ Misses        580      542      -38

Impacted Files
Coverage Δ

.../scala/zhttp/service/server/WebSocketUpgrade.scala
100.00% <ø> (+83.33%)
:arrow_up:

zio-http/src/main/scala/zhttp/http/HExit.scala
74.46% <0.00%> (+2.12%)
:arrow_up:

zio-http/src/main/scala/zhttp/http/Response.scala
40.54% <0.00%> (+2.70%)
:arrow_up:

...tp/src/main/scala/zhttp/socket/SocketDecoder.scala
4.76% <0.00%> (+4.76%)
:arrow_up:

zio-http/src/main/scala/zhttp/socket/Socket.scala
8.00% <0.00%> (+8.00%)
:arrow_up:

...o-http/src/main/scala/zhttp/socket/SocketApp.scala
36.36% <0.00%> (+27.27%)
:arrow_up:

...ala/zhttp/service/server/ServerSocketHandler.scala
30.76% <0.00%> (+30.76%)
:arrow_up:

...p/src/main/scala/zhttp/socket/SocketProtocol.scala
47.82% <0.00%> (+47.82%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 94a9212...9c9f2cf.


example codesystem needs to be clearly identified

http://profiles.ihe.net/ITI/mCSD/CodeSystem/mcsd-example-hierarchy
I think the current trend is to to have these marked as example codeSystems.

This has been updated and the QA is now happy.  The change involved making the URL use example.org for the CodeSystem.
https://github.com/IHE/ITI.mCSD/blob/main/input/fsh/example-mcsd.fsh#L36


Set default value of PreserveHost for new routes to true.

What this PR does / why we need it:
Currently it's impossible as far as I know to integrate with cert-manager due to the default value of PreserveHost being false.
By changing the default to true we align with the nginx ingress controller and enable seamless integration with cert-manager.
Which issue this PR fixes:
This PR doesn't fix any open issues, but here are some references to the discussion:
#162
#163
jetstack/cert-manager#958
Special notes for your reviewer:
I realise this may not be the ideal long term solution for this and it has been discussed in #162 that a default KongIngress resource would be a good solution.
Also cert-manager is exploring an ingress template solution in jetstack/cert-manager#1097.
However this is such a small change to enable interoperability in the mean time that I thought it worth proposing the pull request.

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

Codecov Report

Merging #248 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #248      +/-   ##
=========================================
- Coverage   20.11%   20.1%   -0.01%
=========================================
  Files          24      24
  Lines        2516    2517       +1
=========================================
  Hits          506     506
- Misses       1963    1964       +1
  Partials       47      47

Impacted Files
Coverage Δ

internal/ingress/controller/kong.go
5.14% <0%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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

I've changed the base from master to next as this will be a breaking change.

Sure thing, I've opened #257 based on next, so I'll close this one.


Fix facet filters

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
yes

Related Issue
Fix #505

Need Doc update
yes

Describe your change
This change introduces a new FacetFilters class to replace the List<String> type of the facetFilters parameter used in the Query and in the ConsequenceParams (extending Query).
What problem is this fixing?
Currently, because a user can only define facetFilters as a list of strings, filters can only be ANDed. To be able to OR filters, the facetFilters should accept lists of list of strings as well. This is what this change is all about. Using the new FacetFilters abstract class that is introduced, and its FacetFiltersAsListOfString/FacetFiltersAsListOfList concrete types, one can easily set the facetFilters parameter of a Query/ConsequenceParams using List<String> or List<List<String>> types. As any other multi-type object of this Java client, deserialization of object returns an Object that can be cast to one of the concrete classes, however, this shouldn't be used that often as the facetFilters parameter is only sent but never returned by the API, but anyway, it's doable.
@ElPicador As it's one of the rare times I add custom (de)serializers myself, could you tell me if there's better/more idiomatic ways to write them?

@ElPicador Forgot to commit the tests actually. Will do.

@ElPicador Forgot to commit the tests actually. Will do.

I see what you did there :suspect:


Adding OperationEnum include to be able to void payments

Adding OperationEnum include to be able to void payments
Solves Issue #8

I’m currently on vacations. I’ll review this when I get back and I’ll
improve the integration test.
On Sun, 13 Feb 2022 at 19:36 Clement Gutel @.***> wrote:

@.**** commented on this pull request.
In QuickBooksSharp/Services/DataService.cs
https://github.com/better-reports/QuickBooksSharp/pull/9#discussion_r805425824
:

     {

       var res = await _client.PostAsync<IntuitResponse>(new Url(_serviceUrl).AppendPathSegment(GetEntityName(typeof(TEntity))), e);

       var url = new Url(_serviceUrl).AppendPathSegment(GetEntityName(typeof(TEntity)));

       if (include != OperationEnum.Unspecified)

       {

           url = url.SetQueryParam("include", include);

@fbridger https://github.com/fbridger why is the param called
'include'. As far as I can see in the docs, it should be called 'operation'
—
Reply to this email directly, view it on GitHub
https://github.com/better-reports/QuickBooksSharp/pull/9#pullrequestreview-881053649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABROYOD77L4KT7BCYVYRMQTU3AW73ANCNFSM5M7DSKBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--

Saludos,
Federico

Checked in the documentation and verified it should be INCLUDE
https://developer.intuit.com/app/developer/qbo/docs/api/accounting/most-commonly-used/payment#void-a-payment
Example:
POST /v3/company//payment?operation=update&include=void
Content type:application/json
Production Base URL:https://quickbooks.api.intuit.com
Sandbox Base URL:https://sandbox-quickbooks.api.intuit.com
On Sun, 13 Feb 2022 at 23:54 Federico B @.***> wrote:

I’m currently on vacations. I’ll review this when I get back and I’ll
improve the integration test.
On Sun, 13 Feb 2022 at 19:36 Clement Gutel @.***>
wrote:

@.**** commented on this pull request.
In QuickBooksSharp/Services/DataService.cs
https://github.com/better-reports/QuickBooksSharp/pull/9#discussion_r805425824
:

     {

       var res = await _client.PostAsync<IntuitResponse>(new Url(_serviceUrl).AppendPathSegment(GetEntityName(typeof(TEntity))), e);

       var url = new Url(_serviceUrl).AppendPathSegment(GetEntityName(typeof(TEntity)));

       if (include != OperationEnum.Unspecified)

       {

           url = url.SetQueryParam("include", include);

@fbridger https://github.com/fbridger why is the param called
'include'. As far as I can see in the docs, it should be called 'operation'
—
Reply to this email directly, view it on GitHub
https://github.com/better-reports/QuickBooksSharp/pull/9#pullrequestreview-881053649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABROYOD77L4KT7BCYVYRMQTU3AW73ANCNFSM5M7DSKBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--

Saludos,
Federico

--

Saludos,
Federico

Ah, I was looking at the docs to void an invoice.
https://developer.intuit.com/app/developer/qbo/docs/api/accounting/all-entities/invoice#void-an-invoice
The API seems to be inconsistent. It uses a parameter called operation to void invoice and one called include to void payments.
I'll get your PR merged in, but the fact different API endpoints use different parameters could be a problem in the long run. I'm not sure what the best way to manage this is...


Please make this page available in Spanish (ES-ES)

🚨 Looks like you arrived from the internal review site
🚨 Do not enter Microsoft confidential information here
Please make this page available in Spanish (ES-ES)

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

ID: 7a667d12-913e-e88d-f4a4-953c60814616
Version Independent ID: 909d3d3b-25fd-5e7c-6da5-c64be452462e
Content: Viva Insights insights for managers
Content Source: Viva/insights/use/pm-home.md
Service: viva
Sub-service: viva-insights
GitHub Login: @madehmer
Microsoft Alias: helayne

Thank you for submitting this localization issue.
I submitted a translation bug: 274380
I'm closing this issue because the Global Services team will handle it from here.


Faster page load times

At the moment at every click: Load wrapper -> Wrapper creates iframe and loads page html -> Init websocket connection -> Load page details using websocket api
It could be much faster by sharing the wrapper and the websocket connection between and only change the inner frame html.
Need a wrapper api command that allows change the url of the page.

Reference: https://github.com/HelloZeroNet/ZeroNet/issues/71.


Helpers nodes as UI dropdown nodes

Not urgent, but would be nice eventually

Would be nice for the Paths nodes as well


Fixing tooltip UI inside Form field

Fixes #2752

Could you add a changeset and a test for this fix please

Changeset and test case has been added for this fix


Optimize multiple breakdowns into fewer db calls

Change the way breakdowns are queried. Previously, each breakdown made 3
queries. Each rollup has 3 breakdowns. Each series of rollups (whether
Rollups or as part of Release) could have many days for a significant number of
queries. Multiple rollups could be request at once, causing a noticable delay
when requesting rollups.
Now, 3 queries are used to get the breakdowns for ALL dates. With the 3
breakdowns per rollups, 9 queries are used to craft all the necessary rollups.
Also made a few small readability changes.

Bleh commits on top of open PRs.  Maybe should have just made it all one PR...


Roll Flutter Engine from 7ffa1355f718 to b41f9f7138c3 (22 revisions)

https://github.com/flutter/engine/compare/7ffa1355f718...b41f9f7138c3
2023-06-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from aBEyk5N2HlwOqvpwo... to BuzniDS9u_hdghV5f... (flutter/engine#42970)
2023-06-18<EMAIL_ADDRESS>Roll Skia from dc2e0ebbd045 to f489be63a30a (1 revision) (flutter/engine#42968)
2023-06-18<EMAIL_ADDRESS>Roll Skia from 85b4788bc3f1 to dc2e0ebbd045 (1 revision) (flutter/engine#42967)
2023-06-18<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from -PsA0LDB2FdS_4_vK... to xS5RzQLfpMEK1rNEd... (flutter/engine#42966)
2023-06-18<EMAIL_ADDRESS>[Impeller] Use 32 bit for positional stuff in the rrect blur (flutter/engine#42797)
2023-06-18<EMAIL_ADDRESS>Revert "[Impeller] correct default PSO pixel format and sample count." (flutter/engine#42962)
2023-06-18<EMAIL_ADDRESS>[Impeller] dont use concurrent runner to decode images on Android. (flutter/engine#42944)
2023-06-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from I1G8Ll0KcxmAoMT8u... to aBEyk5N2HlwOqvpwo... (flutter/engine#42964)
2023-06-18<EMAIL_ADDRESS>Roll Skia from c56f38d79fe0 to 85b4788bc3f1 (2 revisions) (flutter/engine#42963)
2023-06-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from Au1gd3XLMuSt1Z4oh... to -PsA0LDB2FdS_4_vK... (flutter/engine#42959)
2023-06-17<EMAIL_ADDRESS>Roll Dart SDK from d1faf69c8253 to 5a9e97eebbed (1 revision) (flutter/engine#42958)
2023-06-17<EMAIL_ADDRESS>Roll Skia from 2a9e11fd9d37 to c56f38d79fe0 (1 revision) (flutter/engine#42955)
2023-06-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from JMStJe6efcLuWMT_I... to I1G8Ll0KcxmAoMT8u... (flutter/engine#42954)
2023-06-17<EMAIL_ADDRESS>Roll Dart SDK from 1a04f451a934 to d1faf69c8253 (1 revision) (flutter/engine#42953)
2023-06-17<EMAIL_ADDRESS>Roll Dart SDK from b3bcbdf7de8d to 1a04f451a934 (1 revision) (flutter/engine#42951)
2023-06-17<EMAIL_ADDRESS>Roll Skia from dc11c7ddb33f to 2a9e11fd9d37 (1 revision) (flutter/engine#42950)
2023-06-17<EMAIL_ADDRESS>[Impeller] Print malioc errors on CI (flutter/engine#42906)
2023-06-17<EMAIL_ADDRESS>Roll Dart SDK from 8eaed3382237 to b3bcbdf7de8d (10 revisions) (flutter/engine#42949)
2023-06-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from -NW1eatBbmjvLaIcV... to Au1gd3XLMuSt1Z4oh... (flutter/engine#42948)
2023-06-17<EMAIL_ADDRESS>Fix analyzer warnings (prefer_final_in_for_each is now the default) (flutter/engine#42943)
2023-06-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from fXVcR5tdj5wSd_OUz... to JMStJe6efcLuWMT_I... (flutter/engine#42942)
2023-06-16<EMAIL_ADDRESS>Roll Skia from 64fa632d3b01 to dc11c7ddb33f (3 revisions) (flutter/engine#42939)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from -NW1eatBbmjv to xS5RzQLfpMEK
fuchsia/sdk/core/mac-amd64 from fXVcR5tdj5wS to BuzniDS9u_hd
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Automatically open a pre-configured project file when QGIS starts

Context
Breaking out of @geohacker's comment here -
We will create a project file with all the layers and configuration we need, and then have QGIS automatically load the project file on startup.
Proposal
The cleanest way is to probably write a startup script that QGIS will execute on startup: https://docs.qgis.org/3.28/en/docs/pyqgis_developer_cookbook/intro.html#running-python-code-when-qgis-starts
In this script, we can handle opening of a project file.
Questions:

Should we add the project file to this repository and have it built as part of this image? OR
Should we manage the project file separately and have it at some URL that gets pulled in at build time? OR
Should the project file live at a URL and get pulled in at run-time?

Anyways, we'll start off just making sure the startup script works to open a project file, and then we can figure details.
cc @yuvipanda @geohacker
Updates and actions
No response

@geohacker - in your notes above you recommend writing a python script to load the project file on startup. Looking at the qgis docs, one can also start qgis with a project file with like qgis -p <path/to/project.file> - is there a good reason to not just use that? I can see that putting it in a python script gives us more flexibility, but just trying to understand if we can start off just using the CLI flag to open a project file when starting QGIS.
From the output of qgis --help:
[-p, --project projectfile]	load the given QGIS project


Gomod dependency gets downgraded after manual update

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
gomod
Package manager version
go 1.19
Language version
go 1.19
Manifest location and content before the Dependabot update
No response
dependabot.yml content
https://github.com/osbuild/images/blob/main/.github/dependabot.yml
version: 2
updates:

  # Maintain dependencies for GitHub Actions
  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
      interval: "weekly"
      time: "04:00"
    open-pull-requests-limit: 5
    rebase-strategy: "disabled"

  # Maintain dependencies for Go
  - package-ecosystem: "gomod"
    directory: "/"
    schedule:
      interval: "daily"
      time: "04:00"
    groups:
      go-deps:
        patterns:
          - "*"  # group all dependency updates into one PR
    open-pull-requests-limit: 1
    rebase-strategy: "auto"

Updated dependency
github.com/containers/image/v5
What you expected to see, versus what you actually saw
What I expect: dependabot should not be downgrading a dependency
What happens: dependabot PRs downgrade a dependency every time it runs
Description
About a week ago, a dependabot PR pulled in updates that were incompatible with a specific dependency.  I manually updated the dependency from v5.22.1 to v5.29.1 and included it in the PR.

dependabot PR: https://github.com/osbuild/images/pull/390
Breaking commit: https://github.com/osbuild/images/commit/c8d9c5eff6928931faf7ff9e92d09d4f53572d1e
Manual dependency update commit: https://github.com/osbuild/images/commit/13622d01ba62ec9a4cb946c5c82ac97bf1bdf0b7

Since then, every dependabot PR reverts the change to v5.22.1 of github.com/containers/image/v5, but fails because of the incompatibility.
Most recent example: https://github.com/osbuild/images/pull/416 (see: https://github.com/osbuild/images/pull/416/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6L17)
See also:

https://github.com/osbuild/images/pull/397
https://github.com/osbuild/images/pull/401
https://github.com/osbuild/images/pull/403
https://github.com/osbuild/images/pull/406

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs

https://github.com/osbuild/images/pull/397
https://github.com/osbuild/images/pull/401
https://github.com/osbuild/images/pull/403
https://github.com/osbuild/images/pull/406

Smallest manifest that reproduces the issue
No response

Dependabot invokes the Go CLI to do updates, I can reproduce this outside of Dependabot like so:
$ git clone --depth=1 https://github.com/osbuild/images.git
Cloning into 'images'...
remote: Enumerating objects: 8250, done.
remote: Counting objects: 100% (8250/8250), done.
remote: Compressing objects: 100% (5604/5604), done.
remote: Total 8250 (delta 2400), reused 6776 (delta 2140), pack-reused 0
Receiving objects: 100% (8250/8250), 18.23 MiB | 8.98 MiB/s, done.
Resolving deltas: 100% (2400/2400), done.
Updating files: 100% (7194/7194), done.
$ cd images
$ go get<EMAIL_ADDRESS>go: downgraded github.com/Microsoft/hcsshim v0.12.0-rc.1 => v0.10.0-rc.3
go: downgraded github.com/containerd/containerd v1.7.0 => v1.6.19
go: downgraded github.com/containers/image/v5 v5.29.1 => v5.22.1
go: downgraded github.com/containers/storage v1.51.0 => v1.48.1
go: downgraded github.com/google/go-containerregistry v0.16.1 => v0.11.0
go: downgraded github.com/opencontainers/image-spec v1.1.0-rc5 => v1.1.0-rc.6
go: downgraded github.com/sigstore/fulcio v1.4.3 => v1.0.0
go: downgraded github.com/sigstore/rekor v1.2.2 => v1.0.1
go: downgraded github.com/sigstore/sigstore v1.7.5 => v1.4.4
go: downgraded github.com/sylabs/sif/v2 v2.15.0 => v2.8.3
$ grep image\/v5 go.mod
	github.com/containers/image/v5 v5.22.1

It looks like there are some competing dependencies:
$ go mod graph | grep github.com/containers/image/v5
github.com/osbuild/images<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Both of those depend on the same dependency, so I was curious what would happen if we tell Go to update both of them at the same time:
$ go get<EMAIL_ADDRESS><EMAIL_ADDRESS>go<EMAIL_ADDRESS>requires<EMAIL_ADDRESS>not<EMAIL_ADDRESS>
That seems to be the root cause of the issue. In grouped updates, Dependabot runs go get on each dependency individually so it doesn't surface this since the Go tooling found a way to make it work via downgrades.

Ah someone has noticed the same thing:

https://github.com/dependabot/dependabot-core/issues/8847

It seems the pre-release format has changed and it might be messing up the update. You can try ignoring v1.1.0-rc.6 of github.com/opencontainers/image-spec since it sorts larger than rc6 and then Dependabot will choose it to give to the go get command. That should avoid the downgrades.

Ah someone has noticed the same thing:

Thanks!  Sorry I missed that.

Duplicate of https://github.com/dependabot/dependabot-core/issues/8847


Update README to use correct ember install command

Thanks for making this addon.
During install, I just copy/pasted the command in the readme and it took me a while to figure out why ChartJS was not working as expected - I was using the old version from the repo you cloned.
Changed the README ember install command from
$ ember install ember-cli-chart
to
$ ember install ember-cli-chartjs

@sstephens ^^

@stevetarver good catch and thanks for the help.


Segment can't use Cutout, dropout, etc.

image can cutout, but mask image can't do the same process.

help!!!


errorbars buggy for large x values

using Makie, CairoMakie

start = 4e6
errorbars(start:start+9, randn(10), randn(10))

Things look good for smaller values of start; the error bars only start getting weird around start=1e5.

could be that this is because the coordinate transforms are done in Float32, at least that's usually what causes such "quantization" problems in Makie (GPU needs everything in Float32)


Proposal: frint-props

(Intended to be done outside of the monorepo)
Currently
We have frint-react package that ships with observe HoC and streamProps helper function that enables you to stream props to your base component.
It also gives you access to app instance, allowing you to generate your props having access to your providers too.
Problem
The streaming concept requires developers to understand RxJS and Observables, which is becoming a barrier to entry for beginners.
We can do more to make things easier.
Proposal
Given FrintJS aims to be rendering library agnostic, I propose two packages that can help developers write mostly pure functions and help avoid direct RxJS usage unless really needed for complicated use cases:

frint-utils: Exports pure functions for composing your props
frint-utils-react: Composes all functions to create a React HoC

Example usage
This code snippet is just for showing an example, do not take the API too seriously:
import React from 'react';

// just functions
import {
  startWith,
  withStore,
  withApp,
  shouldUpdate,
} from 'frint-props';

// this would use `observe` from `frint-react` interally
import { hoc } from 'frint-props-react';

// some action creators (Redux or frint-store)
import { incrementCounter } from '../actions/counter';

// your base stateless component
function MyComponent(props) {
  return <p>...</p>;
}

// compose as needed, and export
export default hoc(
  // defaultProps
  startWith({
    foo: 'foo value',
  }),

  // store (frint-store or Redux)
  withStore(
    state => ({ todos: state.counter.value }), // mapStateToProps
    { incrementCounter }, // mapDispatchToProps
    'store' // provider name (optional),
  ),

  // app
  withApp((app, props$) => {
    // return Object or Observable
  }),

  // should update or not
  shouldUpdate((prevProps, nextProps) => {
    return true;
  })
)(MyComponent);

Implentation
frint-react and RxJS will be used by the compose HoC internally, and should not result in adding to the bundle size significantly.
I am looking at a size around 1kb to 2kB max for the initial release.
Dependencies

[x] #390 would make it easier for this proposal to be implemented

Development has begun in this repository: https://github.com/frintjs/frint-props
Proposal will be closed once we have our first release on npm. We can then take it from there.


HTML Sorting

We're having trouble sorting on a column that contains HTML. We've noticed that this seems to only be happening when certain attributes are present. For example, the aria-label attributes prevent correct sorting.
See http://live.datatables.net/fuzapano/1/edit.

Hi,
I'd appreciate if you could post support requests in the DataTables forums as per the Contributing.md document that was linked on the new issue page.
The issue is the > character in the attribute. DataTables HTML removal isn't smart enough to strip that out at the moment. I need to have bit of a think about the best way of handling that...

Ah! I see. I've tried to encode these characters (so > instead of >) but it still didn't work.
Would you like me to post this issue in the forums or simply further issues I may have ?

Let's leave it here. I'll get confused if it is in two places :-).
Interesting that > didn't work. I need to look into how to improve the regex DataTables uses - possible strip out attributes inside HTML elements first.

Still looking into this, but it isn't proving to be easy unfortunately. I think probably I might have to provide an option that will give the ability to get the text using the browser's DOM parser, rather than regex. I don't want to make that the default as it is so much slower than string processing.


Missing parenthesis around in inside of a for loop

for ((x in a);;) {}

outputs
for (x in a;;) {}

which is invalid
https://jlongster.github.io/prettier/#{"content"%3A"for ((x in a)%3B%3B)%20%7B%7D%5Cn%5Cn%22%2C%22options%22%3A%7B%22printWidth%22%3A80%2C%22tabWidth%22%3A2%2C%22singleQuote%22%3Afalse%2C%22trailingComma%22%3Afalse%2C%22bracketSpacing%22%3Afalse%2C%22doc%22%3Afalse%7D%7D

This is trickier,
for (a=(a in b);;) {}

outputs
for (a = a in b; ; ) {
}

which is invalid.
So it's anything left of a for that isn't parenthesized already... Not super sure how to figure this out.


refactor: Drop JSON debugging columns from Kingdom Spanner DB

These are no longer needed now that Cloud Spanner has native protobuf support.

This change is


Making ASPP-Layer in DeepLab more generic

🚀 Feature
At the moment in the ASPP-Layer defined here the number of output channels is predefined as a constant, which is good for DeepLab but not if someone else - like me ;) , want to use it in another project, where another out-channel Nr. is required.
Also the number of "atrous rates" is fixed to three, which also could be sometime more or less dpeending on the notwork. Again these fixed values may make sense in DeepLab-Model but not necessarily in other type of model, which in my case is applied on Lidar data.
that means something like this would be at least IMO better:
class ASPP(nn.Module):
    def __init__(self, in_channels, atrous_rates, out_channel=256):
        super(ASPP, self).__init__()
        modules = []
        modules.append(nn.Sequential(
            nn.Conv2d(in_channels, out_channels, 1, bias=False),
            nn.BatchNorm2d(out_channels),
            nn.ReLU()))

        rates = tuple(atrous_rates)
        for rate in rates:
             modules.append(ASPPConv(in_channels, out_channels, rate))

        modules.append(ASPPPooling(in_channels, out_channels))

        self.convs = nn.ModuleList(modules)

        self.project = nn.Sequential(
            nn.Conv2d(5 * out_channels, out_channels, 1, bias=False),
            nn.BatchNorm2d(out_channels),
            nn.ReLU(),
            nn.Dropout(0.5))

This looks reasonable to me, and is backward compatible. Do you want to open a pull request for this?

Ok, thank, yes i will open a poll req.

Fixed via https://github.com/pytorch/vision/pull/2174


Rational.Simplify gets stuck in endless loop

When attempting to simplify a fraction using Rational.Simplify, the library seems to get stuck in an endless loop:
var numerator = Infix.ParseOrThrow("0.1*x");
var denominator = Infix.ParseOrThrow("x+x^2");
var fraction = numerator / denominator;

var x = Infix.ParseOrThrow("x");
var simplified = Rational.Simplify(x, fraction); // hangs

There must be something wrong with how we handle approximations like 0.1 there. Indeed, when replacing 0.1*x with x/10 simplify returns 1/(10 + 10*x) as expected - but of course that's a very different expression and thus simplification problem.

This is fixed in v0.22.0. Thanks for reporting!

Thanks for the fix. The above example now works fine. Unfortunately I still experience problems with slightly different problems, e.g.:
var numerator = Infix.ParseOrThrow("a + a*b + a^2*b + a*b^2");
var denominator = Infix.ParseOrThrow("a + a*b + a^2*b + a^2");

var fraction = numerator / denominator;

var simplified1 = Rational.Simplify(Infix.ParseOrThrow("a"), fraction); // result is a long expression
var simplified2 = Rational.Simplify(Infix.ParseOrThrow("b"), simplified1); // does not seem to finish

For this (and also the earlier example) the exact output is not that important to me, however it is important in my use case that the software does not hang unexpectedly.


sudo: Tweak the password prompt

This has been bugging me for a while; without the colon, it seems that something is missing and that I should wait for the rest of the prompt to appear.  Add a ": " to the end of the sudo prompt (as is usual) so this doesn't happen.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Tweak the password prompt to be closer to what other distros do.
Change Log

Change a parameter passed to the configure script to change the sudo prompt.

Does this affect the toolchain?

NO
Test Methodology

Untested.

Alright, will send another PR because changing the base from the web interface sort of made a mess here.

Alright, will send another PR because changing the base from the web interface sort of made a mess here.