Anisotropic Phantoms

Phantom functions accept length 2 (or length 3 for brain_phantom3D) vectors for the subsampling and upsampling parameters. Scalers still accepted and should be backwards compatible. Some validity checking.
"""
    obj = brain_phantom2D(; axis="axial", ss=4, us=1)

Creates a two-dimensional brain Phantom struct.
Default ss=4 sample spacing is 2 mm. Original file (ss=1) sample spacing is .5 mm.

# References
- B. Aubert-Broche, D.L. Collins, A.C. Evans: "A new improved version of the realistic
    digital brain phantom" NeuroImage, in review - 2006
- B. Aubert-Broche, M. Griffin, G.B. Pike, A.C. Evans and D.L. Collins: "20 new digital
    brain phantoms for creation of validation image data bases" IEEE TMI, in review - 2006
- https://brainweb.bic.mni.mcgill.ca/brainweb

# Keywords
- `axis`: (`::String`, `="axial"`, opts=[`"axial"`, `"coronal"`, `"sagittal"`]) orientation of the phantom
- `ss`: (`::Integer or ::Vector{Integer}`, `=4`) subsampling parameter for all axes if scaler, per axis if 2 element vector [ssx, ssy]
- `us`: (`::Integer or ::Vector{Integer}`, `=1`)  upsampling parameter for all axes if scaler, per axis if 2 element vector [usx, usy], if used ss is set to ss=1

# Returns
- `obj`: (`::Phantom`) Phantom struct

# Examples
```julia-repl
julia> obj = brain_phantom2D(; axis="sagittal", ss=1)

julia> obj = brain_phantom2D(; axis="axial", us=[1, 2])

julia> plot_phantom_map(obj, :ρ)

"""

See https://github.com/JuliaHealth/KomaMRI.jl/discussions/311#discussioncomment-8921717 for an example.

Sorry for all the commits, I couldn´t find a way to do them simultaneously on GitHub.
There is a part where tabs are used instead of spaces, and I also included a suggestion to wrap some checks in a function.

I completely forgot to merge this.

Thanks 🙏

hen an official release of Koma will be tagged, I'll share my example from pypulseq to qMRI for the MP2RAGE.
It will be a good example to show how to label the AcquisitionData to get 2 TI images in the reconstruction.

And I will also try to label directly the profiles in the RawAcquisitioDdata in order to get a standard cartesian acquisition data rather than using the nufft reconstruction :)

Ok, all the tests are running. I will be merging this.
@beorostica can you release a new set of patch package versions? (after checking everything looks good with the docs, plots, etc...)
@aTrotier Tell us if you need any help.

Oh god, thanks for noticing! merged!


Is it possible to import svg from file?

Hi, I don't see anything in docs for including svg from file.
So, is it possible to use svg files with this library?
For example something like this <Svg source={require('./src/logo.svg')} />

I have the same issue.
Any thoughts?

I have tried with react-native-svg-uri, and it works

@Creemli it doesn't work either with these dependencies:
"dependencies": {
  "react": "16.0.0-alpha.12",
  "react-native": "0.45.1",
  "react-native-svg-uri": "1.2.0"
}

It throws error:
~/react-mobile-app/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/SvgViewManager.java:16: error: cannot find symbol
import com.facebook.yoga.YogaNodeAPI;
                        ^
  symbol:   class YogaNodeAPI
  location: package com.facebook.yoga
~/react-mobile-app/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/SvgView.java:130: error: method obtain in class TouchEvent cannot be applied to given types;
            TouchEvent.obtain(
                      ^
  required: int,TouchEventType,MotionEvent,long,float,float,TouchEventCoalescingKeyHelper
  found: int,TouchEventType,MotionEvent,float,float,TouchEventCoalescingKeyHelper
  reason: actual and formal argument lists differ in length
~/react-mobile-app/node_modules/react-native-svg/android/src/main/java/com/horcrux/svg/SvgViewManager.java:38: error: cannot find symbol
                YogaNodeAPI node,
                ^
  symbol: class YogaNodeAPI
3 errors
:react-native-svg:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-svg:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

However, my question is for this lib/repository exactly.
Should I use react-native-svg-uri to achieve the above?
Thanks!

@yavorski  I also meet this error, and it is not because this library but react-native-svg.
you can simply open SvgView.java, and change the method like this:
TouchEvent.obtain(
                                                                 mTargetTag,
                                                                 type,
                                                                     ev,
                                                                     // this params is lost in this library
                                                                     SystemClock.nanoTime(),
                                                                 ev.getX(),
                                                                 ev.getY(),
                                                                 mTouchEventCoalescingKeyHelper));

@Creemli 10x for your suggestion I will try it asap. Should we suggest to fix it in the other library too?

PR is better I think. @yavorski  Have you solved this problem?

Hmm not, exactly, i am thinking of how to start new project without so much difficulties around the way ...
Actually because, of this i just put  and applied some basic styling to it ...

disclaimer: actually just first project with RN, just trying to put some some stuff together and feel what it is.

Can also try this: https://github.com/target/react-native-svg-parser


update dependencies

This PR updates dependencies to the latest release. For packed_simd this was also necessary to be able to run on the latest nightly.

Codecov Report

Merging #319 (4da3ee9) into master (c863a2c) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #319      +/-   ##
==========================================
- Coverage   82.52%   82.52%   -0.01%
==========================================
  Files         162      162
  Lines       44007    44007
==========================================
- Hits        36316    36315       -1
- Misses       7691     7692       +1

Impacted Files
Coverage Δ

parquet/src/encodings/encoding.rs
94.85% <0.00%> (-0.20%)
:arrow_down:

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 c863a2c...4da3ee9. Read the comment docs.

These versions should already be taken care of as they are SemVer compatible to the versions in Cargo.toml according to the SemVer rules of cargo (= means semver compliant in crates). I also checked it by running  cargo update` and looking into the Cargo.lock file (which is 0.3.5 and 1.5.4).


refactor: Make name and payload fields is optional

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

name and payload are required
What is the new behavior?
name and payload is optional
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Hey @fjodor-rybakov,
have you checked that everything is displayed as intended in case name & payload are not provided?

Hey @fjodor-rybakov, have you checked that everything is displayed as intended in case name & payload are not provided?

Hi! I checked and fixed optional payload type... The absence of type throws an error. Mark only the name prop as optional, no problem with it.


First fully featured draft

Todo:

[ ] Tests
[ ] Psalm
[ ] Code style

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@67f08b5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   54.94%
  Complexity      ?       97
=======================================
  Files           ?       18
  Lines           ?      293
  Branches        ?        0
=======================================
  Hits            ?      161
  Misses          ?      132
  Partials        ?        0

Flag
Coverage Δ

unittests
54.94% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


feat: Introduce new Dump function to run pods with two containers

Change Overview
In order to separate data dump generation by database tools and export by kando, a pod performing database dump can have two separate containers and set up the pipe over file instead of anonymous pipe
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[x] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test
[ ] :building_construction: Build

Issues

fixes #issue-number

Test Plan

[ ] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

There is an example in tests and docs.
Another example of blueprint:
apiVersion: cr.kanister.io/v1alpha1
kind: Blueprint
metadata:
  name: dump-bp
  namespace: kanister
actions:
  dump:
    phases:
    - func: Dump
      name: dump
      args:
        namespace: kanister
        sharedStorageMedium: Memory
        sharedStorageSize: 1Gi
        sharedStorageDir: "/tmp/"
        dumpImage: ubuntu:latest
        dumpCommand:
        - bash
        - -o
        - errexit
        - -o
        - pipefail
        - -c
        - |
          mkfifo /tmp/pipe-file
          for i in {1..10}
          do
            echo $i
            sleep 0.5
          done > /tmp/pipe-file
        exportImage: ghcr.io/kanisterio/kanister-tools:v9.99.9-dev
        exportCommand:
        - bash
        - -o
        - errexit
        - -o
        - pipefail
        - -c
        - |
          while [ ! -e /tmp/pipe-file  ]; do sleep 1; done
          kando output numbers "$(cat /tmp/pipe-file)"

Hi @hairyhum ,
Are we doing this to solve any performance issue that were reported? Can you please share some background just for my understanding.

Hi @hairyhum , Are we doing this to solve any performance issue that were reported? Can you please share some background just for my understanding.

The main goal of this change is to separate image management for databse dump blueprints. Not the performance.

Renamed function into KubeTaskParallel and the arguments for containers to background and output containers.

Renamed function to `MultiContainerRun'


[SDK] Training Client Conditions related unit tests

What this PR does / why we need it:
The following unit tests are added with this PR.

get_job_conditions
is_job_created
is_job_running
is_job_restarting
is_job_failed
is_job_succeeded

You can verify each unit test with the following command substituting <test_name> for one of the tests above.
pytest -v sdk/python/kubeflow/training/api/training_client_test.py -k <test_name>

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Related to #2161
Checklist:

[ ] Docs included if any changes are user facing

Let me know if this is too large for 1 PR.
Thought I would group them all together as they are very similar tests.

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

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

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
66

Relevant Lines:
66

💛  - Coveralls

Thanks @andreyvelich I will take a look at resolving these today


Forward search after compilation only works the second time

As brought up implicitly in #30
TeXiFy 0.7.7-alpha.2, pdf viewer 0.12.0-alpha.1
Expected:

Type \newpage Page $n$ in LaTeX
Compile
Forward search is executed but focus (caret) should stay in LaTeX

Actual:

Type \newpage Page $n$ in LaTeX
Compile
Document reloads but stays at same position
Focus is transferred to pdf
Place caret back and compile second time
Now forward search is executed, although the marker disappears immediately and focus is still transferred

@slideclimb will have a look, the behaviour seems to have changed since 0.11

Current behaviour seems to be that the forward search is executed and puts a marker in the pdf, which immediately disappears because the pdf is reloaded.

After #76, it seems that the above issue (forward search executed but the marker disappears) can still occur but much less frequently. Now, it is more common that the forward search will be executed fine, except the marker does not appear. However, sometimes it does appear.
A second manual forward search worked in all cases I tried it.
Also saw this exception once:
Uncaught Wu: Required value was null.
java.lang.Throwable: [LOGSEVERITY_ERROR][webpack://light-pdf-viewer/./application/viewer.js?:1]:
Uncaught Wu: Required value was null.
	at com.intellij.openapi.diagnostic.Logger.error(Logger.java:202)
	at com.firsttimeinforever.intellij.pdf.viewer.jcef.JcefUtils$createDefaultConsoleMessageListener$1.invoke(JcefUtils.kt:104)
	at com.firsttimeinforever.intellij.pdf.viewer.jcef.JcefUtils$createDefaultConsoleMessageListener$1.invoke(JcefUtils.kt:101)
	at com.firsttimeinforever.intellij.pdf.viewer.jcef.JcefUtils$addConsoleMessageListener$1.onConsoleMessage(JcefUtils.kt:89)
	at com.intellij.ui.jcef.JBCefClient$3.lambda$onConsoleMessage$4(JBCefClient.java:305)
	at com.intellij.ui.jcef.JBCefClient$HandlerSupport.lambda$handle$0(JBCefClient.java:726)
	at java.base/java.lang.Iterable.forEach(Iterable.java:75)
	at java.base/java.util.Collections$SynchronizedCollection.forEach(Collections.java:2131)
	at com.intellij.ui.jcef.JBCefClient$HandlerSupport.handle(JBCefClient.java:726)
	at com.intellij.ui.jcef.JBCefClient$HandlerSupport.handleBoolean(JBCefClient.java:731)
	at com.intellij.ui.jcef.JBCefClient$3.onConsoleMessage(JBCefClient.java:304)
	at jcef/org.cef.CefClient.onConsoleMessage(CefClient.java:352)

I can still reproduce this issue:
Expected:

Type `\newpage Page $n$ <caret>` in LaTeX

Compile

Type `\newpage Page $n+1$ <caret>`

Forward search is executed and the text just typed has a marker

Actual:

Type `\newpage Page $n$ <caret>` in LaTeX

Compile

Type `\newpage Page $n+1$ <caret>`

Document reloads but stays at page n, sometimes with a marker. Sometimes it goes to page n+1, so this may be a race condition

Compile second time

Now forward search is executed and the text on page n+1 has a marker (sometimes it has not)


Clear Sorting

There should be an easy way to remove the sorting of a table.

May be a bit out of topic but if I'm correct the way where sort starts are event listeners of elements with data-sort event. How about creating an unique id on sort initialization and use it to name('click.id') events, then provide an destroy method...use the id to remove event listeners and clear rest of the stuff.


make sure that no pekko version specified to build against is same as "default"

That fixes the nightly tests which only accidentally worked before and when fixing the akka->pekko prefix in f47c735e20d62fa58ce81e43cb4f5d82a1c9272b broke because "default" started to refer to pekko 1.0.0 which is not yet released.

Code is not formatted

:+1: Nightlies green again: https://github.com/apache/incubator-pekko-http/actions/runs/4232112501/jobs/7351447416


Add reserved namespace recon and medical-aegis

Signed-off-by: Patrick Dwyer<EMAIL_ADDRESS>

based on https://github.com/CycloneDX/cyclonedx-property-taxonomy/issues/42 and https://github.com/CycloneDX/cyclonedx-property-taxonomy/issues/40


refac: Migrate get datasets operation to drizzle on NextJs

closed https://github.com/lmnr-ai/lmnr/issues/82

 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.

hey @dinmukhamedm just wanna confirm before I delete this, this response any_in_project isn’t used in any frontend component, right?

btw just info that the POST method for updating a dataset in frontend/app/api/projects/[projectId]/datasets/[datasetId]/route.ts is not used anywhere in the codebase

this response any_in_project isn’t used in any frontend component, right?

Yes, this is for consistency with get evaluations and get traces where we used it to show a page placeholder if there are none. However, this is already done separately in the frontend. We can remove this, this is safe.

I think we no need to move the delete method for the dataset to Next.js, as there is semantic search that handles the deletion operation.

Great catch! Let's keep it for now. Later we may want to move that request to the frontend too, but it's gRPC, so it is a little bit more work

btw just info that the POST method for updating a dataset in frontend/app/api/projects/[projectId]/datasets/[datasetId]/route.ts is not used anywhere in the codebase

yep, saw that, we can delete that.


feat: ibc ante handlers and upgrade cleanup

Description
streamming down https://github.com/umee-network/umee/pull/1425 changes.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #1441 (73b8bff) into main (421da96) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1441      +/-   ##
==========================================
- Coverage   52.77%   52.75%   -0.02%
==========================================
  Files          71       71
  Lines        6759     6761       +2
==========================================
  Hits         3567     3567
- Misses       2922     2924       +2
  Partials      270      270

Impacted Files
Coverage Δ

app/upgradev3/migrations.go
78.26% <0.00%> (ø)


Adding new Resources SDK -  StandbyPool

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run prepare-pipelines


CA Policy templates

[Enter feedback here]
This page does not reflect the same settings made in the "Require password change for high-risk users" located at https://portal.azure.com/#view/Microsoft_AAD_ConditionalAccess/CaTemplates.ReactView. This is an issue because currently, you can not set a sign-in frequency of "every time".  The save button will either be greyed out or you will receive the below error.
Message from server. The server could not process the request because it is malformed or incorrect. 1101: The tenant must be explicitly authorized to use the private preview feature: ReAuth.

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

ID: a0863ea0-5143-2cef-84d9-d5920fb72656
Version Independent ID: bc430f9a-7872-19df-2b6f-bfd236c2a7da
Content: User risk-based password change - Microsoft Entra
Content Source: articles/active-directory/conditional-access/howto-conditional-access-policy-risk-user.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

#label:"awaiting-product-team-response"
#label:"assigned-to-author", "product-question", "triaged"
#reassign:@MicrosoftGuyJFlo

Engineering team is aware and investigating. Thanks for flagging this! #please-close

This has been resolved by the developers.


MSAL Angular with ASP.NET core Auth Guides (updated to match current Azure)

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
Are there any tutorials that explain how to get this working with asp.net core? Say angular frontent using msal and asp.net core backend with AddAzureADBearer. I tried all of the ones i could find on google and couldnt get any of them to work.
MSAL on the front end works fine but i get bearer errors on the back end, invalid authority, invalid issuer and even after disabling those checks i just get a 401 with no error. I think ive gone through 4 or 5 tutorials on how to setup the backend API and front end in Azure and they are all completely different. Please tell me you have a standard one hidden somewhere that matches what azure currently displays ?
Thanks,

@whittssg would this sample help with your case?

@whittssg would this sample help with your case?

Yeh that helped. Thanks alot.


#1575 - Improving Edit itinerary UI

Making some tweaks to the ui so the checkobox is more easily clickable (and much smaller), I also made a change so "address line 1" would apear on one line and not two.

Thank you for the contribution, happy new year!

Thank you for the contribution, happy new year!


Add option to install *.vsix files headless

VSCode Version: 1.3.1
OS Version: Mac OS X 10.11.5

it would be nice to be able to install extensions that are packeged as vsix files headlessly e.g.
code --install-extension my.vsix

https://code.visualstudio.com/Docs/extensions/install-extension
What do you mean by "headless", unless the documented approach is not what you wanted?

I want to be able to install plugins that are not in the marketplace the same way as I can do it for plugins that are in the marketplace. Means I want to be able to call code --install-extension my.vsix so that the plugin gets installed without opening a code window. This would allow me to provision code without opening closing Windows.
So basically dealing with vsix and marketplace should be the same

@cdietrich Thanks. Yes, that's not yet supported and should be a good feature request.

That was super fast, thanks a lot! 👍

it works like a charm, thanks 👍


Ability to compare 2 or more load tests

Hi,
I'm in the situation that I need to store JMeter test results as CSV files instead of into DB.
Would I be able to somehow compare 2 or more load tests stored in CSV files with this grafana-csv-datasource?
I mean, to compare in the Graph panel, which requires a Time field and a Number field. The problem to me is that different tests have different timestamps. So,

How can I chart 2 or more load tests in the same Graph panel?
How to make sure they line up nicely instead each in their own time period?

Thx!

Currently, each data source corresponds to a single CSV file. If you have multiple CSV files, you can create multiple CSV data sources and combine them in a graph using the Mixed data source. It's a little cumbersome, but if you have an idea of what would make it easier, I'd love to hear it!

Currently, each data source corresponds to a single CSV file. If you have multiple CSV files, you can create multiple CSV data sources and combine them in a graph using the Mixed data source. It's a little cumbersome, but if you have an idea of what would make it easier, I'd love to hear it!

Gotya. I'll play around and see how things can possibly be improved. Thanks!

Gotya. I'll play around and see how things can possibly be improved. Thanks!


fix memory leak

merge pleaaaase :) (otherwise my tests on another project never pass because of this mem leak)

whoops

thank you :):)


I have "fixed" my implementation with some casts (need to verify that it really works). See https://github.com/raff/goble/commit/811260b1d735999cdc026ab89dc513ad6658e29a

I can confirm this issue.

+1

+1

+1

It seems like there might be an error when trying to connect my OnePlus X to my Mac Mini with server.go (example). The phone never connects properly returning the "incorrect passkey" error regardless of the password typed.

The changes made by @roylee17 solve this issue for me!


Added Readme into 'Background generator"

Description 📜

Added Readme File into pre-exisiting project background-generator

Fixes #1294

Type of change 📝

[x] Improve the readme of any project

Domain of Contribution 📊

[x] Added Readme Using Markdown Syntax

Checklist ✅

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

https://user-images.githubusercontent.com/77873383/167236157-a764951e-1132-4907-b71e-6aea4b702087.mp4

@gurjeetsinghvirdee , please check


Accessory is detected as a new one after every reboot

Describe The Bug:
When the computer restarts, the same accessory is detected as a new accessory (and the previous one becomes unreachable.
Thus, every scenes gets messed up.
(The accessory is always pluged in when the OpenRGB server is up)
To Reproduce:
Shutdown the computer, restart the computer.
Start OpenRGB: openrgb --server (doing it via systemctl)
Expected behavior:
The accessory is recognized as a previously added accessory.
Logs:
[6/13/2023, 5:52:56 PM] [OpenRGB] Adding new accessory: NZXT Kraken X3
Plugin Config:
{
"name": "OpenRGB",
"servers": [
{
"name": "xxxx",
"host": "192.168.xxxx.xxxx",
"port": 6742
}
],
"discoveryInterval": 60,
"preserveDisconnected": true,
"suppressConnectionErrors": false,
"platform": "OpenRgbPlatform"
}
Environment:
Ubuntu for the computer AND homebridge. NZXT Kraken X3 device

Plugin Version: v1.4.0

Homebridge Version: [v1.6.1]

Node.js Version: v16.20.0

Operating System:  Ubuntu for the computer AND homebridge

Could you provide the information that the OpenRGB application lists in the Information tab for that cooler? Name, location, serial number, etc. Does any of that information change after a reboot? Does this happen with other devices or just that cooler?

Sure
Before reboot:
0: NZXT Kraken X3
  Type:           LED Strip
  Description:    NZXT Hue 2 Device
  Version:        2.1.0
  Location:       HID: /dev/hidraw5
  Serial:<PHONE_NUMBER>1A
  Modes: [Direct] Static Fading 'Spectrum Cycle' Marquee 'Cover Marquee' Alternating Pulsing Breathing Candle 'Starry Night' 'Super Rainbow' 'Rainbow Pulse' 'Rainbow Flow'
  Zones: 'Hue 2 Channel 1' 'Hue 2 Channel 2' 'Hue 2 Channel 3'
  LEDs: 'Hue 2 Channel 1, LED 1' 'Hue 2 Channel 1, LED 2' 'Hue 2 Channel 1, LED 3' 'Hue 2 Channel 1, LED 4' 'Hue 2 Channel 1, LED 5' 'Hue 2 Channel 1, LED 6' 'Hue 2 Channel 1, LED 7' 'Hue 2 Channel 1, LED 8' 'Hue 2 Channel 1, LED 9' 'Hue 2 Channel 1, LED 10' 'Hue 2 Channel 1, LED 11' 'Hue 2 Channel 1, LED 12' 'Hue 2 Channel 1, LED 13' 'Hue 2 Channel 1, LED 14' 'Hue 2 Channel 1, LED 15' 'Hue 2 Channel 1, LED 16' 'Hue 2 Channel 2, LED 1' 'Hue 2 Channel 2, LED 2' 'Hue 2 Channel 2, LED 3' 'Hue 2 Channel 2, LED 4' 'Hue 2 Channel 2, LED 5' 'Hue 2 Channel 2, LED 6' 'Hue 2 Channel 2, LED 7' 'Hue 2 Channel 2, LED 8' 'Hue 2 Channel 3, LED 1'

After reboot:
0: NZXT Kraken X3
  Type:           LED Strip
  Description:    NZXT Hue 2 Device
  Version:        2.1.0
  Location:       HID: /dev/hidraw1
  Serial:<PHONE_NUMBER>1A
  Modes: [Direct] Static Fading 'Spectrum Cycle' Marquee 'Cover Marquee' Alternating Pulsing Breathing Candle 'Starry Night' 'Super Rainbow' 'Rainbow Pulse' 'Rainbow Flow'
  Zones: 'Hue 2 Channel 1' 'Hue 2 Channel 2' 'Hue 2 Channel 3'
  LEDs: 'Hue 2 Channel 1, LED 1' 'Hue 2 Channel 1, LED 2' 'Hue 2 Channel 1, LED 3' 'Hue 2 Channel 1, LED 4' 'Hue 2 Channel 1, LED 5' 'Hue 2 Channel 1, LED 6' 'Hue 2 Channel 1, LED 7' 'Hue 2 Channel 1, LED 8' 'Hue 2 Channel 1, LED 9' 'Hue 2 Channel 1, LED 10' 'Hue 2 Channel 1, LED 11' 'Hue 2 Channel 1, LED 12' 'Hue 2 Channel 1, LED 13' 'Hue 2 Channel 1, LED 14' 'Hue 2 Channel 1, LED 15' 'Hue 2 Channel 1, LED 16' 'Hue 2 Channel 2, LED 1' 'Hue 2 Channel 2, LED 2' 'Hue 2 Channel 2, LED 3' 'Hue 2 Channel 2, LED 4' 'Hue 2 Channel 2, LED 5' 'Hue 2 Channel 2, LED 6' 'Hue 2 Channel 2, LED 7' 'Hue 2 Channel 2, LED 8' 'Hue 2 Channel 3, LED 1'

The location seems to change. However it's seems like a normal behavior in Ubuntu (at least)?

I've rebuild your project after removing the location from the uuid calculation and it works well. I don't know if the location stays the same on Windows platform but on Linux, the HID will change after reboot because it will depend of the order they are mounted.
A quick fix would be to add a setting to disable the location from the UUID calculation (even tho it means that users should be sure that the name of their OpenRGB devices are differents).

Hmm, it's tricky because from what I've seen, it's common for devices to report a fake or empty serial number (all but one of my devices do, for instance) and device names can easily be the same (e.g. a pair of RAM sticks). That's why location is used in the UUID calculation since #1. On Windows, it does stay the same. I'm not sure how to make it reliable for Linux too. Maybe OpenRGB could be updated to provide the device's UUID from Linux instead of its /dev path? I'll look into it.


Docs: compress png images

This PR compresses the png images in the docs/images folder to reduce overall package size (image directory size from ~9MB to ~5.7MB).  This is in response to issue #352.  Upon visually inspecting the compressed images, there does not seem to be a noticeable difference in quality.
Note: optipng -o7 filename.png was used for compression.

Thanks!


thorw an error if maxSize not specified

Right now, we are not checking for maxSize is null or not.
This PR adds a null if maxSize is not specified and will throw an error saying maxSize not specified.

Hey, thanks for taking the time!
I'm a little busy this week. I will test and merge this early next week.
Thanks again!

Redundant with https://github.com/siddharthkp/bundlesize/pull/40 and https://github.com/siddharthkp/bundlesize/pull/62


use spirv-tools parser

Replace the spirv parser with the one from https://github.com/KhronosGroup/SPIRV-Tools

As agreed in https://github.com/KhronosGroup/SPIRV-Cross/pull/767, this integration work will be done outside SPIRV-Cross' codebase.


DBZ-1491 Support default values in SQL Server

Rebase of https://github.com/debezium/debezium/pull/1113

@jpechane, can you add a high-level description of the change? Also, can you remind me what was the issue with @grzegorz8's original PR and how is it fixed now? Thanks!

@gunnarmorling There was no issue with it, it just staled a little bit and was no longer applicable to the master.

@gunnarmorling Questions answered, code changes addressed

@jpechane, re the test failure -- do you know what it is about? Saw this a few time recently: build failed but not apparent which test it is. Should be unrelated in any case, but worth tracking in Jira unless you're already working on a fix.

@jpechane, looks like a few comments are still unaddressed?

@gunnarmorling Should be pushed now

Ok, now that 1.2.1.Final is out and master truly represents 1.3, this can finally be merged. Will rebase and apply it. Thanks a lot, @grzegorz8 and @jpechane.

Applied; thanks again all!


CLOUD-56730 spark reverse proxy settings

@akanto

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2719/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/210/


Provide planner that just takes user ask to generates a plan

Currently developers need to write their own prompt to instruct OpenAI to return expected plan object. There could be cases that developers do not have complicated requirement to the planner: they don't want to write their own prompt. Instead, they just want to provide what functions are available and what they want to do, then the SDK could generate a Plan for them and execute.

@govil1811  FYI for planning :)

do you mean pass the prompt template in as a string instead of specifing a file? In order for functions to be called or variables to be referenced there would still need to be a prompt template, since they are referenced via {{ $myVar }} or {{ myFunc }}.

@aacebo I was thinking we can have a default prompt in our planner, so developers can use the default one directly (without providing the prompt files or template) if they don't have complicated scenarios. The key thing is the action (not function) list should be populated into prompt automatically instead of requires developer to update the prompt manually. This was inspired from semantic kernel which provides a default prompt inside its planner for developer to use.
The experience may be:

Developers register some actions through Application.AI.action(...)
Deploy the Teams app and it can response to app users with the registered actions.

As there're many improvements since this issue was created and the action list now can be populated automatically. I feel it can be closed if having a default built-in prompt template inside our planner is not what we want to do currently.


All views should return a 'profile' object in context

For our app... we wrap all the userena forms around a base 'profile' template...  in most userena views there is a 'profile' object returned with the view allowing us to use 'profile.whatever' within our view.  Unfortunately some views don't return this (like change email, change password, etc)... which is a real pain.

Is there a reason you did not use {{ user.profile }} in your templates?

Yes, because we've implemented 'organization' support (which I hope to maybe submit to you eventually)... where a user may have permissons to modify another user's profile (like... an organization user).  Using 'user.profile' would always use 'request.user' which in our case... we want to use the profile that matches the URL.

Ah, valid point. Will merge your pull request. Thanks for your work.


Is it possible to remove a path prefix?

I’m using filesystem with Laravel and the AWS (v3) adapter. When configuring disks in Laravel, you can specify both url and root for the S3 client. I was planning on setting the url option to a CloudFront distribution so that files would be uploaded to an S3 bucket, but then accessed via CloudFront URLs.
My files are stored in an /assets folder in my S3 bucket and the root of my CloudFront distribution is set to the folder. So a file at /assets/foo.jpg would be accessible via https://example.cloudfront.com/foo.jpg (no /assets prefix).
Therefore, I need to remove the prefix from URLs, rather than prepend. Sort of the opposite of what the third parameter to the AwsS3Adapter class constructor would do.
Is it possible to “remove” a prefix from paths?

This is outside the scope of Flysystem, you can just use substr for this.


How to merge multiple MAF files into a single MAF files? How to create sample/tumor_sample_barcode in MAF?

Hi, I am new to use maftools. I have problems about input data files.
Could you help me how to merge MAFs from all samples into a single MAF? And how to create sample/tumor_sample barcode in MAF?
Many thanks!

Hi,
Have you already generated MAF files ? If so, it should contain tumor_sample_barcode column.

Hi,
I have already generated MAF files. I can find the tumor_sample_barcode in MAF file.
But I have another problem. How do I read the MAF file?
I try to use these code:
 source("https://bioconductor.org/biocLite.R")
Bioconductor version 3.6 (BiocInstaller 1.28.0), ?biocLite for help
biocLite("maftools")
BioC_mirror: https://bioconductor.org
Using Bioconductor 3.6 (BiocInstaller 1.28.0), R 3.4.2 (2017-09-28).
Installing package(s) ‘maftools’
try URL 'https://bioconductor.org/packages/3.6/bioc/bin/windows/contrib/3.4/maftools_1.4.0.zip'
Content type 'application/zip' length 12863554 bytes (12.3 MB)
downloaded 12.3 MB
package ‘maftools’ successfully unpacked and MD5 sums checked
The downloaded binary packages are in
C:\Users\josie.wang\AppData\Local\Temp\Rtmpkb8iQn\downloaded_packages
installation path not writeable, unable to update packages: mgcv
library(maftools)
error: package or namespace load failed for ‘maftools’ in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]):
there is no package called ‘circlize’
df <- read.maf("CBCP00001-R1.maf", removeSilent = TRUE, useAll = FALSE)
Error in read.maf("CBCP00001-R1.maf",  :
no this function "read.maf"
I am pretty new to use bioconduter and cancer field for detecting significant somatic mutation variants.
Could you tell me how to conduct maftools?
Many thanks.

Hi, my another question is how to merge multiple MAF file into a single MAF file?

Hi,
MAF files are just tab delimited text files, so you can simple use cat command from terminal to append all files into a single file.
Regarding using maftools, no need to use removeSilent = TRUE as it has been deprecated. What version of the maftools are you using ? See maftools Vignette for a complete case study.

In my case, I used a cat command to merge multiple MAF files into one. I used:    cat /.maf > filename.maf
I hope you find it useful.

In my case, I used a cat command to merge multiple MAF files into one. I used: cat /.maf > filename.maf
I hope you find it useful.

Hello,
For now there is no function in maftools to do so. You can merge them into a single MAF with unix cat command and process it.

Hello,
For now there is no function in maftools to do so. You can merge them into a single MAF with unix cat command and process it.

Hi, I'm trying to use the merge_mafs() function to merge multiple maf files into a single MAF. however, it returned "Error in merge_mafs() : could not find function "merge_mafs"". It confused me because when I ?merge_mafs, it really returned the information of this function. I guess whether it's due to the warning messages when I library the maftools as below:
Warning messages:
1: no function found corresponding to methods exports from ‘GenomicRanges’ for: ‘concatenateObjects’
2: no function found corresponding to methods exports from ‘GenomicAlignments’ for: ‘concatenateObjects’
Any help would be greatly appreciated!
Many thanks!

Hi, I was trying to use the merge_mafs() function in maftools package to merge multiple maf files into a single MAF. However, it returned "Error in merge_mafs() : could not find function "merge_mafs"". It confused me because when I ?merge_mafs, it really returned the information of this function. I guess whether it's due to the warning messages when I library the maftools:
Warning messages:
1: no function found corresponding to methods exports from ‘GenomicRanges’ for: ‘concatenateObjects’
2: no function found corresponding to methods exports from ‘GenomicAlignments’ for: ‘concatenateObjects’
Any help would be greatly appreciated!
Many thanks!

Hi,
Could you post your sessionInfo ? I wonder if you're still using an older version.
Also could you try calling merge_maf usinf namespace ? e.g, maftools::merge_mafs()
Let me know.

Hi, thank you for your reply!

maftools::merge_mafs()
Error: 'merge_mafs' is not an exported object from 'namespace:maftools'
sessionInfo()
R version 3.5.3 (2019-03-11)
other attached packages:
[1] maftools_1.8.0      Biobase_2.42.0      BiocGenerics_0.28.0

Hi, thank you for your reply! I tried maftools::merge_mafs() before, but it didn't work.
maftools::merge_mafs()
Error: 'merge_mafs' is not an exported object from 'namespace:maftools'
sessionInfo()
R version 3.5.3 (2019-03-11)
other attached packages:
[1] maftools_1.8.0 Biobase_2.42.0 BiocGenerics_0.28.0
And then I installed the latest version (maftools_1.8.10) from Bioconductor, merge_mafs() still can not be used.

library(maftools)
merge_mafs()
Error in merge_mafs() : could not find function "merge_mafs"
maftools::merge_mafs()
Error: 'merge_mafs' is not an exported object from 'namespace:maftools'

Thanks!

Hi,
You're using older version of maftools. Could you try to install from GitHub and try again ?

Hi, thank you. The version is maftools_1.99.60 now.

library(maftools)
merge_mafs()
Error in merge_mafs() : could not find function "merge_mafs"
maftools::merge_mafs()
Error: 'merge_mafs' is not an exported object from 'namespace:maftools'
?merge_mafs()
starting httpd help server ... done
sessionInfo()
R version 3.5.3 (2019-03-11)
other attached packages:
[1] maftools_1.99.60    Biobase_2.42.0      BiocGenerics_0.28.0

Hello,
Sorry for the late response. Do you still have the issue ?

Hi,
Yeah, this function still doesn't work. I used the cat command instead to combine all MAF files.
Thanks!

Oh no! I just realised I never exported this function :|  No wonder you can't access it.
But you can still access it with ::: namespace. maftools:::merge_mafs().
I am so sorry and thanks for opening the issue, I will update it in later versions.

Hi,
Thank you so much for your update! Right, maftools:::merge_mafs() works! :)

Can any of you please guide how can I add sample/tumor_sample barcode in already generated MAF?

Hi,
If you have already generated the MAF, it should already have the Tumor_Sample_Barcode column.

@PoisonAlien ,
Yes it has. But all my samples have same string “Tumor” in that column. Can i replace it with the sample name, as I want merge all these MAF in-order to make an oncoplot.

If you have used mergeMafs for merging, can you check if your (merged) maf has a column called Source_MAF? Maybe you can use it as a sample name by renaming it to Tumor_Sample_Barcode

Hi @PoisonAlien
After merging 3 MAF files into 1, I got following columns in my merged MAF:
maf_1@data$Hugo_Symbol
maf_1@data$Entrez_Gene_Id
maf_1@data$Center
maf_1@data$NCBI_Build
maf_1@data$Chromosome
maf_1@data$Start_Position
maf_1@data$End_Position
maf_1@data$Strand
maf_1@data$Variant_Classification
maf_1@data$Variant_Type
maf_1@data$Reference_Allele
maf_1@data$Tumor_Seq_Allele1
maf_1@data$Tumor_Seq_Allele2
maf_1@data$dbSNP_RS
maf_1@data$dbSNP_Val_Status
maf_1@data$Tumor_Sample_Barcode
maf_1@data$Matched_Norm_Sample_Barcode
maf_1@data$Match_Norm_Seq_Allele1
maf_1@data$Match_Norm_Seq_Allele2
maf_1@data$Tumor_Validation_Allele1
maf_1@data$Tumor_Validation_Allele2
maf_1@data$Match_Norm_Validation_Allele1
maf_1@data$Match_Norm_Validation_Allele2
maf_1@data$Verification_Status
maf_1@data$Validation_Status
maf_1@data$Mutation_Status
maf_1@data$Sequencing_Phase
maf_1@data$Sequence_Source
maf_1@data$Validation_Method
maf_1@data$Score
maf_1@data$BAM_File
maf_1@data$Sequencer
maf_1@data$Tumor_Sample_UUID
maf_1@data$Matched_Norm_Sample_UUID
maf_1@data$HGVSc
maf_1@data$HGVSp
maf_1@data$HGVSp_Short
maf_1@data$Transcript_ID
maf_1@data$Exon_Number
maf_1@data$t_depth
maf_1@data$t_ref_count
maf_1@data$t_alt_count
maf_1@data$n_depth
maf_1@data$n_ref_count
maf_1@data$n_alt_count
maf_1@data$all_effects
maf_1@data$Allele
maf_1@data$Gene
maf_1@data$Feature
maf_1@data$Feature_type
maf_1@data$Consequence
maf_1@data$cDNA_position
maf_1@data$CDS_position
maf_1@data$Protein_position
maf_1@data$Amino_acids
maf_1@data$Codons
maf_1@data$Existing_variation
maf_1@data$ALLELE_NUM
maf_1@data$DISTANCE
maf_1@data$STRAND
maf_1@data$SYMBOL
maf_1@data$SYMBOL_SOURCE
maf_1@data$HGNC_ID
maf_1@data$BIOTYPE
maf_1@data$CANONICAL
maf_1@data$CCDS
maf_1@data$ENSP
maf_1@data$SWISSPROT
maf_1@data$TREMBL
maf_1@data$UNIPARC
maf_1@data$RefSeq
maf_1@data$SIFT
maf_1@data$PolyPhen
maf_1@data$EXON
maf_1@data$INTRON
maf_1@data$DOMAINS
maf_1@data$GMAF
maf_1@data$AFR_MAF
maf_1@data$AMR_MAF
maf_1@data$ASN_MAF
maf_1@data$EAS_MAF
maf_1@data$EUR_MAF
maf_1@data$SAS_MAF
maf_1@data$AA_MAF
maf_1@data$EA_MAF
maf_1@data$CLIN_SIG
maf_1@data$SOMATIC
maf_1@data$PUBMED
maf_1@data$MOTIF_NAME
maf_1@data$MOTIF_POS
maf_1@data$HIGH_INF_POS
maf_1@data$MOTIF_SCORE_CHANGE
maf_1@data$IMPACT
maf_1@data$PICK
maf_1@data$VARIANT_CLASS
maf_1@data$TSL
maf_1@data$HGVS_OFFSET
maf_1@data$PHENO
maf_1@data$MINIMISED
maf_1@data$ExAC_AF
maf_1@data$ExAC_AF_AFR
maf_1@data$ExAC_AF_AMR
maf_1@data$ExAC_AF_EAS
maf_1@data$ExAC_AF_FIN
maf_1@data$ExAC_AF_NFE
maf_1@data$ExAC_AF_OTH
maf_1@data$ExAC_AF_SAS
maf_1@data$GENE_PHENO
maf_1@data$FILTER

But cannot locate the Source_MAF column in it.
Please have a look.

Can you post your command? Also are you merging maf objects or files?

Here are these
sam1_syn4985519_maf <- read.maf(maf ="tumorVsNormal_pat1_tumor_syn4985378_vs_norm_syn4985519.maf.gz", useAll = T, verbose = T)
sam1_syn4985596_maf <- read.maf(maf ="tumorVsNormal_pat1_tumor_syn4985596_vs_norm_syn4985519.maf.gz", useAll = T, verbose = T)
sam1_syn4987466_maf <- read.maf(maf ="tumorVsNormal_pat1_tumor_syn4987466_vs_norm_syn4985519.maf.gz", useAll = T, verbose = T)

maf_1=maftools:::merge_mafs(maf=c(sam1_syn4985519_maf,sam1_syn4985596_maf,sam1_syn4987466_maf), verbose = TRUE)

No need to perform first three steps. Just pass all your maf.gz to merge_mafs function. Let me know if you still have issues (which I think will be the case :|)

Ok will try and update you on this.
Thanks!

Hi @PoisonAlien
Thank you, now I can see the Source_MAF after merging files instead of MAF object.
I have another question related to oncoplot(), I am using following command to rename my samples:
maf_merged@data$Tumor_Sample_Barcode=maf_merged@data$Source_MAF
Will that be enough or do I need change something else before parsing the maf_merged object to oncoplot()?

This should work just fine. You may want to do the same for<EMAIL_ADDRESS>slot.

No need to perform first three steps. Just pass all your maf.gz to merge_mafs function. Let me know if you still have issues (which I think will be the case :|)

Hi @PoisonAlien could you please share a way to pass many mafs into the merge_mafs function? I have more than 40 mafs, and the merging works only if I write them by hand...
Like, this works just perfectly:
merged <- merge_mafs(c(PRM_ICH_1,PRM_ICH_15))

But I can't make it work in a loop:
for(i in names){
  maf_merged_08092021 <- merge_mafs(i)
  return(maf_merged_08092021)
}

where "names" is the samples names' list that worked perfectly in the 40 mafs import into R.

Hi,
You could just grep all the files maf files with list.files and pass them to the merge_mafs

If you have used mergeMafs for merging, can you check if your (merged) maf has a column called Source_MAF? Maybe you can use it as a sample name by renaming it to Tumor_Sample_Barcode

Hi, I am trying to merge all the MAFs in-order to make an oncoplot but get an error. Can you please help me with that?
code
`#change the Tumor_Sample_Barcode to what I want to show
hcc_merge_all_copy@data$Tumor_Sample_Barcode = hcc_merge_all_copy@data$Source_MAF
hcc_merge_all_copy@data$Tumor_Sample_Barcode <- gsub("_.*", "", hcc_merge_all_copy@data$Tumor_Sample_Barcode)
#determine the final order what I want to show
sample_order <- gsub("_.*", "", mafs)
sample_order <- substr(sample_order, start = 3, stop = nchar(sample_order))

#draw
pdf("./waterfall_rename_4.pdf", bg="white",height = 5,width = 7)
oncoplot(hcc_merge_all_copy,
showTumorSampleBarcodes = T,sampleOrder = sample_order)
dev.off()
waterfall_rename_4.pdf
`
I use the orignal maf object and the Tumor_Sample_Barcode renamed maf object to draw oncoplots respectively. The latter one miss the bar chat showing TMB.


Please provide means to download updated cell database

Hello,
I suggest that instead of shipping the cell database with the APK (which renders the program to be huge, please bear in mind that some devices don't have much space in /data), that you would add an activity which allows the user to download the update cell database from OpenCellID, then convert the CSV to a SQLite database that is stored on the sdcard for example (or maybe the app's data dir), that is how mar-v-in implemented the old offline GSM location provider (though user had to manually download the database).
I have a question about the current design though, is the cell database extracted everytime the service is started ? or is it just the first time the user uses the service ?

What is the current status of the database format on the phone?  I've found the sources to download the CSV on my computer and would love to transfer it to my phone in its entirety, but if CSV is not the appropriate format then:

what exactly is?
is that format accessible? or still being developed?
are there any developed binaries to convert the CSV to the appropriate format?

I'm decent with text manipulation.  I can write an AutoHotkey script (and even compile an EXE for Windows users) to convert the huge CSV to the appropriate format, though it would take time for me to learn to work with memory limitations.
Loving this app btw, so glad to be GApps free!!!!!


fix some typos and errors

in the context, this means that --sink flag takes an argument of the form PREFIX:CONFIG[?OPTIONS], heapster can store data into multiple sinks at once not because of taking this argument form, but because of specifying multiple --sink flags
fix some typos
Google Compute Enginer should be Google Compute Engine
opentsdb should be OpenTSDB in normal context.
ElasticSearch should be Elasticsearch in normal context

Signed-off-by: bruceauyeung<EMAIL_ADDRESS>

This change is

@bruceauyeung Thanks for your contribution!
/lgtm

@vishh @mwielgus
hi, do you have a moment to check if this is a reasonable PR? thanks!


Feature/endian

I added an Endian optional parameter to the methods in ReadWriteExtension, to match the parameter on ByteData.  I tested it briefly, everything seems fine to me.  I also lumped in another change - I added methods for readInt8, writeInt8, readUint8, and writeUint8, again to match ByteData (and because it wasn't clear to me whether the existing readByte etc. was signed or unsigned).
Thanks for the library; other libraries kept giving me combinations of fixed buffer sizes and separate reading/writing and a different interface than I was previously using, etc.

Another way of dealing with endianness might be to add a field to set it semi-permanently, but I don't think that'd work in the extension.

The only thing im going to change is the default value, I will make the endianness nullable and create a global default endianness variable for QOL.

Seems good to me, too.  Thanks!


[FLINK-16202][sql] Support JSON_QUERY for blink planner

What is the purpose of the change
Support JSON_QUERY function for blink planner
Brief change log
(for example:)

Introduce JSON_QUERY to FlinkSqlOperatorTable
Add corresponding test cases

Verifying this change
(Please pick either of the following options)
This change added tests in ScalarFunctionsTest.scala
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

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

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

hi, @wuchong I have addressed it.

@XuQianJin-Stars Do you think that we need to add some tests like:

fail on invalid input and path, and validate the exceptions are threw correctly?
fail on wrong field types?

2. fail on wrong field types?

invalid path already exists，invalid input can add.
Tried this type of error can not be a catch.
Tried this type of error can not be a catch.

    try {
      testSqlApi("json_query(f7, 'lax $' error on empty)", "null")
    } catch {
      case e: Exception => {
        assertEquals(e.getClass, expectedException)
      }
    }

@XuQianJin-Stars

invalid path indeed exists, what if we do not specify error handler? like json_query(f8, 'invalid $.name')
about testing the expected exception, you could refer ExpressionTestBase.thrown.

hi, @libenchao I addressed it.

@XuQianJin-Stars Thanks for the updating. LGTM generally.
One more question, do you plan to support JSON_QUERY in table api? If yes, then we may could add it in this pr or open another issue to track this.

@XuQianJin-Stars Thanks for the updating. LGTM generally.
One more question, do you plan to support JSON_QUERY in table api? If yes, then we may could add it in this pr or open another issue to track this.

I see @wuchong  has opened a new PR to implement TableApi, and I plan to implement it separately.

LGTM, +1 for merging.

Finalized in #16858.


Fix code scanning alert - Replacement of a substring with itself

Tracking issue for:

[ ] https://github.com/jsenv/core/security/code-scanning/5

Fixed


[Code structure] Broken highlighting after certain edits

Delete and reinstate the colon after "happy" in the (partially minimised) code below. Observe highlighting is broken.
It is sensitive to comment length (e.g. you can delete some but not all of the "random 99" comment). A CM incremental editing bug?
from microbit import *
import music
from random import *
program_list = []
global program_counter
program_counter = -1
variables = [None]*26
operators = '+-/*'

def parse(instruction):
    global program_counter
#    print(program_counter)
    if instruction.startswith('goto '):
        a = 1
    elif instruction == 'stop':
        a = 1
    elif instruction.startswith('beep '):
        a = 1
    elif instruction.startswith('random '):
        # random a 99 puts a random number between 1 and 99 in variable a
        var_rnd = instruction[7]
        var_index = ord(var_rnd) - 97
        rnd_range = int(instruction[9:])
        variables[var_index] = randint(1,rnd_range)
    elif instruction.startswith('if '):
        var1 = ord(instruction[3]) - 97
        operator = instruction[4]
        var2 = ord(instruction[5]) - 97
        goto = int(instruction[12:])
        if goto  < 0 or goto > len(program_list):
            print('Goto error: line doesn\'t exist.')
        else:
            if operator == '=':
                if variables[var1] == variables[var2]:
                    program_counter = goto - 1
            elif operator == '>':
                if variables[var1] > variables[var2]:
                    program_counter = goto - 1
            elif operator == '<':
                if variables[var1] < variables[var2]:
                    program_counter = goto - 1
            else:
                print('If error: comparsion operators must be =, < or >')
    elif instruction.startswith('scroll '):
        split = instruction.find(' ') + 1
        display.scroll(instruction[split:])
    elif instruction.startswith('input ') and len(instruction) == 7:
        var_input = instruction[6]
        var_index = ord(var_input) - 97
        if var_index >= 0 and var_index <= 26:
            input_text = input()
            try:
                variables[var_index] = int(input_text)
            except ValueError:
                variables[var_index] = input_text
        else:
            print('Variable names must be letter a-z.')
    elif instruction.startswith('print '):
        if len(instruction) == 7 and instruction[6].isalpha():
            var_print = instruction[6]
            var_index = ord(var_print) - 97
            print(variables[var_index])
        else:
            split = instruction.find(' ') + 1
            print(instruction[split:])
    elif instruction[3] in operators and instruction[1] == '=' and ord(instruction[0]) > 96 and ord(instruction[0]) < 123 and ord(instruction[2]) > 96 and ord(instruction[2]) < 123 and ord(instruction[4]) > 96 and ord(instruction[4]) < 123:
        var1 = instruction[0]
        var2 = instruction[2]
        var3 = instruction[4]
        operator = instruction[3]
        if operator == '+':
            variables[ord(var1)-97] = 1
        elif operator == '-':
            variables[ord(var1)-97] = 1
        if operator == '/':
            variables[ord(var1)-97] = 1
        if operator == '*':
            variables[ord(var1)-97] = 1
    elif '=' in instruction:
        split = instruction.find('=') + 1
        var_name = instruction[:split-1]
        var_contents = instruction[split:]
        if len(var_name) == 1 and var_name.isalpha():
            try:
                variables[ord(var_name)-97] = 1#nt(var_contents)
            except ValueError:
                variables[ord(var_name)-97] = var_contents
#            print(var_name, '=', var_contents)
        else:
            print('Variable names must be one letter a-z.')
    elif instruction == 'heart':
        display.show(Image.HEART)
    elif instruction == 'happy':
        display.show(Image.HAPPY)
    elif instruction == 'sad':
        display.show(Image.SAD)
    elif instruction == 'meh':
        display.show(Image.MEH)
    elif instruction == 'confused':
        display.show(Image.CONFUSED)
    elif instruction == 'angry':
        display.show(Image.ANGRY)
    elif instruction == 'asleep':
        display.show(Image.ASLEEP)
    elif instruction == 'yes':
        display.show(Image.YES)
    else:
        display.show('?')

The outstanding CM bug has been fixed. Matt to review and plan upgrade. For independent reasons there's already an upgrade PR but that hit unrelated issues.

Hopefully fixed by #563

Fix in #563. Closing.


Quick question : can you disclose the type of rng hakaru is using

To compute samples using probability there's typically rng used so out of curiosity I wonder what is it

Haskell's System.Random.MWC from the mwc-random package.  See https://hackage.haskell.org/package/mwc-random-<IP_ADDRESS>/docs/System-Random-MWC.html for more details.


Correctly close service store result

What type of PR is this?

What does this PR do / why is it needed ?

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

/easycla

/easycla


Add Shiawase (VIP) Low Quality  by Dion Timmer

Song Name
Shiawase (VIP) Low Quality
Artist Name
Dion Timmer
Source
Youtube
Youtube Link
0zmE65ptla4
Direct File Link
No response
Song ID
1266014
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 104356905

reject

bad remix


add operational need section

closes #90

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#73  👈
#70
#69
#67
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dyc3 and the rest of your teammates on  Graphite

Merge activity

May 11, 2:33 PM EDT: @dyc3 started a stack merge that includes this pull request via Graphite.


Masking variables as secret

After consulting with our Microsoft Affiliate I was able to find that variables can be masked as secrets to prevent them from showing up in the logs, however, I could not find that on this page.
The documentation is apparently found in the reference documentation but I would expect it here as well, in the setvariable in scripts part of the document.

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

ID: dd7e0bd3-1f7d-d7b6-cc72-5ef63c31b46a
Version Independent ID: dae87abd-b73d-9120-bcdb-6097d4b40f2a
Content: Define variables - Azure Pipelines
Content Source: docs/pipelines/process/variables.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks, I'm adding a link now to the logging page and a note about issecret.


maidenhead.c  needs Grad Seconds Minutes conversation to decimal

The RMC sentence delivers coordinates used like in APRS.  So i wonder why the QRA locator shows not exact the right one.
The maidenhead function :
char* get_mh(double lat, double lon, int size)
needs an converting like this:

double gms2dec(double gms) {
double degrees;
double minutes;
degrees = (int)(gms);
minutes = (gms - degrees) * 100;
degrees = degrees + minutes/60;
return degrees;
}

and insert the two lines:

char* get_mh(double lat, double lon, int size) {
static char locator[11];
double LON_F[]={20,2.0,0.083333,0.008333,0.0003472083333333333};
double LAT_F[]={10,1.0,0.0416665,0.004166,0.0001735833333333333};
int i;
// if lat, lon in APRS format then convert to dezimal
lat = gms2dec(lat);
lon = gms2dec(lon);

I think the RMC values are always the same format over all GPS chips. So it was here with two types of ublox units.
Please check it with your type of GPS.
Thanks
Wolf

so my understanding is that the GNRMC format gives latitude as ddmm.mmmmmmm. The code on lines 213 and 228 of gpstime.c is supposed to convert it to pure degrees. (Or at least pure degrees multiplied by large value to keep the format 64bit integer...)
The earlier versions of this code (including the one that flew around the world) was not doing this correctly. It was only the commit I made on April 6, 2024 that fixed this problem.
At least I think it fixed the problem. The code on lines 213 and 228 is not very clear or elegant, but it was my intention that it is doing essentially the same calculation (minutes = (gms - degrees) * 100; degrees = degrees + minutes/60;) that you showed in your comment.
If you think its incorrect please let me know, but my bench testing currently is showing accurate values.

An i would knowledge that i checked this with the last version. It is fixed as told.


Add Enabled: true to Rake department config

Follow up of https://github.com/rubocop-hq/rubocop-minitest/issues/25#issuecomment-543504420.
This PR adds Enabled: true to Rake department config. It resolves the following warning:
# .rubocop.yml
require: rubocop-rake

AllCops:
  DisabledByDefault: true

Rake:
  Enabled: true

% bundle exec rubocop
Warning: Rake does not support Enabled parameter.

Supported parameters are:

  - Include

Inspecting 1 file
.

1 file inspected, no offenses detected

Sorry for the late reply. Thanks!


Publish 7.4.2-20220325130423+0000

Publish 7.4.2-20220325130423+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.


Update 0000-01-01-intro.md

No description provided.

No description provided.

No description provided.


Add Tests

I had a look on codecov and I saw a few places we could easily and quickly add tests, so I did so :)

Thank you!


Added dark red star history chart

Added dark red star history chart

Hey @github-star-history, thank you for your efforts towards this contribution. Unfortunately, I can't merge this PR to main because we already have star-history in place.


JobLog log UI fix and isUITrigger() 'trigger' of null" TypeError

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1809
Originally created by @derek-knox on 2020-02-26 21:52:02

View 5 previous comments from the original issue on GitLab


Adding GetChannelsWithPortPrefix function to 04-channel

Description

closes: #2245

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #2304 (b5e1e9f) into main (c2ce323) will decrease coverage by 0.08%.
The diff coverage is 21.73%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2304      +/-   ##
==========================================
- Coverage   79.51%   79.43%   -0.09%
==========================================
  Files         175      175
  Lines       12077    12087      +10
==========================================
- Hits         9603     9601       -2
- Misses       2049     2062      +13
+ Partials      425      424       -1

Impacted Files
Coverage Δ

modules/core/04-channel/keeper/keeper.go
79.72% <0.00%> (-5.04%)
:arrow_down:

...nterchain-accounts/controller/keeper/migrations.go
81.25% <80.00%> (+1.25%)
:arrow_up:

...nterchain-accounts/controller/keeper/msg_server.go
100.00% <100.00%> (+8.33%)
:arrow_up:

modules/apps/27-interchain-accounts/module.go
52.57% <0.00%> (+2.06%)
:arrow_up:


Support for the v2 POST API

Do you have any interest in supporting the v2 POST API? It requires authentication (use your “S3-like API credentials” from https://archive.org/account/s3.php), but has a lot of super useful features.
I’ve been poking at it a fair amount lately, and would be happy to try and help add support for it here if you’re interested.

(Update: looks like they are concerned about extra load on that API under the current situation, so I dropped the docs link.)

@Mr0grog Would appreciate if you could tell me whats the difference between v1 and v2 POST api besides the outlinks?. Can't find the api docs.

Can't find the api docs.

@vegetableman see above comment — docs are not yet public, so I had to remove the link. I got ahead of myself here and thought they were ready for broader use after this blog article: https://blog.archive.org/2019/10/23/the-wayback-machines-save-page-now-is-new-and-improved/

Would appreciate if you could tell me the difference between v1 and v2 POST api

The POST-based API has some pretty fancy features. You can:

Poll for completion or just make the request to save and forget it
Save a screenshot
Save error pages (normally it only saves 2xx responses)
Set a time limit
Set cookies to use when requesting the page to save
Set basic auth credentials to use when requesting the page to save
Get a huge amount of useful metadata about the saved result

@Mr0grog Thanks Rob for sharing the details 👍. I was able to dig some details on the new api through devtools on the spn page, specifically, save/status/<jobId> that fetches the save completion status through jobId.
Although I do get the new snapshot details for a url, the issue is, even after the completion of the job, the new snapshot details are still not readily available for the url through this api: https://archive.org/wayback/available?url=<url>
It takes about ~ 10 - 30 minutes or more for the details to be available.

Hmmmm, my experience has been that the availability time is about the same as an old-style GET request to SPN. (But I’ve been using the CDX API, not the availability API.) Are you sure it’s not just that SPN has been under very heavy load since the current coronavirus situation started?

Alright. So, the CDX api /search/cdx?url= is what I should have been using. Unlike the available api, I am getting the new snapshot details immediately through it. Thanks Rob  👍🙂.

I don't think the virus situation has anything to do with this. Also, I don't think SPN is at fault. My guess is, the data source for the availability api is updated through a queue of some sort. Where as, the data source for CDX is updated immediately .

I'd be open to including such a thing, but I'm clearly pretty far behind you on the learning curve. If you had time to prepare a pull request I'd be open to it.

Like I said in July, I'd take a pull request if someone had one. In the meantime, I'm going to close this ticket as stale.

@palewire Sorry for the radio silence! This was the kick in the pants I needed to check back in with folks at the Internet Archive and they are 👍 on it now, so I will try and find some time to start implementing.

@palewire Sorry for the radio silence! This was the kick in the pants I needed to check back in with folks at the Internet Archive and they are 👍 on it now, so I will try and find some time to start implementing.

OK, I posted a draft of this as #31. It probably gets the job 80% done, but needs some feedback before spending a whole lot more time on it.

OK, I posted a draft of this as #31. It probably gets the job 80% done, but needs some feedback before spending a whole lot more time on it.


fix(bin): add js extension to binary settings

Current dependencies on/for this PR:

main

PR #21   👈

This comment was auto-generated by Graphite.


No more discord server?

I tried looking for it in the README, but also looking through the commit history, seems like it got taken out

https://github.com/ianmarmour/nvidia-clerk#discord-server


AppConfiguration 2024-09-01-preview API version

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

A required check has failed - please fix.
https://github.com/Azure/azure-rest-api-specs/pull/29677/checks?check_run_id=27297663464

Please ensure the first iteration contains the base-api version as is, according to the guidelines here - https://armwiki.azurewebsites.net/rp_onboarding/process/api_review.html
This helps reviewers only review the updates to new versions and not have to review the full API files.


Rsync error after vagrant up

Version: coroot-node-agent-v1.21.4
Vagrant's error info:
Host path: /root/Source/obser/coroot-node-agent-v1.21.4/
Guest path: /tmp/src
Command: "rsync" "--verbose" "--archive" "--delete" "-z" "--copy-links" "--no-owner" "--no-group" "--rsync-path" "sudo rsync" "-e" "ssh -p 22 -o LogLevel=FATAL   -o ControlMaster=auto -o ControlPath=/tmp/vagrant-rsync-20240910-2986741-fuu8t5 -o ControlPersist=10m  -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i '/root/Source/obser/coroot-node-agent-v1.21.4/ebpftracer/.vagrant/machines/ubuntu2004/libvirt/private_key'" "--exclude" ".vagrant/" "/root/Source/obser/coroot-node-agent-v1.21.4/" "vagrant@<IP_ADDRESS>:/tmp/src"
Error: symlink has no referent: "/root/Source/obser/coroot-node-agent-v1.21.4/proc/fixtures/123/fd/5"
symlink has no referent: "/root/Source/obser/coroot-node-agent-v1.21.4/proc/fixtures/123/fd/4"
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]

I solved this by manually removing symlink: rm -r ./proc/fixtures/123/fd/.

Maybe we can tell Vagrant to skip those symlinks


Support reconnect/create in JupyterUriProvider

Currently JupyterUriProvider supports providing a list of jupyter server uris to the Jupyter extension but when the endpoint is no longer alive, and Jupyter still holds a connection to the endpoint, there isn't a way to reestablish a new connection easily, other than

Clear that connection
Request the provider extension to create a new connection

It would be great that if we can reinitiate/create a live connection to replace the legacy one. The JupyterUriProvider can reuse the same handle and when Jupyter asks for getServerUri(handle), it can then reinitiate a new connection and provide the new url/token to Jupyter to replace the old one.

@rebornix not sure i see the purpose for this.
I think we can get the exact same behaviour today by:

Disposing that related url/handle
Then user will have to select the server (for the new handle) and from the server select a new kernel

Why wouldn't that work, would the end workflow/ux be the same?

Lets close this issue for now, been open and we haven't run in to any issues here


Missing close, minimize button on the workspace opened in fullscreen mode

Type: Bug
Dear vscode team,
I hope this message finds you well.
Currently I'm not able to see the three buttoons for the vscode workspace that are opened in fullscreen mode on macOS.
Please see the attached pics:

And the 3 buttons that I am referring to:

I really hope this gets fixed.
Best regards,
Dongmin Lee
VS Code version: Code 1.95.1 (Universal) (65edc4939843c90c34d61f4ce11704f09d3e5cb6, 2024-10-31T05:14:54.222Z)
OS version: Darwin arm64 24.1.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Pro (11 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
2, 2, 3

Memory (System)
18.00GB (3.24GB free)

Process Argv
--crash-reporter-id 6ce5b2a3-c772-4c9c-930d-5587ec7b684f --crash-reporter-id 6ce5b2a3-c772-4c9c-930d-5587ec7b684f

Screen Reader
no

VM
0%

Extensions (31)

Extension
Author (truncated)
Version

js-auto-backticks
cha
1.2.0

vscode-markdownlint
Dav
0.56.0

vscode-eslint
dba
3.0.10

gitlens
eam
15.6.2

vscode-html-css
ecm
2.0.10

vsc-material-theme
Equ
34.7.7

prettier-vscode
esb
11.0.0

auto-close-tag
for
0.5.15

copilot
Git
1.243.0

copilot-chat
Git
0.22.1

vscode-pull-request-github
Git
0.100.0

vsc-python-indent
Kev
1.18.0

vscode-gutter-preview
kis
0.31.2

vscode-language-pack-ko
MS-
1.95.2024103009

debugpy
ms-
2024.12.0

python
ms-
2024.18.0

vscode-pylance
ms-
2024.10.1

jupyter
ms-
2024.10.0

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.21

vscode-jupyter-cell-tags
ms-
0.1.9

vscode-jupyter-slideshow
ms-
0.1.6

live-server
ms-
0.4.15

indent-rainbow
ode
8.3.1

vscode-gitignore-generator
pio
1.0.3

material-icon-theme
PKi
5.12.0

gitmoji-vscode
sea
1.2.5

pdf
tom
1.2.2

intellicode-api-usage-examples
Vis
0.2.9

vscodeintellicode
Vis
1.3.2

JavaScriptSnippets
xab
1.8.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
01bff139:31013167
dvdeprecation:31068756
dwnewjupytercf:31046870
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013

@ssoaora a window that is in macOS fullscreen mode typically does not have these three buttons, did you find an app otherwise that does this? The only exception is that if you move your mouse up outside of the window, it should appear:

@ssoaora a window that is in macOS fullscreen mode typically does not have these three buttons, did you find an app otherwise that does this? The only exception is that if you move your mouse up outside of the window, it should appear:

@bpasero Yes the Jetbrains IDEs support the feature I described.
screenshot from CLion:

@ssoaora but are you certain the window is macOS native fullscreen in that case or just maximised? When you click on the green icon do you to maximise further?

@bpasero Yep if I hover over with my mouse the message shows 'Exit fullscreen mode' like in the screenshot.

Ok interesting, must be a window mode I was not aware of. Treating as feature request.


Time-consuming installation of the library

I am trying to install library ojd_daps_skills in Google Colab. It takes to long, to install all requirement (46 minutes and its not done yet). Same problem when I try to install requirement manually, pip tries to match versions so that there are no conflicts, which makes the download process very slow (Below will be pip info). I would be very grateful, if you specify versions of all libraries.
INFO: This is taking longer than usual. You might need to provide the dependency resolver with stricter constraints to reduce runtime. If you want to abort this run, you can press Ctrl + C to do so. To improve how pip performs, tell us what happened here: https://pip.pypa.io/surveys/backtracking

many thanks for this @Delmark1904 - we will try to do this soon

Address requirements properly:
move analysis/none-user requirements to requirements_dev

This should be much better now @Delmark1904


Category Search and Set stream Info

Sorry for the messy commit history, i've been trying to fix it and this was the best I could do.  As for the highlight nomenclature, so far it seems names are tightly coupled with kick's naming scheme, so i just copied the names of the keys from the api.  if you've got a name suggestion i'm happy to change it.

Could you add the attribute descriptions into the property docstrings?

Could you add the attribute descriptions into the property docstrings?

Changes made


Show Documentation and Implementations Details in storybook

Storybook has addon configurations available for showing the code used for a given story.
It can also show the props available to pass in to a component.
These would be beneficial for showing how these layout systems work at a glance.

I tried 2 plugins that were supposed to do it but I wasn't able to get it to work with HOC, It just listed everything as 'Component' even when setting display Name.
👍 if you get them to work

Is this complete now with the JSX plugin or do you want to document all props as well?

This can be closed since we now have implementation details within Storybook.
See https://github.com/obartra/reflex/issues/80 for additional documentation needs.


Remove BailOutExpectingObject.

As per issue 266, this bailout has been deleted because it was never assigned.
https://github.com/Microsoft/ChakraCore/issues/266

Hi @epsom-software, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@epsom-software, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Thanks for your PR, @epsom-software . Your changes look good :+1:


Feature Request: Fluent Converters

Fluent converters for serialization would be a wonderful alternative to using attributes and having to create another poco to keep the attributes out of the model. Something similar to the syntax described here: https://stackoverflow.com/questions/26801453/fluent-converters-mappers-with-json-net
If there is an extension point this could be added to and someone could provide an example i'd be glad to dive in!

I don't want to add this to the core library. Someone could create it as its own NuGet library over the top of the existing API.

+1 That would be really nice to have it in the light of new System.Text.Json coming in.
I'm trying to ship a lightweight package with contracts without dependency on serializer.
@JamesNK As I understand key classes to look at are JsonContract and DefaultContractSerializer ?


Use assertThat#instanceOf and assertThat#isNotInstanceOf [Buffer]

Motivation:
We use assertTrue(obj instanceOf Cat) for asserting object instances. While this is correct and works, it becomes impossible to debug when the assertion fails because we have no clue at all which instance type made the assertion fail.
Modification:
To address this, we should use assertThat#instanceOf which will do the exact same thing but in a much cleaner way. Also, it will give us more information when it fails like why the assertion of instance check failed and what exactly we got in place of the expected one.
Result:
Cleaner and easier to debug test case failure
This PR migrates the entire buffer module completely to the new API.
Previous PR: #13130

Build stalled. Re-trigerring.


Add params_to_tune for DeepARModel and TFTModel

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes
Look at #1197.
Closing issues
Closes #1197.

Codecov Report

:exclamation: No coverage uploaded for pull request base (automl-2.0@db8cd3a). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 854fce1 differs from pull request most recent head 5ca505f. Consider uploading reports for the commit 5ca505f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##             automl-2.0   #1210   +/-   ##
============================================
  Coverage              ?   7.74%
============================================
  Files                 ?     177
  Lines                 ?   10536
  Branches              ?       0
============================================
  Hits                  ?     816
  Misses                ?    9720
  Partials              ?       0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Putting spark-hdfs on BackendModule

Part of #2216

@rabbitonweb is this ready for review? If so, can you assign someone?
And also it needs a semantic version label.

@alissapajer not yet ready for review, will be soon.

@wemrysi this is ready for review, ChrootedInterpreter added as discussed :)

@wemrysi ready


Where can I find the ISO 19139 files?

I need to create a custom profile as explained at https://github.com/Esri/geoportal-server/wiki/Add-a-Custom-Profile and the customer wants to based their profile on ISO 19139, with some modifications.
Which is the best existing file to use as a starting point?
I can't see ISO 19139 in the geoportal\WEB-INF\classes\gpt\metadata\ directory, although iso-19119-definition.xml does contain referenes to 19139.
(I'm not an expert at the various ISO standards so I'm a bit confused by it all... thanks)

I can see some files under \geoportal\WEB-INF\classes\gpt\gxe\iso\iso19139 but 19139 is not mentioned in my schemas.xml file, so 19139 doesn't show when I choose > Add > Use dedicated editor to create metadata manually
I tried adding 19139 to schemas.xml but this doesn't cause 19139 to show:
  <schema fileName="gpt/metadata/iso/iso-19115-definition.xml"/>
  <schema fileName="gpt/metadata/iso/iso-19119-definition.xml"/>
  <schema fileName="gpt/gxe/iso/iso19139/iso19139-dataset-editor.xml"/>

I suspect I'm misunderstanding something fundamental here.

hi Stephen, ISO 19139 is just the xml encoding of 19115/19119. the two editors apply 19139 encoding rules already, so I would not expect a separate 19139 editor.
in the .../gpt/metadata folders you just make the definition of the schema. a good example ISO profile is .../gpt/metadata/iso/inspire. there is a template (an example file used with defaults), a definition file (where some stylesheets are listed for details, csw responses etc), the interrogation rule (how does geoportal server recognize the schema?), and most importantly: a reference to the Geoportal XML Editor </editor>.
That editor (for INSPIRE for example) is in .../gpt/gxe/iso/iso19139/profiles/inspire.


fix(datasets): Add compatibility shim to tensorflow datasets

Description

Continuation of #255.
Development notes

Checklist

[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes

Is this PR still needed @astrojuanlu ?

It is not, since there's no compatibility layer anymore


userinfo ?

Is there a way to add userinfo to nstimer ? so i can access a specific timer when making multiple timers dynamically ?

Could you please elaborate on your use case?

i need to create multiple instances of timer dynamically, to play music at specific times.
but then also be able to to invalidate timers, in case user decides to cancel
i was thinking of passing custom userinfo and then be able to access that particular nstimer based on that

I still don't understand. Why user info? Why can't you just assign your timers to an array or something?

sorry, yes, im sure there are lots of ways of accomplishing what im asking.
im fairly new to swift, and was able to figure how to do that for user notifications using userinfo, so thought i could re use a similar code.

Alright. I hope you've figured it out. Sorry, user info doesn't sound at all like a good approach for this.


Add tests for RefTypeSyntaxWrapper

This wrapper type was added in #2370 without tests. It's not too difficult to add them; see commit c31beae and other examples from that pull request for how to implement good coverage of these types.

Grabbing this


feat(cloudnative-pg): clusterrole aggregated to user facing roles

create rbac to view/edit cnpg crds and aggregate to user facing roles
view/edit/admin clusterroles are bound to a user & namespace, to manage said namespace, and now allow to create a cluster/backup/etc  with this change.

Can you remove the bump to the version? Otherwise looks good to me, thanks for switching to false by default 🙏

Is this still relevant or should we close it?
@jouve

forgot about this one, it was implemented by #39


Permission group problems

Sometimes the <player.groups> flag returns an empty list... (li@)
... althoug the player has at least one group.
The "older" Groups like Owner, Admin, Builder, Mod, ... work as expected.
But newer Groups like Youtuber, Donator, Streamer, ... don't work.
I'm using latest Spigot, Pex and Vault.

we recommend <player.in_group[group].global> to check for groups :)


Update sqlite.json to include $HOME/.sqliterc

SQLite version 3.41.0 includes support for moving the .sqliterc file into the $XDG_CONFIG_HOME folder.

Thank you!


Extend timestamp variable parameters in File name format

Scenario Overview
We use s3 filename template prefix/'%Y_%m_%d__%H_%M_%S_%f to sort filenames alphabetically.
The next new file is guaranteed to receive the following name in alphabetical order.
In kafka, we have several partitions of one topic, each of them must be written with the same prefix (prefix=topic_name) in order.
It's possible to ensure the files order with this template by running no more than 1 connector task.
Issue:
Timestamp variable have next parameters:
unit parameter values:
yyyy - year, e.g. 2020 (please note that YYYY is deprecated and is interpreted as yyyy)
MM - month, e.g. 03
dd - day, e.g. 01
HH - hour, e.g. 24

Consequences:
With these parameters, files recorded within 1 hour will not differ in name.
Adding the partition number and offset to the file name in the template can solve this problem, but it makes working with the root prefix more difficult.
Uniqueness can be ensured by adding minutes, seconds, milliseconds to the timestamp variable.
Details:
Looks like it's enough to extend the following functionality :
    private static final Map<String, DateTimeFormatter> TIMESTAMP_FORMATTERS =
            Map.of(
                    "yyyy", DateTimeFormatter.ofPattern("yyyy"),
                    "MM", DateTimeFormatter.ofPattern("MM"),
                    "dd", DateTimeFormatter.ofPattern("dd"),
                    "HH", DateTimeFormatter.ofPattern("HH")
            );

with next parameters:
"%M" - Minutes in two-digit format.
"%S" - Seconds in two-digit format.
"%f" - Microseconds.

@BulgakovKD thanks for reporting this.

Uniqueness can be ensured by adding minutes, seconds, milliseconds to the timestamp variable.

Not sure this is the case. At least it won't be guaranteed as there's still a chance for messages from different partitions to be in the same file (if I'm understanding your case correctly).
By reducing the time unit we are just hoping that within a shorter time window there's only messages from one partition.
I guess adding minutes is a valid request -- as some users may want to have messages rotated more frequently -- but I'm not sure going down to microseconds is the right way to proceed. Even more that this will add pressure to the connector task as it will lead to a large number of keys to keep in memory before all messages are flushed to S3.
Do you require ordering only between files? or also ordering within the messages in the file?
You may be solving the first ordering with your proposed approach, but I don't think the second ordering is guaranteed as it's mixing messages from different partitions in the same file.
Not sure yet how to handle this, but let's see if we can find a workaround with the existing configurations before considering changing the connector.


[Bug]In the survey form component, the stars are being selected from right to left, also another bug is that it accepts 0-star ratings

Description
In the components-> forms-> survey form, when user tries to submit (say) a 2-star rating, it is selecting 2 stars from right to left instead of left to right as shown.

Also, a zero star rating is being accepted, which should not be allowed as shown:

I would like to correct this functionality , please assign me this issue under GSSOC 2024.
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100 Please assign me this issue

Proceed @NidhiTornekar


[Snyk] Security upgrade alpine from 3.14.0 to 3

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

Dockerfile

We recommend upgrading to alpine:3, as this image has only 1 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Out-of-bounds Read SNYK-ALPINE314-APKTOOLS-1533752
  714

Buffer Overflow SNYK-ALPINE314-OPENSSL-1569445
  714

Buffer Overflow SNYK-ALPINE314-OPENSSL-1569445
  714

Out-of-bounds Write SNYK-ALPINE314-ZLIB-2976174
  714

Use After Free SNYK-ALPINE314-BUSYBOX-1920733
  614

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Use After Free

 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.


Generate library and asset routes and populate with unstyled content

When data is coming in from the applied schemas, stub out library and asset pages so we have placeholder pages ready for the next prototype, and so the catalog items have a click destination.

https://docs.github.com/en/rest/reference/search#considerations-for-code-search

Only the default branch is considered. In most cases, this will be the master branch.

Wanting to search for files in this branch, which may not be possible.
https://github.com/mattrosno/carbon/tree/feat/schema-basics


The XML response has changed

Waiting on confirm from @mzahed to push this live.

Lets push this!