Add another Rust remote client that directly utilizes the flyteidl-rust Python bindings

Tracking issue
As a result of the initiative, issues, discussions and serial trials and errors in flyrs branch  1 2 this PR aims to create another FlyteRemote inremote/remote_rs.py which  wrapping flyteidl-rust[^1] as its core. Python package,  flyteidl-rust, was built directly from Rust. With the helps of works in flyte/flyteidl/lib.rs, this PR eliminates the protobuf objects serialization overhead[^2] while calling gRPC services. Unlike the original approach in flyrs, this method optimizes performance and reduces the maintenance cost.

Use case ( can work ):from flytekit.remote.remote_rs import FlyteRemote

def test_fetch_execute_sync_string_task(register):
    remote = FlyteRemote(Config.auto(config_file=CONFIG), PROJECT, DOMAIN)
    flyte_task = remote.fetch_task(name="basics.echo_string_task.test_echo_string_task", version=VERSION)
    # `wait` to sync execution
    execution = remote.execute(flyte_task, inputs={"s": "hey"}, wait=True)
    assert execution.is_done
    assert execution.outputs["o0"] == "hey"

[^1]: Temporarily registered in PyPI test index as test package that will not affect the real PyPI registry in production.
[^2]: In previous PRs, it serialize the python types into bytes string and pass into PyO3-exposed Rust functions.
Why are the changes needed?
With flyteidl-rust Python bindings, Flytekit can get rid of not only grpc but also protobuf Python dependencies, ultimately removing the model layer between protobuf and remote entities. This approach leverages  Rust structures directly.
For Rust side, developers still can wrap flyte/flytidl as their remote gRPC clients or use its API directly.
For Python, developers can build another remote client by wrapping flyteidl-rust as its core.
What changes were proposed in this pull request?
To this end, we have to replace all types import from model files to use flyteidl-rust package instead. Including,

Modify remote entities to inherit from flyteidl-rust classes  instead of model files.

Refactor type transoformers in  core/type_engine.py and get_serializable() in translator.py to adapt with flyteidl-rust classes.

get_serializable() in tools/translator.py

[ ] ReferenceEntity
[X] PythonTask
[ ] WorkflowBase
[X] Node
[ ] LaunchPlan
[ ] BranchNode
[ ] FlyteTask

SimpleTransformer in core/type_engine.py

[X] int
[X] float
[X] bool
[X] str
[ ] datetime
[ ] timedelta
[ ] date
[ ] none

Other types' Transformer in core/type_engine.py

How was this patch tested?

[!WARNING]
For now, we havn't migrate the test cases to use flyteidl-rust and other type transformation still works in progress, so the CI must fail, don't run.
FlyteRemote only works in Task register / fetch / execute strictly with primitive python types, Workflow and LaunchPlan still works in progress.

Setup process

Install flyteidl-rust
pip install -i https://test.pypi.org/simple/ flyteidl-rust

Run the tasks which only using string, integer, float as input or output. For example,

pyflyte register ./echo_string_task.py and run on web console, or
pyflyte run --remote ./echo_string_task.py test_echo_string_task --s "hey"

Screenshots

pyflyte register echo_int and echo_float task.

re-run echo_string task from console.

pyflyte run --remote  from flytekit import task

  @task()
  def test_square_task(n: float) -> float:
      return n**2

Check all the applicable boxes

[ ] I updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] All commits are signed-off.

Related PRs
Docs link
Cross References

https://github.com/pydantic/pydantic-core
https://github.com/pola-rs/pyo3-polars/tree/main/pyo3-polars

Thanks, Ketan, for reminding me not to compromise on the earlier serialization approach, and thanks to Kevin and Troy for providing helpful references as they are developing the rust-fs project. I also appreciate others’ insightful discussions.

There are several implementation details need to discuss (in regards to the developer experience):

Enum comparison as integer or as string or as enum itself via different magic function
Since we've add __repr__ magic function for rust enum type to show as int string, we can use int(str(literal_type.type)) to compare literal types. Or we can just leverag Rust Prost's as_str_name() function by exposing it to python binding through PyO3. Or we can add compare operator mentioned in https://pyo3.rs/v0.22.0/class/protocols .

The positional arguments of tuple variants in complex enum

Some Rust code generated enum variants comes in tuple form, like
pub struct Literal {
      pub value: ::core::option::Option<literal::Value>,
}

/// Nested message and enum types in `Literal`.
pub mod literal {
    #[::pyo3_macro::with_pyclass]
    #[derive(::pyo3_macro::WithNew)]
    #[allow(clippy::derive_partial_eq_without_eq)]
    #[derive(Clone, PartialEq, ::prost::Oneof)]
    pub enum Value {
        /// A simple value.
        #[prost(message, tag = "1")]
        Scalar(::prost::alloc::boxed::Box<super::Scalar>),
        /// A collection of literals to allow nesting.
        #[prost(message, tag = "2")]
        Collection(super::LiteralCollection),
        /// A map of strings to literals.
        #[prost(message, tag = "3")]
        Map(super::LiteralMap),
    }
}

So the usage in Python will be like, lit = flyteidl.core.Literal(value=flyteidl.literal.Value.Scalar(flyteidl.core.Scalar(flyteidl.scalar.Value.Primitive(flyteidl.core.Primitive(flyteidl.primitive.Value.StringValue("x"))))), hash="", metadata={}), which has no named arguments. It's not that trivia to use: lit.value[0].value[0].value[0]


chore: deprecation notice

Codecov Report

Merging #297 (733b50a) into master (2752492) will increase coverage by 2.33%.
The diff coverage is 70.13%.

@@            Coverage Diff             @@
##           master     #297      +/-   ##
==========================================
+ Coverage   74.10%   76.43%   +2.33%
==========================================
  Files          25       27       +2
  Lines        1255     1515     +260
  Branches      136      182      +46
==========================================
+ Hits          930     1158     +228
  Misses        297      297
- Partials       28       60      +32

Impacted Files
Coverage Δ

metadata_service/api/popular_tables.py
100.00% <ø> (ø)

metadata_service/api/user.py
100.00% <ø> (ø)

metadata_service/entity/dashboard_query.py
100.00% <ø> (ø)

metadata_service/entity/dashboard_summary.py
100.00% <ø> (ø)

metadata_service/entity/description.py
100.00% <ø> (ø)

metadata_service/entity/tag_detail.py
100.00% <ø> (ø)

metadata_service/proxy/statsd_utilities.py
81.25% <ø> (ø)

metadata_service/util.py
100.00% <ø> (ø)

metadata_service/proxy/shared.py
28.57% <28.57%> (ø)

metadata_service/api/badge.py
61.29% <61.29%> (ø)

... and 21 more

Continue to review full report at Codecov.

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


ci: set action to use node lts

Description:
Sets the workflow files to use node lts version in e2e
Checklist:

[ ] My PR follows the contribution guidelines of this project
[ ] My PR is free of third-party dependencies that don't comply with the Allowlist
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked the PR title to follow the conventional commit specification

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 72.134%

Files with Coverage Reduction
New Missed Lines
%

src/domain/utils/DER.ts
3
35.14%

Totals

Change from base Build<PHONE_NUMBER>7:
-0.07%

Covered Lines:
2774

Relevant Lines:
3655

💛  - Coveralls


joblist/preview/addjob styled

Description
Please include a summary of the change and a link to which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Job Post working with Modal after with fresh token

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Nice work.  styling is coming together.  merging request.


3/17/2020 AM Publish

Docs Build status updates of commit 9f1007d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureadps-2.0-preview/AzureAD/Get-AzureADMSGroup.md
:white_check_mark:Succeeded

azureadps-2.0-preview/AzureAD/New-AzureADMSGroup.md
:white_check_mark:Succeeded

azureadps-2.0-preview/AzureAD/Set-AzureADMSGroup.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


load example error

Problem reported by @nicksavers
$ docker run --rm -v "$HOME/bigchaindb_docker:/data" --link bigchaindb bigchaindb/bigchaindb load
INFO:bigchaindb.config_utils:Configuration loaded from `/data/.bigchaindb`
INFO:bigchaindb.commands.bigchain:Starting 1 processes
Process Process-1:
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/rethinkdb/net.py", line 271, in __init__
    self._socket = socket.create_connection((self.host, self.port), timeout)
  File "/usr/lib/python3.4/socket.py", line 509, in create_connection
    raise err
  File "/usr/lib/python3.4/socket.py", line 500, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.4/multiprocessing/process.py", line 254, in _bootstrap
    self.run()
  File "/usr/lib/python3.4/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/src/app/bigchaindb/commands/bigchain.py", line 201, in _run_load
    b.write_transaction(tx)
  File "/usr/src/app/bigchaindb/core.py", line 113, in write_transaction
    .insert(signed_transaction, durability=durability))
  File "/usr/src/app/bigchaindb/db/utils.py", line 47, in run
    self._connect()
  File "/usr/src/app/bigchaindb/db/utils.py", line 62, in _connect
    db=self.db)
  File "/usr/local/lib/python3.4/dist-packages/rethinkdb/net.py", line 661, in connect
    return conn.reconnect(timeout=timeout)
  File "/usr/local/lib/python3.4/dist-packages/rethinkdb/net.py", line 572, in reconnect
    return self._instance.connect(timeout)
  File "/usr/local/lib/python3.4/dist-packages/rethinkdb/net.py", line 430, in connect
    self._socket = SocketWrapper(self, timeout)
  File "/usr/local/lib/python3.4/dist-packages/rethinkdb/net.py", line 337, in __init__
    (self.host, self.port, str(ex)))
rethinkdb.errors.ReqlDriverError: Could not connect to localhost:28015. Error: [Errno 111] Connection refused

File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 384, in _make_request
six.raise_from(e, None)
File "", line 2, in raise_from
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 380, in _make_request
httplib_response = conn.getresponse()
File "/usr/local/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/local/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/local/lib/python3.6/http/client.py", line 258, in _read_status
line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
File "/usr/local/lib/python3.6/socket.py", line 586, in readinto
return self._sock.recv_into(b)
socket.timeout: timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 445, in send
timeout=timeout
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 638, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/local/lib/python3.6/site-packages/urllib3/util/retry.py", line 367, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/usr/local/lib/python3.6/site-packages/urllib3/packages/six.py", line 686, in reraise
raise value
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 600, in urlopen
chunked=chunked)
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 386, in _make_request
self._raise_timeout(err=e, url=url, timeout_value=read_timeout)
File "/usr/local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 306, in _raise_timeout
raise ReadTimeoutError(self, url, "Read timed out. (read timeout=%s)" % timeout_value)
urllib3.exceptions.ReadTimeoutError: HTTPConnectionPool(host='<IP_ADDRESS>', port=9984): Read timed out. (read timeout=20)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "sample1.py", line 40, in
sent_creation_tx = bdb.transactions.send_commit(fulfilled_creation_tx)
File "/usr/local/lib/python3.6/site-packages/bigchaindb_driver/driver.py", line 411, in send_commit
headers=headers)
File "/usr/local/lib/python3.6/site-packages/bigchaindb_driver/transport.py", line 82, in forward_request
backoff_cap=backoff_cap,
File "/usr/local/lib/python3.6/site-packages/bigchaindb_driver/connection.py", line 91, in request
**kwargs,
File "/usr/local/lib/python3.6/site-packages/bigchaindb_driver/connection.py", line 120, in _request
response = self.session.request(**kwargs)
File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 512, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.6/site-packages/requests/sessions.py", line 622, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.6/site-packages/requests/adapters.py", line 526, in send
raise ReadTimeout(e, request=request)
requests.exceptions.ReadTimeout: HTTPConnectionPool(host='<IP_ADDRESS>', port=9984): Read timed out. (read timeout=20)
i'm getting error like this...any suggestions please


Dependencies seem outdated

Hello, I'm trying to install the repo and noticed some dependency conflicts.
Namely, environment.yaml specifies:
  - python=3.7
  - pytorch=1.8.0

However, the install instructions for the package pytorch3d require the following:

Python 3.8, 3.9 or 3.10
PyTorch 1.12.0, 1.12.1, 1.13.0, 2.0.0, 2.0.1, 2.1.0, 2.1.1, 2.1.2 or 2.2.0.

Attempting to install with a later python version (I tried 3.8 and 3.9) seems to create other install issues (pip attempts to install pickle5 which should not be needed in python > 3.7).
Any advice for how to resolve this?

Hi~
How did you resolve the dependency problem?


Fixed go install usage

Hi, the current instruction is failed like:
$ go install -u github.com/DarthSim/overmind/v2
flag provided but not defined: -u
usage: go install [build flags] [packages]
Run 'go help install' for details.

I fixed it to correct usage of go install.

Hi @hsbt!
Thank's for your contribution!
I changed github.com/DarthSim/overmind@v2 back to github.com/DarthSim/overmind/v2 as github.com/DarthSim/overmind@v2 installs an old version

Thanks. But the current instruction is not working.
$ go install github.com/DarthSim/overmind/v2
go: 'go install' requires a version when current directory is not in a module

It should be:
go install github.com/DarthSim/overmind/v2@latest


Looking for microphone audio format

When using amoled.startmicrophone and grabbing audio from it, what format, samplerate and codec is the audio using?
I am trying to take audio from the t-glass and send it over to a speech to text server and then send back the transcription to display but I am unable to ascertain the formatting to properly reformat to the models desired setup.

PDM format SPM1423HM4H-B microphone, default left alignment

That did the trick, thank you very much for the information. I was able to get speech to text working and displaying on the t-glass thanks to this info.


Different transfer_functions and matter_power_spectrum

When I generate a cosmology object I have the option to specify a transfer_function and a matter_power_spectrum. I could not find an explanation of the difference between these two things in either the README, Doxygen docs or the pdf note that can be generated (000-ccl_note.pdf).
When specifying a cosmological model I can understand that one needs to specify a means of generating the linear power spectrum, and possibly a non-linear power spectrum, but I'm not sure how the transfer_function or matter_power_spectrum attributes of the cosmology structure map onto this.
It was also not clear to me what to make of the list of transfer_function keys, which includes 'emulator' and 'fitting_function'... ?
Apologies if this question is moronic or if I've missed something critical from a section of the documentation.

Thanks Elisa, this is super useful! When you say matter_power_spectrum=any in the last two items, there are situations where this would fail I thought? e.g. transfer_function=ccl_eisenstein_hu and matter_power_spectrum=ccl_emu wouldn't work?
I think having the flexibility you mentioned is worthwhile, but I feel that it's too confusing to have separate lists for the transfer_function and matter_power_spectrum settings when they are often (but not always) tightly coupled to one another. As an alternative, could we replace these two settings with a single power_spectrum setting (the name matter_power_spectrum is too verbose!), and just add each possible power spectrum + transfer function pair as a separate option in the list?
For example, the combinations you listed above would turn into:
ccl_class_linear, ccl_class_halofit
ccl_emulator
ccl_bbks_linear, ccl_bbks_halofit, ...
ccl_eh_linear, ccl_eh_halofit...

I got rid of some duplicate definitions, like fitting_function, to keep the list short. (We could potentially even get rid of the linear settings, as I'm not sure what the use-case for those is given that we have separate linear and nonlin matter power spectrum functions.)
That's only a handful of combinations, and there's no risk of choosing an incompatible transfer/matter power spectrum pair, as only allowed combinations are in the list. It's also easy to add new methods, e.g. ccl_emulator_ostrich or whatever. Would this alternative work for you? @elisachisari @alexander-mead ?

Okay, I think I understand this a bit better now. I think my suggestion would be the following:
Could it be set such that when you initialise a cosmology object you set a method for creating the linear power spectrum (e.g., CLASS, Eisenstein & Hu, BBKS, CAMB, ...) and then, optionally, also a method for creating the non-linear power (e.g., PD96, halofit, CosmicEmu, halo_model, HMcode, FAST-PT, perturbation theory, ...)?
The linear power is almost always useful to have and most of the non-linear methods take this as an input and then apply a 'correction' to the linear power to get some non-linear power. The only exception to this rule from the list above would be the CosmicEmu, where some linear power spectrum has been assumed in the running of the simulations that were used to generate the emulator. In this case, I would suggest that the user be forced to get the power spectrum from a Boltzmann code when using the emulator, so that, for example, 'BBKS + emulator' was not allowed.
Let me know if I am missing something crucial here. Or if anyone has a better suggestion.

@alexander-mead

Could it be set such that when you initialise a cosmology object you set a method for creating the linear power spectrum (e.g., CLASS, Eisenstein & Hu, BBKS, CAMB, ...) and then, optionally, also a method for creating the non-linear power (e.g., PD96, halofit, CosmicEmu, halo_model, HMcode, FAST-PT, perturbation theory, ...)?

We could do this, and I can see some advantages to it, but my feeling is that many of these would be "incompatible". I think we need to figure out what combinations make sense for the user if we phrase it this way, compared against our current implementation. Worth running a poll on Slack?

The only exception to this rule from the list above would be the CosmicEmu, where some linear power spectrum has been assumed in the running of the simulations that were used to generate the emulator. In this case, I would suggest that the user be forced to get the power spectrum from a Boltzmann code when using the emulator, so that, for example, 'BBKS + emulator' was not allowed.

Currently, if you call the emulator, you get the linear power spectrum from CLASS. 'BBKS+emulator' is effectively not allowed and if you try to do that, CCL should quit.

I think the new docs in v2 make this more clear. I am closing for now. Please feel free to reopen!


PDF: Cover page document identifier is likely always 2 separate lines

Document:

https://github.com/metanorma/mn-samples-bsi/blob/main/sources/bs-iso-44001/iso-44001.adoc

Left generated, right original:

Done:


mqtt5-source: Issue with wildcard topic. Character encoding issue? The same topic works with the paho-mqtt5 component

The mqtt5-source seems to have issues if the wildcard is in the middle of the topic. For example this doesn't work:
- route:
    id: route-05dc
    nodePrefixId: route-4ca
    from:
      id: from-c730
      uri: kamelet:mqtt5-source
      parameters:
        topic: test/+/incoming
        brokerUrl: tcp://localhost:1883
      steps:
        - log:
            id: log-ff85
            message: ${body}

The route just doesn't receive any messages:

If the kamelet is replaced with paho-mqtt5, things work:
- route:
    id: route-05dc
    nodePrefixId: route-4ca
    from:
      id: from-5a40
      uri: paho-mqtt5
      parameters:
        topic: test/+/incoming
      steps:
        - log:
            id: log-ff85
            message: ${body}

try with RAW(xxxx) for that value in the kamelet


Unable to parse unsupported language

Not sure what I've done wrong here but the app is unable to parse when attempting to error check. It says unsupported language.

Just read about the parser being limited to javascript only.

It only works for javascript

Just released v2023.1.0 where I completely removed error checking. It'll be rewritten from scratch in the future due to large amount of problems


changeTheme does not work

If i set Text("this is a message", style: Get.textTheme.headline6),
When I click to switch theme, the text will change state, but when I set it to Text("this is a message", style: Theme.of(context).textTheme.headline6),
When, clicking to switch themes does not work.
Flutter Version:
1.17.5
Get Version:
3.3.0
Describe on which device you found the bug:
iPhone 11 Pro Max Simulator
Minimal reproduce code
import 'package:flutter/material.dart';
import 'package:get/get.dart';

void main() {
  runApp(
    GetMaterialApp(
      debugShowCheckedModeBanner: false,
      theme: AppTheme.normalTheme,
      home: TestView(),
    ),
  );
}

class TestView extends StatelessWidget {
  const TestView({Key key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
             //does not work
            // Text("this is a message", style: Get.textTheme.headline6),
            //work
            Text("this is a message",
                style: Theme.of(context).textTheme.headline6),
            RaisedButton(
              onPressed: () => Get.changeTheme(AppTheme.otherTheme),
              child: Text("Change Theme"),
            )
          ],
        ),
      ),
    );
  }
}

class AppTheme {
  static ThemeData normalTheme = ThemeData(
      textTheme: TextTheme(
          headline6: TextStyle(
              fontSize: 30, fontWeight: FontWeight.bold, color: Colors.pink)));
  static ThemeData otherTheme = ThemeData(
      buttonTheme: ButtonThemeData(buttonColor: Colors.green),
      canvasColor: Colors.amber,
      textTheme: TextTheme(
          headline6: TextStyle(
              fontSize: 15, fontWeight: FontWeight.bold, color: Colors.green)));
}

Your code:

From what I could see, and even testing with your code, everything works fine.
If you think there is a bug, please provide a reproduction case, or mention what the problem is in this video.
I recommend using flutter clean before building a project, sometimes caching old versions can cause strange behavior.

 //does not work
            // Text("this is a message", style: Get.textTheme.headline6),
            //work
            Text("this is a message",
                style: Theme.of(context).textTheme.headline6),

if i use Text("this is a message", style: Get.textTheme.headline6) , it does not work. I don't want to use context,😁

Thanks for the explanation, now I understand.
Get.context is immutable, so everyone else that inherits from it is also, like MediaQuery, Theme, etc. are also.
There are context extensions for some of these functions, I don't remember if you have textTheme, but I'll add that in the next update if I don't.

context.theme added on 3.4.1

@jonataslaw  I use context.theme to change theme but it rebuild widget many times. If I use Get.textTheme.headline6 and define style it does not work.


can't install example

go install github.com/reeflective/console/example
github.com/reeflective/console
/home/tester/go/pkg/mod/github.com/reeflective/console@v0.0.0-20230120113638-77a4695d6f8a/tab-completer.go:34:27: undefined: carapace.Complete

Hello,
My install instructions are wrong: I forgot that the classic go install command does not work when there are replace directives in the go.mod. These directives should go away in the near future.
In the meantime, this will work:
git clone https://github.com/reeflective/console
cd console/example
go install

Then
example

will drop you the console app.

For anyone reading this, please know that a simple go install github.com/reeflective/console/example is now working for installing the example application.
All previous go replace directives that were forbidding this have been removed.


docs: eip1559 conflict

fix https://linear.app/matterlabs/issue/DOC-388/eip-1599-conflict

I did (probably) a better addition in the other PR. I'm gonna close this now.


FR - Display Message in Skillet Load with no variable

Skillets that have no variables do not show any runtime information in the configure/submit window. Can we have a way to show messages and amplifying information to users during that stage for Skillets that have no user input variables?
Thanks!,
~Richard

this was done in dev some time ago


Question: can we extract KOCACHE instead of saving it to disk to be able use it between stateless builds?

AFAIK, KOCACHE only accepts directories as a value. But we couldn't use this cache between stateless builds. For example, each workflow run executes in a fresh VM in GitHub Actions.
It'd be nice if we could use OCI registries to save the cache for ko builds.

cc @jonjohnsonjr

kindly ping @jonjohnsonjr

I think we can store this information in the annotations of the image manifest (which might be verbose) or labels of the image config, as BuildKit did, and read that information over here instead of the disk.
WDYT?

Hey sorry, I was on leave for a while 😅
Yes, this is definitely something we could do! I jotted down some notes a while back, let me just dump them here:
KOCACHE_REPO=gcr.io/jonjohnson-test/kocache

Tag = BuildId

Repo acts as map[BuildId]CacheManifest

CacheManifest {
  InlinedConfig
  Layer[0] = OriginalDescriptorBlob
  Layer[1..n] = ForeignDesc # Optional...
  Annotations
}

InlinedConfig gives access to diffids.

Layer[0] preserves the descriptor of the layer we built so it keeps the media type, annotations, etc. (Need some way to invalidate that if preserveMediaType is flipped? Annotation that is a checksum?) Inlined for speed via data field.

Layer[1..n] track where we've pushed this layer before, maybe we keep it in the kocacherepo as well. Annotations for staleness? Use foreign layer to reference other repos/registries in urls.

Annotations contain ko and go version metadata for determining staleness.

Cache repo can be used to store all built images as well so we can easily mount into target registry if possible but just copy if not.

New `ko cache` command for managing this repo (GC, sync, etc) and the dir. Directory is local so much faster to try first and we can cache through to it if both are set. May need a way to shard cacherepo for large number of tags?

I think what you're describing is very similar to what I had in mind here?


css hex colors turned into tags

Quick summary
Related ticket: 8924019-zd-a8c
Any css hex color codes in the post_content get treated as potential hashtags, and become tags on the post.
e.g.: if your post body includes this
text

Then the resultant post will get a tag ffffff which cannot be removed, as it is reapplied every time the post is edited.
Steps to reproduce

Create a new Post, use the code editor.
Enter an HTML tag with a css hex color code in its style. e.g.:
Save the post
Observe that the post now has a tag ffffff.
Try removing the tag - either in the "Edit Post" screen or via "Quick Edit"
The tag is re-added immediately upon removal.

What you expected to happen
Text that begins with a # inside an HTML attribute should be ignored when generating hashtags for a post.
What actually happened
CSS color codes from the post content are turned into tags.
Impact
One
Available workarounds?
Yes, easy to implement
Logs or notes
No response

Haven't we fixed that @akirk ?

Aaaahhh, I see! It will not transform it in a clickable link, but adds it as a tag!?!


Passing props to the rendered Component in

Hi, I am facing a problem with
How do I pass props to the rendered component?
I have this piece of code
class Deals extends Component {
  componentWillMount () {
    const { fetchDeals, navigation } = this.props
    fetchDeals()
  }
  render () {
    const { isFetchingDeals, dealsList, navigation } = this.props
    if(isFetchingDeals) return <ActivityIndicator/>
    else {
      return(
        <Container>
          <View>
            <DeckSwiper
              looping={false}
              dataSource={dealsList}
              renderItem={(item) => <DealCard item = {item} navigation = { navigation }/>}
            />
          </View>
        </Container>
      )
    }
  }
}

I want to pass navigation prop to the child element but it throws an error. any Ideas? Thanks in advance

@ed-zm renderItem={(item, navigation) => <DealCard item={item} navigation={navigation} />

Closing this due to inactivity


feat: bump version to 2.0.0

BREAKING CHANGE

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Loading spinner page on pharos.org doesn't load

Expected behavior
The page should load as the rest of the component pages do with documentation.
Actual behavior
The page is blank and appears to be broken
Steps to reproduce the issue

Go to https://pharos.jstor.org/components/loading-spinner
Observe a blank page.

Additional information
It is possible that this component does not have written documentation and that that the page on pharos.org needs to be designed.

There does exist a mild amount of existing guidance, so although we can probably stand to enhance it, doesn't seem to be a direct cause of the issue. Going a step further in the debugging, I see this in the browser console on that page:
react-dom.production.min.js:188 TypeError: d.trim is not a function
    at jo (CodeBlock.tsx:53:27)
    at go (react-dom.production.min.js:160:137)
    at xi (react-dom.production.min.js:289:337)
    at bs (react-dom.production.min.js:279:389)
    at vs (react-dom.production.min.js:279:320)
    at gs (react-dom.production.min.js:279:180)
    at ls (react-dom.production.min.js:270:88)
    at as (react-dom.production.min.js:267:429)
    at S (scheduler.production.min.js:13:203)
    at MessagePort.R (scheduler.production.min.js:14:128)
fu @ react-dom.production.min.js:188

react-dom.production.min.js:127 Uncaught TypeError: d.trim is not a function
    at jo (CodeBlock.tsx:53:27)
    at go (react-dom.production.min.js:160:137)
    at xi (react-dom.production.min.js:289:337)
    at bs (react-dom.production.min.js:279:389)
    at vs (react-dom.production.min.js:279:320)
    at gs (react-dom.production.min.js:279:180)
    at ls (react-dom.production.min.js:270:88)
    at as (react-dom.production.min.js:267:429)
    at S (scheduler.production.min.js:13:203)
    at MessagePort.R (scheduler.production.min.js:14:128)

standalone.mjs:23 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'languages')
    at standalone.mjs:23:694
    at Array.flatMap (<anonymous>)
    at Ct (standalone.mjs:23:681)
    at Sn (standalone.mjs:26:12000)
    at Kn (standalone.mjs:32:273)
    at standalone.mjs:34:800
    at Object.xo (standalone.mjs:34:869)
    at jo (CodeBlock.tsx:21:5)
    at go (react-dom.production.min.js:160:137)
    at xi (react-dom.production.min.js:289:337)

I haven't yet made heads nor tails of that, but I suspect something in the chaing of Gatsby + React + react-live or something of the sort.

Unable to load content on the page for paginations: https://pharos.jstor.org/components/pagination


Name space not being recongnzied

Issue Description
Namespace not being recongized sometimes.
I have to resetart the ide several times to make it work.
Once it is working fine, it will be fine but I have to redo this every time I restart my comptuer
Steps to Reproduce
Expected Behavior
Actual Behavior
Logs

C# log
Post the output from Output-->C# here
C# LSP Trace Logs
Post the output from Output-->C# LSP Trace Logs here.  Requires `dotnet.server.trace` to be set to `Trace`
Environment information
VSCode version: 1.89.1
C# Extension: 2.28.11
Using OmniSharp: false
Dotnet Information
.NET SDK:
 Version:           8.0.203
 Commit:            5e1ceea679
 Workload version:  8.0.200-manifests.8cf8de6d
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.203\
.NET workloads installed:
There are no installed workloads to display.
Host:
Version:      8.0.3
Architecture: x64
Commit:       9f4b1f5d66
.NET SDKs installed:
7.0.202 [C:\Program Files\dotnet\sdk]
8.0.202 [C:\Program Files\dotnet\sdk]
8.0.203 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.28 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Visual Studio Code Extensions

Extension
Author
Version
Folder Name

copilot
GitHub
1.194.0
github.copilot-1.194.0

copilot-chat
GitHub
0.15.2
github.copilot-chat-0.15.2

cpptools
ms-vscode
1.20.5
ms-vscode.cpptools-1.20.5-win32-x64

csdevkit
ms-dotnettools
1.5.20
ms-dotnettools.csdevkit-1.5.20-win32-x64

csharp
ms-dotnettools
2.28.11
ms-dotnettools.csharp-2.28.11-win32-x64

gitlens
eamodio
15.0.3
eamodio.gitlens-15.0.3

intellicode-api-usage-examples
VisualStudioExptTeam
0.2.8
visualstudioexptteam.intellicode-api-usage-examples-0.2.8

markdown-preview-enhanced
shd101wyy
0.8.13
shd101wyy.markdown-preview-enhanced-0.8.13

vscode-dotnet-runtime
ms-dotnettools
2.0.5
ms-dotnettools.vscode-dotnet-runtime-2.0.5

vscodeintellicode
VisualStudioExptTeam
1.3.1
visualstudioexptteam.vscodeintellicode-1.3.1

vscodeintellicode-csharp
ms-dotnettools
2.1.11
ms-dotnettools.vscodeintellicode-csharp-2.1.11-win32-x64

vstuc
visualstudiotoolsforunity
1.0.1
visualstudiotoolsforunity.vstuc-1.0.1

@Heedoplays can you provide more details on what you mean by 'Namespace not being recongized sometimes.'?  Is it missing from completion?  Is go to definition not working, something else?  A gif of the behavior your seeing is generally useful.
Also, if you wouldn't mind, set dotnet.server.trace to Trace, reload the window, reproduce the issue and then attach the contents of the 'C#' output window here.

Closing - not actionable without the additional requested information. If you're able to provide it, please re-open with the requested details.


fix: filter unique scanned handlers

🔗 Linked issue
Resolves #806

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Hi! I think this PR unintentionally caused that important issue: https://github.com/nuxt/nuxt/discussions/18515. I believe the filtering function should only be applied to api routes and not middlewares


[usecase] Update the tests description

Fail BugID: XWALK-4476

Impacted TCs num(approved): New 0, Update 2, Delete 0
Unit test Platform: iOS
iOS test result summary: Pass 0, Fail 2, Block 0

LGTM


Cannot connect to Metamask through integrating web3modal with chrome extension

Describe the bug
Trying to integrate web3model with chrome extension wallet, and I have Metamask in my browser but at the same time the web3modal is not able to read it and it outputs a page for installing MM.
SDK Version
      "@web3modal/ethereum": "^2.0.0-alpha.4",
       "@web3modal/react": "^2.0.0-alpha.4",
        "react": "^16.13.1",

To Reproduce
Steps to reproduce the behavior:

click on connect wallet button
choose MM button
then it will appear the page of installing MM

Expected behavior
To open MM extension in the same browser
Screenshots

Desktop (please complete the following information):

OS: [e.g. Arch Linux]
Browser [e.g. chrome, brave]

Smartphone (please complete the following information):
Additional context

Interesting, it looks like fonts are broken as well for some reason. Any chance you are running in some sort of restrictive mode that prevents certain things from loading?
I'm unable to replicate this in brave / chrome / firefox on macOS, opens metamask as expected.
As a side note, we also depend on minimum version of react v17.x in @web3modal/react

Interesting, it looks like fonts are broken as well for some reason. Any chance you are running in some sort of restrictive mode that prevents certain things from loading?

Could you please illustrate more on what do you mean with restrictive mode for chrome extension?

As a side note, we also depend on minimum version of react v17.x in @web3modal/react
I will update react and test again.

Could you please illustrate more on what do you mean with restrictive mode for chrome extension?

Just any non-standard browser settings you might've set up for say privacy reasons etc.
We also have a team member who uses Arch Linux and they were able to go through this flow without issues.

Also, could you please outline what you are working on i.e. is this an issue with some custom extension or with official metamask one?

We tried to replicate this, even on same os and version of linux, but to no success. Metamask behaves as expected. Could you give alpha.5 a go and see if this issue is still heppening there?

@iskdrews we are now on alpha.8 and multiple enhancements were made. Since we can't replicate the issue on same os I'll be closing this for time being. Feel free to ping me if you are still seeing this on newer versions and can provide those further details from above comments.

Hi @IljaDaderko,
Thanks for helping on resolving that, and sorry for pausing on this issue for a while.
Packages
I finished updating all the packages including react to 18.2.0:
        "react": "^18.2.0",
        "react-dom": "^18.2.0",
        "react-jazzicon": "^1.0.4",
        "react-redux": "^8.0.5",
        "react-router": "^6.4.3",
        "react-router-dom": "^6.4.3",

And installed the latest versions for web3modal:
        "@web3modal/ethereum": "^2.0.0-beta.7",
        "@web3modal/react": "^2.0.0-beta.7",
        "wagmi": "^0.8.8",

Issue
But unfortunately still getting the same issue where Metamask button in Web3Modal in my extension, can't reach out to my Metamask extension installed in the same browser, as you can see from the screenshot below:

Re-produce the issue
I tried also to make it easier for you to reproduce this issue. I forked this react chrome extension repo (https://github.com/iskdrews/chrome-extension-web3modal-boilerplate-react) which has react v17 and installed the latest web3modal, and got the same issue after building it.

Here are the steps to reproduce the same issue in your end:

Clone: https://github.com/iskdrews/chrome-extension-web3modal-boilerplate-react
npm install
npm build
Load your extension on Chrome following:
i.  Access chrome://extensions/ from your chrome browser
ii. Check Developer mode
iii. Click on Load unpacked extension
iv. Select the build folder.
You can find the extension appeared in the extensions bar.

Potential solution
I think you need to integrate this package (https://github.com/MetaMask/extension-provider) for reaching out to Metamask extension provider from another WebExtensions.
As you can see from this script (https://github.com/MetaMask/extension-provider/blob/master/index.js) it tries to create a connection stream to the Metamask chrome extension, so I think that might solve this issue.
Thanks in advance!

Don't think there is anything we can do here on our end. Under the hood we are using wagmi (https://wagmi.sh/) for connection which in turn uses plain ethersjs / window.ethereum, so perhaps these are having issues of looking for mm outside of the extension? Could be something to do with how extension persmissions work with window object, not sure

FYI I have the same issue- Chrome on Windows (Version 108.0.5359.125 (Official Build) (64-bit)). MetaMask Chrome extension version 10.23.2.

Don't think there is anything we can do here on our end. Under the hood we are using wagmi (https://wagmi.sh/) for connection which in turn uses plain ethersjs / window.ethereum, so perhaps these are having issues of looking for mm outside of the extension? Could be something to do with how extension persmissions work with window object, not sure

When using the wagmi examples page (https://wagmi.sh/examples/connect-wallet) I'm able to connect to MetaMask.
Also when using ConnectKit (https://docs.family.co/connectkit/try-it-out) that also works.
The web3modal example at https://web3modal-dev.pages.dev/ is unable to detect the MetaMask plugin is installed.

@TomSeldon Do you by any chance have deployed example / repo I could look at? And is there a chance to test this on a non windows machine (to pinpoint this down to windows os and chrome). Bellow is my test on Chrome via macOS and everything seems to work on all urls above.
I'll try to get something going on windows next week as well to test this, but os shouldn't play a role here to be fair, just browser.
https://user-images.githubusercontent.com/3154053/209807742-869aaada-a2bb-42a8-8353-183a883f12ea.mov

Hi @0xAsimetriq I'm sorry I don't have an example repo- I just tried the demo page (https://web3modal-dev.pages.dev/) and had this issue.
Whilst playing about with it I think I just found the problem- I had both MetaMask and Coinbase Chrome extensions installed and enabled. If I disable Coinbase, then when selecting MetaMask in web3modal it does trigger the MetaMask connection popup.
With both extensions enabled, it thinks MetaMask isn't installed.
I've not looked at the code, I'm guessing Coinbase is overwriting window.ethereum and so MetaMask's window.ethereum.isMetaMask is undefined instead of true.
wagmi examples and ConnectKit do seem to handle this scenario though, so I'm not sure what's different. Maybe they're doing something like looping through window.ethereum.providers and checking for MetaMask there? Complete guess though.

Hey, I too am making a chrome extension that I'm trying to connect with MetaMask. I've tried MetaMask-extension-provider but it didn't work. Have you had any luck connecting?

I have the same issue.

Image from https://web3modal.com/.
The one at the top of this image does not connect to the extension. The bottom one does.
This is definitely confusing.

@ThomasB013 please update to the latest version.


sorting the retry options and created a separate submenu under retry with button the menu

fixes #316
Changes Made:

[x] Created a submenu menu list under retry with menu button
[x] sorted the list alphanumerically

Adding @kliu57 for review, since she's been focused on ChakraUI this week, and has also worked on code that's related to this.

It's not clear that the "Retry With" menu item is clickable, or a sub-menu:

Compare that to native menus on my Mac:

We need some kind of arrow to indicate that you can open a list if you click. Compare to how the `Ask...` button works:

In this case, we might want the arrow pointing down or to the right.

Thank you for your suggestion. I just made the changes.

A few things to fix:

This shouldn't look like a button, it should look like the other menu items (e.g., no blue background, same font colour, etc)
It shouldn't be centered, but left-aligned like the other items

Can we make this just about sorting and sort out submenu in a followup?

Sure, sorry for scope creep (though he's fixed it now).

that menu does not work well for me on mobile

Can we make this just about sorting and sort out submenu in a followup?

Done #344


sed: 1: ".env": invalid command code when running ./register-dids.sh on mac

When running the ./register-dids.sh script on Mac, the script fails with errors sed: 1: ".env": invalid command code . at this point in the script:
if [ "$ARCHITECTURE" = "Mac" ]; then
            sed -i'' '/'"$1"'=/c\'"$1"'='"${SEED}"'' $DEST_FILE
else
...

I could reproduce and fix this on my Mac running 12.3 Can you test again after merging from main?

confirmed, thank you!


✨ [Feature request]: Add 'Edit Task' functionality to To-Do app

Is your feature request related to a problem? Please describe.
The To-Do app currently does not have an option to edit a task.
Describe the solution you'd like
An 'edit task' feature in the 'To-do App' which enables users to correct any typos or add extra info to the existing tasks.
Describe alternatives you've considered
NA
Additional context
Requires creation of additional states for edit and edit id.

#260


Added LocalizedData logic for Write-Verbose and errors to xComputer

Additional work to xComputer to progress the HQRM status and address issue #51

This change is

Hi @TheFish92, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Really awesome job @TheFish92 ! Thank you!
I've reviewed this with an eye to the move to HQRM, so there are some things I've requested that although aren't critical to this PR they may need to be done for the move to HQRM.
The reason why we use the New-InvalidArgmentExcepton or New-InvalidOperationException calls rather than "Throw" now is that it will create a custom exception object that we can actually test for.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 7 at r1 (raw file):

# Forcing en-US for dev purposes
$LocalizedData = Import-LocalizedData -FileName "MSFT_xComputer.strings.psd1" -UICulture 'en-US'

Any chance you could grab the localization code that is now being used for PSDSCResources? We're trying to standardize on this method where possible.
See https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/MSFT_GroupResource/MSFT_GroupResource.psm1#L65
No need to define the $LocalizedData as script scope though.
You'll need to copy https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1 into this module.

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 169 at r1 (raw file):

        if ($DomainName)
        {
            throw ($LocalizedData.SetDomainJoinNoCredential -f $DomainName)

Rather than "Throwing" an exception like this, can you call New-InvalidArgmentExcepton or New-InvalidOperationException found in the CommonResourceHelper.psm1 (that is copied in from https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1#L24)?

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 245 at r1 (raw file):

        if(!($Credential))
        {
            throw ($LocalizedData.TestDomainCredentialsNotSpecifiedFailure -f $DomainName)

Rather than "Throwing" an exception like this, can you call New-InvalidArgmentExcepton or New-InvalidOperationException found in the CommonResourceHelper.psm1 (that is copied in from https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1#L24)?

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 283 at r1 (raw file):

    if ($DomainName -and $WorkGroupName)
    {
        throw $LocalizedData.TestDomainOrWorkGroupFailure

Rather than "Throwing" an exception like this, can you call New-InvalidArgmentExcepton or New-InvalidOperationException found in the CommonResourceHelper.psm1 (that is copied in from https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1#L24)?

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 295 at r1 (raw file):

    catch [System.Management.Automation.MethodInvocationException]
    {
        Write-Debug $LocalizedData.TestDomainComputerNotMemberOfAny

Could you change this to a named parameter?

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 403 at r1 (raw file):

    )

    throw ($LocalizedData.TestNameIsValidFailureError -f $Name, $FailureReason)

Rather than "Throwing" an exception like this, can you call New-InvalidArgmentExcepton or New-InvalidOperationException found in the CommonResourceHelper.psm1 (that is copied in from https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1#L24)?

DSCResources/MSFT_xComputer/MSFT_xComputer.psm1, line 424 at r1 (raw file):

    if($Throw)
    {
        throw $Output

Rather than "Throwing" an exception like this, can you call New-InvalidArgmentExcepton or New-InvalidOperationException found in the CommonResourceHelper.psm1 (that is copied in from https://github.com/PowerShell/PSDscResources/blob/dev/DscResources/CommonResourceHelper.psm1#L24)?

Comments from Reviewable

@PlagueHO  No problem at all, I'll implement these changes tonight and let you know when I've amended the commit 😄

Sorry for the delay on this. Had a horrible coding block last week so decided to stay away from computers at the weekend. I'll finish the changes this week at some point. :)

Assigning abandoned label for now as there has been no response since @lfshr last comment

I'm going to close this one as I don't think it is required anymore because most of the changes have been incorporated into other PR's.


Bid Ask Prices

Is there a way to get the bid/ask prices of a stock? They used to be returned in the get_quote() method but they don't appear there anymore. I checked the requests log and it seems there is a request made to 'https://quotes-gw.webullfintech.com/api/stock/tickerRealTime/getDepth?tickerId=XXXXXX' to get the bid/asks. I tried making a request to this url with all the same headers that were used, but it doesn't return the bid/asks for some reason.

the get_quote() methord works.
I used this to get the ask/bid:
print(wb.get_quote("SPY")['askList'])
print(wb.get_quote("SPY")['bidList'])
I didn't modify the class so it should work for you.


Documentation restructure

Extended this PR a bit:

restructures readme layout to make it more consistent with other (large) react repos
added "Installation" section mentioning the stylesheets
updated the entry example (MyFirstGrid) to make things more understandable for new users (by using string keys)
made the entry example "copy/paste'able"
added a proper MIT license in LICENSE.txt

Please let me know if you would like to see anything changed in the license file mr. @STRML . Otherwise this should be good to go.
refs https://github.com/STRML/react-grid-layout/issues/135, https://github.com/STRML/react-grid-layout/issues/187

D**, I missed a copy/paste, for the npm install of all things. https://github.com/STRML/react-grid-layout/issues/195


gmpy prerequisite is broken on windows systems

On windows systems, pip install gmpy is broken, alas it needs a lot of prerequisites to be built correctly (first error message you get, is that the wheel cannot be built due to missing gmp.h, for anyone else out there trying to search for this issue, and on windows we don't have libgmp3-dev to install).
Now a "simple" pip install ludwig now fails due to missing header files and more.
I know you are not explicitly supporting Windows installs, but up to the H3 feature support (when gmpy was added) the package was able to be built\deployed.
I suppose it could be possible to pip install ludwig up to the point where all other packages are correctly installed and then do a final pip install ludwig --no-deps, and then manually commenting all H3 references in ludwig\features\feature_registries.py but it's a really broken way to workaround over this.
(in my case, with an added pip uninstall tensorflow and pip install tensorflow-gpu to leverage GPU processing)
Preliminary tests show that this "half-and-half" workaround with --no-deps is running fine with train\test\predict\visualize operations (of course not including anything relative to H3). I am also seeing that 0.1.2 models cannot be used with 0.2 due to Not found: Key optimizer/intent/fc_0/biases/Adam not found in checkpoint          [[{{node save/RestoreV2}}]]
I am looking into this to find if it's related to H3 commenting or if it deserves a separate bug report (even just to give a readable message stating the non retrocompatibility of 0.2 towards 0.1.2)
Thanks!

I'm looking to an alternative to gmpy at the moment. Will keep you posted.

FYI, gmpy2 2.0.8 has available pre-built wheels for recent python versions (not on pypy, and I don't understand why they don't mirror them over there).
https://www.lfd.uci.edu/~gohlke/pythonlibs/#gmpy
I could see that as a "clean" workaround for windows users (like: manually install gmpy2 by doing pip install gmpy2-2.0.8-cp36-cp36m-win_amd64.whl with the correct cpXX version depending on python environment and win32\amd64 architecture).
I don't know if that could be enough, but I suppose that migrating from gmpy to gmpy2 should be fairly straightforward (I haven't seen anything about that, nor I know\understand the purpose of this module, but I hope they have a similar shape\size :))
gmpy2 at least is actively mantained, and 2.1.0 beta came out in May, so we could hope for an extended support timeframe.

Unfortunately it is not straightforward as the functions I'm using are not available anymore in gmpy2. But don't worry the solution that I have working is a simple pure python alternative, will land in a couple days at most.

Thank you for your updates! Unfortunately, it still appears that gmpy is in the requirements list or somewhere in the ludwig install. On windows 10, pip install ludwig outputs all the requirements already satisfied and this:
Collecting gmpy (from ludwig)
Using cached https://files.pythonhosted.org/packages/26/37/2184c13cee81e1dbeaebbb13570195247e73ab2138a3db0c9d2c5347e372/gmpy-1.17.zip

Install from master.


Add a deploy step/job or new action

As part of the CI/CD process, we should have the ability for github to automatically pkg up and deploy updates to the website.
We will likely need to tar/untar the contents vis ssh/scp
Here is an example of how we may achieve something similar. The below is just an outline for a windows server, and we will want to modify it to work with our server (linux) and the build .json job you already completed.
We will need to add a few secrets to github to be used

[ ] ssh username
[ ] ssh password
[ ] server host
[ ] ....

name: CI/CD Pipeline - Build & Deploy

on:
  pull_request:
    types:
      - closed
    branches:
      # Pull Request Closed event for branches matching 'main'
      - 'main'

# Define global variable scoped to entire workflow
env:
  REPO_NAME: ${{ github.event.repository.name }}

jobs:
  Lint-Test-and-Validate:
    name: Lint, Test and Validate
    # If check for if pull request event was merged
    if: github.event.pull_request.merged
    runs-on: [ self-hosted, linux, x64 ]
    steps:
      # Get the code
      - name: Checkout Code
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      # Install Node 18.x
      - name: Set Node.js 18.x
        uses: actions/setup-node@v3
        with:
          node-version: 18.x

      # Install pkg manager pnpm
      # likely want to use NPM
      - name: Install pnpm
        uses<EMAIL_ADDRESS>        id: pnpm-install
        with:
          version: 7
          run_install: false

      # Get the cached version of pnpm
      - name: Get pnpm store directory
        id: pnpm-cache
        shell: bash
        run: |
          echo "STORE_PATH=$(pnpm store path)" >> $GITHUB_OUTPUT

      # Setup pnpm cache
      - name: Setup pnpm cache
        uses: actions/cache@v3
        with:
          path: ${{ steps.pnpm-cache.outputs.STORE_PATH }}
          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
          restore-keys: |
            ${{ runner.os }}-pnpm-store-

      # Install pkg dependencies
      - name: Install package.json dependencies
        run: pnpm install

      - name: Check output
        run: echo ${{ steps.vars.outputs.short_ref }}

      # Run super-linter action with the below variables
      - name: Super-Linter
        uses<EMAIL_ADDRESS>        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          LINTER_RULES_PATH: /
          CSS_FILE_NAME: stylelint.config.js
          JAVASCRIPT_ES_CONFIG_FILE: .eslintrc.js
          TYPESCRIPT_DEFAULT_STYLE: prettier
          VALIDATE_BASH_EXEC: false
          VALIDATE_BASH: false
          VALIDATE_CSS: false
          VALIDATE_EDITORCONFIG: false
          VALIDATE_GITHUB_ACTIONS: false
          VALIDATE_JAVASCRIPT_STANDARD: false
          VALIDATE_JSCPD: false
          VALIDATE_JSON: false
          VALIDATE_MARKDOWN: false
          VALIDATE_NATURAL_LANGUAGE: false
          VALIDATE_POWERSHELL: false
          VALIDATE_SHELL_SHFMT: false
          # Add a validate for python?

  # This workflow contains a single job called "build"
  Build:
    # Specify the environment to use branch rules
    environment: PROD_APPROVAL
    if: github.event.pull_request.merged

    # The type of runner that the job will run on
    runs-on: [self-hosted, linux, x64]

    # describe what checks are required for this build to proceed
    needs: Lint-Test-and-Validate

    # Env variables to be used in build step
    env:
      API_URL: 'www.example.com'
      API_KEY: ${{ secrets.API_KEY_PROD }}
      ROUTER_BASE: /"$REPO_NAME"
      NUXT_ENV: 'production' # not using Nuxt or React

    # Steps represent a sequence of tasks that will be executed as part of the job
    steps:
      # Checks-out your repository under "$GITHUB_WORKSPACE", so your job can access it
       # Get the code
      - name: Checkout Code
        uses: actions/checkout@v3
        with:
          fetch-depth: 0

      # Install Node 18.x
      - name: Set Node.js 18.x
        uses: actions/setup-node@v3
        with:
          node-version: 18.x

      # Install pkg manager pnpm
      - name: Install pnpm
        uses<EMAIL_ADDRESS>        id: pnpm-install
        with:
          version: 7
          run_install: false

      # Get the cached version of pnpm
      - name: Get pnpm store directory
        id: pnpm-cache
        shell: bash
        run: |
          echo "STORE_PATH=$(pnpm store path)" >> $GITHUB_OUTPUT

      # Setup pnpm cache
      - name: Setup pnpm cache
        uses: actions/cache@v3
        with:
          path: ${{ steps.pnpm-cache.outputs.STORE_PATH }}
          key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
          restore-keys: |
            ${{ runner.os }}-pnpm-store-

      # Install pkg dependencies
      - name: Install package.json dependencies
        run: pnpm install

      - name: Check output
        run: echo ${{ steps.vars.outputs.short_ref }}

      # Run command to generate the build
      - name: 'Generate Static Resources for Test environment'
        run: NUXT_ENV='${{ env.NUXT_ENV }}' ROUTER_BASE=${{ env.ROUTER_BASE }} pnpm generate

      # Tar and Gzip the nuxt build .output/public dir
      - name: Tar & Gzip build output
        run: tar -C "$GITHUB_WORKSPACE"/dist -czvf ${{ github.head_ref }}.tar.gz .

      # Upload artifact for persistence across steps/jobs
      - name: Upload build artifact
        uses: actions/upload-artifact@v2
        with:
          name: ${{ github.head_ref }}.artifact
          path: ${{ github.head_ref }}.tar.gz

  Deploy-to-NYMS:
    name: Deploy build to NYMS_HOST
    runs-on: [ self-hosted, linux, x64 ]
    needs: [ Build, Lint-Test-and-Validate ]

    steps:
      # Uses uploaded artifact from previous job
      - name: Download build artifact
        uses: actions/download-artifact@v2
        with:
          name: ${{ github.head_ref }}.artifact

      # Verify the tar file exists before moving on
      - name: Verify ${{ github.head_ref }}.tar.gz exists
        run: ls -l ${{ github.head_ref }}.tar.gz

      # Secure copy the tar file to the remote server
      - name: Secure copy ${{ github.head_ref }}.tar.gz to NYMS_HOST
        run: scp ${{ github.head_ref }}.tar.gz ${{ secrets.SSH_USERNAME }}@${{ secrets.NYMS_HOST}}:C:/_apps/builds/${{ github.head_ref }}.tar.gz

      # SSH into remote server, see if directory exists make one if not, extract the tar file and check if a tar file exists, delete if so
      - name: SSH and untar ${{ github.head_ref }}.tar.gz on NYMS_HOST
        run: >
          ssh ${{ secrets.SSH_USERNAME }}@${{ secrets.NYMS_HOST}} << "EOF"
            if (!(Test-Path C:\_apps\${{ github.event.repository.name }})) { mkdir C:\_apps\${{ github.event.repository.name }} }
            cd C:\_apps\${{ github.event.repository.name }}
            if (!(Test-Path ${{ github.head_ref }})) { mkdir ${{ github.head_ref }} }
            tar -xzvf C:/_apps/builds/${{ github.head_ref }}.tar.gz -C C:\_apps\${{ github.event.repository.name }}
            if (Test-Path C:/_apps/builds/${{ github.head_ref }}.tar.gz) { del C:/_apps/builds/${{ github.head_ref }}.tar.gz }
          EOF

Added a PAT for the club and made it a  secret... not sure if I made it the write one (been a minute) but give it a go and or see if you can find out which type of secrete/variable we need to create.
One in question is called NYMS_PAT


sink,scheduler (ticdc): add slow table and big txn log.

What problem does this PR solve?

Issue Number: ref #7437
What is changed and how it works?
Add log to help debug.

Log txn that contains more than 1024 rows, which can tell us there are some big txns.
If changefeed's checkpoint lag is larger than 30s, log the 4 slowlest table infos every minute, which can help us to find the problematic tables.

Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/run-integration-tests

/run-all-tests

/merge

/run-integration-tests

/run-integration-tests

/run-verify


How to display multi_part data

Hi,
This is more a question rather than an actual bug report.
I am using a Keyence Linescan camera that creates several images from a single capture. I am trying to display several of these images at once in the GUI but with no success. It seems to me this should be possible as I can see in the gui code that different gentl payload types are supported (including multi-part which is what I am interested in):
in canvas.py :
class Canvas2D(CanvasBase):     _visible_payloads = [         PAYLOADTYPE_INFO_IDS.PAYLOAD_TYPE_IMAGE,         PAYLOADTYPE_INFO_IDS.PAYLOAD_TYPE_CHUNK_DATA,         PAYLOADTYPE_INFO_IDS.PAYLOAD_TYPE_MULTI_PART,     ]
The Camera's proprietary GUI can show multiple images at once so I am assuming it is possible to do so with GenICam and harvester.
I am wondering what it is that I am missing to use this multi-part payload type. I can't seem to find this parameter in the Attribute Controller.
Could this be linked to the genTL producer that I am using and that does not support this?
I am using :

OS : Windows 10
Python 3.9.0
Harvester 1.4.0
Harvester GUI 1.0.2
Genicam 1.2.0
GenTL producer : Matrix-Vision Acquire SDK x64 2.48

Thanks a lot!
Jpl

Update on my previous issue. The problem was coming from my camera that was not sending a multi-part buffer in the first place. I needed to do new Image Acquirer acquisition without taking a new capture (had to change the MultiCaptureUpdateImage node value to False to do so).
The full explanation can be found in the online documentation to the camera if this can be useful to anyone. It is the Keyence VJ Series System Integration Manual :

I haven't found a way to display the images together in the UI however, but this actually isn't that much of an issue for me anyways.
Cheers,
Jpl


Better handling for token expiration

The app needs much better robust handling for when the token expires on the backend to detect that, stop future refresh updates (to prevent additional errors warning), show the notification (or maybe not) and then push the user to the SSO login page.  In dev mode, there is no SSO login page, so an error component or page would be nice (like TokenExpired :wink:).
Originally posted by @sjd78 in https://github.com/oVirt/ovirt-web-ui/pull/968#issuecomment-469367125

@bond95 @sjd78 do we still have work to do here?

@sgratch yes, we still don't have robust token expiration handling.

Here's a PatternFly 4 pass at an improved logged out empty state.


code-server 3.9.1 uses nodejs packages with vulnerabilities

We use code-server in docker images with reverse proxy and when you scan the image afterwards with trivy, we get a couple of hits in vulnerabilities:
+---------+------------------+----------+-------------------+---------------+---------------------------------------+
| LIBRARY | VULNERABILITY ID | SEVERITY | INSTALLED VERSION | FIXED VERSION |                 TITLE                 |
+---------+------------------+----------+-------------------+---------------+---------------------------------------+
| tar     | CVE-2018-20834   | HIGH     | 2.2.1             | 4.4.2, 2.2.2  | nodejs-tar: Arbitrary file            |
|         |                  |          |                   |               | overwrites when extracting            |
|         |                  |          |                   |               | tarballs containing a hard-link       |
|         |                  |          |                   |               | -->avd.aquasec.com/nvd/cve-2018-20834 |
+---------+------------------+----------+-------------------+---------------+---------------------------------------+

usr/lib/code-server/yarn.lock
=============================
Total: 2 (HIGH: 2, CRITICAL: 0)

+------------+------------------+----------+-------------------+---------------+--------------------------------------+
|  LIBRARY   | VULNERABILITY ID | SEVERITY | INSTALLED VERSION | FIXED VERSION |                TITLE                 |
+------------+------------------+----------+-------------------+---------------+--------------------------------------+
| ini        | CVE-2020-7788    | HIGH     | 1.3.5             | 1.3.6         | nodejs-ini: prototype pollution      |
|            |                  |          |                   |               | via malicious INI file               |
|            |                  |          |                   |               | -->avd.aquasec.com/nvd/cve-2020-7788 |
+------------+------------------+          +-------------------+---------------+--------------------------------------+
| node-forge | CVE-2020-7720    |          | 0.7.6             | 0.10.0        | nodejs-node-forge:                   |
|            |                  |          |                   |               | prototype pollution via              |
|            |                  |          |                   |               | the util.setPath function            |
|            |                  |          |                   |               | -->avd.aquasec.com/nvd/cve-2020-7720 |
+------------+------------------+----------+-------------------+---------------+--------------------------------------+

These seem to be part of the yarn.locks/package.jsons from the code-server base. I have tried to patch said packages by using resolutions in the package.jsons, but updating these packages seem to break code-server as it won't load vscode properly anymore.

Web Browser: Chrome
Local OS: Ubuntu
Remote OS: Ubuntu
Remote Architecture: Docker image
code-server --version: 3.9.1 e0203f2a36c9b7036fefa50eec6cf8fa36c5c015

Hey, so for each of those packages, looking at yarn.lock
I think I can update tar in the vscode yarn.lock, and ini in the code-server one, but node-forge is pulled in by parcel-bundler, which hasn't seen an update in over a year.
I'll see what I can do in the next hour.

Hey, so I meant to open a PR and everything but I fricked up and ended up committing directly to main...
https://github.com/cdr/code-server/commit/da65b8e8d7b539f5a8c34a918d78783e7742b630
Oops.

Thanks for solving it so quickly though!


Correct kernel-output example

you instead of we
ReactWidget instead of VDomRenderer
example instead of tutorial
update preview
rewording
correct broken behavior: showing the dataframe was creating a panel each time instead of using the existing panel (regression when porting to JLab v1)

Thanks @fcollonval for updating this example!
It looks great overall, and I left a couple of minor comments.

Thanks again for the review @jtpio


[BUG] Fix version supportability matrix in official guide.

What is the bug?
In supportability matrix, it's mentioned corresponding to "OpenSearch 1.x to 2.x" older versions are supported with "Yes via version setting" But version setting is no longer available in opensearch 2.x onwards hence we should only mention "Opensearch 1.x"
https://opensearch.org/docs/latest/clients/agents-and-ingestion-tools/index/
Refer this issue for details https://github.com/opensearch-project/OpenSearch/issues/3023

@nknize: Can you confirm that we should go ahead and make this change? I thought we added the version override setting back in for 2.1 (https://github.com/opensearch-project/OpenSearch/pull/3530).

@nknize: Can you confirm that we should go ahead and make this change? I thought we added the version override setting back in for 2.1 (opensearch-project/OpenSearch#3530).

We did add the override back in for 2.1 so I don't think we should make the change.

@nknize @ervikrant06 @Naarcha-AWS Is a documentation update still needed on this issue? The issue is closed but was assigned to me. I search documentation and haven't located any files with the verbiage "OpenSearch 1.x to 2.x."

@vagimeli I am good with closure. thanks


Upload 2 times the same subtitle file on the row does not work

Bug Report
Expected behavior/code
The upload should work or give an alert.
Actual Behavior
If I upload 2 times the same subtitle file on the row, that does not work.
Steps to Reproduce

Create a VOD room as a instructor
Upload a srt file in the subtitle widget
Delete the subtitle
Try to upload again
And then the bug happens!

Environment

Marsha version:
Platform: Windows

I can't reproduce.
Are you selecting the same language again ?

Reproduced only on chromium based brosers (chromium, chrome, brave, etc)

Do you have Bitwarden extension enabled ? I just found out that Bitwarden sometimes adds an unwanted attribute to html elements : data-com.bitwarden.browser.user-edited="yes"
This is an old issue which still seems to be pending on their side : https://github.com/bitwarden/clients/issues/725
In our case, the bitwarden attribute goes on the hiddenFileInput that represents the file we want to upload. When removing it the button works as expected.

In my case I don't have the bitwarden extension installed in any browser.


shell: uart: simplify RX ISR handling

Simplify code for handling RX data in ISR, but throwing out SMP_SHELL
ifdeffery out to smp_rx_byte() helper function. This function is nicely
thrown away when SMP_SHELL is disabled, resulting also in throwing away
'while (rd_idx < rd_len)' inside uart_rx_handle(). When SMP_SHELL is
enabled, then smp_rx_byte() is just a noop wrapper around
smp_shell_rx_byte().
This patch also solves the case when SMP bytes are received from UART
fifo after shell bytes (AABB, where AA means shell data, BB means SMP
bytes). Previous implementation was not handling it properly, because
first non-SMP byte resulted in breaking the loop. Currently we continue
while loop unconditionally.
Signed-off-by: Marcin Niestroj<EMAIL_ADDRESS>

I've added DNM and RFC, because I have not tested this change besides building it. I would like to hear opinion on this approach first. So far I've noticed reduced ROM usage when SMP_SHELL is enabled.

Just resolved merge conflicts.


Build error, why doesn't it use all the files in the directory?

https://travis-ci.com/probonopd/appimage/jobs/251219352#L359
https://stackoverflow.com/questions/17539407/how-to-import-local-packages-without-gopath

All the .go files in a single directory are part of the same package, and you don't need to import files in the same package (i.e., the same directory). You mentioned working outside

Why does it not work?
All the complicated stuff about GO111MODULE, vendor, modules... is un-go-like. Why can't it "just work"?

Found something that works, but it's really a mess


Preserve compiled libs

ATM emscripten image compiles all ports, but do not preserve output (due a size ~1GB), this might be done as a separated image emscripten-full

I'm also running into the issue that libclang.so is missing, which makes eg Rust builds fail.

@remram44 Hi, libclang is a part of https://packages.debian.org/stretch/libclang-3.9-dev
Can you try to run: sudo apt install libclang-3.9-dev before running the compilation?
I can complete the emscripten image with the missing dependency.

I'm getting:
/emsdk_portable/sdk/system/include/emscripten/emscripten.h:135:71: error: 'deprecated' attribute takes no more than 1 argument, err: true

if I install libclang-dev from apt first.

Interesting,
https://github.com/kripken/emscripten/blob/incoming/system/include/emscripten/emscripten.h#L135
It looks that this doesn't help at all to install the libclang-some of clang tools is used in compilation (from version 3.9 what has different signature for deprecated method).
I'll look if we can compile / preserve libclang from emscripten compilation step.

@remram44 I found it, it can be preserved. Do you have some minimal test case what I can run to verify if my patch works?

I'm happy to test this. I can try to make a smaller Rust example.

Here's my minified example: https://github.com/remram44/test-rust-emscripten

Seems to work with the libs, and not without it. I'll let you know if my full project compiles, seems to be taking a while (would it be building the SDL2 port behind the scenes?)

Yes, none of ports are baked into the image.
I was able to reproduce error and test a hack fix. I'll test the branch from and once it works fine I'll update sdk-incoming.
Thank you for reporting

I can't seem to build the port, emcc ends up spawning more processes endlessly until my system runs out of memory (827 python /emsdk_portable/sdk/emcc processes right now); But I might be doing something wrong? (just ran emcc -s USE_SDL=2 helloworld.c)

Turns out setting EMMAKEN_CFLAGS='-s USE_SDL=2' causes the port build to recurse indefinitely. Looks like an upstream bug. Got everything working now, thanks for your very quick help!


Problem with FbxImportUI

Hi there,
I think that the FbxImportUI is not working properly. I have an FBX file with two boxes. When I import using the Unreal GUI, I can specify if I want to "CombineMeshes" or not and depending on my settings it will import one combined or two unique assets from this single FBX file. When I do that with Python, it imports fine, but it ignores the python settings and it uses the latest settings the user used.
Bellow you can see a code sample I tried and doesn't work.
import unreal_engine as ue
from unreal_engine.classes import FbxImportUI, PyFbxFactory

fbx_import_ui = ue.new_object(FbxImportUI)
fbx_import_ui.set_property('bCombineMeshes', False)
ue.set_fbx_import_option(fbx_import_ui)
ue.import_asset(r"C:\Users\n.moutafis\Desktop\TwoBoxes.FBX", "/Game/FBX", PyFbxFactory)

This has been fixed in latest releases, check here for an example: https://github.com/20tab/UnrealEnginePython/blob/master/examples/kaiju_builder.py#L20


All bottomnavigation tabs, Notification with redesign Implemented

Screenshots

Description

[x] Run the unit tests with ./gradlew check to make sure you didn't break anything

[x] If you have multiple commits please combine them into one commit by squashing them.

@abhijitramesh Please review it.


Update ita-it.xml

Updated typos and generally misconstrued sentences. Basically fixed Italian.

Thanks!


Track what is missing

I should've done this from the beginning, but we should track what is missing from the keybinding file due to something not being supported in Zed.  Some sort of table for each keybinding file - maybe something like this?

intelij action name
intelij key combo
zed action name
zed key combo

action
combo
action
combo

action
combo
N/A
N/A

Maybe it's best to not add the overhead of maintaining these.  Maybe it's best to just allow the community to open PRs when support for new actions in the keymaps land.  I think that is the way to go with this, so I'm going to close this issue out.


Display Tokens Brainstorming

We need a way to display tokens... before we implement a solution lets brainstorm.
If you have examples or ideas please share, then we can implement.

Design were going for...


App Analytics

Tried to get App Analytics working, but did not get all the way through with the following:
// Retrieve app analytics data for the TestFlight app
const getAppAnalyticsData = async (api: any) => {
  try {
    // Define the time period for the app analytics data you want to retrieve
    const startTime = Math.floor(Date.now() / 1000) - (24 * 60 * 60);
    const endTime = Math.floor(Date.now() / 1000);

    // Define the query parameters for the App Store Connect API request
    const queryParams: QueryParams = {
      'filter[timeRange][start]': startTime,
      'filter[timeRange][end]': endTime,
      'filter[version]': bundleId,
      'group[granularity]': 'DAY',
      'measure': 'installs,uninstalls'
    };

    const url = `/v1/analytics/applications/${appId}/data`;
    const method = 'GET';
    const data = await api.request(url, method, queryParams)
    return data;
  } catch (error) {
    console.error(error);
  }
};

Others are trying as well

https://developer.apple.com/forums/thread/718674
https://developer.apple.com/forums/thread/29421
https://developer.apple.com/forums/thread/115392?page=2

I am also looking for a way to query analytics data. Unfortunately, Apple does not currently open the relevant API.


Add check for facet provider requirements

Contains
Fixes #4924
Todo

[x] Test that CoreGameplay world gen still works as expected
[x] Test that focus gameplays' world gen still works as expected (JS, MR, LaS)
[x] Test that logic correctly fails any attempt to add a facet provider with unsatisfied requirements
[ ] Fix failing tests

@MrGizmo123 I pushed a PR slightly changing the error log and adding throwing an IllegalStateException.
Thus, world generation fails and developers get direct and clear feedback when trying to test their world gen provider.
This can be tested, for instance, by adding FloraFacet.class to the required facets in CoreWorlds:BiomeProvider and trying to start a CoreGameplay world.

Seems like the tests failed because the Facet1Provider in the test cases in WorldBuilderTest requires Facet2Provider but was added before it so far. This should be fixed now.

I added a test case that tests that if facet providers are added in an incorrect order, an exception is thrown.


Release Event Hubs April 2020 update for Python

This is a parent issue tracking the work planned for Event Hubs for the April update.
Code complete date: April 3rd, 2020
API design

[ ] In Progress Propose smarter send APIs such that the user doesnt have to one by one fill the EventDataBatch to be able to send events
[ ] In Progress Propose API to control greedy claiming of partitions by the Event Processor

Features

[x] Receive in batches
[x] Code sample showing how to convert IotHub connection string to an Event Hubs compatible one which points to the built-in Event Hubs endpoint
[x] Code sample showing workaround needed for Event Hubs in Azure Stack
[ ] Receiver Redirect feature
[ ] Review current logging experience, come up with a plan to streamline and be consistent across languages for ease of customer support, tackle low hanging fruits

cc @YijunXieMS

The April update which is a preview contained the below

receive in batches
Code sample showing workaround needed for Event Hubs in Azure Stack
Add in missing system properites when reading from Iothub. #10408

Rest of the work is being moved to the May update.
@YijunXieMS, Please provide a link to the April update here and close the issue

Released 5.1.0b1 on 04/06/2020
https://pypi.org/project/azure-eventhub/5.1.0b1/


added health management system program

Type of change

[x] Added new project
[ ] Bug fix
[x] New features
[x] Enhanced documentation

Changes proposed in this pull request

Project name: Health management system

Short description: This program keeps a record of patients wherever they executed for food or exercise.

Checklist

[x] The code is Python 3 compatible
[ ] I have added relevant comments in my file to make the code understandable
[x] I have added my project to a folder that includes all the necessary files

Looks good , thanks for contributing 👍


run.py on windows 11 fails

python .\run.py
Lynx Version 1.0.11ceddf
INFO     {acceleratesupport.py:17} : No OpenGL_accelerate module loaded: No module named 'OpenGL_accelerate'
INFO     {special.py:36} : Using NT-specific GLUT calls with exit callbacks
Traceback (most recent call last):
File "C:\Users\pc\Lynx\lynx\main.py", line 89, in
utils.adblock.read_filter(confvar.BASE_PATH + "adblock/filter.txt")
File "C:\Users\pc\Lynx\lynx\utils\adblock.py", line 36, in read_filter
filter_set.add_filter_list(f.read())
File "C:\Users\pc\miniconda3\lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 1287723: character maps to

The adblock filter should be read in utf-8 format. Append argument encoding='utf8' in open(path) Line: 35, file: adblock.py
This should force the file be read in utf-8 format, instead of Windows cp1252 format.


ISSUE Focus using the F button and focus centre.

I am having a few issue with this, it seems a bit counter intuitive to what the focus point is.
When focusing with F, some of the buttons are getting cut outside the viewer,  it seems counter productive, since it requires the user to zoom out and recentre with the mouse.
The way i would see this working is like this :
With No buttons selected inside the viewer, pressing F would focus the entire buttons layout and zoom to fit in the scaled UI view.(you would see everything)
With Multiple buttons selected, pressing F would centre focus on the middle point between the selected buttons and zoom the UI to fit the view of such buttons.(eg 3 buttons are selected, they are fully visible in the UI view)
With One button selected, pressing F would focus and zoom the UI view to the selection.

You can configure the focus center by picker using that button:

An auto fit-content is wip when the user is adding a button out of the picker.
Design a Focus fitting to content is a lot of work, I'll add it to our TODO but do not wait it for too soon.

Thank you for the quick update, i was wondering if a "focus on the current mouse point on the UI" would be a quicker more viable solution to implement.
That way perhaps, when you select something and press F, the center of focus moves to the mouse position on the canvas, and when nothing is selected, the view gets simply zoomed out to show the full background of the layout.
Either ways, thank you again for the tool, it is very solid work , i know how tricky it is to implement these type of behaviors in Qt.

Hello, when embedding a picker window in maya, it intercepts hotkeys such as:
ctrl+z
ctrl+s
F

This is another topic, i created another issue

Hello,
We just added the feature in the latest commit. It will be there in the next release !


[AutoPR hdinsight/resource-manager] [HDInsight] Fixed PUT application parameter

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3565

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/1986


NIFI-4816: Allow name to be updated for ReportingTasks

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@mattyb149 thanks for the update. This is definitely something that I think we should have added at the start. I'm curious about why the update to all of the reporting tasks, though. None of them make use of the name, as far as I can tell, so why both storing it off? Moreover, if we do want to store it off, then why not just have the setName() method have an @OnScheduled annotation and take in the ConfigurationContext. Then the abstract impl will automatically have the name set. It would not necessarily be available if being called from other @OnScheduled methods, but I think that's okay as long as it is documented - in such a case, you should call the method on the ReportingContext itself.

The idea for storing off the name for all reporting tasks is to support #2431 , in fact that's where this behavior was discovered.
I like the @OnScheduled setName() idea, I may call it something more specific (rather than a bean method) since it will take a context. Will make the change, thanks!

@mattyb149 this looks good to me now! Thanks for updating. I'm not in a position to merge right now, but I will give it a +1 if you want to merge yourself. Otherwise I will merge when I get my local branch squared away. Thanks!


Use atomic commits for DRM rendering

Use atomic commits for DRM rendering, and also add support for using pure python kms module if available, instead of current C++ bindings

@will-v-pi Thanks for this! LGTM. Are you happy to go ahead and merge?
It seems a bit annoying that we have to invoke the garbage collector explicitly, this does seem to be a bit of a sore point with Python sometimes.

Yeah, I'm happy to merge - with the garbage collection, I added that as the card only seems to get garbage collected once a new preview is started rather than when a preview is stopped, which was causing issues in the drm stop restart test


[CLOSED] AccessDeniedException

 Issue by richardpayne
Tuesday Mar 22, 2016 at 16:40 GMT
Originally opened as https://github.com/into-cps/INTO-CPS_Application/issues/8

When running a co-simulation using the INTO-CPS COE I get an AccessDeniedException. I am not sure what is causing this!
java.nio.file.AccessDeniedException: C:\Users\Richard\AppData\Local\Temp\fmi-shm-Windows-amd64\sharedmemory.dll
at sun.nio.fs.WindowsException.translateToIOException(Unknown Source)
at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
at sun.nio.fs.WindowsFileSystemProvider.implDelete(Unknown Source)
at sun.nio.fs.AbstractFileSystemProvider.deleteIfExists(Unknown Source)
at java.nio.file.Files.deleteIfExists(Unknown Source)
at java.nio.file.Files.copy(Unknown Source)
at org.intocps.fmi.jnifmuapi.NativeLibraryLoader.extractNativeLibrary(NativeLibraryLoader.java:101)
at org.intocps.fmi.jnifmuapi.NativeLibraryLoader.loadLibrary(NativeLibraryLoader.java:38)
at org.intocps.java.fmi.shm.SharedMemory.(SharedMemory.java:31)
at org.intocps.java.fmi.service.ProtocolDriver.(ProtocolDriver.java:20)
at org.crescendo.fmi.ShmServer.main(ShmServer.java:23)
ERROR main - Faild to load native Library file: lib/Windows-amd64/sharedmemory.dll
java.io.FileNotFoundException: Unable to find the native lib: lib/Windows-amd64/sharedmemory.dll
at org.intocps.fmi.jnifmuapi.NativeLibraryLoader.loadLibrary(NativeLibraryLoader.java:42)
at org.intocps.java.fmi.shm.SharedMemory.(SharedMemory.java:31)
at org.intocps.java.fmi.service.ProtocolDriver.(ProtocolDriver.java:20)
at org.crescendo.fmi.ShmServer.main(ShmServer.java:23)
Exception in thread "main" java.lang.ExceptionInInitializerError
at org.intocps.java.fmi.service.ProtocolDriver.(ProtocolDriver.java:20)
at org.crescendo.fmi.ShmServer.main(ShmServer.java:23)
Caused by: java.lang.reflect.UndeclaredThrowableException
at org.intocps.java.fmi.shm.SharedMemory.(SharedMemory.java:36)
... 2 more
Caused by: java.io.FileNotFoundException: Unable to find the native lib: lib/Windows-amd64/sharedmemory.dll
at org.intocps.fmi.jnifmuapi.NativeLibraryLoader.loadLibrary(NativeLibraryLoader.java:42)
at org.intocps.java.fmi.shm.SharedMemory.(SharedMemory.java:31)
... 2 more

 Comment by CThuleHansen
Monday Apr 04, 2016 at 10:12 GMT

Is this related to #15 ?

 Comment by CThuleHansen
Monday Apr 04, 2016 at 11:43 GMT

Do you have any data so I can reproduce?

 Comment by lausdahl
Thursday May 19, 2016 at 10:51 GMT

Hi @richardpayne can yo try updating the overture-fmu export and check if this still happens. I just added a fix which should create a new temp folder for each process and delete it on exit.
I will close the issue so please re-open if it happens again.


Ref #4716: java-joor-dsl - Add RegisterForReflection support

fixes #4716
Motivation
All the annotations added to a class managed by the extension java-joor-dsl are not considered by Quarkus since it is not part of the Jandex index.
This change aims to support the annotation RegisterForReflection to easily configure the reflection in Camel-K.
Modifications:

Produce the corresponding ReflectiveClassBuildItem and  LambdaCapturingTypeBuildItem if a RouteBuilder is annotated with RegisterForReflection.
Add some doc to explain the limitations
Add a unit test to ensure that it works as expected

Just a nitpick question, beyond Will Smith PR looks good to me ;)

I'm still a great fan of Will Smith, sorry for that 😅


feat(DDI-1001): Adding form item examples

Removed Form Row example.
Added examples of the form item wrapping: Radio, Checkbox, Dropdown, Button

:tada: This PR is included in version 4.0.0-alpha.67 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.111 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Browser zooming

Had a user of my theme complain that they were losing the grid when zoomed in 125% at a certain browser width.
For example, if you zoom in 125% in Chrome 52.0.2743.116 m and resize the browser so it's at the exact pixel before the regular grid turns into the tablet grid (min-width: 1025px in my case), no grid applies for that 1px.
This doesn't happen at regular zoom, it goes from min-width:1025px to max-width:1024px flawlessly.
It only happens when zoomed in.
If I switch it to max-width:1024.9px, it works fine.
I'm assuming the browser is adding decimal points to the width, which is causing the issue.
Is there a better way around this? Maybe there's something logically wrong with my code?
Here's an example of what I'm doing: http://codepen.io/tomusborne/pen/dXaxOj
Thanks!

Found a fix that works: http://stackoverflow.com/questions/38981132/media-queries-browser-zoom
Might be worth considering adding to the responsive file :)

Interesting. Sorry for the delayed reply, I haven't had time to dig into this yet.
So, is that due to "retina" operating at a half-pixel level?

No worries.
I was pulling my hair out, one of my users kept bringing it up and it made no sense to me.
I don't think it's retina, it's just the way Chrome (maybe others) sets the browser size when zoomed in. At some point neither 1024px or 1025px are true (when it's in the middle).


'Error: L is not defined' ?

Hi,
New to leaflet and this module. Just cloned it and put it into my app andi am getting this in the console:
Error: L is not defined
_getDefaults@http://localhost/assets/angular-leaflet-directive/dist/angular-leaflet-directive.js?body=1:1667:13
.setDefaults@http://localhost/assets/angular-leaflet-directive/dist/angular-leaflet-directive.js?body=1:1740:31
This is the latest version. Something i am doing incorrectly?
Thanks for the cool mod!
cheers. --mark

You must load leaflet library first. Closing.