XML entity shows up as "&" instead of "&" after repacking apk

Describe the bug
It appears that APKEditor cannot correctly handle XML entities in resource strings.
String in source code: <string name="test_string">\"Alma\" & Körte</string>.
Expected behaviour: The string should show up as "Alma" & Körte after repacking the apk.
Actual behaviour: The string shows up as "Alma" & Körte after repacking the apk.
To Reproduce
Steps to reproduce the behavior:

Used version '1.2.5'
Operating system 'Linux <IP_ADDRESS>-microsoft-standard-WSL2 x86_64 GNU/Linux'
Commands java -jar APKEditor-1.2.5.jar d -dex -i app-debug.apk  and java -jar APKEditor-1.2.5.jar b -i app-debug_decompile_xml -o packed.apk

Log/Stacktrace
Used apk file
app-debug.apk.zip
Additional context

Source code for apk (android studio, new project, empty activity)
strings.xml
<resources>
    <string name="app_name">APKEditorTest</string>
    <string name="test_string">\"Alma\" & Körte</string>
</resources>

MainActivity.kt
package com.example.apkeditortest

import android.os.Bundle
import androidx.activity.ComponentActivity
import androidx.activity.compose.setContent
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.material3.MaterialTheme
import androidx.compose.material3.Surface
import androidx.compose.material3.Text
import androidx.compose.runtime.Composable
import androidx.compose.ui.Modifier
import androidx.compose.ui.tooling.preview.Preview
import com.example.apkeditortest.ui.theme.APKEditorTestTheme

class MainActivity : ComponentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            APKEditorTestTheme {
                // A surface container using the 'background' color from the theme
                Surface(
                    modifier = Modifier.fillMaxSize(),
                    color = MaterialTheme.colorScheme.background
                ) {
                    Greeting(getString(R.string.test_string))
                }
            }
        }
    }
}

@Composable
fun Greeting(name: String, modifier: Modifier = Modifier) {
    Text(
        text = "Hello $name!",
        modifier = modifier
    )
}

@Preview(showBackground = true)
@Composable
fun GreetingPreview() {
    APKEditorTestTheme {
        Greeting("Android")
    }
}

I think it has something to do with #49

I confirmed this and will be fixed soon

The issue seems to be fixed in version 1.2.6

I am having the issue is APKEditor 1.3.2

to add more details:
I believe this line in resources.arsc.json is where the string is.
        "entry_name": "reset_phone_sdcard_title",
        "value": {
         "value_type": "STRING",
         "data": "Phone & SD card"
        },
        "id": 3384
       },

Here is how it looks on the phone:

I guess my situations is a bit different because it doesnt show "&amp", it shows something else strange.
link to apk: https://filetransfer.io/data-package/6UuDwzLZ#link
Mirror: https://murena.io/s/kCTBjTXAHQLAs8y
Keep in mind that this is a system app and you probably cannot install it on your phone

Update
Hey folks, quick update on this!
After a day of digging around (and probably way too many Google searches), I think I’ve cracked the mystery.
Here’s what was happening:

On Windows:
It was showing this weird behavior:

On Debian Linux (via WSL):
It looked like this instead:

After some sleuthing, I found out that this is likely a file encoding issue. A helpful StackOverflow thread pointed me in the right direction.
Turns out, Java 18 and above defaults to UTF-8 encoding (source), while I was still running Java 17. Once I upgraded to Java 23, everything started working perfectly on both Windows and Linux.
🎉 Problem solved!
For anyone still using older Java versions, it would probably help if apkeditor (or arsclib maybe? Not sure) explicitly set file encoding to utf-8. I’m not super familiar with Java, so I won’t be submitting a PR (sorry), but hopefully, this info helps someone out there.
Cheers!


Text Edits on Teachings Page

https://github.com/atheycreek/atheycreek/blob/5d58bf1c9d51527c0e8f5140df1e824a810f7067/workspace/xsl/teachings.xsl
Line 13 should be:
This page contains every single teaching we have done, ordered by book of the Bible.
Line 61 should be:
This page contains every single teaching tag, ordered alphabetically.
Thank you.

@davidfrost I fixed it! I actually made it smarter. Instead of being in plain text, I stored the values in the backend of Symphony so Athey staff can edit them whenever they want. To edit the values go to Misc and find teachings-by-book or teachings-by-tag. Or here are the direct links:

teachings-by-book
teachings-by-tag

@davidfrost Note that I already added your edits so this is for if/when you need to change them in the future.

Thanks so much!
Congrats on the cute kid too!
On Jan 27, 2016, at 3:06 PM, Jonathan Simcoe <notifications<EMAIL_ADDRESS>wrote:
@davidfrosthttps://github.com/davidfrost I fixed it! I actually made it smarter. Instead of being in plain text, I stored the values in the backend of Symphony so Athey staff can edit them whenever they want. To edit the values go to Mischttp://atheycreek.com/admin/publish/misc/ and find teachings-by-book or teachings-by-tag. Or here are the direct links:

teachings-by-bookhttp://atheycreek.com/admin/publish/misc/edit/16809/
teachings-by-taghttp://atheycreek.com/admin/publish/misc/edit/16810/

—
Reply to this email directly or view it on GitHubhttps://github.com/atheycreek/atheycreek/issues/54#issuecomment-175904599.


Dtk package refactor

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b40cb03). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #215   +/-   ##
=========================================
  Coverage          ?   97.34%
=========================================
  Files             ?       42
  Lines             ?     1960
  Branches          ?        0
=========================================
  Hits              ?     1908
  Misses            ?       52
  Partials          ?        0

Impacted Files
Coverage Δ

packages/Interface/test/tstKokkosView.cpp
100% <ø> (ø)

packages/Search/test/tstDetailsAlgorithms.cpp
100% <ø> (ø)

packages/Interface/src/DTK_UserApplication.hpp
100% <ø> (ø)

packages/Interface/src/DTK_InputAllocators_def.hpp
100% <ø> (ø)

packages/Interface/test/tstView.cpp
100% <ø> (ø)

...kages/Search/src/details/DTK_DetailsAlgorithms.hpp
91.78% <ø> (ø)

packages/Interface/src/DTK_EvaluationSet.hpp
100% <ø> (ø)

...es/Search/src/details/DTK_DetailsTreeTraversal.hpp
96.15% <ø> (ø)

packages/Search/src/DTK_BVHQuery_decl.hpp
100% <ø> (ø)

packages/Interface/src/DTK_NodeList.hpp
100% <ø> (ø)

... and 28 more

Continue to review full report at Codecov.

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

Related to #217.


Profile data from the registrationForm model are not being save in profile table. #541

Refering to the issue #541 Execuse me to take long it because of internet issue and it is solved now.
I followed exactly that tutorial but what is there is in the user model there aren't code to save these data add on registration form. In the user model the gravatar email is the only one thing that is being save in aftersave; I think that why the other profile data are not being saved. I try to add things manually as explain about but nothing worked out for me. In the user Model there isn't even setProfile method.

Please stop duplicating issues on one topic. I've answered to you in #541

I would like to give you the feedback but no way since the issue has been closed.
When I try to install the version 0.9.5 I am getting this error from the composer:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- yiisoft/yii2-jui 2.0.4 requires bower-asset/jquery-ui 1.11.@stable -> no matching package found.
- yiisoft/yii2-jui 2.0.3 requires bower-asset/jquery-ui 1.11.@stable -> no matching package found.
- yiisoft/yii2-jui 2.0.2 requires bower-asset/jquery-ui 1.11.@stable -> no matching package found.
- yiisoft/yii2-jui 2.0.1 requires bower-asset/jquery-ui 1.11.@stable -> no matching package found.
- yiisoft/yii2-jui 2.0.0 requires bower-asset/jquery-ui 1.11.*@stable -> no matching package found.
- dektrium/yii2-user v0.9.5 requires yiisoft/yii2-jui ~2.0 -> satisfiable by yiisoft/yii2-jui[2.0.0, 2.0.1, 2.0.2, 2.0.3, 2.0.4].
- Installation request for dektrium/yii2-user v0.9.5 -> satisfiable by dektrium/yii2-user[v0.9.5].
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your minimum-stability setting
see https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion for more details.

Read https://getcomposer.org/doc/articles/troubleshooting.md for further common problems.
Installation failed, reverting ./composer.json to its original content.
I don't know what to do so that it works; I even remember that what sent me to choose * for version for the first time I was trying to install the extension to. Thanks for your support.
Date: Mon, 23 Nov 2015 11:29:27 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Closed #548.
—
Reply to this email directly or view it on GitHub.

Do you have assets plugin installed?

I installed Yii2 manually and think probably it is not installed. I am looking how to check it is installed and how to install it.
I'll let you know how it goes. I am stack in my project of the problem.
Date: Wed, 25 Nov 2015 12:43:50 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Do you have assets plugin installed?
—
Reply to this email directly or view it on GitHub.

I've installed everything and yii2-user 0.9.5 I see codes that could do the work I'm gonna try it today. Thanks a lot for leading me
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Date: Thu, 26 Nov 2015 09:03:47 +0200
I installed Yii2 manually and think probably it is not installed. I am looking how to check it is installed and how to install it.
I'll let you know how it goes. I am stack in my project of the problem.
Date: Wed, 25 Nov 2015 12:43:50 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Do you have assets plugin installed?
—
Reply to this email directly or view it on GitHub.

Now as I am trying to override the loadAttributes of registrationForm; I am getting the error :
            PHP Strict Warning
            – yii\base\ErrorException            Declaration of

app\models\RegistrationForm::loadAttributes() should be compatible with
dektrium\user\models\RegistrationForm::loadAttributes(dektrium\user\models\User
$user)
What I did is :
protected function loadAttributes(User $user)
{
$user->setAttributes([
'email'    => $this->email,
'username' => $this->username,
'password' => $this->password,
]);
/** @var Profile $profile */
$profile = \Yii::createObject(Profile::className());
$profile->scenario = 'signup';
$profile->setAttributes([
'gravatar_email' => $this->email,
'pays' => $this->pays,
'ville'=>$this->ville,
'sexe'=>$this->sexe,
'name'=>$this->name,
'location'=>$this->location]);
$user->setProfile($profile);
}
I dont really know what is different from the method I am trying to override:
/**
* Loads attributes to the user model. You should override this method if you are going to add new fields to the
* registration form. You can read more in special guide.
*
* By default this method set all attributes of this model to the attributes of User model, so you should properly
* configure safe attributes of your User model.
*
* @param User $user
*/
protected function loadAttributes(User $user)
{
$user->setAttributes($this->attributes);
}
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Date: Thu, 26 Nov 2015 10:14:53 +0200
I've installed everything and yii2-user 0.9.5 I see codes that could do the work I'm gonna try it today. Thanks a lot for leading me
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Date: Thu, 26 Nov 2015 09:03:47 +0200
I installed Yii2 manually and think probably it is not installed. I am looking how to check it is installed and how to install it.
I'll let you know how it goes. I am stack in my project of the problem.
Date: Wed, 25 Nov 2015 12:43:50 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Do you have assets plugin installed?
—
Reply to this email directly or view it on GitHub.

You have to import User class at the beginning of your overridden Profile class since it is located in other namespace:
use \dektrium\user\models\User

Thanks a lot !!! it is working perfectly;
I would like to know when a user confirm email address by clicking the link in his email address does that login him automatically ?
Date: Thu, 26 Nov 2015 07:50:15 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
You have to import User class at the beginning of your overridden Profile class since it is located in other namespace:
use \dektrium\user\models\User
—
Reply to this email directly or view it on GitHub.

And I would also like to know how can I access the registered user Id directly after the registration in actionregister
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: RE: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
Date: Mon, 30 Nov 2015 11:39:09 +0200
Thanks a lot !!! it is working perfectly;
I would like to know when a user confirm email address by clicking the link in his email address does that login him automatically ?
Date: Thu, 26 Nov 2015 07:50:15 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [yii2-user]  Profile data from the registrationForm model are not being save in profile table. #541  (#548)
You have to import User class at the beginning of your overridden Profile class since it is located in other namespace:
use \dektrium\user\models\User
—
Reply to this email directly or view it on GitHub.

hello i have set up phone number field everything is working except on user/settings/account page phone number comes empty from database but its exists in db. here is my overridden settingsForm.php code
`<?php
namespace app\models;
class SettingsForm extends \dektrium\user\models\SettingsForm
{
/**
* @var string
/
public $phone;
/*
* @inheritdoc
*/
public function rules()
{
$rules = parent::rules();
$rules['phoneRequired'] = ['phone', 'required'];
$rules['phoneLength']   = ['phone', 'integer'];
return $rules;
}
public function attributeLabels()
{
$attributeLabels = parent::attributeLabels();
$attributeLabels['phone'] = 'Tphone';
return $attributeLabels;
}
protected function loadAttributes(User $user)

{
$user->setAttributes([
'email' => $this->email,
'username' => $this->username,
'password' => $this->password,
]);
/** @var Profile $profile */
$profile = \Yii::createObject(User::className());
$profile->scenario = 'settings';
$profile->setAttributes([
'phone' => $this->phone
]);
$user->setProfile($profile);
}
}`


[Backport main]Feature: rollover (#607)

feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: add write index in aliases

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: remove conditions section

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: merge data streams

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: add rollover cypress test

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: make cypress run

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat cypress optimize

Signed-off-by: suzhou<EMAIL_ADDRESS>
fix: rollover with mappings

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: make rollover works in index form

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: add placeholder

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: alignment

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update rollover description

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update snapshot

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: make cypress and unit test run

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
feat: update

Signed-off-by: suzhou<EMAIL_ADDRESS>
Signed-off-by: suzhou<EMAIL_ADDRESS>(cherry picked from commit 497d5f18570352dc2ded0cc224c51aab32b9c3fb)
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #635 (6291eaa) into main (a901f75) will decrease coverage by 0.27%.
The diff coverage is n/a.

:exclamation: Current head 6291eaa differs from pull request most recent head ff0e032. Consider uploading reports for the commit ff0e032 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #635      +/-   ##
==========================================
- Coverage   59.81%   59.55%   -0.27%
==========================================
  Files         274      291      +17
  Lines        9040     9617     +577
  Branches     1564     1685     +121
==========================================
+ Hits         5407     5727     +320
- Misses       3219     3447     +228
- Partials      414      443      +29

Impacted Files
Coverage Δ

...onents/FormGenerator/built_in_components/index.tsx
64.70% <0.00%> (-5.30%)
:arrow_down:

...c/pages/CreateIndex/containers/IndexForm/index.tsx
83.72% <0.00%> (-2.26%)
:arrow_down:

...ublic/pages/Aliases/containers/Aliases/Aliases.tsx
87.82% <0.00%> (-1.60%)
:arrow_down:

...ement-dashboards-plugin/public/pages/Main/Main.tsx
5.37% <0.00%> (-0.65%)
:arrow_down:

...gement-dashboards-plugin/public/utils/constants.ts
95.23% <0.00%> (ø)

...dashboards-plugin/public/services/CommonService.ts
100.00% <0.00%> (ø)

...ic/pages/Aliases/containers/AliasActions/index.tsx
100.00% <0.00%> (ø)

...pages/CreateIndex/components/AliasSelect/index.tsx

...CreateIndex/components/IndexDetail/IndexDetail.tsx

...ages/CreateIndex/components/IndexMapping/helper.ts

... and 24 more

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


Selector .tabs in a div wrapper - not in an ul - Reasonable advantages

I use a component based framework which for example has a navigation component that produces a standard ul list. I can place my navigation list on different positions (like dropdown header navigation - or a simple navigation list on the side)
Preferably i want the responsibility of setting the style and behaviour in the template itself. In my opinion there are advantages when there is a wrapper which holds the .tabs class attribute instead of placing the ".tabs" in  the ul (the navigation component html code) itself.
So for example css code like
.tabs .dropdown {
}

would only apply to navigation components placed  between a wrapper element with the class ".tabs"
Otherwise the navigation list would behave normally.

  #defined by template

Home

                            Dropdown

Secondary link

Something else here

Something else here

 #defined by template

I don't think the extra markup provides any additional benefits here. You mentioned there are many, but I don't see any listed. Sure you have an extra div, but what does that get you in terms of ease of styling? In this case, the ul acts as a wrapper and does everything we need it to.


Incorrect tooltip on the Install button

Here's what I see in the latest docker stack. Here the example is AWB, but other apps suffer from this as well.

Note that the button is active, but the tooltip says the app is incompatible, even though it clearly is. I've looked at the code in the app_manager.py but couldn't see an obvious problem. It's possible the underlying issue is in the aiidalab package.

Could be the description of the button is updated but not the tooltip. I'll have a look at it.


Getting Listings for one section

Hi,
Thanks for writing this gem! :) Can you tell me how to get the Listings for a single section of a single shop? I have the section I want but can't figure out how to work with it...
https://www.etsy.com/developers/documentation/reference/listing#method_findallshopsectionlistings says I need to pass at least shop_id and shop_section_id - is the below the correct syntax for that? I'm hoping that I'm just missing something from *identifiers_and_options but I don't know what it is. Can you help?
irb(main):044:0> user.shop.id
=> 5308141

irb(main):045:0> sect
=> #<Etsy::Section:0x00000001af7728 @result={"shop_section_id"=>7794481, "title"=>"Cabochons", "rank"=>5, "user_id"=>5569833, "active_listing_count"=>15}, @token=nil, @secret=nil>

irb(main):046:0> Etsy::Listing.find({:shop_id => user.shop.id, :shop_section_id => sect})
Etsy::EtsyJSONInvalid: Etsy::EtsyJSONInvalid
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/response.rb:92:in `validate!'
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/response.rb:58:in `result'
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/model.rb:67:in `get_all'
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/model.rb:24:in `get'
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/model.rb:96:in `find_one_or_more'
	from /var/lib/gems/1.9.1/gems/etsy-0.3.0/lib/etsy/listing.rb:75:in `find'
	from (irb):46
	from /usr/bin/irb:12:in `<main>'

Cheers,
Gavin.

Hm,
I got this to work using:
irb(main):077:0> listings = Etsy::Request.get("/shops/#{user.shop.id}/sections/#{sect.id}/listings")
=> #<Etsy::Response:0x000000020b8388 @raw_response=#<Net::HTTPOK 200 OK readbody=true>>

Is this the most optimal way?
Cheers,
Gavin.

I don't use the "models" (e.g. Listing), I always just use the requests directly. It might not be the most optimal way, but it's certainly how I would do it :)

Nice - then it's good enough for me, too - thanks for the response!


Onboarding mini-guides

### Tasks
- [ ] #267
- [ ] #268
- [ ] #269
- [ ] #270
- [ ] https://github.com/kleros/kleros-v2/issues/1301
- [ ] #271
- [ ] https://github.com/kleros/kleros-v2/issues/1450
- [ ] https://github.com/kleros/kleros-v2/issues/1458

fixeed


RFC: create a list of device pointers per driver subsystem

This the starts of a means to allow a list of devices per driver
class/subsystem for things like the shell to utilize.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

The idea here is to help replace the need for things like DT_ADC_{0..n}NAME that get used by the shell to know which devices are ADCs.  The problem with the scheme we've used is that it requires setting the magic #defines somewhere.  It also doesn't deal with the case of non-SoC devices well.  Additionally something like the I2C shell makes assumptions of "names" for I2C being prefixed with "I2C" which isn't always true.
So the idea is we add some kinda of REGISTER macro that will create a list of device pointers per driver subsystem.  We can then utilize that list in the shell or tests as we need.
The PR is mostly to start the discussion, as well as bike shed the names of things.

If we do decide it has to be done by recording a set of devices, I would prefer to see that use a more efficent representation. I've been pushing for integral (16-bit) handles as the canonical device identifier,

Good point, I proposed something similar for the PM dependency tree in  #22941 but it could be more widely used. Also, on a 100% DTS based build, the size size could be reduce to 8bits.


Transparent window on Linux ARM-64

Describe the bug
it doesn't show anything.

To Reproduce
dotnet new avalonia.mvvm
dotnet run

Expected behavior
show UI elements
Avalonia version
11.0.10
OS
Linux
Additional context
Ubuntu 22.04.3 LTS, Xfce 4.18
CPU: Snapdragon 8 Gen1 SM8450
GPU: Adreno 730
I'm using Termux PRoot to boot Linux distro in Android. and also using virgl to accelerate the GPU.

Having this issue too on Windows

Same issue on Windows after updating packages from 11.0.2 to 11.0.10. Adding Avalonia.Skia nuget fixed it for me.


Convert to VS Code project for development

Adds the necessary gitignores, configuration files, and usage instructions to use VS Code for flashing and debugging this project.
I'm working with the extension maintainer to work out the rest of the quirks, but this is ready to start using imo.
@robotoshi and @NuclearTea, can you please follow the usage instructions in the README (in this branch) and make sure they work/make sense on your systems. Flash your dev kit, and verify that the light blinks and you get a heardbeat UART message. Try using the debugging breakpoints also.
Fixes #20 - VS Code setup.

The .extensions file is in the .vscode folder in the repo, not the .vscode folder inside the firmware folder. In my case this meant I  did not get the pop-up to "install recommended extensions" and had to find the extensions manually.

Change the note in about Breakpoints in VS Code because "Flash STM32" runs the program normally and "Debug STM32" runs in debug and obeys breakpoints

Implemented those changes; am merging now.


[openssl] Enable builds with clang-cl

clang always uses /Z7 so i deactivated the PDB install rules. Allowing /Z7 on normal windows requires removing /Zi /Fd from the internal build rules.

@Neumann-A Is this PR ready for review?

@Neumann-A Is this PR ready for review?

Yes.

@Neumann-A Can you confirm that you are OK with the nitpick fix I pushed in https://github.com/microsoft/vcpkg/pull/25869/commits/7fbbf7fe64c88025c2dd9f1e3a2d7f4996e36733 ?

@Neumann-A Can you confirm that you are OK with the nitpick fix I pushed in https://github.com/microsoft/vcpkg/commit/7fbbf7fe64c88025c2dd9f1e3a2d7f4996e36733 ?

LGTM.


[light-client] allow multiple RelayStates to belong to different parentchain types

We have made the light-client generic to support parachains and light client. Currently, we try to initialize the finality verifier dynamically as of #773. However, I made a mistake in my thinking there. It is currently not used yet, but we also want to be able to connect to different chains at the same time. This is why the RelayState looks like this:
#[derive(Encode, Decode, Clone, Debug)]
pub struct LightValidationState<Block: ParentchainBlockTrait> {
	pub num_relays: RelayId,
	pub tracked_relays: BTreeMap<RelayId, RelayState<Block>>,
}

Where each RelayId corresponds to a dedicated chain. Now, as these chains may have different finality algorithms, it is no longer the case that we initialize the light-clients finality type dynamically. Rather, it always needs to support both.
So I suggest:
pub struct RelayState<Block> {
	pub last_finalized_block_header: Block::Header,
	pub penultimate_finalized_block_header: Block::Header,
	pub header_hashes: Vec<Block::Hash>,
	pub unjustified_headers: Vec<Block::Hash>, // Finalized headers without grandpa proof
	pub verify_tx_inclusion: Vec<OpaqueExtrinsic>, // Transactions sent by the relay
        pub finality_state: FinalityState,
}

pub enum FinalityState<Block> {
      Grandpa {
           pub current_validator_set: AuthorityList,
	   pub current_validator_set_id: SetId,
	   pub scheduled_change: Option<ScheduledChangeAtBlock<Block::Header>>
      }
     Parachain {
         /// The finalized relay chain block that includes the last `last_finalized_block_header` of the parachain.
         pub backed_in_relay_chain_block_header: Block::Header
         /// The [`RelayId`] of the [`LightValidatationState`] for the corresponding relay chain.
         pub relay_chain_relay_id: RelayId
     }
}

pub trait IdentifyFinaliy {
    fn finality_type(&self) -> FinalityType
}

impl IdentifyFinality for FinalityState {
     fn finality_type(&self) -> FinalityType {
           match ...
     }
}

pub enum FinalityType {
    Grandpa
    Parachain
}

I'm not sure if I follow here, but I was also not part of the PR discussions. I'll simply place my concerns and apologize if they are baseless:
Every chain needs to have it's dedicated light client anyway, according to our design proposals in our wiki in Light Client Planning:

So instead of making one light client that is usable on multiple chains with different finalization consensuses (if I understood your proposal correctly?) , would it not make sense to first proceed with the proposed light client component, and make it possible to include mulitple light clients where each is able to connect to one parentchain?

Ok, I think this needs a meeting of all developers before we continue here. I did not know/forgot that there was such a proposal.

Closing in favor of the modular light client approach.


[📑 Docs]: Write down all the custom validation rules that are not in JSON Schema

What Dev Docs changes are you proposing?
There is a need to write down what is not in JSON Schema and we wrote custom validation code for it here https://github.com/asyncapi/parser-js/blob/master/lib/customValidators.js so it is transparent and clear what happens if someone decides to validate AsyncAPI document by using directly the AsyncAPI JSON Schema document.
Even our Parser JS is not yet 100% accurate with validation, so https://asyncapi.github.io/tck/asyncapi-parser_js_detailed_report.html should also be treated as source of truth
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The goal of this task should be to update the README.md

add Overview section and explain:

that this repository contains JSON Schema files for all the versions of AsyncAPI specification. You can link to https://json-schema.org/ for reference
that JSON Schema files are not 1:1 reflecting the specification and should not be treated as specification itself but rather as a tool for things like validation for example
that these JSON Schema files should not be used as the only tool for validation of AsyncAPI documents, that some rules described in the AsyncAPI specification cannot be described with JSON Schema

add Custom Validation Needs (not sure if this is the best name for the section) section and explain:

JSON Schema does not cover all validation cases, and if users decide to validate AsyncAPI tools with only JSON Schema provided in this repo, they risk that their AsyncAPI documents may not work will all AsyncAPI tools and not pass validation there.
point out that there is a need for custom validation implementation, what we already do in https://github.com/asyncapi/parser-js and we recommend using it as a validation tool
we need to list what additional custom validations need to be provided:

Server object related validations:

Validate if variables provided in the url property have corresponding variable object defined and if the example is correct
Validate if operationIds are duplicated in the document
Validate if messageIds are duplicated in the document
Validate if server security is declared properly and the name has a corresponding server securitySchemes definition in components with the same name
Validate if given server securitySchemes is a proper empty array when security type requires it
Validate if parameters specified in the channel name have corresponding parameters object defined and if name does not contain url parameters.
Validate that all servers listed for a channel in servers property are declared in the top-level servers object.
Validate if tags specified in the objects have no duplicates. Check is done for: root, operations, operation traits, channels, messages and message traits.

point out that even parser-js do not cover all validation cases yet, and that all test cases and parsers coverage can be found here -> https://asyncapi.github.io/tck/

solved in https://github.com/asyncapi/spec-json-schemas/pull/243 thanks to @Dindihub


What is a workflow to update Backend environments when Web Previews is enabled?

Please describe which feature you have a question about?
When Web previews is enabled on a branch and the option to create a new backend environment for each branch is enabled, what is the workflow to update backend environments? It is not clear to me from the documentation if this new backend environment is suitable to update, and if so, the workflow to do so.
Provide additional details
For example, an amplify project is deployed to github on the main branch, which automatically deploys a frontend environment (main) and a backend environment (prod) in amplify on every commit.
A developer wants to add a feature to the backend to the project, they create a new branch.
git checkout -b new-feature
From here, if the developer wants to add an amplify backend feature, the only environment they have locally is prod (reflecting the environment of the main branch), as the pull request and subsequent ephemeral backend environment has not been created yet. Given that, they enter the following:
amplify add api
Imagine the developer goes through the interactive prompts and chooses the following options:

The developer does not amplify push here, as they do not want to update the production environment, so they commit their changes and push it to github.
git add .
git commit -m 'add api'
git push origin new-feature
Afterwards, they open a PR on github, which Amplify Previews notices and starts provisioning a frontend and backend environment.
The backend environment that Amplify provisions includes the new api changes that were made, and let's say was named prb.
Through practice, here I know that some files are needed to be pulled from the environment and committed to the repository.
The developer then wants to amplify env checkout prb to amplify pull those changes, but the environment does not exist and is met with an error:

Running amplify env list does not list the new backend environment, prb

Side note: , here I tried amplify env pull and was met with unexpected results.  I expected amplify env pull to make no local changes, as the prod environment has not been pushed, but the results are the following:

Here, it appears that amplify env pull in the prod environment pulls down something that refers to a graphQL api, even though nothing was pushed. Strangely, running amplify env pull in a fresh clone of the repository, after an initial amplify init to set it up, did not produce these results, why did the previous amplify env pull pull a change to the previous file?

Continuing on, the developer ends up browsing the Amplify Console to find the backend environment that was created, prb, and uses the amplify pull --appId APP_ID --envName prb command to force their local environment to recognise the ephemeral environment prb:

The developer can now commit the code changes that were produced by the Previews deployment back to the branch:

Unfortunately, this includes adding the prb environment to amplify/team-provider-info.json, which the developer knows will no longer exist once the pull request is merged and the branch deleted, how would the developer avoid this? Should they not commit the change to amplify/team-provider-info.json, or does a new commit need to be made to main after the PR is merged, removing the environment from the file.
Is this the intended workflow when working with web previews?
Should the developer instead work in a sandboxed backend environment locally, so that they can amplify push and get the code changes required in before opening the PR? Meaning that two backend environments is created per PR (the sandbox, and the automatically created one by Previews).
What AWS Services are you utilizing?
Amplify
Provide additional details e.g. code snippets
In summary:

What is the recommended workflow for a team when developing backend environments while Previews is turned on for a github repository?
How can a developer pull changes made by Previews provisioning changes to a new backend environment back into the codebase?
Why does amplify env pull modify local files after an amplify add api and git push to a Previews enabled repository, without amplify push-ing any changes to the original environment the amplify env pull was made in? (And why do we not see this on a fresh clone of a repository).

The App ID is d14291rfs1uwyz and region is ap-southeast-2

So sad to see an incredible effort to explain a basic workflow and get no answer from AWS.
And everyone that comes here hoping to solve the problem and leaves wondering what Nick did to solve that 😞


What about providing a CDN ?

It can help us to keep the latest version possible in sync on our servers and reduce the load while parallelizing.
The guys from CDNJS could do that easily : https://github.com/cdnjs/cdnjs

As mentioned in #158, this is done:
https://github.com/PrismJS/prism/issues/158
So I agree with @apfelbox that we should close this.

@Golmote I think we can close this; #158 has the discussion on releasing a new version out to CDN JS (which is the next step for this)

Yup. Discussion goes on in #158.


Update link to "installing" documentation

The link was broken, probably pointing to an old url.

Thanks.


Working days and other classification

Working days
AbstractProvider::isWorkingDay() returns true for any defined holiday, including e.g. Valentines Day. This is surprising. I belive Valentines Day is a working day for everyone, even the most romantically inclined :-)
I guess all TYPE_OFFICIAL holidays are official non-workdays. But there are also some holidays TYPE_OBSERVANCE holidays that are de-facto non-workdays, at least for some companies/public offices.
For a large number of use-cases for showing holidays in business software, the only holidays of interest are those that affect the business, i.e. when other businesses (banks in particular) are closed. And not so much when people buy flowers to their sweethearts :-)
I know that “working day” is not a boolean property. But I think holidays can be tagged with different fairly objective properties that allow users to get a list of holidays that are relevant for a specific purpose.
Other use-cases include listing all bank holidays or holidays where shops are closed.
I suggest adding a new attribute in addition to the current TYPE_xxx property. This should be a bitmask or tag list that allows multiple flags/tags per holiday.
Currently a holiday can only have one type, so if a holiday is both a public holiday and a bank holiday, it can only be classified as one of them. This means that you cannot get a list of all bank holidays.
The list of flags could look something like this:

Official non-working holiday.
Other official or de facto non-working day, all companies/public offices are closed.
Other official or de facto non-working day, a significant number of companies/public offices are closed.
Official holiday or officially recognised day, working day.
All banks are closed.
All shops are closed.
All schools are closed.
Festive day celebrated by a significant part of the population.

The exact meaning of the words “all” and “significant” should be defined more clearly, e.g. >95% and >30%, respectively.
What do you think of adding such extra data?

Thank you very much for this! It is indeed not correct that isWorkingDay always returns true :(.
When I started with this library, I was already aware that the current types/categories wouldn't be sufficient or cover all situations. The current implementation is simple and would cover the majority of scenarios.
That said, I really like your proposal. Of course when introducing additional data, appropriate helper methods (API) should be present as well. To make it easier for people to use.

This illustrates what I had in mind:
https://github.com/azuyalabs/yasumi/compare/develop...c960657:holiday-tags
I may have gone a bit overboard. On the other hand, as long as the tags are reasonably well-defined, it doesn't take a lot of work for a person with knowledge about a given country to add the relevant tags.
I have tried to think of tags for a variety of reasons, but I have probably missed some. In particular, my knowledge about non-Western cultures is very limited.
The syntax for adding all these tags feels a bit clunky. I guess this is a necessary evil.
What do you think?

Having this would be very beneficial indeed. Is there any progress going on for this suggestion?

A different approach would be to have specific providers for different use-cases. E.g. Yasumi\Provider\Denmark\SchoolHolidays, OfficeHolidays, BankHolidays.
Possibly with provider-specific holiday types, e.g. for Yasumi\Provider\India\ReligousHolidays, type could be TYPE_CHRISTIAN, TYPE_BUDDHIST etc.
Such providers may be easier to implement, because they each have a more specific scope.

The implementation may indeed be more simpler, however the current way feels a bit more natural to me. It simply communicates: 'What are all the holidays in India?', irrespective of the classification.
Creating classes that encompass a specific scope would require a developer to make (coding) decision what class to use.
Maybe not a strong argument, but just my initial feeling :) I would prefer the initial proposal rather.

I would like to help with this. If someone adds the tags, I will go through USA holidays and add proper tag to the each one.

@siburny Thank you for willing to help. Unfortunately this feature is not at the stage where there is a conclusive design nor even a proof of concept. This still needs to be vetted out.

I think this idea of "tags" is spot on. Each holiday date can be assigned one or more tags. However, I do think we should avoid mixing "types" and "tags"; we should stick to one classification system.


Feature Request: Copy screen shot as svg rather than png

Could we make it so that copying the image in obsidian is a svg rather than a png. There are cases when the mind map is big and the copied screenshot is a low res image. It would be best to copy the svg xml code than a plain raster image.

Came here to report this as a bug... The Readme.md for this repository says that Copy Screenshot does copy an SVG to the clipboard.


RFC: Event handler return value will be ignored in 4.0.0

Angular currently supports cancelling an event's default behavior by returning false from an event handler. Unfortunately all event handlers must pay the cost of performing this check whether they return a bool or not. In 4.0.0 we're dropping support for this feature in favor of improving performance and generating less code.
For example, currently placing the following directive on a link will prevent navigation when clicked.
import 'package:angular2/angular2.dart';

@Directive(
  selector: 'inert',
  host: {
    '(click)': 'onClick()',
  },
)
class InertDirective {
  bool onClick() => false; // prevents 'click' event default behavior.
}

From 4.0.0 onward, explicit use of Event.preventDefault() will be required.
import 'dart:html';

import 'package:angular2/angular2.dart';

@Directive(
  selector: 'inert',
  host: {
    '(click)': r'onClick($event)',
  },
)
class InertDirective {
  void onClick(Event event) => event.preventDefault();
}

Closed by https://github.com/dart-lang/angular2/commit/00ac8ddc37f44eed9514a2e65b4f6c0eaf0072a4. Woohoo!
Thanks @leonsenft!


Add -lntdll and ignore warnings on Windows oldrel

A follow-up of https://github.com/extendr/rextendr/pull/285.

Thanks for approving. While there might be some better option to deal with the warning on oldrel, I believe this is better than doing nothing and let the CI keep failing. I'm merging this for now.


No Idea How To Run 0.5.0 With WSL

The documentation says

Neovide supports displaying a full gui window from inside wsl via the --wsl command argument.

However, I have no idea how to make that work. I downloaded both the neovide and neovide.exe 0.5.0 files from https://github.com/Kethku/neovide/releases.

The .exe ran fine in Windows, but doesn't start if I provide a --wsl argument via a shortcut or from the command line.
Running ./neovide --wsl from WSL (Ubuntu) responds with "nvim: Unknown option argument: "--wsl"`.
Running ./neovide.exe  from WSL launches neovide in windows, but it isn't much different from running it from windows directly. All paths are Windows paths, and there is no sense that it is using the "wsl copy of neovim" as the documentation claims.

The instructions leave me clueless. I don't use visualstudio, and I'm not about to install and set up VS to work with WSL just to get an idea about how neovide might work.
I'd like to explore neovide to see if it could work for me, but at this point I don't see how I can?

I'm happy to help you out. The commands have just changed significantly and I'm guessing the docs haven't caught up.
Some tips:

You have to have neovim installed in wsl for it to work. You have to run the neovide exe from the windows side and it will try to start up neovim via the nvim.exe on the wsl side.
If that doesn't work, you can also start up neovim in wsl via tcp socket. There are some details here: https://github.com/Kethku/neovide/tree/opengl#remote-tcp-support although this is currently on the opengl branch. I'm going to fix the CI build so you can download a current build of that tonight.
If all that fails, you could try running with --log to get a log file written out to disk which might give some further hints.

If any of that works let me know, I'd like to update the docs to make it clearer. If not, I've got some other things to try. Please be patient. Its just me and a few other folks building this and supporting mac, linux, and windows is not trivial especially when using a language as new as rust.

Good to know I wasn't missing something!

You have to have neovim installed in wsl for it to work. You have to run the neovide exe from the windows side and it will try to start up neovim via the nvim.exe on the wsl side.

How does neovide.exe know to start up neovim in WSL? Here's what I'm thinking:
I have WSL running, and neovim can be run from the WSL command line (an Ubuntu instance) via "vi", "vim", or "nvim" (there is no "nvim.exe" in WSL to my knowledge). I want to run neovide, say to edit files in my cwd. What do I do? Normally I would simply run vi [filename], or even just vi.
What you said suggests that if I run neovide.exe on windows, it will automagically connect to and run from WSL, but how does it know what directory I'm in or what file I want to edit?

I tried the Remote TCP Support documentation without success as well. I ran the nvim headless command as stated in the documentation, and I ran the remote-tcp option as well (both from WSL and from a shortcut I created with the TCP options). Here's an image from my terminal session running byobu, where you can see both commands:

neovide did not start up when I provided the --remote-tcp option. It starts fine without it (but of course I'm not connecting to nvim in WSL).

Sorry for stringing my replies out; I'm trying your options one at a time.
using the --log option gives me this with the --remote-tcp command:
INFO [neovide::bridge] Close watcher started
ERROR [neovide::bridge] Neovide requires version 0.4 or higher

NVIM is at v.0.4.3, and neovide is at version 0.5.0.
I wonder if part of my confusion is that I don't know what the different released are for? I've been assuming that "neovide" is the linux binary, "Neovide.app.zip" is an iOS binary, and "neovide.exe" is the Windows binary. I don't know if I only need the Windows binary, or if I need something else.

On a hunch, I uninstalled a windows version of neovim I had installed. Now when I run neovide.exe --log, I see the following:
ERROR [neovide::error_handling] Could not locate or start the neovim process: The system cannot find the file specified. (os error 2)
INFO [neovide::window] window created
INFO [neovide::window] renderer created
INFO [neovide::window] Starting window event loop
TRACE [neovide::editor::grid] Editor resized
TRACE [neovide::editor::grid] Editor cleared
TRACE [neovide::editor::grid] Editor cleared
TRACE [neovide::redraw_scheduler] Next frame queued
TRACE [neovide::redraw_scheduler] Next frame queued
TRACE [neovide::redraw_scheduler] Next frame queued
...
DEBUG [neovide::window] Render Triggered
DEBUG [neovide::window] Render Triggered
TRACE [neovide::bridge] UiCommand queued: MouseButton { action: "press", position: (57, 20) }
ERROR [neovide::error_handling] Could not send UI command from the window system to the neovim process.: channel closed

When I click a button the window closes. It doesn't appear to be able to find neovim.

How does neovide.exe know to start up neovim in WSL?

try opening neovide like:
./neovide.exe --wsl

How does neovide.exe know to start up neovim in WSL?

try opening neovide like:
./neovide.exe --wsl

As Kethku mentioned, that doesn't work on version 0.5.0

I think maybe there are some misunderstandings that I need to clear up in the readme.
You mentioned wanting to use vi, or vim on the wsl side. Neovide only supports neovim though.  If it doesn't find one, it will just fail to run, but if it does, it will start up the neovim instance and communicate with it over the windows-wsl boundary via standard input output.
I think the issues you are running into have been fixed in the opengl branch. I've been fighting ci build issues all day, and I think I've gotten it hammered out, but I will update this thread when I have a built exe for sure. I recommend trying that rather than keeping on with the main branch/release builds.

Ok a run finally finished: https://github.com/Kethku/neovide/actions/runs/887728507 I recommend downloading the exe from here.

You mentioned wanting to use vi, or vim on the wsl side. Neovide only supports neovim though.  If it doesn't find one, it will just fail to run, but if it does, it will start up the neovim instance and communicate with it over the windows-wsl boundary via standard input output.

I think I wasn't clear in my previous message when I started talking about using 'vi' and 'vim'. I have those commands aliased to neovim via update-alternatives, and I no longer use vim. neovim is installed in WSL, and it is my daily driver. Sorry for the confusion! Here are more details:
root@DESKTOP:/$ ll $(which vi)
lrwxrwxrwx 1 root root 20 Aug  4  2020 /usr/bin/vi -> /etc/alternatives/vi*
root@DESKTOP:/$ ll /etc/alternatives/vi
lrwxrwxrwx 1 root root 13 Oct 22  2020 /etc/alternatives/vi -> /usr/bin/nvim*
root@DESKTOP:/$ ll $(which vim)
lrwxrwxrwx 1 root root 21 Aug  4  2020 /usr/bin/vim -> /etc/alternatives/vim*
root@DESKTOP:/$ ll /etc/alternatives/vim
lrwxrwxrwx 1 root root 13 Oct 22  2020 /etc/alternatives/vim -> /usr/bin/nvim*

I think the issues you are running into have been fixed in the opengl branch. I've been fighting ci build issues all day, and I think I've gotten it hammered out, but I will update this thread when I have a built exe for sure. I recommend trying that rather than keeping on with the main branch/release builds.
Ok a run finally finished: https://github.com/Kethku/neovide/actions/runs/887728507 I recommend downloading the exe from here.

I tried running the executable from that link, but it exited right away. When I ran it with --log, it created a file with one line in it:
ERROR [neovide::bridge] nvim not found!

As I'm beginning to understand how neovide works, I think a few more details about my setup might be helpful. Here is the output of wsl --list:
> wsl --list
Windows Subsystem for Linux Distributions:
Ubuntu-20.04 (Default)

I hope that helps! I'm running an older version of neovide one where the --wsl option works, and I'm really liking it so far - good job! It's a bit slow to start up though, so I can't use it as a replacement for nvim, only for dedicated editing sessions. I find that the graphics help me confirm when I run some habitual commands, such as :w, because the cursor jumping to the console is evident in my peripheral vision.

Hello.
D:\>neovide.exe --wsl

works great.
Is there an option to set the path to nvim executable?
I'm using both stable(nvim) and nightly(nvim-nightly) version and neovide always runs stable version.

Is there an option to set the path to nvim executable?
I'm using both stable(nvim) and nightly(nvim-nightly) version and neovide always runs stable version.

Please don't hijack this issue @horseinthesky. If you have a question, please start a new issue so that this issue stays on track, and so that others can find help when they have similar issues. It's hard to track an issue in an issue tracker when threads are treated like a forum.
It could be that the issue is more specific than the title suggests, but so far I don't have any reason to think otherwise.

D:\>neovide.exe --wsl

works great.

It still does not for me (I just re-downloaded v0.5.0 and tested it again). Note that it does work on older versions for me, I do have a few questions if it is working for you though:

What version of neovide are you running with --wsl?
What version of linux do you run in WSL?

On a side note, I'm not sure that v0.5.0 is supposed to use the --wsl argument, based on the author's feedback:

The commands have just changed significantly
...
You have to have neovim installed in wsl for it to work. You have to run the neovide exe from the windows side and it will try to start up neovim via the nvim.exe on the wsl side.

Didn't want to hijack the issue. I think my problem also relates to the issue since I'm not able to run wsl NVIM 0.5.0 with neovide.
D:\>neovide.exe --wsl

works with 0.4.3 which I have in /usr/bin/nvim.
My 0.5.0 is located in /usr/local/bin/nvim-nighly. Even if I set alias nvim='/usr/local/bin/nvim-nighly' in my wsl .zshrc, neovide still runs 0.4.3.
This is why I am asking if it is possible to set up an nvim binary path for neovide.

Oh, OK. I think there's some confusion due to an unfortunate conflation of version numbers. I wasn't even thinking about the next release of neovim.
I'm talking about being unable to run neovide 0.5.0. I have neovim 0.4.3 installed in my WSL instance, and neovide quits right away unless I use an older version of neovide with the --wsl option.
I'll try to clarify my original post to avoid future confusion.

Then in terms of NVIM version, we are alike - it is 0.4.3.
I've just checked
neovide (0.5.0):
neovide --wsl

quits right away.
neovide (0.7.0) - Smooth Scrolling Beta:
neovide --wsl

works just fine.

That's interesting to me. What distribution(s) are you using for WSL? Here's mine:
I am on Windows 21H1 (Build 19043.1052) and also Ubuntu-20.04

I'm not sure if it's worth mentioning or not, but I'm running the neovide.exe in Windows, and I don't have the executable in my path (so I have to run it from the directory the executable is in).
The same for me. Just run it from my D:\ disk with some stuff out of the PATH.

On a side note, I have a guess as to why neovide isn't picking up your neovim v0.5.0 install though @horseinthesky. I doubt that neovide starts a shell login to check for nvim, in which case it wouldn't pick up the alias you set up. Instead. you could try a symlink, such that
/usr/bin/nvim -> /usr/local/bin/nvim-nighly

This works cuz (I assume) neovide just looking for nvim in PATH over the windows-wsl boundary via standard input output. It is also probably using the same user as the host (Windows) system.
So would be nice to have an option to give neovide the name of executable or customize the path to it.

So would be nice to have an option to give neovide the name of executable or customize the path to it.

Either that, or we could try to run actual bash including the bashrc in order to get the correct user configured path to neovim. Honestly I don't know how that might work as I don't use linux that often myself. Especially with the recent changes to enable commandline arguments in th eopengl branch, I don't think it would be that hard...

Either that, or we could try to run actual bash including the bashrc in order to get the correct user configured path to neovim.

How are you currently obtaining the WSL neovim executable?

Had the same issue - the trick that helped me - Ubuntu WSL should be default one - check using 'wsl -l'

I'm running into an issue that is similar or identical. I have installed neovim both in the Windows host and in the Ubuntu WSL system. The Windows version is NVIM v0.5.0; the WSL version is NVIM 0.4.3. I suspect that Neovide is finding the Windows nvim even when using the --wsl argument.
With the above setup, when I run neovide.exe --wsl --log, I get the behavior described above, and the log reads:
INFO [neovide::bridge] Close watcher started
INFO [neovide::window] window created
ERROR [neovide::bridge] Neovide requires version 0.4 or higher

When I remove the Windows nvim.exe from the WSL PATH, leaving only the WSL "native" nvim on the path, Neovide opens successfully with the --wsl argument. This indicates to me that as long as nvim.exe (the Windows version) is on the path, that's what Neovide finds when using --wsl, which seems wrong.
However, even when nvim.exe has been removed from the path and Neovide has successfully opened, as soon as I press any key or move the window, it closes, and the log says something like:
TRACE [neovide::bridge] UiCommand queued: Keyboard("a")
ERROR [neovide::error_handling] Could not send UI command from the window system to the neovim process.: channel closed

I'm guessing this should be considered a separate issue, though.

I think this is fixed by https://github.com/neovide/neovide/pull/900. Please reopen if problems continue.


nodejs_demo doesn't work as described on web site; does via readme

On the page https://www.transcrypt.org/docs/html/installation_use.html, there's a description of how to compile and run the nodejs_demo. However, following the instructions there lead to errors. There is a readme.txt in the nodejs_demo directory. It says to use rollup, which bundles the javascript into a CommonJS format which node can run. That necessary  instruction is omitted from the web site version. Also the arguments to the transcrypt command are different; I'm not sure why.
I could offer a PR to fix the docs, but I'm not sure if there's some reason for the different arguments to transcrypt or for not using rollup in the description on the web site. The web site calls transcrypt with:
transcrypt -b -p .none nodejs_demo
whereas the readme.txt uses:
transcrypt -b nodejs_demo

Related to issue #832

Yes, it's the same issue!


Servlet: Filter API

This closes #4142

Hello @ahmadmardeni1 , @hectorkambow  and @BrianaNzivu ? I've made a few changes to this article. Could you please have a look at it?

@BrianaNzivu can we get a status on this PR ?
If any more major revisions are needed - we will be closing this PR asap. 👀

@BrianaNzivu should this PR be reassigned?

@BrianaNzivu should this PR be reassigned?

No, working on it ASAP.

@section-engineering-education/content-moderator-team .
I am kindly requesting a plagiarism check on this article.

Plagiarism check done.

@BrianaNzivu, how are you?I would like to know the progress of this pr.

jumping on this one

@Tumorbenson I believe we are missing your author details.
@BrianaNzivu

@Tumorbenson I believe we are missing your author details.
@BrianaNzivu

Hello @hectorkambow? I have added the author information. Thank you.

@Tumorbenson 👋 .  Please change that ASAP.

Done

@BrianaNzivu
We should have closed this PR - please take a very keen look at future pull requests.

jumping on this one

Hello @hectorkambow , not yet received payment of this pr.


add BCERT svg icon

Add blockcerts BCERT token icon

📝 Description
Add logos for BCERT token - BCERT to ledger. Support for tokens was added to ledger live desktop already
Contract addresses (ethereum)-
BCERT - 0xbb1f919e70ecc55335548d76f5c15a9dc8f5808e
etherscan url: https://etherscan.io/token/0xbb1f919e70ecc55335548d76f5c15a9dc8f5808e
❓ Context

Impacted projects: ``
Linked resource(s): ``

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[ ] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

Hi, any other action we can do to move this forward :) ?

Still no reply on that ? How can we move this further ?

hi :wave: in order to integrate BCERT icon to ledger live could you reduce the number of paths of your icon to a minimal number. keeping it as plain as possible. All the paths will be filled with the same color in the end so if you need to work with negative space you need to merge the paths together keeping the difference.

hi wave in order to integrate BCERT icon to ledger live could you reduce the number of paths of your icon to a minimal number. keeping it as plain as possible. All the paths will be filled with the same color in the end so if you need to work with negative space you need to merge the paths together keeping the difference.

Is there any software that could help doing that ? We're having hard time getting that solved.

All done what expected, thanks for the clarification.

Isn't a changeset required before merging ?

Bypassing branch protections, the tests are failing because the runners are different for a fork


Added includeCommon, includeLocal, includeRemote parameters for mirakle plugin extension to assign more precise rsync filtering rules set

Hi, thanks for your great plugin! I have a small pull request to offer. It allows to add --inlcude commandline arguments to rsync command call along with the --exclude arguments.
What problem does it solve?
I have made a small repository with scripts that allow you to seamlessly move building process of an apk from a local machine to a remote server (https://github.com/LuigiVampa92/RemoteAndroidBuilds). It uses an optimized set of rsync params that allows to pass just necessary source code from a local machine to a remote server and pass result apks back from server after the build. It also passes generated classes (build/generated) so Intellij Idea code completion and code navigation features would be able to find them and not show any errors if you build project remotely. Still the basic idea is to pass as little data as possible between the machines because in a huge project network throughput can become a very real time-consuming bottleneck.
My scripts used to work great but at a certain AndroidGradlePlugin update (7.0.2 if I figured it out right) Google decided to implicitly split the process of building an apk depending on whether it is build by calling gradle command directly or pressing the RUN button in Android Studio UI. This change broke my scripts. Historically we had a single place inside the gradle project's build directory where apks were stored - build/outputs/apk. No matter which way project was build all apks were dropped there. Now AGP uses standard build/outputs/apk output if it was built by a gradle command and another dedicated directory if it was built by pressing RUN button - build/intermediates/apk.
This is not only confusing and make apks be stored in two different locations at once, second directory is technically not a part of the result output directory, but rather a part of intermediate build files that in my scripts were completely excluded from passing from a remote server to a local machine. In a small project passing that directory is OK, but in a huge project that I have been working with it could easily be from 1 to 2 GB. And all those files have nothing to do with a result artifact that will be run and tested on a device. The build/intermediates directory contains a lot of subdirectories (and they can change with AGP updates) and it is not easy to manually exclude all of them except build/intermediates/apk that is now absolutely required to push an apk to a device or an emulator after build is complete.
Since my scripts does not originally download anything from build/intermediates, when I press RUN button nothing happens and I get an error. If I manually run a gradle command and then adb commands to upload an apk and run the app, everything works, but it is not convenient, because the whole idea was about setting up the remote builds once and forget the commandline forever, just press UI buttons and everything should work seamlessly.
Solution
So the right solution would be to make a set rules for rsync where you first explicitly include what subdirectories you need and then exclude the rest. This plugin already has an extension that allows to conveniently add --exclude rules in a gradle script, but not --include rules. So I added them in a similar way to exclude rules.
With include rules this problem about second apk directory is solved easily this way:
excludeRemote += ["**/build/intermediates/**"]
includeRemote += ["**/build/intermediates/apk/***"]
includeRemote += ["**/build/intermediates/apk_ide_redirect_file/***"]

This changes let me set precisely what I need to download from remote build/intemediates directory to run a build with RUN button, and skip other heavy stuff.
Summary
This PR does not alter any of the current behaviour, only adds an extra option to set rsync filters more precisely. Include rules will not certainly be a very popular option because they are most likely not needed for simple projects, but will certainly be beneficial for those who work with huge projects and that's what is when this plugin is most useful.
I have built and tested the changes locally and they works great.
NOTE! I haven't raised plugin version in this PR. It is still 1.6.0, so you will have to manually make a commit to raise plugin version before publishing an update.
I hope you'll merge this and I'll be able to fix my scripts and keep using your great plugin.
Thank you!

Really needed.
APK doesn't download from remote host.

Yeap, this pull request will solve the problem you mentioned. You can cherry pick this commit and build the plugin for your local maven repository on your local computer, but that is not very convenient.
Btw, if you are interested I am currently making an Android Studio plugin that integrates Mirakle build plugin into UI (actually a fork of it, with some extra features and a few fixes to add support of windows). It adds a toggle button on a build toolbar where you can switch "build mode" between local and remote, no special scripts required, no changing the code, no adding build plugins etc - just push "remote builds on" button and hit "Run" button on a toolbar - the build will be performed on a remote server.
Here is the GitHub repo. It's empty for now, but I will upload the source code in a couple of days, when I will finish the readme and the plugin will be published in jetbrains plugin repo


Update US19JanTaxR1.md

federal

Docs Build status updates of commit c320de8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/payroll/US19JanTaxR1.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fcf2057:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/payroll/US19JanTaxR1.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Symphony 2.6: Remove calendar widget

Due to timezone problems (see #2312), this pull request removes the new calendar widget again.
It should be fixed and reapplied in https://github.com/symphonycms/symphony-2/tree/calendar (Symphony 2.7, if possible).

Am I right in saying that the only reason we are removing this is because of the Feature Request to adjust the start of the week? This FR led to incorrect calendar rendering when combined with a system language that is not English.
Is it possible to just revert the FR and leave the calendar as is? Do the timezone issues affect the calendar otherwise?

No, the feature request has been added and is working as intended. The problem is that the calendar shifts days one weekday back depending on your timezone.
This issue was found while adding the week offset but it exists indepently.

Can you provide a exact set of steps to replicate?

There aren't any specific steps, it's more the general setup:

German OS (in my case Mac OS X Yosemite)
German locales in the browser (in my case Firefox 35 with Deutsch [de] as primary language)
German date and time (in my case CET which is UTC+1)

In this given setup, using the calendar will result in the first January starting on the wrong day (Wednesday instead of Thursday).

Ok, well I think this may be fixed then:

German Timezone in OS (Europe/Berlin)
German Timezone in Symphony (Europe/Berlin)
German Language set in Firefox/Chrome (de)

Creating a new entry, and flipping the calendar back to Januar 2015 results in the 1st being on Donnerstag, which is correct. Care to give integration a try and see if you can replicate?

Ah that's no good!
—
Reply to this email directly or view it on GitHub
https://github.com/symphonycms/symphony-2/pull/2325#issuecomment-72434117
.

I just checked and I can confirm that the calendar displays date/weekday connections correctly.

Ah that's no good! Hope you're still getting some sleep :)

Max is sleeping most of the time, so we get sleep as well. Nothing serious.

Revisiting this issue, I ask myself how your changes to the to the calendar script might have fixed the issue: it should work even without these changes.
I've got an idea (and I'd like @animaux to check this as well): Is it possible that the turn of the year fooled us and that we looked as January 2014 which of course started on a Wednesday.

This pull request has been rendered obsolete by https://github.com/symphonycms/symphony-2/commit/57fda1f7e7b3ccd8a472f56747d60b82b03e29ec#commitcomment-9540637.

@nilshoerrmann

Is it possible that the turn of the year fooled us and that we looked as January 2014 which of course started on a Wednesday.

Definitely not. I double- and triplechecked.


New version: AdalmPluto v0.1.5

Registering package: AdalmPluto
Repository: https://github.com/JuliaTelecom/AdalmPluto.jl
Created by: @RGerzaguet
Version: v0.1.5
Commit: 404c3a19a04579fcbbc5c7fcb9879b6193daa44e
Reviewed by: @RGerzaguet
Reference: https://github.com/JuliaTelecom/AdalmPluto.jl/commit/404c3a19a04579fcbbc5c7fcb9879b6193daa44e#commitcomment-50964053

@RGerzaguet please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.  In particular, I believe you have a typo in your Project.toml, I think
julia = "1.5,1,6"

should be
julia = "1.5"

[noblock]

Oh yes, my bad I have not seen that.
Thanks for the feedback it should be ok right now.
[noblock]

@RGerzaguet note that
julia = "1.5, 1.6"

is identical to
julia = "1.5"

:wink:


MQTT json attributes

Description:
Based on #10753.  Allow MQTT sensor to process json values as attributes.
Related issue (if applicable): fixes
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4334
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Is it possible to read nested JSON into attributes?  With the RESTful sensor, if the json_attributes key contains JSON it creates an attribute for each nested key. See this post

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


Releasing the 2020.4 SDK [Will merge only after 2020.4 ships]

Will update the master branch with the changes https://github.com/tableau/connector-plugin-sdk/pull/658/files , after this MR is merged post the tableau 2020.4 release. Will also send out another MR for gh-pages merging and will post a blog post of the release updates in the end

Hi, @jainam1995! Thanks for opening this PR!

In the file connector-packager/connector_packager/xsd_validator.py:

Line 168:121: E501 line too long (129 > 120 characters)
Line 165:67: E231 missing whitespace after ','
Line 164:55: E231 missing whitespace after ','

In the file connector-packager/tests/test_xsd_validator.py:

Line 99:121: E501 line too long (123 > 120 characters)

In the file tdvt/tdvt/tabquery.py:

Line 44:1: E302 expected 2 blank lines, found 1

In the file tdvt/tdvt/tdvt.py:

Line 729:41: E201 whitespace after '('
Line 729:10: E117 over-indented
Line 729:10: E111 indentation is not a multiple of four
Line 727:40: E201 whitespace after '('
Line 718:12: F405 'tabquerycli_exists' may be undefined, or defined from star imports: .tabquery
Line 717:1: E302 expected 2 blank lines, found 1
Line 499:121: E501 line too long (169 > 120 characters)
Line 498:154: E241 multiple spaces after ','
Line 498:121: E501 line too long (169 > 120 characters)
Line 497:121: E241 multiple spaces after ','
Line 497:121: E501 line too long (136 > 120 characters)
Line 496:121: E501 line too long (192 > 120 characters)
Line 495:5: E265 block comment should start with '# '
Line 432:136: W291 trailing whitespace
Line 432:121: E501 line too long (135 > 120 characters)

In the file tdvt/tdvt/tdvt_core.py:

Line 712:87: E251 unexpected spaces around keyword / parameter equals
Line 712:85: E251 unexpected spaces around keyword / parameter equals
Line 712:1: E302 expected 2 blank lines, found 1
Line 47:1: W293 blank line contains whitespace
Line 42:121: E501 line too long (133 > 120 characters)

For reference: Error Code Reference, PEP8, Python code style

Let's pull in master then take a look again, looks like some old code is in here.
Pulled in the master branch to 2020.4 : https://github.com/tableau/connector-plugin-sdk/pull/659/commits/5391e46e10a64ac83dce211cca89b0a3254c9826

Let's pull in master then take a look again, looks like some old code is in here.

Already done

We need a minor version bump for TDVT along with all the other changes that go with that (changelog, readme)
Also can you see if your changes work when the tdvt path is specified with envvar rather than in a config file? Looking at the code, this may not work for run_file either

Made the changes


Transition from CircleCI to GitHub actions entirely

This is a draft of a first version, that replicates (presumably) all current Zenith CI functionality except coverage reports.
Coverage file reports are generated, but fail on merging for some reason, complaining on the unknown format, I plan to either investigate it or create an issue for follow-ups later.
Also extends macOs set of tests with the python ones:

other-tests suite is disabled currently, since they get completed up to ~94% and get stuck on one of the tests
the rest of the tests surprisingly pass, albeit benchmarks being slow and flacky

IMO, before starting using this approach, we need to boost the runners: the ones hosted by GitHub are even weaker than those from CircleCI.
@cicdteam had plans on custom runners which we can discuss.
More context: https://neondb.slack.com/archives/C036U0GRMRB/p1648838481452179

Cool! .github/config.yml is now obsolete, right?

It will be when the coverage will be ported: I'm working on it, but somehow the runner now just gets killed for not reason? Trying to look into that.
Otherwise, yes, this file will be removed if we decide to move onto GH Actions.


Homeassistant Proxy Addon [FATAL tini (7)] exec /app/entrypoint.sh failed: Exec format error

Describe the issue you are experiencing
Hi,
i've installed the HomeAssistant Proxy Addon for Raspberrymatic.
i just installed, set the ip address of Raspberrymatic and started the addon.
Log File:
[FATAL tini (7)] exec /app/entrypoint.sh failed: Exec format error
:)
Describe the behavior you expected
Addon should be started
Steps to reproduce the issue

Install Raspberrymatic Proxy Addon on Homeassistant
Start Addon

...
What is the version this bug report is based on?
Homeassistant 2024.11.2
Which base platform are you running?
ova (Open Virtual Infrastructure, AMD64/x86_64)
Which HomeMatic/homematicIP radio module are you using?
RPI-RF-MOD
Anything in the logs that might be useful for us?
[FATAL tini (7)] exec /app/entrypoint.sh failed: Exec format error

Additional information
No response

Sorry, but that new Proxy addon is not yet ready. That's why it is flagged as "experimental". Thus, please wait until is it done and properly announced and distributed...

no problem, is see the flag :)

So, now the 0.3.0 version of the RaspberryMatic Proxy HA-Addon should work as expected and has left its experimental status. So please give it a try and enjoy to link your RaspberryMatic CCU into HA...


unused argument in batch_processor()

https://github.com/open-mmlab/mmdetection/blob/cd4e716b0470bde842c20c00f18cffa20c2673c1/mmdet/apis/train.py#L59
train_mode is not used.

Yes it is currently a reserved argument.


chore(resources): rename service-info to metadata

Description of change
Rename shuttle-service-info to shuttle-metadata since it will contain more information than only the service in the future.
How has this been tested? (if applicable)
Na.

I didn't rename the internal types as requested @jonaro00

We have a regression in main regarding a clippy lint, maybe you can address that with the PR too, to have a green CI: https://app.circleci.com/pipelines/github/shuttle-hq/shuttle/3891/workflows/d88406e0-d280-44ba-9f95-d1f8f7420677/jobs/80954

@iulianbarbu addressed in 566b50ab79cb5ef5a16d60f47778f67a3ea5f6a6


[zh-cn] sync 5 files in /task/administer-cluster

Sync these files with en page:
content/zh-cn/docs/tasks/administer-cluster/controller-manager-leader-migration.md
content/zh-cn/docs/tasks/administer-cluster/coredns.md
content/zh-cn/docs/tasks/administer-cluster/cpu-management-policies.md
content/zh-cn/docs/tasks/administer-cluster/developing-cloud-controller-manager.md
content/zh-cn/docs/tasks/administer-cluster/dns-custom-nameservers.md

Hi, thanks for your contributions, here are some tips:

Provide a title that can be easily identified and found when some contributors are searching in the PR list
You may need to confirm all differences have been synced in those 5 files

@windsonsea 我想的是改动不大，就没有单独提，是不是这种改动小的，可以暂时先不管？

/lgtm
/approve