syscall: can't call Windows function that returns float

What version of Go are you using (go version)?

$ go version
go version go1.13.8 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/rich/Library/Caches/go-build"
GOENV="/Users/rich/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY="jaxf-github.fanatics.corp/"
GONOSUMDB="jaxf-github.fanatics.corp/"
GOOS="darwin"
GOPATH="/Users/rich/go"
GOPRIVATE="jaxf-github.fanatics.corp/"
GOPROXY="direct"
GOROOT="/usr/local/Cellar/go/1.13.8/libexec"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.8/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/s9/9s3fk2mx5f936dkcp66ltwsc0000gn/T/go-build181499683=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

This is essentially a request for https://github.com/golang/go/issues/3588 to be solved rather than just closed out and ignored.
Currently, it is not possible to retrieve a floating point value returned by a syscall. As noted in the original report, that's not very common in the old Windows APIs, but it does occur in many of the newer APIs, especially around graphics. My particular case is the Direct2D and DirectWrite APIs, although I've seen the need elsewhere as well.
I've looked, and it seems to be fairly straight-forward to add this support via a new set of functions (SyscallFloat, perhaps) -- however, it currently seems impossible to do so without doing it inside the runtime and syscall packages, which would require a fork, since there is various internal state that an outside package cannot access.
If we can't have the additional call(s) created for some reason, can we instead at least expose enough of the internals that an external module could provide the functionality?
I'd be happy to create the necessary PR to make either scenario work.
One thing to note: I realize I could get around this by using cgo. However, as most would likely agree, using cgo on the Windows platform is a nightmare. It's generally quite tolerable on platforms like macOS and linux, where the compiler toolchain is easily installed, but Windows makes that difficult and I've found getting downstream developers to be able to get their Windows machine setup properly for cgo to require a lot of hand-holding, unlike with other platforms, essentially making it a non-starter.

It would help if somebody suggested specific new API that would work for all known cases.  It seems to me that just adding syscall.SyscallFloat may not be enough.  What about syscall.(*Proc).Call and syscall.(*LazyProc).Call?

Good point. I had not yet encountered a case where I needed this in my own projects, but that should be considered as well. I’ll try to come up with a list and post it here.

Are the APIs you reference unimplemented in x/sys/windows?
See also discussion in #6510.
cc @alexbrainman @zx2c4 @mattn

Looking through both package syscall and golang.org/x/sys/windows, it appears these are the APIs that need a "Float" variant for the return:
syscall.Syscall -> syscall.SyscallFloat
syscall.Syscall6 -> syscall.SyscallFloat6
syscall.Syscall9 -> syscall.SyscallFloat9
syscall.Syscall12 -> syscall.SyscallFloat12
syscall.Syscall15 -> syscall.SyscallFloat15
syscall.Syscall18 -> syscall.SyscallFloat18
syscall.(*Proc).Call -> syscall.(*Proc).CallFloat
syscall.(*LazyProc).Call -> syscall.(*LazyProc).CallFloat

windows.(*Proc).Call -> windows.(*Proc).CallFloat
windows.(*LazyProc).Call -> windows.(*LazyProc).CallFloat

Regarding @networkimprov 's question: golang.org/x/sys/windows doesn't implement the syscall code itself -- it just calls through to the implementation in syscall.

Another possibility, which would require only a single code change in function runtime·asmstdcall in runtime/sys_windows_amd64.s and no visible API changes: make use of the currently-unused r2 result by always copying the X0 register into it on return, much like is currently done for the first 4 potential floating point arguments. This value is used on 32-bit systems, but I don't think the float problem exists there and no change is needed in runtime/sys_windows_386.s.

Do Windows function that accept floating point arguments work correctly?
Is there a need to fix this in the syscall package, which is more or less frozen, or can we just fix it in golang.org/x/sys/windows?

In my usage of them, yes, floating point args do work since the fix to fill the first four registers was put in. I found I do have to pass them by calling the conversion function in the math package, but that’s workable.
As stated earlier, the fix has to be in the runtime package, as that’s where the info is and the plumbing is private to that package so can’t be fixed externally.

My particular case is the Direct2D and DirectWrite APIs, although I've seen the need elsewhere as well.

Which API are you talking about?

Another possibility, which would require only a single code change in function runtime·asmstdcall in runtime/sys_windows_amd64.s and no visible API changes: make use of the currently-unused r2 result by always copying the X0 register into it on return, much like is currently done for the first 4 potential floating point arguments.

That sounds reasonable. Is X0 register documented by Microsoft to receive returned value on amd64?

This value is used on 32-bit systems, but I don't think the float problem exists there and no change is needed in runtime/sys_windows_386.s.

Fair enough.
Alex

My particular case is the Direct2D and DirectWrite APIs, although I've seen the need elsewhere as well.

Which API are you talking about?

As stated above, both the Direct2D and DirectWrite APIs are the ones I'm currently using. Did you want a specific function call? If so, here's one off the top of my head: https://docs.microsoft.com/en-us/windows/win32/api/d2d1/nf-d2d1-id2d1brush-getopacity
I'm not sure it matters, though -- this is basic functionality that should be possible, even if official Microsoft APIs didn't have FLOAT return args.

Another possibility, which would require only a single code change in function runtime·asmstdcall in runtime/sys_windows_amd64.s and no visible API changes: make use of the currently-unused r2 result by always copying the X0 register into it on return, much like is currently done for the first 4 potential floating point arguments.

That sounds reasonable. Is X0 register documented by Microsoft to receive returned value on amd64?

Yes. From here: https://docs.microsoft.com/en-us/cpp/build/x64-calling-convention?view=vs-2019:
A scalar return value that can fit into 64 bits is returned through RAX; this includes __m64 types. Non-scalar types including floats, doubles, and vector types such as __m128, __m128i, __m128d are returned in XMM0. The state of unused bits in the value returned in RAX or XMM0 is undefined.

... Did you want a specific function call?

I did. I was not aware of its existence.

If so, here's one off the top of my head: https://docs.microsoft.com/en-us/windows/win32/api/d2d1/nf-d2d1-id2d1brush-getopacity

Sounds good. Thank you.

I'm not sure it matters, though -- this is basic functionality that should be possible, even if official Microsoft APIs didn't have FLOAT return args.

No argument.

... From here: https://docs.microsoft.com/en-us/cpp/build/x64-calling-convention?view=vs-2019:
A scalar return value that can fit into 64 bits is returned through RAX; this includes __m64 types. Non-scalar types including floats, doubles, and vector types such as __m128, __m128i, __m128d are returned in XMM0. The state of unused bits in the value returned in RAX or XMM0 is undefined.

Looks good. Thank you.
Alex


Adding .Net SDK support for setting Azure AD administrator on Managed Instance

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: SanjaMalesevic sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Can one of the admins verify this patch?

/azp run net - mgmt - ci

This changes for .NET client are generated using managedInstanceAdministrators.json from PR  https://github.com/Azure/azure-rest-api-specs/pull/5095.

Hello @SanjaMalesevic , please provide the corresponding Spec review PR, and also please follow these instructions when you generate .Net SDK. Thanks.

Hi, I've just added comment below with link to corresponding PR on azure-rest-api-specs branch.

Hi @SanjaMalesevic , there are some test failures. Please fix them then assign the PR back to me. Thanks

Hi @SanjaMalesevic , there are some test failures. Please fix them then assign the PR back to me. Thanks

I made some changes in creating vnet, new route is added there. So, I need to record all test that create vnet and add new session records. Because of that build has failed. Execution takes lot of time, so I couldn't finish today and I will be out of office next week. It will be solved for 10 days.


lint:0

coverage :85
comment:functionality for formAcl

Please first create authorization in keyclaok and then raise this MR. closing it for now.
Dont


Nanoc live is extremely slow

Steps to reproduce

Run bundle exec nanoc live
Modify a file in the content directory
Save it

Expected behavior
Nanoc quickly rebuilds the corresponding file and serves it
Actual behavior
Nanoc takes over 10 seconds to rebuild the file.
Details
Note that compiling the whole site (with an empty tmp and output) dir takes 6 seconds.

Note that it's possible that I'm doing something stupid, but because of #1449 it's very hard to know what's going on!

Was not using the right gem (guard-nanoc instead of nanoc-live). See #1450


Add github action to add auto reply whenever a new pr is opened

Proposal
(A clear and concise description of what the proposal is.)
Message like -  " We got your pr !! Please kindly wait for team to review it " or any other message as per the mentor's wish
would be sent to the contributors when they open a pr

I would like to work on this issue


Introduce uni01alpha scenario for 17.1 adoption

Adds a new scenario for 17.1 adotpion of the uni01alpha deployment topology.
Depends-On: https://github.com/openstack-k8s-operators/data-plane-adoption/pull/706

recheck

The depends on seems to point to itself?


minimumCharactersToTrigger

Hi
I try to set the "minimumCharactersToTrigger" to zero, to have all the suggestion when keyboard appear, doesn't work.
(by the way, nice work!)

Here is how I worked around this issue:
- (void) textFieldDidBeginEditing:(UITextField *)textField
{
    // Force displaying all suggestions when beginning edition of text field
    id<UITextFieldDelegate> autocompleteInputView = (id<UITextFieldDelegate>)textField.inputAccessoryView;
    [autocompleteInputView textField:textField shouldChangeCharactersInRange:NSMakeRange(0, 0) replacementString:@""];
}


Add icon and minimum Kubernetes version to CSV

The icon and minKubeVersion are required to pass some linting checks.
/assign @thejasn @alebedev87

/lgtm

/label docs-approved
/label px-approved
/label qe-approved

/lgtm


fix: Alignment Issue with Sign Up/Sign In Page Close Button

Fixes #64
Description:
I have resolved the alignment issue affecting the close button on the sign-up and sign-in pages. Correcting this alignment is crucial to maintain a seamless user experience.
Preview:
Desktop:

Mobile:

I appreciate your contribution @LakshanRukantha !! It was really helpful !! Please star this repo for us 🫂


Provide some fanfare if the image you provide is perfectly centered

Currently, if you're image is perfectly centered, it will display "The center is at 50% - 50%
You can visual center this image if you" and obviously provides no instructions of how much to adjust it.
So, the text should be replaced with "Congrats! Your image is perfectly centered!" It will make the user feel a bit more like a hero after they've repeatedly adjusted and uploaded an image to test :)

Yeeeey! Thanks @stowball, already added that to the app 💪


[bug]Data loss found during data writing

environment

序号
组件名称
组件版本

1
canal
canal.deployer-1.1.4

2
mysql
5.6.40-log MySQL Community Server (GPL)

3
rocketmq
V4_2_0_SNAPSHOT

4
system
3.10.0-862.14.4.el7.x86_64

question
For the convenience of description, the abnormal topic is named A. The normal topic is named b。
Use the same data source, consumed by two canal instances into two topics，At a certain point in time, operating the database produced 23 records. In the rocketmq control interface, you can see that A has received 3 messages. B 23 messages received.

A is the topic just created
B is has been running for a while

What consumption model is used? It may be related to the setting of the initial consume offset. In addition, it is recommended to upgrade to the latest version of rocketmq.


WIn32 build error: pFilename is not a member of sequence::FilesystemEntry

Parser.cpp.patch.txt

Thx for reporting this.


admin feature permissions: update with ability for admin-ccs-data-controller to find users by email…

Fixup for change made in https://github.com/alphagov/digitalmarketplace-admin-frontend/pull/535 for https://trello.com/c/dBEA3HaN

Ugh. Unless I start the release of the feature immediately after merging this.

Yeah just added the tag rather than giving me a juggling act to do.


Update technical-reference.md

Removing the change added

@spunukol : Thanks for your contribution! The author, @MarkusVi, has been notified to review your proposed change.

#sign-off


NEUBURG-7: Add flow type checking

Added flow to every file.
Added types for every warning flow returned for those files.

In order not to have @flow in every file, we can add
[options]  all = true
to the .flowconfig and exclude the ones we don't want to have checked, like node_modules, dist, etc.
Concerning your second comment:
What I did so far is: Add @flow to a file, run flow and fix everything that popped up. I think everything that is not annotated yet is automatically set by flow, but I'm not sure. If we really want to have the maximum number of annotations, I think the plugin you suggested is a good option.

@NKlug I would suggest, we do both the .flowconfig and the eslint plugin, since nobody said something against it. Unless you are against it ofc :D

If you start a project from scratch with flow in mind you can set all to true :) :+1: for that.

Let's do the eslint-plugin-flowtype in a separate task though.

I'm currently trying to set all=true, but I'm facing some issues:
Flow does check every file, even those without annotation, including the whole 'node_modules' folder, which is quite large and this process causes my laptop to crash :(
So what I tried is ignoring the node_modules folder in the .flowconfig, but then I get some flow errors because it can't find specific modules (because it's ignoring them). I haven't found a workaround for this yet and it seems like others are also struggling with the same issue.
At the moment I suggest setting removing the all=true again and writing the @flow tag in every file.
In the eslint-plugin we could then enable forcing said tag so we won't forget it anywhere.
@Schedulaar @maxammann What's your opinion on this and do you by chance know a solution to the problem?

Also see this and this.
I decided to remove all=true again.


Which one is the correct license, MIT or GPL-3?

The LICENSE file mentions the MIT license but the pom.xml file mentions the GPL-3 license.
Which one is correct?

Made a pull request to switch the pom.xml over to MIT as this seems to be the desired license.


Add padding bottom to the grid

This is to ensure the spacing between the ol and the "is there something
wrong with this page" link or tagged content is always at 60px.
cc/ @alextea
Before

After

:+1:


Transfer Form. Improve mosaic amount input

Display mosaic balance in the amount text box
Display an error message when the transfer amount exceeds the balance

User Story
As a user that wants to send mosaics,
I want to be warned when I want to send more mosaics than I currently hold,
so I can make adjustments when needed.
Acceptance Criteria
AC1. We will show a non-intrusive validation alert when the amount typed in the box, is higher than the current available balance.
AC2. We will not block the action of attempting to send, and will wait to receive a REST error if the user wants to send a higher amount. This is to avoid issues with source of truth.
This task comes from an Out of scope from #1530.

The only problem I've found is that currently we are blocking "Send" button when the amount is too big which is not correct according AC2 (https://github.com/symbol/desktop-wallet/issues/1562#issuecomment-905336857)

Currently validation is still on wallet side: https://share.getcloudapp.com/jku4GXyk

There is also small issue in offline txs (it does not allow to create offline tx):


Extension doesn't work V 2.2.10

Starting on extension version 2.2.10 it no longer works in bitbucket 4.11.1
It worked fine on 2.2.8

I merged a PR which made this error. I'll fix this soon.

should be fixed with v2.2.11

Thank you, it works! :+1:


[WIP]Try to fix conda env

Issue #, if available:
Description of changes:

Get rid of removal and always create a new environment.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1638-7 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1638/7/index.html

Job PR-1638-20 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1638/20/index.html

Conda create doesn't make a fresh environment. We should keep remove to have a fresh environment for reproducible builds:
$ conda create -n tmp -y
$ conda activate tmp
$ pip freeze | grep numpy
numpy @ file:///opt/concourse/worker/volumes/live/a1b5ea96-b05c-40ba-5dfb-4542ece1c186/volume/numpy_and_numpy_base_1603491228242/work

$ pip install -U numpy
Requirement already satisfied: numpy in ./anaconda3/lib/python3.7/site-packages (1.19.2)
Collecting numpy
  Using cached numpy-1.21.5-cp37-cp37m-macosx_10_9_x86_64.whl (16.9 MB)
Installing collected packages: numpy
  Attempting uninstall: numpy
    Found existing installation: numpy 1.19.2
    Uninstalling numpy-1.19.2:
      Successfully uninstalled numpy-1.19.2
Successfully installed numpy-1.21.5

$ pip freeze | grep numpy
numpy==1.21.5

$ conda deactivate
$ conda create -n tmp -y
$ conda activate tmp
$ pip freeze | grep numpy
numpy==1.21.5


[Feature] TailsOS User Guide

Issue originally created by @jdjnvjdnajns
This issue was created on gitlab and then migrated here. Only the original post was migrated, not the comments. Please take a look at the discussions on the original Gitlab issue before commenting here: https://repo.getmonero.org/monero-project/monero-site/-/issues/987

Feature: TailsOS User Guide
Why?: TailsOS is a popular OS among privacy enthusiasts and users with high-threat models.
However, the current guides to get Monero on TailsOS are hard to find and not easy for the not-so-technical.
Hopefully, this would make it easier for TailsOS users to utilize Monero.
Related:

https://web.getmonero.org/resources/user-guides/cli_wallet_daemon_isolation_qubes_whonix.html

#1155
Would be good to also add @tobtoht's guide 'Setting up a Monero full node on Tails'. See the discussion on reddit. I'll contact them asking the progresses on this, if they are busy, i will adapt it and upload it.


Question regarding running predictions on new data

Hi,
I am trying to test and the nnUNet_baseline on some local data. When I run the test.sh in the nnUNet_baseline folder. After your suggestion with the git lfs pull I am able to run it comfortably.
However I have following 2 questions

In the Dockerfile
ENTRYPOINT python -m process $0 $@ What does the $0 and $@ does ? I tried to find an answer but was not clear to me.

I would like to see how the basemodel performs on my data. Do I need to just copy my data in appropriate folder in the autoPET/test/input folder and then run the autopet_baseline docker container?

Please advice
Thanks a lot.

This just hands over the script name ($0) and all parameters ($@) to the Python script. In our case this is not needed.
You can either replace the test/input samples, but the code runs through only one sample per call. You would either need to create your own loop around or alternatively just use the normal nnUNet framework for running the inferences, please refer to the documentation here: https://github.com/MIC-DKFZ/nnUNet/blob/master/documentation/dataset_format_inference.md

Thank you. When I am done working with it. I can create a pull request, if you want.

Thanks


OSC Channel Number Calculator off by one

Describe the bug
The OSC Channel Number Calculator at the OSC Plugin configuration appears to be off by one.
To Reproduce
Go to Input/Output.
Choose OSC Plugin.
Input /qlc/test to the Channel Number Calculator. You get 46808 as the Channel number.
Create a Widget e.g. a Button in the Virtual Console.
Edit the properties and Auto detect External Input.
Send a test value to /qlc/test and you get 46809.
E.g. with this python script
from pythonosc import udp_client

client = udp_client.SimpleUDPClient('<IP_ADDRESS>', 7700)

client.send_message("/qlc/test", 0.5)

This happens to all OSC paths.
Expected behavior
Same Channel Numbers.
Screenshots

Desktop (please complete the following information):

QLC+ Version 4.13.1

I think there is a misunderstanding here.
What you see in the input profile editor is channel + 1 but what is written in the profile is channel.
Check for example this
https://github.com/mcallegari/qlcplus/blob/master/resources/inputprofiles/TouchOSC-Automat5.qxi#L62
is presented as this

So the calculator si correct, since it outputs what you actually need to write in an input profile or your custom OSC node.
Closing.

Thanks for clarifying, @mcallegari

I think there is a misunderstanding here.

True. And sorry for that.

What you see in the input profile editor is channel + 1 but what is written in the profile is channel. Check for example this https://github.com/mcallegari/qlcplus/blob/master/resources/inputprofiles/TouchOSC-Automat5.qxi#L62 is presented as this

Alright. Could we place a hint somewhere near the calculator? Tooltip, text, some kind of question mark icon, or the like?
I might not be the only one wondering that there is a difference and having a hard time remembering if and where to add 1 (or subtract?) when setting up the VC with channels calculated by the calculator.

Documentation updated
https://docs.qlcplus.org/v4/plugins/osc


1.21 support?

please i beg of you

Until the official ReplayMod has 1.21 support, I cannot do this.


Add c8 for coverage

It's the most lightweight solution I'm aware of.
Happy to adapt the npm scripts to your liking.

I typically default to nyc but I dunno how well that plays along with mocha.


oneoff build for ucb-bar folks

Build the version of libdwarf being used by firesim/firesim to ease the transition to conda-forge deps.

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

We can close this PR.  Definitely don't want to merge it to main.


Problems with running as a User that does not have admin rights

This tool looks great.  When running as an Admin user, it worked well.  I then tried to run it as a limited user and it could not uninstall the bloat for that user.  It would just error out at each attempt to uninstall an item.  I assume if I ran it under the limited user but with admin rights, it would just perform it's actions against the admin user instead of the limited user.
Use case: This PC is used by multiple people.  The Windows bloat repeats for each limited user so it seems like we'll need to run bloatynosy for each limited user.

Appx packages are only removed for the currently logged-in user. That was the case with Bloatynosy and behaves similarly with its successor, Winpilot. You could give it a try and report back.


Support integer isotope values

Many formats (SMI/InChI/SDF/Mol2/CML/HIN/XYZ[non-standard extention]) only support the storage of integer isotopes. This is a reminder to add support for storing these isotopes in these formats.

This is a reminder to add support for storing these isotopes in these formats.

We could also add a warning so that we don't silently drop non-integer masses. At the same time, we will drop other data (velocities, properties), so I don't know if warning every time would be worth it =/

At the same time, we will drop other data (velocities, properties), so I don't know if warning every time would be worth it =/

Well, we could warn once - the first time it happens for a given Frame. The argument against this would be that the user should know which formats have which features and should 'get what's coming to them'. This would only happen if the data being dropped was non-default of course.


HOTT-4444: Re-add 8080 ingress

Pull Request
What?
I have:

Added 8080 ingress to ECS

Why?
I am doing this because:

I missed adding it in the prior PR

It would be good to plumb the infrastructure smoketests for these runs. I'll go and add a ticket for this


Bug: URI class - working with segments is really strange

Describe the bug
I think there is a problem in the way that some methods are handling segments right now.
Here is an example:
$uri = new \CodeIgniter\HTTP\URI('http://something.com/hello/there');

// will return ['hello', 'there']
d($uri->getSegments());
// will return 'hello' - ok
d($uri->getSegment(1));
// will return 'there' - ok
d($uri->getSegment(2));
// will return '' - why? it's not there
d($uri->getSegment(3));
// will throw an exception - ok
// not really easy to work with but this is a different story
d($uri->getSegment(4));

Here we go with a really strange part:
$uri = new \CodeIgniter\HTTP\URI('http://something.com/hello/there');

// will return '' - why? it's not there
d($uri->getSegment(3));
// will NOT throw an exception (even if segment 3 is not set)
d($uri->setSegment(4, 'honey'));
// will return '' - that's surprising considering no error when
// we set it a line before
d($uri->getSegment(4));
// will return 'honey'
d($uri->getSegment(3));

Actually I don't think it's very complicated to fix, but the main problem are BC changes. People can already rely on the empty string that is returned in some cases. So the question is how should it be handled.
Also working with getSegment() and throwing an exception when segment is not there is not very easy to work with. So it would be good to have something to surpass this without the need of catching the exception every time. One idea was proposed here: #2952
CodeIgniter 4 version
All versions
Affected module(s)
URI class
Expected behavior
$uri = new \CodeIgniter\HTTP\URI('http://something.com/hello/there');

// should throw an exception
d($uri->getSegment(3));

$uri = new \CodeIgniter\HTTP\URI('http://something.com/hello/there');

// should throw an exception
d($uri->getSegment(3));
// should throw an exception
d($uri->setSegment(4, 'honey'));
// should throw an exception
d($uri->getSegment(4));

If you want to get the exception use the request Class
$request->getSegment(3)
This works for me.
The uri should not throw exceptions by itself, but i see it return empty string, maybe it should return NULL because there aren't any

@crustamet there is no $request->getSegment() method. You probably think about $request->uri->getSegment() - this method exists and is an alias to the same class I'm referring in this issue.

Oh yes i was talking about that, and now i see that come empty when should throw exception.

I don't know what is happening really with this uri, for me it works good.

So i really don't know exactly what is happening with these segments.
For now i manage to work with them and have no problems.


only normalize http(s) urls

This is a proposed solution for #1080. We'll now restrict our host lower-casing to only schemes we've noted to be "normalizable".

@lslebodn had an interest in tackling this. I think we should have let them submit their change. There isn't any urgency in this.

@sigmavirus24, I believe @lslebodn's primary concern had been seeing this fixed for Debian as he mentioned in the Requests issue.
That said, there definitely isn't a rush on this. If you would like to pick this up @lslebodn, I'll gladly close this out. I'd (possibly mistakenly) inferred your question yesterday to be asking if the work was done yet. The turn around on a release of this will take a while to reach Requests.

I am very glad you provided a fix I would find a time next week :-)
Thank you very much @nateprewitt.

All yours @lslebodn :)

I'm still a little unclear on whether there is other intent to fix #1080, so I'm reopening this PR since we're now a month on.
This should be ready to go (after a test restart), unless you've already started work on this @lslebodn.

I am confused. I thought it's already fixed. Why do we need to reooen it?

Sorry @lsleboldn, @Sigmavirus24 was under the impression you'd wanted to submit the PR, so we closed this one waiting for you to open one.
I think this was just some miscommunication of intentions. The work is done and ready to go here, we just need @Lukasa to push the button. Then hopefully it will make it into 2.14.0.

Want to rebase this onto the current master branch?

Codecov Report

Merging #1089 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #1089   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          21      21
  Lines        1925    1926    +1
======================================
+ Hits         1925    1926    +1

Impacted Files
Coverage Δ

urllib3/util/url.py
100% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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

Should be set after tests finish :)


After setting cilium ds resources such as request and limit - its failing on new nodes

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I'm trying to set requests and limits for cilium ds in eks cluster.
Once the resources are set, pods are working fine.
But when the scaling of nodes is happening and on the new nodes we are facing below error from cilium
level=error msg=“error while opening/creating BPF maps” error=“Unable to create map /sys/fs/bpf/tc/globals/cilium_nodeport_neigh4: cannot allocate memory” subsys=daemon
level=fatal msg=“Error while creating daemon” error=“error while opening/creating BPF maps: Unable to create map /sys/fs/bpf/tc/globals/cilium_nodeport_neigh4: cannot allocate memory” subsys=daemon

Cilium Version
1.12.1
Kernel Version
5.15.0-1019-aws
Kubernetes Version
1.22.6
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

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

What are the limits you are setting? Have you considered bumping them? I'd expect to see that error when the limits are set too low.

The resources such as requests and limits were set as below
"resources.requests.cpu" = "300m"
"resources.requests.memory" = "600Mi"
"resources.limits.cpu" = "600m"
"resources.limits.memory" = "1G"
We increased the limit 50% more than this, even though the avg utlization was far below this threshold and yet we faced the errors.

We don't recommend setting CPU or memory limits at all. See https://twitter.com/thockin/status/1134193838841401345.
Requests are recommended and we don't recommend CPU request below 1 core and below 1G of memory.

As far as I can tell, this is not a bug in Cilium as the errors you're facing are directly resulting from the resource configuration you are applying.


added deployment resources support

Fixes #
Jetstream dispatcher handles messages delivery for NatsJetStreamChannel kind, for broker nad channel scenarios. Jetstream controller programmatically creates dispatcher deployment and it has it's drawbacks. One of them is limited control over deployment configuration, especially when it is created by broker controller, which reads only NatsJetStreamChannel spec part.
Proposed Changes

🎁 Add new feature: support k8s api to manage programmatical deployments of jetstream dispatcher.

Release Note
support k8s api to manage programmatical deployments of jetstream dispatcher.

@pierDipi , @aliok , it is ready for initial review of the idea behind this PR. Is it legal approach at all? Maybe you can advice based on your experience on how this is solved for other channel types. Thank you

/assign @pierDipi

cc @dan-j

/cherrypick main


docs(readme): add pf3/pf4 compatibility css

fixes https://github.com/patternfly/patternfly-next/issues/1567
@dgutride per our discussion, I added a blurb to the README for this. I think it's inevitable this will become a stylesheet, and I imagine anyone using PF3 and PF4 together will run into this issue and benefit from this CSS.
I was chatting with @seanforyou23 about it and he mentioned that this is really similar to https://github.com/patternfly/patternfly-next/wiki/IE11-Support#step-2-add-ie11-compatible-styling-overrides and we could host these in a common place. Just wanted to start the discussion about whether the README is the best spot for these, or if we want to explore other options.

Deploy preview for pf-next ready!
Built with commit 09d83f46e8545b0d76b0cd1acb8afcc1f3024309
https://deploy-preview-1568--pf-next.netlify.com

FWIW, I'm seeing this in UHC, too, since they're using PF3/PF4 together. And like I mentioned when we met about this, there are issues popping up from UHC not using our global reset styles, too. Note the white bullets beside the dropdown menu items.

If we offer styles to correct these types of issues, I think since problems like these are likely going to keep popping up and will likely be an issue with lots of apps, having them copy/paste or grab styles from somewhere manually likely isn't the best approach. I'm more in favor of having a stylesheet that folks using PF3/PF4 together and aren't using the global reset can include in their project to automatically get these kinds of updates. To that point, we could also move towards asking products to include the global stylesheet by default and make it a requirement for using PF4, and then we would only need to provide overrides for PF3/PF4 compatibility.
cc @janwright73

+1 to all of this, I strongly prefer these types of styling patches be centralized in a single place that's included as part of the core distribution so it can be consumed in an automated fashion. Placing it in a README I think will not be good for us or our consumers, it forces a manual workflow for each update.
If we're going to do this for PF3/PF4 compatibility styles, it would make sense to go ahead and decide where the IE11 compatibility styles should live also. Seems like they might be found along side one another in a compat directory or something of the sort.
Anything I can do to help?

Looks like just needs a rebase? Also, you can group this along side the ie11 stylesheet by adjusting this path https://github.com/patternfly/patternfly-next/blob/master/build/npm-scripts/ie-convert-all.js#L7

@dgutride do you know if this has continued to be an issue? I haven't come across a product needing support for this since the PR was opened. Curious if we should close this PR for now and revisit if it comes up again?


Improve versioning by setting the version in a single place and automatically increasing the build number

(And of course, thank you for the changes, which improve the current status a lot.)

Thank you very much for your review!

Also, regarding having to depend on a server to bump the build number, could you please extend the commit message, why a server has to be used and a the git commit hash (git describe --tags --dirty) is not enough. If a server is needed, could you please elaborate and document who maintains https://softether.network.

Done.

We actually do the same for Mumble, however we make use of all parts of the version there.
What do you think about doing the same for this project as soon as we bump the version?

We actually do the same for Mumble, however we make use of all parts of the version there.

Do you mean https://github.com/mumble-voip/mumble? Wow, you are quite active. Thank you for your work.

What do you think about doing the same for this project as soon as we bump the version?

I’d favor it, but as I am only a lurker and not a developer please don’t give too much weight to my opinion, and do what you think is best.

Do you mean https://github.com/mumble-voip/mumble? Wow, you are quite active. Thank you for your work.

Yep, that's the one. When I'm not working on this project I'm usually working on Mumble.

I’d favor it, but as I am only a lurker and not a developer please don’t give too much weight to my opinion, and do what you think is best.

No worries, your opinion is important even if you're not a developer.


fix(client): improve connection closing behavior

This PR resolves some problems that occurred when closing a DTLS client connection.

Codecov Report
Base: 12.63% // Head: 12.19% // Decreases project coverage by -0.43% :warning:

Coverage data is based on head (fe54685) compared to base (42a8693).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
- Coverage   12.63%   12.19%   -0.44%
==========================================
  Files           8        8
  Lines         285      287       +2
==========================================
- Hits           36       35       -1
- Misses        249      252       +3

Impacted Files
Coverage Δ

lib/src/dtls_client.dart
10.05% <0.00%> (-0.11%)
:arrow_down:

lib/src/lib.dart
78.94% <0.00%> (-5.27%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


error http://datables.net/tn/7

After generate a module addNew a item for module will get error http://datables.net/tn/7
Default module still get this error

Please send details of error from network. Also the laravel.log file

This is my logfile
laravel.txt

You need to fix this.
Regards,
[2016-11-16 02:24:20] local.ERROR: exception 'PDOException' with message 'SQLSTATE[42S22]: Column not found: 1054 Unknown column 'id' in 'field list'' in D:\Laravel\la1\vendor\laravel\framework\src\Illuminate\Database\Connection.php:333

Column not found: 1054 Unknown column 'id' in 'field list' (SQL: select `id`, `wor`, `mean` from `words` where `deleted_at` is null order by `id` asc limit 10 offset 0)'

you can change GET to POST method, for fix this problem

If your module has a lot of fields,  it also has this issue. Reduce display list in controller then can fixed this issue

Hi, this is a nice package but I think that the developers have stopped working on it. Maybe you can try this package: https://github.com/Muhammadinaam/speed-admin. It supports Laravel 8+.


Can't build on android ndk r22

Dobby version latest master (Version: Dobby-20210102-cf50347)
ANDROID NDK: android-ndk-r22-darwin-x86_64
Compiler: Clang
.../bin/_deps/dobby-src/source/UserMode/ExecMemory/clear-cache-tool-all.cc:47:20: error: write to reserved register 'R7'   __asm __volatile("svc 0x0" : "=r"(start_reg) : "r"(syscall_nr), "r"(start_reg), "r"(end_reg), "r"(flags));                    ^ 1 error generated.
I've managed to compile my project with ndk r19, but not with r22. As this change
https://reviews.llvm.org/D76848 seems to break functionality

Read carefully about what efriedma says in the link you posted. He says

if you use -fomit-frame-pointer, it should compile successfully

Read carefully about what efriedma says in the link you posted. He says

if you use -fomit-frame-pointer, it should compile successfully

fixed


Support testing in toolbox

Add Makefile target for

Setting up from inside toolbox
Testing in toolbox (checks if exists + sets it up if necessary)

Script that checks if toolbox exists, otherwise sets it up
Playbook with necesary targets

Signed-off-by: Matej Focko<EMAIL_ADDRESS>

I'm not sure I understand what the intended utility for these targets is... Why not just use one of the existing images to create a toolbox? How is check-inside-toolbox different from check_in_container? (besides using toolbox vs podman/docker)

@csomh It seems to me it's more swift compared to running in container where I've encountered rebuilds of image even throughout the day
(also it was more straightforward to set up local mirror for toolbox rather than usercont/base that managed to keep both images from registry and local and continued to use local ones -__-)

Why not just use one of the existing images to create a toolbox?

I don't think this will be easy. The toolbox images do not have easy build instructions.

How is check-inside-toolbox different from check_in_container?

For users that don't use toolbox, there is probably only a small benefit (maybe easier access to the test environment).
Otherwise, it's more a question of "Why use toolbox at all?." For me, it's really handy, especially because I am using Silveblue and have all the development tools (GUI included) inside toolbox.

I don't think this will be easy. The toolbox images do not have easy build instructions.

Strange, I thought something like toolbox create --container packit-service --image usercont/packit-service:prod is enough. Trying it out seems to be working and it seems to me that it's the right environment. What am I missing?

I don't think this will be easy. The toolbox images do not have easy build instructions.

Strange, I thought something like toolbox create --container packit-service --image usercont/packit-service:prod is enough. Trying it out seems to be working and it seems to me that it's the right environment. What am I missing?

pytest and flexmock from files/recipe-tests.yaml (you probably have them installed in $HOME through pip? iirc there were some problems with pytest from pip instead of RPM, maybe they're resolved)


Connecting to Azure Storage Bolb

I want to make a data connector which will connect to Azure Blob storage account bring in the data make some transformation on the data and provide to my client in a client understandable manner.
Can this be done ? If so how ? I am not able to find any examples doing the same.
Can I directly make use of AzureStorage.Blobs and proceed further ?

You can connect to anything.  Just follow a basic sample like Trip Pin and replace all of the Web API access calls with blob storage access methods.
imo blob storage isn't a good target, because it's not structured data.  But you must understand that already.
There's already built-in support in Power BI for accessing table storage data, which is a much more natural application.

There's not currently any way to make use of the blob connector from within your own connector. We hope to change that in the next half-year or so. Meanwhile, the trickiest part of the blob storage API is probably signing the request. You can see some sample code at https://gist.github.com/CurtHagenlocher/23a692099186da26d322181b3742ba39 which does something like that.

@CurtHagenlocher Has there been made any progress on this matter? I would also like to make usage of the Azure Blob Storage connector including the AzureStorage.Blobs function within another custom connector. It would take me a lot of time to rebuild the entire logic via Web.Contents just seems to be an authentication issue?

The blob storage account I would like to connect is made public and works without authentication using the Azure Blob Storage Connector.
Thanks,
Patrick


Used torchtext to refactor relation prediction model - much cleaner!

Refactored the code for Ferhan's relation prediction model
Used torchtext
Code looks much cleaner

still waiting on a CR for this.

@tuzhucheng - can you pls do a CR on this?


Upgrade to latest apollo packages

This upgrades from react-apollo@2 to @apollo scoped packages.

Please merge this pr finally I need it :)


Updated keras-applications version requirement

Summary
This is about as simple as a PR can get- The keras-applications requirement needs to be incremented to support the new models brought in with #11203 .
Related Issues
#11203
PR Overview

[n] This PR requires new unit tests [y/n] (make sure tests are included)
[n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[n] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR.
This update process is in progress. Due to outstanding breaking changes in Keras-Applications, we won't be merely upgrading the version number.
The new models will be available in the next release of Keras.


Active Directory Users can't login with Kerberos

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
When connected to Microsoft Active Directory the Kerberos Users cant login to Portainer.
Only Microsoft Active Directory Users who still use NTLM can login to Portainer.
With the Release of Windows Server 2025 NTLMv1 and NTLMv1 will no longer be available.
Expected Behavior
All users shown in Settings > Authentication are able to Login into Portainer
Actual Behavior
Only user who still use NTLM are able to Login into Portainer.
Steps to Reproduce

Connect Microsoft Active Directory to Portainer in Settings > Authentication
Configure your Users / User Sources in Settings > Authentication
Check if Users are Loaded.
Try to Login with an Kerberos-Only User - It will not work
Try to Login with an NTLM User - It will work

Portainer logs or screenshots
No response
Portainer version
2.21.2
Portainer Edition
Business Edition (BE/EE) with 5NF / 3NF license
Platform and Version
Docker 27.0.1
OS and Architecture
Ubuntu 24.04.1 LTS, AMD64
Browser
No response
What command did you use to deploy Portainer?
docker run -d -p 9443:9443 --name portainer --restart=always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer-ee:2.21.2 --http-disabled

Additional Information
No response

I am on same situation. Is there any chance that Kerberos will be implemented on near future (next updates)?

I am on same situation


Add Cloudflare Workers example

Let me know if this is right, not sure about the isOslo part but everything else should be fine!

Hello @FTCHD
Thank you for the valuable contribution, we are going to review and test it soon


ProxyExchange discards all incoming request headers when making a request with a body.

Affects: spring-cloud-gateway-webflux:3.1.1

Setup
Define an MVC RestController that accepts a ProxyExchange as an argument to a request handler method, and use the ProxyExchange object to issue a POST request (or any request that contains a body) to an upstream service, using a body created (or manipulated) in the request handler method.
Expected Behaviour
The ProxyExchange forwards the original request headers, minus filtered sensitive headers, and Content related headers in the upstream request.
Observed Behaviour
The ProxyExchange discards all original incoming headers, appending only the X-Forwarded-For and X-Forwarded-Proto headers. This behaviour is in contrast to the behaviour observed when using a ProxyExchange object to make requests to upstream services  that do not contain bodies (GET, OPTIONS, etc).

Any update on this?

Could you share your controller please?


Allowing GraphiQL to handle rendering when there is no query passed in

This will allow a pure GET request to the GraphQL endpoint and return back the GraphiQL interface without any queries defined.

Thanks! Will modify this to add some instructions and update the GraphiQL version also


⚠ RPC server - Figment Solana has degraded performance

In 9fa6311, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 216 ms

Resolved: RPC server - Figment Solana performance has improved in da0df95.


Ensure collapsed expandable is visible when overflow is overridden

Fixes sidebar that was hidden from the change in https://github.com/cfpb/design-system/pull/406
Changes

Ensure collapsed expandable is visible when overflow is overridden

Testing

Sidebar in e.g. https://cfpb.github.io/design-system/components/expandables should be visible.

🙇


whenever the plugin is active, shows save dialogue everytime even if nothing is edited

Would it be possible to make this go away. it becomes annoying when I haven't even edited anything.
godot4.2.1 mono

This is fixed as of Release v1.6.0


Writify bug

Describe bug
when i add a note to a song the only way to view the note is by right clicking and clicking add note. it doesn't show up next to the song
Steps to replicate
add writify to spicetify. right click on a song in a playlist. note does not appear.
Necessary screenshots
https://imgur.com/a/zVQNTQY
https://imgur.com/a/b1hjXI9
Spicetify and Spotify Version
0.8.9
please email me at<EMAIL_ADDRESS>if you need more info.

Yeah I still need to fix that but I haven't really been coding much over the summer

Yeah I still need to fix that but I haven't really been coding much over the summer

there's no rush it's just this is a thing i've wanted in spotify for a long while and was bummed that it was bugged

Yup, sorry about that. Issues is that when I coded this extension I coded it to work with one specific theme (Comfy's) grid layout and it didn't work for any other themes

Yup, sorry about that. Issues is that when I coded this extension I coded it to work with one specific theme (Comfy's) grid layout and it didn't work for any other themes

are you planning on patching this soon?

Yup, sorry about that. Issues is that when I coded this extension I coded it to work with one specific theme (Comfy's) grid layout and it didn't work for any other themes

I installed Writify and downloaded that theme but the notes column was not showing. I then moved writify.js to Local/spicetify/Extensions and changed the adress to the globals.d.ts file since it was looking for it in the Roaming/spicetify-cli but my file was in Local/spicetify. I also uncommented the TODO part from line 25 and got the notes column to show up but empty, without any notes. My notes do get saved but not displayed in the column. Is there something I can do to get it to work for the Comfy theme again?


How to make it work with python3.8

I am using python3.8.2 and it is continuously giving me syntax errors. How can I make it work with python3.8? I am using flake8.

Hey! I don't understand your question. PEP8Speaks (flake8 as linter) should work on any Python version.
PEP8Speaks does not have its own linter. Are there any unexpected errors which flake8 does not raise locally for you?

Thanks for your reply! It gives me syntax error when I use walrus operator.
Is there any way I can specify the the python version in pep8speaks config
file?
On Fri, May 22, 2020, 6:15 AM Himanshu Mishra<EMAIL_ADDRESS>wrote:

Hey! I don't understand your question. PEP8Speaks (flake8 as linter)
should work on any Python version.
PEP8Speaks does not have its own linter. Are there any unexpected errors
which flake8 does not raise locally for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OrkoHunter/pep8speaks/issues/173#issuecomment-632417524,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKKLEUACB6WL23BLWUGMDYLRSXDKLANCNFSM4NHGCKXQ
.

Oh interesting! Looks like I will have to update the flake8 and pycodestyle versions that I am using. May I ask you to create a PR? :)

@OrkoHunter Hey, this issue still persists for me. Please take a look at this PR.

@OrkoHunter Hi, I get the already mentioned problem with the walrus operator in this PR

@vhirtham I wonder if you use pycodestyle instead of flake8 as your linter, the issue goes away? We might need to update some versions of the linters, so that newer versions of Python are supported well.

Same problem here: https://github.com/StingraySoftware/stingray/pull/630


389-ds-base: fix homepage url

Motivation for this change
Fix homepage url. Related to #30636.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] 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.

There is also http://www.port389.org/ which is the URL given in the package README

@c0bw3b Thanks a lot. Fixed

Rebased to master in 436c8fd28739b20ab3f4448c249c528ba3267034. Thanks!


Question : Client Credentials to authorize a controller on Identity Server

Is it possible to authorize  an api controller , that is implemented on identity server app , to a particular client  by using client credentials grant type?
For example on identity server i have a controller such as;
[Route("/users")]
    public class UserController : ApiControllerBase
    {
        [HttpGet]
        public async Task<IActionResult> GetAllUsers()
        {
            return Ok();
        }
    }

and i have a client  with following config;
  {
      "ClientId": "web.client",
      "ClientName": "web Client",
      "AllowedGrantTypes": [ "client_credentials" ],
      "ClientSecrets": [ { "Value": "CLIENT_SECRET_HERE" } ],
      "AllowedScopes": [ "idvsrapi"]
    }

I want  GetAllUsers  method to be authorized only to the client above . Is this possible without user login?

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use one of the our free or commercial support options
See here for more details.
Thanks!


Preview images wont show up.

When I load the folder, I get blank picture squares. The tags come through but the picture previews doesn't show.

I also encountered the same symptoms, but I discovered that the 'Settings > Maximum resolution of gallery thumbnails' value was set to 0. After correcting this, the thumbnails displayed correctly.

Please see the troubleshooting
Gradio limits directory of images to display and you have to whitelist them or use temporal thumbnails.


tests: update tests to use ardrive_io

Closed in favor of https://github.com/ardriveapp/ardrive-web/pull/678


Command rate-limiting

Feature rate-limiting for commands on a user/guild-based context.

Does the Watson library contain any user/guild classes for retrieving user metadata? I think we could check on every request whether or not a specific user has exceeded a rate limit during a specific timespan.

Yeah, I think we could implement this pretty easily. The data doesn't really need to be persistent so we just store it in memory. A map should work just fine for this but redis might also be worth a try.
A very basic implementation would look something like this:
In the RouteHandlerFactory, we add a map for caching command calls for each user as well as their guild.
In the createCommandHandler, which basically creates the function that is called once we match a certain command from a user message, we add an additional step that handles the rate-limiting.
I've created a new branch with an example implementation of how this could look like on a per guild/user basis: Commit


Bug in mega-story-dalle & tokenizer

Hi,
I am trying to train the mega-dalle version, but there is one bug in the dataloader, e.g. in flintstones_dataloader.py in line 160 there is:
tokens.append(self.tokenizer.encode(text.lower()))

while the TextTokenizer has no encode function. I changed it for tokenize and modified slightly as follows:
t = self.tokenizer.tokenize(text.lower())
if len(t) > 64:
    t = t[:64]
t = t[1:(len(t)-1)]
t += [0]*(64-len(t))
tokens.append(t)

Now there is no error, but I wonder wether this is the way it was supposed to be.

it also happens with Pororo data

Maybe it has a version difference with the author's local code.
I hope this gets fixed.
Training:   0%|                                        | 0/1273 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/tqdm/std.py", line 1195, in __iter__
    for obj in iterable:
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in __next__
    data = self._next_data()
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1376, in _next_data
    return self._process_data(data)
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1402, in _process_data
    data.reraise()
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/_utils.py", line 461, in reraise
    raise exception
AttributeError: Caught AttributeError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 302, in _worker_loop
    data = fetcher.fetch(index)
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/root/anaconda3/envs/ldm/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 49, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/my/storydalle/mega-story-dalle/pororo_dataloader.py", line 201, in __getitem__
    tokens = self.tokenizer.encode(caption.lower())
AttributeError: 'TextTokenizer' object has no attribute 'encode'


Improve handling of exponentiations

Codegen for something like a^b where both a and b are constants results in something like:
(E::from(a_u64)).exp(E::PositiveInteger::from(b_u64))

Ideally it should be something like:
E::from(a_u64.pow(b))

and eventually this should be further optimized to just be
E::from(8_u64)

where a = 2 and b = 3.
This will involve making changes to both IR and codegen where evaluating the exponent before generating code could be an optimization pass.
Originally proposed by @bobbinth in https://github.com/0xPolygonMiden/air-script/pull/136#discussion_r1107759302

Not that I have any knowledge of the current status nor any say in this, but constant folding sounds like a good idea in general, not just for exponentiation.

@jan-ferdinand Thanks for pointing it out. Agreed, constant folding is something we plan to do as part of adding some optimization passes to the IR in the future.

Superseded by #333.


PTVS messes up hanging indents (Convention PEP 8)

PEP 8 says:

The closing brace/bracket/parenthesis on multi-line constructs may either line up under the first non-whitespace character of the last line of list or it may be lined up under the first character of the line that starts the multi-line construct.

Example:
        records = loadData.fetch_json_data_from_file(loc_filelist,
                                                     continue_number,
                                                     config.is_stacked_json)

However, if I do anything with that line in Visual Studio 2017 with PTVS installed, the code gets reformatted to:
        records = loadData.fetch_json_data_from_file(loc_filelist,
                                           continue_number,
                                           config.is_stacked_json)

As a result, pylint complains:

Wrong continued indentation (add 9 spaces).
continue_number,
^        | (bad-continuation)

Is there any way to fix this?

Do you mean that:
a) renaming loadData or fetch_json_data_from_file led to lines 2 and 3 no longer lining up with the first one
b) some edit on line 1 actually changed the start column position for lines 2 and 3? I can't figure out how to do that.
Regardless, we should make Format Document operation fix up the indent when it is like in your second code snippet. Right now it doesn't do that.


WIP: Fix error handling for status updates on CJI

Reverted the revert pushed in 501c26e1c94f97a644de1ea147386b4938a290d6 and applied PR 462 changes on top

'Completed' status looking good in tests now:
status:
  completed: true
  conditions:
  - lastTransitionTime: "2021-10-11T16:18:39Z"
    status: "True"
    type: ReconciliationSuccessful
  - lastTransitionTime: "2021-10-11T16:18:39Z"
    status: "False"
    type: ReconciliationPartiallySuccessful
  - lastTransitionTime: "2021-10-11T16:18:39Z"
    status: "False"
    type: ReconciliationFailed
  - lastTransitionTime: "2021-10-11T16:28:03Z"
    message: All ClowdJob invocations complete
    status: "True"
    type: JobInvocationComplete
  jobMap:
    iqe-fd47f775-iqe-9dnewig: Complete


Are you considering a stand-alone offline version

Hey, man.
Are you considering a stand-alone offline version?
I think many people will need.
Thanks.

Nope, no offline version planned.


parachain-collator executable

hi there, I'm just trying to work with https://substrate.dev/cumulus-workshop/#/en/3-parachains/1-launch and I don't seem to have an executable for parachain-collator - do you know where I might find that?
Also, random question - I got the relay chains working and I’m not 100% clear on is what all these hashed blocks are if no transactions are happening in the network … are blocks generated just to encapsulate pings between nodes etc.?

I see a parachain-collator exectuable in https://github.com/substrate-developer-hub/substrate-parachain-template#parachain-nodes-collators - should I have that command from this other repo, or perhaps there is some way to build it within the cumulus workshop code?

ah, okay, I see - doh!  the parachain-collator is in that other repo we set up at the start of the tutorial - so I need:
$ ../substrate-parachain-template/target/release/parachain-collator

to run it


[AutoPR security/resource-manager] Adding paging declaration to Microsoft.Security/complianceResults

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5929

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4804


refactor(ast_tools): remove const params for Visit generator

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

#6928
#6927  👈
#6926
#6925
#6924
main

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

Join @overlookmotel and the rest of your teammates on  Graphite


[BUG] yb_autovacate_keep_slots

yb_autovacate
If not zero, bots will automatically leave to make room for human players, when they join the server.

yb_autovacate_keep_slots
How many slots autovacate feature should keep for human players
Default: "1", Min: "1", Max: "8"

This's my setting.   version: 4.5.1 the latest
yb_quota "16"
yb_quota_mode "fill"
yb_autovacate_keep_slots "6"
yb_autovacate "1"

My understanding is  that  yapb will always keep at least 16 players.
And when a real player join server, it will kick one bot automatically, until 6 slots all used.
After that, with more players joining, the total player will increase one by one, until fill the max limit.
But I found that when the num of  real player under 6(maybe), it seems works well.  When people more than 8 or 10, the total number of players is only 12 to 13!!!!  It almost kicked out all bots.
But the total num of players should be more than 18 for expect!!

I believe you're misunderstood this option, autovacate keeps free  slots regardless  player counts.
max players = 30, autovacate = 1, autovacate_keepslots = 5, means means bots can occupy only 25 slots, no matter how many players are playing or quota number.


Button color

Hi,
When a sweetalert opens up, the button (for example 'Ok') has the background-color of the hover. But when you hover over the button and then unhover, the button has the actual background-color it suppose to have (or at least the background-color is now lighter than before hovered).
So the background-color is different before first hover and when a button has been hovered.

Thank you @Tklaversma for reporting this, fixed in v4.0.12


Parsing fails with negative value in numerical questions

When using a numerical question, having a negative value is considered an error where it shouldn't be.
Here is an example:
import { parse } from "gift-pegjs";

const question = "What is the value of -1? {#-1}."

const quiz = parse(question)

Indeed! The grammar doesn't take into consideration signed numbers (how silly!).
I don't have a lot of bandwidth right now to work on this - it's an easy fix, but requires adding tests (such as your example that doesn't work).
The key fix is changing the lines at https://github.com/fuhrmanator/GIFT-grammar-PEG.js/blob/0cb57354f4d493f2a6609047efc1af68c5c2a1f9/GIFT.pegjs#L294-L301
with
Number
  = value:DecimalValue { return parseFloat(value.join('')); }

DecimalValue
  = IntegerPart ('.' FractionalPart)?

IntegerPart
  = Sign? Digits

FractionalPart
  = Digits

Sign
  = [+-]

Digits
  = [0-9]+

I created a PR for this issue.

Thanks @Sho-ki for your PR


Create dataset loader for idner-news-2k

Dataloader name: idner_news_2k/idner_news_2k.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?idner_news_2k

Dataset
idner_news_2k

Description
A dataset of Indonesian News for Named-Entity Recognition task. This dataset was previously provided by Syaifudin & Nurwidyantoro (2016) (https://github.com/yusufsyaifudin/Indonesia-ner). We manually re-annotated the dataset with a more standardized NER tags. We split this dataset into three files, namely train.txt, dev.txt, and test.txt. Each file consists of three columns which are Tokens, PoS Tag, and NER Tag respectively. The format is following CoNLL dataset which split each token into one line and each sentence is separated by one empty line. For the NER tag, we use the IOB format as illustrated in the example below. In terms of PoS tag, we tagged the data using UDPipe (http://ufal.mff.cuni.cz/udpipe), a pipeline for tokenization, tagging, lemmatization and dependency parsing whose model is trained on UD Treebanks.

Subsets
-

Languages
ind

Tasks
Named Entity Recognition

License
MIT (mit)

Homepage
https://github.com/khairunnisaor/idner-news-2k

HF URL
-

Paper URL
https://aclanthology.org/2020.aacl-srw.10/

#self-assign


Get client+server tests working

We had a start of this in #72, but that has aged.
Currently we have client side tests, but they don't actually test a round trip to the server.  At least some preliminary session test should be created.

Also clean up the construction of the client test HTML

Ideally, the build task for this file should use grunt-contrib-jade with as much as possible of the template data provided statically (possibly using underscore-style templates) in the grunt config.

from https://github.com/Kitware/minerva/pull/201#issuecomment-157399228

Minerva uses Girder's test infrastructure and it's working now.


Search is failing due to missing id attribute

$zdclient.tickets.search(query: "subject:ticket subject", reload: true).fetch
I, [2017-10-16T08:51:58.439901 #1]  INFO -- : get https://zenpayrollhelp.zendesk.com/api/v2/tickets/search?query=subject%ticket+subject&reload=true
D, [2017-10-16T08:51:58.440082 #1] DEBUG -- :   Accept: "application/json"
  Accept-Encoding: "gzip;q=1.0,deflate;q=0.6,identity;q=0.3"
  User-Agent: "ZendeskAPI API 1.9.5"
I, [2017-10-16T08:51:59.059882 #1]  INFO -- : Status 400
D, [2017-10-16T08:51:59.061785 #1] DEBUG -- :   server: "nginx"
  date: "Mon, 16 Oct 2017 15:51:59 GMT"
  content-type: "application/json; charset=UTF-8"
  content-length: "163"
  connection: "close"
  x-zendesk-api-version: "v2"
  x-zendesk-application-version: "v44.28"
  x-rate-limit: "700"
  x-rate-limit-remaining: "582"
  strict-transport-security: "max-age=31536000;"
  x-ua-compatible: "IE=Edge,chrome=1"
  cache-control: "no-cache, private"
  x-zendesk-origin-server: "[param]"
  x-request-id: "[param]"
  x-runtime: "[param]"
  x-rack-cache: "[param]"
  x-zendesk-request-id: "[param]"
{"error"=>{"title"=>"Invalid attribute", "message"=>"You passed an invalid value for the id attribute. Invalid parameter: id must be an integer"}}
[]

Looks like the search method is on the client not the tickets object, my bad...


Update slot claiming strategy

closes #750 :

Slot claiming is now done with the authorities found according to the newly imported block (thanks to peeking before importing)
Removal of redundant check in sidechain/consensus/aura/src/verifier.rs verify_author function that sidechain block and parentchain block header match.

Additional info: It passed when running it locally. This makes it quite hard to debug.

I think I have found another problem in the slot timing calculation that we have. PR #790 attempts to fix that

closing in favour of #811


HOTP tokens 'time out'

Reported by mhyzon on 24 Jun 2014 14:09 UTC
HOTP tokens disappear (at least on iOS) after 30 seconds like they are TOTP codes, There is no time constraint on HOTP codes.  Could we either have a tunable timeout or no timeout on HOTP code display?
If you aren't quick enough to enter the code in your client (like you are distracted, or something), then you 'lose' the code.  Do it enough, and sync with the server can be lost

Comment by npmccallum on 25 Jul 2014 21:28 UTC
The timeout is already tunable via the creation URI. I'll keep this bug open as a reminder to add the timeout parameter to the token creation UI.

I don't think this issue is worth addressing. The cost is additional UI complexity. I don't think there is a major problem.


setup.py imports firebase_admin which requires six

[REQUIRED] Step 2: Describe your environment

Operating System version: MacOS 10.13
Firebase SDK version: 2.9.0
Library version: 2.9.0
Firebase Product: any

[REQUIRED] Step 3: Describe the problem
If you don't already have python six installed, you can't install firebase
Steps to reproduce:
pip install .
Relevant Code:
setup.py has: import firebase_admin and firebase_admin.__init__.py has import six so if it isn't installed before installing the installation fails with:
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/rh/fs2krkgj1hz_2ftgxgl20d980000gn/T/pip-mpnc1wob-build/setup.py", line 24, in <module>
        import firebase_admin
      File "/private/var/folders/rh/fs2krkgj1hz_2ftgxgl20d980000gn/T/pip-mpnc1wob-build/firebase_admin/__init__.py", line 21, in <module>
        import six
    ModuleNotFoundError: No module named 'six'

Also required for that setup.py to install from source are google-auth and requests

Install from source is a low priority for us. I think it's fair to ask developers to run pip install -r requirements.txt before attempting that.

Fair enough, but at least this is here if someone else needs to install from the latest, a git tag, a fork ;), etc and this is just a dependency


is there any library for node.js or extension for building EIR ?

do i need to compile the whole thing and just insert it as a stringed buffer ? or there's is a way to build it dynamically using some functions with lead to the proper buffer ?
i'm asking as if there's no way for this, i may start implementing an easy way & see how can we integrate this in BLENO
sample code for the same thing on C
ble.gap().accumulateAdvertisingPayload(GapAdvertisingData::KEYBOARD);         ble.gap().accumulateAdvertisingPayload(GapAdvertisingData::COMPLETE_LOCAL_NAME,                 (const uint8_t *) DEVICE_NAME.data(), sizeof (DEVICE_NAME));         ble.gap().accumulateAdvertisingPayload(GapAdvertisingData::SHORTENED_LOCAL_NAME,                 (const uint8_t *) DEVICE_NAME.data(), sizeof (DEVICE_NAME));

do i need to compile the whole thing and just insert it as a stringed buffer ? or there's is a way to build it dynamically using some functions with lead to the proper buffer ?

You need to compose the whole buffer using the Node.js buffer API: https://nodejs.org/api/buffer.html
I've only scene advlib for decoding: https://www.npmjs.com/package/advlib


phred_offset failure, should we fix?

Background: I'm working again on processing the RISK files in qiita and figure out the last issue so we can process. As a reminder, the Broad sent us per sample files as the raw data. The issue is that by default skbio auto detects the phred_offset value (either 33 or 64) which normally works great but on these files it seems like the reads were generated using 33 but the header was generated using the 64 standard. Thus, it doesn't work and we need to preprocess by hand.
Now, the question is: do we expect this to happen often? If that's the case we need a fix. The fix is simple: add a new column in the qiita.preprocessed_sequence_illumina_params table so we can pass this value; add a new row in that column for per sample FASTQ where we make the phred_offset 32 and leave the other defauls; and finally changing to this new processing params the list of studies that have been processed with it: 1998 and 1939.
Note that we didn't add it cause it was never used before and this is a really special case.

I don't think this is something we should commonly expect to happen, not
sure what others think. At the very least I'm not aware of any other
study that has a similar setup.
On (Oct-26-15|10:27), Antonio Gonzalez wrote:

Background: I'm working again on processing the RISK files in qiita and figure out the last issue so we can process. As a reminder, the Broad sent us per sample files as the raw data. The issue is that by default skbio auto detects the phred_offset value (either 33 or 64) which normally works great but on these files it seems like the reads were generated using 33 but the header was generated using the 64 standard. Thus, it doesn't work and we need to preprocess by hand.
Now, the question is: do we expect this to happen often? If that's the case we need a fix. The fix is simple: add a new column in the qiita.preprocessed_sequence_illumina_params table so we can pass this value; add a new row in that column for per sample FASTQ where we make the phred_offset 32 and leave the other defauls; and finally changing to this new processing params the list of studies that have been processed with it: 1998 and 1939.
Note that we didn't add it cause it was never used before and this is a really special case.

Reply to this email directly or view it on GitHub:
https://github.com/biocore/qiita/issues/1512


Safe Search Provider selection not syncing

Describe the bug
Sync of Safe Search Providers does not work.
Toggle of SafeSearch at all is syncing.
To Reproduce
Steps to reproduce the behavior:
General Settings --> Use Safe Search --> disable one or more of the search providers
Expected behavior
Sync should also include the selection of the search providers

Implemented in https://github.com/bakito/adguardhome-sync/releases/tag/v0.5.0


Add "process" command

If this should skip previously used processors by default would I be correct in assuming that it will need to update the records with any additional processors that were used during the run?

I did try to reduce it to the one commit but I'm pretty sure I failed at that so apologies there.

Please try this:
http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

That is decidedly similar to what I did do, though since they were all the same file I dropped the older two. On my fork I do now only have that one commit but pushing here it doesn't appear to have removed the previous two, is that expected?

Hi! You don't want to be "dropping" the older commits, but instead you want to be squashing your commits into a single commit. By dropping commits you are losing the information in those commits. Looking at your fork, it appears that all of your commits are still there.
My advice would be do to the following:

git checkout the changes you have on your fork - these are likely on origin/next, but depending on your naming scheme this can be different.
Next, again going by your fork, you will want to do a git rebase HEAD~3. The simplest solution here would be to leave the first (as represented in when rebasing) commit as is, squash the second commit and the third commit into the first. For each squash you have the ability to update the commit message, so be sure to modify the commit message appropriately.
Check your work with git log and git show to be sure you still have all your changes
Finally, push your changes to your fork with git push --force origin next (again, assuming your fork is called 'origin'). Note that the force is required as you are re-writing history, and otherwise the push will fail.

I'm not entirely sure what has happened here so I think the easiest way to fix this might be to just backup your process.py file outside of your git repository, checkout the latest version of the next branch from WA and create your commit again.
So first thing to check, do you have the main WA repository set up as a remote? And what is the output if you run:  git remote -v?

I already took a manual backup of the process.py file as soon as things started going wrong.
I do believe I have the main WA set up as remote the output of the requested command is below.

matthew@linux-drzu:~/PycharmProjects/workload-automation> git remote -v
origin  https://github.com/Rahlubenru/workload-automation.git (fetch)
origin  https://github.com/Rahlubenru/workload-automation.git (push)
upstream        https://github.com/ARM-software/workload-automation.git (fetch)
upstream        https://github.com/ARM-software/workload-automation.git (push)

Ah ok that's good.
So in that case you should be able to do git reset --hard upstream/next which will remove any changes you've done locally and take your branch back to the upstream version.
After this you should be able to restore your file, create a new commit with your changes and then force push that back to your origin/next.

The reset has reset the commits, git log shows the commit history for the upstream now. It has not removed the file from my fork or my local folder though and so I can't recommit the file.
I tried using git checkout upstream/next and git fetch upstream but neither of these has aligned my files with the upstream.

Was the reset command with the --hard argument while you were on your origin/next branch?
git fetch upstream should have retrieved the latest version of upstream from github and your git checkout upstream/next would have taken you into a detached state so you will need to run a git checkout next to get back to your branch before continuing.
And just to check, when you're back on your next branch, what is the last commit that git log shows you?

I think I have my fork sorted now, I just ran git push origin next --force even though my local copy looks wrong and that worked. My local is still wrong though somehow.
After the checkout next it was my most recent commit but I did the reset again after and so the following is the last entry.

commit 72f2f8259480717c31d036047caf4380d1916121 (HEAD -> next, upstream/next, origin/next,
origin/HEAD)
Author: Sergei Trofimov<EMAIL_ADDRESS>Date:   Thu Mar 22 12:19:47 2018 +0000
fw/config: better error when merging augs

If one or more entries for augmentations in configuration contains an
invalid value, raise ConfigError with the entry name.

Ok, so you appear to have a clean branch that is up to date with the latest upstream version now.
In which case you should now be able to restore your process.py file and re-commit it?

Yes, I found the problem with my local copy so I'll get this redone shortly. Thank you for your patience.
it appears to have gone to #701 so I guess this one needs closing or is there a way I can get it to pull to this one?

It appears I closed it myself, not quite sure how.

Ok cool, no problem.
It may have closed itself when all the commits were removed and I'm not aware of a way to combine the PR's again so continuing on #701 is fine.


No speech output using PicoTTS with the "system" language

Thank you @relan for reporting this bug in fdroiddata merge request 5900. I've pasted your report and steps below.

I stumbled upon the following issue initially. I have a tablet with AOSP 8.1. Pico TTS has a language setting set to system, which is none of the supported ones. When I was trying to speak some English text aloud, it didn't work. Steps to reproduce:

Check the languages in the Pico TTS settings. Mine has only English, Spanish, Italian, German and French. Set to System.
Set the system language to any but mentioned above.
Start TTS Util and try to speak some text.
Nothing is spoken, "Speech in progress" notification appears, "Stop speaking" button does nothing. Expected result: an error message saying that I should choose a supported language in the Pico TTS settings.

I have reproduced this issue. It will be fixed in the next TTSUtil version. I should be releasing that sometime in the next few days.


fix: Conditionally compile base64 command for bb binary

resolves #1850
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Note: we don't test on Mac so CI will pass regardless


Update ingress.md

'spec' is just another field that any Kubernetes object (Ingress here) needs.

/approve


Question about wallhack colors

Hey thanks for the trainer !! in 1.5.1 i got more then one color ESP in 2.0 everything is red (scav boss players bots) i wonder if i do something wrong ? Thanks !

Thanks, and what is the file size of your EscapeFromTarkov_Data\outline file ? This is the shader we use for coloring players. Perhaps something altered the file. Mine is 13.6 KB (13,943 bytes).

13,6 kB (13.943 bytes) i got both version on my computer 1.5.1 and 2.0 never opened  1.5.1 after installing 2.0

i will delete both and reinstall 2.0 when i can and report if it still show red only thanks for the help

Yep, let's try that, first time I'm seeing this.


fix(core): type not matching to the function

Thanks for your contribution!
Released to NPM as<EMAIL_ADDRESS>


Debugger is not attached after several hangs in internal python module

Describe the bug
While I was testing Issue-6456 I encountered a problem with debugging but then helped me reinstalling of VS. @int19h asked me to open new issue when problem will be reproduced. One week it was working and now I cannot again debug any python code in VS19.
I'm using .pyd file created in c++ and trying to test it with pytest. Unfortunately, after some errors in my c++ package (hang processes etc.) I cannot debug python code independently of native code debugging settings. Debugger is not attached after select debug options on test. Looks like VS pass wrong arguments to testlauncher.py - always the same. It is reproducing for all python projects in VS19.
I created new python solution with one file and test. Output from Test's console when I clicked debug on test in Test Explorer:
C:\Python37\python.exe "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\PROFESSIONAL\COMMON7\IDE\EXTENSIONS\MICROSOFT\PYTHON\CORE\testlauncher.py" D:\DebuggingTest\DebuggingTest\DebuggingTest\ pytest "" 0 "" "" dummyfilename .\test_example.py::TestExample::test_debugging --junitxml=D:\DebuggingTest\DebuggingTest\TestResults\junitresults_09150ba4-3c0e-4bd1-a198-481922678cd2.xml --rootdir=D:\DebuggingTest\DebuggingTest\DebuggingTest -o junit_logging=all -o junit_family=xunit1
This pytest "" 0 "" "" dummyfilename doesn't change even after enabling/disabling native code debugging.
Expected behavior
Debugger is working
Sample Code

python solution created in VS19
inside one python file: test_example.py
content of that file:

class TestExample():
    def test_debugging(self):
        pass

Configuration information
VS Version: 16.9.2
PTVS version:

Python   16.9.21026.1

Python - Conda support    16.9.21026.1

Python - Django support   16.9.21026.1

Python - IronPython support   16.9.21026.1

Python` - Profiling support   16.9.21026.1

Python version:
Python 3.7.9

I downloaded source code and now I'm trying to debug problem in PTVS. It is difficult because symbols does not load for all project (especially for TestAdapter.Executor where is logic related to running tests).
I discovered where is problem. 'app' argument, passed to this method in ExecutorService is always null:
GetDebugSettings(VisualStudioProxy app, IRunContext runContext, PythonProjectSettings projectSettings, out PythonDebugMode debugMode, out string debugSecret, out int debugPort)
Then in the same method:
if (runContext.IsBeingDebugged && app != null) {
    debugMode = projectSettings.EnableNativeCodeDebugging ? PythonDebugMode.PythonAndNative : PythonDebugMode.PythonOnly;
}

And I know that 'app' is getting from here but it returns nulll:
_app = VisualStudioProxy.FromEnvironmentVariable(PythonConstants.PythonToolsProcessIdEnvironmentVariable);
Do you have any clues how to resolve this problem?

Do you have any update on this issue?

@int19h could you look and give some tips?

My apologies for a very late followup! I can't repro it locally on VS 2019 v16.11 + Python 3.7; if I create a simple project as described, set a breakpoint on def test_debugging, and do Debug Test, it gets hit. In case this was an issue with some earlier version of VS and/or PTVS, I would suggest trying the most recent first.
That aside, it looks like the environment variable isn't being inherited by the test runner process for some reason. When you change SetEnvironmentVariable to use global or user scope, it's actually persisted when set, so inheritance isn't needed for the other process to see it; but the default is process-scope. It should still be inherited by child processes, though.
What happens if you try to debug a simple app that just prints out the variable? e.g.:
import os
print(os.environ["_PTVS_PID"])

Hi, thank you for your answer. I am using the most recent VS. I can check env variable from python side but I checked it while debugging PTVS and it was null. Like I wrote earlier, it happens where projects that are being debugged hang. Only reinstall VS can fix the problem for a while. Is child process always created in the same way? Maybe here is some place to improvement. Could you point me where I should look for process creation? Then I could check env variable in main process and in child process after creation.

I've ran into this same fundamental issue while investigating https://github.com/microsoft/PTVS/issues/6796 - that issue has a detailed write-up of the problem, so I'm going to use it to track, and resolve this one as a dupe.


[zuul] Enable ansibletest and tobiko with stages

With this PR in the test-operator role [1] we introduced the concept of stages. By default only tempest stage is enabled. In order for us to continue to test tobiko and ansibletest CR we have to explicitly enable the testing via the cifmw_test_operator_stages parameter.
[1] https://github.com/openstack-k8s-operators/ci-framework/pull/2374

recheck


Add text searching manual

From @jgrzebyta on February 22, 2018 20:33
There is no manual how to create SPARQL text search query for both:

[ ] LuceneSearch
[ ] LuceneSpinSail

Copied from original issue: eclipse/rdf4j-doc#49

From @barthanssens on March 30, 2018 22:10
Do you mean something like http://docs.rdf4j.org/programming/#_full_text_search ? (actually more like an example, not a real manual)


vscode-extensions.eugleo.magic-racket: 0.5.7 -> 0.6.4

Description of changes
I also updated the default settings.
Changelog: here

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 168090 run on x86_64-linux 1

1 package built:

vscode-extensions.eugleo.magic-racket


refactor CLI code

long time no talk!
do you still need a review on this?

Hey @cvan, a review would be great! I'm especially interested in your thoughts on the architectural changes, and whether they'll make it easier (or harder) to maintain and enhance this code over time.
I've updated the branch to fix the conflict with master, so it's ready for your review!

@mykmelez awesome, sure thing!
do you have an idea what's wrong with the Taskcluster test failures on Windows? (I recall prior issues re: PR #69; same issue or different?)

do you have an idea what's wrong with the Taskcluster test failures on Windows? (I recall prior issues re: PR #69; same issue or different?)

I think this was a regression from https://bugzilla.mozilla.org/show_bug.cgi?id=1410564. I fixed it in https://github.com/mozilla/qbrt/commit/2b82e72747e1f9e361090c18a750f3cbb6a2ad2a, then resolved another Taskcluster failure on Windows, so the tests now pass there.


I added my project to readme. My first ever pull request!

This is my first ever pull request. I added my project version to the readme file of the coding challenge 59.

Great message!


Databases on version 0.65.0: missing destination name firewall_configuration in *[]snowflake.Database

Provider Version
+ provider registry.terraform.io/aidanmelen/snowsql v1.3.3
+ provider registry.terraform.io/hashicorp/aws v4.66.1
+ provider registry.terraform.io/hashicorp/random v3.5.1
+ provider registry.terraform.io/hashicorp/vault v3.15.2
+ provider registry.terraform.io/snowflake-labs/snowflake v0.65.0

Terraform Version
Terraform v1.4.6
on darwin_arm64

Describe the bug
When running plan, or import, terraform will fail to parse the output of SHOW DATABASES and error on the firewall_configuration key.
Expected behavior
Output of the SHOW command is handled and the plan is issued.
Existing databases are recognized.
Code samples and commands
This is an example, for us 11 on 12 did fail.
Running plan:
╷
│ Error: unable to scan row for SHOW DATABASES LIKE 'SNOWFLAKE' err = missing destination name firewall_configuration in *[]snowflake.Database
│
│   with snowflake_database.this_db["SNOWFLAKE"],
│   on main.tf line 1, in resource "snowflake_database" "this_db":
│    1: resource "snowflake_database" "this_db" {
│
╵

Where the result of the query is
[
  {
    "created_on": "2021-08-06 09:34:46.074 +00:00",
    "name": "SNOWFLAKE",
    "is_default": "N",
    "is_current": "N",
    "origin": "SNOWFLAKE.ACCOUNT_USAGE",
    "owner": "",
    "comment": "",
    "options": "",
    "retention_time": "0",
    "firewall_configuration": null,
    "share_events_with_provider": ""
  }
]

And the state:
# snowflake_database.this_db["SNOWFLAKE"]:
resource "snowflake_database" "this_db" {
    data_retention_time_in_days = 0
    id                          = "SNOWFLAKE"
    is_transient                = false
    name                        = "SNOWFLAKE"
}

Additional context

12 databases exists in our account
12 databases appear in state
On plan, 1 database managed to be planned (though it already exists), and the plan fails on the 11 others

Adding extra context, I am comparing the same database SNOWFLAKE between my personal snowflake account, and my workplace's:
field | PERSONAL | WORKPLACE
-- | -- | --
created_on | same | same
name | same | same
is_default | same | same
is_current | same | same
origin | same | same
owner | same | same
comment | same | same
options | same | same
retention_time | same | same
resource_group | <null> | MISSING
firewall_configuration | MISSING | <null>
share_events_with_provider | MISSING |

this is now solved by https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/1840


[CognitiveServiceMgmt] Fix location

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1532

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1363   +/-   ##
=======================================
  Coverage   54.82%   54.82%
=======================================
  Files        3012     3012
  Lines       75844    75844
=======================================
  Hits        41578    41578
  Misses      34266    34266

Continue to review full report at Codecov.

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

Includes https://github.com/Azure/azure-sdk-for-python/pull/1175

in https://github.com/Azure/azure-sdk-for-python/pull/1415


Capacitor vs. Cordova Version

Hello,
thank you for creating this awesome plugin! What are the advantages of using this Plugin instead of using the cordova version?
Kind Regards,
Marius

Usually Cordova plugin are a bit messy to use and config compare to capacitor native one.
Like you need ionic lib + Cordova one for reactivity. Here only capacitor.
But both are valid, plus this one will become officially supported by Revenue Cat soon

Thank you! Will the officially supported Capacitor plugin have the getProducts method?

i did added it today here so yes it will as that one have it too now :)

Thank you very much for your work! I really appreciate it!