[Bug]: Create variable for dependently typed functions add targetType as variable type

Description
I have a dependently typed function in a client. When I call a that function and try to create the variable using the Create variable code action in VS code, it creates a variable with type targetType which is incorrect.
Steps to Reproduce
Use the following code with Ballerina SwanLake Update 9 in VS code:
import ballerina/http;

type Album record {|
    string name;
    string artist;
    int year;
|};

type ErrorMsg record {|
    string message;
|};

type AlbumOk record {|
    *http:Ok;
    Album body;
|};

type AlbumNotFound record {|
    *http:NotFound;
    ErrorMsg body;
|};

client class AlbumClient {
    resource function get albums/[string id](typedesc<AlbumOk|AlbumNotFound> targetType = <>)
        returns targetType = external;
}

public function main() returns error? {
    AlbumClient albumClient = new;
    albumClient->/albums/["1"];
}

Try the Create Variable code action for the client call:
https://github.com/ballerina-platform/ballerina-lang/assets/63336800/5c4961b8-5d81-4920-aedb-f663877fc5dc
Expected something like this:
public function main() returns error? {
    AlbumClient albumClient = new;
    AlbumOk|AlbumNotFound targetType = albumClient->/albums/["1"];
}

Affected Version(s)
Ballerina SwanLake Update 9
OS, DB, other environment details and versions
Mac OS
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

There are two concerns with this scenario. The first one is a bug which produces the Create variable code action in place of the create variable with type code action. This only occurs when the type consists solely of record type caused by the following code.
https://github.com/ballerina-platform/ballerina-lang/blob/2b9e2332af7adcf91ed148ad437a5c1ae6abbd09/language-server/modules/langserver-core/src/main/java/org/ballerinalang/langserver/codeaction/providers/createvar/CreateVariableWithTypeCodeAction.java#L276
Ideally, it should produce the following code actions:

Create variable with 'AlbumOk'
Create variable with 'AlbumNotFound'

The second concern is an improvement in which the user expects the union type as a code action as well. One solution would be to provide the respective code action along with the already existing ones, i.e.:

Create variable with 'AlbumOk|AlbumNotFound'
Create variable with 'AlbumOk'
Create variable with 'AlbumNotFound'


No questions to display working

I put in all these comments and then realized that you were still going to do more work on this later, as we discussed in our meeting this afternoon. Forgot! Anyways more food for thought here. :)

Closing this old PR for now as it isn't in a merge-able state and we're working on it as part of https://trello.com/c/46s9TZ9i/321-if-i-have-no-questions-there-should-be-some-message-when-i-click-mine-in-review-questions-to-show-that-instead-of-a-blank-page.


Send Email to 2 Email address

I want to integrate 2 email addresses as delivery emails. I tried manipulating the lambda code "voice-mail\handler\voicemail.js" but I'm not able to get an exact solution to modify it.
Also, how do I roll out new updates to code? Just zip it and upload to S3 or we need CodeBuild pipeline.

Hi, you would be able to test your code changes with a simple upload. No need for CodeBuild pipeline.
Please follow the instructions here https://github.com/amazon-connect/voicemail-for-amazon-connect/tree/master/source/aws-connect-vm-serverless to set up your development environment

Hi, @angieyu thanks for the reply.
Is there any timeline or change that I could implement in order to send emails to multiple recipients?


feat(module): azure vm module

Warning
Avoid pasting images in issues!

Purpose
New reusable terraform module for azure virtual machines.
Detailed description
The Idea is to create  a reusable terraform module for azurerm provider that will simplify creating windows and linux virtual machines.
Following links refers to current standard from terraform docs azurerm provider:
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/windows_virtual_machine
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/linux_virtual_machine
Tasks

[ ] Create reusable module with name terraform-azurerm-vm

Following needs to be done withing a virtual machine module or created as separate modules:

Vnet
Subnet
Network Security Group
NIC
Disc

Affected Items

[x] None

Observations:

Vnet and subnet can be created using existing module network: https://github.com/equinor/terraform-azurerm-network.

Suggestions:

Create virtual machine using existing module vm: https://github.com/equinor/terraform-azurerm-vm.
Create network security group module nsg as submodule of network.
Create network interface module nic as submodule of network.
Create disk module disk as submodule of vm.

That is great. This is our Internal VM Module for reference

Updated task list.

The module repo has been created, although it needs configuring. Is there any plans for further work on this module? @equinor/terraform-baseline
Ref.: https://github.com/equinor/terraform-azurerm-vm

@BouvetMorten You were talking about implementing an initial module based on your experience?

That is the plan. We have created a task to convert and move our local VM module to the terraform-azurerm-vm

@hknutsen @helenakallekleiv Will you do the configuring of the repo that is needed? And we'll have a look at creating the actual model.

@BouvetMorten Repo has now been configured with the appropriate accesses and settings.

@sefornes Hijacking this until initial module has been approved by all teams 😉

Waiting for feedback on initial POC from Azure Champions team.

Organized issue title tasks/pull requests list in main comment.
Since this issue includes several tasks and pull requests, we are considering it as a collection (Collection is a group of tasks/issues)
Task: #106 added.

just report back to @hknutsen when you have something new there @vildeaar.

Pull Requests in VM module repo added as well in main description.


eqos-1931 Scripted portlet - HTML rendering

Create a new scripted portlet
Enter a Title and the text below in the Freemarker/HTML section

Test1
Test2

Test3
Test4

The new scripted portlet should be added and the  and  should be rendered correctly
 should produce a numbered list,
 should produce a list with bullet points.
Ordered list works, bulleted list does not.

Also broken in 6.4QA3

Fixed


Cli testing and acorn rm refactor

Issue: #868
Issue: #346

I branched this off of my acorn rm branch to write tests against that so this includes both unit testing for cli and acorn rm refactor changes @cjellick


Allocation test failure on main

This test failure shows up sometimes. Possibly since #1399.
No Captured Logs for test item "Allocation level control" at test\allocation_test.jl:308 on worker 7960
Error in testset "Allocation level control" on worker 7960:
Test Failed at D:\a\Ribasim\Ribasim\core\test\allocation_test.jl:364
  Expression: ≈(storage[stage_5], u_stage_5.(t[stage_5]), rtol = 0.0001)
   Evaluated: [1654.8209307353425, 1611.6209217322994, 1596.0102056697474] ≈ [1654.8209307353425, 1611.6209307353424, 1568.4209307353424] (rtol=0.0001)

Error in testset "Allocation level control" on worker 7960:
Test Failed at D:\a\Ribasim\Ribasim\core\test\allocation_test.jl:[372](https://github.com/Deltares/Ribasim/actions/runs/8802274133/job/24157582821#step:6:373)
  Expression: ≈(storage[stage_6], u_stage_6.(t[stage_6]), rtol = 0.0001)
   Evaluated: [1533.1275614894184, 1499.9856889683385, 1499.9856107363594, 1499.985610656249, 1499.985610653905, 1499.9856106538211, 1499.9856106538184, 1499.9856106538184, 1499.9856106538184, 1499.9856106538184, 1499.9856106538193] ≈ [1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184, 1533.1275614894184] (rtol=0.0001)

In table form:

evaluated
expected
difference

1654.820931
1654.820931
0

1611.620922
1611.620931
-9E-06

1596.010206
1568.420931
27.58927

and

evaluated
expected
difference

1533.128
1533.128
0

1499.986
1533.128
-33.1419

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

1499.986
1533.128
-33.142

This was probably solved with https://github.com/Deltares/Ribasim/commit/e84ae3a180136a394965a2de87ab9728883f868f, let's merge that separately.

That makes sense, see #1424.


HoA snap pkg not upgrading since 425 while 446 available

This maybe why I may not be getting feature/bugfix updates.

Due to my lack of understanding about snap, I haven't been aware that the snap pkg hasn't been upgraded, nor I have even stopped and re-started this HoA instance.
snap info
uooooo@localhost:~$ snap info home-assistant-snap
name:      home-assistant-snap
summary:   Open source home automation that puts local control and privacy first
publisher: Giaever.online (giaever-online)
store-url: https://snapcraft.io/home-assistant-snap
license:   Apache-2.0
description: |
  Home Assistant is a free and open-source software for home automation that is designed to be the
  central control system for smart home devices with focus on local control and privacy.

  It can be accessed via a web-based user interface, via companion apps for Android and iOS, or
  using voice commands via a supported virtual assistant like Google Assistant or Amazon Alexa.
  Home Assistant integrates with over a thousand different devices and services.

  Read more about Home Assistant at
  - https://en.wikipedia.org/wiki/Home_Assistant
  - https://www.home-assistant.io/

  **NOTE!** This is Home Assistant Core (not supervised).

  Your new installation will be available through your browser at http://localhost:8123/ (or change
  localhost to any other IP address where the snap packages is installed). You will get a launcher
  available in your application menu on desktops.

  **Important information about updates**

  Since each major versions of Home Assistant might bring breaking changes, you will **only**
  auto-update for minor releases. Each major release must be upgraded manually, by switching
  channel.

  See available channels in the software center or with
  `snap info home-assistant-snap`
  Please use the stable versions, e.g `2021.8/stable`.

  You can switch channel with:

  `
  $ snap switch home-assistant-snap --channel=<channel>
  `

  where `<channel>` could be `2021.8/stable`.

  There is a binary sensor `binary_sensor.updater` that will flip to `true` from `false` whenever
  there are updates available in a newer channel. Use this to show a notification whenever a new
  major release is available.

  **Need help?**

  Please use https://www.home-assistant.io/docs/ for documentation and
  https://community.home-assistant.io for  help from the community.

  **Necessary plugs**
Can't connect to your USB (e.g Zigbee, Z-wave, removable drive)?

  See `snap connections home-assistant-snap` for necessary plugs (such as access to USB, removable
  media etc) and connect them with

  `$ snap connect home-assistant-snap:<plug>`

  where `<plug>` can be in example `raw-usb`.

  **Required plugs**

  - *network:* allows Home Assistant access to the network.
  - *network-bind:* allows operating as a network service, to run the Home Assistant server.
  - *network-control:* enables the configuration of networking and network namespaces via `ip
  netns`, providing a wide, privileged access to networking. This is required for the DHCP discovery
  integration to work which will watch the network for DHCP requests for supported devices and
  services. To disable this feature you will have to disable the `default_config` from you
  `configuration.yaml` and disconnect the `network-control` plug with `snap disconnect
  home-assistant-snap:network-control`.

  **NOTE!** Removing the `default_config` will require you to build up a whole new configuration for
  Home Assistant. More on what is enabled by default at this link:
  https://www.home-assistant.io/integrations/default_config/

  **Optional plugs**
  - *raw-usb:* allows access to all connected USB devices via a raw interface.
  - *removable-media*: allows read/write access to mounted removable storage in /media, /run/media
  and /mnt.
  - *serial-port*: enables access to a specific serial port. This interface is restricted because it
  provides privileged access to configure serial port hardware. Might require hot-plug support `snap
  set system experimental.hotplug=true` followed by `systemctl restart snapd`.

  **Available slots**
  - *components:* Load custom components to Home Assistant. You will find the Home Assistant
  Community Store package here: https://snapcraft.io/home-assistant-hacs - Installations on own risk
  and note that provided integrations are not tested or guaranteed to work.
  - *bin:* Missing binaries might be in the Home Assistant Toolbox, such as cURL etc. See
  https://snapcraft.io/home-assistant-toolbox
  - *configurator:* Edit your configuration files with HASS configurator. There is a ready package
  called Home Assistant Configurator here: https://snapcraft.io/home-assistant-configurator.
commands:
  - home-assistant-snap.check-config
  - home-assistant-snap.open-localhost
services:
  home-assistant-snap: simple, enabled, active
snap-id:      BYa8IlUQqqTyhwCjoPWb8Pf2ZaBKQdSv
tracking:     2021.10/stable
refresh-date: 27 days ago, at 06:27 UTC
channels:
  2021.11/stable:    2021.11.5  2021-11-26 (446) 471MB -
  2021.11/candidate: ↑
  2021.11/beta:      ↑
  2021.11/edge:      ↑
  latest/stable:     2021.11.5  2021-11-26 (446) 471MB -
  latest/candidate:  ↑
  latest/beta:       ↑
  latest/edge:       2021.12.0  2021-12-11 (449) 472MB -
  2021.10/stable:    2021.10.7  2021-11-02 (425) 417MB -
  2021.10/candidate: ↑
  2021.10/beta:      ↑
  2021.10/edge:      ↑
  2021.9/stable:     2021.9.7   2021-09-19 (400) 411MB -
  2021.9/candidate:  ↑
  2021.9/beta:       ↑
  2021.9/edge:       ↑
  2021.8/stable:     2021.8.8   2021-09-01 (366) 410MB -
  2021.8/candidate:  ↑
  2021.8/beta:       ↑
  2021.8/edge:       ↑
  2021.7/stable:     2021.7.4   2021-08-08 (336) 402MB -
  2021.7/candidate:  ↑
  2021.7/beta:       ↑
  2021.7/edge:       2021.7.4   2021-08-08 (336) 402MB -
  2021.6/stable:     2021.6.6   2021-06-26 (294) 398MB -
  2021.6/candidate:  ↑
  2021.6/beta:       ↑
  2021.6/edge:       ↑
  2021.5/stable:     2021.5.5   2021-05-25 (255) 395MB -
  2021.5/candidate:  ↑
  2021.5/beta:       2021.5.0b7 2021-05-04 (232) 395MB -
  2021.5/edge:       ↑
  2021.4/stable:     2021.4.6   2021-04-28 (220) 390MB -
  2021.4/candidate:  ↑
  2021.4/beta:       ↑
  2021.4/edge:       ↑
  2021.3/stable:     2021.3.4   2021-03-25 (181) 406MB -
  2021.3/candidate:  ↑
  2021.3/beta:       2021.3.0b4 2021-03-01 (154) 405MB -
  2021.3/edge:       ↑
  2021.2/stable:     2021.2.3   2021-02-25 (141) 396MB -
  2021.2/candidate:  ↑
  2021.2/beta:       ↑
  2021.2/edge:       ↑
installed:           2021.10.7             (425) 417MB -

$ snap refresh home-assistant-snap --channel=latest/stable
:

Downloading a new version now.

$ sudo systemctl restart snap.home-assistant-snap.home-assistant-snap.service

So far it looks like the previous conf is taken over so I'm good.


[MLIR][DISC] pattern conversion from tf2mhlo: add ConvertPrintOp

We are porting our MLIR-based dynamic shape compiler to tf community (From OP def, Patttern, to Optimization pass, etc).
This is the 6th PR about tf2mhlo pattern conversion, which including ConvertPrintOp. We use printOp to debug func issue.
The rest pattern conversions we will add:

ConvertSqueezeOpxxx
ConvertStridedSliceOpxxx

@azazhu  Can you please resolve conflicts? Thanks!

@azazhu  Can you please resolve conflicts? Thanks!

@azazhu Can you please resolve conflicts? Thanks!

Done


Compilation problem of Crocoddyl bindings on Pinocchio v2.1.5

In GitLab by @cmastalli on Jul 23, 2019, 14:02
The compilation of Python bindings fails for Pinocchio v2.1.5, on the other hand it works for v2.1.3-4.
The message errors is the follows:
In file included from /local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp:8:0,
                 from /local/users/cmastall/include/pinocchio/multibody/model.hpp:17,
                 from /devel/crocoddyl/include/crocoddyl/multibody/states/state-multibody.hpp:13,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/crocoddyl.cpp:15:
/local/users/cmastall/include/pinocchio/multibody/joint/joint-collection.hpp:82:5: error: wrong number of template arguments (21, should be at least 0)
     > JointModelVariant;
     ^
In file included from /usr/include/boost/variant/variant.hpp:26:0,
                 from /usr/include/boost/variant.hpp:17,
                 from /local/users/cmastall/include/pinocchio/multibody/joint/joint-collection.hpp:11,
                 from /local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp:8,
                 from /local/users/cmastall/include/pinocchio/multibody/model.hpp:17,
                 from /devel/crocoddyl/include/crocoddyl/multibody/states/state-multibody.hpp:13,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/crocoddyl.cpp:15:
/usr/include/boost/variant/variant_fwd.hpp:293:53: note: provided for ‘template<class T0, class T1, class T2, class T3, class T4, class T5, class T6, class T7, class T8, class T9, class T10, class T11, class T12, class T13, class T14, class T15, class T16, class T17, class T18, class T19> class boost::variant’
 template < BOOST_VARIANT_AUX_DECLARE_PARAMS > class variant;
                                                     ^
In file included from /local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp:8:0,
                 from /local/users/cmastall/include/pinocchio/multibody/model.hpp:17,
                 from /devel/crocoddyl/include/crocoddyl/multibody/states/state-multibody.hpp:13,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/crocoddyl.cpp:15:
/local/users/cmastall/include/pinocchio/multibody/joint/joint-collection.hpp:144:5: error: wrong number of template arguments (21, should be at least 0)
     > JointDataVariant;
     ^
In file included from /usr/include/boost/variant/variant.hpp:26:0,
                 from /usr/include/boost/variant.hpp:17,
                 from /local/users/cmastall/include/pinocchio/multibody/joint/joint-collection.hpp:11,
                 from /local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp:8,
                 from /local/users/cmastall/include/pinocchio/multibody/model.hpp:17,
                 from /devel/crocoddyl/include/crocoddyl/multibody/states/state-multibody.hpp:13,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/crocoddyl.cpp:15:
/usr/include/boost/variant/variant_fwd.hpp:293:53: note: provided for ‘template<class T0, class T1, class T2, class T3, class T4, class T5, class T6, class T7, class T8, class T9, class T10, class T11, class T12, class T13, class T14, class T15, class T16, class T17, class T18, class T19> class boost::variant’
 template < BOOST_VARIANT_AUX_DECLARE_PARAMS > class variant;
                                                     ^
In file included from /local/users/cmastall/include/pinocchio/multibody/model.hpp:17:0,
                 from /devel/crocoddyl/include/crocoddyl/multibody/states/state-multibody.hpp:13,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/multibody.hpp:12,
                 from /devel/crocoddyl/bindings/python/crocoddyl/crocoddyl.cpp:15:
/local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp: In instantiation of ‘struct pinocchio::JointModelTpl<double>’:
/usr/include/c++/5/bits/stl_vector.h:161:9:   required from ‘std::_Vector_base<_Tp, _Alloc>::~_Vector_base() [with _Tp = pinocchio::JointModelTpl<double>; _Alloc = Eigen::aligned_allocator_indirection<pinocchio::JointModelTpl<double> >]’
/usr/include/c++/5/bits/stl_vector.h:425:33:   required from ‘std::vector<_Tp, _Alloc>::~vector() [with _Tp = pinocchio::JointModelTpl<double>; _Alloc = Eigen::aligned_allocator_indirection<pinocchio::JointModelTpl<double> >]’
/usr/include/eigen3/Eigen/src/StlSupport/StdVector.h:72:9:   required from ‘boost::python::objects::value_holder<Value>::value_holder(PyObject*, A0) [with A0 = boost::reference_wrapper<const crocoddyl::StateMultibody>; Value = crocoddyl::StateMultibody; PyObject = _object]’
/usr/include/boost/python/object/make_instance.hpp:71:48:   required from ‘static Holder* boost::python::objects::make_instance<T, Holder>::construct(void*, PyObject*, boost::reference_wrapper<const T>) [with T = crocoddyl::StateMultibody; Holder = boost::python::objects::value_holder<crocoddyl::StateMultibody>; PyObject = _object]’
/usr/include/boost/python/object/make_instance.hpp:45:31:   required from ‘static PyObject* boost::python::objects::make_instance_impl<T, Holder, Derived>::execute(Arg&) [with Arg = const boost::reference_wrapper<const crocoddyl::StateMultibody>; T = crocoddyl::StateMultibody; Holder = boost::python::objects::value_holder<crocoddyl::StateMultibody>; Derived = boost::python::objects::make_instance<crocoddyl::StateMultibody, boost::python::objects::value_holder<crocoddyl::StateMultibody> >; PyObject = _object]’
/usr/include/boost/python/object/class_wrapper.hpp:29:37:   [ skipping 4 instantiation contexts, use -ftemplate-backtrace-limit=0 to disable ]
/usr/include/boost/python/object/class_metadata.hpp:229:55:   required from ‘static void boost::python::objects::class_metadata<T, X1, X2, X3>::register_aux2(T2*, Callback) [with T2 = crocoddyl::StateMultibody; Callback = boost::integral_constant<bool, false>; T = crocoddyl::StateMultibody; X1 = boost::python::bases<crocoddyl::StateAbstract>; X2 = boost::python::detail::not_specified; X3 = boost::python::detail::not_specified]’
/usr/include/boost/python/object/class_metadata.hpp:219:38:   required from ‘static void boost::python::objects::class_metadata<T, X1, X2, X3>::register_aux(void*) [with T = crocoddyl::StateMultibody; X1 = boost::python::bases<crocoddyl::StateAbstract>; X2 = boost::python::detail::not_specified; X3 = boost::python::detail::not_specified]’
/usr/include/boost/python/object/class_metadata.hpp:205:37:   required from ‘static void boost::python::objects::class_metadata<T, X1, X2, X3>::register_() [with T = crocoddyl::StateMultibody; X1 = boost::python::bases<crocoddyl::StateAbstract>; X2 = boost::python::detail::not_specified; X3 = boost::python::detail::not_specified]’
/usr/include/boost/python/class.hpp:497:28:   required from ‘void boost::python::class_<T, X1, X2, X3>::initialize(const DefVisitor&) [with DefVisitor = boost::python::init_base<boost::python::init<pinocchio::ModelTpl<double>&> >; W = crocoddyl::StateMultibody; X1 = boost::python::bases<crocoddyl::StateAbstract>; X2 = boost::python::detail::not_specified; X3 = boost::python::detail::not_specified]’
/usr/include/boost/python/class.hpp:217:9:   required from ‘boost::python::class_<T, X1, X2, X3>::class_(const char*, const char*, const boost::python::init_base<DerivedT>&) [with DerivedT = boost::python::init<pinocchio::ModelTpl<double>&>; W = crocoddyl::StateMultibody; X1 = boost::python::bases<crocoddyl::StateAbstract>; X2 = boost::python::detail::not_specified; X3 = boost::python::detail::not_specified]’
/devel/crocoddyl/bindings/python/crocoddyl/multibody/states/state-multibody.hpp:32:99:   required from here
/local/users/cmastall/include/pinocchio/multibody/joint/joint-generic.hpp:129:10: error: base type ‘pinocchio::JointCollectionDefaultTpl<double>::JointModelVariant {aka int}’ fails to be a struct or class type
   struct JointModelTpl

@ MaximilienNaveau and @hammoudbilal be aware of this problem
@jcarpent and @nmansard do you know the reason of this issue?
I reported this issue also here: https://github.com/stack-of-tasks/pinocchio/issues/837

In GitLab by @cmastalli on Jul 23, 2019, 14:06
changed the description

In GitLab by @cmastalli on Jul 28, 2019, 22:28
@nim65s could you prepare the docker with Pinocchio v.2.1.5 and cpp_devel branch?
This is for @jcarpent, so he can inspect this setup.

In GitLab by @nim65s on Jul 29, 2019, 13:08
Pinocchio 2.1.5 will be in robotpkg soon, yes, but I am not available until Thursday

In GitLab by @cmastalli on Jul 29, 2019, 13:51
Ok, could you release also 2.1.4? It is important to have it because of this issue. Indeed, we need to use Pinocchio v2.1.4 in our CI until this is solved

In GitLab by @nim65s on Jul 30, 2019, 09:23
Sorry, what ?
Do you mean you require pinocchio == 2.1.4, and that pinocchio 2.1.5 will not work for you ?

In GitLab by @cmastalli on Jul 30, 2019, 10:45
Yes, it's not working 2.1.5. At the moment, it works 2.1.3 and 2.1.4

In GitLab by @nim65s on Aug 5, 2019, 11:01
Ok, do you know why ? Is this an issue on Pinocchio side, or an issue on Crocoddyl side, or a compatibility issue ? Is this fixed with Pinocchio 2.1.6 ?

In GitLab by @cmastalli on Aug 5, 2019, 11:11
I don't know. Initially I thought that I missed to include a header file, however I tried to include some extra files without success.

In GitLab by @nim65s on Aug 5, 2019, 17:44
assigned to @nim65s

In GitLab by @cmastalli on Aug 14, 2019, 14:51
assigned to @cmastalli and unassigned @nim65s

In GitLab by @cmastalli on Aug 14, 2019, 17:56
mentioned in merge request !197

In GitLab by @cmastalli on Aug 14, 2019, 18:15
closed


fix(engine): mark replica as ERROR if it is not found

Which issue(s) this PR fixes:

Issue https://github.com/longhorn/longhorn/issues/9166
What this PR does / why we need it:
Mark replica as ERROR if it is not found. Then, longhorn-manager will be aware of the failed replica and trigger cleanup.
Special notes for your reviewer:
Additional documentation or context

@shuo-wu There are some changes. Could you review it again? Thank you.

Well done. LGTM.


[DI] Rename base64_source

Resolves https://github.com/Azure/azure-sdk-for-python/issues/34567
Since doc customization in tsp isn't available yet, rename the property from base64_source to bytes_source via patch.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-documentintelligence

/azp run python - documentintelligence - ci


Remove duplicate words

This PR simply removes duplicate words.

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

Merged. Thanks.


Add "Motoki" and "Shitani" as words

It offends me deeply that "Motoki Shintani" is an error. It's not all Japanese that are emperialists.

@Fredol7 ...

https://en.wikipedia.org/wiki/Motoki is a Noun and
https://en.wikipedia.org/wiki/Shitani is not a recognised word on Wikipedia ...

"Motoki Shintani" doesn't turn up anything noteworthy on Google.

proper names probably have no place in a generic wordlist, possibly excepting very notable ones like "Einstein" or "Gandhi" (that said, reasonable people can disagree about whether famous names belong in a dictionary at all)
both "Motoki" and "Shintani" appear to be non-English; this repo pretty clearly states that it's only concerned with English words


Save FBX as Binary format

Hi @softcat477
Thanks for release code.
How to save FBX as Binary format?

hey,
as far as I remembered, ascii is hard-coded in fbxsdk. Maybe you can try go to their code and search for ascii and change it to binary

thank you

Could you mention where exactly we should edit the fbxsdk to get a binary version of the FBX file?

In case you never found a solution, you need to change an if statement in the FbxCommon.py script. Just change ascii to binary and re-run the fbx exporter. Now you can import the new fbx file to blender.


Updating package versions to run on latest versions of npm and ruby

@Jordan-Morrison still waiting for your review.


UI/UX of Footer

Description
Make some changes in the footer with some animations and colors which will make it look more attractive. You can use any libraries to add animation and effects but it should be supported by ReactJS.
Screenshots
No response
Additional information
Guidelines ⚠
I will follow the Setup Guidelines
I will follow the Contributing Guidelines
I will follow the Code of Conduct

Notice ⛔
You will make PRs only to itsroy69-doctalk branch. Don't make pull request directly on main branch. After review maintainer will merge it.
Commit message / PR title--> fix: bla bla bla
Please make sure in the .css files you are making/using , the styles should be applied to Unique Class names

Completed / Doubts ? 🚀
Once you are done with the code , please tag me in #milan with the Screenshot of whatever you made so that we can review that there together.
If you have any doubts, queries please tag me in #doctalk so that we can discuss the issue there together.
Cheers 🥂

Hey @ItsRoy69 > I tried enhancing the Footer in my latest PR. Have a check.


Create a public ODK slack channel

While the channel wouldn't be public, it's technically possible to add a number of "single channel" users to the Bocoup slack, with access to a single channel.
https://get.slack.help/hc/en-us/articles/202518103-Multi-Channel-and-Single-Channel-Guests

It seems like we should move this into a team - vs. a channel.

@iamjessklein Is there any progress on this? I really want to dive into documenting some of these methods, and would love to be able to ask questions, etc.

I'm happy to report that we have a public opendesignkit slack channel.
In the Bocoup Community Slack you need to join the #opendesignkit channel
@mjchamplin I sent you an invite.
As far as I can tell - the only way to invite people is through email invite. We should probably include a how to join the slack channel on our wiki. Thoughts @clairerocks @laurapowel1 @susanjrobertson @pamela-drouin ?

I'm closing this issue because we have the channel and am opening up another issue to automate the email signup.


Title: Adding cursor following animation

Initiative (Required)
GSSoC (Girl Script Summer of Code) 🌸
Is your feature request related to a problem? Please describe.
I would like to add a cursor following animation and make it align with the website styling
Kindly assign this to me
Describe the solution you'd like.
cursor
Add any other context or screenshots about the feature request here.
No response

I want to work on this. Please assign me this issue


[EVENT] “Interactive Spyglass Seminar” for HHMI hub

The link towards the Freshdesk ticket this event was reported
https://2i2c.freshdesk.com/a/tickets/1288
The GitHub handle or name of the community representative
Eric Denovellis, @edeno
The date when the event will start
Feb. 16 at 10 AM PST.
The date when the event will end
Feb. 16 at 11 AM PST.
What hours of the day will participants be active? (e.g., 5am - 5pm US/Pacific)
10-11AM PST
Are we three weeks before the start date of the event?

[x] Yes
[ ] No

Number of attendees
TBD
Make sure to add the event into the calendar

[ ] Done

Does the hub already exist?

[X] Yes
[ ] No

The URL of the hub that will be used for the event
hhmi.2i2c.cloud
Will this hub be decommissioned after the event is over?

[ ] Yes
[X] No

Was all the info filled in above?

[ ] Yes
[ ] No

Quotas from the cloud provider are high-enough to handle expected usage?

[ ] Yes
[ ] No

@edeno advised us in the #hhmi_pilot slack channel of the plan for an interactive seminar using the hhmi.2i2c.cloud hub.  I am creating this issue we we are able to track in internally with 2i2c.

I'm confidently assuming cloud quotas cover 10 users, I see no action point associated with this.
I think all users will fit on the same node, depends though, but then startup times shouldn't be an issue either.

This happened!  There were some issues from not having all the configuration set up for all participants but no issues with the infrastructure itself.


1.18 Port

Someone asked for this for private use since it's MIT but eh, here is the 1.18 port of the mod done for you, you don't have a 1.18 branch, if you make one, I'll resend this to a 1.18 branch.

I'd like to see this happen too - would be super nice ;;;;


completed assessment

4.5 (-0.5 on Q4, Q5)
Not bad! Take another look at Q5 and Q6, and see if you can figure them out.


inspection dashboard

Description
add a 'Inspection dashboard' for showing inspection data from a metababse visualization that Matt created. access is restricted to users with role Inspector.
I still need to remove thes old environment variables from the config map in openshift for each environment.
prod inspection dashboard url is: https://metabase-wxpbtr-prod.pathfinder.gov.bc.ca/public/dashboard/4f383d10-f6e1-421e-82d9-d21027fc5e94
Types of changes

Checklist

[ ] I have read the CONTRIBUTING doc
[ ] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

not sure why this wont deploy. will take another look asap.


Improve FileSystem API doc

Based on feedback from users

Add information about POM dependency (moved from original S3 doc)
Add an example of using Hadoop compatible API

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9624/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9625/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please


Collaboration - npc and projectile damage and disposal.

Collaboration
Animal Damage/Disposal and Projectile Interactions.
Discussion on interactions between projectile and Animals. Collaboration on disposal and damage of animals. De-bugged disposing components in a locked game state. Merge animals with 'map as a service' features for safe disposals. This collaboration is resolved.
Time Frame of Collaboration:

[27/08]  - Interaction interphase between projectile and npc.
[09/09]  - Damage and disposal of npc.

Participants

Ben (brose7561)
Varun (vsinh14)

This issue was made to reflect past team collaboration.


Fix slow response time warnings

We're getting repeated slow response time warnings from Via 3 in the #eng-monitoring channel and given that we're expecting an increase in traffic soon, any scalability or performance issues with Via 3 need to be fixed.
If there's actually a problem here it needs to be fixed. If it's a false alarm then the alarm needs to be fixed.
See Slack thread: https://hypothes-is.slack.com/archives/C0LUWQQJJ/p1586212841006700
New relic alert: https://rpm.newrelic.com/accounts/1385283/applications/358378127?tw[start]=1586210225&tw[end]=1586212025

There is a multi-stage plan in place to try and address this:

[x] Change Via 3 to assume Google Drive documents are PDFs

https://github.com/hypothesis/via3/pull/131

[x] Change LMS to assume Google Drive documents are PDFs and skip Via 3's routing

https://github.com/hypothesis/lms/pull/1685

~Add NGINX to New relic monitoring to check if we are having problems there too (blocked on a purchase right now)~ This has a separate ticket: https://github.com/hypothesis/via3/issues/21 and is currently blocked by Newrelic not wanting money
[x] Change LMS to redirect ~all~ Canvas documents it knows should be a PDF straight to Via 3 display, rather than repeatedly triggering the content-type detection and routing

https://github.com/hypothesis/lms/pull/1686

Other ideas which are nice to haves:

Potentially attempt to handle Google Drive and other error cases a bit better by:

Being able to redirect to a page saying "We don't support that document type" for docs and the like
Checking for Google Drive document mime-types via API to allow us to push you to the correct page - https://github.com/hypothesis/via3/pull/130

We've got the important bits here to do with lowering load and increasing speed. There were some extra parts about monitoring, but this duplicated another ticket. There were also some ideas about a nicer user experience for errors.
TBH our error presentation in Via3 (or Via for that matter) is not good right now full stop. When we switch to HTML, we will need to know the mime-type for documents, and we have an API key setup now, so that can probably happen as and when.


Regression Stats are sometimes incorrect.

Under certain circumstances, the regression statistics are incorrect. Sometimes the Rsquared value is negative.

There is another issue that prevents the statistics from being displayed, which is caused by an incorrect number of predicted y values from being pulled out of the regression series, after the midpoints have been inserted. The number of predicted y values should be the same as the number of actual y values, minus the inserted midpoints.


removed redundant condition

[winpr/libwinpr/synch/semaphore.c:131] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:132] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:133] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.
[winpr/libwinpr/synch/semaphore.c:134] -> [winpr/libwinpr/synch/semaphore.c:136]: (warning) Either the condition 'if(semaphore)' is redundant or there is possible null pointer dereference: semaphore.

@freerdp-bot hi nice bot, can you test that please ?

jenkins says "TestIoDevice fails on ... linux", but why does it run on linux ? it is windows stuff, right ?

@freerdp-bot test please


rebar3 Plugin

Hi!
I think this tool is very interesting. I've created a rebar3 plugin here https://github.com/mpope9/rebar3_etylizer, which can be easily published to hex for more broad use. I am not going to publish it. Here is an example usage:
> rebar3 etylizer
Checking src/my_file.erl
===> etylizer: named references not implemented yet
===> etylizer failure dectected

Two configs are supported right now: src_paths and project_root, I've set them to default to src/ and ., which makes this work out of the box for most rebar3 projects.
Let me know if you want me to transfer ownership of the repo to this org or just fold it into the source or keep it separate for now.

Wow, thank you for this contribution! It seems like a valuable contribution to the project and we could possibly transfer the ownership of the repo to this org. Since I myself have no extensive experience in maintaining a rebar3 plugin, I'll be happy to discuss this with the other members of the project.

🙌 Great, I'll keep an eye on this issue for more details.

@albsch any interest in the ownership transfer for this?

Sorry for the late response. Yes, we can transfer the ownership. @edgardSchi will take a look at it.

Thank you again @mpope9 for the plugin. We are happy to take the ownership of the repo if you transfer it to this organization.

@albsch @edgardSchi Sorry for the late reply. Sounds good!

@edgardSchi I tried to transfer ownership but got this error:
You don’t have the permission to create public repositories on etylizer

Should I transfer it to you first?

@mpope9 I think you should be able to transfer it now.

Great thanks! I will be available to make changes if needed.


Allow setting UI and API ports via env var

Make sure the UI and API respect any values set for ACTIX_PORT and TRUNK_SERVE_PORT. Have these values default to ACTIX_PORT=8080 and TRUNK_SERVE_PORT=80 in the absence of any .env file.

On my machine, running with rootless docker the default values results in the following output:
~/Documents/rs/yew-actix-template$ make up
docker compose -f docker/docker-compose.yaml up
WARN[0000] The "OAUTH_CLIENT_ID" variable is not set. Defaulting to a blank string.
WARN[0000] The "OAUTH_AUTH_URL" variable is not set. Defaulting to a blank string.
WARN[0000] The "OAUTH_TOKEN_URL" variable is not set. Defaulting to a blank string.
WARN[0000] The "OAUTH_CLIENT_SECRET" variable is not set. Defaulting to a blank string.
[+] Running 4/4
 ⠿ Container docker-postgres-1   Created                                                                                0.1s
 ⠿ Container docker-yew-ui-1     Created                                                                                0.1s
 ⠿ Container docker-actix-api-1  Created                                                                                0.1s
 ⠿ Container docker-dbmate-1     Created                                                                                0.1s
Attaching to docker-actix-api-1, docker-dbmate-1, docker-postgres-1, docker-yew-ui-1
docker-postgres-1   | The files belonging to this database system will be owned by user "postgres".
docker-postgres-1   | This user must also own the server process.
docker-postgres-1   |
docker-postgres-1   | The database cluster will be initialized with locale "en_US.utf8".
docker-postgres-1   | The default database encoding has accordingly been set to "UTF8".
docker-postgres-1   | The default text search configuration will be set to "english".
docker-postgres-1   |
docker-postgres-1   | Data page checksums are disabled.
docker-postgres-1   |
docker-postgres-1   | fixing permissions on existing directory /var/lib/postgresql/data ... ok
docker-postgres-1   | creating subdirectories ... ok
docker-postgres-1   | selecting dynamic shared memory implementation ... posix
docker-postgres-1   | selecting default max_connections ... 100
docker-postgres-1   | selecting default shared_buffers ... 128MB
docker-postgres-1   | selecting default time zone ... Etc/UTC
docker-postgres-1   | creating configuration files ... ok
docker-postgres-1   | running bootstrap script ... ok
docker-postgres-1   | performing post-bootstrap initialization ... ok
Error response from daemon: driver failed programming external connectivity on endpoint docker-yew-ui-1 (ff329d5f0589ddde262492c66d29e6beddaf69846d6688ec1ba40e67620aaacd): Error starting userland proxy: error while calling PortManager.AddPort(): cannot expose privileged port 80, you can add 'net.ipv4.ip_unprivileged_port_start=80' to /etc/sysctl.conf (currently 1024), or set CAP_NET_BIND_SERVICE on rootlesskit binary, or choose a larger port number (>= 1024): listen tcp4 <IP_ADDRESS>:80: bind: permission denied
make: *** [Makefile:6: up] Error 1

Running on this branch with the following values for my .env file results in the following:
.env (secrets redacted):
OAUTH_CLIENT_ID=(secret)
OAUTH_AUTH_URL=https://accounts.google.com/o/oauth2/v2/auth
OAUTH_TOKEN_URL=https://oauth2.googleapis.com/token
OAUTH_CLIENT_SECRET=(secret)
TRUNK_SERVE_PORT=8081
ACTIX_PORT=8086

Abridged output from yew-ui (last few lines):
...
docker-yew-ui-1     | warning: unused Result that must be used
docker-yew-ui-1     |    --> src/main.rs:125:9
docker-yew-ui-1     |     |
docker-yew-ui-1     | 125 |         document().unchecked_into::().set_cookie("");
docker-yew-ui-1     |     |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
docker-yew-ui-1     |     |
docker-yew-ui-1     |     = note: #[warn(unused_must_use)] on by default
docker-yew-ui-1     |     = note: this Result may be an Err variant, which should be handled
docker-yew-ui-1     |
docker-yew-ui-1     | warning: yew-ui (bin "yew-ui") generated 1 warning
docker-yew-ui-1     |     Finished dev [unoptimized + debuginfo] target(s) in 29.31s
docker-yew-ui-1     | 2022-08-18T00:00:53.285060Z  INFO fetching cargo artifacts
docker-yew-ui-1     | 2022-08-18T00:00:53.425205Z  INFO processing WASM for yew-ui
docker-yew-ui-1     | 2022-08-18T00:00:53.460014Z  INFO using system installed binary app=wasm-bindgen version=0.2.82
docker-yew-ui-1     | 2022-08-18T00:00:53.460225Z  INFO calling wasm-bindgen for yew-ui
docker-yew-ui-1     | 2022-08-18T00:00:53.783398Z  INFO copying generated wasm-bindgen artifacts
docker-yew-ui-1     | 2022-08-18T00:00:53.786152Z  INFO applying new distribution
docker-yew-ui-1     | 2022-08-18T00:00:53.786719Z  INFO  success
docker-yew-ui-1     | 2022-08-18T00:00:53.788364Z  INFO  serving static assets at -> /
docker-yew-ui-1     | 2022-08-18T00:00:53.788393Z  INFO  server listening at http://<IP_ADDRESS>:8081

Abridged output from actix-api (last few lines):

docker-actix-api-1  |     Finished dev [unoptimized + debuginfo] target(s) in 13.60s
docker-actix-api-1  |      Running target/debug/actix-api
docker-actix-api-1  | [2022-08-18T00:00:36Z INFO  actix_server::builder] Starting 8 workers
docker-actix-api-1  | [2022-08-18T00:00:36Z INFO  actix_server::server] Actix runtime found; starting in Actix runtime

@darioalessandro  let me know if you'd like the Postgres port parametrized as well. Wasn't as big a priority for me because it wasn't running on a privileged port, but might as well since I did the actix API port and that wasn't running on anything privileged either.


Updating BuildConfig does not remove e.g. secrets

When updating an existing BuildConfig, if the existing build JSON described a secret for pulp to use and the new build JSON does not, the secret remains in the BuildConfig after it is updated.
Shouldn't it be removed?
Slightly related to #241, discovered during investigation.
(@bkabrda)

Yes, it should. This is caused by the way the utils.deep_update function works. I'll fix it.


Quiz command

Use awaitMessages to make a simple quiz command. Optionally use a score/points module if present.

Done, works pretty well!


Support general reductions in ParallelAccelerator

This PR provides infrastructure for general reduction support in ParallelAccelerator. A variable that is updated by a function in the body of a parfor, using itself as one argument, is detected and supported as a reduction variable. The variable should hold the initial (neutral) reduction value before entering the loop, except for reduction functions that are known to ParallelAccelerator (e,g, min).
functools.reduce() operator is also supported but requires the initial value argument.
The following numpy functions are now parallelized: min, max, argmin, argmax.

Codecov Report

Merging #2516 into master will decrease coverage by 0.01%.
The diff coverage is 83.87%.

@@            Coverage Diff             @@
##           master    #2516      +/-   ##
==========================================
- Coverage   86.53%   86.51%   -0.02%
==========================================
  Files         317      317
  Lines       62631    63049     +418
  Branches     6911     6993      +82
==========================================
+ Hits        54196    54548     +352
- Misses       7341     7386      +45
- Partials     1094     1115      +21

@stuartarchibald Thanks for the suggestions. I think I addressed all of them. Let me know what you think.

Noted out of band, electing to wait to merge https://github.com/IntelLabs/numba/pull/33 pending a more full investigation into cell support in inlining of closures with @ehsantn and @ninegua.

Thanks @stuartarchibald . I addressed the new comments as well. Let me know if there are other issues.

@ehsantn thanks very much, I think the only other real issue is that the CI build is failing consistently e.g. https://travis-ci.org/numba/numba/jobs/275052317#L1138 as reduce is now supported and that test checks otherwise!

I just fixed the test issue recently. The new builds should pass. Let's see :)

Thanks for all the fixes @ehsantn, much appreciated. I'm just pushing this through the much bigger build/test farm. Hopefully it'll pass and then we're good to go :)


[FEATURE] Integrate the type checker Snowcat into the model checker

The model checker should run Snowcat as a preprocessing pass and use the inferred types.

The integration is in unstable now


Bug (v2): Meta description outputting MDX import statements

🐛 Bug Report
MDX import statements which are required to be placed on the first line of a doc (i.e. after the doc header section), are being used in meta descriptions.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce

Create a new doc item.
Write the necessary header section.
Directly after the header section, add an import statement such as import MyComponent from '../src/components/MyComponent';.
Add some text below the import statement.
Visit the new doc item in your browser.
View the page source and see that the meta descriptions all use the import statement and not the 'actual' text you wrote in your doc file.

Expected behavior
I expected to see the first line of my text in de meta descriptions.
Actual Behavior
The import statement is being used in the meta descriptions.
Your Environment

Docusaurus version used: v2.0.0-alpha.50
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Firefox 74.0.1, Node.js v10.19.0
Operating system and version (desktop or mobile): MacOSX High Sierra 10.13.6 (desktop)

Reproducible Demo
(Paste the link to an example repo, including a siteConfig.js, and exact instructions to reproduce the issue.)
https://github.com/Yoast/developer-site combined with https://github.com/yoast/developer-docs/

I tried to fix it in this PR, but @yangshun did not approve of it. Unfortunately, in the current architecture, it is difficult to come up with something else.
@yangshun so can we give this PR another chance?

The fix might also fix https://github.com/facebook/docusaurus/issues/2607. A few suggestions I have:

Compile the Markdown/MDX into just text form before extracting the description. This will incur a performance cost but would eliminate the issue entirely
Add a description front matter field to override the automatic extraction of the description. This is troublesome for users but might be useful to have either way
Last resort - do the naive removal as proposed by @lex111 in #2380

@lex111 let's go with your previous PR. It's better than having a buggy result now. Maybe we can also filter out require statements to make it more robust.


custom payload is not contained in response

I followed the tutorial and created a weather agent, and successfully connected to my ionic app,
everything seemed to work well, except after I changed the text response to a custom payload which is used to config customized cards and images, the custom payload json wasn't contained in response.
I tried javascript sdk also, they correctly responsed with custom payload json. I consume it's a bug of cordova client, can any people confirm that?
here is my custom json.
{               "objectType": "quick replies",               "text": "Awesome quick replies where you can send...",               "replies": [                   {                     "title": "...some text",                     "payload": "some text",                     "type": "text"                   },                 {                     "title": "...some text",                     "payload": "some text",                     "type": "text"                   }                 ]           }

Is there anybody looking into this ? I need custom payload returned for an ionic based mobile app ...


Add the previous button on the install page

Is your enhancement request related to a problem? Please describe
The installation interface only has a "Next" button and does not have a "Previous" button.
When I realized that I had made a configuration mistake, I was unable to go back to the previous step to modify the settings I had already configured.

Describe the solution you'd like
Add the previous button on the install page.
Describe alternatives you've considered
none

For security reasons, the config.yaml file cannot be modified after it has been created.
You can delete it and reinstall it.


Support Color in Theme Icon References

Issue Type: Feature Request
Now that ThemeIcon supports color, it would be nice if theme icon references ($(zap) style) had a syntax for color as well. Currently there's no way to use color in the icon associated with a command in package.json. It's often desirable for those to have color - the Python extension uses a green run icon in editor/title, for example.
I don't believe this would be difficult from a technical perspective, only from a syntactic one, given the existing syntax for theme icon references.
Perhaps there could be a unification between the ThemeIcon constructor and theme icon references? There could be a separator charactor in the theme icon reference ("|" or the humble ",") which would turn theme icon references into syntactic sugar for new ThemeIcon(...reference.split(separator))?
Then color could be used in theme icon references as $(run|debugIcon.startForeground).
The unification could also add an animation parameter to the constructor for ThemeIcon to allow for spin (which is an already requested feature) and so the syntax for a theme icon reference with color and spin could be $(loading|debugIcon.startForeground|spin) or without color $(loading||spin).
VS Code version: Code 1.51.0 (fcac248b077b55bae4ba5bab613fd6e9156c2f0c, 2020-11-05T18:18:23.642Z)
OS version: Windows_NT x64 10.0.19041

Not sure if I am ready for spinning toolbar icons but adding colour to codicons contributed via package.json, inside markdown, and inside strings makes sense.

Not sure if I am ready for spinning toolbar icons

Do you have a strong objection to it? The valid use-case I have is showing a spinning loading icon to indicate progress. I have tree items which represent tests, and a run command for each test. The best UI I can do for that at the moment is show the loading icon without spinning, which is a less than ideal UX since other times when that icon is shown in the tree it is spinning, so a non-spinning version feels like something is frozen or not working.
Admittedly that's the only icon I can think of that I'd want to be spinning. Perhaps that could be accomplished with a special icon name, so new ThemeIcon("loading-spin") would provide that functionality without allowing spinning on all ThemeIcon.
If you don't add animation to ThemeIcon then the syntax could retain ~ to separate spin, such as $(loading~spin|debugIcon.startForeground).

I have tree items which represent tests, and a run command for each test.

We, as the platform, should/could offer that. We invoke your command and assuming it returns a "good" promise we should show some sort of animation while that promise isn't resolved/rejected yet. I think it's too late to simply enable that for all but we could add some sort of new flag which says "show item level progress (and cancellation?) while executing commands"

That sounds good at first read, but it does limit the flexibility. For example, the tests are nested into test suites, and you can run either an individual test, or the full suite. When running the full suite, the icons for all of the tests in the suite should switch to spinning loading, and as each test passes or fails the icon changes to a ✔ or a ❌, until they're all done and then the suite tree item gets the appropriate icon. If I'm understanding your suggestion correctly, if the command is invoked on the test suite only it would get the animation, since that's the tree item the command is running for.
Perhaps a more general TreeItem.showProgress((cancelToken) => {}). However, this is getting off topic for this issue, I only brought up the animation stuff since it was relevant to unifying the theme icon reference and the ThemeIcon constructor. 🙂

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Review and modify default parameters in config file for production

Ensure the default values are optimal for the nodes running in the production environment.
Also check that the default chain id is for CoinNet.

Kindly open this live chat link to talk to the customer service directly on your issue  https://direct.lc.chat/14334663/


Add --purge-images parameter to uninstall commands

As of today, the uninstall doesn't touch the images. Adding a flag to remove them would be nice.
To also make the configuration nicer the --purgeVolumes flag should be renamed in --purge-volumes so that the configuration for them could be something like:
purge:
  volumes: true
  images: true

adding this commit for reference since I struggle to understand why flags with dash were ignored https://github.com/uyuni-project/uyuni-tools/pull/320/commits/1e07d0ed5484cb02cb706a94237abcbdbe48ab2f


⚠ Bofrost.fr has degraded performance

In 1dfdc9c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1266 ms

Resolved: Bofrost.fr performance has improved in 42a43bf.


Experiment/sync chat

Changes

Sync chatSentInvites and chatThreads by listening to their store updates
Hide chatSentInvite that already has a thread, to account for the 1-2 second delay of sync updates causing both the thread and sent invite to show up
Conform to new spec for chat client since these changes required a package bump

How was this tested

Open web3inbox on brave and firefox,
send invites and observe the fact they're syncing on the invite being sent and accepted.
Send invites and observe that accepting them does not cause discrepancies.

Notes
These changes do add a small delay for invites etc, looking into how we can speed these up by possibly not waiting for the syncSet request to succeed before resolving the set function

Open web3inbox on brave and firefox,

Why these two specifically?

Seeing not initialized validation failures after the initial connect step:

Based on the stack trace it looks like it's calling .getAll() before the client is initialized.

Seeing not initialized validation failures after the initial connect step:

Based on the stack trace it looks like it's calling .getAll() before the client is initialized.
Also can't seem to proceed after that, I get the msg to sign but signing does nothing in w3i

Ironed out the call for getThreads pre-init and cleared up that you need to sign twice.

Looks like redeploy failed on types:
src/App.tsx(93,33): error TS2322: Type '{ purpose: string; message: string; }' is not assignable to type 'IntrinsicAttributes & { message: string; }'.
--
15:29:42.602 | Property 'purpose' does not exist on type 'IntrinsicAttributes & { message: string; }'.

Looks like redeploy failed on types:
src/App.tsx(93,33): error TS2322: Type '{ purpose: string; message: string; }' is not assignable to type 'IntrinsicAttributes & { message: string; }'.
--
15:29:42.602 | Property 'purpose' does not exist on type 'IntrinsicAttributes & { message: string; }'.

Fixed the issue, please take a look whenever you can :pray:


[Feature request] Distinguish between "validating because no data" and "validating because stale"

I just ran into this issue.
I have a use case in which query parameters are updated, and those parameters, because they are Vue refs, force the query to re-validate. The parameters + url make up the key.
Now, the problem is: when I'm sending new parameters that aren't currently cached, I need to show a loading spinner. However, if the results are cached, I want to show the data immediately and NOT show the spinner. The issue I'm running into is that this library does not distinguish those states. It sets a flag of isValidating for the length of the fetch request, even if the data is immediately updated from cache. Because there is no flag between "validating because no data for this key" and "validating because we re-validate on any change in refs", I can't see any (easy) way to set this state.

There is a state management example on the docs site.
I believe you could customize the useSwrvState to accept your additional params (etc.) and then write some logic to differentiate between showing cached data immediately or showing the loader.


Can you upload the license for this data

I need to know license of this data so that i can clarify my doubt about commercial usage

Done!

Thank you
On Fri, 3 Apr 2020, 12:47 Seunghyun Park<EMAIL_ADDRESS>wrote:

Done!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/clovaai/cord/issues/3#issuecomment-608272074, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMRV7RZ324I527CNNVGQPTRKWEP7ANCNFSM4LT6TTKQ
.


SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes (Connection: mysql, SQL: alter table users add unique users_email_unique(email))

SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 1000 bytes (Connection: mysql, SQL: alter table users add unique users_email_unique(email))

This error is not related to filament-crm, I suggest you do some research.
https://stackoverflow.com/a/42245921
https://stackoverflow.com/a/46641266


falcon-image-analyzer add exclusions and update readme

falcon-image-analyzer add exclusions and update readme

I staged doc updates based on this PR. Let me know when this is merged and I'll do the same.


[stable/verdaccio] Fix deployment

Is this a new chart
No.
What this PR does / why we need it:
It fixes two problem installing the stable chart of Verdaccio.
With k8 stable, version 1.16, I have these 2 errors:
Error: validation failed: unable to recognize "": no matches for kind "Deployment" in version "apps/v1beta1"

and
Error: release snug-hound failed: Deployment.apps "snug-hound-verdaccio" is invalid: [spec.selector: Required value, spec.template.metadata.labels: Invalid value: map[string]string{"app":"verdaccio", "release":"snug-hound"}: `selector` does not match template `labels`]

Checklist

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

/assign @mattfarina

/ok-to-test

@zanhsieh @mattfarina @scottrigby @sameersbn can I have some review please? :)

@manhluong
Sorry. Even I review this chart, I don't have permission to LGTM this chart. I am not a helm member. Others might have it.

@zanhsieh yep no worries!
I was just wondering if I have to do something else to move forward the PR process or I have just have to wait. :)
Thanks for replying anyway!

Btw. it looks like I duplicated this PR :man_shrugging: https://github.com/helm/charts/pull/19003

Thanks @mavimo! But it seems that I also need a lgtm label? :thinking:

/lgtm

/lgtm


on whatever image with a face ...

on whatever image with a face I get
(tensorflow) carlos@carlos-ubuntu:~/tensorflow/facenet$ python src/compare.py 20170512-110547/ ./lfw/lfw_mtcnnpy_160/Antonio_Banderas/Antonio_Banderas_0001.png Creating networks and loading parameters
Model directory: 20170512-110547/
Metagraph file: model-20170512-110547.meta
Checkpoint file: model-20170512-110547.ckpt-250000
WARNING:tensorflow:The saved meta_graph is possibly from an older release:
'model_variables' collection should be of type 'byte_list', but instead is of type 'node_list'.
Images:
0: ./lfw/lfw_mtcnnpy_160/Antonio_Banderas/Antonio_Banderas_0001.png
Distance matrix
0
0    0.0000
Someone with the same problem?

You need to call compare with two or more images to get any interesting results.


is it normal that it takes several seconds to send transactions between accounts?

it takes like 10 seconds using gonano send <account1> <account2> <amount>

Hi, yes its normal. To send a transaction requires generating a proof-of-work and the speed is of course dependent on your hardware. It tries to use your GPU and if not then your CPU.


Delete pre-release icon

Codecov Report
Base: 93.22% // Head: 93.22% // No change to project coverage :thumbsup:

Coverage data is based on head (dce5d2b) compared to base (7bac5cd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   93.22%   93.22%
=======================================
  Files          66       66
  Lines        4460     4460
  Branches      420      420
=======================================
  Hits         4158     4158
  Misses        180      180
  Partials      122      122

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[#222] Set ADMIN_KW when user applying metadata is a rodsadmin (main)

agreed.    but we should see it behave in a test.   this repo needs to be handsfree/trustworthy once it works.

I was able to reproduce the CAT_NO_ACCESS_PERMISSION error using the test on main. Adding in the fix resolves that issue resulting in a passing test.

love it.

that was close.


Cleanup 500s due to Google oauth2 errors.

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

Merged.


"My repositories" on home page when logged in

Similar to the way GitHub does it, I'd like to see all the repos I'm assigned to. At the moment I have to go to the org that repo is in and then view the repo that way which can be annoying

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/40078162-my-repositories-on-home-page-when-logged-in?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github).

Currently there is My Repositories and Collaborative Repositories on the home page when you logged in. This is almost the same with Github. I think you mean Collaborative Repositories and Gitea already satisfied with your requirement.


[BUG] Large QR-Code

In raising this issue, I confirm the following:

[X] I have read and understood the contributors guide.
[X] The issue I am reporting can be replicated.
[X] The issue I am reporting is directly related to the pivpn installer script.
[X] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Describe the bug
Changes made in commit 615c2a41f4293d517237125c2aad6f88f9226db0 result in a very large QR code
Screenshots
In the attached graphic, I have indicated how large the QR is, and so it is not visible in its entirety because there are still 11 lines missing above.
Tested the recommended change by @4s3ti, unfortunately it causes the problem:
Please enter the Index/Name of the Client to show: 1
::: Showing client TestUsr below
=====================================================================
Version must be specified to encode a Micro QR Code symbol.
=====================================================================

Additional context

PiVPN server site:

Fresh installation (6 hours ago) Raspberry Pi OS Lite
OS is fully updated
Changed a couple of security settings
Before installing PiVPN, UFW was installed

Windows 10 + PuTTY

@orazioedoardo if it helps ... this is mine:
┌─[f0rs3ti@deathstar]─[~]
└──╼ $echo $TERM
xterm-256color

I was able to understand the issue even tough to me doesn't really make much difference due my screen size and resolutions.

xterm-256color

Same, @SilkeKohl what about yours?


Feature - CircularFlow by Rodrigo Dominguez

CircularFlow
I add a class called CircularFlow which allows us to group views with circular references.
constraint_referenced_ids : It receives id's of the views that will add the references (Eg. "view2, view3, view4,view5,view6").
circularflow_viewCenter : It receives the id of the view of the center where the views received in constraint_referenced_ids (Eg. "view1")
circularflow_angles : Receive the angles that you will assign to each view (Eg. "45,90,135,180,225")
circularflow_radiusInDP : Receive the radios in DP that you will assign to each view (Eg. "90,100,110,120,130")
How add a view at runtime?
With method addViewToCircularFlow you can add new views to CircularFlow.
 /**
 * Add a view to the CircularFlow. The referenced view need to be a child of the container parent.
 * The view also need to have its id set in order to be added.
 * The views previous need to have its radius and angle set in order to be added correctly a new view.
 * @param view
 * @param radius
 * @param angle
 * @return
 */

Eg.
findViewById<View>(R.id.view7).setOnClickListener {
            findViewById<CircularFlow>(R.id.circularFlow).addViewToCircularFlow(
                it, 140, 200F
            )
        }

Demo add view in emulator
https://user-images.githubusercontent.com/24611045/107442324-ad3eee80-6b15-11eb-89b5-7f3221f2295d.mp4
How remove a view at runtime?
With method removeViewFromCircularFlow you can remove a view from CircularFlow.
 /**
     * Remove a given view from the CircularFlow.
     *
     * @param view
     */

Eg.
findViewById<View>(R.id.view2).setOnClickListener {
            findViewById<CircularFlow>(R.id.circularFlow).removeViewFromCircularFlow(
                it
            )
        }

Demo remove view in emulator
https://user-images.githubusercontent.com/24611045/107656012-80e4b880-6c62-11eb-924f-b0fa1244a2bf.mp4
Demos in AS:
https://user-images.githubusercontent.com/24611045/106652697-9aa44280-6574-11eb-819b-cb09f012f748.mov
https://user-images.githubusercontent.com/24611045/106653120-25853d00-6575-11eb-86ab-dca1a1c85ed9.mov

@googlebot I consent.

@googlebot I consent

@googlebot I consent.

Thanks for your review Nicolas 🚀


[Website Request] The Athletic

New Site Requests
[The Athletic] (https://theathletic.com/)

I don't think this extension is being maintained anymore.
Check out BPC (Bypass Paywalls Clean), it supports bypass of The Athletic.


fix(typeahead): close dropdown on tab with focus-first="false"

When an item isn't selected and you hit "tab", the dropdown stays open even though the focus moves to the second field. An alternative solution might handle this using the 'blur' event. Happy to discuss this.
See this plnkr: http://plnkr.co/edit/zR0HYGVSlMdLgRvCEIZv?p=preview

This is good stuff. I'd actually add the same behavior for the Enter key press, since it is basically the same issue in my opinion, i.e. the user has no intent in modifying the input anymore, either because he changed to a different input by pressing Tab or because he wants to submit the form by pressing Enter.
Either you can add this or I'll do a pull request with that change later on if my opinion is not shared.

This is mostly there, just fix those two things up :) .

@wesleycho I deliberately left the == comparisons as they were. I'm not sure of the ramifications of changing them... Perhaps somewhere they are being set as strings and the coercion is necessary? I don't think they should be changed as a part of this PR, since this PR is concerned with fixing the issue mentioned above rather than refactoring existing code which will carry more risk of introducing a bug.

@fernandoneira I agree with you, however the comment suggested this was by design. If others are in support, I'd be happy to make it work like this for the "enter" keypress too.

I will be merging this shortly, along with the change to === - I did an investigation and it looks like it is only being set to integers, so this should be safe.


Glossary word popover

Update (12 Feb)

The popover is there, but it isn't styled like the new design yet

Acceptance criteria

The first instance of a glossary term in an article will show the grey underline
Hover over that word reveals a popover, which includes

Glossary term
Definition (make a note to writers to cap the definition at N number of character; not sure what this would be honestly since different characters have different widths)
Link to article page (sometimes exists sometimes doesn't)
Image (optional)

Min size of 202px width and 304px height (at the least just put a note to uploaders to make sure image is at least this size, perhaps in Coda)
If image is larger than this, crop it either vertically centered or horizontally centered

Link
https://www.figma.com/file/qqfeYAh6Z7QdyJXsQjIUfh/Stampy?type=design&node-id=2073%3A8816&mode=dev&t=5zH2EaswPLkKJBPS-1

@mruwnik
Can you make it so the word is not repeated twice?

In cases like "agentic" can you just make the word "agent" show?

I also want it to appear under the word rather than to the right, and some styling fixes but I think I can probably fix all that.

That's mainly a copy issue, see the glossary doc.
Each glossary item has a main phrase and aliases, all of which will be underlined. Do you always want the main phrase to be the bold word, even if a different word is underlined in the main text?

Update:

Editors will upload exact right photo size
Re: copy length, added ellipses if it gets too long

Ask editors to keep things at a certain length too

Melissa get in touch with copy

Update: Use term instead of alias

Update: the term looks right, just checking to make sure the ellipsis will appear
Then just a few small styling things

(sorry)

Why is the line height off? Did you use the class .paragraph-small? (sorry, line height is one thing I'm a bit of a perfectionist about)

also seems more text appears after the ellipsis

update: Melissa fix line height

update: last issue is just that there is a bug on brave where there is more text after the ellipsis


modify resource tasks to not chain input properties

This fixes issue #16
Rather than chaining properties for task inputs, this calls getInputs().property(String,Object) within each task that uses it.

Coverage increased (+0.01%) to 98.271% when pulling dd4632b8af2580e0557a19ff42f611a61eb085c6 on gordolio:issue-16-gradle-3.3-deprecation into 4601224feb6488235c7703807430645b4b37dcc6 on ksoichiro:master.

Coverage increased (+0.01%) to 98.271% when pulling dd4632b8af2580e0557a19ff42f611a61eb085c6 on gordolio:issue-16-gradle-3.3-deprecation into 4601224feb6488235c7703807430645b4b37dcc6 on ksoichiro:master.

Thank you!


Automatically infer from source data

I have problems to automatically infer from source data in the metadata section. I get the message "No data mappings! Please create data mapping and try again". It is the same for Geographic Coverage, Taxonomic Coverage and Temporal Coverage. What kind of data mapping is this?
The tables are imported and mapped to DwC. We have IPT version 3.01. Earlier these functions worked.

Thank you for contacting us.
This might be a bug. Can I have the IPT logs please?

Thanks for following up this! Here is the log:
https://www.dropbox.com/scl/fi/o0x9ekb4uwmdljgx5yq28/Log.txt?rlkey=nl4xyusgpk1levw8axf9vdy33&dl=0

Btw, have you tried to do Re-infer? Doesn't it help?

Sorry, how do we re-infer?

There is a button, see example:

Yes, several times. When I tick it I get: No data mappings! Please create data mapping and try again

Yes, that worked. Thank you!

Good. I'll create an issue to make it more obvious that that option exists


[Studio]{Stories}Enable AutoSave or a message to save before changing screen

Hi,
I use to forget clicking on the "Update Story" button.
Maybe it would be possible to add a modal to give the ability to save before changing from this screen to another
Or a toggle button in settings that enable autosave?
With Entities it's already OK: When you add some entities values are saved without a "Save Entity" Button.
Regards
Mat

partial save already exist ;)


Introduce a flag to control cstyle failures in auto merge script

Introduce -s flag to control if a cstyle failure should cause
the merge process to fail. Corrected some style as well.
Signed-off-by: Giuseppe Di Natale<EMAIL_ADDRESS>

@gmelikov would you mind taking a look?


Allow user to supply custom TickIntervalFunction to the Axis

Allowed user to defined a tickIntervalFunction inc ase the user wants to do custom operation on tickInterval.
In my case, I did not want the first value to be the min, but min + step. I also wanted to clip the top.

Thank you
Can you share an example tickIntervalFunction please.
Appreciate any documentation you can add for this feature to make it easy to discover for others

Yes, here's what I have for the yGrid which uses the tickIntervalFunction:
  const yGrid = {
    axisAt: 'right',
    orient: 'right',
    innerTickSize: -1 * gridWidth,
    tickStrokeDasharray: 'Solid',
    tickStrokeOpacity: 0.6,
    tickStrokeWidth: 1,
    tickInterval: Math.floor(gridHeight / 60),
    tickIntervalFunction: (min, max, tickInterval) => {
      const step = (max - min) / tickInterval
      let tickValues = d3Range(min + step, max, (max - min) / tickInterval); // eslint-disable-line
      tickValues.length == tickInterval? tickValues.pop(): '' // eslint-disable-line
      return tickValues
    }
  }

Basically, I am hiding the first step, then I am making sure that the last one does not show up in case the last value == max. (It's an edge case, but one that I saw happening, so wanted to get rid of it)
I'm happy to add this to the doc, but I'm not sure where. Can u link? I'll edit.

@ghost where is your d3Range func?


How to show step indicator horizontally when there are 10 steps?

Hi, I have more than 10 steps that I need to present horizontally. But when I'm doing that I'm getting all steps attached to each other as below.

Even I tried to wrap the step indicator with ScrollView but still, the issue isn't solved.
<ScrollView horizontal>
    <StepIndicator
        customStyles={indicatorStyles}
        currentPosition={0}
        stepCount={data.length}
        renderStepIndicator={renderImages}
        onPress={position => onStepPressHandler(position)}
    />
</ScrollView>

Can someone please help me with this

May be you can just reduce StepIndicatorSize and size of icons inside step as well

Same issue

same issue


Enable basic certificate auth for resource service

This adds the ability to configured the gRPC connection from the dashboard to a resource service to use certificates for authentication.
Such auth is not used in the local dev scenario, so the app host is changed to suppress auth via the DOTNET_RESOURCE_SERVICE_DISABLE_AUTH environment variable. The dashboard will now require certificates unless this variable is set to 1. This discourages self-hosting the dashboard in an insecure manner.

A follow-up PR will cover OIDC for the web app.
Microsoft Reviewers: Open in CodeFlow

We currently don't have a resource service implementation that works with certificates. To test this, I created certs with:
# create root CA details
openssl genrsa -aes256 -out rootCA.key 4096
openssl req -x509 -new -nodes -key rootCA.key -sha256 -days 3650 -out rootCA.crt -subj "//CN=MyRootCA"

# server
openssl genrsa -aes256 -out server.key 4096
openssl req -new -key server.key -out server.csr -subj "//CN=localhost"
openssl x509 -req -in server.csr -CA rootCA.crt -CAkey rootCA.key -CAcreateserial -out server.crt -days 365 -sha256
openssl pkcs12 -export -out server.pfx -inkey server.key -in server.crt

# client
openssl genrsa -aes256 -out client.key 4096
openssl req -new -key client.key -out client.csr -subj "//CN=localhost"
openssl x509 -req -in client.csr -CA rootCA.crt -CAkey rootCA.key -CAcreateserial -out client.crt -days 365 -sha256
openssl pkcs12 -export -out client.pfx -inkey client.key -in client.crt

Then the following changes to the app host project:
diff --git a/src/Aspire.Hosting/Aspire.Hosting.csproj b/src/Aspire.Hosting/Aspire.Hosting.csproj
index 0707a57c..bc25bdd0 100644
--- a/src/Aspire.Hosting/Aspire.Hosting.csproj
+++ b/src/Aspire.Hosting/Aspire.Hosting.csproj
@@ -26,6 +26,7 @@
   <ItemGroup>
     <PackageReference Include="Grpc.AspNetCore" />
     <PackageReference Include="KubernetesClient" />
+    <PackageReference Include="Microsoft.AspNetCore.Authentication.Certificate" />
     <PackageReference Include="Microsoft.Extensions.Hosting" />
   </ItemGroup>

diff --git a/src/Aspire.Hosting/Dashboard/DashboardService.cs b/src/Aspire.Hosting/Dashboard/DashboardService.cs
index 16cec459a..4c2b1da2d 100644
--- a/src/Aspire.Hosting/Dashboard/DashboardService.cs
+++ b/src/Aspire.Hosting/Dashboard/DashboardService.cs
@@ -4,6 +4,7 @@
 using System.Text.RegularExpressions;
 using Aspire.V1;
 using Grpc.Core;
+using Microsoft.AspNetCore.Authorization;
 using Microsoft.Extensions.Hosting;

 namespace Aspire.Hosting.Dashboard;
@@ -15,6 +16,7 @@ namespace Aspire.Hosting.Dashboard;
 /// An instance of this type is created for every gRPC service call, so it may not hold onto any state
 /// required beyond a single request. Longer-scoped data is stored in <see cref="DashboardServiceData"/>.
 /// </remarks>
+[Authorize]
 internal sealed partial class DashboardService(DashboardServiceData serviceData, IHostEnvironment hostEnvironment, IHostApplicationLifetime hostApplicationLifetime)
     : V1.DashboardService.DashboardServiceBase
 {

diff --git a/src/Aspire.Hosting/Dashboard/DashboardServiceHost.cs b/src/Aspire.Hosting/Dashboard/DashboardServiceHost.cs
index 20ee29d0..71d2df3a 100644
--- a/src/Aspire.Hosting/Dashboard/DashboardServiceHost.cs
+++ b/src/Aspire.Hosting/Dashboard/DashboardServiceHost.cs
@@ -3,13 +3,16 @@

 using System.Diagnostics;
 using System.Net;
+using System.Security.Cryptography.X509Certificates;
 using Aspire.Hosting.ApplicationModel;
 using Aspire.Hosting.Dcp;
+using Microsoft.AspNetCore.Authentication.Certificate;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Hosting.Server;
 using Microsoft.AspNetCore.Hosting.Server.Features;
 using Microsoft.AspNetCore.Server.Kestrel.Core;
+using Microsoft.AspNetCore.Server.Kestrel.Https;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
@@ -72,6 +75,53 @@ internal sealed class DashboardServiceHost : IHostedService
             // Turn on HTTPS
             builder.WebHost.UseKestrelHttpsConfiguration();

+            #region TEMPORARY TEST CODE
+
+            // Auth
+            builder.Services
+                .AddAuthentication(CertificateAuthenticationDefaults.AuthenticationScheme)
+                .AddCertificate(options =>
+                {
+                    // Disallow self-signed certs.
+                    options.AllowedCertificateTypes = CertificateTypes.Chained;
+
+                    // Revocation checks require an online CA, which we don't have during testing.
+                    options.RevocationMode = X509RevocationMode.NoCheck;
+
+                    options.Events = new CertificateAuthenticationEvents()
+                    {
+                        OnAuthenticationFailed = context =>
+                        {
+                            _logger.LogError(context.Exception, "Failed authentication.");
+
+                            return Task.CompletedTask;
+                        },
+                        OnCertificateValidated = context =>
+                        {
+                            // TODO validate certificate is as expected
+
+                            _logger.LogInformation("Authentication complete.");
+
+                            return Task.CompletedTask;
+                        }
+                    };
+                });
+
+            builder.Services.AddAuthorization();
+
+            builder.Services.Configure<KestrelServerOptions>(options =>
+            {
+                options.ConfigureHttpsDefaults(options =>
+                {
+                    options.ServerCertificate = new X509Certificate2(@"d:\repos\tmp\certs\server.pfx", "server", X509KeyStorageFlags.DefaultKeySet);
+                    options.ClientCertificateMode = ClientCertificateMode.RequireCertificate;
+                });
+            });
+
+            #endregion
+
             // Configuration
             builder.Services.AddSingleton(configuration);

@@ -80,7 +130,7 @@ internal sealed class DashboardServiceHost : IHostedService
             builder.Services.AddSingleton(loggerOptions);
             builder.Services.Add(ServiceDescriptor.Singleton(typeof(ILogger<>), typeof(Logger<>)));

-            builder.Services.AddGrpc();
+            builder.Services.AddGrpc(options => options.EnableDetailedErrors = true);
             builder.Services.AddSingleton(applicationModel);
             builder.Services.AddSingleton(kubernetesService);
             builder.Services.AddSingleton<DashboardServiceData>();
@@ -91,6 +141,13 @@ internal sealed class DashboardServiceHost : IHostedService

             _app = builder.Build();

+            #region TEMPORARY TEST CODE
+
+            _app.UseAuthentication();
+            _app.UseAuthorization();
+
+            #endregion
+
             _app.MapGrpcService<DashboardService>();
         }
         catch (Exception ex)

The config must be updated to ensure DOTNET_RESOURCE_SERVICE_ENDPOINT_URL is set to localhost, as by default it finds a random port on <IP_ADDRESS> which then fails certificate name validation. The host name must match the CN of the cert.

We currently don't have a resource service implementation that works with certificates.

Integration tests for this would be valuable. You can create a very simple ASP.NET Core app with a dummy OTLP endpoint and cert auth, then call it from the dashboard.
I added a number of integration tests for various OTLP scenarios. You can look to them as examples for how to do integration tests with ASP.NET Core apps.
Launching the dashboard to call the OTLP endpoint would be the most realistic, but I don't know a could way to then assert results. Could just test DashboardClient in isolation.

Integration tests for this would be valuable

This is on my radar. I won't have time to add them before the p5 snap.

@kvenkatrajan @JamesNK @davidfowl this is ready for review.
I'll investigate the test failure tomorrow.

It would be great to have an example using this with keycloak. Could be a playground project (and sample). Not required to get this in but we should do it (especially since it'll serve as a way of testing the functionality).

@javiercn @halter73 @SteveSandersonMS can I get you to take a look at how we're doing auth in Blazor server? One thing I think we may want to do is show the user information somewhere.
@kvenkatrajan @leslierichardson95 do we want more UI around logging in? The current mode will just challenges for auth immediately without a user gesture.

@davidfowl I've taken a look and it looks good.
The user should be automatically available in Blazor during SSR render and in interactive Server.

Should we be using the top level attribute or the authorize view on the router so we can show login screen options?

Should we be using the top level attribute or the authorize view on the router so we can show login screen options?

Do you mean the AuthorizeView with the NotAuthorized fragment?

It would be great to have an example using this with keycloak. Could be a playground project (and sample). Not required to get this in but we should do it (especially since it'll serve as a way of testing the functionality).

This would be possible for the web app's auth, but not for the resource service's.
Would the user need to provide their own OIDC authority, client ID and secret? I'm not familiar with keycloak, if that's related.

This would be possible for the web app's auth, but not for the resource service's.

Yep.

Would the user need to provide their own OIDC authority, client ID and secret? I'm not familiar with keycloak, if that's related.

Yes. We have an example of using keycloak here. @DamianEdwards can help since he spent all of that time setting it up

There are failing integration tests. Even with an Unsecured setting, requests still require auth.
I used OtlpCompositeAuthenticationHandler to meet the authorization requirement when auth was disabled. I don't know if it's the best way, but it worked for me.
How much testing have you done that the different endpoint's auth don't conflict with each other? I couldn't see code to select which auth is run, although I haven't that before so I'm not sure how to go about doing it.

At the very least I think one integration test with browser auth enabled and the server returning a 401 is required to ensure this is working.
There are a lot of complex moving parts here and we don't develop with auth enabled. It is very easy to create regressions. The integration tests around OTLP auth have been merged only a week and have caught a number already.

At the very least I think one integration test with browser auth enabled and the server returning a 401 is required to ensure this is working.

I took a look at this today, as I agree a simple test would be good. We want to validate that we receive a 302 Found with a Location at the OpenID provider to which the user would be redirected.
The challenge with this is that it has to be a live OpenId provider, as the server internally reaches out to https://<authority>/.well-known/openid-configuration. I'm going to investigate spinning up a dummy OpenId provider during the test. I might not complete that in time for p5.

I've pushed an update that unifies the OTLP and web app authentication/authorization code where possible, and have validated both work as expected.
Also fixed conflicts with main.
@JamesNK, as discussed offline, some OTLP configuration values are renamed in this PR:

DOTNET_DASHBOARD_OTLP_AUTH_MODE → Otlp:AuthMode
DOTNET_DASHBOARD_OTLP_API_KEY → Otlp:ApiKey
DOTNET_DASHBOARD_INSECURE_ALLOW_ANONYMOUS is no longer needed → set Otlp:AuthMode to Unsecured
For Otlp:AuthMode, the option None → Unsecured

I'm going to investigate spinning up a dummy OpenId provider during the test.

You can take a look at a dummy OpenId provider I just added to test a trimming issue in Microsoft.AspNetCore.Components.WebAssembly.Authentication. I only did enough for oidc-client-js to let me complete the flow. It's possible that the server-side OpenIdConnectHandler might need more endpoints for stuff like JWKS, but it should still serve as a decent starting point.
https://github.com/dotnet/aspnetcore/blob/3f99d45b0b7d8f0427a3d98acc63098694613362/src/Components/test/testassets/Components.TestServer/RemoteAuthenticationStartup.cs#L37-L94

Should we be using the top level attribute or the authorize view on the router so we can show login screen options?

As you know from https://github.com/dotnet/aspire/pull/3011, endpoint routing doesn't integrate super cleanly with older Blazor concepts like <NotFound>. Same goes for top-level authorize attributes and <AuthorizeView>/<NotAuthorized>, because the authorization middleware will have the authentication handler issue a challenge before Blazor gets a chance to interact with the request.
I recommend using the top-level authorize attribute for when the entire page needs authorization. You just need to setup the login redirect using the authentication handler like you would do for any typical server rendered MVC/Razor Pages/Minimal APIs/etc... app. <AuthorizeView> is still useful for non-page components that need to show different content based on whether the user is logged in or not. For example, we use this to show a login/logout link/form in the Blazor project template.
Since you're using global InteractiveServerRenderMode, normal server-side auth approaches should work fine. If you want to try auto render mode (which I don't necessarily recommend), you can look at the custom server and client AuthenticationStateProvider implementations in the Blazor project that use PersistentComponentState to flow a customized subset of claims from the server to the client. This is also demonstrated in the latest Secure an ASP.NET Core Blazor Web App with OIDC docs which go further to demonstrate using the BFF pattern so the client APP can authenticate with a backend using the JwtBearerHandler via a YARP proxy.


add RTL8189FTV driver to HI3518EV300

Hi,
I tried adding the RTL8189FTV driver driver to OpenIPC firmware for HI3518EV300, but the built images does not contain any RTL8189FTV related files.
branch: https://github.com/OpenIPC/firmware/compare/master...danielk117:firmware:master
firmware: https://github.com/danielk117/firmware/releases/download/latest/openipc.hi3518ev300-br.tgz
build log: https://github.com/danielk117/firmware/commit/2d15250e3cf7e5f75bf3f428f75cd63e21cdb531/checks/8034925614/logs
I missed something or did something wrong. Has someone a hint for me, how to fix this? Thank you!

Have you tried local build? it doesn't look like driver is built, probably there's something in it's makefile that is preventing it from build and gives graceful exit though

https://github.com/jwrdegoede/rtl8189ES_linux/blob/master/Makefile#L38 i'd start from here, there is no CONFIG_RTL8189FTV there

Yes, CONFIG_RTL8189FTV makes no sense. I have no local enviroment for testing the build.
I just followed https://linux-sunxi.org/Wifi#RTL8189FTV and the Makefile in that branch https://github.com/jwrdegoede/rtl8189ES_linux/blob/rtl8189fs/Makefile has
CONFIG_RTL8188F = y and CONFIG_SDIO_HCI = y as default. So the outputted files could be named something like 8189fs.

I tried a couple of ENV combinations, but I wasn't able to get the driver build. It always results in something like this.
This whole buildroot and make things are really new to me. So someone with more make and driver knowledge will probably have to do it.

I use openipc source code ,add "BR2_PACKAGE_RTL8189FS=y" in hi3518ev200 defconfig,and add "CONFIG_RTL8189FS=m" in hi3518ev200.generic.config, then "make BOARD=unknown_unknown_hi3518ev200_openipc all".Eventually 8189fs can used .
The problem is that when I run "load_hisilicon -i -sensor jxh62 -total 64", dmesg gets "himci: card disconnected!" SDIO WIFI
become unavailable.

At the telegram channel for the alternative micam mjsxj02hl firmware, the maintainer of the firmware tells @ZigFisher how he build the driver for openipc. Hopefully he will add the driver soon. 😉
https://t.me/mjsxj02hl_firmware/4920


Related section not collapsed after first reply

https://stacker.news/items/426061
We want to keep this functionality but it should collapse related posts on the first comment submission.

This issue was fixed by https://github.com/stackernews/stacker.news/commit/dc8d35fdcfdaa55b7ba30035fee5bd3a8cd00b9c

https://github.com/stackernews/stacker.news/commit/dc8d35fdcfdaa55b7ba30035fee5bd3a8cd00b9c is related but this change only collapses the related section when you load the page again. This ticket is about updating the cache so you don't have to reload the page after you replied. See this video:
https://github.com/stackernews/stacker.news/assets/27162016/125101ef-f28c-4cac-b678-24b9ba1b442d

Turned out that the issue wasn't really to do with Apollo cache but the Accordion component instead. Tricky one to find but added a PR here #1207

Oh, I thought it might be controlled via the Apollo cache but I didn't look at the code. Sorry for the false pointer


Allow choosing the server alongside character name for profiles/editor

This is useful when people have several characters with the same name on different server.

Implemented in <IP_ADDRESS>


specify user-agent in tmy3 remote request

[ ] Closes #493
[ ] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests must pass on the TravisCI and Appveyor testing services.
[ ] Code quality and style is sufficient. Passes git diff upstream/master -u -- "*.py" | flake8 --diff and/or landscape.io linting service.
[ ] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[ ] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.

@wholmgren Thanks for tracking this one down.

@cwhanse any concerns here? if not, please go ahead and "squash and merge".


TileSprite seams visible when using atlases and WebGL

If you load a tilesprite from an atlas and use WebGL for rendering, you can see a faint seam were the tilesprite loops. It doesn't happen with Canvas.

The texture has crisp borders and proper paddding
It happens at least on Chrome 34.0.1847.137(win7-64) and Firefox 30.0(win7-64)
It affects at least Phaser 2.0.4 and 2.0.5
It might be hard to notice if the texture is too complex or noisy

I'm adding an image in case the description isn't clear enough.

See this alternative for a different approach:
Render Layer as Texture


switch to git-flow branching model

master branch should point to the latest tag.
(Based on Slack discussion.)
@vors I'll take a look at git-flow, but feel free to add any details that need to be sorted out for this.

UPD on it: http://endoflineblog.com/gitflow-considered-harmful
Once we switch to EditorSyntax, we will just have master with the latest EditorSyntax release