Stuck on Spotify Auth

I'm sure this is probably a "user error" however I've set up Maloja and Multi-Scrobbler on a remote server via Docker. As of now, I can scrobble directly to Maloja, Plex scrobbling works via Multi-Scrobbler, however I'm stuck on Spotify auth.
When I click "Reauth and restart polling", it brings up a window that has me login to Spotify, returns an "OK" and then I can close. Multi-Scrobbler seems to have some troubles though with the following error consistently logged:
 2023-01-31T17:04:04+00:00 error : [Source - Spotify - unnamed ] Cannot start polling because user interaction is required for authentication

The setup is via docker-compose, shared below:
version: '3.3'
services:
  multiscrobbler:
    image: foxxmd/multi-scrobbler:latest
    container_name: multiscrobbler
    ports:
      - 9078:9078
    environment:
      - SPOTIFY_CLIENT_ID=redacted
      - SPOTIFY_CLIENT_SECRET=redacted
      - SPOTIFY_REDIRECT_URI=http://multiscrobbler.jansen.sh:9078/callback
      - MALOJA_URL=https://maloja.jansen.sh
      - MALOJA_API_KEY=redacted
      - PLEX_USER=frailtyy
    volumes:
      - multiscrob:/home/node/app/config
    labels:
      - diun.enable=true

volumes:
  multiscrob:

I'm guessing there might be an issue writing the obtained token to a file but it's just a Docker volume, SELinux disabled and everything:
[root@despair docker-multiscrobbler]#  docker exec -it multiscrobbler ls -larth /home/node/app/config
total 36K
-rw-r--r--    1 node     node         726 Apr 28  2022 tautulli.json.example
-rw-r--r--    1 node     node         500 Apr 28  2022 subsonic.json.example
-rw-r--r--    1 node     node        1.2K Apr 28  2022 spotify.json.example
-rw-r--r--    1 node     node         726 Apr 28  2022 plex.json.example
-rw-r--r--    1 node     node         377 Apr 28  2022 maloja.json.example
-rw-r--r--    1 node     node        1.4K Apr 28  2022 lastfm.json.example
-rw-r--r--    1 node     node         610 Apr 28  2022 jellyfin.json.example
-rw-r--r--    1 node     node        1.1K Apr 28  2022 deezer.json.example
-rw-r--r--    1 node     node        1.7K Apr 28  2022 config.json.example
drwxr-sr-x    1 node     node         254 Apr 28  2022 ..
drwxr-xr-x    1 node     node         352 Jan 31 07:18 .

Any thoughts on troubleshooting this one? There's likely an easy fix here staring me right in the face.

If it helps, I'm seeing the following logs in the container:
2023-01-31T17:31:11+00:00 info   : [App                        ] Redirecting to spotify authorization url
2023-01-31T17:42:59+00:00 info   : [Spotify                    ] Received auth code callback from Spotify [{"label":"Spotify"}]
TypeError: Cannot read property 'handleAuthCodeCallback' of undefined
    at file:///home/node/app/index.js:479:50
    at wrappedMiddleware (/home/node/app/node_modules/@awaitjs/express/index.js:116:13)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/node/app/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/node/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at /home/node/app/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/home/node/app/node_modules/express/lib/router/index.js:335:12)
    at next (/home/node/app/node_modules/express/lib/router/index.js:275:10)
    at SessionStrategy.strategy.pass (/home/node/app/node_modules/passport/lib/middleware/authenticate.js:337:9)
2023-01-31T19:55:04+00:00 info   : [Spotify                    ] Received auth code callback from Spotify [{"label":"Spotify"}]
TypeError: Cannot read property 'handleAuthCodeCallback' of undefined
    at file:///home/node/app/index.js:479:50
    at wrappedMiddleware (/home/node/app/node_modules/@awaitjs/express/index.js:116:13)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/node/app/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/node/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at /home/node/app/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/home/node/app/node_modules/express/lib/router/index.js:335:12)
    at next (/home/node/app/node_modules/express/lib/router/index.js:275:10)
    at SessionStrategy.strategy.pass (/home/node/app/node_modules/passport/lib/middleware/authenticate.js:337:9)
2023-01-31T19:55:09+00:00 info   : [Spotify                    ] Received auth code callback from Spotify [{"label":"Spotify"}]
TypeError: Cannot read property 'handleAuthCodeCallback' of undefined
    at file:///home/node/app/index.js:479:50
    at wrappedMiddleware (/home/node/app/node_modules/@awaitjs/express/index.js:116:13)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/node/app/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/node/app/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/node/app/node_modules/express/lib/router/layer.js:95:5)
    at /home/node/app/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/home/node/app/node_modules/express/lib/router/index.js:335:12)
    at next (/home/node/app/node_modules/express/lib/router/index.js:275:10)
    at SessionStrategy.strategy.pass (/home/node/app/node_modules/passport/lib/middleware/authenticate.js:337:9)

Thanks those logs helped. Can you try the foxxmd/multi-scrobbler:develop image and see if that fixes things for you?


Add property for resetPage

Hello,
I noticed that the makeResetUrl of the ResetPassword component look for a "resetPage" property before building the URL that will be sent to the email button.
As I separated the reset and restore function in 2 separates pages, I could define that "resetPage" property in an onInit() method in the php code Tab of my reset page.
But it would be even more convenient to have it available as component property, I wonder why you didn't define it in the defineProperty method, it would be a nice improvement.
Best regards,

Hi @PubliAlex
This must have been an oversight. We've added the resetPage property to the component in v1.6.1 of this plugin.
Thanks!

Hello @daftspunk
Ah nice, that was fast, thank you !


Restructure Vector code in ML-KEM to split functionality

This PR makes the structure of the 3 vector implementations uniform by using the same module names for AVX2, Neon, and Portable. It also breaks up the monolithic portable code in vector.rs into multiple modules.

I added a drive-by fix for a C compilation error (implicit conversion from 'int' to 'short' changes value from 32768 to -32768)

I regenerated and tested the C code, but I would love it if someone else could also check that this works (once CI is happy)


Mention NPM in README

@RomanTsukanov
Now that we have https://www.npmjs.com/package/devextreme-aspnet-data would you please update README accordingly?

Also I think the Installation section needs a better separation between server- and client-side instructions. Subheaders?

Closed in #97


Not able to intall for MacOS M1

No able to install it to MacOS: Ventura 13.4 with M1 processor. Log is attached.
google-cloud-gui_install.log

Any update on the fix for this?


Docs don’t match current code base/API

Thanks for the awesome library however trying to implement it over the past few days I believe I’ve found cases where the docs don’t match the actual code/API. I would like to take a crack at a pull request to help here, but wanted to confirm this is the case — or am I just bonkers?

What did you find?
--
Sent from my iPhone.

@davidjbradshaw sorry I didn't put more detail in my original issue - I was on mobile/on the go.
For example, the docs at https://github.com/davidjbradshaw/iframe-resizer/blob/master/docs/iframed_page/events.md speak of having onReady and onMessage events for the iframed/child page, however I could not get these to work, and looking at the source code I see readyCallback and messageCallback - which I assume are newer names for those events. Example code that works:
window.iFrameResizer = {
        readyCallback: function () {
            window.parentIFrame.sendMessage('IFRAME_LOADED');
        },
        messageCallback: function (message) {
            alert('messageCallback: ' + message);
        }
    };

Oh yes please make a PR for anything like that

Sorry just ready that again, what version of the code are you using?

Just checked the code and it should be onReady and onMessage
https://github.com/davidjbradshaw/iframe-resizer/blob/master/src/iframeResizer.contentWindow.js#L247
This was changed in version 4, in version 3 it was what you have

Oh wow, apologies. You're correct. I thought I had installed latest via npm, but apparently not.


Components labels customization: Toggle, Checkbox, Radio Button.

PR targeted on UI components labels customization. Added labels enabled and disabled colors into corresponding components themes. Saved ability to override it by setting inline styles.

Please add the new labelStyle prop to propTypes and also the docs site. Thanks!

@hai-cea done.

Thanks @misha-panyushkin !


Lines V2

DONE. READY FOR REVIEW.

[X] Convert Line2 to functional API; line2
[X] Convert Line3 to functional API; line3
[X] Enhance line2 API to V2 standards
[X] Enhance line3 API to V2 standards
[X] Add comprehensive tests for line2 API, 100% coverage
[X] Add comprehensive tests for line3 API, 99.26% coverage

Lots of huge cool work in there @z3dev , big kudos :)

@kaosat-dev please review line2 API now. this is done.
I just noticed the comment about transform versus transformMat4. sigh
Actually, line2.transformMat4() actually does transforms on the line using the matrix4. And calls vec2.transformMat4() as well to tranform the points.
Let me know which name is more appropriate.

@kaosat-dev please review the latest code commits.
I have a couple of questions as well.

What to do with the old Line2 and Line3 code? move to the 'moveOutOfCode' directory?
line2.doLinesIntersect() does not belong to this object. this is a calculation for a bounded 2D line segment. I think we should have seg2 and seg3 as well. Can I create seg2, and move this function?


WINC1500 firmware update

It appears that Microchip has recently released a new firmware update (19.7.3):
19.7.3 Release Notes
Firmware FW_19_7_3
Could you please advise if it's safe to update?
Also, I don't have access to a windows machine, is it possible to use the serial_bridge_usb_pitaya_go_vx.x.x.hex with a Linux machine?
Thanks!

@verjus We are testing v19.7.3 and will post a tutorial if it's ready. Currently the used update tool is under windows machine, but a python script will be available soon. Just follow this issue and I will notify you. Thanks.

Answering my own questions.... May be useful to others.
I was able to update the firmware on Linux using the following steps:

nrfjprog -f nrf52 --program serial_bridge_usb_pitaya_go_v1.0.0_mbr.hex --sectorerase
nrfjprog -f nrf52 --reset
Find the serial port using dmesg (e.g. ttyACM2: USB ACM device)
git clone https://github.com/Microchip-MPLAB-Harmony/wireless_wifi.git
cd wireless_wifi/utilities/wifi/winc/tools/
chmod +x winc_programmer_uart
chmod +x image_tool
cd ../../../
ln -s wifi/winc/tools/ .
cd wifi/winc/
./winc_flash_tool.sh -p /dev/ttyACM2 -d WINC1500 -v 19.7.6 -e -x -i prog -w
Done!

This seems to work for me, but proceed at your own risks. I take no responsibility for bricked devices!
Full instructions can be found here


chore: add .gitattributes for cassettes file

Description
Context in Dosu Community Slack
Adds .gitattributes file to mark VRC test cassettes as generated files, so Dosu ignores them when apply size labels.

This worked great, as shown in https://github.com/Future-House/paper-qa/pull/465:


Tincture usage missing from Timeline module

In one of my logs, I used 3 tinctures in the fight with the last one being used near the end of the fight. My last tincture usage is recorded in the "Tinctures" module, but it's missing from the "Timeline" module (Medicated) . I'm guessing it's because the fight ended before the tincture ran out, but not sure. The job being used here is GNB.
Link to the log: https://xivanalysis.com/fflogs/aJ4A2GzHZXBCtjmy/82/1773

This was fixed about 10 days ago, if the issue is still present for you please comment and we'll reopen the issue.


Update quick-create-cli.md

@tysonn : Thanks for your contribution! The author, @iainfoulds, has been notified to review your proposed change.


Add default mode support

Purpose

Add default mode support
Fix https://github.com/ballerinax/kubernetes/issues/495

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

Codecov Report

Merging #507 into master will increase coverage by 0.14%.
The diff coverage is 94.44%.

@@             Coverage Diff              @@
##             master     #507      +/-   ##
============================================
+ Coverage     64.26%   64.40%   +0.14%
- Complexity      950      955       +5
============================================
  Files            98       98
  Lines          4900     4900
  Branches        503      504       +1
============================================
+ Hits           3149     3156       +7
+ Misses         1530     1523       -7
  Partials        221      221

Impacted Files
Coverage Δ
Complexity Δ

...g/ballerinax/kubernetes/models/ConfigMapModel.java
90.00% <66.66%> (+17.27%)
9.00 <4.00> (-1.00)
:arrow_up:

...lerinax/kubernetes/handlers/DeploymentHandler.java
87.05% <100.00%> (+0.23%)
33.00 <0.00> (+2.00)

.../org/ballerinax/kubernetes/models/SecretModel.java
90.00% <100.00%> (+1.11%)
9.00 <1.00> (+1.00)

...netes/processors/ConfigMapAnnotationProcessor.java
78.26% <100.00%> (+0.98%)
14.00 <0.00> (+1.00)

...bernetes/processors/SecretAnnotationProcessor.java
82.41% <100.00%> (+2.87%)
16.00 <0.00> (+2.00)

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 09675d8...8e76d9d. Read the comment docs.

@hemikak Fixed the issues.


Add support for anonymous ciphers

To use it a user must provide their own CipherSuite. Anonymous Ciphers
are vulnerable to MITM attacks so we aren't providing them in public
API.
Resolves #337

https://gist.github.com/Sean-Der/1e4192b37c8a9ef5cf18d6272d22e503
@daenney @jkralik don't bother reviewing this until #337 has been merged. This is that and the anonymous specific changes. I want a review just for this since it is security sensitive. Lots of changes in the state machine etc...

https://gist.github.com/Sean-Der/1e4192b37c8a9ef5cf18d6272d22e503
@daenney @jkralik don't bother reviewing this until #337 has been merged. This is that and the anonymous specific changes. I want a review just for this since it is security sensitive. Lots of changes in the state machine etc...

I just tested the branch against iotivity-lite device (mbedtls) and it works 👍

I just tested the branch against iotivity-lite device (mbedtls) and it works 👍

@jkralik hopefully this covers your needs/exposes everything you need

@jkralik hopefully this covers your needs/exposes everything you need

@daenney mind giving it a quick look over as well, no rush :)

@daenney mind giving it a quick look over as well, no rush :)

@jkralik 99% of the credit goes to you! I was really just cherry-picking your code. Thanks for starting the PR, and sorry it took so long :( I put you down as the co-author

Thx.  I was really just cherry-picking your code. Thanks for starting the PR, and sorry it took so long :( I put you down as the co-author

Thx.  Mind giving it another look over?

@daenney fixed! Mind giving it another look over?


[5.9] Fix backward compatibility in host triple checks

Cherry-pick of https://github.com/apple/swift-package-manager/pull/6819.
When setting host triple in Swift SDKs to arm64-apple-macosx13.0 to allow cross-compiling from an older version of macOS, this triple is not recognized as directly matching arm64-apple-macosx14.0 on a newer version of macOS.
We should support backward compatibility with Swift SDKs that were built for older version of macOS.
(cherry picked from commit 82b1e83728092ff82a1941248410832086b6c88a)
# Conflicts:
#	Sources/PackageModel/SwiftSDKBundle.swift
#	Tests/PackageModelTests/SwiftSDKBundleTests.swift

Resolves rdar://113967401.

@swift-ci smoke test

@swift-ci smoke test

@swift-ci test windows

@swift-ci test windows

@swift-ci test windows


Update Rails to v<IP_ADDRESS>

What happened
Update Rails to the latest stable version v<IP_ADDRESS> to avoid the issue of failing to start because of yanked dependency.
Insight
Rails depends on the MimeMagic gem (ActionMailbox -> ActiveStorage -> Marcel -> Mimemagic). But that gem recently yanked all v0.3.x versions, making the new Rails apps fail to start and the old ones fail to rebuild. Mimemagic also released new versions but changed the Licence from MIT -> GPL2 which isn't compatible with Rails' license anymore.
Rails release a new version v<IP_ADDRESS> to solve that issue by removing Mimemagic dependency.

https://github.com/rails/rails/issues/41750

https://github.com/rails/rails/releases/tag/v<IP_ADDRESS>

https://github.com/rails/marcel/releases/tag/v1.0.0

Proof Of Work
Nimble CX after upgrading to this version no longer experience the failed build due to mimemagic gem.

A small comment, let update the README too :)

Updated in 2d54580.


Cannot back-deploy mangled names including isolated parameters.

Isolated parameters were introduced with concurrency, so don't mangle
names including them in back-deployed code.

@swift-ci please smoke test and merge


adoptopenjdk-bin, zulu8, zulu, graalvm8-ce, graalvm11-ce: remove demo, sample, and broken man pages

Recreating #106806 to rebase on staging.
See: #106716.
Note that AdoptOpenJDK doesn't have demo.

Motivation for this change
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"
[X] 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.

The packages still have demo and sample folders.


[improve] PIP-335: Pulsar with Oxia integration test

PIP: 335
Motivation
Added integration test for Pulsar & BookKeeper cluster with Oxia metadata store.
Modifications

Removed the init-cluster.sh script that was invoked in the ZK container. Instead use a separate container and call pulsar initialize-cluster-metadata from the Java code.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/merlimat/pulsar/pull/11

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

Comparison is base (a83e9ed) 36.63% compared to head (d59c952) 73.69%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #22045       +/-   ##
=============================================
+ Coverage     36.63%   73.69%   +37.06%
- Complexity    12445    32442    +19997
=============================================
  Files          1729     1865      +136
  Lines        132076   138968     +6892
  Branches      14452    15236      +784
=============================================
+ Hits          48381   102415    +54034
+ Misses        77296    28651    -48645
- Partials       6399     7902     +1503

Flag
Coverage Δ

inttests
24.47% <ø> (+0.33%)
:arrow_up:

systests
24.30% <ø> (+0.29%)
:arrow_up:

unittests
72.96% <ø> (+40.93%)
:arrow_up:

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


Add performance measurement to CI/CD pipeline and create GitHub page to check the results

https://medium.com/nerd-for-tech/ci-build-performance-testing-with-github-action-e6b227097c83

Done


feature: outputs analyzer

make a summary file with analysis of some common issues:

full partitions
high uptime
non-emply rabbitmq queues
full swap
low free ram
kernel crashes
etc etc

preliminary implementation since https://github.com/openstack/timmy/commit/abdb347e6b9c5a23a43421228aeb77edc997f0b9


tfrecord export button

I believe tensorflow is a popular enough library where we can build an export button that can generate file.tfrecord files straight from labelimg. Am I crazy or has anyone not thought of this yet? If not, I'd be happy to contribute!

Hi John,
If you can add this, it will be great!


Add download links for datasets

The links for datasets should be downloadable and should not show them as raw files. That's why I think the datasets do not get automatically uploaded to the data library. The links in "Basics of machine learning" tutorial has downloadable links mentioned in the yaml file
Thanks!
ping @bgruening

Cool, thanks are the others trainings also correct?

I am not sure, maybe not .. We have this webinar trial run call now. Once it is over, I will check for other tutorials as well.

@anuprulez can you please check the others. If we merge tm this will be deployed by monday.

@bgruening  I have copied the format from https://github.com/galaxyproject/training-material/blob/master/topics/assembly/tutorials/debruijn-graph-assembly/data-library.yaml and updated the URLs. I cannot test if it works. I checked the links are working and the description is correct.


Allow zmin/zmax again

Sometimes 1 value in the whole lattice got too big, and it messed up the whole scale. I think it's better to allow zmin/zmax optionally, and if not provided use the inferred values

fixed with another PR


deterministic show_doc and DocmentTbl repr

@hamelsmu @jph00 what do you think about this for less polluted diffs?

Excellent idea!


Bump llhttp-ffi to 0.3.0

Updates the llhttp-ffi dependency to v0.3.0. The only difference is that it now uses lhttp v6.0.1, the latest release.

@bryanp do you think I should cut another release with this?

@tarcieri I don't think it's absolutely critical. None of the new features in llhttp are used by http.rb and the bug fixes are unlikely to affect http.rb users. llhttp-ffi v0.0.1 (the version used by the current version of http.rb) uses llhttp v4.0.0 which addressed CVE-2020-8287—looks like no more security related changes since.

@tarcieri @ixti It would be good to cut a release that includes this PR. See https://github.com/metabahn/llhttp/pull/22 (just released to 0.3.1).

Released in v5.0.1


pricing inputs show only numbers on mobile keyboard

https://github.com/bhaden94/va-calculator/commit/c60b6bc5e163ffdbc54e2ff2e45a47ddbbf8aa46


chore: fix prettier and use strict mode

:pencil: Description

:link: Related Issues

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


1.4.0-pre-6: UnboundLocalError: local variable 'texts' referenced before assignment with "-u" option

@Aetopia hi,
Here's an issue I came across with 1.4.0-pre-6:

Contents of "%TEMP%\nvddl.log"
INFO: Namespace: Namespace(help=False, list=True, download=None, extract=None, update=False, studio=False, standard=False, full=False, setup=False, components=[], output='F:\\Temp\\DVALSA~1', flags=[], no_stdout=False)
INFO: Arguments: ['-ls']
INFO: Getting Driver Versions...
INFO: Selected Driver Type: Game Ready
INFO: Selected Driver Type: DCH
INFO: Detected: GP107GLM [Quadro P1000 Mobile]
INFO: API Link Generated: https://www.nvidia.com/Download/processFind.aspx?psid=543&pfid=874&osid=57&lid=1&whql=1&ctk=0&dtcid=1
INFO: Finished!

Contents of "%TEMP%\nvddl_traceback.txt"
Traceback (most recent call last):
  File "F:\Temp\DVALSA~1\ONEFIL~2\main.py", line 19, in main
  File "F:\Temp\DVALSA~1\ONEFIL~2\core\cli.py", line 89, in cli
  File "F:\Temp\DVALSA~1\ONEFIL~2\core\cli.py", line 151, in parse
  File "F:\Temp\DVALSA~1\ONEFIL~2\core\functions.py", line 247, in update
UnboundLocalError: local variable 'texts' referenced before assignment

Dimitrios

Well this is a quick and easy fix.

Fixed in 1.4.0-pre-7. Thank you @Aetopia :)


Integrate B standard extension text

Should wait for ratification prior to merge.
Integrates text from: https://github.com/riscv/riscv-b/blob/main/b.adoc

Is there a need to wait all the way until ratification.  We don't do this for other extensoins (e.g. we include not only frozen extensions but also draft extensions).
From a community perspective, it would seem more desirable to integrate B sooner than later (with of course the standard up top marking as to its official state).

The process discussed in the chairs was to provide a PR as a step in the Ratification Ready milestone qualification. This extension completed public review and is on its way to IC/HC chairs for Ratification Ready vote. I do not think integrating now in Frozen state and updating the state later to Ratified would be an issue.


[theme] Add capacity to apply styleOverrides with styled() instead of with createTheme()

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
There are certain properties that I cannot seem to access from the theme-level customization for a given component that I would like to access and define at the component level with styled() or even the sx prop.
Examples 🌈
Link to Material Design specification: https://mui.com/material-ui/customization/how-to-customize/
Expected implemented behavior:
import { styled } from '@mui/material/styles';
import MuiChip, { ChipProps } from '@mui/material/Chip';

const Chip = styled(MuiChip)<ChipProps>(({ theme }) => ({
  colorPrimary: {
    backgroundColor: theme.palette.primary.light
  },
  colorSecondary: {
    backgroundColor: theme.palette.secondary.light
  }
}));

export default Chip;

Current implemented behavior:
let myTheme = createTheme({
  palette: {
    ...
  },
  typography: {
    ...
  }
});

myTheme = createTheme(myTheme , {
  components: {
    MuiChip: {
      styleOverrides: {
        colorPrimary: {
          backgroundColor: myTheme .palette.primary.light
        },
        colorSecondary: {
          backgroundColor: myTheme .palette.secondary.light
        }
      }
    }
  }
});

Motivation 🔦
I'm trying to accomplish a pure separation of style overrides between the component and theme level. I want to treat the theme I'm creating as more of a way to define global styles and to handle what many designers and developers would consider design tokens (typography, color palette, etc.). And then, at the component-level, I want to be able to handle any customizations for a given component by wrapping it and applying any component-specific styles or usages of the theme being passed in with styled().

I don't really understand what you cannot achieve. Looks like you want to use the styled() instead of the theme.components[*].styleOverrdes, which is perfectly fine and works as expected. You can use the callback for accessing the props & theme. For the slots (the nodes inside the component) you can use class selectors (I believe this is different to the styleOverrides and you are referring to this in terms of difference). See https://mui.com/material-ui/customization/how-to-customize/#2-reusable-component for accessing the theme and adding styles to inner elements (the thumb).

Thanks for sharing the link to the documentation. The following approach seems to be able to work for me for style overrides at the component-level:
const Chip = styled(MuiChip)<ChipProps>(({ theme }) => ({
  '&.MuiChip-colorPrimary': {
    backgroundColor: theme.palette.primary.light
  },
  '&.MuiChip-colorSecondary': {
    backgroundColor: theme.palette.secondary.light
  }
}));

I do think that being able to use the Rule Names, instead of the Global Class, would improve readability of code. For example, the following does not work for me, but then again, I may not be using the right syntax:
const Chip = styled(MuiChip)<ChipProps>(({ theme }) => ({
  colorPrimary: {
    backgroundColor: theme.palette.primary.light
  },
  colorSecondary: {
    backgroundColor: theme.palette.secondary.light
  }
}));

I could see this being particularly useful when defining new variants or removing out-of-the-box variants at the component-level. Again, that may be possible, given the working syntax in the first code snippet in this comment. However, it doesn't seem intuitive, and most of the documentation around customizing component styles is centered around using createTheme and not styled().


Add proposal app skeleton for styling configs handling

@RamezIssac Hello Ramez. Here's what i am thinking of for our styling app inside kn_defaults. Basically we can have overwritable linters/stylers configs and use a command to copy those into project's root.
Also there should be a check to ensure pre-commit app is installed, so all those linters/stylers will work  once user installs pre-commit hook.
Current configs are stolen from Odoo project, and we need to discuss them.

migration files should be excluded from any linter

@jab3z i think they are excluded on all python-related stuff. Is eslint trying to find there something?

@ecugol i said just an added measure, don't know if they are excluded by default.


client certificate does not match Azure AD user

I want to use SSO with Azure AD to login openvpn. After importing profile then input multi-factor authencation. I met error message as below. I don't know how to fix this issue. Thanks for help.

root@vpn:~# openvpn-auth-azure-ad --client-id xxxxxxx -H <IP_ADDRESS> -P 7505 --token-authority https://login.microsoftonline.com/xxxxx.onmicrosoft.com
2022-04-14 18:56:49,815 INFO Connection to OpenVPN management interfaced established.
2022-04-14 18:56:49,816 INFO Running openvpn-auth-azure-ad 1.2.0
2022-04-14 18:56:54,200 INFO [cid: 4]: Received client connect
2022-04-14 18:56:54,200 INFO [cid: 4]: Start to authenticate using device token flow
2022-04-14 18:56:55,286 INFO [cid: 4]: Received client disconnect event
2022-04-14 18:57:21,425 INFO [cid: 5]: Received client connect
2022-04-14 18:57:21,425 INFO [cid: 5]: Continue to authenticate using device token flow
2022-04-14 18:57:21,865 INFO [cid: 5]: client certificate does not match Azure AD user.
2022-04-14 18:57:22,491 INFO [cid: 5]: Received client disconnect event

OpenVPN Clinet log

[Apr 14, 2022, 18:03:18] OpenVPN core 3.git::d3f8b18b win x86_64 64-bit built on Mar 17 2022 11:42:02
⏎[Apr 14, 2022, 18:03:18] Frame=512/2048/512 mssfix-ctrl=1250
⏎[Apr 14, 2022, 18:03:18] UNUSED OPTIONS
4 [resolv-retry] [infinite]
5 [nobind]
6 [persist-key]
7 [persist-tun]
11 [auth-nocache]
13 [ignore-unknown-option] [block-outside-dns]
14 [block-outside-dns]
15 [verb] [3]
⏎[Apr 14, 2022, 18:03:18] EVENT: RESOLVE ⏎[Apr 14, 2022, 18:03:18] EVENT: WAIT ⏎[Apr 14, 2022, 18:03:18] WinCommandAgent: transmitting bypass route to <IP_ADDRESS>
{
"host" : "<IP_ADDRESS>",
"ipv6" : false
}
⏎[Apr 14, 2022, 18:03:18] Connecting to [<IP_ADDRESS>]:1194 (<IP_ADDRESS>) via TCPv4
⏎[Apr 14, 2022, 18:03:18] EVENT: CONNECTING ⏎[Apr 14, 2022, 18:03:18] Tunnel Options:V4,dev-type tun,link-mtu 1523,tun-mtu 1500,proto TCPv4_CLIENT,cipher AES-128-GCM,auth [null-digest],keysize 128,key-method 2,tls-client
⏎[Apr 14, 2022, 18:03:18] Creds: DynamicChallenge
⏎[Apr 14, 2022, 18:03:18] Peer Info:
IV_VER=3.git::d3f8b18b
IV_PLAT=win
IV_NCP=2
IV_TCPNL=1
IV_PROTO=30
IV_CIPHERS=AES-256-GCM:AES-128-GCM:CHACHA20-POLY1305
IV_GUI_VER=OCWindows_3.3.6-2752
IV_SSO=webauth,openurl,crtext
[Apr 14, 2022, 18:03:19] SSL Handshake: peer certificate: CN=server_tQbNQlk72SYn6Xol, 256 bit EC, curve:prime256v1, cipher: TLS_AES_256_GCM_SHA384  TLSv1.3 Kx=any      Au=any  Enc=AESGCM(256) Mac=AEAD
[Apr 14, 2022, 18:03:19] Session is ACTIVE
⏎[Apr 14, 2022, 18:03:19] EVENT: GET_CONFIG ⏎[Apr 14, 2022, 18:03:19] Sending PUSH_REQUEST to server...
⏎[Apr 14, 2022, 18:03:20] Sending PUSH_REQUEST to server...
⏎[Apr 14, 2022, 18:03:20] AUTH_FAILED
⏎[Apr 14, 2022, 18:03:20] EVENT: AUTH_FAILED ⏎[Apr 14, 2022, 18:03:20] EVENT: DISCONNECTED ⏎

Server.conf

port 1194
proto tcp
dev tun
user nobody
group nogroup
persist-key
persist-tun
keepalive 10 120
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
ifconfig-pool-persist ipp.txt
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
push "redirect-gateway def1 bypass-dhcp"
dh none
ecdh-curve prime256v1
tls-crypt tls-crypt.key
crl-verify crl.pem
ca ca.crt
client-cert-not-required
cert server_tQbNQlk72SYn6Xol.crt
key server_tQbNQlk72SYn6Xol.key
auth SHA256
cipher AES-128-GCM
ncp-ciphers AES-128-GCM
tls-server
tls-version-min 1.2
tls-cipher TLS-ECDHE-ECDSA-WITH-AES-128-GCM-SHA256
client-config-dir /etc/openvpn/ccd
status /var/log/openvpn/status.log
verb 3
management localhost 7505
#management socket-name unix [pw-file]
management-client-auth

Hi,
i extend the logging in #9, you may able to see the difference in the future.
The error appears, if the common name different from preferred_username token claim.

It works, thanks!


no av1 support , VaapiVideoDecodeLinuxGL seems to be broken

ubuntu, Thorium Version - latest
chrome://gpu - no av1 support , VaapiVideoDecodeLinuxGL seems to be broken
 --enable-features=VaapiVideoDecoder,VaapiVideoDecodeLinuxGL
here is photo from my chromium - it have hw support all formats

same from thorium with same flags - VaapiVideoDecodeLinuxGL seems to be broken only in thorium

Please test this with ubuntu 22.10 or older (or other linux distro). 23.04 seems to be broken.

ubuntu 22.10 or older (or other linux distro)

the same on 22.10 and 22.04
this does not depend on the distribution in any way, this is most likely a problem of patches that break VaapiVideoDecoder

Wayland?  Hardware video decode on linux/wayland with Chromium-based browsers[*] has been half-working for years  but it seems the last few releases broke it entirely for radeonsi users. Intel and nVidia users seem to be having mixed success.
LIBVA_MESSAGING_LEVEL=2 in /etc/libva.conf yields nothing useful.  Chrome and Edge don't even try.  Thorium tries then falls back:
00:00:00.547 | kVideoDecoderName | "VDAVideoDecoder"
00:00:00.547 | kIsPlatformVideoDecoder | true
00:00:00.547 | info | "Selected VDAVideoDecoder for video decoding, config: codec: h264, profile: h264 high, level: not available, alpha_mode: is_opaque, coded size: [1920,1080], visible rect: [0,0,1920,1080], natural size: [1920,1080], has extra data: false, encryption scheme: Unencrypted, rotation: 0°, flipped: 0, color space: {primaries:BT709, transfer:BT709, matrix:BT709, range:LIMITED}"
00:00:00.547 | pipeline_state | "kPlaying"
00:00:00.547 | pipeline_state | "kSeeking"
00:00:00.547 | pipeline_state | "kPlaying"
00:00:00.551 | error | "VDA Error 4"
00:00:00.551 | warning | "video decoder fallback after initial decode error."
00:00:00.551 | info | "Cannot select VpxVideoDecoder for video decoding"
00:00:00.551 | info | "Cannot select Dav1dVideoDecoder for video decoding"
00:00:00.553 | kIsVideoDecryptingDemuxerStream | false
00:00:00.553 | kVideoDecoderName | "FFmpegVideoDecoder"
00:00:00.553 | kIsPlatformVideoDecoder | false
00:00:00.553 | info | "Selected FFmpegVideoDecoder for video decoding, config: codec: h264, profile: h264 high, level: not available, alpha_mode: is_opaque, coded size: [1920,1080], visible rect: [0,0,1920,1080], natural size: [1920,1080], has extra data: false, encryption scheme: Unencrypted, rotation: 0°, flipped: 0, color space: {primaries:BT709, transfer:BT709, matrix:BT709, range:LIMITED}"

This is thorium-browser-bin 112.0.5615.166-1 from the Arch AUR with amdgpu (Polaris10).  Recently people started having success with LIBVA_DRI3_DISABLE=1 but that too seems to have stopped working just within the past 1-2 weeks.
* except for Firefox... ¯_(ツ)/¯

Adding to the above, launching thorium from the terminal yields the following when playing an h264 vid:
ERR: Display.cpp:1019 (initialize): ANGLE Display::initialize error 12289: Could not make the initialization pbuffer current.
ERR: Display.cpp:1019 (initialize): ANGLE Display::initialize error 12289: Could not make the initialization pbuffer current.
[46886:46886:0519/232242.038868:ERROR:gl_display.cc(504)] EGL Driver message (Critical) eglInitialize: Could not make the initialization pbuffer current.
ERR: Display.cpp:1019 (initialize): ANGLE Display::initialize error 12289: Could not make the initialization pbuffer current.
ERR: Display.cpp:1019 (initialize): ANGLE Display::initialize error 12289: Could not make the initialization pbuffer current.
[46886:46886:0519/232242.040374:ERROR:gl_display.cc(504)] EGL Driver message (Critical) eglInitialize: Could not make the initialization pbuffer current.

Wayland?

x11
on stock chromium - all works fine

For me it stopped working under X11, never worked under wayland. Intel video.
This is what I currently get in console:
[995740:995740:0520/145036.498882:ERROR:vaapi_wrapper.cc(2721)] vaPutSurface failed, VA error: invalid parameter
[995740:995740:0520/145036.498985:ERROR:vaapi_video_decode_accelerator.cc(287)] Failed putting surface into pixmap
[995740:995740:0520/145038.692706:ERROR:vaapi_wrapper.cc(2721)] vaPutSurface failed, VA error: invalid parameter
[995740:995740:0520/145038.692872:ERROR:vaapi_video_decode_accelerator.cc(287)] Failed putting surface into pixmap

And this seems to be the answer: https://bbs.archlinux.org/viewtopic.php?pid=2079752#p2079752

And this seems to be the answe

works in stock chrome
in this patch it does not work

@oksakhartman @dmig Also because of https://github.com/Alex313031/thorium/issues/162 it definitely has to do with those patches. I will remove them one by one and rebuild each time and test, to find the offending one. Then see if I can refactor it to fix, or just remove it all together.


MRG: Add modified Hughson-Westlake tracker

Adding the MWH tracker to the trackers, which is what audiometers use.

Travis and Azure are not happy (ignore AppVeyor for now), and this will need tests.

I've run various tests on the tracker and created an example. It has passed all but 2 tests, and the failures seem unrelated to my code.

The errors seem to be unrelated, but the warnings are real:
/home/circleci/project/doc/auto_examples/index.rst:494: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)
/home/circleci/project/doc/auto_examples/stimuli/sg_execution_times.rst:26: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)
generated/expyfun.analyze.sigmoid.examples:68: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)
generated/expyfun.analyze.sigmoid.examples:76: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)
generated/expyfun.stimuli.TrackerMHW.examples:14: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)
generated/expyfun.stimuli.TrackerMHW.examples:22: WARNING: undefined label: sphx_glr_auto_examples_stimuli_tracker_staircase_mhw.py (if the link has no caption the label must precede a section header)

Take a look also at how the link to the example is rendering on the test server:
https://671-11614571-gh.circle-artifacts.com/0/html/auto_examples/index.html#stimulus-generation-examples

@mpolonenko now that #403 is merged can you rebase? It will make Travis run properly

@larsoner yes, I pushed the changes you suggested, and I'll learn how to rebase

@larsoner all tests have now passed except codecov...not sure why _tracker.py is reducing coverage? Is there something else I should change? Thanks for the help.

@larsoner thanks for the link. I’ve made more tests for the tracker and this updated version passed all tests, including coverage.

LGTM. Nice addition :)

Thanks @mpolonenko !


[FEATURE REQ] SecretClient should support existing MSAL authenticated Sessions

Is your feature request related to a problem? Please describe.
Documentation currently describes there are several Credential classes used to provide authorization to the SecretClient as shown here.
None of these solutions allows me to use a user principal that I have already established.
Describe the solution you'd like
After authenticating a user with the MSAL library I want to be able to reuse the existing context to access other resources (like Azure Key Vault). I do not want my users to be prompted multiple times during their session in my client application.
           var publicClientApp = PublicClientApplicationBuilder.Create(AzureAppRegistrationConfig.ClientId)
                        .WithAuthority(AzureCloudInstance.AzurePublic, AzureAppRegistrationConfig.TenantId)
                        .WithRedirectUri("http://localhost")
                        .Build();

           var authResult = publicClientApp.AcquireTokenInteractive(_scopes)
                        .WithAccount(accounts.FirstOrDefault())
                        .WithPrompt(Prompt.SelectAccount)
                        .WithUseEmbeddedWebView(false) //without this line the new window prompt can sometimes fail to open and the method will hang/do-nothing
                        .WithSystemWebViewOptions(options)
                        .ExecuteAsync(cancellationToken).Result;

Now that I have an authResult (and token) I'd like to give this context to the SecretClient object so that I can access Key Vault Secrets.
Describe alternatives you've considered
I have switched to using the InteractiveBrowserCredential object instead of using the above PublicClientApplicationBuilder.
Additional context
The documentation does not show whether or not this is a feature that already exists.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x ] Description Added
[x ] Expected solution specified

@schaabs can you take a look?

@KSchlobohm Thank you for the feedback. As you have already discovered the InteractiveBrowserCredential credential provides the interactive authentication flow. If this serves your purposes I'm glad to hear it. If not we welcome any feedback on how it might better fit your scenario.
The InteractiveBrowserCredential is actually built on top of the MSAL PublicClientApplication, but it doesn't expose the MSAL client through it's API. In the case you're application requires more control over authentication, or you need to share the PublicClientApplication instance with other portions of your program which authenticate clients outside of the Azure SDK, it is possible to provide your own TokenCredential implementation to use with the SecretClient or any other client which supports authenticating with a TokenCredential.
Here is an example of an implementation which would perform interactive authentication when first called, and then silently authenticate any subsequent calls. It is modeled after the InteractiveBrowserCredential implementation.
    public class MyPublicApplicationCredential : TokenCredential
    {
        private readonly IPublicClientApplication _publicClientApp;
        private readonly IAccount _account;
        private SystemWebViewOptions _options;

        public MyPublicApplicationCredential(IPublicClientApplication publicClientApp, SystemWebViewOptions options)
        {
            _publicClientApp = publicClientApp;
            _options = options;
        }

        public override AccessToken GetToken(TokenRequestContext requestContext, CancellationToken cancellationToken)
        {
            return GetTokenAsync(requestContext, cancellationToken).GetAwaiter().GetResult();
        }

        public override async ValueTask<AccessToken> GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
        {
            if (_account != null)
            {
                try
                {
                    AuthenticationResult result = await _publicClientApp.AcquireTokenSilent(requestContext.Scopes, _account).ExecuteAsync(cancellationToken).ConfigureAwait(false);

                    return new AccessToken(result.AccessToken, result.ExpiresOn));
                }
                catch (MsalUiRequiredException)
                {
                    return await GetTokenInteractiveAsync(requestContext.Scopes, cancellationToken).ConfigureAwait(false);
                }
            }
            else
            {
                return  await GetTokenInteractiveAsync(requestContext.Scopes, cancellationToken).ConfigureAwait(false);
            }
        }

        private async Task<AccessToken> GetTokenInteractiveAsync(string[] scopes, CancellationToken cancellationToken)
        {
            var authResult = await _publicClientApp.AcquireTokenInteractive(scopes)
                         .WithPrompt(Prompt.SelectAccount)
                         .WithUseEmbeddedWebView(false)
                         .WithSystemWebViewOptions(_options)
                         .ExecuteAsync(cancellationToken);

            _account = authResult.Account;
            return new AccessToken(authResult.AccessToken, authResult.ExpiresOn);
        }

Your application could then use your custom credential type to instantiate Azure SDK clients such as the SecretClient.
    var publicClientApp = PublicClientApplicationBuilder.Create(AzureAppRegistrationConfig.ClientId)
                        .WithAuthority(AzureCloudInstance.AzurePublic, AzureAppRegistrationConfig.TenantId)
                        .WithRedirectUri("http://localhost")
                        .Build();

    var myCred = new MyPublicApplicationCredential(publicClientApp, options);

    var secretClient = new SecretClient("https://myvault.vaults.azure.net", myCred);

Thanks @schaabs, the MyPublicApplicationCredential example you provided is the class I was expecting the framework to provide.
In my situation, the InteractiveBrowserCredential class is even easier to work with but in other situations where I would want to use the authResult to access the authenticated user's information this MyPublicApplicationCredential class demonstrates a more flexible approach.
Is this a class that can be added to the documentation and/or Azure.Identity NuGet package?

@KSchlobohm we probably won't add this class to Azure.Identity at least at this time as we are using the MSAL client more as an implementation detail and we have consciously avoided exposing it's API through Azure.Identity to give us flexibility in the future. However I do agree that we should have a sample which demonstrates creating and using such a credential type. I've created issue https://github.com/Azure/azure-sdk-for-net/issues/8694 to track this.

Thanks @schaabs that makes sense!


Bad link to SLG on API web page

At https://luxcorerender.org/luxcore-api/ there is a link to "SLG (aka SmallLuxGPU)" http://www.luxrender.net/wiki/SLG
It looks like the previous owners of luxrender.net lost control of that domain, as it now just shows ads.
Here is a link to archived version of site https://web.archive.org/web/20180413222648/http://www.luxrender.net/wiki/SLG

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Thanks, fixed.


TiddlyWiki multiple active notes

Setup
Validated in:
Browser: Chromium 112.0.5615.49
Editor: Sublime Text Build 4143
But I also validated it in my main setup:
Browser: Mozilla Firefox 111.0.1
Editor: vim 9.0
Description
This is a followup on #236 and #285.
I just tried the new TiddlyWiki-focused frame updates with web-ext --target=chromium, and it works great for one note, but it cannot sync a second note.
I ran into a similar issue when I was writing PR #284, but I suspect the solution is going to be different because, I think, a separate instance of GhostText runs in each iframe?
Reproducibility Steps

Open https://tiddlywiki.com
Start editing one note
Start GhostText
Start editing a second note
Try to start GhostText and select the second note (will not start synching successfully)

Strangely enough, the following steps will start synching two notes?

Open https://tiddlywiki.com
Start editing one note
Start editing a second note
Start GhostText

Next Steps
I want to see if I can diagnose the problem, but I figured I'd post this issue here.

Thanks for the report. This issue is described by the last point here:

https://github.com/fregante/GhostText/issues/204

In short: GhostText will only be executed once per tab, so if frames are added they can’t be activated.
That “was injected” check actually already runs once per iframe, but only the first result is considered. PR welcome


Fix median calculation in Redshift

Description & motivation

Median function and some other aggregate types won't work in context of distinct (per #84 ). Note, there is no new macro, so no tests to write, or need to update the README. There is nothing new the user needs to do, other than what is already listed in the documentation for dbt-profiler.
Checklist

[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered):

[ ] Postgres
[ ] BigQuery
[ ] Snowflake
[x] Redshift
[ ] SQL Server
[ ] Databricks

[ ] I have written tests for new macros (either as dbt schema tests in integration_tests/models, dbt data tests in integration_tests/tests or integration tests in the CI workflow)
[ ] I have updated the README.md (if applicable)

Thank you! The code looks good. I'm merging this now.


xSQLServerSetup: When running InstallFailoverCluster with BrowserSvcStartupType installation will fail

Details of the scenario you tried and the problem that is occurring:
When running a cluster installation (InstallFailoverCluster) with the BrowserSvcStartupType set the installation will fail with the following message in summary.txt:
The setting 'BROWSERSVCSTARTUPTYPE' is not allowed when the value of setting 'ACTION' is 'InstallFailoverCluster'.
While it would be simple in most circumstances to simply remove the setting I am writing a config which can be either a standalone node or cluster and want to minimize duplication.  I believe that it would be more intuitive to simply ignore the value during the set command
I had a quick look at a fix: in MSFT_xSQLServerSetup.psm1 line 1160
    if ($null -ne $BrowserSvcStartupType)
    {
            $argumentVars += 'BrowserSvcStartupType'
    }

To:
    if ($null -ne $BrowserSvcStartupType)
    {
        if ($Action -in @('Install','InstallFailoverCluster','PrepareFailoverCluster','CompleteFailoverCluster'))
        {
            Write-Verbose "BrowserSVCStartupType Ignored as unsupported by SQL Installer"
        }
        else
        {
            $argumentVars += 'BrowserSvcStartupType'
        }
    }

I'm happy to create a PR if people are happy with the change
The DSC configuration that is using the resource (as detailed as possible):
xSQLServerSetup "InstallSQLServer"
{
Action = "InstallFailoverCluster"
InstanceName = $Node.SQLServerInstanceName
Features = "SQLENGINE"

FailoverClusterNetworkName = $Node.SQLServerClusterNetworkName
FailoverClusterIPAddress = $Node.SQLServerClusterIPAddress

BrowserSvcStartupType = "Automatic"

#DSCConfig
DependsOn = $FeatureResources
PsDscRunAsCredential = $Node.InstallCredential
}

Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Windows Server 2016
WMF 5.1
SQL 2016
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
xSQLServer
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
DEV

Thanks for submitting this issue!
I would like opinions from the community on this one.
If we do this change, then we need to do all other variations as well. This would create a lot of logic for all kind of different setting options that are not permitted in all kind of different scenarios. That would become a lot of logic code and test code. If there would be logic for this, I would want to throw an error saying to the user that the configuration is not correct, not just skipping the problem.
But thinking about it I rather just add to the parameter description, in the README.md, that the parameter is not allowed when installing a failover cluster.
Lets hear what the community thinks before deciding.

Looking at the code itself it looks like the majority of the fields already have the conditional inclusion which I describe.  A good example is the cluster network name here or the inclusion of analysis services here.
While I'm happy to defer to the wider community the inability to splat parameters into DSC resources would cause a massive code bloat for me as a resource would be required for each combination of feature and cluster type.  IMO this dynamic use of parameters is one of the strongest features of the entire Resource Pack.

When looking at this again I think those parameters that are ignored in the resource today is the wrong practice. I think if a property is assigned a value in a configuration it should be uphold. If it can't an error should be thrown. As it is now a configuration can have values that are just ignored and it looks like the configuration is set when it is not.
I think a configuration should not contain properties that can't be set. I think the resource should throw an error message saying that the configuration can never be in desired state.

The issue is that if you specify the value in the installer as automatic the installer will fail even though the installer actually configures it as automatic.  How about checking for the value and if set to automatic the parameter is dropped to allow it though and if it isn't then throw an invalid argument exception?

I think since the property is not an acceptable argument with that action (see Integrated Install Failover Cluster Parameters) I don't think we should add code that allows it to be passed. I rather is throws if the parameter is used when /ACTION is 'InstallFailoverCluster'.


Upgrade to 5.1.3

Redo of #1124

This looks like a duplicate of #1206

Sorry, didn't even check on that 🤦


Upgrade Pixie integration

Upgrade the version of  the Pixie integration

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.icorrales seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Rework plot backend picking especially for matplotlib

This PR reworks the plot backend picking to simplify it and solve issues with matplotlib backend
closes #2821

Does your PR is compatible, in case i want a curve not to be selectable?

Yes, the selectable/draggable properties are handled in PlotWidget, the backend does not have to handle it.
This removes some left over in the backend that created issues.
The OpenGL backend was already not using selectable and draggable argument while the matplolib backend was using them to make matplotlib items pickable.
This creates issue because in some situations you want to be able to pick an item no matter if it is selectable or not.

matplotlib picking fixed.
ready for review.

Added a few tests for picking with scatter grid visualizations (which use the picking underneath)


AppImage fails to build on Arch Linux

When running npm run tauri build on Arch Linux, the AppImage fails to build with:
Error failed to bundle project: error running appimage.sh

Running with the verbose flag shows the underlying issue (truncated for brevity):
Calling strip on library lrcget.AppDir/usr/lib/libxslt.so.1

ERROR: Strip call failed: /tmp/appimage_extracted_f792e8e8b92755a0433a3895ce0e35fc/usr/bin/strip: lrcget.AppDir/usr/lib/libxslt.so.1: unknown type [0x13] section `.relr.dyn'

/tmp/appimage_extracted_f792e8e8b92755a0433a3895ce0e35fc/usr/bin/strip: Unable to recognise the format of the input file `lrcget.AppDir/usr/lib/libxslt.so.1'

Calling strip on library lrcget.AppDir/usr/lib/libyuv.so
Calling strip on library lrcget.AppDir/usr/lib/libzstd.so.1
       Error [tauri_cli_node] failed to bundle project: error running appimage.sh

I did some digging and it looks like it's an issue with linuxdeploy and has caused issues in other libraries depending on it. This likely hasn't been encountered yet as the last commit was prior to the linuxdeploy update in Debian. I suspect by now this change is in Debian and the next build will fail. Until the issue is resolved in linuxdeploy, the AppImage will compile if NO_STRIP=true is in the build environment.

There are more similar issues on upstream such as this one: https://github.com/tauri-apps/tauri/issues/8929, they also suggest NO_STRIP=true.


"Last seen active" text and color indicator chnages.

-Contacts are showing Last seen a few seconds ago while grey indicator? This is incorrect. This may be as these are contacts before we upgraded and so their data structure is different. But as we know they were seen a few seconds ago then we should have a green indicator. However in this specific scenario, I think that these contacts are not active for a while and so there indicator is correct while the statement is not correct.

-I don’t see the logic for seen few seconds ago. It should only be in days. Last seen today and otherwise last seen x days back. In dev also the time may be in minutes but it should show days as text. So less that 1 min in Dev is same as less than 1 day in Live and so should say, Last seen today.

@CypherPoet this is because .fromNow() for undefined returns "a few seconds ago"

Noting here that after discussing this, we've decided to not show the "last seen text" if the time is undefined. (The grey indicator will still display, though)

-I'm still able to see the text "Last seen a few seconds ago" only in android device.
-Tested on Dev app version 1.3.2(168)

Verified this issue on Dev app version 1.3.2(170)
-Now it is showing "last seen today" on both the devices.


esp8266 was not declared in this scope

I am using the above code
`#include
include
include
define SSID "SSID"
define PASSWORD "Password"
SoftwareSerial mySerial(3, 2); /* RX:D3, TX:D2 */
ESP8266 wifi(mySerial);
void setupesp8266()
{
while (1) {
Serial.print("restaring esp8266...");
if (esp8266.restart()) {
Serial.print("ok\r\n");
break;
}
else {
Serial.print("not ok...\r\n");
Serial.print("Trying to kick...");
while (1) {
if (esp8266.kick()) {
Serial.print("ok\r\n");
break;
}
else {
Serial.print("not ok... Wait 5 sec and retry...\r\n");
delay(5000);
}
}
}
}
Serial.begin(9600);
mySerial.begin(115200);
Serial.print("setup begin\r\n");
Serial.print("FW Version: ");
Serial.println(esp8266.getVersion().c_str());
if (esp8266.setOprToStation()) {
Serial.print("to station ok\r\n");
break;
} else {
Serial.print("to station err\r\n");
Serial.println("Wait 5 seconds and try again...");
delay(5000);
}
while(1){
if (esp8266.joinAP(SSID, PASSWORD)) {
Serial.print("Join AP success\r\n");
Serial.print("IP: ");
Serial.println(wifi.getLocalIP().c_str());
break;
} else {
Serial.print("Join AP failure\r\n");
Serial.println("Wait 5 seconds and try again...");
delay(5000);
}
}
Serial.print("setup end\r\n");
}
void setup(void)
{
setupesp8266();
}
void loop(void)
{
}
`
but is giving following error
ConnectWiFi:12: error: 'esp8266' was not declared in this scope
 if (esp8266.restart()) {

     ^

ConnectWiFi:35: error: 'esp8266' was not declared in this scope
Serial.println(esp8266.getVersion().c_str());
            ^

ConnectWiFi:38: error: break statement not within loop or switch
 break;

 ^

Using library ESP8266Wifi in folder: C:\Users\admin\Documents\Arduino\libraries\ESP8266Wifi (legacy)
Using library SoftwareSerial at version 1.0 in folder: C:\Program Files (x86)\Arduino\hardware\arduino\avr\libraries\SoftwareSerial
exit status 1
'esp8266' was not declared in this scope
please help me

Hi..
this code is error in various place.where are you get this code
`#include ------------- include some thing (esp8266.h or wifi.h)
#define SSID "SSID"---------------------- place Here your network SSID and password
#define PASSWORD "Password"
SoftwareSerial mySerial(3, 2); /* RX:D3, TX:D2 */------------------------ Include softwareserial in esp8266.h file
void setupesp8266()--------------------- don't understand about esp8266
You have to write a clear and perfect sketch.

I had this problem with the variables if their value does not fit or they are out of place, eg ledPin = D4, I simply set ledPin = 4 and that was resolved.


chore: debug jest in vscode

ref #360
I can't match file in original way, and https://github.com/microsoft/vscode-recipes/pull/108 added a workaround configuration for windows users. doc here. I work fine in the config, but i didn't test other systems.

I'm sorry for submitting this pr in a hurry without check diff, i rebase the commit.


OSR-deopt thrashing

The tests
standalone/io/large_file_read_small_file_test
vm/dart/spawn_uri_from_kernel_blob_test

are failing on configurations
vm-eager-optimization-linux-release-ia32

/======================================================================================\
| standalone/io/large_file_read_small_file_test broke (Pass -> Timeout, expected Pass) |
\======================================================================================/

--- Command "vm" (took 08:00.000111s):
DART_CONFIGURATION=ReleaseIA32 out/ReleaseIA32/dart --optimization-counter-threshold=5 --random-seed=1524871612 --sound-null-safety -Dtest_runner.configuration=vm-eager-optimization-linux-release-ia32 --ignore-unrecognized-flags --packages=/b/s/w/ir/.dart_tool/package_config.json /b/s/w/ir/tests/standalone/io/large_file_read_small_file_test.dart

...

--- Re-run this test:
python3 tools/test.py -n vm-eager-optimization-linux-release-ia32 standalone/io/large_file_read_small_file_test

--trace-compiler --trace-deoptimization shows repeated OSR and deopt at the same BinarySmiOp or GuardField (depending which test). Usually a deopt would add a deopt reason to the ICData to prevent the optimization from being repeated.
@alexmarkov

Small repro:

main() {
  for (int i = 0; i < 10000000; ++i) {
    if (i > 2*1024*1024*1024) {
      print(1);
    }
    if (i % 1000000 == 0) {
      print(i);
    }
  }
  print('ok');
}

ApplyICData pass creates a few BinarySmiOp instructions:
      v9 <- Constant(#2) T{_Smi}
      v10 <- Constant(#1024) T{_Smi}
...
    v11 <- BinarySmiOp:28(*, v10, v9) T{_Smi}
    v12 <- BinarySmiOp:30(*, v11, v10) T{_Smi}
    v13 <- BinarySmiOp:32(*, v12, v10) T{_Smi}

After that, Canonicalize pass converts them to
      v55 <- Constant(#2097152) T{_Smi}
      v56 <- Constant(#10)
...
    v57 <- BinarySmiOp:32(<<, v55, v56) T{_Smi}

Then, LICM pass moves this instruction out of the loop and assignes a different deop_id:
    v57 <- BinarySmiOp:78(<<, v55, v56) T{_Smi}

The new deop_id doesn't have a corresponding descriptor with kIcCall  kind, so AddDeoptReason is not called for it when deoptimization happens.
Also, ICData::kHoisted / ProhibitsHoistingCheckClass mechanism which would prevent LICM in case of deopt only works for a few instructions, but not BinarySmiOp.
We can probably extend ICData::kHoisted / ProhibitsHoistingCheckClass mechanism to BinarySmiOp which should disable LICM on the first deopt and then disable BinarySmiOp on the 2nd deopt.
@rmacnak-google @mraleph @mkustermann WDYT?

I think proposed solution is fine.


Check database has table before altering.

Work around to prevent several issues with silverstripe-framework#4451 and cwp#0031726.
Preferred to fix it here globally as otherwise checks were needed for framework, versioned and CMS.
Parent Issue

https://github.com/silverstripe/silverstripe-framework/issues/7660

As per the comment originally, I could check hasTable at the callee level but it means patches for several of the modules (and in the short term mantaining forks for 3+ repos I've found this issue in.
This keeps the change clean and reduces the amount of boilerplate code each module needs to do (i.e I don't think you would ever not check before altering)

I would rather renameTable throws a LogicException.

I think I concur with this. You run the risk of the application silently ignoring what could be a real bug.

Ok fair.. 2 v 1 democratic process and all. I've updated to throw a LogicException and will update the submodules individually to listen for it and update accordingly.


Testing on Edge/IE and Mobile Platforms

Embed needs to be tested on all widely used platforms.

DartPad embed does not currently work on IE. Seems to be a concurrency issue.

Probably cause by the script loading in too early.
https://github.com/Polymer/docs/issues/1187

https://github.com/Polymer/docs/pull/1256/files#diff-3539bcefe74ec5abf435ecd46e5ad367R159

https://github.com/Polymer/vulcanize/issues/113

Imports.html vulcanization causes Polymer to fail horribly.

Addressed


stix2_validator.py doesn't work with examples found on https://oasis-open.github.io/cti-documentation/stix/examples.html

Running the sample stix2.1 reports found here fails using https://github.com/oasis-open/cti-stix-validator/blob/master/stix2validator/scripts/stix2_validator.py:

https://oasis-open.github.io/cti-documentation/examples/example_json/apt1.json
https://oasis-open.github.io/cti-documentation/examples/example_json/poisonivy.json

Outputs:
stix2_validator.py apt1.json
================================================================================
[-] Results for: apt1.json
[X] STIX JSON: Invalid
    [X] Fatal Error: 'charmap' codec can't decode byte 0x9d in position 15181: character maps to <undefined>

stix2_validator.py poisonivy.json
================================================================================
[-] Results for: poisonivy.json
[X] STIX JSON: Invalid
    [!] Warning: malware--591f0cb7-d66f-4e14-a8e6-5927b597f920: {101} Custom property 'is_family' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: malware--591f0cb7-d66f-4e14-a8e6-5927b597f920: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: malware--591f0cb7-d66f-4e14-a8e6-5927b597f920: {101} Custom property 'malware_types' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: identity--81cade27-7df8-4730-836b-62d880e6d9d3: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: campaign--752c225d-d6f6-4456-9130-d9580fd4007b: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: campaign--d02a1560-ff69-49f4-ac34-919b8aa4b91e: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: campaign--721976f9-56d7-4749-8c69-b3ac7c315f05: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.
    [!] Warning: attack-pattern--19da6e1c-69a8-4c2f-886d-d620d09d3b5a: {101} Custom property 'spec_version' should have a type that starts with 'x_' followed by a source unique identifier (like a domain name with dots replaced by hyphen), a hyphen and then the name.

etc..

was running an older version of the library (v1.1.2) vs v2.0.2


request: TreeIndex::peek_entry

Currently it's impossible to retrieve inserted key from TreeIndex
pub fn peek_entry<'g, Q>(&self, key: &Q, guard: &'g Guard) -> Option<(&'g K, &'g V)>
    where
        K: Borrow<Q>,
        Q: Ord + ?Sized,

should solve this.

=> SCC 2.1.18


SystemStackError in did_you_mean gem when using undef

This is with ruby 2.3.0preview1 (2015-11-11 trunk 52539) [x86_64-linux] which uses did_you_mean-1.0.0.beta3. I was about to file a bug at bugs.ruby-lang.org but then thought here might be better.
Example:
undef :a

Output when using ruby -d:
Exception `LoadError' at /home/justin/.rvm/rubies/ruby-2.3.0-preview1/lib/ruby/2.3.0/rubygems.rb:1241 - cannot load such file -- rubygems/defaults/operating_system
Exception `LoadError' at /home/justin/.rvm/rubies/ruby-2.3.0-preview1/lib/ruby/2.3.0/rubygems.rb:1250 - cannot load such file -- rubygems/defaults/ruby
Exception `LoadError' at /home/justin/.rvm/rubies/ruby-2.3.0-preview1/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:54 - cannot load such file -- did_you_mean
Exception `SystemStackError' at /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:10 - stack level too deep
/home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:10:in `to_s': stack level too deep (SystemStackError)
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/spell_checkers/method_name_checker.rb:15:in `initialize'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:25:in `new'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:25:in `spell_checker'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:21:in `corrections'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:14:in `to_s'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/spell_checkers/method_name_checker.rb:15:in `initialize'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:25:in `new'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:25:in `spell_checker'
	 ... 9010 levels...
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:25:in `spell_checker'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:21:in `corrections'
	from /home/justin/.rvm/gems/ruby-2.3.0-preview1@global/gems/did_you_mean-1.0.0.beta3/lib/did_you_mean/core_ext/name_error.rb:14:in `to_s'
	from test.rb:1:in `<main>'

This example came from trying to use safe_yaml which loads psych/deprecated which uses undef here: https://github.com/tenderlove/psych/blob/c1decb1fef5c77d84a93d5ac19a4961da914dd5c/lib/psych/deprecated.rb#L81

@presidentbeef Thanks for reporting! I'll take a look.

As of https://github.com/yuki24/did_you_mean/commit/d71740ac822af0badcbcefbe2ffc319068768a70, this gem no longer uses TracePoint API, thus this issue should be gone now. Thanks again for reporting!


Porch failing to connect to remote Git private repo with self-signed SSL certs

Expected behavior
kpt alpha repo register  should register a new, remote Git repo as explained in Porch instructions:
https://kpt.dev/guides/porch-user-guide
Actual behavior
The two registered repos, app-health-5g-blueprint and app-health-5g-deployment, are not ready.
$ kpt alpha repo list
NAME                       TYPE   CONTENT   DEPLOYMENT   READY   ADDRESS
app-health-5g-blueprint    git    Package                False   https://my-gitlab.com/dimitar.georgievski/blueprint.git
app-health-5g-deployment   git    Package   true         False   https://my-gitlab.com/dimitar.georgievski/deployment.git
free5gc-packages           git    Package   false        True    https://github.com/nephio-project/free5gc-packages.git
mgmt                       git    Package   true         True    http://<IP_ADDRESS>:3000/nephio/mgmt.git
mgmt-staging               git    Package   false        True    http://<IP_ADDRESS>:3000/nephio/mgmt-staging.git
nephio-example-packages    git    Package   false        True    https://github.com/nephio-project/nephio-example-packages.git

Information

kpt version: 1.0.0-beta.38
Porch version: v0.0.20

Steps to reproduce the behavior
kpt and Porch were deployed as part of nephio sandbox deployment in a kind cluster installed on a VM box:
https://github.com/nephio-project/docs/tree/main/install-guide#installing-on-a-pre-provisioned-vm
I tried to register two private GitLab repositories with self-signed SSL certificate which caused the following error on the Porch server side
cannot fetch repository default/app-health-5g-blueprint: Get "https://my-gitlab..com/dimitar.georgievski/blueprint.git/info/refs?service=git-upload-pack":
tls: failed to verify certificate: x509: certificate signed by unknown authority

Could Porch server be configured to either skip SSL validation of the remote Git server or deploy the custom CA/SSL certificate?

It is worth mentioning that as long as the not-ready repos are registered with Porch operations with other ready repos are not possible which should be considered a bug.
$ kpt alpha rpkg get --name nephio-workload-cluster -v 10
I1208 15:44:38.625379 2251690 loader.go:374] Config loaded from file:  /home/ubuntu/.kube/config
I1208 15:44:38.641047 2251690 round_trippers.go:466] curl -v -XGET  -H "Accept: application/json;as=Table;g=meta.k8s.io;v=v1,application/json" -H "User-Agent: kpt/1.0.0-beta.38" 'https://<IP_ADDRESS>:34759/apis/porch.kpt.dev/v1alpha1/namespaces/default/packagerevisions?fieldSelector=spec.packageName%3Dnephio-workload-cluster'
I1208 15:44:38.641714 2251690 round_trippers.go:510] HTTP Trace: Dial to tcp:<IP_ADDRESS>:34759 succeed
I1208 15:44:38.700053 2251690 round_trippers.go:553] GET https://<IP_ADDRESS>:34759/apis/porch.kpt.dev/v1alpha1/namespaces/default/packagerevisions?fieldSelector=spec.packageName%3Dnephio-workload-cluster 500 Internal Server Error in 58 milliseconds
I1208 15:44:38.700113 2251690 round_trippers.go:570] HTTP Statistics: DNSLookup 0 ms Dial 0 ms TLSHandshake 15 ms ServerProcessing 42 ms Duration 58 ms
I1208 15:44:38.700132 2251690 round_trippers.go:577] Response Headers:
I1208 15:44:38.700161 2251690 round_trippers.go:580]     Content-Length: 339
I1208 15:44:38.700193 2251690 round_trippers.go:580]     Audit-Id: 0c956862-a3ea-4d16-a9b3-303314203984
I1208 15:44:38.700217 2251690 round_trippers.go:580]     Audit-Id: 0c956862-a3ea-4d16-a9b3-303314203984
I1208 15:44:38.700234 2251690 round_trippers.go:580]     Cache-Control: no-cache, private
I1208 15:44:38.700275 2251690 round_trippers.go:580]     Cache-Control: no-cache, private
I1208 15:44:38.700304 2251690 round_trippers.go:580]     Content-Type: application/json
I1208 15:44:38.700327 2251690 round_trippers.go:580]     Date: Fri, 08 Dec 2023 15:44:38 GMT
I1208 15:44:38.700349 2251690 round_trippers.go:580]     X-Kubernetes-Pf-Prioritylevel-Uid: 6abfd417-66e8-45a9-9542-54d899bea11d
I1208 15:44:38.700372 2251690 round_trippers.go:580]     X-Kubernetes-Pf-Prioritylevel-Uid: 6abfd417-66e8-45a9-9542-54d899bea11d
I1208 15:44:38.700395 2251690 round_trippers.go:580]     X-Kubernetes-Pf-Flowschema-Uid: 18e84185-1b8a-43b9-a989-8cec2e934432
I1208 15:44:38.700417 2251690 round_trippers.go:580]     X-Kubernetes-Pf-Flowschema-Uid: 18e84185-1b8a-43b9-a989-8cec2e934432
I1208 15:44:38.700530 2251690 request.go:1154] Response Body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"cannot fetch repository default/app-health-5g-blueprint: Get \"https://my-gitlab.com/dimitar.georgievski/blueprint.git/info/refs?service=git-upload-pack\": tls: failed to verify certificate: x5           509: certificate signed by unknown authority","code":500}
I1208 15:44:38.700661 2251690 request.go:1345] body was not decodable (unable to check for Status): no kind "Status" is registered for version "v1" in scheme "pkg/runtime/scheme.go:100"
Error: an error on the server ("unknown") has prevented the request from succeeding (get packagerevisions.porch.kpt.dev)

I consider support for self-signed certs more of a feature request.
Porch has moved to Nephio, so we should recreate this issue there.

@johnbelamaric
I work at Casa Systems which is a participating member of Nephio LF networking foundation. Considering we are very interested in using Porch and Nephio it would be great if we could help with implementing and testing this Porch enhancement.
I went through the Porch code and have the impression from todo comments you are planning to add abstractions for git repositories. I am not sure if our membership status would allow us to contribute to planning, but if nothing else we could help with testing. Please let me know.

Hi @dgeorgievski, please raise this as a feature request here: https://github.com/nephio-project/nephio/issues. You can reference this issue there, but that's where all future work will be done.
You are more than welcome to join in the planning and execution of any of this. There are no organizational requirements for Nephio for day-to-day choices, only really for the TSC which are very high-level directional choices. Just jump in the pool, the water is warm!
You may want to read up on my proposed roadmap for Porch:
https://github.com/nephio-project/nephio/discussions/450
None of that is set in stone; it is just my initial suggestion. The community will decide where to take it from there.
FYI you may also want to take a look at the status of the transition:
https://github.com/nephio-project/nephio/issues/465
The transition is in effect already; for R2 (releasing very soon), Nephio will use the Porch code built from the Nephio repository.

Great. We are looking to wet our feet :-)


Components Panel: fix for widget visibility depende to screen shape

Issue: https://github.com/Samsung/TAU-Design-Editor/issues/281
Problem: Selector widget is not displayed on the widget list for first load
Solution:

change for render method in case too much calls in short time
change in method for filtering components by profile and screen shape

Signed-off-by: Tomasz Lukawski<EMAIL_ADDRESS>

The PR fixes the problem, but during opening wearable project I can see a mobile component for a short moment. Maybe we should increase insensitive time?

The PR fixes the problem, but during opening wearable project I can see a mobile component for a short moment. Maybe we should increase insensitive time?

In my opinion, problem of mobile components visibility on start should be solved in different way.


"Format" action breaks references to MySQL global variables

Describe the bug
When referencing a global variable such as @@system_time_zone in a SQL query, clicking the "Format" button inserts a space between the two @ symbols.  This creates a syntax error when attempting to "Run" the query.
To Reproduce
After connecting to a mysql database ...

Create a new tab by clicking "+"
Enter SELECT @@system_time_zone
Click "Format"
Observe 2nd line has become @ @system_time_zone

Expected behavior
The variable name prefix should remain @@
Application (please complete the following information):
Client is

Antares
0.5.17
Ubuntu Software for 20.04 LTS

Environment (please complete the following information):
Server is

Amazon Linux AMI
4.14.77-70.59.amzn1.x86_64
MySQL
Community Server 5.7.37

Hi @bart-schaefer,
thanks for the report!
This issue was solved by recent sql-formatter module update (874dc6298b1587c84122a7e684d39c59a6352f79).
You will find the fix in next Antares update.


Implement Basic Enums in Derive

At present the derive feature does not support basic enums.

Implemented in https://github.com/mitsuhiko/deser/commit/e769392314aa9028d970b17896991600bfdb6c34


Adding stats to l2vtgate.

Same stats that vtgate exposes. So it's easier for monitoring. Also
adding end-to-end tests for them.

LGTM


TMFS: Add tempo to tune playback

It works nice on folkrnn.org, so let's add it to the tunes on the machine folk session.

It's there already: add a setting with a tempo.

Why not in this view?

A few reasons. The overall philosophy is that those tunes are frozen and any changes happen in settings. For folk-rnn tunes, that's a nice division between computer generated and hand-tweaked. And practically, the work a user does there for tempo would be lost.
I made it so there is a tempo control in settings, so without even seeing ABC you have a slider to drag (or stepper, rather).
Perhaps it's a question of signage: if the user is not signed in, they're not going to see that.


Send email to user after email verification is completed

Description of problem
Currently, when you setup a portal to use Verified Registration, it means the user will be sent an email with a link they need to click in order to have their user fully active on the portal.
The problem is that most often than not you'd like to send a full welcome message maybe with extra information details to the user once he has verified his account. This cannot be done on the initial email, since the user has not yet verified the email and so is not yet a "valid" user on the site.
So it makes sense to have a second email to be sent when the user verifies his email.
Description of solution
When using verified registration, send a confirmation email to the user after the email has been verified by using the link that the user has received on the initial registration email.
A new email template should be defined for this scenario as a means of confirming that the user has been approved.
Description of alternatives considered
None
Screenshots
Not applicable
Additional context
Not applicable
Affected browser
All

Subject and Body may come from a resource key. If either or is empty, the message should not be sent.

@sleupold do you mean that this behaviour is already existing but the keys may be empty or is this a suggestion on how it should be implemented ?

@valadas these are my ideas, how it should be implemented.

Cool, thanks, I'll tag that as an enhancement then...

I like the ideas as well.  I will see if I can work this out sometime soon as I would love to have this feature added.  I have a few projects to hopefully move off my plate and that should free up some time to focus here a bit more and I will put this on my to do list unless someone can please beat me to it :)  It would be nice to work with a resx file along with an email to send.  I can actually visualize this process so maybe I can contribute here with a little time and luck!


[TA] Prepare patch release for Text Analytic

This patch version includes the Java Doc improvements and the Core and Netty Httpclient version upgrade changes.

API change check
API changes are not detected in this pull request.


[feat]: MultiSelect and MultiSelectNested component's value prop supports number[]

A clear and concise description of what the feature is
MultiSelect and MultiSelectNested component's value prop supports number[]
Why should this feature be included?
value prop sometimes will be like [1, 2, 3]
Please provide an example for how this would work
<MultiSelect value={[1, 2, 3]} ...{restProps}>...</MultiSelect>

Our select is based off – https://www.radix-ui.com/docs/primitives/components/select#root and I don't think we want to support numbers as values as the Option's values are also strings, i think this would just make the API more complicated than it needs to be to be honest, e.g. your code could be:
<MultiSelect value={[1, 2, 3].map(v => v.toString())} ...{restProps}>...</MultiSelect>


[MRG] Add link to mybinder.org

The postBuild script get's executed at the end and install skopt.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #565   +/-   ##
=======================================
  Coverage   86.24%   86.24%
=======================================
  Files          23       23
  Lines        1716     1716
=======================================
  Hits         1480     1480
  Misses        236      236

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 262c130...2ac9715. Read the comment docs.

Looks good! merging


dune not installed ?

Hello,
just installed the windows 64-bit native installer. It seemed to work (I can execute utop) but when I execute this :
dune --help
I get :
'dune' n’est pas reconnu en tant que commande interne
ou externe, un programme exécutable ou un fichier de commandes.
Thanks

You will need Visual Studio and Git for that to work, and after you'll need to use:
opam exec -- dune build
Please watch the video I released:
https://youtu.be/33niX94tn3U?si=T_oy-AMGX3Cm-2Rh
Thanks.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: roromino @.>
Sent: Saturday, December 9, 2023 8:42:02 AM
To: diskuv/dkml-installer-ocaml @.>
Cc: Subscribed @.***>
Subject: [diskuv/dkml-installer-ocaml] dune not installed ? (Issue #88)
Hello,
just installed the windows 64-bit native installer. It seemed to work (I can execute utop) but when I execute this :
dune --help
I get :
'dune' n’est pas reconnu en tant que commande interne
ou externe, un programme exécutable ou un fichier de commandes.
Thanks
—
Reply to this email directly, view it on GitHubhttps://github.com/diskuv/dkml-installer-ocaml/issues/88, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AREG4PKAB7BBPRWSK5YDXPTYISIFVAVCNFSM6AAAAABAN45LIOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZTGOJTGM4TMMQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Card doesn't exist on iPad..

Having an issue with the slider button card "existing" on my ipad.

The slider works, however, on my fire tablets, android and iPhone, PC and MacBook. I have tried clearing cache, clearing browser history, restarts on the ipad and home assistant.

Looks like you're in a browser... Force close the app, then try again.
I would also recommend getting the app from the App Store.

I can not get the app, since Apple in it's infinite wisdom will not allow my 6 year old ipad to be updated.
However, that should not matter, since I am running chrome on every other tablet and pc in the house and this card works just find on them.


Implement JS search

The search is based on the icon titles and their index in the list. A 250ms debounced input event listener hides/shows elements according to the current input value.
The layout of the search box is very simple, so feel free to update it.
Imho a notification if nothing matches would be nice, but I didn't want to change too much in .grid.
The implementation should work on all current browsers as well as >=IE9.

I wish to unsubscribe please
Sent from my iPhone

On 26 May 2017, at 12:25 am, nietonfir<EMAIL_ADDRESS>wrote:
subscribed

There's a button to the right of the PR (shy over the participants badges) that reads "unsubscribe".

Rebased branch to current develop HEAD.

@danleech Is there something I need to modify to get this merged?

@nietonfir Hi! No, I’m just not organised and haven’t looked at this PR yet. I will get round to it though! Thanks for your patience! 😄

No problem at all, take your time! ;-) Thanks for the response!

Implemented all of the feedback except the shortcut and the normalization.
Regarding the shortcut: I'm not a fan of overriding default browser shortcuts. And after conducting a quick search I couldn't find a site where this is done, so I wouldn't recommend it either.
Regarding the normalization: I don't see any issue with that as long as the search terms keep being the brand names. I'll look into that…

Regarding the shortcut: I'm not a fan of overriding default browser shortcuts. And after conducting a quick search I couldn't find a site where this is done, so I wouldn't recommend it either.

Fair enough, I can see where you're coming from :)

This is now live :)


Are there any other tutorials available?

Hi. This seems to be a very robust repo, but I must say it does also seem a bit hard to tackle on your own, just by following the README. Do you have or plan to have any other "getting started" materials?
I closely followed the README and did manage to get this running (sort of), but a couple of aspects or areas are still quite blurry to me.
Why is there a CDN involved? Can we not use it? What use-case scenarios would this layer be desirable or, perhaps, even necessary?
I see there's also a gas-root project/app. What is it for? Is that the actual "GAS backend" that gets pushed to Google (clasp push)?
What steps should one take in order to swap the current gas-frontend project/app for one based on Vue.js?
My current  and most immediate goal is to get a Google Sheet sidebar as my frontend working along with a GAS backend. And, if at all possible, have a local dev environment were I could test this architecture out without having to push to Google at every single change in code.
Sorry for all the questions, but, again, this repo seems very promising, but a bit hard to grasp all at once.

Hi @mpenna ,
Thank you for the questions.
This repo/template is indeed a bit hard to get started with. The main reason (in my opinion) is that it can handle a lot of different scenarios and use cases.
Let's answer the questions:
Q. Do you have or plan to have any other "getting started" materials?
A. Initially I had plans to write a bunch of tutorials on different use cases but later got busy working on my projects. Also, no one asked about it until now. So, I will be adding docs in the coming days.
Q. Why is there a CDN involved? Can we not use it? What use-case scenarios would this layer be desirable or, perhaps, even necessary?
A. The CDN is for hosting only the bundled frontend assets (i.e. js, css, font files). Neither the apps-script code nor the html files will be uploaded to the CDN.
You can surely not use the CDN feature.  How ? in the gas-frontend package, there are two vite config files, 1) vite.config.ts and 2) vite.config.ts.non-cdn-example . Rename the vite.config.ts.non-cdn-example file to vite.config.ts and skip the CDN configuration steps.
Why is there a CDN feature ? So, in the non-cdn configuration, the client-side javascript and css will be bundled and inlined into the html files. If you are not using a js framework or external libraries, it should be fine. But, i was using React and a bunch of npm packages in my frontend code. As a result the output html files were reaching ~1MB in size. Using CDN feature means that the bundled js/css will not be inlined into the html, as a result the html files would stay ~1KB in size. Also because the js/css assets are hosted on a CDN, it cached on user's browser too. Overall it's there to improve the performance on the client side.
Q. I see there's also a gas-root project/app. What is it for?
A. You guessed it right. gas-root folder collects the files that are to be uploaded to Google. This happens automatically during build process and we don't need to think about it. For us, it's a folder to store the environment variables (for both frontend and backend). Prefix the environment vars with PUBLIC_ to make them available on the frontend code.
Q. What steps should one take in order to swap the current gas-frontend project/app for one based on Vue.js?
A. I have not tried this. Basically, you need to configure vite. Install vue and update the vite.config.ts. Take a look at this: https://github.com/vitejs/vite-plugin-vue/tree/main/packages/plugin-vue . If you are stuck, let me know.
The command to install vue in gas-frontend should be:
$ pnpm --filter gas-frontend add vue

Here's a diagram of the build process (simplified) for gas:deploy:staging/production.  It would give you some idea how it works. The workflow for gas:dev is a bit different and more complicated.

If you have more questions or need help, let me know.

Hi @rupampoddar,
First off, let me congratulate you for such a comprehensive and very well structured answer. I'm amazed!
Along with the schematics, your thorough explanations surely helped in building a better understanding I believe I now have of this architecture. Later on, as I mentioned being my goal, I'll try to swap out the built-in frontend for another one based on Vue.js, letting you know how that went.
Before I tackle that, however, could you offer a bit more clarification on the inner-workings regarding the frontend <--> backend communication/integration? Where should we look and what should we be paying attention to?
Oh, and one last (for now) question, please. You mentioned that one of the reasons for deploying a CDN for housing the frontend assets was, along with enabling caching, to bring down the overall package size to be deployed to Google. Did you follow this route solely for a performance reason or are there perhaps any restrictions imposed by Google regarding the final package/app size that you're aware of?
Once again, thanks.

Hi @mpenna,
I'm glad that you found the information helpful.
I have created a new branch with-vue. It has a working Vue.js integration. It's just example code but should be a good starting point. https://github.com/rupampoddar/apps-script-template/tree/with-vue .
Regarding your questions,
1. Frontend <--> backend communication/integration
Thank you for asking about it. This is actually a very important part of this repo and I have missed to mention it on the readme (along with other important things).
The official way of doing this is to use the google.script.run API. See: https://developers.google.com/apps-script/guides/html/communication .
But, we cannot use this directly if we want local development. So, Elisha Nuchi created a package gas-client, which is a wrapper around google.script.run API along with other things. I just forked his repo and made some changes.
Here's my forked repo: https://github.com/rupampoddar/gas-client-fork .
You can read the docs of this repo to understand how to use it.
Also, I have added an example code that you can refer. It's on this file: gas-frontend/src/components/HelloWorld.vue.
2. Restrictions imposed by Google regarding the final package/app size
I did not face any issue uploading 1 MB+ html files to Google.
I prefer not to inline the javascript/css due to some reasons:
a) last time I checked, there was no official way to inline javascript/css using vite. I'm using a vite plugin vite-plugin-singlefile, which works but it's not an official plugin.
b) if we have html files that are ~1MB in size, the initial load time will be significant for the end users. And, there are users who are still using 3G internet.
If your output html files are small, in that case, this shouldn't be an issue.
Hope this helps.

Hi @rupampoddar,
Fantastic! Thank you again for another very detailed answer.
I was planning to test this out with a vue frontend, so this new branch you put together is one great help in that direction. Thanks. I'll try that out later on and let you know how that goes.
I'll have a closer look at the gas-client package also, along with your fork, to better understand how it can help during local development.
Lots of things to learn!
BTW, the vite-plugin-singlefile is one which I was already familiarized with. For practice and learning purposes, I have been trying out some other GAS projects and I eventually got to learn and use this one plugin. I agree with you: it seems to work OK, but it's not something "official".

Oh, and I forgot to mention one thing...
If possible, could you please expand and clarify a bit more on the whole "makecert/ssl" aspect of the setup?
I usually develop all my projects inside Docker container, and more lately having it combined and controlled via VSCode's Dev Containers. I guess I did manage to install the CA inside the container via that makecert command, but I'm not sure how to get that to work with browser (Firefox, in my case), which is running on the host machine (Ubuntu).
Any help on this front will be greatly appreciated.