Connection to postgresql fails

### Question
How do I grant select privilege to user?
Iam getting this error when trying to run "terraform apply"
could not start transaction: read tcp <IP_ADDRESS>:64725-><IP_ADDRESS>:5432: read: connection reset by peer
Code
main.tf
resource "google_sql_database_instance" "instance" {
  name             = "my-database-instance"
  region           = "us-central1"
  database_version = "POSTGRES_14"
  settings {
    tier = "db-g1-small"
  }

  deletion_protection = "true"
}
resource "google_sql_database" "database" {
  name     = "my-database"
  instance = google_sql_database_instance.instance.name
}
resource "google_sql_user" "mohith" {
  name     = "<USER>"
  instance = google_sql_database_instance.instance.name
  password = "<PASSWORD>"
}
resource "postgresql_grant" "readonly_tables" {
  database    = google_sql_database_instance.instance.name
  role        = "mohith"
  schema      = "public"
  object_type = "table"
  privileges  = ["SELECT"]
}
terraform {
  required_providers {
    postgresql = {
      source  = "cyrilgdn/postgresql"
      version = "1.18.0"
    }
  }
}

provider "google" {
  # Configuration options
  project     = "<MY_PROJECT>"
  region      = "us-central1"
  zone        = "us-central1-a"
  credentials = "keys.json"
}
provider "postgresql" {
  host             = "<PUBLIC_IP>"
  username         = "<USER>"
  password         = "<PASSWORD>"
  expected_version = "10.1"
}

Additional Details
No response

By default, Cloud SQL prevents any traffic from the public internet to your instance. Meanwhile, the Postgres provider assumes a Postgres database is reachable at the Public IP you've provided.
The easier way around this is to run the Cloud SQL Proxy on localhost and point the Postgres provider at it, e.g.,
./cloud-sql-proxy <INSTANCE_CONNECTION_NAME> --port 5432 # starts a listener on localhost:5432

Then:
provider "postgresql" {
  host             = "<IP_ADDRESS>"
  username         = "<USER>"
  password         = "<PASSWORD>"
  expected_version = "10.1"
}

But, my host id is not that I want to access, Can you please elaborate on this, anyways I tried this, now I see different error-
Error: could not start transaction: pq: SSL is not enabled on the server

So, anyways , I resolved this error. But I have a db where I need to use tunneling for connecting to db, so, how should I configure the provider, so that, I could grant privileges to user.In this the user is being created, but when granting privileges, Iam getting connection timed out.

I’d suggest looking at the Postgres provider docs. Since this isn’t a Proxy issue, I’d suggest asking on StackOverflow if you’re still stuck.


Reddit BOT

Description
This Reddit Bot Pick a subreddit to scan,Designate a specific comment to search forSet your bot's replyautomatic responce.
Fixes #1072

Type of change

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

Checklist:

[X] My code follows the style guidelines(Clean Code) of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md
[X] My documentation follows Template for README.md
[X] I have added the project meta data in the PR template.
[X] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt needed/present, then type none in Requirments.
Category:

[ ] Calculators
[ ] AI/ML
[ ] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[X] OS_Utilities
[ ] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[X] Others

Title: Reddit BOT
Folder: Reddit BOT
Requirements: Readme.md
Script: reddit_bot.py
Arguments: none
Contributor: @bose07
Description: This Reddit Bot Pick a subreddit to scan,Designate a specific comment to search forSet your bot's replyautomatic responce.

@bose07 your code is exactly same as this code


Add skeletons for Mizar GUI

This adds some skeletons of Widgets needed for that part
of Mizar GUI dealing with sampling with frametrack comparison.
Bug http://b/237284844
Test Compile, Manual

https://screenshot.googleplex.com/7UPY65oQyaw3G9a.png

screenshot/7UPY65oQyaw3G9a.png

Just a heads up, I don't think I will find time to do this review today.


ci: Remove unused flakey test dep that is breaking on win

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@adisuissa i think we want to include this one - it prevents windows from failing due to flakes


dictionary builders expects costs[1] as a backward_size

Each dictionary builders set forward_size to cost[0]  and backward_size to costs[1].
e.g. ipadic, neologd, unidic, and ko.
However, the load method at ConnectionCostMatrix reads the backward_size from conn_data[0].
IPADIC, IPADIC-neologd, Unidic has the same value, forward and backward size.
But, ko has different values. So the cost method returns the wrong value.

Relate https://github.com/lindera-morphology/lindera-ko-dic-builder/issues/6

And now, I'm confusing this... I'll take a look a usage...
https://github.com/lindera-morphology/lindera/blob/bc4e3756c2dccda417926518652bcd6fb5e683c5/lindera-core/src/core/connection.rs#L18-L19


Allow client code to determine whether a timezone was explicitly specified

If RETURN_AS_TIMEZONE_AWARE is set to True, and TIMEZONE is None, a timezone aware date is only returned if the input string explicitly specifies the time zone, otherwise an unaware date is returned.
This change allows client code to determine whether or not a timezone was explicitly specified in the date string.

Feel free to add to the discussion here: #161, thanks @thomasst!

We will have to make relevant changes here as well before we can merge this.


Updating GHZ_game kata

Updating kata to meet styling standards, and removing unnecessary Reset.

@tcNickolas can you review this PR?


proxy setting discuss

url = this.proxy + url
above is the code, so it will visit url like this: http://localhost:4042/api/proxy/test/http://api.chbtc.com/data/v1/depth?currency=eth_cny&size=3&merge=0.1
this don't follow url pattern. it 's hard to do url router map in nodejs expressjs.
expressjs seems not able to parse the original url correctly.
can we change it to like this:
http://localhost:4042/api/proxy/test?url=xxxxx
in which url is component encoded.

I think we should allow more flexible proxy pass configuration, maybe allowing to configure the proxy setting not just like this:
    'proxy': 'https://crossorigin.me/', // string URL

...but in this way too (by allowing to pass a custom url mapping function):
    'proxy': url => 'https://crossorigin.me/' + url

This will enable user to set his own proxy URL passing schema, for example:
    'proxy': url => 'https://localhost:4042/?url=' + encodeURIComponent (url)

yes. that would be better.

btw, i think timeout probably should also be put into option

@86chenjie which one? There is a timeout param already )

@kroitor
const ex = new ccxtexchange;
ex.timeout = 2000;
this is the way i use timeout.
setting timeout in options also work ?

@86chenjie yep!) new ccxt[exchange] ({ timeout: 2000 })

Ok, I added support for proxy url function overrides and uploaded a usage example here: https://github.com/ccxt-dev/ccxt/blob/master/examples/js/custom-proxy-url.js
Use it like this:
"use strict";

const ccxt = require ('ccxt');

(async function main () {

    const kraken1 = new ccxt.kraken ({
        proxy: function (url) {
            return 'https://example.com/?url=' + encodeURIComponent (url)
        },
    })

    console.log (await kraken1.loadMarkets ())

    const kraken2 = new ccxt.kraken ({
        proxy: function (url) {
            return 'https://cors-anywhere.herokuapp.com/' + url
        },
    })

    console.log (await kraken2.loadMarkets ())

}) ()

Let us know if it doesn't work for you. I tested it locally, and it should work. Thx!

A shorter version of it:
const kraken1 = new ccxt.kraken ({
    proxy: url => 'https://example.com/?url=' + encodeURIComponent (url),
})


Can not install autodp with version >= 2.1.0

I tried to install the requirement.txt, but it couldn't install autodp with version 2.1.0b0. Basically, it's a problem of autodp. When I tried to run with autodp version 2.0, the scipy optimizer can not minimizer_scaler with method Brent or Bounded. Is there any option that I can sneak through the use of autodp?

Please install the latest version of AutoDP directly from GitHub:
pip install git+https://github.com/yuxiangw/autodp
Also, please install scipy=1.9.1 as AutoDP is not compatible with the newer versions of scipy.

Thank you, I could run it now, just need to downgrade scipy to 1.9.1


[Question] How can i setup fail2ban?

code-server version: Docker Latest
OS Version: Ubuntu 18.04.3 LTS

Description

I want setup Fail2Ban With Docker
Fail2Ban, nginx reverse proxy is located host.
code-server only running docker.

We're going to add rate limiting to code-server. See #1320


add myself as a contributor

Contributing:

https://github.com/rucio/containers/issues/215

https://github.com/rucio/containers/pull/217
https://github.com/rucio/helm-charts/pull/131

https://github.com/rucio/helm-charts/pull/132

ping @bari12 for review


feat(nim): added queries and tests

How and from there did you install the Nim parser? When I execute TSInstall nim I get Parser not available for language "nim".

The pr to add nim to nvim-treesitter was only merged 2 weeks ago. Maybe you need to update you plugin?

I have made a couple of changes:

Formatted the patterns to be more like other queries
Renamed the test file to be more like the other test files
Fixed the cast pattern by merging both patterns into one

We are no longer bound to the limitation of just one opening and one closing node per delimiter. Both the [, ] and the (, ) can be part of the same container on the same level.
(cast
  type: "[" @delimiter
  type: "]" @delimiter
  value: "(" @delimiter
  value: ")" @delimiter @sentinel) @container

If you agree with my changes please squash the commits into one, then I can merge your PR.

Thank you :)
All done.

Merged, thank you very much.


Use Riverpod for adding, updating, and deleting wallets

The current code uses function propagation, but a wallet provider is added and can be used instead.

WIP:
Removed all props and used all wallets from the notifier. Also, used the removeWallet from the notifier. Currently working on implementing add & update wallet in the notifier then use them in wallet screens.

WIP:
All done except for changing the info screen title after editing the wallet name. Also, while I was testing, I found another bug. After logging out and logging in with a diff account, old wallets will be displayed.

Work Completed:
Created clear function wich sets _isListed to false to force listing wallets on logout then login. Reverted changes of onEditName to update screen title.


Fix #825 : More flexible S2I binary build data format

#825

Awesome, thanks a lot @rohanKanojia ! Unfortunately I only can have a look at it as I'm on PTO for the rest of the week (+ Monday next week). I hope that I can then work fulltime on fmp as the last two weeks were eaten by my other work (we are about to release Fuse 7.1).
sorry for that.

@rohanKanojia Thanks a lot for your changes which adhere to the comments I did. However while reviewing and some minor refactorings, I thought again about this PR and I now believe, that a Generator should decide actually be responsible for the creation of the binary input.
Why? Well, because a generator is only about creating the configuration of an image which then can be created in multiple ways (docker, s2i binary, s2i docker, ...). So the generator should be agnostic of the concrete creation of the image.
What now? The generator selects a base image and the base image is actually that part which in the s2i binary input case determines the format of the binary input. So my suggestion would be to either examine the generated images for the 'from' images or evaluate a certain meta-data (label) which could be set by the generator. Then a kind of 'BinaryBuildSelector' would examine this base image and creates the binary input which fits this s2i builder image.
Does this make sense ? If so, I would like to postpone this story for the next sprint implement it in a generator independent way.

@rhuss : ah, okay


Warnings when using the plugin with Typescript

Hi Simon, I'm already using your plugin 👍
When using the package, I'm getting two warnings / errors:
This could be a configuration issue in the package.json?
Maybe type="module" missing?

Package types seem to be not working:

If I have time, I try to provide a pull request, fixing the issue.
But I wanted to keep track of the errors as a issue.
Thanks for your work!

v1.0.2 should now contain correct typescript definitions.


Pushing app marked for debugging hangs

Selecting the "Configure for remote debugging" option in the deployment dialog causes the app deployment to hang:

This occurs specifically in an Azure test vm built with the Visual Studio 2022 image

Best guess is that this is happening in environments that don't trust the downloaded "GetVsDbg.ps1" script.
This seems likely, based on the recent observation in #360 that this test VM doesn't trust (refuses to execute) the recently-included cf-ssh.ps1 script.
This theory is backed up by the hanging behavior; the last line of output printed came from DotnetCliService.PublishProjectForRemoteDebuggingAsync (line) -- this method includes a call to wait for the powershell script to finish executing (line). Once that finishes, the installation script (GetVsDbg.ps1) should be deleted, but we see that script still present in the file system during the hang:


Move UI test snapshots for beta and stable build flavors to peer directories

What
Move UI snapshots around.

build flavor
before PR
after PR

beta
Along side test files
<test-root>/snapshots/beta/

stable
<packlet-root>/stable-snapshots
<test-root>/snapshots/stable/

Why
Housekeeping. There is no need to treat the two build flavors differently.
Similar in spirit to #1887

The mother of all ui change PRs :D


Region.getRectangles is memory-hungry

getRectangles allocates a new dynamic array each time it's called. Unfortunately due to D's memory-munching GC this means an app will keep allocating memory each time this function is called.
This function should be replaced with an input range since Cairo already has indexing operations for each individual rectangle (or clip).
I'm the one who made getRectangles use an array, so I'll fix this asap by replacing it with a range interface.

Correction, it should be a random-access range.


Listeners not working on iOS

The listeners (at least onDurationChanged, onAudioPositionChanged and onPlayerCompletion) don't work on iOS while it does work on Android.
Full Description
As I can guess, none of the listeners work on iOS while they do work on Android.
Code to Reproduce
The example code should do.
Log Errors
None as far as I can see.
Files/URLs/Sources
Not applicable.
Screenshots
Not applicable.
Platforms

OS: iOS.
OS version: 14.2.
Device: Both simulator (iPhone 12 Pro Max, iOS 14) and a physical iPhone 6s Plus also running iOS 14.2.
flutter version: 1.22.4.
audioplayers version: 0.16.2.
release or not release: Debug on simulator and release on the physical iPhone.

With the new update (the Darwin to Swift conversion) they are now working.


Move configuration and logging out of SYSTEM PROFILE

MS doesn't want us to use it - and it's confusing for users. Move the configuration files and logs out of the profile

logs were moved to the install folder some time ago - all that's left is the config file

logs are moved out with 1.6.x release stream - config/identities are remaining

logs were moved but config remains in %appdir%


Investigate - DDSs that don't explicitly serialize handles do not allow handles to be in its data

From https://github.com/microsoft/FluidFramework/pull/3975#discussion_r521627438
DDSs such as Ink, Counter, SharedSummaryBlock do not serialize any handles. This issue it to investigate and make sure that they do not support handles in their data. Otherwise this should break GC which (currently) requires handles to be serialized.

@agarwal-navin Have these DDSes been reviewed to see if they need handle support? Should there be some type-level way of expressing this? E.g. DDSes like this implement ICantStoreHandles or something like that?

I don't think the have. The idea behind this work item is that if a DDS does not support handles, we should have some way of validating that the data being stored is not a handle. I am not sure how we enforce that but this work is about coming up with that plan.
For example, a new DDS added should either support handles or not and we should make sure the right thing happens when data is added to the DDS.

Tracked by AB#407


node 0.10 support

I had to do a couple of things to get this to work on node 0.10:

include babel-runtime to support Symbols
polyfill execSync

Let me know what you think.

Thank you for the PR!
This looks almost good, but just one, babel-runtime should be in dependencies instead of devDependencies.

Looks like tests are failing due to #8


Set up accounts for web services

[x] Github
[x] Slack
[x] Google Docs
[ ] Zotero

@FlavioZeska welcome to Github!
Zotero might be the only remaining thing here. When you're done you can close the issue.

@FlavioZeska welcome to Github!
Zotero might be the only remaining thing here. When you're done you can close the issue.


lucid.misc.io.reading.read doesn't have a mode argument

The documentation says read has a mode argument that defaults to 'rb':
https://github.com/tensorflow/lucid/blob/master/lucid/misc/io/reading.py#L56
Unfortunately, there is no such mode argument.  The lack of mode is unfortunate, since I need to set it to 'r' to get svelte to work in Python 3.

The downside would be that setting encoding=utf-8'would decode in Python 2 as well, and presumably what one wants isstr` in both cases.  The specific problematic read call is
https://github.com/tensorflow/lucid/blob/master/lucid/scratch/web/svelte.py#L56

What I have in my local tree to fix this: https://github.com/girving/lucid/commit/9d45e639f47bd220b455f417f568b47e9d9e92f5.

v0.2.0 adds the mode property to all misc.io.reading functions, which is tested, but I have no integration tests for the svelte stuff. If you update and find that this did not help, please reopen.

@ludwigschubert Confirmed that lucid now works with no change.  Thanks!


Correct 'scope' for test libraries

Corrected scope of test libraries to be 'test' only (not compile)

recheck

recheck


upgrade to 1.5.14

Upgrade haproxy to 1.5.14 as it resolves this CVE.

We are actually running 1.5.14 using a fork but sending a PR seemed a bit pointless as we can't upload the blob.

Added it @cromega

We would love to have a release with this change, thanks.


[Outlook] (preview) New Mac UI now supports saveAsync and compose win…

…dow pop-out

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

File
Status
Preview URL
Details

docs/outlook/compare-outlook-add-in-support-in-outlook-for-mac.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add NRMS/Hypernet tags to cluster resource group in CI mode

Which issue this PR addresses:
The RP fails cluster installs if NSGs are attached to subnets. Simply Secure v1 applies NSGs in our subscriptions when these tags aren't present, resulting in a race condition.

What this PR does / why we need it:
Adds the tags needed to stop SSv1 from applying NSGs to subnets: https://eng.ms/docs/security-compliance-identity-and-management-scim/security/azure-security/security-health-analytics/network-isolation/netiso-teamdocs/netiso-program-overview/manage/hypernet-and-nrms-simply-secure-v1-network-security-rules

Test plan for issue:
Deploy this commit to central us euap (where the issue is faced)

Is there any documentation that needs to be updated for this PR?
No

/approve

hold for now as tagging the resource group didn't have the desired effect. We'll need to update these tags with a temporary SSv1 exception number and re-test.

@ross-bryan we can open a new PR with exception tags later if needed. Closing this one.


hotfix: Editing project would not change slug (if title didnt change)

related to Giveth/giveth-dapps-v2#265
@aminlatifi the addProject and editProject is not used anymore we just kept it for being backward compatible, because painting test cases for those mutations took time I removed them completely
PS If you think it's better to merge it to staging first, we can change the target branch

Thanks @mohammadranjbarz


JS: Add failing test for npm subdependency updates

Updating npm subdependencies would be awesome because it would allow us to patch security vulnerabilities in sub-dependencies.

@feelepxyz - would love your help on this one next week!


cut v2.27.0

/test integration-crc

/test e2e-crc

e2e-crc is failing on the cert rotation test:
 --- Failed steps:
  Scenario: Start CRC "in the future" and clean up # features/cert_rotation.feature:9
    When checking that CRC is running # features/cert_rotation.feature:14
      Error: not found: running. Timeout
33 scenarios (32 passed, 1 failed)
270 steps (266 passed, 1 failed, 3 skipped)

integration-crc is failing with the invalid tls cert error when pulling podman bundle from quay.io:
  initializing source docker://quay.io/crcont/podman-bundle:4.4.4: pinging container registry quay.io: Get "https://quay.io/v2/": x509: certificate signed by unknown authority

Both are known issues, this should be good to merge!


Added NATS subscribers to connect

tested connect on amd64 (with and without docker-compose), receiving messages from the local connect and from connect on arm64.

Hey @ccorley - thank you for making the additional OCI changes. I have verified that the OCI runtime is working as expected.


Don't use redux logging/development helper middleware in production builds, fixes #523

Signed-off-by: Victor Porof<EMAIL_ADDRESS>https://github.com/mozilla/tofino/issues/523
@jsantell

Coverage remained the same at 42.464% when pulling 01015ec7cfb5f2563081492f1877942bbc575ac0 on victorporof:rm-redux-middleware into afee003cdfd7e148051c9df6c845ce1158e7c3e3 on mozilla:master.

Coverage remained the same at 42.018% when pulling 191a9f0c9388053a13266267b1e7f752a19bd290 on victorporof:rm-redux-middleware into 7d9f2caca29274736885d47903acf6522692eae1 on mozilla:master.


Confusing error message when using tryParse with onChanged in a TextFormField

Hi, I think this is pretty minor, but it threw me for an hour or so.  There are a few things interacting so I will try to explain each step.
I have a TextFormField with a number type keyboard.
In the onChanged callback, I use tryParse to convert the text to an int.  Because text can be null, tryParse requires the ! operator
Now if the user types in some numbers and then deletes all to them tryParse throws a null check error.  Now this seems fair enough since the text in the onChanged call back is empty.  However, it is not null it is ""
However, if you try to put a null check into the logic, the compiler knows that the text in onChanged can't be null and so will flag it as not necessary and ignore it (which is technically correct.... I think).
   onChanged: (text){`
        if(text !=null){          // <= (1)`
            int.tryParse(text)!
    }

The check in (1) is ignored as it is deemed necessary, but the resulting error is that a null check operator was used on a null value.  This leaves the dev is left in a weird position that the error reported is that a null value has been passed, but the fix for it is invalid.   Eventually, I worked out that it works if you check that text is not ""
onChanged: (text){`
          if(text !=""){          // <= This fixes the null check error`
     int.tryParse(text)!
    }

As i say, I don't think this is a bug per se, but the error reported doesn't help fix the issue.

Hi @Strain-solutions, Thanks for filing the issue. Please share the output of flutter doctor -v along with the minimal and complete code sample and the error logs.


Enabling Dark mode , form validation, dynamic content loading (GSSOC CONTRIBUTOR 2024)

Enabling Dark Mode: Allows users to toggle between light and dark color schemes for improved readability and reduced eye strain during nighttime browsing.

Form Validation: Validates user input in web forms in real-time to ensure data accuracy and completeness before submission, enhancing user experience and reducing errors.

Dynamic Content Loading: Loads content asynchronously without refreshing the entire page, providing a smoother browsing experience and reducing page load times for better performance.

please assign me this issue for GSSoc'24. i will be able to handle the theme of the page along with improving the UI

hey @Its-Aman-Yadav
I would like to add Dark mode for the website. please assign me. Thank you!


Breedx2/response bytes

Let the Response object use bytes[] and don't assume String. Provide convenience methods to get the bytes as a String. This breaks backwards compatibility with callers that expect getBody() to return String. Callers will need to call getBodyString() instead.

Looks yummy :ice_cream: to me.


Docs

Still working on this, but wanted to get some feedback. How does it look? Is it targeting the right level? Would it be better if we used concrete values in the examples, rather than showing them "algebraically"? Is it too much to go into use of applicative style, etc.
Also, psc-docs sorting the output alphabetically is quite unhelpful when doing things this way, as I was trying to build on info from Functor with Apply, etc. but obviously that doesn't work so well in the output.

I really like the new docs :)
This looks good.
One passing comment - I wonder if it is better to explain Apply in terms of things like liftA2, or in terms of something like zip. To me, <*> is either "a way to lift pure functions of N >= 2 arguments to take arguments of type Maybe a, or, possibly even simpler, zip for lists of length 0 or 1.

Maybe that explanation is best left for the actual typeclass. It explains things more generally, whereas I would think we'd want to be more specific for an actual data type.

Does the case of "pure functions to take arguments of type Maybe a" not come across in the latter part of the Apply description then? Although that mentions using it with <$>, that was what I was trying to convey.

I think it does. I would probably just change "... to a function that does not usually expect them". We're not passing Maybe a where we expect an a, but turning a function of type a -> b -> ... into a function Maybe a -> Maybe b -> ..., right?

Ah right, I see what you mean. Makes sense :+1:

Not a complaint at all, by the way. I think this is great.

Alright, done now I think. I carried on commenting the instances although that may be a little redunant, as well, I'd already done most of them. ;)

Looks great! :ship:


npcx: support SHI enhanced mode for npcx4

This PR adds:

Add the enhanced mode support to SHI driver for npcx4.
Add the PM_POLICY lock to prevent the chip from entering the deep sleep state during SHI transaction

I'm not able to verify enhanced. Host Command code looks good. I'll test it on hardware tomorrow.

Works for me with npcx7. Are you going to change also https://source.chromium.org/chromiumos/chromiumos/codesearch/+/main:src/platform/ec/zephyr/include/cros/nuvoton/npcx.dtsi;l=105 ?


on android, handle file received from intent action_view

On Android this allows to receive media also from the intent action_view.
Example use case : clic on an email attachment to open it with our app

intent.action.VIEW is handled in package as url/text type. When user try open a file, android system send the uri of this file to intent.
So, you can handle action.VIEW with this code in flutter side:
// For sharing or opening urls/text coming from outside the app while the app is closed
ReceiveSharingIntent.getInitialText().then((String? value) {
  setState(() {
    _sharedText = value;
    debugPrint('path of file: ${Uri.parse(value!).path}');
  });
});

intent.action.VIEW is handled in package as url type. When user try open a file, android system send the uri of this file to intent.
if you need open file with dart:io functions like readAdBytes or readAsString, you should convert the Uri to full path. You can do this with uri_to_file package.
So, you can handle action.VIEW with this code in flutter side:
import 'package:uri_to_file/uri_to_file.dart';
...
// For sharing or opening urls/text coming from outside the app while the app is closed
ReceiveSharingIntent.getInitialText().then((String? value) async {
    String _sharedText = value ?? "";
    debugPrint('path of file: ${Uri.parse(_sharedText).path}');
    File file = await toFile(_sharedText);
    var fileContent = await file.readAsString();
    ...
});

This won't work for all files as you'll get permission issues, this plugin should still be able to send the content set when adding the intent extra "EXTRA_STREAM".

This pull request, looks very promising as I am facing the same issue. Can it get merged?

Can this branch get merged?

Can this branch get merged?

doesn't look like this repo is maintained anymore. Soon there will be more forks than stars 😢


TF Chat Room

Added the TensorFlow Chat Room on Stackoverflow URL in the resources section.

I'm unsure where we draw the line on what we include in the README and what not. Adding wicke@ for policy

The initial commit (f5d859777b3340319d44cc4f8897cad9039d0ddd) is fine, although we should add something like "(not actively monitored by the TensorFlow team)".
@theadactyl where do you think this pointer would be best? I'm actually thinking the community site: https://www.tensorflow.org/community/forums, rather than resources, which seems to be more about consumable content than interactive forums.

But yeah, the other two commits look like a mistake to me.

@martinwicke @mihaimaruseac The other two commits are indeed a mistake, I messed up my TensorFlow fork branches while working on another contribution to the repo, I will fix this.

@martinwicke @mihaimaruseac I have fixed the accidental commits. I apologize yet again for this confusion.

LGTM
On Tue, Jun 30, 2020 at 10:16 AM Mihai Maruseac<EMAIL_ADDRESS>wrote:

@mihaimaruseac approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/40828#pullrequestreview-440227196,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABREJVPWU5U6URIBVIEGUITRZIMV7ANCNFSM4OJIYOCQ
.


net shell updates

Add option to start / stop DHCPv4/6 client
Add attach/detach sub-command to net virtual command. This allows user to manually attach virtual network interfaces on top of other interfaces.
DHCPv6 status printing was missing from net iface command
Allow user to disable not used net-shell commands. It can save a bit of flash if the unused shell commands can be discarded. Currently the default is still the same, so disabled commands are shown in the net-shell listing.

updated according to comments
I did not create a separate NET_SHELL_COMPLETE config as the NET_SHELL_SHOW_DISABLED_COMMANDS kind of provides the same functionality. We can certainly the name etc here.


Add #111 Switch to lefthook for git hooks

This PR addresses this issue.
Even though it is mentioned in the issue to use lint staged this PR sets them up using lefthook after a discussion with the maintainers.

Now this can be merged right?  😃 @Akalanka47000


[WIP] Provider for holidays in Estonia

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
-

Will add tests tomorrow.

Thank you @lukosius for the PR! I'll have a look once you have added the unit tests. Without unit tests it would make it harder for me to verify and it ensures the quality of the library.


fix(github-actions): updated github actions

Introduction :pencil2:
Ensure Google release please deprecated plugin has been replaced with the recommended script
Resolution :heavy_check_mark:

Replaced GHA plugin with release-please CLI command.
Added few additional GHA github.yml and pr.yml.
Added prettier check in lint.yml.

Miscellaneous :heavy_plus_sign:

Dependencies updates
Lint fix
Re-build govUkFrontend.js accomponied with SRI update.

@ttbarnes should we merge main into no-pdf first?

Agree, I think it'd be best to merge main into no-pdf instead.
However, there seems to be an issue with the release version, where it has the incorrect version? PR was closed
I think we should fix that before proceeding

@ttbarnes should we merge main into no-pdf first?

Agree, I think it'd be best to merge main into no-pdf instead.
However, there seems to be an issue with the release version, where it has the incorrect version? PR was closed
I think we should fix that before proceeding

We need to merge this PR first in main-application-no-pdf to fix the release version of this base branch.


We can't change the label style

Hi @zahinafsar 👋
For styling the Y-Axis labels we can use the yAxisTextStyle prop.
For X-Axis labels -
We can give our own custom label component to the entries of the data array using the labelComponent property.
Since it is a custom component, you can give it any styling of your choice.


allow render_react_component(name, props, class: "...")

We need to specify HTML class attributes to the wrapper div elements.
Can you review it please?
BTW I can't pass the test (#8) so the CI will fail :(

Can you elaborate on why you need this? Specifically, why does any code besides hypernova's clientside JS need to reference this div?

This is because we need to apply CSS.
Out code includes CSS something like this:
foo {
  *[data-hypernova-key] {
     // blah blah blah
  }
}

This works but I think it is ugly. If this PR is merged, I just add a CSS selector to the hypernova's div.

@gfx what CSS are you trying to apply that can't be applied to the root element of your react tree?

Here you are:
  *[data-hypernova-key] {
    height: 100%;
  }

The height of a block element must be specified to the very element.

To clarify, does this mean you're rendering the entire HTML content with react, and you want it to take up the whole screen?

@ljharb This is definitely a required feature when you are server-side rendering your app's critical css, along with the markup.
As it stands right now, say I have a login page using a Flexbox based layout, and I'm centering the login form on the page. My critical css is injected styling the body to be flex and so on.
The div that Hypernova creates will break the layout because it's unstyled, isn't a flex element and has a height which will be whatever the contents is - ergo a broken layout.
The fallback right now is as @gfx says, you end up writing this one silly rule that targets the data-hypernova-key when I'd rather have an explicit class on the div itself.

I have a couple of use-cases:

Reduce unnecessary
s by styling the generated div instead of another wrapping
.
Apply a grid area name to the container to make the generated div fit an area of the page (sub-grid feature is not yet supported)
Apply a sticky position on either the generated div to make the react component sticky against the document.

Put it all together:

    <%= render_react_component('nav', { ... }, { style: "position: sticky; top: 0" }

  ...

I could move the grid-area: nav to the generate div and make the react root component sticky. But either way, AFAIK, there is a necessity to style the generated div.


Fix docs for v0.57.2

What this PR does / why we need it:
https://sonobuoy.io/docs defaults to https://sonobuoy.io/docs/v0.57.2
and we missed adding content for v0.57.2.
This commits copies over the content from the main branch
and updates some metadata to point to v0.57.2.
Special notes for your reviewer:
This PR also has an additional fix to github actions to fix the goreleaser command. We download the latest version of goreleaser and it doesn't support the --rm-dist flag.
Release note:
NONE

@mohitsharma-in can you review?


Correctly unset energy_entity attribute on config entry for consistent behaviour

See #2476

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

7 of 7 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
4450

Relevant Lines:
4450

💛  - Coveralls


Fixed the login page text while checking responsive

Related Issue or bug

Info about Issue or bug

Fixes: #95
Describe the changes you've made
I have fixed the text in the login page looking larger just like the register page.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project.
[x] My changes generate no new warnings.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots

@swapnilsparsh Please review the PR and suggest the changes you want in this.

Issue resolved by @jatiinyadav


Write README

What else should I put in readme file?
I could describe functionality, but for now its not really impressing. Once bigger part of beprof is ready I will make changes in README.md file.

2-3 sentences about the purpose of the project
installation (and deinstallation procedure) using pip+git (I prefer copy-pasteable instructions)
some example usage - cuople of lines of the code which:

import beprof
get some profile data from the web
create profile object
calculate FWHM or other features
plot profile using matplotlib

Already done. Take a look and give me feedback: is it OK or something is missing/needs to be changed?


Cannot install recent near-shell

Running the following and getting an error message:
❯ npm install -g near-shell
npm ERR! path /usr/local/lib/node_modules/.staging/near-shell-9f5d58de/node_modules/@babel/code-frame
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall rename
npm ERR! enoent ENOENT: no such file or directory, rename '/usr/local/lib/node_modules/.staging/near-shell-9f5d58de/node_modules/@babel/code-frame' -> '/usr/local/lib/node_modules/.staging/@babel/code-frame-940ef598'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR!      /Users/zaver/.npm/_logs/2019-09-18T13_01_53_606Z-debug.log

It was a fluke


Adding Sprite and load

Pull Request Test Coverage Report for Build 63

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.934%

Totals

Change from base Build 61:
0.0%

Covered Lines:
820

Relevant Lines:
1324

💛  - Coveralls

Pull Request Test Coverage Report for Build 63

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.934%

Totals

Change from base Build 61:
0.0%

Covered Lines:
820

Relevant Lines:
1324

💛  - Coveralls


Add event source CloudEventIngress

Fixes #151
Proposed Changes

Add CloudEventIngress CRD for managing this event source
A Knative Service is created for every CloudEventIngress, waiting for CloudEvent message and forwarding to a sink URI.

Release Note

I think we could discuss whether to use a ContainerService or a Knative Service. I prefer a Knative Service because it can leverage the existing mechanism, e.g. the scalability, the security, the ingress, and etc.
@Harwayne  @vaikas-google here need your inputs too. Thanks.

Close this pull request because a lot of changes have happened. A new PR will be submitted later.


Partial discriminator-maps not working anymore since 1.2.7

BC Break Report

Q
A

BC Break
yes

Version
1.2.7

Summary
The changes calculating the discriminator mapping introduced a BC break in our mappings, because the discriminator map in the mapped-superclass only contained one of the documents. That may be "smelly", but it worked before.
Previous behavior
Having a partial discriminator map did automatically add the missing discriminator-mappings using the FQCN as discriminator-mapping value.
Current behavior
The documents not in the discriminator-map can be saved but not fetched anymore.
If I add the other documents or remove the discriminator-map completely it works again.
How to reproduce
Current mapping
Mapped superclass
<doctrine-mongo-mapping
        xmlns="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping
        http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping.xsd"
>
    <mapped-superclass
            name="Eventjet\Ticket\AbstractTicket"
            collection="tickets"
            inheritance-type="SINGLE_COLLECTION"
    >
        <field name="id" id="true" type="uuid" strategy="NONE"/>
        <field name="number" fieldName="number" type="string"/>
        <field name="scan_code" fieldName="scanCode" type="string"/>
        <field name="revoked" type="boolean"/>
        <reference-one
                field="ticket_type"
                fieldName="ticketType"
                target-document="Eventjet\Ticket\TicketType\AbstractTicketType"
        />
        <discriminator-field name="type"/>
        <discriminator-map>
            <discriminator-mapping value="standing_room" class="Eventjet\Ticket\StandingRoomTicket"/>
        </discriminator-map>
        <indexes>
            <index name="tickets_revoked_seats">
                <key name="seat_id"/>
                <key name="revoked"/>
            </index>
        </indexes>
    </mapped-superclass>
</doctrine-mongo-mapping>

document mapping contained in discriminator-map:
<doctrine-mongo-mapping xmlns="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping"
                        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                        xsi:schemaLocation="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping
                        http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping.xsd">
    <document name="Eventjet\Ticket\StandingRoomTicket">
        <field name="event" type="event"/>
    </document>
</doctrine-mongo-mapping>

one of the document mappings not contained in discriminator-map:
<doctrine-mongo-mapping xmlns="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping"
                        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
                        xsi:schemaLocation="http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping
                        http://doctrine-project.org/schemas/odm/doctrine-mongo-mapping.xsd">
    <document name="Eventjet\Ticket\Seatmap\SeatTicket">
        <field name="seatmap" type="string"/>
        <field name="seat_id" fieldName="seatId" type="string"/>
    </document>
</doctrine-mongo-mapping>

Consider the following test cases:
    public function testStandingRoomTicketEventIsSaved(): void
    {
        $ticketType = $this->createStandingRoomTicketType();
        $event = $ticketType->getEvent();
        $this->eventRepository->save($event);
        $this->ticketTypeRepository->save($ticketType);
        $ticket = new StandingRoomTicket($ticketType, $event);

        $ticket = $this->saveAndReloadStandingRoomTicket($ticket);

        $this->assertSame($event->getId(), $ticket->getEvent()->getId());
    }

    public function testPersistSeatTickets()
    {
        $seat = ObjectFactory::concreteSeat();
        $seatmap = $seat->getSeatmap();
        $this->getContainer()->get(ConcreteSeatmapRepositoryInterface::class)->save($seatmap);
        $ticketType = ObjectFactory::seatmapCategoryTicketType($seatmap);
        $this->ticketTypeRepository->save($ticketType);
        $ticket = ObjectFactory::seatTicket($ticketType, $seat);

        $loaded = $this->saveAndReloadSeatTicket($ticket);

        $this->assertSame($ticket->getSeat()->getId(), $loaded->getSeat()->getId());
        $this->assertSame($ticket->getTicketType()->getId(), $loaded->getTicketType()->getId());
    }

The first one still workes with 1.2.7, the other doesn't. Adding the discriminator-mapping of the document or removing the whole discriminator-map in the mapped-superclass makes the tests pass again.

Yeah, this looks like it would be my fault in #1962. I have to see what I can do here - looks like this is one of those situations where we can't fix one use-case without breaking another.
For the record even though I've already mentioned it in Slack, partial discriminators will be deprecated in the upcoming 1.3 release and dropped entirely in 2.0, so you'll have to update the logic soon.
I'll see if I can fix the issue without breaking more things 😑

Both could be fixed by saving all parent discriminators in metadata, then using $nin specifying all parent discriminators (or discriminators of other classes on same level), instead of current $in with discriminators of current class. This is more complicated approach though and not sure if worth it when use case in this issue is going to be removed in 2.x. Personally I would just close this issue as it's relying on undefined behaviour and reverting #1962 would cause another regression between 1.2.7-1.2.8

reverting #1962 would cause another regression between 1.2.7 and 1.2.8

Since it never worked before 1.2.7, I'm willing to take that risk.

Personally I would just close this issue as it's relying on undefined behaviour

Relying on "undefined" behaviour is what happens after a software gets older. Remember, the first beta versions of 1.x are almost 9 years old - people come to rely on certain things, especially when you tag a final release. Whether it's undocumented or not, I don't like breaking people's workflows, no matter how ridiculous it seems. Also keep in mind that ORM exhibits the same behaviour here, so this is not just "relying on undefined behaviour", you could almost assume a pattern here.
Fact is, this regression causes a BC break while updating. This is made worse by the fact that we're only deprecating partial discriminator maps in 1.3 for removal in 2.0, so I can hardly go out and break this now.
I'm making another attempt at fixing this, I'll see if diffing discriminator values from the map produces a fix for the issue that doesn't break valid use cases. If not, this PR is getting reverted today, with 1.2.9 fixing the regression introduced by this patch.

We've reverted #1962 in #1978 for the 1.x branch. The fix will be added only to 2.x to avoid the regression with partial discriminator maps.


Feedback

gslides version: (you need to do a release and tags for the version to be consistent)
Python version: 3.7
Operating System: Linux

docs

Probably should bold or make it more obvious which API we need, right now it's in the notes which actually happens to be smaller font than regular doc
getting auth is confusing, it would appear to me that I should/would have to enable the scope of the app for the required API? (wasn't obvious till the code), also I didn't realize it had to be a web application I downloaded a desktop application
https://michael-gracie.github.io/gslides/basic.html (jupyter notebook link broken)
also wasn't obvious that I had to pick myself as a tester but that might be a Google doc issue

other feedback

Picky but it's very strange that the child class of something called a Series should be things like Line/Scatter/Area etc....wouldn't these come under something like Plot or Chart or something? If something inherited from a Series I'd expect series manipulation (I don't know what that would be to be honest but feels like the children classes are mostly plotting kind..)
Is there a particular reason why one must call the execute method? This comes back to the naming issue. If you name something CreateSpreadsheet I would expect the class to create as opposed to having to do an execute function. Is there a particular reason why the execute has to be called explicitly?
Also, just starring at https://github.com/michael-gracie/gslides/blob/main/gslides/sheetsframe.py, looks like a lot of things are repeating itself, you could definite do with a super class and inherit, at least the CreateSpreadsheet and CreateSheet
I second Kun's addSlide/rmSlide idea, it feels so much more natural than calling a construction method everytime. A follow up to this is that it's actually kind of strange that everything is a class. It seems that the only method I would ever call on every class is execute, which goes back to the previous point. It definitely feels like those Line/Area/Scatter can be methods as opposed to classes
personal habit but maybe not zero logging? This is hard to debug if any bug.

Overall though the functionality is good and it's pretty cool. I haven't seen a similar package in pypi so this might be useful to some people (if their internal organisation opens the Google API). Advanced usage I could think of would be porting the presentation back into python and manipulate the graphs, because currently this is only going one way (data -> plot). What if I want to get the data back and do something else?

Picky but it's very strange that the child class of something called a Series should be things like Line/Scatter/Area etc....wouldn't these come under something like Plot or Chart or something? If something inherited from a Series I'd expect series manipulation (I don't know what that would be to be honest but feels like the children classes are mostly plotting kind..)

I think that goes back to Kun's comment about aligning naming. Series is a naming that google uses but I am not consistent across the package

I second Kun's addSlide/rmSlide idea, it feels so much more natural than calling a construction method everytime. A follow up to this is that it's actually kind of strange that everything is a class. It seems that the only method I would ever call on every class is execute, which goes back to the previous point. It definitely feels like those Line/Area/Scatter can be methods as opposed to classes

Definitely will do the full refactor to make the class into objects

personal habit but maybe not zero logging? This is hard to debug if any bug.

Good point. The one valuable piece of information that would make sense logging is the json blob that is being passed to the Google API


Exec tachikoma update<PHONE_NUMBER>0338

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io

Coverage remained the same at 68.0% when pulling 665cfb2ce241f6131eb801dab68071621e585c5f on tachikoma/update-20150320150338 into 68e5c0f4004e386f9c6f06d01dee9ecf2e132fab on master.


Flutter 3.0 drawer strange flick when click an item inside it

Description
I was developing an app before update my Flutter from 2.15 to 3.0. When I updated my Flutter version, I started notice a "flick" on drawer when I click an item inside it. The flick can be seen on screen below:

Flutter doctor
[✓] Flutter (Channel stable, 3.0.0, on Microsoft Windows [Version 10.0.22000.675], locale en-US)
• Flutter version 3.0.0 at C:\Users\Josel\fvm\default
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ee4e09cce0 (4 days ago), 2022-05-09 16:45:18 -0700
• Engine revision d1b9a6938a
• Dart version 2.17.0
• DevTools version 2.12.2
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at C:\Users\Josel\AppData\Local\Android\sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[✓] Visual Studio - develop for Windows (Visual Studio Build Tools 2019 16.11.13)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
• Visual Studio Build Tools 2019 version 16.11.32413.511
• Windows 10 SDK version 10.0.19041.0
[✓] Android Studio (version 2021.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
[✓] VS Code (version 1.67.1)
• VS Code at C:\Users\Josel\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.40.0
[✓] Connected device (4 available)
• Redmi Note 5 (mobile) • 44d6160 • android-arm64  • Android 9 (API 28)
• Windows (desktop)     • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.675]
• Chrome (web)          • chrome  • web-javascript • Google Chrome 100.0.4896.127
• Edge (web)            • edge    • web-javascript • Microsoft Edge 101.0.1210.39
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
Code Sample
class PreOrderInfoDrawer extends StatelessWidget with PreOrderFunctionsMixin {
  const PreOrderInfoDrawer({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Drawer(
        child: Column(
          children: [
            ListTile(
              leading: const Icon(Icons.add_shopping_cart),
              title: const Text('Novo pedido'),
              onTap: () => createNewOrder(context),
            ),
            ListTile(
              leading: const Icon(Icons.person_add),
              title: const Text('Selecionar cliente'),
              onTap: () => searchCustomer(context),
            ),
            ListTile(
              leading: const Icon(Icons.search),
              title: const Text('Localizar pedido'),
              onTap: () => searchOrder(context),
            ),
          ],
        ),
      ),
    );
  }
}

@lucas404x
Can you try the official drawer code sample and see if you still get same behavior ?

@lucas404x
I tried the same code sample on latest master, but don't see the behavior.
https://user-images.githubusercontent.com/67046386/168521901-42a65a64-2361-49d8-9614-7190e0eab85c.mov
Can you provide a short video of the behavior ? Also try the same on latest master and see if it persists or not. Moreover check on other devices and see if it works on them or not.

@darshankawar
It's really strange. This behavior happens with Redmi Note 5, but It doesn't happen with the emulator (Pixel 4 API 29).
Redmi Note 5 (scrcpy)
https://user-images.githubusercontent.com/42918677/168566355-0655c232-6fd2-4259-9946-06cedf3bcad0.mp4
Pixel 4 API 29
https://user-images.githubusercontent.com/42918677/168567775-3ad70145-57f4-4990-b717-cc8a43b1b786.mp4

@darshankawar

I can't test in more devices right now.
I didn't test on emulator because I tested before record. And yes, this not happens.
Yes, this happens on real devices. Actually, the Redmi Note 5 is my real device. The scrcpy just stream the screen of my smartphone to PC.

I don't know if It's possible to see it on video, but in the windows version I noticed a mis-rendering happening as well. It's more discreet compared to my real device, but if you see at right edges you can observe the bug.
https://user-images.githubusercontent.com/42918677/168608332-7479e861-62ed-4249-9066-11dde15abdef.mp4

@lucas404x
Thanks for the update. I tried the same on Samsung device but don't see the behavior you reported. I don't have the Redmi device so can't possibly confirm the behavior.
Per your original description, this behavior wasn't there in stable 2.10.5, but when you upgraded to 3.0, it started happening ?
Can you provide videos of both version's behavior ?

@darshankawar
Redmi note 5 (scrcpy) with Flutter 2.10.5
https://user-images.githubusercontent.com/42918677/168793776-de91e21e-5033-4eae-bbba-d01172268d95.mp4

I tried again on Samsung on which I am unable to see the behavior.
https://user-images.githubusercontent.com/67046386/168810879-0e5f4b34-f967-4a7c-ac3e-089d5041e4c3.mov
Maybe or maybe not it could be related to : https://github.com/flutter/flutter/issues/103566 which has been fixed and pushed to master. Can you switch to latest master and try again on RedMi to see if behavior still persists ?
flutter channel master
flutter upgrade

@darshankawar
I noticed a few improvements but the flick persists somehow
https://user-images.githubusercontent.com/42918677/168814611-24247695-9835-43b0-9678-da7050ea9211.mp4

@lucas404x
Can you try using hand gestures instead of mouse and see using it also the behavior persists or not ?

Hi @lucas404x,  I tried to reproduce the issue on Redmi K20 pro (Android 11) with the below code sample, But I don't see any flickering while the drawer animates.
https://user-images.githubusercontent.com/31410839/169035378-de94fb7c-2172-4bc6-bea8-56320c532976.mov

code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);
  final String title;
  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      drawer: PreOrderInfoDrawer(),
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

class PreOrderInfoDrawer extends StatelessWidget {
  const PreOrderInfoDrawer({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Drawer(
        child: Column(
          children: [
            ListTile(
              leading: const Icon(Icons.add_shopping_cart),
              title: const Text('Novo pedido'),
              onTap: () {},
            ),
            ListTile(
              leading: const Icon(Icons.person_add),
              title: const Text('Selecionar cliente'),
              onTap: () {},
            ),
            ListTile(
              leading: const Icon(Icons.search),
              title: const Text('Localizar pedido'),
              onTap: () {},
            ),
          ],
        ),
      ),
    );
  }
}

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 3.0.0, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 3.0.0 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (6 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your
        plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.66.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 101.0.4951.64

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Thanks @maheshmnj .
Based on our verification which we tried on different Android devices but were unable to replicate, it seems the issue could be specific to Redmi note 5 device.
I'll keep it open and label for further insights from the team.


Makes tooltip directive

Ticket Link
GROUPINGS-1325
List of squashed commits

Makes jasmine test for tooltip directive
Replaces all tooltips with the directive
Adds to the tooltip directive comment

Test Checklist

[X] Unit Tests Passed:
[X] Jasmine Tests Passed:
[X] General Visual Inspection:

Sorry, one more thing, I realize the comments would be better as a jsdoc at the top of the directive. Make a summary and list of parameters the directive can take in the jsdoc instead.

Sorry, one more thing, I realize the comments would be better as a jsdoc at the top of the directive. Make a summary and list of parameters the directive can take in the jsdoc instead.

I'm sorry, what do you mean by parameters?

Sorry, one more thing, I realize the comments would be better as a jsdoc at the top of the directive. Make a summary and list of parameters the directive can take in the jsdoc instead.

I'm sorry, what do you mean by parameters?

Like how tooltip can be on its own or as tooltip="text". Also how you have to use data-placement to set the position.

The jsdoc looks great.


Add SQL decorator in new Python programming model

See https://aka.ms/pythonprogrammingmodel

azure-functions Python library v1.18.0 has been released with SQL decorator support.


seek a podcast mp3 then lost duration

This podcast:

https://pdst.fm/e/chtbl.com/track/95538/traffic.megaphone.fm/ADL7076246752.mp3

Playing it is fine, but when you do "seek", the player lost duration(duration == 0).
And it can not perform seek anymore( will cause crash ).
There are also other podcast have same issue.
I'm looking into the issue, but unless you've already addressed it :)

Hey
Since I can't reproduce this could you please let me know more of the environment.
I'll need to setup an issue template to have more context of the issues raised.

I'm using Xcode 12.2, iOS 14.2
With latest master branch of project Example.
It's not all podcast can reproduce this.
please try this mp3:

https://pdst.fm/e/chtbl.com/track/95538/traffic.megaphone.fm/ADL7076246752.mp3

I reproduced this issue.
This seems to be with the header fields of the response and I'm handling it, I didn't take into account the case insensitive that sometimes servers send back.
This is fixed by #7


Feature - Combine predefined and action based roles

Description
This branch introduces middleground between the "action based" roles and the "predefined" role. The branch allows the company representative to invite members that can have the role speaker, booth owner and company member. The branch restricts also the option to change roles once they're set in.
User invited as a speaker can request presentation (if there is a space for presentation in the company) or join already existing presentation.
User invited as a booth owner can request booth (if such is not requested yet) or join the other booth owners.
User invited as a company member can request presentation (if there is a space for presentation in the company) or join already existing presentation, request booth (if such is not requested yet) or join the other booth owners. Once they take an action towards booth or presentation they'll become speaker or booth owner. If they don't take a decision they'll remain company members.
closes #510 (and maybe others I am not sure)
Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected) (MAYBE)
[X] This change requires a documentation update

What needs to be tested

[x] Register as company representative and approve the company (mod)
[x] Invite like 6/7 users or so
[x] Register/Log in as user who is company member and check that you can request a presentation (reject it as a mod so that you don't constantly migrate:fresh)
[x] Register/Log in as user who is company member and check that you can request a booth (reject it as a mod so that you don't constantly migrate:fresh)
[x] Register/Log in as user who is speaker and check that you can request a presentation (approve it as a mod)
[ ] Register/Log in as user who is speaker and check that you can join the other presentation as speaker
[ ] Register/Log in as user who is company member and check that you can join the other presentation as speaker
[x] Register/Log in as user who is booth owner and check that you can request a booth (approve it as a mod)
[ ] Register/Log in as user who is booth owner and check that you can join the other booth owners
[ ] Register/Log in as user who is company member and check  that you can join the other booth owners

The gold sponsor was working as well but check it as well (so 2 presentations but make sure you free the gold sponsor route either from mod or from the database)
Checklist:

[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

@TimKardol
I checked all of the scenarios we talked about and in my environment the following worked:

Register/Log in as user who is speaker and check that you can join the other presentation as speaker
Register/Log in as user who is company member and check that you can join the other presentation as speaker
Register/Log in as user who is booth owner and check that you can join the other booth owners

There was an issue with:

Register/Log in as user who is company member and check that you can join the other booth owners

I believe it was fixed with the new commits


Fix 2D Particle velocity with directed emission mask

Changed CPU velocity calculation for EMISSION_SHAPE_DIRECTED_POINTS to follow the same logic as in the GPU version:
mat2 rotm;
rotm[0] = texelFetch(emission_texture_normal, emission_tex_ofs, 0).xy;
rotm[1] = rotm[0].yx * vec2(1.0, -1.0);
VELOCITY.xy = rotm * VELOCITY.xy;

Now both CPUParticles2D & CPUParticles3D (z disabled) show the same results as their GPU counterparts and take the initial velocity settings into account.
Tested on 3.2 branch only because CPU particles are currently not supported on master.
Fixes #40807

Thanks!

Cherry-picked for 3.2.3.


add ticket_form_id to tickets

https://developer.zendesk.com/rest_api/docs/core/tickets
add the ticket_form_id in the ticket

thanks


Handle code blocks nested within Markdown

Frequently Asked Questions

[x] I have read the FAQ before opening the issue

Workflow version
v2024.17
Alfred version
5.5.1 [2273]
macOS version
Sequoia 15.1 (24B83)
Debugger output
[23:19:34.204] ChatGPT / DALL-E[Text View] Running with argument ''
[23:19:34.260] ChatGPT / DALL-E[Text View] Script with argv '' finished
[23:19:34.265] ChatGPT / DALL-E[Text View] {"response":"> Give me examples of markdown, including nested and multiple code blocks\n\nCertainly! Markdown is a lightweight markup language used to format text. Here are some examples, including nested and multiple code blocks:\n\n### Basic Text Formatting\n```markdown\n# Heading 1\n## Heading 2\n### Heading 3\n\n**Bold Text**\n\n*Italic Text*\n\n[Link to OpenAI](https://www.openai.com)\n\n- Bullet list item 1\n- Bullet list item 2\n\n1. Numbered list item 1\n2. Numbered list item 2\n```\n\n### Code Blocks\n```markdown\nInline code: `print(\"Hello, World!\")`\n\nCode block with syntax highlighting (Python):\n```python\ndef greet(name):\n    return f\"Hello, {name}!\"\n\nprint(greet(\"World\"))\n```\n```\n\n### Nested Code Blocks\nMarkdown doesn't support nested code blocks directly, but you can achieve a similar effect using lists:\n\n```markdown\n1. Step involving code:\n\n    ```python\n    def do_something():\n        pass\n    ```\n\n2. Another step with different code:\n\n    ```bash\n    echo \"Hello from bash\"\n    ```\n```\n\n### Multiple Code Blocks\n```markdown\nHere is a JavaScript example:\n\n```javascript\nfunction sayHi() {\n    console.log(\"Hi!\");\n}\nsayHi();\n```\n\nAnd here is a JSON example:\n\n```json\n{\n    \"name\": \"John\",\n    \"age\": 30,\n    \"isStudent\": false\n}\n```\n```\n\n### Blockquotes\n```markdown\n> This is a blockquote.\n>\n> > Nested blockquote with more information.\n>\n> Concluding remarks.\n```\n\n### Tables\n```markdown\n| Syntax | Description |\n|--------|-------------|\n| Header | Title       |\n| Paragraph | Text     |\n```\n\n### Horizontal Rule\n```markdown\n---\n\nor\n\n***\n```\n\nThese examples cover some of the basic and slightly more complex structures you can create with Markdown.\n\n","behaviour":{"scroll":"end"}}

More details
When another code block is nested within a markdown code block, Alfred has trouble formatting the output.
Alfred's Markdown Syntax doesn't specify how nested code blocks should behave.
This is an excerpt from ChatGPT's response using a prompt similar to "Examples of Markdown":
### Multiple Code Blocks
```markdown
Here is a JavaScript example:

```javascript
function sayHi() {
    console.log("Hi!");
}
sayHi();
```

And here is a JSON example:

```json
{
    "name": "John",
    "age": 30,
    "isStudent": false
}
```
```

Current

Updated

Thank you for the submission and the heads-up. This is worth thinking about, though I’m not convinced of the approach. One important (but fixable) aspect is the code goes against the functional style of the whole script. Seeing as I’d have to maintain this code after merging and that this is in essence a hack (I don’t say that pejoratively), it’s particularly important that it follows the code style and be consistent with the rest of the logic.
While reflecting and experimenting with this, I bumped into an interesting observation. ChatGPT itself doesn’t do this correctly:

Nor does GitHub, for that matter. This (which I had to render render with HTML for demonstration purposes):

```markdown
# This is an h1 header

Some Python code

```python
def inner_function():
    print("Inner function")

inner_function()
```

Some more text

## An h2 header

More text here

```

Becomes:

Which on first glance makes me think this will probably be more complicated than it looks at first glance. Not because of ChatGPT, but GitHub puts a lot of effort into Markdown. And while I appreciate the effort on the fix, I also don’t think it’s the right approach to work around this only on this workflow, because the Text View is generic. It should either be done in Alfred as a whole, or not at all.
I’ll open an issue internally to discuss this, to determine how viable it is. But since you marked this as draft I wanted to let you know sooner rather than later so you wouldn’t spend too much time on it. Although, as always, you’re welcome to use your own changes in your own local installation, as long as those are disclosed when opening bug reports.
Thank you again for the submission, and especially for bringing this to my attention. Have a relaxed Sunday!


Rename _az_is_finite to _az_isfinite to match STL and update call sites.

Fixes https://github.com/Azure/azure-sdk-for-c/issues/983

You are doing renamings, can you please fix one other thing? I think it can be done within this PR.

Fixed that separately: https://github.com/Azure/azure-sdk-for-c/pull/1013


Trigger image selection from javascript frontend

Hi noam148, is there any way to trigger the image selector in frontend, with javascript/jquery.. for example with a button calling your widget from server (controller action)? I'm sticking in a landingpage generator (which composes elements on frontside in browser) and I would like to access the uploaded images from your extension. I would need to have either a javascript script rendering the field with selector or a button calling it, but have absolutely no idea on how I could handle this.... any input or help will be really apreciated! Many thanks in advance, Oliver

Hi @gasserol, it is not clear for me what you mean. Can you explain it to me more clearly, maybe with screens or something like that?

Hi noam148, thanks for replying me. I want to make a cms-similar site with drag-and-drop-Editor.

on the Clientside everything is jQuery. on top is a input type=file field, which opens up the normal desktop  file upload.  on bottom is a button which I use to open already imagemanager/manager/index via a view form rendered by an action in controller.
The question is: do you have any idea on how I could render the input field of your widget directly in jQuery not to have to pass over various modals? I would like to get the path of the selected image in imagemanager directly in a input field and then with jquery update the image (as it is done with the normal file uploader from desktop. Many thanks for your precious help, Oliver

Hi Oliver,
So if i'm correct you want the follow:

On click "change photo": select a foto with the file upload browser and do a file post to ?r=imagemanager/manager/upload

On click "photo noam": open imagemanager module and select a photo. And after select a photo, you want get the full image path back?

If this is what you want, I thick with some tricks it's possible. If you confirm this I can send you some instructions for how to.
One question, does the file upload work already or do you want also instructions for that?
Sorry for my late response, i'm very busy at the moment.

Hi noam148, thansk for your replying. First point is absolutely correct, this is what I would realize. Second point is also correct. I can open your imagemanager module in modal, this works, but when selecting a photo only its thumbnail path is given back, not the original file path stored in db.. and another 'ugly' thing is that if you open this in modal, then the imagemanager shows up, but image selection is in modal again, somehow modal inside modal .. if this could be bypassed without to have to modify too much you code, this would be very good..
By the way, if your time is too short for this for the moment, I'm working on other parts of my software, so please do not hurry on my request :-) Many thanks for your precious help.

Hi @gasserol,
Sorry for my late respone but finally I have some time again, and as I promised i searching for a solution see:
https://github.com/noam148/yii2-image-manager/files/949950/example.zip
I have test the code and everything work fine here. It's a stand alone site and it needs only:

jquery
Bootstrap
Bootstrap-file input (https://github.com/kartik-v/bootstrap-fileinput)

Luckily there was no need to modify the code of the yii2-image-manager.
You can download the zip file and test the code. let me know if there any questions!

Thanks @noam148 I will tryout this as soon as I have a little time, actually we have lots of customer work to do :-) many thanks anyway, I really appreciate. Will let you know. BR, Oliver

@gasserol please, re-open it, if you have any questions or need more help.


[Feat] Redesign UI of the Landing Page

The current font size of the website is small, it would be better if we increase it as it enhances the text readability and user experience.

@Vimall03

@poorvikaa08, Will you be able to do a complete redesign of the landing page (index.html file only)? If you are interested then I will assign it level 2.

/assign

Hi @Sarthakkashyapp,
Let's wait for a response from @poorvikaa08. If we don't hear back soon, I'll go ahead and assign the issue to you. In the meantime, feel free to open a new issue if you find something you'd like to contribute to.
Thank you!

Okay @Vimall03 I'll definitely look for other issues.

@Vimall03
I will create a separate index.html under new design. I’m ready to work on it.

@poorvikaa08 cool, I'll assign this issue to you. You can get started.

Please make sure the design is visually appealing. and the links for login and sign up is working.
for signup - href="user_signup.php"
for login -  href="user_login.php"

Hi @poorvikaa08, Have you made any progress ?

Closing this issue since @poorvikaa08 is inactive.

Hey @Vimall03 can you assign me this ?

Hi@Sarthakkashyapp, I have opened a new issue. Read the description of  #110  & comment on that issue if you are interested

@Vimall03 I've commented.


Doesn't work in diff editor view

viewColumn is undefined. Possible workaround is to create separate window for undefined view columns

Is there currently any workaround for this? It's the main thing that's keeping me using the other vim extension, since I use the diff editor extensively.

Unfortunately there is no workaround here, but i'm working on supporting diff/other composite views right now.


Relax Python version to "python3"

Hi there,
Python 3.7 was not installed on my machine and I believe it might be fine to just say python3 in the dependencies instead of python3.7. If something fails on that, people using this will probably have to go to pyenv anyway.
With kind regards,
Andreas.

blocked on https://github.com/crate/crate-docs/pull/36

@amotl this should be changed in crate-docs-theme as well, as the two projects work in tandem
https://github.com/crate/crate-docs-theme/blob/master/setup.py
see "classifiers", "python_requires"

https://github.com/crate/crate-docs/pull/36 is merged. if you rebase on master, the tests should pass now

oh, wait. there is one more thing to consider:
https://github.com/crate/crate-docs/blob/master/.github/workflows/docs.yml
so, atm, we specifically test using python3.7. this makes it easy for us to say that the docs build is supported for python3.7 and above
what would you suggest we do with the tests if we're removing the specificity of python 3.7? there are some pretty important differences between python 3.6 and 3.7 that have bitten me before. should we modify the GitHub tests to specifically test the docs build on the lowest version of python 3 available?
I think that if we're dropping the ".7" part, then we should be able to say that we work on python3.0, or else we should probably figure out a way of testing for the presence of different python versions and then use the highest 3.x version that can be found on the host os
I would say, we could drop the python version to 3.6, or whatever is the lowest python 3.x version that works. but that doesn't solve the issue as you reported it: namely, we still depend on a specific minor version being installed. which is why I am thinking that we could test for available versions and use the highest
thoughts?

Hi Naomi,

This should be changed in crate-docs-theme as well, as the two projects work in tandem. There is python_requires=">=3.7", see [1].

I am fine with that requirement.
Regarding your other concerns: I well hear you. Whether it would justify any other amendments to the code base is currently beyond what I am able to grasp. I believe there will be Python>=3.6 on many systems around these days and if anything breaks, we might adjust by putting a check into the main Makefile which asserts a specific minor release like >=3.7.
So, the constraint within crate-docs-theme is just fine from my perspective and we should keep it like that without worrying whether the stack could work on Python 3.6.
With kind regards,
Andreas.
[1] https://github.com/crate/crate-docs-theme/blob/master/setup.py

Dear Naomi,

We might adjust by putting a check into the main Makefile which asserts a specific minor release like >=3.7.

I've just amended the patch to add this idea. Let me know what you think about it.
With kind regards,
Andreas.


[Issue]: NVM installation failure inside Azure VM

What happened?
I have followed below Github to install the NVM
https://github.com/coreybutler/nvm-windows
and for that I created VM with windows server 2022, both VM and OS are 64 bit but not sure why it is giving following error
Here are the steps I have followed :

I have ran below command as per your instruction

however when I navigate to that file and execute giving following error:

and Deleted and downloaded again, it is same error. Please suggest.
What did you expect to happen?
I have followed below Github to install the NVM
https://github.com/coreybutler/nvm-windows
and for that I created VM with windows server 2022, both VM and OS are 64 bit but not sure why it is giving following error
Here are the steps I have followed :

I have ran below command as per your instruction

however when I navigate to that file and execute giving following error:

and Deleted and downloaded again, it is same error. Please suggest.
Version
1.1.11 or newer (Default)
Which version of Windows?
No response
Which locale?
None
Which shell are you running NVM4W in?
No response
User Permissions?
Administrative Privileges, Elevated
Is Developer Mode enabled?
None
Relevant log/console output
Please see above s

Debug Output
No response
Anything else?
Please see above description

That curl command doesn't look correct. I think you want curl -O https://github.com/coreybutler/nvm-windows/releases/download/1.1.11/nvm-setup.exe
Check the size of what was downloaded. If the executable isn't at least 5MB, then you're just not getting the download.
Windows has a habit of blocking .exe downloads, so you might want to try with the zip file too.

Hi Core Bulter,
I tried as per your suggestion, seems to be above error is gone, but came up with following error  and when clicked on close I see "access denied" please suggest further

Please suggest further, thank you.

Hi Core Bulter,
I tried as per your suggestion, seems to be above error is gone, but came up with following error  and when clicked on close I see "access denied" please suggest further

Please suggest futher, thank you.

When access is denied, try running it again a few times. Windows may block unknown executables the first time they're downloaded. It's part of Windows' annoyingly intense security. If the app is continually denied, you may need to make sure Windows Defender isn't blocking it.


Merge existing JSDoc comments

This PR is a followup to the infer-from-usage codefix PR. Instead of generating comment text in textChanges.ts, it uses the existing infrastructure in the emitter and formatter. That means that:

The emitter can now emit JSDoc.
The formatter is slightly better (probably) at emitting multiline comments. Because I didn't want to take the time to completely understand and restructure the formatter, I left the existing code for single-line comments intact.
The factory functions now include a few JSDoc tags, but only the ones I needed for this PR. It's easy to add these, so I think it's fine to leave them out.

It additionally means that
4. inferFromUsage.ts constructs JSDoc tags instead of its own data structures.
As a result, you'll see a lot of new, very predictable code in the emitter and lots of deletes in textChanges.ts.
I also wrote code to merge with existing JSDoc comments. The biggest change to existing comments will be the merging of multiple JSDoc comments into one, eg
/** @param x one */
/** @param y two */

Will become
/**
 * @param {number} x one
 * @param {number} y two
 */

@amcasey you are not required for this review, but your comments, combined with @andy-ms' noImplicitThis codefix, led to it, so I thought you'd be interested.

I skimmed it and it seems sensible.  Thanks!

In reply to: 431172674 [](ancestors = 431172674)

I think this is ready for another look now.

Here's my take on improving jsdoc comment merging:
function annotateJSDocParameters(changes: textChanges.ChangeTracker, sourceFile: SourceFile, parameterInferences: ReadonlyArray<ParameterInference>, program: Program, host: LanguageServiceHost): void {
    const signature = parameterInferences.length && parameterInferences[0].declaration.parent;
    if (!signature) {
        return;
    }
    const paramTags = mapDefined(group(parameterInferences, i => String(getNodeId(i.declaration))), inferences => {
        const param = first(inferences).declaration;
        // only infer parameters that have (1) no type and (2) an accessible inferred type
        if (param.initializer || getJSDocType(param) || !isIdentifier(param.name)) return;

        // TODO: improve combining inferences
        const type = firstDefined(inferences, i => i.type);
        const isOptional = inferences.some(i => !!i.isOptional);
        const typeNode = type && getTypeNodeIfAccessible(type, param, program, host);
        return typeNode && createJSDocParamTag(createJSDocTypeExpression(typeNode), param.name, isOptional, "");
    });
    addJsdocTags(changes, sourceFile, signature, paramTags);
}

function addJsdocTags(changes: textChanges.ChangeTracker, sourceFile: SourceFile, parent: HasJSDoc, newTags: ReadonlyArray<JSDocTag>): void {
    const comments = mapDefined(parent.jsDoc, j => j.comment);
    const oldTags = flatMap(parent.jsDoc, j => j.tags);
    const unmergedNewTags = newTags.filter(newTag => !oldTags || !oldTags.some((tag, i) => {
        const merged = tryMergeJsdocTags(tag, newTag);
        if (merged) oldTags[i] = merged;
        return !!merged;
    }));
    const tag = createJSDocComment(comments.join("\n"), createNodeArray([...(oldTags || emptyArray), ...unmergedNewTags]));
    changes.insertJsdocCommentBefore(sourceFile, parent, tag);
}

function tryMergeJsdocTags(oldTag: JSDocTag, newTag: JSDocTag): JSDocTag | undefined {
    if (oldTag.kind !== newTag.kind) return undefined;

    switch (oldTag.kind) {
        case SyntaxKind.JSDocParameterTag: {
            const oldParam = oldTag as JSDocParameterTag, newParam = newTag as JSDocParameterTag;
            return isIdentifier(oldParam.name) && isIdentifier(newParam.name) && oldParam.name.escapedText === newParam.name.escapedText
                ? createJSDocParamTag(newParam.typeExpression, newParam.name, newParam.isBracketed, oldParam.comment)
                : undefined;
        }
        case SyntaxKind.JSDocReturnTag:
            return createJSDocReturnTag((newTag as JSDocReturnTag).typeExpression, oldTag.comment);
        default:
            return undefined;
    }
}

addJSDocTags replaces insertMergedJSDoc with no map or callbacks needed. The caller just passes in the new tags as an array and the function is responsible for looking for matching existing tags and merging them, so it only needs 4 parameters. It also avoids modifying existing tags unless a new tag matches it. This is technically O(oldTags * newTags) but I expect the number of new tags is rarely more than 10.
This does break the test codeFixInferFromUsageMultipleParametersJS, but that test has two different parameters with the same name so there's no reason we should merge their tags.

I like your suggestion, although I note that the group(parameterInferences, i => getNodeId(i.declaration), ...) doesn't make much sense, since each parameter maps to exactly 0 or 1 parameterInference. I think you can mapDefined(parameterInferences, ...) directly. I'll try that.


feat(Product): Show only 3 first tags

This PR has to goal to display only the first three tags in the cards of the product page.
Current Behaviour : Content cards in the product page show all tags that the contributor / curator has added. However, when the content card has more than 5 tags its hard to read the rest of the card.
Desired Behaviour : We should only show 3 tags on the card. If the content card has more than 3 tags, the third tag should be truncated to show just the number of hidden tags. Sample UX which shows behaviour

I added a function to sort the tags in the cards so that the tags selected in the filters are always displayed first in the cards. Because, sometimes the selected tag is hidden in the +X tag, so the user can't understand why some results appear and don't match the current filter.


add to ccm

👷 Deploy Preview for fancy-puffpuff-df775f processing.

Name
Link

🔨 Latest commit
b9fd07f63bb082c636a84cb843541258a98c9e32

🔍 Latest deploy log
https://app.netlify.com/sites/fancy-puffpuff-df775f/deploys/65a9296560b6390008e72fdd


VS Code

Extension Name: php-intellisense
Extension Version: 2.3.5
OS Version: Windows_NT x64 10.0.17134
VSCode version: 1.28.0

We have written the needed data into your clipboard. Please paste:

{
  "activationTimes": {
    "startup": false,
    "codeLoadingTime": 128,
    "activateCallTime": 10,
    "activateResolvedTime": 293,
    "activationEvent": "onLanguage:php"
  }
}

Please provide a description of the issue


New package: kurogame.PunishingGrayRaven version <IP_ADDRESS>

[ ] 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.4 schema?

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

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110762&drop=dogfoodAlpha

@wingetbot waivers Add Validation-No-Executables

This package appears to be a tray-based application that pops up into an always on top window, despite having no executables detected.

this was not supposed to be the program

https://drive.google.com/file/d/1sj2yyDZYNzmtuQBdBU_5-GgwseDMr55s/view?usp=sharing this was supposee to be the app

The new Math panel above confused me before. This package appears to do nothing when given the /S switch, which might seem similar to the Nullsoft silent install. But with any other switch combination, it launches an interactive install:


add a consistent close window key fun for all windows

pretty much always want to be able to do this via keyboard.
btw Ctrl+Return does close dialog-type windows with an accept action..

A somewhat annoying thing is that close window is in the File menu, not the app menu -- we are fully auto-generating the app menu so it would be convenient to add it there.
also, I noticed that chrome uses command-W for close tab -- have to add shift to close window..  so people might want to customize this..
going to leave with Ctrl+Return for time being.


Enable syntax highlighting in f-strings

Makes syntax displayed inside fstring highlight as if it were not.

Also allows nested f strings i.e f"{f'{1+1}'}"

Thanks!


NetworkManagerImpl#getReservedPortMappings works wrong

this method does not parse mappings correctly and returns empty list

How to test:
in Console->Advanced section in Karaf tab run command container-ports,
Result: should print available mappings similar to output of subutai map ls


WIP 2502 mongo now

A not so ideal implementation of now() in agg. pipeline.
Just putting this up for discussion.
There is a bson representation that is a backend managed now() - Bson.JavaScript(Js.New(Js.Call(Js.Ident("Date"), List()))) - but its usage is quite limited. E.g. select fieldA, now() from tableA does not work, nor does select ts - now(), now() - ts from days
I also tried using $let tricks such as below, but this has the same limitations.
val jsNewDate = Bson.JavaScript(quasar.javascript.Js.New(quasar.javascript.Js.Call(quasar.javascript.Js.Ident("Date"), List())))
Bson.Doc("$let" -> Bson.Doc(
       "vars" -> Bson.Doc("now" -> jsNewDate),
       "in"   -> Bson.Text("$$now"))
)

I see another backend-managed now solution that may work, but it's a long shot. Not sure if it's worth trying... In a nutshell: we can try writing (ugh) now() via https://docs.mongodb.com/manual/reference/operator/update/currentDate/ and then looking up that value.
Alternatively, we can calculate now from quasar and then pass that to mongo, but this has the problem of clock skew. Anyway, this PR does that in a quick and dirty way (with some additional problems that 1) it's not referential transparent 2) may be different for each occurrence of now()).
I am hoping someone sees a way that doesn't involve too much refactoring to provide this value for now once, so that we can solve issue 1) and 2) (adding some kind of evaluation context maybe?).

I don't think Javascript is the right approach. Won't that cause a query that uses NOW() to be run as map/reduce? Javascript can't run in the agg pipeline. Or at least couldn't in older versions. Also it's important to have the guarantee that NOW() in one part of a query matches NOW() in another part — i.e. referentially transparency. Better to spend a little extra time developing machinery to just propagate the "query start time" date/time into the QScript or whatever is required to solve this without relying on MongoDB. Backends that provide good support for NOW() can use a different approach.

@jdegoes - thanks! Somehow my mind was glued to solving it in mongo connector, but I didn't think of solving this in QScript. Now, my mind is unglued somewhat, I'll try moving Now from MapFuncCore to MapFuncDerived and give it a default implementation in  QScript.
BTW the Javascript I was talking about above does avoid map/reduce - it is a part of bson that I stumbled upon, see type number 13 here: https://docs.mongodb.com/manual/reference/bson-types so it generates an agg pipeline query, but yeah runtime it results usually in an error (though not always).


Define an XSD schema to validate the configuration file

An XSD schema should be define and used to validate the XML configuration file. It would avoid any error while reading unvalid file.

closed by #75


SOLR-15337: Avoid XPath in solrconfig.xml parsing (take 2)

@madrob
I opened a new PR here to address your review comments

This issue https://issues.apache.org/jira/browse/SOLR-15337 has three main-branch PRs #104 , #160 and this. The JIRA and both the other PRs are merged. Is this last PR supposed to be merged as well or can it be closed?


Implementation of DEX exchanges ?

2018 will be the year of development of DEX exchanges.
The decentralized nature of DEX make more difficult to call API.
XChange and other will adapt their code to call smart contract ( ETH / NEO / EOS and others ).
How will XChange handle this major change in a near future ?
That's just an open discussion.

there's not much example code in the java space for ethereum contracts but if it helps, this is an Idex implementation https://github.com/jnorthrup/XChange/tree/swagger-gen/xchange-idex/src/main/java/org/knowm/xchange/idex

#2544 is available for review

Hi, anyone  had any more thoughts on the DEX integration, @jnorthrup example looks like it is leveraging a rest api, so akin to exchange today. Some smart contracts wrap solidity with javascript, anyone had much experience using the  javax.script package  to wrap the javascript api's?