Theming options for Gtk and Qt

Motivation for this change
In place of #70276
When I started to use NixOS at desktop I was wondered that there is no module to set Gtk and Qt widget and icon themes (in bare WM environment). Its no joy to use lxappearence, qt5ct and similar tools. These modules fix that issue by adding a way to configure themes declaratively.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @worldofpeace
TODO: ping jtojnar mid february

@worldofpeace

Could we configure cursor themes for gtk as well?
I believe it's just another key/s in gtkrc.

Done, thanks!

@worldofpeace @jtojnar merge please?

@wordofpeace @jtojnar ping

@gnidorah Could you expand the manual chapter "GTK/Qt themes" with a mention to this module and an example?

@rnhmjoj Done, thank you!

@rnhmjoj Thank you! arc-kde doesn't provide native qt style, instead it provides settings for 3rd party theme engine (kvantum)

doesn't provide native qt style

I figured it was different somehow, that explains it.

@rnhmjoj Thank you!

Thank you

:-1: I feel like at least @jtojnar @ttuegel and myself should have approved on this before it was merged. I even expressed that to @gnidorah. With release management I've had almost no time to look at these things, which I'm sorry about. I probably should have at least mentioned that here also for everyone to see, instead of the mentions I made on IRC.

IMHO this has waited long enough, I hoped it would be in 20.03 already. Let's merge it.

As mentioned, it may look like it has waited and received improvements off feedback, but there really hasn't been any.

👎 I feel like at least @jtojnar @ttuegel and myself should have approved on this before it was merged. I even expressed that to @gnidorah.

I'm sorry. I saw the changes you requested were completed and @gnidorah had to fix conflicts several times so I thought it would be better to merge it now.

-1 I feel like at least @jtojnar @ttuegel and myself should have approved on this before it was merged. I even expressed that to @gnidorah.

I'm sorry. I saw the changes you requested were completed and @gnidorah had to fix conflicts several times so I thought it would be better to merge it now.

Indeed, it's sad for something like this to go unnoticed.

#80854. Could you open another PR for this?
And I'd request review of @jtojnar myself @ttuegel on the qt module.
I'd for sure like to see @Infinisil :heavy_check_mark: on it since it's a new module.

It would also help, though I think you'd have to rewrite your history :frowning:, if the gtk module and the qt module weren't in the same PR. It just makes it easier to review.

Is there any way to override theme colors declaratively? I'd like to replace some colors with custom ones I have defined to be used throughout my configuration in variables

@pixpox You could try something like this. Note that Qt theming is broken due #98009
  environment.etc."xdg/gtk-2.0/gtkrc".text = ''
    gtk-theme-name="Adwaita-dark"
  '';
  environment.etc."xdg/gtk-3.0/settings.ini".text = ''
    [Settings]
    gtk-theme-name=Adwaita-dark
  '';
  # https://github.com/NixOS/nixpkgs/issues/87667
  environment.variables.QT_QPA_PLATFORMTHEME = "gtk3";
  environment.variables.QT_STYLE_OVERRIDE = "adwaita-dark";
  services.xserver.windowManager.i3.extraPackages =
    options.services.xserver.windowManager.i3.extraPackages.default ++ [ pkgs.adwaita-qt ];

Hm, that only sets the the gtk theme, if I'm not mistaken. I was referring to changing the individual colors. For now I ended up doing it like this: https://github.com/pinpox/nixos-home/blob/1cefe28c72930a0aed41c20d254ad4d193a3fa37/gtk.nix#L11
Maybe someone knows a simpler solution?

@pinpox That looks quite nice. Under which license is your configuration? I'd like to include a setup like yours in my the gtk.nix file but need it to be compatible with MIT.

@rycee Sure thing, go ahead. I have added MIT to the files, let me know if you have any questions

@pinpox Thanks, I've added it now.


Broken web build

Version
0.15
Operating system & version
MacOS, building for Web
What you did

Used leafwing-input-manager in a project
Web build:

cargo install trunk
rustup target add wasm32-unkown-unknown
trunk build --release

What you expected to happen
A working build, like for non-web builds.
What actually happened
The build crashed, with following error:
error[E0308]: arguments to this method are incorrect
   --> /Users/knarf/.cargo/registry/src/index.crates.io-6f17d22bba15001f/leafwing-input-manager-0.15.0/src/action_state/mod.rs:203:51
    |
203 |             .for_each(|action_datum| action_datum.tick(_current_instant, _previous_instant));
    |                                                   ^^^^ ----------------  ----------------- expected `std::time::Instant`, found `bevy::bevy_utils::Instant`
    |                                                        |
    |                                                        expected `std::time::Instant`, found `bevy::bevy_utils::Instant`
    |
    = note: `bevy::bevy_utils::Instant` and `std::time::Instant` have similar names, but are actually distinct types
note: `bevy::bevy_utils::Instant` is defined in crate `web_time`
   --> /Users/knarf/.cargo/registry/src/index.crates.io-6f17d22bba15001f/web-time-1.1.0/src/time/instant.rs:15:1
    |
15  | pub struct Instant(Duration);
    | ^^^^^^^^^^^^^^^^^^
note: `std::time::Instant` is defined in crate `std`
   --> /Users/knarf/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/time.rs:156:1
    |
156 | pub struct Instant(time::Instant);
    | ^^^^^^^^^^^^^^^^^^
    = note: `bevy::bevy_utils::Instant` and `std::time::Instant` have similar names, but are actually distinct types
note: `bevy::bevy_utils::Instant` is defined in crate `web_time`
   --> /Users/knarf/.cargo/registry/src/index.crates.io-6f17d22bba15001f/web-time-1.1.0/src/time/instant.rs:15:1
    |
15  | pub struct Instant(Duration);
    | ^^^^^^^^^^^^^^^^^^
note: `std::time::Instant` is defined in crate `std`
   --> /Users/knarf/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/std/src/time.rs:156:1
    |
156 | pub struct Instant(time::Instant);
    | ^^^^^^^^^^^^^^^^^^
note: method defined here
   --> /Users/knarf/.cargo/registry/src/index.crates.io-6f17d22bba15001f/leafwing-input-manager-0.15.0/src/action_state/action_data.rs:40:12
    |
40  |     pub fn tick(&mut self, _current_instant: Instant, _previous_instant: Instant) {
    |            ^^^^            -------------------------  --------------------------

For more information about this error, try `rustc --explain E0308`.
error: could not compile `leafwing-input-manager` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

Fixed in #590 already, but needs a point release <3 Pointing to that branch for now will get you building for now.

Any planned release date?

This should be live in 0.15.1 :)

thanks @alice-i-cecile ! it works


Autodeploy graphql gateway

Summary by OpenAI
Bug fix: Fixed a data race issue in the process function of my_module.py.

"Data races no more,
Our code is now secure.
With this fix in place,
We can now keep up the pace."

Closing in favor of #935


[Accessibility] Update aria-label for the pin "Add to status bar" under "Editor Languages status" {{}) control at the bottom of status bar.

VS Code Version: 1.88.0
OS Version: MacOS Sonoma (v14.6.1)

Issue: There are identical accessible names under "{} Javascript" inside the status bar at the bottom. Instead update the aria-label = "Add JavaScript version to Status Bar" accessible name.
[Recommendation]
Ensure headings and labels are descriptive and unique. This includes the accessible names of buttons. The accessible name of a button can be set with internal text, an aria-label attribute, or an aria-labelledby attribute. Good accessible names are both concise and descriptive. Avoid including the word "button" in the accessible name, as this information is already supplied by the button's role.
Actual behaviour: The accessibility user finds it difficult to determine between the two controls with same accessible name.
Expected behaviour: Need unique and descriptive accessible name for the pins under status bar's "{} Javascript" tab.
[Compliant Code Example]

Screenshot of the issue:

it looks like all pin buttons still have the same aria-label - "Add to Status Bar tool bar\nAdd to Status Bar button"

I see it working on macOS with VoiceOver: a pin action is now distinguishing between pinning and unpinning.
I would suggest to report a new issue for the Windows voice over issue.


Hello World

My second batch change!
Created by Sourcegraph batch change ryan.aznar/Ryan-Test.

Adding a test comment here!


JS

issue with printing humidity and date.
currently stating undefined

Completed,
Problems were: misspelling humidity, and syntax error with wind.


errors in log

In the synchronization log, I am seeing an error. Can you tell me how to remove them? Or can I ignore them?
2021-01-13 14:08:42,349 - ERROR: NetBox returned: PATCH /api/dcim/interfaces/253/ Bad Request
2021-01-13 14:08:42,349 - ERROR: NetBox returned body: {'type': ['Virtual and wireless interfaces cannot be connected to another interface or circuit. Disconnect the interface or choose a suitable type.']}
2021-01-13 14:08:42,349 - ERROR: Request Failed for interface. Used data: {'name': 'vmk0', 'type': 'virtual', 'mtu': 1500, 'mac_address': 'AC:1F:6B:BD:32:05', 'description': 'Management Network (vSwitch0, vlan ID: 150)', 'mode': 'access', 'untagged_vlan': 17, 'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: my-example'}]}
2021-01-13 14:08:42,350 - INFO: Updating NetBox 'interface' object 'vmk1 (<IP_ADDRESS>)' with data: {'name': 'vmk1', 'type': 'virtual', 'mtu': 1500, 'mac_address': 'AC:1F:6B:BD:32:04', 'description': 'VMkernel (vSwitch0, vlan ID: 150)', 'mode': 'access', 'untagged_vlan': 17, 'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: my-example'}]}
2021-01-13 14:08:42,396 - ERROR: NetBox returned: PATCH /api/dcim/interfaces/254/ Bad Request
2021-01-13 14:08:42,397 - ERROR: NetBox returned body: {'type': ['Virtual and wireless interfaces cannot be connected to another interface or circuit. Disconnect the interface or choose a suitable type.']}
2021-01-13 14:08:42,397 - ERROR: Request Failed for interface. Used data: {'name': 'vmk1', 'type': 'virtual', 'mtu': 1500, 'mac_address': 'AC:1F:6B:BD:32:04', 'description': 'VMkernel (vSwitch0, vlan ID: 150)', 'mode': 'access', 'untagged_vlan': 17, 'tags': [{'name': 'NetBox-synced'}, {'name': 'Source: my-example'}]}

It seems that you previously connected a cable from a swith to the server's network interface.
While syncing it probably found the interface by MAC address.
You need to connect the cable, in NetBox, to the newly created correct physical interface and then try to sync agian.

It seems that you previously connected a cable from a swith to the server's network interface.
While syncing it probably found the interface by MAC address.
You need to connect the cable, in NetBox, to the newly created correct physical interface and then try to sync agian.

thanks, I fixed the errors for the servers - and the errors in the log were gone

thanks, I fixed the errors for the servers - and the errors in the log were gone


prefer-immutable-types How to require readonly on the class variable's type, but not the property field?

I want to combine prefer-immutable-types with @typescript-eslint/prefer-readonly, so that I can keep all public and protected fields mutable but keep the data type immutable. But now seems prefer-immutable-types's variables setting apply to both the field immutability and also the variable type, and the ignoreClasses: "fieldsOnly" setting will turn off both. Is it possible to handle them separately? Thank you!

Could you give a code example of what you would like to be able to do?


[red-knot] Alphabetize rules

Summary
Follow-up from #14950.
Test Plan
Purely stylistic change. Shouldn't affect any functionalities.

Thank you!


Pinned Version of Proof General cannot parse newer Coq versions

What package(s) should be bumped?
proof-general
Why should Doom bump these packages early?
Proof General fails to parse a new kind of version string from Coq due to a spurious regex.
The result of this is a hard-fail right now if one enables coq-mode, it is rendered unusable.
The most recent relevant commit that just fixes the version parsing is 38ae8fdf23f37af71b8a4f9717ba48c52653db63.
I don't think there is anything else to do other than a simple version bump.

This issue should now be resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!


Don't include timestamp in generated manpages

This makes manpage generation reproducible. I found this problem using Debian's reproducible builds infrastructure.
--- old/dasel.1	2024-05-02 15:59:19.342436672 -0300
+++ new/dasel.1	2024-05-02 15:52:35.300986397 -0300
@@ -1,5 +1,5 @@
 .nh
-.TH "DASEL" "1" "May 2024" "Auto generated by spf13/cobra" ""
+.TH "DASEL" "1" "May 2024" "" ""

 .SH NAME
 .PP
@@ -73,8 +73,3 @@
 .SH SEE ALSO
 .PP
 \fBdasel-completion(1)\fP, \fBdasel-delete(1)\fP, \fBdasel-man(1)\fP, \fBdasel-put(1)\fP, \fBdasel-validate(1)\fP
-
-
-.SH HISTORY
-.PP
-2-May-2024 Auto generated by spf13/cobra

Thanks for the contribution @guilherme-puida


style(package): sort list of npm packages by name

If we keep this alphabetical, we'll avoid issues where npm install
automatically sorts and messes up what appear as git changes.

@joshdmiller any issues with that?

Nope. Sounds good!


Launching a file in the root of a project

Hello,
Fantastic plugin, has help me work a lot faster.
But I am having a problem:
When I try to launch a file with an ambiguous name (like index.html) that resides in the root directory of my project, the plugin is unable to target the 'index.html' in my root when there is an 'index.html' in a sub-folder. The plugin always goes to first using the following request:
http://localhost:8091/?message=index.html:1
I tried to change the file path to this but didn't have any luck:
http://localhost:8091/?message=/index.html:1
Please advise,
Thank you in advance,
Jamie

Hi @jalara,
I'm not going improve the plugin anymore, but you can fix the issue by yourself. There is a code that tries to define which file should be opened: https://github.com/zolotov/RemoteCall/blob/master/src/main/java/com/farpost/intellij/remotecall/utils/FileNavigatorImpl.java#L27
For now the recommended way to open files by http requests in using built-in PhpStorm Rest API: http://develar.org/idea-rest-api/#api-Platform-file

I know this is not the best place to ask this but develar.org site is down (404).
Is that API official? I can't find any documentation on google :(

@suside yes, the API is official. develar.org is up now, thank you.


Support bash style alias's to support tab completion in command string aliases

Summary of the new feature/enhancement

In bash, you can do alias's like so:
alias gb="git branch"

which is really useful for typing less.
In PowerShell we can do:
function gb {
    git branch @args
}

But here's why bash's are so much better:

tab completion still works. Where as for PowerShell they don't.
By not supporting this in some way, it means that alias's to native executables are better in bash because the experience is more fluid.
Also, the function approach that we have means we lose out on the work @rjmholt did in UnixCompleters.
Proposed technical implementation details (optional)

Perhaps a new kind of alias that will funnel tab completion all the way down to the alias'd command string?
Set-Alias gb 'git branch' -AliasType CommandString # vs ... CommandLiteral or something... naming is hard but you get the idea

In PowerShell we can do:

function gb {
    git branch @args
}

and if  someone add   gb alias  it  shadow  your gb function.  dont work as expected.

This is pretty much the first thing people (including myself) try to do with aliases. I've seen countless threads/discord conversations with a ton of folks tripping on this.
I'm tempted to say that Set-Alias should do this automatically if the definition:

Has spaces
Doesn't resolve to a existing command
Has a first "part" that does resolve

This is pretty much the first thing people (including myself) try to do with aliases. I've seen countless threads/discord conversations with a ton of folks tripping on this.

Really? The first thing you do is to do complex aliasing?
An alias is a short cut to a command name only and has been since V1. And in all the years I have moderated the PowerShell forum on Spiceworks this is not a highly requested feature. There have been requests but certainly not many.
I'd say this might be a nice to have - but only if the implementation does not break things.

If we pull on this string a bit:

How does the AST capture something like set-alias gb 'git branch', what gets passed in to existing mechanisms for argument completion or pseudoparameterbinding?
What happens in the following cases:

set-alias gb 'git bran'
set-alias gc 'git commit --a'
set-alias gc 'git commit --'
set-alias gi 'get-item -path'
set-alias gi 'get-item -f'
set-alias gi 'get-item -'

Basically, I'd like to understand where the breakdown lies between PowerShell's parser/AST design vs bash's expand and execute approach. Once we have a better understanding of that, we'll be better placed to evaluate whether this feature is achievable.

@rjmholt I don't think the parser would need to change.  Just the parameter binder and possibly a new CommandInfo (or alternatively, just some different code in AliasInfo.Parameters).  Well I say "just" but it's a pretty substantial work item, though I think it's worth it.
For native command argument completers, it might be fine to just expand the text and reparse before invoking them.  Kinda dirty, but maybe doable just for completions.

@doctordns

Really? The first thing you do is to do complex aliasing?

Yes. People who came from bash/zsh/fish will do this.
Take a look at this guide:
https://www.netlify.com/blog/2020/04/12/speed-up-productivity-with-terminal-aliases/
every single example is complex in the sense that it's a command string and not a particular command.

Really? The first thing you do is to do complex aliasing?

I'm pretty sure the first alias I tried to make way back when was to ipconfig /all.  I know that fits well into a function as well, but I don't think I'd call Set-Alias dap 'dotnet add package' all that complex.

And in all the years I have moderated the PowerShell forum on Spiceworks this is not a highly requested feature. There have been requests but certainly not many.

Haven't heard of Spiceworks, but I see it a lot on reddit/discord.  It's up there with Select-Object -Property vs ExpandProperty in starter walls.

I'd say this might be a nice to have - but only if the implementation does not break things.

That's fair, luckily I'm pretty sure breaks won't be needed for it.

/cc @JamesWTruher @daxian-dbw @SteveL-MSFT

Perhaps a more PowerShelly way to implement this would be to add a parameter like -Parameters e.g:
set-alias gc git -parameters commit, "--a"
# OR maybe parameter allows a single string with all parameters?
set-alias gc git -parameters "commit --a"

That would work for many cases, but doesn't handle 'weird' syntax, so you probably still need to have --% for those any way. I like the & --% idea as it reuses the existing operator.

It would handle it just the same way as you'd need to handle it even if there wasn't a separate -Parameters thing to work with.
You'd probably have to wrap everything in quotes there; you can't directly use %-- for this since you're passing data to a cmdlet, not a native executable, so you don't really have the option of not using PS's parsing rules for the input.

Another approach is Fish abbreviations. Implementing them might be simpler in terms of auto completion because they work by expanding the alias at the prompt itself.

Quote from https://github.com/PowerShell/PSReadLine/issues/1753#issuecomment-703863585:

IMO this is more of an editor feature (command line or other) that shouldn't require any PowerShell engine support.

We may consider supporting this in PSReadLine instead of PowerShell engine, so that it stays as an interactive feature only and won't be supported in a script execution.

@daxian-dbw this means that intellisense in vscode won't get this feature.

@TylerLeonhardt True, but then PSSA will just turn around and tell you that you shouldn't be using aliases in scripts.  😉

Really? The first thing you do is to do complex aliasing?

Yes. People who came from bash/zsh/fish will do this.

I'd like to add, this difference in mindsets comes from the fact that commands in UNIX land (unlike PowerShell) are usually already short enough that you won't need to alias them.  So the primary use case indeed is for stacking up some commonly used arguments.
@doctordns

Just want to add a pressing scenario that can't easily implement without aliasing -
Ttrying to implement a 'Take top one' function and following would only work functionally but have bad perf.
PS C:\Users\liangzhu> function toponef {$input | Select-Object -First 1}
PS C:\Users\liangzhu> 1,2,3 | toponef
1

This brings a performance penalty in the case of pipeline input - if I run dir c:\ -s | topeonef it will get to wait for the base stream completes (Compare to that - dir c:\ -s | Selelct-Object -First 1 works just fine doing early termination)
More in #15855 on the technical challenge to get parity in function

Just want to add a pressing scenario that can't easily implement without aliasing -

"filter" function works more fast. :-)

reopening as I think there is value in having this in future irrespective of however it gets implemented

I haven't followed this thread in detail, so there may be a better solution, but note that you can pass a [byte[]] array directly to the -Body argument, so to emulate the 7.3- behavior you can try to ISO-8859-1-encode the entire $body string:
$body = [System.Text.Encoding]::Latin1.GetBytes((
    "--$boundary",
    "Content-Disposition: form-data; name=`"PWD`"`n",
    $certPw,
    "--$boundary",
    "Content-Disposition: form-data; name=`"FILE`"; filename=`"certificate.pfx`"",
    "Content-Type: application/x-pkcs12`n",
    [System.Text.Encoding]::Latin1.GetString($bytes),
    "--$boundary--`n"
  ) -join "`n")

Note that any single-byte, 8-bit, fixed-width text encoding (all of whose code points must be valid) can be used to losslessly encode arbitrary binary data.
ISO-8859-1 encoding (which is closely related to, but not the same as Windows-1252) is special in that it is a true subset of Unicode, covering the 8-bit range of Unicode code points (� (NULL, U+0000) through ÿ (LATIN SMALL LETTER Y WITH DIAERESIS, U+00FF)
As such, you can cast arbitrary binary data to a .NET string, and the Unicode code points of the resulting characters will reflect the original byte values; e.g.:
[byte[]] $bytes = 0..255

$str = [string]::new([char[]] $bytes)

# -> 0..255
[int[]] [char[]] $str

Let me try a summary of sorts:
Supporting Bash-style alias definitions would provide the following benefits:

Ease of definition, closer alignment with POSIX-compatible shells.

The most frictionless syntax is the one suggested by @SeeminglyScience above, so that, say
Set-alias gb 'git branch' would just work, with no need for additional parameters.

If implemented that way, implicit pipeline-input support with support for streaming processing (line by line), whereas using a function to support that currently requires a cumbersome implementation as a proxy function  based on a steppable pipeline (because a function based on $input invariably collects all input up front; note that use of a filter is not an option in this case).

The tab-completion aspect is really an independent one, and for current aliases, which are merely name mappings, already works for PowerShell target commands, but not for external (native) programs.

For instance, say you've used Register-ArgumentCompleter to define completions for pwsh, and you've also defined Set-Alias psh pwsh; currently, alias psh then unexpectedly does not use the tab-completions defined for pwsh.

This asymmetry should be fixed: #21609

Given that the proposal at hand also requires the enhanced aliases to know the external executable name, fixing the above asymmetry could also provide tab-completion support for enhanced aliases.


New relationship type for modification

see helpdesk ticket
Re: Confirmation of curation session for PMID:30726745
Can we add a relation "affected_by" or some other phrase so that we can capture direct OF indirect modification event?
https://www.pombase.org/documentation/modification-data-bulk-upload-format

I think @Antonialock  will need this new relationship for  Ye De TAys phosphorylation sites so putting at high priority.
Let us know if it isn't clear what is needed.

Yeah I thought w could talk about this. I've always done these as phenotypes?

@Antonialock  is anything else holding up this paper?

Yeah I thought w could talk about this. I've always done these as phenotypes?

I thought these were orb6 phosphorylation sites (proteomics data)
like the Mathew Swaffer dataset?
https://www.pombase.org/reference/PMID:27984725
(at least that is what Ye Dee Tay implied in an early e-mail.
Isn't this the case?  (I did not look at the paper yet)

they determine phosphorylated sites in WT + in orb6 mutant
they then compiled a list of the proteins that are differentially phosphorylated in the mutant compared to wild-type.
then they looked at protein motifs and narrowed down a list of "strong candidates"
they tested two of their strong candidates experimentally

checking the Swaffer/Nurse papere
To rigorously define a class of direct CDK substrate sites, we identified phosphorylation events at the minimal CDK consensus sequence (S/T-P), which decreased more than 2-fold after CDK inactivation and whose dephosphorylation was immediate and continuous, fitting an exponential decay (n = 275) (Table S1) (Figures 1C–1E and andS1G).S1G). These sites are enriched for the full CDK consensus site (+3 K/R) (Songyang et al., 1994) (Figure S1I) and do not overlap with non-specific 1-NmPP1 effects (Koch et al., 2011) (Figures S1E and S1F). It is possible a fraction of these sites are phosphorylated by kinases regulated very rapidly downstream of CDK. Phosphosites at the consensus site of other kinases do change in this experiment (e.g., Plk1 and Aurora), although the vast majority of these were not dephosphorylated immediately and did not fit an exponential decay (data not shown). Many of the proteins we identified have been previously characterized as CDK substrates, including orthologs of proteins with mitotic functions, such as SMC4 (Cut3), Cdc25, Plk1 (Plo1), Survivin (Bir1), INCENP (Pic1), Ask1, and TOG (Dis1 and Alp14), and proteins involved in DNA replication, such as Sld2 (Drc1), Sld3, Fen1 (Rad2), Orc1, and Orc2. Furthermore, these sites are enriched for cell-cycle-related GO categories as well as being enriched on the orthologs of human and S. cerevisiae CDK targets (Table S2) (Holt et al., 2009, Hornbeck et al., 2015). This, and the fact that these data can be used to derive half-lives that are quantitatively reproducible between biological repeats (Figure 1B), corroborates our conclusion that this set of 275 sites predominantly constitutes direct in vivo CDK substrates.

What is the consensus site for orb6 and how many candidate substrates do they have?
Matthew was probably confident because of the combination of rapid phosphorylation and a strong CDK consensus, although they say some may be indirect they think it unlikely (I remember @mah did check, and lower confidence sites that did not satisfy all these criteria were excluded- it wasn't the full proteomics dataset).
Even if Tay's experiment was the same, unless there is a strong phosphosite associated with orb6 maybe we should be a bit more conservative?
but we can include all as modifications "affected buy ' orb6

So basically we want to  include modification data for all of the proposed targets with the new relation affected_by orb6 . OK?
https://github.com/pombase/pombase-chado/issues/726

Yeah I thought w could talk about this. I've always done these as phenotypes?

Do we have many other examples? I can't think of any. In small scale they would probably do some more experiments to confirm. Or we have historically used "phosphorylation" process with these as targets. This makes them easy to locate and add  "affected_by"  with the kinase if we do not know that it is direct.
We need to capture this information because it is really informative about the network, even if we cannot make a direct connection.
In this session we have no record of the other targets.
https://www.pombase.org/reference/PMID:30726745

So the action for this itcket is to add the relation affected_by for use with modification data extensions.

So the action for this itcket is to add the relation affected_by for use with modification data extensions.

That was done on August 14th.
I'll add it to the documentation now.

@Antonialock  I think  you could also use the affeted_by list
to provide phenotypes of orb6, in addition to the modifications


[Question] Problems switching from 1.16 to 1.17 - v2 to v3 static configuration

Hi, after several attempts of trying i gave up and cant figure out how to make this configuration work with latest envoy version 1.17.
Could you please help?
Note: Working with 1.16 (though some warning about v2), but not working with 1.17
I understand that i have to switch to
extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
But this does not seem the only change necessary
admin:
  access_log_path: /tmp/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 9901
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address:
        protocol: TCP
        address: <IP_ADDRESS>
        port_value: 443
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
          stat_prefix: ingress_http
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains: ["*"]
              cors:
                allow_origin_string_match:
                  - prefix: "*"
                allow_headers: "content-type, authorization, origin, referer, user-agent, x-grpc-web, x-user-agent"
                allow_methods: "POST,OPTIONS"
              routes:
              - match: { path: "/status" }
                direct_response: { status: 200, body: {inline_string: "OK"} }
              - match:
                  prefix: "/rest/myinternalservice"
                route:
                  prefix_rewrite: "/"
                  cluster: myinternalservice
          http_filters:
          - name: envoy.cors
          - name: envoy.router
            config: {}
      tls_context:
        common_tls_context:
          alpn_protocols: "h2"
          tls_certificates:
            - certificate_chain:
                filename: "/etc/letsencrypt/live/DOMAIN/fullchain.pem"
              private_key:
                filename: "/etc/letsencrypt/live/DOMAIN/privkey.pem"
  clusters:
  - name: myinternalservice
    connect_timeout: 1.0s
    type: LOGICAL_DNS
    dns_lookup_family: V4_ONLY
    lb_policy: ROUND_ROBIN
    load_assignment:
      cluster_name: myinternalservice
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: <IP_ADDRESS>
                port_value: 9600

@masbaehr
can you try:

admin:
  access_log_path: /tmp/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 9901
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address:
        protocol: TCP
        address: <IP_ADDRESS>
        port_value: 443
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          stat_prefix: ingress_http
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains: ["*"]
              cors:
                allow_origin_string_match:
                  - prefix: "*"
                allow_headers: "content-type, authorization, origin, referer, user-agent, x-grpc-web, x-user-agent"
                allow_methods: "POST,OPTIONS"
              routes:
              - match: { path: "/status" }
                direct_response: { status: 200, body: {inline_string: "OK"} }
              - match:
                  prefix: "/rest/myinternalservice"
                route:
                  prefix_rewrite: "/"
                  cluster: myinternalservice
          http_filters:
          - name: envoy.cors
          - name: envoy.router
      transport_socket:
        name: envoy.transport_sockets.tls
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.DownstreamTlsContext
          common_tls_context:
            alpn_protocols: "h2"
            tls_certificates:
              - certificate_chain:
                  filename: "/etc/letsencrypt/live/DOMAIN/fullchain.pem"
                private_key:
                  filename: "/etc/letsencrypt/live/DOMAIN/privkey.pem"
  clusters:
  - name: myinternalservice
    connect_timeout: 1.0s
    type: LOGICAL_DNS
    dns_lookup_family: V4_ONLY
    lb_policy: ROUND_ROBIN
    load_assignment:
      cluster_name: myinternalservice
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: backend
                port_value: 9600

Thanks the transport_socket part makes it work again, probably it did not recognize tls_context
👍
May I also ask if i have to change anything for the JWT config? Tried to add this and the service does not start up anymore.
This is the correct documentation?
https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/http/jwt_authn/v3/config.proto
     http_filters:
      - name: envoy.cors
      - name: envoy.filters.http.jwt_authn
        config:
          providers:
            cognito-pool:
              issuer: https://cognito-idp.eu-central-1.amazonaws.com/eu-central-1_000000000
              forward: true
              audiences:
              -<PHONE_NUMBER>000000000000
              local_jwks:
                filename: /home/xx/envoyconf/jwks.json
              #remote_jwks:
              #  http_uri:
              #    uri: https://cognito-idp.eu-central-1.amazonaws.com/eu-central-1_000000000000/.well-known/jwks.json
              #    cluster: service_google
              #    timeout:
              #      seconds: 5
              from_headers:
              - name: Authorization
                value_prefix: ""

im not certain - i havent any experience with that filter
if you check the envoy startup logs it should point you to problems in the config
for example - your original config had the following error
testing_1  | Protobuf message (type envoy.config.bootstrap.v3.Bootstrap reason INVALID_ARGUMENT:(static_resources.listeners[0].filter_chains[0]) tls_context: Cannot find field.) has unknown fields

The time out value in yaml should be
<INDENT>timeout: 5s

How should you find an error when an error message looks like
" Unknown field in: { ...dump of whole config file... }"
Could at least the name of the offending field be put into the error message?
" Unknown field 'some.offending.field' in: { ...dump of whole config file... }"
Sorry that reminds me of some linux tools in the 90s when you had to use m4 macro compiler to compile a config and got basically no information when something was wrong. I love envoy and you are certainly doing an amazing job. But the error
reporting needs more love in general ;)

Downgraded to envoy 1.16.2 - that shows a lot more error messages with details.

@RobIsHere I also have come across the same scenario. And then I tried with --bootstrap-version 3 argument. Then it provided a meaningful message indicating where exactly is the issue.


feat: Re-issue/send PDAs when first-party endpoint certificates are renewed

Fixes #185. It's also worth highlighting the following:

It introduces the instance method FirstPartyEndpoint.authorizeIndefinitely(), which is like FirstPartyEndpoint.issueAuthorization(), except that it tracks which 3rd party endpoints are authorised in order to renew their PDAs automatically.

For example, Awala Ping will continue to use FirstPartyEndpoint.issueAuthorization() but Letro will use FirstPartyEndpoint.authorizeIndefinitely().
To stop renewing an indefinite PDA, it's enough to delete the respective 1st or 3rd party endpoint.

This PR introduces the concept of "channel" (e.g., ChannelManager), which exists in the Awala protocol suite and refers to an established communication channel between two endpoints (which is end-to-end encrypted and where both are mutually authorised to send messages to each other).
Because #175 isn't implemented yet, I haven't actually integrated the function to reissue+send PDAs (FirstPartyEndpoint.reissuePDAs()). This should be done as part of #175.
This PR integrates CertificationPath from the core Awala lib, which effectively duplicates the existing AuthorizationBundle data class. Once this PR is merged, I'll create a separate PR to replace the remaining uses of AuthorizationBundle.

Review notes
Of things that I'm likely to have got wrong here, I'd rank highly the use of SharedPreferences and coroutines (esp. where I have to specify a context explicitly), so please pay special attention to those.

Hmm, one thing I just realised is that FirstPartyEndpoint.authorizeIndefinitely() and FirstPartyEndpoint.issueAuthorization() must actually return ByteArrays -- otherwise we'd be leaking the Awala lib to the apps using this lib.


added REST API guide

NOTE:

Please fill out the template below

Did you read the instructions in the README?

Y

Have you read the instructions in CONTRIBUTING?

Y

Have you written this article by yourself? Have you taken inspiration from anywhere and then written it in your words?

[I have used the quora link to study it]

@RN0311 done

Merging this.


Add docker server registry

Fixes: #27937

Hi @sftim  if i understood it correctly i have to make two examples for users who want to use docker server registry and one for the user who don't want to use docker server registry?

You can either:

have just one example. The one example sets --docker-server, and the JSON output matches that.
have two examples. The first example doesn't set --docker-server, and the JSON output matches that. There is a second tab that does set --docker-server and for that tab the JSON output does show a custom registry URL.

Either is fine - pick which you prefer.

@sftim Ok, i would like to go with option 1 , i will add commit accordingly

@sftim added a commit for the same . PTAL

@sftim Could you please take a look on this.
Thanks

@sftim  Could you please take a look on this and let me know if any other  changes are needs to be done on this.
Thanks!

/lgtm

/approve


Do not validate details when status code is OK

Related PR: https://github.com/grpc/grpc/pull/22015


typo on plugin distribution page

Accept my pre-entive apologies for probably not logging this right:
On the page https://getkong.org/docs/0.5.x/plugin-development/distribution/
There is a sentence that starts with "It can be" and that's it. That's the full sentence.
I think you accidentally a word, or two.

Has been fixed, thanks!


Update README.md

Updated Readme to add a high level overview of OpenDevin

Thanks a lot for the suggestion! But we completely did the README in this commit, which is somewhat duplicated with this one: https://github.com/OpenDevin/OpenDevin/pull/28
We look forward to future contributions!


remove duplicate 'use strict' statements when concatenating?

When using grunt-ng-annotate like this:
ngAnnotate: {
  app: {
    files: {
      "concat.js": ["a.js", "b.js", "c.js"]
    },
  },
}

...if you have "use strict'' statements in those source files the concatenated file then contains multiple repetitions of the statement.
An option  to prevent this would be great for cleanup purposes.

+1


Build(deps-dev): Bump cypress from 9.6.1 to 9.7.0 (#3534)

Pull request opened by the merge tool on behalf of #3534

Merged in 2c2855984f37446f4320ccfc48ac2a93e086fb9b


Fix warning compiling for macOS 13

Short description 📝
There's a deprecation warning when building with Xcode 14 targeting macOS 13:
Sources/XcodeProj/Extensions/String+md5.swift:63:16: warning: 'init(bytesNoCopy:length:encoding:freeWhenDone:)' was deprecated in macOS 13: String does not support no-copy initialization
        return String(bytesNoCopy: ptr, length: hexLen, encoding: .utf8, freeWhenDone: true)!
               ^

This is because String.init(bytesNoCopy:...) was deprecated in macOS 13: https://developer.apple.com/documentation/swift/string/init(bytesnocopy:length:encoding:freewhendone:)
Further discussion here: https://forums.swift.org/t/does-string-bytesnocopy-copy-bytes/51643/11
Solution 📦
Use String.init(cString:) instead.
Implementation 👩💻👨💻

[x] Use String.init(cString:) instead of String.init(bytesNoCopy:...)
[x] Deallocate bytes from pointer after creating the string

I made a few updates after realizing that the initial code wasn't safe. Should be good to go now.

CI didn't run on the latest commits of this PR.
Tests failed on main, but they pass for me locally when running swift test with Xcode 14b4.

Sorry folks had to revert this 😞 as it was causing some reference generation instability (tests caught this when run repeatedly)

I understand. I don't see how what I wrote is unsafe, but it clearly is.


Add discriminative margin based clustering loss

Cleanup of the previous PR. History can be found at https://github.com/chainer/chainer/pull/4525

Regarding this comment about the error propagation through mean values, sometimes we intentionally stop error propagation from the intermediate values (e.g. Target Q-Network in Deep Q Network). So, it could be possible the current implementaion is correct. I'm wondering how the original implementation deal with it.

@delta2323 I fixed the build issues

OK, I'll take a look.

jenkins, test this please.

Jenkins CI test (for commit 2b877bf490569ed0654a957bf9eecc7ce3ab475f, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

jenkins, test this please.

Jenkins CI test (for commit 0afcc47187c34ded60ccf0234749f5b34c1a903a, target branch master) succeeded!

LGTM


Templates: Beauty Base Animations & Links

Summary
Adds animations to the Beauty template as close to mock up as we can with existing primitives & links.
Relevant Technical Choices
NA
To-do
https://docs.google.com/spreadsheets/d/1iJ8Adqs4zGWteMoxVCZ2MM4LAza0LQmtPUX6I9qdshY/edit#gid=21496487
User-facing changes
Only user facing changes should be the links being added in which shouldn't effect anything other than hovering over the link circles on page 4
Testing Instructions
Links
Go to page 4 and hover over the circles with your mouse in the editor. you should see a link to Example Domain pop up.
Animations
set enableAnimations to true in includes/Dashboard.php. then go view the template in explore templates and see the initial pass of animations.
Screenshots

(Recorded at 15fps to keep file size down. to view in high fidelity, run this branch live)
![Kapture 2020-07-07 at 16 31 31](https://user-images.githubusercontent.com/35983235/86850961-62d12d00-c06f-11ea-8750-feeb25b7781a.gif)

Fixes #2211

Looks awesome!

@googlebot I consent.


Add greek cities geo data

We can get data from this site: https://geodata.gov.gr

Visiting https://geodata.gov.gr/dataset/poleis & choosing GeoJson && WSG format is the most suitable resource for this open issue.

Let's add the below helper fns for the cities:
getCities({ locale = "el" }: options) => City[]
searchCityByName({searchTerm: "Αθήνα", locale? = "el"}: options) => City[]
getCityById({ id, locale = "el" }: options) => City
getCityRelations({ id, locale = "el", type: "region" | "unit" | "municipality" | "prefecture" }: options) => Region | Unit | Municipality | Prefecture | null

fixed with https://github.com/tsevdos/elUtils/pull/83


Release google-cloud-java v0.86.0

This pull request was generated using releasetool.
04-03-2019 10:22 PDT
Asset

Regenerate asset client (#4837)
Add Cloud Asset v1 API. (#4834)

AutoML

Regenerate automl client (#4882)
Regenerate automl client (#4838)
Regenerate automl client (#4822)

Bigtable

Regenerate bigtable client (#4841)

BigQuery Data Transfer

Regenerate bigquerydatatransfer client (#4839)

BigQuery Storage

Regenerate bigquerystorage client (#4840)

Container

Regenerate container client (#4842)

Container Analysis

Regenerate containeranalysis client (#4843)

Core

auto close input stream (#4878)

Data Labeling

Regenerate datalabeling client (#4883)
Adding datalabeling. (#4872)

Dataproc

Regenerate dataproc client (#4844)

Dialogflow

Regenerate dialogflow client (#4845)
Regenerate dialogflow client (#4823)

DLP

Regenerate dlp client (#4846)
Regenerate dlp client (#4824)

Error Reporting

Regenerate errorreporting client (#4847)

Firestore

Regenerate firestore client (#4848)
Regenerate firestore client (#4825)

IAM Credentials

Regenerate iamcredentials client (#4849)

IoT

Regenerate iot client (#4850)
Regenerate iot client (#4826)

KMS

Regenerate kms client (#4851)

Language

Regenerate language client (#4852)

Logging

Allow user to specify custom LoggingOptions (#4729)
Regenerate logging client (#4853)

Monitoring

Regenerate monitoring client (#4854)

OS-Login

Regenerate os-login client (#4855)

Pub/Sub

Regenerate pubsub client (#4856)
Regenerate pubsub client (#4827)

Redis

Regenerate redis client (#4857)

Scheduler

Regenerate scheduler client (#4858)

Security Center

Regenerate securitycenter client (#4859)

Spanner

fix flaky TransactionManager tests (#4897)
Regenerate spanner client (#4860)

Speech

Regenerate speech client (#4861)

Talent

Regenerate talent client (#4884)
Regenerate talent client (#4862)
Regenerate talent client (#4828)

Tasks

Add tasks v2 client (#4879)
Regenerate tasks client (#4863)

Text to Speech

Regenerate texttospeech client (#4864)

Trace

Regenerate trace client (#4865)

Translate

Add translate v3beta1 API. (#4870)
Adding the translate beta samples (#4880)

Video Intelligence

Regenerate video-intelligence client (#4866)
Regenerate video-intelligence client (#4829)

Vision

Regenerate vision client (#4867)
Regenerate vision client (#4830)

Web Risk

Change .yaml file name for webrisk client. (#4898)
Add Cloud Web Risk client. (#4881)

Web Security Scanner

Regenerate websecurityscanner client (#4868)

Dependencies

Upgrade gax to 1.43.0. (#4836)

Release is at https://github.com/googleapis/google-cloud-java/releases/tag/v0.86.0


Enabling multi-Output via Variant and multi-Query via VariantF

I'm not sure how desirable this is, but I thought it would be worth raising for discussion.
Right now, a Hooks-based component can only emit a single message. However, if we could force the component's message type to be a Variant, one could define hooks that can raise their own messages
-- current
Hooks.raise outputToken messageValue

-- future
Hooks.raiseV outputToken sproxy messageValue

A similar idea could apply to queries:
-- current
Hooks.query queryToken queryHandler

-- future
Hooks.queryV queryToken sproxy queryHandler

If a Hooks-based component uses useQuery multiple times, only the last one is used. Thus, I don't think I can implement this idea on top of Halogen Hooks without using unsafePerformEffect at some point.

The reason a Hooks-based component only supports one type of query or output is simply because a Halogen component can only receive one type of query and raise on type of output. And that's the same reason why you can only use useQuery once (or, put differently, why if you use it many times only the last one is used).
Ultimately the useQuery gets used as the handleQuery function in the component, so there's not really a way to have many of them.
Put differently, are you running into something you can do in a Halogen component that you can't do with Hooks?

Put differently, are you running into something you can do in a Halogen component that you can't do with Hooks?

Not at the moment. However, I do think adding more functionality might be helpful in various as-yet-unknown situations.
Correct. A component can only have one action type and one query type. However, if you forced the component's action type to be Variant / forced the component's query type to be VariantF but left the rows to be defined by the end-user, then one could define hooks that can also handle their own actions / queries.
Do I currently see a use-case for this right now? No, but the idea exists and I'm curious what would be possible if it was implemented (assuming it can be).

Working with Variant and VariantF is a lot more tedious than working with normal sum types for messages and queries in components. I'm hesitant to make any change that would require using them unless there were a really good reason to do so. That's even more the case given that using Variant or VariantF for these types is quite rare.
If I'm understanding you correctly, you can use event sources to emulate what you're describing. You can have an event source that emits messages from the hook to the parent component, for example.

Ok. I'll close this then. This idea might not be practical, and until there's a definite use-case for it, I won't really know whether it's worth it.
Thanks for your thoughts on the matter though!


Re-add the "frame" span for tracy comparisons

Objective
In https://github.com/bevyengine/bevy/pull/6503 there were two changes to the top level tracing span "frame. Firstly it was renamed to "main app" and secondly the scope was reduced to only the main app. This means that there is no longer a top level span that can be used to compare frame times.
In addition to making it easier to compare across versions again, it also helps when running without the render feature. Previously the "frame" span was present in both cases but now to compare "headless" and render the comparison is between "main app" and "winit event_handler" and Tracy doesn't handle comparing non-matching frames well.
Before (0.9)

After (0.10)

Solution
This PR reintroduces the "frame" span so comparisons across versions and features is again easy.

Alternative
As an alternative route, the tracy-client crate that is used by tracing-tracy supports the ability to set multiple "Frame" markers using the secondary_mark_frame function. It may be possible to create a PR for tracing-tracy that exposes that functionality and then publish an "update" frame marker instead. This might have additional benefits in the future as it could be used for other systems we might want to track as frames without endless nesting.

I don't like the term "frame" for this span. The frame time is the time between when one frame is presented to the next. Without pipelined rendering the span misses capturing the i/o processing time. When pipelined rendering is enabled, this span is only vaguely related to the frame time. With the current behavior the new span is more like "main app + sub apps".

I don't like the term "frame" for this span.

That's fair. What about something like "update"? It's explicit about what the span is and "updates per second" is a fairly common metric.

Without pipelined rendering the span misses capturing the i/o processing time.

Sorry, I don't understand, which i/o processing is missed? Don't all the services run within an update cycle? Should I add something else to the span to help cover this case?

"update" sounds good to me.

which i/o processing is missed?

The winit runner processes events and then calls app.update(). You'll see these as the small winit event_handler spans between the bigger ones that includes the main app spans. It's hard to create a span that covers these since we'd need to create the span on one call to the callback and drop it on a different call to the callback function. https://github.com/bevyengine/bevy/blob/main/crates/bevy_winit/src/lib.rs#L652

What about calling it specifically "app_update"? Just "update" sounds a bit vague to me, and what if you have more subapps?.

In context I think "update" should seem less vague. The two places it will appear is in the Tracy view and in printed log messages. In both cases there will be context either side that helps. Related, because it's printed in line with log messages, there is an incentive to keep it short if possible.
e.g.
2023-04-19T09:07:21.407456Z  INFO bevy_app:update:main app:schedule{name=Outer}...

Anyway, I can change it to "app_update" if you still prefer it.

@MJohnson459 once merge conflicts are resolved I'm ready to merge this.


NIFI-3363: PutKafka NPE with User-Defined partition (0.x)

This PR is meant for 0.x branch.

Marked PutKafka Partition Strategy property as deprecated, as
Kafka 0.8 client doesn't use 'partitioner.class' as producer property, we don't have to specify it.
Changed Partition Strategy property from a required one to a dynamic property, so that existing processor config can stay in valid state.
Fixed partition property to work.
Route a flow file if it failed to be published due to invalid partition.

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@ijokarumawak please close this PR as it was merged to 0.x branch

Thanks @olegz !


fix: toggle secure for clickhouse migration

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1630  👈 (View in Graphite)
#1629
main

How to use the Graphite Merge Queue
Add the label merge-queue to this PR to add it to the merge queue.
You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.


Eliminate hardcoded object names in tests (and elsewhere?)

Lots of tests rely on hardcoded object names in the simulator, etc.  It would be good to refactor these out to a common module so we didn't have hardcoded names scattered throughout different places in the code, and just call methods/attrs in that module when we need an example valid object name.

Yes please! We might not need a module to do it, just using some methods in the backend object to get a variantset (say) and then storing the ids as instance variables on the test case should be fine. Ideally this should be done along with ID obfuscation, as this will break a lot of test cases and we might as well do it all in one go.

Fixed in #558


[Hexagon] Allow execution on target or simulator from HexagonLauncher

Setting ANDROID_SERIAL_NUMBER=simulator will execute the tests on simulator instead of a hardware device.
This patch also introduces an environment variable HEXAGON_RPC_LIB_DIR to specify the location of the hexagon_api binaries. If unset, the code will look for the binaries in the same way as before this patch.

cc: @mehrdadh @masahi

please fix the lint issues

@mehrdadh Yes, I'd like to do that.  I'm also going to be switching our downstream CI to use the HexagonLauncher mechanism.


user control: overridable OnDoubleClick event

Is it possible to make the benchmark hint disabled?
If yes, is it possible to allow for a DoubleClick event on the plot for implementing custom functionality (like opening the chart in a bigger window if double clicked)?
Thanks in advance for your help.

Is it possible to override the double-click event to do something custom (instead of toggling benchmark display)?

I like this idea! I'll implement this shortly and add a demo which shows how to override the event.

@FadyDev2 I just committed changes which I think solves this issue. I created a demo program which demonstrates these features too. These features aren't on NuGet yet, but I'll probably make a release this weekend.
Disable double-click benchmark toggling
https://github.com/swharden/ScottPlot/blob/bc6b1d6c3cf12601a2f3d309a09892c10a83fb45/demos/WinForms/Demos/FormOverride.cs#L28
Perform custom action on double-click
FormsPlot has some custom events which you can bind to:

https://github.com/swharden/ScottPlot/blob/3524818f6f90013154ade97ec35e1781e42eddf7/demos/WinForms/Demos/FormOverride.cs#L55-L59

this feature is now in the latest NuGet package
https://github.com/swharden/ScottPlot/releases


Slow performance reading SEG with 98 segments

I have this notebook testing the time to run the seg example on a file with ~100 segments.  It takes over 8 minutes to load this file and print the simple metadata.
Is this expected?  Can we expect any optimization as the library matures?  Working with the same data in a research format such as nrrd is orders of magnitude faster.
@hackermd @fedorov

Here is the notebook:
https://colab.research.google.com/drive/1ZLqJwDIO1XKnnjOzClkSq8RIawm3sp9M?usp=sharing

Hi @pieper
The iter_segments was an early solution for accessing segmentation pixels, but has largely been replaced by a set of methods on the Segmentation class itself. Unfortunately this has not yet made it into the User Guide yet, though I do have a branch on it. It is fairly well documented in the API docs for the segmentation class however. Iterating through segments is not efficient nor convenient in many situations. Would something like this be more suited to your use case? This runs considerably faster for me (though there may still be room for performance gains):
import pydicom
import highdicom as hd

seg = hd.seg.segread("/tmp/ABD_LYMPH_008_SEG.dcm")
print('read file')

source_uids = seg.get_source_image_uids()
source_sops = [uids[2] for uids in source_uids]

pix = seg.get_pixels_by_source_instance(
    source_sops,
    ignore_spatial_locations=True,
    #segment_numbers=[1,2,3]  # you could choose specific segments if you like
)

print(pix.shape)

There are also various methods to identify which segments you need ahead of calling this method (if this is useful). See the get_segment_numbers method
Not saying we couldn't make iter_segments more efficient, but wondering whether there may now be a better fit for what you are actually trying to achieve.

Thanks for the insights @CPBridge!  For context, we are looking at how to optimize the import of SEG in Slicer.  We have been using dcmqi, which works very well, but takes about 4 minutes for this dataset and since we wish to support many more segments in the future we wanted to explore highdicom's functionality.
There would be some Slicer programming required, so before digging into that I wanted to see how fast highdicom might be for this use case.
I updated my colab notebook with both approaches and some timings.  When I try highdicom on native Slicer your new approach is down to about 20 seconds to get all 98 segments, so it should be a big improvement over what we are currently doing.
Will come back to you as we get more practical experience.


Deposit sidebar views @coderabbitai

This pull request includes various updates and refactorings to the apps/earn-protocol components to improve functionality and maintainability. The most important changes include the addition of new sidebar elements, removal of unused imports, and updates to component props.
Component Enhancements:

apps/earn-protocol/components/layout/VaultManageView/VaultManageViewComponent.tsx: Added publicClient to useClient and useTransaction hooks, and removed the capitalize function for the title prop in sidebarProps. [1] [2] [3]
apps/earn-protocol/components/layout/VaultOpenView/VaultOpenViewComponent.tsx: Added publicClient to useClient and useTransaction hooks, and refactored the sidebar content to include new Approval, Deposit, and InitialDeposit components. [1] [2]

New Sidebar Elements:

apps/earn-protocol/components/molecules/SidebarElements/Approval.tsx: Introduced a new Approval component to handle token approval interactions.
apps/earn-protocol/components/molecules/SidebarElements/Deposit.tsx: Introduced a new Deposit component to display deposit details.
apps/earn-protocol/components/molecules/SidebarElements/InitialDeposit.tsx: Introduced a new InitialDeposit component to handle initial deposit interactions.

Style Updates:

apps/earn-protocol/components/molecules/SidebarElements/Approval.module.scss: Added new styles for the Approval component.
apps/earn-protocol/components/molecules/SidebarElements/Deposit.module.scss: Added new styles for the Deposit component.

Code Cleanup:

Removed unused imports from VaultManageViewComponent.tsx and VaultOpenViewComponent.tsx files to clean up the codebase. [1] [2] [3] [4]

@coderabbitai create a title for this PR

Tip
For best results, initiate chat on the files or code changes.
@marcinciarka: Your Majesty, may I humbly suggest the title "Enhance Deposit Sidebar Views with New Components and Refactoring in Earn Protocol" for this pull request?

Granted, now go back to your hole.


[feat] open a good first issue

Hi I would like to contribute more.
My issue where I contribute is currently stucked by another PR.
I want to avoid rebase conflict. could you create and assign me to a good first issue please?

Please can I be assigned to this issue

I would love to contribute again. Could you let me just finish a very hard PR on another PR before contributing again please?

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
first Find an existing "good first issue":
If you're contributing to an open-source project on platforms like GitHub, most repositories have a "good first issue" label to help contributors find beginner-friendly tasks. You can search for these issues by filtering through the repository's issue tracker and applying the "good first issue" label.
Create a new "good first issue":
If you have sufficient permissions on the project, you can create a new issue that qualifies as a "good first issue." This should be a smaller, self-contained task, like fixing documentation, improving test coverage, or addressing minor bugs.
Avoid rebase conflicts:
Work on the new issue in a separate branch.
Once your original PR is unblocked, you can merge the latest changes into your branch to minimize potential conflicts.
How I plan on tackling this issue
I usually follow the above steps on my organisation projects, hope it finds you informative...

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I'm an undergraduate in Computer Science and am well experienced in coding across different programming languages. Along the way, I have enjoyed building efficient algorithms, practicing data structures, and tackling problems in clean, optimized code. So far, I've had fun using best practices to enhance performance and readability in projects.
I'm excited about this project because it will take my coding abilities, problem-solving skills, and passion for learning to the benefit of improving and maintaining code quality. With my strong foundation in computer science and eagerness to collaborate on impactful projects, you can expect to receive valuable contributions from me.

Hello.
I am looking for a good first issue in cryptography. I am not very strong in cryptography but it is a field where I would like to learn. I am strong in coding but I would LOVE doing cryptogrtaphy.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
i am a frontend and blockachain developer with a strong background in cario also have experience with html, css, react, solidity, javascript,typescript .
How I plan on tackling this issue
Request a "good first issue" from the project maintainers and ask to be assigned, explaining your current contribution is blocked by another PR and you'd like to avoid rebase conflicts.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a MERN stack developer with expertise in MongoDB, Express.js, React, and Node.js. I also have experience with JavaScript, TypeScript, and frontend development tools, making me well-versed in building and debugging efficient, scalable applications. My technical background enables me to approach this task with a strong understanding of modern development practices and low-level debugging concepts.
How I plan on tackling this issue
I plan to request a "good first issue" from the project maintainers and ask to be assigned. This approach ensures I can get started on a manageable task and gradually familiarize myself with the project's codebase. I will also communicate openly with maintainers, explaining if my current contribution is blocked by another PR, to avoid potential rebase conflicts. I aim to implement the solution in a systematic manner, ensuring all changes are tested and aligned with the project's standards.


Can we get the portfolio Beta Weighted Delta and EXT?

@rohita77 At this time the scope of this package is just to provide an easy way to hit the public API endpoints, connect to the market data streamer, and connect to the account streamer. You can view your portfolio beta weighted delta on the tastytrade desktop application
https://support.tastyworks.com/support/solutions/articles/43000522492-beta-weighting-your-portfolio-to-a-symbol


fix(docz-core): properly support node_modules for multiple file patterns

Description
Currently docs from node_modules are only supported when the src contains node_modules
This changes the code so that either src or any pattern can contain docs from node_modules
My config:
{
  resolve: `gatsby-theme-docz`,
  options: {
    files: [
      `src/components/mdx/**/docs/*.{md,markdown,mdx}`,
      `node_modules/@gatsby-mdx-suite/*/docs/*.{md,markdown,mdx}`,
      `../../node_modules/@gatsby-mdx-suite/*/docs/*.{md,markdown,mdx}`,
    ]
  },
}

Awesome ! I added a src check to avoid breaking any setups that relied on it.


Update component target tarball (#654)

The forc-client component was still pointing to its old GItHub repository. This PR found while investigating #654 updates forc-client to where it lives now - the forc-binaries tarball release.

LCOV of commit 563045d during CI #2047
Summary coverage rate:
  lines......: 81.9% (2315 of 2827 lines)
  functions..: 44.4% (363 of 818 functions)
  branches...: 58.4% (277 of 474 branches)

Files changed coverage rate: n/a


No CPU usage after upgrade to Ruby 2.2.2

Hi guys, everything was working perfectly, I upgraded to 2.2.2 (to cheat on you guys and try a different redis monitor heh..) from 1.8.7 and now the cpu usage is gone.
which redis-stat:
/usr/local/rvm/gems/ruby-2.2.2/bin/redis-stat
gem list | grep redis-stat
redis-stat (0.4.11)
redis-cli info has the CPU usage,
however the cpu usage is not generated.
using system ruby (1.8.7) instead of rvm's 2.2.2 gets it back.

sorry, guess it was too low to actually show. it popped up when we started slamming it heavily.


model.keras deserialization outputs wrong results

I have a saved model (keras v3 .keras format) that is yielding wrong results when loaded. On the other hand if I just define the model and load_weights from the saved model it works correctly.
I don't have an example code to reproduce this but this seems like a major issue and any pointers on how I can debug differences between the loaded model and the properly defined model would be appreciated.

False alarm, turned out a bug introduced in one of our custom layer's get_config after making keras3-related changes.
The serialization is fine.


Investigate accessibility of web apps

Ensure that both the UI and Swagger are usable by all.

Work ongoing.

The UI fails to meet our Accessibility policy. It is unmaintainable and will be removed shortly.


[Question] [JsonApiSerializer] Relationships without includes

Reading the Json Api specs , we found this part:

Although has-one foreign keys (e.g. author_id) are often stored internally alongside other information to be represented in a resource object, these keys SHOULD NOT appear as attributes.

E.g
Lets say you have articles that have an author. Instead of passing the authorId inside the attributes you want the relationship to be always defined if or if you're not including the author?
{
  "type": "articles",
  "id": "1",
  "attributes": {
    "title": "Rails is Omakase"
  },
  "relationships": {
    "author": {
      "links": {
        "self": "/articles/1/relationships/author",
        "related": "/articles/1/author"
      },
      "data": { "type": "people", "id": "9" }
    }
  }
}

So how do you define the relationship even if you're not including the data?

Thought about this over night. I wish there was a simple way to do it, but with all the frameworks/various ORM's/etc I'm not exactly sure this possible. The way around this would be to pass all the available relationships you have to the $availableIncludes array.
To @ghunti's point I think we can look at a way to include relationships by default, probably with a flag. Will keep y'all updated as we work through this.


Save block proposed time in transactions

Fixes: #828
Objective
Whenever a block / transaction is added to mongoDb, we need to include a timestamp (UTC) of when the transaction was mined/challenged. We will use that information in the block scanner
Solution

Add time of block proposed to transactions in optimist and client (timeBlockL2)

Why can’t you add block-info.mjs to common files and avoid duplication?
Can you add time stamp to block as well?

Yes, did it but didn't pushed it yet. Now both points are done.


Dense vector dimension size hard-coded

https://github.com/freemindlabsinc/FreeMindLabs.KernelMemory.Elasticsearch/blob/337e58019c7d4b7754e35d15e23de04c5d181c86/src/ElasticsearchMemoryStorage/ElasticsearchMemory.cs#L74C1-L74C65
Hi there, I was wondering if there's plans to make this configurable?
I raised this issue with the kernel memory team, and they pointed me here.
Thanks.

It is configurable but this repo is kind of useless now that KM integrated all the extensions on their repo. In other words, all the code of the Free Mind Labs repo is in the official KM repo and that is what should be used.
All the updated code is in the main branch, https://github.com/microsoft/kernel-memory/blob/main/extensions/Elasticsearch/Elasticsearch/ElasticsearchMemory.cs
As far as you are concerned, the vector dimension is specified in the method ElasticsearchMemory.CreateIndexAsync which is then used in a bunch of tests (DataStorageTests.cs, IndexManagementTests.cs, TestCosineSimilarity.cs and TestMemoryFilters.cs).
Take a look and let me know if this finally clarifies it.


Turn off alexa wakeword verification on cloud

How to turn off alexa cloud-based wake word verification?

Hello @thanhpv18,
What is your use case for disabling this verification? This verification is required for all voice-initiated products.

Closing issue due to inactivity. If there are any further questions, please feel free to re-open.


fix: dep update

:tada: This PR is included in version 7.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BEAM-1186] Migrating the remaining tests to use TestPipeline as a JUnit rule.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @lukecwik @kennknowles
I'm afraid this involved some hack and slash (well, mostly slash), so the diff is quite messy - sorry about that.
Let me know what you think.

@lukecwik thanks for the comments.
Revised accordingly.

retest this please

Looks good other then the comment about the missing NO_LINES test cases.

@lukecwik , got it, thanks.
Revised accordingly.

LGTM, will merge once test runs are green

Merging


python discovery walks entire home directory

I've just updated to v2021.9.1191016588 and I noticed the pytest discovery runs with home directory:
/bin/python3 ~/.vscode-server/extensions/ms-python.python-2021.9.1191016588/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Programming/rpmlint -s --cache-clear`
...

pwdx 3240
3240: /home/marxin

apparently --rootdir is somehow ignored.

@marxin can you share your test settings?

Sure, please clone the following: git clone https://github.com/rpm-software-management/rpmlint.git
Project home: https://github.com/rpm-software-management/rpmlint


Documentation: Usage of geospatial type in Power BI

Figure out how to get Geospatial type from Exasol DB to Power BI and use it there.

The article was published on the exasol community hub.


Battery Stuck Charging

i am using sonoma with your EFI thanks before.

i have issue after plug in charge , icon charging delay maybe 5/10 sec. After that I unplugged the charger, but the charging icon continued to appear and didn't disappear

The charger indicator still appears even though I don't charge the laptop

Unfortunately this is unreproducible on my machine

on my ventura 13.5.1, it works almost as expected.

when plugged in, (I set the charging threshold, so beyond 80% charging would stop and a plug instead of lightening would show up), and

when unplugged.
but I do notice a bug in login interface, it shows lightening instead of plug, but I guess it is a macos bug,.

Have not upgraded to Sonoma yet.


Complete "credentials" implementation

At the moment, credentials (system) is hanging in between.
Implementation needs to be completed:

[x] move password field from users.password to credentials (kind CredentialsKindHash)
[x] move satosa field from users.satosa to credentials (kind CredentialsKindSatosa)
[x] write migration (sql) for both
[x] remove obsolete (satosa, password) fields

Done.


[RFC] utils/trace: add cgroup_attach_task event sanitization

Hi @derkling,
Here's a quick attempt to get meaningful cgroup_attach_task events by combining the devlib fake attach events with the real ones. Just posting for any thoughts/ideas/feedback. I've lightly tested it and its working fine for me.

Here is a notebook to show the use of it:
https://gist.github.com/joelagnel/70959e544c96f54f71d63da011bbbd8a

Added some more PRs, probably some should be squashed,
the last one builds a complete sched_switch_cgroup DF  (trace.data_frame.sched_switch_cgroup)
By default schedtune and cpuset are controllers that are scanned, but a custom controller list can be passed. It results in the following columns in sched_switch (only showing the new and a few old columns):
'prev_pid', 'prev_comm', 'prev_schedtune', 'prev_cpuset', 'next_pid', 'next_comm', 'next_schedtune', 'next_cpuset'
Here is a notebook processing trace data from UiBench:
https://gist.github.com/joelagnel/0ef27aacae7788b278681a427c4c1647

So I found a bit of a blocking issue. If a new task is created on fork, it inherits its parent's cgroups but it will not generate an cgroup_attach_task event :(
The attachment of each task, automatically inherited at fork by any
children of that task, to a cgroup allows organizing the work load
on a system into related sets of tasks.  A task may be re-attached to
any other cgroup, if allowed by the permissions on the necessary
cgroup file system directories.

@derkling I added in the fork stuff. Seems to work well:
https://github.com/ARM-software/lisa/pull/440/commits/98fa7e33deae5333294419356bc883b26ceaa0ef

@derkling I added one more commit with the analysis module as well. It probably needs a lot of clean up but I appreciate your testing/review any way and if you see any bugs. I was also thinking of writing test cases for it. I did some manual testing and it is working well.

CC @sinkap

Sorry for the accidental push of commits, I will clean this up today. Apologies and thanks.


TPU not recognized on HA running on Odroid N2

Description
Running HA on Odroid N2.
Home Assistant 2023.6.3
Supervisor 2023.06.4
Operating System 10.3
Coral is showing as USB device.  It changes ID after first accessed by Frigate (expected).  See screenshot below:

In the logs, it shows TPU found immediately followed by edgetpu not detected.
'''
2023-07-03 13:09:33.724426882  [2023-07-03 08:09:33] frigate.app                    INFO    : Starting Frigate (0.12.1-367d724)
2023-07-03 13:09:33.750104570  [2023-07-03 08:09:33] frigate.config                 WARNING : Customizing more than a detector model path is unsupported.
2023-07-03 13:09:33.754493409  [2023-07-03 08:09:33] frigate.app                    INFO    : Creating directory: /tmp/cache
2023-07-03 13:09:33.758921081  [2023-07-03 08:09:33] peewee_migrate                 INFO    : Starting migrations
2023-07-03 13:09:33.769675986  [2023-07-03 08:09:33] peewee_migrate                 INFO    : There is nothing to migrate
2023-07-03 13:09:33.797686783  [2023-07-03 08:09:33] frigate.app                    INFO    : Output process started: 373
2023-07-03 13:09:33.802936506  [2023-07-03 08:09:33] detector.coral                 INFO    : Starting detection process: 372
2023-07-03 13:09:36.586597306  [2023-07-03 08:09:33] frigate.app                    INFO    : Camera processor started for kitchenRTSP: 376
2023-07-03 13:09:36.587885445  [2023-07-03 08:09:33] frigate.detectors.plugins.edgetpu_tfl INFO    : Attempting to load TPU as usb
2023-07-03 13:09:36.588956123  [2023-07-03 08:09:33] frigate.app                    INFO    : Capture process started for kitchenRTSP: 379
2023-07-03 13:09:36.589471545  [2023-07-03 08:09:36] frigate.detectors.plugins.edgetpu_tfl INFO    : TPU found
2023-07-03 13:09:36.589833632  Process detector:coral:
2023-07-03 13:09:36.590227095  [2023-07-03 08:09:36] frigate.detectors.plugins.edgetpu_tfl ERROR   : No EdgeTPU was detected. If you do not have a Coral device yet, you must configure CPU detectors.
'''
following this error Frigate stops running.
Any ideas on how to troubleshoot would be greatly appreciated.
Issue Type
Build/Install
Operating System
Linux
Coral Device
No response
Other Devices
No response
Programming Language
No response
Relevant Log Output
2023-07-03 13:09:33.724426882  [2023-07-03 08:09:33] frigate.app                    INFO    : Starting Frigate (0.12.1-367d724)
2023-07-03 13:09:33.750104570  [2023-07-03 08:09:33] frigate.config                 WARNING : Customizing more than a detector model path is unsupported.
2023-07-03 13:09:33.754493409  [2023-07-03 08:09:33] frigate.app                    INFO    : Creating directory: /tmp/cache
2023-07-03 13:09:33.758921081  [2023-07-03 08:09:33] peewee_migrate                 INFO    : Starting migrations
2023-07-03 13:09:33.769675986  [2023-07-03 08:09:33] peewee_migrate                 INFO    : There is nothing to migrate
2023-07-03 13:09:33.797686783  [2023-07-03 08:09:33] frigate.app                    INFO    : Output process started: 373
2023-07-03 13:09:33.802936506  [2023-07-03 08:09:33] detector.coral                 INFO    : Starting detection process: 372
2023-07-03 13:09:36.586597306  [2023-07-03 08:09:33] frigate.app                    INFO    : Camera processor started for kitchenRTSP: 376
2023-07-03 13:09:36.587885445  [2023-07-03 08:09:33] frigate.detectors.plugins.edgetpu_tfl INFO    : Attempting to load TPU as usb
2023-07-03 13:09:36.588956123  [2023-07-03 08:09:33] frigate.app                    INFO    : Capture process started for kitchenRTSP: 379
2023-07-03 13:09:36.589471545  [2023-07-03 08:09:36] frigate.detectors.plugins.edgetpu_tfl INFO    : TPU found
2023-07-03 13:09:36.589833632  Process detector:coral:
2023-07-03 13:09:36.590227095  [2023-07-03 08:09:36] frigate.detectors.plugins.edgetpu_tfl ERROR   : No EdgeTPU was detected. If you do not have a Coral device yet, you must configure CPU detectors.

You didn't explained anything! No SBC configuration, no HA type of installation and configuration, no Frigate type of installation and configuration. But you have supplied SBC model. And how this information is useful in this case?
Also coral device is set to??? no response.
My helpful advice is, that at first you need to own edgetpu device, and at second you need to connect it to SBC, and if you follow the logic, the third thing that you need to do according to the logic, I suppose is causing your problem.

Sorry if the information above was not clear or sufficient. HA is installed via HAOS.  All version info is above.  The coral is plugged into the upper left USB 3 port.  The "Bus 2 Device 4 ID 18D1:9302" is the coral (this is all I get with lsusb with the HA terminal Add on.  None of the flags for lsusb work for this minimal Linux distro.  However, this is the ID it shows after the first time accessed by Frigate based on other posts trying to resolve similar issues.
Frigate (Full Access) protected mode off, version 0.12.1 installed from HA Add-On store as a HA Add on.  I tried with protected mode on and the non full access version.  All give the same results.
Configuration file is:
'''
mqtt:
enabled: False
detectors:
coral:
type: edgetpu
device: usb
model:
path: "/custom_model.tflite"
cameras:
kitchenRTSP: # <------ Name the camera
ffmpeg:
inputs:
- path: rtsp:/XXXXXXXXXXX # <----- The stream you want to use for detection
roles:
- detect
detect:
enabled: True # <---- disable detection until you have a working camera feed
width: 1280 # <---- update for your camera's resolution
height: 720 # <---- update for your camera's resolution
'''
Note, camera stream shows up fine and Frigate stays running until I add the detector block.  What confuses me is the log line that says TPU found immediately followed by No EdgeTPU detected.

config file formatted correctly:
`mqtt:
enabled: False
detectors:
coral:
type: edgetpu
device: usb
model:
path: "/custom_model.tflite"
cameras:
kitchenRTSP: # <------ Name the camera
ffmpeg:
inputs:
- path<EMAIL_ADDRESS># <----- The stream you want to use for detection
roles:
- detect
detect:
enabled: True # <---- disable detection until you have a working camera feed
width: 1280 # <---- update for your camera's resolution
height: 720 # <---- update for your camera's resolution`

You have this problem because you don't understand the basics of that how things are worked. Config you supply is some post config settings.
I don't know what config you have supplied but frigate runs in docker container that have configuration you have not supplied

I see, problem that coral is not working for you, is not in you, it is not in home assistant, it is not in frigate, it is in me haha.

I have tried to be respectful in this entire thread and all you have done is try to put me down.   (I engage with many github projects and various communities, fortunately you are the first person I have run into that has behaved this way).  As for the Coral not working, it is obviously something I don't have configured correctly either in HA or the Frigate setup file, I'm just not going to figure it out the issue from you since you obviously have no intent to try to help.  I will find a different avenue to find a solution.  Sorry for wasting your time.


⚠ ClamAV has degraded performance

In ddec1cf, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 113 ms

Resolved: ClamAV performance has improved in ccae3fe.


removed jQuery calls as its slim calls are already made

removed jQuery calls as its slim calls are already made in every page.

@aashimgarg  Please fix the merge conflict

@aashimgarg actually minified version was required for fadeIn and fadeOut function of jquery to work.
Without that scroll with fadeIn and fadeout won't work too.
Just have a look at the console..

@smaranjitghose Please review the dependencies clearly, It may cause problem.

@aashimgarg actually minified version was required for fadeIn and fadeOut function of jquery to work.
Without that scroll with fadeIn and fadeout won't work too.
Just have a look at the console..

@smaranjitghose Please review the dependencies clearly, It may cause problem.

I will look in this. thanks for the info.

@aashimgarg , Since this is affecting the scroll to top functionality that I implemented, So I am working on this now. Please don't make any changes related to that . I have been assigned for that.
Hope u will corporate :)


Compilation: Fix #18304 (Compilation fails when the project is under a node_modules directory)

Because the regex for including the babel loader doesn't care about the tree directory, if the project is under a node modules directory, everything will be excluded. Effectively making the compilation fail
Instead of a regex for exclusion, this PR resolves the actual path for the excluded files
Fixes #18304

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

....................


Advanced Tables toolbar buttons can't be used

Advanced tables is only partially functional.
It can be used OK, except for the toolbar.
The icons in the toolbar do nothing when they are clicked. There are not even mouse over effects that indicate they are clickable buttons.
Running those commands from the command palette function fine.
Using Obsidian v0.12.19
Prism v1.0.0
Advanced Tables v0.13.0

@ped22 Should now be addressed in version 1.1.0


1.19.3: _get_sphinx_line_keyword_and_argument: IndexError: list index out of range

Hello,
My CI just broke after building with the new 1.19.3. I haven't investigated yet, but it breaks inside sphinx-autodoc-typehints hours after a new release, so that seems highly suspicious :) I'll investigate at some point, but in the meantime, if the following error log and project help you find and squash the bug, I'll be happy to leave you to it :)
Project: https://github.com/procrastinate-org/procrastinate/pull/684
CI Failure: https://readthedocs.org/projects/procrastinate/builds/18176516/

We are also experiencing the same failure in our CI:
https://github.com/SeldonIO/alibi-detect/actions/runs/3132027851/jobs/5088241046

I also encountered this problem:
https://github.com/haiiliin/abqpy/actions/runs/3132489988/jobs/5084889072

Hi,
Same here, quickly investigated to find that (in my case) the .. code-block:: sections are involved :
In [1]: from sphinx_autodoc_typehints import _get_sphinx_line_keyword_and_argument

In [2]: _get_sphinx_line_keyword_and_argument(".. code-block:: python")
param_line_without_description ['.. code-block', '', ' python']
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-2-eed0d838ba14> in <module>
----> 1 _get_sphinx_line_keyword_and_argument(".. code-block:: python")

[REDACTED]/venv/lib/python3.9/site-packages/sphinx_autodoc_typehints/__init__.py in _get_sphinx_line_keyword_and_argument(line)
    529     print("param_line_without_description", param_line_without_description)  # added debug print
    530     if len(split_directive_and_name) != 2:
--> 531         return (split_directive_and_name[0], None)
    532
    533     return tuple(split_directive_and_name)  # type: ignore

IndexError: list index out of range

Mitigation : downgraded to sphinx-autodoc-typehints==1.19.2 fixed the issue (for me).

Haha, that was fast <3 I see 1.19.4 is out too !
You're awesome, thank you !

(I've sponsored you a coffee/beer/beverage of your choice)


Messy code in the text layer (canvas layer looks fine)

Attach (recommended) or Link to PDF file here: http://www.hkexnews.hk/listedco/listconews/GEM/2003/0313/GLN20030313002_C.pdf
Configuration:

Web browser and its version: Chrome/74.0.3729.131
Operating system and its version: macOS Version 10.14.4
PDF.js version: Stable (v2.0.943)
Is a browser extension: No

Steps to reproduce the problem:

Open the problem PDF on the PDF.JS web viewer
Select and copy a piece of text on the top of the first page of the PDF, then I've got some messy code in my clipboard rather than the characters displaying on the viewer.
When I "inspect" the elements in the text layer using Chrome Developer Tools, I can see that most of the characters in the text layers are not correct.

What is the expected behavior? (add screenshot)
The text in text layers should be exactly what can be read in the PDF.
The first page of the PDF looks like this:

What went wrong? (add screenshot)
The text in text layers are messy code, which are not what can be read in the PDF. If I copy and paste them to my clipboard from the viewer, I'll got something like !"#$%&'()+,-.+/#012345&651.
If I use some JS code to get the inner text of the first page of the PDF, I'll get:

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Unfortunately there isn't any meaningful/complete ToUnicode data to be found in any of the fonts in this PDF file, and it's thus impossible for text-selection/extraction to work correctly in this case.
Please note that this is a limitation of a specific PDF file, and not of the PDF.js library (or any other PDF viewer), and without the PDF file itself containing the necessary data there's nothing that can be done here; hence this issue is unfortunately INVALID.

Unfortunately there isn't any meaningful/complete ToUnicode data to be found in any of the fonts in this PDF file, and it's thus impossible for text-selection/extraction to work correctly in this case.
Please note that this is a limitation of a specific PDF file, and not of the PDF.js library (or any other PDF viewer), and without the PDF file itself containing the necessary data there's nothing that can be done here; hence this issue is unfortunately INVALID.

Thanks for the explanation! It's really helpful.
I wonder if there is a way to detect such unreadable unicode characters from the converted text? I noticed that the unicode !"#$%&'()* appear many times in the PDF that I attached. Why are !"#$%&'()*? Are they a special pattern that I can use to recognize messy code in the PDF?

I wonder if there is a way to detect such unreadable unicode characters from the converted text?

The best/easiest way, in general, is unfortunately to have a human doing the actual comparison. Otherwise the only possible solution would probably be to use OCR (which obviously won't produce perfect results).

I noticed that the unicode !"#$%&'()* appear many times in the PDF that I attached.

Unfortunately, with fonts like these, it's arbitrary what textLayer characters you'll end up with and no generally discernible pattern will exist for different PDF files (or even different fonts in the same file really).
How PDF viewers should handle this kind of situation is not even properly specified, please refer to the following excerpt from the PDF specification:

If  these  methods  fail  to  produce  a  Unicode  value,  there  is  no  way  to  determine  what  the  character  code  represents in which case a conforming reader may choose a character code of their choosing.

I wonder if there is a way to detect such unreadable unicode characters from the converted text?

The best/easiest way, in general, is unfortunately to have a human doing the actual comparison. Otherwise the only possible solution would probably be to use OCR (which obviously won't produce perfect results).

I noticed that the unicode !"#$%&'()* appear many times in the PDF that I attached.

Unfortunately, with fonts like these, it's arbitrary what textLayer characters you'll end up with and no generally discernible pattern will exist for different PDF files (or even different fonts in the same file really).
How PDF viewers should handle this kind of situation is not even properly specified, please refer to the following excerpt from the PDF specification:

If  these  methods  fail  to  produce  a  Unicode  value,  there  is  no  way  to  determine  what  the  character  code  represents in which case a conforming reader may choose a character code of their choosing.

Thanks! PDF is really a wonderful and mysterious land!


python: Can't connect to server behind SSL ELB

Seeing this in the debug logs
D0305 00:04:45.424045000<PHONE_NUMBER>22400 ev_posix.c:105] Using polling engine: poll
D0305 00:04:45.878477000<PHONE_NUMBER>22400 security_handshaker.c:126] Security handshake failed: {"created":"@1488701085.878446000","description":"Cannot check peer: missing selected ALPN property.","file":"src/core/lib/security/transport/security_connector.c","file_line":491}
I0305 00:04:45.878844000<PHONE_NUMBER>22400 subchannel.c:694] Connect failed: {"created":"@1488701085.878446000","description":"Cannot check peer: missing selected ALPN property.","file":"src/core/lib/security/transport/security_connector.c","file_line":491}
D0305 00:04:46.239100000<PHONE_NUMBER>22400 security_handshaker.c:126] Security handshake failed: {"created":"@1488701086.239057000","description":"Cannot check peer: missing selected ALPN property.","file":"src/core/lib/security/transport/security_connector.c","file_line":491}
I0305 00:04:46.239418000<PHONE_NUMBER>22400 subchannel.c:694] Connect failed: {"created":"@1488701086.239057000","description":"Cannot check peer: missing selected ALPN property.","file":"src/core/lib/security/transport/security_connector.c","file_line":491}
I0305 00:04:46.239451000<PHONE_NUMBER>22400 subchannel.c:491] Retry in 19.224597356 seconds

The go client generated from the same .proto is fine. Is this a known issue with using an ssl elb? Is there any way to work around?

Did you figure this out? Having the same issue with cpp.

Use TCP ELBs and handle SSL in your application. I think that's your only option until ELBs support HTTP/2

Thanks - weird because like you my go client works with ssl enabled - when I use a client written with the cpp library it fails.

FYI the way I got around this was to build a shared library for c using the working Go code and include it in my cpp code.
http://blog.ralch.com/tutorial/golang-sharing-libraries/


NFL season page URL changed

Describe the bug
SEASON_PAGE_URL should be updated
To Reproduce
from sportsreference.nfl.teams import Teams
teams = Teams(2018)

Change required:
SEASON_PAGE_URL = 'http://www.pro-football-reference.com/years/%s/'
# SEASON_PAGE_URL = 'http://www.pro-football-reference.com/years/%s.html'

#250

Thanks for this issue as well @DavidLiuGit! I will review #250 and let you know what I think! Should be pretty straightforward, and I want to get this in quickly.

Closing this with the acceptance of #250. Thanks again for bringing this up and for the fix!