how to get the upcoming movies for a given country.

Hi
Thanks a lot for this great api. I am struggling to get a list of upcoming movies for india. I tried changed the default language and default country properties, but nothing works.
help will be appreciated.
regards
Parminder

Hi,
According to the API docs for Upcoming (http://docs.themoviedb.apiary.io/#reference/movies/movieupcoming), you cannot submit Country. So you will get movies for all countries (but their names will be in your chosen language).
I believe this is partly because the data is either non-existent, or a given movie can be "upcoming" in many countries at once. You can ask in the TMDb forums for more details.
What you could do instead, is fetch each upcoming movie and inspect its country property(ies) to filter for Inidia.
Mike.

This Feature is now implememted into the API via the region parameter: https://developers.themoviedb.org/3/movies/get-upcoming
Adding this should not be too hard :)

@hendrikbl actually, they explicitly state: "We do not yet support regional queries but this is coming soon." :)
If they do suddenly implement it - poke us again.

@LordMike Oh okay, sorry for that. The regions page said "Region supoprt is finally here!" so I thought they implemented it. And the parameter was listed so I did not read the rest of the page.
Thank you anyways :)

Sounds funny, but they suddenly implemented it, so heres my poke: 👈 😄

Hi @LordMike,  I've just started building an application with TMDb integration and your library is making life very easy so thanks!  Is the region support for Showing/Upcoming movies on the road-map or do I need to roll my own methods for the time being?

If possible, you could make a PR for this library?
It should be relatively straightforward to add a new (optional) option to the upcoming API/methods. (remember to extend one or more unit tests :))

fixed in #issue-146 @LordMike can you review and merge?

@orangefrog the code contributed by @Naliath should cover your use case - could you try it out by referencing the TMDbLib project directly?
If all is well, I'll have a new version pushed to Nuget.


Dockerfile changes

Issue: Not able to build docker image on macbook m2.
Fix: Changed Java Image to eclipse-temurin.

merging


[Bug] When to fix the problem that the pdf cannot be exported?

Is there an existing issue for this?

[X] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (https://rxresu.me)
Current Behavior
the PDF export is not working.
Expected Behavior
No response
Steps To Reproduce
No response
What browsers are you seeing the problem on?
No response
What template are you using?
None
Anything else?
No response

I found a solution:

Open Sharing: toggle Public On, copy URL, open in new tab
Print to PDF: click Three-dot menu on top right, click Print, choose Save as PDF, Save. (Ctrl/Command + P)
Close Sharing: toggle Public Off !!!

PS: If we had a guide to a private preview page, it would be easy to indirectly solve this problem of exporting PDF and significantly reduce resource consumption.


Remove superfluous tests for EmailAddressLocalPart

Based on the feedback in PR #472, also remove the superfluous unit tests in EmailAddressLocalPart.

Thank you for your contribution :+1:
Since this is a test-only change, there's no release associated with this pull request.


Websocket 10 raises NegotiationError for permessage-deflate

Hey,
When trying to create a connection to a websocket that doesn't support permessage-deflate, Websocket 10 raises a NegotiationError where Websocket 9.1 does not.
I haven't had the time to investigate the issue in more detail yet, but I assume that Websocket 9.1 falls back to uncompressed communication.

This sounds more like #1068 (same cause = a poorly implemented server; slightly different symptoms).

Yeah this looks like it's #1068 Thank you both for the response and sorry for creating a duplicate issue.


Version 2 of Xcode Build Task, Decrypt File Task

Adds in a large set of improvements to the Xcode build task and introduces a new Decrypt File task that uses OpenSSL to decrypt files in a project.

Hi @Chuxel, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Chuck Lantz). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Merge sdf4 -> sdf6

Includes changes from #248.
I'm trying out the workflow of merging changes forward.

How did you do it? I see there's only a single commit authored by you. When we start porting lots of changes, it would be nice to preserve some of the history.

yeah, that's a good point. I did the following:
git checkout sdf4
git checkout sdf6
git merge sdf4
# then used the following to resolve conflicts
git mergetool

I'll make a parallel PR using cherry-pick to see if that's any better

see #258 for an alternative approach

merged #258 instead


FAILED: Build did NOT complete successfully

uname -a
Linux tv3--c9d4bf90 5.15.0-52-generic #58-Ubuntu SMP Thu Oct 13 08:03:55 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
gcc --version
gcc (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
llvm-as --version
Ubuntu LLVM version 14.0.0
Optimized build.
Default target: x86_64-pc-linux-gnu
Host CPU: broadwell
clang --version
Ubuntu clang version 13.0.1-2ubuntu2.1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
ldd --version
ldd (Ubuntu GLIBC 2.35-0ubuntu3.1) 2.35
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.
bazel version
Build label: 6.0.0-pre.20220421.3
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue May 3 16:41:11 2022<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>
envoy tag: v1.24.1
bazel build
--jobs=12
--test_filter="skip-all-test"
--copt="-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1"
--cxxopt="-DENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1"
--define=ENVOY_IGNORE_GLIBCXX_USE_CXX11_ABI_ERROR=1
--cxxopt='-std=c++17'
--//bazel:http3=False
--subcommands
--verbose_failures
--sandbox_debug
--cxxopt="-Wno-type-limits"
--copt="-Wno-array-bounds"
--copt="-Wno-vla-parameter"
--copt="-Werror=range-loop-construct"
-c dbg //source/exe:envoy-static
error:
In file included from external/com_google_absl/absl/base/config.h:86,
from external/com_google_absl/absl/hash/internal/city.h:53,
from external/com_google_absl/absl/hash/internal/city.cc:22:
external/com_google_absl/absl/base/policy_checks.h:79:2: error: #error "C++ versions less than C++14 are not supported."
79 | #error "C++ versions less than C++14 are not supported."
|  ^~~~~
1671159953.759135207: src/main/tools/linux-sandbox-pid1.cc:498: wait returned pid=2, status=0x100
1671159953.759154293: src/main/tools/linux-sandbox-pid1.cc:516: child exited normally with code 1
1671159953.759500705: src/main/tools/linux-sandbox.cc:233: child exited normally with code 1
Target //source/exe:envoy-static failed to build
INFO: Elapsed time: 329.238s, Critical Path: 2.04s
INFO: 15 processes: 14 internal, 1 linux-sandbox.
FAILED: Build did NOT complete successfully

uname -a
Linux tv3--c9d4bf90 5.15.0-52-generic https://github.com/envoyproxy/envoy/pull/58-Ubuntu SMP Thu Oct 13 08:03:55 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
gcc --version
gcc (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
llvm-as --version
Ubuntu LLVM version 14.0.0
Optimized build.
Default target: x86_64-pc-linux-gnu
Host CPU: broadwell
clang --version
Ubuntu clang version 13.0.1-2ubuntu2.1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
ldd --version
ldd (Ubuntu GLIBC 2.35-0ubuntu3.1) 2.35
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.
bazel version
Build label: 6.0.0-pre.20220421.3
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue May 3 16:41:11 2022<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>
envoy tag: v1.23.1
error:
1671164080.534610671: src/main/tools/linux-sandbox-pid1.cc:481: child started with PID 2
external/grpc_httpjson_transcoding/src/path_matcher_node.cc: In member function 'void google::grpc::transcoding::PathMatcherNode::LookupPath(std::vector<std::__cxx11::basic_string >::const_iterator, std::vector<std::__cxx11::basic_string >::const_iterator, google::grpc::transcoding::HttpMethod, google::grpc::transcoding::PathMatcherLookupResult*) const':
external/grpc_httpjson_transcoding/src/path_matcher_node.cc:171:27: error: loop variable 'child_key' of type 'const string&' {aka 'const std::__cxx11::basic_string&'} binds to a temporary constructed from type 'const char* const' [-Werror=range-loop-construct]
171 |   for (const std::string& child_key :
|                           ^~~~~~~~~
external/grpc_httpjson_transcoding/src/path_matcher_node.cc:171:27: note: use non-reference type 'const string' {aka 'const std::__cxx11::basic_string'} to make the copy explicit or 'const char* const&' to prevent copying
cc1plus: some warnings being treated as errors
1671164081.623130871: src/main/tools/linux-sandbox-pid1.cc:498: wait returned pid=2, status=0x100
1671164081.623151105: src/main/tools/linux-sandbox-pid1.cc:516: child exited normally with code 1
1671164081.623560805: src/main/tools/linux-sandbox.cc:233: child exited normally with code 1
Target //source/exe:envoy-static failed to build
INFO: Elapsed time: 3823.861s, Critical Path: 75.84s
INFO: 1131 processes: 13 internal, 1118 linux-sandbox.
FAILED: Build did NOT complete successfully

https://github.com/envoyproxy/envoy/issues/24464#issuecomment-1355469686
please also stop cc'ing various folks, people will see your comments on https://github.com/envoyproxy/envoy/issues/24464 without needing the extra pings


test/e2e: Ensure the Bundle's metadata ConfigMap gets re-created

Signed-off-by: timflannagan<EMAIL_ADDRESS>

(just dumping what I've had sitting locally since earlier this week so we're not duplicating work while I'm on PTO)


SWS/BR: Split selected items at stretch markers

"SWS/BR: Split selected items at stretch markers" skips some stretch markers, particularly if multiple items are selected.

I can't reproduce this... any chance you have an RPP (no media needed) that shows this?

Thanks Tim! :)

I think Pipelineaudio and I just ran into this issue. None of the stretch markers after bar 42.3.00 in the attached project are split. Selecting the leftover item and running the action splits some of it, but continues leaving smaller and smaller portions intact all the way to the splitting the last marker by itself.

RPP: https://www.dropbox.com/s/rg7mw5isegkanjg/split stretch issue.rpp?dl=1


Implementation of decoder for 'C' instructions

Hello, I tried to add some check for supported base type. This is to understand whether the instruction, for example, is supported by RV32C or not. I want to know if you accept this idea of mine or not. Also I add entries for c_addiw and compressed load/store of 64-bit values.

Please merge your branch with the latest master to start CI procedure.

1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:30: FAILED:
1:   CHECK( RISCVInstr<uint32> (0xb001).get_disasm() == "c_j -2048" )
1: with expansion:
1:   "c_j, -2048" == "c_j -2048"

OK, I agree we need an additional print type for that. Could you please call it RELATIVE_JUMP?
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:31: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x2ffd).get_disasm() == "c_jal 2046" )
1: with expansion:
1:   "c_jal, 2046" == "c_jal 2046"

Same here
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:37: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x6405).get_disasm() == "c_lui $s0, 1" )
1: with expansion:
1:   "c_lui $s0, 0x1" == "c_lui $s0, 1"

The test is not accurate here — we're using hexadecimal prints for LUI of MIPS and RISC-V.
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:38: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x647d).get_disasm() == "c_lui $s0, 31" )
1: with expansion:
1:   "c_lui $s0, 0x1f" == "c_lui $s0, 31"

Same here
1: -------------------------------------------------------------------------------
1: RISCV invalid instruction
1: -------------------------------------------------------------------------------
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:61
1: ...............................................................................
1:
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:63: FAILED:
1:   CHECK( RISCVInstr<uint32>(0x0).get_disasm() == "unknown" )
1: with expansion:
1:   "c_addi4spn $s0, $sp, 0" == "unknown"

That's wrong. In RISC-V ISA, page 79, is stated that:

A 16-bit instruction with all bits zero is permanently reserved as an illegal instruction.

Please pay attention that builds with Clang and MS lead to segmentation faults and weird outputs.

Almost perfect.
These functions are not covered by unit tests:
constexpr uint32 get_C_SQSP_immediate() const noexcept;
constexpr uint32 get_C_LQSP_immediate() const noexcept;

Additionally, your code to check instruction by ISA bit width is not called for RV64 and RV128 cases. Could you please mark some of proper instructions with these identifiers?

File "riscv-opcodes/opcodes-rvc-pseudo" should contain a field for c_lqsp, but I don't have permission to edit this file. That's why I temporary add instr_c_lqsp in "riscv_instr.cpp". What do you think?

File "riscv-opcodes/opcodes-rvc-pseudo" should contain a field for c_lqsp, but I don't have permission to edit this file. That's why I temporary add instr_c_lqsp in "riscv_instr.cpp". What do you think?

Open a pull request to their repository: https://github.com/riscv/riscv-opcodes/blob/master/opcodes-rvc-pseudo. That's a good opportunity to contribute to RISC-V community!

(however, please keep the temporal solution for now)


adding gradient checkpoint

Hi there,
thank you for your outstanding work!
i'm trying to replicate the result, but the best i get is 0.382 with seeds 2021, batch_size 32 on two 80G a100 (total 64) on eth1 512_96. I saw previous answer mentioned better try 8*a100, but it's impossible for me.
May I ask whether I should keep make my batch size much more bigger? Or do you have any recommend seeds?
I tried different version of llama, and its influence is not significant. So now, i'm working on add gradient checkpoint to reduce ram, but seems it's not helpful. Do you have insight to help me.
thank you!

not helpful on reduce ram.
maybe i add wrong place? since the model parameters is frozen, so do i should add on FlattenHead?

not helpful on reduce ram.
maybe i add wrong place? since the model parameters is frozen, so do i should add on FlattenHead?

Hi, you could try adjusting the batch_size to 1 for a test, or consider switching the base model to gpt2 or bert, as both can effectively reduce the memory consumption.


Suggestion

Hey @kkyr, thanks a lot for this script! I'm very newbie on VPS topics so it helped me a lot.
I have a suggestion which can be also very silly one :D
I'm using my raspberry pi as a VPS and I'm using cloudflare tunnel for opening it to the public. Is it possible for you to add some controls or checks for this kind of 3rd parties (especially cloudflare tunnel 😄) to make sure it opens for public properly; I'm not sure whether it's possible technically but I just wanted to ask.
Thanks!

Hey, thanks for the suggestion.
Can you elaborate on what it means opening for public properly? Trying to understand exactly what the check you'd like to see is.

Thanks for fast reply @kkyr!
So basically I run your script on my VPS (aka Raspberry Pi) and fixed the issues related to that. But I'm using cloudflare tunnel for publishing my VPS to the public so I'm wondering is it possible to check this cloudflare part via script as well? As I said, maybe I'm expecting too much - sorry about that, I'm newbie about these topics
I also created basic diagram, I hope it gives better view

No problem, just trying to understand! Which part is it that you want covered: that a VPS is accessible only via Cloudflare Tunnel, or that a specific component of Cloudflare Tunnel is not misconfigured?


Obsolete property in JsonSerializerSettings

Bud Description
ReferenceResolver property in JsonSerializerSettings is obsolete.
Exception or Stack Trace
Customization\JsonSerializerExtensions.cs(34,17): warning CS0618: 'JsonSerializerSettings.ReferenceResolver' is obsolete: 'ReferenceResolver property is obsolete. Use the ReferenceResolverProvider property to set the IReferenceResolver: settings.ReferenceResolverProvider = () => resolver' [C:\git\azure-sdk-for-net\src\sdks\eventgrid\DataPlane\Microsoft.Azure.EventGrid\Microsoft.Azure.EventGrid.csproj]
NB: sdk folder has changed
To Reproduce
Remove  <NoWarn>$(NoWarn);CS0618</NoWarn> from Microsoft.Azure.EventGrid.csproj then build the project.
Expected behavior
Warning is thrown. Warning is treated as error on the CI pipeline.

@chidozieononiwu do we need to remove that warning? It looks like the owners of that library are aware that the property is obsolete and chose to use it anyway. Unless we have a reason to remove that error, I would leave the project as-is.

@chidozieononiwu do we need to remove that warning? It looks like the owners of that library are aware that the property is obsolete and chose to use it anyway. Unless we have a reason to remove that error, I would leave the project as-is.

I was the one who suppressed the warning during the repo restructure. That library wasn't being pulled into the build previously so the warning wasn't showing up.

Why are we pulling it into the build now?
I think we can try and fix it using the suggestion provided in the error message but I want to understand a bit more about why we are doing it first.

Why are we pulling it into the build now?
I think we can try and fix it using the suggestion provided in the error message but I want to understand a bit more about why we are doing it first.

So, the build was using the folder name 'data-plane' to include dataplane libraries. For EventGrid that folder was named 'DataPlane' so it has not been building for a while which is why the warning seems new. I had to add it t0  to get it to build in CI since warnings are reported as errors.

Do you know if we shipped this at all? It looks like a version was released just last month. If we didn’t build it, any idea how it got built?

Do you know if we shipped this at all? It looks like a version was released just last month. If we didn’t build it, any idea how it got built?

As far as I can tell since this change #5133 some of the client libraries have not been building in the CI and batch is one of them. cc @weshaggard

Do you know if we shipped this at all? It looks like a version was released just last month. If we didn’t build it, any idea how it got built?

It was built from the management plane jenkins jobs. It didn't ship through our pipelines yet.

Ok. And is there a way to see why this warning wasn’t send there? I am hesitant to make a change to this code until we understand why we are seeing this error.
I will reach out to the service owners to see if they have more insight.

I suspect they just ignored the warning before, whereas we are a little more strict about treating warnings as errors. It is also possible that we have updated the version of Json.Net which now has the obsolete warning whereas the older version didn't.

I propose that we just don't do anything here. In our next wave of updates there will be no JSON.NET dependency and this code will just go away. Agree?


WARN http response content type header not set for a http 302 response

Describe the bug
for a http 302 response the response content-type is pointless, but salvo gives warn.
To Reproduce
Steps to reproduce the behavior:

start server
run  curl -i https://<IP_ADDRESS>:5800/efun
See the WARN

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser: curl
Version: 8.4.0

Additional context
Add any other context about the problem here.

Fixed by #616


Cleanup NetBeans license header for module maven.htmlui

Review and cleanup the license header in maven.htmlui
Oracle donated NetBeans to the Apache Software Foundation.
Your task now is to replace the old license header that has Oracle (and/or Sun Microsystems
and/or NetBeans) in it with the standard Apache Software Foundation license header.
For an empty file that supports headers you must add the Apache license header.
Note that you have to use the proper license header for Java, .properties or HTML files.
Look at the nearby files for templates and copy-paste the license header.
When done commit the changes with a commit message that starts with first line
"[NETBEANS-54] Module Review " and a message body describing your review.
See a longer explanation about what we are doing here: https://cwiki.apache.org/confluence/display/NETBEANS/List+of+Modules+to+Review
Please announce in a comment when you have started work.
Files that need to be looked at in maven.htmlui:
maven.htmlui/src/org/netbeans/modules/maven/htmlui/Bundle.properties
maven.htmlui/src/org/netbeans/modules/maven/htmlui/UIDefaults
maven.htmlui/src/org/netbeans/modules/maven/htmlui/description.html
maven.htmlui/src/org/netbeans/modules/maven/htmlui/wizard-gtk.css
maven.htmlui/src/org/netbeans/modules/maven/htmlui/wizard-mac.css
maven.htmlui/src/org/netbeans/modules/maven/htmlui/wizard-metal.css
maven.htmlui/src/org/netbeans/modules/maven/htmlui/wizard-nimbus.css
maven.htmlui/src/org/netbeans/modules/maven/htmlui/wizard-win.css

Dibs.


explore the use of a task manager for long queries

We can have long-running queries that currently block a user from doing anything else.  For example, searches on SDSS programs, without any other constraints can result in up to ~15 minutes queries.  We should explore implementing a task-manager, e.g. https://taskiq-python.github.io/ as a way to offload long-running jobs.
I think we'd want something that manages the background task or query, and does not block Valis or Zora.  Ideally we'd have a way to track the status of the job: e.g. running, completed, error, and potentially have a dashboard of jobs.  Another consideration is anonymous versus user tasks and if we need one or the other or both.
FastAPI has some support for Background Tasks  .  I'm not sure if this is enough for our purposes.   Other related packages:

Celery - https://docs.celeryq.dev/en/stable/index.html
Dramatiq - https://dramatiq.io/
Huey -  https://huey.readthedocs.io/en/latest/

I do like Taskiq as a simpler, more modern approach, but we may want to consider alternatives.

I looked a bit into this for lvmapi and ended up settling for TaskIQ, and I've been liking it quite a lot. It's really easy to set up and plays well with FastAPI. It may have fewer bells than Celery but I actually consider that an advantage.
I looked a bit into the other options (except Dramatiq, I think). Background Tasks sounded promising but I don't think there's a way to track task completion (you just launch something in the background and then lose track of it). It doesn't seem different from just having a pool of asyncio tasks, and in the end one would need to build something to track them. I'm also not sure how well it works with the uvicorn gunicorn worker.
Celery has everything one would need but it simply does not support async and all the hacks I tried failed dramatically. Maybe that's a bit less of a blocker for valis since currently all the DB connections are, in practice, sync. But that's a future limitation and it may not work well with FastAPI.
Huey's implementation of asyncio is a bit hacky and I could not make it work properly.
For what you describe I think TaskIQ would work out of the box with the exception of tracking all running tasks. But I think that's easy to implement with a Middleware that would record when a task starts (for example to a table in sdss5db, or to a Redis DB if we use that as the results backend) and when it finishes. Then it's easy to query that to get a complete picture of what tasks are running and where.
Another things that we should consider is how this and asynchronous DB connections are related. If one uses peewee in a worker, and the query takes 15 minutes, that worker won't be able to do anything else for 15 minutes even if all the action at that point is happening in the Postgresql server. As long as the workers are light one can imagine spinning 100 workers (and maybe using the knowledge of how many queries are running to throttle the start of new queries).
But if the database connection is async (with SQLAlchemy 2 or with some hack that would allow peewee to use psycopg 3 async support) then a few workers could handle all the tasks. At that point TaskIQ is not really doing any work other than tracking query completion. I'm actually not sure if TaskIQ allows workers to run concurrent tasks or if a single worker can only run one task at a time.


Update ContributedValues model

Description
This PR updates Dataset's ContributedValues model for the 0.12 release.
@doaa-altarawy Is this enough for you to go on when implementing the contributed values table as we discussed today? Let me know if there is any more information or clarification I can provide.

Please put back the doi, then will merge.

Now part of #460.


Konnectivity server Service doesn't work on Metal LB

When enabling Konnectivity for a Tenant Control Plane in an environment backed by Metal LB as a solution to expose Load Balancer services, the resulting Service is never exposed due to the following error.
$: kubectl describe svc test-konnectivity-service
Name:                     test-konnectivity-service
Namespace:                default
Labels:                   <none>
Annotations:              <none>
Selector:                 kamaji.clastix.io/soot=test
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       <IP_ADDRESS>
IPs:                      <IP_ADDRESS>
IP:                       <IP_ADDRESS>
Port:                     <unset>  8134/TCP
TargetPort:               8134/TCP
NodePort:                 <unset>  30449/TCP
Endpoints:                <IP_ADDRESS>:8134
Session Affinity:         None
External Traffic Policy:  Cluster
Events:
  Type     Reason            Age   From                Message
  ----     ------            ----  ----                -------
  Warning  AllocationFailed  59m   metallb-controller  Failed to allocate IP for "default/test-konnectivity-service": can't change sharing key for "default/test-konnectivity-service", address also in use by default/test

As a workaround, we should expose the Konnectivity Server in the same Service used by the Tenant Control Plane, since it's pointing to the same subset of Endpoints, although to a different port.

Hey @prometherion, what the default/test Service represents in that scenario?

It's the Tenant Control Plane Service that exposes the Kubernetes API Server.


cv.imread() overload in types definition

cv.imread() should be able to accept either an id string or a reference to a canvas or image element. The element reference overload seems absent in the types definition.

Thanks for pointing it out, still actively working on this so if you come past anything feel free to open more issues. Updated here

Thanks for the quick response, good job!


controller/client: Fix RunJobAttached.

Oops.
Signed-off-by: Blaž Hrastnik<EMAIL_ADDRESS>

LGTM

The commit message could be more descriptive.


Azure SDK Review - [Introduction to ACS WebUI]

New SDK Review meeting has been requested.
Service Name: ACS WebUI
Review Created By: Prabhjot Kaur
Review Date: 05/04/2023 02:05 PM PT
Hero Scenarios Link: here
Architecture Diagram Link: Not Provided
Core Concepts Doc Link: here
APIView Links:
Description: Azure Communication Services UI Library is adding support for Blurred and Custom Backgrounds to its existing set of controls including Composites and UI Components. With this new feature, users can blur the local video background and replace the background with a provided set of custom images.
Detailed meeting information and documents provided can be accessed here

Cancelled by: Prabhjot Kaur


ClassCastExpection with WirelessChargers

Issue Description:
I had 3 Wireless Chargers and when I would break the item it would cause a Internal server error for everyone on the server.  When you logged back in the Charger would be gone but if you placed a block where the Charger was it would kick you out with the same error and same trace. Restarting the server will fix the problem and allow me to place a block where the charger was. I am trying to find a way to reproduce it but cannot. I will reply back if I do find a way.
Thank you guys.
Modpack: Direwolf20 1.12.2 - version 2.2.0 (Release)

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.12.2-5.0.29
EnderCore: EnderCore-1.12.2-0.5.35
Minecraft: 1.12.2
Forge: 2739
SpongeForge? Yes
Optifine? No
Single Player and/or Server? Server

Your most recent log file where the issue was present:
https://pastebin.com/raw/f384cY5J

Henry, thank you for looking into this. If you need any help or any additional information let me know.

I have this exact same problem, on my dedicated server apparently someone did the same thing. My stack trace is the same. Here's a full server crash report for you to inspect. http://paste.ubuntu.com/p/s5n5QZDHQH/

This is also causing a nasty mem-leak. You seem to have pushed a fix for this as-well
Thank you, Henry.

Same issue here, interesting to know it is the cause of a memory leak as I'm having the same issue with the Foolcraft 3 pack using more than it should after a while.
This has been the cause of numerous server crashes too.
Crash report: https://pastebin.com/zEn6xVYZ

This is still happening on EnderIO-1.12.2-5.0.31 and EnderCore-1.12.2-0.5.37

Yeap, you are right that's in build 32 not 31.


⚠ Bofrost.lu has degraded performance

In d75d709, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 837 ms

Resolved: Bofrost.lu performance has improved in 8b365b2 after 6 minutes.


⚠ Processors has degraded performance

In cd7e5e1, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 86 ms

Resolved: Processors performance has improved in 412c4fd.


Why don't we have import_dataset.sh inside pyiron?

https://github.com/pyiron/pyiron-publication-template/blob/f93ceddad4cc23c7858e16300126a96eff2bf2dd/.github/ci_support/import_dataset.sh#L1-L13
I think this functionality is useful in a general way. Why don't we have it inside the standard pyiron package?

Ah sorry I just saw that pack and unpack were solidly available inside pyiron XD. I guess it's a super nice feature that we should promote more.

Yes, the rest is just some magic to copy any file no matter the file name.


feat: implement WithBrowserPaginationComponent

fix: #809
Changes proposed in this PR:

Implement WithBrowserPaginationComponent

[ ] I have followed (at least) the PR section of the contributing guide.
@90milesbridge/tigger

Pull Request Test Coverage Report for Build 3375

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.004%

Totals

Change from base Build 3368:
0.0%

Covered Lines:
2795

Relevant Lines:
3378

💛  - Coveralls


Revert "Fix working_dir negative test error expectation (#1348)"

This reverts commit 2028de8b8d5e0516e0e65664f9085dab02a6a5e2.

I think I messed up my local testing and was using gcs with an older version of security policy when doing the fix: https://github.com/microsoft/hcsshim/pull/1322. As you can see, the quotations aren't there. However, I later merged a PR, which added them: https://github.com/microsoft/hcsshim/pull/1311.


DOCUMENTATION: Readme Cleanup Add Nuget Refs

Closes #133
@hassanhabib @ElbekDeveloper
Took some time today and reviewed Nuget packages that are using images. All seem to be using logo graphic some with an incorporated package name and others not.
In conclusion Elbek is correct markdown syntax does not support centering, and all that found in nuget were left aligned. This PR incorporates the logo in markdown, nuget package links, and the latest videos.

@glhays I see some tags for nuget that shows even the amount of downloads on a given nuget package.
Can you find out how to do that?

Sure no problem.
Also can you take a look at my comment Elbek's issue #138 I know that Christo said the  tag is not allowed in markdown which is true and false. As long as it is separated from markdown syntax with blank lines it is, just not embedded within the markdown.
My honest opinion is and I remember other parsing engines not liking div tags and they just spit them out as text and everything between them.
I am when a free moment arises going to mock a library up and do some test publishing to nuget and see what may work.

@hassanhabib downloads badge added.


How to run portainer without auth?

Question:
Hi, How to run portainer without authorization?
Or with long-long session TTL.
Just for local development, not for production :)
I'm running portainer as docker-container.

You need to use the --no-auth flag when deploying Portainer, have a look at https://portainer.readthedocs.io/en/stable/configuration.html#disable-authentication

This seems to have been relevant in the past, but I'm unable to find this option in the docs now. That link just goes to the main configuration page. Is this option still available?

Its still available, however we do not recommend it, so have removed information about it from our documentation.
We were subjected to a CVE regarding having this option as it seen as very insecure (as many users disabled auth and still put Portainer on the internet).

Ah, I understand. I'm using Portainer as a replacement for Kitematic, as Kitematic sucks pretty hard, so I really like not needing auth. Thanks!

It can also be useful when we want to use our own identity provider like keycloak gatekeeper.

It can also be useful when we want to use our own identity provider like keycloak gatekeeper.

Agreed, I am just setting up Authelia for SSO and either disabling internal auth or trusting the forwarded REMOTE_USER header is very helpful in these situations.

It can also be useful when we want to use our own identity provider like keycloak gatekeeper.

Agreed, I am just setting up Authelia for SSO and either disabling internal auth or trusting the forwarded REMOTE_USER header is very helpful in these situations.

Same i just set up authelia and disabled portainer auth.
However this seems to remove the capability to duplicate/edit containers. Is there a way around this or are we forced to use internal authentication??

So how do I run portainer without auth?

So how do I run portainer without auth? The following command crashes
docker run -d \
  -p 8000:8000 \
  -p 9000:9000 \
  --name=portainer \
  --restart=always \
  -v /var/run/docker.sock:/var/run/docker.sock \
  -v portainer_data:/data \
  portainer/portainer-ce --no-auth

--no-auth was removed from from Portainer v2.0. It's no longer an option.
See here: https://documentation.portainer.io/v2.0/troubleshooting/deprecated/

... and it’s not coming back, they’ve explicitly said that. But setting a very long session expiration (eg 1year) is a reasonable workaround for now https://github.com/portainer/portainer/issues/2101#issuecomment-666070463

You need to use the --no-auth flag when deploying Portainer, have a look at https://portainer.readthedocs.io/en/stable/configuration.html#disable-authentication

--no-auth has been deprecated and removed


Make sure LD -L option is calculated correctly with complex SRCDIRS.

When the project Makefile has specified SRCDIRS components that are not directly below Makefile folder level, the LD -L option is not calculated correctly.
For example,
SRCDIRS = comp_a happy/comp_b /c/dev/comp_c
Then the following are built:
build/comp_a/libcomp_a.a
build/comp_b/libcomp_b.a
build/comp_c/libcomp_c.a

But when LD is run, the -L is calculated as follows
-L build/comp_a
-L build/happy/comp_b
-L build//c/dev/comp_c

This means libcomp_b.a and libcomp_c.a are not found by LD. With this change set, -L is calculated correctly for comp_b and comp_c

 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.

Thanks @eyob-boye . This looks good to me, I've pushed it into our internal review & merge queue.

Cherry-picked as cdc7396, thanks!


Curl downloads result in long build times and 500 errors are not retried

Certain RPM mirrors misbehave and cause very slow downloads that causes curl to get stuck for a very long time. This can sometimes cause an image build to take up to 10-20+ minutes (when it would otherwise take about 2-3 minutes).
Also, if a mirror throws a 503 error (which Fedora mirrors sometimes do during a sync), curl will not retry. A single failed RPM download can cause the entire build to fail.
It would be great if we could find a way to use multiple mirrors for downloads or some more reliable method for downloading them. There's aria2, but it's not currently in RHEL. There's also the multithread python module as well.

We also have issues with timeouts:
Traceback (most recent call last):
  File "/run/osbuild/lib/stages/org.osbuild.rpm", line 161, in <module>
    r = main(args["tree"], args["sources"], args["options"])
  File "/run/osbuild/lib/stages/org.osbuild.rpm", line 101, in main
    osbuild.sources.get("org.osbuild.files", checksums)
  File "/run/osbuild/lib/osbuild/sources.py", line 87, in get
    raise RuntimeError(f"{source}: " + reply["error"])
RuntimeError: org.osbuild.files: curl: error downloading {'url': 'https://mirror.arizona.edu/fedora/linux/updates/32/Everything/x86_64/Packages/l/linux-firmware-20200619-109.fc32.noarch.rpm'}: error code 28

I think this relates to https://github.com/osbuild/osbuild/issues/882


Push and pull should return immediately, along with session info for checking completion status

Probably optionally.  Check kubo commands for similar CLI options related to showing progress or running in background.

Looking in ipfs for comparison, ipfs pin remote ... has a background option.  This seems like the closest match to what we might add to carmirror to signal that the command should return right away with the session id and continue running in the background.
Currently carmirror isn't parsing options, just assuming they occur at specific indexes in the args.  I'm going to explore using ipfs/go-ipfs-cmds for the parsing, as it positions us better to incorporate the commands into kubo's cli later.

Background option has been added.  Currently go-car-mirror client sessions use URL as the session id, so update this to return the URL and this issue is done.


Allow passing namespace for MatchPeerCertificatesFromSecret

In Eventing, we need to use the system namespace rather than the test namespace.

/cc @creydr

/cherry-pick release-1.11

/cc @Cali0707 @Leo6Leo

Thank you!


ISPN-12617 Transport Security

https://issues.redhat.com/browse/ISPN-12617

@pruivo updated with the JGroups 4.2.17.Final release

integrated! thanks @tristantarrant !


[BUG] APP Jawline Exercises (KamoHacks)crashes

Describe the bug
When the Jawline Exercises app is selected and run, LiveContainer crashes and when opened the LiveContainer menu is shown.
Instructions to reproduce
Open LiveContainer app select Jawline Excersises and press run.
What version of LiveContainer are you using?
2.0
Other info
🤫🧏♂

Should work in latest build now.


Lowercase column headers

This PR lowercases all column headers. The changes contribute to #81 and will enable using of #39

@siddharth-krishna, the idea this one is to use lowercase for column headears, but stick to uppercase for column values. Initially I was thinking to uppercase everything, but I find it easier to seperate table dimensions from values this way.

Please feel free to make any improvement directly. I am happy I've managed to get it to pass the tests! 😆

Thanks Sid, good point! Let's open an issue. :-)


Updated link to bleach documentation

Fixed broken link to bleach documentation

Thanks @joopeed but this change conflicts with another PR.
Could you please merge master to resolve & then I'll take another look 👍

Done @marksweb


Google API Not working?

Not a coder, but I appreciate this plugin and would like to use it better, but for some reason, my API key doesn't seem to work with the plugin. Not very familiar with APIs, and regex, so would appreciate any help with this.
Is there some detailed procedure that I'm currently missing? How exactly do I use my API Key for Google Maps geocoding? Thanks.

Sorry for being unable to help on a timely manner, did you close this because you managed to fix it?


Convert css to sass file

Would it be possible to translate the animation.css to sass. Then we would be able to chose only style we need.
For example, in my case, I only use fadeIn style, but still the css file is 55ko.

We don't have an internal css file and I think it would be possible to replace a sass version of animate.css. I'm marking this as a enhancement.

sorry for the delay @geosenna. we don't have any css files and we are just using the animate.css for animations. Googled and found this one: https://github.com/geoffgraham/animate.scss
Hope this helps.


Fix commit icon position

Before:

After:

@lafriks please backport :)

sorry, this does not need backporting, this is new bug just in master :)


The behaviour of FIFO queues does not correspond with AWS specifications

Reporting Bugs
The test code reported in issue #1655 does not correspond with the behaviour of AWS SQS and the specifications of boto3.
As the boto3 documentation says:

When you set this attribute (FifoQueue), you must also provide the MessageGroupId for your messages explicitly. For more information, see FIFO Queue Logic in the Amazon Simple Queue Service Developer Guide.

This is currently not enforced in moto.
Furthermore, as the AWS says:

When receiving messages from a FIFO queue with multiple message group IDs, Amazon SQS first attempts to return as many messages with the same message group ID as possible. This allows other consumers to process messages with a different message group ID.

Currently only one message for each message group ID is returned and the next one is not returned until the first one is deleted.
Currently:

It is possible to call send_message for a FIFO queue without specifying MessageGroupID
When calling receive_messages() on a queue with multiple messages with the same Message Group ID only one message is returned and the next one is not returned until the first one is deleted.

If I am not mistaken the specification should be something like this:
@mock_sqs
def test_receive_messages_with_message_group_id():
    sqs = boto3.resource('sqs', region_name='us-east-1')
    queue = sqs.create_queue(QueueName="blah.fifo", Attributes={'FifoQueue': 'true', 'ContentBasedDeduplication': 'true'})
    queue.send_message(
        MessageBody="message-1",
        MessageGroupId="group"
    )
    queue.send_message(
        MessageBody="message-2",
        MessageGroupId="group"
    )

    messages = queue.receive_messages(MaxNumberOfMessages=10)
    messages.should.have.length_of(2)

@mock_sqs
def test_send_messages_to_fifo_without_message_group_id():
    sqs = boto3.resource('sqs', region_name='us-east-1')
    queue = sqs.create_queue(QueueName="blah.fifo", Attributes={'FifoQueue': 'true', 'ContentBasedDeduplication': 'true'})

    with self.assertRaises(Exception):
        queue.send_message(
            MessageBody="message-1"
        )

Thanks; going to set this as a feature request to implement the entire spec for this.

This is now fixed in Moto >= 1.3.15.dev874


fix: appropriate whitespace to misaligned elements.

Fixes Issue
My PR closes #515
👨💻 Changes proposed(What did you do ?)

Added gap in-between 'Preview' heading and thumb fixing the overlapping issue
Added whitespace between 'Track color' , 'Thumb Color' and the input fields
Added whitespace between the heading and the input in thumb size selection section

✔ Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] This PR does not contain plagiarized content.
[X] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

:tada: This PR is included in version 2.72.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Hey @Dun-sin I noticed that in mobile view the Include Thumb Radius section breaks
Current Behavior:-

Expected Behavior:-

Should I create a new issue and submit the PR??


Replace QGIS 3.38 docs links with QGIS testing

This is the proposed fix #330

Replace QGIS 3.38 with QGIS testing
Update some of the links the links to /testing/: all links related to docs.qgis.org
Some links still use the latest version in the link as they don't have the testing version available: https://qgis.org/api/3.38/, https://qgis.org/pyqgis/3.38/, https://github.com/qgis/QGIS/blob/release-3_38/INSTALL.md
Remove language changing feature for all links in QGIS testing as they don't have translated versions

Why testing instead of latest?

@jef-n I think testing refers to all versions greater than the current LTR as mentioned by @DelazJ in #330. Also, I've checked on the docs website (https://docs.qgis.org/latest/en/docs/user_manual/index.html for example) and noticed the latest version is redirected to 3.34.

Maybe better "QGIS testing (>3.34)" as in the old website https://web.archive.org/web/20240305010548/https://www.qgis.org/en/docs/index.html

Thanks @agiudiceandrea , I've applied the change


Replaced tacotime with binaryFate on the team page

Maybe move tacotime to the Special Thanks section in his honor?

Done @stoffu


Sounds

KF uses VAB and SEQ files for music and sound effects. It would be nice if we were able to to edit these.

This is a pretty sweet project and I'll be watching it eagerly :)
I happened across it when I was trying to work out how to extract the samples and music from the KF games for remixing purposes. I managed to get the sound effects and samples using PSound which is nice but I didn't work out how to access the actual music tracks yet. What I really want is the crazy dialogue from the ending of KF3 but I'm sure it's baked into the FMV somehow (which I'm also not sure how to extract).
Anyway, just mentioning it in case someone knows or it's a feature that's likely to get added here one day. I'll for sure be giving your tools a go when I get a chance.

Audio modification isn't currently a high priority amongst all the other cool stuff. We do have some framework for it but I don't see it happening soon.
You can use jPSXdec to get the audio from a PS1 movie (.s files on the KF2j/KF3j discs) but the music and VA is mixed so you'll have to use an AI to seperate them. PSSound is still best for sound effects right now.

Oh that's a really useful tip, thanks! This tool works really well, and turns out the japanese audio doesn't have the voice over at all so maybe I can just subtract it to get pretty close to a voice-only track :D ❤


Onboarding: token settings

BYOT
1. Address of the existing token (e.g. `TKN`,`Token Name`)
2. New Garden token. Add a help bubble with the following text:  `We recommend sticking with the default syntax here unless you have a good reason not to`
    1. Symbol (the existing token symbol appears by default with a lowercase g in front: `gTKN`)
    2. Name (the name of the existing token appears by default with the word Garden in front: `Garden Token Name`)

Native

Symbol
Name
List of initial holders (seeds) specifying address and quantity for each one.
Initial amount of tokens in the common pool

Add a help bubble with the following text: This is the shared resource of the community which will be managed by the token holders using Conviction Voting

I would remove "Initial amount of tokens in the common pool" since it should match issuance.maxRatio * token.totalSupply since the beginning.

I am not sure if I fully understood. Does it means that given a list of seed holder we can derive the Common Pool initial balance with: issuance.maxRatio * token.totalSupply where totalSupply is the sum of the balances of the list of seed holders?

The complete formula would be:
targetRatio = commonPoolAmount / initialTotalSupply = commonPoolAmout / (seedsAmount + commonPoolAmount)

If we solve for commonPool we obtain the following formula:
commonPoolAmount = seedsAmount * targetRatio / (1 - targetRatio)

We can sum up all initial tokenholders to obtain seedsAmount and obtain targetRatio from the issuance configuration. targetRatio can not be set to one and we should disable the next button in that case, but that's part of the issuance policy screen.

Make sense. I probably  not fully understand how issuance works. But I wonder if there are use cases where a community would like to set up a custom balance for the Common Pool?

Yes, actually that's the case of 1hive. We started being few seeds, and wanted to grow quick at the beginning, so we set up a high inflation to delude initial token holdings.
So maybe a community wants to start with a lot of GDN in their common pool to distribute fast using conviction voting, but they want that over time, the relation commonPool/totalSupply to reduce.
For this case, I would set up an new slider in the issuance policy screen (maybe in advanced?) called initialRatio, so people can decide how to do the initial distribution of the token.

For this case, I would set up an new slider in the issuance policy screen (maybe in advanced?) called initialRatio, so people can decide how to do the initial distribution of the token.

I agree. Most likely we want to support this on the issuance app.


Update UUIDs of Services

Process for updating the UUIDs of Services

[x] Thorsten to check categories of OperationServers and OperationClients in ServiceList
[x] Iswaryaa to review categories of OperationServers and OperationClients in ServiceList
[ ] Katharina to add new UUIDs based on category of operation (but not old UUID) in ServiceList
[ ] Iswaryaa to review new UUIDs in ServiceList
[ ] Katharina to CTRL+h for old UUIDs and replace them by new UUIDs in ForwardingList, CONFIGfile and OAS.
[ ] Katharina to step through ForwardingList, CONFIGfile and OAS and compare every UUID against new one in ServiceList
[ ] Iswaryaa to review new UUIDs in ForwardingList, CONFIGfile and OAS
If no errors found
[ ] Iswaryaa to delete old UUIDs in ServiceList
[ ] Iswaryaa to merge the feature branch into develop
[ ] Iswaryaa to close this issue and delete the feature branch on GitHub

Transfer of service-records from OldRelease to NewRelease has been removed in v1.0.0. This implies that in v1.0.0, once NewRelease comes into architecture, it will start with a fresh record list instead of continuing with service-records from old-application. Kindly correct me if my understanding is wrong

Starting with 2.0.0 there shall be no handover of application data during bequeath-your-data-and-die anymore.
(The information for connecting to the DATAfile or database shall be passed instead.)
I made a mistake by deleting the NewRelease://v1/record-service-request while updating the ServiceList by ignoring that v1.0.0_spec will be created afterwards.
It doesn't play a role, because v1.0.0 has already been implemented.
This is why I didn't change it back.


Copy and paste from subs and definitions doesn't work

This was reported via email by Nicolás Castañeda and also mentioned in issue #20. I think it used to work, perhaps there was a regression. This is especially annoying when trying to copy definitions/etc into the Anki import window.

I'm not sure if something changed in a past version, but in the current code the behavior I'm seeing is that if there is a green "special" selection of subtitle text, that text will always get copied if the user does a copy, even if they had some other selected text they meant to copy (e.g. in a definition). I am going to fix this by just not doing the special copy of green-selected subtitle text if there is any other
selection in the document.


Not compatible with net461

Tried creating new projects as described and adding the prerequisite packages only to receive compatibility errors.
Downloaded the sample source code and opened it in VS2017 only to immediately receive compatibility errors.

@newgui Is there any particular doc you're following? If you provide a link to the sample code, I can dig into this for you.

Hi Scott,  Thanks for responding.  I am following this article:  https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/windows-service?tabs=aspnetcore2x
Step 1 is to install Microsoft.AspNetCore.Hosting.WindowsServices.  When I attempt to do that, I get an error saying that it is not compatible with net461 which the prereqs indicate as required.
So, I then just downloaded the source code from https://github.com/aspnet/Docs/tree/master/aspnetcore/host-and-deploy/windows-service/sample and loaded it up in VS2017 and get a page full of compatibility errors.
I'm sure it's my inexperience here but I'm just not sure what to do next.

@newgui Thanks for the info! I'm looking into this for you.

@newgui If you don't mind, can you include the compatibility errors you're seeing?

"Install" might not be the best choice of words, since what the app will need is a package reference to Microsoft.AspNetCore.Hosting.WindowsServices.
@newgui With regard to running the sample app that accompanies the topic, do you have a recent version of VS (15.3 or later; 15.5.3 is latest) with the ASP.NET and web development and .NET Core cross-platform development workloads installed?
When we did the update on this topic, I pulled the sample down, and it built ok. Therefore, I think a tooling (VS) setup issue is to blame.
I had a slightly earlier version than the latest (15.5.3) back then, so what I'm doing now is taking my VS to 15.5.3 to confirm the sample loads, restores, and builds with latest VS. I'll ping back with the result.

I can confirm that the sample app also builds for me with VS 2017 15.5.7 and the 2 workloads specified by @guardrex.
@guardrex I'm going to create an issue to make some updates to this doc. I see a few other things that need attention.

Thanks for the reply!  I am upgrading VS now and will see how it goes...

OK... I update VS to the latest version and the sample project will load and run.
However, as indicated in the document that I am following the ASP.NET Core Application (.NET Framework) template is no longer available.  Is there another that I should use or is there a way to get that one back?

OK...  I believe I have this figured out now.   Thanks for all of your input.

I think it would be (maybe :smile:) ...
Visual C# > .NET Core > ASP.NET Core Web Application
Set 1st DDL to ".NET Framework" (leave the 2nd DDL on "ASP.NET Core 2.0") ... and select the "Empty" template.

I also noticed that the template indicated in the doc is incorrect. This will be addressed in https://github.com/aspnet/Docs/issues/5578.

Hi Scott,
I seem to be stuck again.  I have created the projects as described here:  https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/windows-service?tabs=aspnetcore2x#tabpanel_SENY+PN+lu_aspnetcore2x
When running in VS I am able to debug the project and browse to the API.
I have also used the sc command to create and start the windows service.  However, when running as a Windows Service I am unable to browse to the url.  I get either a 500 or 400 error.
Can you provide any direction?  Here is a link to my project:  https://www.dropbox.com/s/n3svj2yjtedpmgy/DocumentsApi.zip?dl=0
Thanks,
Kerry
From: Scott Addie<EMAIL_ADDRESS>Sent: Friday, March 02, 2018 10:33 AM
To: aspnet/Docs<EMAIL_ADDRESS>Cc: newgui<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [aspnet/Docs] Not compatible with net461 (#5577)
Closed #5577 https://github.com/aspnet/Docs/issues/5577 .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/aspnet/Docs/issues/5577#event-1501511829 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABGF0cOB5-ASXEvocQNG9oCkr4-zilb8ks5taXRKgaJpZM4SYc3l .  https://github.com/notifications/beacon/ABGF0ZA6Du82eQNCZrKF61ioSdtdPPzHks5taXRKgaJpZM4SYc3l.gif

@newgui I noticed that you're missing the following in DocumentsApi.csproj:
<RuntimeIdentifier>win7-x64</RuntimeIdentifier>

This line causes a Self-Contained Deployment to occur, which is necessary for this to work. Let me know what happens after you add this and try again.

@scottaddie  I just not got back to this and tried adding the win7-x64 configuration and republishing...  However, I am still getting HTTP Error 502.5 - Process Failure.

@newgui

Here is a link to my project

I'm still out-of-town right now, but I'll be back home on Sunday. If @scottaddie is busy and can't get back to you by then, I'll try to help.
I don't use Dropbox, so can you put your project up in a GitHub repo for me to pull down? You can cut out all of the logic and just leave one API endpoint in it that returns a string or something.


Added TFP Versions of Several Chapters

Added Tensorflow Probability implementations of Chapter 3, Chapter 5, and an updated version of the TFP implementation of Chapter 1.

nice, lgtm!


Just survey

Feature to query shodan and make the request to the remote servers without prompting download. i.e. it will simply make the requests parse the interesting words and alert what was found with no download option.  Option to have all output log to a file and or stdout to allow for headless deployment to a vps without user interaction.

Feature already exists with -q -s closing


Feature request: indicator of overall plan size when zoomed in

It'd be really nice to see where we are in the view when zoomed in, with a zoom % and ideally some scrollbars so that we can use keyboard buttons / the left and right scroll function on a mouse to move instead of clicking and dragging.

Hey Kevin, just published a new version that throws out the previous version's handling of scrolling and zooming to use simple scrollbars and buttons for zooming. Should make it a bit more intuitive...I hope
https://github.com/phil-scott-78/azure-data-studio-queryplan-show/releases/tag/0.0.2

Thanks for fixing!


Support SVG icons for the toolbar

Description
This change enables to use of SVG icons for the toolbar.
Motivation and Context
png images do not always look good at HiDPI screens since it requires to generate multiple copies of the image.
In practice, JMeter's toolbar does not provide 2x scaled images, so the pixels are clearly visible.
Screenshots (if appropriate):
Note the icon in the middle

ThemeBeforeAfter
IntelliJ

Solarized dark

If the svg icon doesn’t contain any colour, which needs to be resolved at runtime, there is still DarkSVGIcon. Also I think at some point the icons should be overhauled as they are too detailed to work in the scale they are used.

Also I think at some point the icons should be overhauled as they are too detailed to work in the scale they are used.

That is true. It should be something like "use this URL for icons 31px+, this URL for 17..31px, and this for 0..16px".
That, however, is a bit different story (==> different PR).

That is true. It should be something like "use this URL for icons 31px+, this URL for 17..31px, and this for 0..16px".

Or simply an icon that works in all sizes.

That, however, is a bit different story (==> different PR).

That is true :D

I just learned IntelliJ platform requires plugin logos to be in SVG format: https://www.jetbrains.org/intellij/sdk/docs/basics/plugin_structure/plugin_icon_file.html#plugin-logo-file-format


Citations (mergeable)

prefix/suffix/locator for cite
no more missing key mysterious error
update README
citations refactoring/cleanup

@ronaldtse ignore branch citations (please delete that) and merge this one instead.
Also: hound got angrier this time. I switched to the ´.rubocop.yml´ from ´oss-style´ repo, but there should be 14 offenses. There seem to be a configuration mismatch.

@paolobrasolin you're so fast 👍  Thanks!

@paolobrasolin feel free to delete that stale branch! :-)


Not able to see full error in console.

Sorry this error isn't reproducible because I am not able to figure out yet who is causing this error.
Screenshot:

Flutter Doctor:
[✓] Flutter (Channel dev, 1.23.0-7.0.pre on macOS)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] IntelliJ IDEA Community Edition (version 2020.2.2)
[✓] VS Code (version 1.49.0)
[✓] Connected device (4 available)

• No issues found!

Let me know if more information is needed.

Hi @iLoveDocs
can you please try with fluter run -v to see the full logs?
Thanks.

@Abhishek01039  Not on the machine at this moment, but I know it would work. The question was why the logs are not displayed in the console.

@iLoveDocs Because your verbose flag(-v) is not there.
Android studio and vs code doesn't provide the verbose flag by default you have to configure it.
Thanks.

@iLoveDocs
Can you purposely try to fire an exception in the log and see if you get similar logs or in full ?
Do you see all details when you run flutter run --verbose ?
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Add PreserveUnknownFields marker

Add +kubebuilder:pruning:PreserveUnknownFields processing marker to preserve initContainers, sidecarContainers and ephemeralContainers fields from being pruned.
Resolves #772

@mumoshu agreed, that is a better solution, closing this in favour of #803 then 👍

@donovanmuller Thanks a lot for your confirmation!


[SR-16108] Inconsistency between Any.Protocol and (any Any).Type

Previous ID
SR-16108

Radar
None

Original Reporter
templar (JIRA User)

Type
Bug

Environment
Xcode Version 13.3 (13E113), playground.

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug, TypeChecker

Assignee
None

Priority
Medium

md5: 4161fe58e12b7763211c3196a2aed02c

Issue Description:
According to the proposal, https://github.com/apple/swift-evolution/blob/main/proposals/0335-existential-any.md#metatypes:

P.Protocol, becomes (any P).Type

also,

The protocol metatypes for Any and AnyObject are spelled (any Any).Type and (any AnyObject).Type, respectively.

However, the following code shows the two are not the same considering Any:
Int.self is Any.Protocol // false
Int.self is (any Any).Type // true <- how come?
protocol P {}
P.self is Any.Protocol // false
P.self is (any Any).Type // true <- how come?

It seems, (any Any).Type is transformed into Any.Type by the compiler.
Also, it is illegal to write something like
let something = (any Any).Type.self // error: expected ',' separator.

A same problem exists for AnyObject.
(any AnyObject).Type.self == AnyObject.Protocol.self // false <- how come?
(any AnyObject).Type.self == AnyObject.Type.self // true <- how come?

@hborla could you please have a loot into this?

All examples behave correctly now. Except for the one with a compiler error, which I reported separately in #78042.


Improved compability check error handling.

Improved compability check error handling.

Added specific message when time out error happens.

@lu-ohai we can close this PR, the UI will handle this for now as discussed on slack.


Feature Request: Double click to confirm timeline item creation

I frequently find myself clicking on the timeline and accidentally creating a bunch of new items because of it. I would find it useful if there was a way to prevent misclicks such as requiring a double click instead.

Duplicate #341


Add Zeroize Support

Change Summary:

Adds support for the zeroize crypto domain endpoint which clears the cryptographic keys and non-compliance mode settings within the given domain (which must have an access-mode of "control-usage") of the given crypto adapter.

Endpoint details:

POST /api/partitions/{partition-id}/operations/zeroize-crypto-domain
Request:

crypto-adapter-URI; string (struct CryptoAdapterDetails)
domain-index; int (struct CryptoAdapterDetails)

@rahulbhalla3521 would you please resolve the conflict before merge?


bump release

Codecov Report

Merging #161 (f936017) into main (4213b41) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #161   +/-   ##
=======================================
  Coverage   77.79%   77.79%
=======================================
  Files          18       18
  Lines        2265     2265
=======================================
  Hits         1762     1762
  Misses        503      503

Impacted Files
Coverage Δ

preliz/__init__.py
76.47% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


popup toast doesn't appear if isPresented is .constant(true)

In this code:
            .popup(isPresented: .constant(true),
                   type: .toast,
                   position: .bottom,
                   dragToDismiss: true,
                   closeOnTap: false,
                   closeOnTapOutside: true,
                   dismissCallback: {
                   // on dismiss
                  }) {
                CardView { ... }

PopupView bases the appearAction on isPresented changing:
        .valueChanged(value: isPresented) { isPresented in
            appearAction(isPresented: isPresented)
        }

But if the binding is a .constant(true) since PopupView's inception, it doesn't appear.

Hey @ronyfadel, great catch! Thank you very much for bringing this to my attention. I'm happy to say, I fixed the issue (added onAppear call as well as on change). I added my fix to version 1.0.4, you are most welcome to give it a go. Have a marvellous day!

Hello @ronyfadel, it's been a while so I'm gonna this issue, but please do let me know if the problem is not fixed. Have a good day!


Fix for 2021.8.0 update

Fixing current and voltage constants on new 2021.8.0 update

Thank you very much @Keltere . I was going to merge directly but then I thought this fix would have broke whoever wants or need to run on previous versions since the new symbols would be missing then.
I'm anyway in the process of releasing some 'minors' and so I'll include your patch (patched of course ;) in the upcoming release


mesh prediction error

I run this code:
!python3 infer_wild_mesh.py --vid_path ./4.mp4 --json_path ./alphapose-results.json --out_path /content/MotionBERT
I have saved the best_epoch here:
/content/MotionBERT/checkpoint/mesh/FT_MB_release_MB_ft_pw3d/best_epoch.bin
Traceback (most recent call last):
File "/content/MotionBERT/infer_wild_mesh.py", line 64, in
smpl = SMPL(args.data_root, batch_size=1).cuda()
File "/content/MotionBERT/lib/utils/utils_smpl.py", line 62, in init
super(SMPL, self).init(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/smplx/body_models.py", line 133, in init
assert osp.exists(smpl_path), 'Path {} does not exist!'.format(
AssertionError: Path data/mesh does not exist!

Hi, please first download the data and put them to data/mesh following the instruction.

Hi,
Sorry to re-open the issue but 2 issues here :

According to the readme inference, they never say to donwload the files to do the mesh inference. Then, why would we need it ?
I tried to find the basicModel_neutral_lbs_10_207_0_v1.0.0.pkl file, with no success. Anyone could tell me where this file is ?

Thanks!


If no image is requested, a white image will be given by default

When loading tiles, if the image of layer 0 is lost or the image request is unsuccessful, the entire Earth will be lost. Can this issue be fixed? If no image is requested, a white image will be given by default

Hi @kak0na,
Sorry, but I'm a bit confused about what you're describing. Can you share more information?

What version of Unreal Engine & Cesium for Unreal are you using?
Please post a screenshot or video of the behavior that you're seeing.
Please post step-by-step instructions for how to reproduce that behavior.

Thank you!

I am using version V2.3.0.
I am using the WMTS url, which only has layer data 6-12. It does not have layer 0 data, which results in the default, the viewing angle is far, so there is no earth. I hope that if there is no img, it will also display a white image, is it ok?

Hi @kak0na,
As @kring mentioned, this bug was addressed in Cesium for Unreal v2.9.0. We recommend upgrading your version so you can get access to the latest features and bugfixes :smile:

I'm closing this issue since there's nothing actionable for us. If you experience this issue on a later version of the plugin, please let us know and we'll be happy to help. ☺


Animation: No need of unnecessary displacement of text while hover.

Describe the bug
When the user hovers in the section just bottom of the feature page the text unnecessarily moves to the right side that's is not a good experience.
Expected behavior
The spotlight animation should only be there with this animation removed.
Desktop (please complete the following information):

OS: MacOS
Browser Safari

Hey @DeadmanAbir, Assign this issue to me with the label gssoc.

Hey @DeadmanAbir, Assign this issue to me with the label gssoc.

Share a screen recording of the problem


2-color indicator.

Check out my fork at https://github.com/refdev/HZActivityIndicatorView, it contains this modification.


drivers: can: inconsistent behavior when calling can_stop() with pending transmissions

Describe the bug
The in-tree CAN controller drivers exhibit inconsistent behavior when calling can_stop() with pending CAN frame transmissions:

SJA1000 based drivers (ESP32 TWAI) and Bosch M_CAN based drivers automatically abort any pending transmission on can_stop().
NXP FlexCAN and STM32 bxCAN resumes pending transmissions when later calling can_start().
MCP2515 fails to enter configuration mode if there are pending transmissions.
I have not been able to find the behavior of the Renesas R-Car CAN driver as I do not have access to the hardware.

To Reproduce
Steps to reproduce the behavior:

Disconnect the CAN controller from the CAN bus.
Using the CAN shell, call can start <device>.
Using the CAN shell, call can send <device> 010.
Using the CAN shell, call can stop <device>.
Connect the CAN controller to a CAN bus with at least one other node for ACK'ing frames and monitor any received frames on that node (e.g. use candump on GNU/Linux).
Using the CAN shell, call can start <device>.
Observe the different behavior across the Zephyr CAN drivers as described above.

Expected behavior
The behavior should be the same across the Zephyr CAN drivers. Since some CAN controllers automatically abort any pending transmissions, the common behavior should be to abort any pending transmissions and notify the senders of this.
Impact
The behavior of the SJA1000 based drivers (ESP32 TWAI) and Bosch M_CAN based drivers is exceptionally bad as it will silently abort pending transmissions without notifying the senders and leave the transmit mailboxes occupied.
Environment (please complete the following information):

Commit SHA: 7e6e4579c29f269626d52bb70d5eed30113d90ee

Additional context
The can_stop() API was introduced in #50102. The above issue only occurs when calling can_stop() with pending transmissions.

I have opened #50546 for dealing with the (likely) issue in the Renesas R-Car CAN driver as I do not have access to the hardware to verify its behavior.


Fix deadlock in Python console upon isrExit() + a couple of smaller things

As mentioned in #268, there's a deadlock between our code that tries to execute the ISR (and obtain the GIL, more specifically) and the pthread_join() call in  isrExit() - it's getting stuck on futex() calls at both ends. This only happens when Python is in the console mode (running from script is okay, futexes are there, but don't deadlock).
I don't see any better way than to introduce a transient flag in mraa_gpio_context indicating that the ISR thread is being shut down. Even though the problem at hand is in some sense a corner case, it's anyway not good that we try to execute the ISR upon thread exit.
The chain before the patch is: pthread_cancel()->poll() returns like there's an interrupt->read()->return MRAA_SUCCESS->we're executing the ISR in mraa_gpio_interrupt_handler(). There seems to be no better way to distinguish poll() being interrupted by pthread_cancel() and a normal interrupt.
But I wouldn's say I like this approach (adding a flag is so-so), so I'd like to run this by you @arfoll. Any comments or suggestions are more than welcome.
The other two commits are a typo fix and some more elaborated error output in case our ISR fails (helped me during debugging and I think it would be generally useful).

@arfoll, reworked to use mraa_boolean_t & rebased onto current master.

Thanks alot, great work as usual! merged with 3c3136d43a70ad20160410fd5802cad430525cdf

Thanks :)


gallery: group and app links in galleries render badly

Describe the bug, optionally with screenshots
For group links in collections, the text is squished and the join button is on top of the text

To Reproduce
Steps to reproduce the behavior:

Go to collection with group links
looks bad

Expected behavior
should look good
Additional context:

Ship type: planet
Instance type: self-hosted
OS/browser: qutebrowser on arch linux

Relatedly, app links could be improved as well:

seen, brave on windows

Semi-recently, a revamp/refresh of Gallery interfaces resulted in the following updates to both reference types:

The reasoning for the particular variations:

Groups: Groups are a full bleed block, with a top-left justification for the group name in order to closely approximate the way groups are rendered throughout the system. What we're seeing here is the block as the "header" of the group, with the group's avatar and name situated within the header, as they are elsewhere.
Apps: Apps are explicitly not full bleed in order to preserve the specific styling of the tile, and to afford for instances where an agent/desk don't have a tile representation, but are linked regardless. App tiles are additionally not full bleed here in order to more clearly distinguish the forms of a gallery block and an app tile.

@thelifeandtimes This is still pre-release and in the design stage.

Tracking internally as LAND-366


Perma diff with cloudflare_ruleset

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.3.7
on linux_amd64
Affected resource(s)

cloudflare_ruleset

Terraform configuration files
resource "cloudflare_ruleset" "zone_level_managed_waf" {
  zone_id = data.cloudflare_zone.urbantzdotcom.id
  kind    = "zone"
  name    = "Cloudflare Managed WAF"
  phase   = "http_request_firewall_managed"

  rules {
    action = "skip"
    action_parameters {
      rules = {
        # Cloudflare Managed Ruleset :  SQLi - String Concatenation
        "efb7b8c949ac4650a09736fc376e9aee" : "3b0c61407d0b4f7d87e516472116d2fe"
      }
    }
    expression  = "REDACTED"
    description = "REDACTED"
    enabled     = true
  }

  rules {
    action = "execute"
    action_parameters {
      # TODO unable to get the value from api yet
      id      = "efb7b8c949ac4650a09736fc376e9aee"
      version = "latest"
    }
    expression = "true"
    enabled    = true
  }
}

Link to debug output
Not applicable
Panic output
No response
Expected output
I expect an empty diff every time I run a plan
Actual output
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # cloudflare_ruleset.zone_level_managed_waf will be updated in-place
  ~ resource "cloudflare_ruleset" "zone_level_managed_waf" {
        id      = "REDACTED"
        name    = "Cloudflare Managed WAF"
        # (3 unchanged attributes hidden)

      ~ rules {
            id          = "********************776a0"
            # (4 unchanged attributes hidden)

          - logging {
              - enabled = false -> null
              - status  = "enabled" -> null
            }

            # (1 unchanged block hidden)
        }

        # (1 unchanged block hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Steps to reproduce

terraform init
terraform apply
terraform plan

Additional factoids
The cloudflare_ruleset was created from terraform from the get go.
We migrated from the old system using https://support.cloudflare.com/hc/en-us/articles/5995821690637-Migrating-from-WAF-managed-rules-to-WAF-Managed-Rulesets
References
No response

I'm affected by this, and updating to the latest has not fixed it.

please see the note at the top of the docs - https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/ruleset
enabled = true is no longer valid. it should be status = "enabled"

Hey @jacobbednarz feel free to ignore this comment.
In our case we simply added the block:
logging {
      status = "enabled"
}

manually since it was indeed the wanted behaviour but I think there might still be an issue here.
We initially created these rules without any logging block since we wanted the default behaviour (not really obvious which is but it was fine). That is the behaviour we wanted, not needing to specify a logging block. I think the problem is that the field status defaults to the empty string instead of to enabled.
Cloudflare reports the logging to be enabled and the terraform module "sort of tries to remove that" by setting the empty string as the default. I believe this is the source of the perma-diff. Maybe the solution is to switch the default to "enable" or mark this field as computed if not specified.
But like I said this is no longer an issue for us.
Thanks in advance


Add support for images and attachments again

Add php7-fileinfo and php7-gd back in after they were (probably) removed during a merge.

Thanks!


skull owner with texture value not working

Trying to generate a custom skull with a texture
getting this error: https://pastebin.com/4T1ph9EA
code snippet
String owner = "eyJ0ZXh0dXJlcyI6eyJTS0lOIjp7InVybCI6Imh0dHA6Ly90ZXh0dXJlcy5taW5lY3JhZnQubmV0L3RleHR1cmUvZTVlODYyYzUzYTgzMGRiZDAzMzdkMzgxNDM2M2Y1MjRjOGI5ZGExNzE0ZWQ1NmI1ZDQyMDgwMDI1MTQ0MjgyMyJ9fX0=";
ItemStack skull = ItemStackBuilder.from(Material.PLAYER_HEAD).skull().owner(owner).itemStack();
builder = ItemStackBuilder.from(skull); // line 251

wait now it works... I didnt change anything :/

nvm it was another part of the code that worked


Not working on IE9 (tiles being hosted on MapBox)

I wonder if there is a workaround (IE9, cross-domain failure).
I tried XDomainRequest on IE9, but no luck. Not sure if responeType being 'arraybuffer' adds more difficulty on the request or not.
IE10, 11 both work fine.

Yes, from what I recall, arraybuffer doesn't exist in IE9.


Reusing Containers, multiple saga injected.

React Boilerplate
Issue Type

[ ] Bug (https://github.com/react-boilerplate/react-boilerplate/blob/master/.github/CONTRIBUTING.md#bug-reports)
[ ] Feature (https://github.com/react-boilerplate/react-boilerplate/blob/master/.github/CONTRIBUTING.md#feature-requests)

Description
I have a container that loads up resources from API based on the url src props that's being passed into the container. When I use the container for more than once in a page, the API calls would be doubled (due to reinjection of saga, as expected). I've done a few things to workaround this, each with their cons.

Workaround 1. Insert mode: ONCE_TILL_UNMOUNT in injectSaga. This prevents it for calling API too many times, however it also prevents it from calling the API forever.

Workaround 2. Move the saga injection logic to the page level, works fine but it seems like I would need to "remember" to add the saga to the page that contains the container. There must be some better ways to insert and forget?
I've struggled with this for a few days and I couldn't get any advises on how to handle such cases (Maybe I am not hitting the right keywords on google). How would one approach this instead?
Steps to reproduce
Tried uploading to codesandbox but it's too big. Created a git repo instead. Do take note that this is just a reproduction of the issue, so the logic of separating the components/containers might be silly according to the example's needs.
https://github.com/lunateck/react-boilerplate-container-reuse

Thanks for the detailed report!
I'm not knowledgeable enough on the Saga setup to properly help with the multiple injections but here are some thoughts that might help you avoid the issue altogether:
If I have a Container with sagas but it doesn't make sense as a Page, it's generally because it's something that I want mounted all the time. For example: Header, auth system, notifications/toasters. Those, I'll render outside of my router . That way you avoid any issues with saga injections.
All my other Containers are Pages (which only render Components) and I haven't had any trouble with saga injections. This does mean using a slightly different architecture than what you've done with your Counter container though.
If this doesn't satisfy your need, you may want to take a look at the multiple conversations on saga injections going on in the repo:
#2063 (An unmerged PR from which you might be able to get a fix)
#1420 (One of several long convos on this topic)
Good luck!

If I have a Container with sagas but it doesn't make sense as a Page, it's generally because it's something that I want mounted all the time. For example: Header, auth system, notifications/toasters. Those, I'll render outside of my router . That way you avoid any issues with saga injections.
All my other Containers are Pages (which only render Components) and I haven't had any trouble with saga injections. This does mean using a slightly different architecture than what you've done with your Counter container though.

@julienben thanks for your feedback! What I have is something in between, it doesn't need to be rendered always and is only used in some pages (yet in a frequent basis). :(
One use case that I have is an image gallery (that loads different set of images). I've also started out with page containers but I found out that I was duplicating codes. Hence, I went for the above architecture, but it came with a hefty performance cost due to my inability to handle the sagas (I haven't given up yet on trying to solve it like so). I'll check out the topics you've included. Again many thanks for your time to help!

Closing due to lack of activity, but please reopen if further investigation is needed.


#160916394 This adds the database connection file

What does this PR do?

This adds the database connection file:
The file contains functions to initialize both the main database and the test database, and a function to tear down the connection

Pull Request Test Coverage Report for Build 158

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.97%

Totals

Change from base Build 146:
0.0%

Covered Lines:
96

Relevant Lines:
99

💛  - Coveralls

Pull Request Test Coverage Report for Build 162

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.97%

Totals

Change from base Build 146:
0.0%

Covered Lines:
96

Relevant Lines:
99

💛  - Coveralls


Fix: do not allocate a Vec to iterate over CfgEdge nodes

Resolves the todo unncecessary collect in lib/relooper/src/graph/cfg/mod.rs.
The issue there was that the filter call takes a closure that captures the borrow on self which then prevents us from borrowing self to insert the node in the body of the loop. So here I have replaced the call to filter with a simple if statement that does the same thing. I also directly borrowed out_edges to avoid multiple borrows on self (this is also more efficient because it saves us from building the HashSet).
While I was at it, I replaced the to_vec implementation on CfgEdge with something else that does the same job (creates an iterator over the nodes of the edge), but avoids the heap allocation.

Heh, I tried almost the same, just didn't do `let out_edges = &mut self.out_edges;``


Clean problem

After recompiling(hot reload + watch) in my project folder i see old .css and .js files(i use [cahe] for them) and new files. What should i do to delete old files on hot reload?
My config file:
https://jsfiddle.net/9rmt24yp/1/
And what i see in directory on hot reload:
https://i.gyazo.com/95db01780779f041f0596c134075e04b.png

Also seeing this. Apologies of this is obvious but I'm very new to webpack.

@Shepless
I resolved this problem by adding hash only for production version. In development files will be rewrite.

@binjospookie Simple solutions are often the best! Thanks for sharing!

Can we close this one or is there still an open bug?

@johnagan It can be closed from my perspective...

Awesome! Thanks folks ✨

so is there no way to clean folder in watch method ? can add a feature to clean on the watch method?


State history no longer keeps a local history

It is useful to have some history of your state at reasonably frequent intervals in case you accidentally deselect your neuron of interest (which should not be as difficult to retrieve than it is...) or you accidentally move off of the neuroglancer website. After the introduction of the local storage/local_id state storage, such save states were part of the history under the $ (Save History) button. However, at some point this feature either stopped working or was intentionally removed and now the Save History button only stores manual saves.

In order to resolve some issues, Save state was change to reuse the ID per tab. The ID only changes using a URL that doesn't have a local ID or if two tabs are using the same local ID. As a result, on state changes it gets updates the same entry rather than push a new one.
For save history, returning that old functionality shouldn't be a problem. The main issue is that save history is now just storing the URL to the manually saved states. Going back means at worst storing the whole state for every change and local memory will quickly be filled (even faster if the state has many annotations). Maybe some combination of storing a diff and using a timer to store state will work.
Once annotations are moved out of the state, it won't be as serious but that's a long term project.

Going back means at worst storing the whole state for every change and local memory will quickly be filled (even faster if the state has many annotations). Maybe some combination of storing a diff and using a timer to store state will work.

This feels like a good solution to me. The current solution with a static local id per tab makes total sense, and adding a side save in the background every 5 minutes or so (configurable in the settings) would be great. My colleagues have gotten good at using the server-based save feature for states that they want to keep, so this extra local save is basically as a revert/undo if something goes awry. As such, getting rid of the oldest files when storage runs out would be fine. I did a quick check and a state with 1000 annotations points is ~90KB once its compressed (which could be done with something like this), so there should be room for several history files even in of bulky states.
I'm a bit confused by what you mean with "once annotations are moved out of the state..." — I can see annotations also coming from a database in addition to user input and (sometimes quite large) programmatically generated neuroglancer states, but that will be an additional feature not one one that replaces other methods. Is there another storage angle that is in the works?

Iirc Dynamic Annotations (read/write to server) would replace locally stored annotations.

Yeah, Dynamic Annotations were what I was referring to above about the database. They will be extremely useful, but not the only game in town.


Change Get-Service calls to ignore errors

Change Get-Service to ignore errors, for when there's a service that's malformed or would otherwise tank the Get-Service command
A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the develop branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.
For status checks on the main branch, please use microsoft.ProjectReunion
(https://dev.azure.com/ms/ProjectReunion/_build?definitionId=391&_a=summary)
and run the build against your PR branch with the default parameters.

/azp run


Getting a parse issue

bot works until you hit submit details I think it is to do with the python_telegram_bot==13.1 requirement being out dated? tried to update to the latest but doesnt work at all
error log
bot.py:219: DeprecationWarning: count is deprecated. Use estimated_document_count or count_documents instead. Please note that $where must be replaced by $expr, $near must be replaced by $geoWithin with $center, and $nearSphere must be replaced by $geoWithin with $centerSphere

Hey @hodlape  can you post the full error message here, i'm getting a similar kinda issue

This is the same issue I'm having when I click submit details:
Error Log:
2023-06-24 02:52:54,101 - telegram.ext.dispatcher - ERROR - No error handlers are registered, logging exception.
Traceback (most recent call last):
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/ext/dispatcher.py", line 432, in process_update
handler.handle_update(update, self, check, context)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/ext/conversationhandler.py", line 517, in handle_update
new_state = handler.handle_update(update, dispatcher, check_result, context)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/ext/handler.py", line 156, in handle_update
return self.callback(update, context)
File "/home/alonsome/Projects/Python/CyberVerse/utils/handlers.py", line 76, in follow_telegram
update.message.reply_text(
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/message.py", line 603, in reply_text
return self.bot.send_message(self.chat_id, *args, **kwargs)
File "/home/alonsome/.local/lib/python3.10/site-packages/decorator.py", line 232, in fun
return caller(func, *(extras + args), **kw)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/bot.py", line 135, in decorator
result = func(*args, **kwargs)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/bot.py", line 458, in send_message
return self._message(  # type: ignore[return-value]
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/bot.py", line 281, in _message
result = self._post(endpoint, data, timeout=timeout, api_kwargs=api_kwargs)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/bot.py", line 245, in _post
return self.request.post(f'{self.base_url}/{endpoint}', data=data, timeout=timeout)
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/utils/request.py", line 349, in post
result = self._request_wrapper(
File "/home/alonsome/.local/lib/python3.10/site-packages/telegram/utils/request.py", line 272, in _request_wrapper
raise BadRequest(message)
telegram.error.BadRequest: Can't parse entities: can't find end of the entity starting at byte offset 66

bot works until you hit submit details I think it is to do with the python_telegram_bot==13.1 requirement being out dated? tried to update to the latest but doesnt work at all
error log
bot.py:219: DeprecationWarning: count is deprecated. Use estimated_document_count or count_documents instead. Please note that $where must be replaced by $expr, $near must be replaced by $geoWithin with $center, and $nearSphere must be replaced by $geoWithin with $centerSphere

How did you solve it bro, I'm having quite the same issue.


[SPARK-42789][SQL] Rewrite multiple GetJsonObject that consumes same JSON to single JsonTuple

This PR takes over https://github.com/apache/spark/pull/40419 as the original author @wangyum is busy with other works.

What changes were proposed in this pull request?

This PR adds a new rule to rewrite multiple GetJsonObject to one JsonTuple if their json expressions are the same and the path is .<name>. For example:
=== Applying Rule org.apache.spark.sql.catalyst.optimizer.RewriteGetJsonObject ===
!Project [get_json_object(a#0, $.c1) AS c1#3, get_json_object(a#0, $.c2) AS c2#4, get_json_object(b#1, $.c1) AS c3#5, get_json_object(b#1, $.c2) AS c4#6, get_json_object(c#2, $.c1) AS c5#7]   Project [c1#13 AS c1#3, c2#14 AS c2#4, c1#15 AS c3#5, c2#16 AS c4#6, get_json_object(c#2, $.c1) AS c5#7]
!+- LocalRelation <empty>, [a#0, b#1, c#2]                                                                                +- Generate json_tuple(b#1, c1, c2), false, b, [c1#15, c2#16]
!                                                                                +- Generate json_tuple(a#0, c1, c2), false, a, [c1#13, c2#14]
!                                                                                +- LocalRelation <empty>, [a#0, b#1, c#2]

Please note that it needs to rewrite the path, otherwise it will not get the correct value. For example:
spark-sql (default)> SELECT get_json_object('{"name": "alice", "age": 5}', '$.name') AS name;
alice
spark-sql (default)> SELECT name lateral view json_tuple('{"name": "alice", "age": 5}', '$.name') a AS name;
NULL
spark-sql (default)> select name lateral view json_tuple('{"name": "alice", "age": 5}', 'name') a AS name;
alice

Why are the changes needed?

Improve query performance by eliminating multiple-time expensive JSON parsing.
Does this PR introduce any user-facing change?

A new configuration spark.sql.optimizer.rewriteGetJsonObjectToJsonTupleThreshold is introduced, while the default value None keeps the current behavior.
How was this patch tested?

Unit test and benchmark test.
Benchmark result:

No. of GetJsonObjects
Default(ms)
Rewrite to JsonTuple(ms)
Relative

2
37914
24887
1.5X

3
52862
26752
2.0X

4
71680
28452
2.5X

5
108479
36830
2.9X

10
153952
32436
4.7X

15
224950
34806
6.5X

20
293155
38148
7.7X

25
366037
45725
8.0X

30
439567
52949
8.3X

35
526481
60586
8.7X

Was this patch authored or co-authored using generative AI tooling?

No.

cc @wangyum @cloud-fan @yaooqinn @ulysses-you


beta -> latest (4.51.0)

My main focus here was updating package dependencies for the backend.
The dependencies in ui/package.json I have updated where it didnt bring any build errors.

⚠ Also bump minimum supported node version to 18

Notable Changes

Node Support

⚠ Node 16 has reached end-of-life and is no longer supported by the Homebridge UI.

Starting with v4.50.7, the Homebridge UI will no longer work with Node 16, only Node 18 and 20
Homebridge UI follows the same Node support schedule as Homebridge. See the How-To-Update-Node.js page in the Homebridge wiki for more information

Plugins

Use plugin display name if available (#1582)

Other Changes

System:

Updated backend dependencies (#1571 and #1569)
Updated frontend dependencies (#1572)
Update @homebridge/node-pty-prebuilt-multiarch to version v0.11.1 ( Node 20 Support and upstream refresh )
Update @homebridge/node-pty-prebuilt-multiarch to version v0.11.3 (#1583)

@bwp91 Ready for review/release?

@bwp91 Ready for review/release?

not quite yet, I think @NorthernMan54 wanted to do a bit more testing re the nodepty repo + since this beta has quite a few changes, would be good to get some other collaborators to just install and make sure there are no immediate issues

I have 6 different homes that I manage and all have been working good.


Add tests for TRSettingsController

This is the first in a series of PRs to facilitate porting more files to Swift. A future PR will actually port TRSettingsController to Swift; this PR is intended to add test coverage.
While adding tests I also ran into some issues with build settings related to the PCH file. This PR also removes PCH files in favour of explicit imports.

Small questions. Looks good.

Thanks @dazmuda ✨


refactor if checks to be more compact

This proposes a refactoring, which simplifies and makes more compact if-else expression of type var if var else to the equivalent var or.
Reference one-liner to obtain all var if var else occurrences:
grep -oPrnH '\b(\w+) if \1\b else' .

Additionally, some unused imports have been removed.

we don't merge style only pr unless it's in core tinygrad with a new linter check

we don't merge style only pr unless it's in core tinygrad with a new linter check

Hi @chenyuxyz,
Could you confirm what the "new linter check" entails, please?
Also, isn't tinygrad/helpers.py considered core?
Thank you for the info provided!