Stubbing a Req.Request in a Phoenix.LiveView.assign_async

Disclaimer: I will note that this is an "advanced" use case which may not be solvable, but I wanted to have a discussion first.
Hello @wojtekmach, thank you for your amazing library, I love Req!
This issue is to determine if there's a possible solution for testing a  Req.Request inside of async LiveView assign. Imagine the following live view (btw I can totally appreciate that this might need a change in LiveView but I'll start here if thats ok)
defmodule MyApp.Live do
  use Phoenix.LiveView

  def mount(_, _, _) do
    {:ok, assign_async(socket, :foo, fn -> {:ok, Req.get!("http://something.com").body} end)}
  end

  def render(assigns) do
    ~H"""
    <.async_result :let={foo} assign={@foo}>
      <%= inspect(foo) %>
    </.async_result>
    """
  end
end

The problem here is that in order to test this LiveView and stub out the call to Req.get!, there is a race condition, as the request to the LiveView will have already been made before you get the pid back that you can call Req.Test.allow/3` against.
e.g.
test "assync_assign with a Req.Request", %{conn: conn} do
  {:ok, live, _html} = live(conn, ~p"/my-route")

  # Race condition as `Req.get!` may have already run
  Req.Test.allow(MyApp.Req, self(), live.pid)
  Req.Test.stub(MyApp.Req, fn conn -> Req.Test.json(conn, %{}) end)
end

Of course, this could be solved with some sort of conditional that uses environment config, or a mocking library like Mox or Mimic.
However, I'd like to imagine there could be another way to propagate ownership so that the allow could be called prior to the LiveView creation. This may require changes to LiveView, it might be impossible, or it might not be something you want to support.
Thanks for the consideration and feel free to close if this is something you are not interested in supporting

I suppose this generalizes to GenServers that call Req in init/1 or a handle_continue/2 from init, so the mention of LV is maybe not necessary

Mox allows you to pass a function to allow in case you need to delay evaluation of the allowed pid. Could something like this work?
parent = self()

Req.Test.allow(MyApp.Req, self(), fn ->
  send(parent, {:allow, self()})
  receive(do: (pid -> pid))
end)

{:ok, live, _html} = live(conn, ...)

assert_receive {:allow, pid}
send(pid, live.pid)

This is a little clunky. It means that, if a function is passed, it needs to be run in its own process to prevent deadlocks, but perhaps a pattern like this could work? It could be wrapped in a helper to make it a bit more ergonomic.

Another solution would use telemetry events to hook into the process

Yeah speaking of telemetry events, we have this Broadway guide: https://hexdocs.pm/req/Req.Test.html#module-broadway
I think it is fine when people do that in their own GenServers and Broadway pipelines which I’d considered intermediate level but this is too much for people using LV so we should definitely improve this. Since Ecto sandbox works in LV we should have infrastructure in place to make that happen. Or it does in LV but not in async assigns? In any case, yes, this is definitely a priority and any help is appreciated.

Wait, this seems to work already? https://gist.github.com/wojtekmach/2ee023315dd6ea209d67b6d3ef8cd985
Could you modify the snippet to show the bug?

For me, the problem was that my stub was defined before the render_async but after the live call, like this:
defmodule DemoLiveTest do
  use ExUnit.Case
  use PhoenixPlayground.Test, live: DemoLive

  test "it works" do
    {:ok, view, _html} = live(build_conn(), "/")

    Req.Test.stub(:foo, fn conn ->
      Req.Test.json(conn, %{foo: :bar})
    end)

    assert render_async(view) =~ "bar"
  end
end

Simply moving the stub definition down, as in your snipet @wojtekmach, fixed the issue.


Changed Color Combination and avatar

As per issue #302, I changed the color combination and make it more attractive and also changed the avatar.
In the below images you can see some changes

What was the problem of the new theme?
I have used the updated theme and asked people about it and they said that
new style is better. You can see it here:
ashish-the-coder.github.io/Portfolio
Please let me know what is the problem of new theme?
Regards,
Ashish
On Tue, Mar 30, 2021, 5:50 AM David Miller @.***> wrote:

Closed #303
https://github.com/StartBootstrap/startbootstrap-freelancer/pull/303.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StartBootstrap/startbootstrap-freelancer/pull/303#event-4524479219,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASVKH6RDMTJDQP6YXDVXLLTTGEKLTANCNFSM4ZZGW22A
.

Okay thanks now I understood the main reason for not changing the theme.
And my website is running well, thank you for asking!!
On Tue, Mar 30, 2021, 10:19 PM David Miller @.***>
wrote:

This is a style choice that we leave up to the developer. Style and theme
are subjective to the user, so this isn't an update that we would add into
the base theme. I hope that your site does well though!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StartBootstrap/startbootstrap-freelancer/pull/303#issuecomment-810416871,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASVKH6TXN472SN7NVBYSRQDTGH6IPANCNFSM4ZZGW22A
.


[Feature] Support for new OpenAI multimodality

Is this your first time submitting a feature request?

[X] I have searched the existing issues, and I could not find an existing issue for this feature
[ ] I am requesting a straightforward extension of existing functionality

Describe the feature
Enabled Canopy to also allow uploading for images, or indexing of images, for the RAG operation. This also means that inference API from Canopy should be able to respond with a proper REST format (JSON + base64 enc for bin?) to return said produced images.
Describe alternatives you've considered
Considering creating something like this myself.
Who will this benefit?
Multimodal RAG users who wish to use multimodality within the ChatCompletions API from OpenAI , for example.
Are you interested in contributing this feature?
yes, but I've just discovered canopy 10 minutes ago :)
Anything else?
No response

Hi @sivang !
Sounds great, and also something that we have on our roadmap i hope we will publish soon.
What would be the interface for loading images?
Looking at todays "Document" model:
class Document(BaseModel):
    id: str = Field(description="The document id.")
    text: str = Field(description="The document text.")
    source: str = Field( default="", description="...")
    metadata: Metadata = Field(default_factory=dict, description="...")

What would you like to see? an extension of the current model? a new model?
Thanks!,
Roy from Pinecone

Thanks @sivang sharing this idea!
As @miararoy has mentioned, this is definitely on the roadmap.
We're currently deliberating between two approaches:

Create distinct KnowledgeBase classes, each designed for a specific data type. For instance, DocumentsKnowledgeBase for documents, ImagesKnowledgeBase for images, etc. To add a new type, we'd need to define a new data model and a corresponding KnowledgeBase subclass for managing it.
Make the KnowledgeBase fully robust - you define an arbitrary data schema on init, and the KnowledgeBase handles each of the data fields (texts, images, etc.) based on that schema.

Both approaches have their own pros and cons in terms of user experience and ease of use. We're keen to hear the community's feedback on which API they find more practical and user-friendly.

thank you @miararoy and @igiloh-pinecone for being so responsive!
What I envisaged is a common data structure (so I guess a new model!)  that would allow me a similar mental model as I use now with OpenAI ChatGPT WebUI, such that

I can transparently trigger calls to whatever modality needed for my prompting
All of the produced artifacts are accessed centrally , just like in the WebUI for the ChatGPT interface.

So for example, I could have a Trail or Conversation or whatever we choose to name it object, which would contains the specialized objects for each data items regardless of its modality. In a sense I guess I'm thinking along the lines of https://en.wikipedia.org/wiki/MIME , but obviously I'm expecting the user friendliness of Python objects and collections.
How does that sound? Does it make sense in view of the roadmap and general architecture? (and btw, I'm happy to help contribute the feature)

Thanks @sivang sharing this idea! As @miararoy has mentioned, this is definitely on the roadmap.
We're currently deliberating between two approaches:

Create distinct KnowledgeBase classes, each designed for a specific data type. For instance, DocumentsKnowledgeBase for documents, ImagesKnowledgeBase for images, etc. To add a new type, you'd need to define a new data model and a corresponding KnowledgeBase subclass for managing it.
Make the KnowledgeBase fully robust - you define an arbitrary data schema on init, and the KnowledgeBase handles each of the data fields (texts, images, etc) based on that schema.

Both approaches have their own pros and cons in terms of user experience and ease of use. We're keen to hear the community's feedback on which API they find more practical and user-friendly.

I think the underlying implementation is important, as long as we can create , or use said KnowledgeBases as one when providing context for the LLM. I admit that I haven't dived into the docs that far, but when asking for a retrieval, do I need to specify which KnowledgeBases to use and can I combine a few to allow OpenAI to kick DALLE generation on it's own accord? (as it done through the ChatGPT webUi ?)


chore: runs build-and-publish workflow on schedule

adds option to run it manually against supported branches
ensure workflow jobs will only run against redhat branches
updates hack/storage.sh to use ODF operator
cleanup registry deployment on hack/teardown.sh script

/cherry-pick redhat-3.6


Update VS to 17.10

Test failures must be due to some caching somewhere - built locally and vswhere picks up 17.10 no problem.

Yeah, I just downloaded the build agent from the URL we use and it had the new version. Did the build run before VS released?

Yeah, I just downloaded the build agent from the URL we use and it had the new version. Did the build run before VS released?

No, I checked the test agent locally before I ever submitted the PR.


Broken jasmine typings

Seems that the new 2.5.46 version of @types/jasmine is broken due to this commit https://github.com/DefinitelyTyped/DefinitelyTyped/commit/ba29010d1438767c5db5e73d716447178d85f0f3.
In the meantime for the fix, the version will be fixed to the previous working one.

Here's the related PR: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14875


Copy ga javascript into /vendor during static build

Adds build changes from #258 to static's build.sh file so that npm run static no longer results in a 404 error when it looks for vendor/angulartics.ga.

:+1:

Coverage remained the same at 33.633% when pulling 289d93e14939e994a44f5327519116ed1f47570b on thevoiceofzeke:vendorFix into 4f009036ed477cd64e48ce83c5a6a346bfa27cf7 on UW-Madison-DoIT:master.

:+1: Hmm, interesting. It should have been copied over as part of the grunt process-html run.

I think maybe the build comments were left out? I'm not familiar with process-html, but it looks like you have to have special comments in HTML in order for the grunt task to do anything.

Ah, that could be. Alas, this work.


Add a skills list/rank section

Where there are skills listed and how well I know them.
EX:
Below are tech skills based on knowledge 1 none and 10 meaning expert.
PHP  4/10        Python 8/10       JavaScript 7/10
CSS 8/10         HTML 8/10          CLI 7/10
OOP 7/10         Agile 6/10            GIT 7/10
Flask 7/10        SQL 6/10

That has been added and may be edited to reflect recent skills in the future.


Firefox uses PulseAudio only and doesn't work with just ALSA

Since ALSA is included in Arch Linux by default, the most common deployment scenarios include PulseAudio with ALSA.

This calls into question whether the following are still needed to be installed:

alsa-utils
gnome-alsamixer
pnmixer

https://github.com/phR0ze/cyberlinux/commit/c04a0d1708b6046cfcced39a3d41ec064b4958c4


  (See above for error)
Traceback (most recent call last):
File "/opt/3d-model-convert-to-gltf/server/convert.py", line 33, in
main()
File "/opt/3d-model-convert-to-gltf/server/convert.py", line 20, in main
out_convert_gltf_path = Convert.convert_by_type(sys.argv[1], os.path.abspath(sys.argv[2]), is_bin)
File "/opt/3d-model-convert-to-gltf/server/service/Convert.py", line 219, in convert_by_type
result = model.handler(file_path, is_bin)
TypeError: handler() missing 1 required positional argument: 'need_draco'

v1.5 mirror file is good.v1.6 and latest have this bug

Thanks for your clearly feedback, the error seems like convert.py params pass problem, I'm already fixed but forget to push image before.
I will remind you again after this problem solved.
https://github.com/wangerzi/3d-model-convert-to-gltf/blob/master/server/service/Convert.py#L178


Oauth2config (depends on #559)

Change dependant on https://github.com/meanjs/mean/pull/559
Aim is to move oauth switch out of routing. It should allow to create dynamic oauth setup and allow to re-use some controllers in case you connect to various endpoints, i.e. GitHub and GitHub Enterprise.

@sielay  Let's review this after #559 is merged.


Refactor Authorization

As briefly mentioned in #243

This would be quite a big refractor.

postponed

I just looked at the discussion in #243 and it didn't seem like it was a problem in the past semester—we don't have rogue TAs that go around editing other submissions.
So I'm going to close this.

Yes, I agree.


2.6.0 missing in helm repo rancher-stable

Repo rancher-stable still lists 2.5.9.  Will 2.6.0 be added later to the stable repo?
● λ helm search repo rancher-latest
NAME                    CHART VERSION   APP VERSION     DESCRIPTION
rancher-latest/rancher  2.6.0           v2.6.0          Install Rancher Server to manage Kubernetes clu...
● λ helm search repo rancher-stable
NAME                    CHART VERSION   APP VERSION     DESCRIPTION
rancher-stable/rancher  2.5.9           v2.5.9          Install Rancher Server to manage Kubernetes clu...

2.6.0 will be in the stable repo when its declared stable, which it is not currently. It will probably be v2.6.1.


bloop: 2.0.4 -> 2.0.5

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354264

aarch64-darwin

:white_check_mark: 1 package built:

bloop

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354264

x86_64-darwin

:white_check_mark: 1 package built:

bloop


Investigate devcerts --trust on Linux

This issue is to look into support for the devcerts '--trust' option on Linux.
It may not be possible to make this work on any flavor of Linux, but maybe we can make some improvements to enable (partial) support on some flavors.
CC @omajid @natemcmaster

enable (partial) support on some flavors.

What did you have in mind?

We don't have anything in mind yet. But we want to use this issue to share some options as we explore them.


Bash File names with periods

Bash syntax highlighting struggles with files with version numbers. The line below will highlight the .7 as a number.
sudo dpkg -i vagrant_1.7.2_x86_64.deb

Adding this to bash component seems to work for me, but didn't see a test suite to feel comfortable just submitting pull request.
number: {
    pattern: /(\s|=)-?(0x[\dA-Fa-f]+|\d*\.?\d+([Ee]-?\d+)?)\b/,
    lookbehind: true
}

I think your suggestion would not work for expressions like z=$(($z+3)) (http://tldp.org/LDP/abs/html/arithexp.html).
Maybe we just need to consider the dot . as a word-character and do something like:
number: {
    pattern: /([^\w\.])-?(0x[\dA-Fa-f]+|\d*\.?\d+([Ee]-?\d+)?)\b/,
    lookbehind: true
}

What do you think?
(Regarding test suite, it's a known problem. For now, the closest thing we have is the Examples page... So I'll add an example for this specific issue once we agree on the way to fix it.)

That's where I started, but wasn't sure on the regex and switched to what I posted. What you posted works for my few brief examples. Would you say this is a clike problem or a bash problem? Let me know if you'd prefer I submit the pull request.

I'll commit this directly. For now, I'll consider this only applies to bash, because I don't think the case can happen in all c-like languages.


Extend the StrimziKafkaContainer to allow starting Kafka with KRaft.

The goal of this PR is to enable Kraft and reduce startup time. With it, I'm around 4-5 seconds to start the Kafka container. With such startup time, this Test Container can be used as the Quarkus Dev Services for Kafka. It also allows projects using Kafka to have an easy way to verify their behavior when KRaft is used.
This PR:

Adds support for Kraft in the StrimziKafkaContainer
Allows configuring the storage UUID (advanced option)
Updates scripts to support Kraft (changed override and avoid starting Zookeeper)
Extends test suite with specific Kraft tests

@see-quick @tombentley Thanks for your comments! I've pushed an updated version:

mentions KIP-500
adds waitForRunning
extracts the common configuration outside of the if(kraft) blocks

Thanks for the PR :)


LaunchPlan discovery fails silently if an error occur

You can test it by creating a new launch plan and passing the same input twice with different types to a workflow. For example:
new Workflow())
    .withName("boom")
    .withFixedInput("foo", "foo")
    .withFixedInput("foo", 42)

nvm.


Update Java MVVM discovery baselines + minor style fix

Current coverage is 78.49% (diff: 100%)

Merging #570 into master will not change coverage

@@             master       #570   diff @@
==========================================
  Files           248        248
  Lines          9245       9245
  Methods           0          0
  Messages          0          0
  Branches       1214       1214
==========================================
  Hits           7257       7257
  Misses         1678       1678
  Partials        310        310

Powered by Codecov. Last update de342e2...eca4499

LGTM

I don't see the review changes. Did they get lost?

@saicheems and I talked offline. They way this is coming about is rather roundabout: a single delete request message is being bound to multiple methods, so  for all but one, the sample string they provide is incorrect. From our POV, the correct solution would be to remove the sample string entirely, but then that leaves a delete request message undocumented in the proto, which is probably not good. We could have more specific one-off workarounds, but given that this API will be manually  wrapped, and that we've not seen this in other APIs, let's defer this.


Revisit supported ruby version

We nominally support ruby 2.1 or greater, and have Rubocop set up accordingly. However, it turns out this has not been true since we added the ActiveSupport dependency:
activesupport-5.1.1 requires ruby version >= 2.2.2, which is incompatible with the current version, ruby 2.1.8p440

Should we:
A) Change the dependency to fall in line with shipit-engine's minimum version as originally intended; or
B) Set an independent, higher requirement.
Given the extent to which we work with variable parsed JSON in this gem, the safe navigation operator introduced in  2.3 would be tremendously useful. And although we certainly designed this gem around Shipit's use case, it isn't all that functionally tied to Shipit. For those reasons, I vote for (B), and making that requirement ruby 2.3.
@Shopify/cloudplatform @kirs

👍 for 2.3 as long it's available on Shopify's Shipit setup

👍 for 2.3, I don't see any reason for supporting older versions than what Shipit is using as long as theres no other major users

2.3 sounds fine to me.


Panic with large body sizes

using current main 62dec83b1310edcdb4e86dc7b2f0a82d02741b4f (v2.9.2)
Error output:
$ go run . --authentication=false -d localhost -i localhost:8888
2023/05/27 - 23:28:42 | Starting SSH service on address: localhost:2222
2023/05/27 - 23:28:42 | Loading ssh_key as ssh-ed25519 host key
2023/05/27 - 23:28:44 | Accepted SSH connection for: <IP_ADDRESS>:42184
2023/05/27 - 23:28:44 | HTTP forwarding started: http://vp0.localhost:8888 -> /tmp/<IP_ADDRESS>_42184_8098909803 for client: <IP_ADDRESS>:42184
2023/05/27 - 23:29:07 | vp0.localhost:8888 | 200 |   551.48962ms |       <IP_ADDRESS> | GET      /testfile
2023/05/27 - 23:29:59 | vp0.localhost:8888 | 200 |  2.193983129s |       <IP_ADDRESS> | GET      /testfile
2023/05/27 - 23:30:16 | vp0.localhost:8888 | 200 |  2.334790643s |       <IP_ADDRESS> | GET      /testfile
2023/05/27 - 23:30:35 | httputil: ReverseProxy read error during body copy: unexpected EOF

2023/05/27 23:30:35 [Recovery] 2023/05/27 - 23:30:35 panic recovered:
net/http: abort Handler
/usr/lib/go/src/net/http/httputil/reverseproxy.go:519 (0x9bcc68)
	(*ReverseProxy).ServeHTTP: panic(http.ErrAbortHandler)
~/go/pkg/mod/github.com/antoniomika/oxy@v1.1.1-0.20210804032133-5924ea01c950/forward/fwd.go:540 (0xa1cab8)
	(*httpForwarder).serveHTTP: revproxy.ServeHTTP(w, outReq)
~/go/pkg/mod/github.com/antoniomika/oxy@v1.1.1-0.20210804032133-5924ea01c950/forward/fwd.go:267 (0xa1a186)
	(*Forwarder).ServeHTTP: f.httpForwarder.serveHTTP(w, req, f.handlerContext)
~/go/pkg/mod/github.com/antoniomika/oxy@v1.1.1-0.20210804032133-5924ea01c950/roundrobin/rr.go:147 (0xa150e7)
	(*RoundRobin).ServeHTTP: r.next.ServeHTTP(w, &newReq)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/utils.go:49 (0x9d4108)
	WrapH.func1: h.ServeHTTP(c.Writer, c.Request)
~/projects/sish/httpmuxer/httpmuxer.go:311 (0xb0f4a3)
	Start.func3: gin.WrapH(currentListener.Balancer)(c)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/context.go:173 (0x9cd3e1)
	(*Context).Next: c.handlers[c.index](c)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/recovery.go:101 (0x9cd3cc)
	CustomRecoveryWithWriter.func1: c.Next()
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/context.go:173 (0x9cc506)
	(*Context).Next: c.handlers[c.index](c)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/logger.go:240 (0x9cc4e9)
	LoggerWithConfig.func1: c.Next()
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/context.go:173 (0xb1007e)
	(*Context).Next: c.handlers[c.index](c)
~/projects/sish/httpmuxer/httpmuxer.go:71 (0xb0fee7)
	Start.func1: c.Next()
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/context.go:173 (0x9cb6ac)
	(*Context).Next: c.handlers[c.index](c)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/gin.go:652 (0x9cb695)
	serveError: c.Next()
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/gin.go:645 (0x9cb3ec)
	(*Engine).handleHTTPRequest: serveError(c, http.StatusNotFound, default404Body)
~/go/pkg/mod/github.com/gin-gonic/gin@v1.8.1/gin.go:572 (0x9cad7c)
	(*Engine).ServeHTTP: engine.handleHTTPRequest(c)
/usr/lib/go/src/net/http/server.go:2936 (0x76b315)
	serverHandler.ServeHTTP: handler.ServeHTTP(rw, req)
/usr/lib/go/src/net/http/server.go:1995 (0x766831)
	(*conn).serve: serverHandler{c.server}.ServeHTTP(w, w.req)
/usr/lib/go/src/runtime/asm_amd64.s:1598 (0x467e80)
	goexit: BYTE	$0x90	// NOP

2023/05/27 - 23:30:35 | vp0.localhost:8888 | 500 |   5.01434782s |       <IP_ADDRESS> | GET      /testfile

Commands to reproduce the issue:
$ ssh -R test:80:localhost:8080 -p 2222 localhost
Press Ctrl-C to close the session.

The subdomain test.localhost is unavailable. Assigning a random subdomain.
Starting SSH Forwarding service for http:80. Forwarded connections can be accessed via the following methods:
HTTP: http://vp0.localhost:8888

and
$ dd if=/dev/urandom of=testfile bs=1M count=800
800+0 records in
800+0 records out
838860800 bytes (839 MB, 800 MiB) copied, 2.68018 s, 313 MB/s
$ darkhttpd .
darkhttpd/1.14, copyright (c) 2003-2022 Emil Mikulic.
listening on: http://<IP_ADDRESS>:8080/
<IP_ADDRESS> - - [27/May/2023:23:30:35 +0200] "GET /testfile HTTP/1.1"<PHONE_NUMBER>47 "" "Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/113.0"

and now downloading http://vp0.localhost:8888/testfile via Firefox results in the sish panic. Downloading via port 8080 works fine.
In this example Firefox stopped after about 260MiB, but when using a remote server it already stops ~50Mib. Given the output I suspect that sish (or some library/golang) has a 5s request timeout, after which the response is aborted.
Expected behaviour: sish keeps the connection open as long as the server and client keep the connection open. Usecases: Long Polling APIs and large file downloads.

You can disable the idle connection logic by setting idle-connection=false either in the command line or in your sish config.

Thanks, but this issue shows that the idle connection logic is implemented wrong. The documentation says Duration to wait for activity before closing a connection for all reads and writes (default 5s), but in the example the connection is closed after 5 seconds regardless of activity

The idle timeout exists for both sides of the tunnel. If one side of the tunnel (the writer) doesn't receive any messages, the connection is considered dead. This is currently implemented with golang's connection deadlines, which doesn't really have a way to apply deadlines differently (single deadline which includes read or write as extending the deadline). This is the expected behavior of how this is implemented.
We could maybe make sish smarter by looking for when a deadline occurs and having retry logic in if the other side of the connection has still not hit its deadline, but this feature can just as easily be disabled and timeouts handled by the ping timeout instead

I'm gonna open a PR with https://github.com/antoniomika/sish/blob/main/utils/conn.go#L181 changed to just call SetDeadline instead of the individual read or write deadlines. This would theoretically push both deadlines continuously as either read or write is called. Mind helping me test out those changes @bibo38 ?

Thanks. Pushing both deadlines solves my problem :+1:
Here is the diff that worked for my downloading file usecase:
diff --git a/utils/conn.go b/utils/conn.go
index 45b1df1..6927c94 100644
--- a/utils/conn.go
+++ b/utils/conn.go
@@ -184,7 +184,7 @@ type IdleTimeoutConn struct {

 // Read is needed to implement the reader part.
 func (i IdleTimeoutConn) Read(buf []byte) (int, error) {
-       err := i.Conn.SetReadDeadline(time.Now().Add(viper.GetDuration("idle-connection-timeout")))
+       err := i.Conn.SetDeadline(time.Now().Add(viper.GetDuration("idle-connection-timeout")))
        if err != nil {
                return 0, err
        }
@@ -194,7 +194,7 @@ func (i IdleTimeoutConn) Read(buf []byte) (int, error) {

 // Write is needed to implement the writer part.
 func (i IdleTimeoutConn) Write(buf []byte) (int, error) {
-       err := i.Conn.SetWriteDeadline(time.Now().Add(viper.GetDuration("idle-connection-timeout")))
+       err := i.Conn.SetDeadline(time.Now().Add(viper.GetDuration("idle-connection-timeout")))
        if err != nil {
                return 0, err
        }

Handling the errors is probably also a good idea to print meaningful status errors like The [HTTP/SSH/TCP] connection with X was longer than Y idle and therefore terminated, instead of a panic (which let's you think it's a bug in sish).

Cool! Appreciate you went through building and testing. I'll make a PR and get a version patched!


Use rclone.conf from rclone executable directory if already existing

What is the purpose of this change?

To run rclone in "portable" mode, as in "portable app", by putting rclone.exe together with rclone.conf in a folder on a memory stick. Rclone will pick up the local config file without having to always remember to specify the --config command line argument, and not having to resort to launcher script like described in forum.
Was the change discussed in an issue or in the forum before?

The idea came from this forum post, although it does not describe the solution itself.
Checklist

[x] I have read the contribution guidelines.
[x] I have added tests for all changes in this PR if appropriate.
[x] I have added documentation for the changes if appropriate.

[x] I'm done, this Pull Request is ready for review :-)

This is a nice idea :-)

There are a few other applications that does similar things, although with variations (e.g. ILSpy checks for file ILSpy.xml to store settings in, VSCode checks for folder "data" to store settings, cache, extensions etc in, Notepad++ checks for dummy file "doLocalConf.xml" to indicate it should store settings, plugins etc in various hard coded locations beneath the application folder). Command line argument is "required", but to also be able to run it "portable" without having to remember to always use the same argument, or to resort to launcher script, is nice.

I'm a little concerned it might interfere with the already complicated finding the config file code, but it looks OK to me.

Sure, I can agree with that. I think it should be safe though (and it is a smaller change than my previous pr in the same code, #2515 :)

I've given the CI another kick to see if we can get it to pass!

Didn't seem to help, unfortunately...

Finally!  I'm getting closer to figuring out why the CI sometimes fails but not there yet! (#3154 )
I'll merge this now - thank you very much for this.


[Feature] Vela cli should support workflow subcommand

Is your feature request related to a problem? Please describe.

Hope to operate  application workflow on the command line, such as suspend and resume the workflow
Describe the solution you'd like

Vela cli should support workflow subcommand
vela workflow suspend
vela workflow resume
vela workflow stop
vela workflow restart

/assign


How to get validation loss and display it in Tensorboard

The current project contains only training loss and learning rate curves, how can I modify def train_one_epoch() to compute the validation loss during the training session?

Additionally, is there a way to modify the code so that the x-axis of the TensorBoard graphs represents epochs instead of batches？

I would also like to ask if there's a simple solution to this.

Getting train loss is simple, the model function in train_utils.py/train_one_epoch() returns all of that.
Creating predictions each epoch is also simple by just evaluating after each epoch, as explained here: #840. This evaluation also returns other metrics like recall and aos.
However, I'm not able to get loss values from evaluation - the model function doesn't return it and there doesn't seem to be a built-in utility function that calculates it from the given config, ground truths, and predictions.
I'm currently using a custom implementation of GIoU-3D for this (I'm mainly interested in positional loss and less so in classsification loss), but that doesn't seem like the right way to go.

I'm new to object detection networks, so I'm not sure if loss is utilised in the same way as elsewhere. I want to use validation loss for hyperparameter tuning and both train and validation set losses to check for overfitting. Is it more common to use accuracy metrics PCDet provides for this here?

Okay, I think I figured out how to get validation loss working without custom implementations and other silliness. This applies to the AnchorHead dense head, so your mileage may vary.

Make sure targets are generated even during validation by getting rid of this condition or editing it to always be True. An alternative that doesn't require PCDet's code (which I'm not a fan of) would be to manually call assign_targets() and edit forward_ret_dict, but I haven't looked into obtaining the data_dict parameter from outside.
Call the dense head's get_loss() function after each prediction, for instance here in this simplified bit from eval_utils.py (you still need to output it somewhere to console, Tensorboard, or a file):

losses = []
for i, batch_dict in enumerate(dataloader):
        load_data_to_gpu(batch_dict)
        with torch.no_grad():
                pred_dicts, ret_dict = model(batch_dict)
        losses.append(model.dense_head.get_loss()[1]['rpn_loss'])
loss = np.average(losses)

If anyone with more knowledge can chime in whether this is the right way or I'm doing something wrong, I'd be grateful. However, it appears to work.

Hi did it work for u??


Kite not responding in Spyder.

Description
What steps will reproduce the problem?

Please help to sole this issue. Kite plugin is there but code completion is not showing.
Traceback
Traceback (most recent call last):
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/preferences/widgets/configdialog.py", line 136, in button_clicked
    configpage.apply_changes()
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/preferences/api.py", line 110, in apply_changes
    self.apply_callback()
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/preferences/api.py", line 142, in <lambda>
    self._apply_settings_tabs(self.changed_options))
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/preferences/api.py", line 170, in _apply_settings_tabs
    self.apply_settings(options)
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/api/preferences.py", line 104, in wrapper
    func(opts)
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/api/plugins/new_api.py", line 528, in apply_conf
    self.after_configuration_update(list(options_set))
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/completion/plugin.py", line 353, in after_configuration_update
    self.register_statusbar_widget(provider_name)
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/completion/plugin.py", line 451, in register_statusbar_widget
    self.statusbar.add_status_widget(currrent_widget)
  File "/home/rodo/anaconda3/envs/rodo_main/lib/python3.8/site-packages/spyder/plugins/statusbar/plugin.py", line 109, in add_status_widget
    raise SpyderAPIError(f'Status widget `{id_}` already added!')
spyder.api.exceptions.SpyderAPIError: Status widget `kite_status` already added!

 Do you have Kite installed? The thing is their service is unavailable right now so there is not much we can do about that: https://www.kite.com/kite-is-temporarily-unavailable/?source=download
Also, how is the error being trigger for you? Any new info in order to reproduce this is greatly appreciated

Yeah, so the kite is installed and enabled for Visual Studio Code. But again in the kite app, I can't see that it is enabled for Spyder and there it was instructed to do it manually, specifically by going to preferences and enable code hinting by Spyder. Also I have tried to enable code hinting and suggestions by kite in Preferences section which is in parentheses and it was beforehand mentioned that it's not available. And after checking the required boxes, when I clicked on enable the box or window popped up to put your issue.
And now here I'm.

I was able to reproduce this @rodosingh (the issue related to the error message).
Regarding Kite completions, could you try to restart the Kite copilot and under settings click on Redo setup?
Under the status bar for kite you should see ready

Just in case, related issues: https://github.com/spyder-ide/spyder/issues/16837 https://github.com/spyder-ide/spyder/issues/16458

Will do and try to update you soon.

Above Image is in Kite window. Showing not available.

Here it's showing Notify me when Kite can provide missing completions (but is unavailable). And I have done the Redo Setup but nothing has happened.

Could it be that the jedi is not installed that's why the autocompletion is not available?

Hey @dalthviz I've solved the error. The problem is due to missing package of jedi>=0.17.2, <0.19.0.
Just uninstalled the old jedi to run the new one.
Now working fine.

Thanks @rodosingh to let us know how you manage to fix the issue with the completions :+1:

Since my issue was resolved, I'm closing this thread.


schema.org range included too strict if range includes both Text and Thing

Looking specifically at schema:itemListElement, range includes ListItem, Text and Thing. if the value is a typed resource, it fails at https://github.com/ruby-rdf/rdf-reasoner/blob/develop/lib/rdf/reasoner/schema.rb#L130-L140. Given that Thing is in the range, it should not fail.
See https://github.com/structured-data/linter/issues/37
cc/ @JHTrans

Fixed in ruby-rdf/rdf-rdfa#3


[Feature] Impliment "JumpList" like recent items

To add better access to recent files, I am suggesting that we implement partially some of the JumpList functionality in Windows Explorer. this will allow a more seamless experience in Cairo and possibly promote quicker adoption.
When a user right-clicks a "taskbar" item, the recents for that item are displayed.
I will start working on this in my fork an will add some of my findings to the "Issue" item as it may interest some followers.
Additional context

This request is a bit old but I think it is very useful, not only to open recent files but also to access other options such as opening a private tab in Firefox
In this way it would also be compatible with programs such as JumplistExtender which allows you to customize pinned applications from the taskbar.
We could even add this feature directly to Cairo and have more control over the Dock applications.

https://docs.microsoft.com/en-us/dotnet/api/system.windows.shell.jumplist?view=net-5.0


Dot Files Exercise Terminal Error

Bug Report
There seem's to be an issue with Line 59 of the .bash_profile file included as part of the Dot Files exercise error. After moving and renaming files to the home directory and starting a new terminal process the following error pops up:
-bash: unalias: git: not found
Line 59 reads:
export PS1=' \[\033[4;1;${PROMPT_COLOR}m\]\w\[\033[0m\]$(unalias git; __git_ps1 " $(git config --get user.email) (%s)") \n$ '
Removing the unalias git portion of the line seems to correct the error.
export PS1=' \[\033[4;1;${PROMPT_COLOR}m\]\w\[\033[0m\]$( __git_ps1 " $(git config --get user.email) (%s)") \n$ '

fixed. Thanks!


can't enter phyton state for Attach Obj Control node in houdini 19.5

It seems that Attach Obj Control works perfectly for houdini 19.0.x but unable to enter phyton state in houdini 19.5.x. Manually inserting the @name="joint name" works.

Currently, the tools only work with Houdini 19.0. At the current stage, we will skip the release for 19.5 and release an improved version for 20.0

I've released an update for H19.5, which includes the new Attach Geometry SOP. Please refer to the updated installation instructions in the tutorial. After you've set it up, please test it and let me know if everything works as expected. Thank you!


Rollup.js example

Do you have any example of code and CSS splitting with rollup?
Thks in advance

I don't have much experience using Rollup for code splitting unfortunately.
I'd recommend following https://github.com/rollup/rollup/issues/372


chargent: added threshold

Hi,
this changes the method so it would notify early instead of waiting for the peak to materialize. All charge curves I recorded so far show fast convergence at the peak, justifying this approach. powermanager does the same but relies on user supplied threshold, so I think this is not a duplicate.

Thanks!


Lack of doc for creating subtask

Describe the Change
It is hard to understand how to create a subtask for new contributors
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

fighting！

fighting！

Yours now!

New doc 'how to create a plugin' will contain this


sqlite: select * from sessions - only one entry?

Hi guys,
caused by my OpenDTU havin troubles for some days, some of my loads' solar percentages are false.
This is not an error of evcc!
Since I can re calculate the percentages based on other meter's infos, I wanted to edit the entries in table sessions via sqlite directly.
But ... I cannot fetch any session statics - there's only one entry...

Besides sessions, there's no other suitable table:

Anyone can help here?
Thanks in advance!
BR
Chris

Just one idea: Did you look at the correct database?
When you start evcc it is printing which database it uses.


K8S - kubectl CLI unable to connect

Rancher Version: 0.63
Docker Version: 1.9.1
OS: Rancher - 4.2.3 Server - OSX local
Steps to Reproduce:

Create K8s cluster
Copy config from kubectl window to local machine.
Attempt to run any kubectl command, e.g  kubectl version

Results: error: couldn't read version from server: an error on the server has prevented the request from succeeding
Expected:  kubectl to show client and server version.,
I am using Github auth, is this preventing the client access?

Kubectl only allows sending credentials if the server is SSL.  This means you need to run rancher behind SSL.  Refer to http://docs.rancher.com/rancher/installing-rancher/installing-server/basic-ssl-config/ for more info about SSL.

We have a new CLI coming for Rancher that will allow you to do rancher kubectl get po that will make this process easier.

We have made some magical unicorn changes such that even if Rancher server isn't running with SSL enabled kubectl will still work.  This is because port 8080 is actually now capable of serving http or https.  In short, in the next release the generate kubectl config will work regardless of whether you have SSL or not, or if you've enabled authentication or not.

@DaveBlooman Would you be able to test to see if this works for you using v1.0.0-rc1?

@DaveBlooman This has been fixed in our v1.0.0-rcX releases. If you still have issues, please re-open.

Rancher version: 1.1.1
Docker Version: 1.11.1
Rancher cluster: a list of ubuntu 14.04 instance in GCE
OS with kubectl installed: Windows 7 professional
Steps to Reproduce:

Created a 4-host kubernetes cluster

Generate and Copy config from kubectl page to local machine (i.e. placed config under C:\Users\XXXXX.kube ).

run kubectl command

Results: Unable to connect to the server: http: server gave HTTP response to HTTPS client
and when I change the server address from https:// to http:// , I get
Results: error: couldn't read version from server: an error on the server has prevented the request from succeeding
I am currently using http for the rancher server. I have tried v1.3.0 for both windows/amd64 and windows/386 but get the same error.
I remember that I have successfully use command like kubectl get service with my remote cluster before with using HTTP, but when I enable the access control function and change the config, it cannot work anymore.
May I know if this problem is related to SSL or other config issues? Thanks.


use private class fields

The benefit of using hard private class fields # instead of a soft typescript flavored private keyword makes it possible for esbuild to mangle the names do something like 1-2 letter word
i added the --minify --target=es2022 to esbuild build process and i compared the size it generated afterwards

before using #: 18 637 byte
after using #: 16 127  byte

So as you can see it mangles name to something short
beside, they should never have been exposed in the first place.
What i like about # better over the TS private keyword is that you can instantly recognize that this for instance
          if (this.#isHashPrefix()) {
            this.#changeState(State.HASH, /*skip=*/1);
          } else if (this.#isSearchPrefix()) {
            this.#changeState(State.SEARCH, /*skip=*/1);
            this.#internalResult.hash = '';
          } else {
            this.#changeState(State.PATHNAME, /*skip=*/0);
            this.#internalResult.search = '';
            this.#internalResult.hash = '';
          }

is using private class fields that aren't exposed. without having to hover over some variable or looking at where xyz where defined and wether or not it used the private keyword or not # is a instant recognizable token/symbol and it's how you truly define something as being really private
closes #116

@jimmywarting Thanks for the PR, but changing the target did break node-14 compatibility.
As LTS support for node 14 ended only a few weeks ago, I don't think we should be dropping support for it just yet.
When we are going to drop V14 support, that will make this a breaking change, and we should up the version number again.
I'm talking with @kenchris, to decide what we are going to to. But perhaps you can make some adjustments so this isn't needed?

Only change needed to keep older support would be to change the target version to whatever you like.
It can still utilize # for private stuff and mangle name to short names.
The only different would be that it instead would use WeakSet or WeakMap in the compiled version.
However i do think NodeJS v14 should be dropped.
in other case i think that it could also be useful to compile a new/extra ESM version with newer syntax.
like dist/urlpattern-es2022.js or something

@kenchris, What do you think about dropping V14 support? I would not mind that much.
@jimmywarting I think adding back in support for V14 is looking for the ??= things and refactor those to what we had before. I don't mind if the class's privates are a bit less private in old versions node versions.
either way, one of those needs to be resolved before we can merge this.


Integration of Witness Policy Tool into Witness with New Command witness policy

Introduction:
Proposing the integration of the Witness Policy Tool into Witness, including the addition of a new command: witness policy. This integration aims to streamline processes and improve usability.
Proposal for Integration and New Command:
Integrating the Witness Policy Tool's features directly into Witness with a new command witness policy for a cohesive experience.
Features for Integration with witness policy Command:

Policy Creation and Validation: Capability within witness policy to create and validate policy files.
Rego Module Generation: Inclusion of Rego module generation for attestation validation in the witness policy command.
Complex Policy Support: Handling sticky values, custom Rego modules, and certificate constraints through witness policy.

Command Integration and Usage:

Integrate Witness Policy Tool’s commands into Witness under witness policy, ensuring seamless command syntax and functionality.

Advanced Features:

Focus on custom Rego modules, certificate constraints, and diverse attestation data types under the witness policy umbrella.

Benefits:

Streamlined Workflow: A unified command enhances user experience and efficiency.
Enhanced Functionality: Expands Witness’s capabilities for SDLC security management.
Increased Adoption: A more intuitive and powerful tool fosters wider usage.

Conclusion:
The integration of the Witness Policy Tool into Witness, crowned by the witness policy command, will create a comprehensive and user-friendly solution for managing SDLC security and integrity.
ref github.com/testifysec/

We need a way to map flags to steps, there are two approaches I am considering:
Approach 1: Mapping Flags to Steps
In this approach, users explicitly map each flag to a step using a key-value pair. It requires users to be explicit about which step each flag applies to.
Command Example:
policytool generate --step "build" --step "deploy" --root-ca "build=rootCA.pem" --root-ca "deploy=deployCA.pem" --public-key "build=buildKey.pub" --public-key "deploy=deployKey.pub"

Here, the user specifies each flag with a corresponding step name (build or deploy) and the appropriate file.
Approach 2: Ordering-Based Flag Association
In this approach, flags are assumed to apply to the most recently defined step, based on their order in the command. Users list flags immediately after the step to which they apply.
Command Example:
policytool generate --step "build" --root-ca "rootCA.pem" --public-key "buildKey.pub" --step "deploy" --root-ca "deployCA.pem" --public-key "deployKey.pub"

In this example, the root-ca and public-key flags are applied to the nearest preceding --step flag. This approach is more intuitive for users as it follows the natural flow of command-line input.
User Experience Consideration:

Mapping Approach: More explicit, suitable for complex configurations, but can be verbose.
Ordering Approach: More intuitive and streamlined, easier for simple to moderately complex setups.

One vote for the mapping approach. Generally prefer the UX of unordered cli flags and it'll retain flexibility of potential future cli flag placement for new features. Maybe can also be paired with an assumption that if only one step is given, then step-specifiers for associated flags not needed.

I'd like to dynamically generate policies, given a subject.  We can build to the provenance graph as long as we have a good entry point to that graph, say the hash of the pipeline URL.
ex:
multiple steps
--sticky deploy=sticky.yml
--sticky deploy,test,build=sticky.yml
--sticky *=sticky.yml
single step
--sticky sticky.yml

Upvote on the mapping. If that support unordered flags its much less prone to making mistakes. Ordered flags are much more prone for me to make mistakes. And less easy to adapt scripts while working on it. So from a UX perspective, although more explicit, it more flexible in use in my opinion.

+1 for Approach Mapping Flags to Steps

I am going to start by adding a witness policy check command, there is a draft PR up


⚠ 366.ru has degraded performance

In 39d8863, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1094 ms

Resolved: 366.ru performance has improved in 6e27af8.


[TYPO] SAR quickstart notebook section numbers are incorrect

Description
SAR quickstart notebook section numbers are incorrect.
Specifically, the number jumps after Section 2.2 to 5
5. Evaluate how well SAR performs

Thanks for the quick fix!


[stable/datadog] fix: DD_TAGS including [] due to quote instead of join

What this PR does / why we need it:
According to https://docs.datadoghq.com/agent/docker/?tab=standard this should be formatted as key1:value1 key2:value2 not [key1:value1 key2:value2].
Which issue this PR fixes

fixes #21663

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Hi @nesl247
Thank you for your contribution. We have included you change in the pull request #22006 that fix severals reported issues.
I think you can safely close this PR.
Cheers
Cedric


Ability to disable the component

In the case where it takes time to process the file, the user may want to disable the file upload control until the processing is complete.

I think there should be better integration between the UI (file input) and the upload.  I don't think the consuming code of this component should be left to manage the state (for example, while uploading, in the event of errors, etc.).

I'm not just talking while this component is processing the file though. Our FRA takes the uploaded file and pushes it to S3 which can be a fairly lengthy process depending on how large the file is. This component wouldn't be aware that the S3 upload is happening so our component that wraps this would need to be able to control the disabled state.

Yep.  I think the application code should not be acting as the controller between the file upload and the file input.  There's a whole bunch of scenarios that can come up related to upload (to S3 or where ever) that potentially affect the state of the UI, and we don't want the app code to know about all of that.


groups: Use a single group configuration for all groups

There is no separate groups configuration in the section definition
anymore. Instead, all the attributes that were described there are added
to the group widget itself so there's only one place to configure the
group
** Breaking change **

The groupName property is renamed to title
The groupShortname does not exist anymore as it was not used
The groups definitions in the sections need to be moved to the
corresponding widget of type group.
Specific group templates may need to be updated.
The group widget config's shortname field must match the last part
of the path where are stored those objects.

En fait non, cette PR est juste un commit puisqu'une autre PR acceptée avait déjà ajouté le premier commit...


Clean Up .lock file on SIGTERM

Issue #, if available:
https://github.com/aws-robotics/aws-robomaker-sample-application-objecttracker/issues/14
Description of changes:
Remove .lock file on receiving SIGTERM as a cleanup, so that the exit routine is more graceful.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing this pull request as the source branch got messed up.


Replace input configuration with Hydra

Codecov Report

Merging #71 (9c1bd4b) into master (7cb75c1) will increase coverage by 0.46%.
The diff coverage is 50.60%.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   59.40%   59.86%   +0.46%
==========================================
  Files          16       18       +2
  Lines         574      608      +34
  Branches       56       59       +3
==========================================
+ Hits          341      364      +23
- Misses        205      214       +9
- Partials       28       30       +2

Impacted Files
Coverage Δ

mloq/cli/main.py
0.00% <0.00%> (ø)

mloq/cli/setup_cmd.py
0.00% <0.00%> (ø)

mloq/config/generation.py
90.47% <ø> (ø)

mloq/config/logic.py
48.64% <ø> (+8.64%)
:arrow_up:

mloq/directories.py
81.57% <ø> (ø)

mloq/requirements.py
54.54% <ø> (-1.71%)
:arrow_down:

mloq/templating.py
85.71% <ø> (ø)

mloq/version.py
0.00% <0.00%> (ø)

mloq/workflows.py
75.00% <ø> (ø)

setup.py
0.00% <ø> (ø)

... and 7 more

Continue to review full report at Codecov.

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


1621 card selection

Closes #1621
Summary
Add isSelected prop to base Card component that will allow for selection styling.  The component spreads props that will allow for event handlers for click, focus and blur, allowing teams to customize their selection logic.
Change List (commits, features, bugs, etc)

added isSelected prop to Card.jsx
added stateful component to story that showcases selection.

Acceptance Test (how to verify the PR)

click and tab through the Card component story and see the styling change.

:tada: This PR is included in version 2.141.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Install via NPM makes my Alfred Preferences unusable

I installed the workflow as suggested via the readme using npm install -g alfred-npms. On opening my Alfred preferences all I get is a white window 😕
Fortunately enough everything works again after uninstalling via npm.

Do you already have the npms package present in Alfred which was installed via Packal? If so, remove that one first. Alfred can't handle two workflows with identical names.

Was just gonna comment that this is probably an issue with alfred-link instead^^
I don't have another npms workflow installed. I do have a custom search set up that takes "npms" as an argument. Can that get in the way as well?

Might be, not familiar with custom searches. Is it possible to remove that one and try again?

I've tried, unfortunately to no avail. I'm not sure what could get into the way here.

What does this show you?
$ ls ~/Library/Application\ Support/Alfred\ 3/Alfred.alfredpreferences/workflows/

If you changed the workflow directory in Alfred preferences, ls that directory instead. I'm curious to see what's in there.

A bunch of "user.workflow.id"s and alfred-npms as a symlink to /usr/local/lib/node_modules/alfred-npms, which doesn't seem to exist (anymore).
Remove all the alfred-npms references and try installing again?

Yes try that. And you are 100% sure that one of those user.workflow.id doesn't have the <key>name</key> tag set to npms?

Took a bit to figure out how to do that without just reading through Alfred :D
find . -name "info.plist" -print0 | xargs -0 -L1 /usr/libexec/PlistBuddy -c "Print :name"
Definitely no npms in that output.

Strange. Can you try installing alfred-ng2. Still a white screen?

That worked just fine. Umm... I'm not sure what to say. I have absolutely no clue why npms isn't working.

Aha, I know why! It's indeed an issue with alfred-link. At first, alfred-link added the name property dynamically. That behaviour was then removed and I did a patch release, which off course should've been a feature release. Let me fix that.

@kiliankoe Did a PR. Have to wait for @sindresorhus though as I can't release this workflow to npm.

Thank you so much!

Well, thank you for letting me/us know :).


daily_walk_limit

Short Description
Please add ability to limit maximum amount of km walked per day. This is indirectly possible through maximum walk speed parameter and other resting/stop parameters. But if there is a direct config option to limit max walk distance per day/hour etc, that would make it easier to simulate real human behavior. I dont think it is practical to walk 20+km every day for most people.
Possible solution
Add max_distance_per_day or max_distance_per_hour parameter.
record amount of distance walked in db.
max_distance_per_day parameter should be checked against db before walking.
How it would help others
This would make it easy to simulate real human behavior.

set 'car speed' :-D

I don't think Niantic will ban people for walking 20 km. I agree with MZorzy, setting a speed would be enough.


IdEmbedding not support PyTorchEngine

IdEmbedding userEmbedding = new IdEmbedding.Builder()
.setDictionarySize(userCount)
.setEmbeddingSize(64)
.build();
userEmbedding.initialize(manager, DataType.FLOAT32, usersND.getShape());
userEmbedding.forward(ps, new NDList(manager.create(new int[] {1,2,3,4})), true).singletonOrThrow();
Exception in thread "main" java.lang.UnsupportedOperationException: Not supported!
at ai.djl.ndarray.BaseNDManager.invoke(BaseNDManager.java:285)
at ai.djl.nn.transformer.MissingOps.gatherNd(MissingOps.java:31)
at ai.djl.nn.transformer.IdEmbedding.forwardInternal(IdEmbedding.java:72)
at ai.djl.nn.AbstractBlock.forward(AbstractBlock.java:121)
at ai.djl.nn.Block.forward(Block.java:122)

@dxjjhm Thanks for reporting this issue. Currently DJL only implemented gather in MXNet. We prioritize this for PyTorch.

Do you have some methods to get embedding-batch by input-batch

by using  NDArrayEx Interface, I am already found a solution as follow.
NDArray inputUserEmbedding = userInput.getNDArrayInternal()
.embedding(userInput, userEmbeddingTable, SparseFormat.DENSE)
.singletonOrThrow();

gather has been implemented in PyTorch.


github repository "README.md" contains link to careers page that doesn't exist anymore

The portainer github.com repositry contains a link to your main web page for careers and that link
points to a non-existant web-page.
best regards,
Florian La Roche

@laroche
Thanks for pointing that out.
We will sort it out asap.

The link should now be working correctly again.


[master] Update BitBucket links

Signed-off-by: Louise Poubel<EMAIL_ADDRESS>

btw I think we can just remove bitbucket-pipelines.yml. We can do that in a separate PR if necessary

I think we can just remove bitbucket-pipelines.yml

I think release-tools's auto jobs may still be using it to figure out the Ubuntu version. CC @j-rivero


adds preflights to check disk space for containerd, rook and openebs

What this PR does / why we need it:
To ensure that the disk has enough space for the add-ons containerd, rook and openebs
Which issue(s) this PR fixes:
Motivates by # [sc-71717]
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Adds host preflight check to verify ephemeral disk space for the add-on(s) containerd, rook and openebs

Does this PR require documentation?

Hi @emosbaugh,
Thank you for the review. The suggestions were addressed.

Lets just wait for others to chime in


BUILD-261: create outside of git control release.yaml to push to repository release page for dev preview install

/assign @adambkaplan

How about something like this in hack/generate-release-yaml.sh?
#!/bin/bash

FILES=(
"00-namespace.yaml"
"0000_10_projectedresource.crd.yaml"
"01-service-account.yaml"
"02-cluster-role.yaml"
"03-cluster-role-binding.yaml"
"csi-hostpath-driverinfo.yaml"
"csi-hostpath-plugin.yaml"
)

rm -f release.yaml

for FILE in ${FILES[@]}
do

	echo -e "\n---\n" >> release.yaml
	cat deploy/${FILE} >> release.yaml

done

Then make generate-release-yaml can just be:
./hack/generate-release-yaml

How about something like this in hack/generate-release-yaml.sh?
#!/bin/bash

FILES=(
"00-namespace.yaml"
"0000_10_projectedresource.crd.yaml"
"01-service-account.yaml"
"02-cluster-role.yaml"
"03-cluster-role-binding.yaml"
"csi-hostpath-driverinfo.yaml"
"csi-hostpath-plugin.yaml"
)

rm -f release.yaml

for FILE in ${FILES[@]}
do

	echo -e "\n---\n" >> release.yaml
	cat deploy/${FILE} >> release.yaml

done

Then make generate-release-yaml can just be:
./hack/generate-release-yaml.sh

sure I can mess with it next time I circle back to this - thanks

/retest

changes pushed @coreydaley PTAL
given @adambkaplan busy state wrt kubecon, going to
/unassign @adambkaplan
and
/assign @adambkaplan
if need be we can iterate on this more once Adam actually tries running make generate-release-yaml and uploading the content to the repository's release page

duh :-)
/unassign @adambkaplan
/assign @coreydaley

assuming that the new hack/.sh is executable.
/lgtm

assuming that the new hack/.sh is executable.
/lgtm

thanks, and yes, it is:
gmontero ~/go/src/github.com/openshift/csi-driver-projected-resource  (gen-rel-yaml)$ ls -la hack
total 24
drwxrwxr-x.  2 gmontero gmontero 4096 May  4 10:49 .
drwxrwxr-x. 12 gmontero gmontero 4096 May  4 10:41 ..
-rw-rw-r--.  1 gmontero gmontero   76 Sep  4  2020 boilerplate.go.txt
-rwxrwxr-x.  1 gmontero gmontero  399 May  4 10:49 generate-release-yaml.sh
-rwxrwxr-x.  1 gmontero gmontero  390 Sep  4  2020 update-crd.sh
-rwxrwxr-x.  1 gmontero gmontero  380 Sep  4  2020 update-generated.sh
gmontero ~/go/src/github.com/openshift/csi-driver-projected-resource  (gen-rel-yaml)$

assuming that the new hack/.sh is executable.
/lgtm

thanks, and yes, it is:
gmontero ~/go/src/github.com/openshift/csi-driver-projected-resource  (gen-rel-yaml)$ ls -la hack
total 24
drwxrwxr-x.  2 gmontero gmontero 4096 May  4 10:49 .
drwxrwxr-x. 12 gmontero gmontero 4096 May  4 10:41 ..
-rw-rw-r--.  1 gmontero gmontero   76 Sep  4  2020 boilerplate.go.txt
-rwxrwxr-x.  1 gmontero gmontero  399 May  4 10:49 generate-release-yaml.sh
-rwxrwxr-x.  1 gmontero gmontero  390 Sep  4  2020 update-crd.sh
-rwxrwxr-x.  1 gmontero gmontero  380 Sep  4  2020 update-generated.sh
gmontero ~/go/src/github.com/openshift/csi-driver-projected-resource  (gen-rel-yaml)$

Thanks.
I didn't see a way to tell if it was on github


Add coverage reporting

Adds support for publishing the code coverage to codecov.io similar to what we do for clkhash.
Example coverage report here.
Screenshot:

Once we install the codecov GitHub appliction (I can't myself) the bot should comment on PRs that lower the code coverage.

codecov app has been added too


proxy报错：bad resp CRLF end

2015/09/08 17:47:00 [INFO] session [0xc20fe60e80] closed: {"sid":157266540,"ops":10,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:33075"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a66240] closed: {"sid":157266550,"ops":3,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:33100"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc20fe614c0] closed: {"sid":157266544,"ops":2,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:25923"}, quit
2015/09/08 17:47:00 [INFO] session [0xc20e0f8900] created, sid = 157266552
2015/09/08 17:47:00 [INFO] session [0xc2208209c0] created, sid = 157266553
2015/09/08 17:47:00 [INFO] session [0xc20e0f8900] closed: {"sid":157266552,"ops":1,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:36036"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a66b80] created, sid = 157266554
2015/09/08 17:47:00 [INFO] session [0xc23978f380] closed: {"sid":157266446,"ops":22,"lastop":1441705620,"create":1441705619,"remote":"<IP_ADDRESS>:32861"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc2e5324440] closed: {"sid":157265473,"ops":19,"lastop":1441705620,"create":1441705615,"remote":"<IP_ADDRESS>:33422"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a67380] created, sid = 157266555
2015/09/08 17:47:00 [INFO] session [0xc2345252c0] closed: {"sid":157265742,"ops":28,"lastop":1441705620,"create":1441705616,"remote":"<IP_ADDRESS>:34068"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc2542fab00] closed: {"sid":157265195,"ops":11,"lastop":1441705620,"create":1441705614,"remote":"<IP_ADDRESS>:58151"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc23c20fe40] closed: {"sid":157265727,"ops":37,"lastop":1441705620,"create":1441705616,"remote":"<IP_ADDRESS>:34040"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc20e0f92c0] created, sid = 157266556
2015/09/08 17:47:00 [INFO] session [0xc20e0f92c0] closed: {"sid":157266556,"ops":3,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:33108"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc23e910580] closed: {"sid":157118384,"ops":194,"lastop":1441705620,"create":1441704045,"remote":"<IP_ADDRESS>:21053"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a67380] closed: {"sid":157266555,"ops":2,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:25926"}, quit
2015/09/08 17:47:00 [INFO] session [0xc242d39c40] closed: {"sid":157265889,"ops":7,"lastop":1441705620,"create":1441705617,"remote":"<IP_ADDRESS>:34429"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc243933780] closed: {"sid":157266506,"ops":15,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:33007"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc220821ec0] created, sid = 157266557
2015/09/08 17:47:00 [INFO] session [0xc220821ec0] closed: {"sid":157266557,"ops":1,"lastop":1441705620,"create":1441705620,"remote":"<IP_ADDRESS>:33116"}, error = bad resp CRLF end

协议解析错误。redis 协议是 \r\n 结束的。确定协议格式正确？
On Tuesday, September 8, 2015, kmephistoh<EMAIL_ADDRESS>wrote:

2015/09/08 17:47:00 [INFO] session [0xc20fe60e80] closed:
{"sid":157266540,"ops":10,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:33075"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a66240] closed:
{"sid":157266550,"ops":3,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:33100"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc20fe614c0] closed:
{"sid":157266544,"ops":2,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:25923"}, quit
2015/09/08 17:47:00 [INFO] session [0xc20e0f8900] created, sid = 157266552
2015/09/08 17:47:00 [INFO] session [0xc2208209c0] created, sid = 157266553
2015/09/08 17:47:00 [INFO] session [0xc20e0f8900] closed:
{"sid":157266552,"ops":1,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:36036"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a66b80] created, sid = 157266554
2015/09/08 17:47:00 [INFO] session [0xc23978f380] closed:
{"sid":157266446,"ops":22,"lastop":1441705620,"create":1441705619,"remote":"
<IP_ADDRESS>:32861"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc2e5324440] closed:
{"sid":157265473,"ops":19,"lastop":1441705620,"create":1441705615,"remote":"
<IP_ADDRESS>:33422"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a67380] created, sid = 157266555
2015/09/08 17:47:00 [INFO] session [0xc2345252c0] closed:
{"sid":157265742,"ops":28,"lastop":1441705620,"create":1441705616,"remote":"
<IP_ADDRESS>:34068"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc2542fab00] closed:
{"sid":157265195,"ops":11,"lastop":1441705620,"create":1441705614,"remote":"
<IP_ADDRESS>:58151"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc23c20fe40] closed:
{"sid":157265727,"ops":37,"lastop":1441705620,"create":1441705616,"remote":"
<IP_ADDRESS>:34040"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc20e0f92c0] created, sid = 157266556
2015/09/08 17:47:00 [INFO] session [0xc20e0f92c0] closed:
{"sid":157266556,"ops":3,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:33108"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc23e910580] closed:
{"sid":157118384,"ops":194,"lastop":1441705620,"create":1441704045,"remote":"
<IP_ADDRESS>:21053"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc253a67380] closed:
{"sid":157266555,"ops":2,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:25926"}, quit
2015/09/08 17:47:00 [INFO] session [0xc242d39c40] closed:
{"sid":157265889,"ops":7,"lastop":1441705620,"create":1441705617,"remote":"
<IP_ADDRESS>:34429"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc243933780] closed:
{"sid":157266506,"ops":15,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:33007"}, error = bad resp CRLF end
2015/09/08 17:47:00 [INFO] session [0xc220821ec0] created, sid = 157266557
2015/09/08 17:47:00 [INFO] session [0xc220821ec0] closed:
{"sid":157266557,"ops":1,"lastop":1441705620,"create":1441705620,"remote":"
<IP_ADDRESS>:33116"}, error = bad resp CRLF end
—
Reply to this email directly or view it on GitHub
https://github.com/wandoulabs/codis/issues/416.

协议错误是指业务程序没有向proxy提交合法的redis查询请求吗？是应为用了不支持的codis命令吗？

跟命令无关，应该是协议不合法，你用的什么client？

就是php的标准client，问题是开发人直接对后端的codis做了replication

问题是开发人直接对后端的codis做了replication

没看懂这句，replication是啥？


Feature: add snapshot testing

Type of feature
🍕 Feature
Current behavior
right now the tests we have are for functions (I hope) and we have nothing to test the desing itself, some minor changes in the tailwind can break the whole site.
Suggested solution
I am suggesting to use: Snapshot Testing
so that we can ensure that the design does not break with minor things. Relying on human behavior's is not working always.
https://jestjs.io/docs/snapshot-testing
Additional context
Related https://github.com/open-sauced/insights/issues/1526
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

We switched to Vitest but will revisit this


No test case name in summary output for TEST_CASE_END_OVERRIDE assertion

If there is a failing assertion in TEST_CASE_END_OVERRIDE in the summary the originating test case name does not appear.
Currently the output looks like e.g.
  ? Assertion "CHECK_EQUAL_VAR(bugCount_ts, 0)" failed in TEST_CASE_END_OVERRIDE?CheckForBugMessages (UTF_HelperFunctions.ipf, line 144?767)
It would be very helpful if also the name of the originating test case is mentioned in the message.

verify if assertions in TEST_CASE_BEGIN_OVERRIDE have the same issue

Well we do treat the hook functions now as test cases so the behaviour you are seeing is kind of by design I guess. Still I agree that this would be helpful.


not_in with an empty array produces no results

Doing
User.where(User.arel_table[:id].not_in([]))

performs the query
SELECT "users".* FROM "users" WHERE ("users"."id" NOT IN (NULL))

which produces an empty result set. This seems a little strange, both from the SQL semantics of NOT IN (NULL) not matching anything, and from Arel. From the usual use case, I would expect not_in([]) to match all records.

For anyone else coming across this, it looks like this issue has been fixed.  When I ran the arel statement above in Rails 3.2.18 the SQL generated did not have this issue.

In Rails 5.2, I see a similar scenario when explicitly excluding nil:
User.where(User.arel_table[:id].not_in([nil]))

Zero results are returned from the query.


Link ppa not updated on Ubuntu since version 23.1.1

Description
Hello, I've found some similar issues at different moments.
Could you update the version on ppa?
Thank you!
DBeaver Version
Community Edition 23.1.1
Operating System
No response
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Thank you for report. We'll update it.

PPA package has been updated


extraDependencies for createQuery and createJsonQuery

fixes: https://github.com/igorkamyshev/farfetched/issues/350

There is no way to provide custom sourced fields using createQuery and createJsonQuery. It PR allows to pass stores as relation for query to handle sourced fields relation side-effects

Check internal implementation of createQuery. You can write something like this in your code.

Sure
I also can fork whole farfetched to my codebase if i need 1 simple extra parameter. But i dont want all of that complexity and extra maintaining cost for new releases, so i decided to contribute to main repo
If u think its wrong fit just say it and close PR/issue. I will found a way to deal with my cases in my codebase instead

Sorry, without strong motivation and any documentation I simply do not get why this feature has to be in core.
Please, provide proper RFC and documentation and reopen PR with it 🙏


Throwout with button

I want make throw out with button exactly as Tinder  in controller $scope

I think you can do it manually. But I need to inform you. I need to completely remove current card in the stack in my case. When I need action like that, I did it with the following;
$scope.throwOut = function () {
  $element = $element.closest('li[swing-card]:last');
  $element.addClass('animated rotateOutUpLeft');

  # this will work when animate is completed.
  $timeout(function () {
    $scope.cards.splice(-1);
  }, 400);
};

So as you see, I used animate.css to animate the current card. Then I removed the card when animate is completed.
If you need to keep the card in the stack after action, I think it's a little bit complex than this. Maybe you can do it to just modify the animation.

This could be solved with card.throwOut, or not? See the original, not angular wrapped, docs
@gajus what would it take to do this?


[Question] Get information of change of a model

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
I use Monaco Editor and React Monaco Editor to build our own editor.
An event onChange  is emitted when the content of the current model has changed, where we can get newValue of the model.
Now, I would like to get information of comparison of newValue and oldValue: for instance, insertion of certain characters at a location, deletion of certain characters at a location, etc.
Does anyone know how to get these information?

I also want to know.

Perhaps editor.onDidChangeModelContent((a) => console.log(a)); is what you want
{
    "changes": [
        {
            "range": {
                "startLineNumber": 9,
                "startColumn": 6,
                "endLineNumber": 9,
                "endColumn": 6
            },
            "rangeLength": 0,
            "text": "o",
            "rangeOffset": 74,
            "forceMoveMarkers": false
        }
    ],
    "eol": "\n",
    "versionId": 7,
    "isUndoing": false,
    "isRedoing": false,
    "isFlush": false
}


#4366 New project added

Title and Issue number
#4366 Add background template for websites
Close #4366

Checklist:

[x] I have mentioned the issue number in my Pull Request.

@TusharKesarwani  @MohitGupta121  please review this pr under gssoc'23


Training other dataset

Hi @zhjohnchan @kkakkkka ,
Could you please let me know how to train this model for a different dataset?

Hi there!
Thank you for your interest in our project.
To ensure compatibility, please structure your dataset in the same format as the one we use in our repository. Once you've done that, you can follow the preprocessing steps outlined in the README to prepare your data for training.


Release <IP_ADDRESS> - google maps hotfix

Fix for google map display in resource landing page

Can one of the admins verify this patch?


Fix typo for Command R finetuning notebook

small typo to make Jupyternotebook to work

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


Add tests that mock s3 to unit test S3TaskStorage

Changes in #5 does not support unit testing S3TaskStorage with a mechanism to mock S3. This prevents the newly added tests from working in CI.

We've got this in the current (and upcoming) version, so I'm closing as done.


[CodeClean] error case in mosquitto impl

Code clean, handle error case if failed to create mosquitto handle.
Signed-off-by: Jaeyun<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #124. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jaeyun-jung, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-edge/ci/repo-workers/pr-checker/124-202211021706040.97212100028992-c5670349878605644b95d518af88f2927f96b1cd/.

:octocat: cibot: @jaeyun-jung, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-edge/ci/repo-workers/pr-checker/124-202211021740320.4170138835907-ee7c5f2d77a62ccb4146499d347fb0626c69b8d6/.


Typescript's autocomplete under vscode doesn't seem to work

Describe the bug
In my vscode development environment, the typescript extension is installed.
But there is no auto-complete prompt in this project. Autocomplete can work in other projects.
There may be more than just typescript that cannot work, there are no prompts, no member variable autocomplete, no css autocomplete, and no path autocomplete.
To Reproduce
Steps to reproduce the behavior:

Enter code in any file

Expected behavior
vscode will do it autocomplete.
Screenshots

Desktop (please complete the following information):

OS: Linux
Browser chrome
Node Version v20.10.0
Vscode Version v1.85.0

Additional context

I don't have any issues with autocomplete in my vscode, and there doesn't seem to be anything in this project that would affect autocomplete, so I doubt it's a problem with other settings or anything.

I'm closing this, feel free to reopen if it's necessary


Release for ES 7.8.1

Release notes: https://www.elastic.co/guide/en/elasticsearch/reference/7.8/release-notes-7.8.1.html

Version <IP_ADDRESS> has been released today.


update Project info on app from Google Sheets

Working w/ @jaschallert, we want to update Projects tab with new information provided by Sharon:

Google Sheets
To update the app sections, Google Sheets pull from here:

Projects and other tabs in the app:
data | marineenergy.app - Google Sheets
EXCEPT for BioAssessments tab, which uses:
ba-docs - Google Sheets

R Scripts
For instance, to update Projects, you'll want to run the update_projects() here:
https://github.com/marineenergy/apps/blob/19af11c6ebccde14ff1247f3c5598dd3a1e064e7/scripts/update_all.R#L1-L20

Debugging with @jaschallert
Note, that I am NOT seeing an active Project, nor a Git tab in the upper right pane of RStudio:

The problem is that the paths to scripts are based on opening an RStudio project with a *.Rproj file. This sets the working directory, which you can check with the command getwd() in the R Console. The here() command looks for this *.Rproj file as the "root" of the project from which to base all other paths.

Here's the error that @jaschallert was getting:
Error in file(filename, "r", encoding = encoding) :
  cannot open the connectionIn addition: Warning message:In file(filename, "r", encoding = encoding) :  cannot open file '/home/jschallert/scripts/common.R': No such file or directory

And @jaschallert, if you want to clarify documentation, please feel free to proceed editing here:

Chapter 3 R Technicians | A Minimal Book Example

source code: marineenergy/help: 02-rtech.Rmd


LICENSE

Can you clarify the license under which you are releasing this port?  Preferably with a file in the repo or mention in the README.
The godocs for the repo include a copy of the original google copyright and license, but it seems this does not cover your port.

I've updated the README to clarify things:
License
As this is just a fairly literal port of Google's code consider this under
the same license (MIT license). For details see COPYING
Where COPYING is the license you found in the godocs documentation
Best regards
Lee McLoughlin CTO LMMR Technologies
http://uk.linkedin.com/pub/lee-mcloughlin/1/973/481
On 19 September 2016 at 20:16, Marty Schoch<EMAIL_ADDRESS>wrote:

Can you clarify the license under which you are releasing this port?
Preferably with a file in the repo or mention in the README.
The godocs for the repo include a copy of the original google copyright
and license, but it seems this does not cover your port.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/leemcloughlin/gofarmhash/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKxT7zr1PeDGpamp2zYDfbL92Y9Lufc8ks5qrt-KgaJpZM4KA2_c
.

Thanks


adapt to calculator

I created a calculator ros package. But I need to modify a little bit of this package.
It seems that this package is still functional. But please check it again.

Have you made any changes in monitor.py? It looks similar to the existing monitor script.

I didn't change any in monitor.py. I just rewrite some functions in my code.

I didn't change any in monitor.py

Then can you remove monitor.py from the PR?


[coolprop] Fix usage, re-fix installation

Fix export macro.
Fixes #18281.

@LiuZhexuan Can you please test my changes?
Thanks.

Cool, thanks @JackBoosY :)


[Type checker] When type resolution fails, don't check bridgeable conformances

Should address rdar://problem/34760012, although there isn't enough detail
there to be certain.

@swift-ci please smoke test and merge

@swift-ci please clean smoke test Linux


verification-pending route doesn't works

Hi guys, something wrong with that route ?? im using the default accounts-entry configuration ...

here the screen:


Feature/generic oauth provider updates

Split out the environment (Environment) and client specific (OpenIdOptions) properties.
Split out the OpenIdClient into:

OAuth2Client - Specification
OAuth2JWTClient - Specification
OpenIdClient - Specification
SalesforceClient - Specification

Update flow, grant, middleware, revocation and userinfo for these change.
Refactor the system tests
Add a Google system test
Add middleware for Salesforce Identity URLs

Only parts of the specifications required for the Salesforce Client are implemented

@sfryffdc This should address the comment in https://github.com/financialforcedev/orizuru-auth/pull/24#pullrequestreview-212254469.
I had a thought that it would be good to have the type of client in the environment too and split out the OAuth2/OAuth2JWT/OpenID/Salesforce related stuff, hence this PR is a bit larger than I intended, but more extendible.
@rcorfieldffdc It should also address your comments of reducing the risk of parameter injection.


Fix block count corruption on writer.flush() after an auto-sync

Small change - I think it seems right to reset the block count to zero whenever you do writer.dump() (whether through an auto-sync or a manual flush()). Otherwise dump() can be called with no available blocks, which seems to be bad.

Thanks!
For some seemingly unrelated problem travis is failing. I'll run some tests locally and fix this before final merge.

Can you merge with master so I'll see travis passing? Thanks

Thanks!

Cheers (and sorry I didn't get a chance to do the merge for Travis)!


Custom Calender Dates (Min- Max)

Hi i want to  show the calendar from the dates which i am giving like from date and end date . Is it possible?.

The best you can do is use the selectDate: method which will set the current month to that of the passed in date. Custom date ranges are not supported.


.Net: Use Arrays instead of Lists for DuckDB

Motivation and Context
Fixed-length Arrays have better performance than Lists: https://duckdb.org/2024/02/13/announcing-duckdb-0100.html#fixed-length-arrays
Description
Since version 0.10.0 DuckDB supports fixed length Array type that performs better than List type.
Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

@microsoft-github-policy-service agree [company="Space International"]

@microsoft-github-policy-service agree

@markwallace-microsoft Build fails with this error, should I update the NuGet package as part of this PR?

error NU1903: Package 'System.Text.Json' 8.0.2 has a known high severity vulnerability

@Giorgi we have a separate PR to handle this one.

#7185

@RogerBarreto Now that the PR is merged can you run the checks again for this PR?

I think it supports both but Array was introduced for better performance


xeus-cling broken

Describe the bug
With a recent (as of 2024-04-25) nixos-unstable, the executable provided in xeus-cling, xcpp, errors out immediately after launch with the following error:
❯ xcpp
: CommandLine Error: Option 'lto-embed-bitcode' registered more than once!
LLVM ERROR: inconsistency in registered CommandLine options
[1]    178364 IOT instruction (core dumped)  xcpp

Steps To Reproduce
Steps to reproduce the behavior:
With the following flake.nix
{
  inputs = {
    nixpkgs.url = "nixpkgs/nixos-unstable";
    flake-utils.url = "github:numtide/flake-utils";
  };

  outputs = { self, nixpkgs, flake-utils }:
    flake-utils.lib.eachDefaultSystem (system:
      let
        pkgs = import nixpkgs { inherit system; };
        jupyterBundle = pkgs.jupyter.override {
          definitions = { cpp17 = pkgs.cpp17-kernel; };
        };
      in
      {
        devShell = pkgs.mkShell {
          name = "demo";
          buildInputs = [
            jupyterBundle
            pkgs.xeus-cling
          ];
        };
      });
}

The corresponding nixpkgs is 572af610f6151fd41c212f897c71f7056e3fb518.
Note that my NixOS system is grossly out-of-date.
Expected behavior
xcpp should run
Supplemental info
core dump:
       Message: Process 178364 (xcpp) of user 1000 dumped core.

                Module linux-vdso.so.1 with build-id da5c6c53fbe35d8b03e6dfe95e7c028e2bfcd7e7
                Module libuuid.so.1 with build-id 37d35a84946281110191ba2b51bece1082238885
                Module libpthread.so.0 with build-id efbe91a94c283a4f5ef9d35e527b2275771e24bf
                Module libxml2.so.2 without build-id.
                Module libffi.so.8 without build-id.
                Module ld-linux-x86-64.so.2 with build-id 6bd610665dfe9b03d95b980402a36f034b20b75c
                Module libc.so.6 with build-id c0e1a19579c6b6aebef99235c7d0dc5a6fc03b23
                Module libgcc_s.so.1 without build-id.
                Module libstdc++.so.6 without build-id.
                Module libxeus.so.9 without build-id.
                Module libcrypto.so.3 with build-id f524b178e83c82d459e9cd6b56bc76214fc48792
                Module libzmq.so.5 without build-id.
                Module libxeus-zmq.so.4 without build-id.
                Module libncursesw.so.6 without build-id.
                Module libz.so.1 without build-id.
                Module libm.so.6 with build-id 69d287bfba1da08f7d73e7b7c73cb473b9d5f301
                Module libdl.so.2 with build-id 46595fe8b08ac7a5b3cbcc31f41025b893bdea5c
                Module librt.so.1 with build-id f3cc3d7a2b288b2d7f621924c19357d93e7b963c
                Module libLLVM-13.so with build-id 3627016ca2b8b34e7f42fa849057f8abb626dbb1
                Module libxeus-cling.so.0 without build-id.
                Module xcpp without build-id.
                Stack trace of thread 178364:
                #0  0x00007efff68b9efc __pthread_kill_implementation (libc.so.6 + 0x8fefc)
                #1  0x00007efff6869e86 raise (libc.so.6 + 0x3fe86)
                #2  0x00007efff6852935 abort (libc.so.6 + 0x28935)
                #3  0x00007efff80b59d1 _ZN4llvm18report_fatal_errorERKNS_5TwineEb (libLLVM-13.so + 0xc3c9d1)
                #4  0x00007efff80b5b5a _ZN4llvm18report_fatal_errorEPKcb (libLLVM-13.so + 0xc3cb5a)
                #5  0x00007efff8095642 _ZN12_GLOBAL__N_117CommandLineParser9addOptionEPN4llvm2cl6OptionEPNS2_10SubCommandE (libLLVM-13.so + 0xc1c642)
                #6  0x00007efff8096537 _ZN4llvm2cl6Option11addArgumentEv (libLLVM-13.so + 0xc1d537)
                #7  0x00007effff2287cd _Z41__static_initialization_and_destruction_0ii.constprop.0 (libxeus-cling.so.0 + 0x13cf7cd)
                #8  0x00007f000668008e call_init (ld-linux-x86-64.so.2 + 0x508e)
                #9  0x00007f000668017c _dl_init (ld-linux-x86-64.so.2 + 0x517c)
                #10 0x00007f0006696830 _dl_start_user (ld-linux-x86-64.so.2 + 0x1b830)
                ELF object binary architecture: AMD x86-64

Notify maintainers
@thomasjm

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.21, NixOS, 22.11 (Raccoon), 22.11.20230325.da26ae9`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - channels(root): `""`
 - nixpkgs: `/nix/store/rgszd759xg7rj5rg5h2k97sgj42ym6j6-source`

Add a :+1: reaction to issues you find important.

Starting to investigate this. It looks like the problematic commit is b1ef46706f5f566419b6798ea888b6d60e3387ef. That commit switched it from using LLVM 9 to 13.
There are some reports that xeus-cling isn't compatible with LLVM 13: https://github.com/jupyter-xeus/xeus-cling/issues/513

Hi @vgvassilev, is there any chance you could tell us why this is happening?
The problem arose after we upgraded cling and xeus-cling from LLVM 9 to LLVM 13. My understanding is that this issue happens when some package has LLVM components compiled into it statically, and then also tries to dynamically link against libLLVM.so, resulting in an error about double-registering command line options.
I saw that xeus-cpp runs against later LLVM versions, so presumably solved this problem somehow.
(I was also wondering, should I be thinking about switching my efforts from the old cling + xeus-cling to CppInterop + xeus-cpp? The latter is better supported and is the approach of the future, right?)
Thanks in advance for any help!

Hi @vgvassilev, is there any chance you could tell us why this is happening?
The problem arose after we upgraded cling and xeus-cling from LLVM 9 to LLVM 13. My understanding is that this issue happens when some package has LLVM components compiled into it statically, and then also tries to dynamically link against libLLVM.so, resulting in an error about double-registering command line options.

It looks like we are hitting a similar issue to the issue in https://github.com/conda-forge/cppyy-cling-feedstock/pull/56. I've added a comment here that explains why. In part the way llvm is built is "broken" as when it is built in mixed mode (dynamic libraries and static libraries) when we choose to link static libraries the llvm's cmake automatically appends libLLVM.so.

I saw that xeus-cpp runs against later LLVM versions, so presumably solved this problem somehow.

There is a hacky way to "fix" the broken package used in CppInterOp here: https://github.com/compiler-research/CppInterOp/pull/165

(I was also wondering, should I be thinking about switching my efforts from the old cling + xeus-cling to CppInterop + xeus-cpp? The latter is better supported and is the approach of the future, right?)

The idea of this work is to supersede xeus-cling and we are getting there. It'd be good to start shipping them, too...

I believe you need to use that flag when building llvm.

I got it to work by passing enableSharedLibraries = false when building Cling. No change was need to xeus-cling. Commit here: https://github.com/codedownio/nixpkgs/commit/9fcd1d95e3df6b2cd75bcba0923c8dd4266a28ce.
However, this requires a whole rebuild of LLVM, which we probably don't want to do in Nixpkgs. So now I'm looking for another way to accomplish the same thing.
I tried making a copy of llvmPackages_13.llvm.lib and manually deleting the relevant .so files, but that didn't work -- the build found them anyway.
Perhaps I can tweak Cling's CMake configs as done in https://github.com/compiler-research/CppInterOp/pull/165, although I'm scared to add so much CMake code.
I was also looking at https://github.com/root-project/cling/commit/d73fdd2198e40f7978899277eee599392d393ea6, and wondering if that commit might improve matters. Any chance a new release of Cling might be coming soon that would include this?

I’d claim that the way the llvm package is built is incorrect for the static case. Rebuilding it is probably the right way to address this sets of issues not only for cling but other projects.

@thomasjm can you please guide me how I can use your version of cling until we don't have a solution?
I appreciate if you can share a flake.nix file here

@rodfersou you can just use the commit I mentioned: https://github.com/codedownio/nixpkgs/commit/9fcd1d95e3df6b2cd75bcba0923c8dd4266a28ce
I.e. if you want it as a flake, just point the Nixpkgs input at it like this:
inputs.nixpkgs.url = "github:codedownio/nixpkgs/9fcd1d95e3df6b2cd75bcba0923c8dd4266a28ce";

@thomasjm I'm using your version, but things are still broken here..
/nix/store/m6zw8s1l3a0i19idb8g8ygsxbiwq2q71-libcxx-16.0.6-dev/include/c++/v1/cstdlib:103:1: note: using declaration annotated with 'using_if_exists' here
using ::size_t _LIBCPP_USING_IF_EXISTS;
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
Replaced symbol at_quick_exit cannot be found in JIT!
Error: signal 11:
0   xcpp                                0x0000000104dd23d0 _Z7handleri + 48
1   libsystem_platform.dylib            0x000000018df23584 _sigtramp + 56
2   libxeus-cling.0.15.3.dylib          0x000000010f8dad68 _ZN5clang11DeclContext10removeDeclEPNS_4DeclE + 568
3   libxeus-cling.0.15.3.dylib          0x000000010f8dad68 _ZN5clang11DeclContext10removeDeclEPNS_4DeclE + 568
4   libxeus-cling.0.15.3.dylib          0x000000010c691550 _ZN5cling12DeclUnloader9VisitDeclEPN5clang4DeclE + 256
5   libxeus-cling.0.15.3.dylib          0x000000010c691588 _ZN5cling12DeclUnloader14VisitNamedDeclEPN5clang9NamedDeclE + 28
6   libxeus-cling.0.15.3.dylib          0x000000010c70d9c0 _ZN5cling19TransactionUnloader18unloadDeclarationsEPNS_11TransactionERNS_12DeclUnloaderE + 292
7   libxeus-cling.0.15.3.dylib          0x000000010c70dcec _ZN5cling19TransactionUnloader17RevertTransactionEPNS_11TransactionE + 448
8   libxeus-cling.0.15.3.dylib          0x000000010c6de3a4 _ZN5cling11Interpreter6unloadERNS_11TransactionE + 240
9   libxeus-cling.0.15.3.dylib          0x000000010c6d23a8 _ZN5cling17IncrementalParser17commitTransactionERN4llvm14PointerIntPairIPNS_11TransactionELj2ENS0_12EParseResultENS1_21PointerLikeTypeTraitsIS4_EENS1_18PointerIntPairInfoIS4_Lj2ES7_EEEEb + 976

oh.. maybe I should add the unwrapped version

sorry for the noise, I just figured out how to use @thomasjm version

thanks again


Should imported Icons be removed from the core package?

After a quick glance, it looks like the following components import the current Icon component:

Hug

CloseButton

IconButton

Select

Radio

Checkbox

Banner

FormField

The FormField will not require the Icon import with the changes in #361

Because the CloseButton, Select, Radio, and Checkbox components import specific icons, they do not require the entire icon library, and can be updated to import the icons individually.

The IconButton component, could be included in the new pcln-icons package, instead of the core design system package

The Hug and Banner component APIs could be updated to be more composable and not import the Icon component directly

Since completely removing the Icon component would require updates to all these components and increase the scope for migration, I'd suggest postponing the complete removal until after v2 is out and slating the change for a smaller v3 to follow later, but open to hearing thoughts from the team

Resolved by v2 release.