Extraneous string "ID" added to surname with PDF and authors with ORCID links

Hi,
We have noticed that when processing a PDF with ORCID URL links labeled "ID", the "ID" string is concatenated to the author string.
Steps to reproduce:
For instance with this PDF:
https://journals.plos.org/plosbiology/article/file?id=10.1371/journal.pbio.3000097&type=printable
Article page:
https://journals.plos.org/plosbiology/article?id=10.1371/journal.pbio.3000097
Using process header document + Consolidate header=true (but we have the same result with Consolidate header=false)
We get :
<persName xmlns="http://www.tei-c.org/ns/1.0">
    <forename type="first">Thomas</forename>
    <surname>Burgoyne Id</surname>
</persName>

Whereas the expected string is  <surname>Burgoyne</surname>
We are using application version="0.5.6". It also happened with previous versions.
Thank you for the new release 0.5.6 and all the work.

Thank you very much Raphaël for reporting the issue!
In addition to skipping the ID string, let's also extract the orcid of the author as it is done with the DOI link/annotations in the reference (it will be added as an <idno> element under <persName>).

Thank you very much! it will be really useful, we look forward to upgrade our submission process with it ;-)

Following #630, this is working very well now with PLOS, thanks @Aazhar !
The consolidation however would need some improvement, because it tends to discard the identified/attached ORCID, while we could probably keep it whatever its source is (original PDF or CrossRef record). For instance for this PDF https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0244981
or
https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0245039
-> everything is fine without consolidation, but merging with crossref record discard the good ORCID extractions.
Still to be done too: for the consolidation, get the orcid from the crossref/glutton record under utilities/crossref/WorkDeserializer.java when available.

Following #630, this is working very well now with PLOS, thanks @Aazhar !
The consolidation however would need some improvement, because it tends to discard the identified/attached ORCID, while we could probably keep it whatever its source is (original PDF or CrossRef record). For instance for this PDF https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0244981
or
https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0245039
-> everything is fine without consolidation, but merging with crossref record discard the good ORCID extractions.
Still to be done too: for the consolidation, get the orcid from the crossref/glutton record under utilities/crossref/WorkDeserializer.java when available.

The remaining problem with consolidation and orcid via crossref record should be covered by 480b080e6735106f2a4e5e3629eb63ffe273cdc9 - consolidation for sure could still be improved, but I think it's a more general redesign that would be needed then.

The remaining problem with consolidation and orcid via crossref record should be covered by 480b080e6735106f2a4e5e3629eb63ffe273cdc9 - consolidation for sure could still be improved, but I think it's a more general redesign that would be needed then.


Add FormData::Multipart#boundary reader method

I wanted to measure performance of the multipart-parser gem, to compare it to the performance of Rack's multipart parser. I'm using http-form_data to generate test multipart data, and multipart-parser needs me to pass in the boundary. I can call MultipartParser::Reader.extract_boundary_value to extract the boundary from FormData::Multipart#content_type before parsing, but since the FormData::Multipart object already knows the boundary, I thought I could eliminate this extra step.

@ixti Updated 😀

Awesome! Thanks!


https://github.com/ElectronNET/Electron.NET/issues/72

@GregorBiswanger Works on my machine (the XmlDoc cleanup is not important... but resulted in build warning...)
Important part: Setting the current working dir. Any problems with that? Seems "ok"

@GregorBiswanger I will merge this, ok?


Should this render the panel automatically or should I be explicitly adding some HTML within Twig to render it or call a function?

Ignore that, I passed the wrong value to something :(

No problem.
the main thing you have found a solution.
Do not need touch templates, Tracy append after </body></html>.
For show Tracy you only need enable Debugger::enable();
this minimum you must see Tracy panel.
all other is Tracy addons.

@1f7 Debugger::enable(); does not works for me. Already follow the guide, but panel didn't render

Already fix, by 'addContentLengthHeader' => false,

@arma7x, thanks i add it in readme


Playground: importsNotUsedAsValues tsconfig option is missing

Yeah, I'd 👍🏻 a PR adding it to the 'interop constraints' section - if you'd like to set it manually you can use the twoslash comments system, which you can read about in the playground help section

happy to give it a go, would you be able to show me where these options are in the repo? i'm having trouble finding it

happy to give it a go, would you be able to show me where these options are in the repo? i'm having trouble finding it

I encountered the same problem, the help documentation is as follows:
https://www.typescriptlang.org/play?target=99&jsx=4&module=6&ts=4.8.4#handbook-11 ,
Or you can directly look at the example below:
https://www.typescriptlang.org/play?strictPropertyInitialization=false&importsNotUsedAsValues=1#code/PTAEAEEsFsAcHsBOAXAzgOXsgqqgpgCYCCqAagIYA2ArnqgFyyJ16IBueAsAFAwIqgA3qACyoAL6gAZonjRQAcgB0wZHWQKePPAA9+yUAGN4AO1QGi9UaAC8oAIw9jZ+JTxLK8AOYAKQwEogA


Pre-built binary depends on nix paths

What happens
Trying to execute the  austral-linux that comes with the 0.2.0 results in the following error message:
$ ./austral-linux
bash: /path/to/austral-linux: No such file or directory

This error happens because the austral-linux executable is dynamically linked, and since it was built in a nix-shell, the linker is tied to the Nix path /nix/store/yaz7pyf0ah88g2v505l38n0f3wg2vzdj-glibc-2.37-8/lib/ld-linux-x86-64.so.2.  The user will not have this path unless they previously used the nix-shell to build the compiler.
What I expected to happen
The compiler to run and display the usage message on stdout.
Steps to reproduce

Either start on a system or container that doesn't have nix installed, or issue sudo mv /nix/store/yaz7pyf0ah88g2v505l38n0f3wg2vzdj-glibc-2.37-8/lib/ld-linux-x86-64.so.2 /some/other/path (exercise extreme caution doing the latter, it may render your system inoperable if you're on nixos or if you're a heavy nix user!)
Download the austral-linux executable from the 0.2.0 release.
Issue chmod +x /path/to/austral-linux.
Try to execute the compiler by issuing /path/to/austral-linux.

Environment
The bug was communicated to me by Ivan-Velickovic from UNSW's Trustworthy Systems group. I managed to reproduce it in the following environment:
$ uname -a
Linux 6.2.0-32-generic #32~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC x86_64 GNU/Linux

Impact
Mostly affects new users, and users who just want to try Austral. Running the nix-shell is a viable workaround. There is probably a way to configure nix to avoid the issue in future releases, but this is outside of my domain of expertise.

I spent most of the afternoon going crazy trying to debug this in GitHub Actions: I couldn't figure out why it was saying "No such file or directory" when ls -l was clearly telling me that austral existed in the working directory and had executable permissions. It would be great if future release builds publish binaries that don't depend on Nix.


Support multiple ManyToMany fields that are using the same junction table

This PR is fixing issue #17 by introducing new option "customDeleteCondition".
This approach looks better and easier for me then moving removal code out of the main foreach.

@voskobovich, please review.

I believe that we need a more realistic example.
For example the relationship between order and products. The binding table stores the sum and the number of positions.
Or you can use an example from your practice.

@voskobovich I've updated usage example from mine project. Hope that will help.


Cannot open distributed_portals_with_elixir.livemd on Elixir v1.12.2

Environment

Elixir & Erlang/OTP versions (elixir --version):

Erlang/OTP 24 [erts-12.0.3] [source] [64-bit] [smp:16:16] [ds:16:16:10] [async-threads:1] [jit]

Elixir 1.12.2 (compiled with Erlang/OTP 24)

Operating system: macOS Big Sur
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): livebook server
Livebook version (use git show if running with mix):
Browsers that reproduce this bug (the more the merrier): Chrome, Safari, Firefox
Include what is logged in the browser console: Nothing is logged
Include what is logged to the server console:

10:58:12.272 [error] GenServer #PID<0.1438.0> terminating
** (FunctionClauseError) no function clause matching in Livebook.LiveMarkdown.Import.build_notebook/3
    (livebook 0.1.0) lib/livebook/live_markdown/import.ex:171: Livebook.LiveMarkdown.Import.build_notebook([{:metadata, "{\"livebook_object\":\"cell_input\",\"name\":\"Other node\",\"type\":\"text\",\"value\":\"\"}"}, {:cell, :markdown, [{"p", [], ["Now paste the result of the other node name and its cookie in the inputs below:"], %{}}, {"pre", [], [{"code", [{"class", "elixir"}], ["IO.puts node()\nIO.puts Node.get_cookie()"], %{}}], %{}}, {"p", [], ["In order to connect across notebooks, open up ", {"a", [{"href", "/explore/notebooks/new"}], ["a new empty notebook"], %{}}, "\nin a separate tab, copy and paste the code below to this new notebook,\nand execute it:"], %{}}, {"h3", [], ["Notebook connections"], %{}}, {"p", [], ["Now we have everything we need to connect across notebooks."], %{}}]}, {:cell, :elixir, "Node.get_cookie()"}, {:cell, :markdown, [{"p", [], ["Now there is one last piece of the puzzle: in order for nodes to connect to each\nother, they need to have the same cookie. The cookie serves as a very simple\nauthentication mechanism. We can read the cookie of our current notebook runtime\nlike this:"], %{}}, {"p", [], ["Much better! We see one node, which is the Livebook server itself."], %{}}]}, {:cell, :elixir, "Node.list(:hidden)"}, {:cell, :markdown, [{"p", [], ["The reason why we get an empty list is because, by default, Erlang Distribution\nis a fully mesh. This means all nodes can see all nodes in the network. However,\nbecause we want the notebook runtimes to be isolated from each other, we start\neach runtime as a ", {"em", [], ["hidden"], %{}}, " node. We can ask Elixir to give us all hidden nodes\ninstead:"], %{}}, {"p", [], ["If you execute the code above, you get... an empty list!?"], %{}}]}, {:cell, :elixir, "Node.list()"}, {:cell, :markdown, [{"p", [], ["We can also get a list of all nodes our runtime is connected to by calling\n", {"code", [{"class", "inline"}], ["Node.list()"], %{}}, ", let's give it a try:"], %{}}, {"p", [], ["By executing the code above, we can see node names are atoms. By default,\nLivebook only connects to nodes running on the same machine, but you can\nalso configure it to connect to runtimes across machines."], %{}}]}, {:cell, :elixir, "node()"}, {:cell, :markdown, [{"p", [], ["When Livebook executes the code in a notebook, it starts a separate Elixir\nruntime to do so. Since Livebook itself is also written in Elixir, it uses\nthe Erlang Distribution to communicate with this Elixir runtime. We can\nget the name of the Elixir node our notebook is running on like this:"], %{}}, {"h3", [], ["Distribution 101"], %{}}, {"blockquote", [], [{"p", [], ["The feature we are going to implement will allow us to share data across\ntwo separate notebooks using the Erlang Distribution. This section is a great\nexperiment to understand how things work behind the scenes but ", {"strong", [], ["sharing data\nacross notebooks as done here is a bad idea in practice"], %{}}, ". If this topic\ninterests you, we recommend picking up one of the many available learning\nresources available for Elixir, which will provide a more solid ground to\nleverage the Erlang VM and its distributed features."], %{}}], %{}}, {"p", [], ["With our portals working, we are ready to give distributed transfers a try.\nHowever, before we start, there is one big disclaimer:"], %{}}]}, {:section_name, ["Distributed transfers"]}, {:cell, :markdown, [{"p", [], ["There is just one topic left..."], %{}}]}, {:cell, :elixir, "Portal.close(portal)"}, {:cell, :markdown, [{"p", [], ["Feel free to reevaluate the cell above a couple times. Once you are done,\nrun the cell below to clean it all up and close the portal:"], %{}}]}, {:cell, :elixir, "Portal.push_right(portal)"}, {:cell, :markdown, [{"p", [], ["Sweet! Look how Livebook automatically picked up the new representation.\nNow feel free to call ", {"code", [{"class", "inline"}], ["push_right"], %{}}, " and see what happens:"], %{}}]}, {:cell, :elixir, "Portal.shoot(:red)\nPortal.shoot(:purple)\nportal = Portal.transfer(:red, :purple, [1, 2, 3])"}, {:cell, :markdown, [{"p", [], ["That's all we need! Let's start a new transfer and see how it goes:"], %{}}, {"p", [], ["Then we call ", {"code", [{"class", "inline"}], ["inspect"], %{}}, " multiple times, to get a text representation of both\n", {"code", [{"class", "inline"}], ["left"], %{}}, " and ", {"code", [{"class", "inline"}], ["right"], %{}}, " doors, as well as to get a representation of the data\ninside the doors. Finally, we return a string containing the portal presentation\nproperly aligned."], %{}}, {"p", [], ["In the snippet above, we have implemented the ", {"code", [{"class", "inline"}], ["Inspect"], %{}}, " protocol for the\n", {"code", [{"class", "inline"}], ["Portal"], %{}}, " struct. The protocol expects one function named ", {"code", [{"class", "inline"}], ["inspect"], %{}}, " to be\nimplemented. The function expects two arguments, the first is the ", {"code", [{"class", "inline"}], ["Portal"], %{}}, "\nstruct itself and the second is a set of options, which we don't care about\nfor now."], %{}}]}, {:cell, :elixir, "defimpl Inspect, for: Portal do\n  def inspect(%Portal{left: left, right: right}, _) do\n    left_door  = inspect(left)\n    right_door = inspect(right)\n\n    left_data  = inspect(Enum.reverse(Portal.Door.get(left)))\n    right_data = inspect(Portal.Door.get(right))\n\n    max = max(String.length(left_door), String.length(left_data))\n\n    \"\"\"\n    #Portal<\n      \#{String.pad_leading(left_door, max)} <=> \#{right_door}\n      \#{String.pad_leading(left_data, max)} <=> \#{right_data}\n    >\\\n    \"\"\"\n  end\nend"}, {:cell, :markdown, [{"p", [], ["For example, every time something is printed in Livebook, or in Elixir's\nterminal, Elixir uses the ", {"code", [{"class", "inline"}], ["Inspect"], %{}}, " protocol. Since protocols can be extended\nat any time, by any data type, it means we can implement it for ", {"code", [{"class", "inline"}], ["Portal"], %{}}, " too.\nWe do so by calling ", {"code", [{"class", "inline"}], ["defimpl/2"], %{}}, ", passing the protocol name and the data\nstructure we want to implement the protocol for. Let's do it:"], %{}}, {"p", [], ["Yes, we can! Elixir provides protocols, which allows behaviour to be extended\nand implemented for any data type, like our ", {"code", [{"class", "inline"}], ["Portal"], %{}}, " struct, at any time."], %{}}, {"p", [], ["We already know that Elixir data structures data can be printed by Livebook.\nAfter all, when we type ", {"code", [{"class", "inline"}], ["1 + 2"], %{}}, ", we get ", {"code", [{"class", "inline"}], ["3"], %{}}, " back. However, can we customize\nhow our own data structures are printed?"], %{}}]}, {:section_name, ["Inspecting portals with Protocols"]}, {:cell, :markdown, [{"p", [], ["We have made some good progress in our implementation, so now let's work\na bit on the presentation. Currently, the Portal is printed as a struct:\n", {"code", [{"class", "inline"}], ["%Portal{left: :orange, right: :blue}"], %{}}, ". It would be nice if we actually\nhad a printed representation of the portal transfer, allowing us to see\nthe portal processes as we push data."], %{}}]}, {:cell, :elixir, "Portal.close(portal)"}, {:cell, :markdown, [{"p", [], ["Our portal transfer seems to work as expected! Now let's clean up and\nclose the transfer:"], %{}}, {"p", [], ["Since the list is reversed, we can see that we pushed the number ", {"code", [{"class", "inline"}], ["3"], %{}}, "\nto the right, which is exactly what we expected. If you reevaluate the\ncell above, you will see data moving to the right, as our portal doors\nare stateful."], %{}}]}, {:cell, :elixir, "Portal.push_right(portal)\nPortal.Door.get(:blue)"}, {:cell, :markdown, [{"p", [], ["Note the list is reversed - and we knew that! - as we always add items\non the top. But we will use that to our advantage soon. Let's start pushing\ndata to the right:"], (truncated)

Current behavior
This repository is cloned to /Users/Shared/repositories/personal/elixir/livebook/

Install Livebook using the command mix escript.install hex livebook.
Update asdf using the command asdf reshim elixir.
Start Livebook using the command livebook server.
Open Livebook at http://localhost:8080/.
Navigate to lib/livebook/notebook/explore/.
Click distributed_portals_with_elixir.livemd to select it.
Click the Open button.
The loading progress bar starts and then completes but nothing opens.
The server stacktrace above is shown in the log when clicking over to the terminal window.

Expected behavior
The notebook should open to view contents.
Notes
I want to immediately point out that this was fixed by updating Elixir to 1.12.3-otp-24 via asdf.
This is what I see when opening Livebook for the first time on v1.12.2:

This is what I see on v1.12.3:

I see the line You will need Elixir v1.12 or later. in the readme but I seem to get the "preferred experience" on 1.12.3. I cloned this repo looking to load the portal notebook when it was less work to move to a later Elixir revision.
I don't know what would've saved me from going down this path. A banner saying something like "For the best experience, run this on v1.12.3 or later" may have helped. Understanding that I was missing an Explore section through screenshots could've also been useful.
I hesitated to open this when there is a relatively easy fix but someone else may run into this by accessing this or other notebooks in the future.

Feel free to switch to Elixir v1.12.2, update hex and install livebook. It should be the latest version and work fine :)

That could explain it, @jonatanklosko. My version of hex on v1.12.2 was installed a bit ago so I believe mix escript.install hex livebook would've used that version, not the one that fixes dependency resolution.
I was able to verify that works by downgrading to v1.12.2, running mix local.hex --force, and seeing the explore section 🙌. It seemed odd that a language version change would've made a difference but my brain completely skipped over the fact that a newer hex was installed after switching.
Thanks, @jonatanklosko and @josevalim.


Install instructions incorrect?

Hi, I'd like to install following these instructions from the README but am not sure how to get the binary built:
CGO_ENABLED=0 go get -u github.com/liamg/traitor/cmd/traitor

This creates a go/pkg folder with a bunch of subfolders, but I have no idea where the built binary is.
Any help greatly appreciated! Using go 1.16.8.

@taoeffect What version of Go do you have installed?

@prologic I mentioned it in the issue, see the very last sentence

@prologic I mentioned it in the issue, see the very last sentence

My apologies I didn't see that due to my vision impairment 😅 Soo according to the Go 1.16 Release Notes you shoudl now use go install <package>@<version> to install packages that contain a package main such as what cmd/traitor provides.
Example:
$ go install github.com/liamg/traitor/cmd/traitor@latest

Tested on my iMac.

When I run that I get this:
$ go install github.com/liamg/traitor/cmd/traitor@latest
go/pkg/mod/github.com/liamg/traitor@v0.0.9/pkg/exploits/all.go:5:2: //go:build comment without // +build comment

Sorry I can't explain what's going on there :/

What architecture are you building on? If building is an issue, you can probably take one of the prebuilt binaries from the latest release.

@liamg fedora linux.

If building is an issue, you can probably take one of the prebuilt binaries from the latest release.

Thanks, when I checked last they weren't there for 0.0.14, but now they are.

*I edited the issue title to "Build instructions incorrect?", can this issue be reopened since they still aren't working?


Create a page to display fair principles youtube video for the Open Science practice

So rather than directing the user to Youtube directly, the user will be directed to a new page in the website that has the video embedded and a summary about fair principles, whiles maintaining the same header, footer and other colours and design from the original website.

Hello, I am an outreachy applicant, please i will like to be assigned this issue
thank you

@BatoolMM i would like to be assigned to this task. thank you.


cemu-ti: unstable-2022-06-29 -> 2.0

Description of changes
https://github.com/CE-Programming/CEmu/releases/tag/v2.0
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

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

1 package built:

cemu-ti


Reference set name via set rather than set_name

It seems that at least with version 3.9.0 of aerospike that the set name is referenced by they key "set" rather than set_name. For me this fixes the unit tests on aerospark.

Thanks. We made the modification locally, since there is a conflict. Appreciate pointing this out!


EPIC - ReturnValueEvaluator

ReturnValueEvaluator in every place an expression that produces and outcome is required.

@tiagobento This cannot be done in one go. (well it is feasible but the reviewer will go crazy)
At present in every place the expression that produces some sort of value is done by different pieces of code.

This is not good for sustainbility and it is pretty error prone as any change in this part requires to change the logic everywhere. This lead to a situation where some of this expression are producing different outcomes depending where you execute.
This does not allow for writing how expressions work as you need to go one by one in every place how they work or how variables work.
we cannot consolidate FEEL instead of MVEL as expression language (which is far better)

Examples of this expression could be assignments, node input/output, conditions (nodes or boundary events), split/join node completion conditions. activation conditions, loops (multiinstandce) and so forth.


Search product

Can you provide supplementary information to this issue??

search products by the product name

I can try this out

This is going to be an extension of the get all products right??

This issue has already been fixed!


Dependabot alerts security upgrade alert

This is for upgrading these upstream dependencies, dependabot is reporting these potential vulnerabilities:
Known security vulnerabilities detected

Dependency jinja2 	Version < 2.11.3 	Upgrade to ~> 2.11.3
Defined in dev.txt
Vulnerabilities 	CVE-2020-28493 Moderate severity
Dependency Flask-Cors 	Version < 3.0.9 	Upgrade to ~> 3.0.9
Defined in dev.txt
Vulnerabilities 	CVE-2020-25032 High severity
Dependency ipython 	Version >= 6.0.0 < 7.16.3 	Upgrade to ~> 7.16.3
Defined in dev.txt
Vulnerabilities 	CVE-2022-21699 High severity
Dependency pyjwt 	Version >= 1.5.0 < 2.4.0 	Upgrade to ~> 2.4.0
Defined in dev.txt

Landed on #33

FYI @jab1982 the security issues that you reported got fixed.


fetch top donated project for each month of the year

For comms campaign around the holidays we should fetch each project that received the highest USD value of donations for each month of the year.
https://www.notion.so/giveth/EOY-12-Days-of-GIVing-Campaign-14a3ab28d48c8012a989cd964e87c72b?d=14a3ab28d48c80edbf13001cf699cc33
So querying for 11 months  the start date to the end date of a given month, which project received the highest USD value of donations.
We should get 11 responses, 1 each for

January
February
March
April
May
June
July
August
September
October
November

cc: @oyealmond

@divine-comedian should this be a endpoint or can be a one time query thing I get from the db?

One time query

@divine-comedian
WITH "MonthlyDonations" AS (
  SELECT
    p.id AS "project_id",
    p.title AS "project_title",
    DATE_TRUNC('month', d."createdAt") AS month,
    COALESCE(SUM(d."valueUsd"), 0) AS "total_donations",
    RANK() OVER (
      PARTITION BY DATE_TRUNC('month', d."createdAt")
      ORDER BY COALESCE(SUM(d."valueUsd"), 0) DESC
    ) AS rank
  FROM
    public.donation as d
  INNER JOIN
    project p ON p.id = d."projectId"
  WHERE
    d."createdAt" >= '2024-01-01' and d."createdAt" <= '2024-11-30'
  GROUP BY
    p.id, p.title, DATE_TRUNC('month', d."createdAt")
)
SELECT
  "project_id",
  "project_title",
  month,
  "total_donations"
FROM
  "MonthlyDonations"
WHERE
  rank = 1
ORDER BY
  month ASC;

Query Used.

Sent you the results by DM

I think a few adjustments need to be made... that the comms team didn't consider
We should only consider givbacks eligible projects AND omit the giveth matching pool

ok new query
WITH "MonthlyDonations" AS (
  SELECT
    p.id AS "project_id",
    p.title AS "project_title",
    DATE_TRUNC('month', d."createdAt") AS month,
    COALESCE(SUM(d."valueUsd"), 0) AS "total_donations",
    RANK() OVER (
      PARTITION BY DATE_TRUNC('month', d."createdAt")
      ORDER BY COALESCE(SUM(d."valueUsd"), 0) DESC
    ) AS rank
  FROM
    public.donation as d
  INNER JOIN
    project p ON p.id = d."projectId" and p.verified = true and p."isGivbackEligible" = true and p.title != 'Giveth Matching Pool'
  WHERE
    d."createdAt" >= '2024-01-01' and d."createdAt" <= '2024-11-30'
  GROUP BY
    p.id, p.title, DATE_TRUNC('month', d."createdAt")
)
SELECT
  "project_id",
  "project_title",
  month,
  "total_donations"
FROM
  "MonthlyDonations"
WHERE
  rank = 1
ORDER BY
  month ASC;

Excellent! Closing this one


Fix file max size upload

Hello!
When uploading a file indicating in the config a specific size I have seen that the File Class of the rules, the max function is not static so it cannot be called in the way it is.
exception :  "Error" file :  "/var/www/html/packages/furrutia-c/laravel-file-explorer/src/Requests/UploadItemsRequest.php" line :  80 message :  "Non-static method Illuminate\\Validation\\Rules\\File::max() cannot be called statically"

Hello, you have already corrected it, since you are not going to use the File class in max, I have also changed the type to save you importing this class just for that.


:hammer: remove etl-metadata-export from wizard instructions

Generating YAML metadata with etl-metadata-export ... isn't needed anymore now that wizard generates reasonable YAML file and we don't have to repeat metadata in the garden step.
I removed the optional instruction to run etl-metadata-export and also fixed the function in case anyone uses it to prune fields like origins or licenses that aren't useful anymore.

Wondering if files under steps/data/garden/dummy/2023-09-25 are possibly spurious?

Oh man 🤦 Yes, they are spurious. Merging just the parts that make sense.


Installation Failed

The Laravel framework version I used is 8.60.0 with PHP version 7.3.12. I want to use this package for my live auction system, however, there is a lot of problems installing this package.
This is what I get when I run composer require beyondcode / laravel-websockets

Then I run composer require beyondcode / laravel-websockets --with-all-dependencies, more problems occur.

Can someone provide a working instruction for installing laravel-websockets on laravel v8.x? Or does it really work in the latest laravel version?
Thank you in advance!

Duplicate of / see https://github.com/beyondcode/laravel-websockets/issues/784 for a potential resolution. (The question will be whether the forced downgrade of guzzlehttp/psr7 causes problems elsewhere.)


a-frame demo

Hi, im trying to make a porting in angular and a-frame (framework built on top of three.js) but im not sure if i can just use the THREE lerp with the frame times of TalkingHead for the lip sync animations
code: https://github.com/Roxcsas/aframe-talking-avatar-ia
this is the result: https://streamable.com/flpvvh

Very interesting. Thank you for sharing.
I don't see any problem with using the THREE lerp. It does more or less the same thing as my valueAnimationSeq method. One difference here is that you use a linear animationProgress with lerp, whereas I use a sigmoid function to ease in and out. If you want to experiment with this, you can check out my sigmoidFactory method and do the following:
this.easing = this.sigmoidFactory(5);
...
let animationProgress = Math.max( ...
animationProgress = myComponent.easing(animationProgress);

However, I don't know if this makes much difference and there is no single right way to do this. Mostly, it is just trial-and-error and some fine-tuning, and in the end, it all comes down to what looks most natural lip movement to you. Based on the video clip, your code is already working very well.


Update egress cluster policy docs

/lgtm for text

/lgtm for text

There is no / command robot in this repo, it has been added to give you write permission


NPE when solving ArrayInitializerExpr

I'm getting a NPE while trying to parse an ArrayInitExpr, I think maybe you haven't considered that here? https://github.com/javaparser/javasymbolsolver/blob/master/java-symbol-solver-core/src/main/java/com/github/javaparser/symbolsolver/javaparsermodel/JavaParserFactory.java

There is no bug: the problem was the user expected to solve expressions not inserted in an AST and that can work only in certain cases. Added tests anyway and provided better error messages.


NextJS - Critical dependency: the request of a dependency is an expression

Description
web:dev:  ⚠ ../../node_modules/.pnpm/@google+model-viewer@3.3.0_three<EMAIL_ADDRESS>web:dev: Critical dependency: the request of a dependency is an expression
I have added three package in my package.json and still keeps complaining

How do you solve this issue? I am using Next 13 and it's a warning not an error.

Version

model-viewer: v3.3.0

Browser Affected

[x] Chrome, version: xx.x.xxxx.xx
[ ] Edge
[ ] Firefox
[ ] IE
[ ] Safari

OS

[ ] Android
[ ] iOS
[ ] Linux
[x] MacOS
[ ] Windows

Sounds like a NextJS issue? Was it working before with v3.2?

Even with v3.2, it was giving me the same issue


Throw for empty string baseURL.  (Fixes #29)

@kenchris PTAL.  This adds some test cases.  I will upstream them in crbug.com/1240702.  Thanks.


Problems with Fasterxml 2.10.0 and Springboot 2.2.1

I cannot use Faker with my latest Springboot project (2.2.1.RELEASE)
java.lang.NoSuchMethodError: com.fasterxml.jackson.module.kotlin.KotlinModule.<init>(IZZZILkotlin/jvm/internal/DefaultConstructorMarker;)V

	at io.github.serpro69.kfaker.Mapper.<clinit>(Mapper.kt:12)
	at io.github.serpro69.kfaker.FakerService.readCategory(FakerService.kt:121)
	at io.github.serpro69.kfaker.FakerService.load(FakerService.kt:65)
	at io.github.serpro69.kfaker.FakerService.<init>(FakerService.kt:30)
	at io.github.serpro69.kfaker.Faker.<init>(Faker.kt:13)
	at io.github.serpro69.kfaker.Faker.<init>(Faker.kt:12)

I fixed it by upgrading fasterxml to 2.10.1.

I'm also encountering the same issue but with Jackson 2.11. Exact same stack trace.

What spring version are you using @rpmoore ? Will try to reproduce it.

We're not using spring. I also tried down reving Jackson to 2.10.1 and that didn't resolve the issue. I can play with some of the other dependencies to see what it is.

Doing a little digging I found this issue https://github.com/FasterXML/jackson-module-kotlin/issues/279
Seems like if you update to 2.10.2 that should resolve some of the issues going forward.

Thanks for digging that up @rpmoore Will update all of the dependencies and push the change on the weekend. Hopefully that will help.
I've also tried shading the dependencies, but there's no documentation on using gradle shade plugin with kotlin-dsl syntax, so that's still in progress.

@rpmoore please check if faker version 1.1.2 fixes it. I've bumped jackson dependencies to 2.11.0

I'll give it a try today and let you know.

That seemed to do the trick for me! I can run my tests without issue now.

Great, thanks for confirming @rpmoore
Closing this for now. @ksinkar feel free to reopen if this is not fixed for you.

Yeah, it's working like a champ now. This has been super helpful!


Update code in InputEvents doc

Overview
Fixes two issues in this doc:

Two lines ended in ) instead of ;.
Debug logs were identical between the source detected event and the source lost event, instead of logging the correct state.

Changes

Fixes: #6830

/azp run


Sending message doesn't show list of connections

When trying to message between two connected agents, I don't see the names of the connected agents in the drop down - instead I get some unicode text that I can't select.  See screenshot.

Steps done:
I'm running the toolkit and plugins from the main branches of the repo, with the toolkit on Windows 10.   I'm using the Alice and Bob docker file in the Plugin demo folder.

Started the toolbox and agents
Added Alice and Bob to the Toolbox via the invitations
Confirmed Discover Features
Alice created an invitation for Bob, copied it
Bob pasted in the invitation and processed it.
Connected
Went to messages (either agent) and tried to select the other to send the message

Any advice on what is happening?

Looks like it is happening on all of the Select dropdowns.  I have a screenshot from @dbluhm in a tutorial he's done that shows the same issue, so looks like others are having it as well.

Are you using a recent version of the toolbox? The "Connections [New]" and "Invitations New" was in a commit close to a year old now. The version confusion may have been caused by some out of date instructions in that tutorial (apologies); using Hyperledger main of the toolbox and toolbox plugin repos should be functional today.

Pretty sure I was using main on both, but I'll rerun and report back.

Yup...old code.  Sorry about that.  Thanks for the quick response. Closing.


HDA2 cars: fix torque limits

Our torque limits and steering pressed logic assumes the original Hyundai scaling, but the signal on HDA2 cars is 4x in max value, causing torque to wind down when not touching the wheel.

[x] Update torque rate limits and steering pressed threshold

https://imgur.com/a/9SfUxMq

Done in 221086857aac770f5d22273119e6175eeb06c729

This PR did this: https://github.com/commaai/openpilot/pull/24901


[Issue] Allow email subscriptions

What is the issue about?
Allow readers to subscribe to new content by email.
Code of Conduct

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

waiting for #24


fix: Resolve permission denied issue on OpenShift cluster for NodeJS

Co-authored-by: Harikrishnan Balagopal<EMAIL_ADDRESS>Signed-off-by: Akash Nayak<EMAIL_ADDRESS>

Related https://github.com/konveyor/move2kube-demos/pull/197


Fixing the integration variable

In the definition of the Cumulative Distribution Functions the integration variable was incorrectly defined as $\theta$ instead of $y%.

I'll just fix this with the next manual update and thank you for the correction in the acknowledgements.  Thanks!

Oops, forgot the reference:
https://github.com/stan-dev/stan/issues/2336#issuecomment-331960060


Make dashboard (i.e. Gene, Name) search case insensitive

Dear Florian Bayer,
I would be highly grateful if you could make the search function in the dashboard case INsensitive. My pinky already hurts from pressing down the shift key.
Thanks a lot!
Stephan Eckert

I added a checker box that can enable and disable case-sensitive text searches.
The new feature looks like this :

Some regex queries may only be available in the case-sensitive mode


Feature: NAT use different port

Description
Add port configuration for NAT, can be configured HAProxy(sidecar) on a single machine without port conflicts
nat flag change:
server --nat X.X.X.X  // default port

server --nat X.X.X.X:1479  // custom port

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[ ] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[ ] I have added the relevant labels
[ ] I have updated the official documentation
[ ] I have added sufficient documentation in code

Testing

[x] I have tested this code with the official test suite
[x] I have tested this code manually

Manual tests
root@644365c223d2:/work# make build && mv main dogechain
go build -ldflags="-X 'github.com/dogechain-lab/dogechain/versioning.Version=v1.1.1+a8ae32e8+2022-09-06_07:35:30'" main.go
root@644365c223d2:/work# go test -coverprofile coverage.out -timeout 28m ./...
?   	github.com/dogechain-lab/dogechain	[no test files]
ok  	github.com/dogechain-lab/dogechain/archive	0.039s	coverage: 62.6% of statements
ok  	github.com/dogechain-lab/dogechain/blockchain	0.018s	coverage: 59.8% of statements
?   	github.com/dogechain-lab/dogechain/blockchain/storage	[no test files]
ok  	github.com/dogechain-lab/dogechain/blockchain/storage/leveldb	4.865s	coverage: 27.0% of statements
ok  	github.com/dogechain-lab/dogechain/blockchain/storage/memory	0.002s	coverage: 77.8% of statements
ok  	github.com/dogechain-lab/dogechain/chain	0.066s	coverage: 38.5% of statements
?   	github.com/dogechain-lab/dogechain/command	[no test files]
?   	github.com/dogechain-lab/dogechain/command/backup	[no test files]
?   	github.com/dogechain-lab/dogechain/command/genesis	[no test files]
?   	github.com/dogechain-lab/dogechain/command/helper	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/candidates	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/helper	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/propose	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/snapshot	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/status	[no test files]
?   	github.com/dogechain-lab/dogechain/command/ibft/switch	[no test files]
?   	github.com/dogechain-lab/dogechain/command/license	[no test files]
?   	github.com/dogechain-lab/dogechain/command/loadbot	[no test files]
?   	github.com/dogechain-lab/dogechain/command/loadbot/generator	[no test files]
?   	github.com/dogechain-lab/dogechain/command/monitor	[no test files]
?   	github.com/dogechain-lab/dogechain/command/peers	[no test files]
?   	github.com/dogechain-lab/dogechain/command/peers/add	[no test files]
?   	github.com/dogechain-lab/dogechain/command/peers/list	[no test files]
?   	github.com/dogechain-lab/dogechain/command/peers/status	[no test files]
?   	github.com/dogechain-lab/dogechain/command/root	[no test files]
?   	github.com/dogechain-lab/dogechain/command/secrets	[no test files]
?   	github.com/dogechain-lab/dogechain/command/secrets/generate	[no test files]
?   	github.com/dogechain-lab/dogechain/command/secrets/init	[no test files]
?   	github.com/dogechain-lab/dogechain/command/server	[no test files]
?   	github.com/dogechain-lab/dogechain/command/status	[no test files]
?   	github.com/dogechain-lab/dogechain/command/txpool	[no test files]
?   	github.com/dogechain-lab/dogechain/command/txpool/status	[no test files]
?   	github.com/dogechain-lab/dogechain/command/txpool/subscribe	[no test files]
?   	github.com/dogechain-lab/dogechain/command/version	[no test files]
?   	github.com/dogechain-lab/dogechain/consensus	[no test files]
?   	github.com/dogechain-lab/dogechain/consensus/dev	[no test files]
?   	github.com/dogechain-lab/dogechain/consensus/dummy	[no test files]
ok  	github.com/dogechain-lab/dogechain/consensus/ibft	0.343s	coverage: 59.3% of statements
?   	github.com/dogechain-lab/dogechain/consensus/ibft/proto	[no test files]
?   	github.com/dogechain-lab/dogechain/contracts/abis	[no test files]
?   	github.com/dogechain-lab/dogechain/contracts/bridge	[no test files]
?   	github.com/dogechain-lab/dogechain/contracts/systemcontracts	[no test files]
ok  	github.com/dogechain-lab/dogechain/contracts/validatorset	0.003s	coverage: 82.6% of statements
ok  	github.com/dogechain-lab/dogechain/crypto	0.023s	coverage: 75.1% of statements
ok  	github.com/dogechain-lab/dogechain/e2e	457.102s	coverage: [no statements]
?   	github.com/dogechain-lab/dogechain/e2e/framework	[no test files]
?   	github.com/dogechain-lab/dogechain/graphql	[no test files]
ok  	github.com/dogechain-lab/dogechain/graphql/argtype	0.015s	coverage: 42.0% of statements
?   	github.com/dogechain-lab/dogechain/helper/bridge	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/common	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/daemon	[no test files]
ok  	github.com/dogechain-lab/dogechain/helper/enode	0.014s	coverage: 56.2% of statements
?   	github.com/dogechain-lab/dogechain/helper/hex	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/ipc	[no test files]
ok  	github.com/dogechain-lab/dogechain/helper/keccak	0.001s	coverage: 0.0% of statements [no tests to run]
?   	github.com/dogechain-lab/dogechain/helper/keystore	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/progress	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/tests	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/validatorset	[no test files]
?   	github.com/dogechain-lab/dogechain/helper/vault	[no test files]
ok  	github.com/dogechain-lab/dogechain/jsonrpc	3.025s	coverage: 72.4% of statements
?   	github.com/dogechain-lab/dogechain/licenses	[no test files]
ok  	github.com/dogechain-lab/dogechain/network	89.720s	coverage: 80.1% of statements
?   	github.com/dogechain-lab/dogechain/network/common	[no test files]
ok  	github.com/dogechain-lab/dogechain/network/dial	1.002s	coverage: 91.5% of statements
ok  	github.com/dogechain-lab/dogechain/network/discovery	0.031s	coverage: 41.8% of statements
?   	github.com/dogechain-lab/dogechain/network/event	[no test files]
?   	github.com/dogechain-lab/dogechain/network/grpc	[no test files]
ok  	github.com/dogechain-lab/dogechain/network/identity	0.003s	coverage: 22.7% of statements
?   	github.com/dogechain-lab/dogechain/network/proto	[no test files]
?   	github.com/dogechain-lab/dogechain/network/testing	[no test files]
ok  	github.com/dogechain-lab/dogechain/protocol	26.448s	coverage: 76.5% of statements
?   	github.com/dogechain-lab/dogechain/protocol/proto	[no test files]
ok  	github.com/dogechain-lab/dogechain/secrets	0.002s	coverage: 9.1% of statements
?   	github.com/dogechain-lab/dogechain/secrets/awsssm	[no test files]
?   	github.com/dogechain-lab/dogechain/secrets/hashicorpvault	[no test files]
?   	github.com/dogechain-lab/dogechain/secrets/helper	[no test files]
ok  	github.com/dogechain-lab/dogechain/secrets/local	0.015s	coverage: 84.3% of statements
?   	github.com/dogechain-lab/dogechain/server	[no test files]
?   	github.com/dogechain-lab/dogechain/server/proto	[no test files]
ok  	github.com/dogechain-lab/dogechain/state	0.016s	coverage: 14.2% of statements
ok  	github.com/dogechain-lab/dogechain/state/immutable-trie	0.016s	coverage: 58.2% of statements
?   	github.com/dogechain-lab/dogechain/state/runtime	[no test files]
ok  	github.com/dogechain-lab/dogechain/state/runtime/evm	0.018s	coverage: 34.9% of statements
ok  	github.com/dogechain-lab/dogechain/state/runtime/precompiled	0.173s	coverage: 68.4% of statements
ok  	github.com/dogechain-lab/dogechain/tests	260.916s	coverage: 81.6% of statements
ok  	github.com/dogechain-lab/dogechain/txpool	55.065s	coverage: 78.3% of statements
?   	github.com/dogechain-lab/dogechain/txpool/proto	[no test files]
ok  	github.com/dogechain-lab/dogechain/types	0.002s	coverage: 34.9% of statements
ok  	github.com/dogechain-lab/dogechain/types/buildroot	0.231s	coverage: 67.4% of statements
?   	github.com/dogechain-lab/dogechain/versioning	[no test files]
root@644365c223d2:/work#

Sidecar pattern

Manual tests

The test case codes are official test suite, either you add or already exists.
Manual tests are those network building and tests manually, which sometimes required.
e.g. attack-behavior branch for the fix PR, manual test when upgrading node of breaking change, etc.


[async_client] Add custom retry strategy per request

Motivation
This PR introduces the option of passing in a custom retry strategy on a per-request basis rather than having to use the retry strategy used at time of instantiation. This allows clients to leverage the client's retry mechanism while having the flexibility to use different strategies when needed.
Test Plan
Current tests cover all the code paths introduced by this PR.

/land


Add some syntax sugar

This PR will add an comment with an discription of what spawn do, as
well as an removed console.log in order to make use of string template
in node.
Not sure if this is needed, but why not 💙

This is greatly written by the way. Thumbs up to Christopher, Dan and the rest working on this.

Hi, thanks! I don’t think we’ll take this.
We use spawn pretty extensively in the codebase so it doesn’t make sense to me to document what it does in just one place. One would need to understand spawn anyway to contribute to this project, so I feel like documenting this is a bit out of scope.
Similarly, we would rather avoid taking purely stylistic changes. They clutter the changelog, and this code changes fast anyway. Also, I’m not sure I’m seeing the benefit of proposed over existing form.

No problem, I get it :) You'll have one console line instead of two, which makes a bit more sense, as you are making the code a bit more efficient in that matter. However, the changes to not have a very big perf gain, so I understand.
As with the commenting, I like that the .sh has been fairly commented, so people that want to contribute know how the bash script works. It was a nice move. It makes it easier to understand the code if you see something you've not encountered before, as well as getting knowledge of how it works, before eventually making a PR that's related to source code.
Also, if it's not a too big of a deal,mind answering me on Twitter, so I can ask a few questions about how you're making a temp directory.


Port UWP taskbar behavior when fullscreen to Win32 apps

Proposal: [Port UWP taskbar behavior when fullscreen to Win32 apps]

Summary

If a UWP app is in full screen if I swipe up from the bottom part of the screen with touch or otherwise with a touchpad/mouse I just hover the bottom part of the screen.
In Win32 apps nothing happens. The only way to switch to apps is using multitasking view, which is quite annoying.
Another useful feature of this uwp feature is the ability to retrieve the title bar analogously by swiping down from the upper part of the screen or putting the mouse hover on top.
This could be invoked with the useful shortcut Win+Shift+Enter like in UWP apps so that you can also manually opt it in
Rationale

Having the same behaviour in Win32/UWP apps is ok for the purpose of Project Reunion
Being able to switch among apps or just check the time or info on taskbar would be made definitely easier and with basically no drawback

Scope

Capability
Priority

Important Notes

Open Questions

Will this behavior cause problems to games where the taskbar could be popping-out continuously?
My idea is to make this one the default behavior, even if an option to disable it should be present. For those who continue to argue about games, I just think that they aren't the main usage of WinAppSDK, that I think is instead new generation WinUI 3 apps

The problem is UWP apps use the built-in API to do this which handles this correctly but Win32 apps don't have such API and have their own methods of achieving so it's up to the developers unless Microsoft provides a built-in way to go full screen in a Windows desktop app.

This proposal is incomplete (e.g. scope section is invalid).

This proposal is incomplete (e.g. scope section is invalid).

@MarcAnt01 Optional != Provided w/ invalid values. I understand it's minor but I read it and was briefly confused.

@MarcAnt01 Optional != Provided w/ invalid values. I understand it's minor but I read it and was briefly confused.

Ah yes, removing it might make sense. Done

I think this outright cannot work for exclusive full screen apps (because DWM is not involved here, so start cannot show up over the game), and might cause more pain than benefits for some kinds of games or interfaces.
 Thus the taskbar pops up and breaks DirectFlip, making the game becomes stuttery until the taskbar goes away. Some FPS games don't reset the pointer position often enough to the center, so sometimes you might hit the bottom and the same issue happens (I've had that one when using Mouse Without Borders). It gets even worse if you end up clicking on the taskbar too, because you could trigger long loading processes (e.g. Visual Studio or another game is pinned to the taskbar).
In the cases of interfaces, I think of YouTube where somebody would want to access settings or something and because the mouse went to the bottom of the screen, start pops up and completely blocks out the settings icon, forcing the user to move their mouse back towards the middle, wait for the taskbar to go away and then move their mouse back towards the settings icon more carefully to not hit the bottom of the screen.

I think this outright cannot work for exclusive full screen apps (because DWM is not involved here, so start cannot show up over the game), and might cause more pain than benefits for some kinds of games or interfaces.
 Thus the taskbar pops up and breaks DirectFlip, making the game becomes stuttery until the taskbar goes away.
Some FPS games don't reset the pointer position often enough to the center, so sometimes you might hit the bottom and the same issue happens (I've had that one when using Mouse Without Borders). It gets even worse if you end up clicking on the taskbar too, because you could trigger long loading processes (e.g. Visual Studio or another game is pinned to the taskbar).
In the cases of interfaces, I think of YouTube where somebody would want to access settings or something and because the mouse went to the bottom of the screen, start pops up and completely blocks out the settings icon, forcing the user to move their mouse back towards the middle, wait for the taskbar to go away and then move their mouse back towards the settings icon more carefully to not hit the bottom of the screen.
In all 3 examples, this is very frustrating behavior.

I find it way more annoying not to have a way to go back in certain scenarios. Using touch it's even worse, because sometimes you cannot go back unless there's a "go back button". I agree it's not a perfect behavior, but the one which is less annoying in the overall imho

This must be an optional behavior. UWP is already hobbled by this behavior and IMO not suitable for game development on desktop. And that's because we cannot fully suspend overlay in UWP, and so Windows captures the top and bottom pixel row of the screen for taskbar / titlebar activation. Even with the mini-bar, it is still captured. That is a breaking feature for many games, they could not be run without incident, because this mouse capture blocks normal interaction such as camera pan. And even besides those titles that use the screen edge already, the issue with titlebar and taskbar popping up uninvited is already observed for games such as Minecraft. It just shouldn't happen if the app developer doesn't want it happening.
You could propose an OS feature to enable taskbar activation, or restrict it to swipe-up only, and no mouse responsiveness.
I've just moved back to Win32 after a few years learning to code on UWP, I love UWP but this is one of the few boxes I've crossed for UWP that has caused me to abandon it. Because by now, Microsoft would have given us an option to disable overlay if they were serious about supporting game development on UWP. They are not seemingly. It was a relief to open my Win32 app stub and move the mouse around without any overlay popping up. That is the control I need to make games. Games have particular UI/UX needs that not all app dev's understand. And I bet there are even some apps that require full control of the view space.

If a UWP app is in full screen if I swipe up from the bottom part of the screen with touch or otherwise with a touchpad/mouse I just over the bottom part of the screen.

This sentence does not make any sense.

If a UWP app is in full screen if I swipe up from the bottom part of the screen with touch or otherwise with a touchpad/mouse I just over the bottom part of the screen.

This sentence does not make any sense.

If you want to discuss about it, starting with it "does not make any sense " isn't a good start, since I don't usually like to answer to rude questions

There is no discussion, because i do not understand your problem, because your description does not make any sense. Please write an english sentence. Have a nice day.

There is no discussion, because i do not understand your problem, because your description does not make any sense. Please write an english sentence. Have a nice day.

If you don't understand, you may ask for an explanation like polite people do.   Making your point clear in the OP is something YOU should strive for.
Now you have answered multiple times and still not made your point clear.

OK folks let's calm down. Does the updated description help? It was a typo (over vs. hover).

At least it is an english sentence now. I am sorry but i could not figure this out. I still do not think it is a good description, because i am expected to know what the UWP behaviour is.
My best guess is that he wants that the taksbar shows itself when you hover there?

@Balkoth  If you don't understand something, it doesn't mean that my issue is "useless" and not clear, also have you noticed that 7 people added a thumb up emoji and that I'd had a conversation with @sylveon about possible drawbacks of this behavior?  Try to question yourself, before blaming other people next time

If we wanna go back to the point of the issue, the taskbar using mouse/touchpad appears if you put the cursor for some millisecs in the bottom part of the app. Using touch you can see it swiping up from that area (beginning from out of the screen). This happens only in UWP app as far as I know. Feel free to ask any question if it's still not clear to you

OK, so some maybe this will help:

Title: Allow access to Taskbar when desktop app is fullscreen
Summary:
When Universal Windows Platform (UWP) apps are in full-screen mode, users can access the Taskbar by moving the mouse to the edge of the display where Taskbar resides. Taskbar would then appear. This proposal seeks to extend this behavior to all apps covered by the Windows App SDK.

If you're going to force this on us, then we also need a way to completely suspend mouse events except for delta's while in fullscreen to prevent the titlebar and taskbar from being activated.

If you're going to force this on us, then we also need a way to completely suspend mouse events / mouse position except for delta's while in fullscreen to prevent the titlebar and taskbar from being activated. And just to be clear, this would force us (game developers and some app developers) to use a software mouse in all cases because I don't see any proposal here to resolve the issue of the actual mouse capture problem - there can't be any interaction with the mouse and explorer while a game is running, so no overlay popup triggers. And yes, you are exactly asking for something that many of us cannot permit - it will kill Windows AppSDK for us, just as UWP is unusable.

I think an option to disable this behavior should be present, even if the uwp behavior should be default.

I don't think that forcing such a disruptive behaviour on all applications by default is a good idea. There is a reason why fullscren win32 apps never did this. If an option for this is considered, the default should be that it is disabled.

Yes, it's very disruptive to both the user and application, and the point I was making before. It can't even work in the case of exclusive fullscreen, too, making the taskbar behavior inconsistent.
An option for the apps themselves to disable the behavior fails to consider the thousands of games pre-App SDK that would be negatively affected by this change, and cannot be updated anymore. The UWP behavior should not be default, but rather a per-app opt-in.

I think the UWP behaviour should be the default for new desktop apps using WinUI 3 and others through a new simplified API for full screen but not apps which use their own methods including existing ones. Further more, a new value could be added to the enumeration FullScreenSystemOverlayMode that would suppress task bar and title bar overlay plus give exclusive full screen for the window.

Yes, it's very disruptive to both the user and application, and the point I was making before. It can't even work in the case of exclusive fullscreen, too, making the taskbar behavior inconsistent.
An option for the apps themselves to disable the behavior fails to consider the thousands of games pre-App SDK that would be negatively affected by this change, and cannot be updated anymore. The UWP behavior should not be default, but rather a per-app opt-in.

You see how this behavior is disruptive for games, but you don't seem to see how the Win32 behavior is disruptive for touchpad users or simply people that want to go back without using a keyboard (especially those who are using only a mouse or a touch device).

What are those fullscreen apps that would need sich functionality anyway? There now have been a number of cases in which this feature would break apps, while you have given none where this would be useful.

Most fullscreen apps do provide a way to leave the app with a mouse, as a fullscreen app without a way to exit is called a virus. The only apps I've seen which provide a keyboard-only way to exit it are games which require a keyboard to play to begin with, so it isn't a concern here.
The amount of legitimate fullscreen apps without a way to leave is low, so this feature would negatively affect thousands of legit apps just to make a few apps (which are still unnamed, please give examples) more usable from touch.

The way I see it, it's not simply about being able to exit full screen, but being able to access to the taskbar while in a full screen is useful. It's annoying to have to exit full screen just to do something simple. For example, to quickly check the time or the status of an operation in another app while watching a YouTube video full screen.
Also, I agree that this would break many apps, and should be opt in.

The way I see it, it's not simply about being able to exit full screen, but being able to access to the taskbar while in a full screen is useful. It's annoying to have to exit full screen just to do something simple. For example, to quickly check the time or the status of an operation in another app while watching a YouTube video full screen.
Also, I agree that this would break many apps, and should be opt in.

Exactly, imagine I am watching a YouTube video and want to know the time or just check notifications. In the old Edge I would just swipe up from the bottom, now I have to exit full screen and this sounds like a terrible UX to me.
Most apps aren't games, so it makes no sense to enforce a behavior that makes actually sense only on games.

It makes no sense to enforce such behaviour for your niche scenario. Most apps do not need this, as they do not run fullscreen.

It makes no sense to enforce such behaviour for your niche scenario. Most apps do not need this, as they do not run fullscreen. If an app willingly chooses fullscreen, it willingly chooses that no other os elements are visible or reachable.

I could say about your scenario to be niche and that most apps do need these and an app choosing fullscreen just wants additional space and could want system os elements to be visible 🤷

You would be lying if you would do that 🤷♂

Ok, thanks for providing your objectively superior point of view

Another useful thing would be to have the shortcut full screen behavior that UWP does. Every UWP, if you press Win+Shift+Enter, it goes full screen. I find that super useful.

Another useful thing would be to have the shortcut full screen behavior that UWP does. Every UWP, if you press Win+Shift+Enter, it goes full screen. I find that super useful.

I never use keyboard shortcuts, but this sounds really useful. Adding to the description

Most apps aren't games, so it makes no sense to enforce a behavior that makes actually sense only on games.

But games are a significant portion of apps which make use of fullscreen, so they must be considered.

Most apps aren't games, so it makes no sense to enforce a behavior that makes actually sense only on games.

But games are a significant portion of apps which make use of fullscreen, so they must be considered.

Apps which use fullscreen =! apps targeted for WinAppSDK

You contradict yourself, the title and your description states win32 which means just about every application.

You contradict yourself, the title and your description states win32 which means just about every application.

You cannot port uwp behavior to uwp, because it's already there

You contradict yourself, the title and your description states win32 which means just about every application.

This is a WindowsAppSDK repo, not Feedback Hub or Windows OS issue repo. Any feedbacks here it's for apps using WindowsAppSDK, it's crystal clear.

You realize that in order for this to work, changes have to made at the OS level. Both UWP and WIN32 have their roots in the OS itself.

And your point being? Project Reunion feature requests might require changes in the system itself to be implemented

The original text of the issue never mentionned the App SDK, so I assumed this was an attempt to get the feature to all apps, regardless of whether they are using the App SDK or not. This could have been clearer.
If it's the case that the change is suggested for App SDK only, then I don't mind as long as there's an easy way to completely opt-out of it.

The original text of the issue never mentionned the App SDK, so I assumed this was an attempt to get the feature to all apps, regardless of whether they are using the App SDK or not. This could have been clearer.
If it's the case that the change is suggested for App SDK only, then I don't mind as long as there's an easy way to completely opt-out of it.

Yep, ofc as Vitor pointed this isn't feedback hub, so I didn't think it was necessary to point that out that this ticket is aimed at it

I think AppWindow need to support two behaviors, what I will call TaskBar overlay and TitleBar overlay (that are mandatory  on UWP). This two behaviors can also be dissociated to have a "À la carte" behavior. For the TitleBar it should also be interesting to automatically show an "Exit fullscreen" button in place of the maximize button exactly like UWP fullscreen windows.

"Most apps aren't games, so it makes no sense to enforce a behavior that makes actually sense only on games." ROFL. I've played more game that I've used apps. And most people I know will be in that category.

"Apps which use fullscreen =! apps targeted for WinAppSDK" Is that so? Where does Microsoft say that? Where do they say they have no intention of supporting fullscreen apps and games in Windows API's moving forward?

This is a WindowsAppSDK repo, not Feedback Hub or Windows OS issue repo. Any feedbacks here it's for apps using WindowsAppSDK, it's crystal clear.

You say it's crystal clear, but I agree with Balkoth, WindowsAppSDK is for all apps, not just some apps. Isn't project reunion meant to be a unifying platform. Or are particular parties trying to keep things separated still and perhaps even separate things even more. WIndowsAppSDK is not just for your kind of apps, it's for my kind of apps too, which includes games.

@andrewleader Can we kill this issue? Nothing good to be had here, I'm afraid.

I think the new behavior of tablet optimized taskbar, that makes it always accessible with a swipe from the bottom completely solves this problem at least for touchscreen, I don't don't if it's worth to keep this open for mouse/touchpad


Spark CI is not working - issues with latest run of spark tests

Branch : spark-branch
Context :
Spark tests should run successfully on github
Problem :
Tests on spark-branch are currently failing with https://github.com/pandas-profiling/pandas-profiling/runs/4126235396?check_suite_focus=true

#1133 closes this.

Draft PR does not trigger CI.
There were a lot of other small warnings and errors but it's all fixed by: https://github.com/ydataai/pandas-profiling/pull/1250/commits/166195d891847e49145f56e528eee41c531e9589


Replace merge with assign

https://deprecations.emberjs.com/v3.x/#toc_ember-polyfills-deprecate-merge
This one was missed in #296

Any idea when this should be merge in?


Unable to build kmonad with stack

Using an up to date manjaro I have done those steps:
yay -S docker  stack ghc
sudo systemctl enable --now  docker.service
sudo usermod -aG docker $USER
git clone https://github.com/kmonad/kmonad.git ~/Comp/kmonad

cd ~/Comp/kmonad
stack build

But now I got that message error that I cannot find associated to kmonad elsewhere on the internet:
docker: Error response from daemon: invalid volume specification: '/home/ulys/.stack:~/.stack': invalid mount config for type "bind": invalid mount path: '~/.stack' mount path must be absolute.

Hey there Pietrodito,
First off, sorry for not responding sooner. Illness gets in the way occasionally.
Secondly, I'm afraid I have no idea about docker, I've never used it once in my life. @lierdakil, would you know what is going on? I think you contributed the dockerfile?

the volume is mapped from you local folder to the docker container's folder which does not support relative path
try something like /home/ulys/.stack:/home/root/.stack

Okay, I'm confused. The

done this steps

part mentions docker, but doesn't actually invoke it; the stack build part shouldn't really do anything with docker... I think? If you actually got a kmonad binary built, then it also shouldn't invoke docker. Either some steps aren't mentioned in the OP, something went mysteriously very wrong, or I'm completely misunderstanding the issue.
For context, the Dockerfile is a convenient way of building a static binary, see https://github.com/kmonad/kmonad/blob/master/doc/installation.md#using-docker -- that's its only purpose (well, that and CI), you absolutely neither need nor want docker to build with stack.

Also, side note: you don't actually need to build on Linux, just grab the static binary from actions' artefacts, e.g. https://github.com/kmonad/kmonad/actions/runs/2675437618 (scroll down to "artefacts" section; the list of all builds is available under the Docker Image CI tab in Actions)

I am also unable to install kmonad with stack. I tried installing it in /usr/bin/. I saw some text where it looked like it was doing something but which kmonad is empty

@userrand unfortunately most people aren't psychic, so unless you post exhaustive logs somewhere, it's unlikely anyone could help you :wink:
Side note, you really shouldn't manually put binaries into /usr/bin, at least consider /usr/local/bin, or if you only need to run as a single user, $HOME/.local/bin (which is where stack puts binaries by default).

@userrand unfortunately most people aren't psychic, so unless you post exhaustive logs somewhere, it's unlikely anyone could help you wink
Side note, you really shouldn't manually put binaries into /usr/bin, at least consider /usr/local/bin, or if you only need to run as a single user, $HOME/.local/bin (which is where stack puts binaries by default).

Hi, sorry, I managed to fix the problem and documented the process of the different issues I encountered here : https://github.com/kmonad/kmonad/issues/264#issuecomment-1569398698
Why is it bad to to put binaries in /usr/bin/ ?

why is it bad to put binaries in /usr/bin/ ?

/usr/bin/ is managed by your package manager. In brief, you don't generally want to mix manually and automatically managed things, it has very high potential for future headaches.


Compatibility with Symfony 4.0+ instead of 4.3+

Lowering the requirements for symfony/* libs from  4.3+ to 4.0+.
According to current dependencies definitions in composer-symfony4-min.json tests of such are passing:
https://github.com/propelorm/Propel2/blob/3087361bd27f3598f3ecd70e9fd272a5c2575b88/tests/composer/composer-symfony4-min.json#L20-L26
https://travis-ci.org/github/propelorm/Propel2/jobs/726959500

< 4.3 is kind of EOL, but if it is supported/green I don't see the harm :+1:


fix(utils): Duplicate package metric - count all duplicate instances

Duplicate metric is counting only unique package names that are bundled multiple times. The metric is missing the case when the bundle will include another instance of the same package causing an unexpected increase of the bundle size.

Packages
Duplicate packages

Before
node_modules/package-anode_modules/package-a/node_modules/package-cnode_modules/package-bnode_modules/package-b/node_modules/package-anode_modules/package-b/node_modules/package-cnode_modules/package-c
2

After
node_modules/package-anode_modules/package-a/node_modules/package-cnode_modules/package-bnode_modules/package-b/node_modules/package-a node_modules/package-b/node_modules/package-cnode_modules/package-c
3

closed by #1447


Minimal documentation about how to use the command line tools

TL;DR: This issue is about how to install the bin/ scripts.
How to install now

The hxl2example (simply output CSV from direct from libhxl) and the hxl2tab at this moment are one-file-put-on-your-path scripts.
As long as the dependencies are already installed they are likely to work.

Python 3 is an requirement
The python library "libhxl" (https://github.com/HXLStandard/libhxl-python; https://pypi.org/project/libhxl/) is an requeriment for every tool from this repository
Some additional tools may require other libraries. At bare minimum this will be documented per script

Installation via pip
At this moment, there is no way to install these scripts via pip.
Maybe this will not be implemented at all, in special if the dependencies between different exporters get too complicated to keep it from breaking a system.
How to create new exporters
Even if this set of tools get complicated over time, at least the bin/hxl2example may be kept as minimum viable product to create some exporter that actually could be shared with others.
Also, without refactoring, other scripts like hxl2tab may still this way for long time. This may be useful, even if for testing, because some exporters may have to implement opinionated HXL attributes, and on worst case scenario if some the conversion table is not external (like remote spreadsheet or local HXLated csv) this can work for ad hoc changes.

One way to install is using the Python pypi package HDP Toolchain at https://pypi.org/project/hdp-toolchain/.
Some scripts still have possibility to simply copy and paste the file to some some place on searchable path. Also each script already have extra inline documentation.


Tooltips

We want the ability to incorporate some sort of tooltip for communicating information about the inputs and outputs
https://m2.material.io/components/tooltips

See @artemiystlr's figma design


packer template for building machine image

Moved to here: https://github.com/wjimenez5271/k3os-vagrant


add set(tuple<...>) implementation for insert and update statements

resolves #216

Very nice, thanks!


Fix for btrfs migration with raid (usb drive) does not work

This prevents that the usbdrive is identified as hdd

@holgern Why the change in blitz.datadrive.sh? When there are no partitions why the test for the biggest partition?
Did a clean install without errors. I will test again on my VM.

My 32 GB USB drive was detected as hdd. The changes on blitz.datadrive.sh prevent that the smaller USB which is /dev/sdb1 with only one partition is overwriting hdd, hddDataPartition and sizeDataPartition

I think the root of this error is in blitz.migration.sh not in blitz.datadrive.sh
blitz.datadrive.sh should be called with the correct parameters (different with ext4/btrfs)
Working on a pr.

@holgern Is 'isBTRFS' in line x the old format or the format from the menu? I think it's the old one. I have a different solution in #1706.

+cp into dev


BCH Atomic Swap - Auto conversion address

I've been doing an atomic swap with someone sending the BCH from coinbase. The wallet was displaying to the BCH sender a BCH address of type CashAddr for the lockaddress. The sender which was using coinbase to send the BCH input the cashAddr and on the coinbase confirmations sending page, shown the BCH cashAddr lockaddress in the Legacy format, which is a bit confusing for the sender of BCH, if he's not aware of such address type.

I think the easiest way to display all type addresses.  Adding settings will not make UI clear.

Here are screenshots.
https://discord.com/channels/539232227360243712/570686711655432222/821952185877135361
Testing: Try to do BCH swap and send BCH to locking Legacy address.


"Make Dead Links..." doesn't let me continue..

Challenge Make Dead Links using the Hash Symbol has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.111 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Can anyone point out what's wrong with my code if any???
Thanks.

Hey dude.
Your issue is that you have included the URL still - all that is required is the hash.
So change this:
Click here for cat photos.
to this:
Click here for cat photos.

@AX3M, the code is wrong. It should be :
a href="#" instead of a #href="".
Kindly close this issue.


Need theory of operations/architecture documentation

The documentation currently lacks coverage of architecture and theory of operations.  As a consequence it's difficult to scrutinize chef-vault to determine whether it's adequately secure and whether it's a good fit for one's environment.
Good job on the usage documentation, though!

I've created a THEORY.md document on the issue_109 branch if you'd like to take a look and see if it provides the level of detail you're looking for.
@moserke, if you can also take a gander and make sure that I haven't mis-spoken that would be great.
If this looks good, I'll make it part of v2.5.0, which I hope to ship tomorrow.


Missing trailing comma in lines with "typing-extensions>=4.1"

I tried to install, but it failed, so I realized it's missing a trailing comma in the setup.py in lines 13 and 27. After adding that, the package installed successfully. 🎉
https://github.com/Arthur151/ROMP/blob/a349c6bf4d6229b8fba2e900d38ef210888937d0/simple_romp/setup.py#L13
https://github.com/Arthur151/ROMP/blob/a349c6bf4d6229b8fba2e900d38ef210888937d0/simple_romp/setup.py#L27

Thanks a lot for reporting this.
I have fixed it.
Best,
Yu


[Lerna]: handle parallel with storybook

What does it do?

removes storybook from develop scope
uses workspaces in lerna

Why is it needed?

because deps were being confused
the packages need to be built before storybook and with parallel (which is meant for long running tasks) it would fail – a lot.

@joshuaellis As a note: while this fixes the storybook crashes for me, running yarn develop and yarn storybook at the same time leads to a "component not found" screen in storybook once a change has been made to a component. I'm not sure if this is introduced by this PR or whether this was already introduced when moving storybook out of the DS package?

@joshuaellis As a note: while this fixes the storybook crashes for me, running yarn develop and yarn storybook at the same time leads to a "component not found" screen in storybook once a change has been made to a component. I'm not sure if this is introduced by this PR or whether this was already introduced when moving storybook out of the DS package?

This is at the moment expected behaviour (although not ideal), I need to into solutions in the meanwhile – my greatest piece of advice would be don't reload the webpage and it will correct itself.


Portuguese translation for CONTRIBUTING.md

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of freeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[x] Add new translation (feature adding new translations)
Checklist:
[x] Tested changes locally.
[ ] Addressed currently open issue #18309
Updates #18309

Description
Adds Portuguese translation for the CONTRIBUTING.md file

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
Thanks @JorgeAndd
Happy Contributing!


unused_variables, false positives

https://github.com/larshp/abapGitServer
Variable "CLIENT" not used

looks like a problem in the UPDATE SQL,

looks like a problem in the UPDATE SQL,


Limit bpf recorder ttl cache size

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
In theory we could exceed memory in an unlimited way when using the TTL
caches. We now limit them by size, having enough room and space.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

/lgtm

Codecov Report

Merging #735 (3136d77) into main (e29b91b) will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #735      +/-   ##
==========================================
+ Coverage   53.93%   53.94%   +0.01%
==========================================
  Files          41       41
  Lines        4120     4121       +1
==========================================
+ Hits         2222     2223       +1
  Misses       1831     1831
  Partials       67       67


refactor(devtools-view): Replace usages of TooltipHost with Tooltip

We were using TooltipHost from a legacy version of FluentUI's React component suite. This PR updates those instances to use the newer Tooltip component from their new library.

Change is good but I think almost all of these want to use relationship="description". The "refresh containers list" button seems ok with label, but for all buttons that already have text, description seems better?

Ah, you're probably right. I misread the label description on first read. Will update.


Multiple psycopg2 errors during migration

Performing startup checks...
Running migrations (setting up DB)...
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
Operations to perform:
  Apply all migrations: account, admin, analytics, auth, contenttypes, core, db, sessions, sites, socialaccount
Running migrations:
  Applying analytics.0008_session_is_bounce...    return self.cursor.execute(sql, params)
psycopg2.errors.ObjectInUse: cannot CREATE INDEX "analytics_session" because it has pending trigger events

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/shynet/./manage.py", line 21, in <module>
    main()
  File "/usr/src/shynet/./manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 85, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 243, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 229, in apply_migration
    migration_recorded = True
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 115, in __exit__
    self.execute(sql)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 142, in execute
    cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.OperationalError: cannot CREATE INDEX "analytics_session" because it has pending trigger events

Migrations failed, exiting

System info
root@lhr1:~# docker version
Client: Docker Engine - Community
 Version:           20.10.2
 API version:       1.41
 Go version:        go1.13.15
 Git commit:        2291f61
 Built:             Mon Dec 28 16:17:32 2020
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.2
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       8891c58
  Built:            Mon Dec 28 16:15:09 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.3
  GitCommit:        269548fa27e0089a8b8278fc4fc781d7f65a939b
 runc:
  Version:          1.0.0-rc92
  GitCommit:        ff819c7e9184c13b7c2607fe6c30ae19403a7aff
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0
root@lhr1:~# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 18.04.5 LTS
Release:        18.04
Codename:       bionic
root@lhr1:~#

psql info:
root@lhr1:~# sudo -u shynet psql
could not change directory to "/root": Permission denied
psql (10.15 (Ubuntu 10.15-0ubuntu<IP_ADDRESS>))
Type "help" for help.

shynet=>

I have no damn clue how this happened other than the fact that watchtower causing heavy IO issues.

It's possible that your database was locked at the time of the migration. Version 0.8.0 has many performance improvements, so the migration process will require creating several indexes. While Django's migrations are quite robust, if you're running into issues I recommend temporarily stopping your Shynet instance, performing the upgrade, and then restarting.
Let me know if you still run into issues.

Note that these migrations can take upwards of 20 minutes depending on the size of your database!

Thanks!
Is there a direct command to perform the migration?

Shynet will automatically perform all pending migrations at startup; I’d try just starting a single Shynet instance.

Sorry for bothering again but it doesn't seem to work:
root@lhr1:~# docker run -p "<IP_ADDRESS>:8099:8080" --env-file "/opt/analytics/prod.env" --name "shynet-space-2" "milesmcc/shynet:latest"
Performing startup checks...
Running migrations (setting up DB)...
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
Operations to perform:
  Apply all migrations: account, admin, analytics, auth, contenttypes, core, db, sessions, sites, socialaccount
Running migrations:
  Applying analytics.0008_session_is_bounce...    return self.cursor.execute(sql, params)
psycopg2.errors.ObjectInUse: cannot CREATE INDEX "analytics_session" because it has pending trigger events

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/shynet/./manage.py", line 21, in <module>
    main()
  File "/usr/src/shynet/./manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 330, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 371, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 85, in wrapped
    res = handle_func(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/commands/migrate.py", line 243, in handle
    post_migrate_state = executor.migrate(
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/usr/local/lib/python3.9/site-packages/django/db/migrations/executor.py", line 229, in apply_migration
    migration_recorded = True
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 115, in __exit__
    self.execute(sql)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/base/schema.py", line 142, in execute
    cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/usr/local/lib/python3.9/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/usr/local/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.OperationalError: cannot CREATE INDEX "analytics_session" because it has pending trigger events

Migrations failed, exiting
root@lhr1:~#

Got it working! Had to do some psql magic (aka disabling the triggers on some tables as postgres user)

Ok, good! I'm glad you got it to work. To preempt issues like this in the future, I just pushed an update to dev that splits the schema and update steps into two different migrations, which hopefully would prevent this in the future.


retention: Prevent deletion of partially-archived messages.

Previously, this code:
old_archived_attachments = ArchivedAttachment.objects.annotate(
    has_other_messages=Exists(
        Attachment.objects.filter(id=OuterRef("id"))
        .exclude(messages=None)
        .exclude(scheduled_messages=None)
    )
).filter(messages=None, create_time__lt=delta_weeks_ago, has_other_messages=False)

...protected from removal any ArchivedAttachment objects where there
was an Attachment which had both a message and a scheduled
message, instead of either a message or a scheduled message.
Since files are removed from disk when the ArchivedAttachment rows are
deleted, this meant that if an upload was referenced in two messages,
and one was deleted, the file was permanently deleted when the
ArchivedMessage and ArchivedAttachment were cleaned up, despite being
still referenced in live Messages and Attachments.
Switch from .exclude(messages=None).exclude(scheduled_messages=None)
to .exclude(messages=None, scheduled_messages=None) which "OR"s
those conditions appropriately.
Pull the relevant test into its own file, and expand it significantly
to cover this, and other, corner cases.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

This is great, merged, thanks for the thoughtfully written test suite @alexmv! Also marking this as a backport candidate for inclusion in the 7.x branch.

Backported to 7.x as 97d4f71f0fb5fa9ac3ca13238b2dda8b6353b564 and 9628cc92788a22a4d743c0435051db1f1bbd8743.


#317 - added httpsRedirectCode

I did not yet have time to review it... Did you close it cause there's a problem with this PR or...?

reviewing it myself I found a hardcoded value that needs to be changed, I'll reopen it when I'm done :)

I made a commit before merging this PR and that produced a few conflicts. The commits I made to this PR were only to resolve them. Also, I added an entry to the docs after merging this PR.
Thank you @leostarcevic.


Provide an override for guava

Provide an override for guava, refresh (a few of the JSON files updated)

@mcarlett @cunningt before merging this PR, we should make sure that each component tests (that uses guava) are successful, I'll come up with a list of component to be tested with this version of guava

openshift-maven-plugin
camel-google-bigquery
camel-openapi-java
camel-salesforce
camel-opentelemetry
camel-fhir
camel-google-pubsub
camel-jira
camel-grpc
camel-minio

openshift-maven-plugin
camel-google-bigquery
camel-openapi-java
camel-salesforce
camel-opentelemetry
camel-fhir
camel-google-pubsub
camel-jira
camel-grpc
camel-minio

All tests are successful @cunningt good to merge


Evaluation Measures

I made a draft gdoc with the description of the task we came to in #19, as well as the definition of evaluation measures for tasks 1 (classification) and 2 (retrieval). All this information will be available from the challenge's website http://cosmir.github.io/open-mic/
Each task has measures that ignore the instrument taxonomy and measures that take it into account.
Please take a look and leave comments before Nov 2nd at https://docs.google.com/document/d/1RF9aUCFh6_-YnoZfM6XhANZbGxQcP84B32saJCKyD3E/edit?usp=sharing

Details to finalize:

[ ] Description of data and annotations.
[ ] Description of taxonomy.
[ ] Size of system output for task 2.


development/libraries/libdap: fix hash

Motivation for this change
Fixes hash for https://github.com/NixOS/nixpkgs/issues/80379
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Do we know why the hash changed?

--- a/README
--- b/README
@@ -1,9 +1,11 @@
Please find the libdap4 API documentation here: https://opendap.github.io/libdap4/html/

+[![DOI](https://zenodo.org/badge/DOI/10.5281/zenodo.3641778.svg)](https://doi.org/10.5281/zenodo.3641778)
+
 Updated for version 3.20.5

 Memory leaks. Minor bug fixes. Lots of work on CI.

 Updated for version 3.20.4

 Memory leak fixes and C++11 features

@Xe can you retarget this to master please? It should be fixed there first, and then can be backported.

I sent an email to their mailing list imploring them to never do this again, but it’s a Google Group so I’m not confident my message will even be delivered.

Happy ending from the mailing list:

OK, my apologies. I really appreciate the work done by downstream packagers.

@Ma27 I don't think this should have been merged straight to a release
branch.  It should have gone to master and backported.

Thanks for the heads-up! Did a "forward-port" to master: 09b47478f8f480aded09c3c621a6ca8af224d276


pythonPackages.pyfuse3: 3.1.1 -> 3.2.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] 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.

Result of nixpkgs-review pr 122954 at c92457f2 run on x86_64-linux 1

2 packages built successfully:

python38Packages.pyfuse3
python39Packages.pyfuse3

2 suggestions:

warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/development/python-modules/pyfuse3/default.nix:1:39:
  |
1 | { lib, buildPythonPackage, fetchPypi, fetchpatch, pkg-config, fuse3, trio, pytestCheckHook, pytest-trio, which }:
  |                                       ^

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/development/python-modules/pyfuse3/default.nix:31:5:
   |
31 |     license = licenses.gpl2;
   |     ^


Problems to compile mraa on Edison running node 0.12.7

Hi, I was able to build on Edison (OS 2.1) mraa with node 0.10.x following the steps in the build doc.
Unfortunately when I try the same steps with node 0.12.7 (installed from alexT repo) I got following errors:
root@max01:~/mraa/build# make
[ 21%] Built target mraa
[ 46%] Built target _python-mraa
[ 47%] Building CXX object src/javascript/CMakeFiles/mraajs.dir/mraajsJAVASCRIPT_wrap.cxx.o
/home/root/mraa/build/src/javascript/mraajsJAVASCRIPT_wrap.cxx:778:13: error: 'FunctionCallbackInfo' in namespace 'v8' does not name a template type
 typedef v8::FunctionCallbackInfov8::Value SwigV8Arguments;
...
(lots of errors like before)
...
/home/root/mraa/build/src/javascript/mraajsJAVASCRIPT_wrap.cxx:9740:18: note: in expansion of macro 'SWIGV8_SYMBOL_NEW'
 exports_obj->Set(SWIGV8_SYMBOL_NEW("Uart"), _exports_Uart_obj);
                  ^
src/javascript/CMakeFiles/mraajs.dir/build.make:60: recipe for target 'src/javascript/CMakeFiles/mraajs.dir/mraajsJAVASCRIPT_wrap.cxx.o' failed
make[2]: *** [src/javascript/CMakeFiles/mraajs.dir/mraajsJAVASCRIPT_wrap.cxx.o] Error 1
CMakeFiles/Makefile2:231: recipe for target 'src/javascript/CMakeFiles/mraajs.dir/all' failed
make[1]: *** [src/javascript/CMakeFiles/mraajs.dir/all] Error 2
Makefile:123: recipe for target 'all' failed
make: *** [all] Error 2

Make sure to wipe the build directory so that cmake doesn't reuse cached values for nodejs versioning decisions. This worked on edison with a node 0.12.7 I compiled from source so no reason it shouldn't work here.

+1, I was building it fine as well.

It works, I forgot to upgrade nodejs-dev. Thanks!