Analyzer warns about function that always returns

When a function returns from a switch statement on an enum and covers all enum values but does not have a return statement at the end, the analyzer show a warning, gist at https://gist.github.com/anonymous/31caf98143f8d25c7e06f6647b0c03a2, snippet below:
enum FancyEnum {
  first,
  second,
  third
}

int fancyFunction(FancyEnum param) {
  switch(param) {
    case FancyEnum.first:
      return 1;
    case FancyEnum.second:
      return 2;
    case FancyEnum.third:
      return 3;
  }
}

This might be correct given the language specification, is just confusing as a user, specially since the analyzer warns for each missing value in the switch statement itself.

Yep. This is a known issue, and one that will probably get better if we get non-nullable types in the language.
The problem is that param can have the value null, which isn't covered by any of the cases, in which case the method will return null, which might not be the intended behavior.
It would be great, though, to improve the error message to explain this to users, because it's not obvious to most users.


FIX compilation error

This change fixes a compilation error on Xcode 7 beta 4 .

Thanks for your pull request and sorry for late response!


Move core interface into XAIBase.jl package

Closes #151.

This introduces some breaking changes through XAIBase:

Explanation field neuron_selection has been renamed to output_selection
Explanation field heatmap that describes heatmapping presets has been added
heatmap keyword argument cs has been renamed to colorscheme

Closed #151.


Update CONTRIBUTING.rst and docs.

We turn on Circle CI to test it somewhere else; and it listens to us pushing. Since master and anything not testing Circle CI doesn't have the config, it fails.
Related PR: https://github.com/BlueBrain/nmodl/pull/1155


Adding AWS multi-node deployment capabilities.

This PR is based on all-in-one AWS deployment #39.
Applying changes make deploy_ocp3_cluster role to create a simple 2 node - 1 master AWS instance setup, and then deploy a v3.x cluster with OA.
Steps to reproduce the deployment:

Prepare environment by supplying SUB_USER, SUB_PASS, AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY variables.
Execute the ansible-playbook get-openshift-ansible.yml.
Prepare the instances and deploy the cluster with ansible-playbook deploy_ocp3_cluster.yml.

Deprecated by #65


Add bytes_mut and into_bytes for ResponseData

I needed these functions to avoid unnecessary copy so hopefully adding them is ok.

@kristianhasselknippe thank you


@preact/signals-react-transform not working in React Native

[ ] Check if updating to the latest version resolves the issue

Environment

[ ] I am using @preact/signals-core
[ ] I am using @preact/signals
[X] I am using @preact/signals-react

[X] I understand usage changed in v2, and I've followed the React Integration instructions

Describe the bug
I added babel plugin but it doesn't work in React Native. it throws error: Cannot import useSignals outside of an ESM module file

Well the error is correct, you cannot use import outside of an ESM context.
Our export map is correct & providing the right formats, do you mean to use require instead?

Hi @rschristian, I followed the guide to add Babel transform, My project is just a React Native app but the babel  has the errorCannot import useSignals outside of an ESM module file .  The signal can work but I need to add manual import { useSignals } from '@preact/signals-react/runtime';. it should be works automatic

My project is just a React Native app

We need an actual reproduction in order to help, this isn't enough to go on.
We do ship both CJS & ESM builds, your config may be pulling the wrong one?

Hi @rschristian, I think the issue is in my repo, I compile everything to CommonJS. Do you know how to signal works for this case?

I reckon because you preface the babel import with module: it will default to import syntax, no?

I provide support of react-native in @preact-signals/safe-react
I would like to contribute this feature to the mainstream library when I will have time

I appreciate your support! But after I upgrade the latest version of @preact/signals-react-transform, I got the error

We need a reproduction

@XantreDev

I appreciate your support! But after I upgrade the latest version of @preact/signals-react-transform, I got the error

I faced the same problem (it is related to babel plugin) as soon as I enable it this error appears (if I use useSignals() ) everything works.
Playback can be done simply by creating a new project and connecting signals as it is written in the documentation (and when using a signal this error appears).
npx create-expo-app DemoSignalApp --template tabs

@URVL Which version of the babel plugin do you use?

<EMAIL_ADDRESS>"^0.4.0",

сб, 21 сент. 2024 г. в 09:41, Valerii Smirnov @.***>:

@URVL https://github.com/URVL Which version of the babel plugin do you
use?
—
Reply to this email directly, view it on GitHub
https://github.com/preactjs/signals/issues/575#issuecomment-2365029644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGJTAP454DFRN35KRXVCQDZXUIIVAVCNFSM6AAAAABJY4N3NCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGAZDSNRUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Interesting. I will try to figure out why it happens in my free time
Btw, you can try @preact-signals/safe-react, because I've made it especially for a react native app. It worked on the 0.72 version of react native. I hope it still works

I get this and similar combination of problems, can we reopen this?
When using react 0.74+ the get the error Cannot import useSignals outside of an ESM module file when the version of the babel plugin is < 0.4.0
when the version of the plugin is 0.4.0, the error becomes:
 ERROR  RangeError: Maximum call stack size exceeded

This error is located at:
    in AppContainer
    in app(RootComponent), js engine: hermes
 ERROR  RangeError: Maximum call stack size exceeded

This error is located at:
    in AppContainer
    in app(RootComponent), js engine: hermes
 ERROR  RangeError: Maximum call stack size exceeded

This error is located at:
    in AppContainer
    in LogBox(RootComponent), js engine: hermes
 ERROR  RangeError: Maximum call stack size exceeded

This error is located at:
    in AppContainer
    in LogBox(RootComponent), js engine: hermes

@XantreDev
@preact-signals/safe-react seems to be working


Issue with version control policy

First, thank you for providing a great Flutter library.
However, I think the version management method of this project is wrong.
Minor version changes should guarantee the operation of previous versions.
However, due to the 1.6.0 version update, Config's showRecentsTab is not available, so
Many projects that were based on the assumption that showRecentsTab is available have encountered problems.
(For example, chatview)
Please remember that minor versions guarantee previous versions, which is why we can use ^ in Flutter dependencies.
If I am wrong, please give me feedback.
Thank you.

@MarsMan13 Thanks for pointing this out. I will try to be more careful with versioning from now on 🙏


Annoying warning

I am kind of tired of seeing this every time I run my test suite :o) - I hope it can be fixed.
/usr/local/lib/ruby/gems/2.2.0/gems/simplecov-html-0.8.0/lib/simplecov-html.rb:58: warning: possibly useless use of a variable in void context

The offending line even says "FIXME" :o)

This was fixed in 43141a04. It will be in the next release. @colszowka we should probably patch release, given doesn't look like this repo gets much activity.

ping @colszowka gem owner simplecov has me and @xaviershay, but gem owner simplecov-html has you alone :(

@bf4 Thanks for the heads up, you, @sferik and @xaviershay should now all be also owners of the simplecov-html gem. I'll verify #36 works fine and will ship 0.9.0 today.

Arghs, so it turns out we have a ~> 0.8.0 version dependency in the simplecov gemspec, which prevents me from shipping this as 0.9.0 for SimpleCov 0.9.1.
@bf4 @xaviershay What do you think, should we rather ship this as 0.8.1 to avoid having to bump simplecov as well, or rather ship SimpleCov 0.9.2 with an updated gem dependency of ~> 0.9.0 and then ship this as 0.9.0 to keep version parity?

Change it to gem.add_runtime_dependency 'simplecov-html', '~> 0.8' and it works for everyone, no?

@bf4 People will still need to update simplecov to 0.9.2 then, and would be allowed to use the 0.8 tree of simplecov-html, so some might miss the bugfixes. I'll make it ~> 0.9.0 so we have parity between simplecov version numbers and simplecov-html

seems good


Logo links to home page

Should we use Link from react-router now that this is internal?

Should we use Link from react-router now that this is internal?

Sigh... You're right. Let's do it right. Done.

Should we use Link from react-router now that this is internal?

Sigh... You're right. Let's do it right. Done.

Wait is this just a styled  from MUI? Or do they use react-router under the hood


Sb module & HDD Drive

Added some more functionality to SB Module and possibly fixed HDD module on Windows.

Due to your latest changes this will not merge cleanly. Will see if I can find fixes to merge it.

Just had a quick glance and it looks good - glad that you've got the disk space widget working in Windows, people will be pleased.
In the future though, if you could make pull requests for just one thing (so in this case, one for changes to the Sickbeard module and another for the HDD module) then that will make it a lot easier to review, as I can approve/reject each thing independent of the others. Reviewing everything all mashed together for v0.1 was a nightmare that I hope never have to repeat :)

(although don't bother splitting this pull request into two, I don't mind this time as the changes are pretty small, I just meant for future reference)

My modifications weren't very big and I didn't expect anything to change in such a short time for exactly the files I modified. However, I should have made a pull request earlier.
As you can see from the name of my branch (SB_Module) it was going to be specifically for SB stuff but as everything was "minor" I thought I could just quickly do the HDD stuff too. Next time will make pull request for most modifications.

I understand, just trying to promote best practice and good habits and all that.
In fact I should probably write a programming style guide on Lighthouse. Will knock something quick together now...

FYI: https://maraschino.lighthouseapp.com/projects/85614/messages/53811-programming-style-guide

Just saw your comment re. it not merging cleanly. Don't worry, I'm happy to resolve the conflicts when I review the code - will probably be a bit easier for me to do as I know my way around what has changed.

Ok now we can merge. I  had a look at the conflict and it was a very silly one. Will test it out and if I don't find any problems I will merge it OK?

Added to experimental.

This is on master now. Nice changes, especially the ability to change episode status from within the SB module, that's genuinely useful :) You might want to check the code though, it didn't merge cleanly so I ended up making a few changes (mostly to what set_episode_status in sickbeard.py returns).

This is on master now. Nice changes, especially the ability to change episode status from within the SB module, that's genuinely useful :) You might want to check the code though, it didn't merge cleanly so I ended up making a few changes (mostly to what set_episode_status in sickbeard.py returns).

Ye it's giving an error when changing status. The idea was to throw an error if something wrong happened thats why I was returning 1/0 (true/false) I think. So, if false the popup should appear otherwise no notifications are needed.

Feel free to make whatever changes you like, but it's best if we always stick to returning either JSON or a snippet of HTML as appropriate; for example, in this case:
{ status: 'success' }
or
{ status: 'error' }
vs 1 or 0.

I already changed set_episode_status to return JSON, and the JS to use it. Would be a good idea to change it elsewhere if you've used anything different, though.

Great it seems to be good now.


can't ref in

Issue description

components: Input
reactstrap version #6.0.1
import method umd/csj/es
react version #16.3.2
bootstrap version #4.1.1

What is happening?

I just put ref on Input tag but, I can't I use typescript, so I used any type, but doesn't work
What should be happening?
can use ref in

Steps to reproduce issue

... Declare variables     ex) public password: any = null
... link ref  in <Input ref  => { this.password = ref />
... do it! this.password.focus()
... Error!

Error message in console
TypeError: _this.password.focus is not a function

Code

class Login extends React.Component {
public password: any = null

someMethod = () => {
     this.password.focus()
}

render() {
    return (
                <Input
                  type="password"
                  placeholder="Password Here"
                  ref={ref => {
                    this.password = ref
                  }}
                />
                <button onClick={this.someMethod}> Click me! </button>
      )
   }
}

ref can only get a reference to the thing you use it on, Input. Use innerRef to get a reference to the underlying input.

Thank You!

ref can only get a reference to the thing you use it on, Input. Use innerRef to get a reference to the underlying input.

Thanks man,you save my day


Make Svelte packages community-maintained

@jordansexton the Svelte packages are published in npm:
https://www.npmjs.com/package/@svelte-on-solana/wallet-adapter-core
https://www.npmjs.com/package/@svelte-on-solana/wallet-adapter-ui
https://www.npmjs.com/package/@svelte-on-solana/wallet-adapter-anchor
Let me know if you need more info or support about them.
Thanks!


Change of Name

We are still using twitter but it should be changed to X or X previously known twitter. I would like to work on this one

please change it to X (fka Twitter)

Hey @angiejones, I see this issue has been idle for a couple of days. Can I pick this up?

@aniketh-varma thanks for offering your help! Sure thing go for it, I'm assigning it to you!

This one is already completed I have merged the pull request for this one

@Pratham271 oh, I did not know. Can you please share the PR with me?


"File modified since transfer scheduled" when syncing symlinks

Which version of the AzCopy was used?
10.2.1
Which platform are you using? (ex: Windows, Mac, Linux)
Arch Linux
What command did you run?
sync /home/xuanrui/arch-repo https://archlinuxxuanruiqi.blob.core.windows.net/$web --recursive
What problem was encountered?
Some files failed to sync with the error: UPLOADFAILED: /home/xuanrui/arch-repo/repo/xuanrui.db.sig : -REDACTED- : File modified since transfer scheduled. Coincidentally, all of those files were symlinks. All other files synced without problems.
How can we reproduce the problem in the simplest way?
Create a folder, add a file, make a symlink to it in the same directory, and sync it to a blob.
Have you found a mitigation/solution?
None yet. I can force copy the files to the blob, but this is more of a non-solution.

May be related to other symlink issues, so assigned to Adele, so this one can be looked at with those.

Once I have my Linux environment entirely going, I'll be sure to check these out-- While I'm able to reproduce at least some of them on WSL, it certainly isn't optimal for debugging.

Strangely enough, I can't seem to reproduce this. I've tried the following arrangements:
Symlinking in the same directory as we're syncing

testDir/aaa
testDir/bbb (bbb => aaa)

Symlinking in a different directory than we're syncing

testDir2/ccc
testDir/ddd (ddd => ccc)

Symlinking to a subdirectory of the one we're syncing

testDir/myDir/eee
testDir/fff (fff => myDir/eee)

I could be misunderstanding your scenario, if so, could you clarify?

Hmm, maybe try issuing the command repeatedly? It only began failing for me after a few times.

Still failing for me after multiple attempts. azcopy is the only way to copy symlinks reliably as I know. BTW, my use case is the 1st: sym-linking in the same directory as we're syncing.

I also find that azcopy copy isn't following symlinks properly...

Out of curiosity, for azcopy copy, are you using the flag --follow-symlinks?

Furthermore, it seems as though cmd/sync:520 has a comment saying that we've disabled following symlinks at the moment.

So, I've been totally incapable of reproducing this, even on a Linux VM. Are you sure there's not some other program using your files?
I may be incorrectly reading your scenario, but are you potentially trying to keep a backup of your dotfiles?

If so, other programs using the files may actually interfere with LMTs.

My file system is Btrfs. Maybe this is actually a Btrfs problem.

I am totally sure that no userspace program is using my files, yes.
I didn't know that the --follow-symlinks option existed. I'll give it a try. OTOH, is the fact that azcopy sync does not correctly copy symlinks probably related to the problem that azcopy sync cannot follow symlinks in the first place?

I'm using Azure Blob Storage to host a package repository for Arch Linux (https://wiki.archlinux.org/index.php/Unofficial_user_repositories). I put my packages in the $web directory and run my repository as a public, static website.

I have this same issue. I am copying veeam backup files from a Buffalo Terastation to azure blob using azcopy 10.2.1.
Some files fail immediately every time with the error "File modified since transfer scheduled". The file has not actually been modified.
I am using azcopy on server 2016.
Running "azcopy sync" command.

Update:
I built a linux box to access the terastation via nfs.
Currently running azcopy and initially it seems to be working without the "File modified since transfer scheduled" error I was getting when running azcopy on my server 2016 box.
Will post again if this is successful.

Using a linux box seems to be a successful workaround at this time.

The original sync implementation doesn't support following folder symlinks and accidentally enumerates them as files.
Oddly enough, we record the LMT of the symlink itself and check against the LMT of the destination.

As a part of the copy refactor, I'll modify this enumerator to ignore symlinks outright if follow symlinks is off.

@adreed-msft

we record the LMT of the symlink itself

In cases where followSymlinks = true, will this be fixed by the refactoing too?

Yes, this has been fixed by the refactoring, and has been released. Enjoy!


Request for method Drop

There is a case in my project, when  I have to delete database completely including main db file and it's journal, if it exists. I implemented this logic on my side, but it would be really useful to have such a method in LiteDatabase class.

Really thought
var col = liteDB.GetCollection<SolarSystem>("systems"); 			col.DropIndex("systems");
Would do it.

Hi! With the objective of organizing our issues, we are closing old unsolved issues. Please check the latest version of LiteDB and open a new issue if your problem/question/suggestion still applies. Thanks!


siconos bullet installation

Error in siconos local installation.

Dear Ritesh,
Could you give some details about your environment? Linux? distro? compiler? verion of swig and gcc?
For the first error:

first of all, try to clean your build directory. It is very curious that you get targets with the same.
how do you install bullet ? could you give me some details ?

For the second error, vtk is not installed. please install it.

Environment Linux
SWIG Version 4.0.2
gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
CMakeError.log
CMakeOutput.log
I use following cmd for bullet installation
cmake -S /home/rigupta/Bureau/siconos -DBULLET_INSTALL=ON

vtk is now installed and siconos_build (manually created directory) is cleaned and the following command is relaunched.
cmake -S /home/rigupta/Bureau/siconos -DBULLET_INSTALL=ON -DWITH_BULLET=ON -DCMAKE_INSTALL_PREFIX=/home/rigupta/.local/bin/siconos
The error of same name existing still persists.

CMakeError.txt
CMakeOutput.txt


Added custom history folder setting – defaults to .history

Allows a user to specify a folder other than .history for the history folder.

Hi, thank you for this PR, but it's not complete, see HistoryController#internalDecodeFile

Very stale, replaced by #117


Error Creating VM

Hi I was trying to use your script but I got this error below:

Any idea why?
@edit:
It looks like that thise line is causing an error:

After proxmox update i get almost the same error but other line:
[ERROR] 4@127 Unknown failure occured.

Hi, I've got a similar error too

could someone help me solve it? Thanks!

I had the same issue trying to install hassos this evening, using this script. While I don't know what caused the problem, updating Proxmox and rebooting the host solved the issue. Here is where I found the soltuion (I have no other hasos installs and no issues with my Proxmox setup). The install went ahead flawlessly and I am now in the Home Assistant web UI.

Is this still a problem?

I think it might still be an issue.  I ran up a fresh proxmox install today, so the server has been rebooted.  When using the script I get the following (data is zfs storage):
bash -c "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/master/install.sh)"
[INFO] Using 'data' for storage location.
[INFO] Container ID is 104.
Getting URL for latest Home Assistant disk image...
Downloading disk image...
Extracting disk image...
Creating VM...
[ERROR] 4@135 Unknown failure occured.

@dmshimself I am unable to reproduce your results. Please run the following command in a Proxmox shell and post the entire output.
bash -xc "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/master/install.sh)"

Thanks for that and here is the output:
root@pve:~# bash -xc "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/master/install.sh)"

set -o errexit
set -o errtrace
set -o nounset
set -o pipefail
shopt -s expand_aliases
alias 'die=EXIT=$? LINE=$LINENO error_exit'
trap die ERR
trap cleanup EXIT
++ mktemp -d
TEMP_DIR=/tmp/tmp.geHgA51VP1
pushd /tmp/tmp.geHgA51VP1
read -r line
++ awk 'NR>1'
++ pvesm status -content images
++ echo data zfspool active<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER> 61.97%
++ awk '{print $1}'
TAG=data
++ echo data zfspool active<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER> 61.97%
++ awk '{printf "%-10s", $2}'
TYPE='zfspool   '
++ echo data zfspool active<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER> 61.97%
++ numfmt --field 4-6 --from-unit=K --to=iec --format %.2f
++ awk '{printf( "%9sB", $6)}'
FREE=' 1001.51GB'
ITEM='  Type: zfspool    Free:  1001.51GB '
OFFSET=2
[[ 38 -gt '' ]]
MSG_MAX_LENGTH=38
STORAGE_MENU+=("$TAG" "$ITEM" "OFF")
read -r line
++ echo local-lvm lvmthin active<PHONE_NUMBER> 9887899<PHONE_NUMBER> 0.55%
++ awk '{print $1}'
TAG=local-lvm
++ echo local-lvm lvmthin active<PHONE_NUMBER> 9887899<PHONE_NUMBER> 0.55%
++ awk '{printf "%-10s", $2}'
TYPE='lvmthin   '
++ awk '{printf( "%9sB", $6)}'
++ numfmt --field 4-6 --from-unit=K --to=iec --format %.2f
++ echo local-lvm lvmthin active<PHONE_NUMBER> 9887899<PHONE_NUMBER> 0.55%
FREE='    1.63TB'
ITEM='  Type: lvmthin    Free:     1.63TB '
OFFSET=2
[[ 38 -gt 38 ]]
STORAGE_MENU+=("$TAG" "$ITEM" "OFF")
read -r line
'[' 2 -eq 0 ']'
'[' 2 -eq 1 ']'
'[' -z '' ']'
++ whiptail --title 'Storage Pools' --radiolist 'Which storage pool you would like to use for the container?\n\n' 16 61 6 data '  Type: zfspool    Free:  1001.51GB ' OFF local-lvm '  Type: lvmthin    Free:     1.63TB ' OFF
STORAGE=data
'[' -z x ']'
info 'Using '''data''' for storage location.'
local 'REASON=Using '''data''' for storage location.'
local 'FLAG=\e[36m[INFO]\e[39m'
msg '\e[36m[INFO]\e[39m Using '''data''' for storage location.'
local 'TEXT=\e[36m[INFO]\e[39m Using '''data''' for storage location.'
echo -e '\e[36m[INFO]\e[39m Using '''data''' for storage location.'
[INFO] Using 'data' for storage location.
++ pvesh get /cluster/nextid
VMID=100
info 'Container ID is 100.'
local 'REASON=Container ID is 100.'
local 'FLAG=\e[36m[INFO]\e[39m'
msg '\e[36m[INFO]\e[39m Container ID is 100.'
local 'TEXT=\e[36m[INFO]\e[39m Container ID is 100.'
echo -e '\e[36m[INFO]\e[39m Container ID is 100.'
[INFO] Container ID is 100.
msg 'Getting URL for latest Home Assistant disk image...'
local 'TEXT=Getting URL for latest Home Assistant disk image...'
echo -e 'Getting URL for latest Home Assistant disk image...'
Getting URL for latest Home Assistant disk image...
RELEASE_EXT=vmdk.gz
++ cat
++ python3
URL=https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz
'[' -z https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz ']'
msg 'Downloading disk image...'
local 'TEXT=Downloading disk image...'
echo -e 'Downloading disk image...'
Downloading disk image...
wget -q --show-progress https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz
hassos_ova-4.11.vmdk.gz            100%[================================================================>] 237.69M  1.36MB/s    in 3m 38s
++ basename https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz
FILE=hassos_ova-4.11.vmdk.gz
msg 'Extracting disk image...'
local 'TEXT=Extracting disk image...'
echo -e 'Extracting disk image...'
Extracting disk image...
gunzip -f hassos_ova-4.11.vmdk.gz
++ pvesm status -storage data
++ awk 'NR>1 {print $2}'
STORAGE_TYPE=zfspool
'[' zfspool = dir ']'
for i in {0,1}
disk=DISK0
eval DISK0=vm-100-disk-0
++ DISK0=vm-100-disk-0
eval DISK0_REF=data:vm-100-disk-0
++ DISK0_REF=data:vm-100-disk-0
for i in {0,1}
disk=DISK1
eval DISK1=vm-100-disk-1
++ DISK1=vm-100-disk-1
eval DISK1_REF=data:vm-100-disk-1
++ DISK1_REF=data:vm-100-disk-1
msg 'Creating VM...'
local 'TEXT=Creating VM...'
echo -e 'Creating VM...'

A colleague re-installedanother fresh install, so the sorage usage changed.  However I re-ran the first script and this second one, so the results are still consistent.
root@pve:~# bash -xc "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/master/install.sh)"

set -o errexit
set -o errtrace
set -o nounset
set -o pipefail
shopt -s expand_aliases
alias 'die=EXIT=$? LINE=$LINENO error_exit'
trap die ERR
trap cleanup EXIT
++ mktemp -d
TEMP_DIR=/tmp/tmp.bLk8dSvpK5
pushd /tmp/tmp.bLk8dSvpK5
read -r line
++ awk 'NR>1'
++ pvesm status -content images
++ echo local-zfs zfspool active<PHONE_NUMBER> 15113492<PHONE_NUMBER> 0.87%
++ awk '{print $1}'
TAG=local-zfs
++ echo local-zfs zfspool active<PHONE_NUMBER> 15113492<PHONE_NUMBER> 0.87%
++ awk '{printf "%-10s", $2}'
TYPE='zfspool   '
++ echo local-zfs zfspool active<PHONE_NUMBER> 15113492<PHONE_NUMBER> 0.87%
++ awk '{printf( "%9sB", $6)}'
++ numfmt --field 4-6 --from-unit=K --to=iec --format %.2f
FREE='    1.57TB'+ ITEM='  Type: zfspool    Free:     1.57TB '
OFFSET=2
[[ 38 -gt '' ]]
MSG_MAX_LENGTH=38
STORAGE_MENU+=("$TAG" "$ITEM" "OFF")
read -r line
'[' 1 -eq 0 ']'
'[' 1 -eq 1 ']'
STORAGE=local-zfs
info 'Using '''local-zfs''' for storage location.'
local 'REASON=Using '''local-zfs''' for storage location.'
local 'FLAG=\e[36m[INFO]\e[39m'
msg '\e[36m[INFO]\e[39m Using '''local-zfs''' for storage location.'
local 'TEXT=\e[36m[INFO]\e[39m Using '''local-zfs''' for storage location.'
echo -e '\e[36m[INFO]\e[39m Using '''local-zfs''' for storage location.'
[INFO] Using 'local-zfs' for storage location.
++ pvesh get /cluster/nextid
VMID=102
info 'Container ID is 102.'
local 'REASON=Container ID is 102.'
local 'FLAG=\e[36m[INFO]\e[39m'
msg '\e[36m[INFO]\e[39m Container ID is 102.'
local 'TEXT=\e[36m[INFO]\e[39m Container ID is 102.'
echo -e '\e[36m[INFO]\e[39m Container ID is 102.'
[INFO] Container ID is 102.
msg 'Getting URL for latest Home Assistant disk image...'
local 'TEXT=Getting URL for latest Home Assistant disk image...'
echo -e 'Getting URL for latest Home Assistant disk image...'
Getting URL for latest Home Assistant disk image...
RELEASE_EXT=vmdk.gz
++ python3
++ cat
URL=https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz
'[' -z https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz ']'
msg 'Downloading disk image...'
local 'TEXT=Downloading disk image...'
echo -e 'Downloading disk image...'
Downloading disk image...
wget -q --show-progress https://github.com/home-assistant/operating-system/releases/download/4.11/hassos_ova-4.11.vmdk.gz
hassos_ova-4.11.vmdk. 100%[=======================>] 237.69M   215KB/s    in 11m 43s
++ basename https://github.com/home-assistant/operating-system/releases/download/4.11/
hassos_ova-4.11.vmdk.gz
FILE=hassos_ova-4.11.vmdk.gz
msg 'Extracting disk image...'
local 'TEXT=Extracting disk image...'
echo -e 'Extracting disk image...'
Extracting disk image...
gunzip -f hassos_ova-4.11.vmdk.gz
++ awk 'NR>1 {print $2}'
++ pvesm status -storage local-zfs
STORAGE_TYPE=zfspool
'[' zfspool = dir ']'
for i in {0,1}
disk=DISK0
eval DISK0=vm-102-disk-0
++ DISK0=vm-102-disk-0
eval DISK0_REF=local-zfs:vm-102-disk-0
++ DISK0_REF=local-zfs:vm-102-disk-0
for i in {0,1}
disk=DISK1
eval DISK1=vm-102-disk-1
++ DISK1=vm-102-disk-1
eval DISK1_REF=local-zfs:vm-102-disk-1
++ DISK1_REF=local-zfs:vm-102-disk-1
msg 'Creating VM...'
local 'TEXT=Creating VM...'
echo -e 'Creating VM...'
 It seems that others having same problem are also using ZFS and some have eventually succeeded after tricks like reboots or retrying several times.
My theory is that everyone experiencing this problem is using ZFS for datastore and have either slow disk subsystem or enough load on disks causing "pvesm alloc" call to fail during "zfs create" step. Proxmox enforces 5 second timeout for zfs create which is simply too low for some of us.
After removing /dev/null redirection of pvesm output I get following error message:
"command 'zfs create -s -V 1024k rpool/data/vm-119-disk-0' failed: got timeout"
Trying to run "zfs create -s -V 1024k rpool/data/vm-119-disk-0" manually fails as volume is actually created just fine, simply not fast enough for pvesm to be happy.
Extending timeout from 5 seconds to 15 seconds by patching Proxmox code makes this work (for me) but is obviously not a good solution as it will be overwritten by Proxmox updates. Waiting fix from Proxmox might take awhile as this problem was first reported over 5 years ago.
If Whiskerz is willing to maintain workaround for Proxmox side bug this could be solved as part of install.sh. If "pvesm alloc" returns error code 4 then enter loop that sleeps for couple seconds, check if new volume was created successfully despite pvesm error, repeat for n times if necessary and only if it still doesn't exist then bail out with error. Otherwise continue to next step of install.
As said this is actually old and known bug: https://forum.proxmox.com/threads/zfs-plugin-timeouts.18882/#post-110786
https://github.com/proxmox/pve-storage/blob/74b724a6993fbe859a76d46dd5f2f9be395a2002/PVE/Storage/ZFSPoolPlugin.pm#L311
https://github.com/proxmox/pve-storage/blob/74b724a6993fbe859a76d46dd5f2f9be395a2002/PVE/Storage/ZFSPoolPlugin.pm#L168
zfs_create_zvol doesn't specify timeout when calling zfs_request which causes timeout to be 5 seconds. It's not enough in all cases which triggers timeout.

Very useful information - many thanks

Try running the following and compare to the original script. Please report your findings.
bash -c "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/patch-slow-zfs/install.sh)"

It worked perfectly.  I put the server under as much load as I could. Lots of IO on the same zfs pool, lots of CPU use and it sales through.  Specifically I got:
Getting URL for latest Home Assistant disk image...
Downloading disk image...
Extracting disk image...
Creating VM...
Adding serial port and configuring console...
Installing 'kpartx'...
[INFO] Completed Successfully! New VM ID is 120.

Unfortunately that doesn't solve problem since both "pvesm alloc" and "qm importdisk" call "zfs create" with same 5 second timeout.
One can simulate how script behaves with slow zfs disk subsystem by playing with timeout setting on ZFSPoolPlugin.pm and setting it to something really low such as 100ms.
--- /usr/share/perl5/PVE/Storage/ZFSPoolPlugin.pm.orig  2020-07-14 14:58:24.000000000 +0300
+++ /usr/share/perl5/PVE/Storage/ZFSPoolPlugin.pm       2020-08-24 21:46:18.028085724 +0300
@@ -173,7 +173,7 @@
     my $msg = '';
     my $output = sub { $msg .= "$_[0]\n" };

-    $timeout = PVE::RPCEnvironment->is_worker() ? 60*60 : 5 if !$timeout;
+    $timeout = PVE::RPCEnvironment->is_worker() ? 60*60 : 0.1 if !$timeout;

     run_command($cmd, errmsg => "zfs error", outfunc => $output, timeout => $timeout);

hi,
same problem here some time on a zfs install.
i have try : bash -c "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/patch-slow-zfs/install.sh)"
but without success.
Finally, i got a successfull install with a simple sleep 15 between qm create and pvesm alloc

Hello,
same problem on my zfs with running another VM
Did like below:
wget https://github.com/whiskerz007/proxmox_hassos_install/raw/master/install.sh to download script
firstly I got an error: Unable to handle file extension 'zip', so I did:
apt install unzip sed '/*"gz") gunzip -f $FILE;;/a\ \ *"zip") unzip -o $FILE;;' install.sh > temp.sh mv temp.sh install.sh chmod 755 install.sh
Later I added sleep 30  between qm create and pvesm alloc
type nano install.sh and add sleep 30  between qm create and pvesm alloc in "VM Create" section
run script ./install.sh
My output:
root@proxmox:~# bash -c "$(wget -qLO - https://github.com/whiskerz007/proxmox_hassos_install/raw/patch-slow-zfs/install.sh)" [INFO] Using 'local-zfs' for storage location. [INFO] Container ID is 103. Getting URL for latest Home Assistant disk image... Downloading disk image... Extracting disk image... Creating VM... [ERROR] 4@138 Unknown failure occured. root@proxmox:~# nano install.sh root@proxmox:~# ./install.sh [INFO] Using 'local-zfs' for storage location. [INFO] Container ID is 103. Getting URL for latest Home Assistant disk image... Downloading disk image... Extracting disk image... Archive:  haos_ova-6.2.vmdk.zip   inflating: haos_ova-6.2.vmdk        Creating VM... Adding serial port and configuring console... Installing 'kpartx'... [INFO] Completed Successfully! New VM ID is 103.


When cmd_vel is set to zero it tries to go back to the origin in the twist planner

Problem: any new plan by keyboard input in the twist planner will start from origin instead of the current position, though the second step in the plan is the current position.
https://user-images.githubusercontent.com/31764928/111193521-9ebe7980-8590-11eb-8ff1-c29a2b4ee7c3.mp4
Estimated reason: when cmd_vel is set to zero it tries to go back to the origin in the twist planner
Test version: 777d0efaf3f58619a3c9d08ef4f76419a6968c12
To reproduce it:
roslaunch spirit_utils spirit_gazebo.launch paused:=false
roslaunch spirit_utils remote_driver.launch
rostopic pub /control/mode std_msgs/UInt8 "data: 1"
roslaunch spirit_utils execute_plan.launch body_planner:=twist
then input "i",  "stop", and "i" again on the keyboard,
Thanks!

@jcnorby Sorry for the late response. Yes, this works perfectly for me, thanks!


Rye's version of Python leads to clang not found errors

Steps to Reproduce
First, let me thank you for creating rye and improving it so quickly! It is truly a game-changer and makes managing Python projects a breeze. :rocket:
On Ubuntu 22.04:

Install rye with uv as preferred package installer, Python installed and managed by Rye and<EMAIL_ADDRESS>as default toolchain.
Create a project with rye: rye init test
Add pycairo using rye add pycairo.

This issue happens with other Python packages, e.g. spatialgeometry.
Expected Result
pycairo should be correctly installed in the virtual environment created by rye.
Actual Result
I get the following error:
Added pycairo>=1.26.0 as regular dependency
Reusing already existing virtualenv
Generating production lockfile: /home/sdiebolt/Documents/Work/physmed-seminars/test/requirements.lock
Generating dev lockfile: /home/sdiebolt/Documents/Work/physmed-seminars/test/requirements-dev.lock
Installing dependencies
   Built file:///home/sdiebolt/Documents/Work/physmed-seminars/test                                     Built 1 editable in 298ms
Resolved 1 package in 0ms
error: Failed to download distributions
  Caused by: Failed to fetch wheel: pycairo==1.26.0
  Caused by: Failed to build: pycairo==1.26.0
  Caused by: Build backend failed to build wheel through `build_wheel()`:
--- stdout:
running bdist_wheel
running build
running build_py
creating build/lib.linux-x86_64-cpython-312
creating build/lib.linux-x86_64-cpython-312/cairo
copying cairo/__init__.py -> build/lib.linux-x86_64-cpython-312/cairo
copying cairo/__init__.pyi -> build/lib.linux-x86_64-cpython-312/cairo
copying cairo/py.typed -> build/lib.linux-x86_64-cpython-312/cairo
running build_ext
--- stderr:
error: [Errno 2] No such file or directory: 'clang'
---
error: Installation of dependencies failed

Indeed, I don't have clang installed on my machine. However, clang isn't required to build Python packages when using the OS Python.
I've noticed that the version of Python installed by rye is compiled using clang, whereas the OS Python from my machine is compiled using gcc. Indeed, when creating a virtual environment using my OS gcc-compiled Python and installing pycairo, the build is successfully done using gcc. I'm not at all familiar with how Python packages are built however, but I would expect rye to install a Python version that uses the building toolchain available on the host? Or is clang a requirement to use rye?
Version Info
rye 0.27.0
commit: 0.27.0 (43ee4fce0 2024-02-26)
platform: linux (x86_64)
self-python<EMAIL_ADDRESS>symlink support: true
uv enabled: true

Stacktrace
No response

I saw this too and just set CC=gcc and didn't think so much about it (so that should work - does it work for you?), but it sounds like can be improved.

This is basically what #646 should resolve. Work has not started yet.
Closing as duplicate of #646

I saw this too and just set CC=gcc and didn't think so much about it (so that should work - does it work for you?), but it sounds like can be improved.

Yes, installing clang or setting CC=gcc works as expected! However, this is the first time I encountered such an issue, and it isn't very practical especially for CI/CD pipelines that need to be modified to account for this...
Thanks for the quick update @mitsuhiko!

I had the similar error to @sdiebolt when I tried to install gdal package with rye add gdal.
In my case, I also had clang++ not found error, but the following worked so far.
CC=gcc CXX=g++ rye add gdal

I share this as a temporary solution for those who are facing similar problems.

experienced this on many dependencies (especially ones that require cpyparsing). triple-checked that clang and gcc were on the ephemeral shell, which silenced the clang version of the error but didn't work for gcc. amano-takahisa's solution got me through it!


Regressor model and extra features

Why is QM9RegressorDiscrete designed to not use extra_features? Couldn't the spectral features of the noisy graph possibly help with the regressor on predicting target properties of a clean graph?

If there are differentiable functions, they can be included inside the model, and I think that it becomes safe yes.

On 26 Apr 2023, at 11:27, Haoming Li @.***> wrote:
Is this related to whether extra_features_X and extra_features_E are differentiable functions of X and E? Do you consider it safe when they are differentiable functions?
—
Reply to this email directly, view it on GitHub https://github.com/cvignac/DiGress/issues/31#issuecomment-1523088396, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEJOOTUOVH75WKEQGMW2AW3XDDS7ZANCNFSM6AAAAAAWZ3QFCE.
You are receiving this because you commented.


Helper tool needs Admin credentials.

When apps are installed through MDM tools like Jamf, the current update system causes a couple of headaches:
If the helper tool doesn’t have the right permissions, users get a prompt asking for admin credentials. This is disruptive and unnecessary.
Users with standard accounts can uninstall the app if we follow the required permissions , which isn’t ideal in managed environments.
Here’s what would help fix this:
No Update Prompts: Add an option to completely skip the admin credential prompt. If the helper doesn’t have permissions, just don’t run the update.
Better Permissions Setup: Provide a .pkg installer or something similar to properly configure permissions for the updater so it can run without needing admin rights.
Centralized Control: Create a preference file (like a plist) that lets admins disable updates entirely or control how updates and uninstalls work.
These changes would make things much cleaner and easier for both users and admins in managed setups.

Squirrel.Mac is a tool for app developers, and they decide policy for app updates. The current recommendation is that you engage with specific vendors for this issue


Remove packaging of Org.WebRtc.dll

WebRtc is consumed via the webrtc.lib
Fixes #87

Looks good, thanks.
For context this is to fix #87, and MixedReality-WebRTC consumes WebRTC UWP via Org.WebRtc.WrapperGlue.lib, which is the final static library consumed also by Org.WebRtc.dll, the WinRT wrapper. But MixedReality-WebRTC directly links against Org.WebRtc.WrapperGlue.lib, and does not use the WinRT interop layer, so Org.WebRtc.dll is useless for us.

We will also need to respin some NuGet packages from that.


Change participant RU headers

'Количество' in english 'count'. 'How much money to spend on a gift' in russian is more close to 'amount to spend'

Nobody on the team speaks Russian so your contribution is much appreciated. I just deployed your fix to production.


Included link to solution of alphabet-cipher

Awesome, thank you! Sorry for the slow merge - holiday break :)


picks up 425497 packets and doesnt show me any information

How long would this take? i gave it 30 mins. this is my home wifi btw with wpa1

Hi there, zizzania will continue capturing until a WPA handshake occurs on the target network (i.e. for a device to (re)connect).


WINNF.FT.C.HBT.11 Grant Renew test case will continue forever in case all Heartbeats with AUTHORIZED include GrantRenew=True

Executing with SAS Harness v<IP_ADDRESS> the test case WINNF.FT.C.HBT.11 for Grant Renew, if the "grantRenew=true" from the very first Heartbeat Request with "AUTHORIZED" state, and also in all the following Heartbeat Requests, then the SAS Harness continues this test forever always giving in Heartbeat Response a new grantExpireTime with a newer date in the future.
See log below.
This test is finishing properly only if the "grantRenew=true" appears once during the test in only one of the Heartbeat Request messages (not in the first Heartbeat Request).
The situation below occurs when CBSD/DP UUT have their internal Grant Renew timer configured to be 1% of the GrantExpiryTime. So the grantRenew is included in the ALL the Heartbeat Requests with AUTHORIZED state.
If a CBSD/DP UUT have their internal timer configured to 50% of the GrantExpiryTime then test WINNF.FT.C.HBT.11 will be executed successfully.
I am NOT asking for a fix to this issue, just to note it.
2018-06-06T11:14:30.226Z - INFO - heartbeat request from CBRS  : {
"heartbeatRequest": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantId": "414394837",
"grantRenew": false,
"operationState": "GRANTED"
}
]
}
2018-06-06T11:14:30.243Z - INFO - engine sent successfully, the response to CBRS  : {
"heartbeatResponse": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantId": "414394837",
"response": {
"responseCode": 0
},
"transmitExpireTime": "2018-06-06T11:17:50Z"
}
]
}
2018-06-06T11:15:00.325Z - INFO - heartbeat request from CBRS  : {
"heartbeatRequest": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantId": "414394837",
"grantRenew": true,
"operationState": "AUTHORIZED"
}
]
}
2018-06-06T11:15:00.325Z - INFO - Time interval between two heartbeat request messages is: 30.099, limit is: 65.0
2018-06-06T11:15:00.342Z - INFO - grantRenew received in HBT request message
2018-06-06T11:15:00.342Z - INFO - engine sent successfully, the response to CBRS  : {
"heartbeatResponse": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantExpireTime": "2018-06-06T11:21:00Z",
"grantId": "414394837",
"response": {
"responseCode": 0
},
"transmitExpireTime": "2018-06-06T11:18:20Z"
}
]
}
2018-06-06T11:15:30.427Z - INFO - heartbeat request from CBRS  : {
"heartbeatRequest": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantId": "414394837",
"grantRenew": true,
"operationState": "AUTHORIZED"
}
]
}
2018-06-06T11:15:30.427Z - INFO - Time interval between two heartbeat request messages is: 30.102, limit is: 65.0
2018-06-06T11:15:30.444Z - INFO - grantRenew received in HBT request message
2018-06-06T11:15:30.444Z - INFO - engine sent successfully, the response to CBRS  : {
"heartbeatResponse": [
{
"cbsdId": "airspancbrs1Mock-SASDA5A17CD1214",
"grantExpireTime": "2018-06-06T11:21:30Z",
"grantId": "414394837",
"response": {
"responseCode": 0
},
"transmitExpireTime": "2018-06-06T11:18:50Z"
}
]
}

Agree that this test case, as implemented in Test Harness, does not properly handle all the possible cases (i.e. grantRenew in first heartbeat message, grantRenew in all heartbeat messages, etc.).   To handle these cases requires changes to Test Harness code (not just .csv or .json files for test case).
Since this test is listed in WINNF-TS-0122-V1.0.0 as OPTIONAL for certification, this is a lower-priority issue.

Will be handled by CBRS-A 3rd part company.


SITE: Refactor Notifications page to have the same code style with CMO Notifications UI

CMO Notifications UI page enhancements are completed and it's pages and templates could be used as an example for the SITE Notifications. Please use the same html files structure with the same layout and elements hierarchy on order to have a CSS rule matching with the existing theme.
You should not write additional CSS code for this task.
Provided mockup:

I have modified the notification UI for Site. Kindly review the changes and merger it with the main branch.

The html files were 100% copy-pasted from cmo-ssapp. As you know, there are differences between CMO and SITE, but setting  const ACTOR = cscServices.constants.Roles.Site;  in app-config.js is not enough. There are some differences also in html files. You have to identify which elements are required in SITE and copy only those elements. Meanwhile your PR is closed without merge.

I have refactored the notification page along with the history modal for SITE.


Adding support for s390x for changing GetCycleCounterFrequency

Closing #12180 and re-creating the PR based on review comments.

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


Problem undefined symbol in tf_sampling

Hello,
Thank you for releasing your code and for your publication!
I am trying to train semantic_seg_synthia. The script stops just after launching
sh command_train_direct.sh and give the following error:

Do you have any idea about what is going on?
I compiled the Makefiles and obtained circular dependency problems for sampling and grouping operations.
Thank you,
Looking forward to your reply.

It seems like a problem with the compilation of TF Ops. What are your python and Tensorflow versions? The best is to follow the instruction in the README file.

Hi,
I am using Pyhton 3.5 et Tensorflow-gpu 1.9.0 but the difference is that I am using Debian 9 and g++ 6.3.0.
But I finally succeed to compile TF Ops without circular dependency problems!! And thus manage to train the network with removing the following option in Makefiles : -D_GLIBCXX_USE_CXX11_ABI=0
Thank you for your attention,
Iris


Refactoring custom Key Vault data plane client

-moving custom code from root to custom subfolder
-moving generated code to root
-removing custom 'convenience' KeyVaultClient
-adding KeyVaultId class to azure.keyvault namespace
-updating generated/init.py and generated/models/init.py for backwards
compat with warning on import for end users

@RandalliLama @lmazuel @tjprescott These are the changes I was proposing to the KeyVault data plane client.  It should be noted that this is a backwards breaking change, as we discussed offline.  The changes aren't complete yet, as I'm still working on getting all the test code updated and validating all the changes, but I wanted to create the pull request to start getting feedback on the changes.  PTAL

@lmazuel @tjprescott I've updated the PR with changes from your requests.  I've satisfied almost all of them I believe.  PTAL

Codecov Report

Merging #1137 into master will decrease coverage by 0.01%.
The diff coverage is 99.04%.

@@            Coverage Diff             @@
##           master    #1137      +/-   ##
==========================================
- Coverage   55.14%   55.13%   -0.02%
==========================================
  Files        2162     2169       +7
  Lines       60328    60449     +121
==========================================
+ Hits        33269    33329      +60
- Misses      27059    27120      +61

Impacted Files
Coverage Δ

azure-keyvault/azure/keyvault/models/key_item.py
100% <ø> (ø)

...vault/azure/keyvault/models/key_sign_parameters.py
100% <ø> (ø)

...eyvault/azure/keyvault/models/issuer_attributes.py
100% <ø> (ø)

...e-keyvault/azure/keyvault/models/key_attributes.py
100% <ø> (ø)

...ure/keyvault/models/x509_certificate_properties.py
100% <ø> (ø)

...ault/azure/keyvault/models/organization_details.py
100% <ø> (ø)

...e/keyvault/models/certificate_update_parameters.py
100% <ø> (ø)

...ult/azure/keyvault/custom/http_bearer_challenge.py
89.09% <ø> (ø)

...re-keyvault/azure/keyvault/models/secret_bundle.py
100% <ø> (ø)

...e-keyvault/azure/keyvault/models/key_properties.py
100% <ø> (ø)

... and 75 more

Continue to review full report at Codecov.

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


one possible solution for issue #7

This just puts an extra on the intent (cordova_autostart = true), if the app has been started after boot.
Then you can use some plugin to access the intent and determine how the app has been launched.
To do so, first install a plugin to read the intent - e.g.:
cordova plugin add https://github.com/napolitano/cordova-plugin-intent --save
Then use it to access the intent - e.g.:
// It seems, this code can even be run before cordova's deviceready event is triggered:
window.plugins.intent.getCordovaIntent(function(intent) {
   if (intent.extras
        && intent.extras.cordova_autostart) {
     console.log("App has been launched automatically after boot.");
   }
   else {
     console.log("App has been launched manually.");
   }
});

thank you i will try it

@lentschi
I tried it, and it's work fine


[WIP] #47 Increase width for Ask/Bid orders

Increased width for Ask/Bid orders #47

@yivo if we increase more width then the charts will get too small. Let me know your thoughts on this.

@dinesh-skyach Can you, please, try and separate columns so they will not get over each other and tide titles to them? This way user will be able to just scroll to the Bid or Ask side of the panel, tho the scrol option is not so obvious.
Please, note that that maximum number is<PHONE_NUMBER>999999.99999999. Make the panel wider if needed to.

@pkucherenk0 I have made some changes. Please check again.


chore(release): Version bump

Checklist

Issue number for this PR: #995
Docs included?: yes
Test units included?: yes
e2e tests included?: no
In a clean directory, npm install, npm run build, and npm test run successfully? yes

Description
Releasing rc1
Code sample
See #996

Hi. Was this released in rc1?
I'm getting warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>


Typos

Pull Request Template
Checklist

[x] Confirm that run-checks.sh has been executed.

Changes
Adding spell checking as part of CI

Haha I share the feeling, it's good to have something to help us with typos just in case 😅

@antimora  I relate so much 😅


Resize Components

I figured out how to recolor different components (ie. shoe laces) but can I also resize them? Thanks.

<mesh scale={2}>


Replace SI adapters' MessageConverters by Inbound/OutboundMessageMapper

The current MessageConverter implementations do the opposite of what we need in the adapters.
For example, MappingJackson2MessageConverter expects a String or byte[] already representing a JSON object and turns them into the JSON's original type, whereas we want to turn an object into its String or byte[] representation to go on the wire.
InboundMessageMapper and OutboundMessageMapper sound like better interfaces. For example, JsonOutboundMessageMapper contains the following method, which sounds just like what we need:
    public String fromMessage(Message<?> message) throws Exception {
        return this.jsonObjectMapper.toJson(this.shouldExtractPayload ? message.getPayload() : message);
    }

Obsolete.


[SystemRequest/OnSystemRequest] SDL expected behaviour

Hello, @robinmk
Please help to clarify some questions regarding proposal:
0083-Expandable-design-for-proprietary-data-exchange.md
The proposal says:

So, while the 'requestSubType' parameter can be used with any value of 'RequestType' and 'fileType', it should not be used with the above specified combination in an OnSystemRequest notification.

What is SDL expected behaviour in case
HMI uses in the notification params mentioned?

What is SDL expected behaviour in case
HMI sends OnSystemRequest with RequestType='OEM_SPECIFIC' but without 'RequestSubType'?

What is SDL expected behaviour in case:
a) mob app sends SystemRequest
with 'RequestType' of 'PROPRIETARY' && 'fileType' of 'JSON' or a 'RequestType' of 'HTTP' and 'fileType' of 'BINARY'
b) with  'RequestType' of 'PROPRIETARY'  but without 'RequestSubType'

@AByzhynar, please be informed about the question.

@KhrystynaDubovyk ,
Please find my responses below:

What is SDL expected behaviour in case
HMI uses in the notification params mentioned?

SDL should process the message and forward it to the app. SDL should not apply any filter as that would be artificial. For eg: Proxy may in the future change their behavior for the combination of RequestType and FileType under this discussion.
HMI should be prudent to not use the said combination

What is SDL expected behaviour in case HMI sends OnSystemRequest with RequestType='OEM_SPECIFIC' but without 'RequestSubType'?

SDL should process the message and forward it to the app. Again, HMI should be prudent to fill in the proper details before sending the notification.

What is SDL expected behaviour in case:
a) mob app sends SystemRequest with 'RequestType' of 'PROPRIETARY' && 'fileType' of 'JSON' or a 'RequestType' of 'HTTP' and 'fileType' of 'BINARY'

How does SDL process such requests today?

b) mob app sends SystemRequest with  'RequestType' of 'PROPRIETARY'  but without 'RequestSubType'

Did you mean RequestType of OEM_SPECIFIC but without the requestSubType param? If yes then SDL Core should send it to HMI for further processing.

@mrapitis , @JackLivio ,
Any objection to the above?

@robinmk I have some more questions:
 If a newer proxy sends a SystemRequest RPC with the new "requestSubType" parameter to an older system, that RPC would be rejected by the system with a response of INVALID_DATA. If a newer system sends an OnSystemRequest notification with the new "requestSubType" parameter (and not in the above specified combination) to an older proxy, that notification would be forwarded to the application layer.

We don't have such SDL behavior on older systems. SDL will consider such parameter as fake and cuts off from mobile request, forwards the rest to HMI

@GetmanetsIrina Please consider information above and Robin's answer

@AByzhynar ,

We don't have such SDL behavior on older systems. SDL will consider such parameter as fake and cuts off from mobile request, forwards the rest request parameters to HMI.

I meant when the systemRequest is of type OEM_SPECIFIC. I understand that if requestSubType was sent with an existing requestType to an older head unit, then the requestSubType field would be cut off.

@robinmk How do you propose to add such check to older system? Within this proposal it is impossible as new RequestType is added. Such behavior should be added to SDL core where no such RequestType exists in API.  Please advise.

@robinmk, please confirm the assumption.
According to description for requestSubType in OnAppPermissionConsent
   If the app sends a requestSubType which is not specified in this list, then that request should be rejected.
   An empty array signifies that any value of requestSubType is allowed for this app.
   If this parameter is omitted, then a request with any value of requestSubType is now allowed for this app```
policy dictionary must be expanded with requestSubType array of strings in app_policies groups.

@robinmk Also please consider that allowed/disallowed RPC parameters check is made globally according to the following rules:
If "parameters" field exist in PT section of incoming RPC but empty
all  params considered as DISALLOWED

and this proposal breaks (inverts) this logic. So it will be very strange if RPC parameter requestSubType will be checked according common logic and application allowed/disallowed requestSubType will be checked according to inverted logic.
@robinmk Please reply ASAP as implementation is blocked by correct understanding of expected SDL behavior.

@GetmanetsIrina

Please clarify 'If this parameter is omitted, then a request with any value of requestSubType is now(or not?) allowed for this app'.

This should be "Not". Sorry for the confusion.

According to current rules in preloaded_pt in case of parameter has empty array in pt any value of requestSubType is not allowed. In case parameter is omitted any value of requestSubType is allowed.

We are confused by this. Could you please provide clarity on how the system behaves today with the 'requestType' param?

@AByzhynar

@robinmk How do you propose to add such check to older system? Within this proposal it is impossible as new RequestType is added. Such behavior already exists in SDL core where no such RequestType exists in API. So it will be the same behavior for any RequestType which is absent in API. Is this behavior expected?

When an older system receives a SystemRequest with the RequestType=OEM_SPECIFIC and with requestSubType parameter, then such requests shall be rejected by the system since a RequestType of OEM_SPECIFIC is not supported by such systems.
If the older system receives a SystemRequest with a RequestType= any of the accepted values (like NAVIGATION or CLIMATE etc) and with requestSubType parameter, then such requests shall be accepted by the system and the requestSubType parameter will be ignored.
This is my understanding. Hope we are in-sync here.

@AByzhynar
What would happen on an older system where OEM_SPECIFIC is not defined in HMI_API/MOBILE_API but is received as part of PTU (to be sent as part of OnAppRegistered)?

@robinmk , we have investigated the SDL behavior with omitted parameter and empty array in RequestType.
SDL allows any types in both cases, this is SDL defect.
According to requirements in case of requestType is omitted SDL must not allow all values,
in case of parameter has empty array in pt any value of requestType is allowed.

@mrapitis ,

According to requirements in case of requestType is omitted SDL must not allow all values,
in case of parameter has empty array in pt any value of requestType is allowed.

I think this is the behavior that we need. Do you agree?
@GetmanetsIrina

What would happen on an older system where OEM_SPECIFIC is not defined in HMI_API/MOBILE_API but is received as part of PTU (to be sent as part of OnAppRegistered)?

I am still not clear on answer to the above question.

@robinmk , SDL omits OEM_SPECIFIC value, and responds with INVALID_DATA resultCode to SystemRequest(OEM_SPECIFIC) and does not resends OnSystemRequest(OEM_SPECIFIC) to mobile app.

Perfect. Ok, Thank you!


Proposal: Make OpenAI api key configurable at runtime

Feature request
I'd like the option to pass the OpenAI api key to the openai.Openai client at runtime.

ConfigurableField (docs) already supports passing this variable to the client, and in fact allows me to change any config except openai_api_key.
I suspect the original intent may have been to make it configurable: if openai_api_key is defined with a configurable_field there is no warning or Exception raised.

I can submit a PR; I'm already using the proposed change in my code.
Motivation
There are many scenarios where I'd like to change the API key depending on the task I'm performing. This is especially true when using Langchain in the context of a microservice or API with a high volume of requests: building "cloned" modules or re-initializing modules is impractical.
Your contribution
I can submit a PR in short order but am soliciting input here first.
The proposed change is to the validation function found here (source):
Proposed change at the bottom
    @root_validator()
    def validate_environment(cls, values: Dict) -> Dict:
        """Validate that api key and python package exists in environment."""
        if values["n"] < 1:
            raise ValueError("n must be at least 1.")
        if values["n"] > 1 and values["streaming"]:
            raise ValueError("n must be 1 when streaming.")

        values["openai_api_key"] = get_from_dict_or_env(
            values, "openai_api_key", "OPENAI_API_KEY"
        )
        # Check OPENAI_ORGANIZATION for backwards compatibility.
        values["openai_organization"] = (
            values["openai_organization"]
            or os.getenv("OPENAI_ORG_ID")
            or os.getenv("OPENAI_ORGANIZATION")
        )
        values["openai_api_base"] = values["openai_api_base"] or os.getenv(
            "OPENAI_API_BASE"
        )
        values["openai_proxy"] = get_from_dict_or_env(
            values,
            "openai_proxy",
            "OPENAI_PROXY",
            default="",
        )

        client_params = {
            "api_key": values["openai_api_key"],
            "organization": values["openai_organization"],
            "base_url": values["openai_api_base"],
            "timeout": values["request_timeout"],
            "max_retries": values["max_retries"],
            "default_headers": values["default_headers"],
            "default_query": values["default_query"],
            "http_client": values["http_client"],
        }

        ##### PROPOSAL: REMOVE MARKED IF STATEMENTS FROM THIS CODE #####
        if not values.get("client"): # <--- REMOVE
            values["client"] = openai.OpenAI(**client_params).chat.completions
        if not values.get("async_client"): # <--- REMOVE
            values["async_client"] = openai.AsyncOpenAI(
                **client_params
            ).chat.completions
        ##### END PROPOSAL #####

This would require no change in user behavior (putting OPENAI_API_KEY in the environment variables still works).
I'm not sure why these IF checks are here. If they're necessary to avoid re-defining openai.OpenAI, I'd suggest the benefit of a dynamic api key outweighs the cost of re-instantiation (and could be solved regardless by a quick check on whether the client_params changed).
Thanks for any input!

Looks like while I was writing this, the feature issue category was removed, leading to a blank submission when I finally hit send. I can't delete it, so I just edited the body. Happy to re-open elsewhere if needed. Thanks!

@cvansteenburg what's the bottleneck with re-initializing the model?

@cvansteenburg I hit this issue as well: did you end up submitting a PR?  I couldn't see it...

@maspotts I got slammed at work and had to go dark for a bit. Will return shortly with a more thoughtful response :).

Thanks would be great: I'm currently reaching in and modifying chain.llm_chain.llm.openai_api_key, or index/embedding_function.openai_api_key, or whatever, just before I call a method that uses it, but that seems needlessly error-prone and not at all future proof!

@cvansteenburg @maspotts did you have any luck submitting a PR? If not, can you show me the code modification that needs to be done to get this to work? Currently stuck here

In the end, I just modified my code so I create a list of OpenAI or ChatOpenAI instances at the beginning and then cycle through them at the point of use.

What if you want to pass in the API key from the client

I think the easiest is to create a new OpenAI or ChatOpenAI instance using the specific API key when you receive it.

🪄✨POOF✨ I have reappeared!
It's been awhile since I worked this issue but I recall the pain-point being that there isn't a simple or performant way to achieve the desired API key change. Re-initializing the model wasn't performant (to very belatedly answer your question @eyurtsev), and I found the other solutions here, while valid, to be neither performant or simple. This may reflect my own inexperience.
As stated in the original proposal, the source code suggests the original intent may have been to make openapi_api_key configurable: if openai_api_key is defined with a configurable_field there is no warning or Exception raised.
My proposed PR fixed the issue for me (at the time, anyway - I haven't checked for recent changes to source) by removing the IF checks that prevent re-defining openai.OpenAI and thus allowing the class to dynamically use openai_api_key when passed via configurable_field.
There may be an important reason the original author decided to prevent redefinition of the openai.OpenAI class. (safety related? Perhaps redefinition creates problems?) Git blame shows @efriis, a more experienced dev and certainly more familiar with the code base, so I propose a change only with trepidation ;).

@cvansteenburg another approach, which does not require changing the underlying class:
class CustomChatOpenAI(ChatOpenAI):
    @root_validator(pre=True)
    def validate_environment(cls, values: Dict) -> Dict:
        values["async_client"] = values["client"] = None
        return values

did they fix this ?
this code is from their server examples but still not working
https://github.com/langchain-ai/langserve/blob/main/examples/configurable_chain/server.py
dynamic_auth_model = ChatOpenAI(openai_api_key="placeholder").configurable_fields(
openai_api_key=ConfigurableField(
id="openai_api_key",
name="OpenAI API Key",
description=("API Key for OpenAI interactions"),
),
)

Has anyone tried this with the new Langchain V3 version? I'd love to know if there have been any changes to how API key configuration works with the update. Thanks in advance for sharing any experiences or insights!


[AMBARI-23800] Using the proper concatenation delimiter when using the 'append' variable replacement function in a Kerberized cluster

What changes were proposed in this pull request?
We use the append variable replacement function var/lib/ambari-server/resources/stacks/HDP/2.5/services/HIVE/kerberos.json for hadoop.proxyuser.HTTP.hosts. The problem was that we used the wrong concatenation delimiter - \, instead of , - and this caused an issue when there are more than one host is listed here.
There are two places where I needed to fix this:

overwriting the old value upon upgrade to 2.7
fixing this in HIVE's kerberos.json

How was this patch tested?
Modified the appropriate JUnit test; latest unit tests in ambari-server:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 27:01 min
[INFO] Finished at: 2018-05-10T15:05:05+02:00
[INFO] Final Memory: 161M/758M
[INFO] ------------------------------------------------------------------------

In addition to this I executed the following E2E test steps:

installed Ambari <IP_ADDRESS>-3
installed a cluster (HDFS, Hive, YARN/MR2, ZK, Pig, Slider, Tez, AMS)
Kerberized the cluster
prepared to upgrade to <IP_ADDRESS>-437
just before the ambari-server upgrade step I implemented my changes in the code; built the new JAR and replaced it in my test environment; I overwrote the kerberos.json in question too
upgraded to <IP_ADDRESS> (w/o any issue) and started the server and agents
logged in to Ambari UI and checked HDFS's config; so far so good
added mulitple hosts in hadoop.proxyuser.HTTP.hosts
regenerated the keytabs and checked HDFS's configuration again: the proper value has been shown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2221/
Test PASSed.


Log airflow webserver to stderr

Also bind to <IP_ADDRESS>

Debug is to log access. The host name is not needed.
Actually can close the pr because it's not needed at the moment.
On Mon, 16 Jan 2017, 20:11 Chris Sng<EMAIL_ADDRESS>wrote:

@chrissng commented on this pull request.
In entrypoint.sh
https://github.com/datagovsg/airflow-pipeline/pull/28#pullrequestreview-16792115
:

@@ -49,7 +49,7 @@ if [ "$1" = 'afp-scheduler' ]; then
elif [ "$1" = 'afp-webserver' ]; then
echo "Starting webserver"
python "${AIRFLOW_HOME}"/setup_auth.

py

exec gosu "${USER}" airflow webserver

exec gosu "${USER}" airflow webserver --hostname=<IP_ADDRESS> --log-file=- --debug

Hmm don't quite understand why these flags are needed
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/airflow-pipeline/pull/28#pullrequestreview-16792115,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8APQA2lBKx6bSTRWqDHaKCvGL1NiThks5rS16KgaJpZM4LkOsj
.

Actually this is not needed for now. I'll close this for now.


State not being written to Azure rm

Hi, Thanks a lot for writing this plugin. I am facing an issue where state not being written to Azure Blob storage. I was able to create a Azure blob storage via "Terraform Init" successfully. Also, i've included
"terraform {
backend "azurerm"{}
}"
snippet in my main.tf.  When i see the console output in Azure Pipelines for init task, I see the init is is called with proper "backend-config" options. Also i see "Initializing backend..." but couldnt see anything related to it after that.
Here's the part of the init's console output for your reference.
 [command]C:\hostedtoolcache\windows\terraform\0.12.0\x64\terraform.exe init -backend-config=storage_account_name=cdmstfremotestate -backend-config=container_name=cdmstfremotestateblob -backend-config=key=terraform.tfstate -backend-config=resource_group_name=rg_terraform_remotestate -backend-config=arm_subscription_id=f8b9d8ba-ca38-4e13-8969-408d28bc3c67 -backend-config=arm_tenant_id=*** -backend-config=arm_client_id=*** -backend-config=arm_client_secret=*** 2019-05-29T23:58:50.9632471Z  2019-05-29T23:58:50.9634380Z Initializing the backend... 2019-05-29T23:58:50.9647661Z  2019-05-29T23:58:50.9648220Z Initializing provider plugins... 2019-05-29T23:58:50.9648281Z - Checking for available provider plugins... 2019-05-29T23:58:51.1283219Z - Downloading plugin for provider "azurerm" (terraform-providers/azurerm) 1.29.0... 2019-05-29T23:58:52.4868982Z  2019-05-29T23:58:52.4910303Z The following providers do not have any version constraints in configuration, 2019-05-29T23:58:52.4910423Z so the latest version was installed. 2019-05-29T23:58:52.4911304Z  2019-05-29T23:58:52.4911413Z To prevent automatic upgrades to new major versions that may contain breaking 2019-05-29T23:58:52.4911533Z changes, it is recommended to add version = "..." constraints to the 2019-05-29T23:58:52.4911622Z corresponding provider blocks in configuration, with the constraint strings 2019-05-29T23:58:52.4911710Z suggested below. 2019-05-29T23:58:52.4911750Z  2019-05-29T23:58:52.4911805Z * provider.azurerm: version = "~> 1.29" 2019-05-29T23:58:52.4911863Z  2019-05-29T23:58:52.4911926Z Terraform has been successfully initialized! 2019-05-29T23:58:52.4912009Z  2019-05-29T23:58:52.4912095Z You may now begin working with Terraform. Try running "terraform plan" to see 2019-05-29T23:58:52.4912175Z any changes that are required for your infrastructure. All Terraform commands 2019-05-29T23:58:52.4912258Z should now work. 2019-05-29T23:58:52.4912295Z  2019-05-29T23:58:52.4912358Z If you ever set or change modules or backend configuration for Terraform, 2019-05-29T23:58:52.4912461Z rerun this command to reinitialize your working directory. If you forget, other 2019-05-29T23:58:52.4912537Z commands will detect it and remind you to do so if necessary. 2019-05-29T23:58:52.5070668Z ##[section]Finishing: terraform init
Appreciate youer help/inputs on this. Thank you!

I was able to resolve it and its working as expected now. There was a small misconfiguration in my pipeline script which was overwriting my terraform files. Thanks again for the plugin.

@prongcs thanks for using the extension! I am glad to hear you were able to resolve the issue.


Replace moment

moment is 66 kB<EMAIL_ADDRESS>use any other lighter alternatives

dayjs (2.8 kB)
date-fns (16.9 kB)
luxon (20 kB)

source: https://github.com/you-dont-need/You-Dont-Need-Momentjs

I want to do this but am kind of blocked at the moment due to not knowing what to do with the security rating metric, i.e. if I want to keep it or not and what to replace the UI with, but I agree with you. Because the code for calculating the security rating is pretty garbage at the moment, it’s a non-sensical metric that only takes into account the user’s password and when they changed it last so that’ll need some work.

What's preventing you from replacing moment?

Will not replace moment in the foreseeable future. I like working with moment and I think others do as well.

But it's huge


Inheritance (Java OOP's Concepts)

Description 📜
I have made-up a documentation on Inheritance in Java. It is one of the important topic in java. I have made this with my hardwork and I added everything that you will understand it with a ease.
Fixes #2101
Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Backend Dev (Java)

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Congratulations your pr is merged.

Congratulations your pr is merged.

Thank you

Thank you
On Mon, Sep 27, 2021 at 3:11 PM Satyam175 @.***> wrote:

Congratulations your pr is merged.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/2964#issuecomment-927699636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOHU2K44J5PMQSGWYNBKPTUEA35FANCNFSM5EY6YD7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Application insights working without config

I'm building a small library that use Application Insights.
Is there a possibility Application Insights working without ApplicationInsights.config file?
I tried remove and add the modules manually at constructor, but it didn't work.

Hi @jonnovaretti,
Please try these instructions and let us know if you run into any problems:
https://docs.microsoft.com/azure/application-insights/application-insights-console

hi @MS-TimothyMothra,
I already have tried this sample. It works when I add the ApplicationInsights.config file, but when I remove the file, it stop working.

Can you share your code here of how you're configuring ApplicationInsights?
remove anything sensitive (ex: ikey).

Sure! No problem...
Just adicional information. When I exclude the config file, it doesn't work, but when I just remove the content of file, it works.
My code:
`public Telemetry(ILogger logger)
{
TelemetryConfiguration configuration;
ApplicationInsightsConfig applicationInsightsConfig;
        configuration = TelemetryConfiguration.Active;
        configuration.ApplicationIdProvider = new ApplicationInsightsApplicationIdProvider();

        configuration.DisableTelemetry = false;

        applicationInsightsConfig = ReaderConfig.ReadConfig();

        if (!applicationInsightsConfig.InstrumentationKey.Equals(EMPTY_TOKE))
            configuration.InstrumentationKey = applicationInsightsConfig.InstrumentationKey;

#if DEBUG
configuration.InstrumentationKey = string.Empty;
#endif
configuration.TelemetryInitializers.Add(new HttpDependenciesParsingTelemetryInitializer());
configuration.TelemetryInitializers.Add(new AzureRoleEnvironmentTelemetryInitializer());
configuration.TelemetryInitializers.Add(new AzureWebAppRoleEnvironmentTelemetryInitializer());
configuration.TelemetryInitializers.Add(new BuildInfoConfigComponentVersionTelemetryInitializer());
        configuration.TelemetryProcessorChainBuilder.Use((next) => { return new AutocollectedMetricsExtractor(next); })
                                                    .Use((next) => { return new QuickPulseTelemetryProcessor(next); })
                                                    .UseAdaptiveSampling()
                                                    .Build();

        configuration.TelemetryChannel = new ServerTelemetryChannel();

        var telemetryLocal = new DeveloperModeWithDebuggerAttachedTelemetryModule();
        telemetryLocal.Initialize(configuration);

        TelemetryModules.Instance.Modules.Add(telemetryLocal);

        client = new TelemetryClient(configuration);
        this.logger = logger;
    }`

@MS-TimothyMothra, I'm debugging Microsoft.ApplicationInsights and I've see different behavior between delete file config and remove all content from file.
In this part of code:
// Load configuration from the specified configuration
               if (!string.IsNullOrEmpty(serializedConfiguration))
               {
                   try
                   {
                       XDocument xml = XDocument.Parse(serializedConfiguration);
                       LoadFromXml(configuration, modules, xml);
                   }
                   catch (XmlException exp)
                   {
                       CoreEventSource.Log.ConfigurationFileCouldNotBeParsedError(exp.Message);
                   }
               }

When there is empty file,

This line looks suspicious to me:
applicationInsightsConfig = ReaderConfig.ReadConfig();
and
configuration.InstrumentationKey = applicationInsightsConfig.InstrumentationKey;
Can you confirm for me that this is not the applicationinsights.config that you're removing from your project?
I would expect that you store your Ikey in the app.config for your project.

If that doesn't work, try testing with a simpler configuration and only adding extra as you need them.
I know for a fact that this works in a sample console app:
TelemetryConfiguration tcConfig = TelemetryConfiguration.Active;
tcConfig.InstrumentationKey = "myikey";
TelemetryClient tc = new TelemetryClient(tcConfig);

I'm closing this ticket as out of date.
If you're still having issues, please reopen.


GCP

gcloud compute instances create mac --provisioning-model spot --scopes https://www.googleapis.com/auth/devstorage.full_control --boot-disk-size 200 --enable-nested-virtualization --machine-type n1-standard-8 --image-project ubuntu-os-cloud --image-family ubuntu-minimal-2304-amd64 --boot-disk-type pd-balanced

sudo apt install apt-file qemu-system-x86 qemu-utils dmg2img git htop screen nano

cat /volumes/untitled/var/log/install.log

Stumped in reading the log outside in normal text. Input appreciated
sudo modprobe nbd max_part=8
sudo qemu-nbd --connect=/dev/nbd0 macOS.qcow2
sudo fdisk /dev/nbd0 -l
sudo fsapfsinfo /dev/nbd0p2
sudo fsapfsmount -f 4 /dev/nbd0p2 /mnt/mac

gcloud compute instances create mac --provisioning-model spot --scopes https://www.googleapis.com/auth/devstorage.full_control --boot-disk-size 200 --enable-nested-virtualization --machine-type n1-standard-8 --image-project ubuntu-os-cloud --image-family ubuntu-minimal-2304-amd64 --boot-disk-type pd-balanced

sudo apt install apt-file qemu-system-x86 qemu-utils dmg2img git htop screen nano

Same thing happened to me. Try getting an offline installer for ventura like an iso.

I don't immediately understand the procedure. This should work. I am again stumped by getting the log
dev@mac:~$ sudo tree -a hfs                   hfs                                           ├── .HFS+ Private Directory Data\x0d          ├── .fseventsd                                │   └── fseventsd-uuid                        ├── .journal                                  ├── .journal_info_block                       ├── macOS Install Data                        │   ├── InstallAssistant.pkg.integrityDataV1  │   ├── InstallAssistant.pkg.partial          │   ├── Locked Files                          │   │   ├── .disk_label                       │   │   ├── .disk_label.contentDetails        │   │   ├── .disk_label_2x                    │   │   └── Boot Files                        │   └── index.sproduct                        └── ␀␀␀␀HFS+ Private Data                                                                   7 directories, 9 files                        dev@mac:~$

I think the way the newest ventura installer grabs files from online is different then before. Try Monterey first

Stumped in reading the log outside in normal text. Input appreciated

sudo modprobe nbd max_part=8

sudo qemu-nbd --connect=/dev/nbd0 macOS.qcow2

sudo fdisk /dev/nbd0 -l

sudo fsapfsinfo /dev/nbd0p2

sudo fsapfsmount -f 1 /dev/nbd0p2 /mnt/mac

That corresponds to

ls /volumes/untitled\ 1

In the recovery boot ??? !  ??
/Volumes/untitled is the actual content but I don't know how it magically conjures that mount. Any insight appreciated
It doesn't have many commands to help

findmnt /volumes/untitled

Command not found

This might be the opencore disk.

If you want to open the installer log press "window" in the top bar and select "installer log"

Maybe I can be so snyde as to ask for a Ventura image. Pre installed. There was one on vagrant but alas, parallels only :-[
vagrant box add logantbond/ventura
vagrant box list
It was a pre installed parallels image :-\

I do have offline installers but they're 15 gigs so I don't know how to share em.

Just search up "ventura iso" on Google and it'll be there. But do not share links here please. I don't wanna get this repo taken down.

So there is NO OFFICIAL OFFLINE INSTALLER? How are they created? What about InstallAssistant.pkg. There is no way to mount that in qemu?

@john-peterson apple is dumb. for a 100% official installer, u need a mac. If you know any online service that can host a 15gb file, I can make an offline installer for you, since I have a mac I can use.

Just for the record, trying to install ventura using setupArch.sh throws the same error to me.
OSISHelperProxy blessUpdateBundleAtPath:mutableProductPath:enclosingMountPoint:]_block_invoke_2: Error Domain=NSCocoaErrorDomain Code=260 "The file “bootbase.efi” couldn’t be opened because there is no such file." UserInfo={NSFilePath=/Volumes/Mac SSD/macOS Install Data/UpdateBundle/AssetData/./boot/Firmware/System/Library/CoreServices/bootbase.efi, NSUnderlyingError=0x600002df8930 {Error Domain=NSPOSIXErrorDomain Code=2 "No such file or directory"}}
I took this from other thread but the error is exaclty the same. Monterey installs ok.
So to update Ventura you need to get an ISO, you won't be able to do it via script right?

Sadly it seems the answer for that is yes.


Added support for account signing keys.

When account signing keys change the validity of the client JWT and imports requiring tokens need to be checked as well as it is possible for the signing key used to sign the user or import token to have been removed from the source account.

Coverage decreased (-0.06%) to 90.391% when pulling 0cbcbff35e6d1457429a4e086680c2bc12e5bbb7 on account-signingkeys into f17c5142dd3d77add2f580a99f8ca7149fd53526 on master.

See inline.
Also why does jwt repo have a Makefile?

Because make test is easier to type during the dev cycle than:
	gofmt -s -w *.go
	goimports -w *.go
	go vet ./...
	go test -v
	go test -v --race
	staticcheck ./...


Accesibility Change One: Show Acessible Info

This change does two things

If a widget is accessible with a title, use that title for the melatonin sidebar rather than clasname and preferentially over juce name. (If it has both an accessible title and juce name and they differ the juce name is shown additionally)

collect and display basic accessible info in the inspector panel. these are title, type, value, and handler class.

I'd appreciate a review and feedback here
the one thing I couldn't figure out is how to make the sub section in a properties panel indent properly. It works and folds but doesn't look right. Heres' what I mean

Your thoughts welcome!

Awesome!
Properties have been flat so far, which is why nested styles aren't happy out of the box. If we wanted to go with nested, I guess we'd want to match the styles in the main navigation tree and ensure that toggling the section was "remembered" across
Part of me wonders if "accessibility" should be a collapsable top level section (ala preview/color/properties)... I'll have to check the branch out to weigh in on that, so I'll get back to you!

OK! So I made the changes you suggested

Name is no longer concatenated on accessible name and is in the properties panel
Accessibility moves to its own panel

You are right that the properties panel being last is 'special' and I didn't figure out quite how, but I solved that problem by having the accessibility panel come before that. LOL. Here's a screenshot from surge.
Happy to make any other changes. This is immediately become useful for me though with even just this change.

Love it!!!

I solved that problem by having the accessibility panel come before that

Heh, I figured :)
I'll find some time to check out the branch and merge tomorrow. I was hoping the tests would run too, that always trips me up on GitHub...

You probably need to approve me as a first time pr contributor - there’s usually a button but you may also need to do a review first
but we have pretty good pr actions in the clap juce extensions and clap wrapper projects if that helps too?

Yeah, just strange as I have Actions set to Require approval for first-time contributors who are new to GitHub (meaning it shouldn't restrict contributors with long standing GitHub accounts).
This keeps being a problem on my repos, so I opened a ticket with support, I don't see any way for me to run the actions here:

This is perfect! Nice work! Thanks for being willing to move stuff top level, I think it works great.
Apparently I have 0 components with accessibility titles on my app! I guess it's time I get to work. I'll have to learn about roles and actions too somewhere — almost all of my components are hand rolled, I avoid JUCE's widgets where possible (except for text editors)

On actions your 'on' seems odd
on:
  workflow_dispatch:
  push:

vs in clap-wrapper we have
on:
  pull_request:
    branches:
      - main
      - next


Identify populations X kilometers from nearest health facility

We need to help a government identify the populations most in need of infrastructure and planning for ebola response.
Given the following data sets:

a list of population points (50k points)
a list of health facilities (1.4k points)
shapefile for DRC at admin level 2,
roads shapefile for DRC

...create a Jupyter notebook that serves all population points x kilometers from nearest health facility.
The final project should be an open-source, standalone GitHub repository, with a README and at least some test coverage. The readme should address (1) what this repo is or does, and (2) how to install and use it.

Hello. I have been working recently on something similar. You can take a look at my repository here. In a nutshell, I created a map that shows the population (and % of a population) in a district (second sub-national level) that falls within X kilometers isoline of the nearest health facility (or specialist). Isolines can be computed using the following methods:

Openrouteservice API - up to 2500 isolines a day (100 per minute, max 120 km isolines)
OSMNX and networkx libraries to create a graph that is used to determine the isolines. The graph can be built from data downloaded using OSMNX or from a shapefile after conversion to OSM format (building the graph can take up to several hours)

If there is still a need for help, then I would like to contribute, either by making any necessary changes to my repository or by creating something similar from scratch that will meet the expectations.  I am open to any suggestions.
Sincerely,
Radek


Update Android package version for schema v1.5

android version in properties.gradle is already set to 2.8.0, no more work is necessary


AwesomeOpenSource

We are creating a github org for all awesome open source repos. I am working with a few individuals who are also managing their own awesome repos, and I think to gain more traction we need to work together. What are your thoughts about collectively moving your awesome repos to https://github.com/AwesomeOpenSource
The idea is that visibility for these repos would be increased due to more traffic to the org. What are your thoughts? Let me know if there is anything you'd like to discuss!

i think it's not relevant to us migrate this repo into some org, because i really don't see any advantage in this. Can you describe me some reasons for do that?


Solana explorer

Clone the Ethereum one, build the map_block_full and map_transaction_full with a simple filter, outputting the same model as what the chain offers (so import the solana spkg).
Goal is to be able to use something like:
substreams gui -e mainnet.eth.streamingfast.io:443 https://storage.googleapis.com/substreams-registry/spkg/ethereum-explorer-v0.1.1.spkg map_block_full --start-block 1397588 --stop-block +1

just for inspecting stuff.
Even better if one can filter a given transaction, or a given thing, and have a reduced block, so there's less confusion between transactions when debugging.
This way, we can also add that to streamingfast.io in the widget.. would show multichain support directly on the front page, and make the data model discoverable.
https://github.com/streamingfast/substreams-explorers

This was done by @colindickson


Ensure that we only reject the '-i key' part of command execution

Ensure that we only reject the '-i key' part of command execution and not parts that contain '-i' in larger strings.
We ran into the case where if node['jenkins']['master']['prefix'] contains '-i' the -s (server address) array element was removed.
For example:
node['jenkins']['master']['prefix'] = '/this-is-my-jenkins'
generates the command
java, -jar /.../jenkins-cli.jar -s http://<IP_ADDRESS>:8080/this-is-my-jenkins/ -i /.../jenkins-key install-plugin /.../ace-editor-....plugin -name ace-editor
This fails because we have not yet installed and configured the ldap plugin.
The retry then tries
java, -jar /.../jenkins-cli.jar install-plugin /.../ace-editor-....plugin -name ace-editor
because /-i/ matches against 'this-is-my-key' and removes the '-s ...' element.
Obvious fix.

Merging under the obvious fix policy


Make sidebar link display as block.

Thank you @overtrue for contributions, but can you provide a screenshot of the diff. you made with this PR?

Yes, This changes just make a wide clickable area:
before:

after:

The red area is clickable.

Cool, Thanks 👻

@saleem-hadad 😆


2.5 fixes

Several misc fixes to the 2.5 release.

Fix exception message when disabling a listener

Fix inability to download files from agents

Remove overly verbose messages when agents begin their staging process (only output to debug now)

Fix event types for agent checkin, tasking, results, and rename

@mr64bit Thanks, really appreciate it!


(HOLD - WIP) ⚠ Replace kube-rbac-proxy with controller-runtime metrics authentication/authorization

This commit removes the use of the kube-rbac-proxy image and replaces it with metrics authentication/authorization provided by controller-runtime. The kube-rbac-proxy image is deprecated and will no longer be maintained, which introduces risks to production environments. For more details, see: kubernetes-sigs/kubebuilder#3907
Key changes:

Updated to configure metrics server options with secure authentication/authorization using controller-runtime filters.
Added support for disabling HTTP/2 by default to mitigate vulnerabilities (e.g., HTTP/2 Stream Cancellation CVE).
Removed the kube-rbac-proxy container from deployment configurations.

/hold

I confirmed that by reverting the changes described above, I can successfully run the e2e and the upgrade-e2e locally. A similar "undo" might be needed in operator-controller.

/hold
We need add a test to get the metrics before the change
Then, we can get it merged after the test be merged

Rebased with the tests to ensure backwords compatibility: https://github.com/operator-framework/catalogd/pull/483

Hi @joelanford
Can we uphold and move forward with this one?


Add EqualDistributionWithAffinityWorkerSelectStrategy which balance w…

Add EqualDistributionWithAffinityWorkerSelectStrategy which balance work load within affinity workers.
In my use case, I need bind a datasource to some worker hosts and I want tasks distribute in balance.

👍  Can you please update the docs too

@niketh docs updated

👍

@JackyWoo have you signed the CLA - http://druid.io/community/cla.html ?

@pjain1 The CLA is signed.
I have tested the feature in my cluster and it works well.


Fix windows_task configuration

![(shipit)]
(http://qph.is.quoracdn.net/main-qimg-0bf1d3de309a89d9ed70ae91f6271a7f)

@juliandunn, I made the changes requested. Hoping to merge this in for the next release.
It fixes the windows_task recipe.

LGTM.


[ARCTIC-265][Site] Benchmark notebook

Why are the changes needed?
Add docs of benchmark
Link: #265
Brief change log
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before making a pull request

Documentation

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

@shidayang Could you please rename the PR's title to match pattern [ARCTIC-{issue number}][{Moduel}]{description}

@shidayang You may want to replace Link #265  to Resolve #265 to help github link this PR to issue.


NIFI-2866: The Initial Max Value of QueryDatabaseTable won't be case

I merged the commit logs.
Thank you for your review. @mattyb149
Original issue: #1101
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

+1 LGTM, ran the unit tests and tried in NiFi with mixed case identifiers. Thank you for the contribution! Merging to master


Styled-components and microdata

Microdata attributes doesn't appears at the DOM node.
Person - it's a styled component.
<Person itemscope itemtype="http://schema.org/Person" />
The attributes itemscope and itemtype not present in rendered DOM.

That's probably not StyledComponent's fault. In React, they should be spelled as itemScope and itemType (note the camelCase)

Oh, thx!


Custom dock with minimal size appears bigger than necessary

Godot version:

3.2.4 beta2
Issue description:

The dock has minimum horizontal size. It looks like the panel is resized to this size, but it's also added to the final size, so the left panel is wider than necessary. But when you manually shrink the width to minimal, when the dock is switched out the additional size is still subtracted resulting in tiny panel.
Minimal reproduction project:

ReproductionProject.zip

Ugh I just checked on master and it doesn't happen there.
Not sure if it's worth keeping this open. It affects my plugin, but it's not very critical bug.

Seems like it happens to the import dock too:

Seems to be still valid on master, see #64273

Reopening as #64800 was reverted due to regressions.


Migration of pipeline owners

The following pipeline file owners need to be updated
T3000 workflows
.github/workflows/t3000*.yaml @tapspatel
TG workflows
.github/workflows/tg-*.yaml @tapspatel
TGG workflows
.github/workflows/tgg*.yaml @tapspatel
tests/scripts/t3000/ @tapspatel
tests/scripts/tg/ @tapspatel
tests/scripts/tgg/ @tapspatel
tests/scripts/run_performance.sh @tt-rkim @tapspatel
scripts/install @tapspatel (remove)

PR: https://github.com/tenstorrent/tt-metal/pull/9310

Merged: b081f5ebb234b46f6ea574df722def135bd65b3f