Update rustls to 0.23 and quinn to 0.11

Need the rustls-platform-verifier 0.4.0 for CA cert support on Android/iOS.

Thanks, these updates were long overdue.
I updated a few things on your PR:

I kept RustlsServerName in ServerName. Did you have a strong reason to move to String ?

I also changed the configure_client implementation for the CertificateVerificationMode::SignedByCertificateAuthority case.
The call to install_default in the following snippet
let _ = rustls::crypto::CryptoProvider::install_default(rustls::crypto::ring::default_provider());

was bothering me. Seeing the rustls documentation, libraries should not set the default per-process crypto provider and instead use the one set once by the application or specify a custom one.
I decided to specify the usual ring default provider rustls::crypto::ring::default_provider(). In order to do this, rustls-platform-verifier 0.4 was added as a direct dependency since Quinn's helper ClientConfig::with_platform_verifier does not let us directly set the provider and instead relies on the per-process one.

I kept RustlsServerName in ServerName. Did you have a strong reason to move to String ?

No strong reason, other than to avoid the new need for a lifetime parameter because of the 'static lifetime on DnsName member of ServerName. I just wanted to play it safe so opted to generate the CertStore key up front.

I decided to specify the usual ring default provider rustls::crypto::ring::default_provider(). In order to do this, rustls-platform-verifier 0.4 was added as a direct dependency since Quinn's helper ClientConfig::with_platform_verifier does not let us directly set the provider and instead relies on the per-process one.

Actually I have a PR awaiting final review for rustls-platform-verifier to update the jni dependency to 0.21 so hopefully it will be 0.5.0 in the next few days. And I would actually prefer the ability to use it without waiting for quinn to upgrade to it.

Great. I will merge this and keep an eye on the rustls-platform-verifier release


Tutorial adding command

Codecov Report

Merging #4 (b3fe124) into master (1cc4863) will decrease coverage by 0.07%.
The diff coverage is 69.73%.

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
- Coverage     72.15%   72.08%   -0.08%
- Complexity      399      415      +16
============================================
  Files            70       73       +3
  Lines          1232     1293      +61
  Branches        125      132       +7
============================================
+ Hits            889      932      +43
- Misses          311      325      +14
- Partials         32       36       +4

Impacted Files
Coverage Δ
Complexity Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)
8.00 <0.00> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
12.00 <0.00> (ø)

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <0.00%> (ø)
1.00 <0.00> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

.../seedu/address/logic/parser/EditCommandParser.java
85.71% <33.33%> (-6.60%)
11.00 <0.00> (ø)

.../java/seedu/address/storage/JsonAdaptedPerson.java
95.91% <66.66%> (-4.09%)
11.00 <0.00> (ø)

...c/main/java/seedu/address/model/person/Remark.java
75.00% <75.00%> (ø)
4.00 <4.00> (?)

...java/seedu/address/logic/commands/EditCommand.java
97.22% <100.00%> (+0.20%)
12.00 <0.00> (ø)

...va/seedu/address/logic/commands/RemarkCommand.java
100.00% <100.00%> (ø)
10.00 <10.00> (?)

... and 8 more

Continue to review full report at Codecov.

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

Tutorial PR irrelevant to project.


Removing UIKitView when dragging and scrolling from CustomScrollView breaks the scrolling animation

When by any chance we drag the UIKitView when scrolling a CustomScrollView and then we hide UIkitView tends to break the whole scroll animation of the CustomScrollView widget.

Steps to Reproduce

Clone the test project from the following repo
Run in an iOS device and slowly start scrolling down until see the UIKitView (the red rectangle)
Scroll and drag it until it vanish.
Check the whole scroll animation gets laggy.

Target Platform: iOS
Devices: All iOS devices
Flutter Doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.26.0-18.0.pre.187, on macOS 11.2 20D64
    darwin-arm, locale en-DO)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] Connected device (1 available)

• No issues found!

Was able to replicate it on latest stable. Dragging the red rectangle makes the scrolling animation slow and laggy.

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.6, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-IN)
    • Flutter version 1.22.6 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 9b2d32b605 (13 days ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (1 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 14.2

! Doctor found issues in 1 category.

cc @cyanglaz  - sounds like the gesture issue you looked at a while back

The fix has rolled into the framework at https://github.com/flutter/flutter/commit/b957144136c36f189f00876b71379a0a7cae7efc
Please try out Flutter on master see if it resolves the issue :)


APT whitelist request for checkstyle + dependencies

Some dependencies are already whitelisted, here's the full list:
The following NEW packages will be installed:
  antlr checkstyle java-wrappers libantlr-java libapache-pom-java
  libcommons-beanutils-java libcommons-cli-java libcommons-lang-java
  libcommons-logging-java libcommons-parent-java libgoogle-collections-java
  libjsr305-java

Thanks for the PR. Please follow the approval process outlined here: https://github.com/travis-ci/apt-package-whitelist#package-approval-process

@BanzaiMan I went through the process for checkstyle, although it was quite exhausting, there's nothing malicious.
What else do you want me to do? Are you merely looking for comment with confirmation that I've done it?
I guess I'm missing the point here.


License Request

Applying in hopes of advancing "Into to Computer Science" class with code with me plugin for PyCharm, applying on behalf of CTE Teacher.

@CyberZ3ro First stage of verification passed, official domain is bend.k12.or.us as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Any updates?


Create an API for calculating the number bytes and shares used by compact and sparse data

As pointed out in this comment and this comment, it will be easier to maintain and modify share encoding code if we also have a canonical API for calculating the number bytes and shares that compact and sparse share use. There are many different uses of this estimation logic, so getting a perfect API or modifying the existing code to use such an API could take a few PRs. We should track them here.

We now have the following exported functions which handle the number of shares usecase:
https://github.com/celestiaorg/celestia-app/blob/0b447adf776689cc8f14823c6ef925b8f8249b5a/pkg/shares/share_sequence.go#L101-L103 https://github.com/celestiaorg/celestia-app/blob/0b447adf776689cc8f14823c6ef925b8f8249b5a/pkg/shares/share_sequence.go#L80-L83
I don't see an immediate need for a separate API that returns number of bytes but it would be easy to implement (number of shares needed * share size) so proposal to mark this as resolved for now? @evan-forbes


Change CuArrays type from Float64 to Float32

This PR aims obtain a better performance on GPU by using CuArrays with Float32 type.

 Performance test
Google Cloud VM:

n1-standard-2 (2 vCPUs, 8gb memory)
1 x NVIDIA Tesla T4 (details)

code:
using ImageQuilting

TI = rand(100, 100, 100)
tilesize = (10, 10, 10)

@time ImageQuilting.iqsim(
  TI,
  tilesize
)

Result:

using Float64: 33.592089 seconds (18.75 M allocations: 40.698 GiB, 3.04% gc time)
using Float32: 21.736202 seconds (18.95 M allocations: 28.386 GiB, 4.11% gc time)

Codecov Report

Merging #46 (78887c7) into master (6166ef6) will not change coverage.
The diff coverage is 0.00%.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


PhantomJs requires flags for HTTPS sites

Hey there,
I tried implementing this gem, but my site uses ssl encryption and PhantomJs requires the following flags to handle ssl sites (courtesy of Penthouse):
to run on HTTPS sites two flags must be passed in, directly after phantomjs in the call:
    --ignore-ssl-errors=true --ssl-protocol=tlsv1

I've tried digging through the PhantomJs gem but can't seem to find a solution for this, any thoughts?

Little more detail on this, here's the error that's thrown:
Error opening url 'https://www.example.com/': SSL handshake failed

I've added the recommended flags for PhantomJS via #16. I'll close this issue for now, but please feel free to reopen if the problem persists.

The problem persists on Heroku:
remote: -----> Preparing app for Rails asset pipeline
remote:        Running: rake assets:precompile
remote:        Phantomjs does not appear to be installed in /app/.phantomjs/1.9.8/x86_64-linux/bin/phantomjs, installing!
remote:        % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
remote:        Dload  Upload   Total   Spent    Left  Speed
remote:        0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
remote:   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
remote:        19 12.5M   19 2539k    0     0  14.7M      0 --:--:-- --:--:-- --:--:-- 14.7M
remote: 100 12.5M  100 12.5M    0     0  42.7M      0 --:--:-- --:--:-- --:--:-- 80.5M
remote:        Successfully installed phantomjs. Yay!
remote:        Removed temporarily downloaded files.
remote:        Error opening url 'https://example.com/': SSL handshake failed
remote:        Asset precompilation completed (8.04s)

It would be really nice to have it fixed because there is still no reliable way to activate mod_pagespeed on heroku!

@darmenise , I just tried this on one of our Heroic instances, and here are my results:
remote:        % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
remote:        Dload  Upload   Total   Spent    Left  Speed
remote:   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
remote:        100 12.5M  100 12.5M    0     0  32.7M      0 --:--:-- --:--:-- --:--:-- 32.7M
remote:        Successfully installed phantomjs. Yay!
remote:        Removed temporarily downloaded files.
remote:        Cache write: critical-path-css:/ ({:namespace=>"critical-path-css", :expires_in=>nil})
remote:        Dalli::Server#connect XXXXXXX.com:11211
remote:        Dalli/SASL authenticating as 6a59e1
remote:        Dalli/SASL: 6a59e1
remote:        Asset precompilation completed (7.07s)```

In my `critical_path_css.yml`, I'm using a `base_url` with a known working SSL certificate. Are you using the most up to date version of this gem, `0.3.1`?

One thing I still need to do is update PhantomJS, but barring that, I'm currently unable to replicate the SSL handshake issue.

I'm using the edge version actually. I'm also using a custom domain and Cloudflare, maybe something goes wrong there... But the SSL certificate is definitely working:

If you want I could privately give you more details.

Update: if I don't use the custom domain the SSL handshake is successful.
In other words if i put https://appname.herokuapp.com as base_url for production it works.
Maybe the problem is in some Cloudflare configuration..

Just tried your fix @darmenise and it worked for me as well. My company uses GoDaddy for our domain names, so maybe it's something to do with Heroku's SSL implementation. Regardless, using the base Heroku url seems to fix the issue!

That being said, I'm fine with the issue being closed if you're happy with the solution @michael-misshore

I'm glad to know that :-)
I'm still in trouble because my app uses subdomains and so I can't use that fix but it would be for another issue. This one I think it could be closed too.

Closed!


the problemuse hamachi to create the server

When i use my Intranet IP to create the server,the DarkSoul3 can connect to the server.
Then I tried to use Hamachi create a LAN,and modify the value of 'ServerPrivateHostname' to the virtual IP assigned to me from Hamachi,restart the server.
Then the DS3 can not connect to the server.

when i use hamachi,the server log is like

loginServer : 'xxx.xx.xx.xx' Client connected.
a few seconds later
loginServer : 'xxx.xx.xx.xx' Client disconnected.


Task/english content

Resolves #482
Resolves #483
Resolves #484
Resolves #485
Resolves #486
Resolves #487
Resolves #488
Resolves #489
Resolves #490
Resolves #491
Resolves #492
Resolves #493
Resolves #494
Resolves #495
Resolves #496
Resolves #497
Resolves #498
Resolves #499
Resolves #500
Resolves #501
Resolves #502
Resolves #503

(so basically all the English content)
Currently a draft because there are some dead keys I need to remove

As an update, @YedidaZalik and I talked and we're going to merge this (assuming it looks good to you @nmakuch and @pcraig3 ). Any updates we'll log as new issues :)


Perfer gradle and add support for multi-branch testing.

Most of the code here is copied and pasted from other plugins, such as beats input.

@guyboertje - i needed to make some more changes, mind taking another look ?

Merging on @jakelandis behalf
Going to rebase PR 19 on top of this.

@guyboertje - thanks!


EVP_MAC_size()

We seem to have an EVP_MAC_size() function:
size_t EVP_MAC_size(EVP_MAC_CTX *ctx);

However rather than taking an EVP_MAC parameter it takes an EVP_MAC_CTX.
Shouldn't the function be called EVP_MAC_CTX_size()? Or is this deliberate?

I gotta wonder why this function got singled out...  but I'll play.
I can't really remember the exact reason, but my immediate intuition now says that "the size of the MAC" is more naturally spelled EVP_MAC_size() than EVP_MAC_CTX_size().
What I don't want to see is both EVP_MAC_size() and EVP_MAC_CTX_size()...  that's just confusing.

I gotta wonder why this function got singled out... but I'll play.
I can't really remember the exact reason, but my immediate intuition now says that "the size of the MAC" is more naturally spelled EVP_MAC_size() than EVP_MAC_CTX_size().

Virtually all of our functions start with the name of the object it operates on. Notable exceptions are the init/update/final functions. Aside from those, AFAICT, all the other EVP_MAC_* functions do conform to that convention - making this one stick out as being the odd one out. I tripped over this when trying to find the size of the mac given an EVP_MAC_CTX. Grepping the source failed to find EVP_MAC_CTX_size() as I expected...

Okie, than change it.  You should do the same for EVP_KDF_size(), then

Needs to be renamed for 3.0: EVP_MAC_CTX_get_mac_size().  Likewise for the KDF.

EVP_KDF_reset() also renamed..

Could we instead remove EVP_MAC_size()? The size is returned as part of the final EVP_MAC_final().
The same would be true of EVP_MD_size() and EVP_DigestFinal_ex().

Do you mean completely remove it? I do not think it would be nice idea. In an application I might want to know a size of particular MAC (or MD) before I actually try to create one.


[Fix] Fix wrong index in splice

Caused errors + not working correctly most of the time with some invites

I thought i had merged this lol


Change update to getState

In thunks, we do not know the current state because maybe some other actions has changed the state during the thunk. So you offered an update function.
But a better solution is offer a getState function...

@xialvjun The update function can take a function that receives the state. We find this method a small improvement to React/Redux.
For example, when you don't need the state to calculate the next state.
update({ count: 1 })

And if you need the state to compute the next state.
update(state => ({ count: state.count + 1 }))

Does that make things clear?

But if the state has been changed during the thunk, then state.count is wrong.

@xialvjun Right, that problem is solved using:
update(state => ({ count: state.count + 1 }))

The state passed to the inner thunk above is always up-to-date.

@xialvjun In other words:
const actions = {
  upAsyncNaive(state) { // Hey! state may be out-of-sync here!
    return update => update({ count: state.count + 1 })
  },
  upAsyncSmart(state) { // No problem here!
    return update => update(state => ({ count: state.count + 1 }))
  }
}

got it. thanks very much.

Glad you figured it out! 🎉


Update: description

Codecov Report

Merging #417 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #417   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         150    150
  Lines        1481   1481
=====================================
  Hits         1481   1481

Continue to review full report at Codecov.

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


Found some more errors

I also get some graphical errors with the texture loading with SixLabors.ImageSharp on Linux/Mono. I tried just changing the Texture.cs class to use System.Drawing to load the texture instead and the errors disappeared. Perhaps that would be better to use in the tutorial since it is also one dependency less.

cc: @devvoid

Either one is just a stopgap for the moment, neither are a great solution. Still, with ImageSharp's licensing problems, this is good for now.


Support for tfupdate

Integrate tfupdate
Run https://github.com/minamijoyo/tfupdate/ on commit to pin the version of terraform and providers.
That will allow users to pin versions without efforts, at least for terraform itself and the providers.
How could pre-commit-terraform help solve your problem?
I have a simple PR : https://github.com/minamijoyo/tfupdate/pull/63 but could rewrite it to be a bit more dynamic, ie detects the provider and update, overall it does the job.
If you are interested I can apply it to this repo.

Hi @jrottenberg
Yeah, it will be useful.
Check out https://github.com/antonbabenko/pre-commit-terraform/blob/master/.github/CONTRIBUTING.md as a start point and feel free to ask if you'll have any questions.


vrc-get resolver as a alternative of vpm-resolver

this would be a amazing feature

I've started making this at https://github.com/anatawa12/vrc-get-resolver. It's working however not yet ready for release.

it looks vrc-get-resolver is working so I mark this as completed.


"go get" broken out of the box

A colleague is attempting to install with "GO111MODULE=on go get github.com/kyleconroy/sqlc/..." and getting Offset syntax errors

Here's what we see when we try to do it
$ go install github.com/kyleconroy/sqlc/cmd/sqlc
# github.com/lfittl/pg_query_go/parser
pg_query_parse_plpgsql.c: In function ‘pg_query_parse_plpgsql’:
pg_query_parse_plpgsql.c:437:10: warning: ignoring return value of ‘asprintf’, declared with attribute warn_unused_result [-Wunused-result]
  437 |    (void)asprintf(&new_out, "%s%s,\n", result.plpgsql_funcs, func_json);
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# github.com/kyleconroy/sqlc/internal/engine/dolphin
go/src/github.com/kyleconroy/sqlc/internal/engine/dolphin/convert.go:302:28: n.OldTable undefined (type *"github.com/pingcap/parser/ast".RenameTableStmt has no field or method OldTable)
go/src/github.com/kyleconroy/sqlc/internal/engine/dolphin/convert.go:303:29: n.NewTable undefined (type *"github.com/pingcap/parser/ast".RenameTableStmt has no field or method NewTable)

Duplicate of https://github.com/kyleconroy/sqlc/issues/654


feature request

First of all thanks for the effort of making such useful tool and god bless you.
Second I have a request of a feature
Description of feature:
It would help a lot to include LoLalytics ARAM builds into ARAM mode/map. As well as metasrc.com for both ranked 5v5 and aram.
Thank you

Dup.
https://github.com/dustinblackman/Championify/issues/233
https://github.com/dustinblackman/Championify/issues/358


Minify Truncating CSS

We have a copy of the 3.0.3 version of minify running on a test site where we are doing some work and it appears that minify is truncating the CSS of the main style.css file of the WordPress stock twentyseventeen theme for some reason.
The test environment is running PHP 7.0.26, no NFS, no concatenation (single file)
Truncated CSS - http://testsite.vrazer.net/wp-content/mu-plugins/wordkeeper-speed/minify/?f=/wp-content/themes/twentyseventeen/style.css
Source CSS - http://testsite.vrazer.net/wp-content/themes/twentyseventeen/style.css
This appears to be a bug in the latest version of Minify as alterations in the config file do not change anything.  No errors are logged in PHP.  Debug mode seems to alleviate the problem, but I suspect that is because it's not actually doing minification in debug mode.
Is there any fix for this?

I don't have anything to do with the WP plugin, sorry.

Hi Steve,
Just curious, did you actually read the bug report or just see the word
“WordPress” and assume that this had something to do with a plugin?  I
never mentioned a plugin - namely because this bug report has nothing to do
with a plugin.
This is a stock install if your Minify library with no other code
involved.  The only thing this even has to do with WordPress is that we
provided a CSS file to your minifier that is part of a stock WP theme. CSS
is just CSS, though.  I could have named a CSS file from anywhere.  But
since the bug is only manifesting for me so far with this example, I wanted
to give you that information so you can see the problem with the
minification with a real css file and track down the problem.
On Thu, Dec 14, 2017 at 12:49 PM Steve Clay<EMAIL_ADDRESS>wrote:

I don't have anything to do with the WP plugin, sorry.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrclay/minify/issues/620#issuecomment-351801284, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAXiVeAYoiCOXQbNU_JQiGBjD03hXnCQks5tAW2bgaJpZM4Q7uiA
.

--
--
Lance Dockins
Minister of Magic
VRAZER, LLC
Office:<PHONE_NUMBER> <javascript:void(0);>
Cell:<PHONE_NUMBER> <javascript:void(0);>

Sorry I did misread. You want to file this with CSSmin or just try updating it via composer. It may already be fixed.


Folder icon inheritance for sub-folder/nested folders

Housekeeping

[X] I'm sure this issue is not a duplicate

Information:
Hi all,
I was playing around with custom folder icons as described here in the documentation: https://github.com/vscode-icons/vscode-icons/wiki/FineTuning
Everything works pretty well, but I was not able to find any information about how to inherit icons to subfolders.
I have following folder structure:

What I want to achieve is, that "audit", "message", "name.route" does all have the same icon as the parent folder "routes", but I don't want to explicitly add the names of the folders  in the vscode-icon configuration.
My current configuration looks like this:
{"icon": "interfaces", "extensions": ["routes"], "format": "svg"},
I tried to add "route" to the extensions array and hoped, that at least the folder "name.route" would take over the icon configuration, but it didn't do the trick.
Thanks in advance for your feedback :-)

Hi @koanplaned, unfortunately that's not currently supported.

Hi @robertohuertasm, thank you for the feedback


Update index.html

I have added some css in it
I wish you like it
Thankyou!

Sir is your repository participating in hacktoberfest 2021

pls tell


Allow filtering of free plans

Reason for change
We need to add free plan filtering that satisfy the following requirements:

[ ] No changes are required on the user end end (i.e. the list should be able to be updated without changing any attributes)
[ ] It’s a temporary shim until it’s supported by the API

Testing
Test the <manifold-plan-selector> component with the new platform attribute, and see if the plan list filters (free plans only—LogDNA, Mailgun, etc.)

Should we hide the plan selector on the left if there's only a single plan to show?

Should we hide the plan selector on the left if there's only a single plan to show?

I think not; this is a temporary change, and long-term consumers will almost always show multiple plans. So even if we hid it, we’d just un-hide it very soon

are we hiding paid products altogether?

No; that’s a function of manifold-marketplace. We could add that functionality, but for now our consumers aren’t using that component so it’s fine. They’re managing the product list, so the only thing they don’t manage will be plan lists (nor should they!)


Added safety check to PR workflow

Pull Request Checklist

[X] Make sure to read the contributor guide here.
[X] Make sure you are making a pull request against the staging branch.
[X] Make sure your pull request title matches the requested format.
[X] Make sure to lint, type-check, and run unit and API tests.

Description
Added safety check to PR workflow

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore(logs): logging  based on log level type

Signed-off-by: mittachaitu<EMAIL_ADDRESS>This PR fixes the logging based on the type of log level.
If log level is LOG_LEVEL_ERR  then code will redirect to stderr else redirect to stdout.

As per discussion, standard to be followed logging the long-running process is to redirect to the standard error(fd: 2) so that logs will persist upon restarts also.


pgdriver error codes as consts

Hey, thanks for this awesome lib!
I was wondering if it would be nice to have the pgdriver error code as consts, the same thing we have in the pgerrcode lib.  I think it would add more flexibility on how we can handle errors over there.  Let me know if it makes sense and I can open a PR if you think it's worth

That looks useful and we probably should just use that module instead of bundling it with Bun? pgx is one of the drivers officially supported by Bun so it should not be a problem...
Instead we could add more helpers like https://github.com/uptrace/bun/blob/master/driver/pgdriver/error.go#L24-L35, but nothing obvious comes to mind (except documenting it :)...

That looks useful and we probably should just use that module instead of bundling it with Bun?

I would agree on this one. I guess pgerrcode is already feature complete and you just need to document the error handling part somehow. Let me know if you need any help on this one

Added an example to https://bun.uptrace.dev/guide/drivers.html#pgdriver


Typescript

(https://github.com/justsomederpystuff/activitysdk/pull/1/files)
How to build:
npm install & npx tsc  to compile typescript to javascript. Change tsconfig if you want.
How to use:
You can use a bundler such as Vite to use node packages in web development.
Follow the documentation to test/build the website.
Example:
import ActivitySDK from "activitysdk"

const sdk = new ActivitySDK("886415887287066664", "client secret here");
sdk.on("READY", async () => {
  try {
    await sdk.commands.authenticate(["rpc", "identify"]);
  } catch (e) {
    console.error("couldn't authorize/authenticate: " + e);
  }
  await sdk.commands.setActivity(999, {
    state: "In a Group",
    details: "Competitive | In a Match",
    assets: {
      large_image: "numbani_map",
      large_text: "Numbani",
      small_image: "pharah_profile",
      small_text: "Pharah"
    },
    party: {
      id: "99996696-xd-rrr",
      size: [3, 6]
    },
    secrets: {
      join: "025ed05c71f639de8bfaa0d679d7c94b2fdce12f",
      spectate: "e7eb30d2ee025ed05c71ea495f770b76454ee4e0",
      match: "4b2fdce12f639de8bfa7e3591b71a0d679d7c93f"
    },
    instance: true
  });

  console.log(await sdk.commands.getGuild("851813327289581619"));
});

i forgot to remove the debugging console logs

i'll continue on the project using a different branch
once you merge this pull request make sure you reverse the last 2 commits


lxqt.lxqt-qtplugin: 0.12.0 -> 0.13.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/lxqt-qtplugin/versions.
Version release notes (from GitHub):
These checks were done:

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/c6381d7911d526753a3a9f2ef17dc34e

du listing: https://gist.github.com/bc019cd05de1faa42bc8d394d3390267

meta.description for lxqt.lxqt-qtplugin is: '"LXQt Qt platform integration plugin"'.
cc @romildo for testing.
This build is cached with Cachix. To use the cache follow these instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 5cf6e3aa6a366f5f1d47ee3ab68d458ea08204cc && nix-shell --pure -I nixpkgs=. -p lxqt.lxqt-qtplugin

Success on x86_64-linux (full log)
Attempted: lxqt.lxqt-qtplugin
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0
shrinking /nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0/lib/qt-5.11/plugins/platformthemes/libqtlxqt.so
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0/lib
patching script interpreter paths in /nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0
checking for references to /build in /nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0...
postPatchMkspecs
postPatchMkspecs
/nix/store/rdhk3pfn2f2fj1sha4v5l459cg8gb6ip-lxqt-qtplugin-0.13.0

Success on aarch64-linux (full log)
Attempted: lxqt.lxqt-qtplugin
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0
shrinking /nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0/lib/qt-5.11/plugins/platformthemes/libqtlxqt.so
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0/lib
patching script interpreter paths in /nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0
checking for references to /build in /nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0...
postPatchMkspecs
postPatchMkspecs
/nix/store/l35zmmcn2dac8cg7c217fj48jr5w2l78-lxqt-qtplugin-0.13.0

Failure on x86_64-darwin (full log)
Attempted: lxqt.lxqt-qtplugin
Partial log (click to expand)

trying https://github.com/lxde/lxqt-qtplugin/archive/0.13.0.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   125    0   125    0     0    224      0 --:--:-- --:--:-- --:--:--   224
100   126    0   126    0     0    168      0 --:--:-- --:--:-- --:--:--   168
100 26280  100 26280    0     0  20091      0  0:00:01  0:00:01 --:--:-- 11.2M
unpacking source archive /private/tmp/nix-build-source.drv-0/0.13.0.tar.gz
cannot build derivation '/nix/store/8a5j683azbadaw5bn96n2257pnxysyaf-lxqt-qtplugin-0.13.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/8a5j683azbadaw5bn96n2257pnxysyaf-lxqt-qtplugin-0.13.0.drv' failed


doc: translate zh/installation/ha-installation

What type of PR is this?
/kind documentation

What this PR does / why we need it:
translate zh/installation/ha-installation
Which issue(s) this PR fixes:
Special notes for your reviewer:

/cc @zhzhuang-zju


[FEAT] Configure dark mode for pages other than the landing page

This includes the wallet page and the wallet adapter page

/assign

/assign


Recommending several papers

Great job! I wish that I had seen this earlier.
I would like to recommend

When and why PINNs fail to train: A neural tangent kernel perspective
On the eigenvector bias of Fourier feature networks: From regression to solving multi-scale PDEs with physics-informed neural networks
Learning the solution operator of parametric partial differential equations with physics-informed DeepOnets
Long-time integration of parametric evolution equations with physics-informed DeepONets

Can I also recommend this paper on an Active Learning strategy for generating collocation points in PINNs?
Optimal Transport Based Refinement of Physics-Informed Neural Networks


Fixes MAISTRA-1167: RetryHostPredicateFilter in protocol_integration_test suite passes now

istio-proxy tests are passing with this change.


speed doesn't improve at all

Hi @NathanGithub2013, thank you for your feedback. Could you provide a bit more info? What have you tried and what how long did it take before and after upgrading DAPLink?

I had measure the time to download my file before and after the update and
it's 39 sec before and 36 sec after
Vào 15:54, Th 3, 11 thg 4, 2023 Carlos Pereira Atencio <
@.***> đã viết:

Hi @NathanGithub2013 https://github.com/NathanGithub2013, thank you for
your feedback. Could you provide a bit more info? What have you tried and
what how long did it take before and after upgrading DAPLink?
—
Reply to this email directly, view it on GitHub
https://github.com/microbit-foundation/DAPLink/issues/18#issuecomment-1502937002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7A4GL47KAMBSSSVRIUIUALXAUL4ZANCNFSM6AAAAAAWWLFYVI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the additional info @NathanGithub2013!
Wow, 39 seconds is quite long. Could you provide a bit more info about your test computer?
What kind of flashing did you perform? Was it copy and pasting a hex file into the MICROBIT drive? Or using the online editors with WebUSB?
Could you post the hex file here? (email attachments won't work, you'd have to add it to a comment directly from the GitHub website). It might be better if the file is first compressed into a zip file or similar.
What kind of computer specs do you have? Operating system version, etc.
Do you run any antivirus software of similar that could be interacting with the flashing?

Web usb
Vào 18:10, Th 3, 11 thg 4, 2023 Carlos Pereira Atencio <
@.***> đã viết:

Thanks for the additional info @NathanGithub2013
https://github.com/NathanGithub2013!
Wow, 39 seconds is quite long. Could you provide a bit more info about
your test computer?
What kind of flashing did you perform? Was it copy and pasting a hex file
into the MICROBIT drive? Or using the online editors with WebUSB?
Could you post the hex file here? (email attachments won't work, you'd
have to add it to a comment directly from the GitHub website). It might be
better if the file is first compressed into a zip file or similar.
What kind of computer specs do you have? Operating system version, etc.
Do you run any antivirus software of similar that could be interacting
with the flashing?
—
Reply to this email directly, view it on GitHub
https://github.com/microbit-foundation/DAPLink/issues/18#issuecomment-1503135472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7A4GL6P5DSPCRBX26K5GFTXAU33XANCNFSM6AAAAAAWWLFYVI
.
You are receiving this because you were mentioned.Message ID:
@.***>

MBP 2011 MacOS 10.15
Vào 21:24, Th 3, 11 thg 4, 2023 Ralph Nathan @.***>
đã viết:

Web usb
Vào 18:10, Th 3, 11 thg 4, 2023 Carlos Pereira Atencio <
@.***> đã viết:

Thanks for the additional info @NathanGithub2013
https://github.com/NathanGithub2013!
Wow, 39 seconds is quite long. Could you provide a bit more info about
your test computer?
What kind of flashing did you perform? Was it copy and pasting a hex file
into the MICROBIT drive? Or using the online editors with WebUSB?
Could you post the hex file here? (email attachments won't work, you'd
have to add it to a comment directly from the GitHub website). It might be
better if the file is first compressed into a zip file or similar.
What kind of computer specs do you have? Operating system version, etc.
Do you run any antivirus software of similar that could be interacting
with the flashing?
—
Reply to this email directly, view it on GitHub
https://github.com/microbit-foundation/DAPLink/issues/18#issuecomment-1503135472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7A4GL6P5DSPCRBX26K5GFTXAU33XANCNFSM6AAAAAAWWLFYVI
.
You are receiving this because you were mentioned.Message ID:
@.***>

No antivirus
Vào 21:25, Th 3, 11 thg 4, 2023 Ralph Nathan @.***>
đã viết:

MBP 2011 MacOS 10.15
Vào 21:24, Th 3, 11 thg 4, 2023 Ralph Nathan @.***>
đã viết:

Web usb
Vào 18:10, Th 3, 11 thg 4, 2023 Carlos Pereira Atencio <
@.***> đã viết:

Thanks for the additional info @NathanGithub2013
https://github.com/NathanGithub2013!
Wow, 39 seconds is quite long. Could you provide a bit more info about
your test computer?
What kind of flashing did you perform? Was it copy and pasting a hex
file into the MICROBIT drive? Or using the online editors with WebUSB?
Could you post the hex file here? (email attachments won't work, you'd
have to add it to a comment directly from the GitHub website). It might be
better if the file is first compressed into a zip file or similar.
What kind of computer specs do you have? Operating system version, etc.
Do you run any antivirus software of similar that could be interacting
with the flashing?
—
Reply to this email directly, view it on GitHub
https://github.com/microbit-foundation/DAPLink/issues/18#issuecomment-1503135472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7A4GL6P5DSPCRBX26K5GFTXAU33XANCNFSM6AAAAAAWWLFYVI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Does not detect Textarea

Hello, I am using PhantomJS with Selenium and whenever I try to find a textarea, it says No Such Element.

can u provide the select tag code

Thank you for submitting this ticket!
This issue tracker primarily serves as the tool to report and track bugs, defects, and development tasks.
For general question, code review request, or other related discussion, please post it to either:

the official PhantomJS users mailing-list
Stack Overflow (with phantomjs tag)

If you have not read it yet, please check also a few tips on Effective Q&A.


⚠ Fetchers has degraded performance

In f1c0644, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 364 ms

Resolved: Fetchers performance has improved in 120d9c4.


Runnable track crossing eventlisteners (iOS)

If you try to slide the runnable track on iOS-Safari (iPhone 13) the eventListener input and change are fired at the same time.

Fixed by setting a onclick on the HTML head if the touchpoints is > 0


README.md scripts sorted

scripts in README.md  sorted and duplicate entries removed

Thanks For the help buddy😊


pythonPackages.py-multihash: init at 0.2.3

Motivation for this change
pythonPackages.py-multihash: init at 0.2.3
Depends On:
https://github.com/NixOS/nixpkgs/pull/83202
https://github.com/NixOS/nixpkgs/pull/83200
https://github.com/NixOS/nixpkgs/pull/83197
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

please see the reviews of the other two pr's, they are still blocking this PR

please see the reviews of the other two pr's, they are still blocking this PR

ok. Thanks.

Is this ready to be merged?


TypeError with Field type

Argument 2 passed to thejoshsmith\prismsyntaxhighlighting\services\PrismSyntaxHighlighting::parseDefinitions() must be an instance of thejoshsmith\prismsyntaxhighlighting\services\object, instance of stdClass given, called in /home/vagrant/sites/nystudio107/vendor/thejoshsmith/craft-prism-syntax-highlighting/src/services/PrismSyntaxHighlighting.php on line 70

Just removing the object typehint fixes the issue:
before:
    protected function parseDefinitions(array $config, object $themeDefinitions): array

after:
    protected function parseDefinitions(array $config, $themeDefinitions): array

It looks like the object typehint wasn't implemented until PHP 7.2
PHP version:
vagrant@homestead ~/sites/nystudio107 (develop) $ php -v
PHP 7.1.11-1+ubuntu16.04.1+deb.sury.org+1 (cli) (built: Oct 27 2017 13:49:56) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.1.11-1+ubuntu16.04.1+deb.sury.org+1, Copyright (c) 1999-2017, by Zend Technologies
    with Xdebug v2.5.5, Copyright (c) 2002-2017, by Derick Rethans
    with blackfire v1.18.0~linux-x64-non_zts71, https://blackfire.io, by SensioLabs

Thanks for reporting and providing a detailed bug report! I've removed the object typehint so it should be compatible with PHP versions <= 7.1 now.


Icon change suggested

The icon for unsubscribe looks so close to the icon for "mark as read" that it's hard to tell the difference if you don't know what you're looking for.
Suggest a change, like maybe the envelope with a circle and line through it, or an arrow jumping from the envelope to the garbage can, or some such.

Hi @mikecole79, we are updating the icon to better match the Nylas icon sizing, so we may tweak it be clearer. Either @colinking or I will post an update

Hey @mikecole79. The new 0.2.0 release contains a new icon that is a modified version of the previous icon. It focuses on the mute icon itself, rather than the email + mute icon combination from before. When you get the chance, try it out and let us know if you have any ideas for improvements.

Much easier to tell the difference between the buttons now.  Thank you.


Dps 5015 - current consumption fault

I hope you guys can help me…
I have a dps5015 for 3-4years, it’s pcb v2.3, and software 1.6, everything original.
Suddenly it started to show current consumption of 0.01amps. At first I tough it’s caused by my leads or by other things around, but today i took it out from the enclosure to see what’s wrong with it.
The behavior is the same.
At first i replaced the output caps and one smd transistor underneath the shunt bars as they were getting hotter then the other ic’s (i used thermal camera)
Nothing changed.
When psu is cold (first boot) everything is ok, after a few minutes it starts to show the 0.01amp consumption.
The drawn power will be directly propotional with the set voltage… for example if i set 30v it shows 0.30w if i set 19v it shows 0.19w and so on
It’s very annoying as i mainly use this psu to monitor the power sequence of different electronics.
I’m not familiar with psu repairs so i can figure out what’s wrong with it, so i’m asking you guys, maybe someone encountered this issue…

It turned out to be the input psu.... replacing it solved the issue.
Case closed!
PS: still using this dc psu for over 7 years!!

PS: still using this dc psu for over 7 years!!


How to control the LLM to generate mathematical formulas that conform to latex syntax?

any ideas with guidance lib? thx

That would be great. You would need a guidance function that represented the grammar of valid latex formulas. Then you could just add (+) that grammar to a model to make it generate a formula. If such a grammar exists on the web we could convert it, otherwise we would need to build up a grammar from scratch. If you have a basic subset of equations you would like to support and are up for trying to write latex_formula grammar I would be happy to give feedback / help a bit :)


Use the list styling that Sphinx explicitly notes

What's happening?
From #235:

Furo misses to define a list-style for non-arabic lists. This makes nested list look very ugly, as they are all numbered with 1., 2.

Reproducer
See https://pradyunsg.me/furo/kitchen-sink/lists_tables/#enumerated-lists vs https://sphinx-rtd-theme.readthedocs.io/en/stable/demo/lists_tables.html#enumerated-lists.
Expectation
The lists show up as Sphinx marks them.
There's a bit of CSS suggested for this in #235, by @latosha-maltba!
Code of Conduct

[X] I agree to follow the Code of Conduct.

This is a critical issue for me which makes me go back to RTD theme (which is uglier than Furo). :(


Fix transforming internal urls to relative urls from algolia

This fixes: https://github.com/timlrx/pliny/issues/158
You should return the item if it's an anchor or an internal link.
When it's not either of those, it is a valid url where you can transform the url and strip the hostname.
This could be extended by having a way to differentiate internal and external domains, maybe passing down siteMetadata.siteUrl to this component.

Thanks for the fix!


feat(pathmatch): improve the inclusion and exclusion pattern matching

This pull request enhances the pattern matching capabilities for .gitignore by transitioning to regex-based patterns. The initial suggestion to employ fnmatch or filepath.Match for this task has been set aside because these methods do not support case modification. After careful consideration, I concluded that adapting .gitignore patterns to fit regular expressions would be a more effective approach. Although this method may introduce a slight performance overhead, it ensures a more flexible and robust matching mechanism that is suitable for our current requirements. This update aims to deliver more precise control over the files we choose to ignore within our version control system.
This PR addresses and resolves issue #2.
To safeguard against potential future issues, tests have been added. These tests are designed to prevent any regressions in the event that the current implementation is modified down the line.

re-open with PR checks enabled.


Patch for Spade -> Filament

NOT READY FOR REVIEW

Ok don't actually think I need to expose SigMap? but I'll keep it in for now in case I do


15630 fixed like escaping

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#15630

The test fails

Yes. But the issue is fixed. It just seems that LIKE conditions are database-specific so test itself should be adjusted.

I would like to review it tomorrow if it’s possible.

I simplified test case, this should be enough to test #15630.

Accidentally squashed your tests as my commit. Hope you don't mind...

Thanks!


Count ua references after UA_EVENT_SHUTDOWN in ua_destroy

Resolves #701
Count ua references after UA_EVENT_SHUTDOWN in case that triggers clean-up in other modules.

there should not be a need for this change.
the change that was done in
commit 1406caf404a0d402210e86529feb0e8b6e0ac207
does not change the behaviour.
can you please include the full log of a session where your problem appears?

The problem occurs with my proprietary application module which holds a reference to the UA instance, so a relevant log is not easily available.
When the quit command is invoked, baresip simply does a ua_stop_all. My module removes all UA references upon receiving a  UA_EVENT_SHUTDOWN event (generated by ua_stop_all), thus allowing shutdown to proceed.

Referring to the diff in 1406caf:
Prior to the change, UA_EVENT_SHUTDOWN is first generated in ua_stop_all, allowing modules to remove their references before mem_nrefs(ua) is called to determine whether clean-up can proceed.
Afterwards, ua_stop_all has been changed such that mem_nrefs(ua) is done first, prior to ua_destroy. For application modules that hold references to the UA, this means that they will always contribute to the initial reference count, even if they remove all their references upon processing UA_EVENT_SHUTDOWN. This all means that another quit command is required to actually shutdown the application.

I notice that the mwi module also holds a reference and would hold up shutdown in the same manner as my module. That one seems even more problematic as it does not always immediately mem_deref its UA reference even upon receiving a UA_EVENT_SHUTDOWN as it may have further asynchronous actions to perform on shutdown.
With that module enabled and active, I cannot see how baresip can be shutdown without invoking quit repeatedly - it would be counted as holding a reference at the point of quit being invoked, causing baresip shutdown to be deferred. Upon mwi clean-up completing, it calls mem_deref on its UA reference, but there is nothing to tell baresip to continue application shutdown from there.
Have I misunderstood how such application modules are supposed to tie into shutdown mechanisms, particularly regarding the quit command? Is the idea that you just have to invoke the quit command repeatedly until everything has cleaned up to allow it to finally proceed? Please advise.

weili-jiang writes:

I notice that the mwi module also holds a reference and would hold
up shutdown in the same manner as my module. That one seems even more
problematic as it does not always immediately mem_deref its UA
reference even upon receiving a UA_EVENT_SHUTDOWN as it may have
further asynchronous actions to perform on shutdown.

I have not noticed any shutdown problems with mwi module in my app.  At
quit, the app calls ua_stop_all(false) and sets five second timer.  If
it fires before baresip has died, the app calls ua_stop_all(true).  I
haven't seen the latter happening.
Below is an example of Quit:
4-28 19:55:04.036  7679  7679 D Baresip : Quiting
04-28 19:55:04.086  7679  7679 D Baresip Service: Received onStartCommand action Stop
04-28 19:55:04.090  7679  7679 D Baresip Lib: ua_stop_all upon baresipStop
04-28 19:55:04.090  7679  7847 D Baresip Lib: calling ua_stop_all with force 0
04-28 19:55:04.090  7679  7847 D Baresip Lib: ua event (SHUTDOWN)
04-28 19:55:04.090  7679  7848 D Baresip Lib: ua: stop all (forced=0)
04-28 19:55:04.090  7679  7848 D Baresip Lib: mwi: shutdown of<EMAIL_ADDRESS>04-28 19:55:04.091  7679  7847 D Baresip Lib: ua event (SHUTDOWN)
04-28 19:55:04.092  7679  7848 D Baresip Lib: mwi: shutdown of<EMAIL_ADDRESS>04-28 19:55:04.092  7679  7679 D Baresip : Seconds remaining: 4
04-28 19:55:04.092  7679  7848 D Baresip Lib: ua: in use (2) by app module
04-28 19:55:04.591  7679  7847 D Baresip Lib: ua event (UNREGISTERING)
04-28 19:55:04.591  7679  7847 D Baresip Lib: sending ua/call<PHONE_NUMBER>36/0 event unregistering
04-28 19:55:04.592  7679  7847 D Baresip Service: uaEvent got event unregistering/506863377936/0
04-28 19:55:04.595  7679  7847 D Baresip Lib: ua event (UNREGISTERING)
04-28 19:55:04.595  7679  7847 D Baresip Lib: sending ua/call<PHONE_NUMBER>36/0 event unregistering
04-28 19:55:04.595  7679  7847 D Baresip Service: uaEvent got event unregistering/506863379536/0
04-28 19:55:04.852  7679  7847 D Baresip Lib: ua event (EXIT)
04-28 19:55:04.852  7679  7847 D Baresip Lib: ua exited -- stopping main runloop
04-28 19:55:04.852  7679  7847 D Baresip Lib: main loop exit
04-28 19:55:04.852  7679  7847 D Baresip Lib: closing ...
04-28 19:55:04.852  7679  7848 D Baresip Lib: ua: sip-stack exit
04-28 19:55:04.871  7679  7848 D Baresip Lib: module: unloading app mwi
04-28 19:55:04.875  7679  7848 D Baresip Lib: module: unloading app debug_cmd
04-28 19:55:04.875  7679  7848 D Baresip Lib: module: unloading app contact
04-28 19:55:04.875  7679  7847 D Baresip Lib: unloading modules ...
04-28 19:55:04.876  7679  7848 D Baresip Lib:  [   zrtp main]: DESTROYING LIBZRTP...
04-28 19:55:04.877  7679  7848 D Baresip Lib:  [   zrtp main]: DESTROYING LIBZRTP - DONE
04-28 19:55:04.877  7679  7847 D Baresip Lib: tell main that baresip has stopped
04-28 19:55:04.878  7679  7847 D Baresip Service: 'stopped' from baresip
04-28 19:55:04.884  7679  7679 D Baresip : Handling service event 'stopped'

Thanks @juha-h for your example. I think my problem is caused because my modified main function is no longer calling the right shutdown routines as per the default baresip main. I will update it to follow the baresip one.

On further analysis, this change is still required. See my update to 701.

@weili-jiang thanks for the patch and the detailed report.
it turns out that you are correct.
a module that deref ua syncronously on shutdown event, will need a fix like this.


Create Paginator Component

Issue Description
We need to build a common paginator component for Terra.
Issue Type
New Feature
Expected Behavior
Render a paginator component from provided props, in accordance with UX designs.

Tech Design
Proposed Inputs
Forward/Back: URL, start, limit, startParam, limitParam
First/Last: Total count
Paged: Page selected
General: Variant
The basic proposed functionality is the forward/back option. This requires a base URL and the key/values associate with start and limit. I'm taking the params as props because we can't guarantee that all consumers will use start/limit for their query params.
The first/last functionality is dependent on knowing how many pages exist, which can be calculate with the limit and total count.
Listing the page numbers is also dependent on knowing how many pages exist, but additionally needs to know currently selected page to build the relative linking and selected-page-display.
The variant will allow the consumer to flex between the Search Paginator and the Progressive Paginator. The progressive paginator will additionally be dependent on selected page as a "required" prop.
Proposed API
<Paginator
	baseUri={base_uri_for_pagination}
	startParam={some_start_param}
	limitParam={some_limit_param}
	start={some_start}
	limit={some_limit}
	page={selected_page} // Optional
	totalCount={totalCount} // Optional
	// onPageChange is intended to support optional Ajax loading of content on pagination.
	onPageChange={callbackToLoadContentToContainer()} // Optional
	variant={‘search’ || ‘progressive’} // Hate these names. Don’t have better ones. Optional. Defaults ‘search’.
/>

Proposed URL Builder Utility
var isPaged = (totalCount, page) => (!(typeof(totalCount) === 'undefined' || typeof(page) === 'undefined'));

var getPageCount = (limit, totalCount) => (Math.ceil(totalCount / limit));

// May redesign into a more complex function to account for more situations and be a little more readable.
// Ex: Setting previous to undefined if previous would result in a negative page.
var urlBuilder = (baseUrl, startParam, limitParam, start, limit, totalCount = undefined, page = undefined) => (
  {
    href: baseUrl,
    next: `${baseUrl}?${startParam}=${start + limit}&${limitParam}=${limit}` + (page ? `page=${page + 1}` : ''),
    previous: `${baseUrl}?${startParam}=${start - limit}&${limitParam}=${limit}` + (page ? `page=${page - 1}` : ''),
    first: totalCount ? `${baseUrl}?${startParam}=0&${limitParam}=${limit}&page=1` : undefined,
    last: totalCount ? `${baseUrl}?${startParam}=${getPageCount(limit, totalCount) - limit}&${limitParam}=${limit}&page=${getPageCount(limit, totalCount)}` : undefined,
    // The 'page' property below may be pulled out and built in the main component, or I might parse and modify as-needed.
    page: isPaged(totalCount, page) ? `${baseUrl}?${startParam}=${start}&${limitParam}=${limit}&page=${page}` : undefined,
  }
);

export default urlBuilder;

Let's rename the page prop to currentPage so that it's more clear as to it's intent.

@neilpfeiffer Some of our old pagination used show more style paginators that didn't know how many pages existed. Do we need to account for scenarios when we are unsure of how many total pages there are?

How about themability variables? We have posted them on several tech designs.

Also, should we do these as two different paginators instead of using a prop to switch between the two?

@neilpfeiffer Some of our old pagination used show more style paginators that didn't know how many pages existed. Do we need to account for scenarios when we are unsure of how many total pages there are?

The various components (like page numbers) are listed as conditional at the top of the basic implementation. I took this to mean that the component should flex based on what information is provided.

Also, should we do these as two different paginators instead of using a prop to switch between the two?

Is there a technical/UX/whatever reason you think that splitting them into separate top-level components is better than just providing a variant that achieves basically the same thing?

If props or other aspects change for the API on one paginator but not on the other, it's easier to adjust for. Otherwise, we could end up having a large paginator component that has a large set of props for which only half are used at a given time.

Is that a realistic possibility, though? Sure, the design could change, but that is true for literally all of our components. Having said that, I know the updates to button caused a bunch of headaches for everyone involved.
@bjankord: Thoughts?

The above tech design is out-of-date per discussion last week. Rather than providing a mechanism to build and manage URLs in-component, we will require the user to pass in a callback function that has the Ajax/page-reload logic on their end.
A new formal design will be forthcoming.

+1 on tech design

+1 on tech design

+1 on tech design

Get get the page number to display, will the selectedPage, totalCount, and itemCountPerPage need to be included? Also, will itemCountPerPage have a default?

I say for now that we make all those props required. As for default itemCountPerPage, I think different teams across Cerner had different values for that, so I wouldn't default that for now.

They shouldn't be required because the design accounts for showing fewer UI elements (just next/previous) when page number isn't known.
I could see having totalCount and itemCountPerPage required, but not 100% sold on it.

Sorry. I was vague. I meant requiring them all in order to get the display. I'm fine with leaving them as optional with the descriptions you provided.

Going with the v2 design for my PR.

JIRA created


Custom sb.roles.ts are not being precompiled (to be honest only components have precompile feature)

Added precompile feature tyo components, should add it to every stuff:

datasources
roles

fix: https://github.com/sb-mig/sb-mig/pull/228


Update runtime-config.md

correct the cli

Thanks for the PR! Since it's a simple typo, we can forego the CLA; but feel free to still sign it for future, larger improvements. Thanks again!


Allow Time::Span.new with positional + nanoseconds

Fixes #8825

I don't follow why we need this to fix #8825. What's wrong with my suggestion from https://github.com/crystal-lang/crystal/issues/8825#issuecomment-589006913 ?
I suppose we could still follow with this because nanoseconds is more likely to be passed as named parameter than other arguments. I'm not sure, but it might have even been a named-only argument at one point.

The idea was to use named argument for subseconds units since the nano/micro was not as obvious as the rest of the parts.
Some db drivers use the named argument so without this we are breaking that code without a deprecation cycle. People are forced to update in order to use nightly rather that noticing a deprecation warning on nightly.


[vcr-2.0] Emit netty internal metrics

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (8093c80) 72.58% compared to head (706da15) 17.10%.

Files
Patch %
Lines

...rc/main/java/com/github/ambry/cloud/VcrServer.java
0.00%
6 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2669       +/-   ##
=============================================
- Coverage     72.58%   17.10%   -55.48%
+ Complexity    11552     2918     -8634
=============================================
  Files           809      809
  Lines         65633    65639        +6
  Branches       8018     8020        +2
=============================================
- Hits          47640    11230    -36410
- Misses        15381    53593    +38212
+ Partials       2612      816     -1796

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


Copilot Chat - Analyzers, detach from dotnet props

Description
Removing Copilot Chat's import of the /dotnet/ folder's common props:
  <Import Project="$(RepoRoot)/dotnet/Directory.Build.targets" />
  <Import Project="$(RepoRoot)/dotnet/Directory.Build.props" />

This allows the project folder to be copied out of the repo and run independently.
Adding the useful props to Copilot Chat's csproj file. Adding the analyzers in use by dotnet, and fixing style violations they pick up. (Note: these analyzers are optional and can be added/removed in Copilot Chat without affecting the rest of the repo).
Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[X] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Let's wait for @adrianwyatt @TaoChenOSU or @hathind-ms to approve. This one's for you folks, so if you're happy with it, go ahead and merge.


Bugfix: Make computeVertexNormals normalize the normal vectors correc…

…tly: Float32Array.subarray must be called with [begin, end) indices.
Visually, this fix will make neurons appear slightly brighter overall and remove artifacts where certain vertices were much too dark before.
Optimization: Removed vertexFaceCount: With normal vector normalization working correctly, dividing by the number of adjacent triangles has no effect and can be removed.

Thanks for catching this bug!


Home page

fixes #165

sync with master and fix CI errors, otherwise LGTM

@albertcui should we feature our sponsors on the home page?

SEXY AF -signoff-


Add typescript and function arguments support.

This PR will add vt context support for function argument, interface and enum.

Hey, I can merge if you fix the indentation :).
Thank you!

What? But I didn't find any problems with it!!
Please tell me what went wrong.(It's best to have screenshots)
Thanks!

You can see it here: https://github.com/haringsrob/nvim_context_vt/blob/9d32d1368159eba8d0bcc35317806e96a3dd9dcc/lua/nvim_context_vt.lua

That's OK.

I think you have used tabs instead of spaces.

我认为您使用了选项卡而不是空格。

Yes, fix it.


Add _ComplianceBanner unit tests

TODO: Tests are failing when run together. Need to figure out where isolation is failing.


Add isTrusted method to check events

JIRA: https://rockabox.atlassian.net/browse/RIG-1083
Helper which checks whether an event is trusted or not.
More about trusted events here: https://developer.mozilla.org/en-US/docs/Web/API/Event/isTrusted
If a browser doesn't support isTrusted we fallback to clientX and clientY properties. X and Y have to be greater than 0 to treat an event as trusted.
More about clientX: http://www.w3schools.com/jsref/event_clientx.asp

Coverage decreased (-70.2%) to 25.688% when pulling 9750623bfb03c539348d0013db48f6731908e307 on 1083/is-trusted into 7b84532b7859e1d8b038f0e223f5953f0ded078b on master.

🍪 ☕


Codebase: For non upgradeable contracts consider using upgradeable contract as parents

E.g. AccessAdmin is non upgradeable contract which uses AccessManagedUpgradeable as parent.
Consider using AccessManaged + Initializable in such cases

please provide the list of impacted contracts


Support JRuby

Removed mysql2 from gemspec
Updated Gemfiles to use mysql2 or jdbc depends on ruby platform.

dead


added permission to DELETE DBClusterSnapshots.

Without this permission, the lambda starts failing after the first of several DBClusters, because of not being able to purge older snapshots for the first DBCluster.

Hi @timbaileyjones
Thanks for the PR, good spot! Can you add the permission to the source file for the template as well, please: https://github.com/pbudzon/aws-maintenance/blob/master/infrastructure/src/rds-cross-region-backup.py#L136

Done.  Thanks for the quick response!


Restructure 2-stream shortwave solver

Purpose
Restructure 2-stream shortwave solver

[x] I have read and checked the items on the review checklist.

The new code structure looks good to me, thanks! We do need flux_dn_dir, e.g. for the land.

If we just need the flux_dn_dir at the surface, we can further reduce memory footprint!
I can take care of that in a different PR!

The new code structure looks good to me, thanks! We do need flux_dn_dir, e.g. for the land.

If we just need the flux_dn_dir at the surface, we can further reduce memory footprint! I can take care of that in a different PR!

Sounds good, thanks!


Add card “when charger is plugged in”

Hi,
Loving your app (although I need to restart it quite often to resolve the “no access to user account” error others have seen.
What I would love to see added is a card stating whether or not the chargecable is in the car. I only start charging around midnight but would like to know whether or not the cable has been put in so that I won’t have any Nasty surprises that next morning where it didn’t charge at all.
Is this possible :)?

Hi.
There is an option for ‘charge port open’ but personally I use the ‘scheduled charging pending’
This is only true if you have set scheduled charging in the car (f.ex. at 04:00) AND the cable is connected.
This way you can start the charging at will from Homey and if all fails the charging starts at 04:00
I have set this it to check the current utility price and start charging if the power is among the 6 cheapest hours of the day/night.
--
Mvh
Torstein Wikene 
www.wikene.no
"Ulempen ved å gjøre noe riktig første gangen, er at ingen forstår hvor vanskelig det egentlig var"
On 25 Nov 2019, 21:33 +0100, BBEHannes<EMAIL_ADDRESS>wrote:

Hi,
Loving your app (although I need to restart it quite often to resolve the “no access to user account” error others have seen.
What I would love to see added is a card stating whether or not the chargecable is in the car. I only start charging around midnight but would like to know whether or not the cable has been put in so that I won’t have any Nasty surprises that next morning where it didn’t charge at all.
Is this possible :)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This sounds exactly what I am searching for but I do not see these cards when creating a flow..?

would this do what I am searching for (check if schedules charging is active);


fix: only listen for the 'storage' event if using 'local' storage

Fixes #150.
Notes
jsdom doesn't raise the 'storage' event because this event only fires when values change in a different window, which is something we don't have (we only have one window).  So to unit-test, one must dispatch the event manually.
Also:  The event handler uses getValueFromStorage to obtain the value, but the event.newValue property already has this value.  This seems like an unnecessary performance hit, and may be best to simply deserialize event.newValue.
New Situation/Issue
I noticed that there is no deletion.  I think that whenever this.#current becomes undefined, the value in storage should be deleted.

Two more things:  Unsure if there's anything else required, so I made this a draft PR.  The second is that I added the delay() function to the testing helpers so as to not continuously drag the timeout thing in a new promise every time one must await for effects to run.

Done.  Let me know if there are any more pendings.  If not, I'll add a changeset and take the PR out of Draft status.

Feel free to move it out of draft @webJose !

Thank you!

Ha, you were in a hurry.  I came but the branch was merged already.


favorites collection is not synced

If you set favourites in quick manner then not all of them are present in favourites collection.

I can't reproduce. You have to be more specific, please.

I scraping changes sth then game is not "favorite" any more.


FIX: npm audit, 2 vulnerabilities

Fixed 2 medium vulnerabilities with npm audit fix

Thx for updating, need to add dependabot to the project


Labels for nav icons

We would like css for text instead of icons in the nav.

I liked Kasper's take on this problem:

I'd like to have a breadcrumb style text only navigation on the left side of the navigation bar.


Default values for materials

Hello again!
I’m trying to set up my materials with default values. For example, given the material button.html:
---
value: Default button text
---
<button class="button">{{value}}</button>

when included in form.html, I would expect the following code:
<form action="">
	{{> button value="My new button text"}}
</form>

to generate:
<form action="">
	<button class="button">My new button text</button>
</form>

but instead, the text defined in button.html remains. How can I override placeholder values in materials. Is this possible?
Thanks,
Paul

Front matter gets name-spaced internally, so doing default values like this is a little tricky.
I use a helper default to achieve this.
assemble({
  helpers: {
    default: function (value, defaultValue) {
      return value ? value : defaultValue;
    }
  }
})

{{default value "Default Text"}}

@paulrobertlloyd if it doesn't need to be in the front matter you could use an if:
<button class="button">{{#if value}}{{ value }}{{^}}placeholder{{/if}}</button>

I got this snippet from http://frontend.github.io/toolbox/components.html

@LukeAskew Oh, that helper is useful (I had previously used if statements @ptibbetts, but they can get unwieldy). Thanks for the tip :smile:
I guess the problem is that the example I provided above is relatively simple compared to most components. Ideally a component can be provided with a default context (data that provides dummy copy, images, button states etc), but which can be overridden by another context as provided via a partial (BTW, only started getting to grips with handlebars, and the concept of contexts etc.)
Looking at PatternLab, it seems to handle this use case well (with a data.json file used to provide these default values, which can be overrides when partials are called in individual views/pages). Is there a way to achieve the same with fabricator as it exists today?

It's been a while since I've visited this concept, but I totally understand the desire to associate default values with a partial more easily.
I should be picking up another Fabricator project at work next month, so I can make a note to revisit this feature.

Awesome, thanks @LukeAskew!

Hey @LukeAskew any chance there's been progress on this? I'm trying to do what I think is the same thing.
I've got a component that uses front-matter to define some values, but when I include that component into a structure, I want to have some different values.

Just started work on v2.0 this week. Hoping to move quickly, so stay tuned!

Thanks. That'll be awesome and will keep the code much cleaner instead of needing to pass a bunch of overrides into our partials and use a bunch of if statements to look for the existence of the overrides!


expose webpack config

My attempt to implement #40. I think it's the simplest possible solution.
To customize config user have to create webpack.js in project root and export function that adjusts config as needed.
module.exports = (config, hotReload) => {
  if (hotReload) {
    const babelLoader = config.module.loaders.find(({loader}) => loader === 'babel')
    babelLoader.query.plugins.push(require.resolve('babel-plugin-styled-components-named'))
  }
  // Style loader not works for some reason, but it should be configured similar to this:
  config.resolveLoader.root.push(require('path').join(__dirname, 'node_modules'))
  config.module.preLoaders.push({ test: /\.css$/, loader: "style!css" })
}

I think this is a great solution. I was also thinking of using a function to decorate the suggested webpack config next wants.
What do people think about:

Defaulting to webpack.js as name

Should we add a config flag in package.json? next.webpack?

Should we instead roll this together with #25 and then you can initialize the server and supply the webpack config?
next({
  webpack: (cfg) => cfg
})

I would suggest the name webpack.config.js for consistency with webpack itself, though admittedly the API is somewhat specialized here.
http://webpack.github.io/docs/configuration.html

@giladgray I will change it to use returned value.
Actually I don't like both names. webpack.config.js  suggests that it exports config object. webpack.js looks like it exports webpack instance itself.
Maybe webpack.next.js would be better?
Actually, I think that it would be better to configure everything in the single point as suggested by @rauchg, but I don't feel like diving so deep in next.js code for now.
next({
 webpack: (cfg) => cfg
})

@vdanchenkov my comment on the code above (and the author's response) suggests that this file will in fact export the config object.
however webpack.next.js makes some sense cuz it's similar to webpack's own usage but makes it clear that we're doing something slightly different.
a code-less solution should be high priority here because the typical usage will be via NPM script. i don't want to have to create a file just to invoke next() with some config when that could all be done via the CLI. (now i'm thinking a CLI flag to specify the file path might also be helpful, but that could come later.)

I think it would be better to use a centralised config for Next called something like next.config.js. I made a PR for it here: https://github.com/zeit/next.js/pull/222. I think this could work great together with this PR.
You could for example do something like this:
// next.config.js

module.exports = {
  webpack: (cfg) => cfg
}

@CompuIves How should we proceed? Will you apply webpack configuration functionality to #222 before it's merged? In that case I'll close this PR in favour of yours.

@vdanchenkov I added your functionality to #222!


Update stale.yml

Add Category:Research to the list of skipped labels; since the nature of this issues is pretty long lasting.

context: https://github.com/owncloud/ocis/issues?q=is%3Aopen+is%3Aissue+label%3AStatus%3AStale+label%3ACategory%3AResearch
17 issues labeled as Research should not be really flagged as research.


[stable/traefik] Traefik fails when ACME is enabled

Is this a request for help?: yes

Is this a BUG REPORT or FEATURE REQUEST? (choose one): bug?

Version of Helm and Kubernetes:
helm: v2.9.1
Kube: 1.10.5-gke.3
Which chart: traefik
What happened:
# values-override.yaml

image: traefik
imageTag: 1.6.5-alpine

rbac:
  enabled: true

dashboard:
  enabled: true
  domain: traefik.mydomain.com

#ssl:
#  enabled: true
#  enforced: true

acme:
  enabled: true
  logging: true
  email<EMAIL_ADDRESS>  staging: true
  domains:
    enabled: true
    domainList:
      - main: "*.mydomain.com"
    challengeType: dns-01
  dnsProvider:
    name: route53
    route53:
      AWS_REGION: "us-east-1"
      AWS_ACCESS_KEY_ID: "..."
      AWS_SECRET_ACCESS_KEY: "..."

helm install stable/traefik  --name ingress --namespace traefik --values ./values-override.yaml

traefik container dies with error
level=fatal msg="Unknown entrypoint \"https\" for provider ACME configuration"

If I uncomment ssl section, pod starts and I'm able to access https://traefik.mydomain.com. However the cert in not from LE, it's the default one for *.example.com.
I'm also now seeing errors like this in the log
level=error msg="Unable to obtain ACME certificate for domains \"traefik.mydomain.com\" detected thanks to rule \"Host:traefik.mydomain.com\" : cannot get ACME client ACME challenge not specified, please select HTTP or DNS Challenge"

Is there something I'm missing?
What you expected to happen:
To be able to access the dashboard at https://traefik.mydomain.com with my LE wildcard cert
How to reproduce it (as minimally and precisely as possible):
see above

I see now that I have challengeType in the wrong spot.

I have exactly the same but I think skipping the ssl-part would also skip this default entrypoint. Have you really only fixed the challengeType parameter, @sundeer ?

Oh dear, my fault: I forgot to explicitely set staging: false.


github issue template: add sandboxing info

Motivation for this change
Whether sandboxing is enabled or disabled may help for some issues and facilitate reproducing.

This seems a little hacky. "brew" has a command called "brew config", maybe we could add that into Nix?

@rasendubi good idea. I think this can be improved with a || echo no for systems that don't have that file at all.
@matthewbauer it's also a good idea, probably better in the long run but it's expanding the scope of the PR


Suggestion: Guest-user support

Add support to upload files without a username/password that will get automatically deleted after a set time/never

Automated deletion is unsupported: this is a stateless API without any kind of database, tracking upload times and running a task in the background to delete files would break that premise. Please use a fully-featured project like Lutim which allows anonymous and temporary uploads if you need that, with client-side encryption pretty much like ZeroBin/PrivateBin as well.
Regarding the unauthenticated access, it would be a liability more than anything else, but you can already create a public token that you can share with everyone, it can be anything you want of course. Disabling the authentication entirely is also possible by editing the code a bit of course, but it will never be a readily available option due to the aforementioned security and liability reasons!

Automated deletion is unsupported: this is a stateless API without any kind of database, tracking upload times and running a task in the background to delete files would break that premise. Please use a fully-featured project like Lutim which allows anonymous and temporary uploads if you need that, with client-side encryption pretty much like ZeroBin/PrivateBin as well.
Regarding the unauthenticated access, it would be a liability more than anything else, but you can already create a public token that you can share with everyone, it can be anything you want of course. Disabling the authentication entirely is also possible by editing the code a bit of course, but it will never be a readily available option due to the aforementioned security and liability reasons!


Update tutorial-use-disks-powershell.md

This change fixes the incorrect use of Add-AzVMDataDisk and Remove-AzVMDataDisk for operations on a scale set VM.

@microsoft-github-policy-service agree

@ju-shim
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Fixed #29808 -- Fixed initial migration detection when identifiers are case-insensitive.

Ticket 29808.
Not sure about the solution.
PR just created for proposing the solution.

@felixxm Thanks for the review. I added a fix for mixed-case column names based on the ignores_table_name_case feature flag.
wouldn't it better to change the feature flag name to ignores_identifier_case?

@charettes , Thanks for the review.
I've changed the patch to address your comments.
Please let me know if something else is needed.

@hramezani Thanks :+1: I pushed minor edits.


03/30/2018 02:44 : Create Pull Request for branch prTesting_2018033002 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

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


Issues reported are for when something is published, not when patches were released

As an example, CVE-2022-30190 was patched in June 2022.
However, running patch_review.py 2022-jun will report no mention of it.  Why?
Well, MSRC lists "it" as being released in May rather than June.  "It" being something said about the vulnerability, not the updates themselves.
I suspect that users of patch_review.py may expect the tool to list what updates are released in the specified month.  Not what bulletins.

The issue seems to be related to the MSRC API and the nature of the information contained therein. The script patch_review.py pulls data from the API, which may not align perfectly with the patch release dates.
In the case of CVE-2022-30190, it seems that the vulnerability was disclosed or reported in May, but the patch was not released until June. The MSRC data, and therefore the patch_review.py script, appears to track the disclosure dates rather than the patch release dates. I agree this discrepancy would confuse users expecting to see patch release dates.
To address this issue, the script would need to pull data from a source that accurately tracks patch release dates.
Another possibility might be to modify the script to pull data for a range of months rather than a specific month to ensure that all relevant patches are included. For example, if you're looking at June, you might pull data for May, June, and July. This is not a perfect solution, as it could include some patches that weren't released in June, but it would help ensure you're not missing any patches.
If you haven't already, you might look at the MS Powershell module, which already seems to have some of your desired functionality. https://github.com/Microsoft/MSRC-Microsoft-Security-Updates-API/blob/main/src/README.md


Parameter description are treated as plain text

monaco-editor version: 0.19.3
Browser: Firefox 72.0.2
OS: MacOS
Playground code that reproduces the issue:
monaco.languages.typescript.javascriptDefaults.setDiagnosticsOptions({
	noSemanticValidation: true,
	noSyntaxValidation: false
});

monaco.languages.typescript.javascriptDefaults.setCompilerOptions({
	target: monaco.languages.typescript.ScriptTarget.ES6,
	allowNonTsExtensions: true
});

monaco.languages.typescript.javascriptDefaults.addExtraLib([
	'declare class Facts {',
	'    /**',
    '     * @param t **Some bold text** Some other text',
	'     */',
	'    static next(t:string):string',
	'}',
].join('\n'), 'ts:filename/facts.d.ts');

var jsCode = "return Facts.next(/* hit ctrl + shift + space */);";

monaco.editor.create(document.getElementById("container"), {
	value: jsCode,
	language: "javascript"
});

Description
The description of the parameter on function next is treated as plain text when just showing the description of the parameter. Additionally, the displayed comment is missing a leading *. It says *Some bold text**. It should be **Some bold text**

The incorrect parsing issue also arises when hovering the method directly. Here the text is treated as markdown, but because a leading * was stripped, the result is wrong. The text Some bold text should be bold and not cursive

Expected behaviour:
The description of the parameter is not missing the leading *, when showing the hint on the parameter it should be treated as markdown and rendered accordingly.

Fixed by https://github.com/microsoft/monaco-typescript/pull/66


Add Grab etcd use case

Contributing guidelines
Please read our contribution workflow before submitting a pull request.

Can you fix the commit title to Documentation: add Grab etcd use case?
Thanks.

@gyuho Fixed, thanks

LGTM. Thanks!

Merging.

Codecov Report

Merging #7458 into master will decrease coverage by -0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7458      +/-   ##
==========================================
- Coverage   70.44%   70.35%   -0.09%
==========================================
  Files         237      237
  Lines       21238    21238
==========================================
- Hits        14962    14943      -19
- Misses       5143     5156      +13
- Partials     1133     1139       +6

Impacted Files
Coverage Δ

auth/jwt.go
0% <ø> (ø)
:white_check_mark:

pkg/adt/interval_tree.go
82.77% <ø> (-4.73%)
:x:

pkg/fileutil/purge.go
73.68% <0%> (-7.9%)
:x:

proxy/grpcproxy/register.go
57.14% <0%> (-3.58%)
:x:

rafthttp/msgappv2_codec.go
69.56% <0%> (-1.74%)
:x:

etcdmain/etcd.go
45.64% <0%> (-1.25%)
:x:

etcdserver/v3_server.go
61.84% <0%> (-1.1%)
:x:

etcdserver/server.go
78.92% <0%> (-0.36%)
:x:

clientv3/watch.go
89.81% <0%> (+0.26%)
:white_check_mark:

... and 5 more

Continue to review full report at Codecov.

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


prevent some common JIT issues

keep shapes stable when possible
prime with common scenarios

cc @krisselden

looks good to me, closes #57

and closes #56


Adds a item abstration to Memoizable and Callable Items

Hello, I love dry-rb tools, I try to use it in my projects everytime that I can (Great work dry-rb Team!), then I decided to make a change code proposal.
I created a Factory to create a Memoizable Item or a Callable item. When options has the memoize option as true the Memoizable object is created, but if isn't true, the Callable object is used.

Wow, thanks @GabrielMalakias, nice work :+1:
Thanks for the PR and using dry-rb :smile:


Clean up imports and exports to help with package and bundle size

While looking into some bundle size issues internally, I noticed that there were a lot of third party libraries coming from Perses packages that probably shouldn't be. This made me go an take a look at our JavaScript bundles for the Perses app (i.e. npm run analyze -w app). This was what it looked like when I started:

You can see we have 2.7 MB of Javascript total (before gzipping which would happen on the wire). After doing some digging around and cleaning up a bunch of things, including adding a bit of code splitting to the app, this is what things look like now:

This ended up cutting almost 800 KB of JavaScript from our bundles. This should also help folks using the Perses packages in their own bundles. I'll add a few comments inline about a few of the more interesting changes.

If anyone's interested, the webpack docs explain the sideEffects stuff here: https://webpack.js.org/guides/tree-shaking/

Really impressive. Kudo @LukeTillman for doing it !


Add addresses in Newport, OR

Status:

[ ] Research sources and propose best source (and it's data license)
[ ] Review proposed source
[ ] Approve source based on review comments
[ ] Add to OpenAddresses

NOTE: Proposed source notes should include basic column mapping to OpenAddress data model.

will have to call:
Lincoln County Assessor
225 West Olive Street, Room 207
Newport, Oregon 97365
tel:<PHONE_NUMBER>
fax:<PHONE_NUMBER>

Will likely made redundant with https://github.com/openaddresses/openaddresses/issues/2148.

Oregon statewide data in #2148 doesn't include Lincoln county :(

Spoke to assessor, no data... Called e911 dispatch.<PHONE_NUMBER> Robbie hurry.  Left a voicemail.

Robbie called back.  Spoke for 15 minutes,. Need to formally request data.  Person in charge of Salem and three counties data requests will return Wednesday.  Will follow up first thing Wednesday morning.

called last wednesday (12/21/2016) - left a voicemail.  Called 12/27/2016 and left another voicemail.

called Salem IT GIS dept<PHONE_NUMBER> and left a voicemail

Okay, we are good to go. Need to sign a license agreement, terms are outdated (by ten years), but she still needs it signed, and we just need to explain our use of the data. Since we are incorporating in a derivative, there is no problem with us using the data in OA. We just need to explain that the data will be incorporated into OA, and what OA is. I know it says we can't, and I explained that to her, but she said we can. So I'm not sure what our play is here? They are working on a open data site, and she is working with their legal department to get the terms updated, but no idea when that will be.

@nvkelso let me know how you would like me to handle this.  Thanks!

The license in question is GIS_PLA20161228.docx. Unless we get it in writing that it's OK to ignore the no-redistribution and no-commercial-use sections of this license we can't use their data.

sent an e-mail asking if the terms can be changed; they have an open site coming soon with the data. Worst case, we have to wait...

Hi Justin,
No, I have no way of expediting Legal, and having them prepare a single license just for your use.  No, I cannot commit to a time when open data will be final for us.
What you can do is send me a narrative on how you intend to use the data.  And, if there is specific language in the PLA that you want to strike or re-write, then identify it… and I will take it from there.
Thanks,
Sue

@nvkelso can we fill out form - cross off what we don't want and submit? Worse case scenario is they say no. They are in process of creating open data portal - no idea when it will be finished. The terms she sent are outdated, but they have to send them.

@iandees Is the form striking and offering back to this source something you'd need to do as an OA legal officer?

I crossed out the Permitted Use and Restrictions on Use sections of the license and posted it here: GIS_PLA20161228.docx
Alternatively, we could write something about how we want to use the data and try to make it fit in with 2.2's "authorized in advance in writing by the City of Salem".
It would probably be better/simpler/less confusing to ask them to release their data under something like CC-BY rather than whatever this document is.

http://opensalem-salem.opendata.arcgis.com/datasets/d686030c3b1a474bbe019320b77ce7e6_0

they are reviewing so we can get data for the three counties instead of just city. fingers Crossed!

here is my POC: Blohm Susan GIS Lead Analyst City of Salem<EMAIL_ADDRESS>(503) 763-3429

just followed up with a call. She never gave it to their legal team to look at..... So now she is ;)  Stay tuned

sent another follow-up email...

#6170 adds addresses for Newport
Are there any updates for obtaining addresses from Lincoln County?

@arch0345 I'll check tomorrow

I've joined the shapefile with the other dbf here: taxlotaddress.zip
@iandees would you be able to upload this to data.openaddresses.io?

Uploaded to https://data.openaddresses.io/cache/uploads/iandees/f75ee2/taxlotaddress.zip


microsoft_provider  - review images

As a developer I should use style lib icons for images that are vectors and all versions of image densities for the other images
Acceptance criteria
1 - Create the vector icons for the lib
2 - Create the missing image sizes

updating for TikiStyle


Can't resolve 'stylus-loader'

I have alread add stylus-loader config to webpack. When I require a stylus file, it successfully installed the dependencies, but it comes up with the following error:

When I restart the cmd, it works correctly.

I use happypack in my project, when I remove happypack config, it works correctly.

Sorry plugin in happypack, also it no need for webpack v5, sorry for big delay, plugin will be live in near future


Sort roots before serialization

Also update example program

This was causing failures in regression tests needed for grouping of sentences and would also affect parallel operation.

Thanks @kwalcock !


Crash when using aim assistance against mob.

Describe the bug:
Tried out the mod for 1.17.1, when I tried using aim assistance on a zombie or creeper it crashed the game. Crash report attached. Picture of aim assistance settings attached.
To Reproduce (or what were you doing when it happened):
Using aim assistance.
Aim assistance setup:

Crash log:
crash-2021-08-28_13.44.35-client.txt

Have you tried running the mod using the option debug=true: yes / no
no

Have you tried deleting the config files? (minecraft/config/bc_filename.properties): yes / no
yes - still crashes.

Thanks I'll take a look into it. It definitely needs more testing, it must be a dumb mistake since I just ported the current version, I did not remove or add anything new

Fixed in https://github.com/lilgallon/Bettercolors/releases/tag/7.4.1-MC1.17.1fa


Question: Prepend a string to a file within a pipeline?

First, thanks for creating and maintaining this package!  It's something I've wished for repeatedly since discovering Go a few years ago AND the implementation is really clean.
Upon discovering script yesterday, I put it to work in query handler for sqlite.
// Query executes a SQL query contained in a file at qrypath against the sqlite3
// db file located at dbpath.  The result is returned as a string and err will
// be non-nil if an error is encountered.
// You may pass zero or more sqlite3 special commands in dotcmd. These are
// typically used to change the output format, e.g. ".mode json" but other
// special commands may be useful.  Sqlite3 currently recognizes more than 60
// dot commands. See https://www.sqlite.org/cli.html for more info.
func Query(qrypath string, dbpath string, dotcmd ...string) (result string, err error) {
	qrybytes, err := ioutil.ReadFile(qrypath)
	if err != nil {
		return
	}
	qry := strings.Join(dotcmd, "\n") + "\n" + string(qrybytes)
	cmd := fmt.Sprintf("sqlite3 %s", dbpath)
	result, err = script.Echo(qry).Exec(cmd).String()
	return
}

So is there any way to construct a pipeline that would catenate the joined dotcmd lines and the content of the sql file (qrypath)  before passing it to Exec?

You're welcome @Michael-F-Ellis! This looks like an interesting application.
I don't think we can do this in one pipeline at the moment; script.Slice will produce a pipe containing your slice of dotcmd strings, one per line, and script.File will produce a pipe containing the contents of your query file. But currently we'd have to extract these to strings and then write another pipeline to send them to the command.
Can you suggest a neat syntax for doing this operation all in one pipeline? If so, I'd certainly consider including it in the library.

dots := strings.Join(dotcmd, "\n") + "\n"
cmd := fmt.Sprintf("sqlite3 %s", dbpath)
result, err = script.Echo(dots).AppendFromFile(qrypath).Exec(cmd).String()

That sounds good!

@bitfield Can I pick this issue up?

This doesn't seem like something that's in high demand. If we got, say, ten thumbs-up on this issue (perhaps not an unreasonable requirement for a project with over 5,000 stars), I'd at least consider it worth looking at, but not until then.


How do you log?

If I use Console.WriteLine for logs, the electron main process encounters an error because the logs are not JSON.

You have two options, either use Console.Error.WriteLine and that will show up in the console window if you are running just Node. If you are running electron it doesn't display (I don't know exactly why).
Alternatively, and this is probably the easiest option, create a request handler for a 'log' request type in Javascript, then in .NET just do connection.Send('log', 'the thing you want to log');
The handler in javascript can be something like:
connection.on('log', logMessage => {
  console.log(`From .NET: ${logMessage}`);
});

And you can see that in the dev tool's console if you are running this in Electron.
Let me know if this helps and if not I can elaborate more on how this works

Thanks, I ended up doing exactly that before I even opened this issue :D Guess I'll keep doing it!


New version: MoritzBunkus.MKVToolNix version 82.0

This pull request is automatically generated by 🥟 Dumplings in #3029
Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,

This PR's version number 82.0 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 81.0.0, which has 3 parameters.

Is this intentional?
(Automated response - build 510)