Natural Aqua Affinity for merling doesn't seem active

As it says on the tin.
Using origins: <IP_ADDRESS>
MC: 1.16.5
Forge: 36.1.24
Underwater mining is still acting slow, even when shifting down onto the block so I'm not floating up. Thankfully I can resort to magical mining with ars noveau. I have since removed better diving so it shouldn't be related to my previous issue. Mods are... still numerous. But I'm not entirely sure what could be denying it if anything.

As far as I am aware this is a mod interaction issue as it doesn't occur in my dev environment.

darn it, I had a feeling. Guess I'll have to take a poke around... :V

oh! shoot, I never came back to report on what fixed this. it was actually a mod called annointed items because of some kind of implementation with it's "mining rate" annointment that can be applied to equipment. Despite never having been annointed yet it would affect all forms of non-vanilla aqua affinity and such.
https://www.curseforge.com/minecraft/mc-mods/anointed-items

While the fix is trivial on their end (Changing a method call), I don't have high hopes that they will implement it in a timely manner.
If you want to fix them manually, just drop the files in the following zip file in either origins or annointed items META-INF directory in the jar file. This will fix the bug. (It's a simple coremod that replaces a method call)
Bugfix.zip

intriguing. I don't have the mod installed anymore, but if thats a fix, hopefully they can be contacted to fix it and upload it, so that eventually I can include it in the pack when it's finished eventually.

Just saw that you wanted more info on what mod interactions break Aqua Affinity, one mod that causes this is Project MMO. I already mentioned this on their discord but got no answer, but afaik their dev is pretty busy at the moment. I played around with the bugfix posted above, but I don't know Java and couldn't get it (or something slightly altered) to work. I believe the problem is similar but not quite the same (but again, don't know Java). Even if it's something on PMMO's side I'd love something like the bugfix EdwinMinecraft posted if that is possible.

I also had this problem with the mod Axes Are Weapons. Is this on their end or this end?
https://www.curseforge.com/minecraft/mc-mods/axes-are-weapons

Also having this exact same issue (same mod and all) here in the future of 2024, was there ever any fix?


Bolt failed to start on ubuntu 16.04

Describe the Bug
I installed bot by following the docs using apt get and when trying to run it I'm getting:
bolt
/opt/puppetlabs/bolt/bin/ruby: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by /opt/puppetlabs/bolt/lib/libruby.so.2.5)

Expected Behavior
Bolt running
Steps to Reproduce
Steps to reproduce the behavior:

Install on ubuntu 16.04
Try to run it

Environment

Latest
Platform Ubuntu 16.04

Additional Context
Add any other context about the problem here.

Hi @omerlh - I don't think this is a problem with Bolt, it looks like you'll just need to upgrade your libc6 package on the system you're installing Bolt on (source):

@omerlh I'm going to close this because I'm reasonably certain it's not an issue with Bolt, as we can't reproduce it with any Bolt version. Let me know if you'd like to keep discussing and we can reopen the issue!


profanity: 0.5.1 -> 0.6.0

Motivation for this change
I noticed the announcement of the profanity.im release version 0.6.0
Things done

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

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

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

@GrahamcOfBorg build profanity

Build failure on Darwin. I guess the packages-osx patch is still needed?

I am also unsure about the osx-patch. I asked on the irc what to do since I cannot build the package with the patch. I was told to leave it out.
Do I need a Mac to test this?

@GrahamcOfBorg build profanity

@eri451 ideally yes, or ask around on IRC if a Darwin user is willing to do the test
I refreshed the Darwin patch for v0.6.0 sources so it is no longer a blocker.
The bot Darwin build wasn't complete only because it took to much time building gpgme

thank you


enable autosave

Opening this for discussion purposes. This autosave isn't exactly what I'm used to, which is it essentially saves as you type (well, when you go back to normal mode in vim), but it is an autosave of sorts.
This will save whenever atom or a pane loses focus. In other words, only one file can be modified at a time. It may be annoying in Guard/--watch'd things, but less annoying than the vim version.
Thoughts?

Oh I should mention some pros:
Lots of time is wasted by people forgetting to save. I've seen devs and designers refreshing pages wondering why their change didn't happen only to realize they didn't save. This particular type of auto save would prevent that while also preventing save spamming that negatively affects --watch'd things.
I think autosave is pretty common for sublime text users and users of my vim config, so there's a set of people that are used to it.

Ok, I'ma merge this in and see how people respond.


chore: consolidate the Superset python package metadata

SUMMARY
Following up on https://github.com/apache/superset/pull/27859, where
I introduced a new file pyproject.toml to set up the
python project metadata, here I'd like to consolidate and modernize
further by pushing as much as possible into pyproject.toml,
except where things need to be dynamic, for which we'll keep using
setup.py:

get rid of setup.cfg, everything there should be able to move to
pyproject.toml
remove duplicate keys from setup.py and pyproject.toml
minimize/simplify setup.py

NOTE: we may be able to chisel a bit more at setup.py, but it's going to me needed in some capacity since we have some dynamic parameters, notably the version number we read and align from package.json. There's a tricky thing related to entry-points we could tackle.
Also note that having setup.py provides continuity for commands like python setup.py sdist which is kind of nice.
If we decided to make that a goal, we could fully kill setup.py and centralize on pyproject.toml, we're well on our way there.

@john-bodley let me know what you think as to whether tox.ini should be consolidate as well. I'm fine either way, but note that I had to use "legacy_tox_ini" since not all tox.ini configs have a direct translation in toml, and we have quite a bit of things going on in there...


Add ServerExtension#httpClient()

Motivation:
This is to provide a convenient way to make httpClient with ServerExtension.
Modifications:

Add ServerExtension#webClient()
Add ServerRule#webClient()
Add ServerXXX#configureWebClient(wcb)

Result:

Closes #3761

For me, I usually create WebClients with specific SessionProtocols.
(e.g. WebClient.of(server.uri(SessionProtocol.H1C)), etc.)
Would it be worth adding such method so that a user can easily create the client?
@ParameterizedTest
@EnumSource(value = SessionProtocol.class, names = { "H1C", "H2C" })
void foo(SessionProtocol protocol) {
    WebClient client = server.webClient(protocol);
    ...
}

For me, I usually create WebClients with specific SessionProtocols. (e.g. WebClient.of(server.uri(SessionProtocol.H1C)), etc.) Would it be worth adding such method so that a user can easily create the client?
@ParameterizedTest
@EnumSource(value = SessionProtocol.class, names = { "H1C", "H2C" })
void foo(SessionProtocol protocol) {
    WebClient client = server.webClient(protocol);
    ...
}

How about adding webClient(SessionProtocol, Consumer<WebClientBuilder>) as well? 🤔

webClient(SessionProtocol, Consumer)

I think it's a bit too much. 😆 Users could just build their own client using server.uri(...).

server.webClient(protocol);

Have thought about it again and I think we don't need to implement it in this PR. Let's implement it in the follow-up PR if it's needed. 😄


Add space option to formatPhoneNumberString

I have added space option to formatPhoneNumberString, like we have in formatNumberString.
I think it can be useful in some cases. For example, we can replace plain space by nonbreakable space.

Why just not use the non breakable space by default?

Why just not use the non breakable space by default?

Okay. Also, I've increased major number of version, because this PR breaks backward compatibility.

Published as<EMAIL_ADDRESS>Thanks!


Allow any type that implements FromServerFnError as a replacement of the ServerFnError in server_fn

Closes #3270
Fixes #3153
Addresses #3155

[x] Added FromServerFnError
[x] Deprecate the ServerFnError::WrappedServerError variant and manual error conversion helpers
[x] Removed ServerFnErrorErr::WrappedServerError and impl From<ServerFnError> for ServerFnErrorErr
[x] Update server_fn_macros to use the FromServerFnError trait
[x] Added ServerFnErrorErr::Middleware for a better context of the errors
[x] Added an example in examples/server_fn_axum
[x] cargo +nightly make check

The implementation is complete. The only breaking changes should be the third ones mentioned above, and other behaviors should remain retained as expected.

@nicolas-guichard Hi, this fix is similar to #3253, but takes a different approach, addressing #3153 and #3155. It would be great if you could take a look and review whether this approach works.

Thanks for this work!
While I have not had time to manually test it, and have not reviewed every line, this looks like it is the right approach, and it certainly seems to make custom errors more usable, which I think is great.
It comes at a slightly awkward time in our release cycle as we are planning a 0.7 release this weekend and this makes some breaking changes that won't have time to be tested in the beta/rc process before that release.
So here's my suggestion:

@benwis plans to test this out and we will welcome other testers and reviews
We will release 0.7 without this included
We will also plan a 0.8 some time in late December/early January, that includes the final version of this PR and any breaking-change-requiring bugfixes from 0.7

Thank you for your feedback! That release schedule is reasonable to me.


unrobust CMake search logic

ROCM 4.5 and hip-lang package assume hip is installed as <ROCM_ROOT>/hip and tries to access paths outside <ROCM_ROOT>/hip and inside ``<ROCM_ROOT>` for example
https://github.com/ROCm-Developer-Tools/HIP/blob/cddb52549b4f4fce9165cac8b2ccf25173ba3157/hip-lang-config.cmake.in#L108
reproducer
cmake_minimum_required(VERSION 3.21.0)

project(test_hip CXX)
enable_language(HIP)

When using AOMP where hip is installed inside /usr/lib/aomp_14.0-0
$ cmake ..
-- The CXX compiler identification is Clang 14.0.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/mpicxx - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- The HIP compiler identification is Clang 14.0.0
-- Detecting HIP compiler ABI info
CMake Error in /home/yeluo/opt/cleanup/qmcpack/build_R7_aompbuild_offload_cuda2hip_real_check/test/build/CMakeFiles/CMakeTmp/CMakeLists.txt:
  Imported target "hip-lang::device" includes non-existent path

    "/home/yeluo/rocm/aomp_14.0-0/../include"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

CMake Error in /home/yeluo/opt/cleanup/qmcpack/build_R7_aompbuild_offload_cuda2hip_real_check/test/build/CMakeFiles/CMakeTmp/CMakeLists.txt:
  Imported target "hip-lang::device" includes non-existent path

    "/home/yeluo/rocm/aomp_14.0-0/../include"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

CMake Error at /home/packages/spack/opt/spack/linux-ubuntu20.04-zen2/gcc-9.3.0/cmake-3.21.4-t47xgwhz7ql3mvemzhzc7zwsrly6na3s/share/cmake-3.21/Modules/CMakeDetermineCompilerABI.cmake:49 (try_compile):
  Failed to generate test project build system.
Call Stack (most recent call first):
  /home/packages/spack/opt/spack/linux-ubuntu20.04-zen2/gcc-9.3.0/cmake-3.21.4-t47xgwhz7ql3mvemzhzc7zwsrly6na3s/share/cmake-3.21/Modules/CMakeTestHIPCompiler.cmake:29 (CMAKE_DETERMINE_COMPILER_ABI)
  CMakeLists.txt:4 (enable_language)

-- Configuring incomplete, errors occurred!

issue 1. When looking for __clang_cuda_math.h, the path should be derived on where clang is not where hip libraries are
issue 2. Should not add path blindly ${_IMPORT_PREFIX}/../include without verification.

Have the exact same problem with hip-config.cmake for target hip::device.

@ye-luo Apologies for the lack of response. Can you please test with latest ROCm 6.0.2 (HIP 6.0.32831)? If resolved, please close ticket. Thanks!

Hi @ye-luo, I see that both the cases brought up in https://github.com/ROCm/ROCm/issues/1766 and https://github.com/ROCm/HIP/pull/2776 have been addressed and compile correctly in the latest ROCm release. I will close this ticket as the issue has been resolved.


[help] Garbage truck: Capacity , Pickup&Delivery, Time Window VRP.

This problem is complex. I try model this in ORTools but i cannot figure it out. How combine Capacity and Pickup & Delivery in my scenarion (there is single delivery point and multiple pickups)? Could You can help me?
I assume that Time Window Constraint should have each visited points from (i.e. 8-16).
Thanks

Any updates?

No

I think this is straightforward to model as a pickup and delivery problem, once you wrap your head around how to think about pickup and delivery nodes.
Model it like this.  Each location i has a certain number of bins with some weight W.  The bin origin is the location i, and the destination is the dump, but it has to be a unique node, so lets call it d_i.  Your demand callback takes a single parameter for the node, and if the node is a pickup location i, it replies with the appropriate weight.  If the node is the dump node for i, d_i, then the callback function replies with the negative of the pickup weight at i, reflecting that the load is dropped off and the occupied capacity is restored.
The dump trucks can have the same capacity, or a vector of capacities, following the usual approach.  When the total picked up weight exceeds the capacity, the truck must "deliver" each of its garbage loads to their destinations at the dump.  These dummy nodes are all the same location, of course, so the travel cost from d_1 to d_2 is zero, but the "demand" at d_1 is the negative of the pickup at 1; the demand at d_2 is the negative of the pickup at 2, and so on.
You can set time windows on both pickups (the i nodes) and the dump (the d_i nodes), again following the usual way to set time windows.
The trick is to get over the fact that the dump is one location in the real world, and instead recognize that it is many nodes in model space.
Some other fun wrinkles might be requiring that the truck completely empty all of its garbage when visiting the dump, minimizing the weight carried per mile so as to reduce fuel costs, etc etc.

@jmarca thank You very much.
My conclusion: for simple model Pickup and Delivery problem, solving works quiet good.
I have some wired feeling that I have to split collecting garbage bins to two or more days. In time window there is no chance to model it. Am I right?

in case:
search_parameters.first_solution_strategy = (routing_enums_pb2.FirstSolutionStrategy.PARALLEL_CHEAPEST_INSERTION)

finished immediately with status : 2 -> ROUTING_FAIL

On Mon, Aug 19, 2019 at 01:07:25AM -0700, pietrucha wrote:

@jmarca thank You very much.
My conclusion: for simple model Pickup and Delivery problem, solving works quiet good.
I have some wired feeling that I have to split collecting garbage bins to two or more days. In time window there is no chance to model it. Am I right?

Yes you can model it I'm sure.  You can require that the vehicles
return to depot overnight (so even though the time window at the
pickup location is active, the vehicles are all "busy" serving their
"sleep overnight" "pickup" task.  Or you can split the track bin into
multiple logical (dummy) pickups and deliveries to the dump, but link
all of them with a disjunction such that only one of the logical set
can be served.  Then each of the N bins can have its own time window
tailored to the needs of the day (say Saturday pickup must be later
than weekday pickup, etc).

--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/google/or-tools/issues/1428#issuecomment-522460146

--
James E. Marca

As to your second question, you can't parallel search in transportation problems in OR Tools.  Single thread only.
Without seeing a lot more of your code, I can't really hazard a guess as to why your solver run is failing for such a small problem.  And that is a small problem:

(12 nodes - 1 depot, 11 bins, 1 dump - so I have distance & time matrix 24x24 )

is tiny, you should have no trouble solving it.  My guess is you've messed up the specification of the time windows in the dimensions definitions.
As to the solution strategy questions, the first solution strategy is variable---I usually go with parallel cheapest insertion as a first try, but in almost every non-trivial case I've ever set up, at some point I'll try almost all of the initial solutions to see what works the best.  Very problem dependent.  For example, I just cut and paste this from an old project:
    search_parameters.first_solution_strategy = (
        # routing_enums_pb2.FirstSolutionStrategy.LOCAL_CHEAPEST_ARC)  # works
        routing_enums_pb2.FirstSolutionStrategy.GLOBAL_CHEAPEST_ARC) # works
        # routing_enums_pb2.FirstSolutionStrategy.SAVINGS)    # works
        # routing_enums_pb2.FirstSolutionStrategy.FIRST_UNBOUND_MIN_VALUE)    # crazy suboptimal
        # routing_enums_pb2.FirstSolutionStrategy.AUTOMATIC)    # suboptimal
        # routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)   # suboptimal
        # routing_enums_pb2.FirstSolutionStrategy.PATH_MOST_CONSTRAINED_ARC) # suboptimal
        # routing_enums_pb2.FirstSolutionStrategy.PARALLEL_CHEAPEST_INSERTION) # no assignment
        # routing_enums_pb2.FirstSolutionStrategy.LOCAL_CHEAPEST_INSERTION)    # no assignment
        # routing_enums_pb2.FirstSolutionStrategy.ALL_UNPERFORMED) # no assignment
        # routing_enums_pb2.FirstSolutionStrategy.BEST_INSERTION)    # no assignment
        # routing_enums_pb2.FirstSolutionStrategy.CHRISTOFIDES)    # no assginment
        # routing_enums_pb2.FirstSolutionStrategy.SWEEP)    # no assignment
        # routing_enums_pb2.FirstSolutionStrategy.SEQUENTIAL_CHEAPEST_INSERTION) # no assignment

@jmarca

As to your second question, you can't parallel search in transportation problems in OR Tools. Single thread only.

OK.

is tiny, you should have no trouble solving it. My guess is you've messed up the specification of the time windows in the dimensions definitions.

I made mistakes making travel time limitation. Even though, CHRISTOFIDES strategy return 2.
After testing on matrix 580x580 I found out that truck does'n fully unload. Load with ** is when truck deliver trashes
Route for vehicle 3:  0 Load(0) ->  288 Load(110) ->  289 Load(220) ->  579 Load(**110**) -> 286 Load(220)  -> 276 Load(**110**)  ->  578 Load(**0**) ->  0 Load(0)
Questions:

In my model each node have different numbers of bin and the load time is different. I add load time in RegisterTransitCallback like that. Does my aproach is correct?

return data['time_matrix'][from_node][to_node]+ data['load_time_demands'][from_node]

How add unload time on a dump when I have multiple virtual d_i? It costs me only first delivery d_i.
I am also facing situation when I have to visit some node 2 times in a week. Should I create 2 virtual nodes for that kind? Complexity will be very high!!!

On Wed, Aug 21, 2019 at 09:09:42AM -0700, pietrucha wrote:

@jmarca

As to your second question, you can't parallel search in transportation problems in OR Tools. Single thread only.

OK.

is tiny, you should have no trouble solving it. My guess is you've messed up the specification of the time windows in the dimensions definitions.

I made mistakes making travel time limitation. Even though, CHRISTOFIDES strategy return 2.
After testing on matrix 580x580 I found out that truck does'n fully unload. Load with ** is when truck deliver trashes

It will only "fully unload" if you require it to do so.
If you have one unload node for each load node, then you can forbid
travel from unload nodes to load nodes unless truck is empty, using a
conditional constraint.  Something like (I'm just typing this in, it
is not real code):
for unload_node in unload_nodes:
    unload_idx = manager.NodeToIndex(unload_node)
    for possible_load_node in load_nodes:
        possible_load_idx = manager.NodeToIndex(load_node)
        if routing.NextVar(unload_idx).Contains(possible_load_idx):
            # the routing model has a possible link from unload node to load node
            # the condition: must be emtpy when arriving at pickup node
            empty_at_load = my_dimension.CumulVar(possible_load_idx) == 0
            # going from memory on this, you must double check

            # check if the next node planned really is this pickup node
            next_node_condition = routing.NextVar(unload_idx) == possible_load_idx
            # formulate a conditional expression
            expression = solver.ConditionalExpression(next_node_condition,
                                                      empty_at_load,
                                                      1)
            # apply the conditional expression to the model
            solver.AddConstraint( expression >= 1 )

The conditional is saying if the vehicle moves from unload node to
load node ( the next_node_condition is true), then the constraint that
the truck is empty at the pickup node (the empty_at_load constraint)
must also be true (must be enforced).
If it cannot be true for whatever reason, then next_node_condition
must also be false, and the truck cannot go from the unload node to
the potential load node.  Instead it must visit another unload node to
empty more trash, and try again.

Route for vehicle 3:  0 Load(0) ->  288 Load(110) ->  289 Load(220) ->  579 Load(**110**) -> 286 Load(220)  -> 276 Load(**110**)  ->  578 Load(**0**) ->  0 Load(0)
Questions:

In my model each node have different numbers of bin and the load time is different. I add load time in RegisterTransitCallback like that. Does my aproach is correct?

return data['time_matrix'][from_node][to_node]+ data['load_time_demands'][from_node]

That seems fine to me.  Try it on a tiny problem and check the timings by hand.

How add unload time on a dump when I have multiple virtual d_i? It costs me only first delivery d_i.

You'll have to be clever.

I am also facing situation when I have to visit some node 2 times in a week. Should I create 2 virtual nodes for that kind? Complexity will be very high!!!

Yes you must create dummy nodes.  The solver can only visit a node at
most one time.  Once the node is visited, it is checked off the list
of places that must be visited.
And yes the complexity is going to go up with the more nodes you have
to visit in a week.  That's why these problems are fun to solve and
why I do this for my job.

--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/google/or-tools/issues/1428#issuecomment-523528629

--
James E. Marca

I am also facing situation when I have to visit some node 2 times in a week. Should I create 2 virtual nodes for that kind? Complexity will be very high!!!

Yes you must create dummy nodes.  The solver can only visit a node at
most one time.  Once the node is visited, it is checked off the list
of places that must be visited.

@jmarca Example nodes:
- 0 - depot
- 1 - visited twice a week, TW (mon-fri: 6-7 day of the week doesn't matter, this node shouldn't be visited day by day), I create 5 virtual node representing 5 days like this:
    1' mon 6-7
    1'' tue 6-7
    1''' wed 6-7
    1'''' thu 6-7
    1''''' fri 6-7

- 2 - visited once a week, TW (mon-fri: 6-14, day of the week doesn't matter),
    2' mon 6-14
    2'' tue 6-14

- 3 - ...

1000 - ...

When truck visit node 2'', other 2 nodes should be marked as visited. How to mark those nodes?

When truck visit node 2'', other 2 nodes should be marked as visited. How to mark those nodes?

you make a disjunction.  read up on the exact semantics in the docs, but essentially
for duplicate_nodes in big_list_of_lists:
duplicate_indices = [manager.NodeToIndex(c) for c in duplicate_nodes
routing.AddDisjunction(duplicate_indices, some_big_penalty)
On Fri, Sep 13, 2019 at 02:23:38AM -0700, pietrucha wrote:

I am also facing situation when I have to visit some node 2 times in a week. Should I create 2 virtual nodes for that kind? Complexity will be very high!!!

Yes you must create dummy nodes.  The solver can only visit a node at
most one time.  Once the node is visited, it is checked off the list
of places that must be visited.

@jmarca Example nodes:
- 0 - depot
- 1 - visited twice a week, TW (mon-fri: 6-7 day of the week doesn't matter, this node shouldn't be visited day by day), I create 5 virtual node representing 5 days like this:
    1' mon 6-7
    1'' tue 6-7
    1''' wed 6-7
    1'''' thu 6-7
    1''''' fri 6-7

- 2 - visited once a week, TW (mon-fri: 6-14, day of the week doesn't matter),
    2' mon 6-14
    2'' tue 6-14

- 3 - ...

1000 - ...

When truck visit node 2'', other 2 nodes should be marked as visited. How to mark those nodes?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/google/or-tools/issues/1428#issuecomment-531167026

--
James E. Marca


Autolinking in React-Native 0.60+ for Android

Is your feature request related to a problem? Please describe.
yes, there's no autolinking for android.
Describe the solution you'd like
use the autolinking feature
Describe alternatives you've considered
use the autolinking feature
Additional context
no other additional context

not stale, still waiting an answer

@wamry please provide more details. In what way are you trying to initiate autolinking, and why is it related to Detox?


BUG: Fix bug with parent dir check

Turns this cryptic error (from MNE-BIDS-Pipeline but the idea is clear I think):
│13:37:40│ 🐛 sub-01 run-noise A critical error occurred. The error message was: /mnt/ssds2/larsoner/mcq/natural-conversations-bids/derivatives/mne-bids-pipeline/sub-emptyroom/ses-20230412/meg/sub-emptyroom_ses-20230412_task-noise_proc-filt_split-01_raw.fif

Starting post-mortem debugger.
<traceback object at 0x7f11e2640d80>
> /home/larsoner/python/mne-python/mne/io/base.py(2606)__init__()
-> assert op.isdir(op.dirname(fname)), fname

into a useful one:
13:46:43│ 🐛 sub-01 run-noise A critical error occurred. The error message was: parent directory does not exist: "/mnt/ssds2/larsoner/mcq/natural-conversations-bids/derivatives/mne-bids-pipeline/sub-emptyroom/ses-20230412/meg"

Thanks @larsoner


"Connection" field  in media isn't recognized

sdp file
c=IN IP4 <IP_ADDRESS> is media field , but it's recognized as "Session Description Connection".
https://tools.ietf.org/html/rfc4566#page-9

@zombiecong thank you!
Should be fixed in v0.9.2. Please check if works for you.

@ernado Yes, it works. Thanks.


Skip forward and backward doesn't work.

skipping doesn't work play/ pause does.

Google's changed how playlists work on YouTube. Unfortunately, this means that skipping is broken. I'd be happy to accept a pull request fixing it!


Fix ffmpeg dependency and don't import project in setup.py

Fixes #1.
Also it's a bad practice to import the project in setup.py, as at the time of setup the environment likely doesn't have the required dependencies like numpy. There are a couple ways to resolve this while keeping the 2 versions in the same place, see https://packaging.python.org/en/latest/guides/single-sourcing-package-version/.

@kalvdans do you have write access to the original repository?

@kalvdans do you have write access to the original repository?

No, sorry. I can't merge, just helping out reviewing.


IDEA - ES6 destructuring

Hi,
ES6 is great for destructuring object and my though is about using it when the injection happened, so it will be much easier when having a lot of dependencies, to not need to take care about the order of the dependencies. My idea is something like that :
`module.exports = function ({ logger, mqttServer }) {
// work is done here
}
module.exports.$name = "JobBusiness"
module.exports.$inject = ["logger", "mqttServer"]`
Would it be possible ? Is-it a huge work ?
Thanks

Hi,
I did a first version with es6 destructuring .. maybe I can help with my code, if someone is interested in


T.Push is not a function error

I have an chart.appendSeries() inside a loop to add series that constantly updating every other time., After iterating im getting this error.
apexcharts.min.js:6 Possible Unhandled Promise Rejection: TypeError: t.push is not a function

I guess this answers the question - https://stackoverflow.com/a/62633281/1247864

I was having this issue


Add tests to guarantee to remove only expected resources

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Thank you.
/approve


Tag LfMerge branches when release is pushed (FW8)

GHA workflow will now tag the repo with same version number being put into the Docker image tags.
We also separate out the release workflow into its own file, as a first step towards someday building a truly reusable Docker-release workflow. (It's not there yet; this version of the workflow has some LfMerge assumptions, like branch structure, baked into it).
This will only run when the live branch is pushed, but I'm putting this PR into both the master and fieldworks8-master branches so that they'll continue to have identical workflows.

This change is

Thoroughly tested in my test repo before making this PR, so not waiting for a review.


y21ga | BFUS | v0.2.2: Continuous "click sound" from the cameras after updates?

Installed few Yi Hack cameras (also Dome X) around April 15 (previous firmware). Updated firmware in June.

The cameras are not rebooting.
There is SD on everyone with swap enabled.

If the camera is not in a boot loop the reason could be a the light of the room.
Dark to switch on ir leds and filter but not enough to mantain it.
So the cam switches on the night mode and then comes back to day mode.

My h51ga camera do the same sound, but only when switch video is closed and camera parked, I disabled the IR led and the sound is still there.

Another test, hide camera using PTZ commands (and status switch always ON) and no click sounds detected.

One more test, compiling with the latest commits make this sound disappear.

Which commit?

Sorry, I don't remember, I compiled it about 2 weeks ago, where I can find this information (SD, linux compilation dir)?

"2 weeks ago" is enough, thank you.

Hi is there a solution for this

I didn't find a fix, but I'm, not able to repeat this behavior with my cams.


Get-DbaAgentJob added test

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[X] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Added test to cover -Job parameter, should be 100% covered now!

awesome, thank you!


Update datediff-transact-sql.md

@pmasl : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

@pmasl : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.


Replace header-parsed vk_enum_string_helper.h with xml-based version

This version is generated from vk.xml requiring no changes to vulkan.py.  Other helper files will also use this generator in the future.

Updated, rebased and pushed to Master.


Fix typos in Keyberon entry

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @ryankurte (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks!


Stack plot bug part 1 issue 266

Please confirm that this PR has done the following:

[x] Tests Added
[ ] Documentation Added
[x] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
Addresses the first problem identified in #266

This fixes the first part of #266. The wrong image can be seen here with the correction available here. Do we want this added to the tutorial too or not worry about it as this is a bug fix?

Coverage decreased (-0.3%) to 87.848% when pulling f667f850ec4c53945fcdaa6b4c6e562a983e57d7 on znicholls:stack-plot-bug-part-1-issue-266 into dd7ae0cb63eee15e18dc74ca05f6b323293061fb on IAMconsortium:master.

thanks @znicholls! I don't think that this needs to go into the tutorial, as it just fixes unexpected behaviour...
Can you rebase, then I'll merge?

Can you rebase, then I'll merge?

cool done thanks @danielhuppmann !

It seems that appveyor upgraded to a new version of matplotlib, causing this error. I think I solved a similar problem by downgrading.
@gidden, does it make sense to do that here as a quickfix, too, or is it worthwhile to make the underlying use of matplotlib-internals more robust?

Took a quick look at the logs, specifically conda list.
Last success:
...
markupsafe                1.1.1            py37he774522_0
matplotlib                3.0.3            py37hc8f65d3_0
mistune                   0.8.4            py37he774522_0
...

This PR:
...
matplotlib                3.0.3            py37hc8f65d3_0
matplotlib-base           3.1.1            py37h2852a4a_1    conda-forge
mistune                   0.8.4            py37he774522_0
...

Let me do a quick test in #274

ok, looks like #274 has a fix. mind incorporating it?

mind incorporating it?

yep done


Cannot run individual tests when using "pytest"

Description of the feature request:
When using pytest, there are no play icons ▶ for running individual tests (individual methods, classes, modules) using Bazel.
It appears that only unittest.TestCase is supported, based on python/src/com/google/idea/blaze/python/run/PyTestUtils.java.
I'm happy to submit a PR for this if we can agree on the implementation.
Which category does this issue belong to?
Intellij, PyCharm
What underlying problem are you trying to solve with this feature?
Want to run run filtered pytest tests from the IntelliJ/PyCharm editor
What operating system, Intellij IDE and programming languages are you using? Please provide specific versions.
No response
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

To add support for pytest, we would need to detect:
a. Test Classes: any class with name "Test*"
b. Test Methods: any method with name "test_" (possibly inside a test class)
c. (for bonus points) Test Modules: any module with name "test_"
(a) and (b) are relatively easy - not so sure about (c) though.
In any case, we would need to agree...

Should this be an opt-in feature? If so, how should it be opted into and where should it be documented?
Should we use the "Python Integrated Tools -> Default test runner" setting?

Hi @jpgxs
I'm a user of the plugin and I'm able to select individual pytest tests to run. Debugging also works. See the screenshot.
I would suggest you check that your using the latest version of pycharm/intelij (+python plugin)

Having said that. When running an "individual test" it ends up running all test tests in that target (I'm not sure if this is expected behaviour or a bug). A question to the maintainers, is there a reason something like the following isn't passed?   bazel test //path/to/target:test --test_arg='-k {test_function_name}'

Thanks @JeroenSchmidt, I seem to get some interesting behaviour now that I'm looking closely.
If I define a test at the module level (not inside a class), I get the same behaviour as you:

However, for test classes (Test*.py) and their methods (test_*.py), I either get:

No play button
A play button with "Nothing here"
A play button to run all the tests in the Bazel test

It's not clear to me why (1) happens as it seems to have changed over the course of today; I think it's related to IntelliJ's native pytest integration + caching.
Currently, I get "Nothing Here" for all tests in the file:

This changes in all cases if I either:
(a) Define a test at the module level:

(b) Define a test class with name "*Test":

I think this logic is being handled here: https://github.com/bazelbuild/intellij/blob/76f1972336538639059dc26018dfcb49f4d41515/python/src/com/google/idea/blaze/python/run/PyTestUtils.java#L26

Of course, these aren't very useful if they run all the tests in the suite anyway; some of our py_test targets take 10+ minutes to run if you're running everything!
The unittest integration seems to pass though the --test_filter argument with TestClass.test_method / TestClass / test_method.
For pytest, I think it should either:
a. Pass it through with exactly the same scheme as unittest.
b. Pass it through in a pytest native format module.py::TestClass::test_method
Personally, I'm leaning towards (b) but it might be a bit trickier

I'm happy to submit a PR for this if we can agree on the implementation.
Thank you @jpgxs , you're welcome! cc @agluszak


dotnet aspnet-codegenerator scaffolder should put controller in controllers folder

dotnet aspnet-codegenerator scaffolder puts the controller in the project directory and the views in the correct location (Views folder)
dotnet aspnet-codegenerator controller -name Controllers/MoviesController  -m Movie -dc MvcMovieContext
puts the controller in the correct folder, but then the views are in the wrong place.

@Rick-Anderson You would need to pass in --relativeFolderPath  option to tell which folder to generate the controller in.
dotnet aspnet-codegenerator controller -name MoviesController -m Movie -dc MvcMovieContext --relativeFolderPath Controllers

Closing this.


Creating medusa2.

Turns out creating entire weights for the lm_heads costs a huge amount of VRAM (specially for multilingual models like Gemm) and is not necessary at all to get good speculation.
This PR modifies the legacy code to create new medusa models without duplicating this lm_head making it much more efficient to run. It also increments the version number of the config so users can know if how to actually run the model.

Good catch @Narsil.  This will probably lead to more stable training as well.


Is it possible to increase the history in event data?

At the moment the history array in event data contains the last 20 check exit codes. I my handler filter I do some logic based on the history to reduce noise to Slack.
  # The fifteenth occurrence after state change (role:windows 60th) or resolve after at least 15 times critical
  # Noise can be reduced for certain checks even more by setting the check.occurrences attribute larger than 15
  state_change_with_occurrences_critical:
    negate: false
    attributes:
      occurrences: "eval: (':::client.role:::' != 'windows' && value == :::check.occurrences|15:::) || (':::client.role:::' == 'windows' && value == :::check.occurrences|60:::) || ':::action:::' == 'resolve'"
      check:
        history: "eval: (value.last == '0' && value[-16...-1].uniq.length == 1 && value[-16...-1].uniq[0] == '2') || (value.last == '2' && value[-15...-1].uniq.length == 1 && value[-15...-1].uniq[0] == '2')"

But I like to have more than 20 history entries, let's say 100. Is that possible?

What I want to accomplish is improve the  ':::action:::' == 'resolve' part. To receive only a resolve message after the error/warning was during at least X time.

Hi @pietervogelaar.  Sensu stores the last 21 check result statuses to enable the flap detection algorithm and the number of entries is not currently configurable.
It seems like you might be trying to use filters to implement your own flap detection. Have you had a look at the check flap threshold attributes  that control the flap detection algorithm? These settings might help you make appropriate adjustments without needing to change Sensu's history registry, as you can use filters to evaluate whether or not the event action is 'flapping' .
I'd also recommend configure your handler(s) to use the built-in occurrences filter. This filter ensures that your configured check occurrences have been met before allowing the event to be handled. Because Sensu will AND together the filters defined on a handler, there shouldn't be a need to reproduce the occurrence logic in your own filters.

Hi @cwjohnston,
Thanks for your suggestion about flapping. However I don't think that is sufficient. On Slack I only want to receive events that require human attention. So even for a check that never flapped, but is for the first time critical, I only want to get notified if it's critical for a certain amount of time or X occurences.

I only want to get notified if it's critical for a certain amount of time or X occurences.

This is the purpose of the built-in occurrences filter. Please see our built-in filter reference documentation for details and examples.
Closing this issue for now as I believe the occurrences filter provides sufficient coverage for this use case.


Default material is assigned to meshes without material for glTF export

This fixes #57955 by assigning a default StandardMaterial3D to meshes without any material assigned.

Thanks!


Alto Sax Facts

https://github.com/instructlab/taxonomy/issues/803
If your PR is related to a contribution to the taxonomy, please, fill
out the following questionnaire. If not, replace this whole text and the
following questionnaire with whatever information is applicable to your PR.
Describe the contribution to the taxonomy

...
...
...

Input given at the prompt

   ...

Response from the original model

  ...

Response from the fine-tuned model

  ...

Contribution checklist

[ ] The contribution was tested with ilab generate
[ ] No errors or warnings were produced by ilab generate
[ ] All commits are signed off (DCO)
[ ] The qna.yaml file contains at least 5 seed_examples
[ ] The qna.yaml file was linted and prettified (yaml-validator can do both)
[ ] An attribution.txt file in the same folder as the qna.yaml file
[ ] Content does not include PII or otherwise sensitive or confidential information
[ ] Content does not include anything documented in the project's Avoid these Topics guidelines

Closing per: https://github.com/instructlab/taxonomy/pull/1177


Write CodePush guide

It took as a while to figure it out - it's not supported by default, so let's be proactive and describe how we made it.

I'm on it


Weird behavior when trying a horizontal swipe on a vertical calendar

(Required) Version Number: Version 8
Description

Set scroll mode to stop at each calendar frame.
Set scroll direction to vertical
Try multiple horizontal swipes
The odd behavior is it scrolls up and stops at an illegal location.
See video for better explanation.

Screen recording video link: https://streamable.com/qs0dw0
Here are my configs:
    calendarView.scrollingMode = .stopAtEachCalendarFrame
    calendarView.scrollDirection = .vertical

    ConfigurationParameters(startDate: startDate, endDate: endDate, calendar: self.calendar, generateOutDates: .tillEndOfRow)

I am seeing a corner radius effect.
There is no extra code like this in this library.
Do you have any corner radius code in your project?

It's just how a modally presented view behaves. When it senses the view being dragged, it adapts to the rounded edges of the iPhones display.
It there a way to somehow add another gesture recognizer in addition to the calendar view vertical one?

Nvm, I've added a pan gesture recognizer to the view of the modal, and it's eating up all the horizontal swipes. So that fixes the odd vertical scrolling behavior.
Regardless, If you think this is worth looking into, I think it has to do with the modally presented view, and a vertical scrolling calendar that stops at every frame.

I will investigate. But if someone can help looking into this i'd be grateful.

closing issue.
It seems like you have said.
Thanks.


fix(pay-form): ensure payment form shows on top

This fixes an issue where the payment form - initiated from an external request payment link - would display under the connect dialog if that was already open. This change ensures that the payment form always displays on the top most layer.
Fix #284

utACK e586dc5


Drop dependency on func command line tool

Plugin name and version
azure-functions-maven-plugin 1.13
Plugin configuration in your pom.xml
Irrelevant
Expected behavior
package and run goals do not require .net functions sdk and everything can be done without any additional tools
Actual behavior
[INFO] Step 8 of 8: Installing function extensions if needed
Microsoft (R) Build Engine version 16.10.2+857e5a733 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  All projects are up-to-date for restore.
  extensions -> C:\Users\User\IdeaProjects\azure-function-runtime\target\azure-functions\functionapp\bin\extensions.dll

Steps to reproduce the problem
Just install the .net function sdk for windows.
Looking at the source code, azure-functions-maven plugin just maps parameters to func tool, which is outright wrong for any java based project. In its current state the plugin does not even permit adding additional command line parameters, nor logs the commandline used to run the tool, forcing the user to monitor the running processes using get-wmiobject -filter "name = 'func.exe'" powershell scriptlet to even guess what the plugin is doing and mimic that while providing the missing parameters yourself.
In short, the plugin must be self contained.

Ping @Dragas

function core tools are written in .net and it is required for running functions locally. so I think .net function sdk is required, could you let me know what parameter you can add to 'func host start' to avoid the requirement for .net functions sdk?

What? How is that a requirement when a new JVM is spawned to run the function application?

Hi @WoodenAndrew , the function core tools repository is located here (managed by Azure Functions team), it must adopt one language since it support multiple languages, personally I don't think it is right to have multiple distributions of function core tools using different languages since this is an endless work.


Consider revising definitions (4)

The following 4 definitions contain disjunctions (OR statements), which will obstruct their classification in a multispecies context. It also seems that this makes them unnecessarily complex;

'cell death defective': Here I would consider reducing to GO:cell death, rather than regulation of cell death?
'courtship behavior defective': should probably simply be inheres in courtship behaviour.
'mating behavior defective': Same as above
'mating defective': Same as above

Maybe best wait with this change until https://github.com/FlyBase/drosophila-phenotype-ontology/pull/5 is merged, @Clare72

'cell death defective': I think removing GO 'regulation of cell death' better matches the text definition - the cause (e.g. regulation) does not matter. However, if we made a phenotype for 'regulation of cell death defective', this would then not be a 'cell death defective' phenotype because GO 'regulation of cell death' does not (and should not) have an is_a relationship to GO 'cell death'. Personally, I think this would be fine as defective regulation may not necessarily result in actual changes to the number of cells dying (which is the phenotype here).
'courtship behavior defective': In GO, 'male courtship behavior' is not a 'courtship behavior', but a 'mating behavior' that is part_of some 'courtship behavior'. Looks like this happened due to this ticket: https://github.com/geneontology/go-ontology/issues/4165, but it is not clear why part_of rather than is_a. We need 'courtship behavior defective' to also cover gender-specific behaviors, so I think this needs fixing in the GO. Current logical def for 'courtship behavior defective' looks like a workaround for this problem.
'mating behavior defective': Looks like this should be fine as everything in GO (including 'male courtship behavior') is linked up by is_a relationships (although tree structure is not perfect).
'mating defective': Changing this would mean that 'mating behavior defective' is no longer an is_a child, which is probably fine. I don't think a small change in pulse song frequency or wing extension angle should infer 'mating defective' as males might otherwise mate normally, but others may disagree. This is similar to part_of issues discussed on the calls i.e. should an abnormal component of a thing mean an abnormal thing.
@dosumis - looks like you made all four of the terms in question, do you have anything to add?

Noticed these inferences were lost in new simple release:
+ SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000399> <http://purl.obolibrary.org/obo/FBcv_0000721>)
+ SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000721> <http://purl.obolibrary.org/obo/FBcv_0000387>)
+ SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000721> <http://purl.obolibrary.org/obo/FBcv_0000400>)

These ones were added in the new dpo-simple release:
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000399> <http://purl.obolibrary.org/obo/FBcv_0001347>)
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000401> <http://purl.obolibrary.org/obo/FBcv_0000400>)
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000402> <http://purl.obolibrary.org/obo/FBcv_0000387>)
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000402> <http://purl.obolibrary.org/obo/FBcv_0000400>)
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000402> <http://purl.obolibrary.org/obo/FBcv_0000721>)
- SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000721> <http://purl.obolibrary.org/obo/FBcv_0001347>)

@Clare72 I think this is all due to what you discovered here, but if you want, you can verify whether this is ok for now.

Translation:

SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000399 http://purl.obolibrary.org/obo/FBcv_0000721) [removed]
SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000399 http://purl.obolibrary.org/obo/FBcv_0001347) [added]

'courtship behavior defective' falls under 'phenotype' rather than 'mating behavior defective'

SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000721 http://purl.obolibrary.org/obo/FBcv_0000387) [removed]
SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000721 http://purl.obolibrary.org/obo/FBcv_0000400) [removed]
SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000721 http://purl.obolibrary.org/obo/FBcv_0001347) [added]

'mating behavior defective' falls under 'phenotype' rather than 'behavior defective' and 'mating defective'

SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000401 http://purl.obolibrary.org/obo/FBcv_0000400) [added]

'mating rhythm defective' additionally falls under 'mating defective'

SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000402 http://purl.obolibrary.org/obo/FBcv_0000387) [added]
SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000402 http://purl.obolibrary.org/obo/FBcv_0000400) [added]
SubClassOf(http://purl.obolibrary.org/obo/FBcv_0000402 http://purl.obolibrary.org/obo/FBcv_0000721) [added]

'song defective' additionally falls under 'behavior defective', 'mating defective' 'mating behavior defective'

I also see these 'changes' in the old edit file when reasoned with ELK, reasoning with hermit fixes it. The new edit file is taking a ridiculously long time to reason with hermit, but I suspect that it will solve the problem.

If course!!! I am so dense! That’s why I wanted these patterns changed in the first place. Elk does not do OR! I will try to convince David today and that’s it. All else later.

The last subclass axiom missing is this one:
SubClassOf(<http://purl.obolibrary.org/obo/FBcv_0000721> <http://purl.obolibrary.org/obo/FBcv_0000387>)


[release-3.0] copr, future_pool: Add more metrics

Signed-off-by: Breezewish<EMAIL_ADDRESS>
What problem does this PR solve?
This PR is a cherry pick for https://github.com/tikv/tikv/pull/7398

/run-all-tests tidb=release-3.0 tidb-test=release-3.0 pd=release-3.0

/run-all-tests tidb=release-3.0 tidb-test=release-3.0 pd=release-3.0


feat(datepicker): initial implementation

This is just the simplest implementation ever.  Not to merge yet.
I just want to test the waters, get feedback and give it some extra love.
So the idea is to extract all the logic into a separate Engine so we can release (in a different repo) a MomentEngine and people can implement their own if needed.
I created a bindings file that every people will need to add to their bootstrap:
bootstrap(App, [NGB_BINDING])

I guess we will have more stuff to add to that with other components.
That engine has just one method (for now) that simply generate an array of days.
I put the styling inline because I have no idea in how to make it on a separate file and make it work with TSC and the UMD bundle.
And of course, the whole design is just a prototype (but I would like to keep it simple, our uibs one is a space shuttle with that many buttons).
http://plnkr.co/edit/y8YSQrLl39ECLXRxaqNJ?p=preview
I also realized that my design is coupled with the glyphicons.css I use on my playground.

Another thing to mention is bindings or providers or ... ?

Also, wasn't sure where to put that bindings file nor how to export it, I accept suggestions.

To be clear on this, the DatepickerEngine will be provided upon bootstrap of the application by the user, and that is how it will be injected into the component?

Yes, all users will need to inject the bindings on bootstrap. Now we will have 1 with this Engine but we can potentially have 5-10 (for upcoming components).
So the idea to inject it in the bootstrap is that it makes the engine available everywhere to inject (including our datepicker).
So if we have bootstrap -> A -> B -> Datepicker (And A component using a B component that uses the datepicker).Datepicker will ask B if he defined a engine, nope, A, nope, on bootstrap? Yes.
The nicety in here is that you can do in B (or A):
providers: [provide(DatepickerEngine, {useClass: MyCustomDatepickerEngine})]

Now when the datepicker ask B if it has a engine, well, it does so the datepicker is injected with that one instead.
So having the interface as a contract, people can create their own engines and use them where they like to.

So I guess the thing we really need to get right is the engine then - it's a little different than I was originally thinking of for an implementation, but could be fine.
I need to think about this some.

I think that this work is stale to the point that there is no reason to keep this PR open. Let's close it to keep the queue clean and re-open when this PR is ready.


[ENHANCEMENT] Prohibit users from resizing volume when vm is stopping.

Is your enhancement related to a problem? Please describe.
Currently, this PR https://github.com/harvester/harvester/pull/4794 solution doesn't fix the problem which users could resize the volume when vm is stopping.
When we stop vm from kubevirt, its RunStrategy and PrintableStatus will be like this:

RunStrategy: RerunOnFailure, , PrintableStatus: Running
RunStrategy: Halted, , PrintableStatus: Running
RunStrategy: Halted, PrintableStatus: Stopping
VMI still exists
At this moment, it takes some time to stop vm, so users could edit vm config to resize volume.
RunStrategy: Halted, PrintableStatus: Stopped
VMI is gone

Describe the solution you'd like
We should change condition order, move PrintableStatus != Stopped to first one.
Describe alternatives you've considered

Additional context

Test PASS on harvester-v1.4.0-dev-20240830.
Environment

harvester-v1.4.0-dev-20240830

Profile: 2 nodes QEMU/KVM (8C/16G/500G per node)
ui-source: Auto

Prerequisite

 Fresh install harvester with any nodes

 Create a VM, check Running.

Steps

 VM => ⋮ => Edit Config, should NOT be able to resize. :heavy_check_mark:

 Stop VM => ⋮ => Edit Config when Stopping, should NOT be able to resize. :heavy_check_mark:

 Check VM Stopped => ⋮ => Edit Config, should be able to resize. :heavy_check_mark:

Resizing

Ready

 Start VM, check Running and disk do resized. :heavy_check_mark:

VM Running

Volume do resized

Note

harvester/harvester/pull/5409


ESC exit issue in v1.3 configuration menu

It appears that the v1.3 configuration program will not update the DVI output when using ESC to exit from a sub-menu. The program does in fact exit, and the composite output shows the exit message and a floating point prompt, but the DVI output still shows the configuration menu.
As an example (using the DOS 3.3 disk image), press S for save to flash, and immediately press ESC. The configuration menu will remain on the DVI output. only a reset will clear the configuration menu from DVI.
On the other hand, If you were to have pressed a key that brought you back to the main menu, the program would exit normally on DVI and composite.

Thanks for the report. I see what has happened. Will be an easy fix.
For the moment, just press any other key after entering a submenu or pressing save - except for ESC. Then press ESC afterwards.
Or just press CTRL-RESET - like one usually needs to anyway, to boot into any other program.


Getting started tutorial and child device tutorial documentation

Proposed changes
Updated getting-started.md, added images. Added child-device.md

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[x] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue
#1512
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments

Superseded by https://github.com/thin-edge/thin-edge.io/pull/1576


TypeError: Cannot read property 'connect' of undefined

Hi, I'm receiving
TypeError: Cannot read property 'connect' of undefined
at socketFactory (http://localhost:3000/bower_components/angular-socket-io/socket.js:31:44)
by calling:
socket.emit('client', newValue);
I have 'btford.socket-io' on my dependencies and added:
.factory('socket', function (socketFactory) {
var mySocket = socketFactory();
mySocket.forward('server');
return mySocket;
});
and on my controller I injected 'socket'
What am I missing?

/facepalm, opened this on the wrong project.


Overflow Boxes for Dialogs and Windows

Feature Request:
I use Modal dialogs in Metro CSS (Moving an former UIKit 3 project to Metro CSS) for adding and editing data from forms, and kinda messes up when is used in mobiles, specially when the form has many fields, and once they overflow, there's no way to scroll back to the rest of the controls in the dialog, only the page itself, the dialogs will remain static.
UIKit3 has an component called uk-overflow as seen here which allows multiple content and controls to be inside of an dialog.
Thanks.

https://codepen.io/olton/pen/RyVjxw


Make APNSClient Sendable if coders are Sendable

JSONEncoder and JSONDecoder also conform to Sendable, making the most commonly used APNSClient then Sendable as well.

@FranzBusch sure, running checks

ThreadSanitizer: unexpected memory mapping

Hmm, is this TSan failure related to the new conformance? Seems unlikely to me...

GitHub have screwed up their runners on Linux so any LLVM ASAN/TSAN checks are failing. They have a fix that's due to deploy this week (I believe Friday) but as a workaround if you want it you can check out the issue which contains details and the workaround https://github.com/actions/runner-images/issues/9491#issuecomment-1999992492


Option to force a text case on all new tags

It would be nice to have some way to force a given case on all tags (uppercase, lowercase and none).

It would be awesome to add a way to force inflection as well...
titleize
singularize
pluralize
etc.

The proposed behavior can be easily achieved by using the onTagAdded option, so there's no need to bake it into the directive's code.


UnicodeDecodeError for new version

Description of the problem
When I read some .edf files by read_raw_edf() function, the codes from Version 0.24.1 succeed to read all the files. However, the codes from Version 1.1.1 succeed to read some files, but report an error while reading one file as:

UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc8 in position 34915709: invalid continuation byte

Steps to reproduce
self.raw_data = read_raw_edf(raw_data_path, preload=False)[:][0]
where `raw_data_path` includes a .edf file.

Link to data
No response
Expected results
Succeed to read all files.
Actual results
File "/home/xxx/anaconda3/lib/python3.7/site-packages/mne/io/edf/edf.py", line 1387, in read_raw_edf
verbose=verbose)
File "", line 12, in __init__
File "/home/xxx/anaconda3/lib/python3.7/site-packages/mne/io/edf/edf.py", line 158, in __init__
onset, duration, desc = _read_annotations_edf(tal_data[0])
File "/home/xxx/anaconda3/lib/python3.7/site-packages/mne/io/edf/edf.py", line 1590, in _read_annotations_edf
triggers = re.findall(pat, tals.decode('utf8'))
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc8 in position 34915709: invalid continuation byte
Additional information
Platform:         Linux-4.15.0-188-generic-x86_64-with-debian-buster-sid
Python:           3.7.11 (default, Jul 27 2021, 14:32:16)  [GCC 7.5.0]
Executable:       /home/xxx/anaconda3/bin/python
CPU:              x86_64: 256 cores
Memory:           755.8 GB
mne:              1.1.1
numpy:            1.20.3 {blas=mkl_rt, lapack=mkl_rt}
scipy:            1.7.3
matplotlib:       3.5.1 {backend=agg}
sklearn:          0.24.2
numba:            0.54.1
nibabel:          Not found
nilearn:          Not found
dipy:             Not found
cupy:             Not found
pandas:           1.3.5
pyvista:          Not found
pyvistaqt:        Not found
ipyvtklink:       Not found
vtk:              Not found
qtpy:             2.0.1 {PyQt5=5.9.6}
ipympl:           Not found
pyqtgraph:        Not found
pooch:            v1.6.0
mne_bids:         Not found
mne_nirs:         Not found
mne_features:     Not found
mne_qt_browser:   Not found
mne_connectivity: Not found
mne_icalabel:     Not found

or changelog https://mne.tools/stable/whats_new.html says:

Annotations contained in EDF files are correctly read as UTF-8 according
to the EDF specification (#10963
https://github.com/mne-tools/mne-python/issues/10963 by Clemens Brunner
https://github.com/cbrnr)
So I would conclude that the EDF file you have does not follow the EDF file
standard but I am not an EDF expert
how were these files written?
Message ID: @.***>

这是来自QQ邮箱的自动回复邮件。

您好呀~我是陈俊儒，我已收到您的邮件~我会尽快查阅的。如果您不认识我，请核对下您的发送地址。

I have exported these .edf files from original data by the Neuro Workbench software (from NIHON KOHDEN). The transformation is executed in the software and I did not use explicit codes to do this.

I am not sure that whether the commercial software still follows old encoding type latin-1, which causes the error report.

do you have the problem with other EDF readers?

Message ID: @.***>

According to the EDF specification, TAL channels are encoded as UTF-8. Previously, we used ISO 8859-1 (aka Latin-1), which worked with files that are apparently very common in practice, but are not compliant with the standard.
I prefer sticking to the standard UTF-8 encoding, but we could think about introducing a new parameter to let users change the annotation encoding if this is really common. Alternatively, we could also try/except UTF-8 and Latin-1 encodings, which would neither require a new parameter nor any user input.

I would be ok with a new param to specify the encoding. Maybe
tal_encoding or just encoding?

Message ID: @.***>

Fine with me. I think encoding="utf8" would be a good choice if we document it that this refers to annotation channels (other encodings cannot and should not be changed since e.g. header encoding is limited to ASCII).

works for me.
Can you take care of the PR? thx a lot

Message ID: @.***>

Can you take care of the PR?

Sure, will do!

@agramfort @cbrnr Thanks a lot for considering this situation. I will use the new function later.


Add Options to FSx Lustre tests

Signed-off-by: Sean Smith<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

might be bad form to give an approval when the CI check is failing, but seeing as how it's a "line is too long" error i'll do so anyways.


NEXT-8566 Fix mailto/fax/tel links in WYSIWYG editor

1. Why is this change necessary?
To fix issue https://issues.shopware.com/issues/NEXT-8566
2. What does this change do, exactly?
Update the REGEX to also detect 'mailto:', 'fax:' and 'tel:' links
3. Describe each step to reproduce the issue or behaviour.
Add a<EMAIL_ADDRESS>link in the wysiwyg editor. It will add http:// in front of it.
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-8566
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey : )
We've just set your PR to “Scheduled”.
This means our initial screening is passed.
Your PR will now be reviewed and tested before being put trough our internal extended testsuite and review process.
We will keep you informed wether something has to be done by you or when the PR is merged : )
If you have any questions feel free to ask.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-16437
Please use this issue to track the state of your pull request.

thanks for your contribution :+1: :tada:


404 at /desktop/extensions/

I found a broken link : https://docs.docker.com/desktop/extensions/

Hey! @oleksis Thanks for reporting the broken link issue. Could you let us know how you arrived at the broken link?

Could you let us know how you arrived at the broken link?

I was testing the Docker Extensions [Docker Desktop 4.7.0 (77074)] link for build my own extension.

Fixed in Docker Desktop 4.8.0 or a later.


Composite physical layers even if elevation is 0

Description
Based on pre-existing logic, we were only compositing physical layers if the elevation was > 0.  This causes RenderFlex to not actually clip overflows, which broke internal tests.  I've added a golden test for this case and changed it so we always composite physical models.
Related Issues
#27891
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors (See Test Coverage).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes).
[x] No, this is not a breaking change.

I'm not comfortable landing this without some further investigation on the impact on Fuchsia.  Closing for now and will revert instead while I investigate.


Unit test fail 'ACL HTTP Origin Should PUT new ACL file'

Cloned repo, did npm install. Then ran the tests, npm test. One test is failing:
  1) ACL HTTP Origin Should PUT new ACL file:
     Uncaught AssertionError: expected 403 to equal 201
      at Request._callback (test/acl.js:243:24)
      at Request.self.callback (node_modules/request/request.js:198:22)
      at Request.<anonymous> (node_modules/request/request.js:1035:10)
      at IncomingMessage.<anonymous> (node_modules/request/request.js:962:12)
      at _stream_readable.js:908:16

Am I missing any setup or configuration steps?
(Let me know if you need any debug log messages).

Ha! After a few inspection we basically create some files that we never remove, so if you run the test twice the file is already there in place, hence this test fails because you can't create something that is already there.
So, an easy fix would be to remove the file/folder at the beginning of the test

@nicola - I notice that the test/acl.js Cleanup step was changed to skipped, here: https://github.com/linkeddata/ldnode/commit/cd42e336e397af3b535a0ad77d5cddef680f8f1b#diff-c65e0d598fa4659ff66a65afb2057ee3L1044
Was that intentional? The acl tests don't seem to clean up after themselves now (here's my git status after the tests run):
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	test/resources/acl/origin/test-folder/
	test/resources/acl/write-acl/default-for-new/
	test/resources/acl/write-acl/empty-acl/another-empty-folder/
	test/resources/acl/write-acl/empty-acl/test-file
	test/resources/acl/write-acl/empty-acl/test-folder/
	test/resources/acl/write-acl/test-file
	test/resources/acl/write-acl/test-file.acl

Do we need to re-do the cleanup code, or add the test/resources/ dir to .gitignore?

Thanks for pointing this out. I plan to fix this issue, once done with the #164.
The cleanup step has been commented because it is from the old tests (soon to be deleted) - that I refactored - mostly completely - just before christmas.
I would not put them on .gitignore, I would just make sure I remove the folders and files once the test is done (or if possible at each test, in order to make the tests independent of each other) and put a cleanup function at the beginning of the test (not as a test, but as a before())

This is solved by https://github.com/linkeddata/ldnode/commit/b76a9f2e55b54beb4c7b8ab5a7d2821335aa86f3


[Bug] yazi missed environment variables

Steps to Reproduce

Use sway keybind mod+t to run yazi
Try to open any file via the editor.

Expected Behavior
It'll fail with 127 code, no vi program
Additional Context
vi is default editor, so yazi don'tm catch my .zshrc variable

.zprofle doesn't work, because I'm not login to the shell directly (tuigreet+greetd).

Hm, what if try to use .zprofile with kinking it from .zshenv, and then import to sway?


Allow customizing format output

I often want to sanity check some math or similar, and evcxr_repl works wonderfully for this for the most part (previously I had been using lldb/gdb for this which works about as well as you might imagine...).
Unfortunately, if doing bitwise math, I often want to see hex output. Doing so requires outputting something like format!("{:x}", <expression I want to check>); each time, which is tedious.
Taking a look at the source, it seems somewhat tricky since you don't know the type in advance, but one (somewhat hacky) option would be something like allowing specifying a list of formatting options to try, which would be used here, or something. e.g. if I set :fmt ["{:x}", "{:?}"] (or whatever), then the system would try {:x} followed by {:?} for things where evcxr_display doesn't apply.
That said, I'm flexible and willing to accept something that requires annotating things manually if the above solution is undesirable for some reason.

Interesting. I'd not rule out something like that if there was sufficient demand for it. In the meantime, perhaps the following would suffice?
pub trait HexDisplay { fn evcxr_display(&self); } impl HexDisplay for i32 { fn evcxr_display(&self) {println!("EVCXR_BEGIN_CONTENT text/plain\n0x{:x}\nEVCXR_END_CONTENT", self);} }
This adds a custom text display format for i32, after which i32 values will be printed as hex:
>> 8191
0x1fff

For anybody interested, I actually ended up solving this for myself with a "prelude" of sorts in my init.evcxr: https://gist.github.com/thomcc/8b308c30ddc35eb07646b4bb2ade480f (contains examples)
The downside of this is that init.evcxr is kinda a pain to work with (long lines), but other than that it works like a charm.

Wow that's a lot of code to put on one line. Fixing it might just be a matter of removing the line splitting code from the init.evcxr handling code. I think at the the time that code was written evcxr didn't support an input containing multiple commands, but now it does.


feat/atlas react website

Update main page with new deisgn

Hello, I'm from the Algolia crawler team (that powers DocSearch), and we've noticed recently bursts in bandwidth coming from your website. The timing seems to indicate that it can be related to this change.
Basically most of your pages now weight 1.55MB:
$ curl https://atlasgo.io/getting-started/guides/testing/guides -I
HTTP/2 200
content-type: text/html
content-length: 1548783

The problem is that even the 404 have this behaviour, they are actually responding 200 and have a 1.55MB content-length, to only later display a 404 message. Also they expose relative links, leading to infinite paths such as: https://atlasgo.io/cloud/getting-started/guides/cloud/getting-started, https://atlasgo.io/cli/declarative/apply/guides/mysql/getting-started/getting-started, ...

Thanks for reporting this to us, @sbellone. I appreciate it 🙏
We'll start by fixing these relative links immediately and investigate the 404 behavior. I'll keep you updated on our progress.

Hello @sbellone, we pushed a fix to production today. The not found page no longer returns 1.55MB in size anymore. And the status returns correctly to 404.
Thanks for your report here. We appreciate it.
$ curl https://atlasgo.io/getting-started/guides/testing/guides -I
HTTP/2 404
content-type: text/html
content-length: 12061

Thanks for notifying! I can confirm that your DocSearch crawler is back to crawling a small amount of pages which all have a reasonable payload :slightly_smiling_face:


Fix render doesn't recognize relative partial path with haml

This is a 🐞 bug fix.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[ ] The test suite passes (run bundle exec rspec to verify this)

Summary
Haml gem injects a middle layer between render and Jets::RenderingHelper. https://github.com/haml/haml/blob/master/lib/haml/helpers/action_view_mods.rb#L6. Because of this, folder cannot be extracted correctly from callers.
Context

Version Changes

Released in v2.3.8


Permission denied running Git hooks

When attempting to run NPM scripts in a Git pre-commit hook, wslgit fails with Permission denied.
What's odd is I can run the NPM script manually in WSL with the same user used by wslgit (checked with echo $USER in the pre-commit hook).

Can you check if you can run it manually from outside of WSL using something like:
wsl -e /bin/bash -c "git commit -m 'My commit message.'"

This is more or less the command that gets run by wslgit. If that does not work, you could check if it works when running an interactive shell (using the i flag):
wsl -e /bin/bash -ic "git commit -m 'My commit message.'"

An interactive shell is usually required to apply your .bashrc file, and for some commands (like push, pull, etc.) wslgit automatically uses interactive shells. This can be configured as described here.

I got the same Permission denied error for both

I managed to narrow down the issue. Somehow the PATH is not set correctly when running in wslgit/wsl -e.
I am using asdf-nodejs to manage my Node.js versions, but its not working when running in wslgit/wsl -e because ~/.asdf/shims is not in the PATH.

Where does asdf setup the correct PATH? After a quick look in the documentation, it seems this is usually done in .bashrc for the Bash shell. .bashrc should be loaded by an interactive shell (with the i flag) in most configurations.
You can check if .bashrc is executed by putting something like echo "Executing .bashrc" at the top of .bashrc, and repeat the manual commands from above (for using wslgit you must remove this output again, it will not work otherwise).
If there is no output, what distribution are you using? Do you have a .profile, .bash_profile or .bash_login file in your home directory?

Ah, my bad, you hit the nail on the head, turns out I have both ~/.bashrc and ~/.bash_profile, which was causing the ~/.bash_profile to be ignored. Thanks and sorry for the trouble


[BUG] CheckExistenceById throws 400 Bad Request

Describe the bug
I'm trying to check if an Azure resource exists by it's full Id, but I always get back a 400 bad request from ARM when using the ResourcesManagementClient class.
Expected behavior
The response should return either 404 or 204, depending on if the resource exists
Actual behavior (include Exception or Stack Trace)
ARM always returns 400 Bad Request
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)
// Make sure you run "az login" and set your account to the proper subscription before running this code.
string subscriptionId = "YourSubscriptionid"
string resourceId = $"/subscription/{subscriptionId}/...." // Full resource id here
var resourceManagementClient = new ResourcesManagementClient(subscriptionId , new AzureCliCredential());
var response = resourceManagementClient.Resources.CheckExistenceById(resourceId, "2021-04-01"); // latest API version

// The above line throws a RequestFailedException

Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0]: Azure.ResourceManager.Resource 1.0.0-preview.2
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [e.g. Azure AppService or Windows 10 .NET Framework 4.8]:

dotnet --info
.NET SDK (reflecting any global.json):
 Version:   5.0.204
 Commit:    84d1fe1bb7

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19043
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\5.0.204\

Host (useful for support):
  Version: 5.0.7
  Commit:  556582d964

.NET SDKs installed:
  3.1.410 [C:\Program Files\dotnet\sdk]
  5.0.204 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET runtimes or SDKs:
  https://aka.ms/dotnet-download

IDE and version : [e.g. Visual Studio 16.3]: Visual Studio Code 1.57.0

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@HarveyLink please help on this issue

Hi @nobody-msft ,
Azure.ResourceManager.Resource 1.0.0-preview.2 has been deprecated. You could use our re-designed: Azure.ResourceManager.Resource as a work around. Code will like this:
ArmClient armClient = new ArmClient(new DefaultAzureCredential());
Subscription subscription = await armClient.GetDefaultSubscriptionAsync();
var result = await subscription.GetGenericResources().CheckIfExistsAsync("resourceId");


Fixed the broken link in ctc_asr.md

Fixed the broken link for png in "Visualize the data" section

The link is correct, it's just relative to keras.io


Update alpha API to beta

What type of PR is this?
/kind api-change
What this PR does / why we need it:
Change alpha API to beta
Does this PR introduce a user-facing change?:
Renamed v1alpha1 API to v1beta1. The API is the same, but users must use the new version.

/lgtm

I didn't expect that to go through without approval, apologies @xing-yang

I didn't expect that to go through without approval, apologies @xing-yang

Looks like something causes me to auto-approve my own PRs. I wonder if that can be reversed.

@xing-yang Any feedback/concerns you have I can address in a new PR, just let me know.

@chrishenzie @bswartz no worries.  I'll take a look.  If I have comments, Ben can address with a followup PR.


C#: offload work from GrpcThreadPool by default.

Tentative fix for https://github.com/grpc/grpc/issues/11824
There's noticeable performance impact (600 -> 380KQPS  ping pong roundtrips on 32core), but offloading to a different thread has some extra safety features (it's hard to deadlock youself) and might even increase throughput in scenarios where it takes long time to serialize/deserialize requests or processing them is CPU intensive. Adding an experimental knob to allow turning off the offloading for those who really want to.

CC @jskeet and @benwulfe  as this should fix #11824.
CC @chwarr I remember you mentioning the issue this is trying to address.

If a request still needs to hit the grpc thread before being offloaded,
then it wouldn't fix it.
On Jul 25, 2017 6:00 AM, "Jan Tattermusch"<EMAIL_ADDRESS>wrote:

CC @jskeet https://github.com/jskeet and @benwulfe
https://github.com/benwulfe as this should fix #11824
https://github.com/grpc/grpc/issues/11824.
CC @chwarr https://github.com/chwarr I remember you mentioning the
issue this is trying to address.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/11930#issuecomment-317729409, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH7cpZRAPpTZ8A-EQzO0KCsbozE24Lbxks5sRebUgaJpZM4OihEt
.

Oh this should hide grpc thread from the outside world another way. Lgtm
then and thanks for fixing!
On Jul 25, 2017 7:34 AM, "Benjamin Wulfe"<EMAIL_ADDRESS>wrote:

If a request still needs to hit the grpc thread before being offloaded,
then it wouldn't fix it.
On Jul 25, 2017 6:00 AM, "Jan Tattermusch"<EMAIL_ADDRESS>wrote:

CC @jskeet https://github.com/jskeet and @benwulfe
https://github.com/benwulfe as this should fix #11824
https://github.com/grpc/grpc/issues/11824.
CC @chwarr https://github.com/chwarr I remember you mentioning the
issue this is trying to address.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/11930#issuecomment-317729409, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH7cpZRAPpTZ8A-EQzO0KCsbozE24Lbxks5sRebUgaJpZM4OihEt
.

Hey Jan, I know you had previously suggested using fewer completion queues to fix this.  Is there any advantage to having more than one completion queue?  It seems that with a single queue serviced by all threads, you achieve more fairness (and far lower deadlock chances).  With multiple queues, you could get an operation stuck indefinitely behind a fairly long operation while the other queues are happily processed, right?
This change lgtm -- but if this change has drawbacks, maybe the immediate fix would be to make the default value of  completionqueuecount = "1"?

One comment:
It seems like issues with the current thread pool are not just for sync unary RPC types. But if getting rid of the deadlock issue for sync RPC types was enough, then could we use grpc_completion_queue_pluck for these call types and completely avoid the thread pool for them, in order to keep the current thread pool for it's performance?


delete and redeploy of same app sometimes uses old revision

i tried deleting (from the UI) and redeploying my app on Fermyon cloud and sometimes it seems to be serving old revision (until I deploy the app 2nd time).

Does #870 affect this?  I saw there was a lengthy chat thread, are there any additional details or conclusions that are worth capturing here?

just that we were able to theoretically narrow down issue to this line here: https://github.com/fermyon/spin/blob/4f2251244fa17f36abca899bb3e775ae718aa45f/src/commands/deploy.rs#L274
may be @kate-goldenring can add more meaningful insights here.

We are using UseRangeRule when an app is being created and UseSpecifiedRevision when an app is being modified (redeployed with changes). Since a revision does not exist until an app is created, UseRangeRule is used to have Cloud select a revision. Then, on updates, deploy queries for an app and uses the existing revision to deploy the changes to. There may be some race case here where the old revision is not being cleaned up so we are seeing it and using it rather than creating a new revision with a UseRangeRule.

Here is where we check in deploy.rs if the app exists. This call to get existing apps may be incorrectly returning the app that was deleted causing that revision to be reused rather than creating a new one.

@rajatjindal did this cause any user errors? I dont see it affecting the user experience but it is a clear inconsistency that could cause problems in the future and probably makes testing/data gathering inconsistent

if a user deletes and recreates the app, he will see old version of his app running, which IMO will confuse the end user.
I was able to verify this by deploying an app with small changes, and on 3rd attempt, it reused the version from 1st attempt and therefore ran the old code.

I believe what is happening is that during create app command, we trigger Import Revisions command which imports all existing revisions from bindle using the storageId and associates them with the current app id
and now when we try to deploy  using ‘Range Rule’, it picks up the random version  from that list of revisions and deploy that.
but because we already have bindle_id during deploy command, we can get the active-revision-id as follows and then call add_channel with specific version.
 let active_revision_id = self
                    .get_revision_id_cloud(&client, bindle_id.version_string().clone(), app_id)
                    .await?;

i was able to verify this by using patched spin to deploy and it seems to be working as expected.

i am still  trying to see if there is a better way to fix this. will report back