apps/btshell: add bletiny clone that uses new shell

Features improved command naming and tab completion functionality.
In the future it could replace bletiny. Later some commands
implementations could me placed inside respective packages and added
to shell as separate modules, e.g. l2cap, gatt, gap, etc.

Hi, Chris. Thanks for the review. Fixed the things you mentioned.
Of course, the documentation will need to change if btshell replaces bletiny. For now, here is the mapping between bletiny and btshell commands:
adv     -> advertise
chrup   -> gatt-notify
conn    -> connect
datalen -> conn-datalen
disc    -> gatt-discover-{characteristic,descriptor,service,full}
find    -> gatt-find-included-services
l2cap   -> l2cap-{update,connect,disconnect,create-server}
mtu     -> gatt-exchange-mtu
passkey -> auth-passkey
read    -> gatt-read
rssi    -> conn-rssi
scan    -> scan
sec     -> security-{pair,start,encryption,set-data}
set     -> set{-adv-data}
show    -> gatt-show{-local,-conn,-coc,-addr}
store   -> keystore-{add,del,show}
svcchg  -> gatt-service-changed
term    -> disconnect
tx      -> test-tx
update  -> conn-update-params
wl      -> white-list
write   -> gatt-write


Microsoft Teams integration unclear

In https://github.com/UtrechtUniversity/getting-started/blob/main/docs/third-party-applications.md it is unclear whether to use the Teams GitHub or GitHub enterprise integration. After messing around and polluting a channel with many useless config messages (all the people in my Teams are confused) it's still not clear to me.
Should we use Teams' GitHub Enterprise connector (seen from ... next to channel name, Connectors) or the GitHub teams integration (which I can no longer find).
When I try to configure the GitHub Enterprise, it asks me to configure the repo itself to post to a certain webhook and I can't do this because I don't have permissions. I couldn't find how to use the organization's webhooks.
When trying to configure the Github app, I could subscribe to updates from within the chat with the Github bot, but not from within the channel of interest. (When I tag @GitHub subscribe UtrechtUniversity/clumped-processing to connect, it in stead creates a popup to create a GitHub issue from within Teams).

Thanks! For now it is possible to use: https://teams.github.com/
If you would like to use any other integration, let us know.
The teams.github.com application will create a github chat channel in Teams. Here you can subscribe to a repository  (e.g. subscribe UtrechtUniversity/getting-started. You will have to authorize the app for the repository via a pop-up window. After subscribing you will receive notifications in Teams about changes in the Github Repo. We will update the documentation to further clarify this.


docs(error/btstrpd): Additional scenario you get the error added

additional scenario when we could get this error is "You can also get this error when the path to templateURL for a route in any of the $stateProvider.state() calls was wrong."

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Fix/Feature: Docs have been added/updated
[ ] Fix/Feature: Tests have been added; existing tests pass

Other information:

Hi, I don't think we should add this as $state is not a core AngularJS feature (it's part of ui router).
We could add  a general note that this can happen if you include a template / templateURL that contains another angular app.

We could add a general note that this can happen if you include a template / templateURL that contains another angular app.

So this does actually result in the specific error? As inside the same angularjs application, the error is different indeed. See: https://plnkr.co/edit/q97QLSMB853Vxi5FDwnX?p=preview

@frederikprijck well, uiRouter is not ngRoute, so mileage may vary. What could happen is if you leave the templateUrl part empty, in ngRoute it downloads the current page which inserts another ngView .. and causes an infinite loop. Maybe uiRouter does something else and causes the already bootstrapped error. However, let's wait for @dasariramacharan to respond.

I'm going to close this PR because we cannot merge it in its current state, as we are either waiting for more information, tests are missing, or required code changes haven't been made.
You can still leave a comment with updates in this PR or open a new PR if you have remedied the issues in the discussion.


fix: #1237 better detection of function call

Ref #1237

PR itself is good, need to merge it manually into 1.9 instead of 2.x

Thank you @funivan.


Add pgAdmin to PsotgreSQL

Microsoft Reviewers: Open in CodeFlow

@MarNicGit can you take a look? The error identifies a preexisting root,PostgreSQL but it is not available in master yet. This is adding a second folder to that root. My assumption is the root is getting flagged as a dupe when this should be a subfolder, not a new root. Here is the reference https://github.com/microsoft/winget-pkgs/pull/1728


iGNN: All menus in the sidebar are closed as a default.

All menus in the sidebar are closed as a default. Whenever anything in a menu is changed, it closes automatically and you have to open it again to keep changing things.
A change in the menus should not cause the menu to close.
Same problem as #6

The close/open settings are set in pn.VCard. I've elaborated both settings, but I do agree that the behaviour you describe is annoying. For settings that the user is unlikely to change, e.g., map settings, this is probably fine, but less so in other cases.

Ive added a general feature request for this as it is true for the entirety of the website. #55


Use codevise/teaspoon for logging backtraces on console

Coverage remained the same at 99.058% when pulling d07461c7ed089faa97984142adcb4dffd5ab7790 on aviav:use-codevise-teaspoon into 486cf6a79b3b5956384a541541199528828c5ea0 on codevise:master.

Coverage remained the same at 99.058% when pulling d07461c7ed089faa97984142adcb4dffd5ab7790 on aviav:use-codevise-teaspoon into 486cf6a79b3b5956384a541541199528828c5ea0 on codevise:master.


Remove the special handling for nameddests that look like standard pageNumbers

PR #7341 added special handling for nameddests that look like pageNumbers, to prevent issues since we previously incorrectly supported specifying a pageNumber directly in the hash; i.e. #10 versus the correct #page=10 format.
Since this behaviour wasn't correct, PR #7757 fixed and deprecated the old format, which means that we no longer need to maintain the nameddest hack in multiple files.

/botio unittest

/botio-linux preview


Fail to install zlib

Describe the bug
Fail to install zlib
Environment

OS: [Windows]
Compiler: revision

To Reproduce
vcpkg install zlib:x64-windows
Expected behavior
A clear and concise description of what you expected to happen.
Failure logs
-- Downloading https://github.com/madler/zlib/archive/v1.2.13.tar.gz -> madler-zlib-v1-44b834fb.2.13.tar.gz...
error: https://github.com/madler/zlib/archive/v1.2.13.tar.gz: WinHttpReceiveResponse failed with exit code 12002
Additional context
zlib file url is
https://github.com/madler/zlib/releases/download/v1.2.13/zlib-1.2.13.tar.gz
NOT
https://github.com/madler/zlib/archive/v1.2.13.tar.gz

I can't reproduce this issue in my side. I could download the https://github.com/madler/zlib/archive/v1.2.13.tar.gz successfully.
Also, I can download both urls you mentioned successfully.
zlib-1.2.13.tar.gz
I think this may be caused by your network problem, as a workaround, please download the attachment and save it into vcpkg\downloads folder.

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Adding kubernautic

@milindchawre thanks for this contribution, can you please include what is included in the free trial?

@salaboy done, updated the PR.

@milindchawre awesome thanks!


missing a "s"

pip install -r requirements.txt

@qiaoyan thanks. fixed!

@qiaoyan thanks. fixed!


HA installation problems

Checklist

[X] I have installed the latest release (or BETA) version of the integration and home assistant.
[X] I have prepared DEBUG log output (for technical issues) | In most of the cases of a technical error/issue I would have the need to ask for DEBUG log output of the integration. There is a short tutorial/guide 'How to provide DEBUG log' here
[X] I confirm it's really an issue | In the case that you want to understand the functionality of a certain feature/sensor Please be so kind and make use if the discussion feature of this repo (and do not create an issue) - TIA

Add a description
I tried to add a new addon repository, but it fails (please see picture)

Add your DEBUG log output
Logfile:
Logger: homeassistant.components.hassio
Quelle: components/hassio/websocket_api.py:135
Integration: Home Assistant Supervisor (Dokumentation, Probleme)
Erstmals aufgetreten: 17:24:17 (2 Vorkommnisse)
Zuletzt protokolliert: 17:27:38
Failed to to call /store/repositories - https://github.com/marq24/ha-evcc is not a valid add-on repository

... well
this is an integration and not an addon

Sorry, ich bin schaffe es nicht. Wenn ich eine Github Integration hinzufüge, dann will er sich mit meinem Account verknüpfen. Könntest Du bitte etwas Hilfestellung geben? Besten Dank!

https://github.com/marq24/ha-evcc?tab=readme-ov-file#installation


Not able to access StringValue, IntValue, TemplateValue but all the details will be now come under the value section.(Version 1.3.4)

Hello @SeppPenner ,
Is there any enhancement or specific reason for not giving the data in IntValue, StringValue, or any other type? (It is properly working before the version release of 1.3.4)

According to the below SS is not able to access the StringValue,IntValue, etc.:

Basically, the internal metrics have changed in version 1.3.4 which means that there is only Value that is filled according to the data type specified. So, if you input / receive a metric with type String, the value will be a string value, but its generic type is object for convenience issues / simplification.
You can try metricValue.Value as string stringValue or something like
if (metricValue.Value is string stringValue)
{
    // Do something with the string value from 'stringValue'
}

This is not a bug, but intended behaviour as I wanted to have a generic API that works with all data types equally.

Thanks for the information @SeppPenner .
I am confused as it is not mentioned in the release version of 1.3.3 and 1.3.4.
(https://github.com/SeppPenner/SparkplugNet/releases/tag/1.3.4)


Eliminate $ from identifiers

Hi there,
I'm currently using lest to test a project of mine, and noticed it was producing some warnings when compiling tests for my project using clang (3.6 and 3.7):
/home/travis/build/genbattle/dkm/src/test/lest.hpp:121:41: error:
      '$' in identifier
      [-Werror,-Wdollar-in-identifier-extension]
  ...$section = 0, $count = 1; $secti...
                               ^
/home/travis/build/genbattle/dkm/src/test/lest.hpp:121:52: error:
      '$' in identifier
      [-Werror,-Wdollar-in-identifier-extension]
  ...0, $count = 1; $section < $count...
                               ^
/home/travis/build/genbattle/dkm/src/test/lest.hpp:121:60: error:
      '$' in identifier
      [-Werror,-Wdollar-in-identifier-extension]
  ...= 1; $section < $count; $count -...
                             ^
/home/travis/build/genbattle/dkm/src/test/lest.hpp:121:73: error:
      '$' in identifier
      [-Werror,-Wdollar-in-identifier-extension]
  ...< $count; $count -= 0==$section++ )

Clang is not happy about dollar characters $ being used in identifier names. This seems reasonable given the discussion here. Using such characters outside of the normal alphanumeric character set is implementation defined in C++11, and so may or may not work. On clang with -Wall -Wextra it happens to generate a warning.
Use of $ in identifiers should be removed if it can be without seriously impacting the readability or clarity of the code.

Thanks Nick,
I've become aware of $ being non-standard after already using it. It helps if someone else starts to mention it :)
I might change the names to:

lest_env
lest_count
lest_section

The 'problem' is that the user may not use these names for her own purpose.
Any other ideas?

Global namespace clashes are a difficult problem so solve with macros. Prefixing the names with lest_ seems like a reasonable compromise.

As an aside:
For some time lest does contain :
#ifdef __clang__
# pragma clang diagnostic ignored "-Wdollar-in-identifier-extension"
...

@genbattle Ah, you're using lest version 1.24.1, current is 1.24.5.
1.24.2: This release fixes and suppresses several warnings for clang and gcc and fixes the version number for VC14 and higher.

I tried the latest version and it does indeed resolve all of my issues! Sorry for the trouble.

You're welcome.
Nonetheless it may be a good idea to stop using the $character.

what about lest_internal__ as a prefix, very unlikely that someone else would use that too...

Thanks @Kosta-Github,
I chose to 'encode' the $ as lest_ and internal as _, obtaining somewhat shorter and easier to read names, such as lest__section.
The lest environment parameter named $ becomes lest_env as this may be of use to the user.


new logo

Coverage decreased (-0.2%) to 96.809% when pulling b40307ed341f21178108f0ae1eba841ab8f990fa on feature/new-ascii-logo into eaf3c827888797e5d40649d9f04af7685f5c0ba4 on master.

Coverage decreased (-0.2%) to 96.774% when pulling b6e94d3ab332d2602474e1774fbc990653ac6212 on feature/new-ascii-logo into 16869fcd5dca55192114b45759a325b605f1ad72 on master.


UserInfoValidator: print value that caused the failure

PR: https://github.com/NationalSecurityAgency/skills-service/pull/564

:+1:


RetroSyn data should include reaction types to allow for combined data split of reactants, product, reaction type

Current situation: I can load data for retrosynthesis with both reactants and product:
from tdc.generation import RetroSyn
data = RetroSyn(name = 'USPTO-50K')

I can also load the reaction types (labels, classes) for this data (presumably in the same order):
from tdc.utils import get_reaction_type
reaction_types = get_reaction_type('USPTO-50K', output_format='df')

I could combine these two data manually, but then I would lose the data split functionality, since it is (presumably) only applied to the original RetroSyn data. So, this sample code does not split/include the reaction types:
split = data.get_split()
train_data = split['train’]

Feature request: I would like (to merge) a dataset with reactants, product and reaction type and then apply the split functionality to this dataset.

Hi @hogru thanks for the feature request! we will work on it asap.

Hi it is resolved in: https://github.com/mims-harvard/TDC/commit/e62c890141c321f2f2433dfd5594ffc1ae4fb166
Now you can add split = data.get_split(include_reaction_type = True)

Well, that was super fast; will experiment with it this week. Thank you!


Migrating to Gatsby 4

It's been a while since Gatsby v4 released. Seems like there are a lot of new features in the new version.
I should take some time to:

Publish a new version that is compatible with Gatsby v4
Explore the new features and see if this theme can utilize them (like DSG)

Known issues:

[x] gatsby-remark-wiki-link doesn't convert double brackets into links, probably a gatsby-plugin-mdx issue . Temporarily fixed by changing plugin export, see https://github.com/hikerpig/gatsby-project-kb/pull/37/files#diff-c3254fcb81fff9e3b77cc30fdbbd3e6216f355adc1b5df1f33ec2963d115a218R128

Thanks for starting this. I'm planning to migrate my site to use this template, but I'll hold for when this is done. I'm not very familiar with the Gatsby ecosystem, but I'll try to contribute if possible. Again, thanks for the resource.

@aadimator Actually I've published<EMAIL_ADDRESS>some time ago and tested in my own wiki site, it is compatible with Gatsby 4 and works just fine for me.
But I don't have the absolute confidence to publish it for stable channel.  Besides that, there are some breaking changes in Gatsby 4 that I may need to address in my release log. I don't have many time currently so this kind of stalled.
Speaking of the template, do you mean foam-template-gatsby-kb ?

Yup, the foam-template.
Thanks for the quick response. I'll try to follow your wiki site config and set up my notes using that. I will let you know if everything goes smoothly. Thanks


Empties with children are duplicated when exporting selected

It turns out that empties, and also point lights, get duplicated when objects are parented to them. The issue only happens when the "Export selected only" is enabled. Both have the exact same transformation definitions, extra properties, and name. The only difference is that one lists children and the other one does not. I could not find anything in the specification to indicate that this is intentional.
"nodes" : [
    {
        "mesh" : 1,
        "name" : "Cube",
        "translation" : [
            -1.9036643505096436,
            -4.57893180847168,
            -1.1928699016571045
        ]
    },
    {
        "mesh" : 0,
        "name" : "Cube.001",
        "translation" : [
            -1.9036645889282227,
            -4.57893180847168,
            -1.192870020866394
        ]
    },
    {
        "children" : [
            0
        ],
        "name" : "Empty",
        "translation" : [
            1.9036645889282227,
            4.57893180847168,
            1.1928701400756836
        ]
    },
    {
        "name" : "Empty.001",
        "translation" : [
            -2.578963279724121,
            3.178974151611328,
            -0.6574060916900635
        ]
    },
    {
        "children" : [
            1
        ],
        "name" : "Empty.002",
        "translation" : [
            5.122903347015381,
            3.958202838897705,
            1.3997524976730347
        ]
    },
    {
        "name" : "Empty",
        "translation" : [
            1.9036645889282227,
            4.57893180847168,
            1.1928701400756836
        ]
    },
    {
        "name" : "Empty.002",
        "translation" : [
            5.122903347015381,
            3.958202838897705,
            1.3997524976730347
        ]
    }

Hi! How recently did you install or update the exporter? @emackey submitted a fix July 11 that may have solved this.


[MRTK Shell Parity] [Pointer] Tether line visual fit & finish

Overview
Adds a grab pointer to tether line, updates default materials & width for tether line and default pointer line, updates default pointer line to be less wobbly.
Part of #4200 UX Controls Parity with HoloLens 2 Shell
Changes

Fixes: #4407

/azp run mrtk_pr

/azp run


LBType field for AWS in infra cluster object.

In the infra config cluster object we will need to add api for LBType for AWS which will store and persist the information of the desired LB type mentioned by the installer config API so that the ingress operator will refer the LB type from this object whenever someone accidentally deletes the ingress controller. Currently, the default LB type i.e CLB/ELB gets set if you delete the ingress controller object even when the desired the LB type for default ingress controller was set to NLB as ingress operator does not have a way to check the LB type as we don't store it.
Enhancement Proposal - https://github.com/openshift/enhancements/pull/1148

/assign @rfredette

@rfredette PTAL. I need to get this merged ASAP as the epic has high priority. Thanks in advance!

/lgtm

@JoelSpeed PTAL. I have made the changes you asked for.

@JoelSpeed based on our discussions it looks like the right place is ingress cluster object and not the infrastructure cluster object.  The default ingresscontroller can have either an internal or external LB.  However, The API uses a separate load balancer.
As you mentioned ->
"Should this be on the infrastructure object? Infrastructure to me implies things around the control plane, so i'd expect the internal and external control plane load balancers to be here, but, if this is a workload load balancer, maybe it's better suited somewhere else?"
over here https://github.com/openshift/enhancements/pull/1148#discussion_r932289883
I think ingress cluster object seems to be the right place.
I have made the necessary changes. PTAL.
Copying @sadasu from the installler team as she had voted for this field to be added in the infrastructure object.

/retest

Were there arguments already made for why this would be on either the ingress or infrastructure objects? Keen to see what the other perspectives were
If this is going onto the ingress object, we will want to add a PlatformSpec style discriminated union to house the data
spec:
  platformSpec:
    platform: AWS
    aws:
      loadBalancer:
        defaultType: NLB

Were there arguments already made for why this would be on either the ingress or infrastructure objects? Keen to see what the other perspectives were
If this is going onto the ingress object, we will want to add a PlatformSpec style discriminated union to house the data
spec:
  platformSpec:
    platform: AWS
    aws:
      loadBalancer:
        defaultType: NLB

No there were not arguments made as such. But as you also mentioned that "Infrastructure to me implies things around the control plane, so i'd expect the internal and external control plane load balancers to be here, but, if this is a workload load balancer," the right place will be the ingress object as this is for workload balancer.   The default ingress controller can have either an internal or external LB.  The API uses a separate load balancer.
I will add the changes you suggested.

@JoelSpeed  PTAL

Were there arguments already made for why this would be on either the ingress or infrastructure objects? Keen to see what the other perspectives were
If this is going onto the ingress object, we will want to add a PlatformSpec style discriminated union to house the data
spec:
  platformSpec:
    platform: AWS
    aws:
      loadBalancer:
        defaultType: NLB

@JoelSpeed No there were not any arguments made as such. There was one comment from @sadasu over here openshift/enhancements#1148 (comment) . But as you also mentioned that "Infrastructure to me implies things around the control plane, so i'd expect the internal and external control plane load balancers to be here, but, if this is a workload load balancer," the right place will be the ingress object as this is for workload balancer. The default ingress controller can have either an internal or external LB. The API uses a separate load balancer.
I will quickly add the changes you suggested.

@JoelSpeed PTAL. We have a google where we had asked where to place this field. But we were asked to ask the installer team.  https://drive.google.com/file/d/1NcbZHpRWyqIvMN4Bxv6MLqVxUg_IN8ba/view?t=22m15s
However as mentioned earlier @sadasu from installer team  did vote for infrastructure object https://github.com/openshift/enhancements/pull/1148#discussion_r919410024  but as we discussed this is a workload balancer we will need to place in another object than infrastructure. So, placing this in the ingress cluster object will be the appropriate option.

@JoelSpeed I have made some major changes as we would need the api to be built in such a way that it could handle LB option like Scope which is not platform specific and also platform and  LB Type specific settings like connection idle time out and load balancer source range.
To set LBType the api will look like this
Spec.LoadBalancer.Platform.AWS.defaultType
To set the scope of the Load balancer(in the future). This is not platform or LB Type specific.
Spec.Loadbalancer.Scope
To set the LoadBalancerSourceRange which is LBType specific(in the future)
Spec.LoadBalancer.Platform.AWS.AWSClassicLoadBalancerParamaters.LoadBalancerSourceRange

@JoelSpeed I have made the changes as suggested. PTAL.

I see the verify test failing for other PR's as well. After running make verify on my local machine.

API rule violation: omitempty_match_case,github.com/openshift/api/operator/v1,StaticPodOperatorStatus,LatestAvailableRevision
API rule violation: omitempty_match_case,github.com/openshift/api/operator/v1,StaticPodOperatorStatus,LatestAvailableRevisionReason
hack/../hack/update-openapi.sh: line 43: ../../../github.com/openshift/api/verify_openapi/openapi.json: No such file or directory
make: *** [Makefile:55: verify-scripts] Error 1
Also I had to run  go get<EMAIL_ADDRESS>and go install<EMAIL_ADDRESS>or else it complains openapi-gen: No such file or directory
@JoelSpeed  @deads2k can you please let me know how I can fix the make verify test ?

API looks good now, for the verify, try running the openapi-gen and commiting the result, ./hack/update-openapi.sh.
You may want to try running it by picking from #1257 as this should be fairly environment detangled

@JoelSpeed this is the output ->

API looks good now, for the verify, try running the openapi-gen and commiting the result, ./hack/update-openapi.sh.
You may want to try running it by picking from #1257 as this should be fairly environment detangled

2022/08/24 08:53:40 Arguments validation error: output package cannot be empty
[miheer@localhost api]$
[miheer@localhost api]$ ./hack/update-openapi.sh
Generating OpenAPI definitions for apiserver:v1 apps:v1 authorization:v1 build:v1 config:v1 helm:v1beta1 console:v1 console:v1alpha1 image:v1,docker10,dockerpre012 imageregistry:v1 kubecontrolplane:v1 legacyconfig:v1 cloudnetwork:v1 network:v1 networkoperator:v1 oauth:v1 openshiftcontrolplane:v1 operator:v1 operator:v1alpha1 operatorcontrolplane:v1alpha1 operatoringress:v1 osin:v1 project:v1 quota:v1 route:v1 samples:v1 security:v1 securityinternal:v1 servicecertsigner:v1alpha1 sharedresource:v1alpha1 template:v1 user:v1 machine:v1beta1 machine:v1 monitoring:v1alpha1 at github.com/openshift/api/openapi
E0824 08:49:09.173510   11128 openapi.go:487] [github.com/openshift/api/machine/v1.NutanixResourceIdentifier]: union members must be optional: github.com/openshift/api/machine/v1.NutanixResourceIdentifier.Type
API rule violation: list_type_missing,github.com/openshift/api/apiserver/v1,APIRequestCountStatus,Conditions
API rule violation: list_type_missing,github.com/openshift/api/apiserver/v1,APIRequestCountStatus,Last24h
API rule violation: list_type_missing,github.com/openshift/api/apiserver/v1,PerNodeAPIRequestLog,ByUser
API rule violation: list_type_missing,github.com/openshift/api/apiserver/v1,PerResourceAPIRequestLog,ByNode
...
...
...
API rule violation: names_match,k8s.io/apimachinery/pkg/util/intstr,IntOrString,Type
API rule violation: omitempty_match_case,github.com/openshift/api/operator/v1,StaticPodOperatorStatus,LatestAvailableRevision
API rule violation: omitempty_match_case,github.com/openshift/api/operator/v1,StaticPodOperatorStatus,LatestAvailableRevisionReason

ci/prow/verify-client-go — Job failed. @JoelSpeed @deads2k can you please check this  issue ?
./vendor/github.com/openshift/api/openapi/openapi.json
F0824 12:38:29.971600   16614 main.go:53] Error: Failed executing generator: some packages had errors:
com.github.openshift.api.config.v1.AWSIngressSpec: discriminator set to type, but no fields in union

/retest

ci/prow/verify-client-go — Job failed. @JoelSpeed @deads2k can you please check this issue ?

we fixed the generator, you're green now.

question about handling "" so that upgrades work, looks ok otherwise.  You may need to install this on a cluster to confirm.

@JoelSpeed @deads2k  PTAL

/approve
/hold
holding for updates to Joel's comments.  The API lgtm otherwise.

@JoelSpeed PTAL

/retest

@JoelSpeed any idea why the verify is failing ?

/go/src/github.com/openshift/api/hack/update-deepcopy.sh
Generating deepcopy funcs
F0826 02:04:41.618098    8762 main.go:80] Error: Failed executing generator: some packages had errors:
errors in package "github.com/openshift/api/config/v1":
output for "v1/zz_generated.deepcopy.go" differs; first existing/expected diff:
"PlatformSpec) DeepCopyInto(out AWSPlatformSpec) {\n\tout = *in\n\tif in.ServiceEndpoints != nil {\n\t\tin"
"IngressSpec) DeepCopyInto(out AWSIngressSpec) {\n\tout = *in\n\treturn\n}\n\n// DeepCopy is an autogenera"

on my local machine make verify works with no errors. I have generated and pushed all the files.

@miheer There are other changes in master that affect this deepcopy file since your branch was created, if you rebase on latest master and then run the deepcopy generator you'll see there is a diff

Thanks @miheer, this looks good. Could you please manually test the new API and verify that the validations are working as expected (eg the optional fields can be omitted, the enums work correctly), then I'm happy to merge this

@JoelSpeed I was able to test this and it seems to be working fine
Please edit the object below. Lines beginning with a '#' will be ignored,
and an empty file will abort the edit. If an error occurs while saving this file will be
reopened with the relevant failures.

apiVersion: config.openshift.io/v1
kind: Ingress
metadata:
creationTimestamp: "2022-08-30T01:03:17Z"
generation: 11
name: cluster
resourceVersion: "60888"
uid: de89cc95-449e-4ae5-a837-f4171fe61e36
spec:
domain: apps.testnlbapi.devcluster.openshift.com
loadbalancer:
platform:
aws:
type: NLB
type: AWS
status:
componentRoutes:
works fine ->
removed loadbalancer It is not shown in the CR after saving the CR
removed platform It is not shown in the CR after saving the CR
removed aws It is not shown in the CR after saving the CR
removed NLB ....told to add a value
removed type: AWS ....It is not shown in the CR after saving the CR (Is this the right behavior ?)

Great thanks!
/lgtm

/hold cancel

@ahardin-rh can you please docs-approved this PR ?
@hongli can you please qe-approved this PR ?

/label qe-approved

/label px-approved

/label docs-approved


Create a separate PhysMemAllocator trait

I propose to more clearly separate out the notion of a Physical Frame Allocator. This PR creates a new PhysMemAllocator trait and uses it in the recursive page table. This also happens to solve a couple of compile errors around borrowing closures.
r? @phil-opp

Thanks for the PR, @mark-i-m!
What kind of compile errors do you mean?

I'm a bit unsure about this PR, because it requires much more than needed from the frame_allocator arguments. For example, we only need to allocate 4KiB frames in map_to, but with this PR the passed allocator must be able to allocate and deallocate frames of all sizes.
So we can no longer do things like the following:
let mut frames = [None; 3];
for frame in &mut frames {
    *frame = Some(frame_allocator.allocate_frame());
}
// do something with `frame_allocator`

recursive_page_table.map_to(page, frame, flags, || {
    frames.iter_mut().find(|f| f.is_some()).take()
});

Here we preallocate three frames and use a closure around the frames array as allocator argument. This allows use to do something different with frame_allocator because we don't need it anymore for the map_to call. With this PR this would become much more complicated since we would also need to provide code for allocating 2MiB and 1GiB frames and code to deallocate frames, which isn't even needed here.

On the current master, I get: […]

That's strange. For me and travis it compiles fine. I made some recent changes, could you try again with the current master?

These are fair concerns. I had kind of imagined a different setup: One physical memory allocator for the whole kernel (a simple buddy allocator)

This should be possible too, of course. You should be able to do:
recursive_page_table.map_to(page, frame, flags, || {
    buddy_allocator.allocate_frame()
});

I would be fine with a PhyMemAlloc trait. I don't think that closures are unidiomatic, but I guess that you're right that a trait makes the common case more convenient.
I think I would choose different names, maybe AllocateFrame or FrameAllocator.
Another thing that I just noticed: The unmap method should not have an allocator parameter at all and instead return the freed frame. This allows the caller to reuse it for something else without going through an allocator. Also, it makes the method symmetrical to map_to, where the frame is supplied by the caller and not allocated from an allocator.

I think that would conflate two different things: there is the page that was unmapped, but there are also possibly a few more pages freed that used to be used for page tables. I agree that the former should be returned to the caller, but I think it makes more sense for the latter to be freed to the allocator.

The thing is that we don't free any page table frames, because there is no simple and fast way to check whether a page table is empty. A function that returns a frame and takes no allocator would be less confusing in this regard.

because there is no simple and fast way to check whether a page table is empty.

Oh, hmm... that's a bit troubling. There are a lot of cases, where one knows that all mappings in some range will be reclaimed, so attempting to reclaim page tables makes sense (e.g. process termination).
Perhaps we could add another method map_reclaim that does attempt to reclaim pages (by iterating over the present bits of the page tables) and return them to the allocator?

@4e554c4c said that he had figured out a way of quickly checking if a page table is empty, I believe.

Actually, checking present bits is not sufficient because there could be shared pages.
Linux solved this problem with struct page and reverse mappings.
It seems that a bit more thought is required to come up with an api that allows genericly plugging in a way to find out if you should reclaim a page...

The problem is that that all approaches are opinionated, so this library is
not the right place to handle this in my opinion.
I think the best way forward is:

Return a the unmapped frame from unmap instead of freeing it in an
allocator. This allows the user to decide whether that frame should really
be freed. So they could use a reverse mapping, additional page table bits
or some other mechanism to identify shared frames.
Add some sort of reclaim_unused_page_tables method that frees unused
page table frames in an allocator. The method could take an address range
as argument to limit the scanning to a part of the table. This method could
be called by the user if they know that there should be reclaimable tables,
e.g. after they unmapped a process. Alternatively, it can be called
regularly by a background thread.

What do you think?

That sounds reasonable to me.
I will try to come around to updating the PR later today...

@mark-i-m Perfect, thank you!

@phil-opp Is it ok to use RangeBounds (which was stabilized 2 weeks ago)? That would limit the compiler version, but it would allow a nicer API...

(Still WIP, don't review yet)

@phil-opp Let me know what you think. I'm not sure how to test this, so if you have any ideas, let me know.

Is it ok to use RangeBounds (which was stabilized 2 weeks ago)? That would limit the compiler version, but it would allow a nicer API...

We already rely on unstable features, so I don't think that requiring an up-to-date nightly is fine.

I'm not sure how to test this, so if you have any ideas, let me know.

I'm planning to test it with a minimal kernel and the upcoming bootimage test command. This way, the tests run in QEMU and manipulate real page tables.

@phil-opp Do you mean that requiring an up-to-date nightly is okay, or not okay?

Could you split off the reclaim_page_table changes to a separate PR? This would allow us to quickly merge the unmap change and release version 0.2.0. Adding reclaim_page_tables later is not a breaking change, so we don't need to block the release on this.

Done. I will open another PR soon.

@phil-opp Opened #32
We can continue discussion about reclaim_page_tables there.

Thanks a lot!

I just updated the bootloader to the new trait based interface:
 pub(crate) fn map_page<'a, S>(
     page: Page<S>,
     phys_frame: PhysFrame<S>,
     flags: PageTableFlags,
     page_table: &mut RecursivePageTable<'a>,
     frame_allocator: &mut FrameAllocator,
 ) -> Result<MapperFlush<S>, MapToError>
 where
     S: PageSize,
     RecursivePageTable<'a>: Mapper<S>,
 {
-    page_table.map_to(page, phys_frame, flags, &mut || {
-        frame_allocator.allocate_frame(MemoryRegionType::PageTable)
-    })
+    struct PageTableAllocator<'a, 'b: 'a>(&'a mut FrameAllocator<'b>);
+
+    impl<'a, 'b> paging::FrameAllocator<Size4KiB> for PageTableAllocator<'a, 'b> {
+        fn alloc(&mut self) -> Option<PhysFrame<Size4KiB>> {
+            self.0.allocate_frame(MemoryRegionType::PageTable)
+        }
+    }
+
+    page_table.map_to(page, phys_frame, flags, &mut PageTableAllocator(frame_allocator))
 }

It's much more boilerplate now and we need two explicit lifetimes. So I think I liked the closure based interface more. I just wanted to bring it up here, maybe we can improve ergonomics somehow.
(Commit https://github.com/rust-osdev/bootloader/commit/83ae580c5e0bff4f9e307e4c70428874de8b82a5#diff-9c959995cefca11a6ba2160e735da057R161)

@phil-opp I'm confused. Why not just impl paging::FrameAllocator for frame_allocator::FrameAllocator?

The problem is that paging::FrameAllocator is general purpose, i.e. the trait could be used for all kind of frame allocations, not only for page table allocations. The frame_allocator::FrameAllocator in contrast wants to keep track of the purpose of the allocated frames to create the bootinfo later. So if we implemented the trait for frame_allocator::FrameAllocator we would need to specify a fixed region type, which does not seem like a clean solution (because we might want to allocate different region types using the trait in the future).
I guess that's a relatively niche use case, but if similar cases appear we might think about implementing the trait for FnMut() -> PhysFrame, so that we can use the old closures based interface too.

Interesting. What does MemoryRegionType do? Why can't you allocate an arbitrary frame of physical memory?
This problem suggests that perhaps FrameAllocator should take some sort of options argument?

We do allocate an arbitrary memory frame, we just remember for what it is used to create the bootinfo when control is passed to the kernel. This is really an uncommon use case, so adding an options argument is not worth it.

Actually, when I thought about it, I don't think this is an unusual use case: https://github.com/torvalds/linux/blob/master/include/linux/gfp.h

Ah, good point. So I guess we have three options:

Add a generic option argument to the trait.
Do nothing and let people create wrapper types (like in the bootloader).
Implement the trait for FnMut() -> Option<PhysFrame> to support the previous closure based interface.

I don't like option 1 because we would need to also add the option parameter to map_to and all other mapping methods. Option 2 is the current state and requires no modification to the mapper methods, but requires some boilerplate. Option 3 would eliminate that boilerplate by allowing any closure that returns frames as allocator.
The difference between option 2 and 3 is whether the caller must explicitly define a frame allocator or can implicitly use any closure that returns frames. The question is if there's any danger from this implicitness, i.e. is it possible pass a non-allocator closure by accident.

I'll comment more later, but I wanted to leave some notes in addition to your comments:
If you want to use a third-party allocator, you still need to create a wrapper type that impls the trait. This is a bit annoying. We might consider adding some stock implementations. The one I wrote is here:
https://github.com/mark-i-m/os2/blob/288f1e3edd28234ad6a6e096be9d9f76fda41a7f/kernel/memory/paging/mod.rs#L47-L55
We could also consider a NoAllocator implementation of FrameAllocator, where any allocation just cause a panic. This could be used when we want to ensure there is no allocation (e.g. under memory pressure or in the memory reclamation algorithm itself (to prevent recursion (to prevent recursion (to prevent recursion (...))))).


feat: enable algolia docsearch

Prerequisites for Code Changes

[x] This pull request follows the code style of the project
[x] I have tested this feature

Changes Proposed

enable algolia docsearch

Are we missing the search box element https://docsearch.algolia.com/docs/DocSearch-v3#implementation

the search is integrated into the docusaurus theme, just need to configure the properties for the plugin - https://docusaurus.io/docs/search#using-algolia-docsearch


Highlight the portions of the boilerplate that specifically demonstrate the configuration and usage of React Hot Loader 3

The goal of this PR is to tighten the focus of this boilerplate on the utility of React Hot Loader 3.
It results in a net reduction in lines of code while adding functionality and addressing a couple issues:

Closes https://github.com/gaearon/react-hot-boilerplate/issues/63 - Add production build
Closes https://github.com/gaearon/react-hot-boilerplate/issues/65 - Use webpack-dev-middleware (for Error Overlay)

Hmm, not sure about hjs-webpack, I'd rather not hide the base Webpack config. Want to just do the production build side?

Going to close this because of lack of activity and I still would prefer not to use hjs-webpack. Let me know if you still want to add the production build.


Fix the issue with changing order by height

So when we click to sort by height there is a short (or longer, depending on the RPC performance) moment when it changes the order of already downloaded blocks and then after few seconds it shows correctly sorted data (so starting from block 1).
The issue is visible on the video. I know this happens because of the fetch time, but we should show some spinner instead of changing the order of the visible blocks.
https://user-images.githubusercontent.com/3756160/217575546-31afbe60-0d5f-4635-bb5c-59083afa689c.mov

Fixed in: https://github.com/casper-network/casper-blockexplorer-frontend/issues/180


What is the difference between setup.py and setup_cuda.py ?

Is setup.py for CPU only version and setup_cuda.py for GPU version?
and if I use GPU, whether we should compile setup.py and setup_cuda.py sequentially?
Thanks a lot.

setup is for compiling cpu version, setup_cuda is for gpu version


Fix: requests nil error

Fix example.rb requests method:
example.rb:71:in 'remap_headers': undefined method 'each' for nil:NilClass

When does this error? Would you provide an example? Is there a test for coverage?

Sorry, I can't provide an example because I don't have access to the repository where this issue was met.


Set TX:MSC_PCRE_LIMITS_EXCEEDED variable is limits exceeded

There was a PR (#2737) which fixes an earlier shortcoming, namely that the @rx (and @rxGlobal) operator(s) do not handle PCRE limit issues.
I didn't follow it, but unfortunately seems it implements a different behavior from the other engine (mod_security2).
In mod_security2's reference the relevant behavior is explained as:

MSC_PCRE_LIMITS_EXCEEDED: Set to nonzero if PCRE match limits are exceeded. See SecPcreMatchLimit and SecPcreMatchLimitRecursion for more information.

May be the documentation is a bit ambiguous, but it means the TX.MSC_PCRE_LIMITS_EXCEEDED will be set, not the "regular" MSC_PCRE_LIMITS_EXCEEDED variable.
This patch corrects this behavior.
Please note, that the introduced variable is not mentioned in v3's documentation.
Why is this important?
The OWASP Core Rule Set team has a plan for the rule set to handle these types of errors. Without the compatibility, we can't do that.

Cc @brandonpayton.

Thanks for noticing this, @airween. I was not aware of this. Perhaps we should entirely remove the standalone variables added in #2736 and replace them with TX collection entries. What do you think?

Thanks for noticing this, @airween. I was not aware of it. Perhaps we should entirely remove the standalone variables added in #2736 and replace them with TX collection entries. What do you think?

I do not know yet. If a rule writer can make a rule both for mod_security2 and libmodsecurity3 (the two are compatible), then - I think - it's not distracting.

I think it is a little confusing to have two different sets of vars indicating MSC PCRE errors. At least I personally find it confusing. :)

I do not see a need why compatibility had to be broken here. Anything bringing it back is good from my perspective.

@brandonpayton, @dune73 - thanks guys.
In the meantime, a new issue appeared: #2902, so I think there are three votes to remove the regular variable.
@martinhsv - what would be the better: should I add a new commit to this PR which removes the variable by #2736, or (if this PR is fine) after merging this, I will create a new PR?

I have made some comments in #2901 .
In brief: to me it would seem confusing to have pcre error flags set by the engine to use TX:, when we don't do that with any other such errors (REQBODY_ERROR, the many MULTIPART_ errors, etc.)
In other words, the existing v3 functionality may reasonably be considered an improvement.
With the functionality as implemented in #2737 , TX: variables continue to be reserved for transaction variables that are created by individual rules -- either through direct naming or via the capture action).
(Of course, if left as is, the v3 doc and rule 200005 in modsecurity.conf-recommended need updates.)
For anyone wishing to have a single rule work in both v2 and v3, writing a single SecRule to do so is pretty straightforward.

For anyone wishing to have a single rule work in both v2 and v3, writing a single SecRule to do so is pretty straightforward.

Sorry for the request, but could you show an example? mod_security2 does not support MSC_PCRE_LIMITS_EXCEEDED variable, only inside of the TX collection. That's our problem, that's why I created this patch.

Ugh. I see your point. I didn't think that through fully from the perspective of a v2 installation. Handling both v2 and v3 might indeed be annoying to try to do with a single SecRule. I'll give that a bit more thought.

Ugh. I see your point.

No problem.

I'll give that a bit more thought.

Thanks. Let me know if I can help you anything.

All right. There is a way to do the test without creating this TX variable within the code. However, the simplest, non-contrived way that I could think of doing so, involves creating a TX variable within a chained rule.
Having a synonym for backwards-compatibility reasons isn't terrible. I'm ok with this and will proceed to merge.

@martinhsv thanks.
Now what should we do with the "regular" MSC_PCRE_LIMITS_EXCEEDED variable? As the author of that patch mentioned, he would revoke that.
Should I prepare a PR?

A good way to handle such inconsistencies would be to add the version number in a variable.
I personnally added it in an environment variable in hook_request_early (v2):
apr_table_set(r->subprocess_env, "ModSecVersion", MODSEC_MODULE_VERSION);
This allowed me to have rules analyzing MULTIPART_PART_HEADERS compatible with all versions.
Using an environment variable also allows to perform some checks with mod_header.


Spec File Icons

Can we have a different icons for a spec.js files vs js files ?

Sure. I'll try some things, don't hesitate to post ideas if you have any

Here is some mix of the javascript icon and the test folder icon :

However I don't know if that's distinguishable enough.


Can't import client

Hello, I am trying to use the module and as soon as I import GfycatClient from gfycat.client I get the following error:
Traceback (most recent call last):
  File "<pyshell#2>", line 1, in <module>
    from gfycat.client import GfycatClient
  File "C:\Users\Vinicius\AppData\Local\Programs\Python\Python35-32\lib\site-packages\gfycat\client.py", line 4, in <module>
    from constants import (FETCH_URL_ENDPOINT, FETCH_URL_LAZY_ENDPOINT,
ImportError: No module named 'constants'

What am I doing wrong? Code is exactly that:
from gfycat.client import GfycatClient

Hi @mvinoba, there was an issue with imports for some users. I have updated the package on pypi to fix the error. Please reinstall the package, and it should work fine.


Fix for #39

FlexSearch should be loaded before the integration script is loaded; this PR changes the ordering accordingly, thus fixing #39.

Thanks1


WIP: Uni v3 Zap

Description
TODO

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

0 of 55 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.8%) to 87.432%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contracts/src/zaps/UniV3Zap.sol
0
55
0.0%

Totals

Change from base Build<PHONE_NUMBER>0:
-1.8%

Covered Lines:
2400

Relevant Lines:
2745

💛  - Coveralls


Change: M3-2669 Paginate Disks and Configs with Paginate Render Props Component

Description
Remove Pagey from Disks and Configs panel in favor of Paginate render-props component
Type of Change

Non breaking change ('update', 'change')

Applicable E2E Tests
N/A
Note to Reviewers
Also removed the auto scroll-to-top from disks and configs because they don't have their own dedicated pages, so it would make sense. We should implement some sort of scroll-to-id feature, similar to scrollToError

@johnwcallahan alternatively just request all disks and configs when the Linode Details view is mounted?

@martinmckenna yeah, I guess that's the best option here. Probably solves several problems.

@johnwcallahan all good for re-review


Add print to confirm that the failed_test_stories.yml has been saved

Proposed changes:

Added a message at the end of rasa test core, showing that the failed stories were saved in results/failed_test_stories.yml, resolving issue #3998

Default path (results)

User specified path

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

Please, run the command black . your code have a lint issue

I think for this contribution changing documentation will not be needed, but update the changelog file, see how other lines is made to follow the pattern.

I made the changes you asked for. Please check if it is correct and if anything else is missing.

Pull Request Test Coverage Report for Build 334634746

20 of 20 (100.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 86.207%

Files with Coverage Reduction
New Missed Lines
%

rasa/core/run.py
1
88.39%

rasa/telemetry.py
2
87.21%

Totals

Change from base Build 332399642:
0.01%

Covered Lines:
20207

Relevant Lines:
23440

💛  - Coveralls


PropertyListEncoder / PropertyListDecoder

Implemented with https://github.com/apple/swift-foundation/pull/468


[BUG] "Error starting manager: require backing-image-manager-image"

Describe the bug
The longhorn-system does not start at all. After applying the standard manifest, the pods which are going to CrashloopBackoff
lars@d04:~/kubernetes-cluster/iz/misc/longhorn/old$ kubectl -n longhorn-system get all
NAME                                            READY   STATUS             RESTARTS   AGE
pod/longhorn-driver-deployer-6b755bf76f-lgl8v   0/1     Init:0/1           0          5m49s
pod/longhorn-manager-b8s5c                      0/1     CrashLoopBackOff   5          5m49s
pod/longhorn-manager-bl56m                      0/1     CrashLoopBackOff   5          5m49s
pod/longhorn-manager-ppnnw                      0/1     CrashLoopBackOff   5          5m49s
pod/longhorn-manager-td9s2                      0/1     Error              6          5m49s
pod/longhorn-manager-z2vc4                      0/1     CrashLoopBackOff   5          5m49s
pod/longhorn-ui-67954f5b9b-g57rq                1/1     Running            0          10m

NAME                        TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
service/longhorn-backend    ClusterIP   <IP_ADDRESS>   <none>        9500/TCP   10m
service/longhorn-frontend   ClusterIP   <IP_ADDRESS>    <none>        80/TCP     10m

NAME                              DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR   AGE
daemonset.apps/longhorn-manager   5         5         0       5            0           <none>          5m49s

NAME                                       READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/longhorn-driver-deployer   0/1     1            0           5m49s
deployment.apps/longhorn-ui                1/1     1            1           10m

NAME                                                  DESIRED   CURRENT   READY   AGE
replicaset.apps/longhorn-driver-deployer-6b755bf76f   1         1         0       5m49s
replicaset.apps/longhorn-ui-67954f5b9b                1         1         1       10m

To Reproduce
Steps to reproduce the behavior:

kubectl apply -f https://raw.githubusercontent.com/longhorn/longhorn/v1.1.1/deploy/longhorn.yaml

Expected behavior
A clean longhorn environment
Log
lars@d04:~/kubernetes-cluster/iz/misc/longhorn/old$ kubectl -n longhorn-system logs pod/longhorn-manager-b8s5c
2021/05/20 06:52:50 proto: duplicate proto type registered: VersionResponse
time="2021-05-20T06:52:50Z" level=fatal msg="Error starting manager: require backing-image-manager-image"

Environment:

Longhorn version: 1.1.1
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s

Number of management node in the cluster: 3
Number of worker node in the cluster: 3

Node config

OS type and version: Debian Buster
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: currently 0

Additional context
Add any other context about the problem here.

HI! Could you update Node Config? And provide support bundle.  I suspect there is a resource/network issue.


Mods

Hello,
First of all, thank you for this excellent work.
Now, the question is: is it possible to use Minecraft mods with this add-on?
Regards.

I haven't tried this but you can check out the documentation of the underlying docker container: https://docker-minecraft-server.readthedocs.io/en/latest/mods-and-plugins/


It would be nice if we could support importing full components into vue

Here's a demo of an element-plus I wrote
and this is element-plus official demo
I've seen a lot of community implementations like https://element-plus.run, which is actually bad for developers, and the pain point is that we can't bring in a whole component library at once. As you can see, the lack of el-rows in my demo caused my code to only partially render
So I have a proposal
// if it works will be great
import { use } from 'vue'
import ElementPlus from 'element-plus'
use(ElementPlus)

thanks bro, it works

@Sight-wcg in the playground link you sent, how does the playground know to import the element-plus npm package? does that happen automatically


feat: support proto & json encoding

Here's the code to implement this, which is pretty straightforward in your encoder/scan switch statements. If you're open to adding this, I can add some tests
fixes #196

Closed per discussion in #196


ImportError: cannot import name 'build_dist_index' from 'pipdeptree'

In a new venv I'm unable to run python-brewer successfully with python 3.9.6.  With the example from the README I receive the following:
$ cd "$(mktemp -d)"
$ python3 -m venv venv
$ source venv/bin/activate
$ pip install python-brewer
$ pybrew \
    -n "Statik" \
    -d "Python-based, generic static web site generator aimed at developers" \
    -H https://getstatik.com \
    -g https://github.com/thanethomson/statik.git \
    -r https://github.com/thanethomson/statik/archive/v0.14.2.tar.gz \
    statik \
    statik.rb
Traceback (most recent call last):
  File "/usr/local/bin/pybrew", line 5, in <module>
    from pythonbrewer.cmdline import main
  File "/usr/local/lib/python3.9/site-packages/pythonbrewer/__init__.py", line 4, in <module>
    from pythonbrewer.deplist import *
  File "/usr/local/lib/python3.9/site-packages/pythonbrewer/deplist.py", line 6, in <module>
    from pipdeptree import build_dist_index, construct_tree, sorted_tree
ImportError: cannot import name 'build_dist_index' from 'pipdeptree' (/usr/local/lib/python3.9/site-packages/pipdeptree.py)

It's an old project with the last commit in 2018. But if you're interested my MR solves an issue with the import error, caused by wrong dependency to pipdeptree.

@shmygol, thanks.  Is there a project for python that is either more widely used or up to date?  Most of the python homebrew tools I’ve found have one problem or another.

pip uninstall pipdeptree
pip install pipdeptree==0.9.0

works for me


Question: send crafted TRAP PDU in SNMPv1

Hi,
Example available in documentation is for v2.
My problem is I didn't find example to send Trap with EnterpriseOID so I do: pdu.setComponentByName('enterprise', mytrap.getEnterpriseOid())
As read in #78 : "when you are sending v1 TRAPs you still need to pass v2 TRAPs to pysnmp"
So I don't know if I should:

do a v2TrapPDU, convert it to v1, setEnterpriseOid and use sendPdu()
do a v1TrapPDU, setEnterpriseOid and convert it tov2 (and use sendPdu())

Or there is a way to set EnterpriseOid using sendNotification?
Thank you

TRAP v1 messages can be sent via high-level API, according to the documentation,

https://docs.lextudio.com/pysnmp/examples/hlapi/asyncio/agent/ntforg/common-notifications
/examples/hlapi/asyncio/agent/ntforg/default-v1-trap.py


[Dropdown] onChange doesn't fire if used with action: 'hide'

I have a simple dropdown on icon and I dont want to have '.selected .active' on selection of the particular item, so I am simply using action: 'hide'. I also want to do custom action on change, so I have onChange too as suggested here : http://beta.semantic-ui.com/modules/dropdown.html#specifying-select-action
But onChange is not firing. If i change to action: 'select' or remove action , it works.
Here is fiddle of my case: http://jsfiddle.net/v4b088th/

Hide does not store values.
Use action: 'select'
http://jsfiddle.net/h3f6010r/

I dont want to have '.selected .active' on selection of the particular item, that is why I used hide.
Anyways, if this is not supported then you should remove it from examples. This code is taken exactly from the last example as described here:  http://beta.semantic-ui.com/modules/dropdown.html#specifying-select-action
The above example led me to believe that this will work.
Also, for my particular case of having to take action on select, but not having the selection highlighted, what do you suggest? Should I use onclick on the item and just action: 'hide' on dropdown?


Need to get to underlying ID3D11Texture2D pointer from a Veldrid texture... how?

I am trying to write a SteamVR (OpenVR) overlay app.  OpenVR requires a handle to the D3D11 texture to use to render the overlay.  I have a Veldrid texture that I set as render target (drawn on by ImGui) and the texture has the sampled flag on it as well.  All that works.
Now I need to pass the underlying D3D11 texture to OpenVR so it can display it on my HMD as an overlay... how do I get to that pointer?
Note I am using GraphicsBackend.Direct3D11 when calling VeldridStartup.CreateWindowAndGraphicsDevice.
In Unity I would just call texture.GetNativeTexturePtr().  What's the equivalent with Veldrid?

Never mind - I figured this one out.  The answer is -
BackendInfoD3D11 _backendInfo;
_graphicsDevice.GetD3D11Info( out var _backendInfo );
var texture_t = new Texture_t();
texture_t.handle = _backendInfo.GetTexturePointer( _texture );
texture_t.eType = ETextureType.DirectX;
texture_t.eColorSpace = EColorSpace.Gamma;
OpenVR.Overlay.SetOverlayTexture( _overlayHandle, ref texture_t );


Fix MoveMesh for 3D cases + add 3D FSI test

This PR fixes a code-breaking bug in MoveMesh (missing import statement for 3D case).
A new 3D FSI test with a floating cube has also been added to make sure that this is covered for the future

This looks okay. Although again, it'd be good to know the size of the added mesh files.
And to clarify, the reason this test is needed is because the existing FSI cylinder test case is only 2D whereas the observed bug was in the 3D code?

@zhang-alvin that's right, the bug only appeared in 3D. This test ensures that the whole workflow (two-phase, FSI, ALE) is working in 3D as well.
Concerning the mesh, it is very coarse and the files are relatively small in kb


Dolphin doesn't show all (WEBP) image previews

Describe the bug
Dolphin supports image previews, however it does not work with WEBP out-of-the-box.
Steps To Reproduce
Steps to reproduce the behavior:

set services.desktopManager.plasma6.enable = true
Open Dolphin with previews enabled and icon view
Open a folder with WEBP images
Images show a placeholder icon
Can open in Gwenview as usual

Expected behavior
I expect that all image formats supported by Gwenview would also be supported by Dolphin for previews.
Gwenview has a dependency on qtimageformats, and is installed with plasma by default. Adding qtimageformats to my config fixes this issue. Perhaps qtimageformats could be added as an optional dependency of plasma6, since kimageformats is already a required dependency in this way.
This is possibly also applicable to plasma5, since it doesn't use qtimageformats either.
Screenshots

Notify maintainers
@SuperSandro2000 @K900 @omentic @oxalica @stuebinm @peterhoeg @Stzx @eclairevoyant @MinerSebas @fpletz @mjm
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.52, NixOS, 24.05 (Uakari), 24.05.5366.fbca5e745367`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.8`
 - channels(root): `"home-manager-24.05.tar.gz, nixos-24.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

I have a similar issue in unstable. But for me the
/nix/store/cy5gab6y2hiqbx7a35cmr0z3j7c6y7wq-kimageformats-6.7.0/lib/qt-6/plugins/
in the QT_PLUGIN_PATH of my Plasma 6 session. That breaks e.g. jpegxl backgrounds.

It looks like Qt 6.8 broke things wrt plugin discovery: https://bugreports.qt.io/browse/QTBUG-130109
We have a patch and I'll send it to staging-next once it's merged upstream.

Thanks!

I believe I have a similar issue, except that in my case not even gwenview can open the images, let alone Dolphin preview them. If I try to open e.g. an avif image it says Gwenview cannot display documents of type image/avif even though I have explicitly installed libavif and kimageformats and qtimageformats

Yes, this is the same regression, fix is in staging-next.

How long until it hits unstable?

A couple days, probably.

This is fixed for me now on unstable


Alias Handler for SCIM Users

see issue #2505
Preparation for alias feature for SCIM users; adds subclass of EntityAliasHandler that handles alias entities of SCIM users.
Prerequisites:

#2765
#2824

Hi, as this PR is a stepping stone toward the overall "alias feature." And it looks like the changes are limited to the specific code paths related to the feature. So I would defer the approval of this PR to @strehle, in terms of judging whether this PR accomplishes its goal.
I'll also approve as long as our prior understanding the "alias feature" are still true after this PR: That the functionality & performance of all the existing UAA features are not impacted (do not experience breaking changes / regressions) when the "alias feature" is not turned on (and the feature is off by default).

@adrianhoelzl-sap please cleanup PR , see
https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=2769&id=cloudfoundry-identity-parent
Rest is ok

@adrianhoelzl-sap please cleanup PR , see https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=2769&id=cloudfoundry-identity-parent
Rest is ok

@strehle This link is not showing no results for me:

@adrianhoelzl-sap please cleanup PR , see https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=2769&id=cloudfoundry-identity-parent
Rest is ok

@strehle This link is not showing no results for me:

there where updates and now smells are solved
See overview https://sonarcloud.io/summary/new_code?id=cloudfoundry-identity-parent&pullRequest=2769

@Tallicia do you have open issues here ?

I merge this PR as all questions seem to be addressed and two approvals exist.


Fixed OpenGL version string being parsed incorrectly on some platforms

Fixes #1249.

std::strncmp compares two possibly null-terminated character arrays. If the null terminator is encountered in either array before n characters are compared, it ends the comparison. In our case, there is no danger because both parameters are guaranteed to be null-terminated.

I was talking about what happens inside the blocks:
m_settings.majorVersion = version[13] - '0';
m_settings.minorVersion = version[15] - '0';

I guess you don't want to have to assume that the OpenGL implementation is standard compliant? 😁

What standard? We are "parsing" strings we know nothing about 😀

Fixed.

Looks very robust now 😆
What about a helper function to avoid all these repetitions?
bool parseVersionString(const char* prefix, unsigned int versionPosition, unsigned int &major, unsigned int &minor);

Naaah... an extra function just to move out 4 if statements? 😨

It would makes things more readable and shorter. 👍
Maybe even one if statement:
if (parseVersionString(...) || parseVersionString(...) || ...) {
    m_settings.majorVersion = major;
    m_settings.minorVersion = minor;
} else { warning }

with some better formatting, ofc.

Fixed.

Is there a reason to use strncmp with the last argument being strlen of one string? If it stops at the first encountered \0 anyway, maybe we can just use strcmp?

Using std::strncmp we can check for prefixes. std::strncmp("A", "ABC", 1) will return 0 while std::strcmp("A", "ABC") will return non-zero.

Fixed.

Does it now look okay, @LaurentGomila?

@kingnobody8 Can you try the code? Need more of an actual test than review. 🙂

@LaurentGomila Fixed.

Looks good to me! tested on iPhone 8

Looks good. Tested on Windows, Android, Mac, and iOS, all successfully.
Just for reference, only the apple versions will do the "old way" of getting the gl version. Windows and android do the "new way".


[Bug]: changing the column type to tag kills the DB..

Contact Details
<EMAIL_ADDRESS>What happened?
when changing a column type to tag It freezes and wipes out the DB (leaves with zero bytes)
this is what happens during the initial freeze

What platform were you using?
Desktop
Version of the plugin
2.7.4
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

I will check this issue as priority.
In your video looks like there is too much files to edit the color property at the same time

thx!
happy to help in anyway I can (though I cant code so only testing :))
you are a legend with this plugin!
Z

Was a concurrency problem =(
Commit b50ce35c
When the plugin is trying to add a new option/color there is another row with another petition and overwrite it. Also the entire row is refreshed and trigger AGAIN the add function because the override thing
Will be added with the next version


feat: provide mechanism for overriding _rtl.ccs files

Thank you @wincent for detailed review.

Thanks for updating, @naderjafari. CI is failing and yarn test (in repo root) fails locally for me too. Lots of this kind of thing in the console:
liferay-js-themes-toolkit/node_modules/graceful-fs/polyfills.js:282
        if (!stats) return cb.apply(this, arguments)
                              ^

TypeError: Cannot read property 'apply' of undefined

I'm suspicious of the promise-based version here. Can we go back to the callback-based one that I suggested here?
I've reviewed this for code correctness but I'd like to get a second or third opinion on this from @julien or @jbalsas, especially in terms of the higher-level "is this a good idea" question.

Just started reviewing :)

Thanks for updating, @naderjafari. CI is failing and yarn test (in repo root) fails locally for me too. Lots of this kind of thing in the console:
liferay-js-themes-toolkit/node_modules/graceful-fs/polyfills.js:282
        if (!stats) return cb.apply(this, arguments)
                              ^

TypeError: Cannot read property 'apply' of undefined

I'm suspicious of the promise-based version here. Can we go back to the callback-based one that I suggested here?
I've reviewed this for code correctness but I'd like to get a second or third opinion on this from @julien or @jbalsas, especially in terms of the higher-level "is this a good idea" question.

This error occur when i run yarn test in windows10 command line :

if your suggested will resolve this issue I will change it as your suggestion.

@naderjafari, it looks like your machine tries to open .js files with Microsoft Script Host and not with node.js - I'm not a Windows expert but you might need to run yarn.cmd test instead of yarn

@naderjafari, it looks like your machine tries to open .js files with Microsoft Script Host and not with node.js - I'm not a Windows expert but you might need to run yarn.cmd test instead of yarn

this resolved by change  "test": "jest" to  "test": "yarn jest" in package.json file.
other scripts also start with yarn so i think this is a bug.

@naderjafari, it looks like your machine tries to open .js files with Microsoft Script Host and not with node.js - I'm not a Windows expert but you might need to run yarn.cmd test instead of yarn

this resolved by change "test": "jest" to "test": "yarn jest" in package.json file.
other scripts also start with yarn so i think this is a bug.

I'm not sure about that: we have a Windows laptop for testing and I'm able to run yarn test and yarn run test without issues.

@naderjafari I've got some feedback from @marcoscv-work (who has much more knowledge concerning themes than I have) and who thinks it would actually be better to keep on doing this in a custom task (like you mentioned in #330)  rather than in the build tasks.

@naderjafari I've got some feedback from @marcoscv-work (who has much more knowledge concerning themes than I have) and who thinks it would actually be better to keep on doing this in a custom task (like you mentioned in #330) rather than in the build tasks.

@julien this feature is useful in right-to-left languages for example we can override directions of code or pre dom elements to ltr or text-align to left to show text in this elements in correct way.
r2 does not works perfectly as we excepted for all CSS rules and via this way we can override all incorrect CSS rules that converted by r2.
also this feature is available in other modules that builds with gradle BuildCss.
This feature comes form 6.2 :
https://dev.liferay.com/develop/tutorials/-/knowledge_base/6-2/supporting-right-to-left-languages-in-plugins

The two doubts I had about this feature were: about the implementation complexity (as noted here) and also the higher-level question of whether this is the appropriate pattern in the first place (this pattern of creating a named file that ends up getting appended to an build product of the same name may be a little too opaque). So it is interesting to see that at least in 6.2, that was the way that overriding worked; ie. in that tutorial, it says:

DEFINING CUSTOM CSS FOR RTL LANGUAGES
You may, however, want to extend the generated CSS by defining your own custom styles for RTL languages.
You’ll need to create a CSS file with the suffix _rtl in the same location as a CSS file that you want to extend. For example, create a file main_rtl.css to extend a file named main.css.
After you have deployed your plugin, check its css/.sass-cache folder to see that the generated _rtl.css file not only contains the CSS from the original file, but also contains your custom CSS code at the end.

So the question is: do we want to follow that precedent, should we search for a better way, or is doing nothing and recommending the use of hooks the right call? @julien, @marcoscv-work: what do you think?

Hey @naderjafari, @wincent, please see https://github.com/liferay/liferay-js-themes-toolkit/issues/330. Meant to answer here, but apparently I misplaced it 😢

Hey @naderjafari, @wincent, please see #330. Meant to answer here, but apparently I misplaced it 😢

Hi @jbalsas
Just started reviewing


ISP Dns Leak?

Hi. I have been testing the last beta this days, using the original toml config and also trying several scenarios (all servers, 1 server, 2 servers, 3 servers, 4 servers, only ipv4 servers, only ipv6 server, both ipv4+ipv6, the 3 require_* set to true and/or false, etc...). On all the cases, visiting this url https://ipleak.net/, at some point, it leaks to my ISP. I don't know if it have something to be with the way dnscrypt-proxy try to handle the 100+ connexions that the site does, or what. Any idea? Thanks in advance.

Browser automatic updates working 100%. As expected, using dnscrypt-proxy.
Proof: I have logs about these events in dnscrypt-proxy.
Of course, I have additional rules for the Browser also (for all Group at once with programs inside, including installer/updater too). It's not about global rules for all system (as I described above), but for the Browser only.
Rules from Top -> Bottom (First -> Last):

DNS (UDP) 53 - Out (DNSCrypt) ........ ALLOW
Outgoing from Remote Port: 53 to Local Ports: 1024 - 65535
For addresses: 127.0.0.XX1,127.0.0.XX2 (listen_addresses) - I keep 2 resolvers working

Public (UDP) ................................................... BLOCK
All Outgoing/Incoming queries for Public Network

Internet (TCP) - Out .................................... ALLOW
Outgoing for Any address

So, nothing can use any other DNS, only those I want: dnscrypt-proxy + fallback resolver.
Even such 3-rd party resolvers as DANE (or any other) will be blocked for sure using this settings.
They are confirmed by time and positive experience.

:flags options for chromium browsers are also important
Search by keywords:
[Disable]
preconnect / predictor / preresolve / prefetch / suggestions / Google / SendBeacon / USB / sensor / credit card ... autofill / WebRTC / network (if you don't need net queries)
[Enable]
reduce ... 'referer' ... granularity /  disable ... auditing / memory coordinator / isolation / AppContainer Lockdown

As you will see, overall bias of Google web-core is reducing user safety and privacy. Most of unwanted options are turned on, instead there are very few useful options are turned off by default.

Or just use Brave.

Or just use Brave.

600 Mb of garbage about new way of monetization.


change position

Hi, after make text by this code:
Crouton.makeText(activity, "ddddd", Style.ALERT).show();

its show top of my toolbar and i cant change position of that, please help me, Thanks

Crouton.makeText(activity, "ddddd", Style.ALERT, ViewGroup).show();

Basically what @amervelic said.

I'm post new issue as your help


Updating instructions for installing KVM2 on Ubuntu & CentOS

The current instructions on the website are already out-of-date as we noticed recently. Updating to reflect current valid procedure.

This change is

/assign @richardsliu

/lgtm
/approve


phpfpm container renamed to php

The phpfpm container has also phpcli, so maybe it would be better to rename it to just php.
Then you can simply run
docker-compose exec php bin/magento ...

(as of docker-compose 1.7.x).

I don't want to rename this, because the base image is built from phpfpm. It's possible that in the future we will create image(s) from the php image, and then we would be in trouble.

You know what, I'm going to disagree with myself :) Since this is just in the docker-compose, I think it does make sense to rename this to php to keep things simple. We are already doing this with the database layer in naming it db.

You can also call db container as mysql/mariadb and alias it as db for the app :)
and about phpfpm vs php... you named app the "...-nginx..." container, so I don't think this is an issue. But it would be easier to call, as I wrote above :)

Let's think about this for a while. I think we should probably avoid renaming everything at this point, because it might cause some confusion. I understand the aliases here, but I think its simpler to not use aliases at all. Since nginx config at https://github.com/mageinferno/docker-magento2-nginx/blob/master/Dockerfile references phpfpm, we should probably just keep that naming convention for now.
I understand this approach isn't used for nginx (app) and mariadb (db), however neither of those has a dependency such as phpfpm within their configurations, so we should probably keep the current approach going forward.


[development] Bump Quarkus version to 2.13.2.Final

https://github.com/kiegroup/optaplanner/pull/2238
https://github.com/kiegroup/optaplanner-quickstarts/pull/426

@rsynek There seems to be an issue with school-timetabling and the quarkus update ... Can you have a look ?

@rsynek There seems to be an issue with school-timetabling and the quarkus update ... Can you have a look ?

Only with Gradle? That's weird. I will look into it.

The issue looks like a regression in the Quarkus Gradle plugin, introduced in 2.13.1.Final.


SOLR-16228: Audit how we refer to Solr Modules

https://issues.apache.org/jira/browse/SOLR-16228
Description
Our docs reflect years of evolution of how we add things to solr, and don't put Solr Modules front and center.
Solution

Be consistent about referring to Solr Modules, remove use of the word for non Solr Modules where possible.

Make sure EVERY Module in the Ref Guide lists the name of the module using the same terminology.

Sets us up for better documentation around where just enabling a Solr Module isn't enough, and you have to do werid stuff.

Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[ X] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ X] I have created a Jira issue and added the issue ID to my pull request title.
[ x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ X] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[X ] I have added documentation for the Reference Guide

@HoustonPutman If we don't put [Solr Module] in the links, then the text is plural:
This is provided via the jaegertracer-configurator Solr Modules that needs to be enabled before use., so what do you think?

@HoustonPutman If we don't put [Solr Module] in the links, then the text is plural:
This is provided via the jaegertracer-configurator Solr Modules that needs to be enabled before use., so what do you think?

Never mind, I see what you suggested.  Thanks.

Seriously thanks for the great work here! It should be much much clearer to users going forward 🙂

This ship has probably sailed, but are we going to run into issues with Solr Modules v Java Modules at some point?

This ship has probably sailed, but are we going to run into issues with Solr Modules v Java Modules at some point?

I don't think so... It would be worse if we were mainly a java library like Lucene.
But yeah, the ship has sailed IMO 😅

🚢 🚢 🚢 😄

I think we are good?   We should spin off into a new issue clarifying the docs on the gson dependency for Jaeger.   I re-requested reviews from folks, and would love to merge on Friday.


Zero Withdraw - Tweaks

Todo

[ ] @Brean0 Itemized list of changes in this PR & their rationale; add to this GitHub comment
[ ] @Brean0 Relay list of changes to Halborn in next meeting for review
[ ] @Brean0 Natspec documentation and inline comments for everything in scope of Zero Withdraw
[ ] @silochad Code review
[ ] @silochad Documentation and naming review
[ ] @pizzaman1337 Code review

This may be out of the scope of this MR, but is there a public view function that returns a boolean on if we're currently in the "morning" for a particular sunrise? one that does a check like if (block.number - s.season.sunriseBlock <= EARNED_BEAN_VESTING_BLOCKS) { and returns true/false if in the morning


Why the batch_size is 1?

I don't understand why you set the batch_size=1, i.e., only 1 picture will be trained in one iteration. Have you ever tried other batch_sizes?

I have the same doubt

You can use larger batch size if the GPU memory is enough.


Inform laxar about last moment the buffer with events was read

Fixed on master (v0.3.0) and release-0.2.x (v0.2.1).


Failed to load TOML file. Connection failed.

Hello Stellar Expert team,
I’m encountering an issue with loading my stellar.toml file on Stellar Expert. The platform shows an error:
Failed to load TOML file. Connection failed. Previously, I successfully launched test tokens on this domain, and everything worked correctly up until last week. I have not made any changes to the server configuration or the TOML file since then. Despite this, the file is no longer loading on Stellar Expert, while it remains accessible and functional on other platforms. However, I’ve tested my server and configuration extensively, and everything seems to work correctly on other platforms and through manual requests. Below are the details of my setup and the tests I performed to ensure compliance.

https://stellar.expert/explorer/public/asset/HOLIO-GDZYRVBSO4UQOVVNDW3P4TUQFQR2ZPPL7ISGHRA7SJQHKVD2MT6OOG5W-2
https://testiodnud.online/.well-known/stellar.toml
https://stellar.sui.li/testiodnud.online

Changes and Verifications Made:

Configured CORS headers to allow unrestricted access (* for Access-Control-Allow-Origin).

Ensured the MIME type for TOML is text/plain.

Tested the server for SSL/TLS compliance and time-to-first-byte metrics, all of which are within acceptable ranges.

Verified the file content and encoding for correctness.

Issue Status: Despite the correct server configuration and successful tests, Stellar Expert fails to load the stellar.toml file. I suspect there might be a caching issue or an additional validation check on Stellar Expert that isn't documented.
Please advise on any additional steps I can take or clarify if this issue could be related to platform-side caching or validation.
Thank you for your assistance! Let me know if more details are needed.

Same with new domain


Tp 1946 set second passphrase

Merge after https://github.com/unik-name/docs.uns.network/pull/28

https://spacelephant.tpondemand.com/entity/1946

Merged into https://github.com/unik-name/docs.uns.network/tree/sandbox-master_tp_1931


[SPARK-36096][CORE] Grouping exception in core/resource

What changes were proposed in this pull request?
This PR group exception messages in core/src/main/scala/org/apache/spark/resource
Why are the changes needed?
It will largely help with standardization of error messages and its maintenance.
Does this PR introduce any user-facing change?
No. Error messages remain unchanged.
How was this patch tested?
No new tests - pass all original tests to make sure it doesn't break any existing behavior.

Can one of the admins verify this patch?

cc @mridulm @cloud-fan can you review this


HLIL invalidly simplifies unsigned comparisons

This PPC assembly
addi    r0, r3, -0x12
cmplwi  r0, 0x1
ble     0x8006e5b4

incorrectly ends up as the following HLIL:
r3_2 u> 0x13

The offending transformation obtained through Function.request_debug_report("high_level_il") appears to be:
https://i.imgur.com/OaUd9Ts.png to https://i.imgur.com/GtX5tsi.png

This optimization should have been done only on equality comparisons. Fixed in build 2145.


NameError: name 'decode_predictions' is not defined

I had this error when inferring the inputs: NameError: name 'decode_predictions' is not defined.
pred = predict_bert_multitask(
        problem = problem,
        inputs = inputs,
        model_dir = model.params.ckpt_dir,
        problem_type_dict = problem_type_dict,
        processing_fn_dict = processing_fn_dict,
        decode_prediction = True
        #return_model = True
        )

How to fix, thank!

This is a bug. Thanks for reporting.


add support for QueryBuilder-map based queries in Query Packager

This is a fairly minor change which adds support to the Query Packager for QueryBuilder queries in key=value form. The key advantage of this is that QueryBuilder supports relative date range predicates (via http://docs.adobe.com/docs/en/cq/5-6-1/javadoc/com/day/cq/search/eval/RelativeDateRangePredicateEvaluator.html) something which if done just with XPath or SQL-2 would require constant updating of the query.

LGTM! Great addition - QB FTW!


Feat/sign up and login

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #2
Type of change
Please delete options that are not relevant.

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

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

[x] Manual Testing

Checklist:

[x] My changes generate no new warnings

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


⚠ Bofrost.de has degraded performance

In cd61706, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1557 ms

Resolved: Bofrost.de performance has improved in a9dfe42.


bug in resolving JSON files within module dependencies

I've made a reduced test case for this:

https://github.com/tmcw/require-bug

Requiring JSON files within modules from subdirectories of a project fails and the error message indicates that browserify is looking for a /package.json file after the json file path. I can replicate this newly in new installs of browserify 8.x so it's likely this is because of an update in one of browserify's dependencies.

bisected down to this commit in node-resolve: https://github.com/substack/node-resolve/commit/5b737d58b38ce891ef3f06d600d0562dbbc8539c

It's not limited to json files. Requiring a regular old .js file has the same problem.

Fixed in https://github.com/substack/node-resolve/commit/503c746a6e64d50f2c9b18b4476ffcfed49947f2

Cheers!


Create additional Info Blocks

Description
You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.


improve add ut for column_array

What problem does this PR solve?
this pr do :
0/ add ut cases for column_array
1/ delete convert_to_full_column_if_const and add convert_to_full_column_if_const in create action
2/ improve replicate function for column_array according ut test result, eg before we deal with decimal type with replicate_generic is much slower than now we deal with filter_number
before impl:

now impl:

Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?


Make bottom app bar safe area friendly

#16821

This is tricky. Do we really want to make the bottom app bar unconditionally fit within the safe area?
Wouldn't it be better to have the developer decided by adding a SafeArea themselves?

We'd want to keep the behaviour consistent with the bottom nav bar. It's by definition at the bottom and should have a bottom friendly layout by default.

But the BottomAppBar can be set up to accommodate the notched out unsafe areas and if we do this, we'll disable bottom app bars like that.
If we document the fact that you might want to wrap your BottomAppBar in a SafeArea, then we're supporting both use cases.

LGTM


Can Pannellum get custom plugin ?

If i want use degree of compass for create mini map plugin for pannellum.
how i can get it. or how i can create plugin for pannellum?
//Sorry for my wording. I can write English little bit
//custom plugin look like https://forgejs.org/plugins. it plugin for forge.js

now i have solve of get angle value
viewer = pannellum.viewer(...)
viewer.getPitch()
sorry for my poor question.

Were you able to answer your own question?


Video of Switch to Spring

Create a video for switching to Spring for non-producers.
Create a video for switching to Spring for producers.

completed


Matplotlib doesn’t work on mobile Safari

Need to add some styling and figure out why graph isn't appearing when ran on mobile safari.
Haven't tested other mobile browsers

I suspect this one's going to be tricky, looks like the same issue in desktop Safari:

I suspect this one's going to be tricky, looks like the same issue in desktop Safari:

Tricky sounds fun :) knowing this is an issue on safari in general narrows it down a lot.

Indeed! These are the console errors when running the default plt code:
[Error] Recursive call to fatal_error. Inner error was:
	(anonymous function) (pyodide.js:1:27994)
	(anonymous function) (pyodide.js:1:10551)
	(anonymous function)
[Error] TypeError: Invalid argument type in ToBigInt operation — pyodide.asm.js:14:2026049
	(anonymous function) (pyodide.js:1:28064)
	(anonymous function) (pyodide.js:1:10551)
	(anonymous function)
[Error] Internal error: Argument 'undefined' to hiwire.get_value is falsy (but error indicator is not set).
	(anonymous function) (pyodide.asm.js:14:234444)
	(anonymous function) (pyodide.asm.js:14:234986)
	(anonymous function)
[Error] Error: Internal error: Argument 'undefined' to hiwire.get_value is falsy (but error indicator is not set).
	(anonymous function) (pyodide.asm.js:14:234670)
	(anonymous function) (pyodide.asm.js:14:234986)
	(anonymous function)
[Error] Unhandled Promise Rejection: TypeError: Invalid argument type in ToBigInt operation
	dispatchException (main-c4f2541b93e4ae8b71f8.js:1:75462)
	(anonymous function) (main-c4f2541b93e4ae8b71f8.js:1:71659)
	r (688-5ecc4d12a9763f21465d.js:1:26179)
	c (688-5ecc4d12a9763f21465d.js:1:26419)
	promiseReactionJob

Invalid argument type in ToBigInt operation is probably our best clue

Throwing a breakpoint in at [Error] TypeError: Invalid argument type in ToBigInt operation — pyodide.asm.js:54990
Function is:
        function invoke_viiii(index, a1, a2, a3, a4) {
            var sp = stackSave();
            try {
                dynCall("viiii", index, [a1, a2, a3, a4]) // <-- this is line 54990
            } catch (e) {
                stackRestore(sp);
                if (e !== e + 0 && e !== "longjmp")
                    throw e;
                _setThrew(1, 0)
            }
        }

It gets hit first with these local variables:
a1: 8494200
a2: 33341996
a3: 33347822
a4: 0
index: 5568
sp: 8494192
this: undefined

And then it throws that error. That function might mean more to you than it does to me because I don't yet know anything about wasm :)

Great info! I'll try to dive more into this tomorrow also :)
Thanks for all the details.
Another good place to look is issues in the pyodide repo talking about safari

I did take a look at their issues and couldn't find anything that looked super close to this. Might have missed it though, seems weird if we're the first to hit this.
There are fixed Safari issues in previous versions though, so hopefully worst case we can get a fix upstream there. Eg. If you've tried the iodide notebooks in Safari they're on version 0.15.0 and fail when you try to import any package - that's been fixed in the version we're on.


Speed issues on iOS and Windows

When I set   _audioPlayer.Speed = 0.6 it works on android as expected. On iOS  _audioPlayer.Speed is set to 0.60000123 but it sounds at normal speed.  On Windows usually it takes from 2 to 10 attempts to set _audioPlayer.Speed  to desired value. For widows I use code like this:
 for (int attempt = 1; attempt <= 20; attempt++)
 {
     _audioPlayer.Speed = 0.6;

     if (Math.Round(_audioPlayer.Speed, 1) == 0.6)
     {
         break;
     }
     else
     {
         await Task.Delay(10);
     }
 }

It sounds like we have an inconsistency with the accuracy being reported back between Android and iOS which should be easy enough to fix.
As for the behaviour on Windows. Do you see an error or anything reported to the console?

Both for Android and iOS if I set _audioPlayer.Speed = 6 then _audioPlayer.Speed != 0.6. It is something like 0.60000004234. But in Android I hear slower sound. In iOS I hear the sound at original speed.
For Windows it is set exact to 0.6 but not from the first attempt. No any messages in Output window

The same issue

stackoverflow
rate must be enabled in AVAudioPlayer in order to change the speed (rate) of the audio player.
the following should be set in constructor:
player.EnableRate = true;


add ALB metrics

https://clever.atlassian.net/browse/INFRA-2272
This pulls ALB metrics into the services dashboard for services that use ALBs (currently everything in clever-dev).
The metrics pulled are identical to the ones pulled for ELBs, except for surge queue and spillover count, which don't exist for ALBs. I have used TargetConnectionErrorCount and RejectedConnectionCount in place of surge queue and spillover count, since they are somewhat analogous (http://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-cloudwatch-metrics.html#load-balancer-metric-dimensions-alb).
I've manually tested this locally

I tried spinning up grafana-service in a dev env, since I was curious to see if the ALB graphs would look. Right now they're showing 500s. Any idea what's up?
ALB graphs in "clever-dev" env show 500s:
https://n--grafana-service.int.clever.com/dashboard/script/scripted_services.js?env=clever-dev&app=hall-monitor
ELB graphs in "production" env look good
https://n--grafana-service.int.clever.com/dashboard/script/scripted_services.js?env=production&app=hall-monitor

@nathanleiby this PR requires https://github.com/Clever/ecs-metrics/pull/31
you can fake it by hardcoding an alb name and target group in addALBDashboard. You can look up example values going to cloudwatch -> metrics -> applicationelb -> grab the alb name and target group dimensions from one of the metrics there

👍 thanks for clarifying, I'll probably wait til the other one is merged.


Distrobox Gone ?

The link in the description to distrobox leads to 404 page, it seems that the  distrobox project is gone ?

@89luca89's GitHub account and all of its repositories are being flagged by GitHub for some unknown-yet reasons.
He has made a mirror on GitLab here for the time being.
Reference: https://t.me/distrobox/604

Thanks to @castrojo help, all is back to normal :-)


Updated index.html

Added my profile and fixed some formatting to keep things neat.

Me

Extra formatting

Tidyness


feat: add scroll into view to FluentDataGrid

🙋 Feature Request
I would like a ScrollIntoView() method for FluentDataGrid, if such a thing is doable.
🤔 Expected Behavior
Let's say the Items property is a List<SomeClass>. I should be able to pass in a SomeClass item/object as a parameter to ScrollIntoView() and then the item is brought into view.
😯 Current Behavior
No such method exists. User has to scroll (potentially) thousands of records to find the correct item.
💁 Possible Solution
Ideally the method signature would be something like this:
public void ScrollIntoView(T item, int columnIndex = 0, bool scrollIntoCenter = true)
It's helpful to scroll both axies, thus good if you can provide the column that needs to be brought into view as well.
It's good to scroll the item into the center screen by default. Helps the user to visually find it faster.
When using a FluentDialog panel, I had to resort to JavaScript for the scroll functionality:
        function ScrollIntoView(element) {
            if (!element) return;
            else element.scrollIntoView({ behavior: "smooth", block: "center", inline: "center" });
        }

I could then pass in a reference to a Name FluentTextField for example, that had validation errors.
But this approach does not work with FluentDataGrid, because the tabel is dynamically generated via the Items property and I don't have references to the matching <fluent-data-grid-row>.
Perhaps the JavaScript method approach would work, if FluentDataGrid would offer a converter method, eg:
public FluentDataGridRow ConvertItemToGridRow(T item)
🔦 Context
Let's say the DataGrid contains 1000 items. User modifies a couple of rows.
While saving, my validation finds some errors. I would like to scroll the first row with an error into view.
💻 Examples
You can look into WPF DataGrid for example. Although it does not offer a scroll to center functionality.

Hi,
We've received this request from others as well. Jus have not gotten round to it yet and need to investigate if it is possible. Concern I have is around Virtualized grids.
It will most probably not work with grids that use a remote data source.
So, it is on the list but no concrete plans and no time to investigate yet.

Maybe we can start by allow setting the ID of each row?
<FluentDataGrid Items="@people" RowId="@(p => $"grid-row-{p.PersonId}")">
    <PropertyColumn Property="@(p => p.PersonId)" Sortable="true" />
    <PropertyColumn Property="@(p => p.Name)" Sortable="true" />
    <PropertyColumn Property="@(p => p.BirthDate)" Format="yyyy-MM-dd" Sortable="true" />
</FluentDataGrid>