operator vsphere-kubernetes-drivers-operator (0.1.0)

Please review the instructions for submitting an operator bundle for certification.

Please review the instructions for submitting an operator bundle for certification.

@tonytcampbell Thanks for pointing out to the guide, Just curious to know if there is any way to see the error logs to identify the failed tests?

Please review the instructions for submitting an operator bundle for certification.

@tonytcampbell Thanks for pointing out to the guide, Just curious to know if there is any way to see the error logs to identify the failed tests?

You will be able to see logs in Connect but there is a known issue with that feature that we are working on at the moment.

Please review the instructions for submitting an operator bundle for certification.

@tonytcampbell Thanks for pointing out to the guide, Just curious to know if there is any way to see the error logs to identify the failed tests?

You will be able to see logs in Connect. There is a known issue with that feature that we are working on at the moment but you should have logs for your last attempt.

Thanks @tonytcampbell , currently we are not able to see even the last logs in Red Hat Connect Portal, I tried running the same thing in local hosted crc with minimal pipeline run, I am currently facing below problem
[yaml-lint : lint-yaml-files] [Errno 13] Permission denied: 'operators/vsphere-kubernetes-drivers-operator/0.1.0'

It would be great if you can help us to point out the issues with this

@asifdxtreme Please change the namespace field in your CSV to "placeholder" instead of "vsphere-kubernetes-drivers-operator".

@asifdxtreme Hello! There are some concerns around this Operator in future versions of OpenShift. I tried reaching out to the email from your account but it won't allow external emails. Can you please share an email address I can reach out to you with?

@taylorjordanNC you can contact us on<EMAIL_ADDRESS>for any queries related to this operator, we will be happy to make this compatible with future releases of Openshift


Fix extension replacement

Replace only the extension at the end of the filename
Existing code would replace the first instance of "ts" and the . in the extension was interpreted as a RegExp "match anything" rather than a dot literal.

Oh awesome, thanks!


entering an "=" unexpectedly breaks "in" at a distance

given:

entering "=" (was trying to insert arrow type before remembering we dont have them) yields:

breaking the in

hmm interesting... I've been allowing breaking the edit site's parent bidelimited container bc that's needed for eg turning [a,b,c,d] into [a,b]@[c,d] just by typing over middle comma. but maybe should just be a brace-specific thing


[BUG] No "reward" in Tensordict, works good after my patch

I am sharing here a module I had to create to work with torchrl, at least for the environment I am creating (which can be found here).
__all__ = [
    "EnvBase",
    "ParallelEnv",
    "step_and_maybe_reset",
]

from loguru import logger
import typing as ty
from tensordict import TensorDict, TensorDictBase
from torchrl.envs import (
    DTypeCastTransform,
    EnvBase as EnvBaseRL,
    ParallelEnv as ParallelEnvRL,
)
from torchrl.envs.utils import _terminated_or_truncated, step_mdp
from torchrl.data import TensorSpec

class EnvBase(EnvBaseRL):
    """Patched EnvBase..."""

    def step_and_maybe_reset(
        self,
        tensordict: TensorDictBase,
    ) -> ty.Tuple[TensorDictBase, TensorDictBase]:
        """Patched."""
        return step_and_maybe_reset(self, tensordict)

class ParallelEnv(ParallelEnvRL):
    """Patched ParallelEnv..."""

    def step_and_maybe_reset(
        self,
        tensordict: TensorDictBase,
    ) -> ty.Tuple[TensorDictBase, TensorDictBase]:
        """Patched."""
        return step_and_maybe_reset(self, tensordict)

def step_and_maybe_reset(
    self: EnvBaseRL,
    tensordict: TensorDictBase,
) -> ty.Tuple[TensorDictBase, TensorDictBase]:
    """Runs a step in the environment and (partially) resets it if needed.

    Args:
        tensordict (TensorDictBase): an input data structure for the :meth:`~.step`
            method.

    This method allows to easily code non-stopping rollout functions.

    Examples:
        >>> from torchrl.envs import ParallelEnv, GymEnv
        >>> def rollout(env, n):
        ...     data_ = env.reset()
        ...     result = []
        ...     for i in range(n):
        ...         data, data_ = env.step_and_maybe_reset(data_)
        ...         result.append(data)
        ...     return torch.stack(result)
        >>> env = ParallelEnv(2, lambda: GymEnv("CartPole-v1"))
        >>> print(rollout(env, 2))
        TensorDict(
            fields={
                done: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False),
                next: TensorDict(
                    fields={
                        done: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False),
                        observation: Tensor(shape=torch.Size([2, 2, 4]), device=cpu, dtype=torch.float32, is_shared=False),
                        reward: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.float32, is_shared=False),
                        terminated: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False),
                        truncated: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False)},
                    batch_size=torch.Size([2, 2]),
                    device=cpu,
                    is_shared=False),
                observation: Tensor(shape=torch.Size([2, 2, 4]), device=cpu, dtype=torch.float32, is_shared=False),
                terminated: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False),
                truncated: Tensor(shape=torch.Size([2, 2, 1]), device=cpu, dtype=torch.bool, is_shared=False)},
            batch_size=torch.Size([2, 2]),
            device=cpu,
            is_shared=False)
    """
    logger.trace(f"step_and_maybe_reset...")
    tensordict = self.step(tensordict)
    # done and truncated are in done_keys
    # We read if any key is done.
    tensordict_ = step_mdp(
        tensordict,
        keep_other=True,
        exclude_action=False,
        exclude_reward=False,  # NOTE: Patched this
        reward_keys=self.reward_keys,
        action_keys=self.action_keys,
        done_keys=self.done_keys,
    )
    any_done = _terminated_or_truncated(
        tensordict_,
        full_done_spec=self.output_spec["full_done_spec"],
        key="_reset",
    )
    if any_done:
        tensordict_ = self.reset(tensordict_)
    # if isinstance(tensordict, (TensorDict, TensorDictBase)):
    #     assert "reward" in tensordict.keys(), f"{tensordict}"
    # if isinstance(tensordict_, (TensorDict, TensorDictBase)):
    #     assert "reward" in tensordict_.keys(), f"{tensordict_}"
    logger.trace(f"tensordict: {tensordict}")
    logger.trace(f"tensordict_: {tensordict_}")
    return tensordict, tensordict_

Basically I patch the step_and_maybe_reset methods with my own step_and_maybe_reset, which is an exact copy of torchrl's except the line exclude_reward=False.
In the original code, it is exclude_reward=True.
Now, am I really patching something and correcting it, or am I breaking something by doing this? How come the original source code on purpose excludes the reward, but then I hit a "reward" not in keys error?
To help reproduce, this is the test that fails for me:
import pytest
from loguru import logger
import typing as ty
import sys, os

from tensordict import TensorDict
from torchrl.envs import EnvCreator, check_env_specs, ParallelEnv
from torchrl.collectors import RandomPolicy, SyncDataCollector

from shark.env import ChessEnv # This is my libray, which passes the tests below

# THIS IS THE TEST THAT RAISES AN ERROR
@pytest.mark.parametrize("num_envs", [3])
def test_parallen_env(engine_executable: str, num_envs: int) -> None:
    """Test usage of `ChessEnv` in `ParallelEnv`."""
    make_env = EnvCreator(lambda: ChessEnv(engine_path=engine_executable))
    # Makes identical copies of the env, runs them on dedicated processes
    env = ParallelEnv(num_envs, make_env)
    check_env_specs(env)
    rollout = env.rollout(2)
    logger.info(f"Rollout: {rollout}")
    assert len(rollout) == num_envs
    # Test integration with collector
    policy = RandomPolicy(env.action_spec)
    collector = SyncDataCollector(
        env,
        policy,
        frames_per_batch=2,
        total_frames=10,
        device="cpu",
        reset_at_each_iter=True,
    )
    for i, td in enumerate(collector):
        assert isinstance(td, TensorDict)
        if i > 1:
            break
    logger.success(f"Passed for {env.__class__.__name__}")

# THIS TEST PASSES
def test_chess_env(engine_executable: str) -> None:
    """Test we can initialize the chess environment."""
    # Create env
    env = ChessEnv(
        engine_path=engine_executable,
        play_as="black",
        device="cpu",
    )
    # Reset
    state = env.reset()
    logger.info(f"Reset state: {state}")
    # Step
    action = env.sample()
    out = env.step(action)
    logger.info(f"Tensordict: {out}")
    # Rollout
    td = env.rollout(3)
    logger.info(f"Rollout: {td}")
    # Sanity check
    check_env_specs(env)
    # Sample
    env.sample(from_engine=False)
    env.sample(from_engine=True)

Regardless of whether I use ParallelEnv or not, I hit the error KeyError: 'key "reward" not found in TensorDict with keys [\'_reset\', \'action\', \'done\', \'observation\', \'terminated\']' when using my env with the SyncDataCollector.

IIUC you would like to get the reward at the root level, not only in the "next"?
The convention we adopted was to store the reward and done states in the "next" entry.
This is in accordance to the MDP definition from Sutton and Barto where an action at time t gives you an output (s, r, done) tuple that are grouped together:

Thanks for your reply.
I'm fine with keeping the reward in the "next" entry, but then I am wondering why I hit that error? Beware that the error is raised because the SyncDataCollector code looks for that missing "reward" key, not my code.

My guess is that it comes from the fact that you provide the reward during reset

I think that may be it! It's a pleasure to work with you. :)


perf: extract regex patterns

🔗 Linked issue

📚 Description

Extracting regex patterns across the codebase can contribute to overall performance by not requiring recompilation with each usage, given that they act as an object and would therefore hold a reference

I also followed up on it in the warning itself, but I believe the CodeQL is inaccurate. The pattern is the exact same, the only difference is the placement.
And given that marking this sends a notification, I decided not to mention you additionally.
I assume you would like ui-templates to be left unchanged? I didn't make any changes there yet because I remember from my other perf PR that you decided it is not a concern

import { run, bench } from 'mitata'

// Version 1: regex inline
function regexInline(version) {
  return version.replace(/-\d+\.[0-9a-f]+/, '')
}

// Version 2: regex outside
const SEMANTIC_VERSION_RE = /-\d+\.[0-9a-f]+/
function regexOutside(version) {
  return version.replace(SEMANTIC_VERSION_RE, '')
}

// Test
if (regexInline('1.0.0-1.0a') !== '1.0.0') throw new Error('1 failed')
if (regexOutside('1.0.0-1.0a') !== '1.0.0') throw new Error('2 failed')

bench('regex inline', () => {
  regexInline('1.0.0-1.0a')
})

bench('regex outside', () => {
  regexOutside('1.0.0-1.0a')
})

run()

Hey, Poooya!
In simple patterns and little iteration counts, I agree the difference may not be that noticeable. However, if it'll happen in a loop (like most of these patterns are) that's where the performance will really improve. It is also logical, because each interation will encounter the pattern again as opposed to a reference.
I will be able to provide a more thorough benchmark soon

@GalacticHypernova On the benchmark with 100,000 iterations, the performance improvement is 1.1-1.5ms. That seems negligible, and possibly within the margin of error of any test.
It's so tricky - with tiny improvements like this on the one hand there's no real downside, on the other I wouldn't be surprised if the engine could optimise this kind of thing itself - and will we rewrite things back in three months if/when it does? Also, are there any readability costs here?

@danielroe The current benchmark, as expected, indeed is somewhat of a small improvement, because the pattern is simple and there is overall little to no resource usage outside of the benchmark. In a real world application there will be many things happening at once, and it would be important to optimize everything as much as possible. In more complex patterns, in a functional application, this can easily reach over tens of milliseconds of performance difference for much less than 100,000 iterations.
As for the ability of the engine to optimize it on its own, this is quite unfortunately impossible, due to the implementation of regex. It's the same as to why duplicating functions is still creating 2 instances. Each time the JS compiler reaches a regex literal, it compiles it. Then, since it's an object, if it's placed into a variable, it saves the reference to the compiled regex. It would not make sense for the engine to internally reuse it, because that essentially nullifies the use of variables.
The readability here is not overly impacted, because the change is not that big, it's just pre-compiling the regex and then using the variable.

let's give it a go!

Thanks! I'm sure this will have a great impact


[NEW-UI] Backup options page

Questions
Answers

Description?
Please be specific when describing this change. What did you change? Why?

Type?
bug fix / improvement / new feature / refactor

BC breaks?
yes / no

Deprecations?
yes / no

Fixed ticket?
Fixes #{issue URL here}, Fixes #{another issue URL here}

Sponsor company
Your company or customer's name goes here (if applicable).

How to test?
Indicate how to verify that this change works as expected.

Excepting #976 to be merged before unblocking this one.


Blast 3D

I have a question: is this project going to be part of Blast 3D? Which 3D engine is used for Blast 3D itself? Because from the screenshot and the description of some commands I have read on blitzcoder and from the demo I tried, I have the impression that the engine used is OpenB3D. Is that correct?

Hey there @angros47. Happy New Year! 🎉
Yes, this is part of Blast3D and the 3D engine is OpenB3D.
Cheers.

Happy new year to you!
I am always glad to see someone using OpenB3D. Have you had to make any change to it, or are you using the version provided on github/sourceforge? I would be interested to hear any feedback, in case.

Awesome. It became an obvious choice since it has history and closest to Blitz3D.
I'm still using v1.25 and will update to the latest eventually.
Yes, I'd be happy to post some suggestions and feedback soon.


imposssible to formate blade code

Template
Please write your blade template that occurs issues.
Context (Environment)

write your environment

Description

Any other informations

Closed by #372


[ML][Pipelines]If else multiple blocks

Description
Support multiple blocks in if_else
E2E tests not enabled yet.
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Path to Windows Kits is incorrect

In the first paragraph of https://docs.microsoft.com/en-us/uwp/api/Windows.UI.Xaml.Controls.AppBar#control-style-and-template section, the path to the SDK is incorrectly suggesting it's the 64 bit programs folder.
"For design purposes, generic.xaml is available in the (Program Files)\Windows Kits\10\DesignTime\CommonConfiguration\Neutral\UAP\ <SDK version>\Generic folder from a Windows Software Development Kit (SDK) installation. "
It is actually in the 32 bit folder:
Program Files (x86)\Windows Kits
NOT
(Program Files)\Windows Kits\10\

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3e19cf53-1136-11fc-ad60-ac54746a5e16
Version Independent ID: eadefb56-bfcf-0027-bc8a-9966efda834f
Content: AppBar Class (Windows.UI.Xaml.Controls) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.appbar.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: jwmsft
Microsoft Alias: jimwalk

@LanceMcCarthy I created a PR to fix this, but in the future: you can too! Instead of reporting the bug just go ahead and click "Edit" on the doc page and submit the proposal yourself.

I usually see this button (have opened several such fixes in the past), but it wasn't visible here. Not sure why, do I need to be signed in with a specific account (i.e. my MVP account?).

OH derp. You mean Edit here on GitHub :)

The docs page has an "Edit" button, e.g. this page you reported has this near the top:

You're right that many pages don't have it and that's usually because the page isn't backed by GitHub content. You shouldn't need to be signed in to docs to see the button vs not.


Error: 15

Just added an account, tried to run Pyrelay and am getting this error. Doesn't look like the game has updated since the latest release, so I don't believe its packet related. Tested it on a different machine and still getting the same error. No clue what is going on.
Connecting to <IP_ADDRESS>
Error: 15
Client forcefully disconnected from <IP_ADDRESS>
Where do these error codes come from? is it a packet id?

https://github.com/Maxi35/pyrelay/blob/64837cb3c2f626bda71309403fba25e3c64835a7/Client/Client.py#L359-L370
Looks like you received a FailurePacket with and errorId of 15, which is that the server is full and you were placed in the queue.
From a quick search of this repo I don't think pyrelay handles server queuing, but what should happens is:

Receive FailurePacket (don't reconnect)
Wait for QueueMessagePacket
Wait x seconds and attempt to reconnect
Repeat

Can always count on Extacy to have the method ;)
Appreciate your help.


Table不支持拖拽怪改变列宽？

I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Support request => Please do not submit support request here

Current behavior

Expected behavior

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

ng-zorro-antd version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

not supported yet.


Fix Innerloop Solution

The image icon check in broke the inner loop solution because the shared helper MakeIconElementFrom now references ImageIcon which isn't always included in the inner loop solution. We can fix this using the preprocessor variable defined in the compile step of the ImageIcon shared project.

/azp run


Duplicate address issues

Live Better store front:
Able to simulate issues Duplicate address issue: --> Duplicate address gets created when i place an order with guest/loggedin user. if i fill same address in shipping and billing
Other issue found:

User gets logged in as undefined with guest user. Not Able to move to register screen from checkout page.
Duplicate addresses are getting created on betterstore for the guest user.
If I try to register with same guest user from register page then it pass alert as email already in user.

This is fixed:

User gets logged in as undefined with guest user.

This is fixed:

Duplicate addresses are getting created on betterstore for the user.

Address(es) entry with same shipping and billing address is showing single address record for the user.


Improve support for extending/overriding fields in Pydantic

Currently this code is broken:
class User(pydantic.BaseModel):
    age: int
    password: Optional[str]

@strawberry.experimental.pydantic.type(User, fields=["age", "password"])
class UserType:
    name: str = "Michael"

https://github.com/strawberry-graphql/strawberry/pull/826/files#diff-998cad7d53bd87d5dc5e6e4cf3c76e6757da8a4691b786cd76cfc8b3f63852daR175-R181

This is broken for me too

I think this was fixed in a recent version, this is working for me now.

@jerber thanks! will close this issue now :)


Load latest server log on console ws connection.

Could we add the ability to send the latest server log to the client on a web socket connection?
It should basically be this https://bitbucket.org/phybros/swiftapi/src/41595c76fc2f97d93da188c8dcafd404cd82d60b/src/main/java/org/phybros/minecraft/SwiftFilter.java#lines-48:61

So you're saying that you would open the connection and it would immediately send the last 1000 log lines through the newly opened connection @srmullaney?
Can also make the 1000 configurable in the config file (default to 1000).

@phybros Yes.
I love the idea to make it configurable.

Cool. Almost finished!

Cool! Will the lines sent have all the color codes etc?

Yes

Okay, cool.


Display Changes Requested

Request changes to display:

Delete extra (redundant) “R” and “T” in the upper left corner of the display that indicate receive and transmit.

Make the scan group indicators consistent.  Call them “I” and “II” on the screen in both the top left while scanning, to the right of the channel label, and in the manual.

“><” indicates the response timer is running for “dual rx respond”. But it erroneously also shows when in “main tx dual rx”.
Current implementation:
dual rx respond: shows "DWR" and then shows "><" for 4 seconds after a transmit or receive
main only: shows nothing
main tx dual rx: shows "DW" and then "><" for 4 seconds after a transmit or receive
crossband: shows "XB" all the time

I suggest deleting the “><” indicator and replace with two “>” indicators, one by each channel, just as done in the stock firmware. This indicates not only the respond 4 seconds is active, but also what channel will be responded to. Edit: it appears the ">" indicator exists and is also active in the display. So perhaps just delete the extra "><" indicator, and leave "DW" or "DWR" showing.

Why change. Isn't it QS-default ?

Why change. Isn't it QS-default ?

The AM bug is QS default. Change for the better. Removing this redundancy might even give us a few more bytes free code space.


IAsyncRequestHandler should be replaced by IRequestHandler

In 4th section under "Register the types used by MediatR" header, there is registered IAsyncRequestHandler interface in MediatR module. Since 4.0.1 MediatR have allowed to use only IRequestHandler.
https://github.com/jbogard/MediatR/releases
https://github.com/jbogard/MediatR/issues/224

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b3920c19-388e-0e84-960f-0bb0c8340fe0
Version Independent ID: c728d4ba-50e8-798d-72a0-c4a5b6f13ca2
Content: Implementing the microservice application layer using the Web API
Content Source: docs/architecture/microservices/microservice-ddd-cqrs-patterns/microservice-application-layer-implementation-web-api.md
Product: dotnet-architecture
Technology: microservices
GitHub Login: @nishanil
Microsoft Alias: nanil

+1, spent some time comparing if IAsyncRequestHandler is still valid with MediatR 4.0.1, thanks @pawel-zolty for highlighting this.


Refactor

Replaced deprecated typing with native typing
Applied De-Morgan identities to logical comparisons
Replaced native open() methods with pathlib context handlers
Simplified boolean logic

@nkato Fixed!


ipvlan slaves must inherit the mac-address from parent

Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

ping @nerdalert

@sanimej the reason I removed it is because kernel fails with operation not supported if we try and set a mac-address for a ipvlan sub-interface. Also in certain kernel versions, even if we set a mac-address, it seems to be ignored. I guess it is safer to not set it up even though the mac-address is empty in inspect.

@mavenugo Its ok to not set the mac. But we should try to show the mac in docker inspect by getting the auto generated mac. If the priority is to fix the basic issue now lets file an issue for the inspect issue and address it later.

make sense. btw... any changes as you suggested requires driver api changes on the libnetwork core side .. :(

any changes as you suggested requires driver api changes on the libnetwork core side
We can create the ipvlan slave interface in the CreateEndpoint and set the mac assigned by the kernel.

LGTM


Positioned inside Opacity gives unhelpful error message.

Failed assertion: line 26 pos 12: 'other.runtimeType == this.runtimeType' is not true.

We should add an assertion in ParentDataNode and its descendants that verifies that it was put under the right parent, and give that a much more useful error message.

Should have a better error message now.


Changes to GUI front Page

Change 3rd tab to "Setup and Test"
Make a large rectangle to the right of the taps rest of screen wide x ~down to a little above the plots. This will be for alarms. Move the Alarm Status indicator to this box. For red & green, flash the background. We will add more text.
Move the plot legends to the left side of each plot. Make background transparent so plot line overwrites.
Move plots to the right side and enclose in a black border.
Change the font everywhere to Oxygen (yes,  that is the name of the font).
Make the On/Off bigger.

Triple the height of the Alarm Box and increase font size.
Make the font bigger on control and status blocks. The entry box is wider than needed.
See 1, 5, and 6 above.


Adopt for new multi-threaded cursive capabilities

Note: this includes https://github.com/BonsaiDen/cursive_table_view/pull/40
In 1 cursive now requires Send + Sync, and the reason for this is:
The View now requires Send + Sync, to allow accessing or moving views between threads.
This prevents using Rc/RefCell, and may require using Arc/Mutex instead.
This should eventually open the way for more multi-threaded processing of the view tree.

This PR passes cargo build and cargo test with cursive-core = "0.3.5" and cursive = "0.20.0" and then I've tested grin (latest master) which passes too.
Would be nice to have it as version 0.15 maybe?
If so, it would allow grin to use an updated cursive and avoid this error:
   Compiling grin v5.3.0-alpha.1 (/home/user/1tmp/grin)
error[E0599]: the method `with_name` exists for struct `TableView<WorkerStats, StratumWorkerColumn>`, but its trait bounds were not satisfied
   --> /home/user/1tmp/grin/src/bin/tui/mining.rs:238:31
    |
238 |                 Dialog::around(table_view.with_name(TABLE_MINING_STATUS).min_size((50, 20)))
    |                                           ^^^^^^^^^ method cannot be called on `TableView<WorkerStats, StratumWorkerColumn>` due to unsatisfied trait bounds
    |
   ::: /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cursive_table_view-0.14.0/src/lib.rs:117:1
    |
117 | pub struct TableView<T, H> {
    | --------------------------
    | |
    | doesn't satisfy `_: Nameable`
    | doesn't satisfy `_: View`
    |
    = note: the following trait bounds were not satisfied:
            `TableView<WorkerStats, StratumWorkerColumn>: cursive::View`
            which is required by `TableView<WorkerStats, StratumWorkerColumn>: Nameable`
            `&TableView<WorkerStats, StratumWorkerColumn>: cursive::View`
            which is required by `&TableView<WorkerStats, StratumWorkerColumn>: Nameable`
            `&mut TableView<WorkerStats, StratumWorkerColumn>: cursive::View`
            which is required by `&mut TableView<WorkerStats, StratumWorkerColumn>: Nameable`
    = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope; perhaps add a `use` for it:
    |
17  + use cursive_core::view::nameable::Nameable;
    |

and an updated cursive-core to avoid this error:
   Compiling cursive_table_view v0.14.0
error[E0277]: `Rc<(dyn for<'a> Fn(&'a mut Cursive, H, std::cmp::Ordering) + 'static)>` cannot be sent between threads safely
   --> /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cursive_table_view-0.14.0/src/lib.rs:995:21
    |
995 | impl<T, H> View for TableView<T, H>
    |                     ^^^^^^^^^^^^^^^ `Rc<(dyn for<'a> Fn(&'a mut Cursive, H, std::cmp::Ordering) + 'static)>` cannot be sent between threads safely
    |
    = help: within `TableView<T, H>`, the trait `Send` is not implemented for `Rc<(dyn for<'a> Fn(&'a mut Cursive, H, std::cmp::Ordering) + 'static)>`
note: required because it appears within the type `Option<Rc<(dyn for<'a> Fn(&'a mut Cursive, H, std::cmp::Ordering) + 'static)>>`
   --> /usr/lib/rust/1.76.0/lib/rustlib/src/rust/library/core/src/option.rs:570:10
    |
570 | pub enum Option<T> {
    |          ^^^^^^
note: required because it appears within the type `TableView<T, H>`
   --> /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cursive_table_view-0.14.0/src/lib.rs:117:12
    |
117 | pub struct TableView<T, H> {
    |            ^^^^^^^^^
note: required by a bound in `cursive::View`
   --> /home/user/1tmp/cursive/cursive-core/src/view/view_trait.rs:34:33
    |
34  | pub trait View: Any + AnyView + Send {
    |                                 ^^^^ required by this bound in `View`

error[E0277]: `Rc<(dyn for<'a> Fn(&'a mut Cursive, usize, usize) + 'static)>` cannot be sent between threads safely
   --> /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cursive_table_view-0.14.0/src/lib.rs:995:21
    |
995 | impl<T, H> View for TableView<T, H>
    |                     ^^^^^^^^^^^^^^^ `Rc<(dyn for<'a> Fn(&'a mut Cursive, usize, usize) + 'static)>` cannot be sent between threads safely
    |
    = help: within `TableView<T, H>`, the trait `Send` is not implemented for `Rc<(dyn for<'a> Fn(&'a mut Cursive, usize, usize) + 'static)>`
note: required because it appears within the type `Option<Rc<(dyn for<'a> Fn(&'a mut Cursive, usize, usize) + 'static)>>`
   --> /usr/lib/rust/1.76.0/lib/rustlib/src/rust/library/core/src/option.rs:570:10
    |
570 | pub enum Option<T> {
    |          ^^^^^^
note: required because it appears within the type `TableView<T, H>`
   --> /home/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cursive_table_view-0.14.0/src/lib.rs:117:12
    |
117 | pub struct TableView<T, H> {
    |            ^^^^^^^^^
note: required by a bound in `cursive::View`
   --> /home/user/1tmp/cursive/cursive-core/src/view/view_trait.rs:34:33
    |
34  | pub trait View: Any + AnyView + Send {
    |                                 ^^^^ required by this bound in `View`


#574 Issue: [GSSOC'23] Fixing the Signup page.

Fixes Issue

I made the SignUp page Responsive.

Changes proposed

The issue was that the Sign-Up page was not Responsive.

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

hii..could you sync your branch once..it's giving error to be deployed...there may be some syntax issues or something like that..


[User-Sign-Up] user can register for a particular role

A user should be able to register as a teacher or a trainer.
A teacher is one who will be involved in directly impacting members of the community he/she is (will be) assigned to.
The job of the teacher is to inculcate knowledge, and educate members of his/her assigned community.
Request data

first name: must be between 2 and 30 characters long
last name: must be between 2 and 30 characters long
email
phone number
password: must include at least a number; must be between 6 and 30 characters long; must include at least a special character in the range of full stop (.), asterisk (*), ampersand (&), caret (^), exclamation (!), dash(-), and underscore(_)
country of residence
state/province residence
city/town of residence
fluent languages
role: must be either teacher or trainer

Implementations

Make use of the express-validator middleware
Encrypt user password with bcrypt
Send an activation message and link to the user email
The activation link must include certain crypted info about the user

Response data

Send user an informative success message to activate his/her account in the provided email

PS

the encrypted data in the link being sent to the user should be a JWT with a 20 mins lifetime
the JWT should be attached to the link as a URL parameter called activationToken


[http-client-python] bump pyright version to latest

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


[Feature] Option to remove/hide/disable "Run | Debug" in editor

This is pretty straightforward, I'm a shortcut user when it comes to running my tests, and having these links appear above every test/block just adds visual clutter. It would nice if I had the ability to hide them. :-)

there is already a option in the vscode settings to disable codeLens
jestrunner.disableCodeLens

@firsttris yea, that doesn't seem to really be working.
I have these settings:
"jestrunner.codeLensSelector": "**/*.{bueller}.{bueller}",
"jestrunner.disableCodeLens": true,

But debug is still showing up.

@metal-gabe , just figured out you have to reload the window / restart vscode after setting disableCodeLens to true for this to work


bug: Highlights not working when opening a file

Did you check the docs and existing issues?

[X] I have read the docs
[X] I have searched the existing issues

Neovim version (nvim -v)
0.9.4
Operating system/version
MacOS Sonoma 14.0
Describe the bug
Hello, first of all, Thank you so much @stevearc, this file explorer was the plugin that I really needed to manage all of my files and directories in a project.
I'm just experiencing a strange bug. Sometimes, when I open a file, the highlights are just broke.

I can't really get a pattern out of here, it's seems to be random on the files that that happens.
I also found out that, sometimes, when I open the file with some other plugin, or even manually with :e for example and the highlights work well, if I open oil and I hit <C-c> to close oil, it breaks the highlights again.
I use treesitter to manage the highlights.
Using neotree or telescope it works and always worked correctly but somehow, oil is not on this aspect and it's maybe the last piece of the cake to use this plugin on 90% of the times.
Gonna also be talking about this plugin on an upcoming YouTube video as well for all of the amazing work you did :D
Steps To Reproduce

Open a file with oil.nvim

Expected Behavior

This is how it shows the file whenever it's opened with either neotree or telescope.
Directory structure
No response
Repro
local lazypath = vim.fn.stdpath("data") .. "./.repro/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable", -- latest stable release
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)

local opts = {
	-- Use this colorscheme for the Lazy popup
	install = {
		colorscheme = { "nightfly" },
	},
	-- Automatic checks for plugin updates and notify
	checker = {
		enabled = true,
		notify = false,
	},
	-- detection of any file change on the modules module
	change_detection = {
		notify = false,
	},
}

require("lazy").setup({
	{ "bluz71/vim-nightfly-colors", name = "nightfly", lazy = true, priority = 1000 },
	{
		"nvim-neo-tree/neo-tree.nvim",
		-- version = "v3.x",
		dependencies = {
			"nvim-lua/plenary.nvim",
			"nvim-tree/nvim-web-devicons", -- not strictly required, but recommended
			"MunifTanjim/nui.nvim",
		},
		config = function()
			-- Unless you are still migrating, remove the deprecated commands from v1.x
			vim.cmd([[ let g:neo_tree_remove_legacy_commands = 1 ]])
			require("neo-tree").setup({
				close_if_last_window = true,
				-- filesystem = {
				-- 	follow_current_file = {
				-- 		enabled = true,
				-- 		leave_dirs_open = true,
				-- 	},
				-- },
				-- sources = {
				-- 	"document_symbols",
				-- },
			})
		end,
	},
	{
		"stevearc/oil.nvim",
		-- opts = {},
		-- Optional dependencies
		dependencies = { "nvim-tree/nvim-web-devicons" },
		config = function()
			local oil = require("oil")
			-- require("oil").setup()
			oil.setup()
			vim.keymap.set("n", "-", "<CMD>Oil<CR>", { desc = "Open parent directory" })
		end,
    },
		{
			-- {
			"nvim-treesitter/nvim-treesitter",
			event = { "BufReadPre", "BufNewFile" },
			dependencies = {
				"nvim-treesitter/nvim-tree-docs",
				"nvim-treesitter/nvim-treesitter-context",
				"windwp/nvim-ts-autotag",
			},
			build = ":TSUpdate",
			config = function()
				require("nvim-treesitter.configs").setup({
					-- A list of parser names, or "all"
					ensure_installed = { "lua", "vim", "sql" },
					-- Install parsers synchronously (only applied to `ensure_installed`)
					sync_install = false,
					-- Automatically install missing parsers when entering buffer
					-- Recommendation: set to false if you don't have `tree-sitter` CLI installed locally
					auto_install = true,
					-- List of parsers to ignore installing (for "all")
					-- ignore_install = { "javascript" },

					---- If you need to change the installation directory of the parsers (see -> Advanced Setup)
					-- parser_install_dir = "/some/path/to/store/parsers", -- Remember to run vim.opt.runtimepath:append("/some/path/to/store/parsers")!

					highlight = {
						-- `false` will disable the whole extension
						enable = true,
						-- NOTE: these are the names of the parsers and not the filetype. (for example if you want to
						-- disable highlighting for the `tex` filetype, you need to include `latex` in this list as this is
						-- the name of the parser)
						-- list of language that will be disabled
						--disable = { "c", "rust" },
						-- Or use a function for more flexibility, e.g. to disable slow treesitter highlight for large files
						--disable = function(lang, buf)
						--    local max_filesize = 100 * 1024 -- 100 KB
						--    local ok, stats = pcall(vim.loop.fs_stat, vim.api.nvim_buf_get_name(buf))
						--    if ok and stats and stats.size > max_filesize then
						--        return true
						--    end
						--end,

						-- Setting this to true will run `:h syntax` and tree-sitter at the same time.
						-- Set this to `true` if you depend on 'syntax' being enabled (like for indentation).
						-- Using this option may slow down your editor, and you may see some duplicate highlights.
						-- Instead of true it can also be a list of languages
						additional_vim_regex_highlighting = false,
					},
					autotag = {
						enable = true,
						enable_close_on_slash = false,
					},
					tree_docs = { enable = true, spec_config = { jsdoc = {} } },
					context_commentstring = {
						enable = true,
						enable_autocmd = false,
					},
					incremental_selection = {
						enable = true,
						keymaps = {
							init_selection = "gnn", -- set to `false` to disable one of the mappings
							node_incremental = "grn",
							scope_incremental = "grc",
							node_decremental = "grm",
						},
					},
					indent = {
						enable = true,
					},
				})

				require("treesitter-context").setup({
					enable = true, -- Enable this plugin (Can be enabled/disabled later via commands)
					max_lines = 0, -- How many lines the window should span. Values <= 0 mean no limit.
					min_window_height = 0, -- Minimum editor window height to enable context. Values <= 0 mean no limit.
					line_numbers = true,
					multiline_threshold = 20, -- Maximum number of lines to collapse for a single context line
					trim_scope = "outer", -- Which context lines to discard if `max_lines` is exceeded. Choices: 'inner', 'outer'
					mode = "cursor", -- Line used to calculate context. Choices: 'cursor', 'topline'
					-- Separator between context and content. Should be a single character string, like '-'.
					-- When separator is set, the context will only show up when there are at least 2 lines above cursorline.
					separator = nil,
					zindex = 20, -- The Z-index of the context window
					on_attach = nil, -- (fun(buf: integer): boolean) return false to disable attaching
				})
			end,
			-- },
		},
}, opts)
vim.g.nightflyVirtualTextColor = true
vim.cmd([[colorscheme nightfly]])

Did you check the bug with a clean config?

[X] I have confirmed that the bug reproduces with nvim -u repro.lua using the repro.lua file above.

Hmmm...I've seen this issue before, but it was fixed a while ago. The problem before was related to when the filetype got set; if it was set during another autocmd then the FileType autocmd wouldn't be processed the same and syntax highlighting wouldn't kick in.
The fact that it breaks sometimes when you <C-c> out of oil is extremely weird because at that point we're not even entering a new buffer, we're just switching back to a previously visible buffer.
I tried out the config you provided, but I haven't been able to get the issue to reproduce. Do you have any ideas what specific situations might cause it to happen?

Have been testing and debugging as well but it’s a bit strange. It seems to be random files but the files that previously failed the highlight have a big chance to not show the highlight anymore on following neovim sessions. It’s really strange

It looks like you are using treesitter for syntax highlighting, correct? I've tried looking around for other clues on what could cause the syntax highlighting to be cleared, and I found this thread which looks similar but seems like the issue may have been a timeout with the typescript regex syntax highlighting. Do you see anything in :messages when this happens?
I read up more on treesitter highlighting and I don't see any obvious way for this to happen. If you manage to get a more solid repro let me know, otherwise I don't think there's much I can do.

There's nothing on :messages unfortunately. Do you think that this typescript timeout can also be triggering on lua highlighting? Because, the image that I used on the example is lua.
It's so strange this because, I really love this plugin but it get's me really annoyed since I don't have highlighting in there :(

Update: Now, in another file that doesn't have the correct highlight, I have found this error which is from Comment.nvim but seems that is caused by oil:

Update: Looks like I may have found the reason. Inside of my plugin, at directory, after/plugin/functions.lua:

I have a mkview as you can see:

If I comment out this autocmd it works just fine.

Interesting. I still don't know how that could cause the highlighting issue, and I still wasn't able to repro the issue. Some things you could try:

Wrap the callback function in a vim.schedule
Only conditionally call mkview/loadview if vim.bo.buftype == ""
The <C-c> behavior for oil just calls oil.close(). The source for that is here
https://github.com/stevearc/oil.nvim/blob/4088efb8ff664b6f1624aab5dac6c3fe11d3962c/lua/oil/init.lua#L396-L402

You could try to get a minimal repro using those autocmds, winsaveview, winrestview, and buffer switching. If you can get that minimal repro, it's probably something we can file as an issue upstream

I've run into this issue specifically when working with norg files. A simple :e fixes the problem.
I was curious, so I commented out the ft = "norg" line in my neorg config (using Lazy as my package manager) and now the highlighting works as expected upon opening a file. I'm not smart enough to further explain this, but hopefully it provides some understanding to those who are.

For me, the reason was really the event to read and store the folds that I have (views) that was breaking it. I had problems in the past with this thing but in the last year, I haven't any problem. I love to always keep the foldings and cursor position but to use oil.nvim, I can't have this event so I just disabled.
I wanted to investigate further but, haven't had time to do it yet but most probably, if there's an error on an event, since some are silence, oil.nvim just breaks highlight for some reason, that's my opinion as far as I investigated it.

If anyone can get a repro that would be super helpful, but in the meantime you could also try out the change in #214. That PR is intending to fix behavior on Windows, but it also changes how oil opens files in general so there's a chance that it will have some effect on this bug.

@stevearc don't know if will be useful or not but, I was trying to find out why sometimes loadview breaks the highlight and I found this issue on telescope: https://github.com/nvim-telescope/telescope.nvim/issues/559 seems to be a bit similar with what happens with oil but they still have the issue open.
Most of the people just use :e as a quick workaround but seems to be a more complex issue.


⚠ Thumbnails API Endpoint has degraded performance

In ca6a402, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 4241 ms

Resolved: Thumbnails API Endpoint performance has improved in c278e3f after 21 hours, 24 minutes.


deeplink work success relese mode on IOS

Actually I didn't do anything,
I gave my account to relax mode with XCode and generated pod and deeplink worked.
				LD_RUNPATH_SEARCH_PATHS = (
					"$(inherited)",
					"@executable_path/Frameworks",
				);
				LD_RUNPATH_SEARCH_PATHS = "$(inherited) @executable_path/Frameworks";

I thought maybe because of these and I did PR
let's try to publish it or we'll see it again

I have set up appcenter to build on that branch. However, I am confident that this is not the fix, the issue is that you need to configure Xcode to generate the certificate for the production/release build, but currently you only do it for the development build.
Appcenter trashes failed builds for a while. So I need to wait again for it to build to see the error.

btw. why do we have development here and not release? https://github.com/encointer/encointer-wallet-flutter/blob/54c5ba18c6cf2d9327e0b1395e4649f09077e6c7/ios/Runner/Runner.entitlements#L6

This is the error we get:
/Users/runner/work/1/s/ios/Runner.xcodeproj: error: Provisioning profile "Encointer AppStore Deployment" doesn't support the Associated Domains capability. (in target 'Runner' from project 'Runner')
/Users/runner/work/1/s/ios/Runner.xcodeproj: error: Provisioning profile "Encointer AppStore Deployment" doesn't include the com.apple.developer.associated-domains entitlement. Profile qualification is using entitlement definitions that may be out of date. Connect to network to update. (in target 'Runner' from project 'Runner')

** ARCHIVE FAILED **

##[error]Error: /usr/bin/xcodebuild failed with return code: 65
##[section]Finishing: Xcode build (signed)
##[section]Starting: Xcode build (signed)


Release version v3.0.1-beta

This PR releases v3.0.1-beta with an updated Grafana dependency of 9.x.

Pulled in the test fix to #359.


Error when trying to upgrade to Capacitor 6 Beta(2).

Hi
I get this message when I try to upgrade my project to Capacitor 6 Beta(2).
Your version of capacitor-blob-writer is not compatible with @capacitor/core 6.0.0-beta.2.
Thanks
Roland

Fixed in v1.1.15, please test.


I always get an error on the frontend web!

I don't know what i should do, please help!

Hi,
The frontend cannot connect to the backend.
Can you screenshot the backend output ?

Yes, how should i install ffmpeg, because i thought i have it already installed.

I've changed the drive and now i get this error.

I've managed to fix it, I had "ffmpeg" AND "ffmpeg-python" installed, so i did this.
`pip uninstall ffmpeg
pip uninstall ffmpeg-python
pip install ffmpeg`


Getting a StickySidebar is not defined error

I used the example html, css and js code but I can't get the sticky sidebar to stick!
Have a look to https://audio-gear-dev.org/companys7/
Chrome states the errors:

Any idea what might be wrong?
Thanks
Robert

I had this issue because I was grabbing the .js file from src folder. That is the pre-compile source code. Grab the same file from dist folder and it should be the distribution file

I had this issue because I was grabbing the .js file from src folder. That is the pre-compile source code. Grab the same file from dist folder and it should be the distribution file

That work for me, thank you.


Question About Print Output

Hello, I'm wondering why there are single quotes used around the filename after 'Adding metadata' but double quotes after 'Adding thumbnail'.
Sorry to ask again, but I'm really interested.
[ffmpeg] Adding metadata to 'Leonard Nimoy reads Ray Bradbury's 'There Will Come Soft Rains' from 'The Martian Chronicles'-LzhlU8rXgHc.mp3'
[ffmpeg] Adding thumbnail to "Leonard Nimoy reads Ray Bradbury's 'There Will Come Soft Rains' from 'The Martian Chronicles'-LzhlU8rXgHc.mp3"

Thank you in advance.

Are you trolling or what?

No, I'm not -- really. :-/

I'm genuinely, 100% interested in the difference between the print output in these two lines. I'm sure there might be a completely logical reason for it, but I don't know what it is. Sorry.
[ffmpeg] Adding metadata to 'Leonard Nimoy reads Ray Bradbury's 'There Will Come Soft Rains' from 'The Martian Chronicles'-LzhlU8rXgHc.mp3'
[ffmpeg] Adding thumbnail to "Leonard Nimoy reads Ray Bradbury's 'There Will Come Soft Rains' from 'The Martian Chronicles'-LzhlU8rXgHc.mp3"

I'll repeat it 3rd time: there is no logical reason. First uses single quotes hardcoded in string literal, second uses double quotes.


bug: Tall(ish) custom labware throwing LabwareHeightError

#Overview
API doesn't tollerate tall custom labware, even those shorter than the 150 mm limit.
Current behavior
For custom labware, LabwareHeight > LABWARE_DEF_JSON["dimensions"]["zDimension"]
This then throws a LabwareHeightError error when running scripts e.g.

LabwareHeightError [line 65]: The Cobas V3 24 Well Plate 4300 µL on 3 has a total height of 291.0 mm, which is too tall for your current pipette configurations. The longest pipette on your robot can only be raised to 179.7 mm above the deck. This may be because the labware is incorrectly defined, incorrectly calibrated, or physically too tall. Please check your labware definitions and calibrations.

Steps to reproduce
app version 3.17.0
import json
from opentrons import protocol_api, types

CALIBRATION_CROSS_COORDS = {
    '1': {
        'x': 12.13,
        'y': 9.0,
        'z': 0.0
    },
    '3': {
        'x': 380.87,
        'y': 9.0,
        'z': 0.0
    },
    '7': {
        'x': 12.13,
        'y': 258.0,
        'z': 0.0
    }
}
CALIBRATION_CROSS_SLOTS = ['1', '3', '7']
TEST_LABWARE_SLOT = '3'

RATE = 0.25  # % of default speeds
SLOWER_RATE = 0.1

PIPETTE_MOUNT = 'left'
PIPETTE_NAME = 'p20_single_gen2'

TIPRACK_SLOT = '5'
TIPRACK_LOADNAME = 'opentrons_96_tiprack_20ul'

LABWARE_DEF_JSON = """{"ordering":[["A1","B1","C1","D1"],["A2","B2","C2","D2"],["A3","B3","C3","D3"],["A4","B4","C4","D4"],["A5","B5","C5","D5"],["A6","B6","C6","D6"]],"brand":{"brand":"Cobas v3","brandId":["LONDON BIOFOUNDRY"]},"metadata":{"displayName":"Cobas V3 24 Well Plate 4300 µL","displayCategory":"wellPlate","displayVolumeUnits":"µL","tags":[]},"dimensions":{"xDimension":127.76,"yDimension":85.48,"zDimension":101},"wells":{"A1":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":18.88,"y":69.74,"z":6},"B1":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":18.88,"y":51.74,"z":6},"C1":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":18.88,"y":33.74,"z":6},"D1":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":18.88,"y":15.74,"z":6},"A2":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":36.88,"y":69.74,"z":6},"B2":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":36.88,"y":51.74,"z":6},"C2":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":36.88,"y":33.74,"z":6},"D2":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":36.88,"y":15.74,"z":6},"A3":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":54.88,"y":69.74,"z":6},"B3":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":54.88,"y":51.74,"z":6},"C3":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":54.88,"y":33.74,"z":6},"D3":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":54.88,"y":15.74,"z":6},"A4":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":72.88,"y":69.74,"z":6},"B4":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":72.88,"y":51.74,"z":6},"C4":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":72.88,"y":33.74,"z":6},"D4":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":72.88,"y":15.74,"z":6},"A5":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":90.88,"y":69.74,"z":6},"B5":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":90.88,"y":51.74,"z":6},"C5":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":90.88,"y":33.74,"z":6},"D5":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":90.88,"y":15.74,"z":6},"A6":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":108.88,"y":69.74,"z":6},"B6":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":108.88,"y":51.74,"z":6},"C6":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":108.88,"y":33.74,"z":6},"D6":{"depth":95,"totalLiquidVolume":4300,"shape":"circular","diameter":15,"x":108.88,"y":15.74,"z":6}},"groups":[{"metadata":{"displayName":"Cobas V3 24 Well Plate 4300 µL","displayCategory":"wellPlate","wellBottomShape":"u"},"brand":{"brand":"Cobas v3","brandId":["LONDON BIOFOUNDRY"]},"wells":["A1","B1","C1","D1","A2","B2","C2","D2","A3","B3","C3","D3","A4","B4","C4","D4","A5","B5","C5","D5","A6","B6","C6","D6"]}],"parameters":{"format":"irregular","quirks":[],"isTiprack":false,"isMagneticModuleCompatible":false,"loadName":"cobasv3_24_wellplate_4300ul"},"namespace":"custom_beta","version":1,"schemaVersion":2,"cornerOffsetFromSlot":{"x":0,"y":0,"z":0}}"""
LABWARE_DEF = json.loads(LABWARE_DEF_JSON)
LABWARE_LABEL = LABWARE_DEF.get('metadata', {}).get(
    'displayName', 'test labware')

metadata = {'apiLevel': '2.0'}

def uniq(l):
    res = []
    for i in l:
        if i not in res:
            res.append(i)
    return res

def run(protocol: protocol_api.ProtocolContext):
    tiprack = protocol.load_labware(TIPRACK_LOADNAME, TIPRACK_SLOT)
    pipette = protocol.load_instrument(
        PIPETTE_NAME, PIPETTE_MOUNT, tip_racks=[tiprack])

    test_labware = protocol.load_labware_from_definition(
        LABWARE_DEF,
        TEST_LABWARE_SLOT,
        LABWARE_LABEL,
    )

    num_cols = len(LABWARE_DEF.get('ordering', [[]]))
    num_rows = len(LABWARE_DEF.get('ordering', [[]])[0])
    well_locs = uniq([
        'A1',
        '{}{}'.format(chr(ord('A') + num_rows - 1), str(num_cols))])

    pipette.pick_up_tip()

    def set_speeds(rate):
        protocol.max_speeds.update({
            'X': (600 * rate),
            'Y': (400 * rate),
            'Z': (125 * rate),
            'A': (125 * rate),
        })

        speed_max = max(protocol.max_speeds.values())

        for instr in protocol.loaded_instruments.values():
            instr.default_speed = speed_max

    set_speeds(RATE)

    for slot in CALIBRATION_CROSS_SLOTS:
        coordinate = CALIBRATION_CROSS_COORDS[slot]
        location = types.Location(point=types.Point(**coordinate),
                                  labware=None)
        pipette.move_to(location)
        protocol.pause(
            f"Confirm {PIPETTE_MOUNT} pipette is at slot {slot} calibration cross")

    pipette.home()
    protocol.pause(f"Place your labware in Slot {TEST_LABWARE_SLOT}")

    for well_loc in well_locs:
        well = test_labware.well(well_loc)
        all_4_edges = [
            [well._from_center_cartesian(x=-1, y=0, z=1), 'left'],
            [well._from_center_cartesian(x=1, y=0, z=1), 'right'],
            [well._from_center_cartesian(x=0, y=-1, z=1), 'front'],
            [well._from_center_cartesian(x=0, y=1, z=1), 'back']
        ]

        set_speeds(RATE)
        pipette.move_to(well.top())
        protocol.pause("Moved to the top of the well")

        for edge_pos, edge_name in all_4_edges:
            set_speeds(SLOWER_RATE)
            edge_location = types.Location(point=edge_pos, labware=None)
            pipette.move_to(edge_location)
            protocol.pause(f'Moved to {edge_name} edge')

        set_speeds(RATE)
        pipette.move_to(well.bottom())
        protocol.pause("Moved to the bottom of the well")

        pipette.blow_out(well)

    set_speeds(1.0)
    pipette.return_tip()

Expected behavior

LabwareHeight = LABWARE_DEF_JSON["dimensions"]["zDimension"]
The above script should run without a LabwareHeightError.

@hainesm6 this protocol is set to metadata = {'apiLevel': '2.0'} which is quite old. I don't necessarily think it'll solve the issue, but I would recommend checking this with {'apiLevel': '2.11'} just to be safe.
Also, as the error message says...

This may be because the labware is... incorrectly calibrated

...so the second thing I would try would be a factory reset of your robot's labware calibration data, since this calibration data which affects the total height used during motion planning

Thanks for the suggestions @mcous. As you say it might be because of the apiLevel being old but then I created this issue 18 months ago. Unfortunately I now don't have the ability to check if this solves the error. I've therefore closed the issue.


[✨]contributors

@all-contributors please add @IanDxSSXX  for infrastructure, tests and code

@all-contributors please add @orange04  for code and design


CXF-7989: org.apache.cxf.jaxrs.JAXRSInvoker should handle CompletionException

The Problem
In case when CXF is used along with CompletionStage / CompletableFuture, in most cases the error handling is propagated back through CompletionException instance (http://cs.oswego.edu/pipermail/concurrency-interest/2014-August/012911.html). It works but it forces users to introduce own implementations of ExceptionMappers and manually unwrap the CompletionException into something more meaningful.
Use cases
There are 3 leading use cases which cover most typical usage of the CompletionStage / CompletableFuture.

@Suspended AsyncResponse

public void get(@PathParam("isbn") String isbn, @Suspended final AsyncResponse resp) {
        client.target("http://localhost:19092/services/catalog/" + isbn)
            .request()
            .rx()
            .get(Book.class)
            .whenComplete((r, ex) -> {
                if (ex != null) {
                    resp.resume(ex);
                } else {
                    resp.resume(r);
                }
            });
    }

Return CompletableFuture<Book>

public CompletableFuture<Book> get(@PathParam("isbn") String isbn) {
        return client
            .target("http://localhost:19092/services/catalog/" + isbn)
            .request()
            .rx()
            .get(Book.class)
            .toCompletableFuture();
    }

Return constructed instance of the CompletableFuture<Book>

public CompletableFuture<Book> get(@PathParam("isbn") String isbn) {
    final CompletableFuture<Book> future = new CompletableFuture<Book>();
    future.completeExceptionally(...);
    return future;
}

Suggested Solution
Instrument JAXRSInvoker::handleAsyncResponse to distinguish CompletionException, unwrap the cause and use it to select the appropriate ExceptionMapper (if any).
@rmannibucau why JAXRSInvoker::handleAsyncResponse and not JAXRSInvoker::checkFutureResponse, the 1st use case which uses suspended async response does not flow through JAXRSInvoker::checkFutureResponse but all three use cases run through JAXRSInvoker::handleAsyncResponse
@coheigea @dkulp guys would really appreciate if you could take a quick look, small change but may introduce undesired side effects, thanks a lot.

@reta agree, your fix is better than my proposal which was ignoring suspended case. Thanks a lot to work on it.

Thanks for review, @coheigea , sorry I didn't get the whitespace changes thingy, could you please clarify a bit, thanks a lot!

@reta If I apply the patch via "git apply" I get a lot of whitespace warnings:
526.patch:37: trailing whitespace.
526.patch:39: trailing whitespace.
cause = throwable.getCause();
526.patch:41: trailing whitespace.
526.patch:77: trailing whitespace.
526.patch:82: trailing whitespace.
return CompletableFuture.supplyAsync(() -> {
warning: squelched 15 whitespace errors
warning: 20 lines add whitespace errors.

@coheigea done, the patch applies smoothly now, thanks!


fatal: 'origin' does not appear to be a git repository

My environment: Windows 7 enterprise, git 2.6.2, python 2.7.11.
When trying to initiate repository with command
repo init -u https://android.googlesource.com/platform/manifest
I got an error like:
fatal: 'origin' does not appear to be a git repository
fatal: Could not read from remote repository.
Is it possible to solve it?

Thank you for the feedback.
Could you provide the full trace? If nothing else than the two lines are shown, could you set the env variable "REPO_TRACE" to 1 before calling repo?
`export REPO_TRACE=1`

Beside this, be aware of the fact that you get errors on Windows with the Android source (most surely) since it is not case sensitive regarding files and dirs.

origin is your fork: your own repo on GitHub, clone of the original repo of GitHub (stackoverflow)
if you have already set up git username/email with git config, try specifying a branch
repo init -u https://android.googlesource.com/platform/manifest -b master
I've had it pull the android source properly but can't remember the exact details. It successfully repo sync too, in cmd/cli (not minGW)


Describe second qubit in 'prepare entanglement'

For some reason only the first qubit being set to the initial state is mentioned - to make things less confusing, it is worth adding that the second qubit is set to Zero.
This helps to understand the next sentences (why the second qubit will always be the same as what we measure for the first).

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/quickstart.md
86
63
3
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 9b4f48c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 390: [Error] Error happened in build.entrypoint.ps1. Please build your target branch first and then reopen this pull request to do another round of verification. If the issue still happens, open a ticket in https://SiteHelp and include the following error message for our team to troubleshoot: Cannot process argument transformation on parameter 'defaultBoolValue'. Cannot convert value "" to type "System.Boolean". Boolean parameters accept only Boolean values and numbers, such as $True, $False, 1 or 0.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Release Google.Cloud.Container.V1 version 2.7.0

Changes in this release:
New features

Multiple new messages and properties (commit fa099d3)

Created release for Google.Cloud.Container.V1-2.7.0


Is there a way, i could browse documents(pdf)

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
Version Number of Plugin:
Device Tested On:
Simulator Tested On:
Version of VS:
Version of Xamarin:
Versions of other things you are using:
Steps to reproduce the Behavior
Expected Behavior
Actual Behavior
Code snippet
Screenshots

It's not possible at the moment to pick documents.
You can try using File picker plugin
https://github.com/Studyxnet/FilePicker-Plugin-for-Xamarin-and-Windows


After editing, the cursor can be left on an invalid cell.

Make a new document, hit i to insert "hello" and hit <ESC>. The cursor will be filling the cell immediately after the "o," even though there is no character there. If the cursor is moved off this cell is cannot return.
This is wrong. Probably the right way to handle it is to allow the cursor to be "one past" the end of the document. This will also make an empty document somewhat more uniform.

The two commits above help a lot (and "fix" the original issue), but there's a knock-on. The fix above was to make the placement of the cursor as originally described legal (e.g., that one-beyond placement is now valid). Now, however, if you insert text from that position, when you're done you end up in the same relative position (one-beyond). But if you append text, you end up at the last character you appended.
The append behavior should match the insert behavior.


tf.function decorator converts ragged tensors to their dense equivalent

System information

Have I written custom code: No
OS Platform and Distribution: Windows 10 (will also test on Ubuntu)
TensorFlow installed from: binary
TensorFlow version: 2.0 preview
Python version: 3.6.6
CUDA/cuDNN version: 10
GPU model and memory: GTX 1070 8gb

Describe the current behavior
When using @tf.function a function with a ragged input it will return the dense equivalent of the ragged tensor.  When the decorator is omitted it returns a ragged tensor.
Describe the expected behavior
Return a ragged tensor regardless of the use of @tf.function
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
@tf.function
def do_nothing(x):
    return x

rt = tf.ragged.constant([[[1, 2, 3], [4]],
                         [[5], [], [6]],
                         [[7]],
                         [[8, 9], [10]]])

do_nothing(rt)

<tf.Tensor: id=51310, shape=(10,), dtype=int32, numpy=array([ 1,  2,  3,  4,  5,  6,  7,  8,  9, 10])>

# @tf.function
def do_nothing(x):
    return x

rt = tf.ragged.constant([[[1, 2, 3], [4]],
                         [[5], [], [6]],
                         [[7]],
                         [[8, 9], [10]]])

do_nothing(rt)

tf.RaggedTensor(values=<tf.RaggedTensor [[1, 2, 3], [4], [5], [], [6], [7], [8, 9], [10]]>, row_splits=tf.Tensor([0 2 5 6 8], shape=(5,), dtype=int64))

The ragged tensor documentation hints that this might be intended or necessary, is it?
https://www.tensorflow.org/guide/ragged_tensors#rank_and_ragged_rank

@edloper I thought we supported raggedtensors in tf.function due to the changes in nest. Can you take a look?

@alextp I want to work on this issue, can you please guide me on this?

@shashvatshahi1998 the right way to fix this is to instrument calls to the ragged tensor to tensor function and see when they fire in the tf.function codepath. Whatever's triggering that probably needs to call nest.flatten with expand_composites=True

Sorry, @alextp i am not getting what you are trying to say.

In third_party/tensorflow/python/eager/function.py, nest is used in several
places to unpack and repack collections of tensors.  Most likely, the
appropriate fix is to add "expand_composites=True" to some (or all?) of
these nest.xyz(...) calls.  This option causes composite tensors to be
expanded into their components.  E.g.:

x = {'a': t1, 'b': RaggedTensor(t2, t3), 'c': tf.SparseTensor(t4, t5,
t6)}
nest.flatten(x)
(t1, RaggedTensor(t2, t3), SparseTensor(t4, t5, t6)
nest.flatten(x, expand_composites=True)
(t1, t2, t3, t4, t5, t6)

Similarly, when expand_composites=True is used with nest.pack_sequence_as,
it will reconstruct the composite tensors from their components.
On Wed, Apr 3, 2019 at 3:02 PM Shashvat Chand Shahi <
<EMAIL_ADDRESS>wrote:

Sorry, @alextp https://github.com/alextp i am not getting what you are
trying to say.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/26505#issuecomment-479616856,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFhajDPzTbrtBjXkDa9tdNp8OTti1pYkks5vdPqugaJpZM4bmf0e
.

I took an initial stab at this, and got something basic working.  It ends up being a bit more involved than I thought at first (though the basic approach I suggested is the right one).  I'll work on it a bit more and get back to you with an update.

@edloper can I also contribute to it.

I added expand_composites to all nest.flatten() and nest.pack_sequence_as() what about other functions of nest in the file @edloper

Fixed by #40006c3.


raise wrong Exception: Invalid ResourceConsumption of Implementation

Some of Arxml do not have RESOURCE-CONSUMPTION in its SWC-IMPLEMENTATION in AUTOSAR version 4.0.3

Will you provide the ARXML for analysis?


Add optional classes param to ImageFolder and add docs.

Sorry for the time to response.
I think it might be better to follow an approach similar to the one presented in #145, so I'll close this PR for now.


Firebase UI Auth: What is the correct way to link provider Phone and Email/Password?

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?
Yes
Step 2: Describe your environment

Android device: Any
Android OS version: Any
Google Play Services version: 4.3.13
Firebase/Play Services SDK version: 21.0.7
FirebaseUI version: 8.0.1

Step 3: Describe the problem:
We are using email/password for authenticating user, we would like to use phone auth of Firebase UI to verify their mobile number that will be save to Firestore. We will not use phone for authenticating user. Our flow starts with authentication using email/password then launch another screen for phone auth, there we will the phone auth credential and link it like this.
val response: IdpResponse? = IdpResponse.fromResultIntent(result.data)

if (result.resultCode == Activity.RESULT_OK && response != null)
   Firebase.auth.currentUser!!.linkWithCredential(response.credentialForLinking!!)

This throws NPE
Failure delivering result ResultInfo{who=null, request=1236788821, result=-1, data=Intent { (has extras) }} to activity {com.sample.app/com.sample.apppresentation.AuthActivity}: java.lang.NullPointerException
Not sure if the response.credentialForLinking!! or the user is null but we just sign in with phone auth so it should not.
Not sure if this is the correct flow to link email/password and phone in Firebase since it looks like what happen here is we are overwriting the previous session of email/password with phone auth. What we think the correct way is to run phone authentication first then get its linking credential, use it after signing it with email/password and link them via linkWithCredential but does Phone Auth returns credentialForLinking?
Steps to reproduce:

Implement your own simple email/password sign in with Firebase Auth
Integrate Firebase UI and use only Phone Auth as provider
Create account using email/password provider and launch Firebase UI phone auth for OTP

Observed Results:
NPE
Expected Results:
Link phone auth provider to email/password provider.

Just check it again, the credentialForLinking is null. Why?

Update
What we tried so far is sign in with email/password (verified) then open Firebase Auth UI (Phone). Finish OTP then link the account using this approach.
val authCredential =<EMAIL_ADDRESS>"password")

viewModel.linkAccountCredential(user, authCredential)

This returns however an error:

Hi @ArcherEmiya05, one thing to note is that you cannot link existing accounts. Similarly, I believe credentialForLinking only applies if you're trying to link an account that already exists. Also, if you link a phone number your users will be able to use SMS sign in as well, but in your first post you say

We will not use phone for authenticating user

If your goal is to verify phone numbers but not use them for authentication, I think Firebase Auth is not the best solution.


Version tagging

Continuing the conversation from #119 :
I think it might be about time to start tagging versions. Then use that to add a deprecation warning like @jedahan said. This means we should start adding a CHANGELOG and keep a list of deprecation dates in there.
What do you think of that option?

Deprecation messages at least for now are not that important IMO.
I would start with a minimal CHANGELOG.md, and v0.1.0 then move from that. As of now I think we can go for it.

[ ] Create CHANGELOG.md with v0.1.0 entry, just "Initial release" would suffice
[ ] Update README.md with a release badge (see https://img.shields.io/github/release/qubyte/rubidium.svg)
[ ] Create release draft in https://github.com/frmendes/geometry/releases, thank our active contributors (castellanprime, ChauTNguyen) and link to https://github.com/frmendes/geometry/graphs/contributors

A future possibility is to have release schedules. But given our codebase and activity it seems unnecessary.
We may automate this but for now this is all we need to release v0.1.0.
Thoughts? @frmendes @jedahan

I'd like to take this on. Start with a minimal version and a CHANGELOG and build it up from there.

@frmendes Please do! I'm up for any question.

@jedahan @desyncr any features you would like to be built upon the implementation of version tagging? We already discussed deprecation messages.

Yeah I think the smallest subset of features for now, until we need them is good. We can probably put all message logic in the setup function on startup for people for now.

@frmendes Is there anything missing to implement versioning? Can I help with anything?

Well, time for me, tbh. Though, I promise I'll add it tonight 🤞

Fixed by #146


Digimon Scripts

This is what Ashley and I have done so far, we just set up the foundation tbh but its playable and works lol

I will update as we go

Another To-Do is to make Protect guaranteed for all Pokemon and make it 3 moves (and Protect + finisher)

I'm brainstorming that at the moment. I've never made two guaranteed moves & have it randomize an exceeding amount of moves lol

So far no conflict :3

C'mon Travis be nice

and he failed me

Tbh it isn't as bad of a travis error since I changed my coding style to tabs not spaces lol

wtf I got more errors when I solved the ones it gave me

mfw no more conflicts :D

Is that all?

@DeathlyPlays Good Job ! Impressed :P

I suggest we make a separate branch including the current code for the digimon project so its easier to commit to the project


Contribution and documentation changes

Moved development guide to CONTRIBUTING.md (GitHub can use this file)
Made some changes to CONTRIBUTING.md
Small README changes
Renamed MIT_LICENSE to LICENSE (more standard naming convention)

@kevgo @charlierudolph

This is 100% awesome. The documentation is now more professional, and a lot more mature and friendly to read. Thanks Allen!  :+1:
:rocket:

whoops wrong button, sorry. I got too excited :smile:

Great improvements to the documentation

BTW, I've noticed that the builds fail sometimes -- seems like a race condition introduced by parallel_tests?

Yeah noticed that too. I will open an issue


Circular solution failed in nodejs (NextJs - Api function area)

Describe the bug
Deal Tsyringe team,
First i want say thankyou for your masterpiece, tsyringe is wonderful and fast! Below is a issue i found when implement tsyringe to my project use NextJs as base:
in your document has section explains about circular dependencies when example:
@injectable()
export class Foo {
  constructor(@inject(delay(() => Bar)) public bar: Bar) {}
}

@injectable()
export class Bar {
  constructor(@inject(delay(() => Foo)) public foo: Foo) {}
}

// construction of foo is possible
const foo = container.resolve(Foo);

// property bar will hold a proxy that looks and acts as a real Bar instance.
foo.bar instanceof Bar; // true

This work well if put into test file and test use jest + ts-jest. but failed when add to a nextjs-api file. Look like circular not working correct in Nodejs ENV. Error shown:
ReferenceError: Cannot access 'Bar' before initialization.

Attention that Nextjs use typescript along with Babel. After read next example in document i see other case you show how to resolve circular issue with Interface, this make me idea maybe issue start from how babel reference mismatch class with interface, so i try to change constructor(@inject(delay(() => Bar)) public bar: Bar) {} to constructor(@inject(delay(() => Bar)) public bar: typeof Bar) {} and code working.
I'm has not enough experience to go deeper issue, but i guess issue because code after compile think ...: Bar as a class , not reference as interface, and at the time 2 classes circular one of them is not initialization yet.
Of couse use interface as middleware is a option, but it duplicate a lot of work in project if project service classes is huge. Maybe you guys can research to have better solution.
To Reproduce

create new project nextjs use yarn create next-app. Setup file and typescript follow nextjs instruction.
modify tsconfig (see below)
add file babelrc and add plugins (see below). This part in tsyring's document still lack info. If someone has time maybe can help clear exactly what babel plugins require for tsyringe in browser/node env.
create new file in pages/api folder named test.ts add content (see below).
run api , error will show.
change constructors of 2 classes : Foo/Bar to : typeof Foo/Bar and run api again.

tsconfig.json
{
  "compilerOptions": {
    "target": "ES6",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext"
    ],
    "allowJs": true,
    "skipLibCheck": true,
    "skipDefaultLibCheck": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve",
    "baseUrl": ".",
    "paths": {
      "@/*": [
        "./*"
      ]
    },
    "experimentalDecorators": true,
    "emitDecoratorMetadata": false,
    "strictPropertyInitialization": false,
  },
  "include": [
    "next-env.d.ts",
    "**/*.ts",
    "**/*.tsx"
  ],
  "exclude": [
    "node_modules"
  ]
}

.babelrc
{
  "presets": [
    [
      "next/babel",
      // {
      //   "class-properties": {
      //     "loose": true
      //   }
      // }
    ]
  ],
  "plugins": [
    "babel-plugin-parameter-decorator",
    "babel-plugin-transform-typescript-metadata",
    [
      "@babel/plugin-proposal-decorators",
      {
        "legacy": true
        // "decoratorsBeforeExport": true
      }
    ],
    [
      "@babel/plugin-proposal-class-properties",
      {
        "loose": true
      }
    ]
  ]
}

next js test api - pages/api/test.ts
import 'reflect-metadata';
import { NextApiHandler } from 'next';
import { container, delay, inject, injectable } from 'tsyringe';

@injectable()
export class Foo {
  constructor(@inject(delay(() => Bar)) public bar: Bar) {}
}

@injectable()
export class Bar {
  constructor(@inject(delay(() => Foo)) public foo: Foo) {}
}

const api: NextApiHandler = async (req, res) => {
  const test = container.resolve(Foo);
  console.log(test);
  return;
};
export default api;

Expected behavior
Find solution where we can reference Class as Interface.
Version:
"tsyringe": "^4.4.0",

@TrungRueta Did you try the example giving each class a different file ?

@TrungRueta Did you try the example giving each class a different file ?

@emilioastarita Yes, in large project i tried to write each class in separate file and import together. as i can guess now issue only comes from ...: <class ref as interface> syntax. if i replace with any everything work, but in IDE we will lost type hint.
For better workaround we used this:
type BarType = Bar;

@injectable()
export class Foo {
  constructor(@inject(delay(() => Bar)) public bar: BarType) {}
}

With extra line on top i can force typescript understand that BarType is 100% type/interface not class and it look like bypassed every compilers in next steps.

@emilioastarita Yes, in large project i tried to write each class in separate file and import together. as i can guess now issue only comes from ...: <class ref as interface> syntax. if i replace with any everything work, but in IDE we will lost type hint.
For better workaround we used this:
type BarType = Bar;

@injectable()
export class Foo {
  constructor(@inject(delay(() => Bar)) public bar: BarType) {}
}

With extra line on top i can force typescript understand that BarType is 100% type/interface not class and it look like bypassed every compilers in next steps.

I've also been struggling with this issue. Any help would be appreciated.

After a few hours of investigation, I think, the problem here is the isolatedModules: true.
This test will fail
https://github.com/microsoft/tsyringe/blob/master/src/__tests__/errors.test.ts#L55
if you add isolatedModules: true to jest.config.js: https://github.com/microsoft/tsyringe/blob/master/test/jest.config.js#L14
  globals: {
    "ts-jest": {
      isolatedModules: true,
      tsconfig: "src/__tests__/tsconfig.json"
    }
  },

With isolated modules, the Reflect provides Object instead of the other class name, so the detect circular dependency fails earlier.
    Expected pattern: /Cannot inject the dependency "b" at position #0 of "A01" constructor\. Reason:\s+Cannot inject the dependency "a" at position #0 of "B01" constructor\. Reason:\s+Attempted to construct an undefined constructor. Could mean a circular dependency problem. Try using `delay` function./
    Received message: "Cannot inject the dependency \"b\" at position #0 of \"A01\" constructor. Reason:
        Cannot inject the dependency \"a\" at position #0 of \"B01\" constructor. Reason:
            TypeInfo not known for \"Object\""

If you use babel as a transpiler, you have to use isolatedModules. Also, it is a good practice in Jest as well, because it heavily speeds up the test runner. (https://kulshekhar.github.io/ts-jest/docs/getting-started/options/isolatedModules/)


CLOUD-45463 instantiate default views

@martonsereg

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1904/


Full dark theme

I love pure dark theme & Goodtime has Dark theme by default :)
But this part of app is too bright👇🏿 & annoying when switching different apps...Especially night time kindle/eBook reading... (when we click on device recent app or enable freeform multiwindow)

Make it full dark, maybe like MaterialFileManager as shown in the screenshot.
Thankyou🙏🏿
(Idk what this part of the app is called. Recent app status_bar? Change title of this issue according to it)

What device and Android version are you using?

Model: Lenovo Tab4 8 plus (TB-8704X)
Android version: 8.1.0
Rooted: No


Add Content-MD5 header

This adds support for RFC1864: The Content-MD5 Header Field.

Added sunsided/hyperium-headers-content-md5 to publish it without upstream support. Will yank, should this be integrated in https://github.com/hyperium/headers/pull/142.


Flaky test: AsymmetricSyncTests ► insert[macos]

From: https://github.com/realm/realm-kotlin/runs/20418016911
Error:
  kotlin.AssertionError: Number of documents was: -321716944

It has also been happening on Jenkins: https://ci.realm.io/blue/organizations/jenkins/realm%2Frealm-kotlin/detail/PR-1581/27/tests/
It has only surfaced on macOS...not Android/JVM.

Seems to be caused by https://youtrack.jetbrains.com/issue/KT-64139/Native-Bug-with-while-loop-coroutine-which-is-started-and-stopped-on-the-same-thread
Upgrading to 1.9.22 makes the problem go away. Since we cannot upgrade the entire project to this version we probably just need to re-write the code to work around it.

Also failed here: https://github.com/realm/realm-kotlin/runs/20528124152

Looks like this might be the root cause: https://github.com/realm/realm-core/issues/7279
It was apparently a bug introduced in Core 13.24.1

Fixed by https://github.com/realm/realm-kotlin/pull/1645


Add validation pass for arithmetic operations

The pass checks if arithmetic operations (such as OpFMul) receive
correct operands.

I've also checked this against the Vulkan CTS.  Fortunately, the Vulkan CTS shaders continue to pass validation. :-)

Rebased and pushed onto master as 82df4bbd688a


[GHSA-jjjh-jjxp-wpff] Uncontrolled Resource Consumption in Jackson-databind

Updates

Affected products

Comments
it seems like CVE-2022-42003 was fixed in jackson-databind with version <IP_ADDRESS> (12-Oct-2022) according to the release notes: https://github.com/FasterXML/jackson-databind/blob/04a6719805a26b4e5699285b7110ec1bfd655365/release-notes/VERSION-2.x#L561

The reason for my request is to eliminate the difference between databases.
Having different severities in different databases is common, but having different information about which versions are vulnerable is inconvenient when using different scanning tools.
Snyk says <IP_ADDRESS> is not vulnerable but Grype says it is vulnerable.
In our case, our partner using Snyk was confident that his application didn't contain vulnerable 3rd party dependencies and delivered a release but we got different results using Grype.
In the opposite case, I would ask Snyk to revise the versions but in this particular case, it seems technically correct to make changes here.
I hope consistency is a good reason to make the change.

Definitely a fair reason to raise the concern, but we're going to keep <IP_ADDRESS> as the first fixed version, so that we do not impose an unnecessarily bad experience on gradle users. We've done similar things in the past as well
ex. https://github.com/github/advisory-database/pull/4714
Forcing bad user experience in exchange for a minor technical correction is not a good trade off. That said, I will add some language to the advisory to make this more clear.


Second code snippet missing semicolon

Second code snippet missing semicolon on the GetDeferral(); line.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1e9daac3-b416-9a90-39ff-55ada1d9d461
Version Independent ID: fecefe30-8a56-5da4-1123-7e822d96ae8a
Content: Create and register an out-of-process background task - Windows UWP applications
Content Source: windows-apps-src/launch-resume/create-and-register-a-background-task.md
Product: uwp
GitHub Login: @eliotcowley
Microsoft Alias: elcowle

Thanks for catching this, @tjmullen2! The fix should be reflected in the live docs soon.


Improved language for the compose parameter tool...

... the workflow parameters tutorial.
The word repeat doesn't appear so I guess we should avoid it. I was thinking about including this screenshot also but maybe that is overkill / would go stale too quickly?

thanks @jmchilton!


Implement SQL Agent API - Backend

Description
Tasks

[ ] Create a simple form in NextJS for input
[ ] Build a microservice Fast API
[ ] API accepts natural language input
[ ] Returns SQL Query with LangChain and OpenAI
[ ] Add Logging
[ ] Unit testing for SQL generation

Notes
Do not focus on frontend. Backend is important here.


Please specify "old browsers"

This is a (multiple allowed):

[ ] bug
[ ] enhancement
[ ] feature-discussion (RFC)
[x] Question

Readme.md

[...] or you need old browsers support [...]

Which browsers are considered "old" or "modern".
Please add a list of supported browsers (browser-matrix)

In my experience, most things work on IE10+ including Edge
The swiper works a charm on Chrome, Safari and Mozilla Firefox

Moving away from github. Feel free to reopen if interested.


Cannot disable manifest mode

vcpkg crashes on install if a parent directory happens to have a file or directory named vcpkg.json. Here is the setup:
/cluster/home/vcpkg.json  # some file over which I have no control and no permission and want to ignore
/cluster/home/username  # my home directory
/cluster/home/username/vcpkg  # my vcpkg install

Result:
$ cd /cluster/home/username/vcpkg
$ ./vcpkg install --triplet x64-linux-release any-port-name
Failed to load manifest from directory /cluster/home
try_read_contents("/cluster/home/vcpkg.json"): Permission deniednote: updating vcpkg by rerunning bootstrap-vcpkg may resolve this failure.

I need to disable vcpkg's manifest mode and tell it to ignore this file. It seems that there is no way to do so.

This was fixed in https://github.com/microsoft/vcpkg-tool/pull/1535 / https://github.com/microsoft/vcpkg/pull/42115


error generating xkmfile?

I have been very happily using crouton with my trusty Ubuntu chroot for a while.
Suddenly after the latest chrome-os update,  when I try to start my window manager within the chroot (via "startx"), the screen blanks and then returns back to the chrome-os, reporting the following error in the terminal (copied from /tmp/Xorg.crouton.log.1):
(I should note that I did do a "crouton -u" after the chrome-os update.)
It seems to be a permissions problem, but I cannot figure it out. Any help would be appreciated.
[  2147.840] (II) XKB: generating xkmfile /var/lib/xkb/server-B20D7FC79C7F597315E3E501AEF10E0D866E8E92.xkm
[  2147.873] [xkb] Can't rename /tmp/fileJJodNq to /var/lib/xkb/server-B20D7FC79C7F597315E3E501AEF10E0D866E8E92.xkm, error: Invalid cross-device link
[  2147.873] (EE) XKB: Couldn't compile keymap
[  2147.873] (EE) XKB: Failed to load keymap. Loading default keymap instead.
[  2147.877] (II) XKB: generating xkmfile /var/lib/xkb/server-B20D7FC79C7F597315E3E501AEF10E0D866E8E92.xkm
[  2147.897] [xkb] Can't rename /tmp/fileNF1px9 to /var/lib/xkb/server-B20D7FC79C7F597315E3E501AEF10E0D866E8E92.xkm, error: Invalid cross-device link
[  2147.897] (EE) XKB: Couldn't compile keymap
[  2147.897] XKB: Failed to compile keymap
[  2147.897] Keyboard initialization failed. This could be a missing or incorrect setup of xkeyboard-config.
[  2147.897] (EE)
Fatal server error:
[  2147.897] (EE) Failed to activate core devices.(EE)
[  2147.897] (EE)

Update: seems related to this .
If I set TMPDIR to $HOME/tmp and then startx, the screen goes blank and stays that way (rather than returning to chrome-os). The only way to return to chrome-os now is to reboot.

Try updating your chroot, and then try using xinit, because startx has never worked reliably in crouton (due to various coexistence hacks).


chore: version bump

Adress vulnerabilities

https://github.com/coveo/push-api-client.java/security/dependabot/1
https://github.com/coveo/push-api-client.java/security/dependabot/2

Version bumps already performed by Renovate

#57
#58


drivers: clock-control: st: add MCO support for H7 family

Create clock_stm32_ll_mco.h file to bring stm32_clock_control_mco_init, mco1_prescaler, mco2_prescaler, MCO1_SOURCE and MCO2_SOURCE definitions which were previously in clock_stm32_ll_common.{c,h}. This is done so that stm32_clock_control_mco_init can be called from clock_stm32_ll_h7.c.
Also update Kconfig.stm32 and add new MCO sources to allow H7 support.

Thank you all !