fix: flag descriptions / response messages

Proposed changes
This is just a simple pr fixing some cmd descriptions I noticed when I wanted to export a backup to s3.

The create job should give the ID. Far more useful than printing the bucket ID
The watch command operates on jobId, not bucketId
Problem with sharing flags cross packages. The description for snapshotId is not suitable in the export command. Nothing is restored, it's exported. Rewritten to be more generic.

Checklist

[x]  I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[ ] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[ ] I have updated test/README.md (if an e2e test has been added)
[ ] I have run make fmt and formatted my code

Further comments

Hi @raffis thanks for your contribution, can you please run make gen-docs to also update the docs?


dts: bindings: pinctrl: add '^pinctrl-[0-9]+$ pattern property

Instead of hardcoding up to 5 states.
Depends on https://github.com/zephyrproject-rtos/zephyr/pull/62921

Is there some case that is driving this?

Is there some case that is driving this?

Align with Linux bindings, it’s actually a documented Zephyr requirement unless it’s not possible.


[Question] About enabling the extra features.

I have notice there's some extra features (i.e. smart charging, charging plan, charging profile...)  available on the Dashboard but I am not sure how to enable them. I have browse through the page and issue page and I still couldn't find related instruction to enable those features. I would really appreciated if assistants can be provided, even the slightest hint is fine.
Thank you in advance.

According to my research, some of them do are available by default (i.e. charging plan, you just have to connect a charger that supports it), but others I think are paid feature, which you could consult with the SAP team. Also, I remember seeing smart charging in other repository from SAP, but I am not sure whether it is the one required for implement into ev-server


Fix missing const & bool return type of stopServer()

The global function stopServer() in the Vnc namespace has some problems: The header declares it as returning bool, but the cpp defines it returning void. Nevertheless, the cpp returns in its body the return value of invoking the singleton vncManager's member function stopServer, which is... void again. Also, the header declares the QWindow * parameter as const, whereas the cpp does not.
This PR aligns the global stopServer's signature with the VncManager's member function's signature, which is returning void, taking a const QWindow *, and fixes the definition's body by no longer returning something.

Thanks


Moved to protobuf-net.<IP_ADDRESS>0

Hello Richard,
I've moved your very usefull addon to protobuf-net.<IP_ADDRESS>0.
Please, check it and if you saw it as usefull, update nuget.

Brilliant stuff - merged and new downloads package, should be in NuGet soon

Thnak you!


[question] Could we build blazor project into pure js library?

While I have seen js interop in blazor. I was wonder that, could we build some C# utility library with blazor but not use any blazor component, and instead build it into pure js/wasm then include it into normal HTML/JS project. Then using DotNet.invokeMethod to call it in js file
Is there a project template for this pattern?

@Thaina thanks for contacting us.
This is a question better suited for the mono team.
In general I don't think it would be practical given that you would need to bring along the runtime and other code for it to work. It would also not be possible to call it with DotNet.invokeMethod as that's not something we've explored.


Compress Expected Large Tables

I wonder if you can add a flag to the GUI to allow for using SQL native DATA_COMPRESSION = PAGE for qualified SQL versions?  If it checks the repository database server and is developer or enterprise, allow the check box, otherwise grey out with a tooltip.  For tables like SlowQueries, keeping the default retention can mean a very large table if it is a very busy enterprise being monitored.  We have around eight repos for different things (separation of logic regions, etc.) and by enabling data compression on the SlowQueries table on all of them we saved 40% on the disk space.  Obviously we had to add it to the Switch.SlowQueries as well to allow the purging to happen for the retention.

Data compression is already used in a few places but it's possible I missed some cases where it might be beneficial.  We don't need to worry about versions/editions. The repository db requires SQL 2016 SP1 or later so we have compression on all editions.
Changing it for existing tables could be quite a time-consuming operation depending on the size of the tables.  The partition switching also adds some additional complexity as you noted.  If I implement the changes for #640 it should make it much easier for people to change the compression settings - The Switch tables would be removed.

Okay cool...yeah that will help and then it's just up to you to go compress yourself.  Sounds good.


Override Image resizeMode ?

Is this a bug report or a feature request?
Feature Request
Is it possible to override images resizeMode ? I've tried custom styles and renderersProps but neither reach the Image component.

Duplicate of #172, closing.


How can I format input field like real usa number

For example if someone type 9032XXXXX it should formatted something like  (903)-000-000. Or is there any way  I can define the id or the input box

It's an easy task with using https://github.com/nosir/cleave.js
Here's the live demo: https://jsfiddle.net/ad3quksn/127/


Add messages to auth login and logout views.

@maliabbas101 Thanks for this patch :+1: However, we don't want to add messages as a requirement for auth.


{storage-blob-preview} Fix Azure/azure-cli#23536: az storage blob upload: Clarify permissions for adding tags

This task can be performed by a Storage Blob Data Owner or a security principal that has been given permission to the Microsoft.Storage/storageAccounts/blobServices/containers/blobs/tags/write
More Info: https://docs.microsoft.com/en-us/azure/storage/blobs/storage-blob-index-how-to?tabs=azure-cli#upload-a-new-blob-with-index-tags
fixes Azure/azure-cli#23536

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

AzureStorage

Hi @navba-MSFT, thanks for contribution. Currently storage-blob-extension needs test re-record so the CI is blocked. Will merge this PR after I fix the tests.
BTW, can you make the change in Azure/azure-cli repo too?

@evelyn-ys I have created a PR for Azure CLI too https://github.com/Azure/azure-cli/pull/23686

Hi @navba-MSFT, thanks for contribution. Currently storage-blob-extension needs test re-record so the CI is blocked. Will merge this PR after I fix the tests.
BTW, can you make the change in Azure/azure-cli repo too?

@evelyn-ys I see the other PR (https://github.com/Azure/azure-cli/pull/23686) has been merged now. Thanks for your review.

Hi @navba-MSFT, thanks for contribution. Currently storage-blob-extension needs test re-record so the CI is blocked. Will merge this PR after I fix the tests.
BTW, can you make the change in Azure/azure-cli repo too?

@evelyn-ys Any update on this PR ?

@navba-MSFT Will really appreciate if you can help re-record all the failed tests

@navba-MSFT Will really appreciate if you can help re-record all the failed tests

@evelyn-ys I am unable to find the reason for test failure. It just says below:

/azp run

@navba-MSFT Will really appreciate if you can help re-record all the failed tests

@evelyn-ys I am unable to find the reason for test failure. It just says below:

It's just because the build has been too long to exist


The Code  Cannot be downloaded

On this web page(https://docs.microsoft.com/zh-cn/aspnet/web-forms/overview/data-
access/introduction/creating-a-data-access-layer-cs),
I want to download the code by clicking the link("下载示例应用程序或下载 PDF"),
but it is not successful. What should I do?

@ZhouHelen Thanks for your feedback. Your issue is transferred to Asp.Net team repo. They will help you there.


no module named 'adafruit_ble'

hello,
I used microbitV2 and esp32 with CircuitPython 9.0.3,but they all report the err：
ImportError: no module named 'adafruit_ble'
How could i install or find the adafruit_ble module ?
Thank you.

This guide explains how to install libraries like adafruit_ble: https://learn.adafruit.com/welcome-to-circuitpython/circuitpython-libraries

This guide explains how to install libraries like adafruit_ble: https://learn.adafruit.com/welcome-to-circuitpython/circuitpython-libraries

Is BLE supported on the ESP32 in CircuitPython?
https://forums.adafruit.com/viewtopic.php?p=1011494&hilit=LYWSD03MMC#p1011494

Is BLE supported on the ESP32 in CircuitPython?

Not yet: https://learn.adafruit.com/welcome-to-circuitpython/frequently-asked-questions#faq-3129409
Space is also limited on the ESP32 and if we add BLE we may need to remove some other things.


RM-156486 Release v0.5.2

Purpose
Adding WK CI so that it builds correctly,
Solution
High level overview of what was done. This is the roadmap for those who are going to CR.
Semantic Versioning (check one)

[ ] The following were changed in a non-backward compatible way and requires a major version bump:

[link to the breaking change in the diff]

[ ] Something public was added or changed in backward compatible way, this requires a minor version bump
[x] No public changes nor new features (backwards-compatible refactor or bug fix), so this can be included in a patch
release

How to QA

[ ] go build ./... runs successfully
[ ] Build passes
[ ] Skynet passes
[ ] Aviary passes
[ ] Rosie runs her checks

Security Insights
The items listed below may not capture all security relevant changes. Before providing a security review, be sure to review the entire PR for security impact.

(40) Vulnerable direct dependencies were detected
1 vulns in ajv < 6.12.3 via webui/internal/assets/yarn.lock
1 vulns in ansi-regex < 3.0.1 via webui/internal/assets/yarn.lock
1 vulns in async < 2.6.4 via webui/internal/assets/yarn.lock
1 vulns in color-string < 1.5.5 via webui/internal/assets/yarn.lock
1 vulns in cryptiles < 4.1.2 via webui/internal/assets/yarn.lock
1 vulns in deep-extend < 0.5.1 via webui/internal/assets/yarn.lock
1 vulns in diff < 3.5.0 via webui/internal/assets/yarn.lock
1 vulns in extend < 3.0.2 via webui/internal/assets/yarn.lock
1 vulns in fstream < 1.0.12 via webui/internal/assets/yarn.lock
1 vulns in glob-parent < 5.1.2 via webui/internal/assets/yarn.lock
1 vulns in gopkg.in/yaml.v2 < 2.2.8 via go.sum
1 vulns in hawk < 9.0.1 via webui/internal/assets/yarn.lock
1 vulns in hoek < 4.2.1 via webui/internal/assets/yarn.lock
1 vulns in hosted-git-info < 2.8.9 via webui/internal/assets/yarn.lock
1 vulns in ini < 1.3.6 via webui/internal/assets/yarn.lock
1 vulns in jsdom < 16.5.0 via webui/internal/assets/yarn.lock
1 vulns in json-schema < 0.4.0 via webui/internal/assets/yarn.lock
1 vulns in macaddress < 0.2.9 via webui/internal/assets/yarn.lock
1 vulns in mem < 4.0.0 via webui/internal/assets/yarn.lock
1 vulns in nth-check < 2.0.1 via webui/internal/assets/yarn.lock
1 vulns in nwmatcher < 1.4.4 via webui/internal/assets/yarn.lock
1 vulns in path-parse < 1.0.7 via webui/internal/assets/yarn.lock
1 vulns in postcss < 7.0.36 via webui/internal/assets/yarn.lock
1 vulns in randomatic < 3.0.0 via webui/internal/assets/yarn.lock
1 vulns in react-dom < 16.2.1 via webui/internal/assets/yarn.lock
1 vulns in sshpk < 1.13.2 via webui/internal/assets/yarn.lock
1 vulns in stringstream < 0.0.6 via webui/internal/assets/yarn.lock
1 vulns in underscore < 1.12.1 via webui/internal/assets/yarn.lock
1 vulns in y18n < 3.2.2 via webui/internal/assets/yarn.lock
1 vulns in yargs-parser < 13.1.2 via webui/internal/assets/yarn.lock
10 vulns in handlebars < 4.7.7 via webui/internal/assets/yarn.lock
2 vulns in braces < 2.3.1 via webui/internal/assets/yarn.lock
2 vulns in elliptic < 6.5.4 via webui/internal/assets/yarn.lock
2 vulns in is-svg < 4.3.0 via webui/internal/assets/yarn.lock
2 vulns in js-yaml < 3.13.1 via webui/internal/assets/yarn.lock
2 vulns in node-fetch < 2.6.7 via webui/internal/assets/yarn.lock
3 vulns in minimist < 1.2.6 via webui/internal/assets/yarn.lock
3 vulns in ua-parser-js < 0.7.24 via webui/internal/assets/yarn.lock
6 vulns in tar < 4.4.18 via webui/internal/assets/yarn.lock
7 vulns in lodash < 4.17.21 via webui/internal/assets/yarn.lock

(1) Security relevant changes were detected
Watched file aviary.yaml added

Action Items

Request infosec review via Slack: #support-infosec
Review dependencies for available updates
See this Splunk dashboard for more CVE details
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

Test comment to see if Rosie is listening

Rosie please try again

QA +1

@Workiva/release-management-p


done(err) when err.output.payload has been modified to string

When err returned through done(err) callback has been modified to a non-object such as a string the use of JSON.stringify() will encapsulate the string in extra quotations. For example if middleware modify the err.output.payload to be in XML format as type string the HTTP returned payload will include extra quotes breaking XML formatting and likely external app parsing.
Also consider adding http 'content-type' headers when outputting errors assuming that isn't against some standard.
done() handler correctly identifies this case when being called without err on line 96;
if (isStream(val)) {
stream = val
} else if (typeof val === 'object') {
res.setHeader('Content-Type', 'application/json')
stream = fromString(stringify(val))
} else if (typeof val === 'string') {
stream = fromString(val)
}

Nice catch; do you by any chance have any example input / output we could use to generate a regression test from? Thanks!

Sure, also I borked the pull request using instanceof my apoligies. It should be if(typeof payload == 'string').
if i use the done callback as follows:
var err = merry.error({status.Code: 404));
err.output.payload = js2xmlparser.parse('response', err.output.payload);
return done(err);
The payload == "\n\n    404\n    Not Found\n"
Including the literal quotes as shown

Merged in #62

v4.2.0


Remove what3words integration

Coverage increased (+1.3%) to 42.13% when pulling 4bc3a61a175fc42b4c21f76d1b7ed31c616c447a on remove_w3w into 598c588f7b8dadeb29da28e38d99862311486cfa on master.

Coverage increased (+1.4%) to 42.281% when pulling f91991d486fd01dd1630cefdc023f9b339f87f9f on remove_w3w into 598c588f7b8dadeb29da28e38d99862311486cfa on master.

Coverage increased (+1.4%) to 42.303% when pulling 51357441a74da620f1cf72756f8f0882414ebc1e on remove_w3w into 598c588f7b8dadeb29da28e38d99862311486cfa on master.


fix dynamic properties PHP 8.2

Hi,
on php 8.2 I get errors like this:
Deprecated: Creation of dynamic property Ratchet\Server\IoConnection::$httpBuffer is deprecated

@krajcikondra The test suite doesn't execute the tests with PHP 8.2 currently and we can't see if your changes are completely fixing the issue, but I am looking into that, see #1003 for more details. Once I get everything updated, we can come back to this one. Depending on how far I go with #1003, I might need to apply the same changes as you did, I'll keep you posted :+1:


ImportError: cannot import name 'Client'

In my screenshot below you can see i successfully installed python-binance (via pip), but when i run my script, it is not recognizing your module. I am using "from binance.client import Client"

Hi @queezyy, the problem is that you named your script "binance" and it interferes with the correct package import. Rename it to something different and try to re-run it. Hope this helps.

In my screenshot below you can see i successfully installed python-binance (via pip), but when i run my script, it is not recognizing your module. I am using "from binance.client import Client"

I had the same issue.
A lot of the .py files were empty or largely commented out - I realised I had multiple instance of python installed, so I cleared them, then uninstalled python-binance "pip uninstall python-binance" then re-installed without cached files, making sure to use the last working version which is 0.7.4 "pip --no-cache-dir -v 0.7.4 python-binance" (I think that's how the command is structured)
Hope that helps!

Same problem here !! It suddently fails

update your pip and install python-binanece again,it will done.

from binance import *
api_key = 'you_key'
api_secret = 'you_key'
client = set(api_key,api_secret)
this work for me

client = set(api_key,api_secret)
this work for me

Does it ?
    client = set(api_key, api_secret)
TypeError: set expected at most 1 argument, got 2

For me the issue was fixed when I renamed local binance.py file to something else .py


feat(ci): Check that scope has been included in every PR

Motivation
We would like to include scope in PR titles, as otherwise the sheer volume of PRs makes it hard to find PRs affecting a specific canister.
Changes

Introduce a GitHub action that checks PR titles.

TODO: A repository admin would have to make this check required, or make all jobs with names ending with :required required.
Tests
This is the same incantation I am pushing in in various other repos.  I tested it extensively here: https://github.com/dfinity/orbit/pull/288

Just provided a PR with incorrect label therefore I allow myself to merge this PR that way I start using correct titles as we decided.


Todo

[ ] update Travis script to use latest approach (see [this] (https://www.fpcomplete.com/blog/2016/02/updated-haskell-travis-config))
[ ] add GHCJS build on Travis
[ ] relax base dependency to allow version 5 too (remove upper bound)
[ ] remove git locations and use packages from Stackage

use packages from Stackage
Currently all our packages is in nightly Stackage snapshot.  It is bound to GHC 8, but GHCJS built with it does not work properly yet.

relax base dependency to allow version 5 too (remove upper bound)
No need to allow base-5 because GHC 8 uses version 4.9.

Closing this due to #6.


BUG? Unchecked runtime.lastError: Cannot find menu item with id vue-inspect-instance

Version
4.1.5-beta.1
Browser and OS info
chrome73.0.3683.75
Steps to reproduce
I install vue-devtools from chrome store, and in chrome devtools I drag Vue from last to second place, that is happened. and I could not find Vue devtools item anymore, and it has a error in extensions:Unchecked runtime.lastError: Cannot find menu item with id vue-inspect-instance,but my other project could find Vue devtools in chrome devtools.

What is expected?
Vue devtools should in chrome devtools
What is actually happening?
Vue devtools not in chrome devtools

Hi, xuejiangjun.
It was similar to my case.
Did you install the vuejs devtools while you make the app?
In my case, I restart the vuejs app project and solved.

Hi, xuejiangjun.
It was similar to my case.
Did you install the vuejs devtools while you make the app?
In my case, I restart the vuejs app project and worked.

I restart and reinstall it  many times but dosent work, so I give up. I do not know what is the reason, maybe it is chrome's bug.

i have the same problem, same Devtools version,same chrome version, (and my computer is mac pro).
Has your problem been solved?

i have the same problem, same Devtools version,same chrome version, (and my computer is mac pro).
Has your problem been solved?

no

@renyuns 我复现一下这个问题：我把vue-devtools安装之后，把它从最后拉到前面 console后面，然后一会就不起作用了，我就关了F12重新打开，然后发现Vue没有了，不管怎么重装活着是禁用开启，都是报这个错误，估计是谷歌或者是插件的问题。

@xuejiangjun 那就不知道了🤣

Duplicate of #629


DisplayName not displaying

Hello ! I have LuckPerms 1.12.2 with VaultChatFormatter but the displayname isn't working.

Do you mean display name, or prefix/suffix? Because these are both entirely different things.

Also 1.12.2 is the server version, not the LP version

I mean displayname.. I don't know how to set prefixes/suffixes..

And the version of luckperms is:
LuckPerms-Bukkit-4.4.1

VaultChatFormatter doesn't change the displayname of the player. It is only for adding prefixes and suffixes and for basic chat formatting (setting colours)
You need another plugin like EssentialsX and EssentialsXChat to display the displayname.

The wiki is your friend: https://github.com/lucko/LuckPerms/wiki/Prefixes,-Suffixes-&-Meta

Ok i Will set the prefix when i'm back To my home. I Will inform you for the advencement

Ok it's working ! Thank you for the support !


Reading Nihon files does not give correct voltage

Describe the bug
I am using the mne.read_raw_nihon() function to read in an EEG file along with its accompanying 21E, LOG, and PNT files. However, when I plot() the raw data, some of the channels have voltages that are 10,000x the voltages of the other channels. I tried using a different tool to convert my Nihon file into an EDF, and after opening this EDF, I saw that the voltage amplitudes were much more reasonable. Without knowing too much about Nihon's file format, I'm guessing that there is some sort of unit_multiplier that MNE isn't reading in that the other tool did notice.
For my personal workflow, I can just use the other tool to generate EDFs and then read them into MNE, but I think it would benefit the project to address this bug.
Steps to reproduce
My file contains patient data, so I cannot share it at the moment. However, I did manually inspect the EEG file, and it looks like the header is EEG-1100C V01.00. Not sure if that helps
Expected results

Actual results
Notice how far off T7, T8, P7, and P8 are. I tried zooming out within MNE, and the signals look fine if you zoom out enough, so I'm pretty sure it's just a scaling issue

Additional information
Platform:      Windows-10-10.0.19041-SP0
Python:        3.6.4 (v3.6.4:d48eceb, Dec 19 2017, 06:54:40) [MSC v.1900 64 bit (AMD64)]
Executable:    c:\Users\seand\eeg_proj\venv\Scripts\python.exe
CPU:           Intel64 Family 6 Model 142 Stepping 9, GenuineIntel: 4 cores
Memory:        7.9 GB

mne:           0.23.4
numpy:         1.19.5 {blas=D:\\a\\1\\s\\numpy\\build\\openblas_info, lapack=D:\\a\\1\\s\\numpy\\build\\openblas_lapack_info}
scipy:         1.5.4
matplotlib:    3.3.4 {backend=Qt5Agg}

sklearn:       0.24.2
numba:         Not found
nibabel:       Not found
nilearn:       Not found
dipy:          Not found
cupy:          Not found
pandas:        1.1.5
mayavi:        Not found
pyvista:       Not found
vtk:           Not found

Okay, I think I found the problem! The problematic channels are T7, T8, P7, P8, FT9, and FT10. The reason for this is in the definition of the function _map_ch_to_specs, which will use a different digital-to-physical conversion for channel names that are not in _default_chan_labels, and apparently these channels that I have are not default. My proposed solution would be to add these channels to the list of default channels, since I think that these labels will always be associated with EEG channels.

But what is this funtion "_map_ch_to_specs" doing? What is the meaning? I got no clue.


fix: Correct default_order for not case-sensitive StringBones

Without the .idx suffix an empty result is returned

Hm, indeed this is a possible fix, but on the other side the order setting should be generally handled by the bones, and not in the prototypes, like in BaseBone.buildDBSort.

Yeah, absolutely. After I found this bug I thought the same thing. But after looking at buildDBSort, I saw this only works on queries, we don't have here. Therefore, it was too much work for a hotfix. I'd appreciate to use the help of the bones, but for now let's go with this fix to get things working.

But after looking at buildDBSort, I saw this only works on queries, we don't have here.

I've made an alternative proposal for this PR in #1169. It's not perfect, but uses the provided facilities.
Furthermore, #1077 was updated and tested now, and I would like to merge it into develop.

Replaced by https://github.com/viur-framework/viur-core/pull/1169


Buy side representation

The buy side is not only the part of the market "where the money comes from", but advertiser are technically deep integrated into the ecosystem, providung identities with associated data. To leave them outside of a governing structure would be a fundamental flaw, which which would make it difficult to impossibl to establish market-wide support for the concept. Industry frameworks like the European Transparency and Consent Framework (TCF) made the same mistake and are paartially crippled by this up to the present.

I am well aware of the role the buy-side plays — that is why it is flagged as an issue. If you have suggestions for how to develop a legitimate polity on that side, I will definitely take them.
The TCF was primarily designed to enable intermediaries to continue business as usual (and revenue for IAB Europe). It's certainly not a model to follow. (Publishers weren't exactly taken into account, either.)
Anyway — if you have suggestions, I'll take them.


Added a Request Filter (blocks 'seen' receipts, 'typing' indicator)

Hi @Aluxian;
I re-implemented this NW.js-based component from version 1.0.
The feature is implemented as a request-filter module (common -> service), which exports an object with enable() and disable() method. It's URL-agnostic, can be repurposed in the future (e.g., ad-blocking). It adds a new 'Privacy' main top-level menu.
It uses Electron's WebRequest API, which has become stable and performant.
Code-style specification was applied as per your ESLint configuration. Closes #542.
Please review & comment, cheers,
S

Hey, that’s fantastic! I’m slightly pressured with exams right now so it might take me 2 weeks to merge this, but I’ll surely do it.
Thank you! :) I see you closed this, i assume you’re going to open a cleaner PR?
--
Alexandru Rosianu
Full-Stack Developer | aluxian.com

On 10 Jan 2017, at 10:26, Sidney<EMAIL_ADDRESS>wrote:
Hi @Aluxian https://github.com/Aluxian;
I re-implemented this NW.js-based component from version 1.0.
The feature is implemented as a request-filter module (common -> service), which exports an object with enable() and disable() method. It's URL-agnostic, can be repurposed in the future (e.g., ad-blocking). It adds a new 'Privacy' main top-level menu.
It uses Electron's WebRequest API http://electron.atom.io/docs/api/web-request/, which has become stable and performant.
Code-style specification was applied as per your ESLint configuration.
Please review & comment, cheers,
S
You can view, comment on, or merge this pull request online at:
https://github.com/Aluxian/Messenger-for-Desktop/pull/1159 https://github.com/Aluxian/Messenger-for-Desktop/pull/1159
Commit Summary
Merge pull request #520 from Aluxian/staging
Merge pull request #524 from Aluxian/staging
Update CHANGELOG.json
Merge pull request #546 from Aluxian/staging
Remove screenshot
Update package.json
Merge pull request #1141 from Aluxian/develop
Merge pull request #1142 from Aluxian/develop
Merge pull request #1143 from Aluxian/develop
Merge pull request #1144 from Aluxian/staging
Merge pull request #1145 from Aluxian/develop
Merge pull request #1146 from Aluxian/develop
Merge pull request #1147 from Aluxian/develop
Merge pull request #1149 from Aluxian/staging
Merge branch 'master' into deploy
Merge pull request #1150 from Aluxian/deploy
menu(main): add 'privacy' template
menu(main): add 'blockSeenTyping' method to click expressions
utils(prefs): add 'block-seen-typing' to defaults
services(common): add 'request-filter' module
config(gitignore): add IntelliJ .idea
File Changes
M .gitignore https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-0 (1)
M CHANGELOG.json https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-1 (2)
M src/package.json https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-2 (6)
M src/scripts/browser/menus/expressions/expr-click.js https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-3 (17)
M src/scripts/browser/menus/main.js https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-4 (1)
A src/scripts/browser/menus/templates/main-privacy.js https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-5 (17)
M src/scripts/browser/utils/prefs-defaults.js https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-6 (1)
A src/scripts/common/services/request-filter.js https://github.com/Aluxian/Messenger-for-Desktop/pull/1159/files#diff-7 (33)
Patch Links:
https://github.com/Aluxian/Messenger-for-Desktop/pull/1159.patch https://github.com/Aluxian/Messenger-for-Desktop/pull/1159.patch
https://github.com/Aluxian/Messenger-for-Desktop/pull/1159.diff https://github.com/Aluxian/Messenger-for-Desktop/pull/1159.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Aluxian/Messenger-for-Desktop/pull/1159, or mute the thread https://github.com/notifications/unsubscribe-auth/ABWmdpkGURHBEJ3jfN67eEzki-Woj5Xaks5rQ1zrgaJpZM4LfP6N.


✨ Implement RAW capture ($2,000)

What feature or enhancement are you suggesting?
It would be great to be able to capture RAW images (Apple ProRAW or Bayer) in VisionCamera. See this apple guide.
For $2,000, I will implement RAW support in VisionCamera.
What Platforms whould this feature/enhancement affect?
iOS, Android
Alternatives/Workarounds
RAW Capture is not possible today.
Additional information

[X] I agree to follow this project's Code of Conduct
[X] I searched for similar feature requests in this repository and found none.

$2,000
I will implement RAW capture for $2,000. Donate to this polar pool to support this feature.

if i implement it can i get that?

No, because no one donated to the pool yet. It's at $0.

Is there any update on this?

Again, this issue is a sponsor pool. Once it's full, I'll implement this.


Added title, description, and og-image to og:metadata (#1113)

Below is what the og link looks like on the facebook debugger: (I don't know how to get the facebook debugger to get the image even with a tunnel)

Here is an example of what is currently generated from the platformIcon list on the og-image:

Here is an example as to how the og-image will readjust automatically to the inclusion of more icons in the list:

Updated design:

Updated design:

Final:


[SPARK-37818][DOCS] Add option in the description document for show create table

What changes were proposed in this pull request?
Add options in the description document for SHOW CREATE TABLE  command.

Why are the changes needed?
#discussion
Does this PR introduce any user-facing change?
No
How was this patch tested?
SKIP_API=1 SKIP_RDOC=1 SKIP_PYTHONDOC=1 SKIP_SCALADOC=1 bundle exec jekyll build

@Peng-Lei could you post a screenshot of the new doc in the PR description?

@Peng-Lei could you post a screenshot of the new doc in the PR description?

done. Thank you very much @gengliangwang

Thanks, merging to master/3.2


Charity Events page/section

Add a section to the site to advertise charity events.  At the basic, include a link to more information (such as the Eventbrite page).  Then, include past events, and advertising assets
I want to use this for Would I Lie To You.
Checklist:

[x] Event header and section
[x] Add WILTY
[ ] Add ability to add House Event & fundraisers
[ ] Optimise for mobile
[x] Centre the "Next week is..."
[ ] Handling of past events
[ ] Add past events
[ ] Add event dashboard for event info, like money raised, people who attended, text updates from organisers - see #12, not for this issue's PR.

Was thinking something like this:

Have it be controlled by a JSON file, which has the PR image set by a URL to what is in public.

ad30c60 completes up to the layout and design.
Implementing a JSON based system to "automagically" handle event listing shouldn't be too hard

Past events (list such that most recent is first on the website):

The Zoom Quiz - June 2020
Weakest Link? - Unknown, say Spring 2020
WILTY 2021 - January 2021
Echo Dot Raffle - February 2020
House Touch Rugy - Beaufort won. Autumn 2020.

Future events:

House Cross Country - date unknown, but in progress.

I think for events that do not have a header image either something abstract and generic, or just put the event name in a different font maybe

I think for events that do not have a header image either something abstract and generic, or just put the event name in a different font maybe

Event name in a different font?

Event name in a different font?

In terms of house events, what to include:

Start date/term
Event name
Coming up, ongoing, done
If ongoing/done, who winning/won?
?

In terms of house events, what to include:

Start date/term
Event name
Coming up, ongoing, done
If ongoing/done, who winning/won?
?


Add Private Swarm Key Support

This PR follows the release of go-libp2p v0.24.0 by now providing a -swarmkey argument which allows users to supply a multicodec-encoded v1 PSK when launching their relay daemon.
Closes #16
Signed-off-by: Oleg S<EMAIL_ADDRESS>

@marten-seemann I've opened PR #29 to address dependency issues, besides that, was there anything else you needed from me to get this PR merged?

Sorry, I missed that PR. Merged now. Can you rebase please?

@marten-seemann I've rebased, let me know if there's any issues or anything else you'd need.


debian postinst should not move file installed by package

Summary
debian postinst should not move file installed by package
Steps To Reproduce

# deb -i kong-community-edition-0.14.0.stretch.all.deb

$ dpkg -L kong-community-edition | grep kong.conf.default
/usr/local/lib/luarocks/rocks/kong/0.14.0-0/kong.conf.default

$ ls /usr/local/lib/luarocks/rocks/kong/0.14.0-0/kong.conf.default
/bin/ls: cannot access '/usr/local/lib/luarocks/rocks/kong/0.14.0-0/kong.conf.default': No such file or directory

Additional Details & Logs

Kong version ($ kong version)
0.14.0
Kong debug-level startup logs ($ kong start --vv)
N/A
Kong error logs (<KONG_PREFIX>/logs/error.log)
N/A
Kong configuration (the output of a GET request to Kong's Admin port - see
https://getkong.org/docs/latest/admin-api/#endpoint)
N/A
Operating system
Debian Stretch

Cause
The postinst script in the deb contains:
mv /usr/local/lib/luarocks/rocks/kong/0.14.0-0/kong.conf.default /etc/kong/kong.conf.default
Please review the requirements for debian package scripts at https://www.debian.org/doc/debian-policy/ch-maintainerscripts.html .  As one example, this postinst script fails to be idempotent.

Thanks for the report! This behavior has been changed in how we package Kong for all OS distributions to no longer require a postinst script. This behavior went out with the Kong 1.0 release.


Make min_max_speed_davis default wave speed estimate for FluxHLL()

See https://github.com/trixi-framework/Trixi.jl/issues/1721 and https://github.com/trixi-framework/Trixi.jl/issues/1726
In this version, we keep min_max_speed_naive.

The changes in the test directory stem from those elixirs for which the speed estimates have been exchanged.


Update READM.md file with new usage instructions

[ ] Make a docker-compose.yml
[ ] Update README.md

solved


Is this crate maintained?

If not, I believe that this repository and crate should either be transferred or a fork should replace it.
If you anyone is available, can you please respond with any preferences as to how you'd prefer this crate be treated?
Thanks!

I have tried contacting the author on that matter on the 23rd of January but have not yet received a response.
I have also contacted crates.io (out of curiosity) to know whether this issue had already happen and if an action was taken. They will not transfer ownership of a crate without approval of the author (which is understandable), and they suggested development to be revived through a new crate (whether a fork or not) and that people should update their dependencies.

Hi @Ethiraric! Thanks for the quick response!

Dupe of #160 and #192

chyh1990, owner of this crate, may be Chen Yuheng(陈宇恒), who is one the co-founders of SenseTime(商汤科技). SenseTime is one of the best artificial intelligence (AI) software providers in China. If I'm right, he must be really busy and have little spare time.
His email address proves this<EMAIL_ADDRESS>


Can we add stakers to this network? New validators were not discovered

When network was up using the preset validators and stable and producing blocks, I added a new validator and tried to add it to the network. However, it was not detected as a new validator to stake.

I run the network and made sure it is working with 2 geth nodes, 2 beacon nodes, and 1 validator node connected with the first beacon node.
I created another validator key using the deposit tool deposit new-mnemonic starting from index 0
I called my webmethod that makes the deposit, passing it my filled geth account public and private keys, and the deposit key generated by the deposit tool

echo {\"keys\":$(cat `ls -rt /root/validator_keys1/deposit_data* | tail -n 1`), \"address\":\"0xF359C69a1738F74C044b4d3c2dEd36c576A34d9f\", \"privateKey\": \"0x28fb2da825b6ad656a8301783032ef05052a2899a81371c46ae98965a6ecbbaf\"} > /root/validator_keys1/payload.txt
curl -X POST -H "Content-Type: application/json" -d @/root/validator_keys1/payload.txt http://localhost:8005/api/account/stake

My web method calls the deposit contract's deposit method

export const deposit = async(depositKeys, account, pvtKey) => {
    const provider = new ethers.providers.JsonRpcProvider(process.env.RPC_URL); // connect to the network

	// Getting the contract
    const signer = new ethers.Wallet(pvtKey, provider); // create a wallet with the provided private key
    const contract = new ethers.Contract(process.env.DEPOSIT_CONTRACT_ADDRESS, contractAbi, signer); // get the deposit contract

    for (let key of depositKeys) {
        const { pubkey, withdrawal_credentials, signature, deposit_data_root } = key; // get the relevant data from the raw depositData

        // call the deposit function on the contract (and prepend 0x to the deposit data)
        const tx = await contract.deposit(prefix0X(pubkey), prefix0X(withdrawal_credentials), prefix0X(signature), prefix0X(deposit_data_root),
            {
                chainId: parseInt(process.env.CHAIN_ID), // add the chain ID to the transaction
                value: ethers.utils.parseEther('32') // the transaction value
            }
        );

        const w = await tx.wait();

        // stop on failure
        if (w.status == 0) throw new Error('TRANSACTION FAILURE: ' + JSON.stringify(w));

        console.log('>>> TX', tx);
        console.log('>>> W', w);
    }
}

The deposit function returned json with the transaction info and status=1
The deposit transaction was logged in geth to my contract address 0x4242...
Submitted transaction hash=0x944ce52c06d9518c436847dcda2bd144552daf96257b132afe16e188e824af6e from=0xF359C69a1738F74C044b4d3c2dEd36c576A34d9f nonce=0 recipient=0x4242424242424242424242424242424242424242 value=32,000,000,000,000,000,000
Started a new validator node connecting to the second beacon node:

clients/lodestar validator \
	  --dataDir "./data/consensus/1" \
	  --beaconNodes "http://<IP_ADDRESS>:9597" \
	  --suggestedFeeRecipient "0xF359C69a1738F74C044b4d3c2dEd36c576A34d9f" \
	  --graffiti "YOLO MERGEDNET GETH LODESTAR" \
	  --paramsFile "./consensus/config.yaml" \
	  --importKeystores "/root/validator_keys1" \
	  --importKeystoresPassword "/root/validator_keys1/password.txt" \
	  --logLevel debug

The new validator is not discovered (Count=0)

Lodestar network=mergednet, version=v1.2.1/e76d4c9, commit=e76d4c9cda242f5f3465f5168d8b36a2c3031d53
Connecting to LevelDB database path=data/consensus/1/validator-db
1 local keystores
0xa6f3a9bc85181812cd003d0cc476c591e7ef5f278f5ca5450d3a07e1f32463ac29e92caaeb5c17ade8f56f9836031f9c
Genesis fetched from the beacon node
Verified connected beacon node and validator have same the config
Persisted genesisValidatorRoot 0xe63460dc044e056f26ca8f7406a18867d31f1ec195322f428b5918d4b0153050
Persisted genesisTime<PHONE_NUMBER>
Verified connected beacon node and validator have the same genesisValidatorRoot
Discovered new validators count=0

All the logs are here
Is there something I am doing wrong?

We tried setting ETH1_FOLLOW_DISTANCE: 5
We also tried setting SLOTS_PER_EPOCH: 6 and EPOCHS_PER_ETH1_VOTING_PERIOD: 2, but these values were reverted back to the large values from mainnet when we queried http://<IP_ADDRESS>:9596/eth/v1/config/spec
curl http://<IP_ADDRESS>:9596/eth/v1/beacon/states/head/validators shows 8 validators, we hoped it would be 9 as we added our validator.
we waited half a day and still the same results

Everything looks good. After the deposit it takes 8-12 hours usually to activate the validator. Maybe wait a bit longer?
There is one thing:
Discovered new validators count=0

It looks like your new validator does not correctly load the keys. Can you check that?

I generated many validator_key groups.
A group I put in the consensus/mnemonic.yaml file is discovered when I run the validator.
But a group that I don't put in mnemonic.yaml is not discovered.
What else can I do?


message_list_hover: Convert module to TypeScript.

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Rebased to resolve conflicts.

Fixed the bug above and merged this. Hopefully the lesson of that is helpful -- is you see some dead code, be very careful about fixing it to work as apparently intended without testing, because it might be that the code being dead is what's correct.


Removing hyphen

since user ids on telegram doesn't require a hyphen there's no need for that here.

-1 is an invalid value for user_id and it is intended to be like that. User should replace the whole trunk instead of just the numbers.


Dispatcher::fire Error

DELETE--plz

Please don’t use dev master, as it has non working and untested code.


Unable to build

Trying to install on Linux Mint 20.1 (Ubuntu 20.04) with no success due to obsolete dependences g++-9-multilib (>= 9.3.0-3~) and gcc-9-multilib (>= 9.3.0-3~). Any suggestions?
Thx in advance.

I am unable to duplicate this problem. Did it occur when running ./toolChain or ./load.nmr

The problem is during ./toolChain
whatsin script shows:
edi@YogaEdi:~/ovjbuild/bin$ ./whatsin
../logs/makeovjlog
/usr/include/stdio.h:27:10: fatal error: bits/libc-header-start.h: No such file or directory
/usr/include/c++/9/iomanip:38:10: fatal error: bits/c++config.h: No such file or directory
/usr/include/stdio.h:27:10: fatal error: bits/libc-header-start.h: No such file or directory
/usr/include/stdio.h:27:10: fatal error: bits/libc-header-start.h: No such file or directory
/usr/include/stdio.h:27:10: fatal error: bits/libc-header-start.h: No such file or directory
src/IMAGE/user_templates/ib_initdir/math/functions/src/ufuncs.c:12:10: fatal error: sys/param.h: No such file or directory
/usr/include/stdlib.h:25:10: fatal error: bits/libc-header-start.h: No such file or directory
scons: *** [asyncIo.o] Error 1
scons: *** [AcodeBuffer.os] Error 1
scons: *** [CSfuncs.o] Error 1
scons: *** [asyncIo.o] Error 1
scons: *** [CSfuncs.o] Error 1
scons: *** [src/IMAGE/user_templates/ib_initdir/math/functions/src/ufuncs.o] Error 1
scons: *** [src/IMAGE/user_templates/ib_initdir/math/functions/src/addms.o] Error 1
scons: building terminated because of errors.
scons: building terminated because of errors.
scons: building terminated because of errors.
scons: building terminated because of errors.
scons: building terminated because of errors.
scons: building terminated because of errors.
In ../logs/makeovjlog there are :
Total of  0  ERRORS .
0  Fatal errors .
7  fatal errors .
0  ld: fatal: .
0  acomp failed .
0  cannot access .
0  cannot find .
7  No such file .
0  syntax error .
0  undefined reference .
7  Error 1 .
0  Error copying .
0  does not exist .
0  Permission denied.
0  Unable to remove.
0  File format not recognized.
0  File exists.
6  building terminated.
6  warnings .
I can also attach makeovjlog if desired ;)
Thx

./toolChain shows the annoying error:
edi@YogaEdi:~/ovjbuild/bin$ ./toolChain
If requested, enter the admin (sudo) password
[sudo] password for edi:
Ign:1 http://dl.google.com/linux/earth/deb stable InRelease
Ign:2 http://ddebs.ubuntu.com focal InRelease
Hit:3 http://archive.canonical.com/ubuntu focal InRelease
Hit:4 http://ppa.launchpad.net/scribus/ppa/ubuntu focal InRelease
Hit:5 http://archive.ubuntu.com/ubuntu focal InRelease
Ign:6 http://ddebs.ubuntu.com focal-updates InRelease
Hit:7 https://dl.winehq.org/wine-builds/ubuntu focal InRelease
Hit:8 http://dl.google.com/linux/earth/deb stable Release
Hit:9 http://ddebs.ubuntu.com focal Release
Hit:10 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Hit:11 http://ddebs.ubuntu.com focal-updates Release
Ign:12 http://packages.linuxmint.com ulyssa InRelease
Hit:13 https://repo.skype.com/deb stable InRelease
Hit:14 http://archive.ubuntu.com/ubuntu focal-backports InRelease
Hit:15 http://security.ubuntu.com/ubuntu focal-security InRelease
Hit:16 https://packages.microsoft.com/repos/ms-teams stable InRelease
Hit:17 http://packages.linuxmint.com ulyssa Release
Reading package lists... Done
Building dependency tree
Reading state information... Done
All packages are up-to-date.
Reading package lists... Done
Building dependency tree
Reading state information... Done
g++ is already the newest version (4:9.3.0-1ubuntu2).
g++ set to manually installed.
gcc is already the newest version (4:9.3.0-1ubuntu2).
gfortran is already the newest version (4:9.3.0-1ubuntu2).
gfortran set to manually installed.
libglu1-mesa-dev is already the newest version (9.0.1-1build1).
libglu1-mesa-dev set to manually installed.
make is already the newest version (4.2.1-1.2).
scons is already the newest version (3.1.2-2).
git is already the newest version (1:2.25.1-1ubuntu3.1).
libx11-6 is already the newest version (2:1.6.9-2ubuntu1.2).
libx11-dev is already the newest version (2:1.6.9-2ubuntu1.2).
libx11-6:i386 is already the newest version (2:1.6.9-2ubuntu1.2).
libx11-6:i386 set to manually installed.
libc6-dev is already the newest version (2.31-0ubuntu9.3).
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies.
g++-multilib : Depends: g++-9-multilib (>= 9.3.0-3~) but it is not going to be installed
gcc-multilib : Depends: gcc-9-multilib (>= 9.3.0-3~) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.
Ubuntu package installation complete

Apparently, the Mint download mirrors are not equally up to date. Could you use the iso from the USA kernel.org site, which is the one I used. The checksum is
cksum linuxmint-20.1-cinnamon-64bit.iso
914753724<PHONE_NUMBER> linuxmint-20.1-cinnamon-64bit.iso

Finally managed to successfully install.
After googling answer was sudo aptitude install gcc-multilib -> n -> y (to older version)
and then sudo aptitude install g++-multilib
After that ./toolChain worked properly and ./buildovj also. Then ./load.nmr successfully installed the system.
Thx for ur help, DanIverson 😊


Comments

Hi,
Thanks for this, it's impressive how you squashed a playable game into 512 bytes! Just a few questions and comments after completing the game.

Seems like armour and health always spawn in pairs like this:

.........
...X.X...
.........

Is that intentional?

On one level, I think I spawned on top of a monster which was a bit confusing

Seems like gold doesn't have a purpose?

If you could find a way to fit them, indicators for dungeon level, armor and weapon level would improve the game a lot in my opinion. Even if you drop gold or some other features :)

On the way out of the dungeon with the amulet, it seems like my armor was so strong that no monster could hurt me. Maybe that's your intention though :)

Thanks again.

Hi.

The random number generator is guilty of this pattern.
It's because the level is filled before start, but no check is done when placing player.
Only to make it nicer.
It's pretty hard, each of these features would take at least 8 bytes.
Not my intention but there are no detailed armor/weapons, these only increase and there's no limit to their appearance.

Thanks!

Thanks for the explanations, feel free to close the issue if you prefer :)


Create instance fails

with the latest release, when trying to create instance it fails with message:
2017-12-15T13:35:36.7990583Z OperationId = 00000000-0000-0000-0000-000000000000
2017-12-15T13:35:36.8028992Z Status = Failed
2017-12-15T13:35:37.3705454Z ##[error]Errors encountered : InitialUserPrincipalName cannot be null or empty.
I set the initial user email to the login user but still get the above error.

@andrehe001 If you login to the admin portal using the same user I presume you are able to provision a new instance using the UI?
I would also check if all the main fields for that user are populated in office 365 (i.e. firstname etc...)

It’s working fine via the UI. Can you send me the version where I can switch between 8.x and 9.x? With the actual version I see only version 9.x?
Get Outlook for iOShttps://aka.ms/o0ukef

From: WaelHamze<EMAIL_ADDRESS>Sent: Friday, December 15, 2017 6:59:24 PM
To: WaelHamze/xrm-ci-framework
Cc: Andre Heim; Mention
Subject: Re: [WaelHamze/xrm-ci-framework] Create instance fails (#68)
@andrehe001https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandrehe001&data=02|01||264e5dcc11604925958308d543e59479|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636489575664851021&sdata=hzEbyXbxHG8jdnMImPUkzcu5Mv0mGAG8RfyMHfebXYQ%3D&reserved=0 If you login to the admin portal using the same user I presume you are able to provision a new instance using the UI?
I would also check if all the main fields for that user are populated in office 365 (i.e. firstname etc...)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FWaelHamze%2Fxrm-ci-framework%2Fissues%2F68%23issuecomment-352071352&data=02|01||264e5dcc11604925958308d543e59479|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636489575664851021&sdata=DwAyVaAnfQUcYezSTVmTORvbgUAgg8N9jx9wQLf6BT8%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANH-hzbzLsUQPiORyRrSRcq9UtMLc82vks5tArN8gaJpZM4RDf5Z&data=02|01||264e5dcc11604925958308d543e59479|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636489575664851021&sdata=3r9aWmCx%2BQIZObKSj8xpC7532DIFxSbfpxUK9Y6uLQ0%3D&reserved=0.

This is separate API so there is no difference between 8 and 9. Can you share the parameter supplied to the task? Might be worth running the build/release with system.debug = true and see if there is any helpful info in the log too.

I just tried to re-run one of my existing builds that was working before and it fails now with the same error. There maybe have been a change on the vendor side but I can't see any new API being released. Has this ever worked for you?

@andrehe001 @devkeydet is this working for you? If not I will open a case with MS.

No, still the same problems with creating an instance. Would be great, if you could open up a case and point to your Add-in

Opened the case. Will let you know how it goes.


GUACAMOLE-1876: Display points of interest heatmap in history recording player.

This change adds heatmaps to the in-browser history recording player, one for frames rendered, and one for key events (if the recording has any - otherwise only the frames heatmap will display).

OK @mike-jumper all your comments should be addressed now.


(GH-2281) Add logging guide

This adds a brief guide to logging in Bolt, clarifying that there are
two output methods that are separate, and both configurable. This should
help clarify the difference between --verbose and --log-level for
users.
Note: This is from the hack day, I just forgot to demo + push
Closes #2281
!no-release-note

CLA signed by all contributors.


feat: scheduler (12/): add more scheduler logic

Description of your changes
This PR is part of the PRs that implement the Fleet workload scheduling.
It features more scheduling logic for PickAll type CRPs.
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

[x] Unit tests

Special notes for your reviewer
To control the size of the PR, certain unit tests are not checked in; they will be sent in a separate PR.

A special note:
Currently the API has a limit of 100 for cluster decisions, but it also dictates that we need to keep all decisions from selected clusters. Considering that we do not have a schema-side range set for numOfClusters, do we drop decisions when there are over 100 selected clusters or do we set a limit of 100 to numOfClusters as well?

A special note:
Currently the API has a limit of 100 for cluster decisions, but it also dictates that we need to keep all decisions from selected clusters. Considering that we do not have a schema-side range set for numOfClusters, do we drop decisions when there are over 100 selected clusters or do we set a limit of 100 to numOfClusters as well?

I wonder where is the 100 limit come from? I think k8s scheduler has a parameter about how many nodes to consider. Do we need to add that?

A special note:
Currently the API has a limit of 100 for cluster decisions, but it also dictates that we need to keep all decisions from selected clusters. Considering that we do not have a schema-side range set for numOfClusters, do we drop decisions when there are over 100 selected clusters or do we set a limit of 100 to numOfClusters as well?

we cannot limit the numberOfClusters for the selectAll type and still need to handle the 100 limit size.

A special note:
Currently the API has a limit of 100 for cluster decisions, but it also dictates that we need to keep all decisions from selected clusters. Considering that we do not have a schema-side range set for numOfClusters, do we drop decisions when there are over 100 selected clusters or do we set a limit of 100 to numOfClusters as well?

I wonder where is the 100 limit come from? I think k8s scheduler has a parameter about how many nodes to consider. Do we need to add that?

100 limit comes from https://github.com/Azure/fleet/blob/main/apis/placement/v1beta1/policysnapshot_types.go#L77

A special note:
Currently the API has a limit of 100 for cluster decisions, but it also dictates that we need to keep all decisions from selected clusters. Considering that we do not have a schema-side range set for numOfClusters, do we drop decisions when there are over 100 selected clusters or do we set a limit of 100 to numOfClusters as well?

I wonder where is the 100 limit come from? I think k8s scheduler has a parameter about how many nodes to consider. Do we need to add that?

Hi Ryan! The API has a maxItems = 100 limit on ClusterDecisions, so it may be in conflict with the status updating process if the customer picks over 100 clusters. This does not concern the node threshold much I think; the threshold does not kick unless the cluster has over 100 nodes IIRC. I guess it's kind of safe right now for us to assume that most fleets will not have over 100 member clusters (the design target was 500?)

I will keep the old name for now and see if there's a better name.


How to set route filters on ui.router states?

In Laravel, setting route filters is quite easy. We use a beforeFilter([]) and specify which functions in the controller should be accessible on authentication, and also have except in exceptional scenarios.
Now, I just started using Angular's ui.router and the concept of states is certainly the way forward, but I am asking a asking a noob question here. How do I set route filters on my states. I definitely don't want to do it on individual routes using resolve.
Here's some code. This is what I use for my profile route. I use resolve to make sure it is only accessible when authenticated. But the problem is, my login and signup routes are still accessible when I am logged in. They shouldn't be. It should just redirect to home.
Now I could add resolves to states I don't want accessible when I am logged in, but is that the right way? What if there are many. That would be repeating code. Is there a 'ui.router` way to do this?
.state('profile', {
                url: '/profile',
                templateUrl: 'js/app/partials/profile.html',
                controller: 'ProfileCtrl',
                resolve: {
                    authenticated: function($q, $location, $auth) {
                        var deferred = $q.defer();

                        if (!$auth.isAuthenticated()) {
                            $location.path('/login');
                        } else {
                            deferred.resolve();
                        }

                        return deferred.promise;
                    }
                }
            })

This is a good question for stackoverflow. Here is a page that might help you though (from a quick google search) http://brewhouse.io/blog/2014/12/09/authentication-made-simple-in-single-page-angularjs-applications.html

I did post it there, but the answer was too complicated and didn't make use
of resolve. I think someone with more experience using ui.router would
guide me well.
I am at the noob stage but I make sure I am on the right track :-)
On Mon, Jul 13, 2015 at 8:21 PM, Eddie Monge<EMAIL_ADDRESS>wrote:

This is a good question for stackoverflow. Here is a page that might help
you though (from a quick google search)
http://brewhouse.io/blog/2014/12/09/authentication-made-simple-in-single-page-angularjs-applications.html
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-router/issues/2094#issuecomment-121028679
.

--
Ali Gajani
Founder at Mr. Geek
www.mrgeek.me
www.aligajani.com

Resolve is not a good place to do what you are describing. Try following the tutorial in the link I posted. It should help

Okay I think I know .. The person who helped me on SO pointed out the same.
Resolve does seem a good solution for what then ?
On Monday, July 13, 2015, Eddie Monge<EMAIL_ADDRESS>wrote:

Resolve is not a good place to do what you are describing. Try following
the tutorial in the link I posted. It should help
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-router/issues/2094#issuecomment-121038519
.

--
Ali Gajani
Founder at Mr. Geek
www.mrgeek.me
www.aligajani.com

Resolve if for loading data that the view will need. It is fetched before the view renders so you can show the view when the data is ready

In Satellizer's client example, the author uses Resolve to check if the
user is authenticated, using a promise, and then returns the profile page.
Why is this not useful in state filtering?
On Mon, Jul 13, 2015 at 11:25 PM, Eddie Monge<EMAIL_ADDRESS>wrote:

Resolve if for loading data that the view will need. It is fetched before
the view renders so you can show the view when the data is ready
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-router/issues/2094#issuecomment-121079470
.

--
Ali Gajani
Founder at Mr. Geek
www.mrgeek.me
www.aligajani.com


Change heuristics for showing missing .env error

Pull Request Description

Fix #9479

Important Notes
None
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

Do we use these configurations for unit tests only?

these configurations are used for all builds

these configurations are used for all builds

I meant vitest.config.ts, sorry for confusion.

ah, yeah:

vitest (gui) is for gui unit tests only iirc?
vite (gui) is for the regular app
vite.test (dashboard) is for dashboard unit tests only.
vite (dashboard) is for both unit tests and the regular app

I think, if we have to specify env variables for unit tests - we're doing something wrong, and the problem is not with the env, but with our code (maybe bacause our env variable spread across the app instead of staying located in the single place?)

@MrFlashAccount @PabloBuchu @indiv0 mind reviewing?


Multi-stage cluster addons: Templating issue leads to panic

/kind bug
What steps did you take and what happened:
We get a panic with the current code in values, ok := unmarshalData.(map[string]interface{})["values"].(string), which indicates an internal Go conversion error of interfaces.
What did you expect to happen:
This panic shouldn't happen and we should template it properly

solution is to upgrade to gopkg.in/yaml.v3 from v2


TimeoutOverflowWarning on NodeJS 10.13.0

Please answer the questions the best you can:

What version of bluebird is the issue happening on?
3.5.3

What platform and version? (For example Node.js 0.12 or Google Chrome 32)
Nodejs 10.13.0 on Linux OpenSuse

Did this issue happen with earlier version of bluebird?
Yes, it happened in 3.5.1, 3.5.2, 3.5.3 after nodejs was updated.

This warning started appearing after NodeJS version was updated from 8.9.0 to 10.13.0.
Please note that it does not affect execution of code.
(node:14343) TimeoutOverflowWarning: Infinity does not fit into a 32-bit signed integer. Timeout duration was set to 1.     at new Timeout (internal/timers.js:59:15)     at setInterval (timers.js:503:19)     at fallback (/home/developer/project1/node_modules/async-listener/index.js:619:15)     at /home/developer/project1/node_modules/async-listener/index.js:649:53     at /home/developer/project1/Apps/DevTestApp/DevTestApp.js:230:13     at /home/developer/project1/CorePublic.js:423:41     at tryCatcher (/home/developer/project1/node_modules/bluebird/js/release/util.js:16:23)     at Promise._settlePromiseFromHandler (/home/developer/project1/node_modules/bluebird/js/release/promise.js:512:31)     at Promise._settlePromise (/home/developer/project1/node_modules/bluebird/js/release/promise.js:569:18)     at Promise._settlePromise0 (/home/developer/project1/node_modules/bluebird/js/release/promise.js:614:10)     at Promise._settlePromises (/home/developer/project1/node_modules/bluebird/js/release/promise.js:694:18)     at _drainQueueStep (/home/developer/project1/node_modules/bluebird/js/release/async.js:138:12)     at _drainQueue (/home/developer/project1/node_modules/bluebird/js/release/async.js:131:9)     at Async._drainQueues (/home/developer/project1/node_modules/bluebird/js/release/async.js:147:5)     at Immediate.Async.drainQueues (/home/developer/project1/node_modules/bluebird/js/release/async.js:17:14)     at runCallback (timers.js:705:18)     at tryOnImmediate (timers.js:676:5)     at processImmediate (timers.js:658:5)

That looks like someone did a .delay(Infinity), can we get reproducing code?

Hi, I've double checked my code after reading you comment.
It's 100% my bad as I've added Number.POSITIVE_INFINITY in setTimeout to keep a test process alive.
Thanks.

I have the same Timeoverflow issue please siggest me the solution


Move CI to GitHub Actions

Q
A

Type
improvement

BC Break
no

Summary
This PR moves a number of CI tests to GitHub Actions, reducing the number of builds run on travis-ci.
In this PR:

The performance tests and tests against standalone are moved from travis-ci to GitHub Actions
Tests with coverage have been removed as none of us really bothered to check it. We can bring it back if we find we want to take a look, but until then it's better to reduce the build load
Actions run on pull requests to master and stable branches and all pushes. This is to ensure that we test branches, releases, and it gives contributors the opportunity to have actions run in their forks when they push a branch, allowing them to fix errors before creating a pull request. Note that this mirrors our setup from travis-ci.

This PR will be followed by a separate PR to run tests on sharded clusters and replica sets. These require additional work as they currently use mongo-orchestration to start clusters. I'll see how to best migrate these to GitHub Actions.

@greg0ire Added extension caching, we'll see what that does for performance.

Ok, the cache is warm now, so I restarted the build to see how performance will improve.

I noticed that performance tests were much faster.
Sadly, there are segmentation faults here: https://github.com/doctrine/mongodb-odm/runs/1462657688?check_suite_focus=true#step:6:21

There is an issue with pecl where pecl install ... segfaults at the end of the install process due to replacing the .so file while it's in use (by the CLI process that is started by pecl install): https://github.com/mongodb/mongo-php-driver/issues/1177.
I'll take another look and add another build step that prints information about the installed extension version (php --ri mongodb) so we can confirm that the correct version was installed and is available.

I reported the issue at https://github.com/shivammathur/setup-php/issues/348

Added, and it's showing the correct versions for the time being. I'll expand the build pipeline over time and keep an eye on this.

I just restarted the jobs, the bugfix seems to work great, and the speedup is significant.

Cc @ostrolucky , the setup on DoctrineBundle could be improved, but it would be a harder because as long as we support PHP 7.1, we need 2 PHP installs (one with XDebug, one with PCOV).


Empty Pull request and empty ISSUES Menu, settings for queries done

Hi,

Extension version: v0.79.2024011609 Pre-Release
VSCode Version: 1.85.2 (user setup)
OS: Windows 11
Repository Clone Configuration (single repository/fork of an upstream repository):
Github Product (Github Enterprise version):

I have configured the extension but it just is not working, neither Pullrequests nor issues

I am trying to use it with github enterprise.
Best regards
Eckard.

After some reloads I see now the different queries but no entries.

Try to see if you have the builtin git extension enabled. You can find it by entering @builtin in the search box in extensions.

In the meantime I see also PRs but no issues :-(   and I do have some.

@ecgehrke can you share the output from GitHub Pull Request?

Closing as no further info was provided.


Merge production Web Chat

Now, we can use production Web Chat without modifications in Emulator:

No custom build of Web Chat and Direct Line JS, we are using production version from NPM directly
Chat history preservation/restoration is done thru Emulator Core, not hacking into Web Chat redux store
Chat tabs behind the current tab will no longer unmount, but hidden and kept in DOM until close

Caveats:
After merging this PR, we will show send box and interactivity even we load transcript, this is because the PR for hiding send box is pending approval on Web Chat. Once it's in Web Chat, we can bump version and get that fixed without code change. Will track in another bug.
Some design considerations:

Bot ID and user ID are now fixed to "bot" and "default-user" respectively

Otherwise, we will need to load physical transcript files before instantiating the document editor (need context switching: client -> main -> client)
Web Chat is not reactive to bot ID and user ID change
No one complain about random bot ID and user ID for Chatdown-generated transcript, seems no one really care about bot ID and user ID yet

Tabs that hide behind active tab are now hidden, instead of unmounted

Otherwise, we will need to preserve Web Chat redux store, which means

Preserving history in Web Chat is controversy and we probably need to keep custom Web Chat, instead of production
Emulator redux store will need to keep a copy of another redux store (as of today), seems weird, and Web Chat redux store is not plain JSON too

@compulim  so good to see so much custom code get deleted! Awesome PR 👍


Feature Request: Convert SLC to SICD

I'm looking for something similar to https://sarpy.readthedocs.io/en/latest/utils/convert_to_sicd.html#module-sarpy.utils.convert_to_sicd (but that actually works well).
I'm sorry that I'm unfamiliar with the AIR Centre or what the goals of this package are, but does conversion to SICD fall within the scope of features for this package?

Hi Josh, Rafael Schouten reached out to me about this as well.
So far it is not something we have discussed, but it seems to me that it would be nice to implement it here or in a subpackage as we have the goal of being able to process SLC data from several sensors. We have a developer meeting Wednesday where I will bring it up.

Hi Josh,
I have looked a bit more into the format. It makes good sense to support io for SICD in SARProcessing.jl so it becomes easy to work with.
The format looks quite complex so I don't think that any of the current contributors will work on it soon. And even when we start on SICD we will probably begin by just importing the python or c++ library from the "national geospatial-intelligence agency" to test out the functionality.


Win11 doesn't seem to connect to any internet services or sharing drives

Describe the issue
I got my win11 working seriously well, but then I am facing two issues now. "No internet + shared drive ain't working" and the moment I try to plug in a usb drive, my macOS desktop (UTM containing multitask window) crashes, but the virtual machine doesn't nor do any other window of my macOS.
Configuration

UTM Version: 2.2.4 (36)
OS Version: Big Sur 11.6
Intel or Apple Silicon? M1

Crash log
App doesn't crash, my macOS cursor does for the particular multitask window wherever UTM recides.
Anyone else facing such issues?
Also, I have tried installing and deleting the Win11 for like 6 times, and I faced these issues everytime. The cursor going invisible issue actually did not occur on my 4th and 6th installation.

The cursor problem is unfortunately a known problem, and a fix is not yet available. #2628 (you seem to have discovered this yourself)
Have you tried switching the Network to Bridge Mode #3033 ? This is in the VM config → Network tab.

As far as the "my macOS desktop (UTM containing multitask window) crash" please attach a screen recording. Thank you.

unfortunately, I can't right now. Removed it... will set it up on external ssd soon tho (memory issues, I'm already low on memory and had to install a heavy application for college purpose)
I can describe the cursor issue as, which ever tab(multitask window/desktop#) UTM.app is placed/stays in, the cursor of macOS for that particular tab gets stuck. Non unfunctional unless I switch to another MacOS tab.
This occurs while I run win11.

https://user-images.githubusercontent.com/7568075/159358571-77f4df97-5852-46f7-8963-b1deb52806ac.mp4
I think it's the same error. I can't install it because there's no network

@alexgfaria your issue is likely that the network card in use is not compatible with Windows out of the box. Edit your VM to choose a different network card. (like the first choice, e1000e)

Same issue.

@Curtisspope Please save the debug log according to these instructions. Then make a new comment here and attach the log file.

Same issue over here. Please find attached the requested log with the default network adapter (virtue-net-pci)
WithDefaultNetworkAdapter.log
With the suggested e1000e I have the same issue, please see this log:
Withe1000enetwork.log
Note I have a MacBook Pro M1 16" with a wifi connection. (no network cable attached through dongle)
Hop this helps. Is more data is needed please let me know.

@jvkranenburg Please switch your VM from "Shared" to "Bridged" network. From my experience the "Shared" does not work with Windows guest (but works fine with Linux guest).
If you don't want to use "Bridged" because your host's network device changes frequently, you can also choose "Emulated VLAN" but it might create DNS problems in the Windows guest.

@conath In Bridged mode I have the same issue for both network connectors (virtue-net-pci and e1000e)
BridgeMode.log
BridgedWithe1000eNetwork.log

@jvkranenburg Can you access <IP_ADDRESS> in a web browser in the guest?

Yes I can.

@jvkranenburg I meant to try to access that website inside your Windows VM.

@conath No, because I cannot install Windows 11 as shown in the video from @alexgfaria.

Try pressing Shift + F10 when it wants internet during install and run
OOBE\BYPASSNRO
it will restart and this time you can click I don't have internet

NOT WORKING WITH YOUR COMMAND OOBE....

you put the wrong \

@PhaseAngle  Tnx! That works. After installing Spice network is working as expected!

@PhaseAngle
pressing Shift + F10 does nothing for me :/

@alexgfaria I had to do Shift + fn + F10. Maybe that helps.

@jvkranenburg I successfully skip the internet check during installation win11. But after installing Spice, the network still not working. Which mode (bridged or shared) and emulated network card you choose?

@jvkranenburg I successfully skip the internet check during installation win11. But after installing Spice, the network still not working. Which mode (bridged or shared) and emulated network card you choose?

I used the bridged mode with virtue-net-pci adapter.

@jvkranenburg I successfully skip the internet check during installation win11. But after installing Spice, the network still not working. Which mode (bridged or shared) and emulated network card you choose?

I used the bridged mode with virtue-net-pci adapter.

It work, thank you so much!

Try pressing Shift + F10 when it wants internet during install and run
OOBE\BYPASSNRO
it will restart and this time you can click I don't have internet

Thanks! This should definitely go into the setup tutorial at https://mac.getutm.app/gallery/windows-11-arm

Sem.nome-2-2.mp4
I think it's the same error. I can't install it because there's no network

here is the solution for the wifi problem
https://www.youtube.com/watch?v=jPsrI5fS8zc

Try pressing Shift + F10 when it wants internet during install and run
OOBE\BYPASSNRO
it will restart and this time you can click I don't have internet

I CREATED AN ACCOUNT TO SAY THAT I LOVE YOU THANK YOU SO MUCH

This worked for me: https://www.reddit.com/r/mac/comments/tktihx/comment/i3puy58/?utm_source=share&utm_medium=web2x&context=3

Running UTM 3.1.5
Mac OS 12.3.1
Not able to get UTM to recognize me WiFi adapter. It loads the Ethernet, but I need WiFi. I've installed spicetools, tried Bridge mode, and still not working. Any ideas?

Using a m1 mac on Mac OS 12.4
I have got the windows to open but am completely usure about how to get the wifi working. I have installed spicetools.

Launch normally the windows and mount the spice guest (https://mac.getutm.app/support/)
Press Shift + F10 (FN need be press too, if you see mute icon)
Now you need to have a cmd and you have to write:
D: [press Enter]
spice-guest-tools-0.164.exe  [press enter]

D is the ISO, check with command "dir" to see if spice-guest-tools-0.164.exe is in directory

An install window will show up, complete the installation and reboot, after this, the network need to work.

You can add name server under network then it works.

@mistercorea this completely solved my issue after searching for an entire week. Thank you so much!

@jvkranenburg Please switch your VM from "Shared" to "Bridged" network. From my experience the "Shared" does not work with Windows guest (but works fine with Linux guest).
If you don't want to use "Bridged" because your host's network device changes frequently, you can also choose "Emulated VLAN" but it might create DNS problems in the Windows guest.

Thank you...its work for me

F10

Had issues with both shared / bridged modes, solved the installation using cmd open and bypass command

it works while installing spice guest tool using this method
https://github.com/utmapp/UTM/assets/135052782/80f5f8ab-0139-4d0b-b18f-f349df181b9d

use it this method it will work

use this method it will fix the Network
Make sure u enabled spice tool
then at setup in this page

then D:
AND THEN dir
then spice-guest-tool-0.164.exe
then after restart u are connected

before that do Fn+Shift+F10

Try pressing Shift + F10 when it wants internet during install and run
OOBE\BYPASSNRO
it will restart and this time you can click I don't have internet

Thanks! This should definitely go into the setup tutorial at https://mac.getutm.app/gallery/windows-11-arm

use my method it will work

Hello I am having this problem whereby in the setup page it wants me to connect to WiFi but the networks don’t connect and i cannot click next I am on 2021 MacBook Pro m1

I have gotten into Windows via the I don’t have WiFi but I cannot configure the wifi in settings either does anybody have a solution for this

Emulated VLAN and "virtue-net-pci" adapter worked for me.

I have gotten into Windows via the I don’t have WiFi but I cannot configure the wifi in settings either does anybody have a solution for this

hello i think this video may be help you solve the problem https://www.youtube.com/watch?v=GyvKJjS3m5M
or https://www.youtube.com/watch?v=Ub3gHDBQuAI

@conath In Bridged mode I have the same issue for both network connectors (virtue-net-pci and e1000e)
BridgeMode.log BridgedWithe1000eNetwork.log

Changing my network interface setting to this one solved to issue for me!


Fix test-templates for starters

Fix starters failing in template-tests.sh due to choosing the wrong working directory. Slightly improves the working directory changes to be isolated within a given function to avoid context-sensitivity.

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
No changes detected.

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2760203&view=results should be green if this works.


Hard-coded connection string in DB Context after using scaffolding tool

I may be misunderstanding the intent of the tool: once I run dotnet ef dbcontext scaffold ..., all of my files are generated as expected, but the provided connection string is hard-coded in the dbcontext class itself (with a warning directing the developer to follow best-practices on storing a connection string).
I had planned to do my database schema updates externally (more specifically from a SQL Server database project), so I will need to run this tool whenever there are any schema changes, which means, each time it runs, I'll have to manually modify the generated database context (or script the change).
Is there an option/pattern I'm missing, or I am not following the intended usage of the tool?

I think we could add an option to skip generating the connection string which you could use if you already have it in the right place. Would that help?

Yes, that would be great!

We would have to generate code to wire up the context to use the existing connection string though, so depending on the app the code we'd have to generate could be ConfigurationManager in OnConfiguring, or if it's an ASP.NET core app then the code is probably already there in Startup.cs. Perhaps we need to take another look at whether we can realistically work out what kind of app we are generating into.

I don't know the details around the experience you are looking to provide, but as a developer, I think it's perfectly reasonable for it to be my responsibility to worry about how the connection string is set on the DbContext.

My main point was that if we skipped generating the connection string then you wouldn't have to delete it from code... but you would still be in the position where you have to hand edit the context file to wire back up the connection string (or add the DI-friendly constructor if you are using ASP.NET Core).

@rowanmiller Fair point.  I think having an option to generate the constructor that takes external DbContextOptions<TContext> and no OnConfuguring() method is a better way to go about this.

We decided we aren't going to do anything right now, but opened a more general item to look at better support for the different connection string patterns https://github.com/aspnet/EntityFramework/issues/6746. We think this may be intertwined with our reverse engineer template story, possibly with some built-in templates you can easily chose between.

@rowanmiller am I to understand this discussion as "the connection string will be hard coded for now" ?
Why not add a flag to disable it as @divega  suggested?

Being unfamiliar with the EF Core codebase, I could have probably implemented this better-- but it seems pretty straight forward...   ElanHasson@2d2c0dd
I didn't yet implement any tests, and have not submitted a PR for it.
Based on the above discussion, would it even be accepted?

@ElanHasson suppressing OnConfiguring only gets you part of the way there, as you need to also add the constructor that takes options. We discussed adding this flag (implemented as you did) but decided it was just a stop gap that we would want to remove once we have proper support for doing something better with connection strings (i.e. something that tailors to the specific app model you are using).

@rowanmiller sorry, didn't get an alert for your response.
Scaffolding is a major part of the DB first workflow especially with regard to automation.. currently the only other options I can see are using Roslyn/String manipulation to remove the method and this is not ideal.
Regarding the "something better", is that even slated/planned/identified? I'd be happy to take a stab at it if it isn't.
Perhaps the low-effort, half measure  is what I've implemented in ElanHasson@2d2c0dd .
It is already generating all the classes as partials, so perhaps it's an exercise left to the implementor?
In my scenario, we're injecting DbContextOptions into the ctor in a different partial class, and having a hard-coded connection string in the OnConfiguring method is actually preventing the injected DbContextOptions<> from being used due to configuration priority.

Perhaps we can modify the generated method to include (!optionsBuilder.IsConfigured)... then use the hardcoded connection string.
This should be a great low-effort modification that can satisfy the DI use case without introducing additional overhead, such as documentation, many net-new unit tests, etc.

Perhaps we can modify the generated method to include if (!optionsBuilder.IsConfigured)... then use the hardcoded connection string.

Interesting idea, re-opening to discuss in triage

Great to see this moving along... Can I claim this one?

@rowanmiller and @smitpatel thanks  for taking care of this so quickly.
Glad I was able come up with a simple, fast solution!!


Adding 'Check Palindrome of integers' in Racket issue #36

Adding 'Check Palindrome of integers' in Racket issue #36
guys, pls. do add "hacktoberfest-accepted" label when merging this PR

thanks @ephremdeme
please if you can add the "hacktoberfest-accepted" label
cheers


refactor(semantic): use a simpler way to resolve reference for ReferenceFlag::Type

#3430  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite

Let'll merge once we have some test passing PRs 😅


[Android] Add support for subcomposition and sharing state

Changes

Add support for subcomposition. By adding a new a mode that doesn't register any state updates, the editor can be composed purely for measurement purposes and not trigger a state update.
Add support for sharing state between editors at different call-sites; allow the editor to be constructed at a new call site without side effects. The AndroidView factory and update blocks can be called again by the replacement editor instance safely.
Preserve trailing new lines on restoring state by saving and restoring the internal HTML representation of the editor rather than the message HTML.
Make the state observable so that, when an editor leaves the composition, it doesn't need to clean up its reference from the state and cause unwanted state updates.
Fix some issues with unstable classes in the state.
Update some compose dependencies including compose compiler.

Context
To enable the editor to be embedded in complex layouts that require pre-measurement and/or multiple layout trees.

Part of https://github.com/vector-im/element-meta/issues/1973

Although I don't understand what's the reason for switching off registerStateUpdates in a subcomposition. Without state updates the RTE is pretty much useless isn't it?

In a subcomposition the editor is composed twice, once for measurement and once again to be placed in the layout given the first measurement. During the first measurement I need the editor to just be a dumb view and read the state but not try to update it in any way that might impact the second step. Hope that makes sense?

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (7fa6e63) 87.06% compared to head (36e44f7) 89.88%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #822      +/-   ##
============================================
+ Coverage     87.06%   89.88%   +2.82%
============================================
  Files           155       82      -73
  Lines         17983    14725    -3258
  Branches        901        0     -901
============================================
- Hits          15656    13236    -2420
+ Misses         2068     1489     -579
+ Partials        259        0     -259

Flag
Coverage Δ

uitests
?

uitests-android
?

unittests
89.88% <ø> (+3.36%)
:arrow_up:

unittests-android
?

unittests-ios
?

unittests-rust
89.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 73 files with indirect coverage changes

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

Although I don't understand what's the reason for switching off registerStateUpdates in a subcomposition. Without state updates the RTE is pretty much useless isn't it?

In a subcomposition the editor is composed twice, once for measurement and once again to be placed in the layout given the first measurement. During the first measurement I need the editor to just be a dumb view and read the state but not try to update it in any way that might impact the second step. Hope that makes sense?
An alternative could be to create a new instance of the state for every subcomposition but we'd also have the issue of the first composition requesting focus each frame, which I'm not sure how to avoid.

I still did not understand this completely. Can you point me to a piece of code that shows this? (using the RTE in a subcomp).


Examples have incorrect syntax

https://docs.microsoft.com/en-us/powershell/module/exchange/powershell-v2-module/get-exomailboxfolderpermission?view=exchange-ps#examples
Appears to be older related cmdlet

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

ID: e94700a3-3834-fdb7-2acc-13eb22eec9b5
Version Independent ID: 144b44d0-d97f-8afb-b9e1-c2c42a8d88b4
Content: Get-EXOMailboxFolderPermission (exchange)
Content Source: exchange/exchange-ps/exchange/powershell-v2-module/Get-EXOMailboxFolderPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

@dariomws  Thank you very much for the contribution and updating the documentation with PR. @nickiapalucci Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Geometries and labels

Hi again :)
This is more a reflection/remark than an issue.
When using vectortiles with Mapnik to create map, geometries and labels are paint with a subtle difference:

geometries are paint using the "painter algorithm", which means that every layer is paint over the layer above, so in a way the last layer to be paint has more priority than the ones that came before
on the other hand, labels are paint following the rule "first paint blocks following labels", i.e. the first label paint will prevent other labels to be paint on the same place (there is some configuration that can change this on the edge), so we can say that the first layer to be paint has more priority than the ones that came after

For this reason, when doing "old school" mapping with direct connection to the data (say a PostGIS database), for a given type (say the roads) we usually have one layer for the geometries and one for the labels; just to take an example, that could be "roads" and "roads_labels" layers.
When using vector tiles as sources, as it is now, we have only one layer for both geometries and layers, so this means, for example, that one can only put roads before or after places, while in reality we would like to put the places labels before the roads labels (so a place label has priority over a road label) but we want the place label to be after the roads geometries, so that roads are not paint over the places labels.
I understand this repository is meant for generic purpose only, and may not go in great details, but I feel like this one hurts even when doing very basic cartography.
So it may that the answer would be to have separate layers again for geometries and labels. Not sure how this would impact performance, though.
Now, I may just have missed something, or I may not be considering the issue from the right point of view.
Yohan

@yohanboniface thanks for raising this. The way I've imagined handling this is to have the client do two  different symbolization/render passes on the layer, once for the actual geometries, and once for the labels. It seems that the underlying data is only needed once, but that different rendering rules should be applied to them at different times. Of course, I'm unsure how compatible that is with current renderers, but this repo is also mean to focus more on forward-looking use cases for vector tiles and is itself a bit experimental.) Does this make sense?

Yep, it makes sense, sure. :)
But in practice this prevent from using this repo as source for mapnik based rendering. AFAIK, it's not possible, at this stage at least, to have twice a given layer in the mapnik stylesheet (I may be wrong). For sure it's not possible within a CartoCSS toolchain.
So clearly, for performance, both server side and bandwidth side, not duplicating info would be much better. But in the meantime, my opinion is that Mapnik is the minimum target for a vector source, so unless there is a smarter option, I would go on splitting geometries and labels layers when it makes sense.
cc-ing @springmeyer that may have good lights on this.

I admit it's been years since I mucked with Mapnik XML files, I'm all WebGL these days :)  We should look further into the options, perhaps it's possible to duplicate the data source in Mapnik and just point it to the same source twice (the datasource element is replicated for each layer if I recall). Not sure about Carto...

Separate layers for labeling are not a hard restriction in mapnik/cartocss/mapbox-studio, but rather a performance optimization common to a lot of sample projects and vtile sources.
For example, for fastest rendering and most reliable label placement it is ideal to use pre-calculated centroids for labeling polygons. Or for labeling lines you'll commonly want to filter out a reduced selection of lines that make sense to label in order to avoid clutter. Finally once you've decided that a custom layer for labels makes sense it then also makes sense to drop all attributes on the (non-label) layer containing all the geometries to save space. Doing all this at vtile production can produce better and faster rendering results with very little extra payload. Or in some cases your vtiles might even be smaller because attributes take up a lot of space. So, this is the reason you see a variety of custom label layers available in the vtile sources for Mapbox.
But again this is not a hard restriction at all. And generic vtile sources that try to be "one size fits all" are valid, quicker to produce, and can still be used with renderers like Mapnik. For example @yohanboniface if you want to use one linear layer to render both lines and labels you can simply render them in two passes using CartoCSS attachments (https://github.com/mapbox/mapbox-studio/blob/mb-pages/docs/0000-01-01-symbol-drawing-order.md#attachment-ordering). And if you want to render many different layers, many with labels, and maintain all the labels on top you can also do that. It's just more complicated that a single CartoCSS attachment. If using Studio you can use the {id}.{class} syntax to refer to the same layer twice in whatever order you need: https://github.com/mapbox/mapbox-studio/blob/mb-pages/docs/0000-01-01-advanced-features.md#custom-layer-order. If not using Studio then it may be even easier - you just need to form up a Mapnik XML that refers to the same vtile layer twice by name.

Thanks @springmeyer! All makes sense, I imagine we might add label layers at some point for all the reasons you mention, but I had always been assuming they could be rendered from one generic source like this in the meantime.

Thanks @springmeyer :)
About attachment, my experience was that they allow to draw twice a same geometry, but the order they are drawn is still the order of the data layers, not the order they are defined in the stylesheets. What I mean is that an attachment is still drawn with the layer it belongs to. Am I wrong?
I was not aware about the syntax trick to have a given layer listed more than once in the project.yml file, and that really nice! As you may guess, I'm using Kosmtik, which doesn't know about that trick, but I know the guy who maintains it, so I may convince him to support this nice feature :)
Thanks again!

It works, thanks a lot, Dane :)
So you and @bcamper deserve a Thank You Map :)

Amazing :)
Closing this and will open an issue to consider label layers in the future.


bugfix filesystem with google storage

1. Why is this change necessary?
The class Shopware\Core\Framework\Adapter\Filesystem\Plugin\CopyBatch can't handle multiple target files when using Google Storage Adapter. GuzzleHttp\Psr7\Stream closes the resource in the __destruct() function after the resource has been uploaded. The $handle is then a closed resource for the second $targetFile and League\Flysystem\Filesystem throws the InvalidArgumentException.
2. What does this change do, exactly?
The change checks if $handle is still a valid resource in each iteration and opens the source file again if necessary.
3. Describe each step to reproduce the issue or behaviour.

Use Google Storage as filesystem adapter.
Run bin/console theme:compile

4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @pawelnapierala could you please sign the cla?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-11758
Please use this issue to track the state of your pull request.

Hello @pawelnapierala , we're closing this pull request due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline.

Hi, I've created a ticket for this bug so we can have a look into this more thoroughly: https://issues.shopware.com/issues/NEXT-14202

I would probably put the if (!is_resource($handle)) above like this:
if (!is_resource($handle)) {
  if (is_resource($batchInput->getSourceFile())) {
    $handle = $batchInput->getSourceFile();
  } else {
    $handle = fopen($batchInput->getSourceFile(), 'rb');
  }
}

Otherwise the $handle = $batchInput->getSourceFile(); part will run for every target file even if the adapter does not close the resource.
But more importantly. Why is this PR declined???

Hey @dl-mbegerau, the PR was declined because it was incomplete.
We do close incomplete PRs after 2 Weeks of no reaction to prevent orphaned PRs, as stated in our contribution guidelines.
As written above the PR can be reopened, or a new one can be created at any time for this.


Update GyroE optimism factory contract to proxy contract address

Description
Update GyroE optimism factory contract to proxy contract address

I'm closing in favor of #484


WebCamTexture

Is there a way to get this to work with the webcam as input source?

Yes. You can use the Source Texture property instead of the Source Video property.