Sidebar fixes

Ok, this PR's ready for review.
Basically, it does two things:

It offers a partial fix to @steveklabnik's #858 by not showing a horizontal scrollbar, and only showing the vertical scrollbar when there's actually something to scroll. It also adds a bit more space between the sidebar and the spec text. The vertical scrollbar, when present, remains pretty obtrusive on non-OSX platforms, but I don't think there's much we can do about that; it's just those platforms' scrollbar. All these changes are the first commit.

The second commit has the whole list slide up as the footer comes on screen, to prevent the sidebar from overlapping with the footer, which has always been a problem. I also used this as an opportunity to refactor the JS a bit, so reviewing the new source will probably be easier than trying to follow the diff.

As @wvteijlingen pointed out in another issues, making the sidebar a scrollable column on small screens doesn't work reliably in Safari. That is, in Safari, when the user's cursor is over the sidebar and they scroll, about half the time it's the main section of the document that scrolls, rather than the sidebar scrolling. I have no idea why this is, and I tried a couple things to fix it, but neither ultimately worked. These were: using z-index to try to move the sidebar forward, to hint to Safari that it's what the user is focusing on, and making the links in the sidebar display: block, again to try to increase the odds that Safari thinks the user is interacting with the sidebar when their cursor is over the sidebar.
The alternative to this Safari unreliability (if we can't find a fix) is to make the sidebar position: static when it can't fit, rather than making it a fixed-but-scrollable region. I'm not sure if that's worth it, since it means that the nav items will be offscreen for users on small viewports once they've scrolled down a bit into the spec text (which defeats the point a bit). But it's an option, and Ward's PR in #785 gets us most of the way there (I just don't think it accounts for the footer yet in computing whether there's space for the sidebar to fit with position: fixed).

Updated this to also narrow the sidebar. See commit message for the details.

Definitely an improvement. Thanks @ethanresnick :shipit:


Added an argument for protobuf version in Dockerfile

Added an argument to freeze protobuf version in the Dockerfile because with default installation we face an error as mentioned here
Version freeze for base docker image to avoid any error with future python release

Issue  has been solved, so PR is no longer needed.


Update blog-music-identification.qmd

added two more existing categories

Thanks, @alanngnet ! We've been reserving the data tag for data files you can download for model training, transfer learning, etc. Does your website offer that ability? If not, I thought "Audio search" could better capture this one, but let me know if the data tag is more appropriate.

I do host downloadable CSI benchmarking test datasets at https://www.irishtune.info/public/MLdata.htm , although it hadn't occurred to me to mention that in my blog post. Should I call that out, and then add the audio-data category?

Oh, great! That sounds like a good plan. I'll wait until the extra commit is added before I merge this one.


How to build instructions reference incorrect version of Albacore

The how to build instructions (/src/how_to_build.txt) recommend installing the latest version of Albacore, but Rake fails when using the latest version with the error message NameError: uninitialized constant Albacore::Task. The Travis build script forces version 1.0.0.rc2. The build instructions should be updated to reference the correct version.

PR 2045 resolves this issue.


Add deploy to Netlify section to docs

We don't have instructions of how to deploy to Netlify in our docs.
I realized that we use plugins as default set up for deployment.
I tried the hexo-deployer-netlify plugin but it doesn't quite work out of the box.
I'm proposing adding a few lines to provide some instructions about deploying sites on Netlify for now so that it's visible to everyone.

Maybe also add method 3: push to github, gitlab, or whatever is supported and netlify should pick up the change automatically. Or web hooks? Maybe you can find out that method or others and that would help.

It grew! 😯
Are the images referencing the wrong path?

[ ] fix img paths

It was not correct yes, my bad! Should be fixed now

Thanks for entertaining my nitpick. :neckbeard:

Thank you @lunaceee for your contribution!


Banner & readme enhancements

I would like to add a banner in the readme for doc2pen.
This will include a personalised gif / poster.

Sure! Assigned for 2 days

@smaranjitghose I have made a PR, do check and let me know, also don't forget to add the level tag to this issue

Hey, do add the level tag for this issue, I am not able to fetch points otherwise. @smaranjitghose


Fix typo

Hey @orta 👋 really cool notes here, thanks for this!
Just a quick PR for this typo I caught

Thanks!


Fixed the position of the quotation marks

Fixes:  #1717
Description
Fixed the position of the quotation marks to make them fit inside the box

Type of PR

[x ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
before:

after:

Checklist:

[ x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ x] My changes generate no new warnings.
[x ] I have tested the changes thoroughly before submitting this pull request.
[x ] I have provided relevant issue numbers and screenshots after making the changes.

Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!

okay I will look into it and resolves the conflict

okay I will look into it and resolves the conflict


case #1334 - case #1336

case #1334 - false negative with namespacing/qualifications using jsx-pascal
case #1336 - false postive with names starting with a number using jsx-pascal
[Can besaid it is fixed, but inside it is not accepted anyway.  Seems just fixed the regex]

#1336 is not an issue; identifiers can not start with numbers.

@mfyuce any interest in completing this PR?


Problem on install betfair.py

Hi.
I'm trying to install betfair.py but I've got this problem :
Microsoft Windows [Versione 6.1.7601]
Copyright (c) 2009 Microsoft Corporation. Tutti i diritti riservati.

C:\Users\MyUser>pip install betfair.py
Collecting betfair.py
  Using cached betfair.py-0.1.4.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "c:\users\MyUser\appdata\local\temp\pip-build-c0m5pp\betfair.py\set
up.py", line 79, in <module>
        license=read('LICENSE'),
      File "c:\users\MyUser\appdata\local\temp\pip-build-c0m5pp\betfair.py\set
up.py", line 61, in read
        with open(fname) as fp:
    IOError: [Errno 2] No such file or directory: 'LICENSE'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in c:\users\MyUser
\appdata\local\temp\pip-build-c0m5pp\betfair.py

C:\Users\MyUser>

Can someone help me?
Thanks!

Try uninstalling and reinstalling betfair.py. A bug fix for the issue you're describing was pushed to PyPI yesterday in release 0.2.0.

All ok now.
Thanks!
PS : I'd like to create certificate and I think I must call 'invoke ssl'.
How can I invoke this command?
Sorry I'm a real python beginner ;-)


Use of checkers in docker_cli_pull_test.go

Signed-off-by: James Carey<EMAIL_ADDRESS>

Thanks!
ping @vdemeester for review :)

LGTM 🐰

LGTM


Paste Option in right click menu [Feature Request]

Hi,
I am trying to make StyleBot work in Kiwi browser. The main problem I encounter is to get the css into the editor.
I have all the css coded already and just want to paste it into a specific style but there is no option for that
and Android does not have a CTRL-V option either.
Maybe I am missing something? On regular windows StyleBot is awesome!!
Best regards,
Wouter

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context or screenshots about the feature request here.

To expand on this.. I tried to Import JSON content exported from the windows version. But that does not do anything. I also tried to activate clipboard as a key, but inside the style editor that key desactivated (greyed out).


[Snyk] Security upgrade ubuntu from trusty to trusty-20190515

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

vendor/golang.org/x/net/http2/Dockerfile

We recommend upgrading to ubuntu:trusty-20190515, as this image has only 292 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

614
NULL Pointer Dereference SNYK-UBUNTU1404-OPENSSL-1049144
No Known Exploit

614
Access of Resource Using Incompatible Type ('Type Confusion') SNYK-UBUNTU1404-OPENSSL-3314800
No Known Exploit

829
Off-by-one Error SNYK-UBUNTU1404-SUDO-1065770
Mature

686
Improper Privilege Management SNYK-UBUNTU1404-SUDO-3234995
Mature

686
Improper Handling of Exceptional Conditions SNYK-UBUNTU1404-SUDO-473059
Mature

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
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Improper Privilege Management
🦉 NULL Pointer Dereference

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 85.195%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2670

Relevant Lines:
3134

💛  - Coveralls


Misc improvements around build

This PR consists of three independent commits.

Add compile-riak-test to dependency of copile target c2f8057
Skip deps in clean-riak-test 03628e9
Add --all option to "git add" to suppress deprecation warning 01c7b81

Example of warning message from git add without --all is like this:
warning: You ran 'git add' with neither '-A (--all)' or '--ignore-removal',
whose behaviour will change in Git 2.0 with respect to paths you removed.
Paths like 'current/dev/dev1/data/generated.configs/app.2<IP_ADDRESS>.21.35.config' that are
removed from your working tree are ignored with this version of Git.

* 'git add --ignore-removal <pathspec>', which is the current default,
  ignores paths you removed from your working tree.

* 'git add --all <pathspec>' will let you also record the removals.

Run 'git status' to check the paths you removed from your working tree.

@borshop merge


Custom tags support

It would be better if we can apply custom tags to the gist, not just the language tags.

checkout the 1.0.0-beta.3: https://github.com/hackjutsu/Lepton/commit/303554f53177ab44a15bedced25949c09309beea


Add the routing for the "Landing Page" aswell as set a custom background

https://github.com/sopra-fs23-group-20/client/commit/85b0e6c66ccb2f77223be048a41e00a2a8c36527


Bump to Go 1.18

Signed-off-by: ShutingZhao<EMAIL_ADDRESS>Related Issue(s)
Closes https://github.com/kyverno/policies/issues/387.

Description

This PR bumps to Go 1.18.
Checklist

[x] I have read the policy contribution guidelines.
[] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.

Already fixed in https://github.com/kyverno/policies/pull/388

I don't see it on the release-1.8 branch.

Shoot, sorry, I didn't see this was for a release branch.


Update Terms intro

Update as provided by Frank Tees

@palemieux Jerry doesn't have write access to this repo? The above is saying it needs review required from someone with write access, can you update him?


prohibit fluent logger to chocke on its own log messages in case of failure

if the fluent logger is used as a logger for slf4j and it fails to send messages it will choke on its own failure messages (cannot send message) once the connection to fluentd is broken
->this change breaks this circle by only logging the failure to send messages max once per minute

@danischroeter https://github.com/fluent/fluent-logger-java/commit/afc9e84e0210cd106124a4904c6a2eb140d036b3 mitigates the issue?

@komamitsu sorry was off...
nope the referenced commit only reduces the problem to the case when reconnect is enabled. For reconnect the problem stays the same.
If you agree I resolve the merge conflict so you could merge.

I see. Can I ask you to resolve the merge conflict and address this comment https://github.com/fluent/fluent-logger-java/pull/59/files#r80361139 ? Thanks


Always On Top Memory

Description of the new feature / enhancement
Always On Top is a very useful utility, so thank you for creating it!
Have AOT remember which windows had AOT applied and reapply that status automatically when those windows open (this should be an optional setting, per application).
Scenario when this would be used?
While the quick keyboard shortcut is nice, it would be even better if there was an option to have the utility remember which window (including Edge/Chrome/etc. web apps) was set to be Always On Top each time said window opens and have the AOT applied automatically.
Supporting information
No response

The similar-issues-ai bot properly found similar issues and I'll close this as a duplicate and go add my two cents to the original #51295.

Hopefully, someone can revise the resolution so that it shows Resolution-Duplicate (I didn't have rights to do so myself).


fix(deps): update dependency @dcos/ui-kit to v3.64.1

This PR contains the following updates:

Package
Type
Update
Change

@dcos/ui-kit
dependencies
minor
3.63.0 -> 3.64.1

Release Notes

dcos-labs/ui-kit
v3.64.1
Compare Source
Bug Fixes

sets the 'type=button' on buttons by default (827b47d)
infobox: wraps actions to a new line without media queries (dacdbb7)
wraps words in empty states to deal with long URL strings (6d45fe4)

v3.64.0
Compare Source
Features

dropdownmenu: adds a maxwidth prop (28952a0)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Enabled.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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


Trying to focus a text field with dropdown menu opened requires one to click twice

Describe the bug
Is the behavior intentional or a bug? When I click on the dropdown menu and then try to type in the text field with the menu open, it doesn't automatically focus on the text field. Instead, I have to click on it again for it to become active.
Reproduction
Open the input docs then click on the theme changer button at the top as if you want to change the theme, with that menu open try to focuse the input. It will require you to click twice which is not very user friendly.
Logs
No response
System Info
Windows 11

Severity
annoyance

I think the issue is that the user can still interact with the input while the select is open, whereas in the original shadcn you cannot. This is likely an issue to be solved at the bits level so will transfer there.


not updated for unstable release

My package.json has "husky": "^0.15.0-rc.8" in deps and the latest version is 0.15.0.rc.12, but it is not updated by renovate.
Reading the docs,

Renovate won’t update any packages to unstable versions (e.g. 4.0.0-rc3) unless the package version was already unstable (e.g. it was already on 4.0.0-rc2).
https://renovateapp.com/docs/configuration-reference/configuration-options#ignoreunstable

I expect that husky is updated, but actually not.
npm show husky shows
'dist-tags': {latest: '0.14.3', next: '0.15.0.rc.12' },

I guess renovate uses only latest tag, not the other tags like next.

Your package.json is not updated because 0.15.0.rc.12 satisfies your range ^0.15.0-rc.8 and hence there is no need to update.
The reason to use ranges in package.json files is if you want to support a range of multiple versions and not be overly restrictive. If it's your goal to always have the latest version then you should be pinning your dependencies instead. As husky is typically a devDependency you should be pinning them if your expectation is to restrict them to the latest version anyway.

I guess renovate uses only latest tag, not the other tags like next.

No, that's not the case. next is a non-standard tag anyway so has no effect. You could have a tag called previous or really-old-version-from-2015 and they would be ignored too.

You might also be interested in the option upgradeInRange but I think you might end up confusing yourself or other users if you (a) look like you support ranges, but (b) always end up with a range containing 1 dependency anyway. Better to pin and make it clear to everyone about which version is used/installed.

@rarkins upgradeInRange is enabled and Renovate sends PRs to update other deps/devDeps except for husky.
I agree that pinning version is reasonable for devDeps (yes, husky is in devDeps), but it's not suitable for deps.
I want to know the spec or behavior of Renovate for the future.
My following understand is correct?

If upgradeInRange is enabled and the version is specified with range,

if the version specifier is stable, the PR to update package.json and lock file is sent.
if the version specifier is unstable (like 1.0.0-rc.1), the PR is NOT sent.

Hi @teppeis thanks for the additional details. It is not intentional/deliberate that the new husky update should be missed out if indeed you have enabled upgradeInRange, but maybe that feature is not working nicely with the stable/unstable logic. Could you raise an bug report in the main Renovate repository so that I can put it in my queue to look into and fix if necessary?


chore: enclose lint:sol glob expression

Need quotes around the glob expression to ensure it isn't evaluated too early. On my machine not all files are found when quotes are emitted.

Thanks


Need to Fix Shebang in Health Architectures Zip File

I opened an issue on the Health Architectures repo which causes an issue for Mac users.
https://github.com/microsoft/health-architectures/pull/22
The shebang lines need to be changed. I'm not sure how this zip is created, so can the person who owns that process make that change? 😄

We could also update the lines that use bash 4+ parameter expansion for lowercase to use something POSIX compliant (vs. having mac users upgrade their system bash).
For example, this will require someone on OSX to install a secondary bash next to their system shell:
defdeployprefix=${defdeployprefix,,}

This will work regardless of 3.x or 4.x:
defdeployprefix=`echo $defdeployprefix | tr '[:upper:]' '[:lower:]'`

tr is POSIX, so it should be available on all POSIX-compliant *nix installs (and WSL).


Initial draft of EPS32 PWM HAL

An simple implementation of PWM HAL for ESP32 with 4 available channel/timer slots. Please review.

修改已提交


Current master opens preview window after saving commit

environment

vim: neovim 0.1.6
vim-airline: since https://github.com/vim-airline/vim-airline/commit/3899f848a751b2f8e6d45fa1fe3f723491999f6f
OS: Debian GNU/Linux testing
Have you reproduced with a minimal vimrc: yes
What is your airline configuration: does happen without specific configuration
if you are using terminal:
terminal: urxvt
$TERM variable: tmux-256color
color configuration (:set t_Co?): 256
if you are using Neovim:
does it happen in Vim: yes

actual behavior
After saving and closing (with :wq) the gitcommit window opened by :Gcommit (fugitive), a preview window with .git/index is shown.
Minimal vimrc:
call plug#begin('~/.vim/bundle')

Plug 'tpope/vim-fugitive'
Plug 'vim-airline/vim-airline'

call plug#end()

expected behavior
.git/index is not opened in a preview window.

please open at vim-fugitive repo instead.

Is there any reason you think the bug is in fugitive? The bug is definitely triggered by https://github.com/vim-airline/vim-airline/commit/3899f848a751b2f8e6d45fa1fe3f723491999f6f.

Well, how would that open a preview window?

I was having the same issues, so what I did was uncomment out all my plugins except for vundle and vim-fugitive, and it was working fine, and so one by one I reinstalled my plugins and tried testing this issue by making test commits and fugitive was working fine, but when I reinstalled airline, this issue came back. Thus I have to agree with @paretje that this may be caused by airline. I hope this can be resolved though because I love using this package, besides that keep up the great work!

Am 2016-11-17 04:00, schrieb Christian Valdez:

I was having the same issues, so what I did was uncomment out all my
plugins except for vundle and vim-fugitive, and it was working fine,
and so one by one I reinstalled my plugins and tried testing this
issue by making test commits and fugitive was working fine, but when I
reinstalled airline, this issue came back. Thus I have to agree with
@paretje [1] that this may be caused by airline. I hope this can be
resolved though because I love using this package, besides that keep
up the great work!

It would help, if someone could actually debug this issue to find out,
why the
preview window is opened.
cc @gregorias as he is the commit author, perhaps he has an idea.

Thanks for cc, I'll look into this in the weekend.
On Thu, Nov 17, 2016, 09:16 Christian Brabandt<EMAIL_ADDRESS>wrote:

Am 2016-11-17 04:00, schrieb Christian Valdez:

I was having the same issues, so what I did was uncomment out all my
plugins except for vundle and vim-fugitive, and it was working fine,
and so one by one I reinstalled my plugins and tried testing this
issue by making test commits and fugitive was working fine, but when I
reinstalled airline, this issue came back. Thus I have to agree with
@paretje [1] that this may be caused by airline. I hope this can be
resolved though because I love using this package, besides that keep
up the great work!

It would help, if someone could actually debug this issue to find out,
why the
preview window is opened.
cc @gregorias as he is the commit author, perhaps he has an idea.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vim-airline/vim-airline/issues/1317#issuecomment-261182764,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAsF0bQ3blaPo6VM2OHgQbibJD8tWkSwks5q_A1OgaJpZM4KvqRT
.


connecting/disconnecting

Hi, I added the addon to kodi and it says  its connected and few seconds later it says disconnected. What did i do wrong?
                                                                                Steve

This is caused by incorrect credentials in the addon settings

Like if put the secret key in wrong?
On Tue, 7/10/18, Tomas Kislan<EMAIL_ADDRESS>wrote:
Subject: Re: [kodi-connect/kodi-connect-addon] connecting/disconnecting (#3)
To: "kodi-connect/kodi-connect-addon"<EMAIL_ADDRESS>Cc: "steve55792"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Date: Tuesday, July 10, 2018, 2:05 AM
This is caused by incorrect
credentials in the addon settings
—
You are receiving this because you authored the thread.
Reply to this email directly, view
it on GitHub, or mute
the thread.

Yes exactly


Allow to specify the max websockets frame size when using the upgrader.

Motivation:
At the moment its not possible to adjust the max websockets frame size when using the upgrader while its possible when directly use the WebSocketFrameDecoder.
Modifications:
Allow to specify max frame size via an init argument (with default value).
Result:
More flexible way to use the upgrader.

also @vlm

@Lukasa done


HTTP link previews do not show up in Desktop App

On desktop app v1.1.0, OS X.
Repro Steps: Post an image link starting with http://
Observed: On browsers, a preview of the image will automatically be shown below the link. In the desktop app, no image shows up.

Duplicate of #35


Bug 2101335: Clone action enabled for CrashLoopBackOff

📝 Description

Add a brief description

🎥 Demo

Please add a video or an image of the behavior/changes

@avivtur @upalatucci @hstastna
please note all the possible vm statuses here
I fixed isFailedPrintableStatus and clone is now only enabled if the vm is paused to stopped

/lgtm


Updating PodDisruptionBudgets not supported

When attempting to update a PDB tanka returns an error from the k8s api:
The PodDisruptionBudget "..." is invalid: spec: Forbidden: updates to poddisruptionbudget spec are forbidden.

Should tanka attempt to handle this by deleting and recreating the PDB?  or is this behavior expected and it should be handled manually by the operator?

/cc @malcolmholmes @woodsaj

This same problem exists for statefulsets, where we need to issue a kubectl delete --cascade=false sts <stsName> before doing a tk apply.
Deleting a resource before re-creating sounds dangerous, if tanka did support it, we would definitely want it to be gated behind a cmdline flag, or prompt for confirmation.

I think we might be able to catch the The PodDisruptionBudget ... error and delete the pdb, but ask for confirmation first, the same way as we do when issuing tk apply

@sh0rez do we want this or should we just let it close?


AWS Settings?

Can you throw your basic settings for AWS Cognito in the readme or somewhere?
I had a heck of a time getting things set up to work on the AWS side. Could be helpful to others.

I'm struggling to get set-up in AWS, any help would be greatly appreciated. Thank you!


Encrypting Previously entered data

Thanks for making this. Worked like a charm, but I noticed an unexpected issue when i started using the app.
I had a previously unencrypted CharField of first_name and last_name'
I installed django-encrypted field and updated my model using the EncryptedCharField. When I enter new data, everything appears as I expected: The view shows me the unencrypted first_name and last_name while the database shows the fields encrypted.
However, with the data that was already entered, the view shows the encrypted fields as they appear on the database - basically gibberish. Is there anything that can be done about this? I'm working with a local version, but want to update the deployed version with the encrypted fields, but I'm a bit concerned that I would freak out users because of the above issue.

The encrypted fields assume that the data in the db is encrypted. So if it pulls some plaintext out of the database, it will be treated as cyphertext, which will turn it into gidderish.
I would recommend a data-migration strategy. There would be three migrations needed.

Add a new field for the encrypted data with a different name
Write a data migration that transfers the data from plaintext field to the new encrypted field
Delete the plaintext field and rename the encrypted field to original name

After running those, you should have all your old data safely encrypted in the new field.


[ko] Wrong Note box scoping in Korean "Volumes" page

This is a Bug Report

Problem:
In the 'RBD CSI migration' section of English "Volumes" page, the Note box ends right before the 'secret' section starts:

But the Note box in Korean "Volumes" page does not end where it should, spanning improperly.

Proposed Solution:
Put {{< /note >}} right before the heading ### secret.
Page to Update:
https://kubernetes.io/ko/docs/concepts/storage/volumes/#rbd-csi-migration

/language ko
/assign

/triage accepted

In upstream website: Resolved by #33566 and #34817.
In v1.23 website: Will be resolved by a future PR that merges dev-1.23-ko.3 into release-1.23.
/remove-lifecycle stale

/close
Since dev-1.23-ko.3 into release-1.23. is complete


Exp voting

Motivation
This PR adds the optional HARE weighted consensus
Closes #
Changes
It changes handling HARE messages to use node weights instead of messages count
Test Plan
TODO

No, it can be closed and deleted.


Unable to checkout '70821217ca0db0280af537002839dbb340f77d68' in submodule path '_notebooks'

Hi! I'm following the first steps of the tutorial, running: pip install -r requirements.txt
And I get this error:
Collecting lightning@ git+https://github.com/Lightning-AI/lightning@master (from -r requirements.txt (line 2))
  Cloning https://github.com/Lightning-AI/lightning (to revision master) to /private/var/folders/3w/bsb667zn42gdhbqq3z8ptmgw0000gn/T/pip-install-rdi4y7c2/lightning_01d8224b706a4bfdbddd328a10f43098
  Running command git clone --filter=blob:none --quiet https://github.com/Lightning-AI/lightning /private/var/folders/3w/bsb667zn42gdhbqq3z8ptmgw0000gn/T/pip-install-rdi4y7c2/lightning_01d8224b706a4bfdbddd328a10f43098
  Resolved https://github.com/Lightning-AI/lightning to commit d4867cba56e63321ad94b7176bef2d654d9b1c9a
  Running command git submodule update --init --recursive -q
  git-lfs filter-process: git-lfs: command not found
  fatal: the remote end hung up unexpectedly
  Unable to checkout '70821217ca0db0280af537002839dbb340f77d68' in submodule path '_notebooks'
  error: subprocess-exited-with-error

Thanks!

The line git-lfs filter-process: git-lfs: command not found is suspicious. Can you try installing it?
https://git-lfs.com/

The line git-lfs filter-process: git-lfs: command not found is suspicious. Can you try installing it?
https://git-lfs.com/

Yep... I should have read better the error log. Sorry and thanks for the help!


NoneType in vanilla_mask_3d

Hi,
I would like to create the saliency map for a DNN that I use for classification of 4 types of images. I manage to calculate the gradient_saliency but when I try to calculate any of the masks e.g vanilla_mask_3d I get the following error:
vanilla_mask_3d = gradient_saliency.GetMask(im, feed_dict = {neuron_selector: prediction_class}) Traceback (most recent call last):
File "<ipython-input-19-7508dfaae246>", line 1, in <module>     vanilla_mask_3d = gradient_saliency.GetMask(im, feed_dict = {neuron_selector: prediction_class})
File "/usr/local/lib/python2.7/dist-packages/saliency/base.py", line 97, in GetMask     return self.session.run(self.gradients_node, feed_dict=feed_dict)[0]
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 889, in run     run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1105, in _run     self._graph, fetches, feed_dict_tensor, feed_handles=feed_handles)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 414, in __init__     self._fetch_mapper = _FetchMapper.for_fetch(fetches)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 231, in for_fetch     (fetch, type(fetch)))
TypeError: Fetch argument None has invalid type <type NoneType'>
Could you please tell me where might this error be coming from since I don't seem to have a NoneType argument in my prediction_class.
Cheers,
Pouyan

I managed to solve it in case anyone had the same problem. The error was coming from tf.gradients() function which was returning "None" due to mishandling the tensors which were built upon in the graph.

Could you please elaborate on how you solved the problem? I've spent hours trying to find a solution.

The problem comes from the fact that the correct tensor is not being passed to the  tf.gradients()  function. In my case I was not passing the tensor with the softmax layer which calculate the prediction to the function:
gradient_saliency = saliency.GradientSaliency(graph, sess, y, images)).
Therefore the gradient ended up being None.

The problem comes from the fact that the correct tensor is not being passed to the tf.gradients() function. In my case I was not passing the tensor with the softmax layer which calculate the prediction to the function:
gradient_saliency = saliency.GradientSaliency(graph, sess, y, images)).
Therefore the gradient ended up being None.

Hello, have you solved this problem? I met the same problem but have no idea what's wrong. the y? I noticed that tensor y has no shape


Add ability to remove blank space between results from multiple SELECT statements when the result is only 1 row

Is your feature request related to a problem? Please describe.
When pulling a single row from multiple tables Azure creates a large blank space between each result row and the next. This isn't an issue when there are multiple rows in the results of a single SELECT statement, as then the rows will continue until they hit the next result set.
Describe the solution or feature you'd like
Create a more flexible result grid that adjusts to the size of less result rows.
Describe alternatives you've considered
While UNION can bring data from the two tables together into one result set, it comes with restrictions - such as losing column headers and only allowing the same amount of columns in each select statement. This is not a good solution when looking at an object that has data across many tables. JOINs are not a good solution either, as they put all data into a single row, making data difficult to view and compare as it stretches on horizontally. Pulling only certain columns from the different tables only works when you're interested in a handful of columns.
Additional context
Here's an example from a project I'm working on:

As you can see, there's a bunch of blank space between the select results.
It's possible a setting to adjust this already exists, but I was unable to find it within the settings, nor through different searches online. I recently switched from SQL server management studio where it automatically adjusts like so:

The blank space is there to make room for the actions on the right side of the grid. We will need some ux review on this to understand what we can do in this area.

I would like to see this addressed. It's frustrating for users coming from SSMS where multiple result sets are handled well. Needing to see a single row from several different tables is a common scenario.

Bump. Super annoying "feature".


Bottts - robot illustration generator

Just wanted to share - not sure if feasible…
https://bottts.com/
Create robot illustrations in Sketch App with this free library. Combine frames, antennas, sensors, accessories, and colors. 🤖
/cc @pablostanley

Bottts fits perfectly! Will be officially supported in the coming weeks.

Will be published this Thursday. :tada:

Already available today!

Awesome @FlorianKoerner – Thank you, beautiful done 🙇♂


dapr-component glitch in UX and inconsistency between docs & CLI

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[x] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Managing to successfully create a dapr-component with missing properties breaks the portal UX
Steps to reproduct
I managed to create a component via CLI with the following schema:
name: commonpubsub
componentType: pubsub.azure.servicebus
version: v1
metadata:
- name: connectionString
  secretRef: pubsub-servicebus-connectionstring
secrets:
- name: pubsub-servicebus-connectionstring
  value: Endpoint=sb://da...........

Actual behavior [What actually happened.]
If I retrieve the dapr-component via CLI az containerapp env dapr-component show I get
{
  "id": "/subscriptions/xyz/resourceGroups/dapracargdb/providers/Microsoft.App/managedEnvironments/xyz/daprComponents/commonpubsub3",
  "name": "commonpubsub2",
  "properties": {
    "componentType": "pubsub.azure.servicebus",
    "ignoreErrors": false,
    "metadata": [
      {
        "name": "connectionString",
        "secretRef": "pubsub-servicebus-connectionstring"
      }
    ],
    "secrets": [
      {
        "name": "pubsub-servicebus-connectionstring"
      }
    ],
    "version": "v1"
  },
  "resourceGroup": "dapracargdb",
  "systemData": {
    "createdAt": "2022-06-07T17:31:21.6881338",
    "createdBy": "xyz",
    "createdByType": "User",
    "lastModifiedAt": "2022-06-07T17:48:50.0800949Z",
    "lastModifiedBy": "xyz",
    "lastModifiedByType": "User"
  },
  "type": "Microsoft.App/managedEnvironments/daprComponents"
}

In the output above you notice there is no scopes section or value in metadata, as I did not add it to the input.
If I open the Dapr component from portal UX I get the following error:

Expected behavior [What you expected to happen.]
Only if I create the dapr-component from UX, or CLI with the empty scopes and value in metadata, I get the UX to comply
{
  "id": "/subscriptions/xyz/resourceGroups/dapracargdb/providers/Microsoft.App/managedEnvironments/xyz/daprComponents/commonpubsub",
  "name": "commonpubsub",
  "properties": {
    "componentType": "pubsub.azure.servicebus",
    "ignoreErrors": false,
    "metadata": [
      {
        "name": "connectionstring",
        "secretRef": "pubsub-servicebus-connectionstring",
        "value": ""
      }
    ],
    "scopes": [],
    "secrets": [
      {
        "name": "pubsub-servicebus-connectionstring"
      }
    ],
    "version": "v1"
  },
  "resourceGroup": "dapracargdb",
  "systemData": {
    "createdAt": "2022-06-07T17:14:41.3484488",
    "createdBy": "xyz",
    "createdByType": "User",
    "lastModifiedAt": "2022-06-07T17:14:41.3484488",
    "lastModifiedBy": "xyz",
    "lastModifiedByType": "User"
  },
  "type": "Microsoft.App/managedEnvironments/daprComponents"
}

From the UX, I can see the dapr-component created via CLI

Screenshots
In addition to the issue above, following the schema at https://docs.microsoft.com/en-us/azure/container-apps/dapr-overview?tabs=yaml fails with Invalid YAML provided. error.

The schema that seems to work is at https://docs.microsoft.com/en-us/azure/container-apps/microservices-dapr?tabs=powershell.

Additional context
Using CLI with extension 0.3.4

I also saw the YAML syntax issue for the servicebus sample - when adapting it to the below format for blobstorage, I got it to work - I had opened a separate bug for that already in the MicrosoftDocs repo.


webgui: match cipher suites and openssl config commands

Hi!
ref. https://forum.opnsense.org/index.php?topic=29284
ref. https://forum.opnsense.org/index.php?topic=30048.0
It turned out that the administrator can break access to the gui by selecting  TLSv1.3 ciphersuites only.
OpenSSL uses different config file options and commands to define ciphersuites for TLS1.2 and below (CipherString) and for TLS1.3 suites (Ciphersuites):
https://www.openssl.org/docs/manmaster/man3/SSL_CONF_cmd.html#SUPPORTED-CONFIGURATION-FILE-COMMANDS
PR tries to follow this logic and use matching command for defined ciphers.
RFC8446 Compliance Requirements (https://www.rfc-editor.org/rfc/rfc8446#section-9.1) is also fulfilled when using TLS1.3 suites: the presence of the TLS_AES_128_GCM_SHA256 is controlled.
Thanks!
UPD. RFC8446 Compliance Requirements check implemented in https://github.com/opnsense/core/commit/d31bff3c3975fe8c043538bcccbe30674068eae8 and removed from pr

Assigning @AdSchellevis since he integrated this and likely has an opinion on integration.

sorry, is
https://github.com/kulikov-a/core/commit/41043fce8d4deb0dd0f59a8f2041c925fbd78b66
what we talking about?

@kulikov-a When it comes to ensuring the correct contents, I would suggest we move that to the "model" (knowing we don't have one here). As the model is responsible for proper data. This https://github.com/opnsense/core/commit/d31bff3c3975fe8c043538bcccbe30674068eae8 would improve the validation. This just leaves us with writing the correct config.

@AdSchellevis true! removed this check

updated (no static array) and made shorter

@kulikov-a looks good, thanks! it would have been nice if we could easily determine if a cipher belongs to 1.3 so we shouldn't have to ask openssl, unfortunately that doesn't appear to be the case (unless injecting the list statically).

@AdSchellevis  Thanks!! (agree, openssl is rather obscure in some places)


Enhancing Deploy Pipelines with Swift Integration Tests

Objective:
We aim to introduce thorough integration tests into our services deployment pipelines to prevent breaking releases.
How?
On every swift release we should prepare build for testing artifacts that contains at least one test for each service. Artifacts should be isolated and have minimal dependencies, including service under test. Builds for testing can be prepared based on a dedicated test plan for each artifact. Service under test should run Swift tests against it's staging instance.
Following services should be tested:

[X] rs-relay
[ ] keyserver - ChatTests.testMessage()
[ ] echo-server - PN UITest - blocked with GH M1
[ ] rpc-proxy - ENSResolverTests
[x] cast server - PushTests.testWalletCreatesAndUpdatesSubscription()

closing for now


FEATURE PopUp initial build

Codecov Report

Merging #2 into master will increase coverage by 27%.
The diff coverage is 85.18%.

@@            Coverage Diff             @@
##             master       #2    +/-   ##
==========================================
+ Coverage     41.17%   68.18%   +27%
- Complexity        8       11     +3
==========================================
  Files             4        5     +1
  Lines            17       44    +27
==========================================
+ Hits              7       30    +23
- Misses           10       14     +4

Flag
Coverage Δ
Complexity Δ

#php
68.18% <85.18%> (+27%)
11 <3> (+3)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

src/Model/Violator.php
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/Admin/SiteNotificationsAdmin.php
0% <ø> (ø)
4 <0> (ø)
:arrow_down:

src/Model/PopUp.php
85.18% <85.18%> (ø)
3 <3> (?)

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 edda631...2fa8e7f. Read the comment docs.


Handle streaming errors and return to user proc, with result_type

Currently when streaming, errors are ignored and never returned to the user's code.  This includes non-starter errors (eg wrong API key) as well as errors encountered in the course of usage (eg context length exceeded).  These are discussed  in https://github.com/alexrudall/ruby-openai/issues/256 and https://github.com/alexrudall/ruby-openai/pull/270 .
This PR (the one you are reading) offers an alternative solution:

Errors are returned to the user's code as well as data blocks
Every returned structure has a "result_type" injected into the top level of the JSON, one of:

"data"
"error"
"unknown"

Specs have been updated, but in short:
data: { "some": "json" } chunks are returned as data  result_type
error: { "some": "json" } chunks are returned as error result_type
{"error": {"some": "json"} } chunks are returned as error result_type
{"some": "json"} chunks are returned as unknown result_type
Note that this is carefully crafted to ensure backward compatibility with existing code!  For example, this sample code will work unchanged:
stream: proc do |chunk, _bytesize|
  print chunk.dig("choices", 0, "delta", "content")
end

.... it will just ignore the result_type field and look for chunks matching the expected shape.
Feedback or change requests welcome.
Thanks BTW for an awesome gem, we're super happy with it!
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

Would love to see this merged, I got stuck because I was missing errors for a bit. @alexrudall or others?

For @gregschwartz, or anyone else who wants to use this branch before merge, you can temporarily point your Gemfile to:
gem "ruby-openai", github: "harlantwood/ruby-openai", branch: "streaming-error-handling"

@alexrudall any thoughts on this one?  Happy to make changes if you have requests.

@harlantwood thank you for this PR!

@alexrudall  any thoughts on this one vs #338 ? Happy to update this one and fix tests, but if you plan to merge #338 I won’t bother.

Big thanks for this @harlantwood -- going with #344 as simpler (and outsources some of the work to Shopify via event_stream_parser!) - but thanks a lot for taking the time to contribute this, it all helps find a solution.

Sure thing!  I think #344 is a great solution, if you’re willing to go with a breaking change.
Thanks for creating a great lib — we’ve been using it in production (streaming via hotwire) and it’s been super solid.


Added Language Downloading and Settings

Added basic downloading of languages from our language server and a dynamic settings panel which allows the user to change the language between the downloaded ones.
With the help of the awesome (which wrote parts of the original code, woop woop), @MaxiBoether !

Working on making it react/redux.

Closing in favor of #455.


WDIO update to 8.23.0

PR Details
First iteration with set of changes that allow to execute beacon related e2e tests with WDIO v8
Description
Created lib version 2.0.0 with updated dependencies and latest version of related WDIO v8 packages. Adapted tsconfig to be ESM compatible and adapted imports and other methods for WDIO v8.
Related Issue
#39
Motivation and Context
Keep library up to date by using latest version of dependencies
How Has This Been Tested
This has been tested by using it within the Beacon project. E2E tests execute successfully although it has been detected already some related tasks that will need to be done -> refactor of test-case.reportes.ts. Other changes might be needed.
Types of changes

[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x ] I have read the CONTRIBUTING document
[x ] My code follows the code style of this project
[x ] Increase version in package.json following Semantic Versioning
[x ] Add the issue into the right project with the proper status (In progress)

Created systelab-components-wdio-test v2.0.0 base wtih WDIO v8 - completed


[Question] onEnter, onLeave hooks

I was wondering if onEnter and onLeave hooks are already supported in preact-router. If not, is it something on a roadmap?

Nope! You should use lifecycle events, onEnter and onLeave aren't really necessary.

Thank you. I missed the fact that pages are just components (with URL based conditions).

No problem! Sorry for being abrupt there, was trying to get through 150 Github notifications in 2 hours. haha

No worries. I can't thank you enough for your work on preact :heart: :heart:


Randomly perform garbage collection on database tables

The following three database tables need garbage collection, clearing any rows that have been marked as expired:
users_remembered
users_resets
users_throttling

For most use cases, the best solution may be to trigger the garbage collection for a table randomly, e.g. during related operations, using an implementation like this:
function random_float($min = null, $max = null) {
    $min = isset($min) && \is_numeric($min) ? (float) $min : 0;
    $max = isset($max) && \is_numeric($max) ? (float) $max : 1;

    return $min + \mt_rand(0, \mt_getrandmax()) / \mt_getrandmax() * ($max - $min);
}

$probability = 0.02;

if (\random_float() < $probability) {
    // perform garbage collection ...
}

For the users_confirmations table, garbage collection would be more complex. For each user_id and for each email in that table, the row with the largest id must be preserved so as not to break the re-sending of previous confirmations.

If you considering to implement garbage collection, you should think about converting DB from MyISAM to InnoDB.

@ponasromas Can you explain this? Is there any relation between the storage engine and the garbage collection that we’re planning to implement?
As noted above, we’re planning to clear the unused rows from those database tables via an implementation in the library’s code (i.e. in PHP).
Switching from MyISAM to InnoDB for MySQL is something we always have in mind. But it may be better to do this in a future update when there are other changes to the database schema as well. This issue here can probably be resolved without any such breaking changes.

I have three thoughts on this topic:
1.) The garbage collection might include:

rows in users_confirmations without id in the table users (account cancellation)
rows in users_confirmations where the user is verified and the email is equal to the one in the table users
users that registered but never logged/verified after a grace period
(non-active members some days after an information message)

2.) The garbage collection should be initiated by a scheduled cronjob of the server (not managed within the library). One advantage would be that the timing of the garbage collection can be better managed.
3.) I find there is a connection between garbage collection and the issue regarding InnoDB because with InnoDB it would be possible to use foreign key constraints for a customized table with additional user information. That would mean that updates and deletions could cascade automatically between the table users and the table with additional user information. Data consistency could be improved and the general handling would be easier. I would like to see a change to InnoDB.

One additional thought:
If users that never logged in/are not verified are deleted after a grace period in the users table then the garbage collection of the users_confirmations table simply would be to delete all rows that are expired (taking the above mentioned grace period into account).

@Aristotle22
(1) I see no reason why all this cannot simply be handled by deleting expired entries only. Do you have strict (storage/performance) requirements that make it impossible for you to have obsolete entries in those tables for a few days (until they’re actually recognized as expired based on the date)?
(2) This is why it’s not here yet. That question is not as easy as you think. We have many users who are on shared hosting or on platforms where they can’t easily schedule cron jobs or similar things. But I also see how some people might not want this to be randomly performed on regular requests. So we might need both.
(3) Foreign keys and cascading deletions really are a tiny fraction of the problem we have here. The vast majority of garbage is entries that have expired simply because of their age, not because there has been some deletion in another table. You can check this in your own database tables if you want. Finally, we maintain support for other database systems, too.

@ocram
(1) My suggestions are only additional entries that could be deleted.
(3) The foreign keys could be used for additional user information saved in a separate table independent from the library. You don't have to use foreign keys at all.
https://github.com/delight-im/PHP-Auth/issues/10


Package Project

See this for how to prepare a Python package
Users should have to merely install and import as a library, just as is done with Pygame.

https://pypi.org/project/pyEngine2D-ddd/0.0.0/


CircularLinkedList.java

Added CircularLinkedList data structure.

Hi Prakhar, Thanks for proceeding on the suggestion. By the way, i have given some basic cosmetic comments. And also write a Junit for this data structure in tests folder. You can refer to other data structures test cases. Make sure you test all the corner cases and even if you can introduce anything new in testing, which others have not yet added. Make the changes and the test case, i will then merge the request. Thanks.

Hi Prakhar,
Why dont you submit the josephus problem's code written using CircularLinkedList? Make it a seperate class.


Unsupported vtype when lmul<1

In section 3.3, spec_0.10 says
For a given supported fractional LMUL setting, implementations must support SEW settings between SEWLMUL1MIN and
LMUL * SEWLMUL1MAX, inclusive.
Does it means that:
1.unsupported vtype will set vill
when lmul=1/8,  sew must be set 8,  if not, vill will be set,  is it?
2.supported vtype, but eew of instruction does not belong to [SEWLMUL1MIN,LMUL * SEWLMUL1MAX]
when vlen=128, lmul=1/8,  sew=8, for vle64.v instruction, eew is 64, the load data is wider than maximum width that  vtype can express(128x1/8=16), this instruction will be illegal or not?

thanks
As spec(3.3) says
For a given supported fractional LMUL setting, implementations must support SEW settings between SEWLMUL1MIN and
LMUL * SEWLMUL1MAX, inclusive.
SEW between SEWLMUL1MIN and LMUL * SEWLMUL1MAX, inclusive, must be supported. it means SEW settings are only relevant to SEWLMUL1MIN, LMUL, SEWLMUL1MAX, and not relevant to VLEN, that will be easy for programers.  issue 656 has shown
https://github.com/riscv/riscv-v-spec/issues/656#issuecomment-805357460
then, unsupported settings when lmul<1 will set VILL in vtype or not?


Feature request: Parcelable Advertisement on Android

I wonder if you would consider making the AndroidAdvertisement implement Parcelable?
The Android native ScanResult implements the Parcelable interface, so I think this should be possible, though I don't know if there are internal details of the Advertisement implementation that make it more difficult.
The use case is an android view that shows a list of devices in range, implemented using Jetpack Compose. I would like to be able to seamlessly handle app configuration changes using rememberSaveable.
Yes, I could store the list of advertisements in a global singleton, outside of the view hierarchy, but that feels less in line with Android architectural idioms.

And the same thing would be nice for Peripheral. Android's BluetoothDevice implements Parcelable.

Sounds like a good feature to have. 👍
I'm currently pretty busy on other projects at the moment, but I'll try to find some time to work on this.
PRs are also welcome. 😁

And the same thing would be nice for Peripheral. Android's BluetoothDevice implements Parcelable.

Having Kable's AndroidAdvertisement be Parcelable can be done, but unfortunately: I don't see a feasible way to do the same for AndroidPeripheral. As AnroidPeripheral is stateful (holds onto a connection) which cannot be written to a Parcel. This was done to make a peripheral have a more intuitive API.
Android's BluetoothDevice is Parcelable because it is stateless (is a thin wrapper around a BLE address), and from that stateless object a BluetoothGatt is created (which holds the underlying connection).

@eddieSullivan, #652 makes AndroidAdvertisement Parcelable.
You can test it with:
repositories {
    maven("https://oss.sonatype.org/content/repositories/snapshots")
}

dependencies {
    implementation("com.juul.kable:core:0.29.1-issue-610-1-SNAPSHOT")
}

Shipped in 0.30.0.


Adding custom control overrides

In our deployment we need to add custom control overrides. I'm trying to find a way to add this into superset so that allows us to avoid maintaining cherry picks. I have a method that overrides files using NormalModuleReplacementPlugin during deployment in a webpack config that merges with the open source config (here's a doc that describes the process).
In order to easily override files, I need to make a few changes I've included in this PR. Ideally extraOverrides would have been an object that I could add to the end of controlOverrides, but sometimes a control I'm adding an override for already has an overrride, so the extraOverrides needs to know about existing controlOverrides.
I also have a fix for MetricsControl, isEqual was not being used correctly and doesn't catch in the case that the objects are different lengths. And we were also clearing metrics control too frequently, we only need to clear when the changed metrics list no longer contains the selected metric.
@mistercrunch @kristw thoughts?

Great work! I took a closer look at this and have a few suggestions.
Instead of applying the overrides at individual control panel file (, which will be moved into plugins). Add a single hook at explore/controlPanels/index.js and pass all controlPanelConfigs to it.
Current index.js
const controlPanelConfigs = {
  area: Area,
  bar: Bar,
  big_number: BigNumber,
  ...
}

Modified index.js
import extraOverrides from './extraOverrides';

const controlPanelConfigs = extraOverrides({
  area: Area,
  bar: Bar,
  big_number: BigNumber,
  ...
})

extraOverride.js
export default function extraOverrides(controlPanelConfigs) {
  return controlPanelConfigs;
}

Custom extraOverride.js in Airbnb deployment
export default function extraOverrides(controlPanelConfigs) {
  const newConfigs = {...controlPanelConfigs};
  newConfigs.line = {
    ...newConfigs.line
    controlOverrides: ... // apply override here
  };
  newConfigs.table = {
    ...newConfigs.table
    controlOverrides: ... // apply override here
  };

  return newConfigs;
}

Codecov Report

Merging #6956 into master will increase coverage by <.01%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master    #6956      +/-   ##
==========================================
+ Coverage   64.76%   64.76%   +<.01%
==========================================
  Files         421      422       +1
  Lines       20384    20387       +3
  Branches     2249     2250       +1
==========================================
+ Hits        13202    13204       +2
- Misses       7049     7050       +1
  Partials      133      133

Impacted Files
Coverage Δ

...src/explore/components/controls/MetricsControl.jsx
72.59% <0%> (-0.55%)
:arrow_down:

superset/assets/src/explore/controlPanels/index.js
69.23% <100%> (ø)
:arrow_up:

...assets/src/explore/controlPanels/extraOverrides.js
100% <100%> (ø)

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 3ae02d1...aafb622. Read the comment docs.

thanks for the good idea @kristw that is simpler 👍

I think this is ready to go, any other thoughts @mistercrunch?

LGTM, I'm confused by the old isEqual calls though, I'm not sure if it did make any sense as it was before...

I'm not sure how the old isEqual code worked, maybe it didn't. I noticed because it wasn't catching if columns was a different length.
Sorry for the churn, I noticed an issue with the way I was clearing metrics in the MetricsControl so I fixed that logic.


feat(print): add pdf generation logic

This implements / fixes: https://github.com/zskarte/zskarte-client/issues/259
It is based on changes from https://github.com/zskarte/zskarte-client/pull/400
The QR-Code deeplinks only work together with https://github.com/zskarte/zskarte-client/pull/405
The new print sidebar is shown by clicking on "Print" menu point in menu sidebar or if the user presses Ctrl+P / Meta+P or the Browser Print function is called from menu.
In the last case the browser print view is still shown (cannot be bypassed).
Pressing Esc closes now any sidebar.
The attribution was missed for OSM Tiles, I added them.
Also updated the attribution on other layers to work and match the ones in geoAdmin Layers.
The attribution was not shown on the page, I added the corresponding ol-component, and move it to "center bottom" of the view.
Based on the logic the sidebar goes "over" the map and not change the size of it, it's not optically center if an sidebar is open. This sidebar behavior was also the reason I had to move it to center because it would not be visible otherwise.
I need to add crossOrigin: 'anonymous', to all tile layers/sources because otherwise the canvas was "tainted" and could not be exported anymore.
I added an global css class like-mat-form-field-outline to be used with fieldset to have the same border style for radioGroup and checkboxes as for other input fields.
The option emptyMap and the available paperFormat allow to use the print/pdf function also to create maps for drawing physically on it (up to A0), a feature our ZSO miss on all other online map/mapping tools...
Normal mode is to allow user to select the area to print(click on the map), and based on the other options the corresponding scale is calculated. It is also possible to select an specific scale in the dropdown.

While printing the map interactions are deactivated to prevent interrupting the print view settings.
While printing there is also a progress bar, showing progress of loading needed tiles (0-80%) and do the other pdf generation work.(80-100%)

And "direct print" function as mentioned in issue is not implemented, as this have no added value than use the print on the directly opened pdf.
example pdf:
Druck Test Operation_A5.pdf

Ah darn, sorry @swerder I did not realise until now that your PRs are from a forked repo, since I have not access to your repo I will not be able to rebase this stuff properly and merge your PRs in a timely manner since I might have to wait for your changes every time I merge something into our dev branch.
Would it be possible for you to give me access to your repo? Then I should be able to handle everything by myself and I won't need to bother you next week or so :)

@thegnuu as I'm not part of your group/collaborator I had to fork to do changes.
I send you a colleborate invate to my repos/forks.

@swerder no worries, that's totally fine. Thanks a lot for the access! But since I have to merge a lot of stuff it will be easier if I can pull those changes into your repo and do everything at once, so you don't have to do anything and I will not have to wait for you, I am absolutely sure you have other things to do right now ;)
I will merge all your changes in the upcoming two to three weeks :)

@swerder  and if you have plans to make some changes in the future I can tell the guys to add you to the repo as well...

@thegnuu I used the most of my spare time in the first half of the year for my changes here.
currently I make a break.
but what's still missing, and I plan to do is the "extension" of #429 to also allow "continue" work offline also with page refresh, and allow to push all to the server when have again internet.
also there may be some conceptual problems for longer offline time/switching operations with the immer patches I want to verify/fix.


Revise "Find, Download, and Explore Data" section with updated screenshots and functionality.

taken care of when merging tutorials-update with main (screenshots have been updated in tutorials-update branch)


attr, data improvements

Because I was usable in value other than string in jquery, I did it like that in cash.

Fixes implemented in a different way have been put into place for $.fn.attr and $.fn.prop and should be done for $.fn.data soon.


Support non-diagonal weight matrices

This can be accomplished by premultiplying basis functions by the Cholesky square root of the weight matrix (a generalization of how diagonal weight matrices are currently handled).  See equation (7) in [1].
[1] Green, P. J. (2011). Iteratively Reweighted Least Squares for Maximum Likelihood Estimation, and some Robust and Resistant Alternatives. Journal of The Royal Statistical Society, Series B (Methodological), 46(2), 149–192.

This will be hard.  Also, does anybody want it?  Marking for 0.2.


[meta] Version 2.0.3 not on npmjs.org

Vue.js version: 2.6
consumed using: Webpack
FULL error message (including stack trace):
relevant code:
steps for reproducing the issue:

The steps above are not really relevant. I tried vue-table-2 and noticed with(this) bug #29 and it seems to have been fixed in v2.0.3. However, v2.0.3 is not on npmjs, resulting in vue-tables-2 throwing this error in my project.
What was the reason for unpublishing 2.0.3? It is still listed, but not as latest version

I was trying to fix #42 and got myself in trouble. For some reason, I can't find 2.0.3 on master, Also can't unpublish 2.0.1. Trying to resolve

Okay, I already monkeypatched v2.0.3 into my yarn.lock and that worked 🙈
Thanks for the quick reply

Upgraded package to new version. Also reflected in vue-tables-2 reference


native geolocation deprecated on Android

I struggle to make the geolocation works on Android, using ngCordova and of course the org.apache.cordova.geolocation plugin.
I came across this Jira:
https://issues.apache.org/jira/browse/CB-5977
Is ngCordova aware of this deprecation?
What is the best way today to use geolocation on Android with ngCordova?
Indeed, it seems that, when the plugin is installed, Android always try to use the native way, that..isn't available any more.

Thanks a lot,
Michael

I found a tricky bug:
Having this code:
$cordovaGeolocation.getCurrentPosition({
                    enableHighAccuracy: true,
                    timeout: 15000
                })

and in the Android phone settings about "Position", the radio button set on "GPS only",
I ALWAYS run in timeout.
All users argue me that my requests were too long (more than 15 seconds in this case), but when I tell them:
"Try to set the Position mode to 'High accuracy'",  they say: wow now it's fast !
Question so: why is there a timeout even when  'enableHighAccuracy: true' when using GPS ONLY ?

I'm having this exact same issue, any updates to this?

I also, didn't find solution for that issue.

+1 seeing the same issues on Galaxy S3 mini running Android 4.4.4

@Kukiwon Did you try to install Crosswalk? Because this is remedy. Also you can create function with setinterval, at first run it will fail but next one will execute properly.

@mikeon yes i'm using crosswalk. I'll try using an interval for getting a location and will keep you posted.

@mikeon your proposed solution using an interval did not work for me. Eventually we reinstalled Android on the phone and now it works. Looks like it was an issue with the device. Thanks again!

same issue as mica16... Is there a way out, or is there some other plugin that I can use?

I've tried cordova-plugin-geolocation with ngCordova but it always returned me Timeout error, meaning it can't get device's position in my even 5 minutes set timeout, so the final solution for Android 5.1 at least:
i've added permissions to AndroidManifest.xml
    <uses-permission android:name="android.permission.INTERNET" />
    <uses-permission android:name="android.permission.NETWORK_ACCESS" />
    <uses-permission android:name="android.permission.ACCESS_WIFI_STATE" />
    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
    <uses-permission android:name="android.permission.ACCESS_LOCATION_EXTRA_COMMANDS" />

it wasn't working without ACCESS_WIFI_STATE  permission. Now it works even without a sim card, with wifi connected and location enabled.
In case someone will find it helpfull.


Deprecate thanosSelector and expose mixin._config.thanos config variable for thanos sidecar

Description
This commit deprecates existing thanosSelector and exposes a single config variable mixin._config.thanos to customize thanos sidecar mixins. It follows same structure as https://github.com/thanos-io/thanos/blob/d2d74dac9878f633f1fb0e253bcc0b1cdea376d3/mixin/config.libsonnet
Signed-off-by: Arunprasad Rajkumar<EMAIL_ADDRESS>Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[ ] FEATURE (non-breaking change which adds functionality)
[ ] BUGFIX (non-breaking change which fixes an issue)
[X] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
Please put a one-line changelog entry below. Later this will be copied to the changelog file.

- Deprecate `thanosSelector` and expose `mixin._config.thanos` config variable for thanos sidecar

@paulfantom Can you please merge this? Thank you.


[help/question] Help with implementing Android lambda de-sugaring

I have written some basic code that re-sugars lambdas to a certain extent. So far the code is only supposed to recognize an anonymous class which was desugared from a lambda because Android prefixes them all as -$$Lambda$, after that the code drops the last part of the class name, which on Android is the MD5 hash of the source code that the lambda originally contained at compile time, prefixed with a $ to separate it from the central part, which is in the form ClassName$OptionalSubClassName$MaybeMore$optionalMethodName. The code fallbacks whenever that part is missing and doesn't handle well Proguard/R8 minified classes due to how I handle the part relative to the methodName, but I have worse issues going on at the moment.
So far I managed to get the code to work, but it only works partially, the system renames classes based on the interface they implement, nests them under the class that invokes them, but here lies the issue:
Lambda classes are not inner classes and I rename them in https://github.com/skylot/jadx/blob/master/jadx-core/src/main/java/jadx/core/dex/visitors/RenameVisitor.java#L76 by switching to my code if the name contains $$Lambda$ and doing all the messy renaming job. This results in the following issue, which I don't really know how to fix since I don't know well how JaDx works, it's my first time working on it.

I just realized lambda inlining has been implemented and merged 15 hours ago, my approach was a more "complex" idea to:

create a class for the interface implementation
move the lambda code inside it
inline the class at a later date by actually writing it as a lambda

But since the current implementation is much cleaner, I'm closing this issue and dropping my plans.

@HellPie I really appreciate your efforts to improve jadx! Thank you!
But next time before writing a lot of code I suggest opening issue with your thoughts and plans so I can help with implementation and also for work coordination.

I was planning to open a PR, but first I wanted to know if it was actually possible to implement what I wanted. I didn't waste a lot of time on it. I'm partly happy to see I can stop worrying about it because the current implementation is so much cleaner than mine.


[BEAM-1265] Fix PValue input in _PubSubReadEvaluator

This incorporates @davidcavazos' change proposed in #3483.

R: @aaltay

LGTM

I will merge after the tests pass.

Coverage remained the same at 70.85% when pulling 4dfd37fed7540b2748c60debc63367ab8f711f91 on charlesccychen:fix-te-inputs into ea9f7fbdde5ede5e5f5060da5dfe2749992a2fa7 on apache:master.

Merged. @charlesccychen  Could you send a PR to cherry pick it to the release branch?

Ignore the previous comment. CP PR: https://github.com/apache/beam/pull/3504


Allow node key to be specified on cmd line as optional argument

While this is not super safe, an optional way to do this might be useful in certain scenarios

Closing as there is no need for this now.


Should check server SSL capability

It is also MySQL 5.6 compatibility problem for #32 .
MySQL 5.6.x Community Edition does not support SSL, so if the user has enabled SSL and the MySQL server is 5.6.x Community Edition, the user will receive an error message containing a handshake failure.
In this case, checking the MySQL server capabilities in HandshakeV10Message will be more useful than checking the MySQL server version.
My personal opinion is that for MySQL, SSL cannot support only enable/disable, and should support the following 3 modes:

REQUIRED (high security): if server not support SSL, then throw a exception to channel (connection will be close by this exception)
OPTIONAL (default mode): if server not support SSL, then disable SSL.
DISABLED (plain mode): use plain connection without SSL.

R2DBC ConnectionFactoryOptions.SSL is a boolean option, I try treat true same as OPTIONAL, and append a SSL option Option.<Boolean>valueOf("sslRequired") for enable REQUIRED mode.
@mp911de Should I support optional SSL mode for MySQL 5.6 compatibility?
Another plan is just throw an exception and close the connection when the user has enabled SSL and the MySQL server does not support SSL. Then r2dbc-mysql still only has two modes, ENABLED or DISABLED. (Equivalent to ENABLED same as REQUIRED, and the OPTIONAL mode is removed)

What does the MySQL JDBC do? In doubt, R2DBC should try to find answers in JDBC.

@mp911de mysql-connector-j use two options for choice SSL mode, useSSL and requireSSL.

As long as the server is correctly configured to use SSL, there is no need to configure anything on the Connector/J client to use encrypted connections (the exception is when Connector/J is connecting to very old server versions like 5.6.25 and earlier or 5.7.5 and earlier, in which case the client must set the connection property useSSL=true in order to use encrypted connections). The client can demand SSL to be used by setting the connection property requireSSL=true; the connection then fails if the server is not configured to use SSL. Without requireSSL=true, the connection just falls back to non-encrypted mode if the server is not configured to use SSL.

Reference from document: https://dev.mysql.com/doc/connector-j/5.1/en/connector-j-reference-using-ssl.html

Note: mysql-connector-j 5.1.x target MySQL server version is 5.5.0 - 5.7.x

More intuitive:

useSSL is most like OPTIONAL, but it has 3 statuses: true, false and undefined.
requireSSL is most like REQUIRED.
mysql-connector-j check more things for optional SSL mode (including but not limited to checking server capabilities)

Thanks for clarifying. Let’s keep a single flag. In SQL Server we have a similar arrangement. The server can not support SSL, support SSL and demand SSL.
I would keep it the way: if the server wants ssl, then enable SSL. If the client wants ssl we should require SSL and fail if SSL is not available.

I am not sure if I understand the meanings. I tried to clarify: if the user sets ConnectionFactoryOptions.SSL to true, it means SSL required. If it is set to false, it means SSL disabled. If neither is configured, choose whether according to the SSL support of the MySQL server, it is also means SSL optional.

It’s slightly different. Not setting SSL to true means SSL is set to false. We had a similar discussion at https://github.com/r2dbc/r2dbc-postgresql/pull/104

MySQL has the similar arrangement as PostgreSQL in client version 5.7.23 or higher, connector-j version 8.0.13 or higher.

https://dev.mysql.com/doc/refman/5.7/en/connection-options.html#option_general_ssl-mode
https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-reference-using-ssl.html

Agree your suggestion, let's keep single flag and support Option.valueOf("sslMode") configuration.

SSL_MODE has supported yet, and single flag SSL would been keep.


Handle device-lost for text in DrawingIslands sample.

Description
This enables the text rendering in the DrawingIslands sample to properly recover when the D3D device is lost.
The DeviceLostHelper class listens for device-lost events on the specified DXGI device. This code is adapted from https://learn.microsoft.com/en-us/windows/uwp/composition/composition-native-interop.
The DrawingIsland class now includes an instance of DeviceLostHelper and uses it to listen for events on the device associated with the TextRenderer object. On device lost, the DrawingIsland::OnDirect3DDeviceLost method is called and does the following:

Tells the TextRenderer to recreate its device
Re-renders all the text items
Starts watching the new device

The TextRenderer class has new code to recreate its device. It also catches and ignores the exception in the unlikely event that a DXGI_ERROR_DEVICE_REMOVED error occurs during TextRenderer::Render.
I tested this by triggering multiple device losses using dxcap -forceTDR.

Thanks! Right now, this introduces multi-threading in, which gets us unblocked. I'll refactor this around later to use a DispatcherQueue to post back to the UI thread.


Backend: Item Introduction

@anselmbradford
@kurtw

Not related to the changes in this PR, but I noticed the link underlines extending too far in the screenshot. @kave could you collapse lines 64–66 into one line with no whitespace between the a tags and the inside content tag?

@Scotchester sure no problem, done

Beautiful, thanks!


Do not replay (flush) cached data from pushBestEffortsData after promotion

After a promotion, BestEffortsMonitoring.flushAfterActivePromotion(BestEffortsMonitoring.java:77) is called.
This as the effect of sending in the monitoring service some data that were created in a passive server related to some passive entities that do not exist anymore.
The goal of pushBestEffortsData is to be non reliable. I don't think data should be cached and replayed after a promotion, especially because 1) this is a non reliable channel and 2) data is constructed related to an old context where the server was passive and had passive entities. So this data is now completely obsolete and should be discarded.

talked with @myronkscott : a solution will be implemented in PR https://github.com/Terracotta-OSS/terracotta-platform/pull/413/commits


Warn about IonicPage Lifecycle

Warn IonicPage Lifecycle events are only available on IonicPage component, others components need Angular Lifecycle Hooks.
Short description of what this resolves:
In the NavController documentation, only Ionic Page Lifecycle events are present. At first, we have the impression that these events replace the Angular Lifecycle hooks, but that's not the case!
Changes proposed in this pull request:
Add warn
Ionic Version: 3

LGTM, will merge in a bit.

Thanks for the PR. Looks like this got lost in the shuffle. Merging.


Compatibility certification request for jaxb-ri 3.0.1-b02

[x] Organization Name ("Organization") and, if applicable, URL:
Eclipse Foundation
[x] Product Name, Version and download URL (if applicable):
Eclipse Implementation of JAXB 3.0.1-b02
[x] Specification Name, Version and download URL:
Jakarta XML Binding 3.0
[x] TCK Version, digital SHA-256 fingerprint and download URL:
Jakarta XML Binding TCK 3.0.0, SHA-256: 4ee5f5c12edf5212c7fe16f0a51e74ae48c7b7a72ffc40450dfdb07204fb5c0b
[x] Public URL of TCK Results Summary:
TCK results summary
[x] Any Additional Specification Certification Requirements:
None
[x] Java runtime used to run the implementation:
Oracle JDK 1.8.0_191, Oracle JDK 11.0.7
[x] Summary of the information for the certification environment, operating system, cloud, ...:
Linux
[x] By checking this box I acknowledge that the Organization I represent accepts the terms of the EFTL.
[x] By checking this box I attest that all TCK requirements have been met, including any compatibility rules.

approved by lazy consensus


Make diff syntax highlight more readable

Use red color for deleted nodes, greed – for added (like on GitHub).
GitHub Example:
-
Submit

+<button type="submit" class="cta-button">Submit</button>

I not sure why would I need that.
And could you also please not steal the design of my blog? I paid for it and fonts as well. It’s not a template.
https://alies.dev/

Hi @pepelsbey

And could you also please not steal the design of my blog?

I'm very sorry that this looks like I stole the design. I basically checked license (it's MIT), forked your site to build my personal blog plus learn best practices directly from your code. The idea was to change design step by step and only then officially launch the blog. Yesterday I made the first deployment and forgot to "hide" the unfinished version of the site.

I paid for it and fonts as well. It’s not a template.

Sorry for this also. I didn't know that, but I'm going to change fonts anyway. BTW, shouldn't this information about fonts be reflected in LICENSE.md?
Anyways, thanks for amazing work here (and only here). I'm promising you to change design and fonts by the site launch date.

I added the license section to the readme. I’m sorry it wasn’t clear enough before. Copying and reusing the parts of the code that are licensed under MIT is fine, but not the website as a whole because the design belongs to me. I hope you won’t use my website’s design on a public website anymore.


I have added a Data Science Interview Questions and answers PDF... Please merge it into your Repo

Hello, and thank you for your interest! I closed this PR because it is not appropriate to add to this repository a PDF authored by another person. I marked this PR as spam because this appears to be related to hacktoberfest, which has been generating a lot of noisy PRs lately. Please let me know if I've assumed incorrectly. All the best!


Aiven operator fails after upgrading to v0.9.0 from v0.4.0

Hi,
This is running on AKS with k8s 1.24.
Already removed namespace and all resources.
Re-added CRDs and waited. Deploying v0.9.0 helm chart will yield the following error

MountVolume.SetUp failed for volume "webhook-server-cert" : secret "webhook-server-cert" not found
and, after some time
Unable to attach or mount volumes: unmounted volumes=[webhook-server-cert], unattached volumes=[webhook-server-cert kube-api-access-bhsct]: timed out waiting for the condition

Any hints on what I may be doing wrong?
Thanks

It seems cert-manager vanished from server. Sorry for the false alarm.


Standardize boot.rb and rely on bundler to load all gems

Change config/boot.rb to match fresh Rails v4.2.1 app.

The syck gem is included in the Gemfile and so we don't need the
require 'rubygems' statement, as long as we move the require 'syck'
statement below the require 'bundler/setup' statement. require 'yaml'
can appear anywhere before the YAML::ENGINE.yamler assignment,
because YAML is part of the standard library.

These changes mean that our application boot is more like that of a standard
Rails app and allows people to avoid needing to use bundle exec to run the
tests using this technique.

:+1: the closer we can get to expected behaviour the better.

I was wondering why binstubs do not work on smartanswers.
:shipit:


can't use GPTSimpleVectorIndex with Microsoft Azure OpenAI

I can't use GPTSimpleVectorIndex with Microsoft Azure OpenAI
I have the following settings:
openai.api_type = "azure"
openai.api_base = "https://xyz.openai.azure.com/"
openai.api_version = "2022-12-01"
openai.api_key = os.getenv("OPENAI_API_KEY")

from gpt_index import GPTSimpleVectorIndex, SimpleDirectoryReader, LLMPredictor

documents = SimpleDirectoryReader('docs/test-data', recursive=True, required_exts=[".md"]).load_data()
llm_predictor = LLMPredictor(llm=AzureOpenAI(deployment_name="xyz-Curie", openai_api_key=openai.api_key))
index = GPTSimpleVectorIndex(documents=documents, llm_predictor=llm_predictor)

Error:
openai.error.InvalidRequestError: The API deployment for the resource does not exist.

i haven't really tested the AzureOpenAI stuff, i think we only have embedding support atm and not the full llm? https://github.com/jerryjliu/gpt_index/pull/254/files

Any update on this? Not sure where to get AzureOpenAI

It works if you specify both embedding and completion model:
embed_model = OpenAIEmbedding(deployment_name="text-search-ada-doc-001")
llm_predictor = LLMPredictor(llm=AzureOpenAI(deployment_name="davinci-003", openai_api_key=openai.api_key))
index = GPTSimpleVectorIndex(documents, embed_model=embed_model, llm_predictor=llm_predictor)

should be resolved with the nb provided in #402


Composer version swtich from .env dile

hi,
can you add WARDEN_COMPOSER_VERSION
parameter to .env file?
Thanks

You can use COMPOSER_VERSION= 1 or 2.

@navarr i didnt know this thanks


Creating Function/Method-Based Sequence Diagrams

Is your feature request related to a problem? Please describe.
"Spectest" has the functionality to create sequence diagrams. However, due to the coarse granularity of the generated sequence diagrams, I want to create sequence diagrams based on method calls.
When creating function/method-based sequence diagrams, it is straightforward to perform static analysis of the code. This method eliminates the need to execute it every time during testing, addressing concerns about slow performance.
Therefore, a command-based sequence diagram generation feature is developed to facilitate verification.

Ref. https://github.com/ondrajz/go-callvis


It's possible configure in Packer?

Question or Suggestion
Hi everyone,
I'll like to use this awesome theme, but current I use Packer to manager my plugins, so trying to do that don't work:
packer.lua
    use {
        'scottmckendry/cyberdream.nvim',
        lazy = false,
        priority = 1000
    }

after/plugin/colors.lua
require("cyberdream").setup({
    -- Enable transparent background
    transparent = true, -- Default: false

    -- Enable italics comments
    italic_comments = true, -- Default: false

    -- Replace all fillchars with ' ' for the ultimate clean look
    hide_fillchars = true, -- Default: false

    -- Modern borderless telescope theme
    borderless_telescope = true, -- Default: true

    -- Set terminal colors used in `:terminal`
    terminal_colors = true, -- Default: true

    theme = { -- Default: nil
        variant = "light", -- Default: "deafult" (dark)
        highlights = {
            -- Highlight groups to override, adding new groups is also possible
            -- See `:help highlight-groups` for a list of highlight groups

            -- Example:
            Comment = { fg = "#696969", bg = "NONE", italic = true },

            -- Complete list can be found in `lua/cyberdream/theme.lua`
        },

        -- Override a color entirely
        colors = {
            -- For a list of colors see `lua/cyberdream/colours.lua`
            -- Example:
            bg = "#000000",
            green = "#00ff00",
            magenta = "#ff00ff",
        },
    },
})

Result:

I use the default and most simple configuration, maybe losing something in the config.
Thanks!!

Sorry, but I made work partial
I add this function in my after/plugin/colors
function ColorMyPencils(color)

    color = color or "cyberdream"
    vim.cmd.colorscheme(color)

end
ColorMyPencils()

the result was

but telescope continues not working in transparent mode

Thanks @daltondiaz 🙂
I'll get some Packer install instructions added to the readme.
Telescope looks correct based on your configuration. You can set borderless_telescope to false in your config to make the window transparent.

@scottmckendry perfect!!!
Everything working, thanks!!


[Docs]: Code of Conduct checkbox

Issue category
Formatting or readability improvement
Description
See screenshot:

Have you read the Contributing Guidelines on reporting issues?

[X] I have read the Code of Conduct.

Self-service

[X] I'd be willing to address this documentation request myself.

Awesome. Thank you for reporting this @botgina


Freeze after import (maybe ROCm only)

When installing exllamav2 with pip install exllamav2, after importing exlamav2 python freezes and I have to exit with Ctrl + C.
I also wasn't able to get it running with the prebuild wheels for ROCm before (undefined symbol <some symbol in exllamav2_ext.cpython-310-x86_64-linux-gnu.so> - I unfortunately did not copy the exact error). But I wasn't able to reproduce this in another venv (did not want to break my installation again in my main venv xD)
I got it running by cloning the repository and building from source.
Exllama worked for me before. It broke right around when I added a second GPU to my system. But because problems with Pytorch kept me from using Exllama for some weeks I'm not sure if that is related or it's just that something changed with v0.0.11
p.s.: Is there a reason for using python setup.py install --user in the readme. This did not work for me. I use venv for virtual environments and pyenv for handling multiple python versions.

Freezing for me on CUDA 12.1 as well.

With the JIT version installed, the first time you import the library it compiles the C++/CUDA extension. This also happens if there's a change to the configuration such as adding a new CUDA device.
This can be a little slow and Torch doesn't really give any feedback while the build is going on, unless you ask it to be verbose (set verbose = True at the top of ext.py if you want) but then it dumps a huge amount of information to the console even when there's no work to do. I'll see if I can find some hacky way to get feedback just when it's building the extension, maybe. In the meantime, the latest dev version should reduce the compile time quite a bit by splitting the template instances into multiple compilation units.
Here are some things to try:

clone the repo, set the "EXLLAMA_NOCOMPILE" env variable and run python setup.py install --user. E.g. on Linux that's EXLLAMA_NOCOMPILE= python setup.py install --user. This is equivalent to pip install exllamav2 except you get the dev version instead of the latest release.
clone the repo and run python setup.py install --user without setting the env variable. This will install the library while building the extension locally, and you'll get feedback to see if anything's going wrong with the build. Subsequently importing the library should be "instant"
Install one of the prebuilt wheels. Make sure to match the CUDA version  (cuxxx) of your Torch installation, and your Python (cpxx) version.

I understand that the JIT version will install on first Import but I waited really long and still nothing happened (I waited way longer than it takes to build from source).
As I said, when building from source it works fine. (I'm still unsure why I suggest the --user flag)
After reinstalling with pip install exllamav2 I'm now able to reproduce the alternative Error I referred to before. I get the following output on my first import of exllamav2
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/data/linux_data/AI/LLM/WebUI/.venvs/llm1100/lib/python3.10/site-packages/exllamav2/__init__.py", line 3, in <module>
    from exllamav2.model import ExLlamaV2
  File "/data/linux_data/AI/LLM/WebUI/.venvs/llm1100/lib/python3.10/site-packages/exllamav2/model.py", line 17, in <module>
    from exllamav2.cache import ExLlamaV2CacheBase
  File "/data/linux_data/AI/LLM/WebUI/.venvs/llm1100/lib/python3.10/site-packages/exllamav2/cache.py", line 2, in <module>
    from exllamav2.ext import exllamav2_ext as ext_c
  File "/data/linux_data/AI/LLM/WebUI/.venvs/llm1100/lib/python3.10/site-packages/exllamav2/ext.py", line 15, in <module>
    import exllamav2_ext
ImportError: /data/linux_data/AI/LLM/WebUI/.venvs/llm1100/lib/python3.10/site-packages/exllamav2_ext.cpython-310-x86_64-linux-gnu.so: undefined symbol: _ZN3c107WarningC1ENS_7variantIJNS0_11UserWarningENS0_18DeprecationWarningEEEERKNS_14SourceLocationESsb

The --user flag installs the package for the current user rather than system-wide, which (at least in my experience) avoids some potential complications. I've seen system-wide installs require root access to install, and while maybe that's just because of some misconfiguration in the environment, it's always been my impression that the --user flag is the safer bet in any case.
As for that ImportError, I've seen it before, referring specifically to "UserWarning" and "DeprecationWarning" symbols, but I can't remember what the solution was. I think it had to do with an extension being compiled for one version of Torch and then loaded by another. Is it possible you have an old version of the extension cached in ~/.cache/torch_extensions/? Are there maybe multiple versions of Torch on your system in separate envs?

The --user flag installs the package for the current user rather than system-wide

So this is only needed when not using a venv? When I used the flag It installed it to a strange location, where python in my venv could not find it.

Is it possible you have an old version of the extension cached in ~/.cache/torch_extensions/?
Are there maybe multiple versions of Torch on your system in separate envs?

This could be possible. Since some time I use the nightly build of torch (2.3) as it fixed an error I had with my GPU. Before I used torch (2.2) and some venvs might still have this version installed.

Same here, it was working until I trained my own model with axolotl and converted it to exl2.
Now it freezes on importing (from exllamav2 import) no matter what.
I deleted whole env and reinstalled exllamav2, but no matter what versions of exllamav2 or torch I use it freezes on import.
FYI I'm on Ubuntu.

how to check if it is using ROCm or not for generating text . please help me

Not sure what change but for me the everything is working fine again with version 0.0.12

Alright this is weird, after I fixed freezes by reinstalling everything I could multiple times it finally worked for a while.
Just now, I made some changes to my code (generation/prompt part) it now freezes AGAIN! I did not touch pip at all!!


Enabling ingress V1 in helm chart does not work

If ingress.v1beta1.enabled is False and ingress.v1.enabled is True, no ingress rule gets deployed. This is because if condition at Line 1 here disables the entire file.
Ingress rules should be placed in separate files.

cc @xiangfu0

Please assign the issue to me. Thanks

Hi, here is the PR


Revisit git submodule magic

We need to look into the git submodule magic because there are some corner cases where data can get lost. One of the cases is when files are added in a submodule but not committed.
Example steps to replicate:
echo FOO > Tools/jMAVSim/new_file.txt
(cd Tools/jMAVSim && git add new_file.txt)
make clean
make

Great, thanks!
FYI: @georgehines

This is one the problems with submodules where there might not be a good solution. I think we're going to find that running git submodule update --init before cmake fixed a lot of problems with people not cloning recursively or having out of date submodules.
Having make clean recurse into each build directory and run clean might be what you really want instead of the current behaviour.

CMake is not designed to have external libraries living in the source tree, specifically, it is not appropriate to do a add_subdirectory(external_library)
All external libraries should be built how I built nuttx, in the build directory. Then linked to at compile time. This is standard cmake procedure, and  you can think of it as similar to a ros dev workspace.
See the mechanics here: https://cmake.org/cmake/help/v3.0/module/ExternalProject.html

For developers who want to edit  git submodules within the source of qgroundcontrol, this might be another option: http://www.diracprogram.org/doc/release-12/programmers/external_projects.html, although I have never tried it yet.


Add required attribute to register inputs

Like the login inputs, it is better that the required attribute added to register view inputs. #1

Thanks!


Guide to Searching for Materials

Search Terms and Limiters for Finding Articles
Time Range
We’re looking for historical articles between 1880 and 1930, so making sure to limit your search timeframe will make (mostly) everything relevant to the period. While searching on some databases, even with date ranges active, you may still continue to get one or two articles that aren’t from the time period—so watch out for that!
Historical Figures
Searching specifically by name is very helpful toward pinpointing exactly what you need. There are some figures you can search simply by last name (i.e. "Tesla", "Edison") but there are others you may need to search by full name to specifically find the person and not the company (i.e. "George Westinghouse" and not "Westinghouse"). Play around with the search terms and see what works best for what you're looking for.
Article Type
If you're looking in a database, you should be able to limit the results by the type of text. We're looking for newspaper articles, journal articles, and even propaganda or advertising for or against the two current types. You'll want to take down the information of the publisher, author (if there's a listed author), sponsor (if you can find one), date of publication, and provide a URL so that we can come back to it later. You can find a template I made for us right here!
Places to Search
Here is a list of online resources we can search through to find what we need. When we find more, we'll add them here:

Inventions & Experiments of Nikola Tesla (Articles & Interviews) https://teslaresearch.jimdo.com/articles-interviews/
Tesla Universe: https://teslauniverse.com/nikola-tesla/articles
JSTOR*: http://www.jstor.org.pitt.idm.oclc.org/
*You'll need to log in with your Pitt ID to access JSTOR

At some point, we may be incorporating modern articles to compare to historical articles. The Pitt Physics & Astronomy Database may help us with that when the time comes. We'll talk more about this on our next meeting Friday! For now, don't worry about using this.
We'll need to post the articles we find into the container folder I made specifically for this purpose. I have two folders in there, one labelled "AC" and the other "DC". If you find an article about (or pro-) AC current, please sort it into that folder, and vice versa with DC.
File Naming Convention for Uploading Articles to the Container:
Type = (JN - Journal) (NP - Newspaper) (AD - Advertising)
Date = (YYYY-MM-DD)
Example: NP1880-03-28.txt

@Jamielynn92 Jamie--Aaron and I continued to talk for about an hour after you left. If you have any questions or need any clarification on anything, please let me know! I can always find a time to meet with you and help walk you through anything if you'd like a more visual walkthrough rather than online direction.