threema-desktop: 1.0.3 -> 1.2.0

Description of changes
https://github.com/threema-ch/threema-web-electron/releases/tag/1.2.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 167953 run on x86_64-linux 1

1 package built:

threema-desktop


[Feature Request] Dialog component does not support changing the close icon

Existing Component
Yes
Component Name
Dialog
Description
Please provide an attribute to support changing the close icon

Sorry, we have no plan to support this. You can set show-close to false to hide the default close icon, and position your own icon using CSS.


I dont know how to install this. I dont have any info about coding.

Please Describe The Problem To Be Solved
Im having a hard time installing this on my pc. I tried asking someone but they said i needed Linus or WSL, which i dont really know how to use. Can somebody please tell me how can i install this on my pc like they are teaching a 9 year old.

My discord is "trxae" if you want to talk.


Shapes don't appear in legend of "scatter" example

Note that the shapes do appear in the scatter-solid example, which is puzzling.

Can't reproduce locally, and the test image has the shape legend so it appears in CI as well.

Good to know. If it's only a problem for me, I'll go ahead and close this.
On Fri, Aug 4, 2017 at 12:36 PM Jeff Baumes<EMAIL_ADDRESS>wrote:

Can't reproduce locally, and the test image has the shape legend so it
appears in CI as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kitware/candela/issues/511#issuecomment-320294939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACxNJLHnJEALwsOTY98O38x9CeovRj2rks5sU0hwgaJpZM4OjC6p
.

Yes, it appears to be. I'll close for now.


⚠ PassThePopcorn has degraded performance

In e1c5d17, PassThePopcorn ($PTP) experienced degraded performance:

HTTP code: 200
Response time: 1623 ms

Resolved: PassThePopcorn performance has improved in 65f1b17 after 5 minutes.


Make 18 compatible and include rebar for ease of use

Record is not actually a type so used tuple instead. Update to the new crypto:hash function.

Thank you very much @hazardfn!!

No problem me and a friend of mine are using some of your code to create a simple websocket server because we hate cowboy and wanted something more pure and simple. Our aim is to deliver a talk at EUC next year for a rate-limiter project in erlang called valvex!
Thanks for putting your work on GitHub, your name will of course be mentioned etc.

This is very exciting, I'll be pending of your talk!, thank you again for using this project, I can help you if you have any doubts!,


Problem with PNet and RNet conversion

Hi all,
I used this tool to convert matcaffe trained model to pycaffe model. The converted ONet model can work properly but not PNet or RNet.
The converted PNet and RNet still needs transposed image to produce the correct output. It seems that they are not converted at all.
Anyone has any idea what may have caused this?
Thanks

Hi, I think the MTCNN model in this repo can be successfully converted. Can you somehow provide your model (or link) and other things like test data? Thanks.

Thank you for your reply. I've gone through the whole process again and it is now working as expected.
Thank you for sharing your awesome work!

@conanhujinming  Hi, can I ask an off-topic question about caffemodel conversion?
I trained a model using BGR image(since opencv reads images in BGR order), the result is good and when I want to deploy it, the input somehow has to be in RGB order. Is there any method to modify the model to take input of RGB order and still outputs the exact same result?
The model is not complicated, only contains convolution, innerproduct, max pooling and PRelu.
Many thanks.

Hi, I think you can try to transpose the weights of filters in the dimension of channels. Maybe you can try to do this based on my code: just change the weights where I change them. I am not quite sure about it but I guess it may work.

@conanhujinming Thank you for your reply.
I reconsidered this and found that only modifying the weights of the first conv layer will suffice. Just switch the weights channels as you mentioned. The rest of the networks' weights should be unchanged. I'll try to implement this base on your code.
Thank you again for your advice.

Well, I don't think modifying the first conv layer would be sufficient. I guess all the conv layers before the first FC layer should be modified. You can have a try. I guess it is not so difficult to implement. Good luck.


Dependabot doesn't update the PR when the master branch has a force-push

Package ecosystem
ruby / bundler
Package manager version
Bundler version 2.1.4
Language version
ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]
Manifest location and content prior to update
/Gemfile.lock - see https://github.com/jyn514/jyn514.github.io/blob/5e02d18696f849b0cc99287fa27843c158ce46f5/Gemfile.lock
dependabot.yml content
N/A
Updated dependency
nokogiri (1.10.4 to 1.13.1)
What you expected to see, versus what you actually saw
dependabot generates a PR that only changes the Gemfile. Instead, the PR modifies other files as well.
Native package manager behavior
N/A
Images of the diff or a link to the PR, issue or logs
https://github.com/jyn514/jyn514.github.io/pull/5

Thanks for including direct link to the PR, makes it much easier to understand what you're describing.
When you force pushed master, then the existing PR branch still had the old commit that you blew away from master. You can see that the SHA you force pushed from is the one that's listed on the PR.
Given that you don't have a .github/dependabot.yaml file, that means this is a security update PR rather than our more typical version update PR's.
This one will require a little more digging, but wanted to jot down my notes from an initial look at the issue.

I asked a teammate who told me that Dependabot only auto-rebases when there's a conflict with the branch. Auto-rebasing on every commit to the default branch would be pretty expensive from a compute standpoint, especially as right now @dependabot rebase re-runs the job, not a normal git rebase.
Since your force push didn't create any conflict with the PR, it never triggered teh rebase.
The workaround is to do a manual @dependabot rebase comment and that will trigger Dependabot to re-run and would clear it out.

I asked a teammate who told me that Dependabot only auto-rebases when there's a conflict with the branch.

This seems like the wrong behavior, though? My understanding is that if I'd merged the PR as-is, it would have reverted the changes from my force-push.

Auto-rebasing on every commit to the default branch would be pretty expensive from a compute standpoint

It doesn't need to be on every push, only when there's a force-push.

It doesn't need to be on every push, only when there's a force-push.

True, we should add that to the list of conditions where we auto-rebase the PR.

Related but not a duplicate:

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

@jeffwidman that's this issue, did you mean to link something else?

Yes, fixed thanks @jyn514


Trigger next gc if needed after a gc

Please add PR description

Codecov Report

Merging #36 into master will increase coverage by 0.15%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   84.12%   84.28%   +0.15%
==========================================
  Files          43       43
  Lines        2577     2602      +25
==========================================
+ Hits         2168     2193      +25
  Misses        409      409


Add a suitable FAQs section to the Website

Add a new FAQ section to the Website just above Contributor's section

Take Reference from -
https://codyhouse.co/gem/css-faq-template
https://bootsnipp.com/snippets/z8e5d
Example -

Must Have in PR -

Add proper styling and Wrapping effect using JavaScript
The color and design should match the website theme
Smooth gliding is preferred

Rules -

Must Follow all the steps from Instructions
The PR Should be sent within 24 hours of Assignment, else will get reassigned to the next person.
The PR should be valid and have no Merge Conflicts
DON'T spoil the Code indentations with any formatting tools such as prettier or VSC format

Assigned.

@BGS89 You can add  5 random community questions, our team will be updating the data

Hi @fykaa I have submitted a PR for review : )


Combo 4

This concept is currently being used out in the wild, but there is no data associated with it, and it’s completely uninformative/unnecessary. Note: Seems to be investigator-added term.
Recommendation: terminate existence out in the wild

Terminated


Bugfix: Should not crash when parsing ISO string with missing fractions

Summary of changes
Found this by fixing a warning from dialyzer
Checklist

[x] New functions have typespecs, changed functions were updated
[x] Same for documentation, including moduledocs
[x] Tests were added or updated to cover changes
[x] Commits were squashed into a single coherent commit

Coverage increased (+0.05%) to 67.96% when pulling a608ee3e0b30d1a852e658fce1c16cf58a6ad470 on alvinlindstam:parse-crash-fix into 5d2180da3676140878ad5538ebf96c13cae9098f on bitwalker:master.

Thanks!


Initial Sankey barchart visualization

Sankey Barchart is a visualization based on http://bl.ocks.org/terrancesnyder/227e02f3e2c8eef23f96 which describes a flow and relationship between nodes in a graph, represented in tabular form. Initial Sankey barchart visualization

Hi @bradsa, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

Hi @bradsa, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

I'm in the process of Rebasing to upstream\master .. the pull request contained a set of non-related changes.

I'll try again with a new pull request ?


Lower cased label for Theoretical Models and upper cased OF in SSP

The "factor of safety'' label for Theoretical Models is now lowercased and cannot fix until the table generation automatically titleizes the label. If I try to titleize "factor of safety'' the "of" gets capitalizes with it since I am not (and cannot) use an 'of' combinator. The for this is that I need to have a custom plural: "factors of safety''.

I should be able to temporally fix the capitalization using: nounPhrase' s p c = Phrase (S s) (S p) c CapWords

Why can you not use one of the of combinators here?

of'' returns a sentence whereas of and of' return a NP

factor = npnc "factor"    (cn' "factor")
safety = npnc "safety"    (cnIES "safety")
If I use for'' I will have to use it in my body with these defined terms above rather than using the term from the below concept with is what I'm trying to reference. @szymczdm
fs_rc :: RelationConcept
fs_rc = makeRC "fs_rc" (nounPhrase "factor of safety" "factors of safety") fs_desc fs_rel

for now I will use
nounPhrase'' :: Sentence -> PluralForm -> CapitalizationRule -> CapitalizationRule -> NP
nounPhrase'' = Phrase
and manually tell drasil what to do

you should now be able to create a NounPhrase for "factor of safety" using the of_'' combinator. If you have a problem, reopen this issue.

This has now fixed all but one case "factor of safety" shows up. The remaining issue is still in theoretical models table. "Factor of Safety" is still not capitalized and this is done internally somehow since all that is in my body.hs is s4_2_2_tmods = map Definition [Theory fs_rc]

That's a separate issue that will be solved alongside #29 since we're going to need to reimplement some things before it'll be done. I'll make a note in #29 to ensure we confirm it's done, and close this issue for the time being.


HttpException create method self vs static

Q
A

Bug?
no

New Feature?
yes

Version
1.1.0

Actual Behavior
method create returns HttpException instance
Expected Behavior
Should return an instance of calling class eg ServerErrorException::create() should return
ServerErrorException
Steps to Reproduce
call ServerErrorException::create()
Possible Solutions
Change self to static keyword in create method of HttpException class

ServerErrorException is not part of this repository - did you extend HttpException in your code? but you are correct, this seems unexpected. HttpException could have been made final, but we did not, and it seems okay to extend it.
can you create a pull request to change the exception to use new static instead of new self? and until we release a fixed version, you can overwrite the create method in your extending class, as a workaround - but we should fix it in the HttpException, it is quite unexpected behaviour.


Very slow rendering in tmux (split-mode or multi-attached mode) (Linux)

When in tmux with a split pane, running
ls -al

in a folder with a few hundred files, alacritty needs 1-2 seconds to render it. If I do the same in urxvt, it's almost immediate.
If I use alacritty and urxvt side by side, both attached to the same tmux session but without split panes, urxvt also finishes drawing almost immediately, while alacritty takes quite a long while.
Running the same command in the same folder without using tmux, alacritty renders much faster
The speed is does not noticably change when comparing split vs. attached vs. attached&split
I'm using mesa drivers (in case it's relevant)
My system:

Does urxvt implement margins (DECLRMM)? Margins essentially allow the application to only render the half of the screen that is updated.
Do you see the same speed difference when splitting horizontally?

No I don't. It's quite fast when split vertically, but very slow when splitting horizontally.
No difference for not split but tmux dual-attached tho.

Sounds like the problem here is Mesa drivers. Really need to figure out what's going so wrong there....
cc #125

@d3rrial we landed #410 recently. Mind giving this test a try on the latest master?

I'd say the limiting factor is tmux now.
I've just run it head to head against urxvt and st, and without timing tools I couldn't tell the difference when running journalctl |cat on a 2 year old system with verbose logging.
If anything it's a bit faster. (not by much, with my phone's stopwatch I measured st with 6.4 seconds, urxvt with ~7 seconds and alacritty with ~6.0 seconds.)
the time command gives me:
Alacritty:
journalctl  5.85s user 0.55s system 99% cpu 6.406 total
cat  0.12s user 2.26s system 37% cpu 6.406 total
Urxvt:
journalctl  6.62s user 0.55s system 99% cpu 7.179 total
cat  0.13s user 2.40s system 35% cpu 7.179 total
st:
journalctl  5.99s user 0.56s system 99% cpu 6.558 total
cat  0.10s user 2.29s system 36% cpu 6.558 total
The comparison isn't very fair, since I have urxvt running at a much smaller font-size and I'm having other applications running in the background tainting the results, but it's a good enough approximation.

Sounds like there's no further action required here


Doc/edits root of application folder

@MaxJPRey  I'm guessing there will be code coverage issues as I edited several messages to be complete sentences, with punctuation. I also know I removed at least one occurrence of "please" given that style guides say not to include in technical documentation.

@MaxJPRey All checks but codecov/patch are passing.

@MaxJPRey All checks but codecov/patch are passing.

Hi @PipKat , I am working on it.


dep init crashes

dep init crashes with the following stack trace
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x133ed2c]

goroutine 53 [running]:
github.com/golang/dep/vendor/github.com/sdboyer/gps.(*SourceMgr).SyncSourceFor(0xc4200849c0, 0xc420558000, 0x19, 0x0, 0x0, 0x0, 0x0)
/go/src/github.com/golang/dep/vendor/github.com/sdboyer/gps/source_manager.go:461 +0x10c
main.getProjectData.func1(0xc420558000, 0x19, 0xc4200849c0)
/go/src/github.com/golang/dep/cmd/dep/init.go:226 +0x6c
created by main.getProjectData
/go/src/github.com/golang/dep/cmd/dep/init.go:251 +0xab5

Thanks for the report! Sorry you ran into that. It's fixed upstream, we just haven't integrated latest gps in a while. (See e.g. #250)


Please invite me to the GitHub Community Organization

Name
Rohit Saini
Discord Username (if applicable)
Rohit Saini#1017
Additional Context
Hi there, I am Rohit and I am following Eddie from a long time. I want to thank him for guiding everyone to contribute and encourage open source contributors I am also one of them. He is such an inspiration. I am an open-source enthusiast and I think this community would be perfect for me to grow more in the field of open source as there are already experienced members in the community.
It would be my great pleasure if I can join this community.

Welcome to EddieHub, @rockingrohit9639! We hope you learn more about open source and grow your skills. Only remember the most important thing: community first, code second! 😄👋🎉


Searching for text that is in the metadata pane does not work

Summary
A feature of IIIF Print is being able to search for text that is in the metadata pane, currently this does not work.
Acceptance Criteria

[ ] In a parent/child view, searching for metadata that exists in the pane should return a hit

Screenshots or Video
Testing Instructions

[ ] create a parent/child work
[ ] search for a metadata term that is in the metadata pane
[ ] observe that the search returned the result which should be the work that the metadata is on

Notes

This ticket passes SoftServ QA see attached screenshots.
[x] Create parent/child work

[x] Search for a metadata term that is in the metadata pane

[x]  observe that the search returned the result which should be the work that the metadata is on


release v1.1.0

Milestone https://github.com/containers/podman-desktop-extension-ai-lab/milestone/7
Required items to release

[x] https://github.com/containers/ai-lab-recipes (need release v1.1.0)
[x] https://github.com/containers/podman-desktop-extension-ai-lab/pull/1032
[x] https://github.com/containers/podman-desktop-extension-ai-lab/pull/1041
[x] https://github.com/containers/podman-desktop-extension-ai-lab/pull/1042
[x] https://github.com/containers/podman-desktop-extension-ai-lab/pull/1043
[x] https://github.com/containers/podman-desktop-extension-ai-lab/pull/1044

Post release

[ ] Update the catalog extension

Done 🚀


Git prompt

Brian Carper posted a very compelling zsh prompt for git repositories earlier today: http://briancarper.net/blog/570/git-info-in-your-zsh-prompt
It's really gorgeous and would probably fit in nicely with oh-my-zsh.

Based on Brian's description of the symbols, I think the lib/git.zsh is implemented slightly differently than was described in the blog. It's not difficult to adapt, though—I modified the relevant function to look at staged changes vs. unstaged changes vs. unmerged changes vs. untracked changes and count the number of each type of change in a theme I'm working on (https://github.com/lietk12/oh-my-zsh/blob/lietk12-theme/themes/lietk12.zsh-theme); Adam Lindberg's sunrise theme also modified the original function from lib/git.zsh to provide a bit more detail, but I don't recall any other themes which made major changes.
Here's the modified function I wrote:
function git_prompt_status_concise() {
  local status_index="$(git status --porcelain 2> /dev/null)"
  local prompt_status=""
  local untracked="$(echo $status_index | grep '^?? ' 2> /dev/null)"
  if [[ -n $untracked ]]; then
    prompt_status="${ZSH_THEME_GIT_PROMPT_UNTRACKED_PREFIX}$(echo $untracked | wc -l)${ZSH_THEME_GIT_PROMPT_UNTRACKED_SUFFIX}${prompt_status}"
  fi
  local unmerged="$(echo $status_index | grep '^UU ' 2> /dev/null)"
  if [[ -n $unmerged ]]; then
    prompt_status="${ZSH_THEME_GIT_PROMPT_UNMERGED_PREFIX}$(echo $unmerged | wc -l)${ZSH_THEME_GIT_PROMPT_UNMERGED_SUFFIX}${prompt_status}"
  fi
  local unstaged="$(echo $status_index | grep -E '^ D |^.M |^AM |^ T ' 2> /dev/null)"
  if [[ -n $unstaged ]]; then
    prompt_status="${ZSH_THEME_GIT_PROMPT_UNSTAGED_PREFIX}$(echo $unstaged | wc -l)${ZSH_THEME_GIT_PROMPT_UNSTAGED_SUFFIX}${prompt_status}"
  fi
  local staged="$(echo $status_index | grep -E '^D  |^R|^M|^A' 2> /dev/null)"
  if [[ -n $staged ]]; then
    prompt_status="${ZSH_THEME_GIT_PROMPT_STAGED_PREFIX}$(echo $staged | wc -l)${ZSH_THEME_GIT_PROMPT_STAGED_SUFFIX}${prompt_status}"
  fi

  echo $prompt_status
}


Convert bytes in memory BarChart properly

Fixes regression caused by #450
Signed-off-by: Lauri Nevala<EMAIL_ADDRESS>

Is it possible to write a unit test to make sure this doesn't regress in the future?


create stream: Show current user on top of "People to add" list

This is a modification of #7524 that does two things:

It changes a more fragile object creation to prepend into the people object to just fetching the current user’s object.

Changing all_user => all_users.

Fixes #7475

This is good to merge @showell, and we can close this PR and PR #7524 then.

merged thanks @brockwhittaker

Thanks @brockwhittaker this is nice trick :)


How to rotate 2D scene?

Hi,
I have a hard time figuring out how to rotate 2D plot such as scatter or barplot. The things is that when I use rotate!  I can figure out the right quarternion to use. It seems that my plot essentially slanted in the 3D space. What is the correct way to do this?

Try passing a 2-tuple of angles?

Hi,
I did put it does not seem to work
s = barplot(ones(10), color =cl, rotations = (\theta1, \theta2) )
does not seem to do anything.

If I do this it give incorrect result
rotate!(s, qrotation(SVector{3}(1.0, 0.0, 0.0), π/2))

As I understand it, you're looking to transpose the graph - swap the x and y axes, right?

In this case yes, but I would love to know how to do rotation of 2d scene
in general. The problem I think is that the camera field does not get
update correctly. It treat the 2D as if it is a 3D object,
On Sun, Mar 31, 2019 at 3:54 PM Anshul Singhvi<EMAIL_ADDRESS>wrote:

As I understand it, you're looking to transpose the graph - swap the x and
y axes, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaPlots/Makie.jl/issues/333#issuecomment-478374910,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXjro4GgOhPdIbooFxQRM8yHDzatjjfpks5vcRJ8gaJpZM4cUfb-
.

I suppose you could try to pull off just using a cam3d! in the camera argument and rotating it that way, but it's hardly elegant...

Yep, that is what I attempt to do put it is really complicated.
On Sun, Mar 31, 2019 at 4:01 PM Anshul Singhvi<EMAIL_ADDRESS>wrote:

I suppose you could try to pull off just using a cam3d! in the camera
argument and rotating it that way, but it's hardly elegant...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaPlots/Makie.jl/issues/333#issuecomment-478375763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXjro1O_Do0WcrrUnc_0u5rcucuwOWt1ks5vcRQsgaJpZM4cUfb-
.

Maybe put camera = cam3d! and then rotate_cam! instead?

Do you want to rotate the whole scene or just a single plot object?

ping @wsrinin

I put some code in https://github.com/JuliaPlots/Makie.jl/issues/1298 that rotates a node in the scene graph. Note that there is an issue with CairoMakie, however.
I also asked a related question here: https://discourse.julialang.org/t/permute-x-and-y-axes-of-a-makie-axis/67826


[refactor] Change default behavior to collect pretty print summary report and license reports into a zip file by default

Change default behavior to collect pretty print summary report and license reports into a zip file by default

I added this functionality in the license_report feature branch. In order to minimize dependencies, it is left to the user to use the compression tool of their choice. KICK generates license reports, as well as a JSON representation of the pretty print summary, and stores those files in the specified output folder.


Implement a basic tiling implementation using a binary tree.

Similar to this for each layer and output:
             Root *
                 / \
SurfaceToplevel *   * SplitWindow
                   / \
  SurfaceToplevel *   * SurfaceToplevel

Moving onto this now.
My initial thought here is that I don't know if I should force myself into using a binary tree.
I think what may work well is only on the tiling layer, for every surface node that is a direct child of of the root (meaning the tiling scene_tree) the split will be horizontal. Then if there is another child tree, then for every child of this, the split will be in the opposite direction.
So if the root tree has 3 SurfaceToplevels, and 1 SplitWindow; then it will be a 4 way horizontal split, then in the space of SplitWindow if there are 2 SurfaceToplevels, it will be a horizontal split.
This is going to be my first implementation attempt. I'll see how it goes then I might re-evaluate.


Generated scala code may improperly group list patterns

Describe the bug
If an elm pattern match statement has nested list expressions, the generated scala code may improperly group them.
To Reproduce
Start with the following elm source code:
describe : List (List Int) -> String
describe ll = case ll of
    (_ :: _) :: _ -> "This starts with a list of at least one element"
    _ :: _ :: _ -> "This contains at least two sublists"
    _ -> "Other"

listDescribeTest1 : Int -> String
listDescribeTest1 x = describe [[x]]

listDescribeTest2 : Int -> String
listDescribeTest2 x = describe [[], [x]]

describe operates on lists of lists. The first arm of the match statement should match any list which has a first element which itself has a first element. The second arm should then catch lists of at least two elements.
This is the behavior in both native elm and the morphir-elm interpreter, which show each of the first two arms being matched by listDescribeTest1 and listDescribeTest2 respectively.
However, if morphir-elm gen is called, the following scala code is produced for the describe function:
def describe(
    ll: morphir.sdk.List.List[morphir.sdk.List.List[morphir.sdk.Basics.Int]]
  ): morphir.sdk.String.String =
    ll match {
      case _ :: _ :: _ =>
        "This starts with a list of at least one element"
      case _ :: _ :: _ =>
        "This contains at least two sublists"
      case _ =>
        "Other"
    }

Here, the grouping information has been lost and the first arm completely shadows the second. (In this example it will at least result in a scala compiler warning, but in others - such as if the second case were omitted - it would produce apparently legal scala code with the incorrect behavior.)
Expected behavior
Generated scala patterns should be grouped the same way as the elm source code.
Desktop (please complete the following information):

OS:  OSX
Version: 2.87.6

@AttilaMihaly I revisited this and it seems to no longer be happening - can you confirm that this bug has been closed?

Confirmed that as of 2.89.0, this behavior no longer appears.


Add persistent block room settings

Codecov Report

Merging #114 into master will decrease coverage by 0.13%.
The diff coverage is 65.11%.

@@             Coverage Diff              @@
##             master     #114      +/-   ##
============================================
- Coverage     68.38%   68.25%   -0.14%
- Complexity      496      508      +12
============================================
  Files            91       91
  Lines          1683     1723      +40
  Branches        182      187       +5
============================================
+ Hits           1151     1176      +25
- Misses          469      479      +10
- Partials         63       68       +5

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
77.27% <0.00%> (ø)
20.00 <0.00> (ø)

src/main/java/seedu/address/model/person/Room.java
43.47% <30.76%> (-26.53%)
7.00 <5.00> (+4.00)
:arrow_down:

...rc/main/java/seedu/address/model/person/Block.java
71.42% <83.33%> (+1.42%)
7.00 <5.00> (+4.00)

src/main/java/seedu/address/model/UserPrefs.java
86.36% <100.00%> (+6.36%)
14.00 <4.00> (+4.00)

Continue to review full report at Codecov.

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


Add or modify support for supabase

@ccrsxx is usually always busy so I doubt he'd want to do this. But, you're more than welcome to contribute to the project and open a PR. :)

Sorry but no plans on doing this, it should be pretty similar tho with firebase tho. You can implement it yourself.


Added Binary Search in Java

Thanks for contributing! But this problem doesn't exist.
You can find active problem statements here. If you'd like to add a problem instead, refer to this


west: hostap: update hostap revision to PR

Update hostap revision to PR: Add chan_list_changed API

The manifest file will be automatically updated by #77215 so closing this one.


feature needed: object pattern recognition

Detecting Objects like Laptop, Books, TV.
If Object is Laptop , then we should generate memes on WorkFrom home.
If object is books, we should generate memes on Exams, Academic Institutions.


Error on Oni load

Upon loading Oni, using the latest Master, I get this error:
Uncaught (in promise) TypeError: Cannot read property 'request' of undefined
    at NeovimInstance.eval (NeovimInstance.ts?e8a2:538)
    at NeovimAutoCommands.eval (NeovimAutoCommands.ts?f73a:116)
    at Generator.next (<anonymous>)
    at eval (563:8)
    at Promise (<anonymous>)
    at __awaiter (563:4)
    at NeovimAutoCommands.executeAutoCommand (NeovimAutoCommands.ts?f73a:115)
    at NeovimEditor.enter (NeovimEditor.tsx?b7dc:669)
    at OniEditor.enter (OniEditor.tsx?ed1d:189)
    at AugmentedWindow.enter (WindowManager.ts?db84:95)
eval @ NeovimInstance.ts?e8a2:538
(anonymous) @ NeovimAutoCommands.ts?f73a:116
(anonymous) @ 563:8
__awaiter @ 563:4
executeAutoCommand @ NeovimAutoCommands.ts?f73a:115
enter @ NeovimEditor.tsx?b7dc:669
enter @ OniEditor.tsx?ed1d:189
enter @ WindowManager.ts?db84:95
_focusNewSplit @ WindowManager.ts?db84:287
createSplit @ WindowManager.ts?db84:204
(anonymous) @ startEditors.ts?d033:56
(anonymous) @ 301:9
__awaiter @ 301:5
startEditors @ startEditors.ts?d033:25
(anonymous) @ index.tsx?0c97:185
(anonymous) @ 200:16
__awaiter @ 200:12
initializeAllEditors @ index.tsx?0c97:184
(anonymous) @ index.tsx?0c97:205
fulfilled @ 200:13

Following back the stack trace, I think its due to the changes that were added in #1786.
Specifically, the following line:
https://github.com/onivim/oni/blob/6d9cd90e1c20bc2c4ae3ce087f8695d5856457b0/browser/src/Editor/NeovimEditor/NeovimEditor.tsx#L669
Which I think is calling on Neovim, when it isn't fully setup yet.
If I pause on exception, the error occurs when calling exists('#FocusGained'), so I am assuming that is the cause of the error.

@CrossR , nice find! I saw this came up but didn't realize it was a regression - I think you're right that #1786 breaks this. I'll take a look at it today. Thanks for catching this!

Seems the error persists despite #1800, with the addition of notifyGoal of undefined

Sorry for adding another one in there, I just wanted to raise that the issue persists unfortunately. Regarding the focusedGained and lost functionality this feature exists for the workspace which is what the git status uses to update the branch I wonder if that functionality could be used to dispatch the focusedGained?

Thanks @Akin909 for the update! I just fixed this in #1800
Those errors should've been caught in our automation - so there was a hole in our safety net. That fix addresses the hole (we weren't always checking the logs for errors), so hopefully it'll be harder to introduce regressions like this in the future 👍

(The fix just got merged in - so might be worth pulling the latest master if you're still hitting it)


Remove unnecessary device_guard usage

pulled out of #938, we don't need to set the device ID kernels, because it already gets set by the executor->run function.

That's the issue with global state - we have no way of knowing whether it was modified somewhere, so in the worst case users change the device ID inside a custom LinOp::apply and we run solver kernels on GPU 1 while the data resides on GPU 0, which gives a hard to identify access violation.

I see. Okay, in general, that makes sense.


Production of content and posts on social networks LATAM [Ambassador]

EdTech Latin America
Grant category

Educational Material

Description
Problem statement
The situation in Brazil where there are a large number of developers who are interested in utilizing the flow blockchain and programming in cadence. However, the availability of English-only content is limiting their ability to do so. This suggests that there is a demand for resources in languages other than English in order for these developers to fully utilize the flow blockchain.
Proposed solution
It is important to provide high-quality content to the Latin American (LATAM) community in order to support and enrich their learning and development. This can involve creating materials that are accurate, informative, and relevant to the needs and interests of the community. It may also involve ensuring that the content is presented in a clear and concise manner, and is accessible to individuals with varying levels of knowledge and expertise. By providing quality content to the LATAM community, individuals can improve their understanding and skills in a particular area, and have access to resources that can support their personal and professional growth.
Impact
The goal of this proposal is to facilitate the transition of thousands of web2 developers to the web3 world, with a focus on utilizing flow solutions. Brazil, as the largest economy in Latin America, has a significant number of highly skilled developers who have the potential to create innovative solutions using the flow blockchain. These solutions have the potential to address a range of issues within Brazil, including green certification of agribusiness, environmental concerns related to forest devastation, and the distribution of wealth and hunger. This proposal is equivalent to six months of work, and its successful implementation could lead to significant benefits for Brazil and its developers.
More about me:
I am an advocate for flow initiatives and have published articles and videos on social networks. In November 2022, I was selected from over 3300 applicants to be one of the first 16 ambassadors worldwide. In 2018, the launch of flow inspired me to open a company in Brazil called Coflow. Our company shares the same vision as Dapper Labs, which is to create industry-leading web3 experiences and develop dapps on the flow blockchain that provide users with a simple way to manage their assets and use the blockchain. Our mission is to make the world a more accessible and enjoyable place through consumer adoption of decentralized technologies.
Milestones and funding
Deliverables : Production of content and posts on social networks LATAM
https://discord.com/channels/613813861610684416/1049804829221978153

Milestone
Deliverables
Timeline
FLOW proposal

Phase 0
Only in Portuguese Published (December)
ready
Free

Phase 1
30 articles in Portuguese and Spanish. (January)
4 weeks
7.000

Phase 2
30 articles in Portuguese and Spanish. (February)
4 weeks
7.000

Phase 3
30 articles in Portuguese and Spanish. (March)
4 weeks
7.000

Our site was created with the intention of being multilingual and is designed to be a collaborative platform where the community can request content updates through pull requests on GitHub. In this way, together, we can work to expand and improve the available content.
Adoption value will be reverted to the LATAM community
| 2 months               | 50 FlowLatam advocates     |  8 weeks     | 5.000          |
| 4 months               | 100 FlowLatam advocates   |  8 weeks     | 10.000        |
| 6 months               | 150 FlowLatam advocates   |  8 weeks     | 15.000        |
We consider engaged advocates as those who help with promotion and also manage the community in some way, assisting each other and fairly receiving financial incentives for their productivity. Our goal is to assign small tasks to the members so they can further engage.
Article Publications
Blog: flow.coflow.com.br/blog
Strategy: Every month, 30 articles covering a variety of topics related to the Flow ecosystem will be made available. Additionally, all articles published on the website will be tracked for performance on a weekly basis.
Accountability: https://www.notion.so/flowambassadors/b301a63e452b4c3bb6499a7ea4bc475d
Videos
Channel: https://www.youtube.com/@blockchainflow
Strategy: In addition to videos produced from the top-performing articles on the website, we are also forming partnerships with other channels to provide additional high-quality content. For example, this type of video: https://www.youtube.com/watch?v=rOXTWIQ_sU8&list=PLJdSIZeX6heAluuSyva_wPM_LUyr8_MPY
Other content portals that the FlowLatam community will disclose
I commit to fostering the growth of the FlowLatam community by allowing new advocates to publish content on the platform in their native language. As a way of recognizing and valuing their contributions, a portion of tokens will be donated. In addition to publishing on the main blog, community advocates will also have the opportunity to share their posts on other platforms and social networks, increasing their chances of being rewarded with Flow tokens.
Mapped so far: Mirror, Medium, tabnews, Twitter, Linkedin, Instagram, Web3Dev, dev.to, hackmd.io, hackmd.io and Substack.
ADOPTION GOALS:
If all of the above is implemented, the following goals should be achieved within a 6-month period for recruiting FlowLatam advocates:
2 months - 50 FlowLatam advocates will be engaged
4 months - 100 FlowLatam advocates will be engaged
6 months - 150 FlowLatam advocates will be engaged
In addition to quantity, we will develop quality metrics to assess the engagement of the advocates, in order to ensure that only the best ones are present in the FlowLatam program, which is ideated and led by the CoFlow team.
Team

Name
Role
Bio
Contact

Diego Alcantara
Official Ambassador at Flow
Founder Coflow
<EMAIL_ADDRESS>

Bianca Lima
Head of Operations
CoFounder Coflow
<EMAIL_ADDRESS>

Adriano Monteiro
Bilingual Assistant
Assistant Coflow
<EMAIL_ADDRESS>

https://github.com/diegofornalha/Mais1Code/tree/master/content/pages/blog
I have prepared a platform for the OnFlowBR community and am in the process of expanding it to cover all of Latin America. I have migrated all of the content to the platform and will be using google analytics to allow others to monitor the real growth of the community. I am committed to fostering the growth of the community and will be allowing new flow advocates to post on the platform. As a reward for their contributions, I will be donating a portion of tokens. In addition to posting on the main blog, I will also be rewarding those who publish their posts on other content platforms and social networks with a flow token.

@chrisackermann I will create targeted content for the hackathon participants, including the use of the emeraldBot to record and measure the engagement of advocates.

@diegofornalha great idea! There are many people to reach in the LATAM space.

@fotescodev The proposal update included the consideration of Spanish speakers, ensuring that they also have access to our content. In this way, we expand our audience and strengthen our LATAM presence. This inclusion reinforces our commitment to meet the needs of everyone.

Ok gents, let's give it a try!

@fotescodev @alxflw Due to the restructuring of the grant program and delays in decision-making for proposal acceptance, the bilingual developer responsible for translating Spanish content accepted a new full-time job opportunity in another blockchain. As a result, we've chosen to reduce the amount of Flow tokens we are requesting, while still committing to deliver the Portuguese-translated content.
We are delighted to share that our community has been expanding and now includes about 200 developers, all members of the Emerald LATAM community, each one eager to participate in the Hackathon.
Moreover, we are excited to announce the launch of the BR token. Our goal is to add liquidity to this token and foster a token economy within regional projects.
Our target is to reach 6,000 members by the end of the year with this support. We are eager to accelerate our growth!

@Aliserag

This is the amount held by the DAO. The strategy is straightforward: whenever the value of the Flow token drops, we secure a loan in dollars, which is then converted into Flow. We sell at a high to repay the loan. This method serves as a practical, educational example of DeFi in action. The profits enable us to increase our holdings of tokens from partner projects, such as the 236,000 DUST tokens we possess, viewable at https://www.flowdiver.io/account/diego.find?tab=tokens.
https://nft.flowverse.co/user/0x7bf07d719dcb8480?tab=sale
Additionally, we have Flovatar NFTs listed for sale, priced closely to their real market value, as can be seen at https://flov.dev/. This adds up to over 70,000 FLOW tokens in NFTs.

@diegofornalha Hello Diego! If you need any help in the future, just tag me!

We appreciate your proposal and taking the time to fill it out. After careful review we've decided not to proceed at this time. Please stay connected to our Twitter and working groups to stay on top of the latest developments in the ecosystem and new grant opportunities.


helm is not upgrading new added labels at deployments pod metadata.label: section

It seems that my newly added labels are not taken over as part of the upgrade although no complains  from client.
Version:"v3.6.0-rc.1

Hi @cforce. In the future, can you please provide a test case with your issue? That way we can reproduce the issue without having to do some guesswork on our end to try and reproduce a similar environment. Thanks.
I attempted to reproduce the issue with both helm upgrade and kubectl apply to see if there were any differences in behaviour between the two. I found that both did not apply the new label during the upgrade, which indicates to me that this is an intentional behaviour coming from Kubernetes.
Here's how I tested with kubectl:
 ><> cat deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 3
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80

><> kubectl apply -f deployment.yaml
><> vim deployment.yaml
><> cat deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
    foo: bar
spec:
  replicas: 3
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx
        image: nginx:1.14.2
        ports:
        - containerPort: 80

Notice there's a new label: foo: bar.
><> kubectl apply -f deployment.yaml

None of the pods were removed. After removing the pods with kubectl delete, none of the new pods applied the new label.
So, this is either an upstream bug or this is intentional behaviour. Given Kubernetes' restrictions around selector label updates I would take a guess it's more likely intentional behaviour.

I could not reproduce the issue. I tested by hard-coding a label in the default deployment template (notice foo: bar in the pod labels):
><> cat templates/deployment.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: {{ include "test-9848.fullname" . }}
  labels:
    {{- include "test-9848.labels" . | nindent 4 }}
spec:
  {{- if not .Values.autoscaling.enabled }}
  replicas: {{ .Values.replicaCount }}
  {{- end }}
  selector:
    matchLabels:
      {{- include "test-9848.selectorLabels" . | nindent 6 }}
  template:
    metadata:
      {{- with .Values.podAnnotations }}
      annotations:
        {{- toYaml . | nindent 8 }}
      {{- end }}
      labels:
        foo: bar
        {{- include "test-9848.selectorLabels" . | nindent 8 }}
    spec:
      {{- with .Values.imagePullSecrets }}
      imagePullSecrets:
        {{- toYaml . | nindent 8 }}
      {{- end }}
      serviceAccountName: {{ include "test-9848.serviceAccountName" . }}
      securityContext:
        {{- toYaml .Values.podSecurityContext | nindent 8 }}
      containers:
        - name: {{ .Chart.Name }}
          securityContext:
            {{- toYaml .Values.securityContext | nindent 12 }}
          image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
          imagePullPolicy: {{ .Values.image.pullPolicy }}
          ports:
            - name: http
              containerPort: 80
              protocol: TCP
          livenessProbe:
            httpGet:
              path: /
              port: http
          readinessProbe:
            httpGet:
              path: /
              port: http
          resources:
            {{- toYaml .Values.resources | nindent 12 }}
      {{- with .Values.nodeSelector }}
      nodeSelector:
        {{- toYaml . | nindent 8 }}
      {{- end }}
      {{- with .Values.affinity }}
      affinity:
        {{- toYaml . | nindent 8 }}
      {{- end }}
      {{- with .Values.tolerations }}
      tolerations:
        {{- toYaml . | nindent 8 }}
      {{- end }}

Calling helm upgrade caused a pod restart, and the new pod had the correct labels applied.
><> k get po chart-1624463173-test-9848-557b67675-v9zlx -o jsonpath="{.metadata.labels}"
{"app.kubernetes.io/instance":"chart-1624463173","app.kubernetes.io/name":"test-9848","foo":"bar","pod-template-hash":"557b67675"}
``

Strange indeed that it works for you. Using same aks version
I tried adding tags according to https://docs.datadoghq.com/getting_started/tagging/unified_service_tagging/?tab=kubernetes
tags.datadoghq.com/env: {{ .Values.stage.environment | quote  }}
tags.datadoghq.com/service: {{ .Chart.Name | quote  }}
tags.datadoghq.com/version: {{ .Values.app.version | quote }}

Deployment was successful but labels has not been added
Is its possible that updgrade does ignore if the container  image does not change?
I will retry with 3.6.1

I don't think that's the case. I never changed the container image in my example test, and the labels updated just fine.


[Excel] (Custom Functions) Fix filepath in shared runtime tutorial

Fixes #3104.

Docs Build status updates of commit ab7dc60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tutorials/share-data-and-events-between-custom-functions-and-the-task-pane-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 6acdc12:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tutorials/share-data-and-events-between-custom-functions-and-the-task-pane-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 19ad81e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tutorials/share-data-and-events-between-custom-functions-and-the-task-pane-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


'backup --files-from' results in 'ls latest' producing header with all filenames

Output of restic version
restic 0.16.2-dev (compiled manually) compiled with go1.20.6 on linux/amd64
it's restic_v0.16.2-128-g76f507c77_linux_amd64
same problem with restic 0.16.2
What backend/service did you use to store the repository?
local filesystem (SFTP has same problem)
Problem description / Steps to reproduce
When using 'backup --files-from', the 'ls latest' of that snapshot consists of an ugly, unformatted header and the actual file list.
The header is 'snapshot xyz of [ (ALL files listed here) ]' which is true, but the listing is not easy to read.
If using 'backup path', the header contains [ path ].
To reproduce (any find command will do):
host: find /etc/network | ~/bin/beta-restic -r ~/restic-repo/ --files-from -
repository 94e619c2 opened (version 2, compression level auto)
no parent snapshot found, will read all files
[0:00] 100.00%  7 / 7 index files loaded
error: open /etc/network/interfaces: permission denied
Files:           7 new,     0 changed,     0 unmodified
Dirs:            7 new,     0 changed,     0 unmodified
Added to the repository: 0 B   (0 B   stored)
processed 7 files, 16.844 KiB in 0:00
snapshot 46e891a9 saved
Warning: at least one source file could not be read
host: ~/bin/beta-restic -r ~/restic-repo/ ls latest
repository 94e619c2 opened (version 2, compression level auto)
[0:00] 100.00%  7 / 7 index files loaded
snapshot 46e891a9 of [/etc/network /etc/network/interfaces.d /etc/network/if-pre-up.d /etc/network/if-pre-up.d/wireless-tools /etc/network/if-pre-up.d/wpasupplicant /etc/network/interfaces /
etc/network/if-post-down.d /etc/network/if-post-down.d/wireless-tools /etc/network/if-post-down.d/wpasupplicant /etc/network/if-down.d /etc/network/if-down.d/resolved /etc/network/if-down.d/
resolvconf /etc/network/if-down.d/wpasupplicant /etc/network/if-up.d /etc/network/if-up.d/resolved /etc/network/if-up.d/wpasupplicant /etc/network/if-up.d/000resolvconf /etc/network/if-up.d/
mountnfs] at 2024-01-07 12:08:08.573929019 +0100 CET by restic-test@host filtered by []:
/etc
/etc/network
/etc/network/if-down.d
/etc/network/if-down.d/resolvconf
/etc/network/if-down.d/resolved
/etc/network/if-down.d/wpasupplicant
/etc/network/if-post-down.d
/etc/network/if-post-down.d/wireless-tools
/etc/network/if-post-down.d/wpasupplicant
/etc/network/if-pre-up.d
/etc/network/if-pre-up.d/wireless-tools
/etc/network/if-pre-up.d/wpasupplicant
/etc/network/if-up.d
/etc/network/if-up.d/000resolvconf
/etc/network/if-up.d/mountnfs
/etc/network/if-up.d/resolved
/etc/network/if-up.d/wpasupplicant
/etc/network/interfaces.d

Expected behavior
Something sensible, maybe [-] to indicate the header might be too complex to display neatly because stdin was used.

Actual behavior
The header of the ls listing contains an unformatted list of all the files in the backup.

Do you have any idea what may have caused this?
No.

No.
Did restic help you today? Did it make you happy in any way?
I'm still considering restic for future backups, so yes.

Why am I doing it this way?
The problem is reproduced on Linux, but it was identified on Windows, where my 'backup tree' contains a number of symlinks.
These are not followed by restic, so I 'find' them in the backup tree and pass the list to restic.

Just updated to 0.16.3: same problem.
I just started testing my restore-script, unfortunately restic also there produces a one-line header with all 5000+ filepaths in it - quite disturbing though it's possible to filter the messages in a pipe.
Is this a bug, or does it work as designed?
Regards

The reason this happens is that the path(s) specified when taking a backup (even from --files-from option) 'implicitly' identify the backup snapshot (in combination with the host and the time).  If you run restic snapshots for instance, you will see that the list will be shown in the output against that particular snapshot ID as well.
This and many other issues reveal an underlying design flaw in how restic operates.  There is a proposed solution (#4026) that I submitted some time ago, but based on the current developer priorities and project roadmap, it not under consideration at this time.

Thanks - I agree with the proposals described in your issue, they could bring som clarity and flexibility to my backups using paths;
The backup mentioned in this issue uses a file list produced by the find+grep programs (following symbolic links and using exclude patterns) - it cannot be easily transformed (by me) into paths and excludes in restic, so I'll live with each snapshot being isolated - and discard the list with 5000+ paths...
Maybe addition of the .gitignore feature will help me, I'll certainly try it out.

Maintaining snapshots made using a file list:
Having used restic a few months I'm starting to test the forget policies - I'm out of luck because the snapshots are considered different: the filelists in the snapshots are always a little bit different because they're made dynamically outside of restic.

@ogregpt The --group-by option is probably useful in your case, see https://restic.readthedocs.io/en/stable/060_forget.html#removing-snapshots-according-to-a-policy .

@ogregpt The --group-by option is probably useful in your case, see https://restic.readthedocs.io/en/stable/060_forget.html#removing-snapshots-according-to-a-policy .

I'm not so sure Michael.

the filelists in the snapshots are always a little bit different because they're made dynamically outside of restic.

Because of this, I think the path component of @ogregpt snapshots vary, so I don't think --group-by can't help here.
Entertaining #4026 would resolve the two issues raised by @ogregpt, and many other issues as mentioned in issue 4026.
While it would be a considerable undertaking, my view is that it would be worth it in the long run, especially because it would reduce the number of support requests that appear related to the flaws of the current model of using the path to identify a backup-set.  But, I do acknowledge that because I am not a core developer, I don't have the full picture.

@MichaelEischer @damoclark :
Thank you both - to my relief Michael's suggestion does indeed work: the --group-by option overrides the path component...

Entertaining #4026 would resolve the two issues raised by @ogregpt, and many other issues as mentioned in issue 4026.

I don't have time to look into that issue before restic 0.17.0 is done.

I don't have time to look into that issue before restic 0.17.0 is done.

Perfectly understandable Michael.  I checked out the roadmap and it appears a mammoth undertaking, with some exciting new features coming.

Thank you both - to my relief Michael's suggestion does indeed work: the --group-by option overrides the path component...

Seems my skepticism was wrong - I'm glad that this did work for you.  :)
I think I was misguided by my own context.  I have configured restic to backup using separate snapshots for each mounted filesystem.  So most of my hosts have more than one path defined.  I am guessing that your use case at the moment has just a single 'dynamic' backup for your host, so grouping by host alone allows you to apply your forget policy against all your snapshots.
If in the future, you need to add additional backups to the same host, you should be able to make that work by using tags to separate them from your dynamic backup, and can --group-by host,tag


Add codeMirrorInstance prop to editors

This PR adds a codeMirrorInstance prop to the editor, as inspired by the same prop in react-codemirror.
I'm trying to migrate my company from react-codemirror to react-codemirror2, but without this prop (or another way of defining codemirror extensions) I'm not yet able to.

Coverage decreased (-0.03%) to 79.836% when pulling 4c22705320241b283fcbcf956214503733d5a2b5 on zachkirsch:codeMirrorInstance-prop into 3d490109229e8592772db922e9028c9a1dee3336 on scniro:master.

@scniro are you able to take a look here when you have a chance?

@zachkirsch depending on what you're trying to achieve, I was able to use a React ref and pass that into the react-codemirror2 component, for example:
const editorInput = useRef(null);

...

<ReactCodeMirror ref={inputref} options={options} value={content} onBeforeChange={handleChange}/>

Then elsewhere in my code, I could access the codemirror instance like so:
editorInput.current.editor.focus();

You may be able to do the same in order to setup extensions?

Thanks @davidwinter! The feature I wanted to use was CodeMirrorInstance.defineExtension, which I couldn't figure out how to do with this package. But I actually figured out a workaround yesterday for our use case (custom search bar) so I'm no longer blocked on this!


multiple account

hi Dude can you add some future for multiple Account supporting?

I also want to do this through multiple accounts. Is there a way to do that? Because it seems that even by changing the project folder and deleting the connection-related files, it still connects to the previous account when I run it.

Soon, the multi-account feature will be added!


backstretch slideshow with title/caption, multiple blocks of text (at least one  and a 'readmore') and ability to add link?)

Hi,
I need to set up a backstretch slideshow with a title/caption, a short summary and a 'read more'. text elements should be linkable.
anybody willing to help?
thanks a lot in advance,
Alessio

This is a hefty task to ask someone else to do for free, but I can send you down the right path if I know a little more about your project. Is the backstretch for a full page background or meant to be contained within a fixed width space?  Do you have need for thumbnails or some kind of gallery system?


IO Exception throwing when using DevSkim with Output file

Running .\devskim.exe analyze C:\Projects\ConsoleApp1\ConsoleApp1\ sample.txt command throwing exception like this.
Unhandled exception. System.IO.IOException: The process cannot access the file 'C:\Project\DevSkim\DevSkim_0.4.118\sample.txt' because it is being used by another process.

Am I missing anything?

Hi Anuraj, thanks for your report.
Were you intending to run on "sample.txt" in the local directory?
It looks like we should be catching that exception, but in this case there wouldn't be anything for us to do with it - this is pretty unrecoverable from a reading the file standpoint. It looks like a file locking issue with that file itself.

No. I am trying to run on a C# project directory and I want to store the results in sample.txt file. So if I run the command like this
 & "C:\Project\DevSkim_windows_0.4.118\DevSkim_0.4.118\devskim.exe" analyze C:\Projects\ConsoleApp1\ConsoleApp1\

I am getting the following output

But if I provide an output file, like this.
 & "C:\Project\DevSkim_windows_0.4.118\DevSkim_0.4.118\devskim.exe" analyze C:\Projects\ConsoleApp1\ConsoleApp1\ output.txt

It is throwing an exception like this.

Hope you understood the problem.

Thank you for your report. I will investigate.


Failed to delete Internal LB resources when k8s service was deleted on GKE

Describe the bug
This only happens on internal LB (not on external LB). When we uninstall SR cluster, the LB related GCP resource is not cleaned up, making the next round of SR helm install failing. It complains the LB IP I pre-alloc and put in helm yaml file - still IN USE.
I had to delete the related resources to enable SR cluster reinstall. Including:

the fwd rules. gcloud compute forwarding-rules delete xxx
the backend-services. get from gcloud compute backend-services list.

StarRocks Operator now does not handle GCP related resources. It just delete the resource it created.
StarRocks Operator does not do anything adaptation on public provider.


Syntax errors on older ruby version

Hi I'm using ruby 1.9.3 and I'm using sablon latest version. I'm getting syntax errors while using it because of the older ruby version obviously. I've tried to fixed it and successfully did it with no errors and used it to create word document. Recently I needed to print HTML with list so I tried but couldn't make it work because of this error.
ArgumentError: Could not find w:abstractNum definition for style: ''
Obviously its because of the changes I made for removing syntax errors and couldn't fix it.
I've tested latest version of gem in a new project and it worked fine with list. So I decided to add the latest version into my current project but having syntax errors again. I need help to fix the syntax errors because If I do it it would eventually end up in breaking something like last time. These are the syntax errors I'm having:
SyntaxError:
/lib/sablonnn/configuration/configuration.rb:17: syntax error, unexpected tPOW
def register_html_tag(tag_name, type = :inline, **options)
^
/lib/sablonnn/configuration/configuration.rb:18: syntax error, unexpected tPOW, expecting ')'
tag = HTMLTag.new(tag_name, type, **options)
^
/lib/sablonnn/configuration/configuration.rb:56: unknown type of %string
...s: :root, allowed_children: %i[_block _inline] },
...                               ^
/lib/sablonnn/configuration/configuration.rb:56: syntax error, unexpected $end
...s: :root, allowed_children: %i[_block _inline] },
P.s. I can't upgrade my ruby version.
Thanks in advance

@newbieeeeeee Typically a missing w:abstractNum style is an issue with the template not the gem itself. Sablon makes no effort to try and create some of the complex internals of MS Word and expects them to already exist in the template. Judging by the syntax errors listed I wouldn't think correcting them would break the code that processes HTML lists without breaking everything else.
If you still have your code modified to work with v1.9.3  I would try it again, however this time save an unordered list into the template document somewhere. Alternatively you could try using this template https://github.com/senny/sablon/blob/master/test/fixtures/insertion_template.docx with a context containing lists. That template document has all the required Word styles so if your code is correct it should work.

@stadelmanma Thanks for your help. I've fixed the syntax issues successfully. And it's generating word file. Although I got a new error which is:
ArgumentError: Could not find w:abstractNum definition for style: "ListBullet"
Which I successfully resolved too. After a lot of debugging I came to know that the version of the word template I was using was old. So I created a new template with same content and it worked.
Thanks again for your help.

@newbieeeeeee, I'm glad you were able to get things working!


Need to add tests for New-xDscResource and New-xDscResourceProperty commands

Partially fixed in PullRequest #7 by @dlwyatt


gpio_button (MEGH-4093)

gpio_button is necessary, can it be removed？
Now component espressif/button is used, there is a conflict；

Older esp idfs do not use component manager by default and so, this component has been retained. It was renamed from button to gpio_button to prevent some conflicts in cmake dependency resolution. Can you elaborate what is the exact issue you are seeing? I believe your compilation must be finding 2 instances of the same APIs. Is that so?

yes，compilation find 2 instances of the same APIs.
I noticed that the function name has been changed here to prevent conflicts, but the newly added component espressif/button use the same name as you.
I'll ask the maintainer of button component if they can handle it.
Thanks!


not getting relevant answers from asks

From my initial tests on some embeddings Im not getting consistent results.
.I can see the GetNearestMatchesAsync method called but the results are not related to the questions i ask .

Hi,
Can you share a sample code ?

sure .
For example a have stored in the database the following   memory record for an index named blogposts:
Elon Reeve Musk is a business magnate and investor.
Musk is the founder, chairman, CEO and chief technology officer of SpaceX; angel investor, CEO, product architect and former chairman of Tesla, Inc.
founder of the Boring Company; co-founder of Neuralink and OpenAI; and president of the Musk Foundation.

next i run a search where theask ="who is elon musk"
It returns all records even ones without elon musk.
Next I run theask ="who is steve jobs"
it returns again all records even though i have nothing on steve jobs
  var kernel = Kernel.Builder
                .WithLogger (_logger )
                .WithOpenAITextEmbeddingGenerationService("text-embedding-ada-002", decryptedOpenAiKey )
                .WithMemoryStorage(await SqlServerMemoryStore.ConnectAsync(_dbConnectionString))
                .Build();
             var memoryCollectionName = index;
             try
            {
                var memoriesResponse =   kernel.Memory.SearchAsync(memoryCollectionName, theAsk, limit: limit, minRelevanceScore: relevanceScore);
                await foreach (var memory in memoriesResponse)
                {
                   // Console.WriteLine($"Result {++counter}:\n  >> {memory.Metadata.Id}\n  Text: {memory.Metadata.Text}  Relevance: {memory.Relevance}\n");
                 }

            }

Ok thank you for this sample.
I'll take a look into it in the next week.
I will give you in touch.

@giannik,
After a try on Azure SQL Server, I get a match for the data you provided to me.
The relevance between your data and the ask is about 0.88 which seems to be good.
I also tested the same program with an Azure Cognitive Search (vector search). The relevance is about 0.89 (which is different, and I don't know why, be not so far).
Can you try to adjust your settings (min relevance), or see if other entries are simply more relevant to your question?
Regards,


[SPK] service, project, hld commands unit test hardening

As a spk 'services' developer, I should have better unit test coverage on commands and libraries I own. Figure out how to test and verify error cases where applicable.

@mtarng we should be adding testing goodness every time we touch an area. Creating a task to retrofit testing on existing code is very hard to estimate. Can we just do testing as we code?


Add easier feed management

[x] i.e. moving sources between folders

[ ] moving multiple sources at the same time

[ ] make folder nesting possible

ability to drag & drop folders, or reorder them in some other way would be good too
re: nesting folders - i haven't seen that in any other FF reader and i'm not sure how necessary that is, but that's just my opinion

ability to drag & drop folders, or reorder them in some other way would be good too
re: nesting folders - i haven't seen that in any other FF reader and i'm not sure how necessary that is, but that's just my opinion

Brief (firefox addon) has those capabilities, but SmartRSS looks better and is faster :(
i don't know wich one to choose

drag and drop is useful but gets messy with long lists, I'll look into it when I have some time
nesting may be useful and doesn't seem too complicated
@samthelion I picked up this project mostly because it was the easiest way of dealing with pain points in other readers, Newsfox was pretty much perfect but slow and not supported by quantumfox
as I resolved most of my personal issues development slowed down a lot, but I hope to get back to it sooner than later

actually i'm going to revoke my request for this feature
i've gotten used to having the 'folders' in alphabetical order

won't work on nesting, at least not in foreseeable future


vis.video with OpenCV 4

Bug Description
vis.video throws UnboundLocalError with OpenCV version 4:
Traceback (most recent call last):
  File "/Users/ohwi/Practice/visdom/samples/06_1. etc.py", line 24, in <module>
    title="Alohomora",
  File "/anaconda3/lib/python3.7/site-packages/visdom/__init__.py", line 335, in wrapped_f
    return f(*args, **kwargs)
  File "/anaconda3/lib/python3.7/site-packages/visdom/__init__.py", line 1064, in video
    fourcc,
UnboundLocalError: local variable 'fourcc' referenced before assignment

Reproduction Steps
vid = cv2.VideoCapture("assets/Alohomora.mp4")
	frames = []
	while True:
		ret, frame = vid.read()
		if not ret:
			vid.release()
			break
		frames.append(frame)

	vis.video(
		np.array(frames),
		opts=dict(
			title="Alohomora",
		)
	)

Expected behavior
Visdom writes video.
I think elif cv2.__version__.startswith(('3', '4')): at line 1055 of __init__.py may solve the problem.

Ah we haven't been doing a good job keeping up on visdom's video implementation at the moment. If adding the given line solves the problem for you, feel free to open a PR with it!


Archive this repo?

Hey @chrisjsewell,
Can we archive this repo? I assume you don't have the bandwidth to work on it right now, so I figure we should hide it from view :)

Heya, yeh no problem


Fix bug in concat where keys were improperly assigned

What do these changes do?

Now creates a MultiIndex correctly from the keys provided
Better error checking (closer to pandas)
Correct order of keyword arguments
Minor refactor

Related issue number
Resovles 316

[ ] passes git diff upstream/master -u -- "*.py" | flake8 --diff
[x] passes black --check modin/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-PRB/199/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/137/
Test PASSed.


Spinner not working since 10 nov 2017

I'm getting the following error on the console.
Uncaught SyntaxError: Unexpected token export
Incident:1654 Uncaught ReferenceError: Spinner is not defined
Please help as we are using SpinJs in production.
Thanks,
Shahrukh

me too.
http://fgnass.github.io/spin.js/spin.min.js
this link is broken

yes I already copied the code locally and then did that change, worked for me too. Thanks!

Check the release notes and #343; the project was upgraded to ES6. I'm guessing you should probably stick with the version 2 branch if you're not transpiling your code and bundling modules with Babel/Webpack.

I updated the website to clarify that spin.js shouldn't be loaded from a <script> tag directly referencing the official website, as this is likely to break your app whenever a major new version is released.


Add container resource reservation and limit specifications to docker-compose and ansible role definitions

Notes:

compose file resource directive

CPU resource settings:

cpus: setting
cpu_percent: setting

MEM resource constraint settings:

mem_reservation: setting
mem_limit: setting

NOT supported outside of swarm mode w/ v3 format: https://github.com/docker/compose/issues/4513


reorganize and extend readme

Checklist

[x] No CHANGELOG update needed

[x] No new tests needed

[ ] No documentation update needed

IMHO it's fine to leave as is.

Yes, all great points and ideally we refine the readme and documentation here a bit more thorough very soon. I'll make appropriate changes, open issues, and merge here then.


AllAnime webview issue

Source information
AllAnime 14.31
Source language
English
Steps to reproduce

Enter to an anime page
Press webview

Expected behavior
Opening of the webview of the anime
Actual behavior
It doesn’t open
Aniyomi version
r7801
Android version
14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Fyi, this problem still continues after the latest update 14.32 @almightyhak @imper1aldev


nodejs: use .finalPackage for ease of overriding

Description of changes
Note: This should be a trivial refactor with no rebuild (tested locally). Ofborg agrees.
Previously, the nodejs passthru.pkgs attribute refers to the package fixed point derived in the current context, without considering potential overrides to the main nodejs package.
This is fixed here with the .finalPackage attribute, which refers to the final package fixed point, after potential overrides.
For example, previously, after overriding nodejs with nodejs.override, nodejs.pkgs would still refer to the original package, instead of the overridden one. After this fix, nodejs.pkgs would be correctly based on the overridden nodejs package.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

The patch is designed to have minimal code changes so that it would not interfere with ongoing projects such as #327653.

However, I personally think the final result will look better if you'd replace the self usages with finalAttrs.finalPackage. I understand that this will enlarge the diff, so probably it will be better to do it after https://github.com/NixOS/nixpkgs/pull/327653 is merged, […]

The aforementioned PR already removes unnecessary uses of self, leaving us with pkgs attribute and tests. For this, I’ve been thinking that we could use nodejs argument from callPackage (perhaps with an explicit __splicedPackages where needed) because splicing does not work for finalAttrs.finalPackage. That said, splicing also doesn’t for with self, so this change looks OK to me. And thanks for making this change small!

Thank you! If the related changes can be directly incorporated into #327653, that would be great! This PR simply serves as a proof that such change would lead to no rebuild, as confirmed by ofborg.

If this PR is urgent for you @bryango we can merge this, and at https://github.com/NixOS/nixpkgs/pull/327653 they would deal with potential merge conflicts / rebasing issues after master will be merged into staging.

I am okay with either way! I do need this PR right now, but I can easily work around the issue by manually overriding passthru.pkgs, therefore it is not so urgent. I will leave you (tie and doron) to decide whether to merge this as-is or to wait for a better version later on! :wink:

I’m OK with merging this now, doesn’t look like there would be any conflicts. I’d like to avoid PR #327653 becoming de-facto fork of Node.js package, even if temporarily 😅
Nixpkgs review process for non-trivial changes is already slow as hell and occasionally adding small changes on top wouldn’t help with that.

I’d like to avoid PR #327653 becoming de-facto fork of Node.js package, even if temporarily 😅

What do you mean by making the PR a de-facto fork? Do you mean you use a Nixpkgs fork with #327653 applied to it?

What do you mean by making the PR a de-facto fork?

This PR significantly improves build times and fixes some platform support issues we’ve had before (i.e. being unable to compile between platforms with different bitness), so I assume that some people might rely on it.


Integrate with AWS Lambda Builders

Demonstrate how to easily build and package Serverless apps staying in a single language (Go, Python), possibliy leveraging AWS Lambda Builders https://github.com/aws/aws-lambda-builders in some fashion.
This note taken from @corymhall on-boarding document as there are no issues currently under org:pulumi mentioning this particular library.

Closed in favor of https://github.com/pulumi/pulumi-aws/issues/3980

@mjeffryes @t0yv0 both issues are closed. I am trying to figure out how to use Functions with Makefile. Any path forward for this?

@alankritjoshi sorry for the confusion, we were having some trouble figuring out where to put it ha! Here is the issue you should use, please upvote!
https://github.com/pulumi/pulumi-awsx/issues/1290


added x64 as a prefix to search for cl.exe in the toolchain file

this makes things work with VS '15' if you manually specify the triplet

Hi @barcharcraz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Thanks for working this out!


Design study for new owner drawn styles

A first attampt for owner drawn styles. For discussion only :^)

To check it, start the Mapsui.Samples.Forms.UWP sample and go to "Special". There select "Special Style". You could click also on the points. It works because of the new MapInfo function. Good work.

You could use Mapsui.Sample.UWP or Mapsui.Sample.WPF too.

And only the last part is relevant. Should have made an new branch for this. Sorry.

Close this and added a new one. See #839.


⚠ Bofrost.be has degraded performance

In 9e63305, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1988 ms

Resolved: Bofrost.be performance has improved in e6e3da5.


Insights Add-In:  Qualifying Messages

Team,
I'm wondering about trying to collect analytics for an email newsletter I send to specific distribution lists (DLs).  I have read about Qualifying Messages, but  am looking for further understanding.
In my case, I am looking to track the "Open" rates and (potentially) follow through clicks on links contained within the email message.
When I check MyAnalytics for these specific email messages, I notice I can't find them showing up in the Insights Add-In.  So then I saw the "Exceptions to qualifying messages" and this is what my feedback relates to.  Of the four listed exceptions, the only one that applies to my use case is:

Email that was sent more than 14 days ago -- is this something that can be expanded to, say, 90 days?  I'm looking for quarterly analytics.

Additionally, when it comes to "Open Rate", do I understand correctly that MyAnalytics can only provide data if all recipients have opted into MyAnalytics?

Hello gitvpic,
I contacted the product team with your questions. Here's the answer to your second one:
Q. Can MyAnalytics provide data on "Open rate" only if all recipients have opted into MyAnalytics?
A. The data for a recipient is reported only if that person has a MyAnalytics license and is opted in for processing.

For info about MyAnalytics licenses, see Assign licenses with a MyAnalytics service plan.
For info about opting in, see Opt back in.

As for your first question, I need to ask for clarification:
Do you mean that you want to see email-read data for all of the 90 days after the email was sent OR do you want to see that same 14 days of email-read data for up to 90 days after the email was sent?
Thank you,
-- paul9955


Make HTTP response code when backend can't be found configurable

Issue Template
Title: Make HTTP response code when backend can't be found configurable
Description:
We've historically returned 502s when a backend can't be found. Envoy currently returns 503s. It'd be nice if we could configure this rather than it being hard-coded.
I'm not sure where the optimal place to configure this would be, so wanted to start the discussion before implementing it.
cc: @AndresGuedez

@ahedberg I think this is a duplicate or closely related to https://github.com/envoyproxy/envoy/issues/7537. See also proposed API in https://github.com/envoyproxy/envoy/pull/8126. Would you mind collaborating with @lukidzi on this? I would love to see the entire problem of customizing local replies solved holistically as this keeps coming up.


UI: Consistent UX everywhere for choosing Users & Teams

The UX should be consistent across all the places we let the user choose Users & Teams including Glossary Reviewers, Entity pages, etc.

@vivekratnavel these two are related right https://github.com/open-metadata/OpenMetadata/issues/4537

Yes, they are both related


ASE not protected by Azure DDOS Protection

Please specify that this protection does not apply to App Service Environments. This phrase seems as they would be protected since they are deployed in a VNET, however they are not. "Policies are applied to public IP addresses associated to resources deployed in virtual networks, such as Azure Load Balancer, Azure Application Gateway, and Azure Service Fabric instances."

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

ID: 6f38b1bd-2ab0-71f1-a6e9-925b7af3389d
Version Independent ID: 9528f5cf-16e1-0550-dc3e-edc135c82167
Content: Azure DDoS Protection Standard Overview
Content Source: articles/virtual-network/ddos-protection-overview.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@torfulle Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @torfulle
I fixed your issue as requested in this PR:
https://github.com/MicrosoftDocs/azure-docs-pr/pull/55042
#please-close @TravisCragg-MSFT

@torfulle We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Remove deactivated users from groups

For current servers starting with https://github.com/zulip/zulip/pull/29498 (server-10, ZFL 303), deactivated users are no longer members of any groups.
On the client side, the change this requires is that when we get a realm_user/update event saying a user was deactivated, we should remove them from all groups. See API docs.
Chat thread: https://chat.zulip.org/#narrow/stream/378-api-design/topic/Groups.20events.20for.20deactivated.20users/near/1958907

@dashwave /fix

@dashwave /fix

@dashwave /fix

@dashwave-ci-dev /fix


Upgrade to Angular 10

fixes #4725
cannot update ngx-monaco-editor from 9.0.0 to 10... they only have a beta and it's pretty broken (loader.js does not exist)

Codecov Report

Merging #4724 into master will decrease coverage by 26.17%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #4724       +/-   ##
===========================================
- Coverage   54.44%   28.27%   -26.18%
===========================================
  Files        1120       58     -1062
  Lines       38961     4503    -34458
  Branches     4904        0     -4904
===========================================
- Hits        21211     1273    -19938
+ Misses      17574     3054    -14520
  Partials      176      176

Prodded this some more, looking good, just need to get the merge commit passed gates


Add cat history

This is based on #97, please merge it first

Rebased.

This was rebased to cleanup-commands. Please merge #110 before.

I think it's be nice to:

Show the created account state in the same format as the modified
Show modifications starting with the most recent

current:
./bin/pick cat test2 --history
Enter your master password
>
test2 was created on Thu Jan 5 21:03:47 2017
modified: Thu Jan 5 21:03:47 2017
  username: asdf
  password: lll
modified: Thu Jan 5 21:08:24 2017
  username: asdf
  password: 8HpjSoFzM4F5HWC8m2dFFBpcl

purposed:
./bin/pick cat test2 --history
Enter your master password
>
modified: Thu Jan 5 21:08:24 2017
  username: asdf
  password: 8HpjSoFzM4F5HWC8m2dFFBpcl
created: Thu Jan 5 21:03:47 2017
  username: asdf
  password: lll

Show the created account state in the same format as the modified

Good idea! modifiedOn and createdOn seem to be redundant information for the initial account creation, as they always equal :) I find the proposed format a bit confusing, but whatever 👍 Updated.

Show modifications starting with the most recent

I intentionally choose this way of representing an account history. Just image having edited an account 100 times and wanting to check the previous (i.e. second most recent) account history: You would have to scroll up quite a few pages. As you are most likely to search for recent account modifications, IMO it's best to have the newest one available at the very bottom (your Terminal history, chat logs, .. use the same order).

Agreed re the ordering of account history.


Method for quickly calculating spatial statistics for multiple images together (different samples from same cohort and replicates)

...I'm hoping do some of the spatial analyses presented in squidpy but across many different multiplexed images (e.g. different donors, sample types, and replicates). All of the examples online were for individual sample analysis. Is there a way to do this type of analysis across multiple images (use as replicates) for a given condition? I know I can load all of my sample data into one adata object, with columns for observations, but the x, y positions of each image will overlap. Loading each into its own adata object seems clunky, especially when I want to start collecting replicate information about spatial statistics. For example if I want to use sq.gr.co_occurrence, I would need to iterate through adata/unique conditions (images) then I may not have the same "cell type" in each image replicate making it hard to concatenate replicates there, since the output is not linked to the "cell type" key but just an nxn numpy array. I'm hoping that there is some simple implementation within squidpy that I am missing and you could point me there to help subset and perform statistics on replicates and subconditions.

hi @johnhickey22 ,
thanks for the interest in squidpy, we are very aware of this problem and are working toward making the analysis tools working on such settings. For now you could try simply concatenating anndata objects like this:\https://github.com/theislab/squidpy/issues/318#issuecomment-886541340\
in order to create a block diagonal obsp sparse matrix. The spatial statistics should work out of the box in that settings but again we haven't really tested it so it's something we don't know for sure. We  are hoping to cut a new release in ~a month in which that feature would be included.

Fantastic. Thanks so much about sending the workaround. I'll try it out and see how it works with images that do not share all the same clusters. Will look forward to the next release. Thanks for creating such useful tools for the community.

Is there a reason why do you not plot the co_occurance probability ratio on a log scale as default since this is a ratio?

hi @giovp - Thanks for pointing out the previous workaround. I've tried it for my data but does not look like it is working. I've tried to do another workaround by computing the probabilities individually and then concatenating the probabilities for each cell pair and location after the fact. However, I am running into issues with the interval scales being different between samples. My tissues are of different sizes, would it be possible to instead of putting in the n_steps and determining the interval automatically from _find_min_max function, and just specify the total interested distance (often the biology of interest is smaller scales than the entire image) and then also the step size. This would also allow me to concatenate replicates.
Another suggestion for plotting the co_occurance probability ratio on a log scale as default since this is a ratio, lower ratios tend to be underrepresented. Thanks for your help!

hi @johnhickey22 , with new release out soon we'll add support to build spatial graph across multiple slices. You can then run that function as is since the graph will be built by only considering points in the same slide. You can follow development here #516

#516 has been merged, you can now use the co-occurrence score across multiple slides, if you build the graph by setting the library_key argument. Let me know if you try it out, will close this but feel free to reopen if not solved.


OCPBUGS-15991: use ignition-proxy Service to populate ignitionEndpoint with strategy NodePort

https://issues.redhat.com/browse/OCPBUGS-15991

Thank you again for quickly putting in this change!
I've tested the changes out on @elsapassaro's environment (which reproduced the initial issue) and saw that it fixes the issue and the cluster has installed the additional workers.

/cherry-pick release-4.13


Refactor AnonSocket::read/write for blocking socketpair

This PR moves the actual read/write operations of AnonSocket::read/write into separate functions, in preparation for #3665. The new functions will be used as blocking callback.

@rustbot ready

Lgtm. Please squash


Move installation towards the top

It only makes logical sense to have the installation and setup towards the top as most other node module documentation does

Thanks, but I prefer keeping the API documentation as the most prominent, since many users already know how to use Node.js/npm/Chai.

Fair enough


Release v1.1.1

This seems to work. Thanks @bleykauf .

Thank you @doronbehar . Just released to Pypi


--check gives 'SarifFileSet' object has no attribute 'get_result_count_by_severity'

Seems there is regression in check command because of recent Python API change. See:
root@76bdb3291e3f:/# python3 --version
Python 3.8.10
root@76bdb3291e3f:/# sarif --version
SARIF tools v3.0.3
root@76bdb3291e3f:/# sarif --check error summary /root/2024-11-12T15\:03\:16.3207055+00\:00_vs_2024-11-12T15\:03\:38.5907415+00\:00_summary.Sarif

error: 0

warning: 2
 - SetGid: 1
 - SetUid: 1

note: 552
 - Default Level: 552
Traceback (most recent call last):
  File "/usr/local/bin/sarif", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.8/dist-packages/sarif/cmdline/main.py", line 61, in main
    exitcode = args.func(args)
  File "/usr/local/lib/python3.8/dist-packages/sarif/cmdline/main.py", line 402, in _summary_command
    return _check(input_files, args.check)
  File "/usr/local/lib/python3.8/dist-packages/sarif/cmdline/main.py", line 229, in _check
    counts = input_files.get_result_count_by_severity()
AttributeError: 'SarifFileSet' object has no attribute 'get_result_count_by_severity'

The same command works fine on sarif-tools 2.0.0.

Thanks for raising.  This is fixed in the main branch but it doesn't look like the fix is in the latest release.
L230 in main.py in main branch is good:
ret += input_files.get_report().get_issue_count_for_severity(severity)
L229 in main.py in 3.0.3 is bad (and 3.0.2 is broken too)
counts = input_files.get_result_count_by_severity()
Tagging @debonte , possible to make a new release?

Yes, this is a duplicate of https://github.com/microsoft/sarif-tools/issues/73 which is already fixed. I'll ship a new release today.

@danielszot, I just shipped 3.0.4 with the fix for this issue.


Duplicate "Data Collection" section

The section is available here and here.
Consider de-duplicating.

I believe both sections are now required by policy, such that I'll go on and close this as by-design. Thank you!


Fix Bug event page

Fix Bug event page, Responsive page

can i do this? please assign it to me

can i do this? please assign it to me

@ashutosh7i u can proceed with this issue

i dont see any bug in the page, please specify or i merge optimized code

I want to solve these issue

@shrutimeg you can proceed with this issue

I'd like to contribute to this, if its still available

@Athi223 u can proceed