google_maps_flutter example not working on API 21 and below

Examples works on API 22 and later.
Issue: Getting a blank map on device with API 21 (Android 5.0)

Log:
Launching lib/main.dart on Android SDK built for x86 64 in debug mode...
✓ Built build/app/outputs/flutter-apk/app-debug.apk.
Connecting to VM Service at ws://<IP_ADDRESS>:50660/yUJQP1dAMtA=/ws
D/EGL_emulation( 4506): eglMakeCurrent: 0x7fe05b17fba0: ver 3 0 (tinfo 0x7fe06320fd60)
I/zzbz    ( 4506): Making Creator dynamically
I/DynamiteModule( 4506): Considering local module com.google.android.gms.maps_dynamite:0 and remote module com.google.android.gms.maps_dynamite:202414000
I/DynamiteModule( 4506): Selected remote version of com.google.android.gms.maps_dynamite, version >= 202414000
V/DynamiteModule( 4506): Dynamite loader version >= 2, using loadModule2NoCrashUtils
D/        ( 4506): HostConnection::get() New Host Connection established 0x7fe05df962b0, tid 4521
W/ResourcesManager( 4506): Asset path '/system/framework/com.android.media.remotedisplay.jar' does not exist or contains no resources.
W/ResourcesManager( 4506): Asset path '/system/framework/com.android.location.provider.jar' does not exist or contains no resources.
D/        ( 4506): HostConnection::get() New Host Connection established 0x7fe0460ef0f0, tid 4538
I/DynamiteLoaderV2( 4506): [71] Mapsdynamite
I/Google Maps Android API( 4506): Google Play services client version: 12451000
I/Google Maps Android API( 4506): Google Play services package version: 202414014
D/EGL_emulation( 4506): eglMakeCurrent: 0x7fe05d7b7200: ver 3 0 (tinfo 0x7fe05d7badc0)
D/        ( 4506): HostConnection::get() New Host Connection established 0x7fe0476f0710, tid 4577
D/EGL_emulation( 4506): eglCreateContext: 0x7fe047ac37a0: maj 1 min 0 rcv 1
D/EGL_emulation( 4506): eglMakeCurrent: 0x7fe047ac37a0: ver 1 0 (tinfo 0x7fe047806560)
D/        ( 4506): HostConnection::get() New Host Connection established 0x7fe05e29b0f0, tid 4520
W/DynamiteModule( 4506): Local module descriptor class for com.google.android.gms.googlecertificates not found.
I/DynamiteModule( 4506): Considering local module com.google.android.gms.googlecertificates:0 and remote module com.google.android.gms.googlecertificates:4
I/DynamiteModule( 4506): Selected remote version of com.google.android.gms.googlecertificates, version >= 4
I/DynamiteLoaderV2( 4506): [71] Googlecertificates

Flutter doctor:
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/dearwolves/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/dearwolves/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode11.3.1.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C505
    • CocoaPods version 1.9.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.3.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

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

[✓] Connected device (1 available)
    • Android SDK built for x86 64 (mobile) • emulator-5558 • android-x64 • Android 5.0.2 (API 21) (emulator)

Probably token issue. did you replace map api key for your own?
related: https://github.com/flutter/flutter/issues/25556

Issue replicable using latest master. Tried the official google_maps_flutter plugin example on emulator api 21 and observed that the map is not displayed. Probably related to platform-views.

flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.107, on Mac OS X 10.15.2 19C57,
    locale en-US)
    • Flutter version 1.22.0-10.0.pre.107 at
      /Users/deeptibelsare/Documents/Fluttersdk/flutter
    • Framework revision 54ade88365 (61 minutes ago), 2020-09-08 22:08:57 -0700
    • Engine revision d1d848e842
    • Dart version 2.10.0 (build 2.10.0-98.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/deeptibelsare/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • ANDROID_HOME = /Users/deeptibelsare/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/deeptibelsare/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

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

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

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (3 available)
    • macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.2 19C57
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome
      85.0.4183.83

! Doctor found issues in 1 category.

Similar to 62496

Probably token issue. did you replace map api key for your own?
related: #25556

yes, it works on api 22 and later.

/cc @blasten

I was able to reproduce this issue. I will update this thread once I find out more.

I tried using Flutter v1.7.8+hotfix.4  from 7/24/2019, and VD platform view doesn't appear to work.  I debugged this a bit more, and it seems image is nullptr in https://github.com/flutter/engine/blob/7e962ba91133f2a0c3751b4cbf0a5643d53c3166/shell/platform/android/android_external_texture_gl.cc#L60.
@amirh Has VD ever worked in API level 21?

I tried using Flutter v1.7.8+hotfix.4 from 7/24/2019, and VD platform view doesn't appear to work. I debugged this a bit more, and it seems image is nullptr in https://github.com/flutter/engine/blob/7e962ba91133f2a0c3751b4cbf0a5643d53c3166/shell/platform/android/android_external_texture_gl.cc#L60.
@amirh Has VD ever worked in API level 21?

It should work starting API 20, though unfortunately we do not have a test covering it on old APIs :sad @blasten can you confirm that you're not using software rendering?

I confirmed that the emulator is using hardware rendering, and that the engine is instantiating AndroidContextGL.  I'm unsure if this issue is affecting the emulator only or devices running API level 21/20.

I tested on xiaomi mi4i which is using android 5.0.2, I can confirm the maps are working fine with the latest flutter version.
and the plugin google_maps_flutter: ^2.0.6
I would request you to upgrade to the latest stable version of flutter and verify.
Screenshot taken on mi4i

Heres my flutter doctor -v output

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-arm, locale en-IN)
    • Flutter version 2.2.2 at /Users/mahesh/Documents/flutter
    • Framework revision d79295af24 (6 days ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

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

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 57.0.5
    • Dart plugin version 211.7233

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

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • d9dac45d • android-arm64  • Android 10 (API 29)
    • macOS (desktop)        • macos    • darwin-arm64   • macOS 11.4 20F71 darwin-arm
    • Chrome (web)           • chrome   • web-javascript • Google Chrome 91.0.4472.106

• No issues found!

Thank you

@blasten Just wondering if this issue should be kept open still for tracking ?


Hyper hype beast refactoring 6

@ityonemo same refactoring done in #554 (hhb-4) applied to Rooms and UserBlocks modules :smiley:

@benawad this looks good to me

Should be rebased to staging, shouldn't it?

yeah @arthurysong can you set this to staging

@benawad oh yeah mb, i set the base to staging :+1:


being able to cancel "starting bot"

Is your feature request related to a problem? Please describe.
The "starting bots" is taking too long. I want to cancel it. see screenshot below.

Also applies to other ...ing, e.g. "generating dialog", "publishing", etc. Sometimes it takes time, other times, it's broken. Users shouldn't need to wait for so long to get the results.

Similar feedback in discussion #7183. Adding this to fit-and-finish EPIC #8107.


minimap preCluster

hi ,
I am delighted to use minimap for preprocessing when using large data. However, the number of transcripts in the different partition is unbalanced, with one of them reaching tens of thousands, resulting in a very slow division and memery costing of the gene family.I would like to know there are no parameters to solve this problem or other methods，Looking forward to your reply，thanks.
yang

Hi,
Yes I am aware of this issue and trying to improve it. Note that the imbalance is also somewhat natural --- some bins (transcripts) are simply more abundant! But it does look like right now the preclustering processing is a bit too relaxed, sometimes binning not-so-related sequences together.
I encourage you to update to the latest version of Cupcake (v.3.9 https://github.com/Magdoll/cDNA_Cupcake/tree/v3.9) and try the --dun_use_partial option.
-Liz

@Magdoll Thank you very much for your reply and expect you to improve this problem and I will try the new version of Cupcake

@Magdoll Hello, I have installed the latest version of cupcake, but can not function properly. Have you ever tried to run this version and the unbalanced partition has not been improved? I am very anxious to deal with this problem, whether you can change a parameter to control the problem?

Hi @yangtried ,
Do you mean you tried --dun_use_partial and the preclustering results remain the same? Or just that it didn't improve the largest partitions any better. I would need more details on the largest precluster bin sizes to have some idea.
--Liz

hi @Magdoll ，
I did not run the latest version of cupcake smoothly because of the following error:
“ pkg_resources.run_script('cupcake==3.9', 'run_preCluster.py')
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 739, in run_script
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 1500, in run_script
File "/ifs/TJPROJ3/PB/yangjie/cogentlib/Python2/lib/python2.7/site-packages/cupcake-3.9-py2.7-linux-x86_64.egg/EGG-INFO/scripts/run_preCluster.py", line 182, in
main(args.cpus, args.dun_make_bins, args.dun_use_partial)
TypeError: main() takes at most 2 arguments (3 given)”

Hi @Magdoll ，
I did not run the latest version of cupcake smoothly because of the following error:
“ File "/Python2/bin/run_preCluster.py", line 5, in
pkg_resources.run_script('cupcake==3.9', 'run_preCluster.py')
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 739, in run_script
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 1500, in run_script
File "/Python2/lib/python2.7/site-packages/cupcake-3.9-py2.7-linux-x86_64.egg/EGG-INFO/scripts/run_preCluster.py", line 182, in
main(args.cpus, args.dun_make_bins, args.dun_use_partial)
TypeError: main() takes at most 2 arguments (3 given)”

Hi,
There was an issue with v3.9 when I merged two branches. It should be corrected now in the v4.0 version: https://github.com/Magdoll/cDNA_Cupcake/tree/v4.0
Here's a comparison of the top 5 bins by using and not using --dun_use_partial:
using --dun_use_partial
cluster   size
5234    907
5060    845
3117    807
5094    749
3976    618

not using --dun_use_partial
cluster   size
7124    6570
4658    988
7123    974
4472    808
3637    525

hi @Magdoll ，
I have used the 4.0 version, and there is a big improvement in partition balancing, but unlike your top 5 bins, there are still thousands of transcripts that form a partition. I have a different question, why do some transcripts can form a gene family, and some are classified as unassigned? Looking forward to your reply ~

Hi @yangtried ,
I tested the --dun_use_partial option on a very small dataset (65k FLNC sequences), so naturally, my top bins are still very small. Having a few thousand in a partition is still going to run decently fast.

why do some transcripts can form a gene family, and some are classified as unassigned?
the preClustering step classifies all sequences that do not have any hits (if you use --dun_use_partial, it would have to be a full hit) to any other sequence, based on minimap results. This means these sequences are orphans and they get put in the file preCluster_out.orphans.fasta and will have the stat as orphan in the output CSV file preCluster.output.csv. Additionally, preClustering also identifies possible PCR chimeras and put them in the file preCluster_out.chimeras.fasta, marking them as chimera in the output CSV.

Let me know if you have more questions.
--Liz

hi @Magdoll ，
Thank you very much for your reply, I use the 330M full length transcript, use --dun_use_partial get about 10,000 bins, each bin contains about 3 to 10 transcripts, only one bin contains 2000 transcripts, I do not know Is this normal? Another question, when refactoring, I get a very short contig, as shown below, path1.why is this happening? I think this contig is not desirable.
“>path0
TCCTCCATAGCCGCC.........

path1
GATCTTGT"

Hi @yangtried ,

It really depends on the diversity and abundance distribution of your transcripts in the sample, so I can't comment on that. minimap is pretty promiscuous, so if it truly can't find more than 3-10 sequences that have decent (full hits, because you used --dun_use_partial), then it is what it is.

The short "path1" will occasionally happen with heavily trimmed sequences. Cogent by default trims away all lower case bases at the beginning and end of each sequence in in.fa. So take a look at that bin at the in.trimmed.fa. Is there one that is heavily trimmed?

--Liz

hi @Magdoll ，
Thank you very much for your reply. Does it seem normal to have an over-pruning sequence in in.trimmed.fa?
You have already replied to how the orphan was formed, I am very sorry I did not understand what is the meaning of hit here. Can you explain it for me? Why do you think that the sequence containing more than 16 PolyTs in the first 20 bases of an orphan sequence is a PCR chimeric sequence? Looking forward to your reply.

Hi @yangtried ,
For short sequences (< 500 bp) that have a lot of lower case, i.e. indication of low quality, bases at the flanking regions, trimming them away would lead to a very short sequence that is of little use to the Cogent reconstruction process. If you think the lower case sequences are actually trustworthy and should not be trimmed, you can provide an input sequence that is all upper case. Otherwise, Cogent prefers not to deal with low-quality bases.
In terms of whether that's "normal", I guess the answer is I have certainly seen that happen with Iso-Seq data, but not that often.
re: orphan and chimeras --- a "hit" in the preclustering step means a sequence (from isoseq_flnc.fasta) must have at least one alignment record with another sequence based on the minimap result. Minimap is very sensitive, capable of detecting sequence similarities at as low as 20-30%. So, if a sequence has no hits, then it is an "orphan", and will be thrown out in the preclustering step. Further, sequences that have more than 16 poly Ts in the first 20 bases have a very high chance of being a PCR chimera, because it indicates a structure that is:
cDNA primer --- TTTTTTTT --- part of transcript1 --- part of transcript 2 --- AAAAAAA --- cDNA primer.
For a graphical representation, see this wiki on different kinds of artifacts.
--Liz

hi @Magdoll ，
Hi, after testing, I also found a problem that when using the argument, --unun_partial, the orphan sequence increased, as I got 7743 orphan sequences and 1522 chimeric sequences before using this parameter, however, after using the parameters , The same test data, I got 64559 orphan sequences and 11251 chimeric sequences, and these sequences in the subsequent gene family and reconstruction process is no longer used, this will lose a lot of data, is that reasonable?

Hi @yangtried ,
Yes it is totally expected that using --dun_use_partial would increase the number of orphan sequences. Because --dun_use_partial means any sequence that only has a partial hit will now be considered not a hit and is more likely to become "orphans".
--Liz


React Overdue Tasks Application Customizer

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
React Overdue Tasks Application Customizer

The sample illustrates how to display a warning message that current user has overdue tasks.
Features:

application customizer
usage of Page Placeholders
usage of SP PnP JS

@AJIXuMuK,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

thx Alex, great sample. Will add usage tracking image to readme, but otherwise all good.

Hi Alex, would you be available to demonstrate this in August 2017 monthly community call on 8th of August at 8 AM PST? - Would be typical 10-15 min demo to show the capability and code. I know that this is a short notice, but hopefully good for you?

Hi Vesa,
Yes, I'm available for the call. You can count on me.

thx Alex, I'll get this included in agenda, which will be released later today together with August release notes.


FIX: Refused to execute script

Chrome browser rejects the file 'http://domain/tinymce/filebrowser/'. This message get in the console browser:
Refused to execute script from 'http://domain/tinymce/filebrowser/'
because its MIME type ('text/html') is not executable, and strict MIME
type checking is enabled.

The patch fixes this of course.

It could be nice to add a test to make sure of the content-type.


[BUG] Websocket connection failed in JupyterLab with Chrome

Describe the bug
Using Chrome on Ubuntu Linux I can open the JupyterLab portal, but cannot connect to any kernels.  This has been a persistent issue for me over the past year or more.  My workaround is to use Firefox on the same computer, but I prefer to use Chrome.
On the Javascript console, I find the following repeated errors:
index.js:114 WebSocket connection to 'wss://data.lsst.cloud/nb/user/msk/api/events/subscribe?token=...' failed:
(anonymous)	@	index.js:114
_subscribe	@	index.js:107
factory	@	index.js:42
Poll._execute	@	index.es6.js:374
execute	@	index.es6.js:313

WebSocket connection to 'wss://data.lsst.cloud/nb/user/msk/api/kernels/9f38e77e-e981-4f8e-9c00-421c675f3951/channels?session_id=...' failed:
KernelConnection._createSocket	@	default.js:73

WebSocket connection to 'wss://data.lsst.cloud/portal/app/sticky/firefly/events?channelID=ffChan-msk-nb-1692364030-53' failed:
NPe	@	firefly-c4dab20….js:2
(anonymous)	@	firefly-c4dab20….js:2
kPe	@	firefly-c4dab20….js:2
t	@	firefly-c4dab20….js:2

To Reproduce
Steps to reproduce the behavior:

Log on to RSP with Chrome
Open Notebooks
Start a server (e.g., small).  This seems to work fine.
Wait for the JupyterLab landing page to open.
Open the Javascript console and see websocket connection failed messages.
Open a Notebook: see more websocket connection failed messages.  The browser never connects to the kernel.
Open a Console: the console tab opens.  Try any command and nothing happens.
Open a Terminal: the terminal tab opens and a cursor appears.  I cannot type anything.

Editing files (e.g., text or python files) and browsing directories works fine.
Expected behavior
That I could do anything with a python kernel.
Desktop (please complete the following information):

OS: Ubuntu Linux 22.04.3 LTS (and up to date)
Browser: Chrome 116.0.5845.96 (Official Build) (64-bit)

Additional context
This jypterlab issue might be relevant, although that report is not using wss: https://github.com/jupyterlab/jupyterlab/issues/12222
When I use Firefox on the same system, everything appears to work.  There are no websocket failures on the console.

Thanks for the report. Using Chrome under Linux is something that should certainly work, a couple of our engineers use that themselves.
We're looking it it, meanwhile:

Can I get you to paste the status bar of your Jupyterlab window, the one that looks like this:

Can you try disabling your Chrome extensions (temporarily) and trying from a private browser window?

One more thing - can you go to this 3rd party website https://www.piesocket.com/websocket-tester
from:

Your Firefox which works
Your Chrome which does not work

click Send (their default settings are fine) and report whether you get the same behavior on both browsers, and include a screenshot of your output. This is what I would expect:

I have identical results in Chrome and Firefox with the piesocket tool:

The status bar with Chrome is:

Also, neither disabling extensions nor private browsing mode was successful.

The errors are different today.  Now I am seeing 302 errors, e.g.,:
WebSocket connection to 'wss://data.lsst.cloud/nb/user/msk/api/events/subscribe?token=...' failed: Error during WebSocket handshake: Unexpected response code: 302
(anonymous) @ index.js:114

So we have been unable to reproduce this and the most likely (though not certain) explanation is that there is something in your configuration that is disabling websocket connections. Is there any way you could try from a different device and/or network?

I've tried this on three different computers on three different networks,
including just now from a hotel.  All three computers have ubuntu, although
different installations.  All three use the latest google-chrome-stable
from http://dl.google.com/linux/chrome/deb .  My only option left for
testing is to try it on an EC2 instance or something similar, unless you've
already done that.

Hi I am sorry we're kinda out of ideas since we can't reproduce it any way we try. If your timezone permits can you sign up for our zoom office hour and we can try and watch what's happening? You can sign up here: https://fantastical.app/frossie/ask-square

(by the way if you're always logging onto your Chrome wit the same google account, the different computers aren't really testing anything different if you have a Chrome setting disabling websockets, but if you drop in for office hours we can check for that)

I have been able to run a notebook from an identical version of Chrome installed on a different linux distribution, so I can now rule out Chrome version and logging into my browser as the cause.  I have also successfully run notebooks with Chrome on my same computer, just using a different (fresh) user account.  It clearly seems to be a problem hyper local to myself.  I'll close the issue.  Thanks for the help!

Ah. If you ever figure out why your (older) user account had an issue, do let us know, but glad you're up and running!


Feature Request: constraints removal and/or puzzle editor

Sometimes, to solve a puzzle on a computer, I open it in the editor so that I can get as many functions and commands as I want, make the solution, which works, then shorten it gradually, until it fits in the puzzle constraints.
It would be nice if android application allowed that too. It could be done by adding an option to puzzle solution screen which would add solution cells to the limit and remove them with another click. The additional cells could be highlighted with different color.
An alternative would be to implement puzzle editor and have an option to open an existing puzzle in the editor, similar to PC version.

In addition to the main report, I want to have a way to remove or adjust step limit.
Some of harder puzzles, for example "Flag on flagpole", have solutions, which would work, but take more steps then allowed. It would be nice to see the puzzle solution working to completion even if the rules of the game do not allow it to be submitted.


Can't connect to Jira user Account

Hello
Followed documentation, bot needs to connect Jira user Account, popup does open but Jira response with an error :
could not get user info for client, err: No response returned: Get "https://crit-digital.atlassian.net/rest/api/2/myself": oauth2: cannot fetch token: 400 Bad Request Response: {"error":"invalid_grant","error_description":"Invalid aud: https://auth.atlassian.io"}
Similar to https://github.com/mattermost/mattermost-plugin-jira/issues/945
Regards ;

Hi @supergeoff, thanks for reporting this issue. I have a few questions:

What version of the Jira plugin are you using?
How long has your MM server been integrated with Jira? How many users are connected? (just for me to have some context)
Are you using the cloud instance type for the Jira Cloud connection? If so, we recommend using the newer cloud-oauth instance type, which can be installed with /jira instance install cloud-oauth (Jira URL). This solves the in your description. This is available in v4.0.0 of the plugin.

Hi @mickmister, I had the same problem (reported here: https://community.atlassian.com/t5/Jira-Software-questions/Error-connecting-to-Jira-with-Mattermost-plugin/qaq-p/2501718). I updated to version 4 and installed with cloud-oauth (after I've removed old app in jira management app). I followed the bot's instructions for setting permissions on jira, but the "Configure" button to press in mattermost jira's bot doesn't woks and it's not possible to enter the jira details (client ID and secret) to continue.
Thanks for your suppport

Hi @elrancid, Just so I understand correctly, you are using /jira setup or /jira instance install cloud-oauth (url)? Then you get to the step with the "Configure" button, but clicking the button doesn't open a modal?
I'm thinking starting over on the setup process may fix something happening here. Are there any relevant log messages in your server logs?

No, I did't do /jira setup, it isn't documented. I've done only /jira instance install cloud-oauth (url). If I do /jira setup bot say:
Just a few more configuration steps to go!
1. Choose your Jira edition.
2. Create an incoming application link.
3. Configure a Jira subscription webhook.
4. Connect your user account.

There's no documentation on how to achieve those tasks, because the documentation link https://mattermost.gitbook.io/plugin-confluence/plugin-jira/setting-up/configuration doesn't works.
For the Jira edition: how to set it?
The incoming application link should be done with /jira instance install cloud (url) process? bot say it's deprecated.
Clicking on "Continue" button log say:
{"timestamp":"2023-10-17 12:53:36.209 Z","level":"debug","msg":"Action integration error.","caller":"web/context.go:114","path":"/api/v4/posts/3wtdgiunnpdu3yyt1qokq61e5c/actions/11pk4djrrtbn5e31kbgo3zafwr","request_id":"t8715xcmsbf73ctnsw55wuc4ur","ip_addr":"<IP_ADDRESS>","user_id":"moepmz8jbinu7pbfo8te4shqbc","method":"POST","err_where":"DoActionRequest","http_code":400,"error":"DoActionRequest: Action integration error., err=Post "https://mattermost.joule.video/plugins/jira/setup-wizard/button\": tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2023-10-17T12:53:36Z is after 2023-10-15T10:07:06Z"}
{"timestamp":"2023-10-17 12:53:36.209 Z","level":"debug","msg":"Received HTTP request","caller":"web/handlers.go:173","method":"POST","url":"/api/v4/posts/3wtdgiunnpdu3yyt1qokq61e5c/actions/11pk4djrrtbn5e31kbgo3zafwr","request_id":"t8715xcmsbf73ctnsw55wuc4ur","user_id":"moepmz8jbinu7pbfo8te4shqbc","status_code":"400"}

err=Post \"https://mattermost.joule.video/plugins/jira/setup-wizard/button\": tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2023-10-17T12:53:36Z is after 2023-10-15T10:07:06Z"

@elrancid It seems the SSL certificate being used for your Mattermost server may be invalid. The above error implies that the certificate is expired: current time 2023-10-17T12:53:36Z is after 2023-10-15T10:07:06Z. Is it possible the certificate needs to be updated?

If you mean the root server certificate, no, the certificate expire in 2024. Could it be a certificate with Jira Cloud?

@elrancid It's reporting this for the service that will receive a request at https://mattermost.joule.video/plugins/jira/setup-wizard/button, which would be your Mattermost server.
The underlying issue here is that the server isn't handing this request off to the plugin directly, and is instead performing an actual HTTP request to itself, to be processed by the plugin. I was under the impression we were handing it off directly in the case of interactive messages, and we are doing so based on the code below, but there may be an issue there:
https://github.com/mattermost/mattermost/blob/e1db660f70388a02acbd2e1d988d6ac9476294e1/server/channels/app/integration_action.go#L318-L321
Either way, the immediate solution for you will be something with your server's certificate. I'm not sure exactly what the solution is though.

@hanzei Any ideas why we would see the above error when we are indeed using a.DoLocalRequest for this kind of request? In the code linked above, I think we may need to change
rawURLPath := path.Clean(rawURL)

to
rawURLPath := path.Clean(inURL.Path)

Hi @supergeoff, we've released a version of the Jira plugin that fixes this issue about 2 months ago. Have you had the chance to try this version of the plugin?

Hi @supergeoff, we've released a version of the Jira plugin that fixes this issue v4.0.0 about 2 months ago. Have you had the chance to try this version of the plugin?

Closing this issue for now. Can reopen If upgrading to latest plugin version and using cloud-oauth does not fix the issue for someone.


Support private networks via PSK flag

Pre-sharked key passphrases can be provided via the PSK flag, starting from common repo v0.6.1+.
Allocator and proxy currently needs to support this feature.

[x] allocator
[x] proxy

Completed. Changes are made on top of branch for #31, so I'll wait until that's merged to master before trying to merge these commits.


Compatibility with openvr_fsr (AMD FidelityFX SuperResolution for VR)

Hi There,
@fholger made a modified openvr_api.dll that allows you to apply FidelityFX SuperResolution upscaling to many SteamVR games, as long as they use D3D11 (https://github.com/fholger/openvr_fsr).
I've runned a few successfull testing on nomai VR and it seems to improve performance greatly (cost a bit of image quality, but still totally worth imo)
I had to put openvr_api.dll on read only since it seems to be erased each time when you start nomaiVR.
I'm playing on an Oculus Quest 2 with Air link (4128x2096 / 90Hz) on an R5 3600 / RTX 2080 / Medium settings in game
Without FidelityFX > 33fps :

With FidelityFX (quality) > 45fps :

With FidelityFX (performance) > 80+fps :

Would be really nice to merge this into nomai VR !
Thanks for the mod anyway :)
Regards.

Yeah I've also tried it and it works great. I might try to include it, but I'd want to figure out a way for people to configure the resolution in-game. I've been busy with the Firewatch VR mod, but maybe when I come back to Outer Wilds for the DLC I can try to figure this out.

It would be helpful if you would specify where to put the DLL. I know it normally goes into Plugins folder for Unity games, but this is a mod, so I assumed it would be different
I found 3 possible locations:

OWML\Mods\Raicuparta.NomaiVR (no openvr dll originally, but it is main mod folder and there was another Valve-named dll, so this was my first though. No dice)
OWML\lib (this one had openvr dll in it, but replacing it did not appear to do anything)
steamapps\common\Outer Wilds\OuterWilds_Data\Plugins (replacing this dll actually did the trick)

Added in #460


README: update sponsors

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

We have two new sponsors to mention by name so let's add them to the README.

Review period will end on 2021-03-16 at 00:00:00 UTC.

Review period skipped due to critical label.

Thanks @jonchang!


Fix SynonymQuery equals implementation

The term member of TermAndBoost used to be a Term instance and became a BytesRef with #11941, which means its equals impl won't take the field name into account. The SynonymQuery equals impl needs to be updated accordingly to take the field into account as well, otherwise synonym queries with same term and boost across different fields are equal which is a bug.

@romseygeek @mkhludnev, this bug was introduced since 9.4 version, can this PR be back-ported to 9.4.2 to fix the issue?

Hi, @mingshl
I'm able to cherrypick this fix into branch_9_4, but I'm not sure if there'll be release 9.4.2 ever.

Thank you! @mkhludnev

I'm able to cherrypick this fix into branch_9_4, but I'm not sure if there'll be release 9.4.2 ever.

Indeed, there won't be.


People using this for not-its-intended purpose could burn through Twilio $$ very quickly

So, how can we avoid that? What things can we do to ensure its only getting used when it really matters?

You'd need to be able to put some sort of deterent into the system after they dial the number?
Maybe some kind of prompt, then a redirect to the recording? Other than that a black list of numbers that misuse the service?

@christopher-wilson Yah, I don't want to make it slower for people to make the call; otherwise they'll lose out on the ability to hit "dial" and just put it on speakerphone.
Black list is definitely possible; I've already added disclaimers to the site saying "I reserve the right to listen to your recordings if I'm suspicious you're abusing it." It's very manual, but it may be the only option

What do you think about a counter on the twilio cookie that flags the account and notifies you if it goes over x. I believe twilio cookies last for 4 hours and are linked to the number calling.

Problem is their usage isn't connected to the cookie--they're calling from
a phone.
But it would be nice to keep an eye out for any account with many calls and
or many minutes. They may not all be problematic but a single admin
dashboard that surfaces all that info could be super helpful.
On Thu, Mar 3, 2016 at 8:24 AM Richard Delorier<EMAIL_ADDRESS>wrote:

What do you think about a counter on the twilio cookie that flags the
account and notifies you if it goes over x. I believe twilio cookies last
for 4 hours and are linked to the number calling.
—
Reply to this email directly or view it on GitHub
https://github.com/mattstauffer/pulledover/issues/9#issuecomment-191760109
.

Actually twilio matches the cookie by phone number so its basically acts like their phone number is their browser https://www.twilio.com/help/faq/twilio-basics/how-do-twilio-cookies-work.
I agree a dashboard would be more useful with out without the flag just so you can easily review/blacklist/whitelist accounts.

Oh I think the cookie thing may only be for sms, scratch that.

Started working on this here
@mattstauffer what sort of rate do you want to use for throttling?

It seems like the simplest solution is to not make this service free. Keep an active credit card on file, and make billing terms for recording. Then charge users of the service an adequate amount to cover the Twilio charges for their recording.

@franksteinberg totally. The biggest problem is that I wanted to reduce barriers to people using this, and requiring them to pay and register credit cards is a high barrier.


[PROPOSAL] Require the OpenAPI spec to be re-generated for pull requests

What/Why
What are you proposing?
With the addition of coverage in #179 I propose we move the build into PRs as well.

Make ./gradlew build output/overwrite OpenSearch.openapi.json at the root.
Make CI fail if that file is not up-to-date.

What problems are you trying to solve?
Currently CI/CD re-generates the OpenAPI spec and makes a PR. This causes several issues.

Warnings are ignored, whereas they should be errors (see #184).
Coverage can't know what has changed next to the actual API spec change.
We have 2 commits for every PR.

What is the developer experience going to be?
Require that developers run ./gradlew build before a PR can be merged.

@dblock will https://github.com/opensearch-project/opensearch-api-specification/pull/211 resolve this one as well?

Yes, closing.


feature request: launch debloat script with one liner

Something similiar to what MAS has. Launch the script with one liner (method 1) without having to download the entire github repo.
example screenshot:

Actually i just made one for you pls review my idea
pull request related to this issue:
https://github.com/Raphire/Win11Debloat/pull/56

and heres the one liner that i just made it works perfectly tested on windows 11 22h2 home edition
(must be ran with admin perms)
irm https://raw.githubusercontent.com/loadstring1/Win11Debloat/master/get.ps1 | iex

I'm not sure if this one liner could be combined with the RemoveAppsCustom list. But I assuming that it may be possible...

Heya,
This has actually been requested before in issue https://github.com/Raphire/Win11Debloat/issues/39

Heya,
This has actually been requested before in issue #39
I'll look into this and your PR this weekend when I have more time :)

thank you my script won't support every each windows version because it uses winget to install git then clones your repo and runs "run.bat" which then starts your debloat script
its not the best but works like i wanted it to work

@loadstring1 Thanks for taking the time to work on Win11Debloat. I have adapted your solution provided in #56 to include support for parameters and, among a few other changes, added a check to ensure WinGet is installed. See: 7e89991
The command to run doesn't look as clean, but it'll do the job and avoids using Invoke-Expression (iex). This article by Microsoft explains why iex should be avoided if possible.
Closing issue as resolved 😎

@loadstring1 Thanks for taking the time to work on Win11Debloat. I have adapted your solution provided in #56 to include support for parameters and, among a few other changes, added a check to ensure WinGet is installed. See: 7e89991
The command to run doesn't look as clean, but it'll do the job and avoids using Invoke-Expression (iex). This article by Microsoft explains why iex should be avoided if possible.
Closing issue as resolved 😎

thank you for allowing me to contribute to this project

We are trying to dpeloy this using our RMM for onboarding automation. How do you suggest we fix cases when winget is not installed? Install Winget followed by running the line?

We are trying to dpeloy this using our RMM for onboarding automation. How do you suggest we fix cases when winget is not installed? Install Winget followed by running the line?

You could alternatively look into one of the other methods to run the script, but I think in your case simply installing WinGet may be the best option
This article has more information about WinGet that may be helpful: https://learn.microsoft.com/en-us/windows/package-manager/winget/


feat: i18n routes collapse in accordion

🆒 Your use case

routes unreadable when in use i18n
🆕 The solution you'd like
let's group the other languages into each one.
for example, use accordion from server routes:

🔍 Alternatives you've considered
No response
ℹ Additional info
No response

v2

v3

v3.1 with current popper solution:

Looks neat! Let's discuss in a PR?


Update release notes to 2023.12 / other changes

PREVIEW: https://rpaladin.github.io/notes#2023.12
CHANGELOG
New

INDEX: Monthly release bump (2023.12)
NOTES/IMG: New releases notes (2023.12)

Fixed

NOTES: Fixed wrong data attribute for 2023.10 & 2023.11 navigation links

Misc

MAKE: Fix support for PNG format

QUESTIONS

If you have time, would you mind answering my question from the last PR: https://github.com/armory3d/armory_web/pull/46#issuecomment-1789727800

Thanks and happy holidays!

Happy Holidays to you too! Hope you have a great rest of the month.


loihi_realization test_lohi.py

WARNING:DRV:  elementType would be deprecated in 0.9 in favor of messageSize, which provides more flexibility
WARNING:DRV:  elementType would be deprecated in 0.9 in favor of messageSize, which provides more flexibility
INFO:DRV:  SLURM is being run in background
INFO:HST:  srun: error: Unable to allocate resources: Invalid node name specified
WARNING:DRV:  Connection is taking longer than usual.
WARNING:DRV:  Boards might be busy or consider reasons below:
WARNING:DRV:  1. If you are working on INRC Cloud, ensure setting SLURM=1.
WARNING:DRV:  2. Run sinfo to check if all boards are down.
WARNING:DRV:  3. Check squeue below for any unfinished jobs. Run scancel  to cancel hung jobs.
WARNING:DRV:  JOBID     NAME                          PARTITION TIME      NODELIST(REASON)    USER
Do you know how to ensure the first two items?

error: PHDR segment not covered by LOAD segment
board.createSnip Error

Hi, this question is related to the Loihi hardware and environment you are using, please contact<EMAIL_ADDRESS>for general questions about running code on Loihi.


EuroSCORE II archetype, guidelie and test fixture

I agree with the conditions before performing the calculations. However, the editor won't allow me to check this for local variables.
I have made updates according to your other comments. Thanks!


No module named 'pysurvival.utils._functions'

Hi,
I can't load this module 'pysurvival.utils._functions'. I looked into pysurvival/utils/ directory but didn't find it.
How do I fix this? Thank you!

Please let us know
@Author
@jqmp

Have resolved this issue? @jqchen24 jqchen24

This is resolved for me by installing pysurvival again with pip. Also version conflict with python was resolved.
Python=3.6.7
pyarrow >=1.12


Botv1

@knowbase seems like this change may need more updates as we have added count. All places in data & locals folders where infra is called, we need to add infra.0 to make it work


moment.js does not use UK daylight saving time by default

Confirm if we want to always use UK time when persisting records to the database and update generation of dates to use the build function in date-formatter.js, which checks for DST.

this should be done for the Visitor's date of birth also, as if their DOB is in DST, it will currently save it for the day before


Uncaught SyntaxError: Unexpected identifier

Hey all, I'm trying to use Swup with a combination of PHP, HTML, JS and CSS and did exactly what the instructions says but for some reason it doesn't work.
(I included certain files w/ the code but as far as I know that shouldn't be a problem)
My console says Uncaught SyntaxError: Unexpected identifier in main.js - line 16 where I try to enable Swup (I installed the package w/ npm install)
16: import Swup from 'swup'; //This line seems to be the problem
17: const swup = new Swup();

Does someone know what I did wrong? Thanks in advance!
-Ruben

It seems you're missing the bundling part of your JS workflow. import... line shouldn't really make it to the browser and should be processed by bundler (like webpack), which would generate a compiled code for you.
Alternatively, you can include swup from /dist folder with a script tag, and then you don't need to import it from npm.

It seems you're missing the bundling part of your JS workflow. import... line shouldn't really make it to the browser and should be processed by bundler (like webpack), which would generate a compiled code for you.
Alternatively, you can include swup from /dist folder with a script tag, and then you don't need to import it from npm.

Does it not support php? I'm trying to include it but it still doesn't seem to work.
My test repo: https://github.com/RFlintstone/page_transitions
Video that I followed: https://www.youtube.com/watch?v=mWeYMyN5-oo&t=775s
~Ruben

This has nothing to do with php actually, which is only executed on a server. The problem you are having is that syntax like import is not supported by a browser, so you need to bundle your JavaScript if you want to use it that way with tools like webpack or much more beginner friendly ParcelJS. Swup itself is actually bundled the same way.
Swup demo, where webpack is used should be a helpful source, specifically the commands used in the scripts section of package.json.
Alternatively, as already mentioned, and also used by a project in the YT tutorial, swup can be included from dist folder, and then the import... line can be left out.
In any case, this is not a swup issue. Closing.


⚠ Bofrost.at has degraded performance

In a1259db, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1235 ms

Resolved: Bofrost.at performance has improved in 47f34ba after 7 minutes.


[Feature Request] Log client-side (JS) errors too, for FREE.

Feature Request
What I am trying to achieve
We can log client-side (JS) errors too.
There are many third-party services available to track JS errors. Why not offer a way to do the same for FREE?
How I see it implemented

Implement an API to receive errors via Ajax call on the error event.
A small JS snippet for developers to copy-paste into the layout view

DONE🎉

Nope. Out of scope.


Fixed the issue where iOS would fail to launch a remote desktop due t…

…o memory issue.
"free(inst->settings->HomePath);"
The bug ticket this goes with: https://github.com/FreeRDP/FreeRDP/issues/2593

The real issue is that ios_context_new is wrong. Wrong signature and missing error checking.

@phongleq I've created PR #2652.
Please let me know if that fixes your issue. Thanks.

Closing this PR since it would break all other clients.


fix: follow oauth2 spec for redirect uri

The OAuth2 spec explicitely forbids using fragments in redirect uri. Any IDP that follows the spec can currently not be used without changing the uris in netbird.
I think the default should be following the spec.
Closes #91
rfc-editor.org/rfc/rfc6749#section-3.1.2
"""
The redirection endpoint URI MUST be an absolute URI as defined by
[RFC3986] Section 4.3. The endpoint URI MAY include an
"application/x-www-form-urlencoded" formatted (per Appendix B) query
component ([RFC3986] Section 3.4), which MUST be retained when adding
additional query parameters. The endpoint URI MUST NOT include a
fragment component.
"""

 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.

@PatrickDaG these are just defaults, and the redirect URLs are configured according to the IDP using the environment variables:
NETBIRD_AUTH_REDIRECT_URI="/auth"
NETBIRD_AUTH_SILENT_REDIRECT_URI="/silent-auth"

The ones set there allow folks using auth0 or authentik to connect without setting them, and changing the defaults will break these deployments.

The ones set there allow folks using auth0 or authentik to connect without setting them [...]

What do you mean? They still have to configure the redirect uri in their IDP config, regardless of including a fragment or not.
In any case this is a clear violation of the oauth spec. There is absolutely zero guarantee that auth0 or authentik won't change their behavior in the future to follow the spec closely, which would then break all existing setups permanently. Fixing this violation of the oauth spec warrants a breaking change in my opinion.


94/t/STORY-3309/Migrate to Adwords api v201509

WIP

@ibiziiac-sv - What is the source of these changes?  Specifically, NoriLoader?  It isn't doing what you mean for it to do.  Specifically, it creates a class method and not an instance method.

Never mind, I was using an old SHA when I was reviewing the code.  Note that I had to do bundle update savon_zuora to get the latest version of the gem installed.  bundle install when using a branch doesn't fetch the tip of the branch.  I didn't realize that.

Okay, I get what is going on here now.
You are replacing NoriSavon.configure with just calling Nori.new with the correct parameters.  That looks better.
I assume the idea is not to change the global (shared) Nori configure, but rather use savon_zuora specific ones.  Is that correct?
Scott

I am intending to drop the forked nori_savon v.1.1.5 and use the up to date v2.4
I'm creating an isntance of Nori with NoriLoader because the new version uses instance methods, but the old version was using the class method NoriSavon.parse.

@sherriman please review

@sherriman can you please take a look? I don't have a lot of context on the need for this change.

This looks good.  Comments given.
Specific Comments
Please address these:

Can we afford to construct a new instance of nori on every soap response or fault?  That is, is the cost appropriate?
What lead you to bump the gem dependencies in the gem spec?  For example,  Part of our confusion was changing HTTPI versions.  Did other changes require that?  How do we know this works with the new version of HTTPI, nori and wasabi?

You should bump the version, right?

@sherriman I addressed the comments regarding nori_options.
Specific comments:

With the new version of nori we have to create an instance to be able to use parse method, but the old version had the class method parse.
The gems nori, httpi and wasabi are used by both active_zuora and google-api-ads-ruby. In order to keep google API client up to date, I decided to update these dependencies. But knowing that updating active_zuora and savon to newer versions would lead to massive changes to be applied in the web repository, I decided to adapt the existing compatibility versions, that were forked earlier and customized earlier, to work with the updated versions of nori, httpi and wasabi.

:+1:
Thank you for answering my comments.  Looks good.

I missed this, we should probably bump the version.rb of this file as well.  Same as active_zuora.
Not merge in the latest gem, just change the version.rb.

But what version would it be? It's the same version as before, but customized with the updated version of nori usage.

I think that 0.9.10 is correct.


Fix Excel2003: set namespace if not exist

Summary
Hi! This PR will help to open/read hard readable files Excel2003 without namespaces in workbook.

Coverage decreased (-0.6%) to 82.353% when pulling e1f509cfbd66a9aeb4e0916fa5e6a154775014f0 on evrone:fix/excel-2003-namespacing into 561e295335a0faddb4384021f748fa293882643b on roo-rb:master.

@chopraanmol1 Hi!🤝 Now PR looks good and test coverage fixed. 🙌🏼

@amatsuda @Empact @scottwillson Hi! 🙌🏼 Could you check this PR, please.
Thank you in advance!

Thanks for the fix - could you add a line to the test checking the namespace is as expected?


Add ability to profile rubocop execution via --profile and --memory options

As per discussion -  https://github.com/rubocop/rubocop/pull/11326#issuecomment-1364477250
With these changes, users will be able to easily profile rubocop within their projects (and possibly send PRs with improvements or report back identified performance problems).
Example usage:
rubocop --profile --memory

--profile - will do a CPU profiling (via stackprof) and create a dump file within tmp directory of the project's root
--memory - will additionally profile memory usage (via memory_profiler) and create a memory profiling info file within same tmp directory

Tests are passing on my machine (Mac OS), but are failing on CI (Windows). Would appreciate if someone suggest why.

The problem was that stackprof does not support windows - https://github.com/tmm1/stackprof/issues/45.
@bbatsov @koic Can you take a look at this PR?

@fatkodima Might be good to document somewhere how to profile RuboCop with the built-in options.

Added some docs. Writing docs is not my strong skill, sorry 😄

Don't worry. This will do for start. Thanks!


Gk rba overhaul

Removed all exercises, except the weather API, and resources directory as per Trello board https://trello.com/b/YBteLzHS/client-side-course-summit

👍


Replace non-experts Authorships with a citationStyle authors field.

@rakunkel-ucd let me know if you have any issues with this
@Vensberg close this issue if it's okay.


code and instructions don't match

Awesome plugin. Consider changing the code for the command from pconnect to pancakes to match the readme, or changing the readme to match the Plugin. @bensheldon and @ari-gold prefer pancakes, while I prefer pconnect.

Actually, Ben's preferred breakfast food is waffles!


New package: QTBase v0.6.1

Registering package: QTBase
Repository: https://github.com/USCqserver/QTBase.jl
Created by: @neversakura
Version: v0.6.1
Commit: 5594440a457c80e7c9a294f02fd1c97478bc1ea9
Git reference: v0.6.1
Release notes:

First release.

I changed the name for the package and this pull request should be closed.


Added allowOverMax and customMaxAttribute to the documentation #108

In regards to issue #108

Nice fix :+1:.


Automated PR for Prettybot

This is an automated PR.

Superseded by https://github.com/OctopusDeploy/OctoVersion/pull/225.


Minimize

In the tray it says "Mimize" instead of "Minimize"

Fixed on v1.1.0:

Linux (.deb): https://github.com/Electra-project/storage/releases/download/v0.0.0/electra-desktop_1.1.0_amd64.deb
Linux (.rpm): https://github.com/Electra-project/storage/releases/download/v0.0.0/electra-desktop-1.1.0.x86_64.rpm
MacOS: https://github.com/Electra-project/storage/releases/download/v0.0.0/electra-desktop-1.1.0.dmg
Windows: https://github.com/Electra-project/storage/releases/download/v0.0.0/electra-desktop-setup-1.1.0.exe


Unable to retrieve workspace key

[Enter feedback here]
We have ARM template for Azure Container Instances – ACI to add log analytics workspace, this is the ARM template piece:
"workspaceId": "[reference(parameters('logAnalyticsResourceId'), providers('Microsoft.OperationalInsights', 'workspaces').apiVersions[0]).customerId]",
"workspaceKey": "[listKeys(parameters('logAnalyticsResourceId'), providers('Microsoft.OperationalInsights', 'workspaces').apiVersions[0]).primarySharedKey]"

When it runs I’m getting following exception:
ERROR: {"status":"Failed","error":{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"Forbidden","message":"{\r\n "error": {\r\n "code": "AuthorizationFailed",\r\n "message": "The client 'f1e1532b-b5ad-4ba2-b169-15ba1d8a6675' with object id 'f1e1532b-b5ad-4ba2-b169-15ba1d8a6675' does not have authorization to perform action 'Microsoft.OperationalInsights/workspaces/listKeys/action' over scope '/subscriptions/832d2ee6-742b-4789-a57e-00bf5013fdc0/resourcegroups/operations-eu/providers/Microsoft.OperationalInsights/workspaces/c-loganalytics-eu' or the scope is invalid. If access was recently granted, please refresh your credentials."\r\n }\r\n}"}]}}
In documentation here:
https://docs.microsoft.com/en-us/azure/azure-monitor/logs/manage-access?tabs=portal#access-control-mode
you are only mentioning:

Read the workspace keys to allow sending logs to this workspace
Microsoft.OperationalInsights/workspaces/sharedKeys/action

There is no information on the:
Microsoft.OperationalInsights/workspaces/listKeys/action
Once we added this action:
Microsoft.OperationalInsights/workspaces/listKeys/action
to custom role that SPN was granted we were able to successfully retrieve workspace key in ARM template

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

ID: 8dec993a-34a8-ccbf-de46-057c3125247a
Version Independent ID: 7482ad70-ac38-ab52-b007-083c620ddc49
Content: Manage access to Log Analytics workspaces - Azure Monitor
Content Source: articles/azure-monitor/logs/manage-access.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@cryptodinoadamuso I see the issue is closed the I believe it's self-resolved. Let me know if that's not the case.


Github API Client

This should solve https://github.com/danger/swift/issues/17

Warnings

:warning:

Any changes to library code should be reflected in the Changelog.
Please consider adding a note there and adhere to the Changelog Guidelines.

Messages

:book:

Danger CI

  Generated by :no_entry_sign: dangerJS

Looks good - want to add a changelog entry then you can merge 👍


connection closing before write

Currently using tap-facebook from singer.io to read from facebook marketing api and writing to Redshift using the target-redshift package. The connection establishes with Redshift and reads the data, but before i get to the write step I'm getting the error:
Traceback (most recent call last):
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/postgres.py", line 129, in write_batch
    cur.execute('BEGIN;')
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/postgres.py", line 53, in execute
    return super(_MillisLoggingCursor, self).execute(query, vars)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/psycopg2/extras.py", line 440, in execute
    return super(LoggingCursor, self).execute(query, vars)
psycopg2.OperationalError: SSL SYSCALL error: EOF detected

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_redshift/__init__.py", line 48, in main
    target_tools.main(redshift_target)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/target_tools.py", line 25, in main
    stream_to_target(input_stream, target, config=config)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/target_tools.py", line 69, in stream_to_target
    raise e
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/target_tools.py", line 63, in stream_to_target
    _flush_streams(streams, target, force=True)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/target_tools.py", line 88, in _flush_streams
    _flush_stream(target, stream_buffer)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/target_tools.py", line 81, in _flush_stream
    target.write_batch(stream_buffer)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_redshift/redshift.py", line 58, in write_batch
    return PostgresTarget.write_batch(self, nullable_stream_buffer)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/postgres.py", line 192, in write_batch
    cur.execute('ROLLBACK;')
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_postgres/postgres.py", line 53, in execute
    return super(_MillisLoggingCursor, self).execute(query, vars)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/psycopg2/extras.py", line 440, in execute
    return super(LoggingCursor, self).execute(query, vars)
psycopg2.InterfaceError: cursor already closed

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "venv--target-redshift/bin/target-redshift", line 10, in <module>
    sys.exit(cli())
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_redshift/__init__.py", line 54, in cli
    main(args.config)
  File "/singer.io/venv--target-redshift/lib/python3.7/site-packages/target_redshift/__init__.py", line 48, in main
    target_tools.main(redshift_target)
psycopg2.InterfaceError: connection already closed

@rthirumulu looks like we don't have the ability to pass along SSL information currently. target-postgres has the ability to pass along SSL information to allow psycopg2 to deal with certs and whatnot.
I think we've avoided doing this for target-redshift because we assumed most folks would be using an ssh bastion to connect to their Redshift instance.
If you can provide guidance on what values are needed to get PsycoPG2 to connect to Redshift with SSL support (if that is indeed the issue you're facing) we'd be happy to get that baked into target-redshift.
Do you have any other projects connecting to this Redshift instance via python?

@rthirumulu we're looking to wrap up features/whatnot for a release this week. Any updates on the above?

We never quite identified this as a reproducible issue with the target, so I'm closing this out.


add SET to the UpdateExpression

Hello, I am using architect for a few projects and am really impressed with what you have built here. This is an amazing tool which helps a frontend focused dev like myself quickly setup a solid serverless backend. Thanks so much for building and open sourcing all this fantastic stuff. I love the latest documentation, all these examples make it so much clearer how to work with DynamodB via architect. I needed to add that SET expression in order to get my copy pasted version of this example working.
Thank you!

awesome!!! thx for the kind words and the PR! please don't hesitate to make any suggestions and/or feature requests to improve! cheers 🍻


Add discriminator length checks

This PR adds discriminator length checks in a few places where they appear to be missing.

this is not a vulnerability because the array_ref will fail if the data is shorter than the discriminator but an error msg is still better than a panic!

@paul-schaaf I agree with what you said.


Failed to connect

Basically trying to configure the ipmi integration it flakes out with the error, never mind what I do...
Then having played around with ipmitool it also fails to connect, except if I add in the "-I lan or -I lanplus" option, then all is well and working, but I have no idea whether this is an option that can be set for the integration...
HELP!

Did you install the add-on?

there is no log that I can see and looking in
/root/config/custom_components/ipmi/server.py
I see...
def connect(self) -> pyipmi.Ipmi:
   interface = pyipmi.interfaces.create_interface('rmcp',
                                       slave_address=0x81,
                                       host_target_address=0x20,
                                       keep_alive_interval=0)

which is not the lanplus option, but I'm not a python programmer and thus I could be wrong... either way, with the ipmitool from the HA machine I get data back, trying to configure the connector times out...
Where do I enable/check the logs?

Ok, there is a singular line in home-assistant.log... here from a ssh session...
[core-ssh config]$ tail -n 0 -f home-assistant.log
2023-11-08 16:19:44.511 ERROR (SyncWorker_2) [custom_components.ipmi.server] Error connecting to IPMI server idrac-FL2D25J.gariep.local: timed out
^C
[core-ssh config]$ ping -c 3 idrac-FL2D25J.gariep.local; ipmitool -I lanplus -H idrac-FL2D25J.gariep.local -U user -P passw chassis status
PING idrac-FL2D25J.gariep.local (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=63 time=0.427 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=63 time=0.725 ms
64 bytes from <IP_ADDRESS>: seq=2 ttl=63 time=0.596 ms
--- idrac-FL2D25J.gariep.local ping statistics ---
3 packets transmitted, 3 packets received, 0% packet loss
round-trip min/avg/max = 0.427/0.582/0.725 ms
System Power         : off
Power Overload       : false
Power Interlock      : inactive
Main Power Fault     : false
Power Control Fault  : false
Power Restore Policy : always-on
Last Power Event     : command
Chassis Intrusion    : inactive
Front-Panel Lockout  : inactive
Drive Fault          : false
Cooling/Fan Fault    : false
Sleep Button Disable : not allowed
Diag Button Disable  : allowed
Reset Button Disable : not allowed
Power Button Disable : allowed
Sleep Button Disabled: false
Diag Button Disabled : true
Reset Button Disabled: false
Power Button Disabled: false
Thus, the machine responds, even though it is powered down, but the ipmi connector does not work... HELP!

Install the add-on: https://github.com/ateodorescu/home-assistant-addons/tree/main/ipmi-server
The integration can use the add-on if it's installed otherwise will fallback to the python library which does not support lots of devices.

Oh boy, if I panel beat ipmi-server into place and the restart home-assistant, then go to Add-ons and Add-on Store, there it now sits as a local repository, but then trying to Install it, the end result is...
The command '/bin/bash -o pipefail -c apk -U update && apk -U upgrade && apk -U add --no-cache ipmitool nginx=1.24.0-r6 php$PHPVERS php$PHPVERS-fpm php$PHPVERS-curl php$PHPVERS-dom php$PHPVERS-gettext php$PHPVERS-xml php$PHPVERS-simplexml php$PHPVERS-zip php$PHPVERS-zlib php$PHPVERS-gd php$PHPVERS-openssl php$PHPVERS-mysqli php$PHPVERS-pdo php$PHPVERS-pdo_mysql php$PHPVERS-pgsql php$PHPVERS-pdo_pgsql php$PHPVERS-sqlite3 php$PHPVERS-pdo_sqlite php$PHPVERS-json php$PHPVERS-mbstring php$PHPVERS-ctype php$PHPVERS-opcache php$PHPVERS-session php$PHPVERS-tokenizer && mkdir -p /app && mkdir /app/cache && mkdir /var/tmp/nginx' returned a non-zero code: 1
Again, HELP! this is beyond me...

There was a problem with the addon but I have fixed it: https://github.com/ateodorescu/home-assistant-addons/issues/5
Please try again.

Hi there, thanks, I got one machine working now, the Dell, as above, now an older IBM is not coming up, again ,via command line, it works, but cannot add it as a device, it claims "Failed to connect"... in the home-assistant.log, I get:
2023-11-10 10:14:56.752 ERROR (SyncWorker_3) [custom_components.ipmi.server] Error connecting to IPMI server X3650.gariep.local: CompletionCodeError cc=0xc7 desc=Request data length invalid

Hi there any idea about the cc=0xc7 error as above, the IPMI integration is now installed working (mostly), at least the Dell, is connected and I have control via your server/device, but it will not register the IBM X3650, it comes up with the error as posted previously.

Hi there any idea about the cc=0xc7 error as above, the IPMI integration is now installed working (mostly), at least the Dell, is connected and I have control via your server/device, but it will not register the IBM X3650, it comes up with the error as posted previously.

Sorry, I have no idea what that means and there's no way I can debug such a thing.

Ok, one more question, does your device query or try and set the IPMI entity SMTP  parameters?
On https://pubs.lenovo.com/mgt_tools_smm/smm_ipmi_command under OEMCMD GET and SET SMTP_CONFIG_PARAMETERS the error 0xC7: Request data length invalid appears.

Ok, one more question, does your device query or try and set the IPMI entity SMTP parameters? On https://pubs.lenovo.com/mgt_tools_smm/smm_ipmi_command under OEMCMD GET and SET SMTP_CONFIG_PARAMETERS the error 0xC7: Request data length invalid appears.

Please read this one: https://github.com/ateodorescu/home-assistant-ipmi/issues/16#issuecomment-1715018263
Basically the addon could run any ipmitool command that you want. So, if you want to set anything on your server via ipmitool then you can do that. The user Adumbird created his own rest commands to control the fans of his server. He most likely assigned those hass rest commands to some entities/buttons in hass. You could do the same with any commands that you like.


Docs:

I think your document is suspected of misleading
Page URL:
https://www.typescriptlang.org/docs/handbook/typescript-in-5-minutes.html
Issue:

Even if you change the string type to the number type, it will report an error :ReferenceError: backpack is not defined
Recommended Fix:
As a beginner, he may not know that there will be errors after the modification, I think you need to add this hint, or change to a more suitable example
Browser:

Thanks for the issue. The example explains that it is declaring an constant which doesn't exist on the line above - I think this is fine 👍🏻

I would have to agree with @gui-gu-zi
As someone who is attempting to gain a deeper understanding of Typescript, unnecessarily adding additional concepts into such a supposedly simple example , not only creates needless frustration but wastes time.


windowing generator should return an AxisArray not List[AxisArray]

I'd like to change the behavaiour of the windowing generator method. Currently, its output is a List[AxisArray]. The unit then iterates over the list, yielding one at a time.
When the newaxis parameter is not None, segments / windows (even if only one) appear on a new axis named something like "step" or "win", and the generator return always has len=1. If newaxis is None then a list is returned with len for the number of windows found on this iteration.
I would like to make newaxis in the generator non-optional with a default value of "win". This means that we always get a new "win" axis. Then, if the Unit was configured with newaxis=None, it's up to the node to multiple yields of single-window AxisArray messages without the "win" axis present.
The main reason for this change is that at present the windowing generator cannot be composed with other generator functions. A common use case that this change will simplify is a STFT where the output of windowing goes into spectrum ('time' axis becomes 'freq') and modify_axis to convert the 'win' axis to 'time'.
This will not change the functioning of the Window(ez.Unit) in any way, only the generator method.
Pleaset let me know what you think.

This sounds good to me.  Feel free to submit a PR and we can get it merged in a jiffy.


Bring isolated workspaces to beta

This issue tracks feedback on the "isolated workspaces" feature that allows a workspace to be mounted to specific steps and sidecars, and the work required to move this to beta stability. Any issues related to this feature can be linked here.

/lifecycle frozen

Closing as a duplicate of https://github.com/tektoncd/pipeline/issues/6109


Using quackling with LangChain

Hi,
Is there any sample demonstrating usage with langchain?
-tansu

Hi @tansut, LangChain integration has just been introduced with v0.4.0.
You can find a basic example here.
If this answers your question, feel free to close the issue.

Thanks.


Fix Incorrect Inclusion of API_KEY in chat.completions.create Call

This PR addresses a bug  where the API_KEY was erroneously included in the parameters of the chat.completions.create() method, leading to a TypeError.  The error in question:
Traceback (most recent call last):
  File "/Users/easto/Library/Mobile Documents/com~apple~CloudDocs/Projects/spine-reader/better.py", line 4, in <module>
    text = betterocr.detect_text(
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/easto/Library/Mobile Documents/com~apple~CloudDocs/Projects/spine-reader/venv/lib/python3.11/site-packages/betterocr/detect.py", line 83, in detect_text
    completion = client.chat.completions.create(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/easto/Library/Mobile Documents/com~apple~CloudDocs/Projects/spine-reader/venv/lib/python3.11/site-packages/openai/_utils/_utils.py", line 299, in wrapper
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
TypeError: Completions.create() got an unexpected keyword argument 'API_KEY'

The provided fix prioritizes the user-specified API_KEY when initializing the OpenAI client and ensures that the API_KEY is not passed along with other options to the completions.create() call.

omg I missed that, thanks!

@junhoyeo Thanks for merging! :D

@junhoyeo Thanks for merging! :D

👍👍 Just shipped https://github.com/junhoyeo/BetterOCR/releases/tag/v1.1.2 to PyPI


[Bug]: Intermittent RangeError: Invalid array length when page.screencast

Minimal, reproducible example
'use strict';

const puppeteer = require('puppeteer');
const { setTimeout } = require('timers/promises');
const main = async () => {
  const browser = await puppeteer.launch({
    headless: 'new',
    args: ['--window-size=1920,1080'],
    defaultViewport: {
      width: 1920,
      height: 1080,
    },
  });
  const page = await browser.newPage();
  const recorder = await page.screencast({ path: '/tmp/screencast.web' });

  // go to the tool this will redirect to the login page
  await page.goto('https://online.mapcreator.io');
  await page.click('button[title="login"]');
  await setTimeout(10000);
  await recorder.stop();
  await browser.close();
};

main();

Error string
RangeError: Invalid array length
Bug behavior

[X] Flaky
[ ] PDF

Background
When creating a screencast intermittently the error RangeError: Invalid array length is thrown, I've narrowed it down to this line https://github.com/puppeteer/puppeteer/blob/main/packages/puppeteer-core/src/node/ScreenRecorder.ts#L180 trying to create an Array(-1), wrapping Math.round(DEFAULT_FPS * (timestamp - previousTimestamp)) in abs seems to solve the problem.
Clicking the login button combined with the timeouts seems to trigger it, without those it doesn't appear to happen.
This is running in a docker container pulling ghcr.io/puppeteer/puppeteer:21.4.1 and installing ffmpeg.
It doesn't happen every time but running for i in {1..10}; do time node test.js; done seems to trigger it at least once.
Expectation
Not throw an error
Reality
/vagrant/node_modules/puppeteer-core/lib/cjs/puppeteer/node/ScreenRecorder.js:168
                return (0, rxjs_js_1.from)(Array(Math.round(DEFAULT_FPS * (timestamp - previousTimestamp))).fill(buffer));
                                           ^

RangeError: Invalid array length
    at /vagrant/node_modules/puppeteer-core/lib/cjs/puppeteer/node/ScreenRecorder.js:168:44
    at h (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:64384)
    at y (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:64315)
    at f._next (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:11821)
    at t.next (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:6459)
    at /vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:78513
    at f._next (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:11821)
    at t.next (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:6459)
    at /vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:59971
    at f._next (/vagrant/node_modules/puppeteer-core/lib/cjs/third_party/rxjs/rxjs.js:3:11821)

Node.js v18.18.1

Puppeteer configuration file (if used)
No response
Puppeteer version
21.4.1
Node version
v18.18.1
Package manager
npm
Package manager version
9.8.1
Operating system
Linux

Flaky is checked, no idea what the bot doesn't like, if someone can elaborate it would be helpful.

@jrandolf please check once you are back.


fix the drawing cjk(utf8) rune.

The fix is much more complex, because of the edition mode. Please, check #74 and the linked issues/PRs.


feat: add support for Google VertexAI based PaLM API

This PR adds support for the set of LLM from Google based on PaLM. It's using VertexAI as PaLM is available in public preview via the VertexAI Model Garden and can be used already, instead of waiting for the GenAI API.

https://cloud.google.com/ai/generative-ai
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Describes source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[ ] Passes all golangci-lint checks.

@alvarowolfx as an FYI I have a basic example hitting the PaLM 2 apis here: https://github.com/tmc/cbison


Presets for only logsCollection and hostMetrics

Implements presets for logsCollection and hostMetrics.  config continues to contain default config.  Use append to add filelog and hostmetrics receivers into user config.

@dmitryax I have run make generate-examples and pushed the results. so we can see how the changes affect things.  Once we are good with this approach i'll bump the chart and regenerate the examples.

@dmitryax I believe I've addressed all unresolved comments.  If all looks good I'll regenerate the examples for final review.

@dmitryax @Allex1 ready for final review


Avoid returning None on init

It's a bit inconvenient that that init may return None, especially since I have to manually check before using with when using mypy. Looks like the following TODO is an intended fix for that, considering how easy of a fix it'd be I'd love to see it happen.
https://github.com/wandb/client/blob/7cc4dd311f3cdba8a740be0dc8903075250a914e/wandb/sdk/wandb_init.py#L742

Hey @agrinh , thanks for the request. I'm bringing this up with the team, to see if we could include a fix for this in the upcoming release.

Seconding this feature request.

@d-miketa and @tyomhak we return None in case of a failure. Would a DummyRun or a DisabledRun would more of an api you are looking to see?

issue seems to be fixed. If it is still an issue please provide a repro


Including zdb in the precompiled initramfs

The zdb command  can be very useful to diagnose possible issues with the pools. I wonder if it could be included in the precompiled initramfs so it's available in the zfsbootmenu interactive shell.

This shouldn't be a problem to do. zdb uses the same shared libraries as zfs and zpool, so it'll just pad the initramfs by a small amount. I'll get this added in to release binaries by the time 1.12.0 is ready.

Thanks!

v1.12.0 will ship with zdb and lsblk by default. We'll tag a new release in early January, so that we can ship OpenZFS 2.1.2.


COM-477 Getting Started + Storybook improvements + IDEA folder

What is the ticket / issue associated with this change? (Link to the ticket or issue)

Updates Storybook to latest
Adds useful addons for storybook
Improves gitignore file
Adds IDEA folder
Add script to run the app at root level
Improves READMEs with getting started

You have to update the Dockerfile too to use the new commando to build the storybook

thank you, fixed it!


Test failure in VS2019

After being able to overcome #210, I've encountered a test failure:

It looks like VS' test runner uses some slightly outdated (or too modern?) .NET version than we require in our tests.
I believe we should somehow fix that by adding a specialized double serializer for tests. The numbers in question, 0.52778099999999994 and 0.527781, are exactly equal (at least my current pwsh session tells me that). We could try to experiment with .ToString("r") to use a round-trippable format or something like that. Looks like we unnecessarily expect more precision than is required by the modern approach to double serialization.

The problem will disappear if you choose x64 architecture for testing
Thanks to the so-answer

0.52778099999999994 and 0.527781

Just round to single precision

You can try using the settings file
Configure unit tests by using a .runsettings file


migrate hasHierarchy -> inHierarchy to conform the spec (in sync with @zazuko/cube-hierarchy-query)

Describe the bug
upstream bugs
current cube-creator needs to produce cubes respecting the spec.
zazuko/cube-creator#1243 and zazuko/cube-hierarchy-query#6
hence visualize needs to adopt the changes in sync with upstream.
NB: do you reuse @zazuko/cube-hierarchy-query at all or are you reinventing an implementation?
cc @ortnever @zellersabine @pierredumas

/label upstream


Use Travis CI to build the sphinx documentation

Partially addressing #193.
The sphinx documentation is built by ReadTheDocs when changes are pushed on master.
This will ensure that the doc can be generated without errors.
Make sure the matrix build on Travis CI already ran the check before you merge.

I removed the -q flag when calling sphinx-build to get a verbose output.  I amended the changes and pushed force.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files         380      380
  Lines       28126    28126
=======================================
  Hits        26921    26921
  Misses       1205     1205

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 803b940...4d938f7. Read the comment docs.

OK to merge once jenkins is happy (Travis worked).


ASM-7600 Disconnecting the host from storagegroup

Running "disconnect" command will fail if host is not
connected
Adding a condition to run "disconnect" command  only
if host is connected to storage group

@sushilrai mind taking a look at this one?

+1


Multiple Serval failures

There have continued to be a steady stream of failing Google Chat messages from Serval (5 a day?).  These should be investigated and addressed so that either (1) the underlying issue is addressed or (2) the logging is reconfigured to no longer make the Google chat message.

I think all of these alerts are from/connected to #449, #450, and #453.

I'm going to close this since I'm fairly confident that the recurring alerts we've received are the result of issues tracked above.


Rename this project to goat

Would be neat wouldn't it?
P.S.
Great project, keep it up :+1:

haha I'm afraid someone else already thought about it :) thanks for the suggestion and the encouragement though!


Refactor 2.0 split proto

TODO

[x]  IE testing—in version 2.0 (before release)
Deeper scrollTo testing—in version 2.0.1
Docs and examples—in version 2.1.0
Party 🎉

Why stickybits 2.0?

more tests
open API so it's more hackable for customer—b/c that's what they want

Key files

src/stickybits.js

Note for Code Reviewers

The changes are mainly new tests
The key file linked above should be well documented and is a prototype that is hopefully readable.

@jonathanong @briangonzalez @someguynamedmatt Here we go!


Remove version: Meta.Oculus version 1.97.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://securecdn.oculus.com/binaries/download/?id=3754726804808697
OK
(Automated message - build 910)

Either you can close the pull request directly without mentioning me.


Need a way to relate a pool created block to the pool which created it

I think that block.vrf_key should match up with pool_update.vrf_key but the former is VerKeyVRF PraosVRF and the latter is a Hash Blake2b_256 (VerKeyVRF PraosVRF).

Turns out that the hash field of the slot_leader table as the verification key which it should have been the hash (Blake_224) of the verification key.

Does block.vrf_key not correlate with pool_update.vrf_key though?

No, the first is a "verification key" and the second is a "hash of a verification key".

The first is a "verification key" and the second is a "verification key hash".

Ok. Might be worth suffixing it with _hash before the next major version?


$executeRawUnsafe: incorrect binary data format in bind parameter 6

Bug description
As Prisma does not support batched upserts, I wanted to implement them with Kysely ... this worked fine when using also the Kysely DB provider (pq driver), but when using prisma-extension-kysely which is officially recommended from your side, the insert into query failed, with the error in the header. Internally it uses the $executeRawUnsafe method, which has a bug to send the correct bytes to my Postgres database ... the transaction model has this structure:
type in Prisma
{
      projectId: string
      storeId: string
      fromTimestamp: Date
      toTimestamp: Date
      productName: string
      productPrice: number
      productAmount: number
      invoiceAmount: number
    }

type in Kysely
{
    projectId: string;
    storeId: string;
    fromTimestamp: Timestamp;
    toTimestamp: Timestamp;
    productName: string;
    productPrice: number;
    productAmount: number;
    invoiceAmount: Generated<number>;
}

this is the method that produces the error:
async storeTransactionGroups(
    projectId: string,
    transactionGroups: TransactionGroupDto[],
    chunkSize: number = DEFAULT_DATABASE_BATCH_SIZE,
  ): Promise<void> {
    const transactionModels: Transaction[] = transactionGroups.map((x) =>
      this.convertTransactionGroupToTransactionModel(projectId, x),
    );
    await this.prismaService.client.$transaction(async (tx) => {
      await Promise.all(
        [...chunks(transactionModels, chunkSize)].map(
          (transactionModelChunk) => {
            const batchUpsertQuery = tx.$kysely
              .insertInto('Transaction')
              .values(transactionModelChunk)
              .onConflict((ocb) =>
                ocb
                  .columns([
                    'projectId',
                    'storeId',
                    'fromTimestamp',
                    'toTimestamp',
                  ])
                  // eslint-disable-next-line max-nested-callbacks
                  .doUpdateSet((eb) => ({
                    productName: eb.ref('excluded.productName'),
                    productPrice: eb.ref('excluded.productPrice'),
                    productAmount: eb.ref('excluded.productAmount'),
                    invoiceAmount: eb.ref('excluded.invoiceAmount'),
                  })),
              );
            return batchUpsertQuery.execute();
          },
        ),
      );
    });
  }

somehow the productPrice is transmitted incorrectly ... when inserting multiple values isolated as single values the above exception is thrown on the second and third entry, never on the first ... furthermore when hardcoding productPrice to 0 or 12.7 also no exception is thrown, I am completely puzzled what your Prisma engine is doing under the hood to produce such kind of errors ... anyway in the meantime I just use kysely with the native pg driver which works just fine and has no such bugs ...
If you need more input just ask
How to reproduce
check the description
Expected behavior
query finished with your recommended extension, there must be an error in your engine, as the data seems fine, I double and triple checked my input data and its also validated with class-validator ...
Prisma information
generator prisma-client-js {
  provider      = "prisma-client-js"
  binaryTargets = ["native"]
}

generator prisma-client-kysely {
  provider = "prisma-kysely"
  fileName = "kysely.types.ts"
  output = "../backend/bff/lib/service-utils/src/lib/database"
}

generator prisma-client-py {
  provider             = "prisma-client-py"
  interface            = "sync"
  recursive_type_depth = -1
}

datasource db {
  provider = "postgresql"
  url      = env("DB_TIMESCALE_DB_URL")
}

model Transaction {
  projectId     String
  storeId       String
  fromTimestamp DateTime
  toTimestamp   DateTime
  productName   String
  productPrice  Float
  productAmount Float
  invoiceAmount Int      @default(1)

  @@id([projectId, storeId, fromTimestamp, toTimestamp])
  @@index([projectId, storeId, fromTimestamp, toTimestamp])
}

model Event {
  projectId     String
  storeId       String
  fromTimestamp DateTime
  toTimestamp   DateTime
  eventType     String
  eventName     String

  @@id([projectId, storeId, fromTimestamp, toTimestamp])
  @@index([projectId, storeId, fromTimestamp, toTimestamp])
}

model WeatherMeasurement {
  projectId     String
  storeId       String
  fromTimestamp DateTime
  toTimestamp   DateTime
  temperature   Float
  humidity      Float?
  cloudCoverage Float?
  windSpeed     Float?
  precipitation Float?

  @@id([projectId, storeId, fromTimestamp, toTimestamp])
  @@index([projectId, storeId, fromTimestamp, toTimestamp])
}

model Forecast {
  id            String                 @id @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  projectId     String
  storeId       String
  fromTimestamp DateTime
  toTimestamp   DateTime
  timezone      String
  status        ForecastStatusEnum
  resolution    ForecastResolutionEnum
  results       ForecastResult[]
  message       String?
  createdAt     DateTime               @default(now())
  lastUpdatedAt DateTime               @default(now()) @updatedAt
}

model ForecastResult {
  id                       String   @id @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  forecastId               String   @db.Uuid
  forecast                 Forecast @relation(fields: [forecastId], references: [id], onUpdate: Cascade, onDelete: Cascade)
  fromTimestamp            DateTime
  toTimestamp              DateTime
  predictedValue           Float
  upperBoundPredictedValue Float
  lowerBoundPredictedValue Float
}

enum ForecastStatusEnum {
  processing
  finished
  error
}

enum ForecastResolutionEnum {
  daily
  hourly
  half_hourly
}

Environment & setup

OS: macOS (my dev computer)
Database:PostgreSQL
Node.js version: 20

Prisma Version
5.11.0

Can you create a minimal project that reproduces this problem and put it on GitHub? That would help a lot here to be able to see how this fails and then hopefully figure out why and fix it. Thanks.

Similar problem confirmed in https://github.com/prisma/prisma/issues/22482

Thank you for reporting the issue @Oidlichtnwoada!
If you are not using driver adapters for Postgres, can you please try this version and tell us if it fixes the issue? Note that this is a dev version and it is not suitable for production use.
npm i<EMAIL_ADDRESS>npm i<EMAIL_ADDRESS>

Thanks everyone!
The issue should be fixed and the fix will be shipped as a part of 5.15.0 release next week.
As my co-worker mentioned, you can try out the fix now by using 5.15.0-dev.47 snapshot. We do not recommend using snapshot versions in production, but it's enough to verify the fix.
Feel free to comment if that version does not fix the problem for you.


Upgrade highlightjs in common foundation component

Upgrade highlightjs in common foundation component to support multiple languages.

Closing as repo is deprecated.