Setting TreeExplainer Base Value to 0: possible? How?

Hi,
First thank u very much for this awesome library!
I am interested on the TreeExplainer described here https://github.com/slundberg/shap

My question would be if it is possible to set in advance the "Base Value" to 0?
In the example mentioned (with Boston DS) what i would like to achieve it ll be the values that added sum the output from scratch, instead from the "Base Value". Is that possible? How?

Hey! The base value is just expected value of the model. So you could
change the number of samples recorded in the tree (the training dataset
counts for the splits) or perhaps easier you could just draw a bar from
zero to the base value and name it “bias term” or “base rate bias” or
something similar.
On Sun, Sep 9, 2018 at 4:22 PM Bakkat2<EMAIL_ADDRESS>wrote:

Hi,
First thank u very much for this awesome library!
I am interested on the TreeExplainer described here
https://github.com/slundberg/shap
[image: image]
https://user-images.githubusercontent.com/3450005/45269990-ae175600-b46d-11e8-9018-6a60081f12d1.png
My question would be if it is possible to set in advance the "Base Value"
to 0?
In the example mentioned (with Boston DS) what i would like to achieve it
ll be the values that added sum the output from scratch, instead from the
"Base Value". Is that possible? How?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/slundberg/shap/issues/257, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADkTxTzbZOmv62UqF1aPuFgj9Q1zAvlPks5uZaKXgaJpZM4WggYq
.

Thanks for your answer, and great to know it is possible :D
Sorry, but I did not get how to do it, u mean that i have to pass the "bias term" as a parameter of the

?

I mean attach a bias term as another column of the shap_values matrix that is equal to explainer.expected_value, then just pass 0 for the expected value when calling force_plot

Thank u very much! I ll try that.

HI,
I have tried what you sugested, but i think is not what i was looking for.
Sorry, i think i did not explain myself properly.
What i would need is that when the "TreeExplainer" fuction is called, and the contribution of each parameter to the output is calculated, the base value used was 0.
Would that be possible?
thank u very much

Could you post code that shows what you did? That might help clarify things.

The way the SHAP values are computed in the code assumes we are using
conditional expectations, so they will always sum up across all features to
the difference in the model's mean output and current output (there is not
a way to simply change the definition of the conditional expectations to
sum from 0). You can however for plotting purposes change the "base value"
to 0 or 1 and then add the difference between the expected_value and 0 or 1
and an extra bar in the plot. That's the closest thing I can think of.
On Sat, Nov 10, 2018 at 7:27 AM Bakkat2<EMAIL_ADDRESS>wrote:

Please, don t mind. I really preciate your answers. They are very
helpfull! Thank you very much for your time, i mean it.
I got it, its clear that theres a part of the output that is not explained
by the variables and is razonable to have an expected value.
But if i am not wrong , you define the expected value in advance (i mean
before finding the shap values) as the mean of the target in the training
data set.
What i would like to do is to set it in advance as 0, or as 1, or another
number and check the shap values generated. Probably this would be
theorically wrong, since i would be asuming that the variables are capable
of explaining the hole output (or a part arbitrary choosen), but I would
like to use them, even under those strong asumptions.
Would that be possible?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/slundberg/shap/issues/257#issuecomment-437591593, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADkTxX7WrXL7_4jLUeD6fDC5UOw8f-qmks5utvBUgaJpZM4WggYq
.

Just wanted to leave a note here:
@slundberg's  solution works very well. Attached is an example of the output along with the complete code I used.

  shap.force_plot(
        0, # set base value to zero
        np.hstack([shap_values[0, :], explainer.expected_value]), # add expected value as a bar in the force plot
        pd.concat(
            [c_features, pd.DataFrame({"Base value": [explainer.expected_value]})],
            axis=1,
        ).iloc[0, :], # grabs the identities and values of components
        text_rotation=30, # easier to read
        matplotlib=True,
        show=False, # allows saving, etc.
    )

The features are whited out because they contain potentially sensitive info, not because there was a problem with SHAP's output. Also, in this example "base value" is obscured because "output value" is too close, but this is also something we can fix later for presentation.

Has someone figured a way to change the base value to 0 so that the Shap values sum is equal to the predicted value?

I think you can set the background, the model will set the basevalue based on the background.If the background has only one sample x, then the predicted value y of that sample is the basevalue. But I haven't found a way to change the basevalue directly.


Prebuilt Classification Utils

To add here after init review: test case histograms/barplots/ROC/confusion matrix.

Fixed in https://github.com/kolenaIO/kolena/pull/155


Adding docs for manage synonym tab

Adding docs for manage synonym tab

Docs Build status updates of commit dd85316:
:clock10: Pending: waiting for processors (32 builds ahead of you)
:warning: Docs Build is busy, currently there are 32 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: MSFTChao sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/manage-synonyms.md
67
45
31
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

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

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
67
45
31
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 3898ff0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
67
43
31
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 5884a2e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
86
67
13
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 70cf9a6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
86
73
12
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 9215e52:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
95
84
4
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 874f4a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/intent-classifier.md
90
78
15
link
:white_check_mark:

enterprise-graph/manage-synonyms.md
86
65
11
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 277859c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:bulb:Suggestion
View
Details

enterprise-graph/intent-classifier.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-named-entity.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/add-utterance-radio-buttons.png
:white_check_mark:Succeeded
View

enterprise-graph/media/intent-classifier-tutorial/ic-summary-metrics.png
:white_check_mark:Succeeded
View

enterprise-graph/manage-synonyms.md

[Suggestion] Invalid value for ms.author: 'MSFTChao' is not a valid Microsoft alias. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/manage-synonyms.md
86
65
11
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

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

File
Status
Preview URL
Details

enterprise-graph/manage-synonyms.md
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym2.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/add-synonym3.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/manage-synonym.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/synonym_example.png
:white_check_mark:Succeeded
View

enterprise-graph/media/conflation-synonym/tab-list.png
:white_check_mark:Succeeded
View

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


Updating Portal Accelerator MDFC VM Vulnerability Assessment to use mdeTVM by default

Overview/Summary
Updating Portal Accelerator MDFC VM Vulnerability Assessment to use mdeTVM by default
This PR fixes/adds/changes/removes

Changed the recommended default to Microsoft Defender vulnerability management:

Breaking Changes

N/A

Testing Evidence
As above
Testing URLs
Azure Public

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Ensured contribution guidance is followed.
[X] Updated relevant and associated documentation.
[X] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

For completeness lets also update these:

https://github.com/Springstone/Enterprise-Scale/blob/31505/eslzArm/eslzArm.json#L143
https://github.com/Springstone/Enterprise-Scale/blob/31505/eslzArm/managementGroupTemplates/policyAssignments/DINE-MDFCConfigPolicyAssignment.json#L57

I don't know what I was thinking... wow. Sorry about that. All updated.


Support for detecting if an object is "connected" to Qml in some way.

public static class Qml
{
    static bool IsConnected(this object instance)
    {
        // Return true if the Qml is currently alive in Qml in any way, otherwise false.
    }
}

This should be done before #7.
This will require us to track .NET instances that are currently alive in the Qml world.
This is the rough idea:
We store the instances in .NET with a Dictionary<object, NetInstance>. When sending an object to Qml, we need to check if an existing NetInstance already exists in the dictionary and use it, otherwise create one. Within the deconstructor of NetInstance, we need to reach out to the .NET world and remove the instance from our global dictionary.
IsConnected will simple do a instance.ContainsKey to determine if object is currently wrapped in Qml/V8 somewhere.

no longer needed?

Yeah, not sure the use case anymore. I was initially only doing to do this to ensure the underlying facilities are in place for doing signals, but that was solved on your end.


Files with undetermined MIME type are not forwarded from Matrix to Discord

Files with undetermined MIME type (dd if=/dev/zero of=test_file bs=7M count=1) are not relayed from Matrix to Discord.
Perhaps should the bridge fallback to some generic value when it's not able to guess the MIME type.
Log snippet:
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]: Jun-16 15:40:13.484 [MatrixEventProcessor] warn: There was an error sending a matrix event TypeError: Cannot read property 'match' of undefined
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at Mime.extension (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/node_modules/mime/mime.js:79:23)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at MatrixEventProcessor.GetFilenameForMediaEvent (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/src/matrixeventprocessor.ts:493:59)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at MatrixEventProcessor.<anonymous> (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/src/matrixeventprocessor.ts:309:27)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at Generator.next (<anonymous>)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at /nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/build/src/matrixeventprocessor.js:22:71
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at new Promise (<anonymous>)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at __awaiter (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/build/src/matrixeventprocessor.js:18:12)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at MatrixEventProcessor.HandleAttachment (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/build/src/matrixeventprocessor.js:272:16)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at MatrixEventProcessor.<anonymous> (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/src/matrixeventprocessor.ts:173:33)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at Generator.next (<anonymous>)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at fulfilled (/nix/store/jljsvhk13ckz4rqfs7rnkbfvs0pxc4dr-node_matrix-appservice-discord-0.5.1/lib/node_modules/matrix-appservice-discord/build/src/matrixeventprocessor.js:19:58)
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at <anonymous>
Jun 16 15:40:13 matrix matrix-appservice-discord[7986]:     at process._tickCallback (internal/process/next_tick.js:189:7)

fixed in #501


Potential unauthorized access to underlying database

I was looking through the HTTP handler code, and noticed that access to the /metastore route is not protected in the same way that access to the /query or /write endpoints are.  Specifically, the /metastore handler would not be protected with an authenticate middleware because the method signature does not opt-in to authentication.
// If it's a handler func that requires authorization, wrap it in authorization
if hf, ok := r.handlerFunc.(func(http.ResponseWriter, *http.Request, *influxdb.User)); ok {
	handler = authenticate(hf, h, requireAuthentication)
}

This unprotected pathway results in a call to BoltDB's Tx.Copy method which apparently copies the entire database to the given writer.
// Copy writes the entire database to a writer.
// A reader transaction is maintained during the copy so it is safe to continue
// using the database while a copy is in progress.
// Copy will write exactly tx.Size() bytes into the writer.
func (tx *Tx) Copy(w io.Writer) error { /* ... */ }

So, bottom line, this is probably a bug.  Personally, I'd want this operation to be restricted to cluster admins and other authenticated nodes (it looks like this capability is used as part of the clustering work for 0.9.0?)

Thanks @politician -- yes, this is part of the 0.9.0 clustering, and the 0.9.0 code is still very much in progress. We'll get around to securing the routes you mention soon. The are somewhat different since they are not meant to be accessed directly by users.

Great, thanks.

/metastore responds with 404, I assume it no longer exists. Was likely ripped out when raft was redone.


Decide: Should we encourage people to use ipfs-pack as the primary way of adding files to IPFS?

Should we remove the Files on IPFS Lesson from the Decentralized Web Primer and tell people to use ipfs-pack instead?
The old Files on IPFS lesson revolves around using ipfs add -w to add file content and wrap it in directory info. Should we still include that in the "introductory" material?
If we keep that older lesson, which gets closer to the core of how IPFS works, how should we frame the lesson that covers creating ipfs-packs and serving their contents?
This decision is important because it will influence how people encounter IPFS for the first time and how they think it's supposed to be used.

Im not sure at this moment. Some impressions:

I want people to put content into ipfs and forget about posix as much as reasonable
I want applications to work from running ipfs services, either in the app, or in the host OS
I want a properly-working /ipfs and /ipns that people can rely on (FUSE or something like)
I want PackManifests to be associated with "content that MUST remain in the POSIX host FS (for perf or other reasons)" (dataset archives/libraries, media archives/libraries, package managers, etc)
I don't want PackManifests lying around throughout a user's personal computer.
The majority of data people deal with in their personal lives (data generated by most apps) should be version controlled and immutably kept in IPFS (git style)

Therefore, i do not think we should push for ipfs-pack to be THE main way people interact with "files on ipfs". I think it should be the way people interact with "large archives/bundles of files on ipfs".

Thanks. That gives me the info I need.


update.Message.MessageId is always 0 on webhook

Steps to reproduce
install update 13

start a webhook
receive any messages and check for messageId (it's 0)

Expected  behavior
parse the correct message_id
Actual behavior
probably cannot parse message_id to MessageId?
Environment data
NuGet Package Version: 4.2.0
.NET Version: 4.6.2
IDE: VS2017
App: Desktop
More Info
Got this update as Json
{"update_id":969****25,
"message":{"message_id":68,"from":{"id":964***14,"first_name":"***","username":"***","language_code":"en-US"},"chat":{"id":-1001******332,"title":"***","type":"supergroup"},"date":1502169772,"text":"1"}}

And then this had been parsed
ParsedUpdate:{"Id":0,"Message":{"MessageId":0,"From":{"Id":964***14,"FirstName":null,"LastName":null,"Username":"***","LanguageCode":null},"Date":"\/Date(-62135596800000)\/","Chat":{"Id":-1001******332,"Type":3,"Title":"***","Username":null,"FirstName":null,"LastName":null,"AllMembersAreAdministrators":false,"Photo":null,"Description":null,"InviteLink":null},"ForwardFrom":null,"ForwardFromChat":null,"ForwardFromMessageId":0,"ForwardDate":null,"ReplyToMessage":null,"EditDate":null,"Text":"1","Entities":[],"EntityValues":[],"Audio":null,"Document":null,"Game":null,"Photo":null,"Sticker":null,"Video":null,"Voice":null,"VideoNote":null,"Caption":null,"Contact":null,"Location":null,"Venue":null,"NewChatMember":null,"NewChatMembers":null,"LeftChatMember":null,"NewChatTitle":null,"NewChatPhoto":null,"DeleteChatPhoto":false,"GroupChatCreated":false,"SupergroupChatCreated":false,"ChannelChatCreated":false,"MigrateToChatId":0,"MigrateFromChatId":0,"PinnedMessage":null,"Invoice":null,"SuccessfulPayment":null,"Type":1},"EditedMessage":null,"InlineQuery":null,"ChosenInlineResult":null,"CallbackQuery":null,"ChannelPost":null,"EditedChannelPost":null,"ShippingQuery":null,"PreCheckoutQuery":null,"Type":1}

It looks like every underline can't be parse correctly? but why? am I missing something here? this thing was working perfectly before...

WorkAround
I could make it by doing this at the start of the Post controller:
var req = Request.InputStream;
req.Seek(0, SeekOrigin.Begin);
string json = new StreamReader(req).ReadToEnd();
var update = JsonConvert.DeserializeObject<Update>(json);

now update has been bind perfectly

How did you parse the update before you used Newtonsoft.Json?

@Olfi01 Sorry I couldn't understand what are you trying to say,
I've added using Newtonsoft.Json; to the top of the file and then trying to deserialize it after getting the value manually with Request.InputStream. Then I read from 0 to the end of the Input and it'll give me a string that telegram would send me on update, so I just parse it with JsonConvert.DeserializeObject<Update> Command to Update class

I mean, how was the input handled before you used that workaround?

@Olfi01 Oh the ASP.NET MVC would automatically bind the form data into action's method arguments (which in this case Update class) however It won't bind the first_name into FirstName if our Update class name had first_name property instead of FirstName telegram it could be bind automatically to Update class and so on...

that seems to be a problem in your asp.net config, can you show the file with the url mapping to me?

@Olfi01
Well I don't have a url mapping I'm using Asp.Net MVC and I have a Controller named Bot and an Action named Post so ...
public class BotController : Controller
{
    [HttpPost]
    public async Task Post(Update update)
    {
    ....
    }
    ....
}

I use webhook so I already run this command
await Bot.SetWebhookAsync("https://domain.net/Bot/Post", new FileToSend("FileName.pem", System.IO.File.OpenRead(Server.MapPath("~/Path/To/File.pem"))), 100, new Telegram.Bot.Types.Enums.UpdateType[] { Telegram.Bot.Types.Enums.UpdateType.All });
And the RouteConfig is
	public class RouteConfig
	{
		public static void RegisterRoutes(RouteCollection routes)
		{
			routes.IgnoreRoute("{resource}.axd/{*pathInfo}");

			routes.MapRoute(
				name: "Default",
				url: "{controller}/{action}/{id}",
				defaults: new { controller = "Home", action = "Index", id = UrlParameter.Optional }
			);
		}
	}

Which is the default route config of an ASP.NET MVC application default routing config, everything is basically a default config...
Web.Config is the default one of course with my database connection inside added.
And that's it.

wait, let me research how asp.net handles json data...

okay, found it. apparently the serialization is done by a class that does not recognize jsonObject attributes. that means you should just change the argument of the controller to string input and then call Update u = JsonConvert.DeserializeObject<Update>(input);

okay, found it. apparently the serialization is done by a class that does not recognize jsonObject attributes. that means you should just change the argument of the controller to string input and then call Update u = JsonConvert.DeserializeObject<Update>(input);

It does not work like that. I had to use @sijav 's workaround. That means reading the input stream from the beginning and deserialize it.

Even until now with Nuget version 16.0.2, I still need to use @sijav's code and a custom DTO class to make it work properly.

Even until now with Nuget version 16.0.2, I still need to use @sijav's code and a custom DTO class to make it work properly.

read this https://github.com/TelegramBots/Telegram.Bot/issues/835


Proper way to include javascript file

I have files at:
/views/index.pode
/public/scripts/main.js.pode
In /views/index.pode, I use the following code to try and include /public/scripts/main.js.pode
$(include scripts/main)
or even using
$(include scripts/main.js.pode)
I get a "file not found at path" error.
A comment within Responses.ps1 on line 309 seems to indicate that only the views path is looked at:
   # only look in the view directory
In the rest of the Include function, I don't see any code that would look through the public folder.
I am able to use the following successfully:
$(include ../public/scripts/main.js.pode)
Am I making a mistake somewhere?

Ah yeah, the include in only used to pull in partial views - so you can re-use view logic.
For javascript, css, images etc; you can use them in the normal way, so for javascript files:
<script type="text/javascript" src="scripts/main.js.pode"></script>


DBT ingestion errors

Affected module
Does it impact the UI, backend or Ingestion Framework?
DBT ingestion
Describe the bug
A clear and concise description of what the bug is.
2024-09-26 12:16:52.750  Expected ColumnLineagebut gotdictwith value{'fromColumns': ['bq-gorg...nies_upsert.segment_id'}- serialized value may not be as expected
Errors:

To Reproduce
Screenshots or steps to reproduce
Expected behavior
A clear and concise description of what you expected to happen.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: [e.g. 0.8]
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

fixed with https://github.com/open-metadata/OpenMetadata/pull/17708
also @aballiet this is just a warning and can be ignored.


AJAX call on add/edit

Can I add ajax calls to add and delete buttons? If yes, can you please provide an example?
Also, Will this plugin work in mobile only since it uses jquery mobile?

Can I add ajax calls to add and delete buttons? If yes, can you please provide an example?

I don't think you can directly attach events to add or delete buttons. But you can listen to the listviewchange which is triggered when the editable-listview comes out of the edit mode and then make an Ajax call based on any change in the list contents. See the Events page for more information.
If you can share the context of your particular use case, then may be I'll be in a better position to suggest a solution.

Also, Will this plugin work in mobile only since it uses jquery mobile?

jQuery mobile is just a requirement to run this plugin. The plugin does not care if it's being run on a mobile or not. If you design a Tablet or Desktop UI using jQuery mobile, then there's no reason that this plugin should not work.


[CRITICAL] OCI Grafana stop working on Grafana 8

Hi,
I've updated grafana to the new version and all the graphs stop working, please check.

Same with the DS

Is there anything you could share about how to resolve this issue?

Please refer to this page
https://grafana.com/grafana/plugins/oci-metrics-datasource/?tab=installation
Once you follow it and install your plugin, you should see a signed
datasource.
Please select and you should be good to go..
On Tue, Jun 15, 2021 at 10:02 AM Aleksandar Kocic @.***>
wrote:

Is there anything you could share about how to resolve this issue?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oracle/oci-grafana-metrics/issues/91#issuecomment-861671338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3CLHCYQRU36QW73DAMVRLTS6BQVANCNFSM46KLBKXQ
.

--
Athul

Thanks,
It is still not working. AlI I got from logs:
t=2021-06-15T18:24:44+0000 lvl=eror msg="Metric request error" logger=context userId=16 orgId=2 uname=xxx error="plugin unavailable" remote_addr=xxx

And this:
t=2021-06-15T18:24:43+0000 lvl=eror msg="Request Completed" logger=context userId=16 orgId=2 uname=xxx method=POST path=/api/tsdb/query status=500 remote_addr=xxx time_ms=2 size=34
referer=https://xxx

Any update?
I have tried both local and OCI instance with the same result.

Just one change . In my previous comment, I pasted a different plugin link by mistake.
Since, this is logging, the following should work.
Please refer to this page

https://grafana.com/grafana/plugins/oci-logs-datasource/?tab=installation

Once you follow it and install your plugin, you should see a signed
plugin.

Use it to create a datasource with tenancy details

Please select the new signed datasource

Now, test your data  source.

We should have a call to discuss it. I am available now or later.

Not really interested in the oci logs data source; however, it doesn't work either.

Still not working, i've created a brand new installation and configured oci-cli and oci-metrics-plugin 2.2.2, after run the grafana update to version 8 still not working with error 500

Please, we need Oracle to prioritize this because it's avoinding us to deliver a new dashboard with histograms to our customers that we can only build on Grafana 8. The other cloud vendors are OK on Grafana 8, just Oracle's its not. Now we are blocked to deliver our new dashboard. This is critical team.

Any update?

Hello, I just updated two servers with grafana and it stopped working, these are the same errors presented above.
From what I've debugged, the plugin version is not compatible with the grafana 8 API version.
Here is the log:
t=2021-06-30T23:11:03-0400 lvl=eror msg="Failed to start plugin" logger=plugins.backend pluginId=oci-metrics-datasource error="Incompatible API version with plugin. Plugin version: 1, Client versions: [2]"

PALLIATIVE SOLUTION: I downgraded to version 7.5.4 and it works again. Follow my procedure.
$wget https://dl.grafana.com/oss/release/grafana_7.5.4_amd64.deb
$ sudo dpkg -i grafana_7.5.4_amd64.deb
dpkg: warning: downgrading graphane from 8.0.3 to 7.5.4
(Reading database ... 217484 files and directories currently installed.)
Preparing to unpack grafana_7.5.4_amd64.deb ...
Unpacking graphane (7.5.4) over (8.0.3) ...
Setting up graphane (7.5.4) ...
Restarting grafana-server service... OK
Processing triggers for systemd (237-3ubuntu10.48) ...
Processing triggers for ureadahead (0.100.0-21) ...
$ sudo service grafana-server restart
Okay, just access that all the charts of Oracle Cloud Metrics are back to work.

@claudiocastelo  Grafana 8 is not supported yet.
@MaikiGirardi
We are working to add the support.
https://github.com/oracle/oci-grafana-metrics/pull/92
Currently, we are blocked by an issue
We have reached out to grafana developers and support team to help us move further.
Apologies for the inconvenience caused.
Will keep you posted.

@claudiocastelo  Grafana 8 is not supported yet.
@kocic11
@MaikiGirardi
We are working to add the support.
https://github.com/oracle/oci-grafana-metrics/pull/92
Currently, we are blocked by an issue
We have reached out to grafana developers and support team to help us move further.
Apologies for the inconvenience caused.
Will keep you posted.

Right, i'm following #92

Hi, any update about this issue?

Hello Maiki,
Apologies  for the delay.
We are still working on it with the Grafana engineering team to get this
resolved.
Also, I have added you to the Grafana support discussion thread in email to
keep you in the loop.
Thanks,
Athul
On Mon, Jul 19, 2021 at 7:34 AM Maiki Girardi @.***>
wrote:

Hi, any update about this issue?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/oracle/oci-grafana-metrics/issues/91#issuecomment-882599634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE3CLHEAIOJDBECFOPMS543TYQZXJANCNFSM46KLBKXQ
.

--
Athul

Guys, it's been a long time since the issue was open, nothing yet?

HI @MaikiGirardi  Please have a look into #106. I have changed it so that it can work multiple profile, cross tenancy policy and it supports Grafana 8 with alerting. It is still on developing phase. But you can have a look and try.
If you need any help or clarification feel free to let me know.

Hi, i opened this bug again because there is no release solving the problem.

Hi, this bug was closed, but there is no release, still waiting.


Make kubernetes domain configurable

Towards https://github.com/giantswarm/giantswarm/issues/8979
This will allow setting the proper search domain if is has been changed in the installation, the value is being taken from the same place as CoreDNS.

ping @rossf7  :)

This is basically an improvement over https://github.com/giantswarm/chart-operator/pull/139

Agree, looks good


Support for Multi-Exec and BLPOP

Based on the following:

https://github.com/lettuce-io/lettuce-core/wiki/Connection-Pooling#is-connection-pooling-necessary
https://github.com/lettuce-io/lettuce-core/wiki/Pipelining-and-command-flushing
https://github.com/lettuce-io/lettuce-core/issues/480

It appears that you should not use a shared connection when you are making MULTI - EXEC (transaction) or BLPOP (blocking list pop) calls.
From what I understand from the above docs, the application will need a non-shared connection for these calls.
This could be done either by creating new connector instances (possibly isolated to some worker thread) or via a connection pool if these calls are dynamic (not already isolated to some worker thread that contains its own connection).

Thanks for the super fast turn around on this.


Update Delaunay.gd

Hi,
Thank you for the wonderful plugin.
I wrote a helper function to return the clipped boundary of border VoronoiSite. Might not be the comprehensive solution you have in mind but I thought it can be a useful tool for anyone who need to find get the border Site boundaries without changing the underlying values.
Also in triangulate() function it should read:
if (_rect.has_no_area()):
set_rectangle(calculate_rect(points))
--

Add function to return clipped boundary for border VoronoiSite.
Fix a bug in triangulate function

Thanks!


Running CITE-seq-Count on ASAP-seq

Hi,
Thanks for making this useful tool!
I have a question about running CITE-seq-Count on ASAP-seq. We did ASAP-seq to hashtag cells.
There are three FASTQ files and according to this document (https://citeseq.files.wordpress.com/2020/09/asap_seq_bridge_a_hashing.pdf) and they are structured in this way:
R1 containing Genomic fragment including UBI (analogous to UMI) at positions 1-10.
R2 containing cell barcode at positions 1-16.
R3 containing Genomic fragment including hashtag at positions 1-15.
Since CITE-seq-Count accepts only two sets of FASTQ files (R1 and R2), I appended R2 and R1 together and provided the appended FASTQ file as an input for [-R1] parameter, and R3 FASTQ file for [-R2] parameter. And given the lengths of cell barcode and UBI, this parameter was provided [-cbf 1 -cbl 16 -umif 17 -umil 26].
I'm wondering whether the above approach is sensible for ASAP-seq? Reason I'm asking is I obtained an output that doesn't match up with the accompanying scATAC-seq data - no common cell barcodes across them.
Thanks!

Hello @jhjlee
your strategy is totally fine here. What might have happened is that the V3 chemistry of 10x has two different barcodes per cell, one for the RNA one for the FB.
When you give a specific list of RNA, it will probably not overlap at all with the FB barcodes.
From here you have two options.

run the current release without providing a specific list of cells to find but rather a number.
Once you get this back, "translate" the barcodes based on the translation list found here
Run the experimental branch https://github.com/Hoohm/CITE-seq-Count/tree/feature/cells_argument and this should provide an MTX output with both RNA and FB barcodes in the barcodes.tsv output. Please try on a lower portion of the data if you choose this option. There is an argument that allows you to get only the top N reads, just to see that it runs through. This version will ask you for a Reference "Translation" list, the link I posted in the previous paragraph.

Option 1 is safer, but a bit more manual work, option 2 might crash, I've been working on it off and on lately. So there are definitely bugs in there I have not accounted for.
Please let me know if you need any help to go through this.

Hi thanks for you reply!
I'm a little confused though because what I'm trying to do is to demultiplex ASAP-seq data (accompanying scATAC-seq). When I did option 1 and tried translating the barcodes with the translation list, I had no hits. Isn't this translation list for scRNA-seq instead? Sorry if I'm misunderstanding you.

Totally possible, I imagined it would be the same chemistry.
Did you try and grep some barcodes in the library to find out if they are
in there at all?
On Wed, 19 May 2021, 11:52 Hyun Jae (Joshua) Lee, @.***>
wrote:

Hi thanks for you reply!
I'm a little confused though because what I'm trying to do is to
demultiplex ASAP-seq data (accompanying scATAC-seq). When I did option 1
and tried translating the barcodes with the translation list, I had no
hits. Isn't this translation list for scRNA-seq instead? Sorry if I'm
misunderstanding you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Hoohm/CITE-seq-Count/issues/154#issuecomment-843940888,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJVO2F3Z2NW5YOC24JNX5DTOOC5FANCNFSM45D5XLAA
.

Yes I did and couldn't get any matches. I think the barcodes for scATAC-seq (and by extension ASAP-seq) and barcodes scRNA-seq are different.

Yeah I'm not familiar with the technology.
Any chance there is a translation table?
If not, could you maybe send top 10 barcodes from a couple of millions of
reads?
On Wed, 19 May 2021, 13:29 Hyun Jae (Joshua) Lee, @.***>
wrote:

Yes I did and couldn't get any matches. I think the barcodes for
scATAC-seq (and by extension ASAP-seq) and barcodes scRNA-seq are different.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Hoohm/CITE-seq-Count/issues/154#issuecomment-844012572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJVO2CH4DSSWQYGJSCJSB3TOOOJRANCNFSM45D5XLAA
.

Hi I think I figured out the problem here. Apparently the cell barcodes in the ASAP-seq data needs to be reverse complemented (source: https://github.com/caleblareau/asap_to_kite). When I did this and ran CITE-seq-Count again, I could pick up signals from the data.
Thanks for looking into this and happy to close the issue.

Nice!
Happy to help :)
And I got to learn something as well, thanks
On Thu, 20 May 2021, 02:57 Hyun Jae (Joshua) Lee, @.***>
wrote:

Closed #154 https://github.com/Hoohm/CITE-seq-Count/issues/154.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Hoohm/CITE-seq-Count/issues/154#event-4769954799, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJVO2GZW6R2AFCKUCVZY5TTORNA3ANCNFSM45D5XLAA
.


Need unit tests covering the different states for the advanced export box

There are quite a few variants of this box:

Test coverage should exercise all of them, since the logic is a little unclear.
https://github.com/simonw/datasette/blob/fdfbbbb9ee0d02fd4d43dfc42382252fa2287d6d/datasette/templates/table.html#L140-L159

No primary key => no "object" option: https://latest.datasette.io/fixtures-dcc1dbf/no_primary_key
Has a primary key => show "object" option: https://latest.datasette.io/fixtures-dcc1dbf/complex_foreign_keys
Has a next page => has "stream all rows" option: https://latest.datasette.io/fixtures-dcc1dbf/no_primary_key
Has foreign key references = show default-checked "expand labels" option: https://latest.datasette.io/fixtures-dcc1dbf/complex_foreign_keys
Does not have a next page => do not show "stream all rows" option: https://latest.datasette.io/fixtures-dcc1dbf/complex_foreign_keys


Add admin room commands for force joining rooms and making yourself admin

https://matrix-org.github.io/synapse/latest/admin_api/rooms.html#make-room-admin-api
https://matrix-org.github.io/synapse/latest/admin_api/room_membership.html

This has been implemented for a while now


Update Pages “index”

Automatically generated by Netlify CMS

👷 Deploy Preview for nervous-engelbart-a15395 processing.
🔨 Explore the source changes: 724706625de1ad21d8119e9bbda8ce51efc61058
🔍 Inspect the deploy log: https://app.netlify.com/sites/nervous-engelbart-a15395/deploys/61ad3d902076de0008fb941a


Add phi_ext to more models

Expand documentation for models that already support phi_ext
Add phi_ext to fermions_spinless and hubbard models

You are of course right regarding the check in coupling_strength_add_ext_flux.
Without it though, the phi_ext parameter for models on 1D lattices silently ignores phi_ext.
To fix this, I added a different check to coupling_strength_add_ext_flux, that checks if the number of specified fluxes matches the dimension of the lattice.
This is implicitly assumed by the implementation anyway.
Since this is not what you agreed to above, could you confirm the merge again?


Links for downloads to pLinux, zLinux and AIX missing

The previous download page when hosted on the old side included links to where you could download versions for these platforms.  This section was added under this PR https://github.com/nodejs/nodejs.org/pull/101#issuecomment-87781755
and it looked like this

@jasnell FYI

Thanks, added the section again.


embeddings: remove unnecessary mostly-duplicated implementations

PR Checklist

[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

They're only removed from the embeddings directory.
Can you provide a minimal, reproducible example of what you're trying to accomplish?

Sorry, I'm still having trouble understanding what you're trying to accomplish. The code that was removed duplicated all the embeddings implementation with the only difference being that it forwarded the CreateEmbedding method to the underlying client. This design didn't make a lot of sense, but it also wasn't clear what the goal is.
Can you provide an example of the actual problem you're trying to solve? Do you have a llm.Chat but not the underlying llm.Client and you're trying to get embeddings from it? How does this situation arise? Can you get embeddings from the client, not the chat? I'm confused about the duplication here.

BTW, I recommend opening a new issue to discuss this - for more visibility. Feel free to @ me if you want to create one and provide an example

@eliben opened the issue, if you like you can find me on the discord as well to chat about this or even hop on the call if needed.


[1.9] Controller world crash

Forge: <IP_ADDRESS>7
Version: Tested on both 0.6.4 & 0.6.5
Description: When I had a hopper going into a disk drive and attempted to import items into storage this way, I got a server crash and the world crashed on startup again until I used mcedit to remove the controller block.
Crash Report

Thanks for reporting, this is fixed for the next version.

Thanks for the quick reply =)

This bug also works on a bunch of other machines, so I'm very happy I know about this.

Great work!

Version 0.6.6 fixes this


Pack cpp and third_party files to model_perf python package

Pack cpp and third_party files to model_perf python package

Abandon this PR because we choose to use git submodule.


feat: use dynamic config instead of static

Motivation: it allows to avoid redeploying process and improves service maintenance.
Candidates

https://doc.traefik.io/traefik/providers/consul/
https://doc.traefik.io/traefik/providers/etcd/
https://doc.traefik.io/traefik/providers/http/
https://doc.traefik.io/traefik/providers/consul-catalog/

won't do because I split dynamic config https://github.com/octomation/stub/tree/main/traefik/dynamic


Added an environment to control if Firehoe is preferred or not when present

If the environment is not set, the current behavior and Firehose is automatically picked up if there is endpoints configured with it.
If the environment set, firehose is preferred only if the variable is true and there is endpoints configured with it.

You can also use this helper
https://github.com/graphprotocol/graph-node/blob/8155ed5299e96a79a1554642486bcf88254b4a59/graph/src/env.rs#L43
I find it confusing that this is an Option<bool>, if the default is to prefer firehose then it should just default to true.

@leoyvens You are right, it's simpler without, it's was overcomplicated for the same net result, changed.


Add MockStream using tokio::io::DuplexStream

We have a mock stream in the mock example but we should add that implementation to tonic proper so users can use it without needing to maintain that code.

@SuperFluffy yeah thanks for the reminder we should try to fix that when add this as well.

MockStream has been removed and instead its recommended to just use tokio::io::DuplexStream instead.
https://github.com/hyperium/tonic/pull/923


cleanup(ci): properly set a concurrency for CI workflows.

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area CI
What this PR does / why we need it:
See https://github.com/falcosecurity/libs/pull/1029 and https://github.com/falcosecurity/libs/pull/1028
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.35.0

Note: conversely from the libs PR, i decided to stop all workflows triggered by the branch/pr on Falco (not just the CI one), because we have eg: codeQL that is pretty heavy too since it performs a full build.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


cargo: Fix sparse registry error on update

We're seeing some errors like this:
error: failed to get `arrow` as a dependency of package `geoengine-datatypes v0.7.0 (/home/dependabot/dependabot-updater/dependabot_tmp_dir/datatypes)`

Caused by:
 failed to load source for dependency `arrow`

Caused by:
 Unable to update registry `crates-io`

Caused by:
  usage of sparse registries requires `-Z sparse-registry`

This is due to the sparse registry we enabled in https://github.com/dependabot/dependabot-core/pull/6995. It seems this setting is required for the update to work as well as the environment variable.

This isn't a complete solution, we also need to consider a range of unstable channel releases. But this should improve our success rate with Cargo updates, so we can ship incrementally.
The -Z flags only apply to unstable channels so we can ignore older stable toolchains. They will continue to use the non-sparse protocol.

How frequently is this problem occurring?
Given it's only cropping up on nightly/unstable, and anyone using nightly/unstable generally is quick to update their toolchains, I suspect most folks affected by this will be updating soon if they haven't already...
I'm not objecting to merging since this is already written, but any code we add is also code we have to maintain later... so I'd be inclined to simultaneously open a draft PR to remove this workaround, and consider it blocked for another 6-12 months and then pull it out.
This also gets into the territory that we've discussed for later this quarter of setting clear policy guidelines around what versions we support within different ecosystems (cc @abdulapopoola for visibility as we discuss that initiative). Eg, for Rust Cargo, in general we probably don't want to be trying to support workarounds required for nightly/unstable for very long once a feature has been merged into stable.
Two other ideas:

don't try to use the sparse protocol at all for nightly/unstable that are older than the stable version where the sparse protocol landed... this has a perf hit for us internally, but again I suspect the frequency is pretty darn low so it'd be okay.
throw an error message to the user of "you are on an older nightly/unstable, please upgrade to a stable >= "

Any reason we can't go with 1⃣ ??
Also, my apologies if I misunderstand completely how channels work in Cargo/Rust, I've read the docs but haven't worked directly with them so I may be misunderstanding something here that limits the implementation options.

The errors are hidden behind the HelperSubprocessFailed exception, so we can't tell exactly how many updates this is affecting.
I created a PR to gather toolchain channel data that should be helpful going forward: https://github.com/dependabot/dependabot-core/pull/7080
Consider that this feature only stabilized in January, so anyone who pinned to a nightly last year are now broken. I don't want to disable the sparse protocol and risk an incident unless we know more about who is using what channels. So, do we want to wait on this and proceed with https://github.com/dependabot/dependabot-core/pull/7080?

Initial numbers indicate <7% of our jobs involve a toolchain, and even less of that are toolchains before 1.68, maybe ~1%?
Based on that I think we can go with recategorizing the error as a resolution error and recommend upgrading toolchains to >= 1.68.

@ilya-bobyr Yep, see https://github.com/dependabot/dependabot-core/pull/7122.
Is there a reason you can't update your toolchain to 1.68.0?

We saw failures in some dynamically loaded code.
And no one spend enough time fixing it yet.

Sorry for the trouble! Dependabot was putting too much pressure on GitHub without the sparse registry so it forced us to update. Usually we're not this aggressive about forcing recent versions.

Sorry for the trouble! Dependabot was putting too much pressure on GitHub without the sparse registry so it forced us to update. Usually we're not this aggressive about forcing recent versions.

No worries, and thank you for the explanation!
After you merge the warning, it would be even easier for the users to understand that it is time to update :))


Fix imagenet for windows to use if name == "main" idiom

According to Sikon-san's report on Google Group, imagenet example does not run properly on Windows environment. If the main scripts uses multiprocessing module, we need to disable side effects in main scripts by using __name__ == "__main__" idiom.
See the official document for details.
Related to #483

Fixed already.


chore: generate docs

TODO: Still need generate examples using the rosetta library, and
generate docs on properties, methods, as well as one the different go
data structures generated from the same jsii type (i.e. the struct and
interface of a class).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 206c0fb2565ac619953628f4ec1898790e0666c7
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: b2674376bc59e56c0640fae12ce77342a6626a69
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 93bd02926be1f461e9c113cb76fb5159ec7e2129
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 686113cf30e7e67f8d72c308e9e4192d5b86b441
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[Bug?/Config Issue] Mob Crusher doesn't collect essence from passive mobs

Industrial Foregoing Version: 1.12.1-1.4.2-69
TeslaCoreLib Version: 1.12-1.0.9
if this is intended can there be a config option to allow it

It is intended, the Mob Crusher collects the dropped experience when mobs are killed and transform it into essence, if a mob doesn't drop experience it won't generate essence.


Pull Request 16

Pull Request 16
Branches Compared

Discobots-1104A-master :arrow_left: Discobots-1104A-marco-oct_2020
Proposed Changes

:heavy_check_mark: Change PD point turn to solely use the IMU.
:heavy_check_mark: Get rid of magic numbers wherever possible.
:heavy_check_mark: Add more comments because VEX likes that apparently.
:heavy_check_mark: Probably kink out most of the bugs in the PD code.
:heavy_check_mark: Add more text to the init screen and controller to clarify program initialization.
:heavy_check_mark: Update the PR template to imclude branches being compared.
:bulb: Have not checked PD point turn code, but will do soon.

Nvm gonna add more stuff before I PR


fix(jest-config): use the value of process.env.CI to determine option.updateSnapshot

Fixed #12288
Summary

The cause of this problem is that the current implementation for determining option.updateSnapshot does not take into account process.env.CI. We can know the value of process.env.CI in DEFAULT_CONFIG.ci.
Test plan

This PR added test cases for option.updateSnapshot and also fixed type issues by as assertions.

Codecov Report

Merging #12378 (ab6db25) into main (faef0b4) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main   #12378      +/-   ##
==========================================
+ Coverage   68.47%   68.48%   +0.01%
==========================================
  Files         324      324
  Lines       16967    16969       +2
  Branches     5060     5061       +1
==========================================
+ Hits        11618    11622       +4
+ Misses       5317     5315       -2
  Partials       32       32

Impacted Files
Coverage Δ

packages/jest-config/src/index.ts
20.73% <ø> (ø)

packages/test-utils/src/config.ts
0.00% <ø> (ø)

packages/jest-config/src/normalize.ts
88.86% <100.00%> (+0.40%)
:arrow_up:

Continue to review full report at Codecov.

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

Okey
On Sun, Feb 13, 2022, 5:35 PM Simen Bekkhus @.***>
wrote:

@.**** commented on this pull request.
In packages/jest-config/src/normalize.ts
https://github.com/facebook/jest/pull/12378#discussion_r805307171:

@@ -1132,7 +1132,7 @@ export default async function normalize(
}

newOptions.updateSnapshot =

argv.ci && !argv.updateSnapshot

(!!argv.ci || DEFAULT_CONFIG.ci) && !argv.updateSnapshot

I don't think this makes sense - we should always respect the arguments
the user passes to us
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/12378#pullrequestreview-880957727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXNZNGS276L2ZUUHLQ3CHJLU253OJANCNFSM5OHALFFQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Create DEV Helm Repository for Tractus-X

This PR implements:

a GO binary, pulling released Helm charts from GH Org repositories, build DEV Helm repository
GH Workflows for building the GO binary and executing/building the Helm repository.
README.md file which will be shown in repo root (→ GH page).

The Helm repository will be available at https://eclipse-tractusx.github.io/tractus-x-release/charts/dev
For now this PR is a draft. After Code Review I will remove the draft flag from this PR.
Go binary helmRepoIndex

get a list of all GH Org repositories
for each repository:

check if GH Pages are enabled, if not continue
If a index.yaml in repos GH Page is available download
merge downloaded  index.yaml with local one
delete downloaded index file

GH Workflows
Build Go - helmRepoIndex
Build the Go binary and stores it at .github/bin.  .github/bin directory is created if not presend. Go binary is pushed back to repository to make it available for the second workflow.
Build Helm Repository - DEV
This workflows fetches from all GH Org repositories their local Helm repository file (index.yaml) and merges it into the central charts/dev/index.yaml. No released Helm charts are fetched and stored localy.
Updated charts/dev/index.yaml is pushed back to repository.
Things to discuss/ ToDo's

Is it best practice to build a binary and push it back to the repo and use this binary with a workflow. If not, does it make sense to create a separate Git repo for this binary (and maybe upcoming other helper binaries) and to release them there? This would make the bins reusable.
ToDo: Authenticated github client isn't nesseccary for workflow execution. This was integrated during local testing and hitting the GH API rate limit. Either remove, or add argument to Go binary to use it or not.

As per Tractus-X/Catena-X decision the Helm repositories should be managed/located in a different repository than tractus-x-release. Close this PR due to this decision.


scopehal-apps: darwin support

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354815

x86_64-linux

:white_check_mark: 1 package built:

scopehal-apps

aarch64-linux

:white_check_mark: 1 package built:

scopehal-apps

x86_64-darwin

:x: 1 package failed to build:

scopehal-apps

aarch64-darwin

:white_check_mark: 1 package built:

scopehal-apps

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354815

x86_64-linux

:white_check_mark: 1 package built:

scopehal-apps

aarch64-linux

:white_check_mark: 1 package built:

scopehal-apps

x86_64-darwin

:white_check_mark: 1 package built:

scopehal-apps

aarch64-darwin

:white_check_mark: 1 package built:

scopehal-apps


added forked fragment plugin

@BenediktLa Please re-add the plugin, so the change takes affect in our Project


double quote and use addslashes on disqus title

Previously if you had a single quote in your post title a javascript error would prevent the page from functioning correctly. With this functionality we will double quote the post.title in the inject javascript and use the builtin swig filter addslashes to escape characters that need to be escaped.

Thanks. :+1:


fixed Bot.create() and Bot.post()

Parameters for bot creation and posting messages from a bot were being passed as URL parameters and not as the POST body.
Even though GroupMe's API docs for bot creation suggest that the data should be passed as URL parameters, this actually doesn't work. You need to pass them in the body of the POST as JSON, as is described here.
Posting messages works the same way; data must be sent as a JSON block in the POST body and not as URL parameters.
There was a companion error to this one in the Bot class initializer. The dictionary returned by the API encapsulates the expected data within an entry with key 'bot', but the full dictionary is returned.
Each of these issues has been fixed; the diff is self-explanatory.

Awesome, thanks!


Flutter 1.7.8.hotfix.4-1 crashes in linux

flutter_05.log
Hi!
I use flutter sdk in Arch linux and frequently it crashes. I attach the log crash to this issue.
Flutter version: 1.7.8.hotfix.4-1
Linux version (uname -a): Linux msj 4.19.66-1-MANJARO #1 SMP PREEMPT Fri Aug 9 18:01:53 UTC 2019 x86_64 GNU/Linux
Crasch message: Oops; flutter has exited unexpectedly. Sending crash report to Google. Crash report sent (report ID: dc3c4ace7a7c5c8a) Crash report n to /home/carlos/flutter_05.log;              please let us know at https://github.com/flutter/flutter/issues.

can you post contents of /home/carlos/flutter_05.log ?

The file is attached to the first message. Here is the content:
`Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter doctor
exception
FileSystemException: FileSystemException: Cannot open file, path = '/opt/flutter/bin/cache/flutter_version_check.stamp' (OS Error: Permission denied, errno = 13)
#0      _File.throwIfError (dart:io/file_impl.dart:643:7)
#1      _File.openSync (dart:io/file_impl.dart:487:5)
#2      _File.writeAsBytesSync (dart:io/file_impl.dart:612:31)
#3      _File.writeAsStringSync (dart:io/file_impl.dart:636:5)
#4      _LocalFile&LocalFileSystemEntity&ForwardingFile.writeAsStringSync (package:file/src/forwarding/forwarding_file.dart:152:16)
#5      Cache.setStampFor (package:flutter_tools/src/cache.dart:243:35)
#6      VersionCheckStamp.store (package:flutter_tools/src/version.dart:469:20)
<asynchronous suspension>
#7      FlutterVersion._getLatestAvailableFlutterDate (package:flutter_tools/src/version.dart:391:31)
<asynchronous suspension>
#8      FlutterVersion.checkFlutterVersionFreshness (package:flutter_tools/src/version.dart:296:52)
<asynchronous suspension>
#9      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:372:41)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#11     _rootRun (dart:async/zone.dart:1124:13)
#12     _CustomZone.run (dart:async/zone.dart:1021:19)
#13     _runZoned (dart:async/zone.dart:1516:10)
#14     runZoned (dart:async/zone.dart:1463:12)
#15     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#16     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#17     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#18     new Future.sync (dart:async/future.dart:224:31)
#19     CommandRunner.run (package:args/command_runner.dart:112:14)
#20     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#21     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:22)
<asynchronous suspension>
#22     _rootRun (dart:async/zone.dart:1124:13)
#23     _CustomZone.run (dart:async/zone.dart:1021:19)
#24     _runZoned (dart:async/zone.dart:1516:10)
#25     runZoned (dart:async/zone.dart:1500:12)
#26     run.<anonymous closure> (package:flutter_tools/runner.dart:60:18)
<asynchronous suspension>
#27     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#28     _rootRun (dart:async/zone.dart:1124:13)
#29     _CustomZone.run (dart:async/zone.dart:1021:19)
#30     _runZoned (dart:async/zone.dart:1516:10)
#31     runZoned (dart:async/zone.dart:1463:12)
#32     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#33     runInContext (package:flutter_tools/src/context_runner.dart:56:24)
<asynchronous suspension>
#34     run (package:flutter_tools/runner.dart:51:10)
#35     main (package:flutter_tools/executable.dart:62:9)
<asynchronous suspension>
#36     main (file:///b/s/w/ir/k/archive/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#37     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#38     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

flutter doctor
[✓] Flutter (Channel stable, v1.7.8+hotfix.4, on Linux, locale es_ES.UTF-8)
    • Flutter version 1.7.8+hotfix.4 at /opt/flutter
    • Framework revision 20e59316b8 (hace 5 semanas), 2019-07-18 20:04:33 -0700
    • Engine revision fee001c93f
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /home/carlos/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling
      support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1343-b16-5323222)
    • All Android licenses accepted.

[✓] Android Studio (version 3.4)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 38.2.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1343-b16-5323222)

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

duplicate https://github.com/flutter/flutter/issues/28210


Possible to use peripherals in panic handler?

Hi, first thank you for the effort put into this wonderful ecosystem.
I'm writing a heater controller with this library, and one thing I want to make sure of, is that when panic happens, the PWM (which controls how much heat is produced) is disabled/zeroed and the program goes into an infinite loop, so that it doesn't burn down my house.
From what I gathered, peripherals are intentionally made hard to get for safety reasons. I can't take in the handler because that won't work. Maybe I should try global mutable variable to store the PWM instance, but it's considered a bad practice in rust?

Since I pressume you're not going to do anything after the panic handler, you can use the steal function on the periperal, re-setup the peripherals and turn off the PWM. Alternatively, you can probably just call reset on the whole thing after stealing.

@TheZoq2 thanks for the reply.
Re-setup sounds like a hassle, I'm especially worried about re-initializing these registers causing some undefined behaviour.
Is there someway I can directly access the PWM object that I constructed in main?

A global PWM variable is probably the best option. It's not "best practice", but it's also not terrible because you'll have to access it via unsafe

Thanks!  I tried this with the following program. But obviously my borrow-fu is not very good yet.
static mut pwm_ : Option<&mut Pwm<TIM3, C1>> = None;
#[panic_handler]
unsafe fn panic(_info: &PanicInfo) -> ! {
    match pwm_ {
        Some(&mut pwm) => {
            pwm.disable();
        },
        None => {},
    };
}
#[entry]
fn main() -> ! {
let mut pwm = ...;
unsafe {
        pwm_ = Some(&mut pwm);
    }
pwm.set_duty(duty);

error[E0507]: cannot move out of `"*pwm_.0"` as `"pwm_"` is a static item
  --> src/main.rs:77:11
   |
77 |     match pwm_ {
   |           ^^^^ help: consider borrowing here: `&pwm_`
78 |         Some(&mut pwm) => {
   |                   ---
   |                   |
   |                   data moved here
   |                   move occurs because `pwm` has type `stm32f1xx_hal::pwm::Pwm<stm32f1::stm32f103::TIM3, stm32f1xx_hal::pwm::C1>`, which does not implement the `Copy` trait

error[E0596]: cannot borrow `pwm` as mutable, as it is not declared as mutable
  --> src/main.rs:79:13
   |
78 |         Some(&mut pwm) => {
   |                   --- help: consider changing this to be mutable: `mut pwm`
79 |             pwm.set_duty(0);
   |             ^^^ cannot borrow as mutable

error[E0596]: cannot borrow `pwm` as mutable, as it is not declared as mutable
  --> src/main.rs:80:13
   |
78 |         Some(&mut pwm) => {
   |                   --- help: consider changing this to be mutable: `mut pwm`
79 |             pwm.set_duty(0);
80 |             pwm.disable();
   |             ^^^ cannot borrow as mutable

error[E0597]: `pwm` does not live long enough
   --> src/main.rs:169:21
    |
89  | #[entry]
    |        - `pwm` dropped here while still borrowed
...
169 |         pwm_ = Some(&mut pwm);
    |         ------------^^^^^^^^-
    |         |           |
    |         |           borrowed value does not live long enough
    |         assignment requires that `pwm` is borrowed for `'static`

error[E0499]: cannot borrow `pwm` as mutable more than once at a time
   --> src/main.rs:247:9
    |
169 |         pwm_ = Some(&mut pwm);
    |         ---------------------
    |         |           |
    |         |           first mutable borrow occurs here
    |         assignment requires that `pwm` is borrowed for `'static`
...
247 |         pwm.set_duty(duty);
    |         ^^^ second mutable borrow occurs here

Try making the static mut the owner of the peripheral, rather than just having a mutable reference.

Thanks! it works.
Although I still hope to use reference since that would be simpler for me, but i'm closing this issue anyway.


Azure Pipeline Error On Initialize due to node 10 dependency.

Describe the bug
As of this morning, Azure Pipelines is throwing warnings in Job Initialization step for both the TerraformInstaller@1 and TerraformCli@1.
The errors are as follows:
##[warning]Task 'Terraform Installer' version 1 (TerraformInstaller@1) is dependent on a Node version (10) that is end-of-life. Contact the extension owner for an updated version of the task. Task maintainers should review Node upgrade guidance: https://aka.ms/node-runner-guidance
##[warning]Task 'Terraform CLI' version 1 (TerraformCLI@1) is dependent on a Node version (10) that is end-of-life. Contact the extension owner for an updated version of the task. Task maintainers should review Node upgrade guidance: https://aka.ms/node-runner-guidance
To Reproduce
Steps to reproduce the behavior:

Setup pipeline using TerraformInstaller@1 and TerraformCli@2
Execute pipeline
See warnings in Job Initialization

Expected behavior
The job should initialise without throwing the warnings above
Screenshots
If applicable, add screenshots to help explain your problem.

Pipeline Logs
Include logs that help demonstrate the problem. Please make sure to redact any sensitive info such as secrets.
Agent Configuration

OS: ubuntu-22.04
Self Hosted
Terraform version used: N/A
AzureCLI version used: N/A

Additional context
Add any other context about the problem here.

Duplicate of #430


rt: Add framework for creating heap statics

Normal Rust binaries will initialize heap statics on start, and then essentially leak the memory. This works fine because the memory is cleaned up during process exit. We can't assume the same with a FAP, so if we want to support "statics", we should provide a way in the runtime for initializing them before the user's app code runs, and freeing them after it exits.

In this case you're talking about statics that are stored on the heap (from lazy_static, once_cell), rather than static data stored in the .rodata right?

Yep. People could still use one of those crates, but I believe it would get reported as a memory leak by the FZ tooling.

It might be possible to provide a factory function for OnceCell that registers the static for cleanup. Calling the take method takes ownership of the inner value allowing them to be dropped.

We can't use once_cell because it requires std for Mutex. I'm instead planning on using generic_once_cell, which works for any mutex that implements lock_api::RawMutex. I've implemented the latter locally, and opened #74 with an Instant impl that I need.

We can't use once_cell because it requires std for Mutex.

I think it's possible to use with #[no-std] as long as you provide a critical-section implementation. I might take a look at that today, since a number of embedded crates are based around it.


How can I finetune the large bitrate model (λ=0.05) to obtain the low bitrate model (i.e. λ=0.013)

For saving computational resources, I think the training strategy from well-trained high bitrate model to obtain a low bitrate model is necessary.
Problem> when I type "--lambda 0.0013 --load-checkpoint /../0.05checkpoint_best.pth.tar" and the training process doesn't start with  "epoch0". It just exit without any error.
How can I work on this training strategy? Thanks a lot!

The epoch is saved inside the checkpoint, along with the LR scheduler state, and other information:
            save_checkpoint(
                {
                    "epoch": epoch,
                    "state_dict": net.state_dict(),
                    "loss": loss,
                    "optimizer": optimizer.state_dict(),
                    "aux_optimizer": aux_optimizer.state_dict(),
                    "lr_scheduler": lr_scheduler.state_dict(),
                },
                is_best,
            )

When loading, the checkpoint you may want to skip loading some of this information. For example, modify the training code to:
def parse_args(argv):
    ...
    parser.add_argument("--start_fresh", action="store_true", help="Reset training state.")
    ...

def main(argv):
    ...

    last_epoch = 0
    if args.checkpoint:  # load from previous checkpoint
        print("Loading", args.checkpoint)
        checkpoint = torch.load(args.checkpoint, map_location=device)
        net.load_state_dict(checkpoint["state_dict"])
        if not args.start_fresh:
            last_epoch = checkpoint["epoch"] + 1
            optimizer.load_state_dict(checkpoint["optimizer"])
            aux_optimizer.load_state_dict(checkpoint["aux_optimizer"])
            lr_scheduler.load_state_dict(checkpoint["lr_scheduler"])

    ...

Then, just run your training command with an additional --start_fresh flag.


fix_unknown_devices_quick_reply

Display an alert when some unknown devices are found while using the quick replay
Should fix https://github.com/vector-im/riot-android/issues/1069

Whats about the issue that the message is lost in this case?

@krombel
The message won't be lost : it will be stored in the dedicated room history as "unsent".
The user will have to validate the unknown devices list before sending them.

I could not find the message that was send via quick-reply in the history. It seems that the message somehow was added to local room history but I cannot see it. Just the resend possibility is shown.
After choosing the resend option I could see the message again
(Just tested a few moments ago with current dev and stable)

@krombel
When you says "history", do you means the home/recents history or the room history (i.e the room page) ?

I mean the list of messages that contains the send/received messages inside a room

@krombel
on my side, the message is displayed at the bottom in red.

I tested again and the message was not there when I send thequick-reply and checked instantly.
But now I see the unsend message when I left and refocused the room


Fix #515

Closes #515

Can you add tests for this?

Can you add tests for this?

That's what I'm currently working on


Update system_advanced_misc.php

This adds the "normal" power state to the powerd - system_advanced_misc options.
If the powerd utillity can not determine the system power options (ac or battery) it will use the "normal" setting rendering the "ac" and "battery" settings useless for these systems.
powerd -v (from the command-line, on my intel Avoton system)
Quote
powerd: unable to determine AC line status <=== Important for "-n" option of powerd !!!
load  10%, current freq 1200 MHz (13), wanted freq 1200 MHz
load   6%, current freq 1200 MHz (13), wanted freq 1200 MHz
load   8%, current freq 1200 MHz (13), wanted freq 1200 MHz
load  10%, current freq 1200 MHz (13), wanted freq 1200 MHz
load   6%, current freq 1200 MHz (13), wanted freq 1200 MHz
powerd manpage:
Quote
-n   mode    Selects the mode to use normally when the AC line state is
unknown.

Whoops, nice catch. This seems to have gone missing in action even before 15.1 came out:
https://github.com/opnsense/core/commit/b899e523

Edited to add the "help" text.

Looks good, thank you very much! 👍


remove dead code

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
44f06b2f7c4a0158835c95559ce17e623ed008e7
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [geeknoid,rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Possible SQL injection vulnerability in table viewer

Hi Jerome,
I was looking at the database table viewer code and think there are a few SQL injection vulnerabilities.
Most of these situations are handled by some kind of processing of user input beforehand, like implode(',', array_map('absint', (array) $_REQUEST['id'])), making them difficult to exploit, but in all cases, $wpdb->prepare() would have been the right method.
https://github.com/jerome-barbato/wp-steroids/blob/f4466d0f8e02f5c3916bf7c8a5cb9d2eb6cbcc02/includes/lib/list-table.php#L138-L142
https://github.com/jerome-barbato/wp-steroids/blob/f4466d0f8e02f5c3916bf7c8a5cb9d2eb6cbcc02/includes/lib/list-table.php#L265-L268
https://github.com/jerome-barbato/wp-steroids/blob/f4466d0f8e02f5c3916bf7c8a5cb9d2eb6cbcc02/includes/lib/list-table.php#L314-L317

Hi @markeasting, you are right. Now, the file uses "prepare" in all places.


Update to support React 16

These are the breaking changes: https://github.com/facebook/react/releases/tag/v16.0.0.
I tried running your tests with react 16.0.0 and made some fixes, still this test won't pass. If you have any idea of what it could be I can fix it and send a PR.

Maybe it's because of React 16's new SSR strategy? The style won't be updated in some situations. https://github.com/facebook/react/issues/10591

@cyl19910101 any ideas of what could fix it?

Should be fixed with v4.2.0


remove depreceated zappr

This PR removes the leftover from depreceated zappr

Done in https://github.com/hellofresh/kangal/pull/116


Documentations on how to use with typescript

Hi, I'm trying to use this with Angular2, but I don't see a documentations explaining how to use with typescript, is a detailed documentation a possibility in the future?

Hi, I added the definition files to typings (https://github.com/DefinitelyTyped/DefinitelyTyped/pull/8932). So to use it you have to do the following:

npm install qrcode-generator
typings install qrcode-generator --save --ambient
import qrcode = require('qrcode-generator'); inside your ts file

that is all to use it with intellisense and what not...

Hi, thanks for your response, but I know 2 ways of loading libraries like this:
import qrcode from 'qrcode-generator';
// or
let qrcode = require('qrcode-generator');

but I never saw that before:
import qrcode = require('qrcode-generator');

anyway, I tried those the only one that worked for me is:
let qrcode = require('qrcode-generator');

I am getting this error
Attempted to compile "qrcode-generator" as an external module, but it looks like a global module.

Hi, when do you get this error? At what step?

while installing typings, but tried it with --global flag it fixed the errors but there is another error,  It seems like createImageTag() (in typings) function has some issues may its createImgTag() instead.

Ok. I see the changes. At the time of writing this typing there were ambient typings, which are now global: https://github.com/typings/typings#updating-from-0x-to-10. I can check this other error later today. If you find anything feel free to tell us

I'm still can't get this to work with my settings even though it's working in plunker, in my app I'm using angular-cli.
Steps to reproduce:
make new app:
ng new newAppName
cd newAppName
ng serve

it works.
npm i qrcode-generator // (note this is missing the svg support).
ng serve // stil work

edit new-app-name.component.ts and import the qrcode-geenerator in it like this
// vscode underline the qrcode-generator string and complai about not finding it
import * as qrcode from 'qrcode-generator';

then ng serve which is still running errors out with this message:

/path/to/project/newAppName/tmp/broccoli_type_script_compiler-input_base_path-jscpZEq5.tmp/0/src/app/new-app-name.component.ts (3, 25): Cannot find module 'qrcode-generator'.

I tried installing the typings for it by adding this to the typings.json file:
"globalDependencies": {
    "qrcode-generator": "registry:dt/qrcode-generator#0.0.0+20160412152159"
  }

then running:
typings i

install successful, but still the same error.
angular-cli version:
angular-cli: 1.0.0-beta.5
node: 6.2.0
os: linux x64
am I missing something?
Is there other configuration to make for this to work?

As I see it angular-cli is still using typings version < 1, so it needs to be amientDependencies, or you manually update typings to version > 1 in package.json.

I tried that but nothing was installed, I removed the global folder under the typings folder, then I moved this line:
"qrcode-generator": "registry:dt/qrcode-generator#0.0.0+20160412152159"

from globalDependencies to ambientDependencies:
and then run this:
typings install --ambient qrcode-generator --save // or
typings install qrcode-generator --save

but I get:

└── (No dependencies)

and nothing was installed, typings in package.json is v0.8.1.

Hi, I was able to import this finally thanks to @JavascriptMick from angular-cli's gitter I did the following:
first make the format global:
'vendor/qrcode-generator': {
    format: 'global',
    main: 'qrcode.js'
  }

then when importing do it this way:
import 'qrcode-generator';
declare let qrcode;

Hope this help.

Example usage:
    l```

et qrCode = new QRCode();
qrCode.setErrorCorrectLevel(ErrorCorrectLevel.M);
qrCode.setTypeNumber(4);
qrCode.addData("test123");
qrCode.make();
    let base64ImageString = qrCode.toDataURL();

Example usage:
imports
import { QRCode ,ErrorCorrectLevel, QRNumber, QRAlphaNum, QR8BitByte, QRKanji } from "qrcode-generator-ts/js";

then
        let qrCode = new QRCode();
        qrCode.setErrorCorrectLevel(ErrorCorrectLevel.M);
        qrCode.setTypeNumber(4);
        qrCode.addData("test123");
        qrCode.make();
        let base64ImageString = qrCode.toDataURL();

Nice to see some sort of usage on here, in the complete absence of any kind of documentation.  However, while this is great, I have no clue what setTypeNumber even is, and why I need it.


Redirect user to same page before login page

Redirect user to same page before login page
Motivation and context
Should close #683
https://user-images.githubusercontent.com/30602242/162568877-f91a1fd7-9b1d-4baf-9cb1-f04e49983301.mp4

Ok. I will check.

@kachar I think now is much better.


Fix Cephfs plugin to return false to ValidateVolumeCapabilities if Block volume is specified

Cephfs doesn't have a feature to provide Block Volume, therefore it should return false to ValidateVolumeCapabilities if Block Volume is specified.
Fixes #44

@mkimuram thanks!


Create the authorization endpoint

Implemented by https://github.com/duckduck-team/quackit-api/commit/93458ff40e8336ca2f8f9161c9c47d7e4506dcac


[Android][InputEvolution] Fix number input default behaviour

Related Issue
Fixes #4522
Description
This fix changes the comparisson behaviours to match the new use of nullable properties for value, min and max in Input.Number. Now there are null checks instead of comparissons against Integer.MIN_VALUE or Integer.MAX_VALUE constants.
How Verified
The fix was verified manually and the results can be seen below:
Value set, Min/Max null/unset

Value null/unset

Microsoft Reviewers: Open in CodeFlow

@almedina-ms please remember to cherry pick this into release/20.07

@almedina-ms / @golddove, do we have the sample card for this test checked in ? If not please let;s get it in.