[Event Request] Codeunit 64 - Sales-Get Shipment - OnBeforeCopyItemChargeAssgnt

Dear ALAppExtensions team,
Through this issue we would like to request event publisher OnBeforeCopyItemChargeAssgnt to be added to Codeunit 64 Sales-Get Shipment with the following signature:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeCopyItemChargeAssgnt(var SalesOrderLine: Record "Sales Line"; var SalesShptLine: Record "Sales Shipment Line"; var QtyToAssign: Decimal; var QtyFactor: Decimal; var IsHandled: Boolean)
    begin
    end;

with all var parameters.
which would be called as follows from CopyItemChargeAssgnt:
    local procedure CopyItemChargeAssgnt(SalesOrderLine: Record "Sales Line"; SalesShptLine: Record "Sales Shipment Line"; QtyToAssign: Decimal; QtyFactor: Decimal)
    var
        SalesShptLine2: Record "Sales Shipment Line";
        SalesLine2: Record "Sales Line";
        ItemChargeAssgntSales: Record "Item Charge Assignment (Sales)";
        ItemChargeAssgntSales2: Record "Item Charge Assignment (Sales)";
        InsertChargeAssgnt: Boolean;
        LineQtyToAssign: Decimal;
        IsHandled: Boolean; // NEW VARIABLE
    begin
        // NEW EVENT >>
        IsHandled := false;
        OnBeforeCopyItemChargeAssgnt(SalesOrderLine, SalesShptLine, QtyToAssign, QtyFactor, IsHandled);
        if IsHandled then
            exit;
        // NEW EVENT <<

        with SalesOrderLine do begin
            ItemChargeAssgntSales.SetRange("Document Type", "Document Type");
            ItemChargeAssgntSales.SetRange("Document No.", "Document No.");
            ItemChargeAssgntSales.SetRange("Document Line No.", "Line No.");
            if ItemChargeAssgntSales.FindSet then
                repeat
                    if ItemChargeAssgntSales."Qty. to Assign" <> 0 then begin
                        ItemChargeAssgntSales2 := ItemChargeAssgntSales;
                        ItemChargeAssgntSales2."Qty. to Assign" :=
                          Round(QtyFactor * ItemChargeAssgntSales2."Qty. to Assign", UOMMgt.QtyRndPrecision);
                        SalesLine2.SetRange("Shipment No.", SalesShptLine."Document No.");
                        SalesLine2.SetRange("Shipment Line No.", SalesShptLine."Line No.");
                        if SalesLine2.FindSet then
                            repeat
                                SalesLine2.CalcFields("Qty. to Assign");
                                InsertChargeAssgnt := SalesLine2."Qty. to Assign" <> SalesLine2.Quantity;
                            until (SalesLine2.Next() = 0) or InsertChargeAssgnt;

                        if InsertChargeAssgnt then begin
                            ItemChargeAssgntSales2."Document Type" := SalesLine2."Document Type";
                            ItemChargeAssgntSales2."Document No." := SalesLine2."Document No.";
                            ItemChargeAssgntSales2."Document Line No." := SalesLine2."Line No.";
                            ItemChargeAssgntSales2."Qty. Assigned" := 0;
                            LineQtyToAssign :=
                              ItemChargeAssgntSales2."Qty. to Assign" - GetQtyAssignedInNewLine(ItemChargeAssgntSales2);
                            InsertChargeAssgnt := LineQtyToAssign <> 0;
                            if InsertChargeAssgnt then begin
                                if Abs(QtyToAssign) < Abs(LineQtyToAssign) then
                                    ItemChargeAssgntSales2."Qty. to Assign" := QtyToAssign;
                                if Abs(SalesLine2.Quantity - SalesLine2."Qty. to Assign") <
                                   Abs(LineQtyToAssign)
                                then
                                    ItemChargeAssgntSales2."Qty. to Assign" :=
                                      SalesLine2.Quantity - SalesLine2."Qty. to Assign";
                                ItemChargeAssgntSales2.Validate("Unit Cost");

                                if ItemChargeAssgntSales2."Applies-to Doc. Type" = "Document Type" then begin
                                    ItemChargeAssgntSales2."Applies-to Doc. Type" := SalesLine2."Document Type";
                                    ItemChargeAssgntSales2."Applies-to Doc. No." := SalesLine2."Document No.";
                                    SalesShptLine2.SetCurrentKey("Order No.", "Order Line No.");
                                    SalesShptLine2.SetRange("Order No.", ItemChargeAssgntSales."Applies-to Doc. No.");
                                    SalesShptLine2.SetRange("Order Line No.", ItemChargeAssgntSales."Applies-to Doc. Line No.");
                                    SalesShptLine2.SetRange(Correction, false);
                                    SalesShptLine2.SetFilter(Quantity, '<>0');
                                    if SalesShptLine2.FindFirst then begin
                                        SalesLine2.SetCurrentKey("Document Type", "Shipment No.", "Shipment Line No.");
                                        SalesLine2.SetRange("Document Type", "Document Type"::Invoice);
                                        SalesLine2.SetRange("Shipment No.", SalesShptLine2."Document No.");
                                        SalesLine2.SetRange("Shipment Line No.", SalesShptLine2."Line No.");
                                        if SalesLine2.FindFirst and (SalesLine2.Quantity <> 0) then
                                            ItemChargeAssgntSales2."Applies-to Doc. Line No." := SalesLine2."Line No."
                                        else
                                            InsertChargeAssgnt := false;
                                    end else
                                        InsertChargeAssgnt := false;
                                end;
                            end;
                        end;

                        if InsertChargeAssgnt and (ItemChargeAssgntSales2."Qty. to Assign" <> 0) then begin
                            ItemChargeAssgntSales2.Insert();
                            QtyToAssign := QtyToAssign - ItemChargeAssgntSales2."Qty. to Assign";
                        end;
                    end;
                until ItemChargeAssgntSales.Next() = 0;
        end;
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Unable to show image in notification when app is in backghround

Hi, i am sending notifications to my android app using Firebase. Device is showing image in notification when the app is in foreground. But when the app is in background, it doesn't show image in notification, it only shows simple notification.

Looks like you are sending a notification message from the console. In that case when your app is in the background you cannot dynamically update the image that is used with a notification message.
Since you cannot specify the notification icon in the console you can set the default notification icon in your app's manifest.


Modifications to antiporter (and symporter) branch

Hello,
Protégé cannot count items in a logical definitions, such that terms like  GO:0062158 chloride:proton antiporter activity, which transports a positively charged and a negatively charged ions, ends up under both GO:0015491 cation:cation antiporter activity
and GO:0015301 anion:anion antiporter activity:

I propose to get rid of the double change in the term labels (and logical definitions), and use the superclass terms

GO:0140323 solute:anion antiporter activity and
GO:0015299 solute:proton antiporter activity

Dear all,
The proposal has been made to obsolete

GO:0005451 monovalent cation:proton antiporter activity
GO:0015491 cation:cation antiporter activity
GO:0015301 anion:anion antiporter activity

The reason for obsoletion is that these unnecessary grouping classes and cause incorrect inferences (since the reasoner cannot 'count' the number of items in the logical definitions).
There are 72 manual annotations and 5 InterPro annotations, see https://docs.google.com/spreadsheets/d/1ZqlyeAS5UmwUh1MgJ9FvePR8wCfwKMoixGnruoOiz3s/edit#gid=0.
Including 5 InterPro domains
Thanks, Pascale

Created a new term to group some cation:cation antiporters
[Term]
id: GO:0140828
name: solute:metal cation antiporter activity
namespace: molecular_function
def: "Enables the transfer of a solute or solutes from one side of a membrane to the other according to the reaction: solute(out) + Na+(in) = solute(in) + Na+(out)." [GOC:pg]
intersection_of: GO:0015297 ! antiporter activity
intersection_of: has_input CHEBI:25213 ! metal cation
intersection_of: has_primary_input CHEBI:24431 ! chemical entity
created_by: pg
creation_date: 2022-05-23T14:30:46Z

The reason for obsoletion is that these are unnecessary grouping classes

But sometimes the "grouping" follows the actual biology. For example, human SLC9B2 can transport either Na+ or Li+, a function that is captured in a single reaction enabled by "monovalent cation transporter". Splitting the reaction into two, one that mediates Li+ transport enabled by GO:0010348 lithium:proton antiporter activity and one that mediates Na+ transport enabled by GO:0015385 sodium:proton antiporter activity, is straoghtforward to do but obscures a useful bit of information.
Thoughts @pgaudet?

Thanks for noting this @deustp01
The GO guidelines (that we discussed for example for the NAD/NADP case) is that if a protein catalyses two reactions (that exist in GO), we annotate both reactions. Annotating to the grouping class means that we dont know whether the activity is more specific. We can always create a term that indicates broad substrate specificity, but I dont think this works for monovalent inorganic cation (CHEBI:60242) ?
Other children (in addition to Na+ and Li+) are:

ammonium   (CHEBI:28938)
caesium(1+) (CHEBI:49547)
cobalt(1+) (CHEBI:85033)
oxidonitrogen(1+) (CHEBI:29120)
potassium(1+) (CHEBI:29103)
rubidium(1+) (CHEBI:49847)
silver(1+) (CHEBI:49468)

So, if you annotate SLC9B2 to 'monovalent inorganic cation ' it means it also transports ammonuim and potassium, which is not true.
What do you think ?
Thanks, Pascale

That being said, I agree we could keep more granularity under the 'solute' part of the antiporter -

CHEBI:23906 monoatomic cation
CHEBI:23905 monoatomic anion

That would exclude oxidonitrogen and ammonium.
Does that work ?

We can always create a term that indicates broad substrate specificity

That leads to a proliferation of term. For example, for human SLC9B2 we'd want a term for transport of Na+ and Li+. And there are a number of transporters that each act on overlapping combinations of amino acids. Despite the NAD(P) discussion, there is still a discussion needed, I think, about tradeoffs between specificity in GO MF terms and specificity in annotation. For example, a few months ago @ukemi and I were pushing for multiple catalytic activity terms to be used to annotate the activity on the same enzyme on each of a series of nucleotide derivatives, and the decision then was to use the existing more general MF term and capture the substrate details at the level of GO-CAM annotation. That implies that if an MG+F definition lists multiple substrates, they are related by logical ORs, so the objection that using the general transport MF term means the transporter must handle all of the possible substrates should be wrong, so similar use of a monovalent cation transport MF term, coupled at the level of annotation to a list of relevant substrates seems reasonable.
Of course that does not solve the counting problem by itself because we would need consistent annotation rules (to be developed and implemented) and scripts to parse combinations of MF terms and substrates out of latest-generation GO-CAM models to get reliable tallies - not trivial.
But that is a future discussion. Current usage requires that if we want SLC9B2 to transport either of two cations, we need two reactions, so I will make the change and note it in the spreadsheet.

@deustp01 I added this to next week's ontology call

Note that I did not yet obsolete "monovalent cation transporter"

But meanwhile I've re-annotated the reaction that used it, to use terms for transport of specific cations, so we don't need this particular term. What does remain, I think, is the hard issue of how fine-grained a GO MF term should be, and how to trade off granularity in MF terms against granularity in GO-CAM annotations: does coupling a gene product to a broad MF term and a specific substrate in a GO-CAM annotation unit reliably capture all the information? Can we come up with general standards for addressing this issue? @ukemi @vanaukenk

@deustp01 @pgaudet @ukemi
It looks like the specificity question wrt this ticket is on the agenda for the next ontology call.  I've also added #23378 to the agenda to discuss specificity wrt terms capturing information about specific gene products.  We do need general guidelines about this.

[x] Obsoleted GO:0005451 monovalent cation:proton antiporter activity
[x] AS A TEST: GO:0015299 solute:proton antiporter activity: changed LD from 'has primary input' some 'chemical entity' to the more narrow 'organic molecular entity'
[x] and GO:0140848
-name: amino acid:monoatomic cation antiporter activity
+name: amino acid:solute antiporter activity
-intersection_of: has_primary_input CHEBI:23906 ! monoatomic cation
-intersection_of: has_primary_input CHEBI:33709 ! amino acid
+intersection_of: has_primary_input CHEBI:35238 ! amino acid zwitterion
+intersection_of: has_primary_input CHEBI:50860 ! organic molecular entity

Some edits:
Changed amino acids to match RHEA and added RHEA xrefs when I found them:

[x] GO:0000515
-name: aspartate:glutamate, proton antiporter activity
+name: L-aspartate:L-glutamate, proton antiporter activity

[x] GO:0015496
-name: putrescine:ornithine antiporter activity
+name: putrescine:L-ornithine antiporter activity

added +xref: RHEA:28827
+xref: TC:2.A.3.2.1

[x]  GO:0043858
-name: arginine:ornithine antiporter activity
+name: L-arginine:L-ornithine antiporter activity

[x]  GO:0043862
-name: arginine:agmatine antiporter activity
+name: L-arginine:agmatine antiporter activity
+xref: RHEA:29651

[x] GO:0043872
-name: lysine:cadaverine antiporter activity
+name: L-lysine:cadaverine antiporter activity
+xref: RHEA:28895
+xref: TC:2.A.3.2.2

[x] Created grouping class
+[Term]
+id: GO:0140849
+name: amino acid derivative:cation antiporter activity
+def: "Enables the transfer of an amino acid derivative from one side of a membrane to the other according to the reaction: amino acid derivativ(out) + cation(in) = amino acid derivativ(in) + cation(out)." [GOC:curators]
+intersection_of: GO:0015298 ! solute:cation antiporter activity
+intersection_of: has_primary_input CHEBI:83821 ! amino acid derivative
+created_by: pg
+creation_date: 2022-05-31T06:01:27Z

Done in https://github.com/geneontology/go-ontology/issues/24479


question

Do you use the Euclidean distance for calculating the distance between two feature vector?

Yes!

Yes

Thank you


Gym provisioning export with odd names

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Gym provisioning profile resolution for exporting the ipa when detecting profiles from project is not clear in the documentation.
Description
Gym detects by default the provisioning profiles. The documentation states that it is possible to define the export provisioning profiles. However, the actual resolution is consistent with the specs of CodeSigningMapping. This MR only clarifies the spec for the mapping.

@googlebot I signed it!

@joshdholtz I'm glad you liked it 😁. I also want to change the documentation for the specific part of provisioning profile export, but I don't know where to do it. I had some problemas with how things work, because the name of my profiles are "non-standard", and gym behaved weird from my perspective. That behavior lead to improve this part of the spec, which actually behaves as expected per the spec, but it is not clear to the final user.


Add syslog logger support

Hi,
I want to use syslog with elasticsearch. syslog can be configure like here:
https://github.com/elasticsearch/elasticsearch/issues/527#issuecomment-553427
Best Regards
Jan

Hi,
You can supply your own logging.yml file to the module to manage those kind of things.

Closing as no feedback has been given for 3 weeks.


RLS creation

With the new "Open data model" feature in Power BI service, roles can be created in Service too

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

ID: e0ba6184-412b-7e75-ef74-46ae14d62dd6
Version Independent ID: bf97c0bc-6511-e0f0-8df3-9e4f83492fe9
Content: Comparing Power BI Desktop and the Power BI service - Power BI
Content Source: powerbi-docs/fundamentals/service-service-vs-desktop.md
Service: powerbi
Sub-service: pbi-fundamentals
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @saiprasadb365. This issue has been assigned to the doc author for review.

Why was this not picked up in the communication plan when editing in the service was rolled out?   Just sayin.

Thanks for pointing this out, @saiprasadb365.
The Venn diagram will be updated accordingly.


GEODE-1942: Limiting random strings to [A-Z] chars

This test was generating random strings, but some strings were invalid
regions names, for example strings starting with __. Limiting the
strings the alphabet.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

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

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

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@balesh2 @mhansonp


refactor: Unified build and optimize bootstrap process

Currently, each of our modules depends on the build.sh file in its own folder
This causes code duplication - each module has a similar build file. At the same time, we also have a simple build process and have written some shell scripts for it.
Now we hope to unify the build:

The build depends on a unified build script (it may be located in the build folder), so that we don't have to worry about script errors when building each module

Optimize our startup process (bootstrap), we expect a standard process like this:

Start Program -> Read Args -> Read Remote Config -> Init -> Serving

The remote configuration center address is provided by args

Now our program lacks an engineering implementation of program parameters. It only relies on a simple "flag" package. We should use a more engineering design.

Although we use a remote configuration center, the parameters mentioned above refer more to the options when the program is initialized. For example, we can set a useHook flag to decide whether to use eshook

tips: you can choose a suitable flag package or still use "flag" package

solved at #63


[8.x] Fix event discovery with custom application paths

This PR fixes an issue that was occurring when a listener's namespace was different from its path in the file system. Specifically, classFromFile could infer a wrong FQCN.
For instance, if I have a src/App/Listeners/LoginSuccessful.php listener, the inferred FQCN would be Src\App\Listeners\LoginSuccessful instead of App\Listeners\LoginSuccessful (with the first uppercase, too).
This issue is fixed by using composer.json's PSR-4 definition to replace a path by its associated namespace.

This doesn't actually seem to fix your situation when I test it manually?

This implementation works for me:
    /**
     * Extract the class name from the given file path.
     *
     * @param  \SplFileInfo  $file
     * @param  string  $basePath
     * @return string
     */
    protected static function classFromFile(SplFileInfo $file, $basePath)
    {
        $class = trim(Str::replaceFirst($basePath, '', $file->getRealPath()), DIRECTORY_SEPARATOR);

        $psr4 = Arr::get(
            json_decode(file_get_contents(app()->basePath().DIRECTORY_SEPARATOR.'composer.json'), true),
            'autoload.psr-4',
            []
        );

        return str_replace([DIRECTORY_SEPARATOR, '/'], '\\', str_replace(
            array_values($psr4), array_keys($psr4), Str::replaceLast('.php', '', $class)
        ));
    }

Not sure if it will break anything else though. Changing to reading the composer.json file is in itself probably somewhat of a breaking change.

@taylorotwell It's actually my bad, my implementation breaks with Unix paths. Yours doesn't work with Windows though, because $class will have backward slashes, so the PSR-4 replace won't work. If you fix $class before transforming it, it'll work.
- Str::replaceLast('.php', '', $class)
+ str_replace([\DIRECTORY_SEPARATOR, '/'], '/', Str::replaceLast('.php', '', $class))

Final working implementation looks like that:
$class = trim(Str::replaceFirst($basePath, '', $file->getRealPath()), \DIRECTORY_SEPARATOR);

$psr4 = Arr::get(
    json_decode(file_get_contents(app()->basePath() . \DIRECTORY_SEPARATOR . 'composer.json'), true),
    'autoload.psr-4',
    []
);

return str_replace('/', '\\', str_replace(
    array_values($psr4),
    array_keys($psr4),
    str_replace([\DIRECTORY_SEPARATOR, '/'], '/', Str::replaceLast('.php', '', $class))
));


AEMCEDS-72 button alignment in cards

Please always provide the [Jira issue(s)] your PR is for, as well as test URLs where your change can be observed (before and after):
Fixes #AEMCEDS-72
Test URLs:

Before: https://main--aem-competency--aem-comp.hlx.live

After: https://aemceds-72-button-alignment--aem-competency--aem-comp.hlx.live

[ ]  If you are adding a new block or a variation to an existing block, please fill below:
Block library path: NA

@shashanksing29 Layout is broken

@mkbansal1 This is an authoring issue with this card, an extra  tag containing '`' symbol is there which is disturbing button alignment. We have to fix this card authoring.


v0.1.2

Current dependencies on/for this PR:

main

PR #75   👈

This comment was auto-generated by Graphite.


Event filter + dialogs cause the application to crash

Using Quamash 0.6.1.
Consider the following program, that opens a dialog window within the main event loop, and uses a global event filter:
#!/usr/bin/env python3
import asyncio
import sys
import typing as T

from PyQt5 import QtCore, QtWidgets
from quamash import QEventLoop

async def show_error(msg: str) -> int:
    dialog = QtWidgets.QMessageBox()
    dialog.setIcon(QtWidgets.QMessageBox.Critical)
    dialog.setWindowTitle("Error")
    dialog.setText(msg)
    future: "asyncio.Future[int]" = asyncio.Future()
    dialog.finished.connect(lambda result: future.set_result(result))
    dialog.open()
    return await future

class Window(QtWidgets.QWidget):
    def __init__(self) -> None:
        super().__init__()
        self.setLayout(QtWidgets.QHBoxLayout())
        self.button = QtWidgets.QPushButton("Ready", parent=self)
        self.button.clicked.connect(self.on_button_click)
        self.layout().addWidget(self.button)
        QtWidgets.QApplication.instance().installEventFilter(self)

    def on_button_click(self) -> None:
        asyncio.ensure_future(show_error("test"))

    def eventFilter(
        self, source: QtCore.QObject, event: QtCore.QEvent
    ) -> bool:
        print(source, event, event.type())
        return False

def main() -> None:
    app = QtWidgets.QApplication(sys.argv)
    loop = QEventLoop(app)
    asyncio.set_event_loop(loop)
    window = Window()
    window.show()
    with loop:
        loop.run_forever()

if __name__ == "__main__":
    main()

Here's what happens – I'm getting a rather cryptic error in Python's internals:

StopIteration: 1024

The above exception was the direct cause of the following exception:

SystemError: <class 'PyQt5.QtWidgets.QWidget'> returned a result with an error set
zsh: abort (core dumped)  python3 test

Figured it out. My issue was caused by QMessageBox having set no parent widget. Passing window to it has fixed the problem.


poetry installation "Unable to find installation candidates for nvidia-cudnn-cu11"

When installing the package using peotry on a device that does not have cuda installed I get the error:
 Unable to find installation candidates for nvidia-cudnn-cu11
since the package works on CPU as well is it possible to fix this install issue so that i don't have to have a gpu enabled device to install the package.

Thanks for reporting. I am not very familiar with poetry, but it looks like this is due to the dependency on torch, which we can't remove.
Here is a relevant thread:
https://github.com/python-poetry/poetry/issues/6939
Scanning that thread, it appears that a fix for torch has been merged, but perhaps it has not yet made it to pypi.

I believe the torch 1.13.1 version that should have the fix has indeed been posted to PyPI: https://pypi.org/project/torch/

The requirements.txt is only used in my continuous integration, so I don't think it will make a difference in the poetry installation. PyMDE imposes no max versions on torch/torchvision, so you shouldn't need to do anything special to use their latest versions.

I've never used poetry before, but I scanned their docs just now. Is my understanding correct in that you have a poetry project, and PyMDE is one of its dependencies? Then typing poetry install fails when it tries to install PyMDE?
If so, can you add an explicit dependencies on using the latest torch and torchvision versions in your pyproject.toml? Maybe just figuring out which versions are required, or how to specify them, will be most of the work.

In the meantime, if you post the entire output from the poetry command, as well as your entire pyproject.toml file, I might be able to help further.

Ha!  I didn't realize that the latest version of torch had just been posted to PyPI yesterday.  I ran the poetry install today, and that seems to have fixed everything.  Woohoo!


Fix: Cannot de-select tag/skill in search page when coming from profile.

Fixes Issue
Closes #7340
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Go to a profile

Select any tag.

Try de-selecting it and it should work.

Note to reviewers

It only improves consistency between the urls, how does it fix th deselect issue?

@krishstwt have you gone through the source code?

Yes I had a brief look, it is just a question I am curious about

It only improves consistency between the urls, how does it fix th deselect issue?

@krishstwt have you gone through the source code?

I think the inconsistency was the problem, when going to the search page it did not match the case, so could not be unselected
Thank you for your contribution @ribhav-pahuja and fixing the bug 👍
Btw just a small recommendation, it is better to write a helpful reply to questions on your PR, after all, open source and pull requests are about collaboration over the code. Plus, a future employer might see this sort of reply and make a negative decision based on it.

@eddiejaoude gotcha! I will keep this in mind.
Also, thanks for merging the PR!


Re #751

Due to the rewrite we need to impl this again

Not as hard though, just need to support pi types

#751


Autocomplete for String Union Types

Let's say I have the following type:
 type Cars = 'Ferrari' | 'Volvo' | 'Porsche';

It will be awesome if I can treat it like an enum, meaning I can write Cars.       and get intellisense for Ferari, Volve, and Porsche.
Today, you do not get that.
You can have funky workarounds to make the number based enum become a string
Like this (runnable sample):
enum Cars {
	Ferrari = <any> "Ferrari",
	Volvo = <any> "Volvo",
	Porsche = <any> "Porsche"
}

But that's hacking, even though it opens the question why not make this a core feature.
Or, you can use namespaces, like (runnable sample):
type Cars = 'Ferrari' | 'Volvo' | 'Porsche';
namespace Cars {
	export const Ferrari : Cars = "Ferrari";
	export const Volvo : Cars = "Volvo";
	export const Porsche : Cars = "Porsche";
}

But obviously, this way you can get into errors, with both the type and namespace potentially going out of sync due to human error.
Ideally, the very first line should in itself allow you to have autocomplete, similar to what enum does. Or alternatively of course support string enum types.

But that's hacking, even though it opens the question why not make this a core feature.

that is the plan. we need to come up with a detailed proposal. we have talked about this in the past in https://github.com/Microsoft/TypeScript/issues/5740 and https://github.com/Microsoft/TypeScript/issues/5943.
the feature is tracked by https://github.com/Microsoft/TypeScript/issues/1206
For auto completions in IDE's this already coverd by https://github.com/Microsoft/TypeScript/issues/8428

unless this is more to this request that allowing string enums, i would close this in favor of https://github.com/Microsoft/TypeScript/issues/1206.


system admin option to reset domain last modified timestamp

if we need to make sure a domain is synced from zts to zms, this option would allow system admins to change the last modified timestamp thus marking the domain as changed rather than asking the domain admin to make some change that is not necessary in the domain.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

👍


DLedgerServer class Code format problem

@Override annotation should  above the method

5 class has this code format problem


Fix typo in readme.

YMAL -> YAML

This isn't helpful because it's based off the command line output.

Thanks! Looks great now 👍

20:01  [iPhoenix] Due to your accidental transposition of the inner letters of that acronym, user experience and trust are decreased substantially. Studies  say users do not trust documents with misspellings, and it drags the user out of their project. With these commits, I fix your issue and restore faith and domestic tranquility and promise to preserve, protect, and defend convimg.


[Regression] vi_mode segment doesn't work in next

vi_mode in next always shows INSERT.
How to reproduce:
First with master branch:
docker run -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 -e TERM=$TERM -it --rm ubuntu bash -uexc '
  apt update && apt install -y zsh git
  git clone -b master https://github.com/bhilburn/powerlevel9k.git ~/powerlevel9k
  echo "
    POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=()
    POWERLEVEL9K_RIGHT_PROMPT_ELEMENTS=(vi_mode)
    source ~/powerlevel9k/powerlevel9k.zsh-theme
    bindkey -v" >~/.zshrc
  zsh -i'

Press ESC. Observe that INSERT on the right changes to NORMAL. Press i and it goes back to INSERT. This is expected behavior.
Now replace master with next in the command and observe that the prompt always shows INSERT.

Hey @romkatv !
Thanks for the reports (this one and #1196). I scheduled both for v0.7.0, but I am out of time to fix them for the next days. So if anyone wants to take over, I'd be happy.

This diff should fix vi_mode. But if the second option of [tmp] is implemented this would be unnecessary.
diff --git a/segments/vi_mode/vi_mode.p9k b/segments/vi_mode/vi_mode.p9k
index b9911a3..79425a8 100644
--- a/segments/vi_mode/vi_mode.p9k
+++ b/segments/vi_mode/vi_mode.p9k
@@ -232,6 +232,8 @@ function zle-keymap-select {
       *)          cursorShape ${P9K_CURSOR_SHAPE_DEFAULT};;
     esac
   fi
+  __p9k_prepare_prompts
+  zle .reset-prompt
 }

I will create a PR anyway since atm the segment is completely broken and background_jobs "merely" has an inconsistency that is harder to fix than this (for me at least).

I just noticed that Powerlevel9k in next overrides zle-line-init. This will break configs of people who use the documented recipe for making vicmd the default keymap.
From http://zsh.sourceforge.net/Doc/Release/Zsh-Line-Editor.html:

The following example puts the line editor into vi command mode when it starts up.
zle-line-init() { zle -K vicmd; }
zle -N zle-line-init

I don't know how many people use it, maybe none. There is https://gist.github.com/dbb/3893091, which seems to do the same thing.
If users put their config before sourcing powerlevel9k, it just won't work. If they put it after, then prompt will always get rendered twice, becoming twice as slow. On the first render it'll show INSERT, and then on the second render (after refreshing everything, including vcs), it'll show NORMAL.
This makes vi_mode in next more broken than in master. Both branches have the double rendering issue, and both cause jarring freezes each time keymap changes (a very common operation for vi keymap users), but only next can override user's zle-line-init so that it has no effect.
FWIW, Powerlevel10k has flawless vi_mode segment. No zle-line-init hijacking, no double rendering, no freezes, instant switching.

@romkatv thx, I don't intend to rewrite the segment so I'll stick with my solution. But if anyone wants to do it, feel free.

I merged #1232 yesterday. So this issue is fixed for now. Thanks @romkatv and @Syphdias

I am experiencing similar behavior. vi_mode display string remains as POWERLEVEL9K_VI_INSERT_MODE_STRING value.  Currently using the master branch at 3dafd79 of Powerlevel9k.
POWERLEVEL9K_MODE='nerdfont-complete'
ZSH_THEME="powerlevel9k/powerlevel9k"
...
plugins=(git vi-mode jenv fzf ripgrep colored-man-pages gradle rbenv conda-zsh-completion zsh-completions)
...
source $ZSH/oh-my-zsh.sh
...
POWERLEVEL9K_VI_INSERT_MODE_STRING="\ue3aa"
POWERLEVEL9K_VI_COMMAND_MODE_STRING="\u26A1"
...
# LEFT PROMPT
POWERLEVEL9K_LEFT_PROMPT_ELEMENTS=(vi_mode dir vcs dir_writable)

POWERLEVEL9K_VI_MODE_INSERT_BACKGROUND="deepskyblue2"
POWERLEVEL9K_VI_MODE_INSERT_FOREGROUND="gold3"
POWERLEVEL9K_VI_MODE_INSERT_VISUAL_IDENTIFIER_COLOR="black"
POWERLEVEL9K_VI_MODE_NORMAL_BACKGROUND="deepskyblue2"
POWERLEVEL9K_VI_MODE_NORMAL_FOREGROUND="gold3"

Please provide guidance on this issue.


Improve the installation docs

Issue Description
All first time users need to download the CRDB binary to connect to their cluster (whether on CRDB or CC, this is usually a required step), but if you are not an experienced developer our download docs are extremely confusing. This page makes it seem like you need to follow every single step listed, but in reality, you would just be downloading the binary a bunch of times in different ways. In addition to running into this problem personally when I first interviewed at CRL, we've recently seen CC interviewees who are Hackathon students get totally tripped up by this page.
Suggested Resolution
Add a sentence at the top of the page "Use one of these methods to install the CockroachDB binary". Label each download method as an option, eg "Option 1: Install with Homebrew", "Option 2: Download the Binary", "Option 3: Use Kuberenets", etc.
Doing this will make it much clearer that all of these steps are not required and are in fact different variations of the same thing.

@emily-horing, we've rarely heard that the install page is "extremely confusing". On the contrary, when I look at "Was this page helpful?" responses since the beginning of February, across all install pages, we received 12 positive submissions and just 3 negative submissions. I'm not against helping users understand that the page contains "options", either through intro text or heading language, but do you have any examples to support that the current approach is very confusing to developers?

In any case, @emily-horing, I just opened a PR with an easy fix, I think. Assigned you as reviewer: https://github.com/cockroachdb/docs/pull/10180

I can share a video with you from a recent user session -- I think for people who are newer to development (which we are getting more of based on our survey data!) it will help a lot to clarify. "Extremely" may be a bit extreme... probably just channeling the frustration I had when I first went through it ~2 years ago :)


VTT | For big period (~13 days) result from connector is not ready in 1 min

1.Create product for VTT data Source
2. Fetch product with a relevant period, many days ( from what I tried smth 13days+ make this behavior reproducible)
Result
When connector takes more than 1 min to return response, broker shoots the 502 or 504 response.

Wanted
Investigate which component requires so much time? Data source or connector? Could we improve the time response? or maybe the values for period should be limited to smth smaller?

The data source is extremely slow.

@artoranta Data source is extremely slow so can we run some background worker to call and store the data in caching layer?

@artoranta Data source is extremely slow so can we run some background worker to call and store the data in caching layer?


priority support for http gw peers

There is no support for a fallback mode, when the second peer will only be used if the first is unreachable.
Please add a priority parameter, the same way as SRV record does. It also be nice to have a support for peers configuration via SRV DNS record.

It seems this issue should be moved to https://github.com/nspcc-dev/neofs-sdk-go because the pool must be change to support this feature.


add "Sanatize tag option values" rule

Summary
Proposed changes:

add "Sanatize tag option values" rule (incl. summary, why, how and examples)

Checklist

[X] The changes only affect or contain one style guide rule or section.
[X] The changes are in style guide format.
[X] The new style guide section has an entry in the Table of Contents (only if changes introduce new section).
[X] The changes can be merged into the target branch without conflicts.

I find the update /  reduce example rather difficult. Especially when looking at the implementation example <output-sum numbers="{ [1, 2, 3, 4] }" /> <!-- outputs: 10 -->. Looking at this implementation, I assume that you put in a list (and expect that list to be preserved). However, that list ends up being a number. Which is quite the contrary to what the rule is actually dictating.
Can't we extend the first recommended block like:
this.items = opts.items || []; // default to empty list
this.score = Number(opts.score) || 0;  // default to 0
this.foo = this.foo || 'bar'; // default to 'bar'

I get what you want to explain in the example, but maybe we can change the naming from:
<output-sum total-score="{ [1, 2, 3, 4] }" /> <!-- outputs: 10 -->

Ah... I completely misread the example (twice). I did not see the naming of the tag and the sum tag property.
But I still think the example is not clear. This is a rule about option values. For this example to make sense, you need to know that the element is outputting a single value.

I think this rule has nothing to do with Riot, but with basic programming practices. I would scratch it entirely
On 21 Mar 2016 09:45 +0100, Peter<EMAIL_ADDRESS>wrote:

Ah... I completely misread the example (twice). I did not see the naming of the tag and thesumtag property.
But I still think the example is not clear. This is a rule about option values. For this example to make sense, you need to know that the element is outputting a single value.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly orview it on GitHub(https://github.com/voorhoede/riotjs-style-guide/pull/36#issuecomment-199179469)

@phortuin I'd say it's a generic software pattern (defensive programming) specifically applied to Riot. I've now seen issues in multiple projects which could have been avoided by using this pattern. Since to me the guide aims to help more robust Riot apps which are predictable in their use for other developers as well, I do think it's relevant.
Perhaps we can rephrase it to:
## Harness your tag options

In Riot your tag options are your API. A robust and predictable API makes your tags easy to use by other developers. Therefore you should **harness your tag options**.

...
How? --> defaults, type conversion, ...

@petergoes if we are going through with this PR (pending discussion with @phortuin) I agree that we can simplify the example. It was just one I still had lying around: http://plnkr.co/edit/RvPuBP?p=preview . I do think that the naming fits the component though as it "outputs the sum of the given numbers".

That sounds reasonable, esp with the argument that the tag is your API. I would still keep it fairly generic, with a simple sanitization example for typing & fallback (something like const score = parseInt(opts.score, 10) and const fruits = opts.fruits || []). Even then, it's dangerous to explain too much before having to write a book on this (because what happens if opts.fruits is not an array in the before example, etc)

I don't think there is anything wrong with the naming of the rule and its meaning. I found just the example a little confusing. I don't see the need to rephrasing it.
Maybe we can have another rule explaining the your-tag-is-your-api thing?

Hm interesting, I wanted to state that explaining 'tag as api' is not necessary as it has become a common paradigm thanks to Angular mostly, but when I google HTML as API and everything related, I get nothing. So I still think that it shouldn’t be part of this guide, but maybe we need to rephrase 'the tag is your api' and then write a blog post about this paradigm and the cons/pros?

I added a new question #37 so we can discuss the HTML as API thing.
How do we proceed with the Sanatize tag option values rule?

@phortuin I also updated this PR based on feedback of you and @petergoes: https://github.com/voorhoede/riotjs-style-guide/pull/36/commits/00e0a9cfb2fdf57d853f29f3dc7ed9654284a5a6


[WIP] Enable validation for migrations

Adds DRUPAL_DEFAULT_MIGRATIONS_VALIDATE environment variable for switching validation for migrations on or off

By default, it is true

Adds use_env function for test suites to add environment variables to .env, scoped to a single test suite

This is used to disable validation for most test suites, via use_env "DRUPAL_DEFAULT_MIGRATIONS_VALIDATE=false"
The only test suite performing migrations that does not disable validation is 10-migration-backend-tests

Fixes some invalid CSVs in 10-migration-backend-tests

Note:  The process of cleaning up CSVs in 10-migration-backend-tests is incomplete!  I can't figure out the remaining failure(s?)
Remaining work
/10-migration-backend-tests/migrations/media-image.csv results in a validation failure.  as far as I can tell, the migration module does not tell you exactly which constraint is violated, but merely recites the list of constraints.  I suspect this is may be failing due to permissions.
The error is:
media_img_00001 | 1 | [media]: field_media_image.0=You do not have  access to the referenced entity (file: 531).\|\|field_media_image.0=Only files  with the following extensions are allowed: png gif jpg jpeg tif tiff jp2.

To reproduce the error

do make up.
do make test test=10-migration-backend-tests.  You'll see it terminate with a migration error.
Go to https://islandora-idc.traefik.me/admin/structure/migrate/manage/idc_ingest/migrations/idc_ingest_media_image/messages and look at the error message

I suspect it's a permissions error.  If so, this is probably a bug, because ingests are being performed by admin.
Please fix this PR, solicit review, and merge it in my absence!
Resolves jhu-idc/idc-general#368

The reason media validation is failing is because the file_import call in the migration file defaults a uid to 0 (anonymous user!) if it's not specified.   That code can be seen here:  https://git.drupalcode.org/project/migrate_file/-/blob/2.0.x/src/Plugin/migrate/process/FileImport.php#L45
Not sure why it doesn't just use the current user's uid.
To fix this:

we can set the uid to some static id in the file.  1 is Admin.  All the files will be imported as this particular user then.  Maybe that's okay.
maybe use another way to move files around?

I tried a quick test to change the UID with no success. I will try again tomorrow.

There were two things so far that I've changed.  I added uid: constants/ADMIN to the file_import parameter list. Otherwise it tries to add the file using uid 0, which is an anonymous user. This doesn't pass validation.
The other change is using the actual file name, versus the media file's name. The file_managed table is a lot happier with that in there and Files do actually delete now via the UI.
I think there are more things to fix, so this is just a start.

I also rebased on development, to bring in the unique id work on nodes.

So when validate: true is enabled, we cannot run ingests more than once because of the unique_id constraint.    Not sure what to do about this yet.

This PR should fix the issue I was seeing in running migrations multiple times:  https://github.com/jhu-idc/idc-isle-dc/pull/198.  We should be able to do that again and I'll write a test to specifically test that.

I've gotten this a lot farther along and am having trouble reproducing the last few errors in tests locally.  I did not do the TEST_ENVIRONMENT variable setup.  I'm going to try one more thing and then shift back to the unique_id work so I leave behind some PRs to be looked at next week to keep pushing the stable data format forward.


Control point placement by line length ratio

https://github.com/tangrams/tangram/pull/441

Resolved when https://github.com/tangrams/tangram-docs/pull/163 is merged.


Adding support for default values in FindInMap

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/intrinsic-function-reference-findinmap-enhancements.html

Thanks! Merged locally to fix formatting via 7533fbabcf3e51b0ad5785884c2d5508fe301414


Vanished After PF2 Update

Howdy. First off, love this mod. Full automation kind of takes away from the flat check experience, so this mod is perfect.
I think I may have just upgraded my PF2 system a little further than the mod is ready for, but since updating I no longer see PF2 Flat Check Helper in the modules list and it's not in my chat window.
So this is probably a "wait for the update and it'll work again" situation, but I'm finding myself a little lost in the Foundry modules and systems updates world lol.

I'm running the latest foundry and pf2e system version, and it's working fine. There's no maximum core/system version in the manifest, so it really should not disappear after an update.

Okay, it's probably a conflict with another mod that updated then. I'll isolate and find it! Thank you!

Just a follow up in case anyone else runs into the issue. 1. The name change threw me off, I wasn't finding the mod because I was looking for the wrong name LOL. and 2. I had to uncheck the "Show Flat Check buttons" option and reload, then check it again to make it reappear. Works fine now.

No idea how I didn't spot this while testing, but the "Enable flat check buttons" now works correctly.

Lol ain’t coding fun. And thanks!

Whoops

Whoops


lnwire: simplify fuzz targets

Reduces code duplication and fluff.
A pure refactor built on top of https://github.com/lightningnetwork/lnd/pull/9251.

Needs a rebase, then I'll merge.

Rebased.


Memory leak

ERROR: Allowed memory size of 536870912 bytes exhausted (tried to allocate 53248 bytes) {"userId":1,"exception":"[object] (Symfony\Component\ErrorHandler\Error\FatalError(code: 0): Allowed memory size of 536870912 bytes exhausted (tried to allocate 53248 bytes) at /home/...redacted.../public_html/vendor/intervention/image/src/Intervention/Image/Gd/Decoder.php:33)
[stacktrace]
#0 {main}
"}
The line in question is $core = @imagecreatefrompng($path);

Seems to be related with the setting of PHP's memory limit.


mesonpy: add heuristics to try mapping unknown install paths to wheel

This will map files that do not use {py_purelib}/{py_platlib}
placeholders, which is the case of targets that receive the install
location as a string in the Meson build files (eg. configure_file).
Signed-off-by: Filipe Laíns<EMAIL_ADDRESS>

Blocked by #9288, as Meson will still apply the hardcoded Debian handling on the Github Actions Python, which does not have the Debian modifications. Meson will try to install to dist-packages.

Verified that the logic works when we disable the Debian Python check.


Update Set-MsolCompanySettings.md

Added an advisory note to the -UsersPermissionToReadOtherUsersEnabled parameter to warn of implications of setting the boolean value to $False. Setting this to false will result in tenant wide issues where Team owners can no longer add Members to the teams they are Owners of.

@jebalert
Per the read me file:
Please be aware that if you have any updates to automatically generated reference content, you should create an issue and not a PR. The issue will then be created into a bug and triaged accordingly. We do this because the reference content must mirror the *dll-help.xml file and subsequent updates to the reference content may overwrite any changes made through the repo. Capturing it in the source code ensures that this does not occur. Any PR with changes to automatically generated reference content will be closed.
Automatically generated reference content is any of the content under this Reference node, such as Get-AzureADUser
This only applies to reference content and not conceptual content. Pull requests for changes to conceptual content are accepted and encouraged. An example of the conceptual content is the Overview or a scenario such as importing data.
Thank you!
Bill

@jebalert
Per the read me file:
Please be aware that if you have any updates to automatically generated reference content, you should create an issue and not a PR. The issue will then be created into a bug and triaged accordingly. We do this because the reference content must mirror the *dll-help.xml file and subsequent updates to the reference content may overwrite any changes made through the repo. Capturing it in the source code ensures that this does not occur. Any PR with changes to automatically generated reference content will be closed.
Automatically generated reference content is any of the content under this Reference node, such as Get-AzureADUser
This only applies to reference content and not conceptual content. Pull requests for changes to conceptual content are accepted and encouraged. An example of the conceptual content is the Overview or a scenario such as importing data.
Thank you!
Bill


Bot down, again.

I don’t know why this keeps happening, is it a monthly thing?

Sorry about the lack of response. I made an error that necessitated restarting the bot. Updates like these are posted in the bot's support server, available from the help or feedback command.


Handle gaps in the MovingWindow data

Provide an interface for handling gaps

the user should be able to tell what to do with gaps, e.g. replace with constant number, bfill ffill, interpolate
should be able to get a list of all gaps

Accessing behavior of uninitialized or missing data
We need to check if data is potentially uninitialized in the __getitem__
method and define what to do in that case.
The check can be performed by using the ringbuffers gaps. Thus we'd need to
check if the requested data subwindow intersects with any interval stored as gap.
possible actions when uninitialized data is accessed:

raise an exception
return the biggest subset

Gaps here refers to the gaps found in the OrderedRingBuffer class and how they can be handled

@cwasicki can we close this issue?

Do you remember who requested these features?

the user should be able to tell what to do with gaps, e.g. replace with constant number,

This is available now.

bfill ffill, interpolate

This is not supported but could be done by the user on the array.

should be able to get a list of all gaps

This request was just closed: https://github.com/frequenz-floss/frequenz-sdk-python/issues/558

Do you remember who requested these features?

I can't remember but might come from myself :). I'll close the issue now.


Documentation : proof reading

The dev guide would need some proof reading and some more work (atom/enaml and also a git section)

Closing following #65


Failed to create MOF files

After typing .\Build.ps1 -ResolveDependency no MOF has been created.
I got this type of error for each MOF:
DSCFile01 : DSCFile01 : MOF__0.0.0 NA
    DSCFile01 : FileServer ::> FilesAndFolders .....................................................OK
    DSCFile01 : FileServer ::> RegistryValues ......................................................OK
    DSCFile01 : FileServer ::> SecurityBase ........................................................OK
    DSCFile01 : FileServer ::> WindowsFeatures .....................................................OK
    DSCFile01 : FileServer ::> NetworkIpConfiguration ..............................................OK
    DSCFile01 : FileServer ::> DscLcmController ....................................................OK
    DSCFile01 : FileServer ::> DscLcmMaintenanceWindows ............................................OK
    DSCFile01 : FileServer ::> DscTagging ..........................................................FAILED
        Compilation Error: System.Management.Automation.ParentContainsErrorRecordException: You cannot call a method on a null-valued expression..Message
Error occured during compilation of node 'DSCFile01' : Compilation errors occurred while processing configuration 'RootConfiguration'. Please review the errors reported in error stream and modify your configuration code appropriately.
Compilation errors occurred while processing configuration 'RootConfiguration'. Please review the errors reported in error stream and modify your configuration code appropriately.
At C:\Windows\System32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917 char:5
+     throw $ErrorRecord
+     ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (RootConfiguration:String) [], InvalidOperationException
    + FullyQualifiedErrorId : FailToProcessConfiguration
`PSDesiredStateConfiguration\Configuration : You cannot call a method on a null-valued expression.` `At C:\Temp\DscWorkshop-dev\DscWorkshop-dev\DSC\DscConfigurations\CommonTasks\0.2.85\DSCResources\DscTagging\DscTagging.schema.psm1:1 char:1` `+ Configuration DscTagging {` `+ ~~~~~~~~~~~~~~~~~~~~~~~~~~` `    + CategoryInfo          : InvalidOperation: (:) [Configuration], ParentContainsErrorRecordException` `    + FullyQualifiedErrorId : InvokeMethodOnNull,Configuration`
Exception calling "InvokeWithContext" with "2" argument(s): "You cannot call a method on a null-valued expression."
At C:\Windows\System32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2232 char:13
+             $result = $Body.InvokeWithContext($functionsToDefine, $va ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : RuntimeException

Can you give us more details about how you called the script?
I have just cloned the repo, started the build script with 'ResolveDependency' and it worked right away. However, I remember to have fixed something that look like the error you are getting.
For now, please use the code in the dev branch. I am going to merge the latest working version to master very soon.

Hi Raymund!
Here is what I've done:

Downloaded the repository
Copied/pasted cmdlets from step 3. Setup your laptop
Launched .build.ps1 -ResolveDependency

Ok, waiting for your new master branch...
No hurry, it's just to learn how to use Datum.

Hi Luc,
Both branches should have a working version now. Can you give it another try, please?

Luc, while derlivering the DSC workshop @ PSConf.asia, we have seen the same issue that you are running to. It happens, if you download the project and not clone it. Please use
git clone https://github.com/automatedlab/dscworkshop

Working in a git repository is required because of this line: https://github.com/AutomatedLab/CommonTasks/blob/cf0973a3f62acd661a4eb201fc550c2792f5a771/CommonTasks/DscResources/DscTagging/DscTagging.schema.psm1#L34

It works fine with the command line.
Thanks Raymund!
I leave the issue opened as a reminder so you can decide if you want to improve the code to take into account the possibility to download the project or if you prefer to remove this possibility from the 2.Git clone your fork locally section...

Good point, we will need to rework the main readme anyway. it is quite old and a lot has changed in the project. I leave the issue open until this is done.

This is now fixed in the 'DscTagging' resource. If the folder is not a Git repo, it write 'NoGitRepo' for the Commit Id.


slug for signals and hubs

This feature is not in the scope of current project, so we will leave to future versions.


Polyfill path silently fails with Netflix test data

Data used is here: http://download.opencontent.netflix.com/?prefix=AV1/Chimera/AVIF/
This decodes fine in Chrome 74 (which has native support) and Firefox 66 (with AV1 enabled), but fails on Safari (Safari 12.1, in this case). Console did not leave any useful backtraces.

Thanks, I will look into it.

Btw, are you sure you didn't try 10-bit files? That's currently not supported.

I should have mentioned that detail - I tried both 8 and 10. Both don't decode.

In safari, Sometimes rendering is scattered and It is displaying error of out of memory
[Error] FetchEvent.respondWith received an error: Error:
Out of bounds memory access (evaluating 'this.FFI.djs_alloc_obu(e.byteLength)')

[Error] Failed to load resource: FetchEvent.respondWith received an error:
Error: Out of bounds memory access (evaluating 'this.FFI.djs_alloc_obu(e.byteLength)')
(A13502.1bf9b45c.avif, line 0)


OutofMemoryException

Thanks for this greet job!
I have got a problem when i start to use the new version( I used the old version on codeplex before)
it throws outofmemory exception when I used old version text2vec tool generated vector.bin file.if there are a way to convert it to new version.
WordEMWrapFeaturizer funcion
model.LoadBinaryModel(filename);
i found that there is no
bw.Write(0);//no VQ
in the old version,maybe this rises the exception,is there any solutions for this,I donot want to retrain the word_vector.bin.

Yes, model format was changed for VQ which can reduce model size significantly, so Txt2Vec model is not compatible with the previous one.
To avoid retraining word_vector.bin, you could revert Txt2Vec model load code (model.LoadBinaryModl(...)), and still use the previous version without VQ. And then compile RNNSharp by using modified Txt2Vec.dll, then it should work.


🐛 Added logic to avoid updating the updated_at field when migrating

no issue

we have to explicitly reset the updated_at field, because Bookshelf auto-updates this field on each update
detect and respect options.migrating

@kevinansfield Don't merge it, i am adding a test.

@kevinansfield Ready to test :)


Option to invert the returned value for binary_sensors

This adds an option to invert the returned value of a binary_sensors. I needed this to report the value of a dry contact checking if my garage door is closed or not.
Thanks for this integration, it works well for me, just needed this little change :)

Thanks for the review, I addressed your comments and added an entry in the CHANGELOG.

I pushed a commit to enhance readme and set invert value in parent class

can you test with the last modifications and tell me if everything is working ? (I don't have a V4 now to test)

Just tested your changes, they work fine.

thanks @ydubreuil


Legitify

@Flet

+1

My goal with this repo (and the reason for the naughty find/replace) was to have the only deviation be semicolons with all other changes automatically inherited from the standard module.
This was done explicity to avoid having a full fork that could potentially deviate further from the standard. Also this avoids maintainence for the most part, falling back on the dependency (not a perfect way to do it, but still less work).
Thinking this one over...

Check out that second commit, it contains just the semicolon change plus some trolling in the README. I'm thinking we can keep rebasing it in top of the latest standard, so we keep the same versions and the same small changeset. Check out the commit message for details on how this works.

PS re "automatically inherited from the standard module", we'd have to publish this module anyway every time standard is published if we want the versions to stay in sync.

we'd have to publish this module anyway every time standard is published if we want the versions to stay in sync

This is not quite true. If the dependency is 2.x.x it will be resolved to the latest minor version at the time  the user installs (I agree this is not perfect).
In any case, the only other solutions I could think of to make this cleaner involve @feross changing standard in some way... for example if standard had an option to provide a different path for rc or had some way of overlaying rc files or overriding individual rules..
Would you be oppose to a PR that added an option to override the path to rc, @feross?


Fix android TransactionTooLargeException

Fix the following:
https://github.com/Telerik-Verified-Plugins/ImagePicker/issues/137
https://github.com/Telerik-Verified-Plugins/ImagePicker/issues/107
https://github.com/Telerik-Verified-Plugins/ImagePicker/issues/45

@EddyVerbruggen
Would you merge this and release a new version? This is quite an issue in android actually. Thanks.

@hkjeffchan Sure thing, you can now grab 2.3.0 from npm.
And thanks, on behalf of the users of this plugin! 🤝


The price filter is not working properly

Bagisto Version: Master
Steps To reproduce--
1. Log in as a user.
2. Search product by category.
3. Try to filter the products by price filter..
Expected result: The price filter should be work properly.
Actual result: The price filter is not working properly. Please find the screenshot Here

Hi @Sanjaybhattwebkul,
Please share the full steps.
How you reproduce, which product you have added.

It's working fine and I am not able to reproduce this.

@devansh-webkul
Please find the screen recording Here to reproduce this issue


Quick readgroup question

Quick question: assuming that I want to feed BAMs with one read group per BAM into bcbio, but I want the samples to be named (and associated with the metadata) by the basename of the BAM file rather than by the name of the read group, is there a way to enforce that? Ignoring the read group all together and just taking all reads from the BAM would work as well. BAMs will be re-mapped by bcbio and I am aiming for variant2 with bwa+vardict, if that makes a difference.

Sven-Eric;
Are you running into an issue during the templating step? I'm not sure if I totally understand but with a BAM bcbio will auto assign the description from the existing read group, but then also give you the chance to over-write that to whatever you want through the CSV metdata:
https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/workflow/template.py#L358
so it should use either the original description or the base BAM name, whatever you put in the first column of your metadata CSV file.
For the actual run, bcbio uses the read group specified in description, if realigning, and does not keep the original read group.
Does that answer your question? Let me know if not and happy to discuss more.

I see; so if a fill in the description field in the metadata CSV then the read group name will never be used within bcbio? That's exactly how I want it, thanks. Feel free to close.

Belay that, another question since I have your attention: if I use BAMs as inputs, and want to remap, will bcbio also split the mapping across multiple compute nodes (as it does with the bgzipped FASTQs by streaming batches of FASTQ records to multiple mapper processes)?

That's exactly right. The only time we use an existing read group is if you feel a pre-aligned BAM file and don't do alignment or have bam_clean: fixrg turned on. Glad this works for you.

Sven-Eric;
It will split mapping as normal. The first step is to convert BAMs into bgzipped fastqs then they get processed as normal. So for realigning BAM inputs should be no different than fastq. Hope this helps.

Excellent, thanks a bunch. So it always does the right thing whatever you throw at it. I should have known that, naturally ;)


Flash and render target clearing

https://github.com/aduros/flambe/blob/master/src/flambe/platform/flash/Stage3DBatcher.hx#L263
Log.warn("Changing render target, clearing it first as required by Stage3D");
_context3D.clear(0, 0, 0, 0); // Required :(

There is no need to clear the render target in AIR 15/FP 15 and up. This limitation can now be fixed:
http://blogs.adobe.com/flashplayer/2014/09/relaxing-render-target-clear-operation-in-stage3d.html

Prior to Flash Player 15.0 and AIR 15.0, there was a strict restriction that the clear function must be called on a render target before drawing, so that the content in a render texture does not get preserved after invoking the clear function. In Flash Player 15.0 and AIR 15.0, we have optimized the old internal rendering system and have removed this requirement.

Thanks!

It wouldn't be too hard to detect which Flash Player version the browser (or AIR) is at runtime. Then you could just disable the clear() call if you're running the right Flash version with something like this:
var majorVersion :Int = Std.parseInt(flash.system.Capabilities.version.split(" ")[1].split(",")[0]);
clearContexts = (majorVersion < 15); // Clearing rendering target is not needed in Flash / AIR 15 and up.


x/crypto/ssh: ssh_message_ignore (message type 2) causing an error on Dial()

What version of Go are you using (go version)?
go1.7 darwin/amd64
What operating system and processor architecture are you using (go env)?
darwin, amd64
What did you do?
attempted to connect to an ssh server via /x/crypto/ssh using ssh.Dial()
What did you expect to see?
A successful connection.
What did you see instead?
unexpected message type 2 (expected one of [31])
RFC4253 states "all implementations must understand and subsequently ignore this message at any time" but the package is treating it as an invalid message type, causing the transmission attempt to fail. (see https://tools.ietf.org/html/rfc4253#page-26)

We're running into this server-side as well. The issue appears to be that in the handshake transport, it's not ignored properly, as during the handshake, there are multiple messages exchanged in a specific order. However, it looks like it's valid for msgIgnore to come between those.
At the end of enterKeyExchangeLocked we use t.conn.readPacket which can return msgIgnore. As an example, in (t *handshakeTransport) readLoop() msgIgnore is ignored as expected, but when we fall back to the underlying connection, we still get those messages.

CC @agl @hanwen

There is one caveat in that the requirement to accept this message type is only levied after the identification string is sent which should happen after the connection is established. I don't think that is the case here since the message type it is expecting is related to key exchanging. Key exchange comes after the identification string  has been sent.

I'm not as familiar with this part of the protocol (yet) but it seems like the identification string is always the first line sent and it's not a part of a packet. So, anywhere packets are sent and received, msgIgnore can show up.

I feel like it should be fine to ignore msgIgnore at the transport level... unless there's any pressing need, couldn't t.conn.readPacket just never return msgIgnore?
Or does that mean other stuff has to be refactored because we translate messages into msgIgnore to drop them in at least one instance.

you should be able to drop in the code that decrypts a packet.

I encounter the same issue when I use crypto/ssh for client side.
Thanks @belak and @treehau5, I made a hacking and solved my problem now.
My solution is doing a retry the t.conn.readPacket every time it return the msgIgnore.
I would like to contribute my code, but I don't think it is a good way to fix.

couldn't t.conn.readPacket just never return msgIgnore?

I feel it was a good suggestion, had voted +1.

you should be able to drop in the code that decrypts a packet

could you explain more?

It looks like this should have been closed a few days ago. https://github.com/golang/crypto/commit/7c6cc321c680f03b9ef0764448e780704f486b51


I can't download any images or files in Pake apps

Search before asking

[X] 我在 issues 列表中搜索，没有找到类似的内容。
I searched in the issues and found nothing similar.

Pake version
latest
System version
macbook pro m2
Node.js version
18.20
Minimal reproduce step
Create any app, try download an image
What did you expect to see?
download in my downloads folder
What did you see instead?
Nothing happens
Anything else?
it's sad because now i cannot use pake for any apps
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

did you try to use the name field in lowercase. I had some issues similar when I didn't write the name field in lowercase(deepblue)


Regex

Watching my Udemy Course has really helped learn about regex, however I wonder how common this will be in my work when I finally get a job

Brennan and I had some very different answers


As soon as i add this package i get the following error

this is right after when i add readsms package. Can you help?
The supplied phased action failed with an exception.
A problem occurred configuring project ':readsms'.
Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.
If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Let me see, Maybe some issue with AGP version.

Can you also post the output of flutter doctor


provision.py: Don't create py2 venv in py3 mode.

When running tools/provision.py in python3 mode, we used to create a python2 venv called zulip-py2-twisted-venv. This was needed because Zulip couldn't run tools/run-dev.py in python3. So we switched to
this virtualenv when running tools/run-dev.py.
Now that Zulip can run tools/run-dev.py in python3, we don't need to create this virtualenv anymore.

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

Good catch!  Merged, thanks @sharmaeklavya2!


Add markdown to HTML conversion with syntax highlighting

Codecov Report
Attention: Patch coverage is 85.24590% with 9 lines in your changes missing coverage. Please review.

Project coverage is 88.21%. Comparing base (2fcfa03) to head (ad00d04).

Files with missing lines
Patch %
Lines

src/index.ts
85.00%
9 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   87.88%   88.21%   +0.33%
==========================================
  Files           6        6
  Lines         553      611      +58
  Branches       49       52       +3
==========================================
+ Hits          486      539      +53
- Misses         66       71       +5
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Fixes #8


Add new ballista-cli crate

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/2493 and https://github.com/apache/arrow-datafusion/issues/2433
Rationale for this change

Allows us to release DataFusion and Ballista separately. This was not practical before because the datafusion-cli depended on ballista which in turn depended on datafusion, leading to issues such as https://github.com/apache/arrow-datafusion/issues/2433)
Allows DataFusion and Ballista CLIs to evolve separately without Ballista adding development overhead to the DataFusion CLI

Here is the new crate dependency diagram:

What changes are included in this PR?

[x] Create new ballista-cli crate by copying all of datafusion-cli crate
[x] Remove ballista features from datafusion-cli crate
[x] Update dependency diagram
[x] Refactor to have ballista-cli re-use some of the code from datafusion-cli instead of duplicating all of the code
[x] Update CI scripts
[x] Update user guide

Are there any user-facing changes?

Yes

Thanks @andygrove  looks much better to me

Thanks for the review @alamb. I pushed another commit to fix the datafusion-cli lock file that had conflicts after merging from master and also added the ballista-cli lock file that was previously ignored. I also updated the CI integration test to build the ballista-cli.

AMD64 tests failed with No space left on device ... trying again

AMD64 tests failed with No space left on device 3 times in a row now.

@alamb All PR builds are failing on AMD64 with no space left on device. Any idea what we need to do to resolve?

@alamb All PR builds are failing on AMD64 with no space left on device. Any idea what we need to do to resolve?
Perhaps we can try to update the cache-key in https://github.com/apache/arrow-datafusion/blob/master/.github/workflows/rust.yml#L55 to see if something about the old caches is too large?

Thanks. I suppose this cache just continues to grow forever and doesn't clear out old versions of deps that we no longer use? Perhaps we should include the month name in the cache so it resets periodically?

@alamb Moving the ballista tests to a separate job fixes this. Ok to merge?


feat(fake-workloads): WIP Enable fake workloads in OCP

Description

This enables the fake workloads in OCP
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test ?

/test gke-scale-tests
/test gke-perf-scale-tests

Could you elaborate in the PR description why these changes lead to enabling fake workloads in OCP?
I'm not very familiar with fake workloads, so I could use some context to understand how they tie into the bigger picture

@Maddosaurus I've updated the description with the why it was failing and why it doesn't fail now 🙂


Running a cell (Ctrl+Enter) should put us in Control mode

Expected behaviour
After running a cell with Ctrl + Enter, we can type b or a to add a new cell.
Actual behaviour
b or a is typed inside the cell
Steps to reproduce:

While editing a cell, press Ctrl+Enter
Press a or b

Environment data

VS Code version: 1.57.0-insider
Jupyter Extension version (available under the Extensions sidebar): v2021.6.914258744
Python Extension version (available under the Extensions sidebar): v2021.6.908196464-dev (pylance v2021.6.1-pre.1)
OS (Windows | Mac | Linux distro) and version: Local Mac, Remote Ubuntu 18.04
Python and/or Anaconda version: N/A
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Remote

Yes, ctrl+enter executes the cell, but doesn't put in control mode.
This is for code cells.
For markdown, I cannot get them to render with Shift/Ctrl + Enter, but this is already covered in the other issue. My workaround to get them to render is to switch it to code (y), then back to MD (m), with an esc in between since y puts us in edit mode.

validated


Cannot use createUser

Hi, I am developing an app and I am not able to use this:
User user =<EMAIL_ADDRESS>"password");
Is the correct way to use it? I can make the login but cant create users that way?
Thanks

Hi @cingolanifede, According to https://loopback.io/doc/en/lb2/Android-SDK.html, I think it does create a user for you if you follow the steps in the tutorial (it extends the LoopBack base User model). Can you provide a link to a test project with steps to reproduce the issue? See https://github.com/strongloop/loopback/wiki/Reporting-issues#bug-report. For future reference, please post questions at https://groups.google.com/forum/#!forum/loopbackjs. See https://github.com/strongloop/loopback/wiki/Questions for more details.

Closing this ticket. @cingolanifede If you are still experiencing this issue, feel free to re-open this issue with more info and we can investigate further.


Added Black Hole Number Calculator

Fixes Issue
Closes #729
Description👨💻
Added a folder in Calculators named Black-Hole-Number-Calculator containing HTML, CSS & JS file.
Type of change📄

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

How this has been tested✅
Tested with different number inputs.
Checklist✅

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added demonstration in the form of GIF/video file
[X] I am an Open Source Contributor

Screenshots/GIF📷

Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳🎉 @nilamkumari11


[EventHubs] fix tracing bug

fixes bug in: #27986
changes:

removed duplicate tracing which was adding the Azure.EventHubs.Message span twice when sending a batch message/list of messages.
adding the tracing info the uamqp/pyamqp.Message application_properties rather than event_data. Ensures that the diagnostic ID is actually being added on the outgoing Message, rather than the EventData after outgoing Message has already been created.
Adding the FakeSpan class to tests folder to test telemetry.

TODO:

follow-up issue created to address implementation so it behaves correctly in a separate PR: #28141

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

perf run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2091776&view=logs&j=42ded549-05ee-5cdb-7ba7-7a948a0cc056&t=a103fa30-9714-51c6-dfba-9bd5390ed023

/azp run python - eventhub - tests


OpenXR Toolkit 1.1.0 not working for me

I am using version 1.1.0 as I am using a Pimax 8kx. I cannot get DCS or MSFS to show up in the box in order to enable them. I also get an error when i try to open the logfile. The logfile is NOT present in the AppData/Local fiel that is suggested to manuallly check. Can anyone help? I know its not a bug as this version works for others. Thanks.

I somehow muddled through and got it to work. No idea how

I now get an error when lauching DCS with a CTD.

XR_APILAYER_NOVENDOR_toolkit.log

These errors are from OpenComposite. The log file you attached seems older than the error, so I think DCS crashed even before OpenXR Toolkit (you can delete the log file and try again to confirm).

I just tried deleting a log and it prevented me from making another (error message), so i restored it but with the older info. How do  I delete and try again?

I just tried deleting a log and it prevented me from making another (error message)

This confirms that the issue is with OpenComposite and not OpenXR Toolkit. There isn't anything I can do. I don't write OpenComposite.

I see from your log that you are using Pimax-OpenXR. I really don't think the issue is with OpenXR Toolkit and you've showed yourself that OpenXR Toolkit is not invoked.
The issue might be with Pimax-OpenXR, and that's the log file I would need.

dcs.20220608-062427.crash.txt
Does this help?

No but this would: https://github.com/mbucchia/Pimax-OpenXR/wiki/Capturing-debug-traces

The cmd prompt says it can't find the file path, even though I can see it right there in the folder

Actually I think I can make it work by modifying the file path

These instructions are for v0.1.1 that is not public yet.
In the command line replace C:\Program files\PimaxXR\... with c:\program files\Pimax-OpenXR\.... Otherwise wait for version 0.1.1.

It says I cannot do it. The file path i had to use is below:
wpr -start "c:\Program Files\Pimax-OpenXR\PimaxOpenXR.wprp" -filemode

It's telling you that you have a trace running, because you already started the command. Try wpr -cancel then retry from the beginning.

It actually worked, it said the trace was saved. Where do I find the file though?

in the folder you opened the prompt with... whch looks to be c:\windows\system32 (try elsewhere next time so you don't mess up your system files!)

Good to know, I didn't realise that's what was happening.

This confirms that the error happens outside of my code. So I can't do much. Have you tried deleting the D3DCOMPILER_47.DLL file from your DCS\bin folder?

I have not, but as far as I can tell it is a required file that I had to paste in with 2 other files. But I will try and see what happens. Thanks for trying to help. It did work last night and managed to change some settings with the OpenXR menu. Could I have broke it with a setting in there?

Try safe mode in Companion app then, but according to your log it's not getting anywhere near.
Also, if you had to copy D3DCompiler_47.dll, it means you are using a super old deprecated version ( OC-ACC )
That version is not supported anymore by the author of OC.

Ok, so the tutorial I used provided the files because its an older version of the toolkit that works with the Pimax. So I guess a newer compiler wont work with the older version of the toolkit? I am using version 1.1.0 which according to the tutorial is the latest version that is compatible with the Pimax.
It was working for a while between closing the the initial thread and reopening again. I just can't see why it would suddenly stop working.

Ok, so the tutorial I used provided the files because its an older version of the toolkit that works with the Pimax

No you are mixing up OpenXR Toolkit and OpenComposite. Please do not use OpenComposite from the Jabbah repository as it is deprecated.

It was working for a while between closing the the initial thread and reopening again. I just can't see why it would suddenly stop working.

I asked you to try to disable OpenXR Toolkit or use Safe mode. You have not answered those.
At this point, I am inclined to close this issue because none of the code of OpenXR Toolkit nor Pimax-OpenXR is being invoked according to your logs. I cannot debug issues not in my code. Please get in touch with the OpenComposite folks.

This is the tutorial I used and where I got the links. At the 2 minute marks he says you cannot use a later version of the toolkit.

I have been telling you to use a newer OpenComposite not OpenXR Toolkit. You don't seem to grasp that these are 2 different things, developed by different people.
Here is the OpenComposite site:
https://gitlab.com/znixian/OpenOVR/-/tree/openxr

disabling the Toolkit allows DCS to launch with STEAM.

But you are no longer trying to do this? You have since then installed Pimax-OpenXR which gets rid of Steam, so your answer is outdated. My question is right now does it work when you disable OpenXR Toolkit?
I do not know what you issue is. Error -32 is returned by the OpenXR Loader, a product that I also do not develop.
I'm sorry you are having issues but I've been through this same process 100 of times now:

things work. yay
next day, things no longer work. People claim things just like you "It was working for a while [...]. I just can't see why it would suddenly stop working."
I spend hours investigating
Eventually 2-3 days later, people come back to me: I found the issue, it was XYZ user error, I changed a setting, I installed a new driver etc.

So for short, if it worked, and it no longer works now, you need to find the one thing you have changed and that broke it.
My final recommendation to you is to try to reinstall this: https://aka.ms/vs/17/release/vc_redist.x64.exe as it is a common source of error -32.
But for your next step, you are going to need to ask the OpenComposite folks, as I am not able to help with issues in products that I do not develop.

Ok. Thank you for your time, I really do appreciate it. I'll continue on according to your recommendations.


Sub-commands for benchmark

Changes:

:point_right: Breaking: The pallet benchmarking was moved to the benchmark pallet sub-command.
Move shared code into a shared module
Add a test for benchmark block
Unify docs of sub-commands

Many of the changes are just moved files, the interesting part is in node/cli/src/.
The output of the benchmark command now looks like this:
Sub-commands concerned with benchmarking. The pallet benchmarking moved to the `pallet` sub-command

USAGE:
    substrate benchmark <SUBCOMMAND>

OPTIONS:
    -h, --help       Print help information
    -V, --version    Print version information

SUBCOMMANDS:
    block       Benchmark the execution time of historic blocks
    help        Print this message or the help of the given subcommand(s)
    overhead    Benchmark the execution overhead per-block and per-extrinsic
    pallet      Benchmark the extrinsic weight of FRAME Pallets
    storage     Benchmark the storage speed of a chain snapshot

Polkadot companion: https://github.com/paritytech/polkadot/pull/5247
Closes #11140, #11153

bot rebase

bot merge


Lose the ref to OBI in the definition of definition

From<EMAIL_ADDRESS>on May 05, 2011 11:17:31
http://purl.obolibrary.org/obo/IAO_0000115 currently begins: "The official OBI definition..."  If this is to be a generally used term then the reference to OBI needs to go.
Original issue: http://code.google.com/p/information-artifact-ontology/issues/detail?id=109

From<EMAIL_ADDRESS>on May 05, 2011 09:01:15
Had been fixed in trunk - see http://information-artifact-ontology.googlecode.com/svn/trunk/src/ontology/ontology-metadata.owl and https://code.google.com/p/information-artifact-ontology/issues/detail?id=38#c1 Will be pushed with next release.
Status: Done


Fix warning during compiling tests.

Motivation:
When compiling the tests the following warning is printed:
swift-nio/Tests/NIOTests/SelectorTest.swift:361:43: warning: result of call to 'wait()' is unused
}.wait().forEach { return try! $0.wait() } as Void)
Modifications:
Ignore return value to silent the compiler warnings.
Result:
No more warnings.

@swift-nio-bot test this please

@swift-nio-bot test this please

@Lukasa addressed


Ditch TopoJSON support?

@mourner: I also had some thoughts about whether I made a mistake by pushing TopoJSON support instead of keeping things simple and limited to GeoJSON
The size compression benefits are good but it makes the format significantly more complex, this will harm adoption rate
well, not very complex currently, but it'll be much more complex when we make it streamable
@tmcw: hm, i think yes, we should dump it.
i'm divided on topojson because of the dual purpose of topology in it
like, a really good open source implementation of a topology-supporting geometry system... super useful
but topojson is mainly doing it to save bytes

@mourner What about a separate implementation for topojson instead?  Seems like both benefit from the building blocks established here, but both geometry systems don't need to realized in the same container...  Could be a separate project in a separate repo if needed for better isolation.
Would having separate implementations help reduce the complexity of each to a tolerable level?  Or put differently, is the aggregate complexity of doing both here in the same container higher than the sum of the complexities of doing each individually?

Separate repo may be a good option, since the two repos could be developed at a different pace then, while we could focus on perfecting an awesome + simple GeoJSON compression that has higher chances of adoption.

Removed and feeling pretty good about it — makes iterating on the spec so much easier, and simpler is pretty much always better. Topobuf would be a good name for a project this branches out into. I also think that sum of complexities of two different projects will be lower than trying to squash two pretty different formats into one spec.

So is anyone currently developping a "Topobuf" project? It looks like no one is maintaining a .proto file for topojson since it has been ditched here, which is sad.

@ramsestom I needed it myself for a project, so I've forked the 1.0 version of this library with geojson support removed.
It's on npm as topobuf and hosted on Github https://github.com/azavea/topobuf/


x/net/http2: requests experience excessive time delays when encountering network errors

If the client loses its connection to the server, and multiple requests are sent. There would be only one dial start connecting to the server. If the dial timeout due to network issue, only the first request who creates the dial would be finished, and the other requests would continue trying to connect to the server even if their context has been canceled due to timeout.
For getClientConn multi request would creat one call to dial server.
func (p *clientConnPool) getClientConn(req *http.Request, addr string, dialOnMiss bool) (*ClientConn, error) {
	//...
	for {
                 //....

		// if there is no active connection
		traceGetConn(req, addr)
		// for a specific addr, only one call would be created.
		call := p.getStartDialLocked(req.Context(), addr)
		p.mu.Unlock()
		<-call.done

		// If should retry dial for the request, which do not respect to request's context.
		if shouldRetryDial(call, req) {
			continue
		}
		cc, err := call.res, call.err
		if err != nil {
			return nil, err
		}
		if cc.ReserveNewRequest() {
			return cc, nil
		}
	}
}

For shouldRetryDial function, even if the request's context has been canceled, the result can be true.
func shouldRetryDial(call *dialCall, req *http.Request) bool {
	if call.err == nil {
		// No error, no need to retry
		return false
	}
	if call.ctx == req.Context() {
		// If the call has the same context as the request, the dial
		// should not be retried, since any cancellation will have come
		// from this request.
		return false
	}
	if !errors.Is(call.err, context.Canceled) && !errors.Is(call.err, context.DeadlineExceeded) {
		// If the call error is not because of a context cancellation or a deadline expiry,
		// the dial should not be retried.
		return false
	}
	// Only retry if the error is a context cancellation error or deadline expiry
	// and the context associated with the call was canceled or expired.
	return call.ctx.Err() != nil
}

@Taction Do you have a minimal example that reproduces this issue?

CC @neild

@hardikmodha Sorry for the late reply. The scenario I've encountered is when there is a network issue, the request consumes more time than the request timeout + dial timeout. I created a test to simulate this situation(also committed in my pr).
http2/clientconn_test.go
// TestConnectTimeout tests that a request does not exceed request timeout + dial timeout
func TestConnectTimeout(t *testing.T) {
	tr := &Transport{
		DialTLSContext: func(ctx context.Context, network, addr string, cfg *tls.Config) (net.Conn, error) {
			// mock a net dialler with 1s timeout, encountering network issue
			// keeping dialing until timeout
			var dialer = net.Dialer{Timeout: time.Duration(-1)}
			select {
			case <-time.After(time.Second):
			case <-ctx.Done():
			}
			return dialer.DialContext(ctx, network, addr)
		},
		AllowHTTP: true,
	}

	var sg sync.WaitGroup
	parentCtx, cancel := context.WithCancel(context.Background())
	defer cancel()

	for j := 0; j < 2; j++ {
		sg.Add(1)
		go func() {
			for i := 0; i < 10000; i++ {
				sg.Add(1)
				go func() {
					ctx, _ := context.WithTimeout(parentCtx, time.Second)
					req, err := http.NewRequestWithContext(ctx, "GET", "http://<IP_ADDRESS>:80", nil)
					if err != nil {
						t.Errorf("NewRequest: %v", err)
					}

					start := time.Now()
					tr.RoundTrip(req)
					duration := time.Since(start)
					// duration should not exceed request timeout + dial timeout
					if duration > 2*time.Second {
						t.Errorf("RoundTrip took %s; want <2s", duration.String())
					}
					sg.Done()
				}()
				time.Sleep(1 * time.Millisecond)
			}
			sg.Done()
		}()
	}
	sg.Wait()
}


Any release notes or changeset?

I can't find any release notes or changeset. Am I missing something?

https://github.com/tannerlinsley/react-virtual/releases


Add in tests to check for overflow in unbounded window [databricks]

This fixes #4084
The actual fix went into CUDF, but this adds in a test to show that it is working.

build


Revert "Provide service-monitor and application-model"

Reverts vespa-engine/vespa#3615
This broke 6.161.2

👍


checking isDragging from persisted this.selected drag state

Fixes #1410.
In onMouseMove, the isDragging check was previously conditioned on hasBeenDragged, which is recomputed every mouse move event.  Now, it is conditioned on this.selected.hasBeenDragged which is set the first time the user's mouse is more than 0.02 away from the drag start position.

Closed by https://github.com/KittyCAD/modeling-app/pull/4341


private-bamoe-issues#1461: add content in documentation forms code-gen cli tool

Closes: https://github.com/bamoe/private-bamoe-issues/issues/1461

@LightGuard thank you for the review, feedback and hints, all should be incorporated now.

PR updated, feedback incorporated


Replace repo name for containerd

Hi
The repo name for containerd seems to be bad on https://containerd.io/docs/
Cheers

LGTM

Thanks @WTFKr0 !


puppeteer.launch error

ProcessException (ProcessException: Operation not permitted
Command: .local-chrome/113.0.5672.63/chrome-mac-x64/Google Chrome for Testing.app/Contents/MacOS/Google Chrome for Testing --allow-pre-commit-input --disable-background-networking --disable-background-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-background-pages --disable-component-update --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=Translate,BackForwardCache,AcceptCHFrame,MediaRouter,OptimizationHints,DIPS --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --enable-automation --enable-blink-features=IdleDetection --enable-features=NetworkServiceInProcess2 --export-tagged-pdf --force-color-profile=srgb --metrics-recording-only --no-first-run --password-store=basic --use-mock-keychain --headless --hide-scrollbars --mute-audio about:blank --remote-debugging-port=0 --user-data-dir=/var/folders/dy/q8b1npfn7d978xwc68vh4tp00000gn/T/com.example.yudi/puppeteer_dev_profile-FJ3WIl)
----- flutter version 3.10.0 ------
I modified this code, there was another error "Websocket url not found", help me?
"< key  >com.apple.security.app-sandbox< /key >< false />"

@zsbox Can you share a full project that I can run and see the error by myself?

@xvrh sorry，I think I get it, this plugin only supports dart running, and does not support chrome in-app operation, right, for example, I click a button?

It worked for me on debug mode but when I export the macOS app, that was the error:
creation failed, path = .local-chrome

Getting the same error

import 'dart:developer';

import 'package:flutter/material.dart';
import 'package:puppeteer/puppeteer.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(),
      home: const HomeView(),
    );
  }
}

class HomeView extends StatelessWidget {
  const HomeView({super.key});

  @override
  Widget build(BuildContext context) {
    Future run() async {
      try {
        // Download the Chrome binaries, launch it and connect to the "DevTools"
        var browser = await puppeteer.launch();

        // Open a new tab
        var myPage = await browser.newPage();

        // Go to a page and wait to be fully loaded
        await myPage.goto('https://dart.dev', wait: Until.networkIdle);

        // Do something... See other examples
        await myPage.screenshot();
        await myPage.pdf();
        await myPage.evaluate<String>('() => document.title');

        // Gracefully close the browser's process
        await browser.close();
      } catch (e) {
        log(e.toString());
      }
    }

    return Scaffold(
      body: Center(
        child: ElevatedButton(
          child: const Text("Run"),
          onPressed: () => run(),
        ),
      ),
    );
  }
}

@Shiba-Kar where do you run the app? MacOS? iOS? Android?

@Shiba-Kar where do you run the app? MacOS? iOS? Android?
macOS(darwin)

[✓] Flutter (Channel stable, 3.13.6, on macOS 14.0 23A344 darwin-arm64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.83.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

@Shiba-Kar did you disable sandbox?
https://github.com/xvrh/puppeteer-dart/issues/154#issuecomment-944487927
Additionally you can use already installed chrome path:
final browser = await puppeteer.launch(executablePath: BrowserPath.chrome);


Store the state of the table view

Store the table state in the querystring as it changes, and load it when the page first loads.
This requires changing the municipalities filter value to by IDs, rather than objects, so that they're easier to work with.

Merge #61 into this before merging this.

Ok, this can be merged when someone has had a look and is happy. @jbothma