Key press not registered (only key release) for some keys

For certain keys an event is only registered when the key is released, not when it is pressed. If the key is held down for a bit and then released, a bunch of key events are registered.
Some keys that do this are KeyEnter KeyTab KeyCtrlA... (I think most of the control keys), but keys like KeySpace KeyLeft KeyBackspace2 don't have this behavior.
Setting VTIME to 0 solves the problem, but then the program's CPU usage goes way up.
I am using iTerm2 on Mac OS X 10.11.2.
EDIT: this also seems to happen on Linux (Ubuntu 15.10)

Here is a short example script that illustrates the issue:
package main

import "github.com/gdamore/tcell"
import "os"

func main() {
	s, _ := tcell.NewScreen()
	s.Init()

	defStyle := tcell.StyleDefault.
		Background(tcell.ColorDefault).
		Foreground(tcell.ColorDefault)

	s.SetStyle(defStyle)

	var enterNum int
	var spaceNum int
	s.Clear()
	for {
		event := s.PollEvent()

		switch e := event.(type) {
		case *tcell.EventKey:
			if e.Key() == tcell.KeySpace {
				s.SetContent(5, spaceNum, 'S', nil, defStyle)
				spaceNum++
			} else if e.Key() == tcell.KeyEnter {
				s.SetContent(10, enterNum, 'E', nil, defStyle)
				enterNum++
			} else if e.Key() == tcell.KeyEscape {
				s.Fini()
				os.Exit(0)
			}
		}
		s.Show()
	}
}

Holding down Space will make a bunch of 'S' characters appear as you hold the key down, but 'E' characters for holding down Enter only appear after the key has been released.

I’ll try to look at this later today.
On Fri, Mar 18, 2016 at 7:20 AM, Zachary Yedidia<EMAIL_ADDRESS>wrote:

Here is a short example script that illustrates the issue:
package main
import "github.com/gdamore/tcell"import "os"
func main() {
s, _ := tcell.NewScreen()
s.Init()
defStyle := tcell.StyleDefault.
    Background(tcell.ColorDefault).
    Foreground(tcell.ColorDefault)

s.SetStyle(defStyle)

var enterNum int
var spaceNum int
s.Clear()
for {
    event := s.PollEvent()

    switch e := event.(type) {
    case *tcell.EventKey:
        if e.Key() == tcell.KeySpace {
            s.SetContent(5, spaceNum, 'S', nil, defStyle)
            spaceNum++
        } else if e.Key() == tcell.KeyEnter {
            s.SetContent(10, enterNum, 'E', nil, defStyle)
            enterNum++
        } else if e.Key() == tcell.KeyEscape {
            s.Fini()
            os.Exit(0)
        }
    }
    s.Show()
}

}
Holding down Space will make a bunch of 'S' characters appear as you hold
the key down, but 'E' characters for holding down Enter only appear after
the key has been released.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/gdamore/tcell/issues/85#issuecomment-198380375

What is your terminal type?  Can you send me the value of echo $TERM?  Also, did you regenerate the terminal database or are you using the one I shipped?

I think I've figured this out.  I need to add the various codes for keys to the t.keys array; their absence means they are getting handled as unknown/undecoded sequences.  Which is definitely not optimal.
Probably also I should deal better with that case.

Solution is to add the control characters (being careful not to overwrite the ones that are ambiguous, i.e. all those starting with ESC) to the prepared keys.


Lock husky version & uses latest instead of *

locks husky version (the new slot in package.json for hooks doesn't work with env vars)
changes * from package.json to latest for clarification

Pull Request Test Coverage Report for Build 1914

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

Totals

Change from base Build 1907:
0.0%

Covered Lines:
1023

Relevant Lines:
1070

💛  - Coveralls


Duplicate videos on the React Native list

Hello people
I have a list of videos that comes from the API, the IDs are different, but the players appear the same video.

List

card YouTube

I didn't find the reason :-(

Multiple  instances on Android
The YouTube API for Android is a singleton. What it means is that unlike the iOS implementation, no two players can be mounted and play a video at the same time. If you have two scenes that happen to live together, or come one after the other (such as when navigating to a new scene), The new  Will take the focus of the singleton and play the video, but after being unmounted, the older mounted  will not be able to take the role back, and will need to be re-mounted.
:-(

Any solution for this?


XIOS error handling

          just a thought, not necessarily for this PR. Would it be sensible to throw an exception if any of the getter objects are still `NULL`? Ill be honest... I didn't really check what happens if `xios` getters fail. If `xios` throws the error anyway then it may not be necessary.

Originally posted by @TomMelt in https://github.com/nextsimhub/nextsimdg/pull/555#discussion_r1607849575

Closed by #576.


Minted LandNFT shows no information

Currently, minted lands show no information such as description and other metadata.
However, the minted landnfts should show information about the land. We can take from the existing land datastcrure setup.
https://sepolia.voyager.online/contract/0x07a03d379480d83b968b804c39e3717e11fb041f2ec8e908db78f25e7f1d0a1b?portfolio=erc20

Mind if I try this one? I write cairo smart contracts with test.

I am a Smart Contract developer as well as a full stack de4veloper and would like to work on this issue.
I will modify the existing LandNFT contract to include metadata storage for minted lands.
On the backend, I will create an API endpoint to fetch land details from the blockchain.
For the frontend, I will develop a React component to display land information clearly.
Integrate web3.js to connect the frontend with the blockchain, enabling real-time updates.
Finally, optimize data retrieval using caching mechanisms to improve performance

I'm a fullstack engineer with years of experience and my ETA for this is 48 hours.

Can I attempt this issue?

Could I take a shot at this?

I can handle this task confidently. I have successfully contributed to this project before,  I'm fully capable of taking on this responsibility.

Would love to tackle this!

I’d like to resolve this.

gm, Suhas here, having worked in the last od, where  I’ve worked on integrating on-chain mechanics into games and have contributed to optimizing user experience with tools like SPV wallets and Web3 integrations with expertise in writing and testing smart contracts in Cairo.
To resolve this issue, I will updaete the LandNFT contract to link minted tokens with metadata from the existing land data structure. I will ensure metadata is retrievable both onchain and via an apifor seamlesss frontend integration. Additionally, I’ll enhance the UI to display land details dynamically while optimizing performance through caching mechanisms.
ETA: 24 hours.

can i work on this, i would love to contribute here and make my first od contribution.

Can I attempt this issue?  Am a web 2 frontend developer, a solidity smart contract developer/auditor and a cairo smart contract developer. Am a graduate of Starknet Cairo Bootcamp. I have successfully done tasks assigned to me in OnlyDust. If assigned this i would complete it within 4 days.


UnicodeDecodeError parsing PKG-INFO

On Python 3.4.2 in some environments (maybe when LANG=C), setuptools_scm raises a UnicodeDecodeError when it encounters non-ascii in the PKG-INFO file:
wget https://pypi.python.org/packages/source/i/irc/irc-12.2.tar.gz#md5=efbb4d9a6bb92b24ab6109e04716ef38 -O - | tar xz
cd irc-12.2
python
>>> import setuptools_scm
>>> setuptools_scm.get_version()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/app/env/lib/python3.4/site-packages/setuptools_scm-1.5.2-py3.4.egg/setuptools_scm/__init__.py", line 43, in get_version
  File "/app/env/lib/python3.4/site-packages/setuptools_scm-1.5.2-py3.4.egg/setuptools_scm/__init__.py", line 21, in version_from_scm
  File "/app/env/lib/python3.4/site-packages/setuptools_scm-1.5.2-py3.4.egg/setuptools_scm/hacks.py", line 8, in parse_pkginfo
  File "/app/env/lib/python3.4/site-packages/setuptools_scm-1.5.2-py3.4.egg/setuptools_scm/utils.py", line 64, in data_from_mime
  File "/usr/lib/python3.4/encodings/ascii.py", line 26, in decode
    return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 8768: ordinal not in range(128)

This causes the version inference to fail and all versions of the irc package to install as 0.0.0.

Bitbucket: https://bitbucket.org/pypa/setuptools_scm/issue/28
Originally reported by: Jason R. Coombs
Originally created at: 2015-06-11T04:12:40.903

Indeed, in the environment where this is failing, LANG is not set.

Original comment by: Jason R. Coombs

Setuptools forces UTF-8 for the encoding of package metadata. Use it also for reading. Fixes #28.
→ <>

Original comment by: Jason R. Coombs


🌟 [EPIC] Integration Figma and Storybook

🌟 EPIC

For the project flow we want to give the designer the possibility to define Components for the Frontend with Figma.
Figma should then be linked with the Story Book of the Frontend.

closing for now - pipeline itself is set


[Go Live][May 17]

Push content to live

Docs Build status updates of commit 5e4ae8b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/samples/app-templates.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


C# to Golang

Is it possible to do it in reverse to convert C# to Golang

Unfortunately that task would be much, much harder:
https://go2cs.net/#c-to-go


Playlist and Title display features

Hi John
I'm using your MediaElement js library since a while now and I think it's a really strong help for getting HTML5 audio and video to everybody.
I have used the player for many music artists website (realized with my company, 1nterval): Roberto Alagna, The Puppini Sisters, the Jazz Masters, Juliette Gréco, Rolando Villazon, and others to come !
As you can see on those site, a playlist is displayed when you move the mouse over the note. You can also browser the artist discography and load another playlist from an album tracklist panel like here.
The website listed here are using a custom javascript code to implement this playlist feature, but I have now taken the time to reshape this code into a plugin for MediaElement. You will find the playlist feature itself in src/js/mep-feature-playlist.js and the album name and track title display feature in src/js/mep-feature-title.js. I have also created a demo file if you want to have a quick look at it. I have put more mp3 files from Air Review, so that's why the commit is so heavy, sorry for that.
The playlist management mechanism is separated from the playlist display panel (triggered by the 'playlist' string in the feature array) so that it can be used with a custom playlist display. I have only dealt with the audio player for the moment, but I think that few is missing to adapt it to the video one.
Please tell me what you think about my code. I would be very pleased if you plan to integrate it in a future release.

@Fab1en Thanks for the work on this PR; however, the changes were set on the build files, and they must be implemented in the src folder; if it's still your desire to get this PR reviewed, please submit a new one with the changes on the src folder.

@ron666 I did implement the changes in the src folder (see commit 5ec2e128), but then I build the code and committed the result (commit f253f70).

Yes please nothing in the build area can be changed


Support for assuming IAM role with web identity token

Terragrunt currently only supports assume role for IAM roles, With the introduction of OIDC providers, we can assume IAM role with web identity token, which is officially supported by https://github.com/aws-actions/configure-aws-credentials
With iam_role in terragrunt, can we also have support to assume IAM role with web identity token with the token or the token file passed as an additional input ?
If this can be supported, we can directly run teragrunt on github actions without a need for using configure-aws-credentials action. This helps in assuming maintaining multiple roles in multiple modules

Yes please

Yes! I am having to use a workaround right now. I use OIDC for my CICD provider and this would be huge!

Yes! I am having to use a workaround right now. I use OIDC for my CICD provider and this would be huge!

Would you mind sharing your workaround?

This is a big thing for me.
We have a complex setup of accounts and being able to utilise OIDC will greatly reduce the overhead needed to maintaining cross account relationships.

I'd like to share our workaround. We have this in our root terragrunt.hcl:
# Generate an AWS provider block
generate "provider" {
  path      = "provider.tf"
  if_exists = "overwrite_terragrunt"
  contents  = <<EOF
variable "oidc_web_identity_token" {
  type        = string
  description = "OIDC Web Identity Token used by AWS Provider."
  default     = null
}

provider "aws" {
  region = "${local.region}"

  assume_role_with_web_identity {
    role_arn           = "arn:aws:iam::${var.account_id}:role/${var.role_name}"
    web_identity_token = var.oidc_web_identity_token
  }
}
EOF
}

Before we trigger the pipeline we write the OIDC token we obtained to terraform.auto.tfvars which gets read by Terraform and thus fills in the oidc_web_identity_token var.
echo "oidc_web_identity_token = \"${OIDC_TOKEN}\"" > terraform.auto.tfvars

This is not native Terragrunt like iam_role is but it gets the job done. We rolled this out last week and it works perfectly for our use-case.

@botagar @syphernl I ended up actually finding this in the Terraform documentation. Dynamic Provider Credentials.
This allowed me to setup terraform cloud as a custom Identity Provider in AWS, then when I ran my terragrunt configuration with the terraform cloud (using terraform cloud API token) I didn't have to do any setup of the AWS credentials. The IAM role used for authentication was setup in the project variables.
Check out the proof of concept I worked on. Terragrunt TFC integration

If this can be supported, we can directly run teragrunt on github actions without a need for using configure-aws-credentials action.

Has anyone actually even managed to get Terragrunt to assume the OIDC role when using aws-actions/configure-aws-credentials?
I have not had any success and the same seems to be true in this issue (dating from back in 2021 no less). If you've managed to get it to work it would be great to know how you did it.
So, even having Terragrunt work with OIDC via configure-aws-credentials without any workarounds would be good for me. OIDC is now pretty much the recommended way to grant temporary auth with your provider in github actions workflows so this really ought to work cleanly with Terragrunt.
Is there any way we promote this issue in priority? I'm not a Go developer myself but if anyone has any insight in how to patch this maybe we can make an effort work on it.
Thanks for all the effort on the project, it's very much appreciated.

I couldn't get TG to use TF remote state using OIDC.
I created a small but realistic test stack and set the provider to be TF 1.6-beta3 but TG never seemed to try access the state in the correct account. I verified that the generated backend block was all correct and was able to see that the role I has setup was assumed, but then TG dropped the ball in the last stretch by trying to access the state bucket in our Agent Hosting account (which doesn't exist). All things I confirmed through cloudwatch and cloudtrail.
Same stack without TG, i was able to use TF remote state via Assume Web Role AND deploy resources across accounts via Assume Web Role.
I used assume_role_with_web_identity in the provider block, and the same in the back-end block.
See TF 1.6 RC1 here
As for passing the token into that block, in our pipeline we developed a small utility (node app) to get the OIDC token and turned it into a shared GH action.
In the Job, we save the token to a file and have that file path set in the assume_role_with_web_identity block.
As for what we're doing right now for the rest of our stuff already deeply in TG, we're breaking out the jobs into their respective target AWS accounts and calling aws-actions/configure-aws-credentials in each Job for each account role.
We then target the appropriate directories for each account using the --terragrunt-include-dir parameter.
This is a major pain though, and defeats one of the main reasons we introduced TG into our tech stack in the first place.
The promise of being able to just shotgun all our infra out there in a single command and have the whole dependency graph taken care of for us was quite the selling point, but now it's another layer ontop of what we were doing back when we were pure TF.
If OIDC support isn't going to be even considered in TG for a while, we might need to consider going back to pure TF.
This would be very unfortunate as we just wrapped up moving everything into TG, but well, the universe loves pulling pranks.
I don't see much discussion happening here on this feature tbh, so maybe we're a tiny minority who want to use OIDC.
OIDC or Assume Web Identity aren't anywhere in the Roadmap so I don't think we should hold our breath for this feature.
I too am not a GO dev and I just haven't had the time to be able to sit down with the TG codebase to understand what's going on and how it does all it's role assumptions.
TG as a tool so far has been great. But no OIDC is a bit of a show stopper.

Thanks for the reply @botagar. I can't say I disagree with any of what you say in general, it's definitely a bit of a weird oversight that OIDC doesn't work yet transparently and I can see how it might be nearly a deal breaker and very frustrating for anyone who has sunk a lot of time and effort into Terragrunt and then discovers OIDC is not a first-class citizen.
Yesterday though after a bit of a slog I did manage to get Terragrunt running with OIDC using the same kind of method as described above with  the assume_role_with_web_identity block in the provider. I had to make some alterations to get it working though so I'll post a summary of the code for posterity (the source is not OSS yet but it will be eventually) in case anyone else needs OIDC in their workflow.
# terragrunt.hcl

generate "provider" {
  path      = "provider.tf"
  if_exists = "overwrite_terragrunt"
  contents  = <<EOF
# Workarounds for terragrunt's (current) inability to handle OIDC role chaining.
variable "oidc_session_name" {
  description = "The OIDC session name."
  type        = string
  default     = null
}

variable "oidc_web_identity_token" {
  description = "The OIDC session token."
  type        = string
  default     = null
}

provider "aws" {
  allowed_account_ids = ["${var.aws_account}"]
  region = "${var.aws_region}"

  dynamic "assume_role_with_web_identity" {
    for_each = var.oidc_web_identity_token == null ? {} : { oidc_enabled = true }

    content {
      role_arn                = "arn:aws:iam::${var.aws_account}:role/${var.gha_oidc_role_name}"
      session_name            = var.oidc_session_name
      web_identity_token      = var.oidc_web_identity_token
    }
  }
EOF
}

Here we use variables with default null values and a dynamic block for assume_role_with_web_identity. I had to come up with this extra check using dynamic otherwise I would end up with a validation error where Terraform complains that either web_identity_token or web_identity_token_file have to be set in the assume_role_with_web_identity block. By using dynamic we can get around this in a context-sensitive way while still having the variable set to null which is clean.
Then as described by the OP, in the pipeline what we will do is write oidc_session_name and oidc_web_identity_token to terraform.auto.tfvars and then in the CI/CD context those values will be picked up by Terraform and the assume_role_with_web_identity block will be used. So make sure you have a terraform.auto.tfvars file somewhere in your working tree without those values in it that you can use for this purpose. If we are just running terraform locally then it will fallback to whatever credentials you have configured (no autovars).  Obviously you'll need to configure your OIDC provider and role so that the ARN in role_arn matches whatever you inject via the pipeline.
In your Github actions (for example) we need to do the rest of the dynamic injection and call Terragrunt. (This was adapted from a larger workflow file so forgive any typos or mistakes.)
# deploy.yml

name: Provision
concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true
permissions:
  contents: write
  id-token: write  # OIDC permission.
env:
  ACCOUNT: 12345678
  REGION: us-east-1
  ROLE: gha-oidc-role.  # Make sure this matches your OIDC role name in terraform.
  ROLE_SESSION: GithubActionsProvision
  TF_ROOT: ./tf  # Wherever the root of your terraform configuration is located in your repo.

jobs:
  provision-oidc:
    name: 'Provision OIDC'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4

      - id: set-oidc-role
        run: |
          echo "ROLE=arn:aws:iam::${{ env.ACCOUNT }}:role/${{ env.ROLE }}" >> $GITHUB_OUTPUT

      - uses: aws-actions/configure-aws-credentials@v4
        id: oidc-credentials
        with:
          aws-region: ${{ env.REGION }}
          role-to-assume: ${{ steps.set-oidc-role.outputs.ROLE }}
          role-session-name: ${{ env.ROLE_SESSION }}

      - id: write-oidc-vars
        env:
          AUTO_VARS: terraform.auto.tfvars.  # The name of your autovars file.
        run: |
          vars=${{ env.TF_ROOT }}/${{ env.AUTO_VARS }}
          token=${{ steps.oidc-credentials.outputs.aws-session-token }}
          session=${{ env.ROLE_SESSION }}
          echo "oidc_session_name = \"${session}\"" >> ${vars}
          echo "oidc_web_identity_token = \"${token}\"" >> ${vars}

      - uses: hashicorp/setup-terraform@v2
        with:
          terraform_wrapper: false

      - id: setup-terragrunt
        uses<EMAIL_ADDRESS>        with:
          token: ${{ secrets.GITHUB_TOKEN }}

      - id: tf-plan-oidc
        env:
           STACK: staging  # Set this to whatever configuration directory you need.
        run: terragrunt run-all plan --terragrunt-working-dir ${{ env.TF_ROOT }}/${{ env.STACK }}

      - id: tf-apply-oidc
        env:
           STACK: staging  # Set this to whatever configuration directory you need.
        run: terragrunt run-all apply  --terragrunt-working-dir ${{ env.TF_ROOT }}/${{ env.STACK }}

Obviously this is quite a lot of heavy lifting to have to do, but it works and Terragrunt is provisioning via OIDC in the pipeline.
I wouldn't try and provision an entirely new stack in this way. I suppose it might be possible to do so, but I build my stack out manually first before activating automation. The workflow above is intended to be used only for updates during a CI/CD process on an existing set of resources. Ensure the OIDC role has the correct permissions to access the dynamodb tables and state buckets that already exist and with something like the above you should be away to the races.

Thanks for sharing that @jhrr !

I hadn't actually thought of using the dynamic blocks that way.
If first class OIDC support were at the very least acknowledged on the roadmap, we could probably stick with the interim solution we have now.
Maybe it's time to learn a little GO too... :sweat_smile:

FYI: I just sent out a PR that implements this


Add missing css3 columns to _bourbon.scss

Was getting "Undefined mixin 'columns'". Seemed to be missing from _bourbon.scss.

I haven't found the time to test the new columns mixins yet. I've merged your pull request, so if you run into any issues, please report back. Thanks @msonnabaum.


CS Project file contains many absolute and relative paths

The CS Project file for LethalSDK contains a lot of absolute and relative paths, assuming a file structure that isn't guaranteed. Perhaps using environment variables for this might make it easier for others to self-compile the project should the need arise

It was done by someone else for lethal expansion
I don't know how I'm sorry
But you can PR if you know how and if you think that may help peoples


Add WKT definition parsing into proj4 parameters

Add the ability to parse Well-Known Text into proj4 parameters HERE for the following implementations:

OpenGIS® 01-009 Implementation Specification: Coordinate Transformation Services Revision 1.0

http://portal.opengeospatial.org/files/?artifact_id=999

OGC® 12-063r5 Geographic information - Well-known text representation of coordinate reference systems

http://docs.opengeospatial.org/is/12-063r5/12-063r5.html

See ngageoint/GeoPackage#4

Implemented in projections-ios and made a dependency in release 5.0.0.


Normalize Transform should cater for channel >1

A quick look at the code shows if you have your --nChannel > 1, then this line would throw an error trans_valid = transforms.Compose([transforms.Normalize(mean=[0.5],std=[0.5])])  because its only expecting 1 channel and not 3.
I'm I missing something?

Thanks for raising the issue. Fix is in this commit.


dasd_get_host_access_count libdasd/dasd_sys.c fails when referencing a loopback mounted device

When using "fdasd -f /dev/loop0" the dasd_get_host_access_count routine in libdasd/dasd_sys.c tries to access the file /sys/bus/ccw/devices/"busid"/host_access_count. For DASD images mounted via a loopback device, that fails.
This results in the following message being output:
WARNING:
Disk /dev/loop0 is online on operating system instances in 263849296 different LPARs.
Ensure that the disk is not being used by a system outside your LPAR.
Note: Your installation might include z/VM systems that are configured to
automatically vary on disks, regardless of whether they are subsequently used.
util_sys_get_dev_addr in libutil/util_sys.c gets a -1 return code from the call to readlink, and then itself returns -1 to dasd_get_host_access_count. The following if statement
if (!util_sys_get_dev_addr(device, busid))
return 0;
fails, so instead of returning a 0, execution continues to the next statements:
path = util_path_sysfs("bus/ccw/devices/%s/host_access_count", busid);
util_file_read_l(&value, 10, path);
free(path);
    return value;

Since the variable value is not initialized, it returns a random non-zero number to the caller.
Looking further, if I test with a real DASD volume, the dasd_get_host_access_count call to util_sys_get_dev_addr gets a zero return code, and so returns a value of 0 to the caller. That doesn't seem right, since it never tries to read the /sys/bus/ccw/devices/"busid"/host_access_count file, so it's not returning an actual value from there. If I force the execution of:
path = util_path_sysfs("bus/ccw/devices/%s/host_access_count", busid);
util_file_read_l(&value, 10, path);
Then the call to util_file_readl happens, and in that routine, the call to file_gets(buf, sizeof(buf), path)) fails so it returns -1.
Out of curiosity, I tried to manually look at /sys/bus/ccw/devices/0.0.0100/host_access_count on my system. Everything I tried resulted in an "Operation not supported" error message.
To sum up, it looks like there's two problems here:

The if (!util_sys_get_dev_addr(device, busid)) is "backward."
The sysfs host_access_count file doesn't seem to be readable, at least on any of my systems.

Hi Mark,
the main issue is being fixed internally already. You're right, this check
https://github.com/ibm-s390-tools/s390-tools/blob/885ff0a03f5561acb76e7201ddb2c9de30a5e123/libdasd/dasd_sys.c#L217
has to be turned over. I tested the loop back device problem with our fix and it works.
About the "Operaion not supported", I don't see this issue on our systems.
If you have details on how to reproduce it and you think it is s390-tools related, please
open another issue for it. If you think it might rather be Kernel related, use the usual
channels to report the problem. Thanks!
This issue will be closed once the commit goes upstream. Thanks for reporting!

Fixed with https://github.com/ibm-s390-tools/s390-tools/commit/353403824b5112c254f6788bad5edbbb19b96683


Luis tool does not support Get/Set publish settings

Tool
Name: LUIS
Version: 2.6.2
OS: os version
Describe the bug
Luis tool does not support Get/Set publishsettings, which declared here:
https://github.com/microsoft/botbuilder-tools/blob/278155eb71a58c8367b912593feacc60387d5395/packages/LUIS/src/operations/apps.ts
Line 439
References should be added here also:
https://github.com/microsoft/botbuilder-tools/blob/c74b7e210e06d166a042ad04c2a1bbab69e1261a/packages/LUIS/bin/operations.json
otherwise, the following api won't work with luis tool:
https://westus.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/5ae32174d5b81c02cc64c6d3

We are rewriting the LUIS API tool in our R7 release. This will be done as part of the BF CLI.
This work is being done against the following spec:
https://github.com/microsoft/botframework-cli/blob/master/specs/LUIS-CLI-IN-BF.md
INCLUDED in that spec is the get publish settings. Suggestions for additional scenarios should be created as issues in that repo, referencing that spec.

This is coming soon in the new version of the tool, if you feel there is an issue with the spec please open an issue in the respecting repository.


Initialize consensus data

This change moves the consensus fetching calls into closures
I'll apply some of the changes in #755 on top of this PR.
Note: the chainstate subsystem call in block_maker is only used to read in data for PoW mining, and not the mining itself. This way, PoW mining will stays within the blockprod subsystem

I've rebased #755 onto this PR, so I'm going to close this PR and move the latest changes needed there.


Several fixes around the H3 and QUICHE build.

This PR tackles a few issues:
1 - ATS wasn't built with quiche. (TS_HAS_QUICHE=0)
Fixing the rpath  in the  quiche.m4 solves this issue as the autoconf now successfully detects quiche, so it does define the macro we need.
2 - Build with the latest quiche.
This was a problem because quiche introduced this change which was incompatible with our code.
This is a short term solution, the proper fix is to update our code to the latest API This is tracked here
Environment config finished. Running AuTest...
Running Test qlog_quiche:..... Passed
Running Test quic_no_activity_timeout:... Passed
Running Test quic_poll_timeout:.. Passed

Generating Report: --------------
Total of 3 test
  Unknown: 0
  Exception: 0
  Failed: 0
  Warning: 0
  Skipped: 0
  Passed: 3

3 - There were few issues when building h3 related to the new libtscore archive.

diff --git a/iocore/net/quic/Makefile.am b/iocore/net/quic/Makefile.am
index aeb98a391..fce167482 100644
--- a/iocore/net/quic/Makefile.am
+++ b/iocore/net/quic/Makefile.am
@@ -62,6 +62,8 @@ test_MTHashTable_LDADD = \
        $(top_builddir)/iocore/eventsystem/libinkevent.a \
        $(top_builddir)/src/tscore/libtscore.a \
        $(top_builddir)/src/tscpp/util/libtscpputil.la \
+  $(LIBPCRE) \
+  $(LIBCAP) \
        @SWOC_LIBS@ @OPENSSL_LIBS@

 test_MTHashTable_SOURCES = test_MTHashTable.cc

With this added change the make -j check passes.
diff --git a/iocore/net/quic/Makefile.am b/iocore/net/quic/Makefile.am
index aeb98a391..fce167482 100644
--- a/iocore/net/quic/Makefile.am
+++ b/iocore/net/quic/Makefile.am
@@ -62,6 +62,8 @@ test_MTHashTable_LDADD = \
        $(top_builddir)/iocore/eventsystem/libinkevent.a \
        $(top_builddir)/src/tscore/libtscore.a \
        $(top_builddir)/src/tscpp/util/libtscpputil.la \
+  $(LIBPCRE) \
+  $(LIBCAP) \
        @SWOC_LIBS@ @OPENSSL_LIBS@

 test_MTHashTable_SOURCES = test_MTHashTable.cc

Thanks Bryan, just added that + other stuffs.


Create bower.json

Hi, Angular directive uses eCharts as dependency and without bower.json it does not inject the echarts.js file. Please review and merge.
Thanks

Hi, I don't think that adding a  bower.json would be a good idea, considering that the bower projects recommends using other tools like yarn and webpack.

...psst! While Bower is maintained, we recommend yarn and webpack for new front-end projects!

While bower may still be maintained for the years to come, it will not be as active as before, and will lack of features already present yarn like a lock file.
https://github.com/bower/bower.github.io/pull/268


Disable individual dates

I'm trying to create an event calendar that allows people to select dates. Yet once someone selects a date, I need to disable it for others.
Thus how do I pass a disabled prop to individual dates?

Great! That would be very useful!

close by #49

Hi @aboveyunhai,
I noticed you've added a disabledDates field to disable specific days.
However, it seems this field doesn't support disabling weekends on the calendar dynamically. To achieve this, consider implementing a similar approach to react-select, using a callback function like filterDate to dynamically disable days at runtime.

Hi @aboveyunhai, I noticed you've added a disabledDates field to disable specific days. However, it seems this field doesn't support disabling weekends on the calendar dynamically. To achieve this, consider implementing a similar approach to react-datepicker, using a callback function like filterDate to dynamically disable days at runtime.
Would you be interested in receiving a pull request for this feature?

Feel free to create the PR! just wonder why the disabledDates doesn't work in your case
doesn't the following work?

const disabledDates = useMemo(() => {
   // your filter logic here, which is the same as the filterDate function
  // put dynamic dates into the Set
   return new Set();
}, [])

disabledDates = {disabledDates}

Hi @aboveyunhai,
Background: I'm working on a timelog system where weekends should be disabled for selection on the current calendar panel.
To dynamically disable weekends on the calendar:
It seems there's no way to directly access the current date range within the component. However, I can still pass all weekend days of the current month to the disabledDates prop. While this approach might not be the most efficient, it should achieve the desired result.
I'm considering adding a callback function type to the disabledDate prop. This function would accept a date object and return true if the date should be disabled or false if it should be enabled.


[eslint] Enforce new rules

Focus on the rules that can be automaticaly fixed and
can lead to possible errors.
@shaurya947 Please review and merge

@shaurya947 I'm using the autofix option and I'm trying to follow the more used coding style.
I think that it can be merged safely.
I have added some property labeled as # Wishlist, one day.
That I think we should enforce.


Argv

By the way, if you want a free T-shirt, you can mark this PR (and 3 other PRs you create this month on open-source repos) with hacktoberfest-accepted and sign up for https://hacktoberfest.com


Support websockets

https://github.com/spring-projects/spring-framework/tree/master/spring-webflux/src/test/java/org/springframework/web/reactive/socket

http://nginx.org/en/docs/http/websocket.html
https://www.nginx.com/blog/websocket-nginx/

Is support for web-socket support slated for 1.0? or will this be implemented later? The largest problem we currently have with Zuul 1 is the lack of socket support.

It is planned for the initial release in December.

Closed via 924c484622cb849e54084189ecc754018fa3c102


How to support multi token in env?

When the constructor is initialized, the possibility of adding a flag for different tokens should be included

Hey!! This is just a simple port of ozq/moodle-client, i've didn't work too much on this package.
Unfortunately i can't help you


feature request: type keyword + tab to select site to search

love this plugin! one of my favorite features from raycast
it would be even better if it mimicked chrome's omnibar UX and allowed you to select the site to search with e.g. "chatgpt  " instead of "?? chatgpt "
not sure if this conflicts with powertoys run's global keywords, but saving a few extra keystrokes every time i use this extension would be great.

Hi @bradyab , thank you for your love!
I think this reply can help you: https://github.com/Daydreamer-riri/PowerToys-Run-WebSearchShortcut/issues/1#issuecomment-2014895380

amazing, works perfectly!
On Wed, Jun 12, 2024 at 11:56 AM Riri @.***> wrote:

Hi @bradyab https://github.com/bradyab , thank you for your love!
I think this reply can help you: #1 (comment)
https://github.com/Daydreamer-riri/PowerToys-Run-WebSearchShortcut/issues/1#issuecomment-2014895380
—
Reply to this email directly, view it on GitHub
https://github.com/Daydreamer-riri/PowerToys-Run-WebSearchShortcut/issues/22#issuecomment-2163396630,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4RNKRI7YPFHRAYC5273SLZHBVRFAVCNFSM6AAAAABJFR2XDSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRTGM4TMNRTGA
.
You are receiving this because you were mentioned.Message ID:
<Daydreamer-riri/PowerToys-Run-WebSearchShortcut/issues/22/2163396630@
github.com>

Hi @bradyab , thank you for your love!
I think this reply can help you: #1 (comment)

hi, this link doesn't work, but i have the same question

Hi @areographe, you can juse enable this option.

Hi @areographe, you can juse enable this option.

great, thank you!


fit_resonator: Naming conventions for resonator data files and folders

Change fit_resonator outputs to the following naming convention:
Folder: FitType_Date
Fit params file: FitType_DataFileName.csv
Fit image file: FitType_DataFileName.png

Done.


Moving frameworks to utils folder

The go files below are moved to the utils folder.

db.go
logging.go

Have created a file called constants, which are global , keeping it here unless someone finds a better solution on where to keep rather than importing the constants on around the place.
Done as part of the issue : https://github.com/pivotal-gss/gpmt2/issues/8

@faisaltheparttimecoder  see my recent issue #10 - I think logging should be globally initialized, then set in cmd. That should be possible. If it can't be done with go-logging, it was working with logrus and we can toy with the formatting separately.
Setting logging globally should allow us to just import the logger package and then call log with the write to filesystem and stdout handled easily.
We might need to take good time to organize this, but the root structure should allow the rest of the project to be more maintianable.

@Tylarb Yes that can be done, when you push your ssh utils just move that out to the root folder. or you want I can push that one now ? what you think ?

@faisaltheparttimecoder  With go-logging, can we just call logging.debug in any sub-package as expected? That was one reason I started with Logrus, because you can import it as log and all configuration will be inherited.
If so, we can merge this then I can adjust. If not, let's hold off on this PR, get logrus working from package main, then continue from there.

The way go-logging works is

you setup a logger name.
then from any package ( on any file ) just call that logger like

log = logging.MustGetLogger(utils.ToolName)

and then any logging works as normal with go-logging format


Proper scrolling

Right now when you scroll to the bottom of tofi it doesn't give you the next entry... it goes back to the top, regardless of how many entries there should be.

Awesome, and with that (assuming this works perfectly I haven't tested it yet I'm not home) tofi is the perfect launcher for me. Thank you so much for the hard work!

No problem! Thanks for all the suggestions, it helps motivation when you know someone's actually getting some use out of what you're doing :)


[Feature request] Using origyear in a custom citation key.

I've sent my Hume and Wenar entries, which I expand on below, as Report ID: B9GKHH82
BBT's custom citation key feature generally works well
At the moment I have a custom citation key ...
Zotero > Tools > Preferences > Better Bibtex > Citation Keys > Citation Key Format: [Auth:condense,_:lower]_[year]_[veryshorttitle:lower]
... and that works well enough.
So from a Zotero Entry of ...
Author: Wenar, Leif
Date: 2015-09-09
Title: Rights
Publication: The Stanford Encyclopedia of Philosophy

... BBT will use my custom citation key format to produce a Citation Key "wenar_2015_rights". Generally that's perfect.
My desire for origyear handling
However, I also have some Zotero entries that include BBT custom fields, and the specific custom field origdate, a field which biblatex supports (and some third party biblatex style's support).  For example in Zotero I might have ...
Title: An Enquiry Concerning the Principles of Morals
Author: Hume, David
Book Title: Enquiries Concerning Human Understanding and Concerning the Principles of Morals
Date: 1975
Extra: bibtex[origdate=1751]

... which BBT will reliably export, complete with my custom citation key, as ...
@incollection{hume_1975_enquiry,
  author = {Hume, David},
  date = {1975},
  origdate = {1751},
  title = {An {{Enquiry Concerning}} the {{Principles}} of {{Morals}}},

However, for entries with an origdate (in addition to a date) I'd rather have a citation key that uses the origyear. In this example to produce "hume_1751_enquiry" rather than "hume_1975_enquiry".
The call is for origyear rather than origdate as such
Note it's the origyear rather than origdate I'd be after. In Biblatex origdate is parsed out into relevant date parts origyear, origmonth, origday, etc. and made available for styles. In the Hume example there's no difference between origyear and origdate. However in something like, from Zotero ...
Author: Wenar, Leif
Date: 2015-09-09
Extra: bibtex[origdate=2005-12-19]
Title: Rights
Publication: The Stanford Encyclopedia of Philosophy

I'd want BBT to generate a citation key of "wenar_2005_rights" rather than "wenar_2005-12-19_rights" (nor would I want, as would occur with my current customization, "wenar_2015_rights"). That is, I'd like to end up with an export like ...
@article{wenar_2005_rights,
  author = {Wenar, Leif},
  date = {2015-09-09},
  origdate = {2005-12-19},
  title = {Rights},
  journaltitle = {The Stanford Encyclopedia of Philosophy},

[Edit: If I was citing this entry in a real article I'd probably only by interested in the latest date and not use origdate at all. But I include the example as you, as BBT dev, would probably need to programatically handle origdates that include a full date, rather than merely a year.]
What the solution might entail
I note from https://github.com/retorquere/zotero-better-bibtex/wiki/Citation-Keys

... in addition to the 'special' fields listed JabRef also allows all 'native' fields as key values; the [BBT] plugin does the same but allows for Zotero native fields (case sensitive!) not Bib(La)TeX native fields

And I don't think my feature request amounts to adding custom citation key support for Biblatex native fields, as such. Rather it perhaps amounts to a request for:

Adding custom citation key support for BBT custom fields (as inserted into Zotero's extra field with 'Bibtex[fieldname1=fieldvalue1;fieldname2=fieldvalue2; ...]'; and
A special handling of 'Bibtex[origdate=...]' such that BBT parses this and makes it available to custom citation key formatting as origdate (holding the full value) and origyear (holding just the year. I could see some future potential requests for parsing into other components origmonth, origday, etc - but for the moment that's nowhere part of my interest - I'm only interested in the year part (and perhaps the whole part of origdate).

My motivations
Just to expose my motivations. Obviously I'm interested in writing articles where the original publication date is more important. So I'm wanting to end up with in-text citations and bibliographic entries that look like ...

Lorem (Hume 1751).
Hume, David. 1751. "An Enquiry Concerning the Principles of Morals." In Enquiries Concerning Human Understanding and Concerning the Principles
of Morals, 3rd ed., edited by L. A. Selby-Bigge and P. H. Nidditch. Note.
New York: Oxford University Press, 1975-06-12. [Bold added]

When writing in latex, and using a biblatex file, that this entry might have a modern date in the citation key, e.g. "hume_1975_enquiry", is not too great of a problem. In one's tex document this might be referenced as \autocite{hume_1975_enquiry} and the modern date is appropriately regulated, using one's chosen style vodooo, to a less important position on the rendered PDF output.
However, I'm attempting to identify, and if need be create, a toolchain that allows lightweight markup source documents (like markdown) to be transformed into various end user formats (pdf via latex, epub, html, Amazon Kindle). Pandoc looks promising here. In this context the readability of the source markup is more important than, say, the readability of a tex document. The whole motivation for using a lightweight source document is so that it can be handed around directly between non expert users.
So, in the context of writing articles where the original date is more important, and that the writing will be an a lightweight markup language, I'd much rather have a citation key that uses the original year.
Edit: Added edit as marked.
Edit: "the" to "that".

In the meantime a workaround can be used by leveraging the feature documented at https://github.com/retorquere/zotero-better-bibtex/wiki/Citation-Keys#set-your-own-fixed-citation-keys
For example, in Zotero my Hume entry currently has a citation key hume_1975_enquiry and I want to end up with hume_1751_enquiry. To do this:

In Zotero right click on the entry > "Generate BibTex Key".  "bibtex: hume_1975_enquiry" will be appended to the Extra field (and so, excellently, does not interfere with existing values in the Extra field).
Manually edit "bibtex: hume_1975_enquiry" to be the desired key,  "bibtex: hume_1751_enquiry".
Tab out of the Extra field to save.
Click on some other entry. Click again on the Hume entry. This will refresh the Citation Key which now should be taking its value from the Extra Field "bibtex: hume_1751_enquiry". That is, the Citation Key should now be "hume_1751_enquiry".

In BBT5 you will be able to use [origyear] in your pattern; origyear will be origyear if present, year otherwise; year the other way around.


Failed to logsparse multiprocess functions

Does log_sparse function in viztracer fail to work if the function we are trying to profile is of multiprocess?
I run the following sample code and the programs stops exactly after 50 seconds everytime.
Command to run the code : viztracer --log_sparse sample.py
Ubuntu 20.04.3
Python 3.8.10
viztracer 0.15.3
Sample Code

Output

Output in terminal

Thanks in advance for your help

@log_sparse means "log THIS function", which is the hello function(not any of its descendants), and viztracer actually did what's expected - logged the only call of hello.
Do you want the function calls inside hello? You could try to add stack_depth argument to the @log_sparse decorator. However, in that case, I don't know why you want to use @log_sparse. What data are you trying to get rid of?

The @log_sparse is logging the hello function as it should but the problem is the hello function or any other function (that has multiprocessing in it) that I am trying to log is stopping exactly after 50 seconds without interference. May I know why its stopping after 50 seconds and how to keep it running as long as the functions runs. Also, please point out if I am doing anything wrong.
Thanks

Because you have a timer in your while loop - total_time, and break out of your while loop when it reaches 50. This is not related to viztracer, I'll close it. Next time I would suggest that if you have any program related issue(not directly viztracer related), turn off viztracer and run with python directly to see if you can reproduce it.


Access to /tmp directory

It would be nice to have access to /tmp directory for opening torrents directly from firefox. Now I have to save .torrent file  to downloads and then open it with qBittorrent.

Accessing /tmp an open problem in snapd at the moment,  https://forum.snapcraft.io/t/sharing-files-via-tmp/1613/16
The gist is that the /tmp firefox is seeing is specific to its mount namespace.
A fix is in the pipeline for Firefox 77, which will make Firefox use $HOME/Downloads/firefox.tmp instead of /tmp for opened files. It'll also improve file handling in general.


Support newer java versions

This is a major rework of the memory project structure that provides support for Java 8, as well as Java 9 through 13.
The changes leverage improvements in the Java ecosystem such as the Java Module System (JPMS), and multi-release JARs (MR-Jars).
The PR here is a consolidation of work conducted together with @leerho over several months, and documentation has been provided to the rationale for the changes.
For further information, see this issue: https://github.com/apache/datasketches-memory/issues/121

This is a great PR. I really appreciate the work you put into the documentation!!  So many of the files were just a path name change that the review process went a lot faster than I expected.
I still have potentially 1 or 2 files I want to move in from datasketches-java, but we could do that after this merge.
If you can fix the LGTM problem that would be great, but if it is too much of a hassle, I wouldn't worry about it.

This is a great PR. I really appreciate the work you put into the documentation!! So many of the files were just a path name change that the review process went a lot faster than I expected.

Thanks for the positive feedback!

I still have potentially 1 or 2 files I want to move in from datasketches-java, but we could do that after this merge.

If you can fix the LGTM problem that would be great, but if it is too much of a hassle, I wouldn't worry about it.

This is currently logged with asf-infra, but if it takes too long to resolve we can discuss alternatives.


Replace whilp/git-urls module by chainguard-dev/git-urls

We discovered a vulnerability on the module github.com/whilp/git-urls GHSA-3f2q-6294-fmq5. This repository doesn't look to be maintained at this moment. That is why we decided to fix the vulnerability and move the repository over our organization. We plan to maintain this repository in the future.

@hiddeco I don't see why we couldn't bump that limit to 2048 on our side. Let me handle that change and publish a new release.

@hectorj2f can you please run make tidy and force push the changes, this will trim go.sum and fix the CI.

Thanks for the reviews!


allow setting both absolute and relative tolerance of sparse iterative solvers

Currently a single tolerance (tol argument) is used, which is impractical, when using an iterative solver as a preconditioner. In that case it is desirable to set a relative tolerance to some not-so-small value (e.g. 1e-2), without caring about the absolute tolerance. The current behaviour prevents that, as something like setting tol=1e-2 often leads to absolute convergence in iteration zero of the preconditioner, hence no preconditioning done at all.
This behaviour breaks the IMHO very good lgmres solver. For example, the following happens when using lgmres with cg (tol=1e-2) as a preconditioner for symmetric positive definite diagonal blocks:
<snip>
sfepy: cg-s: iteration 1
sfepy: cg-s: iteration 2
sfepy: cg-s: iteration 3
sfepy: cg-s: iteration 4
sfepy: cg-s: iteration 5
sfepy: cg-s: iteration 6
sfepy: cg-s: cg convergence: 0 (successful exit)
rhs norm: 8.751133e-02, solution norm: 4.484927e+00
sfepy: iterative-s: iteration 4: |Ax-b| = 2.323006e-03
sfepy: cg-s: iteration 1
sfepy: cg-s: cg convergence: 0 (successful exit)
rhs norm: 4.101975e-04, solution norm: 0.000000e+00
sfepy: cg-s: iteration 1
sfepy: cg-s: cg convergence: 0 (successful exit)
rhs norm: 2.286502e-03, solution norm: 0.000000e+00
Traceback (most recent call last):
<snip>
  File "....local/lib/python2.7/site-packages/scipy/sparse/linalg/isolve/lgmres.py", line 145, in lgmres
    "|v| ~ %.1g, |M v| = 0" % rnorm)
RuntimeError: Preconditioner returned a zero vector; |v| ~ 0.002, |M v| = 0

This happened because the rhs norm was 2.286502e-03 < 1e-2 (see the line above the traceback), so no solution was attempted.
So I propose having both rtol and atol arguments (or allow tol to be a tuple), and possibly adding a tol_mode argument - either 'and' or 'or', defaulting to 'or', that tell whether both tolerances have to be met ('and' mode) or a single one is enough.
I can make a PR implementing this for lgmres, as it is written in Python. I am not sure if I am brave enough to dive into the legacy fortran of the other solvers.

When a similar issue came up with differential_evolution we kept tol for relative tolerance and added an atol for absolute tolerance:
https://github.com/scipy/scipy/pull/6449#issuecomment-239268592
Seems like doing the same thing here would be appropriate.

This has been on the todo list forever, so it's good if you want to fix
it finally...
.
For backward compat: I would add new arguments rtol and atol. The tol
argument sets both to the same value, and an error is raised if the user
gives both tol and rtol or atol. We might also want to issue a
DeprecationWarning if the user sets the tol argument.
.
The second question is what to do with left-preconditioning. I'm not
sure applying an absolute tolerance to preconditioned residuals is very
sensible --- this is just a surprise waiting to happen.
.
Note that there's some refactoring of LGMRES here
https://github.com/scipy/scipy/pull/5910, plus some minor fixes.

(Btw, review of gh-5910 would be appreciated in case you're interested
in the topic ;)

Yes, leaving tol and adding rtol, atol was another option I considered :). I will give it a shot after I resolve my compilation issues with the scipy master.

The fortran-based solvers seem all to use the relative-only test (contrary to what the docstrings say), as defined in  STOPTEST2.f.src, which is equivalent to r_norm <= tol * b_norm.
The python-based solvers (lgmres, and gcrotmk) use both the relative and absolute test.
So the PR should

modify STOPTEST2.f.src for RTOL, ATOL
update iterative.py for rtol, atol
update lgmres (and gcrotmk) for rtol, atol

Now the condition above measures |r_k| / |r_0| only for the zero initial guess. Does it make sense to use |b - A x0| instead of |b|? I guess no - I have just checked that petsc uses also just |b|, so it is what people probably expect, but what do you think?
I would also like to introduce the  tol_mode argument (described above) - do you agree?

The fortran solvers also do absolute termination checks, iirc, but this
is elsewhere in the code than in STOPTEST2. The extra check may also in
some cases be done only for the first step.

Yes, I can see the absolute test for the first step, e.g. in CG. Thanks!

Heretical thought: having the fortran solvers rewritten in Python, following the examples of lgmres and gcrotmk, would be much more maintainable.

Heretical thought: having the fortran solvers rewritten in Python, following the examples of lgmres > and gcrotmk, would be much more maintainable.

Doesn't seem heretical to me (I've wished for the same thing in fact), though Cython might be even better. I'd guess that the issue here is just someone having time to do it.

As a fun side-effect it would also make the solvers reentrant.

Would Cython help here? IMHO the solvers spend most of the time for non-tiny problems in blas/matvec calls, that are compiled anyway. But I did not measure that :)

Would Cython help here?

I doubt it would speed things up much. IMO the benefit would come from possibly exposing the Cython interface directly in addition to a Python interface.

Well, there is [1] - a ready python implementation with MIT license and compatible function arguments. I guess we should not reinvent wheels :)
[1] https://github.com/pyamg/pyamg/tree/master/pyamg/krylov

@rc: did you manage to make headway with fixing the tolerances?

@pv not yet - I have been a bit afraid of diving into the fortran sources. :(  (And holidays interference in the meantime.) The fortran code seems to have some tolerance related issues/discrepancies (like https://github.com/scipy/scipy/issues/7623), which I did not analyse properly.
I was exploring a bit the Krylov solvers available through PyAMG, as a potential candidate for replacing the fortran code in scipy, but even those seem to have some discrepancies in how the error is measured among the methods and between the implementation and docstrings (see e.g. https://github.com/pyamg/pyamg/issues/203, where a possible better stopping criterion is also discussed). It might be that the pyamg implementation was inspired by the same source as the scipy one. In any case, the PyAMG Krylov solvers are way more readable (for me). What do you think about contacting the PyAMG team in the respect of using their solvers in scipy? But I guess this would not be for the 1.0 milestone, while I can still try to hack the fortran functions in time.

No open PRs yet, so I'm pushing this back to the 1.1 milestone.

Suggested fix in gh-8400

Thanks @pv! I will try the preconditioning use with the https://github.com/scipy/scipy/pull/8400 branch.

Now that you mention preconditioning...
I now realize that the fortran code termination conditions actually use
the left preconditioned residual. So gh-8400 needs still further work.

Ok, preconditioner issues should now be fixed in gh-8400.
In the end, this only affects gmres and lgmres, because only they have an inner loop stoptest that looks at the preconditioned residual.
Because for preconditioned problems the preconditioned residual and actual residual may behave differently, I had to add an adaptive tolerance adjustment that ensures the inner loop tolerance is reduced if the preconditioned residual check passes, but the actual residual check not. This is somewhat different from a "traditional" GMRES implementation that only deals with relative tolerance of the preconditioned residual, however.


Changes across the board

Notable changes below.
Streams:

Use confluent.version 2.1.0-alpha1
Use Scala 2.11.7
Add ZK connect config to examples

All = Streams + other examples:

Improve docs (READMEs)
Restructure layout of examples repository.  Streams examples are now under kafka-streams, everything else is under kafka-clients.
Updated 2 of the 4 kafka-clients examples (producer and consumer;  I didn't touch the two Avro examples) to use CP 2.0.1 / Kafka <IP_ADDRESS>-cp1.
Add licensing information (Apache v2) and NOTICE files for all projects.

LGTM. I will add the staging repo after merging PR.


[Bug]: CodeAct agent console log skips user messages

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
The user message is now an action, when it used to be an observation. I think the observation was displayed/logged in console, the action is not.
The effect is that CodeAct seems to talk to itself, without presenting the other side of the conversation.
Note: while this seems weird and affects logs,
Current Version
Current main (https://github.com/OpenDevin/OpenDevin/commit/8bfae8413e74e1b6095d5380a7ea93b6b73a4537)

Installation and Configuration
`make build`
`make run`

Model and Agent

Model - GPT-3.5-turbo
Agent - CodeAct

Reproduction Steps

run with UI, CodeAct
start a task
say something in the UI
=> everything I say won't be in the console, but the agent does talk to someone, a ghost, an alter ego... 😅

Logs, Errors, Screenshots, and Additional Context
Example console display CodeAct talking to itself:
STEP 0

22:30:43 - PLAN
make a bash script to solve the 17 queens problem
22:31:34 - ACTION
MessageAction(content='Sure, I can help with that. The N-Queens problem is the problem of placing N chess queens on an N×N chessboard so that no two queens threaten each other. Thus, a solution requires that no two queens share the same row, column, or diagonal.\n\nThe 17-Queens problem is a specific case of the N-Queens problem where N is 17. This problem is computationally expensive to solve, especially with a bash script, due to the large number of possibilities to check.\n\nHowever, I can provide a simple bash script that uses a backtracking algorithm to solve the N-Queens problem. This script will not be efficient for large N like 17, but it can solve the problem for smaller N like 8.\n\nHere is the bash script:\n\n```bash\n#!/bin/bash\n\n# N-Queens problem solver in Bash\n\nN=$1\nsolution=0\n\n# Initialize the board\nfor (( i=0; i<$N; i++ )); do\n  for (( j=0; j<$N; j++ )); do\n    board[$i,$j]=0\n  done\ndone\n\n# Check if a position is safe\nis_safe() {\n  local row=$1\n  local col=$2\n  local i=0\n  local j=0\n\n  # Check this row on left side\n  for (( i=0; i<$col; i++ )); do\n    if (( board[$row,$i] )); then\n      return 1\n    fi\n  done\n\n  # Check upper diagonal on left side\n  for (( i=$row, j=$col; i>=0 && j>=0; i--, j-- )); do\n    if (( board[$i,$j] )); then\n      return 1\n    fi\n  done\n\n  # Check lower diagonal on left side\n  for (( i=$row, j=$col; j>=0 && i<$N; i++, j-- )); do\n    if (( board[$i,$j] )); then\n      return 1\n    fi\n  done\n\n  return 0\n}\n\n# A recursive utility function to solve N-Queens problem\nsolve_n_queens() {\n  local col=$1\n  if (( $col == $N )); then\n    (( solution++ ))\n    echo "Solution $solution:"\n    print_solution\n    return\n  fi\n  for (( i=0; i<$N; i++ )); do\n    if is_safe $i $col; then\n      board[$i,$col]=1\n      solve_n_queens $(( $col + 1 ))\n      board[$i,$col]=0\n    fi\n  done\n}\n\n# Function to print a solution\nprint_solution() {\n  for (( i=0; i<$N; i++ )); do\n    for (( j=0; j<$N; j++ )); do\n      echo -n "${board[$i,$j]} "\n    done\n    echo\n  done\n  echo\n}\n\n# Solve the N-Queens problem\nsolve_n_queens 0\n```\n\nYou can run this script with the number of queens as an argument, like this:\n\n```bash\nbash n_queens.sh 8\n```\n\nThis will print all solutions to the 8-Queens problem.\n\nPlease note that this script uses a simple backtracking algorithm and is not optimized for large N. It may take a very long time to find all solutions for N=17.', wait_for_response=True, action='message')

==============
STEP 1

22:34:05 - ACTION
MessageAction(content="I'm sorry for the confusion, but as an AI text-based model, I don't have the capability to execute bash scripts directly. However, you can run the script on your local machine by following these steps:\n\n1. Copy the script provided in the previous response.\n2. Open a text editor, paste the script, and save it as `n_queens.sh`.\n3. Open a terminal and navigate to the directory where you saved `n_queens.sh`.\n4. Make the script executable by running `chmod +x n_queens.sh`.\n5. Run the script with the number of queens as an argument, like this: `./n_queens.sh 3`.\n\nPlease note that the N-Queens problem has no solutions for N=2 and N=3. The first non-trivial case, with a solution, is N=4.", wait_for_response=True, action='message')

==============
STEP 2

22:36:26 - ACTION
**CmdRunAction**
THOUGHT:I apologize for the confusion. You're right, I can execute bash commands. However, creating and executing a bash script in this environment is a bit tricky. Let's try it step by step.

Fixed by #2035


input[range] max is interpolated after model value is set

Fiddle here: https://jsfiddle.net/dserrano_lnk/fy93ptkk/
Assume all atributes are interpolated (min and max) and bound (using ng-model) on the input[range].
If the range's max is over 100 (eg. 200), and the value is set over 100, the range's value will be "stuck" at 100 (the default max value for a range). When the range's max attribute is updated, the range's value stays at 100, while in the model the value is still 200.
I worked around on this, by using a $timeout.

input[type="range"] is not a supported type in Angular (e.g. see #5892), so (when using ngModel on it) it falls back to input[type="text"]. This can lead to all sorts of weird behaviors.
Among other things, the min and max attributes are not watched by Angular for changes, so the will probably be no re-rendering one they change from {{vm.max}} to 200 for example.
--
Basically, here is what is going on:

Initially, the element max and min properties are set to {{vm.max}}/{{vm.min}} respectively.
Since these are invalid value, the actual limit fall back to the default: 0 - 100
ngModelController applies the modelValue (200) to the input. Since the max allowed value is currently 100, the input's value is set to 100.
Interpollation kicks in asynchronously and sets max/min to 200/0 respectively.
The input's limit are updated in the UI automatically.
Since there has been no change in the modelValue or the viewValue, ngModelController doesn't "understand" it has to re-$render() (so it doesn't). As a result, the input's viewValue remains at 100 (and the modelValue remains at 200).

This is a problem for input[type="range"], because (unlike all other input types) its "display value" (what the user sees, i.e. the position of the knob) may change even if it's actual value hasn't changed.
Basically, you need to let ngModelController know, that max/min/ngMax/ngMin needs to be watched over and when the corresponding attribute value changes, it needs to re-$render().
Here is a quick demo: Updated fiddle
--
Closing as currently there is no support for input[type="range"] and adding support for it is tracked in #5892.

Thank you for your input and fix =)


Add Translation Sensitivity to Editor Settings

This for 3D Navigation. Currently in Godot, you cannot adjust Panning sensitivity. So for people who uses non-default Orbit sensitiviy, the default panning speed will most likely feel off. This PR adds a Translation Sensitivity option that fixes this issue.
The max value increase also help addressing #75855 (for trackpad users). Original code change comes from @geminax from that thread. All credit should go to him; I just tested and prepared the PR.
I also want to add a bit more context as to why this is important to me. I have 15+ years of muscle memory from Maya and Unity. I tried to get into Godot in the past but the 3D navigation always felt off on my setup. I know Godot has been improving on this end as well. There are now more options I can tweak in this regard. And I think this Translation Sensitivity setting is the last missing piece I need.

Squashed everything and added missing entry to doc/classes/EditorSettings.xml

Are you building a double precision build? Then that's the reason, not a bug in that case

Are you building a double precision build? Then that's the reason, not a bug in that case

Thanks for the info! Yes, you are right. After building and using the x86_32 version, no more extraneous xml updates were generated for me.
So I think the doc is making it slightly confusing then. On the Updating class reference when working on the engine doc page, it gives example of running --doctool with the x86_64 version.

If that's the case, I can try to contribute to the doc repo.

Meant the --precision=double option, it often changes the docs by using for example PackedFloat64Array over PackedFloat32Array, and changing some default values

If I'm not mistaken there, I can also try to contribute to the doc repo (adding a note about the possible doc generation difference between x86_32 and x86_64).

Documentation output should be identical between 32-bit and 64-bit builds. (This isn't to be confused with the floating-point precision build-time option, which is entirely independent of the architecture's bitness.)

Documentation output should be identical between 32-bit and 64-bit builds

On my end, running .\bin\godot.windows.editor.x86_64.exe --doctool generates 4 more updated xml files:
https://github.com/CreepGin/godot/commit/2e23ca275e2b69c870e6f757270e38dc0df52ee8
Running the x86_32 version does not produce any extra updates.

Those are built with a different version I'm 99% sure, the redundant_for_variable_type setting was removed two days ago, have you built both versions from the same branch?

have you built both versions from the same branch?

Yes

Just to be sure build both again and see, because all these changes were from PRs merged in the last two days, so it'd be a fantastic coincidence

Okay finally got it. It's definitely my silly mistake. I must've mixed up the exe names (with/without ".dev") for some of the runs. Sorry for the trouble! 😓

Happy it got worked out! It's an extremely easy mistake that I've done myself a few times

Hey!  Who forgot about this?  It's almost done and kind of a big deal for anybody working from a lap top with experience in other 3d software.  It's not necessary by any means, but it's pretty bad feeling to not be able to navigate comfortably.

I think it was ignored for a long time. Let me know if I can do anything more. I just pushed to the PR to resolve a recent conflict.

I'm waiting for this. When will this be released?

is it a matter of waiting for next release?

Thanks! Congratulations on your first merged contribution! 🎉


SMART launcher auth token endpoint URL

While launching the smartform.io from the smarthealthit launcher using provider and patient context, where does the app get the /token from? Why is the app following a Auth code flow (authorisation_code) as well?

Also confirming if this is the implemented flow for establishing the provider and patient context? is the /auth token call for this highlighted step?

Going though the entire launch auth flow step-by-step:

Launch
This is the action of clicking on the "Launch" button in the SMART Launcher. It launches the app.

Get auth server
The app initially doesn't know the location of the auth server, so it has to request the auth server's endpoint from the CMS server. In this case it is a GET request:
https://launch.smarthealthit.org/v/r4/fhir/.well-known/smart-configuration
In the response, auth server endpoint is defined as https://launch.smarthealthit.org/v/r4/auth. The app now knows where to go to be authenticated.

Request authorisation
Now, since the app knows where to be authenticated, it asks the browser to navigate to the auth server's authorize endpoint to get an authorization code. In this case:
https://launch.smarthealthit.org/v/r4/auth/authorize
This navigation request consists of all the launch params defined in the first step i.e. client id, scopes, redirect uri, etc. If the auth server accepts this request, it generates an authorization code.

Redirect to app
If an authorization code is generated, the user is redirected back to the app with the authorization code.
Otherwise, you get blocked from proceeding further. A page showing "unauthorised" might be shown along with the reasons why you aren't authorised (possibly due to mismatched scopes or a client id).

Token exchange
Now that the app has an authorization code, it performs a POST request to the auth server's token endpoint to trade its authorization code for an access token. In this case:
https://launch.smarthealthit.org/v/r4/auth/token
This access token can now be used when making requests to the CMS's FHIR API.

Here's where the steps fit In your screenshot:

Hope that helps.

Hey @dattachandan, are you happy for me to close this issue given it has been stale for a bit?


Fix View.propTypes -warning on RN 0.44

Fixes this:

Warning: View.propTypes has been deprecated and will be removed in a future version of ReactNative. Use ViewPropTypes instead.

by using ViewPropTypes

@ollija thanks 👍


Add missing Konsole kpart to various KDE applications

Motivation for this change
I missed a few runtime Konsole dependencies in the wrapper updates! Fixes #17419.
@FRidh As the Kile maintainer, could you check that it works with the new wrapper?
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@ttuegel, yep, now it works. Thanks!

I've changed Yakuake (2e538a0f55e3c40b2b02ee1c750f145eeb1cd27f) and Konversation (65c109f8875e37726a96ec8cd9e8a090c921b08c) to use kdeWrapper as well.
I like this new method, instead of invoking the wrapper explicitly in a phase its now declarative (at least on the outside). In my opinion we need more of this in Nixpkgs.


Remove ignore marked_as_spam column

Now that the marked_as_spam column has been removed in
https://github.com/alphagov/email-alert-api/pull/1353, we can safely delete this code.

merged into https://github.com/alphagov/email-alert-api/pull/1353.


Allow creating custom clients

This PR allows to re-use the existing logic of setting up a reqwest client, for custom purposes.
The intended use case is that you want to re-use the config loader logic, but want to further work with the client builder, adding your own options.

Hey. Thanks for this. It looks sensible to me, but appears to be failing tests.

Hey. Thanks for this. It looks sensible to me, but appears to be failing tests.

Cool. Should be fixed. Just learned that you cannot use ? in a comment. So now I use "expect" like you did in the other comment. This fixed the tests.

Yeah, haven't imported all the Result stuff in there. The stack trace is generally pretty good in tests. Thanks for fixing it up! Looks good to me!


Missing Connect to Twitch button

At  the top right, before I updated I had the connect to twitch button so I could download subbed videos but, was getting an error when trying to download so I updated. Now the button is gone completely.

Here you go:
https://github.com/Franiac/TwitchLeecher/releases/tag/v1.8.0


Question about raw videos

Hi @QinghongLin! May I ask that where I can find the raw videos for TACoS and QVFS datasets? It seems that the original authors did not release them. Thank you!

Hi @yeliudev ,
For QFVS videos, please visit this instruction: https://github.com/facebookresearch/EgoVLPv2/tree/main/QFVS
For TACoS videos, please visit here: https://www.mpi-inf.mpg.de/departments/computer-vision-and-machine-learning/research/human-activity-recognition/mpii-cooking-2-dataset
:)

Got it. Thank you so much!


[test] use explicit functions

Current dependencies on/for this PR:

main

PR #2   👈

This stack of pull requests is managed by Graphite.


feat(rest-api): create documentation page on api create

Issue
https://gravitee.atlassian.net/browse/APIM-4328
Description
Introduce create documentation on api creation
Additional context

📚  View the storybook of this branch here

Closing. It seems to be a WIP PR, no more necessary


Add test case for endpoint with both predictor and explainer.

What this PR does / why we need it:
Verified the explainer ksvc is created correctly
Then verified the route for explainer is created correctly when status is ready
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #414

This change is

/assign @rakelkar @ellis-bigelow
In PR #465 we added unittest for virtual service which is good. This PR add test from controller point of view, to create ksvc and update status, then finally got virtualservice and route created correctly.

awesome test!
/lgtm
/approve

/retest


question: does it work the same way as c++ std::partial_sort?

hello, does it work the same way as c++ std::partial_sort?
if not, how to make some change to work same way as std::partial_sort?
thank you!

No, it works like https://en.cppreference.com/w/cpp/algorithm/nth_element. To get something like partial_sort, you would need to sort the 0...k elements in place afterwards.


Additional shipment options

Description
Added support for extra shipment options according to https://dhlparcel.github.io/api-gtw-docs/#a-shipment-options

BOUW | Delivery to construction site
EXP | Expresser
HANDTPS | Signature on delivery at parcel shop
PERS_NOTE | Email to the receiver (for MDP Business app only)
REFERENCE2 | Extra reference label
ADD_RETURN_LABEL | Include extra label for return shipment
NO_TRACK_TRACE | No track and trace for shipment

Motivation and context
I needed this to add a PERS_NOTE to my shipments and there was already a request in #70 for ADD_RETURN_LABEL
How has this been tested?
I've added unit tests to test all the new options
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
Please, please, please, don't send your pull request until all of the boxes are ticked.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[ ] Each individual commit in the pull request is meaningful.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] If my change requires a change to the documentation, I have updated it accordingly.
[x] My code follows the code style of this project.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

I've tried to add some new funtionality to support more shipment options. But I messed up a little bit with my commits. How can I remove them?
And how can I make all those tests pass?

Can this MR please be merged?


Added link to MSAL for Go

Thanks @ktoliver for reaching. Please reassigning this to @mmacy as I moved from this docset.

Hi @mmacy - Could you review the pull request? Would you be able to update the metadata, too? Thanks!

@roberthstrand I hadn't heard this had gone to public preview. I see in the MSAL Go README that it's being called "production ready preview," but I'm not sure that means it's treated as a public preview product.
I'll check with the team and get back to you.

Adding members of the team that might be able to clarify whether MSAL Go has officially gone public preview.
#assign-reviewer:henrik-me #assign-reviewer:abhidnya13 #assign-reviewer:SaeedAkhter-MSFT

The following reviewer(s) could not be added because they aren't a valid GitHub ID, aren't a valid collaborator on this repo, or are the author of the PR: henrik-me abhidnya13 saeedakhter-msft

The following reviewer(s) could not be added because they aren't a valid GitHub ID, aren't a valid collaborator on this repo, or are the author of the PR: henrik-meabhidnya13saeedakhter-msft

Thanks, @abhidnya13!
@roberthstrand Definitely appreciate your submission, but closing this one for now. Once we're ready to call this public preview, we will update the documentation to reflect that. Thanks for the PR!
#please-close


Transition spring modules to New Codesnippet Tooling

See #24958 for details.

This is a prerequisite to getting JDK17 CI legs running in the repo.
@alzimmermsft could you add links to the pending CI work?

This issue is no longer a blocker for the transition to Java 17 in CI, and as of https://github.com/Azure/azure-sdk-for-java/pull/26452 the changes need to resolve this issue have changed. Before that PR this issue was to remove usage of the old codesnippet tooling and resolving any Javadoc warnings and errors that happened due to that but the PR has removed the codesnippet tooling and the Spring libraries were given a build flag that ignores Javadoc errors. Now, this issue should track the following work for all Spring libraries:

Remove the <doclint> property configuration.
Run Javadoc generation.
Resolve all Javadoc errors and warnings.
Run Javadoc generation again to ensure build passes.

For convenience all locations that need this change have a comment of <!-- TODO: Remove this once all Javadoc warnings and errors are resolved -->.
As an example using azure-spring-cloud-messaging,

I would remove the <doclint> property configuration in it's POM file.
I would run mvn clean install to do a full build.
I would update the locations with Javadoc errors, these are almost exclusively missing Javadocs on public APIs.
I would again run mvn clean install to do a full build.
I would repeat steps 2-4 until build passes without Javadoc generation errors.

closed in https://github.com/Azure/azure-sdk-for-java/pull/28239 and https://github.com/Azure/azure-sdk-for-java/pull/28242


boolean OR?

I want to say something like this:
        let assert = Command::new("cargo").args(&["clean", "-p", "simple_project"])
            .current_dir("./tests/simple_project")
            .assert()
            .success()
            .or().stdout("package ID specification `simple_project` matched no packages");

But I don't see an or() function, or any function taking an arbitrary Predicate instead of a built-in assertion. In the meantime I can use https://doc.rust-lang.org/std/process/struct.Command.html#method.output, but it would be nice to have something built-in.

As of right now, we can only handle predicates on one field at a time.  I'm doubtful we'll be enabling ore complex ones.
In considering more complex expressions, one of the problems is the termination case, what causes us to evaluate all of it.
assert_cmd (at that time assert_cli) supported building expressions (before the execution happened).  We simplified to being a thinner wrapping around Command but without us carrying any data because of that, we had to move the assertions to happen after the command runs.  The problem we ran into was evaluating the expression.  Its very easy to forget (though maybe we could play with must_use).
Then next problem is naming.
After that is how do we enable these arbitrary declarative-like  logical conditions.  Do we explicitly not support nested expressions? If we support them, what would the API look like?
With all of this, what is the ergonomics of the API compared to writing a more imperative logic.


Use most recent React version

It seems like we’re currently using the React version defined in package.json.
Instead we should use the most recent React version when creating a new project.
Please feel free to look into this and/or send a PR.

I'm happy to take a look at this and work on a PR.

Thanks

Does this apply to react-dom as well?


Error: DOMException: The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission

I get Error: DOMException: The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission When trying to do this:
const response = await fetch(${apiUrl}/api/challenge?${params.toString()}, {
method: 'GET',
headers: {
'Content-Type': 'application/json',
},
});
if (!response.ok) {
throw new Error('Network response was not ok');
}
const challenge = await response.text();
console.log('Challenge:', challenge);
try {
const registration = await client.register(${formData.firstName} ${formData.lastName}, challenge, {
authenticatorType: "auto",
userVerification: "required",
timeout: 60000,
attestation: true,
userHandle: "",
debug: false
})
console.log('Registration:', registration);
these are the logs:
Result: 99b5c557-51d3-4945-a9b5-999889cc6b77 [index.tsx:139:10](webpack://_N_E/pages/add/index.tsx?2ab1)

Error: DOMException: The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission.

Dunno. Too vague.

I closed it seems it seemed more like a problem with your code than this library. Feel free to reopen it if you have more information.


[Snyk] Security upgrade jwcrypto from 1.5.2 to 1.5.6

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.test.txt

⚠ Warning
django-oauth-toolkit 1.6.1 requires jwcrypto, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

733/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 6.8
Denial of Service (DoS) SNYK-PYTHON-JWCRYPTO-6405821
jwcrypto:  1.5.2 -> 1.5.6
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

Long time no see @yuritoledo!


Clear control in all input types

We want to have a control that allows the users to easily clear the value for all input types. Currently in recordedit, there is a clear control for date inputs, timestamp inputs, Asset selector inputs, and Foreign Key selector inputs. In the range inputs, we already have integer/float inputs and date/timestamp inputs with an x control to remove the values.

@jrchudy @RFSH  Should this be part of the style guide?

This has been done as part of #1847.


Prepare automatic performance tests for Backend

Expected Behavior
We should have at least one performance test to be sure we can handle multiple users simultaneously.
Actual Behavior
MedTagger doesn't have any automatic performance tests. This results in failure during initial and first demo as our server was overloaded by 50+ people trying to pull heavy Scans.
Additional comment

It should be an easy-to-use script that could be run anywhere.
It should be configurable to target any MedTagger instance.
It will help us to improve quality and measure benefits from server changes,
Cover fetching Slices and using API for selecting random Scan (most impacting use cases).

We have a NodeJS script that simulates user's behaviour but it's not so scalable. It can simulate only one user but it would be nice to have some wrapper that will allow to simulate N users at once.

Don't forget to write some docs describing how to use it and how to prepare environment for such simulation (npm install ...).


Question: translating a monolingual HF Dataset

Hey guys.
Was jus wondering if the translation steps fits also datasets with one language only. I am asking as I saw that there is a default parameter which specifies the column which includes the language.
If possible, how to overcome that scenario (having no columns with indication of the language as the dataset is - let's say - all English .. ?
big thanks.

The scripts as-is won't directly work as they rely on a language column indeed. You have 2 options for now I think:

Modify the code to allow for fixed language as input
Use Pandas to quickly add such a column, something like:

df = pd.read_csv(...)
df['lang'] = 'en'
df.write_csv(...)

I suspected it. I'll move on. Thanks a lot.


Suggestion: change "cimgui" mentions to something unique

Even though dear_bindings is a completely separate project from https://github.com/cimgui/cimgui, it still uses cimgui terminology - there's a cimgui namespace, defines like CIMGUI_IMPL_API, etc. I understand the name in both cases simply means C+ImGui, however I also personally feel like in order to reduce confusion, dear_bindings could change it to something unique, purely for differentiation reasons.
I've been personally confused because of this multiple types. When just seeing a file named cimgui.cpp, I have no clue what generator does it come from. When I first stumbled upon dear_bindings, this also made me think it's simply a modern reimplementation of https://github.com/cimgui/cimgui that aims to keep the output the same, which is not the case.
The new name doesn't have to be anything fancy, just something that allows for mental separation. Ideas:

dcimgui (dear c imgui)
dbcimgui (dear bindings c imgui)

The only downside I can think of is, it will be a breaking change for anyone currently using dear_bindings... But, we're only v0.11, anything reasonable goes?

Personally I kinda of disagree. Both headers have mentions at the top of their file.
I suppose there are 2 questions: one for the (default) filenames and one for in-code prefixes and namespaces.
Tangentially, at least one person told me they were reluctant to switch from cimgui to dear bindings because they prefer the ig prefix e.g. igButton() over ImGui_Button(). I personally don't fancy the ig prefix too much but it's nice that it is short, but I don't think it would be a bad property if most names were similar.

The only downside I can think of is, it will be a breaking change for anyone currently using dear_bindings...

I think it would be fine to rename file and symbols at this point, if that was deemed desirable.

The idea initially was to make it easy for people to switch, but I can definitely see that if you're in the situation where both at floating around the filenames don't make things easier.
dcimgui.h feels simpler than dbcimgui.h (even just typing the latter now I had to think about it - I'm not sure why by dc comes much more naturally to me than dbc). It's also a relatively benign change in the scheme of things because it's very easy to provide a filename override on the command line.
As for the symbol names.... hm.... personally I'm in the "igButton() feels kinda ugly, somehow?" camp, but I can't argue with the "there's less characters to type" aspect. Although in C++ land I'm pretty used to typing "ImGui::Button()", which is even longer - I'm not sure how prevalent it is for people to include the ImGui namespace since if you do that you end up with just Button() which is both confusing and very likely to clash with something.
That said, the idea of a --custom-imgui-namespace-prefix flag has some appeal - then if people want cimgui-style names they can get them with --custom-imgui-namespace-prefix ig (which would internally just effectively rename the ImGui_ prefix to ig after flattening).
I'm going to leave this issue to sit for a week or to so that if anyone else has feedback they can chime in (and I'll ponder it in the meantime), but if nothing else comes up I think I'll try renaming the default output files and adding an optional namespace prefix modifier as described.

That said, the idea of a --custom-imgui-namespace-prefix flag has some appeal - then if people want cimgui-style names they can get them with --custom-imgui-namespace-prefix ig (which would internally just effectively rename the ImGui_ prefix to ig after flattening).

this is a good idea.

even just typing the latter now I had to think about it - I'm not sure why by dc comes much more naturally to me than dbc

prob cuz of DC comics :) anyway, changing filename is good change.

So I've gone ahead and implemented this - the default output name is now dcimgui.h, and there are now switches to control the prefixes... to quote the new readme file:
By default, Dear Bindings will use namespaces/class names to generate prefixes for names in the output - for example, ImGui::Begin() will become ImGui_Begin().
However, you may want to customise these, especially if you are looking for compatibility with cimgui or similar. To achieve this, you can specify  --custom-namespace-prefix to replace ImGui_ with something else - for example, --custom-namespace-prefix ig will result in ImGui::Begin() becoming igBegin() instead. For more detailed customisation, you can use --replace-prefix, which allows any arbitrary name prefix to be replaced - for example,  --replace-prefix ImFont_=if will result in ImFont_FindGlyph() becoming ifFontGlyph() (and all other ImFont_ names following suit). You can specify as many --replace-prefix arguments as you like (although the results are undefined if you specify overlapping prefixes).
(for reference, --custom-namespace-prefix Foo is just a simplified syntax for --replace-prefix ImGui_=Foo)

Awesome stuff @ShironekoBen :)
Slight related, for language bindings it might also be useful to have no prefix at all (don't know if this is currently possible), for example look at this Zig code which uses Zig's 'translate-c' feature on the traditional cimgui API. It would be nice if the ig prefix could be dropped, so that it looks like ig.SetNextWindowPos() (to be fully 'Zig idiomatic' it should be ig.setNextWindowPos(), but that's probably too much to ask ;)
    ig.igSetNextWindowPos(.{ .x = 10, .y = 10 }, ig.ImGuiCond_Once, .{ .x = 0, .y = 0 });
    ig.igSetNextWindowSize(.{ .x = 400, .y = 100 }, ig.ImGuiCond_Once);
    _ = ig.igBegin("Hello Dear ImGui!", 0, ig.ImGuiWindowFlags_None);
    _ = ig.igColorEdit3("Background", &state.pass_action.colors[0].clear_value.r, ig.ImGuiColorEditFlags_None);
    ig.igEnd();

Hmm, an interesting idea. I can't say I've tested it extensively, but --replace-prefix ImGui_= seems to have the desired effect there... although given that in virtually all cases these symbols are going to get exported as C symbols from a library at some point, I'm not sure how much chaos having likely-to-collide things like Begin defined is going to be?
It feels like for most use-cases doing that sort of renaming in the higher-level language layer is likely to be more useful, but if anyone does want it then the option is there!

Btw, I just discovered that building the C bindings without prefix produces a collision with some Win32 function names (GetWindowPos, SetCursorPos, GetCursorPos) - so probably not a good idea to go without a prefix in the C bindings :)
I think it's better to deal with this problem in language-specific bindings which then call down into the C bindings (e.g. if I want a Zig bindings naming convention like ig.setNextWindowPos(...), then this should be handled by a separate Zig module (which can also be generated from the JSON metadata).


Improve model laziness

This code in the AbstractModelLoader triggers a lot of completions:
            if (parameter.isSequenced() &&
                    // FIXME: store info in Sequenced
                    typeFactory.isNonemptyIterableType(parameter.getType())) {

We should implement the FIXME, with proper backwards-compat if the new attribute is missing.

We could store it as a boolean flag in the Parameter itself, couldn't we?

We could store it as a boolean flag in the Parameter itself, couldn't we?

Fixed.


Default to the current session window

If tmux session and window are blank, then the current session and current window are chosen. These commits make minimal changes to attain this functionality.

If I understand correctly, this is useful if you're running vim in tmux and have ipython running in a different pane of the same session, correct ? This sounds like a good idea.
I have a minor nitpick were I would prefer if can keep the current default behavior, see my comment.

That's exactly what I'm doing, running vim in one pane and ipython in another. Yes, I see the value in naming the default session and window. That way one will not accidentally send a set of commands to their current pane. Do you want me to rebase this branch, keeping the defaults the same?

Yes, if you can just add a commit (or rebase, as you like) that preserve the defaults, I'll be happy to merge the PR.

I've updated the PR!

Cool, thank you for your contribution !


Install does not work on Mac (capitan)

I installed both with curl & with git clone, and the command nvm does not work for me. After including . ~/.nvm/nvm.sh in my .bashrc.
Is there something different about the new mac os ?

Is it a fresh install? I assume you're using the stock bash. Do you have a ~/.bash_profile file? If that's present, I think OS X doesn't source ~/.bashrc.

@realisation Also, please try . $NVM_DIR/nvm.sh on the command line, and confirm that nvm does at least work properly when sourced!

please try . $NVM_DIR/nvm.sh on the command line, and confirm that nvm does at least work properly when sourced!

@ljharb That does indeed work. However, I did have the ~/.bash_profile but it didn't include the script in there when I curled it.

OK - this is just an issue with the install script then. To fix for your case, manually add these two lines in the last section of https://github.com/creationix/nvm#manual-install

@ljharb re: Do you have a ~/.bash_profile file? If that's present, I think OS X doesn't source ~/.bashrc.
That worked! Thank you!!
Steps to fix:

open ~/.bashrc and cut the lines that add nvm.sh to the $PATH
create the file bash_profile: touch ~/.bash_profile
open that file in your favorite editor, and paste in the cut lines:

export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" # This loads nvm

restart terminal

What is the final recommendation for this:

i have no .bashrc and in .bash_profile I have

export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" # This loads nvm

but this is not enough to work :-(

@simonh1000 nvm alias default node if you want a version auto-used on shell startup. By default, you always have to nvm use something every time you open a new shell.

@ljharb thansk so much - I must have done that months ago on the old machine and forgotten about it

This appears resolved?