Add just-in-time tips to discover keyboard shortcuts and other techniques in context

VS Code has several keybindings to speed up or enhance workflow that users are unaware of.
Since it's not practical to memorize all keybindings upfront.  I believe users will benefit from just-in-time suggestions based off what they're doing in context.  (e.g. if a user is inserting the same characters repeatedly on multiple lines, we could suggest the keybinding to create multi-cursors.)
This could help users learn relevant keybindings while they work.  Perhaps this could be in the status bar or in a toast-like notification.  The key is to make it non-obtrusive and immediately relevant and thoughtful to the user's present needs.  There should be a global setting to disable it, if users don't want it.  We should also be very careful to avoid spammy suggestions that are repetitive or irrelevant.
@waderyan also had a similar idea before this about introducing tips into the product.  I propose we extend his idea to be in context.

This will also help solve #4100

pinging @stevencl for feedback.


Correct reporting of CAPEX / OPEX

n.statistics.capex and .opex do not actually report CAPEX and OPEX but fixed and marginal costs. Here we compute the Capex and Opex from the overnight_cost, lifetime, discount rate and fixed costs in the post networks. In the mid-term this should all be directly available from the PyPSA Network
Before asking for a review for this PR make sure to complete the following checklist:

[ ] Workflow with target rule ariadne_all completes without errors
[ ] The logic of export_ariadne_variables has been adapted to the changes
[ ] One or several figures that validate the changes in the PR have been posted as a comment
[ ] A brief description of the changes has been added to Changelog.md
[ ] The latest main has been merged into the PR
[ ] The config has a new prefix of the format YYYYMMDDdescriptive_title

[ ] At the moment grid CAPEX is only for assets built after 2020. How to harmonize this with the other sectors, which consider assets build before 2020 as well??

CAPEX:
understood as annualized investments
WACC is 7%,  but for decentral assest like solar rooftop it is 4%
For transmission infrastructure we only consider the annuities of assets built between 2020-2045, because older infrastructure likely is already amortized
For other infrastructure we would be able to consider all annuities. However for consistency we follow the way it is done for grid infrastructure
known issues:
[ ] decentral biomass boiler have WACC 7% which is inconsistent
[ ] Some assets built before 2020 likely have not yet repaid their annuities -> CAPEX may be underestimated
[ ] There might be mismatches of the lifetime recorded in the network component and the one in the cost-data, affecting both the annuity and the FOM computation
OPEX:
FOM is computed as the difference of CAPEX and aggregated fixed costs, then added onto...
n.statistics.opex, which aggregates marginal cost
energy costs should be reported separately, see below
the opex of fossil generators in the model would correspond to fuel costs. since these generators are not covered by the exporter script, this is not an issue for us
differing from CAPEX, the FOM of existing assets before 2020 is included in the computation because they still need to be maintained
known issues:
[ ] Offwind grid connection has FOM == 0%
[ ] Some components remain in the network, even though they are no longer used (e.g. coal) -> FOM may be overestimated
Energy costs:
not reported at the moment
imported e-fuels and H2 should be included


Add bust rendering

Closes #47
Also, I bumped the cache bust because I changed the numbering for the RequestKind enum.

Add the bust to the examples

Add the bust to the examples

Added


Organize Imports: Imports issue with class declared outside the package block (in same file)

Hi Josh,
I just discovered a small issue related to classes declared outside the package block (in the same file). The code action proposes to fix the missing import but the organize imports feature then cleans it.
This gif demonstrates the issue:

Best,
Aurélien

Wow, I never knew that if you need to reference the class in the package block from outside of the block in the same file, that you are required to import it (unless it's in the top-level package, of course). I confirm that all AS3 compilers require this. Crazy how this hasn't come up before now!

Thanks Josh for the fix.
Yes, this was a discovery for me as well. I think this was the first time I had to do a such thing 😅


Support nacos registry.

Motivation:
Add nacos registry support.
Modification:
Nacos current version is 0.2.1, production version is 0.8.0, so this support is preview version.
Result:
Fixes #245

Codecov Report

Merging #327 into master will decrease coverage by 0.04%.
The diff coverage is 82.53%.

@@             Coverage Diff              @@
##             master     #327      +/-   ##
============================================
- Coverage     69.58%   69.53%   -0.05%
+ Complexity     1084     1082       -2
============================================
  Files           364      363       -1
  Lines         15655    15656       +1
  Branches       2538     2539       +1
============================================
- Hits          10893    10887       -6
- Misses         3460     3462       +2
- Partials       1302     1307       +5

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/alipay/sofa/rpc/common/RpcConstants.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../sofa/rpc/registry/zk/ZookeeperConfigObserver.java
18.03% <100%> (ø)
0 <0> (ø)
:arrow_down:

...ofa/rpc/registry/zk/ZookeeperOverrideObserver.java
40.98% <54.54%> (-1.21%)
0 <0> (ø)

...ofa/rpc/registry/zk/ZookeeperProviderObserver.java
75.51% <68.75%> (-2.76%)
0 <0> (ø)

.../alipay/sofa/rpc/registry/utils/RegistryUtils.java
93.61% <97.05%> (+0.67%)
0 <0> (ø)
:arrow_down:

.../rpc/transport/rest/SofaResteasyClientBuilder.java
88.88% <0%> (-3.71%)
0% <0%> (ø)

...pay/sofa/rpc/message/bolt/BoltInvokerCallback.java
83.63% <0%> (-1.82%)
7% <0%> (-1%)

...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java
83.92% <0%> (-1.79%)
0% <0%> (ø)

.../sofa/rpc/bootstrap/DefaultClientProxyInvoker.java
87.09% <0%> (-1.62%)
0% <0%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

@ujjboy Nacos module already add to all/pom.xml and get all test cases done.

文档有编写吗？

文档有编写吗？ @JervyShi

文档我加一下

https://github.com/alipay/sofastack-doc/pull/73 先合并这个，然后再改文档。@JervyShi


customEmails property syntax

I believe this:
"customEmails": "[variables('customEmails')]"
Should be this instead:
"customEmails": ["[variables('customEmails')]"]

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

ID: c3b9fa00-b319-daa4-3bd5-1f83f96590d4
Version Independent ID: 69d25f7e-30e4-0c6a-1d67-25e282d71a3c
Content: Create a classic metric alert in Azure with a Resource Manager template
Content Source: articles/monitoring-and-diagnostics/monitoring-enable-alerts-using-template.md
Service: azure-monitor
GitHub Login: @johnkemnetz
Microsoft Alias: johnkem

@pokorourk Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

These alerts have been retired and no one should be using them anymore.
#please-close


Deprecate Corpora

Remove corpora.cziscience.com domain

[ ] https://corpora.cziscience.com is removed (only PROD)
[ ] https://api.corpora.cziscience.com is removed (only PROD)

Turn off legacy landing page

[ ] @mckinsel notifies the author of the affected paper about the URL change

[x] Update https://chanzuckerberg.github.io/cellxgene/ to include the link to cellxgene.cziscience.com

[x] Return a 404 for https://chanzuckerberg.github.io/cellxgene/posts/cellxgene_cziscience_com

[x] Retain a copy of the HTML for https://chanzuckerberg.github.io/cellxgene/posts/cellxgene_cziscience_com around just in case

Update (8/4/2020):
Requirement:

[x] Update the related repos's README with the following message to avoid confusion:

UPDATE (X/X/2020): Starting today, the name Corpora will only be used as the internal project name, with cellxgene Data Portal being the official product name

============= Archive - no need to the the following for M1
Given the decision to rebrand corpora to cellxgene, we will need to update our internals to ensure that the branding matches. This work includes:

[ ] Updating Route53 with cellxgene.cziscience.com and deleting corpora.cziscience.com (dev and staging).
[ ] Updating databases name with cellxgene.
[ ] Update variable names in code that reference Corpora.
[ ] Sort out S3 buckets names and any conflicts with existing cellxgene-prefixed buckets.
[ ] Work with infra to rename roles related to Lambda functions used by Corpora.
[ ] Update API Gateway to reflect name change
[ ] Update IAM policies that reference Corpora
[ ] Update secret names
[ ] Update any references to Corpora in single-cell-infra repo
[ ] Rename Github repo
[ ] Update public-facing documentation that references Corpora

Could we just change all of the outward facing documentation to say cellxgene and keep the internals as corpora. Corpora will be the internal name we use to distinguish cellxgene the tool from cellxgene the data browser. It will save us a ton of refactoring work, resolving name conflicts, migrating buckets, and makes it easier to communicate what part of the project is being modified.

I can see why having Corpora namespace internally could cause confusion, since now publicly everything is Cellxgene. But I think I'm leaning towards @Bento007 's idea in terms of renaming, because in order to distinguish between cellxgene the tool and the data browser, the data browser will still need to have a different name than cellxgene.
So data browser's internal project name could be either just data portal or dp for short, to match our external product name, as seen in API URL api.cellxgene.cziscience.com/dp/*, or just keep it as corpora
Personally I feel that as long as we update public facing name to be Data Portal (DP) and on https://github.com/chanzuckerberg/corpora-data-portal's README to say:

UPDATE (8/1/2020): Starting today, the name Corpora will only be used as the internal project name, with Cellxgene Data Portal being the official product name

It should clear all the potential confusions?
Thank you!

Also I don't think we have a name for the "integration data corpus" if we drop Corpora. Ambrose suggested just calling it "remix".

Per September 24 Sprint Planning, @maniarathi and @mckinsel agreed that we should merge the three remaining issues related to corpora deprecation into one issue and assign an estimate of 1 (rather than three tickets each estimated at 1).

@signechambers1 can you verify the https://chanzuckerberg.github.io/cellxgene/ part? Thank you!


Change dependency version of org.wso2.stratos.appserver.dashboard.ui to 6.0.0-SNAPSHOT

Change dependency version of org.wso2.stratos.appserver.dashboard.ui to 6.0.0-SNAPSHOT

While building the product-as I'm getting following error
Cannot complete the install because one or more required items could not be found.
Software being installed: Key Store Management Feature 4.3.4.SNAPSHOT (org.wso2.carbon.security.mgt.feature.group 4.3.4.SNAPSHOT)
Missing requirement: Key Store Management Feature 4.3.4.SNAPSHOT (org.wso2.carbon.security.mgt.feature.group 4.3.4.SNAPSHOT) requires 'org.wso2.carbon.core.feature.group [4.3.0,4.4.0)' but it could not be found


Add support for subsample ratio 4:1:0 and missing 4:1:1

I was getting panics with an image with a subsample ratio that was not supported, this adds support for the two additional subsample ratios that are supported by Go.

Thanks for the patch !
Can you also add one commit & remove go versions <= 1.4 from travis ?

done, thanks Benoît


What's the place CNN exactly?

Hi, I couldn't find what pre-trained network you used for the place CNN from the paper or the website. Where does it come from?

Search for "Pretrained Models" on the demo website (https://projects.csail.mit.edu/soundnet/).


Upgrade e3nn to fix batch normalization runtime issue with final_conv module

Starting with version 0.5.1 of e3nn, the authors of the library are now guarding against attempting to aggregate scalar means if no scalar irreps are given as input: https://github.com/e3nn/e3nn/blob/bb71b3c75c38c6d633cfebe9988b1fb1563432c0/e3nn/nn/_batchnorm.py#L191
(This PR also adds wandb to the installation instructions for new Conda environments)
It's possible version 0.5.0 of e3nn was pinned accidentally, even though the source code will only run with 0.5.1 of e3nn. I hope this helps!

@amorehead Have you encountered this e3nn bug while running DiffDock?

Hello. I haven't tested to see if DiffDock-L's codebase now raises this error, but for DiffDock 1 it did.


Error: timeout reached while waiting for sockets response

<EMAIL_ADDRESS><EMAIL_ADDRESS>Calls made to the adapter methods as follows:
`
const sockets = await io.of('/').adapter.sockets();
console.log(sockets); // a Set containing all the connected socket ids
// this method is also exposed by the Server instance
const sockets = io.in('room3').allSockets();
console.log(sockets); // a Set containing the socket ids in 'room3'
`
Causes the below exception being thrown:
(node:20818) UnhandledPromiseRejectionWarning: Error: timeout reached while waiting for sockets response     at Timeout._onTimeout (/var/app/node_modules/socket.io-redis/dist/index.js:286:28)     at listOnTimeout (internal/timers.js:549:17)     at processTimers (internal/timers.js:492:7) (node:20818) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag--unhandled-rejections=strict(see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 4)

same error

Getting this. Any clue for fixes?

I am having the same issue and I'm at a loss on how to stop it.
Error: timeout reached while waiting for sockets response
at Timeout._onTimeout<EMAIL_ADDRESS>

getting this error randomly

Same error using ioredis

Happening to me too.
Using ioredis for connection and fetchSockets instead of allSockets due to the deprecation of allSockets sice<EMAIL_ADDRESS>but the result is the same.
Error: timeout reached while waiting for fetchSockets response
    at Timeout._onTimeout (/node_modules/@socket.io/redis-adapter/dist/index.js:559:28)
    at listOnTimeout (node:internal/timers:569:17)
    at process.processTimers (node:internal/timers:512:7)

Same problem when the number of clients is large.

Same error here. Looking at the lib seems like the error is generated on this line: https://github.com/socketio/socket.io-redis-adapter/blob/ef5f0da0b4928fd422afc985aec0e233d34400c0/lib/index.ts#L759
@darrachequesne Can you help us? What is the logic of this if that generates the error?
    if (this.requests.has(requestId)) {
      reject(
        new Error("timeout reached while waiting for fetchSockets response")
      );
      this.requests.delete(requestId);
    }

I am seeing this issue as well intermittently

seems to happen when numSub and msgCount are off by 1
{   type: 5,   numSub: 3,   resolve: [Function (anonymous)],   timeout: Timeout {     _idleTimeout: 5000,     _idlePrev: [TimersList],     _idleNext: [Timeout],     _idleStart: 5650,     _onTimeout: [Function (anonymous)],     _timerArgs: undefined,     _repeat: null,     _destroyed: false,     [Symbol(refed)]: true,     [Symbol(kHasPrimitive)]: false,     [Symbol(asyncId)]: 391,     [Symbol(triggerId)]: 0   },   msgCount: 2,   sockets: [] }

me too

allSockets

allSockets works for me, my app deployed on k8, on single instance it was fine, fetchsocket timeout on multiple instances

anyone found a solution??

The timeout might happen if the requesting server does not receive enough responses from the other servers in the cluster.
Catching the exception and retrying upon failure should account for those temporary issues:
async function fetchSockets() {
  for (let i = 0; i < MAX_RETRIES; i++) {
    try {
      return await io.in("room1").fetchSockets();
    } catch (e) {
      // timeout error
    }
  }
  throw "too many retries";
}


[WIP] Style the bookmark/subscribe button

Looks nicer! Made some changes to SubscribeIcon component based on BookmarkIcon, dunno if I broke anything.

Is good to go now! Club page will be slightly restructured and background changed to snow

Is this design the most intuitive? not sure it totally communicates what it wants but it's def better than it was

aight is gtg methinks

I've pushed changes that allow this PR to also cover the mobile club rework, currently the page looks like this on mobile after the changes:

@valenciafu is it okay if the buttons are ordered this way instead of the way they were ordered in the mockup, it's a lot easier to code this way.

@sigmachirality yep, go for it!


Non-interactive quickstart

Is it possible to use quickstart on an existing cluster without having to use the prompt? e.g.
% kn quickstart kind --name local
Running Knative Quickstart using Kind
✅ Checking dependencies...
    Kind version is: 0.15.0

Knative Cluster kind-local already installed.
Delete and recreate [y/N]:

Ideally, we could run the entire command non-interactively like: kn quickstart kind --name local --no-tty or a -y or -n flag. Your thoughts?

Quickstart wasn't intended to be an installer, so I don't think that's something we'd do. Instead, you might want to look into the operator plugin or scripting the yaml install like @csantanapr did with KonK.


Don't transform class methods

Related to #10.
function createClass() {
  var C = class {
    constructor(a,b) {
      this.a = a;
      this.b = b;
    }

    f() {
      return () => 'yo';
    }
  };

  return C;
}

to
'use strict';

var _createClass = function () { function defineProperties(target, props) { for (var i = 0; i < props.length; i++) { var descriptor = props[i]; descriptor.enumerable = descriptor.enumerable || false; descriptor.configurable = true; if ("value" in descriptor) descriptor.writable = true; Object.defineProperty(target, descriptor.key, descriptor); } } return function (Constructor, protoProps, staticProps) { if (protoProps) defineProperties(Constructor.prototype, protoProps); if (staticProps) defineProperties(Constructor, staticProps); return Constructor; }; }();

function _classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError("Cannot call a class as a function"); } }

function _ref2() {
  return 'yo';
}

function _ref() {
  return _ref2;
}

function createClass() {
  var C = function () {
    function C(a, b) {
      _classCallCheck(this, C);

      this.a = a;
      this.b = b;
    }

    _createClass(C, [{
      key: 'f',
      value: _ref
    }]);

    return C;
  }();

  return C;
}

@jhen0409 , I run your test with, and without your changes
In both cases, I got same transformedWithPlugin.code, and equal your code above
Please, check transformedWithPlugin.code at your local repo, after reverting your changes
If has some difference with fixed output - send here your npm-shrinkwrap.json - maybe different versions of babel&plugins?
If has no difference - try write another test, or maybe not really need this fix?

@Gvozd the babel-preset-es2015-node5 can be reproduce this problem, originally I thought it would in the babel-preset-es2015. Sorry, I'll add test case for create-class with es2015-node5.

@jhen0409 , your right, and I'm wrong
Maybe need add custom plugins for different test-cases, but I has no time for this
I've merge, and added comment to test-case
see 1.0.3 on npm
Thanks for your work


Introduce Continuous Integration

Continuous Integration pipeline. Right now it's executed on PHP 7.3 and there are two builds - one with latest requires and one with lowest requires, lowest is required to make sure things work on minimum requirements.
Resolves #9 #11

@piotrooo They can be squashed on merge, guthub allows to do that.

You are responsible for your commits. You exactly know what you did. So who can better squash your commits than you? :)

GitHub can do it as good as I can (screenshot from other project):

ping @NoelDavies lets merge PRs :)


Fix crashing on empty regexps

Show rules with empty strings, regexps and regexp-s that match empty strings now panic instead of crashing.
Fixes #1790
(note that is doesn't fully fix initial issue as it has \b things, so initial #1790 is mostly like more general #229, but without them it is fixed)

Thanks!


Dataset browser

It would look complete if there is a way to look at the images in the Dataset created. Right now it is difficult to peep in to lmdb and see how the images looks like (especially if we choose to not store the original image to save space)

See #21.


Leak when using Metal delegate

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
2.7.3
Custom Code
Yes
OS Platform and Distribution
iOS 15.5
Mobile device
iPad
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Whenever we enable Metal delegate, a fast memory leak is observed. Using just CoreML delegate, and the same exact app/code, the memory use is steady.

Most of the leaked memory looks like

_objc_rootAllocWithZone
-[MTLDebugCommandBuffer computeCommandEncoder]
0x102775ec8
0x11379ec3c
tflite::Subgraph::Invoke()
tflite::Interpreter::Invoke()

The app is a Swift app, that calls into native C/C++ code. Native C++ threads then run inference calling into TFLite C API.
Any insight into what can be tried is appreciated!

### Standalone code to reproduce the issue

```shell
n/a

Relevant log output
No response

Hi @w3sip ! Is this issue replicating in the 2.8/2.9 version too? Could you provide a minimal standalone code to reproduce this issue. Thank you!

@mohantym - can't try 2.9 because of https://github.com/tensorflow/tensorflow/issues/56250
Considering we're on iOS, providing a standalone sample may end up being a project. Is there a sample/demo project for IOS that could be used for that?

I've been able to test with 2.9.1, definitely leaks there as well.
The growth is very linear, and very rapid -- after ~22,000 inferences, the memory is up close to 1.5GB.

Could you please check the issue here https://github.com/tensorflow/tensorflow/issues/56239 and let us know if this is relevant to your issue. Thanks!

@sachinprasadhs - most definitely not; we only create the model once upon initialization. The leak seems to originate in TfLiteInterpreterInvoke.
To remind you, the inferences run in native C++ code/threads, but in context of iOS app. Is that something that is covered by TFLite tests?

any update on this issue? for now we wrap the invoke by @autoreleasepool as work around

Any updates? I can't really use auto release pool as everything is written in C++

Hi, sorry for the delay. Will take a look asap.

Hi @yishuangP any updates on this?

Hi, just to clarify, are you using the MetalDelegate.swift to initialize the metal delegate?

@yishuangP , yes, correct!

@yishuangP I got same issue as explained on this ticket https://github.com/tensorflow/tensorflow/issues/57717 but still not find the right way to fix it!

Does wrapping TfLiteInterpreterInvoke with autoreleasepool solve the issue? I can add it before we figure out the root cause. Thanks!

Hi, I just tried the pose estimation example with our latest nightly version 0.0.1-nightly.20230405. I don't see this memory leak issue. Could you try our latest version? Thanks!


[CT-1667] [Bug] Vulnerability on cryptography dependency

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
As of yesterday, Snyk has reported on a vulnerability for the cryptography package here: https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3172287
However dbt-snowflake is locking the package:
cryptography>=3.2,<37.0.0"

Would it be possible to update this package in dbt-snowflake?
Expected Behavior
I expect the package to be updated as to not expose this vulnerability issue.
Steps To Reproduce
Using the snyk-cli.
$ snyk test

Relevant log output
Issues with no direct upgrade or patch:
  ✗ Denial of Service (DoS) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3172287] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>><EMAIL_ADDRESS>and 2 other path(s)
  No upgrade or patch available

Environment
- OS: macOS Ventura 13.0.1
- Python: 3.0
- dbt-core: 1.3.1
- dbt-snowflake: 1.3.0

Additional Context
No response

Thanks for opening @Surbias !
I just opened https://github.com/dbt-labs/dbt-snowflake/pull/360 as a result. I don't think cryptography has published a new version yet that includes this fix, so this won't be completely resolved until then.
Thank you again for letting us know 🙌


The paper says it also need a number of labeled SQL to train?

Thank you very much.

Am I right?

Hi, the model doesn't need labeled SQL to train. It only uses question and answer pairs as weak supervision. Please see Section 4 for more details.

Thank you！

What does load_programs in experiment.py do?
@crazydonkey200 Thank you!

It refers to programs found from random search, see section 5.1 training details.


[Fix: bag]: add try catch to bag test

Description
As reported in this issue the bag test is flaky and can timeout while executing.
This is due to some Zenoh internal over which we have no control.  To solve this we have few options:

Remove the test
Change all the ipc code to abstract zenoh behind an interface
Wrap the unit test in a try/catch so that we avoid the failure (but don't really solve the problem).

The first point is not an option. The second point is also not an option as we decided as part of the design that we were going to keep the interface simple at the cost of full testing capabilities.

your solution is as good as removing the test…can we increase the timout for the service call?


Add dataset description table to README

Closes #191.

Do not merge yet, I want to wait till this pull request 230 is done so that I might add a table for this attack as well.

Heyhey, thanks for the great work @marisanest. I allowed myself to improve a little upon it, as the issue stated something related to "purpose", in addition to the data set information. So I tried to incorporate that. Let me know what you think. Since the table for the first two attacks is identical, I once again thought about moving the 3 tables to the introduction of the MIA attacks, but I couldn't come up with an elegant merge of the tables, so I left them there. I think its fine if they are split like that.

can we merge this?

Yes, I just added a table for MembershipInferenceAttackOnPointBasis after Jakob now changed the README accordingly. Can you please check it again @Friedrich-Mueller? And I also changed the description a tiny bit, so @blauertee, could you also have a look? If everything is fine we can merge.

Yes, I just added a table for MembershipInferenceAttackOnPointBasis after Jakob now changed the README accordingly. Can you please check it again @Friedrich-Mueller? And I also changed the description a tiny bit, so @blauertee, could you also have a look? If everything is fine we can merge.

The table LGTM

Ok, I checked the implementation again, and it seems that indeed both datasets are needed, although I can't say clearly why. So I will keep the changes I added in the README and merge them for now.


Fix broken chat message footer styling

@soey I made a comment on your PR introducing the conversations feature that your changes broke the styling of the chat message footer and it's actions. I proposed there the change that fixes it, but I think you missed implementing it because I ran into the same broken styling while reviewing another PR.
See https://github.com/tactilenews/100eyes/pull/1849#discussion_r1627566780
This PR introduces the proposed fix.

Mh.. i think I touched this back then .. but nevertheless thanks for fixing. CSS ist not my best friend yet.


[BUG] ShareManager pod doesn't restart on terminating the node when both sharemanager and workload pod is on the same node

Describe the bug
ShareManager pod doesn't restart on terminating the node when both ShareManager and workload pod is on the same node.
To Reproduce
Steps to reproduce the behavior:

Deploy Longhorn v1.1.0-rc1 on a K3s cluster of 3 nodes.
Create a StatefulSet (1 pod) with RWX volume. Make sure both the shareManager pod and the workload is attached to the same node.
Power down the attached node.
The ShareManager doesn't get restarted and the workload pod stuck in terminating state.

Logs:
longhorn-support-bundle_432fcb24-4f12-4d8c-a037-f3caf95d528a_2020-12-09T07-08-14Z.zip
Time of error : ~ 2020-12-09 6:55:00 UTC
Volume name - pvc-852ea68c-340a-4fc1-a0d0-75e67508f4ac
Workload name - wk-1
Environment:

Longhorn version: v1.1.0-rc1
Kubernetes version: v1.19.4+k3s2
Node config

OS type and version: ubuntu 18.04
CPU per node: 2 vcpu
Memory per node: 4 GB
Disk type: EBS

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): AWS

This could be because of the setting Pod Deletion Policy When Node is Down. The setting Pod Deletion Policy When Node is Down was do-nothing in this cluster, so the workload stuck in terminating state.

From the logs it looks like this is not a bug, there was no longer an active workload.
Which meant that the kubernetes_pv_controller set the lastPodRef, here:
https://github.com/longhorn/longhorn-manager/blob/master/controller/kubernetes_pv_controller.go#L490
This means that there is no need for the share manager any longer and the share manger went into the stopped state and detached the volume. Can be seen in the logs shareState=stopped, @khushboo-rancher can you file an issue to include the sharemanager resources during support bundle generation?

@khushboo-rancher once you scale the workload up or create a new workload, new deployment refering the same pvc, the share manager should be coming up again, correct?

From the pods.yaml the sharemanager pod: pvc-852ea68c-340a-4fc1-a0d0-75e67508f4ac also no longer shows up so it correctly got force deleted as well.

@joshimoo Yes, on scaling up the workload, the ShareManager came up. I think, we can close this.

Closing this. Not a bug.


[Other]: Add example of tract extraction to documentation

Summary
Updates to documentation for new analysis workflow
Description
Add a brief example to the docs to demonstrate how the tract extraction workflow can be used.

Done with #42


Missing releases

In npm is published version 0.1.14, but here in GH is as latest version 0.1.19. So few builds are missing in npm

ok, closing


Add command "deta deploy --force"

Add a command that will ignore .deta/state and load all files, while writing new hashes to .deta/state. For example deta deploy -f/--force.

good idea! cc @rohanshiva @aavshr


Help with our Harvester setup

Hi,
We are setting up a 1 node cluster running harvester as bare metal, then rancher as a management VM to handle a kubernetes cluster (which will be VMs on said host)
Our setup will be

Single server with Harvester installed on Bare Metal
Dual 25Gbe Network linked in an LACP bond (as configured on harvester installation)
10TB of NVMe storage on a hardware raid 5, Volume is visible in the node view.

We plan to run Rancher on a Suse MicroOS virtual machine (hosted on the above harvester node) then configure rancher to point back to the harvester node via integration and have it deploy a kubernetes cluster for us (6x etcd, 6 worker).
Questions i have are

Does our above setup look OK (we understand a single node is not ideal, but this is research only and a rather high end node with plenty of inbuilt resiliency)

When we create a VM network, It only allows us to attach it to the "mgmt" network - is that normal practice?

The inbuilt storage is seen (this is not where harvester is installed tho, harvester is on another dual ssd, raid 1 vol)

Is this where VMs will naturally sit?

Can we connect our DELL Powerstore SAN to this host (we have it wired via fibre channel)

Is there a list of website URLs that Rancher and Harvester ideally need access to in order to do updates, pull drives etc? I need to white list in our proxy

We are trying to connect the BMC out of band but get the following (we tested the IP and credentials work, the server supports redfish)

Sorry for all the questions, come from a strong vmware / windows background so this is a steep learning curve for me
Thank you

Hi, just some thoughts on some questions.

6x etcd, this doesn't sound right. I'd suggest using an odd number like 3.

mgmt network is handy if you want to create VLAN networks on the same NICs belonged to the management network.

Is this where VMs will naturally sit?

Yes. You can log in to the physical node and check what block device is mounted under /var/lib/harvester/defaultdisk.


Consider moving StringIndexType to Azure.Core

There might be other services that use this type, so better have it generic and not library-specific

For the upcoming release, we have decided to hide the type in .NET and wait until other CS uses it to make sure the values are the correct ones in Azure.Core


Kubectl workspace plugin semantic change proposal

Kubectl workspace plugin change proposal, now that Home workspaces PR is merged:

Change a bit the semantic of the ws command to  better match the typical cd semantics:

ws without parameters == ws '~' == use the home workspace
ws . == show the current workspace
ws ..  == use the parent workspace
ws -  == use the previous workspace

cd takes me home
cd . takes me nowhere
cd .. takes me up one
cd - takes me to last

sgtm

+1 from me for the change

We better change this with v0.6 such that users don't get a breaking change in a few weeks.


Pushing object in array from defined variable adds them by reference, and they can't be modified without affecting the variable.

Version
2.5.9
Reproduction link
https://jsfiddle.net/arwany/uyndonrp/
Steps to reproduce
After the page loads, the array "persons" will be loaded with one item (Thomas).
Now I have provided two buttons to demonstrate the issue/bug I am having:

"  Add - OK": this button pushes new object to the array by passing the object in clear js object format.
"  Add - NOT OK": this button pushes new object to the array by passing an already defined object above (newPerson).

If you click the "Add NOT OK" button two times, and try to change the value of the array in the gray boxes, you will notice that both the object newPerson and the array objects are being changed, as if vue has added them by reference instead of adding them by value (is it even possible?).
What is expected?
To copy the newPerson variable into the array and leave it alone.
Also, when adding multiple objects, I expect to be seperated from each other, but now they seem to be changed all at the same time.
What is actually happening?
I suspect that pushing the object into the array is being added by reference instead of being added by value.

Is this an issue/feature with Javascript or with vue itself?

It should be working as expected. You should clone it by your self.
this.persons.push(Object.assign({}, this.newPerson));
or
this.persons.push({...this.newPerson});

This is how javascript works, you have to clone the object as @JounQin said

I am facing the same problem but using assign function, so it should be a copy.

@JounQin hi it worked for can u explain it in details please

@ua3167 It's just how JavaScript works...
They are sharing a same reference.


PORTALS-2367: add a breakpoint for categories, and flex-wrap the feat…

…ured tools to fix x-overflow

This is all code specific to the NF portal (Tools Database).  Pulling in and letting Robert validate


SAK-40099 - Update CKEditor to 4.9.2, elFinder to 2.1.39 and update all plugins

I was 100% sure that this will break Master, the dependencies will be populated to the Maven repos soon.

@jonespm I've published the elfinder dependency on webjars, it's in maven central:
http://search.maven.org/#search|ga|1|g%3A"org.webjars.bower"
How can I publish a dependency in the Sakai maven repo?
http://search.maven.org/#search|ga|1|g%3A"org.sakaiproject.webjars"
Thanks!

The image2 release failed I must have missed a step, I sent instructions on how you can release in the future. I have to leave for airport but can look at it as soon as I get a chance.


[QUESTION] Is the driver fully compatible with non-typed null ?

Question
I'm working with Spring boot and mssql, but there is a performance problem in spring's jdbctemplate where it searches to discover the type of the field when null is entered as a parameter and this greatly worsens the application's performance and I want to disable this behavior , but I need to know if the mssql driver always expects a type to be informed when passing a null value, or if reporting a null without a type is something fully supported, I know that the problem is not from this repository, but I couldn't find any document that says whether or not the driver supports untyped null arguments.
Link to the jdbctemplate issue if the issue was not well explained: https://github.com/spring-projects/spring-framework/issues/25679 (Although the issue talks about postgresql the same problem occurs in mssql).

hi @viniciusxyz
I am unsure about the issue you are having I took a quick look at the issue linked as you mentioned this is about another driver but the API referenced there do work properly in the Microsoft JDBC Driver for SQL Server. As a matter of you if you look through the tests in the repo there are numerous cases setObject(paramIndex, null) is called.
If you have a specific case you are having problems with the driver please provide a standalone java repro that uses the Microsoft JDBC Driver for SQL Server directly (ie without Spring) along with the schema used for the repro so we can investigate.
Thanks


Templates in VS11 Beta

The first thing I noticed is that the Script# templates do not work in VS11 Beta, at least not out of the box. However, I am able to successfully load and compile a Script# solution that had been created using VS2010.
In attempt to get them to work, I copied the templates from VS10 into VS11, re-registered templates and created a new Script# Script Library project. This works fine.
I then added an Import Library project, which fails because it cannot find various files in the template, like "AssemblyInfo.cs" and "Class1.cs". These files are present and correct in the template zip file.
I tracked it down to one difference - the .zipinfo file. This file exists in the Script Library project template but not in the Import Library template. After copying it from the Script Library template to the Import Library template (and modifying the file to suit its new template name and contents, etc) I can now successfully create an Import Library project in VS11 Beta.
Unfortunately I can't find out a lot of information about this file - is it something proprietary to Script#? I've had a look through some other template types and I can't see it being used anywhere else. It's also interesting that only 1 Script# project seems to use it, and VS11 has some sort of problem loading templates that do not have it.
Do you think it would be appropriate to modify the templates project so that all templates now have this file?

The .zipinfo shouldn't actually be there. It is a remnant of a file that directed the build process to list what to zip (which has since been replaced by msbuild tasks), and was specific to the homegrown script# build process. Looks like one .zipinfo remained in the source tree.
Its odd that the existence of a .zipinfo makes a template work...
I wonder if something has changed in VS11 that must be accounted for in the .vstemplate files … and the existence of a .zipinfo file (or perhaps any other file) is simply side-stepping that logic. I suspect you're hitting a bug in VS itself...


Directional typing

The absolute killer feature for this IDE would be if it were possible to just type out linear sections of code in any direction. This is roughly how I imagine this working:

The cursor has a direction associated with. This could probably use the same little wedge icon that's used for IP directions, but in black/white (depending on dark mode setting).
Have buttons and maybe keyboard shortcuts to change the direction.
Typing a character generally advances the cursor in the current direction after typing, applying edge wrapping if necessary.
Typing a space advances the cursor without changing the current character.
Backspace deletes the current character and moves the cursor backwards. (Or maybe it shouldn't even delete the character, so that there's an easy ways to move the cursor forwards and backwards along its current direction without affecting the grid.)
Del deletes the current character without moving the cursor.
Typing a mirror automatically adjusts the cursor position.
Not entirely sure how to deal with branches (either <> or grid corners). I would say either default to positive, or have a toggle somewhere that indicates whether to follow positive or negative branches.

There's probably some weird edge cases where this behaviour doesn't quite do what you expect, but I think in the majority of use cases this would feel very natural and intuitive for typing code into the grid.

When editing the current cell while running the typing direction would be initialized by the current execution direction.
Ideally, but not necessarily, this would be done while execution is paused.

When moving backwards, branches aren't ambiguous. But It is ambiguous when moving backwards through a corner in the direction of the corner, because two paths lead there. I the use has just moved forward through that corner, that information could potentially be stored to help decide which path to move through when going backwards.

How about: backspace moves you to the negative edge, shift or ctrl+backspace moves you to the positive edge. Gives the user access to both paths without requiring any fancy logic for keeping track of where they came from.

I think I prefer tab and shift+tab for navigating forward and backwards on the typing axis. Space could also navigate forwards if you want. I would like backspace to delete text as you move backwards.

Oh yeah, of course. Tab and/or space (+shift) for moving along the typing axis makes a lot of sense.

The only thing missing at this point is a way to set the typing direction with buttons. It would be particularly useful on mobile.

Stepping back over $ is currently a bit weird. It skips the $ itself. Perhaps when navigating backwards, it should be treated like a noop?
Typing [ sends the cursor to the default location of IP 5. Typing ] or # sends the cursor to the default location of IP 1. This seems reasonable enough, but the behavior could be modified so that these characters don't affect navigation, if desired.

Yeah, I think []# should probably do nothing special for directional typing. This default behaviour is going to be helpful exactly once per program, and afterwards is probably going to be a bit annoying. For $, ignoring it when stepping backwards seems reasonable.

I suppose for $ the most correct thing to do is to skip the instruction immediately following it when stepping backwards.
The logic would be something like the following. When moving backwards, first assume the preceding instruction is a no-op and take another step in the same direction. If we arrive at a $, then stay there. Otherwise, simulate the execution of the preceding instruction as normal.

The above suggestion wouldn't work correctly for two $ in a row. I think it would be necessary to keep navigating backwards while encountering $ in order to count them, then determine what to do based on the count. I can't just save the movement history, because the use may navigate backwards without ever having navigated forwards.

I'm not sure trying to skip stuff while navigating backwards is desirable. If you reach a $/ while going backwards, it's just as likely that you were planning to move back through the mirror instead of the $, so I think not skipping is the less confusing behaviour.

Works for me. It's also simpler and I hadn't started implementing it yet.


[WIP] feat(volume) Add softAntiAffinity and zoneSoftAntiAffiny for volume

https://github.com/longhorn/longhorn/issues/5358

I have tested the UI
LGTM, thanks!


Node: Add binary variant to PUBSUB NUMSUB and PUBSUB SHARDNUMSUB

@Yury-Fridlyand: for these 2 commands, I am not sure how to verify binary in the test. Any suggestion?


Set up automatic app publishing to beta channels

https://github.com/Apple-Actions/upload-testflight-build


Programatically access trace from a standalone hardhat node

Use case
I would like to run a standalone node using hardhat node, deploy a contract from a script on it and access the trace from within the same script.
Suggested solution
Custom RPC endpoint(s) could be implemented to allow calling functions such as tracer.lastTrace() remotely.
Would you be willing to accept a PR implementing such a solution?

Custom RPC endpoint(s) could be implemented to allow calling functions such as tracer.lastTrace() remotely.
Would you be willing to accept a PR implementing such a solution?

Sure, but how are you planning to implement this? Since to add an RPC method, I believe you need to modify the hardhat repository.  Is it possible to add an RPC method to hardhat network via a plugin? Then this makes sense.
Some info that might be useful: this plugin hijacks the ethereumjs VM object which does all the low-level evm operations and collects the relevant trace data as tx keeps on executing on the VM and finally makes it available to consume through the lastTrace(). However, this cannot work if you are using an rpc node (even if it is hardhat node over localhost, it is possible to hh node --trace but it only works in the node terminal, not in your tests).

Yes, it is possible. I plan to override _send method of provider here. Hardhat-tracer already implements it's own provider so I guess it will be easy


Refactor tests

Codecov Report
Attention: Patch coverage is 82.83737% with 248 lines in your changes are missing coverage. Please review.

Project coverage is 70.15%. Comparing base (f854d2a) to head (005581f).

Files
Patch %
Lines

src/cmd/send_bulk.rs
45.55%
49 Missing :warning:

src/cmd/send.rs
38.77%
30 Missing :warning:

src/sources/postgres.rs
0.00%
23 Missing :warning:

src/email_builder/receiver.rs
83.82%
22 Missing :warning:

src/email_transmission/client.rs
60.37%
21 Missing :warning:

src/cmd/init.rs
37.93%
18 Missing :warning:

src/email_builder/mime.rs
93.10%
16 Missing :warning:

src/email_builder/message.rs
93.56%
15 Missing :warning:

src/email_provider/aws.rs
0.00%
13 Missing :warning:

src/email_transmission/smtp.rs
0.00%
13 Missing :warning:

... and 6 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main       #6       +/-   ##
===========================================
+ Coverage   14.98%   70.15%   +55.17%
===========================================
  Files          29       28        -1
  Lines        2329     1997      -332
===========================================
+ Hits          349     1401     +1052
+ Misses       1980      596     -1384

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


Update test requirements to latest versions

Updated each test requirement to latest
Removed oslotest since this is an openstack thing
Bumped pylint to just 1.9.4 because that's the latest py2 and py3
compatible version

Self merged, sorry.  Wanna get these test requirements up-to-date


⚠ Bofrost.be has degraded performance

In deaefe1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1909 ms

Resolved: Bofrost.be performance has improved in aa621de after 6 minutes.


Improve outer menu

Outer Menu is not very practical yet.
As of now, the outer menu is collapsed by default when working on a clientWindow that is smaller than 1981px.
While extended, it is floating above the content.
This is not ideal, forcing the user to navigate the menu or expand it manually to better locate connections.
Suggestion is to apply the large screen layout when extended and apply it as the default status.

I believe it would be interesting to have the possibility to collapse the sidebar of the selected DB. So it would make it possible to view the editor broader perspective (I speak from experience).

Hi @Leonardofreua ,
By sidebar you refer to the area of database tree + properties/ddl?
So, a toggler quick-action to collapse it entirely?

Hi @Leonardofreua ,
By sidebar you refer to the area of database tree + properties/ddl?
So, a toggler quick-action to collapse it entirely?

Exactly. Today have a feature to drag the sidebar, but it would be interesting to collapse it.

This implementation contemplates a few structural changes to the layout besides the toggling action.

Removing the fixed style of the menu.
The main container of the primary tab is now styled with css GRID with template set in place: 'menu content' 100% / auto minmax(0, 1fr). With this, the outer_menu still allows internal scroll content when heavily populated, while the content area can expand/shrink.
The default status of the outer_menu is expanded, but the user can collapse it.
Collapsing/Expanding the menu changes the space available for the content, and many components are not fluid. This means that the toggling action also triggers a resize on these elements, defaulting to the context available. This has been centralized in a refreshHeights method that calls a v_tab_tag.resize() method. Each type of inner_tab has this resize function callback assigned to them.
Floating panels, such as the snippets global panel and the full-view panel have been re-worked to account for the new structure as well.
Added a toggle to the main workspace that collapses/expands the content from the left (database tree + properties/ddl). This also triggers resizes for components.


Multiple local sources

Am I missing something or is it just not possible to have multiple local sources? I have a vector source and like to add some rasters but that's not allowed in one source. And I can't figure out how to add a second local source to my project?!?
Isn't this possible?

It is not possible to combine local sources in Mapbox Studio. In order to "composite" (combine) sources, you need to upload them.
Mapbox Studio accepts mapbox ids or a single tilejson url.
A workaround would be to use https://github.com/tnightingale/tilelive-vector-composite and tessera to combine different local sources and expose them via single tilejson url.
See here: https://github.com/mapbox/mapbox-studio/issues/368#issuecomment-62240537

Problem is that my GeoTiffs are 100GB+ ... But I'll have a look at the tilelive-vector-composite. Thanks!

+1 for this feature. It would be nice to have multiple sources available to style/composite somehow.
I tried hacking around this by manually setting my source in the
project.yml file:
...
source: "composite:///opt/mapping/styles/test-multi-source.tm2/sources.json"
...

and the project.xml file:
...
  <Parameter name="source"><![CDATA[composite:///opt/mapping/styles/test-multi-source.tm2/sources.json]]></Parameter>
...

But Mapbox Studio Classic did not appreciate the "composite:" protocol.
Just an FYI for those who thought they could be sneaky.

Is there an update on that issue? Is there a way to combine local sources or even local with remote sources and style them in Mapbox Studio Classic?
Any hints are very much appreciated!


B 18186 remove ppm completer

B-18186
Summary
How To Test
Because this story focuses on removing unused code. It is unlikely it should affect currently active code. However the closest means to do so would be to:

Ensure that all Jest/Playwright/Go tests pass (or that no new tests are failing).
Do a brief run through with all activities relating to creation, approval,deny, delete, update operations on PPMs.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against a2aa373eb0409cbe0340963f0c0e1c2415f02de5

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 0f56aba37f3c16e28139b5826145125c3665a01a


Refactor ngsi-go

Proposed changes
This PR refactors ngsicmd, ngsilib and e2e test.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[X] I have signed the CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you
did and what alternatives you considered, etc...

Coverage decreased (-0.03%) to 99.969% when pulling b9be2f9f51e820b3d4c5d4d6f635e53e6ba7fded on fisuda:update/refactoring into 205c82f08553b98a470edba1ef9e0f513591586c on lets-fiware:main.

Coverage decreased (-0.03%) to 99.969% when pulling b9be2f9f51e820b3d4c5d4d6f635e53e6ba7fded on fisuda:update/refactoring into 205c82f08553b98a470edba1ef9e0f513591586c on lets-fiware:main.


Add diagnostics settings for function app

This PR is:

To add extra security on bicep files
To add diagnostics settings to the function app resource
To update GitHub Actions workflow

LGTM - Just tested and confirmed all works as expected. Thanks @justinyoo !


Remove GA verification and custom dimensions

We moved these to a new repo
https://github.com/alphagov/digitalmarketplace-govuk-frontend/pull/15

This will be a breaking change, so we'll need to add a major version bump & update the Changelog so we know not to pull it in to apps that haven't been upgraded yet.


Bottom of tabli menu hidden.

Not sure if anyone else has this issue. With many tabs open the bottom of the tabli menu is hidden (after scrolling down completely), as well as the tab/window count. Tested on windows 10 chrome Version 51.0.2704.84 m. Tabli v0.9.3. Changing body max-height from 537 to about 330px seems to fix it, though it does affect the pop out window as well.

Just realized it was messing up because of zoom was set at 125%


Not able to generate chairs from Trained Model

Hi Meet,
I successfully trained the model for 20000 epochs without any error. But why I try to generate chairs like your readme suggests using the following command
python 3dgan_mit_biasfree.py 1 ./models/biasfree_19960.cptk
Nothing happens. The code opens cuda but does nothing
This is my trace
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcublas.so.8.0 locally I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcudnn.so.5 locally I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcufft.so.8.0 locally I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcuda.so.1 locally I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcurand.so.8.0 locally All dependencies not loaded, some functionality may not work Tensor("gen/Relu:0", shape=(32, 4, 4, 4, 512), dtype=float32) g1 Tensor("gen/Relu_1:0", shape=(32, 8, 8, 8, 256), dtype=float32) g2 Tensor("gen/Relu_2:0", shape=(32, 16, 16, 16, 128), dtype=float32) g3 Tensor("gen/Relu_3:0", shape=(32, 32, 32, 32, 64), dtype=float32) g4 Tensor("gen/Tanh:0", shape=(32, 64, 64, 64, 1), dtype=float32) g5 W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE3 instructions, but these are available on your machine and could speed up CPU computations. W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations. W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations. W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations. W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations. W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations. I tensorflow/core/common_runtime/gpu/gpu_device.cc:885] Found device 0 with properties:  name: TITAN X (Pascal) major: 6 minor: 1 memoryClockRate (GHz) 1.531 pciBusID 0000:01:00.0 Total memory: 11.90GiB Free memory: 11.01GiB I tensorflow/core/common_runtime/gpu/gpu_device.cc:906] DMA: 0  I tensorflow/core/common_runtime/gpu/gpu_device.cc:916] 0:   Y  I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: TITAN X (Pascal), pci bus id: 0000:01:00.0) I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: TITAN X (Pascal), pci bus id: 0000:01:00.0)
Please do let me know if I am missing something.
Thanks,
Nitin

You need to input the threshold here:
https://github.com/meetshah1995/tf-3dgan/blob/master/src/3dgan_mit_biasfree.py#L251


clarified regex message

Previous wording made it sound like the regex allows special characters, but it really only supports alphanumeric ones

@quinn-mchugh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@quinn-mchugh - thanks for this fix.

#sign-off


 However zipping that said folder itself results in a file of a format called CPGZ when it is unzipped.

Is zipping a folder not supported or am I doing something wrong? Here's the snippet of code.
do {
    let folderPath = NSBundle.mainBundle().bundleURL.URLByAppendingPathComponent("My Images")
    try Zip.zipFiles([folderPath], zipFilePath: zipPath, password: nil, progress: { progress in
        print(progress)
    })
} catch {
    print("Zipping failed :(")
}

I've uploaded a demo project to Dropbox here.

zipFiles is currently expecting NSURLs of filePaths and not a folderPath. You can grab the contents of the folder path and pass them to zipFiles.
That said, being able to zip a folder path would be useful. I'll add this to a future version.

Hi @Isuru-Nanayakkara, this is now solved with https://github.com/marmelroy/Zip/commit/075ee6cbdbdc5833809511d3c90d3690da7cfcfe
I want to add some folder-specific tests before bundling this feature in the next release but you can try it in the meantime.
Thanks again for the question. Closing this issue.

Released.


Getting 'this._teardown is not a function' on open-wc-haunted-demos

EnvInfo

Browser: Chromium 77.0.3865.120 (Official Build)

To reproduce

open the haunted demos on StackBlitz
wait for the demo to load
click between Beginner/Intermediate/Advanced

Hmm, looks like it could be a bug in Haunted? cc @matthewp
I can reproduce it by:

going to: https://open-wc-haunted-demos.stackblitz.io/basic
click intermediate, it loads fine
click basic, it gives the error

Looks like a bug in Haunted. useEffect is not working with async functions and assumes that any return value is a teardown function, which is not the case here. You can file an issue at https://github.com/matthewp/haunted or if not I will after I get off work.

https://github.com/matthewp/haunted/issues/144
should have it fixed shortly.

Fixed in Haunted 4.6.2


Add how to make Datadog interpret error logs as errors

Sounds good, I haven't found any docs that actually tells how the status field is used, but I'll add this one which at least describes what it is.


glfwInit hangs

When i try to call glfwInit function with glfw x86 version 3.2.1 library on Windows 10 intel graphics, it hangs on _glfwInitJoysticksWin32 call. When i replace glfw version from 3.2.1 to 3.1.2 it solves.

Are you able to test with latest master? There have been lots of changes in the upcoming 3.3 release, and this issue might have already been fixed.

I have just tested with latest master and it works. it seems resolved 👍


Merge in spahan's tree

While I was busy in The Big Blue Room it seems @spahan made a bunch of really useful changes. So I'll work on merging those in.

Put those changes in branch spahan. Suspect my recent changes have caused merge conflicts, but shouldn't be too hard to resolve.

(and in case Hanspeter reads this, I'll fix them since I caused them! No worries.)

First pass at fixing them is in the spahan-merge branch.

There we go. Will merge this into master and do an rc release.


increase the width

I want it to fill the width dynamically.

For now, you can update the tileSize of the theme dynamically. Does that help?

Ah yes, I missed that. What unit does it use? How would I dynamically generate 90% of the screen width?

To use 90% of the available space I would use a LayoutBuilder and divide de maxWidth of the constraints instance by 7. Otherwise I would use a MediaQuery and also divide the width of the screen by 7.
Envoyé à partir de Outlook pour iOShttps://aka.ms/o0ukef

De : Lee Casey @.>
Envoyé : Monday, June 12, 2023 7:19:30 AM
À : GaspardMerten/date_range_picker @.>
Cc : Gaspard Merten @.>; Comment @.>
Objet : Re: [GaspardMerten/date_range_picker] increase the width (Issue #1)
Ah yes, I missed that. What unit does it use? How would I dynamically generate 90% of the screen width?
—
Reply to this email directly, view it on GitHubhttps://github.com/GaspardMerten/date_range_picker/issues/1#issuecomment-1586600848, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJE35ODJLQODX3WPMS6NNDLXK2RGFANCNFSM6AAAAAAYGBMBME.
You are receiving this because you commented.Message ID: @.***>

Perfect! Thank you 😊
Lee
On 12 Jun 2023 at 08:04 +0100, Gaspard Merten @.***>, wrote:

To use 90% of the available space I would use a LayoutBuilder and divide de maxWidth of the constraints instance by 7. Otherwise I would use a MediaQuery and also divide the width of the screen by 7.
Envoyé à partir de Outlook pour iOShttps://aka.ms/o0ukef

De : Lee Casey @.>
Envoyé : Monday, June 12, 2023 7:19:30 AM
À : GaspardMerten/date_range_picker @.>
Cc : Gaspard Merten @.>; Comment @.>
Objet : Re: [GaspardMerten/date_range_picker] increase the width (Issue #1)
Ah yes, I missed that. What unit does it use? How would I dynamically generate 90% of the screen width?
—
Reply to this email directly, view it on GitHubhttps://github.com/GaspardMerten/date_range_picker/issues/1#issuecomment-1586600848, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJE35ODJLQODX3WPMS6NNDLXK2RGFANCNFSM6AAAAAAYGBMBME.
You are receiving this because you commented.Message ID: @.>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.>


merge sdk config demo into main page

Summary
This PR merges the SDK configuration work into the main index page for more seamless testing
How did you test your changes?

yarn dev
localhost:3001
spin up scw locally to test the ingested values

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Including guidance for Windows redirection GPO

We have been receiving feedback from many customers that they are lacking this specific guidance.  In customer meetings we have reviewed this guidance and it has helped, so I'm proposing that we include this guidance on our support page for admins.

@kaarins
@gacarini

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Event Macro Option

Hi. This is a great module. Very good!
Is it possible to add an option to call a macro on a time?
Ex.:
When you move the time to 6 A.M. it will call a macro that will play the sound of a rooster.

Thanks for the suggestion, but About Time already does this, so I'm not going to also implement it here. :)


position enum

Should use an enum for position instead of strings

Should make the position an int in the database that we map to an enum. This allows for automatic mapping instead of manual mapping.


Backend refactor

This is quite a large ticket and it fundamentally changes the backend... I'll be writing docs so it's explicitly clear how it's behaving on the frontend (I'll use postman to gen them). For now, use postman to test it. It's only got a couple requests... I've exported my settings and you can find them on the discord.
I know it's quite large but it should make the code much, much more readable for everybody : D!
P.S.,

To generate a JWT hit the login endpoint, copy the token, go to environment (top right where it says devisle-tutorial-site_LOCAL) and hit the cog.


Feature Request: Ubuntu Version Control

Currently the default Ubuntu version is 18.04, but I'm wondering if there is a way to make this configurable so that 16.04 is the target version? I notice that in the runtime containers there are 16.04 versions, so I was hopeful. Thanks!

Infrastructure changes to support a wider variety of base images and configurations is being planned as part of #48, so you can track the progress of the feature there. Out of curiosity, what use case is motivating the need for an Ubuntu 16.04 version?

We have a software stack which targets 16.04, and we were looking for something which  is compatible. We actually found a way to circumvent that need, though it would still be cool to have it if free. Thanks for the response!

Just for the record in case anyone else finds this issue and isn't clear on this distinction: the binaries produced by packaging an Unreal Engine project in a ue4-docker container image based on Ubuntu 18.04 will happily run in an Ubuntu 16.04 variant of the ue4-runtime container image, since UE4 bundles its own clang toolchain and CentOS 7 sysroot in order to ensure wide compatibility with different versions of Linux.
The only reason someone would need an Ubuntu 16.04 version of the ue4-docker container images (as requested here) is if the software you are using at compile time is something that requires that version of Ubuntu. Even in that case, you can still copy the Installed Build of UE4 from the ue4-minimal or ue4-full container images into a separate image based on a different version of Ubuntu, since the Engine itself is also built with the same toolchain and sysroot. So if you find yourself in the same boat as @AutonomousHansen was but can't find a way around the requirement for a specific version of Ubuntu, there's still a solution readily available to you while the infrastructure redesign process is ongoing.


New feature "Do not store my password"

I have written a feature to be able to configure an account so that the ingoing and outcoming password won't be stored in the settings. It works like this:

I have added a checkbox "Don't store my password" in the account options.
At start such accounts are disabled (I put a text "[-]" in front of the account name in the account list to identify such accounts easily)
When you open the account, K-9 will ask you for the password(s) but won't store it in the properties, so that in remains in memory only until K-9 is closed. You can also press cancel to access the stored messages without entering a password.
There is also a new menu item in the menu on the account list to forget all sessions passwords without having to exit K-9.

Please tell me if you have any comment on my modifications.
PS: To disable an account as long as the password hasn't be entered, I have reused the existing "disabled" feature. This option was for imported accounts, for which K-9 doesn't know the password yet. The first time you open this account, K-9 ask you for the password and store it. I simply added the notion of session passwords, which won't be stored.
PS2: This modification would cover http://code.google.com/p/k9mail/issues/detail?id=1243

I created issue #677 for this feature. Closing this PR as quite some functionality is missing and it's unlikely we get this rebased without much effort.


pkg/k8s: hold mutex while adding events to the queue

As there are certain code regions where the mutex is held it might
occur the events are put in to the channel out of the original order
they got the mutex.
Fixes: 7efa98e03b9b ("k8s: Factor out service and endpoints correlation into a new ServiceCache")
Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

Coverage increased (+0.03%) to 44.222% when pulling a7c351ef288e91a83358ec9718d790050a3a9d60 on pr/fix-mutex-for-svs into f1f3d2c1d10866cd85309dee782a58a488907cbd on master.

test-me-please


PR Watcher and Manual Validation update.

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Also some analyzer warnings in your other scripts -
PS D:\Git\winget-pkgs> Invoke-ScriptAnalyzer D:\Git\winget-pkgs\Tools\ManualValidationProfile.ps1

RuleName                            Severity     ScriptName Line  Message
--------                            --------     ---------- ----  -------
PSUseDeclaredVarsMoreThanAssignment Warning      ManualVali 15    The variable 'SharedFolder' is assigned but never used.
s                                                dationProf
                                                 ile.ps1
PSUseShouldProcessForStateChangingF Warning      ManualVali 31    Function 'Set-Status' has verb that could change system
unctions                                         dationProf       state. Therefore, the function has to support
                                                 ile.ps1          'ShouldProcess'.
PSUseShouldProcessForStateChangingF Warning      ManualVali 65    Function 'Update-Stuff' has verb that could change system
unctions                                         dationProf       state. Therefore, the function has to support
                                                 ile.ps1          'ShouldProcess'.
PSUseApprovedVerbs                  Warning      ManualVali 41    The cmdlet 'Run-Validation' uses an unapproved verb.
                                                 dationProf
                                                 ile.ps1
PSAvoidUsingWriteHost               Warning      ManualVali 6     File 'ManualValidationProfile.ps1' uses Write-Host. Avoid
                                                 dationProf       using Write-Host because it might not work in all hosts,
                                                 ile.ps1          does not work when there is no host, and (prior to PS 5.0)
                                                                  cannot be suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      ManualVali 27    File 'ManualValidationProfile.ps1' uses Write-Host. Avoid
                                                 dationProf       using Write-Host because it might not work in all hosts,
                                                 ile.ps1          does not work when there is no host, and (prior to PS 5.0)
                                                                  cannot be suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      ManualVali 36    File 'ManualValidationProfile.ps1' uses Write-Host. Avoid
                                                 dationProf       using Write-Host because it might not work in all hosts,
                                                 ile.ps1          does not work when there is no host, and (prior to PS 5.0)
                                                                  cannot be suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.

PS D:\Git\winget-pkgs> Invoke-ScriptAnalyzer D:\Git\winget-pkgs\Tools\PRWatcher.ps1

RuleName                            Severity     ScriptName Line  Message
--------                            --------     ---------- ----  -------
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 58    'where' is an alias of 'Where-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1225  'where' is an alias of 'Where-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1279  'where' is an alias of 'Where-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1312  'where' is an alias of 'Where-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1492  '%' is an alias of 'ForEach-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1493  '%' is an alias of 'ForEach-Object'. Alias can introduce
                                                 ps1              possible problems and make scripts hard to maintain. Please
                                                                  consider changing alias to its full content.
PSAvoidUsingCmdletAliases           Warning      PRWatcher. 1507  'iwr' is an alias of 'Invoke-WebRequest'. Alias can
                                                 ps1              introduce possible problems and make scripts hard to
                                                                  maintain. Please consider changing alias to its full
                                                                  content.
PSUseDeclaredVarsMoreThanAssignment Warning      PRWatcher. 14    The variable 'appName' is assigned but never used.
s                                                ps1
PSUseDeclaredVarsMoreThanAssignment Warning      PRWatcher. 289   The variable 'nvalidColor' is assigned but never used.
s                                                ps1
PSUseDeclaredVarsMoreThanAssignment Warning      PRWatcher. 1285  The variable 'matchVar' is assigned but never used.
s                                                ps1
PSUseDeclaredVarsMoreThanAssignment Warning      PRWatcher. 1465  The variable 'i' is assigned but never used.
s                                                ps1
PSPossibleIncorrectComparisonWithNu Warning      PRWatcher. 1233  $null should be on the left side of equality comparisons.
ll                                               ps1
PSPossibleIncorrectComparisonWithNu Warning      PRWatcher. 1248  $null should be on the left side of equality comparisons.
ll                                               ps1
PSUseBOMForUnicodeEncodedFile       Warning      PRWatcher.       Missing BOM encoding for non-ASCII encoded file
                                                 ps1              'PRWatcher.ps1'
PSUseShouldProcessForStateChangingF Warning      PRWatcher. 1513  Function 'Set-PadRight' has verb that could change system
unctions                                         ps1              state. Therefore, the function has to support
                                                                  'ShouldProcess'.
PSUseApprovedVerbs                  Warning      PRWatcher. 1484  The cmdlet 'Create-Sandbox' uses an unapproved verb.
                                                 ps1
PSUseApprovedVerbs                  Warning      PRWatcher. 1499  The cmdlet 'Check-ForManifestEntries' uses an unapproved
                                                 ps1              verb.
PSUseSingularNouns                  Warning      PRWatcher. 16    The cmdlet 'Watch-PRTitles' uses a plural noun. A singular
                                                 ps1              noun should be used instead.
PSUseSingularNouns                  Warning      PRWatcher. 1499  The cmdlet 'Check-ForManifestEntries' uses a plural noun. A
                                                 ps1              singular noun should be used instead.
PSAvoidUsingWriteHost               Warning      PRWatcher. 33    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 35    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 37    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 41    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 43    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 45    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 46    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 47    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 83    File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1212  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1276  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1307  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1318  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1331  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1355  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1387  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1429  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1436  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1437  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingWriteHost               Warning      PRWatcher. 1446  File 'PRWatcher.ps1' uses Write-Host. Avoid using
                                                 ps1              Write-Host because it might not work in all hosts, does not
                                                                  work when there is no host, and (prior to PS 5.0) cannot be
                                                                  suppressed, captured, or redirected. Instead, use
                                                                  Write-Output, Write-Verbose, or Write-Information.
PSAvoidUsingEmptyCatchBlock         Warning      PRWatcher. 1385  Empty catch block is used. Please use Write-Error or throw
                                                 ps1              statements in catch blocks.
PSAvoidUsingEmptyCatchBlock         Warning      PRWatcher. 1509  Empty catch block is used. Please use Write-Error or throw
                                                 ps1              statements in catch blocks.

Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information.

The problem with this error is that the alternatives are not equivalent. Write-Output has no option similar to Write-Host's -nonewline to suppress adding a newline at the end of output. This suggests that PowerShell shouldn't be used to write console applications such as these.

Avoid using Write-Host because it might not work in all hosts, does not work when there is no host, and (prior to PS 5.0) cannot be suppressed, captured, or redirected. Instead, use Write-Output, Write-Verbose, or Write-Information.

The problem with this error is that the alternatives are not equivalent. Write-Output has no option similar to Write-Host's -nonewline to suppress adding a newline at the end of output. This suggests that PowerShell shouldn't be used to write console applications such as these.

Precisely why I provided the snippet for suppressing the Write-Output item

For these,

Functions which modify state not supporting ShouldProcess
Approved Verbs
Use Singular nouns

To suppress the error I'm using the "antipattern of make everything start with Get-".
This one doesn't make sense to me - isn't null additive and commutative?

Incorrect null comparison ($null should always be on the left side of equality comparisons)

This one doesn't make sense to me - isn't null additive and commutative?

Incorrect null comparison ($null should always be on the left side of equality comparisons)

https://learn.microsoft.com/en-us/powershell/utility-modules/psscriptanalyzer/rules/possibleincorrectcomparisonwithnull?view=ps-modules

Precisely why I provided the snippet for suppressing the Write-Output item

My local client doesn't enjoy the snippet:
PowerShell 7.4.0
Manual Validation build: 510
ParserError: C:\repos\winget-pkgs\Tools\PRWatcher.ps1:46
Line |
  46 |  [Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSAvoidUsingWrite …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Unexpected attribute 'Diagnostics.CodeAnalysis.SuppressMessageAttribute'.
PS C:\ManVal>

Precisely why I provided the snippet for suppressing the Write-Output item

My local client doesn't enjoy the snippet:
PowerShell 7.4.0
Manual Validation build: 510
ParserError: C:\repos\winget-pkgs\Tools\PRWatcher.ps1:46
Line |
  46 |  [Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSAvoidUsingWrite …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Unexpected attribute 'Diagnostics.CodeAnalysis.SuppressMessageAttribute'.
PS C:\ManVal>

Edit: it only errors if present in both files. Will validation need it in both, or will one be sufficient?

@stephengillie - Can you try the PR that I raised into your fork?

@ stephengillie - Can you try the PR that I raised into your fork?

I'm not sure how to do this. I don't see a PR in my tree. Am I looking in the wrong place?

@ stephengillie - Can you try the PR that I raised into your fork?

I'm not sure how to do this. I don't see a PR in my tree. Am I looking in the wrong place?

https://github.com/stephengillie/winget-pkgs/pulls
Looks like I have some merge conflicts now that I'll need to resolve

I updated these earlier today, but wanted to have a full round of validations and approvals to catch most of the errors.

@microsoft-github-policy-service agree company="Microsoft"


Update README.md

Hi,
This small PR adds two buttons to get your app on F-Droid and Google Play 👍
close #12

Thanks @Poussinou !


Benny/single nft endpoint

Changes:

Changed many files' formatting with go fmt ./... (I apologize this changed so many files, it happens when I save in vscode)
Added endpoint for retrieving nft by id
Added function for retrieving nft by id from mongo
Added test function for creating an nft then retrieving that nft and finally deleting the nft by id from mongo

Files to review that were not just adjusted by go fmt:

glry_db/t__nft_test.go
glry_db/glry_nft.go
glry_lib/glry_http_handlers.go

Nice work!! that was fast  🔥
Looks great overall - I have one comment about the NOT FOUND case, and left a smaller nit comment above.
I see that for the NOT FOUND case where the requested nft ID doesn't exist, we return {"data":{"nfts":[]},"status":"OK"}
Could we return an error instead of status OK and an empty array of nfts? Similar to how you return an error if no id param is provided?
That would make it easier to handle NOT FOUND cases on the client, and also makes it easier to classify as an error case for logs and other uses

@kaitoo1 agreed that we should return an error in this scenario

example error creation: https://github.com/gallery-so/go-gallery/blob/f50e1b3b846dcac6a6be74203d80c64b212ad6e1/glry_lib/glry_auth_jwt.go#L104
i believe you just need to return this error out of the top-level HTTP handler and it'll send down a 200 with an error message attached

Thank you guys! Will handle the not found case as soon as I can with a similar error as the id not found error.


Unable to retrieve notification token due to unsupplied origin

I'm running the Firebase Cloud Messaging example and caught up with this error:

Failed to load https://fcm.googleapis.com/fcm/connect/subscribe: The 'Access-Control-Allow-Origin' header has a value 'https://localhost:8081' that is not equal to the supplied origin. Origin 'http://localhost:8081' is therefore not allowed access. Have the server send the header with a valid value, or, if an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Has anyone encountered the same issue and figured out how to solve it? Many thanks.

Looks like you are using https somewhere. Could you confirm that you did not change the sample to use https://localhost:8081 anywhere?

No, I'm not using https and also didn't change anything from the original source code except setting the public Vapid key value in index.html